entity manager through core on dev #11359
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
alecsavvy
commented
Feb 12, 2025
| audiusd: | ||
| container_name: audiusd | ||
| image: audius/audiusd:prerelease | ||
| image: audius/audiusd:local |
Contributor
Author
There was a problem hiding this comment.
TODO: change once this is merged AudiusProject/audiusd#51
alecsavvy
commented
Feb 12, 2025
package.json
Outdated
| "verify": "turbo run verify", | ||
| "CORE=======================================": "", | ||
| "buf:generate": "buf generate buf.build/audiusd/protocol:main --include-imports && npm run buf:generate:postinstall", | ||
| "buf:generate": "buf generate buf.build/audiusd/protocol:as/em --include-imports && npm run buf:generate:postinstall", |
Contributor
Author
There was a problem hiding this comment.
TODO: change once this is merged AudiusProject/audiusd#51
alecsavvy
commented
Feb 12, 2025
Comment on lines
+93
to
+106
| return { | ||
| status: true, | ||
| transactionHash: txhash, | ||
| transactionIndex: 0, | ||
| blockHash: res.blockHash, | ||
| blockNumber: Number(res.blockHeight), | ||
| from: signer || "", | ||
| to: signer || "", | ||
| cumulativeGasUsed: 10, | ||
| gasUsed: 10, | ||
| effectiveGasPrice: 420, | ||
| logs: [], | ||
| logsBloom: "" | ||
| } |
Contributor
Author
There was a problem hiding this comment.
dress up core response to look like an acdc (ethereum) tx
alecsavvy
commented
Feb 12, 2025
Comment on lines
+225
to
+241
| indexed_slot = index_core_plays( | ||
| logger=logger, | ||
| session=session, | ||
| challenge_bus=challenge_bus, | ||
| latest_indexed_slot=latest_indexed_slot, | ||
| indexing_plays=indexing_plays, | ||
| block=block, | ||
| ) | ||
|
|
||
| indexed_em_block = index_core_entity_manager( | ||
| logger=logger, | ||
| update_task=self, | ||
| web3=self.web3, | ||
| session=session, | ||
| indexing_entity_manager=indexing_entity_manager, | ||
| block=block, | ||
| ) |
Contributor
Author
There was a problem hiding this comment.
refactored a lil bit, can optimize this later. this organization makes it very readable
phelpsdb
approved these changes
Feb 12, 2025
rickyrombo
approved these changes
Feb 13, 2025
audius-infra
pushed a commit
that referenced
this pull request
Feb 13, 2025
[3932be3] entity manager through core on dev (#11359) alecsavvy [e8eb855] fix lint (#11362) alecsavvy [42944f4] Fix plays indexing starvation (#11361) Isaac Solo [9fa5f75] Revert "filter chain health on these nodes" (#11358) alecsavvy [e85347a] [C-5763 C-5603] Migrate profile collections to tan-query (#11285) Dylan Jeffers [48ba8aa] [C-5767] Migrate related-artists to tan-query #2 (#11303) Dylan Jeffers
schottra
added a commit
that referenced
this pull request
Feb 13, 2025
* origin/main: (21 commits) Test listen streak challenge 2 listens in 1 day (#11373) Listen streak challenge fix 2 plays in one day (#11372) Randomize storage node ordering in StorageNodeSelector. (#11371) Disable endless listen streak challenge, delete rows (#11369) Fix user-link overflow (#11370) MRVR country fixes (#11118) [C-5270 C-5269 C-5268 C-5629] Comment layout improvements (#11360) 🪦 poa ganache (#11367) Fix playlist invalid sort test (#11368) core block diff health check (#11365) Audius Protocol v0.7.90 Hide endless challenge audio (#11364) [C-5769] Use batch fetching to normalize core entity hooks (#11355) entity manager through core on dev (#11359) fix lint (#11362) Fix plays indexing starvation (#11361) Revert "filter chain health on these nodes" (#11358) Tag audiusd with protocol releases to allow reverts (#11347) [C-5763 C-5603] Migrate profile collections to tan-query (#11285) [C-5767] Migrate related-artists to tan-query #2 (#11303) ...
schottra
added a commit
that referenced
this pull request
Feb 14, 2025
…ibles * origin/main: (71 commits) [PAY-3881][PAY-3883] Add indexing/endpoints for separated collectibles data (#11287) [C5778] Update web profile page layout (#11374) Test listen streak challenge 2 listens in 1 day (#11373) Listen streak challenge fix 2 plays in one day (#11372) Randomize storage node ordering in StorageNodeSelector. (#11371) Disable endless listen streak challenge, delete rows (#11369) Fix user-link overflow (#11370) MRVR country fixes (#11118) [C-5270 C-5269 C-5268 C-5629] Comment layout improvements (#11360) 🪦 poa ganache (#11367) Fix playlist invalid sort test (#11368) core block diff health check (#11365) Audius Protocol v0.7.90 Hide endless challenge audio (#11364) [C-5769] Use batch fetching to normalize core entity hooks (#11355) entity manager through core on dev (#11359) fix lint (#11362) Fix plays indexing starvation (#11361) Revert "filter chain health on these nodes" (#11358) Tag audiusd with protocol releases to allow reverts (#11347) ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
How Has This Been Tested?
audius-cmd test should pass, user creates and track uploads
create a couple users, like, repost, create playlists, everything should work as if it's on acdc