Skip to content

Conversation

@imbeauf
Copy link
Collaborator

@imbeauf imbeauf commented Nov 6, 2025

… onto develop

Draft PR for the moment, just to test if the AL manages to compile with these extensions
@prasad-sawantdesai
@olivhoenen


📚 Documentation preview 📚: https://imas-data-dictionary--172.org.readthedocs.build/en/172/

@github-actions
Copy link

github-actions bot commented Nov 6, 2025

@deepakmaroo
Copy link
Contributor

deepakmaroo commented Nov 14, 2025

This changes complaining AL-Java compilation issue code too large.
AL-Java code generation to be fix

…ade-off agreed with the author of the ticket)
@imbeauf
Copy link
Collaborator Author

imbeauf commented Nov 19, 2025

Does it work with this lighter version of the IDS ? I think this would satisfy 90% of the needs of the author of the ticket

@github-actions
Copy link

@deepakmaroo
Copy link
Contributor

I apologise, missed this updates and notification.
Tested gyrokinetics_local ids lighter version with AL-Java v5.5.0 and IMAS-Core v5.5.2. AL-Java compilation issue disappeared and it is working fine.
For reference, created AL-Java branch to validate gyrokinetics_local ids https://git.iter.org/projects/IMAS/repos/al-java/commits?until=IMAS-5682-test-gyrokinetics-validate

imbeauf and others added 2 commits December 3, 2025 09:38
Co-authored-by: Simon Pinches <SimonPinches@users.noreply.github.com>
Co-authored-by: Simon Pinches <SimonPinches@users.noreply.github.com>
@github-actions
Copy link

github-actions bot commented Dec 3, 2025

@github-actions
Copy link

github-actions bot commented Dec 3, 2025

@github-actions
Copy link

github-actions bot commented Dec 3, 2025

@github-actions
Copy link

@imbeauf
Copy link
Collaborator Author

imbeauf commented Dec 16, 2025

Dear reviewers, the technical issues are solved for this branch, can you please complete your review of this PR ? It's not a draft anymore. Thanks

Copy link
Collaborator

@SimonPinches SimonPinches left a comment

Choose a reason for hiding this comment

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

lgtm

Co-authored-by: Simon Pinches <SimonPinches@users.noreply.github.com>
@github-actions
Copy link

@imbeauf imbeauf requested a review from olivhoenen as a code owner December 17, 2025 07:38
@github-actions
Copy link

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.

8 participants