Music Player - Step 58 Use the OR operator unclear

Describe the Issue
The current instructions are a little unclear
create a song constant and set it to the current song of userData, or the first song in the userData?.songs array.

It's not too clear if you are to set song to one of those, or if you are to use the OR operator in the assignment. I think there could be an instruction or clue to make this more clear:
create a song constant and use the OR operator ( || ) to set it to the current song of userData, or the first song in the userData?.songs array.

Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-basic-string-and-array-methods-by-building-a-music-player/step-58
Your code
const setPlayButtonAccessibleText = () => {

};

Expected behavior
I think there could be an instruction or clue to make this more clear:
create a song constant and use the OR operator ( || ) to set it to the current song of userData, or the first song in the userData?.songs array.

Additionally the error feedback and the console feedback is different. The feedback in the console is much more explicit. (See screenshot)
Screenshots

System

Device: e.g. iPhone 6, Laptop
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context
No response

That sounds like a good change to me.
I'll open it up for first timers only 👍

I will work on this right now.

Can someone tell me how to run this web application?

@huyenltnguyen freecodecamp website..

How can I run this locally?

@huyenltnguyen
I followed all the instructions to run this project locally, but when try to run the project with pnpm run develop command, I got this error.

form-group.tsx
import React, { createContext } from 'react';
import { FormGroupProps } from './types';

export type FormContextProps = Pick<
  FormGroupProps,
  'controlId' | 'validationState'
>;
export const FormContext = createContext<FormContextProps>({});

const defaultClasses = 'mb-3.5 relative';

export const FormGroup = ({
  className,
  validationState,
  controlId,
  as,
  ...props
}: FormGroupProps): JSX.Element => {
  const context = {
    controlId,
    validationState
  };
  const componentClass = as;
  const Component = componentClass || 'div';

  const classes = [defaultClasses, className].join(' ');
  return (
    <FormContext.Provider value={context}>
      <Component
        className={classes}
        as={as}
        id={controlId}
        {...props}
        validationstate={validationState}
      />
    </FormContext.Provider>
  );
};

../../../../../node_modules/@types/react/index.d.ts:3436:13 - error TS2717: Subsequent property declarations
same type.  Property 'rect' must be of type 'SVGProps<SVGRectElement>', but here has type 'SVGProps<SVGRectEl

3436             rect: React.SVGProps<SVGRectElement>;
                 ~~~~

  ../../node_modules/.pnpm/@types+react@16.14.51/node_modules/@types/react/index.d.ts:3322:13
    3322             rect: React.SVGProps<SVGRectElement>;
                     ~~~~
    'rect' was also declared here.

../../../../../node_modules/@types/react/index.d.ts:3437:13 - error TS2717: Subsequent property declarations
same type.  Property 'stop' must be of type 'SVGProps<SVGStopElement>', but here has type 'SVGProps<SVGStopEl

3437             stop: React.SVGProps<SVGStopElement>;
                 ~~~~

  ../../node_modules/.pnpm/@types+react@16.14.51/node_modules/@types/react/index.d.ts:3323:13
    3323             stop: React.SVGProps<SVGStopElement>;
                     ~~~~
    'stop' was also declared here.

../../../../../node_modules/@types/react/index.d.ts:3438:13 - error TS2717: Subsequent property declarations
same type.  Property 'switch' must be of type 'SVGProps<SVGSwitchElement>', but here has type 'SVGProps<SVGSw
3438             switch: React.SVGProps<SVGSwitchElement>;
                 ~~~~~~

  ../../node_modules/.pnpm/@types+react@16.14.51/node_modules/@types/react/index.d.ts:3324:13
    3324             switch: React.SVGProps<SVGSwitchElement>;
                     ~~~~~~
    'switch' was also declared here.

../../../../../node_modules/@types/react/index.d.ts:3439:13 - error TS2717: Subsequent property declarations
same type.  Property 'symbol' must be of type 'SVGProps<SVGSymbolElement>', but here has type 'SVGProps<SVGSy
3439             symbol: React.SVGProps<SVGSymbolElement>;
                 ~~~~~~

  ../../node_modules/.pnpm/@types+react@16.14.51/node_modules/@types/react/index.d.ts:3325:13
    3325             symbol: React.SVGProps<SVGSymbolElement>;
                     ~~~~~~
    'symbol' was also declared here.

../../../../../node_modules/@types/react/index.d.ts:3440:13 - error TS2717: Subsequent property declarations
same type.  Property 'text' must be of type 'SVGTextElementAttributes<SVGTextElement>', but here has type 'SVtributes<SVGTextElement>'.

3440             text: React.SVGTextElementAttributes<SVGTextElement>;
                 ~~~~

  ../../node_modules/.pnpm/@types+react@16.14.51/node_modules/@types/react/index.d.ts:3326:13
    3326             text: React.SVGTextElementAttributes<SVGTextElement>;
                     ~~~~
    'text' was also declared here.

../../../../../node_modules/@types/react/index.d.ts:3441:13 - error TS2717: Subsequent property declarations
same type.  Property 'textPath' must be of type 'SVGProps<SVGTextPathElement>', but here has type 'SVGProps<Sent>'.

3441             textPath: React.SVGProps<SVGTextPathElement>;
                 ~~~~~~~~

  ../../node_modules/.pnpm/@types+react@16.14.51/node_modules/@types/react/index.d.ts:3327:13
    3327             textPath: React.SVGProps<SVGTextPathElement>;
                     ~~~~~~~~
    'textPath' was also declared here.

../../../../../node_modules/@types/react/index.d.ts:3442:13 - error TS2717: Subsequent property declarations
same type.  Property 'tspan' must be of type 'SVGProps<SVGTSpanElement>', but here has type 'SVGProps<SVGTSpa

3442             tspan: React.SVGProps<SVGTSpanElement>;
                 ~~~~~

  ../../node_modules/.pnpm/@types+react@16.14.51/node_modules/@types/react/index.d.ts:3328:13
    3328             tspan: React.SVGProps<SVGTSpanElement>;
                     ~~~~~
    'tspan' was also declared here.

../../../../../node_modules/@types/react/index.d.ts:3443:13 - error TS2717: Subsequent property declarations
same type.  Property 'use' must be of type 'SVGProps<SVGUseElement>', but here has type 'SVGProps<SVGUseEleme

3443             use: React.SVGProps<SVGUseElement>;
                 ~~~

  ../../node_modules/.pnpm/@types+react@16.14.51/node_modules/@types/react/index.d.ts:3329:13
    3329             use: React.SVGProps<SVGUseElement>;
                     ~~~
    'use' was also declared here.

../../../../../node_modules/@types/react/index.d.ts:3444:13 - error TS2717: Subsequent property declarations
same type.  Property 'view' must be of type 'SVGProps<SVGViewElement>', but here has type 'SVGProps<SVGViewEl

3444             view: React.SVGProps<SVGViewElement>;
                 ~~~~

  ../../node_modules/.pnpm/@types+react@16.14.51/node_modules/@types/react/index.d.ts:3330:13
    3330             view: React.SVGProps<SVGViewElement>;
                     ~~~~
    'view' was also declared here.

Found 182 errors in 3 files.

Errors  Files
     5  ../../node_modules/.pnpm/@types+react@16.14.51/node_modules/@types/react/index.d.ts:3126
     1  src/form-group/form-group.tsx:29
   176  ../../../../../node_modules/@types/react/index.d.ts:3254
/mnt/c/Users/Ashish Gupta/Desktop/open-source/freeCodeCamp/tools/ui-components:
 ERR_PNPM_RECURSIVE_RUN_FIRST_FAIL<EMAIL_ADDRESS>build: `pnpm run build:css && pnpm run build:js &&
Exit status 2
 ELIFECYCLE  Command failed with exit code 1.
 ELIFECYCLE  Command failed with exit code 1.
 ELIFECYCLE  Command failed with exit code 1.
 ELIFECYCLE  Command failed with exit code 1.
 ELIFECYCLE  Command failed.
 ELIFECYCLE  Command failed.
ERROR: "develop:client" exited with 1.
 ELIFECYCLE  Command failed with exit code 1.
 ```

@iprime2 Instead of asking for help here, please join us in our chat room or the forum.
Thank you and happy coding.


Using latest dev version of Worldguard, FAWE. Cannot use "allow-block-place" "allow-block-break" flags.

I'm pretty sure my solution is to use normal WorldEdit, but I don't want to obviously because I value FAWEs performance efficiency. Is there a way to be able to use these flags?

use skript for this its very easy to disable ...

ok how then

Any progress on fixing this?

Would love to get these flags back for WG7, any progress on the issue?

Feel free to try this fork, it may or may not work with FAWE, but the flags on this fork work on WG7
https://github.com/OnyxianSoul/WorldGuardExtraFlags-1.17

Related #152


jump to comic

Is your feature request related to a problem? Please describe.
Sometimes, using the app I'd like to be able to open a specific comic, either by typing in the comic number/name like on the explainxkcd wiki or by  searching for it in google then opening using the app.
Describe the solution you'd like
I think a "search" or "jump to comic" feature in the app would work, as well as support for opening xkcd links.

Hey @ToadtoadGD thanks for the request!
I'll close #19 and keep this issue open for implementation.


[Unit Testing]: More test cases

Add more unit testing test cases.
Related to issue #630 and extending PR #643
Signed-off-by: Arthur Chan<EMAIL_ADDRESS>

Not needed anymore, using another approach in PR #663


Enable support for apps that are run locally at a sub-uri.

For example app_1 runs at dev.co/ and app_2 runs at dev.co/app_2/
If app_2 has BetterErrors installed, the repl will not show up because
it sends its /__better_errors/... requests to app_1.
This feature lets the user specify the sub-uri (or as its being called
here uri_prefix) in Rail's configuration block config.better_error.uri_prefix = "/app_2"
or by calling it directly BetterErrors.uri_prefix = "/app_2".

We have multiple apps running on different paths, requests are directed to the correct app using haproxy.
We can't use SCRIPT_NAME because our paths are dynamic (/:account_slug/schtuff).This PR would solve our issue. We do run the whole stack locally with haproxy serving each piece. For now my only solution is to fork this repo and apply this PR's patch.
I think this is worth thinking about.


Exception when importing private key

zcash4mac 1.0.4-208
Steps:
Wallet -> Import one private key -> Paste the private key -> Click on "Import"
What happened:
The key is actually imported but an exception is thrown right when the import ends.

Expected result:
No exception.

This project is dead so retracting.


nixos/nimdow: module updates

Description of changes

Make nimdow package configurable
Ensure nimdow can start a terminal in it's default configuration
Add a simple module test

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Looks good!
Could you add a passthru.tests reference to nixosTests.nimdow in the nimdow derivation? In that way, ofborg and other tools would run these tests for new updates.

Yeah, I already had that in progress, just wasn't sure if I should add it to this PR or separately. Added it here now.

Version test fails on darwin x86: https://github.com/NixOS/nixpkgs/pull/296479/checks?check_run_id=22757547826
building '/nix/store/5sn0iihbw09lc7yy95rklpg6dbmbj3qq-nimdow-0.7.37-test-version.drv'...
nimdow --version returned a non-zero exit code.
could not load: libXft.so
(compile with -d:nimDebugDlOpen for more information)
error: builder for '/nix/store/5sn0iihbw09lc7yy95rklpg6dbmbj3qq-nimdow-0.7.37-test-version.drv' failed with exit code 1;
       last 3 log lines:
       > nimdow --version returned a non-zero exit code.
       > could not load: libXft.so
       > (compile with -d:nimDebugDlOpen for more information)
       For full logs, run 'nix log /nix/store/5sn0iihbw09lc7yy95rklpg6dbmbj3qq-nimdow-0.7.37-test-version.drv'.

