Skip to content

Logic for Bayesian update procedure#96

Merged
vredchenko merged 11 commits intomainfrom
prior-update
May 23, 2025
Merged

Logic for Bayesian update procedure#96
vredchenko merged 11 commits intomainfrom
prior-update

Conversation

@d-j-hatton
Copy link
Contributor

@d-j-hatton d-j-hatton commented Apr 28, 2025

Also includes command line tools for rough simulations of the prediction and update processes.

  • Test command line tools
  • Test timestamp ordering logic for Bayesian update method

@vredchenko
Copy link
Collaborator

Nice! I have a couple of remarks:

@staticmethod
    def generate_uuid():
        return urlsafe_b64encode(uuid4().bytes).decode('ascii').rstrip('=')

@d-j-hatton
Copy link
Contributor Author

  • I already use /tools and /tests dirs outside src/ - these scripts could arguably be categorised as either. I don't mind changing away from tools and tests, introducing cli to consolidate all command-line stuff, and/or moving any of them under src/ - whatever you prefer?

Putting them in tools makes sense I think, I have no problem with them going there

@vredchenko
Copy link
Collaborator

  • I already use /tools and /tests dirs outside src/ - these scripts could arguably be categorised as either. I don't mind changing away from tools and tests, introducing cli to consolidate all command-line stuff, and/or moving any of them under src/ - whatever you prefer?

Putting them in tools makes sense I think, I have no problem with them going there

Cool, I guess this PR depends on #92 so we can leave this hanging for now - if you like I'll pick this up and consolidate it with 92 once that one's finished

@d-j-hatton
Copy link
Contributor Author

Sounds good. Gives me time to test it properly. Swamped with paperwork today

@vredchenko vredchenko changed the title [WIP] Logic for Bayesian update procedure Logic for Bayesian update procedure May 22, 2025
@vredchenko vredchenko merged commit 31dc883 into main May 23, 2025
5 checks passed
@vredchenko vredchenko deleted the prior-update branch May 23, 2025 12:51
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