Skip to content

Conversation

@jacob-indigo
Copy link

@jacob-indigo jacob-indigo commented Jan 11, 2024

Addresses issue #1912.

This is my first time contributing to Pint so I wasn't sure what your deprecation and versioning practices are. I added a placeholder in CHANGES for now but let me know what version you'd include this in (given that it's technically a breaking change) and how/if some notice should be given ahead of time (e.g. warn("The definition of acre will change in version 0.x to conform the 2022 revision by NIST") or whatever).

@jacob-indigo jacob-indigo requested a review from dalito January 17, 2024 14:23
Copy link
Contributor

@dalito dalito left a comment

Choose a reason for hiding this comment

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

Looks fine. Data and source are correct.

Many will look at the diff of this unit definitions file. So you could make it a bit easier to review the changes (see comments).

rod = 16.5 * foot = rd = pole = perch

square_rod = rod ** 2 = sq_rd = sq_rod = sq_pole = sq_perch
acre = 43560 * square_foot
Copy link
Contributor

Choose a reason for hiding this comment

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

You could leave it as is was to minimize diffs. acre = 10 * chain ** 2

Copy link
Author

Choose a reason for hiding this comment

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

I did this to be consistent with this statement across all of the units here

but we've followed the NIST's lead here and defined all of the units directly based on "common" units (feet and miles)

@dbhart
Copy link

dbhart commented Feb 27, 2024

According to NIST HB 44-2024, the original citation is Federal Register (85 FR 62698, Oct. 5, 2020) pp. 62698-62708. Despite the deprecation, the old units are still available for working with historical data, prefixed by the letter "s" - i.e., sft and smi (which are already in pint), but also sli, srd, sch, sfur, and sac for link, rod, chain, furlong and acre, respectively. So, it might be appropriate to still have those definitions in USCSLengthSurvey, and put the new survey measures into USCSLengthInternational? Then the deprecation notice can go into USCSLengthSurvey?

https://www.govinfo.gov/content/pkg/FR-2020-10-05/pdf/2020-21902.pdf

@hgrecco
Copy link
Owner

hgrecco commented Mar 13, 2024

@jacob-indigo what do you think about @dbhart proposal?

@jacob-indigo jacob-indigo force-pushed the update_usgs_survey_length branch from c1f76e3 to e56fead Compare May 8, 2025 18:37
@jacob-indigo
Copy link
Author

jacob-indigo commented May 8, 2025

@dbhart @hgrecco @dalito I think that proposal makes sense. I just pushed up a change to make that happen. I would appreciate a fresh review of this PR!

@jacob-indigo jacob-indigo requested a review from dalito May 8, 2025 18:55
@jacob-indigo
Copy link
Author

jacob-indigo commented Jul 16, 2025

@dgoeries @andrewgsavage @maurosilber Is this something you could help me get reviewed?

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.

USCSLengthSurvey group is defined based on the survey foot which is now defunct

4 participants