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

Adiciona link para visualizar os top100 artigos acessados (coleção e periódico) #265

Open
wants to merge 56 commits into
base: master
Choose a base branch
from

Conversation

pitangainnovare
Copy link
Contributor

@pitangainnovare pitangainnovare commented Jun 26, 2024

O que esse PR faz?

A PREENCHER

Onde a revisão poderia começar?

A partir do commit de9e243. Os anteriores fazem parte do PR #263

Como este poderia ser testado manualmente?

A PREENCHER

Algum cenário de contexto que queira dar?

  • O parâmetro TIMEOUT foi ajustado para 20 segundos, que é o tempo que o SciELO Usage API usualmente leva para responder a queries mais pesadas (relacionadas a idiomas e países). Quando esses elementos forem migrados para índice Solr, os dados serão mais rapidamente obtidos. O TIMEOUT, no contexto do fetch_data, não é o expoente, mas sim o tempo que um requests.get (ou post) espera pela resposta. No contexto do fetch_data (em requests_utils.py), os retries baseados no tenacity são exponenciais e iniciam em 4, como indicado em https://tenacity.readthedocs.io/en/latest/#waiting-before-retrying. Esse é o tempo para fazer uma nova tentativa, configuradas como 5 ao todo, intervaladas de maneira exponencial (sem relação com o TIMEOUT).
  • É necessário ter um SciELO Usage funcional com os dados do PR Cria índice metrics para apoiar SciELO Analytics usage#27
  • É importante fazer com que o container analytics-webapp-1 enxergue a rede em que a aplicação Usage está inserida. Em ambiente dev, isso é feito por meio de docker network connect usage_default analytics-webapp-1.
  • Também é necessário indicar no development.ini o endereço IP do índice Solr/Usage. Para isso, execute docker inspect scielo_usage_local_solr e anote o valor de IPAddress (por exemplo, "172.18.0.7"). Esse valor deve ser preenchido em development.ini-TEMPLATE, na seção [app:main] por meio da linha usage_solr_api_host = http://172.18.0.7:8983. Note a inclusão da porta 8983.

Screenshots

Menu de acesso
Screenshot_20240625_195218

Top 100 artigos acessados para uma coleção
Screenshot_20240625_205948

Top 100 artigos acessados para um periódico
Screenshot_20240625_210115

Quais são tickets relevantes?

N/A

Referências

N/A

…ocale/analytics.pot -D analytics -d analytics/locale
rafaelpezzuto and others added 25 commits June 14, 2024 20:15
…e params - facets é um JSON grande, que, se enviado junto ao params, fica truncado)
@pitangainnovare
Copy link
Contributor Author

@robertatakenaka Criei a estrutura para apresentar os top100 articles. Quando for possível, preencherei o corpo do PR. Creio que é possível revisar o PR.

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