Reduce allocations in String::sprintf

This prevents an allocation of a new string for every += char32_t. The example string was rendered as a label every frame in GDScript:
text = "%d FPS (%.2f mspf)" % [Engine.get_frames_per_second(), 1000.0 / Engine.get_frames_per_second()]

To validate the improvement, allocations were captured for a 5-second window. Note the number of allocations in the sprintf function of the following images.
Before: 43,830 allocations

After operator += (char32_t): 17,474 allocations

[!NOTE]
60% reduction in allocations.

After static const String: 12,934

[!NOTE]
A further improvement was to use some static const String for common padding and sign characters, which reduced the allocations a further 26%

[!NOTE]
Total reduction in allocations for the 5-second window was 70%

Related: #93697

I feel like intead of static const String, methods like insert_char should be introduced, similar to #92475

I feel like intead of static const String, methods like insert_char should be introduced, similar to #92475

Totally agree. Adding appending APIs would be helpful. Also, being able to keep the memory around for a String that could be used as a temporary would be beneficial too, as there are a number of temporaries created in sprintf

Also, being able to keep the memory around for a String that could be used as a temporary would be beneficial too, as there are a number of temporaries created in sprintf

There's quite a lot of examples of thread_local LocalVector used for similar purposes, so I wonder about introducing a LocalString class, which would use LocalVector inside of it. Not sure of it would be worth it if there's not many uses for it though

Just passing by. A few thoughts:

T const & is inconsistent with the rest of codebase. const T & is preferred.
There is string appening functionality, StringBuilder. I'm not sure how to use it for the late sign insertion, but there's likely a clever way to solve that.
String str = "(" deserves the optimization, too.

There's quite a lot of examples of thread_local LocalVector used for similar purposes, so I wonder about introducing a LocalString class, which would use LocalVector inside of it. Not sure if it would be worth it if there's not many uses for it though

The thread_local thing is being done with LocalVector because it features a capacity feature, which Vector doesn't. In addition, LocalVector doesn't have a thread-safe refcount, which makes it more performant and the right option when you know you don't need such a thing. My point is that thread_local Vector and thread_local String would be possible if Vector featured the aforementioned capacity-reserve feature. Adding it would avoid having to emulate it in certain parts of the engine where the Vector is sized to what it would be the capacity and the size is managed externally.
In other words, at some point my view is that it would be ideal that Vector and LocalVector were just two specializations of the same template, with better names. So, Vector would be LocalVector plus copy-on-write
I opened a proposal time ago about this: https://github.com/godotengine/godot-proposals/issues/5144.
We could even have a copy-on-write specialization which is not thread-safe, which may be a better fit for strings. I'd say that all of this would need to be discussed carefully because there are things yelling to be improved, such as these vector types getting more telling names and finding out the most precise set of features each needs across the codebase.

Discussion of future improvements aside, I think we can go ahead and merge this once it is updated to use const T & formatting as requested by RandomShaper.

T const & is inconsistent with the rest of codebase. const T & is preferred.

I'll fix that.
As an aside, I wrote a blog about const 13 years ago, indicating a more intuitive placement, with a simple rule for reading it:
http://blog.stuartcarnie.com/post/constant-confusion/

@RandomShaper / @clayjohn changed to const T &

As an aside, I wrote a blog about const 13 years ago, indicating a more intuitive placement, with a simple rule for reading it:

Lol, if I had to debug an issue arising from const int const * const * i = &j; // ok I would cry

Thanks!


Cannot P/Invoke a function named "Process()"

Steps to reproduce
Define the following function in C and compile as a DLL:
__declspec(dllexport) int Process()
{
    return 42;
}

Run the following in PowerShell with the path to the DLL in the DllImport:
$type=Add-Type -MemberDefinition @'
[DllImport("C:\\PathToRepro.dll")]
public static extern int Process();
'@ -Name TestProcess -PassThru

$type::Process()

Expected behavior
Function returns 42.
Actual behavior
The following error occurs:
The result type 'System.Int32' of the dynamic binding produced by the object with type 'System.Management.Automation.PSObject' for the binder 'PSInvokeMember: static Process ver:0 args:0 constraints:<>' is not compatible with the result type 'System.Object' expected by the call site.
Changing the name of the function to something else such as Process2() works as expected.
I was not able to reproduce this when using a C# program instead of PowerShell.
Environment data
Name                           Value
----                           -----
PSVersion                      7.1.0
PSEdition                      Core
GitCommitId                    7.1.0
OS                             Microsoft Windows 10.0.20231
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

I guess PowerShell is not designed to call directly native dll-s. I suggest to wrap in C# code.

From the PowerShell documentation (link), it sounds like the -MemberDefinition parameter should support this. As I mentioned, this error only happens if the function is named Process. It appears to work correctly for other function names.

PowerShell Extended Type System defines extensions for System.Diagnostics.Process type and I guess PowerShell binding catches this. So you could try either wrap in C# code or put the code in custom namespace.

How do I wrap in C# code? I tried the following, but it gives the same error:
$typeDefinition=@"
using System;
using System.Runtime.InteropServices;

namespace Sample{
    public static class SampleClass
    {
        [DllImport("C:\\PathToRepro.dll")]
        public static extern int Process();
    }
}
"@

Add-Type -TypeDefinition $typeDefinition

[Sample.SampleClass]::Process()

Setting the -Namespace parameter to Add-Type does not appear to make a difference either.

Looks like it's any static method named either Process, or End (incidentally, not Begin because it has a void return) without arguments that returns a value type.
Easier repro:
Add-Type -TypeDefinition 'public class Test { public static int Process() => 10; }'
[Test]::Process()

I would guess that it's caused by the special casing here:
https://github.com/PowerShell/PowerShell/blob/db94377b3a69511280d329e64375475f6d7513b8/src/System.Management.Automation/engine/runtime/Binding/Binders.cs#L6915-L6927
It's not immediately obvious how that's happening, but it sure seems like it is.
/cc @daxian-dbw you might find this interesting.  Looks like the binder is somehow returning true here and exiting early, skipping the cast to object right afterwards.

Since no workaround works I set Issue-Bug.

@SeeminglyScience From your example Begin should work the same?

Since no workaround works I set Issue-Bug.

Reflection or an intermediate C# method with a different name can serve as a workaround.  Definitely a bug though.

@SeeminglyScience From your example Begin should work the same?

So because the method SteppablePipeline.Begin has a void return, the expression generated will be something like { target.Begin(); return null }. That skips the whole "this call site should return object but doesn't box the int" business.

Oh also probably a better repro:
class Test {
    static [int] Process() {
        return 10
    }
}

[Test]::Process()

Still able to reproduce in PowerShell 7.3.9.


RLOS output is pink

I installed the latest build of the Pro Add-In today, and ran it against the test "ElevationUTM_Zone10" data. One single observer, but the output is PINK:

... and now it is gray/blue and not square...

I have confirmed that I get the same thing in today's and yesterdays build. Going back to check previous builds

Build 457 shows the correct symbology

The build number is 4857. I just tested this on 4861 and I am seeing the same issue.

This was an issue in ArcGIS Pro 1.3 and was fixed then. This is no longer an issue.


Added DataPickerTextField

Great! Needs some thorough documentation on par w/ other field types in this codebase. Also Data makes it sound tied to NSData, and Text might be misleading since it's not a text entry. How about PickerField?

Excellent, thanks @vikmeup!


refactor/udp Remove udp/utp

This release does not require udp connections

This change is

r? @canndrew
(maidsafe_highfive has picked a reviewer for you, use r? to override)

r? @Viv-Rajkumar

Reviewed 12 of 18 files at r1, 7 of 7 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Can't query Users projects - INTERNAL ERROR

I'm trying to query information on a user's projects on -
https://mainnet.serve.giveth.io/graphql
and it's throwing an INTERNAL SERVER ERROR

the user does exist and we can isolate the error specific to this if we remove projects from the query.

@divine-comedian pls test this in this url https://serve.giveth.io/graphql
with this query
query{
  user(userId:1){
    firstName
    lastName
    projects{
      title
    }
  }
}

and if it was ok please tell @MoeNick  to move this to Done

@divine-comedian pls test this in this url https://serve.giveth.io/graphql
with this query
query{
  user(userId:1){
    firstName
    lastName
    projects{
      title
    }
  }
}

and if it was ok please tell @MoeNick to move this to Done

The problem is particularly on the production database, I'm checking what's wrong with those users and then improve that query

@divine-comedian pls test this in this url https://serve.giveth.io/graphql
with this query
query{
  user(userId:1){
    firstName
    lastName
    projects{
      title
    }
  }
}

and if it was ok please tell @MoeNick to move this to Done

The problem is particularly on the production database, I'm checking what's wrong with those users and then improve that query

@mateodaza  No it was in staging as well, after deploying PR it resolved, please see https://github.com/Giveth/impact-graph/pull/135#issue-737120234

user projects seem to be returning data now!

no errors, hopefully pushing this to productions will solve the problem

@mohammadranjbarz  can you ping me when this has been merged to master so I can carry out the task I need to do with the  prod Database?

@mohammadranjbarz  can you ping me when this has been merged to master so I can carry out the task I need to do with the  prod Database?

Sure mitch,
I dont know whats our release plans for now, but I'll let you know

@mohammadranjbarz can you ping me when this has been merged to master so I can carry out the task I need to do with the prod Database?

@divine-comedian
It's in production now

thank you man! thanks for pushing this out so quickly


Support hairpin mode

Problem
Can't access service from it's own pod in a k3s cluster. Even passed kebelet args like --hairpin-mode="promiscuous-bridge".
feature
Would you consider keep hairpin support ? It's necessary in many scenarios and  doesn't require a lot of compute resource.
work around
Watching /sys/devices/virtual/ for new virtual device  then echoing 1 to hairpin_mode   as a work around.

thumb up

Instead of watching and run echo 1 > xxx  for every veth, just run ip link set cni0 promisc on once to enable promiscuous mode on every node.

Hairpin mode is now enabled for flannel in v0.10.0-alpha1

Hairpin mode is enabled by default in v0.10.0.  grep -r hairpinMode /var/lib/rancher/k3s/agent/*


Documentation

Add arch diagram and project documentation

Added wiki for project and link to the wiki from the readme.


Throws errors when trying to type the name of an #included header

Here are the steps:

Create a new file (touch derp.c)
Edit the file (nvim derp.c)
Type the following: #include <
Once I start typing the name of the header (e.g. stdio.h) deoplete throws the following errors:

[deoplete] Traceback (most recent call last):
[deoplete]   File "/Users/hiphish/.local/share/nvim/site-packages/deoplete.nvim/rplugin/python3/deoplete/deoplete.py", line 116, in gather_results
[deoplete]     ctx['candidates'] = convert2candidates(ctx['candidates'])
[deoplete]   File "/Users/hiphish/.local/share/nvim/site-packages/deoplete.nvim/rplugin/python3/deoplete/util.py", line 51, in convert2candidates
[deoplete]     return [{'word': x} for x in l] if l and isinstance(l[0], str) else l
[deoplete] TypeError: 'generator' object is not subscriptable
[deoplete] Could not get completions from: clang2.  Use :messages for error details.

This happens every time. Any idea?

Any idea?

Yup.  Apparently deoplete doesn't like generators now.


Wownar react native example, Sign in with Neynar does not work on Expo Go

Testing out Wownar, found an issue with the build; Sign In with Neynar does not work for ios,ios sim
Steps to recreate the issue

Clone the repo
Go to Wownar-react-native example
Copy env values and install all the dependencies
Start ios and run using Expo Go
Clicking on SignIn with Neynar does not work.

Also found that it works great, on android emulator, but not with ios sim, or ios

@gendev1 thanks for creating the issue. We'll look into this early next week.

@gendev1 What do you see after clicking on SignIn with Neynar?

The button is not clickable, I use iPhone 13, it does not work on iphone sim too ( tried on windows, mac dev env)

@gendev1 I tried it in the iPhone simulator the button is clickable.

I feel it is more my ios device issue, I opened up another app and was not able to click on anything in that app too
So closing this issue


Introduce image loading performance test.

This example reports how long it takes to load 5 images over the network(locally spawned http server).

cc @mkustermann

Had to revert this commit as the buildbot turned red on this.

I think there is bigger buildbot infra outage that is unrelated to this PR.

Pub is down


Swaylock/waylock lose focus after resume

Hyprland Version
1b48642fd15c433c53876f1b933dcd46265caf8f
Bug or Regression?
Bug
Description
I have this issue where sometimes swaylock/waylock (it happens for either of them) don't get focus when resuming from sleep. This is not always the case so I haven't been able to pin-point the issue. When it happens, sometimes the mouse cursor is visible but then goes away after pressing any key. Trying to input the password does not work at all and the screen remains black (but still lit). Then I have to kill swaylock from another tty to be able to access the system.
I have swaydle setup so that is turns off the screen before it locks it and the screen is locked using a the lockscreen script below.
I hope this is not a duplicate issue since I have not been able to find anybody else having the same problem.
hyprland config:
exec-once = swayidle -w timeout 400 'hyprctl dispatch dpms off' resume 'hyprctl dispatch dpms on' timeout 700 'lockscreen' timeout 1000 'systemctl suspend' lock 'lockscreen' before-sleep 'lockscreen'

lockscreen script:
#!/bin/sh
playerctl -a pause

dunst_old=$(dunstctl is-paused)
dunstctl set-paused true # pause dunst

# mute
pactl set-sink-mute 0 true

pidof swaylock > /dev/null || swaylock -d >> /tmp/lockscreen.log 2>&1

# wait for swaylock to finish
wait $!

dunstctl set-paused "$dunst_old" # restore dunst state

swaylock config:
color=000000
show-keyboard-layout
daemonize
ignore-empty-password

How to reproduce

Setup swayidle and swaylock as above.
Make the system sleep (either by the timers expiring or by closing the laptop's lid)
When the system wakes up, try to enter a password and fail

This is very inconsistent and I haven't been able to find out what exactly triggers it. But I have been having the problem quite frequently.
Any suggestions on how to pin-point the cause are greatly appreciated! I'm guessing it's probably missconfiguration.
As a final remark: I haven't had the same problem with gtklock, which does not support wayland's screen lock protocol, so maybe that has something to do with it. However, I haven't used gtklock for very long so maybe it would show up eventually.
Crash reports, logs, images, videos
No response

This is the log of the last time this happened (the last time the session lock I was unable to unlock it, the sessions above worked fine):
lock_error_hypr.log

by doesnt get focus you mean keyboard focus? So you can still type behind it?

Yes keyboard focus, I can't type in the password.

just because you can't type in the password doesn't mean focus is lost, it could be e.g. swaylock being hung?
Launch a terminal in the background, get it to do this, and try to type in and send pkill Hyprland

You are right, swaylock is still in focus because I tried what you said and it does not work. However, I can still not type my password.

I think I face the same issue. I can confirm that swaylock has lost its focus and hyprland receives the keyboard inputs instead.

This only seem to happen when swayidle triggers the swaylock command in the before-sleep hook. For me there is no difference in how I initiate suspend/hibernate (e.g. wlogout, cli, lid, ...). If swaylock is active before suspend is triggered, the problem disappears:
swaylock && systemctl suspend
Workaround to recover unlockable session:

Have allow_session_lock_restore set to true
Log into a TTY
Kill / restart swaylock

pkill -SIGUSR1 swaylock ; WAYLAND_DISPLAY=wayland-1 swaylock

I also tried to use waylock instead and stumbled upon this. Might be an issue with Hyprland or swayidle?

This is most likely an issue with hyprland, i could reproduce similar behavior when adding a monitor when locked (eg. plugging in a monitor or it coming back from sleep). For me it also happened when starting swaylock from a terminal. I was able to reproduce the same issue with waylock. I could not reproduce the issue disappearing when swaylock was active before suspending (could also be me, I switched from sway to Hyprland yesterday)
As a workaround I put this line into the config (inspired by @mindrunner )
bindl = $mainMod, BackSpace , exec, pkill -SIGUSR1 swaylock && WAYLAND_DISPLAY=wayland-1 swaylock

can you check on git I made a commit rn

Thanks for the fix, works except when closing a docked laptop, unplugging it and then opening it (I have a monitor connected to my docking station), but i can use the workaround I posted before for that situation.

Hi, I wanted to say I was also affected by this problem, but I think, I succesfully managed to isolate when this occurs for me:

I use multiple monitors
I use swayidle
in after-resume section of swayidle's config, I have hyprctl reload.

Now that I think about, I think it's possible that swayidle is not the culprit, but hyprctl reload is (and I have no way of triggering it right after sleep othen than swayidle at the moment, I'm too lazy to setup a systemd service.
When either of these two conditions doesn't happen, then swaylock is happily taking my keyboard. (i.e. only laptop without monitors plugged in, or not using hyprctl reload right after sleep) I was initially using hyprctl reload in order to trigger things specified in exec in my config, but I can directly put it into swayidle's after-resume section.
I did not test changing the monitor setup while the laptop was sleeping.

I just now tested to put laptop into sleep by systemctl suspend (with swayidle running, therefore triggering before-sleep section, which locks the laptop), unplugging the monitors and waking up and everything works fine without hyprctl reload in after-resume. The other direction (sleeping without monitors, plugging in, waking up) worked as well.
Btw, I'm not on git version, but on 0.34.0-1 from Arch repos.

Happened to me without any external screens and not additional setups reloading hyprland.

The lockscreen stopping to receive input happens to me randomly on NixOS when using swaylock-effects in Hyprland. Not just after sleep/resume, also occasionally after locking normally through swayidle.
I can confirm that the emergency recovery methods illustrated above seem to work for me:

set allow_session_lock_restore = true in hyprland.conf
add keybinding for resetting swaylock:bindl = SUPER, BackSpace, exec, pkill -SIGUSR1 swaylock && WAYLAND_DISPLAY=wayland-1 swaylock -f

When the issue occurs and input focus is lost on swaylock, I can use the keybinding to reset it. The screen will flash red briefly and then a new swaylock instance spawns which receives keyboard input again, enabling me to unlock the screen.

For reference are the software versions I am using:

swaylock-effects 1.6.11
swayidle 1.8.0
Hyprland 0.36.0 (from git via flake package)
GDM 45.0.1 (as display manager)
nixpkgs stable 23.11

I just encountered this again (I do almost once a day) and after I hit the emergency shortcut and unlocked my screen using a new swaylock instance, I could see parts of my password that I attempted to type until I realized swaylack is not reacting inserted into the terminal window that was visible on the workspace the lockscreen was covering.
So swaylock is not freezing but is actually loosing focus as the original description states. Furthermore, something on the previously active workspace might steal the focus.
As long as this isn't fixed anybody affected by this should be very careful when typing a password and hitting enter. Always check if swaylock is actually reacting to input before hitting enter. Otherwise you might post your password into a chat or something.

I thought I fixed this.

patch.txt
try

patch.txt try

Thank you. I will thoroughly test-drive the patch this week.

I have managed to hit this already:
/tmp/hypr/356414639fc06ace7045236ec41d673342981057_1711394832/hyprland.log:[ERR] BUG THIS: Session locked but focused isn't session lock on key
I was still unable to unlock it though.

To me, it happens mainly if I change the display setup. Like, I make the laptop sleep while plugged into a monitor and then wake it up without it, if I connect the monitor again then the focus works.

patch.txt
try this then

I have managed to hit the bug log but it seems to recover fine afterwards and I've always been able to log in so far.
[LOG] Switch Lid Switch fired, triggering binds.
[LOG] Switch Lid Switch turn on, triggering binds.
[LOG] New session lock!
[LOG] Session got locked by 2d60e40
[LOG] Registered signal for owner 2d60e40: 2d60e58 -> 249e5c0 (owner: wlr_session_lock_v1)
[LOG] Registered signal for owner 2d60e40: 2d60e68 -> 249e628 (owner: wlr_session_lock_v1)
[LOG] Registered signal for owner 2d60e40: 2d60e78 -> 249e690 (owner: wlr_session_lock_v1)
[LOG] Registered signal for owner 2d7cfd0: 2d72ed8 -> 2d7cfe8 (owner: SSessionLockSurface)
[LOG] Registered signal for owner 2d7cfd0: 2d41528 -> 2d7d050 (owner: SSessionLockSurface)
[LOG] Registered signal for owner 2d7cfd0: 2d72ec8 -> 2d7d0b8 (owner: SSessionLockSurface)
[LOG] SessionLockSurface 2d7cfd0 mapped
[LOG] Set keyboard focus to surface 2d72bf0
[LOG] New Gamma event at 7ffc8a1029c0
[LOG] New Gamma event at 7ffc8a1029c0
[LOG] Switch Lid Switch fired, triggering binds.
[LOG] Keybind handling only locked (inhibitor)
[LOG] Switch Lid Switch turn off, triggering binds.
[LOG] Keybind handling only locked (inhibitor)
[LOG] Callback 2d86a78 -> 2d86a70, Keyboard removed.
[LOG] Callback 2d86940 -> 2d86938, Keyboard removed.
[LOG] Callback 2d869a8 -> 2d869a0, Keyboard removed.
[LOG] Callback 2d86a10 -> 2d86a08, Keyboard removed.
[LOG] Destroyed keyboard 2d86910
[LOG] New Gamma event at 7ffc8a1029c0
[LOG] Registered signal for owner 2dd23f0: 2e0d918 -> 2dd2828 (owner: layerSurface)
[LOG] Registered signal for owner 2dd23f0: 2da5318 -> 2dd26f0 (owner: layerSurface)
[LOG] Registered signal for owner 2dd23f0: 2e0d928 -> 2dd2758 (owner: layerSurface)
[LOG] Registered signal for owner 2dd23f0: 2e0d938 -> 2dd27c0 (owner: layerSurface)
[LOG] Registered signal for owner 2dea9d0: 2da5328 -> 2deab50 (owner: CPopup Head)
[LOG] LayerSurface 2da5250 (namespace notifications layer 3) created on monitor eDP-1
[ERR] BUG THIS: No CWLSurface for surface in damageSurface!!!
[WARN] Orphaned wlr_surface 2e0d640 in setPreferredScaleForSurface
[WARN] Orphaned wlr_surface 2e0d640 in setPreferredTransformForSurface
[LOG] LayerSurface 2da5250 mapped
[LOG] Registered signal for owner 2dd25d0: 2e0d958 -> 2dd2618 (owner: CWLSurface)
[LOG] Registered signal for owner 2dd25d0: 2e0d918 -> 2dd2680 (owner: CWLSurface)
[LOG] CWLSurface 2dd25d0 called init()
[LOG] New Gamma event at 7ffc8a1029c0
[LOG] New Gamma event at 7ffc8a1029c0
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[LOG] cursorImage request: surface 0
[LOG] Callback 1bab070 -> 1bab068, CWLSurface removed.
[LOG] Callback 1bab0d8 -> 1bab0d0, CWLSurface removed.
[LOG] CWLSurface 1bab000 called destroy()
[LOG] cursorImage request: surface 2d38fe0
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
[LOG] New Gamma event at 7ffc8a1029c0
[LOG] Attached a keyboard with name Headphones (AVRCP)
[LOG] Registered signal for owner 2d86910: 2dce598 -> 2d86918 (owner: Keyboard)
[LOG] Registered signal for owner 2d86910: 2dce588 -> 2d86980 (owner: Keyboard)
[LOG] Registered signal for owner 2d86910: 2dce468 -> 2d86a50 (owner: Keyboard)
[LOG] Registered signal for owner 2d86910: 2dce5a8 -> 2d869e8 (owner: Keyboard)
[LOG] ApplyConfigToKeyboard for "headphones-(avrcp)", hasconfig: 0
[LOG] Attempting to create a keymap for layout ch-qwerty with variant  (rules: , model: , options: caps:swapescape)
[LOG] Updating keyboard 2d86910's translation state from an active index 0
[LOG] Set the keyboard layout to ch-qwerty and variant to  for keyboard "Headphones (AVRCP)"
[LOG] New keyboard created, pointers Hypr: 2d86910 and WLR: 2dce458
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] BUG THIS: key not found in m_dPressedKeys
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Keybind handling only locked (inhibitor)
[LOG] Session Unlocked
[LOG] Callback 249e6b8 -> 249e6b0, wlr_session_lock_v1 removed.
[LOG] Callback 249e5e8 -> 249e5e0, wlr_session_lock_v1 removed.
[LOG] Callback 249e650 -> 249e648, wlr_session_lock_v1 removed.
[LOG] Set keyboard focus to surface 2d2c810, with [Window 2d32b20: title: "YouTube — Mozilla Firefox"]
[LOG] SessionLockSurface 2d7cfd0 destroyed
[LOG] Callback 2d7d0e0 -> 2d7d0d8, SSessionLockSurface removed.
[LOG] Callback 2d7d078 -> 2d7d070, SSessionLockSurface removed.
[LOG] Callback 2d7d010 -> 2d7d008, SSessionLockSurface removed.

[ERR] MEGA BUG THIS: No session lock found at all to focus to??

wtf? sounds like a bug on the side of the lock?

[ERR] MEGA BUG THIS: No session lock found at all to focus to??

wtf? sounds like a bug on the side of the lock?

not sure its bug on the side of lock, because i tried swaylock and hyprlock this week.  have 2 same bugs on both of them. 1 - described here (no ability to input something after suspend + resume ) , 2 - no ability to change keyboard layout on lockscreen(but this one i have not tried to fix yet, so doesnt matter).

wtf? sounds like a bug on the side of the lock?

I don't think so:
[LOG] New session lock!
[LOG] Session got locked by 2d60e40
[LOG] SessionLockSurface 2d7cfd0 mapped
[LOG] Set keyboard focus to surface 2d72bf0
...
[LOG] LayerSurface 2da5250 (namespace notifications layer 3) created on monitor eDP-1
[LOG] LayerSurface 2da5250 mapped
[LOG] Set keyboard focus to surface 2d72bf0
...
[ERR] BUG THIS: Session locked but focused isn't session lock on key
[LOG] Set keyboard focus to surface 2d72bf0
[ERR] MEGA BUG THIS: No session lock found at all to focus to??
...
[LOG] Session Unlocked
[LOG] SessionLockSurface 2d7cfd0 destroyed

There's a lock surface present the entire time, so not much more for the client to do - that's the surface that should be given focus. Set keyboard focus to surface XYZ is also printing for each the "BUG THIS" lines, which during a lock can only happen if the surface passed to CCompositor::focusSurface is a session lock surface passing all checks.
Maybe print where wlr_seat_keyboard_notify_enter is called, print the resulting focused_surface after the call, and print then same focused_surface where you do the checks that fail to make sure they're all the same? Also make sure that the logs printing surface-related things use comparable pointers, e.g. all printing the underlying wlr_surface so you can match SessionLockSurface lines with focus lines directly.

wlroots stuff is long gone, if this bug is still there I'll revisit it after #6608 is done. Haven't experienced it ever myself so I dunno if it's still a thing

We just got another poke on that issue on swaylock, so I took a quick look before I closed the issue as being on the Hyprland side of things to make sure it wasn't an unreasonable assumption.

wlroots stuff is long gone,

heh, yeah I was looking at an old checkout - at least it it matched the logs. The business logic in question looks quite similar now though, so the issue might have carried over.

Is there any solution for this issue? Experience this as well every time I put laptop asleep with lock on. This is a critical issue for me


Support lts-3 in store-core

For store, th-reify-many and th-orphans are still needed, but this simplifies things.

Makes sense!


Referencing another variable defined within the same or different group, or in a pipeline

In the value of a group variable, is it supported to reference another variable within the same group ?  For example:
group_var_root = "\UncShare\Base\Path"
group_var_deep = "$(group_var_root)\some\deep\folder"
What about referencing a variable in a different variable group (seems like a bad approach, but someone might have a need)?  What about referencing a variable defined in a pipeline (effectively making the library group variable require a specific pipeline variable to be defined)?
In essence, this question boils down to "Is all variable group linking resolved before variable value substitution is begun?"  Please document this behavior regardless of the answers.  Thanks!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4827aa99-ae62-bd63-6f2f-a87a4065ed27
Version Independent ID: c9e461ee-7f42-3503-9460-18fa3a807bbb
Content: Variable groups for Azure Pipelines and TFS - Azure Pipelines
Content Source: docs/pipelines/library/variable-groups.md
Product: devops
Technology: devops-cicd
GitHub Login: @alexhomer1
Microsoft Alias: ahomer

Can we reuse the value of a key in the same variable group? If this is possible, what is the syntax to refer to the key?
As in, I have a variable App.URL : www.contoso.com. I am using the key to substitute value in my next variable like this Login.URL : $(App.URL)\Login and this doesn't work.


adding changes to assignment_1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Added code and markdown to answer the questions in the assignment
What did you learn from the changes you have made?
About the linear and multiple linear regression models, how to fit, summarize, plot and how to drive observations and conclusions out of it on the fitness and accuracy of the model
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
no
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
working on the syntax of add_constants resulted in error as it was expecting the input to be a 2D array for multiple regression
How were these changes tested?
on the vs code
A reference to a related issue in your repository (if applicable)
NA
Checklist

[ Y ] I can confirm that my changes are working as intended

A1: Q2 missing hypotheses statements

hi @amanda-ng518 , are you mentioning about this one in question 2. I had this answer submitted, is that statement correct ?
(ii) In the context of the Boston dataset, state the null and alternative hypotheses.
The The F-statistics is significantly high (247) and the p-value associated with the F-statistic (1.84e-75) is extremely low, much lower than the typical significance level (e.g., 0.05). The low p-value for the F-statistic indicates that the model is statistically significant, meaning that the predictors have a relationship with the response variable. So the null hypothesis can be rejected and alternative hypothesis holds good.


postgres: Reflect decimal.handling.mode in money conversion and allow to specify scale

https://issues.redhat.com/browse/DBZ-1931

* Do we have a test which doesn't provide the decimal handling mode setting, so to make sure that default mapping for `MONEY` column doesn't change?

Good point, AFAICT there's no such test, adding it.

* Could you update the docs (see https://github.com/debezium/debezium/blob/main/documentation/modules/ROOT/pages/connectors/postgresql.adoc) accordingly; we should describe the mapping of montary columns (currently not listed in the table with type mappings), and we should describe that `decimal.handling.mode` impacts monetary columns; probably best to search for references to that option name so to see where adjustments should be made

Sure, I wanted to check if you are fine the proposed changes first, before spending time with documentation :-)
Doc changes may take me some time, will push it in next version. Now pushing new test testing default mapping and change you've proposed for testStreamMoneyAsString()

Added documentation updates.


Some love for pcapfile_info

Current coverage is 85.83% (diff: 100%)

Merging #32 into master will not change coverage

@@             master        #32   diff @@
==========================================
  Files            15         15
  Lines          1221       1221
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1048       1048
  Misses          173        173
  Partials          0          0

Powered by Codecov. Last update 4a19ce1...da61f1d

Thanks, @asergi.


Upgraded to SPFx 1.6.0

Q
A

Bug fix?
no - yes?

New feature?
no - yes?

New sample?
no - yes?

Related issues?
fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?
Changes necessary to upgrade react-command-dialog web part to SPFx 1.6.0
Guidance
No special additional guidance necessary

 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.:x: msekkappan sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Please let me know if I can be of any help

thx @msekkappan, appreciated.


[Docs] Relay session & claim/proof diagrams

Summary
Incorporates RPC Session Life-Cycle into docusaurus docs.
Issue

MVT features table
#159

Type of change
Select one or more:

[ ] New feature, functionality or library
[ ] Bug fix
[ ] Code health or cleanup
[ ] Documentation
[ ] Other (specify)

Testing

[ ] Unit Tests: make go_develop_and_test
[ ] LocalNet E2E Tests: make test_e2e
[ ] DevNet E2E Tests: Add the devnet-test-e2e label to the PR. THIS IS VERY EXPENSIVE, so only do it after all the reviews are complete.
[ ] Documentation changes: make docusaurus_start

Sanity Checklist

[ ] I have tested my changes using the available tooling
[ ] I have commented my code
[ ] I have performed a self-review of my own code; both comments & source code
[ ] I create and referenced any new tickets, if applicable
[ ] I have left TODOs throughout the codebase, if applicable

@bryanchriswhite Aiming to review this by mid week.

@bryanchriswhite Just a heads up that I started reviewing this!

%% ---
%% displayMode: compact
%% ---
gantt
    title Session Relay / Claim / Proof Windows
    dateFormat ss
    axisFormat %S
    tickInterval 1second

    section Relays
        Session N Start : milestone, sns, 00, 0s
        num_blocks_per_session          :nbps, 00, 4s
        Session N End :milestone, sne, after nbps, 0s
        grace_period_end_offset_blocks : gpof, after sne, 2s
        Session N+1 Start : milestone, after sne, 0s
    section Claim Window
        claim_window_open_offset_blocks : cwob, after sne, 10ms
        Session N Claim Window Open : milestone, cwo, after cwob, 0s
        claim_window_close_offset_blocks : cwcb, after cwo, 4s
        Session N Claim Window Close : milestone, cwc, after cwcb, 0s
    section Proof Window
        proof_window_open_offset_blocks : pwob, after cwc, 10ms
        Session N Proof Window Open : milestone, pwo, after pwob, 0s
        proof_window_close_offset_blocks : pwcb, after pwo, 4s
        Session N PRoof Window Close : milestone, pwc, after pwcb, 0s

timeline
    title Session Claim/Proof Windows

    section Session
    Session
    : session N starts
    : relays for session N
    : num_blocks_per_session blocks elapse
    : session N ends
    section Claim Window
    Claim window & grace period
    : late relays for session N
    : session_grace_period blocks elapse
    : grace period ends

    : claim_window_open _offset_blocks - session_grace_period blocks elapse
    : claim window opens
    : claims for session N
    : claim_window_close _offset_blocks - session_grace_period blocks elapse
    : claim window closes
    section Proof Window
    Proof window
    : proof_window_open _offset_blocks  blocks elapse
    : proof window opens
    : proofs for session N
    : proof_window_close _offset_blocks blocks elapse
    : proof window closes


from .async import process_pool

File "F:\Face-Recognition-Attendance-System-master\attendance\recognizer.py", line 2, in
import xlwrite,firebase.firebase_ini as fire;
File "C:\Users\HP\AppData\Local\Programs\Python\Python37\lib\site-packages\firebase_init_.py", line 3
from .async import process_pool
^
SyntaxError: invalid syntax

Check this out - https://www.youtube.com/watch?v=TiMACTNbNl8
It might help

thanks, bro its helped me to solve the issue


Quality control edits

I edited the hands-on material and slides for the quality control tutorial about FastQC and TrimGalore!. In particular, I applied modifications for:

fixing spelling and syntax
re-wording explanations

So many nice things included! Thanks a lot!


Added model parameter for Google Cloud Speech API

Based on documentation: https://cloud.google.com/speech-to-text/docs/reference/rest/v1p1beta1/RecognitionConfig

Thanks.
SpeechRecognition 3.13.0 supports recognize_google_cloud(model=...)


Opening terminal in a k8s pod attempts to activate venv if named k8s

Type: Bug

Both Kubernetes and Python extentions enabled
Open a terminal on a k8s pod
See terminal open and attempt to active a local virtualenv named k8s

source /home/user/.virtualenvs/k8s/bin/activate
bash-4.2$ source /home/user/.virtualenvs/k8s/bin/activate
bash: /home/user/.virtualenvs/k8s/bin/activate: No such file or directory

This k8s venv is something I created, if I rename it, this doesnt happen
Extension version: 2022.18.2
VS Code version: Code 1.73.1<PHONE_NUMBER>f055b99068d3688932416f2346dd3b, 2022-11-09T04:27:29.066Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: No
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-10850H CPU @ 2.70GHz (12 x 2712)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
31.73GB (15.98GB free)

Process Argv
--crash-reporter-id 2c833a45-a7cd-4fb4-a5f0-d73c3424a0c5

Screen Reader
no

VM
0%

Item
Value

Remote
WSL: Ubuntu-20.04

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
Intel(R) Core(TM) i7-10850H CPU @ 2.70GHz (6 x 2712)

Memory (System)
19.54GB (16.05GB free)

VM
0%

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyone:30548225
pyindex848:30577860
nodejswelcome1:30587005
282f8724:30602487
gswce1:30612156
3d0df643:30613357
dbltrim-noruby:30604474
f6dab269:30613381

Was the k8s environment your selected environment for your workspace? If so, then this is expected. Otherwise the environment exists in a place where only virtual environments should exist and thus that activate file should be there.

Because we have not heard back with the information we requested, we are closing this issue for now. If you are able to provide the info later on then we will be happy to re-open this issue to pick up where we left off.


Raising proposal based on PR discussion for: Enhance KafkaBridge resurce with consumer inactivity timeout and HTTP consumer/producer parts enablement

Thanks for the proposal. I left few initial comments.

Comments resolved @scholzj.

@scholzj, thanks for explaining :)
Though it was merged and then discussed after, but that makes sense - my bad.

@k-wall I would try to avoid doing the deep nesting and having consumer sections everywhere. So I would not put it under .soec.http.consumer and should it be related only to consumer, I would go for .spec.consumer.timeoutSeconds. WDYT @ppatierno?

@scholzj if timeoutSeconds should be available for both consumer and producer, I think thats a better solution, more in line with version 2 presented.

@scholzj if timeoutSeconds should be available for both consumer and producer, I think thats a better solution, more in line with version 2 presented.

If it applies to both, it makes more sense under spec.http as you have it I guess. That also coresponds to http.timeoutSeconds as it exists in the bridge itself. But I would defer to @ppatierno as to what he thinks.

@scholzj if timeoutSeconds should be available for both consumer and producer, I think thats a better solution, more in line with version 2 presented.

If it applies to both, it makes more sense under spec.http as you have it I guess. That also coresponds to http.timeoutSeconds as it exists in the bridge itself. But I would defer to @ppatierno as to what he thinks.

Sure thing. Sorry for the potential misunderstanding, what I meant was if its relevant for those timeoutSeconds to differ from consumer to producer.

https://github.com/strimzi/strimzi-kafka-bridge/blob/main/src/main/java/io/strimzi/kafka/bridge/http/HttpConfig.java#L37 shows this property applies only to the consumer.  I there should be a change considered against the Bridge itself that would rename the property http.consumer.timeoutSeconds (retaining but deprecating the old name). The API being proposed by this change should be .spec.consumer.timeoutSeconds.

The choice is not so obvious :-( and I think that the initial choice of http.timeoutSeconds was bad (sorry for that), because it didn't make it clear that the configuration is for the consumer only (as pointed by @k-wall).
The thing is that the bridge has two parts: http and kafka. But they are not that clear in the KafkaBridge CRD (i.e. consumer.config is about Kafka related parameter so everything like kafka.*, but it's not consumer.kafka.config which could have been better).
Another idea ... I know it's going to use a different order from the application.properties file but what about using consumer.http.timeoutSeconds, consumer.http.enabled, producer.http.enabled ? I know they will be the opposite of what we have in the application properties starting with http.* but we already have the approach of consumer and producer sections in the KafkBridge custom resource.
At the same time it means an additional inner level, which we could avoid.
@scholzj @k-wall @steffen-karlsson wdyt?

@ppatierno I would try to avoid the deep nesting and the confusing use of http in multiple places. So maybe you could do something like:
spec:
  http:
    consumerEnabled: true
    producerEnabled: true
    consumerTimeoutSeconds: 60

if you think these things belong to HTTP?

@scholzj also your suggestion is fine but I am really struggling to take a strong position. I am really regretting the choices made in the past for the current names in the application.properties file.
Regarding the enabled field for example, while it's enabling/disabling the HTTP part of consumer/producer, in reality it's enabling/disabling the consumer/producer as whole (disabling HTTP, no Kafka client is created) so the best should have been just consumer.enabled and producer.enabled (without the http prefix). So the timeout seconds should have been just consumer (if we assume that any other Kafka related timeout should go in the config part).
At this point I would be for having these params directly under consumer (enabled and timeoutSeconds) and producer (enabled) sections.

APIs are hard :-(. When it comes to bad decisions in the KafkaBridge custom resource, I guess the v1 api will be an opportunity. To change the application.properties is I guess less complicated technically, but would break compatibility. But in any case I don't think the KafkaBridge schema absolutely has to follow the application.properties if it doesn't make sense.

Isn't calling it a http timeout is actually misleading?   It could be confused with the timeouts applied to the HTTP protocol itself.   The timeout we are talking about is intrinsic to the consumer bridge.  So that train of thought leads me to:
consumer:
    enabled: true
    timeoutSeconds: 60
    config:
      auto.offset.reset: earliest
      enable.auto.commit: true

@ppatierno Sure, let me do that :)

Not sure exactly what you mean by "Also I would avoid the code formatting on "http enablement", "consumer" and "producer" but using it for actual code and fields." @ppatierno
Everything else is updated @ppatierno and @k-wall.

Not sure exactly what you mean by "Also I would avoid the code formatting on "http enablement", "consumer" and "producer" but using it for actual code and fields." @ppatierno

The usage of the single apex which provides the formatting like this formatting with single apex is typically for code and yaml fields while you are just mentioning "http enablement", "consumer" and "producer". So I would avoid having something like http enablement, consumer and producer unless you refer to fields in the spec or code.

@steffen-karlsson maybe I was not clear but now your removed single apex from everywhere even where it makes sense :-D ... for example http.consumer.timeoutSeconds makes sense having the single apex and formatted as code, but you removed from it having just http.consumer.timeoutSeconds.
As I said, YAML fields should have such format.

@steffen-karlsson just adding one more thing. The 071 number was just taken by another merged proposal.
You should rename the file using the next available number and also update the README file with this proposal (number and title).

@steffen-karlsson as for the other PR, I see a lot of commits which are not related to the proposal. What's happened? Can you try to clean it please? Closing and opening is not good because we'll lost all the approvals we already have so you should fix on this PR imho.

@steffen-karlsson as for the other PR, I see a lot of commits which are not related to the proposal. What's happened? Can you try to clean it please? Closing and opening is not good because we'll lost all the approvals we already have so you should fix on this PR imho.

Git happend, tried to update from main due to conflict and then my fork was out of sync.
Let me squash it all into one

This proposal has enough votes to be approved and merged (4 binding, 1 non-binding).
Because it's has been opened for long time, I will wait the end of tomorrow (April 16th) and then approve/merged if no further feedback are provided.

This proposal is approved with 4 binding and 1 non-binding votes. Going to merge.
@steffen-karlsson you can even back to work on the implementation PR changing it accordingly. Thanks!


WIP: add support for live logs for Sauce Orchestrate

Proposed changes

Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] I have updated the json schema (if appropriate)

Further comments

supersided by https://github.com/saucelabs/saucectl/pull/869


Notebooks - variable explorer icon taking long time to appear in toolbar

Environment data

VS Code version: 1.72.0 - insider
Jupyter Extension version (available under the Extensions sidebar): v2022.9.1002791758
Python Extension version (available under the Extensions sidebar): v2022.16.0
OS (Windows | Mac | Linux distro) and version: Windows
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: Local

Expected behaviour
Variable Explorer icon to be in toolbar reasonably quickly
Actual behaviour
When opening notebooks or creating new notebooks it takes a really long time for the variable explorer icon to appear in the toolbar
Steps to reproduce:

create new notebook
wait for variable explorer icon to appear

The only thing that would slow it down is this part of the "when" clause notebookKernel =~ /^ms-toolsai.jupyter\\//, we can't add the button until we know that it's compatible with the selected kernel. For me, the button appears pretty much instantly, along with the kernel. It's also possible that this got faster in the past couple months with all of the work around kernels we've done. If it's still slow, then that's probably because the kernel selection is slow and we can't really improve the button. Let me know if this is still bad for you on the latest, especially if it seems like it takes longer than it takes to select the kernel.


Create bootstrap_5_layout.html.twig

Changelog
Added

Compatibility with bootstrap5

Hi. This works fine. Can it be merged?

Hi,
It would be great if this could be merged, it works fine and fixes the tabs not working in Symfony5 / EA 3 !

Thanks!

@webda2l are you going to create a new release with this change or should I use master in my composer? Thank you!


how do i make it vertical

Vertical switches are not yet supported, but are already planned.

@shreyas-jadhav Currently you can use RotatedBox for rotating your switch and its content:
RotatedBox(
   quarterTurns: 1,
   child: AnimatedToggleSwitch(
       iconBuilder: (value) => RotatedBox(quarterTurns: -1, child: ...),
       ...
   ),
)

Does this work for you?

I have now added a vertical() method to AnimatedToggleSwitch. You can use this in your pubspec.yaml to test this new feature :
animated_toggle_switch:
    git:
      url: https://github.com/splashbyte/animated_toggle_switch.git
      ref: 61-vertical-switch


accessToken

Hi
I've tried getting grabbing the accessToken from the  Developer-Console > Console / F12  both Chrome and FireFox both return NULL
Yes I've logged in

Run this in console
for (var a in localStorage) {    console.log(a, ' = ', localStorage[a]); }
Returns the following

OnlyFans has changed a few things and this downloader no longer supports downloading as it used to.


Doesn't work on Deno

This repo has published into https://jsr.io but if I follow the typescript example when running on Deno it just says

window not found

Which of course it doesn't exist in Deno.

First of all, on JSR I didn't enable Deno compatibility.
And this package is supposed to be run in a webview window to communicate with tauri core.
It should not be used in Deno runtime.

You should look at the “works with” part above this button. I can’t control the installation button , it’s always there.

Ok, I'll close this but I was assuming since it was rust it was compiling to wasm and being loaded with ffi, which deno supports. Theoretically this should be able to work without the window when running in deno but I haven't dug in that deeply yet.

No, this is a plugin for tauri, the js part only runs in browser to communicate with the core process written in rust. You can find my video talking about plugin on YouTube.

Do you know what tauri is? If you are trying to access clipboard data with deno, then this plugin is not for you.
you could try https://github.com/CrossCopy/clipboard

BTW, you can install the package with deno as a package manager, but not as a runtime.
e.g. If you are running react dev server with deno, then it's OK to install the package with Deno, but the package's JS code still runs in browser.


HasOne doesn't work in the default tab

If there is a HasMany relationship in the first tab, it works, but it doesn't work for HasOne, only if you switch to another tab and then the first tab.

Fixed: v2.1.5


golangci: disable comment warnings

Comments will just slow us down right now.  We'll do a comment sweep when the codebase is stabilized.

this was fixed in #250


Test fails: PageNotFound

I installed ratebeer on my Mac using python 2.7 by running:
virtualenv testA
source testA/bin/activate
pip install git+https://github.com/alilja/ratebeer.git
wget https://raw.githubusercontent.com/alilja/ratebeer/master/test.py
Running python test.py returns:
======================================================================
ERROR: test_beer (__main__.TestBeer)
Make sure the results for a beer contain the expected data
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test.py", line 20, in test_beer
    results = RateBeer().beer('/beer/new-belgium-tour-de-fall/279122/')
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 136, in beer
    return self.get_beer(url, True).__dict__
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 132, in get_beer
    return models.Beer(url, fetch)
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 72, in __init__
    self._populate()
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 112, in _populate
    raise rb_exceptions.PageNotFound(self.url)
PageNotFound: /beer/new-belgium-tour-de-fall/279122/

======================================================================
ERROR: test_beer_aliased (__main__.TestBeer)
Check that AliasedBeer exception is raised properly
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test.py", line 111, in test_beer_aliased
    self.assertRaises(rb_exceptions.AliasedBeer, rb.beer, "/beer/new-belgium-biere-de-mars/113241/")
  File "/Users/mariel/anaconda/lib/python2.7/unittest/case.py", line 473, in assertRaises
    callableObj(*args, **kwargs)
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 136, in beer
    return self.get_beer(url, True).__dict__
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 132, in get_beer
    return models.Beer(url, fetch)
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 72, in __init__
    self._populate()
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 112, in _populate
    raise rb_exceptions.PageNotFound(self.url)
PageNotFound: /beer/new-belgium-biere-de-mars/113241/

======================================================================
ERROR: test_beer_closed_brewery (__main__.TestBeer)
Handling beers from closed brewers
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test.py", line 42, in test_beer_closed_brewery
    results = RateBeer().beer('/beer/hantens-hildener-landbrau/140207/')
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 136, in beer
    return self.get_beer(url, True).__dict__
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 132, in get_beer
    return models.Beer(url, fetch)
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 72, in __init__
    self._populate()
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 112, in _populate
    raise rb_exceptions.PageNotFound(self.url)
PageNotFound: /beer/hantens-hildener-landbrau/140207/

======================================================================
ERROR: test_beer_closed_contract_brewery (__main__.TestBeer)
Handling beers from closed contract brewers
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test.py", line 47, in test_beer_closed_contract_brewery
    results = RateBeer().beer('/beer/crew-republic-x-11-wet-hop/298026/')
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 136, in beer
    return self.get_beer(url, True).__dict__
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 132, in get_beer
    return models.Beer(url, fetch)
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 72, in __init__
    self._populate()
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 112, in _populate
    raise rb_exceptions.PageNotFound(self.url)
PageNotFound: /beer/crew-republic-x-11-wet-hop/298026/

======================================================================
ERROR: test_beer_contract_brewed (__main__.TestBeer)
Handling contract brewed beers
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test.py", line 53, in test_beer_contract_brewed
    results = RateBeer().beer('/beer/benediktiner-weissbier/157144/')
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 136, in beer
    return self.get_beer(url, True).__dict__
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 132, in get_beer
    return models.Beer(url, fetch)
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 72, in __init__
    self._populate()
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 112, in _populate
    raise rb_exceptions.PageNotFound(self.url)
PageNotFound: /beer/benediktiner-weissbier/157144/

======================================================================
ERROR: test_beer_get_reviews (__main__.TestBeer)
Check to make multi-page review searches work properly
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test.py", line 61, in test_beer_get_reviews
    self.assertIsNotNone(next(reviews))
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 213, in get_reviews
    self._populate()
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 112, in _populate
    raise rb_exceptions.PageNotFound(self.url)
PageNotFound: /beer/deschutes-inversion-ipa/55610/

======================================================================
ERROR: test_beer_no_abv (__main__.TestBeer)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test.py", line 70, in test_beer_no_abv
    results = RateBeer().beer('/beer/deschutes-altitude-amber/92102/')
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 136, in beer
    return self.get_beer(url, True).__dict__
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 132, in get_beer
    return models.Beer(url, fetch)
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 72, in __init__
    self._populate()
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 112, in _populate
    raise rb_exceptions.PageNotFound(self.url)
PageNotFound: /beer/deschutes-altitude-amber/92102/

======================================================================
ERROR: test_beer_no_ratings (__main__.TestBeer)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test.py", line 84, in test_beer_no_ratings
    results = RateBeer().beer('/beer/deschutes-abyssident/194792/')
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 136, in beer
    return self.get_beer(url, True).__dict__
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 132, in get_beer
    return models.Beer(url, fetch)
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 72, in __init__
    self._populate()
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 112, in _populate
    raise rb_exceptions.PageNotFound(self.url)
PageNotFound: /beer/deschutes-abyssident/194792/

======================================================================
ERROR: test_beer_retired_beer (__main__.TestBeer)
Attributes for retired beers display properly
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test.py", line 105, in test_beer_retired_beer
    results = RateBeer().beer('/beer/shorts-funkin-punkin/79468/')
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 136, in beer
    return self.get_beer(url, True).__dict__
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 132, in get_beer
    return models.Beer(url, fetch)
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 72, in __init__
    self._populate()
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 112, in _populate
    raise rb_exceptions.PageNotFound(self.url)
PageNotFound: /beer/shorts-funkin-punkin/79468/

======================================================================
ERROR: test_beer_unicode (__main__.TestBeer)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test.py", line 97, in test_beer_unicode
    results = RateBeer().beer('/beer/stedji-oktoberbjor/292390/')
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 136, in beer
    return self.get_beer(url, True).__dict__
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 132, in get_beer
    return models.Beer(url, fetch)
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 72, in __init__
    self._populate()
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/models.py", line 112, in _populate
    raise rb_exceptions.PageNotFound(self.url)
PageNotFound: /beer/stedji-oktoberbjor/292390/

======================================================================
ERROR: test_str_nonascii_search (__main__.TestSearch)
Test out the search function with a str with more than ASCII characters
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test.py", line 184, in test_str_nonascii_search
    results = RateBeer().search("to øl jule mælk")
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 120, in search
    url = '/beer/{0}/{1}/'.format(beer_data['name'].replace(' ', '-').lower(), beer_data['id'])
UnicodeEncodeError: 'ascii' codec can't encode character u'\xf8' in position 3: ordinal not in range(128)

======================================================================
ERROR: test_unicode_nonascii_search (__main__.TestSearch)
Test out the search function with a unicode string with more than ASCII characters
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test.py", line 202, in test_unicode_nonascii_search
    results = RateBeer().search(u"to øl jule mælk")
  File "/Users/mariel/testA/lib/python2.7/site-packages/ratebeer/ratebeer.py", line 120, in search
    url = '/beer/{0}/{1}/'.format(beer_data['name'].replace(' ', '-').lower(), beer_data['id'])
UnicodeEncodeError: 'ascii' codec can't encode character u'\xf8' in position 3: ordinal not in range(128)

======================================================================
FAIL: test_fetch_by_letter (__main__.TestAlpha)
Make sure the results for a brewery list by index contain the expected data
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test.py", line 216, in test_fetch_by_letter
    self.assertTrue(beer.url == u'/brewers/a-duus-and-co/1668/')
AssertionError: False is not true

----------------------------------------------------------------------
Ran 24 tests in 9.073s

FAILED (failures=1, errors=12)

Any ideas on how to fix this?

@marielvanstav, it's been a little slow around here to merge in the necessary changes, but you can pull from my fork and it should fix most things https://github.com/OrganicIrradiation/ratebeer/issues/51. Searching and getting beer information, at least works. I haven't had the time recently to go in and fix some of the ancillary API calls. Let me know if you have any issues.

@parryc Hi, have you had any progress? I'm trying to use the version you mentioned in the comment, but the functionalities are still limited - I'm having problems getting review and brewery code to work. My main goal was to get the get_reviews code running, do you have any idea how I could fix it?

Hi @jciupis, sorry, no update from me – brewery and reviews are unfortunately not needed for my specific uses, so I haven't been compelled to update my fork to the newest API (as frustrating as it may sound for you.
It appears from a very quick look, that the API call for review information is some combination of the two operations below. Using this beer as an example.
[{"operationName":"beerReviews","variables":{"beerId":"484278"},"query":"query beerReviews($beerId: ID!) {\n  beerReviews(beerId: $beerId, order: TOP_RATERS, first: 5) {\n    reviews: items {\n      score\n      comment\n      createdAt\n      author {\n        username\n        city\n        state {\n          name\n          __typename\n        }\n        country {\n          name\n          __typename\n        }\n        __typename\n      }\n      __typename\n    }\n    __typename\n  }\n}\n"},{"operationName":"tagDisplay","variables":{"beerId":"484278"},"query":"query tagDisplay($beerId: ID!, $first: Int) {\n  tagDisplayArr: beerTags(beerId: $beerId, first: $first) {\n    items {\n      id\n      urlName: plain\n      __typename\n    }\n    __typename\n  }\n}\n"},{"operationName":"beerReviews","variables":{"beerId":"484278","order":"RECENT","first":7},"query":"query beerReviews($beerId: ID!, $authorId: ID, $order: ReviewOrder, $after: ID) {\n  beerReviewsArr: beerReviews(beerId: $beerId, authorId: $authorId, order: $order, after: $after) {\n    items {\n      ...ReviewItem\n      __typename\n    }\n    totalCount\n    last\n    __typename\n  }\n}\n\nfragment ReviewItem on Review {\n  id\n  comment\n  score\n  scores {\n    appearance\n    aroma\n    flavor\n    mouthfeel\n    overall\n    __typename\n  }\n  author {\n    id\n    username\n    reviewCount\n    __typename\n  }\n  checkin {\n    id\n    place {\n      id\n      name\n      city\n      state {\n        id\n        name\n        __typename\n      }\n      country {\n        id\n        name\n        __typename\n      }\n      __typename\n    }\n    __typename\n  }\n  servedIn\n  likeCount\n  likedByMe\n  createdAt\n  updatedAt\n  __typename\n}\n"}]
There does not appear to be an API call to scrape Brewery information, so those information will have to be old school scraped from the web site, following the code in Brewery._populate.  Given the renewed interest (2 people in a month!) I will try to fix the rest of the code sooner rather than later. Please post again if you have any questions or if you are able to update the Review and Brewery code. Feel free to fork the fork above and we can merge everything in together, back into this main repo.

@parryc Actually, I managed to get text of reviews with the snippet you provided. As I need the data for an academic machine learning project, that was basically my goal. Thank you very much for your help!
I have close to no experience in web stuff, so I don't think I would be much help with fixing the whole thing properly. You can consider my issue closed though :)

Great, best of luck @jciupis. Closing issue.


Is there API Documentation?

Is there API documentation somewhere that indicates how these interfaces are to be used?

For example... what exactly is the offset here?
export interface Loc {
    /**
     * @public
     */
    line: number;
    /**
     * @public
     */
    col: number;
    /**
     * @public
     */
    offset: number;
}

A lot of it doesn't have explanations yet, but if you head over to the discord server there's a #plugin-dev channel where other plugin devs can help you.
There's also some docs here that may help: https://marcus.se.net/obsidian-plugin-docs/

any idea what the official roadmap for the API is? atm it is quite underwhelming
I think that obsidian will thrive because of plugins, making it hard to code / maintain plugins will lead to the opposite.
Especially as a lot of major plugins are looking like they have been abandoned already.

There is now official doc: https://docs.obsidian.md/


Adding profile.md

Just adding a profile.md file.

I have created a branch, created a file, and made a commit.


CZ gate is now shown when drawing the circuit using mpl output type.

Summary
A simple change was made to show the CZ gate when drawing the circuit using mpl output type.
For this simple snippet:
import qiskit
# Draw a circuit and plot it
circuit = qiskit.QuantumCircuit(2, 2)

circuit.cx(0, 1)
circuit.cz(0, 1)
circuit.cy(0, 1)

circuit.draw(output='mpl')

Before the change:

After the change

Details and comments
When plotting a circuit using a CZ gate, the user may be confused by the image that is drawn since it's not clear which operation is being applied and where. This changes this by not making the CZ gate a case for the _symmetric_gate function.
P.S: This is my first pull request, apologies in advance for any inconvenience this may cause.

 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.

Thank you so much for your kind response!
For anyone in the future that doesn't understand why this is deliberate, this thread helped me out.


Update EKS-A E2E AMI to latest

Issue #, if available:
Description of changes:
Update EKS-A E2E AMI to latest. Weekly update
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve


OPENSANCTIONS_RESOLVER_PATH issue - downloaded resolve.ijson

Hello,
Environment:
$ uname -a
Darwin Landons-iMac.local 20.6.0 Darwin Kernel Version 20.6.0: Thu Sep 29 20:15:11 PDT 2022; root:xnu-7195.141.42~1/RELEASE_X86_64 x86_64
$ docker -v
Docker version 20.10.21, build baeda1f
$ git pull
Already up to date.

Problem:
I downloaded the file resolve.ijson and put it in data/.  After setting the environment variable OPENSANCTIONS_RESOLVER_PATH the dedupe candidates do not build.  Fails while Building index blocking pairs with UnboundLocalError: local variable 'idx' referenced before assignment.
I don't know where to go from here.
Output:
docker-compose run --rm app bash
root@work:/opensanctions# echo $OPENSANCTIONS_RESOLVER_PATH
/opensanctions/data/resolve.ijson
root@work:/opensanctions# opensanctions xref -l 50000 sanctions
2022-11-30 01:01:56 [info     ] Resolving canonical_id in statements... [opensanctions.core.statements] resolver=<Resolver('resolve.ijson', 185540)>
2022-11-30 01:02:12 [info     ] Xref running, auto merge threshold: 0.990000 [opensanctions.core.xref]
2022-11-30 01:02:12 [info     ] Loading database cache...      [opensanctions.core.loader] scope=<Collection('sanctions')>
2022-11-30 01:02:12 [info     ] Begin xref: <CachedDatasetLoader(<Collection('sanctions')>)>, resolver: <Resolver('resolve.ijson', 185540)> [nomenklatura.xref]
2022-11-30 01:02:12 [info     ] Building index from: <CachedDatasetLoader(<Collection('sanctions')>)>... [nomenklatura.index.index]
2022-11-30 01:02:12 [info     ] Built index: <Index('sanctions', 0, 0)> [nomenklatura.index.index]
2022-11-30 01:02:12 [info     ] Building index blocking pairs... [nomenklatura.index.index]
Traceback (most recent call last):
  File "/usr/local/bin/opensanctions", line 33, in <module>
    sys.exit(load_entry_point('opensanctions', 'console_scripts', 'opensanctions')())
  File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/opensanctions/opensanctions/cli.py", line 99, in xref
    blocking_xref(dataset, limit=limit, auto_threshold=auto)
  File "/opensanctions/opensanctions/core/xref.py", line 21, in blocking_xref
    xref(
  File "/usr/local/lib/python3.10/dist-packages/nomenklatura/xref.py", line 90, in xref
    _print_stats(idx, suggested, scores)
UnboundLocalError: local variable 'idx' referenced before assignment

This was an upstream bug in nomenklatura, run pip install -U nomenklatura to fix. It also only happens if you have no data in the database, so you may want to run opensanctions crawl sanctions or similar before running xref.


Typo ".Sln"→".sln"

https://docs.microsoft.com/en-us/visualstudio/extensibility/internals/solution-dot-sln-file?view=vs-2022
https://github.com/MicrosoftDocs/visualstudio-docs/blob/main/docs/extensibility/internals/solution-dot-sln-file.md
#PingMSFTDocs

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@leslierichardson95
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"

@TerryGLee: I'm sorry - only the author of this article, @leslierichardson95, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off


Explain how the module works in the README

Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

Haha, amazing feedback :)


Upgrading from Pydantic 2.6.4 -> 2.7.0 throws ModuleNotFoundError.

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
I have a project (managed via Poetry) that has Pydantic as a dependency; we've been using Pydantic 2.6.x for some time with no issues.
I updated my project dependencies (including Pydantic) this morning, which included pulling in Pydantic 2.7.0, and previously-running code now produces:
.venv/lib/python3.12/site-packages/pydantic/__init__.py:374: in <module>
    _getattr_migration = getattr_migration(__name__)
.venv/lib/python3.12/site-packages/pydantic/_migration.py:260: in getattr_migration
    from .errors import PydanticImportError
.venv/lib/python3.12/site-packages/pydantic/errors.py:6: in <module>
    from typing_extensions import Literal, Self
E   ModuleNotFoundError: No module named 'typing_extensions'

Environment is Python 3.12, and I've deleted and recreated the virtual environment and re-installed all dependencies to make sure there was nothing odd happening there.
Example Code
from pydantic import BaseModel

Python, Pydantic & OS Version
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/Users/msimpson/Desktop/Projects/i2ns/systems/insight/python/i2ns-insight-api/.venv/lib/python3.12/site-packages/pydantic/__init__.py", line 374, in <module>
    _getattr_migration = getattr_migration(__name__)
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/msimpson/Desktop/Projects/i2ns/systems/insight/python/i2ns-insight-api/.venv/lib/python3.12/site-packages/pydantic/_migration.py", line 260, in getattr_migration
    from .errors import PydanticImportError
  File "/Users/msimpson/Desktop/Projects/i2ns/systems/insight/python/i2ns-insight-api/.venv/lib/python3.12/site-packages/pydantic/errors.py", line 6, in <module>
    from typing_extensions import Literal, Self
ModuleNotFoundError: No module named 'typing_extensions'

Could you give more info about how pydantic was installed in Poetry? typing_extensions is correctly specified as a dependency so something is probably wrong on your end

Hello, I've encountered the same problem.
Seems like that installing<EMAIL_ADDRESS>with poetry, it doesn't install also pydantic_core and annotated_types

Vilcos --
Poetry runs dependency tracking off of pyproject.toml; the relevant section of that file for my prokect looks like:
[tool.poetry.dependencies]
blessings = "^1.7"
email-validator = "^2.1.1"
icecream = "^2.1.3"
pydantic = "^2.6.4"
pyjwt = {extras = ["crypto"], version = "^2.8.0"}
python = "^3.12"
result = "^0.16.1"

The burn-it-down-and-reinstall-from scratch procedure I use involved removing the virtual environment created by Poetry, removing the Poetry lockfile, and then invoking poetry install, which triggers the dependency resolution and installation of the new virtual environment, e.g.:
$ rm -rf .venv
$ rm poetry.lock
$ poetry install
Creating virtualenv in .venv
Updating dependencies
Resolving dependencies... (20.8s)

Package operations: 65 installs, 0 updates, 0 removals

  • Installing certifi (2024.2.2)
  • Installing charset-normalizer (3.3.2)
  • Installing idna (3.7)
  • Installing markupsafe (2.1.5)
  • Installing urllib3 (2.2.1)
  • Installing alabaster (0.7.16)
  • Installing babel (2.14.0)
  • Installing docutils (0.20.1)
  • Installing imagesize (1.4.1)
  • Installing jinja2 (3.1.3)
  • Installing mdurl (0.1.2)
  • Installing packaging (24.0)
  • Installing pycparser (2.22)
  • Installing pygments (2.17.2)
  • Installing requests (2.31.0)
  • Installing six (1.16.0)
  • Installing snowballstemmer (2.2.0)
  • Installing sphinxcontrib-applehelp (1.0.8)
  • Installing sphinxcontrib-devhelp (1.0.6)
  • Installing sphinxcontrib-htmlhelp (2.0.5)
  • Installing sphinxcontrib-jsmath (1.0.1)
  • Installing sphinxcontrib-qthelp (1.0.7)
  • Installing sphinxcontrib-serializinghtml (1.1.10)
  • Installing asttokens (2.4.1)
  • Installing cffi (1.16.0)
  • Installing executing (2.0.1)
  • Installing markdown-it-py (3.0.0)
  • Installing parso (0.8.4)
  • Installing pbr (6.0.0)
  • Installing ptyprocess (0.7.0)
  • Installing pure-eval (0.2.2)
  • Installing setuptools (69.2.0)
  • Installing sphinx (7.2.6)
  • Installing traitlets (5.14.2)
  • Installing wcwidth (0.2.13)
  • Installing colorama (0.4.6)
  • Installing cryptography (42.0.5)
  • Installing decorator (5.1.1)
  • Installing dnspython (2.6.1)
  • Installing matplotlib-inline (0.1.6)
  • Installing nodeenv (1.8.0)
  • Installing pexpect (4.9.0)
  • Installing pluggy (1.4.0)
  • Installing prompt-toolkit (3.0.43)
  • Installing pyyaml (6.0.1)
  • Installing rich (13.7.1)
  • Installing sphinxcontrib-jquery (4.1)
  • Installing stack-data (0.6.3)
  • Installing stevedore (5.2.0)
  • Installing iniconfig (2.0.0)
  • Installing jedi (0.19.1)
  • Installing bandit (1.7.8)
  • Installing blessings (1.7)
  • Installing coverage (7.4.4)
  • Installing email-validator (2.1.1)
  • Installing icecream (2.1.3)
  • Installing ipython (8.23.0)
  • Installing pyjwt (2.8.0)
  • Installing pytest (8.1.1)
  • Installing result (0.16.1)
  • Installing ruff (0.3.7)
  • Installing semantic-version (2.10.0)
  • Installing sphinx-rtd-theme (2.0.0)
  • Installing pydantic (2.7.0)
  • Installing pyright (1.1.358)

Writing lock file

Hope that helps.

Also, just to note, if I lock the Pydantic dependency in pyproject.toml to 2.6.4, i.e. change:
pydantic = "^2.6.4"

to:
pydantic = "2.6.4"

everything starts working again.

Ah, interesting, my Poetry install is downrev'd (1.7.1) -- let me update that to current and see if things resolve.

Nope, same behavior -- let me do a test project with only Pydantic as a dependency and see what happens.

Fascinating. So with a new project, with current version Poetry and this pyproject.toml file:
$ cat pyproject.toml
[tool.poetry]
name = "miketest"
version = "0.1.0"
description = ""
authors = ["Mike Simpson<EMAIL_ADDRESS>readme = "README.md"

[tool.poetry.dependencies]
python = "^3.12"
pydantic = "^2.7.0"

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

I see these dependency pulls when I run poetry install:
$ poetry install
Creating virtualenv miketest in .venv
Updating dependencies
Resolving dependencies... (0.9s)

Package operations: 4 installs, 0 updates, 0 removals

  - Installing typing-extensions (4.11.0)
  - Installing annotated-types (0.6.0)
  - Installing pydantic-core (2.18.1)
  - Installing pydantic (2.7.0)

Writing lock file

Installing the current project: miketest (0.1.0)

So it's definitely pulling the deps that appear to be missing in my earlier runs in my main project ... let me experiment with adding the other deps in one at a time and see if there's some point where the behavior suddenly alters and drops those Pydantic deps, maybe there's some weird version conflict that's causing the problem.

Also, just to document it for posterity, I didn't know about this command, and it's great:
$ poetry show --tree
blessings 1.7 A thin, practical wrapper around terminal coloring, styling, and positioning
└── six *
email-validator 2.1.1 A robust email address syntax and deliverability validation library.
├── dnspython >=2.0.0
└── idna >=2.0.0
pydantic 2.7.0 Data validation using Python type hints
├── annotated-types >=0.4.0
├── pydantic-core 2.18.1
│   └── typing-extensions >=4.6.0,<4.7.0 || >4.7.0
└── typing-extensions >=4.6.1

Sigh. Found the problem -- well, at least I found the source of the problem for my originally-reported issue. We're using AWS CodeArtifact as our upstream default package source, with PyPi as the fallback, e.g. this configuration in pyproject.toml:
[[tool.poetry.source]]
name = "aws-codeartifact-i2ns-pypi"
url = "https://{**** elided ***}.us-east-1.amazonaws.com/pypi/pypi/simple/"
priority = "primary"

[[tool.poetry.source]]
name = "PyPI"
priority = "primary"
priority = "supplemental"

That CodeArtifact repo is supposed to act as a transparent proxy back to PyPi, but the behavior I'm seeing is that with the above config, Pydantic 2.7.0 gets pulled but none of its dependencies. If I change the config so that we're pulling straight from PyPi (rather than pulling from PyPi "through" CodeArtifact) everything works correctly -- Pydantic 2.7.0 gets pulled along with its expected deps.
So for my case at least, this is an issue with the CodeArtifact cache of the upstream PyPi artifacts, I think.
@fabio-gallotti-illimity Are you doing something similar by any chance? If it's the same issue for both of us, I think we can blame AWS and close out this ticket, with my apologies to @Viicos for wasting his time. :)

Great investigation @mgsimpson-i2.
I wonder if CodeArtifact somehow ignores dependencies of packages that come from pypi (in this case because they're not yet cached in code artifact) but use dependencies from code artifact to, to guard against some kind of supply chain attack? Did you look at the hashes in the poetry.lock file?
I'll close this, but happy to reopen if anyone finds an issue with this project.

Just following up for the sake of completeness -- with PyPi configured as "primary", and CodeArtifact configured as "supplemental" -- so preferring direct PyPi pulls for dependencies, the poetry.lock entry for Pydantic looks like this:
[[package]]
name = "pydantic"
version = "2.7.0"
description = "Data validation using Python type hints"
optional = false
python-versions = ">=3.8"
files = [
    {file = "pydantic-2.7.0-py3-none-any.whl", hash = "sha256:9dee74a271705f14f9a1567671d144a851c675b072736f0a7b2608fd9e495352"},
    {file = "pydantic-2.7.0.tar.gz", hash = "sha256:b5ecdd42262ca2462e2624793551e80911a1e989f462910bb81aef974b4bb383"},
]

Flipping the config (preferring pulling through CodeArtifact for all deps) that entry changes to this:
[[package]]
name = "pydantic"
version = "2.7.0"
description = ""
optional = false
python-versions = "*"
files = [
    {file = "pydantic-2.7.0-py3-none-any.whl", hash = "sha256:9dee74a271705f14f9a1567671d144a851c675b072736f0a7b2608fd9e495352"},
    {file = "pydantic-2.7.0.tar.gz", hash = "sha256:b5ecdd42262ca2462e2624793551e80911a1e989f462910bb81aef974b4bb383"},
]

Interestingly, the hashes don't change, but some of the metadata fields are missing.
So, best working theory right now is that AWS CodeArtifact has a bad/partial/something version of Pydantic 2.7.0 cached in their PyPi mirror -- I tried to clear out my own CodeArtifact cache but it looks like AWS is maybe using a single big backing cache for all of those PyPi pulls, which would make sense in terms of efficiency.
My hope is that sometime in the next few days CodeArtifact will do some kind of TTL refresh and pull a good copy; failing that, for my project at least it's no real hassle to use PyPi as primary and CodeArtifact as supplemental for our locally-built artifacts.

with my apologies to @Viicos for wasting his time. :)

No time wasted, glad this is now documented as an issue. I'd be interested to know: what happens if you try installing these without Poetry? (e.g. pip or uv)

Testing in a new virtual environment, pip seems to run clean and as expected (dependencies get pulled). So there's a chance that this is some weird interaction between Poetry and AWS CodeArtifact; but the caveat is that I don't normally point pip at CodeArtifact -- I just tried it for the first time using some boilerplate from the AWS docs -- so I may not be doing an apples-to-apples comparison.

Hum yeah you have options in pip with --index-url and --extra-index-url I believe, but hard to tell if the same behavior applies with Poetry :/

Turns out this is a Poetry issue: https://github.com/python-poetry/poetry/issues/9301


Custom implementation of timescale-paralllel-copy

We can fork/copy the repo https://github.com/timescale/timescaledb-parallel-copy for fitting zindexing needs, these are:

allow to use as library instead of an independent app
should be possible to pass csv/binary from memory instead from only from csv files

should be possible to pass csv/binary from memory instead from only from csv files

https://github.com/timescale/timescaledb-parallel-copy/issues/35#issuecomment-592196000
I've just found this in the docs/an issue:
"-file string
File to read from rather than stdin"


Update to flutter_svg 2.0.0+1

Is there any provision for updating the library to comply with flutter_svg 2.0.1?

Same issue here, thanks!

Had to wait until flutter_svg fixes their caching bugs in the 2.0.0 release. The coast is clear now.
I've upgraded all dependencies now.