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.

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

Docs: Migration notebook suggestions #4996

Closed
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
c3d9231
ci: Define simpler argilla CI workflow
frascuchon Jun 6, 2024
b9651e5
force build
frascuchon Jun 6, 2024
4c39f5d
ci: Ru only unit tests
frascuchon Jun 6, 2024
65b656a
ci: Remove and delete obsoleted workflows
frascuchon Jun 6, 2024
00f99be
ci: Update workflow refs
frascuchon Jun 6, 2024
53738b7
refactor: Rename base module to argilla_v1
frascuchon Jun 6, 2024
866dc90
chore: Fix string references to argilla module
frascuchon Jun 6, 2024
69c7485
chore: Configure project toml
frascuchon Jun 6, 2024
95a0f69
Fix missing argilla naming usage
frascuchon Jun 6, 2024
c38dd87
fix the tests
frascuchon Jun 6, 2024
6237f96
chore: Move argilla project to argilla-v1
frascuchon Jun 6, 2024
8eb495a
ci: Change refs to argilla-v1
frascuchon Jun 6, 2024
d846109
chore: update project setup
frascuchon Jun 7, 2024
d86895c
Define argilla bundle package
frascuchon Jun 7, 2024
3046985
add notebook example for dataset migration
frascuchon Jun 7, 2024
f25da20
Merge branch 'feat/rename-argilla-2-argilla-v1' into feat/argilla/ren…
frascuchon Jun 7, 2024
57753d6
Merge branch 'feat/argilla/rename-argilla-to-argilla-v1' into feat/ar…
frascuchon Jun 7, 2024
b7203ed
feat: Expose v1 SDK in argilla-sdk directly
frascuchon Jun 7, 2024
001fc2c
Merge branch 'feat/rename-argilla-2-argilla-v1' into feat/argilla/def…
frascuchon Jun 11, 2024
5b23de9
[REFACTOR] Rename `argilla-sdk` to `argilla` (#4991)
frascuchon Jun 11, 2024
94f3dfe
refactor: Moving SDK related workflows
frascuchon Jun 11, 2024
c26e317
chore: Rename argilla-sdk folder to argilla
frascuchon Jun 11, 2024
f566d71
chore: Reverting notebook
frascuchon Jun 11, 2024
96e5fc7
Corrected predictions
nataliaElv Jun 11, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
14 changes: 7 additions & 7 deletions .github/workflows/argilla-v1.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ on:

pull_request:
paths:
- "argilla/**"
- "argilla-v1/**"

release:
types:
Expand All @@ -32,7 +32,7 @@ jobs:
defaults:
run:
shell: bash -l {0}
working-directory: argilla
working-directory: argilla-v1

steps:
- name: Checkout Code 🛎
Expand All @@ -56,7 +56,7 @@ jobs:
id: cache
with:
path: ${{ env.CONDA }}/envs
key: conda-${{ runner.os }}--${{ runner.arch }}--${{ steps.get-date.outputs.today }}-${{ hashFiles('argilla/environment_dev.yml') }}-${{ env.CACHE_NUMBER }}
key: conda-${{ runner.os }}--${{ runner.arch }}--${{ steps.get-date.outputs.today }}-${{ hashFiles('argilla-v1/environment_dev.yml') }}-${{ env.CACHE_NUMBER }}

- name: Update environment
if: steps.cache.outputs.cache-hit != 'true'
Expand Down Expand Up @@ -97,8 +97,8 @@ jobs:
- name: Upload package artifact
uses: actions/upload-artifact@v4
with:
name: argilla
path: argilla/dist
name: argilla-v1
path: argilla-v1/dist

# This job will upload a Python Package using Twine when a release is created
# For more information see:
Expand All @@ -114,15 +114,15 @@ jobs:
defaults:
run:
shell: bash -l {0}
working-directory: argilla
working-directory: argilla-v1

steps:
- name: Checkout Code 🛎
uses: actions/checkout@v4
- name: Download python package
uses: actions/download-artifact@v4
with:
name: argilla
name: argilla-v1
path: dist
- name: Publish Package to TestPyPI 🥪
uses: pypa/gh-action-pypi-publish@release/v1
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,8 @@ concurrency:
on:
push:
paths:
- "argilla-sdk/docs/**"
- "argilla-sdk/src/**"
- "argilla/docs/**"
- "argilla/src/**"
branches:
# - "main"
# - "develop"
Expand All @@ -19,17 +19,16 @@ on:

defaults:
run:
working-directory: argilla-sdk
working-directory: argilla

jobs:
publish:
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
python-version: [ "3.10" ]
python-version: ["3.10"]
steps:

- name: checkout docs-site
uses: actions/checkout@v4
with:
Expand All @@ -43,7 +42,7 @@ jobs:
python-version: ${{ matrix.python-version }}
cache: true
cache-dependency-path: |
argilla-sdk/pyproject.toml
argilla/pyproject.toml

- name: Install dependencies
run: |
Expand Down
24 changes: 12 additions & 12 deletions .github/workflows/argilla-sdk.yml → .github/workflows/argilla.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# This workflow will install Python dependencies, run tests and lint with a variety of Python versions
# For more information see: https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-python
name: Build and publish the `argilla-sdk` python package
name: Build and publish the `argilla` sdk python package

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
Expand All @@ -15,17 +15,17 @@ on:

push:
paths:
- "argilla-sdk/**"
- "!argilla-sdk/docs/**"
- "argilla/**"
- "!argilla/docs/**"
branches:
- "develop"
- "main"
- "feat/**"

pull_request:
paths:
- "argilla-sdk/**"
- "!argilla-sdk/docs/**"
- "argilla/**"
- "!argilla/docs/**"
branches:
- "develop"
- "main"
Expand All @@ -37,7 +37,7 @@ on:

defaults:
run:
working-directory: argilla-sdk
working-directory: argilla

jobs:
build:
Expand Down Expand Up @@ -65,7 +65,7 @@ jobs:
python-version: ${{ matrix.python-version }}
cache: true
cache-dependency-path: |
argilla-sdk/pyproject.toml
argilla/pyproject.toml
- name: Install dependencies
run: |
pdm install
Expand All @@ -86,10 +86,10 @@ jobs:
# Upload the package to be used in the next jobs only once
if: ${{ matrix.python-version == '3.8' }}
with:
name: argilla-sdk
path: argilla-sdk/dist
name: argilla
path: argilla/dist

# This job will publish argilla-sdk package into PyPI repository
# This job will publish argilla package into PyPI repository
publish_release:
name: Publish Release
runs-on: ubuntu-latest
Expand All @@ -116,8 +116,8 @@ jobs:
- name: Download python package
uses: actions/download-artifact@v4
with:
name: argilla-sdk
path: argilla-sdk/dist
name: argilla
path: argilla/dist

- name: Setup PDM
uses: pdm-project/setup-pdm@v4
Expand Down
43 changes: 1 addition & 42 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,47 +2,6 @@ repos:
##############################################################################
# argilla specific hooks
##############################################################################
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v4.5.0
hooks:
- id: check-yaml
files: '^argilla/.*\.py$'
- id: end-of-file-fixer
files: '^argilla/.*\.py$'
exclude_types: [ text, jupyter ]
- id: trailing-whitespace
files: '^argilla/.*\.py$'
- repo: https://github.com/Lucas-C/pre-commit-hooks
rev: v1.5.4
hooks:
- id: insert-license
name: "Insert license header in Python source files"
files: '^argilla/.*\.py$'
args:
- --license-filepath
- license_header.txt
- --fuzzy-match-generates-todo
# - --remove-header
- repo: https://github.com/psf/black
rev: 23.10.1
hooks:
- id: black
additional_dependencies: [ "typer==0.7.0" ]
# only run for argilla/ files
files: '^argilla/.*\.py$'
args: [ --line-length=120 ]
- repo: https://github.com/astral-sh/ruff-pre-commit
rev: v0.1.3
hooks:
# Simulate isort via (the much faster) ruff
- id: ruff
files: '^argilla/.*\.py$'
args:
- --select=I
- --fix
##############################################################################
# argilla-sdk specific hooks
##############################################################################
- repo: https://github.com/Lucas-C/pre-commit-hooks
rev: v1.5.4
hooks:
Expand Down Expand Up @@ -87,5 +46,5 @@ ci:
autoupdate_branch: ""
autoupdate_commit_msg: "[pre-commit.ci] pre-commit autoupdate"
autoupdate_schedule: weekly
skip: [ ]
skip: []
submodules: false
55 changes: 0 additions & 55 deletions argilla-sdk/README.md

This file was deleted.

70 changes: 0 additions & 70 deletions argilla-sdk/pyproject.toml

This file was deleted.

22 changes: 0 additions & 22 deletions argilla-sdk/src/argilla_sdk/_api/__init__.py

This file was deleted.

18 changes: 0 additions & 18 deletions argilla-sdk/src/argilla_sdk/records/_io/__init__.py

This file was deleted.

Loading