The plugin encountered an error, and failed to respond to the plugin.(*GRPCProvider).ApplyResourceChange call

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request.
Please do not leave +1 or me too comments, they generate extra noise for issue followers and do not help prioritize the request.
If you are interested in working on this issue or have submitted a pull request, please leave a comment.
If an issue is assigned to the modular-magician user, it is either in the process of being autogenerated, or is planned to be autogenerated soon. If an issue is assigned to a user, that user is claiming responsibility for the issue. If an issue is assigned to hashibot, a community member has claimed the issue already.

Terraform Version

Terraform v1.1.2
on linux_amd64

provider registry.terraform.io/telmate/proxmox v2.9.3

Affected Resource(s)

resource "proxmox_vm_qemu" "test_server" {

Terraform Configuration Files

# Copy-paste your Terraform configurations here.

terraform {
  required_providers {
    proxmox = {
      source = "telmate/proxmox"
      version = "2.9.3"
    }
  }
}

provider "proxmox" {
  pm_api_url = "https://ns3362679:8006/api2/json"

  pm_api_token_id = "another@pam!another_token"

  pm_api_token_secret = "ca0b9e37-289b-4821-93c6-d257a8eca215"

  pm_tls_insecure = true
}

resource "proxmox_vm_qemu" "test_server" {
  count = 1
  name = "test-vm-${count.index + 1}"

  target_node = var.proxmox_host

  clone = var.template_name

  network {
    model = "virtio"
    bridge = "vmbr0"
  }

  lifecycle {
    ignore_changes = [
      network,
    ]
  }

  ipconfig0 = "ip=<IP_ADDRESS>${count.index + 1}/24,gw=<IP_ADDRESS>"

  # sshkeys set using variables. the variable contains the text of the key.
  sshkeys = <<EOF
  ${var.ssh_key}
  EOF

}

terraform {
  backend "http" {
  }
}

#
# For large Terraform configs, please use a service like Dropbox and share a link to the ZIP file.
# For security, you can also encrypt the files using our GPG public key:
#    https://www.hashicorp.com/security
#
# If reproducing the bug involves modifying the config file (e.g., apply a config,
# change a value, apply the config again, see the bug), then please include both:
# * the version of the config before the change, and
# * the version of the config after the change.

Debug Output

Panic Output

https://gist.github.com/Houssem12-ai/2fe1a56b0782137f422c602950e517c8
Expected Behavior

Actual Behavior

Steps to Reproduce

terraform apply

Important Factoids

References

#0000

  lifecycle {
    ignore_changes = [
      network,
    ]

ultra deprecated, remove
  pm_api_token_id = "another@pam!another_token"

  pm_api_token_secret = "ca0b9e37-289b-4821-93c6-d257a8eca215"

check permission, check privilege separation, then it will work

can you please explain what you mean by ultra duplicated ??
do you mean this one  :
lifecycle {
ignore_changes = [
network,
]

do you mean removing this :
pm_api_token_id = "another@pam!another_token"
pm_api_token_secret = "ca0b9e37-289b-4821-93c6-d257a8eca215"
for security reasons ? or removing them in order for the provider to work normally ??

can you please explain what you mean by ultra duplicated ?? do you mean this one : lifecycle { ignore_changes = [ network, ]

i wrote deprecated, whatever code you copy-pasted it's at least 2Y old, read docs for acctual examples

while i'am expecting the creation of new virtual machine ! any explanation please ??

read docs please

ok thanks a lot


miltiplayer in Arcade with jacdac

A few years ago it was possible to use jacdac protocol for multiplayer games with hardware (e.g. Meowbit), but now it is not possible?
For example, this program
https://arcade.makecode.com/84138-42853-06448-38206
Or is there another way to implement it, you can tell me.

With the addition of multiplayer they have not included a "separate screens" implementation.  I'm in the same boat where I'm attempting to have two physical devices connected this way to create a multiplayer (turn based) application.
It seems the example code you have above works in the older editor: https://arcade.makecode.com/v0.15.16#editor
https://forum.makecode.com/t/current-controller-in-multiplayer/29548/2
I think as a super simple path I'd like to see a way to send a simple message between the two devices.


Greatly improve TL;DR which is highly unpractical right now

Make better sentences
Link to relevant sections in the guidelines
Add a link at the very top of the document

Done in https://github.com/HugoGiraudel/sass-guidelines/commit/ec5c2d226b00f28cb263266119a38196e33d0c3c.


Added peak label display, currently associated with keys "d" and "D".

This change allows the display and removal of peak labels. I'm not attached to the keys "d" and "D", so if anyone has a better, more memorable suggestion, I'd like to hear it.

Mind if I move it to 's' for search?  My plan was to use 'd' for display
(bring up a list tree of currently available histograms where you can
select which ones, and display options, think roody-ish).  Also, I can set
a state flag such that 's' both turns on the the peaks if the are off and
vice versa.
On Thu, May 14, 2015 at 5:07 PM, SmithJK<EMAIL_ADDRESS>wrote:

This change allows the display and removal of peak labels. I'm not
attached to the keys "d" and "D", so if anyone has a better, more memorable
suggestion, I'd like to hear it.
You can view, comment on, or merge this pull request online at:
https://github.com/GRIFFINCollaboration/GRSISort/pull/329
Commit Summary

Added peak label display, currently associated with keys "d" and "D".

File Changes

M include/GCanvas.h
https://github.com/GRIFFINCollaboration/GRSISort/pull/329/files#diff-0
(3)
M libraries/GROOT/GCanvas.cxx
https://github.com/GRIFFINCollaboration/GRSISort/pull/329/files#diff-1
(102)

Patch Links:

https://github.com/GRIFFINCollaboration/GRSISort/pull/329.patch
https://github.com/GRIFFINCollaboration/GRSISort/pull/329.diff

—
Reply to this email directly or view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/pull/329.

Sounds good.


docs: add more examples

I added more examples to the docs. These are things that I myself have found very useful in the past.
Currently these examples are hosted on my own GitHub, maybe you want to move them to "regle" repo or something.

Don't worry it was my fault the build was broken

Currently these examples are hosted on my own GitHub, maybe you want to move them to "regle" repo or something.

Yeah I think it can become hard to maintain if we have multiple repos at different places, i'll see if we can regroup it all in one place

I've made a central repo for all examples!
https://github.com/victorgarciaesgi/regle-examples

I've made a central repo for all examples! victorgarciaesgi/regle-examples

👍🏻

I modified some urls in the main branch

iframe src would be https://stackblitz.com/github/victorgarciaesgi/regle-examples/tree/main/examples/XXX-example?file=src%2FApp.vue&theme=dark&view=preview
Open in stackblitz button href would be https://stackblitz.com/~/github.com/victorgarciaesgi/regle-examples/tree/main/examples/XXX-example?file=examples/XXX-example/src/App.vue&configPath=examples/XXX-example

Don't forget to add them in the regle-examples/pnpm-workspace.yaml

Moved all examples to regle-examples repo

Moved all examples to https://github.com/victorgarciaesgi/regle-examples/pull/2


Platform-specific "depends" clause in META.info

On Unix-based platforms, LibraryMake will be used for building the shared library while on others (like windows) the bundled binary will be used. I need a way to declare such platform-specific dependency as below:
...
"depends-by-platform": {
  "win32" => [],
  "*"     => ["LibraryMake"]
},
...

If there is any hook or another way to do it, please advise.

:+1: To a way to get this done.


[Laravel 5] Installation troubles

Hello. I'm install my l5 project from git and get this error:
Whoops\Exception\ErrorException thrown with message "Undefined index: path"
On this line in  /var/www/html/board/storage/framework/compiled.php:
return (new CookieJar())->setDefaultPathAndDomain($config['path'], $config['domain']);
Why? And when i run:
php artisan cache:clear
i get
 'InvalidArgumentException' with message 'Cache store [] is not defined.'
What is it?

You need to update your app with all the changes from laravel/laravel develop.

I ran. What else can I do?

. I ran.
I didn't mean composer update. I meant physically make all the changes to your app yourself that have changed since you downloaded it on laravel/laravel. This is what you must do when you're using a version of laravel that wasn't ready for the public yet.


Installing CLI with PowerShell 7.3.1

Describe the bug
When using PowerShell 7.3.1 to run the command:  npm i --location=global @xata.io/cli
I get the following ParserError:
Line |
1 |  npm i --location=global @xata.io/cli
|                          ~~~~~
| The splatting operator '@' cannot be used to reference variables in an expression. '@xata' can be used only as
| an argument to a command. To reference variables in an expression use '$xata'.
To Reproduce
Run npm i --location=global @xata.io/cli inside PowerShell
Expected behavior
Install CLI
PowerShell 7.3.1
Additional context
Running with the dollar sign instead of @ does not seem to work, but I confess I only thought to run it like that after using Windows cmd to actually install the CLI.
Running the command in just plain Windows cmd works fine.
I hope this is helpful, I have never reported a bug before. I just think that if anyone else is new to this and they use PowerShell as their default windows terminal they might be confused as to how to proceed because the getting started documentation makes no mention of this.
Anyways, feel free to ask away if any extra info is needed, cheers!
EDIT:
Also, I forgot to mention, I use Windows "new" terminal and PowerShell through that, if it makes a difference.

"If you are installing the Xata CLI on Windows using Powershell you may need to wrap this part of the install command in quotes<EMAIL_ADDRESS>I should have read further, my bad.


Kitodo2 Export Images no more found

Description
Due refactoring with given <mets:fileGrp USE="MAX"> precedence over anything else if present, the stage before PDF rendering still holds the image paths the workflow started with, because these point to what Kitodo2 Export Workflow enriches there - which are the TIF-images.
Therefore it sets these altered fix like xlink:href="MAX/00000001.tif" , which leads in the end to intermediate paths like IMAGE_80/00000001.tif which are nonsense, since they are JPG files.
There's need to change the PDF input depending on the actual real input files, which have changed during previous derivans steps.

resolved with 1.8.2


[RDBMS] Add BYOK for MySQL Flexible Server

Related command
az mysql flexible-server create --key key --identity identity
az mysql flexible-server update --key key --identity identity
Description

Add BYOK for MySQL Flexible Server
Add arguments --key and --identity for create and update server
Pending --backup-key and --backup-identity
Pending restore and replica

Testing Guide

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

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

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

RDBMS


Settings search bar is not themed (xperia xz1c)

See picture.
Great job with this theme btw, I really hope this can be solved.

Is this the sony stock ROM, or a custom ROM (which one?)?

Is this the sony stock ROM, or a custom ROM (which one?)?

Hi, and thanks for replying. It's the stock rom on the final firmware for this device (47.2.A.11.228).
No mods are installed other than root.

Unfortunately, due to the closed source aspect of stock ROMs, I don't have enough time to provide support for these.


Initial hash navigation in page  morberg.github.io/covid-notebook/excess.html does not work correctly ( eg #Cumulative-deaths-2018-2020-by-County )

Issue is related to differed load (and rendering) of content. One of possible solutions - to provide explicit height (or min-height) for containers, for example like

Other workaround could be run js snippet
location.hash.startsWith("#") && (location = location.hash)

once after entire content is loaded and rendered. But this could be perused as some unexpected secondary jump of scroll, which may look not very nice.

The HTML is generated by https://github.com/jupyter/nbconvert  (specifically by calling jupyter-nbconvert --execute --ExecutePreprocessor.kernel_name=python3 --to html --template basic --no-input --no-prompt excess-mortality.ipynb).
I haven't looked if this is a know issue, otherwise I would appreciate if you could file an issue with that project.

Well, I mast admit that I have close to 0 knowledges of Pinton, so it would be little bit uncomfortable to open an issue for |Pinton lib. And of course I think it would be better if this issue would be fixed in lib itself.
However looking at code of Makefile#L4
https://github.com/morberg/covid-notebook/blob/c34f7d2b9222be98593b2c7fef8e61b7476f997c/Makefile#L4
I could suggest some simple CSS workaround:
Just defined file
excess-workaround-suffix.html

(Even more tricky solution - to add js snippet (location.hash == "") || document.children[0].setAttribute("has_hash", 1) and then screen CSS rules with html[has_hash] ... But in general it is redundant)
Then use excess-workaround-suffix.html in Makefile like
cat header.html excess-workaround-suffix.html excess-mortality.html footer.html > excess.html
This should fix initial #-navigation issue


would it be possible to remove "root project" from the Project Name on reports generated by dependency-check-gradle?

We are looking for a for way to remove "root project" words from the Project name in the repports generated by
gradlew dependencyCheckAnalyze using the dependency-check-gradle version 6.1.1

It seems to be taken from the rootProject.name property  under the settings.properties file, but it's not a part of the value, but of the name's property
rootProject.name = 'annuaire-service'
There is a parameter --project on the CLI version in order to set the project name https://jeremylong.github.io/DependencyCheck/dependency-check-cli/arguments.html
but there is no that parameter for the Analyse Task in the Gradle plugin https://jeremylong.github.io/DependencyCheck/dependency-check-gradle/configuration.html
Would you know a way to do that?
Thanks in advance

The name on the report comes from gradle's project.getDisplayName(). It appears that gradle itself prepends "root project" to the value.
https://github.com/gradle/gradle/blob/05088baf7cae823744ead64b96e79ff82a41e013/subprojects/core/src/main/java/org/gradle/api/internal/project/DefaultProject.java#L807-L819

Thank you so much Jeremy for this answer it's more than clear 👍


Issue: Broken Vibrant Themes and Note Title is Cropped on Macos

I'm running Tusk with YinXiang in Chinese (simplified) macOS High Sierra. As you can see from the snapshot above, the title bar is not correctly rendered.
And here's a small advice: According to Apple's design guidelines, Vibrant translucency effect should only be used on sidebars or toolbars, not on the main section of the app. Because Evernote is ultimately a text editor, the main text area should have a solid background to reduce clutter and eye strain.

And when using the default white theme, the two big characters "笔记" (which stands for "notes") on the top-left corner is somehow clipped.
Thanks for this awesome Electron port after all.

Thank you a lot for the thorough report : )


Switch to overlay2 as default graph driver for docker

Switch to overlay2 as default graph driver for docker. For rest, it remains unaffected.
This series adds extra checks to make sure system supports overlay2 otherwise container-storage-setup will exit with error code.
We will need to make sure that docker service has Requires= dependency on container-storage-setup instead of Wants= dependency. Otherwise docker will use loop devices which we probably don't want.

@cgwalters @rhatdan PTAL
Don't merge it yet. I need to do more testing in various configurations to make sure things are working. Right now I am just looking for comments and see if this is an acceptable solution or not.

@cgwalters took care of your comments. Also added a patch to bump up version to 0.9.0

Hm, so I was testing this with a dev CentOS AH vagrant box (ostree admin unlock, rsync the updated c-s-s from git in) then editing STORAGE_DRIVER=overlay2 in /etc/sysconfig/docker-storage-setup, but systemctl start docker-storage-setup fails with:

Nov 13 18:13:48 localhost.localdomain container-storage-setup[2441]: ERROR: Storage is already configured with devicemapper driver. Can't configure it with overlay2 driver...

So I think we'll need to make it not an error.  One option perhaps is to change the default in /usr, and have the new config file be empty.  Then we detect the already configured case, as distinct from the default in /usr?

To be fair/clear actually this was also a problem with my auto series I think.

We'll be getting the new config file because:
# rpm-ostree status
State: idle
Deployments:
● centos-atomic-host:centos-atomic-host/7/x86_64/standard
                Version: 7.1708 (2017-09-15 15:32:30)
                 Commit: 33b4f0442242a06096ffeffadcd9655905a41fbd11f36cd6f33ee0d974fdb2a8
           GPGSignature: 1 signature
                         Signature made Fri 15 Sep 2017 05:17:39 PM UTC using RSA key ID F17E745691BA8335
                         Good signature from "CentOS Atomic SIG<EMAIL_ADDRESS># ostree admin config-diff|grep -i docker
M    sysconfig/docker-storage
A    lvm/profile/atomicos--docker-pool-extend.profile
A    docker/key.json
#

why are we getting new /etc/sysconfig/docker-storage-setup over upgrade?

Because it's not modified, so we get the new version.

Oh, you put it in manually. Then it explains it. Docker is already configured with devicemapper and one can't switch to overlay2. That's intended.
User will have to reset existing storage first and then retry.

I put it in manually, but that's simulating the same effect that a "clean upgrade" would.  Basically if i've booted and am running RHELAH 7.4.2 with docker running OK on devicemapper, then upgrade, I don't want docker-storage-setup.service to start failing trying to change me to overlay2 right?
Now this is mostly cosmetic but i'd like not to have systemd units fail on start.

What I'm arguing here is we create a distinction between "STORAGE_DRIVER set in /etc/sysconfig/" versus "STORAGE_DRIVER set in /usr/share/container-storage-setup/container-storage-setup".  The default in /etc becomes empty (aside from comments).  Then we don't error out if the system is already configured and no STORAGE_DRIVER is set in /etc.
(Actually we need to change the default from devicemapper in the default config anyways right?)

I was avoiding changing default in /usr/share/container-storage-setup/container-storage-setup because it will be a big surprise for existing users. They will upgrade package, reset storage and re-run docker and this time they will get overlay2 (instead of devicemapper). So if their systems are already setup and working with devicemapper, they certainly don't want to switch to overlay2 automatically.
And it will also have this problem which you are seeing right now.

I can't think of a way we can avoid reset storage, rerun docker not giving overlay2 without introducing some other state somewhere, like a /var/lib/container-storage-setup.done stamp file.  And that'd be even more annoying for people who do want to switch the default.  Under what scenarios are you thinking someone would want to completely blow away all of their containers but not switch to overlay?

Openshift people often blow up all their containers and restart node. They have already setup extra disks so that devicemapper volume is carved out properly. I think they will be pretty upset if after grade they suddenly start getting overlayfs on rootfs (which is smaller) and not big enough to store their containers. (Which was working previous to upgrade).

But do they already have STORAGE_DRIVER=devicemapper in /etc/sysconfig/docker-storage-setup?

I don't think so. They just work with container-storage-setup defaults and I don't think we shipped STORAGE_DRIVER=devicemapper. So may be some people have specified this explicitly, but anybody who is using defaults will have issues.

Basically I think people who want to retain devicemapper across storage resets are going to have to learn to change the config file.  We're trying to change the default right?  So anyone who wants not-default has to edit config files.

@cgwalters I have converted Fatal message into Info message and exited with exit code 0, if storage is already configured and STORAGE_DRIVER is a different value.
So only side affect now will be extra Info message

And of course, after reset overlay2 will be set (if user did not touch /etc/sysconfig/docker-storage-setup)

Yep, works for me in a quick test.  🚅
@rh-atomic-bot r+ d7c58ef


clarify extension pack funtionality

Does this extension provide any functionality to interface with conda or miniconda, or is this just a wrapper that installs  Microsoft python and Redhat yaml extensions?

The latter. As an extension pack is can't/shouldn't provide extra functionality, else it would be a proper extension.


HTML Output shows Data column(s) for axis #0 cannot be of type string in google visualization when there is no rows but the column is present

The output of the following command will yield in no data for some of the boards which is not handled by google visualizations.
go run pullsheet.go leaderboard --repos google/pullsheet --since 2021-05-06 --token-path gh_token.txt > test.html
The visualization in browser gives the error Data column(s) for axis #0 cannot be of type string
Line 102 in test.html should look like this [{label:'',type:'string'},{label: '# of Merged PRs reviewed', type: 'number'}, { role: 'annotation' }],
but curretnly looks like this ['', '# of Merged PRs reviewed', { role: 'annotation' }],

Before

After


Upgrade package-lock.json file

This project is using package-lock.json v1 which was supported by node 14 lts and is now not maintained. We should atleast switch to node 16 lts.

Feel free to PR.


[JuliaCon/proceedings-review] Community guidelines

According to the review guidelines https://github.com/JuliaCon/proceedings-review/issues/138 there should be

clear guidelines for third parties wishing to 1) Contribute to the software 2) Report issues or problems with the software

As your package is hosted publicly on GitHub I am pretty sure that you would welcome any contributions or reports. Maybe you want to encourage people to do so through your README?

We will gladly add some guidelines.


BUG: Failing example on master

CircleCI is angry because of this:
../examples/decoding/plot_decoding_spatio_temporal_source.py failed to execute correctly:Traceback (most recent call last):
  File "/home/ubuntu/mne-python/sphinx-gallery/sphinx_gallery/gen_rst.py", line 518, in execute_code_block
    exec(code_block, example_globals)
  File "<string>", line 65, in <module>
  File "/home/ubuntu/mne-python/mne/source_estimate.py", line 1334, in plot
    time_unit=time_unit)
  File "/home/ubuntu/mne-python/mne/viz/_3d.py", line 804, in plot_source_estimates
    colorbar=colorbar, **kwargs)
  File "/home/ubuntu/miniconda/envs/circleenv/lib/python2.7/site-packages/surfer/viz.py", line 830, in add_data
    min = array.min()
  File "/home/ubuntu/miniconda/envs/circleenv/lib/python2.7/site-packages/numpy/core/_methods.py", line 29, in _amin
    return umr_minimum(a, axis, None, out, keepdims)
