fix: proxy resolution from pac file for ws requests on windows and macOS (3-0-x)

Description of Change
Backports https://chromium-review.googlesource.com/c/chromium/src/+/1143093
Fixes https://github.com/electron/electron/issues/15344
This is not required for versions greater than 3.x because we changed to use chromium proxy resolver implementation with https://github.com/electron/electron/pull/14656
And also it is not required for versions less than 3.x as the switch to platform proxy resolver was made temporarily only for ch 66 upgrade due to lack of support for mojo network context apis.
Checklist

[x] PR description included and stakeholders cc'd
[x] Patch information is added to appropriate .patches.yaml
[x] script/update runs without error
[x] PR title follows semantic commit guidelines

@deepak1556 thanks for the description of why this isn't needed except for in 3.x :+1:


Delete metrics when the reporting CR is deleted

remove policy_schedule_interval_seconds metrics when drpolicy is deleted

deleting of DRPC metric is in progress


Change trig functions to operate in radians; add degree trig functions

We change the existing trig functions to operate in radians. We add sind, cosd, etc. as functions that operate in degrees. This follows Fortran, MATLAB, Wolfram Alpha, etc.
In addition, we remove atan2, and move its functionality to the 2-arg variants of atan and atand. This follows Lisp and Scheme, and is appropriate since CoffeeScript has optional arguments with default values. atan(y, x = 1) == atan2(y, x)
And we fix the accuracy of these functions by removing their use of roundEpsilon. That function seems to be trying to ensure that values that should be rational (e.g. sind 30 == .5), look "nice". Instead, we just explicitly check for such values. The fact that there is a small number of such values is ensured by http://www.jstor.org/stable/2304540.

This is cool.  Since it's a breaking change it'd be nice for us to do two things:
(1) Write a short blog entry (mostly so that we have a web page where we can point people who are confused)
https://github.com/PencilCode/blog/tree/master/documents/posts
(2) Do a short sed script to change exiting bare sin(90) to sind(90).
Here is a list of egrep results of code that matches sin|cos|tan|atan etc.
http://david.pencilcode.net/home/maybebroken.txt

Here's a python3 program that will make the appropriate changes.
You will want to change it to do the right thing to actually update the database,
probably by removing the filter to check for particular functions (used for debugging),
changing the file name handling, and writing unchanged lines.
import re
import string
files_to_skip = [# Code is in an unfinished state. Looks like it wants to be in radians.
"belto15/Conic_Sections",
# Copies of jquery-turtle.js:
"dkogan/myjquery_turtle.js",
"drm/my_jquery_turtle.js",
# Uses sin, looks confused.
"belto15/Blood_pressure",
# Just confused.
"misterv/Experiments/Experiment/Triginometry",
# This code is just confused. It uses tan, but probably wants to be using atand.
"monaghant15/DirectionTest"]
A bunch of these use Math.sin, but use sin as a variable name.
degrees_to_radians = re.compile('degrees[-]?(to)?[-]?radians|D_2_R|degrees', re.I)
file_line = re.compile('^(.?):(.)$')
trig_line = re.compile('^(.?\b)(sin|cos|tan|asin|acos|atan|atan2)(\b.)$')
turtle_color_operation = re.compile('\b(pen|dot|polygon|background|color|fill|box|wear)\b')
def all_whitespace(s):
return s.strip(string.whitespace) == ''
def fix(line):
match_trig = re.match(trig_line, line)
if match_trig:
start = match_trig.group(1)
trig_function = match_trig.group(2)
rest = match_trig.group(3)
if start.endswith('.') or rest.startswith(':') or
re.search('["']', start) or
re.search('["']', rest) or
trig_function == 'sin' and
(start.endswith('unpardonable ')) or
trig_function == 'tan' and
(re.search(turtle_color_operation, start) or
rest.startswith(' sneakers') or
re.search('\bbeige\b', start) or
all_whitespace(start) and all_whitespace(rest) or
re.search('\[', start) or re.search('\]', rest)):
return start + trig_function + fix(rest)
else:
if trig_function == 'atan2':
trig_function = 'atan'
return start + trig_function + 'd' + fix(rest)
else:
return line
with open('/Users/bcassels/Downloads/maybebroken.txt') as input:
with open('/Users/bcassels/Python/test-out.txt', 'w') as output:
for line in input:
match_line = re.match(file_line, line)
if match_line:
file = match_line.group(1)
if not(file in files_to_skip or
file.endswith('.html') or
re.search(degrees_to_radians, file)):
code = match_line.group(2)
match_trig = re.match(trig_line, code)
if match_trig and match_trig.group(2) == 'tan':
output.write(file + ':' + fix(code) + '\n')

Here's my blog post explaining the change:
https://docs.google.com/a/google.com/document/d/1YskbkZBJunG-T4hlJih_2sSyZ1UkOQ6AwpQ9ZxiCrow/edit

Thanks bob!  Send me your conversion script.  Also I'll be doing the log->debug change soon.


implement the alibaba cs service support

provide only the regions and products that support alibaba cs

ACSK is available in all regions, instance types depend on the zones.

#74


How do I save an array of pixels?

I'm trying to save an array of pixels but can't seem to do so.
I can't access the pixel data directly (mutate it), can't seem to load pixel data directly (eg using loadFromMemory.
I tried using packed (gapless) data and allPixelsAtOnce on an initialized Image, but it only seems readable.

What would have help you achieving this more readily?

@HuskyNator perhaps there is a documentation issue we could fix?

It'd be nice to be able to use loadFromMemory using a plain pixel array?
I feel like I'm hacking my way around this by using an assignment like allPixelsAtOnce()[] = pixelsArray[].
Apart from that, it wasn't really clear whether this was possible from the docs. (Though I got it to work :) )

Ah yes, loadFromMemory() is to load en encoded-image.
You are supposed to use image.createViewFromData() and then you have an Image which is a reference to that data (that you can .clone).
Maybe a new call that do that internally would be cool, please suggest a name maybe?

Ah I see. I totally missed that comment! Totally my bad :)
In what way would the two be different?
I'd normally say making it doable using loadFromMemory, but that might be confusing/complex.
I think loadFromPixels or loadFromPixelData works well. :)
Also slightly confused about the existence of LoadFlags in this case.
They're not used by createViewFromData right? How do you implement the same change starting from pixel data?
Honestly quite confused about the difference between Layout, PixelFormat and LoadFlags.
They seem to overlap a bit?


Update footer links to use new focus style

V3 focus styles

V3 focus styles with legacy mode turned on

Pre-v3 focus styles

Closes https://github.com/alphagov/govuk-frontend/issues/1320

Sorry if this wasn't clear but the links that are inline within a paragraph should have an underline. In this case that would be Government Digital Service and Open Government Licence v3.0
Regarding the legacy mode, should probably keep the underlines if it's not too much trouble.

Spoke to people on Slack, since people using govuk_template wont be using this component, it seems like it's not a big importance to add the old behaviour.

Did you manage to make the inline link change? the preview is dead now

Note that this style was updated in this pull request: https://github.com/alphagov/govuk-frontend/pull/1491

Some context on why we removed underlines:

I believe the intention was to treat the links in the footer in the same way we treat other navigational elements – like the header, or in the case of the Design System the side navigation. Where the ‘thing’ is a ‘list of links’, the thinking is/was that they don’t need the underlining because there’s no wider context in which they need to be distinguished. The exception is any link that does appear within e.g. a paragraph, in which case it does need the underline to distinguish it. It has been questioned a few times on support recently though.
The benefit, I believe, is that it allows us to have a clear hover state for those links, which we didn’t previously have


Splice item from [rows] not removing correct row

I'm submitting a ...  (check one with "x")
[x ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior

 <ngx-datatable #table class="material" [rows]="filteredItems"

The array filteredItems contains 10 items.  If I use splice to remove element two, the ngx-datatable re-renders but it removes the bottom-most row of the table.
Expected behavior

Only item two should be removed.
Reproduction of the problem
Upgrading to 11.1.7 introduced the issue.   Worked as expected with 11.0.4
What is the motivation / use case for changing the behavior?
No change.   Should exhibit same behavior as 11.0.4
Please tell us about your environment:

Table version: 11.1.7

Angular version:  5.0.0

Browser: Chrome

Language: TypeScript 2.6.2

The '_.reject' only works if you are using a certain library. However, someone helped me with this. Apparently the problem is that you first have to set the array of rows to another variable and then do the slice from that variable, then reset the array of rows to the variable.
So:
deleteFilteredItem(index) {
let temp = [...this.filteredItems];
temp.splice(index, 1);
this.filteredItems = temp;
}

Somehow the assigning
let temp = [...this.filteredItems];
just with spread operator is important here.
I tried it with simple assigning one array to another
let temp = this.filteredItems
and it was still buggy as described above...

@Kligerr This is because using the spread operator that way creates a new array which gets detected using the OnPush change detection strategy. The array is still the same even if you temporarily assign it to a local variable.


Validate funding stream addresses

Motivation
We need to validate the coinbase output addresses against the funding stream receivers. Part of https://github.com/ZcashFoundation/zebra/issues/338
Specifications
[TBA]
Designs
[TBA]
Solution
This is based on https://github.com/ZcashFoundation/zebra/pull/3017 and it still needs a bunch of work. It is published so we can have the big picture of all we need to get done.
It needs adjustments in almost all calls and further test coverage. Right now it is passing test blocks and a quick test will be to sync up to tip however i didnt made this yet as the code is constantly changed.
Review
Will be useful for @teor4321 as they are reviewing https://github.com/ZcashFoundation/zebra/pull/3017
Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

Follow Up Work

Replaced by https://github.com/ZcashFoundation/zebra/pull/3040


Can't link MbedTLS after last commit (CON-664)

After last commit, in which refrigerator and room_air_conditioner were added, I cannot compile anymore my project because the compile cannot find MbedTLS:
../esp-idf/tools/esp-matter/connectedhomeip/connectedhomeip/config/esp32/third_party/connectedhomeip/src/crypto/CHIPCryptoPALmbedTLS.cpp:330: undefined reference to mbedtls_hkdf'`
Tried updating esp-idf to 5.1 version and freshly installed esp-matter again, all with the same result.
 esp-idf % git log
commit cbce221e88d52665523093b2b6dd0ebe3f1243f1 (HEAD, tag: v5.1)
Merge: c570f67461 e44a6888ea
Author: Jiang Jiang Jian<EMAIL_ADDRESS>Date:   Wed Jun 28 17:33:53 2023 +0800

    Merge branch 'bugfix/mac_ext_order_for_c6_h2_v5.1' into 'release/v5.1'

    efuse: Fix byte order in MAC_EXT (for C6 and H2) (v5.1)

    See merge request espressif/esp-idf!23990

esp-matter % git log
commit 85963d2ca8f4ecb49fa3234f5b0273cb2df411b7 (HEAD -> main, origin/main, origin/HEAD)
Merge: 714233a de530ef
Author: Shu Chen<EMAIL_ADDRESS>Date:   Thu Jul 27 20:22:48 2023 +0800

    Merge branch 'nvs/change_default_nvs_size_to_48k' into 'main'

    change example default nvs size to 48k

    See merge request app-frameworks/esp-matter!443

@Okywan You need to enable HKDF algorithm by setting MBEDTLS_HKDF_C = y in sdkconfig.
You can also do this by running idf.py menuconfig from your project directory. You can find HKDF algorithm under component config --> mbedTLS

Right, that was it. Is this explained somewhere and did I miss it?


Why Copy Api is So slow???

I have tested some one drive api method.
And I found that one drive copy api is very slow...
I tried 20 times copy api, and the the fatest time is 8 second, and the slowest is 2 min and 2 second.
everage secont is 40 sec....
File size is only 13.62 KB docx type file...
Is there some way to make speed up copy speed? or Do you have any plan to make speed up copy speed???

Hi @lolhyuck
This is one of the reasons the copy API is still "in preview". We're working on improving how the copy API works and will continue to iterate on that before taking the API out of preview.


Update/install with pnpm fails with latest version of kit

Describe the bug
I wanted to check out the new changes, tried to update and tried a clean install
$ pnpm update
Packages: +24 -10
++++++++++++++++++++++++----------
Progress: resolved 231, reused 210, downloaded 0, added 0, done
node_modules/.pnpm/@sveltejs+kit@1.0.0-next.406_svelte@3.49.0+vite@3.0.7/node_modules/@sveltejs/kit: Running postinstall script, failed in 197ms
.../node_modules/@sveltejs/kit postinstall$ node svelte-kit.js sync
│ > The filesystem router API has changed, see https://github.com/sveltejs/kit/discussions/5774 for details
│     at create_manifest_data (file:///home/s0001325/work/sersophane/node_modules/.pnpm/@sveltejs+kit@1.0.0-next.406_svelte@3.49.0+vite@3.0.7/node_modules/@sveltejs/kit/src/core/sync/c
│     at create (file:///home/s0001325/work/sersophane/node_modules/.pnpm/@sveltejs+kit@1.0.0-next.406_svelte@3.49.0+vite@3.0.7/node_modules/@sveltejs/kit/src/core/sync/sync.js:25:24)
│     at Module.all (file:///home/s0001325/work/sersophane/node_modules/.pnpm/@sveltejs+kit@1.0.0-next.406_svelte@3.49.0+vite@3.0.7/node_modules/@sveltejs/kit/src/core/sync/sync.js:58:
│     at file:///home/s0001325/work/sersophane/node_modules/.pnpm/@sveltejs+kit@1.0.0-next.406_svelte@3.49.0+vite@3.0.7/node_modules/@sveltejs/kit/src/cli.js:71:15
└─ Failed in 197ms
 ELIFECYCLE  Command failed with exit code 1.

 $ pnpm install
Packages: +210
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Packages are hard linked from the content-addressable store to the virtual store.
  Content-addressable store is at: /home/s0001325/.local/share/pnpm/store/v3
  Virtual store is at:             node_modules/.pnpm
Progress: resolved 231, reused 210, downloaded 0, added 210, done
<EMAIL_ADDRESS>Running postinstall script, done in 15ms
node_modules/.pnpm/@sveltejs+kit@1.0.0-next.406_svelte@3.49.0+vite@3.0.7/node_modules/@sveltejs/kit: Running postinstall script, failed in 200ms
.../node_modules/@sveltejs/kit postinstall$ node svelte-kit.js sync
│ > The filesystem router API has changed, see https://github.com/sveltejs/kit/discussions/5774 for details
│     at create_manifest_data (file:///home/s0001325/work/sersophane/node_modules/.pnpm/@sveltejs+kit@1.0.0-next.406_svelte@3.49.0+vite@3.0.7/node_modules/@sveltejs/kit/src/core/sync/c
│     at create (file:///home/s0001325/work/sersophane/node_modules/.pnpm/@sveltejs+kit@1.0.0-next.406_svelte@3.49.0+vite@3.0.7/node_modules/@sveltejs/kit/src/core/sync/sync.js:25:24)
│     at Module.all (file:///home/s0001325/work/sersophane/node_modules/.pnpm/@sveltejs+kit@1.0.0-next.406_svelte@3.49.0+vite@3.0.7/node_modules/@sveltejs/kit/src/core/sync/sync.js:58:
│     at file:///home/s0001325/work/sersophane/node_modules/.pnpm/@sveltejs+kit@1.0.0-next.406_svelte@3.49.0+vite@3.0.7/node_modules/@sveltejs/kit/src/cli.js:71:15
└─ Failed in 200ms
 ELIFECYCLE  Command failed with exit code 1.

Reproduction
> git clone git@github.com:ollema/sersophane.git
> cd sersophane
> pnpm install

Logs
No response
System Info
System:
    OS: Linux 5.15 Ubuntu 20.04.4 LTS (Focal Fossa)
    CPU: (16) x64 Intel(R) Core(TM) i9-10885H CPU @ 2.40GHz
    Memory: 19.60 GB / 31.20 GB
    Container: Yes
    Shell: 5.0.17 - /bin/bash
  Binaries:
    Node: 18.7.0 - ~/.nvm/versions/node/v18.7.0/bin/node
    npm: 8.15.0 - ~/.nvm/versions/node/v18.7.0/bin/npm
  Browsers:
    Chrome: 104.0.5112.79
  npmPackages:
    @sveltejs/adapter-node: next => 1.0.0-next.86
    @sveltejs/kit: next => 1.0.0-next.406
    svelte: ^3.49.0 => 3.49.0
    vite: ^3.0.7 => 3.0.7

Severity
blocking an upgrade
Additional Information
No response

Yep, you'll need to npx svelte-migrate routes before you can use the new version

aha, when I read the migrations guide (https://github.com/sveltejs/kit/discussions/5774) I thought the order was:

pnpm update
npx svelte-migrate routes

but I guess it was the other way around

now it works, thanks!

gotcha, clarified the order on https://github.com/sveltejs/kit/discussions/5774

I was reproducing bug for another repo and was clearing everything out from my production project. In the end of that, when my minimal reproduction repo was ready, I decided to bump all my dependencies (in case bug already fixed). After successfully migrating to new sveltekit, I was still getting error after installing dependencies:
The filesystem router API has changed, see https://github.com/sveltejs/kit/discussions/5774 for details
│     at create_manifest_data (file:///.../issue-reproducer/prettier-svelte-kit-2/node_modules/.pnpm/<EMAIL_ADDRESS>│     at create (file:///.../issue-reproducer/prettier-svelte-kit-2/node_modules/.pnpm/<EMAIL_ADDRESS>│     at Module.all (file:///.../issue-reproducer/prettier-svelte-kit-2/node_modules/.pnpm/<EMAIL_ADDRESS>│     at file:///.../issue-reproducer/prettier-svelte-kit-2/node_modules/.pnpm/@sveltejs+kit<EMAIL_ADDRESS>└─ Failed in 144ms

This was really unexpectedly for me, but new version of sveltekit requires +page.svelte in your router directory. For my bug reproduction only __layout (+layout) was enough. Killed about 30 minutes on this.


Update dependencies

https://github.com/knative/release/actions/runs/5681931414/job/15399143450 is complaining in "Verify it is up to date": The following files are changed: vendor/knative.dev/eventing/hack/update-cert-manager.sh
/assign @dsimansk

/assign @pierDipi

Verify job:
Error: Please run ./hack/update-codegen.sh.
diff --git a/vendor/knative.dev/eventing/hack/update-cert-manager.sh b/vendor/knative.dev/eventing/hack/update-cert-manager.sh
old mode 100644
new mode 100755
knative-sandbox/eventing-kafka-broker is out of date. Please run hack/update-codegen.sh
Error: Process completed with exit code 1.

:thinking:


Singleton Handles should be a public API

Singleton is not implemented yet in the Java API. Thus, this should not be merged.


Make a release

We are prevented from using cabal-doctest because our test suite uses NoImplicitPrelude

I'll prepare a new release soon.

I've uploaded cabal-doctest-1.0.3 to Hackage with this change.

Thanks!

On 3 Nov 2017, at 14:14, Ryan Scott<EMAIL_ADDRESS>wrote:
I've uploaded cabal-doctest-1.0.3 to Hackage with this change.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.


Add an option to show the tooltip when hovering on the handles

Describe the feature
It would be nice to have an option to show the tooltip when hovering on the handles.
Describe the solution you'd like
A hover option for the tooltip parameter:
<vue-slider v-model="model" :tooltip="'hover'"></vue-slider>

It would show the tooltip on both hovering and focusing one of the handles.

When `tooltip="hover", is it possible to include "focus" function?
Because when dragging, the mouse is not always on the handle.

Thanks for your feedback, hover and active parameters are supported in version 3.1.0.
active contains both hover and focus functions, and now the tooltip default parameter is active.

Awesome, thank you very much!


mapToProps: support new observe key for Observables

Background

It is difficult for Widgets to communicate with each other
A Pub Sub model can help solve the problem
But our mapToProps handles re-renders based on state only

Attempted solution:

If mapToProps can trigger re-renders based on Observables, it would help

What's done

A new observe key has been introduced in mapToProps. And it is expected to return an Observable.
rxjs has been upgraded to v5.0.0-rc.1

Usage example
// ./components/Root.js
import { Observable } from 'rxjs';
import { createComponent, mapToProps } from 'frint';

const RootComponent = createComponent({
  render() {
    return (

{this.props.appId}

{this.props.total}

    );
  }
});

export default mapToProps({
  observe: (app) => {
    // you have access to `app` instance

    return Observable
      // iterate over these values in a sequence
      .of(1, 2)

      // now convert it to an object, which will be available as props
      .scan(
        // this function would be called twice,
        // first with `1`, and then `2` as number
        (props, number) => {
          props.total = props.total + number;

          return acc;
        },

        // the initial object to start the `scan` with
        {
          appId: app.getOption('appId'),
          total: 0
        }
      );
  }
})(RootComponent);

The example is a very simple one, but given the observe function is passed with the app instance, you can get observables from other Services/Factories, and combine them into one, and then produce the final props object to be made available in the Component.
And every time the observable emits a new value, it would trigger a re-render in your Component.
Further reading
We already use Observables in Region.
You can read further about it here: http://reactivex.io/rxjs/

Coverage increased (+0.4%) to 90.681% when pulling 36cafbd86bf7cc605896d0b51e618c50412fe031 on maptoprops-observable into a3df2c0e3e71ce6189c7c65a6ad4daf24b81bcd9 on master.


Typescript issue

Not working on React typescript

I am also facing the typescript issue. There is no support for it.

Add this code to types.d.ts file
declare module "react-image-zoom" {
  import { ComponentType } from "react";

  interface ReactImageZoomProps {
    width: number;
    height?: number;
    zoomWidth?: number;
    img?: string;
    scale?: number;
    offset?: {
      vertical: number;
      horizontal: number;
    };
    zoomStyle?: string;
    zoomLensStyle?: string;
    zoomPosition?: "top" | "left" | "bottom" | "right" | "original";
  }

  const ReactImageZoom: ComponentType<ReactImageZoomProps>;

  export default ReactImageZoom;
}


Tenants website not working

Description
Hello! I have installed this package successful and followed this tutorial https://blog.usejournal.com/writing-a-full-featured-multi-tenant-laravel-app-from-scratch-a0e1a7350d9d. I can't get the tenant website working, i don't know if it is a vhost(apache2) issue .
I have left webserver with default settings expect for apache2 enable which is true.
Any help would be highly appreciated, thanks!

Actual behavior
..
Information

hyn/multi-tenant version:5.2
laravel version:5.6
database driver and version:mysql
webserver software and version:apache2
php version:7

tenancy.php config

webserver.php config

Error log
Description: Unable to locate the server named "boise1.tenants.blazebytes.com" --- the server does not have a DNS entry. Perhaps there is a misspelling in the server name, or the server no longer exists. Double-check the name and try again.

Seems to work fine for me: http://boise1.tenants.blazebytes.com/
Perhaps you have a hosts file entry for that domain pointing at the wrong IP?

Seems to work fine for me: http://boise1.tenants.blazebytes.com/
Perhaps you have a hosts file entry for that domain pointing at the wrong IP?

No, the vhost file is OK. What do u mean it works fine for you?

OK sir, seems the DNS propagation was the issue. now it works . Thanks!!!


Ability to generate unique serial number for multi-device emulation.

To simulate multiple devices, the emulator must create virtual devices with unique MAC and serial numbers.  Perhaps random or as a parameter upon launch?

Working on this currently. The goal is to have it create deterministic Serial Number + Mac Address from an IP Address / Port combo.


[BUG] Button text in section "Books to Learn Rust" has typo

Description
The button in section "Books to Learn Rust" has typo - it should be "See More Books"
Expected Behavior
n/a
Actual Behavior
n/a
Screenshots (if applicable)

Additional Information
Fix needs to be applied in https://github.com/FrancescoXX/rustcrab/blob/35628d9690e67748f84a196a232c04a16701c339/src/components/BooksSection.tsx#L43

thanks


[BUG] - Shiny Apps don't seem to render correctly

Describe the bug
I created an R environment with r-irkernel and r-essentials dependencies.  I then tried running
library(shiny)
library(ggplot2)

datasets <- c("economics", "faithfuld", "seals")
ui <- fluidPage(
  selectInput("dataset", "Dataset", choices = datasets),
  verbatimTextOutput("summary"),
  tableOutput("plot")
)

server <- function(input, output, session) {
  dataset <- reactive({
    get(input$dataset, "package:ggplot2")
  })
  output$summmry <- renderPrint({
    summary(dataset())
  })
  output$plot <- renderPlot({
    plot(dataset)
  }, res = 96)
}

shinyApp(ui, server)

from https://www.oreilly.com/library/view/mastering-shiny/9781492047377/ch01.html which should look like
.

However upon visiting<EMAIL_ADDRESS>I only see

Expected behavior
Should render as expected
OS and architecture in which you are running Nebari
Linux - Azure
How to Reproduce the problem?
see above
Command output
No response
Versions and dependencies used.
No response
Compute environment
None
Integrations
No response
Anything else?
No response

That worked great.  I do think this is something that will trip up a lot of users.  Perhaps we could set up a rule in traefik to add the last forward slash if missing for endpoints that end with /proxy/*.

This may be a simple change in jupyter-server-proxy (it already has an AddSlashHandler). But it might have some unintended consequences so would require some thorough testing internally before we propose it upstream.

The / vs no / has tripped me up on other packages as well, panel etc.

Closing in favor of https://github.com/nebari-dev/nebari-docs/issues/240


Neovim Support for Elm Syntax

It looks like elm support is still missing in neovim.
Vim:
https://github.com/vim/vim/blob/master/runtime/syntax/elm.vim
Neovim (missing as of today):
https://github.com/neovim/neovim/tree/master/runtime/syntax
I assumed, these runtime patches will find it's way into neovim, too. But maybe somebody has to create an explicit PR? Does somebody want to dig into that?
NeoVim Patch PRs
This is how Vim patches that are merged into neovim look like:
https://github.com/neovim/neovim/pulls?q=vim-patch+
Elm Runtime in Vim
PR
https://github.com/vim/vim/pull/6157
Runtime update
https://github.com/vim/vim/commit/acc224064033e5cea21ef7f1eefb356ca06ff11d#diff-6189986fbc5a8d68cbeb136221f76964ef0b68a961a5d3c4202482aada9b83fe

@andys8 , I can try to handle that.

So, it turns out there is a patch for the file type detection that has already been applied to neovim (https://github.com/neovim/neovim/pull/12403) but the syntax, ftplugin and indent files are missing from said patch. I couldn't find a Vim patch for the those files, so I opened an issue in neovim because I am not sure how to proceed (the contribution readme doesn't help in this case, although I think just porting the files and opening a PR should be fine). Waiting for the neovim guys to reply.

Nice. Thanks a lot

Will be solved with https://github.com/neovim/neovim/issues/13193

Hey, long time.
I opened a PR for the changes: https://github.com/neovim/neovim/pull/13678.
I am waiting for a review but I expect there might be more work involved as I am not sure I created the patch correctly. Lets see.

Hey, long time.
I opened a PR for the changes: https://github.com/neovim/neovim/pull/13678.
I am waiting for a review but I expect there might be more work involved as I am not sure I created the patch correctly. Lets see.

https://github.com/neovim/neovim/pull/13718 was merged. Thanks a lot

https://github.com/neovim/neovim/pull/13718 was merged. Thanks a lot


Update sbt-wartremover to 2.4.15

Updates org.wartremover:sbt-wartremover from 2.4.10 to 2.4.15.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.wartremover", artifactId = "sbt-wartremover" } ]

labels: sbt-plugin-update, semver-patch

Superseded by #1400.


Weapon.onFire documentation lists arguments in wrong order.

This Issue is about (delete as applicable)

An error in the documentation

Docs say weapon first, then bullet instance. Actual order from code is reversed.

Thanks for opening this issue, and for submitting a PR to fix it. We have merged your PR into the dev branch, and attributed the work to you in the README. If you need to tweak the code for whatever reason, please submit a new PR.


Bump version in master after new release

@GuillaumeGomez I guess this can be closed, you just keep it in mind on the next release?

Euuuh... I prefer to keep it open for now.

I prefer to keep it open for now.

Forever you mean?

No, until I think about it. :3


Web Apps for Containers with work/school login

On the page "https://azure.microsoft.com/en-us/try/app-service/containers/" there is the following message
"Web Apps for Containers are not supported with a work or school login."
What on earth does that mean?
I can't run a business on Containers with Web Apps?
Can we get some more explanation added to that page.
Also that same page lists all the different docker images, but with no explanation whatsoever about what they include. There is no .NET image at all. Is .NET (full framework) not supported for docker?

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

ID: 593c3abc-7b38-6673-62b6-7787d0dda4dd
Version Independent ID: cbb2b875-b220-596c-7b4d-c2cf6c644335
Content: Deploy a Docker container app to an Azure web app
Content Source: docs/pipelines/apps/cd/deploy-docker-webapp.md
Product: devops
GitHub Login: @alexhomer1
Microsoft Alias: ahomer

Thank you for your feedback. I have passed this on to the product team for advice.

The normal web apps support authentication from a few identity provider by default configurable from the portal. This just means that you cannot use the authentication stuff on web apps for containers.
It doesn’t mean you can not implement the same in your web apps in containers, but you have to implement it yourself. authentication explained

this problem still persists.

Oddly, I am not even signed in with a work or school account (not that I know of anyway), it's just a<EMAIL_ADDRESS>Microsoft account.


add support for atoum 3.0

fixes #16

the https://github.com/atoum/config-extension/pull/12 needs to me merged a new version taged.

thnaks


Real time quits when an image is recieved

This one is kind of weird.  When an image link is recieved by the bot the rtm just quits no errors or anything.
If I restart it, it quits again and eventually floods itself off of slack ( or if i send a new message so the last message sent isn't an image any more )

i think i might have been causing this issue myself by 'returning' from inside of the real-time block!
I can no longer reproduce this bug after rubocop pointed out this issuee.

👍


ICN/FNCM configuration

Describe the solution you'd like
Automatically setup ICN/FNCM for additional features like Daeja license files, TeamSpaces, EntryTemplates, Social features
Additional context
Consider solutions with DB access, Public Java API, private REST API, UI automation.

Automatically enabled Daeja and Task Manager config in BAN now.


Instructions on adding external source repositories to the application

Amendment to the Guides

Rendered page preview

Rendered page preview

Should I add another column to the table with a list of all of the sources available in that repository? What do you think?

Yeah I think that would look good. Maybe also include the main repository included with Paperback? So there's an easy place to guide users to in case they ask what sites are available in Paperback.

Page has been adjusted


no need of place of supply for customers and vendors with tax treatment as Non GCC

[x] CONVERTING FROM DELIVERY CHALLAN TO INVOICE
[x]

issue fixed !


Unable to determine version

This might be related to #78 but that just got closed it seems.
We have enabled authentication on our Elk stack and now I can no longer run abc import, I get the following:
> abc-0.10.2 import --src_type=csv --src_uri=ff.csv --typename=_doc --transform_file=transform.js http://user:pass@elkhost:9200/index_ff_test --log.level=debug
INFO[0000] Created temp file pipeline_1556188021399291439.js
INFO[0000] ES Version:
panic: unable to determine version from http://elkhost:9200, bad status code: 403 [recovered]
        panic: Panic at 71: unable to determine version from http://elkhost:9200, bad status code: 403 [recovered]
        panic: Panic at 71: unable to determine version from http://elkhost:9200, bad status code: 403

goroutine 1 [running]:
github.com/appbaseio/abc/vendor/github.com/dop251/goja.(*Runtime).RunProgram.func1(0xc00048bb10)
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/vendor/github.com/dop251/goja/runtime.go:779 +0x9b
panic(0x1200a80, 0xc00040e0e0)
        /Users/siddharthlatest/.gvm/gos/go1.11/src/runtime/panic.go:513 +0x1b9
github.com/appbaseio/abc/vendor/github.com/dop251/goja.(*vm).try.func1(0xc000403ee0, 0x0, 0xc00048ba10, 0x0, 0x0, 0x0, 0xc00048ba68)
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/vendor/github.com/dop251/goja/vm.go:364 +0x459
panic(0x12e5140, 0xc00040a1e0)
        /Users/siddharthlatest/.gvm/gos/go1.11/src/runtime/panic.go:513 +0x1b9
main.(*Transformer).Save(0xc000497050, 0x1602880, 0xc0000b4180, 0xc00013e2c0, 0x3, 0xc, 0x0, 0x0)
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/cmd/abc/goja_builder.go:304 +0x494
reflect.callMethod(0xc000497740, 0xc00048b848)
        /Users/siddharthlatest/.gvm/gos/go1.11/src/reflect/value.go:663 +0x180
reflect.methodValueCall(0x1602880, 0xc0000b4180, 0xc00013e2c0, 0x3, 0xc, 0x0, 0x0, 0x1602880, 0xc000403ef0, 0xc000403ee8, ...)
        /Users/siddharthlatest/.gvm/gos/go1.11/src/reflect/asm_amd64.s:29 +0x33
github.com/appbaseio/abc/vendor/github.com/dop251/goja.(*vm)._nativeCall(0xc000403ee0, 0xc000495290, 0x3)
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/vendor/github.com/dop251/goja/vm.go:1824 +0x264
github.com/appbaseio/abc/vendor/github.com/dop251/goja.call.exec(0x3, 0xc000403ee0)
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/vendor/github.com/dop251/goja/vm.go:1808 +0x4d7
github.com/appbaseio/abc/vendor/github.com/dop251/goja.(*vm).run(0xc000403ee0)
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/vendor/github.com/dop251/goja/vm.go:288 +0x5d
github.com/appbaseio/abc/vendor/github.com/dop251/goja.(*vm).run-fm()
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/vendor/github.com/dop251/goja/vm.go:375 +0x2a
github.com/appbaseio/abc/vendor/github.com/dop251/goja.(*vm).try(0xc000403ee0, 0xc00048ba70, 0x0)
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/vendor/github.com/dop251/goja/vm.go:370 +0x10c
github.com/appbaseio/abc/vendor/github.com/dop251/goja.(*vm).runTry(0xc000403ee0, 0x145a890)
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/vendor/github.com/dop251/goja/vm.go:375 +0x4e
github.com/appbaseio/abc/vendor/github.com/dop251/goja.(*Runtime).RunProgram(0xc0000a8d80, 0xc000192360, 0x0, 0x0, 0x0, 0x0)
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/vendor/github.com/dop251/goja/runtime.go:790 +0x197
github.com/appbaseio/abc/vendor/github.com/dop251/goja.(*Runtime).RunScript(0xc0000a8d80, 0x0, 0x0, 0xc00007c000, 0x287, 0xc00007c000, 0xc00003c500, 0x287, 0xc00007c000)
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/vendor/github.com/dop251/goja/runtime.go:769 +0x9b
github.com/appbaseio/abc/vendor/github.com/dop251/goja.(*Runtime).RunString(0xc0000a8d80, 0xc00007c000, 0x287, 0x487, 0xc00007c000, 0x287, 0x0)
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/vendor/github.com/dop251/goja/runtime.go:758 +0x47
main.newBuilder(0xc0000c7de0, 0x1f, 0x6e1b8f, 0xc0000c41e0, 0x142697c)
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/cmd/abc/goja_builder.go:55 +0x321
main.execBuilder(0xc0000c7de0, 0x1f, 0xc0001c1d00, 0x1, 0x1)
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/cmd/abc/appbase_import.go:132 +0x39
main.runImport(0xc0000aa020, 0x6, 0x6, 0x6, 0xc00009c058)
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/cmd/abc/appbase_import.go:127 +0xe89
main.main()
        /Users/siddharthlatest/.gvm/pkgsets/go1.11/global/src/github.com/appbaseio/abc/cmd/abc/main.go:46 +0xdb

Is there some setting that my elk user needs to be able to get the version?

@hobbesuk Which version of ElasticSearch are you currently using this with? Any other info about your ES setup that can help us replicate the issue?

It is 6.x - I don't apparently have rights to get the exact version so I'm asking our admins

@hobbesuk We call the following endpoint to determine the version: GET http://user:pass@elkhost:9200/index_ff_test/_settings?human. The error above implies that the endpoint call returned a 403 error code. Can you make a cURL request and see if this route is accessible with the exact URI you passed to abc?

Yep - I get a 403 from that:
{
	"error": {
		"root_cause": [{
			"type": "security_exception",
			"reason": "action [cluster:monitor/main] is unauthorized for user [user]"
		}],
		"type": "security_exception",
		"reason": "action [cluster:monitor/main] is unauthorized for user [user]"
	},
	"status": 403
}


Not getting all the index based metrics

I am using the 10.4rc1/20180817 release candidate of the elasticsearch-exporter.
I have tried adding the option "-es.all" ("-es.indices"), but i couldn't get any metrics based on the index (e.g. documents per index or any similar metrics).
Do I need to enable anything else?
Requesting you to have a look at it and kindly let me know how to proceed further.

@zwopir Any suggestions for this issue?

Hi @biparida,
could you please try the latest RC (1.1.0rc1) with
./elasticsearch_exporter --es.uri http://<your-es-node>l:9200 --es.all --es.indices

and curl the endpoint on http://localhost:9114/metrics.

Hi @zwopir,
I am getting the below metrics. I am looking for metrics with segregation of individual metric.
elasticsearch_index_stats_indexing_index_total

Thanks
Bijay

Hi Bijay,
I don't quite understand what your question is, but I noticed that the index in the above query is logstash-2019.04.20 while it is logstash-2019.05.20 in the second one

Thanks Christoph, getting them after adding--es.indices. I want to get the sum of the counts dynamically.
Something like sum(elasticsearch_index_stats_indexing_index_total{cluster="$cluster",index="$index_name"})
But, I can't pass the dynamically, something like logstash-*
But, the below query works.
sum(elasticsearch_index_stats_indexing_index_total{cluster="$cluster",index="logstash-2019.05.22"})

you can use regexp label matching (see https://prometheus.io/docs/prometheus/latest/querying/basics/#instant-vector-selectors) for the index label. Something like sum(elasticsearch_index_stats_indexing_index_total{cluster="$cluster",index=~"logstash-2019.*"}) or whatever regexp fits your requirements

Thanks a lot. In the mean time I was able to figure out. I will wait for the latest stable release to be available to use it. Closing the issue.
sum(elasticsearch_index_stats_indexing_index_total{cluster=~"$cluster",index=~".*"}) by (index)


check_already_uploaded seems not work properly

Hi, I would like to know how mule-uploader handle the file have been uploaded completely when user select the same file again.
I saw the behavior on sample site (http://mule-uploader.com/) is:

Send request for singing key to backend
Send request to S3 and it response error NoSuchUpload, The specified upload does not exist. The upload ID may be invalid, or the upload may have been aborted or completed.
Send request for singing key with force parameter to backend.
Start a new uploading (it means s3 will have two same files)

Then I tried to remove the force parameter at my environment to see if it will handle the uploaded file checking, I got errors. The flow as below:

Send request for singing key to backend
Send request to S3 and it response error NoSuchUpload, The specified upload does not exist. The upload ID may be invalid, or the upload may have been aborted or completed.
Send request for singing key parameter to backend.
Two OPTION requests to S3 and response 200 ok
Two PUT requests to S3 and response 502 or 404
Show exception on console, Uncaught TypeError: Cannot read property 'region' of undefined line: 759 (it in check_already_uploaded function)

So, what the purpose of check_already_uploaded? How does it check the file uploaded? is this a issue or just un-support this kind of scenario?
And I got the same exception when uploading timeout, it seems the error_callback not work properly.
Its network har file: https://dl.dropboxusercontent.com/u/52456752/localhost.har
its console log: https://dl.dropboxusercontent.com/u/52456752/check_already_uploaded_error.log

Also seen this behaviour. Even if the upload finishes and there is some error, throws back an error on line 1020 "error_callback function undefined" or something like that. I don't think that's working properly.


[BEAM-13999] [Playground] support vertical orientation for graph

I added vertical orientation for graphs. (https://issues.apache.org/jira/browse/BEAM-13999)

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

LGTM thanks all!


Support metadataCharset on Project.zipTree()

Our Zip and Jar tasks support setting the metadata charset when creating archives. But when Gradle consumes a Zip file using Project.zipTree(), it always uses the platform encoding.
Expected Behavior
Project.zipTree() can be configured with the right metadata encoding.
Alternatives:

always use UTF-8 as the default (closer to the ZIP spec which doesn't really allow arbitrary encodings - though everybody ignores that)
use UTF-8 when the file extension is .jar, since the JVM expects jars to be UTF-8 encoded. This would mirror the default on the Jar task

Current Behavior
Project.zipTree() always uses the platform encoding, which breaks when a ZIP has been UTF-8 encoded and someone tries to expand it on Windows for instance.
Context
https://github.com/javaslang/javaslang/issues/1856

That's not what the Javadoc says for ZipFile:

Opens a ZIP file for reading given the specified File object.
The UTF-8 charset is used to decode the entry names and comments.

The version which accepts an encoding needs to be used when you read a Zip file which entries are not using UTF-8.

You're right, but our ZipFileTree uses org.apache.tools.zip.ZipFile instead, which uses the platform encoding by default.

Oh I missed the fact it wasn't the same ZipFile. Thanks for the clarification.

Here is the design doc we used to fix archive metadata charset handling in 3.x: https://github.com/gradle/gradle/blob/master/design-docs/handling-encodings.md
The approach we took was to:

use UTF-8 by default for JAR, WAR and EAR as using another charset is incorrect per the spec ;
keep using the default platform charset for ZIP metadata for backward compatibility reasons, using the Language encoding flag (or EFS).

Missing Project.zipTree was an overlook.
We should at least make it consistent with the archive tasks.
We could also consider using UTF-8 by default for ZIPs in 4.0 as a potentially breaking change.
Note that this also applies to Project.tarTree.


Seeking bar is broken in the frontend if cursor exceeds the bounds of the slider

What version of Music Assistant has the issue?
2.0.0b119
What version of the Home Assistant Integration have you got installed?
2024.3.4
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
When I try to use the seek bar in the music player, if my mouse cursor exceeds the bounds before letting go, nothing happens and the seek bar breaks until I drag it to another value. No server request gets made, nothing gets logged in neither the browser console nor server logs. This is especially a problem on mobile as you are way more likely to overshoot when you can't see the exact touch point.
How to reproduce
Play a song, then try to seek the music to any point. This should work fine, now drag the seek bar "beyond 0" (just go outside the bounds), now the seek bar will break.
https://github.com/music-assistant/hass-music-assistant/assets/25883885/65c412f0-7ca5-48e1-b1d8-899b5d766607
Music Providers
Issue happens regardless of music provider but I'm using Spotify and YouTube Music
Player Providers
I'm seeing this issue when playing to an AirPlay device but I don't believe this is player-specific as no server request gets made from the frontend in the first place (as shown by the console)
Full log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2024.3.3
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

@marcelveldt this is caused by @mouseup in https://github.com/music-assistant/frontend/blob/2c4dd24a489bbed556f02d4f6407aca5168ba202/src/layouts/default/PlayerOSD/PlayerTimeline.vue#L38
The mouseup will never be called when the mouse is outside of the component.
Since Vuetify 3.2.0 there are these @start and @end events that can be used instead of mouseup/mousedown.


Any NavLink label should be able to be any React component

🚀 Feature Proposal
Right now, NavLink labels can only be a string. They should be able to be any React component to allow for more customization and styling.
Motivation
Signed in as label in Navbar.
Technical Notes
export interface NavLink extends NavItem {
  /** The link name */
  label: React.ReactElement
  /** Adds a top border to the link as a list divider */
  dividerAbove?: boolean
  /** Inserts an icon to the left of the link when a valid icon name is passed */
  icon?: string
  /** A click handle which will redirect the user to whenever it is clicked */
  onClick?: (event: React.MouseEvent<any>) => void
  /** Determines the href */
  href?: string
}

:wave:
I've previously helped out here. Am I eligible to help with another good first issue ?
If not (or either way) , if it helps, here is the line I think needs to be modified.

@tehKapa i meant to tag you in the question as well^

@morrme you're more than welcome to take this issue! I've assigned you the issue

ok @jackcmeyer I've made the change. I don't see any tests that need to be modified.  Should one be added?

@morrme Probably a good idea to add tests if they are missing


Not able to send commands through pipeline

I have the following script to connect to my redis cluster:
from datetime import datetime

start_time = datetime.now().strftime('%Y-%m-%d %H:%M:%S')

startup_nodes = [{"host": "redis01-staging-tor", "port": "7000"}]
redis_client = StrictRedisCluster(startup_nodes=startup_nodes, decode_responses=True)
p = redis_client.pipeline()

p.command_stack = []

for i in range(1, 1000):
    p.command_stack.append((["SET", "foo{}".format(i), "bar{}".format(i)], {}))

p.execute()

end_time = datetime.now().strftime('%Y-%m-%d %H:%M:%S')

print("start_time: {}".format(start_time))
print("end_time:   {}".format(end_time))

When I attempt to run the script I get the following error:
Traceback (most recent call last):
  File "./test.py", line 17, in <module>
    p.execute()
  File "/Users/chat/anaconda3/lib/python3.6/site-packages/rediscluster/pipeline.py", line 98, in execute
    return self.send_cluster_commands(stack, raise_on_error)
  File "/Users/chat/anaconda3/lib/python3.6/site-packages/rediscluster/utils.py", line 101, in inner
    return func(*args, **kwargs)
  File "/Users/chat/anaconda3/lib/python3.6/site-packages/rediscluster/pipeline.py", line 144, in send_cluster_commands
    attempt = sorted(stack, key=lambda x: x.position)
  File "/Users/chat/anaconda3/lib/python3.6/site-packages/rediscluster/pipeline.py", line 144, in <lambda>
    attempt = sorted(stack, key=lambda x: x.position)
AttributeError: 'tuple' object has no attribute 'position'

Is there a bug in the code?  I followed the example from the following:
http://redis-py-cluster.readthedocs.io/en/master/pipelines.html#multi-exec-cluster-test-code

The main issue with your example is that it is not a Example code piece on how to use and code pipelines. Appending directly to the command_stack variable is a very low level solution that should not be used unless there is some unsupported feature/command in the lib that you require, then you should use the command stack. The correct way is to create a pipeline object and use the normal SET/GET methods like you would the normal StrictRedisCluster object. The only real error here is that the docs is out of date.

Thanks.  Your suggestion worked.


Detected concurrency problem with Query Gating

There's a problem while trying to save two changes on the same Document concurrently.
I've simplify the issue into two tests as you can see in this PR at my YesSQL fork. The fork is  created now for showing you the problem:
https://github.com/Jag84/yessql/pull/1
If you have Query Gating enabled and you try to update a document concurrently, things couldn't work as it is expected. It seems sometimes document stored contains changes from another not commited changes. It causes that a retry operation attemps to remove an element which failed due to concurrent exception and it fails cause the element doesn't exist (despite its original delete operation failed due to a concurrent exception).
If you disable Query Gating, everything works fine.
If you want to check it by yourself, you only need to clone and run both ShouldUpdateConcurrentDocumentsWithQueryGating and ShouldUpdateConcurrentDocumentsWithoutQueryGating tests.

I can solve the failure removing the element that doesn't exist returning a copy of the Document on ProduceAsync. The changed method would be something like this:
  internal async Task<T> ProduceAsync<T, TState>(WorkerQueryKey key, Func<TState, Task<T>> work, TState state)
  {
      if (!Configuration.QueryGatingEnabled)
      {
          return await work(state);
      }

      object content = null;

      while (content == null)
      {
          // Is there any query already processing the ?
          if (!Workers.TryGetValue(key, out var result))
          {
              // Multiple threads can potentially reach this point which is fine
#if !NET451
              // c.f. https://blogs.msdn.microsoft.com/seteplia/2018/10/01/the-danger-of-taskcompletionsourcet-class/
              var tcs = new TaskCompletionSource<object>(TaskCreationOptions.RunContinuationsAsynchronously);
#else
              var tcs = new TaskCompletionSource<object>();
#endif

              Workers.TryAdd(key, tcs.Task);

              try
              {
                  // The current worker is processed
                  content = await work(state);
              }
              catch
              {
                  // An exception occurred in the main worker, we broadcast the null value
                  content = null;
                  throw;
              }
              finally
              {
                  // Remove the worker task before setting the result.
                  // If the result is null, other threads would potentially
                  // acquire it otherwise.
                  Workers.TryRemove(key, out result);

                  // Notify all other awaiters to return the result
                  tcs.TrySetResult(content);
              }
          }
          else
          {
              // Another worker is already running, wait for it to finish and reuse the results.
              // This value can be null if the worker failed, in this case the loop will run again.
              content = await result;

              // CREATED CONTENT COPY HERE
              if (!(content is int))
              {
                  content = Configuration.ContentSerializer.Deserialize(Configuration.ContentSerializer.Serialize(content), content.GetType());
              }
          }
      }

      return (T)content;
  }
}

Even so, the concurrency problem keeps happening.

It has sense that query gating can produce unexpected problems with two threads reading same document from the db if both want to modify it, reason is they will share same object in memory and both can apply changes to the same object. So, no matter if one thread receives a concurrency exception storing object in db because the other will succeed storing it in db and that one will store the shared object that will contain both modifications.
So, it seems sensible what you did on your second comment to create a clone of the document to return for those workers waiting a result. And for sure this has fixed some problems. What I don't get is what else could be causing that after that at the end of the test not all de nationalities in the document has been removed

I've found time to come back to this and I've found what is causing the bug.
Problem is the first thread adding an object in Workers for a key is sharing with others a reference to result that it is returned by the method. So, this thread after returning the object changes the object that is the same other threads are reciving. So, no matter if the serialize and deserialize it to work with a new copy of the object. Problem is the original object they receive has been changed and it is not the same in database.
If you want to provide me write access to your repo I can make there changes to your pr to make it work

Added the pr to orchard core with original @Jag84 code and my fix

Fix in on myget if you want to try it


Continous auto focus mode for android camera

Solution adapted from https://github.com/mbppower/CordovaCameraPreview/issues/53#issuecomment-137770646

Looks good to merge.


How to create a customized image.

Hi,
Could I have some guide about building an image like "pre-configured Cloudbreak deployer image"? Because we need to use vmdk as our image, need know what components needs to install in the image. Thanks a lot!
Xin

Hi,
The  "pre-configured Cloudbreak deployer image" for other cloud providers are basically:

install docker
install Cloudbreak Deployer
prepull the docker images, so cbd start is quick

So follow the steps below:
Get Docker
You have 3 options:

Use the docker image from Marketplace
Use docker-machine to create an instance on azure
Do it all yourself: start from whatever image, install docker yourself.

Of course i would recommend to use the docker image from marketplace. You just click on the
big green Create Virtual Machine and follow the instructions. There is also docs guiding you.
Install Cloudbreak Deployer
You can read the on-prem docs. But once you have the docker instance from the previous step, its just an unzip:
curl -Ls public-repo-1.hortonworks.com/HDP/cloudbreak/cloudbreak-deployer_1.1.0_$(uname)_x86_64.tgz | sudo tar -xz -C /bin cbd

To check if the installation is succesfull:
$ cbd --version
Cloudbreak Deployer: 1.1.0

[Optional] Warm-up with docker images
This is optional, you can skip it safely, it just makes the first start quicker.
mkdir cloudbreak-deployment
cd $_
cbd init
cbd start

This will download all the docker images needed for CloudbreakDeployer. But if ypu want to create a reusable image, Yo have to delete the Profile as it contains the actual ip address, which wont match new instances starting from that image.
rm Profile

You can create an image from this instance


Don't always display "residue" as "modified residue"

to discuss

I found the code that's doing this but it has turned into a can of worms.  I've found a couple of areas of code that only worked by accident.  When I started tried to fix the residue display those bits of code failed in odd ways.  I think I've worked things out though.
So currently most residue properties in Chado get shown as fake modified_residue() extensions in the website.  This means there is a bit of special case code and config for residues (which is what was only working by accident).
Residues on protein binding annotations are handled as a different special case.  They are turned to "binding site" like here:
https://www.pombase.org/gene/SPAC31A2.16
As a test I changed the code and config locally to show the residues as residue() extensions on the web pages.  That fixed the problem with GO:0052723 above.
Only two other GO annotations were changed by this.
There's a second annotation on the asp1 / SPCC1672.06c page:
ATP hydrolysis activity  modified residue 1-385

For https://www.pombase.org/gene/SPAC22E12.09c
serine-type endopeptidase activity  modified residue S371

Should those be "residue" or "modified residue"?
I've added configuration for the protein modification annotations that have residues so they still display as "modified residue" after the changes.
And I've configured the FYPO annotations that have a residue property in Chado to display as "modified residue".
Sorry, long comment.  I'll commit the changes soon and check the website on Saturday.

I'll commit the changes soon and check the website on Saturday.

I've done some more testing and code-tidying.  The code and config changes are committed for tonight's load.

serine-type endopeptidase activity  modified residue S371
IN this one S371 is an active site residue so just "residue". It might look a bit weird because an active site is more than one residue, but it's fine for now.
We don't really curate active sites etc, but we could probably do them as protein features.
http://www.sequenceontology.org/browser/current_svn/term/SO:0001104
we have talked about importing protein features from external sources
https://github.com/pombase/pombase-chado/issues/1126
to display in the protein feature viewer etc, so we should see what  formats and standards they use.

It now says "residue" on the inositol hexakisphosphate 1-kinase activity (GO:0052723) page:
https://www.pombase.org/term/GO:0052723


Update flobject.py

Updating for consistency.

Implemented in this PR already. https://github.com/ansys/pyfluent/pull/1985


[Feature]  - Addition of Blobs Button

Description
https://github.com/Rakesh9100/Beautiify/assets/144141216/0f1b928c-07b9-4dd2-b17e-c17173bdc8a9
I want to add a button with blob effect in buttons section . This is the demo of button
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[X] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Proceed @ayusharyann


ami-iit: Add scrum masters of ergocub-related teams to ami-iit/mech

As they may want to generate URDF models, see https://github.com/icub-tech-iit/ergocub-software/pull/229#discussion_r1533641951 .

fyi @S-Dafarra @GiulioRomualdi


Fixed one code block in docs

On Combining providers page of the docs site, the following line:
switch (ref.watch(filterProvider)) {

which belongs to filteredTodoListProvider code block would give an error:
Type 'StateController<Filter>' of the switch expression isn't assignable to the type 'Filter' of case expressions.
so, I changed that code block to:
  final filteredTodoListProvider = Provider<List<Todo>>((ref) {
    final filter = ref.watch(filterProvider);
    final todos = ref.watch(todoListProvider.state);

    switch (filter.state) {
      case Filter.none:
        return todos;
      case Filter.completed:
        return todos.where((todo) => todo.completed).toList();
      case Filter.uncompleted:
        return todos.where((todo) => !todo.completed).toList();
    }
  });

Simply adding .state after ref.watch(filterProvider) in switch would solve the problem, but I chose the fix that was in accordance with the official Todo list example.

Nice catch, thanks!


Firebase.Performance missing the sessions feature

The current Xamarin binding (<IP_ADDRESS>) uses the global version 5.8.1 of Firebase Performance monitoring.
We would like to leverage the sessions feature added in version 5.11.0 (October 2018).
Here is a more detailed explanation of the feature.
Are there any plans for a version bump? Is that something you've looked into yet?
Latest version is 6.3.0, but I suppose anything above 5.11.0 would enable that feature automatically.
So far we've had good results with the performance monitoring on iOS, we have network metrics and automatic metrics (app_start etc.) working well.

Firebase v6.5.0 is live now, Performance Monitoring should have the requested feature now. Closing the issue now.


A CI/CD Observability Solution Suggestion

Hi pipenv community,
This is Serkan, Co-Founder & CTO at Thundra, the company behind Foresight.
I am here with the suggestion of Matt Davis and Oz Tiram for our CI/CD observability tool, Foresight.
Foresight brings visibility into CI workflows and test runs, enabling developers to debug CI, build, and test failures and latencies quickly. Furthermore, Foresight’s change impact analysis helps you analyze the impact of the code changes to detect defects early in the release cycle.
With Foresight, you can set rules that will help you to set up robust and customizable policies for your workflows and tests. Receive notifications for fluctuations in key performance metrics as your workflow runs.
Foresight is 100% FREE for open-source projects and I think Foresight can be useful for a repository contributed by many developers, such as pipenv.
Some of the prominent features of Foresight are:

Monitor and optimize your Github Actions Workflows (CI/CD pipelines)

Identify and debug failing, slow and unreliable tests

Automatically assess the level of risk of software changes

Why Foresight for open-source projects

Foresight supports "tokenless authentication" for open-source projects.
So when contributors open PRs to this repository from their forks and workflow run is triggered, collected workflow, test and telemetry data (metrics, logs, etc ...) will be associated to the Foresight account of this repository.
This means that, all the workflow runs from this main repo and its all forks will be able to monitored in a single Foresight account without need to share any credentials.

Foresight also supports custom sub-domain for open-source projects (for ex: it might be pipenv.app.runforesight.com for this repository) and you can share your workflow run and test metrics/insights with all your community as readonly in public without requiring everyone to signup Foresight.

Here are some examples of popular Github projects having Foresight public view with their own custom sub-domains:

Keycloak (https://github.com/keycloak/keycloak): https://keycloak.app.runforesight.com
mathjs (https://github.com/josdejong/mathjs): https://mathjs.app.runforesight.com
statsviz (https://github.com/arl/statsviz): https://statsviz.app.runforesight.com

Best regards,
Serkan

I believe this issue can be closed as we are integrated with Foresight metrics.


Can't install CentOS 6

I'm trying to install CentOS 6. Fetching the filesystem works, but I get an error message during installation (see below). It is the same error message that you got here under "Installing the new rootfs".
If I try to start WSL with bash.exe afterwards, nothing happens. No error message, it just returns me to the prompt.
The issue seems to be that the old CentOS version's adduser doesn't take the --quiet parameter, and that trips up lxrun.exe. However /etc/passwd and /etc/shadow seem to be edited correctly. Any ideas how to get this running?
> python get-source.py centos:6
> python install.py centos:6
[*] Probing the Linux subsystem...
[*] Default user is jason at /home/jason.
[*] Switching default user to root...
[*] Reading /etc/{passwd,shadow,group,gshadow} entries for root and jason...
[*] Copying rootfs_centos_6.tar.xz to /root/rootfs-temp...
[*] Beginning extraction...
[*] Waiting for the Linux subsystem to exit...
[*] Backing up current rootfs to rootfs_ubuntu_trusty...
[*] Switching to new rootfs...
[*] Writing entries of root and jason to /etc/{passwd,shadow,group,gshadow}...
[*] Switching default user back to jason...
/usr/sbin/adduser: unrecognized option '--quiet'
Usage: adduser [options] LOGIN
       adduser -D
       adduser -D [options]

Options:
  -b, --base-dir BASE_DIR       base directory for the home directory of the
                                new account
  -c, --comment COMMENT         GECOS field of the new account
  -d, --home-dir HOME_DIR       home directory of the new account
  -D, --defaults                print or change default useradd configuration
  -e, --expiredate EXPIRE_DATE  expiration date of the new account
  -f, --inactive INACTIVE       password inactivity period of the new account
  -g, --gid GROUP               name or ID of the primary group of the new
                                account
  -G, --groups GROUPS           list of supplementary groups of the new
                                account
  -h, --help                    display this help message and exit
  -k, --skel SKEL_DIR           use this alternative skeleton directory
  -K, --key KEY=VALUE           override /etc/login.defs defaults
  -l, --no-log-init             do not add the user to the lastlog and
                                faillog databases
  -m, --create-home             create the user's home directory
  -M, --no-create-home          do not create the user's home directory
  -N, --no-user-group           do not create a group with the same name as
                                the user
  -o, --non-unique              allow to create users with duplicate
                                (non-unique) UID
  -p, --password PASSWORD       encrypted password of the new account
  -r, --system                  create a system account
  -R, --root CHROOT_DIR         directory to chroot into
  -s, --shell SHELL             login shell of the new account
  -u, --uid UID                 user ID of the new account
  -U, --user-group              create a group with the same name as the user
  -Z, --selinux-user SEUSER     use a specific SEUSER for the SELinux user mapping

[!] Failed to switch default user in WSL: Command '['cmd', '/C', 'C:\\Windows\\System32\\lxrun.exe', '/setdefaultuser', 'jason']' returned non-zero exit status<PHONE_NUMBER>
Error in atexit._run_exitfuncs:
Traceback (most recent call last):
  File "install.py", line 119, in switch_user_back
    subprocess.check_output(['cmd', '/C', lxpath + '\\lxrun.exe', '/setdefaultuser', user])
  File "C:\Users\jdman\AppData\Local\Programs\Python\Python35\lib\subprocess.py", line 626, in check_output
    **kwargs).stdout
  File "C:\Users\jdman\AppData\Local\Programs\Python\Python35\lib\subprocess.py", line 708, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['cmd', '/C', 'C:\\Windows\\System32\\lxrun.exe', '/setdefaultuser', 'jason']' returned non-zero exit status<PHONE_NUMBER>

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "install.py", line 123, in switch_user_back
    exit(-1)
  File "C:\Users\jdman\AppData\Local\Programs\Python\Python35\lib\_sitebuiltins.py", line 26, in __call__
    raise SystemExit(code)
SystemExit: -1

This is actually an issue with WSL, since lxrun is the one running the adduser --quiet command.
One possible workaround for this would be to create a hook script which replaces adduser during installation with a wrapper that removes the unsupported --quiet argument. That, or it updates adduser to one which supports it.

Why not use CentOS 7?

@thebitstick I'm trying to run some legacy software I use at work. The programs exist in binary form and run on CentOS 6 (well, Scientific Linux 6 which is the same). On every other OS like Centos 7 or Ubuntu, I would have to do a lot of hacks with LD_LIBRARY_PATH and so on. It would be very complicated to compile everything from scratch.
@RoliSoft It's tricky... the hook script only runs after switching to the new OS, which is too late. I can't run bash.exe do do anything with CentOS. I am trying to replace adduser during install.py, but that is quite hard. lxrun doesn't run adduser if it is a script, so I have to compile a binary (on another CentOS 6 system). Also, on CentOS, adduser is a link to useradd, which is a different program on Ubuntu...

@jdmansour @RoliSoft you could try adding to the install.py something that checks if installing CentOS 6 or other out-of-date distros and change the command from adduser --quiet to to adduser

The adduser command gets called by lxrun, not directly by the installation script. Anyway, I believe this is not the core problem. I can sidestep this issue by using lxrun /setdefaultuser root before running the install script. In this case there is no error message, but I still can't enter WSL. bash.exe doesn't work, nor does anything like bash.exe ls. Strangely enough, lxrun.exe can successfully start binaries (like adduser, usermod, passwd) in the CentOS image, but bash.exe refuses to do anything, and gives no error message.

@jdmansour Writing to files outside of WSL is very tricky indeed. One thing I found that mostly works, is to not create/delete the file, but rather append to it.
Truncation works with some applications, but not with others. I haven't been able to consistently and successfully truncate files and replace their content from outside of WSL yet.
That being said, files like /etc/passwd are truncated and overwritten by the installer script, and they seem to work.
It would really save a lot of trouble if Microsoft would release some documentation or expose new APIs to write VoIFs metadata to regular files.

I'm running into the same problem, for the same reasons (legacy compatibility).
I wounder if this can be bypassed by copying the centos:6.x root under another minimal root and then chroot-ing into the centos:6.x directory?


Add basic bash completion

Adds basic bash completion for primary and subcommand options per #4

Thank you for the PR.
Can you please update the README.md with the completion installation instruction?

I pushed a commit to install the completion file as part of the package. I believe this should work on most platforms, but I have not tested on OSX yet.

Thank you


False positive syntax error FirebaseRulesTokenType.SERVICE_KEYWORD

I installed the plugin in Webstorm with the version 1.1.0-SNAPSHOT and I get one false positive error on valid rules.
I got currently only one false positive syntax error when I write the following valid rules:
rules_version = '2';

function someFunction() { // Error: "FirebaseRulesTokenType.SERVICE_KEYWORD expected, got 'function' "
  return false;
}

service cloud.firestore {
  ...
}

Same error occurs when I write
rules_version = '2';

// some comment  // Error: "FirebaseRulesTokenType.SERVICE_KEYWORD expected, got '// some comment' "
service cloud.firestore {
  ...
}

P.S. As overall however it is what I was looking for. Good job. :)

Thanks for testing, The changes gonna be available for version: 1.1.1


Certificate verification issue while enabling metrics and trace type of log data for AKS cluster

Discussed in https://github.com/signalfx/splunk-otel-collector-chart/discussions/1087

Originally posted by ajaykumarrc December 18, 2023
i have installed splunk otel connector on AKS cluster version 1.25.15  after enabling metric type logs am getting the below error any workaround for the below error.
error   kubeletstatsreceiver@v0.91.0/scraper.go:77      call to /stats/summary endpoint failed  {"kind": "receiver", "name": "kubeletstats", "data_type": "metrics", "error": "Get \"https://1xx.2xx.xx.xxx:10250/stats/summary\": tls: failed to verify certificate: x509: cannot validate certificate for 1**.2***.**.**03 because it doesn't contain any IP SANs"}

Hey,
I haven't seen this issue so far, but this comes from the kubeletstats receiver. You can check the docs to check how to configure tls. I think as a dirty workaround I'd try to overwrite it first with the following config:
agent:
  config:
    kubeletstats:
      auth_type: serviceAccount
      insecure_skip_verify: true
      collection_interval: 10s
      endpoint: ${K8S_NODE_IP}:10250
      extra_metadata_labels:
      - container.id
      metric_groups:
      - container
      - pod
      - node

This is the default config with  insecure_skip_verify: true option

Closed as complete. Thanks Olga!

can we add insecure_skip_verify as an option in values.yaml with default false?
so that we don't need to add it as a workaround


High CPU useage

Hi,
When I use the software I get a around 20% of my desktop Quadcore i5 being used up. Is this what other users are experiencing?
Thanks.

I've also got a quad core i5 and I see about 1% CPU usage for Precision Gaze Mouse and 5% for Enable Viacam when I'm actively using it. Are you using the original EyeX or 4C? The 4C has an onboard ASIC chip to process eye gaze with very low CPU. Also, make sure the "Show Warp Bar" and "Show gaze tracker" checkboxes are turned off because they each use extra CPU.

I haven't got an EyeX to compare with, but for what it's worth I pretty regularly see high CPU usage from Tobii's 4 processes when using my 4C, which often persists even after PrecisionGaze is turned off or exited.
I haven't tried to prove it but I can't help but feel that Tobii's software is doing needless work.. possibly related to all the other not-so-helpful features the software provides, such as choosing alt-tab targets by gaze (or highlighting its tray pop-up menu via gaze).
EViacam is definitely more CPU hungry though!

I am not sure why it still consumes a high amount of CPU for me. I started using the talon eye tracking and now the CPU usage is extremely low. I also have a strong feeling that the Tobii driver is very wasteful.

Interesting I know the Talon eye tracker has a custom driver the author wrote himself so maybe it bypasses some issue with the Tobii driver. That still doesn't explain why CPU usage is low for me but not you. Perhaps there is some system level difference? For what it's worth I do have most of the optional interaction features turned off.
If the problem is with PrecisionGazeMouse and not Tobii, you could run it in the debugger and see how much CPU is spent in various functions https://docs.microsoft.com/en-us/visualstudio/profiling/profiling-feature-tour?view=vs-2019. That requires some development experience though, and spare time to run the test.

Closing since it seems this requires more information to fully debug. If you do find where the CPU is being used please reopen.


pull request template

creates pull request template
suggestions on what to add or remove are highly welcome !
linked it on the main README, so it'd be easy for new contributors to see and use.

Hi Rose 👋🏾 @zoobot
I'm sorry once again for getting back late ! being super swamped with my exams (school exams now, bootcamp's own done)
but uhm, will include your suggestion now !

Okay then, will do !
Thank You !

I just squash and rebase my commits, but I still see a bunch of commit from before.
not sure how it works.
ready for review now !, suggestions on what to change or remove are welcome ! @zoobot

thanks a lot, this helps @zoobot
will follow your workflow next time to avoid unnecessary merge commits !
I will just go ahead and re-create my branch from an up to date main and add my files again


Search for posts not finding posts with specific keywords

Problem
Although we recently introduced search capability in #66, the current design implementation has some limitations that lead to a poor user experience. To name a few examples:

The search only works for top level parent posts, meaning that users are unable to search for child level posts
The fuzzy search sometimes omits explicit terms
The search experience is slow and results in freezes such as when searching for long text queries

User Story
As a DevHub user, I want to search for any posts that mention a specific topic or keyword even if I don't know the exact phrasing or spelling.
Acceptance Criteria

The search works for any posts (top level or child)
The search function uses fuzzy matching to return posts that match the search term, including partial matches and misspelling. The search should always prioritize the explicit terms first
The search results should load quickly
The search function displays results as they are found, allowing users to see posts matching the search in real-time

Example of searching for terms in child posts
To reproduce:
Here's an example of child terms that get omitted:

Log into the DevHub as theori.near
Filter by zero-knowledge labels and the devgovgigs.near author
Search for "typically"
Search output returns: "No posts matches search"

Expected behavior:
Search output should reveal 2 posts that match these conditions (post 1, post 2)
Desktop:
OS: OS
Browser Chrome
Version: 112.0.5615.49
Screenshots:

https://user-images.githubusercontent.com/110252255/233517605-6e96a58f-371f-4a32-a223-e5378aab7523.mp4

@ori-near Just for context, both of the posts are not top-level. The solution we currently have requires downloading all the posts to the client side, so it was decided to only use top-level posts for search, which I highlighted as a limitation in the PR review https://github.com/near/devgigsboard-widgets/pull/85#issuecomment-1500202212. I will leave it to @ailisp to evaluate the best path forward, but I feel that soon we will need a dedicated server to serve search queries.

@frol 's observation is correct. Searching for typically finds top level posts, which is expected behavior. If you find some keyword where top level posts are not found in search result, please feel free to reopen this ticket!

I would still keep this issue open, so we can get back to it in the future

Done. Note that server side processing is required to complete this task. So I create a separate task near/neardevhub-contract#21, which this task is consider blocked by.


Magic 8 Ball

Project title
Magic 8 Ball
Project Description
It gives you a solid foundation for how to use Math.random to produce randomized results .
You can use this project to impress your friends and make small life decisions.
Stack:
And in order to tick the check box just but x inside them for example - [x] like this. Please delete options that are not relevant.

[x] Html
[x] CSS
[x] JavaScript
[ ] ReactJS

/assign


[TC-SC-5.2]  Test step label has to be updated as per Test Plan

Description:
TC-SC-5.2 - Test step label has to be updated as per Test Plan In Steps 1, 10.
Steps to reproduce:
Step 1, 10
Script References:

https://github.com/project-chip/connectedhomeip/blob/master/src/app/tests/suites/certification/Test_TC_SC_5_2.yaml

Test Plan References::

https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/group_communication.adoc#test-procedure-4

@Ashwinigrl fixed in PR 30869 please verify it and close the ticket.

Issue Fixed in PR https://github.com/project-chip/connectedhomeip/pull/30869


ability to provide secret to agent and generate token

Very nice. This will simplify our demos a bit.

Awesome.

We should add this to the docs!

yes, I've got an updated version of the 0.5 docs I'll push tomorrow. Trying to finalize the docs for a release!


Terminal comma to indicate discarded items in array destructuring fails Squiz.Arrays.ArrayDeclaration.CommaAfterLast

Describe the bug
Array destructuring is a syntax in PHP where items of an array are assigned to single variables, using an array construct on the left of the assignment, e.g.
[$foo, $bar, $baz] = $source_array;

It's a common pattern to include a terminal comma to indicate that we know not all of the array's elements are being assigned.
E.g. in Composer:

                        [$response, ] = $spec;
                    [$defaultUsername, ] = explode(':', $authParts, 2);

However, this fails the Squiz.Arrays.ArrayDeclaration.CommaAfterLast sniff:
124 | ERROR | [x] Comma not allowed after last value in single-line array
     |       |     declaration (Squiz.Arrays.ArrayDeclaration.CommaAfterLast)

Code sample
<?php

[$alpha, ] = explode('/', 'alpha/beta');

To reproduce
Steps to reproduce the behavior:

Create a file called test.php with the code sample above...
Run phpcs --standard=Squiz  --sniffs=Squiz.Arrays.ArrayDeclaration test.php
See error message displayed

 3 | ERROR | [x] Comma not allowed after last value in single-line array declaration

Expected behavior
A clear and concise description of what you expected to happen.
Versions (please complete the following information)

Operating System
MacOS 10.15

PHP version
8.3

PHP_CodeSniffer version
3.10.1

Standard
Squiz

Install type
Composer

Additional context
Add any other context about the problem here.
Please confirm

[x] I have searched the issue list and am not opening a duplicate issue.
[x] I have read the Contribution Guidelines and this is not a support question.
[x] I confirm that this bug is a bug in PHP_CodeSniffer and not in one of the external standards.
[ ] I have verified the issue still exists in the master branch of PHP_CodeSniffer.

@joachim-n Thank you for reporting this. This is due to the sniff incorrectly acting on short lists.
The Squiz.Arrays.ArrayDeclaration sniff is known to be problematic and has been for years.
I've been building up a (highly configurable) NormalizedArrays standard in PHPCSExtra to replace it, but that's not complete yet. Would be lovely if I could find some time to finish that at some point.
Having said that, you could consider excluding the Squiz.Arrays.ArrayDeclaration.CommaAfterLast error code and using the PHPCSExtra NormalizedArrays.Arrays.CommaAfterLast sniff to replace it. That sniff does ignore short lists correctly.

Loosely related to #527

Thanks for the super quick response!
I've passed on the advice about using a different sniff to the issue in Drupal that brought me here - https://www.drupal.org/project/coder/issues/3456683

@joachim-n Can this issue be closed ?

Probably, though if the replacement isn't ready yet, it might be an idea to add some docs to Squiz.Arrays.ArrayDeclaration to explain its limitations.


Gov module restore (+test fixes)

Description
This PR restore gov module and its tests + fix the tests that didn't work on the PR #184.
Closes #173.
Checklist

[x] Targeted PR against correct branch.
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Wrote unit tests.
[x] Wrote integration tests (simulation & CLI).
[x] Updated the documentation.
[x] Added an entry to the CHANGELOG.md file.
[x] Re-reviewed Files changed in the Github PR explorer.

I notice that we're testing with 2mln (for report and profile modules) and 5mln (with posts) of default gas values. Strangely these values cause the failure of some sim tests and I cannot reproduce them locally because those tests passed without error on my machine. So I’ve opted to try with the regular value that we’re using on chain so (200K for report a profile) and (800k for posts which has some scenario where the gas is around 650k.


Jump instead of expand when trigger is present on the snippet

I have been experimenting writing my own snippets for commit messages (so I can learn lua snippets mostly) but I have found a problem that I can't find a proper solution.
I have this snippet:
s( { trig = "feat", name = "feature", dscr = "add new feature to the user" }, fmt("feat{}: {}", { c(1, { t "", fmt("({})", i(1, "scope")) }), i(2) }) ),
I expand it with feat and then I have feat|: (| is the cursor), and If I try to jump to the second node it will expand the snippet again since feat is the snippet trigger.
Is there a workaround?, I can change the trigger to something else but I would like to keep it like it is.

I've had this problem myself, I think the best solution is to get used to distinct jump and expand-keys.
You could try adding some heuristic to determine whether to jump or expand, but I don't think anything there can work in all scenarios.

I've had this problem myself, I think the best solution is to get used to distinct jump and expand-keys. You could try adding some heuristic to determine whether to jump or expand, but I don't think anything there can work in all scenarios.

Thanks for your answer.
I've ended with this, which works apparently fine for my specific needs in this snippet.
local isNested = false return {     -- Feature commit     s(         { trig = "feat", name = "feature", dscr = "add new feature" },         fmt("feat{}: {}", { c(1, { t "", fmt("({})", i(1, "scope")) }), i(2) }),         {         condition = function()             if isNested == true then                 return false             end             return true         end,         callbacks = {             [1] = {                 [events.enter] = function(node, _event_args)                     isNested = true                 end,             },         },     }     ), ...
I may consider switching to the separate jump/expand keys, but for me it's too much right now (previously I used Tab to almost everything -cmp, snippets-)

Ahh, you'll just have to reset isNested on leave.
So
callbacks = {
    -- -1 refers to the snippet itself
    [-1] = { events.enter = set, events.leave = reset}
}

I think that should do it, you could extract it into a function for quickly creating snippets which you know won't be nested.

You're welcome :)


optimize and move sqrt to sdk-core

right now the sqrt function starts with a guess of 0, it should start with a guess that's more like the (1 << most significant bit divided by 2)
https://github.com/Uniswap/uniswap-v3-sdk/blob/87efce041d58bb43e4ead1beb139ddb8a5886137/src/utils/sqrt.ts#L5
once optimized, it should be moved to sdk-core, and incorporated to the v2-sdk (remove babylonianSqrt from the v2-sdk)
this function also needs comprehensive test coverage

done in https://github.com/Uniswap/uniswap-v3-sdk/commit/62c6ec552230ea89371ccedcf15da741498892f4


Passing untyped objects to parameters with custom types no longer works in 0.25.3

Bicep version
Bicep CLI version 0.25.3 (862708594a)
Describe the bug
After upgrading to 0.25.3 we discovered that passing in an untyped object to a parameter with a custom type no longer works, even if the untyped object has the required fields. Bicep produces an error directly in the editor if you try this.
To Reproduce
The following sample code mimics a setup we have been using successfully up until and including 0.24.24.
Create main.bicep:
module data 'data.bicep' = {
  name: 'data-module'
}

module app 'app.bicep' = {
  name: 'app-module'
  params: {
    identity: data.outputs.identity
  }
}

Create data.bicep:
resource identity 'Microsoft.ManagedIdentity/userAssignedIdentities@2023-01-31' existing = {
  name: 'my-identity'
}

output identity object = union(identity, { name: identity.name, id: identity.id })

Create app.bicep:
param identity {
  id: string
}

output id string = identity.id

In main.bicep you now have an error The specified "object" declaration is missing the following required properties: "id".bicep(BCP035):

The output from data.bicep is an untyped object with the following shape:
{
  "outputs": {
    "identity": {
      "type": "Object",
      "value": {
        "apiVersion": "2023-01-31",
        "condition": true,
        "existing": false,
        "id": "/subscriptions/<redacted>/resourceGroups/rg-identity/providers/Microsoft.ManagedIdentity/userAssignedIdentities/my-identity",
        "isAction": false,
        "isConditionTrue": true,
        "isExtensibleResourceReference": false,
        "isTemplateResource": false,
        "location": "swedencentral",
        "name": "my-identity",
        "properties": {
          "clientId": "<redacted>",
          "principalId": "<redacted>",
          "tenantId": "<redacted>"
        },
        "provisioningOperation": "Read",
        "referenceApiVersion": "2023-01-31",
        "resourceGroup": "rg-identity",
        "resourceGroupName": "rg-identity",
        "resourceId": "Microsoft.ManagedIdentity/userAssignedIdentities/my-identity",
        "scope": "",
        "subscriptionId": "<redacted>",
        "tags": {}
      }
    }
  }
}

Was this intentionally broken? Perhaps it was a bug that this was allowed, and now the bug is fixed? :)
Additional context
We are using this pattern a lot where we have "data modules" that expose various resources in similar ways. We use union(<resource>, { name: <resource>.name, id: <resource>.id }) since in general the id and name properties are not included otherwise.

+1

I have already reported this over here #13247

Fixed by https://github.com/Azure/bicep/pull/13272, thanks @cedricbraekevelt


How about quota-available-bytes and quota-used-bytes?

https://tools.ietf.org/html/rfc4331#page-3

No plans for this so far.

Any news on this? I am very interested in this so I can see total disk space and available disk space


Just two more tests

I'm sorry that I haven't responded as I have just now discovered your PR in the process of filing my own issue. Email me if you're still interested.

Your new tests were added into the v2.1.1 release.
Thanks! And sorry it took so long.


Remove jQuery, extend from common component, cleanup a bit

Mainly wanted to remove jQuery here, but went ahead and did some
cleanup as well, while I was making the changes.

nice stuff


SyntaxError: Unexpected token u    at Object.parse (native)

i received error after i put the codes in WelcomeCtrl.js
app.controller('WelcomeCtrl',['$scope','$localStorage',function($scope,$localStorage){
}])
SyntaxError: Unexpected token u
at Object.parse (native)
at fromJson (http://localhost:5000/app/libs/angular.js:1250:14)
at Object.p.$sync (http://localhost:5000/app/libs/ngStorage.min.js:1:1628)
at $get (http://localhost:5000/app/libs/ngStorage.min.js:1:1855)
at Object.invoke (http://localhost:5000/app/libs/angular.js:4473:17)
at http://localhost:5000/app/libs/angular.js:4290:37
at getService (http://localhost:5000/app/libs/angular.js:4432:39)
at Object.invoke (http://localhost:5000/app/libs/angular.js:4464:13)
at extend.instance (http://localhost:5000/app/libs/angular.js:9093:34)
at nodeLinkFn (http://localhost:5000/app/libs/angular.js:8205:36)
i also have app.js
var app=angular.module("myApp",["ngRoute","ngResource","ngStorage"]);
app.config(function($routeProvider){
$routeProvider.when("/create_project",{
templateUrl:"/app/components/project/create_project.html"
})
$routeProvider.when("/login",{
templateUrl:"/app/components/user/login.html"
})
$routeProvider.otherwise({
templateUrl:"/app/components/welcome/welcome.html"
})
});
app.controller('AppCtrl',['$scope',function($scope){
}])

:+1:

this unexpected token often occurs during parsing failures, got to investigate further. meanwhile, I'm struggling to make angular 1.4.0 work with it

I gues "u" in VM21539 angular.min.js:114 SyntaxError: Unexpected token u at Object.parse (native) is for "undefined"
You can acidentally save something weird to local storage like this:
angular.module('app', ['ngStorage'])
.controller('MyController', ['$localStorage', function($localStorage) {
  var vm = this;
  vm = $localStorage.$default({
    foo: 1,
    bar: 2
  });
  // ... more code ...
  vm.total = function total() { return 123;}
}]);

http://plnkr.co/edit/LcN3PkDCFtMnUkJByLCj?p=preview will fail on second run.
Probably an attempt to do this shouldn't pass silently.

I was able to fix it by removing prefix from config block
.config(['$localStorageProvider',
    function ($localStorageProvider) {
        //$localStorageProvider.setKeyPrefix('wf'); removing this fixed above error
    }])


Can't import in node Vue?

I'm trying to install and import this into a node Vue project and I keep getting an error that the module cannot be resolved.
My command to install:
npm install spin-wheel
My code to import:
import { Wheel } from "spin-wheel";
Error I get:
Module not found: Error: Can't resolve 'spin-wheel' in 'C:\Projects\Web\wheel-test\src'
I've checked package.json and it's definitely in there: "spin-wheel": "^4.3.0",. I'm getting the error before I've even begun to setup the config and initialize it. It can't even find the module. Is there a preferred way to import this that I'm missing?

I was importing incorrectly. The correct way to import is: import { Wheel } from "spin-wheel/src/wheel";

Hey @blkhwks19, thanks for raising this!
I did notice a simular error when building for production in a Vue 3 app. It was because there was no entrypoints defined in package.json. This has been fixed in v4.3.1 so hopefully it will make it easier to import and bundle now.


[BUG]: Search does not work properly

Expected Behavior
Search for something, see result.
Actual Behavior
Error: SQLSTATE[HY000]: General error: 1191 Can't find FULLTEXT index matching the column list (SQL: select `users`.`id`, `users`.`email`, `users`.`name`, `user_information`.`user_department` from `users` left join `user_information` on `users`.`id` = `user_information`.`user_id` where `users`.`email` like %Test% or `users`.`email` = SOUNDS LIKE or `users`.`name` like %Test% or `users`.`name` = SOUNDS LIKE or MATCH (users.name) AGAINST (\"+Test\" IN BOOLEAN MODE) or `user_information`.`user_department` like %Test% or `user_information`.`user_department` = SOUNDS LIKE or MATCH (user_information.user_department) AGAINST (\"+Test\" IN BOOLEAN MODE) or `users`.`id` = Test limit 4)
Steps to Reproduce the Problem

Search for something
See issue

Critical Bug?

[X] Yes
[ ] No

More Information
https://github.com/BerkeAras/WorkGroup-API/blob/62cbda68304baf5e742be04bd29b3cd5c147dee6/app/Http/Controllers/SearchController.php#L25-L38

Fixed: https://github.com/BerkeAras/WorkGroup-API/commit/84a8b2a50e8f182a680187684294ed2081a1dd98


[Bug]: Request Timeout

Bug expectation
I installed puppeteer on my server and created a script for generating a screenshot.
When I run the request I get this error: "Request Timeout
This request takes too long to process, it is timed out by the server. If it should not be timed out, please contact administrator of this web site to increase 'Connection Timeout'.".
Bug behavior

[ ] Flaky
[ ] PDF

Minimal, reproducible example
const express = require("express");
const puppeteer = require('puppeteer');

const app = express();

app.get("/pdf", async (req, res) => {

    const browser = await puppeteer.launch({
        headless: true,
      ignoreHTTPSErrors: true,
      args: [`--window-size=1920,1080`],
      defaultViewport: {
        width:1920,
        height:1080
      }
    });
    const page = await browser.newPage();
    await page.goto('https://www.youtube.com/');
    await page.screenshot({path: 'example.png'});

    await browser.close();

});

app.listen(6795, () => {
    console.log("Server started");
});

Error string
Request Timeout
This request takes too long to process, it is timed out by the server. If it should not be timed out, please contact administrator of this web site to increase 'Connection Timeout'.
Puppeteer configuration
No response
Puppeteer version
19.5.0
Node version
14.20.1
Package manager
npm
Package manager version
6.14.12
Operating system
Linux

Unable to reproduce. The problem is likely with your server environment: you need to make sure it has all dependencies for running chromium and that is has enough memory and cpu to launch and stop the browser within a single request.


Update service-fabric-linux-windows-differences.md

fixing the wording a bit.

@imangoodarzi,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@rwike77