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

feat: DAIDE parser for any pattern and automatic validation #37

Merged
merged 14 commits into from
Nov 28, 2022

Conversation

mjspeck
Copy link
Collaborator

@mjspeck mjspeck commented Nov 22, 2022

The parser can now parse not just messages and arrangements, but any
DAIDE pattern.

Keyword objects are now automatically validated upon instantiation.

Closes: #36, #29

The parser can now parse not just messages and arrangements, but any
DAIDE pattern.

Keyword objects are now automatically validate upon instantiation

Closes: #36
@mjspeck mjspeck requested a review from byung-o November 22, 2022 21:51
@mjspeck mjspeck self-assigned this Nov 22, 2022
@mjspeck mjspeck marked this pull request as draft November 23, 2022 17:45
@mjspeck mjspeck marked this pull request as ready for review November 23, 2022 18:10
@mjspeck mjspeck marked this pull request as draft November 23, 2022 20:34
The issue was that there was no `visit_sub_arrangement` method, so that
was being treated as generic visit.
@mjspeck mjspeck marked this pull request as ready for review November 23, 2022 21:04
@mjspeck mjspeck linked an issue Nov 28, 2022 that may be closed by this pull request
src/daidepp/keywords.py Outdated Show resolved Hide resolved
src/daidepp/keywords.py Outdated Show resolved Hide resolved
src/daidepp/keywords.py Outdated Show resolved Hide resolved
src/daidepp/keywords.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@byung-o byung-o left a comment

Choose a reason for hiding this comment

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

looks good

@mjspeck mjspeck merged commit a0d016e into main Nov 28, 2022
@mjspeck mjspeck deleted the feat/#36-expand-parser branch November 28, 2022 16:17
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.

Make parser able to parse any DAIDE string Add Testing to PR
2 participants