Skip to content

Tablecloth restructure #165

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

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Tablecloth restructure #165

wants to merge 6 commits into from

Conversation

cmd-mov
Copy link
Member

@cmd-mov cmd-mov commented Mar 31, 2025

The tablecloth chapter has been split into 3 sections: one on creating, loading and saving datasets, one on working with columns, and one dataset operations. I have also added the list of datasets to the first chapter.

Cvetomir M. Dimov and others added 3 commits March 31, 2025 10:06
… is on creating, loading and saving datasets. The second one is on column operations. The third one is on datset operations.
@daslu
Copy link
Member

daslu commented Apr 2, 2025

Hi @cmd-mov, I think this is a great idea.

Note that we have made more recent updates to this notebook.

One of them is reading the Chicago bikes data file from the web to make it more self-contained:
https://github.com/scicloj/noj/blob/c3e7ab5/notebooks/noj_book/tablecloth_table_processing.clj#L353
So, we no longer include this data file in the Noj repo.

I think your version still tries to read the file the old way, and that is why it currently fails.

Possibly, there are a few other gaps between the current version of the notebook and the one you used.

We can look into this together when we meet.

@cmd-mov
Copy link
Member Author

cmd-mov commented Apr 2, 2025

Ah, thanks. I was wondering why. I believe I might have worked from an older version of the repo. Let's look at it when we meet indeed.

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