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

Principle work to handle conversion between unit Deg/Rad/(H|D)MS #1

Merged
merged 9 commits into from
Sep 26, 2024

Conversation

Republicof1
Copy link
Contributor

@Republicof1 Republicof1 commented Sep 25, 2024

this is the initial commit of the collection of transformations too and/or from various representations of spatial position

Republicof1 and others added 9 commits September 17, 2024 17:23
change the package.json to turn on the ESM capability of jest. Change eslint to most recent compatible version (8.57.0). 
This commit also moves over my forked version of tsofa
Renamed test file to astroLib.tests.ts (note the ts format).
Jest config update to support ESM in ts-jest
version number updated in package.json
Add abstraction between the accessor adjunct and the implementation. This will allow for more managed access to the package contents.
Updated collection of tests and functionality
currently DMS, HMS can be converted to and from Rad and Deg
Roundtrip HMS/DMS <=> Deg/Rad at high precision. Note this is dependent on a change in TSOFA. Precision values set to higher than required level of 14 decimal places for Rad<=>Deg and 10dp for Sexagesimal conversions.

minor correction to tsconfig

correction to vv_ms test rig to manage acceptance of similar output (rather than exact character match)
* note that the output goes into the dist directory
* also special tsconfig just for jest
@pahjbo
Copy link
Member

pahjbo commented Sep 26, 2024

CI still not working - which is irritating - cannot access the public tsofa package in github packages....

@pahjbo pahjbo merged commit cc83d5f into main Sep 26, 2024
1 check failed
@pahjbo pahjbo deleted the 1-exchange-of-formats branch September 26, 2024 14:48
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