-
Notifications
You must be signed in to change notification settings - Fork 67
Enable OpenCRE as selectable standard in map-analysis #720
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
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Ashutosh singh <163310518+ashutosh-engineer@users.noreply.github.com>
|
Dear @ashutosh-engineer , with all due respect: based on that question I have to conclude that your understanding of the OpenCRE data structure fundementals is currently insufficient to make a reliable change. The essence of OpenCRE is that a standard links to CREs and because other standards also link to those CREs, they link to each other. In other words, OpenCRE forms an intrinsic part of the graph. I strongly advice not to go ahead and make changes before this understanding is solid. |
Thanks for the earlier clarification — I spent some time going through the codebase and the map-analysis logic to make sure I fully understand the model. My understanding now is:
Concretely, this means: If this understanding aligns with your expectations, please let me know if it’s okay for me to proceed with backend change following this approach. Thanks for the guidance, Sir |
|
Excellent, please proceed and feel free to abort this task if you feel uncomfortable with the complexity. |
|
@robvanderveer What was done:
How it was verified:
Happy to open the PR in this form or adjust if you’d prefer any changes first.
The limited set of standards was intentional and only meant to validate routing and traversal behavior locally. |
|
@robvanderveer The changes have been implemented and verified locally with minimal test data. I have committed the recent code in the PR, please let me know if you’d prefer this submitted as-is in its current form, or if you’d like any adjustments first. |
- Backend: Add read-only OpenCRE routing to existing graph traversal logic - Frontend: Remove hardcoded OpenCRE entry to prevent dropdown duplication - Make backend standards API the single source of truth for OpenCRE - No data, schema, or ontology changes - only exposes existing relationships
- Remove generated bundle.js file from version control - Add bundle.js and bundle.js.LICENSE.txt to .gitignore - These are build artifacts that should not be tracked
Signed-off-by: ashutosh-engineer <ashutoshsingh6376@gmail.com>




This PR enables OpenCRE to be selected on either the left or right side in the map-analysis view.
Fixes #469