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

Download inputs as a csv #1463

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Download inputs as a csv #1463

wants to merge 2 commits into from

Conversation

louispt1
Copy link
Contributor

Download inputs as a csv with:

  • key
  • min
  • max
  • present value: default
  • future value: user
  • unit
  • share group

at /api/v3/scenarios/2601971/inputs.csv

Closes #1449

@mabijkerk
Copy link
Member

Fast work! Two questions:

  1. Normally when we download a data-export, the SessionID is appended to the file. Instead of inputs.csv I would then get inputs.123456.csv. Is this something we can add here as well?
  2. When I go to the JSON inputs endpoint, all the default values are filled in. For example costs_bio_ethanol has "default": 0.948245519132. When I download the CSV for the same scenario however, this key (and a lot of other keys) have no default value. Is something going wrong here?

@louispt1 louispt1 force-pushed the inputs-csv-endpoint branch 3 times, most recently from 56aea09 to 764c4c1 Compare October 15, 2024 15:44
@louispt1
Copy link
Contributor Author

Good points! I have resolved both I think, and added some additional spec for the csv download - could you verify again @mabijkerk?

@mabijkerk
Copy link
Member

No I get this error I think:

Screenshot 2024-10-16 at 09 02 11

@mabijkerk mabijkerk mentioned this pull request Oct 25, 2024
Open
@louispt1 louispt1 force-pushed the inputs-csv-endpoint branch from 764c4c1 to db5029f Compare October 25, 2024 07:59
@louispt1 louispt1 force-pushed the inputs-csv-endpoint branch from c7ba7b7 to 452ccc9 Compare November 19, 2024 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@noracato noracato Awaiting requested review from noracato

@mabijkerk mabijkerk Awaiting requested review from mabijkerk

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add data-export for inputs
2 participants
@louispt1 @mabijkerk