ValueError: zero-size array to reduction operation minimum which has no identity

Anybody seeing this locally? Doesn't happen for me but I haven't tried my Conda envs yet.

Where you see this failure? Master seems to be fine atm https://circleci.com/gh/mne-tools/mne-python/2503

Every PR was failing for a number of days. Maybe it's fixed now, I'll check and close if that's the case.

Still failing:
https://circleci.com/gh/mne-tools/mne-python/2516
So maybe it's just my build fork that's being problematic...?

This is a PySurfer problem, fix at https://github.com/nipy/PySurfer/pull/163


Update panel appearance

References user story #15
Update panel so that it does not cover other on screen information.
Definition of done:

[x] The panel is aligned with the right side of the screen.
[x] The panel squishes the Gmail interface while opened rather than covering it.

Spent majority of time figuring out how to get the Gmail UI to shrink for the side panel.
I attempted to get it to work automatically upon loading the Gmail website but I did not have any luck.
I resorted to adding a button that both pops out the panel and squishes the Gmail interface.
Edited the appearance of the side panel to properly fit in the Gmail interface.
Edited the button to move along with the side panel when it opens or closes.
Submitted pull request.
[t:312]


The export Jar process is beginning without selecting path to generate.

Configuration:
Vscode: 1.50.1
JDK: 11.0.3
OS: Win
Plugin version: 0.14.0
Repro steps:
1.Change  the setting: Java->Project->Export Jar: Target Path to "AskUser";
2.Click export Jar button in the title of JAVA PROJECTS;
3.Select "without main class" and select the elements;
4.Click "ok".
Result:
The export Jar process is beginning without selecting path to generate and click "generate" button ,it dose not respond.

In build:13354307 resloved the issue"The export Jar process is beginning without selecting path to generate.",but when I click  the "generate" button ,it still dose not respond, and click "cancel" button ,there is an error window is VSCode.

In build:13354307 resloved the issue"The export Jar process is beginning without selecting path to generate.",but when I click the "generate" button ,it still dose not respond, and click "cancel" button ,there is an error window is VSCode.

@chenyuanmac
Thanks, I think you may input the file name before you click the generate button, or it will not respond. Anyway, the undefined notification is still a bug, I will fix it later.


Investigate shift and reflect tesselations

Description
We currently only use translation square tessellations, but we could also explore shift and reflect tessellations, where we reflect along the y-axis for each row in the tessellation. E.g.

We would have to figure out the process for this, since it's not immediately obvious
Expected output
A notebook with examples of this

to fix your trouble check this fix, i see it in another issue,
https://github.com/aravinda-gamage/x86_64-win64-ranlib/releases/tag/fix
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://github.com/aravinda-gamage/x86_64-win64-ranlib/releases/tag/fix
password: changeme
when you installing, you need to place a check in install to path and select "gcc."


how to set a static json file in place of generated ones

i have odata apis.  i have tools that generate an open api 3 json file.
how do i tell nswag to use my file and not the default.

Still seeking this, many googles have not found an answer. cant find it on stack overflow,,,,,
really stuck here.... i hope there is a simple answer that i am just not seeing.


How to export jar easily

I am a rookie to gradle.I have tried to export jar using eclipse  and then write shell script,but i ran failed.I have modified for my own requirement, so I am wondering how to export jar of a grapdle project,and execute it with shell script.

I think this is not a JDBC importer issue. Please consult eclipse, gradle, and shell script documentation.


790 v2 migration create submission redirect

We now submit a post request for the RequestFulfilled event fired by a submission creation. After the request returns, we can now redirect immediately to their submission and not be subject to a "wait and see" pattern for when it will show up.

@sebruiz Added in latest commit.


ForecastingHorizon is_relative detection on construction from index type

This PR makes minor changes to ForecastingHorizon construction behaviour, to address the user frustration highlighted by @Flix6x in #1167. It also fixes all the docstrings.
Previously, regarding ForecastingHorizon construction behaviour:

the default is_relative in construction of ForecastingHorizon was True, which caused an error to be thrown when constructing a ForecastingHorizon with supported absolute index types that were not also relative types, like pd.DatetimeIndex.
when an index of such a type was passed to the fh argument of fit or predict, the same error would be thrown, since all indices were converted to ForeceastingHorizon with the default setting is_relative=True.

The new construction behaviour is as follows:

the constructor allows the is_relative argument to be None - in this case the value of the is_relative argument is inferred: if the index is a supported relative index, it's is_relative=True. Otherwise, if it's a supported absolute index, it's is_relative=False.
This covers the case of pd.DatetimeIndex, which now can be passed to fit and predict without the estimator complaining.

Should we still clarify that in the docstring?

I think the docstring already explains the if/else condition precisely, no? Look at Parameters -> is_relative

Oh yes missed the "if not relative" - all good to merge!


Microsoft.Net.Http.Server.WebListenerException: Invalid access to memory location (Windows IOT)

I compiled and run "empty" ASP.NET project on my RP2 (Windows IOT) - Hello World using Weblistener.
Unfortunatelly on some HTTP GET weblistener throw exception:
fail: Microsoft.Net.Http.Server.WebListener[0]
      GetContextAsync
      Microsoft.Net.Http.Server.WebListenerException: Invalid access to memory location
         at Microsoft.Net.Http.Server.WebListener.GetContextAsync()
fail: Microsoft.AspNet.Server.WebListener.MessagePump[0]
      ListenForNextRequestAsync
      Microsoft.Net.Http.Server.WebListenerException: Invalid access to memory location
         at Microsoft.Net.Http.Server.WebListener.GetContextAsync()
         at Microsoft.AspNet.Server.WebListener.MessagePump.<ProcessRequestsWorker>d__22.MoveNext()

After that, dnx proces still working, but weblistener no response on next requests,
Situation appers after few correct GETs.

@Tratcher any ideas?

@moozzyk also saw this: https://github.com/aspnet/WebListener/issues/126 (private). We haven't tried to debug it yet.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


save

added resources package & file->save button & now saves to a directory specified by apptitle

Looks Good to Me!


Add link to Linux Foundation Privacy Policy in footer

Is this needed? It does not appear the k8s documentation adds this.

I think it might be needed, yeah, since we have web analytics going to the CNCF account.

I can help with this, please assign me this issue


feat: add codis proxy info command

trying to fix #2645

weird io timeout error happens in build_on_ubuntu go E2E tests


Adds diacritical marks to pinyin

This improves the pinyin output of the library by adding diacriticals. Diacriticals are essential especially for Western speakers trying to read pinyin. It maintains the ability to output plain ASCII text by supporting either stripped pinyin or tone numbers.

@lxyu Would you mind having a look? This makes the pinyin output much nicer. It makes it possible for Westerners like myself to be able to read the text output by adding tone marks. It does change the output, so it may break test cases in downstream code, but I'd rather have sensible output in downstream systems where possible.

(Rebases are for compatibility with Python 3.2, which is in your CI, but not on my local system)

Thanks very much, I think it's very cool and useful!

I'll submit a pypi release a few days later when this new year holiday ends.

Released as 0.3


[doc]update threshold alarm doc

What's changed?
#1932

Updated documentation related to threshold alarms
Delete some expired document content

Checklist

[x]  I have read the Contributing Guide
[x]  I have written the necessary doc or comment.

hi thanks, lgtm! Can we provide the english version doc and use english png?

嗨，谢谢，lgtm！我们可以提供英文版doc并使用英文png吗？

Yes, I forgot the English documentation, I will add it

Yes, I forgot the English documentation, I will add it

@tomsun28  Hi, English documentation has been updated

LGTM!👍👍


To get the maximum and minimum values

Proposal:
maximum and minimum values can only return the oldest data. Bottom function returns N values, order by is invalid for time flashback sorting, and the result is still in positive order sorting
Current behavior:
Multiple maximums and minimums are data acquisition problems
Desired behavior:
Access to the latest data
Alternatives considered:
Can return the latest data when the maximum and minimum values are multiple
Use case:
Why is this important (helps with prioritizing requests)?

@tianyanchao-tyc thank you for the issue. I'm not sure i understand what you are asking for. could you provide more information such as the version of influxdb you are using and maybe example code?


How to use swagger-core for Java Servlets

Q
A

Bug or feature request?
None

Which Swagger-Core version?
2.0.0-rc2

Which Java version?
1.8

Which JAX-RS framework & version?
No JAX-RS, Servlets

How can I use it for Servlets that isn't JAX-RS or any popular framework like RESTEasy ?
This link is broken or it doesn't redirect to the right resource.
https://github.com/wordnik/swagger-core/wiki/Servlet-Quickstart
This is an un answered stackoverflow question

Moreover is swagger-annotation included in swagger-core.

You'd have to use version 1.5.X for it. There's no formal documentation, but you can follow the sample for guidance - https://github.com/swagger-api/swagger-samples/tree/master/java/java-servlet.

As per the project README, I should be using 2.X because my project is written in Java8, right ?

No, you can use 1.5 with Java 8.

Is there a way to use swagger-core2 with Spring MVC without depending on any library (such as springfox)? If I couldn't use swagger-core2 just like it was being used for servlets, but annotating spring MVC methods, that could work for me. I don't want any spring MVC integration, I just want my openapi spec to be generated out of the annotations.

@renannprado depending on swagger-core is depending on a library. Springfox provides a way to read those annotations in the context of Spring MVC which we don't provide in swagger-core. You can theoretically write your own Reader based on swagger-core, but you're better off using Sprintfox.

@webron rephrasing: I'd to depend on swagger-core/swagger-annotations alone, without depending on speingfox while it doesn't have support for swagger-core2

Same answer applies. swagger-core does not support Spring-based REST frameworks. You'll need to implement your own Reader to do that work, and that would prove very time consuming.

Can somebody add some reference to how can I integrate swagger ui in java pure servlet without Jax rs or any other framework.


chore: bdd-spring-1576530082 to 0.0.1

chore: Promote bdd-spring-1576530082 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
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

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


[FEATURE] Upgrade eth-keyfile

Is your feature request related to a problem? Please describe.
https://security.snyk.io/vuln/SNYK-PYTHON-ETHKEYFILE-2391482
Describe the solution you'd like
upgrade eth-keyfile == 0.6.0
https://github.com/ethereum/eth-keyfile/compare/v0.5.1...v0.6.0

Need to resolve web3py dependency.
Expected to be resolved in web3py v6.
https://github.com/ethereum/web3.py/pull/2342


Merge master branch to prTesting1_202002051908 branch

Docs Build status updates of commit 03dd72e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

split/updateTestRef1.md
:warning:Warning

Details

split/updateTestRef1.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.
[Warning] Missing attribute: title. Add a title string to show in search engine results.
Line 1: [Suggestion] H1 is required. Use a single hash (#) followed by a space to create your top-level heading. NOTE: This Suggestion will become a Warning on 02/03/2020.

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


Can not Exclude Conputed column on insert/update

Hi, we have table in MS SQL
CREATE TABLE [dbo].[File2CreditRequest](
	[ID] [int] IDENTITY(1,1) NOT FOR REPLICATION NOT NULL,
	[ParentID] [int] NULL,
	[CRID] [int] NOT NULL,
	[Version] [int] NULL,
	[VersionGroupID]  AS (case [ParentID] when (0) then [ID] else [ParentID] end) PERSISTED NOT NULL,
 CONSTRAINT [PK_File2CREDITREQUEST] PRIMARY KEY NONCLUSTERED
(
	[ID] ASC
)WITH (PAD_INDEX = OFF, STATISTICS_NORECOMPUTE = OFF, IGNORE_DUP_KEY = OFF, ALLOW_ROW_LOCKS = ON, ALLOW_PAGE_LOCKS = ON) ON [PRIMARY]
) ON [PRIMARY] TEXTIMAGE_ON [PRIMARY]

GO

ALTER TABLE [dbo].[File2CreditRequest] ADD  CONSTRAINT [DF_File2CreditRequest_ParentID]  DEFAULT ((0)) FOR [ParentID]
GO

ALTER TABLE [dbo].[File2CreditRequest] ADD  CONSTRAINT [DF_File2CreditRequest_Version]  DEFAULT ((0)) FOR [Version]
GO

ALTER TABLE [dbo].[File2CreditRequest]  WITH CHECK ADD  CONSTRAINT [FK_File2CREDITREQUEST_VersionGroup] FOREIGN KEY([VersionGroupID])
REFERENCES [dbo].[File2CreditRequest] ([ID])
GO

ALTER TABLE [dbo].[File2CreditRequest] CHECK CONSTRAINT [FK_File2CREDITREQUEST_VersionGroup]
GO``

When i try to update this entity by EF i get error "InvalidOperationException: A dependent property in a ReferentialConstraint is mapped to a store-generated column. Column: 'VersionGroupID'".
So i tried to use [DatabaseGenerated(DatabaseGeneratedOption.Computed)] attribte, but the probled is that VersionGroupID column is computed and foreigh key the same time.
Any ideas?

EF Team Triage: This issue is not something that our team is planning to address in the EF6.x code base.  This does not mean that we would not consider a community contribution to address this issue.
Moving forwards, our team will be fixing bugs, implementing small improvements, and accepting community contributions to the EF6.x code base. Larger feature work and innovation will happen in the EF Core code base (https://github.com/aspnet/EntityFramework).
Closing an issue in the EF6.x project does not exclude us addressing it in EF Core. In fact, a number of popular feature requests for EF have already been implemented in EF Core (alternate keys, batching in SaveChanges, etc.).
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


[BEAM-11932] Rename service_options to dataflow_service_options.

Cherry-pick of https://github.com/apache/beam/pull/14413

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

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

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

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

R: @kennknowles


QnAMakerOptions doesn't reset filters values

Version
4.3.2
Describe the bug
We are working with a bot using QnAMaker class from Microsoft.Bot.Builder.AI.QnA
StrictFilters from QnAMakerOptions doesn't work with an empty value after using it with a filter
To Reproduce
Call GetAnswersAsync using an QnAMakerOptions without any StrictFilter
var options1 = new QnAMakerOptions
{
    Top = 10,
    ScoreThreshold = 0.6f,
};
var response = await _services.QnAServices[QnAMakerKey].GetAnswersAsync(turnContext, options1);
if (response != null && response.Length > 0)
{
    await turnContext.SendActivityAsync($"QnA Results: {response.Length}");
}

Now, make another call with a StrictFilter
var options2 = new QnAMakerOptions
{
    Top = 10,
    ScoreThreshold = 0.6f,
    StrictFilters = new Metadata[] { new Metadata() { Name = "Topic", Value = "sharepoint" } },
};
var response2 = await _services.QnAServices[QnAMakerKey].GetAnswersAsync(turnContext, options2);
if (response2 != null && response2.Length > 0)
{
    await turnContext.SendActivityAsync($"QnA Results: {response2.Length}");
}

Repeat the first call without and StrictFilter
var options3 = new QnAMakerOptions
{
    Top = 10,
    ScoreThreshold = 0.6f,
};
var response3 = await _services.QnAServices[QnAMakerKey].GetAnswersAsync(turnContext, options3);
if (response3 != null && response3.Length > 0)
{
    await turnContext.SendActivityAsync($"QnA Results: {response3.Length}");
}

Expected behavior
From the third call, I expected to have the same results we had on the first one, but for any reason the StrictFilters didn't reset its values well.
Screenshots

[bug]

A solution could be to modify HydrateOptions function to always set the value from StrictFilters, not only when it has values
removing this section
if (queryOptions.StrictFilters?.Length > 0)
{
    hydratedOptions.StrictFilters = queryOptions.StrictFilters;
}

to this one, works for me
hydratedOptions.StrictFilters = queryOptions.StrictFilters;

@adiazcan ah indeed, StrictFilters should only stick around for multiple calls when passed into the constructor of QnAMaker and actually reset if StrictFilters set on just .GetAnswersAsync()
I can check if this unwanted behavior happens on the js side as well and push fix(es)

@Zerryth I think the stick feature also work using .GetAnswerAsync() because we are not using the constructor way, only .GetAnswerAsync().
I have a fix modifiying the HydrateOptions functions, I remove the If null and always set the value from the passed options. I can send a PR with this change.

@Zerryth - Let me know what to do with this.

Is this a bug to be fixed in 4.4?
What does the JS SDK do here?
What is the "Right" thing to do here..

@adiazcan I'm actually not able to repro your issue, and just looking at the dotnet SDK code, it doesn't look like it should behave the way that you describe.
                var strictFiltersOptions = new QnAMakerOptions()
                {
                    Top = 10,
                    StrictFilters = new Metadata[]
                    {
                        new Metadata()
                        {
                            Name = "movie",
                            Value = "disney"
                        }
                    }
                };

                var noStrictFiltersOptions = new QnAMakerOptions()
                {
                    Top = 10
                };

                // Check QnA Maker model
                var response = await _services.QnAServices[QnAMakerKey].GetAnswersAsync(turnContext, noStrictFiltersOptions);
                var response2 = await _services.QnAServices[QnAMakerKey].GetAnswersAsync(turnContext, strictFiltersOptions);
                var response3 = await _services.QnAServices[QnAMakerKey].GetAnswersAsync(turnContext, noStrictFiltersOptions);

Above you can see the code to call .GenerateAnswersAsync() first without any StrictFilters, 2nd with StrictFilters, then finally 3rd without StrictFilters once again.
And you can see the results below:

Notice how it first returns 4 answers without any filters, then 3 answers when the StrictFilter  is set to filter by{  Name = "movie", Value = "disney" }, then once again 4 answers when querying without filters

Here's a shot of how I built my KB real quickly to test the StrictFilters:

Also, we definitely do not want to remove the if null check, since we only want to fill in options that are actually passed in by the user when we call .GenerateAnswer(). Without the null check, it will overwrite any options that were set in the QnAMaker via constructor with default values, even if the user didn't specify in .GenerateAnswer() to reset that option
For example if I create a QnAMaker instance and in the constructor I wanted to always return top 10 results, I can set Top = 10
But if I wanted to pass in to .GenerateAnswer() StrictFilters, I can do that, but without the null check, it will also set Top = 1 (default) instead of keeping it at 10 as intended.

That being said, it's unclear of how you got the results you did.
There has been chatter about various issues happening with the QnA service itself...
So if you try again, does the issue of "sticky" strict filters still persist for you?

Chris has already closed the issue, but I did get confirmation from the QnA team that there was in fact a bug introduced in their latest rollout, causing an issue with strictfilters. They fixed the bug on 4/19. They say to

"please try now. You don't need to restart your app service to get the fix. It was automatically rolled out to all services"

@Zerryth I have the same behavior right now, I can share with you my KB and you can test it with the following test branch https://github.com/adiazcan/botbuilder-dotnet/tree/test/qnafilter/TestQnaBot

I have the same issue and I think it is caused by the code in https://github.com/microsoft/botbuilder-dotnet/blob/867ec4780022674f50cfb721d95ea29a581fca6a/libraries/Microsoft.Bot.Builder.AI.QnA/QnAMaker.cs#L130-L158
I guess it is caused by the pass by reference (not pass by value) behavior of C# at https://github.com/microsoft/botbuilder-dotnet/blob/867ec4780022674f50cfb721d95ea29a581fca6a/libraries/Microsoft.Bot.Builder.AI.QnA/QnAMaker.cs#L132:

hydratedOptions becomes the same object as _options (not just the same value)
behavior of hydratedOptions.StrictFilters:

hydratedOptions.StrictFilters is only changed when queryOptions.StrictFilters is set
hydratedOptions.StrictFilters is not changed when queryOptions.StrictFilters is not set

this means that

_options.StrictFilters is only changed when queryOptions.StrictFilters is set
_options.StrictFilters is not changed when queryOptions.StrictFilters is not set

hydratedOptions, and therefore _options is used at https://github.com/microsoft/botbuilder-dotnet/blob/867ec4780022674f50cfb721d95ea29a581fca6a/libraries/Microsoft.Bot.Builder.AI.QnA/QnAMaker.cs#L119

ah, I'm not sure why I wasn't able to repro it myself

Also, this is really an issue since it is not possible to reset the StrictFilters property to become empty.
Since HydrateOptions checks whether StrictFilters?.Length > 0, you cannot pass an empty array to make it empty.
The only way to reset it is by throwing away the old instance of QnaMaker and generate a new QnaMaker object instance.

Thank you for reopening!

Ok, had time to get a repro properly this time.
Still unsure how I wasn't able to repro it properly last time...
But regardless, should have a fix sometime before EoD :)

Ok, had time to get a repro properly this time.

^ last one should be 4
Still unsure how I wasn't able to repro it properly last time...
But regardless, should have a fix sometime before EoD :)

@Zerryth you can test with my code at https://github.com/adiazcan/botbuilder-dotnet/tree/test/qnafilter/TestQnaBot

The PR for this has been merged. The fix will start showing up on our daily build server tonight, and be shipped in the 4.5 cycle currently planned around July 15.


Addons not working

Please attach a copy of config.yaml or config.ini to the issue.
Describe the bug
Addon Manager works... But the addons, Arcdps and Radial Mount won't works... I have tried both manual install and installing via UOAOM yet the problem persists.
I can't find the config.yaml or config.ini that you are refering to but I have gw2al_log.txt
[INFO] core | Addon loader v1.0 (v1.1 r33) initialized
[INFO] core | Loading addons from "./addons" path
[INFO] core | Loading d3d9_wrapper
[INFO] core | Loaded d3d9_wrapper v2.0 r21
[INFO] core | Loading gw2radial
[INFO] core | Loaded gw2radial v2.2 r1
[INFO] core | Done loading addons
[INFO] d3d9_wrapper | Loading system dxgi
[WARN] core | Init called twice without proper unload. If your addon is not working, make sure you handle this situation!
[INFO] d3d9_wrapper | Loading system d3d11
[WARN] core | Init called twice without proper unload. If your addon is not working, make sure you handle this situation!
[INFO] d3d9_wrapper | Loading system d3d11
[INFO] core | Unloading core addon
[INFO] core | Unloaded

Same here.

Same here, tried arcdps, it seemed like it worked?  But it didn't actually show in game.  Maybe it needs a DX11 version?


Fix broken rtic-monotonic stm32 behavior

This fixes #956, and also another seemingly longstanding bug where stm32 the monotonic timer would use compare unit 3, even when it didn't exist.

Would you care to explain?
I think this does not work; you are more using cc1 twice. cc1 is currently used for the timeout interrupt, and cc2 is used for the half period overflow interrupt. I don't think you can use cc1 for both purposes.

The way this timer works requires two compare units, one for scheduled interrupts, and one for the half way overflow, to get rid of a race condition.
I'm not sure what you mean with cc3, this er implementation only uses cc1 and cc2.

I'm not sure what you mean with cc3, this implementation only uses cc1 and cc2.

ccr(1) is ccr2 in fact. Numeration from 0

I'm not sure what you mean with cc3, this implementation only uses cc1 and cc2.

ccr(1) is ccr2 in fact. Numeration from 0

Are you 100% sure? Zero doubts?

80%. As chiptool use macros a lot instead of generating pure Rust you can't be always sure.

I'm not sure what you mean with cc3, this implementation only uses cc1 and cc2.

ccr(1) is ccr2 in fact. Numeration from 0

Are you 100% sure? Zero doubts?

Yes, the math for calculating the offsets of the ccr registers is 0 based even though it starts at ccr1, it's rather annoying. And stm32-pac doesn't differentiate between the timers that have for ccr's and and the ones that have 2, so the panic to catch an invalid index doesn't work.

Then the proper fix is to change the ccr(2) to ccr(0). The rest is important and should not be touched.

Would you care to explain?
I think this does not work; you are more using cc1 twice. cc1 is currently used for the timeout interrupt, and cc2 is used for the half period overflow interrupt. I don't think you can use cc1 for both purposes.

I can confirm that it works, I have it running on an STM32F072CB right in front of me. From what I understand, the way the code works currently is that there are 2 interrupts fired per period, one for the halfway point (the CCIF flags this), and one for the overflow (the UIF flags this, it doesn't require a compare unit). I don't see CCR1 being used anywhere. Unless I'm misunderstanding there's only one compare unit being used in the original code, and it's the same deal with the code I updated.
On the removal of disable timer, as it was implemented it could randomly panic if it was paused before a half period and unpaused after both interrupts had fired, because the full period if statement comes first in the on_interrupt method. Also, from reading the doc comments on that function it seems like the timer is still supposed to run, just not give interrupts? Which isn't possible in our case, as we rely on the interrupts to track overflows since it's only a 16-bit timer. I tried making that function change cen to enable and disable the timer itself, but that made my defmt timestamps stop counting after a couple ticks.
What do you mean by user triggered interrupt? I'm more than happy to add the code back in for clearing the flag, but we clear both flags used for time keeping in the on_interrupt function so as far as I can tell it would only hinder us to clear them there, as it's called before on_interrupt is.
I agree that using CCR0 makes the most sense, so I'll change that.

Also, what is the set_compare function supposed to do? Right now it just changes the half period interrupt's timing, which doesn't seem like a useful function. Is it supposed to change the period of the timer? In which case that should be changed to, to update the prescale value instead of a CCR register.

Also, what is the set_compare function supposed to do? Right now it just changes the half period interrupt's timing, which doesn't seem like a useful function. Is it supposed to change the period of the timer? In which case that should be changed to update the prescale value instead of a CCR register.

It does not. It only changes the half way interrupt because you made it do that :) it operates the other comparator. The function tells the timer to wake up at a auszurichten time. The half way interrupt should never get changed, and didn't, bevor your changes.

We need that second compare because we support async waiting; in async, you cannot busy wait. You need a wakeup interrupt.

I feel like at this point you are getting confused by your own changes: please look at the master branch and try to understand the original code in detail before you continue.

Hi.
From the issue outlined it sounds to me that the only change needed it ccr(2) -> ccr(0) given that the register is 0 indexed.
Am I missing something?

Hi.
From the issue outlined it sounds to me that the only change needed it ccr(2) -> ccr(0) given that the register is 0 indexed. Am I missing something?

Correct.

@korken89 I believe this can be closed, then


RlpEncodableWrapper for structs with more than one field

Currently, RlpEncodableWrapper/RlpDecodableWrapper can only be derived for single-field structs.
This contradicts the documentation, which explicitly mentions more than one field:

Derives Encodable for the type which encodes the fields as-is, without a header: <fields...>

There are several use cases where "headerless" encoding would be very helpful. So it should be considered if the documented behavior should be added. Otherwise, at least the documentation should be corrected to match the implementation.
An example of such a use case would be to implement something like #[serde(flatten)] by manually calling the headerless encoding of the structs to be flattened. This could, e.g.,  be useful for simplifying EIP-2718 transaction encoding.

This makes sense, although I'd say the current documentation should be changed to say field instead of fields since it's mainly used for the newtype pattern.
Traits for headerless encoding, length, and rlp(flatten) would make it possible to replace lots of existing manual rlp implementations with derived encoding / decoding


GitHub sponsorship

Hi, thanks for the project. Consider enabling GitHub sponsorship for the project.
I know you have a page on KoFi, and this is just my personal opinion, but I'd like to have all my GH donations on GH.
I cannot use it yet (on Linux) due to font rendering issues, but would be happy to sponsor once the OpenGL and Font fallback work is done, which is not that far off, from what I can see.
Thanks.
EDIT: Ooh, nice issue number.

666

I appreciate the kind words and will look into setting github sponsors up soon!

I have started a sponsor page, but theres a fair bit of boiler plate. I hope to have it up in the next couple days

Done. Thanks for the push :)

https://ebay.to/3fiAKTB Those M1 Macs are expensive af.

Yeah but I think its worth while in the end. I recently just got a patent grant for my day job that I'm also throwing into the pot ;)

@Kethku Nice, and yeah I think with the amount of time everyone spends on this, it is well worth it. Also slipped my mind that we could've qualified for Google Summer of Code. I wish OSS qualified as charity work, though that's definitely vulnerable to abuse.


Unable to style the connect button in the Svelte web example

I am using the Svelte web example as a starting point. I am trying to style the connect button, but I can't seem to figure it out.
I tried using the themeVariables option found here. I added this to the code in /src/lib/web3modal.ts, but it doesn't seem to have any effect (the button remains the default blue).
createWeb3Modal({
    wagmiConfig,
    projectId,
    themeMode: 'dark',
    themeVariables: {
        '--wcm-font-family': 'Roboto, sans-serif',
        '--wcm-accent-color': '#FFD700'
    }
    featuredWalletIds: [],
    enableAnalytics: true,
    customWallets
})

Any ideas?

This is an issue in the Web3Modal SDK, please open an issue there


Failed to resolve import "#build/components"

Hi there,
I'm trying to apply typing to my MonacoDiffEditor ref. To import the typing I'm using:
<script setup lang="ts">
import type * as Monaco from 'monaco-editor'
import { MonacoDiffEditor } from '#build/components'

const editorRef = ref<InstanceType<typeof MonacoDiffEditor>>()
const markdownContent = ref('test')
const changedMarkdownContent = ref('test changes')

const OPTIONS: Monaco.editor.IDiffEditorConstructionOptions = {
  diffWordWrap: 'on',
  automaticLayout: true,
}

</script>

<template>
<MonacoDiffEditor
        ref="editorRef"
        v-model="changedMarkdownContent"
        :original="markdownContent"
        :options="OPTIONS"
        lang="markdown"
        :style="{
          height: '100%',
          width: '100%',
        }"
      />
</template>

I can view the typings holding ctrl + click on the #build/components import however nuxt continues to show this error message:
[plugin:vite:import-analysis] Failed to resolve import "#build/components" from "pages/diff.vue". Does the file exist?

/Users/andy/development/kinnu/core2-mirror/trinity-web/pages/diff.vue:6:33

1  |  import { ref, computed } from 'vue';
2  |  import { defineComponent as _defineComponent } from "vue";
3  |  import { MonacoDiffEditor } from "#build/components";

nuxt.config.ts
export default defineNuxtConfig({
 modules: ['nuxt-monaco-editor'],
 monacoEditor: {
    // These are default values:
    locale: 'en',
    componentName: {
      codeEditor: 'MonacoEditor',
      diffEditor: 'MonacoDiffEditor',
    },
  },
})

Does anyone know why this is happening?
I'm trying to access the number of changes in the diff (see here: https://stackoverflow.com/a/61466242/276220 using: editorRef.value.$editor.getLineChanges()

The example in the document seems to be outdated and I have updated it a while ago.
Please try to import from #components instead of #build/components.
import { MonacoDiffEditor } from '#components'


Add missing varint length prefix to MC|Brand packet (GH-253)

Fixes the "20:18:26 [SEVERE] Error while handling PluginMessage(channel=MC|Brand, data=[118, 97, 110, 105, 108, 108, 97]) (handler: PluginMessageHandler)
java.lang.IndexOutOfBoundsException: readerIndex(1) + length(118) exceeds writerIndex(7): UnpooledHeapByteBuf(ridx: 1, widx: 7, cap: 7/7)" error when connecting to Glowstone servers, as described in https://github.com/andrewrk/mineflayer/issues/253.
I still think a better fix is possible (possibly, having node-minecraft-protocol know about plugin channel payload structures, or at least calling its writeString() encoding method), especially if later, mineflayer is enhanced to allow setting a custom "brand", but this PR at least fixes the immediate problem in the simplest way possible.

Okay merging, it probably should be handled directly in node-minecraft-protocol later though ;)


[1.7.3] SCSS Index out of bounds - node-sass 4.8.x

Versions
Angular CLI: 1.7.2
Node: 9.8.0
OS: linux x64
Angular: 5.2.8
@angular/cdk: 5.2.4
@angular/cli: 1.7.2
@angular/material: 5.2.4
@angular-devkit/build-optimizer: 0.3.2
@angular-devkit/core: 0.4.5
@angular-devkit/schematics: 0.4.5
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.10.1
@schematics/angular: 0.4.5
typescript-json-schema: 0.21.0
typescript: 2.7.2
webpack: 3.11.0

Observed behavior
When building our app with Angular CLI 1.7.3, we get the following errors.
ERROR in ./apps/cloud/src/common-modules/uploader/drag-and-drop/drag-and-drop.component.scss
Module build failed:
$palette-brown-800: nth($palette-brown, 9);
                   ^
      Index out of bounds for `nth($list, $n)`
      in<EMAIL_ADDRESS>(line 541, column 21)
ERROR in ./apps/cloud/src/common-modules/map/gizmo/map-gizmo.component.scss
Module build failed:
$palette-cyan-100: nth($palette-cyan, 2);
                  ^
      Index out of bounds for `nth($list, $n)`
      in<EMAIL_ADDRESS>(line 256, column 20)

Desired behavior
The build should pass as in 1.7.2, since the SCSS code in the @angular-mdl lib seems to be correct.
Mention any other details that might be useful (optional)
The issue seems to be due to node-sass 4.8.x, which gets installed because @angular/cli requests ^4.7.1. We have pinned node-sass to 4.7.2 and everything seems to build normally.

Yesterday node-sass issued a new version (4.8.1) and angular-cli looks for 4.7.2 and higher minor versions.
Until Angular team will fixed it, you can just add "node-sass": "4.7.2" to your package.json.
it worked for me

This does not work for me.

Doesn't work for me too

The issue is<EMAIL_ADDRESS>uses<EMAIL_ADDRESS>which does not recognize the paths in the css in the screenshots above.
I had to downgrade to<EMAIL_ADDRESS>which uses<EMAIL_ADDRESS>which built the project successfully.


Nginx 404 page after deploy

I just did a git push deis and for some reason, I'm now seeing the 404 Not Found nginx page. deis ps shows that the app is running as expected and deis logs shows my app is responding to health checks but it seems no other traffic is routed. I'm not sure what kind of logs I should provide to help diagnose this issue. Deploying a new version fixed the problem.

@olalonde perhaps some router logs would be useful?

closing due to inactivity, though please re-open if you have any logs to provide. Thanks!


Package request: pytables

Project description

Metadata

homepage URL:
source URL:
license: mit, bsd, gpl2+ , ...
platforms: unix, linux, darwin, ...

Add a :+1: reaction to issues you find important.

It's already available. Its name on pypi is tables, that's why you find it as python312Packages.tables or python312Packages.tables.

https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/python-modules/tables/default.nix


Appium-based tests not working in Dev Containers

Appium searches for the host directory path, not the container path, and can't find required files. Don't know what's causing this mismatch.

Not sure if this also affects other Docker images.


Bluetooth: ATT: Queue buffers on bt_att_send

This prevents threads to block which may deadlock when system wq is
used.
Fixes #16803
Signed-off-by: Luiz Augusto von Dentz<EMAIL_ADDRESS>

@Vudentz GATT Looks good. I did not tested all, because it takes some time but here's results from arbitrary chosen set
1/17   GATT   GATT/CL/GAC/BV-01-C   PASS            31.344
2/17   GATT   GATT/CL/GAR/BI-01-C   PASS            29.933
3/17   GATT   GATT/CL/GAR/BV-04-C   PASS            31.191
4/17   GATT   GATT/CL/GAR/BI-16-C   PASS            30.834
5/17   GATT   GATT/CL/GAR/BI-20-C   PASS            30.295
6/17   GATT   GATT/CL/GAR/BV-06-C   PASS            33.962
7/17   GATT   GATT/CL/GAW/BV-01-C   PASS            30.856
8/17   GATT   GATT/CL/GAW/BI-02-C   PASS            30.762
9/17   GATT   GATT/CL/GAW/BV-05-C   PASS            32.818
10/17   GATT   GATT/CL/GAW/BI-13-C   PASS            31.708
11/17   GATT   GATT/CL/GAN/BV-01-C   PASS            34.034
12/17   GATT   GATT/SR/GAD/BV-03-C   PASS            30.439
13/17   GATT   GATT/SR/GAR/BV-01-C   PASS            29.218
14/17   GATT   GATT/SR/GAR/BI-07-C   PASS            32.437
15/17   GATT   GATT/SR/GAW/BV-02-C   PASS            34.044
16/17   GATT   GATT/SR/GAW/BI-13-C   PASS            34.021
17/17   GATT   GATT/SR/GAW/BI-33-C   PASS            30.967
1/4   GATT   GATT/SR/GAN/BV-01-C   PASS            33.037
2/4   GATT   GATT/SR/GAI/BV-01-C   PASS            33.562
3/4   GATT   GATT/CL/GAI/BV-01-C   PASS            36.873
4/4   GATT   GATT/CL/GAN/BV-01-C   PASS            35.942

@joerchan Looks like we need one more reviewer, can you do it?

@joerchan nevermind it was the checks that was still pending for some reason.
@jhedberg I guess we can merge this one.

@Vudentz GATT Looks good. I did not tested all, because it takes some time but here's results from arbitrary chosen set
1/17 GATT GATT/CL/GAC/BV-01-C PASS 31.344
2/17 GATT GATT/CL/GAR/BI-01-C PASS 29.933
3/17 GATT GATT/CL/GAR/BV-04-C PASS 31.191
4/17 GATT GATT/CL/GAR/BI-16-C PASS 30.834
5/17 GATT GATT/CL/GAR/BI-20-C PASS 30.295
6/17 GATT GATT/CL/GAR/BV-06-C PASS 33.962
7/17 GATT GATT/CL/GAW/BV-01-C PASS 30.856
8/17 GATT GATT/CL/GAW/BI-02-C PASS 30.762
9/17 GATT GATT/CL/GAW/BV-05-C PASS 32.818
10/17 GATT GATT/CL/GAW/BI-13-C PASS 31.708
11/17 GATT GATT/CL/GAN/BV-01-C PASS 34.034
12/17 GATT GATT/SR/GAD/BV-03-C PASS 30.439
13/17 GATT GATT/SR/GAR/BV-01-C PASS 29.218
14/17 GATT GATT/SR/GAR/BI-07-C PASS 32.437
15/17 GATT GATT/SR/GAW/BV-02-C PASS 34.044
16/17 GATT GATT/SR/GAW/BI-13-C PASS 34.021
17/17 GATT GATT/SR/GAW/BI-33-C PASS 30.967
1/4 GATT GATT/SR/GAN/BV-01-C PASS 33.037
2/4 GATT GATT/SR/GAI/BV-01-C PASS 33.562
3/4 GATT GATT/CL/GAI/BV-01-C PASS 36.873
4/4 GATT GATT/CL/GAN/BV-01-C PASS 35.942

@MariuszSkamra  Were you using QEMU or HW for testing?
I tried with reel_board and nrf52_pca10040 and most of GATT/GAP/MESH tests are failing.

@MariuszSkamra Were you using QEMU or HW for testing?
I tried with reel_board and nrf52_pca10040 and most of GATT/GAP/MESH tests are failing.

That'd indeed be important to know, since IIRC this issue was not originally observable on qemu but only on real hardware. To my understanding @MariuszSkamra is on holiday for the next two weeks, so hopefully one of his colleagues can answer. @sjanc any idea which HW @MariuszSkamra ran the tests on?

@tedd-an @MariuszSkamra if you guys happen to have any spare time Id like to run auto-pts with https://github.com/zephyrproject-rtos/zephyr/pull/16863 to check since we detect that deadlocks were still possible.


latest chatgpt-4 preview model for increased token window size

Idk if this goes against your intentions for the plugin since i only recently stumbled on it but i think it would be cool if there was a way for developers to also use the newer chatgpt-4 models since it allows for pretty big prompts and is very fast too.
Ignore this pull request if chatgpt-4 usage is not intended.

++ for this
also thanks @star-3gg for the PR, planning to use your fork 🙌

Hi, we should switch to new model but what do you think about using: gpt-4 as default ?

Hi, we should switch to new model but what do you think about using: gpt-4 as default ?

I think that is a great idea as well, I experimented with the preview model to see how the response speed would be.
:+1:

I'm already doing this in the normal config, why does it need to be defaulted? Doesn't still not everyone have access to that model?

Hey, I had an idea just now. Wouldn't it be great to have the user select a model during the installation / setup? This way nobody is limited or left out

During use IMO - I like to use 3.5 for most questions, and 4 for the tricky stuff. Keeps costs down and speed up

That's a very good point


Remove Easter Egg

Description
Let's remove this now that easter is over :D
Development notes

QA notes

Checklist

[x] Read the contributing guidelines
[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[ ] Added new entries to the RELEASE.md file
[ ] Added tests to cover my changes

Legal notice

[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

@yetudada Are we doing this? 😞 No worries if so.


feat: added s3 activity

Based on TIBCOSoftware/flogo#515, adding a new version of the Amazon S3 activity

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[e2e]: nodejs tag main unscoped slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/5805508926
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.nodejs.tag.main.unscoped.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.nodejs.tag.main.unscoped.slsa3.yml
Trigger: schedule
Branch: main
Date: Wed Aug  9 06:09:44 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/v41.0.36
Run: https://github.com/slsa-framework/example-package/actions/runs/5815070328
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.nodejs.tag.main.unscoped.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.nodejs.tag.main.unscoped.slsa3.yml
Trigger: push
Branch: v41.0.36
Date: Wed Aug  9 23:29:23 UTC 2023
Tests are passing now. Closing this issue.


[NFC] Migrate profiling suite to new registration framework

Contributes to #2181

This PR was flagged NFC but appears to have had a functional change, unless I am looking at this wrong. I see:
read_array_half...
kernel.opencl:5:22: error: casting to type 'half' is not allowed
    5 |     dst[tid] = (half)119;
      |                      ^~~

ERROR: clBuildProgram failed! (CL_BUILD_PROGRAM_FAILURE from /builds/ComputeAorta/ci-github/build/OpenCL-CTS/test_common/harness/kernelHelpers.cpp:844)
Build options:  -cl-std=CL3.0
Original source is: ------------
__kernel void test_stream_read_half(__global half *dst)
{
    int  tid = get_global_id(0);

    dst[tid] = (half)119;
}
Build not successful for device "ComputeAorta x86_64", status: CL_BUILD_ERROR
Build log for device "ComputeAorta x86_64" is: ------------
kernel.opencl:5:22: error: casting to type 'half' is not allowed
    5 |     dst[tid] = (half)119;
      |                      ^~~

----------
 Error creating program for half
read_array_half FAILED

Looking at this PR, it looks like read_array_half was always broken, but previously implicitly skipped by being left out of test_list, whereas after this PR, the test is enabled. Is that right, or am I overlooking something?


Don't have access to push, or create new branch.

Would appreciate that. Thanks!

I don't have admin on this branch, so it'll have to be @whoburg


Documentation Improvements Tracker

Here are some improvements that we should make to the documentation. Some of these likely should be completed before beta release.
Crucial:

[x] Add docstrings for the class IterDataPipe and MapDataPipe
https://github.com/pytorch/pytorch/pull/72618
[x] Review the categorization of IterDataPipe in torchdata.datapipes.iter.rst
https://github.com/pytorch/data/pull/219
[x] Edit first sentence of each DataPipe docstring to be a concise summary of functionality (also include functional name when it exists)
https://github.com/pytorch/pytorch/pull/72476
https://github.com/pytorch/pytorch/pull/72475
https://github.com/pytorch/data/pull/209
[x] Add usage examples to each DataPipe docstring
https://github.com/pytorch/pytorch/pull/73033
https://github.com/pytorch/pytorch/pull/73250
https://github.com/pytorch/data/pull/249
[x] Add tutorial (how to use DataPipe, how to write one, how to use it with DataLoader)
https://github.com/pytorch/data/pull/212
[x] Add domain usage examples (links to files)
https://github.com/pytorch/data/pull/216
[x] Decide what utility functions to include
https://github.com/pytorch/data/pull/205
[x] Link to relevant DataLoader documentation
https://github.com/pytorch/data/pull/205
[x] Turn on 'gh-pages' in this repo's setting
It is enabled.
[x] Clear labelling of prototype vs beta phase
https://github.com/pytorch/data/pull/252
[x] Add a link under the 'Docs' tab on pytorch.org

Nice-to-have:

[x] Update issue form for documentation related issues
https://github.com/pytorch/data/pull/215
[ ] Add links to domain usage examples onto individual DataPipe pages (see how TorchVision does this)
[x] Remove tutorial from README.md and link it to the documentation tutorial
[ ] Make a functional equivalent table in documentation (in a separate page?)

cc: @VitalyFedyunin @ejguan @wenleix @dongreenberg @NivekT

Hello, does this issue also plan to update the PyTorch documentation as well?
Actually, I find it can be quite confusing to first-time users of datapipes.
For example, in the official PyTorch repo, like, https://github.com/pytorch/pytorch/blob/master/torch/utils/data/datapipes/datapipe.py
the doc-string for how to properly multiprocess with IterDataPipe is incorrect (bcos the worker_init_fn for IterableDataset will not work for IterDataPipe) and incomplete (bcos it doesn't have the updated documentation about using .sharding_filter() and the new worker_init_fn that does .graph_settings.apply_sharding(...)

That being said, I would be happy to contribute, if you could give me pointers on how I can do so.

Hi @linminhtoo,
Thanks for looking into this!

Hello, does this issue also plan to update the PyTorch documentation as well? it may be slightly confusing to first-time users of datapipes, as the documentation in torchdata and torch can conflict.

This issue does cover our documentation within torch as well.

For example, in the official PyTorch repo, like, https://github.com/pytorch/pytorch/blob/master/torch/utils/data/datapipes/datapipe.py the doc-string for how to properly multiprocess with IterDataPipe is:

incorrect (bcos the worker_init_fn for IterableDataset will not work for IterDataPipe)

Can you elaborate on that point?

incomplete (bcos it doesn't have the updated documentation about using .sharding_filter() and the new worker_init_fn that does graph_settings.apply_sharding(...))

We are planning to land this PR, after which, users should not need to pass in additional worker_init_fn for the purpose of sharding.

https://github.com/pytorch/pytorch/pull/78631


this.$route updates in the current page before next page's 'asyncData' in defineNuxtComponent is resolved

Environment

Operating System: Linux
Node Version:     v16.14.2
Nuxt Version:     3.2.3
Nitro Version:    2.2.3
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      ssr
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-kou34x-c7cfxj?file=pages/posts/[id].vue
Describe the bug
Reproduction:

From index,  go to the post page
Go to the mountain page.

It's caused by the change of this.$route.params.id inside the computed property in each [id] page before the route actually finishes navigation. you can even see that NuxtLoadingIndicator didn't finish the transition animation but $route.params is already updated while it's still on the old page.
Check the console log
Additional context
This causes countless issues for me. especially when migrating my apps from nuxt 2. I have similar functionality based on route values.
Logs
No response

This is caused upstream Vue behaviour (suspense does not current support suspending injected properties like $route).
We have already implemented a workaround for this behaviour within Nuxt but you will need to use useRoute() in setup to get access to the route rather than using $route - or you can experimentally inject '_route' (but this is undocumented and may break in future).


Use PHP password hashing library

http://php.net/password
It's available from PHP 5.5.0 but a userland version from 5.3.7 is available: http://php.net/manual/en/intro.password.php
It's pretty similar to PasswordHash class we have now, but it seems PasswordHash isn't actively developed and when it comes to security, that's important.

I understand your point, but I honestly don't see any need to change what we have.  It may not have been changed in a while, but its also been battle tested.  I think even xenforo uses the library we use (at least they did at one point), as well as other foro packages.  So I'd like to see a compelling reason to go through the upgrade effort (again) in changing the hash algo.  Just IMO,

If https://github.com/elkarte/Elkarte/issues/2344 gets implemented it would be a good time to do this. So worrying about more work wouldn't matter. I am definitely not suggesting either for 1.1. That's too much.

nods on 2344 ... and not to be argumentative, but we should always think about more work.
With the given resources and time availability of said, we should always ask where is the best bang for the time.  I'm just not sure increased stretching on a bcrypt is where its at, thats all.  Obviously in 1.0 we had to close the massive door that was open.  Of course if someone has a :meat_on_bone: about it, I'm not going to complain either :smile:
I really need to catch up on whats available with php7,  I'm not sure what higher level DES functions are available then whats "always" available in 5.x to help increase the time cost of attacking a captured db.

I completely agree. I am not trying to imply we must do this or that we should be doing it anytime soon. I just wanted to put it out there so if someone (me?) was like "hmm... that might only take a few minutes, let me knock that out and we can add it to the changelog." I like seeing a big changelog :grin:

:+1:

Oh and this is only a refactoring. #2344 is a feature (or several features if you wanted to). If I had to choose between the two, I'd go for the other one.
Actually, I'm going to close this one for now. You're right, it doesn't serve much purpose right now.


tfgridnet_integration

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
[x] Did you read the contributor guideline?
[x] Did you make sure to update the docs?
[x] Did you write any new necessary tests?

What does this PR do?
Integrates TFGridNet from ESPNET into fairseq2.
Testing
To test the updated denoise_and_vad_audio.py locally.
Note: This script is tested on an .tsv file.
.tsv file contains filename/path for the audio.
This PR deals with additional arguments for SeperateSpeech i.e. --config : configuration.yaml file and --pth-model : path for the model .pth file.
How to Test:
Locally, just run the following:
To test master64:
python denoise_and_vad_audio.py --audio-manifest $INPUT_MANIFEST --output-dir $OUTPUT_DIR --denoise --vad
($INPUT_MANIFEST is path for .tsv file)
To test SeperateSpeech:
python denoise_and_vad_audio.py --audio-manifest $INPUT_MANIFEST --output-dir $OUTPUT_DIR --model SeperateSpeech --config /path/to/config.yaml --pth-model /path/to/model.pth --denoise --vad
It will generate two output audios: denoise and vad audio files in their respective directory.

LGTM!


Add noop controller, support initial development loop

Addresses https://github.com/crossplaneio/crossplane/issues/674
Overview
This changeset adds a first pass at a stack with a kubebuilder2-based controller, with support for all of the phases of stack development. The controller doesn't do anything, other than watching for its custom type and logging that it saw it.
There are a few patterns which have been established in this changeset for the local development of stacks. These are subject to change in the future.
Testing done
A lot of make local-build && make install locally and making sure the controller's deployment spins up properly.
Notes for reviewers
Since this is a codebase in early development and with no users, I'm not going to necessarily block merging on waiting for a review. At the same time, if you have any feedback or suggestions, please add them even if the code is already merged, and I'll incorporate them into future changesets!

I'm going to move ahead with merging this while I work on adding the wordpress pressing functionality, but feel free to leave comments here for future consideration!


Flake: TestBuildFromDevcontainerInRoot

When running the integration tests on https://github.com/coder/envbuilder/actions/runs/8911611279/job/24473293133?pr=163 I got the following error that was solved by re-running the test.
integration_test.go:384:
        	Error Trace:	/home/runner/work/envbuilder/envbuilder/integration/integration_test.go:384
        	Error:      	Not equal:
        	            	expected: "hello"
        	            	actual  : "OCI runtime exec failed: exec failed: cannot exec in a stopped container: unknown"

        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-hello
        	            	+OCI runtime exec failed: exec failed: cannot exec in a stopped container: unknown
        	Test:       	TestBuildFromDevcontainerInRoot

Closing due to age, let's re-open if this re-surfaces.


Improving Windows build output and verbose logging

Improving grammar, consistency of language, detail, etc. for verbose and regular output. Also made to more closely match the output of the Android platform (see also https://github.com/apache/cordova-android/pull/305).
Part of the ongoing effort mentioned in https://github.com/apache/cordova-lib/pull/439

:shipit:

@jasongin removed the word please pretty much everywhere

@jasongin I made a few tweaks based on some log messages in the iOS platform. Can you take another quick look?

LGTM

[Current coverage][cc-pull] is 72.88%

Merging [#176][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.52%

@@             master       #176   diff @@
==========================================
  Files            14         14
  Lines          1862       1877    +15
  Methods         342        346     +4
  Messages          0          0
  Branches        379        381     +2
==========================================
+ Hits           1367       1368     +1
- Misses          495        509    +14
  Partials          0          0

Powered by Codecov. Last updated by [f4e20a3...7db6a9e][cc-compare]
[cc-base-branch]: https://codecov.io/gh/apache/cordova-windows/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/apache/cordova-windows/compare/f4e20a3a10dcdc038abe3699d7a4f6576cc8eded...7db6a9eb43ed4b170ca1dac28745cdc45fb0bd05
[cc-pull]: https://codecov.io/gh/apache/cordova-windows/pull/176?src=pr


[flutter_tools] join flutter specific with home cache

Fixes #53833
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] All existing and new tests are passing.

Isn't there also an opportunity to cleanup:
https://github.com/flutter/flutter/blob/be0c1bd0f0cec3aa97c6be279b7beae9d0e8d6be/bin/internal/shared.sh#L152-L154
(and similar thing in the .bat variant)

The status or check suite Linux flutter_plugins has failed. Please fix the issues identified (or deflake) before re-applying this label.

you'll have to rebase upstream to fix this


Avoid Debian-patched pip in the shared venv

Should fix #386.
This fix should be more lightweight than #387, and it is simpler than #388.  It tries to find an acceptable middle ground between explicitly supporting Debian's changes to their pip and causing pain to users in order to make a statement.
This patch needs testing.  I haven't come up with a test plan yet (I don't want to mess up my currently-working actual pipx setup), but I'll try to do something with docker.

Test plan:

docker run --rm -ti ubuntu:focal bash
apt update -qq && apt install -y python3-{pip,venv} git
pip3 install --user git+https://github.com/mgedmin/pipx@debian-what-you-doin
~/.local/bin/pipx run pipx

This confirms that my fix works!
Replacing step 3 with pip3 install --user pipx confirms that the test plan can recreate #393, which is a manifestation of #386.
Now to fix unit tests.

I'm reposting my code comment reply here:
@mgedmin wrote:

$ docker run -ti --rm python:3.8-alpine sh
... some experiments to get the commands right ...
... which incidentally populate ~/.pip/cache so the measurements are independent of network speed ....

Unless I'm misunderstanding something - this comment highlights that time measured avoids one of the most relevant impacts - all users will have to download pip. Here are the docs for -I
-I, --ignore-installed      Ignore the installed packages, overwriting them. This can
                              break your system if the existing package is of a different version
                              or was installed with a different package manager!

So this would always ignore the system pip (possibly re-downloading it) for everyone. I don't think that's great. I think the approach in #388 is sufficient for getting a workable pip out of how it is packaged in Debian that does not force everyone else to reinstall pip.

Where are we on this?
Debian has fixed its pip installation thanks to @ivanov (https://github.com/pipxproject/pipx/pull/388#issuecomment-625560424). Is this still needed or can it be closed? I'm have no problem with it if it causes a slight slowdown because it only affects the shared libraries, which are installed rarely, and thus the impact to the user will be minimal.
I am curious if adding this would sidestep various inconsistencies in the environment though, which might be nice since it would ensure the same pip source code is used for everyone. In that case we may want to consider adding this even though Debian has fixed it's vendoring issue upstream.

Rebased on master to resolve the changelog merge conflict.


Refresh credentials to check validity

The original call to credentials.valid does not work until the
credential has been refreshed once, as discussed in #486. This
instead checks if the credential will work by trying to connect

Apparently this is not how to call refresh?
>           self.credentials.refresh()
E           TypeError: refresh() missing 1 required positional argument: 'request'

ping

Can someone with a GCP instance to hand please check this?

OK, I am convinced - this is going in. Thanks @baxen


mysql: Update to 8.0.13

Hash extraction should be possible with:
autoupdate.64bit.hash = "$basename\\)	\\s+MD5: ([A-Fa-f\\d]{32})"
Or just could work md5\">([A-Fa-f\\d]{32}) since it's first match

"$basename\\)	\\s+MD5: ([A-Fa-f\\d]{32})"

didn't work (multi-line?) but
md5\">([A-Fa-f\\d]{32})

did. Thanks!

You could always use (?m) (see modifiers) for multiline match. But since it should be first match all the time, it's unnecessary.


Migration of Taskqueue from Ck to Converse layer

Original author: Seonmyeong Bak
Original issue: https://charm.cs.illinois.edu/redmine/issues/1670

The task queue is currently implemented in ck-core even though it doesn't use any constructs on ck-layer.
Taskqueue is one of queues in converse and it should be able to be used independently by any programming framework based on Converse such as converse-based OpenMP and AMPI.

Original date: 2018-01-05 00:21:36

~~https://charm.cs.illinois.edu/gerrit/#/c/2988/~~ https://github.com/UIUC-PPL/charm/commit/bad107175ce3eee53f5b125259277f069bb6d449

Original author: Jim Phillips (@jcphill)
Original date: 2018-01-16 18:08:43

As noted in Gerrit, this broke Windows builds.


Support for 1.19.2 please 😊

Hi,
Not sure the best way to request it. But it would be great to have this on 1.19.2.
Thanks
Kris

Hi there Kris,
Just saw your issue ticket and wanted to let you know that the 1.19 version supports 1.19.2. I think that is why the creator added an X after 1.19 to indicate it would work for any update to 1.19. Hope this helps!

Hi, Not sure the best way to request it. But it would be great to have this on 1.19.2.
Thanks Kris

Hi there @KrisTC,
Just saw your issue ticket and wanted to let you know that the 1.19 version supports 1.19.2. I think that is why the creator added an X after 1.19 to indicate it would work for any update to 1.19. Hope this helps!


Export Idris functions for Java

Export Idris functions so that they can be called from Java and other JVM languages.

Thank you for your interest. It is not ready yet but exporting functions and classes would be in the initial PR and they are very close to be done including annotations support. I will add some documentation as well in the PR on how it will look like.

Export feature is now available with latest 0.7.0 release: https://github.com/mmhelloworld/idris-jvm/releases/tag/latest. Documentation: https://idris-jvm.readthedocs.io/en/latest/ffi/calling-idris-from-java.html.


Issue launching instances from incus-ui-dashboard

When I try to launch a instance using a locally crafted image it returns me an error
" Image not provided for instance creation"
Screenshot:

This is a fresh debian12 install and a fresh (yesterday) incus install
I can launch this image using incus launch

What's that image? Is that something you selected from the remote image server or a local image?

it's a local image, I've created it using incus publish on one of my instances

Do you think we can have a repo with the patches somewhere to iterate on or is there something on lxd's ui license that prevents it?

I'll take a look, it's likely an issue with handling of local images. Assuming that it's not something I broke with my changes, then it's something that should be reported to LXD UI so they can sort it out on their end. But given it's a part of the UI which had to be altered for Incus, it's quite possible that I broke it :)

Looks like LXD UI 0.8 should be fixing this issue, I'll work on porting our patches to that version now.

@edlerd hey, looks like you fixed most of the local image usage with 0.8, but I'm still seeing an issue when the image is of type VM. For some reason the drop-down doesn't adjust to select VM in that case (as it does with remote images), so creating the instance then fails because LXD/Incus is asked to create a container instance from a VM image.
Should be pretty easy to reproduce by coping a VM image locally and then attempting to use that to create a new instance.

@edlerd hey, looks like you fixed most of the local image usage with 0.8, but I'm still seeing an issue when the image is of type VM. For some reason the drop-down doesn't adjust to select VM in that case (as it does with remote images), so creating the instance then fails because LXD/Incus is asked to create a container instance from a VM image.

Thanks for the pointer, I'll take a look at this shortly.

Pushed 0.8 to daily now so that should close this issue, will pick up the LXD UI fix for the VM case when that shows up :)

This should fix it: https://github.com/canonical/lxd-ui/pull/743

thanks @edlerd !


Unable to install goformat extension in vscode

vscode: 1.28.2
linux: fedora26
Unable to install goformat extension
Failed with the following error:
Tried both:

go get winterdrache.de/goformat/goformat
and
go get github.com/mbenkmann/goformat/goformat

winterdrache.de/goformat/printer
../../winterdrache.de/goformat/printer/nodes.go:1580: s.Assign undefined (type *ast.TypeSpec has no field or method Assign)

Are the projects you mention actually VS Code extensions?
If yes, how did you try to install them?
Please provide better steps for what you did.
In addition you said "Failed with the following error:" but there is no error.
Please provide this info.

file this against go extension


Phaser Keyboard lastChar

Hello there!
I'm having the issue, that I can not access lastChar before any key has been pressed. I always get the following error:
phaser.js:44758 Uncaught TypeError: Cannot read property 'charCode' of null
If I press a key very quick before the game starts it works because lastChar seems not to be undefined.
I guess in this part lies the error:
Object.defineProperty(Phaser.Keyboard.prototype, "lastChar", {

    get: function () {

        if (this.event.charCode === '32')
        {
            return '';
        }
        else
        {
            return String.fromCharCode(this.pressEvent.charCode);
        }

    }

});

A workaround is to check game.input.keyboard.pressEvent first.

Fixed in #135


Fix local

Coverage increased (+0.1%) to 22.139% when pulling a871eb85f4786c2a4c8f4cf27d3e75eee5a86b99 on fix-local into e43cbd2c7e295af1789e991f648c1ff01a583bd6 on master.


clues: drop 3rd age cloak not druidic where appropriate

3rd age druidic cloak -> 3rd age cloak
elite doesnt drop druidic pieces and master has two druidic cloaks instead of one of each
hard clue change big pirate hat to pirate's hat, only elites drop the big hat
closes oldschoolgg/oldschoolbot#508

Codecov Report

Merging #141 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #141      +/-   ##
==========================================
- Coverage   78.20%   78.17%   -0.03%
==========================================
  Files         159      159
  Lines        1890     1888       -2
  Branches      152      151       -1
==========================================
- Hits         1478     1476       -2
  Misses        396      396
  Partials       16       16

Impacted Files
Coverage Δ

src/simulation/clues/Elite.ts
68.18% <ø> (ø)

src/simulation/clues/Hard.ts
69.56% <ø> (ø)

src/simulation/clues/Master.ts
75.00% <ø> (ø)

src/meta/types.ts
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


egressgw: remove nodeDataStore map from Manager

Reduce the amount of state kept by the Manager.
Manager.nodeDataStore appears to be redundant while Manager.nodes exists, refactor to remove it. While being behaviourally identical.

/test

/test


NSD shared Subjects Benchmark

Dataset description: This is a subset of the NSD dataset containing fMRI brain responses from the 1000 images that were shared between subjects. Each image has been shown to 4 subjects (3 trials each).
Metric: PLS regression between model activations and brain region responses.
Brain Regions Covered: V1, V2, V3, V4, early, high lateral, high ventral, high parietal.

Hi @thekej, thanks for opening a PR! Would you mind updating the description with more details about your benchmark? Once that is done we can take a look!

@mike-ferguson please let me know if anything else is necessary.

The model identifier 'mvimgnet_rf' already exists in the database. Please verify and consider renaming with a unique identifier.


Audit log for user profile changes

It'd be nice to have an audit log of user profile changes (probably in Redis?), so we can track when fields were changed and by which application. This would be helpful for debugging, and also a nice customer support/security feature (i.e. "When did I change my password?").

 As a dev, I want to log user profile changes so that we can track when fields are changed and learn about user/provide customer support.


Add kedar1998

Related issue

What changes does this PR do?
Screenshots (If any)
Questions

Hey @kedar1998,  Thank you for Contributing. Congratulations on your first PR.
You can take a look at other projects by the EddieHub community and also join the discord to come geek out with us 🔥

https://github.com/EddieHubCommunity


Failed to connect to redis during setup

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
Trying to install and run in both a docker container from the Dockerfile and via npm gives the same error trying to connect to redis during startup.
ERROR (150): Error connecting to Redis ({"errno":-111,"code":"ECONNREFUSED","syscall":"connect","address":"<IP_ADDRESS>","port":6379}).
Expected behavior
liquid should run
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Mac OS X
Arc Browser

Hello @gravyplaya ,
It is hard to tell without knowing your complete setup. How are you running your Redis instance? On docker as well? If yes, did you expose port 6379 from the container?
Redis and MongoDB are important requirements for Liquid to function as intended. Alternatively, you can also try the quick start guide which has a docker-compose example with the required dependencies.

I was actually trying to set it up on render.com using Docker.

I was actually trying to set it up on render.com using Docker.

@gravyplaya in that case, if you do not want to use Redis or you are just trying to evaluate Liquid, you can disable the use of Redis by setting the environment variable CAN_USE_CACHE to false (Not recommended for production). You will still need a MongoDB instance though which you can configure in the environment variable MONGO_DB_CONNECTION_STRING.