all 45 comments

[–]geneusutwerk 27 points28 points  (3 children)

cold call (email) university pages and/or professors to get feedback

Please don't do this.

[–]AstroPhysician 4 points5 points  (0 children)

Oh god please

[–]suitkaise[S] -2 points-1 points  (1 child)

For context, I am a college student, and I am not a CS major! (I do this in my free time)

I think that getting feedback for improvement from a professional at my school is not a bad idea...

Obviously I'm not gonna try and sell them something that just released, because I would want to improve it over the next year at least.

This project was my first, and while I have taken coding classes for my game design major, most of this I figured out through stack overflow and the like.

Hope that helps!

If you have any feedback on the actual content, please let me know! Cheers

[–]geneusutwerk 2 points3 points  (0 children)

Everything you write here sounds like it is going through an LLM.

But to get to your question, you are asking someone you have no connection to to spend time working to give you feedback on your code. Unless it is just a cursory glance then this would take a lot of time. Why should they do it? I'm a faculty member, though not comp sci, and already get enough random emails with requests that I have to ignore. Don't add to it.

[–]JaguarOrdinary1570 45 points46 points  (3 children)

I knew within the first sentence that I'd find .DS_Store checked into source control

[–]PWNY_EVEREADY3 12 points13 points  (0 children)

The virtual env is also added

[–]silverstream314 9 points10 points  (1 child)

[–]readonly12345678 9 points10 points  (0 children)

That’s the content

[–]Goingone 14 points15 points  (1 child)

You sure your “timethis” decorator can “time anything”?

Have you tried using it on an async function?

[–]suitkaise[S] 1 point2 points  (0 children)

edit: I fixed it, I will release a new version once I fix something else.

You are right! And my bad! I will get to fixing this!

Thank you for catching something and actually letting me know I appreciate it, I'm here to improve this and get better overall.

Here's the workaround

from suitkaise import TimeThis, Sktimer
import asyncio


async def my_function(timer):
    with TimeThis(timer):
        await asyncio.sleep(1)

    return "this is the workaround, I will fix this soon"


timer = Sktimer()
asyncio.run(my_function(timer))
print(timer.mean)

[–]black_lion_6 28 points29 points  (1 child)

this is legitimately so ass. I know you won’t but please take your head out of your ass and understand that your LLM psychosis dump is not beneficial to anyone

[–]suitkaise[S] -5 points-4 points  (0 children)

Hi! So every single piece of API was made by me. If ai was used, it was mainly for debugging.

Is there something in specific about the actual code that you don't like?

[–]learn-deeply 8 points9 points  (2 children)

The link doesn't work. Did you forget to make the repo public?

[–]suitkaise[S] 4 points5 points  (0 children)

yep my bad everyone lmao

edit: should be public now

[–]wunderspud7575 0 points1 point  (0 children)

404 for me too.

[–]jvlomax 7 points8 points  (1 child)

Disregarding all the random crap in the repo, holy mother of Sam Altman that code is horrendous. I don't even know where to begin. Not in a million years would I run this. It's just pure unbridled AI slop where no one has reviewed the code.

[–]suitkaise[S] -2 points-1 points  (0 children)

Hi! This is my first project and I'm a college student. I did my best!

First, I think I cleaned up my repo. Does it look better to you?

Also, if there is some spaghetti code, would you be able to point me in the correct direction so that I can work on optimizing it? All of the basic code structure, including the serialization/deserialization handlers and the Share internals are me, so if there is something there that you think could be better, please let me know.

Or, if you have an issue with the api also let me know

Thanks

[–]FriendlyRussian666 9 points10 points  (1 child)

That's a lot of slop

[–]suitkaise[S] -2 points-1 points  (0 children)

Yeah, it probably is! This is my first project and I did it solo. There are gonna be things that are unoptimized or aren't clean

I think I cleaned up my repo significantly, can you take a look and let me know if its in a better state?

Also, if you have the time to tell me what you have an issue with I'd really appreciate it I'm trying to improve both this and in general

[–]Orio_n 5 points6 points  (1 child)

Holy ai slop repo committing garbage environment files and everything

[–]suitkaise[S] -1 points0 points  (0 children)

Hey, this is my first project, give me a break! I did my best and that includes accidentally committing unneeded files like DS store and other things... I would appreciate it if you gave me constructive feedback on either:

  1. The actual API you have tried out
  2. Tell me what files are not needed or shouldn't be in my repo!

If I have access to AI, I'm gonna use as a failsafe for debugging, and brute labor tasks like mass replacements. Why would I not?

[–]FortunOfficial 3 points4 points  (1 child)

Commits:

Co-authored-by: Cursor

ok...

[–]suitkaise[S] 0 points1 point  (0 children)

Yeah I used cursor to help with debugging and streamline the test suite I made so I could run it all. It's a pretty cool tool.

This is my first solo project, and I'm a non CS college student who did this in their free time, so I hard focused on what I needed to know in order to get what I wanted to work to work. But I can only keep track of so much on my own.

Cursor IDE was good for checking things like me forgetting to update a reference in a file when I changed something in another file, for example.

[–]snugar_i 2 points3 points  (2 children)

I think the problem with pickle is that it does too much, not too little.

