error saving files

I installed the API and run the next couple of code lines.
I got an error when I wanted to save the filing.
from secedgar.filings import FilingType, Filing
from datetime import datetime

filing = Filing(cik_lookup='aapl',
                filing_type=FilingType.FILING_10K,
                start_date=datetime(2015, 1, 1),
                end_date=datetime(2019, 1, 1))

filing.save('./reports')

The following error is rising when I run the last command:
`---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
 in
----> 1 filing.save('FFS')
~.conda\envs\Keras\lib\site-packages\secedgar\filings\filing.py in save(self, directory, dir_pattern, file_pattern)
222
223         loop = asyncio.get_event_loop()
--> 224         loop.run_until_complete(self.client.wait_for_download_async(inputs))
~.conda\envs\Keras\lib\asyncio\base_events.py in run_until_complete(self, future)
568         future.add_done_callback(_run_until_complete_cb)
569         try:
--> 570             self.run_forever()
571         except:
572             if new_task and future.done() and not future.cancelled():
~.conda\envs\Keras\lib\asyncio\base_events.py in run_forever(self)
523         self._check_closed()
524         if self.is_running():
--> 525             raise RuntimeError('This event loop is already running')
526         if events._get_running_loop() is not None:
527             raise RuntimeError(
RuntimeError: This event loop is already running
10it [00:01,  9.88it/s]`

Hi @matinashtiani, thanks for reporting this. Which version of secedgar are you using?

Quick workaround -> probably problem for all using a newer python version with asyncio
!pip install nest_asyncio
import nest_asyncio
nest_asyncio.apply()

Quick workaround -> probably problem for all using a newer python version with asyncio
!pip install nest_asyncio
import nest_asyncio
nest_asyncio.apply()

@fyx99 is this a 3.9 issue? Or does it begin earlier than that?

Quick workaround -> probably problem for all using a newer python version with asyncio
!pip install nest_asyncio
import nest_asyncio
nest_asyncio.apply()

@fyx99 is this a 3.9 issue? Or does it begin earlier than that?

probapbly, I am running on 3.9

@jackmoody11 do the tests run on 3.9?

Yes, they do @reteps. Not sure why they aren't catching this.

Hi @matinashtiani, thanks for reporting this. Which version of secedgar are you using?

I am using 0.3.2.

Hi @matinashtiani, thanks for reporting this. Which version of secedgar are you using?

I am using 0.3.2.
Also, my python version is 3.7.6

Ok, thanks for sharing this @matinashtiani. I will see if I can find the issue.

@reteps I think this might have something to do with this.

I think we just add nest_asyncio as a dependency.

I think we just add nest_asyncio as a dependency.

@reteps Would rather find the underlying issue and fix if we aren't using asyncio like it is intended/supposed to be used.

Hi - I ran into this problem with python 3.8.5 as well on Ubuntu. So far, I'm using nest_asyncio to get around this problem. Looking forward to a proper fix. Thanks.

Actually correcting myself... using nest_asyncio did not solve the problem, it appeared to be doing something, but no file was downloaded into the specified folder...

@jackmoody11 This answer gives us a explanation why this is illegal:
run_until_complete is not for running any number of arbitrary async functions synchronously, it is for running the main entry point of your entire async program. This constraint is not immediately apparent from the docs.

Since libraries like aiohttp will queue it's own entry point to run as a server and block the loop's synchronous operations using run_until_complete or run_forever, the event loop will already be running and you won't be able to run independent synchronous operations on that event loop and wait for it's result within that thread.

I'm not entirely sure how to resolve it, but the issue is most certainly in the wait_for_download_async function

Note:         async with await session.get(link) as response: -> async with session.get(link) as response: since we batch, we don't need to wait for session.

Final notes:

move client and conn to init of network client
remove fetch method, move into fetch_and_save method
make fetch_and_save global method, remove session parameter

I'll open PR to fix these, though I'm not sure that will resolve the issue.

Ok, it is added to 0.4.0

Hi @reteps , thanks for this, has this code been merged to master? If so I'm happy to test it.

It isn't because the code works fine without it on my machine haha. Just ran your script:
retep@desktop:~/repos/secedgar_test$ python3 --version
Python 3.8.5
retep@desktop:~/repos/secedgar_test$ python3
Python 3.8.5 (default, Jul 28 2020, 12:59:40)
[GCC 9.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import secedgar
>>> secedgar.__version__
'0.3.2'
>>>
retep@desktop:~/repos/secedgar_test$ python3 main.py
10it [00:01,  5.11it/s]

@esvhd Can you share your version of aiohttp?
>>> import aiohttp
>>> aiohttp.__version__
'3.6.3'

Here's mine. Got installed when I pip installed secedgar
# Name                    Version                   Build  Channel
aiohttp                   3.7.4                    pypi_0    pypi

I changed it to your version 3.6.3 but it still did not work.. Here's what I need:
# had to use nest_asyncio, otherwise code fails...
import nest_asyncio
nest_asyncio.apply()

from secedgar.filings import Filing, FilingType

# 10Q filings for Apple (ticker "aapl")
my_filings = Filing(cik_lookup='aapl', filing_type=FilingType.FILING_10Q)

my_filings.save('~/data/edgar')
# this runs with tqdm status bar.... but...

Then when in the folder, there is not data...

I’m running Ubuntu 20.04..
Sent from my iPhone

On 1 Mar 2021, at 21:53, Peter Stenger<EMAIL_ADDRESS>wrote:

What platform are you on? I am running this on ubuntu 18.04 inside windows WSL2.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi - I’m actually running Pythian 3.8.5 here.. odd
Sent from my iPhone

On 1 Mar 2021, at 21:58, Peter Stenger<EMAIL_ADDRESS>wrote:

Can you upgrade python to any version >=3.8.1 and test?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

So to confirm, we have identical python, sec-edgar, and platform, but mine works and yours does not.

@esvhd Can you show us what you get when you run pip list?

Appreciate your help folks.

Reply to this email directly, view it on GitHub, or unsubscribe.

@esvhd can you set up a new virtual environment where you only pip install secedgar and then run the script? And then let us know if the problem persists.

@esvhd Here is a link to a guide for something that @jackmoody11 outlined. https://help.dreamhost.com/hc/en-us/articles/115000695551-Installing-and-using-virtualenv-with-Python-3

Hi guys, let me try this later and will let you know. Thanks!
Sent from my iPhone

On 5 Mar 2021, at 23:19, Peter Stenger<EMAIL_ADDRESS>wrote:

@esvhd Here is a link to a guide for something that @jackmoody11 outlined. https://help.dreamhost.com/hc/en-us/articles/115000695551-Installing-and-using-virtualenv-with-Python-3
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@esvhd Just wanted to give you a reminder we are waiting on that.

I am going to mark this as closed and open a new issue for expansion of ~.


decompiler sometimes fails to identify +offset as member acess

Is your feature request related to a problem? Please describe.
often times i see things like

*(pstream->vtbl + 8)

because the decompiler sometimes doesn't see the +8 as a member access even tho i've declared  the types correctly.
Describe the solution you'd like
The type of pstream is build in a way that the above should actually be

pstream->vtbl->addref

Describe alternatives you've considered
A way to manually turn an offset into a member access would be nice too.

The whole thing looks like this in HLIL:
// IStream::AddRef
*(pstream->vtbl + 8)
guardCFDispatchFunction(pstream)

This is a very common pattern in windows binaries that are compiled with a control flow guard.
the guardCFDispatchFunction is essentially a jmp rax so the code puts pstream->vtbl->addref into rax and pstream into rcx as the first this argument. Which is why the whole thing looks kinda funky in HLIL.

this somewhat relates to #1760 but i think an improved detection of the member access would be a good thing regardless of the cfg, i just happened to notice it in those situations.

Are you on the latest dev build? If not could you please try switching builds? If you are on dev could you possibly share a binary? We may be missing an edge case. We can't reproduce this.

yeah im on the latest dev build and. i reanalyzed the binary with the newest build but i didnt start the project with the newest version, hope that doesnt cause any problems.
I cant publicly upload the binary but i can share it privately if you want.
Ill try to give some more context.
this is a hlil of one of these situations:
struct IMFByteStream* bytestream_cpy = bytestream
// HRESULT GetCapabilities(
//   DWORD *pdwCapabilities
// );
// Retrieves the characteristics of the byte stream.
*(bytestream_cpy->vtbl + 0x18)
int32_t errcode = guardCFDispatchFunction(jmp_rax)(bytestream_cpy, &bytestream_capabilities)

The types are:
struct IMFByteStream __packed
{
    struct IMFByteStream_vtbl* vtbl;
};

struct IMFByteStream_vtbl
{
    struct IUnknown_vtbl iunknown;
    void* get_capabilities; // this is 0x18

this is the asm:
mov     rcx, qword [rsp+0x68 {bytestream}]
lea     rdx, [rsp+0x60 {bytestream_capabilities}]
mov     rax, qword [rcx {IMFByteStream::vtbl}]
// HRESULT GetCapabilities(
//   DWORD *pdwCapabilities
// );
// Retrieves the characteristics of the byte stream.
mov     rax, qword [rax+0x18]
call    qword [rel guardCFDispatchFunctionPointer]  {guardCFDispatchFunction(jmp_rax)}

I found out something interesting tho.
I may be completely stupid but in this context IMFByteStream* would be the same as IMFByteStream_vtbl** because the definition of IMFByteStream makes it just another pointer, right?
Im asking because i just noticed that:
struct IMFByteStream_vtbl** bytestream_cpy = bytestream
(*bytestream_cpy)->get_capabilities
int32_t errcode = guardCFDispatchFunction(jmp_rax)(bytestream_cpy, &bytestream_capabilities)

works.
I just set bytestream_cpy to IMFByteStream_vtbl** instead of IMFByteStream*

This is likely a duplicate of #2730


Missing license

Hi,
I would like very much to use this to bootstrap a project, but there is no license, which means that I don't have any right to use this code. This should be clarified so people can actually use this.
Thanks

Hello, @antoine-aubry. I added MIT license now =)

Thanks :)


hyprlock just dosent work

Whenever I run hyprlock through the terminal or through they keybind, it just freezes the screen and dosen't unfreeze until I enter the password. I have set up a configuration file so I am not sure why it is not working. Any help would be appreciated.
Config File

post logs from hyprlock. Sounds like you dont have a config

