Cannot set topic retention policy (error 500)

Describe the bug
I have a namespace with 4 day retention policy:
pulsar-admin namespaces create tenant/ns
pulsar-admin namespaces set-retention --size -1 --time 4d tenant/ns

I also have topic-level policy enabled:
# Enable or disable system topic
systemTopicEnabled=true

# The schema compatibility strategy to use for system topics
systemTopicSchemaCompatibilityStrategy=ALWAYS_COMPATIBLE

# Enable or disable topic level policies, topic level policies depends on the system topic
# Please enable the system topic first.
topicLevelPoliciesEnabled=true

When I attempt to turn off the retention policy for a single topic, I get either an error 504(Bad gateway) when going through pulsar-proxy or an error 500 if issuing the command from the broker with pulsar-admin:
./pulsar-admin topics set-retention -s 0 -t 0 tenant/ns/mytopic
00:20:36.472 [AsyncHttpClient-7-1] WARN  org.apache.pulsar.client.admin.internal.BaseResource - [http://localhost:8080/admin/v2/persistent/tenant/ns/mytopic/retention] Failed to perform http post request: javax.ws.rs.InternalServerErrorException: HTTP 500 Internal Server Error
HTTP 500 Internal Server Error

The result is the same when attempting to change size or time to any value, not just 0. There is nothing in the broker or pulsar-proxy logs to indicate the nature of this failure.  get-retention for the topic appears to work (returns null), and delete-retention appears to succeed, even though no retention policy is set for the topic, but I cannot be sure, since I cannot set the policy for the topic.
Expected behavior
Topic policy changed
Desktop (please complete the following information):

OS: n/a

Additional context
The build is at current HEAD for 2.7 branch, so 2.7.5-SNAPSHOT

I can help with this issue.

Thank you, @mattisonchao, please let me know if I can provide any further information.

@flowchartsman  I can not reproduce this issue at 2.7 branches.
Could you give me more detail to help me?

21:52:06.450 [pulsar-client-io-60-1] ERROR org.apache.pulsar.client.impl.ProducerImpl - [persistent://tenantname/namespacename/__change_events] [null] Failed to create producer: 112 request timedout after ms 30000
21:52:06.451 [pulsar-client-io-60-1] ERROR org.apache.pulsar.broker.admin.impl.PersistentTopicsBase - [null] Failed to perform setRetention on topic persistent://tenantname/namespacename/topicname
org.apache.pulsar.client.api.PulsarClientException$TimeoutException: 112 request timedout after ms 30000
        at org.apache.pulsar.client.impl.ClientCnx.checkRequestTimeout(ClientCnx.java:1076) ~[org.apache.pulsar-pulsar-client-original-2.7.5-SNAPSHOT.jar:2.7.5-SNAPSHOT]
        at org.apache.pulsar.client.impl.ClientCnx.lambda$channelActive$0(ClientCnx.java:201) ~[org.apache.pulsar-pulsar-client-original-2.7.5-SNAPSHOT.jar:2.7.5-SNAPSHOT]
        at io.netty.util.concurrent.PromiseTask.runTask(PromiseTask.java:98) [io.netty-netty-common-4.1.68.Final.jar:4.1.68.Final]
        at io.netty.util.concurrent.ScheduledFutureTask.run(ScheduledFutureTask.java:176) [io.netty-netty-common-4.1.68.Final.jar:4.1.68.Final]
        at io.netty.util.concurrent.AbstractEventExecutor.safeExecute(AbstractEventExecutor.java:164) [io.netty-netty-common-4.1.68.Final.jar:4.1.68.Final]
        at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:469) [io.netty-netty-common-4.1.68.Final.jar:4.1.68.Final]
        at io.netty.channel.epoll.EpollEventLoop.run(EpollEventLoop.java:384) [io.netty-netty-transport-native-epoll-4.1.68.Final.jar:4.1.68.Final]
        at io.netty.util.concurrent.SingleThreadEventExecutor$4.run(SingleThreadEventExecutor.java:986) [io.netty-netty-common-4.1.68.Final.jar:4.1.68.Final]
        at io.netty.util.internal.ThreadExecutorMap$2.run(ThreadExecutorMap.java:74) [io.netty-netty-common-4.1.68.Final.jar:4.1.68.Final]
        at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30) [io.netty-netty-common-4.1.68.Final.jar:4.1.68.Final]
        at java.lang.Thread.run(Thread.java:748) [?:1.8.0_312]
21:52:06.455 [pulsar-client-io-60-1] INFO  org.eclipse.jetty.server.RequestLog - <IP_ADDRESS> - - [09/Feb/2022:21:51:18 +0000] "POST /admin/v2/persistent/tenantname/namespacename/topicname/retention?authoritative=false HTTP/1.1" 500 1119 "-" "Pulsar-Java-v2.8.0" 48367
21:52:06.455 [pulsar-client-io-60-1] WARN  org.apache.pulsar.client.impl.ClientCnx - [id: 0xa662f391, L:/<IP_ADDRESS>:52114 - R:<IP_ADDRESS>/<IP_ADDRESS>:6650] 112 request timedout after ms 30000192.168.2

@flowchartsman
What is your concurrentLookupRequest config value? it seems some resources block cause request timeout.

maxConcurrentLookupRequest=50000

@flowchartsman   This PR may already fix these problems.
https://github.com/apache/pulsar/pull/14503
https://github.com/apache/pulsar/pull/14293

This appears to have been resolved by an update.


Can't build, missing signing key

Currently a normal build (mvn verify) fails as the build wants to sign the artifact. This makes no sense for local-only builds. Signing should only happen when actually deploying to a repository.

Thanks for the hint. Apparently I have to configure Maven so that signing is only executed during a deployment.
mvn package seems to work. Maybe that helps temporarily.

A typical Maven-like solution would be to move the signing into a Maven Profile (so you activate it manually using mvn -P or to bind the signing execution to the deploy Phase explicitly, so it runs at mvn deploy but not at mvn verify. 😃


Correct loadingController options parameter

Short description of what this resolves:
LoadingOptions in the loadingController takes "content" as a parameter instead of "message"
Changes proposed in this pull request:
Change "message" by "content"
Ionic Version: 4.x

Thank you!


VPS powered down

Summary
VPS powerdown and yaruo also ended with sighup.
/var/log/syslog/
Jul 11 00:55:30 qemu-ga: info: guest-shutdown called, mode: powerdown

I created a systemd service file to start the BOT even after rebooting.


User and Password request on public address

Hi , I use souliss connected to the local WiFi and from outside through a public IP ( noip.com ) . My question is if there is a possibility that the gateway requests a username and password when trying to log in from an IP outside the local network for safety.

Hi, in the Wiki in the page Security there is an explanation of the
Lockdown that you can use to avoid that others may access your gateway.
It will ban IP address that guess a connection.
Dario.

Great! Thank you Dario, this is a good option.
Luciano.


CB-12368: Fix permission check on Android

Platforms affected
Android
What does this PR do?
The plugin was checking whether camera permission was granted but then actually requested permission for external storage.
What testing has been done on this change?
Tested on Android 6 & 7.
Checklist

[X] Reported an issue in the JIRA database
[X] Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected.
[ ] Added automated test coverage as appropriate for this change.

Cordova CI Build has one or more failures.
Commit     - Link
Dashboard - Link

Builder Name
Console Output
Test Report
Device Logs

Windows 10  Store
Link
Link
Link

iOS 9.3
Link
Link
Link

iOS 10.0
Link
Link
Link

Android
Link
Link
Link

Cordova CI Build has completed successfully.
Commit     - Link
Dashboard - Link

Builder Name
Console Output
Test Report
Device Logs

Windows 10  Store
Link
Link
Link

iOS 9.3
Link
Link
Link

iOS 10.0
Link
Link
Link

Android
Link
Link
Link

Thanks, I've merged it.
BTW, how did you do to get READ_EXTERNAL_STORAGE permission asked? I get it granted and it never ask for it.

@jcesarmobile It repeatedly asked for "photos, medias, and files" permission (which I suppose is READ_EXTERNAL_STORAGE) each time getPicture() was called, but I observed this behavior only on Android 6.0. On Android 7.x I got asked only once too.


Add a general Dockerfile for poetry-managed python project

How about adding a default Dockerfile? I often need to check the process of installing poetry and pipx while dockerizing a service, a default general Dockerfile is a great help.
For example, like this:
FROM python:3.8-slim

ENV PYTHONUNBUFFERED=1
ENV PATH=$PATH:/root/.local/bin
WORKDIR /project

COPY . /project/

    # Install pipx
RUN python3 -m pip install --upgrade pip && \
    python3 -m pip install --user pipx --no-warn-script-location && \
    python3 -m pipx ensurepath && \
    # Install poetry
    apt-get update -y && \
    apt-get install curl -y && \
    curl -sSL https://raw.githubusercontent.com/python-poetry/poetry/master/get-poetry.py | python - && \
    . $HOME/.poetry/env && \
    poetry config virtualenvs.create false && \
    poetry install --no-dev --no-interaction --no-ansi

Sounds good to me. I thought of this. Just don't have enough time to do so ATM. Feel free to send me a PR. Notice that both poetry and pipenv are supported by this template, you might want to do something like https://github.com/Lee-W/cookiecutter-python-template/blob/master/{{cookiecutter.project_slug}}/tasks/env.py#L9. Also, I'd like to make this an option like "build_pypi_package" https://github.com/Lee-W/cookiecutter-python-template/blob/master/cookiecutter.json#L12 You can remove the undeeded Dockerfile through https://github.com/Lee-W/cookiecutter-python-template/blob/master/hooks/post_gen_project.py

Sounds good to me. I thought of this. Just don't have enough time to do so ATM. Feel free to send me a PR. Notice that both poetry and pipenv are supported by this template, you might want to do something like https://github.com/Lee-W/cookiecutter-python-template/blob/master/{{cookiecutter.project_slug}}/tasks/env.py#L9. Also, I'd like to make this an option like "build_pypi_package" https://github.com/Lee-W/cookiecutter-python-template/blob/master/cookiecutter.json#L12 You can remove the undeeded Dockerfile through https://github.com/Lee-W/cookiecutter-python-template/blob/master/hooks/post_gen_project.py


Fixes 'redirect_options' addition for 'redirect_to' in confirmations

In the following PR, redirect_options was added. However, there was an omission in the confirmation email section when errors occurred, so I have included it.
https://github.com/lynndylanhurley/devise_token_auth/pull/1599

Sorry for not noticing! After performing a 'rebase master', it looks like the tests have passed.


Spatial::rotate() can scale objects!?

As a new user, I was giving rotate() wrongly-formed vectors (they weren't normalized), except instead of a crash or warning, my object shrank, which was very unexpected.
I would have expected auto-normalization, a warning, or an error, but not scaling!
Example:
Repeatedly call rotate(Vector3(0.2,0,0), 0.05) and watch your object spin and shrink.
Arch Linux - Godot 2.1.3
See also: #8113

It can shrink even if you feed vectors coming from the basis of your object. As I understood earlier, it's because there is no auto-normalization so float precision or not-normalized vectors will cause the transformation matrix to drift in an invalid state (remember maths, rotation uses the same matrix cells as scale does).

As I mentioned in #8113, the only proper way (without relying on the user) I can think of to fix this is to split the basis into rotation and scaling parts, and store them separately.
Under the hood, the rotation part could be stored as a quaternion, whose normalization is easier than orthogonalization of matrix, and the scaling part as a vector. Of course, these eventually need to be converted to a matrix (on write?) such that it can be used in rendering. Also, if we go to that route, we'd need to disable write access to basis matrix.

Uh, what's stopping you from enforcing normalization on whatever vectors are passed to Spatial::rotate?

That'd work only if you know beforehand that the object scaling is (1,1,1). Otherwise, you need the scaling information from somewhere else because you can't know whether that scaling is from numerical errors or intentional.

@cooperra If you haven't done it already, can you please try using a proper axis vector, that is Vector3(0.2,0,0).normalized(), and see how fast this happens?

I would have expected auto-normalization, a warning, or an error, but not scaling!

I added errors for unnormalized vectors in 3.0 for debug builds for this kind of mistakes.
To be clear, you'd eventually see this kind of scaling even with a normalized vector, because of the precision errors, so this needs to be address properly somehow.

@cooperra If you haven't done it already, can you please try using a proper axis vector, that is Vector3(0.2,0,0).normalized(), and see how fast this happens?

That's pretty much what I ended up doing, except with my game's vector instead of the example's. It worked perfectly.

Ah, OK. Apparently it happens slower than I thought. Feel free to close this issue, I'll open another one for separating basis into rotation and scaling.


Replaces usage of device UUID with device name

Fixes #757
The issue on Circle CI was fixed :tada:
I had to make a few changes to include not only the device name but also the OS version. Not sure if I took the best approach, suggestions are welcome :wink:

Another idea for how to make this run a little smoother is to swap out the build test here for a build-for-testing test. I have noticed better results that way as it delays the call to open the simulator until the test target has already been built. Therefore you are less likely to have a timeout issue.

For iPhones using the name might not always work. If the simulator has a watch tied to it the OS and the name will not be unique. You can have more than one. I'm guessing this works for iPhone 6s because the watches are now tied to an iPhone 7 simulator in circle.
Let's just keep that in mind if something breaks in the future.


Pull container images from ECR Public instead of Docker Hub

Issue #, if available:
As an Amazon best practice, CI should pull container images from ECR Public instead of Docker Hub.
Description of changes:
This change pulls the base container images from ECR Public.
Testing performed:
CI is successful
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

LGTM
nit: Can we also self-host the zot image? This should remove all of our non-public ECR image dependencies. Not a blocker, though, just might be nice to lump it in this PR.

Looked into this. Unfortunately there does not appear to be a ECR Public image for ZOT so there might be more effort involved. Let's split this and follow-up to not block these changes.


ValueError: Expected more than 1 value per channel when training, got input size torch.Size([1, 128])

When I try to use scvi to train the model, I got the following error. Could you please help me to solve this problem? Thank you for your time!

ValueError("Expected more than 1 value per channel when training, got input size {}".format(size))
ValueError: Expected more than 1 value per channel when training, got input size torch.Size([1, 128])
My code:

scvi-tools version 1.0.4

Hi, this unfortunately occurs when the last batch of the training set contains one observation, leading to the batch norm layer complaining. You can solve this by passing in a batch_size to the train method that is different than the default one.

Hi, this unfortunately occurs when the last batch of the training set contains one observation, leading to the batch norm layer complaining. You can solve this by passing in a batch_size to the train method that is different than the default one.

It works. Thank you so much!


Exception on fedora 38 / Thoughts on Gir.Core

Hi,
I tried your project on a current fedora 38 with dotnet 6 / 7 installed. Unfortunately it did not work out. It looks like the code in GtkSharp is hard coding an old webkitgtk library.
Unhandled exception. System.TypeInitializationException: The type initializer for 'WebKit.WebView' threw an exception.
 ---> System.DllNotFoundException: Webkit: libwebkit2gtk-4.0.dll, libwebkit2gtk-4.0.so.37, libwebkit2gtk-4.0.dylib, libwebkit2gtk-4.0.0.dll
   at GLibrary.Load(Library library)
   at WebKit.WebView..cctor()

As this project looks quite interesting, I wonder what your thoughts are in regard to base it on Gir.Core? Gir.Core supports Gtk4 and current WebKitGtk versions. It follows the binaries inside the current gnome Flatpak SDK. This allows to create Flatpaks  which can be distributed via Flathub.
But the project and API is not complete or guaranteed to be stable, so it is quite possible that the APIs you are relying on are not yet generated or are even not supported anymore as there was some API break on WebKitGTK side.
As I'm actively working on Gir.Core, I would be interested in making the needed API available to you.
I created a small issue on my side https://github.com/gircore/gir.core/issues/931 to keep your project in my mind in case you are not interested as I think it could benefit a lot of users. If you like, the project could even be part of Gir.Core itself as it looks like that not much code is needed maintenance wise.

Hi,
Thanks for reaching out.  Yes, I am interested in getting this to work with Gir.Core.  You can see the APIs used in the following files:
https://github.com/JinShil/BlazorWebView/blob/main/BlazorWebKit/Gtk.cs
https://github.com/JinShil/BlazorWebView/blob/main/BlazorWebKit/WebKit.cs
https://github.com/JinShil/BlazorWebView/blob/main/BlazorWebKit.Test/Program.cs
https://github.com/JinShil/BlazorWebView/blob/main/BlazorWebKit/BlazorWebView.cs
Basically, this project needs a Gtk Window, a WebKitGTK WebView, and a couple of JavascriptCore functions which may or may not be available through WebKitGTK.
Personally, I would prefer to not have to maintain this project.  If you want to make it part of Gir.Core, please do, and, as long as it does what I need, I'll use yours instead.

Hi,
good to hear that you think it would make sense to base your project on Gir.Core 👍
My comment was meant a bit like a request to contribute as I use my coding capacity to improve the generator itself as there is a lot of work ahead. Probably for more than a year.
As support for blazor apps is kind of optional I would probably work on it in version 0.13.0 and currently I'm working on 0.5.0 which still several weeks away. See milestones.
In case you don't have time to contribute don't worry I'm here for the long run. Someday I will be working on 0.13.0.

I think I could contribute if I know what to do.  What did you have in mind?
I suppose I could add reference to your Nuget packeages and see if I could get project to work with what's currently in Gir.Core.  Does that sound like a good first step?

If what you meant by "request to contribute" was a request for you to provide a pull request to this repository, then please do.  That would be most welcome.

Unfortunately I don't want to set my priories to support blazor apps before 0.13.0 as I already committed to the other needed changes for other users.
Perhaps everything needed is already there. From a first glance it looks like JavaScript/ WebKit should be fine. From GTK side it looks like "g_memory_input_stream_new_from_data" is missing but you could probably use NewFromBytes or in case it does not work out try some workaround via the lower level internal api NewFromData. I'm available on matrix for support on those things.
If you could report any missing APIs I would try to add them to the next release.
If you like you could contribute a new project inside Gir.Core via a PR and thus depend on current Gir.Core main branch and get the APIs before an official release. The code for the PR could be in the extensions folder of Gir.Core: https://github.com/gircore/gir.core/tree/main/src/Extensions
A sample could be in the samples folder: https://github.com/gircore/gir.core/tree/main/src/Samples
If the PR is merged you are welcome to improve / adopt the code if needed. I would make sure that the your sample is working throughout new releases to ensure that all features of Gir.Core work like advertised for the future.
If you decide to contribute directly to Gir.Core and not create your own package there are some requirements: the code must be licenseable under MIT or LGPL (in case I want to switch license) and the code style should be similar to the one already used in Gir.Core. So there will probably be feedback from me in regard to your code to be able to accept it as I would need to ensure it works for the coming years.
What do you think?

For now, I'll try to port this project to Gir.Core and let you know if I encounter any problems.
In terms of licensing, some of the code I'm using in this project was taken from Steve Sanderson's Web Window which is Apache Licensed, and that prompted me to choose the same license for this project.
For now, I'll just try to port this to Gir.Core and if successful, I'll work out the other stuff later.

How would I do the following with GirCore?
g_signal_connect(WebView.UserContentManager.Handle, "script-message-received::webview",
                Marshal.GetFunctionPointerForDelegate(HandleWebMessageDelegate),
                nint.Zero);

See the sample: https://github.com/gircore/gir.core/blob/7ac07dd8d53638818b3550faa41d46a0e70ee3b0/src/Samples/WebKit-6.0/JavascriptCallback/Program.cs#L26
The last parameter is the detail argument (in your case "webview").

@czirok I tried testing on one of my ARM devices, but unfortunately they are running Debian Bullseye and there is libgtk4 library yet.  I won't be able to test ARM until I set them up with a more recent OS which will likely not happen for a few more months.
I tried testing your .NET 7 test in a Debian Bookworm VM and I got this:
(process:2582): Gtk-WARNING **: 10:33:33.553: Unknown key gtk-modules in /home/mike/.config/gtk-4.0/settings.ini

(process:2582): Adwaita-WARNING **: 10:33:33.588: Using GtkSettings:gtk-application-prefer-dark-theme with libadwaita is unsupported. Please use AdwStyleManager:color-scheme instead.

(BlazorWebViewTestNet7:2582): GLib-GObject-WARNING **: 10:33:33.715: g_object_get_is_valid_property: object class 'BlazorWebKitBlazorWebView' has no property named 'web-context'
UnhandledException - unhandled exception: System.NotSupportedException: Unable to extract the value for type ''. The type (id: 0) is unknown.
   at GObject.Value.CheckComplexTypes(UIntPtr gtype)
   at GObject.Value.Extract()
   at GObject.Value.Extract[T]()
   at GObject.Property`2.Get(K obj)
   at WebKit.WebView.get_WebContext()
   at BlazorWebKit.WebViewManager..ctor(WebView webView, IServiceProvider serviceProvider) in /home/mike/GitHub/czirok/BlazorWebView/BlazorWebKit/BlazorWebView.cs:line 46
   at BlazorWebKit.BlazorWebView..ctor(IServiceProvider serviceProvider) in /home/mike/GitHub/czirok/BlazorWebView/BlazorWebKit/BlazorWebView.cs:line 18
   at Program.<>c.<<Main>$>b__0_0(Application sender, EventArgs args) in /home/mike/GitHub/czirok/BlazorWebView/BlazorWebViewTestNet7/Program.cs:line 36
   at GObject.Signal`1.<>c__DisplayClass11_0.<Connect>b__0(Value returnValue, Value[] parameters)
   at GObject.Closure.InternalCallback(IntPtr closure, IntPtr returnValuePtr, UInt32 nParamValues, ValueData[] paramValuesData, IntPtr invocationHint, IntPtr userData)

Hello @JinShil
I installed Debian Bookworm. x64 VM, netinstall iso + Gnome. I don't know what could be wrong, @badcel  can tell you.

Here is the package list that includes the word gtk:
gir1.2-gtk-3.0:amd64                    3.24.37-2
gir1.2-gtk-4.0:amd64                    4.8.3+ds-2
gir1.2-gtkclutter-1.0:amd64             1.8.4-4+b1
gir1.2-gtksource-4:amd64                4.8.4-4
gir1.2-javascriptcoregtk-4.0:amd64      2.40.5-1~deb12u1
gir1.2-javascriptcoregtk-4.1:amd64      2.40.5-1~deb12u1
gstreamer1.0-gtk3:amd64                 1.22.0-5+deb12u1
gtk-update-icon-cache                   3.24.37-2
gtk2-engines-pixbuf:amd64               2.24.33-2
ibus-gtk:amd64                          1.5.27-5
ibus-gtk3:amd64                   1.5.27-5
ibus-gtk4:amd64                         1.5.27-5
libcanberra-gtk3-0:amd64                0.30-10
libcanberra-gtk3-module:amd64           0.30-10
libcheese-gtk25:amd64                   43.0-1
libclutter-gtk-1.0-0:amd64              1.8.4-4+b1
libcolord-gtk4-1:amd64                  0.3.0-3.1
libdbusmenu-gtk3-4:amd64                18.10.20180917~bzr492+repack1-3
libgnome-autoar-gtk-0-0:amd64           0.4.3-1
libgtk-3-0:amd64                        3.24.37-2
libgtk-3-bin                            3.24.37-2
libgtk-3-common                         3.24.37-2
libgtk-4-1:amd64                        4.8.3+ds-2
libgtk-4-bin                            4.8.3+ds-2
libgtk-4-common                         4.8.3+ds-2
libgtk2.0-0:amd64                       2.24.33-2
libgtk2.0-bin                           2.24.33-2
libgtk2.0-common                        2.24.33-2
libgtk3-perl                            0.038-3
libgtkmm-3.0-1v5:amd64                  3.24.7-1
libgtksourceview-4-0:amd64              4.8.4-4
libgtksourceview-4-common               4.8.4-4
libgtksourceview-5-0:amd64              5.6.2-1
libgtksourceview-5-common               5.6.2-1
libjavascriptcoregtk-4.0-18:amd64       2.40.5-1~deb12u1
libjavascriptcoregtk-4.1-0:amd64        2.40.5-1~deb12u1
libjavascriptcoregtk-6.0-1:amd64        2.40.5-1~deb12u1
libnma-gtk4-0:amd64                     1.10.6-1
libportal-gtk3-1:amd64                  0.6-4
libportal-gtk4-1:amd64                  0.6-4
libreoffice-gtk3                        4:7.4.7-1
libwebkit2gtk-4.0-37:amd64              2.40.5-1~deb12u1
libwebkit2gtk-4.1-0:amd64               2.40.5-1~deb12u1
libwebkitgtk-6.0-4:amd64                2.40.5-1~deb12u1
software-properties-gtk                 0.99.30-4
transmission-gtk                        3.00-2.1+b1
xdg-desktop-portal-gtk                  1.14.1-1
xdg-user-dirs-gtk                       0.11-1

From the message: (BlazorWebViewTestNet7:2582): GLib-GObject-WARNING **: 10:33:33.715: g_object_get_is_valid_property: object class 'BlazorWebKitBlazorWebView' has no property named 'web-context' I would assume that you use an old webkitgtk library. There was some API break in webkitgtk. Gir.Core expetcts libwebkitgtk-6.0.so.4 to be available, which probably is in package libwebkitgtk-6.0-4.
@czirok I tested your code and the demo worked :+1:

I believe we've fulfilled the gist of this request. If there is anything else that needs to be done or discussed, please reopen, or open a new issue.  Thank you.


Could not set unknown property 'manifestPlaceholders' for object of type com.android.build.gradle.AppExtension.

I upgraded gradle plugin to newest (2.2.0-alpha3) and gradle to 2.14, but still keep getting the error.
Could you please either add instructions how to make this work out of Android Studio or not use manifestPlaceholders?
Thanks.

Closing. I put manifestPlaceholder by mistake under android, not under defaultConfig.


Atomgraphfeat

What does this implement/fix? Explain your changes.
This is an initial version of our 49 features using in classical Graphs neural networks atomic features.
it's 4x faster compare to python version
Any other comments?
We can decide to add more / or custom it in the future

@thegodone It looks like a lot of extra stuff got added here.   .DS_Store, file line endings.
Would it be possible to just have the stuff specific to AUTOFEAT?

I can make a new pull with the 5 main files.


Might consider documenting how to use the results

Customers and support folks might easily assume that these rules generate alerts. They are presented as analytic rules among other rule types with a primary purpose of generating alerts. Also the whole flighting\production tuning topic suggests alert monitoring. That seems like unnecessary overhead for log data...who tunes log data?  I recommend updating the documentation to clearly state there are no actual alerts to avoid confusion. It would also be beneficial to provide a hunter query rule and a workbook to help customers track the results. Customers may not know what to do with this data. The rules themselves should have some instructions/clarification in the description as well since nobody reads these days.
Thanks, Andrew

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

ID: 16c724fa-83ff-b7c5-481d-169fe05d6b9a
Version Independent ID: 617297d6-13c3-52f1-746d-353f77479f45
Content: Work with anomaly detection analytics rules in Azure Sentinel
Content Source: articles/sentinel/work-with-anomaly-rules.md
Service: azure-sentinel
Sub-service: azure-sentinel
GitHub Login: @yelevin
Microsoft Alias: yelevin

@AndrewBlumhardt
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Hi @AndrewBlumhardt, thanks for sending your feedback in to Microsoft docs!
You have some great suggestions here, and I've opened a backlog item for ourselves to take a fresh look at this description with your comments in mind.
We'll also pass on your request for the descriptions in the rules themselves but can also recommend that you use the community resources linked from Comment on our blogs and forums to send feedback.
I'm going to close this issue for now, but feel free to continue commenting here as needed. Thanks for your contribution!
#please-close


Welp, I can't add Blooket coins, it says

https://github.com/Jude-Gideon/Blooket/Global for working tokens!

i like your hacks jude gideon

Thanks for using them!

Ooh, thanks for helping Elmo out I can use these against cookie monster!

?


chore(Topology/PartialHomeomorph): rename type variables

Greek letters are dead, long live X, Y and Z. Same procedure as in previous renames.

Note for reviewers

ignore the first three commits; they are from a dependent PR which has now been merged
the next six commits belong to this PR and can be reviewed individually
the remaining three commits are just master; I'd squash them if I knew how...
[x] depends on: #9630
[x] depends on: #9633

Otherwise LGTM. Thanks!
bors d+

Thank you for the review.
bors r+


14.1 Implement the possibility to add attributes to a passport that are not initially set by a schema.

Implemented in both the program an the API


The Indefinite Article

Played through and enjoyed The Quest for the Holy Grail: found it true to the film and fun to play.
The only quibble I have is that many of the names for monsters and items seem to include the indefinite article: thus "The Black Knight" or "The Holy Hand Grenade" or "an inscription".  While "The Black Knight" is canonically correct, for most of the others, the article is just an annoyance that means that a command like "read inscription" fails.  Can I suggest that deleting the indefinite article from most of the descriptions would help with gameplay?

Articles were added to some artifacts and monsters to make the text read more smoothly in some places. However, you're right that it was interfering with commands. I tend to use a lot of abbreviated commands when I play (e.g., AT BL for ATTACK THE BLACK KNIGHT) and the articles were inconvenient in that situation.
I just released an update which changes the logic on the way articles are handled before artifact and monster names. You can now ignore the article name completely when typing. E.g., "ATTACK BLACK" will still work even if the word "the" appears before his name.
FYI, you can also do partial word matching at the end of artifact and monster names. e.g., "OPEN CHE" and "OPEN ST" both work for opening an artifact named "chest". However, if the name has more than one word, it only works on the beginning of the first word and the end of the last word. So, "ATTACK GI" wouldn't work for "ATTACK FROST GIANT" but "ATTACK FR" or "ATTACK NT" would.


Add support for source target quantity when reconciling shipment

Update data model and test data
Add source/targetQuantity to GraphQL API and update business logic
Show diff on failure when running pre-commit in CI

 2.0 Add support for source/target quantity when reconciling shipment


Running npm test for specific module/suite is running all tests

[X] System Information

[X] Browser type and version: Chrome Version 71.0.3578.98 (Official Build) (64-bit)
[X] OS type and version: MacOS X Mojave 10.14.2
[X] Node version: 6.5.0
[X] Any error messages that may be in the console where you ran npm start: None
[X] Any error messages in the JS console: None

[X] Describe the bug
When trying to run specific module tests, it is running all test suites instead.

[X] To Reproduce
Steps to reproduce the behavior:

Go to the root directory of the project.
Run: npm test dealer
Note the output which says:

./node_modules/.bin/jest --no-cache --watch $2

Note that all test suites ran:

Test Suites: 1 failed, 2 passed, 3 total
Tests:       8 failed, 24 passed, 32 total
Snapshots:   0 total
Time:        2.083s
Ran all test suites related to changed files.

[X] Expected behavior
When running for a specific suite, it should obviously only run that suite. You can hack this by giving a dummy 1st argument to the "test" command: npm test fjksdlfjds dealer

This results in only the dealer suite being ran:

[X] Screenshots (optional)
Screenshot of all tests being run:

[X] Additional context
See this issue which talks about the same thing near the bottom. I think the issue was closed addressing only the initial problem. Also see this possibly related issue.

Perhaps the $2 argument in scripts/test/index.sh is supposed to be $1?

Closing this issue because we've released a new version of the TypeScript course, where this is not a problem anymore


Adding only 2 videos for testing the appearance

🛠 Fixes Issue #359

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

Please review my PR @arpit456jain @singhmansi25 @KKhushhalR2405 @raghavTinker

@muhammadgalhoum add a video

@muhammadgalhoum  ur branch is 48 commits' behind i can't merge it until its update
because due to this we lost latest changes before also

@muhammadgalhoum add a video

@muhammadgalhoum ur branch is 48 commits' behind i can't merge it until its update because due to this we lost latest changes before also


headlines: init at 0.7.2

Description of changes
https://gitlab.com/caveman250/Headlines
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

are we missing a dependency. An attempt to play a video message ends in:
(headlines:633001): GStreamer-Play-ERROR **: 22:08:22.935: GstPlay: 'playbin' element not found, please check your setup
fish: Job 1, 'headlines' terminated by signal SIGTRAP (Trace or breakpoint trap)

GstPlay: 'playbin' element not found

Fixed but another error occurred. Have no idea how this can be fixed:
** (headlines:944317): CRITICAL **: 21:01:26.183: Error from element /GstPlayBin:playbin/GstPlaySink:playsink/GstBin:abin/GstPulseSink:pulsesink0: The stream is in the wrong format.
The stream is in the wrong format.
../gst-libs/gst/audio/gstaudiobasesink.c(1117): gst_audio_base_sink_wait_event (): /GstPlayBin:playbin/GstPlaySink:playsink/GstBin:abin/GstPulseSink:pulsesink0:
Sink not negotiated before eos event.

The latest version always crashes for me after loading the gui.
** (process:1648300): WARNING **: 18:01:03.441: Can't set the global locale to the user's preferred locale.
   locale::facet::_S_create_c_locale name not valid
   The environment variable LANG may be wrong.

(process:1648300): Gtk-WARNING **: 18:01:03.443: Locale not supported by C library.
	Using the fallback 'C' locale.

(headlines:1648300): Gtk-WARNING **: 18:01:03.583: Allocating size to AdwViewSwitcherBar 0x17b62f0 without calling gtk_widget_measure(). How does the code know the size to allocate?

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation height too small. Tried to allocate 899x0, but AdwViewSwitcherBar 0x17b62f0 needs at least 899x45.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation height too small. Tried to allocate 899x0, but GtkActionBar 0x17b6480 needs at least 899x45.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation height too small. Tried to allocate 899x0, but GtkRevealer 0x17c8290 needs at least 899x45.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation height too small. Tried to allocate 899x0, but GtkCenterBox 0x17b6610 needs at least 899x45.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation height too small. Tried to allocate 0x-1, but GtkBox 0x17c07e0 needs at least 0x0.

(headlines:1648300): Gtk-WARNING **: 18:01:03.583: gtk_widget_size_allocate(): attempt to allocate GtkBox box 0x17c07e0 with width 0 and height -1

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation height too small. Tried to allocate 320x-1, but AdwViewSwitcher 0x17b67a0 needs at least 320x44.

(headlines:1648300): Gtk-WARNING **: 18:01:03.583: gtk_widget_size_allocate(): attempt to allocate AdwViewSwitcher viewswitcher 0x17b67a0 with width 300 and height -1

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation height too small. Tried to allocate 0x0, but AdwViewSwitcherButton 0x17d4260 needs at least 0x44.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation width too small. Tried to allocate 0x0, but AdwViewSwitcherButton 0x17d4260 needs at least 16x0.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation height too small. Tried to allocate 0x0, but GtkStack 0x179ef90 needs at least 0x44.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation width too small. Tried to allocate 0x0, but GtkStack 0x179ef90 needs at least 16x0.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation height too small. Tried to allocate 0x0, but AdwViewSwitcherButton 0x17d4470 needs at least 0x44.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation width too small. Tried to allocate 0x0, but AdwViewSwitcherButton 0x17d4470 needs at least 16x0.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation height too small. Tried to allocate 0x0, but GtkStack 0x17d99b0 needs at least 0x44.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation width too small. Tried to allocate 0x0, but GtkStack 0x17d99b0 needs at least 16x0.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation height too small. Tried to allocate 0x0, but AdwViewSwitcherButton 0x17d4680 needs at least 0x44.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation width too small. Tried to allocate 0x0, but AdwViewSwitcherButton 0x17d4680 needs at least 16x0.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation height too small. Tried to allocate 0x0, but GtkStack 0x17ea3f0 needs at least 0x44.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation width too small. Tried to allocate 0x0, but GtkStack 0x17ea3f0 needs at least 16x0.

(headlines:1648300): Gtk-CRITICAL **: 18:01:03.583: Allocation height too small. Tried to allocate 0x-1, but GtkBox 0x17c0960 needs at least 0x0.

(headlines:1648300): Gtk-WARNING **: 18:01:03.583: gtk_widget_size_allocate(): attempt to allocate GtkBox box 0x17c0960 with width 0 and height -1

(headlines:1648300): Gtk-WARNING **: 18:01:03.604: Allocating size to AdwViewSwitcherBar 0x17b62f0 without calling gtk_widget_measure(). How does the code know the size to allocate?

** (headlines:1648300): WARNING **: 18:01:03.663: Json::CharReader error: * Line 1, Column 1
  Syntax error: value, object or array expected.

Segmentation fault (core dumped)

I have fiddled a bit with the package definition.
I fulfilled all pkg-config complaints of missing packages but the app still crashes (maybe it receives malformed json from the api).
Here is what I used:
{ lib
, stdenv
, cmake
, pkg-config
, libmicrohttpd
, curl
, openssl
, jsoncpp
, libxml2
, ffmpeg
, gst_all_1
, youtube-dl
, boost
, websocketpp
, libadwaita
, gtkmm4
, libsecret
, fetchFromGitLab
, makeWrapper
, xdg-utils
, pcre
, pcre2
, util-linux
, libselinux
, libsepol
, fribidi
, libgcrypt
, libthai
, libgpg-error
, libdatrie
, gnutls
, xorg
, libtasn1
, p11-kit
}:

stdenv.mkDerivation rec {
  pname = "headlines";
  version = "0.7.2";

  src = fetchFromGitLab {
    owner = "caveman250";
    repo = "Headlines";
    rev = version;
    sha256 = "sha256-wamow0UozX5ecKbXWOgsWCerInL4J0gK0+Muf+eoO9k=";
  };

  nativeBuildInputs = [
    cmake
    pkg-config
    makeWrapper
  ];

  buildInputs = [
    pcre
    pcre2
    p11-kit
    libtasn1
    gnutls
    xorg.libXdmcp
    fribidi
    util-linux
    libselinux
    libsepol
    libgcrypt
    libthai
    libgpg-error
    libdatrie
    libmicrohttpd
    curl
    openssl
    jsoncpp
    libxml2
    ffmpeg
    youtube-dl
    boost
    websocketpp
    libadwaita
    gtkmm4
    libsecret
    gst_all_1.gstreamer
  ] ++ (with gst_all_1; [
    gstreamer
    gst-libav
    gst-plugins-base
    (gst-plugins-good.override { gtkSupport = true; })
    gst-plugins-bad
  ]);

  cmakeFlags = [
    "-DCMAKE_BUILD_TYPE=RelWithDebInfo"
    "-DDIST_BUILD=ON"
  ];

  postFixup = ''
    wrapProgram "$out/bin/headlines" \
      --prefix PATH : "${lib.makeBinPath [ xdg-utils youtube-dl ffmpeg ]}" \
      --prefix GST_PLUGIN_SYSTEM_PATH_1_0 : "$GST_PLUGIN_SYSTEM_PATH_1_0"
  '';

  meta = with lib; {
    description = "GTK4 / Libadwaita Reddit client written in C++";
    homepage = "https://gitlab.com/caveman250/Headlines";
    license = licenses.gpl3Plus;
    platforms = platforms.linux;
    maintainers = with maintainers; [ chuangzhu ];
  };
}

A lot of dependencies, maybe should be reported upstream and ask for a dependency cleanup..

I have dug out some information about my app crashing and reported upstream: https://gitlab.com/caveman250/Headlines/-/issues/92 ... seems to be an issue with me not being login and the app requesting a login-only resource.

I fulfilled all pkg-config complaints of missing packages

These are just Requires.private dependencies. Since we are not doing static linking here, these can be safely ignored.

The latest version always crashes for me after loading the gui.

I can't reproduce.

Ok, I have tested again with nix-review. This time no crashes but videos would still not play at all.
** (headlines:109595): CRITICAL **: 18:18:46.488: Error from element /GstPlayBin:playbin/GstPlaySink:playsink/GstBin:abin/GstPulseSink:pulsesink2: The stream is in the wrong format.
The stream is in the wrong format.
../gst-libs/gst/audio/gstaudiobasesink.c(1117): gst_audio_base_sink_wait_event (): /GstPlayBin:playbin/GstPlaySink:playsink/GstBin:abin/GstPulseSink:pulsesink2:
Sink not negotiated before eos event.

I dug a bit into the problem and seems that it's because GStreamer cannot parse M3U files with multiple resolution options. Anyway, I don't think it is a bug of packaging. I'll mark this PR ready to merge.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/727


Add support for ed25519

I would like to make a pull request.

@zensh Happy to receive a PR. I asked @benjizhai to bring the repo in sync with the current draft of the interface spec and add Ed25519 but we'd be happy to collaborate with you.

Closed by #3


Update manifest.json

As others have commented, the manifest file need a "version" parameter as of version 2021.6 of Home Assistant. Please add this or the integration will not load!

I've pushed this "patch" months ago :D So it's a duplicate.

close with #58


Logic to convert packages into artifact names and namespaces

What is it?
For example, this model grabs NPM packages
select distinct
  abp.project_source,
  abp.project_name,
  sbom.package,
  sbom.package_source
from `ossd.sbom` as sbom
join `oso.artifacts_by_project_v1` as abp
  on
    abp.artifact_namespace = lower(sbom.artifact_namespace)
    and abp.artifact_name = lower(sbom.artifact_name)
join `oso.projects_by_collection_v1` as pbc
  on abp.project_name = pbc.project_name
where
  abp.artifact_source = 'GITHUB'
  and sbom.package_source = 'NPM

and here is some quick logic to extract naming fields:
def extract_artifacts(pkg):
    pkg = pkg.replace('../','').replace('./','')
    namespace = pkg
    name = pkg
    if '/' in pkg:
        splt = pkg.split('/')
        if len(splt) > 2:
            if 'aztec-packages' in pkg:
                namespace = splt[0]
                name = splt[1]
            else:
                return (None, None)
        else:
            namespace, name = splt
        namespace = namespace[1:]
    return (namespace, name)

See also here


more bitwise trickery

Per this comment from @Th0rN13. According to my microbenchmarks, this is indeed a (probably very modest!) performance win

LGTM
I thought the V8 engine would make the same optimizations for powers of 2, but I see that this PR is around 0.3 seconds faster for 100MB of input on node 12, and around 0.75 seconds faster on older JS engines.
In the original bit set proposal I was trying to determine what the maximum number of bits that could be used in integer bit operations on JS engines. Since a JS double (IEEE 754) has a 53 bit significant I was hoping it would be higher than 32. Not the case.


advent: display leaderboards as a colour image

uses pillow to render leaderboards into an image with a style very close to the website. inline display in the discord app is a bit smaller than i would've liked but alas. it should be much better on mobile at least.
text format is kept alongside the image since it's probably nice to have. maybe this should be behind an argument, but there are already so many.
also added url to website into the reply message
implementation has some weaknesses:

unicode multi-character sequences and emoji aren't rendered
relies very much on a monospace font for alignment
it is also a touch slow, maybe
the resolution on mobile is sometimes bad, depending on how discord scales the thumbnail

let me know if this is something you want.
OR, just inline the leaderboard into the message. it's only done as an attachment because slack let you preview text files on desktop and mobile.

I hate to well-actually, because this is uber cool, but it would probably be a better UX experience to have the boards be an embed with fancy discord interactions (e.g. buttons to scroll down or scroll up)?
I'll let others chime in on what they want AoC to "be" into the future though, because I don't see anyone jumping to write alternatives either - it would be silly to veto a good change just to make way for a "better" version that might not ever get written.

A Discord embed is probably a good way to modernise it. I don't think full pagination is good (way too many clicks), but maybe something like showing the first 20 rows in an image, then a button to expand to the full image. From testing, this is comfortable to read on Discord and mobile.
I do like the images (I just think they're neat) so I'd like to make them work, but also open to feedback

yippee

changes in https://github.com/UQComputingSociety/uqcsbot-discord/commit/344c0f6f7c191cf2e3adc7d61c66528a1d3f3aa2. re fonts, i wanted to have the most supported glyphs. i think this is import important than matching the website exactly. some comparisons: noto ibm sourcecodepro.
noto mono is the most extensive, but it does break the consistent monospace width sometimes. luckliy for us, this is not a problem since the only non-ascii characters are at the far right of each line (discord display names, example), so everything stays aligned.

after https://github.com/UQComputingSociety/uqcsbot-discord/pull/212/commits/1fd826d90ab9bfb285fdb83a51a28ccb271ecd04:
                                  Total Star Total Star            Local
      Star 1   Star 2  Both Stars   1 Time     2 Time   Total Time Order
  1)  0:01:10  0:01:00    0:02:10    0:01:10    0:01:00    0:02:10   316 katrinafyi
  2)  0:01:25  0:01:52    0:03:17    0:01:25    0:01:52    0:03:17   314 Cameron Aavik
  3)  0:02:19  0:00:58    0:03:17    0:02:19    0:00:58    0:03:17   312 bradleysigma
  4)  0:03:05  0:01:33    0:04:38    0:03:05    0:01:33    0:04:38   310 b-paul

this can be separated from the bigger pr if you want

re fonts, i wanted to have the most supported glyphs. i think this is import important than matching the website exactly.

Ah, yeah. I agree, that is more important. Shame pillow doesn't support fallback fonts


[K3s] Promote etcd to stable

Request Summary:

v1.19.5+k3s1 and newer promoted etcd to stable (removed from experimental). The docs need to indicate that etcd is no longer experimental as of v1.19.5+k3s1. This release and future in the 1.19 line and on have etcd as stable.
Details:

Small changes to call out this change in etcd support status.

Note to self: Area to edit is https://github.com/rancher/docs/blob/master/content/k3s/latest/en/installation/ha-embedded/_index.md


compiling error

hi, i have a problem with compiling/upload in my arduino, i'ved tried several ways like uninstall and reinstall arduino ans etc, but everytime i compile it's always turns out like this
fork/exec D:\Arduino\hardware\tools\avr/bin/avr-g++.exe: Access is denied.
Error compiling for board Arduino/Genuino Uno.

thank u

This is not at all an appropriate place for this. Post on the Arduino forum and I'm sure we'll have it solved in no time:
http://forum.arduino.cc/


[cli] expand help text for --dev and --base options

What does this PR do?
Improve some cli option help texts
Motivation
I was confused by these help texts so I thought I'd clarify them a bit based on what I now know.
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

@FlorentClarret can we merge despite the checks? I've updated the PR title btw

@FlorentClarret can we merge despite the checks? I've updated the PR title btw

yup!


cubieboard2 11.1-RELEASE build fails to boot with "No valid device tree blob found!"

drops into loader> prompt. Looks exactly like #186. I think it's caused by the new u-boot-sunxi-with-spl.bin 2017.01 u-boot.
I've ended up with this in my /boot/loader.rc.local:
load /boot/kernel/kernel
load -t dtb /boot/dtb/cubieboard2.dtb

Should this file be added to the cubieboard2 (and probably the rest of the allwinner boards) overlay?

You should ask on the freebsd-arm mailing list and see what those folks recommend.
More likely, recent changes to the U-Boot port have changed the expected DTB file name.  I know Warner and other FreeBSD folks have been trying to make the various U-Boot ports more consistent and some of the file names have been changing around in the process.

Same with my Orange Pi Plus 2, just that I have to do load -t dtb /boot/dtb/orangepi-plus-2e.dtb instead.

I've looked at the official image (FreeBSD-11.1-RELEASE-arm-armv6-CUBIEBOARD2.img.xz).
Attached the console log. It is loading /boot/dtb/sun7i-a20-cubieboard2.dtb and proceeding correctly.
I've removed my  /boot/loader.rc.local and hard-linked  /boot/dtb/sun7i-a20-cubieboard2.dtb  to /boot/dtb/cubieboard2.dtb. It's booting correctly!
I think the dtb name is coming from u-boot, Debian wiki has a list of supported boards and their respective dtbs here.


