Pictopy

[abstract bear]

because on the backend we are using model and python

[coarse sparrow]

Ok thanks, trying for doing some quality work

[worldly pulsar]

Keep it up👍

[abstract bear]

you get stuck somewhere contact us"""

and listen mentor is out for a week he informed us so don't think he isn't active

keep up thee good work

[abstract bear]

I AM PRETTY MUC DONE

[worldly pulsar]

Just making final touch.

[abstract bear]

DONE

WORKING EVREYTHING??

[worldly pulsar]

Yeah.
Having some issues rendering returned image.
will be solved in some time

[abstract bear]

okkk

i am writing a pr

okay would send okkk

[worldly pulsar]

??

[crude prism]

@abstract bear You are now level 19! :confetti_ball:

[pseudo kiln]

Hello I am Divyanshu Shekhar. I am looking forward to contributing to this project.
I have good knowledge of MERN, Go, Python. I have done in the past, simple project in Rust and Tauri too.

[worldly pulsar]

Okay.
so you can go through codebase .
and try to understand workflow.

[worldly pulsar]

Specially Tauri and Rust part.

[pseudo kiln]

Yes sure

[worldly pulsar]

Let me know if you face any issue in set-up

[pseudo kiln]

I will try to understand it to my best

[worldly pulsar]

Great!

[opal cradle]

hey I have a question ....

??

more like a problem

🥲

[worldly pulsar]

What is it?

Don't worry.
We are here to help.

[opal cradle]

whenever i am trying to run " npm run tauri dev" my system is crashing again & again

[abstract bear]

that sound great clone the repository and listen there some issue when u run npm run tauri dev

some bug

typoo error

[abstract bear]

i made the changes and but and pr for the same

our mentor is out for week (he informed us) so we have to wait a little a while

but you send me screen shot i can helpp

[abstract bear]

tuh bas ss bhejjjj mai dekhahta hoon 🧐

[opal cradle]

ok

[abstract bear]

few question u install every dependenices

right ?

[opal cradle]

I mean i followed the steps from the github file

till the npm run step

till the last step you see on this SS

[crude prism]

@opal cradle You are now level 2! :confetti_ball:

[abstract bear]

what is errorr mtlb in the terminall

ss of that

[wraith creek]

yeah

anytime!

[opal cradle]

it just crashed again

[abstract bear]

bro listen trying to run scriptt

go the frontnend folder

and then npn run tauri dev

[opal cradle]

ok

[abstract bear]

run npm run taui dev

[opal cradle]

it is still crashing

[abstract bear]

did u installl all the dependeciess

[opal cradle]

what is the command

??

[abstract bear]

npm install

[opal cradle]

yeah i did

[abstract bear]

cd scripts

and then venv command

[opal cradle]

yeah

[abstract bear]

?

venv?

bro just give me a minute show me your screen i might get to know better what is the problem

?

[opal cradle]

um......no

🙃

i told you i just followed till the npm run trauri dev command

why is this issue coming ??

[opal cradle]

No worries it is solved

[crude prism]

@opal cradle You are now level 3! :confetti_ball:

[abstract bear]

greattt listenn can u add more themesss

brooo i can give you something

good for new bie

good first issue

[opal cradle]

ok sure

[abstract bear]

need any help

just remember we are there

[tranquil spire]

I have a concern about optimizing the overall performance of the application. Why are we querying the database using SQLite in Python instead of Rust? Won't it increase the latency

[pseudo kiln]

Exactly. There is this same issue with image fetching and posting. Why not fetch with Rust itself?

[abstract bear]

you are thinking that while rust is faster than the python

right that is what are u saying?

[abstract bear]

beacuse Rust may complicate maintenance without significant performance gains unless queries are highly compute-intensive.

[tranquil spire]

Oh i see ,so your intuition is that fetching of images is faster with the help of Python ,right?

[abstract bear]

Not exactly. Fetching images using Python can be efficient for smaller-scale applications,

[pseudo kiln]

But the high memory usage of the app ... Don't you think it's due to the overhead of Python only ??

[tranquil spire]

Yes but when the application will be scaled further then for that I was having the concern

[abstract bear]

but for that we are using effecint library like numpy or pillloww

if u saw

Proper profiling and optimization in Python might resolve the issue without the need to migrate to Rust.

[tranquil spire]

Well with libraries it would be better to stick with python but had the general doubt so yeah

[pseudo kiln]

Maybe certainly, just had the idea to do so. I guess I will too have to check some benchmarks like pillow and rust image libraries.

[abstract bear]

share them tooo

i wanna have a look at them

but generally spaeking even the senior engeenir takes around 1 year to be comofortable be rust

[pseudo kiln]

Exactly and it might be tough for us to implement certain advanced features now.

[abstract bear]

its better we stick with python

[worldly pulsar]

Pillow is pretty efficient while it comes to image processing.

[crude prism]

@worldly pulsar You are now level 19! :confetti_ball:

[worldly pulsar]

Its scalable and can be easily integrated.

[worldly pulsar]

And also, we can implement any AI feature easily .

[worldly pulsar]

Yeah.
It will be good for future improvements.

[worldly pulsar]

Pillow uses Numpy for most of the processing and computation.

And Numpy is implemented in C++ . So that makes pillow efficient because of underlying C++ implementation

[worldly pulsar]

New AI models will not be compatible with rust.

[abstract bear]

wahheee

[worldly pulsar]

And Python itself has huge library support to implement anything.

[tranquil spire]

yeah for implementation it's definately easier,i got curious looking at the database codebase and thought of asking . Well got to learn something new !!

[worldly pulsar]

There is so much more to learn.

[rose crane]

Hello everyone, I've been trying to figure out to contibute, can anyone tell me what's under developement and what are the places i can contribrute to

[worldly pulsar]

All the projects in AOSSIE are under development and are being actively maintained.

[worldly pulsar]

What's your preferred tech stack?

[opal cradle]

I am still waiting 🙂

[abstract bear]

ooh listenn my recent pr itt need to be memrgedd

see i have added a new feature through which user can control the entire ui

so in that i have only three themes where you can add more themes

[worldly pulsar]

You can try to find bugs.

[worldly pulsar]

Raise the issue and solve it.
It will be great experience for you.

[worldly pulsar]

It just feels different to solve an issue from scratch.

[opal cradle]

Ok 👌

[worldly pulsar]

Just a suggestion.
You can also pick existing issue.

[abstract bear]

let my recent pr merge then you can work on that

then

well parag @worldly pulsar

what about

the

documentation

[worldly pulsar]

working on it.

[worldly pulsar]

will add today.

[abstract bear]

till when ?

mtlb u have to go somewhere

so bro just make sure pr is there before that

documentation

is necessary

[worldly pulsar]

i mean.

i will finish it till evening.

[worldly pulsar]

I will

[abstract bear]

ok when are u leaving ??

and would be back when ?

[worldly pulsar]

tomorrow.

[worldly pulsar]

on 19 .

[abstract bear]

okk best of luck bro

[worldly pulsar]

thamks!

[abstract bear]

i'll be working on the face detection things live and then sort imagess hope fully it will merge soon to

[worldly pulsar]

It will be nice.

[abstract bear]

and listenn paragg requirement .txt i pretty much added all the dependienies would make a pr for it by tonight ??

sound great and that model we need to soon

[worldly pulsar]

Sure.

[worldly pulsar]

??

[abstract bear]

the model u would add to right

[worldly pulsar]

Yeah.

[worldly pulsar]

Hey @abstract bear .
I have added the documentation .

Check it out .

[abstract bear]

GREATT """"

[abstract bear]

@mystic sigil

hy tell me this is the better place to discuss

would be happy to have your views

?

CAN WE clearify because u were talking about onnx compatibilty and then were saying onnx won't be supported for long

[mystic sigil]

@abstract bear I have mentioned all the problems there https://github.com/AOSSIE-Org/PictoPy/pull/244

for your reference

i. Models size is 4 GB that's too large from my end
ii. As ONNX does not support it so , I have no idea it works for all divices or not

[abstract bear]

but you recently said there is certainity that onnx would be supported in the future

??

model size is a good point

i am working on it

shukriya for a great feedback

[abstract bear]

but can u clearify this

[worldly pulsar]

Yeah.
Whats your point?

[abstract bear]

the current focus is on delivering a functional feature, with flexibility for future adaptations if needed.

[mystic sigil]

I have told "In the future, there is no certainty whether ONNX will continue to be supported." means I have no idea ONNX in future will support or not the model you provided

[abstract bear]

it remains a widely adopted format with strong community backing and frequent updates

[worldly pulsar]

So, Why are we focusing so much on ONNX compatibility?

[abstract bear]

but firstly u were saying why its not onnx compitable

so i am not getting you point

can u elaborate

can i provide u file u can try it out with all the filess

[mystic sigil]

@worldly pulsar @abstract bear , let me clarify what I am saying. Your diffusion model is not supported by ONNX, which violates the backend Pictopy architecture.

Since ONNX does not support it, there is no guarantee that it will work across all devices

[crude prism]

@mystic sigil You are now level 7! :confetti_ball:

[mystic sigil]

And 2ndly it size is too high around 3.8 GB

[worldly pulsar]

On what basis you are saying this?
Have you tested it on multiple devices?

[abstract bear]

hile ONNX doesn’t support the diffusion model,

[mystic sigil]

I have told there is no gurantee

[abstract bear]

size i am working on it

[worldly pulsar]

I agree that size is a concern but it can be optimized in the future.

[abstract bear]

ONNX doesn.t support stable diffusionn models

[mystic sigil]

How can you be sure that it will work on all divice but if ONNX supports then Yes , I can say it will work on all divices

[worldly pulsar]

Its working perfectly fine on Windows and Linux.

[worldly pulsar]

I am sure because I have tested it.

[abstract bear]

The key point is that while ONNX provides a broader guarantee, the current model may still work across a significant range of devices,

[mystic sigil]

@worldly pulsar If that only matters then Yolo model was not used here , for your reference you check the previous discussion of Pictopy

[abstract bear]

okay i wasn,t aware that we have go through the chat around a year go

[abstract bear]

can u add that in the documentation

[worldly pulsar]

Your point was multi-device compatibility.
Which you didnt even back with proof.

[abstract bear]

all my saying @mystic sigil can u just try it out

please so that we can get an feedback and work towards it

i am working on the model reducing its size

[worldly pulsar]

Try it on your system . And then we can have a discussion.

[mystic sigil]

Let it be merged first then surely I will test it in my system

[abstract bear]

well the concern you showed us great full i have already started the workk for the size of the model and onnx

and if possibble can u help us out in testing

@mystic sigil asking as a fellow developer

[worldly pulsar]

We would love to hear your feedback!

[crude prism]

@worldly pulsar You are now level 20! :confetti_ball:

[mystic sigil]

Surely

[abstract bear]

@mystic sigil

just wanted to say you were your conern is real

at mid night going through all website onnx is a must

it would take a lot of effort but right now the model size has been shrink too around 2 gb

i am downloading automatic 1111 to optimize the midel and convert it itno onnx format

shurkiya bhai @mystic sigil

[worldly pulsar]

Looking forward to new Model.

[wraith creek]

do you guys know a good dataset for training images for gallery with which I can add more classes , I searched literally for hours and found no properly labeled dataset , few found were too large which were not possible to train locally

[marsh idol]

you can try imagenet : https://www.image-net.org/

I think its smaller versions exist

https://huggingface.co/datasets/zh-plus/tiny-imagenet

[wraith creek]

it has numeric labels do you have any idea how can I train yolov8n model wiht it?

also when loading it I dont see any labels folder, txt files required for training yolov8

[marsh idol]

https://huggingface.co/datasets/zh-plus/tiny-imagenet/tree/main

in this classes.py has the labels

[wraith creek]

ohh thanks

[worldly pulsar]

Great.

[worldly pulsar]

Now, we also have ONNX compatibility.

[viscid estuary]

how to solve "failed to add folder" in ai tagging page but album is loading images from the folder but not ai tagging

[worldly pulsar]

Can you send console log messages.

[abstract bear]

is your backend up and running

??

[viscid estuary]

INFO: Application startup complete.
INFO: Uvicorn running on http://0.0.0.0:8000 (Press CTRL+C to quit)
INFO: 127.0.0.1:45796 - "GET /images/all-image-objects HTTP/1.1" 200 OK
INFO: 127.0.0.1:45810 - "OPTIONS /images/add-folder HTTP/1.1" 200 OK
INFO: 127.0.0.1:45810 - "POST /images/add-folder HTTP/1.1" 400 Bad Request

[viscid estuary]

yes its running

[abstract bear]

BHAI EK MINUTE WHEN DID U FORK THIS REPO

JUST ASKING

the repo has update a lot since then listen?

@viscid estuary

just go sync you your repo to the lastest pictopy repo

then it should work fine

[worldly pulsar]

You are running both frontend and backend on WSL right?

[worldly pulsar]

If not , please run both on WSL.
You might be facing CORS error.

And also try syncing the latest changes from main repo. Just as @abstract bear said.

Let me know if it works.

[viscid estuary]

ok sure

[abstract bear]

..

[abstract bear]

https://medium.com/@mandalsouvik/safetensors-a-simple-and-safe-way-to-store-and-distribute-tensors-d9ba1931ba04 @worldly pulsar

[crude prism]

@abstract bear You are now level 24! :confetti_ball:

[abstract bear]

have a look at this

this might be the key for the speed

i read some articles about safetenors it we go from onnx to safetensor it would helpp

will help to keep the size low and i talk to some seniors about this

they said it won't effect accuracy if we use safetensor format and try quanitized from fp16 to int8

1.8 gb the size can easily be decreased to 800 mb or even less

@worldly pulsar

[worldly pulsar]

@abstract bear , You are right.
SafeTensors can significantly reduce model size while ensuring fast, efficient, and secure loading. It's perfect for keeping things lightweight without accuracy loss.

Combining SafeTensors with quantization (FP16 to INT8) can shrink a 1.8GB model to 800MB or less, maintaining performance and reducing memory overhead.

Any minor quality drop from quantization is negligible if done right.

SafeTensors boosts loading speed and security, ideal for real-time or edge applications. It’s the key to lightweight, efficient deployments.

[abstract bear]

we should go with safetesnsor

tensor rt supported so no worries

[worldly pulsar]

Sure.
In that way, we can also expect more contributors to add more AI features.

[abstract bear]

wahee

just workk more on the sizee

[worldly pulsar]

Yeah.
It will be great.

[abstract bear]

we must workk on accuracyy

[worldly pulsar]

Yeah.
We will.

[abstract bear]

i am going through some docmentation

[radiant mauve]

hey guys, what all issues are currently people working on?
Also, there have been a lot of merge conflicts lately

what is the issue

[abstract bear]

@radiant mauve we recently added a image genration feature

working on pr on requirement txt and and documentation

and alsoo some feature in ui and face detection for feature in backend

[worldly pulsar]

@radiant mauve, please check it out.

https://github.com/AOSSIE-Org/PictoPy/pull/242

We have also added setup docs for the image-generation model.

I am working on some frontend bugs and also adding detailed documentation of existing AI models

[marsh idol]

I am working on writing workflows for generating executables for all platforms, I need to test on Mac, all the other stuff is ready, will raise a PR soon. Also, I am working along with @mystic sigil to remove the "images" folder from the backend of AI tagging and getting the images directly from the folders.

[abstract bear]

@rahul can u elaborate i was thinking to add a video in readme .md on how to setup pictopy locally so should i waitt for few days

[marsh idol]

You can make the video now also, for setting up the project locally for development.

[abstract bear]

great then

[wraith creek]

I have addded generate memories feature, secure folder , done some optimisations in face tagging backend for more accuracy

[wraith creek]

also I found out their a seperate thumbnails folder generating specially for thumbnails, I dont know who did that but that make things more slow and take even more of user's space

[wraith creek]

also I am now working on improving the existing functionalies, adding necessary features for a gallery app instead of good to haves as good of haves will make the app more bloated only

[wraith creek]

To solve uncessary merge conflicts and errors because of that we can add tests in our app and a github action workflow for it so after someone resolves the conflict it is easy to know if that contains any error or not I'll create an issue for it and start working on it first as it seems more important

I have create issues for it

[marsh idol]

No, no the thumbnails folder was added recently in one of the PRs. It doesn't take much space, only like 50kB per image. But with that we are able to drastically improve performance because we first render thumbnails while displaying rather than full resolution image in the grid.

[crude prism]

@marsh idol You are now level 8! :confetti_ball:

[worldly pulsar]

You have got a point.

It improves performance drastically.

[wraith creek]

ok then its fine, I didnt look at how much space it takes

[crude prism]

@wraith creek You are now level 13! :confetti_ball:

[viral cobalt]

@wraith creek ,

[wraith creek]

yes?

[viral cobalt]

You were saying to collaborate on buliding advance editing features

[wraith creek]

yeah we can if you want I'll dm you after sometime describing everything well discuss

[radiant mauve]

yeah, a pipeline would be great
the project has becoming big now

[wraith creek]

yess Im almost done with tests for fast api then will start with rust test and lastly frontend and a pipeline on push

[radiant mauve]

that's great
would be greatly helpful for me

it's becoming tougher to review each PR manually

[worldly pulsar]

End-to-end testing will decrease your workload exponentially.

[radiant mauve]

i just commented on your PR
great work, but doesn't it diverge from our goals(as of now)?

image generation in a image gallery seems useless to me, until it can mix different images
rather than generation

[radiant mauve]

what do you mean by end-to-end testing here?

[worldly pulsar]

Thank you for your feedback!
I understand your point about the feature diverging from the current goals. My intent with adding the image generation functionality was to explore AI-based capabilities that could enhance the user experience and provide a unique edge to the gallery.

[radiant mauve]

yeah, the work was great
it's just, it's too early for the project and I am not sure, if it's right for the project

[worldly pulsar]

We can definitely consider adding the ability to mix or edit existing images in the future to make this feature more relevant and aligned with the project’s goals.

