Skip to content

add icebug notebooks#1

Open
aheev wants to merge 6 commits into
LadybugDB:mainfrom
aheev:add-lbug-arrow-csr
Open

add icebug notebooks#1
aheev wants to merge 6 commits into
LadybugDB:mainfrom
aheev:add-lbug-arrow-csr

Conversation

@aheev
Copy link
Copy Markdown

@aheev aheev commented May 23, 2026

icebug-memory and icebug-disk notebooks

@aheev
Copy link
Copy Markdown
Author

aheev commented May 23, 2026

@adsharma could you PTAL?

@adsharma
Copy link
Copy Markdown
Contributor

  • why not have both? Create this file under a new name

Future work:

  • would be nice to have a container data structure (IcebugMem or ArrowGraph) that holds the indices/indptr (someone doesn't need to understand what they are to use them)
  • Add a new constructor to icebug to take the container
  • make the flat <-> csr conversion as a utility function

@aheev
Copy link
Copy Markdown
Author

aheev commented May 23, 2026

  • why not have both? Create this file under a new name

yeah right. Will add an icebug-disk one too

Future work:

  • would be nice to have a container data structure (IcebugMem or ArrowGraph) that holds the indices/indptr (someone doesn't need to understand what they are to use them)
  • Add a new constructor to icebug to take the container

these go in icebug?

  • make the flat <-> csr conversion as a utility function

maybe this belongs to icebug-format?

@adsharma
Copy link
Copy Markdown
Contributor

Both the conversion and the container can live in the icebug-format repo.

Incentivizes other graph dbs to adopt icebug-format.

@aheev aheev changed the title refactor ladybug arrow csr add icebug notebooks May 26, 2026
@aheev
Copy link
Copy Markdown
Author

aheev commented May 27, 2026

@adsharma can you try now?

@adsharma
Copy link
Copy Markdown
Contributor

Renders ok now. But there are errors in some cells (missing col names you were discussing)

@aheev
Copy link
Copy Markdown
Author

aheev commented May 27, 2026

Renders ok now. But there are errors in some cells (missing col names you were discussing)

Yes. We can re-reun them once ladybug changes are in

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