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

Mutagen Parse Issue Generic Topic #120

Open
Noggog opened this issue Sep 19, 2024 · 0 comments
Open

Mutagen Parse Issue Generic Topic #120

Noggog opened this issue Sep 19, 2024 · 0 comments
Labels
engine A feature for the analyzer engine itself

Comments

@Noggog
Copy link
Member

Noggog commented Sep 19, 2024

Maybe define a topic that is raised if mutagen has a problem parsing a record, for any reason. This would be fulfilled by parsing the record in a non-lazy fashion, and catching any exceptions thrown.

This might be a prereq to calling any isolated record analyzers? Would prevent them from having exceptions thrown within themselves. Downside being analyzers that wouldn't throw on the fields they are interested in looking at wouldn't run at all.

Maybe the solution is that the user can specify (somehow) that they dont want this to be checked on record XYZ, and that would allow the individual analyzers to run anyway, and throw as needed. Opt in sorta thing.

@Noggog Noggog added the engine A feature for the analyzer engine itself label Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engine A feature for the analyzer engine itself
Projects
None yet
Development

No branches or pull requests

1 participant