[flutter_tools] PathNotFoundException deletion failed in FontConfigManager.dispose

On stable/3.16.1
Command flutter test
PathNotFoundException: Deletion failed, path = '/var/folders/xl/slhmrd4s4kgbhf6rs9t8xd8w0000gn/T/flutter_tools.RCaYjG/flutter_test_fonts.vnvYiE' (OS Error: No such file or directory, errno = 2)
at _checkForErrorResponse(common.dart:55)
at _Directory._delete.<anonymous closure>(directory_impl.dart:185)
at _rootRunUnary(zone.dart:1407)
at _CustomZone.runUnary(zone.dart:1308)
at <asynchronous gap>(async)
at ForwardingFileSystemEntity.delete(forwarding_file_system_entity.dart:66)
at <asynchronous gap>(async)
at FontConfigManager.dispose(font_config_manager.dart:37)
at <asynchronous gap>(async)
at FlutterPlatform.close(flutter_platform.dart:643)
at <asynchronous gap>(async)
at Loader.close.<anonymous closure>.<anonymous closure>(loader.dart:295)
at <asynchronous gap>(async)
at Future.wait.<anonymous closure>(future.dart:523)
at <asynchronous gap>(async)
at Future.wait.<anonymous closure>(future.dart:523)
at <asynchronous gap>(async)
at Loader.close.<anonymous closure>(loader.dart:292)
at <asynchronous gap>(async)
at Runner.close.<anonymous closure>(runner.dart:249)
at <asynchronous gap>(async)
at _execute(executable.dart:171)
at <asynchronous gap>(async)
at main(executable.dart:38)
at <asynchronous gap>(async)
at _DefaultTestWrapper.main(test_wrapper.dart:26)
at <asynchronous gap>(async)
at _FlutterTestRunnerImpl.runTests(runner.dart:217)
at <asynchronous gap>(async)
at TestCommand.runCommand(test.dart:473)
at <asynchronous gap>(async)
at FlutterCommand.run.<anonymous closure>(flutter_command.dart:1350)
at <asynchronous gap>(async)
at AppContext.run.<anonymous closure>(context.dart:150)
at <asynchronous gap>(async)
at CommandRunner.runCommand(command_runner.dart:212)
at <asynchronous gap>(async)
at FlutterCommandRunner.runCommand.<anonymous closure>(flutter_command_runner.dart:348)
at <asynchronous gap>(async)
at AppContext.run.<anonymous closure>(context.dart:150)
at <asynchronous gap>(async)
at FlutterCommandRunner.runCommand(flutter_command_runner.dart:294)
at <asynchronous gap>(async)
at run.<anonymous closure>.<anonymous closure>(runner.dart:112)
at <asynchronous gap>(async)
at AppContext.run.<anonymous closure>(context.dart:150)
at <asynchronous gap>(async)
at main(executable.dart:90)
at <asynchronous gap>(async)

This is the top crasher on stable 3.16


defineLazyEventHandler is not defined

Environment
Stackblitz, local = same results
Reproduction
https://stackblitz.com/~/github.com/martinszeltins/nuxt-from-scratch
Describe the bug
When I run the nitro dev command it shows me the following error. But h3 is installed and defineLazyEventHandler is supposed to be available. I have also extended nitro types in tsconfig and my IDE seems to recognize it as auto-import. I don't understand what is causing this problem.
> nitropack prepare

The CJS build of Vite's Node API is deprecated. See https://vitejs.dev/guide/troubleshooting.html#vite-cjs-node-api-deprecated for more details.

 ERROR  defineLazyEventHandler is not defined                                                        1:19:55 PM

  at eval (nitro.config.ts#cjs:15:14)
  at evalModule (node_modules/.pnpm/jiti@1.21.6/node_modules/jiti/dist/jiti.js#cjs:1:247388)
  at Object.jiti (node_modules/.pnpm/jiti@1.21.6/node_modules/jiti/dist/jiti.js#cjs:1:245316)
  at resolveConfig (node_modules/.pnpm/c12@1.11.1/node_modules/c12/dist/shared/c12.cab0c9da.mjs:365:26)
  at async loadConfig (node_modules/.pnpm/c12@1.11.1/node_modules/c12/dist/shared/c12.cab0c9da.mjs:167:23)
  at async loadOptions (node_modules/.pnpm/nitropack@2.9.7/node_modules/nitropack/dist/nitro.mjs:4782:21)
  at async Module.createNitro (node_modules/.pnpm/nitropack@2.9.7/node_modules/nitropack/dist/nitro.mjs:5119:19)
  at async Object.run (node_modules/.pnpm/nitropack@2.9.7/node_modules/nitropack/dist/cli/prepare.mjs:84:19)
  at async runCommand (node_modules/.pnpm/citty@0.1.6/node_modules/citty/dist/index.mjs:337:16)
  at async runCommand (node_modules/.pnpm/citty@0.1.6/node_modules/citty/dist/index.mjs:328:11)

 ERROR  defineLazyEventHandler is not defined                                                        1:19:55 PM

 ELIFECYCLE  Command failed with exit code 1.

Additional context
No response
Logs
No response

nevermind, just had to import it from h3
import { defineEventHandler, defineLazyEventHandler, fromNodeMiddleware } from 'h3'


Add RadiusX/RadiusY to RectangleGeometry

In WPF RectangleGeometry has RadiusX and RadiusY properties to get rounded corners.
I'd like to use a rounded rectangle as a clipping to a control in Avalonia.
In WPF:
<!-- Create a rectangle with rounded corners
             by giving the RectangleGeometry a RadiusX
             and a RadiusY of 10.  -->
          <RectangleGeometry
            Rect="20,20,150,50" RadiusX="10" RadiusY="10" />

I might get to this over the holidays along with the BackgroundSizing work. What is required here is also similar to what I did for Rectangle itself and I can use it there too.

This is coming in with BackgroundSizing. It's sharing the same code a few other places.


Use rekor signedEntryTimestamp if present

If the SET is present, we store the logEntry as a bundle annotation. On verify, we first try to verify it against the rekor public key (not experimental). if that fails, then we hit the tlog.
Starts to address https://github.com/sigstore/cosign/issues/281, still need to add the --verify-inclusion flag to force checking the tlog

Ehl gee tee emm!


[Refactoring] Add helper functions process

Current code for adding helper functions
	if err := env.addHelpers(vm); err != nil {
		return err
	}
	if err := env.addDatabaseFunctions(vm); err != nil {
		return err
	}
	if err := env.addVolatileFunctions(vm); err != nil {
		return err
	}
	if err := env.addSearch(vm); err != nil {
		return err
	}
	if err := env.addSendMail(vm); err != nil {
		return err
	}

Helper functions may be moved to separated package and added to functions registry via init()
Within Execute - add registered functions
Core code will be more clear
Also it will allow to add custom helper functions

Hey there, I'm not sure I fully understand what you're proposing.
Those expored functions are already just wrapper around existing functionalities. Take the Emailer.Send you just added, the glue code is to handle the translation between goja types and Go's, that's mostly it.
I'm not sure I see the benefits of moving that glue code outside the function package.
Maybe having a file in the function package with all that glue code, removing it from runtime.go, say something like exposed.go (for lack of better word for now haha) and moving all those env.addX there and have one function call from the Execute function say: env.setuop() or whatever.
This could be a nice refactor. Can you showcase what you're referring to by having them in their own package, can you give an example for instance of what could be customized?

Possibly separated package it too much
Later I'll try to implement this staff in exposed.go


AllowJS Not Working?

TypeScript Version:  3.1.1

Search Terms:
allowJS Implicitly has an 'any' type
TSConfig
{
    "compilerOptions": {
      "allowJs": true,
      "module": "commonjs",
      "noEmitOnError": true,
      "noFallthroughCasesInSwitch": true,
      "noImplicitReturns": true,
      "noUnusedLocals": true,
      "experimentalDecorators": true,
      "emitDecoratorMetadata": true,
      "moduleResolution": "node",
      "resolveJsonModule": true,
      "strict": true,
      "target": "es2017",
      "lib": [
        "dom",
        "es2017",
        "es2018",
        "esnext"
      ],
  }

I am importing this JS file:
module.exports = {
  development: {
    username: "postgres",
    password: "",
    database: "development_database",
    host: "<IP_ADDRESS>",
    dialect: "postgres"
  },
  test: {
    username: "postgres",
    password: "",
    database: "test_database",
    host: "<IP_ADDRESS>",
    dialect: "postgres",
    logging: false
  }
}

like this:
import { Sequelize } from 'sequelize-typescript'
import * as sequelizeConfig from '../db/config'

const config = sequelizeConfig[process.env.NODE_ENV || 'development']
...
...

Expected behavior:
On 3.0.3 this compiles fine with no issues.
Actual behavior:
On 3.1.1 I get this error:
Element implicitly has an 'any' type because type '{ development: { username: string; password: string; database: string; host: string; dialect: string; }; test: { username: string; password: string; database: string; host: string; dialect: string; logging: boolean; }; }' has no index signature.

7 const config = sequelizeConfig[process.env.NODE_ENV || 'development']

I'm new to Typescript, but shouldn't allowJS: true prevent this scenario?

When you set allowJs it means we check the JS file. That means we can infer a type for its output, which as you can see from the error message is { development: ..., test: ... }. When you index into that with [process.env.NODE_ENV || 'development'], there's an error because process.env.NODE_ENV could be anything, so we don't know that it's a valid key.
You could fix this with:
const env = process.env.NODE_ENV || 'development';
if (env !== "development" && env !== "test") throw new Error();
const config = sequelizeConfig[env]

(The reason this changed since 3.0 is because we no longer give an index signature to JS object literals in certain positions.)

Yeah, as @andy-ms said, we made it so noImplicitAny (a part of strict) opt you completely out of the implicit {[index: string]: any} index signatures objects originating in JS used to have (since it caused a ton of type safety issues, and the flag is supposed to help remove those).

=== and !== are treated specially by TypeScript while includes is not.


Issue loading external JSON with Chrome

Hi,
When I go on your website with my version of Chrome, I don't have any problem seeing all your tables. However, when I create the page on my computer and then visualize on Chrome, I can't see anything. It works on IE and Firefox but not Chrome. I am wondering why.
Second question, I added the folder to an internal server with local links and now, none of the browsers can read the JSON file. any idea, please ?
Thank you a lot for this useful tool,
Alex

Alex,
Sorry it took me so long to respond.  Make sure that everything is running on the same domain name. Since you are trying to access the JSON using AJAX and you are not allow to make cross domain calls, it would cause the AJAX to fail.  Check your browser console to be sure.
Again sorry for the delay.
Michael


[fix] Removed python yaml dependency that was causing issues.

Description
This fix removes python yaml dependency in the intrinsic calibration node, yaml seemed to have come compatibility issues across platforms. Instead opencv FileStorage and python I/O, which seems consistent throughout versions.
Tested on two platforms:

Ubuntu: 16.04
ROS: Kinetic
Python: 2.7.13
OpenCV: 3.3.1-dev

and:

Ubuntu 14.04
ROS: Indigo
Python: 2.7.6
OpenCV: 2.4.8

Related PRs
See #1621 for bug description

@cirpue49 can you please test it in your environment?


imx.to ripper

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[ ] a bug fix (Fix #...)
[ x] a new Ripper
[ ] a refactoring
[ ] a style change/fix
[ ] a new feature

Description
Download both single images and galleries from imx.to.
Testing
Required verification:

[ x] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[x ] I've verified that this change works as intended.

[ x] Downloads all relevant content.
[x ] Downloads content from multiple pages (as necessary or appropriate).
[ x] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[x ] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ x] I've added a unit test to cover my change.

thank you @japodhidev ! would it be possible you create one commit on top of branch "main"? i delete the branch master - which will automatically close the pull request. just open it again.


TagDeclarators cannot be used as function parameters

This is a weird use of TagDeclarators that we may not care about, but I wanted to write down everything I learned about them.
A TagDeclarator like $a cannot be passed as a parameter and used.
fn pentagon = (len, taga, tagb, tagc) => {
  const sg = startSketchOn('XY')
  |> startProfileAt([-len / 2, -len / 2], %)
  |> angledLine({ angle: 0, length: len }, %,taga) // <-- taga does not modify ProgramMemory here.
  |> angledLine({
       angle: segAng(a, %) + 180 - 108, // <-- After merging #3015, `a` is not defined here.
       length: len
     }, %, tagb)
  |> angledLine({
       angle: segAng(b, %) + 180 - 108,
       length: len
     }, %,tagc)
  |> angledLine({
       angle: segAng(c, %) + 180 - 108,
       length: len
     }, %, $d)
  |> angledLine({
       angle: segAng(d, %) + 180 - 108,
       length: len
     }, %)

  return sg
}

const p = pentagon(32, $a, $b, $c) // <-- `a` gets defined in ProgramMemory here.
  |> extrude(10, %)

This was revealed by #3015. Prior to that, it used to work because there was only a single global scope. The TagIdentifier was defined in that global scope of ProgramMemory, and that's what was being used inside the function.
The issue is that evaluating a TagDeclarator like $foo binds an identifier foo to its corresponding TagIdentifier, but returns the TagDeclarator. If you have a TagDeclarator passed in as a parameter to a function, you can never get its corresponding TagIdentifier.
https://github.com/KittyCAD/modeling-app/blob/6e7e6e96cfadffd5590863546133121f9d58505c/src/wasm-lib/kcl/src/ast/types.rs#L2048-L2059
When a TagDeclarator is stored in a variable like a function parameter, looking up the identifier foo in ProgramMemory doesn't re-evaluate the TagDeclarator; it (sensibly) assumes it has already been evaluated.
This seems like a case where TagDeclarator and TagIdentifier evaluation needs to be revisited, especially now that we have scoped tags #2941.

yeah I think its fine for this not to be supported, its an anti-pattern

Jess, I know you don't like a whole lot of issues open, so I'm going to close this 🙂
I think that in the future we can either add a static check to disallow this or make function calls to functions that can create tags inspect the argument and define it, similar to an output parameter.


Add real grid test

Codecov Report
Base: 72.85% // Head: 72.85% // No change to project coverage :thumbsup:

Coverage data is based on head (48ab9b2) compared to base (60d11d5).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #57   +/-   ##
=======================================
  Coverage   72.85%   72.85%
=======================================
  Files          18       18
  Lines        1315     1315
  Branches      221      221
=======================================
  Hits          958      958
  Misses        288      288
  Partials       69       69

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


AttributeError: '_io.TextIOWrapper' object has no attribute 'compile'

I get this error trying to load a CSV file into Postgres

Could you attach a code snippet I can use to reproduce the error?

Sorry, I changed approach and don't have the code to hand any more.

Got it, feel free to reopen if you run into the issue again.


Remove '$0' special syntax

Fixes #4715 (please discuss the merits of the idea there)

There doesn't seem to be objection to #4715, so it would be great if someone could review and merge this.


Feature Request - Provide ability to execute workflow from Overview page when a recurrence trigger is used

Describe the Bug with repro steps
We expose Run/Run with payload on portal and VS Code but in VS Code when a recurrence trigger is used, there is no way to kick off the workflow other than waiting for it to run on the interval. It would be nice to have an explicit gesture and allow the ability to kick it off similar to portal.
What type of Logic App Is this happening in?
Standard (VSCode)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
No
Workflow JSON
No response
Screenshots or Videos

Portal experience (Consumption

Browser
N/A
Additional context
No response

Hi @kewear thanks for raising this, this would be a great feature, for how different the vscode and portal is its not as easy as just enable the button. The extension bundle exposes one callback url for the "request" type triggers and thats the reason on why we can call it.
I will ask the BE team is there any expose api in the localhost for the extension bundle which we might be able to call and run the triggers that doesn't expose a callback url.


fix: checkout pr in ecosystem ci

As discussed in discord, right now the git ambiguity check always fails because the pr commits are yet to exist in the main history.
in this fix, we checkout to the pull request history and have a look there.

/ecosystem-ci run vite-plugin-vue

It seems I can't test this out here. Let me do it on my own fork.

I confirmed that it works 👍
https://github.com/sapphi-red/vite/pull/7


Ionic 4 mobile app with angular-auth2-oidc

Hi
I could successfully launch the ionic app in mobile. But the identity server login page is getting launched in external browser. But I want to launch the page in inappbrowser  for both android and iOS.
How can I achieve the same with angular-auth2-oidc and ionic cordova.

Hmm, that's a bit broad question, we're but a small community, typically focused on helping out on bugs and feature requests. I suggest trying your luck to find someone who created a tutorial on your scenario, or be the first to write it. Otherwise, we'd have to hope for a community member here to pop up to answer your question...

Hi, sorry if I missed anything.
The existing code angular-auth2-oidc is launching the url in external browser in mobile app.
How to open the same in inapp browser if there is a facility already there.

Hello, @CodeDangler
I would also like to use this library in mobile applications, as it is one of the best I have found.
I tried to implement a solution suggested by @nhance  without success:

https://github.com/manfredsteyer/angular-oauth2-oidc/pull/235
https://github.com/manfredsteyer/angular-oauth2-oidc/pull/369

But I found another library, not as good as angled-oauth2-oidc from @manfredsteyer but that somehow works:

https://github.com/wi3land/ionic-appauth

@jeroenheijmans is it possible to turn this from a question into a feature request? The Ionic community is huge and we would profit a lot from such a well-maintained lib as this one, if it offered the option to use an own redirect instead of a static one.
Another useful feature request for Ionic would be being able to provide your own httpClient in order to use a cordova-http-client (with HTTPS pinning) for logout calls etc.

OP kicked off this thread with:

How can I achieve the same with angular-auth2-oidc and ionic cordova.?

In that I read a question for guidance around using this library within a specific ecosystem.
I'm happy to change the labels (they're just labels anyways) to a 'feature request', but then what is the specific feature being requested?
If it's about getting a specific tutorial for using the library within Ionic, I'm not sure if the community around this library is big enough to support that. It would possibly be written by someone, but then quickly go out of date. I much prefer then someone takes up the gauntlet of writing one, and we add it to the list of Tutorials.
In fact.... if you look at that list, there is already an external "Ionic" sample linked.
What do you think, does this make sense?

@jeroenheijmans Thanks for the quick reply. I am aware of that tutorial, but the solution used there (additionally using an okta plugin) works around many of the features that this lib (angular-oauth2-oidc) already provides, thus using this lib provides only little benefit implementing it like this. If you take a look at this chunk of code from the tutorial:
oktaLogin(): Promise<any> {
  return this.oauthService.createAndSaveNonce().then(nonce => {
    let state: string = Math.floor(Math.random() *<PHONE_NUMBER>).toString();
    if (window.crypto) {
      const array = new Uint32Array(1);
      window.crypto.getRandomValues(array);
      state = array.join().toString();
    }
    return new Promise((resolve, reject) => {
      const oauthUrl = this.buildOAuthUrl(state, nonce);
      const browser = window.cordova.InAppBrowser.open(oauthUrl, '_blank',
        'location=no,clearsessioncache=yes,clearcache=yes');
      browser.addEventListener('loadstart', (event) => {
        if ((event.url).indexOf('http://localhost:8100') === 0) {
          browser.removeEventListener('exit', () => {});
          browser.close();
          const responseParameters = ((event.url).split('#')[1]).split('&');
          const parsedResponse = {};
          for (let i = 0; i < responseParameters.length; i++) {
            parsedResponse[responseParameters[i].split('=')[0]] =
              responseParameters[i].split('=')[1];
          }
          const defaultError = 'Problem authenticating with Okta';
          if (parsedResponse['state'] !== state) {
            reject(defaultError);
          } else if (parsedResponse['access_token'] !== undefined &&
            parsedResponse['access_token'] !== null) {
            resolve(parsedResponse);
          } else {
            reject(defaultError);
          }
        }
      });
      browser.addEventListener('exit', function (event) {
        reject('The Okta sign in flow was canceled');
      });
    });
  });
}

all this response parameters parsing is something that you already have in the lib. It would be better if one could hook that window.cordova.InAppBrowser.open into the plugin itself.
Also, one should be able to provide a custom httpClient for HTTPS-pinning (I would see that in a separate feature).
So basically, I am asking for "real" Ionic support, not documentation or a tutorial. Is this something that this lib could provide in the future or do you not want to open that can of worms?

I'm starting to understand a little better.

I'm asking for "real" Ionic support...
Is this something that this lib could provide?

I think it's good to have interception points in the library, as long as they're not tightly coupled to another framework such as Ionic: we don't want to give all users of the library Ionic-specific features, I think?
I have not read through the tutorial, nor do I know much about Ionic. The code you posted does look off to me indeed, you shouldn't need that when using the library. But it could also be that you could do better than that specific tutorial? I just don't know.
What we really need to know what the actual, concrete feature request is. Because "real Ionic support" is still too vague for me. And with that, a bit too hard to know if it'd be a good fit.
Would you have a concrete prototype or proposed change for the library, that would make it more useful in e.g. Ionic environments?

@jeroenheijmans This is what I had in mind, sounds good 👍
I will make two new feature-requests with detailed requirements

Thx @philly-vanilly!
Fair warning: the community at the library is small, and PRs sometimes move a bit slow. I moderate the issues a lot, but it's largely the core maintainer looking at a bunch of PRs all at once every few months. So you might not get fast or huge interaction (but the effort is still appreciated! <3).

I assume, with code flow, I would need to use customRedirectUri instead of customHashFragment?

Hi @philly-vanilly ,
thanks for your code snippet!
Did you fine a solution for code flow?

I've been looking for a similair solution. After lots of debugging and looking at the source code of this library I figured out what to do when using capacitor.
Our current setup: Angular 8
angular-oauth2-oidc: 10.0.3
capacitor: 2.4.2
Implicit flow was working fine at first, but the code flow+PKCE gave some issues which where caused by an outdated version of angular-oauth2-oidc, but was resolved in this PR
The little bit of code that was needed to get things working:
import {NgZone} from '@angular/core';
import {Plugins} from '@capacitor/core';
import {OAuthService} from 'angular-oauth2-oidc';
import {Router} from '@angular/router';

const {App, Browser, Device} = Plugins;

export class AppComponent {

  constructor(
    private zone: NgZone,
    private oauthService: OAuthService,
    private router: Router,
  ) {
    this.listenForOAuthRedirect();
  }

  private async listenForOAuthRedirect() {
    // example of the important stuff
    const oauthParameters = {
      responseType: 'code',
      // the IDP must redirect to the App URI (it should be whitelisted)
      redirectUri: 'custom-app-uri://localhost',
      // open a new browser window when starting implicit flow
      openUri: url => Browser.open({url})
    };

    this.oauthService.configure(oauthParameters);

    App.addListener('appUrlOpen', async data => {
      const platform = (await Device.getInfo()).platform as Platform;

      // because reasons
      if (platform === 'ios') {
        await Browser.close();
      }

      await this.zone.run(async () => {
        await this.oauthService.tryLogin({customHashFragment: data.url.substr(data.url.indexOf('?'))});

        const hasValidAccessToken = await this.oauthService.hasValidAccessToken();

        if(hasValidAccessToken){
          await this.router.navigate(['/your-protected-route-after-login']);
        }
      });
    });
  }

}

@mrmkrs thank you for that snippet, you saved me from hours (if not days) of hassle !

Hi,
I'm already using the auth lib for a regular Angular project, and I tried to adapt my code for an Ionic version of the app.

Angular 13
angular-oauth2-oidc: 13.0.1
Ionic 6.1.9

I tried the above sample from @mrmkrs  for Ionic but I don't get the App.addListener('appUrlOpen', () => ...) triggered.
What is supposed to trigger the appUrlOpen event? Because obviously we must get inside to execute this.oauthService.tryLogin() method. Maybe I should mention I'm very new to Ionic :).
I don't get neither this part:
// the IDP must redirect to the App URI (it should be whitelisted)
 redirectUri: 'custom-app-uri://localhost',

How should I setup the app URI in Ionic? And by whitelisted I guess you mean the server side config (keycloak) to allow the redirection to the url.
For comparison, here is my (working) code for the web app:
// Called by an APP_INITIALIZER
initialize(): Observable<boolean> {
    const authSubject = new Subject<boolean>();
    this.oauthService.configure(environment.config.authConfig);
    this.oauthService.loadDiscoveryDocumentAndLogin().then(authenticated => {
      if (authenticated) {
        this.oauthService.setupAutomaticSilentRefresh();
        // Perform login in App state
        this.login();
      } else {
        this.oauthService.initImplicitFlow();
      }
      authSubject.next(authenticated);
      authSubject.complete();
    });
    return authSubject.asObservable();
  }

I would really appreciate some help here. Thanks a lot.


Modify Document (pdf-lib) Fiddle doesn't work with other PDFs

How come if I replace the PDF URL (https://pdf-lib.js.org/assets/with_update_sections.pdf) in this Fiddle it with the same PDF, but a different URL ( (https://github.com/Hopding/pdf-lib/blob/master/assets/pdfs/with_update_sections.pdf)) it doesn't work?

Because https://github.com/Hopding/pdf-lib/blob/master/assets/pdfs/with_update_sections.pdf doesn't link to a PDF, it links to an HTML/JS page that renders a PDF preview. You have to provide a link to a raw PDF file, like so: https://github.com/Hopding/pdf-lib/raw/master/assets/pdfs/with_update_sections.pdf.


use travis ci

currently a non-public camunda server is used for building. In case of error, non-camunda-members do not have access.. we should move at least the ci and nightly build to travis so every member of this github org has access.

@hawky-4s-   I need admin rights on this repo to activate travis  .... could you a) give me these rights (seems I have them already for example on camunda-bpm-reactor) or b) enable travis for this repo?

Activated TravisCI

In the future a Community Jenkins will be publicly available with GitHub Pull Request building, notification etc.

Behold: https://travis-ci.org/camunda/camunda-bpm-spring-boot-starter


Errors in cosmos Broadcaster are sometimes are not propagated to caller

When working on a test, I noticed the following behaviour when calling BroadcastTx.
The response returned here did not error, but included an error message in the RawLogs of the response.
The error in question I believe happened in the ValidateBasic of a message that was broadcast.
I think in the case of RawLogs being populated on the linked line, we can return an error with the contents (even though the actual err returned is nil ).
The happy path here is that all fields except for txHash should be zero values.
cc @crodriguezvega

ref: https://github.com/strangelove-ventures/interchaintest/pull/842


Support for Codex

I was wondering whether this has support for Codex.

Sure, all you need to choose codex models, like code-davinci-002 or code-cushman-001.

could you please give a demo, i do not know how to code it


Incompatibility with IntelliJ 2020.3+

Plugin 'logstash config support' (version 'null') is not compatible with the current version of the IDE, because it requires build 202.* or older but the current build is IU-203.5981.155

Hi,
I have noticed that The current build is failing since Dec 5. I Have looked into it and saw that the Class LogstashLanguage is loaded from two different locations in different tests:

build/classes/java/main-instrumented/com/github/redfoos/logstash/LogstashLanguage.class
idea-sandbox/idea-2020.3/plugins-test/logstash-intellij-plugin/lib/logstash-intellij-plugin-0.62-2020.3.jar!/com/github/redfoos/logstash/LogstashLanguage.class

Whichever load is the second, triggers an exception in IntelliJs Language class, since they check whether a Language had been loaded before.
In the tagged version releasebuild_0.62 only the class from the build directory is loaded.
Hope this helps.

Hi,
I have noticed that The current build is failing since Dec 5. I Have looked into it and saw that the Class LogstashLanguage is loaded from two different locations in different tests:

build/classes/java/main-instrumented/com/github/redfoos/logstash/LogstashLanguage.class
idea-sandbox/idea-2020.3/plugins-test/logstash-intellij-plugin/lib/logstash-intellij-plugin-0.62-2020.3.jar!/com/github/redfoos/logstash/LogstashLanguage.class

Whichever load is the second, triggers an exception in IntelliJs Language class, since they check whether a Language had been loaded before.
In the tagged version releasebuild_0.62 only the class from the build directory is loaded.
Hope this helps.

Here is what I did:

I set a breakpoint at the Constructor of com.github.redfoos.logstash.LogstashLanguage
I added Watches with the following expressions:

com.github.redfoos.logstash.LogstashLanguage.class.getClassLoader()

Found different classloaders are used when the breakpoint is hit during execution of LogstashLexerTest and CompletionTest

com.github.redfoos.logstash.LogstashLanguage.class.getClassLoader().getResource("com/github/redfoos/logstash/LogstashLanguage.class").toString()

Different Locations as mentioned before.

I looked at the Stacktrace when the Breakpoint hit.

For the LogstashLexerTest the instantiation of LogstashLanguage happened during the test execution
For the CompletionTest the instantiation of LogstashLanguage happened outside (i assume before) the actual execution of any test method.

Unfortunately I'm not familiar with JUnit3 nor the IntelliJ test framework. I hope that this still helps towards a quick fix.

Here is what I did:

I set a breakpoint at the Constructor of com.github.redfoos.logstash.LogstashLanguage
I added Watches with the following expressions:

com.github.redfoos.logstash.LogstashLanguage.class.getClassLoader()

Found different classloaders are used when the breakpoint is hit during execution of LogstashLexerTest and CompletionTest

com.github.redfoos.logstash.LogstashLanguage.class.getClassLoader().getResource("com/github/redfoos/logstash/LogstashLanguage.class").toString()

Different Locations as mentioned before.

I looked at the Stacktrace when the Breakpoint hit.

For the LogstashLexerTest the instantiation of LogstashLanguage happened during the test execution
For the CompletionTest the instantiation of LogstashLanguage happened outside (i assume before) the actual execution of any test method.

Unfortunately I'm not familiar with JUnit3 nor the IntelliJ test framework. I hope that this still helps towards a quick fix.

I also meet the proplem

I also meet the proplem

Same here

Same here

Hi @redfoos
I tried transforming the plugin to the documented directory-structure from https://plugins.jetbrains.com/docs/intellij/custom-language-support-tutorial.html (moving src/com to src/main/java, tests to src/test/java etc. and fixing paths in build.gradle and in the tests), The issue then still persists.
Do you think this could be a problem with the test-framework?
I searched the IntelliJ issue tracker for issues with the test framework but found nothing that looked as if it could be the reason for this problem. Would it be in your interest to report a bug there and see what happens?

@fibsifan, Hello! Thank you for your work, could you please send PR?
As for the tests, I cannot be sure that this is a problem with the test framework, but, unfortunately, I now have absolutely no time to deal with this, since I am in the process of defending my diploma, it is possible that afterwards I will definitely figure it out. If you have an idea of what you can write to the bug tracker and it's not difficult for you, then I'll be glad!

For the switch to gradle default layout see #18
Regarding the issue with the tests I should probably create an example project to try to reprotuce the error before reporting. Maybe I'll find some time on one of the upcoming weekends and try it.

Guys, it's completely unclear for me why all tests does not work together. It is look like some kind of 'pollution' tests context and I've spent around a day to understand how to fix and I failed.
Now for create plugin release I disabled parser and lexer tests.


DialogFieldTagControl - don't add  for multiselects

when selecting one option from a list, having <None> (or <Choose>) in the list makes sense
when selecting 0 or more options from a multiselect, neither <None> or <Choose> can be in the actual list of options.
This matches what https://github.com/ManageIQ/manageiq/pull/19148 did for DialogTagDropDownList,
(including introducing the multiselect? method).
(Previously, the options were added to tags in https://github.com/ManageIQ/manageiq/pull/14348)
Cc @eclarizio - can you review please?
Cc @skateman,
Fixes https://github.com/ManageIQ/manageiq-ui-classic/issues/6393
and partly addresses the issue in https://github.com/ManageIQ/manageiq-ui-classic/issues/6392
@miq-bot add_label bug, automate

We need to backport this to ivanchuk for https://bugzilla.redhat.com/show_bug.cgi?id=1834219
@miq-bot add_label ivanchuk/yes

Ivanchuk backport details:
$ git log -1
commit 2bfe6d2aea2a2f2716e744ba847532a7261ee021
Author: Brandon Dunne<EMAIL_ADDRESS>Date:   Mon Jan 13 14:24:08 2020 -0500

    Merge pull request #19696 from himdel/dialog-tag-multi

    DialogFieldTagControl - don't add <None> for multiselects

    (cherry picked from commit 9ce9b3df7565a7fe4ad64494e348841f4ada98e0)

    https://bugzilla.redhat.com/show_bug.cgi?id=1834219


Supporting Atom

Hi @fisker ,
Logging process.env in Atom's console shows only one property with "ATOM" in it, namely ATOM_HOME pointing to the .atom folder in one's user directory.
I confirmed that this should work (conditionally throwing when process.env.ATOM_HOME is present, noticing that when running from CLI, it worked, while within Atom, it did not).
There are official docs about it at https://flight-manual.atom.io/using-atom/sections/basic-customization/#custom-home-location-with-an-environment-variable .
Not inclined to make a PR at this time, but offering in case you wanted to use it, as I was curious whether this could be made to work with Atom.

Thanks, I don't use Atom, but PR welcome.
This package not ready yet, I need a babel transform before I can use it in my .eslintrc.js 😄

By the way, let me know when the package is functional so we can add to awesome-eslint.

I think it's already functional, I won't do anything related to ESLint here.

Fixed by #2

Have got something in mind that can use this?
Here is mine:

Disable some rule in my personal config
Use different fix logic for eslint-plugin-prettier

Added to https://github.com/dustinspecker/awesome-eslint#tools . Thanks!

Have you got something in mind that can use this?

I can't recall at the moment, but I've run into issues where Atom wasn't operating in the same way, causing some false positive IIRC. Might have been an import resolver issue, but can't really recall for sure right now.

This wasn't the one I was thinking of, but here's one issue I found looking at one project's config.
My config sets the parserOptions.ecmaVersion dynamically based on the minimum version in engines: https://github.com/brettz9/eslint-config-ash-nazg/blob/master/node.js#L3-L18
However, my detection script which uses process.cwd, gets a bad cwd apparently from within Atom.
If I'm going to have to set it manually, I guess I might just drop the dynamic setting approach, but was hoping to find a workaround. :-)


How is CA Policy for Service Principals deemed to be a separately licensed product?

Having been involved with the preview of CA Policy For 'workload identities' for a very long time, for me it's sad that Microsoft has elected to make this a separately licensed feature somehow bundled up with the "Entra" product and not included with their existing entitlement to use AAD Conditional Access.
Customers who already have top-end Azure AD licensing, and who fully expect that to include all aspects of Identity Protection and Conditional Access, and who may simply wish to implement CA policy for the "Service Account" context of Azure AD Service Principals (e.g. protect Service Principals aka 'service accounts' used in ADO Service Connections, for automation etc), as opposed to multi-tier Azure AD integrated custom application flows, now need to buy yet another Azure AD license.
Entra is a permissions management product. Please explain why is Microsoft moving aspects of Conditional Access into this product, and how does "Conditional Access for Workload Identities" constitute a different product, as opposed to simply an incremental feature uplift on Conditional Access where customers are already paying for that?

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

ID: 60d23869-94f5-22cc-425e-4bf675d84ce0
Version Independent ID: 5105047f-04bf-87ef-ee8e-3e9bf9b5ac3a
Content: Azure Active Directory Conditional Access for workload identities - Microsoft Entra
Content Source: articles/active-directory/conditional-access/workload-identity.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@kahawai-sre
Thanks for your feedback! We will investigate and update as appropriate.

@kahawai-sre
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly
@MicrosoftGuyJFlo
Can you please check and add your comments on this doc update request as applicable.

#label:"awaiting-product-team-response"

More about Entra can be found on the following site https://www.microsoft.com/security/business/microsoft-entra
"Microsoft Entra a family of products that encompasses all identity and access capabilities. Within the Entra family are products such as Microsoft Azure Active Directory (Azure AD), Microsoft Entra Verified ID, and Microsoft Entra Permissions Management."
#please-close


Add option to temporarily disconnect/disable a smart card to allow mapping into Crostini

See more details on the Chromium group and the Crbug that actually appears to be partially by design behavior from the connector.
https://groups.google.com/a/chromium.org/d/msg/chromium-os-dev/7lvuQOVJbBo/JI0YnV7QAQAJ
https://bugs.chromium.org/p/chromium/issues/detail?id=1030778#c_ts1583002164

+1 this would solve a number of problems.  It would be even better if it wouldn't take exclusive use, if possible.  That would mean you wouldn't have to go frob it off, then frob it back on later.

Changing the exclusive use would be pretty hard, both from the implementation perspective (involving significant rework of the CCID free software driver) and also from the practical usefulness (since a smart card middleware like CSSI will anyway keep a constantly opened connection to the reader/token in order to notify Chrome about changes).
Tackling this problem in a correct way might be related to the ongoing effort of converging the smart card stack on Chrome OS, like exposing it to PWAs. That'll be a long road though, especially for Crostini where smart cards haven't been prioritized yet.
So I'm afraid the manual disconnection would have to be our short- and middle-term solution.

The annoying thing here is that the ONLY place this doesn't work is in ChromeOS.  It works with chrome browser on any other supported OS (BSD, Linux, Windows, and MacOS).  For a company that touts 2-factor, physical tokens, and FIDO - it's a bit annoying that their OS doesn't actually allow this.
As much as I would like to continue to use ChromeOS/PixelBooks as a daily portable driver, I've been forced back to a Linux laptop to be able to access gitlab/github, my passwords (gopass), sign releases, etc.  Basically, if you've bought into 2-factor GPG/SSH/FIDO, you have no realistic way of using ChromeOS unless EVERYTHING you do can be done via Chrome Apps/Browser.

The thing that gives me hope is that smart card sharing between multiple applications in a host/guest scenario is something that has been solved by VMware and Parallels and possibly Virtualbox already, so hopefully the incredibly clever folks at Google can work this out, especially with the decline of NaCl applications and the switch to WebASM for a lot of things.

For the sharing smart card support between Crostini and the host OS, please file an issue into the Chromium tracker (https://crbug.com), which in particular tracks the Crostini-related tasks. This would need to be a separate effort, since there's no trivial way how this would work (as you cannot access the same USB device from two OS'es simultaneously, and since the standard PC/SC-Lite daemon always keeps an open connection to the reader).

FWIW, smart card access in Crotstini is now working for me on 92 beta, if I disable the Smart Card Connector app in chrome://extensions/ using the toggle and if I go to settings and attach the USB token to the Linux container under Settings -> Developers -> Linux development environment -> Manage USB devices (or using vmc in crosh).
It would still be nice if the Smart Card connector app could automagically release the smart card or even allow access / attachment to Crostini instead, tho.


Added bids config file for bids-validator

Chris,
I have used the antsCorticalThickness BIDS app successfully on several of my data sets.  Each time I have created a specific BIDS directory with just the T1w images so it passes the  bids-validator.
As you know, the code checks that that BIDS directory is valid before proceeding with running antsCorticalThickness.  This works well if your BIDS directory is compliant without a configuration file. Unfortunately, for me this is normally not the case.  I use the configuration file to avoid certain issues.
For example, when doing the heudiconv conversion the hidden directory .heudiconv is written to the BIDS directory.  This directory is not BIDS compliant and the bids-validator will fail if the directory remains.    I also like to leave README files within the BIDS directory to explain issues with the data to our group.  These README files are not BIDS compliant and the bids-validator will fail. I also like to keep the BIDS validator configuration file in the BIDS directory so people can see what is being ignored during the validation.  This file is not BIDS compliant either.
I deal with the above examples by specifying to ignore these files and directories during the BIDS validation. This works great until I try to run antsCorticalThickness BIDS APP because it doesn't use the BIDS configuration file when performing the BIDS validation.
I would like to recommend that either the bids-validator check is removed from the BIDS apps or the bids-validator configuration file is passed in through argparse.
I  forked this repository and I made the suggested changes. I was able to some simple testing.  However, I am relatively new to python and I am not sure if the subprocess handles the string I am passing it to correctly.
Please consider these simple changes. I believe that it will allow more people to use this BIDS app.
Thanks
Bob

If you only need to ignore files it would be much simpler to update the validator in this BIDS App and include https://github.com/INCF/bids-validator#bidsignore files in your input datasets.

BTW what makes your README files not compliant? Perhaps this is an issue similar to https://neurostars.org/t/how-to-upload-eeg-data/1250/10 ?

I didn't know about .bidsignore.  I have been using the bids configuration file instead.  Using .bidsignore would be easier.  I just can't seem to get it to work.  Here is the contents of my BIDS directory
aging1a ➜  bids ls -a
.              .bidsignore~              participants.tsv  sub-imove1061                  task-rest_acq-pcasl_bold.json
..             CHANGES                   README            sub-imove1071
acrostic.list  dataset_description.json  sub-imove1011     sub-imove9001
.bids.cfg      .heudiconv                sub-imove1031     task-rest_acq-epi_bold.json
.bidsignore    log_file                  sub-imove1051     task-rest_acq-mbepi_bold.json

aging1a ➜  bids cat .bidsignore
.*
!/.heudiconv

The syntax of the .bidsignore file is based upon your comments about similar content as .gitignore, which I found here https://stackoverflow.com/questions/8021441/git-how-to-ignore-hidden-directories

aging1a ➜  bids bids-validator $PWD
        ... (missing text)

	1: This file is not part of the BIDS specification, make sure it isn't included in the dataset by accident. Data derivatives (processed data) should be placed in /derivatives folder. (code: 1 - NOT_INCLUDED)
		./.bids.cfg
			Evidence: .bids.cfg
		./.bidsignore
			Evidence: .bidsignore
		./.bidsignore~
			Evidence: .bidsignore~
		./.heudiconv/imove1011/ses-1/info/dicominfo_ses-1.csv
			Evidence: dicominfo_ses-1.csv
		./.heudiconv/imove1011/ses-1/info/dicominfo_ses-1.tsv
			Evidence: dicominfo_ses-1.tsv
		./.heudiconv/imove1011/ses-1/info/filegroup_ses-1.json
			Evidence: filegroup_ses-1.json

       ... (missing text)

        Summary:                 Available Tasks:                     Available Modalities:
        650 Files, 5.48GB        rest                                 T1w
        6 - Subjects             TODO: full task name for rest        T2w
        1 - Session                                                   dwi
                                                                      sbref
                                                                      magnitude1
                                                                      phasediff
                                                                      bold
                                                                      fieldmap

Can you please tell me what I am doing wrong?
Thanks for your help

Are you using the latest version of the validation?
Best,
Chris
On Feb 22, 2018 6:14 AM, "bkraft4257"<EMAIL_ADDRESS>wrote:

I didn't know about .bidsignore. I have been using the bids configuration
file instead. Using .bidsignore would be easier. I just can't seem to get
it to work. Here is the contents of my BIDS directory
aging1a ➜  bids ls -a
.              .bidsignore~              participants.tsv  sub-imove1061                  task-rest_acq-pcasl_bold.json
..             CHANGES                   README            sub-imove1071
acrostic.list  dataset_description.json  sub-imove1011     sub-imove9001
.bids.cfg      .heudiconv                sub-imove1031     task-rest_acq-epi_bold.json
.bidsignore    log_file                  sub-imove1051     task-rest_acq-mbepi_bold.json
aging1a ➜  bids cat .bidsignore
.*
!/.heudiconv
The syntax of the .bidsignore file is based upon your comments about similar content as .gitignore, which I found here https://stackoverflow.com/questions/8021441/git-how-to-ignore-hidden-directories
aging1a ➜  bids bids-validator $PWD
... (missing text)
1: This file is not part of the BIDS specification, make sure it isn't included in the dataset by accident. Data derivatives (processed data) should be placed in /derivatives folder. (code: 1 - NOT_INCLUDED)
./.bids.cfg
Evidence: .bids.cfg
./.bidsignore
Evidence: .bidsignore
./.bidsignore~
Evidence: .bidsignore~
./.heudiconv/imove1011/ses-1/info/dicominfo_ses-1.csv
Evidence: dicominfo_ses-1.csv
./.heudiconv/imove1011/ses-1/info/dicominfo_ses-1.tsv
Evidence: dicominfo_ses-1.tsv
./.heudiconv/imove1011/ses-1/info/filegroup_ses-1.json
Evidence: filegroup_ses-1.json
   ... (missing text)

    Summary:                 Available Tasks:                     Available Modalities:
    650 Files, 5.48GB        rest                                 T1w
    6 - Subjects             TODO: full task name for rest        T2w
    1 - Session                                                   dwi
                                                                  sbref
                                                                  magnitude1
                                                                  phasediff
                                                                  bold
                                                                  fieldmap

Can you please tell me what I am doing wrong?
Thanks for your help
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/BIDS-Apps/antsCorticalThickness/pull/2#issuecomment-367693254,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOkpztPkn5pfrD5WC8t21r7Bg_XAdR3ks5tXXZhgaJpZM4SOMsW
.

I am running
bids-validator -v
0.23.1
Unfortunately, I couldn't find the current version of the bids-validator on the GitHub page.

It's on npm - https://docs.npmjs.com/getting-started/updating-global-packages
npm update -g bids-validator


QUESTION: Update design to ESP32 to reduce cost, and increase performance?

Hi, since the ESP32's are so cheap now, I think updating the design to that would improve speed and price. Whats your idea on that?

This repository hasn't had a code update since 2018, and the creator hasn't made a commit on GitHub in a year, so it seems effectively dead. I just started (as in within the last few days) working on my own project that aims to do essentially the same thing as this project, but based on an ESP32-C2 module with ESP-IDF instead of Arduino. I can post a link if/when I make it public.


debug breakpoint for interactive window not working

Environment data

VS Code version: 1.29.0
Extension version (available under the Extensions sidebar): 2019.1.0
OS and version: linux ubuntu 18.04
Python version (& distribution if applicable, e.g. Anaconda): python 3.5.2
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): virtualenv
Relevant/affected Python packages and their versions: all

Expected behavior
when placing new debug breakpoint in GUI and running in interactive window, stop at debug point.
Actual behavior
not stopping at debug breakpoint
Steps to reproduce:

put debug breakpoint in code
run in interactive mode

This is actually not implemented yet. There's another issue tracking it:
https://github.com/microsoft/vscode-python/issues/3876


Investigate adding openOCD and GDB to docker build

Closing this since we do not want to spend the time working on integrating a windows -> linux usb setup. We may revisit if we move to a linux os for dev


Update readme file

Create instructions: how to build docker image.
These are my personal notes:
------------- VIRKER ------------------
docker run -d -v ${pwd}/../uploads:/opt/youtransfer/uploads -v ${pwd}/../config:/opt/youtransfer/config -p 80:5000 remie/youtransfer:stable

docker kill //To shut down the container

--------------------------------------

# You can specify a repository and tag at which to save the new image if the build succeeds:
# docker build -t gdpr -f Dockerfile .
# docker build -t imagename .

#husk at kigge i -dockerignore - der ligger nemlig node_modules

#
# RUN mkdir /opt/youtransfer/config ... RUN /opt/youtransfer/ osv...
# WORKDIR /opt/youtransfer/
# COPY . /opt/youtransfer/

# RUN update or??

This issue was moved to prolike/drop-gdpr#5


Transition-group is not working in render function

Versions

nuxt: 2.15.8
node: 14.17.6

Dependencies
"dependencies": {
    "core-js": "^3.19.3",
    "nuxt": "^2.15.8",
    "vue": "^2.6.14",
    "vue-server-renderer": "^2.6.14",
    "vue-template-compiler": "^2.6.14",
    "vuex-persistedstate": "^4.1.0",
    "webpack": "^4.46.0"
  },
  "devDependencies": {
    "@fortawesome/fontawesome-svg-core": "^1.3.0",
    "@fortawesome/free-brands-svg-icons": "^6.0.0",
    "@fortawesome/free-solid-svg-icons": "^6.0.0",
    "@nuxt/postcss8": "^1.1.3",
    "@nuxtjs/fontawesome": "^1.1.2",
    "@nuxtjs/tailwindcss": "^5.0.0-4",
    "autoprefixer": "^10.4.2",
    "postcss": "^8.4.7",
    "tailwindcss": "^3.0.23"
  }

Reproduction
I found that transition-group is not working in render function. I use nuxt/tailwind in this simple demo.
components/Faded.vue

index.vue
<template>

    <button class="bg-green-100" @click="AddIndex">Button</button>
    <Faded>

0

1

2

    </Faded>

</template>

This is replicable without Nuxt. Would you raise an issue to https://github.com/vuejs/vue ? 🙏


[Snyk] Security upgrade nginx from 1.21 to 1.23.2

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

web-server/Dockerfile

We recommend upgrading to nginx:1.23.2, as this image has only 96 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

714
Double Free SNYK-DEBIAN11-CURL-1585150
No Known Exploit

714
Incorrect Default Permissions SNYK-DEBIAN11-CURL-2936229
No Known Exploit

614
XML External Entity (XXE) Injection SNYK-DEBIAN11-LIBXML2-3059797
No Known Exploit

614
Integer Overflow or Wraparound SNYK-DEBIAN11-LIBXML2-3059801
No Known Exploit

714
OS Command Injection SNYK-DEBIAN11-OPENSSL-2933518
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


services/horizon/expingest: StateVerifier should react to shutdown

StateVerifier should check for shutdown signal and if StateVerifier is running expingest.System.Run should exit after StateVerifier graceful shutdown.
Motivation: State verifier in Horizon uses the shared DB connection pool, if go routine is not shutdown gracefully it's possible that it will emit errors because Horizon closes DB connection pool.

Fixed in #1978.


A new setting meant to ignore ACTIVATE_VERSION messages entirely

Feature scope
Targets (data type handling, batching, SQL object generation, tests, etc.)
Description
I just wanted to post an issue that I had with Meltano that I still cannot figure it out. It is about the add_record_metadata configuration.
Basically when I set add_record_metadata=True, I do see all the following columns showing in this link (https://sdk.meltano.com/en/latest/implementation/record_metadata.html) appear. However when I set it to False, I still see the column, _sdc_table_version in my output which is a mssql table. I started to think whether this is a bug. I Just want to ask how I can remove this column from my output.
Here is my meltano.yml:
version: 1
default_environment: dev
project_id: 99290a45-4a7e-42ae-8208-0b2ca725d652
environments:
- name: dev
  config:
    plugins:
      extractors:
      - name: tap-mssql
        config:
          database: LiftStaging
          host: localhost
          port: '1434'
          user: sa
        select:
        - crm-contact_extract.id
        - crm-contact_extract.country_code
      loaders:
      - name: target-mssql
        config:
          database: LiftStaging
          default_target_schema: crm
          host: localhost
          port: '1434'
          username: sa
          add_record_metadata: false
- name: staging
- name: prod
plugins:
  extractors:
  - name: tap-mssql
    variant: wintersrd
    pip_url: tap-mssql
  loaders:
  - name: target-mssql
    variant: meltanolabs
    pip_url: git+https://github.com/MeltanoLabs/target-mssql.git
  mappers:
  - name: meltano-map-transformer
    variant: meltano
    pip_url: git+https://github.com/MeltanoLabs/meltano-map-transform.git
    mappings:
    - name: table-map
      config:
        stream_maps:
          crm-contact_extract:
            __alias__: contact_loaded (edited)

We tried  meltano invoke tap-mssql > output.json and the output looks like this:
{"type":"STATE","value":{"currently_syncing":"crm-contact"}}
{"type":"SCHEMA","stream":"crm-contact","schema":{"properties":{"country_code":{"inclusion":"available","maxLength":2,"type":["null","string"]},"id":{"inclusion":"available","minimum":-2147483648,"maximum":2147483647,"type":["null","integer"]}},"type":"object"},"key_properties":[]}
{"type":"ACTIVATE_VERSION","stream":"crm-contact","version":1732695221437}
{"type":"RECORD","stream":"crm-contact","record":{"country_code":"US","id":1},"version":1732695221437,"time_extracted":"2024-11-27T08:13:41.445465Z"}
{"type":"RECORD","stream":"crm-contact","record":{"country_code":"US","id":2},"version":1732695221437,"time_extracted":"2024-11-27T08:13:41.445465Z"}
{"type":"RECORD","stream":"crm-contact","record":{"country_code":"US","id":3},"version":1732695221437,"time_extracted":"2024-11-27T08:13:41.445465Z"}
{"type":"RECORD","stream":"crm-contact","record":{"country_code":"US","id":4},"version":1732695221437,"time_extracted":"2024-11-27T08:13:41.445465Z"}
{"type":"RECORD","stream":"crm-contact","record":{"country_code":"US","id":5},"version":1732695221437,"time_extracted":"2024-11-27T08:13:41.445465Z"}
{"type":"STATE","value":{"currently_syncing":"crm-contact"}}
{"type":"ACTIVATE_VERSION","stream":"crm-contact","version":1732695221437}
{"type":"STATE","value":{"currently_syncing":"crm-contact","bookmarks":{"crm-contact":{"initial_full_table_complete":true}}}}
{"type":"STATE","value":{"currently_syncing":null,"bookmarks":{"crm-contact":{"initial_full_table_complete":true}}}}

So even if add_record_metadata is set to false, _sdc_table_version will still appear in the table and only its value becomes NULL because of "type":"ACTIVATE_VERSION",...
Also see: https://github.com/meltano/sdk/blob/5ed3be170bfd1f35b72efc40c493cd000bd00ed2/singer_sdk/sinks/sql.py#L379-L387

Here's what target-redshift does, which I think makes sense:
https://github.com/TicketSwap/target-redshift/blob/eb7a814012b70abd7f774b39d76f2c4801b86882/target_redshift/target.py#L44-L52

Thanks for logging this issue @jeff73511!


Jansson Generating Documentation

Hello,
Currently I am trying to build the Jansson documentation.
But even after installing the following library using pip: http://www.sphinx-doc.org/en/master/
The CMake build is unable to locate the binary, and its version after fiddling with FindSphinx.cmake.
How should I setup the environment for generating the documentation?
Currently the README.md points to http://sphinx.pocoo.org/.
Which doesn't looks correct.
Does anyone has any idea on setting the documentation build? Till then I have to disable documentation generation.
-Ameya Vikram Singh

Thanks @ploxiln for the hints.
I was close to fixing it but probably being too late into the night I could not see the regex expression.
I have created a pull-request with the fixes.
-Ameya Vikram Singh

I am not familar with git command to pick commit from your branch, so I have to create a new PR(#502 ) to fix regex expression.

Closing per #502 (unsure why github didn't close automatically).


Ln/twitch embed

In the create post component, it's not clear that you can add Twitch URLs.  We're running out of space in the placeholder text but it would be nice to communicate this somehow:


docs: add info ~/server/utils directory in ~/utils page

❓ Type of change

[x] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
I didn't know that /server/utils exist and assumed that the server uses /utils too. Added info alert to avoid such misconceptions.
📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Thank you ❤


feat(base): add embark-target face

Embark remaps vertico-current to embark-target so to make the selected candidate highlighting consistent it makes sense to align how Doom themes customises embark-target, too.
Before the patch:

After the patch:

(theme: doom-tokyo-night)

LGTM! Thanks for your help!


[FLINK-16560] Forward Configuration in PackagedProgramUtils#getPipelineFromProgram

Before, when using PackagedProgramUtils (for example in the standalone cluster entrypoint or the web ui) the Flink Configuration would not be applied to the execution environment.
This also adds a test that verifies that we forward configuration.
This is a version of #11607 for Flink 1.10. There are slight differences because the plan environments were changed. Also the tests are in a different place because in Flink 1.10 the dependencies between flink-clients and flink-streaming-java are reversed.
Brief change log

change OptimizerPlanEnvironment and StreamPlanEnvironment constructors to accept a Configuration and forward to the correct super constructor.

Verifying this change

see the newly added test

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: indirectly, because we now correctly forward also deployment options
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no

Merged.


code-server --link on Termux fails

OS/Web Information

Web Browser: Latest stable Opera Desktop
Local OS: Windows 10 64-bit
Remote OS: Android 10
Remote Architecture: aarch64
code-server --version: 3.11.1 acb5de66b71a3f8f1cc065df4633ecd3f9788d46

Steps to Reproduce

Get code-server installed on Termux by following this docs: https://github.com/cdr/code-server/blob/main/docs/termux.md
code-server is installed and it works fine without --link, I managed to run it on same local area desktop web browser via !P address.
Stop the program, then rerun it with --link: code-server, the following error shown bellow:

~ $ code-server --link
[2021-08-21T13:29:23.818Z] info  code-server 3.11.1 acb5de66b71a3f8f1cc065df4633ecd3f9788d46
[2021-08-21T13:29:23.828Z] info  Using user-data-dir ~/.local/share/code-server
[2021-08-21T13:29:23.917Z] info  Using config file ~/.config/code-server/config.yaml
[2021-08-21T13:29:23.917Z] info  HTTP server listening on http://<IP_ADDRESS>:38223 (randomized by --link)
[2021-08-21T13:29:23.918Z] info    - Authentication is disabled (disabled by --link)
[2021-08-21T13:29:23.918Z] info    - Not serving HTTPS (disabled by --link)
[2021-08-21T13:29:24.009Z] info  ping server: dial server: failed to WebSocket dial: failed to send handshake request: Get "https://cloud.coder.com/latency": dial
tcp: lookup cloud.coder.com on [::1]:53: read udp [::1]:55865->[::1]:53: read: connection refused
[2021-08-21T13:29:24.028Z] error --link agent exited with 1

Expected
It should works normally.
Notes
I tested it on Google Cloud Shell (it's Debian), get installed via install script and get it running normally. Screenshot is below:

I did the similar procedure, but via Termxu's proot-distro, on Alphine distro with no --isolated flag.
[2021-08-21T14:04:52.275Z] info  code-server 3.11.1 acb5de66b71a3f8f1cc065df4633ecd3f9788d46
[2021-08-21T14:04:52.283Z] info  Using user-data-dir ~/.local/share/code-server
[2021-08-21T14:04:52.395Z] info  Using config file ~/.config/code-server/config.yaml
[2021-08-21T14:04:52.396Z] info  HTTP server listening on http://<IP_ADDRESS>:46389 (randomized by --link)
[2021-08-21T14:04:52.396Z] info    - Authentication is disabled (disabled by --link)
[2021-08-21T14:04:52.397Z] info    - Not serving HTTPS (disabled by --link)
[2021-08-21T14:04:56.148Z] info  Unfortunately we cannot ensure a good user experience with your connection latency (337ms). Efforts are underway to accommodate us
ers in most areas.
[2021-08-21T14:04:56.202Z] error --link agent exited with 1

It isn't isolated: it has access to host file system, included install Termux's code-server. I skipped the installation on Alpine and just get it running. It works without --link too. Same IP and same port, just different config (eg password; it's new).

Apparently, you should be able to get code-server --link run with termux-chroot from package proot (package resolv-conf may be needed too).
~ $ pwd
/data/data/com.termux/files/home
~ $ termux-chroot
~ $ pwd
/home
~ $ code-server --link
[2021-08-21T16:15:07.026Z] info  code-server 3.11.1 acb5de66b71a3f8f1cc065df4633ecd3f9788d46
[2021-08-21T16:15:07.035Z] info  Using user-data-dir ~/.local/share/code-server
[2021-08-21T16:15:07.136Z] info  Using config file ~/.config/code-server/config.yaml
[2021-08-21T16:15:07.136Z] info  HTTP server listening on http://<IP_ADDRESS>:39845 (randomized by --link)
[2021-08-21T16:15:07.138Z] info    - Authentication is disabled (disabled by --link)
[2021-08-21T16:15:07.139Z] info    - Not serving HTTPS (disabled by --link)
[2021-08-21T16:15:13.017Z] info  Proxying code-server, you can access your IDE at https://localhost-gphg.cdr.co

Please note that this code-server is running under proot and has different file system root.

@gphg Thanks for the detailed notes/investigation! I am guessing since you closed this, you were able to get it working?

@jsjoeio it's working. But if possible, I want to get it run without chroot. I'll have this issues reopen.
For unknown reason, I can't get any GitHub notification. I have no idea.


Enable which NPC's to hide in Entity Hider?

Is your feature request related to a problem? Please describe.
When doing some runecrafting at ZMI or some other activities there are some npcs that get in front of the things you need to click, making this an annoying situation. Can't disable all npcs because i need the banking one.
Describe the solution you'd like
An expansion to the entity hider plugin where you would be able to choose which NPC's to hide when typing their names.

Duplicate of #6854

Duplicate of #6854


SonarCloud CodeSmells

Describe the solution you'd like
Opening an issue to track SonarCloud code smells to fix. I am gonna send a comment next in this issue to let everyone know what code smells I will be working on. Please do the same if you are planning to grab some :)
What is the added value?
Fixing some code smells will enhance the overall code quality and health

Hi @Ed-vL,
I would like to work on some of those code smells.
Where do I find the code smell reports of SonarCloud?


compare mode of chart not working

When hovering a chart, it would be nice to see the same behaviour as in plot.ly is implemented. The hovermode compare is not working as expected, although it seems it is overriding the default settings of the layout.
"hovermode":"compare"
In the codepen this is working: https://codepen.io/anthonydickinson/pen/dmxOJy
Settings

Expected result or similar

Actual result

Hi Anthony, thanks for raising this. This currently removes the default tooltip in favour of the plotly version. If you're fine with this then you'll also need to change the hoverinfo setting on each of the series from none to one that suits you e.g. all, x+y, y. See the attached GIF.

Hey,
This is what we were looking for. Thx.


Remove 2017 venue page

@grobie @brian-brazil

Or leave it in place? Fine with removing it, but don't see the need.

@grobie we removed it from 2016 as well and generally always only kept those pages that made sense at a given time. Less mental overhead for people looking for things :)


[WIP][SS][MINOR] "keyWithIndexToNumValues" -> "keyWithIndexToValue"

What changes were proposed in this pull request?
This PR changes keyWithIndexToNumValues  to keyWithIndexToValue.
There will be directories on HDFS named with this keyWithIndexToNumValues. So if we ever want to fix this, let's fix it now.
How was this patch tested?
existing unit test cases.

Test build #82462 has started for PR 19435 at commit c62ba65.

Test build #82462 has finished for PR 19435 at commit c62ba65.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82462/
Test PASSed.

@tdas would you take a look, thanks

@tdas @zsxwing would you take a look, thanks

Thanks! Merging to master.


chore(master): release 0.1.13

:robot: I have created a release beep boop
0.1.13 (2022-08-26)
Features

add ingest support (22715eb)
add support for oauth2 client management (878d223)

Bug Fixes

fix dt import according to protobuf changes (f808e43)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/indykite/jarvis-sdk-node/releases/tag/v0.1.13 :sunflower:


refactor(ContractDefinitionStore): removed upsert semantic and introduced results on CUD operations

What this PR changes/adds
Refactor the ContractDefinitionStore as follows:

Removed upsert semantic
Introduced StoreResult for save, update and delete operations
Removed accept and save(Collections) APIs

Why it does that
to unify the store implementation, keep code paths clean and separate
Linked Issue(s)
Closes #2538
Checklist

[x] added appropriate tests?
[x] performed checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?
[x] assigned appropriate label? (exclude from changelog with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and Etiquette for pull requests for details)

Codecov Report
Patch coverage: 37.37% and project coverage change: -0.02 :warning:

Comparison is base (a13f68e) 64.83% compared to head (04ae235) 64.82%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2555      +/-   ##
==========================================
- Coverage   64.83%   64.82%   -0.02%
==========================================
  Files         872      876       +4
  Lines       17949    17990      +41
  Branches     1074     1078       +4
==========================================
+ Hits        11638    11662      +24
- Misses       5877     5893      +16
- Partials      434      435       +1

Impacted Files
Coverage Δ

.../org/eclipse/edc/azure/cosmos/CosmosDbApiImpl.java
0.00% <0.00%> (ø)

...tore/azure/cosmos/assetindex/CosmosAssetIndex.java
56.89% <0.00%> (ø)

...os/transferprocess/CosmosTransferProcessStore.java
0.00% <0.00%> (ø)

...contractdefinition/SqlContractDefinitionStore.java
0.00% <0.00%> (ø)

...tor/store/cosmos/CosmosDataPlaneInstanceStore.java
0.00% <0.00%> (ø)

...r/dataplane/store/cosmos/CosmosDataPlaneStore.java
0.00% <0.00%> (ø)

.../java/org/eclipse/edc/spi/result/StoreFailure.java
0.00% <0.00%> (ø)

...n/java/org/eclipse/edc/spi/result/StoreResult.java
0.00% <0.00%> (ø)

...tract/spi/offer/store/ContractDefinitionStore.java
0.00% <ø> (ø)

.../eclipse/edc/service/spi/result/ServiceResult.java
52.38% <41.66%> (ø)

... and 11 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Incorporate further ApiView feedback

This PR touches many files, but the volume comes primarily from bulk pattern application.
Changes include:

Per SCM guidance, nullable reference type support is added to the project and a first pass has been made across the public surface; this isn't 100% complete, particularly in the internal details, but it's the majority and allows a clear look at the pattern changes.
Name properties have been been updated to use a more specific name -- either FunctionName or ParticipantName -- in cases where the question of "what is this the name of" isn't trivially inferable from the type.
Public setters on request types have been removed, replaced with init for types where required/init are an intended initialization pattern.
ChatMessageContent's type conversions to string and (new) Uri are now explicit.
ChatRequestAssistantMessage now uses "text" for its content constructor parameter name.
CreatedAt is now present on GeneratedImageCollection (in addition to its emplacement on GeneratedImage).
ImageQuality and ImageStyle are renamed with the consistent Generated* prefix.
OpenAIClientOptions  now derives from the appropriate PipelineClientOptions type.
GetLegacyCompletionClient is removed from OpenAIClient.

Closing this PR in favor of the replacements mentioned above -- those pull apart the aims of this one better informed by the further discussion we've had.


make sure that toggle sidebar class works properly

This PR will:

make sure that toggle sidebar class works properly
fix issue when printing and sidebar-collapse still occupies the space
fix content-wrapper height when expanding treeview menu.

this pushes the footer in relation to sidebar's new height

Did you make sure that this works on small devices too? The class sidebar-open only affects small screens. We implemented it this way so the sidebar is open by default on large screens and collapsed by default on small screens.

Have not tested on actual small device but have tested it by resizing the browser and refreshing the page multiple times. Having these issues on my project and these patches fixes it.

And yes, sidebar is open for large screen and collapse on small screens.

I'll download your fork and test it on small devices. We appreciate the contribution!

I noticed a small issue. The sidebar on small screens doesn't open from the first click. I have to click twice to open the sidebar. After that, the toggle button works fine. Why do you think that is?

I think this is what we should do to make sure that it functions fine on both devices and fixes the print issues. In your commit at line 252 you added .addClass('sidebar-collapse'); which is what we need. However, we should keep the toggleClass function since there is a handler now for small devices that will make sure that sidebar-collapse is set in the body tag.

Let me know what you come up with. Currently I have some other work so I will attend this issue later. Thank you!

I think the problem was there was no initial body class of sidebar-open/collapse which causes the issue. Will work on this and let you know.

"The sidebar on small screens doesn't open from the first click" fixed on https://github.com/yajra/AdminLTE/commit/ecf2e4a5312e3f491456562b3d1ddd6df99b259c. Kindly check again. Thanks!

Hi again. I tested this and it works fine. However, I don't see why is this necessary. If you look at the CSS file, you will see that sidebar-open overrides sidebar-collapse on small screen automatically.
You mentioned earlier that this fixes printing issues, how does it do that? The CSS has a @media print queries that overrides any margining or padding in the layout so sidebar-collapse and sidebar-open don't take effect when printing.
We are still experiencing issues with printing. Especially from chrome, which for some reason is ignoring the print queries. I am still not sure what is causing this issue.

The problem was on cases when sidebar-collapse and sidebar-open are both added on the body at the same time due to lack of initial class.
For printing on V1 when sidebar is collapse, the whole content is printable while on V2, the sidebar still occupies the printing space.

I see the problem now. After a few tests, I noticed the sidebar hides only if the sidebar-collapse class is applied and the sidebar-open is not. The goal that we want to achieve is to have the printing work weather these classes are there or not. I think the way to do that is via CSS. I am currently making this my priority since the template promises an enhanced printing experience.

Hello again,
So sorry for the delay. Could you please create a pr for the second solution? I fixed some printing issues using CSS and now I think we can implement your code without affecting the normal behavior.
Thank you!

No problem. Will create a new PR for each issue later. Closing this now since it's not sync with master branch. Thanks!

thanks and sorry, quite busy atm and was not able to create the new PR!
seems like this were fixed on v2.0.3. :beers:

No problem, I added few things in the css so I decided to implement the fix myself. Thank you for the contribution!


[Bug]: Cell missing value when "*" is in the table widget's column name

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Bug:
When this particular string appears in table widget's column name, cell values will disappear
The string that causes the problem: "*" (double quote, star, double quote connected together)
use case :
The application that I'm developing is a table builder that lets the user build a table template which the user can define column names / how many rows should be in it. The users will define the column name so they made this column name: Answer"*"
how the fix will be useful for you:
Not have the cell value missing.
Additional information that I think might help:

[   {     "RowName": "Row\"*\"",     "Answer*\"\"": "(Editable Text)"   } ] This is the function's return value.
Steps To Reproduce
Have a table widget's data like this:
[   {     "RowName": "Row\"*\"",     "Answer*\"\"": "(Editable Text)"   } ]
Public Sample App
No response
Environment
Production
Issue video log
No response
Version
1.9.12

@BorenTsaiAMD you need to escape the quotes in the table by replacing " with "


breaks refs

Because of its heavy use of cloneWithProps, react-layout breaks refs of any UI element inside a Layout element

Ostensibly the use of cloneWithProps can be replaced with parent-child context on newer versions of React, but I haven't had time to do so yet.

Fixed by https://github.com/jsdf/react-layout/pull/10


Document breaking changes from render_options to optgroups

Django 1.11 support broke some custom widgets that used render_options() to render existing data into widgets. Would it be possible to document how such widgets should use the updated optgroups() method?

More specifically, I'm looking for a snippet that renders an ArrayField and also shows the existing data. By just using the example inlined in the documentation the existing tags never show up.

Here's what I have so far, it works for my purposes. Would love to hear feedback if it's good for the documentation:
class ArrayTagWidget(Select2TagWidget):

    def build_attrs(self, *args, **kwargs):
        self.attrs.setdefault('data-token-separators', [])
        self.attrs.setdefault('data-width', '500px')
        self.attrs.setdefault('data-tags', 'true')
        return super().build_attrs(*args, **kwargs)

    def value_from_datadict(self, data, files, name):
        values = super().value_from_datadict(data, files, name)
        return ','.join(values)

    def optgroups(self, name, value, attrs=None):
        values = value[0].split(',')
        selected = {*values}
        subgroup = [self.create_option(name, v, v, selected, i) for i, v in enumerate(values)]
        return [(None, subgroup, 0)]

@yuvadm thx, this looks good! There is never too much documentation.
The only feedback I have, is that I would make sure to focus on what to show here. If you want to show how to extend optgroups you should focus on that, not throw in the extra attrs.
Cheers! Looking forward to see your PR!
-Joe

@codingjoe great, thanks for the feedback! Will send a PR shortly.


update the Mac runtime to latest

as it says

complete.
marking this closed but I will release when it if time to release v0.6.8


CMake-js run-time environment update checks (feature request)

Hi,
Would be possible to turn off via CLI option the runtime environment update check that cmake-js does for every build?
It would make builds faster and be helpful for situations when there no connectivity.
Thanks!

Do you mean the check of the availability of runtime's headers and libraries? Buy without them the build will fail for sure, so that is a mandatory process.

Correct me if I am wrong, but I believe the dependency check could look at the local file system first for then, download all required packages from the internet if something is missing.

That should behave that way you've described. Maybe you've found a bug, haven't you?
Look, the flow is the following:
https://github.com/unbornchikken/cmake-js/blob/master/lib/es6/buildSystem.js#L32
https://github.com/unbornchikken/cmake-js/blob/master/lib/es6/dist.js#L89
https://github.com/unbornchikken/cmake-js/blob/master/lib/es6/dist.js#L46
So basically it's intended to check the file system first, then invokes the download if something is missing.

Hey,
I double checked this issue out and cmake-js is somehow working properly now.
Since then, we've upgraded our native module to make use of the latest nodejs runtime. Maybe that's why?
Thanks!

In old and new exactly what runtime versions do you mean?


Allow for escaping inputs by default

This adds a default_escape config setting which can be used to toggle whether or not escaping is enabled by default. This allows for a safer default behaviour and avoids the need to chain multiple |escape|typogrify filters everywhere.
I haven’t changed the current default, so escaping is still off unless you override the setting in your local config.

Thanks @domstubbs -- do you think I should document this?

Thanks. I suppose it would be worth a mention in the docs if this setting is likely to stick around.
I didn’t want to open a PR that was very heavy handed, but it seems to me that the most Craft-like approach would be for all tags to escape by default and perhaps to offer a separate, non-escaping |typogrifyHTML filter for use with rich-text fields. That would force people to make a conscious decision to say “let HTML through”, whereas with the current approach the onus is on them to remember to escape inputs manually, which is easy to forget even if you do read the docs.


Beta resource

I'm looking into using some beta-resources from Scaleway. But, when I tried to create eg. an IAM Group i could not find it in the Pulumi provider.
The reason seems to be that the Terraform provider will enable beta resources conditionally depending on an environment variable.
A make provider with the following modification to the Makefile

...

WORKING_DIR     := $(shell pwd)

# Enable beta resources.
export SCW_ENABLE_BETA := true

Produces output that seems to indicate the resources are now available.
What are the attitude towards having the Pulumi provider include beta resources? And is it simple enough that a PR would be accepted (I might need some pointers) or does it require more insight into how this project it set up?

I've implemented the modifications described above to a fork https://github.com/danquah/pulumi-scaleway/pull/1
I'd be happy to make an actual PR with those changes (would be nice with an upgrade to 2.5.0 before the PR as it has more of the IAM resources)

I don't intend to add beta resources to the GA builds of the provider. I'll consider adding the env var to prerelease versions

That fair, I totally get that you want to avoid blurring the lines between whats GA and whats not.
I'm curious. Do you know how other providers are handling things like this? Eg. lets say Scaleway had an official provider - I would expect they also wanted a clear separation between what is GA and whats not.
One way would be to have a whole extra type ala
import * as scaleway from "@pulumiverse/scaleway-beta";

Another approach would to have a runtime variable that prevented a specific resource to be used - but that would not keep you from using the resource in your programs which again would be unfortunate.
Do you know what other people are doing?

I personally haven't seen this pattern in any other bridged provider. In tf, you only have to generate a new binary, but for Pulumi you also have to generate the libraries too.
I think it'd be reasonable to fork this provider and call it "scaleway-beta"

Good point about the libraries. It makes things a bit more messy.
Re the fork - wonder if it would be possible to source both providers from the same repository 🤔
Don't know if you have any contacts at Scaleway, but I'll just try and reach out and ask the whether this beta-pattern is something they expect to use going forward. If this is a one-off thing for the IAM resources it does not make much sense to put any energy into it. But, if we're going to have a new set of beta resources again and again, it might merit some extra work.
In the end, if we're going to need the IAM resources on the project I'm currently working on I would probably have some time to put in to making this work. If IAM all of the sudden goes GA, my problem goes away - so lets see :)

slack link for reference (it's going to expire i 90 days)

A couple of quotes from Slack, we should of cause be aware that this is just a quick dialog and nothing binding:

(Me): Does anyone know if the the pattern where you specify  SCW_ENABLE_BETA to enable beta-resources is something that will be used going forward, or whether it is mostly just used for IAM for now?  ...
(Scaleway representative): It is mostly used for IAM, we are adding other beta products but we do not set them behind SCW_ENABLE_BETA variable. Iam is a specific case
(Me): OK - so it might actually not be worth the work to introduce anything special for SCW_ENABLE_BETA as things stands right now? I understand things might change in the future :)
(Scaleway representative): Yes, I don't think it would be worth the work, we want to avoid using SCW_ENABLE_BETA, it could be removed after iam GA.

So two takeaways:

No point in doing any more work re. SCW_ENABLE_BETA
That being said, the Terraform provider can at any point contain non-GA resources regardless of any flag like SCW_ENABLE_BETA. It might be a good idéa to mention this somewhere to the users of the provider. Ie. you should always verify with Scaleway whether the resources you use are GA or not.

I think this issue is pretty much resolved.

Version 2.8.0 of the terraform provider has now enabled IAM, so I guess we'll have it available with the next update of the Pulumi provider. With the feedback I got from their Slack channel it sound like they won't repeat this pattern for future features, so I'm closing this issue.


How to build only one function?

Plugin Version or Commit ID
v0.8.9
Unity Version
2020.3.30f1
Your Host OS
Windows 10
Target Platform
Windows Standalone
Description
hi~    Because the package built is too large, I only want to build the holistic function of the Android version on the winodws platform. How do I need to modify the build file?   Thanks~
Code to Reproduce the issue
No response
Additional Context
No response

duplicates #534


chore: single dev DB creds

Description
In #658, I removed alternative DB credentials from the server app.
This PR forces the DB settings to be the same in Skaffold as in localdev.
The main point of this is to make development a bit easier by always using the same credentials in all situations. Basically if you're developing on your machine the connection string will always be postgresql://backend:backend@${HOST}/backend_default, and ./manage.py shell won't fail because you're on the profile with the wrong DB creds.
There are a lot of different development set-ups for Substra, so I think it's good to avoid having too much variance.
I'd agree the names (backend:backend and backend_default) aren't necessarily the best, but honestly I just want them to be the same everywhere.
Checklist

[-] changelog was updated with notable changes (no user impact)
[-] documentation was updated (no documentation to update)

End to end tests: :hourglass_flowing_sand: STARTING


Azure EventHub client tool

Creation of a new tool for interacting with Azure EventHubs. Similar to the Kafka Client Tool, the new tool can produce and consume messages to an EventHub using connection credentials.

I'm confused about the CI error on appveyor:
You cannot call a method on a null-valued expression.
At line:1 char:1
+ $fileContent += $env:priv_key.Replace(' ', "`n")
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : InvokeMethodOnNull

Command executed with exception: You cannot call a method on a null-valued expression.

I'm confused about the CI error on appveyor:
You cannot call a method on a null-valued expression.
At line:1 char:1
+ $fileContent += $env:priv_key.Replace(' ', "`n")
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : InvokeMethodOnNull

Command executed with exception: You cannot call a method on a null-valued expression.

The build error seems to be due to secure variables not being available in PR branches. I'll merge this PR. Wonderful contribution!


docs: Fix Style and Image docs, fix CSS ref link

PR fulfills these requirements

[x] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[x] Docs have been added/updated (for bug fixes/features)
[x] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[x] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Describe the reason for change
Docs are broken because of wrong format:

<> should be escaped
examples should contain xml directly, all comments go into <!-- --> html comments

Does this PR introduce a breaking change?
(check only one)

[ ] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[x] No
[ ] Not sure (briefly explain the situation below)

Fix is already pushed to develop by CI action: https://github.com/heartexlabs/label-studio/commit/b9308150b855864c44e87de9bf8413f5147866c4


Implement metrics reader

Signed-off-by: Albert Teoh<EMAIL_ADDRESS>Which problem is this PR solving?

Relates to #2954

Short description of the changes

Implements GetLatencies, GetCallRates and GetErrorRates of the Prometheus MetricsReader.

Friendly nudge on this, if anyone could take a look please; I'd like to move forward on this feature.
I'm cognisant that this PR is on the larger side as I wanted to provide more context for the reviewer; however, if the preference is to make this smaller, I could break it up into smaller PRs too.
The failing unit test is caused by a known flaky test issue, which I'm looking at now: https://github.com/jaegertracing/jaeger/issues/3025

Thank you for the review, @jpkrohling!


[Tracking] State mismatch debugging.

[ ] https://github.com/ChainSafe/forest/issues/3109 - not only related to this, but mainly
[ ] Create a manual with debugging approaches:

[ ] how to compare stack traces in lotus and forest
[ ] how to validate exactly the epoch we need to debug on both lotus and forest
[ ] Attaching a debugger to lotus and forest (we already have a little info on notion: here

[ ] Introduce a way to isolate state mismatches (a minimal set of data that could be then used in a unit-test for lotus and forest)

We could begin to gather some of this info by just posting it here or on notion, how do you guys deal with debugging this? It would be very helpful to avoid reinventing the wheel every other week.

Also we can add how to extract a gas trace for a given message with both Lotus and Forest.

Also we can add how to extract a gas trace for a given message with both Lotus and Forest.

And I think this should probably be added as a feature to at least Forest. Something that can be enabled easily for debugging purposes. I'd like to move away from having to add debug statements and such for the common use-cases. I think is good to support proper instrumentation.
A good example of how that could be done is present in ParityDB codebase for the instrumentation feature.

@ruseinov Excellent initiative. I wanted to achieve something like this in the past but didn't manage to find the capacity to finish and publish it. Here are some ideas I collected and tricks I got from the Lotus rockstars. Link

Here's what I use to dump gas traces, thanks for the tips @elmattic .
debug_patch_forest.diff.txt

A submodule diff on the lotus side to do the same:
ffi-diff.txt
Note, that in order for tracing to be enabled you need to run lotus with LOTUS_VM_ENABLE_TRACING=1.


Run Compute shaders in the same binding context as vertex/fragment shaders

What problem does this solve or what need does it fill?
Executing a compute shader is possible, e.g. :  https://github.com/Kjolnyr/bevy_app_compute
However, in order to do so some separate calls are required, and a different binding context, so buffers can not be shared.
In complex calculations, e.g. raytracing or particle systems, compute shaders are an ideal way to execute code in a well defined massively parallel context.  However if the output of the compute shader is large, it creates a massive overhead to copy the data from GPU --> CPU and then back again just to display the rendered results.
What solution would you like?
Extend  the pipeline with dispatching the compute shaders in the same context, should there be a compute shader available, OR a flag is set to true in e.g. the Material trait.
What alternative(s) have you considered?

Copying data back and forth, which is slooow
Running the compute shaders parallell to the fragment shader, but data did not transfer between them because they ran in a different context.

Additional context
Running compute shaders and the fragment shade in the same context but with different dispatch calls is possible, see(mildly related): https://forum.babylonjs.com/t/data-transfer-from-compute-shader-to-renderer/45576

I'm not really sure what you mean by "context" here. You definitely can create a buffer, dispatch a compute shader to write some data to it, and then bind it in your material to read from during our mesh draws. You don't have to copy data back to the CPU, you can just bind the buffer you wrote to in the compute pass for your material.
Can you expand more on what you're trying to do, and what issues you're running into?

Sorry openGL stuck on me... I meant pipelines here.
Specifically what I would like to do is to dispatch a compute shader, and then have the results being displayed by the fragment shader in a draw pipeline ( specifically in a Material ).
An example of what I did with only the fragment shader: https://github.com/davids91/shocovox/blob/main/examples/bevy_wgpu_render.rs

Have you seen the compute shader game of life example? It uses a compute shader to simulate the game of life in a texture, and then that texture is rendered to the window as a fullscreen sprite. There is no copy back to the CPU. It’s just a compute pass and the transparent 2D main pass where the sprite is drawn.

I saw the example I just couldn't figure out how the texture itself is getting displayed. I see there are some bevy-relevant nodes, and a RenderLabel, which I assume is being displayed, but there are just too many nuts and bolts in play here.
This example right?
https://github.com/bevyengine/bevy/blob/main/assets/shaders/game_of_life.wgsl
https://github.com/bevyengine/bevy/blob/main/examples/shader/compute_shader_game_of_life.rs
I see that the texture being rendered is being rendered by the compute shader, and there's no copy to CPU, thanks for the heads up, that I missed previously!
Even with the example  if I'm about to build a system like this I would rather extend the functionality of bevy, I just don't know that much about its inner workings.

Update!
I've made it ... Should anyone be interested. This is totally possible as it is.