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

Dirichlet concentration parameter is now actually being used #315

Merged
merged 2 commits into from
Oct 4, 2023

Conversation

osorensen
Copy link
Collaborator

This should fix #313.

Copy link
Member

@wleoncio wleoncio left a comment

Choose a reason for hiding this comment

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

Hurray for Open Science! :)

@wleoncio wleoncio linked an issue Oct 4, 2023 that may be closed by this pull request
@osorensen osorensen merged commit ec442b3 into develop Oct 4, 2023
14 checks passed
osorensen added a commit that referenced this pull request Oct 4, 2023
* Removed duplicated output element (#291)

* Removed duplicated output element

* Updated package version and NEWS

* Should fix #292 (#293)

* Updated NEWS and increment version

* Updated citation info in README

* Should fix #299 (#300)

* Should fix #299

* Added missing documentation objects and changed the name of one argument

* Fixed another documentation issue

* Removed deprecated SMC functions (#303)

* Updated build version number

* Increment version number to 1.3.2.9002

* Removed deprecated SMC functions (#301)

* Updated NEWS (#301)

* Logz issue 158 (#304)

* Added exact partition function option

* Updated tests

* Updated SMC Mallows examples

* Styling

* More linting

* Updated linter workflow

Some linter were deprecated, ignored line on vignette needed update

* Trying to please CodeFactor

* Incremented version number and added entry to NEWS.md

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Added BugReport URL (#308)

* Added URL for bug reports

* Increment version number to 1.3.2.9004

* Small typo correction and documentation update

* Website (#310)

* Website

* De-duplicating vignettes

* Adding GH Action for pkgdown (#290)

Literally the output of `usethis::use_github_action("pkgdown")`

* Triggering `gh-pages` when pushing to PR #310 (tests)

This is just for testing the online deployment and should be removed after merge (or just before).

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Small changes (#311)

* Small changes

* Fixed gh-pages bases for deployment (#290)

The PR branch was left over. More importantly, the file listed "main" instead of "master".

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Corrected small typo in documentation

* Cleaner unit test output (#314)

* Increment version number to 1.3.2.9006

* Capturing output of test unit

* Dirichlet concentration parameter is now actually being used (#315)

* forwarded psi argument to run_mcmc

* Incremented version and updated news.md

* ready for cran submission

* CRAN wanted a trailing slash in the URL to the website, since it becomes added automatically if one follows the URL

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>
@wleoncio wleoncio deleted the psi-issue-313 branch October 4, 2023 07:47
osorensen added a commit that referenced this pull request Oct 5, 2023
* Removed duplicated output element (#291)

* Removed duplicated output element

* Updated package version and NEWS

* Should fix #292 (#293)

* Updated NEWS and increment version

* Updated citation info in README

* Should fix #299 (#300)

* Should fix #299

* Added missing documentation objects and changed the name of one argument

* Fixed another documentation issue

* Removed deprecated SMC functions (#303)

* Updated build version number

* Increment version number to 1.3.2.9002

* Removed deprecated SMC functions (#301)

* Updated NEWS (#301)

* Logz issue 158 (#304)

* Added exact partition function option

* Updated tests

* Updated SMC Mallows examples

* Styling

* More linting

* Updated linter workflow

Some linter were deprecated, ignored line on vignette needed update

* Trying to please CodeFactor

* Incremented version number and added entry to NEWS.md

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Added BugReport URL (#308)

* Added URL for bug reports

* Increment version number to 1.3.2.9004

* Small typo correction and documentation update

* Website (#310)

* Website

* De-duplicating vignettes

* Adding GH Action for pkgdown (#290)

Literally the output of `usethis::use_github_action("pkgdown")`

* Triggering `gh-pages` when pushing to PR #310 (tests)

This is just for testing the online deployment and should be removed after merge (or just before).

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Small changes (#311)

* Small changes

* Fixed gh-pages bases for deployment (#290)

The PR branch was left over. More importantly, the file listed "main" instead of "master".

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Corrected small typo in documentation

* Cleaner unit test output (#314)

* Increment version number to 1.3.2.9006

* Capturing output of test unit

* Dirichlet concentration parameter is now actually being used (#315)

* forwarded psi argument to run_mcmc

* Incremented version and updated news.md

* ready for cran submission

* CRAN wanted a trailing slash in the URL to the website, since it becomes added automatically if one follows the URL

* Internalized a bunch of SMC functions and made families #312

* incrementing development versions

* updating news.md

* linting

* fixed most of the thigns

* incremented dev version

* Fixed lint issues

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>
osorensen added a commit that referenced this pull request Oct 5, 2023
* Removed duplicated output element (#291)

* Removed duplicated output element

* Updated package version and NEWS

* Should fix #292 (#293)

* Updated NEWS and increment version

* Updated citation info in README

* Should fix #299 (#300)

* Should fix #299

* Added missing documentation objects and changed the name of one argument

* Fixed another documentation issue

* Removed deprecated SMC functions (#303)

* Updated build version number

* Increment version number to 1.3.2.9002

* Removed deprecated SMC functions (#301)

* Updated NEWS (#301)

* Logz issue 158 (#304)

* Added exact partition function option

* Updated tests

* Updated SMC Mallows examples

* Styling

* More linting

* Updated linter workflow

Some linter were deprecated, ignored line on vignette needed update

* Trying to please CodeFactor

* Incremented version number and added entry to NEWS.md

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Added BugReport URL (#308)

* Added URL for bug reports

* Increment version number to 1.3.2.9004

* Small typo correction and documentation update

* Website (#310)

* Website

* De-duplicating vignettes

* Adding GH Action for pkgdown (#290)

Literally the output of `usethis::use_github_action("pkgdown")`

* Triggering `gh-pages` when pushing to PR #310 (tests)

This is just for testing the online deployment and should be removed after merge (or just before).

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Small changes (#311)

* Small changes

* Fixed gh-pages bases for deployment (#290)

The PR branch was left over. More importantly, the file listed "main" instead of "master".

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Corrected small typo in documentation

* Cleaner unit test output (#314)

* Increment version number to 1.3.2.9006

* Capturing output of test unit

* Dirichlet concentration parameter is now actually being used (#315)

* forwarded psi argument to run_mcmc

* Incremented version and updated news.md

* ready for cran submission

* CRAN wanted a trailing slash in the URL to the website, since it becomes added automatically if one follows the URL

* Internalized a bunch of SMC functions and made families #312

* incrementing development versions

* updating news.md

* linting

* fixed most of the thigns

* incremented dev version

* Fixed lint issues

* removing tagging of Anja as author since she is listed in DESCRIPTION

* lots of work in order to clean up compute_consensus

* linting

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>
osorensen added a commit that referenced this pull request Oct 12, 2023
* Removed duplicated output element (#291)

* Removed duplicated output element

* Updated package version and NEWS

* Should fix #292 (#293)

* Updated NEWS and increment version

* Updated citation info in README

* Should fix #299 (#300)

* Should fix #299

* Added missing documentation objects and changed the name of one argument

* Fixed another documentation issue

* Removed deprecated SMC functions (#303)

* Updated build version number

* Increment version number to 1.3.2.9002

* Removed deprecated SMC functions (#301)

* Updated NEWS (#301)

* Logz issue 158 (#304)

* Added exact partition function option

* Updated tests

* Updated SMC Mallows examples

* Styling

* More linting

* Updated linter workflow

Some linter were deprecated, ignored line on vignette needed update

* Trying to please CodeFactor

* Incremented version number and added entry to NEWS.md

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Added BugReport URL (#308)

* Added URL for bug reports

* Increment version number to 1.3.2.9004

* Small typo correction and documentation update

* Website (#310)

* Website

* De-duplicating vignettes

* Adding GH Action for pkgdown (#290)

Literally the output of `usethis::use_github_action("pkgdown")`

* Triggering `gh-pages` when pushing to PR #310 (tests)

This is just for testing the online deployment and should be removed after merge (or just before).

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Small changes (#311)

* Small changes

* Fixed gh-pages bases for deployment (#290)

The PR branch was left over. More importantly, the file listed "main" instead of "master".

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Corrected small typo in documentation

* Cleaner unit test output (#314)

* Increment version number to 1.3.2.9006

* Capturing output of test unit

* Dirichlet concentration parameter is now actually being used (#315)

* forwarded psi argument to run_mcmc

* Incremented version and updated news.md

* ready for cran submission

* CRAN wanted a trailing slash in the URL to the website, since it becomes added automatically if one follows the URL

* Internalized a bunch of SMC functions and made families #312

* incrementing development versions

* updating news.md

* linting

* fixed most of the thigns

* incremented dev version

* Fixed lint issues

* removing tagging of Anja as author since she is listed in DESCRIPTION

* lots of work in order to clean up compute_consensus

* linting

* got rid of annoying row names in posterior interval dataframe

* made a version with parallel chains that is actually working

* close to a working version with parallel chains

* first iteration

* styling and updating style rules

* attempt at refactoring

* getting there

* styling

* fixing initial value hickup

* styling

* Parallel issue 307 (#320)

* made a version with parallel chains that is actually working

* close to a working version with parallel chains

* first iteration

* styling and updating style rules

* attempt at refactoring

* getting there

* styling

* fixing initial value hickup

* styling

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>
osorensen added a commit that referenced this pull request Oct 13, 2023
#324)

* Removed duplicated output element (#291)

* Removed duplicated output element

* Updated package version and NEWS

* Should fix #292 (#293)

* Updated NEWS and increment version

* Updated citation info in README

* Should fix #299 (#300)

* Should fix #299

* Added missing documentation objects and changed the name of one argument

* Fixed another documentation issue

* Removed deprecated SMC functions (#303)

* Updated build version number

* Increment version number to 1.3.2.9002

* Removed deprecated SMC functions (#301)

* Updated NEWS (#301)

* Logz issue 158 (#304)

* Added exact partition function option

* Updated tests

* Updated SMC Mallows examples

* Styling

* More linting

* Updated linter workflow

Some linter were deprecated, ignored line on vignette needed update

* Trying to please CodeFactor

* Incremented version number and added entry to NEWS.md

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Added BugReport URL (#308)

* Added URL for bug reports

* Increment version number to 1.3.2.9004

* Small typo correction and documentation update

* Website (#310)

* Website

* De-duplicating vignettes

* Adding GH Action for pkgdown (#290)

Literally the output of `usethis::use_github_action("pkgdown")`

* Triggering `gh-pages` when pushing to PR #310 (tests)

This is just for testing the online deployment and should be removed after merge (or just before).

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Small changes (#311)

* Small changes

* Fixed gh-pages bases for deployment (#290)

The PR branch was left over. More importantly, the file listed "main" instead of "master".

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Corrected small typo in documentation

* Cleaner unit test output (#314)

* Increment version number to 1.3.2.9006

* Capturing output of test unit

* Dirichlet concentration parameter is now actually being used (#315)

* forwarded psi argument to run_mcmc

* Incremented version and updated news.md

* ready for cran submission

* CRAN wanted a trailing slash in the URL to the website, since it becomes added automatically if one follows the URL

* Internalized a bunch of SMC functions and made families #312

* incrementing development versions

* updating news.md

* linting

* fixed most of the thigns

* incremented dev version

* Fixed lint issues

* removing tagging of Anja as author since she is listed in DESCRIPTION

* lots of work in order to clean up compute_consensus

* linting

* got rid of annoying row names in posterior interval dataframe

* made a version with parallel chains that is actually working

* close to a working version with parallel chains

* first iteration

* styling and updating style rules

* attempt at refactoring

* getting there

* styling

* fixing initial value hickup

* styling

* Parallel issue 307 (#320)

* made a version with parallel chains that is actually working

* close to a working version with parallel chains

* first iteration

* styling and updating style rules

* attempt at refactoring

* getting there

* styling

* fixing initial value hickup

* styling

* correcting typos in vignette

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>
osorensen added a commit that referenced this pull request Oct 20, 2023
* Removed duplicated output element (#291)

* Removed duplicated output element

* Updated package version and NEWS

* Should fix #292 (#293)

* Updated NEWS and increment version

* Updated citation info in README

* Should fix #299 (#300)

* Should fix #299

* Added missing documentation objects and changed the name of one argument

* Fixed another documentation issue

* Removed deprecated SMC functions (#303)

* Updated build version number

* Increment version number to 1.3.2.9002

* Removed deprecated SMC functions (#301)

* Updated NEWS (#301)

* Logz issue 158 (#304)

* Added exact partition function option

* Updated tests

* Updated SMC Mallows examples

* Styling

* More linting

* Updated linter workflow

Some linter were deprecated, ignored line on vignette needed update

* Trying to please CodeFactor

* Incremented version number and added entry to NEWS.md

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Added BugReport URL (#308)

* Added URL for bug reports

* Increment version number to 1.3.2.9004

* Small typo correction and documentation update

* Website (#310)

* Website

* De-duplicating vignettes

* Adding GH Action for pkgdown (#290)

Literally the output of `usethis::use_github_action("pkgdown")`

* Triggering `gh-pages` when pushing to PR #310 (tests)

This is just for testing the online deployment and should be removed after merge (or just before).

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Small changes (#311)

* Small changes

* Fixed gh-pages bases for deployment (#290)

The PR branch was left over. More importantly, the file listed "main" instead of "master".

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>

* Corrected small typo in documentation

* Cleaner unit test output (#314)

* Increment version number to 1.3.2.9006

* Capturing output of test unit

* Dirichlet concentration parameter is now actually being used (#315)

* forwarded psi argument to run_mcmc

* Incremented version and updated news.md

* ready for cran submission

* CRAN wanted a trailing slash in the URL to the website, since it becomes added automatically if one follows the URL

* Internalized a bunch of SMC functions and made families #312

* incrementing development versions

* updating news.md

* linting

* fixed most of the thigns

* incremented dev version

* Fixed lint issues

* removing tagging of Anja as author since she is listed in DESCRIPTION

* lots of work in order to clean up compute_consensus

* linting

* got rid of annoying row names in posterior interval dataframe

* made a version with parallel chains that is actually working

* close to a working version with parallel chains

* first iteration

* styling and updating style rules

* attempt at refactoring

* getting there

* styling

* fixing initial value hickup

* styling

* Parallel issue 307 (#320)

* made a version with parallel chains that is actually working

* close to a working version with parallel chains

* first iteration

* styling and updating style rules

* attempt at refactoring

* getting there

* styling

* fixing initial value hickup

* styling

* correcting typos in vignette

* Removed unused argument from augment_pairwise() #325

* Updated news

* incremented version

---------

Co-authored-by: Waldir Leoncio <w.l.netto@medisin.uio.no>
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.

psi variable not used in compute_mallows?
2 participants