mirror of
https://github.com/PhotonVision/photonvision
synced 2026-06-19 00:41:41 +00:00
Switch from FasterXML Jackson to Avaje Jsonb (#2503)
## Description WPILib switched from FasterXML Jackson to Avaje Jsonb for speed reasons in https://github.com/wpilibsuite/allwpilib/pull/8721. This does the same for PhotonVision. Some temporary Jackson adapters are present to allow compatibility with alpha-4 ahead of updating Photon's WPILib version. A few old backwards compatibility migrations were also dropped if they were difficult to port to Avaje Jsonb or otherwise complicated the code. ## Meta Merge checklist: - [x] Pull Request title is [short, imperative summary](https://cbea.ms/git-commit/) of proposed changes - [x] The description documents the _what_ and _why_, including events that led to this PR - [ ] If this PR changes behavior or adds a feature, user documentation is updated - [ ] If this PR touches photon-serde, all messages have been regenerated and hashes have not changed unexpectedly - [ ] If this PR touches configuration, this is backwards compatible with all settings going back to the previous seasons's last release (seasons end after champs ends) - [ ] If this PR touches pipeline settings or anything related to data exchange, the frontend typing is updated - [ ] If this PR addresses a bug, a regression test for it is added - [ ] If this PR adds a dependency, the license has been checked for compatibility and steps taken to follow it --------- Co-authored-by: samfreund <samf.236@proton.me> Co-authored-by: Matt Morley <matthew.morley.ca@gmail.com>
This commit is contained in:
@@ -17,13 +17,14 @@
|
||||
|
||||
package org.photonvision.server;
|
||||
|
||||
import io.avaje.jsonb.Json;
|
||||
import io.avaje.jsonb.Jsonb;
|
||||
import io.javalin.http.Context;
|
||||
import org.photonvision.common.configuration.ConfigManager;
|
||||
import org.photonvision.common.configuration.NeuralNetworkModelManager;
|
||||
import org.photonvision.common.hardware.Platform;
|
||||
import org.photonvision.common.logging.LogGroup;
|
||||
import org.photonvision.common.logging.Logger;
|
||||
import org.photonvision.common.util.file.JacksonUtils;
|
||||
|
||||
public class TestRequestHandler {
|
||||
// Treat all 2XX calls as "INFO"
|
||||
@@ -39,12 +40,13 @@ public class TestRequestHandler {
|
||||
ConfigManager.getInstance().load();
|
||||
}
|
||||
|
||||
private record PlatformOverrideRequest(Platform platform) {}
|
||||
@Json
|
||||
record PlatformOverrideRequest(Platform platform) {}
|
||||
|
||||
public static void handlePlatformOverrideRequest(Context ctx) {
|
||||
try {
|
||||
PlatformOverrideRequest request =
|
||||
JacksonUtils.deserialize(ctx.body(), PlatformOverrideRequest.class);
|
||||
Jsonb.instance().type(PlatformOverrideRequest.class).fromJson(ctx.body());
|
||||
Platform platform = request.platform();
|
||||
logger.info("Overriding platform to: " + platform);
|
||||
|
||||
|
||||
Reference in New Issue
Block a user