-
Notifications
You must be signed in to change notification settings - Fork 2
Docs tidy after dropping dependencies #195
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
Open
rwb27
wants to merge
89
commits into
main
Choose a base branch
from
docs-tidy-after-dropping-dependencies
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
89 commits
Select commit
Hold shift + click to select a range
11d705a
Change `add_thing` to use name and instantiate Things.
rwb27 5f59b04
Improve validation of arguments.
rwb27 f670f22
Update DirectThingClient to use name not path.
rwb27 cbe8791
Return the created Thing from add_thing
rwb27 5c60e0d
Update test suite to use new add_thing syntax.
rwb27 fcb4ca8
Update code examples in documentation
rwb27 ed073b9
Update docstrings that mention `add_thing`
rwb27 ac8ec8d
Fix docstring for add_thing.
rwb27 00bce70
Remove vestigial #noqa statements
rwb27 66810fc
Implement ThingServerInterface
rwb27 7a8d9e5
Provide, and test, thing state metadata via ThingServerInterface
rwb27 86cf44d
Create invocation loggers before starting actions.
rwb27 025cd83
Update concurrency docs.
rwb27 8f7b5c7
Replace BlockingPortal with ThingServerInterface.
rwb27 1c18dbc
Mark slow tests to speed up repeated testing.
rwb27 eeddee9
Update test for MJPEGStream
rwb27 0acea11
Update test suite to mock the server interface.
rwb27 4f5e429
Tests for ThingServerInterface.
rwb27 8f06fcc
Typing fixes.
rwb27 0fb1e37
Docstring fixes.
rwb27 9394f4d
Use ThingServerInterface in settings code
rwb27 e774862
Deduplicate `path_for_thing`.
rwb27 2e13f2c
Refactor field-like typing logic into base_descriptor.
rwb27 78a9720
First implementation of ThingConnection.
rwb27 5e333f6
Completed implementation of thing_connection
rwb27 405ac09
Test code for thing_connection
rwb27 e688dd7
Improve handling of field-typed descriptors.
rwb27 28b2709
Docstring fixes
rwb27 f747fc6
Revert to using `typing.get_type_hints` but evaluate lazily.
rwb27 9816338
Support multiple things in a connection
rwb27 8cb258c
Move more error checking into `ThingConnection`
rwb27 24998bf
Got tests passing
rwb27 869bd73
Tidy up some imports
rwb27 31350b4
Testing and improvements
rwb27 739372c
Removed unnecessary override.
rwb27 4464624
Typing fixes.
rwb27 7a266e7
Update tests to take account of different type evaluation.
rwb27 dbcbe72
Check ReferenceError is raised if a Thing gets deleted.
rwb27 2260d64
Add comment to #type: ignore
rwb27 3451344
Explicitly don't support forward references in type subscripts.
rwb27 0e6908c
Add full testing for FieldTypedBaseDescriptor.
rwb27 79c08fb
Mark dependencies as deprecated.
rwb27 997f1b6
Fix broken test
rwb27 a5d61d2
Fix type ignore comments
rwb27 b0c12c6
Fix docstring
rwb27 e9a0e29
Pass Thing Connection config through `add_thing`.
rwb27 602ccf1
Allow thing connections to be specified in config.
rwb27 be6fc96
Spelling fix
rwb27 04499c1
Fix a typo in test code.
rwb27 cc3cb27
Add a documentation page on thing connections.
rwb27 369faaa
Fix tests for Python < 3.12
rwb27 421ac90
Add invocation context module
rwb27 4f9efd1
Migrate actions and add to __init__
rwb27 d489910
Tests and documentation.
rwb27 f2ef644
Docstring fixes
rwb27 fd4b610
Migrate test code
rwb27 a5fa721
More docstring for ThreadWithInvocationID
rwb27 b791072
Split cancellable_sleep
rwb27 bea820d
Don't make loggers per-invocation
rwb27 f784f50
Fix old test and dependency
rwb27 056f61f
Fix an edge case in ThreadWithInvocationID
rwb27 3478e65
Test the logs module
rwb27 e282676
Initialise all Things during ThingServer.__init__
rwb27 15dce7b
Tidy up config models and make them available at module level.
rwb27 5399569
Tidy up Thing lifecycle and leave validation to the Pydantic models
rwb27 d9689a0
Don't print stack traces for validation errors.
rwb27 f3aedd8
Update test code to pass Thing classes to ThingServer.__init__
rwb27 869d7ac
Update example/documentation code examples.
rwb27 3a5aef1
Fix up some docstrings.
rwb27 c7786df
Rename thing_connection to thing_slot
rwb27 728baa5
Remove the defunct object_reference_to_object
rwb27 655bf9d
Remove an unused import
rwb27 1c23151
Typing fixes for the fallback server
rwb27 b1b98e2
Update index and core concepts
rwb27 c496f96
Allow specifying folder in create_thing_without_server
rwb27 aa04aff
Spelling fixes in docs.
rwb27 6aaf7b2
silence a spurious type warning
rwb27 71e2a38
Update src/labthings_fastapi/server/__init__.py
rwb27 28e905a
Reformat example to give `things` a name.
rwb27 c6766aa
Fix some broken references in the docs.
rwb27 d2b03b5
Use sphinx-toolbox to provide decorator links.
rwb27 b79b62f
Fix some ambiguous links in the docs
rwb27 1fcbf9c
Fix links and RST formatting
rwb27 6782063
Update some broken links and don't link to WoT Core Concepts so much.
rwb27 cfb90b3
Properly import ThingServerInterface in `thing`
rwb27 a077e7f
Shorten the explicit reference `_structure`.
rwb27 88dcbf8
Single-source the version number.
rwb27 a79423f
Fix indentation in subclassing notes.
rwb27 c9bfd2e
De-emphasise WoT core concepts
rwb27 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The file I want to comment on doesn't exist, but this is the same directory.
running_labthings.rsthas a/in the path ofmythingon both lines 12 and 31