Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

FixedU128 #6183

Closed
xlc opened this issue May 29, 2020 · 1 comment · Fixed by #6214
Closed

FixedU128 #6183

xlc opened this issue May 29, 2020 · 1 comment · Fixed by #6214
Assignees
@marcio-diaz
Labels
J0-enhancement An additional feature request.

Comments

@xlc
Copy link
Contributor

xlc commented May 29, 2020 •
edited
Loading

We have Fixed128, will be useful to have FixedU128 as well.

I wonder if the implement_fixed macro already support it?

This can also replace our implementation which have different API than Fixed128 https:/open-web3-stack/open-runtime-module-library/blob/master/utilities/src/fixed_u128.rs
@marcio-diaz
Copy link
Contributor

marcio-diaz commented May 29, 2020 •
edited
Loading

I think it should work without much changes. Let me prepare a PR.

@marcio-diaz marcio-diaz added the J0-enhancement An additional feature request. label May 29, 2020
@marcio-diaz marcio-diaz self-assigned this May 29, 2020
@marcio-diaz marcio-diaz mentioned this issue Jun 2, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@marcio-diaz marcio-diaz

Labels
J0-enhancement An additional feature request.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Enable fixed point u128 paritytech/substrate
2 participants
@marcio-diaz @xlc