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

LogisticRegression Models #293

Merged
merged 1 commit into from
Oct 7, 2023
Merged

LogisticRegression Models #293

merged 1 commit into from
Oct 7, 2023

Conversation

kyleskom
Copy link
Owner

@kyleskom kyleskom commented Oct 7, 2023

Not seeing great results with these

@kyleskom kyleskom merged commit 1114beb into master Oct 7, 2023
@STRATZ-Ken
Copy link

@kyleskom A couple of things worth noting as I continue my personal script from last year. Your random state should be adjust randomly as well. When training, the initial set of weights could offset the initial values incorrectly. So you should test training with random values initially to see which are good. I also did not see great results when testing LR. There are too many connections that LR models don't handle well.

@kyleskom
Copy link
Owner Author

kyleskom commented Oct 7, 2023

I played around with the random a bunch as well. I couldn't find any that would come close to the other model so I'm just going to leave it as be for now.

@STRATZ-Ken
Copy link

I would suggest a save to file that writes the output and the accuracy of each model. Some models are trained a million times before a great one will appear. Human recording the values is not possible. Its the same for your NN models. You have a set number of neurons for each layer. However, what if 4 layers is better, or 3, or 512 neurons, higher or lower learning rate etc. All variations should be set and tested then record (Some should be tested 3-4 times to ensure they were just not a fluke).

@kyleskom
Copy link
Owner Author

kyleskom commented Oct 8, 2023

I have played around with all that but your right, I do need a better way to test and document models.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants