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

Move Trace Debugger Bug #20727

Open
Vadim-Kolchanov opened this issue Dec 24, 2024 · 7 comments
Open

Move Trace Debugger Bug #20727

Vadim-Kolchanov opened this issue Dec 24, 2024 · 7 comments
Assignees

Comments

@Vadim-Kolchanov
Copy link

I want to debug the code by running tests in VS code. I use Move Trace Debugger extension .

  1. Execute command sui move test --trace-execution. Folder "traces" was created successfully
image
  1. Open the required file with tests
  2. Run -> Start Debugging
  3. Select a test function
image

Expected Result

The debugger will start successfully

Actual Result

Get an error:
image

System Information

  • OS: Windows 11, WSL Ubuntu
  • Sui ver: sui 1.39.3-5d4350d66d2b
  • Sui move ver: sui-move 1.39.3-5d4350d66d2b
  • Edition: 2024.beta
@stefan-mysten
Copy link
Contributor

@Vadim-Kolchanov thanks for flagging this!

@stefan-mysten
Copy link
Contributor

@awelc would you have any idea what might be wrong?

@awelc
Copy link
Contributor

awelc commented Dec 24, 2024

@Vadim-Kolchanov, thank you for the report indeed! I think I know what the problem is but it would be great if you could share your code so that I can confirm my suspicion, create a fix, and produce tests that will ensure that this will not happen in the future

@Vadim-Kolchanov
Copy link
Author

@awelc, I can't share the source code, because it's still in development. But I created a separate project (with the entire structure preserved), where the error is reproduced. I hope this will help you.
https://github.com/Vadim-Kolchanov/bug_report

@awelc
Copy link
Contributor

awelc commented Dec 27, 2024

@awelc, I can't share the source code, because it's still in development. But I created a separate project (with the entire structure preserved), where the error is reproduced. I hope this will help you. https://github.com/Vadim-Kolchanov/bug_report

Thank you! That helped a lot! The fix is in the works and hopefully a new version will soon land in the VSCode marketplace (will keep you posted)

@Vadim-Kolchanov
Copy link
Author

@awelc Thank you! I see you've done a great job

@awelc
Copy link
Contributor

awelc commented Dec 28, 2024

@awelc Thank you! I see you've done a great job

The new version of the extension has been published, though admittedly support for dynamic fields is somewhat lacking. Will work on trying to improve it shortly, though!

awelc added a commit that referenced this issue Dec 28, 2024
## Description 

This PR fixes trace ingestion for the case when global location is used
in a write to address a problem reported in
#20727

## Test plan 

Added a new test (all new and old tests must pass)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants