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

New examples & tutorials #77

Merged
merged 12 commits into from
Sep 25, 2023
Merged

New examples & tutorials #77

merged 12 commits into from
Sep 25, 2023

Conversation

jrzkaminski
Copy link
Collaborator

@jrzkaminski jrzkaminski commented Aug 14, 2023

  1. New tutorials, old ones are removed.
  2. Some minor fixes

@jrzkaminski jrzkaminski added the documentation Improvements or additions to documentation label Aug 14, 2023
@jrzkaminski jrzkaminski self-assigned this Aug 14, 2023
@jrzkaminski jrzkaminski changed the title old tutorials removed New examples & tutorials Aug 14, 2023
@codecov-commenter
Copy link

codecov-commenter commented Aug 14, 2023

Codecov Report

Merging #77 (4059e66) into master (390e2da) will increase coverage by 0.12%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master      #77      +/-   ##
==========================================
+ Coverage   65.02%   65.15%   +0.12%     
==========================================
  Files          43       43              
  Lines        2891     2896       +5     
==========================================
+ Hits         1880     1887       +7     
+ Misses       1011     1009       -2     
Files Changed Coverage Δ
bamt/utils/composite_utils/MLUtils.py 86.27% <85.71%> (+1.49%) ⬆️

... and 1 file with indirect coverage changes

# Conflicts:
#	tutorials/Nodes.ipynb
#	tutorials/Preprocessing.ipynb
#	tutorials/example_bigbravebn.ipynb
#	tutorials/example_geo.ipynb
#	tutorials/example_socio.ipynb
#	tutorials/parameters_learning_and_sampling.ipynb
#	tutorials/regressor_exp.ipynb
#	tutorials/structure_learning.ipynb
@jrzkaminski jrzkaminski marked this pull request as ready for review September 5, 2023 11:01
@Anaxagor
Copy link
Collaborator

Не очень поняла прикладные примеры, кажется, что оба примера про отбор признаков? Хотя по описанию второй пример должен быть про то, что с помощью БС можно нагенерить дополнительных тренировочных данных, но этого в примере не увидела.

@Anaxagor
Copy link
Collaborator

Ещё в документации есть пустые разделы, думаю, что их стоит убрать пока вообще.

@jrzkaminski
Copy link
Collaborator Author

Не очень поняла прикладные примеры, кажется, что оба примера про отбор признаков? Хотя по описанию второй пример должен быть про то, что с помощью БС можно нагенерить дополнительных тренировочных данных, но этого в примере не увидела.

Примеры действительно похожи, однако во втором примере мы генерим дополнительные данные и добавляем их к основным без feature selection. Я действительно там забыл поменять название подсекции, поправлю

data_concatenated = pd.concat([data_bamt, sampled_data], ignore_index=True)

@jrzkaminski
Copy link
Collaborator Author

jrzkaminski commented Sep 12, 2023

Ещё в документации есть пустые разделы, думаю, что их стоит убрать пока вообще.

уберу

@Anaxagor Anaxagor self-requested a review September 23, 2023 10:44
Anaxagor
Anaxagor previously approved these changes Sep 23, 2023
@jrzkaminski jrzkaminski merged commit dce46f7 into master Sep 25, 2023
4 checks passed
@Roman223 Roman223 deleted the examples-rework branch October 26, 2023 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
4 participants