App Crashes when frameProcessor prop used🐛

What's happening?
android app crashes then i try to use frameProcessor.it works fine when I comment out the frameprocessor prop.i have pasted the logs please look into it
"react-native": "^0.72.3",
"react-native-vision-camera": "^3.6.8",
"react-native-worklets-core": "^0.2.4"

Reproduceable Code
function App(): JSX.Element {
  const { hasPermission, requestPermission } = useCameraPermission();
  const device = useCameraDevice('front');
  const frameProcessor = useFrameProcessor(() => {
    'worklet'
 /*    const faces = scanFaces(frame)
    console.log(`Faces in Frame: ${faces}`) */
    console.log("Frame Processor");
  }, [])

  useEffect(() => {
    if(!hasPermission)
    {
      requestPermission().then((res) => {
        console.log(res);
      }).catch((e) => {
        console.log(e);
      })
    }
  })

  return (
    hasPermission?
     <View style={{
      flex:1,
     }}>
      <Camera
      device={device}
      isActive={true}
      style={{
        flex:1,
      }}
      frameProcessor={frameProcessor}      //app don't crash when i remove the frameProcessor
      />
     </View>:
     <View style={{
      flex:1,
      justifyContent:"center",
      alignItems:"center"
     }}>
     <Text>
     Permission Denied
     </Text>
     </View>
  );
}

Relevant log output
2023-11-22 15:36:10.095 22446-22577 AndroidRuntime          com.test2                            E  FATAL EXCEPTION: mrousavy/VisionCamera.video
                                                                                Process: com.test2, PID: 22446
                                                                                com.facebook.jni.CppException: Compiling JS failed: 1:1:invalid empty parentheses '( )' Buffer size 3 starts with: 280a29
                                                                                at com.mrousavy.camera.frameprocessor.FrameProcessor.call(Native Method)
                                                                                at com.mrousavy.camera.core.VideoPipeline._init_$lambda$0(VideoPipeline.kt:115)
                                                                                at com.mrousavy.camera.core.VideoPipeline.$r8$lambda$EMriwS_FKhtjTFtppM84Z5V8tiI(Unknown Source:0)
                                                                                at com.mrousavy.camera.core.VideoPipeline$$ExternalSyntheticLambda0.onImageAvailable(Unknown Source:2)
                                                                                at android.media.ImageReader$1.run(ImageReader.java:837)
                                                                                at android.os.Handler.handleCallback(Handler.java:938)
                                                                                at android.os.Handler.dispatchMessage(Handler.java:99)
                                                                                at android.os.Looper.loopOnce(Looper.java:241)
                                                                                at android.os.Looper.loop(Looper.java:342)
                                                                                at android.os.HandlerThread.run(HandlerThread.java:67)
2023-11-22 15:36:10.095 22446-22577 ExceptionHandle         com.test2                            I  at com.mrousavy.camera.frameprocessor.FrameProcessor.call(Native Method)
2023-11-22 15:36:10.096 22446-22577 ExceptionHandle         com.test2                            I  at com.mrousavy.camera.core.VideoPipeline._init_$lambda$0(VideoPipeline.kt:115)
2023-11-22 15:36:10.096 22446-22577 ExceptionHandle         com.test2                            I  at com.mrousavy.camera.core.VideoPipeline.$r8$lambda$EMriwS_FKhtjTFtppM84Z5V8tiI(Unknown Source:0)
2023-11-22 15:36:10.096 22446-22577 ExceptionHandle         com.test2                            I  at com.mrousavy.camera.core.VideoPipeline$$ExternalSyntheticLambda0.onImageAvailable(Unknown Source:2)
2023-11-22 15:36:10.096 22446-22577 ExceptionHandle         com.test2                            I  at android.media.ImageReader$1.run(ImageReader.java:837)
2023-11-22 15:36:10.096 22446-22577 ExceptionHandle         com.test2                            I  at android.os.Handler.handleCallback(Handler.java:938)
2023-11-22 15:36:10.096 22446-22577 ExceptionHandle         com.test2                            I  at android.os.Handler.dispatchMessage(Handler.java:99)
2023-11-22 15:36:10.096 22446-22577 ExceptionHandle         com.test2                            I  at android.os.Looper.loopOnce(Looper.java:241)
2023-11-22 15:36:10.096 22446-22577 ExceptionHandle         com.test2                            I  at android.os.Looper.loop(Looper.java:342)
2023-11-22 15:36:10.097 22446-22577 ExceptionHandle         com.test2                            I  at android.os.HandlerThread.run(HandlerThread.java:67)

Camera Device
{
  "sensorOrientation": "landscape-left",
  "hardwareLevel": "full",
  "maxZoom": 4,
  "minZoom": 1,
  "supportsLowLightBoost": false,
  "neutralZoom": 1,
  "physicalDevices": [
    "wide-angle-camera"
  ],
  "supportsFocus": true,
  "supportsRawCapture": false,
  "isMultiCam": false,
  "name": "FRONT (1)",
  "hasFlash": true,
  "hasTorch": true,
  "position": "front",
  "id": "1"
}

Device
Infinix X669C
VisionCamera Version
3.6.8
Can you reproduce this issue in the VisionCamera Example app?
I didn't try (⚠ your issue might get ignored & closed if you don't try this)
Additional information

[ ] I am using Expo
[X] I have enabled Frame Processors (react-native-worklets-core)
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

same here

same here!

are there alternates i can use for frame processing @agiorgiani @whiletruefranz

@preetamra It depends, you could capture a frame on an interval, save e.g. a png to the temp directory and then process that file, but that seems really bad performance wise compared to the frame processing worklet.
Funny thing is: I have a commit with a working version of my file, logging the rgb value at a certain pixel coordinate via frame processor. As soon as I change anything inside that file, be it deleting an unused import or changing the backgroundColor of a view, the frame processor errors out and I have to roll back. The error I get (without changing anything on the working function) is the following:
Regular javascript function '_f' cannot be shared. Try decorating the function with the 'worklet' keyword to allow the javascript function to be used as a worklet.
@mrousavy I am out of ideas with this, could you chime in? The function used is decorated with the worklet keyword and as I said it works as long as I don't change anything on the component file.

Is the babel plugin properly configured?

maybe check your versions of RN, RN-VisionCamera, RN-Worklets-Core and RN-Reanimated to make sure theyre the same as here in the example app

GUYS, I HAVE GOOD NEWS!!!!!
This version config is working. Just copy and paste.
I will waiting for feedback!!
I hope it can be help to you.
//package.json
//package.json
{
  "name": "AwesomeFrameFace",
  "version": "0.0.1",
  "private": true,
  "scripts": {
    "android": "react-native run-android",
    "ios": "react-native run-ios",
    "lint": "eslint .",
    "start": "react-native start",
    "test": "jest"
  },
  "dependencies": {
    "react": "18.2.0",
    "react-native": "0.72.3",
    "react-native-reanimated": "^3.5.4",
    "react-native-vision-camera": "3.6.9",
    "react-native-worklets-core": "0.2.4",
    "vision-camera-face-detector-plugin": "github:lenhathieu96/vision-camera-face-detector"
  },
  "devDependencies": {
    "@babel/core": "^7.20.0",
    "@babel/preset-env": "^7.20.0",
    "@babel/runtime": "^7.20.0",
    "@react-native/eslint-config": "^0.72.2",
    "@react-native/metro-config": "^0.72.11",
    "@tsconfig/react-native": "^3.0.0",
    "@types/react": "^18.0.24",
    "@types/react-test-renderer": "^18.0.0",
    "babel-jest": "^29.2.1",
    "eslint": "^8.19.0",
    "jest": "^29.2.1",
    "metro-react-native-babel-preset": "0.76.8",
    "prettier": "^2.4.1",
    "react-test-renderer": "18.2.0",
    "typescript": "4.8.4"
  },
  "engines": {
    "node": ">=16"
  }
}

//babel.config.js
//babel.config.js
module.exports = {
  presets: ['module:metro-react-native-babel-preset'],
  plugins: [
    ['react-native-reanimated/plugin'],
    ['react-native-worklets-core/plugin'],
  ],
};

//app.tsx
//app.tsx
/* eslint-disable react-hooks/exhaustive-deps */
import React, { useCallback, useEffect, useState } from 'react';
import { StyleSheet, Text, SafeAreaView, View, Image } from 'react-native';
import {
  Camera,
  VisionCameraProxy,
  useCameraDevice,
  useCameraPermission,
  useFrameProcessor,
} from 'react-native-vision-camera';
import { Worklets } from 'react-native-worklets-core';

import {
  FaceDetectionStatus,
  FaceDetectorResponse,
  detectFace,
} from 'vision-camera-face-detector-plugin';

const CAMERA_SIZE = 900;

export default function App() {
  const [errMessage, setErrMessage] = useState<string>('');
  const [base64Frame, setBase64Frame] = useState<string>('');
  const [submitting, setSubmitting] = useState<boolean>(false);
  const [status, setStatus] = useState<FaceDetectionStatus>('success');

  const device = useCameraDevice('front');
  const { hasPermission, requestPermission } = useCameraPermission();
  const id = (Date.now() * 1000);
  const submitSever = (frameData: string) => {
    setSubmitting(true);
    setErrMessage('');
    setBase64Frame(frameData);
    return new Promise((res) => {
      setTimeout(() => {
        setSubmitting(false);
        setBase64Frame('');
        res('data ne');
      }, 4000);
    });
  };

  const onGetFaceDetectorResponse = Worklets.createRunInJsFn(
    async (res: FaceDetectorResponse) => {
      setStatus(res.status);
      switch (res.status) {
        case 'error': {
          if (res.error) {
            setErrMessage(res.error.message);
          }
          break;
        }
        case 'success': {
          if (res.frameData) {

            console.log(res.faces)
            submitSever(res.frameData);
          }
          break;
        }
        default:
          break;
      }
    }
  );

  const frameProcessor = useFrameProcessor(
    (frame) => {
      'worklet';
      if (submitting) {
        return;
      }

      const response = detectFace(frame);
      onGetFaceDetectorResponse(response);
    },
    [submitting]
  );

  useEffect(() => {
    if (!hasPermission) {
      requestPermission();
    }
  }, []);

  const renderCamera = useCallback(() => {
    if (device == null || !hasPermission) {
      return <Text>No camera device</Text>;
    }

    return (
      <Camera
        device={device}
        isActive
        style={styles.camera}
        frameProcessor={frameProcessor}
        //ML Kit use YUV format
        pixelFormat="yuv"
      />
    );
  }, [device, hasPermission, frameProcessor]);

  const renderFrame = useCallback(() => {
    return (
      <Image
        style={{ width: 100, height: 100 }}
        source={{ uri: `data:image/png;base64,${base64Frame}` }}
      />
    );
  }, [base64Frame]);

  return (
    <SafeAreaView style={styles.container}>
      <View style={styles.cameraContainer}>{renderCamera()}</View>
      {renderFrame()}
      <Text>{status === 'error' ? `Error: ${errMessage}` : status}</Text>
    </SafeAreaView>
  );
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    alignItems: 'center',
    justifyContent: 'center',
  },

  cameraContainer: {
    width: CAMERA_SIZE / 2.15,
    height: CAMERA_SIZE - 100,
    borderRadius: CAMERA_SIZE / 2,
    marginVertical: 24,
  },

  camera: {
    flex: 1,
  },
});