Uh it's a x11 window manager, it cannot work on Darwin. The version check isn't new tho?

Doesn't link correctly on macos so set platforms to linux.


Date/time picker : Edit Creation Date

Please give your suggestions about what kind of client-side component we should choose to select a date/time.
There is a lot of pre-requisite work to do before because it will need to use the current culture and timezone (and later, the calendar settings as in O1).
It will be used for the date time field and for the editor of the 'creation date'.

See #434 to manage the cultures.

@DanielStolt and I have recently talked about datetime localization offline. First thing is to reference the issue that was opened in O1 related to the types of fields we actually need (Date, Time and DateTime AFAIR). It explained why. We might not have to care about Cultures for now if you want to make some progress on the fields. Timezone is already supported so you can use it at least.

there is an issue when you add a content item: if the owner part is displayed and you enter a user name, it will create the item using the current one, not the specified one.

Now that the DateTimeField and the Timezone setting exist, we should also think to the editor of the CreatedUtc in CommonPart-Date.Edit.cshtml.
Should we just use an input with type="datetime-local"?

Also: When a date is manually set on creation or update, do we have to convert it to Utc when we store it and convert it to the current timezone when we display and re-edit it, like the DateTimeField ?

Yes save to DB as UTC and display with LocalClock.ConvertToLocal()

In Edit, we should format it to not display the milliseconds.

That's maybe just a string.format

I removed the step="1" because it can lead to a  client validation error in Chrome.
https://stackoverflow.com/questions/41373682/input-type-datetime-local-doesnt-support-seconds-in-chrome


srcsets still point to webflow

The srcset image urls still point to webflow for all pages. Need to move to local.

checked with
grep -Hnor . -e "https://uploads-ssl.webflow.com"| more
https://github.com/SriSathyaSaiVidyaVahini/SSSVV-landing/commit/704caa2f00e6372bd21d781e481bc2d5ca5a56fc
should have fixed this.


Commands could reject invalid addesses early

web3 provides web3.utils.isAddress, which checks offline if a string is a valid address. We should use this in commands that prompt for an address, like balance, to either not progress until a valid one is entered or fail early.

Closing as duplicate of https://github.com/OpenZeppelin/openzeppelin-sdk/issues/1069


Nick: Watch 1 877 6715 087

Nick requests the watch of the watch_number 1 877 6715 087. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Ryan M in Charcoal HQ


Tracking issue for kafka sink v2

At the moment, we use Sarama as the Kafka client to implement our Kafka sink. It's heavily coupled with the Sarama library. We may replace it by other Kafka client library in the future, so it's necessary to get rid of the dependencies on the Sarama.

[x] #8046
[x] #8047
[x] #8076
[x] #8062
[x] #8081
[x] #8082
[x] #8092
[x] https://github.com/pingcap/tiflow/issues/8145
[x] #8141
[x] #8143
[x] #8164
[x] #8184
[x] #8213
[x] #8240
[x] #8245
[x] #8241
[x] #8246
[x] #8250
[x] #8262
[x] #8339
[x] #8340
[x] #8367
[x] #8578
[x] #8749

closed


Use expo-image-picker instead of RN image picker

@keybase/react-hackers Fixes the issue of uses external storage as temp cache on android.

[ ] taking a video crashes ios


Merciless fighting style - fear effect wearing off immediately after the pc who caused it ends their turn instead of after the npc's next turn

Good afternoon,
the fear effect from merciless is wearing off on the end of turn when the character who caused it ends their turn instead of after the npc's who got feared''s turn comes up.
Merciless is on the barbarian fighter as their fighter fighting style choice.
Workshop campaign: Diabloasta
version of game 1.5.74
UB version <IP_ADDRESS>
Merciless fighting style issue.zip

fixed in dev.


[BUG] setup tools related python functions are taking more time after upgrading to 65.3.0 from 57.0.0 as number of calls getting increased

setuptools version
65.3.0
Python version
3.9.16
OS
custom linux distro
Additional environment information
issue observed env: openssl version -> 3.0.8  cryptography -> 40.0.2 setuptool -> 65.3.0
this issue is observed in all usecases that is imporitng setuptool package.
Description
As part of upgrading openssl to 3.0.8 version, there requires indirect dependency with setuptools package, hence upgraded to version 65.3.0 to match with minimum requirement to openssl.
After update ,there is some degrade in performance during the runtime which is observed to be appeared from setuptools approx 0.6 seconds more than usual as the total number of calls are getting increased for same code when executed with two different setuptool version
Is this expected behaviour ?
Expected behavior
number of calls and total time taken should be approx same when setuptools upgraded to (65.3.0)
How to Reproduce
sample Script uscase where  the issue can be observed easily
# cat tesversion.py
#!/usr/bin/env python
import pkg_resources
version = pkg_resources.require("redis")[0].version
print(version)

# cat tesversion.py
#!/usr/bin/env python
import pkg_resources
version = pkg_resources.require("redis")[0].version
print(version)

#  python -m cProfile -o profile.out tesversion.py
# python showprofile.py

Output
profiling of  openssl(3.0.8) + cryptography(  40.0.2) + setuptool upgrade(65.3.0)
Thu Jun 22 11:29:02 2023 profile.out
428104 function calls (412759 primitive calls) in **0.937 seconds**
Ordered by: internal time
List reduced from 2437 to 20 due to restriction <20>
ncalls tottime percall cumtime percall filename:lineno(function)
3185 0.046 0.000 0.105 0.000 /usr/lib/python3.9/sitepackages/
pkg_resources/_vendor/packaging/version.py:261(__init__)
215 0.039 0.000 0.039 0.000 {built-in method marshal.loads}
504/101 0.027 0.000 0.083 0.001 /usr/lib/python3.9/sre_parse.py:494(_parse)
827/775 0.025 0.000 0.162 0.000 {built-in method builtins.__build_class__}
2498 0.025 0.000 0.025 0.000 {built-in method posix.stat}
38302 0.019 0.000 0.040 0.000 {built-in method builtins.isinstance}
13831 0.018 0.000 0.018 0.000 /usr/lib/python3.9/sre_parse.py:234(__next)
651 0.018 0.000 0.062 0.000 <frozen
importlib._bootstrap_external>:1514(find_spec)
2025 0.017 0.000 0.019 0.000 /usr/lib/python3.9/sitepackages/
pkg_resources/_vendor/packaging/version.py:444(_cmpkey)

/usr/lib/python3.9/sitepackages/pkg_resources/_vendor/packaging/version.py
is coming from setuptools and this is always topping the list of functions which is causing more time. This file/methods from this file were not at all listed when profiling on the old versions.
profiling of  openssl(1.1.1o) + cryptography( 3.2.1) + setuptool upgrade(57.0.0)
Thu Jun 22 11:25:08 2023 profile.out
55287 function calls (54295 primitive calls) in **0.130 seconds**
Ordered by: internal time
List reduced from 752 to 20 due to restriction <20>
ncalls tottime percall cumtime percall filename:lineno(function)
18 0.008 0.000 0.010 0.001 {built-in method _imp.create_dynamic}
42 0.007 0.000 0.007 0.000 {built-in method marshal.loads}
142 0.005 0.000 0.016 0.000 {built-in method builtins.__build_class__}
19 0.005 0.000 0.012 0.001 /usr/lib/python3.9/enum.py:179(__new__)
173 0.004 0.000 0.014 0.000 <frozen
importlib._bootstrap_external>:1514(find_spec)
309 0.004 0.000 0.004 0.000 {built-in method posix.stat}
94/24 0.004 0.000 0.011 0.000 /usr/lib/python3.9/sre_parse.py:494(_parse)
8 0.003 0.000 0.003 0.000 {built-in method builtins.eval}
4 0.003 0.001 0.003 0.001 {method 'recv' of '_socket.socket' objects}
38 0.003 0.000 0.006 0.000 /usr/lib/python3.9/sitepackages/
redis/connection.py:759(pack_command)

===========================================================================
Sample scripts output:
profiling of  openssl(3.0.8) + cryptography(  40.0.2) + setuptool upgrade(65.3.0)
Mon Jun 26 18:25:56 2023    profile.out

         318328 function calls (309872 primitive calls) in 0.667 seconds

   Ordered by: internal time
   List reduced from 1342 to 20 due to restriction <20>

   ncalls  tottime  percall  cumtime  percall filename:lineno(function)
     3187    0.046    0.000    0.102    0.000 /usr/lib/python3.9/site-packages/pkg_resources/_vendor/packaging/version.py:261(__init__)
   413/65    0.023    0.000    0.071    0.001 /usr/lib/python3.9/sre_parse.py:494(_parse)
     1896    0.019    0.000    0.019    0.000 {built-in method posix.stat}
       82    0.018    0.000    0.018    0.000 {built-in method marshal.loads}
    11952    0.017    0.000    0.017    0.000 /usr/lib/python3.9/sre_parse.py:234(__next)
     2027    0.017    0.000    0.018    0.000 /usr/lib/python3.9/site-packages/pkg_resources/_vendor/packaging/version.py:444(_cmpkey)

================================================================
profiling of  openssl(1.1.1o) + cryptography( 3.2.1) + setuptool upgrade(57.0.0)
================================================================
Mon Jun 26 18:18:03 2023    profile.out

         311194 function calls (307101 primitive calls) in 0.630 seconds

   Ordered by: internal time
   List reduced from 1046 to 20 due to restriction <20>

   ncalls  tottime  percall  cumtime  percall filename:lineno(function)
     2150    0.027    0.000    0.027    0.000 {built-in method posix.stat}
   452/76    0.025    0.000    0.076    0.001 /usr/lib/python3.9/sre_parse.py:494(_parse)
     2043    0.023    0.000    0.055    0.000 /usr/lib/python3.9/site-packages/pkg_resources/_vendor/packaging/version.py:271(__init__)
     1166    0.019    0.000    0.056    0.000 /usr/lib/python3.9/site-packages/pkg_resources/_vendor/packaging/version.py:205(_legacy_cmpkey)
     8152    0.019    0.000    0.029    0.000 /usr/lib/python3.9/site-packages/pkg_resources/_vendor/packaging/version.py:187(_parse_version_parts)
    12756    0.017    0.000    0.017    0.000 /usr/lib/python3.9/sre_parse.py:234(__next)
      507    0.015    0.000    0.055    0.000 <frozen importlib._bootstrap_external>:1514(find_spec)
     2367    0.013    0.000    0.019    0.000 /usr/lib/python3.9/posixpath.py:71(join)
       59    0.012    0.000    0.012    0.000 {built-in method marshal.loads}

Hi @mbhata, please note that the module pkg_resources is deprecated. We are providing maintenance to it, but I don't think performance parity is at the top of the priorities (specially because we also depend on 3rd party dependencies).
Have you considered using importlib.metadata.version as a replacement for the example you provided?
You can also use importlib.metadata.distribution to get a distribution object...


feat(client): add deis logs -n

how can I get more logs using deis logs --app=<appname>
i can see in the API there is settings.settings.LOG_LINES... can it be passed as a parameter somehow so we can get as much log as we want?
mainly wanted to give developers access to deis without having to give them ssh access.

