Skip to content

flamenco, choreo, app: Integrate Parallel Funk #4557

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

Merged
merged 1 commit into from
Apr 2, 2025

Conversation

topointon-jump
Copy link
Contributor

Latest Funkier integration branch

@topointon-jump topointon-jump force-pushed the topointon/funkier-merge branch 4 times, most recently from bcb1b41 to 755548b Compare March 25, 2025 19:07
@topointon-jump topointon-jump force-pushed the topointon/funkier-merge branch 6 times, most recently from 3166238 to a8bd2e9 Compare March 31, 2025 19:56
@topointon-jump topointon-jump force-pushed the topointon/funkier-merge branch from a8bd2e9 to a82271b Compare April 1, 2025 19:45
if( data ) {
fd_account_meta_t const * meta = fd_type_pun_const( data );
(*ctx->acct_fun)( msg->accts.funk_xid.ul[0], msg->accts.sig, &addr, meta, (uchar*)data + meta->hlen, meta->dlen, ctx->fun_arg );
if( datalen >= meta->hlen + meta->dlen ) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the thinking about adding this?

jumpsiegel
jumpsiegel previously approved these changes Apr 2, 2025
@topointon-jump topointon-jump force-pushed the topointon/funkier-merge branch from 4d822fc to 3f5ea72 Compare April 2, 2025 19:04
@topointon-jump topointon-jump changed the title [wip] Funkier integration #2 Funkier Integration #2 Apr 2, 2025
@topointon-jump topointon-jump changed the title Funkier Integration #2 flamenco, choreo, app: integrate new Funkier Apr 2, 2025
@topointon-jump topointon-jump force-pushed the topointon/funkier-merge branch 2 times, most recently from abc17a7 to 8d8bb40 Compare April 2, 2025 21:12
@topointon-jump topointon-jump changed the title flamenco, choreo, app: integrate new Funkier flamenco, choreo, app: Integrate Parallel Funk Apr 2, 2025
Copy link
Contributor

@cali-jumptrading cali-jumptrading left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok so in general, I only gave this a glance and only looked at the first few files. Would be a good to do a pass to fix alignment :)

@topointon-jump topointon-jump force-pushed the topointon/funkier-merge branch 2 times, most recently from 6539b50 to 9b35974 Compare April 2, 2025 22:45
@topointon-jump topointon-jump force-pushed the topointon/funkier-merge branch from 9b35974 to d2f39d6 Compare April 2, 2025 22:48
Copy link
Contributor

@ibhatt-jumptrading ibhatt-jumptrading left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🇺🇸

@topointon-jump topointon-jump enabled auto-merge April 2, 2025 22:49
@topointon-jump topointon-jump added this pull request to the merge queue Apr 2, 2025
Merged via the queue into main with commit f8262f7 Apr 2, 2025
11 checks passed
@topointon-jump topointon-jump deleted the topointon/funkier-merge branch April 2, 2025 23:06
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.

4 participants