diesel setup doesn't work in powershell 7

Setup
Versions

Operating System: windows 10

Problem Description
If i run diesel setup in powershell 7 nothing happens, but same works fine in command promt. What can be an issue?
Checklist

[v] I have already looked over the issue tracker for similar issues.
[v] This issue can be reproduced on Rust's stable channel. (Your issue will be
closed if this is not the case)

Duplicate of #2015.
Please try to search the issue tracker first for duplicated issues before you open a new issue. The check mark is there for an reason.


DDPG fixes

[ ] make DDPG work for cartpole first, for sanity check
try dis https://github.com/songrotek/DDPG/blob/master/actor_network.py
and try dis https://github.com/pemami4911/deep-rl/blob/master/ddpg/ddpg.py
[ ] batch normalization
[ ] make DDPG work for pendulum
[ ] schedule DDPG for all the experiments
[ ] check random seeds

I think it's really close to working but something is off with the gradient update. The DDPG implementation from permami works, and ddpg_tf implementation results in exactly the same actions up to the first gradient update. I also did some testing around the weights init - the algorithm is not that sensitive to them and will work for different initializations.

working DDPG implemented in #118


Add SIG Docs Spotlight blog article

Spotlight blog series - SIG Docs edition

/retitle Add SIG Docs Spotlight blog article

cc @sftim

@sftim the changes are in and chose July 29th as the DOP

@PurneswarPrasad after updating, please copy the article text into a new k/website PR that is similar to https://github.com/kubernetes/website/pull/35084

/hold cancel

I'm happy with this.
/lgtm
/approve

/hold
Needs approval. Update the publication date if approving after 2022-07-29.
Once this is approved and the publication date is right, it's OK to unhold this. And then a similar check-date-and-unhold for https://github.com/kubernetes/website/pull/35084

/lgtm

/approve
/lgtm

https://github.com/kubernetes/website/pull/35530 is good to go.

/hold cancel


(FM-7442) Convert network_interface to agentless

Codecov Report

Merging #15 into puppet6 will increase coverage by 0.15%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           puppet6     #15      +/-   ##
==========================================
+ Coverage     0.73%   0.89%   +0.15%
==========================================
  Files          173     174       +1
  Lines        21797   21832      +35
==========================================
+ Hits           161     196      +35
  Misses       21636   21636

Impacted Files
Coverage Δ

...b/puppet/provider/network_interface/cisco_nexus.rb
100% <100%> (ø)

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 98ef7ec...ccc98c4. Read the comment docs.


I would like to request for CodeLens integration for better code navigating

Currently the only code lens support seems to be for the status of tests. I would like to see code lens added to code in general that assist with code navigation by showing references and implementation of functions/classes.
EDIT: Also is not possible to vote on closed requests, so I'm opening it again to take a try ...

This is a duplicate of https://github.com/microsoft/pylance-release/discussions/4740. We use discussion items in the Enhancement discussion category to track things that are not currently on our backlog but could be prioritized if there was enough interest from users. Please up-vote the existing discussion item. Thanks!


Admin can upload a theme directly from his computer to the site

Do you have an idea of the zip file structure ?

wwwroot/themes/name_of_theme
themes/name_of_themes


I can't selectable current month

Hello. I can't selectable current month. To be able to select the current month, I need to set the largest month as the next month or the following months. This is an unreasonable method.
I recorded a video for this question. You can understand the problem better after watching the video.
https://user-images.githubusercontent.com/100342868/172051136-ad6c88a3-4643-43bd-9e83-c43afee3fc9e.mp4

Do you have any errors in the console?

Do you have any errors in the console?

No,there are no error messages on the console screen.

@harundogan01 Checked it, it happens because internally dp is working with the entire date object, meaning, that when you provide '2022/06' it is parsed to June 1, 2022 which is < today. To be able to select, add a day larger than today for the current month.
I will see to add a separate check when the month picker is used for validation in the next version.

@harundogan01 Checked it, it happens because internally dp is working with the entire date object, meaning, that when you provide '2022/06' it is parsed to June 1, 2022 which is < today. To be able to select, add a day larger than today for the current month.
I will see to add a separate check when the month picker is used for validation in the next version.

Thank you so much. I will try this solution. I will let you know.

@Jasenkoo  I finded a bug. The method you mentioned in the date selection does not work. I show the reason with the video below.
https://user-images.githubusercontent.com/100342868/172109730-8a276b59-864a-4db0-8430-d88619b8a8c6.mp4
In short, the largest day determined in the current system must be the current day or one day greater than it. This seemed like a problem to me. For example, if I set the biggest day as June 20, when we come to June 21, I will not be able to choose June again. What is your comment on the subject?

There is no workaround for this atm. You can always set the max date as the last day in a month. Since you are using the monthPicker, it doesn't really matter if it is the 1st or 30th as a date.

There is no workaround for this atm. You can always set the max date as the last day in a month. Since you are using the monthPicker, it doesn't really matter if it is the 1st or 30th as a date.

Good news,
I dynamically took the last day of the month and added it to the max parameter. It's working flawlessly now. I wish you a good day.


Claim count status is wrong

The result of the status call nr of claims is not the same thing as we are grabbing. In reality the logic is grabbing the number of times a claim was synced or processed since the last restart of elastic search.
The name of the field should be changed to reflect the logic or the logic should be changed to reflect the name. Preferably the latter.

The status response has been completely rewritten to provide valuable information


✨ Multiple cameras at once

What feature or enhancement are you suggesting?
I think Dual camera support at once should be in react-native-vision-camera ,
Scenario:

Half-screen allocate front camera view
second half screen: allocate back camera view
both should be working at a time

Note: Currently, the multi-camera feature provided by react-native-vision-camera can not fulfill this feature
What Platforms whould this feature/enhancement affect?
Android
Alternatives/Workarounds
I am unable to find this feature in React Native Right now and I need to implement this in next 10 days
Additional information

[X] I agree to follow this project's Code of Conduct
[X] I searched for similar feature requests in this repository and found none.

here is the feature already present in a flutter plugin "multiple cameras at a once"
but React Native still not have...


PennCNV HMM file

Do we need to create an HMM file?
See PennCNV documentation to figure this out, and if so put code to generate this in "scripts" dir.

Looks like the HMM file is bundled with PennCNV. We just need to provide the signal intensity files in proper format.
See first paragraph here

@mtbrown22, I think you are right about the HMM file.  That first paragraph is confusing, though.  It also says the PFB and GC Model file are part of the bundle.  Though there are example files for those, the PFB is specific to the array and the GC Model file is specific for the genome build.  So those two really need to be generated.  I think the HMM will work, though.


Integration with CircleCI

Implement support for CircleCI as an easy to setup automation alternative to Jenkins for running the process automation.

See https://github.com/SalesforceFoundation/CumulusCI-Test/blob/master/circle.yml for a demo setup that runs on CircleCI.
See https://github.com/SalesforceFoundation/mrbelvedereci for the new CI system build to run CumulusCI 2 flows on Heroku as another alternative.


Add example systemd service file

We encourage users to use this file as a basis for systemd service deployments as it contains some basic best-practices to get started.
Related: #68

I just copied the configuration I currently use on my servers which I know to be working. Thanks for the hints, I'll try adding more sandboxing to see if it works, if yes, I will update the file in this repository.


Friendly error messages in example app

Description
When running the example app greet-and-react with missing environment variables, users may see quite a nasty error like this:
TypeError: SlackStrategy requires a clientSecret option
10:43 AM
    at new SlackStrategy (/rbd/pnpm-volume/ac2b9b2a-b6f4-455c-ba84-a33b4efcbe69/node_modules/.registry.npmjs.org/@aoberoi/passport-slack/1.0.5/node_modules/@aoberoi/passport-slack/dist/strategy.js:131:13)
10:43 AM
Jump to
at Object.<anonymous> (/app/src/index.js:37:14)
10:43 AM
    at Module._compile (internal/modules/cjs/loader.js:688:30)
10:43 AM
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:699:10)
10:43 AM
    at Module.load (internal/modules/cjs/loader.js:598:32)
10:43 AM
    at tryModuleLoad (internal/modules/cjs/loader.js:537:12)
10:43 AM
    at Function.Module._load (internal/modules/cjs/loader.js:529:3)
10:43 AM
    at Function.Module.runMain (internal/modules/cjs/loader.js:741:12)
10:43 AM
    at startup (internal/bootstrap/node.js:285:19)
10:43 AM
    at bootstrapNodeJSCore (internal/bootstrap/node.js:739:3)

This is not very helpful to a new user who is just trying to get things running and hasn't yet followed the next step about setting environment variables. Instead, we should catch the errors from the SlackEventAdapter constructor and the SlackStrategy constructor and output a more friendly error message. The following seems a little more pleasant:
Could not start up the application. Have you set your environment variables correctly yet? See  ...
Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

https://github.com/slackapi/node-slack-sdk/issues/818


Fixed some path issues

This fixes some bugs where a group eg have a location set to absolute or relative to project , where the resulting path for the files inside of it would be wrong.
Related to https://github.com/mac-cain13/R.swift/issues/191

Thanks for this fix!
This is merged to develop and is part of https://github.com/mac-cain13/R.swift/pull/194


Default values from properties are not being rendered into UI.

I am not be able to see the defaul value for my requestbody fields.
I know this issue was fixed. I tried several version but no luck
json. path
"/xzxx/xx/xxxx" : {
"post" : {
"tags" : [ "xxxxxxx ],
"summary" : "delete property",
"description" : "PASS",
"requestBody" : {
"content" : {
"application/x-www-form-urlencoded" : {
"schema" : {
"type" : "object",
"properties" : {
"Id" : {
"type" : "integer",
"default" : 5190
},
"UserId" : {
"type" : "integer",
"default" : 122
}
}
}
}
},
"required" : true
},
"responses" : {
"200" : {
"description" : "null"
}
}
}
},
html file

output.

I think you discovered a bug. I dont think we dealt with fixing form-urlencoded request body defaults.
along with default, example also is not respected for form-urlencoded request body
I have added this to our internal issue list. thanks for bringing it up


Implement a User Instructions Menu for Each Fragment

Each fragment should include a menu option that provides users with instructions relevant to the current fragment's functionality. This will enhance the app's usability and assist new users in navigating the app more effectively.

Resolved


Self Apply

Updates toolkit and applies the new coding standard and license headers.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.491%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
55

Relevant Lines:
57

💛  - Coveralls


BAU: Track VC received for TICF CRI

Proposed changes
What changed
Track VC received for TICF CRI
Why did it change
When calling the TICF CRI, what we're actually interested in sending is all the VCs received so far this session. We've modelled this as a reuse/other journey so far, with a reuse journey not sending any VCs, and anything else sending all of them.
Due to how the journey map works and how many places we need to update to integrate the TICF CRI into all journeys it's very fiddly to get this right and not make a mistake.
This is an attempt to alleviate having to think about that. This is a speculative, untested, not even deployed stab at tracking the VCs received in the users session, and then using that to inform which VCs are sent to TICF.

@Wynndow as discussed, this seems like a reasonable approach to me - I'll raise a new task to track this properly.


Request for nonexistent project setting error when opening Godot

Godot version
3.5.beta.custom_build.afadd3d12
System information
Ubuntu 21.10 - Nvidia GTX 970, Gnome shell 40.4 X11
Issue description
When running Godot, then this error is shown
ERROR: Request for nonexistent project setting: logging/file_logging/enable_file_logging.
   at: set_initial_value (core/project_settings.cpp:122)
ERROR: Request for nonexistent project setting: logging/file_logging/enable_file_logging.
   at: set_builtin_order (core/project_settings.cpp:678)
ERROR: Request for nonexistent project setting: logging/file_logging/enable_file_logging.
   at: set_restart_if_changed (core/project_settings.cpp:126)
ERROR: Request for nonexistent project setting: logging/file_logging/enable_file_logging.
   at: set_ignore_value_in_docs (core/project_settings.cpp:136)

it is visible in CI - https://github.com/godotengine/godot/runs/5247414724?check_suite_focus=true
Steps to reproduce
Just run editor/project
Minimal reproduction project
No response

Probably a regression from https://github.com/godotengine/godot/commit/b343660dac50c753d227d296358fe1aa40da9c90

Happens in 3.4.3 (stable) aswell.

CC @BimDav

I don't get the error, but I can see a problem coming from https://github.com/godotengine/godot/commit/b343660dac50c753d227d296358fe1aa40da9c90.
There is kind of an asymetry in has_setting: if it is a test to check if it is legal to do get_setting, then the added checks from https://github.com/godotengine/godot/commit/b343660dac50c753d227d296358fe1aa40da9c90 seem necessary. But it is also used as a check in
	if (!ProjectSettings::get_singleton()->has_setting(p_var)) {
		ProjectSettings::get_singleton()->set(p_var, p_default);
	}
	ret = ProjectSettings::get_singleton()->get(p_var);

	ProjectSettings::get_singleton()->set_initial_value(p_var, p_default);
	ProjectSettings::get_singleton()->set_builtin_order(p_var);
	ProjectSettings::get_singleton()->set_restart_if_changed(p_var, p_restart_if_changed);
	ProjectSettings::get_singleton()->set_ignore_value_in_docs(p_var, p_ignore_value_in_docs);
	```

In that case we want the restrictive version of `has_setting`, so maybe there should be both versions in the API?

I think it can anyway be reversed for now as the case it fixes is rare...

Steps to reproduce
Just run any project

That's not sufficient, which is why some of us couldn't reproduce the issue. #58672 has more details, it happens only when a feature override's default value has been changed (e.g. by disabling file logging on PC).


podman still fails after adding subuid/subgid ranges

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description

Hello, I want to ask how to recover from the following state:
I have large UID, so useradd didn't create subuid and subgid ranges. I ran podman a few times and it failed. So, I discovered the subuid/subgid ranges missing and added them manually, but podman still doesn't work.
I managed to recover in VM by deleting all config and some other stuff and reinstalling podman, but I don't remember how and I possibly broke the system by deleting some other things, so I don't want to do it on my own system.
Is there a good, deterministic way of recovering from this? Thank you!
Btw, ideal solution would be to make podman to recognize newly added subuid/subgid ranges are recover automatically, or at least advise user what to do. For this reason, I am marking this issue as a bug.
Additional note: if I do the steps below without steps 4. and 5., it all works ok.
Steps to reproduce the issue:

In "/etc/login.defs", UID_MAX is set to 60000
# useradd -u 100000 user1
# su - user1
# su -c "podman run --rm -it registry.fedoraproject.org/fedora-minimal" - user1
# su -c "podman run --rm -it registry.fedoraproject.org/fedora-minimal" - user1  # (need to run two times for some reason)
# usermod -v<PHONE_NUMBER>36 -w<PHONE_NUMBER>36 user1
# su -c "podman run --log-level=warning --rm -it registry.fedoraproject.org/fedora-minimal" - user1

Describe the results you received:
WARN[0000] the current user namespace doesn't match the configuration in /etc/subuid or /etc/subgid
WARN[0000] you can use `podman system migrate` to recreate the user namespace and restart the containers
WARN[0000] Error initializing configured OCI runtime crun: no valid executable found for OCI runtime crun: invalid argument
INFO[0000] running as rootless
WARN[0000] Error initializing configured OCI runtime crun: no valid executable found for OCI runtime crun: invalid argument
Trying to pull registry.fedoraproject.org/fedora-minimal...
Getting image source signatures
Copying blob eab6ae2ae0c1 done
Copying config 7bb5a60e8a done
Writing manifest to image destination
Storing signatures
  Error processing tar file(exit status 1): there might not be enough IDs available in the namespace (requested 0:5 for /usr/bin/write): lchown /usr/bin/write: invalid argument
Error: unable to pull registry.fedoraproject.org/fedora-minimal: unable to pull image: Error committing the finished image: error adding layer with blob "sha256:eab6ae2ae0c1b29c62c08b8ebefd7d690e794d426c6db86ce887fab367c395ed": Error processing tar file(exit status 1): there might not be enough IDs available in the namespace (requested 0:5 for /usr/bin/write): lchown /usr/bin/write: invalid argument

Describe the results you expected:
No error, it should run the container.
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
Version:            1.6.2
RemoteAPI Version:  1
Go Version:         go1.12.10
OS/Arch:            linux/amd64

Output of podman info --debug:
debug:
  compiler: gc
  git commit: ""
  go version: go1.12.10
  podman version: 1.6.2
host:
  BuildahVersion: 1.11.3
  CgroupVersion: v1
  Conmon:
    package: conmon-2.0.2-1.fc30.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.2, commit: a89d21975ee86e84e0b0e1c0f887687582f4b0e3'
  Distribution:
    distribution: fedora
    version: "30"
  IDMappings:
    gidmap:
    - container_id: 0
      host_id: 114172
      size: 1
    uidmap:
    - container_id: 0
      host_id: 114172
      size: 1
  MemFree:<PHONE_NUMBER>
  MemTotal:<PHONE_NUMBER>4
  OCIRuntime:
    name: runc
    package: runc-1.0.0-95.rc9.gitc1485a1.fc30.x86_64
    path: /usr/bin/runc
    version: |-
      runc version 1.0.0-rc9+dev
      commit: 0840802d505e28d3adcbc7a275cbeadbdf64ddec
      spec: 1.0.1-dev
  SwapFree:<PHONE_NUMBER>
  SwapTotal:<PHONE_NUMBER>
  arch: amd64
  cpus: 4
  eventlogger: journald
  hostname: pkratoch.usersys.redhat.com
  kernel: 5.3.7-200.fc30.x86_64
  os: linux
  rootless: true
  slirp4netns:
    Executable: /usr/bin/slirp4netns
    Package: slirp4netns-0.4.0-4.git19d199a.fc30.x86_64
    Version: |-
      slirp4netns version 0.4.0-beta.2
      commit: 19d199a6ca424fcf9516320a327cedad85cf4dfb
  uptime: 2h 18m 2.46s (Approximately 0.08 days)
registries:
  blocked: null
  insecure: null
  search:
  - docker.io
  - registry.fedoraproject.org
  - quay.io
  - registry.access.redhat.com
  - registry.centos.org
store:
  ConfigFile: /home/pkratoch/.config/containers/storage.conf
  ContainerStore:
    number: 0
  GraphDriverName: overlay
  GraphOptions:
    overlay.mount_program:
      Executable: /usr/bin/fuse-overlayfs
      Package: fuse-overlayfs-0.6.5-2.fc30.x86_64
      Version: |-
        fusermount3 version: 3.6.2
        fuse-overlayfs: version 0.6.5
        FUSE library version 3.6.2
        using FUSE kernel interface version 7.29
  GraphRoot: /home/pkratoch/.local/share/containers/storage
  GraphStatus:
    Backing Filesystem: extfs
    Native Overlay Diff: "false"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 0
  RunRoot: /run/user/114172
  VolumePath: /home/pkratoch/.local/share/containers/storage/volumes

Package info (e.g. output of rpm -q podman or apt list podman):
podman-1.6.2-2.fc30.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):
Happens on my system, but it's reproducible in virtual machine, too.

as the warning is suggesting you, you may need to recreate the user namespace:
WARN[0000] you can use `podman system migrate` to recreate the user namespace and restart the containers

Could you try using podman system migrate and then try creating the container again?

podman system migrate did not resolve my issue, I still get the same warnings and errors as before. Is there any other way to "recreate the user namespace"?

so if I read above correctly, you UID for user1 is 100000 and you are adding an additional subuid range<PHONE_NUMBER>36?
That won't work as the ID 100000 is present twice, once as your ID, another time as part of the additional ids.

 If I make the ranges e.g.<PHONE_NUMBER>36 instead, the migrate command works. I am closing this as the problem was on my side. Thank you for investigating this and explaining me what's wrong.

@giuseppe seems like something we should report an error to the user.  Something like Your UID can not be in the range of subuids or subgids.


Missing content

Hey guys ... Wanted to submit some proposed documentation updates/typo fixes via pull request but I can't find the Coldcard docs.  I'm new to Github so could definitely be my fault.

There's a bug in this repo, working on a fix! This one will be deleted and another will be created in its place.


[Toggle] Update docs

It took me a while to figure out how to get the new value from onToggle because I didn't realise the function took a second parameter. I've updated the docs to make this clearer.

[-] PR has tests / docs demo, and is linted.
[yes] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[yes] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

@oliviertassinari I don't think we're consistent here across components, but should isInputChecked just be value rather than the internal variable name?

@mbrookes You are right, we have the following description with the Checkbox component.
It would be better to stay coherent:
    /**
     * ...
     *
     * @param {object} event `change` event targeting the underlying checkbox `input`.
     * @param {boolean} isInputChecked The `checked` value of the underlying checkbox `input`.
     */

@GusRuss89 The messages aren't consistent with the Checkbox, still, it's better than what we have before. Thanks.


Failed to Open Stream: Missing "autoload" file

I'm using the step-by-step installation guide here(http://koel.phanan.net/#install). I get an error when I run this command php artisan init. Error details are giving below.
⤷  php artisan init

Warning: require(/Users/jonesagyemang/Projects/koel/bootstrap/../vendor/autoload.php): failed to open stream: No such file or directory in /Users/jonesagyemang/Projects/koel/bootstrap/autoload.php on line 17

Fatal error: require(): Failed opening required '/Users/jonesagyemang/Projects/koel/bootstrap/../vendor/autoload.php' (include_path='.:') in /Users/jonesagyemang/Projects/koel/bootstrap/autoload.php on line 17

which version of php are you using? looks like a version of php which is too old.

@etopian
⤷  php -v
PHP 5.6.16 (cli) (built: Dec 17 2015 16:38:17)
Copyright (c) 1997-2015 The PHP Group
Zend Engine v2.6.0, Copyright (c) 1998-2015 Zend Technologies

http://stackoverflow.com/questions/28468625/laravel-5-failed-opening-required-bootstrap-vendor-autoload-php

Thanks @etopian I am upgrading to PHP 7.0
Then I will try again
I will keep you updated

Run composer install in your root project folder (or php composer.phar install).
PHP 5.6 should be fine.

 () ~/Projects/koel (master)
⤷  php composer.phar install
Could not open input file: composer.phar

you need to get https://getcomposer.org/composer.phar
after that you need to get nodejs. read the install instruction.
it's much quicker to get docker and use that instead.
https://docs.docker.com/engine/installation/windows/

@etopian I'm using MacOS's Mavericks
I enabled OpenSSL in php.ini and re-run composer install.
Thanks for the SO link. That's where I found the answer!

:+1:

There are several reasons why this is caused:

/storage and /bootstrap don't have file permission 775
Old version of PHP
Not running composer update in the project.


implement SFX for Hestmor Breathing when Idle

Creating the issue so it stays organized:
https://github.com/user-attachments/files/16616319/HestmorIdleBreathingLoop.zip

@Pseudo-64 the breathing sound is ready for testing


Flinkcdc 增量读取oralce ，正常输出数据一段时间后 异常关闭

flinkcdc增量读取oralce数据库
报错： java.lang.IllegalStateException: Retrieve schema history failed, the schema records for engine 08c6c43f-e529-439b-9546-d1056524d389 has been removed, this might because the debezium engine has been shutdown due to other errors.
at com.ververica.cdc.debezium.utils.DatabaseHistoryUtil.retrieveHistory(DatabaseHistoryUtil.java:77)
at com.ververica.cdc.debezium.internal.FlinkDatabaseSchemaHistory.configure(FlinkDatabaseSchemaHistory.java:82)
at io.debezium.relational.HistorizedRelationalDatabaseConnectorConfig.getDatabaseHistory(HistorizedRelationalDatabaseConnectorConfig.java:105)
at io.debezium.relational.HistorizedRelationalDatabaseSchema.(HistorizedRelationalDatabaseSchema.java:39)
at io.debezium.connector.oracle.OracleDatabaseSchema.(OracleDatabaseSchema.java:35)
at io.debezium.connector.oracle.OracleConnectorTask.start(OracleConnectorTask.java:54)
at io.debezium.connector.common.BaseSourceTask.start(BaseSourceTask.java:130)
at io.debezium.connector.common.BaseSourceTask.startIfNeededAndPossible(BaseSourceTask.java:207)
at io.debezium.connector.common.BaseSourceTask.poll(BaseSourceTask.java:148)
at io.debezium.embedded.EmbeddedEngine.run(EmbeddedEngine.java:788)
at io.debezium.embedded.ConvertingEngineBuilder$2.run(ConvertingEngineBuilder.java:188)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)
Environment :

Flink version :  1.13.1
Flink CDC version: 2.2.1
Database and version: oracle 11g

To Reproduce
Steps to reproduce the behavior:

Thes test data :
The test code :
The error :

Additional Description
If applicable, add screenshots to help explain your problem.
2022-08-07 03:31:58,515 DEBUG org.apache.flink.runtime.io.network.partition.ResourceManagerPartitionTrackerImpl [] - Processing cluster partition report from task executor container_e92_1642414791080_10224_01_000002: PartitionReport{entries=[]}.
2022-08-07 03:31:58,541 DEBUG org.apache.flink.runtime.resourcemanager.active.ActiveResourceManager [] - Trigger heartbeat request.
2022-08-07 03:31:58,541 DEBUG org.apache.flink.runtime.jobmaster.JobMaster                 [] - Received heartbeat request from 16e2c7709ada250726d6361b0bd7b7c9.
2022-08-07 03:31:58,541 DEBUG org.apache.flink.runtime.resourcemanager.active.ActiveResourceManager [] - Received heartbeat from 2b7e87604695f4e1f737ca3897284ee8.
2022-08-07 03:31:59,841 DEBUG org.apache.flink.runtime.jobmaster.JobMaster                 [] - Trigger heartbeat request.
2022-08-07 03:31:59,843 DEBUG org.apache.flink.runtime.jobmaster.JobMaster                 [] - Received heartbeat from container_e92_1642414791080_10224_01_000003.
2022-08-07 03:31:59,843 DEBUG org.apache.flink.runtime.jobmaster.JobMaster                 [] - Received heartbeat from container_e92_1642414791080_10224_01_000002.
2022-08-07 03:31:59,843 DEBUG org.apache.flink.runtime.jobmaster.JobMaster                 [] - Received heartbeat from container_e92_1642414791080_10224_01_000004.
2022-08-07 03:31:59,843 DEBUG org.apache.flink.runtime.jobmaster.JobMaster                 [] - Received heartbeat from container_e92_1642414791080_10224_01_000005.
2022-08-07 03:31:59,844 DEBUG org.apache.flink.runtime.jobmaster.JobMaster                 [] - Received heartbeat from container_e92_1642414791080_10224_01_000006.
2022-08-07 03:32:00,220 DEBUG org.apache.flink.runtime.jobmaster.JobMaster                 [] - Archive local failure causing attempt a7bb32ade11f8400a55e761081eb0e46 to fail: java.lang.IllegalStateException: Retrieve schema history failed, the schema records for engine 08c6c43f-e529-439b-9546-d1056524d389 has been removed, this might because the debezium engine has been shutdown due to other errors.
at com.ververica.cdc.debezium.utils.DatabaseHistoryUtil.retrieveHistory(DatabaseHistoryUtil.java:77)
at com.ververica.cdc.debezium.internal.FlinkDatabaseSchemaHistory.configure(FlinkDatabaseSchemaHistory.java:82)
at io.debezium.relational.HistorizedRelationalDatabaseConnectorConfig.getDatabaseHistory(HistorizedRelationalDatabaseConnectorConfig.java:105)
at io.debezium.relational.HistorizedRelationalDatabaseSchema.(HistorizedRelationalDatabaseSchema.java:39)
at io.debezium.connector.oracle.OracleDatabaseSchema.(OracleDatabaseSchema.java:35)
at io.debezium.connector.oracle.OracleConnectorTask.start(OracleConnectorTask.java:54)
at io.debezium.connector.common.BaseSourceTask.start(BaseSourceTask.java:130)
at io.debezium.connector.common.BaseSourceTask.startIfNeededAndPossible(BaseSourceTask.java:207)
at io.debezium.connector.common.BaseSourceTask.poll(BaseSourceTask.java:148)
at io.debezium.embedded.EmbeddedEngine.run(EmbeddedEngine.java:788)
at io.debezium.embedded.ConvertingEngineBuilder$2.run(ConvertingEngineBuilder.java:188)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)

I have the same problem

I have the same problem

I add
debezium.log.mining.strategy=online_catalog
debezium.log.mining.continuous.mine=true

it become ok

Closing this issue because it was created before version 2.3.0 (2022-11-10). Please try the latest version of Flink CDC to see if the issue has been resolved. If the issue is still valid, kindly report it on Apache Jira under project Flink with component tag Flink CDC. Thank you!
cc @GOODBOY008

Closing this issue because it was created before version 2.3.0 (2022-11-10). Please try the latest version of Flink CDC to see if the issue has been resolved. If the issue is still valid, kindly report it on Apache Jira under project Flink with component tag Flink CDC. Thank you!

I have the same problem，not solved


Fix: add aliases for IsCountable in ValidatorPluginManager

Q
A

Documentation
no

Bugfix
yes

BC Break
no

New Feature
no

RFC
no

QA
no

Description
Aliases for the IsCountable validator were missing in the ValidatorPluginManager alias definitions.
fixes #110

so @Ocramius @froschdesign ... should I drop the v2 of servicemanager here or in a new PR? thanks!

@rieschl
Sorry for the very late response! 😞
Please create a separate pull request to drop the old version of laminas-servicemanager. This allows us to release the current pull request as a bug-fix.


Use "spawn" when using multiprocessing.Pool

Read https://pythonspeed.com/articles/python-multiprocessing/ first
Read http://0pointer.net/blog/file-descriptor-limits.html second
What changed?

When using multiprocessing.Pool, switch to use "spawn". (Linux default is fork). Parent open fd is inherited by child process.

Rationale

Parent process has a lot of open file descriptors that maybe problematic. It's true that we only use multiprocessing to load YAML, so you can argue maybe we don't want to touch this at all. But be ware of any multiprocessing usage, the effect of default linux model is fork instead of spawn and you need to worry about your file descriptors.

fork is faster because it does a copy-on-write of the parent process's entire virtual memory including the initialized Python interpreter, loaded modules, and constructed objects in memory.

But fork does not copy the parent process's threads. Thus locks (in memory) that in the parent process were held by other threads are stuck in the child without owning threads to unlock them, ready to cause a deadlock when code tries to acquire any of them. Also any native library with forked threads will be in a broken state.

The copied Python modules and objects might be useful or they might needlessly bloat every forked child process.

The child process also "inherits" OS resources like open file descriptors and open network ports. Those can also lead to problems but Python works around some of them.

So fork is fast, unsafe, and maybe bloated.

However these safety problems might not cause trouble depending on what the child process does.
spawn starts a Python child process from scratch without the parent process's memory, file descriptors, threads, etc. Technically, spawn forks a duplicate of the current process, then the child immediately calls exec to replace itself with a fresh Python, then asks Python to load the target module and run the target callable.

So spawn is safe, compact, and slower since Python has to load, initialize itself, read files, load and initialize modules, etc.

However it might not be noticeably slower compared to the work that the child process does.
forkserver forks a duplicate of the current Python process that trims down to approximately a fresh Python process. This becomes the "fork server" process. Then each time you start a child process, it asks the fork server to fork a child and run its target callable.

Those child processes all start out compact and without stuck locks.

forkserver is more complicated and not well documented. [Bojan Nikolic's blog post](http://www.bnikolic.co.uk/blog/python/parallelism/2019/11/13/python-forkserver-preload.html) explains more about forkserver and its secret set_forkserver_preload() method to preload some modules. Be wary of using an undocumented method, esp. before the [bug fix in Python 3.7.0](https://bugs.python.org/issue28779).

So forkserver is fast, compact, and safe, but it's more complicated and not well documented.

How was it tested?
If it was manually verified, list the instructions for your reviewers to follow.

[x] Unit Tests
[ ] Functional Tests
[ ] Manually Verified

Still need to test against linux and lambda github stuff. (because lambda does not have shared memory)

Have you been able to reproduce the error on load_templates?  I'm not sure this is going to solve the file descriptor limitation. We're really just using a process pool to use ruamel without dealing with its mem leak. If this was the problem you should be able to replicate it using the load test repo. I'm going to push what I think may help alleviate some of the issues but at a certain point someone will need to bump their ULIMIT and we give them a formulate to calculate how much it should be.

maybe abandon for now. revisit when we hit the fork problem (fd leaks). otherwise, this is probably too scary to introduce.


Add minor tweaks

Context:
The paper got published, so the reference needs updating in the README. The contraction flops calculation that was used in the paper benchmarks is slightly different than the version in the code right now. So this has also been updated to be consistent.
Description of the Change:

Update Readme to with the new reference to the published article.
Add minor tweak to the contractions flops calculation.

Changes look fine to me, but not sure why the CI is explicitly failing

Also, do you mind documenting these changes in the changelog?

Awesome, thanks guys for the comments! I made the necessary changes.


[AccordionSummary] false positive propType warning with disableGeneration

When AccordionSummary is checking for usage of the deprecated classes.focused prop, the code calls indexOf on classes.focused without checking if that property is defined. If you don't supply classes.focused, this throws Failed prop type: Cannot read property 'indexOf' of undefined.
It appears to have been introduced in this change: https://github.com/mui-org/material-ui/commit/9c48d0bb71a3a9c4c0dfbd6bdf8e9605e69a1444#diff-a3e45615b0516b905db6b26935126e30763f45b8901b2bee960adb5c3be2b997R147
A workaround is to always declare focused as an empty string in the classes prop (note: this does not work in Typescript, since that class was removed from the class keys), but ideally the checker should check if it's defined first.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

An error is thrown if you use AccordionSummary without explicitly passing classes.focused
Expected Behavior 🤔

No error should be thrown.
Steps to Reproduce 🕹

CodeSandbox: https://codesandbox.io/s/thirsty-ives-bwmqv
Steps:

Use ServerStyleSheets with disableGeneration: true option
Use AccordionSummary with a classes prop that does not include focused
Receive propType warning Failed prop type: Cannot read property 'indexOf' of undefined

Context 🔦

This can break tests that check that components are being passed the correct props (for example, using the prop-types-throw library: https://www.npmjs.com/package/prop-types-throw)
Your Environment 🌎

`npx @material-ui/envinfo`
  Don't forget to mention which browser you used.
  Output from `npx @material-ui/envinfo` goes here.

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-latest), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.

Thanks, updated with a CodeSandbox link and repro steps with some additional context.

Fixed in https://github.com/mui-org/material-ui/pull/27385


We update the ConsistentIDv1 SDXL model and pretrained parameters.

We update the ConsistentIDv1 SDXL model and pretrained parameters.
https://huggingface.co/JackAILab/ConsistentID/blob/main/ConsistentID_SDXL-v1.bin
Hope to update synchronously

Thank you for informing about the update.
Currently occupied with integrating CogVideoX
https://github.com/newgenai79/newgenai
Will soon look into this.

@Jandown
Did they released Huggingface demo for SDXL?
There are so many code changes, difficult to merge.

@Jandown
Did they released Huggingface demo for SDXL?
There are so many code changes, difficult to merge.

I don't think so, at least I haven't found it


read_json: error tells to provide columns types but there is no such parameter

In [5]: sf = tc.SFrame.read_json('/Users/me/Downloads/annotations/image_info_test2014.json', orient="records")
Parsing JSON records from /Users/me/Downloads/annotations/image_info_test2014.json
Unable to parse /Users/tmuss/Downloads/annotations/image_info_test2014.json. It does not appear to be in JSON record format. A list of dictionaries is expected
RuntimeError                              Traceback (most recent call last)
 in ()
----> 1 sf = tc.SFrame.read_json('/Users/me/Downloads/annotations/image_info_test2014.json', orient="records")
/Users/tmuss/venv/dev/lib/python2.7/site-packages/turicreate/data_structures/sframe.pyc in read_json(cls, url, orient)
1710             g = SArray.from_json_records(url)
1711             g = SFrame({'X1':g})
-> 1712             return g.unpack('X1','')
1713         elif orient == "lines":
1714             g = cls.read_csv(url, header=False)
/Users/me/venv/dev/lib/python2.7/site-packages/turicreate/data_structures/sframe.pyc in unpack(self, unpack_column, column_name_prefix, column_types, na_value, limit)
5385             column_name_prefix = unpack_column
5386
-> 5387         new_sf = self[unpack_column].unpack(column_name_prefix, column_types, na_value, limit)
5388
5389         # construct return SFrame, check if there is conflict
/Users/me/venv/dev/lib/python2.7/site-packages/turicreate/data_structures/sarray.pyc in unpack(self, column_name_prefix, column_types, na_value, limit)
3258             lengths = [len(i) for i in head_rows]
3259             if len(lengths) == 0 or max(lengths) == 0:
-> 3260                 raise RuntimeError("Cannot infer number of items from the SArray, SArray may be empty. please explicitly provide column types")
3261
3262             # infer column types for dict type at server side, for list and array, infer from client side
RuntimeError: Cannot infer number of items from the SArray, SArray may be empty. please explicitly provide column types

add parameter or update error message?

I think we should add the column_types parameter and it should probably behave the same way as with read_csv. @shantanuchhabra, do you mind taking this one?

I acquired the same JSON file that triggered this error and repro'ed this using Turi Create 5.0 beta 3. I got the following error (which doesn't mention anything about column types):
In [2]: sf = tc.SFrame.read_json('image_info_test2014.json')
Parsing JSON records from /Users/schhabra/Downloads/image_info_test2014.json
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
<ipython-input-2-7cc26e872a18> in <module>()
----> 1 sf = tc.SFrame.read_json('image_info_test2014.json')

/Users/schhabra/Downloads/venv/lib/python2.7/site-packages/turicreate/data_structures/sframe.pyc in read_json(cls, url, orient)
   1559         """
   1560         if orient == "records":
-> 1561             g = SArray.read_json(url)
   1562             if len(g) == 0:
   1563                 return SFrame()

/Users/schhabra/Downloads/venv/lib/python2.7/site-packages/turicreate/data_structures/sarray.pyc in read_json(cls, filename)
    583         """
    584         proxy = UnitySArrayProxy()
--> 585         proxy.load_from_json_record_files(_make_internal_url(filename))
    586         return cls(_proxy = proxy)
    587

turicreate/cython/cy_sarray.pyx in turicreate.cython.cy_sarray.UnitySArrayProxy.load_from_json_record_files()

turicreate/cython/cy_sarray.pyx in turicreate.cython.cy_sarray.UnitySArrayProxy.load_from_json_record_files()

RuntimeError: Unable to parse /Users/schhabra/Downloads/image_info_test2014.json. It does not appear to be in JSON record format. A list of dictionaries is expected

This error does not repro anymore. Closing this issue for now. We can create a new issue for adding column_types to read_json later if needed.

@shantanuchhabra I think the issue is to update the error message right?


Fix #412: just import everything at the outset.

Hi @rabidbit -- I took a quick look at this. Turns out what was happening was that the imports were being placed at class-level:
class StudentCode(object):
    import X
    import Y

    def run():
         # student's code goes here

and for some reason Skulpt wasn't finding them. On investigation, just importing everything that's supported at the top of the file seems to be the simplest way to handle this, so I went ahead and modified the starter code instead so that collections, math, etc. are automatically available.
The one thing I'm not sure about is whether we want to put a comment at the top of the coding area to say something like this:
    # This is a Python environment that you can type code into. The following
    # libraries have already been pre-imported and can be used if you need them:
    # collections, copy, image, math, operator, ...

but I'm not sure. On the one hand it gives info that might be useful. On the other hand I don't want to create the suggestion that the user should be using something from here, and it's not something an interviewer would say. My current leaning is to not have it for now, but add it later if we detect confusion (hopefully the PR as-is will address most of the issues since there won't be random import errors showing up). Wdyt?

I agree that we should not include in starter code, as it's likely users will try to use them even if they don't need them, resulting in distraction and frustration.
Can we detect if users try to import something? If we detect something being imported, we could say something like this in the feedback:

Looks like you are trying to import libraries. TIE currently supports and auto-imports the following libraries: collections, copy, image, math, and operator. If you think you need to use any of these, you can simply call the method you require.

I think we already do that, but only for unsupported libraries -- see the feedback when you type "import x", for example. For supported libraries, nothing happens, and I think including the note in that case would be a distraction.
So, I think the current behaviour is fine as-is, but let me know if you want me to make further changes?

Sorry, I should have been more specific in #412 . Users are typing import statements above the template code, like this:
import someLibrary

def someCode(x):
    stuff = someLibrary.doStuff()
    return stuff

Which results in the following error:
It looks like someLibrary isn't a declared variable.
Did you make sure to spell it correctly?
And is it correctly initialized?

Which is the same error they get if they didn't have an import statement. That's the issue for #412
If users can enter import statements above the template code, then I think the existing flow and feedback would work.

Right -- I believe I had tested that case in my PR and it works as expected. Please could you verify?

I didn't realize that this occurred only for supported but not imported modules. I think I understand the issue and fix. This appears to fix the bug.

Merged. Thanks!


Benchmark publishing

Could you please add a topic on the mainpage that lists benchmark results of all zero-allocation hash functions available? I may be asking for a trouble for komihash, but it would be interesting to know anyway.

Since benchmarking in Java is much more complicated than in C/C++ due to just-in-time optimizations by the JVM, we currently do not want to maintain a list of benchmark results. However, there are ideas for a test infrastructure to automatically run benchmarks on different platforms and JVMs in future.
For now, you can execute the corresponding benchmarks by running ./gradlew jmh. If you are only interested in functions for hashing byte arrays without allocations you could add
jmh {
    includes=['com.dynatrace.hash4j.hashing.*hashBytes.*Direct']
}

to build.gradle.

No problem, thanks!

Thank you. Looks like komihash5 is not that much behind wyhashFinal4. komihash can't be faster due to a bit more complex structure.


Excessive Memory usage after upgrading to 5.1.5

Env :
Elementary OS
2018-08-21 09:24:31.331 - DBeaver <IP_ADDRESS>808130751 is starting
2018-08-21 09:24:31.332 - OS: Linux 4.15.0-32-generic (amd64)
2018-08-21 09:24:31.333 - Java version: 1.8.0_181 by Oracle Corporation (64bit)
2018-08-21 09:24:33.610 - Disable Eclipse action set 'org.eclipse.ui.edit.text.actionSet.annotationNavigation'
2018-08-21 09:24:33.610 - Disable Eclipse action set 'org.eclipse.ui.WorkingSetActionSet'
*** BUG ***
In pixman_region32_init_rect: Invalid rectangle passed
Set a breakpoint on '_pixman_log_error' to debug

With DBeaver 5.1.1
I see following logs
Gtk-Message: Failed to load module "pantheon-filechooser-module"
(dbeaver:6781): Gtk-WARNING **: Unable to locate theme engine in module_path: "pixmap",
(dbeaver:6781): Gtk-WARNING **: Unable to locate theme engine in module_path: "pixmap",
(dbeaver:6781): Gtk-WARNING **: Unable to locate theme engine in module_path: "pixmap",
(dbeaver:6781): Gtk-WARNING **: Unable to locate theme engine in module_path: "pixmap",
Gtk-Message: Failed to load module "pantheon-filechooser-module"
(java:6798): Gtk-WARNING **: Unable to locate theme engine in module_path: "pixmap",
(java:6798): Gtk-WARNING **: Unable to locate theme engine in module_path: "pixmap",
(java:6798): Gtk-WARNING **: Unable to locate theme engine in module_path: "pixmap",
(java:6798): Gtk-WARNING **: Unable to locate theme engine in module_path: "pixmap",

Adding

--launcher.GTK_version
2

to /usr/share/dbeaver/dbeaver.ini file solved the problem

This is no longer possible, since GTK2 is no longer supported. Is there another workaround?


Add PoolArray::to_vec(), improve doc around read()/write()

Closes #806.
I noticed that the documentation around the read() and write() methods was a bit lacking, and tried to make it a bit easier to understand. Also moved trait impls after the inherent impl.
I'm not 100% sure that we really need to_vec(). It's nice as a convenience, but it may cause people to do things like
pool_array.to_vec().map(...).collect()
// or
for elem in pool_array.to_vec()

instead of going via read(). After all, read().to_vec() is not much longer.
On the other hand, we already have PoolArray::from_vec(), so it's somewhat symmetric.
I'd be happy for some input on that.

bors r+


brew formulae question

Q: Is it possible to use this repository as the default brew formulae?

Rather than the default python brew option.

label:enhancement
label:question

I'm not the maintainer of the brew formula, so I would ask this question on the Homebrew repo. I think they'll be able to make the change.


Keep getting 'unsupported attribute' errors on state migration

Bug description 🐞
Hi guys, I'm attempting to run some state migrations from terrakube.io to our Terrakube instance. I keep getting 'unsupported attribute' errors on various states. The attribute that it errors out on isn't always the same, but it happens when there is an attribute in the state file that is set to 'null'. For example:

And this is the error:

Steps to reproduce
Try to migrate any state file that has an attribute with a null value and it will throw that error.
Expected behavior
Ignore the null attribute
Example repository
No response
Anything else?
No response

Hello @jderieg
I am not really sure about this issue because internally we are not doing any kind of processing or modification to the terraform state when using the CLI driven workflow, the terraform CLI is just sending a base64 encoded string to Terrakube  API and we take the value and save it.
Do you see any logs in the API?
Could you share some terraform code that generate a state with a null value?
Maybe it is a terraform CLI or terraform provider issue but not sure....

OK, so I did some more testing.
If I completely delete my local .terraform directory and .terraform.lock.hcl file, do a new 'terraform init' and pull a brand-new fresh state file from the source, it pushes up to Terrakube without error. I'm not exactly sure why though.
I think it's possible that my local .terraform directory had existed for long enough that some state changes had nullified some of the attribute values that existed, so Terraform didn't know what to do with them on the state push. That's my best guess.
Anyway, I guess the workaround is to delete the local .terraform directory and re-init.
I didn't see anything else in the api log when it threw this error, so maybe it didn't even get that far and the error was thrown by my local Terraform binary.
I think we can close this issue as it doesn't actually seem to be related to Terrakube. Thanks!


Set agent's capabilities though configure script

I'd like to set agent's capabilities through configure.exe script when register new agent. Is it possible?
Thanks

@melezhik the only way i can think would be (not ideal):
Create a .env file under agent root and put your capability there in format:
foo=bar
test=test1

the agent will add foo and test as environment variables and report those as system capabilities.

Yeah, thanks @TingluoHuang  sounds like a workaround


Extend 'resolveHours' pipes' functionality

Make it return days passed as well as hours when appropriate

Duplicates #89


Export CommonJS version

The problem to solve
Is your feature request related to a problem you are experiencing that is not a bug? Please describe.
I want to use the project, but the project I'm working on does not support ESModules.
Proposed solution
It'd be nice to have a /commonjs counterpart to the current implementation.
Alternatives
Use something like babel to output a CommonJS version before publishing it

In any async function in commonjs code in Node.js, you can write:
const { generate, count } = await import('random-words');

We plan to use this method ourselves.
I'm interested to know why and whether your project can't use this technique.

It's a legacy project that has +2 years; I'm enhancing the coverage, hoping to have more reliability for further changes.
The thing is that this project is running on CommonJS in an older Node version (before top-level await); my wish was to use the project to generate some params for a jest it.each. All tests in Jest must be declared synchronously, so having it inside an async function would not be an option.
I ended up installing @faker-js/faker on the end :confused:, but having this one portable to CommonJS would be helpful, and I'm sure it'd enhance the project's usage.

FYI, Node 18 supports top level await. Only in .mjs files, and that's not
going to change, but you could do your dynamic import in an async function.
I think it might make more sense though for you to do your own babelization
rather than an actual fork so you don't have to maintain the code. Best
wishes with your project.
On Fri, Dec 8, 2023 at 11:16 AM Wesley @.***> wrote:

We're under v18.14.2 here. I'll close this, and when I have some
availability, I can fork it and add CommonJS support.
Thank you anyway!
—
Reply to this email directly, view it on GitHub
https://github.com/apostrophecms/random-words/issues/43#issuecomment-1847467060,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH27MVI4CPFP7PO4DIQ7TYIM4PPAVCNFSM6AAAAABAMVHY6SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNBXGQ3DOMBWGA
.
You are receiving this because you commented.Message ID:
@.***>

--
THOMAS BOUTELL | CHIEF TECHNOLOGY OFFICER
APOSTROPHECMS | apostrophecms.com | he/him/his

Why are we trying to normalize esm on a backend......

I am compiling TS to code to CommonJS to run on Node 18 and none of the approaches are working:
import { generate } from 'random-words';

async function slugify() => {
    // ...
	const { generate } = await import('random-words');
    // ...
}

Both of them error out with ERR_REQUIRE_ESM

@pksunkara it's returning this error because since you're transpilling into common js, you don't run it in ESM.
About the second version, the exported code uses await import or it's being changed to other thing?

Is this an issue somehow unique to the library or are you having a general
problem using ES modules in your specific environment? If the latter I
suggest taking that to stackoverflow or another appropriate forum and
creating simple test case ES modules to experiment with as it is not
something specific to this library.
On Tue, Jan 9, 2024 at 7:23 AM Pavan Kumar Sunkara @.***>
wrote:

Ofcourse I understand what the issue is.
All I am trying to say is that because of how tsc is compiling the second
version and changing it, even the second version doesn't work.
Basically the conclusion is that this library does not work at all for
commonjs backend with typescript.
—
Reply to this email directly, view it on GitHub
https://github.com/apostrophecms/random-words/issues/43#issuecomment-1882976982,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH27JQYWMJGO4MEGEKPHDYNUZDPAVCNFSM6AAAAABAMVHY6SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBSHE3TMOJYGI
.
You are receiving this because you commented.Message ID:
@.***>

--
THOMAS BOUTELL | CHIEF TECHNOLOGY OFFICER
APOSTROPHECMS | apostrophecms.com | he/him/his

For everyone looking for CSJ alternatives: https://www.npmjs.com/package/random-word-slugs
It is the same issue as with all Sindre Sorhus packages: legacy codebases exist. Splitting ecosystem into opposing camps doesn't help anyone. Add a compile step people.

There's no intention here to split the ecosystem, we just can't support everything and have chosen to offer an ES module. For free, keep in mind. Use what you like, and consider upgrading your tooling to something that can also handle ES modules. Or use something else; that's fine too.

All right, I see your point.

At the time I declined to support this, I had not realized the extent to which it was painful to use from commonjs. Since then I've had occasion to actually try that, with other modules, and you're right: it's a bad scene.
This is actually scheduled to change - node 22 will, at some point, start supporting requires of ES modules as long as they have no top level async code - but in the meantime, it makes more sense to have tooling for it.
Would you like to submit your changes as a PR? As long as the update procedure is clear I see the benefit at this point. Sorry for making this contentious.

FWIW, with even with that support in node 22, ts-node still doesn't work. Ref: https://github.com/TypeStrong/ts-node/issues/1007#issuecomment-2181478441

Yeah, I'm not surprised.

Adding my +1 to this!


Cannot read property 'toCommandArgument' of undefined

Environment data
VS Code version:  1.19.3
Python Extension version:  2018.1.0
Python Version:   3.6.4 (x86)
OS and version:    WIndows 10 x64
Actual behavior
Getting an error when running "Run Python File in Terminal" using a key binding
Error: Cannot read property 'toCommandArgument' of undefined
	at TerminalCodeExecutionProvider.<anonymous> (C:\Users\Szabi\.vscode\extensions\ms-python.python-2018.1.0\out\client\terminals\codeExecution\terminalCodeExecution.js:44:99)
	at Generator.next (<anonymous>)
	at fulfilled (C:\Users\Szabi\.vscode\extensions\ms-python.python-2018.1.0\out\client\terminals\codeExecution\terminalCodeExecution.js:15:58)
	at <anonymous>

Expected  behavior
The python file should be executed without error.
Steps to reproduce:
I was able to reproduce this with a fresh new install of vscode and the python extension only

Install this extension
Run a python file in terminal using ctrl+shit+P
Now you can assign a key binding to this command (I used ctrl+alt+P)
Press ctrl+alt+P on the same file
A WARN is shown and the file is not executed

Logs
Output from Python output panel
##########Linting Output - pylint##########
Using config file d:\Programming\ModernGL\.pylintrc

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

Your code has been rated at 10.00/10 (previous run: 10.00/10, +0.00)

Output from Console window (Help->Developer Tools menu)
[Extension Host] Python Extension: Failed to get conda info from conda null
t.log @ /C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:249
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:1226   ERR Cannot read property 'toCommandArgument' of undefined: TypeError: Cannot read property 'toCommandArgument' of undefined
	at TerminalCodeExecutionProvider.<anonymous> (C:\Users\Szabi\.vscode\extensions\ms-python.python-2018.1.0\out\client\terminals\codeExecution\terminalCodeExecution.js:44:99)
	at Generator.next (<anonymous>)
	at fulfilled (C:\Users\Szabi\.vscode\extensions\ms-python.python-2018.1.0\out\client\terminals\codeExecution\terminalCodeExecution.js:15:58)
	at <anonymous>
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:156 WARNING: Promise with no error callback:123
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:156 Object
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:249 [Extension Host] (node:9212) DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.
t.log @ /C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:249
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:1226   ERR Cannot read property 'toCommandArgument' of undefined: TypeError: Cannot read property 'toCommandArgument' of undefined
	at TerminalCodeExecutionProvider.<anonymous> (C:\Users\Szabi\.vscode\extensions\ms-python.python-2018.1.0\out\client\terminals\codeExecution\terminalCodeExecution.js:44:99)
	at Generator.next (<anonymous>)
	at fulfilled (C:\Users\Szabi\.vscode\extensions\ms-python.python-2018.1.0\out\client\terminals\codeExecution\terminalCodeExecution.js:15:58)
	at <anonymous>
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:156 WARNING: Promise with no error callback:125
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:156 Object
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:1226   ERR Cannot read property 'toCommandArgument' of undefined: TypeError: Cannot read property 'toCommandArgument' of undefined
	at TerminalCodeExecutionProvider.<anonymous> (C:\Users\Szabi\.vscode\extensions\ms-python.python-2018.1.0\out\client\terminals\codeExecution\terminalCodeExecution.js:44:99)
	at Generator.next (<anonymous>)
	at fulfilled (C:\Users\Szabi\.vscode\extensions\ms-python.python-2018.1.0\out\client\terminals\codeExecution\terminalCodeExecution.js:15:58)
	at <anonymous>
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:156 WARNING: Promise with no error callback:137
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:156 Object

For the problem exist when you run the command from the command palette.

Me too:
VS Code version: 1.19.2
Python Extension version: 2018.1.0
Python Version: 2.7.13 (x86); Anaconda
OS and version: Ubuntu x64
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.3 LTS
Release:        16.04
Codename:       xenial
Started happening out of nowhere.

@nicain
Does the problem exist when you run the command from the command palette?
Please could you provide some steps to replicate this.

Same for me:
Version 1.19.3
Commit 7c4205b5c6e52a53b81c69d2b2dc8a627abaa0ba
Date 2018-01-25T10:36:34.867Z
Shell 1.7.9
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64
So i assign a key binding : Ctrl + Alt + R
it is giving the same error, but running command palette work irrespective of above setting,

Yesterday it was working fine. Today in the morning I updated this extension and the problem persists even after a clean install.

Same problem for me, I'm unable to solve it. What data sould I provide? I'm not familiar with VSCode logs.

@richardSin501 we need to know whether the command Run Python File in Terminal works for you (e.g. we need to know whether it's a keybinding thing or the actual command itself).

@brettcannon Yes it works every way other than using the keybinding.

@richardSin501 Thanks for the info! Now we have to reproduce on our side to figure out what has happened.

The same for me.
VSC Version 1.19.3
ms-python 2018.1.0
Context menu and command palette - OK, keybinding not working.
Additional info:
If custom user setting "python.terminal.executeInFileDir" set to True, error became "Path must be a string. Received undefined".

I just realized we don't have the keybindings.json setting that used to work and is now failing for people. Could someone share that?

Found the problem, previously we had a check to ensure the file existed and it was a python file, we're missing that check (which used to indirectly allow shortcuts to work as well).

Once CI passes, you can download the insiders build to verify that we fixed the issue.

Thanks! That solved the problem.  But I'm missing the double quotes around the filename (needed for use in cygwin)

@hajeka please could you file a separate issue for that.

Hello, How do I solve the problem? I have asigned the combination ctrl + b to execute python, but I get an error message "Cannot read property 'toCommandArgument' of undefined".

@Noah-Elbec
Please download and install the insiders build from here

@DonJayamanne Thank you very much, the problem was solved by installing the Insiders Build extension.

The insider build works like a charm. Thank you for the quick fix!

I still get this message with VS Code insiders 1.20.0.

Works Perfectly! Thank you so much!

@DonJayamanne worked great for me too, and an easy update.  Thanks!

use the insiders build is the only way to fix this? nothing to do in the stable version?

@wilfredinni

the insiders build is the only way to fix this? nothing to do in the stable version?

We will deliver this fix in the next version.

This is still in issue in latest stable.

Please read the comment above.

Please read the comment above. Fix will be in the nextt release.
When is the next release?

end of the month

Still an issue with latest release 1.21.0

Still not working for me in the current insiders build.

@listentojohan @tweakimp
You need to install the Insiders Build of the Python Extension, not VS Code.
Please follow the above link

Thanks, that fixed it. Why is this build not the one you get when you download an extension for the insiders build, so confusing  :(

Thanks, that fixed it. Why is this build not the one you get when you download an extension for the insiders build, so confusing :(

VS Code doesn't support releasing multiple versions of the same extension. I.e. whether you use insiders or stable build of VS Code, the marketplace is the same.
FYI - The latest release was just published, you can now install the latest official release of the extension.

We have renamed our a "development build" to disambiguate, but as of right now we don't have something on the marketplace because it's time taken away from working on the extension to constantly be uploading a new .vsix every day.


more features, sources, allow user to add features

would be good to showcase that it's easy to add other features to lookup table
also maybe add oxidation-state-dependent things

When this says "load sources", are you referring to the deepchem.molnet.load_*source*(featurizers, splitters) for each of the molnet datasources? There are also other options methods for loading from disks, dataframes, and arrays

I'm not familiar enough with molnet to know exactly what you're talking about there, but what I'm referring to here is including more features (from other data sources) in the built-in data tables for the package. Currently it's mostly just atomic features from pymatgen and I think a few I pulled from matminer too.

I see. Deepchem contains methods to load the various MoleculeNet datasets and preprocess the data with splitters and featurizers.

Adding to this that it would be good also to build a way for a user to add in their own data source for atomic features that could hook into the automated vector-building functions. One could already in principle add user-sourced features but only by "manually" providing either feature matrices or at least vectors directly, so you couldn't use the functions that build features for you if only a list of feature names is provided.

This is covered by restructure PR and I neglected to link it, closing.


Make rebooting optional

https://github.com/SineQuaNonSoftware/puppet-proxmox/blob/aa2c24aeccb8f1d3675e18ce4759fa8dfb90e64e/manifests/install.pp#L25
Pass a parameter that enables rebooting, just not do it by default. Also, reboot resource defaults to "immediately", maybe delay that until the end of the puppet run at least ?

I think only the reboot after installing proxmox-ve should be mandatory (kernel change). The network should not.
And yes, pushing the reboot to the end of the run is fine.

This is already done isn't it?

Not completely. I switched it to be done at the end of the run, but did not make it optional in any way.

Well, since you also changed how we handle the network, the only moment that reboot will be triggered is right after installing the proxmox kernel for the first time.
I personnally wouldn't use that option and can't think of a reason you'd want to have a partially working admin interface and unusable server until you reboot it yourself?

Network changes are applied through an ifup/down and not a reboot.
This reboot will be triggered on update too.


coreg-GUI:  catch fiff files without digitizer info

Small fix that should not conflict with #4971: Show appropriate error message for previously uncaught exception, for fiff files with measurement info but without digitizer data (e.g. MNE-sample-data/MEG/sample/ernoise_raw.fif)

Test failures are unrelated, thanks @christianbrodbeck


Fix event chunk getting broken after join

Purpose

Fixes #1532

Goals

Support scatter-gather data processing with joins.

Approach

Allow a list of event checks to be processed together.
Remove isBatch is event check and make all event checks to behave as a batch by default.
Introduce empty window to process events as a batch in joins without using lenghtBatch(0) window.

Release note

Support scatter-gather data processing with joins.

Codecov Report

Merging #1559 into master will decrease coverage by 0.1%.
The diff coverage is 70.86%.

@@             Coverage Diff              @@
##             master    #1559      +/-   ##
============================================
- Coverage     67.89%   67.79%   -0.11%
  Complexity       45       45
============================================
  Files           648      650       +2
  Lines         27322    27401      +79
  Branches       4298     4312      +14
============================================
+ Hits          18551    18577      +26
- Misses         6976     7017      +41
- Partials       1795     1807      +12

Impacted Files
Coverage Δ
Complexity Δ

...e/executor/function/CreateSetFunctionExecutor.java
60% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ore/executor/function/MaximumFunctionExecutor.java
74.35% <ø> (ø)
0 <0> (ø)
:arrow_down:

...y/processor/stream/window/SortWindowProcessor.java
91.04% <ø> (ø)
0 <0> (ø)
:arrow_down:

...rocessor/stream/window/SessionWindowProcessor.java
66.41% <ø> (ø)
0 <0> (ø)
:arrow_down:

...i/core/util/parser/CollectionExpressionParser.java
39.73% <ø> (ø)
0 <0> (ø)
:arrow_down:

...cessor/stream/window/TimeBatchWindowProcessor.java
76.12% <ø> (-0.31%)
0 <0> (ø)

...hi/core/aggregation/BaseIncrementalValueStore.java
62.83% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ava/io/siddhi/core/util/parser/OperatorParser.java
51.51% <ø> (ø)
0 <0> (ø)
:arrow_down:

.../io/siddhi/core/util/parser/InputStreamParser.java
72.72% <ø> (-5.85%)
0 <0> (ø)

...core/src/main/java/io/siddhi/core/table/Table.java
27.54% <ø> (ø)
0 <0> (ø)
:arrow_down:

... and 119 more

Continue to review full report at Codecov.

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


Fix literal_is, issue #369

Do not error for equal literals.
Better error message for not equal literal values, and not matching types.
Add literal_inference testcases

The behaviour of rejecting comparisons where both sides are literals is deliberate. See my last comment on issue #369 for an explanation why.
This patch would completely fail for literal expressions, ie expressions containing literals whose type has not yet been determined. For example:
(1 + 2) is 3

If you want to improve the error messages and add unit tests for the intended behaviour (which we really do need), then please submit a new pull request.


Refactoring WindowBackdrop and removing undocumented APIs

The theme switch for new windows is still not working (if this was aimed to be fixed in this PR)

With these changes the application does not work well in dark theme.


CI: Remove main branch trigger

Change-type: patch
Signed-off-by: Josh Bowling<EMAIL_ADDRESS>

@balena-ci I self certify!


Update Docs with Release 1.18

This PR adds the release 1.18 docs to main.

Fixed merged conflicts with recent commit.


Use proper XDG configuration folders instead of just HOME

 # BEGIN: Upsun certificate configuration
Host *.platform.sh *.upsun.com
  Include /Users/shyim/.upsun-cli/ssh/*.config
Host *
# END: Upsun certificate configuratio

The complete .upsun-cli folder should be at ~/Library/Application\ Support/upsun-cli
See https://pkg.go.dev/os#UserConfigDir

@shyim thanks for reporting. We could definitely do something like this, but given that we currently have a rather complex setup with our Legacy CLI and new functionality working side-by-side here, we've decided not to touch on configuration directories for now.
If you'd like, you can still choose your configuration directory with the following environment variable:
export UPSUN_CLI_APPLICATION_USER_CONFIG_DIR=Application\ Support/upsun-cli

If you need this at all times, you can add this to your .zshrc/.bashrc files as well.

Maybe leave this issue open with a label so it won't get lost. I hope the legacy cli will be removed 🙈


spell check - en-US

What is it?

[ x ] enhancement
[ x ] Docs

Description
No logic change whatsoever.  Spell-checked 90% of the files.
Use cases and why
I needed to familiarise myself with the code, and while at it, I decided to clean up a bit as well.
Checklist:

[ x ] My code follows the developer guidelines of this project
[ x ] I have performed a self-review of my own code

Very useful PR!🤩

To harmonize the documentation, I did set my Mac/VsCode to en-US and performed a complete (almost) walkthrough of the code, be it .rs, .html, .css, .ts, .tsx, or anything else.  I skipped the compiled files, such as .md and the .js files.
All attempts were made to not make any logic change.


Not loading, Fitbit Sense

I don’t know why, but after switching to this clock it worked fine on the Fitbit Sense for a few weeks.  (Heart rate stalling aside, see other issue log.)
Then out of the blue I got this:

I tried the classic uninstall and reinstall.  Changing the clock face from the app.  But that didn’t fix it.  I changed the clock face from the watch itself and it has sprung back to life.
Hope that helps.

I witnessed this the other day when a friend tried to install the watch face. It never happened in testing so I need to see if I can replicate this behavior. One question- is your watch connected to an iPhone or Android device?

https://github.com/aidanblack/Fitbit-ClassicClock/commit/f588c2b1df6cd01e9abe0d749ecf4edd5e25722b commit will try to address this. Waiting for approval in the Fitbit Gallery.
After the update is approved, please close this issue if the bug is fixed.

I have the Fitbit Sense connected to an iPad Mini 4 (iOS) rather than an iPhone.

The update has been approved. Select the watch face in the Fitbit app to download the update. I will leave these two issues open for now - let me know if the issue appears to be fixed. Thanks for the feedback!

Installed.  I will test it for a few days and report back.  Thanks so much for looking into this.


Hello, I have a network of questions to ask you, may I add your Facebook, thank you

Hello, I have a network question to ask you, may I add your Facebook, thank you. I have several days in this routing issue, may I ask for help next, thank you.
My Facebook Account is<EMAIL_ADDRESS>I hope you can help us next thanks

@CristianVladescu
I hope you can help me thanks


Migrate nebari slurm docs

Reference Issues or PRs
This PR moves docs from the nebari-slurm repo: https://github.com/nebari-dev/nebari-slurm/tree/main/docs
to this repo.

What does this implement/fix?
Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[x] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[x] Did you test the pull request locally?
[ ] Did you add new tests?

Documentation

Access-centered content checklist
Text styling

[ ] The content is written with plain language (where relevant).
[ ] If there are headers, they use the proper header tags (with only one level-one header: H1 or # in markdown).
[ ] All links describe where they link to (for example, check the Nebari website).
[ ] This content adheres to the Nebari style guides.

Non-text content

[ ] All content is represented as text (for example, images need alt text, and videos need captions or descriptive transcripts).
[ ] If there are emojis, there are not more than three in a row.
[ ] Don't use flashing GIFs or videos.
[ ] If the content were to be read as plain text, it still makes sense, and no information is missing.

Any other comments?

Thanks for this PR, Amit.
Vini and I took a look at it today, and we'll call this PR a "migration" of docs so that we can merge and update it iteratively. For instance, the docs still mention QHub-HPC, and we'll need to update those mentions to begin with.


TypeError: Cannot read property 'near' of undefined

In v1.1.0 I'm seeing this error, but I don't see this error in v1.0.4 or v1.0.2. The specific area I'm seeing this is when using a coroutine, so creed.coroutine(generator)(val).then(x => x) is enough to trigger it for me.

Thanks @Akkuma, I'll look into it.

It seems rollup or buble, not sure which, is incorrectly compiling the source.  Here is the line that is blowing up.  If you look at the line just before it and the line just after it, you can see the problem.  The function expression on the line before it gets named to then$$1 by rollup/buble (I can tell it's rollup/buble since the there is no function expression in the original source).  Then on the following line, rollup/buble incorrectly transforms the call to then, which is a function imported here, to then$$1, which causes the now-in-scope function expression named then$$1 to be called instead of the imported function.  Since then$$1 is called as a function, it's this will be undefined, causing the error you are seeing.
Rollup and buble have been very reliable, so this is quite surprising.  Unfortunately, though, this means the entire 1.1.0 build is broken.
Pinging @Rich-Harris to see if he has any advice--perhaps I'm doing something I shouldn't be.  Otherwise, I may have to carefully rename all the functions whose names overlap with method names.

@Akkuma See #107 for a workaround that I'm planning to release as 1.1.1.  If you have a chance, I'd love to know if that branch works for you.  I used your example to test it (thanks for the example!), and it worked for me, so I'm hopeful.

I just released 1.1.1, which is exactly 1.1.0 built with the known-working rollup+buble configuration from 1.0.4.  It should fix the issue, but please reopen if you still encounter a problem.  Thank again for reporting this!


fix(build): add centos version 7.8.2003

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area cmd

/area pkg

/area docs

/area tests

What this PR does / why we need it:
In order to support CentOS with kernel 3.10.0-1127 we need support for CentOS Linux release 7.8.2003 (Core). This PR adds it. This will make files like test-infra's centos_3.10.0-1127.el7.x86_64_1.yaml build successfully.
Which issue(s) this PR fixes:

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

fix(build): add support for CentOS Linux release 7.8.2003

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: LucaGuerra
To complete the pull request process, please assign leodido after the PR has been reviewed.
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 589f2b8227c017d276d7718c4547114fedeb5ee7

👉  https://github.com/falcosecurity/driverkit/pull/117 , but I'm not sure if we already need it 🤔


Crash after send message to Server in ServiceConnection project

I used these code to connect with the server which is an sample project named "BlinkyService", And I establish an client project, to talk with it via winrtcpp.
this is my code:
void MainWindow::on_WriteBtn_clicked()
{
hstring  requestedPinValue;
	AppServiceConnection connection;
	connection.AppServiceName(L"BlinkyService");
	connection.PackageFamilyName(L"BlinkyService-uwp_gpek5j0d8wyr0");
	IAsyncOperation<AppServiceConnectionStatus> operation =  connection.OpenAsync();
	operation.Completed([&](auto const & sender, AsyncStatus) {
		ui->listWidget->addItem("service connected!");
		AppServiceConnectionStatus status = sender.GetResults();
		if (status != AppServiceConnectionStatus::Success)
		{
			ui->listWidget->addItem("connect failed!");
			return;
		}
		ui->listWidget->addItem("connect success!");
		/*Send a message with the name "requestedPinValue" and the value "High"
		These work like loosely typed input parameters to a method*/
		ValueSet message;
		hstring key = L"requestedPinValue";
		Windows::IInspectable value = PropertyValue::CreateString(L"High");
		message.Insert(key, value);

		IAsyncOperation<AppServiceResponse> resOperation = connection.SendMessageAsync(message);
		//resOperation.Completed([&](const auto & sender2, AsyncStatus) {
		//	AppServiceResponse response = sender2.GetResults();
		//	if (response.Status() == AppServiceResponseStatus::Success)
		//	{
		//		ui->listWidget->addItem("Send message Success!");
		//		//timer = ThreadPoolTimer.CreatePeriodicTimer(this.Tick, TimeSpan.FromMilliseconds(500));
		//	}
		//	else
		//	{
		//		ui->listWidget->addItem("Send message failed!");
		//	}
		//
		//});

		//If the message was successful, start a timer to send alternating requestedPinValues to blink the LED

	});
}

if I delete the sendMessage code, it will print "connect success!", which means it has been connected to the server... but when I add the sendMessage code, it will crash . Do you know why ?Thank you!

I used
AppServiceResponse response = co_await connection.SendMessageAsync(message);
instead of
 IAsyncOperation<AppServiceResponse> resOperation = connection.SendMessageAsync(message);
Then it works.

The co_await expression will ensure that the thread context is restored so that you can update the UI widget from the UI thread rather than some background thread where the Completed handler is called.


Decouple deployment of Northern Australia Map from NationalMap

This is done using the separate branches of NA and NM and running the usual npm run deploy in each directory, following the normal release process. Releases built in this way have been given to GA. Once they're ready to fully transition to this approach, we'll merge the separate branches into master and close this issue.


bar plot value labels are wrong when yOffset is used

Thank you Terbaco for mentioning this over email.
This bug should be fixed, but also it would be nice if users could provide an array of manual plot value labels. Perhaps if this array is null the values will be used, but if it is not the labels will be used.

Interesting, this one isn't "wrong" so much as it's the wrong choice for the default. For example, this could be a worthwhile addition (excuse the poor photoshop, even by my standards):

I could see this being useful for waterfall plots especially, which were added in June #476, however, the default certainly should be to show the value, not the growth/loss.

Probably the best long-term solution would be to let the user define what the labels by passing in a string[]... I'll add it to #412
For now I changed the default behavior based on the principle of least astonishment which I think is good enough to mark this issue resolved 😎


Azure API Management - Subscription - Update - stateComment not working

Describe the bug
As described here https://docs.microsoft.com/en-us/rest/api/apimanagement/2019-01-01/subscription/update it should be possible to use and update stateComment.
We can update all the other Fields but not stateComment - how ever the return code is always 204 which would mean it should have been set, but it's not.
To Reproduce

Go to 'https://docs.microsoft.com/en-us/rest/api/apimanagement/2019-01-01/subscription/update'
Login with Account with permission to create/update subscriptions and click on 'try it'
Fill resourceGroupName, serviceName and sid with valid values which you got by get before
as Body use:
{
"properties": {
"displayName": "testdisplayname",
"stateComment": "testcomment"
}
}
Click run
You should get a response 204 and see that displayName was set but stateComment will still be null

Expected behavior
The stateComment should have the value "testcomment"
Desktop (please complete the following information):

OS: Win Server 2019
Browser Edge
Version [ 81.0.389.2]

Hi @juergstaub , @czullu ,
it's by design. StateComment is applied only when the state is changed to "rejected" by the administrator. Otherwise stateComment field is just ignored. We will update the documentation with these details soon.


testing issue 977838

I've opened an internal bug for the warning
.config is not supported languaging name, alias or extension for parsing code snippet with tag name, you can use line numbers instead when resolving "[!codeC_HowTo-WCFServiceAndASMXClient#1]"
I wanna make sure it's fixed.

@tompratt-AQ this is how they told me to fix this the issue with .config extension snippets.
@rpetrusha @BillWagner do you think it's ok to identify our snippets using config files as XML? Engineering team said they cannot parse generic code references because the content format of the config file are not always known (XML, JSON, etc.)
It certainly beats missing code snippets like we've had since our last migration. 😢

Yes, it’s certainly OK to identify config files as XML. In fact, where I’ve been putting config file contents inline, that’s how I’ve been identifying them.
From: Maira Wenzel<EMAIL_ADDRESS>Sent: Tuesday, April 25, 2017 12:51 AM
To: dotnet/docs
Cc: Ron Petrusha; Mention
Subject: Re: [dotnet/docs] testing issue 977838 (#2002)
@tompratt-AQhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftompratt-AQ&data=02|01|ronpet%40microsoft.com|8e3547038d0b4a152c6f08d48bafd80c|72f988bf86f141af91ab2d7cd011db47|1|0|636287034738372115&sdata=QCYdPRLa8xe0ffS0cBjYeB2m1jY%2FqVWC1J5Ob3UpYfE%3D&reserved=0 this is how they told me to fix this the issue with .config extension snippets.
@rpetrushahttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frpetrusha&data=02|01|ronpet%40microsoft.com|8e3547038d0b4a152c6f08d48bafd80c|72f988bf86f141af91ab2d7cd011db47|1|0|636287034738372115&sdata=PB9lcS%2BDdGilJB3c9Vsa0LTZcXD5TWOVpZm8H%2BR789A%3D&reserved=0 @BillWagnerhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FBillWagner&data=02|01|ronpet%40microsoft.com|8e3547038d0b4a152c6f08d48bafd80c|72f988bf86f141af91ab2d7cd011db47|1|0|636287034738382128&sdata=3IzxKTmKOgptqdtCn9qimziWZvv2e4oHu2TWQl9ZQ0U%3D&reserved=0 do you think it's ok to identify our snippets using config files as XML? Engineering team said they cannot parse generic code references because the content format of the config file are not always known (XML, JSON, etc.)
It certainly beats missing code snippets like we've had since our last migration. 😢
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs%2Fpull%2F2002%23issuecomment-296948027&data=02|01|ronpet%40microsoft.com|8e3547038d0b4a152c6f08d48bafd80c|72f988bf86f141af91ab2d7cd011db47|1|0|636287034738382128&sdata=7wqeyhR0S9yQGwSGz8UlcIPBQLv4ZQrHAT3TCJL7KP8%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKYfMcqsn4x6CqnJH13JdwSBXTn8PUQoks5rzaXvgaJpZM4NG4L4&data=02|01|ronpet%40microsoft.com|8e3547038d0b4a152c6f08d48bafd80c|72f988bf86f141af91ab2d7cd011db47|1|0|636287034738382128&sdata=Xl41v3oUgwCfQgwvGxQcyrTDI7paqVLrDqEwHJh9%2FGY%3D&reserved=0.


Backup and restore etcd

High-level description
This PR rebases https://github.com/dcos/dcos/pull/6957
This PR introduces a utility dcos-etcdctl on /opt/mesosphere that can be used to import/export the etcd state.
Corresponding DC/OS tickets (required)

D2IQ-66110 https://jira.d2iq.com/browse/D2IQ-66110.

Related tickets (optional)

D2IQ-58912 include etcd in backup and diagnostic bundles.

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain.
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot changelog-not-required  Part of add etcd into DC/OS. (DCOS-59004)


Is it possible to config an api authorizer of type aws_iam?

In the serverless framework, I can specify an authorizer of type aws_iam like this:
create:
handler: create.main
events:
- http:
path: notes
method: post
cors: true
authorizer: aws_iam
Is it possible to do this in SAM yaml? I only see support for custom authorizers or or cognito user pool authorizers.

Related issues and requests for this feature:
https://github.com/awslabs/serverless-application-model/issues/512#issuecomment-420324817
https://github.com/awslabs/serverless-application-model/issues/248#issuecomment-393660831


Optional URL param for userName

In reference to my question #115 - This is how I was able to achieve what I wanted.
This will enable an optional param to be passed to the URL to fill in the UserName field.
Ex:
<EMAIL_ADDRESS>

Thank you, let me check the PR.

@geoperez  you're welcome!

@mariodivece thoughts?

@Jrsnow8921 just one thing, can you explain this feature? Maybe inside "Customization and Configuration" section

@geoperez  explanation added to the "Customization and Configuration"  section of README.md

Thank you!

You're welcome! @geoperez


Create a "Paper-ified" version of the plugin (alongside Spigot's)

This I feel is a good example plugin for showing how to work with the Chat Preview API.
Still maintains Spigot compat ofc, just adds the Paper stuff as more of an example and might allow for cooler Paper stuff in the future 🤷🏻

Please adjust to the changes made in https://github.com/treecaptcha/uwu/commit/0e7c81da5707e2f0e5962e9b9ce984fc90b7a519


App risk status screen: misleading wording

In the recently published (German language) screenshots one can see that in case of status "green" the app tells the user "no risky encounters so far" ("bisher keine Risiko-Begegnungen").
This is an unnecessarily imprecise wording which could be misleading and providing a false sense of safety to anyone who is not too familiar with the mechanism of the tracing system (i.e. probably to most users as well as most of the press and politics). Even if the app achieved an enormous penetration rate and if the bluetooth mechanism turned out 100.0% reliable the app would still not be in a position to make such an assertion (due to inevitable testing and reporting delay, untested asymptomatic cases who never report, etc).
I strongly suggest to use a more precise wording such as "no risky encounters detected so far" ("bislang keine Risiko-Begegnungen festgestellt/ermittelt"), which leaves less room for misinterpretation and false impressions of the capability of the tracing system. To some, it might look like a small, insignificant difference in wording, but it could help reduce misunderstandings and false assessments of the personal and public safety situation. Please make sure to be as precise as possible.

Internal Tracking ID: EXPOSUREAPP-2067

it's also been 14 days since i raised this related issue here: https://github.com/corona-warn-app/cwa-wishlist/issues/110
no one appears to have looked at it

I would like to add to this topic:
At least in the German version of the Android App (now v1.3.0), after 14 days the app risk status screen switches from "1 Risiko-Begegnung" to "Bisher keine Risiko-Begegnungen". The "Bisher" (=so far) is actually wrong, since there was one in the past, even if it is not relevant any more.
I'd strongly suggest to change the wording (in the idea of this posting) to "Keine Risiko-Begegnungen festgestellt". Just drop the "Bisher". You could add "in den letzten 14 Tagen" (= in the last 14 days), but this could also be misleading.
The current form is bad UX.

@geos-github
This is quite an old wishlist item and there have been many iterations to the wording and layout of the risk tiles in the app since May 2020. In German the risk tile now says "Keine Risiko-Begegnungen" if that is the case (no more "bisher" in the wording).
In the meantime, screenshots are available on the website https://www.coronawarn.app/de/screenshots/ for reference. This is the CWA 2.23 Android screenshot in German:

https://www.coronawarn.app/de/screenshots/#android_cards goes directly to the German risk card screenshots. There are links to iOS and to English-language screenshots also available on the website.
Do you consider this wishlist item now fulfilled? If yes, then you could close it.

well, the initial point I have been trying to make has not yet been taken into consideration. To reiterate: when the app claims "no risk encounters" users are likely to take that for what it says by assuming that they are safe since there actually were no risk encounters (must be true since the app says so...). Still the app cannot make such a claim. Only because the app was not able to detect any such risk encounters is by no means reason to the claim that there had not been any. That was and still is my point. For someone not too tech-/statistics-savvy, this phase provides an incorrect level of assurance, it  constitutes a claim beyond what the app is actually able to assess.

@geos-github
Thanks for your feedback! You are right that the risk tile information is incomplete. I think it is acceptable for the overview to say only "Keine Risiko-Begegnungen" however the detailed tiles reached through the right arrow don't contain any disclaimer about what is not covered. For example the app can't calculate a risk encounter with a person which occurs after they have reported positive to the app. Also if an infected person is not using any supported Corona app or not carrying a smartphone then you may have a risk encounter, but the app doesn't know about it.
To get an explanation you need to tap on "Risiko-Ermittlung aktiv" where you are given a full explanation of how the risk calculation works.
I think in practice users have not been misled by the text in the app because they have understood through other information sources how it works. Nevertheless the texts in the risk tiles could be improved to be more comprehensive.