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

[playback] Playback failing says existing column is missing #2102

Closed
frosencrantz opened this issue Nov 3, 2023 · 4 comments
Closed

[playback] Playback failing says existing column is missing #2102

frosencrantz opened this issue Nov 3, 2023 · 4 comments

Comments

@frosencrantz
Copy link
Contributor

Small description
Playback failing says existing column is missing

Expected result
No errors due to columns that exist

Actual result with screenshot

https://asciinema.org/a/4Kop1pUil70qoHUWMpoCfvvBW

Using sample_data/benchmark.csv

No error other than in the status log:

  • no "Date" Column on benchmark
  • visidata/cmdlog.py:263:moveToReplayContext()

Steps to reproduce with sample data and a .vd

vd -p /tmp/benchmark_cmdlog.vdj  sample_data/benchmark.csv

cat /tmp/benchmark_cmdlog.vdj

#!vd -p
{"sheet": "benchmark", "col": "Date", "row": "", "longname": "type-date", "input": "", "keystrokes": "@", "comment": "set type of current column to date"}
{"sheet": "benchmark", "col": "Quantity", "row": "", "longname": "type-int", "input": "", "keystrokes": "#", "comment": "set type of current column to int"}
{"sheet": "benchmark", "col": "Unit", "row": "", "longname": "type-currency", "input": "", "keystrokes": "$", "comment": "set type of current column to currency"}
{"sheet": "benchmark", "col": "Paid", "row": "", "longname": "type-currency", "input": "", "keystrokes": "$", "comment": "set type of current column to currency"}

Additional context
Please include the version of VisiData and Python. Latest develop and Python 3.9.2

I will note that on different hosts I see the behavior, in slightly different ways. Sometimes I need to pipe the input on STDIN to see it. But in example here it did not require it. But if you have difficulty reproducing then it could be related to some sort of timing.

@saulpw
Copy link
Owner

saulpw commented Nov 3, 2023

Thanks @frosencrantz. moveToReplayContext needed to make sure the sheet was loaded before moving the cursor. Should be fixed now.

@saulpw saulpw closed this as completed Nov 3, 2023
@frosencrantz
Copy link
Contributor Author

Thank you again, Saul! It looks like it worked.

@reagle
Copy link
Contributor

reagle commented Jun 28, 2024

I'm having a similar problem of it choking on a column name during playback. Should this be reopened or should I create a new issue?

In the attached I'm trying to create a macro that will show Reddit dumps consistently. My goal is to hide all columns but 5 and create an iso column.

https://asciinema.org/a/sKvTyL5MfBqdeZCE5t9xTpOTo

m
C
| ^(author|created_utc|selftext|title|url)$
gt
c width
ge 0
q
c created
= e2iso(created_utc)
^ created
h
-
m

@saulpw
Copy link
Owner

saulpw commented Jun 29, 2024

@reagle Please open a new issue, thank you! [I did for you, #2435]

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

4 participants