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

Parsing Error with SSW #639

Closed
MicahGale opened this issue Jan 21, 2025 · 2 comments · Fixed by #645
Closed

Parsing Error with SSW #639

MicahGale opened this issue Jan 21, 2025 · 2 comments · Fixed by #645
Labels
bugs A deviation from expected behavior that does not reach the level of being reportable as an "Error". parsers are hard Examples of where MCNP syntax is complicated and should be simplified.

Comments

@MicahGale
Copy link
Collaborator

Describe the bug

A parsing error is occurring when reading an SSW input.

To Reproduce

A short code snippet of what you have ran. Please change or remove any specific values or anything that can't be public. For example:

problem = montepy.read_input("foo.imcnp")

Error Message (if any)

If an error message was printed please include the entire stacktrace. If it includes any specific values please change or remove them. For example:

    >37304|  ssw cel= 3000  20000  21000  22000  23000  30000  31000
          |         ^ not expected here.
    There was an error parsing "=".
    sly: Syntax error at line 1, token==

Version

  • Version 0.5.3.post1
@MicahGale MicahGale added bugs A deviation from expected behavior that does not reach the level of being reportable as an "Error". parsers are hard Examples of where MCNP syntax is complicated and should be simplified. labels Jan 21, 2025
@MicahGale
Copy link
Collaborator Author

I think making param only data inputs should be added as parsable.

@MicahGale
Copy link
Collaborator Author

MicahGale commented Jan 23, 2025

This is documented in section 5.8.8 of the 6.3.1 manual. I think the issue is that cell isn't a keyword here.

We can add CEL, PTY, and SYM. Probably could also hit #636, #607 with this.

MicahGale added a commit that referenced this issue Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugs A deviation from expected behavior that does not reach the level of being reportable as an "Error". parsers are hard Examples of where MCNP syntax is complicated and should be simplified.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant