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

Fixed point version #3

Open
Mahdi89 opened this issue May 4, 2018 · 3 comments
Open

Fixed point version #3

Mahdi89 opened this issue May 4, 2018 · 3 comments

Comments

@Mahdi89
Copy link

Mahdi89 commented May 4, 2018

Hi,
I have a data type fix for the fixed-point version (my handle is Mahdi89).
I understand that fixed-point simulation takes longer to converge but afaik it utilises resource better and its integer/fraction ratio could be explored based on the input data.

Thanks

@lavagno
Copy link
Collaborator

lavagno commented May 4, 2018 via email

@max2ma
Copy link
Owner

max2ma commented May 7, 2018

Hi Mahdi89,
Do you have your code ready now? If you like, I could add you as a collaborator so that you can contribute to this repo directly. Or you could make a fork from this repo, make your modification and then send a merge request.
Hope you see your contribution.
Best regards,
Liang

@Mahdi89
Copy link
Author

Mahdi89 commented May 8, 2018

Hi Liang,
Thanks for your reply,
Sure, I have couple of PRs for getting fixed point in, which I think would be easier to submit them as separate PRs and you can review them separately and merge into master after review. My handle is Mahdi89.
Thanks

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

No branches or pull requests

3 participants