Confusing and wrong names

cheminfo / mzData

Read and explore mzData files.

https://cheminfo.github.io/mzData/

MIT License

2 stars 3 forks source link

Confusing and wrong names #3

Closed lpatiny closed 7 years ago

lpatiny commented 7 years ago

@maasencioh

There are two issues related to the name of this project:

It is not reading mzML but mzData. So it should be named mzData.
The npm name has a typo. Furthermore, it should not include a "js" suffix. We propose to use mzdata

maasencioh commented 7 years ago

mzData and mzXML are deprecated projects and the idea is to add those formats to this project, avoiding code replication and simplifying its integration with chromatography

lpatiny commented 7 years ago

Did you have a look about how different those formats are ?

maasencioh commented 7 years ago

Both are quite similar in the fact that both use XML tags to organize and base64 to compress, but in any case, it didn't seem trivial how to reutilize the code for both