Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cannot reload sumocfg if SUMO_HOME is not set #14830

Closed
behrisch opened this issue May 7, 2024 · 6 comments
Closed

cannot reload sumocfg if SUMO_HOME is not set #14830

behrisch opened this issue May 7, 2024 · 6 comments
Assignees
Labels
a:sumo-gui bug regression Works in previous versions
Milestone

Comments

@behrisch
Copy link
Contributor

behrisch commented May 7, 2024

complains about invalid document structure

@behrisch behrisch added this to the 1.21.0 milestone May 7, 2024
@behrisch behrisch self-assigned this May 7, 2024
@namdre
Copy link
Contributor

namdre commented May 7, 2024

is this a regression?

@behrisch
Copy link
Contributor Author

Yes, it has been introduced when we restrained schema checking to local files by default in #11054

@namdre
Copy link
Contributor

namdre commented May 16, 2024

Why does the initial load work but not the reload?

@behrisch
Copy link
Contributor Author

behrisch commented May 17, 2024

It seems there is something broken in the validation initialization. The first pass does not do any validation (but only in the case without SUMO_HOME)

@namdre
Copy link
Contributor

namdre commented May 17, 2024

I would say this is a feature. Much better to load without validation then to not load at all. Giving a warning of validation not possible, please set SUMO_HOME is enough in my view. (And the same for reload).

@behrisch
Copy link
Contributor Author

You are right, the broken part is that it does validation in the second pass

@namdre namdre added the regression Works in previous versions label May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:sumo-gui bug regression Works in previous versions
Projects
None yet
Development

No branches or pull requests

2 participants