Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Moving the Python SDK into this repository? #82

jhoetter started this conversation in Ideas
Discussion options

The Python SDK is an integral part of refinery, as it allows you to both push and pull data directly and further will give some other helpful features such as pushing heuristic results, etc.

We'll arguably commit to Python for a long, long time. So my idea is, should we just move the Python SDK into this repository, so that you have access to the SDK right away? Or at least add it as a dependency so you can use it right out of the box?

Are there any major drawbacks I'm forgetting?

You must be logged in to vote

Replies: 2 comments

Comment options

jhoetter
Sep 13, 2022
Maintainer Author

So far:

Pro

  • No need to install another library, users can just apply the SDK right away

Contra

  • An update in the SDK will ultimately result in a new application version.
  • If the SDK contains dependencies to larger frameworks, this ultimately results in a longer installment of pip install kern-refinery.

Another option would be to have two installment scripts for pip, one with the SDK (default) and one without it. Generally, the goal of this whole discussion is to make it easier for users to apply integrations, etc. to the application. We're going to make the SDK a crucial part of the application.

You must be logged in to vote
0 replies
Comment options

jhoetter
Sep 21, 2022
Maintainer Author

So, we decided to keep them separate, at least for now, mainly for dependency reasons. But we will keep this an open discussion, as this isn't set in stone.

You must be logged in to vote
0 replies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Category
💡
Ideas
Labels
None yet
1 participant
Morty Proxy This is a proxified and sanitized view of the page, visit original site.