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

add caveat and fix typos #78

Merged
merged 1 commit into from
Apr 2, 2024
Merged

add caveat and fix typos #78

merged 1 commit into from
Apr 2, 2024

Conversation

divyasharma-arup
Copy link
Contributor

Adjusted the KPI_Data_Requirements_and_Expectations.md file to reflect the risk of simulation configurations that may affect the accuracy of KPI calculations. Fixed a couple tpos in the output file names.

@divyasharma-arup divyasharma-arup requested a review from mfitz April 2, 2024 10:33
@divyasharma-arup divyasharma-arup linked an issue Apr 2, 2024 that may be closed by this pull request
Copy link
Contributor

@mfitz mfitz left a comment

Choose a reason for hiding this comment

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

👍

@divyasharma-arup divyasharma-arup merged commit 5402c6b into main Apr 2, 2024
1 check passed
@divyasharma-arup divyasharma-arup deleted the outside-mode branch April 2, 2024 12:10
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.

Ability to filter out 'outside' mode from metrics
2 participants