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

A field removed from the rules is still tested #52

Closed
ClemensLinnhoff opened this issue Jan 2, 2024 · 0 comments · Fixed by #61
Closed

A field removed from the rules is still tested #52

ClemensLinnhoff opened this issue Jan 2, 2024 · 0 comments · Fixed by #61
Labels
bug Problems in the build system, build scripts, etc or faults in the interface.

Comments

@ClemensLinnhoff
Copy link
Contributor

Describe the bug

I also to remove a field from the rules, e.g. mounting_position_rmse from the sensor view, but I still get a warning:

SensorView.mounting_position_rmse.is_set(None) does not comply in SensorView

Describe how to reproduce the bug

  1. Run the example trace file and the warning will appear.
  2. Open rules/osi_sensorview.yml
  3. Delete the line "mounting_position_rmse:"
  4. Run the example trace file again. The warning will still appear.

Describe the expected behavior

I would expect the warning to not appear anymore, because there is no rule defining the mounting_position_rmse anymore. I am not sure, where this warning is coming from. Are the rules cached somewhere?

Describe the OS you are using

  • OS: Ubuntu 22.04
@ClemensLinnhoff ClemensLinnhoff added the bug Problems in the build system, build scripts, etc or faults in the interface. label Jan 2, 2024
@ClemensLinnhoff ClemensLinnhoff mentioned this issue Jan 2, 2024
20 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Problems in the build system, build scripts, etc or faults in the interface.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants