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

namada/vm: disable floats in wasm #3209

Merged
merged 2 commits into from
May 21, 2024
Merged

Conversation

tzemanovic
Copy link
Member

@tzemanovic tzemanovic commented May 9, 2024

Describe your changes

closes #1872

Indicate on which release or other PRs this topic is based on

0.35.1

Checklist before merging to draft

  • I have added a changelog
  • Git history is in acceptable state

Copy link

codecov bot commented May 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.24%. Comparing base (4ed6229) to head (7f3a04c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3209      +/-   ##
==========================================
- Coverage   60.24%   60.24%   -0.01%     
==========================================
  Files         303      303              
  Lines       93191    93191              
==========================================
- Hits        56145    56143       -2     
- Misses      37046    37048       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Fraccaman
Copy link
Member

can we move ibc back to wasm now?

@tzemanovic
Copy link
Member Author

can we move ibc back to wasm now?

@Fraccaman I'm not sure, it was an issue with M1/M2 which might still be there (#1831)

@tzemanovic tzemanovic marked this pull request as ready for review May 9, 2024 13:18
@tzemanovic tzemanovic requested a review from grarco May 9, 2024 13:18
brentstone added a commit that referenced this pull request May 9, 2024
* tomas/disable-wasm-floats:
  changelog: add #3209
  namada/vm: disable floats in wasm
@Fraccaman
Copy link
Member

@yito88 do u know if we can compile ibc-rs to wasm now?

@yito88
Copy link
Member

yito88 commented May 15, 2024

@Fraccaman I tried it in #2480 and it worked. I'm not sure whether the latest one works or not. I'll try it with 0.35.1

@tzemanovic tzemanovic merged commit 56f5525 into main May 21, 2024
17 of 19 checks passed
@tzemanovic tzemanovic deleted the tomas/disable-wasm-floats branch May 21, 2024 13:29
@yito88
Copy link
Member

yito88 commented May 22, 2024

tx_ibc.wasm moved back to wasm worked in #3275

@yito88 yito88 mentioned this pull request May 22, 2024
2 tasks
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.

disable floats in wasm
5 participants