Hotfix read fullscan xml sift#2
Open
Heisenberg4322 wants to merge 55 commits intobeckywagner4:mainfrom
Open
Conversation
Each table in read_sift is now wrapped in a tryCatch - this is so that the use of `read_many_sift()` can be improved. If the new `warn` argument is set to TRUE, the first table to fail to be read will produce a warning, and the function will return NULL. If it is FALSE (the default for `read_sift()`, an error will be thrown instead. `read_many_sift()` and `read_many_sift2()` set `warn = TRUE` so a bad file will not disrupt the reading of a whole folder. They have been adjusted to handle NULLs while binding the data.
Add error catching for read_sift
fixed bug in read_proc_sift
New function to add 'data' column from 'start_time' and 'time_s' or 'time_ms' to both raw SIM/fullscan and processed SIM SIFT data. Includes options to filter by phase and to force timezone.
Create add_date_sift.R
Added blank line after @export. Function previously would not appear properly in R.
Minor bug fix
minor bug fix number 2
Update add_date_sift.R
Create read_fullscan_sift.R
document update
Hotfix add date sift 3
update to descriptions and parameters. Added warning if timezone not specified. updated devtools::document() files.
add_date_sift update
Update to read_proc_sift to enable reading in processed sift files containing multiple of the three tables (with numerous start times). Enables faster processing through labsyft.
Update read_proc_sift.R
renamed column "numbers" to "CAS_number"
Update read_proc_sift.R
Update read_proc_sift.R
This function reads a processed SIFT .csv file (which contains multiple tables such as analyte conc, conc per reagent, conc per product and their raw equiavlents) over mutliple start times to create a single data frame with four columns: table, time_s, compound_identity and conc. Subsequent functions will be produced to filter and expand the columns of these tables.
Replaced the read_proc_sift function (which was malfunctioning)
New function to process processed SIFT data output from read_proc_sift
Update filter_proc_sift.R
documentation update
added warning to catch and remove duplicate data
…e_sift Feature - new function to validate fullscan SIFT data as quantitative
…e_sift Hotfix validate as quanititative sift
…_sift-2 Update validate_as_quantitative_sift.R
…_sift-3 Update validate_as_quantitative_sift.R
new functions for reading fullscan xml files
function update
adds column reagent_mass
Update read_fullscan_xml_sift.R
Update filter_proc_sift.R
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.