Remove need of saving images before logging.

Open
#11 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
35/100
Issue type
Refactor
Clarity
Mostly clear
Activity status
Stale
Tech stack
cpp

Research direction

Start by reading the Image support code and tracing how armadillo matrices are logged as images. Review the discussion linked from PR #6 for the existing encoded-format context. Done means matrices can be logged without temporary image files while retaining the current visualization behavior.

Written by the indexing model from the issue text.

Description

enhancement help wanted

While logging armadillo matrices as Images for visualization in mlboard currently it is mandatory to save armadillo matrices as an image temporarily before logging. The reason we do that is to get the armadillo matrices in encoded format in the form of binary string. However, due to this, it needs quite a lot of reading and writing to the file system which can be avoided. It might also lead to increased time while logging thousands of images.
So the best solution to tackle this issue is to remove the need of saving images. This issue might need a good understanding of the Image support code. For more info on the topic you can see the following comment.
https://github.com/mlpack/mlboard/pull/6#discussion_r452971739

Dominant language
C++
Stars
13
Forks
8
PR merge metrics
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from mlpack/mlboard

All issues in mlpack/mlboard

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.