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

Update Readme and the URL for the API documentation #779

Merged
merged 76 commits into from
Jun 26, 2024
Merged

Update Readme and the URL for the API documentation #779

merged 76 commits into from
Jun 26, 2024

Conversation

irfanpena
Copy link
Contributor

Describe Your Changes

  • Update Readme to reflect the new updates
  • Update the URL for the API doc

Fixes Issues

  • Closes #
  • Closes #

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

@irfanpena irfanpena requested a review from louis-jan June 26, 2024 00:37
@irfanpena irfanpena self-assigned this Jun 26, 2024
louis-jan
louis-jan reviewed Jun 26, 2024
View reviewed changes
README.md Show resolved Hide resolved
louis-jan
louis-jan reviewed Jun 26, 2024
View reviewed changes
cortex-js/README.md Outdated Show resolved Hide resolved
louis-jan
louis-jan reviewed Jun 26, 2024
View reviewed changes
cortex-js/README.md Outdated Show resolved Hide resolved
louis-jan
louis-jan reviewed Jun 26, 2024
View reviewed changes
cortex-js/README.md Outdated Show resolved Hide resolved
@irfanpena
Copy link
Contributor Author

@louis-jan Updated, kindly check again

@irfanpena irfanpena requested a review from louis-jan June 26, 2024 04:16
louis-jan
louis-jan reviewed Jun 26, 2024
View reviewed changes
cortex-js/README.md Outdated Show resolved Hide resolved
louis-jan
louis-jan reviewed Jun 26, 2024
View reviewed changes
README.md Show resolved Hide resolved
@irfanpena irfanpena requested a review from louis-jan June 26, 2024 05:03
louis-jan
louis-jan reviewed Jun 26, 2024
View reviewed changes
README.md Outdated Show resolved Hide resolved
louis-jan and others added 21 commits June 26, 2024 15:40
@louis-jan @irfanpena
fix: remove build target (#769)
bdcd108
@vansangpfiev @sangjanai @irfanpena
chore: use customized build for vulkan (#768)
0cd2d53 
Co-authored-by: vansangpfiev <sang@jan.ai>
@louis-jan @irfanpena
feat: add engine pull support for tensorrt-llm (#765)
8d0f678
@louis-jan @irfanpena
fix: remove patch-package dependency (#770)
1b2cb35
@louis-jan @irfanpena
fix: cortex node suppress warning (#771)
0be10c5
@louis-jan @irfanpena
chore: rename npm package cortexso (#772)
6cbcb49
@l2D @irfanpena
chore: Add Swagger UI Log Message and Update README.md with API Acc…
d482c69 
…ess Information (#749)

Signed-off-by: l2D <20911264+l2D@users.noreply.github.com>
@irfanpena
Update the readme and the server URL for the API documentation
e143638
@irfanpena
Update the description
2108c14
@louis-jan @irfanpena
chore: add dependency instructions (#773)
d5ae28f
@irfanpena
update @janhq/cortex -> cortexso
8ccc304
@irfanpena
add prerequisites section
8bb43cd
@irfanpena
update per feedback
37222fc
@irfanpena
add brew installation
f626e42
@irfanpena
Separate npm and brew
65a5d39
@louis-jan @irfanpena
chore: add MacOS pkg installer (#774)
1dba491
@irfanpena
nits
afd79ca
@irfanpena
edit the URL
0944227
@irfanpena
update the desc
f939ef5
@irfanpena
Merge branch 'dev' of https://github.com/janhq/cortex into pena-patch
f91fd54
@irfanpena
Merge branch 'pena-patch' of https://github.com/janhq/cortex into pen…
49061d4 
…a-patch
louis-jan
louis-jan reviewed Jun 26, 2024
View reviewed changes
cortex-js/src/app.ts Outdated Show resolved Hide resolved
louis-jan
louis-jan approved these changes Jun 26, 2024
View reviewed changes
Copy link
Contributor

@louis-jan louis-jan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@irfanpena
Copy link
Contributor Author

@louis-jan Added the v1 slug URL. Also I cannot merge the PR on my own

@louis-jan louis-jan merged commit aae83ae into dev Jun 26, 2024
1 check passed
@louis-jan louis-jan deleted the pena-patch branch June 26, 2024 09:18
@irfanpena irfanpena restored the pena-patch branch June 27, 2024 01:37
@irfanpena irfanpena deleted the pena-patch branch June 27, 2024 01:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@louis-jan louis-jan louis-jan approved these changes

Assignees

@irfanpena irfanpena

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
10 participants
@irfanpena @louis-jan @Van-QA @CameronNg @marknguyen1302 @namchuai @hiento09 @0xSage @vansangpfiev @l2D