-
Notifications
You must be signed in to change notification settings - Fork 12
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
Creating a custom database #17
Comments
Hi @MarcNIOZ, Let's start with the first (less complex) error you are receiving when attempting to recreate the default database. When running your code using a clean version of the component definitions file (e.g., which I just downloaded from https://github.com/vanmooylipidomics/LOBSTAHS/blob/master/inst/doc/csv/LOBSTAHS_componentCompTable.csv), I did not receive the same error. However, I just took a look at the default version of the LOBSTAHS_componentCompTable.csv file you are using (which you kindly sent along by e-mail), and the problem appears to be ~ 130 empty lines of csv data at the end of your file. Here is a sample of what it looks like when I open the file in a text reader:
Those "empty" lines continue for a bit. Perhaps these were somehow retained after you added new entries to the file and then deleted them? This can happen if you don't actually delete the "empty" lines from the file. What were you using to edit your .csv files? Microsoft Excel is terrible about this. Take a look and see if you can fix the problem. Will take a look at the second issue here soon. |
I am also opening an issue #18 for a new feature request with the @vanmooylipidomics/lobstahs-devel-team to see if we can't write something to make LOBSTAHS smart enough to correct/ignore that sort of problem in the future. |
Hello Jamie, Thanks for all the trouble. I used excel for this and it is possible that I tried it with the added compounds and then removed them for the default option. This probably kept some of the lines “open”. I will try to see if this also the case with the added compounds error. |
Oke, I downloaded the original database file to try again. However now I got the same error as with the added compounds file.
|
Hi Marc: I am unable to reproduce the calcComponentMasses building blocks error when I regenerate the default databases in this way, and I am able to generate the databases without issue. Tell me what you get when you try the below:
Result should be 18. |
Hmm, strange. I checked it and it also gives 18. Is there a way to download the original .csv file? Or can you send it to me? Perhaps it has something to do with excel and the convertion to .csv. |
Did you download directly from here: https://github.com/vanmooylipidomics/LOBSTAHS/blob/master/inst/doc/csv/LOBSTAHS_componentCompTable.csv |
I downloaded the excel file including the instructions. From the following page: https://github.com/vanmooylipidomics/LOBSTAHS/blob/master/inst/doc/xlsx/LOBSTAHS_componentCompTable.xlsx |
What if you try using the .csv from the link above, do you still receive the error? I also just tried again, this time by saving the second tab of the LOBSTAHS_componentCompTable.xlsx spreadsheet as a .csv file; it worked fine for me. |
I tried it (by right clicking on raw and then save as). However I couldn't download it as it says "insufficient rights" (translated from Dutch). |
What is "it" and "raw," and where did you attempt to download the file from? Please be specific -- hard to recreate a workflow with nondescriptive terms. |
Sorry, you meant you clicked the "raw" tab on this page: https://github.com/vanmooylipidomics/LOBSTAHS/blob/master/inst/doc/csv/LOBSTAHS_componentCompTable.csv and then attempted to download? I am not sure what that error is about; sounds like problem specific to your browser or OS. What if you just copy the text, paste into a text document, and then save it? |
Ah yes, sorry, it is getting late here. I was about to make a screen shot. You were right, chrome didn't let me download it. I was able to do it with firefox but still got the same error. |
After much discussion and trial and error (mostly via a long email chain between @jamesrco and @MarcNIOZ), determined source of the second error was a very critical commit that did not get passed from the LOBSTAHS GitHub repository to Bioconductor in time to meet the Bioconductor 3.5 release date. I've pushed this commit and a few others to both the devel and release-3.5 Bioconductor branches. Any user installing (or re-installing) the package after roughly noon tomorrow (June 28, 2017) using
will get the correct code, which contains the necessary fixes. The revised code requires that the custom LOBSTAHS_componentCompTable (if used) contain 18 columns, including a column for silicon atoms ("Si"). Thanks to @MarcNIOZ for helping to figure this one out. Leaving this issue open for a day or so to make sure the changes propagate, then will close it. |
On a potentially unrelated note, this also cleared up my earlier issue with the vignette. I was using the bioconductor version to run it before. I cloned the latest master and it seems to be working properly. |
@lee-t Remind me to walk you though how we get files from this Git repo to the correct place on the Bioconductor Subversion server. It is far from obvious, and very easy to screw up. I've messed it up a bunch of times, and in one of those instances it took me a day at the command line to fix everything. Supposedly they're migrating to pure Git soon, but the project seems to be on hold. |
Closing this one out. The Bioconductor release installation now loads LOBSTAHS v1.2.1, which contains the necessary fixes. |
Hello all,
I tried to generate a lobstah database but I encountered a couple of errors.
When I try create a database with the default .csv file I got the following:
When I tried to create a database with some additional compounds I got the following:
Perhaps somebody can help me with this.
Thanks in advance,
Marc Besseling
PhD student at the Netherlands Institute for Sea Research (NIOZ)
The text was updated successfully, but these errors were encountered: