You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@daffodil.apache.org by "lrbarber (via GitHub)" <gi...@apache.org> on 2024/03/08 16:58:52 UTC
[I] Infoset diff view does not work [daffodil-vscode]
lrbarber opened a new issue, #986:
URL: https://github.com/apache/daffodil-vscode/issues/986
Not Sure where this feature came from or when it stopped working.
Inforset file that can be displayed is a temp file....
Solution might be to just remove this option.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Infoset diff view does not work [daffodil-vscode]
Posted by "hdalsania (via GitHub)" <gi...@apache.org>.
hdalsania commented on issue #986:
URL: https://github.com/apache/daffodil-vscode/issues/986#issuecomment-2005655091
After further testing, this feature works as intended in Linux platform. This seems to be an pathing issue with Windows. @shanedell could you please test in MacOS and see if we can expand portability functionality for this?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Infoset diff view does not work [daffodil-vscode]
Posted by "shanedell (via GitHub)" <gi...@apache.org>.
shanedell commented on issue #986:
URL: https://github.com/apache/daffodil-vscode/issues/986#issuecomment-2007397618
This working for me on MacOS. I will test on a windows VM and see if I can figure out the issue
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Infoset diff view does not work [daffodil-vscode]
Posted by "scholarsmate (via GitHub)" <gi...@apache.org>.
scholarsmate closed issue #986: Infoset diff view does not work
URL: https://github.com/apache/daffodil-vscode/issues/986
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Infoset diff view does not work [daffodil-vscode]
Posted by "scholarsmate (via GitHub)" <gi...@apache.org>.
scholarsmate closed issue #986: Infoset diff view does not work
URL: https://github.com/apache/daffodil-vscode/issues/986
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Infoset diff view does not work [daffodil-vscode]
Posted by "hdalsania (via GitHub)" <gi...@apache.org>.
hdalsania commented on issue #986:
URL: https://github.com/apache/daffodil-vscode/issues/986#issuecomment-2007435278
@shanedell This was the issue only with Windows and I found out that Uri.Parse function used within vscode.commands.executeCommand VSCode API function was encoding the path string and replacing "\" with "%5C" encoding. By replacing Uri.Parse with Uri.file in infoset.ts for infoset diff functionality fixes the problem for windows. It works in both Windows and Linux.
@lrbarber will update the inforset.ts with following change in his existing PR #992 and push his changes soon.
ctx.subscriptions.push(vscode.commands.registerCommand('infoset.diff', async () => {
if(sid !== undefined){
let path = ensure(tmp(sid));
let prev = ensure(`${path}.prev`);
vscode.commands.executeCommand ('vscode.diff', Uri.file(prev), Uri.file(path), "Previous ↔ Current");
}
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Infoset diff view does not work [daffodil-vscode]
Posted by "lrbarber (via GitHub)" <gi...@apache.org>.
lrbarber commented on issue #986:
URL: https://github.com/apache/daffodil-vscode/issues/986#issuecomment-2004650029
The code added from the that jw3 added 3 years ago (src/infoset.ts) appears to be unchanged:
```
ctx.subscriptions.push(vscode.commands.registerCommand('infoset.diff', async () => {
if(sid !== undefined){
let path = ensure(tmp(sid));
let prev = ensure(`${path}.prev`);
vscode.commands.executeCommand ('vscode.diff', Uri.parse(prev), Uri.parse(path), "Previous ↔ Current");
}
```
I wonder if this is as simple as someone removing the code that used to create the .prev file.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Infoset diff view does not work [daffodil-vscode]
Posted by "scholarsmate (via GitHub)" <gi...@apache.org>.
scholarsmate commented on issue #986:
URL: https://github.com/apache/daffodil-vscode/issues/986#issuecomment-1986089557
@jw3, In your opinion, is this a feature we should try to retain, if so, how is it supposed to work and what problem(s) does it solve?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Infoset diff view does not work [daffodil-vscode]
Posted by "lrbarber (via GitHub)" <gi...@apache.org>.
lrbarber commented on issue #986:
URL: https://github.com/apache/daffodil-vscode/issues/986#issuecomment-1986131997
This does not seem like something that would need to be automatically opened upon entering the debugger.
From: John Wass ***@***.***>
Sent: Friday, March 8, 2024 12:34 PM
To: apache/daffodil-vscode ***@***.***>
Cc: Larry Barber ***@***.***>; Author ***@***.***>
Subject: Re: [apache/daffodil-vscode] Infoset diff view does not work (Issue #986)
image.png (view on web)<https://github.com/apache/daffodil-vscode/assets/1545372/15bb8946-f14b-4213-b03a-e8381176946f>
how is it supposed to work
Open the diff view, step through a debug, observing the colorized diff that compares the current infoset against the previous step/breakpoint infoset. The intent was to quickly identify changes a span of execution made to the infoset.
In your opinion, is this a feature we should try to retain
If viewing infoset changes as a colorized diff that updates as you step through a debug is useful.
* jw3/example-daffodil-vscode#66<https://github.com/jw3/example-daffodil-vscode/pull/66>
* gif from the old wiki<https://github.com/jw3/example-daffodil-vscode/wiki#infoset-tools>
-
Reply to this email directly, view it on GitHub<https://github.com/apache/daffodil-vscode/issues/986#issuecomment-1986120382>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ARJT6NEQWKWJWHCDKTYZNVLYXHY63AVCNFSM6AAAAABENEBPVSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBWGEZDAMZYGI>.
You are receiving this because you authored the thread.Message ID: ***@***.******@***.***>>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Infoset diff view does not work [daffodil-vscode]
Posted by "lrbarber (via GitHub)" <gi...@apache.org>.
lrbarber commented on issue #986:
URL: https://github.com/apache/daffodil-vscode/issues/986#issuecomment-2004416436
Confirmed that even when invoked as originally intended and shown in the gif referenced in comment by jw3 above, the infoset diff viewer does not work and displays the same error shown at the top of this issue.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Infoset diff view does not work [daffodil-vscode]
Posted by "jw3 (via GitHub)" <gi...@apache.org>.
jw3 commented on issue #986:
URL: https://github.com/apache/daffodil-vscode/issues/986#issuecomment-1986120382
> how is it supposed to work
Open the diff view, step through a debug, observing the colorized diff that compares the current infoset against the previous step/breakpoint infoset. The intent was to quickly identify changes a span of execution made to the infoset.
> In your opinion, is this a feature we should try to retain
If viewing infoset changes as a colorized diff that updates as you step through a debug is useful.
- https://github.com/jw3/example-daffodil-vscode/pull/66
- [gif from the old wiki](https://github.com/jw3/example-daffodil-vscode/wiki#infoset-tools)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Infoset diff view does not work [daffodil-vscode]
Posted by "jw3 (via GitHub)" <gi...@apache.org>.
jw3 commented on issue #986:
URL: https://github.com/apache/daffodil-vscode/issues/986#issuecomment-1986139724
> This does not seem like something that would need to be automatically opened upon entering the debugger.
It was not originally, you can see the flow on the gif from the old wiki
https://github.com/jw3/example-daffodil-vscode/wiki#infoset-tools
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org