[ERR] source= globbing error: found no match
[ERR] Config has errors:
Config error in file /home/nuero/.config/hypr/hyprlock.conf at line 2: source= globbing error: found no match
Proceeding ignoring faulty entries
[LOG]   | got iface: wl_seat v9
[LOG]    > Bound to wl_seat v9
[LOG]   | got iface: wl_data_device_manager v3
[LOG]   | got iface: wp_tearing_control_manager_v1 v1
[LOG]   | got iface: wp_fractional_scale_manager_v1 v1
[LOG]    > Bound to wp_fractional_scale_manager_v1 v1
[LOG]   | got iface: zxdg_output_manager_v1 v3
[LOG]   | got iface: wp_cursor_shape_manager_v1 v1
[LOG]    > Bound to wp_cursor_shape_manager_v1 v1
[LOG]   | got iface: zwp_idle_inhibit_manager_v1 v1
[LOG]   | got iface: zwp_relative_pointer_manager_v1 v1
[LOG]   | got iface: zxdg_decoration_manager_v1 v1
[LOG]   | got iface: wp_alpha_modifier_v1 v1
[LOG]   | got iface: zwlr_gamma_control_manager_v1 v1
[LOG]   | got iface: ext_foreign_toplevel_list_v1 v1
[LOG]   | got iface: zwp_pointer_gestures_v1 v3
[LOG]   | got iface: zwlr_foreign_toplevel_manager_v1 v3
[LOG]   | got iface: zwp_keyboard_shortcuts_inhibit_manager_v1 v1
[LOG]   | got iface: zwp_text_input_manager_v3 v1
[LOG]   | got iface: zwp_pointer_constraints_v1 v1
[LOG]   | got iface: zwlr_output_power_manager_v1 v1
[LOG]   | got iface: xdg_activation_v1 v1
[LOG]   | got iface: ext_idle_notifier_v1 v1
[LOG]   | got iface: ext_session_lock_manager_v1 v1
[LOG]    > Bound to ext_session_lock_manager_v1 v1
[LOG]   | got iface: zwp_input_method_manager_v2 v1
[LOG]   | got iface: zwp_virtual_keyboard_manager_v1 v1
[LOG]   | got iface: zwlr_virtual_pointer_manager_v1 v2
[LOG]   | got iface: zwlr_output_manager_v1 v4
[LOG]   | got iface: org_kde_kwin_server_decoration_manager v1
[LOG]   | got iface: hyprland_focus_grab_manager_v1 v1
[LOG]   | got iface: zwp_tablet_manager_v2 v1
[LOG]   | got iface: zwlr_layer_shell_v1 v5
[LOG]   | got iface: wp_presentation v1
[LOG]   | got iface: xdg_wm_base v6
[LOG]   | got iface: zwlr_data_control_manager_v1 v2
[LOG]   | got iface: zwp_primary_selection_device_manager_v1 v1
[LOG]   | got iface: xwayland_shell_v1 v1
[LOG]   | got iface: hyprland_toplevel_export_manager_v1 v2
[LOG]   | got iface: zwp_text_input_manager_v1 v1
[LOG]   | got iface: hyprland_global_shortcuts_manager_v1 v1
[LOG]   | got iface: zwlr_screencopy_manager_v1 v3
[LOG]    > Bound to zwlr_screencopy_manager_v1 v3
[LOG]   | got iface: wl_shm v1
[LOG]   | got iface: wl_drm v2
[LOG]   | got iface: zwp_linux_dmabuf_v1 v4
[LOG]    > Bound to zwp_linux_dmabuf_v1 v4
[LOG]   | got iface: wl_compositor v6
[LOG]    > Bound to wl_compositor v6
[LOG]   | got iface: wl_subcompositor v1
[LOG]   | got iface: wp_viewporter v1
[LOG]    > Bound to wp_viewporter v1
[LOG]   | got iface: wp_drm_lease_device_v1 v1
[LOG]   | got iface: wp_drm_lease_device_v1 v1
[LOG]   | got iface: wp_single_pixel_buffer_manager_v1 v1
[LOG]   | got iface: wl_output v4
[LOG]    > Bound to wl_output v4
[LOG] [core] dmabufFeedbackMainDevice
[LOG] output 48 make BNQ model ZOWIE RL LCD
[LOG] output 48 name HDMI-A-3
[LOG] output 48 description BNQ ZOWIE RL LCD T2J00961SL0 (HDMI-A-3)
[LOG] output 48 done
[LOG] Running on Hyprland
[LOG] Locking session
[LOG] onLockLocked called
[LOG] got fractional 1
[LOG] got fractional 1
[LOG] configure with serial 198
[LOG] Configuring surface for logical [Vector2D: x: 1920, y: 1080] and pixel [Vector2D: x: 1920, y: 1080]
[LOG] got fractional 1
[ERR] Invalid key down event (stray release event?)
[LOG] In grace and cursor moved more than 5px, unlocking!
[LOG] Unlocking session
[LOG] Unlocked, exiting!
[LOG] Reached the end, exiting

[ERR] source= globbing error: found no match
[ERR] Config has errors:
Config error in file /home/nuero/.config/hypr/hyprlock.conf at line 2: source= globbing error: found no match
Proceeding ignoring faulty entries

Is it not clear?

@nuerocist the wiki is somewhat unclear about this, but the default behavior of hyprlock is to show nothing at all, so you cannot tell its there.
make a file at ~/.config/hypr/hyprlock.conf and start adding elements (background, input field). You can find information on how to configure these on the wiki ^^

[ERR] source= globbing error: found no match
[ERR] Config has errors:
Config error in file /home/nuero/.config/hypr/hyprlock.conf at line 2: source= globbing error: found no match
Proceeding ignoring faulty entries

Is it not clear?

I fixed the error and still nothing happened. The logs are the same except the error at the top is gone now.

@nuerocist the wiki is somewhat unclear about this, but the default behavior of hyprlock is to show nothing at all, so you cannot tell its there.
make a file at ~/.config/hypr/hyprlock.conf and start adding elements (background, input field). You can find information on how to configure these on the wiki ^^
EDIT: i just noticed you linked your config file, it looks fine at first glance... are you sure you put it in the correct place?

I'm pretty sure it is in the correct place. If I place it anywhere else it says "File does not exist."

Right, looking at your config, it seems like this is your issue:
$hypr=~/.config/hypr
source=$hypr/colors.conf

iirc hyprland has issues with ~ when resolving paths, try using $HOME
instead.
idk why i didn't notice this earlier, maybe i got distracted by @vaxerski
jumping to conclusions earlier.
On Sat, 22 Jun 2024, 07:52 nuero, @.***> wrote:

@nuerocist https://github.com/nuerocist the wiki is somewhat unclear
about this, but the default behavior of hyprlock is to show nothing at all,
so you cannot tell its there.
make a file at ~/.config/hypr/hyprlock.conf and start adding elements
(background, input field). You can find information on how to configure
these on the wiki ^^
EDIT: i just noticed you linked your config file, it looks fine at first
glance... are you sure you put it in the correct place?
I'm pretty sure it is in the correct place. If I place it anywhere else it
says "File does not exist."
—
Reply to this email directly, view it on GitHub
https://github.com/hyprwm/hyprlock/issues/374#issuecomment-2183822739,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5LZQBVJMSADLPMCKU4HCLZIUGKPAVCNFSM6AAAAABJPL2VDOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBTHAZDENZTHE
.
You are receiving this because you commented.Message ID:
@.***>

$hypr = /home/nuero/.config/hypr/
source = $hypr/hyprland/colors.conf # for custom color
Like this? I'm not really sure how to use $HOME, sorry.

that would work, but i actually looked through my files and it seems you can also use relative paths:
source = ./colors.conf

It's giving me this error now:
[ERR] source= globbing error: read error
[ERR] Config has errors:
Config error in file /home/nuero/.config/hypr/hyprlock.conf at line 2: source= globbing error: read error
Proceeding ignoring faulty entries
[LOG]   | got iface: wl_seat v9
[LOG]    > Bound to wl_seat v9
[LOG]   | got iface: wl_data_device_manager v3
[LOG]   | got iface: wp_tearing_control_manager_v1 v1
[LOG]   | got iface: wp_fractional_scale_manager_v1 v1
[LOG]    > Bound to wp_fractional_scale_manager_v1 v1
[LOG]   | got iface: zxdg_output_manager_v1 v3
[LOG]   | got iface: wp_cursor_shape_manager_v1 v1
[LOG]    > Bound to wp_cursor_shape_manager_v1 v1
[LOG]   | got iface: zwp_idle_inhibit_manager_v1 v1
[LOG]   | got iface: zwp_relative_pointer_manager_v1 v1
[LOG]   | got iface: zxdg_decoration_manager_v1 v1
[LOG]   | got iface: wp_alpha_modifier_v1 v1
[LOG]   | got iface: zwlr_gamma_control_manager_v1 v1
[LOG]   | got iface: ext_foreign_toplevel_list_v1 v1
[LOG]   | got iface: zwp_pointer_gestures_v1 v3
[LOG]   | got iface: zwlr_foreign_toplevel_manager_v1 v3
[LOG]   | got iface: zwp_keyboard_shortcuts_inhibit_manager_v1 v1
[LOG]   | got iface: zwp_text_input_manager_v3 v1
[LOG]   | got iface: zwp_pointer_constraints_v1 v1
[LOG]   | got iface: zwlr_output_power_manager_v1 v1
[LOG]   | got iface: xdg_activation_v1 v1
[LOG]   | got iface: ext_idle_notifier_v1 v1
[LOG]   | got iface: ext_session_lock_manager_v1 v1
[LOG]    > Bound to ext_session_lock_manager_v1 v1
[LOG]   | got iface: zwp_input_method_manager_v2 v1
[LOG]   | got iface: zwp_virtual_keyboard_manager_v1 v1
[LOG]   | got iface: zwlr_virtual_pointer_manager_v1 v2
[LOG]   | got iface: zwlr_output_manager_v1 v4
[LOG]   | got iface: org_kde_kwin_server_decoration_manager v1
[LOG]   | got iface: hyprland_focus_grab_manager_v1 v1
[LOG]   | got iface: zwp_tablet_manager_v2 v1
[LOG]   | got iface: zwlr_layer_shell_v1 v5
[LOG]   | got iface: wp_presentation v1
[LOG]   | got iface: xdg_wm_base v6
[LOG]   | got iface: zwlr_data_control_manager_v1 v2
[LOG]   | got iface: zwp_primary_selection_device_manager_v1 v1
[LOG]   | got iface: xwayland_shell_v1 v1
[LOG]   | got iface: hyprland_toplevel_export_manager_v1 v2
[LOG]   | got iface: zwp_text_input_manager_v1 v1
[LOG]   | got iface: hyprland_global_shortcuts_manager_v1 v1
[LOG]   | got iface: zwlr_screencopy_manager_v1 v3
[LOG]    > Bound to zwlr_screencopy_manager_v1 v3
[LOG]   | got iface: wl_shm v1
[LOG]   | got iface: wl_drm v2
[LOG]   | got iface: zwp_linux_dmabuf_v1 v4
[LOG]    > Bound to zwp_linux_dmabuf_v1 v4
[LOG]   | got iface: wl_compositor v6
[LOG]    > Bound to wl_compositor v6
[LOG]   | got iface: wl_subcompositor v1
[LOG]   | got iface: wp_viewporter v1
[LOG]    > Bound to wp_viewporter v1
[LOG]   | got iface: wp_drm_lease_device_v1 v1
[LOG]   | got iface: wp_drm_lease_device_v1 v1
[LOG]   | got iface: wp_single_pixel_buffer_manager_v1 v1
[LOG]   | got iface: wl_output v4
[LOG]    > Bound to wl_output v4
[LOG] [core] dmabufFeedbackMainDevice
[LOG] output 48 make BNQ model ZOWIE RL LCD
[LOG] output 48 name HDMI-A-3
[LOG] output 48 description BNQ ZOWIE RL LCD T2J00961SL0 (HDMI-A-3)
[LOG] output 48 done
[LOG] Running on Hyprland
[LOG] Locking session
[LOG] onLockLocked called
[LOG] got fractional 1
[LOG] got fractional 1
[LOG] configure with serial 17298
[LOG] Configuring surface for logical [Vector2D: x: 1920, y: 1080] and pixel [Vector2D: x: 1920, y: 1080]
[LOG] got fractional 1
[ERR] Invalid key down event (stray release event?)
[LOG] In grace and cursor moved more than 5px, unlocking!
[LOG] Unlocking session
[LOG] Unlocked, exiting!
[LOG] Reached the end, exiting

Line 2 is source = $hypr/colors.conf # for custom color, probably colors.conf doesn't exist.

Line 2 is source = $hypr/colors.conf # for custom color, probably colors.conf doesn't exist.

/home/nuero/.config/hypr/hyprland/colors.conf
It exists here.

Line 2 is source = $hypr/colors.conf # for custom color, probably colors.conf doesn't exist.

/home/nuero/.config/hypr/hyprland/colors.conf
It exists here.

In your linked config, you are defining $hypr as:
$hypr = ~/.config/hypr
Your are then sourcing your colors as:
source = $hypr/colors.conf # for custom color
This would resolve to:
/home/nuero/.config/hypr/colors.conf
But in the quoted reply, you said it exists at:
/home/nuero/.config/hypr/hyprland/colors.conf
Try changing line 2 to:
source = $hypr/hyprland/colors.conf # for custom color

Line 2 is source = $hypr/colors.conf # for custom color, probably colors.conf doesn't exist.

/home/nuero/.config/hypr/hyprland/colors.conf
It exists here.

In your linked config, you are defining $hypr as: $hypr = ~/.config/hypr
Your are then sourcing your colors as: source = $hypr/colors.conf # for custom color
This would resolve to: /home/nuero/.config/hypr/colors.conf
But in the quoted reply, you said it exists at: /home/nuero/.config/hypr/hyprland/colors.conf
Try changing line 2 to: source = $hypr/hyprland/colors.conf # for custom color

Did what you said, does not work. When I run hyprlock it just freezes my screen until I type in my password.

what does ls -al ~/.config/hypr output?

total 12
drwxr-xr-x 1 nuero nuero  114 Jul 18 16:38 ./
drwxr-xr-x 1 nuero nuero 1158 Jul  5 17:45 ../
drwxr-xr-x 1 nuero nuero  106 May 29 18:26 custom/
drwxr-xr-x 1 nuero nuero  128 Jul 18 16:40 hyprland/
-rw-r--r-- 1 nuero nuero  597 Jun 18 20:25 hyprland.conf
-rw-r--r-- 1 nuero nuero 5383 Jul 18 16:38 hyprlock.conf
drwxr-xr-x 1 nuero nuero  164 May 29 18:26 shaders/
drwxr-xr-x 1 nuero nuero   20 Jun 18 18:10 wallpapers/

what about ls -al ~/.config/hypr/hyprland?

total 36
drwxr-xr-x 1 nuero nuero   128 Jul 18 16:40 ./
drwxr-xr-x 1 nuero nuero   114 Jul 18 16:38 ../
-rw-r--r-- 1 nuero nuero  1046 Jul 18 16:40 colors.conf
-rw-r--r-- 1 nuero nuero   668 May 29 18:23 env.conf
-rw-r--r-- 1 nuero nuero   922 Jun 12 20:31 execs.conf
-rw-r--r-- 1 nuero nuero  4672 May 29 18:23 general.conf
-rw-r--r-- 1 nuero nuero 11638 Jun 18 18:12 keybinds.conf
-rw-r--r-- 1 nuero nuero  2074 May 29 18:23 rules.conf

can you paste those first couple of lines of your hyprlock.conf?

# This file sources other files in 'hyprland' and 'custom’ folders
# You wanna add your stuff in file in  'custom'
# Defaults
source=~/.config/hypr/hyprland/env. conf
source=~/ .config/hypr/hyprland/execs. conf
source=~/.config/hypr/hyprland/general. conf
source=~/.config/hypr/hyprland/rules. conf
source=~/ .config/hypr/hyprland/colors. conf
source=~/ .config/hypr/hyprland/keybinds. conf
Custon
source=~/.config/hypr/custon/env.conf
source=~/.config/hypr/custon/execs..conf
source=~/.config/hypr/custon/general . conf
source=~/.config/hypr/custon/ules..conf
source=~/.config/hypr/custon/keybinds.conf
These?

Wait sorry I showed my hyprland.conf

was wondering....

$hypr = ~/.config/hypr
source = Shypr/hyprland/colors.conf
Here you go.

try using relative path?
so source=hyprland/colors.conf

Also... you're not sourcing more files in your colors.conf are you?

try using relative path? so source=hyprland/colors.conf

Same as before sadly.

can u maybe add ur colors.conf to that gist?

and does it work if you comment those first two lines out?

Also... you're not sourcing more files in your colors.conf are you?

# exec = export SLURP_ARGS='-d -c DDE1F9BB -b 41465944 -5 00080000"
general {
col.active_border = rgba(E3E2E939)
col.inactive_border = rgba(8F9a9A30) }
misc {
background_color = rgba(121318FF)
}
plugin {
hyprbars {
# Honestly idk if it works like css, but well, why not
bar_text_font = Rubik, Geist, AR One Sans, Reddit Sans, Inter, Roboto, Ubuntu, Noto Sans, sans-serif
bar_height = 30
bar_padding = 10
bar_button_padding = 5
bar_precedence_over_border = true
bar_part_of_window = true
bar_color = rgba(121318FF)
col.text = rgba(E3E2E9FF)
# exanple buttons [fR -> L]
# hyprbars-button = color, size, on-click
hyprbars-button = rgb(E3E2E9), 13, x, hyprctl dispatch killactive
hyprbars-button = rgb(E3E2E9), 13, &, hyprctl dispatch fullscreen 1
hyprbars-button = rgb(E3E2£9), 13, -, hyprctl dispatch movetoworkspacesilent special
}
windowrulev2 = bordercolor rgba(B3CSFFAA) rgba(B3CSFF77),pinned:1
This is my colors.conf

can u maybe add ur colors.conf to that gist?

Like add it into .config/hypr/ instead of /hypr/hyprland?

no no no
make your hyprlock,conf start with:
# $hypr = ~/.config/hypr
# source = $hypr/colors.conf # for custom color

and does it work if you comment those first two lines out?

Config error in file /home/nuero/.config/hypr/hyprlock.conf at line 10: config option <general:ignore_empty_input> does not exist.
Gives this error

oh that means ur colors is working now...

oh that means ur colors is working now... Try commenting out the ignore_empty_input line for now?
We will look at that soon....
Do you know what version of hyprlock you have installed?

hyprlock-git-0.3.0.r30.gc5b8ad03-1
This one

update it
mine is hyprlock-git 0.4.0.r11.g69d37d26-1

If you get an error complaining about libhyprutils.so.0 missing after updating, that appears to be because of changes in the libhypr package, and can be resolved by:
sudo ln -s /usr/lib/libhyprutils.so.1 /usr/lib/libhyprutils.so.0

I decided to do a full system update and it gave me the error:
installing libdisplay-info (0.2.0-1) breaks dependency 'libdisplay-info.so=1-64' required by hyprland-git
It won't let me update

how did you install hyprlock?
use an aur helper? or just makepkg from the aur?
or did you build it from source?

I used yay

ahhh
do yay -S hyprlock

ahhh do yay -S hyprlock-git

Tried doing that. Gave me the error:
CMake Error at /usr/share/cmake/Modules/FindPkgConfig.cmake:634 (message):
The following required packages were not found:
- hyprutils>=0.2.0
Call Stack (most recent call first):
 /usr/share/cmake/Modules/FindPkgConfig.cmake:862 (_pkg_check_modules_internal)
CMakeLists.txt:42 (pkg_check_modules)
I did what you said with the libhyprutils and it said:
ln: failed to create symbolic link '/usr/lib/libhyprutils.so.0': File exists

yay -S hyprlock-git -ddd
That will cause you to have the same error i mentioned above about libhyprutils.so.0

Sorry I don't understand... Do I do yay -S hyprlock-git -ddd?

Assuming that yay passes additional flags to pacman, then yes.
I use paru, and thats how i got mine installed
the hyprutils thing is why i am even here browsing the issues 🤣

or maybe yay -Sddd hyprlock-git

or maybe yay -Sddd hyprlock-git

This gives the same error. lol
should I try with paru??

ohhhh hang on...
Does this work?
yay -S hyprutils-git -ddd
yay -S hyprlock

ohhhh hang on... Does this work?
yay -S hyprutils-git -ddd
yay -S hyprlock-git

Yup that installed

nice...
does hyprlock launch now?

It still doesn't work... I had to comment out a lot of things since It was giving me errors and after I did all that it did the same thing. Just froze my screen

i mean... its SUPPOSED to freeze your screen...
ur just not getting any display because it isnt reading your config...
try renaming ur hyprlock.conf to hyprlock.conf.bak, and putting a minimal one in place for a test...
something like this:
general {
  disable_loading_bar =	true # disables the loading bar on the bottom of the screen while hyprlock is booting up (bool = default: false)
  hide_cursor =	true # hides the cursor instead of making it visible (bool = default: false)
  grace =	0 # the amount of seconds for which the lockscreen will unlock on mouse movement (int = default: 0)
  no_fade_in =	false # disables the fadein animation (bool = default: false)
  no_fade_out =	false # disables the fadeout animation (bool = default: false)
  ignore_empty_input =	false # skips validation when no password is provided (bool = default: false)
  immediate_render =	false # makes hyprlock immediately start to draw widgets. Backgrounds will render background:color until their background:path resource is available (bool = default: false)
  pam_module =	hyprlock # sets the pam module used for authentication. If the module isn’t found in /etc/pam.d, “su” will be used as a fallback (str = default: hyprlock)
  text_trim =	true # sets if the text should be trimmed, useful to avoid trailing newline in commands output (bool = default: true)
}
label {
    monitor =
    text = Hi there, $USER
    text_align = center # center/right or any value for default left. multi-line text alignment inside label container
    color = rgba(200, 200, 200, 1.0)
    font_size = 25
    font_family = Noto Sans
    rotate = 0 # degrees, counter-clockwise

    position = 0, -20
    halign = center
    valign = center
}
input-field {
  monitor =
  size = 250, 50
  outline_thickness = 3
  dots_size = 0.33 # Scale of input-field height, 0.2 - 0.8
  dots_spacing = 0.15 # Scale of dots' absolute size, 0.0 - 1.0
  dots_center = true
  dots_rounding = -1 # -1 default circle, -2 follow input-field rounding
  outer_color = rgb(15, 15, 15)
  inner_color = rgb(200, 200, 200)
  font_color = rgb(10, 10, 10)
  fade_on_empty = true
  fade_timeout = 1000 # Milliseconds before fade_on_empty is triggered.
  placeholder_text = <i>Input Password...</i> # Text rendered in the input box when it's empty.
  hide_input = false
  rounding = -1 # -1 means complete rounding (circle/oval)
  check_color = rgb(204, 136, 34)
  fail_color = rgb(204, 34, 34) # if authentication failed, changes outer_color and fail message color
  fail_text = <i>$FAIL <b>($ATTEMPTS)</b></i> # can be set to empty
  fail_timeout = 2000 # milliseconds before fail_text and fail_color disappears
  fail_transition = 300 # transition time in ms between normal outer_color and fail_color
  capslock_color = -1
  numlock_color = -1
  bothlock_color = -1 # when both locks are active. -1 means don't change outer color (same for above)
  invert_numlock = false # change color if numlock is off
  swap_font_color = false # see below

  position = 0, -80
  halign = center
  valign = center

  shadow_passes = 0 # passes for shadow, 0 to disable (int - default: 0)
  shadow_size = 5 # size for shadow (int - default: 3)
  shadow_color = rgb(0,0,0) # shadow color (color - default: rgb(0,0,0))
  shadow_boost = 1.2 # boost shadow’s opacity (float - default: 1.2)
}

It made me comment out a lot of things, and then did the same thing. Not sure what is wrong.

what does the console output say?

[LOG]   | got iface: wl_seat v9
[LOG]    > Bound to wl_seat v9
[LOG]   | got iface: wl_data_device_manager v3
[LOG]   | got iface: wp_tearing_control_manager_v1 v1
[LOG]   | got iface: wp_fractional_scale_manager_v1 v1
[LOG]    > Bound to wp_fractional_scale_manager_v1 v1
[LOG]   | got iface: zxdg_output_manager_v1 v3
[LOG]   | got iface: wp_cursor_shape_manager_v1 v1
[LOG]    > Bound to wp_cursor_shape_manager_v1 v1
[LOG]   | got iface: zwp_idle_inhibit_manager_v1 v1
[LOG]   | got iface: zwp_relative_pointer_manager_v1 v1
[LOG]   | got iface: zxdg_decoration_manager_v1 v1
[LOG]   | got iface: wp_alpha_modifier_v1 v1
[LOG]   | got iface: zwlr_gamma_control_manager_v1 v1
[LOG]   | got iface: ext_foreign_toplevel_list_v1 v1
[LOG]   | got iface: zwp_pointer_gestures_v1 v3
[LOG]   | got iface: zwlr_foreign_toplevel_manager_v1 v3
[LOG]   | got iface: zwp_keyboard_shortcuts_inhibit_manager_v1 v1
[LOG]   | got iface: zwp_text_input_manager_v3 v1
[LOG]   | got iface: zwp_pointer_constraints_v1 v1
[LOG]   | got iface: zwlr_output_power_manager_v1 v1
[LOG]   | got iface: xdg_activation_v1 v1
[LOG]   | got iface: ext_idle_notifier_v1 v1
[LOG]   | got iface: ext_session_lock_manager_v1 v1
[LOG]    > Bound to ext_session_lock_manager_v1 v1
[LOG]   | got iface: zwp_input_method_manager_v2 v1
[LOG]   | got iface: zwp_virtual_keyboard_manager_v1 v1
[LOG]   | got iface: zwlr_virtual_pointer_manager_v1 v2
[LOG]   | got iface: zwlr_output_manager_v1 v4
[LOG]   | got iface: org_kde_kwin_server_decoration_manager v1
[LOG]   | got iface: hyprland_focus_grab_manager_v1 v1
[LOG]   | got iface: zwp_tablet_manager_v2 v1
[LOG]   | got iface: zwlr_layer_shell_v1 v5
[LOG]   | got iface: wp_presentation v1
[LOG]   | got iface: xdg_wm_base v6
[LOG]   | got iface: zwlr_data_control_manager_v1 v2
[LOG]   | got iface: zwp_primary_selection_device_manager_v1 v1
[LOG]   | got iface: xwayland_shell_v1 v1
[LOG]   | got iface: hyprland_toplevel_export_manager_v1 v2
[LOG]   | got iface: zwp_text_input_manager_v1 v1
[LOG]   | got iface: hyprland_global_shortcuts_manager_v1 v1
[LOG]   | got iface: zwlr_screencopy_manager_v1 v3
[LOG]    > Bound to zwlr_screencopy_manager_v1 v3
[LOG]   | got iface: wl_shm v1
[LOG]   | got iface: wl_drm v2
[LOG]   | got iface: zwp_linux_dmabuf_v1 v4
[LOG]    > Bound to zwp_linux_dmabuf_v1 v4
[LOG]   | got iface: wl_compositor v6
[LOG]    > Bound to wl_compositor v6
[LOG]   | got iface: wl_subcompositor v1
[LOG]   | got iface: wp_viewporter v1
[LOG]    > Bound to wp_viewporter v1
[LOG]   | got iface: wp_drm_lease_device_v1 v1
[LOG]   | got iface: wp_drm_lease_device_v1 v1
[LOG]   | got iface: wp_single_pixel_buffer_manager_v1 v1
[LOG]   | got iface: wl_output v4
[LOG]    > Bound to wl_output v4
[LOG] [core] dmabufFeedbackMainDevice
[LOG] output 48 make BNQ model ZOWIE RL LCD
[LOG] output 48 name HDMI-A-3
[LOG] output 48 description BNQ ZOWIE RL LCD T2J00961SL0 (HDMI-A-3)
[LOG] output 48 done
[LOG] Running on Hyprland
[LOG] Locking session
[LOG] onLockLocked called
[LOG] got fractional 1
[LOG] got fractional 1
[LOG] configure with serial 13649
[LOG] Configuring surface for logical [Vector2D: x: 1920, y: 1080] and pixel [Vector2D: x: 1920, y: 1080]
[LOG] got fractional 1
[ERR] Invalid key down event (stray release event?)
[LOG] Authenticating
[LOG] PAM: The account is locked due to 3 failed logins.
[LOG] PAM: (5 minutes left to unlock)
[ERR] auth: Authentication failed for hyprlock
[LOG] auth: authenticated for su
[LOG] Unlocking session
[LOG] Unlocked, exiting!
[LOG] Reached the end, exiting

did you use the original conf i posted, or the updated one with the background section?

I just used the one you sent

oh you changed it

yea... i hoped to get it before u copied 😛

Also by the way, it says my hyprlock is up to date but its only hyprlock-git-0.2.0.r20.g2448774d-1. It isn't 0.4.0 like yours..

ahhh i think u may be using the aur/hyprlock-git
im using chaotic-aur/hyprlock

interesting. how do I switch??

pacman-key --recv-key<PHONE_NUMBER>B78AEB --keyserver hkps://keyserver.ubuntu.com:443
pacman-key --lsign-key<PHONE_NUMBER>B78AEB
pacman -U --noconfirm 'https://cdn-mirror.chaotic.cx/chaotic-aur/chaotic-keyring.pkg.tar.zst' 'https://cdn-mirror.chaotic.cx/chaotic-aur/chaotic-mirrorlist.pkg.tar.zst'

then add the following to the end of /etc/pacman.conf:
[chaotic-aur]
Include = /etc/pacman.d/chaotic-mirrorlist

then do pacman -Syy to pull the package lists

then you can just pacman -S hyprlock-git

Thanks got it installed. But the hyprlock.conf is still not working. same thing as before

so its just locking the screen, but not displaying anything?

yup

and ur on v0.4.0

yeah im on v0.4.0

try doing the config i posted without commenting anything out?

It just gives me config errors then

what errors?

Config error in file /home/nuero/.config/hypr/hyprlock.conf at line 20: config option <general:no_fade_out> does not exist.

hrmmm.... what is the output of the following:
hyprctl systeminfo
and
hyprctl configerrors

Hyprland, built from branch main at commit a60c7283e677ed81a466d1c0c864c8a05e192cac  (xwayland: verify new xsurf is valid in prop reads).
Date: Wed May 29 19:34:25 2024
Tag: v0.40.0-144-ga60c7283, commits: 4750

flags: (if any)

System Information:
System name: Linux
Node name: coolcomputer
Release: 6.9.4-arch1-1
Version: #1 SMP PREEMPT_DYNAMIC Wed, 12 Jun 2024 20:17:17 +0000

GPU information:
01:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Ellesmere [Radeon RX 470/480/570/570X/580/580X/590] [1002:67df] (rev c7) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

nothing for configerrors

ok....
if you launch it like this: hyprlock -v > hyprlock.log
then log back in, can you post the resulting hyprlock.log file in ur gist?

log back in meaning after the screen freezes I put in my password?

after hyprlock locks your screen and you unlock it

I wonder if your problem is that ur on an older version of hyprland, so have an older version of hyprlang (the parser for the config files)
Try doing pacman -S hyprland

I'm guessing this isnt a custom built setup ur doing, and you have used something like hyprdots or ML4W to install ur system?

I'm guessing this isnt a custom built setup ur doing, and you have used something like hyprdots or ML4W to install ur hyprland setup?

Im using end-4 dotfiles. But i had hyprland installed before that

I wonder if your problem is that ur on an older version of hyprland, so have an older version of hyprlang (the parser for the config files)
Try doing pacman -S hyprland-git
that should get you from version hyprland-git 0.40.0-144-ga60c7283 to hyprland-git 0.41.2.r44.300228b5-1

just updated it

cna you

I'm guessing this isnt a custom built setup ur doing, and you have used something like hyprdots or ML4W to install ur hyprland setup?

Im using end-4 dotfiles. But i had hyprland installed before that

I kinda like the way they modularised the conf files and sourced them all seperately...
Might be time to do a refactor of my setup...

ok.... if you launch it like this: hyprlock -v > hyprlock.log then log back in, can you post the resulting hyprlock.log file in ur gist?

can you do this again?

what does hyprctl systeminfo say now?

Hyprland, built from branch main at commit a60c7283e677ed81a466d1c0c864c8a05e192cac  (xwayland: verify new xsurf is valid in prop reads).
Date: Wed May 29 19:34:25 2024
Tag: v0.40.0-144-ga60c7283, commits: 4750

flags: (if any)

System Information:
System name: Linux
Node name: coolcomputer
Release: 6.9.4-arch1-1
Version: #1 SMP PREEMPT_DYNAMIC Wed, 12 Jun 2024 20:17:17 +0000

GPU information:
01:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Ellesmere [Radeon RX 470/480/570/570X/580/580X/590] [1002:67df] (rev c7) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

quit hyprland and come back, its still reporting v0.40

quit hyprland and come back, its still reporting v0.40 Default keybind to quit is super+M

Hyprland broke lmao.. sending this from my phone. I try to log in and then it just logs me out

guh... ctrl+alt+F5 and login to the tty, then mv ~/.config/hypr/hyprland.conf ~/.conf/hypr/hyprland.old to get a fresh start

wtf?
ls ~/.config/hypr

and dont move the hyprland.conf... i just tried it myself, and hyprland doesn't generate a new one when one isnt found...

oh ok

i'll have to continue this tomorrow since its getting late. please comment what you think might fix it, i'll check tomorrow. thanks for your help

regardless, why couldnt it find ur hyprland.conf?

i'll have to continue this tomorrow since its getting late. please comment what you think might fix it, i'll check tomorrow. thanks for your help

need to see the output of ls ~/.config/hypr

ok so ur hyprland.conf is still there, phew :)
what output do you get if you run Hyprland?

This is what I get..

Ahhh so that is from that error you got when you tried to do a full system update

I decided to do a full system update and it gave me the error: installing libdisplay-info (0.2.0-1) breaks dependency 'libdisplay-info.so=1-64' required by hyprland-git
It won't let me update

This one....

yup

do sudo pacman -S libdisply-info

oh that fixed it!!

Fantastic! now you can msg on ur pc again :)

so how does hyprlock look now? still jsut lock ur screen without any display change?

I noticed hyprlock displays a red screen for a few seconds after I run it. but then it disappears

yea i get a red flash in terminal when i launch it there too... not sure why
but if i let hypridle handle it, no red flash

and if i use a keybind, i get a red flash if a terminal is my active window, but otherwise not... got me stumped why though

thats odd

I noticed hyprlock displays a red screen for a few seconds after I run it. but then it disappears

Are you getting a blurred background when you run hyprlock? or still just locks the screen with no change to your desktop?

it just locks my screen, no blur

thats odd
right?

it just locks my screen, no blur

what output do you get in the terminal when you run hyprlock

