Skip to content

add setup.py#2

Closed
key wants to merge 53 commits into
dtcooper:masterfrom
key:master
Closed

add setup.py#2
key wants to merge 53 commits into
dtcooper:masterfrom
key:master

Conversation

@key
Copy link
Copy Markdown

@key key commented Dec 31, 2011

Hi,

I add setup.py which is draft version of python-fitparse package.
Please fill setup.py with your information.

David Cooper added 30 commits August 8, 2011 23:04
… Disable parse() hook on definition records by default.
…clude scaling if it's = 1. Added TODO comment.
… to be arrays were declared as such and would end up in a binary parse error. I'm not crazy about the fix, but justification is in comments.
…rom having namedtuples in the first place.
@dtcooper
Copy link
Copy Markdown
Owner

dtcooper commented Jan 1, 2012

Hey Mitsukuni,

Thanks for this. I will review and add a setup.py file this week!

Thanks again,

David

On Saturday, December 31, 2011, Mitsukuni Sato <
reply@reply.github.com>
wrote:

Hi,

I add setup.py which is draft version of python-fitparse package.
Please fill setup.py with your information.

You can merge this Pull Request by running:

git pull https://github.com/key/python-fitparse master

Or you can view, comment on it, or merge it online at:

#2

-- Commit Summary --

  • add sample data which is ride on indoor trainer.
  • Merge remote-tracking branch 'dtcooper/master'
  • add setup.py

-- File Changes --

A setup.py (28)
A tests/data/sample-activity-indoor-trainer.fit (0)

-- Patch Links --

https://github.com/dtcooper/python-fitparse/pull/2.patch
https://github.com/dtcooper/python-fitparse/pull/2.diff


Reply to this email directly or view it on GitHub:
#2

David Cooper
Application Developer

T:416.601.1999 x 223
www.trapeze.com
175 Bloor St. E., South Tower, Suite 900
Toronto, ON M4W 3R8

@Tigge
Copy link
Copy Markdown

Tigge commented Sep 2, 2012

Would be really nice to have this. Should be able to publish it to the Python Package Index http://pypi.python.org/pypi then to make it really easy for people to install it with pip or easy_install.

@dtcooper
Copy link
Copy Markdown
Owner

dtcooper commented Dec 8, 2012

setup.py added to my new branch (ng).

dtcooper added a commit that referenced this pull request Dec 9, 2012
Committing the work I've been doing over the past little while on a
rewrite for python-fitparse.

Major Changes
-------------

* New, hopefully cleaner public API with a clear division between
  accessible and internal parts. (Still unstable and partially
  complete.)

* Proper documentation! Going to use Sphinx.

* Unit tests and example programs.

* (WIP) Command line tools (eg a `.FIT` to `.CSV` converter).

* Component fields and compressed timestamp headers now supported
  and not just an afterthought. Closes issues #6 and #7.

* FIT file parsing is generic enough to support all types. Going to
  have specific `FitFile` subclasses for more popular file types
  like activities.

* (WIP) Converting field types to normalized values (for example,
  `bool`, `date_time`, etc) done in a consistent way, that's easy to
  customize by subclassing the converter class. I'm going to use
  something like the Django form-style `convert_<field name>` idiom
  on this class.

* The FIT profile is its own complete python module, rather than
  using `profile.def`.

* Bonus! The profile generation script is less ugly (but still an
  atrocity) and supports every ANT FIT SDK from version 1.00 up to
  5.10.

* A working `setup.py` module. Closes issue #2, finally! I'll
  upload the package to PyPI when it's done.

* Support for parsing one record at a time. This can be done using
  `<FitFile>.parse_one()` for now, but I'm not sure of the exact
  implementation yet.
@schinckel
Copy link
Copy Markdown

Also, you should make sure you register the name fitparse on pypi so no-one else can.

@petergardfjall
Copy link
Copy Markdown

Would it be possible to also add a setup.py to the master branch? I would like to make use of your library but I'm having trouble getting the ng branch code to parse my FIT files (I guess the ng code is in a state of flux at the moment?) so I would like to make use of the old code for now.

Thanks for devoting time to maintaining the library!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants