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.

Sign up for GitHub

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

Jump to bottom

Limit of analysis to be copied #241

Open
Felipe-UnB opened this issue Nov 20, 2015 · 9 comments · Fixed by #244
Open

Limit of analysis to be copied #241

Felipe-UnB opened this issue Nov 20, 2015 · 9 comments · Fixed by #244
Assignees
@johnzeringue
Labels
bug
Milestone
v1.0.0

Comments

@Felipe-UnB
Copy link

Heys guys, how are you? So, I was trying to copy a lot of results from excel to TOPSOIL (exactly 330 analysis). However, when I try to paste from the clipboard, there is a limit of 239 analysis that can be copied. Is this a real limit or it might be some kind of bug? I am using Windows 10 Home and Microsoft Office Professional Plus 2010.
@johnzeringue
Copy link
Contributor

We've yet to see this sort of thing before. If it's a problem with Topsoil, it's certainly not intentional. This could also be an issue with Windows, Excel, or your data. Are you able to send me the file you're having trouble with? If I'm able to reproduce the issue, I can start working on a fix or a workaround.

@johnzeringue johnzeringue self-assigned this Nov 22, 2015
@Felipe-UnB
Copy link
Author

Hey John, I will prepare the file ASAP!

@Felipe-UnB
Copy link
Author

Resultados 91500 - TOPSOIL issue.xlsx

@Felipe-UnB
Copy link
Author

I don´t believe the problem is actually with the data because I was able to plot it using Isoplot. But let´s see what you find out, maybe Isoplot is handling some kind of error in a way that It´s not obvious to me.

@johnzeringue johnzeringue added this to the v1.0.0 milestone Nov 23, 2015
@johnzeringue
Copy link
Contributor

I've found the issue and will have a fix out soon. @Felipe-UnB, thanks for reporting this.

johnzeringue added a commit to johnzeringue/Topsoil that referenced this issue Nov 23, 2015
@johnzeringue
Fixed DSV reader bug
2583e9f 
Fixed CIRDLES#241 by fixing a bug where blank lines (even trailing) would cause
the DSV reader to think the table wasn't square. Now, all blank lines
are ignored, allowing even:

```

A B C

1 2 3

4 5 6

```
@johnzeringue johnzeringue mentioned this issue Nov 23, 2015
Merged
@Felipe-UnB
Copy link
Author

John, the bug was actually related to the number of analyses?

@johnzeringue
Copy link
Contributor

Not the one I found, but I could reproduce the copy-paste failing. After that, everything worked fine. Are you still having issues?

@Felipe-UnB
Copy link
Author

The version with the bug correction for this issue is the v1.0.0-alpha.6, correct? If that's right, I am still experiencing the same problem. I will try exporting the data as CSV and see if I am able to plot all of them.

@johnzeringue
Copy link
Contributor

I'll try again to reproduce and get back to you.

@johnzeringue johnzeringue reopened this Nov 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@johnzeringue johnzeringue

Labels
bug
Projects
None yet
Milestone
v1.0.0
Development

Successfully merging a pull request may close this issue.

Fixed DSV reader bug johnzeringue/Topsoil
2 participants
@johnzeringue @Felipe-UnB