Traefik pilot : invalid token

Do you want to request a feature or report a bug?

Bug

What did you do?

Just try plugin on Traefik Pilot :)
What did you expect to see?
Make a simple docker-compose with :

version: "3.8"
services:
  reverse_proxy:
    image: traefik:2.3
    restart: unless-stopped
    command:
     --api.insecure=true
     --accesslog
     --log.level=DEBUG
     --entrypoints.web.address=:80
     --providers.docker
     --providers.docker.exposedbydefault=false
     --experimental.pilot.token="xxxxxxxxxxxxxxxxxx"
     --experimental.plugins.blockpath.modulename="github.com/containous/plugin-blockpath"
     --experimental.plugins.blockpath.version="v0.1.2"
    ports:
      # The HTTP port
      - "80:80"
      # The Admin port
      - "8080:8080"
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    networks:
      - web

networks:
  web:
    name: traefik_wan

Token generate on https://pilot.traefik.io/.
Traefik does not start with the following error in debug:
reverse_proxy_1  | 2020/07/20 13:46:45 traefik.go:75: command traefik error: failed to download plugin github.com/containous/plugin-blockpath: error: 400: {"error":"invalid token"}

What did you see instead?
Download plugin on instance
Output of traefik version: (What version of Traefik are you using?)

Version:      2.3.0-rc2
Codename:     picodon
Go version:   go1.14.5
Built:        2020-07-15T20:22:27Z
OS/Arch:      linux/amd64

What is your environment & configuration (arguments, toml, provider, platform, ...)?
# (paste your configuration here)

If applicable, please paste the log output in DEBUG level (--log.level=DEBUG switch)
everse_proxy_1  | 2020/07/20 13:46:45 traefik.go:75: command traefik error: failed to download plugin github.com/containous/plugin-blockpath: error: 400: {"error":"invalid token"}

hello,
You have to remove the quote ":
     --experimental.pilot.token=xxxxxxxxxxxxxxxxxx
     --experimental.plugins.blockpath.modulename=github.com/containous/plugin-blockpath
     --experimental.plugins.blockpath.version=v0.1.2

same problem :(
If i remove plugins line, the instance starts and goes back well in pilot ( with quote

could your try (without quotes) and with a fresh token?

same error

I have also try with yaml config instead cli : same

I reproduce, I trying to find why.

The problem is no fixed, could you retry?

Works :1st_place_medal:


Can't Paste Large Amount of Text - "Uncaught RangeError: Maximum call stack size exceeded"

Prerequisites

[X] I have checked the following:

[X] The problem didn't happen in Safe Mode, but disabling packages didn't change anything.
[X] Followed all applicable steps in the debugging guide.
[X] Checked the FAQs on the message board for common solutions.
[X] I couldn't find my issue. Though, atom reported this issue is already reported, that problem was about saving to network, and I couldn't see any similarity.

Description
Pasting a large amount of text leads to Maximum call stack size exceeded.
Full Stack Trace:
Uncaught RangeError: Maximum call stack size exceeded
/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:833

RangeError: Maximum call stack size exceeded
    at Object.spliceWithArray (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/underscore-plus/lib/underscore-plus.js:362:46)
    at TokenizedBuffer.module.exports.TokenizedBuffer.bufferDidChange (/Applications/Atom.app/Contents/Resources/app.asar/src/tokenized-buffer.js:278:18)
    at /Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:671:36
    at Set.forEach (native)
    at TextBuffer.module.exports.TextBuffer.emitDidChangeEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:670:33)
    at TextBuffer.module.exports.TextBuffer.applyChange (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:664:12)
    at TextBuffer.module.exports.TextBuffer.setTextInRange (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:557:19)
    at Selection.module.exports.Selection.insertText (/Applications/Atom.app/Contents/Resources/app.asar/src/selection.js:480:43)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:3023:31
    at /Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:1161:28
    at TextBuffer.module.exports.TextBuffer.transact (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:828:18)
    at TextEditor.module.exports.TextEditor.transact (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:1594:26)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:1155:24
    at TextEditor.module.exports.TextEditor.mergeSelections (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:2572:18)
    at TextEditor.module.exports.TextEditor.mergeIntersectingSelections (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:2534:35)
    at TextEditor.module.exports.TextEditor.mutateSelectedText (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:1153:19)
    at TextEditor.module.exports.TextEditor.pasteText (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:2996:19)
    at TextEditor.core:paste (/Applications/Atom.app/Contents/Resources/app.asar/src/register-default-commands.js:437:21)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/register-default-commands.js:666:34
    at TextBuffer.module.exports.TextBuffer.transact (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:828:18)
    at TextEditor.module.exports.TextEditor.transact (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:1594:26)
    at atom-text-editor.newCommandListeners.(anonymous function) (/Applications/Atom.app/Contents/Resources/app.asar/src/register-default-commands.js:665:22)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:259:29)
    at CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:59)
    at CommandRegistry.module.exports.CommandRegistry.dispatch (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:160:19)
    at AtomEnvironment.module.exports.AtomEnvironment.dispatchApplicationMenuCommand (/Applications/Atom.app/Contents/Resources/app.asar/src/atom-environment.js:1179:28)
    at EventEmitter.outerCallback (/Applications/Atom.app/Contents/Resources/app.asar/src/application-delegate.js:312:25)
    at emitThree (events.js:116:13)
    at EventEmitter.emit (events.js:194:7)

Steps to Reproduce

Copy a large amount of text (around 4MB)
Paste inside a new tab (with Plain Text syntax)

Expected behavior: You should see the text.
Actual behavior: An exception is thrown and the text is not pasted.
Reproduces how often: Always
Versions
This my Atom version. I see there's also a new 1.17, which the autoupdate doesn't detect yet. I couldn't see any resolved issues that was about this error, so I assume nothing has changed.

Atom    : 1.16.0
Electron: 1.3.13
Chrome  : 52.0.2743.82
Node    : 6.5.0

Additional Information
I don't have any content inside init.coffee, and most of the packages I have are supposed to be used when a specific syntax is present (not for plain text). I tried disabling some of the 3rd party ones, and even some of the core ones, but none of them changed anything. Anyway, the stack trace didn't show any 3rd-party package involved in the exception, so I think there's something extra happening when Atom is not run in Safe Mode.

Thanks for taking the time to contribute!
We noticed that this is a duplicate of #4767. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


[MAS3.3.2] [Labels and Relationship-Settings] Legends are not associated with the check boxes

Actual Result:
Legends are not associated with the check boxes and narrator also not narrating the legends information while navigating to the checkboxes
1.User settings 2. Application updates 3. Data collection
Expected Result:
Legends should be associated with the check boxes and should narrate the legends information while navigating to the checkboxes
1.User settings 2. Application updates 3. Data collection
Repro Steps:
1.Open the Bot Framework V4 Emulator.
2.Navigate to Welcome tab screen and select it, screen gets opened
3.Navigate to the open bot and select it
4.Open bot dialog gets opened and connect to the open bot
5.Live chat screen gets appears
6.Navigate to the Edit ngrok settings and select it
7.Emulator screen gets opened
8.Navigate to the “1. User settings 2. Application updates 3. Data collection” section check boxes
9.Observe the issue
User Impact:
User who rely on AT will not able to get the labels information if the labels are not associated with the checkboxes might get confused while navigating to the Emulator settings pane.
Test Environment:
OS Build: Version 1903(OS Build 18362.239)
Browser: Edge
Application: Bot Framework V4 Emulator
Screen reader: Narrator
MAS Reference Link:
https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={4b95a70d-258c-474a-820e-5c8b79d5c0bf}
MAS3.3.2_Setting.zip

Closing as dupe of #1768


Has this project been abandoned?

Hi!
Has this project been abandoned? Seeing as there hasn't been an update in 15 months, i must ask if this project has been abandoned for good.
Thanks!

Well I'm certainly alive. Adding new assets though, even if they are finished as shown in some issues, is very time consuming with little value.
Any PRs that popup to include new assets or include tooling to greatly simplify the time taken to add assets would definitely be helpful. Though not sure if I will have the time to do so in the near future


nixos/bluetooth: hotfix for stupidity

Motivation for this change
Further to #112493
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg eval


On the terraform-aws-ecs-cloudwatch-autoscaling module, upgrade to v0.5.0 to support terraform v14

Describe the Feature
Upgrade the terraform-aws-ecs-cloudwatch-autoscaling source module from v0.4.2 to v0.5.0 in order to support terraform v14
Expected Behavior
Running a terraform plan on this module in terraform v14 fails because the module is linked to an older version of terraform-aws-ecs-cloudwatch-autoscaling that does not support terraform v14.
Use Case
Utilizing this module with terraform v14
Describe Ideal Solution
Increase the version of the terraform-aws-ecs-cloudwatch-autoscaling source module from v0.4.2 to v0.5.0 in order to support terraform v14 on the label module.
Alternatives Considered
Merge in #81
Additional Context
I would like to utilize this cloudposse module with terraform v14 but am currently blocked from doing so. I could fork this repo but would prefer to utilize your maintained repo. Really appreciate all of the open source work cloudposse does!

No longer needed

No longer needed


Where is Merge(eds...) called?

Where is this actually called? I found it being used in func Merge(eds...), but I don't see this latter being used anywhere
Originally posted by @SweeXordious in https://github.com/celestiaorg/celestia-app/pull/837#discussion_r987355330

Round-trip testing the ability to encode and decode block data is an important thing to test given that each piece of that process is vital to the network. While we might not need to decode an entire block in a single call, we will always need to decode each portion.
Therefore, while we might be able to un-export this function and only use it as a testing helper func, but imo removing it entirely is not a good idea.

is there an action item remaining here? cc @SweeXordious
I'm fine with unexporting the function if that is deemed valuable

Round-trip testing the ability to encode and decode block data is an important thing to test given that each piece of that process is vital to the network

So, it is being used in tests, right?

is there an action item remaining here?

Nope, the issue was just to know if this could be deleted, since we didn't see it being used anywhere.

yes it is part of some of the tests for share encoding, can we close this issue then?

Thanks for the help, I don't think any action is required for this


GH-35425: [R] Tests failures on R < 4.0 due to data.frame conversion

Closes: #35425

@github-actions crossbow submit test-r-versions

@github-actions crossbow submit test-r-versions


Monthly Chat Agenda August 2021 (2021-08-02 and 2021-08-16)

