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
Remove duplicates when refilling readline history #1000
Comments
Are you saying that the way to fix this is to put duplicate lines in the history, but not refill readline's? Isn't it better to simply not add them in the first place? |
Our history is supposed to be a record of what you've executed - for instance, running |
Right - we do not want to de-dupe the actual record of your actions, because then it is no longer accurate, and breaks reproducibility. We only want to de-dupe readline, so that we skip repeating entries when hitting the up arrow, etc. |
Agreed. In fact, it used to work without the dupes a while back, I'm not exactly sure when it changed. Was it when we introduced the sqlite backend or later, do you guys know? |
I don't think I accounted for it when I did the sqlite stuff initially, so my guess would be then. I can reproduce it in 0.11. PR coming up in a few minutes. |
Closed by #1003, I forgot to put that in the commit comment. |
Closes ipythongh-1000, earns t-shirt
Closes ipythongh-1000, earns t-shirt
I see duplicates when going back in history in 3.0.0 (QtConsole), it is not something that can be reproduced with confidence but anyone using IPython for long enough time should have noticed this. Maybe QtConsole is not used as much and thus not get tested like the Notebook? |
Readline by default only keeps one copy of consecutive duplicates in history, but when we refill it from the history database, we push those duplicates multiple times. It should be an easy fix in
refill_readline_hist
.The text was updated successfully, but these errors were encountered: