-
Notifications
You must be signed in to change notification settings - Fork 482
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
Create flat and sequential docs structure #1168
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1168 +/- ##
==========================================
+ Coverage 80.33% 80.43% +0.09%
==========================================
Files 95 97 +2
Lines 6602 6684 +82
==========================================
+ Hits 5304 5376 +72
- Misses 1298 1308 +10
☔ View full report in Codecov by Sentry. |
25d496e
to
a327af8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing !!
```bash | ||
$ export SUPERDUPERDB_DATA_BACKEND='mongodb://localhost:27018/documents' | ||
$ python -c 'import superduperdb; print(superduperdb.CFG.databackend)' | ||
mongodb://localhost:27018/documents |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-
We should add more example of configuration,
like how to configure vector search -
we should also add example of environment variable configuration
SUPERDUPERDB_
with multi level configuration example cluster.distributed = True
|
||
```python | ||
from superduperdb import superduper | ||
db = superduper() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should showcase
db = pymongo.client()
db = superduper(db)
|
||
### `db.show` | ||
|
||
This methods displays which `Component` instances are registered with the system. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should give link to Components md file ?
) | ||
``` | ||
|
||
### Vector-search |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could be a seperate vector search md file?
) | ||
``` | ||
|
||
### Coming soon: support for raw-sql |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
already merged
|
||
m = Pipeline(task='sentiment-analysis') | ||
|
||
m.predict( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we have a Listener md file?
```python | ||
from superduperdb import CFG | ||
|
||
CFG.artifact_store = 'filesystem://./data' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What other fields are supported in CFG ?
I think a For example: Milvus Glossary , Product FAQ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- The code in the notebook is old. Some of the new versions of the code no longer work and need to be updated.
- I think we missing a real (or fake) use case to deploy SuperDuperDB for production. Not just a notebook. For example, we had a database, how to deploy superduperdb to handle it, maybe command line, or other?
- There is a lack of a brief process introduction of superduperdb, what happened after adding the model, what happened after adding the listener, etc. Let users understand the general operating mechanism
|
||
- [Applying models](19_apply_models.mdx) | ||
- [Vector search](22_vector_search.mdx) | ||
- [Example Q&A application](/docs/use_cases/items/question_the_docs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
404
- A [**change-data-capture** service](29_change_data_capture.md) | ||
- A [**vector-search** service](30_vector_comparison_service.md), which finds similar vectors, given an input vector | ||
|
||
In the following pages, we describe how to set-up these independent services. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Miss the link or information about the following pages
fields: | ||
|
||
```python | ||
s = Schema('my-schema', fields={'my-text': 'str', 'my-image': my_image_encoder}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we add the context to let users know what this is and how to use it.
) | ||
``` | ||
|
||
Now when the data is loaded from the database, it is loaded as text: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should explain to the user what's going on here. Because we must assume that users are not familiar with related concepts, such as encoder and decoder in Encoder, and uri->bytes
docs/hr/content/docs/17_ai_apis.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is recommended to add a custom API wrapper demo here.
api = 'https://xxxxxxx:12345/predict'
class ModelA(Model):
......
def predict(self, x):
return label
db.add(ModelA(api))
Then users can quickly use superduperdb to test their model services like this
) | ||
|
||
db.add( | ||
Stack( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import Stack?
Hi @jieguangzhou thanks for this in depth feedback. I'm working on adding 1, 3. (I realize the notebooks/ use-cases are out-of-date). For 2. what would you suggest? A mini-code-base? |
e7eec38
to
942e5c4
Compare
This PR aims to reboot the documentation: