Breaking change desc in 5.0.0 can be improved

Currently it states:
https://github.com/webpack-contrib/file-loader/releases/tag/v5.0.0
switch to ES modules by default (the option esModule is true by default)

While restoring previous behaviour is described in the source code. Above description could be changed to:
switch from CommonJS to ES modules. You can go back to CommonJS with:

options: {
              esModule: false
            },

Thank you for creating this issue. However, issues need to follow one of our templates so that we can clearly understand your particular circumstances.
Please help us help you by recreating the issue using one of our templates.


Updated dropbox-experimental to 3.9.32

Thank you for the contribution. It was merged directly as ed08a43abc0edbc9ca9774be5e2c670e285b3492 to keep commit history cleaner. Your contribution is still credited to you.


providing a way to handle success

Hi
Here is an improvement to enable the handling of success cases for json_requests.
Here we can now have the list of all registration_ids that were successful.
There is something to keep in mind that i didn't write in the readme (because it's already in GCM manual) : the registration_ids listed in "success" can also be in "canonical"
well.. i hope, because I admit that your "group_response" method is quite obscure to me :)
Guillaume

Hello! I know this has been posted a while ago, but can you please update this pull request against the current code base?
Also make sure to have a single commit for your changes (code, docs, tests) that are referring to the same thing.
If you have no intention in working on this I'll close this pull request and open an issue that will refer to this for someone else who is interested to work on this feature.

Hello, Sorry, yes, it's been a while !
And I'm not at all into this kind of things at the moment. So you took the right decision : close this and file an issue. thanks :)


[backport] New style clipped_relu

Backport of #3503

jenkins, test this please.

jenkins, test this please

jenkins, test this please

jenkins, test this please.

jenkins, test this please.


Fixing the module did not self-register issue when importing in worker threads

Importing this package in worker threads (Node 12) causes "Module did not self-register" error, this change fixes that

Which node versions support NODE_MODULE_INIT? Node 8 failed in CI.

Fixed in 5.0.0 / 5.0.1.


FEAT: CHANGE ERROR MESSAGES

Changed all text based error messages.

will make the necessary changes.

I had to implement this because I needed to cut down the contract size.


Format rule  to keep together }, { for list of objects

Is there a setting or a rule that will keep this style of formatting ? I mean keeping }, { together and then hugging }] at the end?
trans = [{
    'first': Stuff,
    'second': Other,
    'third': Blah,
    'fourth': Sth
}, {
    'first': Stuff,
    'second': Other,
    'third': Blah,
    'fourth': Sth
}, {
    'first': Stuff,
    'second': Other,
    'third': Blah,
    'fourth': Sth
}]

I tried 0.3.4 and 0.4.3 ruff for this, with enabled preview, shouldn't it keep hugging braces and brackets ?
Related discussion I think

Hi @darowny
No, there's no such formatter setting.
The hugging preview style only applies if the array (or dict or set) is the only argument in a function call or the only content of a prenthesized expression
trans = call([
    {
        "first": Stuff,
        "second": Other,
        "third": Blah,
        "third": Blah,
    }
])

trans = a + ([
    {
        "first": Stuff,
        "second": Other,
        "third": Blah,
        "third": Blah,
    }
])


Huawei E3531 showing up as ethernet interface

Hi guys,
Not really sure if it's some problem or it's me doing something wrong but I can't manage to connect the Huawei E3531 to the network. It keeps showing  up as another ethernet interface with IP <IP_ADDRESS> and it answers to pings but it never showed up as a ppp interface. I tested a Huawei K3520 before and it connects without problems.
It's one of the listed as tested and working OK so I'm a bit confused.
Here's what I get in dmesg (could it be that product ID is different?):
[ 1571.968729] usb 1-1.2.3: new high-speed USB device number 8 using musb-hdrc
[ 1572.070920] usb 1-1.2.3: New USB device found, idVendor=12d1, idProduct=14dc
[ 1572.070951] usb 1-1.2.3: New USB device strings: Mfr=1, Product=2, SerialNumber=0
[ 1572.070965] usb 1-1.2.3: Product: HUAWEI Mobile
[ 1572.070977] usb 1-1.2.3: Manufacturer: HUAWEI
[ 1572.200661] cdc_ether 1-1.2.3:1.0 eth1: register 'cdc_ether' at usb-musb-hdrc.1-1.2.3, CDC Ethernet Device, 00:1e:10:1f:00:00
[ 1572.211280] usb-storage 1-1.2.3:1.2: USB Mass Storage device detected
[ 1572.224798] scsi host0: usb-storage 1-1.2.3:1.2
[ 1573.334324] scsi 0:0:0:0: Direct-Access     HUAWEI   TF CARD Storage  2.31 PQ: 0 ANSI: 2
[ 1573.448929] sd 0:0:0:0: [sda] Attached SCSI removable disk
[ 1573.506815] IPv6: ADDRCONF(NETDEV_UP): eth1: link is not ready
[ 1573.514270] IPv6: ADDRCONF(NETDEV_UP): eth1: link is not ready
[ 1595.687740] IPv6: ADDRCONF(NETDEV_CHANGE): eth1: link becomes ready
[ 1595.687818] cdc_ether 1-1.2.3:1.0 eth1: kevent 12 may have been dropped
Thanks!

Managed to make it work disabling the pin in Windows (apparently the modem has a web interface). Not really sure if there is some way to keep the pin and input it when connecting the modem in resinOS.

@Spidey86 I have seen this before with some of the huawei modems. The ones that have the hilink management panel are not managed by modemmanager, because they have an internal management engine. I know there is a way to change the default so the modem will be in AT modem so that modemManager can manage it, then you could set the PIN from modemManager config. Its often pretty hard to find documentation on how to convert these modems this mode though.


Extract Slack notifier from SirCAL to allow reuse

Description
Extract Slack notifier from SirCAL to allow reuse
Changelog
Itemize code/test/documentation changes and files added/removed.

Creates slack notifier
Refactors SirCAL code to use the extracted code

I think I clobbered you with #675 :(

I think I clobbered you with #675 :(

I think I clobbered you with #675 :(

Fixed.

I think I clobbered you with #675 :(

Fixed.


Dungeon Siege II → no mouse cursor

We should have a mouse cursor, but there is none, so we are stuck on the first menu.
without d9vk:
System information

GPU: Nvidia GT 730M
Driver: 435.21
Wine version: Proton 4.11
DXVK version: #852

Apitrace file(s)
https://mega.nz/#!Duh3RIRQ!h89Z2ypBY10-yqySNVpi6YPzgzto1f4hD1d9W5LhA3c
Log files
steam-39200-d3d9.log
steam-39200-d9vk.log

Go windowed mode, this issue also appears on windows.

I have a cursor withour d9vk in fullscreen:

Found this video quite helpful: https://youtu.be/WMXsZZCo9S8

Found this video quite helpful: https://youtu.be/WMXsZZCo9S8
Feel free to comment if it didnt work (or if it did)

It is already known the cursor works in windowed mode.

Found this video quite helpful: https://youtu.be/WMXsZZCo9S8
Feel free to comment if it didnt work (or if it did)

It is already known the cursor works in windowed mode.

Yip, I know, just seems like it saves time having it in video format ;)

Found this video quite helpful: https://youtu.be/WMXsZZCo9S8
Feel free to comment if it didnt work (or if it did)

It is already known the cursor works in windowed mode.

Yip, I know, just seems like it saves time having it in video format ;)

Sorry to see your thumbs down, im new here, i must have selected incorrect profile options, I thought this was here to help people get a solution or some kind of learning space. What is this page though?

Here is the remaining fixes PART 2 of 2
https://youtu.be/wiZntRumPq8

@Joshua-Ashton can you delete those messages which are not proper fixes but rather ads for a channel on Youtube?

Before I do, its me figuring out a way to show how to resolve the issue in video format to people who like me typed in google and searched the issue i thought i was having alone, i dont like reading all the time and im sure many others feel the same, and i like a challenge, from all the reading doing research, this was my solution that makes the game in a condition that worked for me, i found one video that is similar...saw a Challenge...Took It...but I saw a thumbs up on my video...so...ya...lets see if others want me to keep my posts here. Please Like my video posts above if you here for a similar reason...to find a temporary fix instead of to have a dud game while we wait for some type of update OR perhaps if you prefer video instruction as an easier option to be assisted throughout a fix process...for either...or both...Like my video post...also...dude...you opened this issue on Sep 21, 2019... even if they implement my fix as a update only and include maxwindowsize with transparent borders into it...that id accept as a fix...would you? enough typing for now...can i remove it Sunday night? (People research quite a bit on weekends)

Not to be rude, but is the issue still present when using the version at https://github.com/doitsujin/dxvk? That's where d9vk development is these days.

@Joshua-Ashton can you delete those messages which are not proper fixes but rather ads for a channel on Youtube?

removed. they not ads. they there to help others like me fix a dud game

@CartoonFan yes, the issue is still present, use PROTON_USE_WINED3D=1 for now, it's the best way to play the game in good conditions. It also works with fullscreen=false, but it's windowed and the videos flicker a lot, I don't know if it's related to DXVK or the windowed mode.

Hmm...that's rough. I hope @Joshua-Ashton or @doitsujin are at least aware of the situation. I mean, since everything moved on, the d9vk repo has been basically dead, so I hope they know that some old issues are still present in current dxvk.

@NerosTie If this is this is still a problem with latest dxvk then please move the issue to that repo. 🙂
Hopefully it can be looked at then.

Duplicate with https://github.com/doitsujin/dxvk/issues/3020
I close this one.


JS Styleguide and Unit tests on XOSlib

Added Eslint with Google JavaScript Style Guide.
Started unit test on xos-utils.js and xos-backbone.js

Issue #113 should be fixed with this PR.

Content provider view has been added to this PR, to view it:

Open http://<IP_ADDRESS>:9000/admin/core/dashboardview/
Add a dashboard view with url: template:contentProvider
Add the dashboard in Home Customize tab


Note colors for specific songs?

So, I want to put these note colors in some songs, but they apply to every song, and that glitches other note skins. Is there a way I can make them apply only to specific songs? Any coding? (Lua or HX, but Lua preferably)

your gonna have to hardcode it idrk if you can use lua to help but its extremely complicated code


Backspace and Delete have an errant character

When back spacing or deleting characters in a TEXTAREA an extra Â appears that needs to be deleted. This occurs every 3-4 characters and happens on all browsers.

Jodit Version:  3.5.4
Browser:
Firefox 84.0, Chromium 87.0.4280.88, Chrome Version 87.0.4280.88
OS:
Kubuntu 20.04 (Ubuntu 20.04 base)
Is React App:
False
Code
Expected behavior:
No extraneous characters be added to text when deleting characters within a TEXTAREA.
Actual behavior:
Extra 'Â' characters are typed by Jodit automatically when deleting characters with the Backspace or Delete keys.

How do I fix this in javascript ?

This bug is back in the latest version (Version: v4.2.27) (Chrome Version 129.0.6668.71 (Official Build) (64-bit))


Feature/update vrx launch settings

update launch settings for VRX.

This PR should be merged after this PR merged.


Add Gulp to README dependencies

Gulp is needed globally to run the build/dev server processes, also a minor sentence change for running the generator.

Thanks for the pull request Pam. There's actually some logic in the codebase to check whether a user has gulp installed locally or not (https://github.com/swirlycheetah/generator-angular2/blob/master/app/index.js#L64) but perhaps that's not working as it should be.
If you could try re-running the generator, letting me know if you've got gulp installed globally or not and then seeing what command Yeoman says to run once the generator has finished that'd be useful.
I'd prefer not to have to ask people to install global packages if possible so it'd be good to figure out an alternative solution, perhaps something like wrapping the gulp tasks in npm run scripts. Any thoughts/suggestions welcome.

@swirlycheetah I got it to run once I installed gulp globally ... my testing ground is biased though, because I had to unbork some Node setup on this machine, so the yo process halted and I had to step into it. Generator works great once all that (and gulp) is set up :)
I also dislike global packages, so I'm with you there, and I noticed there weren't any scripts set up for gulp

Sorry, I missed your update. Just for clarification do you mean an npm run task to wrap the gulp tasks? If that's the case I'll sort that, it should mean I can actually tear out quite a bit of that nasty global dependency checking I'm doing.

@swirlycheetah Yes, my suggestion is an npm run task by putting it in the scripts object in package.json :)

Have added the npm run scripts and pushed a new tag, 0.6.2. Any further issues let me know.

Apologies if this is added somewhere else ... but I don't see your changes here, FYI https://github.com/swirlycheetah/generator-angular2/blob/master/package.json#L19-L21

Ah, the package.json that gets created when you run yo angular2 is here https://github.com/swirlycheetah/generator-angular2/blob/master/app/templates/_package.json. I hope we weren't at crossed wire this entire time and you actually meant this projects package.json rather than the one created by the generator!
I have noticed one issue with 0.6.2, if you're using npm >= version 3 there's an issue with the traceur runtime location (#39), I'll get this fixed tonight.

the scripts would need to be more like ./node_modules/gulp/... pointing to the executable for that to work without depending on global gulp.
$ npm start

><EMAIL_ADDRESS>start /Users/----/Sites/foo
> gulp dev

sh: gulp: command not found

npm ERR! Darwin 14.5.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "start"
npm ERR! node v5.0.0
npm ERR! npm  v3.3.6
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR<EMAIL_ADDRESS>start: `gulp dev`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'gulp dev'.
npm ERR! This is most likely a problem with the foo package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     gulp dev
npm ERR! You can get their info via:
npm ERR!     npm owner ls foo
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/----/Sites/foo/npm-debug.log

you'll want
  "scripts": {
    "build": "./node_modules/gulp/bin/gulp.js",
    "start": "./node_modules/gulp/bin/gulp.js dev"
  },

I'm closing this PR since I don't want it merged and you're doing fixes elsewhere :) feel free to make an issue, but I think this is almost done anyhow. Thanks for maintaining the generator!

Thanks for the feedback Pam. You don't actually need to reference the longhand path for local dependencies when using npm run scripts, see https://docs.npmjs.com/cli/run-script;

In addition to the shell's pre-existing PATH, npm run adds node_modules/.bin to the PATH provided to scripts. Any binaries provided by locally-installed dependencies can be used without the node_modules/.bin prefix.

I am a terrible QA person and got the error message in a situation where the node_modules were not yet installed
I have learned much from this PR. Thanks!


⚠ Aset App has degraded performance

In a026c16, Aset App (https://asetapp.com) experienced degraded performance:

HTTP code: 200
Response time: 20568 ms

Resolved: Aset App performance has improved in 9ba27f9.


change font size of code

Hello! I have made this small change so that code scales based on its containing element. For example, code included in a title is title-sized.
Before:

After:

Hopefully I've followed the right procedure and etiquette for this - I've never done a pull request before! Thanks for making this project, it's really useful.

Thank you very much @alexandergill! It's an honor to be someone's first PR.


Fix add wildcard tag bug

The Tag class was updated with additional methods and constructor that accept a boolean flag to determine if wildcard tags should be enabled. Tag default constructor and validity checking methods have been reverted to their original form for ease of use by others.

Codecov Report

Merging #78 into master will decrease coverage by 1.08%.
The diff coverage is 89.28%.

@@             Coverage Diff              @@
##             master      #78      +/-   ##
============================================
- Coverage     63.34%   62.26%   -1.09%
- Complexity      411      424      +13
============================================
  Files            81       85       +4
  Lines          1457     1521      +64
  Branches        155      166      +11
============================================
+ Hits            923      947      +24
- Misses          499      533      +34
- Partials         35       41       +6

Impacted Files
Coverage Δ
Complexity Δ

...java/seedu/address/logic/commands/EditCommand.java
91.66% <25.00%> (-3.40%)
16.00 <1.00> (+1.00)
:arrow_down:

.../seedu/address/logic/parser/EditCommandParser.java
93.10% <100.00%> (+0.51%)
11.00 <0.00> (ø)

...in/java/seedu/address/logic/parser/ParserUtil.java
80.70% <100.00%> (+4.61%)
18.00 <4.00> (+4.00)

src/main/java/seedu/address/model/tag/Tag.java
94.73% <100.00%> (+10.12%)
11.00 <6.00> (+4.00)

...a/seedu/address/logic/parser/AddCommandParser.java
80.00% <0.00%> (-20.00%)
4.00% <0.00%> (-1.00%)

.../main/java/seedu/address/model/person/Address.java
70.00% <0.00%> (-10.00%)
7.00% <0.00%> (+1.00%)
:arrow_down:

.../seedu/address/logic/parser/AddressBookParser.java
80.00% <0.00%> (-8.89%)
12.00% <0.00%> (ø%)

...rc/main/java/seedu/address/model/ModelManager.java
75.38% <0.00%> (-3.65%)
23.00% <0.00%> (ø%)

src/main/java/seedu/address/model/Model.java
100.00% <0.00%> (ø)
1.00% <0.00%> (ø%)

...rc/main/java/seedu/address/model/person/Phone.java
80.00% <0.00%> (ø)
8.00% <0.00%> (+2.00%)

... and 7 more

Continue to review full report at Codecov.

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

lgtm - good catch!


Enable calls with commitments

Besides Fun, Comm and Num pointers should also be callable objects as long as opening them reveals a Fun. For example, the three following expressions should reduce to the same value:

((lambda (x) x) nil)
((commit (lambda (x) x)) nil)
((num (commit (lambda (x) x))) nil)

Yes

@porcuquine I just realized that expressions like (1 2 3) will error in a non provable way because of the failure to open the commitment on the head. Let me know if we should follow the plan to include nums or if we should only support comms.


ROMP and BEV ckpt finetuned on AGORA

Hi,
Thanks for your amazing work. I cannot find how to download BEV_ft_agora.pth on your repo. The train_models.zip does not have this model.
Would it be possible to share the ckpt? Sam for ROMp finetuned on AGORA?
Thanks for your time,

Hi @fabienbaradel
As indicated by this instruction,
https://github.com/Arthur151/ROMP/blob/master/docs/bev_evaluation.md#agora
current BEV model has been fine-tunned on AGORA. So you can just change the name to current model.
About the ROMP model: Sorry, it is trained about two years ago. I didn't keep it.
But if you have deployed the training code of ROMP, it should be easy to get it.
Have a nice day.


Linux - Clicking system tray icon doesn't show window

Expected Behavior

Clicking on the system tray icon should show the main window.
Current Behavior

Clicking on the system tray icon does nothing .
Possible Solution

Looks like it's probably the same as this electron issue, so doesn't look like there's a solve.
Using the Ctrl-Alt-Shift-T shortcut to force the window to show.
Context

Code Sample

Your Environment

Software
Version(s)

Package
1.1.2

Runtime

Package Manager

Operating System
Kubuntu 16.04

Thanks @emlyn-h!
I tested this about a month ago on Ubuntu without issue. With all the variants of Linux I'm aiming for minimum support until I get someone to handle the majority of the work.

One option might be to change the behaviour on Linux.
Context menu on the icon seem to work for me (right click menu style).  But it's an extra user click, which feels a bit redundant.
Example: https://github.com/emlyn-h/tweet-tray/commit/326ac39c54a5bb1fbebd1af76aa5908f9227889b

@emlyn-h if you reckon this will run fine on most common distros then I'd be happy to merge it. It's good to make sure Tweet Tray is passable for Linux users, as opposed to relying on a keyboard shortcut. 😄

@jonathontoon yeah, might be a safest.  The docs seem to have a bunch of Linux limitations listed.


Column background image does not show in editor

Just like a Section, a background image can be set for a Column. While the Section shows the background image in the editor, the Column does not.

https://documentation.mjml.io/#mj-column
Column does not have a background image, easy-email is based on MJML.

Alright, thanks for explaining. The attributes panel for Column does show the background image option though.

We'll fix it this week.

The latest version has been fixed. I have replaced "background" with "background color".


[CI] MacOS CI failed because brew install failed

Describe the enhancement requested
because this is an alternate version of another formula.

If you need to have node@18 first in your PATH, run:
  echo 'export PATH="/usr/local/opt/node@18/bin:$PATH"' >> /Users/runner/.bash_profile

For compilers to find node@18 you may need to set:
  export LDFLAGS="-L/usr/local/opt/node@18/lib"
  export CPPFLAGS="-I/usr/local/opt/node@18/include"
Upgrading c-ares has failed!
Using curl
Installing ccache
Upgrading cmake
Installing flatbuffers
Using git
Installing glog
Installing googletest
Installing grpc
Installing llvm@14
Using lz4
Installing mimalloc
Installing ninja
Upgrading node
Upgrading openssl@3
Using pkg-config
Installing protobuf
Upgrading python
Installing rapidjson
Installing re2
Using snappy
Installing thrift
Using utf8proc
Using wget
Installing xsimd
Using zstd
Homebrew Bundle failed! 1 Brewfile dependency failed to install.
Error: Process completed with exit code 1.
Maybe it's the homebrew or other error?

Example:

Python / AMD64 macOS 12 Python 3 : https://github.com/apache/arrow/actions/runs/7044700065/job/19183502836
C GLib & Ruby / AMD64 macOS 12 GLib & Ruby: :https://github.com/apache/arrow/actions/runs/7044700070/job/19183508096

Component(s)
Continuous Integration

This seems like a duplicate from: https://github.com/apache/arrow/issues/39003

Yes. This is duplicated.


How to manage host files

This is a question about usage because I'm new to the provisioner.
How do I dynamically generate the hosts file? The below configuration doesn't provide me with a hosts file.
If I want to change something in my ansible 'users' role, for instance, and run only the ansible code again, how do I do that? The code below generates no 'hosts' file. Should it? It wasn't clear from the documentation. Or it's at least not working for me.
Thanks in advance!
resource "hcloud_server" "control_plane" {
  count       = 3
  name        = "${var.cluster_name}-control-plane-${count.index + 1}"
  server_type = var.control_plane_type
  image       = var.image
  location    = var.datacenter

  ssh_keys = keys(var.ssh_public_keys)

  labels = {
    "kubeone_cluster_name" = var.cluster_name
    "role"                 = "api"
  }

  provisioner "remote-exec" {
    inline = [var.install_ansible_dependencies ? var.ansible_dependencies_install_command : "sleep 0"]

    connection {
      host        = "${self.ipv4_address}"
      type        = "ssh"
      user        = "root"
      private_key = "${file(var.ssh_private_key)}"
    }
  }

  provisioner "ansible" {
    ansible_ssh_settings {
      insecure_no_strict_host_key_checking = true
    }

    plays {
      enabled = var.run_ansible_playbook
      vault_password_file = var.ansible_password_file
      groups = ["nodes"]
      hosts = [self.ipv4_address]

      playbook {
        file_path = "../ansible/node.yml"
      }

      extra_vars = {
        cluster_name = "${var.cluster_name}"
        server_name  = "${var.cluster_name}-control-plane-${count.index + 1}"
        ansible_user = "root"
      }
    }

    connection {
      host = "${self.ipv4_address}"
      type = "ssh"
      user = "root"
    }
  }
}

@radekg any ideas? or from users?

Hi @laurawadden. Apologies for delayed answer. By design, the provisioner does not expose the hosts. This is because the provisioner is designed for provisioning the hosts created by Terraform during the Terraform run, and only those hosts. Exposing the hosts file directly to the users potentially opens a can of worms where the complete Ansible functionality would have to be reimplemented inside of the provisioner.
For the local provisioner, at this moment in time, there is no way to get hold of the hosts file. For the remote provisioner, if remote.skip_cleanup is true, the file would be left in the temp bootstrap directory.
Having said all of that, if I understand correctly, what you would like to achieve is: provision the host with Terraform Ansible provisioner and potentially afterwards use Ansible directly to run against the already existing host?

Thanks for replying! No worries about being late.

Having said all of that, if I understand correctly, what you would like to achieve is: provision the host with Terraform Ansible provisioner and potentially afterwards use Ansible directly to run against the already existing host?

Yes, this is exactly my use case. But I'm coming from a background using only static hosts with Ansible, so maybe using dynamic hosts is the way to go. Still looking into it.
If you have any ideas/tips on how to set this up, I would very much appreciate it :)

I think, due to the nature of how Terraform is intended to be used, this is not possible out of the box. With Terraform, the intent is to promote immutable infrastructure. To apply changes on a running host, one would have to run Ansible independent from Terraform.
What could be done here, the remote.skip_cleanup could be promoted to a top level skip_cleanup, such that it would be possible to get hold of the hosts file after the local provisioner run. That could be an intermediate solution, at least giving a chance to have the hosts file available.
Alternatively, maybe a mix of outputs with Ansible dynamic inventory operating on Terraform outputs is a good option? The dynamic inventory could use a method similar to https://github.com/radekg/terraform-state-ansible-module for constructing the inventory file from outputs.
The intermediate solution with the cleanup tweaks is easy to implement. In the long run the dynamic inventory would be definitely preferred.

Closing due to inactivity.


fix simultaneous validateAll calls error reporting

could you please npm publish v1.2.9?
thanks

Ohh , i forgot it


I'm trying to build the 'op' for vmamba in onnxruntime.

In visual studio 2022， I'm trying to build the 'op' for vmamba in onnxruntime. Now, I'm compiling CUDA code, and this error occurred. Error C2672 "selective_scan_fwd_cuda": No matching overloaded function found. The error occurred in selective_scan_nrow.cpp at lines 242 and 354.I noticed that the compilation of the nrows mode is commented out in the setup.py file. What is the main purpose of this section? Can I simply not compile the related files? Additionally, many files include references to PyTorch. Is this mainly for registering these algorithms with PyTorch or for utilizing certain algorithms from PyTorch? If I deploy this algorithm in onnxruntime, can I only compile the CUDA-related files and remove the PyTorch-related ones?

Nrow is used to be an acceleration version for selective scan, but that does not work well, so it is not needed now.

I know nothing about onnx, so I am not sure that if Pytorch related functions can be ignored. But if you are talking about the torch.autograd.Function, I think if just for inferencing, you can only refer to the code in forward function and ignore the backward.


resolves #4 add transform-group option to use single wrapper element

using single wrapper to scale slides if option is transform-group
fix tests
add tests for transform-group option

As a side note, tests don't work on Firefox for some reason. However, I did manually testing in Firefox and it works great.

I realized that when transform-group is specified, it still uses zoom in Chrome. This differs from the transform option, which uses transform for all browsers. So we have an extra degree of configuration here. Should we switch to an options map so that we can have a separate flag for whether to wrap each slide or the group of slides?
zoom with transform (on group) fallback
{ wrap: 'group' }

transform (on group)
{ method: 'transform', applyTo: 'group' }

The default would be:
{ applyTo: 'slide' }

When applyTo is "group", I'd also move the zoom to this wrapper element so that the change is consistent.

Or maybe we can simplify the option name to "on":
{ method: 'transform', on: 'group' }

Is there a value you prefer to describe the group of slides instead of "group"?

Another possibility is to use a fallback chain, like:
scale('zoom|transform group')

vs
scale('zoom|transform slide')


feat(learn): Add/Remove Tests for Imperial-Metric Converter Project

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

* Removes the two "Information Security" tests as per #39435

Add additional tests for invalid numbers/units/both number and units and final output check
Improve error reporting for non-XHR errors.  If an assertion fails the error is in the property message, so I have ORed the message for all tests. This provides a better experience for debugging.

Testing
I have tested from GitPod against my personal project (which passes) and against the boilerplate (which fails).
Note on 'invalid *' tests
The current example project does not pass some of these tests.  As noted in #39470, it has some issues with some invalid units.  It also returns the error as { error: "invalid unit" }.  If this is the standard, then the project/boilerplate will need to be updated to state as such.  For the moment I have adopted test strategy of checking for a value in the .error property and plain-text return value.
IE:
data.error === 'invalid number' || data === 'invalid number'

Merge conflict should be resolved.

If you would like to change against my project:
https://metric-imperial.pearlygatesoftware.com/

Thanks for this, Rex.
Where are we on the L vs l return units? Are we going to instruct all lowercase return values?
Pending that, I see this as ready-to-go.

I have made the changes as proposed.
These can be tested against my project, here:
https://metric-imperial.pearlygatesoftware.com/
I don't understand all of the other files showing as changed.  I may have mucked up the Git history.  If so, I can try to fix or I can resubmit as needed.

Holy Toledo! I managed to rebase it properly to remove the spurious commits.  I also accepted @nhcarrigan's suggestions.

I rebased this again quick since it had conflicts.

Do you want to make a PR to the demo-projects repo with your example project @SaintPeter? You should remove the user stories from the landing page so it looks something like this.

Any reason we are not merging this yet, other than a second review @moT01 ?

Nope @raisedadead

I will review it now.

@RandellDawson Amazingly, I guess I don't have a PR open for the example project.  I'll see about spinning mine up.

@SaintPeter I have tested everything against the example project (PR https://github.com/freeCodeCamp/demo-projects/pull/37) and everything passes.  In a separate PR, someone else can work on fulfilling the requirements referenced in issue https://github.com/freeCodeCamp/freeCodeCamp/issues/39721.
Again, thanks for your hard work on all 3 of the PRs related to this project.
@nhcarrigan @Sky020 You show outstanding change requests, so if you are good with the changes, please approve and merge this PR.  Thanks.

For what it's worth, that requirement was previously established and is present in the example project:

@nhcarrigan Thanks for that. I missed it in my project.
Then, we at least need to change this section of the user-story/test:

spelling out units in the format '{initNum} {initial_Units} converts to {returnNum} {return_Units}'

This is my last outstanding PR (of 3).  Are we waiting on some specific changes here?

I believe the changes in @Sky020's review discussion are all that is left.

It seems overly complex to have the user provide the shorthand version of a unit (like mi for miles) and expect the final string property to return the long version of the two (miles for mi and kilometers for km), but I am fine as long as user stories are clear about it and the tests correctly test what is expected.  If we are worried about some kind of backwards compatibility, then the user stories/tests should allow for both versions.

@RandellDawson I don't think that this is the appropriate venue for this discussion. These requirements were in the original project and were unchanged by my additions.
The purpose of this PR was to implement the missing tests for this project.  Along the way I added some clarification and made slight changes to the user stories and tests to be more consistent.  That's about it.
If you would like to propose additional changes to this project, I suggest we open an issue to discuss it.  Right now there are students who are missing out on the benefit of having tests on /learn until this PR is merged.

I created a new issue to add/discuss the changes that are still needed. I think we should merge this as-is since the issues remaining were preexisting.

@Sky020 Are you fine with merging this PR, since @moT01 has created an issue outlining the concerns?


run_guarded using bound function ignores reference

It seems binding a variable to a function, then calling run_guarded on that binding won't honor a reference. Take a minimal example
#include <hpx/lcos/local/composable_guard.hpp>
#include <hpx/util/bind.hpp>
#include <hpx/hpx_init.hpp>
#include <iostream>

void incr(int &il1) {
    // increment variable by 1, implicitly lock guard
    il1 = il1 + 1;
    // implicitly unlock guard
}

hpx::lcos::local::guard l1;

int hpx_main(boost::program_options::variables_map&)
{
    int i1 = 0;

    // run incr with i1 passed as a reference
    run_guarded(l1, hpx::util::bind(incr, boost::ref(i1)));

    std::cout << i1 << std::endl;

    return hpx::finalize();
}

int main(int argc, char* argv[])
{
    boost::program_options::options_description
        desc_commandline("Usage: " HPX_APPLICATION_STRING " [options]");

    return hpx::init(desc_commandline, argc, argv);
}

If you were to compile and run this
> ./a.out
0

It seem's i1 is not updated, although i've bound a reference of it to incr and invoked it via run_guarded. Is this a bug or am I mistaken?

IRC Logs:
<K-ballo> Syntaf: does it ever hit incr?
<Syntaf> K-ballo: inc is called, if I placed a print statement within it, it would print
<K-ballo> the guard executes afterwars
<K-ballo> *afterwards
<K-ballo> weird, I too would have expected it to synchronize
<K-ballo> not bind's fault then
<Syntaf> so run_guarded has fire and forget semantics?
<hkaiser> Syntaf: it seems to execute things during destruction

I think the problem is that at the time you print out i1, it hasn't yet incremented. If you put a print statement inside the guarded function, I think you'll see that it does run. To answer the question about fire and forget, the answer is a qualified yes. While the guard doesn't return a future, attaching another function will happen after the first, the equivalent of a then(). The intent was that you program it in "futurized" style. If a guard did return a future and had a get(), then it would lose the property of being inherently deadlock free.

@stevenrbrandt Is there a way to allow synchronization with the last function running within the guard? Could we add something like
void incr(int &il1) {
    il1 = il1 + 1;
}

hpx::lcos::local::guard l1;
int l1 = 0;
run_guarded(l1, hpx::util::bind(incr, boost::ref(i1)));

// ...

hpx::future<void> f = l1.get_future();

// ...

f.get(); // wait for all guarded functions to execute

Let me give a more thorough explanation of what a guard is supposed to be.
Maybe that will help us clarify what we should do about this ticket.
In the old days we wrote programs with long-lived threads and when we wanted to
have only one of them access a certain region of code, we blocked them using
mutexes. The problem, of course, if thread_a wants to acquire first mutex_1
then mutex_2, and thread_b wants to acquire first mutex_2 then mutex_1, a
deadlock occurs. The wisdom was that you should get your mutexes in canonical
order. Unfortunately, that wasn't possible if you wanted your programs to be
composable.
However, now that we have short-lived user level threads we can think of how to
solve the problem in a new way. What if adding mutexes to a running thread is
the real problem? What if we solve it by making the thread itself the critical
section? This is the idea of the guard. The guard is like a mutex, but you need
to acquire your guard or guards before your thread enters the run queue--and
you release them automatically when the thread exits.
If you need several guards at once, the system can sort them in some canonical
order and acquire them one by one before making the thread that wants to use
them ready-to-run. Voila, composability and no deadlocks. In fact, nothing even
blocks.
Another problem with mutexes is that if you suspend the thread holding it, new
threads can potentially be blocked. Guards don't have this problem. Because
each thread that wants to use a guard doesn't get to enter the run queue until
its guards are ready, no thread is ever blocked by another holding a guard. So
guards solve this problem too.
Consider this problem (switching to pseudo-code):
guard g1;

run_guarded(g1,thread_a(){
  f = run_guarded(g1,thread_b(){...});
  f.get(); // deadlock
});

Potentially, this could be solved if I made guards re-entrant. However, that
still doesn't quite work.
guard g1, g2;

run_guarded(g1,thread_a(){
  f = run_guarded({g1,g2},thread_b() {...});
  f.get();
});
run_guarded(g2,thread_c() {
  f = run_guarded({g1,g2},thead_d() {...});
}

Now we're back to where we started. Effectively, thread_a() first holds guard
g1, then tries to add g2 while it's running. Thread c does the same thing in
reverse, and we have deadlock. The whole idea of guards was to prevent this.
I think "futurization" is the new paradigm. Why call get() when we have then()?
Maybe we should think of guards as a way to futurize code that needs mutexes.
Why not take the original code, and write it like this?

int hpx_main(boost::program_options::variables_map&)
{
    int i1 = 0;

    // run incr with i1 passed as a reference
    run_guarded(l1, hpx::util::bind(incr, boost::ref(i1)));

    run_guarded(l1,[&i1](){ std::cout << i1 << std::endl; });

    return hpx::finalize();
}

Thanks for the explanation, Steve!

guard g1;

run_guarded(g1,thread_a(){
  f = run_guarded(g1,thread_b(){...});
  f.get(); // deadlock
});

It might be possible to detect this and throw if a future is requested from an 'inner' guard.

I think "futurization" is the new paradigm. Why call get() when we have then()?

I fully agree, however eventually you always need to synchronize with all of your asynchronous code - be it at the end of the computation or the end of an iteration.

Hartmut, it is possible to synchronize with all the async code. One of the properties of the guard is that it is guaranteed to execute the methods attached to it in the order in which they were received. Thus, in the example below, f2 always executes after f1.
run_guarded(g1,f1)
run_guarded(g2,f2)

Additionally, I rely on the fact that HPX doesn't terminate until all running threads terminate. That covers end of program synchronization.
However, yes, what you say about detecting and throwing these errors make sense. That breaks composability, but it's less offensive because it tells you immediately where and when you made the mistake.

What about:
guard g;
vector<double> v = { ... };
parallel::for_each(par, v.begin(), v.end(),
    [&g](double d)
    {
        run_guarded(g, [d]() { ... });
    });

// you'd want this:
g.get_future().get();

Did you intend to use the same guard for each method in the loop? Because the guard is going to effectively sequentialize your loop. Anyhow, we should probably discuss this more when I get back.

Yah, the example is not very convincing... What I meant to show is that we might want to be able to synchronize with all functions run using the guard at any point, not only at the end of the execution.

@stevenrbrandt, @Syntaf: Is this still an issue? Can we close this?

We've talked about the possibility of having guards return a future. For various reasons, this is tricky to do inside the guard code itself. One has to put a promise inside the guard data structures and pass it around, which means one has to be very careful about whether hpx::init has been called, as well as complicating some otherwise simple garbage collection issues. I've done the following, however,
{{{
hpx::future run_guarded_future(guard_set& gs,std::function<hpx::future()> fun) {
std::shared_ptr<hpx::promise> prom{new hpx::promise()};
run_guarded(gs,fun,prom{
hpx::future f = fun();
f.then([prom](auto a){ prom->set_value(); });
});
return prom->get_future();
}
}}}
This should have the capability you want. The difficulty (apart from the wrongness of calling get() and not then()) with using it is that it has noticeably lower performance (at least for the Intersim benchmark).  Maybe we should talk more about what you want to do about this.

@stevenrbrandt This code example will fail as promiose is move-only. Hpwever, if you considered using C++14 you could write:
hpx::future<void> run_guarded_future(guard_set& gs,std::function<hpx::future<void>()> fun) {
  std::shared_ptr<hpx::promise<void>> prom{new hpx::promise<void>()};
  hpx::future<void> f = prom.get_future();
  run_guarded(gs,[fun,prom=std::move(prom)]() mutable {
    hpx::future<void> f = fun();
    f.then([prom=std::move(prom)](auto a){ prom->set_value(); });
  });
  return f;
}

But your code was C++14 before so this shouldn't be an issue...
I'll go ahead and close this now. Please feel free to re-open, if necessary.


Handy Control Growl Notification(Global) cause exception

Handy Control Growl Notification(Global) cause exception when it is shown on the application window.
I want to have a notification in my application but as the notification is created in the viewmodel class
 Growl.Success("xxxxxx");
does not work and I have to use
Growl.SuccessGlobal("xxxxxxxx");
but when notification is shown on the application window and mouse is moved on the notification it causes crash.
I shall to say that it works correctly in previous version for example(3.0.0)
My App.xaml is :
<Application x:Class="growlTest.App"
             xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
             xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
             xmlns:local="clr-namespace:growlTest"
             xmlns:hc="https://handyorg.github.io/handycontrol"
             StartupUri="MainWindow.xaml">
    <Application.Resources>
        <ResourceDictionary>
            <ResourceDictionary.MergedDictionaries>
                <hc:ThemeResources/>
                <hc:Theme/>
            </ResourceDictionary.MergedDictionaries>
        </ResourceDictionary>
    </Application.Resources>
</Application>

My mainWindow is:
<Window x:Class="growlTest.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:local="clr-namespace:growlTest"
        xmlns:hc="https://handyorg.github.io/handycontrol"
        mc:Ignorable="d"
        Title="MainWindow" Height="450" Width="800">
    <Grid>
    </Grid>
</Window>

My MainWindow.cs is:
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Data;
using System.Windows.Documents;
using System.Windows.Input;
using System.Windows.Media;
using System.Windows.Media.Imaging;
using System.Windows.Navigation;
using System.Windows.Shapes;
namespace growlTest
{
    /// <summary>
    /// Interaction logic for MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
    {
        public MainWindow()
        {
            InitializeComponent();
            HandyControl.Controls.Growl.SuccessGlobal("hello");
        }
    }
}

when I run the application you will see the:

but after mouse is moved on the notification you will see:

you should notice that notification should be on the main window until you can see the exception.
while it works correctly in the previous version(3.0.0)

duplicate https://github.com/HandyOrg/HandyControl/issues/867


Upload full volumes for Bobby's volume

Includes scale 0 and native annotations.

directing people to ndio/restful calls for now.


New drone icons for the Echo team

The Echo team need some new drone map icons designed here: https://www.figma.com/file/4bp5iNsiWPh8HSiD98oKzU/Echo-Assets-(new)-✏?node-id=501%3A651
Must be reviewed and adjusted in accordance with our EDS icon style guide. It is possible that these icons will be useful to others as well.

The Echo team need some new drone map icons designed here: https://www.figma.com/file/4bp5iNsiWPh8HSiD98oKzU/Echo-Assets-(new)-✏?node-id=501%3A651
Must be reviewed and adjusted in accordance with our EDS icon style guide. It is possible that these icons will be useful to others as well.

All 3 are ready for review @BirteThornquist @benteljones here:
https://www.figma.com/file/BQjYMxdSdgRkdhKTDDU7L4KU/Assets?node-id=86%3A3729
If approved, it is ready to go @mimarz

LGTM 👍🏻

LGTM 👍🏻

released in<EMAIL_ADDRESS>


feat!: Breaking changes from aztec-packages

Development from Aztec.
Trying to fix the subrepo workflow

The backend encountered an error: ""

:this_is_fine:

The integration test seems to error on generateFinalProof. Not sure why tbh.

I think there is now a hidden aggregation object that is wrecking havoc on the JS packages. generateProof splits the bytes of the public inputs but is unable to regenerate a witnessmap since the ABI doesn't contain the aggregation object, so then when submitting to verifyFinalProof the fields of the aggregation object are lost

nargo prove works because it does not try to separate proof and public inputs from the result bb returns

generateProof splits the bytes of the public inputs but is unable to regenerate a witnessmap since the ABI doesn't contain the aggregation object, so then when submitting to verifyFinalProof the fields of the aggregation object are lost

Deflattening the public inputs being returned is really important from a UX perspective when working with proofs generated by an untrusted party. We should aim to retain this.

So if I understand it correctly the size of a proof was given by
proof_size = public_inputs_field_size + plonk_proof_zero_public_inputs_size

where the aggregation object was included in the public inputs. We knew proof_size and plonk_proof_zero_public_inputs_size so could work out public_inputs_field_size, which allowed us to snip off the right number of fields for the public inputs.
We now have the proof size given by
proof_size = public_inputs_field_size + num_proof_verifications * aggregation_object_size + plonk_proof_zero_public_inputs_size

We should be able to keep the current behaviour if we just explicitly tell the backend how many public inputs we want.
This is assuming that the aggregation objects are placed after the rest of the public inputs however.
If we inject the

I think the only issue was that we got:
original_proof => public_inputs + aggregation object + actual_proof
Now we convert it to proofData => witnessmap + actual_proof
But! the witnessmap only contains public_inputs, since the abi doesn't know anything about the aggregation object
Now we want to regenerate original_proof, so we serialize the witness map again, and now our reconstructed original_proof => public_inputs + actual proof
And then we error because our proof doesn't contain the aggregation object anymore

So what I have made as a workaround is that ProofData contains the actual public inputs vector and the witness map just for user convenience. We always work with the actual public inputs vector to reconstruct proofs and everything and the user can have the witness map for convenience or the full public inputs vector. But yeah the fact that we have some public inputs that the abi doesn't know anything about is not great

I think inside  generateProof you can just check whether makeEasyToVerifyInCircuit is true and that should determine how you split the proof. The idea with moving the aggregation object outside of the ABI was so that the user did not have to know about it. However, we do still need to handle it.

I think inside generateProof you can just check whether makeEasyToVerifyInCircuit is true and that should determine how you split the proof. The idea with moving the aggregation object outside of the ABI was so that the user did not have to know about it. However, we do still need to handle it.

But it doesn't depend on the circuit being verified inside another, but rather if the circuit is verifying another right? I think the makeEasyToVerifyInCircuit is for circuits verified inside another

But it doesn't depend on the circuit being verified inside another, but rather if the circuit is verifying another right? I think the makeEasyToVerifyInCircuit is for circuits verified inside another

Ah yes you are right. Then yeah we will have to do what you started where we keep the public inputs vector for the output aggregation state even though it is not part of the witness map.


Too many values to unpack

Running this on a NetGear WNDR3700v3 and receiving the following error after specifying login details and IP address in my configuration.yaml.
16-01-02 02:01:48 homeassistant.components.device_tracker: Error setting up platform netgear
Traceback (most recent call last):
  File "/usr/local/lib/python3.4/dist-packages/homeassistant/components/device_tracker/__init__.py", line 124, in setup_platform
    scanner = platform.get_scanner(hass, {DOMAIN: p_config})
  File "/usr/local/lib/python3.4/dist-packages/homeassistant/components/device_tracker/netgear.py", line 36, in get_scanner
    scanner = NetgearDeviceScanner(host, username, password)
  File "/usr/local/lib/python3.4/dist-packages/homeassistant/components/device_tracker/netgear.py", line 59, in __init__
    results = self._api.get_attached_devices()
  File "/home/pi/.homeassistant/lib/pynetgear/__init__.py", line 77, in get_attached_devices
    ip, name, mac, link_type = info[1:5]
ValueError: need more than 3 values to unpack

I took liberty to do a print before the 3 value unpack and received the following output of the contents of info the line before the exception
>>> print(ng.get_attached_devices())
['12@1', '<IP_ADDRESS>', '--', '28:C6:8E:CE:9C:AB']

Not sure what the data should be to make a working patch here.  Let me know if I can help debug further.

Looks like an off by one error as the data you logged shows signal, ip, name, mac. Do you by any chance have device names that contain @ in your network?

There are no devices in my home network that have an '@' in the device name.

Fix is here.
http://paste.ubuntu.com/14371023/
Tried to figure out generating a PR to make it more proper but the TLDR is sanitized more input checking length of info and if info exists.  Zero'd out link_rate variable and NULL'd link_type in my edge case.  Tested and working on latest pip3 install of homeassistant.

Version 0.3.1 has been published to PyPi including your fix.

Did some more digging and it appears that the first entry aligns with 'wireless' or 'wired'.  Is link_type variable typically assigned to which interface the associated device is connected over, like wired/wireless?  If so I can generate a new PR assign it the correct value instead of null'ing it out.

I only have the R6300 to test with but I get both link type and speed. my response


Populate server_name for sockaddr resolver

Simply use the path of the uri given to the resolver upon creation. This change is mainly motivated by the use of the sockaddr resolver in the grpclb tests.

Thanks. PTAL.


Improper word choice as well as Improper formatting in ReadMe.md

I found some Improper word choice as well as Improper formatting for that I have made changes and corrected that so, now It will be easier for everyone to read.
in pr https://github.com/accordproject/cicero-ui/pull/195
We need to change this in all

Also in https://github.com/accordproject/markdown-transform/pull/97

Also in https://github.com/accordproject/ergo/pull/692

Also in https://github.com/accordproject/ergo/pull/692 ,
https://github.com/accordproject/concerto/pull/145 ,
https://github.com/accordproject/markdown-transform/pull/97

https://github.com/accordproject/cicero-template-library/pull/295

Let me know when you have a PR in for all main repos and we can review all at once.

@irmerk I think I have done for major repos listed
Please let me know any other Repos for correcting it's README

Docs for the following repos need to be consistent, so I don't want to be reviewing piece meal PRs, rather all of them at once to keep consistent:

markdown-transform
ergo
cicero
concerto
markdown-editor
cicero-ui
cicero-template-library
models
techdocs
cicero-vscode-extension
slack-app
concerto-ui
template-studio-v2

Okay @irmerk I will do for all

Progress:

Repo Name
README.md
DEVELOPERS.md

markdown-transform
Done
Done

ergo

Done

cicero

concerto

markdown-editor

cicero-ui

cicero-template-library
Done
Done

models

techdocs

cicero-vscode-extension

slack-app

concerto-ui

template-studio-v2

@irmerk I have updated all the repos Please merge


Look for someone interested in joining as QA

@novellac mentioned it would be helpful to have someone testing our work, and it would be a good opportunity for interested/early in their career.
@alliequintano can act as a mentor for a QA.
Want our QA to focus on:

accessibility

Does anyone have someone in mind or have some networks to reach out to?

I think we may have discussed last week, but just for reference here: we don't have any requirements in terms of experience for having someone join us as a QA. If you know someone interested in tech but who does not have technical experience (or maybe doesn't know yet what role they would/could perform), this opportunity would be a good entry point. We do want the QA to develop some expertise in accessibility requirements and testing, so the person should be interested in that.

We found Cherry Cola!!! Welcome Nia as QA! ❤ 🍒


Geometry crossing dateline disappears in wrapped view.

I unwrap the coordinates so that they extend past the -180, 180 range. Seems like an issue with checking geometry extents.

Also some issues with geometry being clipped when extent gets large.

You should add a at the 0 meridian crossing to every segment that crosses it. Then everything should work as expected.

Add a ... coordinate? I think you skipped a word. I tried adding a coordinate right at the crossing, but it makes no difference.  The only solution I see at the moment is to split it up into multiple features / geometry. But this complicates things with interactions and more complicated polygon geometries that I also use.
What to do in this situation for example:

Sorry, I meant you have to split the geometry at the 180 meridian crossing. You can still have it in a single feature, but with a MultiLineString geometry.

I'm a novice at the internals of the openlayers source, but I've partially fixed this by using the extent of the source instead the extent of the projection when calculating the wrapped replays inside the renderer composeFrame method. It seems to assume all geometry lies within the projection extent, which in my situation is not the case.  I have no idea what other consequences this might have.

Here's my solution to all the issues I'm having, https://github.com/illogicz/ol3/commit/c87a8398b2446223ffb8dd24009dcbfae364dba1

This bug is a little bit more extensive.
Crossing the left dateline, if you are plotting points from <180 to >180, it hides the line. If you are plotting points >-180 to <-180, it doesn't hide line.
Crossing the right dateline, the issue is reversed. So you would be able to see the >180 plotted line. But not the <-180 plotted line.
Creating a multiline geometry with points at the meridians doesn't help. Nor does just adding the points.
MultiLineString: https://codepen.io/anon/pen/EwgpPP?editors=1011
LineString: https://codepen.io/anon/pen/pWEZeM?editors=1011
Zoom in on the -180 meridian and the top line will disappear.
Zoom in on the +180 meridian and the bottom line will disappear.

if you convert the top line into a >-180 to <-180 line, it will show on the left meridian but not the right when you zoom in.
https://codepen.io/anon/pen/ZXpjjW?editors=1011

@raiyni I had to revert your pull request, because it caused a severe regression.
Here is how you properly split your geometries at the 180 meridian: https://codepen.io/ahocevar/pen/gXJeqQ.

Sure but that doesn't help with polygons and circles. What about adding an option to do look behinds and aheads defaulted to false?

Vector tiles do not have this issue, so you might want to consider using those.


fix shipment dependency

Please also add a template here then, this is needed to be able to review.

All issues fixed and Pr overview is ready as well as Release group


Introducing prepare and finish methods

Hi there,
These two methods are very helpful in case someone wants to intercept the request payload or response payload, before or after method execution.
This approach is already used in Tornado: https://github.com/tornadoweb/tornado/blob/6e266a3/tornado/web.py#L1509-L1534
Please be so kind to accept my pull request. Thanks in advance!

Coverage increased (+0.4%) to 71.707% when pulling 16202e43ebc49f014763d3dc55bac3a691ccaadf on puentesarrin:prepare_finish into a6ca79cd8d00206b53536e2e85734e857fb6782a on bodbdigr:master.


Vips::Error: composite: there must be X blend modes

I have a strange problem with ruby-vips on my mac.
If I execute the same piece of code in the ruby shell it works but in the rails console it throws an error.
For testing purposes I just made five copies of the same image and I try to use composite like this:
path = '/path/to/files/prod'
(1..5).each do |i|
	imgs.push(Vips::Image.new_from_file path+i.to_s+'.jpg')
end
res = Vips::Image.composite(imgs,2)
in the rails console I get:

Vips::Error: composite: for 5 input images there must be 4 blend modes

I just can't figure out why it should behave differently in the rails console.

Could they be picking up different version of the gem? Maybe try uninstall all ruby-vipes you can find, then just reinstalling the latest.
It used to insist you gave it N-1 blend modes even if they were all the same. Only the latest version lets you specify a single mode for all the blends.

Yes, that was it... now that I rebooted the Ruby shell and the Rails console are both picking up version 8.6.2 and it's working as expected. Thanx


JsonResource response wrong "Associative Arrays" when use number for key

Laravel Version
10.22.0
PHP Version
8.2.7
Database Driver & Version

Description
Get wrong response when response from with JsonResource . An "Associative Arrays" that have key as number will response in "Indexed Arrays".
Steps To Reproduce
$associative_array = [
    2 => ['x' => 1],
    3 => ['x' => 1],
    6 => ['x' => 1],
];

return new JsonResource($associative_array);

return response()->json($associative_array);

In order to preserve the keys you should create a custom resource class and define a preserveKeys property on it like this:
class MyResource extends JsonResource
{
    protected bool $preserveKeys = true;
}

More info here: https://laravel.com/docs/10.x/eloquent-resources#preserving-collection-keys


Make docker service async

Just use asyncio.to_thread or something

To be honest, it's not really worth it IMO, not a lot is gained in this change.
I guess I could use https://github.com/aio-libs/aiodocker


Add OpenSearch Service Connection support

Service Connection support for Docker Compose and Testcontainers.

rebased and merged at 37c222a5f35cfa685dccf9549004ba50988f7211


YahooDownloader

Describe the bug
YahooDownloader function used to work perfectly to download a list, however today it is not working anymore.
To Reproduce
Steps to reproduce the behavior:
dax = [
'SIE.DE',
'BAYN.DE',
'RWE.DE',
'ZAL.DE',
'HNR1.DE',
'ALV.DE',
'IFX.DE',
'FRE.DE',
'DBK.DE',
'EOAN.DE',
'DTE.DE',
'CON.DE',
'AIR.DE',
'BEI.DE',
'VNA.DE',
'BAS.DE',
'SHL.DE',
'HEI.DE',
'DB1.DE',
'BMW.DE',
'DTG.DE',
'1COV.DE',
'VOW3.DE',
'MTX.DE',
'ADS.DE',
'SY1.DE',
'P911.DE',
'DHL.DE',
'MRK.DE',
'ENR.DE'
]
Select start day data scraping ( future 5 years back)
TRAIN_START_DATE = '2019-03-08'
TRAIN_END_DATE = '2023-09-08'
TRADE_START_DATE = '2023-09-08'
TRADE_END_DATE = '2024-03-08'
df_raw = YahooDownloader(start_date = TRAIN_START_DATE,
end_date = TRADE_END_DATE,
ticker_list = dax).fetch_data()
Expected behavior
a dataframe with stock list- dax OHLCV

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: macOs 15.0 (24A335)
Browser safari Version 18.0 (206<IP_ADDRESS>.6)
Version [e.g. 22]

Same issue. Its broken for the tutorials too.
Temporary hack:
TOP_BRL =["NVDA", "INTC", "AMD", "GOOGL", "MSFT", "AAPL", "AMZN", "BRK-B", "TSLA"]

# Initialize an empty list to store DataFrames
df_list = []

for ticker in TOP_BRL:
    # Fetch data for each ticker
    portfolio_raw_df = YahooDownloader(start_date='2011-01-01',
                                       end_date='2022-12-31',
                                       ticker_list=[ticker]).fetch_data()
    # Append the fetched DataFrame to the list
    df_list.append(portfolio_raw_df)

# Concatenate all DataFrames row-wise
portfolio_raw_df = pd.concat(df_list, axis=0, ignore_index=True)

# Display the concatenated DataFrame
print(portfolio_raw_df)

Maintainers, can I take a stab at fixing this issue?

I just added a line below to the fetch_data() function in yahoodownloader.py and it works.  See screenshot below.  After adding that line, you'd have to re-import the package.
temp_df.columns = ['Adj Close', 'Close', 'High', 'Low', 'Open', 'Volume']

Hi there! I noticed that the issue comes from the downloaded dataframe having multi-level column names, while it’s expecting just a single level.
Adding temp_df.columns = temp_df.columns.droplevel(1) between lines 53 and 54 of  yahoodownloader.py should fix it.
Hope this helps!

https://github.com/AI4Finance-Foundation/FinRL/blob/083b9dccba415a555e7de13c70a90e1e921f944c/finrl/meta/preprocessor/yahoodownloader.py#L53-L54

@kano5266
Does it work now?
if it works, could you pls submit a PR for it?


⚠ Bofrost.fr has degraded performance

In 52785e2, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1073 ms

Resolved: Bofrost.fr performance has improved in 4103a37.


Deprecate Wagtail welcome page

This page can currently only be seen by those users with Django admin permission (developers and admins). Those users don't always get sent to the page upon logging in, but only when logging in with certain ?next=/something/ parameters in the login URL.
Besides that, the content and design of this page has some issues, so we've decided to remove it. This PR does so.
See platform#3800 for background.
How to test this PR

Login and logout and confirm that the normal flow works correctly.
When logged out, try to visit a page like http://localhost:8000/admin/template_debug/jobmanager/job_listing_list/. You should get redirected to http://localhost:8000/login/?next=/admin/template_debug/jobmanager/job_listing_list/, and then, upon logging in, go to the right page.
When logged out, try to visit a page like http://localhost:8000/admin/pages/319/edit/. You should get redirected to http://localhost:8000/login/?next=/admin/pages/319/edit/, and then, upon logging in, get redirected to http://localhost:8000/admin/.  <--  this is the case affected by this PR! Previously this is how you would end up at /login/welcome/. This is actually a bug, I think, as there's no reason why we shouldn't get to the right page, but I am deliberately not addressing that in this PR.

Checklist
[Feel free to delete any checkboxes that are not applicable to this PR.]

[x] PR has an informative and human-readable title

PR titles are used to generate the change log in releases; good ones make that easier to scan.
Consider prefixing with the feature/section you're addressing, e.g., "Mega Menu: fix layout bug" or "Paying for College: Update content on Repay tool"

[x] Changes are limited to a single goal (no scope creep)
[x] Code follows the standards laid out in the CFPB development guidelines

Would reeeeeeally like to get the broken next redirect bug working, though. (I assume it's related to our custom check_permissions?

Yes. I would also like to remove this custom check entirely.


fix(deps): Switching to exact dependencies

:tada: This PR is included in version 1.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Extending formatters to include ending position when available

Feature request
Is your feature request related to a problem? Please describe.
I see that only in the json formatter, there is provision for an end location. It would be helpful if other formatters or new formatters were extended to include ending position when that is available.
Describe the solution you'd like
In https://github.com/eslint/eslint/issues/11176 I discuss some possiblities for extending the "stylish" format.
Describe alternatives you've considered
Finding a package that has provision for this. Or writing a custom formatter that isn't contributed back but just used in the tool I am working on.

@rocky what would you want to use this end position information for? The formatters such as stylish intentionally omit the end position because, in addition to the points nzakas made in the linked thread, it's not really necessary for most users. For most use cases it's wasted space on the terminal.
The JSON formatter does have an end location because, unlike most other formatters, it's intended for consumption by code.

it's not really necessary for most users

I find it helpful obviously since that's why I mentioned it.
How does one determine that it is not necessary for most users? Is the column number necessary for most users? Decades ago this level of precision wasn't around and people got by. So how is it that nowadays that we are comfortable with adding a starting column position, but insist that this clearly is enough, even when more detail is available?

The JSON formatter does have an end location because, unlike most other formatters, it's intended for consumption by code.

The argument against  extending in the link cited is that it might mess up tools that parse the output. In other words, in some cases that output is consumed by code. And one of the points of this project is that the output format is the way it is so it can be parsed to position editors at the point of error.
Conceivably those tools that parse this could also make use of the end location. The editor I use is able to highlight a region of code.

I find it helpful obviously since that's why I mentioned it.

Lol, yes, but why is it helpful? What about the output becomes easier when you also have the ending position?

How does one determine that it is not necessary for most users?

Excellent question. I certainly don't have the ability to answer it. But - generally, TSLint and other CLIs tend to try to keep output as information-dense as possible. In theory we could add loads of information that's not there now, but it would make it harder for users to visually scan through the output and see what's relevant. Unless there's a documented reason why something is worth the visual space, it's probably not worth putting it in.
I don't intend these responses to be blunt arguments against adding it in, by the way. There just should be a good, documented reason why changes are justified before these happen.

but why is it helpful? What about the output becomes easier when you also have the ending position?

If you look at my github projects you'll see I have worked a lot on both debuggers and precise location reporting. Why? Well, because I make a lot of mistakes. I don't do this as a (very labor intensive) academic excercise.
So I guess I have a particular insight into this that perhaps others don't.The reason an error message is an error message is because it is, well,  unexpected or suprising. So yes, you could give me the starting spot and maybe for a number of situations that would be enough. But as someone who makes a lot of mistakes I might make a mistake in understanding the thing you are talking about too.  Here is a contrived example. Suppose I get this error
Type error

      x = a * b + c
          ^

I look and see that a and b are both let say integers so I don't get it. But suppose the error was that c is a string.  Well if you had something like
Type error

      x = a * b + c
          ^^^^^^^^^

as opposed to:
Type error

      x = a * b + c
          ^^^^^

that would more clearly indicate that we were talking about the expression of a*b with c.
Let me give another analogy. In English prepositions are often used and that is good enough for most people. But for some there can still be confusion. A number of jokes are based on switching the meaning of what prepositions refer to.
Therefore, avoiding prepositions while not necessary for most people, is helpful for some. And this leads me to what feels like a little bit of hypocracy given that tslint is in the linting business.
A number of the linting rules are based on this kind of principle like I just mentioned where you should aviod the use of a preposition because it might confuse someone. Specifically there are rules involving the use a consistent indenting scheme or when optional semicolons and commas are not needed or should be used. If I understand this correctly the motivation behind that is to reduce the possibilty of any extra cognative effort or the possibility of a confusion.
But when it comes to how the linting tool reports what's there is now this different point of view, well, it's good enough for most people to figure what I mean.
Now as for the extra verbiage in giving a range. You could get by with an additional 2-4 characters which would cover most of the cases.  So instead o 12:4 for "line 12 column 4" you might have 12:4.6 to mean "line 12 column 4", for a length of 6 characters.

It sounds like the problem you're trying to address is that the stylish reporter isn't precise enough for some errors. Is that common from what you've seen?
Would the codeFrame reporter do it for you?
src/components/Payment.tsx
Parentheses are required around the parameters of an arrow function definition (arrow-parens)
  21 |     public componentDidMount() {
  22 |         this.input.focus();
> 23 |         loadStripe().then(Stripe => Stripe.pay());
     |                          ^
  24 |     }
  25 |
  26 |     public render() {

...or do you want something between the conciseness of stylish and the specificity of codeFrame?
Alternately, how would you propose the stylish formatter's output be changed?

It is about precision, not the format used to show location. An interval conveys more information than a single positon does.

Alternately, how would you propose the stylish formatter's output be changed?

Both here and in the link I suggested possibilities. The simplest and shortest one is to add a dot followed by a length of characters. For example: 12:4.6 would  mean "line 12 column 4", for a length of 6 characters.  I think I have seen this used elsewhere. However if there is existing art that is more widely used, I'd go with that instead.
I haven't focused on that part because it doesn't make sense to investigate this when there has already been this much resistance over its inclusion.

I think if there are specific errors you’re coming up against that are unclear without an interval, that is probably a failure on the rule description.
As mentioned in the eslint issue, any changes to formatters at this point would be a breaking change. Perhaps a change like #3783 may be helpful here, in making it easier to use your own formatter?
Also happy to leave this open to see what the community thinks - if there’s sufficient support then we may want to look into making this change in a future major version.

In your example, you would also have a starting character, no? That should give you a pretty good place to start looking.
To note: I'm not necessarily against the change - just that it's a big one to make, and I don't want to make that decision lightly.
You mentioned that you're writing a parser of some type. Is there a reason you can't use the JSON formatter? I assume this is what the vscode react plugin uses, as it is able to highlight the full width of the error. It's not intended that any of the other formatters should be read programmatically, if that was your intention.
All of that said, if the current selection of formatters still doesn't meet your needs, I think the best bet is probably to develop one yourself, and if we receive substantial community support for this change, we can consider merging it in to TSLint.

the best bet is probably to develop one yourself, and if we receive substantial community support for this change, we can consider merging it in to TSLint.

☝ that!


Add pointwise_divide function

As discussed in issue #1579, there was no function on dlib to perform pointwise division of matrices.
This PR adds that function and corresponding unit tests.
Here are some performance tests I've ran, all of them performing a pointwise division. The number at the end is the number of arguments used for each test.
With a matrix of size 2016x1512
pointwise_multiply2           17 ms
pointwise_multiply3           20 ms
pointwise_multiply4           23 ms
pointwise_division2           15 ms
pointwise_division3           18 ms
pointwise_division4           22 ms
With a matrix of size 4032x3024
pointwise_multiply2           70 ms
pointwise_multiply3           81 ms
pointwise_multiply4           92 ms
pointwise_division2           62 ms
pointwise_division3           72 ms
pointwise_division4           89 ms
We can see an improvement in performance between ~6% and ~12%.
Also, I've found a place were pointwise_multiple was used to actually perform a division, in the affine layer, so I've replaced it.

I've changed identation to be exactly as in pointwise_multiply, and replace all tabs I've found. Sorry about that.
Anyway, have you considered to add some kind of code formatter to the project? For example, I've heard of some like clang-format, but I've never used any of them. Maybe adding something like that to dlib would be another good PR? :)


Clean up some file names

Some file names were added that do not seem to align with the existing approach
Fix these up (just rename) and add a docs section to help as a reference for the future

Thanks @zerolab


bin unable to update itself  in Powershell

Issue: bin.exe is not able to update itself.
OS: Microsoft Windows 11 Pro (10.0.22631 N/A Build 22631)
Shell: Powershell 5.1.22621.2506 & Powershell 7.4.1
PS C:\Users\admin\bin> bin update bin.exe --debug
   • debug logs enabled, version: 0.17.3
commit: 9da887da9ce476a54e25828bad962a1cae2da17c
built at: 2024-01-31T03:39:37Z
built by: goreleaser

   • Config directory is: C:\Users\admin\.bin
   • Download path set to C:\Users\admin\bin
   • Checking updates for C:\Users\admin\bin\bin.exe
   • Getting latest release for marcosnils/bin
   • Found new version v0.17.4 for C:\Users\admin\bin\bin.exe at https://github.com/marcosnils/bin/releases/tag/v0.17.4
   • C:\Users\admin\bin\bin.exe v0.17.3 -> v0.17.4 (https://github.com/marcosnils/bin/releases/tag/v0.17.4)

Do you want to continue? [Y/n] y
   • Getting v0.17.4 release for marcosnils/bin
   • Removing bin_0.17.4_windows_arm64.exe (URL https://api.github.com/repos/marcosnils/bin/releases/assets/155860489) with score 41 lower than 46
   • Keeping bin_0.17.4_windows_amd64.exe (URL https://api.github.com/repos/marcosnils/bin/releases/assets/155860491) with highest score 46
   • Removing bin_0.17.4_windows_386.exe (URL https://api.github.com/repos/marcosnils/bin/releases/assets/155860486) with score 36 lower than 46
   • Removing bin_0.17.4_linux_arm64 (URL https://api.github.com/repos/marcosnils/bin/releases/assets/155860487) with score 6 lower than 46
   • Removing bin_0.17.4_linux_amd64 (URL https://api.github.com/repos/marcosnils/bin/releases/assets/155860488) with score 11 lower than 46
   • Removing bin_0.17.4_linux_386 (URL https://api.github.com/repos/marcosnils/bin/releases/assets/155860485) with score 1 lower than 46
   • Removing bin_0.17.4_darwin_arm64 (URL https://api.github.com/repos/marcosnils/bin/releases/assets/155860493) with score 16 lower than 46
   • Removing bin_0.17.4_darwin_amd64 (URL https://api.github.com/repos/marcosnils/bin/releases/assets/155860492) with score 21 lower than 46
   • Checking binary from https://api.github.com/repos/marcosnils/bin/releases/assets/155860491
   • Starting download of https://api.github.com/repos/marcosnils/bin/releases/assets/155860491
15.15 MiB / 15.15 MiB [----------------------------------------------------------------------------------------------------------------------------------------------------------------] 100.00% 5.97 MiB p/s 3s
   • Overwrite flag set, removing file C:\Users\admin\bin\bin.exe

   ⨯ command failed            error=Error installing binary remove C:\Users\admin\bin\bin.exe: Access is denied.

Expected outcome: bin.exe updated to latest version

I have the same issue on Windows 10 Pro, and can confirm that it also occurs in cmd as well.

hey! thx for reporting. I'll try to get a windows VM to test this out :pray:

Still having the same issue. Any updates on this?


Fix folder structure naming

They're all caps and it's annoying me.

I tried doing this but it was very tricky. git and windows are like both half-case-sensitive. I tried it with git mv using a temp name but it did not go well. might have to commit the temp names or something.

might have to commit the temp names or something.
Yeah that's probably the only way

Fixed with https://github.com/MNES-Emulator/MNES/commit/69138f91690c96965c0e8c323e3f9a34e0dfa072


route_settings unsupported in HTTP API Gateway

Description
Current (1.1.0) version of the api-gateway module has this block:
  #  # bug - https://github.com/terraform-providers/terraform-provider-aws/issues/12893
  #  dynamic "route_settings" {
  #    for_each = var.create_routes_and_integrations ? var.integrations : {}
  #    content {
  #      route_key = route_settings.key
  #      data_trace_enabled = lookup(route_settings.value, "data_trace_enabled", null)
  #      detailed_metrics_enabled         = lookup(route_settings.value, "detailed_metrics_enabled", null)
  #      logging_level         = lookup(route_settings.value, "logging_level", null)  # Error: error updating API Gateway v2 stage ($default): BadRequestException: Execution logs are not supported on protocolType HTTP
  #      throttling_burst_limit         = lookup(route_settings.value, "throttling_burst_limit", null)
  #      throttling_rate_limit         = lookup(route_settings.value, "throttling_rate_limit", null)
  #    }
  #  }

TF documentation tells that stage's routes can have custom settings ( see "route_settings" in https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/apigatewayv2_stage)
As such, I can not do the following instruction, it has no effect:
module "api_gateway" {
  source  = "terraform-aws-modules/apigateway-v2/aws"
  version = "~> 1.1.0"
  protocol_type = "HTTP"
 # ...  options abriged...
  integrations = {
    "GET /auth/myinegration" = {
      #...abriged...
      # THESE INSTRUCTIONS ARE IN https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/apigatewayv2_stage but they are not supported by your module due to the commented bit quoted above.
      throttling_rate_limit=10
      throttling_burst_limit=10
    }

The issue https://github.com/terraform-providers/terraform-provider-aws/issues/12893 seem to be closed but it is hard to say if underlying issue is been fixed.
I am interested into setting throttling_burst_limit / throttling_rate_limit in particular.

I am also curious about the state of mentioned issue.
It would be great to use data_trace_enabled option for default_route_settings resource. :)

This issue has been resolved in version 5.0.0 :tada:


Need support for floatArray in SDKTypeDefinition

We need support floatArray type in SDKTypeDefinition as float64[] type in go sdk.

Expected solved in #62


Single recommendation page: disable/enable recommendation for particular cluster feature

The patch allows users to select a group of clusters that they will disable from the affected clusters table. Also, if the recommendation is disabled for some clusters, the appropriate card is rendered, where users can click and see the list of disabled clusters.
Fixes https://issues.redhat.com/browse/CCXDEV-6310.

[x] Comply with the mock-ups https://marvelapp.com/prototype/g70a1he/screen/79082637
[x] Make the affected clusters table items selectable
[ ] Tests coverage

@doruskova, the patch is missing tests and minor implementation fixes, however, I want to ask you to review the feature itself. If the screenshots are not descriptive enough, please, let me know.

@gkarat Yes! it looks fine. Just the toast alert should say: Recommendation disabled instead of "successfully".


Use dunder prefix convention to mark methods and attributes as private

The Python implementation should follow the reference implementation and implement private methods and attributes as such using the appropriate mechanisms.
If this is a desired extension, I would provide the implementation for it.

Same as #3, could use some PRs to fix whatever needs adjusting 👍

https://github.com/sqids/sqids-python/pull/8


Clarify documentation of the auto_reload option

Thanks a lot for this lib! I'm using it to render component templates that will also be rendered by TwigPHP, and it's working well so far.
I've had the exact same issue as #194 (with the same use case: an express middleware to use with a webpack dev server) and spent a good hour trying every option and possible fix I could think of. My fix was indeed to recreate the Twing environement on every request.
The current documentation says:

auto_reload boolean
When developing with Twing, it’s useful to recompile the template whenever the source code changes. If you don’t provide a value for the auto_reload option, it will be determined automatically based on the debug value.

Reading through #194 and https://github.com/twigphp/Twig/issues/2697 it seems auto_reload only affects the cache behavior, and can be used to bypass the cache if a template file was changed, but there is another in-memory cache attached to the environment that it does not affect at all? How could that be reflected more clearly in the docs, maybe with a warning for long-lived Twing environments?

Hi @fvsch,
I couldn't agree more: this needs to be clarified. As you discovered, the auto_reload option only bypass the explicit cache (the one passed as parameter to the environment) but not the in-memory cache of the environment. You then have to recreate a new environment on each run loop to benefit of it. It makes sens but it's not obvious at all.
I'll clarify this in the doc. Feel free to provide a PR if you feel like taking care of this.

You can clear the cache using this code:
env.on('template', () => env.loadedTemplates = new Map())

It works in JavaScript, not in Typescript. I plan to have loadedTemplates publicly exposed in the future.


[HOSTED] Generated Issue: ConnectionException

{
"status": 500,
"type": "ConnectionException",
"message": "Failed to communicate with Redis.",
"error": null,
"report_url": "https://github.com/jikan-me/jikan-rest/issues/new?title=[HOSTED]+Generated+Issue%3A+ConnectionException&body=Please+fill+out+the+details+below.

**Summary%3A**

**Steps+to+reproduce%3A**

+%23%23%23+Additional+Details+
+**Jikan+Parser+Version**%3A+```v2.16.2%404e3781e1617047921d20acc7bb4390bc8e83502d```
**PHP%3A**+```7.3.21```
**Redis**%3A+``````
**Exception%3A**+```Predis\Connection\ConnectionException```
**Code%3A**+```10061```
**Message%3A**+```+```
**Trace%3A**+```C%3A\xampp\htdocs\jikan-rest\vendor\predis\predis\src\Connection\AbstractConnection.php+on+line+155```
**Request%3A**+`GET+%2F`
"
}
Summary:
Steps to reproduce:
Additional Details
Jikan Parser Version: v2.16.2@4e3781e1617047921d20acc7bb4390bc8e83502d
PHP: 7.3.21
Redis: ``````
Exception: Predis\Connection\ConnectionException
Code: 10061
Message:
Trace: C:\xampp\htdocs\jikan-rest\vendor\predis\predis\src\Connection\AbstractConnection.php on line 155
Request: GET /

Did you install and start Redis?
sudo apt install redis
sudo service redis start

i have setup use redislabs, with detail like that

REDIS_HOST=redis-16891.c1.asia-northeast1-1.gce.cloud.redislabs.com
REDIS_PASSWORD=password
REDIS_PORT=16891

but i get an error

the api is down right now, it is not your fault.
check https://status.jikan.moe/

@westheme Redis port is usually 6379. Check that maybe and other auth details whether they're correct. Or whether your redislabs server allows external connections (I haven't used redislabs so I can't say much)
And try authenticating into Redis server through a terminal on your end with the provided details and see whether that works first.
Jikan throws a Failed to communicate with Redis when it can't connect or is provided invalid credentials (if it's auth based).
Also check error logs on your jikan setup for details. It's in /jikan-rest/storage/logs/daily/*.log
@Tacaton Unrelated to this issue. They're self hosting the API. Can be noted by the [HOSTED] prefix in the title.

Closing due to inactivity. Please feel free to reopen the issue if you face the same issue again.

Closing due to inactivity. Please feel free to reopen the issue if you face the same issue again.


Merge pull request #2 from hyperledger/master

Description

Motivation and Context

Fixes #
How Has This Been Tested?

Checklist:

[] I have added a Signed-off-by.
[] I have either added documentation to cover my changes or this change requires no new documentation.
[] I have either added unit tests to cover my changes or this change requires no new tests.
[] I have run golint and have fixed valid warnings in code I have added or modified. This tool generates false positives so you may choose to ignore some warnings. The goal is clean, consistent, and readable code.

Signed-off-by:
Merging from master

this seems accidental, in any event, we have moved to Gerrit. Closing.


Feature Request - Displaying Multiple Points with the Same Datetime

Hi,
This is a feature request to enable displaying multiple points with the same datetime. For example, suppose I have two distinct markers that share the datetime of "2015-05-18 00:00". I'd like both of these points to be displayed simultaneously. Then on the next step, suppose the markers share the datetime of "2015-05-18 01:00" so they should be displayed simultaneously once again (but the markers with the datetime of "2015-05-18 00:00" should no longer be displayed).
Thanks!

Hi,
I am currently not actively working on this plugin. However, I would be happy to merge a pull request to include this functionality!


Output installing latest echoverse version

Hi,
Just wanted to bring to your attention echoLD and echofinemap had non-zero exit status
Code to install echoverse
remotes::install_github("RajLabMSSM/echolocatoR@echoverse")
Intallation output

Hey, so the echoverse branch is outdated now. All updates are being push to master.
So just use:
remotes::install_github("RajLabMSSM/echolocatoR")

Also, could you please include the full output from the installation messages? It looks like the photo you added is just the very end of it.
It may be that there's an issue with one of the dependencies for those subpackages, but I can't tell without the full messages.

Ok, although I have been trying to install both echoverse, and echolocatoR branches, I thought we still needed to install the echoverse branch. Thanks for clarifying.
I am currently trying to rerun the code to install echolocatoR outside the conda environment, as you mentioned in #102

Sorry, this has been me being clumsy.
Your interpretation in #102 is correct. re-running the code to install and allowing to upgrade is enough.

No worries, thanks for bearing with the growing pains.
I think once all of echolocatoR's functions are stable this should happen less frequently.


[BUG][Kernel] Delta clone does not copy the table comment, only columns

Bug
Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[x] Kernel
[ ] Other (fill in here)

Describe the problem
Delta clone does not copy the table comment to the target table, only the comments related to the columns.
Steps to reproduce
create or replace table test_tbl_comment(id int comment 'PK', name string) comment 'a comment';
insert into test_tbl_comment values (1, 'Andrea'), (2, 'William');
create or replace table test_tbl_comment_dclone deep clone test_tbl_comment;
describe table extended test_tbl_comment_dclone;

Observed results
Only the comment related to id column has been replicated
Expected results
The table comment should be replicated as columns
Environment information
Databricks DBR 15
Willingness to contribute
The Delta Lake Community encourages bug fix contributions. Would you or another member of your organization be willing to contribute a fix for this bug to the Delta Lake code base?

[ ] Yes. I can contribute a fix for this bug independently.
[x] Yes. I would be willing to contribute a fix for this bug with guidance from the Delta Lake community.
[ ] No. I cannot contribute a bug fix at this time.

And BTW this doesn't seem like a kernel issue


In the Angular2 client, a successful login returns to an undefined route

fixed

Hi damienbod. Was update to rc5 the fix to this bug? What should I do to fix this bug, and don't update to rc5?

Hi @ph1lolog1st
my routing was defined incorrect
Greetings Damien

damienbod, very thanks for quick replay. Could you say what exactly was wrong and why, please?

@damienbod, can i fix this bug using router 3.0.0-beta.1?

I found solution: window.location.href = "something". I think it is a poor solution, but it's work.


Default tab title is super-long, only 3 tabs fit the screen

On the most recent version from Releases:

I'm sure there are reasons why it's like this, but it renders tabs unusable for me in the current state: only three tabs fit the screen at a time.
Yes, I realize this has been discussed before and all previous issues were closed, with issue authors often disagreeing with their closing (sometimes a minor related cause was dealt with, but the original issue still stands). Please feel free to close this as well if the whole long default tab title affair is decided WONTFIX.
Yes, I realize one can probably configure the tab title, but I don't consider this as a valid argument. IMO having to configure such a basic thing is a show-stopper when trying out a new terminal.
The screen is a Surface Pro 7 screen if it matters.

Two things, actually:

that tab title was generated by your shell, and is configurable in yuor shell's startup scripts (for bash, that's probably /etc/profile). you can actually configure it to contain anything you want, which is really cool. I know you already enumerated this as an option, but it really is the "best" fix ;P
not all previous issues were closed, actually: this is largely a /dupe of #597 (which is very much still open) :smile:

Well, then please put it in Top of readme, in a popping message box, tooltip or whatever popping into eyes. If I happen to have a long username and a long computer name, Windows Terminal is taxing me by a lot of time figuring out how to reset this. Of course I realize that current tab title is provided by the bash profile, but I don’t think the tab title is informative

I hope that 2 is resolved before 1.0 release

Sorry, https://github.com/microsoft/terminal/issues/597 is vague. It starts by asking for min-width. Then someone suggests it would be good to have max-width, but it is not clearly stated in the OP.
Could you please keep this issue open until something in the underlying tab control gets fixed and then the user-facing problem described in this issue (https://github.com/microsoft/terminal/issues/597 is more about impl details) has an official default solution?

https://github.com/microsoft/terminal/pull/3876 Work seems actively progressing, I hope it gets merged before 1.0 and equal mode is enabled by default. Pretty much everyone would be surprised that WT doesn’t handle tabs like web browser or supports only three tabs visible (without having to reset bash profile env variables)

@vadimkantorov Yes, I agree, three tabs is really bad...
Also if you maximize the window you can't get pass of three tabs (which really sucks).
Until #3876 was not merged, I did override the title of the tab on the terminal settings, using name and suppressApplicationTitle properties on profiles array.

For now, I can have at least 7 tabs.
This was the easiest and cleanly way I found to do that.
I hope this be addressed.


[HDInsight] Fixed OMS operations to be marked as long-running.

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

@olydis This is a bug fix, the service had implemented these operations as long-running. As of today, the terminal status for PUT and DELETE is 202 (Accepted).

I've added 200 as a possible response to fix the violation for now, but I'll make the service fix as part of the next breaking change.

Note that this will influence generated code! If your service does not implement it like that, then it makes no sense to model it differently. Essentially the violation is of the service, so you didn't fix it but muted it, leading to possibly broken SDKs.

That's a fair point. However, what's the recommended course of action in this case? Until the service is fixed, it seems like either the spec must be inaccurate or we'll have to live with the SDK violation.

Accurate Swagger is the first priority. You'll have to live with the violation until it's fixed (or request exception from ARM that your service is allowed to have this behavior forever). :)

Sounds good. I also prefer an accurate spec over masked violations. I've updated the PR accordingly.

No modification for AutorestCI/azure-sdk-for-node

No modification for AutorestCI/azure-sdk-for-ruby

No modification for AutorestCI/azure-sdk-for-python