[ERR] Config has errors:
Config error in file /home/nuero/.config/hypr/hyprlock.conf at line 20: config option <general:no_fade_out> does not exist.
Proceeding ignoring faulty entries
[LOG]   | got iface: wl_seat v9
[LOG]    > Bound to wl_seat v9
[LOG]   | got iface: wl_data_device_manager v3
[LOG]   | got iface: wl_compositor v6
[LOG]    > Bound to wl_compositor v6
[LOG]   | got iface: wl_subcompositor v1
[LOG]   | got iface: wl_shm v1
[LOG]   | got iface: wp_viewporter v1
[LOG]    > Bound to wp_viewporter v1
[LOG]   | got iface: wp_tearing_control_manager_v1 v1
[LOG]   | got iface: wp_fractional_scale_manager_v1 v1
[LOG]    > Bound to wp_fractional_scale_manager_v1 v1
[LOG]   | got iface: zxdg_output_manager_v1 v3
[LOG]   | got iface: wp_cursor_shape_manager_v1 v1
[LOG]    > Bound to wp_cursor_shape_manager_v1 v1
[LOG]   | got iface: zwp_idle_inhibit_manager_v1 v1
[LOG]   | got iface: zwp_relative_pointer_manager_v1 v1
[LOG]   | got iface: zxdg_decoration_manager_v1 v1
[LOG]   | got iface: wp_alpha_modifier_v1 v1
[LOG]   | got iface: zwlr_gamma_control_manager_v1 v1
[LOG]   | got iface: ext_foreign_toplevel_list_v1 v1
[LOG]   | got iface: zwp_pointer_gestures_v1 v3
[LOG]   | got iface: zwlr_foreign_toplevel_manager_v1 v3
[LOG]   | got iface: zwp_keyboard_shortcuts_inhibit_manager_v1 v1
[LOG]   | got iface: zwp_text_input_manager_v3 v1
[LOG]   | got iface: zwp_pointer_constraints_v1 v1
[LOG]   | got iface: zwlr_output_power_manager_v1 v1
[LOG]   | got iface: xdg_activation_v1 v1
[LOG]   | got iface: ext_idle_notifier_v1 v1
[LOG]   | got iface: ext_session_lock_manager_v1 v1
[LOG]    > Bound to ext_session_lock_manager_v1 v1
[LOG]   | got iface: zwp_input_method_manager_v2 v1
[LOG]   | got iface: zwp_virtual_keyboard_manager_v1 v1
[LOG]   | got iface: zwlr_virtual_pointer_manager_v1 v2
[LOG]   | got iface: zwlr_output_manager_v1 v4
[LOG]   | got iface: org_kde_kwin_server_decoration_manager v1
[LOG]   | got iface: hyprland_focus_grab_manager_v1 v1
[LOG]   | got iface: zwp_tablet_manager_v2 v1
[LOG]   | got iface: zwlr_layer_shell_v1 v5
[LOG]   | got iface: wp_presentation v1
[LOG]   | got iface: xdg_wm_base v6
[LOG]   | got iface: zwlr_data_control_manager_v1 v2
[LOG]   | got iface: zwp_primary_selection_device_manager_v1 v1
[LOG]   | got iface: xwayland_shell_v1 v1
[LOG]   | got iface: wl_drm v2
[LOG]   | got iface: zwp_linux_dmabuf_v1 v5
[LOG]    > Bound to zwp_linux_dmabuf_v1 v5
[LOG]   | got iface: hyprland_toplevel_export_manager_v1 v2
[LOG]   | got iface: zwp_text_input_manager_v1 v1
[LOG]   | got iface: hyprland_global_shortcuts_manager_v1 v1
[LOG]   | got iface: zwlr_screencopy_manager_v1 v3
[LOG]    > Bound to zwlr_screencopy_manager_v1 v3
[LOG]   | got iface: wp_drm_lease_device_v1 v1
[LOG]   | got iface: wp_drm_lease_device_v1 v1
[LOG]   | got iface: wl_output v4
[LOG]    > Bound to wl_output v4
[LOG] [core] dmabufFeedbackMainDevice
[LOG] output 47 make BNQ model ZOWIE RL LCD
[LOG] output 47 name HDMI-A-3
[LOG] output 47 description BNQ ZOWIE RL LCD T2J00961SL0 (HDMI-A-3)
[LOG] output 47 done
[LOG] Running on Hyprland
[LOG] Locking session
[LOG] onLockLocked called
[LOG] got fractional 1
[LOG] got fractional 1
[LOG] configure with serial 937
[LOG] Configuring surface for logical [Vector2D: x: 1920, y: 1080] and pixel [Vector2D: x: 1920, y: 1080]
[LOG] got fractional 1
[LOG] Authenticating
[LOG] auth: authenticated for hyprlock
[LOG] Unlocking session
[LOG] Unlocked, exiting!
[LOG] Reached the end, exiting

still not parsing your config correctly...
I would try a sudo pacman -Syu now...
But be aware, thats what broke ur hyprland last time...
But to be fair, you need to be able to run a system update without breaking anything regardless of your display manager... so we should see if its still and issue and sort out whatever dependency issues u have so that u can know u have a nice stable machine

Alright, i'll start updating. this might take a while

No probs... ill be around if u run into any probs mate :)

all done, went smoothly

awesome :)
reboot and make sure everything starts up ok

yup, i rebooted and everything works

what happens now when you run hyprlock ?

same thing as before... red flash and then nothing

and the terminal output?
still:
[ERR] Config has errors:
Config error in file /home/nuero/.config/hypr/hyprlock.conf at line 20: config option <general:no_fade_out> does not exist.
Proceeding ignoring faulty entries

?

yes its still the same thing

ok... we will try the most minimal possible conf, is that cool?

sure. anything that works at this point haha

general {
  pam_module = hyprlock

}

background {
  monitor =
  path = screenshot # supports png, jpg, webp (no animations, though)
  blur_passes = 2 # 0 disables blurring
  blur_size = 7
  noise = 0.0117
}

that should just blur ur screen, until you put ur password

Red flash with this error:
[ERR] Config has errors:
Config error in file /home/nuero/.config/hypr/hyprlock.conf at line 2: config option <general:pam_module> does not exist.

Does not blur

gosh...
try sudo pacman -S hyprlang-git

hyprlang-git-0.5.2.r8.adbefbf-1 and hyprlang-0.5.2-1 are in conflict. Remove hyprlang?
should I?

maybe...
what is the versions you have of all the hyprland packages?

how do I check?

do: pacman -Qe hypr{ctl,idle,land,lock,paper,picker}

that should get most of them

error: package 'hyprctl' was not found
hypridle-git 0.1.2.r5.g7c5747b7-1
hyprland-git 0.41.2.r45.8e15f91c-1
hyprlock-git 0.4.0.r13.g9514925a-1
error: package 'hyprpaper' was not found
hyprpicker-git 0.3.0.r5.8791f71-1

then no, no need to do anything to hyprlang

oh ok

What happens if you remove the general section altogether from your conf?

It dosent give me any errors but it still does the red flash thingy. no blur

also, what is the output of which hyprlock

/usr/local/bin/hyprlock
hmmmmmm

what about ls /usr/bin/hyprlock?

/usr/bin/hyprlock*

i think you have an old version of hyprlock sitting in /usr/local/bin that u should rename, and potentially remove if that fixes things

wish i had asked you that a LOT sooner...
Here's how mine looks...

Glad we finally figured out what was happening mate!
I see were in the same timezone :P

Looks great!! also hey, youre in australia! i live in new zealand lol

Were almost like COUSINS!

thats right lmaoo

i guess its time to close this now! thanks for all of your help

Even happier i could help now, being that ur a fellow AUNZ

and yea... close this... glad we got you sorted

yup! again, thanks for the help


Allow configurable pester verbosity (and ideally inherit from Powershell extension)

When I click the run for the tests, I prefer detailed to normal output. There should be settings to mirror these settings and ideally inherit these settings if not explicity set otherwise.

So luckily the Debug Output Verbosity is used already because vscode-pester-test-adapter uses the PowerShell extension for debugging.
The next change would be to hook up powershell.pester.outputVerbosity to vscode-pester-test-adapter. Pretty simple to do, I think:
const value = vscode.workspace.getConfiguration('powershell').get<string>('pester.outputVerbosity');

and then pass that in to the script:
https://github.com/TylerLeonhardt/vscode-pester-test-adapter/blob/dfd4a1fcde1478d35c98f0c7f267f531961927e4/src/powershellScripts.ts#L110
Interested in touching a little typescript? :)

So luckily the Debug Output Verbosity is used already because vscode-pester-test-adapter uses the PowerShell extension for debugging.
The next change would be to hook up powershell.pester.outputVerbosity to vscode-pester-test-adapter. Pretty simple to do, I think:
const value = vscode.workspace.getConfiguration('powershell').get<string>('pester.outputVerbosity');

and then pass that in to the script:
https://github.com/TylerLeonhardt/vscode-pester-test-adapter/blob/dfd4a1fcde1478d35c98f0c7f267f531961927e4/src/powershellScripts.ts#L110
Interested in touching a little typescript? :)

@TylerLeonhardt if it annoys me enough, maybe :)

@TylerLeonhardt if it annoys me enough, maybe :)


Old ef Version referenced in IdentityBase.EntityFramework.IntegrationTests project

in every project is used the version 2.0.2 of Microsoft.EntityFrameworkCore, except for the IdentityBase.EntityFramework.IntegrationTests, that is still used the 2.0.1 version.
https://github.com/IdentityBaseNet/IdentityBase/blob/develop/test/IdentityBase.EntityFramework.IntegrationTests/IdentityBase.EntityFramework.IntegrationTests.csproj

Fixed in 196bc3c64cf8fe52a11d26a5f74aa457489ae802


feat: new methods for leaf ops

The struct definitions are now non_exhaustive, so they cannot be build with the Noop {ty: ..} syntax outside of this crate.

What are we anticipating adding to these structs in the future?