This is the Agenda for the Monthly CRS Chat.
The general chat is going to happen on https://owasp.slack.com in the channel #coreruleset on Monday, 2021-08-02, at 20:30 CET. That's the 1st Monday of the month. A separate issue chat is happening at the same location, same time on Monday, 2021-08-16. That's the 3rd Monday of the month. Please note that we have a CRS calendar (maintained by @fzipi).
Items on the Agenda:  (see previous meetings decisions: here)
What happend in the meantime since the chat last month
Outside development

Google App Armor signed up as a silver sponsor for the CRS project and paid the invoice. This is now confirmed.
A 2nd Gold sponsor has received the invoice, payment is expected in the first week of August.

PRs that have been merged since the last meeting

#2165
#2164
#2159
#2158
#2157
#2155
#2154
#2152
#2148
#2107
#2078

We merged 11 PRs since the last monthly project chat.
Open PRs

#2168 - in need of a quick review, maybe @airween
#2163 - replacing filenames lead to broken tests. Probably some ftw-fu needed here
#2156 - review needed so we can remove ctl:ruleEngine=Off
#2151 - adding more teeth to 932200 (uninitialized variables)
#2149 - cleanup of regexp assembly script(s)
#2067 - better integration needed, volunteer to finish this wanted
#2050 - reviewer keen on ReDoS wanted
#2049 - ModSec asks for input on their idea to introduce the expanded list of JSON CTs as a separate list
#1993 - no agreement on plugin mechanics, @lifeforms and @dune73 disagree
#1975 - Nextcloud REs update, volunteer wanted

Open PRs marked DRAFT or work in progress or needs action

#2136 - review by @franbuehler pending
#2101 - review (tests) by @azurit pending
#2070 - incomplete
#2068 - still actively being updated

Other items

Status of Coraza WAF
Status of dev retreat in October : 7 participants (full or partial), 2 tentative, at least 2 open
What to do at the dev retreat: List of ideas: https://github.com/coreruleset/coreruleset/wiki/Dev-Retreat-Topics

Open Issues - Separate Issues Meeting (Monday, FIXME)
Status of issues covered last month

Issue slot 1: #FIXME
Issue slot 2: #FIXME
Issue slot 3: #FIXME
Issue slot 4: #FIXME
Issue slot 5: #FIXME
Issue slot 6: #FIXME
Issue slot 7: #FIXME
Issue slot 8: #FIXME
Issue slot 9: #FIXME
Issue slot 10: #FIXME

Stats

Covered in chat: FIXME
Closed: FIXME
Pending: FIXME

This month's issues
We generally cover 10 issues per month in a separate issue meeting. Add them as you see fit.

Issue slot 1: #FIXME
Issue slot 2: #FIXME
Issue slot 3: #FIXME
Issue slot 4: #FIXME
Issue slot 5: #FIXME
Issue slot 6: #FIXME
Issue slot 7: #FIXME
Issue slot 8: #FIXME
Issue slot 9: #FIXME
Issue slot 10: #FIXME

How to get to our slack and join the meeting?
If you are not yet on the OWASP Slack, here is your invite: https://owasp-slack.herokuapp.com/ .
Everybody is welcome to join our community chat.
Meeting Agenda Template
Link

Decisions
PRs

#2168 - in need of a quick review - @airween
#2163 - replacing filenames lead to broken tests. Probably some ftw-fu needed here - @fzipi
#2156 - review needed so we can remove ctl:ruleEngine=Off
#2151 - adding more teeth to 932200 (uninitialized variables)
#2149 - cleanup of regexp assembly script(s)
#2067 - better integration needed, volunteer to finish this wanted
#2050 - reviewer keen on ReDoS wanted
#2049 - ModSec asks for input on their idea to introduce the expanded list of JSON CTs as a separate list
#1993 - no agreement on plugin mechanics, @lifeforms and @dune73 disagree
#1975 - Nextcloud REs update, volunteer wanted

Open PRs marked DRAFT or work in progress or needs action

#2136 - review by @franbuehler pending
#2101 - review (tests) by @azurit pending
#2070 - incomplete
#2068 - still actively being updated

Other items

Open Issues - Separate Issues Meeting (Monday, July 19)

Issue slot 1:
Issue slot 2:
Issue slot 3:
Issue slot 4:
Issue slot 5:
Issue slot 6:
Issue slot 7:
Issue slot 8:
Issue slot 9:
Issue slot 10:


Build Fail with 'config.h' file not found

Hi, Thanks for kind blog.
I'm trying to clone and build your project but it's failed.
** BUILD FAILED **

The following build commands failed:
        CompileC /Users/kangsangkwon/Desktop/FaceParty/react-native-opencv-tutorial/ios/build/Build/Intermediates.noindex/React.build/Debug-iphonesimulator/double-conversion.build/Objects-normal/x86_64/fixed-dtoa.o /Users/kangsangkwon/Desktop/FaceParty/react-native-opencv-tutorial/node_modules/react-native/third-party/double-conversion-1.1.5/src/fixed-dtoa.cc normal x86_64 c++ com.apple.compilers.llvm.clang.1_0.compiler
        CompileC /Users/kangsangkwon/Desktop/FaceParty/react-native-opencv-tutorial/ios/build/Build/Intermediates.noindex/React.build/Debug-iphonesimulator/double-conversion.build/Objects-normal/x86_64/strtod.o /Users/kangsangkwon/Desktop/FaceParty/react-native-opencv-tutorial/node_modules/react-native/third-party/double-conversion-1.1.5/src/strtod.cc normal x86_64 c++ com.apple.compilers.llvm.clang.1_0.compiler
        CompileC /Users/kangsangkwon/Desktop/FaceParty/react-native-opencv-tutorial/ios/build/Build/Intermediates.noindex/React.build/Debug-iphonesimulator/double-conversion.build/Objects-normal/x86_64/fast-dtoa.o /Users/kangsangkwon/Desktop/FaceParty/react-native-opencv-tutorial/node_modules/react-native/third-party/double-conversion-1.1.5/src/fast-dtoa.cc normal x86_64 c++ com.apple.compilers.llvm.clang.1_0.compiler
(3 failures)

Installing build/Build/Products/Debug-iphonesimulator/reactNativeOpencvTutorial.app
An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=22):
Failed to install the requested application
The bundle identifier of the application could not be determined.
Ensure that the application's Info.plist contains a value for CFBundleIdentifier.
Print: Entry, ":CFBundleIdentifier", Does Not Exist

Command failed: /usr/libexec/PlistBuddy -c Print:CFBundleIdentifier build/Build/Products/Debug-iphonesimulator/reactNativeOpencvTutorial.app/Info.plist
Print: Entry, ":CFBundleIdentifier", Does Not Exist

Hi, it works for me: https://github.com/facebook/react-native/issues/14382#issuecomment-420190830

@bushev how is java code interface with react native?

It's just linking problem. I fixed it with manual linking.


Update hero.rs: fix the derive line

Using AsChangeset two times in the derive resulted (at least for me) in a linting/running error, but this does not fix all the issues.

Great catch @zargold !! Thanks for the contribution!!


Replace all instances of dapp to app

#186  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @imranjami and the rest of your teammates on  Graphite


Should QNNs be able to handle lists as y values for fitting?

I guess this is something we want?
Unexpected exception formatting exception. Falling back to standard exception
Traceback (most recent call last):
  File "c:\Users\mcr\Anaconda3\lib\site-packages\IPython\core\interactiveshell.py", line 3369, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "C:\Users\mcr\AppData\Local\Temp\ipykernel_19772\793130201.py", line 16, in <cell line: 16>
    reg.fit(X_train, Y_train)
  File "C:\Users\mcr\PycharmProjects\squlearn\src\squlearn\qnn\base_qnn.py", line 114, in fit
    self._fit(X, y, weights)
  File "C:\Users\mcr\PycharmProjects\squlearn\src\squlearn\qnn\qnnr.py", line 212, in _fit
  File "C:\Users\mcr\PycharmProjects\squlearn\src\squlearn\qnn\qnnr.py", line 185, in partial_fit
    self._qnn,
  File "C:\Users\mcr\PycharmProjects\squlearn\src\squlearn\qnn\training.py", line 521, in regression
    elif weights is None:
AttributeError: 'list' object has no attribute 'shape'

Can sklearn hlms do this? If yes, yes.


hook up the BusyIndicator to the mapDrawing status

Description

For the DisplayDrawingStatus sample, the BusyIndicator no longer appears on Windows. The fix is to hook up BusyIndicator's "running" state to the "mapDrawing" boolean. This gets it going when necessary.
I'm not sure when this regressed; my guess is it's due to the change from Qt 5 to Qt 6.
Type of change

[x] Bug fix
[ ] New sample implementation
[ ] Sample viewer enhancement
[ ] Other enhancement

Platforms tested on:

[x] Windows
[ ] Android
[ ] Linux
[x] macOS
[ ] iOS

Checklist

[ ] Runs and compiles on all active platforms as a standalone sample
[ ] Runs and compiles in the sample viewer(s)
[ ] Branch is up to date with the latest main/v.next
[ ] All merge conflicts have been resolved
[ ] Self-review of changes
[ ] There are no warnings related to changes
[ ] No unrelated changes have been made to any other code or project files
[ ] Code is commented with correct formatting (CTRL+i)
[ ] All variable and method names are camel case
[ ] There is no leftover commented code
[ ] Screenshots are correct size and display in description tab (500px by 500px, platform agnostic)
[ ] If adding a new sample, it is added to the sample viewer
[ ] Cherry-picked to Main branch (if applicable)

This is weird and I'd like help understanding. I'm not sure why the BusyIndicator's visibility needs to be changed. Its visibility should be contingent on the visibility of the mapDrawingWindow which is controlled by the same mapDrawing boolean on line 43. If the BusyIndicator is just running constantly, but is hidden/visible because of the parent rectangle, I don't understand why this works.
Furthermore, the QML sample relies on the same logic. Is QML affected on Windows as well?
Lastly, I verified that the changes work on Android, but then again the originals worked on Android as well.

This change doesn't impact the BusyIndicator's visibility setting at all -- that continues to operate as it always has, which is that it's totally dependent on the parent's visibility. This change  just impacts the running setting. Of course, a BusyIndicator that isn't running doesn't draw anything, so in that sense it looks the same as if visibility is false. The difference now is that when mapDrawing changes, the BusyIndicator both becomes visible (because of its parent), as well as it starts running. So apparently that's enough to get it going. Whereas before, the BusyIndicator was running non-stop, and simply changed visibility back and forth.
I can't say for sure, but there's probably some efficiency code in BusyIndicator to keep it from churning CPU cycles when it's running-but-invisible, and that code is what we were getting tripped up on. Now that it toggles running as well as going visible, everybody's happy.

Cool, thanks for the explanation. Could you make this same change to the QML sample? Then I'll approve it, but I think we should wait for Gela's verification on Windows before merging.

The QML version is implemented differently and doesn't have the same problem. It has an onDrawStatusChanged that checks drawStatus === Enums.DrawStatusInProgress. I guess the difference is because we're able to make a mapDrawing property in the C++ code that we otherwise can't do in pure QML.

Hm, odd that the QML version works despite running: true which caused it to fail in C++. Oh well, so long as it works.


s/it state is a primitive/if state is a primitive/

Coverage remained the same at 96.111% when pulling 37fda6ca425f8f6b5a71653b24609542a1edee2e on christiangenco:patch-1 into 2677cc3e4e12291dac6666ace95372c91dfa2069 on mweststrate:master.

Thanks!


Link to KB invalid

In a 0.20 report a link to
https://daisy.github.io/a11y-kb/docs/scripting/aria.html
is invalid.
Looks like it has been renamed to
https://daisy.github.io/a11y-kb/docs/script/aria.html

The code has already been updated to reflect the new URL so the next release will reflect it:
https://github.com/daisy/ace-core/blob/master/src/report/axe2ace.js

Thanks


Make tabs tabbable again! (skip setting tabindex to -1)

Currently, vaadin-tabs aren't tabbable. Sounds funny, yeah :)
We've implemented the unique behavior in keyboard a11y, even our own demos are inconsistent, tabbing works nicely in  but not in vaadin-tabs examples:

Expected: ability to focus "Tab two" with Tab
Actual: inability to focus "Tab two" with Tab
Our users are asking about workaround and how to set tabindex of vaadin-tabs to 0.
I think we should provide the similar experience as most of tabs plugins provide, namely make tabs tabbable, namely by pressing Tab users will be able to go through all tabs.
Proposal
The current behavior could be optional, users will be able to turn it on/off by setting some property (name to be decided)

According to WAI-ARIA recommendations:
For the tab list:

Tab: When focus moves into the tab list, places focus on the active tab element . When the tab list contains the focus, moves focus to the next element in the page tab sequence outside the tablist, which is typically either the first focusable element inside the tab panel or the tab panel itself.
When focus is on a tab element in a horizontal tab list:

Left Arrow: moves focus to the previous tab. If focus is on the first tab, moves focus to the last tab. Optionally, activates the newly focused tab (See note below).
Right Arrow: Moves focus to the next tab. If focus is on the last tab element, moves focus to the first tab. Optionally, activates the newly focused tab (See note below)

This is specifally so that users can quickly skip over a long tab list to get to the tab content.

We've implemented the unique behavior in keyboard a11y

Here's another example with the same behavior: https://www.webcomponents.org/element/PolymerElements/paper-tabs/demo/demo/index.html

An issue with tabbing through individual tabs is that once there are many tabs, it might get really frustrating to tab through them all to navigate to the next component.
So agree with @jouni, this is a nice feature of <vaadin-tabs> that allows users to skip over the entire tabs widget, no matter how many tabs are there.

This is specifically so that users can quickly skip over a long tab list to get to the tab content.

So agree with @jouni, this is a nice feature of  that allows users to skip over the entire tabs widget, no matter how many tabs are there.

One of the essential skills of good UX designers is to know existed solutions. Please take a look at GitHub UI, there are tabbable tabs and "Skip to content" button.
Once again, I'm proposing adding a new optional feature, which could be added to the next minor version if we're not making the new behavior as a default.

Github tabs are essentially links, they cause a full page reload most of the time. And yes, traditionally web page tabs are implemented with links and hence are in the tab order.
The use case for vaadin-tabs is more on the full-blown application side, where my experience is that tabs are navigated with arrow keys and not the tab key.
Options are fine, and it’s the maintenance team’s call if they want to take on the added cost of maintaining more options. Personally, I’m not seeing enough need for such an option at the moment. If we would decide to add an option, I would keep the current behavior the default one, so we don’t have to make a new major version (might make sense to remove the “requires new major” label as we have an option to not require it).
My suggestion, for now, would be to implement such tabs as custom using  elements, if you need “tabbable tabs”.

my experience is that tabs are navigated with arrow keys and not the tab key.
I don’t have access to any these kinds of apps that I referred

You have an experience with apps you don't have access to. Sounds contradictory :upside_down_face:

I can’t speak for screen reader users (and I don’t have much experience using apps with keyboard-only), but I think my preference would be one tab-stop per tab bar if there are more than a handful of tabs.

The easiest way to check out what is the best keyboard a11y, it's quite logical to test one of a11y-related projects, e.g.

https://www.usability.gov/what-and-why/accessibility.html
https://a11yproject.com/

vaadin-tabs is designed to support the latter case (a large number of tabs).

Isn't a large number of tabs the poor UX by itself? In my opinion, it is. The reason is simple. As a user I either can't see all tabs:

or see an extremely wide app:

But please do not make a desicion based on 1 person opinion and explore related questions/answers https://ux.stackexchange.com/search?q=many+tabs to see different opinions.

Then there is this https://developer.mozilla.org/en-US/docs/Web/Accessibility/Keyboard-navigable_JavaScript_widgets#Grouping_controls

For the guidelines they promote, at least a11yproject has a “Simple Tabs” example that opts for arrow navigation: https://codepen.io/svinkle/pen/edmDF

Please pay attention to this span from their codepen:
Use left and right arrows to navigate between tabs.

which will be read by screen readers. We don't have anything like that.

This is specifically so that users can quickly skip over a long tab list to get to the tab content.

An issue with tabbing through individual tabs is that once there are many tabs, it might get really frustrating to tab through them all to navigate to the next component.

This should be solved in a completely different way, which is the WCAG 2.0 (A) MUST: https://dequeuniversity.com/rules/axe/2.2/bypass?application=lighthouse


Adds ability for soiling.Hsu to accept arbitrary time intervals (other than 1 hour)

[x] Closes #970
[x] I am familiar with the contributing guidelines
[no changes needed] Tests added
[N/A] Updates entries to docs/sphinx/source/api.rst for API changes.
[X] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[X] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[X] Pull request is nearly complete and ready for detailed review.
[x] Maintainer: Appropriate GitHub Labels and Milestone are assigned to the Pull Request and linked Issue.

Other than the small whatsnew changes, the only thing I see is that the test has @needs_pandas_0_22, which is no longer necessary as of #1003

FYI test failure is the shapely segfault

Thanks @jsstein and @kanderso-nrel!


Rich/Verbose console should group output when not attached to a console

Right now, the plain console groups output when attached to a console, but also groups the output if a stream is not sent to a console (i.e. when redirected to a file handle).  However, the rich/verbose consoles simply pump the output directly to the streams with no grouping.  We should do something like the following:

If a stream is attached to a console, use the rich console chain.
If both stderr and stdout are attached to a console, redirect stderr to stdout and group everything together.
If a stream is not attached to a console, use the plain console chain.

PR: https://github.com/gradle/gradle/pull/5443

Merged.


Add Azure DevOps logo

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @aalmada!

:book:

:art: Thanks for submitting a logo. Please ensure your contribution follows our guidance for logo submissions.

  Generated by :no_entry_sign: dangerJS

I think it now follows the guidance for logo submissions. Azure DevOps is one of the supported CI platforms and the logo was minimised using svgomg.

Nice one. Thanks for minifying. Do you think it would be better to make the default logo white so it pops a bit more on the default background:

White seems to be an approved colour for the logo: https://twitter.com/azuredevops?lang=en

I can change it to white but please note that most logos in that folder are in their original colors. The color can be changed using the logoColor parameter.
For example, the GitHub logo is black at https://github.com/badges/shields/blob/master/logo/github.svg but I'm using it in light gray at https://github.com/NetFabric/NetFabric.DoubleLinkedList/blob/master/README.md

@RedSparr0w - do you have an opinion on this one?

It's hard to imagine anyone wanting blue on gray; it doesn't seem like the best default. For what it's worth, the GitHub logo is dark because it was originally designed for the static badge, which has a light background. If white is an approved color, my vote is for that, at least until we add something like #2431.

FWIW - the default logo color is white for the Azure Pipelines badge (although it is a different logo)
https://github.com/Microsoft/vscode#visual-studio-code---open-source

I think we should try to defer all icons to simple-icons.
And if they do not accept it or it doesn't fit their spec for whatever reason then open one here.
What are your thoughts? @paulmelnikow @chris48s
Otherwise if we are going to be merging this PR, i agree the white works better.

I think we should try to defer all icons to simple-icons.

I agree, I don't see a real advantage to keeping our own collection of icons.
So that we don't string along @aalmada, I might suggest:

Merge this with white.
Modify our logo guidelines to say we are no longer accepting new logos
Match up our existing logo collection with simple-icons, and get our existing logos matched in there. Maybe in the process, one of us could ask to join as a co-maintainer.
Remove our existing logos, leaving aliases in place for if needed for backward compatibility.

White it is then... :)
Commits are all squashed to a single one.


check for architecture, download appropriate node version

Fixes issue with Node not working on 32-bit operating systems. See #194.
List of changes:

In VE.sh check whether the machine is 32 or 64 bit and download/install the appropriate version of Node.js

How to test:
Perform normal MediaWiki testing on 32-bit and 64-bit CentOS, with particular attention to making sure VisualEditor works.
Issues:

Closes #194

Tested on CentOS 6 x86. Tests passed. Next will test on 64-bit to make sure 64-bit didn't get broken. Also important to verify that x64 version is actually downloaded, since the x86 version SHOULD work on x64 but we don't want to use it.

Test passed for x64 version, confirmed x64 version of Node installed. Ready for merge.

Thanks for catching my miss. I forgot about the two architecture options. Looks simple enough to not perform duplicate tests. Merging.


Golang support

Overview
Adds golang to PackageType
User Story
https://github.com/phylum-dev/roadmap/issues/102

I think it probably makes sense to merge this and move on for now, however, I created an issue here because it feels like this type could probably be refactored into common: https://github.com/phylum-dev/phylum-types/issues/56

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


Replace Mixins by an HOC

This is a WIP PR for replacing the mixins by an higher order component.

[x] remove mixins
[x] implement HOC withFauxDOM()
[x] update examples
[ ] update tests
[ ] update documentation

Cc @Olical
Cc @djeeg from #90

@djeeg, some comments about the difference with your approach:

I obviously did not go with Typescript.
I didn't merge the connectFauxDOM() call into withFauxDOM(). This is to keep the same API users are already familiar with, and it also makes it possible to hold several faux nodes in a single component.
I used an inheritance inversion so that we can directly manipulate the wrapped component's state, lifecycle methods, and class methods. This is probably the point to get animation sorted in your implementation.

Let me know what you think about these things :).

An additional note: the use-case here is actually very suitable for inheriting a component similarly to PureComponent but that would require users to know about the implementation so they don't override the lifecycle methods we use in their component without using super. That's the advantage of using an HOC with inverted inheritance, we can implement the careful inheritance and the user doesn't have to care.

hey @tibotiber, I have done some testing of your inversion HOC withFauxDOM, lgtm.

Whats your suggestion for rendering multiple instances of the same component?
This is what i have currently.
class NameOfReport extends React.Component<IProps, any> {
    constructor (props) {
        super(props)
        var elementid = "chart_NameOfReport_" + shortid();
        this.state = { elementid : elementid, [elementid]: "loading..." }
    }
    componentDidMount() {
        var faux = (this as any).connectFauxDOM('div', this.state.elementid);
		//...
    }
    render() {
        var elementid = this.state.elementid;
        var result = (

                    {this.state[elementid]}

        );
        return result;
    }
}
export default withFauxDom(NameOfReport);

@djeeg I'm guessing you meant to ask how to render multiple charts in a single component, did you? If that is the case you would call connectFauxDOM() twice in componentDidMount() and render each node separately in render().
// ...
componentDidMount() {
  // ...
  var fauxNode1 = connectFauxDOM('div', 'chart1');
  var fauxNode2 = connectFauxDOM('div', 'chart2');
}

render() {
  // ...
  {this.state.chart1}
  // ...
  {this.state.chart2}
}

Please note that I have never used this behaviour myself and am not advocating it. In many cases, I think rendering different components may be a better idea. But since it was a possibility of the API, I'd rather not remove it. I also think it is easier to reason about the HOC when you don't pass usage arguments to it, it basically injects class methods that you can use but does not originally change the state for you which keeps it pure.

@Olical any idea on how to handle testing for this?

Hm, I'll have a look. I've never been too good at testing more complex UI things, anything more than pure views tend to perplex me :smile:

I rebased and updated to match the API of #89. Will look into HOC testing as well when I have some time, haven't done that before.

@Olical I fixed the tests for the HOC. There is a failing test that has nothing to do with this PR though, I tried it on master and it is failing there as well.

I updated the docs and other links, the PR is ready :)

Added some links to the README, see #92 (did it here to avoid conflicts due to README updates in this PR)

(I haven't forgotten and will sort out the old mergeroo soon, I've just had a couple of very busy works + studying a new coursera course in my free time :smile:)

Thanks for the heads up Oliver. Any ETA on the merge?
Also any idea on the failing test? It doesn't seem to come from these changes, were you aware of it previously?

If I had to guess, it's some new property D3 tries to use, it may just need to be implemented. I'll take a look this week :smiley:

If you would like to merge master in the tests will start to pass again. Also, I was dumb and added a link to your article in a commit, which now conflicts with yours. But I prefer your separate articles section, so feel free to take that during the merge conflict resolution! It should be really easy. Once you're up to date with master and you've replaced my addition of your post link, I'll be happy to merge.
For some reason D3 started using a new method to access some style properties, I wasn't sure on the difference, so I just pointed this new method at the existing style property getter.

I rebased and fixed the conflicts. There is a failing test but apparently unrelated to the HOC. Looking into it. Any idea @Olical?

Oh, that's very odd. I'll take a look.

:tada:

Thanks for the work, you rock :D
On 28 May 2017 16:20, "Thibaut Tiberghien"<EMAIL_ADDRESS>wrote:

🎉
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Olical/react-faux-dom/pull/91#issuecomment-304521469,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AATPXSumSeT2NVsAsOML9HEh2bl3cFrsks5r-ZCvgaJpZM4NVAVG
.


Using parser with envelopes.ApiGatewayEnvelope

Using parser() to parse Event for API Gateway Lambda.
APIGatewayProxyEventModel has version as mandatory attribute
APIGatewayEventRequestContext has authorizer as mandatory attribute
These are not present in ALL Lambda events from API Gateway via Lambda Proxy integration.
What were you trying to accomplish?
Trying to parse payload as a Model according to documentation on parser
Expected Behavior
To correctly parse the API Gateway Event passed to Lambda without throwing an exception
Current Behavior
Validation exception is thrown for missing fields.
Possible Solution
class APIGatewayEventRequestContext(BaseModel):
    ...
    authorizer: Optional[APIGatewayEventAuthorizer]
    ...

class APIGatewayProxyEventModel(BaseModel):
    version: Optional[str]
    ...

Steps to Reproduce (for bugs)
import json
from aws_lambda_powertools.utilities.parser import BaseModel, envelopes
from aws_lambda_powertools.logging import Logger
from aws_lambda_powertools.utilities.parser.parser import parse

# Specify the API payload Model expected
class UserModel(BaseModel):
    username: str
    password1: str
    password2: str

logger = Logger()

def test(event, context):
    parsed_event = parse(event=event, model=UserModel, envelope=envelopes.ApiGatewayEnvelope)
    response = {
        "statusCode": 200,
        "body": json.dumps(event.dict())
    }

    return response

Environment

Powertools version used: 1.17.1
Packaging format (Layers, PyPi): PyPi
AWS Lambda function runtime: 3.8
Debugging logs

[ERROR] ValidationError: 2 validation errors for APIGatewayProxyEventModel
version
  field required (type=value_error.missing)
requestContext -> authorizer
  field required (type=value_error.missing)
Traceback (most recent call last):
  File "/var/task/handler.py", line 34, in test
    parsed_event = parse(event=event, model=UserModel, envelope=envelopes.ApiGatewayEnvelope)
  File "/var/task/aws_lambda_powertools/utilities/parser/parser.py", line 148, in parse
    return envelope().parse(data=event, model=model)
  File "/var/task/aws_lambda_powertools/utilities/parser/envelopes/apigw.py", line 30, in parse
    parsed_envelope = APIGatewayProxyEventModel.parse_obj(data)
  File "/var/task/pydantic/main.py", line 578, in parse_obj
    return cls(**obj)
  File "/var/task/pydantic/main.py", line 406, in __init__
    raise validation_error

Hey @walmsles, thanks for flagging it!
Could you give an example of an event that this isn't true so we can flag the official aws docs too?
Happy to accept a PR to make this optional, or work on this right after a Mypy fix I'm working

Hey have submitted a PR for this (just now :-)).
My PR is consistent with sample events here
The event samples (above and below) are also consistent with output from: sam local generate-event apigateway aws-proxy
Event from API GW:
{
        "resource": "/test",
        "path": "/test",
        "httpMethod": "POST",
        "headers": {
            "Accept": "*/*",
            "Accept-Encoding": "gzip, deflate, br",
            "CloudFront-Forwarded-Proto": "https",
            "CloudFront-Is-Desktop-Viewer": "true",
            "CloudFront-Is-Mobile-Viewer": "false",
            "CloudFront-Is-SmartTV-Viewer": "false",
            "CloudFront-Is-Tablet-Viewer": "false",
            "CloudFront-Viewer-Country": "AU",
            "Content-Type": "application/json",
            "Host": "nlioqcygg3.execute-api.ap-southeast-2.amazonaws.com",
            "Postman-Token": "0d2f11a9-ba6d-49bc-9c1b-795b435592cc",
            "User-Agent": "PostmanRuntime/7.28.0",
            "Via": "1.1 c0abed86b33ff7ae86d6e529296ac06e.cloudfront.net (CloudFront)",
            "X-Amz-Cf-Id": "r3RE7FPFa1B0oq9LlMkCOMKbPUwU5fdD9Zvx6l1GFGExhpgKS8FCXQ==",
            "X-Amzn-Trace-Id": "Root=1-60f286cd-1f3632430b6bbd507a246206",
            "X-Forwarded-For": "<IP_ADDRESS>, <IP_ADDRESS>",
            "X-Forwarded-Port": "443",
            "X-Forwarded-Proto": "https"
        },
        "multiValueHeaders": {
            "Accept": [
                "*/*"
            ],
            "Accept-Encoding": [
                "gzip, deflate, br"
            ],
            "CloudFront-Forwarded-Proto": [
                "https"
            ],
            "CloudFront-Is-Desktop-Viewer": [
                "true"
            ],
            "CloudFront-Is-Mobile-Viewer": [
                "false"
            ],
            "CloudFront-Is-SmartTV-Viewer": [
                "false"
            ],
            "CloudFront-Is-Tablet-Viewer": [
                "false"
            ],
            "CloudFront-Viewer-Country": [
                "AU"
            ],
            "Content-Type": [
                "application/json"
            ],
            "Host": [
                "nlioqcygg3.execute-api.ap-southeast-2.amazonaws.com"
            ],
            "Postman-Token": [
                "0d2f11a9-ba6d-49bc-9c1b-795b435592cc"
            ],
            "User-Agent": [
                "PostmanRuntime/7.28.0"
            ],
            "Via": [
                "1.1 c0abed86b33ff7ae86d6e529296ac06e.cloudfront.net (CloudFront)"
            ],
            "X-Amz-Cf-Id": [
                "r3RE7FPFa1B0oq9LlMkCOMKbPUwU5fdD9Zvx6l1GFGExhpgKS8FCXQ=="
            ],
            "X-Amzn-Trace-Id": [
                "Root=1-60f286cd-1f3632430b6bbd507a246206"
            ],
            "X-Forwarded-For": [
                "<IP_ADDRESS>, <IP_ADDRESS>"
            ],
            "X-Forwarded-Port": [
                "443"
            ],
            "X-Forwarded-Proto": [
                "https"
            ]
        },
        "queryStringParameters": null,
        "multiValueQueryStringParameters": null,
        "pathParameters": null,
        "stageVariables": null,
        "requestContext": {
            "resourceId": "w845zr",
            "resourcePath": "/test",
            "httpMethod": "POST",
            "extendedRequestId": "CmoAGFq7SwMFQ4A=",
            "requestTime": "17/Jul/2021:07:29:17 +0000",
            "path": "/dev/test",
            "accountId": "674048636348",
            "protocol": "HTTP/1.1",
            "stage": "dev",
            "domainPrefix": "nlioqcygg3",
            "requestTimeEpoch":<PHONE_NUMBER>232,
            "requestId": "e227c43b-0957-4354-9f78-b0874f771630",
            "identity": {
                "cognitoIdentityPoolId": null,
                "accountId": null,
                "cognitoIdentityId": null,
                "caller": null,
                "sourceIp": "<IP_ADDRESS>",
                "principalOrgId": null,
                "accessKey": null,
                "cognitoAuthenticationType": null,
                "cognitoAuthenticationProvider": null,
                "userArn": null,
                "userAgent": "PostmanRuntime/7.28.0",
                "user": null
            },
            "domainName": "nlioqcygg3.execute-api.ap-southeast-2.amazonaws.com",
            "apiId": "nlioqcygg3"
        },
        "body": "{\n    \"data\": \"test message\"\n}",
        "isBase64Encoded": false
    },
    "timestamp": "2021-07-17 07:29:17,259+0000",
    "service": "service_undefined",
    "xray_trace_id": "1-60f286cd-1f3632430b6bbd507a246206"
}

Staged for next week's release - Friday the latest. Let me know if this is impacting production and I can bring it forward

All good, not urgent at all.   Thanks 👍

hey @walmsles - This is now fixed in 1.18.0
More details in the release notes: https://github.com/awslabs/aws-lambda-powertools-python/releases/tag/v1.18.0


Align Python build with C# builds

Change target platform for Linux and OSX to match GUI
Fix filename in python to be like App.cs
Align the Python build directory (mostly) with the GUI paths
Change the applicationIdentifier for Android to com.icosa.openbrush (this one was being overwritten correctly at build time anyway, but it'd be better to have the proper value always present)

Thanks!


[dev] Intellisense in project.json is broken / missing

Scenario

Open asp.net project
Open project.json
Type half package name

Expectation
Intellisense prompt potential package candidates.

Fixed in 7c55ca1

We're seeing this today (4/5) discussing OmniSharp (v1.0.993) in VS Code: Some look-ups in project.json are good, but others (Mvc here for example) are not shown ...

Our situation today is covered by this known issue, correct? ... so we just need to give it some time?

For me it only gives one suggestion:

Basically all RC2 stuff "Microsoft.AspNetCore." is not autocompleting at all, but stuff like "Microsoft.AspNet." works.

the default provider (as far as I'm aware) for vscode intellisense only supports the main nuget feed, and one of the ci feeds.

idk if @Ciantic has the same ones I do, but that might at least explain the difference between us on what is getting looked-up. Here's the set I'm using, which came from a very old David F. example app ...
<?xml version="1.0" encoding="utf-8"?>
<configuration>
  <packageSources>
    <clear />
    <add key="AspNetCI" value="https://www.myget.org/F/aspnetcidev/api/v3/index.json" />
    <!-- <add key="AspNetVolatile" value="https://www.myget.org/F/aspnetvolatiledev/api/v3/index.json" /> -->
    <add key="NuGet.org" value="https://api.nuget.org/v3/index.json" />
    <add key="dotnet-core" value="https://www.myget.org/F/dotnet-core/api/v3/index.json" />
    <add key="dotnet-cli" value="https://www.myget.org/F/dotnet-cli/api/v3/index.json" />
    <add key="AspNetvNext" value="https://www.myget.org/F/aspnetvnext/api/v3/index.json" />
  </packageSources>
</configuration>

Side Note: I only go volatile if things get really nasty and only as a temp workaround; I've only un-commented that line twice ever and only used it for a few days each time. It's just there to keep the link handy.

who provides these reference? VS code itself or omnisharp?

@troydai looks like this has nothing to do with OmniSharp, if I start the vscode without omnisharp (checked from process explorer) it still autocompletes same entries.
Maybe this is some VS Code feature, and instead it should be configured to use OmniSharp in projects.json.

Close this here, since it is tracked in VS Code.


change one Publication 231004_0005

👷 Deploy Preview for yzm0518 processing.

Name
Link

🔨 Latest commit
563346760301330583e7adf657884111dd93b53b

🔍 Latest deploy log
https://app.netlify.com/sites/yzm0518/deploys/651c3c29aa3f5e0008b14f56


Swap € and $ in German Layout

There is a function like that for French Layout. Is it also possible to make this work for German Layout. Using Neo 2 German Layout.

Please use new setting by this private.xml:
https://github.com/tekezo/Files/blob/master/Karabiner/private.german_swap_euro_and_dollar/private.xml
How to use private.xml:
https://pqrs.org/osx/karabiner/document.html#privatexml

Wow, thank you for the fast support! I added the Code into private.xml and activated the Key Change by the Checkbox, but Keys didn't change. Maybe i did something wrong.

That setting exchanges shift+4 and alt+e.
I assume these key combination input $ and € in your German Layout.
Is it right?

shift+4 == $
alt+e == €

I am using German Neo Layout2 :-)
The Key  combination is:
Modifier 3 + ö = $
OR Shift + 6 = $
(but want to use Modifier 3 + ö for €)
Shift + 7 = €
Thank you for the Support.

OK.
I updated private.xml. Please replace your private.xml with it.
https://github.com/tekezo/Files/blob/master/Karabiner/private.german_swap_euro_and_dollar/private.xml

I updated private.xml again.

Works perfect. thank you so much.


Fix proxy arg passthrough

proxy argument was broken in recent versions, this should fix it.
Fixes #998 #1282 #1386, and maybe #1387 (were they using proxy?)
Status
Ticker.isin() doesn't work with proxy, can someone with a proxy help investigate? #1399

@ricardoprins Start your proxy testing here.

This fixes #1399. Going to check the other open issues.
def test_ticker_with_proxy(self):
        proxy = 'http://xxx.xxx.xxx.xxx:xxxxx' (address omitted on purpose here)
        ticker = yf.Ticker('AAPL')
        isin = ticker.get_isin(proxy)
        assert isin is not None

test results:
Testing started at 2:56 PM ...
Launching unittests with arguments python -m unittest test_yfinance.TestTicker.test_ticker_with_proxy in xxxxx

DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): query2.finance.yahoo.com:443
DEBUG:urllib3.connectionpool:https://query2.finance.yahoo.com:443 "GET /v6/finance/quoteSummary/AAPL?modules=financialData&modules=quoteType&modules=defaultKeyStatistics&modules=assetProfile&modules=summaryDetail&ssl=true HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): query1.finance.yahoo.com:443
DEBUG:urllib3.connectionpool:https://query1.finance.yahoo.com:443 "GET /ws/fundamentals-timeseries/v1/finance/timeseries/AAPL?symbol=AAPL&type=trailingPegRatio&period1=1673913600&period2=1689724800 HTTP/1.1" 200 247
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): markets.businessinsider.com:443
DEBUG:urllib3.connectionpool:https://markets.businessinsider.com:443 "GET /ajax/SearchController_Suggest?max_results=25&query=Apple%20Inc. HTTP/1.1" 200 3817

Ran 1 test in 1.733s

OK

This fixes #1386:
    def test_download_with_proxy(self):
        proxy = 'http://xxx.xxx.xxx.xxx:xxxxx' # (address omitted on purpose here)
        download = yf.download('AAPL', proxy=proxy)
        assert download is not None

test results:
Testing started at 2:57 PM ...
Launching unittests with arguments python -m unittest test_yfinance.TestTicker.test_download_with_proxy in xxxxx

DEBUG:yfinance:Entering download()
DEBUG:yfinance:Disabling multithreading because DEBUG logging enabled
DEBUG:yfinance: Entering history()
DEBUG:yfinance:AAPL: Yahoo GET parameters: {'period1': '1899-12-31 17:24:11-04:56', 'period2': '2023-07-18 16:57:46-04:00', 'interval': '1d', 'includePrePost': False, 'events': 'div,splits,capitalGains'}
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): query2.finance.yahoo.com:443
DEBUG:urllib3.connectionpool:https://query2.finance.yahoo.com:443 "GET /v8/finance/chart/AAPL?period1=-2208994789&period2=1689713866&interval=1d&includePrePost=False&events=div%2Csplits%2CcapitalGains HTTP/1.1" 200 None
DEBUG:yfinance:AAPL: yfinance received OHLC data: 1980-12-12 14:30:00 -> 2023-07-18 20:00:04
DEBUG:yfinance:AAPL: OHLC after cleaning: 1980-12-12 09:30:00-05:00 -> 2023-07-18 16:00:04-04:00
DEBUG:yfinance:AAPL: OHLC after combining events: 1980-12-12 00:00:00-05:00 -> 2023-07-18 00:00:00-04:00
DEBUG:yfinance:AAPL: yfinance returning OHLC: 1980-12-12 00:00:00-05:00 -> 2023-07-18 00:00:00-04:00
DEBUG:yfinance: Exiting history()
DEBUG:yfinance:Exiting download()

Ran 1 test in 0.662s

OK


Is osiris detected?

I got vacced 2 times
I am using extreme injector v3 with manual map to inject osiris into csgo with all the scramble features enabled.
Is it my injector thats detected or is it osiris?

extreme injector 💀
Injector is hella detected. idk how osiris is doing

It's not VAC detected but I always get a game ban in exactly 40-45 days after the first injection on an account.

There was an unnoticed change to the UserCmd which was likely the cause of trust factor issues. It was fixed last week.

There was an unnoticed change to the UserCmd which was likely the cause of trust factor issues. It was fixed last week.

what injector do you recommend?


Add a utility to update AUTHORS.md

Our authors files are out-of-date in almost all repositories. Due to exactly these reasons, I've never been a fan of such files, users can just use git to request that information.
If we prefer these files, we should have a utility program to extract the list from git and update our files respecting the existing format.
@fabxc @juliusv @brian-brazil @sdurrheimer

Issue with automating is duplication of certain authors (sometimes 5+ times) with different email addresses.

Example script used by docker : https://github.com/docker/docker/blob/master/hack/generate-authors.sh

@fabxc sure, but doing that by hand each time is annoying, and I guess we can put the algorithm we're following into code (like taking the first email, or listing all emails comma separated, and/or maintaining a mapping, or ???).

@sdurrheimer It would need to not touch an existing maintainers section at the top.

We decided to not keep hard-coded authors list in files and maintain a MAINTAINERS.md file instead.


Pelican configuration file not shows

There’s no file name pelican.yaml found in  /etc/pelican. I’ve download all three visions.

This feels to me like a combination of:

Updating documentation to describe how and where Pelican looks for configuration files.
Ensuring that published containers(?) come with reasonable default configuration files in the documented locations. (For example, as I recall, some projects, like Apache, ship configuration files in the default locations that contain nothing but comments that document the default settings.)


BMI Calculator using React

I want to add a BMI Calculator project which is created using react.js. It help an user to calculates his/her BMI.
Tech stack used:

HTML
CSS
React.js
Node.js

By entering an user's height and weight they can see their BMI.

Can you please assign me this issue.

Hey sumitkr2000, can you assign me this task so that I can work on it?

Hey sumitkr2000, can you assign me this task so that I can work on it?

Hey @pranjay-poddar I have completed this issue can u please my merge request and add the "hacktoberfest-accepted" tag to it. Thank you!


Re-enable ssh add-on's network ssh access.

PR #1117 disabled SSH network access for the SSH server add-on.
Users who previous had the "SSH Server" official add-on installed, will be surprised when they update to version 8.4 that ssh access from the network suddenly stops working.
PR #1117 has effectively changed this add-on from something that enabled ssh access over the network into something that only provides access through a web GUI terminal.
This PR simply makes ssh enabled be default restoring the previous behavior to avoid frustrating existing users.
If the intent is to have SSH network access disabled by default. it should be better communicated as a breaking change in a future PR.  There are a number of ways this could be done. At a minimum documentation updates and possibly advertising it as a breaking change in a future version.

Hi @rct,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

We should document this behavior, not re-instate it.

So the solution is to edit the add-on config of "Terminal & SSH"

Add port number "host" in Container | Host | Description
save
The restart the ssh add-on.

I will update the supervisor to copy the install port setting over, so that changes like this did not end up in a breaking change

@pvizeli - That seems like a good idea, anyone who has installed the ssh add-on but hasn’t updated to 8.4 yet will retain ssh network access.

This needs to be relooked at.  There needs to be a way of accessing the system when the addon has failed or the supervisor has stopped loading.

This needs to be relooked at. There needs to be a way of accessing the system when the addon has failed or the supervisor has stopped loading.

You can install the add-on, enable SSH and it will stay enabled. Not sure why you think your use case is related to this (very old) PR?
If you are a developer, and have to deal with brokenn Add-ons and/or Supervisor, SSH access on the host system might be the better option for your case (https://developers.home-assistant.io/docs/operating-system/debugging#ssh-access-to-the-host).


add AudioContextOptions to AudioContext

The user can now specify:

the number of output channels
the sample rate
the output latency
when building its AudioContext.

If the intented options are not met by the audio hardware capabilities, the AudioContext is built with default_config options, and a warning is printed to the user.
If the user options are missing, the AudioContext is built with primary_config options. If the primary_config options are not met by the audio harware capabilities, the AudioContext is built with default_config options, and a warning is printed to the user.
This PR modify DestinationNode:

Adds max_channels_count method
Assert ChannelCountConstraints on DestinationNode

This PR adds merger.rs example to demonstrate new capabilities.
Clippy passes on context.rs and io.rs
related to #35
related to #28

I didn't port the refactoring to the input stream. i leave it as "good first issue" or for another PR

is it Ok with you, if i open a good first issue about this topic?

Go ahead!


Device Update for IoT Hub missing the free tier

In the prerequisites the requirements are specified as follows.

An IoT hub. It's required that you use an S1 (Standard) tier or above.

However if you look at the prerequisites for the ESP32 tutorial the requirements are as follows.

To use Azure Device Update you must have an Azure IoT Hub with scale unit S1, S2, S3 or a free/standard hub.

My issue is that the free tier is missing from the original documentation.

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

ID: 741ab579-291f-5f35-2f4e-199929b227e3
Version Independent ID: 60ff5e0e-4311-914d-ad7d-b0856bc07e0f
Content: Create an account for Device Update for Azure IoT Hub
Content Source: articles/iot-hub-device-update/create-device-update-account.md
Service: iot-hub-device-update
GitHub Login: @eshashah-msft
Microsoft Alias: eshashah

@the-programmer
Thanks for the feedback! We are currently investigating and will update you shortly.

This should be corrected in ESP32 tutorial as free IoT hub tier is not supported for DU. Submitting a PR for this.

Thank you @eshashah-msft for the quick resolution!
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.
Please also do visit Microsoft Q&A the home for technical questions and answers at Microsoft.


Can not excute Set-UnifiedGroup with  App-only authentication

PS C:\windows\system32> Connect-ExchangeOnline -CertificateThumbPrint "" -AppID "" -Organization ""
PS C:\windows\system32> Set-UnifiedGroup -Identity<EMAIL_ADDRESS>-AutoSubscribeNewMembers:$false
Active Directory operation failed on DM6PR01A05DC005.NAMPR01A005.prod.outlook.com. This error is not retriable.
Additional information: Insufficient access rights to perform the operation.
Active directory response: 00002098: SecErr: DSID-03150F94, problem 4003 (INSUFF_ACCESS_RIGHTS), data 0
+ CategoryInfo          : NotSpecified: (:) [Set-UnifiedGroup], ADOperationException
+ FullyQualifiedErrorId : [Server=SL2PR03MB4539,RequestId=2dc2ddf5-5c7e-4a49-8993-a89a06f47264,TimeStamp=8/8/2020
6:46:23 AM] [FailureCategory=Cmdlet-ADOperationException] 276E34A1,Microsoft.Exchange.Management.RecipientTasks.Se
tUnifiedGroup
+ PSComputerName        : outlook.office365.com
PS C:\windows\system32> Set-UnifiedGroup -Identity<EMAIL_ADDRESS>-DisplayName "aaaaa"
Failed to retrieve executing user. Please try again later.
+ CategoryInfo          : NotSpecified: (:) [Set-UnifiedGroup], RecipientTaskException
+ FullyQualifiedErrorId : [Server=SL2PR03MB4539,RequestId=ab94f925-66c2-40bc-b089-c66baeccba7c,TimeStamp=8/8/2020
6:47:12 AM] [FailureCategory=Cmdlet-RecipientTaskException] CAA7D44A,Microsoft.Exchange.Management.RecipientTasks.
SetUnifiedGroup
+ PSComputerName        : outlook.office365.com

Hello @cuicheng11165, thank you for your feedback.
For your inquiry I need to kindly suggest opening a support ticket directly from your tenant's administration, they will be able to help you as here we are limited to documentation issues and improvements.
Support URL: https://docs.microsoft.com/microsoft-365/admin/contact-support-for-business-products.
Thank you!

@dariomws  Thank you very much for the contribution and sharing this explanation. @cuicheng11165 Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. We proceed here to close it.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Ambiguous Conversion Of  'godot::Variant' To 'godot::String' In Godot.hpp

Problem:  Was following every step of the README. However, when running
clang -fPIC -o src/init.os -c src/init.cpp -g -O3 -std=c++14 -Icpp_bindings/include -Igodot_headers -Icpp_bindings/include/ a few errors occur.
Most are easy to resolve (unable to find included files due to incorrect path). But then there is following
In file included from src/init.cpp:1: cpp_bindings/include/core/Godot.hpp:64:42: error: ambiguous conversion for static_cast       from 'godot::Variant' to 'godot::String'   static T _arg_cast(Variant a) { return static_cast<T>(a); }
Same error occurs when using GCC.
Info:

godot3.0 25f742c (from 15 Sept 2017)
godot-cpp e32d7bb (15 Sept 2017)
clang 4.0.1-5 / gcc 7.2.0-2

Ah yes, I should get around to fix that.
The problem is that some compilers like to see a.operator T(), other's like to see static_cast<T>(a), I should figure out which compiler wants what or if there's a proper way to deal with this in C++.
So for now if you just want to get it working then try a.opeator T() or maybe (T) a too.

If some C++ guru knows how to best fix that problem - feel free to tell me or make a PR 😂

#39 should've fixed this


Brave wallet update

Address comment
Update example to include BraveWalletAdapter

Wallet Adapter was intended to be neutral and not provide recommendations of any wallet, or even implicit guarantees that they work properly. It hasn't always done that.
https://github.com/solana-labs/wallet-adapter/pull/498 has removed all adapters from the examples and starter projects, and we will solve this problem with better docs and support rather than appearing to promote or prioritize any wallet.


Replace broken git clone command

Output of previous command:
$ git clone git@github.com:makersacademy/TodoApi.git
Cloning into 'TodoApi'...
The authenticity of host 'github.com (<IP_ADDRESS>)' can't be established.
RSA key fingerprint is

[redacted].
Are you sure you want to continue connecting (yes/no)? yes
Warning: Permanently added 'github.com,<IP_ADDRESS>' (RSA) to the list of known hosts.
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.

Why do we need to change this @david-mears ?

All I can tell you is it fixed the error detailed in the PR comment. @EdwardAndress


chore(deps): loosen up renovate, allowing it to automerge minor changes as well

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat(files): paste files from tobe_ui

[x] I reviewed my own code, you can now review the PR 😘

[x] paste the files from ui project
[x] set up the environments

Moving forwards, we need to remove the concept of "Domain" in the front-end and rename it as "Content"


Label each boundary interface separately

The suggestions of labeling the multiple boundaries came from the following problem:

[x] I want to compute the energy flow through the front and back side of the crystal separately (as those correspond to reflection and transmission). Currently this corresponds to the boundary at x=0 and x=L. -> #191
[x] Additionally, not energy should leak to the sides (y=0, y=y1), I would like to verify this energy flow.
[ ] I'm adding absorbing boundary regions, so that my domain will look like a 5 layer cake: absorber, air, crystal, silicon, absorber instead of the current 3 layer one (air, crystal, silicon). So then I would need to compute the energy flow not at the air boundary, but on the interface between air and the absorber. -> #203

For the internal boundaries I could do most of the work by looking at the names of the adjacent zones. But for the external boundaries I don't have any way of properly distinguishing the several parts (left, right, top, bottom).
Originally posted by @Laar in https://github.com/hpgem/nanomesh/issues/136#issuecomment-985541774

See #203


fix(resolve): Dont show locking workspace members

What does this PR try to resolve?
This is for cargo generate-lockfile and when syncing the lockfile with
the manifest.
We still show it for cargo update because of cargo update --workspace.
We hacked around this previously by filtering out the num_pkgs==1 case
for single packages but this didn't help with workspaces.
How should we test and review this PR?
Additional information
This builds on #14440

This PR had a refactor on it, merging the cargo generate-lockfile and the lockfile sync (e.g. cargo check) reporting but I realized there was a difference that we might want to preserve: cargo generate-lockfile only shows Adds with relevant information to avoid overwhelming the user.

@bors r+
Thanks!

:pushpin: Commit d2ec764995885a74f607669c74cbda60787a924e has been approved by weihanglo
It is now in the queue for this repository.

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing ef854d2f66df7bfcd803a6ca1cfa3b619603142c to master...


error with torchaudio.pyd

I have everything installed without errors still i cant run it. This is the error i get when i run app.py:
E:\audiocraft>python app.py
Traceback (most recent call last):
File "E:\audiocraft\app.py", line 11, in
from audiocraft.models import MusicGen
File "E:\audiocraft\audiocraft_init_.py", line 8, in
from . import data, modules, models
File "E:\audiocraft\audiocraft\data_init_.py", line 8, in
from . import audio, audio_dataset
File "E:\audiocraft\audiocraft\data\audio.py", line 21, in
import torchaudio as ta
File "E:\Python\Python310\lib\site-packages\torchaudio_init_.py", line 1, in
from torchaudio import (  # noqa: F401
File "E:\Python\Python310\lib\site-packages\torchaudio_extension_init_.py", line 43, in
_load_lib("libtorchaudio")
File "E:\Python\Python310\lib\site-packages\torchaudio_extension\utils.py", line 61, in load_lib
torch.ops.load_library(path)
File "E:\Python\Python310\lib\site-packages\torch_ops.py", line 643, in load_library
ctypes.CDLL(path)
File "E:\Python\Python310\lib\ctypes_init.py", line 374, in init
self._handle = _dlopen(self._name, mode)
FileNotFoundError: Could not find module 'E:\Python\Python310\Lib\site-packages\torchaudio\lib\libtorchaudio.pyd' (or one of its dependencies). Try using the full path with constructor syntax.

had the same libtorchaudio.pyd error message when cuda was not installed on torch and torchaudio. after deinstalling torch and reinstalling i got it working with:
Windows 10 - CUDA 117
torch 2.01+cu117
torchaudio 2.02+cu117
torchvision 0.15.2+cu117

had the same libtorchaudio.pyd error message when cuda was not installed on torch and torchaudio. after deinstalling torch and reinstalling i got it working with:
Windows 10 - CUDA 117 torch 2.01+cu117 torchaudio 2.02+cu117 torchvision 0.15.2+cu117

Thanks already reinstalling it, funny thing is it writes as installed with cuda and still makes it cpu version.


No optional locks

Releated #175

https://git-scm.com/docs/git

--no-optional-locks
Do not perform optional operations that require locks. This is equivalent to setting the GIT_OPTIONAL_LOCKS to 0.


Errors without BED file

Is there a way to perform realignment without the use of a BED file, for example, in WGS cases?
Thank you.

Currently, a bed file is always required.  A workaround is to simply specify all chromosomes with a range of 1 to chromosome length in the bed file.  For example, for HPV we use:
NC_001357.1    1       7857
NC_001526.2    1       7905
ABRA does not currently scale well to human WGS.  For our very limited WGS testing we split up BAM files by chromosome and ran ABRA on each independently.  Unfortunately, even that does not scale well - WGS has simply not been a common use case for us to date.


Deep Link URL opening iTunes Store

Hi!
Some users are starting to notice that when they tapped to pay with ATH Móvil iOS tries to open iTunes Store "ATH Móvil" app page. We noticed this first happened when users changed their iPhones. But now it's starting to happen to random people.
I don't know if the universal link https://athm-ulink-prod-static-website.s3.amazonaws.com/e-commerce/mobile it's working correctly.
I will update here if I found a pattern to reproduce it.
I'm using NativeScript to develop my apps.

Hi Pedro
Thanks for comment. You are right the issue is the URL of Universal link https://athm-ulink-prod-static-website.s3.amazonaws.com/e-commerce/mobile.  The problem is not in the SKD, the problem is in the target application (ATH Movil).
ATH Móvil can be opened by Universal Link or URL Scheme. As a work around, you can toggle between URL Scheme and Universal Link. In order to use URL Scheme please change as follows:

Open the class ATHMPaymentRequest
Look for the method named  sendPayment<Handler, Opener>(_ handler: Handler, urlopener: Opener)
Inside the method change TargetUniversalLinks for TargetURLScheme, there is two of them
After that you can compile and test your app again.

Find attached an image of how change would like.
Have a good one

Hi!
Thanks for the quick reply. I just confirmed with a TestFlight user that this workaround works. Thanks a lot.
Should I do a PR to change this or the issue with the UniversalLink will be reported and fixed?

Hi!
Based on this solution I created a PR for Xamarin Forms. https://github.com/evertec/athmovil-xamarin-sdk/pull/6


discourage instanceof

Different versions of vinyl should be able to co-exist, since plugins will now be relying on it directly. The README should discourage usage of instanceof
(related: #103)

I'm partial to Vinyl.isVinyl to match Buffer.isBuffer and Array.isArray.

:+1: to @terinjokes' idea

done (0.5.3)


WIP global: integration of invenio-files-rest

Refactorices the code in order to make it work with
invenio-files-rest and invenio-records.

Adds a macro to generate the iframe which shows the content of the
file.

Signed-off-by: Javier Delgado<EMAIL_ADDRESS>

Addressed in inveniosoftware/invenio-files-rest#8.

Working on:

[ ] Test coverage issue (Test still running over more than 90% of the code but the detector is not working properly).
[ ] Example app.

I have remove one test module (test_invenio_previewer_ext) since it was checking the same behaviour of test_invenio_previwer_views and due to the integration of invenio_files_rest, the test cases were becoming exactly the same.

What is the status of this PR.

@lnielsen The issue with the code detection stills there so the code can be reviewed but the tests are going to be changed (Not the tests per se but the way to run the server and check the response)

@lnielsen The test are finally passing and the module is ready from my side. Could you take a look, please?

@lnielsen Updated.
WRT zenodo/zenodo#465, now it is showing an alert:

Great :-)


[FW]: Add support to missing maps

This will take some time :)

[ ] Blackwater canal
[ ] Drydock
[ ] Angel city
[ ] Colony
[ ] Glitch
[ ] Relic
[ ] Wargames
[ ] Rise

Closing as I dont have the time or motivation for this.
Some notes:

Team territories are defined using trigger entities, I dont know how to set their team
Turrets need an id ( 0 - 8 ) to properly work with the ui

Preferably this would be done using an extra .ent file as modifying the existing ones would just bloat the repository.


How to measuring the checkout process, with ecommerce plugin ?

Hello !
How to load the Ecommerce Plugin ?
To use like this:
ga('require', 'ec');

ga('ec:addProduct', {
  'id': 'P12345',
  'name': 'Android Warhol T-Shirt',
  'category': 'Apparel',
  'brand': 'Google',
  'variant': 'Black',
  'position': 1,
  'dimension1': 'Member'
});

ga('ec:setAction', 'click', {
  'list': 'Search Results'
});

In the issue: https://github.com/MatteoGabriele/vue-gtag/issues/217 you talk about purchase function but purchase does not replace ec:addProduct or ec:addImpression or ec:setAction
thanks you 😄

I'm sorry, I don't really write the API: it's just like this.
analytics and gtag are not the same things. I suggest you check the differences between these 2 libraries. The plugin basically exposes the most common gtag API methods and, if not, you can always use the query method to access directly gtag API


Support claims with verifiable resources

From the README:

CoAtNet is considered the State-of-the-Art model for Image Classification.

There are better models than CoAtNet. It's better to avoid claims like that and include more specific statements like below:

CoAtNet achieves xx.xx% top-1 classification accuracy on the ImageNet-1k dataset (validation split). To know more,  please refer to the original paper [link].

Sure!

From the README:

CoAtNet is considered the State-of-the-Art model for Image Classification.

There are better models than CoAtNet. It's better to avoid claims like that and include more specific statements like below:

CoAtNet achieves xx.xx% top-1 classification accuracy on the ImageNet-1k dataset (validation split). To know more,  please refer to the original paper [link].

Updated
CoAtNet achieves 86.0% ImageNet top-1 accuracy; When pre-trained with 13M images from ImageNet-21K, our CoAtNet achieves 88.56% top-1 accuracy, matching ViT-huge pre-trained with 300M images from JFT-300M while using 23x less data; Notably, when we further scale up CoAtNet with JFT-3B, it achieves 90.88% top-1 accuracy on ImageNet


add parsing for AT TIME ZONE

This commit adds the custom Calcite extension for parsing AT TIME ZONE expressions, but does not yet support executing these queries.
I implemented the @ScalarFunction locally to make sure that it can work when fully wired in but I wanted to commit that separately because we should take care to make sure that we do it in a backwards compatible way. The complications here are that AT TIME ZONE behaves different based on whether the input type already has a time zone associated with it or not. Additionally, AT TIME ZONE returns a timestamp with time zone for non-zoned timestamps, which means we need to support both types before getting the "SQL Standard" behavior here.
Also See: postgres specficiation

Codecov Report

Merging #9477 (370ae53) into master (3818397) will decrease coverage by 8.70%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #9477      +/-   ##
============================================
- Coverage     34.73%   26.03%   -8.71%
+ Complexity      194       44     -150
============================================
  Files          1910     1904       -6
  Lines        101850   101654     -196
  Branches      15452    15430      -22
============================================
- Hits          35379    26462    -8917
- Misses        63483    72541    +9058
+ Partials       2988     2651     -337

Flag
Coverage Δ

integration1
26.03% <0.00%> (?)

integration2
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...apache/pinot/sql/parsers/parser/SqlAtTimeZone.java
0.00% <0.00%> (ø)

...n/java/org/apache/pinot/query/type/TypeSystem.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/server/access/RequesterIdentity.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/query/planner/QueryPlan.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/query/validate/Validator.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/pinot/query/planner/PlannerUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../org/apache/pinot/query/planner/StageMetadata.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...org/apache/pinot/query/planner/stage/JoinNode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...org/apache/pinot/query/planner/stage/SortNode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...g/apache/pinot/query/planner/stage/FilterNode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 610 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Type instabilities?

See this Discourse thread: https://discourse.julialang.org/t/custom-rule-for-differentiating-through-newton-solver-using-forwarddiff-works-for-gradient-fails-for-hessian/93002/20

I think I encountered something related. the ForwardDiff extension flattens arrays to a vector on output (line 42 in ImplicitDifferentiationForwardDiffExt.jl) having the dimensions preserved should fix it in my case at least

Unfortunately this seems required for the linear solvers to work: Krylov.jl doesn't work with arbitrary arrays. I tried to use KrylovKit.jl instead a while back and it ended up being type-unstable for different reasons, but I'm willing to try again

Note that the type stability tests which used to be performed by ChainRules were removed temporarily because I bumped into https://github.com/JuliaDiff/ChainRulesTestUtils.jl/issues/232

I see, but can't you just reshape after the solve?

I see, but can't you just reshape after the solve?

Not sure what you mean, I think this line does the flattening
https://github.com/gdalle/ImplicitDifferentiation.jl/blob/c6c8fc251b2b35d61497b35fb8f24e0b49d0ba0c/ext/ImplicitDifferentiationForwardDiffExt.jl#L35
and this line does the reshaping
https://github.com/gdalle/ImplicitDifferentiation.jl/blob/c6c8fc251b2b35d61497b35fb8f24e0b49d0ba0c/ext/ImplicitDifferentiationForwardDiffExt.jl#L38

@thorek1 I think your issue might have been solved by @YingboMa in #42

And the issue from the Discourse thread will be taken care of by #47

Thanks, I will have a look and let you know.

it didn't resolve my issue. the MWE is a bit involved (the error occurs at line 94). maybe a high level hint: Zygote works, ForwardDiff doesn't. ForwardDiff breaks when I try to use the result returned from implicit diff in computations because the dimension info is lost and he returns a vector instead of a matrix. the function I try to diff has 1 arg and 2 kwargs and returns a matrix and a bool. the conditions accepts three args, the same kwargs and returns a matrix of the same dimension as the first output form the forward function. in principle it should work and it does for Zygote

That fixed it. Thanks!


Don't Merge! PR real time test for target branch full build

Docs Build status updates of commit 765d285:
:clock10: Full build: the average full build time is 6 min(s) 1 sec(s), this is based on the last 30 build(s) for this repository.


<EMAIL_ADDRESS>build error in webpack@4, <EMAIL_ADDRESS>works fine

Reproduction

<EMAIL_ADDRESS>import { URL } from 'whatwg-url';  （es module）
<EMAIL_ADDRESS>(commonjs, no default export)
ModuleDependencyError: Can't import the named export 'URL' from non EcmaScript module (only default export is available)

Expected behavior

Screenshots

Your Environment

software
version

node-fetch

node

npm

Operating System

Additional context

released 2.6.5


Bad Config

hey Badbones69 i know i messed up somewhere in my enchantments config and I was wondering if you could have a look at it. One minute everything was working and I tried to separate some of the Enchants and the next minute my console was being spammed with "unable to pass any event" to CE I have tried to fix this on my own but all the yaml parsers say its valid so i'm at a loss.
Error:
https://pastebin.com/ir3s0Mqk
Enchants.yml:
https://pastebin.com/vHeqD9Bt

You had a yaml format issue. I fixed them and now it should work fine.
https://pastebin.com/raw/nqJbiWiT

omg ty :D


rename precision from Laplace distribution to scale

Closes #293 by using scale, instead of prec to initialize Laplace cuqi.distribution.Laplace

In the end, we moved from prec to scale in iniatilizing cuqi.distribution.Laplace to keep consistent with

CUQIpy itself (cuqi.distribution.LMRF)
Numpy's Laplace
CUQI-PyTorch's Laplace

Thank you @amal-ghamdi for pointing out demos/try12. I just did the update it as you advised and left that line commented out.


KAFKA-15887: Ensure FindCoordinatorRequest is sent before closing

A few bugs was created from the previous issues. These are:

During testing or some edge cases, the coordinator request manager might hold on to an inflight request forever. Therefore, when invoking coordinatorRequestManager.poll(), nothing would return.  Here we explicitly create a FindCoordinatorRequest regardless of the current request state because we want to actively search for a coordinator
ensureCoordinatorReady() might be stuck in an infinite loop forever if the client fail to do so.  Even the consumer would be able to shutdown eventually, this is undesirable.
The current asyncConsumerTest mixes background/network thread shutdown with the consumer shutdown.  As the goal of the module is unit testing, we should try to test the shutdown procedure separately.  Therefore, this PR adds a Mockito.doAnswer call to the applicationEventHandler.close().  Tests that are testing shutdown are calling shutdown() explicitly.

Hi @lucasbru - This PR should have addressed the issue in KAFKA-15887.  I also fixed a couple of issues while reviewing the code.

@philipnee Sounds good. Let me know when it's ready for review.

Hi @lucasbru - I just opened the PR for you to review. I'm not 100% happy with the way tests are setup therefore I made some changes around optionally disabling autocommit in the network thread. Also, I feel the tests here kind of become some sort of integration testing. I thought it kind of against the unit test philosophy.

Yeah. I think the tests may be using spy too much, and should use mock instead. It's definitely running too much code, which makes it more like an integration test, but still introspect a lot, which make them more like a unit test. But it's not something that we are going to fix in this PR. I wonder if we can maybe create a ticket to migrate the AsyncKafkaConsumerTest to a more mocking based approach, is that something that would be feasible in a reasonable amount of time? cc @cadonna who always has strong opinions on unit testing.

We will only auto commit if the configuration is enabled (by default) or if we've got anything to commit at all

Makes sense

We need to enforce finding a coordinator and send an autocommit regardless of the previous commit state because we need to make sure to record the progress before closing

Makes sense

Quite a bit of changes to the testing, because autocommit depends on the current progress, so I need to "seek" for some cases to ensure the test sends an autocommit

Makes sense

Here are the failures:
Build / JDK 21 and Scala 2.13 / testOffsetTranslationBehindReplicationFlow() – org.apache.kafka.connect.mirror.integration.MirrorConnectorsIntegrationExactlyOnceTest
42s
Build / JDK 21 and Scala 2.13 / testResetSinkConnectorOffsetsZombieSinkTasks – org.apache.kafka.connect.integration.OffsetsApiIntegrationTest
58s
Build / JDK 21 and Scala 2.13 / testRackAwareRangeAssignor(String).quorum=zk – integration.kafka.server.FetchFromFollowerIntegrationTest
6s
Build / JDK 21 and Scala 2.13 / testRackAwareRangeAssignor(String).quorum=kraft – integration.kafka.server.FetchFromFollowerIntegrationTest
6s
Build / JDK 21 and Scala 2.13 / testWithGroupMetadata() – kafka.api.TransactionsBounceTest
13s
Build / JDK 21 and Scala 2.13 / testWithGroupMetadata() – kafka.api.TransactionsBounceTest
14s
Build / JDK 21 and Scala 2.13 / testFailureToFenceEpoch(String).quorum=kraft – kafka.api.TransactionsTest
42s
Build / JDK 17 and Scala 2.13 / testWithGroupId() – kafka.api.TransactionsBounceTest
12s
Build / JDK 17 and Scala 2.13 / testWithGroupId() – kafka.api.TransactionsBounceTest
9s
Build / JDK 17 and Scala 2.13 / testWrongIncarnationId() – kafka.server.ControllerRegistrationManagerTest
30s
Build / JDK 17 and Scala 2.13 / testDescribeClusterRequestIncludingClusterAuthorizedOperations(String).quorum=kraft – kafka.server.DescribeClusterRequestTest
<1s
Build / JDK 17 and Scala 2.13 / testDescribeUnderReplicatedPartitionsWhenReassignmentIsInProgress(String).quorum=zk – org.apache.kafka.tools.TopicCommandIntegrationTest
3s
Build / JDK 8 and Scala 2.12 / testOffsetTranslationBehindReplicationFlow() – org.apache.kafka.connect.mirror.integration.MirrorConnectorsIntegrationExactlyOnceTest
42s
Build / JDK 8 and Scala 2.12 / testAlterSinkConnectorOffsetsZombieSinkTasks – org.apache.kafka.connect.integration.OffsetsApiIntegrationTest
24s
Build / JDK 8 and Scala 2.12 / testWithGroupMetadata() – kafka.api.TransactionsBounceTest
12s
Build / JDK 8 and Scala 2.12 / testBalancePartitionLeaders() – org.apache.kafka.controller.QuorumControllerTest
15s
Build / JDK 11 and Scala 2.13 / testReplicateSourceDefault() – org.apache.kafka.connect.mirror.integration.MirrorConnectorsIntegrationExactlyOnceTest
2m 51s
Build / JDK 11 and Scala 2.13 / testReplicateSourceDefault() – org.apache.kafka.connect.mirror.integration.MirrorConnectorsIntegrationSSLTest
2m 28s
Build / JDK 11 and Scala 2.13 / testReplicateSourceDefault() – org.apache.kafka.connect.mirror.integration.MirrorConnectorsIntegrationTransactionsTest
4m 26s
Build / JDK 11 and Scala 2.13 / shouldHaveSamePositionBoundActiveAndStandBy – org.apache.kafka.streams.integration.ConsistencyVectorIntegrationTest
16s

They are unrelated but seems like there's a movement of not merging failing builds.

I totally agree on not using spies. That is my opinion and also the Mockito documentation says to be really careful with spies and to just use them if absolutely needed. Spies are mostly used to test legacy code and they should not be used for new code. We should definitely not wrap the code to test into spies. In my current PR, a unit test was not working because of this wrapping.
I started to improve the test code in a separate PR, but the PRs for the deadline have higher priority.

Test failures unrelated and mapped to existing/new flaky test issues.