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

Bug that causes null constraints #36

Open
maxkratz opened this issue Sep 2, 2022 · 1 comment
Open

Bug that causes null constraints #36

maxkratz opened this issue Sep 2, 2022 · 1 comment
Labels
bug Something isn't working still broken Issue/Bug is not resolved when adding this label

Comments

@maxkratz
Copy link
Member

maxkratz commented Sep 2, 2022

AND tests: There are constraints with null, symptom fixed by b36a8f9

ImportsTests are also affected by this. Symptoms now fixed for all three ILP solvers by 0e12046

@maxkratz maxkratz added the bug Something isn't working label Sep 7, 2022
@maxkratz
Copy link
Member Author

maxkratz commented Dec 15, 2022

Still not fixed. For an example, see test class GipsImportsSimpleMappingTest https://github.com/Echtzeitsysteme/gips-tests/blob/main/test.suite.gips/src/test/suite/gips/imports/GipslImportsSimpleMappingTest.java#L55-L62.

To reproduce the issue, one has to comment out the null checking line within the ILP solvers, e.g., https://github.com/Echtzeitsysteme/gips/blob/master/org.emoflon.gips.core/src/org/emoflon/gips/core/ilp/GlpkSolver.java#L331-L333.

@maxkratz maxkratz added the still broken Issue/Bug is not resolved when adding this label label Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working still broken Issue/Bug is not resolved when adding this label
Projects
None yet
Development

No branches or pull requests

1 participant