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

PR for testing Thinc 703 #11005

Closed
wants to merge 2 commits into from
Closed

PR for testing Thinc 703 #11005

wants to merge 2 commits into from

Conversation

svlandeg
Copy link
Member

Description

dummy PR for testing purposes only - should not be merged

PR to test the CI with the Thinc branch for explosion/thinc#703

@svlandeg svlandeg added the 🔮 thinc spaCy's machine learning library Thinc label Jun 22, 2022
@svlandeg svlandeg changed the title dummy edit PR for testing Thinc 703 Jun 22, 2022
@shadeMe
Copy link
Contributor

shadeMe commented Jun 29, 2022

@explosion-bot please test_slow --thinc-branch refs/pull/703/head

@explosion-bot
Copy link
Collaborator

explosion-bot commented Jun 29, 2022

🪁 Successfully triggered build on Buildkite

URL: https://buildkite.com/explosion-ai/spacy-slow-tests/builds/130

@shadeMe
Copy link
Contributor

shadeMe commented Jun 29, 2022

PSA: Currently testing explosion-bot with this PR. In case you see a lot of related activity, keep calm and continue.

@shadeMe
Copy link
Contributor

shadeMe commented Jun 29, 2022

🪁 Successfully triggered build on Buildkite

URL: https://buildkite.com/explosion-ai/spacy-slow-tests/builds/131

@shadeMe
Copy link
Contributor

shadeMe commented Jun 29, 2022

@explosion-bot please test_slow --thinc-branch refs/pull/703/head

@explosion-bot
Copy link
Collaborator

explosion-bot commented Jun 29, 2022

🪁 Successfully triggered build on Buildkite

URL: https://buildkite.com/explosion-ai/spacy-slow-tests/builds/132

@svlandeg
Copy link
Member Author

@explosion-bot please test_slow --thinc-branch refs/pull/703/head

@explosion-bot
Copy link
Collaborator

explosion-bot commented Jun 29, 2022

🪁 Successfully triggered build on Buildkite

URL: https://buildkite.com/explosion-ai/spacy-slow-tests/builds/135

@explosion explosion deleted a comment from explosion-bot Jun 29, 2022
@explosion explosion deleted a comment from explosion-bot Jun 29, 2022
@explosion explosion deleted a comment from explosion-bot Jun 29, 2022
@explosion explosion deleted a comment from explosion-bot Jun 29, 2022
@explosion explosion deleted a comment from explosion-bot Jun 29, 2022
@svlandeg
Copy link
Member Author

svlandeg commented Jun 29, 2022

Looks like explosion/thinc#703 does in fact cause a failure in spaCy, specifically when processing an empty doc with the textcat. We'll need to look into this - probably in a non-dummy, proper PR ;-)

@svlandeg svlandeg closed this Jun 29, 2022
@svlandeg svlandeg reopened this Jun 29, 2022
@svlandeg svlandeg closed this Jun 29, 2022
@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

🚨 Errors

Usage: @explosion-bot please test_slow [OPTIONS]
Try '@explosion-bot please test_slow --help' for help.

Error: no such option: --thinc-branch

3 similar comments
@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

🚨 Errors

Usage: @explosion-bot please test_slow [OPTIONS]
Try '@explosion-bot please test_slow --help' for help.

Error: no such option: --thinc-branch

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

🚨 Errors

Usage: @explosion-bot please test_slow [OPTIONS]
Try '@explosion-bot please test_slow --help' for help.

Error: no such option: --thinc-branch

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

🚨 Errors

Usage: @explosion-bot please test_slow [OPTIONS]
Try '@explosion-bot please test_slow --help' for help.

Error: no such option: --thinc-branch

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

💬 Help for command test_slow

Usage: @explosion-bot please test_slow [OPTIONS]

  Trigger a build against the spaCy/thinc Slow Test Suite BuildKite pipeline

Options:
  --thinc_branch TEXT  Build thinc from this branch for the test.
  --spacy_branch TEXT  Build spaCy from this branch for the test. Can only be
                       used in the spaCy Transformers repo.

  --run_on TEXT        Execute this command on a different repo but with the
                       changes in the current PR. Valid source -> target repos:
                       spaCy -> spaCy Transformers, thinc -> spaCy, spaCy
                       Transformers

  --help               Show this message and exit.

4 similar comments
@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

💬 Help for command test_slow

Usage: @explosion-bot please test_slow [OPTIONS]

  Trigger a build against the spaCy/thinc Slow Test Suite BuildKite pipeline

Options:
  --thinc_branch TEXT  Build thinc from this branch for the test.
  --spacy_branch TEXT  Build spaCy from this branch for the test. Can only be
                       used in the spaCy Transformers repo.

  --run_on TEXT        Execute this command on a different repo but with the
                       changes in the current PR. Valid source -> target repos:
                       spaCy -> spaCy Transformers, thinc -> spaCy, spaCy
                       Transformers

  --help               Show this message and exit.

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

💬 Help for command test_slow

Usage: @explosion-bot please test_slow [OPTIONS]

  Trigger a build against the spaCy/thinc Slow Test Suite BuildKite pipeline

Options:
  --thinc_branch TEXT  Build thinc from this branch for the test.
  --spacy_branch TEXT  Build spaCy from this branch for the test. Can only be
                       used in the spaCy Transformers repo.

  --run_on TEXT        Execute this command on a different repo but with the
                       changes in the current PR. Valid source -> target repos:
                       spaCy -> spaCy Transformers, thinc -> spaCy, spaCy
                       Transformers

  --help               Show this message and exit.

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

💬 Help for command test_slow

Usage: @explosion-bot please test_slow [OPTIONS]

  Trigger a build against the spaCy/thinc Slow Test Suite BuildKite pipeline

Options:
  --thinc_branch TEXT  Build thinc from this branch for the test.
  --spacy_branch TEXT  Build spaCy from this branch for the test. Can only be
                       used in the spaCy Transformers repo.

  --run_on TEXT        Execute this command on a different repo but with the
                       changes in the current PR. Valid source -> target repos:
                       spaCy -> spaCy Transformers, thinc -> spaCy, spaCy
                       Transformers

  --help               Show this message and exit.

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

💬 Help for command test_slow

Usage: @explosion-bot please test_slow [OPTIONS]

  Trigger a build against the spaCy/thinc Slow Test Suite BuildKite pipeline

Options:
  --thinc_branch TEXT  Build thinc from this branch for the test.
  --spacy_branch TEXT  Build spaCy from this branch for the test. Can only be
                       used in the spaCy Transformers repo.

  --run_on TEXT        Execute this command on a different repo but with the
                       changes in the current PR. Valid source -> target repos:
                       spaCy -> spaCy Transformers, thinc -> spaCy, spaCy
                       Transformers

  --help               Show this message and exit.

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

🚨 Errors

Usage: @explosion-bot please test_slow [OPTIONS]
Try '@explosion-bot please test_slow --help' for help.

Error: no such option: --spacy-branch

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

🚨 Errors

Usage: @explosion-bot please test_slow [OPTIONS]
Try '@explosion-bot please test_slow --help' for help.

Error: no such option: --thinc-branch

1 similar comment
@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

🚨 Errors

Usage: @explosion-bot please test_slow [OPTIONS]
Try '@explosion-bot please test_slow --help' for help.

Error: no such option: --thinc-branch

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

The '--spacy-branch' option can only be used in the following repos: ['spacy-transformers']

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

This repository (spacy) is not supported by me at the moment!

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

The '--run-on' option can only be used in/with the following repos (source: target): {'spaCy': ['spacy-transformers'], 'thinc': ['spaCy', 'spacy-transformers']}

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

No 'Slow Tests' build pipeline found for the repo 'spacy-transformers'!

5 similar comments
@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

No 'Slow Tests' build pipeline found for the repo 'spacy-transformers'!

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

No 'Slow Tests' build pipeline found for the repo 'spacy-transformers'!

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

No 'Slow Tests' build pipeline found for the repo 'spacy-transformers'!

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

No 'Slow Tests' build pipeline found for the repo 'spacy-transformers'!

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

No 'Slow Tests' build pipeline found for the repo 'spacy-transformers'!

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

The '--spacy-branch' option can only be used in the following repos: ['spacy-transformers']

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

❌ The '--spacy-branch' option can only be used in the following repos: ['spacy-transformers']

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

🚨 Errors

Traceback (most recent call last):
  File "/home/madeesh/miniconda3/envs/dev/lib/python3.7/site-packages/click/testing.py", line 329, in invoke
    cli.main(args=args or (), prog_name=prog_name, **extra)
  File "/home/madeesh/miniconda3/envs/dev/lib/python3.7/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/home/madeesh/miniconda3/envs/dev/lib/python3.7/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/madeesh/miniconda3/envs/dev/lib/python3.7/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/madeesh/miniconda3/envs/dev/lib/python3.7/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/home/madeesh/miniconda3/envs/dev/lib/python3.7/site-packages/typer/main.py", line 500, in wrapper
    return callback(**use_params)  # type: ignore
  File "/home/madeesh/dev/explosion-bot/explosionbot/github_commands/test.py", line 188, in test_slow
    test_command(repo_name, test_suite, thinc_branch, spacy_branch, run_on)
  File "/home/madeesh/dev/explosion-bot/explosionbot/github_commands/test.py", line 129, in test_command
    repo = validate_and_resolve_command_parameters(repo_name, test_suite, thinc_branch, spacy_branch, run_on)
  File "/home/madeesh/dev/explosion-bot/explosionbot/github_commands/test.py", line 112, in validate_and_resolve_command_parameters
    raise ValueError(msg)
ValueError: ❌ The '--spacy-branch' option can only be used in the following repos: ['spacy-transformers']

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

🚨 Errors

❌ The '--spacy-branch' option can only be used in the following repos: ['spacy-transformers']

@shadeMe
Copy link
Contributor

shadeMe commented Jun 30, 2022

🪁 Successfully triggered build on Buildkite

URL: https://buildkite.com/explosion-ai/spacy-transformers-gpu-test-suite/builds/11

@svlandeg svlandeg deleted the svlandeg-patch-1 branch June 30, 2022 12:46
@shadeMe
Copy link
Contributor

shadeMe commented Jul 5, 2022

🚨 Errors

Usage: @explosion-bot please test_gpu [OPTIONS]
Try '@explosion-bot please test_gpu --help' for help.

Error: no such option: --spsacy-branch

1 similar comment
@shadeMe
Copy link
Contributor

shadeMe commented Jul 5, 2022

🚨 Errors

Usage: @explosion-bot please test_gpu [OPTIONS]
Try '@explosion-bot please test_gpu --help' for help.

Error: no such option: --spsacy-branch

@shadeMe
Copy link
Contributor

shadeMe commented Jul 5, 2022

🪁 Successfully triggered build on Buildkite

URL: https://buildkite.com/explosion-ai/spacy-transformers-gpu-test-suite/builds/15

@shadeMe
Copy link
Contributor

shadeMe commented Jul 5, 2022

⏳ Triggering 'GPU Tests' for 'spacy-transformers' on Buildkite...

@shadeMe
Copy link
Contributor

shadeMe commented Jul 5, 2022

🚨 Errors

Aborted!

@shadeMe
Copy link
Contributor

shadeMe commented Jul 5, 2022

🚨 Errors

The '--run-on-branch' option must/can only be used together with the '--run-on' option

@shadeMe
Copy link
Contributor

shadeMe commented Jul 5, 2022

🪁 Successfully triggered build on Buildkite

URL: https://buildkite.com/explosion-ai/spacy-transformers-gpu-test-suite/builds/16

@shadeMe
Copy link
Contributor

shadeMe commented Jul 5, 2022

🚨 Errors

current_pr can only be used in PRs of the corresponding repo

@shadeMe
Copy link
Contributor

shadeMe commented Jul 5, 2022

🚨 Errors

Traceback (most recent call last):
  File "/home/madeesh/miniconda3/envs/dev/lib/python3.7/site-packages/click/testing.py", line 329, in invoke
    cli.main(args=args or (), prog_name=prog_name, **extra)
  File "/home/madeesh/miniconda3/envs/dev/lib/python3.7/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/home/madeesh/miniconda3/envs/dev/lib/python3.7/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/madeesh/miniconda3/envs/dev/lib/python3.7/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/madeesh/miniconda3/envs/dev/lib/python3.7/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/home/madeesh/miniconda3/envs/dev/lib/python3.7/site-packages/typer/main.py", line 497, in wrapper
    return callback(**use_params)  # type: ignore
  File "/home/madeesh/dev/explosion-bot/explosionbot/github_commands/test.py", line 225, in test_gpu
    test_command(params)
  File "/home/madeesh/dev/explosion-bot/explosionbot/github_commands/test.py", line 182, in test_command
    pipeline, bk_queue = TEST_PIPELINES[(params.repo, params.test_suite)]
AttributeError: 'Repo' object has no attribute 'repo'

@shadeMe
Copy link
Contributor

shadeMe commented Jul 5, 2022

🪁 Successfully triggered build on Buildkite

URL: https://buildkite.com/explosion-ai/spacy-transformers-gpu-test-suite/builds/17

@shadeMe
Copy link
Contributor

shadeMe commented Jul 5, 2022

🪁 Successfully triggered build on Buildkite

URL: https://buildkite.com/explosion-ai/spacy-transformers-gpu-test-suite/builds/18

@shadeMe
Copy link
Contributor

shadeMe commented Jul 5, 2022

🚨 Errors

This repository (spacy-transformsers) is not supported by me at the moment!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔮 thinc spaCy's machine learning library Thinc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants