Clarify Kerberos decryption key roll over for SSO when using Cloud Sync

The docs for Azure AD Connect highlight the importance of regularly rolling over the Kerberos decryption key of the AZUREADSSO computer account used for SSO. The docs for Cloud Sync describe how to enable SSO, but make no reference to Kerberos key management.
It is unclear if Cloud Sync requires domain admins to regularly roll over this decryption key, if this task is managed by the Cloud Sync Agent, or if Cloud Sync uses a different approach than Connect for SSO that may not necessarily require key rotation to maintain security.
It would be helpful if the Cloud Sync docs provided guidance about Kerberos decryption key management when SSO is enabled.

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

ID: dd791b84-7aff-3e4d-98c5-cf869c538143
Version Independent ID: ac502a77-608e-0552-8c76-9793a2dcbd91
Content: How to use Single Sign-on with cloud sync - Microsoft Entra
Content Source: articles/active-directory/cloud-sync/how-to-sso.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@e3-jarnott
Thanks for your feedback! We will investigate and update as appropriate.

@e3-jarnott
Reviewed your statement. Here is the link provided for roll over the Kerberos decryption key of the AZUREADSSO computer account.
https://docs.microsoft.com/en-us/azure/active-directory/hybrid/how-to-connect-sso-faq#how-can-i-roll-over-the-kerberos-decryption-key-of-the--azureadsso--computer-account-
If this link is not helpful. Please feel free to reach out.
Thanks

I am aware of that process and have successfully executed it in the past on an Azure AD Connect deployment. It is not clear from the docs if this task is required when running Cloud Sync (instead of AD Connect) with SSO enabled, and if it is required, how to do it given the Cloud Sync agent does not include the AzureADSSO PowerShell module referenced in the steps for AD Connect.

Thank you for submitting this issue.  Unfortunately, due to high volumes I was unable to respond to this in a timely manner.  At this point, I am going to close this due to it's age and the possibility that it may have been addressed.  If this is still a problem, please consider submitting a new issue.  Thank you.
#please-close


Fix: Add header to gitlab.js

:star2: What does this PR do?
Fixes error reported by CI:

https://github.com/toggl/track-extension/pull/2156#issuecomment-1397154489

reverse-engineered update-origins.js to understand what the hell the error means, as there's definitely nothing written about this in docs/CONTRIBUTING.md

Should be rather trivial to write a script that:

takes current origins.js
assembles expected comment block for each integration
writes that to each integration replacing existing block if it's there.
git add -p to verify each change
commit the changes

but given past slow and negative experience with this repo, I would not do that unless someone explicitly asks to do that.
also, the error that the script outputs, should guide the user to go to docs/CONTRIBUTING.md#file-header, and that # File header block should be written first.

Also, CI for update-origins is set up incorrectly. it runs in the context of the contributing user repository:

https://github.com/glensc/toggl-button/actions/runs/3960074160

so, it doesn't appear at all under pull request:

https://github.com/toggl/track-extension/pull/2158/checks

probably another reason why other integrations lack the header. I just happened to read my email to see the CI failure.


upgrade go-redis to v8.11.5

Hi @chenyahui
I have just upgrade go-redis to v8.11.5 and test in my local environment, can you please have a look at this.
Thank you.

thanks for you contribution~


adopt InternMap for ordinal scales

Yes it fails because keys were coerced to strings, but now we can distinguish 0 and "0"—which risks to break a lot of applications.
A solution is to keep coercing everything to string when looking in the index:
diff --git a/src/ordinal.js b/src/ordinal.js
index f4513c5..0a34644 100644
--- a/src/ordinal.js
+++ b/src/ordinal.js
@@ -10,19 +10,21 @@ export default function ordinal() {
       unknown = implicit;

   function scale(d) {
-    if (!index.has(d)) {
+    const key = "" + d;
+    if (!index.has(key)) {
       if (unknown !== implicit) return unknown;
-      index.set(d, domain.push(d));
+      index.set(key, domain.push(d));
     }
-    return range[(index.get(d) - 1) % range.length];
+    return range[(index.get(key) - 1) % range.length];
   }

   scale.domain = function(_) {
     if (!arguments.length) return domain.slice();
     domain = [], index = new InternMap();
     for (const value of _) {
-      if (index.has(value)) continue;
-      index.set(value, domain.push(value));
+      const key = "" + value;
+      if (index.has(key)) continue;
+      index.set(key, domain.push(value));
     }
     return scale;
   };

This fixes the tests, but then it's not better than the current implementation…

now we can distinguish 0 and "0"—which risks to break a lot of applications.

This feels like a highly desirable breaking change that merits a major version bump on d3-scale.


Packaged application gives error message: Could not resolve a binding for http://schemas.xmlsoap.org/wsdl/soap/

When using the application with dev mode ( without packaging ) everything works perfectly. But after we package it as an uber-jar it gives the following error:
Could not resolve a binding for http://schemas.xmlsoap.org/wsdl/soap/

We are using quarkus-cxf version 0.5 and quarkus version 1.11.3.Final.
If you need more information, please let me know.

Using the default packaging type for the version 1.11.3.Final it works properly.

But it is very important to me that it also works when packaging the application as an uber-jar.

hmmm
I think we need to add
META-INF/cxf/cxf-servlet.xml
META-INF/cxf/cxf-extension-jaxws.xml
to NativeImageResourceBuildItem in /QuarkusCxfProcessor
@matheusdallrosa  can you try with this patch on master ?

maybe because SoapBindingFactory and XMLBindingFactory must be loaded
so it mean that cxf-rt-bindings-soap must be included in dependencies, is it done ?

another lead : https://cxf.apache.org/docs/bundling-cxf-into-single-jar-with-maven-shade-plugin.html

hmmm
I think we need to add
META-INF/cxf/cxf-servlet.xml
META-INF/cxf/cxf-extension-jaxws.xml
to NativeImageResourceBuildItem in /QuarkusCxfProcessor
@matheusdallrosa can you try with this patch on master ?
I just found this info on stack overflow so maybe wrong.
Anyway, I see that error come from : https://github.com/apache/cxf/blob/75fb6bb56d82f72771a9ee6ecab5d36168303f51/rt/frontend/simple/src/main/java/org/apache/cxf/frontend/AbstractWSDLBasedEndpointFactory.java#L324

The exception definitely comes from the method you mentioned.
I will try to add the files you mentioned.

@matheusdallrosa @dufoli This is a duplicate of https://github.com/quarkiverse/quarkus-cxf/issues/82 ... until https://github.com/quarkusio/quarkus/issues/5677 is fixed, the only good solution would be to create your own pre-shaded CXF bundle, though that may be a bit complex.

maybe because SoapBindingFactory and XMLBindingFactory must be loaded
so it mean that cxf-rt-bindings-soap must be included in dependencies, is it done ?

Tried it and it did not work.

Also, you may have looked into it already - it may be worth exploring the fast-jar option, which is preferred, instead of uber-jar.  Just keep in mind the executable jar will end up in the target/quarkus-app folder.  It works fine (without the issues above) - and provides slightly better performance.

Also, you may have looked into it already - it may be worth exploring the fast-jar option, which is preferred, instead of uber-jar. Just keep in mind the executable jar will end up in the target/quarkus-app folder. It works fine (without the issues above) - and provides slightly better performance.

Yes, i would like to move to fast-jar, but at the moment is not possible for client's reasons.

How can we know which files we need to "give priority"?
Maybe
META-INF/cxf/cxf-servlet.xml
META-INF/cxf/cxf-extension-jaxws.xml

Would be enough?

Hey @matheusdallrosa I think we've already provided several links that will be helpful for you.  I would suggest you start with something like this:  https://cxf.apache.org/docs/bundling-cxf-into-single-jar-with-maven-shade-plugin.html  ... beyond this, I'm afraid I don't have any generic advice, but let us know if you have any specific questions, and please do keep us posted if you can find a solution.

hmm maybe there is a solution by using GeneratedResourceBuildItem with List and if list is not empty we merge all cxf-extension xml file together. I think it will help a lot. no?

btw wsdl.plugin.xml must be merged too:
https://github.com/apache/cxf/blob/master/rt/bindings/xml/src/main/resources/META-INF/wsdl.plugin.xml
https://github.com/apache/cxf/blob/master/rt/bindings/soap/src/main/resources/META-INF/wsdl.plugin.xml

@shumonsharif I have no time to test that but if you have on your own... it is a lead to have equivalent of plugin to merge resources. After all, processor is a precompilation so merge file must be doable with it.

Fair points @dufoli - definitely worth exploring the approach you suggested, though I'm not sure what conflicts may arise with the existing Quarkus uber jar packaging mechanisms.  Will take a look and keep you posted!

I have implement the idea, If you can test the branch on your side before merge...

@dufoli Testing this now, I will keep you posted with findings.  Keep in mind though, I'm not sure why exactly - but I never encountered this issue with native builds - native has always worked fine for me.  I have seen the issue with uber jars only.

@dufoli The issue remains; getting the error below using the uberjar branch.
17:32:09 ERROR [io.qu.ru.Application] (main) Failed to start application (with profile prod): org.apache.cxf.BusException: No binding factory for namespace http://www.w3.org/2003/05/soap/bindings/HTTP/ registered.
        at org.apache.cxf.bus.managers.BindingFactoryManagerImpl.getBindingFactory(BindingFactoryManagerImpl.java:93)
        at org.apache.cxf.frontend.AbstractWSDLBasedEndpointFactory.createBindingInfo(AbstractWSDLBasedEndpointFactory.java:348)
        at org.apache.cxf.jaxws.JaxWsServerFactoryBean.createBindingInfo(JaxWsServerFactoryBean.java:182)
        at org.apache.cxf.frontend.AbstractWSDLBasedEndpointFactory.createEndpointInfo(AbstractWSDLBasedEndpointFactory.java:268)
        at org.apache.cxf.frontend.AbstractWSDLBasedEndpointFactory.createEndpoint(AbstractWSDLBasedEndpointFactory.java:146)
        at org.apache.cxf.frontend.ServerFactoryBean.create(ServerFactoryBean.java:168)
        at org.apache.cxf.jaxws.JaxWsServerFactoryBean.create(JaxWsServerFactoryBean.java:211)
        at io.quarkiverse.cxf.transport.CxfHandler.<init>(CxfHandler.java:141)
        at io.quarkiverse.cxf.CXFRecorder.initServer(CXFRecorder.java:184)
        at io.quarkus.deployment.steps.QuarkusCxfProcessor$startRoute-273993731.deploy_0(QuarkusCxfProcessor$startRoute-273993731.zig:263)
        at io.quarkus.deployment.steps.QuarkusCxfProcessor$startRoute-273993731.deploy(QuarkusCxfProcessor$startRoute-273993731.zig:40)
        at io.quarkus.runner.ApplicationImpl.doStart(ApplicationImpl.zig:872)
        at io.quarkus.runtime.Application.start(Application.java:90)
        at io.quarkus.runtime.ApplicationLifecycleManager.run(ApplicationLifecycleManager.java:100)
        at io.quarkus.runtime.Quarkus.run(Quarkus.java:66)
        at io.quarkus.runtime.Quarkus.run(Quarkus.java:42)
        at io.quarkus.runtime.Quarkus.run(Quarkus.java:119)
        at io.quarkus.runner.GeneratedMain.main(GeneratedMain.zig:29)

@famod @gastaldi @gsmet do you know a way to transform resource to merge ressource for Uber jar and native to avoid to have the duplicate entry issue.
I do not found anything in doc to merge resource whereas it seems doable in quarkus build. I have try to use generatedRessourceBuildItem but do not worked. You can see code in pull request attached to this issue.
[WARNING] [io.quarkus.deployment.pkg.steps.JarResultBuildStep] Duplicate entry META-INF/cxf/bus-extensions.txt entry from org.apache.cxf:cxf-rt-databinding-jaxb::jar:3.4.2(compile) will be ignored. Existing file was provided by org.apache.cxf:cxf-core::jar:3.4.2(compile)

Looks like this file is already inside the org.apache.cxf:cxf-core::jar:3.4.2 JAR. Why are you regenerating it?

@gastaldi indeed this /META-INF/cxf/bus-extension.txt resource is present in all jar of cxf (core but not only).
So we end with duplicate resource.
it is a basic text file and we can have single file which contain all line merged from different files.
I know that mave shade plugin can do the trick. But I think that we can do it with quarkus, it is a pre compilation afterall so combine resource to a new one must be doable. That s why I think to generated resource build item. The issue is that I do not find how delete old resources to avoid duplicate resource errors and seems not working as expected.
here is the process in cxf.
https://cxf.apache.org/docs/bundling-cxf-into-single-jar-with-maven-shade-plugin.html
here is the full list:
core/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/wsdl/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/ws/rm/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/rs/sse/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/ws/mex/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/ws/addr/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/ws/policy/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/management/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/ws/security/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/bindings/xml/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/bindings/soap/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/bindings/corba/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/frontend/jaxrs/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/frontend/jaxws/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/rs/description/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/transports/jms/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/transports/udp/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/frontend/simple/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/transports/http/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/databinding/jaxb/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/features/logging/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/features/metrics/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/transports/local/src/main/resources/META-INF/cxf/bus-extensions.txt
systests/ws-security/src/test/resources/META-INF/cxf/bus-extensions.txt
rt/transports/http-hc/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/transports/websocket/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/transports/http-jetty/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/javascript/javascript-rt/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/transports/http-undertow/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/transports/http-netty/netty-client/src/main/resources/META-INF/cxf/bus-extensions.txt
rt/transports/http-netty/netty-server/src/main/resources/META-INF/cxf/bus-extensions.txt
services/ws-discovery/ws-discovery-service/src/main/resources/META-INF/cxf/bus-extensions.txt

@geoand may I kindly ask you to have a look at ⬆? Do you know a trick that might help here? Thanks!
It seems https://github.com/quarkusio/quarkus/issues/5677 would help here but I guess it's not a low hanging fruit.

I think using the shade plugin could be a good strategy here. If the extension depends on the original JARs the duplicated file will always be present

@geoand may I kindly ask you to have a look at ? Do you know a trick that might help here? Thanks!
It seems quarkusio/quarkus#5677 would help here but I guess it's not a low hanging fruit.

Given that this only occurs with uber-jar, I seriously doubt I will ever get around to it :). Others are welcome to try of course and I can try and help

@dufoli @gastaldi As mentioned previously, the issue does not appear to occur in native mode.  I'm not sure if I'm missing something, but my testing has repeatedly shown it to be limited to uber jar mode only.  Fast jar mode is also not an issue, even though we're not merging any resources there.
It's a really simple test to see what's getting included in the native image, and the output for the snippet shows all the bus-extensions.txt files getting bundled in the native image.
import io.quarkus.runtime.StartupEvent;
import org.jboss.logging.Logger;

import javax.enterprise.context.ApplicationScoped;
import javax.enterprise.event.Observes;
import java.io.IOException;
import java.net.URL;
import java.util.Collections;
import java.util.Enumeration;

@ApplicationScoped
public class CXFResourceLogger {

    private static final Logger LOGGER = Logger.getLogger(CXFResourceLogger.class);

    void onStart(@Observes StartupEvent ev) throws IOException {

        LOGGER.info("Identify resources in native image ...");

        ClassLoader loader = Thread.currentThread().getContextClassLoader();
        Enumeration<URL> urls = loader.getResources("META-INF/cxf/bus-extensions.txt");

        Collections.list(urls).forEach((url) ->
            LOGGER.info(url)
        );
    }

}

Anyway, just wanted to make note of this in case it helps in any way, since we seem to be having some disagreement on the native mode.
Finally - I'm happy to help with this issue @dufoli so please keep me posted if there's anything I can do!

@dufoli I created a PR in quarkusio/quarkus#17077 introducing UberJarMergedResourceBuildItem. Can you give it a try if that works for you?

FTR: This is included in Quarkus 1.13.4.Final which has been released today.

@shumonsharif @dufoli take a look at https://github.com/quarkusio/quarkus/blob/main/core/test-extension/deployment/src/main/java/io/quarkus/extest/deployment/UberJarConfigBuildStep.java#L14
That should be enough to have the files merged in the Uber Jar

@gastaldi Thank you; I think your PR should partially solve the issue we're having.  I noticed you had this ...
https://github.com/gastaldi/quarkus/blob/3f079855681195af7bf96d149ee6c779a4f16d88/core/deployment/src/main/java/io/quarkus/deployment/pkg/steps/JarResultBuildStep.java#L1172
... so we should be half way there.  :-)  The XML merge will also need to be included at some point for this issue to be addressed in full; is that something you may be able to help with?  I can look into it, though I'll need get set up with the quarkus code base first!