Probably nothing, but marking them as non_exhaustive lets us add private fields without breaking semver if we need to.
(We could also add public fields, but I think that's less probable).

@aborgna-q Could a new release be made that includes this change, perhaps a 0.3.1?
Our upgrade to hugr 0.3 is currently broken as new is missing in 0.3.

This is now published in hugr 0.3.1.
https://crates.io/crates/hugr/0.3.1


Adding TeachableHub to MLOps Platforms

Hello,
I would like to add our column to the spreadsheet.
File: MLOps Platforms Comparison Matrix - Public Sheet.xlsx
Changes: Added column 'P' providing information for TeachableHub, no other changes has been made.
Thanks in advance!
Pavel Ivanov
Co-founder and CTO of TeachableHub

Could you expand the documentation links a bit? I currently only see links in 3 out of 15 cells:

It would be good to refer to product documentation for serving features, rollouts, monitoring, model registry, etc.


mocha.opts is not processed properly

I wan't to specify which files are my test files with the following pattern: test/**/*spec.js. Since I run my tests with nodejs using mocha from command line and using karma start tdd.karma.conf.js --single-run, I thought it would be best to put this info into the mocha.opts file, so it would be loaded by both the nodejs version and the browserified version, which is used by karma.
I use karma with the following config file currently:
module.exports = function (config) {
    config.set({
        plugins: [
            "karma-browserify",
            "karma-phantomjs-launcher",
            "karma-mocha"
        ],
        frameworks: ["browserify", "mocha"],
        files: [
            "index.js",
            "src/**/*",
            "test/**/*"
        ],
        preprocessors: {
            "index.js": ["browserify"],
            "src/**/*.js": ["browserify"],
            "test/**/*.js": ["browserify"]
        },
        client: {
            mocha: {
                reporter: "html",
                ui: "bdd"
            }
        },
        browserify: {
            debug: true
        },
        browsers: ["PhantomJS"],
        reporters: ["progress"],
        port: 9876,
        colors: true,
        logLevel: config.LOG_INFO,
        autoWatch: true,
        captureTimeout: 6000,
        singleRun: false
    });
};

So by starting the karma server with karma start tdd.karma.conf.js I will be able to develop and run the tests automatically with watch and by starting with karma start tdd.karma.conf.js --single-run I will be able to automatically test in a pre-commit hook to prevent committing code which does not pass the tests.
By putting test/**/*spec.js into the mocha.opts it runs properly with nodejs, but I got the following error with karma:
PhantomJS 1.9.8 (Windows 7 0.0.0) ERROR
  ReferenceError: Can't find variable: test
  at D:/creation/software developer/projects/javascript-project-template/test/mocha.opts:1

By putting "test/**/*spec.js" into the mocha.opts it runs properly with karma, but I got the following error with nodejs:
D:\creation\software developer\projects\javascript-project-template\node_modules\mocha\lib\utils.js:628
        throw new Error("cannot resolve path (or pattern) '" + path + "'");
        ^

Error: cannot resolve path (or pattern) '"test/**/*spec.js"'
    at Object.lookupFiles (D:\creation\software developer\projects\javascript-project-template\node_modules\mocha\lib\utils.js:628:15)
    at D:\creation\software developer\projects\javascript-project-template\node_modules\mocha\bin\_mocha:326:30
    at Array.forEach (native)
    at Object.<anonymous> (D:\creation\software developer\projects\javascript-project-template\node_modules\mocha\bin\_mocha:325:6)
    at Module._compile (module.js:435:26)
    at Object.Module._extensions..js (module.js:442:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:311:12)
    at Function.Module.runMain (module.js:467:10)
    at startup (node.js:134:18)
    at node.js:961:3

So the mocha.opts is not processed properly by using it with karma-mocha. Is there a way to workaround this issue until it is fixed, e.g. by giving the mocha parameter in the karma config file?

Hi !
I have the same issue.
I have created a PR with a solution to support the opts file.
Only the --require flag is missing in my solution. It would be nice to do a bundle.
Here is the PR: https://github.com/karma-runner/karma-mocha/pull/99
Let me know what you think about it.
Jean.

Now plugin support mocha.opts. Look at this PR


[TEMPLATE] - Object Detection with YOLO - DataScience

What domain will the project template be based on?
Data Science
If 'Others', please specify the preferred domain.
No response
Particular directory path for the template.
templates/Data-Science/Deep-Learning/ObjectDetectionYOLO/
What tech stacks do you want to include?
Python, OpenCV, TensorFlow
Brief Description
This project template implements the YOLO algorithm for real-time object detection focused on detecting ambulances and other vehicles in traffic scenarios.

References : COCO Dataset, Open Images Dataset, Cityscapes Dataset, KITTI Dataset, Berkeley DeepDrive Dataset

Note : A Jupyter notebook in form of a collab notebook should be linked in the README.md, and a user-friendly UI should be built using Flask or Streamlit.
👀 Have you spent some time checking if this issue has been raised before?

[X] Yes
[ ] No

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Would you like to work on this issue?
No

Hi, I have prior experience with working with yolov8 finetuned model on nearly real-time applications.
Would love to contribute.
I have read the code of conduct.

Assigning this issue to you
If you face any issue feel free to contact me via Discord

Hey @shubhamsahu03 are you working on this issue?

so basically you want camera to detect ambulace and other vehicles so that if you integrate with something auto mated it gives abmbulace priority?


tests(help): add tests for help subcommand

#166

With this, only the suggestions and extensively testing subcommand is left to port the python tests to Rust tests.
I am unsure how to go about both of these. I think testing subcommand extensively is unnecessary (or really low priority) as it is just a wrapper of App.

I left the python tests alone because I think it could be removed once 2.0 releases.

Nice!
Yeah as part of the 2x release I'm gonna ensure the tests are good. I'll look at what can be done to remove the python tests as part of that process.
@homu r+

:pushpin: Commit 44cbed1 has been approved by kbknapp

:hourglass: Testing commit 44cbed1 with merge 56e2f75...

:sunny: Test successful - status


Manual: Google Custom Search should only return results for the same manual version

I am using arangodb manual at https://docs.arangodb.com/2.7/
When I used the Google Custom search from this address I received pages labelled for version 2.8.

With the current implementation it is not easy to realise the desired behaviour.

this will be handled as part of the later created #1721 during the 3.0 release cycle. Closing this one as duplicate therefore; thanks for the suggestion.


✨ Make the app data sharable

#20

@giacomocerquone I cannot add you as reviewer unless i assign you collaborator role. Tell me if it is ok for you

I've recorded a better video and deleted the old one, just make sure to check it out! @marilari88

Totally agree with you. We should let user to share also invalid schema.
Thank you for the video with the repro, I'll check the bug soon as I can

Really great work.
I've noticed that when I have an invalid zod schema, and I edit a value, if I click the share button, it seems to be working, but when navigating to the crafted URL, I find the old value there. The number of ValueEditors instead are correctly shared.
https://www.loom.com/share/59dfea5c4c234f11b6f426ea330867d1?sid=cf48f408-f10e-4ee3-965f-1b73998e720d
Up to you on how to handle this you either allow sharing invalid schema (which is what it should be, imagine a guy that doesn't know how to correctly work with zod and wants some help) or you explicitly warn the users that broken zod schema can't be shared.

I just pushed a fix for this issue. I know the code is ugly but it will be removed in the next PR about state refactoring

Great work. Was waiting for this PR to better shape the data so to implement persistence between sessions


Uncomment for TER

In my config.neon TER lines is already uncommented, why it doesn't work?

Have you reimported your experiments? Try to remove temp directory before reimporting the experiments.

Have you checked that tercpp.0.6.2 works on your platform when executed manually? It is executed as an external command: https://github.com/choko/MT-ComparEval/blob/5d48281396e534545574c82521826b4709b15409/libs/Metrics/Ter.php#L48

I am sorry, I meant removing only contents of the temp directory - run mkdir temp && chmod  777 temp.

I've tried and it still can't work.

What doesn't work? localhost or TER?
Set this to TRUE (https://github.com/choko/MT-ComparEval/blob/master/app/bootstrap.php#L13), you might get more information. If that doesn't help, just reinstall MT-ComparEval.

I mean TER, localhost works normally.

In that case download/compile a TER version which works on your machine and then edit the path to the binary: https://github.com/choko/MT-ComparEval/blob/5d48281396e534545574c82521826b4709b15409/libs/Metrics/Ter.php#L12

The download website is empty... can I find it in other place?

I see. Try https://github.com/cservan/tercpp or http://www.cs.umd.edu/~snover/tercom/ (the latter is Java-based and you would have to change also the command on line 42, probably to something like java -Dfile.encoding=UTF8 -jar tercom.7.25.jar -N -s).
Note that I have never used TER, so cannot provide more help.


Few base images updates

This PR contains the following changes:

Add Python v3.11.2 and v3.10.10
Add Fedora 37 (latest) and 38

LGTM

@balena-ci I self-certify!


Jenkinsfile update

This branch delivers an update to the ViaHTML Jenkinsfile. I have documented the problems we have been experiencing here: https://github.com/hypothesis/playbook/issues/433
In essence, we have a single repository that has the potential to be the source for many jobs in Jenkins. The Jenkins file as it is written passes a string viahtml which does not relate to the name of the Jenkins job we have defined, that is viahtml3. I suspect this is the reason we are running into trouble.
The update introduces a new variable, app which has been set to the name of the job in Jenkins "$env.JOB_NAME". The contents of app are then passed to deployApp as part of the onlyOnMaster and milestone stages
Update! New plan
In ViaHTML fix the Jenkins file to match
stage("qa deploy") {
    deployApp(image: img, app: "viahtml", env: "qa3")
}

->
stage("qa3 deploy") {
    deployApp(image: img, app: "viahtml", env: "qa3")
}

Same for the prod one too.

I am closing this PR as we have defined a solution outside of the Jenkinsfile update that has been defined. Review https://github.com/hypothesis/via3/issues/283 for more information.

I've changed this to match the plan outlined in our chat yesterday

I've changed this to match the plan outlined in our chat yesterday

Looks good to me!

I'm going to merge this so it doesn't get forgotten. But we expect a failure in CI after this


Particles affected by nearby light / are not emissive

When testing out the mod I noticed that the particles were quite dark in areas that weren't very well lit. This struck me as odd because typically when you picture an explosion they are usually very bright. It's probably a very quick fix and it feels like a bit of a nitpick but I thought I'd report it anyways.
Tested on a minimal instance using the latest 1.20.1 release.

Yeah, you're right, they aren't emissive. I don't think it would be hard to make the particles emissive, but you never know with this video game sometimes. I'll try to add this as a config option next update(probably enabled by default). Don't really know when that'll happen though, because I'm really busy with other stuff.
Thank you for pointing this out!

It looks a LOT better than I thought it would.
https://github.com/Superkat32/Explosive-Enhancement/assets/89557012/f9e3444b-57b3-421a-a246-33dad40cba74

Added in version 1.2.2! Thanks for the suggestion!


Fix/mobile first

Please contribute continually to this branch as much as you think is needed.  I've covered the overflow with a quick fix.  I've also attempted to have the splash and navbar buttons size according to all resizing modes from this this Chrome Tool.
Needing attention:

[ ] Scribe's inner toolbar (the 'search' + darkmode toggle, etc.)
[ ] Splash's Sitting SVG has way too much padding around it in some modes
[ ] Details could potentially be spaced smaller, wrapped and have the expand button visible (w/o scrolling).

https://5f324afb974c5a6e6dda2018--tpot-toolbox-next.netlify.app/

Chakra's docs for responsive design for are a little cryptic but good enough. However if you couple it with knowledge about how Material-UI does the same thing, you can get an understanding of what Chakra is wanting to do with breakpoint props.
Seeing that the branch is minimal, I would consider merging and closing it. I think a long term goal, which encompasses some of your list items above, it to make the app truly flexible in the UI design to fully support mobile. And I would suggest doing that in another major branch because it is a bigger job.
Since most of the UI is not attached to any business logic, then it should be easier now to take care with each component and craft them to look their best on all screen sizes. The first ones I will do are the layouts, like Dashboard. Doing so will help provide direction on what all the atoms, molecules and organisms need to do.
Also at this time, I would be swapping out material-table with react-table and making a responsive table UI from it. The current package isn't flexible, huge bundle size, etc. This would be the final component to kill to remove MUI entirely and stop hoisting 2 separate UI packages.

Just an FYI, I've been thru the chakra docs to get to the point of the last commit on this branch.

"Seeing that the branch is minimal, I would consider merging and closing it. I think a long term goal, which encompasses some of your list items above, it to make the app truly flexible in the UI design to fully support mobile. And I would suggest doing that in another major branch because it is a bigger job."

That's this branch's intended job, as specified.

I'll be here and there fixing each component and wrapped component to look good on Chrome and Firefox to the best of my ability as I have.

I'm opinionated, but I say why not scrap the material-table and replace it with a chakra grid for now, slap a custom search bar on it for filtering/ordering and call it a day?  Better yet, keep both impl just in case a user prefers Cards or Table.  But I know we have other items preceding this on the big list of things we want to do, so just a thought for now.  I can ably provide Cards, as I have a slotted variant already.


Github automation fails to move issues

Descriptions
Our two github actions, auto-triage.yml and remove-issue.yml currently doesn't work due to the usage of GITHUB_TOKEN. We previously used a personal token, which would expire and overall not recommended. We have since moved to using GITHUB_TOKEN.
It seems however that it doesn't work, but the issue could be that the GITHUB_TOKEN does not have the necessary permissions.
About GITHUB_TOKEN

The settings for the GITHUB_TOKEN are set to read/write, but it still does not work.  According to GitHub Actions that should provide read/write access to the issues, among other resources.
Perhaps we can try explicitly setting the read/write permission for issues on the jobs:
jobs:
  <my-job>:
    permissions:
      issues: write

Otherwise we may be forced to go back to using a personal access token (not ideal).

Adding explicit permissions did not work.  So we need to either go back to using a personal access token or find a different action that works with GITHUB_TOKEN.
Note for testing: After changes are made add a priority label (low. normal, or high) to an issue.  This should cause the Move labelled issue action to run.

Since GITHUB_TOKEN does not work, and we don't want to use our our Personal Access Tokens, the best approach would probably be to create a service account in GitHub, add it as a member of the organization and/or contributor of the repo, then create and use a PAT from that service account.


fix typo

About the retry feature in the lib/key.js, the comment says "retry after 500 ms" but the actual value is 50 (ms). This pull request fixes this.
For now I increased the interval to 500 ms because I feel that 50 ms is too short as an interval.
If the 50 ms was correct and the comment was wrong, please tell me. I'll fix this pull request.
https://github.com/zlargon/google-tts/blob/f7157430d1d8a084e6d43fc10bc643af096d2dee/lib/key.js#L5-L8
https://github.com/zlargon/google-tts/blob/f7157430d1d8a084e6d43fc10bc643af096d2dee/lib/key.js#L26

Coverage remained the same at 94.737% when pulling b611e78e587dfc07fd2ef427e7ed8eeafeeb279c on kmyk:fix/inteval into f7157430d1d8a084e6d43fc10bc643af096d2dee on zlargon:master.

Ah, that's a typo in the comments, so it should be 50 ms. Thanks.

Ok, I've updated this pull request.

Thanks, looks good to me.


[Feature Request] Add a datastructure for Tuple

It would be great to have a data structure for Tuple
Tuple <A>{
  _1 : A
  _2 : A
}

// functions
function fold()
function bimap()

Why do you think this?
I feel a tuple type innecesary in JavaScript because arrays can do the work.
Tuples are interesing in typed languages where you cant mix types in arrays without lost their types, but this does not happen in js.

True, I want to use tile with flow typing.
On Fri, Apr 27, 2018, 1:15 PM Daniel<EMAIL_ADDRESS>wrote:

Why do you think this?
I feel a tuple type innecesary in JavaScript because arrays can do the
work.
Tuples are interesing in typed languages where you cant mix types in
arrays without lost their types, but this does not happen in js.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/funfix/funfix/issues/127#issuecomment-385082268, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADanZTOpbtyXecqxy22wpfjbVkzCUIc4ks5ts3xlgaJpZM4TY1Iu
.

But in the way you suggested this, _1 and _2 have the same type, maybe you are thinking in some like Tuple<A, B> instead?
I am not using flow or ts right now, so I really don't see why this could be interesting :/

🤦 corrected it.
True, it may not be interesting right now, but it is helpful when you want to return 2 types and creating a class with 2 objects is an overkill.
Take a look at https://github.com/fantasyland/fantasy-tuples, it is perfect but just don't have typescript or flow types. Therefore, though it would be perfect for funfix : Functional Programming Library for JavaScript, TypeScript and Flow

Will the authors be willing to include tuples if I work on it and create a PR ?


boards: shields: Use new naming scheme for shield node labels

See https://github.com/zephyrproject-rtos/zephyr/issues/50040

I've missed some nodes (ones w/o preexisting nodelabel). Kept as DNM for now.


Python 3 support.

This is not ready to be merged.  This PR implements:

Preliminary Python3 support
Travis-CI container (faster) builds

@QuLogic can you take a look at this?  You are one of the best Python2/3 conversion experts I known :wink:
Note that we ca use six.iteritems instead simply changing iteritems() to items.  This is just a first draft to get things going.

The less new dependencies the better :)

Is there a reason you added print() but no __future__ statements?

Suggest we include in requirements.txt, not inline.

Looks like the Python 3 tests fail in the test runner: https://travis-ci.org/geopython/pycsw/jobs/80099911#L1132

Suggest we include in requirements.txt, not inline.

Not sure what you mean.  I kind of got lost in all the comments.

Looks like the Python 3 tests fail in the test runner: https://travis-ci.org/geopython/pycsw/jobs/80099911#L1132

I fixed that.  Now all the tests fail on Python 3 :stuck_out_tongue_winking_eye:
There are a few encode/decode to do. Also I guess some dicts and/or json dumps must be ordered.  I see several test failure due to out of xml.
We can continue on this PR (1) or merge this, since I manage to not break python 2, and open a new one to address the remaining issues (2).
Note that I would like to do (1), but I do not know when I will be able to get back to this and (2) might be appealing if someone else is willing to pick up from where I stopped.

We could make a Python_3 branch and move this work there, but it cannot be merged into master IMHO

cc @ricardogsilva
@ocefpaf thanks.  We are close.  Looks like issues are bound to the following:

varying order of XML root element namespaces declarations
as noted, varying order of dicts and json

AFAICT looks like most, if not all, errors can be solved by enforcing order in the codebase.  Making these changes will also require close inspection and update of the expected results which will naturally change.
I agree with @kalxas in making a py3k branch in pycsw proper.

AFAICT looks like most, if not all, errors can be solved by enforcing order in the codebase. Making these changes will also require close inspection and update of the expected results which will naturally change.

Yep.  That is why I am stepping back.  The day job is calling and this will take more time than I thought.  Sorry.

I agree with @kalxas in making a py3k branch in pycsw proper.

OK.  I think I can point this PR to a py3k branch upstream, but I am not sure if I can create that branch.

@ocefpaf branch created for PR: https://github.com/geopython/pycsw/tree/py3k

Done in #379.  Closing this one.


Fixed #28991 -- Added new BlankFieldListFilter admin filter.

https://code.djangoproject.com/ticket/28991

Since I'm always flagged as spam on Trac, I'll post this here:
Dear future reader: I don't have the time to work on this at the moment. Feel free to take it from here on. The PR needs a few small changes.


Question: mt_events Table Design Motivations

I was looking at the SQL script here and tried to understand the motivations behind the design of the mt_events table:
CREATE SEQUENCE {databaseSchema}.mt_events_sequence;

[...]

CREATE TABLE {databaseSchema}.mt_events (
	seq_id		bigint CONSTRAINT pk_mt_events PRIMARY KEY,
	id		uuid NOT NULL,
	stream_id	uuid REFERENCES {databaseSchema}.mt_streams ON DELETE CASCADE,
	version		integer NOT NULL,
	data		jsonb NOT NULL,
	type 		varchar(100) NOT NULL,
	timestamp	timestamptz default (now()) NOT NULL,
	CONSTRAINT pk_mt_events_stream_and_version UNIQUE(stream_id, version),
	CONSTRAINT pk_mt_events_id_unique UNIQUE(id)
);

Note: I've also been through that piece of documentation.
What is the purpose of the seq_id as a primary key?
I don't really understand the purpose of the  seq_id as a primary key, since there is already an id column for which the values are generated on the client-side via CombGuidIdGeneration. From my understanding, those values generated by that algorithm are both sequential and the probability of collisions are very low (plus from what I'm seeing there is already a unique constraint on the id column to avoid any possible redundant values anyway).
So why the id column set as a primary key wouldn't be enough?
Additionally, I thought about potential concurrency issues, but I don't really see how does a sequence of bigint would do a better job here. A sequence might fall out of sync and have gaps anyway due to an expected failed transaction. I'm sure that I'm missing something out and I would like to better understand what was the motivation behind that decision.
Why seq_id is not defined as bigserial column type?
Also, I was wondering why the seq_id is not written as a bigserial column which behind the scenes does pretty much the same job (bigint sequence).
8.1. Numeric Types - 8.1.4. Serial Types
9.16. Sequence Manipulation Functions
Again, I'm probably missing something out.
Why the type column does not have an index?
I know that an index might slow down a little bit the insertion speed but it might come in handy to filter out some events in a given stream based on their types, especially if that stream is becoming pretty big.
Is it about trade-offs and usage?

I'm sorry if my questions sound really stupid but I couldn't really figure out why that specific design has been chosen.

@kerry-perret You might want to phrase this more as suggestions than questions.
"Why the type column does not have an index?" -- I was much more worried about the insert performance at the time that was originally built, but that's probably worth thinking about in v4 because the async daemon (assuming that stays in) queries by type.
"Why seq_id is not defined as bigserial column type?" -- I can't say that I even knew about BigSerial
"What is the purpose of the seq_id as a primary key?" -- This one's a little more involved, and it's been several years since we went through all that. It's actually possible in Marten to configure whether or not you want the events to use Guids or numbers as the primary key. Don't remember why folks wanted that, but it was a big deal for Marten v2 for somebody. The async daemon depends on the seq_id as a table wide event id to track progress and that's critical, so that has to stay. I honestly don't remember why there's also a Guid key option, but that might have something to do with the C# code and needing a way to correlate objects in memory before the sequential id is assigned and absolutely nothing to do with SQL whatsoever.

@kerry-perret Okay, nevermind about the bigserial. Wouldn't really have made anything simpler because it was important to return all the new id's to the client in the function that appends events. Auto-incrementing didn't really help IMO, and with the "tombstone" idea we'll try to play in v4, even less so.
You probably shouldn't be trying to understand things by looking at just a database table without any other context here.

First off, thanks a lot! That is super duper informative!

@kerry-perret You might want to phrase this more as suggestions than questions.

Blurred lines 😄  (couldn't come off with a proper pun, thought about "quegestions")
In all seriousness, that was purely out of curiosity. I wouldn't dare to suggest anything serious until I manage to grasp every tiny little bit of detail (that may take a while (a long one), though).

Why the type column does not have an index?

I was much more worried about the insert performance at the time that was originally built, but that's probably worth thinking about in v4 because the async daemon (assuming that stays in) queries by type.

If you don't mind, I am also taking the liberty to add below what you told me on Gitter, if it could be of any help for someone passing by this "questionissue / issustion":

Oh, but about indexing type. It’s mostly inserts and few reads of that table. When there are reads using the type column in prodution usage, there would always be a WHERE clause against the seq_id column as well that first constrains the results against a 100-200 record range, so I don’t see an index on type as being desirable.

Your answer is sound, I thought about that too, that's exactly why I was wondering if it was due to usage.

What is the purpose of the seq_id as a primary key?

This one's a little more involved, and it's been several years since we went through all that. It's actually possible in Marten to configure whether or not you want the events to use Guids or numbers as the primary key. Don't remember why folks wanted that, but it was a big deal for Marten v2 for somebody. The async daemon depends on the seq_id as a table wide event id to track progress and that's critical, so that has to stay. I honestly don't remember why there's also a Guid key option, but that might have something to do with the C# code and needing a way to correlate objects in memory before the sequential id is assigned and absolutely nothing to do with SQL whatsoever.

I see, things are pretty clear now.

Why seq_id is not defined as bigserial column type?

I can't say that I even knew about BigSerial

Okay, nevermind about the bigserial. Wouldn't really have made anything simpler because it was important to return all the new id's to the client in the function that appends events. Auto-incrementing didn't really help IMO, and with the "tombstone" idea we'll try to play in v4, even less so.

It does makes sense to me now.
Thanks again!


fix: don't use .fromBinary on import

This PR fixes #680, by transforming the b64 feature set to a JSON string describing the message, at buildtime (bootstrap step / npm run -w packages/protobuf bootstrap:featureset-defaults).
This allows to import @bufbuild/protobuf in a context that doesn't provide a TextEncoder globally.
Please let me know if you have any feedback :)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

So I realized this was an issue when I saw https://github.com/bufbuild/protobuf-es/issues/663 last week and wrote a fix, but didn't get to pushing it up until today. You beat me to it - but https://github.com/bufbuild/protobuf-es/pull/683 avoids the dependency cycle. Could you confirm that it works for you?

Superseded by https://github.com/bufbuild/protobuf-es/pull/683, but thanks for the PR ❤


dynamic variables

hey, nice job.
i want to make variables dynamic using css variables, i mean change all of less variables to css variable when we are using less.render.i think we can achieve that with some plugins like https://github.com/miyamarisubs/less-plugin-custom-properties

Hi @amirjr1378, thanks!
Could you explain us a little bit more about this. What are you trying to achive? or maybe the problem you are trying to resolve Any information would be valuable because I don't quite understand the request.

hi again @randomeduc.
for example my primary color is #ddd.
generated css file, would replace all of @primary variable with #ddd , but i want to replace it with var(--primary).
i think it can be achieved with some kind of less plugin that convert less variable to css custom properties instead of value.
if there is no plugin we can achieve this with some kind of regex replacement that replace all of #ddd with var(--primary).
if this feature can be implement with this library we can easily change antd theme in runtime without adding less that is huge library to project in browser.

This might be relevant: https://ant.design/docs/react/customize-theme-variable

IMHO, using a regex to replace them would not be an optimal solution. I think lots of colors get generated using less functions (darken, fade, etc) taking the aforementioned variables as base, doing it so would only work in the ones that match exactly with the color chosen. In the other hand, if two vars end being of the same color it would be impossible to differentiate which one you are trying to replace since the method works in a per-color basis.
I don't know if the plugin handles those scenarios. A quick test just didn't let me compile the .less file using something like lighten(@var, 10%) since the @var argument could not be evaluated to a color. I assume it works well for simple replace cases, though. A workaround would be to use the defined default color instead of the var name to calculate any function using it but in the end you'll be able to only change those base colors and not any derivatives.
While still experimental, seems that antd is working in bringing this feature first-party as jugglingcats pointed us in the comment above. Maybe it'd be preferable to stay tuned to this and try to build something from there.


Update the correct version for the release notes

Signed-off-by: Zelin Hao<EMAIL_ADDRESS>Is your feature request related to a problem? Please provide an existing Issue # , or describe.
Fix the heading for the release notes
Describe the solution you are proposing
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #326 (b300346) into 2.4 (9043118) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                2.4     #326   +/-   ##
=========================================
  Coverage     70.82%   70.82%
  Complexity      372      372
=========================================
  Files            44       44
  Lines          2581     2581
  Branches        174      174
=========================================
  Hits           1828     1828
  Misses          641      641
  Partials        112      112

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Thanks @zelinh! Do you know if backport main tag works?

@dzane17 @kaushalmahi12 This would need backport so please help take care of that later. Thanks.

Thanks @zelinh! Do you know if backport main tag works?

I'm not pretty sure; might need manual efforts on it if not. Thanks!


Multi-line properties don't render properly

At least in my case, with Outlook 2013, properties that contain multiple lines do not render on multiple lines in the email  (this is also true about the Rendered message in bold) :

The initial message in Seq looks like this :

FWIW, it seems that
white-space:pre-wrap;

like in https://github.com/datalust/seq-apps/blob/1cb6a5f9ec682afaf6a4b34815a99a470f0beeb5/src/Seq.App.EmailPlus/Resources/DefaultBodyTemplate.html#L104 is not supported in Outlook
see https://www.campaignmonitor.com/css/text-fonts/white-space/
Just using
white-space:pre;

seems to work better

If I take the contents from https://github.com/datalust/seq-apps/blob/1cb6a5f9ec682afaf6a4b34815a99a470f0beeb5/src/Seq.App.EmailPlus/Resources/DefaultBodyTemplate.html and replace pre-wrap; with  pre; all over the place, it looks nicer, and gives the following:

which is better, and looks closer to the Seq layout :)

