Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Email in API key #59

Open
wants to merge 23 commits into
base: dev
Choose a base branch
from
Open

Email in API key #59

wants to merge 23 commits into from

Conversation

MichaelRipa
Copy link
Member

@MichaelRipa MichaelRipa commented Sep 26, 2024 •
edited
Loading

This update does the following:

  • Checks whether an API key contains an email, throwing a custom error message (redirecting to login.ndif.us)
  • Updates the default error message for invalid API keys to also include a mention of login.ndif.us

Related PR: ndif-team/nnsight#249

@MichaelRipa MichaelRipa mentioned this pull request Sep 26, 2024
Merged
@JadenFiotto-Kaufman
Streaming working with nnsight streaming
c59a88f
@JadenFiotto-Kaufman
Use redis instead of rabbitmq. Logic for handling websocket disconnec…
afb6492 
…ts in model base.
@MichaelRipa
Added basic live update script
6bc9f7c
@MichaelRipa
Added automatic conda environment updates for service rebuilds
f53df0d
@MichaelRipa
cleanup
2fa60a0
@MichaelRipa
Added docstring and debug mode
e6d8626
@JadenFiotto-Kaufman
Set cuda env to be all gpus in base model deployment if no gpus speci…
932d62a 
…fied
@JadenFiotto-Kaufman
API environment requires redis not pika (RMQ ) now
2f66083
@JadenFiotto-Kaufman
Merge remote-tracking branch 'origin/dev' into streaming-protocol
c4605ab
@AdamBelfki3
Merge pull request #67 from ndif-team/streaming-protocol
f96dc7f 
Streamed execution
@JadenFiotto-Kaufman
Fix handing of dtensors. Set cuda vis devices to all if none provided
96576d5
@JadenFiotto-Kaufman
Merge branch 'streaming-protocol' into 0.4
98bc4eb
@JadenFiotto-Kaufman
Ndif working with 0.4 ! streaming protocol next
537c853
@JadenFiotto-Kaufman
Bug was suppressing remote exceptions to the user
c4c6760
@MichaelRipa
Removed update Makefile target
af4063a
@MichaelRipa
removed update script
b72e6ba
@JadenFiotto-Kaufman
Streaming local <-> remote
47e7f2e
@MichaelRipa
Merge branch '0.4' of https://github.com/ndif-team/ndif into live-upd…
d2cc9b6 
…ates
@MichaelRipa
Corrected nnsight repo branch
0c9bec8
@JadenFiotto-Kaufman
Merge pull request #62 from ndif-team/live-updates
dee43ec 
Add live dependency updates and auto-update on container start
@MichaelRipa
Merge branch '0.4' of https://github.com/ndif-team/ndif into api-key/…
4718037 
…custom-error
@MichaelRipa
Fixed slugify path for 405b
8f0db54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JadenFiotto-Kaufman JadenFiotto-Kaufman Awaiting requested review from JadenFiotto-Kaufman

@AdamBelfki3 AdamBelfki3 Awaiting requested review from AdamBelfki3

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@MichaelRipa @JadenFiotto-Kaufman @AdamBelfki3