Skip to content

Change UserWarning to DeprecationWarning for Bus objects which are st…#328

Merged
FBumann merged 1 commit into
mainfrom
fixes/deprecation-warning
Sep 13, 2025
Merged

Change UserWarning to DeprecationWarning for Bus objects which are st…#328
FBumann merged 1 commit into
mainfrom
fixes/deprecation-warning

Conversation

@FBumann
Copy link
Copy Markdown
Member

@FBumann FBumann commented Sep 13, 2025

…ored inside Flows

Description

Brief description of the changes in this PR.

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Code refactoring

Related Issues

Closes #(issue number)

Testing

  • I have tested my changes
  • Existing tests still pass

Checklist

  • My code follows the project style
  • I have updated documentation if needed
  • I have added tests for new functionality (if applicable)

Summary by CodeRabbit

  • Bug Fixes
    • Deprecation notices during flow-to-bus connection are now emitted as DeprecationWarning (instead of UserWarning), improving compatibility with warning filters. The warning is emitted before auto-registering the bus. No functional changes to connection behavior.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Sep 13, 2025

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

📥 Commits

Reviewing files that changed from the base of the PR and between 698bdf4 and f4bac1a.

📒 Files selected for processing (1)
  • flixopt/flow_system.py (1 hunks)
 _____________________________________________________________________________________________________________________
< Use a project glossary. Create and maintain a single source of all the specific terms and vocabulary for a project. >
 ---------------------------------------------------------------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fixes/deprecation-warning

Comment @coderabbitai help to get the list of available commands and usage tips.

@FBumann FBumann merged commit aee77eb into main Sep 13, 2025
6 of 7 checks passed
@FBumann FBumann deleted the fixes/deprecation-warning branch September 13, 2025 16:52
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.

1 participant