Pandora-IsoMemo / ReSources

Food Reconstruction Using Isotopic Transferred Signals (ReSources)
https://isomemoapp.com/app/resources
GNU General Public License v3.0
0 stars 1 forks source link

Issue with Copy-pasting in Target & target-to-source offsets table #99

Closed CarloCocozza closed 11 months ago

CarloCocozza commented 1 year ago

Pergamon Roman.zip

For the attached model, the software crashes when new data is copy-pasted in Target & target-to-source offsets table. This happens in all versions online and local.

@arunge

arunge commented 1 year ago

@CarloCocozza Could you provide more details on this? I did the following which was working without any issues:

  1. Importing your data.
  2. Copying a table (1) into a table calculation
copy paste
image image
  1. Changing some values, copying the table and pasting it into the app
change & copy paste back
image image

The app did not crash, I tested this on Resources main online. I tested with Chrome and with Firefox.

Did you try something else that lead to the crash?

CarloCocozza commented 1 year ago

hi @arunge

I sent this in a rush and I did not explained it properly. I meant when I copy-paste different data from an external file (in my case excel file), using ctrl-V.

arunge commented 1 year ago

Ok, now I could identify a problem that happens when the number of rows to be inserted is smaller than the number of rows that is already in the table. I am looking for a fix.

arunge commented 1 year ago

With version 23.07.2 on Resources beta I fixed the above bug that stopped the app working.

Now, one can paste a table into the target table, even if it contains targets that were not present in the table before. (I used your example data.)

image

However, what should be the standard behavior for Sources and Concentration tables? @isomemo @CarloCocozza Should row and column names be kept after pasting a new target table? This is currently the case in version 23.07.2.

Sources table Concentrations table
image image
arunge commented 11 months ago

As just discussed, the current default behavior is fine and I am closing here.