Skip to content

biwt refactoring and 3d#339

Closed
drbergman wants to merge 1 commit intoPhysiCell-Tools:developmentfrom
drbergman:biwt-refactor-rebase
Closed

biwt refactoring and 3d#339
drbergman wants to merge 1 commit intoPhysiCell-Tools:developmentfrom
drbergman:biwt-refactor-rebase

Conversation

@drbergman
Copy link
Copy Markdown
Collaborator

  • refactoring of biwt code; now fully supports interactive 3d plotting!
  • Modest extensions/refactoring (not API-breaking) of some studio classes.

commit message

  • process_window now abstract method required by all BioinformaticsWalkthroughWindow subclasses
  • create_continue_button to standardize continue button creation
  • confluence shows only 2 digits beyond decimal
    • stores the full value but only displays 2 digits when not editing
    • improve logic for updating other boxes
  • remove extra whitespace from biwt_tab.py
  • fix use of nonnegative_validator for 3d
  • fix current_pars_acceptable
  • fix typo in disc_plotter_3d

- process_window now abstract method required by all BioinformaticsWalkthroughWindow subclasses
- create_continue_button to standardize continue button creation
- confluence shows only 2 digits beyond decimal
  - stores the full value but only displays 2 digits when not editing
  - improve logic for updating other boxes
- remove extra whitespace from biwt_tab.py
- fix use of nonnegative_validator for 3d
- fix current_pars_acceptable
- fix typo in disc_plotter_3d
@drbergman drbergman changed the title refactoring and 3d biwt refactoring and 3d Dec 15, 2025
@rheiland
Copy link
Copy Markdown
Member

rheiland commented Apr 2, 2026

I'm just now seeing this (or reminded of this). Are there indeed conflicts that need resolved?

@drbergman
Copy link
Copy Markdown
Collaborator Author

drbergman commented Apr 3, 2026

This branch has been superseded by the PR by Marwa that we merged back in January. The changes I see here are already in development

@drbergman drbergman closed this Apr 3, 2026
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