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

Merge tiegan-working branch into master #92

Merged
merged 6 commits into from
May 29, 2023
Merged

Conversation

tieganh
Copy link
Collaborator

@tieganh tieganh commented Aug 31, 2022

Mostly tidying up of the repository in preparation for public release, and some work on sensitivity testing and risk disaggregation functions over summer 2022.

  • Changed the names of the cHazard and cDamage folders to include the word 'beta' as they are not part of the official release.
  • Updated the readme in English and French.
  • Centralized most scripts into the 'scripts' folder.
  • Deleted old scripts that have been superseded.
  • Replotted figures in the sensitivity testing folder while preparing report for publication.

There should be nothing in here that affects others work, to my knowledge. Will be making a new release from master once this branch is merged, which will be the official first public release.

DamonU2
DamonU2 previously approved these changes Sep 1, 2022
anthonyfok
anthonyfok previously approved these changes Oct 20, 2022
Copy link
Member

@anthonyfok anthonyfok left a comment

Choose a reason for hiding this comment

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

Looks good to me! Would be nice if you could fix the minor Markdown link formatting error as shown below, but it also could be fixed later. Cheers!

LISEZMOI.md Outdated Show resolved Hide resolved
@tieganh tieganh dismissed stale reviews from anthonyfok and DamonU2 via 9eafd97 May 10, 2023 19:40
@tieganh
Copy link
Collaborator Author

tieganh commented May 10, 2023

Hey @anthonyfok do we still need to approve this or has it been superceded?

@wkhchow
Copy link
Collaborator

wkhchow commented May 10, 2023

Sorry @tieganh missed this PR from August 2022!
The master has been changed since August, and I have a PR ready to be merged. Can the edits from this PR be 'cherry picked' to the master? @anthonyfok I think most of the edits here dont affect any of the newer changes in the master (just geopackage updates).

Also we would need to file a issue to fix for a new revision of opendrr/model-factory as I suspect the change in chazard/cdamage to have beta suffix would need to change some parts of the psra ingest @drotheram.
But since there are no changes at the moment to psra we can still run the build on the dsra for additional scenarios.

@anthonyfok
Copy link
Member

anthonyfok commented May 11, 2023

Hey @anthonyfok do we still need to approve this or has it been superceded?

Hey @tieganh, we still need this PR to be merged eventually. It is not superceded, but there are some merge conflicts which I'll try to resolve tomorrow or early next week.

@tieganh
Copy link
Collaborator Author

tieganh commented May 17, 2023 via email

See commit 8b3be4f
The French text was translated with DeepL.com with minor manual editing
such as changing " " to « », and using non-breaking space where
necessary.
Copy link
Member

@anthonyfok anthonyfok left a comment

Choose a reason for hiding this comment

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

This branch / pull-request has been rebased on top of "master" with merge conflicts resolved.

@tieganh, I took the liberty of changing the date in the README.md file from "11 Aug 2021" to "11 Aug 2022", and to keep what you wrote on 15 Oct 2022 in there too. Please feel free to make further changes if you like in a new pull request.

@anthonyfok anthonyfok merged commit e3e3110 into master May 29, 2023
anthonyfok added a commit to OpenDRR/opendrr-api that referenced this pull request Apr 2, 2024
As part of the OpenDRR/seismic-risk-model publication cleanup
originally intended for v1.0.0 release but was overlooked in 2022,
sourceTypes.csv was moved to scripts/sourceTypes.csv on 2023-05-29
on master branch, making into v1.1.0 release on 2023-09-12.

See OpenDRR/seismic-risk-model#92
anthonyfok added a commit to OpenDRR/opendrr-api that referenced this pull request Apr 2, 2024
Handle the cHazard → cHazard_beta (as well as cDamage → cDamage_beta
even if disabled) rename in OpenDRR/seismic-risk-model
since 2023-05-29 (master) and 2023-09-12 (v1.1.0);
see OpenDRR/seismic-risk-model#92

Fixes #245
anthonyfok added a commit to OpenDRR/opendrr-api that referenced this pull request Apr 2, 2024
As part of the OpenDRR/seismic-risk-model publication cleanup
originally intended for v1.0.0 release but was overlooked in 2022,
sourceTypes.csv was moved to scripts/sourceTypes.csv on 2023-05-29
on master branch, making into v1.1.0 release on 2023-09-12.

See OpenDRR/seismic-risk-model#92
anthonyfok added a commit to OpenDRR/opendrr-api that referenced this pull request Apr 2, 2024
Handle the cHazard → cHazard_beta (as well as cDamage → cDamage_beta
even if disabled) rename in OpenDRR/seismic-risk-model
since 2023-05-29 (master) and 2023-09-12 (v1.1.0);
see OpenDRR/seismic-risk-model#92

Fixes #245
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants