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

Device Name parsing #1490

Merged
merged 1 commit into from
Feb 2, 2022
Merged

Device Name parsing #1490

merged 1 commit into from
Feb 2, 2022

Conversation

zachgk
Copy link
Contributor

@zachgk zachgk commented Feb 1, 2022

This adds support for device names to the Device class and the ability to parse
those names.

This adds support for device names to the Device class and the ability to parse
those names.

Change-Id: I2686dbfeb1836e4610ff6cfde505073776ea900f
@zachgk zachgk requested a review from frankfliu February 1, 2022 22:29
@codecov-commenter
Copy link

Codecov Report

Merging #1490 (467fd36) into master (bb5073f) will decrease coverage by 1.35%.
The diff coverage is 50.07%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1490      +/-   ##
============================================
- Coverage     72.08%   70.73%   -1.36%     
- Complexity     5126     5318     +192     
============================================
  Files           473      493      +20     
  Lines         21970    23285    +1315     
  Branches       2351     2548     +197     
============================================
+ Hits          15838    16471     +633     
- Misses         4925     5534     +609     
- Partials       1207     1280      +73     
Impacted Files Coverage Δ
api/src/main/java/ai/djl/modality/cv/Image.java 69.23% <ø> (-4.11%) ⬇️
...i/djl/modality/cv/translator/BigGANTranslator.java 21.42% <ø> (-5.24%) ⬇️
...odality/cv/translator/BigGANTranslatorFactory.java 33.33% <0.00%> (+8.33%) ⬆️
...nslator/InstanceSegmentationTranslatorFactory.java 14.28% <0.00%> (-3.90%) ⬇️
.../modality/cv/translator/YoloTranslatorFactory.java 8.33% <0.00%> (-1.67%) ⬇️
...i/djl/modality/cv/translator/YoloV5Translator.java 5.69% <0.00%> (ø)
...odality/cv/translator/YoloV5TranslatorFactory.java 8.33% <0.00%> (-1.67%) ⬇️
...pi/src/main/java/ai/djl/ndarray/BytesSupplier.java 54.54% <0.00%> (-12.13%) ⬇️
...i/src/main/java/ai/djl/ndarray/NDArrayAdapter.java 15.23% <0.00%> (-0.83%) ⬇️
...l/training/loss/SigmoidBinaryCrossEntropyLoss.java 64.00% <0.00%> (ø)
... and 176 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c3bd94d...467fd36. Read the comment docs.

@frankfliu frankfliu merged commit 8749e41 into deepjavalibrary:master Feb 2, 2022
@zachgk zachgk deleted the deviceName branch February 3, 2022 22:16
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.

3 participants