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

There are no rules able to parse the input #51

Closed
atorr018 opened this issue Mar 6, 2019 · 7 comments

Comments

@atorr018
Copy link

commented Mar 6, 2019

Hi there,

I am CS student, and I found an issue with the library you are developing. I try it with other .cnv files and is working fine, but this one from 2016 is having an issue. It trows the following exception

CNVError: 'There are no rules able to parse the input. File:

you think it would be fine to send you the file so you can check the issue? or is it something I am doing wrong?

@castelao

This comment has been minimized.

Copy link
Owner

commented Mar 6, 2019

Hi, yes please send me a sample of your failing files.

@atorr018

This comment has been minimized.

Copy link
Author

commented Mar 6, 2019

thanks, here you go:

data.zip

@PabloOtero

This comment has been minimized.

Copy link

commented Jun 5, 2019

Hi Castelão, any update about this issue? I'm experiencing the same problem.

@PabloOtero

This comment has been minimized.

Copy link

commented Jun 6, 2019

Hi again,

I'm not sure why my code in utils.py was indented in lines 98 to 101. By using your updated code in your repository everything works.

@castelao

This comment has been minimized.

Copy link
Owner

commented Jun 27, 2019

I'm sorry for the slow response. So is the latest commit here working for you?

@aer205

This comment has been minimized.

Copy link

commented Aug 13, 2019

Hello,
I'm having the same issue - do you have any advice?

castelao added a commit that referenced this issue Aug 22, 2019
Thanks to @atorr018 for reporting this bug.

Some corrupted files show empty lines which are now ignored, thus if the
rest of the file makes sense it is parsed as usual.
@castelao

This comment has been minimized.

Copy link
Owner

commented Aug 22, 2019

Thanks @atorr018 !

@PabloOtero , these lines should be out of the loop so that if no rule can satisfy, the function returns an error.

@PabloOtero and @aer205 , if the latest version doesn't work for you, I would need to see an example file to understand why the current rules can't parse your data. In that case, please open a new issue with the cnv text that fails. Thanks,

@castelao castelao closed this Aug 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.