[PR #130] [CLOSED] def get_data(data_url, data_file) #193

New Issue

GiteaMirror · 2025-11-02T00:08:46-05:00

GiteaMirror commented

2025-11-02 00:08:46 -05:00

📋 Pull Request Information

Original PR: https://github.com/GokuMohandas/Made-With-ML/pull/130
Author: @tomcohen0079
Created: 2/26/2019
Status: ❌ Closed

Base: master ← Head: master

📝 Commits (10+)

b60357e Merge pull request #27 from GokuMohandas/advanced_rnns
c4120d4 updated advanced rnn notebook
3267109 Merge pull request #28 from GokuMohandas/advanced_rnns
8c4f447 finished advanced rnn notebook
f15beb7 Merge pull request #29 from GokuMohandas/advanced_rnns
4552a33 updated image sizes
2d99160 Merge pull request #30 from GokuMohandas/advanced_rnns
0c759fd started cv notebook
62a8fa0 started cv notebook
2efe2b7 added intro image

📊 Changes

1 file changed (+2228 additions, -0 deletions)

View changed files

➕ notebooks/Copy_of_10_Object_Oriented_ML.ipynb (+2228 -0)

📄 Description

in the components part , the get_data function asks for a url that we provide later through the config file , but in the function we dont use the url that we provided as a parameter, but we go directly to the config file , meaning no matter what we put as a parameter , it wont matter

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/GokuMohandas/Made-With-ML/pull/130 **Author:** [@tomcohen0079](https://github.com/tomcohen0079) **Created:** 2/26/2019 **Status:** ❌ Closed **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (10+) - [`b60357e`](https://github.com/GokuMohandas/Made-With-ML/commit/b60357ecd75346692a60ca7a7ae0bf24a3672cc0) Merge pull request #27 from GokuMohandas/advanced_rnns - [`c4120d4`](https://github.com/GokuMohandas/Made-With-ML/commit/c4120d49c00ebcd297f0538fac1fb645f951f618) updated advanced rnn notebook - [`3267109`](https://github.com/GokuMohandas/Made-With-ML/commit/32671094d42fdde2c49d23b314e7465a9e74d9b2) Merge pull request #28 from GokuMohandas/advanced_rnns - [`8c4f447`](https://github.com/GokuMohandas/Made-With-ML/commit/8c4f4474b6772d45406673015b2c986f610477d6) finished advanced rnn notebook - [`f15beb7`](https://github.com/GokuMohandas/Made-With-ML/commit/f15beb70a999b82ca6ae66236ecb30ec06d0aca7) Merge pull request #29 from GokuMohandas/advanced_rnns - [`4552a33`](https://github.com/GokuMohandas/Made-With-ML/commit/4552a33105000d563fcb16986af2eafc044dd75d) updated image sizes - [`2d99160`](https://github.com/GokuMohandas/Made-With-ML/commit/2d9916009f7a3ff17d474af5e35aaf67c78dac04) Merge pull request #30 from GokuMohandas/advanced_rnns - [`0c759fd`](https://github.com/GokuMohandas/Made-With-ML/commit/0c759fd411bc10dd918d284b171dace9290264d9) started cv notebook - [`62a8fa0`](https://github.com/GokuMohandas/Made-With-ML/commit/62a8fa0a61a149203ea8c6970f16873de10fe243) started cv notebook - [`2efe2b7`](https://github.com/GokuMohandas/Made-With-ML/commit/2efe2b71e1866b363c9e574c87fa802b459e0f2a) added intro image ### 📊 Changes **1 file changed** (+2228 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `notebooks/Copy_of_10_Object_Oriented_ML.ipynb` (+2228 -0) </details> ### 📄 Description in the components part , the get_data function asks for a url that we provide later through the config file , but in the function we dont use the url that we provided as a parameter, but we go directly to the config file , meaning no matter what we put as a parameter , it wont matter --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2025-11-02 00:08:46 -05:00

GiteaMirror closed this issue

2025-11-02 00:08:46 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/Made-With-ML#193