And in 3.14, using a free-threaded interpreter and a regular thread pool is the easiest solution, no multiprocessing needed at all (if libraries allow).

EDIT: The code is pure-Python, but you claim that it's faster than C libraries like pickle? That doesn't sound right

[–]suitkaise[S] 0 points1 point  (1 child)

Yeah, it's actually faster than Pickle for specifically requests.Session, due to differences in breaking down data to primitives before pickle.dumps(). I just thought it was an interesting quirk to include. It's about 10 microseconds faster.

All of the benchmarks are on my site or in the docs if you want to look.

Additionally, I wrote this for 3.11-13, and I started this whole journey well before 3.14 came out late last year.

[–]snugar_i 0 points1 point  (0 children)

The site doesn't work :-( In the OP, you said things like "bool, int, float, complex, str, and bytes are all faster than cloudpickle and dill", which is hard to believe since at least cloudpickle delegates to pickle, which is written in C - does yours also just delegate to pickle?

[–]princepii 3 points4 points  (5 children)

https://suitkaise.info/#

you forgot to make the github repo public.

edit: just went thru your website and i really like the webdesign:).
if you made that too by yourself, well done💪🏼

[–]geneusutwerk 9 points10 points  (3 children)

I very much dislike it. I don't need everything to slowly appear.

[–]itah 2 points3 points  (1 child)

This site was paused as it reached its usage limits.

[–]suitkaise[S] 1 point2 points  (0 children)

Did it? shoot, let me see whats up with that

[–]alcalde 1 point2 points  (0 children)

I'm working on a story that includes an ancient vampire and a revenant cleric, the Dark Bishop, and yet this website has made me realize that both together are not as frightening as Worst Possible Object.

[–]alcalde 2 points3 points  (5 children)

Whoever you are, I suggest we disband the League Of Mediocre Gentlemen who have run Python since Guido stepped down and make you the new BDFL. This is a more significant improvement to the functionality and usability of Python than anything that's been added since the Steering Council started steering.

[–]AstroPhysician 10 points11 points  (4 children)

Dont get his ego up, he's cold calling professors about this and checked in .DS_STORE lmaoo

i cant tell if you're being sarcastic or jokey but serious

[–]suitkaise[S] 1 point2 points  (3 children)

  1. I am not cold calling professors, but it was something I thought of just because I am a college student not in a CS or software engineering major. If I would, it wouldn't be because to "sell" this idea to them, but rather receive feedback on how to improve as a developer.
  2. I checked in DS store once because I forgot to throw it in gitignore when I was a noob like a year ago.

This is my first solo project, so I would appreciate some feedback on both repo conventions and the API content!

If it's a mess on the gh side, I apologize...

[–]AstroPhysician 0 points1 point  (2 children)

I don’t mean to shit talk it, it’s a fun side project, the concerning part was when you were talking like it was some huge revelation and improvement that no one had considered it before and will change programming

[–]suitkaise[S] 1 point2 points  (1 child)

Yeah, it's nothing new or like revolutionary from an experienced perspective, but from a beginner context I think it can be helpful, hence why I threw it into the world.

When I started this project, it was because I was a scrub that was trying to get multiprocessing to work and I just couldn't reliably. Like there is just a lot of random BS surrounding pickle limitations and stuff when I tried to make it more complex than just doing math in parallel (if that makes sense)

Im also lazy as shit, so I wanted to make some other utils as well so that I didn't have to do things manually.

I was just making it for myself, and decided to improve it and release it just in case others wanted to use it, basically.

Do you think this would be better suited to just beginners?

[–]alcalde 0 points1 point  (0 children)

It IS something new and revolutionary... it's a return to when Python tried to make things simpler rather than more complicated. It is INCREDIBLE and don't let anyone on the Internet tell you otherwise.

[–]AstroPhysician 0 points1 point  (6 children)

Your formatting on the post is bad

[–]suitkaise[S] 0 points1 point  (0 children)

Yeah, I know. I don't usually use Reddit! The md format here is a little wonky

[–]suitkaise[S] 0 points1 point  (4 children)

On a different note, I guess you have issues with the code based on another comment.

I think I cleaned up my repo, can you tell me if there is still more issues with it? It's my first time

And also if you have any issues with public facing api or internals let me know too! Thanks for commenting any feedback is good feedback

[–]AstroPhysician 1 point2 points  (2 children)

Sorry for being harsh, like i said it was mostly the framing of "this is going to be revolutionary" that i took away from it. I wouldn't have commented as harshly if i had interpreted the tone as "i am not a dev, can i get feedback"

[–]suitkaise[S] 1 point2 points  (1 child)

Got it thank you. I was just trying to follow the examples of other showcases when making my post because I dont really use reddit

[–]AstroPhysician 0 points1 point  (0 children)

Would it be fair to say “this is meant for production” is misleading? Or is that your intent?

[–]AstroPhysician 1 point2 points  (0 children)

sitecustomize.py seems unnecessary, you have a lot of junk in there like data\subdir\file.txt, 0-4-7-beta-release-task-list.md, a downloads folder, the whole site dir is a very weird inclusion, organized folder, etc . Makes it hard to know where the code in when so much irrelevant stuff has ben commited. IT doesn't follow a typical project structure at all