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

Update README.md #7

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cndesantana
Copy link
Contributor

Adding instructions to run R version of the scraper

Adding instructions to run R version of the scraper
Copy link
Contributor

@anapaulagomes anapaulagomes left a comment

Choose a reason for hiding this comment

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

É uma boa mesmo ter mais informações sobre como executar com R! 👏🏽

@@ -6,7 +6,7 @@ Fonte: http://www.feiradesantana.ba.gov.br/coronavirus/vacinometro/

## Desenvolvimento

Você precisará do Poetry. Instale as dependências com:
Para utilizar o scrapy em Python, você precisará do Poetry. Instale as dependências com:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Para utilizar o scrapy em Python, você precisará do Poetry. Instale as dependências com:
Para executar o raspador em Python, você precisará do Poetry. Instale as dependências com:

@@ -17,3 +17,14 @@ Execute o raspador com:
```
poetry run scrapy crawl vaccines
```

Para utilizar o scrapy em R, as dependências são instaladas automaticamente:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Para utilizar o scrapy em R, as dependências são instaladas automaticamente:
Para executar o raspador em R, as dependências são instaladas automaticamente.

Scrapy é o nome da biblioteca em Python. Pode ficar confuso usar nesse contexto. :)

Para executar, você deve rodar o comando:

```
Rscript scraper.R
Copy link
Contributor

Choose a reason for hiding this comment

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

Tem que instalar esse Rscript, não? Seria bom informar um link pra isso.

@anapaulagomes
Copy link
Contributor

@cndesantana fizemos algumas modificações no README. Tem que atualizar a branch, por favor.

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