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

Adding Export tests function #3

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Adding Export tests function #3

wants to merge 2 commits into from

Conversation

rsogit
Copy link
Contributor

@rsogit rsogit commented Jan 5, 2021

Nesse PR eu adicionei uma função para exportar as frases de teste e corrigi algumas coisas relacionadas ao cadastro de entidades na função de criar frases de teste.

… with entities; Adjusting a language issue in create_example;
Copy link
Contributor

@PeterAlexandre PeterAlexandre left a comment

Choose a reason for hiding this comment

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

Sobe essa alteração que comentei, que fica completo!

O PR estava bom e ainda bem que você continua usando o código! Valeu!

@@ -51,6 +51,21 @@ def export_to_json(filename, *args):
with open(f'{filename}.json', 'w') as outfile:
json.dump(examples, outfile)

def export_evaluate_to_json(filename, *args):
Copy link
Contributor

Choose a reason for hiding this comment

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

Eae, @rsogit! Tava pensando aqui, e vi que essa função é igual a da linha 39 export_to_json, mudando apenas o final do endpoint ("examples/" ou "evaluate/").

Será que ficaria melhor permanecer com uma só função, e adicionar um parâmetro para receber o fim do endpoint examples ou evaluate?

Tava pensando em algo mais ou menos assim:

def export_to_json(filename, endpoint='examples', *args):
    ...
    results = get_all_examples(
        ...
        # alterando o final da string com a variável endpoint
        next_call=f'{base_url}/repository/{endpoint}/'

Para funcionar, chamaríamos no terminal

python main.py nome_arquivo evaluate

Edit 1

Ou algo assim, que considero com menos chances de erro:

def export_to_json(filename, evaluate=False, *args):
    # Getting all sentences from bothub repository
    endpoint = 'examples'
    if evaluate:
        endpoint = 'evaluate'

    results = get_all_examples(
        ...
        # alterando o final da string com a variável endpoint
        next_call=f'{base_url}/repository/{endpoint}/'

Para funcionar, chamaríamos no terminal

python main.py export_to_json nome_arquivo evaluate=True

O que acha? Se puder, aproveita que tá com o código por ai, e aplica essa melhoria, se achar necessário.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants