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.

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

v1.2.0 #12

Merged
merged 26 commits into from
Jul 20, 2022
Merged

v1.2.0 #12

merged 26 commits into from
Jul 20, 2022

Conversation

daynefiler
Copy link
Member

No description provided.

daynefiler and others added 26 commits February 18, 2022 16:17
Added the txpExportGuifunciton. In doing so, added a new txpValueNames method for TxpModel objects. Also functionalized some of the work
within txpCalculateScores, as it would have been duplicated by the
export function.

Still need tests written for txpExportGui.
issue-01: added txpExportGui
issue-08: require R>4.0
The previous version included the param line 'ref = dev', which did not work on my machine. I replaced it with the address 'ToxPi/toxpiR@dev' in the change above. If this is not a general solution, we'll need to change it back and debug.
Added note on negative.value.handling
added comment on negative.value.handling
Merging latest version of introduction.Rmd made on origin/issue-4 with rest of most current updates on /dev
Merging latest version of introduction.Rmd made on origin/issue-4 with rest of most current updates on /dev
Copy link
Member

@davidmreif davidmreif left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me! I just did one more rebuild from /dev to my local machine, and everything seems to be in order.

Will the .io site automatically add the new vignette into the "Articles" dropdown when updated?

@daynefiler daynefiler merged commit 1f4a97e into main Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants