STAR-RG / practical_testing_book

https://damorimrg.github.io/practical_testing_book/
16 stars 13 forks source link

Notebook code has no file extension metadata #14

Closed ViniciusSiqueira54 closed 4 years ago

ViniciusSiqueira54 commented 4 years ago

Descrição: Ao buildar o livro os seguintes WARNINGs estão sendo disparados:

/home/siqueira/practical_testing_book/testgeneration/combinatorial.ipynb: WARNING: Notebook code has no file extension metadata, defaulting to `.txt`
/home/siqueira/practical_testing_book/testgeneration/propertybased.ipynb: WARNING: Notebook code has no file extension metadata, defaulting to `.txt`

Resultado atual: WARNINGs relacionados a extensão dos arquivos .ipynb estão sendo exibidos durante a construção do livro

Resultado esperado: WARNINGs relacionados a extensão dos arquivos .ipynb não deveriam ser disparados

Nota: Outros capítulos que utilizam esse mesmo formato de arquivo não disparam esses WARNINGs

ViniciusSiqueira54 commented 4 years ago

Pessoal consegui resolver esses warnings atualizando alguns dos metadatas dos arquivos de vcs, e limpando os outputs das celulas de vcs.

Vcs precisam exibir esses outputs pro entendimento dos capitulos de vcs? Caso positivo tenho um backup aqui dos arquivos de vcs e posso desfazer as alterações.

rafaelmotaalves commented 4 years ago

@ViniciusSiqueira54 precisamos sim dos outputs do código, esse é o objetivo de usar o python notebook certo? Além disso alguma das suas mudanças fizeram o build automático parar de funcionar, como você pode ver aqui: https://github.com/damorimRG/practical_testing_book/runs/936070273

você poderia por favor voltar as alterações?

ViniciusSiqueira54 commented 4 years ago

@rafaelmotaalves posso sim, sem problemas.

Não tenho certeza quanto a utilização do jupyter se é pra isso, pelo que entendi seria pra o leitor utilizar e ele mesmo criar os outputs dele, mas dai não tenho certeza se o objetivo é esse mesmo, enfim.

Essa questão do build automático eu acho que ta quebrado de antes, @fehaa relatou que teve problemas com ele também, mesmo antes dessa alteração, toda vez que buildo e faço o deploy ele da algum erro, postamos até lá no classroom.

rafaelmotaalves commented 4 years ago

ok @ViniciusSiqueira54 , sobre o problema de build me parece que tem a ver com a versão do jupyter-book, você poderia ver se você está usando a mesma versão do script? Caso não, podemos atualizar o script para usar essa versão.

ViniciusSiqueira54 commented 4 years ago

@rafaelmotaalves eles lançaram duas releases novas esses dias, se tiver como tu atualizar seria uma boa. :)

rafaelmotaalves commented 4 years ago

Pronto, está atualizado

ViniciusSiqueira54 commented 4 years ago

Valeu @rafaelmotaalves , voltei o arquivo de backup do combinatorial.ipynb pro repositório. Obrigado pelo Help com o script de build.

rafaelmotaalves commented 4 years ago

Obrigado @ViniciusSiqueira54 !

gabritto commented 4 years ago

@ViniciusSiqueira54 preciso do output no arquivo propertybased.ipynb, acho que retirar outputs não é uma boa solução. Reverti a mudança no arquivo. Sobre o warning, acho que pode ser ignorado sem problemas, porque a build funciona e tudo parece ok no site. Porém fui investigar a razão do warning. Ao que parece, o jupyter book espera uma informação que o Colab não inclui no notebook gerado. Comparei o que o Binder gera com o que o Colab gera, e acrescentando a seção "language_info" na seção "metadata" do notebook o warning desaparece:

"metadata": {
    "language_info": {
          "name": "python",
          "version": "3.7.8",
          "mimetype": "text/x-python",
          "codemirror_mode": {
              "name": "ipython",
              "version": 3
          },
          "pygments_lexer": "ipython3",
          "nbconvert_exporter": "python",
          "file_extension": ".py"
    }
}
ViniciusSiqueira54 commented 4 years ago

Oi @gabritto, desculpa a demora no retorno, ótimo então.. Sendo assim podemos considerar esse issue fixed.

Obg.