-
Notifications
You must be signed in to change notification settings - Fork 3
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
backslash added to metabolite names #23
Comments
Yes, chemical names can be difficult..... But this is the actual name in the Wikidata entry: I believe the backslash is there, so that R can read the data, right? Or is that not working? |
I've now changes the entry, so the name is 5',5'' (so using the single quote twice, iso using a double quote). |
It messes up in R when there is a |
Forward or backward slash, which one is causing the issue? |
backslashes are added when there is |
I removed them manually in the file, to keep the app running correctly for now. |
How many are there, could you give me some more examples? |
it is around 20 to 30, I think.
|
When I filter the results with a regex in the query of the names itself "FILTER(REGEX(?name, "['"\/]", "i")).", I get: 96963 results in 29005 ms. We could either filter these out before we obtain the data (changing the SPARQL query), or find a way so that R can read these (e.g. like this).... I don't think we should replace these characters with another one (besides changing the double quote to two single quotes). What's your preference @tabbassidaloii ? |
@DeniseSl22 There are options in R to remove them, but I have concerns about causing unnecessary changes in the other values (e.g. there are metabolites with a backslash ( |
When I look at the raw Github data, I don't see this issue....
Might need to be adapted? |
This is weird because I see it in the files I downloaded from GitHub, before opening them in R. |
As discussed:
|
@DeniseSl22 if #9 is solved we can work on #106 considering below
|
@DeniseSl22
In the metabolite files when there is
'
or"
in the metabolite name a\
has been added, see the example below:The text was updated successfully, but these errors were encountered: