pishleback / pishleback/Algebraeon

More operations between Natural, Integer, Rational and builtin types

Open
#54 5 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement good first issue
Dominant language
Rust
Stars
82
Forks
10
PR merge metrics
No merged PRs in 30d

Description

As discussed in #52 it is currently a bit messy to compute the remainder of a Natural by a u32 and obtain a u32, though it is possible by converting Natural to usize first. This issue is for adding more conversions and operations involving a mixture of algebraeons Natural, Integer and Rational types with rusts builtin u8, u16, u32, u64, u128, usize, i8, i16, i32, i64, i128 and isize where appropriate.

  • TryInto<u8 ... u128, usize, i8 ... i128, isize> for Natural (#55)
  • TryInto<u8 ... u128, usize, i8 ... i128, isize> for Integer (#55)
  • TryInto<u8 ... u128, usize, i8 ... i128, isize> for Rational (#81)
  • Natural % T -> T for T = i8 ... i128, isize,u8...u128,usize
  • Integer % T -> T for T = i8 ... i128, isize,u8...u128,usize

Any others to include on the list?

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading issue #52 and the completed conversion work from #55 and #81 to find the existing Natural, Integer, and Rational implementations. Check the current remainder behavior and tests, then add the listed Natural % T and Integer % T operations for the appropriate builtin integer types; done means the operations and coverage work consistently across the checklist.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.