search

MartijnAB / from-toxicological-data-to-toxicological-information

A internship project with the intent to connect existing and/or provided toxicological data, with other toxicological data in order to better understand it.
0 stars 0 forks source link

make WikiPathways class (Story 1) #14

Open MartijnAB opened 5 years ago

MartijnAB commented 5 years ago

zo als vaak met python en ids gaan in imports fout ziet u zo snel de goed manier?

https://github.com/MartijnAB/from-toxicological-data-to-toxicological-information/blob/master/main/main.py

MartijnAB commented 5 years ago

het lijkt mij een goed idee om ook de labels (zoals "Codeine-6-glucuronide") te bewaren zodat ze ook in de nootbook gebruikt kunnen worden. Wat vind u?

mkutmon commented 5 years ago

yes, but that will be part of the chemical compound class, right?

mkutmon commented 5 years ago

zo als vaak met python en ids gaan in imports fout ziet u zo snel de goed manier?

hmm, no

MartijnAB commented 5 years ago

zo als vaak met python en ids gaan in imports fout ziet u zo snel de goed manier?

hmm, no

@mkutmon

Voor nu is het opgelost, want voor nu werk ik van uit de main functie in het bestand zelf.

MartijnAB commented 5 years ago

yes, but that will be part of the chemical compound class, right?

@mkutmon

Ja maar het aan maken van de the chemical compound class gebuurt in de WikiPathways class; van daat de vraag.

MartijnAB commented 5 years ago

@mkutmon

puur op functionaliteit is de class af.

om het regel 60 in het bestand: https://github.com/MartijnAB/from-toxicological-data-to-toxicological-information/blob/master/main/model/WikiPathways.py#L67 zonder exit() te laten verlopen zal er een error in moeten, ik denk dat dat netter is, ook naar de gebruiker; wat denkt u?

mkutmon commented 5 years ago

yeah, I think it's important to tell the user what happened and that no Pathway will be returned!