Skip to content

Conversation

@pavelkomarov
Copy link
Collaborator

@pavelkomarov pavelkomarov commented Sep 2, 2025

To further address #138.

  • made new module for basis-function-based methods
  • changed imports
  • reran jupyter notebooks
  • added an entry for rbfdiff in optimization

"output_type": "stream",
"text": [
"100%|██████████| 13/13 [01:56<00:00, 8.96s/it]\n"
"100%|██████████| 14/14 [01:57<00:00, 8.39s/it]\n"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Possibly got faster due to #154. I'm seeing polydiff takes 30-35 seconds to optimize now. It was formerly nearly a minute.


def test_simulate():
return
# def test_simulate():
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Having real tests in here that don't do anything is odd, because they get run. Yet I don't want to remove the stubs, as a reminder that we should maybe have some tests for these. But in the case of simulations, it's really just "Do they all run okay?", to which the answer is a known yes unless we futz with them significantly, and in the case of evaluations the metrics and plotting functions are defined by hand with intentionality. In both cases the more obvious place to see a failure would be in the notebooks, not in a unit test, although we can probably unit test something for safety.

@pavelkomarov pavelkomarov merged commit a666641 into master Sep 2, 2025
1 check passed
@pavelkomarov pavelkomarov deleted the basis-based-home branch October 3, 2025 20:33
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