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

Problem Graph #51

Merged
merged 92 commits into from
Sep 6, 2024
Merged

Problem Graph #51

merged 92 commits into from
Sep 6, 2024

Conversation

Elscrux
Copy link
Member

@Elscrux Elscrux commented Aug 20, 2024

Reworked solver configuration with graph based approach.

@Elscrux
Copy link
Member Author

Elscrux commented Aug 27, 2024

All solvers show "Solution by null" in the solution Data. (should be the solver name)
The Solver: entry in the Meta Data is always null (probably same issues as solution by null)

This seems to be a backend issue. I'll fix it on #78

This was referenced Aug 30, 2024
@koalamitice
Copy link
Member

@Elscrux the VRP problem mask nearly works now. There is still an issue that the Solution is not shown in the Solution Tab. Everything else seems to work fine.

Screenshot of the problem: (No Solution shown, even though problem was solved successfully)
Bildschirmfoto 2024-09-04 um 14 39 15

@Elscrux
Copy link
Member Author

Elscrux commented Sep 4, 2024

@Elscrux the VRP problem mask nearly works now. There is still an issue that the Solution is not shown in the Solution Tab. Everything else seems to work fine.

Screenshot of the problem: (No Solution shown, even though problem was solved successfully) Bildschirmfoto 2024-09-04 um 14 39 15

Might be a backend issue. Could you confirm with the debugger or so that the solution is properly set?

@koalamitice
Copy link
Member

@Elscrux the VRP problem mask nearly works now. There is still an issue that the Solution is not shown in the Solution Tab. Everything else seems to work fine.
Screenshot of the problem: (No Solution shown, even though problem was solved successfully) Bildschirmfoto 2024-09-04 um 14 39 15

Might be a backend issue. Could you confirm with the debugger or so that the solution is properly set?

Thanks for the input, was actually a backend issue. The clusterers did not set any solutionData. Fixed with: ProvideQ/toolbox-server#86

Copy link

sonarqubecloud bot commented Sep 6, 2024

Copy link
Member

@koalamitice koalamitice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested Changes locally, everything looks good now!

@koalamitice koalamitice merged commit f7c2f0d into develop Sep 6, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants