-
Notifications
You must be signed in to change notification settings - Fork 3
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
Updated Data Formatting #181
Comments
/cc @zohaib-a-ahmed @Massi-Papi @ethan-gray-01 @LikhitaPulijala @BollepalliPranitha @kungfuchicken It appears there are two options going forward. Does the end user wish to pursue either of these options:
|
For testing purposes, when using
|
@BollepalliPranitha Did you merge #183 by accident? |
@adhampton110 |
Could we get confirmation on the suggested inputs for this testing file from the clients? |
@adhampton110 Can you test for these sequences . AYGU-1_MeltR.csv Sample 2 Sample 3 I have provided data for monomolecular, heteroduplex, and homoduplex samples. |
I had successful uploads with Sample 1 & Sample 2. I ran into this error message using Sample 3. Taking a look at the MeltR source code, I am hitting this error. I am wondering if we are to specify the coefficients, as opposed to specifying the sequence |
@adhampton110 |
@adhampton110 Have you tried using any other sequence for Monomolecular |
@BollepalliPranitha I have not, which is why I have been attempting to interact with the MeltR package directly within the R console. With the specified inputs you gave me, if we aren't able to create a meltR.A object when directly interacting with the package, then we also are not going to be able to create a meltR.A object with our application code. Our app code just calls the meltR.A function. |
@adhampton110 |
@BollepalliPranitha I am unable to reproduce their successful creation. Have you attempted to create the meltR.A object with the inputs from the screenshot? The screenshot contains a syntax error, around the |
@adhampton110 I think there are so many outliers present in the data provided that should be manually handled. I am in discussing this with client. The issue persists with the data and Sequence. |
This issue is related to the newly created issue #185 |
Clients introduced a new data format that MeltShiny is going to operate from named as AYGU-1_MeltR that should be used going forward.
This will require rewriting some buried architecture regarding formatting the data going into the MeltR package, which is currently throwing internal errors (within MeltR package). Something is incorrectly being passed into the library post data upload.
This bug needs to be addressed before the application can be used with the new csv format.
The text was updated successfully, but these errors were encountered: