youtube videos doesn't show

playing youtube videos just shows black screen in VideoPlayerView

@bkmalkoc we are adding this functionality! please watch-out here for next update!

very useful lib, thanks for all the effort, Do you have any idea how long (roughly) it will take to add YouTube support, please?

@bastami82 it's in testing stage, we might publish next week!

@bastami82 @bkmalkoc It's added in new library version! please checkout

@ctn-mobile-innovation
thanks, for adding Youtube support so quick, however it crashes on iOS :
 kotlin.native.internal.IrLinkageError: Function 'UIKitView' can not be called: No function found for symbol 'androidx.compose.ui.interop/UIKitView|UIKitView(kotlin.Function0<0:0>;androidx.compose.ui.Modifier;kotlin.Function1<0:0,kotlin.Unit>?;androidx.compose.ui.graphics.Color;kotlin.Function1<0:0,kotlin.Unit>?;kotlin.Function2<0:0,kotlinx.cinterop.CValue<platform.CoreGraphics.CGRect>,kotlin.Unit>?;kotlin.Boolean;androidx.compose.runtime.Composer?;kotlin.Int;kotlin.Int){0§<platform.UIKit.UIView>}[0]'

@bastami82  Let us check this and get back to you

@bastami82 Please share which compose and kotlin version you are using

@bastami82 Please share which compose and kotlin version you are using

compose-plugin = "1.6.11"
jetbrainsCompose = { id = "org.jetbrains.compose", version.ref = "compose-plugin" }
kotlin = "2.0.0"
kotlinMultiplatform = { id = "org.jetbrains.kotlin.multiplatform", version.ref = "kotlin" }
using
simulator iPhone 15 | iOS 17.5

@ctn-mobile-innovation
also got the same error when used
compose-plugin = "1.7.0-alpha03"
kotlin = "2.0.20"

compose-plugin = "1.6.11"
jetbrainsCompose = { id = "org.jetbrains.compose", version.ref = "compose-plugin" }
kotlin = "2.0.0"
kotlinMultiplatform = { id = "org.jetbrains.kotlin.multiplatform", version.ref = "kotlin" }
using
simulator iPhone 15 | iOS 17.5
@bastami82 we have used same version for demo, demo working fine in your device? or while integrating in your project causing this issue?

@ctn-mobile-innovation
I was away yesterday, looking into this as you suggested, the Video player works, but when I import the lib in my project, it does not work on iOS , my best guess is maybe some iOS dependencies are missing when import the project,  I guess it would be nice if we keep the sample/Demo app separate from the main lib project, so this way demo app will pull player lib same as a real use-case , when I compared the two projects , my project only has 4 chaintech libs as externals whereas yours has a few more specially sdp-ssp-compose-multiplatform-iosarm64:1.0.1 and isosimulatorarm64:1.0.1 and isosx64:1.0.1 , please see this comparison in a screenshot (left is what I have and on the right is your dependencies )

@bastami82 We have tried creating separate demo too, it's working fine, can you share your .toml and .gradle file?
Configuration used for demo ->
kotlin = "2.0.20"
compose = "1.6.11"
multiplatform = { id = "org.jetbrains.kotlin.multiplatform", version.ref = "kotlin" }
compose-compiler = { id = "org.jetbrains.kotlin.plugin.compose", version.ref = "kotlin" }
compose = { id = "org.jetbrains.compose", version.ref = "compose" }

here you are:
`[versions]
agp = "8.5.2"
android-compileSdk = "34"
android-minSdk = "24"
android-targetSdk = "34"
androidx-activityCompose = "1.9.2"
compose = "1.6.11"
kotlin = "2.0.20"
ktor = "2.3.12"
landscpapist = "2.3.6"
media-player = "1.0.19"
lifecycleViewModel = "2.8.5"
koin = "3.6.0-alpha3"
koinCompose = "3.6.0-alpha3"
koinComposeMultiplatform = "1.2.0-alpha3"
navigationCompose = "2.8.0-alpha08"
lifecycle-runtime-compose = "2.8.5"
runtimeAndroid = "1.7.1"
serialization = "1.6.3"
[libraries]
androidx-activity-compose = { module = "androidx.activity:activity-compose", version.ref = "androidx-activityCompose" }
media-player = { module = "network.chaintech:compose-multiplatform-media-player", version.ref = "media-player" }
lifecycle-viewmodel = { module = "androidx.lifecycle:lifecycle-viewmodel", version.ref = "lifecycleViewModel" }
koin-android = { module = "io.insert-koin:koin-android", version.ref = "koin" }
koin-androidx-compose = { module = "io.insert-koin:koin-androidx-compose", version.ref = "koinCompose" }
koin-core = { module = "io.insert-koin:koin-core", version.ref = "koin" }
koin-compose = { module = "io.insert-koin:koin-compose", version.ref = "koinComposeMultiplatform" }
navigation-compose = { module = "org.jetbrains.androidx.navigation:navigation-compose", version.ref = "navigationCompose" }
kotlinx-serialization-json = { module = "org.jetbrains.kotlinx:kotlinx-serialization-json", version.ref = "serialization"}
androidx-lifecycle-runtime-compose = { group = "androidx.lifecycle", name = "lifecycle-runtime-compose", version.ref = "lifecycle-runtime-compose" }
androidx-runtime-android = { group = "androidx.compose.runtime", name = "runtime-android", version.ref = "runtimeAndroid" }
ktor-client-core = { module = "io.ktor:ktor-client-core", version.ref = "ktor" }
ktor-client-okhttp = { module = "io.ktor:ktor-client-okhttp", version.ref = "ktor" }
ktor-client-darwin = { module = "io.ktor:ktor-client-darwin", version.ref = "ktor" }
ktor-client-content-negotiation = { module = "io.ktor:ktor-client-content-negotiation", version.ref = "ktor" }
ktor-client-logging = { module = "io.ktor:ktor-client-logging", version.ref = "ktor" }
ktor-serialization-kotlinx-json = { module = "io.ktor:ktor-serialization-kotlinx-json", version.ref = "ktor" }
ktor-client-auth = { module = "io.ktor:ktor-client-auth", version.ref = "ktor" }
landscapist = { group = "com.github.skydoves", name = "landscapist-coil3", version.ref = "landscpapist" }
[plugins]
androidApplication = { id = "com.android.application", version.ref = "agp" }
androidLibrary = { id = "com.android.library", version.ref = "agp" }
jetbrainsCompose = { id = "org.jetbrains.compose", version.ref = "compose" }
compose-compiler = { id = "org.jetbrains.kotlin.plugin.compose", version.ref = "kotlin" }
kotlinMultiplatform = { id = "org.jetbrains.kotlin.multiplatform", version.ref = "kotlin" }
kotlin-serialization = { id = "org.jetbrains.kotlin.plugin.serialization", version.ref = "kotlin" }
[bundles]
ktor = [
"ktor-client-core",
"ktor-client-content-negotiation",
"ktor-client-auth",
"ktor-client-logging",
"ktor-serialization-kotlinx-json",
]`
FYI, I get this error when try to play on ios
kotlin.native.internal.IrLinkageError: Function 'UIKitView' can not be called: No function found for symbol 'androidx.compose.ui.interop/UIKitView|UIKitView(kotlin.Function0<0:0>;androidx.compose.ui.Modifier;kotlin.Function1<0:0,kotlin.Unit>?;androidx.compose.ui.graphics.Color;kotlin.Function1<0:0,kotlin.Unit>?;kotlin.Function2<0:0,kotlinx.cinterop.CValue<platform.CoreGraphics.CGRect>,kotlin.Unit>?;kotlin.Boolean;androidx.compose.runtime.Composer?;kotlin.Int;kotlin.Int){0§<platform.UIKit.UIView>}[0]'
Which is also shared by other people here  https://github.com/Chaintech-Network/ComposeMultiplatformMediaPlayer/issues/20  ,
what I could suggest is, if you can share your working sample app with me, I can compare it and get back to you, Thanks

i saw another library having the same problem here and there it got fixed, so might be able to check from there how they did it. not sure if its the same problem

It's Fixed in latest version

Thank you I can confirm it does work nicely.


Cannot launch gui apps with 2.0.12

Windows Version
11
WSL Version
2.0.12
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
No response
Distro Version
Fedora
Other Software
No response
Repro Steps
I usually launch GUI apps from the windows start menu but after upgrading to 2.0.12, the apps wouldn't start.  wsl --list --running showed the distro running for a few seconds and then it shutdown.  I am able to start WSL from the command line but when I try to launch a gui app such as konsole, I get this error:
qt.qpa.xcb: could not connect to display :0

qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "" even though it was found.

This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.

Available platform plugins are: eglfs, linuxfb, minimal, minimalegl, offscreen, vnc, wayland-egl, wayland, wayland-xcomposite-egl, wayland-xcomposite-glx, xcb.

Aborted (core dump)

There were no issues with 2.0.11
Expected Behavior
GUI apps start as usual
Actual Behavior
GUI apps cannot be launched
Diagnostic Logs
No response

Unfortunately, that didn't work.  :-(

I confirm also here same behaviour.
Temporarily fixed with this command as root (Does not survive at reboot):
rm -rf /tmp/.X11-unix &&  ln -s /mnt/wslg/.X11-unix /tmp/

Ok, I see.  The "does not survive reboot" did not click when I first tried this.
Hopefully a fix comes soon as that is somewhat inconvenient to have to do so I will stick with 2.0.9 for now.
Thanks

Disabling systemd in /etc/wsl.conf is a permanent workaround, but not an option for me

This issue was moved over from WSL but I'm wonder if that was appropriate.  If I'm not mistaken, the WSLg version has not changed since March so it looks like this issue was introduced in WSL not WSLg.  I'm wondering if https://github.com/microsoft/WSL/issues/10818 is the cause as that looks related.

Does your system have a /run/tmpfiles.d/x11.conf file? If not, does the /tmp/.X11-unix folder/symlink get fixed if you create your own empty /etc/tmpfiles.d/x11.conf file? (Reboot of wsl required)

Not sure if this survives a WSL virtual machine reboot.

I'm using Arch Linux and have the same issue. I created a fix for this that runs those commands as a systemd service (iirc Fedora comes with systemd as well, but regardless of your distro, as long as it uses systemd, it should work). This means that the service will run those commands on boot.
This is under the assumption that running those commands does not lead to any unforeseen side effects (I haven't seen any so far, from my understanding, those commands are fine to run...).
1. Creating the systemd service file
Start by creating a systemd service file at the following location, /etc/systemd/system/. You can name the service file whatever you like, however, keep in mind that the name of the file must have a suffix of service. Then open the file with your preferred text editor. In my case, I ran the following:
sudo nano /etc/systemd/system/x11-symlink.service

Note: nano creates a file if it doesn't exist by default, if your text editor doesn't do this, you'll need to create the file first with touch.
2. Write the scripts for the service
To preface this part, I personally never had anything mounted on /tmp/.X11-unix, so running the umount command for that would result in an error. I would recommend running this command on your terminal just to see if you actually need it or not.
In the service file, add the following contents to it:
[Unit]
Description=Setup X11 Symlink

[Service]
Type=oneshot
ExecStartPre=/bin/umount /tmp/.X11-unix
ExecStartPre=/bin/rm -rf /tmp/.X11-unix
ExecStart=/bin/ln -s /mnt/wslg/.X11-unix /tmp/.X11-unix

[Install]
WantedBy=multi-user.target

If you do not need to run the umount command, you should remove it, otherwise the service will fail and exit.
Now you can save and exit the file.
3. Enable the service
To enable the service, we first need to reload systemd to recognize the new service:
sudo systemctl daemon-reload

Then we can enable the service on boot:
sudo systemctl enable x11-symlink.service

Note: the name of the service would be the name of the file you created earlier, make sure this matches.
At this point, you can restart WSL and verify if the service worked by running:
sudo systemctl status x11-symlink.service

If you want to manually start the service without having to restart WSL, you can run the following directly:
sudo systemctl start x11-symlink.service

To verify that it did work, run ls /tmp/.X11-unix and you should see something like X0 or X1, indicating your display.
Hope this helps, lmk if you need extra clarification.

Does your system have a /run/tmpfiles.d/x11.conf file? If not, does the /tmp/.X11-unix folder/symlink get fixed if you create your own empty /etc/tmpfiles.d/x11.conf file? (Reboot of wsl required)

Yes, it does.  Interestingly, the file has a note:

This file is generated by WSL to prevent systemd-tmpfiles from removing /tmp/.X11-unix during boot.

I'm still running 2.0.9 so I don't know if it's still there in later versions.

It seems that with 2.0.15, /run/tmpfiles.d/x11.conf is not created.  Maybe that's the root cause of this problem?

It seems that with 2.0.15, /run/tmpfiles.d/x11.conf is not created. Maybe that's the root cause of this problem?

It definitely feels like the issue you referenced, https://github.com/microsoft/WSL/issues/10818, is related. The fix for that issue, which should disable the above mentioned override file (and possibly further logic) for guiApplications=false actually disabled it unconditionally. It's a shame that wsl part doesn't appear to be open source, so we're forced to poke the Devs through GitHub issues. And if this type of problem goes unnoticed, I get the feeling there's not much automated integration testing involved in the release process.
Does the issue go away on 2.0.15 if you create an empty /etc/tmpfiles.d/x11.conf ? (Which would simply add the override manually instead of thorough wsl)

Unfortunately, the issue doesn't go away after adding an empty /etc/tmpfiles.d/x11.conf.
Maybe this issue would get more attention if it were in wsl/issues where I originally opened it.  As I mentioned previously, the only changes when this issue appeared were to wsl not wslg.

Pretty sure this issue is related to WSL <IP_ADDRESS> release, maybe it's a regression of the fix of microsoft/WSL#10818, could you please move the issue back to microsoft/WSL? @OneBlue

Not fixed in <IP_ADDRESS> + 1.0.60.
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-2
WSLg version: 1.0.60
MSRDC version: 1.2.5105
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.26020.1000

Not fixed in <IP_ADDRESS> + 1.0.60.

same here

I did try to raise the visibility of this issue by reporting it again in the WSL repo (https://github.com/microsoft/WSL/issues/11064) but the ticket got closed because it didn't have "steps to reproduce" even though I pointed to this ticket. :-(

Still there with <IP_ADDRESS>

Is there a specific reason that wslg doesn't seem to honor the [automount] root=/path/here/ config in /etc/wsl.conf? I have to recursively delete /tmp/.X11-unix and ln -s the proper wslg mount point every boot to get graphical programs to run.
Specifically, /mnt/ is not the default mount point in SUSE distros. /run/media/[user]/ is, so I override it as that in my SUSE Tumbleweed WSL2. Having the ability to specify a mount point then not honoring it if used seems fairly strange.

Same here.
Not clear if this behaviour depends on the installed distro (I use arch) or the problem is for everyone (Seems strange that anyone noticed it).

It's not obviously distro-related as I am using Fedora.

I'm going to repeat what I said earlier, this issue should be moved back to WSL  where I originally created it, given that the problem was introduced with a WSL release that did not include any WSLg changes.

It's not obviously distro-related as I am using Fedora.

I just updated another computer from 2.0.9 to 2.1.3, with Ubuntu 22.04... and it works without any workaround!
Here I can see /tmp/.X11-unix/X0 as a file; it is not a link to /mnt/wslg/.X11-unix/X0 (That is present). The two files have the same date/size.
I agree with your last post. I had the same behaviour starting from a specific WSL release that did not include WSLg changes, so it must be something in wsl, or a default configuration changed

I just updated another computer from 2.0.9 to 2.1.3, with Ubuntu 22.04... and it works without any workaround! Here I can see /tmp/.X11-unix/X0 as a file; it is not a link to /mnt/wslg/.X11-unix/X0 (That is present). The two files have the same date/size.

Obviously those are 'virtual' file; both path are mounted:
none on /mnt/wslg type tmpfs (rw,relatime)
none on /tmp/.X11-unix type tmpfs (ro,relatime)
I will compare this with the other non-working installation, with Arch

Still there with <IP_ADDRESS>

It seems that this problem has not been fixed in the latest version, but the systemctl method is very effective, thank you🙏

same issue on my side
C:\Users\ehongya>wsl --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-1
WSLg version: 1.0.59
MSRDC version: 1.2.4677
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.19045.4046

I had the same issue whenever i open any application, but i found a fix and wanted to share it.
If like me you are using Arch Linux (may work on other distros that have the same problem) i found that the wayland socket isn't symlinked automatically, and combined with @rahulshamkuwar's solution, the problem does not occur anymore.
$ firefox

[656] Wayland Proxy [0x7feae4075450] Error: CheckWaylandDisplay(): Failed to connect to Wayland display '/run/user/1000//wayland-0' error: No such file or directory
Error: we don't have any display, WAYLAND_DISPLAY='wayland-0' DISPLAY=':0'

To fix this, add the following lines to your ~/.profile to recreate the symbolic links after every restart. The symbolic links will not survive a restart, that's why i use ~/.profile.
$ cat ~/.profile

if [[ ! -e "${XDG_RUNTIME_DIR}/wayland-0" ]]; then
    ln --symbolic /mnt/wslg/runtime-dir/wayland-0 "${XDG_RUNTIME_DIR}"/wayland-0
    ln --symbolic /mnt/wslg/runtime-dir/wayland-0.lock "${XDG_RUNTIME_DIR}"/wayland-0.lock
fi

Again, if this does not work, give @rahulshamkuwar's solution a try, that fixed some other GUI programs not opening for me.

Here's what's going on for at least some of the cases, when you are running systemd:

Way early in the process, WSL(g) mounts a copy of /mnt/wslg/.X11-unix at /tmp/.X11-unix
systemd mounts a tmpfs at /tmp, thus hiding /tmp/.X11-unix (/usr/lib/systmd/system/tmp.mount).
systemd-tmpfiles then proceeds to create a /tmp/.X11-unix directory (/usr/lib/tmpfiles.d/x11.conf)

A simple workaround then is to disable the mount of the tmpfs on /tmp with systemctl mask tmp.mount. systemd-tmpfiles appears to leave WSLg's existing /tmp/.X11-unix alone.
A nicer solution from the WSL end would be when systemd is configured, instead of the early mystery mount at /tmp/.X11-unix, would be to inject a systemd rule for the mount carefully handling the dependencies so that the mount happened after the tmpfs was created on /tmp.
https://github.com/microsoft/WSL/discussions/8908 was what guided me to understanding what was going on and is making a similar if not identical suggestion...

Here's what's going on for at least some of the cases, when you are running systemd:

Way early in the process, WSL(g) mounts a copy of /mnt/wslg/.X11-unix at /tmp/.X11-unix
systemd mounts a tmpfs at /tmp, thus hiding /tmp/.X11-unix (/usr/lib/systmd/system/tmp.mount).
systemd-tmpfiles then proceeds to create a /tmp/.X11-unix directory (/usr/lib/tmpfiles.d/x11.conf)

A simple workaround then is to disable the mount of the tmpfs on /tmp with systemctl mask tmp.mount. systemd-tmpfiles appears to leave WSLg's existing /tmp/.X11-unix alone.
A nicer solution from the WSL end would be when systemd is configured, instead of the early mystery mount at /tmp/.X11-unix, would be to inject a systemd rule for the mount carefully handling the dependencies so that the mount happened after the tmpfs was created on /tmp.
https://github.com/microsoft/WSL/discussions/8908 was what guided me to understanding what was going on and is making a similar if not identical suggestion...

Totally agree on this one. I really wish the developers could have a closer look at this soon!

After almost two years, I've pretty much given up any hope that this issue will be resolved.

yeah, this seems to be the origin of my several recent cases... Since disabling systemd is not favorable for me, the only workaround would be manually creating symlinks for /tmp/.X11-unix.
BTW, Ubuntu wsl seems not to have some issues with this, did they have some special tweaks?

BTW, Ubuntu wsl seems not to have some issues with this, did they have some special tweaks?

I see this behavior on Ubuntu 24.04, with some gui apps taking 10-20 secs to launch (if ever, sometimes I have to try again).

@wang1zhen
Did this not work for you?
systemctl mask tmp.mount

@wang1zhen Did this not work for you? systemctl mask tmp.mount

Yes, it worked. Yet I think there is tmp.mount service for some specific reasons right?

@wang1zhen

Yes, it worked. Yet I think there is tmp.mount service for some specific reasons right?

Normally, yeah, but I think ms wslg image is taking care of it already.
https://github.com/microsoft/WSL/issues/11857

The workaround that allows keeping tmp.mount is described in https://github.com/microsoft/wslg/issues/1156#issuecomment-1876266025

A better workaround in this comment (configure the existing tmp.mount service rather than creating a new one)

Oddly I can run firefox but not code-oss or xclock.
❯ wsl.exe --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-2
WSLg version: 1.0.65
MSRDC version: 1.2.5620
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.26100.1-240331-1435.ge-release
Windows version: 10.0.26100.2161
❯ ls -la /tmp/.X11-unix
total 0
drwxrwxrwt  2 root root  40 Oct 31 16:07 .
drwxrwxrwt 11 root root 220 Oct 31 16:18 ..

Wonder if systemd+wslg will ever work without manual workarounds.

Oddly I can run firefox but not code-oss or xclock.
❯ wsl.exe --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-2
WSLg version: 1.0.65
MSRDC version: 1.2.5620
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.26100.1-240331-1435.ge-release
Windows version: 10.0.26100.2161
❯ ls -la /tmp/.X11-unix
total 0
drwxrwxrwt  2 root root  40 Oct 31 16:07 .
drwxrwxrwt 11 root root 220 Oct 31 16:18 ..

Wonder if systemd+wslg will ever work without manual workarounds.

All my KDE apps are now failing, if systemd is enabled I can get it to work, but I can no longer access btrfs mounted drives through Windows, not an acceptable trade-off.


Add a short sentence or two to the opportunities page about - 'things coming down the pipe'

This could be something like "we are currently talking to clients about opportunities looking for skills in ....."
Or more vague "we expect to see other opportunities looking for skills in: ...."

[x] draft text
[ ] translate
[ ] post on site

Draft text:

As we continue our search for internal partners to publish more opportunities, our discussions have lead us see that the following programming languages may be in demand: Python, Javascript, TypeScript, Java, C#
Also, teams working on frontend development have mentioned using the following frameworks: Spring, React, Vue

Yes! This is awesome too @gcharest!  Will send this to translation. with the Q&A text. Thank you!

French text is here (scroll to the bottom of the document): https://014gc.sharepoint.com/:w:/r/sites/GCDevEx/Shared Documents/Build and planning/TR0362 - Website content for translation_FR.docx?d=w00805be86ea5465aa5702f6e71246961&csf=1&web=1&e=r0emwq


arch:xtensa:toolchain: add -Wno-atmoic-alignment flags

Summary
to avoid the compiler warning
Impact
xtensa
Testing
Pass CI

@xiaoxiang781216 Could you please share the motivation for disabling this warning? In which scenario does it manifest?
From this Clang patchset, this warning seems useful:
https://reviews.llvm.org/D45319

@xiaoxiang781216 Could you please share the motivation for disabling this warning? In which scenario does it manifest?

@gustavonihei here is the warning generated by Candance's clang:
[2023-01-19T07:31:49.594Z] media_proxy.c:498:9: error: large atomic operation may incur significant performance penalty [-Werror,-Watomic-alignment]
[2023-01-19T07:31:49.594Z]     if (atomic_load(&priv->refs) == 1 && priv->socket) {
[2023-01-19T07:31:49.594Z]         ^

Here is the related struct definition:
typedef struct MediaProxyPriv {
    void                  *proxy;
    char                  *cpu;
    uint64_t              handle;
    atomic_int            refs;
    int                   socket;
    void                  *cookie;
    media_event_callback  event;
} MediaProxyPriv;

The warning looks wrong since refs should be naturally alignment.

From this Clang patchset, this warning seems useful: https://reviews.llvm.org/D45319

If an atomic variable is misaligned Clang will emit accesses to it as a libcall. These calls are likely to be slow and involve locks; they are almost certainly not what the developer intended.

I honestly feel that adding this flag to this broad scope is too overkill, even because the occurrence is not even in upstream code. And this is a kind of warning that shouldn't be overlooked.
Considering the motivation of bypassing a possible false positive from Clang, I would suggest to reduce the scope of this flag to the minimum possible.

@xiaoxiang781216 could you please point me to some good article describing atomic variables and accessing it? I need to educate myself as I was not expecting atomic_load to be used with atomic_int, but was expecting simple priv->refs == 1 if refs is atomic_int.

Here https://en.cppreference.com/w/c/atomic/atomic_load

Seems to be the same as https://github.com/RIOT-OS/RIOT/issues/12048

Maybe that disabling can be done selectively only for Clang?

Ok, I move them to xcc section, @pkarashchenko


Move to Rust 2018 edition

After the preparation done in #83, not much work was left to be done.
Based on #87, that needs to be merged first.

@rnestler ready for review now :slightly_smiling_face:

@dbrgn ping.

awww i rebased it but it looks like I don't have access to push :/ I'm in the spaceapi-community group already, though :/

awww i rebased it but it looks like I don't have access to push :/ I'm in the spaceapi-community group already, though :/

I think that's because it is a branch @dbrgn created? You could just push it to a new branch then.

Rebased!

@g5pw finally :slightly_smiling_face:


Make metrics compatible with official K6 Grafana dashboard

First off, thanks for the extension!
What would be needed to support the metrics used in the official k6 Grafana dashboard?
I was able to get VUs metrics to show up with the following env var K6_PUSHGATEWAY_NAMESPACE=k6, but it seems like there are some aggregated metrics with _total suffix not being emitted by this extension.

It seems the xk6-output-prometheus-remote extension does emit the correct metrics if you need a reference.

@wshirey
Hi, thank you for your valuable suggestion! (And sorry for my belated response...)
Allow me some time to consider how to take in this capability.
I'll update you soon. Stay tuned 😉

Hey @martymarron , sorry for reviving this but were you able to take a look at this? Facing a similar issue of metrics not showing up in Grafana.

@thomas-westfall
Sure, sorry for my belated response 🙇

@thomas-westfall @wshirey guys did your issue solved? I was planning to use this extension to publish my k6 results to pushgateway using this extension.


On experimental free-threaded CPython build, import whenever fails

Not urgent to fix, but importing on free-threaded build fails:
Python 3.13.0rc1 experimental free-threading build (main, Aug  7 2024, 15:09:54) [Clang 15.0.0 (clang-15<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import whenever
Traceback (most recent call last):
  File "<python-input-0>", line 1, in <module>
    import whenever
  File "/Users/arie/code/whenever/pysrc/whenever/__init__.py", line 2, in <module>
    from ._whenever import *
SystemError: init function of _whenever returned uninitialized object
>>>

Even after some experimentation, I can't find the cause. Even after removing the entire module contents, I still get the error. I'll have to do more digging.


Exception on doctor analysing step

Expected Behavior
No JSON parsing exception
Current Behavior
When I shutdown the app, the analysing is just failing... I start my app (it spawns multiples processes): clinic doctor -- npm start
[  ==] Analysing dataundefined:1
[{"pid":28671,"tid":28671,"ts":867159788650,"tts":4991138,"ph":"E","cat":"v8","name":"V8.Execute","dur":0,"tdur":0,"args":{"runtime-call-stats":{}}},{"pid":28671,"tid":28671,"ts":866991947586,"tts":11701,"ph":"M","cat":"__metadata","name":"process_name","dur":0,"tdur":0,"args":{"name":"node"}},{"pid":28671,"tid":28671,"ts":866991947589,"tts":11704,"ph":"M","cat":"__metadata","name":"version","dur":0,"tdur":0,"args":{"node":"10.15.2"}},{"pid":28671,"tid":28671,"ts":866991947590,"tts":11706,"ph":"M","cat":"__metadata","name":"thread_name","dur":0,"tdur":0,"args":{"name":"JavaScriptMainThread"}},{"pid":28671,"tid":28671,"ts":866991947611,"tts":11726,"ph":"M","cat":"__metadata","name":"node","dur":0,"tdur":0,"args":{"process":{"versions":{"http_parser":"2.8.0","node":"10.15.2","v8":"6.8.275.32-node.12","uv":"1.24.1","zlib":"1.2.11","ares":"1.14.0","modules":"64","nghttp2":"1.37.0","napi":"3","openssl":"1.1.1b"},"arch":"x64","platform":"linux","release":{"name":"node","lts":"Dubnium"}}}},{"pid":28671

SyntaxError: Unexpected token p in JSON at position 3698
    at JSON.parse (<anonymous>)
    at Parser._transform (/home/schroeterm/.npm-packages/lib/node_modules/clinic/node_modules/@nearform/trace-events-parser/index.js:41:34)
    at Parser.Transform._read (_stream_transform.js:190:10)
    at Parser.Transform._write (_stream_transform.js:178:12)
    at doWrite (_stream_writable.js:410:12)
    at writeOrBuffer (_stream_writable.js:394:5)
    at Parser.Writable.write (_stream_writable.js:294:11)
    at TraceEventDecoder._transform (/home/schroeterm/.npm-packages/lib/node_modules/clinic/node_modules/@nearform/doctor/format/trace-event-decoder.js:114:19)
    at TraceEventDecoder.Transform._read (_stream_transform.js:190:10)
    at TraceEventDecoder.Transform._write (_stream_transform.js:178:12)

Steps to Reproduce (for bugs)
I can't provide this app, sorry... But maybe I can provide more logs, but where and how ?
Sample upload
N/A
Environment
$ clinic -v
v4.0.3
$ clinic doctor -v
v4.0.4

$ node -v
v10.15.2
$ npm -v
6.9.0

$ uname -a
Linux alienium 4.19.0-4-amd64 #1 SMP Debian 4.19.28-2 (2019-03-15) x86_64 GNU/Linux

Same issue here

Multiple processes are not really supported at the moment. The instrumentation code will output JSON to stdout from every process, and it'll interleave and produce an invalid stream. It might be possible to support in the future but I'm not yet sure what it would take.

@goto-bus-stop that was exactly my issue, thanks! removing child_process from main process solved the issue.


Blacklist earlgrey_verilator_01_05_21 for msys2 platform

Codecov Report

Merging #2632 (3e59da6) into master (f35a7ee) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2632   +/-   ##
=======================================
  Coverage   27.99%   27.99%
=======================================
  Files         191      191
  Lines       33877    33877
=======================================
  Hits         9485     9485
  Misses      24392    24392

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f35a7ee...3e59da6. Read the comment docs.


Enable Enabled parameter

When the project configured as:
# .rubocop.yml

require:
  - rubocop-rspec

AllCops:
  DisabledByDefault: true

RSpec:
  Enabled: true

an annoying warning pops up:
Warning: RSpec does not support Enabled parameter.

Supported parameters are:

  - Include
  - Language

Enabled: true does, however, what it is supposed to do, and RSpec cops are inspecting the source.
This is not the case with e.g. rubocop-rails. Apparently, this is due to our RSpec DSL configuration, as rubocop-rails doesn't have any Rails: YAML configuration in their config/default.yml.
Enabling Enabled: true in our config/default.yml removes the warning.
Wondering if it makes sense to skip this warning for Enabled/Include/Exclude and other universal extension parameters in RuboCop itself? WDYT?

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[-] Added tests.
[-] Updated documentation.
[x] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

Wondering if it makes sense to skip this warning for Enabled/Include/Exclude and other universal extension parameters in RuboCop itself? WDYT?

Yes, we should support rubocop's configuration
The tricky part however, with explicitly stating the values, is whether it would work as expected in all the cases. E.g. if you set AllCops:DisabledByDefault to true, and do not add enabling for rubocop-rspec, will they respect the decision, when our own configuration has Enabled: true :/ (with Enabled: false, would they respect being enabled by default)?

Wondering if it makes sense to skip this warning for Enabled/Include/Exclude and other universal extension parameters in RuboCop itself? WDYT?

Yes, we should support rubocop's configuration
The tricky part however, with explicitly stating the values, is whether it would work as expected in all the cases. E.g. if you set AllCops:DisabledByDefault to true, and do not add enabling for rubocop-rspec, will they respect the decision, when our own configuration has Enabled: true :/ (with Enabled: false, would they respect being enabled by default)?

After giving it another thought, I'm wondering if we will be enabling all RSpec cops this way for those who have DisabledByDefault: true. That would make it inconsistent with rubocop-rails. I'll test this in the next few days.
If so, I'll close this and will try to address this in RuboCop.

After giving it another thought, I'm wondering if we will be enabling all RSpec cops this way for those who have DisabledByDefault: true. That would make it inconsistent with rubocop-rails. I'll test this in the next few days.
If so, I'll close this and will try to address this in RuboCop.

E.g. if you set AllCops:DisabledByDefault to true, and do not add enabling for rubocop-rspec, will they respect the decision, when our own configuration has Enabled: true :/ (with Enabled: false, would they respect being enabled by default)?

So, according to my experiment of running a project that has DisabledByDefault: true against this PR does not inspect with RSpec cops. It only inspects if Enabled: true is explicitly specified in the project's .rubocop.yml.
So we won't be enabling all our cops for those who prefer to disable every cop by default.

E.g. if you set AllCops:DisabledByDefault to true, and do not add enabling for rubocop-rspec, will they respect the decision, when our own configuration has Enabled: true :/ (with Enabled: false, would they respect being enabled by default)?

So, according to my experiment of running a project that has DisabledByDefault: true against this PR does not inspect with RSpec cops. It only inspects if Enabled: true is explicitly specified in the project's .rubocop.yml.
So we won't be enabling all our cops for those who prefer to disable every cop by default.

Thank you for this PR! Looking forward to having it in a future release ☺
This allows us to enable only specific RSpec cops while disabling every other cop. Like in the following example:
RSpec:
  Enabled: false
  Include:
    - '**/*.spec.rb'

RSpec/Focus:
  Enabled: true

This allows for a smoother integration of rubocop-rspec to an existing, large codebase ☺

Thank you for the warm words, @valscion!


can not release gpu memory after trained on ipython notebook

Environment info
Operating System:
ubuntu 16.04
Compiler:
cmake
Package used (python/R/jvm/C++):
python
xgboost version used:
xgboost0.6 with GPU(GTX 1080Ti)
If installing from source, please provide
yes.
git clone --recursive https://github.com/dmlc/xgboost.git
If you are using jvm package, please
no.
If you are using python package, please provide
python2.7
sudo python setup.py install
If you are using R package, please provide
no
Description:
When xgboost was training, GPU worked well.But when training finished, GPU memory was not released automatically. I had to release GPU memory with command: sudo kill -9 SID

Can you show your python script.
Python is garbage collected so often doesn't delete the booster instance. See if manually deleting the booster instance in python helps.

sorry I'm make a mistake.please forget it.


What does freq_cis mean?

I keep seeing freq_cis in run.c. What does it mean?

See RoPE, the relative positional embedding scheme of Llama https://arxiv.org/abs/2104.09864


Add resource predicates for tasks

What this PR does / why we need it:
Add resource predicates for tasks
Which issue(s) this PR fixes:
Fixes https://github.com/kubernetes-sigs/kube-batch/issues/694

If we add Resource predicate check in predicate plugin, it affects preemption also, so have moved that check to Allocate alone

/lgtm
/approve

@thandayuthapani , thanks very much for your effort. I resolved conflict at #717 to catch up internal schedule, sorry for that.


Add ACPR tuto slides and news

Thanks

Ok it seems that there are issues with the post: images are not ok and it does not appear as the latest post (date issue ?).

I have fixed the date (and remaining typo). Can you please fix the images ?

I look it sorry ;(

I have fixed the date (and remaining typo). Can you please fix the images ?

I push directly on the master

@dcoeurjo done ;)

ok thx I've slightly update the text

thanks 👍🏻


PropertyFieldDatetimePicker not calling onPropertyPaneFieldChanged

Category

[x ] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

I'm trying to save two date times using this control but the property is never saved, nor is it present in this.properties. onPropertyPaneFieldChanged isn't being called which I'm guessing is the issue.
Here's my initialization of the picker, along with the versions of pnp and the generator I'm using. I'm sure I'm just doing something wrong with the initialization of the picker but I'm not sure.
PropertyFieldDateTimePicker('startDate', {                   label: 'Start Date and Time',                   initialDate: this.properties.startDate,                   dateConvention: DateConvention.DateTime,                   timeConvention: TimeConvention.Hours12,                   onPropertyChange: this.onPropertyPaneFieldChanged,                   properties: this.properties,                   onGetErrorMessage: (error) => { return error; },                   deferredValidationTime: 0,                   key: 'startDateFieldId'                 })
"@microsoft/generator-sharepoint": "^1.8.0",
"@pnp/spfx-property-controls": "^1.14.1",
"react": "16.7.0",
"react-dom": "16.7.0",

Hi @steevinBradlee,
I tested it out and it works fine both in Reactive and non Reactive modes.
You can find the code here: https://github.com/AJIXuMuK/issues/tree/master/property-controls-186. But basically, it's the same as yours.

@steevinBradlee can you verify the answer from @AJIXuMuK?

@estruyf  @AJIXuMuK  I wasn't able to get this working in the project I added it to but was able to create a new project and it worked fine. I'd like to investigate more but I think it's probably safe to close this. Thanks for your help!


How to save the map in a pcd file, previous issue dont have proper solution.

created a map and how to save in a pcd file. Also my map disappear from the starting after sometime.

You can increase the decay time to fix the max disappear

save map code ; wechat 394467238


Add Continuous Integration builds

Would be nice if we could have a CI build setup to automatically compile and run unit tests on each pull request. This would make it easier for those merging pull requests since they would know automatically if a particular pull request has broken the build.
Suggest using App Veyor for this. http://www.appveyor.com/.

I can help with the setup but this will require someone with admin access to the repo.

I can help with that as I have admin and have done similar for other repos
FYI we currently direct deploy from repo to azure website at http://allready-d.azurewebsites.net/ so just let me know what you want changed to use AppVeyor (which I have used before but not yet for htbox and I am there)
Lastly I can start an htbox account there and give you access as it is free for open source etc but won't get to that until later this evening

Deployment from Repo to azure website is great. I don't think we need to change that.  What I would like to enable is a workflow we trigger a CI build (compile and run unit tests) for each pull request. That way we would have more confidence that the code being merged to master isn't breaking something. Setting this up involves creating an App Veyor account, configuring the App Veyor build and enabling integration via Web Hooks.
It would be great if we can create an App Veyor account and then we can work together on the configuration.

Yep makes snese, I was trying to type out that response while in the middle of something else so thank you.  You should be connected to our AppVeyor account now

Will extend as needed. Basics builds are in place now


Unhandled connection closed

https://github.com/khazhyk/osuapi/blob/19cb80aad8035e103eb402127fef15c1f8d0a2c1/osuapi/connectors.py#L47-L62
I believe your logic was that
raise HTTPError(resp.status, resp.reason,                   (yield from resp.text()))
Would return existing response text even if the response object is closed.
However, this is not the case with the empty body:
https://github.com/aio-libs/aiohttp/blob/master/aiohttp/client_reqrep.py#L913-L922
Forcing a read on a closed object.
elif resp.status == 504 and retries: the loop wouldn't run if the retries variable was negative or 0, so this is unnecessary.

I'm not sure what you mean by "case with the empty body"
import aiohttp, asyncio
loop = asyncio.get_event_loop
a = loop.run_until_complete

s = aiohttp.ClientSession(loop=loop)
res = a(s.head("https://google.com))
res.close()
res._body
a(res.text()) ''
res._body  # b''
a(res.text()) ''

Should be fixed


Tweepy media upload unsuccesful

Hello,
I am quite new to coding and got a bit stuck on this. I have been trying to use tweepy to upload a Image/GIF, unfortunately without any success so far. Please see the code I used below:
`import tweepy
import json
with open('twitterauth.json') as file:
secrets = json.load(file)
auth = tweepy.OAuthHandler(secrets['consumer_key'], secrets['consumer_secret'])
auth.set_access_token(secrets['access_token'], secrets['access_token_secret'])
twitter = tweepy.API(auth)
twitter.update_with_media('/home/pi/gifcam/gifs/test_1.jpg', 'your status update')`
I was able to make a tweet but no luck with Image/GIF.
Every help would be much appreciated.
Thanks and best regards
david

@new-yen
This is because POST statuses/update_with_media is now Deprecated we have to change the working of  update_with_media here and make it work as specified here.
@Aaron1011 @joshthecoder  I can fix it, please assign it to me. Thanks.

@moghya thanks for the information. The missing chunked upload might still lead to problems with bigger file sizes,which was not an issue in my case, as I only tried to upload small images and GIFs.
I figured out, that my problem was, that I tried running my script in Python 3 but it only works in Python 2.
I think this is worth mentioning and maybe should go in a pull request or did I just miss this information somehow.
Thanks and best regards
david

POST statuses/update_with_media, the endpoint that API.update_with_media uses, has indeed been deprecated and is no longer documented now. You should use API.media_upload with API.update_status instead. See Twitter's API documentation for uploading media.
See also the documentation for the in_reply_to_status_id parameter of API.update_with_media and API.update_status.
Tweepy should be fully compatible with Python 3. Without a full traceback or a reproducible example (MCVE/SSCCE), it's impossible to determine what exactly your issue is, although you're uploading a file with a .jpg extension even though your issue specifies that you're uploading a GIF.
As for chunked uploads, see #929.
For code block usage, see https://docs.github.com/en/github/writing-on-github/creating-and-highlighting-code-blocks.


Projects createnew or openexisting has an odd appearing button

Version: 1.25.0-insider (system setup)
Commit: 0b6fb504dc13e8dec935a21b39cb266d27b71c35
Date: 2020-11-23T06:41:43.143Z
VS Code: 1.48.0
Electron: 9.3.0
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042
Steps to Reproduce:

workspace pane select "create new" or "open existing"
button next to location appears oddly but is still clickable

Fixed by https://github.com/microsoft/azuredatastudio/commit/6e0a4f27de9199cf62ac0083c02ed2130d855d30


feat: IsSelfAdjoint.smul_iff

Also generalizes some typeclass assumptions from DivisionSemiring to GroupWithZero and renames a few declarations.

Thanks! Once CI is happy, please merge this PR using bors r+
bors d+

bors r+


chore: Document the readme

The current readme does not contain anything.
It needs to be documented
Acceptance Criteria:

Add README.md fille
Have sections for "Description", "Getting Started", "Features", "Demo", "License", "How To Contribute"
Add MIT file License.

I am re-opening this since I will add more contents over @VrindavanSanap's works.


Gis: Topography view is applied for whole column only

Description
GIS values can't be displayed in the topography view
DBeaver Version
<IP_ADDRESS>207210656
Operating System
Win
Database and driver
PostgreSQL
Steps to reproduce

Open a table with gis values (e.p. PostgreSQL gis.public.planet_osm_line)
open the Value panel, select some GIS values and change the view on the Topography in the panel

Expected: the data is displayed in the Topography view in the Value panel
Actual result: the map view isn't changed
https://user-images.githubusercontent.com/51405061/183298441-3270bf9c-1790-4e73-a84f-ba3a0fc02337.mp4
Additional context
No response

The topography view has a maximum zoom level of 17, while the street view has a zoom level of 20.
You can't switch to a given tile layer when you're zoomed in more than its maximum zoom level.
By selecting multiple points on the map, you've effectively zoomed out, so every point became visible.


PBI Action: Update Parameters not working

Hi Maik, we're trying to deploy a pbi report using your tool. First, we publish the report and then we want to update a few parameters in the report. The publish works fine, but the update parameter step doesn't update the parameters. There is no error in the step either. Based on the code in the function "Update-PowerBIDatasetParameter", I'd expect a log entry for this line "Write-Host "Updated parameters in Dataset- '$($Set.name)'"", but it's not there, which would indicate it's not getting in the if-statement, which would indicate it can't find the parameter, even though it is there.  If there would be a permission issue, I'd expect an error would be thrown, which is also not the case, so a bit puzzled where the problem could be. Might it be a good idea to also log the output of this line: $datasetParameters = Get-PowerBiParameters -GroupPath $GroupPath -SetId $setId ? I don't mind sharing the log, but I'd prefer to do it in a private way, not just publicly in this post.

@c4g-wv: Could you add the system.debug variable and set it to true and then share the log. You can do this via mail: info at msftplayground.com

@maikvandergaag : issue was with passing the update parameters json string from our side, wrong structure. Closing this item.


Use original operationId in annotated GeneratedMethod

The generated annotation @GeneratedMethod currently uses as a value a sanitized name for the operationId. This is necessary for codegen because it is/can be used as java identifier, for instance, in method, variable or class name.  The issue with that, is that the reference to the operationId is lost. For example, an operationId of Jobs doJob gets translated to jobsDoJob, in a method with the same name. In the case of the annotation value, it makes sense that we keep the original String, because is not used for the codegen, it is a simple string. By using the original operationId, without sanitizing, it allows consumer applications to track back with an exact match to the operationId contained in the OpenAPI spec file.

Maybe would make sense to have @GeneratedMethod (operationIdOriginal= "{op.operationIdOriginal}", operationId="@GeneratedMethod ("{op.operationId}"))

Maybe would make sense to have @GeneratedMethod(operationIdOriginal="{op.operationIdOriginal}", operationId="{op.operationId}")

No, I think the PR is fine, this should be the original name

The only problem I see is breaking the compatibility with client code using this annotation expecting the sanitized name. That's fine, we can add a note in the release notes and we don't have a final release yet.

The only problem I see is breaking the compatibility with client code using this annotation expecting the sanitized name. That's fine, we can add a note in the release notes and we don't have a final release yet.

Great @ricardozanini. One problem we have realized might occur and needs to be somehow handled by the extension is when we have two operation ids that leads to the same java method name
This will unlikely occur, for example, if user define two operationids, one name my-method and another named my#method.
In those cases, we should fail and force the user to define a mapping (in configuration) for these "funny" method names.

This naming clash should be handled by the OpenAPI Generator we are using. We can, however, override this behavior. I don't think this should be done here but bring it to that community instead.


Fix max_by/min_by when process extreme value

Remove initial value for fix size type, use group initial null flag to check if it is the first valid value to process.
Timestamp has some problems when represent extreme value, so we skip some tests which comparison type is Timestamp temporarily.
Also, add Timestamp to isNumeric check.
Fixed: #5145

Hi @mbasmanova would you help take a look?

Depends on #5083

I also find isNumericOrDate does not include Timestamp, but it should be include right? @mbasmanova

I also find isNumericOrDate does not include Timestamp, but it should be include right? @mbasmanova

Nice catch. Yes, it should.

BTW, are you wokring on add bool comparison type for this function now? if not, I would like to. @mbasmanova

BTW, are you wokring on add bool comparison type for this function now? if not, I would like to. @mbasmanova

I'm not working on it. It would be great if you could add that support. Thanks.

This issue https://github.com/facebookincubator/velox/issues/5398 block CI.

@mbasmanova Since #5083 merged, I rebased main branch and add Timestamp extreme value UT.


fix: don't create feature export before launch darkly import

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[ ] I have added information to docs/ if required so people know about the feature!
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes
Fixes #3509
Note that it does not make much sense to run an export prior to an LD import since the import is only really useful for new projects. The feature export (I believe) was being done to create a sort of back up but it was not providing the environment id (because there is no contextual environment here).
How did you test this code?
Tested using the production preview but I'm still not seeing the correct behaviour. See comment here.

This change seems to have correctly removed the broken request to create-feature-export but I'm still getting a blank page after confirming the import. @kyle-ssg @novakzaballa might need some further assistance here.

@matthewelwell, I added a fix for a race condition that could have been triggering the issue, could you please review it?

@novakzaballa can you provide more information about the change that you made, I don't quite understand it.

Slack thread on the above topic here.


Write a blog post on https://communityblog.fedoraproject.org/ about the work done for Milestone 1

GSoC Progress Report: Dashboard for Packit :tada:


Render page title successfully after converting from markdown to html?

Let me know if this is not the right place for this. I searched the documentation but didn't find another place to ask.
I have a mostly boilerplate script derived from the example in the documentation that I've used for the last couple years.
However, I've noticed one problem and that is that title's don't get rendered in the HTML. Is there a way to make sure that these are parsed by the context instance successfully and subsequently passed to the site object for rendering in the HTML HEAD's title tag?
This code is getting the title successfully in the site_context function, but I'm not sure how to pass the title (via kwargs or another mechanism) so that it's rendered by the render_site function.
Thanks!
    #!/usr/bin/env python3

    # Convert markdown to HTML

    import os
    import markdown

    from pathlib import Path
    from staticjinja import Site
    from dotenv import load_dotenv

    load_dotenv()
    SRC = os.environ['SRC']
    DOCS = os.environ['DOCS']
    markdowner = markdown.Markdown(output_format="html5")

    def site_context(template):
        markdown_content = Path(template.filename).read_text()
        title = markdown_content.split('\n')[0].strip('# ')
        print(title)
        return {"post_content_html": markdowner.convert(markdown_content), "title": title}

    def render_site(site, template, **kwargs):
        # i.e. posts/post1.md -> build/posts/post1.html
        out = site.outpath / Path(template.name).with_suffix(".html")
        print(out)
        # Compile and stream the result
        os.makedirs(out.parent, exist_ok=True)
        site.get_template("_base.html").stream(**kwargs).dump(str(out), encoding="utf-8")

    print('rendering site')

    site = Site.make_site(
        searchpath=f"{SRC}",
        outpath=f"{DOCS}",
        contexts=[(r".*\.md", site_context)],
        rules=[(r".*\.md", render_site)],
    )

    site.render()

24 hours later, I realized that code I posted works, but I'm not sure why. Can anyone tell me how adding the title key, value pair to the dictionary returned by the site_context function sets the template's title tag successfully? Is that dictionary injected into the global or local namespace of the downstream template generation function? I did see the documentation note in 1.2.3 about env_globals:
    if __name__ == "__main__":
            site = Site.make_site(env_globals={
            'greeting':'Hello world!',
    })
    site.render()

Anything added to this dictionary will be available in all templates:
    <!-- templates/index.html -->
    <h1>{{greeting}}</h1>

But I'm not doing that here. Any insight? Thanks!

No problem @NickCrews Thanks for your response!
I'm super happy that what I tried worked and is working in production. I just didn't realize on this original post that it was working!
I think your explanation makes sense, given the nature of the regex glob.
However,  I would still like to understand the mechanism by which simply adding the title dictionary element to the return from the site_context function has the seemingly magical result of populating the title on the far side of the caller.
I realize this is by way of the fact that what's passed into Site.make_site is a tuple of globs and function pointers, but when I read the source in staticjinja, my head exploded with all the clever abstraction. I'd be grateful if somebody could walk me into understanding the deeper foo.
I see where the make_site method just returns an instance of the constructor:
https://github.com/staticjinja/staticjinja/blob/0daf08bbf645e0051bb521cee73394c206e6244b/staticjinja/staticjinja.py#L276-L285
But how do the (re_glob, fp) tuple's elements, such as title, cross the chasm? I'm just not seeing the magic.
I realize this may not be the right place and I'm hugely appreciative of your reply so I can't thank you enough for going above and beyond here.
To answer your final question, I'm just trying to learn while finding the biggest missing link in the open source ecosystem I'm familiar with is links to things like discord, gitter, irc, or the like.
Thanks again!

Try adding an assert False inside your context function. This will crash, and it should print the call stack so you can see how things are calling each other. If that still is confusing then I can explain more :)


List all Spaces for the Organization documents invalid query parameter

The API documentation for List all Spaces for the Organization describes a organization_guid filter. This filter is invalid as the organization_guid is included in URI instead.

Will be addressed once https://github.com/cloudfoundry/cloud_controller_ng/commit/ebbae0ab4bad7d1ce8fec3402812c45e352d4210 makes it into a cf-release.
You can find docs that include this change at
http://s3.amazonaws.com/cc-api-docs/104228398/index.html


Update workflow to use terragrunt

This PR updates the workflow to use terragrunt on top of terraform.

@lsetiawan - let me know if there is an issue i can help figure out here! also feel free to deploy test infrastructure as needed.


 I suggest preventing kill from being called on dead objects in Phaser (so I don't have to add a boolean check in every place I use kill).

This is literally all the kill function does:
    kill: function () {

        this.alive = false;
        this.exists = false;
        this.visible = false;

        if (this.events)
        {
            this.events.onKilled$dispatch(this);
        }

        return this;

    }

There is no way calling this repeatedly would cause a call size error. Something else is doing it.

Give us the stack trace for that call size error (or post it on the HTML 5 forums) we can help.

Just to add that I need a code example showing this happening, or I'll have to close it off, sorry.

Thanks for opening this issue. As much as we'd love to help you, we need a fully working code sample to do so. Please provide one and then we'll re-open this issue and look at resolving it for you if possible.
If it's easier you can use the Phaser Sandbox to create a code sample (as this is now running the latest build of Phaser). Enter your code, click the 'save icon' in any of the editor tabs, and paste the URL here.


Add [Binance Withdrawals binance.com] #exchange GC5LF6…D6Z6FH

Address: GC5LF63GRVIT5ZXXCXLPI3RX2YXKJQFZVBSAO6AUELN3YIMSWPD6Z6FH
Tags: exchange
Title: Binance Withdrawals  Domain: binance.com
Requested by @jesse6484

forgot to add the tag to my binance XLM account  GAHK7EEG2WWHVKDNT4CEQFZGKF2LGDSW2IVM4S5DP42RBW3K6BTODB4A
335045413

Hello @jesse6484,
You can follow the discussion with your original posted issue https://github.com/stellar-expert/public-directory/issues/442


It would be nice to be able to CTRL F and search for text on the terminal

Description of the new feature/enhancement

Proposed technical implementation details (optional)

ctrl+shift+f, which is rebindable and documented here in the documentation


[ALLUXIO-2815] Use AlluxioStatusException throughout client [WIP]

https://alluxio.atlassian.net/browse/ALLUXIO-2815

Add the capability for AlluxioStatusExceptions to wrap IOExceptions. We have the property that AlluxioStatusException.fromIOException(e).toIOException() == e as long as e instanceof IOException.
Within the boundary of our client public API, all exceptions we throw are AlluxioStatusExceptions.
At the public API borders (BaseFileSystem, AbstractFileSystem, FileInStream, FileOutStream, AbstractLineageClient, LineageFileSystem), we convert any AlluxioStatusExceptions to the exceptions used by our public API. No AlluxioStatusExceptions should escape the boundary of our public API. In 2.0 we will remove the current public API exceptions and either throw a checked version of AlluxioStatusException or directly throw AlluxioStatusException.
Many places where we would catch checked exceptions previously were actually incorrect in the face of unchecked exceptions. This has been fixed.
Update IOException javadoc to say @throws IOException if an I/O error occurs. Given the nature of IOException, any more detailed message we provide can very easily become incorrect.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14645/Failed Tests: 2org.alluxio:alluxio-core-client: 1org.apache.maven.surefire.junit4.JUnit4Provider.alluxio.hadoop.AbstractFileSystemTestorg.alluxio:alluxio-tests: 1alluxio.security.MasterClientAuthenticationIntegrationTest.customAuthenticationDenyConnect
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14646/
Test PASSed.

Merged build finished. Test PASSed.

@calvinjia @apc999 @jsimsa This PR is ready for review

Let me know if you'd like the files to be split into multiple sub-PRs

@calvinjia Thanks for the quick high-level review

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14648/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14653/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14658/
Test PASSed.

@aaudiber thank you for the interation, did another pass and left one more question ... otherwise LGTM

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14664/
Test PASSed.

@calvinjia @jsimsa Comments addressed, PTAL

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14670/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.client.FileSystemIntegrationTest.mountShadowUfs
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14673/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.master.file.ConcurrentFileSystemMasterTest.rootConcurrentDelete
Test FAILed.

Jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14674/
Test PASSed.


update note

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/2826

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@LanaChin PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@LanaChin  Hi! Following up on this issue, please let me know if there is anything I can do to help!.

Reached out to PM for review.

@LanaChin Hi Lana, any updates? Thanks!

Bumped the thread to the PMs.


Test coverage number too low when running test through interface instead of concert class

Hi,
When run test through interface (I'm using typescript), the coverage number is too low.  It drops from 92% to 5%.  Is there any way to get the correct coverage number?
regards,
Jerry

I don't understand what you mean. Could you elaborate?

Sorry, I found out that it is my problem.  I removed the interface and the test coverage number the same.  I guess either I missed some source file in instrumental call or the test file is not included.


A profile page for the user

There should be a profile page for the user which shows his details and an option to edit the profile while leads him to a prefilled form to edit the profile.
Email, username uniquesness should be chexked.
Password change option should not be there.
The profile page link should end like /username and not with  /id
Link that to the blog page and remove details from there. Only paragraph about the user should be present there and a button to his profile.

Can I work on this issue?

@thebishaldeb Please assign this to me

I have made a route as /user/:username. @devyaniChoubey you can continue from creating the content of the page. This will save your time.

@vkgupta857  I have also made it. As the issue was assigned to me I will work on it.

@vkgupta857 I have also made that route. As the issue was assigned to me I will work on it.
OK


fix: default resolver throwing error when association included

As per the title

:tada: This PR is included in version @graphql-gene/plugin-sequelize-v1.1.2-beta.2 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @graphql-gene/plugin-sequelize-v1.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


investigate using a monorepo for the new ArviZ

for all of arviz-[base|plots|stats]

What does this mean?

Specifically it means putting arviz-base, arviz-plots, and arviz-stats into a single repo, but have each package installable on its own if a user needs/wants. Similar to how Jupyterlab has its packages folder, or how Bokeh has its python tool in the same repo as its js tool.


[Doc] zstd.dll installation on Windows

📚 Documentation
Suggestion
Please review and update the following from this page:
https://llm.mlc.ai/docs/install/tvm.html#install-tvm-unity
original: It is likely zstd, a dependency to LLVM, was missing. Please download the precompiled binary, rename it to zstd.dll and copy to the same folder as tvm.dll
Suggested:
It is likely zstd, a dependency to LLVM, was missing. Please download the precompiled binary, rename it to zstd.dll and copy to the same folder as tvm.dll. Hint - Perform a search for "tvm.dll" and identify the folder in which the path includes the name of the current environment eg. mlc-chat-venv. Copy zstd.dll to that folder.
Bug

Link to the buggy documentation/tutorial:
Description of the bug:
This suggestion is based on the following reported by @Sing-Li

Please pardon my (possible) related comment. The Note section above. Downloading and "rename it to zstd.dll and copy to the same folder as tvm.dll" part of the instruction --- is almost impossible for new users to accomplish because they have to understand how conda relates to python, and where miniconda keeps its python site libs before they can find "the same folder as tvm.dll". I hope one of the above fixes will make sure that zstd.dll is always included as part of the tvm bundle/nightly

This is a headache I really wanted to resolve :(

One easier way I found is to install zstd in conda, which even works nicely with Windows:
conda install zstd

I don’t think it’s complete :((  Eventually we should be able to pack zstd with TVM so no extra steps are required

Happy to share the zstd issue seems resolved with a few latest changes!

https://github.com/apache/tvm/pull/16081
https://github.com/apache/tvm/pull/16032
https://github.com/mlc-ai/package/commit/ebe2cd9aa6c3db3a50026b94267e97f8d7ec8d4c

Would you guys mind trying out tomorrow with the latest nightly build?

Closing as the issue has been addressed. Feel free to create new ones if it persists!


Add and remove liquidity modals - when input amount is zero it should be in grey (like betslip)

Design:

this was done previously

this was done previously


Fix issue#3276

Fixes #1
fix issue#3276 Remote IP Address - NULL reference exception

@cpolanec Can you help add me to the contributor to this folk repo so that i can contribute on this?


Update config references in README.md

Fixes instances in the README where [[items]] was used instead of [[item]].

Hey, thanks for the PR!
This is intended. [[item]] is deprecated (but still works) starting from v0.5.1.  The README docs match the sample configuration for the latest version.

I assume you use the AUR version. Thanks for the heads up, i'll go ahead and push and update to it.

Ah, that makes sense, and yep I got it from AUR. Thanks!


npm ERR! code 1 when gulp task 'karma:single-run' fails

Failing a test after executing gulp task 'karma:single-run' returns npm ERR! Exit status 1. The reason this is happening is because of karmaFinishHandler() function in gulp_tasks/karma.js.
Problem code:
function karmaFinishHandler(done) {
  return failCount => {
    done(failCount ? new Error(`Failed ${failCount} tests.`) : null);
  };
}

Whenever failCount is truthy, 'karma:single-run' errors. This only happens for 'karma:single-run' (not an issue with 'karma:auto-run')
Simple fix:
function karmaFinishHandler(done) {
  return () => {
    done();
  };
}

Reproduce

scaffold angular 2 app using FountainJS

paul@mars:~/Development/test$ yo fountain-webapp

     _-----_
    |       |    ╭──────────────────────────╮
    |--(o)--|    │    Welcome to Yeoman,    │
   `---------´   │   ladies and gentlemen!  │
    ( _´U`_ )    ╰──────────────────────────╯
    /___A___\   /
     |  ~  |
   __'.___.'__
 ´   `  |° ´ Y `

Out of the box I include by default Gulp 4, ESLint, Browsersync and Karma.

? Which JavaScript framework do you want? Angular 2
? Which module management do you want? Webpack with NPM
? Which JS preprocessor do you want? TypeScript
? Which CSS preprocessor do you want? SASS
? Which Continuous Integration platform do you want? Travis
? Do you want a sample app? A working landing page
? Would you like a router? @angular/router

open /src/app/header.html and edit text "Fountain Generator" to something else (i.e., "Fountain Generator rocks!" so that karma test fails

open terminal and enter npm run test to execute gulp test (which in turn executes gulp task 'karma:single-run')

Result
paul@mars:~/Development/test$ npm run test

> @ test /home/paul/Development/test
> gulp test

[11:46:56] Loading /home/paul/Development/test/gulp_tasks/browsersync.js
[11:46:56] Loading /home/paul/Development/test/gulp_tasks/karma.js
[11:46:57] Loading /home/paul/Development/test/gulp_tasks/misc.js
[11:46:57] Loading /home/paul/Development/test/gulp_tasks/webpack.js
[11:46:57] Using gulpfile ~/Development/test/gulpfile.js
[11:46:57] Starting 'test'...
[11:46:57] Starting 'karma:single-run'...
ts-loader: Using<EMAIL_ADDRESS>and /home/paul/Development/test/tsconfig.json
25 11 2016 11:47:05.878:INFO [karma]: Karma v1.3.0 server started at http://localhost:9876/
25 11 2016 11:47:05.879:INFO [launcher]: Launching browser Chrome with unlimited concurrency
25 11 2016 11:47:05.964:INFO [launcher]: Starting browser Chrome
25 11 2016 11:47:09.079:INFO [Chrome 54.0.2840 (Linux 0.0.0)]: Connected on socket /#8LyRLrlxOlZiNbf4AAAA with id 72959560
Chrome 54.0.2840 (Linux 0.0.0) header component should render 'Fountain Generator' FAILED
	Expected 'Fountain Generator rocks!' to be 'Fountain Generator'.
	    at Object.<anonymous> (src/index.spec.js:42307:62)
	    at ZoneDelegate.invoke (src/index.spec.js:51054:26)
	    at ProxyZoneSpec.onInvoke (src/index.spec.js:50718:39)
	    at ZoneDelegate.invoke (src/index.spec.js:51053:32)
	    at Zone.run (src/index.spec.js:50936:43)
	    at Object.<anonymous> (src/index.spec.js:50433:34)
	    at src/index.spec.js:4456:21
	    at ZoneDelegate.invoke (src/index.spec.js:51054:26)
	    at AsyncTestZoneSpec.onInvoke (src/index.spec.js:50023:39)
	    at ProxyZoneSpec.onInvoke (src/index.spec.js:50715:39)
	    at ZoneDelegate.invoke (src/index.spec.js:51053:32)
	    at Zone.run (src/index.spec.js:50936:43)
	    at AsyncTestZoneSpec._finishCallback (src/index.spec.js:4451:29)
	    at src/index.spec.js:50012:31
	    at ZoneDelegate.invokeTask (src/index.spec.js:51087:35)
	    at Zone.runTask (src/index.spec.js:50976:47)
	    at ZoneTask.invoke (src/index.spec.js:51157:33)
	    at data.args.(anonymous function) (src/index.spec.js:51792:25)
Chrome 54.0.2840 (Linux 0.0.0): Executed 7 of 7 (1 FAILED) (0.829 secs / 0.778 secs)
[11:47:11] 'karma:single-run' errored after 13 s
[11:47:11] Error: Failed 1 tests.
    at failCount (/home/paul/Development/test/gulp_tasks/karma.js:13:22)
    at removeAllListeners (/home/paul/Development/test/node_modules/karma/lib/server.js:379:7)
    at Server.<anonymous> (/home/paul/Development/test/node_modules/karma/lib/server.js:390:9)
    at Server.g (events.js:292:16)
    at emitNone (events.js:91:20)
    at Server.emit (events.js:185:7)
    at emitCloseNT (net.js:1544:8)
    at _combinedTickCallback (internal/process/next_tick.js:71:11)
    at process._tickDomainCallback (internal/process/next_tick.js:122:9)
[11:47:11] 'test' errored after 13 s

npm ERR! Linux 4.4.0-47-generic
npm ERR! argv "/home/paul/.nvm/versions/node/v7.2.0/bin/node" "/home/paul/.nvm/versions/node/v7.2.0/bin/npm" "run" "test"
npm ERR! node v7.2.0
npm ERR! npm  v3.10.9
npm ERR! code ELIFECYCLE
npm ERR! @ test: `gulp test`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the @ test script 'gulp test'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the  package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     gulp test
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/paul/Development/test/npm-debug.log
paul@mars:~/Development/test$

Environment
ubuntu 16.04 LTS
node -v 7.2.0
npm -v 3.10.9
yo -v 1.8.5
generator-fountain-webapp -v 1.0.0-rc1

Expected 'Fountain Generator rocks!' to be 'Fountain Generator'.


Fixed ReactNative code and modernized "rnfc"

Currently, when you type rnfc
import React from 'react';
import { View } from 'react-native';

export default function componentName() {
  return (
    <View />
  );
}

Clearly, it is missing the opening tag of <View>.
I also changed it so it will look like:
import React from 'react';
import { View, Text } from 'react-native';

export default componentName = () => {
  return (
    <View>
      <Text>textInComponent</Text>
    </View>
  );
};

Honestly, I didn't exactly know how to test this out, so if you know how or can lemme knw how to test it, I can test and report it back

Hi @dsomel21, thanks for the contribution. To accept your pull change the function like this, no use arrow function to keep the default of others snippets.
`
import React from 'react';
import { Text, View } from 'react-native';
export default function componentName() {
  return (
    <View>
      <Text>textInComponent</Text>
    </View>
  );
}

`


[ci:component:github.com/gardener/dependency-watchdog:v1.1.2->v1.2.0]

Release Notes:
DWD no longer depends on checking the reachability of Kube-API server via external load-balancer as kube-proxy path optimizations makes that impossible on infrastructures which have a static IP for the LB. Instead it now depends on node leases that are periodically renewed by each node running in the shoot cluster to determine its scaling action (scale-down/up).

Use `ginkgolinter` instead of self baked `gomegacheck`

Change OCI Image Registry from GCR (`eu.gcr.io/gardener-project`) to Artifact-Registry (`europe-docker.pkg.dev/gardener-project/releases`). Users should update their references.

/close in favour of https://github.com/gardener/gardener/pull/9072


ABC (#26)

testing

commitin hey

commitin hey

Co-authored-by: William Huang<EMAIL_ADDRESS>Co-authored-by: zhengHugo<EMAIL_ADDRESS>

This is a comment


Authenticate without OAuth Server

Hello,
I have been poking around the Mixer client and am trying to find a way to authenticate without the OAuth server. I have a bot that will be containerized and run remotely without user interaction. I'm hoping there's a way that I can autheticate with a bot account using stream key. Is that possible with the client?

I understand what you are asking, but the process of OAuth requires user interaction, for security reasons.  It is possible to generate a token once, and refresh/reuse that token for a long time. If the refresh fails or expires, it will require recreating a new token with user interaction again.  Does that make sense?
Using the shortcode method could allow you to perform that refresh remotely if you could have your service notify you that it needs a refresh... send the code/link to you to perform it remotely/manually. Then you could get an OAuth token that way.  Hope that helps.

@TyrenDe  Thank you for the answer!
So what would the workflow look like?
I'd run the OAuth browser authentication and I'd retrieve the auth code from the connection response? And then the next time I kick up the task I'd MixerConnection.ConnectViaAuthorizationCode()? I tried doing that and got an error stating that the auth code was already used.

I wrote you a detailed e-mail. Let's chat there. :)


Incorrect behaviour with SizeToContent

Incorrect behaviour with window property SizeToContent = "WidthAndHeight"

Workaround:
window.ContentRendered += (o, args) => window.BorderThickness = new Thickness(1);

I'll close this issue.
If you have any problems, please reopen this issue.


Add LDAP Group Lookup for use with OIDC Provider

When using OIDC Providers, add an optional LDAP group lookup to support providers that do not include groups in the ID Token claims.
This PR introduces bonsai, an LDAP library with asyncio support, and the dependencies required to run it.

I have tested the LDAP lookups independently at SLAC, but I have not been able to test running Gafaelfawr yet.

Merged in #390. Thank you!


[Low priority] Prevent users from signing up with an expired invitation

The user_invitation table has an expires_at field which should be taken into account when signing up. If the invite has expired, throw an error and let the user know.

handled in https://github.com/papercups-io/papercups/pull/81


Add CloseOnClickAway property for drawer host

Currently, I needed this property for my project, because I am using DrawerHost as an error reporting mechanism, so I needed a behavior where the drawer will not be closed even if user clicks outside of it.
Although by setting the IsEnabled = false can achieve the same behavior, but it also disabled all the controls in the drawers, therefore I needed the CloseOnClickAway property on DrawerHost
Please do take a look at an example usage :

In this scenario, I need the Group-Clashing hyperlink to be clickable by the user, but at the same time I also wants to disable user from closing it by clicking outside of the drawer.

Not only that, I also need this CloseOnClickAway property because I uses DrawerHost for the following scenarios :

As an alternative to dialog box, because in some situation using dialog box is too aggressive.
To display progress bar while loading something and also disabling UI at the same time.

Hi, thanks for this.
There's been a long term desire of mine (and a few other's requests) to add some sort of "pinned" option, where when pinned the template changes and the content is "pushed" out.
So the template in pinned more would be conceptually be like,
<Grid>
    <drawerContent />
    <mainContent />
</Gird>

I'm keen to follow semver, so I need to get the API right for this.  Including your change I think this would give us three distinct modes, so the new property would probably need to be an enum, something like:
OpenMode
    Standard,
    StaysOpen, // your new behaviour
    Pinned // pin, push main content

Are you interested in helping with this?

@ButchersBoy Yes, I'm interested to help with this. But I need to know what's the difference between Pinned and StaysOpen. By the way, does Standard means the drawer will close on click away?

Is there a plan to also have a sort of "mini mode"? Where when only e.g. some icons are visible?

@ButchersBoy I think I get what is the meaning of pinned already (looks something like this). Let me figure out how to do that.

@adrianhara I don't think I will implement that yet.

@wongjiahau maybe i'll look into implementing it, as I would need it.


Volunteer scheduling

As a social worker or volunteer coordinator, I want to manage needs.

[x] I want to create new needs for my office up to two weeks in advance. I will provide relevant information including, day/time, duration, and the age range(s) of the kids as well as the language spoken.
[x] I want needs to be to be automatically split into 1 hour shifts.
[x] I want to manage the needs I have created, including canceling the entire need, canceling a single shift
[x] As a volunteer coordinator, I want to directly assign a volunteer to a shift.
[x] I want a notification to be sent to available volunteers when my need is created

@SarahOMD Currently, the way the system is built, the age range, preferred culture, preferred language, and number of children can be updated after a need has been created. When those need attributes are changed, do we want to re-notify the relevant volunteers of that change?

For posterity, we talked on the call today and no we don't want to re-notify volunteers about an update to a need if they have already been notified.


Fixed npm install + updates

This PR should fix the npm install issues mentioned here https://github.com/transitland/ember-leaflet-polyline-decorator/issues/1
leaflet and leaflet-polylinedecorator and other dependencies are now pulled in with npm.
I also updated the package versions for most of these dependencies to much later versions.
I have also renamed the package from @transitland/ember-leaflet-polyline-decorator to just ember-leaflet-polyline-decorator to keep it consistent with other ember-leaflet addon names (feel free to override this if you don't want the package name changed - I am not sure how this affects npm publishing)
Once this PR is pulled in I'll make another PR with an updated readme with basic instructions on usage & installation with npm
I haven't really messed with ember addons before this so while I think I did everything correctly, I could have missed something (but it seems to work for me now)

Thank you for taking a look at this, @Techn1x and fixing! It looks like that tricky "normalizeEntityName" empty function was missing. Also it's a good move to transition away from bower towards npm like you suggested. The import function is in line with what I've seen from other addons.
I've tested this PR locally and it works for me (using npm link in the addon directory, npm link ember-leaflet-polyline-decorator in a test app directory and ember generate ember-leaflet-polyline-decorator again in the test app directory). One thing about the Leaflet Draw version is that the Transitland Dispatcher currently relies on an older version of Leaflet Draw, and consequently I think this is why the Leaflet version was older as well (it’s been updated recently). It looks like that functionality is disabled currently, however, so that is a separate issue. Not sure about the name, @drewda will have to weigh in on that.

Thanks for working on this issue, @Techn1x, and thanks for reviewing, @doublestranded!
I'll merge this PR in now and re-publish (under ember-leaflet-polyline-decorator).
If this introduces any new bugs, please do open new issues and/or PRs!


ORDER BY in subquery is wrongly ignored

The query below contains a subquery, which output literals in sorted order. The outer query filters these literals by a prefix. The filtered list should also be sorted but it isn't. In fact, also without the FILTER clause, the result is not sorted. The query execution tree (click on "Analysis" in the QLever UI) does not show an ORDER BY operation.
Bottom line: It seems that ORDER BY clauses in subqueries are simply ignored.
PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#>
SELECT ?label WHERE {
  {
    SELECT ?entity ?label WHERE {
      ?entity @en@rdfs:label ?label .
    }
    ORDER BY ASC(?label)
  }
  FILTER regex(?label, "^\"Xylo")
}

 // Remove all manual ordering from the subquery as it would be changed         // by the parent query.         u->_subquery->_orderBy.clear();
is what I've found. I still have to figure out the best way to only remove the orderBy clauses from subqueries if any of their parents actually performs reordering.


ERROR TypeError: Cannot read property 'emit' of undefined (NEW)

Same issue, same angular-tree-component version (4.0.0) of  #449, try the proposed solutions but I could not solve the problem
In my html
 <input #filter (keyup)="filterNodes(filter.value)" placeholder="Filtrar tareas..."/>
<tree-root class="scrollbar-style-1"       #tree       [nodes]="nodes"       [options]="customTemplateStringOptions"       [focused]="true"       (event)="onEvent($event)"       (initialized)="onInitialized(tree)"     >
In my ts
@ViewChild ('tree') tree: TreeComponent;
I use @ViewChild because in a modal I add a new task and update nodes and after doing this I need to use this.tree.treeModel.update();
this.tree.sizeChanged();
When i am trying to search in input it throws "ERROR TypeError: Cannot read property 'emit' of undefined".

Hi,
can you send the entire code of the component so we can help you debug?

Can you try with later version (5.X.X) and reopen the issue if it works?
Or reproduce with the plunkr:
https://plnkr.co/edit/u5fMmaM14rJ0aqG3k240?p=preview


Testable Cmds - Proposal 1 - Added another step to allow bypassing dispatch later

This PR is a proposal to solve #283
Testing Cmd is quite hard today, because it encapsulates its logic inside a function created on the fly which can't easily be unit tested.
Also this function might be asynchronous, returning unit early, preventing to know when it actually finishes.
So to solve this problem, I've added a new module Fabulous.Core.Cmd.Testing.
This module contains 2 helpers: executeUntil and execute.
These functions execute synchronously the Cmds to aggregate their generated Msgs, allowing to unit test them.
But synchronously waiting for the Cmd to dispatch a message might end up in a deadlock due to some Cmd not dispatching sometimes (like Cmd.toMsgOption and Cmd.toAsyncMsgOption).
To prevent that, I've changed the signature of dispatch to always dispatch an optional message (not changed for view though).
For example:
let timerCmd() =
     async {
         do! Async.Sleep 1000
         return TimedTick
     }

let update msg model =
     match msg with
     | TimedTick ->
         let newCount = if model.TimerOn then model.Count + 1 else model.Count
         let newCmd = if model.TimerOn then Cmd.ofAsyncMsg timerCmd() else Cmd.none
         { model with Count = newCount }, newCmd

[<Test>]
let testTimedTick() =
    let initialModel = { Count = 0; TimerOn = true }
    let actualModel, actualCmd = App.update TimedTick initialModel

   let actualCmdMsg = Cmd.Testing.execute actualCmd

   actualModel |> should equal { Count = 1; TimerOn = true }
   actualCmdMsg |> should equal [ TimedTick ]

Here we want to check that if TimerOn is true, then we return timerCmd(). If false, then Cmd.none.
@dsyme Do you think it's a viable solution?
Is it a good use of ValueOption?
I was also thinking of adding some metadata to Cmd in order to provide helpers that could inspect those, instead of actually executing the Cmd.
Maybe it would be a better solution?

Hi! I did take a look but didn't fully understand it.  Perhaps a few more comments in the code - does ValueNone to the Cmd dispatch represent an error condition in the unit test?  Or...? thanks

@dsyme Ok, I will add more comments to the code.
Essentially what I did was just moving the responsibility to choose or not to dispatch to the main loop from the Cmd.of... helpers to ProgramDispatch<'msg>.dispatch.
Instead of having
/// Command to issue a specific message, only when Option.IsSome = true
let ofMsgOption (msg:'msg option) : Cmd<'msg> =
    [ fun dispatch ->
            match msg with
            | None -> ()
            | Some msg -> dispatch msg ]

We have
/// Command to issue a specific message, only when Option.IsSome = true
let ofMsgOption (msg:'msg option) : Cmd<'msg> =
    [ fun dispatch ->
            match msg with
            | None -> dispatch ValueNone
            | Some msg -> dispatch (ValueSome msg) ]

And then ProgramDispatch<'msg>.dispatch calls the dispatch implementation if there's something to dispatch.
static let dispatch =
    id (fun msgOpt ->
        match msgOpt with
        | ValueNone -> ()
        | ValueSome msg -> dispatchImpl msg)

That way, Cmd always calls its callback even if there is no message to dispatch.

does ValueNone to the Cmd dispatch represent an error condition in the unit test? Or...?

No, ValueNone represents the intent to not dispatch a message.
So for example:
let doSomethingMaybe flag =
     if flag = true then
        Some OtherMessage
     else
        None

let update msg model =
     match msg with
     | SomeMessage flag -> model, Cmd.ofMsgOption (doSomethingMaybe flag)

For flag = true, Cmd.Testing.execute will return a list of 1 message : [ OtherMessage ]
For flag = false, Cmd.Testing.execute will return an empty list : [ ]
Without this PR, we can't know if the Cmd will ever return something in case of flag = false (especially with Cmd.ofAsyncMsgOption)
Hope it's clearer.

Closing this PR because Fabulous will most likely not change the way Cmd behaves today (to stay in line with most Elm-ish implementations)
https://github.com/fsprojects/Fabulous/pull/320#issuecomment-491527076
As this comment suggests, this may be solved by following a simple architectural pattern (applicable at the will of the developer)


Support zeebe user task

https://github.com/camunda/linting/assets/28307541/c8a74bca-301a-47b1-9709-945a17f6b2d1

Add human-readable error messages for Zeebe user task errors.
Add links to properties panel.

Related to https://github.com/camunda/product-hub/issues/2126
Related to https://github.com/camunda/camunda-modeler/issues/4087

Requires https://github.com/camunda/bpmnlint-plugin-camunda-compat/pull/161

We are usually a little bit more descriptive than "Not supported" ⬇:

I took this from Zeebe Properties, but can adjust ofc.

Default value is Job Worker when creating a new user task, is this desired? Potentially we want to make this dependent on the platform type selected, can we accomplish that?

It's based on the XML. For all of the legacy tasks, Job Worker is displayed. When we change the implementation to Zeebe user task, a <zeebe:userTask> extension is added. This way, we don't forcefeed new feature to the legacy diagrams (no surprises), and also avoid the unnecessary decision requirement ("Implementation must be selected").
We can change the default but it's outside of the scope of linting project, so let's discuss it via https://github.com/camunda/product-hub/issues/2126#issuecomment-1972967304

https://github.com/camunda/linting/pull/101#issuecomment-1973064297

Agreed.

We are usually a little bit more descriptive than "Not supported" ⬇:

I took this from Zeebe Properties, but can adjust ofc.

@nikku How about this?

Or:

Better :arrow_up: (as a general pattern). Not Form ID etc. are unsupported, but the form type (or implementation type) chosen. :muscle:

The second one right?

I pushed https://github.com/camunda/linting/pull/101#issuecomment-1973157619

I missed the first one :see_no_evil: .
To be discussed, but I like it as it is clear (what would unsupported mean if you don't have the error panel open).

Reached out to the group. But honestly I think we should just adopt the clear "Supported only in Camunda 8.x and newer" consistently, everywhere.
In this sense I'd be fine to leave the current PR as is (following the existing pattern) and rework the call to action as a follow-up.
What do you think?

I think "Zeebe user task not supported."  guides the user better how to fix the problem. It's not the whole entry, but rather the current value which triggers the error. Perhaps we could make it even better: "Only job worker is supported".

On the other hand, if we want to foster migration, pointing out that migration gives more features, better value, "Supported only in Camunda 8.x" serves that better. So it's a matter of priorities.

I think if we focus a drop down and mark it as an error, then it is clear that the current choice is the issue. So :+1: for (1) focussing the dropdown and marking it as an error and (2) having a descriptive error message that indicates the version compatibility "Supported only in Camunda 8.x".

So now it's https://github.com/camunda/linting/pull/101#issuecomment-1973155463 :D

Great stuff. Could you create an issue to follow-up on this, consistently applying it across the properties panel?
There is other places where we currently validate in the wrong place, and I'd imagine the approach you took could work there, too?

I created a follow-up issue: https://github.com/camunda/linting/issues/102#issue-2163570580


Feature/catenax wrapper path

What this PR changes/adds
Currently, the inbound Filter forgets the actual path that is hit under the base-path/deployment-path.
This also leads to the "name" compartment of the HttpDataSource being left empty.
This fix computes the difference between basepath and requestpath and uses it as the "name" property of the DataAddress in the request --> HttpDataSource gets a (hierarchical) name that is appended to the target url.
Why it does that
For wrapping arbitrary REST APIs using the http dataplane, it is necessary not only to take headers, params and body with the data request, but also the concrete URL that is hit (otherwise you would need indefinitely many data planes/addresses to be registered).
Further notes
Tests have been checked before and after (quite some were already red in the main branch, so this was rather a guess than a profound investigation), but no additional test was added (maybe you should think about it).
Linked Issue(s)
Closes # <-- insert Issue number if one exists
Checklist

[ ] added appropriate tests?
[ ] performed checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?
[ ] added relevant details to the changelog? (skip with label no-changelog)
[ ] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

@bscholtes1A I think you wanted to have a look on it or?

Is related to #881

This PR should be closed in favor of #929

Closed in favour of #929


[GH-#369] Represent naming state using patricia merkle tree

resolves #369

does this include the changes from https://github.com/aeternity/elixir-node/pull/368?

@thepiwo No everything

@DanielaIvanova what do you mean? so it includes changes from #368 or not?

@thepiwo We made branch #369 from branch #366 but we added requested changes from PR #368 that means in #369 is not everything included from #368

thank you, but to merge as #368

@thepiwo We will

@thepiwo Done

so, we wait for the other PMT PR to be merged.

will close this until unblocked


Error in compile

Hello,
I am reproducing the project
http://www.mentalmunition.com/2013/10/measure-air-pollution-in-your-home-or.html
But I use other wi-fi shield, my shield is this
http://www.seeedstudio.com/depot/Wifi-Shield-p-1220.html?cPath=19_20
http://www.seeedstudio.com/wiki/Wifi_Shield_V1.1
I'm following in the footsteps of the link
http://www.mentalmunition.com/2013/06/step-by-step-guide-to-programming-rn-xv.html
but I I downloaded the code project, installed the lib and at the time I tried to run the example:
https://github.com/harlequin-tech/WiFlyHQ/tree/master/examples/tcpclient
But the compiller displays on error in archiev WiFlyHQ.h in line 78 of you lib.
"typedef const char PROGMEM prog_char;"
I commented this line and the code compiled correct, but I dont could connect in wi-fi.

I got the same - I can not compile the example file httpserver.ino
Gives an error message:
...WiFlyHQ-master/WiFlyHQ.h:78:28: error: section attribute not allowed for 'prog_char'
typedef const char PROGMEM prog_char;
What could be the problem? Who was able to solve the problem??

Rolling back to Arduino IDE v1.0 had helped me to avoid this error.


When would you use normal styling?

First off, thank you for creating this lovely package for the community.  It is so simple to use and very effective for achieving responsiveness.  This isn't an issue, as it is a question.
Are there any instances where you would use normal styling over heightToPercentageDP or widthToPercentageDP when trying to achieve full responsiveness?
I've been using this package for almost every attribute in react-native (fontSize, margin, padding, width , height).

Hi @bkkwok,
thanks so much for your kind words :-) I am really very glad that you have found this package useful!
Now, about the package methods, I use them myself for almost every attribute in react-native development. There are a couple of cases that you might not need them:

When your position style property (bottom, right etc) has a value of 0
With property border-radius when I need a big value. I have noticed that if you have border-radius: 250 or border-radius:<PHONE_NUMBER>0 the visual outcome is the same. But if I need a small value, then I use the package methods.

Now and a bit off topic from your original question I would like to share how I personally use the package methods in some ways that might not be obvious!

I always configure my container View component to flex: 1
I try to use flex whenever I can
I always use widthPercentageToDP and heightPercentageToDP instead of "normal" percentage values, even for 100%. I have noticed that there is a difference which is visible by eye. I haven't searched how RN percentage gets calculated and rendered but it's not 100% accurate in my experience (at least in Android).
I have found out that the value 10DP is equivalent with heightPercentageToDP('1.6%') in an average mobile phone. So if you have a project with DP and want to change it to responsive, you can use this analogy as a guide :-)