scipion-em / scipion-pyworkflow

Underlying pyworkflow module for the Scipion framework
GNU General Public License v3.0
6 stars 5 forks source link

Project ifnormation: Modification time and creation date #492

Open Vilax opened 8 months ago

Vilax commented 8 months ago

In the list of the Scipion projects, just below of each project title there are two labels: Modified: It should informs about the time since the last modification (It is ok) Created: It should be the date when the project was created. Currently, it is the date of the last modification.

An example is shown below

image

azazellochg commented 8 months ago

Looks like we are simply displaying stat.st_ctime for the folder which is not correct. I think to get the correct creation time which is stored in each project.sqlite we would need to load() each project.. @pconesa ?

pconesa commented 8 months ago

I'd avoid reading the sqlite for this, actually it was removed to accelerate the loading and also remove one more process accessing the project sqlite.

It seems ctime only works for windows. For linux not.

stat.ST_CTIME The “ctime” as reported by the operating system. On some systems (like Unix) is the time of the last metadata change, and, on others (like Windows), is the creation time (see platform documentation for details).

What about, removing the creation time since is not a stable value?

delarosatrevin commented 8 months ago

It is a useful thing to know when a project is created. Reading a value from the project's SQLite does not have a big impact on performance.

On Thu, Feb 8, 2024 at 9:54 AM Pablo Conesa @.***> wrote:

I'd avoid reading the sqlite for this, actually it was removed to accelerate the loading and also remove one more process accessing the project sqlite.

It seems ctime only works for windows. For linux not.

stat.ST_CTIME The “ctime” as reported by the operating system. On some systems (like Unix) is the time of the last metadata change, and, on others (like Windows), is the creation time (see platform documentation for details).

What about, removing the creation time since is not a stable value?

— Reply to this email directly, view it on GitHub https://github.com/scipion-em/scipion-pyworkflow/issues/492#issuecomment-1934423257, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACAM6WFEX3AUJUQD5VVKFYTYSTYKRAVCNFSM6AAAAABCBZMOWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZUGQZDGMRVG4 . You are receiving this because you are subscribed to this thread.Message ID: @.***>

pconesa commented 8 months ago

Well user un the past have complained about the project list being slow.

El jue., 8 feb. 2024 17:02, Jose Miguel de la Rosa Trevin < @.***> escribió:

It is a useful thing to know when a project is created. Reading a value from the project's SQLite does not have a big impact on performance.

On Thu, Feb 8, 2024 at 9:54 AM Pablo Conesa @.***> wrote:

I'd avoid reading the sqlite for this, actually it was removed to accelerate the loading and also remove one more process accessing the project sqlite.

It seems ctime only works for windows. For linux not.

stat.ST_CTIME The “ctime” as reported by the operating system. On some systems (like Unix) is the time of the last metadata change, and, on others (like Windows), is the creation time (see platform documentation for details).

What about, removing the creation time since is not a stable value?

— Reply to this email directly, view it on GitHub < https://github.com/scipion-em/scipion-pyworkflow/issues/492#issuecomment-1934423257>,

or unsubscribe < https://github.com/notifications/unsubscribe-auth/ACAM6WFEX3AUJUQD5VVKFYTYSTYKRAVCNFSM6AAAAABCBZMOWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZUGQZDGMRVG4>

. You are receiving this because you are subscribed to this thread.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/scipion-em/scipion-pyworkflow/issues/492#issuecomment-1934439471, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAF7ZYMLAFKO5ZDLZYG5FWLYSTZIXAVCNFSM6AAAAABCBZMOWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZUGQZTSNBXGE . You are receiving this because you were mentioned.Message ID: @.***>