@shumonsharif sure, I'll send a PR soon and ping you in the PR for testing, if you like

@gastaldi absolutely, kindly keep us posted.

thanks @shumonsharif  Iwas thinking issue happend in native mode because of some stack trace but indeed it was the providerImpl wich was missing in resource. So, yes you are right it is only in uber jar.
For uberjar, merging of file is great. But if we can have other kind of merging as xml. If we can have something where we transform.
I was thinking to something like defining
TransformResourceBuildItem(String name, (resources) => {return Byte[]});
or
TransformResourceBuildItem(filterFunction, String name, (resources) => {return Byte[]});
old resources are deleted and new one is produce with byte as content .

I am not sure we can define functions inside BuildItems. We could perhaps have the resulting contents of the merged resources at least.
I have created https://github.com/quarkusio/quarkus/pull/17196 for combining XML files properly, but perhaps we could have a build item that replaces the entire resource with something else.

For uberjar, merging of file is great. But if we can have other kind of merging as xml, json,.... If we can have something where we transform.
I was thinking to something like defining
TransformResourceBuildItem(String name, (resources) => {return Byte[]});
or
TransformResourceBuildItem(filterFunction, String name, (resources) => {return Byte[]});
old resources are deleted and new one is produce with byte as content .
And even, if we do it for uberJar, it can be for other thing. Maybe optimize resource with mignification or whatever.... so maybe name it with more generic name.

@dufoli I think you described GeneratedResourceBuildItem?

Okay, now we're getting somewhere. Producing GeneratedResourceBuildItems should be what you' re looking for: https://github.com/quarkusio/quarkus/pull/17199

@gastaldi This is great, about to start testing your new PR, will keep you posted, thanks again.

@gastaldi Testing has been successful, thanks for your help with this.  We're finally in a good spot!  :)
@dufoli I've attached a patch of the changes I made for testing purposes, feel free to review and integrate ... we'll need to wait for the next Quarkus release before we can do much with this anyway.  Also attached the merged file contents for META-INF/cxf/bus-extensions.txt and META-INF/wsdl.plugin.xml, extracted from the uber jar.
quarkus-cxf-194.patch.txt
merged-bus-extensions.txt
merged-wsdl.plugin.xml.txt

@shumonsharif  I check the patch it is great but I do not like to add new dependency just to combine xml so I just update the uberjar branch with xml combine with standard xml reader/writer. based on your work. I am not a big fan of current version either because it add a lot of code for not so much.... I will not merged right away to discuss about it.

Hey @dufoli these are the commands I ran after running mvn clean install -Dquarkus.package.type=uber-jar
unzip -p quarkus-cxf-integration-tests-0.8.0-SNAPSHOT-runner.jar 'META-INF/cxf/bus-extensions.txt'
unzip -p quarkus-cxf-integration-tests-0.8.0-SNAPSHOT-runner.jar 'META-INF/wsdl.plugin.xml'

I have just test and the uberjar work too ;-) @shumonsharif

I have tested with 2.0.0Alpha3 and it is working perfectly!

if we switch to 2.0 branch of quarkus we have to wait at least 1 month the final release
current release alpha 2
we can expect at least 2 RC before final so it mean a month and a half. So I will not switch to 2.0 release and hope next 1.3 final release will get the back port of this commit.

@dufoli It looks like the issue was merged for 1.13.5.Final, so the back port is there (Honor GenerateResourceBuildItem while building UberJar)
https://github.com/quarkusio/quarkus/milestone/126?closed=1
I think 1.13.5.Final will be released very soon, so glad to see your merged this in!

@shumonsharif yes, 1.13.5.Final should be hopefully out on May 26th


cannot cancel pgbench init when running via proxy

I was able to cancel the workload, but not the init phase. It worked for the local configuration without proxy

I couldn't reproduce this. Were you using any flags other than what I have here?

Hm, maybe it was some lucky timing, or a very slow connection. Yesterday I tried to run pgbench from Moscow against our production cluster in US and saw this behaviour, I'll recheck this and try to get some insights from digging into actual network traffic

it's in your hands, @LizardWizzard  :)

I was able to reproduce it with pgbench -i -s500. I hit Ctrl+C after around a second
I'm using mobile internet connection so it may cause problems:

I think it may be a problem with pgbench itself. I'm running wireshark and I can see cancel requests being sent, but I see new queries too which is odd. I'll try to dig deeper

Yeah, I think I've found the reason. When pgbench initializes small tables (pgbench_branches, pgbench_tellers) there is no cancellation handler. So when internet connection is really bad population of these small tables can take significant time.
Following patch fixes the problem for me:
diff --git a/src/bin/pgbench/pgbench.c b/src/bin/pgbench/pgbench.c
index d2c0ed3dfc..e88cc4b17e 100644
--- a/src/bin/pgbench/pgbench.c
+++ b/src/bin/pgbench/pgbench.c
@@ -4189,6 +4189,8 @@ initGenerateDataClientSide(PGconn *con)
 						  "insert into pgbench_branches(bid,bbalance) values(%d,0)",
 						  i + 1);
 		executeStatement(con, sql.data);
+		if (CancelRequested)
+			break;
 	}

 	for (i = 0; i < ntellers * scale; i++)
@@ -4198,6 +4200,8 @@ initGenerateDataClientSide(PGconn *con)
 						  "insert into pgbench_tellers(tid,bid,tbalance) values (%d,%d,0)",
 						  i + 1, i / ntellers + 1);
 		executeStatement(con, sql.data);
