-
Notifications
You must be signed in to change notification settings - Fork 0
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
Refatoração da lista de Eventos de predição e Filtros #750
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…oi removido a autenticação de algumas páginas
…entos de preodição e filtro
…l public pages
…) has been hidden
…nto 734-filter-events
Closed
glaubervila
added a commit
that referenced
this pull request
Jan 15, 2024
* A method was implemented that calculates the prediction path for occultation events. #734 * Implemented first version of user location x event visibilit in Occultation endpoint #734 * Changed Map folder limit. Added more occultation path fields. * Added maps folder highlights * Commands were created to perform the creation of maps and occultation paths manually. * Fix nightside computation and addition lat max min * Foi reescrito os componentes PredictionDataGrid e PredictionFilter. Foi removido a autenticação de algumas páginas * Filters by date, magnitude, name, dynclass and base_dynclass Done * Removed Material ui icons 4 * Replaced material ui card for MUI card * Geo Filter parcialmente implementado * Geo Filter implmentado e funcionando * Lista de eventos de predição e filtros funcionando corretamente * Events datagrid e filter separados como components * finished filter and events datagrid components * Página publica foi atualizada com os novos componentes de lista de eventos de preodição e filtro * implement prediction highlights * reorganization of cards, hidden aladdin * Fixed highlihts * Fix map highlight when asteroid name have - * Fixed asteroid alias when asteroid have - in name * Closes #728 - 'LIneA is supported by' and 'Footer' are together on all public pages * Closes #737 - reorganization of cards as agreed and 'sky-map' (Aladin) has been hidden * Fixed geo filter * Change map size in list events * Fixed map size in prediction list. * Changed pagesize in geofilter * Closes #726 - layout for displaying prediction highlights * Added devcontainer compose file * Change default radius * Changed line height * Added Hide/Show Columns toolbar * Highlight cards formating and color * Text updated in prediction filter * Prediction occultation table alignment * Occultation filter text fix * Cards text formating * Column CA Instant now formated in UTC * Minor fix * Fixed prediction occultation submit interface --------- Co-authored-by: rcboufleur <rcboufleur@gmail.com> Co-authored-by: Jandson Vitorino <jandson1512@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Foi refeito o endpoint Occultations que retorna a lista de eventos e permite filtros.
Os filtros executados no banco de dados tem prioridade sobre os filtros calculados. uma lista com exemplos de requisições para o endpoint está disponivel no arquivo: eventos_filtros.
Filtros por posição do usuario, estão com tamanho de página fixo em 10, cheguei a testar com 25 e funciona bem. leva 1min mais ou menos.
No futuro podemos acelerar o calculo de visibilidade fazendo ele em paralelo. essa funcionalidade ainda tem bastante margem para melhoria de performance, também pode ser implementado uma resposta em stream.
O layout do formulário precisa de melhorias, não deu para dedicar tempo a deixar bonito. mas acredito que o @jandsonrj possa resolver isso no futuro.
@rcboufleur Esta implementação tem uma falha que me incomodou, quando a requisição demora um pouco as informações na tela não estão atualizando direito. é simples resolver mas no momento não tinha mas tempo para concertar isso. mas não atrapalha os resultados.
essa implementação tb pode ter impactados outras telas, já que algumas libs precisaram ser atualizadas/removidas.
mas é melhor separar essas correções em issues novas.