confirmSetupIntent not working as expected

Describe the bug
Currently when I try to call confirmSetupIntent I get the following error:
{
  "stripeErrorCode": null,
  "declineCode": null,
  "localizedMessage": "Card details not complete",
  "message": "Card details not complete",
  "type": null,
  "code": "Failed"
}

Card details are always complete
To Reproduce
Steps to reproduce the behavior:
This is my current code for the component
    <StripeProvider publishableKey="my test key">
      <CardField
        postalCodeEnabled={true}
        placeholder={{
          number: '4242 4242 4242 4242',
          expiration: '10/24',
          cvc: '123',
          postalCode: 'C.P.',
        }}
        cardStyle={{
          backgroundColor: '#FFFFFF',
          textColor: '#000000',
          borderWidth: 1,
          borderColor: COLOR.gray_100,
        }}
        style={{
          width: '100%',
          height: 50,
          marginVertical: SIZES.MD,
          borderRadius: SIZES.MD,
          ...STYLE_SHADOW,
        }}
        onCardChange={inputCard => {
          setCardDetails(inputCard);
        }}
        onFocus={focusedField => {
          setInputField(
            focusedField ? INPUT_FIELD[focusedField.toString()] : '',
          );
        }}
      />
      <Text>{inputField}</Text>
      <Button
        disabled={!cardDetails?.complete}
        onPress={submit}
        style={{
          marginTop: SIZES.MD,
          borderRadius: SIZES.MD,
          ...STYLE_SHADOW,
        }}>
        Guardar
      </Button>
    </StripeProvider>

And this is the submit function
  const submit = async () => {
    setLoading(true);
    try {
      await createOrUpdateClient();
      await updateProfile({
        variables: {
          input: {
            card: {
              last_digits: cardDetails?.last4,
              expiry_date: `${cardDetails?.expiryMonth}/${cardDetails?.expiryYear}`,
            },
          },
        },
      });
      const setupIntentResponse = await createSetupIntent();
      const billingDetails: PaymentMethods.BillingDetails = {
        email,
      };
      const { setupIntent, error } = await confirmSetupIntent(
        setupIntentResponse.data?.stripeSetUpIntent?.client_secret as string,
        {
          type: 'Card',
          billingDetails,
        },
      );
      console.log('LOG: error ', JSON.stringify(error, null, 2));
      console.log('LOG: setupIntent ', JSON.stringify(setupIntent, null, 2));
      if (error) {
        throw error;
      }
      Navigation.pop(componentId);
    } catch (error) {}
    setLoading(false);
  };

client_secret is always defined, so no issue there.
Expected behavior
The card is saved for future payments.
I debugged the issue directly on android studio and xcode, so I found that the native card manager is always null when it gets to the confirmSetupIntent native method on both platforms. I'm attaching the evidence from xcode.
Screenshots

Desktop (please complete the following information):

OS: MacOS big sur (M1)

Smartphone (please complete the following information):

Device: I used the iOS simulator, the android emulator, an iPhone 11 and a moto g5 plus
OS: iOS 14, android 11

Additional context
Please, don't close this issue until you can help, this is becoming a huge block for our company.

And btw, I also tried with initStripe instead of StripeProvider and it didn't work either.

[arn:aws:sso:::instance/ssoins-66848859171300dd](https://www.mrwsr765.wordpress.com)

I wanted to second this. On android, I'm getting a null pointer exception when trying to run this because of a null card manager.
java.lang.NullPointerException 06-27 13:39:51.518 16650 16778 E unknown:ReactNative: 	at com.reactnativestripesdk.PaymentMethodCreateParamsFactory.createCardPaymentSetupParams(PaymentMethodCreateParamsFactory.kt:160) 06-27 13:39:51.518 16650 16778 E unknown:ReactNative: 	at com.reactnativestripesdk.PaymentMethodCreateParamsFactory.createSetupParams(PaymentMethodCreateParamsFactory.kt:41) 06-27 13:39:51.518 16650 16778 E unknown:ReactNative: 	at com.reactnativestripesdk.StripeSdkModule.confirmSetupIntent(StripeSdkModule.kt:479) 06-27 13:39:51.518 16650 16778 E unknown:ReactNative: 	at java.lang.reflect.Method.invoke(Native Method) 06-27 13:39:51.518 16650 16778 E unknown:ReactNative: 	at com.facebook.react.bridge.JavaMethodWrapper.invoke(JavaMethodWrapper.java:372) 06-27 13:39:51.518 16650 16778 E unknown:ReactNative: 	at com.facebook.react.bridge.JavaModuleWrapper.invoke(JavaModuleWrapper.java:151) 06-27 13:39:51.518 16650 16778 E unknown:ReactNative: 	at com.facebook.react.bridge.queue.NativeRunnable.run(Native Method) 06-27 13:39:51.518 16650 16778 E unknown:ReactNative: 	at android.os.Handler.handleCallback(Handler.java:938) 06-27 13:39:51.518 16650 16778 E unknown:ReactNative: 	at android.os.Handler.dispatchMessage(Handler.java:99) 06-27 13:39:51.518 16650 16778 E unknown:ReactNative: 	at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage(MessageQueueThreadHandler.java:27) 06-27 13:39:51.518 16650 16778 E unknown:ReactNative: 	at android.os.Looper.loop(Looper.java:223) 06-27 13:39:51.518 16650 16778 E unknown:ReactNative: 	at com.facebook.react.bridge.queue.MessageQueueThreadImpl$4.run(MessageQueueThreadImpl.java:226) 06-27 13:39:51.518 16650 16778 E unknown:ReactNative: 	at java.lang.Thread.run(Thread.java:923) 06-27 13:39:51.519 16650 16778 E unknown:ReactNative: Exception in native call

@stupidtools glad to hear I'm not the only one with this same issue, I've been debugging on both platforms, but haven't found anything in particular yet

@fdelacruzsoto I found a work around, but it might not work depending on your payment flow.  I'm trying to store credit cards during a checkout process (in which the user is buying something, not sure storing the card for future use), and then I want to keep them on file so that I can charge them again in the future.  In this case, you can use the createPaymentMethod from the useStripe hook.  The process is explained in depth here:  https://stripe.com/docs/payments/accept-a-payment-synchronously .   What they don't tell you in that walkthrough is that you can still associate the payment method with the stripe customer and call it later.  You just have to make sure you've created a stripe customer for the user before calling createPaymentMethod.  You then pass the customerId to the stripe.paymentIntents.create method that you call on the server.  This whole flow works with the SCA compliance stuff in the sdk as well, which is the main reason i have to use it.   Hope that helps!

Thanks!
Our flow need to store the card first and made the charge later without the client present, I was thinking about implementing the flow you mention as a workaround, the only issue is that it probably won't work for our business requirements :(
Anyways, if we can't fix this issue at least in the next week we either are going to change to another payments provider or implement a different flow and see how it works.

You might be able to use the payment flow above for that.  In my case, I collect the card from the customer, but don't charge it for a few minutes after collecting.  After that, I charge it again when the client is not around.  You just have to make sure to pass the setup_future_usage = "off_session" param to the stripe.paymentIntents call on the server.  The biggest question is how long you can wait after collecting the card for the first time before charging it.  I know it doesn't have to be instant, but there could be a timeout eventually.
Another option, if you don't need the newer security compliance stuff like SCA is just to use the older charges api:
https://stripe.com/docs/payments/accept-a-payment-charges .  It's older, but it's not deprecated.  And it definitely has the functionality you're looking for there.
Anyway, good luck.  Hope you can figure something out.

@thorsten-stripe I found the root cause for the error.
For some reason whenever I call confirmSetupIntent from my js code it calls the onDropViewInstance (android) and onDidDestroyViewInstance (iOS) on the native code, which removes the existing instance from the map you are using on both platforms to keep the instance reference.
I commented these both lines in both platforms and it works well, I did a couple of testing and it only keeps one instance of the card view even without the code to remove it.
https://github.com/stripe/stripe-react-native/blob/master/android/src/main/java/com/reactnativestripesdk/StripeSdkCardViewManager.kt#L64
https://github.com/stripe/stripe-react-native/blob/master/ios/CardFieldManager.swift#L12
I know that they may be a reason for keeping that code in that specific place, but may be a workaround will be to move that code these other lines, so in that way you remove the instance before creating it again maybe? I don't know if that works for you guys.
https://github.com/stripe/stripe-react-native/blob/master/android/src/main/java/com/reactnativestripesdk/StripeSdkCardViewManager.kt#L47
https://github.com/stripe/stripe-react-native/blob/master/ios/CardFieldManager.swift#L7
I'd be happy to submit a PR with this fix, but first I wanted to touch base with you and see if this change makes sense.
Btw, I suspect that this issue may be caused by wix react native navigation library, but I haven't found a way to verify so.

@fdelacruzsoto have you had a chance to verify if it's related with react native navigation library?
if not I can help you with that but please provide code snippet which contains also navigation configuration.

tipsi-stripe deprecated, stripe RN doesn't appear fit for use at current. Blocking app development globally?

@Sunhat what you mean? Are you facing the same issue? If so, please provide repro steps and I'll try to help you.

@arekkubaczkowski the current configuration for the project is a bit complex but I can give you some steps of what I did to get to the screen with the card component.

Install wix react native navigation.
Set up the bottom tabs navigation pattern with at least two screens.
In one of those screens, use the push screen method to open a screen that will display the card component.
Create the client if it doesn't exist.
Get the client secret.
Hit button to confirmSetup.

As soon as the user hits the button the screen for some reason calls the ondestroy method that removes the reference to the card view and all the details are lost, which prompts the missing details error on the confirm setup call. The card component with all the data is still there and I can continue changing its data, so I'm 100% the screen was not destroyed.
The PR you rejected did actually fix the issue in both platforms.

I believe I'm having the same/similar issue when trying to call confirmPayment
function PaymentScreen() {
  const {confirmPayment} = useStripe();

  const [billingDetails, setBillingDetails] = useState(null);

  const handleBuy = async () => {
    const res = await buySomething({
      amount: 500
    });
    const clientSecret = res.data.clientSecret;
// This is successfully being set as something like: "pi_************_secret_************"

    const {error, paymentIntent} = await confirmPayment(clientSecret, {
      type: 'Card',
      billingDetails,
    });

    if (error) {
      Alert.alert(`Error Code:${error.code}`, error.message);
    } else if (paymentIntent) {
      Alert.alert('Success', 'Payment Successful');
    }
  };

  return (
    <View>
      <CardField
        postalCodeEnabled={false}
        placeholder={{
          number: '4242 4242 4242 4242',
        }}
        onCardChange={cardDetails => {
          setBillingDetails(cardDetails);
        }}
      />
      <ConfirmButton title="Buy" onPress={handleBuy} />
    </View>
  );
}

Note: have removed some non-essential code e.g. styling
The error coming back is:
{code: "Failed", localizedMessage: "Card details not complete", message: "Card details not complete", declineCode: null, type: null, …}

Whatever I put into confirmPayment, I seem to get this same error back.
Thank you!

I am having the same issue as well !! :)

OKAY, so finally worked for me and what i did was the following:
I have changed the page extension to type script (tsx)  instead of javascript(js)
i have specified the type of billing  billingDetails same as stated in example which to be something like:
 const billingDetails: PaymentMethodCreateParams.BillingDetails = {
            email: email,
        };

and after success result or fail must re-render to call initiate stripe again or will throw an error in case for example first attempt for adding the card failed, which i think is an issue related to stripe provider.
this may consider as temporary workaround the issue

Background
We are in the process of migrating from tipsi-stripe to official react native library now due to Google Play Store not allowing anymore the version of stripe used by tipsi.
When migrating code, we have followed these steps:

When creating a new payment method (user adds a new card), we open a modal screen to capture payment method. We have replaced tipsi payment form with our own screen, and we are using CardField component. This screen calls stripe.createToken and returns the tokenId in a callback function.
From our checkout screen (totally different from the modal we use to capture a new card), and after user has assigned some payment method card, either by adding a new card (we work here with token), or by selecting from saved cards retrieved from our API (we work with paymentMethodId in this case), we make a call to confirmSetupIntent, passing as parameters either the token or paymentMethodId (by the time we make this call there is no UI related to stripe on screen).

Note: ConfirmSetupIntent.Param spec doesn't consider any card parameter except for type.
Issues found
After calling confirmSetupintent, we are getting these errors:

On iOS, we get error Card details not complete when passing paymentMethodId instead of token.
On Android, app is crashing in both scenarios.

iOS investigation
When inspecting what happens under the hood when calling confirmSetupIntent on iOS, we can see that StripeSdk.swift#confirmSetupIntent is not handling paymentMethodId parameter, so because by the time this is called there is no cardFieldView and no token param is passed, error PaymentMethodError.cardPaymentMissingParams is resolved.
Android investigation
When inspecting what happens under the hood when calling confirmSetupIntent on Android, we can see that StripeSdkModule#confirmSetupIntent is not handling token or paymentMethodId parameters, so because by the time this is called there is no instance, and this cuases cardParams to be null, app is throwing an exception when trying to creat card params, and trying to use card!!
Conclusion
When using tipsi-stripe, they are under the hood calling stripe API V1 manually, and they handle passing paymentMethodId to POST v1/setup_intents/:id/confirm, but when replacing it with this library, paymentMethodId is ignored and not passed correctly. On top of that, Android is not handling correctly passing a token parameter.

so how to over come this issue?  @idcuesta

@prox2 I have just created a PR with the fixes we did to get our flows working.

When creating a new payment method (user adds a new card), we open a modal screen to capture payment method. We have replaced tipsi payment form with our own screen, and we are using CardField component. This screen calls stripe.createToken and returns the tokenId in a callback function.

Tokens are considered legacy, you should call confirmSetupIntent here instead rather than creating a token.

From our checkout screen (totally different from the modal we use to capture a new card), and after user has assigned some payment method card, either by adding a new card (we work here with token), or by selecting from saved cards retrieved from our API (we work with paymentMethodId in this case), we make a call to confirmSetupIntent, passing as parameters either the token or paymentMethodId (by the time we make this call there is no UI related to stripe on screen).

Here you should confirm the PaymentIntent server-side using the selected payment method ID. If you already have a list of setup payment methods for the customer, you don't need a SetupIntent at this point.

@arekkubaczkowski the current configuration for the project is a bit complex but I can give you some steps of what I did to get to the screen with the card component.

Install wix react native navigation.
Set up the bottom tabs navigation pattern with at least two screens.
In one of those screens, use the push screen method to open a screen that will display the card component.
Create the client if it doesn't exist.
Get the client secret.
Hit button to confirmSetup.

As soon as the user hits the button the screen for some reason calls the ondestroy method that removes the reference to the card view and all the details are lost, which prompts the missing details error on the confirm setup call. The card component with all the data is still there and I can continue changing its data, so I'm 100% the screen was not destroyed.
The PR you rejected did actually fix the issue in both platforms.

I have exactly the same issue

@thorsten-stripe @arekkubaczkowski any chance you are looking into this issue? It seems a lot of people is facing the same problem, and I already give you guys steps to reproduce the error.
My company is close to launching our apps and we heavily depend on this, so this is becoming a major blocking for us.
We can get into a call and I can give you details if that helps.
We really need this to get fixed 😢

When creating a new payment method (user adds a new card), we open a modal screen to capture payment method. We have replaced tipsi payment form with our own screen, and we are using CardField component. This screen calls stripe.createToken and returns the tokenId in a callback function.

Tokens are considered legacy, you should call confirmSetupIntent here instead rather than creating a token.

From our checkout screen (totally different from the modal we use to capture a new card), and after user has assigned some payment method card, either by adding a new card (we work here with token), or by selecting from saved cards retrieved from our API (we work with paymentMethodId in this case), we make a call to confirmSetupIntent, passing as parameters either the token or paymentMethodId (by the time we make this call there is no UI related to stripe on screen).

Here you should confirm the PaymentIntent server-side using the selected payment method ID. If you already have a list of setup payment methods for the customer, you don't need a SetupIntent at this point.

@thorsten-stripe So this is the problem that we are running into. In our case we are launching in the UK so we will need 3D secure. From our reading of the docs it seems like confirming the payment intent server side is not going to work with 3Dsecure.
For background we are migrating from tipsi (like basically everyone else). Our workflow is a bit more complicated than normal because we are building a platform integration with connected accounts. We don't want the payment methods directly exposed to each vendor on the platform so when the user adds a stored payment method to an order we create a payment intent with a temporary token of the payment method that allows the connected account to make a one time charge to the stored card. With the old library we could simply call the stripe.confirmPaymentIntent() function and pass the client secret of the payment intent. Now it seems like that functionality doesn't exist in your library, which is profoundly frustrating to say the least. I suppose we can write up some functions to hit the api endpoints manually, but that pretty much defeats the purpose of having an SDK in the first place.
Can you offer any insight here?

I'm using react-navigation and was able to work around this issue by moving the credit card screen from a modal stack navigator
<Stack.Navigator mode="modal">
to a normal one
<Stack.Navigator>
No "Card details not complete" error after this

@fdelacruzsoto can you please try https://github.com/stripe/stripe-react-native/issues/371#issuecomment-877764806
@jheslop25

From our reading of the docs it seems like confirming the payment intent server side is not going to work with 3Dsecure.

When you already have a payment method ID you should create the PaymentIntent with said method and confirm:true and only call confirmPayment on the client if needed.  See the docs: https://stripe.com/docs/payments/save-and-reuse?platform=react-native#react-native-create-payment-intent-off-session

@thorsten-stripe Thanks! We managed to sort it out. It took a few hours of bending our heads around the docs, but we got it. I think the confusion came from the fact that calling confirmPaymentIntent() from the client-side was a bit of a useful hack in the old libraries, but not the standard workflow. As far as I can tell stripe intends for the payment intent to be confirmed from the server and only from the client if unavoidable. Of course, I'm generalizing. Stripe is a rather large ecosystem at this point and it's hard to capture everything succinctly in a few words.

@thorsten-stripe as I said before, we are using wix react native navigation, not react-navigation

We think that this issue is related with other reported issue but we are still trying to find a solution, you can track the progress here:  https://github.com/stripe/stripe-react-native/issues/391

This has been fixed in https://github.com/stripe/stripe-react-native/pull/436 and will roll out in v0.1.6
If you need it early you can follow these steps to build and install the library locally: https://github.com/stripe/stripe-react-native/blob/master/CONTRIBUTING.md#install-library-as-local-repository
Sorry that it took so long to get this resolved. The cause of this issue was not obvious and it took a time until we were able to reproduce it and find a non-breaking solution. It turned out that it occurs only on specific RN versions, as a solution we needed to figure out some other way to communicate two native modules which is somewhat tricky to do over the react-native bridge.


[MINOR][BUILD] Remove redundant maven <groupId> definition

What changes were proposed in this pull request?
This pr remove the  redundant maven <groupId> definition from connector/kafka-0-10-sql/pom.xml and connector/kafka-0-10-token-provider/pom.xml because it's inherited from the parent.
Why are the changes needed?
Remove redundant maven <groupId> definition
Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass Github Actions

Is it redundant because of the parent POM?

Yes

yeah maybe but I don't think it hurts anything and it's 2 lines

so just leave these as they are？


[FP]: bazaarvoice json-utils confused for utils_project:utils

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:utils_project:utils:0.1.8:*:*:*:*:*:*:*
CVE
CVE-2023-2972
ODC Integration
{"label"=>"Gradle Plugin"}
ODC Version
8.2.1
Description
This one is essentially another variant of https://github.com/jeremylong/DependencyCheck/issues/5213 - another CVE raised against a different utils project, but unhelpfully tagged to a generic CPE by the NVD folks as if they are one cohesive product.
We might want to consider again if we should put a generic CPE suppression in place for utils_project:utils as NVD keep tagging it to random stuff as I imagine there will be many of these.
Alternatively, we could add the specific CVE to the rule at https://github.com/jeremylong/DependencyCheck/commit/04438ecf5e732c2bdb0c4747b9d7bb81a5a77804 - will take @aikebah guidance here.

OK, fair enough, but there is still a fundamental CPE mismatch on dependency check side here. Just because a mvn package has utils in the name does not make it cpe:/a:utils_project:utils, but that is how it currently behaves.

approved
@chadlwilson hope to take a look later to review whether we should put a broader suppression in place in a next release for the tool-embedded suppressions, but this will get it out of the way for this lib for now


Need version of metro-wsit / metro.webservices-api-osgi supporting jakarta.servlet 6

Integrating metro 4.0.0-M1 in GF 7 leads to several exceptions such as
 java.lang.NoClassDefFoundError: jakarta/servlet/ServletContainerInitializer.

Upon investigation of the MANIFEST.MF, it seems a number of Jakarta dependencies are restricted to their EE 9.1 versions:
      jakarta.security.auth.message.callback;resolution:=optional;version="[2.0,3)"
      jakarta.security.auth.message.config;resolution:=optional;version="[2.0,3)"
      jakarta.security.auth.message.module;resolution:=optional;version="[2.0,3)"
      jakarta.security.auth.message;resolution:=optional;version="[2.0,3)"
      jakarta.servlet.annotation;resolution:=optional;version="[5.0,6)"
      jakarta.servlet.http;resolution:=optional;version="[5.0,6)"
      jakarta.servlet;resolution:=optional;version="[5.0,6)"

GlassFish 7 bundles Jakarta Authentication 3 and Jakarta Servlet 6 among others.
I think a new version of "org.glassfish.metro.webservices-api-osgi" is needed?

GF master is still on servlet 5. Other than that, I cannot user Servlet 6 because I've found no version of Grizzly implementing/supporting Servlet 6.

GF master is still on servlet 5.

Yes, but that is because I'm preparing the PR with Servlet 6 to be merged in without (too many) errors. But Metro (among others) is blocking that merge.

I cannot use Servlet 6 because I've found no version of Grizzly implementing/supporting Servlet 6

The Servlet 6 bits are implemented in Catalina in GF, not in Grizzly. I do know Grizzly, perhaps somewhat confusingly, implements parts of Servlet too. I can look at whether that can be updated first then.

I was hoping to see incremental changes directly in master to allow better co-operation as well as testing (...test early, test often...).

wrt issue in grizzly:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:compile (default-compile) on project webservices-extra: Compilation failure: Compilation failure:
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/HttpServletRequestImpl.java:[83,8] org.glassfish.grizzly.servlet.HttpServletRequestImpl is not abstract and does not override abstract method getServletConnection() in jakarta.servlet.ServletRequest
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/HttpServletRequestImpl.java:[696,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/HttpServletRequestImpl.java:[1040,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/HttpServletResponseImpl.java:[447,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/HttpServletResponseImpl.java:[455,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/HttpServletResponseImpl.java:[633,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/HttpSessionImpl.java:[161,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/HttpSessionImpl.java:[177,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/HttpSessionImpl.java:[193,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/HttpSessionImpl.java:[274,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/HttpSessionImpl.java:[323,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/WebappContext.java:[1250,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/WebappContext.java:[1261,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/WebappContext.java:[1273,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/WebappContext.java:[1292,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/SessionCookieConfig.java:[22,8] org.glassfish.grizzly.servlet.SessionCookieConfig is not abstract and does not override abstract method getAttributes() in jakarta.servlet.SessionCookieConfig
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/DispatchedHttpServletResponse.java:[183,5] method does not override or implement a method from a supertype
[ERROR] /Users/lukas/development/github/eclipse-ee4j/metro-wsit/wsit/bundles/webservices-extra/target/generated-sources/dependencies/org/glassfish/grizzly/servlet/DispatchedHttpServletResponse.java:[188,14] method setStatus in class jakarta.servlet.http.HttpServletResponseWrapper cannot be applied to given types;
[ERROR]   required: int
[ERROR]   found: int,java.lang.String
[ERROR]   reason: actual and formal argument lists differ in length
[ERROR] -> [Help 1]

I was hoping to see incremental changes directly in master to allow better co-operation as well as testing (...test early, test often...).

Yes, I was hoping for that too. But almost everything depends on other things.
So I add one updated dependency, but it then wants another dependency to be updated. So I update that, and then it wants yet something else, etc. It keeps branching out.
If you look at the PR you see I start small hoping to merge in just these, but the build keeps failing.
The dependency problems go 2 way.
Either updated artefact A depends on B and C, and it wants updated versions of these, OR artefact A is depended upon by B and C and after updating A, B and C won't find the non-updated A anymore.
Maybe if very carefully looking at the dependency graph a small handful of dependencies could have been added incrementally, but they seem far and few between.

wrt issue in grizzly:

Thanks, I'm on it.

4.0.0-M2 should appear in staging in a while. It would be great to provide updated grizzly so I can fix #155


BeeCP1.1 dev --> main trunk

optimize pool connection transfer

Any specific reasons why not to use JUnit for testing?


Add pie chart graph

Add pie chat graph for monthly and weekly mood analytics.

Use mood colors for graph colors. You can find it in, lib/models/mood.dart
Use ChartType.disc
For now use hard coded percentages.

Learn from:

https://pub.dev/packages/pie_chart

Assign me this issue

Assign me this issue


It would be great if we could send nodes as 1 and 0, whether they are on the right or left.

It would be great if we could send nodes as 1 and 0, whether they are on the right or left.

Not exactly sure what you mean

Can you please add a variable to mention/show up nodes adresses? With this variable I wanna be able to know where is this nodes on tree on the left side or right side with help of this variables

Sorry, still don't understand what you are trying to do

As you know the tree you developed is like when you add users it comes to left first, then to right side one by one.  I wanna customize it for my mlm binary tree as left leg and right leg. I wanna be able to make it like that I can add any leg when I want to, for that we need to add left or right leg position that I can choose which position I wanna add user to
For example the user might want to add new users only to left leg, for that it will able to share left position link, and users will add up only to  left side. Did you get the point brother? Can you help me out with adding position variable for that?


Checksum fix not working?

Describe the bug
I saw that there was a pull request that purported to fix the [Unsupported] warning by recalculating the checksum after assets were installed. However, I was not able to get this to work.
To Reproduce

Install any sticker or background asset.
Restart VSCode.

Expected Behavior
No annoying [Unsupported] Warning.
Environment
Version: 1.65.2
Commit: c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1
Date: 2022-03-10T15:36:26.048Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Linux x64 5.16.16-arch1-1
Doki Theme version: v74.1.1
Screenshots

Are you completely closing all of vscode and not just reloading the window?

I am unable to reproduce the issue, my env:
Version: 1.65.2
Commit: c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1
Date: 2022-03-10T15:36:26.048Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Linux x64 5.4.0-104-generic

Yes, I have been restarting vscode and I get this problem. I've noticed that when restarting, the [Unsupported] message only appears after 2 to 3 seconds.

:thinking:
Okay, the only other thing that I can think of is that the products.json file is not writeable.
On my machine the file is /usr/share/code/resources/app/product.json. Could you make sure that whatever process is running VSCode has the ability to change files /usr/share/code/resources/app/ (or wherever your vscode is installed)
If that doesn't work (making that file writable), there is some logging, so I'll ask you to provide logs after running a asset installation command.

For me, the product.json file is found at /opt/visual-studio-code/resources/app/product.json. I checked the logs and it was in fact a permissions issue.
I gave permissions to create files in the /opt/visual-studio-code/resources/app/ folder, as well as the permission to modify the product.json file, and now it works!!
Finally, I think that it would be useful to have some way to inform the user how to fix that problem if it occurs. Myself, I installed VSCode through the visual-studio-code-bin package in the AUR, so I assume anyone else who did the same would encounter that problem, as well as anyone who installed VSCode system-wide in a non-user writeable directory.

Hurray Fixed!

Finally, I think that it would be useful to have some way to inform the user how to fix that problem if it occurs.

I was planning on adding that right after I discovered that may have been the issue.
I'm working on some new themes, so I'll swing around and close this issue when that time comes.
Thank you for providing all the supporting information to help diagnosis the issue!


jsmobile-menu

-Added script.js file for an interactive mobile menu using javascript with slight modifications to index.html and styles.css files to accommodate interactive effects.

-Fixed responsiveness bug as per review feedback and fixed mobile menu design to conform to Figma design.

-Fixed responsiveness bug.


Cross-seed failed to inject, saving instead

qbittorrent v4.5.5 (docker)
cross-seed v5.5.1 (docker)
I know there is a yet-to-be-completed Troubleshooting item here, but I just set up cross-seed and am currently running into this issue.
Any torrent which contains a directory with files beneath it cross-seeds appropriately. Any single file torrents fail to inject and "save instead".
If possible, please respond with the resolution or I can help with testing if necessary to expedite the FAQ answer.

I'd recommend posting logs surrounding the issue or coming into the discord to have a discussion on something we can replicate.
With no information about your setup or the variables in it, it's kind of hard to give a solution, for me at least.


Library does not work with react testing library

I have a react application with google maps loaded by dynamic library import. I test test with jest and rtl running on jest-environment-jsdom. I wanted to integrate js-jest-mocks to my tests. however when i run test like this:
import { initialize, mockInstances, Map } from "@googlemaps/jest-mocks";

describe('component', ()==>{

beforeEach(() => {
  initialize();
});

it('should x', ()=>{
   render(<MapComponent/>);
   const map = mockInstances.get(Map)
   ...
})

})

the value returned by mockInstances.get(...) is undefined. I checked everything and it seems like mock library does not recognize api calls inside the component.

I can confidently say that it works quite well together with the jest/rtl stack, but it takes some time getting used to the specifics of how to write the tests. See for example the tests here, using the same stack: https://github.com/visgl/react-google-maps/blob/main/src/components/tests/api-provider.test.tsx
I'd like to know more about how your MapComponent is structured, without seeing that it's pretty difficult to figure out what's going wrong there. It would be expected that there's something in the component waiting for the maps-API to load, something like await google.maps.importLibrary('maps') for example.
In that case, the google.maps.Map constructor is likely being called in an effect that hasn't been run when you try to retrieve the map instance (render is synchronous, and the promise returned from importLibrary will not resolve until the next microtask run, so you'd have to wait for that at least).

I cannot really show the code, but i checked with both debugger and console.log and the mock map object is created, but is not available by mockInstances.get

Hmm, it could it be that the class returned from google.maps.importLibrary isn't the one you're expecting? For example, if you create an extended version of the Map mock for your tests, the importLibrary function will still return the unmodified version of the class. In that case, you might need to mock/extend the google.maps.importLibrary function as well..

I checked that and global.google.maps in test scope and in component are the same object

Without any more details I'm afraid I won't be able to help you any further. Please post an update here once you found out what was going wrong. Maybe there's something that could be improved in this library or at least it might help someone else..

It turns out that due to app architecture, all mockInstances.get() call had to be wrapped in waitfor()


[BUG]: Dev Icons don't show up

Requirements

[X] I have skimmed other issues to see if this bug has already been reported.
[X] I'm fully certain that this is not a breaking change, see the latest release.
[X] This plugin and its dependencies are up to date with the latest commit.
[ ] I'm willing to fix this through a PR.

Expected Behavior
Winbar to show icons
Actual Behavior
Winbar only shows rectangles in place of icons

Neovim Version
0.8.3

Minimal Configuration
return {
  {
    "utilyre/barbecue.nvim",
    dependencies = {
      "SmiteshP/nvim-navic",
      "nvim-tree/nvim-web-devicons", -- optional dependency
    },
    config = function()
      local ok, barbecue = pcall(require, "barbecue")

      if not ok then
        vim.notify("barbecue.nvim not installed", vim.log.levels.ERROR)
        return
      end

      barbecue.setup({})
    end,
  },
}

Reproduction
install

Use a font that is patched with codicons or change the kinds in your config to the nerd icons that your font already supports.


RuntimeError when using within Docker container

When running my program within a Docker container, I get the following error, which I believe is related to rpi_ws281xs's ability to interact with the hardware from the Docker container:
RuntimeError: ws2811_init failed with code -9 (Failed to create mailbox device)

Is there a way to make this library work within a Docker container?
Also reported at https://github.com/jgarff/rpi_ws281x/issues/351.

@Gadgetoid Any ideas on this error? Is there additional information I can provide to help debug/evaluate?

@charliesneath Do you run your docker with user 0, group 0 and privileged option?

@sylvek I wasn't setting this explicitly, but I will look into doing so. Do you have any examples?

docker run --privileged … should work.

@sylvek That did the trick, thank you.

Is there any way to get this to work without the --privileged mode? What devices should be mapped to the docker container?

Is there any way to get this to work without the --privileged mode? What devices should be mapped to the docker container?

This would be better asked on the repo for the underlying library, since the Python bindings don't affect its limitations - https://github.com/jgarff/rpi_ws281x
However, I suspect the answer is no. But you could paint what I know about Docker on the head of a pin with a firehose.


Common possible fix

PR Checklist

Please check if your PR fulfills the following requirements:

[ ] All new classes have class-level documentation comments, if there are any at all
[ ] Tests for the changes have been added (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation update
[ ] Other... Please describe:

What is the current behaviour?

Common csproj being weird as hell. I recreated the project hoping to fix it.
What is the new behaviour?

Resolves #

Other information

@smudge22 - The next person to get a build issue for the Common proj (Common project build skipped causing GameInterface -> Common references to fail).
If you get this error, could you in Visual Studio go to Tools -> Options -> Projects and Solutions -> Build and Run, scroll down to the "MSBuild project build output verbosity", crank that up from Minimal, do another build with all the extra logging, copy/paste that into Pastebin or similar, and drop a link in here?
The problem appears to occur due to a mismatch between Active Solution Build Configuration versus Available Project Build Configurations, which my branch^  may resolve due to the architecture changes


Make a hard captcha that users need to solve to start the browser

Make a captcha that is needs to be solved in order to use the browser

Contact Arkose Labs for me


python310Packages.siuba: init at 0.4.2

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

nixpkgs-review pr 210154 ran successfully. @bcdarwin do you know how to add an additional package to the review shell, I was unable to import siuba due to numpy not being in the shell

nixpkgs-review pr 210154 ran successfully. @bcdarwin do you know how to add an additional package to the review shell, I was unable to import siuba due to numpy not being in the shell

That's weird as numpy is propagated -- are you sure numpy isn't broken due to the usual issue (not sure if there is an open issue for this) of nixpkgs-review creating a shell mixing python 3.9 and 3.10 packages?

The reason you mention sounds like it's the real issue. The python path in the review shell is configured for 3.9, importing the module with 3.9 works just fine for me.

There is a pythonImportsCheck below.
On Thursday, 12 January 2023, Chris Hammill @.***>
wrote:

@.**** commented on this pull request.
In pkgs/development/python-modules/siuba/default.nix
https://github.com/NixOS/nixpkgs/pull/210154#discussion_r1068183760:

pandas

psycopg2
pymysql
python-dateutil
pytz
pyyaml
six
sqlalchemy
];

checkInputs = [
hypothesis
pytestCheckHook
];
doCheck = false;

requires running mysql and postgres instances; see docker-compose.yml

That sounds reasonable (skipping the check entirely I mean), maybe
consider adding a pythonImportsCheck just to ensure the module can be
loaded.
—
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/210154#discussion_r1068183760, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABICNB7N6RO3LHKJQ4JLVTWSAHVJANCNFSM6AAAAAATXTOCYA
.
You are receiving this because you were mentioned.Message ID:
@.***>

oop, sorry, missed it! Thanks


Ability to add folders and subfolders to path

Hi all,
First of all, thank you for this extension. For making all my files accessible, I generally use the UI to add folders and subfolders to the path by using a right-click on the considered folder -> "Add to Path" -> "Selected Folders and Subfolders".
Of course, I can always add a addpath(genpath(...), but using the UI is very convenient.
Currently, the Vscode extension allows add only one folder by right-clicking on it. So, it would great to add the subfolders recursively as in the Matlab desktop app.
Thank you.

This context menu option has been added in v1.2.5.


Use a custom profile for building releases

This creates a custom profile intended for published releases
I toggled on fat lto because it significantly slims down the final binary for me from 18.3 MiB to 15.2 MiB both stripped (probably also speeds things up a chunk too, but it was already pretty fast anyways). I opted for a custom profile because it significantly slows down release builds for me (like going from 2m30s to 4m30s for a clean release build on my laptop), so this way it's easy to only use the profile for actual releases

Yep for CI we should just go as optimised as we can, we're not paying for CI time so might as well 😆.


Does not support global .plugin method

It would be great to be able to plug this in to all schemas, by using the mongoose singleton .plugin method as documented here:
http://mongoosejs.com/docs/plugins.html
let mongoose = require('mongoose');
let beautifyUnique = require('mongoose-beautiful-unique-validation');

mongoose.plugin(beutifyUnique);

Cool idea! This would reduce boilerplate on some projects.
Currently, the module reads information about the schema it's plugged onto as soon as it is applied, so it cannot be plugged in before declaring the indexes―as stated in the docs. This is the underlying issue to fix if we want to use it as a global plugin.
This step is only needed to gather the custom duplicate error messages that the users declare on their schemata. Another solution would be to move custom error messages to the plugin's options; however, this would mean a departure from the standard validators' behavior (eg. the required validator), and is not applicable to the global plugin.
If we want to keep the error messages customizable inside the indexes declaration, we should either:

find some way to hook into mongoose whenever an index is added;
hook up just before schemata are compiled to models.

So, I can't reproduce your issue after further investigating. If I plug in the module globally on the tests, it works. Could you provide more information about your use case?

So I'm adding mongoose.plugin(beautifyUnique) in the app.js file. I've tried adding the plugin to the mongoose singleton before and after instantiating my models.
I'll setup a repo shortly and send you a link.

https://github.com/mickyginger/wdi-ldn-shoes-api
There's a seeds file if you want to run it node db/seeds... Works fine if you add the plugin directly to the userSchema, but not to mongoose.

I tried running node db/seeds but I get the following error:
ReferenceError: User is not defined
    at /home/matteo/Développement/wdi-ldn-shoes-api/db/seeds.js:39:3

Oops... updated. Give it a try now...

If I run node db/seeds, the error is indeed not beautified. But this is because in the db/seeds.js you do not apply the plugin to mongoose. Is it intended?

Hmm, ok, odd... I couldn't replicate that at first, but then after switching to node v6.2.1 and then back to node v4.4.5, everything is working as expected.
I think perhaps Sublime was caching something??
Anyway, you can close this ticket. Thanks for your assistance.

Glad to help! :tada:


Support ensureVisible for 2D viewport

Fixes https://github.com/flutter/flutter/issues/126299
This adds support for the static viewport method ensureVisible to work in the 2D viewport. This means default focus traversal will work as well out of the box! :)
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Ok, this is ready for review. I know there is a breakage in flutter/cocoon from the change in function signature. I am looking to see what options there are to land this with that.

Looking to resolve the customer failure in https://github.com/flutter/cocoon/pull/2838

Customer patch landed, updating customer tests in https://github.com/flutter/tests/pull/271

This means default focus traversal will work as well out of the box! :)

I forget, do these use the same APIs as implicit a11y scrolling? Or did we wire that up differently?

I am going to mark this as a draft while I make some changes. I thought I would add showInViewport and showOnScreen to the 2DViewport as separate changes, but I think they are more tightly coupled than I originally thought and should probably be included here.

Note to self when I get back to this starting net week: https://github.com/flutter/flutter/pull/131641/files#r1280947388

Closing this for now to take out of the review queue, I still need to rework a few things.


Site is down again :(

can confirm... experiencing similar issue.

Sorry guys! It is fixed, thank you for reporting


Link to slack chat broken (404)

The readme and the issue template point to a broken link:
http://angularclass.com/member-join/
A minor problem but I figure you'd like to track it.

@katallaxie
The link is still not working. Could You fix it, or reopen the bug?

How can I join the slack??? The link is not working.


[core-tracing] Enums in core-tracing's API surface break some usage patterns with inheritance

Currently core-tracing has an exported options structure OperationTracingOptions. This structure is defined like this:
interface OperationTracingOptions {
  tracingOptions: {
     // NOTE: expanded here for illustration
     spanOptions: {
        spanKind: SpanKind   // this is a typescript enum
     }
  }
}

Due to a bug in one of our scripts we actually ended up in a situation where we were referencing OperationTracingOptions from 1.0.0-preview.11 and 1.0.0-preview.12. These two versions are actually identical (.12 was just the typical automated version bump but we haven't made any changes to the library itself).
For the most part enums are considered type compatible but we found this situation in our code base where that does not hold:
interface CommonOptions{
  tracingOptions?: OperationTracingOptions;   // (these came from preview.11)
}

interface ServiceSubmitBatchOptionalParams{
  tracingOptions?: OperationTracingOptions;   // (these came from preview.12)
}

interface BlobBatchSubmitBatchOptionalParams extends ServiceSubmitBatchOptionalParams, CommonOptions{
}

This yields a compile error about the tracingOptions field being incompatible:
src/BlobBatchClient.ts(30,18): error TS2320: Interface 'BlobBatchSubmitBatchOptionalParams' cannot simultaneously extend types 'ServiceSubmitBatchOptionalParams' and 'CommonOptions'.
  Named property 'tracingOptions' of types 'ServiceSubmitBatchOptionalParams' and 'CommonOptions' are not identical.

This pattern might not be common in consumers of our packages, but it seems like a sensible thing that you'd expect to work. The enums in this case are coming in because we want to be compatible with opentelemetry/api's exported surface and they use enums.
This issue is primarily just documenting the case for future reference.

We encountered this issue during min-max testing for the azure communication services  sdks, and put in a workaround:
`
//SmsClientOptions is coming from the sms package, which has an internal dependency
// on core-http 1.2.x (where x > 0)
export function createSmsClient(): SmsClient {
return new SmsClient(env.COMMUNICATION_LIVETEST_STATIC_CONNECTION_STRING, {
httpClient: createTestHttpClient()
} as SmsClientOptions);
}
//HttpClient and WebResourceLike are being imported from core-http 1.2.0
function createTestHttpClient(): HttpClient {
const customHttpClient = new DefaultHttpClient();
customHttpClient.sendRequest = async function(
httpRequest: WebResourceLike
): Promise {
//stuff
};
return customHttpClient;
}
`
This is because it was SmsClientOptions was expecting a 1.2.x version of HttpClient, which didn't match with this 1.2.0 HttpClient.
HttpClient was invalid due to its sendRequest  method being incorrect. WebResourceLike has a SpanOptions field much like OperationTracingOptions does.
The type casting tricks the TypeScript compiler to get around this

Here's the TypeScript issue explaining this behavior:
https://github.com/microsoft/TypeScript/issues/1748

These enums are declared separately and are thus distinct in the type system. We don't use structural typing to determine enum compatibility.


Seed Parser Missing Path: function call with arguments

Description
Something like this throws an annoying long warning:
seeds = [
   pda.field.checked_add(1).unwrap().to_le_bytes().as_ref()
]

Since the parser takes everything before the () in unwrap (see this line), therefore generating a long path which can't be found as a name for nothing, just breaking everything.
Current workaround
The current solution for developers is to implement a trait / method for the type in order to have a call like:
seeds = [
   pda.field.checked_add_one().to_le_bytes().as_ref()
]

Basically, have the first method call use no arguments.
Proposed change
Add the missing seed path to pda.rs.

Interesting, so you can't have functions which take arguments on top of the pda seed variables. It kind of makes sense because you don't really want to be doing all sorts of weird stuff with your seeds tbh, they should be pretty clear as to what they are and how you obtain them. If you wanted to make a pr to make the seed parsing more robust (so long as it doesn't break anything or introduce vulnerabilities) it's something I could possibly review.


Test with hopefully the final artifacts

Will merge this one, for the release checklist

Needs to be 11.0.2?

yeah I thought this would need to be the hotfix

Fair, I think this needs to be at least 1.11.1 with the change to the github id endpoint. But the following stuff is just messing with pgp


Tick x-axis spacing with different entries count

I have a series of 100 entries, and the x-axis would correspond to one day, which should be divided into 6 labels. If I have 30 entries, the labels tick spacing is bigger than at 100. Is there a way to keep the same spacing size? I want my graphs not to be scrollable.

Also, is there a way to make the x axis longer than the chart itself? For example, I would have entries, to fit within a day. But the day is not over yet, so I would have entries only until mid day. The x axis would display a step of 2 hours and the line chart would stop at mid day. Could we achieve this?

Hello!

To ensure that we’re on the same page, could you provide an example of this change in label spacing? As such, the entry count shouldn’t influence the label spacing.
Yes, you can achieve this using AxisValuesOverrider—please see here.


Only support on linux?

When I run the cell " Display the multiview images, alignments, masks, and depth of the multiview scene on the fly." The error: 'FileNotFoundError: Could not find module 'libEGL.so.1' (or one of its dependencies). Try using the full path with constr......' occurs, The online solutions all refer to install [sudo apt-get install libgl1-mesa-glx], but I use Windows 10 system. Which platform did you run the project?

Hi, thank you for the interest. We indeed developed our code on Linux, so we would not expect it to work on Windows out of the box.


add redis opentracing

Provide redis opentracing  SDK,  this can help developer see the redis call list and cost, code contains the following sections:

Implements go-redis hook
unit test coverage 100%

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: why444216978:x: weihaoyuweihaoyu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[Feature Request] Disable auto permission organizing.

I don't know what it's called, so I'll just refer to it as "auto permissions organizing".
The request adds a config option to leave comments in flatfile groups without the plugin removing them.
Example:

It helps when organizing permissions for each plugin when directly editing the files. Keeps the permission nodes for a group nice and tidy.
Currently, the plugin just alphabetize the perms:

It's nice, but I don't want my comments to be removed stating which plugin these permissions are coming from.

It's not recommendet to set permissions directly through files, since there are better alternatives like the web editor.
In addition do file storages artificially increase storage space used.

The files only get reorganized when you change the permissions of what the file referrs to. So if you don't want it overriden, don't change the data through any means but the files.
Though I'd highly urge you not to do that. I mean to what plugin a permission belongs should be clear from the get go, as they start with the plugin name.

Yes, I understand that commenting artificially increases the file of the file.
Regards to the permissions starting with the name of the plugin, yeah I get it. It sounds like a personal problem. But, it does make it looks nicer :D

I talk about the files themself and not if the comments increase storage used.
Compared to H2 or MySQL are file-based storages much heavier in terms of storage used.
I don't think this will ever be added, as you're so far the only one suggesting it and we also recommend using alternatives, rather than editing the files directly.

The main issue with this request boils down to:

The currently used YAML library doesn’t allow for persistent comments. As it’s the library shipped by Bukkit, it would require shipping a custom one, which comes within many problems.
It’s a personal problem, as you mentioned. And due to the previous point the effort required to implement this minute feature is several hours. Which is just not proportional to the benefit that feature would provide.

Understood. Thanks!


star tropics

Star Tropics" mapper 68 not supported.
hope this will be supported in the future. plz and ty

You must have a bad ROM or the header was read incorrectly, this mapper is only used for After Burner.

ok got a working rom
after starting a new game it crashes with the error below. So I enabled
allow illegal NOPs, But still got the same error.
unsupported opcode
the CPU attempted to execute opcode 0x02 at NES address 04:CC84
A:   0402    P:          81       DB:    A2
X:   0005    SP:    10E3       DP: 0000
Y:   CC84   PC:809FAE
Build Date:   2022-5-23

I got a bit further by trying different settings but I don't think this game is fully playable yet.
Disable "native return address", disable "NMI mode, at vblank".

So I have it working somewhat now that it doesn't crash on any input.
but when interacting with the villagers the colors flash between
normal and  negative palettes. Game crashes at first save point
before you enter the first dungeon.
Current settings that are enabled:
Zero memory upon reset: checked
Static Range: e000: checked
Truncate ROM: checked
SRAM size: 8
Recompile PRG RAM: checked
Dynamic JSR: Checked
Allow illegal NOPs: Checked
Auto play threshold: 200
Stack emulation, Lazy double return: checked
Stack emulation, Stack underflow detection: checked
Reset stack interrupt on reset: 0xff, 0x00
NMI mode, Auto detect: Checked
Sprite zero hit offset: 8
Improved PPU status loop detection: checked
Mapper hit IRQ offset: 1
VRAM queue buffer size: 0x100
Startropics (USA).txt

Startropics.nes000.spc.zip
Startropics.nes.srm.zip
0

I just tried disabling "Stack page 01" and it seems to fix the crashes.
The flashing colors are caused by the game attempting to change palette in the middle of a frame which Project Nested doesn't support yet.

I just tried disabling "Stack page 01" and it seems to fix the crashes.
The flashing colors are caused by the game attempting to change palette in the middle of a frame which Project Nested doesn't support yet.

I have that disabled already


Roll Flutter (stable) from 7048ed95a5ad to 32fb2f948e9b (7 revisions)

https://github.com/flutter/flutter/compare/7048ed95a5ad...32fb2f948e9b
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-stable-plugins
Please CC rmistry@google.com,stuartmorgan<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter (stable): https://github.com/flutter/flutter/issues/new/choose
To file a bug in Plugins: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Propose to add surveillance:type=camera

Description
Hi,
I would like to propose a "user suggestion". If an element is tagged with camera:angle, camera:direction or camera:mount but has no surveillance:type, the editor should propose the user to add surveillance:type=camera.
Overpass query: https://overpass-turbo.eu/s/1upC

Oh, I just gave the idea for the check, I'm not a code programmer unfortunately so I can't review it :(

Oh, I just gave the idea for the check, I'm not a code programmer unfortunately so I can't review it :(

Ok sir np


Allow custom port # through npm config

First off, great work so far on nuxt.js
For this issue, some documentation here:
https://docs.npmjs.com/files/package.json#config
Ideally, the package.json would then look like this:
{
  "scripts": {
    "start": "nuxt"
  },
  "config": {
    "nuxt": {
      "port": 4000
    }
  }
}

This is critical for folks who want to host multiple nuxt.js apps/sites on one server.

Hi @jaredreich
Actually you can already configure the port via the PORT environment variable.
{
  "scripts": {
    "start": "PORT=4000 nuxt"
  }
}

I already added the option to add the port in the package.json as you suggested, it will be in the next release.

Sweet, great work!

You can now use the version 0.6.5 to set the port with the package.json

Cool, perhaps this could make it into the docs for others to see?
Cheers.

It's planned :)

Awesome. Again great work so far. 👍


JWT-based routing routing feature

This feature implements the feature described here (Internal ticket).
Basically, this feature introduces simple connection redirection (without traffic inspection) based on the JWT token contents. Destination host is provided to the Jet using dst_hst claim. JWT token itself can be injected into the Preconnection PDU blob using "pcb" option for MSTSC/FreeRDP.
When launching Jet proxy, --provisioner-public-key-file should be specified to provide PEM file with the public key for JWT token validation. If it is not specified - JWT token validation is skipped.
Also, it was required to bump The IronRDP version, so a small number of changes were introduced to fix compilation issues.
Use JWT token for RDP connection in MSTSC

Open MSTSC
Enter a JET address in the "computer" field
Press the "Save As..." button under the "Connection settings" panel to save ".RDP" file to you PC
Open saved ".RDP" file with the text editor
Append string "pcb:s:"  to the end of the file (e.g: pcb:s:eyJhbGciOiJSUzI1NiIsInR5cCI6IkpXVCJ9.eyJleHAiOj....)
Save file
In MSTSC press "Open..." and select your edited file
Done. You can start the connection

Use JWT token for RDP connection in FreeRdp
For FreeRDP token can be provided using "/pcb" CLI parameter
JWT token generation for testing
To simplify the testing process of the feature, a small JWT token generation utility was created "jwt-routing-gen"
Utility usage example:
jwt-routing-gen.exe --destination=<IP_ADDRESS> --signing-key=D:\keys\private.pem --validity-period=3m`

This utility will print the generated JWT token to stdout.
Signing key pair for testing can be generated using https://travistidwell.com/jsencrypt/demo
jwt-routing-gen sources and Win64 binary are attached to the current PR
jwt-routing-gen-src.zip
jwt-routing-gen-win64.zip

This ticket requires https://github.com/Devolutions/IronRDP/pull/29 to be merged to correctly handle Preconnection PDU parsing.
Also, it would be great to merge https://github.com/Devolutions/devolutions-jet/pull/91.
After merging PR's above, IronRDP's version change in Cargo.toml is needed, and cargo clippy/fix/fmt should be run again.

@pacmancoder @bcortier-devolutions and I are having trouble figuring out how we should be launching the devolutions-jet relay with a proper RDP listener for TCP-based relaying. Do you have such a command-line sample, or more information how to properly set it up? I'm not even sure which listener should actually be used, and if the RDP TCP-based relayed has been mixed alongside the previous RDP TLS relaying code. We need an identities.json file for the RDP TLS relaying, but we don't need one for the TCP-based relaying. I'll let you know if we figure it out today, otherwise we'll wait for further instructions from you.

@awakecoding The Jet relay start command is the same as before, except we need add an aditional --provisioner-public-key-file option to specify path to public pem key file for JWT signature validation:
devolutions-jet --jet-instance <IP_ADDRESS> --routing-url rdp://anything.com --listener tcp://<IP_ADDRESS>:3389 --protocol "rdp"  --identities-file D:/identities.json --provisioner-public-key-file D:/keys/public.pem

--jet-instance can be anything, --routing-url is just required to start with "rdp://" and does not make any effect for now. Identities file is still mandatory because it is used during "normal" Jet operation, even it is not used for the actual JWT routing. So, you can just place some stub data inside identities.json file.


ol.layer.Raster rendering issue with ol >= 4.4.0

The ol.layer.Raster does not display at start anymore. It renders when zooming.
Can be seen on color manipulation example. At start, no layer is displayed on map but when zooming in, the layer renders.

git bisect shows that 16e6d137004d8797463d25b6c7a0ecd6c440bccd is the first bad commit.


[FEATURE] Firebase test lab

Feature request
Firebase base automated test lab
I have been looking into the firebase testlab with its automated testing. Because cordova/ionic runs in the webview no events are passed to the native thread for testing. Could the plugin be extended to do this just for the testing purposes where you can pass events to the native thread.

This isn't something I'd spend time on adding but I'd be happy to accept a PR that implements it.

Unfortunately firebase test lab does not work for webview. Does not work
for cordova.
On Tue, Apr 21, 2020, 12:29 Ross Rawlins<EMAIL_ADDRESS>wrote:

Do you see value in it? I mean I know there are other automated testers.
This just seems like it was a nice easy way to implement it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dpa99c/cordova-plugin-firebasex/issues/375#issuecomment-617252726,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABH374HDSY26U74I2AOY2TRNW3WXANCNFSM4MNLEOZQ
.

Unfortunately firebase test lab does not work for webview. Does not work for cordova.

Yes that was and still is the case.


Move disconnected_time to websocket

moved disconnected time to websocket, because closed_callback isnt always executed when the websocket disconnects / fails. The websocket attempts to reconnect independently without executing the closed callback in certain situations

Lifetime of  the websocket instance was not suitable for this approach, because a new websocket object can be instantiated on reconnects. This PR is replaced by: https://github.com/EVerest/libocpp/pull/238


(feat): Restore _apply_index in _table_to_df

Feature or Bugfix

Feature

Impact on performance must be checked before merging this PR
Detail

Restore _apply_index in _table_to_df

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 9fd1aeca375fb7fb27b9257747a141d93af277e6
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubLoadTests5656BB24-s6u9F3qN9oFy
Commit ID: 9fd1aeca375fb7fb27b9257747a141d93af277e6
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubLoadTests5656BB24-s6u9F3qN9oFy
Commit ID: 9fd1aeca375fb7fb27b9257747a141d93af277e6
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

Setting to draft for now as load tests are currently failing


Use a layout for budget page

More info on layouts here.
More info on slots here.

A __layout.svelte is not relevant since every UI section on budgets page is interconnected with data.
Maybe create a BudgetLayout.svelte component, but i'm not even sure this is relevant.


PointTransformerV3 Config for S3DIS dataset

When will the S3DIS config for PointTransformerV3 be released?

Hi, I am still on holiday and I will try my best to release all left configs before Feb 2024. But I have already noted your request, I will release the config and exp record for S3DIS immediately when I am back (Within one week).

I am also looking forward to the S3DIS config for PTv3.

Hay, it is already available here (https://huggingface.co/Pointcept/PointTransformerV3), I will try my best to update Pointcept today. (A bit lazy during the weekend :) )

Thank you so much for quick reply : )  👍


koordlet: fix BlkIOReconcile plugin not using WaitForCacheSync method correctly to synchronize PVC resources

Ⅰ. Describe what this PR does
Add the statesInformer.HasSynced method to wait for the PVC resource synchronization to complete.
If the WaitForCacheSync method is not passed a second parameter, the return value is always true
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

@Wangmin362 Thanks for your nice catch! Please check the failure reason of the DCO check before the PR is merging.

/lgtm
Please check the DCO information and sign off your commit to make it mergable


Cannot join a live share when the active VS Code window is a Remote session

Describe what happened:
What was your system configuration?
Product and Version [VS/VSCode]: VSCode
OS Version[macOS/Windows]: Windows_NT x64 10.0.22000
Live Share Extension Version: v1.0.5418
Target Platform or Language [e.g. Node.js]: Rust
Steps to Reproduce / Scenario:

User 1 connects to some existing repository using remote-ssh
User 2 sets up a live share for some local rust repo
User 1 clicks the share link
User 1 selects 'open in desktop vscode' from the website
User 1's vscode opens a blank session without vscode

Please attach logs to this issue:
Sorry didn't capture any

Apologies for the delay here. As is, this issue doesn’t have enough information to be reproduced and we've made improvements in this area. Please reopen if the issue persists with any additional details. Thank you.


Argument of type 'string' is not assignable to parameter of type 'EntityConstructor '.

the example in README fails to pass TypeScript check.

Sounds like you have installed Redis OM 0.3.x. The current README is for the beta version of 0.4.1 which has breaking changes from 0.3.x. This is noted near the top of the README in the Getting Started section.

Sounds like you have installed Redis OM 0.3.x. The current README is for the beta version of 0.4.1 which has breaking changes from 0.3.x. This is noted near the top of the README in the Getting Started section.

thanks!


Error building context tar does not halt build

Our build server builds projects out of source control and dumps the output on a network share.  We then run docker build pointing to the network share for the build context.
In one particularly case we're getting the following error:
PS C:\> docker build \\server\deployment\Builds\Main\22 --pull --tag dkr.reg/main/app:22

ERRO[0000] Can't add file \\?\UNC\server\Deployment\Builds\Main\22\bin\Consul.dll to tar: readlink \\?\UNC\server\Deployment\Builds\Main\22\bin\Consul.dll: The system cannot find the file specified.
Sending build context to Docker daemon  380.4kB
Step 1/10 : FROM dkr.reg/tools/service-base:TC_89
TC_89: Pulling from tools/service-base
Digest: sha256:d90105095d20a7a71197a4dc27343ca651ba4ba14549778733887deee531b62c
Status: Image is up to date for dkr.reg/tools/service-base:TC_89
 ---> d6bf8daf6e79

I suspect there's something wrong with the files themselves since if I copy the "22" folder and run the build again on the copy then it works fine.
However, the error itself, while odd, is not my main concern.  Why is Docker continuing the build when it has errored sending the context to the docker engine?  I would expect the process to terminate with an error at that point.   It makes it hard to know if the build has succeeded or not when viewed from Teamcity.
Is there a way to make it behave in this way?
Docker Version:

Client:
Version:           18.06.1-ce
API version:       1.38
Go version:        go1.10.3
Git commit:        e68fc7a
Built:             Tue Aug 21 17:21:34 2018
OS/Arch:           windows/amd64
Experimental:      false
Server:
Engine:
Version:          18.06.1-ce
API version:      1.38 (minimum version 1.24)
Go version:       go1.10.3
Git commit:       e68fc7a
Built:            Tue Aug 21 17:36:40 2018
OS/Arch:          windows/amd64
Experimental:     false

Hi @Richard-Payne
This repo here is solely for the registry (and related distribution libraries).
Issues with the builder should be reported to github.com/moby/moby to receive proper attention (specifically for windows).
Hope that helps.


Add release to CI folder

Launch Checklist
This is to check that everything is working as expected in terms of CI, including github pages.

[x] Confirm your changes do not include backports from Mapbox projects (unless with compliant license) - if you are not sure about this, please ask!
[x] Briefly describe the changes in this PR.
[x] Link to related issues.

OK, merge to main, let's see what happens.


support for library projects

As far as I can tell this step assumes the typical apk pair or triple that you would see in an application project [main, test] or [main, test, orch]. In and module that uses com.android.library it is possible to build a single test apk which also includes the contents of the AAR library for testing.
Either I am being too lazy about reading the docs or there is not way to do this without some questionable workarounds happening.

Hi @trevjonez !
Thanks for the request.
I have made a ticket for the Tooling Team. They will check it.

I'm wondering if this is still in consideration or if there's a simple workaround. I see this issue thread relating to the Android Build for UI Testing step.

This issue is still unresolved and should not be marked as stale automatically.

I don't remember if I read this somewhere but I gave it a try and it has been working for us for about 3-4 months running connectedDebugAndroidTest on a library module. You just have to add a step to actually build the library module test apk. For our api library module:
  - gradle-runner:
        inputs:
        - mapping_file_include_filter: ''
        - apk_file_include_filter: ''
        - gradle_task: api:assembleDebugAndroidTest
        title: Build api module instrumentation test apk
    - virtual-device-testing-for-android:
        inputs:
        - test_devices: "$INSTRUMENTATION_DEVICE_LIST"
        - test_type: instrumentation
        title: Run api module instrumentation test on virtual devices

Just know that if you are going to run this for a library module and an app module in the same workflow you may experience the problem I faced in #24

You also have to link a random apk next to the test apk generated or else virtual-device-testing-for-android fails, e.g. this small apk or one you generated from a sample app of yours:
Add this step between assembleDebugAndroidTest and virtual-device-testing-for-android:
   - script@1:
        inputs:
        - content: |-
            #!/usr/bin/env bash
            set -e
            set -x
            wget https://some_place_you_uploaded_the_apk_to.apk -O $BITRISE_DEPLOY_DIR/dummy.apk

            envman add --key BITRISE_APK_PATH --value "$BITRISE_DEPLOY_DIR/dummy.apk"
        title: 'workaround: dummy APK'

I also am having this issue. The step looks for an apk in a library project, I think it should be fixed.

@mateherber do you have any news on this?

@mateherber do you have any news on this? I see that @tothszabi already fixed the problem, but the change was never released ☹


Improve make target scripts

[x] [install] Allow overriding Python binary from env
[ ] [install] Add check whether the project is already installed
[x] [install] Fix Poetry being installed in a non-deterministic way.
[x] [all] Remove unused DIR var
[x] [metrics, unit-test, integration-test] Set default poetry home var

For now, Poetry install is locked on release 1.1.12 and deprecated get-poetry.py script that will be replaced in Poetry 1.2 by install-poetry.py.
After the official Poetry 1.2 release, this can be addressed. Issue: #21


Moved pull_up from super to init, close #505

Some PIR sensors are not pull_down by default but pull_up.
At input_devices.py the pull_up=False is hardcoded at super.
Moved up to __init__ so for that case a user could do:
from gpiozero import MotionSensor

pir = MotionSensor(4, pull_up=True)
while True:
    if pir.motion_detected:
        print("Motion detected!")

( @lurch edited this comment to add extra code-formatting )

I'm fine with adding pull_up as a param for MotionSensor - makes good sense. But the doc string needs some tweaking: it's suggesting connecting the "other side" of the sensor but the typical PIR sensors (e.g. from the CamJam kits) have three pins, Vcc, Gnd, and Out so "other side" is a bit confusing. Also it mentions connecting to 3V3 but the CamJam kits are 5V components (hence why the lead paragraph goes on about connecting to the 5V pin).
This is simple enough though that I'm happy to merge and tweak the docs.


Add per-player stats

Adds the following metrics (can be disabled in settings):

factorio_player_connected[name]: 1 if the player is currently connected
factorio_player_last_online[name]: Last tick the player was connected
factorio_player_time_online[name]: Time (in seconds) the player spent online
factorio_player_position_x[name]: The x coordinate of the players (last) position
factorio_player_position_y[name]: The y coordinate of the players (last) position
factorio_player_position_surface[name, surface]: The surface of the players (last) position

Given that player names are in theory a high cardinality label, these new metrics should maybe be disabled by default, like the train metrics mentioned in #32 (although this case should be way less severe).
It seems a bit weird to me to have a disable by default be set to true (ideally would be enable set to false), but I believe renaming the setting would probably cause a conflict with existing setups? Obviously not a problem for this PR, but for the train one, the style of which should be matched by this one's new setting.

Please dont merge this yet. Now that #37 is merged I can also include documentation for the metrics added in this PR to the Metrics.md file.


Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'Could not find a storyboard named 'Manager' in bundle NSBundle

Nice library and work, but cannot use it. Am I missing something?
Added using CocoaPods.
Thanks,

Can you check the pod version you are using please ?
You can find it in your Podfile.lock.

The version is - Dotzu (0.1)
Please tell me that you need any info

update your Podfile with that, and you are ready to go !!
pod 'Dotzu', '~> 1.5'


add vector keda logos

Adds some .svg and .eps versions of the Keda logo to the /images dir
Signed-off-by: flynnduism<EMAIL_ADDRESS>

Thanks for the PR!
It looks like we've lost the lightning effect on the A, would it be possible to add that as well or not?

I think that was actually a modification after the fact by @sjwaight if memory serves me right, just so it pops more on a white background

I'll merge this for now so that it got merged, but would be good to have a version with the effect as well.

I'll take a look at the committed images and get them fixed if possible.

FYI we have them all here now in CNCF https://github.com/cncf/artwork/blob/master/examples/sandbox.md#keda-logos

Based on the link from @caniszczyk do we need to supply all those image styles with the lightning bolt or just the primary logo in blue above? If you could let me know I can brief in the requirement once and get this knocked over in one go. Apologies for the tardiness.

Preferrably all but if that's too much then it's ok to just have these 3 for me!

Thanks and no worries, there's no rush for this.

Does someone have the circular outline source image somewhere as a hi-res input? I had the KEDA text fixed with a lightning bold shadow, but didn't get the other options done (not sure who did those). If someone can source the circular image that would help.

@sjwaight What we have today can be found here:

https://github.com/kedacore/keda/tree/master/images
https://github.com/cncf/artwork/tree/master/projects/keda
https://github.com/cncf/artwork/blob/master/examples/sandbox.md#keda-logos


TREZOR support broken

There should be a string "Bgold" instead of "BitcoinGold" here: https://github.com/BTCGPU/electrum/blob/4d69492782d1017ab50c8449cef44570191a486b/plugins/trezor/trezor.py#L178

Thx for reporting, but the current master doesn't support HW wallets at all. Support for HW Wallets will be changed in the next version i'm working on.


Add ARIA attributes from ARIA 1.1?

ARIA 1.1 is in Candidate Recommendation (https://www.w3.org/TR/wai-aria-1.1/). Would we be able to add support for the new attributes or is it still too soon?

Given that it's been in that state since October, has there been any movement on it? It'd be ideal to get someone from the W3C to weigh in on how likely it is that this spec will change or be withdrawn prior to full standardization.

if @Joanmarie has bandwidth she may be able to weigh in on that 🤞

The ARIA Working Group is currently focused on testing so that we can transition ARIA 1.1 to PR. Unfortunately, it turns out that writing and verifying testable statements is not the sort of work that results in hordes of eager volunteers. Who knew?? 😉
Jokes aside, we do not anticipate changes to the 1.1 spec, and I cannot fathom a condition which would cause it to be withdrawn. Authors are already using ARIA 1.1, user agents are already implementing it, and assistive technologies are already adding support for it.
HTH!

@Joanmarie Thanks for confirming!
@backwardok sounds like we should support the new attributes ASAP, as long as it can be done in a backwards-compatible way (with devices that only support 1.0)

Oooh do we get this for free with @jessebeach's recent addition of aria-query?

Oooh do we get this for free with @jessebeach's recent addition of aria-query

Yes, 😁. The new properties, states and roles are already baked into aria-query

Oooh do we get this for free with @jessebeach's recent addition of aria-query?

@wyattdanger yes. If I'm missing anything, add it to this JSON in a PR and I'll rebuild the modules with the updated data: https://github.com/A11yance/aria-query/blob/master/scripts/roles.json

@rastersysteme agree. Please propose the PR and we'll get it in.


master driver doesn't release sda before get_slave_ack()

It seems that if the last addr/data bit to sda is 0, the sda_out will keep being 0, which causes the sda in i2c_if is hold by the master driver during ack/nack period.

Do you have a pull up on the bus? The pull up should be bringing the
signals back to a logic one.
dov
On Thu, Mar 14, 2024 at 3:38 AM kjhhgt76 @.***> wrote:

It seems that if the last addr/data bit to sda is 0, the sda_out will keep
being 0, which causes the sda in i2c_if is pulled down to 0 during ack/nack
period.
—
Reply to this email directly, view it on GitHub
https://github.com/dovstamler/uvm_agents/issues/5, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZCJRGULA37FT22KGG5UILYYD5QNAVCNFSM6AAAAABEVGQIP6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGE4DKMRRGEYTSOI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

In get_slave_ack() task inside i2c_master.sv,
you can see that it doesn't release the bus after sigs.drv_cb.scl_in === 1'b0 if the previous sigs.drv_cb.sda_out is 0.
In i2c_if.sv, assign sda    = sda_out ? 1'bz : sda_out; will drive a strong0 to sda when sda_out = 0.
In this case, during the ack/nack period the sda bus is still driven by 0. Even you pulled it up externally.


Images not loading - giving error

At first, I was getting this error:
Error:Execution failed for task ':app:transformNative_libsWithMergeJniLibsForDebug'.

com.android.build.api.transform.TransformException: com.android.builder.packaging.DuplicateFileException: Duplicate files copied in APK lib/armeabi-v7a/libblasV8.so
File1: /home/matthew/dev/java/Retrofit2SampleApp/app/build/intermediates/exploded-aar/jp.wasabeef/picasso-transformations/1.3.1/jni
File2: /home/matthew/dev/java/Retrofit2SampleApp/app/build/intermediates/exploded-aar/jp.wasabeef/glide-transformations/1.3.1/jni
File3: /home/matthew/dev/java/Retrofit2SampleApp/app/build/intermediates/exploded-aar/jp.wasabeef/fresco-processors/1.1.1/jni

Then I put this in my build.gradle
packagingOptions {
exclude 'lib/mips/librsjni.so'
exclude 'lib/x86/librsjni.so'
exclude 'lib/armeabi-v7a/librsjni.so'
exclude 'lib/mips/libblasV8.so'
exclude 'lib/x86/libblasV8.so'
exclude 'lib/armeabi-v7a/libblasV8.so'
exclude 'lib/x86/libRSSupport.so'
exclude 'lib/mips/libRSSupport.so'
exclude 'lib/armeabi-v7a/libRSSupport.so'
}
But then, even though it compiles OK, I still can't view any of the images. It might be because I excluded all the .so files above?
 Looks like the latest build tools won't allow duplicate .so files in the apk anymore. Picasso and Glide were moved to separate product flavors as the transformation library used the same .so files.

Thanks, @DeveloperMCD for uncovering this issue and, @sten711 for fixing it.


When moving, hide original

The item you’re sorted stays in the column as you’re moving it around
so if you have 2 items in a column, it looks like you have 3.

Thank you Nikola!
https://codepen.io/nikolasp/pen/povEBwO


build(webpack.config.js): change bundle target to node

Includes a semantic commit that reflects the changes made to support SSR.

:tada: This PR is included in version 1.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix(security): remove disable-web-security due to issues with Zuora

having problem when use this with Zuora's HMAC protocol
https://www.zuora.com/developer/api-reference/#tag/HMAC-Signatures
I found if I delete that option, everything would still work. Curious why that was added to begin with.

@azuruce thanks for your contribution to our open-source solution. --disable-web-security allows running chrome without CORS policy. It's important when you need to test localhost frontend app without any changes from server-side. However, seems Cypress has chromeWebSecurity option, that's why I think we can remove it.

@derevnjuk Thanks for explaining why it was added.


link to getting data into visit manual off llnl site is bad

Describe the bug
Link off the llnl wci site is bad.
https://wci.llnl.gov/content/assets/docs/simulation/computer-codes/visit/GettingDataIntoVisIt2.0.0.pdf
(this is the site I hit when I google "Getting Data Into VisIt")

FYI...a recent update to WCI pages broke all binary download links for Silo project. Its since been fixed by switching .tar.gz to .tgz extensions but I wonder if .pdf may have suffered a similar fate?

FYI...a recent update to WCI pages broke all binary download links for Silo project. Its since been fixed by switching .tar.gz to .tgz extensions but I wonder if .pdf may have suffered a similar fate?

I sent an e-mail to the wci web team about this.

I sent an e-mail to the wci web team about this.

Just sent an email to WCI web team that helped me with Silo's broken links. Oh...so did @brugger1 just now too 😮

Just sent an email to WCI web team that helped me with Silo's broken links. Oh...so did @brugger1 just now too 😮

The WCI web team put the file on the new web site. All is good again.

The WCI web team put the file on the new web site. All is good again.


[4.0RC2] Advanced Import physics colliders issues

Godot version
4.0.rc2.mono
System information
Windows 10, Forward+, GTX960
Issue description
The advanced import setting generates incorrect collision shapes.

the dimensions of the shapes are different in the preview window than in the editor window when the model is opened
setting the position of the collision shape moves the object in dynamic mode
when area mode is selected, the model is not imported
when reopening the importer window, the collision shape is not displayed despite the checkbox being enabled

Steps to reproduce
The example GLTF file is a wireframe of a blender cube of 2x2x2 meters and scale 1.0 in xyz.
Open the GLTF file in the importer, in Generate turn on Physics, in Type select Body Type: Static and Shape Type: Box. Default dimensions 2,2,2, position 0,0,0, rotation 0,0,0

Collision shape preview is half the size of the actual one in the editor

Change the position of the collision shape to 0,2,0 and reimport as static

StaticBody3D position is now 0,2,0 - isn't it the position of the collision body that should change instead of the whole staticbody?
Now reimport collision shape as dynamic

The organization of the nodes has changed and now the entire mesh is shifted upwards (RigidBody3D position is 0,2,0)
Now reimport with collision shape as area

Only collision shape for Area3D is imported and whole mesh is gone.
Minimal reproduction project
cube.zip

duplicate sorry


Since commit 11ab5a3 of Examples/Chip-Tool,  Compiling Examples/Chip-Tool through Ninja failed

Since this commit https://github.com/project-chip/connectedhomeip/commit/11ab5a3c163b677894b912960d1e9252a17aa6a9, when I ran the following command suggested in this doc, I got the error "fatal error: readline/history.h: No such file or directory" which did not happen before the commit 11ab5a3 of Examples/Chip-Tool "Chip tool interactive(#16135)"
scripts/examples/gn_build_example.sh examples/chip-tool out/
Please check the attached file with all details about the error above
errorLogsAboutNinjaCompileFailed.txt

looks like it happened because the new dependency is introduced libreadline-dev
After I installed it, it works for me.
It would be better to update the Installing prerequisites on Linux in https://github.com/project-chip/connectedhomeip/blob/master/docs/guides/BUILDING.md

This needs fixing in the vscode image: we need chip-build-vscode to be able to compile all builds out of the box, right now it does not:
2022-04-02 13:13:25 INFO    FAILED: obj/commands/interactive/chip-tool-utils.InteractiveCommands.cpp.o
2022-04-02 13:13:25 INFO    clang++ -MMD -MF obj/commands/interactive/chip-tool-utils.InteractiveCommands.cpp.o.d --target=aarch64-linux-gnu -O0 -g2 -fno-common -ffunction-sections -fdata-sections -fno-exceptions -fno-unwind-tables -fno-asynchronous-unwind-tables -fPIC -Wall -Werror -Wextra -Wshadow -Wunreachable-code -Wvla -Wformat -Wformat-nonliteral -Wformat-security -Wimplicit-fallthrough …-tool/third_party/connectedhomeip/third_party/jsoncpp/repo/include -I../../examples/chip-tool/third_party/connectedhomeip/examples/common/tracing -c ../../examples/chip-tool/commands/interactive/InteractiveCommands.cpp -o obj/commands/interactive/chip-tool-utils.InteractiveCommands.cpp.o
2022-04-02 13:13:25 INFO    ../../examples/chip-tool/commands/interactive/InteractiveCommands.cpp:21:10: fatal error: 'readline/history.h' file not found
2022-04-02 13:13:25 INFO    #include <readline/history.h>

It looks like this is related to ARM specifically - our images compile for x64 but not for arm build cross-compiles.
I will for now add a noglob and will see about what needs to be done to add libreadline to the arm sysroot.

created #17084 for the glob bit. The build requirements and arm sysroot will have to be done separately (hoping to get time soon ... but it is somewhat tedious).

@andy31415 I see you mentioned the cross-compile in your comment.
I am working on cross compiling the ChipTool for OpenWrt, I get the OpenWrt Cross Compiling Environment ready, however, I could not find an easy way to modify the BUILD.GN in order to use OpenWrt Cross Compiler as well as all other dependencies like headers
Would you mind providing some hint or instructions about how to cross compile the ChipTool for OpenWrt?

@RudyGOGOGO Have you find a way to cross compile the ChipTool for OpenWrt? Thank you.