Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Load more properties from CSV #55

Closed
kwinkunks opened this issue Jan 7, 2016 · 2 comments
Closed

Load more properties from CSV #55

kwinkunks opened this issue Jan 7, 2016 · 2 comments
Labels
bug

Comments

@kwinkunks
Copy link
Member

Right now we only support top, base, description, but it wouldn't be too hard to allow more properties.

The code is a bit gross, so could use cleaning up anyway. Pretty sure most of it is just doing the work of DictReader...
@kwinkunks kwinkunks added the bug label Jan 14, 2016
@kwinkunks
Copy link
Member Author

Calling this a bug, the code needs refactoring.

Should support loading arbitrary data.

@mtb-za
Copy link
Contributor

mtb-za commented Apr 21, 2021

Closing this in favour of #128; the issues are essentially the same. Currently we add arbitrary new columns to data anyway, even if we do not actually do anything special with that. cf #60 which also discusses this.

We do still need to discuss what other aliases we might actually want, but that should happen in #128....

@mtb-za mtb-za closed this as completed Apr 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mtb-za @kwinkunks