convertTo into sequence units should not use Euclidean division
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 65/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- javascript
- Domain
- data
Research direction
Start by locating the convertTo operation for sequence units and trace how it divides negative measurements. Reproduce the -15 inches example and compare its result with the requested (-1ft, -3in) output; done means sequence conversion uses the intended division behavior while calendar arithmetic retains Euclidean division.
Written by the indexing model from the issue text.
Description
I'm not sure if Euclidean division is correct here. We want -15 inches to be (-1ft, -3in), not (-2ft, 9in). We want Euclidean division for calendar arithmetic.
Originally posted by @sffc in https://github.com/tc39/proposal-amount/pull/141#discussion_r3787765640
- Dominant language
- HTML
- Stars
- 58
- Forks
- 12
- Avg merge
- 2h 35m
- Merged PRs (30d)
- 2
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from tc39/proposal-amount
-
Prior Art Open
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
tc39/proposal-amount#43 · 9 comments ·
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
tc39/proposal-amount#158 ·
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
tc39/proposal-amount#157 · 3 comments ·
-
Difficulty 3/5 1-2 days Newbie friendliness 70/100
tc39/proposal-amount#154 · 1 comment ·
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
tc39/proposal-amount#153 · 4 comments ·
All issues in tc39/proposal-amount
Similar issues
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
Schema-level dtype cannot be serialized: to_yaml raises RepresenterError, to_json raises TypeError Open
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
unionai-oss/pandera#2511 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
unitaryfoundation/qldpc-challenge#1651 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 90/100
statsmodels/statsmodels#10271 ·