@mrousavy I made sure the versions match the example projects, did not solve a thing unfortunately.
@canibal2 So you used the Worklets directly instead of the decorator? Will try that.

@whiletruefranz actually I tried just in the android side. Right now, I try for IOS. I will give you feed back asap.

i faced this error only when i forget to configure worklets in babel, you can also try to disable hermes and you probably will get a more helpful description of what going on

@whiletruefranz I checked it. As I said it's worked android and ios side. Make sure run
rm -rf node_module
rm -rf package-lock.json
rm -rf yarn.lock
npm install
yarn install
cd ios
bundle install
bundle exec pod install
Make sure you do as in the config above https://github.com/mrousavy/react-native-vision-camera/issues/2199#issuecomment-1824938125. Just copy and paste.
"dependencies": {
     "react": "18.2.0",
     "react-native": "0.72.3",
     "react-native-reanimated": "^3.5.4",
     "react-native-vision-camera": "3.6.9",
     "react-native-worklets-core": "0.2.4",
     "vision-camera-face-detector-plugin": "github:lenhathieu96/vision-camera-face-detector"
   },

and
//babel.config.js
module.exports = {
  presets: ['module:metro-react-native-babel-preset'],
  plugins: [
    ['react-native-reanimated/plugin'],
    ['react-native-worklets-core/plugin'],
  ],
};

parts is important.
github:lenhathieu96/vision-camera-face-detector package provide the face detection plugin. You can test with this package.

I didnt get the same error after updating my packages

Thanks to @canibal1 my project works right now.
"react-native-worklets-core": "^0.2.4",
"react-native-vision-camera": "^3.6.8",
"react-native-reanimated": "^3.5.4",

is the working combination.

@preetamra is it worked for you?

Using JSC will output this error: Frame Processor threw an error: Unexpected token ')'
I am using the following combination:
"react-native": "0.73.0",
"react-native-vision-camera": "^3.6.16",
"react-native-worklets-core": "^0.2.4",
"vision-camera-dynamsoft-document-normalizer": "^1.0.0"

Thanks to @canibal1 my project works right now.
"react-native": "0.72.3",
"react-native-worklets-core": "^0.2.4",
"react-native-vision-camera": "^3.6.8",
"react-native-reanimated": "^3.5.4",

is the working combination.

Thanks to @whiletruefranz and @canibal1 , I was stuck at this issue for past 2 days, and trying your versions, I was able to make frameprocessors work in my app.

yarn start --reset cache solved it for me


Please output spectral configuration validation errors to console

It is currently very difficult to adjust configurations between openapi validator versions, because errors are not shown to console.
Have spectral file like this:
extends: '@ibm-cloud/openapi-ruleset'
formats: ["oas3"]
rules:
  request-body-object: 'off'
  content-entry-provided:
    description: non-204/404 responses should define a content object
    given:
      - "$.paths[*][*].responses[?(@property != '204' && @property != '404')]"
    severity: error
    resolved: true
    then:
      field: content
      function: truthy
  response-example-provided: 'off'
  string-boundary: 'off'
  prohibit-summary-sentence-style: 'off'
  description-mentions-json: 'off'
  schema-description: 'off'
  property-case-convention:
    description: Property names must follow a camel case naming convention
    given: schemas
    severity: error
    resolved: true
    then:
      function: propertyCaseConvention
      functionOptions:
        type: camel

and use open api validator through code ( something like this):
import ibmValidator from 'ibm-openapi-validator';

    ibmValidator(openAPiSchema)
        .then((validationResults) => {
                 for (const err of validationResults.errors) {
                      console.log(err);
                 }
        });

all validation errors are ignored because "debug" parameter is undefined in spectra-validator.js  checkGetRulesetError and then it silently fallsback to ibm open api default config and causes a lot unrelated errors

Always when I update ibm open api validator I need to manually edit this file:
https://github.com/IBM/openapi-validator/blob/main/packages/validator/src/spectral/spectral-validator.js#L87-L119
to get any understanding why its not working. Also it fails to chalk.magenta (color method?)... well I dont need the colors but the output so I always remove it

After manually editing that file it will output following errors for the above yaml configuration... its difficult to fix it without these messages
  Error at #/rules/property-case-convention/then: must have required property 'function'
  Error at #/rules/property-case-convention/given: must be a valid JSON Path expression or a reference to the existing Alias optionally paired with a JSON Path expression subset

Another question, how do I change the property style to camel case or is it broken?
this does not seem to work?
.spectral.yaml file...

Thanks for bringing this up. I'll work on a change to log these errors.  I'll get back to you on that soon

@Havunen I thought that Spectral had a simpler mechanism for providing configuration to an existing rule, but I misread their documentation. To change this rule to use camel case, you need to use a JavaScript-formatted ruleset. Then, you can import the function we use for validating and provide your own options. Here is an example:
const ibmRuleset = require('@ibm-cloud/openapi-ruleset');
const { propertyCaseConvention } = require('@ibm-cloud/openapi-ruleset/src/functions');

module.exports = {
  extends: [ibmRuleset],
  rules: {
    'property-case-convention': {
      given: [
        '$.paths[*][parameters][*].schema',
        '$.paths[*][parameters][*].content[*].schema',
        '$.paths[*][*][parameters][*].schema',
        '$.paths[*][*][parameters,responses][*].content[*].schema',
        '$.paths[*][*][requestBody].content[*].schema'
      ],
      then: {
        function: propertyCaseConvention,
        functionOptions: {
          type: 'camel'
        }
      }
    }
  },
};

Theoretically, you can continue to use a YAML ruleset and define your own function in a folder called functions that imports/exports the propertyCaseConvention function from our NPM package. Then you should be able to use that function in your YAML file. I put together an example of this but Spectral chokes on it for some reason. Seems there is a bug there. So using a JS file is currently your best bet.

@Havunen To your original issue, debug is now an argument in the programmatic API - added with #386

I believe everything in this issue has been addressed so I'm going to close. Please re-open if there is something that wasn't resolved


Add multi-recipient transactions support

There are likely some issues with babel fees, even though I tried to address them. No tests were performed.
This draft needs to be extended  by some form of communication between individual recipient forms (likely just a parent component passing correctly synchronized props). This is needed so that if there are some assets selected for a recipient A, then these assets are not displayed as fully available for recipient B (available balances are decreased appropriately).

Extract recipient form containing
address and assets into a separate
component.

Keep track of a list of recipients
in SendView.

Extend P2P transaction building such
that it takes a list of receivers
instead of just a single one.

Successfully sent 2-recipient transaction.

This PR is no more compatible with the base branch, closing it for now


Problems using external libraries in Pulumi (installed using pip)

Hi,
I installed jsonSchema using pip into my repo. When I run the file from the command line it works fine.
python3 validate_config.py
However when I run
pulumi up
I get the error
ModuleNotFoundError: No module named 'jsonschema'
I can't seem to find any documentation on how to manage external libraries in Pulumi projects. Do I have to run certain commands to add this module to pulumi's path?

Hi @LiamLairdAG, I think https://www.pulumi.com/docs/intro/languages/python/#packages has the information you need. I'd suggest asking in our community slack if you're still having troubles with it after that.


[SMB] Rework the runasppl module

This PR reworks the runasppl module to remove unecessary execute operation and detect runasppl based on registry keys only.

all good


Themes selector page

Add a page to browse all current themes and select them to edit or download.
Still opting to not add external libraries as we only have a few pages to handle and seems heavy-handed to add react-router just for that.
Also still staying with static site and loading json from public assets, we can make a CI action to fetch all the themes from the zed extension repo.
Preview: https://themes.zed-themes.pages.dev/

Asset links not found on preview

@dfntlynotjorn Hey thanks for checking it preview, it is currently a bit borked. I am looking to use Remix with Vite and Cloudflare (based of this repo) to support KV data for hosting and sharing themes. Currently, it's working locally but deployed seems like Cloudflare routes is intercepting assets routes.
I hoped to have this solved in next couple of days


change package name to OceanPythonPlot.jl

handled in commit a8064a18eacda354bb212d9b87d9c7aefc305bab

title updated to Geo rather than Ocean to be more general


Will It Work with only FTMO Dxtrade Accounts or All DXTrade Accounts.

It will work with any, I believe I made an update where you type the url of where your brokerage is (such as dxtrade.ftmo.com), I believe the parameter is called vendor

Hi
I am getting error when i try to connect Dxtrade '
Post "https://dxtrade.com/api/auth/login": dial tcp <IP_ADDRESS>:443: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.
I am using server as "dxtrade.com"
Thank You


'Remove the extra backslash from the Principal username cofiguration value' part is confusing

Remove the extra backslash from the Principal username cofiguration value in the .json file before entering in the LCS UI. For example, contoso\AXServiceUser should be entered as contoso\AXServiceUser in LCS.
This part is a bit confusing as 'wrong' and 'right' values are no different

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

ID: 3682abff-f1e6-336c-80b2-5c40ab3febd2
Version Independent ID: 14250073-bcce-e22c-ef5b-1194fdf75c1f
Content: Redeploy on-premises environments - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/deployment/redeploy-on-prem.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @sarvanisathish
Microsoft Alias: sarvanis

@VadimGoncharenko, thank you for the feedback.
@sericks007, please review.

@VadimGoncharenko, thank you for pointing out the error.  The example should say: "For example, contoso\AXServiceUser should be entered as contoso\AXServiceUser in LCS."
I've corrected this in the documentation. The updated topic will be published later today.


Migrate to Swift

Rewrite the SDK in Swift.

+1
Would really like to see this.

+1
that would be awesome!

Ok, I will start on a Swift implementation.
If anyone care to help out please let me know.

If you need help with features or testing let me know!

+10

ok ok, I have started and have something, lets see if I can get it to a shareable state.

What is the status here? How can I use Piwik in my Swift Project?

You can use this Objective-C project in a Swift project with a bridging header. More info;
https://developer.apple.com/library/ios/documentation/Swift/Conceptual/BuildingCocoaApps/MixandMatch.html

I'd love to see this too.  How far along are you?  Need any help?  I may be able to help with it, if needed.

Hi @perlguy99 @Erulezz @Urkman @funky-monkey - your help would be very welcome! @mattiaslevin is very busy at the moment and we would welcome new maintainers and help for the official Piwik iOS SDK :+1: if you are interested please consider opening pull requests. We could also give you commit access once you have contributed some things  :-) Thank you for your consideration!

FYI:  we the Piwik team are looking for a maintainer for the iOS SDK. if you can help please comment here #91 - we need you!

I'll need to refactor the SDK to Swift, as I don't want to use AFNetworking. Already using Alamofire. Will comment here after prototyping the approach.

Hi @mattab what's up about the Swift portage for piwik ?

@SteamGorilla we are still looking for a volunteer to step up and take over the Piwik iOS SDK project. Would anyone be interested?
It's a unique opportunity to help participate in a successful open source project and make a difference. We posted on twitter recently asking for help:

@mattab Okay, thanks for your answer. I'm not already skilled enough to realize a portage by myself :/

Hi everyone, great news: @brototyp is working on a Swift 3 port of the iOS SDK. Please check it here: #96 and report your feedback to help him on his mission. The future of iOS SDK is here!

After the first feedback by @mattiaslevin I realized that the pull request is by far to huge. So I will try to chunk if off into reviewable parts.
First will be a pull request with most objc files removed, new target setup and other project settings. Following pull requests will be separated parts. For example event queuing, dispatching, persisting settings and such.
I am looking forward to any feedback on my plan or, of cause, the pull requests.

I got the first pull request prepared with just project setup over here: #97

Seines we now have a base branch for the swift3 rewrite I will start a list of all things to write in swift so single parts can be picked and implemented separately. One might be inspired by the ideas over in #96

PiwikConstants: A collection of all constants. API, Userdefaults, parameters, ...
PiwikUserDefaults: A wrapper onto NSUserDefaults that holds all stored values (except events) such as totalNumberOfVisits, firstVisit and such.
UIDevice-Extension: An extension to return the device platform (iPhone1,1), platform name (iPhone 1G), screen size and version of the operating system.
UIApplication-Extension: An extension to return application specific values such as name, version, build version and such
Models for Event, Campaign, Transaction and CustomVariable
Protocol for an EventQueue
Protocol for a Dispatcher
PiwikTrackedViewController
the PiwikTracker itself

I guess separate tickets for each of those is the best idea. What do you think?
@thorstenstark is there any of those you want to tackle?

Yes, I would agree on this. Perhaps it would make sense to split the last two into seperate tickets. I haven't looked into the current implementation of them, but it sound like these two could be a bit more complex...
I would start with some low hanging fruits like the extensions of Device and Application to become acquainted with this project.

The PiwikTrackedViewController is actually very short. It just tracks on viewDidAppear:. For all those points I am very open to splitting them up into smaller chunks. That goes especially for the PiwikTracker but the Dispatcher and the EventQueue can get big.

Hi, thank you for your Pull Request. We dropped ongoing development for version 3 of this SDK. Please consider looking into version 4, which is completely rewritten in Swift.


[Node.js SDK, Azure Bot Service] Change Configuration Messaging endpoint - azure

Bot Info

SDK Platform:  Node.js
Active Channels: Skype
Deployment Environment: Azure Bot Service - portal.azure.com

Issue Description
Cant change the settings endpoint, error received:
{"readyState":4,"responseText":"{"error":{"code":"InvalidBotData","message":"OwnerIds: OwnerIds must contain at least one email address. "}}","responseJSON":{"error":{"code":"InvalidBotData","message":"OwnerIds: OwnerIds must contain at least one email address. "}},"status":400,"statusText":"Bad Request"}
Reproduction Steps

Login to - https://portal.azure.com
Go to the resource - bot
Settings tab
Change Configuration Messaging endpoint
Save

Expected Behavior
Messaging endpoint save successfully.
Actual Results
Nothing happened.

@liavnave I haven't been able to reproduce this issue with one of my own bots.
Have you tried logging out of the portal and then logging back into it? Similarly, have you tried reproducing the issue in an private/incognito browsing session? If it works there you may need to clear your cookies.

@stevengum I tried to log out and log in again, and test the issue from a private/incognito browser.
also tried to clear cookies.
Still can't change the endpoint.

@liavnave Can you provide your Microsoft App ID and the Bot Handle for further investigation?

@stevengum what is your email?

Also running into this problem on Azure.

Closing due to inactivity.

I tried to access dev.botframework.com to add my email in the settings, but it is not showing in that portal. I am Owner of the bot according to azure portal.

@mintxelas Please look for your bot in the Azure portal.  Bots are managed as Azure Resources now, so access is determined based on Azure Subscription Permissions.

I know, but there's nowhere to put the email of the owner in the portal,
yet that's the error the api throws. And in the permissions of the bot I am
already owner.
El mar., 27 nov. 2018 21:12, Eric Dahlvang<EMAIL_ADDRESS>escribió:

@mintxelas https://github.com/mintxelas Please look for your bot in the
Azure portal. Bots are managed as Azure Resources now, so access is
determined based on Azure Subscription Permissions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/4783#issuecomment-442200495,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAzk-1r81c-ZWh2bckT_gXfm04Q_Wp8zks5uzZy5gaJpZM4U1EsK
.

@mintxelas
Please provide the bot handle and ms application id, so we can investigate further.

Thanks, the bot handle is "AstridBot" and the appId
is c77c3dd6-7278-4f4b-92fc-f3642cb7f29c
El mar., 27 nov. 2018 a las 23:41, Eric Dahlvang<EMAIL_ADDRESS>escribió:

@mintxelas https://github.com/mintxelas
Please provide the bot handle and ms application id, so we can investigate
further.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/4783#issuecomment-442261044,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAzk-82-hmBlIuaGhICeeiydh_yT7dKVks5uzc2egaJpZM4U1EsK
.

@mintxelas, can you please confirm that you are the bot

Closing per original author was non responsive.
@mintxelas, I've opened #5190 to handle your issue. Please refer to that issue going forward.


[FEATURE] Thinking to create a Format Price component in react

Description
currently price is being display statically ie. 100  , ......... what if price has  more digits lets say 18000 ..........Format Price component is needed that will format the price in indian system ........i.e. it will automatically add  rupee symbol ....add commas
Screenshots
No response
🥦 Browser
Google Chrome
Checklist ✅

[X] I checked and didn't find similar issue
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (blank for no)

please assign this issue to me under gssoc'23

any update or any other issue i can work on

Hi @Nitish-hack , this looks like a great issue, make sure it works for all the numbers. The styles musn't change.
Please make sure that you have read our documentation on Commiting guidelines and Pull request guidelines, without proper format the PR might be closed.


remove ineffective invariant usages, fixes #2258

motivation: see #2258
I rewrote selected parts of code that used invariant and were resulting in unnecessary evaluations. For the most part, I replaced code that did some json serialization, or concatenating large/ish strings.
Test plan (required)
I ran the code with my app which has a drawer and several stack. Everything works.

random thought: is invariant necessary at all? Any idea why it was there in the first place? It is, like, more/better performant than standard logic?
This to say, what about removing it completely? @skevy @satya164 @spencercarli any idea of why we are using it?

@kelset working with invariant is convenient. instead of saying
if (something) {
 throw new Error('xyz');
}

you have a one-liner. I'm in favor of keeping it.

Ok thanks for the clarification. I needed a more precise POV for reviewing the code ;)


Crash on client when server shut down

What version of gRPC and what language are you using?
grpc : 1.48.1
language : C++
What operating system (Linux, Windows,...) and version?
Windows 10, Version 20H2(OS Build 19042.1889)
What runtime / compiler are you using (e.g. python version or version of gcc)
VS 2017, cl 19.16.27048
What did you do?
Please provide either 1) A unit test for reproducing the bug or 2) Specific steps for us to follow to reproduce the bug. If there’s not enough information to debug the problem, gRPC team may close the issue at their discretion. You’re welcome to re-open the issue once you have a reproduction.

Run the helloworld server.
Run the the below that just added for loop from sample.

int main(int argc, char** argv) {
// Instantiate the client. It requires a channel, out of which the actual RPCs
// are created. This channel models a connection to an endpoint specified by
// the argument "--target=" which is the only expected argument.
// We indicate that the channel isn't authenticated (use of
// InsecureChannelCredentials()).
std::string target_str;
std::string arg_str("--target");
if (argc > 1) {
std::string arg_val = argv[1];
size_t start_pos = arg_val.find(arg_str);
if (start_pos != std::string::npos) {
start_pos += arg_str.size();
if (arg_val[start_pos] == '=') {
target_str = arg_val.substr(start_pos + 1);
}
else {
std::cout << "The only correct argument syntax is --target="
<< std::endl;
return 0;
}
}
else {
std::cout << "The only acceptable argument is --target=" << std::endl;
return 0;
}
}
else {
target_str = "localhost:50051";
}
GreeterClient greeter(
	grpc::CreateChannel(target_str, grpc::InsecureChannelCredentials()));
for (;;)
{
	std::string user("world");
	std::string reply = greeter.SayHello(user);
	std::cout << "Greeter received: " << reply << std::endl;

	Sleep(3000);
}

return 0;

}

Exit server by Ctrl+c in powershell.

Then I got runtime error in client like below.

What did you expect to see?
The application is crashed.
What did you see instead?
The application should not crash.
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
See TROUBLESHOOTING.md for how to diagnose problems better.
Anything else we should know about your project / environment?

Closing in favor of https://github.com/grpc/grpc/issues/29185


Rotation3 no longer has rotate method?

I get: no method named rotatefound for typena::Rotation<f64, na::U3> in the current scope
I'm using code from a former versions documentation. Did this change somehow? I can't find it in the changelog.

Additionally, when using * as in this documentation:
https://docs.rs/nalgebra/0.16.13/nalgebra/geometry/type.Rotation3.html
I get the trait na::constraint::DimEq<na::U1, na::U3> is not implemented for na::constraint::ShapeConstraint

Yes there is no rotate any more since this is covered by operator overloading.
The error you are getting suggests you are multiplying with the vector on the left-hand-side of the operator or multiplying with a row- vector. Is there any code you can show to help understand the operation you are doing ?

I'm just trying to use a Rotate3 on a Vector3 - Both are row matrices I guess? Surely I should be able to rotate a vector without reshaping a matrix?
The relevant lines:
let rotation = Rotation3::new(rotate);
let positions: Arc<Vec<Vector3<f64>>> = Arc::new(
                  mesh.positions
                      .chunks(3)
                      .map(|i| Vector3::new(i[0] as f64, i[1] as f64, i[2] as f64))
                      .map(|i| rotate * i)
                      .collect()
              );

( Full source https://github.com/peterbraden/rays.rust/blob/transform-mesh/src/shapes/mesh.rs )

Oh man, I'm an idiot! Literally spent hours looking in the wrong place. Thanks!


Development

New version: v1.0.2
This version includes QuickFix #1 and various behind the scene code changes that make the plugin more efficient.

Version v1.0.2
This version includes:

QuickFix 1

Config saving now works, and will save custom names even when the server is turned off.

Various behind the scene code changes that make the plugin more efficient.


Include API spec in repository

To make the code generation reproducible and to give the developer control over the API spec version that is used, it should be included in the repository instead of being pulled directly from nethsmdemo.nitrokey.com.

Which repository do you refer to? It is here available and shouldn't be duplicated to other repositories IMO.

I mean this repository.  For development, it is important to be able to generate the API client in a reproducible way.  For example, I am currently working on fixing https://github.com/Nitrokey/nethsm-sdk-rs/issues/20.  That requires changes to the API generator.  But the current main does not yet include the latest changes to the API spec.  So if I now run the code generator, it pulls in changes that are unrelated to the problem that I’m working on.
Also, having a local copy makes it possible to change the API spec for testing and development, or worst case to fix an incompatibility with the upstream API spec.
This is the same that we approach that we use for the Python SDK.  Adapting to changes in the API spec is a manual task anyway.  The API spec can be easily updated from upstream using the Makefile, for example: https://github.com/Nitrokey/nethsm-sdk-py/blob/54727badd888bb134a78f52fd7d217075f6e2a1a/Makefile#L51-L52

I see, that makes sense.


Keep the form in editable state after submit

I have a form that in some cases I need to show in editable state after the user submits it.
My markup is:
 ...
My expectation was that in such case the form would be show in edit state after submission.
What actually happens is, the user submits the form, and the controls are shown in both edit and view state.
This seems to be a bug (or is there a different way to keep the form in edit stated after submission?)

Is this still an issue?

@eugef There has been no follow up, please close.


Linked Landing Pages to HDH Docs

Linked Landing Pages to HDH Docs
Added Harness OSS to Footer

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[x] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[x] Optional Screen Shoot.

and

Preview environment: https://ci.pr.harness.io/pr-106


Add VPack to Comparison of data serialization formats on Wikipedia

It deserves to be mentioned here:
https://en.wikipedia.org/wiki/Comparison_of_data_serialization_formats
@neunhoef: Please validate below template and fill in the gaps (IDL/API/zero-copy)
|-
| [[VelocyPack]]
| [[ArangoDB]]
| {{n/a}}
| {{no}}
| [https://github.com/arangodb/velocypack/blob/master/VelocyPack.md VelocyPack (VPack) Version 1 Specification]
| {{yes}}
| {{no}}
| {{partial}}{{ref|vpack1refs|g}}
| {{?Schema-Interface description language?}}
| {{?Standard APIs?}}
| {{?Supports Zero-copy operations?}}

And the note regarding references, to be added below the table and the other notes:
*g. {{note|vpack1refs}}VelocyPack offers a value type to store pointers to other VPack items. It is allowed if the VPack data resides in memory, but not if stored on disk or sent over a network.

Everything you have put there is OK, as far as I see.
So far, there is no Schema-Interface description language.
So far, there are no standard APIs, only a reference implementation. However, other formats say "yes for C++, etc. " here. We might as well say "C++ API reference implementation".
For zero-copy operations, we simply say yes. Not entirely sure what they mean there.

Done. Also added VPack to the table a little further down (a bit wonky):
https://en.wikipedia.org/wiki/Comparison_of_data_serialization_formats#Comparison_of_binary_formats
I believe small integers are not zero-copy, because they need to be loaded into a buffer in order to subtract 0x29 or 0x39 respectively to get the actual value.
BTW: shouldn't it be number of bytes is ... here?

0x20-0x27 : signed int, little endian, 1 to 8 bytes, number is V - 0x1f, two's complement
0x28-0x2f : uint, little endian, 1 to 8 bytes, number is V - 0x27


Bug/#96/entities lose idx on concurrent api

fix #96

Pass alternative indices to the function and return them back.
Concurrent calls return arbitrarily but the result contains the index because [1].
Re-order and map to remove the indices.

Codecov Report

Merging #97 (e470592) into master (1957671) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #97   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           40        40
  Lines         1440      1443    +3
=========================================
+ Hits          1440      1443    +3

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

dialogy/constants/__init__.py
100.00% <100.00%> (ø)

dialogy/plugins/text/duckling_plugin/__init__.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Updating binaryen, wasm-install and enabling V8 tests on Mac

This PR includes a number of updates that ended up all being tangled together.

build.rs now uses curl to fetch the appropriate wasm-install package.
Since the waterfall at http://wasm-stat.us now includes Mac builds, this PR enables downloading wasm-install on Mac and running the test suite in V8.
This PR updates the wasm-install versions to more recent builds. The new builds use Wasm binary format 0xd instead of 0xc, so this required...
updating Binaryen to the latest master.
Trans now adds an unreachable expression after calls to non-returning functions such as panic. This is needed to make sure the resulting Wasm bytes typecheck.

@dschuff - can you take a look at this too?


Allow empty lists in streamed responses

It's possible that block request responses end up not containing any blocks, so we need a way to encode this.

Sadly this didn't make v0.9.4, but it looks like a good solution. I'm thinking we should open a networking-only branch for things that pop up during the testnets until v0.10 some time end Jan./mid Feb., and ensure consensus-compatibility with v0.9.3, as this also probably is not the last networking feature to improve testnets with. Sounds good?

@arnetheduck Can you run make check_toc twice (first will find issues in table of contents, second time makes the edits) to fix CI?

oh my, twice?

networking-only branch

or just do 0.9.5 - technically, nothing that worked before is broken with this branch - I don't think we're at the stage yet where we need to be overly strict with compat - this is a much smaller break than signed_root for example

dev is fine. We will be releasing v0.10.0 from dev


Implement pagination for invoices, payments, and transactions

Using Zap iOS with my remote node is getting really difficult. Doing something like generating an invoice takes minutes.
My assumption is because how big my remote node is. 500+ channels and thousands of invoices, payments, etc.
I think we should implement some form of pagination, as I assume my issue is generating an invoice or loading a payment is rendering a fetch of all my payments/invoices or something? Instead of loading everything, maybe we only load the last 50 and enable a scroll pagination UX that fetches more as the user wants to or something

This would also solve this issue of users having timeouts when connecting to large remote nodes

pagination is a very hard problem 😅

in the history we are combining data from multiple api endpoints (transactions, invoices, payments, channels & closedChannels). we somehow have to merge individual pages in a consolidated page 🤯, every individual page might have a different date range
not all lnd api endpoints support pagination (https://github.com/lightningnetwork/lnd/pull/565)

does the desktop app have nice solution for this? @mrfelton @korhaliv


Display known issues in game

[ ] In main menu and in pause menu add a "KNOWN ISSUES" button
[ ] Add a webview widget in game for this that links to https://github.com/orgs/HaywireInteractive/projects/4/views/16

The webview plugin uses a very outdated chromium version and the provided github link fails to load so, instead a button to open on user's default browser will be made.


accepts_incomplete within service instance request body

Looking at the spec it appears that accepts_incomplete should always be a query string parameter, yet in the service instance request spec it says that this should be part of the body. This is a little inconsistent but perhaps it's an oversight or tight RFC2616 compliance (even though a body is supplied). Anyone got a comment on this?

This has been fixed in the latest version of the spec:  https://github.com/openservicebrokerapi/servicebroker/blob/master/spec.md

Thanks @duglin


Inline spinner icon should be offered in a smallest size

Summary
Currently the semantic ranges for a spinner are large, medium and small. However UX rules dictate that the iconography sizing matches the normative size of other Clarity icons. Currently that standard appears to be 16px. For some reason the spinner abides by multiples of 18, roughly +12.5% difference across. That means any forms having inlined spinners will have an oddly larger spinner in relation to other static icons.
Question is, should the sizing multiple of a spinner align to Clarity icons? If so how to achieve that, without tampering CSS classes?

Propose some way to influence a smaller semantic for the spinner. Following the Bootstrap convention, that would be .spinner-xs

Consistency for alignment within forms where other static Clarity icons are positioned

Use case
Provide some specific details about how this change would improve your application and use case.
Examples
Attached is a form workflow illustrating the alignment. See the dotted red box in comparison to another Clarity icon - sized with at a standard 16 x 16.

Workarounds
Yes, hardcode width,height in our application's CSS for this use case.

There is a workaround available for this issue, and we recommend using it for Clarity Angular. We would consider a contribution to address this issue if someone is able to work out a solution. As we build Clarity Core implementations, we expect that this issue won’t be an issue or will be configurable on the application side. To help us clean up our backlog, we are going to close this with a functional workaround available and suggest you follow updates for Clarity Core for enhancements that can support your use case with Clarity Core components.

There is a workaround available for this issue, and we recommend using it for Clarity Angular. We would consider a contribution to address this issue if someone is able to work out a solution. As we build Clarity Core implementations, we expect that this issue won’t be an issue or will be configurable on the application side. To help us clean up our backlog, we are going to close this with a functional workaround available and suggest you follow updates for Clarity Core for enhancements that can support your use case with Clarity Core components.


Using it with Next js

Is there a sample project on how to use it with Next js?

There are a number of samples in various frameworks here: https://github.com/duckdb-wasm-examples

I use it with Next.js in this project:
https://github.com/ilyabo/graphnavi

Thanks for that pointer @ilyabo.
I adapted it into a small demo at ryan-williams/next-duckdb-parquet-demo, also deployed to GitHub Pages at runsascoded.com/next-duckdb-parquet-demo. It shows how to read Parquet files using DuckDB, in Node and the browser, in a few ways (load a local file, fetch a remote URL, load from a buffer).
It's great that DuckDB seems to finally provide a robust way of reading Parquet files in JS (and writing, in Node at least). As this SO describes, there were several previous JS libraries that all seem unsupported / bit-rotted.
A downside seems to be that, in the browser, it requires fetching a 3.6MB (compressed; ≈20MB uncompressed?) wasm file. That fetch seems to happen in O(100ms), but in an end to end demo I am seeing my app take O(seconds) longer to fully load:

ctbk.dev is a Next app that loads a 150KB JSON file via getStaticProps
s3.ctbk.dev is a prototype that instead loads that data as a 22KB Parquet file on the server, and sends that to the client, which decodes it
The Parquet version seems ≈1s slower to reach fully-loaded state.

I'm sure there are low-hanging performance optimizations I should make (I see you used SWR in your example; I'm not familiar with it and didn't attempt that yet), but so far it's unclear whether this approach can let me power my static sites with Parquet data instead of JSON. It would be great if I could fetch arbitrary Parquet files from S3 for each static site page; hopefully we can find a robust and performant way of doing that.

Thanks, @ryan-williams. Your project is not public, it seems.
BTW If you just need to load parquet, there's also https://github.com/kylebarron/parquet-wasm

@ilyabo woops, just set it to public, thanks

One option for repeat visits is to use a service worker to cache the files! I believe they are cached after being parsed, so that should help with the start up time as well after visit #1.

Thanks @alex-monahan, that makes sense. The performance hit I'm seeing doesn't seem specific to the first page load, or to fetching the duckdb wasm. Here's one load of my JSON version and 2 loads of my Parquet version (in Chrome 110):
0. (ctbk.dev; 150KB JSON sent from server): "Load: 1.95s"

1. (s3.ctbk.dev; 22KB Parquet buffer sent from server, parsed on client with duckdb-wasm): "Load: 4.05s"

2. (s3.ctbk.dev; reload of the above): "Load: 3.98s"

ctbk.dev is deployed from neighbor-ryan/ctbk.dev@www, while s3.ctbk.dev is deployed from neighbor-ryan/ctbk.dev@pqt, but there are other differences between them so it's not a perfect comparison (e.g. the Parquet version is parsing data on the client, while the JSON version doesn't have to… though I'm not sure whether the network traces above would notice that).
A simpler test might be to have both pages fetch the data they need directly from S3 (using a plain fetch for the 150KB JSON, vs. pointing duckdb at .s3.amazonaws.com/ for the Parquet). Also varying/increasing the fetched data size would be interesting. I will try to put together such an example.
Another clunky thing I'm doing is round-tripping the duckdb query result through JSON.parse(JSON.stringify(result.toArray())):
/**
 * Run a query against the provided DuckDB instance, round-trip through JSON to obtain plain JS objects
 */
export async function runQuery<T>(db: AsyncDuckDB, query: string): Promise<T[]> {
    const conn = await db.connect()
    const result = await conn.query(query)
    const proxies = result.toArray()
    // TODO: is there an easier / cheaper way to get plain JS objects here?
    return JSON.parse(JSON.stringify(proxies)) as T[]
}

This seemed necessary at some point, but may be introducing some unnecessary overhead. Perhaps I should keep my data in an Arrow table so that it doesn't have to be fully materialized? I am populating danfo.js DataFrames with it and performing various downstream operations.


Package the oracle drivers

Package the oracle db drivers into flyway so they do not need to be downloaded and copied in manually by the user.

Related to https://github.com/flyway/flyway/issues/2513

The concern about ending up with multiple drivers is not a problem - just tried, and if you have both odbc8-x.y.z.jar and odbc8-a.b.c.jar (or odbc10-a.b.c.jar for that matter) then the class loader is quite happy, it takes the first one it comes to. In the absence of us trying to be clever, that means taking the jars in alphabetical order.

Done in 6.2.3

Is this for the docker image, the maven plugin or the cli?


Dubious assignment to be a check in hv_cli_and.c

This assignment that will be a check is dubious. It is the same for the two cases. Suggest to fix and also have a test case that catches it, if possible. Does the code in nodetuple_richcompare work as intended? I don't understand it after these years.
hv_cli_and.c, lines 235-236
       vi = (Py_ssize_t)vt->ob_item[i];
       **wi = (Py_ssize_t)vt->ob_item[i];**

Is it supposed to be wi = (long)wt->ob_item[i];? Hmm... I'll check. (Got really busy with other stuffs recently)

How to get a reference to this class:
$ python
Python 3.7.5 (default, Oct 23 2019, 18:04:59)
[GCC 9.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import guppy.heapy.heapyc
>>> hv = guppy.heapy.heapyc.HeapView(guppy.heapy.heapyc.RootState, ())
>>> d = {}
>>> cli = hv.cli_and((), d)
>>> cli.classify(())
()
>>> type(next(iter(d.keys())))
<class 'guppy.heapy.heapyc.NodeTuple'>
>>>

Woah, that is more complicated than I had expected.

Yeah, it's definitely wrong.
$ python
Python 3.7.5 (default, Oct 23 2019, 18:04:59)
[GCC 9.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import guppy.heapy.heapyc
>>> hv = guppy.heapy.heapyc.HeapView(guppy.heapy.heapyc.RootState, ())
>>> f = lambda l: hv.cli_and(tuple(hv.cli_user_defined(hv.cli_none(), None, lambda obj: i, None) for i in l), {}).classify(None)
>>> a = f((1,)); b = f((2,)); a, b, type(a), type(b), a == b
((1,), (2,), <class 'guppy.heapy.heapyc.NodeTuple'>, <class 'guppy.heapy.heapyc.NodeTuple'>, True)
>>> a = f((1,)); b = f((2,)); a, b, type(a), type(b), a == b, hash(a) == hash(b)
((1,), (2,), <class 'guppy.heapy.heapyc.NodeTuple'>, <class 'guppy.heapy.heapyc.NodeTuple'>, True, False)

This guppy.heapy.heapyc.NodeTuple seems really hard to access from normal Python space. The result from the classifier seems always abstracted away by guppy.heapy.UniSet.Kind. Any idea how to test this?

I could test it by accessing the low level classifier. It is available at hpy().View.hv_cli_and
sverker@sverker-HP-Pavilion-g6-Notebook-PC:~/git/guppy-pe/bugs$ cat test_hv_cli_and.py
from guppy import hpy
h=hpy()
a=h.View.hv.cli_and((h.View.hv.cli_id(),), {})
assert a.classify(1)!=a.classify(2)
# Before fix
python
Python 2.7.15+ (default, Oct  7 2019, 17:39:04)
[GCC 7.4.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import test_hv_cli_and
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "test_hv_cli_and.py", line 4, in <module>
    assert a.classify(1)!=a.classify(2)
AssertionError
# After fix
python
Python 2.7.15+ (default, Oct  7 2019, 17:39:04)
[GCC 7.4.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import test_hv_cli_and
>>>

I can see you accessed the low level classifier too in a previous comment where you used the user defined classifier too, I didn't understand it or read it well enough right away.
Mine was an alternative that was arguably easier to do.
Was your question how we can test it without accessing the low level classifier?

Was your question how we can test it without accessing the low level classifier?

Yeah. Like, does the abstractions above invoke nodetuple_richcompare?

I  find that it is called but I can not produce an error. It is used when partitioning a large enough set with the and classifier. But gdb gives strange results. The richcompare is called with the same results each time, presumably from the dict lookup when there is hash collisions, but why is it getting the same arguments each time? Is it something wrong with gdb interaction?
cat test3.py
from guppy import hpy
h=hpy()
al=[[] for x in range(100000)]
a0=al[:10000]

print h.idset(al).by(h.Id & h.Size) & h.idset(a0[:100]).by(h.Id & h.Size).kind

gdb python
GNU gdb (Ubuntu 8.1-0ubuntu3.1) <IP_ADDRESS>80409-git
Copyright (C) 2018 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "i686-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
<http://www.gnu.org/software/gdb/documentation/>.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from python...(no debugging symbols found)...done.
(gdb) br nodetuple_richcompare
Function "nodetuple_richcompare" not defined.
Make breakpoint pending on future shared library load? (y or [n]) y
Breakpoint 1 (nodetuple_richcompare) pending.
(gdb) run
Starting program: /usr/bin/python
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/i386-linux-gnu/libthread_db.so.1".
Python 2.7.15+ (default, Oct  7 2019, 17:39:04)
[GCC 7.4.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import test3
warning: Error reading shared library list entry at 0xffffb420
warning: Error reading shared library list entry at 0xffffbe60

Breakpoint 1, nodetuple_richcompare (v=0xb748c36c, w=0xb748c38c, op=2)
    at src/heapy/hv_cli_and.c:202
202	{
(gdb) c
Continuing.

Breakpoint 1, nodetuple_richcompare (v=0xb748c36c, w=0xb748c38c, op=2)
    at src/heapy/hv_cli_and.c:202
202	{
(gdb) c
Continuing.

Breakpoint 1, nodetuple_richcompare (v=0xb748c36c, w=0xb748c38c, op=2)
    at src/heapy/hv_cli_and.c:202
202	{
(gdb) c
Continuing.

Breakpoint 1, nodetuple_richcompare (v=0xb748c36c, w=0xb748c38c, op=2)
    at src/heapy/hv_cli_and.c:202
202	{
(gdb) dis 1
(gdb) c
Continuing.
Partition of a set of 100 objects. Total size = 3200 bytes.
 Index  Count   %     Size   % Cumulative  % Kind (class / dict of class)
     0    100 100     3200 100      3200 100 list
>>>

Woah
(venv) zhuyifei1999@zhuyifei1999-ThinkPad-T480 ~/guppy3 $ gdb -ex 'break nodetuple_richcompare' -ex r --args python test3.py
GNU gdb (Gentoo 8.3.1 vanilla) 8.3.1
[...]

Breakpoint 1, nodetuple_richcompare (v=(<type at remote 0x7ffff7d5e960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dce290>), w=(<type at remote 0x7ffff7d5e960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dce290>), op=2) at /home/zhuyifei1999/guppy3/src/heapy/hv_cli_and.c:205
205	    Py_ssize_t vi=0, wi=0;
(gdb) bt
#0  nodetuple_richcompare (v=(<type at remote 0x7ffff7d5e960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dce290>), w=(<type at remote 0x7ffff7d5e960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dce290>), op=2) at /home/zhuyifei1999/guppy3/src/heapy/hv_cli_and.c:205
#1  0x00007ffff7c1c9f2 in do_richcompare (op=2, w=(<type at remote 0x7ffff7d5e960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dce290>), v=(<type at remote 0x7ffff7d5e960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dce290>)) at /usr/src/debug/dev-lang/python-3.7.5-r1/Python-3.7.5/Objects/object.c:695
#2  PyObject_RichCompare (v=v@entry=(<type at remote 0x7ffff7d5e960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dce290>), w=w@entry=(<type at remote 0x7ffff7d5e960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dce290>), op=op@entry=2) at /usr/src/debug/dev-lang/python-3.7.5-r1/Python-3.7.5/Objects/object.c:743
#3  0x00007ffff7a8610c in PyObject_RichCompareBool (op=2, w=(<type at remote 0x7ffff7d5e960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dce290>), v=(<type at remote 0x7ffff7d5e960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dce290>)) at /usr/src/debug/dev-lang/python-3.7.5-r1/Python-3.7.5/Objects/object.c:751
#4  lookdict (mp=0x7ffff5ddf910, key=<optimized out>, hash=<optimized out>, value_addr=0x7fffffffba90) at /usr/src/debug/dev-lang/python-3.7.5-r1/Python-3.7.5/Objects/dictobject.c:757
#5  0x00007ffff7c30bc1 in PyDict_GetItem (op={(<type at remote 0x7ffff7d5e960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dce290>): (...)}, key=<optimized out>) at /usr/src/debug/dev-lang/python-3.7.5-r1/Python-3.7.5/Objects/dictobject.c:1327
#6  0x00007ffff69faab7 in hv_cli_and_fast_memoized_kind (self=0x7ffff5dde7d0, kind=(<type at remote 0x7ffff7d5e960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dce290>)) at /home/zhuyifei1999/guppy3/src/heapy/hv_cli_and.c:42
#7  0x00007ffff69fae83 in hv_cli_and_classify (self=0x7ffff5dde7d0, obj=[]) at /home/zhuyifei1999/guppy3/src/heapy/hv_cli_and.c:117
#8  0x00007ffff6a01320 in cli_epartition_iter (obj=[], ta=0x7fffffffbce0) at /home/zhuyifei1999/guppy3/src/heapy/classifier.c:146
#9  0x00007ffff6a296ba in NyNodeSet_iterate (ns=0x555555580100, visit=0x7ffff6a012ea <cli_epartition_iter>, arg=0x7fffffffbce0) at /home/zhuyifei1999/guppy3/src/sets/nodeset.c:508
#10 0x00007ffff69f8b09 in NyNodeSet_iterate (ns=0x555555580100, visit=0x7ffff6a012ea <cli_epartition_iter>, arg=0x7fffffffbce0) at /home/zhuyifei1999/guppy3/src/heapy/impsets.c:44
#11 0x00007ffff69f8c3b in iterable_iterate (v=<guppy.sets.setsc.ImmNodeSet at remote 0x555555580100>, visit=0x7ffff6a012ea <cli_epartition_iter>, arg=0x7fffffffbce0) at /home/zhuyifei1999/guppy3/src/heapy/heapyc.c:93
#12 0x00007ffff6a01429 in cli_epartition (self=0x7ffff5dc6b90, iterable=<guppy.sets.setsc.ImmNodeSet at remote 0x555555580100>) at /home/zhuyifei1999/guppy3/src/heapy/classifier.c:166
[...]
(gdb) py-bt
Traceback (most recent call first):
  <built-in method epartition of guppy.heapy.heapyc.ObjectClassifier object at remote 0x7ffff5dc6b90>
  File "/home/zhuyifei1999/guppy3/guppy/heapy/Classifiers.py", line 35, in call_with_referrers
    return f(x)
  File "/home/zhuyifei1999/guppy3/guppy/heapy/Classifiers.py", line 114, in partition_cli
    self.cli.epartition)
  File "/home/zhuyifei1999/guppy3/guppy/heapy/Classifiers.py", line 105, in partition
    for k, v in self.partition_cli(iterable):
  File "/home/zhuyifei1999/guppy3/guppy/heapy/Part.py", line 709, in __init__
    for (kind, part) in classifier.partition(set.nodes)]
  File "/home/zhuyifei1999/guppy3/guppy/heapy/Part.py", line 800, in partition
    return SetPartition(self, set, er)
  File "/home/zhuyifei1999/guppy3/guppy/heapy/UniSet.py", line 1692, in get_partition
    p = a.fam.Part.partition(a, a.er)
  File "/home/zhuyifei1999/guppy3/guppy/heapy/UniSet.py", line 1603, in c_str
    a.fam.get_partition(a).ppob(ob)
  File "/home/zhuyifei1999/guppy3/guppy/heapy/UniSet.py", line 344, in __str__
    return self.fam.c_str(self)
  <built-in method print of module object at remote 0x7ffff6c6fd10>
  File "test3.py", line 6, in <module>
    print(h.idset(al).by(h.Id & h.Size) & h.idset(a0[:100]).by(h.Id & h.Size).kind)

Ok, a broken richcompare that is actively used but doesn't break the entire system... is interesting. I'm looking into it.

Ah, I see. The hash "collision" is because the tuple values are the same, and our hash function has been good enough to avoid a collision when the tuple values are not the same.
The bug was a false positive during a richcompare when the tuple are not the same, but in this case, richcompare would not be called if the hash values are different. There wouldn't be a way to trigger the bug unless we can generate a hash collision for different tuple values.
Why is it getting the same arguments each time... probably because we aren't leaking memory and the tuple is recreated and destroyed again and again in hv_cli_and_classify.
Btw, the object that is being classified is [] and the classify result of the and classifier is (<type at remote 0x7ffff7d5e960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dce290>). What sort of classifier is that? Isn't it supposed to be ([], 72) for an id & size classifier?

Btw, the object that is being classified is [] and the classify result of the and classifier is (<type at remote 0x7ffff7d5e960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dce290>). What sort of classifier is that? Isn't it supposed to be ([], 72) for an id & size classifier?

Answer to self: Clodo, this is classifying during print so the set's printing classifer is still clodo.
(gdb) up
#1  0x00007ffff7c189f2 in do_richcompare (op=2, w=(<type at remote 0x7ffff7d5a960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dc5290>), v=(<type at remote 0x7ffff7d5a960>, <guppy.sets.setsc.ImmNodeSet at remote 0x7ffff5dc5290>)) at /usr/src/debug/dev-lang/python-3.7.5-r1/Python-3.7.5/Objects/object.c:695
695	        res = (*f)(v, w, op);
[...]
(gdb)
#7  0x00007ffff69f1e83 in hv_cli_and_classify (self=0x7ffff5dd5640, obj=[]) at /home/zhuyifei1999/guppy3/src/heapy/hv_cli_and.c:117
117	    result = hv_cli_and_fast_memoized_kind(self, kind);
(gdb) p ((NyObjectClassifierObject *)((PyTupleObject *)classifiers)->ob_item[0])->def->classify
$1 = (PyObject *(*)(PyObject *, PyObject *)) 0x7ffff69f513b <hv_cli_type_classify>
(gdb) p ((NyObjectClassifierObject *)((PyTupleObject *)classifiers)->ob_item[1])->def->classify
$2 = (PyObject *(*)(PyObject *, PyObject *)) 0x7ffff69f2a28 <hv_cli_dictof_classify>

class AndClassifier(Classifier):
[...]
    def get_kind(self, k):
        ks = []
        for ki, ci in zip(k, self.args):
            ks.append(ci.get_kind(ki))
        return self.mod.UniSet.fam_And._cons(ks)

class Classifier:
[...]
    def partition(self, iterable):
        items = []
        for k, v in self.partition_cli(iterable):
            k = self.get_kind(k)
            v = self.mod.Use.idset(v, er=self.er)
            items.append((k, v))
        return items

    def partition_cli(self, a):
        ep = self.call_with_referrers(
            a,
            self.cli.epartition)
        return [(k, ep[k]) for k in ep.get_domain()]

Yeah, I see the NodeTuple is iterated through and used to create a list, so I don't think at the high level there is any use of NodeTuple, after the classification. During classification the NodeTuple is only used as keys to the dict, which would be somewhat hard to make a hash collision to hit this bug.
I guess a test at the low level is the way to go then. (Will do later)


Support flutter 1.22.0

../../Dev/flutter/.pub-cache/hosted/pub.flutter-io.cn/flutter_stetho-0.5.2/lib/src/http_client_request.dart:10:7: Error: The non-abstract class 'StethoHttpClientRequest' is missing implementations for these members:
 - HttpClientRequest.abort
Try to either
 - provide an implementation,
 - inherit an implementation from a superclass or mixin,
 - mark the class as abstract, or
 - provide a 'noSuchMethod' implementation.

class StethoHttpClientRequest implements HttpClientRequest {
      ^^^^^^^^^^^^^^^^^^^^^^^
org-dartlang-sdk:///third_party/dart/sdk/lib/_http/http.dart:2045:8: Context: 'HttpClientRequest.abort' is defined here.
  void abort([Object? exception, StackTrace? stackTrace]);
       ^^^^^

Yes!!! Same Error for me also. Please fix it.
Thanks.

I am also facing same issue.

I am also facing this issue. Please fix it


Allow setting tarfile format

For MediaWiki, we would like to be able to use a different tarfile format instead of the default one after identifying regressions in the new 3.8 default format.
We use this as a library, so if it would be possible to add an option to GitArchiver.create to let us specify tarfile.GNU_FORMAT that would be appreciated.
Our downstream ticket is https://phabricator.wikimedia.org/T257102.

Perhaps you can consider conversation after the archive is created, as explained on StackOverflow?

Thanks for the suggestion, I looked into that but bsdtar doesn't support --format=gnu (or at least the Fedora packaged version doesn't), and GNU tar doesn't support the easy conversion method that bsdtar does.
But while we could do it manually, it just seems pretty inefficient to create a tarball in a brokenish format, then uncompress it and recompress it in the correct format when we could just create it in the correct format to begin with.

There are myriad of options if think about it, and not just for tar but for the compressors too. And then there are also their flavors.
What if I extend the archiver to produce an mtree-formatted file, will it suffice? Or perhaps in some other intermediate format which you can easily work with using builtin tools and trivial shel pipelines.

There are myriad of options if you think about it, and not just for tar but for the compressors too. And then there are also their flavors.

True. I haven't fully thought this through, but what about allowing some arbitrary options dict to be passed to ZipFile or Tarfile as kwargs? That would allow us to pass through format without needing you to create a parameter for every single potential option and gives us flexibility in the future too.

What if I extend the archiver to produce an mtree-formatted file, will it suffice? Or perhaps in some other intermediate format which you can easily work with using builtin tools and trivial shel pipelines.

The really nice part about using this library is that it just takes care of everything for us, with very little complexity on our side :) But if that's what you think is best, we'll update our script to make it work.

As an update, we're now monkey patching tarfile.DEFAULT_FORMAT so this isn't a priority for us, but would be nice to have a less hacky way if possible.

Since you are using it as a library, you should be able to call GitArchiver .archive_all_files directly passing custom callable to archive files in a way that suits you best.


Add discord button in escape menu

About the PR

Title
Why / Balance

Technical details

Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: Added Discord button in escape menu.

i do not know how to do that

i do not know how to do that

Does it help if i tell you that we already have that control and its called LinkBanner (should have named it before but forgot the name), all you need to do is spawn it when you open the escape menu, adjust its position and ensure it closes with the escape menu

have no idea

Re-open if you do the review.


OPSEXP-1221: Create checkov action

this should be reverted there is already an official checkov action https://github.com/bridgecrewio/checkov-action


Glitchy vertical video effect

Describe the bug
If the extension thinks that the video is vertical, even if there are bits in the corner, it will try to apply the ambient effect
Making the video full-screen will trigger a glitch that squizzes the video
To reproduce
Steps to reproduce the behavior:

Go to a video
Enable the effect on vertical videos (V)
Go to full screen (F)

Expected behavior
Apply the effect as it'd be applied in non-full-screen mode
Video
https://user-images.githubusercontent.com/6422804/214131118-6cf4a4a7-53e8-426d-82f8-bbe49fe3933f.mp4
Desktop (please complete the following information):

OS: Windows 10 Pro 22H2
Browser: Brave
Browser version: 1.47.171
Extension version: 2.36.33

Other extensions:
Does the bug still persist when you disable all other extensions? If not, which other extension conflicts with the Ambient light for YouTube extension? Haven't tried

I can't seem to reproduce this issue. Can you try to reproduce it yourself with all other extensions disabled?

Yes
I reproduced it even after disabling all non-YouTube extensions

Strange, maybe YouTube is A/B testing a html/css update. Can you post a screenshot of the html and the logs in the Javascript console when it is in that incorrect state?

That's interesting. I just discovered that if I scroll down a little, the issue gets resolved

@avi12 I can't find the video tag in your screenshot. Is the video tag in the ytp-iv-video-content class? It's in the html5-video-container class for me.

The top highlighted is the <video>, and the bottom one is what happens when I use the element picker and move the mouse a bit to the left/right

I have the same style properties as in your devtools screenshot. The only difference is that I have a 1440p resolution and there are slightly different transform values but those are within an expected margin of error:

I suspect that there is sometimes another stylesheet or style attribute value applied with an override of the transform css property on your video tag. Because the html5-video-container tag is correctly scaled down by the scale(0.4504, 1) transformation and the video tag is correctly scaled up by the scale(2.22, 1) transformation defined in the --video-transform variable.

@avi12 I was finally able to reproduce this bug by trying out a different browsers. There seems to be a rendering bug in a specific version of chromium. The reason I was unable to reproduce this was because my Brave browser was not yet updated to the version with the rendering bug.
A workaround seems to be to apply the css property opacity: 0.99 to the video tag. Which hints to a rendering bug in Chromium since that has nothing to do with the transformation itself.
I'll try to find another workaround than the opacity property that does not have any visual impact to the video and report this bug to the Chromium developers.

I usually don't watch videos on full screen so it's not a big issue, but I appreciate that you're trying to fix it 🙏

@avi12 Your report is greatly appreciated because it looks like more users will see this bug in the future. This bug seems to be introduced since Chromium 107. And the actual reason why I was not able to reproduce it is because I have a custom flag enabled in Chrome.
So I'm 2 Chromium versions late at the moment, but I'll find a fix! 😄

@avi12 This is now fixed in version 2.37.2 (Already released to Opera but in review for Edge and Chrome)


feat: add support for parsing mysql intervals

MySQL handles intervals differently from the other dialects. This PR adds support for parsing those intervals when using the MySQL dialect.
One thing to note, in the tests, I have the following case:
mysql().one_statement_parses_to(
    "SELECT NOW() + INTERVAL CAST(6/4 AS DECIMAL(3,1)) HOUR",
    "SELECT NOW() + INTERVAL CAST(6 / 4 AS NUMERIC(3,1)) HOUR",
);

This may be wrong... The destination type probably should remain as DECIMAL(3,1). I say "may" because a bug was reported to the MySQL team in 2017; however, it isn't clear if this bug or documentation issue, and there hasn't been any progress since then.

Hi @kyle-mccarthy  -- I am sorry it looks like this PR never got reviewed and it has now bitrotted. If you would like to pursue this can you please update the PR?
Marking this as a draft to show it is waiting some changes so I can easily filter out which PRs need review. Please mark it as "ready for review" when it is next ready.

Closing as stale -- please reopen if you still plan to work on it


Update whats-new.md

Fixed the wording under Microsoft Priva
fixes#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/10651

@JohanFreelancer9 Please copy review


[lexical-playground] Fix: ListMaxIndentLevelPlugin allows indent beyond maxDepth on paste

Description
The ListMaxIndentLevelPlugin restricts the maximum indent level for list items based on the defined maxDepth. However, when pasting content with a deeper indent, the plugin doesn't automatically enforce this restriction. This allows the pasted content to exceed the allowed indent level, resulting in a higher indent depth than the set maxDepth.
Added logic to restrict paste behaviour for lists in ListMaxIndentLevelPlugin, ensuring that pasted content respects the defined maxDepth and prevents exceeding the allowed indent level.
Closes #5170
Test plan
Before
maxDepth issue
URL: https://d.pr/v/IRF0rH
After
solved maxDepth issue
URL: https://d.pr/v/zw3uMQ
Note: The solution restricts pasting beyond indent level 7, ensuring the list structure stays within the defined limit.

@mdmuhtasimfuadfahim don't forget to do CLA as well. Thanks!

**ivailop7 ** c

Hi @ivailop7. I have updated the code. Please check and signed here https://code.facebook.com/cla with the same email I did the commits. I guess it will take 1 hour. Will it be okay for the Facebook CLA Check pass? If not. Please let me know what I should do for it. Thanks in advance.

Hi @ivailop7. Facebook CLA Check is done now. Please check the code.

I think what this does is prevent pasting at all when the indent is at the limit (even if the paste is just plain text), it doesn't prevent the problem from happening if the pasted content contains multiple levels of indent

Hi @etrepum. If I don't prevent the pasting when the limit hits it generates the issue [#5170](url). Well let me check if I can allow the pasting for text but not to increase the indent level.

Yes, I understand that this prevents one situation where this issue happens, but I don't think it's a complete fix because pasted content can have embedded indentation. I also think it also prevents certain pastes that are expected to work. Possibly the only way to make this solution complete without unintended consequences would be through node transforms and not just commands.


UnicodeDecodeError: don't know how to handle UnicodeDecodeError in error callback

I am trying to create a saved search (type alert) on Splunk using python-sdk.
The problem I am facing is the alert gets created with the proper fields but it still throws an error
UnicodeDecodeError: don't know how to handle UnicodeDecodeError in error callback.
To Reproduce
Steps to reproduce the behavior:

Install python-sdk and write a python code for creating saved search following README.md of python-sdk
Create a saved search having below values

alert_content = {
    "name": "Test Alert",
    "search": "QUERY",
    "alert_type": "number of events",
    "is_scheduled": "true",
    "cron_schedule": "0 9 * * 1",
    "alert_comparator": "greater than",
    "alert_threshold": 0,
    "app": "search",
    "actions": "email",
    "action.email": 1,
    "action.email.to"<EMAIL_ADDRESS>}

It would look something like this

self.service.saved_searches.create(**alert_content)

It creates the alert with proper fields and values but it also throws a below-given error.

UnicodeDecodeError: don't know how to handle UnicodeDecodeError in error callback

Expected behavior
Alert should be created without throwing an error.
Splunk (please complete the following information):

Version: [e.g. <IP_ADDRESS>]
OS: [e.g. centos 7.9.2009]
Deployment: [e.g. cluster]

SDK (please complete the following information):

Version: [e.g. 1.7.3]
Language Runtime Version: [e.g. Python 3.11]
OS: [e.g. MacOS 13.6]

Additional context
I have backtracked the functions where it is throwing error from
So far I found that _load_atom in https://github.com/splunk/splunk-sdk-python/blob/master/splunklib/client.py#L205C5-L205C15 is throwing the error.

Hi @faizanmansuri , we are unable to reproduce the issue. So can you please share other details such as Cluster configurations, as well as the response body that is received from the Splunk Enterprise.
Also could you confirm if the issue exists only for the saved-search APIs or all the other APIs as well.
Thanks!

Hi @ashah-splunk, Thanks for looking into it.
I found that there is an issue with other APIs response as well in our specific Splunk environment  So I don't think it is a SDK problem.


Sidenav should highlight the section you're viewing

For example:

Good eye! This got messed up in my very last push, so I need to investigate.
https://github.com/jhil/jeffanders-2022/issues/33


Currency overrides with bdh

Code Sample, a copy-pastable example if possible
# Your code here, this should be a minimal reproducible example, see https://stackoverflow.com/help/mcve

Problem description
[this should explain why the current behaviour is a problem and why the expected output is a better solution.]
Expected Output
Version Information
[paste the output of pdblp.__version__ here below this line]

Hi I am trying to do the below to get historical data just in a different currency.
con.bdh(['BA US Equity'],['PX_LAST'],'20150629', '20150630', ovrds = [('CRNCY','EUR')])
However, the result is still in USD. It seems it was not override. But if you do the same in BDH in excel, you could get the EUR number by "CRNCY=USD". How can I do that in pdblp?

I have deleted https://github.com/matthewgilbert/pdblp/issues/61 since it was a duplicate. The original post you made is a template which you are meant to fill out but have ignored. Please take a bit of time to research the etiquette of using github issues.


Ninja not found when building a cmake project (assumption that Ninja is already installed.)

The code appears to assume that Ninja is already installed, which it won't be unless one is actively developing Swift itself.
...
http://gb.archive.ubuntu.com/ubuntu/pool/main/z/zlib/zlib1g_1.2.11.dfsg-2ubuntu9.2_amd64.deb – 2.2 MB
http://gb.archive.ubuntu.com/ubuntu/pool/main/z/zlib/zlib1g-dev_1.2.11.dfsg-2ubuntu9.2_amd64.deb – 11.6 MB

Unpacking and copying Swift binaries for the host triple...
8455394 blocks

Unpacking Swift distribution for the target triple...

Copying Swift core libraries for the target triple into Swift SDK bundle...

Unpacking and copying `lld` linker...
CMake Deprecation Warning at /private/var/folders/d6/2m5sz0rx573drxq0b25xttl80000gn/T/swift-sdk-generator-169E0B/cmake/Modules/CMakePolicy.cmake:6 (cmake_policy):
  The OLD behavior for policy CMP0114 will be removed from a future version
  of CMake.

  The cmake-policies(7) manual explains that the OLD behaviors of all
  policies are deprecated and that a policy should be set to OLD only under
  specific short-term circumstances.  Projects should be ported to the NEW
  behavior and not rely on setting a policy to OLD.
Call Stack (most recent call first):
  CMakeLists.txt:13 (include)

CMake Error: CMake was unable to find a build program corresponding to "Ninja".  CMAKE_MAKE_PROGRAM is not set.  You probably need to select a different build tool.
CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_ASM_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!
Error: nonZeroExitCode(1, SwiftSDKGenerator.CommandInfo(command: "PATH=\'/bin:/usr/bin:/usr/local/bin\' cmake -B build -G Ninja -S llvm -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_PROJECTS=lld -DLLVM_TARGETS_TO_BUILD=X86", currentDirectory: Optional("/var/folders/d6/2m5sz0rx573drxq0b25xttl80000gn/T/swift-sdk-generator-169E0B"), file: "/Users/diggory/Code/OpenSource/swift-sdk-generator/Sources/SwiftSDKGenerator/Generator/LocalSwiftSDKGenerator.swift", line: 348))
diggory@MBP-16 swift-sdk-generator %

Would installing ninja from Homebrew work for you as a workaround in the meantime?

Yes, no problem.


Create tukorea.txt

https://tukorea.ac.kr

@fiedith I am sorry, the domain tukorea.ac.kr is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


[BUG]: User can create an event where the end date is before start date in the Power Platform Training Management app

Describe the bug
User can create an event where the end date is before start date in the Power Platform Training Management app
Component (please tell us which flow or app you are experiencing issues with):

Solution Nurture
Version 1.40
Power Platform Training Management

To Reproduce
Steps to reproduce the behavior:

Launch the Platform Training Management app
Create an event where the end date is before the start date
Event is saved

Expected behavior
Error message should be shown to user to prevent them saving the evenvt,
Screenshots

Additional context
Add any other context about the problem here.

I did a quick fix where the submit button only shows if end date >= start date.
Will still allow bad entries if they are bad wrt time but that seems less likely to do on accident and more expensive to fix.
Thanks for raising. We will ship with the next release.

Hello. We have just released the May edition of the toolkit and it contains this fix.
Please be sure to follow the instructions at installing updates in order to remove any unmanaged layers you have in place, or you will not receive the update.
Thank you for helping make CoE great!


Feature: Form for adding libraries

Not really an issue but it would be great to have a form for adding libraries rather than updating the json file currently.
I know this would require a backend db and some sort of authentication so i'm thinking maybe using firebase or a mongo instance for the db and auth using github?
Just some thoughts.
If approved, I'd love the opportunity to take it on and implement it.

The reasons why I like the PR process for adding a library:

I can have a discussion with the person who is adding the library
I can review and approve the library
(subjective) It prevents someone from arbitrarily adding libraries and spamming our list, the list is supposed to be 'curated', we want to be careful with what we add.

I like the authentication idea, but I think its out of scope for now. I appreciate your interest though and I'll keep note of this as the project progresses.

Thanks for the feedback. Understood


Add ability to reorder folds

closes: #717

@nicolaskruchten I think this is what we want would you check please ✨
I didn't add it to other accordions, but this easily could be extended to them

@dmt0 I think this is ready for code review as well

adjusted styles

so I've added the ability to reorder: images, shapes, annotations on top of traces
for transforms, I wasn't sure about the array to reorder (it changes based on the transform applied, and I'm not sure of the side effects) so I didn't do it here.
for rangeselector buttons, the path to get to the array to reorder is a bit less straight forward. I could perhaps use the nested property function of plotly.js. but didn't do it here.
just picked the low hanging fruit. let me know if that's ok @nicolaskruchten

💃 !

✨ although I don't love the "you can't click here" cursor on the greyed out arrows... I'd just leave it as a normal non-link cursor.

although I don't love the "you can't click here" cursor on the greyed out arrows
yeah, agree, removed!


Enable gesture recognizer in selectable rich text

Description
The gesture recognizer is not wired up in render editable. This pr wires it up.
Related Issues
Fixes https://github.com/flutter/flutter/issues/43494
Tests
I added the following tests:
"text span with tap gesture recognizer works in selectable rich text"
"text span with long press gesture recognizer works in selectable rich text"
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I signed the CLA.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] All existing and new tests are passing.
[ ] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[ ] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: Replace with a link to your migration guide

bump for attention

I'd probably recommend making a trivial migration guide that just says "if you hit this error message [...] then you need to make sure you lay out the RenderEditable before you try to hit test with it" or some such.

(and mention that this won't be an issue if you're using widgets)

I'd probably recommend making a trivial migration guide that just says "if you hit this error message [...] then you need to make sure you lay out the RenderEditable before you try to hit test with it" or some such.

Would you recommend to put this in https://flutter.dev/docs/release/breaking-changes? or an email annoucement?

Yes.

Why it isn't in Flutter 1.17?


Need to change ws2811 reset timing

Like here https://github.com/forkineye/ESPixelStick/issues/47
Our standard arduino sketch might similar fix

Our standard arduino sketch might similar fix

fix is done with taking latest fastled ...


Introduce changelog

The project should have a proper changelog so that user can understand differences between versions.

This is an excellent changelog guide: https://keepachangelog.com/en/0.3.0/

A change log is now in the root of the project: 73bff71287d1caf18eb14cb66613fe540131c8b0 (CHANGELOG.md)
Before we can close this ticket we need to add information in the README.md on when and what to add in the changelog


NFL: Failed to parse JSON

URL: http://www.steelers.com/news/article-1/Tomlin-on-Ben-getting-Vick-ready/56399c96-4160-48cf-a7ad-1d17d4a3aef7
c:\Transmogrifier>youtube-dl.py -v "http://www.steelers.com/news/article-1/Tomlin-on-Ben-getting-Vick-ready/56399c96-4160-48cf-a7ad-1d17d4a3aef7"
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.steelers.com/news/article-1/Tomlin-on-Ben-getting-Vick-ready/56399c96-4160-48cf-a7ad-1d17d4a3aef7']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2015.09.28
[debug] Python version 2.7.5 - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg N-71727-g46778ab, rtmpdump 2.4
[debug] Proxy map: {}
[nfl.com] 56399c96-4160-48cf-a7ad-1d17d4a3aef7: Downloading webpage
[nfl.com] 56399c96-4160-48cf-a7ad-1d17d4a3aef7: Downloading player config
[nfl.com] 56399c96-4160-48cf-a7ad-1d17d4a3aef7: Downloading JSON metadata
ERROR: 56399c96-4160-48cf-a7ad-1d17d4a3aef7: Failed to parse JSON  (caused by ValueError('No JSON object could be decoded',)); please report this issue on https://yt-dl.org/bug . Ma
ng the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "C:\Transmogrifier\youtube-dl.py\youtube_dl\extractor\common.py", line 481, in parse_json
return json.loads(json_string)
File "C:\Python27\lib\json_init.py", line 338, in loads
return _default_decoder.decode(s)
File "C:\Python27\lib\json\decoder.py", line 365, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "C:\Python27\lib\json\decoder.py", line 383, in raw_decode
raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded
Traceback (most recent call last):
File "C:\Transmogrifier\youtube-dl.py\youtube_dl\YoutubeDL.py", line 660, in extract_info
ie_result = ie.extract(url)
File "C:\Transmogrifier\youtube-dl.py\youtube_dl\extractor\common.py", line 288, in extract
return self._real_extract(url)
File "C:\Transmogrifier\youtube-dl.py\youtube_dl\extractor\nfl.py", line 160, in _real_extract
url_template.format(id=video_id), video_id)
File "C:\Transmogrifier\youtube-dl.py\youtube_dl\extractor\common.py", line 475, in _download_json
json_string, video_id, transform_source=transform_source, fatal=fatal)
File "C:\Transmogrifier\youtube-dl.py\youtube_dl\extractor\common.py", line 485, in _parse_json
raise ExtractorError(errmsg, cause=ve)
ExtractorError: 56399c96-4160-48cf-a7ad-1d17d4a3aef7: Failed to parse JSON  (caused by ValueError('No JSON object could be decoded',)); please report this issue on https://yt-dl.org
u are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
This link (same vid) does work:
http://www.steelers.com/videos/videos/Tomlin-looks-ahead-to-Ravens-on-a-short-week/12693586-6ea9-4743-9c1c-02c59e4a5ef2
Thanks
Ringo

I don't think "http://www.steelers.com" is supported. As the output says so.

http://prod.hdvideo.steelers.clubs.nfl.com//PIT/videos/dct/video_audio/2015/09-September/092815_TomlinPresser-,50,70,120,200,320,500,0k.mp4.csmil/bitrate=0?v=2.10.3&fp=WIN 19,0,0,185&r=LXIRJ&g=OQFHYIRNDXNC&seek=0
You can get the video here though

Xonshiz
dstftw recently added individual NFL team domains support:
https://github.com/rg3/youtube-dl/commit/5b4c54631a8888d75fa766c0bd6ec1822e6caec5
the direct link to the video on steelers.com does work with latest YouTube-DL builds.
http://www.steelers.com/videos/videos/Tomlin-looks-ahead-to-Ravens-on-a-short-week/12693586-6ea9-4743-9c1c-02c59e4a5ef2


Add proton and newer pressure vessel process names

These processes appear when launching games with Proton 8.0+ adding this hides them fully as intended.

didn't try it but seems reasonable.
also just realized how bad the code in this repo is.
i was just making this for myself didn't think people would use it.
gonna improve the codebase tonight, then release the new version.


Doc software version indicates 0.0.1 when rendered via workflow dispatch

Problem

The version displays 0.0.1 in https://www.diffpy.org/diffpy.fourigui/.
The version is dynamically retrieved from the latest tag. Here, when a single doc rendering workflow is used, it fails to grab the latest tag.
Proposed solution
Begin investigating.

Found the problem. When we checkout, we don't want to just checkout the main branch but also grab the latest tags as well.
Change from
        uses: actions/checkout@v4

to
        uses: actions/checkout@v4
        with:
          fetch-depth: 0  # Fetches all history for all branches and tags

Result
Before:

After:

fixed!  Nicely done:

closing


🐛 BUG: Wrangler crashes when running pages dev if functions contain invalid syntax

Which Cloudflare product(s) does this pertain to?
Pages, Wrangler core
What version(s) of the tool(s) are you using?
Wrangler 3.3.0
What version of Node are you using?
18.17.0
What operating system are you using?
WSL (Ubuntu 22.04)
Describe the Bug
When running wrangler pages dev [...], Typescript functions get auto-transpiled as part of the dev watcher process.
If the TS file contains invalid syntax, the build errors as expected.
When this happens, however, the dev server itself appears to crash and require a restart. I would not expect this. Furthermore, the dev server doesn't shut down cleanly from this crash, instead leaving the old HTTP server live on :8788 (and proxy/proxy port if appropriate), requiring the process to be manually killed to restart the dev server.
Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs

This has been driving me batty for months now... made a PR to fix it.