~fabrixxm

~fabrixxm/confy-dev

Last active a month ago

~fabrixxm/lesanaweb-dev

Last active 8 months ago

~fabrixxm/public-inbox

Last active 9 months ago
View more

Recent activity

[PATCH v2] Per-module logger, add cli switch to log debug a month ago

From fabrixxm to ~valhalla/lesana-devel

when using lesana as a library, having a per-module logger is better
instead of logging everything in root.

When `lesana` cli command is used, logger is configured to print simple
logline and loglevel is set to INFO, unless user pass the option `-v` or
`--verbose` which sets the level to DEBUG
---
Set loglevel as a library makes no sense...

 lesana/collection.py | 34 ++++++++++++++++++----------------
 lesana/command.py    | 40 ++++++++++++++++++++++++----------------
 lesana/types.py      |  9 +++++----
 scripts/lesana       | 11 ++++++++++-
 4 files changed, 57 insertions(+), 37 deletions(-)
[message trimmed]

[PATCH] Per-module logger, loglevel config as a library and as a cli a month ago

From fabrixxm to ~valhalla/lesana-devel

when using lesana as a library, having a per-module logger is better
instead of logging everything in root.
An env var `LESANA_LOGLEVEL` is used to set logging level for the
library.

When `lesana` cli command is used, logger is configured to print simple
logline and loglevel is set to INFO, unless user pass the option `-v` or
`--verbose` which sets the level to DEBUG
---
 lesana/__init__.py   |  8 ++++++++
 lesana/collection.py | 34 ++++++++++++++++++----------------
 lesana/command.py    | 40 ++++++++++++++++++++++++----------------
 lesana/types.py      |  9 +++++----
 scripts/lesana       | 11 ++++++++++-
[message trimmed]

[PATCH] Force LesadaDate.load() return `datetime.date` when parsing a month ago

From fabrixxm to ~valhalla/lesana-devel

---
 lesana/types.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lesana/types.py b/lesana/types.py
index 6e5d21d..677fbdf 100644
--- a/lesana/types.py
+++ b/lesana/types.py
@@ -278,7 +278,7 @@ class LesanaDate(LesanaType):
        # compatibility with dateutil before 2.8
        ParserError = getattr(dateutil.parser, 'ParserError', ValueError)
        try:
            return dateutil.parser.parse(data)
            return dateutil.parser.parse(data).date()
[message trimmed]

Re: [PATCH v3] Fix division by zero for events with zero duration 3 months ago

From Fabio to ~fabrixxm/confy-dev

Il giorno lun, 14/06/2021 alle 02.18 +0200, Evangelos Ribeiro Tzaras ha
scritto:
> --- a/src/pages.py
> +++ b/src/pages.py
> @@ -506,8 +506,14 @@ class EventDetailPage(BasePage):
>          w = self.get_allocated_width()
>  
>          dur = (obj.end.timestamp() - obj.start.timestamp())
> -        pos = (now.timestamp() - obj.start.timestamp())
> -        prc = min(max(pos/dur, 0.0), 1.0)
> +        if dur > 0:
> +            pos = (now.timestamp() - obj.start.timestamp())

If you move pos inside this branch...

Re: [PATCH v2] Fix division by zero for events with zero duration 3 months ago

From Fabio to ~fabrixxm/confy-dev

Il giorno dom, 13/06/2021 alle 02.38 +0200, Evangelos Ribeiro Tzaras ha
scritto:
> 
> +        prc = min(max(pos/dur, 0.0), 1.0) if dur > 0 else int(pos >=
> 0)

While it's correct (and more correct than the code I was writting to
fix this), it's a little to difficult to read.
I would prefer an explicit if/else, with some comments, something like

```
if dur > 0:
	prc = min(max(pos/dur, 0.0), 1.0)
else:

Re: [PATCH] Whitespace cleanups 3 months ago

From Fabio to ~fabrixxm/confy-dev

Merged,

thought I'm not so strict with whitespaces :)

Conf 0.6.0 released 3 months ago

From Fabio to ~fabrixxm/confy-dev

Confy 0.6.0 has been released

see changes here https://git.sr.ht/~fabrixxm/confy/refs/0.6.0

ArchLinux AUR package is updated
https://aur.archlinux.org/packages/confy/

Flatpak package is scheduled
https://flathub.org/apps/details/net.kirgroup.confy

Re: [PATCH] Modify primary menu to better fit GNOME guidelines 4 months ago

From Fabio to ~fabrixxm/confy-dev

Il 07/05/21 16:45, Sebastian Crane ha scritto:
> - Remove 'Quit' item
> - Move 'Keyboard shortcuts' item to below 'Preferences' item
> 
> Please see https://developer.gnome.org/hig/stable/primary-menus.html
> for the relevant GNOME design guidelines.

Thank you Sebastian!

I'll merge this patch as soon as possible 👍