web.load incompatibility with latest tidy 3.9
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Start by locating the web.load implementation and checking how it validates simulation data against tidy. Compare the listed validation errors with the tidy 3.9 and gplugins 3.8 versions identified in the issue. Done means data created before the upgrade loads successfully; the issue does not identify a test or prescribe a compatible change.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the bug
When upgrading to the latest tidy version (3.9), we find that web.load fails and gives the following error:
sim_data = web.load('<HASH>', verbose=False)
Error:
WARNING: updating Simulation from 2.9 to 2.8
WARNING: Could not execute validator
'_source_homogeneous_isotropic' because field 'structures' failed
validation.
WARNING: Could not execute validator 'plane_wave_boundaries'
because field 'structures' failed validation.
WARNING: Could not execute validator 'tfsf_boundaries' because
field 'structures' failed validation.
WARNING: Could not execute validator '_structures_not_close_pml'
because field 'structures' failed validation.
WARNING: Could not execute validator 'bloch_boundaries_diff_mnt'
because field 'structures' failed validation.
WARNING: Could not execute validator
'_warn_monitor_mediums_frequency_range' because field 'structures'
failed validation.
WARNING: Could not execute validator
'_projection_monitors_homogeneous' because field 'structures'
failed validation.
WARNING: Could not execute validator
'diffraction_and_directivity_monitor_medium' because field
'structures' failed validation.
WARNING: Could not execute validator '_warn_grid_size_too_small'
because field 'structures' failed validation.
WARNING: Could not execute validator
'_validate_num_lumped_elements' because field 'structures' failed
validation.
WARNING: Could not execute validator 'check_fixed_angle_components'
because field 'structures' failed validation.
WARNING: Could not execute validator 'data_monitors_match_sim'
because field 'simulation' failed validation.
WARNING: Could not execute validator 'validate_no_ambiguity'
because field 'simulation' failed validation.
ValidationError: 10 validation errors for SimulationData
simulation -> structures -> 0 -> priority
extra fields not permitted (type=value_error.extra)
simulation -> structures -> 1 -> priority
extra fields not permitted (type=value_error.extra)
simulation -> structures -> 2 -> priority
extra fields not permitted (type=value_error.extra)
simulation -> structures -> 3 -> priority
extra fields not permitted (type=value_error.extra)
simulation -> subpixel
value could not be parsed to a boolean (type=type_error.bool)
simulation -> subpixel -> pec -> PECConformal -> edge_singularity_correction
extra fields not permitted (type=value_error.extra)
simulation -> subpixel -> lossy_metal -> SurfaceImpedance -> edge_singularity_correction
extra fields not permitted (type=value_error.extra)
simulation -> subpixel -> pmc
extra fields not permitted (type=value_error.extra)
simulation -> precision
extra fields not permitted (type=value_error.extra)
simulation -> structure_priority_mode
extra fields not permitted (type=value_error.extra)
Expected behavior
We expect to successfully load the data post sim as it always did in the past. From my understanding, tidy is performing validations of the sim and somehow in the new version these validation checks have changed and are incompatible with our version (3.8) of web.load.
Suggested fix
It may require discussion with flexcompute.
Versions
- Lenguaje dominante
- Python
- Estrellas
- 62
- Forks
- 66
- Merge medio
- 1 d 12 h
- PR fusionados (30 d)
- 10
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de gdsfactory/gplugins
-
RCE via pickle.loads on poisoned modes cache in find_modes_waveguide and find_modes_cross_sectionAbiertobug
Dificultad 4/5 3-5 días Aptitud para principiantes 56/100
gdsfactory/gplugins#790 ·
Los mantenedores suelen responder en 1 día
-
s22 in write_sparameters_grating is a numerical floor, not the fibre-port reflection coefficientAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
gdsfactory/gplugins#777 ·
Los mantenedores suelen responder en 1 día
-
bib not properly renderingAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 48/100
gdsfactory/gplugins#749 ·
Los mantenedores suelen responder en 1 día
-
Pin sax with compatible release ~=0.18.2Posiblemente ocupada @jackgdsf la tomó hace 95 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 25/100
gdsfactory/gplugins#738 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 50/100
gdsfactory/gplugins#699 ·
Los mantenedores suelen responder en 1 día
Todos los issues de gdsfactory/gplugins
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
microsoft/agent-lightning#661 ·
Los mantenedores suelen responder en 2 días
-
area/profiles comp/cli P3 type/feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
NousResearch/hermes-agent#146372 ·
Los mantenedores suelen responder en 1 día
-
[BUG] OpenAI-compatible embedding model alias breaks embedder constructionPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
[oblt-aw][autodoc] Document obs-aw-automerge contract and fix automerge doc inconsistenciesPosiblemente ocupada @github-actions la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
GeoSeries scale and skew tests fail with pytest 9.2.0Posiblemente ocupada @jiayuasu la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día