-
Notifications
You must be signed in to change notification settings - Fork 52
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
Explicitly import pyOpenMS #278
Comments
Totally agree. And I like |
good point! agree |
Same here. It has always annoyed me that the star import does not make explicit which functions or methods from OpenMS are being used in the examples. |
Should @greengypsy or I takeover this issue? |
@Kastakin did you already move forward here? Just checking because I would like to prevent duplicated work if @tapaswenipathak or @greengypsy take over. |
Sorry for the late reply, I'm having a busy month due to some university workshops. I think I'll be able to work on the task from the 12th of June forward. If you intend to start working on it maybe I can join in later if we coordinate properly! |
I leave this to @tapaswenipathak or @greengypsy to decide. Not super high priority but nice to have. |
On Wed, Jun 1, 2022 at 9:29 PM Lorenzo Castellino ***@***.***> wrote:
Sorry for the late reply, I'm having a busy month with some university workshops. I think I'll be able to work on the task from the 12th of June forward.
If you intend to start working on it maybe I can join in later if we coordinate properly!
Hi Lorenzo! we'll start on 12th June, see you at
https://discord.com/invite/swkxs72CHB.
…
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
@timosachsenberg |
Yes, either this and then use |
My preference would be the first option to make explicit what is part of pyOpenMS. Explicitly:
import pyopenms as poms
poms.MSExperiment()
(Other abbreviations than poms are fine too, I like poms)
On 8. May 2023, at 16:46, Julianus Pfeuffer ***@***.******@***.***>> wrote:
Yes, either this and then use oms.MSExperiment() etc.
Or just import what is needed (from pyopenms import MSExperiment, FileHandler, ...)
—
Reply to this email directly, view it on GitHub<#278 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AGKDZED5TI3HDWV5VHA53KDXFEBKVANCNFSM5V3HVNAA>.
You are receiving this because you commented.Message ID: ***@***.***>
|
Thanks for the clarifications! My vote for abbreviation is on poms ;) |
haha that is funny - I personally used oms so far ( I guess because it sounds more like a mantra (and poms like the small town Boms ;) ) |
Star imports are a bad practice and should be avoided in the examples proposed in the documentation.
While learning the OpenMS framework I usually imported the package with
import pyopenms as ms
, maybe it's possible to decide on a two letter identifier to import the package.I can work on the issue if it's considered worth by everyone and not just me!
The text was updated successfully, but these errors were encountered: