regex String .match flow type wrong

Currently it's defined as ?Array<string> for return type.
However, the returned object is something like
["12:12a", "a", index: 0, input: "12:12a"]
https://github.com/facebook/flow/blob/v0.41.0/lib/core.js#L286

Hi. Is there any way to override this in local project?

String.prototype.match will return null if there is no match. For example, "abc".match(/foo/) evaluates to null.

@samwgoldman - this is the array extending thing we discussed last week.
Someone tried to improve it in https://github.com/facebook/flow/commit/9611080b787711a392b8d5548241ecb4ed784e6b but I had to revert in https://github.com/facebook/flow/commit/3107fb3394cce82abd3c964e992d1178da33fb2d 😢

I suppose it must be something like this:
declare class RegExpMatchArray extends Array<string> {
    index?: number;
    input?: string;
}

and
match(regexp: string | RegExp): RegExpMatchArray | null;

oh, I misunderstood the original post. sorry about that.

so maybe re-open this issue?
thanks @samwgoldman!

I'd use:
declare class RegExpMatchArray extends Array<string> {
   index: number;
   input: string;
}

and
match(needle: string | RegExp): Array<String> | RegExpMatchArray | null;

and for RegExp.prototype:
exec(haystack: string): RegExpMatchArray | null;

A common usecase is to check that a match result is not null, and then use array destructuring to get the capture groups, like:
lines.map(x => x.match(regex:RegExp))
    .filter(x => x !== null)
    .map(func:([string, string, string])=>({a: string, b: number, c: boolean})))

This causes a collision between Array<string> and some tuple of [string,...,string] sort.
How can I solve this without casting through any? Only by writing typechecks for each string even though the entire match either succeeded or failed?

@sm3ster your fundamental issue is that Flow doesn't understand properly the filter step and so doesn't rule out null results. See here
I believe this is another issue than this one.

@julienw, I already refactored the filter+map into a reduce (ew tho).
The issue is with the signature of func:
If I say it takes an Array<string> (which string#match(RegExp) currently claims to output) I am be accused of unsafe array index access;
But if I declare it as any specific tuple it is incompatible with an Array<string>
Both are silly, but understandable since flow doesn't understand the structure of RegExps, which is what dictates the size of the tuple:
string.match(regexpWithNoCaptureGroups): null | [string]
string.match(regexpWith1CaptureGroup):   null | [string, string]
string.match(regexpWith2CaptureGroups):  null | [string, string, string]
string.match(regexpWith3CaptureGroups):  null | [string, string, string, string]
// etc

Me, I need a better way to cast from arrays to tuples.
I understand the issue I am commenting in is also about the named properties index and input, but considering that unlike JS itself flow introduces Tuple types, Array<string> might not be a good return type even with extra properties.

I'm surprised, as from my testing I thought Flow let's you do whatever you want when converting from string arrays to string tuples. See this for example.

You're right, I can't recreate this anymore today. I will delete the comments.

Hm now I get back undefined :-( Why would that be undefined? https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoVUCuA7AxgFwEs5swBnABxkPwElsAVAJ0KoFN8ywAKQ7CzPgBc5fC2wBzAJQiAgkyYBDAJ4AeMmL4SAfGADeqMGFwkNYTe05gAvGD4D8AOgC2i-LgAW3YI70BGABoAZgBfYGl0I0IoHgsYDi5rJLBsTBgYKX1DI3MPJkQUtiQAUQU4Jm4AA1xFbGw4fHIqGjAAEj17QRC7bHw4cxZLMkqpbJDIsCYOTCZSOITUEKA

?T means T | null | undefined, you only handled the null case in your example (ie. use == instead of === to fix it).

Also see:

Duplicate issue #2450 – Native String#match declaration incorrect
PR #3007 – Fixes String#match and RegExp#exec declarations
PR #3359 – Add new lib def for String#match() result

Seems like the landscape has changed a bit in the last year. Opened a PR to try something like this: https://github.com/facebook/flow/pull/6790


Array of items in a section

I'm trying to setup an accordion that has a nested array of sections, like this
[
  {
    id: 1,
    title: "First",
    content: [
      {
        id: 1,
        section: 1,
        name: "First Section First Item"
      },
      {
        id: 2,
        section: 1,
        name: "First Section Second Item",
      },
    ],
  },
  {
    id: 2,
    title: "Second",
    content: [
      {
        id: 3,
        section: 2,
        name: "Second Section First Item"
      },
    ],
  },
]

How could I go about rendering this in an accordion? Could a flat list be used, if so how would that work?

You should be able to do that with a nested Accordion. You can use the flat list option too, but only for the outer Accordion


dist: redhat: provide only a single version

otherwise, rpm's dependency system interprets it as this package provides all versions.
Refs https://github.com/scylladb/scylladb/issues/13222

@syuu1228 hi Takuya, could you help review this change?

changelog

dropped the Provides section.

Will get into master after https://github.com/scylladb/scylladb/pull/13328.


Support --column-extra-derives

PR Info
Doesn't close any issue.
Functionally related to, but not overlapping with: https://github.com/SeaQL/sea-orm/pull/1934 & https://github.com/SeaQL/sea-orm/pull/1952
New Features

[x] New --column-extra-derives CLI option allows for adding custom derives to column enum.

Bug Fixes
None.
Breaking Changes
Fully backward compatible for the end user.
Changes
None.

Do let me know if you need help!

Hi @billy1624 for the compact entity format there is no column enum so it's passed but not used.

Hi @billy1624 is it ok or I should not pass it + update the assert_serde_variant_results?

Hi @billy1624 can you check again the PR?

Any plans on adding this in the near future? Would be nice to have instead of manually writing PartialEq impl's for the columns enum.

@billy1624 ?


docs: update README.md

Coverage remained the same at 73.423% when pulling f28c65411111e8e990ff0206720efc63f1e3bd88 on docs-update-README.md into 1d939ad69cfa695a0c0c903383f6fdfd3c6e6337 on main.


Implement a 'canary' pass that optimizes well-know patterns in ruby code

This adds a concept of pass-0, which doesn't 'lower' the AST to the c++ one but rather transforms a ruby-ast into a 'simplified' ruby-ast.
By simplified we do not mean during compilation time, but rather at runtime, so we can avoid unnecessary overhead (like allocating an array to get the min/max out of a few numbers).
Inspired by: https://ruby-compilers.com/examples/#example-program-canaryrb

Whoa cool! 😎

Whoa cool! sunglasses

this first version is still a bit naive but I guess it illustrates the process :) I wonder though just how many methods we can improve this way, should I create an issue and start collecting them?

Thanks for the fixes @seven1m !

Thanks for the fixes @seven1m !


No relevant user found

Hello beautiful people,

I use this docker image : jc21/nginx-proxy-manager:latest
NPM version : v2.9.19
Data stored in mariadb.

PROBLEM :
Today i can not connect to NPM GUI.
First when login page loads i see this message in the logs : [1/10/2023] [8:38:46 PM] [Express  ] › ⚠  warning   invalid signature
Next when connecting with my usual credentials (mail + password + Sign in button) a have this red message : No relevant user found.
On the mariadb side i don't see NPM mysql user process connected.
In the mysql user table i see my user i usually use to connect.
Looks like NPM is not connected to mariadb ? But i don't see anything related to mariadb connexion in the logs.

TRYED :
-serched "No relevant user found" issues here
-reapired mysql database
-tested mysql connexion via HeidiSQL and NPM mysql credentials
-had * character inside NPM mysql password : changed to numbers and letters without * character
-verified mysql container IP
-recreated/restarted NPM and mariadb container
-cleared all unused docker images
-restored docker appdata folder from yesturday
-restored synology nas config from yesturday
-verified synology nas firewall and router config
-disabled synology nas firewall and ip block list
-restarted synology nas
-verified internet box config
-verified NPM admin mail and password : no special caracters in mail password only lowercase letters and numbers
-cleared chrome cache
(nothing helped)
Interestingly all cloudflare+npm redirections work well except NPM url witch gives Bad getaway 502 error from web
WEB : https://my-npm-url.mydomain.ovh -> bad getaway 502 error
LAN : http://<IP_ADDRESS>:18581/login -> connexion page -> No relevant user found

DOCKER COMPOSE :

version: '3'
services:
  srv_npm:
    container_name: ${C_NPM}
    image: ${C_NPM_IMG}
    restart: ${C_ALL_RESTART}
    hostname: ${C_NPM_HOST}
    ports:
      - ${C_NPM_80}:80
      - ${C_NPM_443}:443
      - ${C_NPM_81}:81
    volumes:
      - ${REP_APPDATA}/${C_NPM}/data:/data
      - ${REP_APPDATA}/${C_NPM}/letsencrypt:/etc/letsencrypt
    environment:
      - PUID=${PUID}
      - PGID=${PGID}
      - UMASK=${UMASK}
      - TZ=${TZ}
      - DISABLE_IPV6=true
      - DB_MYSQL_HOST=${C_MDB_HOST}
      - DB_MYSQL_PORT=${C_MDB_3306}
      - DB_MYSQL_NAME=${C_MYSQL_NPM_DB}
      - DB_MYSQL_USER=${C_MYSQL_NPM_USER}
      - DB_MYSQL_PASSWORD=${C_MYSQL_NPM_PWD}
    labels:
      - 'com.centurylinklabs.watchtower.enable=${WATCHTOWER_LABEL_ENABLE}'
    networks:
      - web
networks:
  web:
    external: true

LOGS :

[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 01_perms.sh: executing...
Changing ownership of /data/logs to 0:0
[cont-init.d] 01_perms.sh: exited 0.
[cont-init.d] 01_s6-secret-init.sh: executing...
[cont-init.d] 01_s6-secret-init.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
 Skipping.
0 renew failure(s), 1 parse failure(s)
    at ChildProcess.exithandler (node:child_process:402:12)
    at ChildProcess.emit (node:events:513:28)
    at maybeClose (node:internal/child_process:1100:16)
    at Socket.<anonymous> (node:internal/child_process:458:11)
    at Socket.emit (node:events:513:28)
    at Pipe.<anonymous> (node:net:301:12)
[1/10/2023] [8:38:46 PM] [Express  ] › ⚠  warning   invalid signature

Help please,

Best regards :)

Because everything is stored into mariadb database,
i renamed the appdata folder and started with new empty folder :
    volumes:
      - ${REP_APPDATA}/${C_NPM}/data:/data
      - ${REP_APPDATA}/${C_NPM}/letsencrypt:/etc/letsencrypt

After recreating container same situayion, user not recognized,
so i tested with the NPM default user :
Manager username<EMAIL_ADDRESS>Default Proxy Manager password: changeme.

And guess what, i am connected now on an empty NPM : no data.
So it is definitly mariadb database connexion problem.
In the log below i see :
[1/11/2023] [11:43:45 AM] [Global   ] › ℹ  info      Generating MySQL knex configuration from environment variables
[1/11/2023] [11:43:45 AM] [Global   ] › ⬤  debug     Wrote db configuration to config file: ./config/production.json
[1/11/2023] [11:43:47 AM] [Migrate  ] › ℹ  info      Current database version:<PHONE_NUMBER>5214

Looks like he generates mysql connexion file,
but not connected to mysql, i suppose it uses new empty sqlite database instead.

FULL LOG :

[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 01_perms.sh: executing...
Changing ownership of /data/logs to 0:0
[cont-init.d] 01_perms.sh: exited 0.
[cont-init.d] 01_s6-secret-init.sh: executing...
[cont-init.d] 01_s6-secret-init.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
Generating dummy SSL certificate...
Generating a RSA private key
........+++++
......................................................................+++++
writing new private key to '/data/nginx/dummykey.pem'
-----
Complete
Disabling IPV6 in hosts
❯ Disabling IPV6 in hosts: /etc/nginx/conf.d
  ❯ /etc/nginx/conf.d/default.conf
  ❯ /etc/nginx/conf.d/include/assets.conf
  ❯ /etc/nginx/conf.d/include/block-exploits.conf
  ❯ /etc/nginx/conf.d/include/force-ssl.conf
  ❯ /etc/nginx/conf.d/include/ip_ranges.conf
  ❯ /etc/nginx/conf.d/include/letsencrypt-acme-challenge.conf
  ❯ /etc/nginx/conf.d/include/proxy.conf
  ❯ /etc/nginx/conf.d/include/ssl-ciphers.conf
  ❯ /etc/nginx/conf.d/include/resolvers.conf
  ❯ /etc/nginx/conf.d/production.conf
Disabling IPV6 in hosts
❯ Disabling IPV6 in hosts: /data/nginx
[1/11/2023] [11:43:45 AM] [Global   ] › ℹ  info      Generating MySQL knex configuration from environment variables
[1/11/2023] [11:43:45 AM] [Global   ] › ⬤  debug     Wrote db configuration to config file: ./config/production.json
[1/11/2023] [11:43:47 AM] [Migrate  ] › ℹ  info      Current database version:<PHONE_NUMBER>5214
[1/11/2023] [11:43:48 AM] [Setup    ] › ℹ  info      Creating a new JWT key pair...
[1/11/2023] [11:43:52 AM] [Setup    ] › ℹ  info      Wrote JWT key pair to config file: /app/config/production.json
[1/11/2023] [11:43:52 AM] [Setup    ] › ℹ  info      Logrotate Timer initialized
[1/11/2023] [11:43:53 AM] [Setup    ] › ℹ  info      Logrotate completed.
[1/11/2023] [11:43:53 AM] [IP Ranges] › ℹ  info      Fetching IP Ranges from online services...
[1/11/2023] [11:43:53 AM] [IP Ranges] › ℹ  info      Fetching https://ip-ranges.amazonaws.com/ip-ranges.json
[1/11/2023] [11:43:53 AM] [IP Ranges] › ℹ  info      Fetching https://www.cloudflare.com/ips-v4
[1/11/2023] [11:43:53 AM] [IP Ranges] › ℹ  info      Fetching https://www.cloudflare.com/ips-v6
[1/11/2023] [11:43:53 AM] [SSL      ] › ℹ  info      Let's Encrypt Renewal Timer initialized
[1/11/2023] [11:43:53 AM] [SSL      ] › ℹ  info      Renewing SSL certs close to expiry...
[1/11/2023] [11:43:53 AM] [IP Ranges] › ℹ  info      IP Ranges Renewal Timer initialized
[1/11/2023] [11:43:53 AM] [Global   ] › ℹ  info      Backend PID 242 listening on port 3000 ...
[1/11/2023] [11:43:56 AM] [Nginx    ] › ℹ  info      Reloading Nginx
[1/11/2023] [11:43:56 AM] [SSL      ] › ℹ  info      Renew Complete
[1/11/2023] [11:44:38 AM] [Express  ] › ⚠  warning   invalid signature
`QueryBuilder#allowEager` method is deprecated. You should use `allowGraph` instead. `allowEager` method will be removed in 3.0
`QueryBuilder#eager` method is deprecated. You should use the `withGraphFetched` method instead. `eager` method will be removed in 3.0
QueryBuilder#omit is deprecated. This method will be removed in version 3.0
Model#$omit is deprected and will be removed in 3.0.

Problem solved :
I mapped /app/config/ inside docker compose file to check
/app/config/production.json file,
and found there wrong mysql connexion variables values
that connect to my Ombi database instead of NPM database !

      - DB_MYSQL_NAME=${C_MYSQL_NPM_DB}
      - DB_MYSQL_USER=${C_MYSQL_NPM_USER}
      - DB_MYSQL_PASSWORD=${C_MYSQL_NPM_PWD}

when y tryed to connect ombi to mariadb, i copy-pasted NPM connexion variables,
and filled values for ombi connexion,
but forgot to rename new variables for ombi :

C_MYSQL_NPM_DB should have been renamed to C_MYSQL_OMBI_DB ... etc ...

So i had C_MYSQL_NPM_DB twice in my .env file with Ombi connexion values wich overcharged NPM connexion values :

# npm
C_MYSQL_NPM_DB=npm_db
C_MYSQL_NPM_USER=npm user
C_MYSQL_NPM_PWD=npm pwd
# ombi
C_MYSQL_NPM_DB=ombi_db
C_MYSQL_NPM_USER=ombi user
C_MYSQL_NPM_PWD=ombi pwd

:)


Tabular profile json fails to load

The following stack trace was received while calling datapackage.validate(descriptor).
The problem is that the file profiles/tabular-data-package.json contains non-ascii characters, and this fails the loading of the profile json (...such as currencies e.g. ``€95`` or ``EUR 95``.)
   File "/Users/adam/code/os/datapackage-pipelines/.tox/py36/lib/python3.6/site-packages/datapackage/registry.py", line 76, in get
     self._profiles[profile_id] = self._get_profile(profile_id)
   File "/Users/adam/code/os/datapackage-pipelines/.tox/py36/lib/python3.6/site-packages/datapackage/registry.py", line 95, in _get_profile
     return self._load_json_file(path)
   File "/Users/adam/code/os/datapackage-pipelines/.tox/py36/lib/python3.6/site-packages/datapackage/registry.py", line 139, in _load_json_file
     return json.load(f)
   File "/Users/adam/.pyenv/versions/3.6.1/lib/python3.6/json/__init__.py", line 296, in load
     return loads(fp.read(),
   File "/Users/adam/code/os/datapackage-pipelines/.tox/py36/lib/python3.6/encodings/ascii.py", line 26, in decode
     return codecs.ascii_decode(input, self.errors)[0]
 UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 21988: ordinal not in range(128)

It seems the old registry code doesn't work properly. I need to check tomorrow but quick idea:
    def _load_json_file(self, path):
        with open(path, 'r') as f: # with io.open(path, 'r', encoding='utf-8') as f ?
            return json.load(f)

Yes, adding an encoding='utf-8' argument to the open statement fixed the problem.
Will send a PR soon (debugging some other stuff now).


mbrola: convert data to a fixed-output derivation

It's quite annoying that we had to redownload this 0.65G on every rebuild.  Also disk space gets saved, unless you used some deduplication (e.g. nix.settings.auto-optimise-store). And cache.nixos.org space, too.
Things done
The resulting tree of the exposed/aggregate mbrola attribute looks the same to me, though symlinks happen at a bit different names.

Exposing the two parts as top-level attributes might be helpful as well, but I don't want to clutter this PR with that.


Field type build tool

THIS IS NOT READY, this is more of a "hey, the idea works, the implementation isn't ready but anyone have any disagreements about the API?"
This is a continuation of things discussed in #927.
The tool is essentially a fork of preconstruct, there's a bunch of stuff I'm going to rip out though that this tool won't need so I wouldn't recommend reviewing the implementation of this right now.
Something that I would like to hear thoughts on is the API, I've converted @keystone-alpha/fields-wysiwyg-tinymce and it works. Below is essentially the most important part of the API, to import views, you call importView which you import from @keystone-alpha/build-field-types
https://github.com/keystonejs/keystone-5/blob/7eddd38ca4de25aca4f6754e99670e3ccb1c86ce/packages/fields-wysiwyg-tinymce/src/index.js#L1-L20
TODO

[ ] figure out how dev stuff should work inside and outside the monorepo
[ ] build ESM bundles and make chunks resolve to directories that are automatically created in the dist folder with the correct package.json files and bundles
[ ] probably a bunch of other stuff

I really like what this does to the Keystone fields API.
Can you share what it would look like (in terms of file structure, screenshot or comment here would do) for the main fields package?
Also, do you think this changes anything for other views (e.g. for custom pages in the Admin UI)? There aren't any chances to the fields-loader, which we've talked about extending, just double-checking that's a separate thing.
Thoughts unrelated to the API:

The fields introduces flow for server-side code for the first time. I'm ... mixed on that? not sure what it's going to do to the dev loop. Probably fine since build-field-types is a CLI package, so the main Keystone experience isn't affected in any way 🤷♂
The wysiwyg field's index.js also moves behind the build step... which I think is OK since that's now consistent with other packages like utils that preconstruct builds for universal use anyway. But it seems a bit weird, since there's a hard boundary between the server and client parts of each field. Is that change necessary?

Can you share what it would look like (in terms of file structure, screenshot or comment here would do) for the main fields package?

Also, do you think this changes anything for other views (e.g. for custom pages in the Admin UI)? There aren't any chances to the fields-loader, which we've talked about extending, just double-checking that's a separate thing.

It shouldn't change anything about that.

The wysiwyg field's index.js also moves behind the build step... which I think is OK since that's now consistent with other packages like utils that preconstruct builds for universal use anyway. But it seems a bit weird, since there's a hard boundary between the server and client parts of each field. Is that change necessary?

Having the server side parts of field types behind a build step is what enables importView so, yes, it's necessary.
👇  is an explanation of how importView will work which might answer any other questions you might have.
We have a file like this,
import { importView } from '@keystone-alpha/build-field-types';

export let Wysiwyg = {
  ...otherStuff
  views: {
    ...otherViewsStuff
    Field: importView('./views/Field'),
  },
};

It goes through a babel plugin which turns it into this
export let Wysiwyg = {
  ...otherStuff
  views: {
    ...otherViewsStuff
    Field: ___KS_BUILD_SYSTEM_INTERNAL___importView(import('./views/Field')),
  },
};

Rollup does its thing and we have this for CommonJS builds. (and a different thing for ESM but I'm going to ignore that for this explanation) It's going to be written to the dist folder.
export let Wysiwyg = {
  ...otherStuff
  views: {
    ...otherViewsStuff
    Field: ___KS_BUILD_SYSTEM_INTERNAL___importView(Promise.resolve(require('./chunk-hash'))),
  },
};

We then transform it with another babel plugin into this
const _join = require('path').join

export let Wysiwyg = {
  ...otherStuff
  views: {
    ...otherViewsStuff
    Field: _join(__dirname, './chunk-hash'),
  },
};

And rollup will write the ESM and CJS versions of that chunk to dist/chunk-hash/not-sure-about-the-filename-yet.cjs.js and dist/chunk-hash/not-sure-about-the-filename-yet.esm.js and build-field-types will write a package.json with main and module fields to dist/chunk-hash/package.json

❤ it @mitchellhamilton

I've been experimenting and I think I've found a good solution to all of this in dev which also solves some other problems.
I've added a command called dev(I'm not 100% sure about the name) to build-field-types and preconstruct.
For each entrypoint, the command creates a symlink from where the ESM dist files are to the appropriate source file and for CJS it creates a file like below that registers a require hook, re-exports the source file and unregisters the require hook.
'use strict';

let unregister = require('/absolute/path/to/a/thing/in/build-field-types').___internalHook();

module.exports = require('/absolute/path/to/the/source/file');

unregister();

For the ESM file, we basically know the file will be compiled using webpack and Babel so we don't need to worry about how that file will be compiled.
For the CJS file, since we basically know that it'll be running in Node, we can use a require hook to handle running Babel.
Problems this solves:

We can run preconstruct dev and build-field-types dev in a postinstall hook
Each tool doesn't need their own special aliases
You don't need to run a build command after changing any source files (including the utils package)
Probably other problems I can't think of right now because this is second time I've written this comment because I accidentally closed the tab and lost it.

Before I convert the fields package to use build-field-types, I'd like to make sure that people(mainly @timleslie, @jesstelford and @JedWatson) are good with this and everything else in this PR so I don't have to update this all the time and this doesn't become merge conflict hell.

@mitchellhamilton I don't have any major concerns about what you're proposing. looks like it solves quite a few problems so gets a 👍 from me!
generally speaking, I think this whole PR is a big step forward for fields so on the condition that it actually works, I'm in favour of getting it in and then continuing to iterate as necessary.


Fix a bad span of _slf for custom receivers in #[pymethods].

This turned out to be a remnant of #1506. It notably resulted in a very confusing error:
error[E0308]: mismatched types
    --> tests\test_methods.rs:1456:9
     |
1456 |           #[pymethods]
     |           ^^^^^^^^^^^^
     |           |
     |           expected `Py<Issue1506>`, found `*mut PyObject`
     |           arguments to this function are incorrect
...
1461 | / issue_1506!(
1462 | |     #[pymethods]
1463 | |     impl Issue1506 {
1464 | |         fn issue_1506(
...    |
1536 | |     }
1537 | | );
     | |_- in this macro invocation
     |
     = note:   expected struct `pyo3::Py<Issue1506>`
             found raw pointer `*mut pyo3::ffi::PyObject`

The actual cause is that SelfType::receiver is entirely ignored in this case and _slf refers to the original argument, but it sounds like that TryFrom::try_from somehow resulted in *mut PyObject...

Great find! Thanks.
bors r+


HTML

HTML
User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
so that I can easily find and read information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, Git, an JavaScript with associated notes listed

Issue fixed


Add Chissl to contributors

Adds myself to contributors, Kaeso told me to

Hey, this file is for contributors of the SkyBlock mod only. Have you got some prs merged already? If so please link them here. If not please delete this pr as we take this file very serious and don't add others just because

Hey, this file is for contributors of the SkyBlock mod only. Have you got some prs merged already? If so please link them here. If not please delete this pr as we take this file very serious and don't add others just because

they contributed

These PRs are by "Chissl", but maybe it's an impostor with a capital "I" instead of an "L" or something? Very unclear
https://github.com/hannibal002/SkyHanni/pull/2975
https://github.com/hannibal002/SkyHanni/pull/3002
https://github.com/hannibal002/SkyHanni/pull/2996
https://github.com/hannibal002/SkyHanni/pull/3017


Revert "Bump github.com/cloudfoundry/bosh-utils from 0.0.461 to 0.0.463 in /src/acceptance_tests"

Reverts cloudfoundry/uaa-release#869

This is no longer needed.

The job that if it had failed would have caused this revert to merge passed, so this is no longer needed and being closed.


Specify that nuget package is a development dependency

Nuget tool packages should include the following within the metadata element in the nuspec:
<developmentDependency>true</developmentDependency>
This will mark obfuscar as a development dependency automatically in projects that use it. This will prevent it from being listed as a dependency of those projects. This allows authors of tools packages to chain development dependencies as well.

Thanks. Will update the package soon.


Unable to resolve dependencies "typings install && gulp check.versions && npm prune" Proxy Issue

If you are behind a corporate proxy the post install step "typings install && gulp check.versions && npm prune" step will fail. Re: This typings module issue
To fix issue create ".typingsrc" file in angular2-seed project root and set the contents to this;
proxy=YOUR_PROXY
rejectUnauthorized=false

Thanks for pointing this out! I'll add it to the FAQ section of the wiki.

@dhaselden Thhhhhhhhhhhhhhhhannnnnnnnnnnnnnnnnnnnnnnnkkkkkkkkkkkkkkkkkkkkkkksssssssssss :+1:


Fix issue/bug #28

fixed issue/bug 28
changed function downloadDirectoryFromNetSuite body
update split to path.sep

Hello, can you merge the pull request and publish a new version of the extension?

Hello, can you merge the pull request and publish a new version of the extension?


fields with non-exported types from modules crash @composite

Example:
module M
  struct T
  end
end

using .M

struct A
  t :: M.T
end

@composite struct B
  A...
end

In particular this interferes with EnumX.

Fixed by 42d10e9db09d4c8a786200de945b4e69b25ffbf7


Add Ahmad Bamieh as CommComm member

Ahmad is responsible for creating and championing @nodejs/mentorship.

Via the handy tracking tool, it looks to me that Ahmad met the (old track) conditions of membership back in March. Thank you for opening this PR, Dan 🙏🏽
Thank you @bamieh for all your time and effort on the Mentorship initiative! 👏🏽

@dshaw thanks for openning this! I'd be more than happy to become a member in the commcomm.
Thank you @chowdhurian

Given the context above, a vote has been sent to all members! 👍

Congrats @Bamieh! The CommComm has voted and approved you as a CommComm member with a majority vote.
CommComm Membership Checklist:

[ ] Invite to CommComm GitHub Team (and Node.js GitHub org, if not already member)
[ ] Submit PR to add to the the commcomm email alias
[ ] Submit PR to add to the (CommComm README.md](https://github.com/nodejs/community-committee/blob/master/README.md#community-committee-members)

Email PR: https://github.com/nodejs/email/pull/100


Flutter Support for Cobra

Have you checked the docs and existing issues?

[X] I have read all of the relevant Picovoice Platform docs
[X] I have searched the existing issues for Picovoice Platform

SDK
Flutter
Picovoice package version
does not matter
Framework version
does not matter
Platform
Android
OS/Browser version
does not matter
Describe the bug
Is it possible to create a flutter package for COBRA?
Steps To Reproduce
do not have to...
Expected Behavior
Does not matter

Hello @mhamdan91, unfortunately we do not support flutter for Cobra. If this is for a commercial product you can reach out to your sales contact at Picovoice, or otherwise contact us through sales to get the feature added.


Added missing flag 'v8debug' to Global interface

'v8debug' is an optional flag that will be set in NodeJS globals upon using the 'debug-brk' command parameter.
We're using it to dynamically detect when app is running in debug-mode.
(When not on debug-mode, the 'v8debug' flag is undefined)

thanks mate!


Distributed filesystems are awesome

I think Lustre might be failing in a way where /bin/cp exits with non-zero status, but a /bin/ls gives the correct file size.  (The file actually fails halfway through a copy and is truncated, but the local node doesn't know that yet).
By replacing ; with &&, a non-zero exit status will propagate.

man, I love these pull requests so much more than the ones in trac.

Update: I just simulated the problem, but /bin/cp had a zero exit status.  This still can't hurt, but it may not solve the problem.  I'll see if I can keep breaking Lustre.


Add hooks for FargateProfile status and delta

Description of changes:
Add additional hooks to manage the status, and set defaults for the delta, of FargateProfile.

Delta precompare to ignore uninitialised tags
Reject all attempts to update (not supported by SDK) by setting ResourceSynced to False
Prevent controller from attempting multiple deletes by checking for existing DELETING status

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Issue when running setup.sh with importing key-pair to ec2 service

fileb:// needs to be used instead of file:// using AWS CLI v2
AWS CLI version: aws-cli/2.7.1 Python/3.9.11 Linux/4.14.287-215.504.amzn2.x86_64 exe/x86_64.amzn.2
The following command works with the updated fileb://
aws ec2 import-key-pair --key-name "eks-saas" --public-key-material fileb://~/.ssh/id_rsa.pub

Hi @simith Thanks for reporting this. We are currently in the process of moving a new set of features and updates to this project. This is all available in the vnext branch. Could you please try there and let us know. Also, the above command is not required anymore, so you will not run in to that specific issue. Please let us know how it goes.

This has been fixed. Closing issue


[BUG] Paying project when not connected to wallet

Summary
With wallet disconnected, attempting to pay a project does not prompt user to connect their wallet.
What is the expected correct behavior?
Should show wallet connect prompt.
Relevant logs and/or screenshots
https://user-images.githubusercontent.com/96150256/151700684-2a767e8f-16e0-463f-a9b2-1bfd48575fbf.mp4

thanks for logging this! i've noticed this bug on several other buttons that should initiate transactions (it might be all of them)


SASS-3102 - Remove Encryption Service from Employment and use a mecha…

…nism similar to the Encryption and Decryption in CIS.
Description
Please include a brief summary of the change you've made and the issue that it fixes.
Add a link to the relevant story in Jira
SASS-3102
Checklist PR Reviewer
Before Reviewing

[x]  Have you pulled the branch down?
[x]  Have you assigned yourself to the PR?
[x]  Have you moved the task to “in review” on JIRA?
[x]  Have you checked to ensure all dependencies are up to date?
[x]  Have you checked to ensure its been rebased against the current version of main?

Whilst Reviewing

[x]  Have you run the tests?

[x]  Have you looked at the JIRA story to make sure all Acceptance Criteria has been met?

After Reviewing

[ ]  Have you checked for merge conflicts or any changes in the current main that may affect the current pull request? i.e. does it need another rebase?
[ ]  Have you checked to make sure there are no builds in the pipeline before you merge?
[ ]  Have you moved the task to “in pipeline” on Jira?

[x]  Have you rebased against the current version of main?
[x]  Have you checked code coverage isn’t lower than previously?

After PRs been raised

[x]  Have you checked the PR Builder passes?

Closed by mistake


Scroll down to change navbar, scroll up to change it back.

A basic way to change the style of the top navbar as you scroll up and down.
Example
Wondering if there is a built-in way to do it in Reflex. If not, what other options do I have?
Thanks a bunch.

You can embed a JS script to do this but it's not idiomatic.

You can try to use this component: https://github.com/masenf/reflex-intersection-observer
It can trigger an event handler once a component scrolls into view

You can try to use this component: https://github.com/masenf/reflex-intersection-observer
It can trigger an event handler once a component scrolls into view

This works well. Thank you.

You can try to use this component: https://github.com/masenf/reflex-intersection-observer
It can trigger an event handler once a component scrolls into view

This works well, thank you.

You can try to use this component: https://github.com/masenf/reflex-intersection-observer
It can trigger an event handler once a component scrolls into view

This has stopped working in the new reflex update. Any idea how it can be fixed, or any alternatives? Thank you.

You can try to use this component: https://github.com/masenf/reflex-intersection-observer
It can trigger an event handler once a component scrolls into view

This has stopped working in the new reflex update. Any idea how it can be fixed, or any alternatives? Thank you.

We're going to publish 0.0.5 of reflex-intersection-observer
Can you try and see if it resolve your issue?
If it doesn't, please provide more details on your usecase.


EncodeBuilder: Correcting which type of controls should be written in a JSON string

fixes https://github.com/francoispqt/gojay/issues/145

Codecov Report

Merging #149 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #149   +/-   ##
=======================================
  Coverage   99.96%   99.96%
=======================================
  Files          38       38
  Lines        5323     5332    +9
=======================================
+ Hits         5321     5330    +9
  Misses          1        1
  Partials        1        1

Impacted Files
Coverage Δ

encode_builder.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


async run and wait

我也在想这里，很别扭
On Mon, Sep 30, 2019 at 2:04 PM zc<EMAIL_ADDRESS>wrote:

@jschwinger23 commented on this pull request.
In rpc/rpc.go
https://github.com/projecteru2/core/pull/139#discussion_r329418584:

  }

go func() {

	for m := range ch {

		log.Infof("[Async RunAndWait] %v", string(m.Data))

	}

	cancel()

	v.taskDone("RunAndWait", true)

}()

stream.Send 的逻辑似乎可以不区分, 就算 async 也 send 应该没啥问题
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/projecteru2/core/pull/139?email_source=notifications&email_token=AAD3S2HLIOME46ASJDRA5ZTQMGJFVA5CNFSM4I3WE3EKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCGI3VLY#discussion_r329418584,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD3S2BSEN7E4PMVHGHFGTDQMGJFVANCNFSM4I3WE3EA
.

--
-CMGS
A simple coder.
Love travel, sports especially outdoor sports and computer technology.
Have a dream that one day can tour around.


Desktop Docker - GPU (Helpp!)

Is your feature request related to a problem? Please describe.
GPU Support and information.
I'm currently using Windows 10, i7. CompreFace is running on Desktop Docker. This computer has a NVIDIA RTX GPU. How can i enable this instead of using the CPU? I've tried following all online docs relating to the WSL2 but when processing images, it is still using the CPU not the GPU.
Describe the solution you'd like
Possibly, better installation documentation for GPU installs with desktop docker.
Thanks!

Do you use GPU-based custom build?
you can find documentation about custom builds here:
https://github.com/exadel-inc/CompreFace/blob/master/docs/Custom-builds.md

Hi all, the before mentioned documentation is for Linux. What do I need to do in a Windows 11 system with docker? I always get the error that the Nvidia runtime is not found.
Dirk


Generate GuiseFramework API with guise

We should generate the public API for guise so that we can review it

How meta

We need to figure out what the build settings are when building the app with swift build, currently guise uses xcodebuild environment variables

Have you had time to look at this? I'd love to use this for my own tooling and I think it'd be a great addition!

If you're using Xcode it is super easy to add support, but I did not look into swift packages. I know there is a new swiftinterface file to go along with Swift5 so it might be worth looking at that if we can’t figure out a way to do it - I am pretty sure its just fiddling and finding out the right build settings that a package spits out


Fix. typos in leak_tracker

Description
This PR fixes typos in leak_tracker.
What type of PR is this?

[X] Refactor
[ ] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update

[X] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.
Most changes should add an entry to the changelog and may need to rev the pubspec package version.
Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

Thank you, @droidbg


Update cc_contract

Description
Modified cc_contract to have the same format as opt_einsum.
Todos

[x] Modified cc_contract
[x] Update the contraction of 3 tensors with the new contraction function in ccresponse.py.

Status

[ ] Ready to go

Codecov Report

Merging #39 (74925fd) into main (0a5ba5a) will increase coverage by 0.32%.
The diff coverage is 30.00%.


Sometimes the multiplayer sim still appears as teal, even when you're player 1-4

Unclear on root cause, just filing this to track.

As of v1.11.21 which contains https://github.com/microsoft/pxt/pull/9155 I have not been able to repro color issues, so appears that was it for now.


Please add a “time” format

Currently, we can chose between “base 1024 KMGTP”, “KMBT” and “numeric”. Aside from the fact that the first two are not clear to me (add a hover text with an explanation?), I’d like to be able to specify that my data is in a unit of time, e.g. milliseconds, and have the graph automatically label values in the appropriate unit, i.e. milliseconds,  seconds, minutes, …, depending on the values.
The use case is to display latency values.
Thanks for considering this :).

Over in console templates we assume everything is seconds/bytes, and have humanizing functions http://prometheus.io/docs/visualization/template_reference/#numbers (the graphs do similar). Maybe similar can be done here?

Definitely, but note that seconds really is not fine-grained enough for latency metrics: knowing whether round-trip times are below one second is a nice start, but I’d really want to know if it’s 10ms or 900ms :).

Everything is a float in prometheus, I've got things rendering as us in places via this method.

Assuming that everything is in seconds is (for better or worse) at odds with the community decision we had a while ago (weird, I can't find the thread in Google Groups anymore, although it's in my inbox) about units, and we have a lot of existing metrics which are milliseconds, microseconds, etc. So we'll need to be able to configure the scale of the unit explicitly.


Add @Deprecated configurations to the Kotlin accessors

WIP - Add more details
Fixes #10312

At this stage I'm forced to park this work. Whatever I do, IntelliJ IDEA will never see the deprecated methods:

What I discovered is that if I remove the kotlinDeprecation method call, the one which adds the kotlin.Deprecated method to the bytecode, then the method is resolved in IDEA, but is not marked as deprecated, despite the ACC_DEPRECATED access flag.
If I do add the kotlin.Deprecated annotation, then the bytecode looks exactly the same as when I try to build manually the extension, but the method becomes unresolved. I thought it was because of missing visitEnd() or ordering issues in the method calls in ASM, but using javap shows very similar bytecode (the difference is in the line numbers, which are not present in the generated bytecode).
I have spent too much time on this for 0 success. Maybe the Kotlin DSL team has an idea, but I'm running out of them.

Of course I need to mention this works perfectly fine from CLI.

I'm looking for a way to easily test the generated bytecode (so that I can prove that the deprecated configurations actually result in different bytecode, not only sources), but this can be done later.


A fatal error occured while converting a world

Chunker Version
1.2.0 windows
Input World Version
Bedrock 1.20.70
Output World Version
Java 1.21.1, I also tried Java 1.20.6
Platform / Operating System
Windows 11 64 Bit
Issue Description
Dear Chunker Team,
When I start converting my world, it convert normally until it reaches 4.28%. Then it stops for a while and later goes to 8.something%. After a while it gives me and Error: A fatal error occurred during conversion.
If this error continues please report it to us via our [GitHub issues]
Error identifier: 9a06a9a6-2b38-4977-af50-18c5e9325dc7
Could you please help me?
Best regards!
Dominik

I forgot to add the files, sorry.

Sorry for another message but the file that should be the world turned into the fabric logo here on GitHub. Here's the file as a download: https://uploadnow.io/f/gK7rxck

This seems like a duplicate of https://github.com/HiveGamesOSS/Chunker/issues/71, we'll update that issue when a fix is released.

Ok sorry I didn't see that that issue already existed. Thank you for pointing that out to me!


v2ray.com/core/proxy/vmess/outbound: failed to find an available destination

2020/11/16 23:35:28 [Warning] [3942895406] v2ray.com/core/app/proxyman/outbound: failed to process outbound traffic > v2ray.com/core/proxy/vmess/outbound: failed to find an available destination > v2ray.com/core/common/retry: [v2ray.com/core/transport/internet/websocket: failed to dial WebSocket > v2ray.com/core/transport/internet/websocket: failed to dial to (wss://us1102.tomnode.com/cat/34281): 404 Not Found > websocket: bad handshake] > v2ray.com/core/common/retry: all retry attempts failed
2020/11/16 23:35:28 [Warning] [3634160708] v2ray.com/core/app/proxyman/outbound: failed to process outbound traffic > v2ray.com/core/proxy/vmess/outbound: failed to find an available destination > v2ray.com/core/common/retry: [v2ray.com/core/transport/internet/websocket: failed to dial WebSocket > v2ray.com/core/transport/internet/websocket: failed to dial to (wss://us1102.tomnode.com/cat/34281): 404 Not Found > websocket: bad handshake] > v2ray.com/core/common/retry: all retry attempts failed
2020/11/16 23:35:28 [Info] [3942895406] v2ray.com/core/app/proxyman/inbound: connection ends > v2ray.com/core/proxy/http: connection ends > io: read/write on closed pipe
2020/11/16 23:35:28 [Info] [3634160708] v2ray.com/core/app/proxyman/inbound: connection ends > v2ray.com/core/proxy/http: connection ends > io: read/write on closed pipe
2020/11/16 23:35:28 [Info] [2271413108] v2ray.com/core/proxy/http: request to Method [CONNECT] Host [github.com:443] with URL [//github.com:443]
2020/11/16 23:35:28 [Info] [2271413108] v2ray.com/core/app/dispatcher: default route for tcp:github.com:443
2020/11/16 23:35:28 [Info] [2271413108] v2ray.com/core/transport/internet/websocket: creating connection to tcp:us1102.tomnode.com:443
2020/11/16 23:35:28 <IP_ADDRESS>:53284 accepted //github.com:443 [proxy]

wss://us1102.tomnode.com/cat/34281): 404 Not Found > websocket: bad handshake

同样配置手机可以用，只是最近这几天用不了，配置应该没有问题

你的配置是错误的，不可能工作

我是扫码的，而且这个配置一直再用，手机都可以用

刚试了下同样都配置 我用其他软件就可以 了

刚试了下同样都配置 我用其他软件就可以 了

截图上来看看你所说的其他软件同样配置可用。

刚试了下同样都配置 我用其他软件就可以 了

截图上来看看你所说的其他软件同样配置可用。

都是扫码或者复制 手机上都能用

有可能是v2rayU的问题


Fix Exception handling to be the same with older embulk-parser-csv

No problems have been observed about this so far, but trying to let its Exception handling to be the same with the original embulk-standards:
https://github.com/embulk/embulk/blob/v0.9.25/embulk-standards/src/main/java/org/embulk/standards/CsvParserPlugin.java#L263-L266

Thanks!
Hmm, a quick way could be adding gem { ... } in your local build.gradle. I now think how we can make trying a local plugin easier, but not done yet.


Wrong default values prevent basic install k8s ingress support

Bug Description
Background
We were using Istio 1.15.3 installed through the helm charts and decided to upgrade to the latest currently available (1.17.2).
We have cert-manager installed and have a certificate that needs the HTTP01 challenge mechanism which was working fine out-of-the-box with istio 1.15.3.
Upgrade fun
While upgrading, we noticed that the istio-ingress and istio-egress charts were not published anymore and seemed to be deprecated in favor of the new unified chart simply called gateway. By doing so, we also had to adjust our Gateway CRD to match the new app label (istio-gateway rather than istio-ingressgateway). Rather simple fix, so we decided to go ahead and do it.
Real Problem
Once our certificates were getting close to expiry and needed to be renewed, the HTTP01 challenge was spawned, the associated ingress was created with the proper ingress.class annotation set to istio...but nothing worked. It was simply not getting picked up by istio.
Troubleshooting
We enabled all the logs we could think of and all we could find was the gateway's envoy saying there was no route found for the incoming challenge requests which meant that the ingress wasn't taken into account. Unfortunately we were unable to find anything more pertinent...maybe there could be improvements on the logging side or the logs exist and we simply didn't know where to enable them.
Final find
After many hours of research, we finally figured out that the issue was due to the gateway's name being changed in the new merged helm chart and the default values in istio's meshConfig not being updated to reflect this.
So the fix was to reinstall istiod's helm chart by adding the following parameters: --set meshConfig.ingressService=istio-gateway --set meshConfig.ingressSelector=gateway.
After that, the challenges were almost immediately completed and the certificate was issued successfully.
Takeaway
I think this whole ordeal was due to a rename that was done half-way. But, I also believe that finding the documentation to properly use k8s ingress resources was hard. There is this page that says it should just work out-of-the-box, but that's not always the case and then finding how to properly configure istiod was not easy...I basically stumbled upon the meshConfig page by chance and realised there were ingress options there and then had to reverse engineer to figure out how to provide them through the helm chart.
Version
❯ istioctl version
client version: 1.17.2
control plane version: 1.17.2
data plane version: 1.17.2 (15 proxies)

❯ kubectl version --short
Client Version: v1.27.0
Kustomize Version: v5.0.1
Server Version: v1.25.5

Additional Information
There are a few parts that should probably be changed to reflect the new gateway name:

https://github.com/istio/istio/blob/081302afe518f2aec8ee8e67fec3fb3d02a33e04/pkg/config/constants/constants.go#L85

should be renamed to just "gateway"

https://github.com/istio/istio/blob/b437d02ed5aba9fecc3b383138afbb36374e0c4a/pilot/pkg/config/kube/ingress/conversion.go#L367

should probably use a constant rather than a hardcoded string, but otherwise should be updated to "istio-gateway"
the logic also seems flawed, because it doesn't seem to be so true that the label will match the exact name of the service where you will usually have an "istio-" prefix but not on the label...

https://github.com/istio/istio/blob/a44c42b552b07f7cfab7b9ce497b8761e4b3367d/pkg/config/mesh/mesh.go#L85

should be renamed to "istio-gateway"

https://istio.io/latest/docs/reference/config/istio.mesh.v1alpha1/

documentation should be updated to reflect the new values

Affected product area

[ ] Ambient
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[X] User Experience
[ ] Developer Infrastructure
[X] Upgrade
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Control Plane Revisions

There was no "rename". There is no name at all. When you install the chart you pick them name and can put whatever you want there. If you do helm template helloworld istio/gateway you get a helloworld service.
So yes, some legacy code has legacy baked in assumptions about a particular naming scheme - but we should not change these. Instead we can document on the page for Ingress how these configurations work and how to set them.

That is fine with me, since these hardcoded values are just assumptions and wild guesses anyway. I think the main part here is that the documentation was not enough to get this solved easily as it should have been.
Thanks for the quick response.

yep, 100% agree on the docs concern
On Fri, Apr 14, 2023, 2:31 PM Christian Artin @.***>
wrote:

That is fine with me, since these hardcoded values are just assumptions
and wild guesses anyway. I think the main part here is that the
documentation was not enough to get this solved easily as it should have
been.
Thanks for the quick response.
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/44402#issuecomment-1509299533, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXMAWTKAWQO6COJGKATXBG63NANCNFSM6AAAAAAW64CVQM
.
You are receiving this because you commented.Message ID:
@.***>


Update Jq version

Pull Request Test Coverage Report for Build 3444

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 70.111%

Totals

Change from base Build 3439:
0.03%

Covered Lines:
11140

Relevant Lines:
15889

💛  - Coveralls

@andrea11 Thanks for the PR. Please also run
./gradlew generateLock saveLock

so that the new dependencies are pulled in and applied.

@apanicker-nflx thank you for your comment, I am not used to work with gradle.
I updated the PR accordingly 😄


pull request for e2e test at 01/29/2017 12:15:11

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


long running job for firebase functions limited timeout

Firebase functions support only 9mins.
https://cloud.google.com/functions/docs/concepts/exec#timeout
Is there any way to use webhook or gcp pubsub for async long job inside gcp functions?
https://cloud.google.com/community/tutorials/cloud-functions-async

It looks like this question has been answered. I am closing this issue but feel free to reopen.


Simplify the DividerLine component

Which package is this issue related to?
ffe-core-react, ffe-core
Describe your issue (screenshots welcome!)
The DividerLine component doesn't really make much sense in its current implementation. It will render an empty
 element and then
a) Mark it with aria-hidden="true"
b) Mark it with role="presentation"
Looking at the spec there's a couple of reasons why this does not make sense.
For starters, the presentation role indicates "that an element is being used only for presentation and therefore does not have any accessibility semantics" so we're basically just grabbbing a semantic element and then stripping it of its semantics.
Second, the presentation role is ignored by browsers if "[t]he element has any of the twenty-one global ARIA states and properties", so using aria-hidden on the semantic element we wanted to strip of its semantics negates that role.
Third, why are we using aria-hidden on an empty element in the first place?
What is the expected behavior?
This should just be an empty div with the ffe-divider-line class by default and allow the consumer to override that default element with a prop if needed.

Also note that the browser default styling of the
 element is not reset by the the ffe-core CSS. It seems to be margin: 0.5em 0 across browsers but it may be diffferent in different browsers and should be set explicitly.

@42tte While we're doing a11y stuffs anyway, do you have any thoughts on this?

Hi!
I tried to look into this issue, but I'm not sure if I fully understand all the details of it. I've always though of HR as more of a visual break in the page flow, but I see now that this might not always be the case. I found some tips in the WAI Accessibility Guidelines, for example:

For graphics used as section separators (e.g., horizontal rules) provide information about what the graphic represents.  In other words, provide an equivalent textual cue as to what the visual reader perceives (Example 1).  However, we recommend using the HR element with style sheets and the "title" attribute (Example 2).

I'm not sure if changing this to a DIV would solve the actual problem of it being used the wrong way. Again - I might be missing/not understanding something here. Maybe we should enforce a title instead?
Do you have any other suggestions, either here or as code that I can look at?

I don't really know what the correct approach here is, I just thought we were doing it the wrong way. We're still going out of our way to use a semantic element and then at the same time trying to remove its semantics.
I can't recall any longer why I landed on the suggested fix, which is why I reached out for other opinions. As far as I know we've had a few accessibility audits without this being brought up as an issue so maybe it really isn't.


Bug fix: lc.periodogram did not use user-provided frequencies

Fixes #212

Codecov Report

Merging #213 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #213      +/-   ##
==========================================
+ Coverage   88.14%   88.21%   +0.06%
==========================================
  Files          24       24
  Lines        3063     3080      +17
==========================================
+ Hits         2700     2717      +17
  Misses        363      363

Impacted Files
Coverage Δ

lightkurve/tests/test_periodogram.py
100% <100%> (ø)
:arrow_up:

lightkurve/lightcurve.py
91.89% <100%> (ø)
:arrow_up:

lightkurve/periodogram.py
90.9% <100%> (+1.25%)
:arrow_up:

Continue to review full report at Codecov.

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


Please check ioBroker.google-sharedlocations with js-controller 2.0

Hi,
the new js-controller 2.0 will come into latest repository in the next days and we want to make sure that all adapters are working well. We already did a 2 weeks Beta test and so some adapter were aleady checked and some needed slight adjustments.
You can find more information in https://github.com/ioBroker/ioBroker.js-controller/issues/482 and in the ioBroker Forum. If you have more technical questions please write in the referenced issue or in the Developer thread please. General questions are best in the genral thread.
Please update your systems to js-controller 2.0 and check your adapter.
Please close this issue once you have checked your adapter or received successfull reports from users.
Thank you very much for your support. Please contact us in the other Threads or Forum on any question.

@t4qjXH8N Can't we use js-controller 2.0 for signing in with Javascript?


Misc CI bugfixes

This PR addresses 4 separate small issues:

addresses the failure of nightly builds (as seen here. I was too zealous in adding the -nightly suffix. This should fix it.
Address the failing API docs publication script (last seen failing here).
Exclude the new requirements file from the typos check.
stops dependabot from trying to update dependencies outside the .github workflows folder. (Also make the check happen a couple of hours earlier).

@coderabbitai review


Ion-checkbox wrongly showing selected in collection-repeat

As described in the forum here -
http://forum.ionicframework.com/t/ion-checkbox-wrongly-showing-selected/19914
The collection-repeat items list shows the checkbox pre-selected wrongly, when an item is deleted from the list.
here is the pen - http://codepen.io/anon/pen/yywwxN

This issue was moved to driftyco/ionic#3435


Performance fix for PyYAML comments

PyYAML has really really bad performance when adding comments, so I changed it to only add them for the first 100 entries. I also removed the l+i offset which doesn't make sense because the type is a dictionary.

This is a reapplication of https://github.com/riscv-software-src/riscv-isac/pull/89
Please could someone review it so I don't have to recreate it again?

Thanks!


Changed badges with shields.io.

PR of the day.
Can't find something for the JSON-LD badge :(

Even if it's graphically better, I'm not for relying on an external service for that.

And I fully agree with you @dunglas . BitdeliChef (or something like that for statistics) doesn't works anymore and we had to remove all these badges for instance.

Closing for now.


[bzip2] release-only build failure on Windows

Host Environment

OS: Windows 10 2019 from GitHub Actions
Compiler: Latest MSVC 14.26.xxxxx

To Reproduce
./vcpkg install bzip2:x64-windows-static-release --clean-after-build --overlay-triplets=triplets_overlay
(or any package that depends on bzip2, like qt5-base)

Contents of triplets_overlay/x64-windows-static-release.cmake:
set(VCPKG_TARGET_ARCHITECTURE x64)
set(VCPKG_CRT_LINKAGE static)
set(VCPKG_LIBRARY_LINKAGE static)

set(VCPKG_BUILD_TYPE release)

Failure logs
Starting package 2/16: bzip2:x64-windows-static-release
Building package bzip2[core]:x64-windows-static-release...
Could not locate cached archive: C:\Users\runneradmin\AppData\Local\vcpkg\archives\2c\2c926ea673e5e047cbce2782a0a8d80dcb8aaa26.zip
-- [OVERLAY] Loading triplet configuration from: D:\a\qBittorrent\qBittorrent\triplets_overlay\x64-windows-static-release.cmake
-- Downloading https://github.com/past-due/bzip2-mirror/releases/download/v1.0.6/bzip2-1.0.6.tar.gz...
-- Extracting source C:/qbt_tools/vcpkg/downloads/bzip2-1.0.6.tar.gz
-- Applying patch fix-import-export-macros.patch
-- Applying patch fix-windows-include.patch
-- Using source at C:/qbt_tools/vcpkg/buildtrees/bzip2/src/1.0.6-8838f21f8e.clean
-- Configuring x64-windows-static-release
-- Building x64-windows-static-release-rel
-- Fixing pkgconfig
-- Acquiring MSYS Packages...
-- Acquiring MSYS Packages... OK
-- Fixing pkgconfig - release
-- Checking file: C:/qbt_tools/vcpkg/packages/bzip2_x64-windows-static-release/lib/pkgconfig/bzip2.pc
-- Fixing pkgconfig - debug
-- Checking file: C:/qbt_tools/vcpkg/packages/bzip2_x64-windows-static-release/debug/lib/pkgconfig/bzip2.pc
-- CHECK_LIB_bz2d_DEBUG:CHECK_LIB_bz2d_DEBUG-NOTFOUND
CMake Error at scripts/cmake/vcpkg_fixup_pkgconfig.cmake:239 (message):
  Library "bz2d" was not found! If it is a system library use the
  SYSTEM_LIBRARIES parameter for the vcpkg_fixup_pkgconfig call! Otherwise,
  correct the *.pc file
Call Stack (most recent call first):
  scripts/cmake/vcpkg_fixup_pkgconfig.cmake:348 (vcpkg_fixup_pkgconfig_check_files)
  ports/bzip2/portfile.cmake:51 (vcpkg_fixup_pkgconfig)
  scripts/ports.cmake:79 (include)

Error: Building package bzip2:x64-windows-static-release failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: bzip2:x64-windows-static-release
  Vcpkg version: 2020.06.15-nohash

Additionally, attach any relevant sections from the log files above.

Additional context
This is different from https://github.com/microsoft/vcpkg/issues/12689.
Looks like the build process expects to always find stuff related to the debug build, even if just building release (and probably vice-versa).

closed by #12759

What about reopening and merging PR #12772 to fix this issue quickly ?
In my environment #12772 is working just fine, and a similar change was made for pcre in #12629.

Fixed by https://github.com/microsoft/vcpkg/pull/12880.


Add scss import pattern to allow importing all css/scss files from folder

Is your proposal related to a problem?

Since I'm using scss modules, I have a main scss file with some common rules that's being imported in .jsx file. In this main scss file, I need to import all scss files from a folder. They work as template options: each imported scss is one specific template and it can grow over time, increasing the number of templates. Nowadays, I need to import every single file such like this:
mainTemplate.module.scss
// must import every single template
@import "./templates/template01.scss";
@import "./templates/template02.scss";
// ... some files later ...
@import "./templates/template20.scss";

Describe the solution you'd like

I'd like to achieve the same above with the following syntax:
mainTemplate.module.scss
// one single import ...
@import "./templates/*";
// ... and it's done!

Describe alternatives you've considered

I've found this Stack Overflow question:
https://stackoverflow.com/questions/44646201/create-react-app-how-do-i-import-all-scss-files-from-a-directory/44646609#44646609
Wich refers to this npm package:
https://www.npmjs.com/package/import-glob-loader
Additional context

Nothing to add

+1

+1

+1 This will be useful!
Are you also facing something like this?
https://github.com/facebook/create-react-app/issues/7609
I haven't found a solution for modular imports in development mode.
If I import the same file twice it will duplicate the CSS inside the  tags prepended to the head section of the generated HTML.

+1

+1


Mappings for damage command

This PR add mappings for /damage command.

 I close this PR.


Add support to latest msrest package

I'm getting error when importing the VssConnection, with latest msrest package.
It seems like this change broke it.
They moved the ClientRequest class from the msrest.pipeline module to msrest.universal_http

More info:
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-1-9b33f98d18ad> in <module>()
----> 1 from vsts.vss_connection import VssConnection
      2 # from msrest.authentication import BasicAuthentication
      3 # import pprint
      4
      5 # Fill in with your personal access token and org URL

/opt/conda/lib/python3.6/site-packages/vsts/vss_connection.py in <module>()
      9 from ._file_cache import RESOURCE_CACHE as RESOURCE_FILE_CACHE
     10 from .exceptions import VstsClientRequestError
---> 11 from .location.v4_0.location_client import LocationClient
     12 from .vss_client_configuration import VssClientConfiguration
     13

/opt/conda/lib/python3.6/site-packages/vsts/location/v4_0/location_client.py in <module>()
      8
      9 from msrest import Serializer, Deserializer
---> 10 from ...vss_client import VssClient
     11 from . import models
     12

/opt/conda/lib/python3.6/site-packages/vsts/vss_client.py in <module>()
     13 from msrest import Deserializer, Serializer
     14 from msrest.exceptions import DeserializationError, SerializationError
---> 15 from msrest.pipeline import ClientRequest
     16 from msrest.service_client import ServiceClient
     17 from .exceptions import VstsAuthenticationError, VstsClientRequestError, VstsServiceError

ImportError: cannot import name 'ClientRequest'

@templarbr Thank you for reporting this issue!  For now I have released a version of the package that restricts msrest to versions >= 0.5.0 and < 0.6.0.  I will look at moving our dependency to 0.6.0 tomorrow.

This was a bug, but now its considered an enhancement to move to 0.6.0 as we addressed the immediate issue.

https://github.com/Microsoft/azure-devops-python-api/releases/tag/0.1.20


Script is loaded even though the Service is deactivated

Hello,
great work with your plugin so far! I have been enjoying working with it a lot.
However I think I have noticed a bug regarding services and script loading in the Beta Branch.
I have a Category named external which has multiple Services.
Whenever I try to disable for example the Recaptcha Service the script is still being loaded as long as any other service in this category is active.
If the entire category is disabled it works as expected.
<script src="https://www.google.com/recaptcha/api.js" type="text/plain" data-category="external" data-service="Google Recaptcha"></script>

Nice catch!

Fixed in 8f231f62c07a76fb0a183cc0f918e6b148045263.


Export UN matrix error

Desktop (please complete the following information):

OS: Windows 11 Enterprise
Pro 2.9.5

Export Matrix
Parameters
Input Network     C:\ArcGIS\XXX\XXX.gdb\XXX\xXXXXX
Matrix Workbook     C:\ArcGIS\unmatrix.xlsx
Matrix Options     ASSOCIATION_ROLES;CATEGORIES;TERMINALS;RULES
Messages
Start Time: giovedì 16 marzo 2023 16:27:12
udms 2.9.2
Traceback (most recent call last):
File "", line 18, in
File "C:\ArcGIS_resource\Utility-Data-Management-Support-Tools-2.9\Utility-Data-Management-Support-Tools-2.9\UtilityDataManagementSupport.atbx\udms\esri.py", line 274, in wrapper
File "C:\ArcGIS_resource\Utility-Data-Management-Support-Tools-2.9\Utility-Data-Management-Support-Tools-2.9\UtilityDataManagementSupport.atbx\udms_pt\common\wrappers.py", line 459, in wrapper
File "C:\ArcGIS_resource\Utility-Data-Management-Support-Tools-2.9\Utility-Data-Management-Support-Tools-2.9\UtilityDataManagementSupport.atbx\udms\logic_init_.py", line 409, in export_matrix
File "C:\ArcGIS_resource\Utility-Data-Management-Support-Tools-2.9\Utility-Data-Management-Support-Tools-2.9\UtilityDataManagementSupport.atbx\udms\logic\matrix\save.py", line 650, in main
File "C:\ArcGIS_resource\Utility-Data-Management-Support-Tools-2.9\Utility-Data-Management-Support-Tools-2.9\UtilityDataManagementSupport.atbx\udms\logic\matrix\save.py", line 392, in add_rule_sheets
File "C:\ArcGIS_resource\Utility-Data-Management-Support-Tools-2.9\Utility-Data-Management-Support-Tools-2.9\UtilityDataManagementSupport.atbx\udms\logic\matrix\save.py", line 299, in create_empty_matrix
File "C:\Users\PRCGPP78R\AppData\Local\ESRI\conda\envs\arcgispro-py3-clone-1\Lib\site-packages\pandas\core\generic.py", line 5478, in setattr
return object.setattr(self, name, value)
File "pandas_libs\properties.pyx", line 66, in pandas._libs.properties.AxisProperty.set
File "C:\Users\PRCGPP78R\AppData\Local\ESRI\conda\envs\arcgispro-py3-clone-1\Lib\site-packages\pandas\core\generic.py", line 670, in _set_axis
self._mgr.set_axis(axis, labels)
File "C:\Users\PRCGPP78R\AppData\Local\ESRI\conda\envs\arcgispro-py3-clone-1\Lib\site-packages\pandas\core\internals\managers.py", line 221, in set_axis
f"Length mismatch: Expected axis has {old_len} elements, new "
ValueError: Length mismatch: Expected axis has 70 elements, new values have 69 elements
Failed script (null)...
Failed to execute (ExportUtilityNetworkMatrix).
Failed at giovedì 16 marzo 2023 16:27:20 (Elapsed Time: 7,88 seconds)
udms 2.9.2
Traceback (most recent call last):
File "", line 18, in
File "C:\ArcGIS_resource\Utility-Data-Management-Support-Tools-2.9\Utility-Data-Management-Support-Tools-2.9\UtilityDataManagementSupport.atbx\udms\esri.py", line 274, in wrapper
File "C:\ArcGIS_resource\Utility-Data-Management-Support-Tools-2.9\Utility-Data-Management-Support-Tools-2.9\UtilityDataManagementSupport.atbx\udms_pt\common\wrappers.py", line 459, in wrapper
File "C:\ArcGIS_resource\Utility-Data-Management-Support-Tools-2.9\Utility-Data-Management-Support-Tools-2.9\UtilityDataManagementSupport.atbx\udms\logic_init_.py", line 409, in export_matrix
File "C:\ArcGIS_resource\Utility-Data-Management-Support-Tools-2.9\Utility-Data-Management-Support-Tools-2.9\UtilityDataManagementSupport.atbx\udms\logic\matrix\save.py", line 650, in main
File "C:\ArcGIS_resource\Utility-Data-Management-Support-Tools-2.9\Utility-Data-Management-Support-Tools-2.9\UtilityDataManagementSupport.atbx\udms\logic\matrix\save.py", line 392, in add_rule_sheets
File "C:\ArcGIS_resource\Utility-Data-Management-Support-Tools-2.9\Utility-Data-Management-Support-Tools-2.9\UtilityDataManagementSupport.atbx\udms\logic\matrix\save.py", line 299, in create_empty_matrix
File "C:\Users\PRCGPP78R\AppData\Local\ESRI\conda\envs\arcgispro-py3-clone-1\Lib\site-packages\pandas\core\generic.py", line 5478, in setattr
return object.setattr(self, name, value)
File "pandas_libs\properties.pyx", line 66, in pandas._libs.properties.AxisProperty.set
File "C:\Users\PRCGPP78R\AppData\Local\ESRI\conda\envs\arcgispro-py3-clone-1\Lib\site-packages\pandas\core\generic.py", line 670, in _set_axis
self._mgr.set_axis(axis, labels)
File "C:\Users\PRCGPP78R\AppData\Local\ESRI\conda\envs\arcgispro-py3-clone-1\Lib\site-packages\pandas\core\internals\managers.py", line 221, in set_axis
f"Length mismatch: Expected axis has {old_len} elements, new "
ValueError: Length mismatch: Expected axis has 70 elements, new values have 69 elements
Failed script (null)...
Failed to execute (ExportUtilityNetworkMatrix).

Can you right click your Feature Dataset and select Export XML.  Select Schema only and send that to us?

Thank you for the xml file.  I found the issue.  We never accounted for rules that are present, but do not have a matching AssetGroup/AssetType.  Here is a version of the 2.9.2 toolset that handles that issue.
UtilityDataManagementSupport.zip


GitHub org team/member management

From https://github.com/sigstore/sigstore/issues/305:
@lukehinds:

We don't have anything defined as each project has autonomy to manage its own maintainers (codeowners).
As a general guide, I myself view a maintainer as someone who regularly helps review code, finds and resolves bugs and adds features. A good candidate is someone who has a consistent presence in the project.
I hope that helps and sorry for not being more specific. Currently a lot of your contributions (of a varied type) are towards cosign, so that looks like a good trajectory towards being a maintainer.

@dlorenc:

I kind of miss having something like peribolos to manage permissions across an org, but don't really want to have to setup prow just for that. @cpanato do you know of any way to do that easier?

cc: @cpanato @naveensrinivasan

I will move this to the community repository :)


VSCode hangs when I open one C# file

VS Code version: Code 1.38.1 (b37e54c98e1a74ba89e03073e5a3761284e3ffb0, 2019-09-11T13:30:08.229Z)
OS version: Linux x64 5.3.1-arch1-1-ARCH
Steps to Reproduce:

Open this file: https://gist.github.com/exyi/a183e26ad81c7d6a2887961c8d0c8545

Does this issue occur when all extensions are disabled?: Yes. It also happens on my friends PC with a different set of extensions.

Can repro on Windows, seems related to built-in C# grammar?

I can't reproduce instead. It takes a couple of minutes on my machine but it gets parsed in the end


Update babylon.gamepads.ts

Attribute 'ongamepadconnected' is now monitored instead of 'GamepadEvent' (GamepadEvent is supported on Chrome but not 'gamepadconnected') => The gamepad API now works correctly for Chrome.
Added an optional parameter for the constructor to call a custom method when a gamepad is disconnected.
The gamepad api is now based on the gamepad unique ID (instead of the gamepad name).

Ça fait 1 mois que ce pull request n'a pas été valider, y a t'il une raisons a cela ?
It's been one month that pull request has not been validated, y Which is the reasons for this?

Hello,
Yes, I've been reviewing it but it breaks compatibility in IE11. :-( So I need more time to review it and fix it for all browsers. Still, there are some good suggestions in the code.
We're focused on v2 features, so the gamepad will be probably fixed once our v2 features will be finished.
Bye,
David

Does not work at all for several gamepad... PR closed.


Remove more autotools files

Problem
Only GN is the current make system in CHIP. There are still some old autotools files in the source tree.
Summary of Changes
Remove more old autotools files and references in README.md
Readme.md may need updates, however for now this PR only does removal.

@saurabhst @hawk248 @BroderickCarlin @jelderton @chrisdecenzo


[IOS] Delete workspace crash

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

I get this crash in the IOS simulator, the crash seems to come from here:
https://github.com/Expensify/react-native-onyx/blob/91aabb7219ddc8722cc2955de0398b1ae70c546d/lib/Onyx.js#L1089
When we get an Onyx update setting a deleted workspace to null.
From the AsyncStorage.multiMerge documentation, it seems like for a pair of key / value, it is expected that the value is a string (stringified JSON).
I'm wondering if we should be calling JSON.stringify on the value if the value is null (JSON.stringify(null) === 'null'). Alternatively, maybe if the value is null we should be clearing the keys from the storage using Storage.multiClear.
Action Performed:

Login in IOS
Create a workspace
Delete the workspace

Expected Result:
Workspace get deleted, no error
Actual Result:

Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platform:

Where is this issue occurring?

iOS

Version Number:
Reproducible in staging?:
Reproducible in production?:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
View all open jobs on GitHub

Thanks to @aimane-chnaif I have a good idea on how to fix this in the backend. Taking it internally.

PR up and ready for review: https://github.com/Expensify/Web-Expensify/pull/35386

@aldo-expensify This issue just reopened. Do you think it is related to this issue? It was previously closed believing it was related to https://github.com/Expensify/App/issues/10975

@sobitneupane it does look very similar the error and the PR fixing this has not been deployed yet. I'll try to see if I can find more info on the crash to see if it the same.

I looked at crashlytics, but there is no extra information on the data that may have caused the crash. I would say, lets wait for that PR to be deployed.
The crash happened 4 times today, but it was the same device every time, so it is not something common/widespread.

The PR fixing this was deployed a while ago, not sure why this remained open. Closing!


fix: throw io exception instead of storage exception

Fixes #219

@elharo could I get your review on this PR.
I want another perspective.


chore(deps): upgrade cdklabs-projen-project-types

Upgrades cdklabs-projen-project-types dependency. See details in workflow run.

Automatically created by projen via the "upgrade-cdklabs-projen-project-types-main" workflow

@Mergifyio update


Show picker closer to status bar when invoked from there

Investigate to open the picker at the status bar and not in the top center when clicking into the status bar. In that model, the input box should show to the bottom and quick open grow to the top.

Please fix this. Pretty please.


Deployment Error: TASK [download : prep_kubeadm_images | Copy kubeadm binary from download dir to system path

Error:
TASK [download : prep_kubeadm_images | Copy kubeadm binary from download dir to system path] ***
Thursday 02 January 2020  22:01:04 +0530 (0:00:00.778)       0:09:40.886 ******
fatal: [k8s-kubespray-master-0 -> <IP_ADDRESS>]: FAILED! => {"changed": false, "cmd": "sshpass", "msg": "[Errno 2] No such file or directory: b'sshpass': b'sshpass'", "rc": 2}
Details:
Template Using : Ubuntu 18.04
For SSH login using user name and password

Hi,
i've seen this issue in my tests. I've resolved changing "synchronize" with "copy":
  copy:
    src: "{{ local_release_dir }}/kubeadm-{{ kubeadm_version }}-{{ image_arch }}"
    dest: "{{ bin_dir }}/kubeadm"
    remote_src: yes
    mode: preserve

this issue is present in all "Copy ... binary from download dir"

I had the same issue, didn't use the ssh key setup for the template.
I fixed it by installing sshpass in the template VM

I fixed it by adding interpreter_python=python3 in the [defaults] section of the ansible/kubespray/ansible.cfg file. It seems to be an issue with python 2, which was used by default.

The fix for me was to install sshpass on the master node, which also required epel-release:
yum install epel-release
yum install sshpass

Maybe those two steps could be added. In my case my controller was ubuntu and my cluster was on CentOS. Anyway, it was looking for sshpass on the master node.

The fix for me was to install sshpass on the master node, which also required epel-release:
yum install epel-release
yum install sshpass

Maybe those two steps could be added. In my case my controller was ubuntu and my cluster was on CentOS. Anyway, it was looking for sshpass on the master node.

I have the same problem. I followed the recommendation of fastlorenzo but I maintain the problem. Could gelmistefano give me details of path for that code? Which is the name file?


ent: initial support for edge schemas

Edge Schema
The Through option allows setting intermediate entity schemas for edges of type M2M. We name such schemas "edge schemas".
Using this option, users can expose relationships in their API, store additional fields on their edges/relationships,
apply CRUD operations, and set hooks and privacy policy on them.
User Friendships Example
In the following example, we demonstrate how to model the friendship between two users using edge schema with the two
required fields of the relationship (user_id and friend_id) and an additional field named created_at that its value
is automatically set on creation.

// User holds the schema definition for the User entity.
type User struct {
	ent.Schema
}

// Fields of the User.
func (User) Fields() []ent.Field {
	return []ent.Field{
		field.String("name").
			Default("Unknown"),
	}
}

// Edges of the User.
func (User) Edges() []ent.Edge {
	return []ent.Edge{
		edge.To("friends", User.Type).
			Through("friendships", Friendship.Type),
	}
}

// Friendship holds the edge schema definition of the Friendship relationship.
type Friendship struct {
	ent.Schema
}

// Fields of the Friendship.
func (Friendship) Fields() []ent.Field {
	return []ent.Field{
		field.Time("created_at").
			Default(time.Now),
		field.Int("user_id"),
		field.Int("friend_id"),
	}
}

// Edges of the Friendship.
func (Friendship) Edges() []ent.Edge {
	return []ent.Edge{
		edge.To("user", User.Type).
			Required().
			Unique().
			Field("user_id"),
		edge.To("friend", User.Type).
			Required().
			Unique().
			Field("friend_id"),
	}
}

Notes

Similar to entity schemas, the ID field is automatically generated for edge schemas if not stated otherwise.
Edge schema cannot be used by more than one relationship (an edge and its back-reference).
The user_id and friend_id edge-fields are required in the edge schema as they are composing the relationship.:::

User Likes Example
In the following example, we demonstrate how to model a system where users can "like" tweets, and the timestamp of when
exactly the "like" has happened is stored in the database. i.e. store additional fields on the edge.

// User holds the schema definition for the User entity.
type User struct {
	ent.Schema
}

// Fields of the User.
func (User) Fields() []ent.Field {
	return []ent.Field{
		field.String("name").
			Default("Unknown"),
	}
}

// Edges of the User.
func (User) Edges() []ent.Edge {
	return []ent.Edge{
		edge.To("liked_tweets", Tweet.Type).
			Through("likes", Like.Type),
	}
}

// Tweet holds the schema definition for the Tweet entity.
type Tweet struct {
	ent.Schema
}

// Fields of the Tweet.
func (Tweet) Fields() []ent.Field {
	return []ent.Field{
		field.Text("text"),
	}
}

// Edges of the Tweet.
func (Tweet) Edges() []ent.Edge {
	return []ent.Edge{
		edge.From("liked_users", User.Type).
			Ref("liked_tweets").
			Through("likes", Like.Type),
	}
}

// Like holds the edge schema definition for the Like edge.
type Like struct {
	ent.Schema
}

func (Like) Annotations() []schema.Annotation {
	return []schema.Annotation{
		field.ID("user_id", "tweet_id"),
	}
}

// Fields of the Like.
func (Like) Fields() []ent.Field {
	return []ent.Field{
		field.Time("liked_at").
			Default(time.Now),
		field.Int("user_id"),
		field.Int("tweet_id"),
	}
}

// Edges of the Like.
func (Like) Edges() []ent.Edge {
	return []ent.Edge{
		edge.To("user", User.Type).
			Unique().
			Required().
			Field("user_id"),
		edge.To("tweet", Tweet.Type).
			Unique().
			Required().
			Field("tweet_id"),
	}
}

Notes
Since the field.ID annotation is used for configuring a composite identifier (i.e. primary key) for the edge schema,
the ID field will not be generated along with any of its builder methods. e.g. Get, OnlyID, etc.

Thanks for the feedback @fgm. For now, I decided to go with the "explicit way" as mentioned in https://github.com/ent/ent/issues/1949#issuecomment-1137194333. Let's give this feature some time, get some feedback on it, and may improve it in the near future.
Appreciate your help in https://github.com/ent/ent/issues/1949 🙏


[BUG] - Plutus fees unacceptably high for simple token swap

Internal/External
External   otherwise.
Area
Plutus    Related to Plutus Scripts (Alonzo).
Summary
ASTOR has started to pay out tokens for the portion of pool rewards the go back to loyal delegators. A plutus smart contract locks those funds and tokens can be swapped 1:1 against ADA. The process is described here
Because users can currently not be expected to invoke the SC directly, there is a proxy service that monitors a specific address for token input and then does the SC invocation.
As you can see from the individual swap transactions the fees are > 1.2 ADA per swap.
The swap Tx looks like this ...
[01-11-2021 10:49:11] INFO CallerAddr: 67ebac12500d32d9b896bf98470608f6d3832e95223b793517b918c3cabc567d => addr1q90z75a052tvzjhl6ft53lslkj3davtjpv85sx8qe0wwec4f3swpn0mypqccdxy7q7vfusf42cmlgjxlftwyfgv7ed0sa5rtzg
[01-11-2021 10:49:11] INFO Proxy Swap 57 Astor296 from Percy to addr1q90z75a052tvzjhl6ft53lslkj3davtjpv85sx8qe0wwec4f3swpn0mypqccdxy7q7vfusf42cmlgjxlftwyfgv7ed0sa5rtzg =======================================
[01-11-2021 10:49:11] INFO Swap 57 Astor296 from Percy to addr1q90z75a052tvzjhl6ft53lslkj3davtjpv85sx8qe0wwec4f3swpn0mypqccdxy7q7vfusf42cmlgjxlftwyfgv7ed0sa5rtzg
[01-11-2021 10:49:11] INFO Query UTxO for Percy
  67ebac12500d32d9b896bf98470608f6d3832e95223b793517b918c3cabc567d#0 value {lovelace 1444443 3f997b68b1f491c7c2f10af4e2bf9566c5d25bd61df0343065d4fe1c.Astor296 57} datum TxOutDatumHashNone
  14d4ec2c52a66b09e0e00f0d547f3bd8b0e27ab7b0262c0da2084f1d871ea3fb#0 value {lovelace 10094283} datum TxOutDatumHashNone
  9ce0550e8636c9c6fb6af5f89f15e65a88633d52386bf36c5608177abf12edcd#0 value {lovelace 10094283} datum TxOutDatumHashNone
[01-11-2021 10:49:12] INFO Fees: 14d4ec2c52a66b09e0e00f0d547f3bd8b0e27ab7b0262c0da2084f1d871ea3fb#0 lovelace 10094283
[01-11-2021 10:49:12] INFO Collateral: 9ce0550e8636c9c6fb6af5f89f15e65a88633d52386bf36c5608177abf12edcd#0 lovelace 10094283
[01-11-2021 10:49:12] INFO ScriptDatum: Astor296 => {"constructor":0,"fields":[{"bytes":"4173746f72323936"},{"int":1641851091000}]}
[01-11-2021 10:49:12] INFO Invalid after time:<PHONE_NUMBER> => 10-01-2022 21:44:51 UTC
[01-11-2021 10:49:12] INFO Query UTxO for Script
  14d4ec2c52a66b09e0e00f0d547f3bd8b0e27ab7b0262c0da2084f1d871ea3fb#1 value {lovelace 262000000 3f997b68b1f491c7c2f10af4e2bf9566c5d25bd61df0343065d4fe1c.Astor296 710} datum d8a95d4a0b1cd3185fc571eb4ad1c293cdee44fbdf45400616058662b817b827
[01-11-2021 10:49:14] INFO InvalidAfter: 44197763 => 01-11-2021 10:54:14 UTC
[01-11-2021 10:49:14] INFO Build token swap transaction
[01-11-2021 10:49:14] INFO cardano-cli transaction build
 --mainnet
 --alonzo-era
 --tx-in 14d4ec2c52a66b09e0e00f0d547f3bd8b0e27ab7b0262c0da2084f1d871ea3fb#0
 --tx-in 67ebac12500d32d9b896bf98470608f6d3832e95223b793517b918c3cabc567d#0
 --tx-in 14d4ec2c52a66b09e0e00f0d547f3bd8b0e27ab7b0262c0da2084f1d871ea3fb#1
 --tx-in-script-file /var/cardano/local/mainnet/scripts/swaptokens.plutus
 --tx-in-datum-file /var/cardano/local/scratch/script-datum296.json
 --tx-in-redeemer-value 0
 --tx-in-collateral 9ce0550e8636c9c6fb6af5f89f15e65a88633d52386bf36c5608177abf12edcd#0
 --tx-out addr1w9adgfaux7vjj948u5vfmszx6v3ylc9typjjrkke4n6alhcwu2j5z+205000000+767 3f997b68b1f491c7c2f10af4e2bf9566c5d25bd61df0343065d4fe1c.Astor296
 --tx-out-datum-hash d8a95d4a0b1cd3185fc571eb4ad1c293cdee44fbdf45400616058662b817b827
 --tx-out addr1q90z75a052tvzjhl6ft53lslkj3davtjpv85sx8qe0wwec4f3swpn0mypqccdxy7q7vfusf42cmlgjxlftwyfgv7ed0sa5rtzg+57244443
 --change-address addr1vxq39a39juaxn8ydwcejllg5t8ly5dfqqlv9r04lfftnqeqeq8xna
 --invalid-hereafter 44197763
 --protocol-params-file /var/cardano/local/scratch/protocol.json
 --out-file /var/cardano/local/scratch/tx.raw
[01-11-2021 10:49:14] INFO Sign transaction
[01-11-2021 10:49:14] INFO Submit transaction

and the Plutus contract is here
Expected behavior
A simple SC invocation like this should be affordable at large scale. Perhaps I'm doing something wrong with how I construct the Tx (e.g. sending the compiled plutus code every time)?

Are you on your own local testnet? Have you tried submitting a lower fee with build-raw?

This is mainnet. I wouldn't want to get into the business of cpu/ram units when the very purpose of transaction build is to remove the complexity of fee handling. i.e. this is supposed to produce the best possible result

this is supposed to produce the best possible result

Actually, it can be slightly more expensive than build-raw due to how the transaction fee is estimated.
I would need to implement a cli command that returns the cost of running a script to see how much it is actually contributing to the fee. If it's substantial then you will need to take this up with the Plutus team as it's not a cli issue.

I'm closing this. This issue belongs in the plutus repo.


Code sample has a typo - AdditonalOptions instead of AdditionalOptions

One of the code samples has a typo where the end tag for 'AdditionalOptions' is misspelled as 'AdditonalOptions'.

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

ID: 1d408578-140a-fdbe-f385-8309c3f7a565
Version Independent ID: cda01d6c-e26e-e753-79e0-fd2c46cb0275
Content: Converting a WDK sources file to a Visual Studio project - Windows drivers
Content Source: windows-driver-docs-pr/devtest/converting-a-wdk-sources-file-to-a-visual-studio-project.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @EliotSeattle
Microsoft Alias: eliotgra

Thanks for catching that, @WilliamsJason
I made the change and pushed to our main branch. Should be live shortly.
Cheers,
Eliot


Possible to install via go cli?

Is it possible to install via go install? Can't really think of why downloading a prebuilt package while go is really fast enough to build on every machine.

Its built with sql migrations & public files (html, css, js) all bundled into the binary. It isn't done with go install unless it was checked into git.
Its using go-bindata to package it

It'll need some restructuring to be able to do installs without pre-packaging..

You can install via go install but you have to install the cli package


Parallelize RandDistribution.Histogram()

Large number of samples while computing RandDistribution.Histogram(), especially with an expensive Rand() and/or xform, can benefit from batching parallel execution.
As a heuristic, use samples / (2*runtime.NumCPU()) as the batch size, capping by the interval [10..1000]. Use 2*runtime.NumCPU() as the number of workers.

Also, in the same way parallelize RandDistribution.MAD().


bpf: nodeport: only set outer src IP for tunnel encap in XDP

As part of introducing native tunnel support for XDP, we started passing a src_ip parameter to __encap_with_nodeid(). This was needed to manually build the packet's outer headers.
But for TC context we actually don't want to specify the outer src IP, and defer its selection to the kernel stack instead. Worse, specifying the outer src IP means that the skb implementation for ctx_set_encap_info() starts to use the local_ipv4 field in the bpf_tunnel_key. And that's not supported on older kernels, resulting in drops.
Fix this by only setting the src_ip parameter in XDP context.
Reported-by: Yusuke Suzuki<EMAIL_ADDRESS>Fixes: 43dffb298173 ("bpf: encap: manually set src IP and port from nodeport XDP paths")

/test


Return consistent promise object for repeated calls with the same key

Overview
While using memoize in our code we identified an issue where the order of promise execution got scrambled by the use of memoize.
This PR fixes this issue.
Root cause
The first call to memoize with a certain key returns one promise, all subsequent calls to memoize with the same key return a different promise.
This is because the first call creates a new promise with the .catch call and returns it, but caches the original "inner" promise.
Following calls to memoize return only the "inner" promise.
Suggested fix
In the case of having a promise object as the result of the evaluated function, cache the complex promise with the catch call in it.
This way the first and subsequent calls will return the same promise and all will register their events on it.
Details:
We are using memoize to memoize the suggested autocomplete entries for a text input in our web page, they all have the same cache key.
When the user is typing a string into the input, each letter invokes the onchange event that uses memoize and awaits the promise that fetches data from the server.
What we see is that when the promise returned by the memoized function is resolved, the first caller's then method is being invoked last (instead of first as expected).
Repro steps:

Call memoize with a promise returning function with key 'a' and await it.
While it's executing have one (or more) calls to the same key and await it.

Expected behavior - the then statement from item 1 is invoked before the then statement of item 2.
Note: also see the unit test in this PR to illustrate this scenario. This test is failing in the current master branch.

Thanks @anleac for helping out here!

Thanks @anleac for helping out here!

@keithamus should I bump the package version as well?

@keithamus should I bump the package version as well?

@keithamus should I bump the package version as well?

Normally we do that as part of the publishing process after merging the PR.

@keithamus should I bump the package version as well?

Normally we do that as part of the publishing process after merging the PR.

I've simplified the test cases to be more explicit about the behaviours we expect:

That memoized calls return the same Promise each call.
That the memoized Promise can be caught without leaking errors.

I think this is good to go now :wink: :smile:

I've simplified the test cases to be more explicit about the behaviours we expect:

That memoized calls return the same Promise each call.
That the memoized Promise can be caught without leaking errors.

I think this is good to go now :wink: :smile:


[PYTORCH]celu, gelu, selu activations

@masahi please help to review activations of pytorch. Thanks in advance.
-celu
-gelu
-selu
-some datatype conversion issues also fixed.
-testcase coverage added

Thanks @siju-samuel