How to check the applicability of an action at a given state and apply it

User Story
As an AI researcher I want to use UP in my research project to:

Check action applicability: given the current state s0, check if a lifted action a can be applied at s0 (their preconditions are met at that state) or not. Another option is to use a method to obtain all the possible instantations of a at s0 (i.e., obtain the grounded actions corresponding to the lifted action a) and, then, use another method to check the applicability of each grounded action at s0.

Apply an action to the state: after checking that a is applicable, obtain the grounded action a_g by manually selecting the object to instantiate each parameter on and, then, obtain the next state s1 that results from applying a_g at s0. Ideally, this process would support PDDL extensions like :exists, :forall and :when.

In case this functionality is not implemented yet and there are no intentions to do so in the next few days, I would really appreciate some guidance for implementing such functions myself.
Acceptance Criteria

[ ] Action applicability implemented.
[ ] Action transition implemented.

Great, I have similar user stories in mind. I would just split them into two issues/user stories and address them separately. With regard to the first part, here's my version:
User Story
As a UP user I want to get_actual_preconditions() from an ActionInstance to verify that a planned action can be executed when it's its turn.
Rationale: The situation when I get a plan of actions will change over time due to applying these actions but potentially also due to external factors. When I'm about to execute the next planned action, I want to make sure its preconditions are fulfilled as assumed during planning. In my application domain when not using AI planning, I would implement these checks on some sort of state variables. When using AI planning, I need these checks as fluents for planning. Since it is not simple to in general deduce fluents from state conditions but, the other way round, it is easy to map to a state variable check when given a fluent, it makes sense to only implement these checks as UP fluents and also check them during run time.
This information is available from what I see since you can retrieve the action definition from an ActionInstance, and from the action its preconditions. Using the ActionInstance's actual_parameters() it's possible to determine the actual preconditions at execution time. A bit more tricky would be the mapping of parameters to the different signatures of preconditions, so a direct support of this functionality from UP would be potentially more efficient and avoid redundant manual implementations.
From discussions with Oscar I learned the UP team is considering preconditions and effects as part of the returned plan, so this is very related. With this user story, I want to emphasize the demand and present the rationale from a user's point of view. Perhaps you have a more comprehensive solution in mind, which would be welcome even more.
Acceptance Criteria

[ ] Decide whether providing such functionality as part of UP makes sense.
[ ] If so, provide a ActionInstance.get_actual_preconditions() method or similar.
[ ] Optionally, consider related functionality like get_actual_effects() for completeness of the interface.

I totally agree with the idea of splitting my issue into two distinct user stories.
In the end, I have opted for using a different software for my project as UP is in an early stage and does not suit my current needs. Nevertheless, I look forward to seeing the development of this project!

Hi @TheAeryan and @alexander-sung , sorry for the late response.
I think the SequentialSimulator fits this use-case, right?
Let me know if it does what you need it to or not.

Hello @Framba-Luca, thanks for the response still. Sebastian was already looking into the SequentialSimulator from our side, so that should be fine. Let's discuss details in our further meetings.

Hi @Framba-Luca , thanks for the response. However, I no longer need this functionality, as I have implemented it myself as part of Lifted PDDL.


EXIF metadata draft TBP on labs

@mattl and @robmyers, now that SOTC is largely over, do you think we can get a draft up on labs to start a leisurely discussion before xmas eve? We can allow folks to digest through the holiday and then dig in in January. What do you think?

maybe. We have not a lot of time left and a bunch of fundraising things to
do.

Matt Lee
Creative Commons
Austin, TX, USA
3pm-11pm UTC
On Tue, Dec 15, 2015 at 2:50 PM, Jane<EMAIL_ADDRESS>wrote:

@mattl https://github.com/mattl and @robmyers
https://github.com/robmyers, now that SOTC is largely over, do you
think we can get a draft up on labs to start a leisurely discussion before
xmas eve? We can allow folks to digest through the holiday and then dig in
in January What do you think?
—
Reply to this email directly or view it on GitHub
https://github.com/creativecommons/platform-initiative/issues/71.

Ok, let's see where we are at the end of the week/early next. First iteration of platform model + toolkit should at least be done by Fri.

Think we can get a draft of this up by Monday for sharing? Doesn't have to be perfect (shouldn't be actually if we want engagement) but something to go back to platforms and WMF with to kick off the public conversation.


ISPN-6970 Update to JGroups 4.0.0.Alpha1

https://issues.jboss.org/browse/ISPN-6970

unused imports in these classes:
ConcurrentStartChanelLookupTest
ConcurrentStartForkChannelTest
ChannelConnectorBuilder

Updated to remove the TopologyUUID helper class.

Also stopped using a ManagedSocketFactory altogether, since that's identical to the behaviour we have with JGroups 3.6.x.

Fixed some integration tests that were still failing because they were checking for TP.discard_incompatible_packets and TP.enable_bundling, removed in JGroups 4.0.

With the default configuration, performance results with 4.0.0.Alpha1 are almost identical to 3.6.10.Final. There are small differences during the write-only stress test, where 4.0.0.Alpha1 seems to take a little more clients to break down.
We still need to re-evaluate sender-sends-with-timer and the JGroups patches used by JDG 7.0, but that's probably better done after upgrading to 4.0.0.Beta1.

Several  failures caused by the removal of the oob thread pool in the server jgroups  subsystem

@tristantarrant I think I've fixed the test failures, but there are still some performance issues so I'm leaving the Changes Required label.
We could also decide to integrate only the 4.0.0.Alpha1 commits...

@danberindei: @pruivo  and I investigated Pedro's performance issues (IspnPerfTest) on cluster01-04 yesterday and found out that the G1 garbage collector caused roughly a 40% slow down! Removing it improved performance.
I'm investigating why this is the case and what options can be used to tune G1.

Title should be changed to "Update to JGroups 4.0.0.Beta2" :)

I thought I'd leave the upgrade to Beta2 to you, @pruivo ;)

Instead of the Beta2 upgrade, I added another commit to add the NO_TOTAL_ORDER flag to all the response messages, instead of removing it.

Updated.

Local tests look good, but CI shows the functional tests fixed by ISPN-7155 are failing so I need to have another look...

I had lost the ISPN-7155 fix during rebase, should be fine now.

@wburns any other comment or can I merge it?

@pruivo Seems fine to me

integrated! thanks @danberindei !


Add Yosys and Quartus baselines for Intel eval

Intel is BACK baby
Adds Yosys and Quartus baselines.
Progress towards #104
Closes #107
Closes #85

Running eval here: https://github.com/uwsampl/lakeroad-evaluation/actions/runs/6176528135

Latest run https://github.com/uwsampl/lakeroad-evaluation/actions/runs/6178330846


add an option to insert without semicolons

I prefer my JS without semis and it would be nice to be able to configure this to not paste it every time.

I wanna this feature, too!

Just submitted a MR with an option to do so.


DUPLO-4915: Resource duplocloud_aws_dynamodb_table creation times out.

DUPLO-4915: Resource duplocloud_aws_dynamodb_table creation times out.

Task linked: DUPLO-4915 Resource duplocloud_aws_dynamodb_table creation timesout.


Check disk device before saving

Reopening #6307 . Compiled has a count added which is usually a parent class for different devices. count will be used by DiskDevice(and incremented where needed by _might_open function) and is added to avoid mypy errors(since class _Device's __getitem__ has a type annotation of returning Compiled).
Open to comments/suggestions if there is a better way to deal with this.

What increments count?

Device[f"disk:{fn}"] line calls a __getitem__ which is annotated to return a Compiled object. Since Compiled originally doesn't have count attribute, mypy thinks that it tries to access attribute which doesn't exist(which really does but only for DiskDevice). I added the count to "silence" mypy. (We can also consider adding noqa which should be cleaner, since then all other devices won't have the count attribute)

Yea, don't add an unused attribute to silence a typechecker


Compile x86 for linux and windows. Compile aarch64 for linux

Fixes #134

Codecov Report

Merging #135 (203adf9) into main (af98e1e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #135   +/-   ##
=======================================
  Coverage   88.99%   88.99%
=======================================
  Files          11       11
  Lines        2108     2108
=======================================
  Hits         1876     1876
  Misses        232      232

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


Homework 5 submission

Farmer's Market queries based on Homework 5 requirements
What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
I wrote queries to create cross join in order to access data from different tables, and manipulated strings.
What did you learn from the changes you have made?
Understanding the requirement and asking the correct questions helped me write the necessary queries.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
No, it took me time to understand the requirement, not to apply changes.
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
Asking the correct questions is the main issue. To overcome it, I had to learn how the programming language works and what I want it to do of me to read and analyze the data.
How were these changes tested?
All changes were tested as they were written.
A reference to a related issue in your repository (if applicable)
Asking the correct questions seems to be the emerging theme in learning SQL as well as the tools we've learned about in the previous module.
Checklist

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

You have unfortunately made a pull request to the wrong repo. You shouldn't be making a pull request to uoft-dsi/sql, but rather your own forked sql repo. Please read the instructions on your onboarding Assignment Submission Guide more carefully.

Thank you, Daniel!
I made the changes, I can see all 6 homeworks on my PR page.
Regards
Anca Kurian
On Mon, Sep 23, 2024 at 1:33 PM Daniel Razavi @.***>
wrote:

You have unfortunately made a pull request to the wrong repo. You
shouldn't be making a pull request to uoft-dsi/sql, but rather your own
forked sql repo. Please read the instructions on your onboarding Assignment
Submission Guide
https://github.com/UofT-DSI/onboarding/blob/main/onboarding_documents/submissions.md
more carefully.
—
Reply to this email directly, view it on GitHub
https://github.com/UofT-DSI/sql/pull/129#issuecomment-2368929185, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AYEDF3D5FZG7MDHSM6X6HMTZYBGHRAVCNFSM6AAAAABOWMC6ZGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRYHEZDSMJYGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Determine dependency availability in RHEL

otk currently has the following Python dependencies:

click for the command line interface.
rich for pretty formatting of output.
parsimonious for parsing.

We are targeting RHEL 9; let's figure out of these dependencies are available.

Currently none of those are available in BaseOS, AppStream or CRB.


macos

Q Hello MS - it works already on $macos too ?

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

ID: c4b0ea28-4a31-a3ea-fa5f-c32d5c72d4bd
Version Independent ID: 31bd064b-51eb-5505-986e-2c7148784085
Content: Create a Node.js and React app - Visual Studio
Content Source: docs/javascript/tutorial-nodejs-with-react-and-jsx.md
Product: visual-studio-windows
Technology: vs-javascript
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@bgbruno Hi, do you mean does the tutorial work on a Mac (VS for Mac)? Steps are specific to VS on Windows.


Browser and grunt serve issues

This PR fixes

a browser issue when using Firefox, report an incident by clicking on the submit button does not work.
grunt serve try to connect on unbuilt.html instead of index.html

Merged 2dd6907 and 57f7b0e. Thanks @tsamaya!
However, I added 732d3a2acbb386479750c14877ed87e8792e5f8a instead of f813221.
Can you please confirm that this fixes your locale issues w/ the gh-pages site (may require a hard refresh)?

It fixes the issue on the collector page.
but as the dashboard don't use a built version, the map is not loaded (404 from init.js on app/map).

Good catch @tsamaya!
The dashboard page now works after 6819ba6210fb2e6e7e2f1f9ad37e19c5cc6326ac.


Update the tesseract-c_api-demo.py to work with 3.04

The demo for capi in c_api-demo.py uses 3.02 version and the method TessBaseAPIProcessPages(api, filename, None , 0) to get the text from the image. However, in version 3.04 version this method now takes a new parameter TessResultRenderer. So the demo does not work with version 3.04. Additionally, the demo uses TessBaseAPIGetUTF8Text  to get the text which would not get all the text back from multi page tiff documents. Should be using TessBaseAPIProcessPages  to get the text back
This change was made in this commit
I would be more than happy to send a pull request if I can get some help.
I have tried the following but it doesn't seem to work:
outputbase = ctypes.c_char
renderer = tesseract.TessTextRendererCreate(ctypes.pointer(outputbase))
text_out = tesseract.TessBaseAPIProcessPages(self.api, path, None, 0, renderer)

I get an error _type_: must have storage info

See code in this SO answer: https://stackoverflow.com/questions/36871072/segmentation-fault-while-calling-cpp-function-from-python/36876584#36876584


Add tests for simulating external API calls, Fixes #302

Fixes #302
Checklist

[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream master branch.
[x] I have added necessary documentation (if appropriate)

Changes proposed in this pull request:

Added tests for simulating external API calls

@cclauss The travis tests are passing for python 2.7. The app is not deployable on python 3.6 due to issues with relative imports (Travis log) being frequently used in the project. How should I proceed further?

Awesome work!  Thanks a ton for doing this.
You can try marking your tests as expect to fail on Python 3 by:

import pytest  # at the top of the file with other imports
@pytest.mark.xfail(sys.version_info >= (3, 0), reason='Python 3 blocked until #297 goes live')

That must go on the line just before  the def of every function that you expect to fail.

You might have to pytest.mark.skipif https://docs.pytest.org/en/latest/skipping.html

@cclauss I have made changes to the .travis.yml file so that it runs differently for different versions of python.

OK but please help me to move it back to the old way once we get relative imports checked in.

Sure, will do it as soon as the relative imports are checked in!

@cclauss Can you review the PR?

@bhaveshAn @niranjan94 could we please get your reviews on this one?

@ParthS007 @bhaveshAn @niranjan94 Please review the PR.


SameDiff: include "paste a link"

For parity with WordCounter, wondering if we should add a paste a link function to SameDiff? What do you think @rahulbot?
I think this can be a 2.0 thing if we decide to do it unless it's super easy to plop in

duplicate of #6 so I'm closing this


fix(curriculum): prevent passing in case of  element not closed

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #54642

@huyenltnguyen sure, I was just waiting to see if I should update it or not.


Fixing Toggle issue in PillButtonBar Demo

Platforms Impacted

[x] iOS
[ ] macOS

Description of changes
Adding an NSLayoutConstraint to each toggle button to prevent them from being pushed offscreen or overlapping when next to long blocks of text (especially when font size is increased on a small screen).
Verification
Manually tested using the demo app in various font sizes, RTL/LTR languages, different size screens, etc.

Before
After

Pull request checklist
This PR has considered:

[x] Light and Dark appearances
[ ] iOS supported versions (all major versions greater than or equal current target deployment version)
[ ] VoiceOver and Keyboard Accessibility
[x] Internationalization and Right to Left layouts
[x] Different resolutions (1x, 2x, 3x)
[x] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)
[x] iPad Pointer interaction
[ ] SwiftUI consumption (validation or new demo scenarios needed)
[ ] Objective-C exposure (provide it only if needed)

Microsoft Reviewers: Open in CodeFlow

my gut feeling says this might not be the right fix. UISwitch is one of those thigns that has very clear intrinsiccontentsize. if your width anchor fixed this issue, i feel like the problem is in the demo code's uistackview setting.

Closing this PR as the issue has been fixed in commit c14fcca9d4dec2c48a7f5d3277dee9c231c90137 (#880).
@huwilkes @edjamesmsft @harrieshin FYI.


Projecting entity type removes auto-includes added for ownership

Originally filed as https://github.com/aspnet/EntityFrameworkCore/issues/9210#issuecomment-427736113 as comment by @Oblomoff
The bug strikes back in Microsoft.EntityFrameworkCore.SqlServer 2.1.4 (maybe in earlier versions too)! See the attached solution: EfBug.zip
using System;
using System.Linq;

using Microsoft.EntityFrameworkCore;

namespace EfBug
{
    internal class Bar
    {
        public int Id { get; set; }

        public Owned Owned { get; set; }

        public string Simple { get; set; }
    }

    internal class Foo
    {
        public Bar Bar { get; set; }

        public int Id { get; set; }
    }

    internal class MyContext : DbContext
    {
        public MyContext() : base(new DbContextOptionsBuilder()
             .UseSqlServer("Server=(localdb)\\mssqllocaldb;Database=MyContext;Trusted_Connection=True;MultipleActiveResultSets=true").Options)
        {
        }

        public DbSet<Bar> Bars { get; private set; }

        public DbSet<Foo> Foos { get; private set; }

        protected override void OnModelCreating(ModelBuilder modelBuilder)
        {
            modelBuilder.Entity<Bar>().OwnsOne(e => e.Owned);
        }
    }

    internal class Owned
    {
        public string Property { get; set; }
    }

    internal class Program
    {
        private static void Main(string[] args)
        {
            using (var context = new MyContext())
            {
                context.Database.EnsureDeleted();
                context.Database.EnsureCreated();
                context.Add(new Foo
                {
                    Bar = new Bar
                    {
                        Simple = "Simple property is OK!",
                        Owned = new Owned
                        {
                            Property = "Owned property is OK!"
                        }
                    }
                });

                context.SaveChanges();
            }

            // This works fine
            using (var context = new MyContext())
            {
                var bar = context.Bars.Single();

                Dump(bar);
            }

            // Here is the bug
            using (var context = new MyContext())
            {
                var bar = context.Foos.Select(e => e.Bar).Single();

                Dump(bar);
            }

            // ThenInclude workarond does not work!
            using (var context = new MyContext())
            {
                var bar = context.Foos
                    .Include(f => f.Bar)
                    .ThenInclude(b => b.Owned)
                    .ThenInclude(o => o.Property).Select(e => e.Bar).Single();

                Dump(bar);
            }

            Console.ReadKey();

            void Dump(Bar bar)
            {
                Console.WriteLine("Simple property: {0}", bar.Simple);
                Console.WriteLine("Owned property : {0}", bar.Owned?.Property); // <= The bug is here
            }
        }
    }
}

@ajcvickers commented:
Note for triage: this is a different issue than the original issue reported since in the new variation the query root is for Foo which then makes a projection to Bar. Based on how Include currently works, this is by-design. Based on owned types as aggregates the behavior is wrong. I would rather this be tracked by a new issue, rather than re-opening this one, but let's discuss in triage.

@breyed - You are not projecting out any EntityType in your results, so the issue you are seeing is not related to this issue. Please file a new issue if you think there is a bug in EF Core.

Any workaround existing for this bug :) ?

@jonathanantoine, if that can help, the way I got around it was to do a full-fledged Join()
i.e., instead of:
var bar = context.Foos
                    .Include(f => f.Bar)
                    .Select(e => e.Bar)
                    .Single();

do
context.Foos
   .Join(
      inner: context.Bar,
      innerKeySelector: b => b.Id,
      outerKeySelector: f => f.BarId,
      resultSelector: ((f, b) => b)
   )
   .Single();

There may be another, simpler way, but that's the one I found.

Thanks @dstj that will do the job for now :)