Like said by @krancour , follow option to deis logs would be awesome. Today I ssh into host and jounalctl -fu, no better way ?

not at the moment. This is due to the limitations of WSGI, which cannot handle asynchronous requests out of the box.


Add some way to disable the spinner

From tox created by alexrudy : tox-dev/tox#926
Detox prints a file spinner to stdout while tests are running. This is fine for local / console use, but in automated environments, it can send a false signal that work is still going on.

detox has its own issue tracker now :)

fixed in https://github.com/tox-dev/detox/commit/52b2e91c52de796f9b0e98a8600316aa6bb799a3 (released with 0.15)


Fix typing error

fixes the TypeError: 'type' object is not subscriptable error when running __main__.py

@fohrloop Done, can you check please

I will have this as part of the next release!
As support for generic type type annotations with built-in collection types was added in Python 3.9, it essentially means that wakepy <= 0.7.1 supports only Python 3.9+, and not Python 3.7+.
I'm also writing myself a note about adding tests for the CLI / __main__py.
Thank you @Aymane11 !

Cherry picked this and released wakepy 0.7.2 with the fix


fix(ansible/artifacts): update transfusion model

Description
This PR update lidar_tranfusion artifacts (3D detection component).
Related to https://github.com/autowarefoundation/autoware.universe/pull/7890
Related links
Tests performed

Tested by command as below

ansible-playbook autoware.dev_env.download_artifacts -e "data_dir=$HOME/autoware_data" --ask-become-pass

TASK [autoware.dev_env.artifacts : Create lidar_transfusion directory inside /home/scepter914/autoware_data] ***
ok: [localhost]

TASK [autoware.dev_env.artifacts : Download lidar_transfusion/transfusion.onnx] *************************
ok: [localhost]

TASK [autoware.dev_env.artifacts : Download lidar_transfusion/transfusion.param.yaml] *******************
ok: [localhost]

Notes for reviewers
Interface changes
None.
ROS Topic Changes
None.
ROS Parameter Changes
None.
Effects on system behavior
None.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[x] There are no open discussions or they are tracked via tickets.
[x] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@youtalk
Thank you for review.
https://github.com/autowarefoundation/autoware.universe/pull/7890 is merged now, so would you merge this PR?

Done!


IDEA: Support using Bitcoin Core 0.10 directly without using Insight

Using readonly addresses for new wallets / new addresses.
For importing old wallets,  -rescan will be needed.
From https://bitcoin.org/en/release/v0.10.0

One new RPC, importaddress, is added which functions similarly to importprivkey, but instead takes an address or script (in hexadecimal) as argument. After using it, outputs credited to this address or script are considered to be received, and transactions consuming these outputs will be considered to be sent.

The following RPCs have optional support for watch-only: getbalance, listreceivedbyaddress, listreceivedbyaccount, listtransactions, listaccounts, listsinceblock, gettransaction. See the RPC documentation for those methods for more information.

Something similar is already on the roadmap. The plan is to have BWS communicating with bitcore instead of with Insight.


[OpenCL] Fixes transpose operation

Can one of the admins verify this patch?

Jenkins, test this please.


Try to incorporate benchmark gRPC performance tests to the load test

Description:
Better to extend existing Ballerina gRPC performance tests to use these benchmarks examples https://grpc.io/docs/guides/benchmarking/.

Yes +1


Remove forbidden characters in filesystem

Found an error in the Windows file system. It fails when it tries to make a file or folder with one of the forbidden characters. A course or episode with a colon, for example would fail during the mkdir process. Here's a full list of forbidden characters from the prompt:    \ / : * ? " < > |

@vcaiii nice!

Thank you @vcaiii ❤

@vcaiii Could you please check your Facebook Messenger? I've sent you a private message there.


Error with bind_menu

Im using snaked on Archlinux and I get this error
bash$ snaked file
Traceback (most recent call last):
File "/usr/bin/snaked", line 5, in
run()
File "/usr/lib/python2.7/site-packages/snaked/core/run.py", line 45, in run
manager = get_manager()
File "/usr/lib/python2.7/site-packages/snaked/core/run.py", line 29, in get_manager
manager = EditorManager(options.session)
File "/usr/lib/python2.7/site-packages/snaked/core/manager.py", line 75, in init
self.activator.bind_menu('_File#1')
TypeError: bind_menu() takes exactly 6 arguments (2 given)

I dont have the uxie package "python2-uxie-git"
Now I can run snaked


chore(main): release registry-operator 0.0.2

:robot: I have created a release beep boop
0.0.2 (2024-03-16)
Bug Fixes

foobar (a8bb13a)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/shanduur/helm-charts/releases/tag/registry-operator-v0.0.2 :sunflower:


ContentNegotiator and custom formatter issues

I extend a custom formater named plist and add it to my controller behaviors.
I also have some formatters in common/config/main.php
// in application configuration
use yii\web\Response;
    'bootstrap' => [
        'log',
        [
            'class' => 'yii\filters\ContentNegotiator',
            'formats' => [
                'application/json' => Response::FORMAT_JSON,
                'application/xml' => Response::FORMAT_XML,
            ]
        ],
    ],

in my controller
use yii\web\Response;

public function behaviors()
{
    return [
        [
            'class' => 'yii\filters\ContentNegotiator',
            'formats' => [
                'text/plist' => 'plist',
            ],
        ],
    ];
}

they didn't merge correctly,the formatter plist didn't work!
but if I put the plist formatter in application config as below and remove it from controller,it will work!
use yii\web\Response;
    'bootstrap' => [
        'log',
        [
            'class' => 'yii\filters\ContentNegotiator',
            'formats' => [
                'application/json' => Response::FORMAT_JSON,
                'application/xml' => Response::FORMAT_XML,
,               'test/plist'=>'plist'
            ]
        ],
    ],

why should these be merged? You are declaring the configuration in two different places. It's your own responsibility to have these synchronized.


How to configure C++ and Kotlin etc for VS Code

Issue Type: Feature Request
Well I would appreciate if Microsoft can come up with "How to Guides" on configuring compilers such as MinGW-w64 (for C/C++) and coding in the Kotlin language.  In fact, there should be one guide each for all the languages and compilers that VS Code supports.  In fact, these are the languages that I will be facing in my career apart from emerging tech - HTML5 & CSS3 (works), Java 17 (works), Python 3.10 (works), npm & gem (works), JS & TS (haven't tried), C++ (g++ not recognised), Kotlin (kotlinc not recognised), Ruby 2.7 (works), Golang, Clojure and Prolog (haven't tried it).
VS Code version: Code 1.63.2 (899d46d82c4c95423fb7e10e68eba52050e30ba3, 2021-12-15T09:40:02.816Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

Most of mentioned languages aren't supported out of the box, and support depends on 3rd party extensions.
Currently for docs and how-to's you can start with https://code.visualstudio.com/docs/languages/overview and for specific details for languages - check them in sidebar.
If they are not in list, then best way to find extension for that language and check it's docs.
https://marketplace.visualstudio.com/search?target=vscode&category=All categories


Preventing BrokenPipe error

I solved this issue by replacing all print! and println! macros with the ones coming from calm_io library which are stdoutln! and stdout! macros. They are basically doing exactly same thing print*! except it actually returns a Result instead just panicking.
This allows us to catch the result of a text outputting attempt and check for an error. In this case we want to catch std::io::BrokenPipe error and continue in a graceful manner.
Let me know what do you think about using this library, is this fine or would you rather solve this in a different way?
This fixes issue #18

Thanks for this!  #95 was a solution to the broken pipe error. issue #18 is more about avoiding the use of print*(). I think that stdout() is a possible option.
@UlazkaMateusz I wanted to ping you. I know that you wanted to provide a solution to this problem as well.


[GLUTEN-3668][CH] Performance regresses seriously after PR 3169 merge when executing convert string to date

What changes were proposed in this pull request?
PR #3169 add some functions to convert string to date, which leads to performance regression serious. Will be reverted first and raise another pr to fix the issue later.
Test case, 2000W records:
before PR #3169 : 3s
after PR #3169: 3.6m
Related PR: #3169, #3563;
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

@KevinyhZou @lgbo-ustc @taiyang-li please help to review, thanks.

LGTM


Concept: finalize View and WebView class

Variant 1

Create trait ViewTrait with content from View class and mark him as @internal.
View class only use ViewTrait and finalized.
WebView class not extends View but use ViewTrait and finalized.

Variant 2

Create class BaseView with content from View class and mark him as @internal.
View extends BaseView and finalized.
WebView extends BaseView and finalized.

I like variant 2.

What do you want to achieve with the change?

What do you want to achieve with the change? What problem do you want to fix?

Finalize classes. View and WebView must not be inherited outside package.

abstract BaseView would do.


Mojolicious templates are now supported in Camelcade plugin for IntelliJ IDEA

Didn't know where to add this, so, sorry if doing it wrong way.
Anyway, Perl5 plugin for IntellJ IDEA I'm currently developing supports Mojolicious templates starting from today's build.
Would be nice to mention it somewhere.
Here is it: https://plugins.jetbrains.com/plugin/7796

While I'm happy to see this effort undertaken, I'm not sure that it would belong in the main documentation. I don't believe that we reference mojo.vim for example.
While we wait for inspiration if you would prefer to make a wiki page, you are free to do so.

jberger: :+1:

Well, I ment IDE/Editor support of wiki. Just didn't know I could edit it myself :) Thanks!


Changed paths for grayscale- and rivers-map.

Since the actual filenames of grayscale- and rivers-map follow the same pattern as all the other maps, I found it annoying that the generated maps aren't just all placed in the same directory. Kind of makes it difficult to (auto-)generate several maps at once (I wrote a script to generate half a dozen decently sized maps while I'm away and the files being spread around doesn't feel right).
This might seem a bit dirty, there may be other filenames I overlooked. But somebody else shall look at it and decide if the commit is useful or needs some polish.
EDIT: I won the coveralls-lottery.^^

Good enough for me and congrats on the coveralls-lottery!  :)


Enhancement: Table Hints is not passing properly in the BatchQuery.

The changes for this has been checked-in to the master branch. Will be available on builds > v1.9.10.


Taxonomy class has incorrect restrictions

#51 changed to gist:hasMember soem gist:Category in gist 9.0.0

#51 changed to gist:hasMember soem gist:Category in gist 9.0.0

If this is an equivalence, and you use some vs. all, you have an definition that incorrectly allows a vocabulary to have members that are non-categories, e.g. laptops, contracts and buildings.  If it is subclass, then it is correct.

When I made this change Taxonomy == Controlled Vocabulary.  Do we need Taxonomy
I added this

Adding the all values makes the subclassing work out but it does it by not adding anything.  Yes you're right you could add shoes and ships and ceiling was to the taxonomy, but by saying that is the distinction you're allowing the shoes in the ControlledVocabulary.  But really allValues isn't adding to the definition of the class you put it on.  It's adding to the definition of its completment, and its adding to the definition of the filter class.  But really its trying to be a constraint, and Id rather have SHACL manage the constraints.

I don't think of the allValues axiom as a constraint.  I think of it as a true statement that helps to clarify the meaning of the concept.  Meaning can be used in different ways. Being a constraint is just one.
Its always a judgment call whether to put true statements in the model that clarify meaning, and whether to have those statements be in annotations or in formal axioms.

Regarding the inferred equivalence of Taxonomy and ControlledVocabulary and whether we need the former.   They are only equivalent because not enough meaning has been added to Taxonomy.  The definition is Taxon is a bit clunky, it is really just the role of a member of a ControlledVocabulary, and we generally avoid defining Role classes.  Its odd that this is not just easy to do.   It seems useful to have a class for Taxonomy meaning that its members are arranged in a hierarchy. It is less obvious that specifying axioms that make that distinction is useful.

Can we define Taxonomy this way, per one of the above diagrams, but ignoring the navigational parent part.
ControlledVocabulary and
(hasMember some ( (hasSuperCategory some Category) OR
(hasSubCategory some Category)
)
)

Clever, good idea
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: MIchael Uschold<EMAIL_ADDRESS>Sent: Thursday, March 28, 2019 10:24:03 AM
To: semanticarts/gist
Cc: Dave McComb; State change
Subject: Re: [semanticarts/gist] Taxonomy class has incorrect restrictions (#51)
Can we define Taxonomy this way, per one of the above diagrams, but ignoring the navigational parent part.
ControlledVocabulary and
(hasMember some ( (hasSuperCategory some Category) OR
(hasSubCategory some Category)
)
)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/semanticarts/gist/issues/51#issuecomment-477669231, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AABkvh_EgagBngIDfYTF6BIiXFwEdn2lks5vbOyjgaJpZM4Y8Kn8.

Its more fully described above with a question about whether we want to include navigational links too. Maybe not, since a taxonomy categorizes, but a so called navigational taxonomy is not really a taxonomy, in that the nodes are not categorizing anything, or if they are, in a very different looser way.
Might we get rid of our own superCategory and use skos:broader for simplicity's sake?


Document the change from MP_DATA_FILE to MP_DATABASE

My mailpit version is 1.15.1 and MP_DATABASE was only added with https://github.com/axllent/mailpit/releases/tag/v1.16.0
I just lost a bunch of mails because I used MP_DATABASE which I got from here: https://mailpit.axllent.org/docs/configuration/runtime-options/
I would suggest to add a note to the documentation that MP_DATABASE is only available since v1.16.0.

Point noted, however using that logic I would need to document the same for every flag / option which doesn't make sense. Can you please tell me why you are using a much older version of Mailpit?

Symfony has a nice way of documenting features that were added with a specific version.
Some examples:
https://symfony.com/doc/current/mailer.html#disabling-automatic-tls
https://symfony.com/doc/current/components/filesystem.html#readfile

Can you please tell me why you are using a much older version of Mailpit?

Unfortunately it's currently the latest version available for NixOS. For both, the stable channel 24.05 and the unstable channel aswell: https://search.nixos.org/packages?channel=24.05&from=0&size=50&sort=relevance&type=packages&query=mailpit
For many packages a bot can create update PRs to nixpkgs. Maybe the bot couldn't automatically resolve the new dependencies for the update from 1.15.1 => 1.16.0 and nobody did it manually yet.
https://github.com/NixOS/nixpkgs/blob/c5bcace2113d93a989966b4bd2dc1b3d957a0c61/pkgs/servers/mail/mailpit/default.nix

I just created a PR at nixpkgs to update mailpit to the latest version.
https://github.com/NixOS/nixpkgs/pull/323881

Again, I do apologise that you lost emails as a result of using an outdated version of Mailpit with current documentation, though these things should always be tested before being put into production. Unlike flags, unsupported environment variables do not generate errors in any application.
I do carefully deprecate flags though, meaning if users are using flags or environment variables that are no longer relevant, renamed or removed, then a deprecation warning is typically shown for at least a year after the deprecation. This does not address the situation of using an old version of Mailpit with newer documentation though. I have updated the runtime options page to add versions next to some of the more recent options, so hopefully this helps prevent this issue in future. Thanks for the advise.
Also thanks for the PR to Nix :+1: I haven't a clue who maintains that, or why it's not up-to-date. I do know that FreeBSD update usually within an hour or two of new releases, so this is automated on their end.

It wasn't too bad, it was only running for a day in a Staging environment, then my systemd Service was restarted and the mails were gone. Took me quite a while to figure out that it was due to the not-yet-existing MP_DATABASE env var.
Your addition to the docs can definitely help with that, thank you very much!
It can be even improved a little further by linking to the Release Page on GitHub. You write very good release notes, with that someone can follow pretty good what changed, e.g. MP_DATA_FILE to MP_DATABASE.
In fact I did set both variables now, so as soon as my PR is released into unstable I can easily update to v1.19.0.
Usually the update process for nixpkgs is automated for a lot of packages via nixpkgs-update, based on the information from https://repology.org/
The bot also did the last PR from 1.15.0 to 1.15.1: https://github.com/NixOS/nixpkgs/pull/301724
But nix always requires that a hash of the build output is provided, this ensures reproducability. It seems like the bot is not able to do the automatic update when also the npm dependencies (and thus their hash) changed.
I subscribed to mailpit release notifications now. When I have the time I will keep the nixpkgs mailpit version up to date.

@axllent I just found it's possible to declare how packages can be updated by providing the passthru.updateScript attribute.
If I have time I'll investigate how I can make this work so mailpit is also automatically updated in the nix package repository.
https://github.com/NixOS/nixpkgs/blob/master/pkgs/README.md#automatic-package-updates

@RafaelKr That's awesome, I did not know about https://repology.org/. I can see how the node build requirement part can complicate things a bit though.

Just a note about linking to the release page for each of those versions - whilst nice, I'd rather not because I then run the risk of unsuspecting users inadvertently downloading old releases because they "got the link from the website" ;-)  I could link them to the CHANGELOG, but it's not always a nice summary like on the release pages (I sometimes put additional info in the releases).

I just implemented the passthru.updateScript for mailpit in nixpkgs, so their bot can automatically create update PRs :tada:
https://github.com/NixOS/nixpkgs/pull/324026
Now it also doesn't depend on repology anymore, the latest version information is directly fetched from the GitHub API.

Just a note about linking to the release page for each of those versions - whilst nice, I'd rather not because I then run the risk of unsuspecting users inadvertently downloading old releases because they "got the link from the website" ;-) I could link them to the CHANGELOG, but it's not always a nice summary like on the release pages (I sometimes put additional info in the releases).

@axllent Okay fine, I get that point. Then this case is closed, thanks!


Per-Function Metadata for the Executor

Is your feature request related to a problem? Please describe.
I want to implement use case-specific worker threads. The built-in thread pool does not do any specific scheduling, it just randomly assigns tasks to threads.
Specifically, I'd like to have a render thread and a database thread. On the web, it doesn't make sense for either of them to be run on multiple web workers. On native, rendering through wgpu could handle multiple threads, but there are schedulers available that are specifically designed for this kind of work (like the one in bevy).
Implementing an Executor allows me to control thread scheduling, but it's not possible to discern which thread should be used, because there is no possibility to supply metadata from api.rs to the Executor.
Describe the solution you'd like
It would be relatively easy to decide on the thread to use based on the function to be called, but that information is not officially available to the executor (I assume), it only gets a closure to execute.
I think the best solution would be to have another #[frb(...)] annotation for functions that just gets passed to the Executor as a string whenever that function is called. This could also be a unit-like enum to avoid string compares.
Describe alternatives you've considered
There's a debug_name field on WrapInfo that could actually be used for this by adding that annotation to the function name, but besides being clumsy (by requiring all functions to have a prefix or suffix like db or render) the debug part of the name also implies to me that this is not a good idea and might break in future updates.
Alternatively, the thread pool task could also just be used as a jump pad that schedules yet another task on a specific thread, but that's a lot of overhead for what's basically a workaround.

Looks reasonable to me! It looks like adding extra metadata (tags etc) to generated functions.

this bot is really annoying

Feel free to make a PR!

aaaah


Add option so that blame virtual text is displayed when user switches to normal mode instead of a delay.

Is your feature request related to a problem? Please describe.
It's irritating when the blame virt text updates while I'm typing. I know there is an option to delay it but that doesn't really fit my needs.
Describe the solution you'd like
Instead of updating/displaying the virtual text after a set duration, display it when the user exits the insert mode. That way the buffer wouldn't move when the user pauses while typing.
Describe alternatives you've considered
I've set current_line_blame_delay = 10000 (10 seconds). But that doesn't exactly solve the problem. Just makes it less frequent.
Thanks!

Can't you just use an autocmd?

Oh yeah, thanks for the suggestion. For anyone else wanting to do this:
autocmd InsertEnter * :Gitsigns toggle_signs
autocmd InsertLeave * :Gitsigns toggle_signs

Thanks

I think you want toggle_current_line_blame?

Nono i wanted this only. I wrote about the wrong this in the original post. I actually meant the colored lines that appear next to the line numbers.


Multiline Input

safe should really support multiline input somehow, using some special delimiter to terminate.

Here's an idea:
safe set certificate@/path/to/file privkey@/path/to/other/file name=wildcard.github.com


change tray icon when exit node is used

Especially with the new Mullvad VPN exit nodes it would be really nice to quickly see if one is connected to an exit node just by looking at the tray icon.

+1 for this feature.
My typical usage looks like this:

Tailscale is always connected.
My Mullvad exit node is usually connected.
Sometimes I need to temporarily disable the Mullvad exit node.

Being able to glance at the tray icon to determine if the Mullvad exit node is connected or not would be extremely useful.

This should be a pretty easy change, I think, but I do need to come up with an icon variant. Maybe a little shield in the corner or something?

This should be a pretty easy change, I think, but I do need to come up with an icon variant. Maybe a little shield in the corner or something?

A shield would work nicely.
Another option would be the country flag of the connected server.
Another option would be to use a little Mullvad icon, but that would probably run into Trademark issues.  It would have the advantage of being future-compatible if Tailscale ever offers VPN partners beyond Mullvad.
Whatever Trayscale lands on for the tray icon indicator, I think it would be helpful if it were consistent with the icon next to the Mullvad exit node in the main window of the app itself.  Currently the main window of the app shows a yellow circle icon next to the Mullvad exit node.


Instruction category is a function of opcode

So we don't need to cache it.

This implementation is based on a lookup table, why rewrite it with a different technique?

We are storing redundant information in InsnCodes.


Added recipient name for QR code payments on feed

Description
Tested
TODO
Other changes
Describe any minor or "drive-by" changes here.
Related issues

Fixes #[issue number here]

Backwards compatibility
Brief explanation of why these changes are/are not backwards compatible.

Was a temporary work for the MX pilot, will be re-written and merged to master.


Creating Archive Ios - Causing too long file names

VERSIONS

Xamarin.Forms.GoogleMaps - 3.3.0 and 5.0.0 Tested both
Xamarin.Forms - <IP_ADDRESS>5

PLATFORMS

[ ] Android OK
[x] iOS Works fine in another app but not in current (KaBom). Something happend in a week with no changes
[ ] UWP N/A

ACTUAL BEHAVIOR
When I create an Archive for Ios I suddenly get an error message in the build. This Nuget qworks fine in another app wiht same versions as above. It also worked fine in the current problem app one week ago.
Using same version i another app and it builds archive Ios withouth any issues.
I have the text here and below the actual screenshot:
Cannot create an IOS archive 'KaBom.iOS'.
Process cannot be executed on XMA server.
There was an error unzipping the file C\Users\karitalltmp\2022-11-06\f1a083d2.zip; Could not find a part of the path 'C:\Users\karitall\AppData\Local\Temp\Xamarin,Extractions\de6cbab4-b7ca-48a1-b1a8-dBef4856aa91\KaBomiOS 11-06-22 5.26
PM.xcarchive\Products\Applications'\KaBom.iOS,app\<EMAIL_ADDRESS>ACTUAL SCREENSHOTS/STACKTRACE
<[

](!-- Let me show your actual  screenshots or stacktrace. If you want to capture simulator screen to animation GIF, I recomend to LICEcap. -->

// paste stacktrace)

EXPECTED BEHAVIOR
Archive build Ios created
HOW TO REPRODUCE
Just doing a archive build in Ios project

update to the latest vs version

Hi,
The upgrade to latest VS solved the problem.


DeviceVideoTrackSource.GetCaptureFormatsAsync crashes on HL2

Describe the bug
On latest master branch with newest code for enumerating of devices and profiles, the function DeviceVideoTrackSource.GetCaptureFormatsAsync crashes on HL2 ARM. I haven't had a lot of time to test, but I tried with just a VideoCaptureDevice.id and also with VideoCaptureDevice.id and VideoProfileKind.VideoConferencing.
It does not crash in Unity.
Environment

Platform: UWP
Architecture: ARM
Unity version: 2019.4.5
Target device: HoloLens 2

I will have a look now.

I've deployed TestAppUWP on HL2 and there's no crash. It seems though there's an error in enumerating, but I didn't identify if this is a bug in TestAppUWP or in the enumeration code.

OK so I set up a simple test scenario to do some debugging.

Request access to video/microphone
Query devices with DeviceVideoTrackSource.GetCaptureDevicesAsync();
Query formats with DeviceVideoTrackSource.GetCaptureFormatsAsync(captureDevice.id)

On master branch as of both yesterday and today, GetCaptureFormatsAsync crashes on HL2.
I tried reverting the DLLs to the last time I pulled master, which was 21.07.2020, and then the above test scenario did not crash. Everything except the DLLs were kept the same.
So maybe there is something wrong related to building from Unity?

Unity 2019.4.5
Target SDK version: 10.0.19041.0
Minimum platform version: 10.0.17763.0

I'll try and see if I can get a crash log.

Might any of this be useful?
 	ucrtbased.dll!strlen(const char * str) Line 39	C
>	mrwebrtc.dll!std::char_traits<char>::length(const char * const _First) Line 462	C++
 	mrwebrtc.dll!std::string::assign(const char * const _Ptr) Line 2676	C++
 	mrwebrtc.dll!std::string::operator=(const char * const _Ptr) Line 2498	C++
 	mrwebrtc.dll!zsLib::String::operator=(char const *)	Unknown
 	mrwebrtc.dll!<lambda_2780a7ccd217006da50a8eaa82838787>::operator()<winrt::Windows::Foundation::IAsyncOperation<winrt::Windows::Devices::Enumeration::DeviceInformationCollection> const &>(const winrt::Windows::Foundation::IAsyncOperation<winrt::Windows::Devices::Enumeration::DeviceInformationCollection> & asyncResults, winrt::Windows::Foundation::AsyncStatus status) Line 753	C++
 	[External Code]
 	Windows.Devices.Enumeration.dll!GitDelegate2<Windows::Foundation::IAsyncOperationCompletedHandler<Windows::Devices::Enumeration::DeviceInformationCollection *>,Windows::Foundation::IAsyncOperation<Windows::Devices::Enumeration::DeviceInformationCollection *> *,enum ABI::Windows::Foundation::AsyncStatus>::Invoke(Windows::Foundation::IAsyncOperation<Windows::Devices::Enumeration::DeviceInformationCollection *> * a1, ABI::Windows::Foundation::AsyncStatus a2) Line 309	C++
 	Windows.Devices.Enumeration.dll!Microsoft::WRL::AsyncBase<Windows::Foundation::IAsyncOperationCompletedHandler<Windows::Devices::Enumeration::DeviceInformationCollection *>,Microsoft::WRL::Details::Nil,1,Microsoft::WRL::AsyncOptions<-1,0,&GUID_CAUSALITY_WINDOWS_PLATFORM_ID,2>>::FireCompletion() Line 776	C++
 	Windows.Devices.Enumeration.dll!FindAllAsyncOperation<Windows::Devices::Enumeration::DeviceInformationCollection,Windows::Devices::Enumeration::DeviceInformation,DeviceInformationServer,&RuntimeClassNameIAsyncOperationDeviceInformationCollection>::DevQueryCallback(HDEVQUERY__ * __formal, void * pContext, const _DEV_QUERY_RESULT_ACTION_DATA * pActionData) Line 295	C++
 	[External Code]

device_video_track_source.cpp

753   createParams->videoProfileKind =
754        (wrapper::org::webRtc::VideoProfileKind)profile_kind;

Thanks @eirikhollis,
I logged #492 for the actual bug I found, which is slightly different and does not produce any crash for me yet (I will continue testing on HL2). But just FYI.
Looking at the callstack, I am guessing the issue might be on the previous line:
createParams->videoProfileId = (profile_id->empty() ? nullptr : profile_id->c_str());

Likely whatever string type is videoProfileId does not like nullptr as argument; seems trivial to fix by passing an empty string "" instead. If you have the code, would you mind trying that locally? I will also try on my end, but that might take a bit more time than you if you already have a repro all set up.

Note that I just had a look and the fix for #492 is deleting this code. I am hoping to have a PR up by the end of the day if you prefer to wait for that instead, but not sure we'll be able to get a release out with prebuilt binaries today though.

I don't have a lot of time today anyways, so I'll wait for the push to master.
Thanks for looking into the issue, Jerome!


Feat/prompter midi pedal

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Other information:

Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[ ] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Cleaned up.
I think the loops around the RAF are safe, but I'm not sure.
I see memory grow as I run the pedal, but I'm not sure if that is just normal behaviour or a leakage.

@jstarpl care to add the interpolation?

Functionality is in place, code looks good.
Documentation is not done.

@jstarpl can the debug box obey the margin that's being set?

Description added

@jstarpl can the debug box obey the margin that's being set?

Done.

Took a lot just post merge. Looks good.
Confirmed new dependencies are OK.


SQL contains ISNULL function in select clause parsing fails if used with NOT

The fix for ISNULL as a keyword or as a function doesn't work with NOT as a prefix:
    @Test
    public void testSqlContainIsNullFunctionShouldBeParsed3() throws JSQLParserException {
        assertSqlCanBeParsedAndDeparsed("SELECT name, age FROM person WHERE NOT ISNULL(home, 'earn more money')");
    }

results in:
    at line 1, column 46.

Was expecting one of:

    "&&"
    ";"
    "AND"
    "CONNECT"
    "EXCEPT"
    "FOR"
    "GROUP"
    "HAVING"
    "INTERSECT"
    "MINUS"
    "ORDER"
    "START"
    "UNION"
    <EOF>

Originally posted by @damienb-opt in https://github.com/JSQLParser/JSqlParser/pull/610#issuecomment-453487603

already fixed


Restructure as pyproject

Resolves #18

Update project structure
Add pyproject.toml
Update README.md
Add __about__.py, __init__.py, __main__.py

Changed overall structure, closing in favor of #31.


Edit metadata locally

Story
I want to edit metadata locally with my favourite text editor/IDE so that I can create and update it easily.
Given that

The user can edit metadata in someway in the working environment (minimum a text editor with yaml)
When a new request from the working environment/commandline or gui is created, the metadata created is passed as input to create the new record.

Some requirements and considerations

The metadata needs to be validated against the standard before making the request
We are sticking to a basic standard, something like Dublin core (Business rule)

Implementation ideas

Metadata is created in yaml by the user
It is transformed to xml. Package example
Then using some validator against schema. XMLschema package
Then this is passed as a body request to the api.

Any ideas or issues? This raises other issues such as downloading and getting the latest version, with the yml format for instace...

This was implemented in the fairly package


Work item attachments sample fails to attach a file

WorkItemsSample.AddAttachment appears not to attempt to attach a file. Therefore it fails (and so does a subsequent sample, .RemoveAttachment).

Matt I just tested this and it works fine. the catch is you need to make sure the revision value is correct in the patch document.


Add a method to make a graph from a complex matrix

What is the expected enhancement?
Add a method like from_complex_adjacency_matrix , for makiing a graph from an adjacency matrix whose elements can be complex values.
I expect this method to work as follows:
adjacency_matrix = np.array([
    [1.0, 1.0+1.0j, -2.0j],
    [0.0, 2.0, 1.0+1.0j],
    [0.0, 0.0, 3.0]
])

graph = retworkx.PyGraph.from_complex_adjacency_matrix(adjacency_matrix)
list(graph.weighted_edge_list())

output:
[(0, 0, 1.0),
 (0, 1, (1+1j)),
 (0, 2, (-0-2j)),
 (1, 1, 2.0),
 (1, 2, (1+1j)),
 (2, 2, 3.0)]

When the input matrix is not symmetric, it may be convenient to make a graph from the elements of the upper triangular part of it, similar to from_adjacency_matrix.

So I have a draft locally implementing this but the only thing in your example which I will not able to do is reproduce the types in the output edge list. In your example for the real numbers they're floats in the output weight list, but when retworkx looks at the underlying numpy array it will see a 3x3 array complex128 and in the retworkx from_complex_adjacency_matrix() that complex number will be added as a complex so it will be (1.0+0j) not 1.0.
Other than that the PR is ready to go however unfortunately it's blocked on PyO3/pyo3#1798 and won't work yet. I will push it up as a WIP for the time being (and it'll fail on the msrv tests). So unfortunately I don't think this will make 0.10.0 as it's blocked on a pyo3 release.

Thank you for the quick response. Having the output type (1.0+0j) would not matter to me.


Add fuzz-testing

Proposal
Fuzz testing or Fuzzing is a Black Box software testing technique, which basically consists in finding implementation bugs using malformed/semi-malformed data injection in an automated fashion.
Integrate the project with OSS-Fuzz by following the instructions here.
Part of #6085
Use-Case
To increase security of the project.
Is this a feature you are interested in implementing yourself?
Yes
Anything else?
Maintainers help is highly appreciated. For example, helping in identifying the components where fuzz testing will be added.

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/6mkbh6xhau31g
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/6mkbh6xhau31g
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Ping @kedacore/keda-core-contributors

I like this

it can help find subtle bugs not covered by our unit / e2e tests. Sounds good to me.

@harshitasao shall I assign this to you? Are you willing to contribute this?


Wip/6.0 Port 'refresh' cascade testing case

Part of https://trello.com/c/OVdBw7qD/27-verify-cascading
Other types of cascade testing has been taken over by Andrea.
Fix a generic issue when loading entity by id providing existing entity (like in cascade refresh case).

@NathanQingyangXu Why change this from annotations to XML?

@NathanQingyangXu Why change this from annotations to XML (which is cause of the conflict)?

What did you mean? I saw no usage of XML in this PR. Am I missing something?

@NathanQingyangXu Why change this from annotations to XML (which is cause of the conflict)?

The conflict is because Andrea's additional @FailureExpected. Now conflict is resolved.


Update Homepage - Collaborations and Partnerships

Add a "collaborations and partnerships" section towards the bottom of the homepage. It has been added to the Figma design for reference.
The component has a grey colored background. This might also be reused in other sections (see Code of Conduct - Anti-Harassment Email Reporting section).

Can i work on this issue?

@gautamjuyal Sorry for the late reply. This issue is resolved now.

That's cool. Good thing the issue was resolved fast!


bug: Bid for futbin price option is not working.

Prerequisites

[X] I have searched for existing issues that already report this problem, without success.

Script Version
FUT AutoBuyer 1.2.8
Platform
Webapp
Current Behavior
Bid for futbin price option is not working. The following error is encountered when the option is selected.
skip search >>> (No Buy or Bid Price given)
Expected Behavior
When the Bid for futbin price option is selected, the search results must be displayed and bid based on the percentage value given.
Steps to Reproduce
I have no idea
Additional Information
Thank u so much CK :)

Fixed in the latest release


Inconsistent use of write_property/write-property

This has a dash:
https://github.com/cisagov/icsnpp-bacnet/blob/646032358b3aa994aeab8fa7acb0fe1d86d616f7/scripts/icsnpp/bacnet/main.zeek#L396
While this has an underscore:
https://github.com/cisagov/icsnpp-bacnet/blob/35243a1a3e821216df693761bc67ef1b7f3203f5/scripts/consts.zeek#L266
My logs have both dashes and underscores.  Shouldn't they be all  underscore?

Nevermind - I see why they are different now.


GetFrameBufferAsync not work

What can we do for you?
System.IO.EndOfStreamException:“Unable to read beyond the end of the stream.”

Same issue here. I've had a play about with the code but can't find a way around it. An odd thing to note is if I change headerData = new byte[52]; in FrameBuffer.cs to a larger value it gets up to returning the frameBuffer.ToImage() but then fails because it's returning to a different pool.
I was able to replicate this in both a .net 7 and .net 4.6.2 WinForms application using Android API 30.

I see the issue already. Let me see if I can start a pull request tonight.

I had the same issue with "System.IO.EndOfStreamException:“Unable to read beyond the end of the stream.” The BinaryReader.Version = 2 so in  FramebufferHeader Read(byte[] data), the BinaryReader reach 52 before Apha color assignment.
I resolve it by modified the  headerData initialisation from 52 to 56 byte. Can you modify the value in package ?
thanks by advance and for all your work ;)

I was just recently able to reproduce this. Let me see if I can find some time to work on this. I agree with @horhe4 that it should not be hardcoded.


Encrypt secrets too

It would be nice if S3 secrets could do the encryption and upload to S3 too, something like:
$ s3secrets encrypt --key-id=xxxxx --file=myfile.pem --bucket=my-secrets --region=eu-west-1

Yep, I am that lazy :)

@timgent that is probably not a bad idea. I add it soon then :-)

:+1: this would be good actually as keeps it all in one tool


Update log.md

I removed the 'you can log anything' because the specific example won't really work due to circular references. In theory another example could be used, but honestly I don't think it's necessary. If | log means console.log, I already know it means I can use it for anything I can use with console.log. (If folks disagree, it may be nice to put it back but show logging the page variable.

Thank you!


Bump dependency to TF 2.7

Blocking transformers dependency upgrade
Defintion of done

[ ] Verify with PM that performance implications are acceptable
[ ] Check that we don't need to upgrade this for 2.8
[ ] Ensure model regression test pass
[ ] Dependency upgraded to TF2.7

For regression tests, it could make sense to use the current state of the regression testing tool to compare train/test runtimes and computational resources before and after. Reach out if you need any help with that.

cc @ka-bu @usc-m

Updating to 2.7 would also help with support of the Apple M1 and AWS Graviton, based on my recent attempt so build 3.0.1.

https://github.com/RasaHQ/rasa/issues/9199 is affecting us, too. Is there a timeline on when an update with a fix for this will be released?


Oracle 19c container times out with versions <IP_ADDRESS> and onwards of the OJDBC

The Oracle 19c Docker image does not work with Version <IP_ADDRESS> and above of the Oracle JDBC driver, giving a strange "Got minus one from read call" for any ojdbc driver versioned at 19.3 or above.
I have prepared a small demo project which show cases the bug. I have ruled out network problems by sniffing traffic both inside and outside the Docker container, from SQL*Plus, and the traffic is the same. It might be a bug in the driver itself, but I post it here because I cannot rule out that the database container itself is not part of the problem (since Oracle 11g and 12c work fine).
The repository can be found here: https://github.com/danielthegray/ojdbc-bugsample/tree/master along with an explanatory README file.
The full error stack trace is pasted below:
java.sql.SQLRecoverableException: IO Error: Got minus one from a read call
        at oracle.jdbc.driver.T4CConnection.logon(T4CConnection.java:858)
        at oracle.jdbc.driver.PhysicalConnection.connect(PhysicalConnection.java:793)
        at oracle.jdbc.driver.T4CDriverExtension.getConnection(T4CDriverExtension.java:57)
        at oracle.jdbc.driver.OracleDriver.connect(OracleDriver.java:747)
        at oracle.jdbc.driver.OracleDriver.connect(OracleDriver.java:562)
        at java.sql/java.sql.DriverManager.getConnection(DriverManager.java:677)
        at java.sql/java.sql.DriverManager.getConnection(DriverManager.java:228)
        at ojdbc.bugsample.App.main(App.java:14)
Caused by: oracle.net.ns.NetException: Got minus one from a read call
        at oracle.net.ns.NSProtocolNIO.doSocketRead(NSProtocolNIO.java:557)
        at oracle.net.ns.NIOPacket.readHeader(NIOPacket.java:258)
        at oracle.net.ns.NIOPacket.readPacketFromSocketChannel(NIOPacket.java:190)
        at oracle.net.ns.NIOPacket.readFromSocketChannel(NIOPacket.java:132)
        at oracle.net.ns.NIOPacket.readFromSocketChannel(NIOPacket.java:105)
        at oracle.net.ns.NIONSDataChannel.readDataFromSocketChannel(NIONSDataChannel.java:91)
        at oracle.net.ano.AnoCommNIO.p(Unknown Source)
        at oracle.net.ano.AnoCommNIO.e(Unknown Source)
        at oracle.net.ano.AnoComm.readUB4(Unknown Source)
        at oracle.net.ano.Ano.c(Unknown Source)
        at oracle.net.ano.Ano.negotiation(Unknown Source)
        at oracle.net.ns.NSProtocol.connect(NSProtocol.java:368)
        at oracle.jdbc.driver.T4CConnection.connect(T4CConnection.java:1600)
        at oracle.jdbc.driver.T4CConnection.logon(T4CConnection.java:591)
        ... 7 more

If you have any other questions, please let me know!

I experienced the same issue and solved it by providing connection property oracle.net.disableOob=true.

This made my showcase application pass the test with all versions of the driver! Thanks for this. I added the proposed solution to a branch on the bugsample repo: https://github.com/danielthegray/ojdbc-bugsample/tree/connection_prop_fix
where all the versions succesfully connect.
However, I believe that there must be some underlying problem that causes the need for this connection parameter. If I use a modern version of SQL*Plus (or if I use SQLCL, the new CLI tool from Oracle), they also fail to connect to the Oracle 19c Docker container. I don't think I can easily (or legally?) tweak the connection parameter of these applications.
What causes this problem and how can good behavior for this combination of Docker container + modern JDBC drivers be obtained?

See the last entry of the Database FAQ: https://github.com/oracle/docker-images/blob/master/OracleDatabase/SingleInstance/FAQ.md

How to configure this DISABLE_OOB=ON if I use the official container-registry.oracle.com/database/enterprise:<IP_ADDRESS> image?

How to configure this DISABLE_OOB=ON if I use the official container-registry.oracle.com/database/enterprise:<IP_ADDRESS> image?

I have opened a new issue for your question: #1727

The Oracle 19c Docker image does not work with Version <IP_ADDRESS> and above of the Oracle JDBC driver, giving a strange "Got minus one from read call" for any ojdbc driver versioned at 19.3 or above.
I have prepared a small demo project which show cases the bug. I have ruled out network problems by sniffing traffic both inside and outside the Docker container, from SQL*Plus, and the traffic is the same. It might be a bug in the driver itself, but I post it here because I cannot rule out that the database container itself is not part of the problem (since Oracle 11g and 12c work fine).
The repository can be found here: https://github.com/danielthegray/ojdbc-bugsample/tree/master along with an explanatory README file.
The full error stack trace is pasted below:
java.sql.SQLRecoverableException: IO Error: Got minus one from a read call
        at oracle.jdbc.driver.T4CConnection.logon(T4CConnection.java:858)
        at oracle.jdbc.driver.PhysicalConnection.connect(PhysicalConnection.java:793)
        at oracle.jdbc.driver.T4CDriverExtension.getConnection(T4CDriverExtension.java:57)
        at oracle.jdbc.driver.OracleDriver.connect(OracleDriver.java:747)
        at oracle.jdbc.driver.OracleDriver.connect(OracleDriver.java:562)
        at java.sql/java.sql.DriverManager.getConnection(DriverManager.java:677)
        at java.sql/java.sql.DriverManager.getConnection(DriverManager.java:228)
        at ojdbc.bugsample.App.main(App.java:14)
Caused by: oracle.net.ns.NetException: Got minus one from a read call
        at oracle.net.ns.NSProtocolNIO.doSocketRead(NSProtocolNIO.java:557)
        at oracle.net.ns.NIOPacket.readHeader(NIOPacket.java:258)
        at oracle.net.ns.NIOPacket.readPacketFromSocketChannel(NIOPacket.java:190)
        at oracle.net.ns.NIOPacket.readFromSocketChannel(NIOPacket.java:132)
        at oracle.net.ns.NIOPacket.readFromSocketChannel(NIOPacket.java:105)
        at oracle.net.ns.NIONSDataChannel.readDataFromSocketChannel(NIONSDataChannel.java:91)
        at oracle.net.ano.AnoCommNIO.p(Unknown Source)
        at oracle.net.ano.AnoCommNIO.e(Unknown Source)
        at oracle.net.ano.AnoComm.readUB4(Unknown Source)
        at oracle.net.ano.Ano.c(Unknown Source)
        at oracle.net.ano.Ano.negotiation(Unknown Source)
        at oracle.net.ns.NSProtocol.connect(NSProtocol.java:368)
        at oracle.jdbc.driver.T4CConnection.connect(T4CConnection.java:1600)
        at oracle.jdbc.driver.T4CConnection.logon(T4CConnection.java:591)
        ... 7 more

If you have any other questions, please let me know!

What is workaround for this issues?

I experienced the same issue and solved it by providing connection property oracle.net.disableOob=true. This property name is defined by oracle.jdbc.OracleConnection#CONNECTION_PROPERTY_THIN_NET_DISABLE_OUT_OF_BAND_BREAK.

Where is that property file located?

Where is that property file located?

See the FAQ entry: https://github.com/oracle/docker-images/blob/main/OracleDatabase/SingleInstance/FAQ.md#ora-12637-packet-receive-failed

I experienced the same issue and solved it by providing connection property oracle.net.disableOob=true. This property name is defined by oracle.jdbc.OracleConnection#CONNECTION_PROPERTY_THIN_NET_DISABLE_OUT_OF_BAND_BREAK.

Where is that property file located?

It's a connection property which is part of the JDBC URL, e.g. jdbc:oracle:thin:@localhost:1521/service?oracle.net.disableOob=true according to https://docs.oracle.com/en/database/oracle/oracle-database/21/jajdb/.
I don't know if it also works via a system property in general, i.e. -Doracle.net.disableOob=true. But that's how SQL Developer can be configured according to https://stackoverflow.com/a/10100981.

#2607  This PR includes the DISABLE_OOB=ON workaround which would solve this problem


Arcane Pattern Crafter Not Working in Beta 19

Trying to craft flour as shown in the screenshots but it's not working. It will craft fine in my inventory. Running Beta 19 in the latest DW20 pack on a server. Am I doing something wrong?

AFAIK arcane pattern crafter intentionally works with recipes that use single ingredient, like stone bricks, stone brick slabs, block of coal, etc.

Ah, it does work with single item recipes. Though I think it would be way more useful to be able to craft any recipe. Like for a golem ran pizzeria for example. :)


PATH issue finding MAS, when running REMOTELY

Running my playbook on my Mac acting as control node and managing itself as localhost, everything works fine with MAS:
$ ansible-playbook -i inventory/localhost.yml macOS_app-installs.yml --ask-become-pass
...

TASK [geerlingguy.mac.mas : Ensure MAS is installed.] *******************************************************************************************************************************************************
ok: [<IP_ADDRESS>]
...
TASK [geerlingguy.mac.mas : List installed MAS apps.] *******************************************************************************************************************************************************
ok: [<IP_ADDRESS>]
...

However, when I try running the same playbook via SSH remotely, something fails after detecting the installed MAS, and when MAS is called to run (and it comes back, "[Errno 2] No such file or directory: b'mas'"):
$ ansible-playbook -i inventory/mac_ssh.yml macOS_app-installs.yml --private-key ~/.ssh/id_rsa --ask-become-pass
...
TASK [geerlingguy.mac.mas : Ensure MAS is installed.] **************************************************************************************************************
ok: [<IP_ADDRESS>]
...
TASK [geerlingguy.mac.mas : List installed MAS apps.] **************************************************************************************************************
fatal: [<IP_ADDRESS>]: FAILED! => {"changed": false, "cmd": "mas list", "msg": "[Errno 2] No such file or directory: b'mas'", "rc": 2, "stderr": "", "stderr_lines": [], "stdout": "", "stdout_lines": []}

I can correct this problem when coming in remotely by tweaking the PATH in the environment, like this:
  environment:
    PATH: /usr/local/bin:{{ lookup('ansible.builtin.env', 'PATH') }}

  roles:
    - geerlingguy.mac.homebrew
    - geerlingguy.mac.mas

that is, by injecting into the PATH environment variable where (on my Intel Mac) the mas executable is.
$ ls -l $(which mas)
lrwxr-xr-x  1 user  group  27 Dec 20 14:08 /usr/local/bin/mas -> ../Cellar/mas/1.8.6/bin/mas

