drop expected output from common sql functions output

[mattseddon]

follow up to #860

it seems weird that we would manually maintain a text copy of the expected output

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.74%. Comparing base (7f757b3) to head (502f683).
Report is 5 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #886   +/-   ##
=======================================
  Coverage   87.74%   87.74%           
=======================================
  Files         129      129           
  Lines       11462    11462           
  Branches     1545     1545           
=======================================
  Hits        10057    10057           
  Misses       1017     1017           
  Partials      388      388           

Flag	Coverage Δ
datachain	87.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dreadatour]

it seems weird that we would manually maintain a text copy of the expected output

🤔 Not really agree tbh.

It was useful to include this output in the example so people can see what’s happening without running the code. This is common practice.

The same applies to Jupyter Notebooks — we have them "baked" in https://github.com/iterative/datachain-examples. In case of any backward-incompatible changes in the Datachain repo, we need to re-run the corresponding example to generate the new output.

[mattseddon]

it seems weird that we would manually maintain a text copy of the expected output

🤔 Not really agree tbh.

It was useful to include this output in the example so people can see what’s happening without running the code. This is common practice.

The same applies to Jupyter Notebooks — we have them "baked" in https://github.com/iterative/datachain-examples. In case of any backward-incompatible changes in the Datachain repo, we need to re-run the corresponding example to generate the new output.

This isn't an option with Jupyter Notebooks and the process for keeping them updated is more straightforward than remembering to run and then copy/pasting the output into the same file.

[cloudflare-workers-and-pages]

Deploying datachain-documentation with    Cloudflare Pages

Latest commit:	502f683
Status:	✅  Deploy successful!
Preview URL:	https://49c3b187.datachain-documentation.pages.dev
Branch Preview URL:	https://drop-expected-str.datachain-documentation.pages.dev

View logs