Thanks for the report, Thibaud.
I wonder if including both, e.g. white-space: pre; white-space: pre-wrap; might suit Outlook better, but still trigger pre-wrap where it's supported?
I recall pre looking rough when the lines are too wide for the viewport - scrollbars can be ugly in browser clients, but hiding them makes it difficult to read long lines that overflow.

I gave it a try, replacing occurrences of
white-space:pre-wrap;

with
white-space:pre;
white-space:pre-wrap;

seems to do the job for Outlook 2013, multi-line properties are honored (but no change to the Rendered Message, which is a similar behavior to Seq) :

I'll update my PR

Thanks for investigating! Build should be coming through CI shortly 👍


call test done before stopping local streams

fixes #149.
If I remember correctly, previously the code removed all events associated with the video track before calling test done in order to prevent a race condition where on fast machines this.test.done() was executed before the forEach loop finishes and the ended event could not be caught before the object got destroyed, however on slower machines the forEach loop finishes and stops the track thus firing the ended event before this.test.done() is executed and voila, the event has time to fire off an reportError call this failing the test.
Rather than setting all event listeners to null, I just reversed the order to stop the test before stopping the tracks, WDYT? Is this okay or should I explicitly set all events to null in the forEach loop?

Disabling firefox beta in #153

I don't think this fixes the issue. And if it does it could be a bug in chrome (but I don't know enough).
Can a call to track.stop() trigger the event callback to run before track.stop returns? I am not sure, but I would guess no. If it does is that a bug or is it the intended behaviour?

