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

[api] Allow to show NDArray content in Debugger 2 #2080

Merged
merged 7 commits into from
Oct 14, 2022

Conversation

KexinFeng
Copy link
Contributor

@KexinFeng KexinFeng commented Oct 12, 2022

Description

The previous solution still suffers from a slowing down when printing the string outside the debugger mode. Here the solution is to utillize the existing pruning mechanism, and put the execution of init(array) also under the condition that the array is not too large.

A testing code is added.

@codecov-commenter
Copy link

codecov-commenter commented Oct 12, 2022

Codecov Report

Base: 72.08% // Head: 70.24% // Decreases project coverage by -1.84% ⚠️

Coverage data is based on head (c5c5f97) compared to base (bb5073f).
Patch coverage: 72.22% of modified lines in pull request are covered.

❗ Current head c5c5f97 differs from pull request most recent head 77dbd35. Consider uploading reports for the commit 77dbd35 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2080      +/-   ##
============================================
- Coverage     72.08%   70.24%   -1.85%     
- Complexity     5126     6074     +948     
============================================
  Files           473      603     +130     
  Lines         21970    26893    +4923     
  Branches       2351     2912     +561     
============================================
+ Hits          15838    18892    +3054     
- Misses         4925     6588    +1663     
- Partials       1207     1413     +206     
Impacted Files Coverage Δ
api/src/main/java/ai/djl/modality/cv/Image.java 69.23% <ø> (-4.11%) ⬇️
...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java 76.00% <ø> (ø)
...rc/main/java/ai/djl/modality/cv/output/Joints.java 71.42% <ø> (ø)
.../main/java/ai/djl/modality/cv/output/Landmark.java 100.00% <ø> (ø)
...main/java/ai/djl/modality/cv/output/Rectangle.java 72.41% <0.00%> (ø)
...i/djl/modality/cv/translator/BigGANTranslator.java 21.42% <0.00%> (-5.24%) ⬇️
.../ai/djl/modality/cv/translator/YoloTranslator.java 27.77% <0.00%> (+18.95%) ⬆️
...modality/cv/translator/wrapper/FileTranslator.java 44.44% <ø> (ø)
...y/cv/translator/wrapper/InputStreamTranslator.java 44.44% <ø> (ø)
.../modality/cv/translator/wrapper/UrlTranslator.java 44.44% <ø> (ø)
... and 524 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

add message

add message
@KexinFeng KexinFeng merged commit db37e6e into deepjavalibrary:master Oct 14, 2022
KexinFeng added a commit to KexinFeng/djl that referenced this pull request Oct 26, 2022
* [api] Allow show NDArray content in debugger

* add message

add message

add message

* add message

* add message

* add message

* link

* link

Co-authored-by: Frank Liu <frankfliu2000@gmail.com>
@KexinFeng KexinFeng deleted the nd branch March 15, 2023 08:25
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