SyntheticDatasets.jl icon indicating copy to clipboard operation
SyntheticDatasets.jl copied to clipboard

[#1] - Feature/generate checkerboard

Open Conradox opened this issue 5 years ago • 4 comments

Required sub-tasks for this task:

  • [x] Add generate_checkerboard function.
  • [x] Add docstring.
  • [x] Add information in the readme.
  • [x] Add tests.

Conradox avatar Sep 06 '20 04:09 Conradox

Codecov Report

Merging #24 into master will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #24   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines           36        40    +4     
=========================================
+ Hits            36        40    +4     
Impacted Files Coverage Δ
src/SyntheticDatasets.jl 100.00% <100.00%> (ø)
src/sklearn.jl 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ecf7441...0fae669. Read the comment docs.

codecov[bot] avatar Sep 06 '20 04:09 codecov[bot]

@filipebraida, nesse gerador acontece o retorno de três matrizes, então aquele convert não funciona neste caso. Por causa disso criei uma outra função convert e queria que você desse uma olhada nela.

Conradox avatar Sep 06 '20 05:09 Conradox

Esse caso me colocou para pensar igual ao generate_low_rank_matrix.

Não é um problema clássico de um dataset convencional. Não sei se faz muito sentido o DataFrame. Será que vale a pena voltar Tuples de DataFrame? @Conradox

filipebraida avatar Sep 08 '20 14:09 filipebraida

Neste ultimo commit mudei o retorno desta função pra uma tupla, acho que realmente assim fica melhor.

Conradox avatar Sep 10 '20 22:09 Conradox