I feel really sad about the state of Owned Types in  2(.2.2) and seriously regret trying to use them in my project. I think I have actually accumulated days of lost development time trying to understand the issues caused by some of the unresolved bugs. Figuring out #12865 was a thoroughly unpleasant experience. #9005 requires horrible workarounds... and many others. This particular issue is the final, closing chapter of my personal Owned Types odyssey!

Note that in #15150 I mention that I discovered that this is a problem that is seemingly unrelated to the provider used, whereas this issue explicitly mentions Microsoft.EntityFrameworkCore.SqlServer being the culprit.

@alexschrod I don't think there is anything here that means we think this is only an issue on SQL Server. It's just that SQL Server was the provider being used by the person who reported the issue.

I got across this problem today. The simplest way to fix it I found to be to do something like this.
using Microsoft.VisualStudio.TestTools.UnitTesting;
using Shouldly;
using System.Linq;

namespace EFCorePlayground.Tests
{
    [TestClass]
    public class OwnedEntityReadTests
    {
        //Fails
        [TestMethod]
        public void Project_DoesIncludeNested_NoHack()
        {
            var db = new Context();
            db.Database.EnsureDeleted();
            db.Database.EnsureCreated();

            db.Person.Add(new Person { Name = "A", Approved = new AuditInfo(), Requested = new AuditInfo(), WorkDetails = new WorkDetails { Address = new Address { City = "Dummy" } } });
            db.SaveChanges();

            db = new Context();

            var projected = db.Person.Select(x => new ViewModel
                {
                    Name = x.Name,
                    WorkDetails = x.WorkDetails,
                }).First();

            projected.WorkDetails.Address.City.ShouldBe("Dummy");
        }

        //Passes
        [TestMethod]
        public void Project_DoesIncludeNested()
        {
            var db = new Context();
            db.Database.EnsureDeleted();
            db.Database.EnsureCreated();

            db.Person.Add(new Person { Name = "A", Approved = new AuditInfo(), Requested = new AuditInfo(), WorkDetails = new WorkDetails { Address = new Address { City = "Dummy" } } });
            db.SaveChanges();

            db = new Context();

            var projected = db.Person.Select(x => new
            {
                vm = new ViewModel
                {
                    Name = x.Name,
                    WorkDetails = x.WorkDetails,
                },
                x.WorkDetails.Address
            }).First();

            projected.vm.WorkDetails.Address.City.ShouldBe("Dummy");
        }

        public class ViewModel
        {
            public string Name { get; set; }
            public WorkDetails WorkDetails { get; set; }
        }
    }
}

Looking in SQL profiler this doesn't seem to pull down any extra data. It's just annoying to have to write that code.
SELECT TOP(1) [x].[Id], [x].[WorkDetails_WorkName], [x].[Id], [x].[WorkDetails_Address_City], [x].[WorkDetails_Address_PostalCode], [x].[WorkDetails_Address_Street1], [x].[Name]
FROM [Person] AS [x]

I think this is a very important issue to get fixed as soon as possible.
While there are workarounds, for example the one above they have very negative effect on code quality. For example this is one method we now have in our code base.
        private static CompanyViewModel GetCompanyViewModel(Guid org, Context db)
        {
            return db.Companies
                .Select(c => new
                {
                    vm = new CompanyViewModel
                    {
                        Id = c.Id,
                        SenderData = c.SenderData,
                        Image = c.Image,
                        CompanyColor = c.CompanyColor,
                        AddressPosition = c.AddressPosition,
                        Payday = c.Payday
                    },
                    //EF core hack https://github.com/aspnet/EntityFrameworkCore/issues/13546?_pjax=%23js-repo-pjax-container#issuecomment-497477456
                    AddressHack = c.SenderData.Address,
                    PMHack = c.SenderData.PaymentMethod,
                    FPMHack = c.SenderData.ForeignPaymentMethod
                }
                )
                .First(c => c.vm.Id == org).vm;
        }

While it's doable the problem is that this bug affects the code paths for retrieving data which are often much more common the saving data in a code base so there will be way more places where we have to apply workarounds than for example the problem with updating nested owned entities like in #14713.
It's also incredibly unexpected behaviour and hard to explain users, and even if you know about it it's easy to forget because it's not easy to see at a single glance which nested owned types and entity have. The workaround above also have the problem that if someone update SenderData with another nested owned entity. Every single place where we query on this will have to be updated with this hack.
I think this deserves to be fixed in a patch instead of waiting for 3.0. Compared to many other issues I would say that this is a fairly standard case that should just work and the workaround will have to be applied on every projection application wide. The case against a patch would be that there are working hacks to get this to work.

Removing milestone to consider patch request. However, my initial thoughts are that this is too risky to fix in a patch.

@MikaelEliasson We discussed this again, but unfortunately the risk of regression in other queries to too high for us to fix this in a patch release.

@ajcvickers I understand. Do you know where the bug is? I would like to explore if I can automate the hack above but would like to understand the actual reason it happens a bit better first.

@maumar Could you find some pointers as to where @MikaelEliasson should start looking?

@maumar Did you get a chance to look at this?

@MikaelEliasson sorry for the late reply. Here is some background:
The problem is that include pipeline relies on QuerySourceReferenceExpression to figure out what/where needs to be included. This leads to the issue that include only works when the QuerySourceReferenceExpression in directly accessible in the final result (top level projection).
In the IncludeLoadTree QuerySourceReferenceExpression would have to be changed/removed and the EntityType information would have to be stored instead. Then there is matter of finding the entity type in the final projection. Currently we just find QuerySourceReferenceExpressions using QuerySourceTracingExpressionVisitor.
The problem is that when starting from final projection we would need to guess the entity type based on clr type. That's not safe because multiple entity types can manifest as the same clr type. This I would say is the ultimate cause of the entire problem.
In new pipeline we start from the root DbSet (and we always know what those initial EntityTypes are) and track the entity type of an expression as it's being processed up to the final projection so we never need to guess anything.
Finally, once we have the correct include call created (it happens in IncludeCompiler.Compile) we need to inject them into the original expression in the right place. Currently we just do a simple replace of the QuerySourceReferenceExpression, but that would have to change to match based on EntityType instead, and also have a way to track which entity types had their includes added to avoid duplicates. Currently the injection of include expressions to the tree happens in IncludeReplacingExpressionVisitor.


Squad markers

Hi there,
Unless I am missing it, having looked through both the arcdps API and this project I couldn't find anything that indicates when squad markers are put on someone.
It would be great for logs if you could say "X person was the triangle marker" for example.
There are already exposed to the chat (in I assume the format [Squad marker type] has been placed on [Player ID]), but I don't see them exposed as part of the chat output either.
John

Correct we don't have it. It's a great idea and I don't think it would be any issue ToS-wise

Could I also request a bool/enum for Private/Public squad?

That also sounds fine ToS-wise :)

Another request from me, whether action cam is on/off. Can't find this anywhere else.
Apologies for all of the additional things!


Disable SharePoint indexing instructions outdated

In the section to Disable SharePoint indexing the instructions don't line up with the portal experience.

SharePoint Central Administration is not located under Administrative Tools
There is no Operations link or page
There is no "Global Configuration" or Timer job definitions

What are the correct steps supposed to be for MIM 2016 when its underlying SharePoint system is SharePoint Server 2016?

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

ID: b4c298cb-2abb-6ede-c062-41a744b698c0
Version Independent ID: 132d1643-dd10-d034-421a-2281b940c3ea
Content: Microsoft Identity Manager 2016 Best Practices
Content Source: MIMDocs/mim-best-practices.md
Product: microsoft-identity-manager
GitHub Login: @billmath
Microsoft Alias: billmath

Thank for submitting this. I have created a work item for it and will reviewing this in the coming weeks. I am not sure what the timeline will be, so I am going to close this for now. But once I have an update I will drop it in here and let you know.
Thank you!
Bill
#please-close


[Modal] Prevent backdrop to stay open

Closes #12831.
After following the comments in the above issue, I indeed noticed that onExited was not called. However, I've also noticed that onEntered was not called either. This is because the transition is not even rendered.
To solve this, I have added a flag that tells if the transition was entered. If this is not the case, we can remove the modal from the DOM when open is false.
Another simpler solution would be to always set the exited state to true on mount:
  const [exited, setExited] = React.useState(true);

but I wasn't sure of what all the implications could be.

[x] I have followed (at least) the PR section of the contributing guide.

@ValentinH Thanks for giving this issue a try! If you don't mind, I propose a different solution:

Your solution is exactly what I had in mind for the "another solution". It's also good that you added the fix the the Popper and Snackbar, thanks for this! 😊

@ValentinH I'm not aware of any downside either of the exited initially true approach :)

Yes it should be fine: I think it makes sense to only set the exited state when the onExited callback is called, regardless of the open state.

@ValentinH Thanks men, I remember seeing this issue before the hook migration, the logic was scaring, looks à bit simpler now. I really appreciate that you looked for different possible solutions.

Indeed, the code looked quite straightforward even for a newcomer like me :) hooks helped a lot I think but also the clean implementation.


Added log file templates to project and item extensions

Q
A

Bug fix?
no - yes?

New feature?
no - yes?

New sample?
no - yes?

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

What's in this Pull Request?
Please describe the changes in this PR. Sample description or details around bugs which are being fixed.
Guidance
You can delete this section when you are submitting the pull request.

Please update this PR information accordingly. We'll use this as part of our release notes in monthly communications.
Please target your PR to 'Dev' branch, we merge 'dev' frequently to master as needed.

Thanks for your contribution! Sharing is caring.

@eshupps,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot


fix float to i128 cast

caught by datafusion integration test, see: https://github.com/apache/arrow-datafusion/pull/1733

@Igosuki FYI

Thanks a lot for the fix!

👍


protoc-gen-prost-serde: no_std support

Would it be possible to add an option for the generated code to avoid referencing std and things that are only in the prelude when std is linked, e.g. std::result::Result and ToString, and instead use core::result::Result and alloc::string::ToString?

Unfortunately the code generated for protoc-gen-prost-serde comes via the pbjson crate. They would need to provide a release that generates no_std compliant code. We could do some post-processing of their generated code, but that would be a potentially brittle transformation, so it would be better if the upstream could do this.

Thanks! I opened an issue on pbjson: https://github.com/influxdata/pbjson/issues/135


500 error shown for properly filled task form

After error from https://github.com/davydovanton/ossboard/issues/37 I've changed task body and clicked "create" button, but 500 – Internal server error was shown. Also, after refreshing page I was not logged in.
Gif

I fixed. Form for tasks#update didn't generate csrf token. That's why you had the error.
Thanks for bug report 👍


wttrbar: 0.9.3 -> 0.9.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/bjesus/wttrbar/releases.
meta.description for wttrbar is: A simple but detailed weather indicator for Waybar using wttr.in
meta.homepage for wttrbar is: https://github.com/bjesus/wttrbar
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.9.4 with grep in /nix/store/ypljkp0f594k3x66fjhjz5jis8m3n6vz-wttrbar-0.9.4
found 0.9.4 in filename of file in /nix/store/ypljkp0f594k3x66fjhjz5jis8m3n6vz-wttrbar-0.9.4

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
wttrbar

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ypljkp0f594k3x66fjhjz5jis8m3n6vz-wttrbar-0.9.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A wttrbar https://github.com/r-ryantm/nixpkgs/archive/1e0a5d7d084e59df064f862d555103bd0c5a7e20.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/1e0a5d7d084e59df064f862d555103bd0c5a7e20#wttrbar

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ypljkp0f594k3x66fjhjz5jis8m3n6vz-wttrbar-0.9.4
ls -la /nix/store/ypljkp0f594k3x66fjhjz5jis8m3n6vz-wttrbar-0.9.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

wttrbar

Maintainer pings
cc @khaneliman for testing.

Result of nixpkgs-review pr 297535 run on aarch64-darwin 1

1 package built:

wttrbar


Option to fallback to previous API versions

Since ver 0.0.12, api 3.0 subscription is required. Therefore, API calls fail with error 401 since update. Checking on openweathermap's website, the only way to subscribe for this api version still requires users to provide bank card infos, even if you plan to stay in the limits of their free calls allowances.
I'm sure many users would feel uncomfortable giving their banking infos to owm, like me. Would it be possible to add a flag in the container to continue using api 2.0/2.5 instead (with limited capabilities if needed?)
Thank you for your work

or now its time to move to another free API server like WeatherAPI.com, Forecast, Meteostat, Weather Embed,

Hey guys, currently I have no interest to move to another solution as this one works fine for now.
However, if you'd like to continue using the old 2.5 API, I would just stick with v0.0.11 of this container HERE. I don't see a need to code a fallback option when that v0.0.11 only supports 2.5 in the first place and with it's pending deprication i'm not sure the effort warrants the time.

For the record, the 2.5 API will be gone in June 2024: https://openweathermap.org/one-call-transfer
If you are worried about potential charges, you can set the daily limit on OWM side:


kube-proxy: change mode from "userspace" to "iptables"

Fix #185

This seems to require conntrack binary, which isn't typically installed by default


Oddity with messaging

First of all...great code. One thing I did want to point out is that when the form loses focus the theme goes back to normal. I only see a dark theme when I divert the focus to some other application.
Focus on another app

Focus on the dark mode app

btw, for the latest builds of win10 this should be updated:
constexpr bool CheckBuildNumber(DWORD buildNumber)
{
return (buildNumber == 17763 || // 1809
buildNumber == 18362 || // 1903
buildNumber == 18363 || // 1909
buildNumber == 19041 || // 2004
buildNumber == 19042 || // 2010
buildNumber == 19043 || // 21H1
buildNumber == 19044);  // 21H2
}

Regarding the change in color you are seeing when your app window has focus, I beleive this is by design and is actually an option in Windows,
Right click desktop and select "Personalize" -> "Colours"
In here you'll see an option for **Accent Colours" and "Show accent color on title bars and windows border", if this is enabled then the window that currently has the focus gets its title bar changed to whatever color you have selected here in the settings screen.

Ahhh gotcha


⚠ Bofrost.at has degraded performance

In e1dd635, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1814 ms

Resolved: Bofrost.at performance has improved in fde9457.


neptune mlfow integration not working

We encounter some problems with the neptune mlflow --project USER_NAME/PROJECT command.
I am using a virtual environment setup, with python 3.7, running mlflow (1.0.0) without neptune works, but the integration into neptune-mlflow (0.1.2) gives us the following error:
(venv) neptune mlflow --project cladler/sandbox Loading run Default/c93183dbf0a94edfb0a83fa7a66e7c2e SAN-31 https://ui.neptune.ml/cladler/sandbox/e/SAN-31 Traceback (most recent call last):   File "/home/cladler/PycharmProjects/firstProject/venv/bin/neptune", line 10, in <module>     sys.exit(main())   File "/home/cladler/PycharmProjects/firstProject/venv/lib/python3.7/site-packages/click/core.py", line 764, in __call__     return self.main(*args, **kwargs)   File "/home/cladler/PycharmProjects/firstProject/venv/lib/python3.7/site-packages/click/core.py", line 717, in main     rv = self.invoke(ctx)   File "/home/cladler/PycharmProjects/firstProject/venv/lib/python3.7/site-packages/click/core.py", line 1137, in invoke     return _process_result(sub_ctx.command.invoke(sub_ctx))   File "/home/cladler/PycharmProjects/firstProject/venv/lib/python3.7/site-packages/click/core.py", line 956, in invoke     return ctx.invoke(self.callback, **ctx.params)   File "/home/cladler/PycharmProjects/firstProject/venv/lib/python3.7/site-packages/click/core.py", line 555, in invoke     return callback(*args, **kwargs)   File "/home/cladler/PycharmProjects/firstProject/venv/lib/python3.7/site-packages/neptune_mlflow_plugin/__init__.py", line 26, in sync     return run_sync(path=path, project=project)   File "/home/cladler/PycharmProjects/firstProject/venv/lib/python3.7/site-packages/neptune_mlflow/sync.py", line 45, in sync     loader.run()   File "/home/cladler/PycharmProjects/firstProject/venv/lib/python3.7/site-packages/neptune_mlflow/data_loader.py", line 51, in run     exp_uuid = self._create_neptune_experiment(experiment, run)   File "/home/cladler/PycharmProjects/firstProject/venv/lib/python3.7/site-packages/neptune_mlflow/data_loader.py", line 71, in _create_neptune_experiment     with path_utils.Path(os.path.dirname(run.info.artifact_uri)):   File "/home/cladler/PycharmProjects/firstProject/venv/lib/python3.7/site-packages/path/__init__.py", line 211, in __enter__     os.chdir(self) FileNotFoundError: [Errno 2] No such file or directory: Path('file:///home/cladler/PycharmProjects/firstProject/tutorial-dir/mlruns/0/c93183dbf0a94edfb0a83fa7a66e7c2e')
Is there a solution for this given problem? Thanks!

Hi @Aid91,
Thanks for your report. The problem occurred because version 1.0.0 of mlflow broke backward compatibility.
Version 0.1.3 of neptune-mlflow with a fix for this issue was just released. :)

Hi @aniezurawski,
Great! Thanks a lot for the quick fix :)


Updating CONTRIBUTING.md

This PR adds
pip install -r requirements-dev.txt before make test as described #70

This is valid somehow @andreribeirocoelho @regismachado ?

@Perkles I think you were trying to reference @aguimaraesduarte : )


join request for discord linux

Hello!
What is your username and discriminator?
apt-get moo#4431
Why did you join Discord Linux?
Discord Linux client support and tricks for running it on the myriad of Linux-based systems out there.
Where did you hear about Discord Linux?
Discord Testers server #linux-client-chat.
What is the passphrase that was assigned to you by the bot?
black-defense-burst-macrofauna

From Ethan Weegee :
Looks good.  I've given you a temporary role so that you can enter the passphrase in the welcome channel.


Cannot get XML generated

Hey all
I cannot seem to get any XML generated when specifying the said template. If I run the command without the template I do see the docs generated but XML is still a no go for me.
Expected Behavior
XML file to be generated
Actual Behavior
I cannot find the XML file anywhere
Steps to Reproduce the Problem

Downloaded the https://github.com/phpDocumentor/phpDocumentor2/releases/tag/v3.0.0-alpha.2 file as well as the key
cd to my folder
$ php phpDocumentor.phar run -d ./src/ -t ./docs/ --template=xml
No output either in the docs/ folder or the build folder
$ php phpDocumentor.phar run -d ./src/ -t ./docs/ --template="xml"
Same thing

Your environment

Version used: 3.0.0-alpha2
Install method: phar
php version: 7.2
xubuntu 16.04
Link to your project: https://github.com/niden/ide-stubs/tree/refactoring-for-phpdoc

Any pointers would be greatly appreciated.
Thank you

Hi Niden,
Thanks for your report. Somehow the xml templates in phpdoc 3 are not working I will start searching why. Will keep you posted.

It turnsout that we missed the xml templates somehow and that there was a bug in the commandline argument handling for templates.
I checked both issues with our own code base. If you are able to check the latest version in develop that would be nice. There is no phar available yet.

Thank you! Looking forward to the next release.

We released a new version yesterday. I will close this issue. Feel free to reopen when this issue still exists.