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

Strong Motion Azimuth Deployment Guide/Doc #1967

Merged
merged 6 commits into from
May 16, 2024

Conversation

CBurton90
Copy link
Contributor

No description provided.

Copy link
Contributor

@mnaguit mnaguit left a comment

Choose a reason for hiding this comment

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

This is a good documentation, thanks @CBurton90 👍 A similar one for the structural arrays would surely be very handy for our data users.
The content is all good for me, I am just thinking if citing sensor examples for FBA/MEMS would be useful info (although not really relevant with azimuth)?

staylorofford
staylorofford previously approved these changes Feb 13, 2024
@staylorofford
Copy link
Contributor

Following on from what Muriel states, it may simply not be needed to say FBA/MEMS and just say it applies to all SM sensors outside of structural arrays

Copy link
Contributor

@elidana elidana left a comment

Choose a reason for hiding this comment

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

Hi @CBurton90 this one looks really great and incredibly useful!!

I have added some very minor comments, take it or leave it as you see fit!!

docs/STRONG_MOTION_DEPLOYMENT_AZIMUTH.md Outdated Show resolved Hide resolved
docs/STRONG_MOTION_DEPLOYMENT_AZIMUTH.md Show resolved Hide resolved
docs/STRONG_MOTION_DEPLOYMENT_AZIMUTH.md Outdated Show resolved Hide resolved
Copy link
Contributor

@salichon salichon 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

docs/STRONG_MOTION_DEPLOYMENT_AZIMUTH.md Show resolved Hide resolved
@CBurton90 CBurton90 mentioned this pull request Mar 27, 2024
@salichon
Copy link
Contributor

salichon commented Apr 9, 2024

Kia Ora
@CBurton90 It this good enough to be merged ?
cheers :)

@CBurton90
Copy link
Contributor Author

@salichon I just need to tidy up the question here then yes. Will put it on my list today, thanks for the reminder.

@CBurton90
Copy link
Contributor Author

#1967 (comment)

@salichon Is this a question? Yes for case 2 the field tech will measure true north outside of the enclosure/infrastructure and compare instrument alignment from that (to obtain the sensor/recorder alignment within the space constrained enclosure).

@CBurton90
Copy link
Contributor Author

Following on from what Muriel states, it may simply not be needed to say FBA/MEMS and just say it applies to all SM sensors outside of structural arrays

@staylorofford @mnaguit I've removed any mention of FBA/MEMS. Thanks.

@CBurton90
Copy link
Contributor Author

@mnaguit @salichon Could you please merge this one after a review : ) - I have no merge permissions!

@mnaguit mnaguit merged commit f27864b into GeoNet:main May 16, 2024
9 checks passed
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.

5 participants