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.

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

added functionality to export results to txt file #23

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

Conversation

FedericoTartarini
Copy link
Contributor

No description provided.

…These data can be then processed using the Python code in process_data_ashrae.py to generate the tables for the ASHRAE standard. This functionality is disabled by default but it can be activated by setting output_data_for_ASHRAE_table = true
@FedericoTartarini
Copy link
Contributor Author

@thoyt could you please also review the changes and accept the Pull request?

Copy link
Contributor

@thoyt thoyt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The python looks like personal scripting used to analyze and visualize the downloaded data. I think the code in this repo should be for the general purpose application.

It should be possible to focus on the export part and make that very reliable and usable. In this scenario you would have to be a power user and set the special javascript variable to true and read the comments for instructions on how to use it. Is there a reason not to create a csv download button that downloads the whole dataset and leave the postprocessing to users?

Maybe I don't know the goal here but I imagine we want a simple button for downloading.. @praftery ?

ASHRAE 55 Tables/process_data_ashrae.py Outdated Show resolved Hide resolved
ASHRAE 55 Tables/process_data_ashrae.py Outdated Show resolved Hide resolved
js/main.js Outdated Show resolved Hide resolved
ASHRAE 55 Tables/mrt inputs Outdated Show resolved Hide resolved
ASHRAE 55 Tables/process_data_ashrae.py Outdated Show resolved Hide resolved
@FedericoTartarini
Copy link
Contributor Author

You are right, I will remove the folder named ASHRAE 55 Tables from the project. I used the Python code to generate the tables for David however, there is no need to add this code to the GitHub repository. I will now send a new commit to fix the above mentioned issues.

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.

2 participants