Skip to content
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

Brayns 285 fix logic issue in quanta jpeg stream #925

Merged
merged 2 commits into from
Oct 22, 2021

Conversation

Adrien4193
Copy link
Contributor

Now the JPEG stream trigger is only reset once the image is sent instead of each update.

@Adrien4193 Adrien4193 requested a review from NadirRoGue October 22, 2021 07:50
@Adrien4193 Adrien4193 merged commit d7f62a0 into develop Oct 22, 2021
@Adrien4193 Adrien4193 deleted the BRAYNS-285_FixLogicIssueInQuantaJpegStream branch October 22, 2021 08:48
NadirRoGue pushed a commit that referenced this pull request Dec 14, 2021
* Fixed logic issue (+some renaming).

* Removed spdlog added by mistake.
NadirRoGue added a commit that referenced this pull request Dec 15, 2021
* Bumped Pillow and websockets version. (#922)

* Code cleanup (#923)

* Brayns 278 add mock requests in python tests (#924)

* Brayns 285 fix logic issue in quanta jpeg stream (#925)

* Brayns 286 fix python image method (#928)

* Brayns 287 fix python entrypoint methods with one of schema (#929)

* Loaders & CircuitExplorer refactoring (#927)

* Fixed some typo's (#930)

* Update AUTHORS.txt (#931)

* Brayns 258 create a proper logging system for brayns (#932)

* Frame export improvement and static global objects removal: (#933)

* Removes CMake/common dependency from build system (#934)

* Removed VRPN plugin and deps. (#935)

* Brayns 295 rename brayns json macros to a more appropriate name (#936)

* Fixes segfault due to global const std::strings (#939)

* Map and unmap framebuffer added. (#940)

* Fix entry points variable naming (#941)

* Brayns 300 replace free image (#942)

* Brayns 308 Code restructure (#943)

* Brayns 315 2.0.0 release preparation (#944)
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