Skip to content

PALS support#607

Closed
roussel-ryan wants to merge 2 commits intodesy-ml:masterfrom
roussel-ryan:pals
Closed

PALS support#607
roussel-ryan wants to merge 2 commits intodesy-ml:masterfrom
roussel-ryan:pals

Conversation

@roussel-ryan
Copy link
Copy Markdown
Contributor

Description

Motivation and Context

  • I have raised an issue to propose this change (required for new features and bug fixes)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation (update in the documentation)

Checklist

  • I have updated the changelog accordingly (required).
  • My change requires a change to the documentation.
  • I have updated the tests accordingly (required for a bug fix or a new feature).
  • I have updated the documentation accordingly.
  • I have reformatted the code and checked that formatting passes (required).
  • I have have fixed all issues found by flake8 (required).
  • I have ensured that all pytest tests pass (required).
  • I have run pytest on a machine with a CUDA GPU and made sure all tests pass (required).
  • I have checked that the documentation builds (required).

Note: We are using a maximum length of 88 characters per line.

@cr-xu
Copy link
Copy Markdown
Member

cr-xu commented Dec 16, 2025

I believe Axel is working on the python PALS parser (https://github.com/pals-project/pals-python), are we planning to interface with that or just directly with PALS?
Right now it seems to directly convert to a nested dictionary and dump to json/yaml.

@jank324 jank324 changed the title Pals PALS support Feb 25, 2026
@Hespe
Copy link
Copy Markdown
Member

Hespe commented Apr 23, 2026

I would be in favor of building on the existing pals-python. Less code for us to maintain and more likely to be correct if used by more people.

@roussel-ryan
Copy link
Copy Markdown
Contributor Author

Sounds good to me, I can close the PR if we want

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.

3 participants