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

[framework] Add any_map::AnyMap #15574

Draft
wants to merge 2 commits into
base: igor/btree_map
Choose a base branch
from

Conversation

igor-aptos
Copy link
Contributor

Description

How Has This Been Tested?

Key Areas to Review

Type of Change

  • New feature
  • Bug fix
  • Breaking change
  • Performance improvement
  • Refactoring
  • Dependency update
  • Documentation update
  • Tests

Which Components or Systems Does This Change Impact?

  • Validator Node
  • Full Node (API, Indexer, etc.)
  • Move/Aptos Virtual Machine
  • Aptos Framework
  • Aptos CLI/SDK
  • Developer Infrastructure
  • Move Compiler
  • Other (specify)

Checklist

  • I have read and followed the CONTRIBUTING doc
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I identified and added all stakeholders and component owners affected by this change as reviewers
  • I tested both happy and unhappy path of the functionality
  • I have made corresponding changes to the documentation

Copy link

trunk-io bot commented Dec 12, 2024

⏱️ 54m total CI duration on this PR
Job Cumulative Duration Recent Runs
rust-cargo-deny 8m 🟩🟥🟩🟩🟩
rust-move-tests 8m 🟥
rust-move-tests 8m 🟥
rust-move-tests 8m 🟥
rust-move-tests 7m 🟥
check-dynamic-deps 7m 🟩🟩🟩🟩🟩
general-lints 3m 🟩🟩🟩🟩🟩
rust-move-tests 2m 🟥
semgrep/ci 2m 🟩🟩🟩🟩🟩
file_change_determinator 56s 🟩🟩🟩🟩🟩
permission-check 18s 🟩🟩🟩🟩🟩
permission-check 13s 🟩🟩🟩🟩🟩

🚨 1 job on the last run was significantly faster/slower than expected

Job Duration vs 7d avg Delta
check-dynamic-deps 4m 1m +197%

settingsfeedbackdocs ⋅ learn more about trunk.io

@@ -53,5 +53,8 @@ module aptos_std::any {
fun test_any() {
assert!(unpack<u64>(pack(22)) == 22, 1);
assert!(unpack<S>(pack(S { x: 22 })) == S { x: 22 }, 2);

assert!(unpack<S>(pack(S { x: 22 })) == S { x: 22 }, 2);
Copy link

Choose a reason for hiding this comment

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

Remove this line as it uses incorrect method call syntax for unpack. The assertion on line 56 already tests the same functionality with correct function call syntax.

Spotted by Graphite Reviewer (based on CI logs)

Is this helpful? React 👍 or 👎 to let us know.

@igor-aptos igor-aptos force-pushed the igor/copyable_any_map branch from 1457de6 to 6845566 Compare December 12, 2024 18:25
@igor-aptos igor-aptos changed the base branch from igor/remove_if_present to igor/btree_map December 12, 2024 18:25
@igor-aptos igor-aptos changed the title [framework] Add copyable_any_map::AnyMap [framework] Add any_map::AnyMap Dec 12, 2024
@igor-aptos igor-aptos force-pushed the igor/copyable_any_map branch from 6845566 to 3013155 Compare December 12, 2024 18:38
@igor-aptos igor-aptos force-pushed the igor/copyable_any_map branch from 3013155 to 58ebe00 Compare December 16, 2024 22:23
@igor-aptos igor-aptos force-pushed the igor/copyable_any_map branch from 58ebe00 to 6d5e10b Compare December 17, 2024 19:43
@igor-aptos igor-aptos force-pushed the igor/copyable_any_map branch from 6d5e10b to be8233a Compare January 7, 2025 20:58
@igor-aptos igor-aptos force-pushed the igor/copyable_any_map branch from be8233a to d5c471a Compare January 9, 2025 20:40
@igor-aptos igor-aptos force-pushed the igor/copyable_any_map branch from d5c471a to 44d2587 Compare January 10, 2025 06:40
@igor-aptos igor-aptos force-pushed the igor/copyable_any_map branch from 44d2587 to 2a948ea Compare January 10, 2025 07:31
@igor-aptos igor-aptos force-pushed the igor/copyable_any_map branch from 2a948ea to d311703 Compare January 10, 2025 08:35
@igor-aptos igor-aptos force-pushed the igor/copyable_any_map branch from d311703 to c292f09 Compare January 10, 2025 22:41
@igor-aptos igor-aptos force-pushed the igor/copyable_any_map branch from c292f09 to 0ff3f3d Compare January 11, 2025 04:30
@igor-aptos igor-aptos force-pushed the igor/copyable_any_map branch from 0ff3f3d to 7295cea Compare January 11, 2025 04:38
@igor-aptos igor-aptos force-pushed the igor/btree_map branch 3 times, most recently from 9b16865 to f5cf9c7 Compare January 11, 2025 08:00
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.

1 participant