the-algorithm-ml

the-algorithm-ml copied to clipboard

Store returned metrics in variables prior to return statements.

3

I'm stepping through the code and there's a lot here. In general, it's a fantastic step in the right direction and I appreciate the willingness to open source the system....

MarcusSorealheis

Improving README Formatting and Clarity for Increased Accessibility

2

This pull request aims to improve the readability and standardization of the project's README file, in order to make it more accessible and user-friendly for potential contributors and users. -...

faeezshaikh

List removed not used

prasadjivane

custom_training_loop.py

1

clean - removed import

prasadjivane

Hi, I would like to have the ability to search for tweets that I have seen on Twitter. This feature would allow me to easily find and reference tweets that...

hzahera

I just put switch case instead of elif

1

I am a student so i just proposed my idea , might be stupid and less but that what i understood rightnow , also it assume that only one condition...

aayankk

Code cleanup: Replaced 'idx' with '_'. Removed unused parameter & unused import

1

1. The `idx` variable is not used in the code after it is defined, so it can be removed without affecting the behavior of the code. The `idx` should be...

faeezshaikh

Reformat README.md

1

Make this file more readable

yang1fan2

Class DropoutConfig has been defined in line 12 and 113 with the exact same definition.

LakshitF

… is currently ongoing as per @jjh42 in https://github.com/twitter/the-algorithm-ml/pull/46#issuecomment-1493037363 might as well have a meta issue for that, both to track AND to avoid further contextually pointless pull requests

no-identd