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

upload error #80

Open
HansVissers opened this issue Jan 28, 2020 · 14 comments
Open

upload error #80

HansVissers opened this issue Jan 28, 2020 · 14 comments

Comments

@HansVissers
Copy link

Hello, The attached csv table creates an upload error. Thank you for looking into this, Hans

FDAtrain4PredRet.zip

1-28-2020 10-44-03 AM

@stanstrup
Copy link
Owner

Thanks for the report. The file appears to be correctly formatted.

Could you try making the pubchem column empty (but keeping the column)? The pull from pubchem doesn't work anymore. I don't remember if it uses that even if there is an inchi.

Could you also set the Method to "RPMMFDA" instead of "RPMM" so there is no confusion?

Neither of these things should make it break of course. But the system is not the most robust...

@HansVissers
Copy link
Author

HansVissers commented Jan 28, 2020 via email

@stanstrup
Copy link
Owner

stanstrup commented Jan 28, 2020

Yes you have selected that method but the file says "RPMM". As far as I remember what is in the CSV file overrules. Could you try changing that in the CSV?

Edit: incorrect apparently. The selection overrules.

@HansVissers
Copy link
Author

HansVissers commented Jan 28, 2020 via email

@stanstrup
Copy link
Owner

Sorry for the goose chases.

I think I found the bug. At least I was able to crash it. The system uses obabel to clean up the structures and there is a limit to the length of the command that can be send. One of my functions is missing a safeguard for that apparently.

Can you try to dividing your file in two of approximately similar size? That solves that problem here.

@HansVissers
Copy link
Author

HansVissers commented Jan 28, 2020 via email

@stanstrup
Copy link
Owner

As long as they have the same "Method" all data will be used.
If you are unlucky it makes the model based on the first set you upload. Then it will redo the model with all the data on the second pass. The system looks for new data every 10 minutes. It can take longer than that to finish the modeling though. If you give it a few hours it should be plenty to get all predictions possible.

@HansVissers
Copy link
Author

HansVissers commented Jan 28, 2020 via email

@stanstrup
Copy link
Owner

your image didn't survive email --> github. Can you put on github directly?

@HansVissers
Copy link
Author

1-28-2020 2-28-10 PM

Here it is!

@stanstrup
Copy link
Owner

Ah. So it did work!

All good?

@HansVissers
Copy link
Author

HansVissers commented Jan 28, 2020 via email

@stanstrup
Copy link
Owner

A thousand predictions now available ;)

I will leave this open as the underlying bug is not fixed.

@stanstrup stanstrup reopened this Mar 16, 2020
@YunyunZHU
Copy link

Hello, stanstrup,

I've tried to split my file into three parts, each has about 15 compounds. And I have Pubchem column even if they are all empty. But I still get the error message identical to HansVissers. I'm wondering what else I can do to solve this? Looking forward to hearing from you!
Thank you!

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

No branches or pull requests

3 participants