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

Devel #80

Merged
merged 294 commits into from
Mar 5, 2024
Merged

Devel #80

merged 294 commits into from
Mar 5, 2024

Conversation

kamilsi
Copy link
Contributor

@kamilsi kamilsi commented Mar 5, 2024

No description provided.

Laura Bąkała and others added 30 commits November 17, 2023 12:01
recompile package
7bbb444
fix setup skip
734f655
update NEWS
9744f0b
setup pkgdown
f050380
log args too
8a38475
implement study list and details
0a34f9d
document new functions
4fb887d
increase version
8db4afe
inject variables
9cb740f
return study ID upon insert
bb2001c
fix parameter added back
dcf4f01
increment DB schema version
e271d99
@aleksandraduda2
add tests for simple randomization: one-sample prop tests for two and…
26050ed 
… more groups with equal and unequal ratio
@aleksandraduda2
correct number of patients
1d7c568
@kamilsi
moved API to standard location, bumped up renv to R 4.2.3, removed "h…
c91fd59 
…ello world", still WIP
@kamilsi
removed dummy function
2270a58
@kamilsi
Merge pull request #17 from ttscience/feat/simple
bf4e465 
Non-functional simple randomization
@kamilsi
Removed dependency on 7yo dockerfile, added our own; fixed API path i…
f85c8f1 
…n test setup
@kamilsi
fixed dockerfile dependencies
e1ef4ea
@kamilsi
bugfixed test startup
8c41b49
@kamilsi
Fixed R version, API launch command
2f059e8
@kamilsi
removed redundant test
705edb5
@kamilsi
added comments to SQL, added additional DB tests
e176ccd
@kamilsi
bugfix
34a3bc8
@kamilsi
bugifx
16d6d2b
@kamilsi
bugfixed tests
f79c274
@kamilsi
Merge pull request #19 from ttscience/feat/study-def
244d3b9 
Study definition endpoint
@kamilsi
resolved merge conflicts
329fbb2 
Merge branch 'feat/db' into feat/study-list

# Conflicts:
#	Dockerfile
#	inst/plumber/unbiased_api/plumber.R
#	inst/postgres/00-metadata.sql
#	inst/postgres/01-initialize.sql
@kamilsi
Merge pull request #20 from ttscience/feat/study-list
6014329 
Stubs of endpoints, DB and tests, logging of requests
@kamilsi
WIP: core function for dynamic randomization, added usethat and style…
97e8cd2 
…r to packages, tests (stub)
lwalejko and others added 18 commits March 1, 2024 15:17
@lwalejko
Add invalid JSON handling to error-handling.R
976e86f
@JagGlo
Merge pull request #78 from ttscience/vignette-simulation
a44f940 
Issue #77: Changed in age: >55 instead >50
@lwalejko
simplyfy JSON rendering in audit trail get endpoint
fc1b497
@lwalejko
rename methods and simplify checks
377e4ce
@lwalejko
Update documentation
b82633a
@lwalejko
Add IP address and user agent to audit log
712b54f
@lwalejko
Update documentation
4a171c7
@lwalejko
Remove unnecessary JSON validation
604d419
@lwalejko
test for malformed request
0c88cc6
@lwalejko
Merge branch '52-randomization-audit-trail' of https://github.com/tts…
bb61fa3 
…cience/unbiased into 52-randomization-audit-trail
@lwalejko
Add IP address and user agent to audit logs api
72c0a3a
@lwalejko
Update pkgdown build_site_github_pages install parameter
dc0a5c1
@lwalejko
Add setup-renv action for package dependencies
4856c17
@lwalejko
update packages and restore renv in pkgdown workflow
44c626f
@lwalejko
Update pkgdown.yaml workflow
cd20816
@lwalejko
Update Dockerfile to use devtools for package installation
3c7957e
@lwalejko
disable code coverage for default_error_handler
8c34747
@kamilsi
Merge pull request #60 from ttscience/52-randomization-audit-trail
85919a4 
Audit trial
@kamilsi kamilsi added this to the Release 1 milestone Mar 5, 2024
@kamilsi kamilsi linked an issue Mar 5, 2024 that may be closed by this pull request
Release housekeeping #54
Closed
@codecov Codecov
Copy link

codecov bot commented Mar 5, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 99.84779% with 1 lines in your changes are missing coverage. Please review.

❗ No coverage uploaded for pull request base (main@ed1e66e). Click here to learn what that means.

Files Patch % Lines
R/audit-trail.R 98.91% 1 Missing ⚠️
Additional details and impacted files 
@@           Coverage Diff           @@
##             main      #80   +/-   ##
=======================================
  Coverage        ?   99.84%           
=======================================
  Files           ?       11           
  Lines           ?      657           
  Branches        ?        0           
=======================================
  Hits            ?      656           
  Misses          ?        1           
  Partials        ?        0

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

lwalejko and others added 3 commits March 5, 2024 13:33
@lwalejko
Update version number and API version
e8b448c
@kamilsi
Merge pull request #79 from ttscience/fix-pkgdown
df8b7d8 
Fix pkgdown
@kamilsi
Merge pull request #75 from ttscience/71-untested-custom_range-method…
ce589e4 
…-in-randomization-function

Issue #71 added tests custom range method in randomization function
@salatak salatak self-requested a review March 5, 2024 13:48
@kamilsi kamilsi merged commit 3aabf67 into main Mar 5, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@salatak salatak salatak approved these changes

@lwalejko lwalejko lwalejko approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Release 1
Development

Successfully merging this pull request may close these issues.

Release housekeeping
5 participants
@kamilsi @lwalejko @JagGlo @salatak @aleksandraduda2