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

Update main branch with Nick's code #1

Open
wants to merge 39 commits into
base: master
Choose a base branch
from
Open

Update main branch with Nick's code #1

wants to merge 39 commits into from

Conversation

nrjl
Copy link
Member

@nrjl nrjl commented Jul 1, 2019

Not ideal, but combined-nick should be the main branch at this point.

thane and others added 30 commits February 27, 2017 18:02
This isn't working now and the code's a mess from debugging, but it's a
start?
Added estimated derivatives for debugging
- Fixed error with indexing the same point multiple times in the relative ratings derivative, Thane look for ^^FIXED BIT - THANE LOOK HERE^^
- Also changed all the references from X_train to X_rel for relative training data for clarity
- Moved some test data stuff (sample functions etc) into new file test_data
Updated to use obs_kwargs (primarily to know n_ordinals) during construction of PreferenceGP object
Added some stuff to run statrun tools remotely (command line options to disable plots), still working on higher dimensions of input, currently removing uv_rel stuff because it is pointless (just use x_rel and uvi)
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.

2 participants