+		if (CancelRequested)
+			break;
 	}

 	/*

@hlinnaka what do you think? should it be submitted upstream or this can be ignored due to insignificance?

Yeah, whatever we do here, we should do it in upstream.
I'm confused by the one CancelRequested check that there is now. If you hit CTRL-C, it will cancel the data generation, but still proceeds with the steps after that like creating indexes, and reports "done in x s". I would expect some kind of an error message, and non-zero exit status. This query cancellation logic was added by commit 1d468b9ad81b9139b4a0b16b416c3597925af4b0, before that it worked the way I'd expect: if you hit CTRL-C, pgbench exits immediately. I think what that commit tried to do is to also cancel the possibly-long running query in the server, to avoid wasting server resources, especially if you're using server-side generation of the data with pgbench -I dtGvp -s100 -i. But it accidentally changed it so that the pgbench client doesn't exit immediately on CTRL-C anymore. In short, I think 1d468b9ad81b9139b4a0b16b416c3597925af4b0 was broken.
So I don't think the proposed patch that adds CancelRequested is the right fix. Perhaps call setup_cancel_handler() with a callback that does exit() (is that safe from a signal handler? I don't remember). Except that currently the callback gets called before sending the query cancel to the server, and we'd want the exit() to happen after the cancellation. Or perhaps sprinkle those CancelRequested checks like you proposed, but we'll need to have them in more places, and exit() when it happens.
But yeah, please open a thread on this on pgsql-hackers.

it will cancel the data generation, but still proceeds with the steps after that like creating indexes, and reports "done in x s"

Yeah, that felt strange for me too.
Thanks! I'll improve the patch and open a thread on hackers

This is probably from -i flag, that sets a SIGINT handler.
https://github.com/zenithdb/postgres/blob/a260728984e27137130681ab24fa83a4f4815c4e/src/bin/pgbench/pgbench.c#L4451
That tries to send a query cancel and it's pretty heavy, QueryCancel's connect(), send() and recv() all can block ie. pkt drops in the network or just blocking ingress pkts from the db.
https://github.com/zenithdb/postgres/blob/a260728984e27137130681ab24fa83a4f4815c4e/src/interfaces/libpq/fe-connect.c#L4412-L4541

@MMeent could you take a look at this please? PostgreSQL v16 feature freeze has started, so now is a good time to work on bug fixes rather than features :-).

Just noticed that after termination pgbench_accounts contains just one record, because check for CancelRequested is done after generation of first record.
I do not see any PR with proposed fix of this problem at commitfest, did we created one?
I do not think that calling exit from signal handler is goof idea because according to the specification it is not one of the async-signal-safe functions. It is possible to call _exit or _abort. From my point of view it is enough just to add printing message at the end that initialization of the database  was canceled.
There is another related problem: with slow connection pgbench -i takes at lot of time just because each teller is created using separate insert statement so with larger roundtrip even with scale 100, initialization of 1000 tellers may take too much time The obvious solution is to use COPY for initialization of this table just as pbench_account. But I am not sure how useful such optimization  may be, taken in account that main pgbench test will send individual statements in any case, so with ~1 sec latency reported performance will be awful. So it is not clear do we need to speed-up initialization if main benchmark loop will still suffer from this problem.

The easiest way to reproduce is to use a ridiculously large scale. With a large scale, the initGenerateDataClientSide function takes a long time so that it's easier to see the problem:
$ pgbench -i -s500000 postgres
dropping old tables...
creating tables...
generating data (client-side)...
[hit CTRL-C]
^CCancel request sent
[pgbench keeps running]

Update: I have a patch I am pretty comfortable with. I will be submitting it to the mailing list on Monday probably.

I was browsing through old emails to pgsql-bugs list, and saw this thread that's tangentially related: https://www.postgresql.org/message-id/17698-58a6ab8caec496b0%40postgresql.org

I can respond to the thread if you want. Seems like a simple patch given the framework of fe_utils/cancel.h already.

Dropping a link to the mailing list entry: https://www.postgresql.org/message-id/CSSWBAX56CVY.291H6ZNNHK7EO@c3po


add human job name with wildcards

if I've understood the documentation, you use the run_uuid as the job name. In snakemake 7 profiles, I use wildcars to identify jobs. I also use the rules logs so that the slurm logs are in the same directory.
    output: '{log.output}_cluster'
    error: '{log.error}_cluster'
    job-name: '{rule}.{wildcards}'

is it possible to implement these options in this plugin?
thanks in advance

The special job names are there for the reason to limit the number of queries necessary to the slurmctld. Which is a handy feature when many jobs are spawned by snakemake.
However, we had the request in the past and I found this also a little bit annoying. Hence, a "comment" was inserted to contain the rule name. You can inquire with commands like: sacct -o jobid,comment,nodelist,state. I was, however, not able to amend the docs accordingly and indicate shortcuts and the like. Thank you for providing this issue - I will only close it, when sufficient documentation is present.

I'm sorry, but I don't see how to add a wildcard to a job on the documentation, while you are removing a feature available on snakemake 7 ????
I want to use wildcard because if all the jobs have the same name (or comment), you won't be able to see which samples are running...
I also use wildcard for job name to link slurm log to a sample.
Thanks


[bug] HS13A-1 stuck on "LOC"

Not sure this is a bug with this library, but I was playing around with connecting my arduino to the standing desk, and now I've noticed when plugging back in my original HS13A-1 keypad, it's stuck in the LOC state. Not 100% sure how to reproduce, but it will not get out of this setting regardless of how I press the buttons. I've tried:

hold down M for a long time
hold down up and down for a long time
powering on and off the control
powering on and off the control box itself

After all of these it still just says LOC and will not move the desk up and down. I'm not sure how to debug whether it's an issue with the keypad itself, or with the box it connects to. Is any state stored in this box? My box model is CB38M2B(IB)-1
Has anyone else run into this issue? Any ideas how to resolve? Perhaps there's a calibration issue it's experiencing where it thinks the desk is on uneven surface or otherwise has faulty state, and has shut down motor controls into the LOC state? cc @iMicknl @staxDB 🙏

Which code did you use?
I would start with the following steps first:

Disconnect your Arduino from the motor control box
Remove the power cable of your desk
Wait 60 seconds
Plug in the power cable of your desk
Did this solve your problem?

I have exactly the same control box and control panel, however I am using it with ESPHome + Home Assistant at the moment.
By the way, there are some new users figuring out how it works, thus they created a Discord server. Feel free to join us on Discord.

LOC means lock and can be triggered with a 5 seconds press on memory and also released the same way.
It's not a bug it's a feature :)
I haven't attached my desk to an esp but came across this issue and was worried about this bug. But i think everything is fine here.
@tsheaff
Were you able to solve the problem already?

Thanks for the context @mad2xlc! Perhaps we should add this to the FAQ as well.
I will close this issue for now.

Hey @iMicknl @mad2xlc apologies for the incredibly slow reply 😆
I have not managed to fix this issue with the above. Holding down the memory for 5 seconds definitely doesn't fix it, or I would've fixed this months ago. I'm thinking of just buying an new controller. Any thoughts on the best place to purchase one?

Hi @tsheaff,
I am facing the same problem. Have you been able to solve the problem by now?

In case anyone else stumbles on this, I had the same problem with my Sanodesk controller (HS13A-1), though I've never connected it to an Arduino board. It got stuck on child lock mode and pressing and holding M for 5 seconds didn't work.
I fixed it by pressing and holding the M button while removing the power socket. It beeped at me through this procedure until the residual power in the system died. Just before shutting off it flashed '5-'. Once powered back on it was unlocked. Could just be dumb luck but maybe it'll help someone else.

I've just a variation of this bug, googled for the fix I found before and found my own comment (thanks previous me!). However, this time it was different, my keypad was stuck in locked mode but when I pressed the M button it only beeped once, not continuously. The fix above wouldn't work as a result. Instead, after playing around for a long time, I pressed the 1 and 2 buttons together for a few seconds and it beeped continuously at me, then pressing the M caused the same beeping and my fix above worked.

I fixed it by holding M and power off ... still holding M and power on the table ... on display was 888, then I press and hold arrow down and M and table start working


fix issue 17274

fix #17274

/assign

/assign

Great job! Confirmed that the fix renders properly
/lgtm
/approve


Do not start an iteration in search, if chances to finish is low

A match is going on, and this version seems to be much better than master branch

TC 120+1:
Rank Name                          Elo     +/-   Games    Wins  Losses   Draws   Points   WWins  WLoss.  WDraws   BWins  BLoss.  BDraws
   0 zahak_next                     88      26     400     163      64     173    249.5     100      22      78      63      42      95
   1 zahak_master                  -37      35     200      43      64      93     89.5      30      21      49      13      43      44
   2 gopher_check                 -143      38     200      21      99      80     61.0      12      42      46       9      57      34

Finished match

TC 60+1:
Score of zahak_next vs zahak_master: 53 - 54 - 93  [0.497] 200
...      zahak_next playing White: 30 - 21 - 49  [0.545] 100
...      zahak_next playing Black: 23 - 33 - 44  [0.450] 100
...      White vs Black: 63 - 44 - 93  [0.547] 200
Elo difference: -1.7 +/- 35.3, LOS: 46.1 %, DrawRatio: 46.5 %
Finished match

The result is on par with my expectation, that this PR pays of with longer time controls. I might also try to lower the check for next iteration (*2 instead of *3), and see if it gives me the best of both time controls

This yielded a better result than *2 in 1m+1s TC


[Feat] Cmd - Implement Openstack Cookiecutter

Is this a bug report or feature request? (choose one): feature

Python Version (output of python --version): 2.7
Development or Deployment Environment?: All
Release Tag or Master: 0.4
Expected Behavior: Leverage the Oslo config library alongside the cookiecutter scaffolding tool from Openstack
Any Additional Comments: @gardlt
https://github.com/openstack-dev/cookiecutter

@gardlt what is the motivation of using the OpenStack cookicutter, especially, if Armada is not meant to move to OpenStack. If we want certain enhanced project configurations, I believe they can achieved without the cookicutter since it's mostly meant for new projects.

please don't move armada to openstack. there is a high level of interest in Armada being part of Helm or Kubernetes, and in fact we've sold it to the community as such.

we are not going to persue this anymore.


Improve google maps link

Provide more address details in the link for google maps in order to get the start and destination as accurate as possible

Fixed in #128


FR: Additional substitution variable for "project path"

Do you want to request a feature or report a bug?
feature
What is the expected behavior?
see: https://webpack.js.org/configuration/output/#output-filename
There are many variables you can use, additionally it were nice there were a variable for the absolute path to the FOLDER of the used web pack config. like: [projectpath]
If no config is given (called webpack programatically) then an error should be thrown.
If a javascript file is given (which generates the config), the variable is related to this js file's folder.
Other ideas: [context] - to just substitute the path of the given context path. If context is not given, throw an exception.

HOWEVER, currently it vary very which of the following syntax you can use on the different config properties, it should be more consistent. So in most cases the [projectpath]/[context] variable can be spared.
/absolute/path
./relative/path
without/prefix/path

I think every type should be allowed whenever it's possible.
If this is a feature request, what is motivation or use case for changing the behavior?

A further step to write plain json webpack configuration.
Don't require to use the path module all the time.

output.filename doesn't accept absolute paths. where do you want to use [context]?


Does this handle preemptible termination events?

Does this handle preemptible termination events?

That is literally exactly what this repo is for.  :P  lol


pillow: 3.2 -> 3.3

Motivation for this change
Tested with nix-shell -I nixpkgs=. -E 'with import <nixpkgs> {}; stdenv.mkDerivation { name = "foo"; buildInputs = [ python3 python3Packages.pillow ]; }' and:
[nix-shell:~/nixpkgs]$ python3
Python 3.5.2 (default, Jun 25 2016, 21:38:40)
[GCC 5.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from PIL import Image
>>>

Things done

[x] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[x] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

I think travis failed because nova (openstack) depends on both psutil and pillow but one of psutil's tests failed.

psutil_1 is failing regardless of this PR

you might want to check how nox-review calls nix-build and then call nix-build like that yourself but with -k or --keep-going added.

you might want to check how nox-review calls...

k...
15min later: Building in /tmp/nox-review-vdvnew6x: python35Packages.pillowfight calibre python35Packages.pelican python27Packages.sigal python27Packages.librosa hplip_3_15_9 python27Packages.mezzanine python27Packages.scikitimage python27Packages.reviewboard gazebo-headless python27Packages.pydenticon python27Packages.reportlab gnuradio python35Packages.nose_progressive python27Packages.weboob python27Packages.django_silk matrix-synapse python27Packages.basemap python35Packages.binwalk-full gnuradio-osmosdr python35Packages.django_silk python35Packages.av gazebo python35Packages.scikitimage cura winpdb python27Packages.tilestache inspectrumpython35Packages.clint impressive python27Packages.djblets asciidoc-full-with-plugins python27Packages.pillow python27Packages.av python35Packages.wxPython gazeboSimulator.gazebo7 python27Packages.qrcode grass ocropus python27Packages.python-mapnik python35Packages.mahotas printrun torchat python35Packages.internetarchive python27Packages.pelican python27Packages.taskcoach python27Packages.gplaycli python27Packages.internetarchive python27Packages.rainbowstream winswitch python27Packages.wxPython30 mailpile tribler python27Packages.robotframework-ride gnuradio-nacl python27Packages.pillowfight loxodo python27Packages.mahotas python27Packages.mwlib blockdiag jbrout playonlinux actdiag python27Packages.clint python35Packages.mezzanine python27Packages.nilearn python35Packages.librosa mcomix python27Packages.scikitlearn python27Packages.runsnakerun xpra metamorphose2 python35Packages.pilkit salut_a_toi python35Packages.fdroidserver python27Packages.mitmproxy curaLulzbot python35Packages.rainbowstream python27Packages.django_raster python27Packages.wxPython python27Packages.googleplaydownloader python27Packages.willow python27Packages.pyqtgraph gnuradio-with-packages odpdown python27Packages.seqdiag electrum python35Packages.pyopengl python35Packages.pydenticon hplip python35Packages.pyqtgraph nwdiag python27Packages.modestmaps python27Packages.aafigure python35Packages.pillow python27Packages.datadog nova python27Packages.nose_progressive python27Packages.mwlib-rl gqrx python35Packages.sigal python35Packages.python-mapnik python27Packages.pilkit python35Packages.reportlab python35Packages.scikitlearn python35Packages.basemap python27Packages.binwalk-full electrum-dash python27Packages.thumbor python35Packages.datadog python27Packages.pyopengl mirage gazeboSimulator.gazebo7-headless python35Packages.willow python35Packages.aafigure . . . these paths will be fetched (1588.68 MiB download, 4200.03 MiB unpacked)
Hmm... That's more then my whole system... Yeah I think I'll take my chances with travis when it's little packages like this one and avoid commiting anything that will require me to invest in Intel stocks... :/

Hmm... That's more then my whole system... Yeah I think I'll take my chances with travis when it's little packages like this one and avoid commiting anything that will require me to invest in Intel stocks... :/

Yeah, when Nix really takes off I think that's a good idea. I'll push this to a Hydra later.

@FRidh phew... I thought I'll be getting yelled at for committing changes without, eh, committing hardware? :D
Thanks :)

building now at https://headcounter.org/hydra/jobset/iElectric/python-wip
Jobset https://headcounter.org/hydra/eval/330691

Python packages depending on Pillow still build. I haven't tested the others but we'll see.

Thanks!


Update howto-vm-sign-in-azure-ad-linux.md

Add missing supported Linux versions.

@MicrosoftGuyJFlo

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Issue #7584 Kafka Log Directory Not Writeable

Fix quotes in substitution so LOG_DIR is set to a writeable location.
Reproduced error from #7584 with this no error and verified log is written to location:
$ /nix/store/yh8hdd378j1gaaq73gk88ca7gzf590vg-apache-kafka-2.10-<IP_ADDRESS>/bin/kafka-server-start.sh /nix/store/yh8hdd378j1gaaq73gk88ca7gzf590vg-apache-kafka-2.10-<IP_ADDRESS>/config/server.properties
[2015-04-26 19:56:25,370] INFO Verifying properties (kafka.utils.VerifiableProperties)
[2015-04-26 19:56:25,413] INFO Property broker.id is overridden to 0 (kafka.utils.VerifiableProperties)
$ ls /tmp/apache-kafka-logs
total 12
238378449 0 controller.log     238353137 4 kafkaServer-gc.log  238378450 8 server.log
238378448 0 kafka-request.log  238378451 0 log-cleaner.log     238378452 0 state-change.log

cc @ragnard

Looks good to me, thanks @polynomial
@lethalman What's the process from here? I'm happy for this to get merged.

@ragnard was just waiting to hear that. Merged, thanks.


Snakefile detection

Hi @johanneskoester,
thanks for implementing this catalog, this is a great idea!
I noticed that the RADSeq workflow does not appear there. At first I thought that the repository is not small enough and so I removed the results files. But it still did not appear at this state:
https://github.com/AntonieV/NodeRAD/commit/3a6c2244b3c0f6cf5521f33bd77d66a063388092
According to the rules for inclusion the repository may also contains the Snakefile outside the workflow directory. But only when I moved the Snakefile to workflow/Snakefile two weeks later, the workflow appeared in the catalog:
https://github.com/AntonieV/NodeRAD/commit/9546cbcb14abb0c20df1a91c6775da2219a113ae
Thanks and best regards,
Antonie

Fixed in the meantime.


MemoryBackedStore cloned on every FileLoad::map

Consider this snippet of the implementation of the FileLoad trait for MemoryBackedStore (src/storage/memory.rs):
pub struct MemoryBackedStore {
    vec: Arc<sync::RwLock<Vec<u8>>>,
}

impl FileLoad for MemoryBackedStore {
    // ...
    fn map(&self) -> Box<dyn Future<Item = SharedVec, Error = std::io::Error> + Send> {
        let vec = self.vec.clone();
        Box::new(future::lazy(move || {
            future::ok(SharedVec(Arc::new(vec.read().unwrap().clone())))
        }))
    }
}

In the expression vec.read().unwrap().clone() here, if my understanding is correct, it appears that the Vec<u8> underlying the RwLock is being cloned.
I understand that MemoryBackedStore may be primarily intended for testing and, therefore, for relatively small vectors. However, that may not always be the case, and I'd guess that a clone like this could result in excessive memory usage and possibly even a surprise out-of-memory error. (Then again, I could be blowing things out of proportion, pun intended! 💥)
Would it be a good idea to avoid the .clone() here?

It is probably a good idea yes. While it was indeed primarily intended for tests, we are actually using the memory store in terminusdb right now, and I'm sure there's many use cases.
The idea behind the extra clone here is that you get a snapshot view of whatever the contents of the in-memory 'file' is, at the time of resolving this future. There's probably better ways of doing this though.

The idea behind the extra clone here is that you get a snapshot view of whatever the contents of the in-memory 'file' is, at the time of resolving this future. There's probably better ways of doing this though.

Ah, thanks, it's good to know that the clone was intended for a snapshot. Is there an important reason not to replace it with an active view of the data?

It shouldn't be important. The data should not actually change, since all files are write-once, but this is not properly enforced by the types.
For the file backend, we actually give out mmap'ed buffers which could in theory change if the underlying file changes though (although I'm seriously considering replacing that with a version that reads all data into memory instead). But for the current MemoryBackedStore implementation we can't really give out the RwLock-wrapped vec directly though, since it isn't an AsRef<[u8]>.

I don't know the order of operations. Does the MemoryBackedStore need to always be backed by a RwLock? Is the data being written more than once, or is it written once and then read multiple times without being written to again? If the latter is true, then can the RwLock be consumed after writing?


docs(examples): fix bug in Vue infinite scroll example

This fixes a ref not being accessed with it's .value accessor, causing a boolean check to always be truthy.

Thanks for the fix @BrianMitchL


Subtyping constraints in function return specification

The following examples from the eqWAlizer test suite pass, but are marked as fail:
-spec keysort3_neg() -> none().
keysort3_neg() -> lists:keysort(2, [{a, c}, {b, d}]).

-spec min3_neg(atom(), integer()) -> none().
min3_neg(X, Y) -> erlang:min(X, Y).

-spec tc1_neg() -> pid().
tc1_neg() -> timer:tc(fun () -> err end).

The reason is that the specifications all employ subtyping constraints, e.g.:
  -spec tc(Fun) -> {Time, Value}
              when Fun :: function(), Time :: integer(), Value :: term().

If tc/1 would be defined as
  -spec tc(Fun) -> {integer(), term()}
              when Fun :: function().

then the test case fails, as expected.
See #7.

Quite unintuitively, this example passes, too:
-spec foo() -> 0.
foo() -> -1.

Erlang EEP proposal declares :: as type alias, we should change the constraint generation accordingly.


Clone link not found

git clone https://github.com/iotaledger/res-sim-fpc.git as in README.me is not found
try just clone and it works

git clone https://github.com/iotaledger/fpc-sim does work for me

Step 1. Build and run the simulation
Clone the repository
git clone https://github.com/iotaledger/res-sim-fpc.git

@gjeee read my first comment. this link does not work

but git clone https://github.com/iotaledger/fpc-sim DOES work

@gjeee  read my 1st comment ;)

ok you just want the typo to be fixed :-) i was just suggesting a link which does work

I am not 100% sure its a typo. Seems to be referencing another version/project.
Just press clone button and you are fine, but if you follow the instructions you are stuck

It was a typo, now fixed, thanks. I'll close this issue.


75 | Sort Colors | Java

Sort Colors
New Issue(solution) -
Adding New Code(A optimal solution)
Programming Language
Java
Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

Hi @piyush-kash  - Can you please assign this to me?

Can you please assign this to me @piyush-kash .

please assign this issue to me @Yashica7 @SSKale1

Hi @piyush-kash - Can you please assign this to me?

i can't .the owner can

Can you please assign this to me @Yashica7 @SSKale1

@adhi-2311 You can create an issue and get assigned to it

Can i just add a new file in the Java folder of your repo and then make a PR linking my issue?? @Yashica7 @SSKale1

Can i just add a new file in the Java folder of your repo and then make a PR linking my issue?? because its my first time and i dont know any other method @Yashica7 @SSKale1


Bob Ross - Easter Egg :P

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. For example: "I'm always frustrated when [...]"
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Probably best that isn't added to the core package...  ;-)   Happy for you to add it to one of the samples, though!

Core's have to hold onto there Happy Little Vision!
Samples? :D


⚠ Bofrost.de has degraded performance

In c82b5a5, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 941 ms

Resolved: Bofrost.de performance has improved in 6d4ace0.


[Enhancement] preload update state to optimaze lake pk table data load

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：
Fixes #14853
Problem Summary(Required) ：
The upsert and delete data can be loaded into cache when write data done, so we can skip loading those data again in publishing version, for saving publish cost.
Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

https://github.com/Mergifyio backport branch-3.0


service not creating CNAMEs at startup

I have a very simple stack, consisting solely of traefik and traefik-avahi-helper.
version: "3"
services:
  avahi:
    image: hardillb/traefik-avahi-helper
    container_name: avahi
    privileged: true
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - /run/dbus/system_bus_socket:/run/dbus/system_bus_socket

  traefik:
    image: traefik:latest
    container_name: traefik
    restart: unless-stopped
    command: --api.insecure=true --providers.docker
    ports:
      - 80:80
      - 443:443
    labels:
      - traefik.http.routers.traefik.rule=Host(`traefik.local`)
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock

On docker-compose up -d, the avahi log always shows:
><EMAIL_ADDRESS>start
> node index.js
[]
starting cname.py
INFO: All CNAMEs published

and I can't resolve traefik.local, but if I make any change to the traefik service:
-      - 80:80
+      - <IP_ADDRESS>:80:80

and docker-compose up -d, again, traefik.local is published!
Removing [ 'traefik.local' ]
restarting cname.py with /usr/src/app/cnames
starting cname.py
INFO: All CNAMEs published
Adding [ 'traefik.local' ]
restarting cname.py with /usr/src/app/cnames
starting cname.py
INFO: All CNAMEs published

It has nothing to do with the change from TCPv4+TCPv6 to TCPv4, as if I docker-compose down;docker-compose up -d without further changes, the CNAME will once again not be published.
I've tried making traefik depends_on traefik-avahi-helper, and vice versa.
I've tried stopping the  traefik-avahi-helper container and restarting it. Nothing seems to make a difference.
What am I missing?

Add traefik.enable=true as a label to the traefik service iirc

I'll give that a shot. The traefik documentation says it's unnecessary (for traefik itself) if exposedByDefault is unset, but if that's what the helper needs, I'll do it that way.

That did the trick.
I now have 3 other containers I want to have their own .local address and curiously I got this on startup:
 ><EMAIL_ADDRESS>start
> node index.js
[ 'sonarr.local', 'radarr.local', 'jackett.local' ]
starting cname.py
INFO: All CNAMEs published
Adding [ 'emby.local' ]
restarting cname.py with /usr/src/app/cnames
starting cname.py
INFO: All CNAMEs published

With three containers being picked up immediately, and the fourth a little later, but I imagine that's just a quirk of the starting order of the containers.


[VS] Feature Request: Share custom models publicly for Open Source Repos

Product and Version [VS/VSCode]: VS Professional 2017 - 15.9.3
OS Version [macOS/Windows]: Windows 10 - Build
IntelliCode Extension Version:
Target Platform or Language [e.g. Node.js]: C#
Scenario:
I work for a company Umbraco where our CMS is written C# and is available on GitHub. It would be great for anyone who is working with the Umbraco source code and that includes our wider community who contribute bug fixes etc.
It would be a great out of box experience if we can ship the custom model in the repo so that all users contributing to the code get the Intellicode suggestions

@warrenbuckley, thanks for the suggestion. We are considering other ways of sharing models in the future, including having references to models checked in to the repo (although probably not having the model files themselves checked in since they can change somewhat frequently and would bloat the Git history).
The current sharing functionality can come close to mimicking this behavior. If you generate a sharing link and put it in your Readme in GitHub, then anyone can add that model to their IntelliCode profile. There's no automatic model acquisition, but after it is manually added once, any future updates you publish (via retraining the model as the code changes over time) would get automatically picked up for all users of the model.

@warrenbuckley as @jkeech says, thanks for this suggestion, it's definitely a scenario we're thinking about. If you're willing to chat further about this please drop me a line mwthomas at Microsoft dot com.

One (safe and automatic) way to achieve this could be adding a field in the .nuspec file so IntelliCode acquire directly from the nuget package the link to the model

normalizing labels

Hey folks, we've implemented a deep learning model for C# in VS and C# Dev Kit for VSCode, along with Python in VSCode. This should address the ask here!
https://devblogs.microsoft.com/visualstudio/upgrading-from-intellicode-custom-team-completions-models-to-deep-learning/


minor cleanup in manifest.ts

i don't know why copybara is failing, did a local import (cl 300181735) and it succeeded, merging.


Usage without a module?

I'm trying to use const { bigram, trigram, nGram } = require('n-gram');
but I'm getting the error
Error [ERR_REQUIRE_ESM]: require() of ES Module C:\Users\x\x\x\x\node_modules\n-gram\index.js from C:\Users\x\x\x\x\server.js not supported. Instead change the require of index.js in C:\Users\x\x\x\x\server.js to a dynamic import() which is available in all CommonJS modules.     at Object.<anonymous> (C:\Users\x\x\x\x\server.js:8:36) {   code: 'ERR_REQUIRE_ESM'

This package is ESM only.


Using HTTPS?

I am setting up an Ionic app that uses OpenID to authenticate. The authentication endpoint will not accept my request as the callback url is an http url. I searched the docs and the source of ripple but couldn't find anything in the settings that would allow https. Could this be added?

Thanks @iamchairs - would you be able to provide the code you're using to authenticate with OpenID so I can look into this?

Recognising that this comes over five years too late, I had this issue with CORS in newer versions of Chrome, though using Cordova and the old in-browser Ripple emulator - ended up using a separate web server to handle HTTPS and proxy all requests to Cordova. That approach might work here, too.


[Backport] Longhorn k8s 1.27 chart support

Backport of https://github.com/rancher/rancher/issues/41801
This issue is to implement the chart changes & perform the testing required to upgrade to k8s 1.27. Please perform below steps.

[ ] Check whether the chart has any references to the below deprecated features of k8s 1.27. You can find information about deprecated APIs here on Kubernetes Deprecation guide.

storage.k8s.io/v1beta1 API version of CSIStorageCapacity

[ ] Update kube-version annotation in Chart.yaml. catalog.cattle.io/kube-version: '>= 1.23.0-0 < 1.28.0-0'

Note: If this is an upstream chart and if the upstream chart doesn't support k8s 1.27 then do not update the kube-version annotation or update the k8s version accordingly.

[ ] Perform below tests

Fresh install/uninstall/upgrade/downgrade of the charts on K8s 1.27 provisioned on Rancher. Test the chart functionality.
(Please refer to the https://github.com/rancher/rancher/issues/41395 to know what testing was done for k8s 1.26 upgrade for your chart)

Provision rancher on k8s 1.26 or lesser

Install chart on this cluster

Upgrade this cluster in rancher to k8s 1.27

Make sure the existing chart and the app is functional after cluster upgrade

@MKlimuszka Just want to confirm. This is for Rancher 2.7 charts, right?

Verified the chart and no code modification needed:

[x] Check whether the chart has any references to the below deprecated features of k8s 1.27. You can find information about deprecated APIs here on Kubernetes Deprecation guide.
[x] Update kube-version annotation in Chart.yaml. catalog.cattle.io/kube-version: '>= 1.23.0-0 < 1.28.0-0'

For the testing part, we need to test the following charts which are currently available in Rancher 2.7 App & Market Place. cc @khushboo-rancher

101.2.0+up1.4.0
102.2.0+up1.4.1
102.2.1+up1.4.2
102.2.2+up1.4.3
102.2.3+up1.4.4
102.3.0+up1.5.1
102.3.1+up1.5.3

@MKlimuszka Our QA should test with Rancher rancher/rancher:v2.7-head, right?

Verified on rancher/rancher:v2.7.11-b30643b8f2f1a59acfc348fc32825e9d459c0fbb-head 20240202
rancher/rancher:v2.7.11-b30643b8f2f1a59acfc348fc32825e9d459c0fbb-head
Result Passed

[x] 1. v1.26.11+rke2r1 upgrade to v1.27.8+rke2r1 with longhorn 102.2.3+up1.4.4
[x] 2. v1.26.11+k3s2 upgrade to v1.27.8+k3s2 with longhorn 102.2.3+up1.4.4
[x] 3. v1.26.11+rke2r1 upgrade to v1.27.8+rke2r1 with longhorn 102.3.1+up1.5.3
[x] 4. v1.26.11+k3s2 upgrade to v1.27.8+k3s2 with longhorn 102.3.1+up1.5.3
[x] 5. v1.27.8+rke2r1 with longhorn 102.2.3+up1.4.4
[x] 6. v1.27.8+k3s2 with longhorn 102.2.3+up1.4.4
[x] 7. v1.27.8+k3s2 with longhorn 102.3.1+up1.5.3
[x] 8. v1.27.8+k3s2 with longhorn 102.3.1+up1.5.3

Closing the issue as testing was successful and no new chart needs to be released for Longhorn for K8s 1.27 support.


Use scala Int.box instead of Integer constructors

This is a small fix for the test code where it was using new Integer instead of Int.box. The integer constructors were deprecated in JDK 11, so this is not a must have for us, but it is not a great user experience to have the plugin fail to build because of this.
@sameerz up to you if I should target this to 21.12. @jlowe @revans2  fyi.

build

@sameerz up to you if I should target this to 21.12. @jlowe @revans2 fyi.

We do not support Java 8 in 21.12, so let's target this for the next release.

build

build


Prometheus remote-write exporter for Python

This is the Prometheus push exporter for the Python SDK

Completed Q1 2021

Note: Python PRW exporter code completed in Fall 2020 is missing upstream in Python contrib.


error: Failed to find C compiler

I was running Phi-3 notebook and when I ran the training, it gave me this error. I was able to solve it by installing buiild-essential.
sudo apt install build-essential
I am not sure if this dependency is already installed in Google Colab because I ran it on a local notebook. But I think if you add a note in the notebook, that would be beneficial for users who might encounter this problem in the future.

I'll add it into the Wiki - thanks for that!


Cross-Domain problem in Authentication using node.js and angular.js

i have try multiple methods to resolve the problem of CORS but i am unable to solve it . Please solve the issue how i can solve this problem  ----
XMLHttpRequest cannot load https://www.facebook.com/dialog/oauth?response_type=code&redirect_uri=http%…%3A3000%2Fauth%2Ffacebook%2Fcallback&scope=email&client_id=168567676883662. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:3000' is therefore not allowed access.

For security reasons, OAuth servers cannot allow cross domain requests the the authorization endpoint. The error you receive indicates that Facebook is correctly preventing you from doing this.
Sent from my iPhone

On Sep 30, 2016, at 12:06 AM, satyamakgec<EMAIL_ADDRESS>wrote:
i have try multiple methods to resolve the problem of CORS but i am unable to solve it . Please solve the issue how i can solve this problem ----
XMLHttpRequest cannot load https://www.facebook.com/dialog/oauth?response_type=code&redirect_uri=http%…%3A3000%2Fauth%2Ffacebook%2Fcallback&scope=email&client_id=168567676883662. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:3000' is therefore not allowed access.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


M9N UART Broken on Latest Commit

I'm working on testing my M9N based CAN GPS. I rebased against master today and discovered that my GPS was no longer working. gps status showed not okay and it was not communicating. Reverting commit f3c4c97bc87712b84b89cc54f12f51d31ac0038d fixes the issue. @Igor-Misic @bkueng

UART is broken while workin on CAN GPS, and is working again when I2C messages are not enabled. I got lost a bit.  :)

Yeah sorry it is a bit confusing. My board is using an STM32F412 as a PX4 Cannode. I have a U-blox M9N GPS connected to UART0. I'm not sure if this is specific to the M9N. I'll have to test with a Pixhawk 4 GPS that has an M8N.

Just verified with a standalone M9N and a Pixhawk 4 that it is not working there as well.
I also confirmed that the M8N works with f3c4c97bc87712b84b89cc54f12f51d31ac0038d. So this is isolated to the M9N.
Master with f3c4c97bc87712b84b89cc54f12f51d31ac0038d reverted.
nsh> gps status
INFO  [gps] Main GPS
INFO  [gps] protocol: UBX
INFO  [gps] status: OK, port: /dev/ttyS0, baudrate: 115200
INFO  [gps] sat info: disabled
INFO  [gps] rate reading:                  949 B/s
INFO  [gps] rate position:                7.41 Hz
INFO  [gps] rate velocity:                7.41 Hz
INFO  [gps] rate publication:             6.84 Hz
INFO  [gps] rate RTCM injection:          0.00 Hz
 sensor_gps_s
        timestamp: 17226032  (0.117222 seconds ago)
        time_utc_usec: 0
        device_id: 11075589 (Type: 0xA9, SERIAL:0 (0x00))
        lat: 0
        lon: 0
        alt: -17000
        alt_ellipsoid: 0
        s_variance_m_s: 22.5000
        c_variance_rad: 3.1416
        eph: 4294967.5000
        epv: 3750000.2500
        hdop: 99.9900
        vdop: 99.9900
        noise_per_ms: 74
        jamming_indicator: 88
        vel_m_s: 0.0000
        vel_n_m_s: 0.0000
        vel_e_m_s: 0.0000
        vel_d_m_s: 0.0000
        cog_rad: 0.0000
        timestamp_time_relative: 0
        heading: nan
        heading_offset: 0.0000
        automatic_gain_control: 0
        fix_type: 0
        jamming_state: 0
        vel_ned_valid: False
        satellites_used: 0
nsh>

Master with f3c4c97bc87712b84b89cc54f12f51d31ac0038d included.
nsh> gps status
INFO  [gps] Main GPS
INFO  [gps] protocol: UBX
INFO  [gps] status: NOT OK, port: /dev/ttyS0, baudrate: 0
INFO  [gps] sat info: disabled
INFO  [gps] rate reading:                    0 B/s
nsh>

Can you try to comment on this line to see if this will help?
https://github.com/PX4/PX4-GPSDrivers/commit/f3c4c97bc87712b84b89cc54f12f51d31ac0038d#diff-1febc78d26e00434cf70e92631383f784fdb02aa7170afd41fb6eff90114d0b8R647
This is the only one that looks suspicious. F9N doesn't have CFG-I2COUTPROT-RTCM3X  message.

@Igor-Misic Looks like that fixed it.

Solved: https://github.com/PX4/PX4-GPSDrivers/pull/83


Reuse expression parameter if value has already been seen.

This is my solution for #670. This works in the same way for SqlExpression evaluation as SqlBuilder's ReuseParameters, except it is always on.
This is helpful for e.g. search parameters. Currently, query.Where(x => x.Field1.Contains(search) || x.Field2.Contains(search) || x.Field3.Contains(search)) will duplicate the search parameter each time, resulting in a WHERE clause looking like WHERE [Field1] LIKE '@0' OR [Field2] LIKE '@1' OR [Field3] LIKE '@2'.

Its a tiny bit more complex than that, because if the column type you are querying against is different but the value is the same, then you will get errors, as the parameter can't be reused against multiple types.
This is why it was turned off, as it can't be controlled, however we could be smarter in the linq provider as you know the type you are querying against and could reuse ones of the same type. At least you might have a better chance of getting it right anyways.


Regex flags aren't taken into consideration

Regex flags aren't passed to regex101 causing it to default to global and multi-line.
// JavaScript / TypeScript
const pattern = /[abc]/i;

The regex above should open the following URL: https://regex101.com?regex=%5Babc%5D&flavor=javascript&flags=i, but opens https://regex101.com?regex=%5Babc%5D&flavor=javascript.
Every language does flags differently, so fixing this would probably require some smart traversal of PSI elements based on the language, which in turn would probably require different implementations based on the available plugins.
Maybe it's better to just default to &flags= (no flags)?
Here are some more examples that should all pass &flags=i:
// Kotlin
val pattern = Regex("[abc]", RegexOption.IGNORE_CASE)

// Java
Pattern pattern = Pattern.compile("[abc]", Pattern.CASE_INSENSITIVE);

# Python
pattern = re.compile("[abc]", re.IGNORECASE)

Fixed in d73fabf49c944637783fd4ccb13be59bd721237c.
Flags are now passed to regex101. The following languages are specifically supported and tested: JavaScript, TypeScript, Kotlin and Java, but it may still work in other languages depending on if they implement a RegExpModifierProvider or not. I'll take a look at other languages based on feedback and personal need.
Implementation notes:

The global flag is treated as always on
A (probable) bug in IntelliJ adds a DOT_ALL flag ('s') if MULTILINE ('m') is present in JS/TS, I'll create an issue for it

Fixed in d73fabf49c944637783fd4ccb13be59bd721237c.
Flags are now passed to regex101. The following languages are specifically supported and tested: JavaScript, TypeScript, Kotlin and Java, but it may still work in other languages depending on if they implement a RegExpModifierProvider or not. I'll take a look at other languages based on feedback and personal need.
Implementation notes:

The global flag is treated as always on
A (probable) bug in IntelliJ adds a DOT_ALL flag ('s') if MULTILINE ('m') is present in JS/TS, I'll create an issue for it


Set TargetRubyVersion: 2.0 in .rubocop.yml to avoid unwanted autocorrect with Ruby 2.0+

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

There are some rubocop errors with Ruby 2.4.x.
If we run bundle exec rubocop -a with Ruby 2.4.x, it automatically corrects some errors.
But we don't want it, fastlane supports 2.0+.
Description

Update rubocop configuration and avoid unwanted autocorrect by running bundle exec rubocop -a.

Thanks for your review 👍  @giginet @KrauseFx


Altair

1-Altair make often illegal move!
[White "Houdini 6.03 x64"]
[Black "Altair 7.0.0 x64"]
....... 77. Kh2 {-0.45/7 0} Qa6 {+1.00/15 0}
78. Kg1 {-0.75/9 0} Qa5 {+0.97/13 0} 79. Kg2 {-0.79/7 0  Arena
Adjudication. Illegal move!} 1-0

2-Altair crash on quit...
Sandboxie message:
SBIE2101 Object name not found: Unnamed object, error OpenProcess (C0000022) access=001FFFFF initialized=1
thankyou!

Hi,
Can you provide some more details on how Altair crashed or made illegal moves? Such as, what OS you were using, GUI, executable/binary, and settings on Altair?
During testing, Altair 7 has never crashed nor made illegal moves, and this issue has never occurred with others either. I would rather assume there is an issue with how you are running Altair rather than the engine itself. Either way, thanks for the issue post!

Hi!
Thank for respond and congratulations for your chess engine!
Country=Greece
Hardware=Intel(R) Core(TM) i5-4570 3.20GHz 8GB RAM
OS=Windows 10 Pro x64
GUI=Arena 3.5.1
Level=2 0\9/0#100(Timecontrol: 9s+0,1 per game )
[Altair 7.0.0 x64(AVX2)]
Hash=16
Threads=1
I test chess engines only on Sandbox.
here is an part of some game
[White "Carp 3.0.1 x64(AVX2-5Pct)"]
[Black "Altair 7.0.0 x64(AVX2-5Pct)"]
......
98. Kc1 Qb5 {+M2/87 0} 99. Kd1 {Arena Adjudication. Illegal move!} 1-0


Valkyrie use Source and Destination

As a scos maintainer
I want to have kafka inputs and outputs handled by shared libraries
so maintenance and defect costs are reduced and isolated.
AC:

[ ] Valkyrie uses new source and destination protocols like discovery streams does.
[ ] Performance test valkyrie before and after changes and provide results
[ ] Remove and clean up extraneous config no longer needed for broadway
[ ] Ensure valkyrie is set to read from beginning of topics on first connection.

Technical Notes:

We should be able to follow the pattern in discovery streams for source.  Destinations work much in the same way.  You can trace some destinations sample code starting here: https://github.com/inhindsight/hindsight/blob/master/apps/service_gather/lib/gather/extraction.ex
Make sure everything performs well

A quick perf snapshot of Valkyrie (on my machine) for 10,000 SPAT and BSM records - note that is will be ~8 times faster due to Broadway's parallelization. mmb.mmw.lmib.lpc.hpb is the config we're currently using in prod (high prefetch bytes being the key parameter)
Operating System: macOS
CPU Information: Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
Number of Available Cores: 12
Available memory: 16 GB
Elixir 1.8.1
Erlang 21.3.8

Benchmark suite executing with the following configuration:
warmup: 0 ns
time: 30 s
memory time: 500 ms
parallel: 1
inputs: bsm.lmb.lmw.lmib.lpc.lpb, bsm.mmb.mmw.lmib.lpc.hpb, spat.lmb.lmw.lmib.lpc.lpb, spat.mmb.mmw.lmib.lpc.hpb
Estimated total run time: 2.03 min

Benchmarking under_test with input bsm.lmb.lmw.lmib.lpc.lpb...
Benchmarking under_test with input bsm.mmb.mmw.lmib.lpc.hpb...
Benchmarking under_test with input spat.lmb.lmw.lmib.lpc.lpb...
Benchmarking under_test with input spat.mmb.mmw.lmib.lpc.hpb...

##### With input bsm.lmb.lmw.lmib.lpc.lpb #####
Name                 ips        average  deviation         median         99th %
under_test          0.45         2.23 s    Â±16.48%         2.18 s         2.96 s

Memory usage statistics:

Name          Memory usage
under_test       448.21 KB

**All measurements for memory usage were the same**

##### With input bsm.mmb.mmw.lmib.lpc.hpb #####
Name                 ips        average  deviation         median         99th %
under_test          0.38         2.64 s    Â±19.12%         2.57 s         3.67 s

Memory usage statistics:

Name          Memory usage
under_test       487.91 KB

**All measurements for memory usage were the same**

##### With input spat.lmb.lmw.lmib.lpc.lpb #####
Name                 ips        average  deviation         median         99th %
under_test        0.0302        33.13 s     Â±0.00%        33.13 s        33.13 s

Memory usage statistics:

Name          Memory usage
under_test         5.25 MB

**All measurements for memory usage were the same**

##### With input spat.mmb.mmw.lmib.lpc.hpb #####
Name                 ips        average  deviation         median         99th %
under_test          0.20         4.91 s    Â±21.87%         4.47 s         6.51 s

Memory usage statistics:

Name          Memory usage
under_test       775.60 KB

**All measurements for memory usage were the same**
.

Finished in 240.8 seconds
1 test, 0 failures

Randomized with seed 648929

Previous perf with 6 processors
Operating System: macOS
CPU Information: Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
Number of Available Cores: 12
Available memory: 16 GB
Elixir 1.8.1
Erlang 21.3.8

Benchmark suite executing with the following configuration:
warmup: 0 ns
time: 30 s
memory time: 500 ms
parallel: 1
inputs: bsm.lmb.lmw.lmib.lpc.hpb, bsm.mmb.mmw.lmib.lpc.hpb, spat.lmb.lmw.lmib.lpc.hpb, spat.mmb.mmw.lmib.lpc.hpb
Estimated total run time: 2.03 min

Benchmarking under_test with input bsm.lmb.lmw.lmib.lpc.hpb...
Benchmarking under_test with input bsm.mmb.mmw.lmib.lpc.hpb...
Benchmarking under_test with input spat.lmb.lmw.lmib.lpc.hpb...
Benchmarking under_test with input spat.mmb.mmw.lmib.lpc.hpb...

##### With input bsm.lmb.lmw.lmib.lpc.hpb #####
Name                 ips        average  deviation         median         99th %
under_test          0.66         1.52 s    Â±13.19%         1.51 s         1.88 s

Memory usage statistics:

Name          Memory usage
under_test       294.85 KB

**All measurements for memory usage were the same**

##### With input bsm.mmb.mmw.lmib.lpc.hpb #####
Name                 ips        average  deviation         median         99th %
under_test          0.67         1.49 s     Â±8.32%         1.53 s         1.73 s

Memory usage statistics:

Name          Memory usage
under_test       336.32 KB

**All measurements for memory usage were the same**

##### With input spat.lmb.lmw.lmib.lpc.hpb #####
Name                 ips        average  deviation         median         99th %
under_test          0.36         2.80 s    Â±13.31%         2.72 s         3.41 s

Memory usage statistics:

Name          Memory usage
under_test       552.20 KB

**All measurements for memory usage were the same**

##### With input spat.mmb.mmw.lmib.lpc.hpb #####
Name                 ips        average  deviation         median         99th %
under_test          0.38         2.66 s    Â±15.58%         2.70 s         3.07 s

Memory usage statistics:

Name          Memory usage
under_test       532.57 KB

**All measurements for memory usage were the same**
.

Finished in 204.0 seconds
1 test, 0 failures

Randomized with seed 654728

Closed PR for now - https://github.com/Datastillery/smartcitiesdata/pull/969


feat: add Alias to Variance destination doc

Description of the change
Now that we support alias, here's an update to the docs calling it out.
Type of change

[x] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues
NA
Checklists
Development

[X] Lint rules pass locally
[X] The code changed/added as part of this pull request has been covered with tests
[X] All tests related to the changed code pass in development

Code review

[ ]  This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[ ] "Ready for review" label attached to the PR and reviewers mentioned in a comment
[ ] Changes have been reviewed by at least one other engineer
[ ] Issue from task tracker has a link to this pull request

Thanks for the approval @ishachopra0401. @ameypv-rudder I've addressed your feedback and applied your suggestions to the rest of the doc. Mind taking another look?

Thanks for incorporating the suggestions, @gmathieu - I've approved the PR.

Thank you! Who's responsible for merging?

@gmathieu thanks for all the help :)


Any way of querying an item by multiple partition keys?

I couldn't find any way running such queries. The class ResourceFeedReaderClientExtensions is internal

I'm a little confused by the question as ResourceFeedReaderClientExtensions has FeedOptions which only supports a single partition key value.
The samples show how to do a query in the v3 SDK.
If you want to do a query with multiple partition key values then just include the values in the query. For example select * from t where t.pk = "foo" or t.pk = "bar" . The partition key values will be extracted from the query.

Okay. I was looking for a way without writing SQL Queries directly (Lambda for instance)

There is LINQ support for items.

So there is no ddirect way like defining a list of PartitionKey objects (new PartitionKey("somevalue"))?

A single partition key is supported through QueryRequestOptions. For multiple partition keys the only way is through a query.


Ideas about using closures

I'll preface this by saying that my idea is a huge breaking change, and I'm more interested to hear feedback on whether this would even work.
Currently, typed-builder requires the programmer calls .build() at the end, and also exposes *Builder types that can be created and thrown around anywhere. This isn't necessarily a bad thing, but semantically it doesn't make sense to ever allow a builder type to not be built to completion, which is possible by just not calling .build().
I propose that we use a closure instead, which I would argue isn't any worse syntactically:
let conf = Config::build(|builder| builder
    .foo(42)
    .bar("Rust <3".into())
);

This way, we no longer have to call .build() manually; whatever is returned by the closure is what gets built. We can also enforce that the only way (in the public API) to get a *Builder is getting passed one as the closure argument, which can never be taken out of context and must get used. This would also require not implementing Clone.
Thoughts? Is there a reason it's not designed like this?

Why go an extra mile just to decrease usability? The builder is already #[must_use], which is more than enough protection for something that has no side-effect. The ability to move partially built builders around and to clone them was requested by users of this crate, and I see no reason to block it. I'm not protecting them from anything - there are no dangerous things that can be triggered by people explicitly using builders as first class rust values.
Also, I disagree with your claim that:

semantically it doesn't make sense to ever allow a builder type to not be built to completion

Consider this:
let conf = Config::builder()
    .foo(some_string.parse()?)
    .bar("Rust <3".into())
    .build()

Converting this to your closure style won't work:
let conf = Config::build(|builder| builder
    .foo("some_user_string".parse()?) // can't just ? from a any old closure...
    .bar("Rust <3".into())
);

We'd have to add another method:
let conf = Config::try_build(|builder| Ok(builder
    .foo("some_user_string".parse()?)
    .bar("Rust <3".into())
))?;

This is already starting to get a little weird, but it gets worse. That code won't type-check. The reason is that for obvious reasons we want to make try_build's error type generic - let's call it E. People imagine Rust's result? to be like:
match result {
    Ok(ok) => ok,
    Err(err) => {
        return Err(err);
    }
}

But it's actually more like
match result {
    Ok(ok) => ok,
    Err(err) => {
        return Err(err.into());
    }
}

Noticed the into()? Rust will try to convert the result to the result of encasing function. Which is usually a good thing - saves you a .map_err(|err| err.into()) - but in cases like this it can be quite a problem. The reason is that we have two ?s:

The first one, from .parse()?, converts the ParseIntError to E.
The second one, from ::try_build(...)?, converts the E to whatever error type the encasing function has in its Result.

But... Rust doesn't know what E is! It can be any type that supports these two conversions.

At this point, would you not agree that it's at least a little bit "worse syntactically"?

I didn't really consider the case where we might want to ? within the closure, and you're right that my idea totally breaks down here.


Unable to compile Android aar using Linux Docker image

Plugin Version or Commit ID
v0.11.0
Unity Version
2021.3.18f1
Your Host OS
Windows 11 PRO
Target Platform
Android
[Windows Only] Visual Studio C++ and Windows SDK Version
No response
[Linux Only] GCC/G++ and GLIBC Version
No response
[Android Only] Android Build Tools and NDK Version
No response
[iOS Only] XCode Version
No response
Command Sequences
python build.py build --bazel_build_opts "--experimental_scale_timeouts=10.0 --http_timeout_scaling=10.0" --android arm64 --android_ndk_api_level 21 --linkopt=-s -vv
Log
sh-5.1$ ls.txt
Additional Context
I need to compile the library without symbols. The command I used last i found in another thread but doesn't amtter the command I use, the error is always the same. I tried on multiple systems and I managed to compile it only once (but it wasn't with the right flags) and then never again. I need to compile the specific version 0.10.3 but i can't compile any version at all.

I found the problem. Docker Desktop was set integrating with WSL. Once I disabled that it worked perfectly


vfredsum should refine its choice of additive identity

The current spec says that masked vfredsum instructions are permitted to substitute a +0 for a masked-off element, which simplifies some datapath designs.  That's still a fine idea, but to propagate signed zeros correctly, I think we should slightly refine the definition to preserve signed-zero behavior.  Instead of unconditionally using +0, we should use +0 when rounding towards -Inf, and -0 for all other rounding modes.  That way, if a partial sum is zero, adding in the identity element will preserve the partial sum's sign bit.
There shouldn't be any critical-path impact, and the HW cost is pretty trivial.

can you prepare a PR?


Add export to pgfplots (latex)

Add a pgfToPlot() method.
This could be implemented via an additional plotting engine.
See
http://pgfplots.sourceforge.net/

PGFPlots draws high--quality function plots in normal or logarithmic scaling with a user-friendly interface directly in TeX. The user supplies axis labels, legend entries and the plot coordinates for one or more plots and PGFPlots applies axis scaling, computes any logarithms and axis ticks and draws the plots. It supports line plots, scatter plots, piecewise constant plots, bar plots, area plots, mesh-- and surface plots, patch plots, contour plots, quiver plots, histogram plots, polar axes, ternary diagrams, smith charts and some more.

edit Matthias: added clarification and link

What should this method do? Not clear.

Can this be closed?
Not clear what it means and seems outdated. Please close or update.

I'd like. Amthod thst outomarically create pfgplot code for direct pasting
into latex documents. Pgfolot is a great plotting package for latex. Keep
open for now if possible, it's a long term project.
Herbert
On Fri, Oct 6, 2017 at 2:40 AM Matthias König<EMAIL_ADDRESS>wrote:

Can this be closed?
Not clear what it means and seems outdated. Please close or update.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sys-bio/tellurium/issues/9#issuecomment-334708188,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAZDiIjbT_cJtY3dw2s5Bn10V0npouZks5spfVngaJpZM4Bz43L
.

I have added some clarification to the issue.

This might be a project I can do as I am familiar with pgfplots.
Herbert
On Fri, Oct 6, 2017 at 9:15 AM, Matthias König<EMAIL_ADDRESS>wrote:

I have added some clarification to the issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sys-bio/tellurium/issues/9#issuecomment-334801373,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAZDiF5XLuyxx7O2kaj7BOvNS6_DWqIks5splIjgaJpZM4Bz43L
.

Do we still want pgfplot export from tellurium?

Let's close this for now.
H
On Thu, Oct 31, 2024 at 4:59 PM Lucian Smith @.***>
wrote:

Do we still want pgfplot export from tellurium?
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/sys-bio/tellurium/issues/9*issuecomment-2451039148__;Iw!!K-Hz7m0Vt54!iPuBnNDlv9pctbJhuf7VwKErAZnXBEbbm7PaCcOjIwLKpqqnF9Zp7HMzZazZItMkNlTjVZAuE54PXwAVdDyGWPR723ZbRw$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AAIBSDWVJRPZ6U4KSNYPBATZ6K75ZAVCNFSM6AAAAABQ7KWKU6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJRGAZTSMJUHA__;!!K-Hz7m0Vt54!iPuBnNDlv9pctbJhuf7VwKErAZnXBEbbm7PaCcOjIwLKpqqnF9Zp7HMzZazZItMkNlTjVZAuE54PXwAVdDyGWPTfQH_5KQ$
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Herbert Sauro, Professor
Director: NIH Center for model reproducibility
University of Washington, Bioengineering
206-685-2119, www.sys-bio.org, http://reproduciblebiomodels.org/
Mobile:<PHONE_NUMBER>
@.***
Books: http://books.analogmachine.org/

Done!


Duplicate Auto-Generated Types

Describe the Bug
Having updated to 0.150.1 I started experiencing issues with Strawberry's auto-generated types.
I am using a PaginatedConnection type (similar to the Strawberry guide) to return paginated results for a particular object. Previously the relevant output annotation PaginatedConnect[OutputType] could be returned from multiple fields.
Since the update this raises strawberry.exceptions.duplicated_type_name.DuplicatedTypeName. The duplicate is the subsidiary OutputType which is exposed as an edge field with a dynamically generated name OutputTypeEdge.
This seems like a bug not a feature as it is important for multiple fields to be able to return the same type - even when those types are auto-generated by Strawberry.
Additional Context

It seems this release is probably relevant

@gravy-jones-locker this is definitely a bug, do you have a small reproduction?
We had this bug and we fixed it, so maybe there was a regression somewhere

I am also facing same issue. First time strawberry user.
strawberry-graphql version == 0.152.0
Exception: strawberry.exceptions.duplicated_type_name.DuplicatedTypeName: Type PlayerOptionalConnection is defined multiple times in the schema
Reproduction:
# straw/schemas/pagination.py
# Same as in documentation
from typing import List, Generic, TypeVar, Optional
import strawberry
from strawberry import UNSET

GenericType = TypeVar("GenericType")

@strawberry.type
class Connection(Generic[GenericType]):
    page_info: "PageInfo"
    edges: List["Edge[GenericType]"]

@strawberry.type
class PageInfo:
    has_next_page: bool
    has_previous_page: bool
    start_cursor: Optional[str]
    end_cursor: Optional[str]

@strawberry.type
class Edge(Generic[GenericType]):
    node: GenericType
    cursor: str

# straw/schemas/team.py
import strawberry
from typing import TYPE_CHECKING, Annotated
from straw.schemas.pagination import Connection

if TYPE_CHECKING:
    from straw.schemas.player import Player

@strawberry.type
class Team:
    id: strawberry.ID
    name: str

    @strawberry.field
    def players(self, first: int = 2, after: str | None = None) -> Connection[Annotated["Player", strawberry.lazy("straw.schemas.player")] | None]:
        pass

@strawberry.type
class TeamQuery:
    @strawberry.field
    def teams(self) -> Connection[Team | None]:
        return list()

# straw/schemas/player.py
import strawberry
from straw.schemas.pagination import Connection

@strawberry.type
class Player:
    id: strawberry.ID
    name: str

@strawberry.type
class PlayerQuery:
    @strawberry.field
    def players(self) -> Connection[Player | None]:
        return list()

# straw/schema.py
import strawberry
from straw.schemas import team
from straw.schemas import player

@strawberry.type
class Query(
    team.TeamQuery,
    player.PlayerQuery,
):
    pass

schema = strawberry.Schema(query=Query)

@gauthamkamath thank you for the repro, I will look at this tonight!

Hey team, any updates on this?
I am also facing the same issue.

@skilkis Unable to test this because of #2469

@gauthamkamath thanks for the bug report, I'll have a look!

After upgrading to v 0.155.4, I am not getting this issue anymore. Thank you @skilkis !


Set space owner as the resource owner in Keycloak

@alexeykazakov isn't this done ?

No, it's not done yet.

Not relevant anymore. We are not going to use Keycloak for AuthZ.
Closing.


The read metrics

Hi there,
Looking at the attached screenshot, I wonder what is wrong with the answer from my relay, so that the read metrics still stay red.
Can you help understand how to satisfy its logic?

I looked into it briefly, and it does appear that the read results for your relay are incorrect for some reason. Unfortunately, I am away from my workstation today, but I will take a deeper look when I am capable of doing so,
Some context, I am presently refactoring the entire stack and so the library presently responsible for checking relays on nostr.watch is a bit out of date.
Thanks for bringing this to my attention, I’ll let you know when I know a bit more.

Thanks for the rapid reply!
No worries at all. There is nothing urgent here.


xmrig-proxy: 6.4.0 -> 6.10.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/xmrig/xmrig-proxy/releases.
meta.description for xmrig-proxy is: "Monero (XMR) Stratum protocol proxy"
meta.homepage for xmrig-proxy is: "https://github.com/xmrig/xmrig-proxy"
meta.changelog for xmrig-proxy is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 6.10.1 with grep in /nix/store/azpmx8nml5jlk84bsl9widv38am13czq-xmrig-proxy-6.10.1

directory tree listing: https://gist.github.com/e4e3b572da02f13436cf29e111d086f8

du listing: https://gist.github.com/f14efbb7761e096cb197a5d24abf95d7

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

1 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
0 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
xmrig-proxy

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/azpmx8nml5jlk84bsl9widv38am13czq-xmrig-proxy-6.10.1 \
  --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 xmrig-proxy https://github.com/r-ryantm/nixpkgs/archive/1a7de50b77f1892a1bed918f5c50f704ede71b59.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/azpmx8nml5jlk84bsl9widv38am13czq-xmrig-proxy-6.10.1
ls -la /nix/store/azpmx8nml5jlk84bsl9widv38am13czq-xmrig-proxy-6.10.1/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 run on x86_64-linux 1

1 package built:

xmrig-proxy

Maintainer pings
cc @aij for testing.

new release available


SchemaMismatchError: Schema error: Cannot merge types long and double

Environment
**Delta-rs version **: 0.17.2
**Binding **:  Python
Environment:

Cloud provider: AWS
OS:
Other:

Bug
What happened:
Why doing write_to_deltalake(path,df,schema='merge', mode='append',engine='rust', partition_by=['date']) the df is not able to convert the schema of the table from long to double
What you expected to happen:
To Convert the evolution of schema
How to reproduce it:
Create a table with existing field contain int
change the field value to double
More details:

@ion-elgreco this isn't a bug, we don't allow type expansion yet.

Is this on the roadmap?


Addd social media and feedback section in blog page

Addd social media and feedback section in blog page

Hello, I trust this message finds you well. I'm writing to express my eagerness to contribute by adding a Social Media and feedback section to this platform. I believe this addition will not only enhance user engagement but also provide valuable insights for improvement. I'm excited to make this section visually appealing and user-friendly. Could you please assign me this task? I'm looking forward to the opportunity to contribute further.

Ready to roll for GSSoC 24, assigning me an issue would be fantastic

I can fix this issue properly. Kindly assign this issue to me.

@richasriva proceed

Okay, @Anishkagupta04, Thankyou for assigning.

@Anishkagupta04 I am little bit confused that where to merge blog section, in the main page or create a new html page for blog and feedback system? Kindly Confirm.

In the main page
On Sun, 12 May, 2024, 1:40 pm Richa Srivastava, @.***>
wrote:

@Anishkagupta04 https://github.com/Anishkagupta04 I am little bit
confused that where to merge blog section, in the main page or create a new
html page for blog and feedback system? Kindly Confirm.
—
Reply to this email directly, view it on GitHub
https://github.com/Anishkagupta04/RAPIDOC-HEALTHCARE-WEBSITE-/issues/22#issuecomment-2106160004,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDTDDF2GSTQ7BIN6DXXYOKTZB4PXVAVCNFSM6AAAAABHQYK6YGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBWGE3DAMBQGQ
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

@rajlaxmi03 Okay

Hi, @Anishkagupta04 I have solved this issue, kindly check PR #59 for the same.

@rajlaxmi03 Thanks for suggestion. After raising an issue next time, ask to assign the issue. So that we will get clear and assign you. Sorry for the miscommunication had here


✨ Track ks/OTP release 0.1.9

Summary
This PR updates the version of ks/OTP referenced. This PR also updates the release doc's list of where those references are found.
Related issue(s)
Fixes #

/cc @pdettori

/cc @vMaroon

/approve


BUG - Incorrect DrillState when searching in a hierarchy

Related Feature
Hierarchy support in MockServer
Description
When searching within a hierarchy, the returned DrillState property is sometimes equal to leaf even if the node has children
@nlunets

Fixed


Lack of / in paths means files can't located by GH

For instance:

Demo repo at https://github.com/Lawouach/reliably-demo
is this a bug or something I miscofigured?.

@Lawouach could you try with single quotes for your director
        uses: reliablyhq/gh-action@v1
        with:
          dir: './manifests'

You might also want to use the tagged version for the GH action

@Lawouach could you try with single quotes for your director
        uses: reliablyhq/gh-action@v1
        with:
          dir: './manifests'

You might also want to use the tagged version for the GH action

Btw, don't know If you remember, but we also have a public demo repo https://github.com/reliablyhq/action-demo

Btw, don't know If you remember, but we also have a public demo repo https://github.com/reliablyhq/action-demo

Issue is only for SARIF output:
"artifacts": [
                                {
                                        "location": {
                                                "uri": "testsmanifests/pod.yaml"
                                        }
                                }
                        ]

artifacts location is wrongly appended

Issue is only for SARIF output:
"artifacts": [
                                {
                                        "location": {
                                                "uri": "testsmanifests/pod.yaml"
                                        }
                                }
                        ]

artifacts location is wrongly appended

fixed by https://github.com/reliablyhq/cli/commit/db411717919a8cd75713b35f525f6e28386b395c

fixed by https://github.com/reliablyhq/cli/commit/db411717919a8cd75713b35f525f6e28386b395c

hot fix merged into main; can close now

hot fix merged into main; can close now


Implement release notes for v0.6.0

Create release notes for v0.6.0.
Alex, please take care of this task.

Hello Szilveszter,
Note is created . You can is in releases section.