Douplicates

[reza1615]

Thank you for implementing duplicates function. It is very handy.
would you please add these enhancements:

• pleases also add it to the column description pop up window
  
• In the Show duplicates, the preview part, needs a vertical scroll bar. for some cases like city column it has many results and long window
  
• Mentioning the total number of rows and remaining rows in the remove duplicates window
  
• Please make the duplicate preview as pop up window like correlations window or Column Analysis. now it is an instance window.

[aschonfeld]

Not sure I understand the last request. Correlations will always spawn a new tab and Column Analysis will spawn a modal or a popup window depending on the current dimensions of your browser.

Currently the Duplicates has the same behavior as Column Analysis. I did find a bug with when it opens popup window rather than a modal. Other than that everything seems fine.

As for clicking "Execute" theres not much wiggle room on how that opens a new instance of D-Tale. I can add a button to the main menu to "drop this instance" if you'd like?

[reza1615]

The last request:
Show duplicates > Group by ( category_val) > view duplicates >> shows some cases. > select "DDDDD" > press Exceute.
It creates a new instance like http://alphatechadmin.pythonanywhere.com/dtale/main/5 . In my opinion it should create a page like http://alphatechadmin.pythonanywhere.com/dtale/popup/describe/4 which is not instance.
Use case:
for example we have a data set with many duplicates. I want to explore each cases to know should I take care about them. so I will have many instances which are useful just for one time exploration after that they should be removed.
Auto remove instance + Back to Data button are good options other wise please make it like /dtale/popup/describe/4 as a temporary data show

[aschonfeld]

Ok, that is a much longer term change. For the time being just use the new instance for viewing and you can view the "Describe" associated with it and when you don't need it anymore simply go to the "Instances" popup and delete the instance.

[aschonfeld]

fixed in v1.18.1

[reza1615]

@aschonfeld : Thank you for efforts.
The second request isn't resolved and still there is long list

[aschonfeld]

Sorry, already have a fix for it. That will be included in the next release

[reza1615]

I checked and it doesn't have vertical scroll

[aschonfeld]

Hahaha, slow down I haven't released it yet. Hence "next release"