fix 'command-not-found: is a directory' error

Motivation for this change
5a5db609e5bd83bc589f36eef394f3ad172d6648 from https://github.com/NixOS/nixpkgs/pull/24962 caused this issue by changing the way that command-not-found is invoked.
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

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

also cc @Mic92 as a potential reviewer.

@jerith666 i approve of this PR

Thanks!


Missing repo

Hi all
I tried to build the project but Maven was complaining about a missing artefact "swagger2markup-spring-restdocs-ext".
I just added the repo to pom.xml and everything was fine:
     <repository>
        <id>jcentral</id>
        <name>bintray</name>
        <url>http://jcenter.bintray.com</url>
        <snapshots>
            <enabled>false</enabled>
        </snapshots>
    </repository>

Thanks for the cool project.
Regards, S.

fixed in https://github.com/hantsy/angularjs-springmvc-sample-boot/commit/756d198370ba0808281d4d4510baf0133e110cd7
Thanks @steffenhohn


R6 v S3/S4 in HADES

From #77 @chrisknoll has posed an interesting way of viewing Capr potentially through R6 via a pure OOP system. The purpose of this post is to
a) understand the benefits of switching to R6 for Capr and
b) consider the impact of R6 within HADES...when/where to use it, why is it beneficial and ultimately does it even matter
I am hoping to get some feedback or thoughts from others @chrisknoll, @azimov, @ablack3, @anthonysena, @schuemie. I know a post was made a while back referencing this same topic.
Thoughts on OOP in context of Capr
Currently Capr is written in S4 this was done for two (at this point, flimsy) reasons:

R6 was not available at the time. R still used ReferenceClass as its pure OOP system. I explored this in early Capr development but opted to go the S4 route
S4 maintains the "feel" of R. Where as R6 is more amenable to programmers coming from the java and python worlds. S4 is a stricter version of S3 which does a better job of working in a functional programming pipeline. When Capr was originally created it was intended to heavily leverage the pipe operator %>% however this proved to be rather awkward.

Resources to give context to S4 and R6 can be found in chapters 14, 15, and 16 of Advanced R. While the strengths of S4 can be found here.
Starting at Capr v2, there was an intentional effort to transition the feel of the code away from piping and towards nested functions. The construction of cohorts would hence feel like building the ui of a shinyDashboard. A dashboard requires a header, sidebar and body. Within each section the user provides context on the look by adding text, output, boxes etc. Similar to a cohort definition where the user is constructing sections of the definition...the entry, attrition, exit and era. An example of what Capr code should look like now is show below:
library(Capr)

cd <- cohort(

  #entry event (i.e. primary criteria)
  entry = entry(
    drugExposure(metformin, male()), # index query of metformin users who are male
    observationWindow = continuousObservation(priorDays = 365L), #365 min prior obs
    primaryCriteriaLimit = "All" # use all index events
  ),

  # attrition to index event (i.e. inclusion rules)
  attrition = attrition(
    # no t1d any time prior
    'no t1d' = withAll( # start group
      exactly( # start criteria (i.e. count)
        x = 0,
        query = conditionOccurrence(t1d),
        aperture = duringInterval(
          startWindow = eventStarts(a = -Inf, b = 0, index = "startDate")
        )
      )
    ),
    expressionLimit = "All" # include all events for attrition
  ),

  # exit when the person leaves the cohort
  exit = exit(
    endStrategy = drugExit(
      conceptSet = metformin,
      surveillanceWindow = 30L
    ) # create metformin era to determine exit
  ),

  # era logic on how to collapse multiple events
  era = era(eraDays = 30L) # 30 days of metformin use builds an era (bit redundant)

)

If Capr were to switch to R6 the syntax would look more like this:
cd <- cohort$new(

  #entry event (i.e. primary criteria)
  entry = entry$new(
    #list of queries or single query
    list(
      drugExposure$new(
        conceptSet = metformin,
        attributes =
          list(
            male$new()
          )
      ),
      observationWindow = continuousObservation$new(prior = 365L),
      primaryCriteriaLimit = limit$new(type = "All"),
      additionalCriteria = NULL # placeholder
      qualifyingLimit = limit$new(type = "All")
    )
  ),

  # attrition to index event (i.e. inclusion rules)
  attrition = attrition$new(
    list(
      group$new(
        name = 'no t1d occurrence',
        type = "all",
        int = NULL, #placeholder
        criteria = criteria$new(
          type = "exactly",
          int = 1,
          query = conditionOccurrence$new(
            conceptSet = t1d
          ),
          aperture = aperture$new(
            startWindow = eventStarts$new(a = -Inf, b = 0, index = "startDate"),
            ignoreObservationPeriod = FALSE # placeholder
          )
        )
      )
    )
  ),

  # exit when the person leaves the cohort
  exit = exit$new(
    endStrategy = drugExit$new(
      conceptSet = metformin,
      surveillanceWindow = 30L
    ),
    censoringCriteria = NULL #placeholder
  ),

  # era logic on how to collapse multiple events
  era = era$new(
    eraDays = 30L
  )

)

Each class has a new object method where we describe its details. Classes can have further methods such as json coercion, sql builder, print statement, and plot functions. This would be quite nice. I am conscious of not overlapping too much with CirceR.
Thoughts on R6
My main hesitation with R6 is that it removes the "feel" of R. R works best in S3 when you take advantage of its "pipe-ability" and functional programming attributes. Forcing R code into a pure OOP system may tune out "tidy-verse" programmers trying to enter the OHDSI software space. Think there is legitimate fear here given the design of the DARWIN software which are quite "tidy-verse" heavy. Not that its any sort of competition.
Having said this, I am beginning to realize the benefits of using R6 particularly if we begin to think about complex objects (circe definition) and pipelines (strategus modules). Having a strictly encapsulated objects makes it easier to force a complex routine across a network.
This post has gone way too long (of which I have accidentally deleted it twice) but maybe it starts a conversation to think about the HADES codebase as it becomes more and more complex :)

A point on the syntax:
I don't think the use of R6, in API form, has to use the $new syntactic style for initializing an instance of an object. In CohortGenerator for the recent implementation of cohort subsets the piping approach is used by adding constructors as first-class R functions. This isn't especially distinct from how an S3 works as an API.
You could even have an API that is identical to the current version 2 of Capr - your functions conditionOccurrence could just be a wrapper to create an instance of class ConditionOccurrence e.g:
conditionOccurrence <- function(...) {
    ConditionOccurrence$new(...)
}

There would then be no difference with the syntactic sugar of R in this case. For the cases of attrition, exactly, withAll etc there would be some need to handle collections of objects but I don't think the API would need to change.
People who prefer a java like declerative style could also just $new syntax (though I think having multiple functionally identical supported APIs for working with the package would be annoying to support for no real gain from a user perspective).
I agree that the current API is good and shouldn't be changed. So to me this is just a point about the internal design of the package. Would R6 make it significantly better? The main reason I use R6 over S4 is mainly because (as you say) I greatly prefer the model for encapsulation (which is the entire point of OOP) . There are also some minor benefits with inheritance but most OOP developers agree that more than a single level of inheritance is often too confusing and prefer just encapsulating an internal object.
Neither approach really enforces a "Safe Interface" approach like you would get in other languages (or even abstract base classes, like python has good support for). So in my opinion I find the differences between the two approaches are largely just programmer preference - in R6 you're encapsulating an object but its hardly type safe at runtime (Honestly, I still think of an R6 object as a jazzed up hash table pretending to be something more, while S4 is just a hash table with useful attributes).
In conclusion, @chrisknoll and I come from a more traditional OOP background so R6 seems more natural to us. However, we're not maintaining this package...

This sounds like a good topic for the next HADES meeting. My current two cents with all of HADES in mind: R code is hard to maintain (no strong typing, no finding errors at compile time, and lets not get into the ever-shifting dependencies). R6 might help in some areas, but also has its own limitations (e.g. can't use breakpoints in R6 methods). If I were to create a new package from scratch I might use R6, but I certainly wouldn't go back and change old code to use R6 just for the sake of slightly better OOP.

Hi, @mdlavallee92 ,
I didn't mean to have #77 present a critique of the CaprAPI.   At the core, the circe object model is a set of Java classes, (which is OO) and so I was just presenting the OO-view of creating instances and assigning values in an OO-way (which in R syntax would be via R6).
CirceR (the R package) does not have any object model defined in it, it works only off the raw JSON string (or the list-of-lists that comes from jsonlite::parseJson).  Makes it hard to programmatically build circe expressions in R (which is why we have Capr!).  But CapR is solving a secondary problem of 'reusable fragments' such that when you merge a fragment (that can contain criteria and concept set), CapR 'figures it out' to merge those reference IDs into the larger expression.  So, in my mind it makes sense that CapR has CapR-specific objects to support the functionality it uses.
Back to the question of R6 use:  the best example where I have made use of it is in the CohortIncidence package.  It started off by a series of functions in the 2.x version, but 3.x introduced R6 classes (in case you are wondering what it was like to migrate).
What I appreciate about R6 is not just the encapsulation and 'by reference' semantics of the object fields, but documentation is pretty nice too.  Here's the documentation output of an IncidenceDesign, and here is the source code annotation that produces it.  Note how the documentation annotations are very close to the implementation of the fields.
The other reason I leveraged it is that jsonlite has some bad behavior when it comes to rendering single-element vectors:  you can tell it to universally treat single element vectors as non-arrays, but it is such a thing (in JSON) to have a single element value that is actually an array:
> x0 <- jsonlite::fromJSON("{\"x\":1, \"y\":[1]}");
> jsonlite::toJSON(x0);
{"x":[1],"y":[1]}
> jsonlite::toJSON(x0,auto_unbox = T);
{"x":1,"y":1}

The problem is the raw json clearly shows x is not an array, and Y is, but jsonlite will serialize both of those to either non-array or both array based on if you auto_unbox or not.  I had a 'wonderful' discussion with the maintainer of json line in this issue.  Needless to say, it didn't go the way I wanted.
So, enter R6 classes, and implementing a toList() function  on the class.  This method will handle making the value unboxes (ie: won't get turned into an array when you serialize to JSON) when you call asJson() on the class
Not having breakpoints in the R6 implementations hasn't been too much of a problem for me (I'm using these R6 classes as mostly data holders not function holders, so maybe i'm just dodging those issues).  And I do have helper 'factory functions' (example: createincidenceDesign()) that can give you auto-complete prompts on the elements of the object, but this strategy does make it a bit of 'double work' such that if your object introduces a new field, you have to go back to your factory function and provide an argument for it.
Anyways, happy to discuss my experiences with it on a call.   I agree with the sentiment of Martijn that unless you see a big gain of using it, I wouldn't go back and switch.  My primary need was controlling JSON serialization (and I liked the way it handled data validation via checkmate), so I went with it.

Thanks guys!
I did not take it as a critique @chrisknoll. I thought it was an interesting perspective to explore. With R6 popping up more and more in HADES packages I wanted to weigh up its potential within Capr. I strongly considered R6 in the version 2 update but opted not to, due to development time and balance with other projects. It makes a lot of sense to match the underlying nature of circe since Capr is an R interface to it as you stated @chrisknoll.
I agree @schuemie, maybe we can talk about this on the next HADES call or after the symposium as I know things get a bit hectic in the lead up.


binary sensor not created

I added the configuration like this:
binary_sensor:
  - platform: hikvision
    host: <IP_ADDRESS>
    username: admin
    password: *******

and the system logger says:
Logger: pyhik.hikvision
Source: components/hikvision/binary_sensor.py:157
First occurred: 11:15:00 (1 occurrences)
Last logged: 11:15:00

Unable to fetch events. Device firmware may be old/bad.

Am i doing something wrong?
Device info:

Device Name Outdoor Camera
Model DS-2CV2041G2-IDW
Firmware Version V5.5.800 build 210629
Encoding Version V7.3 build 200215
Web Version V4.0.1 build 200218

Any assistance is much appreciated 🙇

I just had the same problem with 4 out of 6 cameras.  Simply fixed by upgrading firmware.
try with new firmwares:
https://www.hikvision.com/en/products/IP-Products/Network-Cameras/Wi-Fi-Series/ds-2cv2041g2-idw/?subName=DS-2CV2041G2-IDW
http://<camera_ip>/ISAPI/Event/triggers should respond with triggers xml payload

Did that, same error though :(


How to download the APK file

This problem appears in the vivo handset browser, downloading the APK file is the downloaded zip compression file, the suffix name is not APK, other browsers and mobile phones have not appeared

I guess need to fix the vivo handset browser if other browsers are ok.


Need to integrate my own custom logs(in html or txt format) with prometheus

What did you do?
I have configured a script that fetch the GCP,Audit log and ssh logs from my cluster environment. The data is stored as html/txt file format.
What did you expect to see?
Now I want to fetch  these logs files by Prometheus and need to display the same in Prometheus GUI.Please let me know whether this is possible.
What did you see instead? Under which circumstances?
 See more here: https://prometheus.io/docs/introduction/faq/#how-to-feed-logs-into-prometheus?
If you want to create metrics from your log data, you can use something like mtail to perform aggregations on your data and make it expose those as metrics.
I'm going to close this here as this is not an issue with the Prometheus Operator, but feel free to comment further should you have more questions.


Doc is missing in ordinary_least_squares.jl

I am actually deleting the related file as it doesn't really bring anything in that form


Add azure passport lib & remove keycloak and google libs

About the changes
In order to be able to use azure oauth strategy we must provide the actual module with the functionality. Official azure library provided. I noticed there is unnecessary libraries such as strategy for google and keycloak oauth, so I removed those for now.

Closes https://github.com/Unleash/unleash-docker/pull/66.
Discussion points
Should I add some more documentation for this, or is it good enough already?
Should we add additional containers for keycloak, google and prepare containers for those as well?
Thanks for providing the community repo 🤟

Thank you for this. Great catch with google dep, I suspect it's a remnant from the internal example repo we have for google auth. I've approved the run

A keycloak build and a google build will also be nice to have. :)

 GitHub PR: docker auth (community)


ardour: add ardour 6 and make it the default

We leave Ardour 5 in, since Ardour 6 has a new latency compensation
algorithm, so your mixes might end up sounding different in the new version.

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

looks like you've missed libpulse-dev and libwebsockes-dev packages as dependency

looks like you've missed libpulse-dev and libwebsockes-dev packages as dependency

pulseaudio is in there,and it claims to build with it. I don't have pulse installed,so have not tested.
seems that in order to build with libwebsockets, you also need libuv (assuming that's what provides uv.h), but it only fails halfway trough the build.
Both are in now, thanks.

I made a PR earlier here: https://github.com/NixOS/nixpkgs/pull/87274 - please make sure all of the changes are included in your PR and then I will close mine in favor of this one.
Also it's always a good idea to check for other PRs before starting your own to avoid duplicate work.

@prusnak Oh,I thought I had checked for other PRs, sorry!
I incorporated most of your changes, but yours has the same issue as @veprbl 's: it replaces ardour5 when installing 6, which what I wanted to avoid, see the commit message about the new latency compensation.

@magnetophon Forgot to push the updated commits?

@veprbl Ah, yes, something went wrong.

@veprbl Thanks, all done.

Something went wrong here, I'm trying to fix it!


Open-AzureADMSPrivilegedRoleAssignmentRequest does not support scoped assignments

The cmdlet does not support creating assignments that are scoped to Administrative Units, this is a critical feature when using PIM for granular delegation.
Looks like Graph exposes the necessary property for that "directoryScopeId"
https://docs.microsoft.com/en-us/graph/api/resources/unifiedroleassignmentscheduleinstance?view=graph-rest-beta

Opened this on docs repository, my bad


How to integrate PHP emails using Mailtrap

Article checklist
Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes
This closes #3620

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

@jumamiller @simonmwaniki, @hectorkambow : I have created this article to help us solve the issue with the previous article #3979.
@jumamiller, kindly review. Thank you!

Added the article update to ensure we don't forget to compensate @simonmwaniki accordingly.
cc @hectorkambow @simonmwaniki

Added the article update to ensure we don't forget to compensate @simonmwaniki accordingly.
cc @hectorkambow @simonmwaniki

I've also added the front-matter to include Simon's name : simon-mwaniki.

@owinowendy, this is a great article! Very easy to follow.

ALERT: This is a special article that should be attended to by @hectorkambow!
Thank you team!

Added the article update to ensure we don't forget to compensate @simonmwaniki accordingly.
cc @hectorkambow @simonmwaniki

Not following? The article update label is used when an author is making an update to an existing (already published) article.
Is that what this is? @jumamiller

@owinowendy, this is a great article! Very easy to follow.

ALERT: This is a special article that should be attended to by @hectorkambow!
Thank you team!

Not following @jumamiller

@owinowendy, this is a great article! Very easy to follow.

ALERT: This is a special article that should be attended to by @hectorkambow!
Thank you team!

Not following @jumamiller

Hi, I sent you a message on Slack!

@jumamiller
Just wanted to remind you that we need to run a plagiarism check as well. (dont forget the labels) 👍
I'll run that as well - just wanted to add the reminder.

Hello @owinowendy,
Upon running your article through our 3rd party plagiarism checker it seemed to raise a few flags and the % was higher than we typically accept.
Please see attached PDF - and revisit the article to ensure we are contributing wholly unique and original content.
Be sure to see our resources page to see more info on plagiarism and what is considered as such.
How to integrate PHP emails using Mailtrap #5404 Copyleaks Report.pdf

@Kayere, does this still need work to improve the plagiarism score? If so, please remove the ready for final review label.

@jumamiller @louisefindlay23  plagariasm issue fixed!

jumping on this one


Update Mythic Plus display to account for changes in 10.2

Enum.WeeklyRewardChestThresholdType.MythicPlus was changed to Enum.WeeklyRewardChestThresholdType.Activities in 10.2, causing the display to show the number of keys run instead of the highest key level.

Closed by commit 37a3b03


use import.meta in worker got error: document is not defined

Describe the bug
use import.meta in worker, in dev mode is ok, but after build got the error
 // source
console.log(import.meta.url);

// after build:
console.log(document.currentScript && document.currentScript.src || new URL("worker.js",document.baseURI).href)

document doesn't exist in web worker
Reproduction
create worker in main.js
import Worker from "./worker?worker";
const w = new Worker();
w.postMessage(1);

worker.js
self.addEventListener("message", () => {
  console.log(import.meta.url);
});

System Info
System:
    OS: Windows 10 10.0.19043
    CPU: (16) x64 11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz
    Memory: 13.14 GB / 31.67 GB
  Binaries:
    Node: 16.4.2 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.10 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 7.19.1 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1023.0), Chromium (92.0.902.73)
    Internet Explorer: 11.0.19041.906
  npmPackages:
    @vitejs/plugin-vue: ^1.3.0 => 1.4.0
    vite: ^2.4.4 => 2.4.4

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I think this issue relate with https://github.com/vitejs/vite/pull/2494/files#diff-3b7bb11b5633d82f82c069037789e105005d657667b1da1e96c18fcf3eb622adR65

I think this is a problem / feature of rollup see https://www.rollupjs.org/guide/en/#resolveimportmeta

In ES modules, import.meta is an object and import.meta.url contains the URL of the current module, e.g. http://server.net/bundle.js for browsers or file:///path/to/bundle.js in Node.

By default for formats other than ES modules, Rollup replaces import.meta.url with code that attempts to match this behaviour by returning the dynamic URL of the current chunk. Note that all formats except CommonJS and UMD assume that they run in a browser environment where URL and document are available.

So we need to tell rollup to handle import.meta.url different in modules.
I will work on a solution on that for a plugin with workers that I wrote and post it here if I found something!

Simple plugin where config is the resolved vite config
{
  name: "worker-env",
  resolveImportMeta(prop, ctx) {
    if (prop !== "url") return null;

    return `new URL('${ctx.chunkId}', location.origin + '${config.base[0] == '/' ? '' : '/'}${config.base}').href`;
  },
}

When this is added to the plugin-list here:
https://github.com/vitejs/vite/blob/main/packages/vite/src/node/plugins/worker.ts#L59
This would fix this problem.

This seems to have already been addressed in rollup. https://github.com/rollup/rollup/pull/4186
Can we "rollup" that fix into vite? :drum: :snake:

@JarekToro Is this still a problem? That fix is in the current rollup version used by vite fixed... So if that was I thought was the problem this should be fixed now.

So it seems to be a tangential problem. I created a  stack blitz to demonstrate the issue.
https://stackblitz.com/edit/vitejs-vite-yqbzxq?file=lib/assets/test.worker.81214889.js
Seems that when reference import.meta from an imported script into a worker it fails to remove the document references
Should check the stack blitz for real example but in short
index.ts
import worker from './test.worker?worker';
console.log(new worker());

test.worker.ts
import './worker_import_script';

addEventListener(
  'message',
  function (e: any) {
      console.log(e)
  },
  false
);
postMessage({ ready: true });

worker_import_script.ts
// This right here adds document references when bundled into the worker.
var _scriptDir = import.meta.url;

console.log(_scriptDir);
// This right here actually works fine
const wasmBinaryFile = new URL('test.wasm', import.meta.url).toString();
console.log(wasmBinaryFile);

output
(function(){"use strict";
var e=document.currentScript&&document.currentScript.src||new URL("test.worker.js",document.baseURI).href;
console.log(e);
// ...etc

EDIT: Problem can be simplified further. All that is actually needed is this code in a worker.
test.worker.ts
var _scriptDir = import.meta.url;
console.log(_scriptDir);

As it gets converted to
(function(){"use strict";var r=document.currentScript&&document.currentScript.src||new URL("test.worker.js",document.baseURI).href;console.log(r)})();

As long this is not fixed you can try https://github.com/mathe42/vite-plugin-worker

Awesome thanks

Still reproducible in Vite 2.9.0-beta.2


QR Code Support

Can't believe I almost missed this! This needs to be supported in the first release, it's pretty much the most common way to add codes!

Implemented by acbf61f, 4c75a61 and df4acd3


Tear down kubecon paris booth infra

I considered creating a static site to replace the ones I'm tearing down but I don't have the bandwidth for that work currently and I think we've already run this infra for longer than expected anyway!

UPDATE: I followed instructions in #34 and applied these changes.

/approve


Pasting into an empty Mobiledoc via the browser File menu does not work

Working on a fix, will be coming in a short moment.

closed in #499


Hi. Is the approach of this paper applicable to the R+F case?

Yes, if flows are available.
Actually, we believe the proposed method is naturally agnostic to the combinations of modalities.


When the gateway is used in combination with the ribbon, there is often an error: java.lang.nullpointerexception: null

Question
When the gateway is used in combination with the ribbon, there is often an error: java.lang.nullpointerexception: null
Bug report
2019.11.19 16:04:28.441 [reactor-http-epoll-8] ERROR [org.springframework.core.log.CompositeLog.error:117] - [33d09a99] 500 Server Error for HTTP GET "/wb/upms/v1/getOrgTree?parentId=-1" java.lang.NullPointerException: null
Enhancement
SpringCloud Version：Greenwich.RELEASE
SpringCloud Alibaba Version ：2.1.0.RELEASE
jdk Version ： 1.8
-->

Question
When the gateway is used in combination with the ribbon, there is often an error: java.lang.nullpointerexception: null
Sometimes it's good, sometimes it's bad, sometimes it's OK for a day, sometimes it'll reappear in a few minutes.
But I have no problem bypassing the gateway to access the service directly.
Bug report
2019.11.19 16:04:28.441 [reactor-http-epoll-8] ERROR [org.springframework.core.log.CompositeLog.error:117] - [33d09a99] 500 Server Error for HTTP GET "/wb/upms/v1/getOrgTree?parentId=-1" java.lang.NullPointerException: null
Enhancement
SpringCloud Version：Greenwich.RELEASE
SpringCloud Alibaba Version ：2.1.0.RELEASE
jdk Version ： 1.8
SpringBoot Version: 2.1.4.RELEASE

Please provide the full stack trace.

I have the same problem. Is there any solution

I have the same problem. Is there any solution


Update build-first-sharepoint-adaptive-card-extension.md

Category

[x] Content fix

What's in this Pull Request?
Updated "Build your first SharePoint Adaptive Card Extension" article with few headings & content fixes

Docs Build status updates of commit babee31:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/spfx/viva/get-started/build-first-sharepoint-adaptive-card-extension.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 7e884d0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/spfx/viva/get-started/build-first-sharepoint-adaptive-card-extension.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Thank you @andrewconnell for the comment. I have made the requested changes & updated the article.


Compile list, set, and dict comprehensions.  Refs #287, 342, 344.

Motivation and Context
This was missing functionality.
Please note: nested comprehensions are not supported yet.
Please note: This PR is built on top of PR #341 (i.e. it includes all the changes of PR #341)
Approach
Comprehensions are compiled by transforming the ast into a form that uses more elementary operations, which are then compiled as usual.  This transform requires knowing the element type of an iterator, so some out-of-band (not via compilation) determination of the element type was implemented in the wrapper.  For example, code was added so that we know if we iterate over:

a ListOf(T), we get a T.
a PythonObjectOfType(range), we get an int.
etc.

The type of a comprehension is a ListOf, Set, or Dict, masqueraded to list, set, or dict, respectively.
Along the way, some fixes were done to Dict operations (Refs #344), and to str operations (Refs #342).
Also, str*int multiplication is now compilable.
These changes made some test cases easier to write.
How Has This Been Tested?

multiple iterators are supported
multiple conditions are supported
different expression types
result type is just 'list', 'set', or 'dict'

Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

a full generator model is implemented in 'dev' now.


[FEATURE] AI Services: support Result return type

AI Services should support returning custom POJOs wrapped in the Result object, so that TokenUsage can be accessed.
interface AiService {

    Result<POJO> extractPOJO(String text);
}

Related: https://github.com/langchain4j/langchain4j/issues/660

Implemented in https://github.com/langchain4j/langchain4j/pull/1015


Create an implementor guide

The spec is focused (properly) on what implementations must and must not do. It's hard for an implementor trying to understand how they can implement the specification. We should create an implementor guide that gives a non-normative overview of the concepts and recommendations for implementing the spec.
Topics to cover include:
mounting secrets as readonly

I would like to see this topic explained:
What is the purpose of "type" and "provider" fields in the binding secret?

Add the discussion from #125 into the guide

Based on the discussion in the Slack:

Question: If there is a change in the ProvisionedService secret data value, how to make it reflect in the application container?
It can be achieved by triggering the application reconciliation through a change in the PodSpec-able resource.
To force trigger the application reconciliation, the implementation can change the volume name.
For example, the volume name could be constructed by combining ServiceBinding resource's .metadata.name and Secret resource's resourceVersion value. e.g., prod-account-service-projection-5731

https://kubernetes.slack.com/archives/C012F2GPMTQ/p1615207548014400

Baiju:
The spec is silent about what causes the service binding reconciler to be triggered.  I can think of few cases when the service binding reconciler should be triggered. 1) When the secret pointed through provisioned service is changed or a direct secret 2) A new application with a label matching the label selector comes up 3) An application is recreated with the given name.  I wonder whether the spec should say something about this?
Scott:
I've viewed this as an implementation detail for the spec itself. The spec is primarily concerned with interoperability by defining the contracts at the edges. These suggestions are useful as part of an implementors guide.

Document how to project ServiceBinding resource's .spec.type and .spec.provider into the container.
See this comment: https://github.com/k8s-service-bindings/spec/pull/154#issuecomment-851386665

I think we can move this issue to the website repo.


LOG4J2-2837: Disruptor and JUL no longer recursively start the AsyncLoggerDisruptor

Previously logging initialization would result in an AsyncLoggerContext
initializing an AsyncLoggerDisruptor, which would access a logger from
JUL, starting the process again. This change avoids recursive calls with
a state check.

Thanks for taking care of this, Carter!

I appreciate the reviews, thanks @remkop :-)
I still need to finish full test runs locally for these changes, then I'll merge along with #361, likely tomorrow.

If you can merge  #361, that would be great! Thank you!

committed to release-2.x and master


Retrieve plugin name from PluginInfoService

LGTM

This was closed without merging by accident, so I merged it just now.


TypeError: locale.split is not a function

Using react native 0.57.
componentDidMount() {
I18n.defaultLocale = "en";
 I18n.locale = this.getStoredLanguage();
I18n.currentLocale();
....
}
All I'm trying to do is fetch whatever locale was set from wherever the persistent data is being stored and set that as the current locale.
Here is the full error log, I don't know if this is helpful
TypeError: TypeError: TypeError: TypeError: locale.split is not a function
This error is located at:
in Main (created by Connect(Main))
in Connect(Main) (at navigationStore.js:448)
in Wrapped (at SceneView.js:9)
in SceneView (at StackViewLayout.js:478)
in RCTView (at View.js:44)
in RCTView (at View.js:44)
in RCTView (at View.js:44)
in AnimatedComponent (at screens.native.js:59)
in Screen (at StackViewCard.js:42)
in Card (at createPointerEventsContainer.js:26)
in Container (at StackViewLayout.js:507)
in RCTView (at View.js:44)
in ScreenContainer (at StackViewLayout.js:401)
in RCTView (at View.js:44)
in StackViewLayout (at withOrientation.js:30)
in withOrientation (at StackView.js:49)
in RCTView (at View.js:44)
in Transitioner (at StackView.js:19)
in StackView (at createNavigator.js:57)
in Navigator (at createKeyboardAwareNavigator.js:11)
in KeyboardAwareNavigator (at createNavigationContainer.js:376)
in NavigationContainer (at SceneView.js:9)
in SceneView (at createTabNavigator.js:39)
in RCTView (at View.js:44)
in RCTView (at View.js:44)
in ResourceSavingScene (at createBottomTabNavigator.js:108)
in RCTView (at View.js:44)
in ScreenContainer (at createBottomTabNavigator.js:98)
in RCTView (at View.js:44)
in TabNavigationView (at createTabNavigator.js:178)
in NavigationView (at createNavigator.js:57)
in Navigator (at createNavigationContainer.js:376)
in NavigationContainer (at SceneView.js:9)
in SceneView (at StackViewLayout.js:478)
in RCTView (at View.js:44)
in RCTView (at View.js:44)
in RCTView (at View.js:44)
in AnimatedComponent (at screens.native.js:59)
in Screen (at StackViewCard.js:42)
in Card (at createPointerEventsContainer.js:26)
in Container (at StackViewLayout.js:507)
in RCTView (at View.js:44)
in ScreenContainer (at StackViewLayout.js:401)
in RCTView (at View.js:44)
in StackViewLayout (at withOrientation.js:30)
in withOrientation (at StackView.js:49)
in RCTView (at View.js:44)
in Transitioner (at StackView.js:19)
in StackView (at createNavigator.js:57)
in Navigator (at createKeyboardAwareNavigator.js:11)
in KeyboardAwareNavigator (at createNavigationContainer.js:376)
in NavigationContainer (at Router.js:97)
in App (at Router.js:122)
in Router (at AppRouter.js:242)
in AppRouter (at App.js:47)
in PersistGate (at App.js:46)
in Provider (at App.js:45)
in App (at renderApplication.js:34)
in RCTView (at View.js:44)
in RCTView (at View.js:44)
in AppContainer (at renderApplication.js:33)
This error is located at:
in NavigationContainer (at SceneView.js:9)
in SceneView (at createTabNavigator.js:39)
in RCTView (at View.js:44)
in RCTView (at View.js:44)
in ResourceSavingScene (at createBottomTabNavigator.js:108)
in RCTView (at View.js:44)
in ScreenContainer (at createBottomTabNavigator.js:98)
in RCTView (at View.js:44)
in TabNavigationView (at createTabNavigator.js:178)
in NavigationView (at createNavigator.js:57)
in Navigator (at createNavigationContainer.js:376)
in NavigationContainer (at SceneView.js:9)
in SceneView (at StackViewLayout.js:478)
in RCTView (at View.js:44)
in RCTView (at View.js:44)
in RCTView (at View.js:44)
in AnimatedComponent (at screens.native.js:59)
in Screen (at StackViewCard.js:42)
in Card (at createPointerEventsContainer.js:26)
in Container (at StackViewLayout.js:507)
in RCTView (at View.js:44)
in ScreenContainer (at StackViewLayout.js:401)
in RCTView (at View.js:44)
in StackViewLayout (at withOrientation.js:30)
in withOrientation (at StackView.js:49)
in RCTView (at View.js:44)
in Transitioner (at StackView.js:19)
in StackView (at createNavigator.js:57)
in Navigator (at createKeyboardAwareNavigator.js:11)
in KeyboardAwareNavigator (at createNavigationContainer.js:376)
in NavigationContainer (at Router.js:97)
in App (at Router.js:122)
in Router (at AppRouter.js:242)
in AppRouter (at App.js:47)
in PersistGate (at App.js:46)
in Provider (at App.js:45)
in App (at renderApplication.js:34)
in RCTView (at View.js:44)
in RCTView (at View.js:44)
in AppContainer (at renderApplication.js:33)
This error is located at:
in NavigationContainer (at SceneView.js:9)
in SceneView (at StackViewLayout.js:478)
in RCTView (at View.js:44)
in RCTView (at View.js:44)
in RCTView (at View.js:44)
in AnimatedComponent (at screens.native.js:59)
in Screen (at StackViewCard.js:42)
in Card (at createPointerEventsContainer.js:26)
in Container (at StackViewLayout.js:507)
in RCTView (at View.js:44)
in ScreenContainer (at StackViewLayout.js:401)
in RCTView (at View.js:44)
in StackViewLayout (at withOrientation.js:30)
in withOrientation (at StackView.js:49)
in RCTView (at View.js:44)
in Transitioner (at StackView.js:19)
in StackView (at createNavigator.js:57)
in Navigator (at createKeyboardAwareNavigator.js:11)
in KeyboardAwareNavigator (at createNavigationContainer.js:376)
in NavigationContainer (at Router.js:97)
in App (at Router.js:122)
in Router (at AppRouter.js:242)
in AppRouter (at App.js:47)
in PersistGate (at App.js:46)
in Provider (at App.js:45)
in App (at renderApplication.js:34)
in RCTView (at View.js:44)
in RCTView (at View.js:44)
in AppContainer (at renderApplication.js:33)
This error is located at:
in NavigationContainer (at Router.js:97)
in App (at Router.js:122)
in Router (at AppRouter.js:242)
in AppRouter (at App.js:47)
in PersistGate (at App.js:46)
in Provider (at App.js:45)
in App (at renderApplication.js:34)
in RCTView (at View.js:44)
in RCTView (at View.js:44)
in AppContainer (at renderApplication.js:33)

i18n.js:312:31
I18n.locales.default
i18n.js:311:12
Object.I18n.locales.get
i18n.js:239:15
Object.I18n.lookup
i18n.js:394:31
Object.
i18n.js:589:29
Object.I18n.translate
i18n.js:586:25
Main.render
Main.js:690:13
Main.proxiedMethod
createPrototypeProxy.js:44:29
And here's an image of the error on an Android Emulator (version 9.0)

Work around
in node_modules/i18n-js/app/assets/javascripts/i18n.js, line 312
 locales.forEach(function(locale) {
      var localeParts = String(locale).split("-"); //forced type String on locale
      var firstFallback = null;
      var secondFallback = null;
      if (localeParts.length === 3) {
...
only problem now is the locale not changing after being set but that's probably a different issue

locale.split is not a function
means the "type/class" of locale is updated to a non string value
I can't help you debug since I don't know what can be returned by this.getStoredLanguage()

I just had to deal with this issue.
The issue is coming like you supposed from this.getStoredLanguage(), this is probably returning a Promise and not a proper value.
I managed to fix it by creating a State with a null value at construction state={locale: null} , then in componentDidMount, i called this.getStoredLanguage().then(value=>this.setState({locale: value})).
If it's the same issue, it's not comming from i18n-js.


Displaying image and video on the webchat

This version allow image and video displaying on the webchat. For more details, please look at the README.md file.
#www.hub4it.com

This version allow image and video displaying on the webchat. For more details, please look at the README.md file.
#www.hub4it.com


Multiple security issues

Our security report shows multiple critical issues regarding the latest (4.19.0) images which should be adressed.

Product
Tool/Package
Vulnerability
Installed
Min Fixed

ui
curl
CVE-2021-22945
7.78.0-r0
7.79.0-r0

api
json-schema
CVE-2021-3918
0.2.3
0.4.0

api
libcrypto1.1
CVE-2021-3711
1.1.1g-r0
1.1.1l-r0

api
apk-tools
CVE-2021-36159
2.10.5-r0
2.10.7-r0

api
class-validator
CVE-2019-18413
0.12.2
None

migration
libssl1.1
CVE-2021-3711
1.1.1g-r0
1.1.1l-r0

migration
json-schema
CVE-2021-3918
0.2.3
0.4.0

migration
apk-tools
CVE-2021-36159
2.10.5-r0
2.10.7-r0

The system packages should automatically be up-to-date when the Docker Images are regenerated.
The npm packages need to be updated. The biggest issue might be class-validator which is unmaintained. Maybe a switch to https://github.com/nestjs/class-validator could easily be done, where the fix is included.

fixed in 20.0.3


APPS-8385 Add/remove instance size fields as per new pricing changes

These changes are to go in as part of Apps relaunch and new pricing rollout in mid May.

Uses new instance slugs in examples and for defaults.
Adds bandwidth_allowance_gib, deprecation_intent, scalable and single_instance_only to app instance size spec.
Removes tier_downgrade_to and tier_upgrade_to from app instance size spec.
Removes app_tier_downgrade_cost and app_tier_upgrade_cost from propose app response.

The PR will be merged post the launch ( when the flipper is at 100%) in mid May.

The rollout of new pricing slugs is planned for May 6th.

The new pricing changes have been released. This PR can be merged.


docs: ✏ add FioriAR 2.0 migration guide

Hi Patrick,
I wrote a migration guide in the wiki (https://github.com/SAP/cloud-sdk-ios-fioriarkit/wiki/FioriAR-2.0-Migration-Guide) and linked it in the README. Can you check the migration guide please?
I tried to focus on the most important breaking changes. We have more, e.g

ViewBuilder for scanView expects an enum
AnnotationLoadingStrategy load now returns a tuple
ARManagement Protocol Removed
Renamed descriptionText to subtitle
Changed CardItemModel` Data Types

but I feel not everything has to be covered in the example as the majority of developers probably won't be affected. But we maybe should list them in the migration guide. I am eager to hear your opinion.
Kind regards,
Marco

On a second thought. I agree that not everything needs to be mentioned, but a few thoughts.

ViewBuilder for scanView expects an enum

Perhaps a mention as its likely ViewBuilders will be used as we have seen evidence that Kevin used them in his demo. This is on the line and could go either way.

AnnotationLoadingStrategy load now returns a tuple

A large change, but I don't think it needs a mention as a custom strategy is highly unlikely

ARManagement Protocol Removed

Same as above, only visible through a custom strategy I believe

Renamed descriptionText to subtitle

Very subtle and straightforward, no mention needed

Changed CardItemModel Data Types

This will impact core usage and probably would feel strange if it's not mentioned, even if its straightforward

Thanks @pgobriensap for your feedback.  Based on your feedback I decided to mentioned all changes to CardItemModel protocol in the migration guide.

ViewBuilder for scanView expects an enum

☑ not mentioned. P.S.: in Kevin's demo he is using cardLabel ViewBuilder.

AnnotationLoadingStrategy load now returns a tuple

☑ not mentioned

ARManagement Protocol Removed

☑ not mentioned

Renamed descriptionText to subtitle

✅ mentioned

Changed CardItemModel Data Types

✅ mentioned


Update dependencies: prost to 0.8 and tonic to 0.5

Which issue does this PR close?
Closes https://github.com/apache/arrow-datafusion/issues/817
Rationale for this change
Examples are failing as they use an incompatible version of tonic/prost that came in with arrow 5.1.0 (https://github.com/apache/arrow-rs/pull/560)  which leads to failures on CI PRs such as https://github.com/apache/arrow-datafusion/pull/808/checks?check_run_id=3223418729
What changes are included in this PR?
Update dependencies: prost to 0.8 and tonic to 0.5
Are there any user-facing changes?
New version of dependent libraries

I am merging this one in to get CI back to green. Please let me know if there are any additional concerns which I will be happy to address


Refactor Datastore/Query Tests

Closes #719
Fixes #731

LGTM, just a small formatting request.

Thanks!


Implemented positional choice conflict checking

This PR addresses issue #8
Implemented a way to validate that nested combinations of positional Parameter choices across multiple Subcommands / Parameters do not overlap in such a way that they would cause significant ambiguity as to what the correct handling should be when parsing.  Some combinations may pass this validation and still cause issues when parsing, but such cases are relatively rare and workarounds exist to avoid the ambiguity.
An attempt was made to update the CommandParser class to also use the new PosNode tree during parsing for Positional params, but relatively deep and invasive changes are necessary to do so.

Codecov Report

Merging #9 (38ecfdf) into main (d741b60) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              main        #9    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files           38        39     +1
  Lines         4306      4601   +295
  Branches       972       899    -73
==========================================
+ Hits          4306      4601   +295

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

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

Impacted Files
Coverage Δ

lib/cli_command_parser/parameters/groups.py
100.00% <ø> (ø)

lib/cli_command_parser/__init__.py
100.00% <100.00%> (ø)

lib/cli_command_parser/__main__.py
100.00% <100.00%> (ø)

lib/cli_command_parser/command_parameters.py
100.00% <100.00%> (ø)

lib/cli_command_parser/config.py
100.00% <100.00%> (ø)

lib/cli_command_parser/exceptions.py
100.00% <100.00%> (ø)

lib/cli_command_parser/nargs.py
100.00% <100.00%> (ø)

lib/cli_command_parser/parameters/choice_map.py
100.00% <100.00%> (ø)

lib/cli_command_parser/parameters/options.py
100.00% <100.00%> (ø)

lib/cli_command_parser/parse_tree.py
100.00% <100.00%> (ø)

... and 3 more

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


updateProfile method should return the same response as getProfile on success

After a successful update of user's details, the updateProfile method should return the same JSON response as the getProfile method, with updated data.

Duplicated by #4


Directory listing missing all files for some directories

Directory and filenames containing ampersand "&" fail to call the directory endpoints and need sanitising.

This issue is duplicated in the Track component


How to use with Blitz

Question
We are currently using Blitz to cache websites. Vite uses a versioning system each time JS and CSS files are compiled.
This causes a issue with Blitz, since it caches whole HTML pages. After every deployment, we get 404 errors on JS and CSS files.
How you would suggest fixing this issue without downtime?
The easy solution would be to delete and generate all cache, but this is  resource intensive, especially when we perform many updates.
Using the vite config, is there another solution ?
Thank you!

The typical solution is to do as you said, delete the caches and regenerate them.
You're changing the file name of the files you're serving up, to ensure that browsers download the newly updated versions, but the file name is hard-coded in the statically cached HTML.
For this to work without having to regenerate the static cached HTML, you'd need to point it to a JavaScript file whose name never changes in your static HTML, that then would read the manifest, and serve up the new version.
You could probably do this either in all JavaScript or in a combination of PHP and JavaScript, but that's probably overcomplicating things, and only really viable in a very high traffic site.

Note that if you use cached includes to include your resources, then you can clear/refresh only those includes on deployment, rather than the entire cache.


Add random forest classifier for tomographer

Moving the random forest classifier from rail_base to here.

#1

See PR #1


Swift_Image->embed('imagePath');

I am wondering if anyone have come across embedding images into twig using this bundle.
At the moment this doesnt work:

But this works:
Controller:
$img1 = $message->embed(\Swift_Image::fromPath('/path/DevToolsBundle/Resources/public/images/background.jpg'));
Twig:

Is there any better approach for that?

You should be able to use:

Although, if you are sending the e-Mails from Command Line, you have to adjust your configuration slightly: http://symfony.com/doc/current/console/request_context.html


Changed Environment-property to use a string+object dictionary

Changed Environment-property to use a string+object dictionary instead of an instance of IOwinContext so that it matches IdentityServer's OwinEnvironmentService.

thx


using cache keys as ArrayIterator keys

With this patch we can access to Item by key:
$pool = new Stash\Pool();
$result = $pool->getItemIterator(array('TEST1', 'TEST2'));
echo $result['TEST2']->get();

P.S. previous pull request (https://github.com/tedivm/Stash/pull/136) I closed.

I'm closing this out as it has negative consequences for memory and there hasn't been any demand for this feature.

negative consequences for memory
This is very strange reason. Ragular arrays and hash-arrays does not significantly different.

We widedly using this feature in our projects.

Are they open source ones? I'm curious if there's something I can test it.

Our projects are comercial sofware (http://www.banki.ru which contains more then 10 subprojects)
One of use cases: we loading list of items from remote service, when storing into cache one by one. Next time we loading another list first from cache, then not found items from remote service. This way helps us to minimize traffic to services and to optimize cache utilization

I see value in this. @tedivm why do you think it has performance impact at all? It should be copy-on-write behavior, so no increase in memory usage.

Submit a new PR and we'll test it. The original PR had merge conflicts and the branch it was based off of was deleted.

#250


Documentation of Library for Standalone Usage

We're interested in reusing the dataflow component on its own (outside of vega). To do that we'd like to understand more about the design and structure of dataflow.
How would I use it from own code? What is its conceptual model (e.g. how are operations defined, what is the structure of input data etc)?
The current README is a little sparse in this regard.
Our own analysis so far can be found in this hackmd.
Much appreciate any guidance you can provide 😄

Yes, there is much documentation missing for stand-alone use. Apologies for that! I hope to get to it later, but it may be a while before I can make a dedicated effort. Documentation and support for the higher-level Vega JSON syntax has taken precedence.
In the meantime, you may find the stand-alone examples in this repo useful: https://github.com/vega/vega-dataflow-examples

We're interested in doing the same, to be able to process data serverside before passing to the client for rendering. Would love more docs here as you're able!

I recommend taking a look at the new dataflow-api, which provides a (documented!) standalone JavaScript API over Vega's dataflow system. Feedback welcome!

Wow, this looks incredible. Did you somehow know we were looking for this, before we did? 😉
We won't be ready to implement for a bit still but will definitely provide feedback, thank you! Feels like enough to close this issue, but will let @rufuspollock decide.


Export facial landmarks

It's possible to export the coordinates of the facial landmarks on a file? Thanks.

Sure, open a file and traverse the landmark and then output to the file


PXP-11315: Explore Page Filter Redesign

PXP-11315
Replaces tabs with dropdown for selecting filter sets and collapse all function for enum facets

When "Data Files" tab is active: the Collapse All button  is not stylized like the Cases tab:

The collapse functionality of the ticket is not yet implemented:

Each facet header needs to be collapsable as in the wireframe. This is third item in the ticket's Task list.


Build kmyth general utilities as a shared library

In order to provide applications utilizing kmyth more flexible access to underlying kmyth utilities, without copy/paste or some other means of including kmyth utility source files, this pull request restructures kmyth to build a 'kmyth-utils' shared library that can be linked in to expose this kmyth utility functionality.
In order to hopefully make this cleaner, the "file_io", "formatting_tools", and "memory_util" sources were moved into a top-level "utils" directory and used to build a "libkmyth-utils.so" library (in the lib folder). The remaining files in "src/util" and "include/util" were put into either "src/network", "src/protocol", "include/network", or "include/protocol".
An attempt was made to change Makefile variable names not related to the "kmyth-utils" library having a "util" or "utility" reference to something different or less confusing.
A "utils-lib" target (make utils-lib) was added to build only the new library. The "libs", "docs", "test", "install", and "uninstall" targets were updated to reflect the changes.
The build information in the INSTALL.md file was updated.

Yes, you are correct. This builds the code in formatting_tools.c, file_io.c, and memory_util.c as a shared library. The rest of the kmyth code (minus the logger code), then, is built by linking against this library. This provides an API with access to these "kmyth utilities" (e.g., base64 encode/decode, ...) by other applications.
In an attempt to avoid confusion, this pull request also reorganizes and renames a little to avoid re-use of the term "utilities".
The changes to the code are indeed minor, but I think the "indent" utility caused some fairly widespread reformatting of files.


Can't delete account when logged in through OAuth

Gitea Version
1.15.6
Git Version
No response
Operating System
No response
How are you running Gitea?
Void Linux package
Database
SQLite
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Description
It is not possible to delete an account when logged in through OAuth as the form for doing it asks for a password.
Screenshots

More details in

https://github.com/go-gitea/gitea/issues/18329


Investigate use of Github Codespaces

Is your feature request related to a problem? Please describe.
Can we use Github Codespaces to containerise our environment? This is intended to help new starts get coding sooner.
Describe the solution you'd like
N/A
Describe alternatives you've considered
Maybe Docker?
Additional context
Does codespaces need a pro account/ premium feature?

Something to bear in mind (as I just discovered when trying to review a PR) is that we'll need to configure the codespace to load in multiple repositories.
See the following links for more details:
https://github.blog/2022-04-20-codespaces-multi-repository-monorepo-scenarios/
https://docs.github.com/en/codespaces/managing-your-codespaces/managing-repository-access-for-your-codespaces


[Enhancement]Set the parallelism of low-cardinality field tasks to 1 (#31249)

Set the parallelism of low-cardinality field tasks to 1

[FE Incremental Coverage Report]
:disappointed: fail : 0 / 2 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/statistic/StatisticExecutor.java
0
2
00.00%
[167, 168]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)


docker build time error

Describe the bug
19.82 store/db/sqlite/sqlite.go:13:2<EMAIL_ADDRESS>Get "https://storage.googleapis.com/proxy-golang-org-prod/966630c75881099b-modernc.org:sqlite-v1.28.0.zip?Expires=1707487199&GoogleAcce
ssId=gcs-urlsigner-prod%40golang-modproxy.iam.gserviceaccount.com&Signature=D7nwkhCSyaJEhsy8nPASVhVyXda5NLaMueCBGVwPEW7S5UMonH406Raz51SVCttooEPgGgpj1b%2B0PNDty%2FJO1BBzVCz9WfB3KazX40g2o5j3ssknBbN5x5WiD
xNkleAnL5ahE3f8y3tSD48LC8TwuV8VbFmb6Xi2eDAJFOtrvEI59jqntkmjxGGFzbaqxlOnuntrnALYSdwv%2BWVdDpIeRth4HEVPY5dEyGYqrBCTj%2Fyi1seuZCYSQfrtgKWdgY1FEBl847TQpNdb17pY1ApprF7ughwBIS6OYptbtJEB4W2CxuV%2Bx15hTSH8NrOKIVDJNiTUzfWel94NkPvanvhygg%3D%3D": dial tcp <IP_ADDRESS>:443: connect: connection refused
Dockerfile:19
17 |     COPY . .
18 |
19 | >>> RUN CGO_ENABLED=0 go build -o memos ./bin/memos/main.go
20 |
21 |     # Make workspace with above generated files.
ERROR: failed to solve: process "/bin/sh -c CGO_ENABLED=0 go build -o memos ./bin/memos/main.go" did not complete successfully: exit code: 1
View build details: docker-desktop://dashboard/build/default/default/tbzf5va17cfa9tgcxzl3m0cq7
Steps to reproduce

docker build -t demo .
This bug occurred

The version of Memos you're using
The latest version
Screenshots or additional context

dial tcp <IP_ADDRESS>:443: connect: connection refused


Validate BigQuery Script with Zetasql

Hi, we have been using zetasql analyzer to validate our generated sql statement for BigQuery. Recently we have decided to generate multi-statement sql like:
BEGIN
CREATE TEMP TABLE TempTable1(columnName STRING) AS select columnName from projectId.dataset.tableName where (((partitionColumn >= '2022-12-31') and (partitionColumn < '2023-01-03')) and ((localDatetimeColumn >= '2023-1-1') and (localDatetimeColumn < '2023-1-2')));
CREATE TEMP TABLE TempTable2(columnName STRING) AS select columnName from TempTable1 where (upper(CompanyGuid) in ('COMPANYGUID') and GroupId in ('groupId'));
select columnName from TempTable2 where DeviceId in ('deviceId') limit 1;
END

But we failed to validate the above statement with zetasql with following config:
LanguageOptions languageOptions = new LanguageOptions();
languageOptions.supportsStatementKind(ZetaSQLResolvedNodeKind.ResolvedNodeKind.RESOLVED_QUERY_STMT);
languageOptions.supportsStatementKind(ZetaSQLResolvedNodeKind.ResolvedNodeKind.RESOLVED_CREATE_TABLE_AS_SELECT_STMT);
languageOptions.enableMaximumLanguageFeatures();
AnalyzerOptions analyzerOptions = new AnalyzerOptions();
analyzerOptions.setLanguageOptions(languageOptions);
Analyzer.extractTableNamesFromScript(sql, analyzerOptions);

I am wondering if anyone could advise? Thanks!

Zetasql doesn't support script parsing currently.

@masterlittle thanks for your reply, I am wondering if there is any timeline or ETA for ZetaSql to support script parsing?


cargo-diet: 1.2.4 -> 1.2.5

Diff: https://github.com/the-lean-crate/cargo-diet/compare/v1.2.4...v1.2.5
Changelog: https://github.com/the-lean-crate/cargo-diet/blob/v1.2.5/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

1 package built:

cargo-diet


Problem with Update Child Records workflow on D365 v9

Hi there,
I'm testing an update to Dynamics 365 V9, and have come across an issue with some functionality in place. The steps that should happen are as follows:

User adds an opportunity product to an opportunity
Synchronous workflow triggers on create of opportunity product
Workflow checks opportunity for boolean value
Workflow updates all opportunity product records with value (including the one that triggered it)

The fields set in the custom workflow steps are as follows:

This worked fine in 8.2, but in v9 I get a business process error that states:
"Instances cannot be resolved and nested lifetimes cannot be created from this LifetimeScope as it has already been disposed."
Any idea what this could be? The full error log is here:
ErrorDetails (3).txt
Workflow tools version: <IP_ADDRESS>
D365 Version: <IP_ADDRESS>9
Many thanks,
Tom

HI,
From my experience it seems that the error you are getting is being thrown by a plugin running on the opportunity. I just implemented this workflow step to update a customer entity that is a child of Order and it worked without any errors.

Hi Leibishr,
Thanks for the suggestion, I've checked this and can't see any other plugins working on this entity, so I assume it's this library causing the issue. It's running as a synchronous workflow so works somewhat similarly I imagine.
I think perhaps the oddity here is that when creating an opportunity product, I'm updating all the opportunity products attached to the opportunity, including the one I've just created. I'm doing this because you can't update opportunity products with workflow, really I just want to update the one I've created. The only other way I could see of doing this was by writing a plugin, which I didn't want to get in to.
Any other ideas from anyone please?


plugin-frames does not process file name comments that include + in the file name

plugin-frames version: 0.35.2
SvelteKit files are regularly prefixed with + in the file name, like +layout.svelte and +page.server.ts. Commented file names that include a + are not processing as titles.
+ is a supported character for file paths on Linux, Windows, and other major operating systems.
+ is not included in the file name regex in utils/getFileCommentRegExpString.
Pull request: #205

Wow, this is great! Thank you very much for this well-described issue AND the accompanying fix PR!

I just released your fix as version 0.35.3 on NPM. Thanks again!


LocaleId reference link incorrect

"
-LocaleId
Specifies the language of this site collection. For more information, see Locale IDs Assigned by Microsoft (https://go.microsoft.com/fwlink/p/?LinkId=242911).
"
This URL does not contain valid Locale IDs for use with the -LocaleId parameter. The LocaleId for English (United States) as per SharePoint Online is 1033, but that link not only doesn't mention a LocaleId, it refers to the Location ID as "0x0409"
The closest Microsoft Reference I could find is here: https://docs.microsoft.com/en-us/openspecs/office_standards/ms-oe376/6c085406-a698-4e12-9d4d-c3b0ee3dbc4a. I would suggest pointing readers to something like that which refers to values that are going to be accepted by this parameter.

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

ID: 2c62bf4a-ac47-96ce-33fb-0d5ea9b178ae
Version Independent ID: 301586d3-ec04-c42d-4681-0bb59443d35c
Content: Set-SPOSite (SharePointOnlinePowerShell)
Content Source: sharepoint/sharepoint-ps/sharepoint-online/Set-SPOSite.md
Service: sharepoint-powershell
GitHub Login: @trent-green
Microsoft Alias: trgreen

Hello @Muxxx Stefano,
Thanks for your feedback, I can see that the link you share could be better, will propose thru a PR.


Add Odoo

Icon Name
hl-odoo
Icon(s)

I'm currently traveling and only have access to the GitHub web-ui. I will add this into the project when I am able to do so.

Thanks!


Make egui work on WebGPU out of the box.

BREAKING: Makes WebGL for wgpu opt-in. By default wasm builds for egui will now use WebGPU. To use WebGL, you need to add wgpu = { version = "0.16.0", features = ["webgl"] } now to your Cargo.toml.
Works around a surface creation issue which made any subsequent call to any method on the surface fail (in a rather arcane manner) - see https://github.com/gfx-rs/wgpu/issues/3710
Regarding the breaking change: Wgpu doesn't allow both WebGPU and a "direct" backend, like WebGL, to live side by side (i.e. wgpu is either in a "wrap WebGPU" or "implement WebGPU" mode). Therefore, egui now inherits the webgl-opt-in behavior. Since Web is memory constraint and availability of WebGPU can be queried easily, two separate builds for WebGPU and WebGL seem to be desirable regardless, so I don't think this is a big issue.
Fixes #2137

./scripts/build_demo_web.sh --release with feature glow changed to wgpu
egui_demo_app_bg.wasm filesize:
before: 6.62mb
after: 4.96mb
not too bad (:

So WebGPU must be configured at compile-time to user either WebGL or WebGPU, but you cannot ever configure it so that you pick at runtime?
So in effect we'll need to have a app_gl.wasm and app_wgpu.wasm and pick the correct one using JavaScript?

yes, that's my understanding of the state of things!

confirmed that now: https://github.com/gfx-rs/wgpu/discussions/3119#discussioncomment-3890627
Concluded that more from the code I debugged, but didn't go into the details. Personally I only ever looked at the backends that go through direct.rs (all backends except WebGPU, including webgl) and never looked at the mechanism that dispatches to web.rs instead.

confirmed non-workaround path working with pending fix on wgpu


Move baseconv/base62 deprecation warning for Django 5.x

fixes #33

Coverage increased (+0.009%) to 90.056% when pulling a4db1af6c889d29f5b21830a2a1bae132f2481b0 on fix-deprecation-warning into 1764c5c8ae4022589b441aaf79253a72f5fae222 on master.


Weird bug with some of the e2s

Issue:
After switching to latest git branch and spawning an e2 (link is down below) started getting this issue. I've never had that problem before on workshop branch. There is no errors in e2, it errors after I spawn/refresh it.
sv: Expression 2 (Aviation instruments ver15): entities/gmod_wire_expression2/core/number.lua:99: attempt to perform arithmetic on a nil value
E2:
aviation_instruments_ver15.txt

Would really appreciate if example chips for bugs could be narrowed down so we could actually try and reproduce it, I will try and do it myself, just saying for the future / anyone else seeing this.

I bisected the tree and found cbc01711 to be the cause

Thought so, I found other people were reporting weird behavior but also giving E2s with thousands of lines which don't help me much..

I've reproduced it:
for(I = 1, 2) { # Needs to iterate at least twice?
    switch (I) {
        default,
        break
    }
}

for(Other = 1, 1) {
    print("repro", Other) # Other is nil
}

Need to figure out a fix now, probably just unbalanced push/pop of scopes with switch :p

Probably gonna merge #2400 after this fix so this stuff doesn't happen again..


IPC will not be available after page refresh

use code like
// backend
const win = await Gluon.open('index.html', {
    windowSize: [800, 500]
  });
win.ipc.test= async (data) => {
    //....
})
// frontend
await Gluon.ipc.test('Message!');

invoke success on first run but after refresh page invoke will got error Uncaught (in promise) TypeError: Gluon.ipc.test is not a function

This is fixed already in 0.14.0 (install from GitHub to use alpha or wait a few days for release).


[jiminy] Allow change in the unit used to log the simulation.

Add Global.TIME_UNIT constant to specify the unit in which time is logged, this enables the user to change the precision with which logging is done.

Ideally, a new terminal condition should be added to the engine before merging:

Done

Done

Nice !


”Pir”, ”Piru Nou”, ”Sărvăzel” are listed in the drop-down menu

Description:
When user writes ”Pir”, ”Piru Nou”, ”Sărvăzel” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Pir"

Expected results
The location appears in the drop-down menu as "Sat Pir, SATU MARE (Pir)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Piru Nou"

Expected results
The location appears in the drop-down menu as "Sat Piru Nou, SATU MARE (Pir)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Sărvăzel"

Expected results
The location appears in the drop-down menu as "Sat Sărvăzel, SATU MARE (Pir)".


key_pressed bug

When you use the key_pressed bug it also prints out some random characters.
Steps to reproduce
function update(deltaTime)
    if key_pressed('w') then
        print("Test") -- Prints also 0 and 1
    end
end

Important Note: key_pressed is now a global variable that gets updated every frame !


Tons of warning logs for EBI but actually caused by Sponge

I am currently running

SpongeForge version: 1.12.2-2838-7.2.4-RC4058
SpongeVanilla version: Nope
Java version: 261
Operating System: CentOS7

Plugins/Mods: Actually I think this issue does not relevant to any mod. Epicbanitem plugin and sponge only.

Issue Description
https://paste.ubuntu.com/p/nsVTSvxd4f/
The text above appear repetitively and infinitely in intermittent periods.
Sorry for troubling, I already asked the developers of ebi that they told me this issue is caused by sponge and I should submit the issue here. So then I do what he said. Thanks for developing sponge which is a very good platform!

If you really need to know what mods and plugins I installed, here it is.
Also, even though I do not think mods are factors, if you need me to test with only ebi and sponge, I will do that. Please comment to inform me.
Thanks!
I'm sorry for some parts of my speaking may disrespect although my true intension is good. Please point them out if exist, because I am currently studying English and I am not aware of some mistakes I made. I was criticized once in the past, so now almost all of my comments include this part of text.
Plugins (34): Minecraft, Minecraft Coder Pack, SpongeAPI, Sponge, AmberShop, AntiAttack, Boop, BungeeTabListPlus-SpongeBridge, ClearMob, Clearall, CommandUtils, DupeFixer, EconomyLite, EpicBanItem, ForceChinese, Holograms, HologramsPlus, Inventory Sync, LuckPerms, MMCReboot, MultiChat, Nucleus, OfflineServerUsernameWebRequestNetworkManager, PlaceholderAPI, ServerListPlus, Sledgehammer, SpongeLoginFix, TitleMOTD, VirtualChest, VirtualChestEnchantments, WorldAlias, WorldEdit, XySafeShulkerBox, spark
Mods (126): Minecraft, Minecraft Coder Pack, Forge Mod Loader, Minecraft Forge, SpongeAPI, Sponge, AE2 Wireless Terminals (AE2WTLib), AI Improvements, Another Common Bug Fix, AppleSkin, Applied Energistics 2, Armory Expansion, Armory Expansion - Construct's Armory, Armory Expansion - Custom Materials, Armory Expansion - Ice and Fire, Armory Expansion - MatterOverdrive, Avaritia, Avaritiaddons, Baubles, BitHop, Bookshelf, Botania, Chameleon, Chisel, Clumps, CoFH Core, CoFH World, CodeChicken Lib, CommonCapabilities, Construct's Armory, CraftTweaker JEI Support, CraftTweaker2, Cyclops Core, Dark Utilities, Ender IO, Ender IO Applied Energistics Conduits, Ender IO Base, Ender IO Conduits, Ender IO Integration with Forestry, Ender IO Integration with Tinkers' Construct, Ender IO Integration with Tinkers' Construct, Ender IO Machines, Ender IO Open Computers Conduits, Ender IO Powertools, Ender IO Refined Storage Conduits, Ender IO endergy, EnderCore, Extra Utilities 2, FTB Backups, FTB Library, FTB Utilities, Flux Networks, FoamFix, FoamFixCore, Forestry, Forge Microblocks, Forge Multipart CBE, GasConduits, GunpowderLib, Hammer Core, Immersive Cables, Immersive Engineering, Immersive Tech, InControl, Industrial Foregoing, InstantUnify, Integrated Crafting, Integrated Dynamics, Integrated Terminals, Integrated Tunnels, IntegratedDynamics-Compat, IntegratedTerminals-Compat, IntegratedTunnels-Compat, Integration Foregoing, Inventory Tweaks, Iron Chest, JEI Bees, JourneyMap, Just Enough Characters, Just Enough Energistics, Just Enough Items, LagGoggles, Lazy AE2, MTLib, Mantle, McJtyLib, Mekanism, Mekanism: Generators, Minecraft Multipart Plugin, Mod Tweaker, NuclearCraft, Nutrition, OpenBlocks, OpenModsLib, OpenModsLib Core, OreExcavation, Pam's HarvestCraft, Pewter, Phosphor Lighting Engine, PlusTiC, RandomPatches, Redstone Flux, Shadowfacts' Forgelin, SpongeForge, Storage Drawers, Surge, Tesla Core Lib, Tesla Core Lib Registries, Thaumcraft, Thaumic Additions: Reconstructed, Thaumic Energistics, The One Probe, Thermal Dynamics, Thermal Expansion, Thermal Foundation, Tinker's JEI, Tinkers Tool Leveling, Tinkers' Construct, UniDict, VanillaFix, WanionLib, Wireless Crafting Terminal 2, XNet, YNot, libnine, p455w0rd's Library

Exception: java.lang.AbstractMethodError:  Method net/minecraft/entity/monster/EntityEnderman.getInventory()Lorg/spongepowered/api/item/inventory/type/CarriedInventory; is abstract
  at net.minecraft.entity.monster.EntityEnderman.getInventory(EntityEnderman.java)

Yeah, that's a Sponge error - I'll ask someone to look at this.


Clink ALT based shortcut keys don't seem to work

Version: 1.0.1
Platform: win32 10.0.17763
Plugins:
Just installed terminus for the first time, opened a shell, saw it was using Clink so tried ALt-H  and other ALT based shortcuts, but none of them seem to work,  other Ctrl oriented commands do seem to wrok

You can switch the Alt/Meta key behaviour in the preferences. It's handled by the OS by default instead of being forwarded to the shell.


Solving override issue

I think there was a mistake here. Either line 131 should add _seq or line 130 should check SBOL_TYPED_URIS. Otherwise you end up with a warning: An object with this identity is already contained in the Document'

@JMante1 thanks very much for this submission. I do not see a related issue, did I miss it? If there isn't one could you please open an issue that documents and demonstrates the problem you are fixing?
Could you please also provide a test case that fails without your fix and passes when your fix is included?
Thanks!

@JMante1 this PR has been idle for 3 months. I'm guessing you found another workaround. Is this ok to close?

Another 3 months have gone by, and no response to the Jan 11 request for unit test or the April 13 inquiry. I am closing this PR due to lack of activity. Please reopen it when you are ready to continue.


npm-run-script  doesn`t working properly

Version
4.0.0
Environment info
mac os && Linux

Steps to reproduce
Dynamic parameter passing to packaging script is needed in the project。
npm run script -- version=xxx
https://docs.npmjs.com/cli-commands/run-script.html
1、package.json script, "build": "vue-cli-service build"
2、npm run build -- version=xxx.xx
3、Entry module not found: Error: Can't resolve  /../path/version=xxx.xx
What is expected?
want to use process.argv to get the parameter through parameter passing
What is actually happening?
build failure.

The script does work properly.
You ran npm run build -- version=xxx.xx so the build command was vue-cli-service build version=xxx.xx. That command runs this script underlyingly https://github.com/vuejs/vue-cli/blob/dev/packages/%40vue/cli-service/lib/commands/build/index.js It takes version=xxx.xx as the entry argument. Of course it failed.


create player UI

Also, the CI check is failed, see the details. It cannot build the frontend. Perhaps due to the change in package-locj.json. It's irrelevant also, I don't know why it changes on your side. npm i on my side does not change it. Anyhow, avoid committing it.

Also, the CI check is failed, see the details. It cannot build the frontend. Perhaps due to the change in package-locj.json. It's irrelevant also, I don't know why it changes on your side. npm i on my side does not change it. Anyhow, avoid committing it.

I don't know why it's not running build, because i didn't change anything in package-lock.json.

Maybe you didn't change it intentionally, however you have committed some changes, therefore I request you to revert them.

Hi @anantakrishna,
I don't why i'm getting this now


bump build

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Naptr feature tests and fixes

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Thank you for your contribution! We will review the pull request and get back to you soon.

More changes to follow after the NAPTR feature is enabled in PPE/Prod when I can test the working. I will record the expected results after the feature is enabled.

Please fix CI issues


chore: Integrate Discord with CI failure

We want to communicate CI failures on discord.
Technical Requirement

Whenever there is a build failure on main we wish to publish that information on our discord channel.

/bounty 50$

/attempt

/attempt #2073


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.8 to before v5.15",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=db5ad6b7f8cdd6d78efef62a1557461d0cbaee54",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=ce7723e9cdae4eb3030da082876580f4b2dc0861"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "db5ad6b7f8cdd6d78efef62a1557461d0cbaee54",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "ce7723e9cdae4eb3030da082876580f4b2dc0861",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "nvme-tcp: fix possible req->offset corruption\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.15 by commit ce7723e9cdae4eb3030da082876580f4b2dc0861, it was introduced in version v5.8 by commit db5ad6b7f8cdd6d78efef62a1557461d0cbaee54. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1002202


ImageResizerContextMenu & PowerRenameContextMenu show in winget list as seperate applications

Microsoft PowerToys version
0.62.1
Running as admin

[ ] Yes

Area(s) with issue?
General
Steps to reproduce
Install PowerToys
open Terminal and run "winget list"
✔ Expected Behavior
Only PowerToys is show in the list like this:
PowerToys (Preview) x64 | Microsoft.PowerToys | 0.62.1 | winget
❌ Actual Behavior
Two extra applications are shown:
PowerRenameContextMenu | PowerRenameContextMenu_8wekyb3d8bbwe | <IP_ADDRESS>
Microsoft.PowerToys.ImageResizerContextMenu | Microsoft.PowerToys.ImageResizerContextMenu_8wekyb3d8bbwe | <IP_ADDRESS>
Other Software
Windows Package Manager Source (winget) | Microsoft.Winget.Source_8wekyb3d8bbwe | 2022.923.2026.688

@stefansjfw , @yuyoyuppe
Wondering about the name of PowerRemameContextMenu package as Microsoft.PowerToys is missing at the beginning. And is it expected that the package version is <IP_ADDRESS> istead of the PT version?

@stefansjfw , @yuyoyuppe Wondering about the name of PowerRemameContextMenu package as Microsoft.PowerToys is missing at the beginning. And is it expected that the package version is <IP_ADDRESS> istead of the PT version?

Here is an image that better shows the problem

(I have removed all the rows from winget that are not relevant to this issue)

is it expected that the package version is <IP_ADDRESS> istead of the PT

this is being fixed by  https://github.com/microsoft/PowerToys/pull/20881
@JulianTompa Regarding 2 extra applications... This is how tier 1 context menu items work in Win11. In order to show PowerRename and ImageResizer in Win11 tier 1 context menu, packages need to be registered and those registered packages are shown by winget list.

is it expected that the package version is <IP_ADDRESS> istead of the PT

this is being fixed by  https://github.com/microsoft/PowerToys/pull/20881

Perfect. 👍

@JulianTompa Regarding 2 extra applications... This is how tier 1 context menu items work in Win11. In order to show PowerRename and ImageResizer in Win11 tier 1 context menu, packages need to be registered and those registered packages are shown by winget list.

Can't winget group them together? They are all belongs to PT and I expected that they are depend on it.

They are all belongs to PT and I expected that they are depend on it.

Not really. Those are being registered during runtime, only if OS is Win11, so not really regular package dependency

They are all belongs to PT and I expected that they are depend on it.

Not really. Those are being registered during runtime, only if OS is Win11, so not really regular package dependency

Can we mark them as RuntimeComponent or something else to not show up?


Update landing page

Docs Build status updates of commit 0ff99f2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

archived-reference/7.1/Microsoft.PowerShell.Core/About/about_Comparison_Operators.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Core/Clear-History.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Core/Connect-PSSession.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Core/Disable-PSSessionConfiguration.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Core/Disconnect-PSSession.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Core/Get-Module.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Core/Get-PSSession.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Core/New-PSSession.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Core/New-PSSessionOption.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Core/Remove-PSSession.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Core/Set-PSSessionConfiguration.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Core/Set-StrictMode.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Core/Stop-Job.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Core/Unregister-PSSessionConfiguration.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Core/Where-Object.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Management/Clear-Content.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Management/Get-HotFix.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Management/Get-ItemProperty.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Utility/ConvertTo-Json.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Utility/Format-List.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/Microsoft.PowerShell.Utility/Get-Verb.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/7.1/PowerShellGet/Find-RoleCapability.md
:white_check_mark:Succeeded
View (powershell-7.1)

archived-reference/docs-conceptual/overview.md
:white_check_mark:Succeeded
View (>=powershell-3.0)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Configure fixer is rule set config for it is null

ping @kubawerlos
there is also a failing test I've to figure out but doesn't seem like a lot of trouble

@SpacePossum I was thinking of moving the call from https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/v2.16.1/src/AbstractFixer.php#L55 to the factory in the current major as well, it is called anyways, so won't be any BC, right?

Should a fixer configure itself with default configuration during construction though? I think it is not the responsibility of the fixer and whatever is using the fixer must configure it (through configure) before calling the fixers fix - method with the appropriate configuration. WDYT?


Update usages of core/lib to reflect current state

This PR's main purpose is to allow rosetta's feature/proof-of-stake branch to properly build. It catches rosetta up to the current state of core on core's feature/proof-of-stake branch by updating usages of core/lib to reflect the updated function signatures.

Current dependencies on/for this PR:

feature/proof-of-stake

PR #59   👈

PR #60

This stack of pull requests is managed by Graphite.

Merge activity

Feb 27, 12:03 AM EST: @lazynina started a stack merge that includes this pull request via Graphite.


Dashboard should be auto-deployed

When we first rolled out the dash, we were auto-deploying it. I asked for it to not be deployed by default because I didn't feel it was ready for public consumption. That isn't the case anymore, so I think we should deploy it by default, unless the user passes a --no-dashboard flag (or something similar)
I have some concern (shared by @JoeyZwicker) about whether it feels weird to include an enterprise feature in the default open source deploy, but I think we agreed that that the concern is outweighed by the better initial user experience so long as:

it's easy to avoid deploying
we're offering free initial usage

I'm still a bit squeamish, but this feels like the right decision, given how frequently we field questions from users who are confused by why they can't access the dash after initial deploy (though admittedly, that is likely exacerbated by #2444)

User PR submitted for this. #2667
On Wed, Feb 14, 2018 at 9:32 AM, stale[bot]<EMAIL_ADDRESS>wrote:

This issue has been automatically marked as stale because it has not had
any recent activity. It will be closed if no further activity occurs. Thank
you for your contributions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pachyderm/pachyderm/issues/2443#issuecomment-365684012,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtjwCbnqrIhPNVntPHXjS9prxRHOEkUks5tUxiogaJpZM4QAVyv
.

This has been fixed


Indium on Curseforge or Modrinth?

With the release of Sodium 0.2.0 on Modrinth, Do you plan on releasing Indium? Or is the plan still to get it implemented into Sodium itself? https://modrinth.com/mod/sodium

Even just having a GitHub release build would be nice.

I'm likely going to wait for Sodium 0.2.0 to release on CurseForge first, but yes I would like to release a build for 1.16 on both platforms.

Sodium 0.3 for Minecraft 1.17.1 has been released. I can confirm that there are big rendering issues if Sodium is used with Indium.

Now under review for Modrinth, and available from Github Releases at https://github.com/comp500/Indium/releases!


Jchaumont/3979/auth login token

Implementation of the auth login --method token flow.

I know it's a draft but I had a look at it anyway :).
Looks good overall, but I am worried about the syntax: ggshield auth login --method token --token $TOKEN means the token can end up in shell history files and other places. Have we considered reading it from an environment variable or from stdin (like gh does) instead?

@agateau-gg Done, I replaced it with a click.prompt :slightly_smiling_face:

@agateau-gg Done, I replaced it with a click.prompt slightly_smiling_face

Looks good :+1:


Click not working

What are you trying to achieve?
SeeElement is working, clicking isn't.
Scenario('click on five teasers', (I) => {
    I.amOnPage('/');
    I.seeElement('.first-teaser');
    I.click('.first-teaser'); //working
    //page redirect
    I.seeElement('a[title="Zur Startseite von Bergfreunde.de klettern"]');
    I.click('a[title="Zur Startseite von Bergfreunde.de klettern"]'); //not working
});

What do you get instead?
Node is not visible,
even though seeElement worked before
Details

CodeceptJS version: 1.1.6
Puppeteer 1.1.1
NodeJS Version: 9.7.0
Operating System: Windows 10 64bit
puppeteer with latest chrome (64.0.3282.186)
Configuration file:

{
  "tests": "./*/*_test.js",
  "timeout": 10000,
  "output": "./output",
  "helpers": {
    "Puppeteer": {
      "windowSize": "1920x1080",
      "url": "https://www.bergfreunde.de",
      "show": true,
      "waitForAction": 800,
      "keepCookies": true,
      "chrome": {
            "executablePath" : "C:/Program Files (x86)/Google/Chrome/Application/chrome.exe",
            "ignoreHTTPSErrors": true,
            "devtools": false
        }
    }
  },
  "include": {
    "I": "./steps_file.js"
  },
  "bootstrap": false,
  "mocha": {},
  "name": "bfFrontendTests"
}

@timbergfreund What html element are you trying to click, is it a <div> or <span>?

Sorry I just read your test case :D I'll need to test this out later.

@timbergfreund Though did you see that the search string matches 3 elements on the page (just tested using the chrome dev tools), is that right?

Thats right, it matches 3 elements on the page. Problem is solved, thanks!


feat(assets): add files for M 317 TF1

up to date

Codecov Report

Merging #602 (6b2eb63) into develop (0afdee6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #602   +/-   ##
========================================
  Coverage    60.40%   60.40%
========================================
  Files          227      227
  Lines         3715     3715
  Branches       570      570
========================================
  Hits          2244     2244
  Misses        1437     1437
  Partials        34       34

Flag
Coverage Δ

unittests
60.40% <ø> (ø)

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

Continue to review full report at Codecov.

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


Migrates Ternary Search Tree to Swift 3 syntax

You're absolutely right @vincentngo
The reason I kept insertData instead of renaming it to insert(data:) was because there are some private methods called findNode and insertNode in the TernarySearchTree class.
But...I agree, insert(data: data, withKey: key) looks better, so I'll change that. And to align things I'll rename the private functions findNode and insertNode so the follow the naming conventions too.
Stay tuned 😄

Aaaand done.
I took the liberty of adding some unit tests as well (just a couple) and they seem to pass 😌
Hope it is as expected

Also it would be awesome if you had time to add some README doc on how ternary search trees work! But that's not part of this pull request haha.


Need Help: how to add games in DB

Is your feature request related to a problem? Please describe.
I have run the program 🥰, but there is no game information in the database. I don't know how to add a new game for development testing.
Describe the solution you'd like
Hope

A tutorial(description) on how to install a game
simple game data (can init the db games)

https://github.com/mantou132/nesbox/tree/master/packages/server#manage-games


[agent] Agent lifecycle managment

Why a daemon?

We want to have a backup of all our data somewhere
We want to have always online device that can replicate data with other devices
We want the ability to run multiple profiles

So what do we need to do?

We need a reliable long-running process (read daemon) with error handling and logs
We need a lock file to ensure only one instance of that process running at once

Spec

[ ] Daemonize process
[ ] Stop/start daemon
[ ] Force stop
[ ] Error handling
[ ] Start time
[ ] Status of currently running daemons
[ ] Log files
[ ] Multiple profiles

Process management design

https://github.com/dxos/dxos/pull/4114