Log Format - Githubissues

CIAT-DAPA / subsets_genebank_accessions

Web tool which allows creating subsets of accessions based on indicators

0 stars 1 forks source link

Log Format #39

Closed stevensotelo closed 3 years ago

stevensotelo commented 3 years ago

The logs output should have the following format

TYPE | DATE | DESCRIPTION

TYPE: INFO | ERROR DATE: yyyy-MM-dd HH:mm:ss

Example:

INFO | 2020-12-04 21:46:19 | Getting accessions passport data for crop(s): african yam bean ERROR | 2020-12-04 21:46:29 | Error in curl::curl_fetch_memory(url, handle = handle): Timeout was reached: [api.genesys-pgr.org] Connection timed out after 10002 milliseconds

stevensotelo commented 3 years ago

@khadijaAouzal

[x] Create a new folder called it "tools" into indicators_scripts
[x] Create a R Script with all functions for logging
[x] Implement logging functions inside of passport module

stevensotelo commented 3 years ago

@juangarc

[x] in master script is pending

@brayan9487

[x] in the indicators code is pending

The package name is: futile.logger

stevensotelo commented 3 years ago

@juangarc and @bmora-0110

It is still pending to implement

stevensotelo commented 3 years ago

@juangarc and @bmora-0110

It is still pending to implement

stevensotelo commented 3 years ago

@juangarc and @bmora-0110

It is still pending to implement
[x] Check the source code that @khadijaAouzal implemented:

https://github.com/CIAT-DAPA/subsets_genebank_accessions/blob/dev/src/indicators_scripts/passport/get_passport_data.R

log_info('passport', message = "Getting accessions passport data for crop(s): %s", crop)

stevensotelo commented 3 years ago

@juangarc

He implemented the log format in the master script

@bmora-0110
He implemented the log format in the master script
[x] He has some issues with source code when he works with this function

stevensotelo commented 3 years ago

@bmora-0110

He solved the issues with the implementation of the function