(I'd imagine, on an Apple Silicon Mac, if Apple Silicon systems have the same problem, one might need to inject "/opt/homebrew" instead of "/usr/local/bin".)
However, it seems strange that "TASK [geerlingguy.mac.mas : Ensure MAS is installed.]" works fine (i.e., it finds the mas executable), but then when it comes it comes time to actually use the detected installed mas in "TASK [geerlingguy.mac.mas : List installed MAS apps.]" then it fails. (And, perhaps, if it got that far, it'd also fail in "TASK [geerlingguy.mac.mas : Ensure configured MAS apps are installed.]" as well.)
I found a couple of past issues referencing what sounds like the same/similar problems, but those both ended up going stale and getting closed.
Problems with the PATH fo mas commands #10
and
Installing apps with mas doesn't work for pubkey ssh tunnels #70
Is there a fix for this that should be put in the role code, and that would work across Intel and Apple Silicon Macs? Or is it just best to stick with tweaking the PATH in the calling playbook? Or is this a problem somewhere else that needs to be corrected instead?
Thanks for any guidance!

I am having the same problem and I am running the playbook locally. I was able to confirm that mas is indeed installed at /opt/homebrew/bin, but the task geerlingguy.mac.mas : List installed MAS apps. fails with [Errno 2] No such file or directory: b''mas'''. 4 tasks before there is a task ensuring that mas is installed.


Make the selinuxd auto-selection logic use environment variables

This patch doesn't change the functionality in any way whatsoever, but makes it easier for tools that reason about bundle or helm manifests to do their job.
I hit such issue with a tool we use in OpenShift that was not able to detect that I forgot to rewrite the quay.io upstream images in the configMap to our downstream images.
We used to put the mapping between selinuxd images and OS directly into a configmap. This is problematic for the aforementioned tools as a container pullspec is hard to find in operator manifests. To make their job easier, the configmap now points to an environment variable name that is defined in the operator deployment manifest.

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Does this PR have test?

Special notes for your reviewer:
Does this PR introduce a user-facing change?

/hold for test

Verified with both 4.13 and 4.12 clusters
4.13 cluster

$ make deploy-openshift-dev
podman build -f Dockerfile.ubi --build-arg version=0.7.1 -t security-profiles-operator:latest .
[1/2] STEP 1/10: FROM registry.access.redhat.com/ubi8/go-toolset@sha256:160ff3bc88c4921f7b602f56b6115fe8bc99212f81f1d5acf5c259210d0261f3 AS build
[1/2] STEP 2/10: USER root
--> Using cache 3972cea6ecbaa4962de158bece746e1d5035ba864cb414e940b469a01abfda00
--> 3972cea6ecb
[1/2] STEP 3/10: WORKDIR /work
--> Using cache 3b29dd32cc809c3c392236951be277f76caef6a2aa19e8ea11d59afa5593bbac
--> 3b29dd32cc8
[1/2] STEP 4/10: RUN dnf install -y         libseccomp-devel
--> Using cache f2ed645e8826f2a608c7a6f5e4d37a7653a833e321de8fbfa4643c2182dfe560
--> f2ed645e882
[1/2] STEP 5/10: ADD . /work
--> 28864e62101
[1/2] STEP 6/10: RUN mkdir -p build
--> 024983830a0
[1/2] STEP 7/10: ARG APPARMOR_ENABLED=0
...
secret/metrics-token created
deployment.apps/security-profiles-operator created
mutatingwebhookconfiguration.admissionregistration.k8s.io/spo-mutating-webhook-configuration created
Setting triggers to track image
oc set triggers -n security-profiles-operator deployment/security-profiles-operator --from-image openshift/security-profiles-operator:latest -c security-profiles-operator
deployment.apps/security-profiles-operator triggers updated

$ oc project security-profiles-operator
Already on project "security-profiles-operator" on server "https://api.bgudi-4131.ibmcloud.qe.devcluster.openshift.com:6443".
$ oc get pods
NAME                                                  READY   STATUS    RESTARTS   AGE
security-profiles-operator-7996887b-7tzxk             1/1     Running   0          3m2s
security-profiles-operator-7996887b-7v4w8             1/1     Running   0          3m11s
security-profiles-operator-7996887b-nqxql             1/1     Running   0          2m53s
security-profiles-operator-webhook-5dcc9587cf-74z2c   1/1     Running   0          3m5s
security-profiles-operator-webhook-5dcc9587cf-krkbh   1/1     Running   0          3m5s
security-profiles-operator-webhook-5dcc9587cf-v557f   1/1     Running   0          3m5s
spod-9pw8p                                            3/3     Running   0          2m29s
spod-jkwgr                                            3/3     Running   0          2m17s
spod-kbdpn                                            3/3     Running   0          2m14s
spod-n28m7                                            3/3     Running   0          2m31s
spod-p99ss                                            3/3     Running   0          2m30s
spod-xkw6r                                            3/3     Running   0          2m28s
$ oc get spod
NAME   STATE
spod   RUNNING
$ oc get pod/spod-9pw8p -oyaml | grep image:
    image: image-registry.openshift-image-registry.svc:5000/openshift/security-profiles-operator@sha256:250d842901ec50e8380247702c43d79569bcd94c7a5dac94e1b6dbd207bcb6ce
    image: quay.io/security-profiles-operator/selinuxd-el9:latest
    image: gcr.io/kubebuilder/kube-rbac-proxy:v0.13.1
    image: image-registry.openshift-image-registry.svc:5000/openshift/security-profiles-operator@sha256:250d842901ec50e8380247702c43d79569bcd94c7a5dac94e1b6dbd207bcb6ce
    image: quay.io/security-profiles-operator/selinuxd-el9:latest
    image: gcr.io/kubebuilder/kube-rbac-proxy:v0.13.1
    image: image-registry.openshift-image-registry.svc:5000/openshift/security-profiles-operator@sha256:250d842901ec50e8380247702c43d79569bcd94c7a5dac94e1b6dbd207bcb6ce
    image: quay.io/security-profiles-operator/selinuxd-el9:latest
    image: image-registry.openshift-image-registry.svc:5000/openshift/security-profiles-operator@sha256:250d842901ec50e8380247702c43d79569bcd94c7a5dac94e1b6dbd207bcb6ce
    image: quay.io/security-profiles-operator/selinuxd-el9:latest

4.12 cluster

$ make deploy-openshift-dev
podman build -f Dockerfile.ubi --build-arg version=0.7.1 -t security-profiles-operator:latest .
[1/2] STEP 1/10: FROM registry.access.redhat.com/ubi8/go-toolset@sha256:160ff3bc88c4921f7b602f56b6115fe8bc99212f81f1d5acf5c259210d0261f3 AS build
[1/2] STEP 2/10: USER root
--> Using cache 3972cea6ecbaa4962de158bece746e1d5035ba864cb414e940b469a01abfda00
--> 3972cea6ecb
[1/2] STEP 3/10: WORKDIR /work
--> Using cache 3b29dd32cc809c3c392236951be277f76caef6a2aa19e8ea11d59afa5593bbac
--> 3b29dd32cc8
...
secret/metrics-token created
deployment.apps/security-profiles-operator created
mutatingwebhookconfiguration.admissionregistration.k8s.io/spo-mutating-webhook-configuration created
Setting triggers to track image
oc set triggers -n security-profiles-operator deployment/security-profiles-operator --from-image openshift/security-profiles-operator:latest -c security-profiles-operator
deployment.apps/security-profiles-operator triggers updated
$ oc project security-profiles-operator
Now using project "security-profiles-operator" on server "https://api.oadp-38790.qe.gcp.devcluster.openshift.com:6443".
$ oc get pods
NAME                                                 READY   STATUS    RESTARTS   AGE
security-profiles-operator-65c7df679-hx49g           1/1     Running   0          2m49s
security-profiles-operator-65c7df679-khq8g           1/1     Running   0          2m54s
security-profiles-operator-65c7df679-lwft2           1/1     Running   0          3m3s
security-profiles-operator-webhook-96868fdb6-5m2qr   1/1     Running   0          2m58s
security-profiles-operator-webhook-96868fdb6-7pbj4   1/1     Running   0          2m58s
security-profiles-operator-webhook-96868fdb6-rr48p   1/1     Running   0          2m58s
spod-ds4l8                                           3/3     Running   0          2m2s
spod-jd6n7                                           3/3     Running   0          2m2s
spod-ln6k5                                           3/3     Running   0          2m2s
spod-lnf4t                                           3/3     Running   0          2m3s
spod-n7rvd                                           3/3     Running   0          2m2s
spod-xkkj8                                           3/3     Running   0          2m3s
$ oc get spod
NAME   STATE
spod   RUNNING
$ oc get pod/spod-jd6n7 -oyaml | grep image:
    image: image-registry.openshift-image-registry.svc:5000/openshift/security-profiles-operator@sha256:915067cad3319c7d7ed77935d16371f3758c62cd8d60139e6c48b23faea52356
    image: quay.io/security-profiles-operator/selinuxd-el8:latest
    image: gcr.io/kubebuilder/kube-rbac-proxy:v0.13.1
    image: image-registry.openshift-image-registry.svc:5000/openshift/security-profiles-operator@sha256:915067cad3319c7d7ed77935d16371f3758c62cd8d60139e6c48b23faea52356
    image: quay.io/security-profiles-operator/selinuxd-el8:latest
    image: gcr.io/kubebuilder/kube-rbac-proxy:v0.13.1
    image: image-registry.openshift-image-registry.svc:5000/openshift/security-profiles-operator@sha256:915067cad3319c7d7ed77935d16371f3758c62cd8d60139e6c48b23faea52356
    image: quay.io/security-profiles-operator/selinuxd-el8:latest
    image: image-registry.openshift-image-registry.svc:5000/openshift/security-profiles-operator@sha256:915067cad3319c7d7ed77935d16371f3758c62cd8d60139e6c48b23faea52356
    image: quay.io/security-profiles-operator/selinuxd-el8:latest

/unhold


Added ppc64le support

This PR will add multi-architecture support to the application-service operator image

/retest


add Exercise::isLocked flag

Coverage increased (+0.02%) to 74.061% when pulling 035b08708e9d23300550ce4160db8b83d28881d5 on exercise-locked-flag into 430f7920ab8d0ab6259ff288b5e2b2f02817e712 on master.


Allow overflow scrolling when in mobile preview mode

In the mobile preview mode, we don't have the proper overflow scrolling for cases where the window isn't large enough to display the entire contents.  This is probably also a problem with desktop mode, but isn't going to get hit as often.

how would we fix this? make the size of the phone smaller? Seems like it won't look too good

Oh wait, nvm I understand now

Fixed with https://github.com/humphd/brackets-browser-livedev/pull/64


fix: show nutrient button when image is present

What

The button to open the nutrition page appears after the image upload is done for nutrient imaged
This pr will work best when the pr #2968 merges, as it will automatically add the button after image upload

Screenshot

Fixes bug(s)

Fixes: #2977

Part of

#1379

thank you @AshAman999 👍


Error 3030000: transaction validation exception when uploading hello contract

With a clean install of DAWN 4 I get the following error when I push the hello contract to a new test account.
Note that I have not installed the system contract. If this is an error caused by lack of system contract please consider this issue a request to correct the response message.
cleos --wallet-url http://wallet:5555 -u http://server:7777 set contract testuser5523 /tmp/build/contracts/hello -p testuser5523
Reading WAST/WASM from /tmp/build/contracts/hello/hello.wasm...
Using already assembled WASM...
Publishing contract...
Error 3030000: transaction validation exception
Ensure that your transaction satisfy the contract's constraint!
Error Details:
!"unresolvable": env.current_time

function current_time does not exist, please ensure your code satisfies the provided api functions


Can we switch to using chan/signal based waits

Is there a way that we can switch from timeoutes to channel based controls for http operations, and for the wait operations?
The concept of "waiting for a process to finish" has many pitfalls.  Using things like bool chans to mark when a process is finished gives lib consumers many more options on coding using the api.
Using something like https://godoc.org/golang.org/x/net/context is pretty standard.  It provides a wrappable context that allows for timeouts, but also provides alternate cancellation approaches, and natively handles "cancel functions".  This functionality is easily integrated into the net/http libraries, and is commonly used by big projects (like docker/libcompose.)
Reasoning:

using timeouts is limiting, as it provides no alternative break methods to be combined
Go suggest  chans and subroutines, over waits

Ideas:

Make all http requests use https://godoc.org/golang.org/x/net/context for all http related actions , instead of just setting a timeout.  This can be done internally first, and then context could be assignable in the service or client, and perhaps also made a part of any service methods (that would be a pretty big API change.)

Make all of the Wait methods use https://godoc.org/golang.org/x/net/context instead of timeouts.  This means that code that uses the library can use any form of context shutdown, and consuming code can then track chans instead of waiting for a function to call.

Goal:
Convert the WaitForXXX methods (and all http requests) to more of a "fire-and-forget" forked approach, as opposed to a functional in-line "wait-for-finish" approach.
Required effort:

The client would need a way to receive a context (either across it's entire scope, or per request)

The client.PerformGetRequest() method would need to attach the context to the http Request using https://golang.org/pkg/net/http/#Request.WithContext

The request.WaitForXXX structs should accept a https://godoc.org/golang.org/x/net/context instead of a timeout.

The service.WaitForXXX functions should check the context->Done() chan for exiting, instead of checking an overall timeout (the duration based sleep should also use a small timeout context instead of a time.Sleep())

What do you think?  I could give you a PR if you can give me some feedback on the ideas.

I must say I haven't used Go that much so I only have a vague understanding of channels and goroutines. Feel free to make a PR, if possible using an approach that changes the public API as little as possible.

Understandable request.  I will start by duplicating the WaitX methods to new chan/context based approaches to demonstrate the change.  Then I can show you some of the advantages.
I will leave the http request mechanism alone for now.
In case it isn't clear, I will create a fork, and start a PR from the fork.

Thanks, please do that!

I did manage to take a first run at this and failed miserable.  Will try again soon.
I am looking at the following goals for the WaitXXX methods:

be able to show some progress (upcloud.Server.Progress)
be able to cancel the wait from outside of the wait (on input event)
be able to run multiple WaitXXXs in parrallel

Hello from 2020. There are new changes to this API/SDK, go update among others, and I am reviewing all the old issues. Thinking of closing this issue, since no PR has been issued and this hasn't bubbled as a request, though it might in the future. Will give you time, before closing, to post a plea for future feature request and we will consider upgrading this feature request again. :)

Hello! We are now using contexts for both requests (see v4.7.0 and v5.0.0) and WaitFor* methods (see v7.0.0), so I'll close this as completed 🚢


Native flora doesn't show up when searched from remote view

Mod source
Factorio Mod Portal
Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[X] Locale (names, descriptions, unknown keys)
[ ] Graphical
[ ] Crash
[ ] Progression
[ ] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
In 2.0's remote view you can type in ore name in the search bar and it'll show all its patches. For example, this is what shows when i type in "iron ore"

However when typing in "Native flora" native flora patches don't show

Weird thing is that when you type in just "ore" (and the list of all ores shows up under the search bar) native flora does show up there (despite not having "ore" in its name)

Steps to reproduce
No response
Additional context
No response
Log file
No response

"Native Flora" is mined from "Bioreserve"

"Native Flora" is mined from "Bioreserve"

Looks like you're right.

This is very confusing and counterintuitive (the fact that the patch's name from the map view doesn't match with the name you type in to search for it) as well as inconsistent with how every other ore in the game is name and searched.

Thanks for the report, I agree in principle that the name should be changed. Changing the internal name of something is hellacious to fix unfortunately (it's a migration hell), but we can do this for the PySE release. Keeping this bug report open until then.


Production styling issue

Figure out why there is extra spacing in the header only in production:

This is not the case for dev, qa, stage, or development environments.

Fixed when moved to k8s.

This is happening again so reopening.


Add --no-debug flag to CLI and use that in boxes

Since we want to distribute artifacts in the boxes and because we include debug info in the artifacts we are getting these annoying diffs upon rebuilding:

We don't need the debug info in the boxes because so we can remove that. Since we use azte-cli to do that the most elegant way to do this is to add the --no-debug flag there.

Not planned


/ issue when using --reuse-values=true

What happened:
root@kube-apiserver-01:~# helm upgrade phpmyadmin test/phpmyadmin --reuse-values=true
Error: UPGRADE FAILED: yaml: line 20: could not find expected ':'
root@kube-apiserver-01:~#
root@kube-apiserver-01:~# helm upgrade phpmyadmin test/phpmyadmin --set ingress.path=\/
Release "phpmyadmin" has been upgraded. Happy Helming!
LAST DEPLOYED: Fri May 11 16:41:45 2018
NAMESPACE: test
STATUS: DEPLOYED

RESOURCES:
==> v1/ConfigMap
NAME        DATA  AGE
phpmyadmin  1     11m

==> v1/Service
NAME        TYPE       CLUSTER-IP    EXTERNAL-IP  PORT(S)  AGE
phpmyadmin  ClusterIP  <IP_ADDRESS>  <none>       80/TCP   11m

==> v1/Deployment
NAME        DESIRED  CURRENT  UP-TO-DATE  AVAILABLE  AGE
phpmyadmin  1        2        1           1          11m

==> v1beta1/Ingress
NAME        HOSTS                 ADDRESS  PORTS  AGE
phpmyadmin  phpmyadmin.localhost  80       11m

==> v1/Pod(related)
NAME                         READY  STATUS             RESTARTS  AGE
phpmyadmin-777bf6748f-xz2p2  0/1    ContainerCreating  0         0s
phpmyadmin-7869d7bf75-t6tq5  1/1    Running            0         9m

root@kube-apiserver-01:~# helm upgrade phpmyadmin test/phpmyadmin --reuse-values=true
Error: UPGRADE FAILED: yaml: line 4: could not find expected ':'

Ingress:
---
{{- if .Values.ingress.enabled -}}
{{- $ingressPath := .Values.ingress.path -}}
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: {{ .Chart.Name }}
{{- with .Values.ingress.annotations }}
  annotations:
{{ toYaml . | indent 4 }}
{{- end }}
spec:
  rules:
  {{- range .Values.ingress.hosts }}
    - host: {{ . }}
      http:
        paths:
          - path: {{ $ingressPath }}
            backend:
              serviceName: phpmyadmin
              servicePort: http
  {{- end }}
{{- end }}

values.yaml:
ingress:
  enabled: true
  annotations:
    kubernetes.io/ingress.class: traefik
  path: "/"
  hosts:
    - phpmyadmin.localhost

Output of helm version:
Server: &version.Version{SemVer:"v2.9.0", GitCommit:"f6025bb9ee7daf9fee0026541c90a6f557a3e0bc", GitTreeState:"clean"}```
Output of `kubectl version`:
```Client Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.2", GitCommit:"5fa2db2bd46ac79e5e00a4e6ed24191080aa463b", GitTreeState:"clean", BuildDate:"2018-01-18T10:09:24Z", GoVersion:"go1.9.2", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.2", GitCommit:"5fa2db2bd46ac79e5e00a4e6ed24191080aa463b", GitTreeState:"clean", BuildDate:"2018-01-18T09:42:01Z", GoVersion:"go1.9.2", Compiler:"gc", Platform:"linux/amd64"}

Seeing this same issue when trying to upgrade any charts with --reuse-values
Version
Client: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}

if I add --set some=thing then its fine, and the upgrade works.

--reuse-values has never really worked for me.  It makes me wonder whether I really understand it's use???
Someone, please correct me if I am wrong.
rawVals is a string slice and I assume it holds all the keyPairs applied with --set or -f.  Somehow, these aren't being passed to the UpdateRelease function or they are????? https://github.com/kubernetes/helm/blob/374a320d0f15c22b736c2d9cd6a0b69d941f14db/cmd/helm/upgrade.go#L219
I'm not familiar with the code base at all so need to look into it further...

I just upgraded to the latest version and I am also running into this issue. --reuse-values used to work with no issues.
Client: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}

Adding --set some=thing (literally this) fixes the issue.

dupe of #4337, which is being fixed via #4364