[abstract bear]

thank you for the feedback

[radiant mauve]

yeah, that would be a pretty cool feature

[worldly pulsar]

We will add AI image generator soon.

[abstract bear]

its a great idea would workk on this right away

[radiant mauve]

yeah
also, try making it modular

like suppose I don't want to use the image generator part, I can have a lighter variation of the app

and suppose I want to use it, I can download the model, and try it out

[worldly pulsar]

Sure.

[abstract bear]

we will give user an ability to generate image with low medieum and high quality

@radiant mauve we have resolve the merge conflicts

[worldly pulsar]

Yeah.
And we are also planning to give user choice to select between generating image with CPU or GPU .

[radiant mauve]

hmm, what about doing this, based on load
like use the GPU, and if you have some CPU left, use that too(upto a limit)

user never knows which setting is best for it

[worldly pulsar]

Thats a great idea.

[worldly pulsar]

We will implement it for sure.

[worldly pulsar]

So, we will dynamically check User system resources and based on that use them.

[radiant mauve]

yeah

[abstract bear]

we would make feature more robust and keep working on the model

[radiant mauve]

great!

[worldly pulsar]

We will add it after Image-Generation PR gets merged.

[radiant mauve]

no, didn't we just now concluded that image generation is not project's goal?

better would be image mixer

also, we decided to have modularity

[abstract bear]

got it

[worldly pulsar]

Got it👍

[worldly pulsar]

Can you elaborate a little bit, on Image-mixer.
So, that we would have a idea of what features to add.

[radiant mauve]

why not you guys work on some other feature currently
image mixer is also too early, although it's a great Idea

if you wanna work, work on packaging the project
the project is still not executable

[worldly pulsar]

Got it, we are getting into it right away! I’ll focus on packaging the project and making sure it’s executable first. Thanks for the heads-up!

[radiant mauve]

yeah, and it would be better, if you make a pipeline
so that we get an executable
after all the changes

[abstract bear]

Sure, I'll work on setting up a pipeline to ensure we can generate an executable seamlessly after implementing all the changes.

[worldly pulsar]

We will work on that.
Without pipeline, it will be manual work every time new changes appear.

[wraith creek]

@marsh idol has added that

he was working on it ig

[radiant mauve]

sorry missed this
@worldly pulsar @abstract bear this is duplicate
seems like Rahul is already working on it

[abstract bear]

no problem then can i make a landing page for picto py from where the picto py is available for the user to download

@radiant mauve

a suttle sweet landing pagee

[worldly pulsar]

@radiant mauve , so now the executables are done. Should I start on something new, or do you have anything specific in mind?

Whatever you suggest, we can make it happen.

For example, we could develop a mobile app for PictoPy, as most people use image galleries on mobile.

[rose crane]

if so, I would love to help you @worldly pulsar on developing it

are you guys also facing issues after the recent merge, there seems to be lots of error after @wraith creek 's recent merge.

[worldly pulsar]

Let see what @radiant mauve thinks about this.

[wraith creek]

I noticed duplicate dependencies and '}' missing in rust after resolving merge conflicts I've fixed that in my latest PR where I'm adding tests, apart from that I checked everything locally their were no errors, when I pulled the main origin their were few lines in mediaview.tsx whiich were causing ig some type errors those lines were not added by me

these merge conflicts problem will be resolved soon after adding tests and a pipeline which I'm adding

[abstract bear]

why nott we need people because adding some major things requires people

@marsh idol there is some merge conflict i noticed it iff possible just resolve it

[abstract bear]

if mentor allows for the landing pagee you can come in we can build it togetherr

[worldly pulsar]

Absolutely.

[abstract bear]

just @rose crane be readyy for some night fixing issue and bugsss🤪 and then going through documentation mid nighttt

[crude prism]

@abstract bear You are now level 25! :confetti_ball:

[abstract bear]

@radiant mauve can u remove this enguage

[wraith creek]

I have fixed all meerge conflicts caused by other PRS too

should I push that

in my tests PR?

[abstract bear]

by the wayy the thing you are about ot make pr

i have alreadyyy workedd on it and pushed

but mentor was out after that

so it was'nt merge

@wraith creek no worries we gotta wait for my pr evreything would be good to go

[wraith creek]

I have fixed all the bugs and pushed

[abstract bear]

it's a;readyy there fixedd

have alook around a week beforee

pleasee

just chill broo tuhh kaare ya maii kya farak padhtha hai

[wraith creek]

ok I thought it was causing problems for many people so I just went ahead and fixed everything every file wherever the error was coming for easy setups

[wraith creek]

yeah

no worries these conflicts will be resolved soon after adding tests

and pipeline

not resolve fully but reduce significantly

[abstract bear]

yupp greatt work

[wraith creek]

what css update? I never updated any css

[abstract bear]

gnn

[wraith creek]

gn

[marsh idol]

I am getting a lot of errors in the MediaView.tsx file after a recent merge.

The app is not able to run and I am not able to resolve conflicts on my PR due to this.

[abstract bear]

okay wait let me checkkkk

let me get my handss dirtyy

give me some time @marsh idol paaji

[abstract bear]

paaji @marsh idol i just synced my repoo to the latest versionn

just give me an hour i'll fixx it

[marsh idol]

No no, you don't need to do that.

[abstract bear]

did it brother

[marsh idol]

I think it should be resolved by the person who coded that part

[abstract bear]

yup i get youu but its just some syntaxx errori 'll fixx it

[abstract bear]

@marsh idol working upon it give me timee

[abstract bear]

@marsh idol hy everything is up and running everything is up running

[abstract bear]

https://github.com/AOSSIE-Org/PictoPy/pull/270 i have madee pr which solve all the issue there were too many typoo too which isfixedd

and listenn everyone please don't use motion(framer-motion) in the frontend just stick to react type sccript tailwind css espicallyyy in src tauriiiiii

[wraith creek]

I saw that too after pulling main I've fixed that with all others in my latest PR yesterday but seems like @abstract bear has also fixed so their should be no issue after merging

[abstract bear]

bro please @wraith creek add your testt as soon as you cann

[wraith creek]

on it! can be a bit of delay because of exams but will be definitely done by the end of this week

[abstract bear]

hmm theek hai thodaa time lagea

if u need helpp in that

i can helpp

[wraith creek]

sure I'll tell you if I need any help in that!

[abstract bear]

i'll be happy to helpp

[mystic sigil]

Someone solving this issue also add a 'bcrypt' module in requirements.txt , as 'bcrypt' used in https://github.com/AOSSIE-Org/PictoPy/blob/main/backend/app/database/albums.py but not in 'requirements.txt'

[worldly pulsar]

@abstract bear ,
Add bcrypt with your PR.

[abstract bear]

okkkkkkk i'll make a commit in that pr itself

let me solve this tooo

[rose crane]

guys is anyone else also facing issues after the last merge

my app is not running after pulling

[worldly pulsar]

There were some errors.
@ KARNX7 has fixed them in his PR.

[worldly pulsar]

The app will be running after that PR gets merged.

[rose crane]

why are the PRs getting merged so late?

[crude prism]

@rose crane You are now level 3! :confetti_ball:

[viral cobalt]

do anyone face same error?

[worldly pulsar]

Hey @viral cobalt , This error has been resolved but PR for that is not yet merged.
So, You will have to wait till that PR gets merged.

Or you can pull changes directly from the PR:

https://github.com/AOSSIE-Org/PictoPy/pull/270

[abstract bear]

arham everything resolved just wait mentor will come and merge it

[viral cobalt]

Thanks for your help @abstract bear @worldly pulsar

[radiant mauve]

Hey guys, now there are lots of issues with merge conflicts
I advise you to, keep on pulling the main repo before making a merge

As now, to resolve the backlog of these
can you all, just create a google doc or something, with all MR's, and after I merge it in that order
you guys resolve conflicts, and fix yours(until your turn comes in the order)

it's needed otherwise, there's going to be a pile of MR's like this

can someone please take the initiative of creating a doc
and adding the PR's in it

[abstract bear]

hy @radiant mauve also do we need landing page for this app from where can download our app(pictopy

[worldly pulsar]

@marsh idol ,
I am getting this error while running frontend after recent merge..
Do, we have to run some commands for creating /dist folder?
If yes, please update Redme.md .

[abstract bear]

@everyone listen i am going to restructure the frontend what i mean by this is that becausee too many things are added so it making look cluttered (i don't think any user is going to use in such a condition ) so would make everything well structured and it should follow a simillar design i will create a notionn dockss for it so pleasee from now onn before you add anything before you add anything update in that notion dockk

[marsh idol]

I will check that after few hrs

I am out right now

[worldly pulsar]

No problem👍

[marsh idol]

I think just creating an empty folder named dist inside the backend folder should do it.

I should probably have updated the readme or created this folder

in my PR.

I will correct this.

[marsh idol]

@worldly pulsar I have opened the PR for the fix, thanks for pointing out.

https://github.com/AOSSIE-Org/PictoPy/pull/285

[worldly pulsar]

Glad I could help!

[wraith creek]

https://docs.google.com/document/d/1UDnSGBjlTkDlibs9J85_cY4G8YCUFIHSsJS_NVg_K3A/edit?usp=sharing
here you go!

[abstract bear]

bro he deosnt meant thatt

[wraith creek]

ohh understood just the PRS with merge conflicts?

but that's variable too for eg if one PR gets merge their is a chance that it would cause conflicts with other
Though if still if doc with all MR are needed I'll update it by tonight

[abstract bear]

yupp

you

i am working on thiss docss

if u are so shall i stop

[wraith creek]

if you want to you can work no problem with me otherwise I'll update this tonight

[abstract bear]

naah if u can just do it

[wraith creek]

updated them now it contains all PRs with MRs

[marsh idol]

The order goes from top to down right?

The first one will get merged first

?

[wraith creek]

Yup

[crude prism]

@wraith creek You are now level 14! :confetti_ball:

[wraith creek]

I guess so that's upto the maintianer the order in which he merges

[rose crane]

guys the frontend looks ugly, is anyone working on it ? @abstract bear you previously mentioned working on it once are you doing ?

[abstract bear]

sorry can u elaborate i did nt get that @rose crane

[rose crane]

the sidebar ui and everything is looking quite ugly, are working on revamping designs?

[abstract bear]

bro just have a look i already

made a pr customization and everything named such

i worked on teh side bar

it not mergedd

but it would be merged soo

soon

[rose crane]

is there anything to be done on frontend where I can help?

[abstract bear]

i am right now working on something backend

and frontend pretty much done

there are some some routing issue in the frontend

people have not used react roiter well so its causing issue

[rose crane]

ok i'll look into it

[abstract bear]

ya sure go ahead if u need any help in that

i am here

@rose crane \

[rose crane]

surre

[worldly pulsar]

Let me know if you face any issue in that.

[rose crane]

I've fixed the AppRoutes.tsx file. There was issue with conditional rendering of Layout Routes

[worldly pulsar]

Do you need help with that?

[rose crane]

Nope, I've fixed it .

Btw thanks for asking

[worldly pulsar]

No problem.

[rose crane]

https://github.com/AOSSIE-Org/PictoPy/pull/287, fixed the routing issue here

[abstract bear]

@rose crane great buddy i saw it but i was busy something could'nt do it

thanx

[raven hill]

https://github.com/AOSSIE-Org/PictoPy/pull/293
Can anyone please have a look in it ?
The PR is regarding the missing package that led to breaking of code in object detection in yolo model and also corrected the file path .

[rose crane]

hey guys the images and design in the memories section look too big and don't go with other pages, planning to change them to have similar look throuhout the app

[crude prism]

@rose crane You are now level 4! :confetti_ball:

[abstract bear]

bro hy i

i am already done with it

i was about to make a pr when i was about to get back group the collage

[rose crane]

So, you are done with it?

[abstract bear]

yup i was working on it till like morning but couldnt push it

as i had like leave for collage

so i thought after collage i would come back and make a pr

[rose crane]

ok then

[abstract bear]

great sorry i should have make a issue for it bea cause i thought i would fix it in my frontend restructring

things

sorry if u like started the work @rose crane

[rose crane]

I hadn't stared yet

[abstract bear]

great then

i'll push it today by the time i am back from collage

[glass seal]

Hey guys, I want to contribute to pictopy. If you need assistance with your assigned isseue related to frontend please do let me know

[abstract bear]

now who did this

??

backend is running images are not coming up

[worldly pulsar]

Is the request being sent to the backend?

[abstract bear]

yup

see na i have loaded images from the frontend it showing uip here in the terminal \

but images are not there frontend

[abstract bear]

hy @everyone

i am make the web ui for the picto py where we will allow user to download it from

would need you all reveiws for it

[rose crane]

great idea, do let me know if you need help

[worldly pulsar]

This is needed.

[worldly pulsar]

Sure.

Let us know when you are done building.

[jagged perch]

Hi @everyone

[jagged perch]

I'm looking forward to contribute to PictoPy

Would love to get connected with you all

[jagged perch]

Hey @radiant mauve ,
https://github.com/AOSSIE-Org/PictoPy/pull/308

I opened a new issue and created a pull request for the same

Can you review it once??

[frail rover]

That's something I need to be good at..cp

[frail rover]

Is it done?

[frail rover]

guys when am trying to upload pic no pic is coming...can someone tell if u r also facing the same prob? images not processing

[marsh idol]

did you start the server?

[worldly pulsar]

It was working fine few days back.

[marsh idol]

please visit http://localhost:8000 and see if you get a message from the server.

[frail rover]

not getting

[marsh idol]

which platform are you using? windows, mac..?

[frail rover]

windows

[marsh idol]

ok, I can help you setup the project. Did you follow the instructions provided in the readme file for the setup? Or you are getting any errors following those

[frail rover]

sure..am woking on eduaid for now..will let uk once i start working again on pictopy

[frail rover]

..

[marsh idol]

👍

[crude prism]

@marsh idol You are now level 9! :confetti_ball:

[dusk sail]

I would like to collaborate

[worldly pulsar]

When did you fork the repo?

Recently or its been many day?

[frail rover]

Today

[crude prism]

@frail rover You are now level 3! :confetti_ball:

[worldly pulsar]

If you are getting a message at localhost:8000,

there must be issue with server setup.

Are you able to see these lines when server is running?

Loading existing face clusters from database...
[2025-02-12 19:25:06 +0000] [6502] [INFO] Running on http://0.0.0.0:8000 (CTRL + C to quit)

[tight wyvern]

Hey, can anyone help me with this, I following the steps in the readme file and trying to run the backend but getting this error

[tight wyvern]

nvm got fixed

[abstract bear]

Great then

[worldly pulsar]

Great.

[worldly pulsar]

it will be better if you run backend in WSL.

[abstract bear]

.

[abstract bear]

i have worked on hero.tsx and just decided more thing

i will contact you when i finalized for some sectionss

[tribal crow]

Hey everyone! 👋

This is my first contribution to this repository! I'm still exploring and understanding the codebase, but as I ran the application for the very first time, this was the first thing that caught my attention. So, I decided to fix it!

It's a minor UI fix for the folder picker icon alignment on the Setup screen when the app opens in a smaller resolution.

You can check it out here: 🔗 https://github.com/AOSSIE-Org/PictoPy/issues/323

Looking forward to learning more and contributing further! 🚀

[tight wyvern]

oh I am using endeavor(arch linux) not sure how wsl will help?

[worldly pulsar]

Oh.

Then it will not be an issue.

I thought you are using windows.

[tight wyvern]

yea, I was actually trying to use uv but ig that dosent work for some reason, I switched back to the normal pip workflow and it started working

[abstract bear]

its great start to start from something small if u get stuck somewhere and you need help message me i'll helpp

[tight wyvern]

I am not able to upload image folder. Not sure what the problem is? Is anyone else having the same problem?

[frail rover]

Yeah I too had this..haven't tried after that..will try later on..

[marsh idol]

To reproduce the problem, I cloned the repository again and setup the project. I am not getting this issue. If you were able to see those "retry" buttons, then did you try to switch to another tab like settings and then going back to the AI tagging tab? This will re-fetch the images and the images will load without any error from then on.

If this still doesn't work, please let me know what do you get when you visit this url: http://localhost:8000

You should see a message like this:

If you are not able to see that message then, the server isn't running.

[worldly pulsar]

I am also not getting this issue.

Folders are being uploaded without any issue.

[mystic sigil]

Everyone here is my implementation idea for https://github.com/AOSSIE-Org/PictoPy/issues/210

I would greatly appreciate any suggestions from your end.

[worldly pulsar]

@mystic sigil , right now, we are saving thumbnails in a folder.
So, I didn't get your point about fixing it.
Can you elaborate a bit?

[worldly pulsar]

Your idea of storing edited images seperately is good one.

[mystic sigil]

My motive , what ever operation we do with our application should not reflect in actual folder , that's why decided to shifted all thumbnails to a single folder images/pitopy.thubnails

[worldly pulsar]

Oh.
Got it.

Go ahead.
It will be nice improvement.

[mystic sigil]

Thanks @worldly pulsar

[hallow saffron]

I've identified a potential memory leak issue in the ONNX Runtime sessions during batch image processing. The problem occurs in both YOLOv8 and FaceNet components where ONNX sessions aren't being properly released.

I've created an issue here: https://github.com/AOSSIE-Org/PictoPy/issues/335

Would love to get some feedback.

[viral cobalt]

I think we should add tone curve in editing , what do you think?

[marsh idol]

I think as Bruno sir mentioned in the GSoC announcement, we need to suggest changes/features which align with the project's vision. This project is an image tagging/classification and face recognition-powered gallery. So adding more photo editing features will not align with our app. This is my take.

[frail rover]

Agreed

[abstract bear]

I have seen such thing have just made the app more cluttered it would be better if u have something in my mind d related to face tagging

Face sort

That would be better

This is my takes

[worldly pulsar]

I agree.

Features related to Face tagging will align with the app.

[abstract bear]

I think when you open pictopy it looks now cluttered

[worldly pulsar]

Yeah.
A little bit.

[abstract bear]

There is too much scope in this project

In the backend part as people pretty much stop and moves forward

So we gotta rethink

And yes one more thing

Are u familiar with electron ??

[worldly pulsar]

Are you asking me?

[abstract bear]

Yup

[worldly pulsar]

Yeah.

I have used it in some hobby projects.

[abstract bear]

I tried to build something with electron but way back

So I think after I spend some more time

I'll be able to get back to it

It must be somewhere back in the mind

[worldly pulsar]

If you have used it once.

Then you catch up on it pretty quickly.

But, there are some issues with the electron.

[abstract bear]

Like?

[worldly pulsar]

Its memory intensive and heavy.

Higher ram usage.

[abstract bear]

That why they choose tauri because of it

[worldly pulsar]

Yeah.

[abstract bear]

There must be some way I feel

Is there library support for this

[worldly pulsar]

There should be.
We need to find a way to solve that problem.

[abstract bear]

We gotta research more about tauri but

Electron

[worldly pulsar]

Yeah.
It has huge library support.

Just like Python.

[abstract bear]

Let see how we are going to take things forwardd

[worldly pulsar]

Yeah.
We will find a solution if we go deep enough.

[radiant mauve]

Hi guys, I don't think we can come up with a fix for so many MR's
so what we are gonna do is, we will discard all the previous MR's, and now review all line by line

in your MR's, what you can do is, add a MR: TITLE(SR. NO.)
as the project has become so big, it has become difficult to maintain the style

as there are lot of contributors

I will be active from today to tomorrow

you guys, create new MR's and ping me up here
do mention the serial number

@marsh idol @wraith creek @worldly pulsar @abstract bear @mystic sigil

and any other contributors, that are currently active

[marsh idol]

https://github.com/AOSSIE-Org/PictoPy/pull/337

I closed my previous PR and reopened it with sr. no. 1

[frail rover]

Sure

[radiant mauve]

queue, your PR after this guys
i am waiting for the pipeline, and will merge this then

[marsh idol]

pipeline meaning workflows?

for the builds?

[radiant mauve]

yeah

oh wait, these are pipelines only

[marsh idol]

yep

[radiant mauve]

just a sec then

Merged
Take the latest Pull, and create MR no. 2 and ping me

[abstract bear]

https://github.com/AOSSIE-Org/PictoPy/pull/309 closed my previous PR and reopened it with sr. no. 3

[wraith creek]

I've also added pipeline to run tests , closing reopening and renaming my PRs

[radiant mauve]

do it after @abstract bear

wait for my signal

let me review his PR

[wraith creek]

ohh ok

[marsh idol]

I think you would have to close the PR and create a new fresh PR, for it to show at the top ig. Or it is the same thing ig.

[radiant mauve]

same thing, ig he took the fresh main pull

[abstract bear]

doing it right away

[marsh idol]

But the workflows won't run on the older PRs I think.

[wraith creek]

they will if we merge the main branch

[radiant mauve]

oh right

then @wraith creek you do yours first then

[marsh idol]

So everyone has to create a new pull request by copying all the description from the older one.

[wraith creek]

yes doing it should I close and reopen my old PR or create a new one?

[radiant mauve]

new

[wraith creek]

btw rahul are you exporting PROD env variable in your workflow because your tests are failing because of that

[marsh idol]

Let me check quickly

[wraith creek]

I'll add that in this workflow PR, just tell me what to export in PROD?

[marsh idol]

I thin due to this change in you PR, the workflow is failing

import.meta.env.PROD is a variable provided by vite

while process.env.PROD is a varable that we have to define in .env file

[wraith creek]

yeah but cant do that otheriwse frontend tests would fail that's why I intentionally added vite-enviroment-variable plugin

and that needs a prod env

[abstract bear]

@radiant mauve hy i wanted to ask i am making a landing page for the pictopy where we can give our app to the end user shall i continue with it?

[radiant mauve]

yes, that's a great idea
go ahead, do add AOSSIE's logo in it

[abstract bear]

thanx would update you in few daysz and would add aoosie logo on the navbaar

[radiant mauve]

great!

[abstract bear]

https://debate-ai-prototype.vercel.app/ i recently made a complete ui for ai for debates ( only frontend depolyed)

[marsh idol]

@wraith creek open your PR with sr. no. 2, because workflows PRs are to be merged before the other

[wraith creek]

ok wait

https://github.com/AOSSIE-Org/PictoPy/pull/338 @radiant mauve

[marsh idol]

Once this merges, I will open my another PR.

[wraith creek]

it is merged you can go ahead

[radiant mauve]

merged

[crude prism]

@radiant mauve You are now level 14! :confetti_ball:

[marsh idol]

https://github.com/AOSSIE-Org/PictoPy/pull/340

3rd PR

[crude prism]

@marsh idol You are now level 10! :confetti_ball:

[marsh idol]

The backend tests failed for the generate thumbnails function because I have made modifications to it

We have to change those tests.

[radiant mauve]

also anything wrong with the build pipeline for windows?
it's failing for every PR

[wraith creek]

Yes I found the issue it was related to environment variables

[crude prism]

@wraith creek You are now level 15! :confetti_ball:

[marsh idol]

The error was in this PR right? https://github.com/AOSSIE-Org/PictoPy/pull/275 @wraith creek

[wraith creek]

Yes I fixed that in my actions workflow now no errors are coming

[marsh idol]

okay then, its fixed

the app will build

The backend tests failed

on my PR

[wraith creek]

Yeah need to change the tests according to your latest changes to the function ig

[marsh idol]

yep

[wraith creek]

I'll change that

[marsh idol]

@radiant mauve Do we want to build the app after every PR merge or we can set the workflow for builds as a manual trigger, which will generate builds with updated version number and detailed changelog in the release section?

[wraith creek]

Ig we can create certain milestones and release versions when those gets completed and trigger a build manually?

We can add few Prs to those version milestones

[marsh idol]

yes, that sounds good

[radiant mauve]

manual trigger
else we might use up all the github resources

or the org

[marsh idol]

The windows PR check passed since the problem was fixed in the last PR, now the app should build without errors for all platforms.

[marsh idol]

Yes

I will modify the workflow for builds

[worldly pulsar]

@radiant mauve ,
please merge this.
https://github.com/AOSSIE-Org/PictoPy/pull/315

[radiant mauve]

@abstract bear build failed for your PR's

[worldly pulsar]

all build tests have passed.

[radiant mauve]

yeah just wait, waiting for Karn's MR
else we will continue with yours

[radiant mauve]

your MR has conflicts

fix it and create a new one

[worldly pulsar]

working on it.

[abstract bear]

i am fixing it right away

[radiant mauve]

@worldly pulsar follow after Karn

[worldly pulsar]

@marsh idol ,
My backend tests are not passing, But I havent changed anything in backend.

[marsh idol]

Yes, it is due to my changes in the backend

Rishab will add the new tests

it will work

[worldly pulsar]

So, what should I do to pass the current tests?

[marsh idol]

I think its fine for now, mainly all the Tauri build tests should pass.

[worldly pulsar]

Okk.

[marsh idol]

beacause the build tests checks for TS errors

[abstract bear]

https://github.com/AOSSIE-Org/PictoPy/pull/341 @radiant mauve i everything is up and running and each test is passed

[worldly pulsar]

@radiant mauve ,
Here's PRs with SR No.- 5
https://github.com/AOSSIE-Org/PictoPy/pull/342

and SR No. - 6
https://github.com/AOSSIE-Org/PictoPy/pull/343

All tests are passed.

[marsh idol]

@radiant mauve I have disabled the auto-triggering of build workflows in my latest PR (Sr. No. 7). Also, I have modified the backend setup instructions and removed the server scripts which were not up to the mark now we are using the standard approach to run the backend, using "fastapi dev" command for the dev. Environment.

reference taken from : https://fastapi.tiangolo.com/tutorial/first-steps/

[hallow saffron]

my backend is running correctly

its just blank

[wraith creek]

@radiant mauve should I put my another pending PR in queue?

[marsh idol]

Let me check I think it is due to PRs merged yesterday

[wraith creek]

found the issue in frontend/src/Config/Backend.ts file remove process.env.backend_url I removed vite enviroment plugin yesterday because windows build was facing few issues of that but forgot to remove process.env in backend.ts

I'll fix this along with my next PR

[mystic sigil]

For now In this file
frontend/src/Config/Backend.ts

replace
export const BACKED_URL = process.env.BACKEND_URL || 'http://localhost:8000';
to
export const BACKED_URL = 'http://localhost:8000';

[wraith creek]

queued my PR, fixed this issue in that
https://github.com/AOSSIE-Org/PictoPy/pull/347

[abstract bear]

i don't think that is due to that because re forked it yesterday after the merge

it worked fines

[marsh idol]

Oh, must be some other PR then

[hallow saffron]

@radiant mauve I have made a PR (https://github.com/AOSSIE-Org/PictoPy/pull/336). I have followed all the guidelines. Bassam Adnan has approved my PR. Can you review it?

[crude prism]

@hallow saffron You are now level 3! :confetti_ball:

[hallow saffron]

I was trying to work on this issue

https://github.com/AOSSIE-Org/PictoPy/issues/349

rust 🏳️

[hallow saffron]

I was trying to work on an issue and i was running into an error endlessly

The error indicated that DATABASE_PATH is missing from the settings file.

I noticed that there are multiple database paths being used (FACES_DATABASE_PATH, IMAGES_DATABASE_PATH, etc.) but DATABASE_PATH is not defined.

is anything like this previously reported?

[hallow saffron]

.

I just opened a PR to fix the face recognition clustering bottleneck. It adds batch processing for embeddings, improved memory usage, and various database optimizations. Some highlights:

• 98.1% performance improvement in clustering operations
• Smart batch processing for face embeddings
• Advanced database optimizations
• Real-time performance monitoring

Before: Processing 100 images took ~45 seconds
After: Same batch now processes in < 1 second!

PR: https://github.com/AOSSIE-Org/PictoPy/pull/354

Would love any feedback on the approach!

[marsh idol]

Your PRs look really good and detailed 😄

[crude prism]

@marsh idol You are now level 11! :confetti_ball:

[marsh idol]

@hallow saffron I think you are using the run-server script to start the backend, In my recent PR which is open, I have simplified the backend setup process,please prefer to use that : https://github.com/AOSSIE-Org/PictoPy/blob/9c918b26c0d643300b36a42787693b989867ab28/README.md

[marsh idol]

can you tell, which python file was giving the error?, refer the stack trace

I think that all the database tables should be in a single .db file, currently there are different DBs for different tasks, which makes foreign key references impossible.

There are many changes which could be made to the backend

[abstract bear]

i am working on the Landing page for the pictopy I JUST COMPLETED hero.tsx and

guz any suggestions

???

[wraith creek]

yes I noticed that too, need to change that

[wraith creek]

I went through it thoroughly and it looks good and very detailed

[marsh idol]

Looks great! In PictoPy, the last letter is small

[wraith creek]

Also when I was contributing to some python libraries I noticed they had comments on every function which made it easy to understand the codebase it would be great if we add hints and comments throughout the fastapi and rust codebase

[wraith creek]

looks good to me but I may not be the right person to comment on UI/UX

[marsh idol]

Yes, I think we can make this as a good first issue

Also the PictoPy documentation is outdated, after so many changes to API and frontend

We can open up small good first issues for new contributors

[wraith creek]

yes totally agree

[marsh idol]

We also need to make a CONTRIBUTING.md file to standardize contribution process and code preferences

[wraith creek]

yess, also need to change the issue template with few checklists like for changes did you update the docs or added unit tests etc

[marsh idol]

Yep.

I think we should go ahead and create these issues with ("Good first issue") in the title after @radiant mauve sir will add good first issue tags, we can remove that

[wraith creek]

yeah sure I'll create the issue for hints and issue template

[wraith creek]

I've created them, you can create for updating docs

[marsh idol]

done

[mystic sigil]

Very nice looking! 👍

[crude prism]

@mystic sigil You are now level 8! :confetti_ball:

[worldly pulsar]

Looks great.

@marsh idol @abstract bear ,
I was thinking to add a script to update requirements.txt automatically whenever new PR is made.

Right now, new depedencies need to be added manually.

[marsh idol]

doesn't running the command pip install locally, update the requirements.txt automatically?

the person needs to include the .txt into their commit

[worldly pulsar]

No.
It doesnt.

[marsh idol]

Oh

[worldly pulsar]

Yeah.

So, this will ensure , that all the requiremnts are up to date.

[marsh idol]

I thought it works like npm install

[worldly pulsar]

thats why npm is better package manager.

[marsh idol]

how will you check for all the changes and find out if a dependency is missing?

[worldly pulsar]

By using .git hooks.
And github Actions .

[worldly pulsar]

and also local script that can run on command to update the requiremnts.txt

[marsh idol]

Okay, so you need to scan each .py file?

Or something else?

[worldly pulsar]

Yeah.
for import statemnts.
Or I can just scan backend conda or venv environments.

[marsh idol]

You can look into another approach also, there is a pip command which lists all the installed packages

[worldly pulsar]

yeah.
pip freeze
.

So, this command scans env.

[marsh idol]

But with this you can't judge which package is redundant.

@worldly pulsar I think this is what you are talking about?

https://stackoverflow.com/questions/31684375/automatically-create-file-requirements-txt

[worldly pulsar]

pip freeze is great when you have seperate virtual env for project.

But if seperate env is not present,

we need to think a way for that situation.

[marsh idol]

@worldly pulsar check this out:
https://github.com/marketplace/actions/automatic-requirements-txt-for-python-projects

Project:
https://github.com/bndr/pipreqs

[crude prism]

@marsh idol You are now level 12! :confetti_ball:

[worldly pulsar]

This is exactly what we need.

I will start working on this immediately.

[hallow saffron]

Thank You :D

[hallow saffron]

I was trying to do pytest in backend folder and was running into errors which basically indicated what i wrote

[hallow saffron]

Thank you :D

[hallow saffron]

if i put a photo to secure folder, how do i remove it from secure afterwards?

[mystic sigil]

This feature not present there

[worldly pulsar]

We need to implement photo removal.

[mystic sigil]

Yes

[mystic sigil]

If you want may work on this ...

[hallow saffron]

Also i saw that we use PBKDF2 for secure folder pw

[crude prism]

@hallow saffron You are now level 4! :confetti_ball:

[hallow saffron]

What if we change that to ARgon2

[worldly pulsar]

What's the benefit of it?

[hallow saffron]

To my knowledge,

argon2 is better than pbkdf2

[worldly pulsar]

Better for what?

Please elaborate.

[hallow saffron]

Argon2 makes it harder to crack the pw than pbkdf2
performance wise, theres almost no significant change for the user
more of an industry standard (from what ive read on reddit and articles)

im working with ref to this issue
https://github.com/AOSSIE-Org/PictoPy/issues/349

[mystic sigil]

pbkdf2 uses SHA (hashing algo) while argon2 uses Blake2b

In terms of Speed and Security I think for modern development using argon2 is better

[hallow saffron]

https://tenor.com/view/sarfaraz-ahmed-virat-kohli-cricket-world-cup-my-answer-is-also-same-no-difference-gif-23392400

[mystic sigil]

There is one drawback also I found , in terms of security argon2 is better but in terms of system compatibility pdkdfd2 is better ...

@worldly pulsar have you any suggestion on this ?

[worldly pulsar]

@mystic sigil , @hallow saffron ,
I think Argon2 will be better because, it’s more resistant to brute-force and GPU/ASIC attacks than PBKDF2 due to its memory-hardness and better key-stretching.

[wraith creek]

it is their

their is an unlock button their

[wraith creek]

you can if you want to but their's not much of a need i think for this

[hallow saffron]

too late already sstarted

[abstract bear]

ooohh haan thanx reminding i think i was working on this last night so made that mistake

[hallow saffron]

This was the issue #349

Added secure folder functionality to PictoPy with:

• Implemented Argon2id for stronger password hashing.
• Added password strength validation (min length, uppercase, numbers).
• Enhanced file deletion with multiple overwrite passes.
• Ensured AES-256-GCM encryption with proper salt and nonce handling.

This is my PR

I'd love to get some feedbacks on these changes!

[abstract bear]

Thanx bro still working on the entire page would keep updating on this

[marsh idol]

Hello everyone, I have opened a PR which fixes issue#209: https://github.com/AOSSIE-Org/PictoPy/pull/364

In this, some important changes are:

1. Accessing images directly from the folder given by the user instead of copying them into the "images" folder.
2. Modified the database structure to contain every table in only one .db file called PictoPy.db.
3. Modified backend tests to align with the latest changes.

Please look into the PR and give any suggestions.

@mystic sigil You can now start your work on the isolation part of the issue#209.

@wraith creek You can start with the progressive loaders that you talked about. Because I have added the folders table in our DB.

[mystic sigil]

Okay 👍

[marsh idol]

To access my changes, you will have to create a branch which is the copy of my branch. Because my changes are not yet merged.

[wraith creek]

yeah

[mystic sigil]

Not a problem , I'll handle

[worldly pulsar]

Hey @marsh idol ,
How can i test the Github workflows locally?

[marsh idol]

means on your github fork or locally on a PC?

[worldly pulsar]

Local PC before commiting.

[marsh idol]

https://github.com/nektos/act

[worldly pulsar]

Got it.

[marsh idol]

I have not tried it

[worldly pulsar]

Thanks!

[marsh idol]

I tested my workflows on my fork.

[worldly pulsar]

How?

[marsh idol]

you can trigger them manually

if you have specified in it in the yml file

[worldly pulsar]

But, workflow is in new brnch and not in main branch.

[crude prism]

@worldly pulsar You are now level 30! :confetti_ball:

[marsh idol]

temporarily you can allow triggering for any branch, you can create a test pull request on your fork

there workflows will run

You want to test which workflows?

[worldly pulsar]

Oh.

Automatic requiremnts.txt generation.

[marsh idol]

yeah, I think creating a dummy PR on your fork will be better

to test workflows

[worldly pulsar]

I will try that.

[marsh idol]

For using this, it will utilize docker

[worldly pulsar]

Yeah.

I tried it once.

[molten kindle]

Hi community, I have contributed to other AOSSIE projects like Edu-Aid. I'm comfortable working with JS and python. Could you assign me any first issues?

[crude prism]

@molten kindle You are now level 2! :confetti_ball:

[abstract bear]

Hy listen

Try cloning the repo 9f picto py

In the frontend navbaar color many of us doesn't like it

You can work on that good for beginners

[molten kindle]

Ok thnx
Will work on it

[wraith creek]

Also @marsh idol and I've created some good first issues you can check them out too

[glass seal]

Hey guys! I really need help setting up locally for frontend. I have tried and When i run the environment, i keep getting white screens. How do you setup locally? please i need a guide to start contributing to this wonderful project

[worldly pulsar]

hey @glass seal , there is some issues with the app.

I think someone has raised a PR to solve those.

but it is not yet merged.

So, wait until it gets merged or pull that pr in your code.

[glass seal]

hey @worldly pulsar , thanks for the update. Can i still apply for the gsoc program without contributing? I don't know when the issue might be merged

[fading pilot]

The issue is about url in that is there in Backend.ts file , go to frontend/src/config/Backend.ts instead of two url just add one url 'http://localhost:8000/'

I hope your problem will be solved , thank you.

[abstract bear]

hy @fading pilot \

recently the backend setup was changed

make sure you have python 3.12 as as all the dependencies goes with it

or you could wait like a 2 days i am a pr on how to setup pictopy locally video

[fading pilot]

sure , thank you

[glass seal]

thanks

[fading pilot]

go on

[young jasper]

Hey @radiant mauve !
I went over the pictopy project and it looked really interesting to me, and the tech stack resonates with mine a lot as well!

I would like to start contributing to it asap, specially the rust backend and tauri, would you be able to point me to any feature requests or issues that I can attend to?

I would potentially like to apply to pictopy for this gsoc term.

I see that electron has been proposed for the native client gui framework, which would be ported from tauri...
Why is that? Isn't tauri supposed to be cross-platform?

Moreover if the backend is written in rust, using Tauri as the gui framework would be a better choice in my opinion. Correct me if I am wrong here.

[abstract bear]

hy penguizz is the only mentor

[young jasper]

I have previously contributed to organizations such as Graphite, https://graphite.rs/.
Even they prefer using Tauri as their gui framework of choice for their upcoming desktop app.

[abstract bear]

mentor

[young jasper]

I see

[worldly pulsar]

Thats great.

[young jasper]

Hoping to hear from you soon ^^

[marsh idol]

Yes, Tauri comes with many advantages, including better efficiency in terms of app size and RAM usage

Tauri is the best option right now.

[young jasper]

Yes absolutely, thats the main advantage of using Tauri over electron.

@marsh idol if you don't mind, would you be open to discuss what exactly are the problems that we are facing in pictopy? And what is being done right now to address them?

[marsh idol]

Sure

[young jasper]

So the idea page mentions there being some problems with using Rust and tauri, what is it specifically?

[crude prism]

@young jasper You are now level 3! :confetti_ball:

[marsh idol]

It is mostly the dependency issue with Tauri, every new contributor faces problems while setting up this project

@abstract bear is working on making setup videos

We have solved the shipping issue of this project with the newly added workflows

I think the ideas page is tentative, Tauri is the framework that we should stick to.

The main issue with our project is the accuracy of the YOLO models

They are not good enough

[young jasper]

Yes I heard about this.

I'm gonna try to work on this issue https://github.com/AOSSIE-Org/PictoPy/issues/237

[young jasper]

@marsh idol what is the preferred way of running the project locally while testing and developing? Through the docker file or just running it as is?

[marsh idol]

Without docker

[young jasper]

I have the project running locally without docker

[marsh idol]

With docker the experience is laggy, because you need to connect to a ip address for remote access to the window.

[young jasper]

have you guys ever considered using poetry as a package managing tool?

it can be used for both virtual environment management and package management, and sharing poetry projects is very easy as well

[marsh idol]

@worldly pulsar

[young jasper]

I'm unable to play any video through the app, is this a bug in the upstream branch or is it just me facing this problem?

[wraith creek]

I think the issue they're referring to in the project idea is we need to install different types of packages and dependencies on different platforms like on windows we need to setup visual studio c++ tools and on linux we need to install other packages

But I agree tauri has lot more advantages, this is a very small issue, i think in repo their already exists few scripts to install platform related dependencies for windows and ubuntu/debian based systems

So I too think porting from tauri to electron is not such a great idea

[marsh idol]

Yes it is issue on linux, you need to install some packages for it. On windows it works fine

[wraith creek]

we can simply put up a PR to fix it and setup will be a breeze then, so yeah setting up is not a issue

[young jasper]

hmm. I am using arch and I installed all the packages which were mentioned in the install scripts

had to do it manually because the packages are named differently in ubuntu and arch, but I believe I installed all of them

[marsh idol]

There are specific packages for playing videos, you can check console by right clicking and then inspecting. You may see some errors when you switch to videos page

[young jasper]

When I run cargo tauri info I get this output, seems like everything is installed properly here

❯ cargo tauri info

[✔] Environment
    - OS: Arch Linux Unknown x86_64 (X64) (KDE on wayland)
    ✔ webkit2gtk-4.1: 2.46.6
    ✔ rsvg2: 2.59.2
    ✔ rustc: 1.85.0 (4d91de4e4 2025-02-17)
    ✔ cargo: 1.85.0 (d73d2caf9 2024-12-31)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-x86_64-unknown-linux-gnu (environment override by RUSTUP_TOOLCHAIN)
    - node: 22.14.0
    - yarn: 1.22.22
    - npm: 11.1.0
    - bun: 1.2.4

[-] Packages
    - tauri 🦀: 2.0.0-beta.23
    - tauri-build 🦀: 2.0.0-beta.18
    - wry 🦀: 0.41.0
    - tao 🦀: 0.28.1
    - tauri-cli 🦀: 2.3.1
    - @tauri-apps/api : 2.2.0 (outdated, latest: 2.3.0)
    - @tauri-apps/cli : 2.3.1

[-] Plugins
    - tauri-plugin-fs 🦀: 2.0.0-beta.10
    - @tauri-apps/plugin-fs : 2.2.0
    - tauri-plugin-dialog 🦀: 2.0.0-beta.10
    - @tauri-apps/plugin-dialog : 2.2.0
    - tauri-plugin-store 🦀: git+https://github.com/tauri-apps/plugins-workspace?branch=v2#66e8dbfc1a5cbeff3846d9ccffb57c45eb0de51e (2.0.0-beta.9)
    - @tauri-apps/plugin-store : not installed!
    - tauri-plugin-shell 🦀: 2.0.0-beta.8
    - @tauri-apps/plugin-shell : 2.2.0

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../dist
    - devUrl: http://localhost:1420/
    - framework: React
    - bundler: Vite

Inspecting the console gives this output

[Error] Unhandled Promise Rejection: NotSupportedError: The operation is not supported.
    play
    togglePlay (NetflixStylePlayer.tsx:69)
    handleContainerClick (NetflixStylePlayer.tsx:147)
    callCallback2 (chunk-RPCDYKBN.js:3674)
    dispatchEvent
    invokeGuardedCallbackDev (chunk-RPCDYKBN.js:3699)
    invokeGuardedCallback (chunk-RPCDYKBN.js:3733)
    invokeGuardedCallbackAndCatchFirstError (chunk-RPCDYKBN.js:3736)
    executeDispatch (chunk-RPCDYKBN.js:7014)
    processDispatchQueueItemsInOrder (chunk-RPCDYKBN.js:7034)
    processDispatchQueue (chunk-RPCDYKBN.js:7043)
    dispatchEventsForPlugins (chunk-RPCDYKBN.js:7051)
    batchedUpdates$1 (chunk-RPCDYKBN.js:18913)
    batchedUpdates (chunk-RPCDYKBN.js:3579)
    dispatchEventForPluginEventSystem (chunk-RPCDYKBN.js:7173)
    dispatchEventWithEnableCapturePhaseSelectiveHydrationWithoutDiscreteEventReplay (chunk-RPCDYKBN.js:5478)
    dispatchEvent (chunk-RPCDYKBN.js:5472:93)
    dispatchDiscreteEvent (chunk-RPCDYKBN.js:5449)

I'm not able to make out what package I'm missing 🤷‍♂️

[marsh idol]

No that package is not part of the rust dependency, you need to install it with your distros' package manager. I forgot that package's name on Ubuntu

[crude prism]

@marsh idol You are now level 13! :confetti_ball:

[marsh idol]

You should check the console when you are running the app, maybe you can find some error being thrown, I was able to find that error via console of the app on Ubuntu

[young jasper]

I understood that part, I was just making sure that tauri is running as it should

[crude prism]

@young jasper You are now level 4! :confetti_ball:

[young jasper]

this is the error message I get in the console, and its not helping much 🤷‍♂️

[Error] Unhandled Promise Rejection: NotSupportedError: The operation is not supported.
    play
    togglePlay (NetflixStylePlayer.tsx:69)
    handleContainerClick (NetflixStylePlayer.tsx:147)
    callCallback2 (chunk-RPCDYKBN.js:3674)
    dispatchEvent
    invokeGuardedCallbackDev (chunk-RPCDYKBN.js:3699)
    invokeGuardedCallback (chunk-RPCDYKBN.js:3733)
    invokeGuardedCallbackAndCatchFirstError (chunk-RPCDYKBN.js:3736)
    executeDispatch (chunk-RPCDYKBN.js:7014)
    processDispatchQueueItemsInOrder (chunk-RPCDYKBN.js:7034)
    processDispatchQueue (chunk-RPCDYKBN.js:7043)
    dispatchEventsForPlugins (chunk-RPCDYKBN.js:7051)
    batchedUpdates$1 (chunk-RPCDYKBN.js:18913)
    batchedUpdates (chunk-RPCDYKBN.js:3579)
    dispatchEventForPluginEventSystem (chunk-RPCDYKBN.js:7173)
    dispatchEventWithEnableCapturePhaseSelectiveHydrationWithoutDiscreteEventReplay (chunk-RPCDYKBN.js:5478)
    dispatchEvent (chunk-RPCDYKBN.js:5472:93)
    dispatchDiscreteEvent (chunk-RPCDYKBN.js:5449)

still im trying to read about it on some forums if I can find the cause of this issue, maybe it is some missing package that I am not able to locate

[worldly pulsar]

@young jasper , I was working on adding a script to generate requirements.txt.

But got busy with other work.

if you want, you can work on it.

[young jasper]

my idea on it was, if we use poetry, we never have to worry about requirements.txt

[worldly pulsar]

Will you explain a bit, how it works?

[young jasper]

whenever you add a python module using poetry its lock file is generated, anyone with that lock file only needs to run poetry install to get all the dependencies installed

[worldly pulsar]

So, we need to use poetry instead of pip, right?

[marsh idol]

so like npm's package.json, the .toml file will get automatically updated whenever someone installs a package?

[young jasper]

yes

[marsh idol]

because pip doesn't update the .txt file

[worldly pulsar]

?

[young jasper]

something like this

[tool.poetry]
name = "dbot"
version = "0.1.0"
description = ""
authors = ["Your Name <[email protected]>"]
readme = "README.md"
package-mode = false

[tool.poetry.dependencies]
python = "^3.10.12"
discord = "^2.3.2"
python-dotenv = "^1.0.1"
discord-py-slash-command = "^4.2.1"
tk = "^0.1.0"


[build-system]
requires = ["poetry-core"]
build-backend = "poetry.core.masonry.api"

[worldly pulsar]

That would be great.

[marsh idol]

then poetry is better option, we don't need to have workflows

[worldly pulsar]

yeah.

[young jasper]

yes, for example-
poetry add <package>
poetry run python3 main.py

[marsh idol]

then what about virtual environments?

[young jasper]

the dependencies are automatically added in this file when you use poetry add, and lock file can be generated with poetry lock

[marsh idol]

we need to activate that right?

[worldly pulsar]

can Poetry add and change package-lock.json when new package gets added?

[young jasper]

poetry install creates a virtual env in ~/.local dir

[marsh idol]

we need to activate it?

[young jasper]

then it can either be activated with poetry shell which activates the env or you can add poetry run before any command and its ran via the virtual env

[round laurel]

Hey everyone, this is really interesting! I’ve heard a lot about Poetry but haven’t had much hands-on experience with it yet. One thing I’m curious about—how does Poetry handle version conflicts between dependencies? For example, if two libraries require different versions of the same dependency, does Poetry resolve it automatically, or do we have to intervene manually? Also, how does it compare to pip-tools in terms of managing lock files and resolving dependency trees? Would love to hear your thoughts

[young jasper]

I've been using poetry for quite some time now and it can even be added to github workflows

[worldly pulsar]

thats great.

It can be set to trigger on every push.

[young jasper]

2 versions of same package cannot exist, in the same environment that is

each project has its own envrionment generated by poetry

[young jasper]

and as for comparison to pip, I would say I have yet to encounter a situation where pip was better than poetry
it automatically resolves dependencies and installs them with whichever package you require it to install

[round laurel]

Got it @young jasper! but one more thing as far as i know pip can reuse packages from a shared system cache, while Poetry installs everything fresh for each project, doesn't this lead to redundant storage and slower setups?

[young jasper]

poetry can retrieve packages from shared cache as well

[crude prism]

@young jasper You are now level 5! :confetti_ball:

[young jasper]

Although I have encountered a bit more storage being used with poetry, but it is not much
And when it saves you from the headache of falling into dependency hell, in my opinion it is worth it.

[round laurel]

Ohh...then this sounds like a great initiative we could take!

[young jasper]

Poetry increases reproducibility, in the end that is what matters more.

[round laurel]

Should I help you with this? If you think I could contribute in any way, let me know about this. I've heard about Poetry before but haven't had the chance to work on it with someone—would love to explore it together! 🙂

[young jasper]

sure

but I think moving to poetry would be the maintainer's call?

[marsh idol]

yep

@radiant mauve sir's call.

[round laurel]

Yeah, obviously, only after the maintainers give the green flag for this.

[young jasper]

should I make a issue on this untill then?

[marsh idol]

yess

[worldly pulsar]

go ahead.

[glass flume]

Hi mentors,I hope you're doing well. I'm a first-time participant in GSoC and excited to contribute. I would love your guidance on understanding the organization’s projects, contribution process, and best practices.

Could you please share any tips or resources to help me get started? I’d really appreciate any advice on how to make meaningful contributions.Im specially interested to contribute to Pictopy @radiant mauve can u help me

[marsh idol]

You can check out good first issues #355 and #356

Also, you should setup the repository using these instructions:
https://github.com/AOSSIE-Org/PictoPy/blob/97286a91a395200e80e532ff2b11672a4107f1dd/README.md

don't refer the readme from the repository directly, the above link has updated readme file

[glass flume]

Thank you so much for your guidance!Rahul I’ll start working on it and will keep you updated on my progress. Really appreciate your help! 😊

[marsh idol]

Before starting the work on any issue, write a comment on its thread so that somebody else also doesn't start to work on it.

[round laurel]

Hey @marsh idol , I already commented on the issue and just saw your message. I really want to align with issue #356—I haven’t asked for #357, so if someone else wants to take it, they can. But I did notice someone had asked for that issue before too, so can I go ahead with #356? Also, I’m new to Pictopy, but I’m a Master Contributor for the Internet Archive. I’d love to work on these cool ideas, so it would be great if you could assign me that issue!

[crude prism]

@round laurel You are now level 2! :confetti_ball:

[marsh idol]

I have asked ritik for any updates, if he doesn't answer, you can start the work.

[round laurel]

Sounds good, @marsh idol ! Let me know if Ritik responds. If there's no update, I’d be happy to take it up

[glass flume]

yes Rahul done I commented on the issue #357 and started working on it

[marsh idol]

That issue has already been taken care of i.e. a PR is open for fixing this issue.

[glass flume]

okay @marsh idol i will find out another issue I can contribute to

Hey @marsh idol I saw issue #355 (Adding comments throughout the codebase). It looks like a simple task—can I take it up? Let me know if it's available!"

[marsh idol]

Yes

comment on its thread

[young jasper]

how does the AI tagging here work?

I have loaded a couple of folders but nothing is coming up in the AI Tagging section

Btw, @marsh idol @worldly pulsar I raised an issue regarding the package management we were talking about. https://github.com/AOSSIE-Org/PictoPy/issues/374

[abstract bear]

hy you can recent sr .no write that in you pr

when you make a pr

[young jasper]

I didn't get it.. what did you mean?

[abstract bear]

refer to the above message

from the mentor

[abstract bear]

this one

[abstract bear]

this one

[young jasper]

So should I name my SR. NO. after the latest merged PR or the most recent PR raised

[worldly pulsar]

Yeah.

That would be better.

[abstract bear]

Hy @rahul

I want to add download files on the landing page

Provide me those link

@marsh idol

Paaji jalde thoda

[marsh idol]

https://github.com/rahulharpal1603/PictoPy/releases/tag/app-v0.0.0

[abstract bear]

Sun x64 windows hai arch is Linux and third one Mac?

[marsh idol]

.dmg wala mac, .deb for ubuntu and .exe windows

[abstract bear]

Paaji free kab ho

Can we get on a call

[marsh idol]

midsems are going on

🥲

[abstract bear]

Even my mid sems are on

Whenever you are free

https://tenor.com/view/me-time-gif-10769821

[marsh idol]

This weekend

[abstract bear]

Ping me up

[abstract bear]

@marsh idol @wraith creek @mystic sigil @worldly pulsar