I agree that it sounds odd but the outer forEach loop seems to finish before stop() hence the events seems to have time fire. At least that's what it I've seen, by either deregister the events in the forEach loop (more explicit) or calling done() before stop seem to prevent the events from firing.

After talking offline to Andre he made it clear that my hypothesis about a race in the forEach loop isn't possible, rather it was closing the peerconnection before stopping the tracks.

lgtm

New logic for how to download firefox has been fixed in travis-multirunner, will update.


:house_with_garden: Home Assistant Refresh

Description of changes
Resolve a bunch of maintenance tasks:

[x] Replace top-level with lib
[x] Allow passing extraArgs to the command line
[x] Make various issues around the NixOS test

[x] restart/reload expectation around exit code 100
[x] replace backup component in dependency change test, since it is a default integration by now
[ ]

[ ] Update various lingo bits

[x] Remove the custom option prefixes (e.g. from customLovelaceModules) and make options names more concrete
[ ] Talk about integrations instead of components. Apparently components is a legacy naming.
[x] Use PEP517 attribute names in custom components

[x] Use runTest instead of handleTest
[x] Autocalling for component and lovelace module package sets

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I was going to open a new PR for this: https://github.com/kira-bruneau/nixpkgs/commit/c0ed7c111355ee1d0f5eb2ed01ecc4807c4c1d20, but I noticed that this was PR was just opened.
Would it ok to include those changes in this PR?

Oh wait nvm, I thought this touched the package, I'll just open a new PR


US total-budget usg:type

US budgets have a usg:type that we probably need to handle.
Via @JamesCoe.

Adding this to the current milestone – we can treat it like budget status.

Fixed in a8dd4fd.


[PE-D][Tester A] Having an incorrect address format does not indicate what the mistake made by the user is

When entering the address format incorrectly, there is no feedback to tell what mistake the user has made.
Neither are the constraints of each parameter for address stated clearly in the user guide.

Labels: severity.Low type.FeatureFlaw
original: awdse22/ped#22

Falls under UG/DG. As parameters constraints are not defined in UG/DG


Output Vcf not in Order

Hi， I found 2 points that could be improved:

The following Warning were reported by Pysam.VariantFile
[W::vcf_parse_info] INFO 'SVSCORE' is not defined in the header, assuming Type=String
[W::vcf_parse_info] INFO 'SCOREMETHOD' is not defined in the header, assuming Type=String

The output Vcf is not in Order, and might induce the following Error:
[E::hts_idx_push] Chromosome blocks not continuous
tbx_index_build failed: Tissue-Case5.vep.vcf.gz

Best regrards!

We're actively working on the release of version 1.4.3 and your two suggestions will be incorporated in the upcoming release.
Thanks a lot for your valuable input.
Best regards.


Docker EE and CE compatibility

Make sure our runtime is compatible with the Docker EE and CE releases

Last bug is console socket @sboeuf  is working on this now and it is the last piece that needs to be complete for a beta release


How to adapt this framework to a new dataset?

Hello, thanks for your repository, it inspires me a lot. For the QA dataset, I wonder how to adapt this framework to the GrailQA dataset? What changes do I need to make to this framework？

Hi, we haven't tried to adapt this framework to the GrailQA dataset, so we can only provide some suggestions based on our understanding. We'd like to follow the sequence in README.md.

(2) Parse SPARQL queries to S-expressions.
This part of code is modified based on RNG-KBQA. Since GrailQA is also based on Freebase, this part of code should work. But it will be better if you run a coverage test on SPARQLs from GrailQA, to check if there are SPARQLs our code fails to handle.
(3) Retrieve Candidate Entities and (4) Retrieve Candidate Relations.
These two steps aims to provide candidate entities/relations for our multi-task model. Both migrating our method for GrailQA and using results of other linking tools should work.
(5) Prepare data for multi-task model.
We guess there should be some file format issue for a new dataset.
(6) Generate Logical Forms through multi-task learning.
We think the multi-task model itself should work without modification. But similar as (2), when converting S-expression back to SPARQL in eval_topk_prediction_final.py, you may need to run a coverage test for unhandled cases from GrailQA.

These are possible changes we can think of at current stage. Hope this helps.

Thanks for your guidance!


handler: allow using the user customization everywhere

I believe there's no reason not to expose the customization for all image types. We actually strive to achieve a functional parity between the hosted service and on-premise IB, so this feels like a low-hanging fruit.
https://issues.redhat.com/browse/HMS-2211

/retest

/retest


Add GitHub Actions: Add labels to opened issue

What

This pull request add GitHub Actions workflow to this repository that automatically add labels to newly opened GitHub Issues
Clarify Issues where Triage and Assign have not been performed and encourage the Benevolent Dictator to do it

Why

I believe that every issues should be judged on the following by Benevolent Dictator

Determine priority of the issue ... Triage
Determine who is in responsible of the issue ... Assign

How

See the file .github/workflows/issues-opened-triage.yml

It runs when a new issue is opened
It gives two labels to the issue

waiting/triage
waiting/assign

Benevolent Dictator will be...

Unlabeled waiting/triage once triage is performed
Unlabeled waiting/assign once assign is performed

Message to @hfu

I have added the following labels to this repository

waiting/triage
waiting/assign
priority/MAY
priority/SHOULD
priority/MUST
Request for Comments

Looks good to me. Let's do this.


Create a clear method for cytoscape graph

implements https://github.com/QuantStack/ipycytoscape/issues/61

I think the issue with this is that it doesn't clear any existing displays. For example if I display after the clear the graph is empty but the original display doesn't change:

Getting that existing one to update is the tricky part that will require doing something on the typescript side

that's interesting, it works for me locally

okay so it doesn't work in jupyter lab interface, but it works in the classic notebook interface.
Any idea what's happening here?

okay so it doesn't work in jupyter lab interface, but it works in the classic notebook interface.

oh man....

Any idea what's happening here?

not at all.
If anything I'm surprised that it works in the notebook. I don't really see the path by which those nodes end up being removed from the cytoscapejs object.

I wonder if there is almost a race condition between the frontend and backend in terms of what the nodes are. And the differences in how notebook apps adn jupyterlab exts handles the comms leads to a different winner?

I wonder if there is almost a race condition between the frontend and backend in terms of what the nodes are. And the differences in how notebook apps adn jupyterlab exts handles the comms leads to a different winner?

no clue 🤷♂
@marimeireles any ideas ?

Hey @MridulS I'm following the issue. Didn't post anything because I'm also lost. Will try to investigate this later tonight.
Maybe @martinRenou has an idea on where to start?

 Sorry for that.
I think this fixes the problem. I couldn't reproduce it only on the notebook, it was failing both on lab and in the notebook to me.
It'd be good to add this to the tests if you have the time.
Following the pattern that Ian started I think we should create a new file with "test_graph_removal" or smth like that.
Thanks @MridulS :)

I just realized I introduced a regression with the versions of jupyter cytoscape and ipycytoscape everybody is fetching from the master :(
I'll fix it now in a new PR, so we can rebase or force push to this.

Should we also add a function to selectively delete nodes/edges? Though maybe that's a can of worms as we sometimes create edges when populating _adj and then we'd have to handle those edges as well

Also, this works for me on jlab now :smile: . So I am generally :+1: on this, but would prefer if an example was added to the documentation prior to merging.
Also for versioning: This along with #109 introduce new features/performance improvements so I think we ought to jump to at least version 1.1.0? Thoughts @marimeireles ?

Also for versioning: This along with #109 introduce new features/performance improvements so I think we ought to jump to at least version 1.1.0? Thoughts @marimeireles ?

I'm always confused towards this... tbh
I think it accumulated enough changes to do it, maybe we should even have done it before. I think these last digit changes are only for minor fixes, and we're adding features for a while now.

I'm going to rebase this and create some tests.

@MridulS do you want to continue work on this? If not do you mind if someone else rebases?

@ianhi feel free to rebase this and continue :)

Can't believe it's been 16 days since I said that, sorry peeps. :(
I'm trying to push to finish this other thing so I have a little bit of time here.

Thanks all! :cherry_blossom: