feat: better error page

Codecov Report
Attention: Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 60.06%. Comparing base (34790f0) to head (3cb1654).

Files with missing lines
Patch %
Lines

server/errorHandler.ts
75.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #175      +/-   ##
==========================================
+ Coverage   60.04%   60.06%   +0.02%
==========================================
  Files         193      193
  Lines        2648     2652       +4
  Branches      435      438       +3
==========================================
+ Hits         1590     1593       +3
- Misses       1010     1011       +1
  Partials       48       48

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


mahesh.raut/RAVCOM-508 Upgrade Calcite Core Module Sub Packages

in this PR,  we incorporated the latest changes from Apache Calcite into specific classes within the following packages:
runtime, sql2Rel, and util

Ok with changes.


force cash chain

see commit comments for more info

I've pushed this without the network.py changes, which I hope are not necessary.  I also added something to remove the recent servers.  The network dialog should now be clean of forks on restart.


Integration tests

I think we need some integration tests for changes that we make.
Right now I just make sure to run a few of the previous examples to check nothing is broken. But running a single script will be easier when adding changes. My proposal is to run a test, and check the resulting time-avaraged NEES value to make sure it's "reasonable" even if its out of bounds.

One integration test that we could run is a purely noiseless filter run, and ensure that we have machine-precision zero errors throughout

Addressed through PR #17


fix: try using it in a piped list

:tada: This PR is included in version 2.0.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


chore: remove unnecessary windows cmd entry point

🎉 Thanks for submitting a pull request! 🎉
Summary
remove the unnecessary file for the windows command as node generates on windows the binaries on their own. no need to ship anything

For us to review and ship your PR efficiently, please perform the following steps:

[x] Open a bug/issue before writing your code 🧑💻. This ensures we can discuss the changes and get feedback from everyone that should be involved.
[x] Read the contribution guidelines 📖. This ensures your code follows our style guide and
passes our tests.
[x] Update or add tests (if any source code was changed or added) 🧪
[x] Update or add documentation (if features were changed or added) 📝
[x] Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

I will do an rc version and test it out on my vm - have a parallels license now ;)

works like a charm on windows:


Add a DRA layer to demo app layers

Add a BC Digital Road Atlas layer to demo app layers. It should be sourced from GeoServer and appear immediately after Parcels in the layer list.

Please provide the service endpoint URL and layer name.

http://openmaps.gov.bc.ca/geo/ows?service=wms&version=1.3.0&request=GetCapabilities
layer name is pub:WHSE_BASEMAPPING.DRA_DGTL_ROAD_ATLAS_MPAR_SP

Verified in fork

Verified


rank_plot changes

@djk2120 the modifications to rank_plot and top_n in analysis.py, specifically the indexing by minmax, appear to not work with the OAAT data format being read in by get_exp. I think this has something to do with how the minmax coordinate is set up. Do you have another processed data location for OAAT output that works with the new rank_plot function?

I found a few more issues with the current analysis.py (e.g., reading h1 tape output with get_exp) doesn't appear to work anymore.
The rank_plot issue is resolved by switching to the version you have in /glade/u/home/djk2120/ppe_tools - perhaps we could merge those changes into the version in this repo? Reading h1 tape doesn't appear to be fixed by this version though.


MM-27346 Fix for excessive analytic events on scroll of LHS

Summary
We have a scroll listener https://github.com/mattermost/mattermost-webapp/blob/f5d93686ed6798813a6d1eb20182a1b6532a46ee/components/sidebar/sidebar_menu/sidebar_menu.tsx#L63 which is triggering handleMenuToggle call
https://github.com/mattermost/mattermost-webapp/blob/f5d93686ed6798813a6d1eb20182a1b6532a46ee/components/sidebar/sidebar_menu/sidebar_menu.tsx#L74-L79
Preventing the call if state hasn't changed to fix this
Ticket Link
https://mattermost.atlassian.net/browse/MM-27346

@sudheerDev
Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin

+++ Returning you to the master branch and cleaning up.

/cherry-pick release-5.26

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin

+++ Returning you to the master branch and cleaning up.


Cannot build plugin on Ubuntu 18.04.3

Build Log:
mkdir -p public && \
cd dist && \
curl -L -o ""mp3enc-wasm-"0.1.6"".tar.gz" -C - "https://github.com/streamer45/mp3rec-wasm/releases/download/v"0.1.6"/"mp3enc-wasm-"0.1.6"".tar.gz" 2> /dev/null && \
cd ../public && \
tar xf ../dist/"mp3enc-wasm-"0.1.6"".tar.gz 2> /dev/null && \
mv "mp3enc-wasm-"0.1.6""/* . && \
mv recorder.js ../webapp/src/client/ && \
rm -rf "mp3enc-wasm-"0.1.6""
Makefile:105: recipe for target 'mp3-wasm' failed
make: *** [mp3-wasm] Error 33

Versions:
Ubuntu 18.04.3
Node: 8.10.0
Go: 1.10.4
GNU Make: 4.1

Looking into this, couldn't reproduce on Archlinux.

I could reproduce the issue on Ubuntu 18.04.2
It appears to be a problem with older versions of curl (version tested 7.58.0) failing if the downloaded file already exists.

Nice

Closing as wontfix for now.


Kineticist Taking Gate's Threshold (Expand the Portal) not displaying options

Go from Level 4 to 5
Select Expand the Portal
Choose Water
Choose Skill
Get a display that I have "No Valid options for Gate's Threshold"

I have

Lv1 Earth gate with Armor in Earth
Lv1 Water Gate with Deflecting Wave
Lv1 Weapon Infusion
Lv2 Winter's Clutch
Lv4 Safe Elements

I wanted or expected to see Winter's Clutch as an option to select on gaining level 5
Actor.json

Hello!
Quick question- did you import your character using Pathbuilder?

No I did not

Something in the actor isn't set right. Go to level 0 then back up and it should fix it.

@TikaelSol thanks, that did fix it.


issue with using cy.api after cy.origin

I have an issue with this code. After replacing cy.request with cy.api, tests are failing.
describe('test', () => {
  it('test', () => {
    cy.origin('https://www.novinky.cz', () => {
      cy.visit("https://www.novinky.cz/p/tiraz")
      cy.contains("Tiráž")
    })

    cy.request({
      method: 'GET',
      url: 'http://httpstat.us/200'
    })
  })
})

the reason for this particular test failing is that the origin actually points to https://cmp.seznam.cz instead of novinky.cz. the error that causes this test to fail is from wrong use of cy.origin rather than cy.api. closing this for now, but if you are experiencing similar issue, feel free to reopen and provide more info


Fix MT4 and MT5 installation

The goal is to check and fix the installation of MT4 platform via Travis CI.
Branch: mt5
Script: scripts/install_mt5.sh (which uses install_mt5.verb).

Could you also test: install_mt4.sh and install_mt4.verb? Maybe branch off to mt5 and change -M5 to -M4.

I think they should fairly work, maybe they need some tweaks. Only installer part needs to be fixed.

You can cherry pick them from master or just copy

Check also https://github.com/FX31337/FX-BT-Scripts/issues/86, another task, let me know if you’d like to check it as well, and if you’ve any questions. You can use VM or Docker to test it out.

I've created branch mt4 out of master and added installation via Docker, but build 384211485 failed due to missing display, so I won't be able to prepare the branch for you this weekend as I'm on holidays. If it's confusing, you can pass that one, unless you know how you can test these scripts using Travis CI.

Here is the example using Docker (you can run on local if you've got it):

docker run -i --entrypoint /bin/bash ea31337/fx-mt-vm <<<'eval.sh install_mt 4'
docker run -i --entrypoint /bin/bash ea31337/fx-mt-vm <<<'eval.sh install_mt 5'

Otherwise, I've created CI tests for it for Travis CI in mt4 branch. See: build 385467827.
Problem:

MT4 installation using AHK hangs
MT5 got 404, but probably is a glitch, so there could be another error.

More recent build: 385469579 where I've sent blind fix for XM's app title.

The installation looks good.


Freedom Fighters low and not stable FPS

The game produces low FPS in places, especially in those places where the effect of explosions and dust occurs.
I am using the game together with dgvoodoo2 + dxvk (anti-aliasing, anisotropic filtering)
System information

GPU: Nvidia GTX 1070
Driver: 452.06
Wine version: i use Windows 10 (2004)
DXVK version: 1.7.1

Apitrace file(s)
https://mega.nz/file/ERYiTCbb#xQQdYBS2vd0UxaYRH_D_gaXCRwOU-d1Ds3KR7eGPGbc
Log files
Freedom.Exe_d3d11.log
Freedom.Exe_dxgi.log

Why are you using DgVoodoo with DXVK on Windows? DXVK is almost certainly gonna be slower than the Nvidia d3d11 driver, especially with dgvoodoo involved.

Почему вы используете DgVoodoo с DXVK в Windows? DXVK почти наверняка будет медленнее, чем драйвер Nvidia d3d11, особенно с dgvoodoo.

To be able to use the Nvidia driver settings, for example, it is convenient to use sharpness when anti-aliasing.

I am using the game together with dgvoodoo2

Not really our problem then. dgvoodoo2 is known not to perform very well.


What is the current state of this project?

I see that it has many issues that remains open. The last commit was last year. @trustmaster do you have any specific plan for this project?
Thank you

This project has been suspended while I've been busy in https://github.com/noflo/noflo lands.
I want to continue work on it but there's no specific time span. Here is my current roadmap for this project:

Drop reactive components support and stick with active components only. In reality active components proved to be way more practical and it would also simplify the runtime.
Complete NoFlo-UI/Flowhub integration.
Use it in real applications and implement whatever comes reasonable while doing so.

Any updates here? Seems like the websocket over at golang.org/x/net/websocket isn't available anymore.

@Chillance I was able to go get golang.org/x/net/websocket today without a problem. Could it be a network problem?
However, package tests are not passing with recent Go versions. I'll have a look at it.

Possibly a network problem, although it seemed to me that it wasn't there anymore. Especially since they also say this:
"This package currently lacks some features found in an alternative and more actively maintained WebSocket package:
https://godoc.org/github.com/gorilla/websocket"
So, could you possibly change to use gorilla instead? I will try again and see if it works in the meantime.

Ok, so a go get of this worked now it seems. Could still be an idea to change to gorillas websocket.

See #35

Cool! Thanks!


Unity 2017.3.1f1 project does not compile anymore: StandaloneFileBrowser.dll not found on OSX (Universal)

In Unity 2017.3.1f1 the .dll can not be found. Unity has removed the support for OSX 32 and OSX 64 and we ran problems on other locations as well where we needed to flag stuff for OSX Universal therefore projects using StandaloneFileBrowser do not compile anymore.

Oh god... its a long day. Tried to press play while OSX was selected on windows....


sqlfluff.executablePath should not sync

I run VSCode and the sqlfluff plugin on several different computers. I use VSCode setting s sync. Some of these computers are Windows and others are Mac. Additionally, some are corporate computers where my access and ability to install and change settings are limited.
Thus, the install location of sqlfluff is not uniform and cannot always be part of the system path. I can get around this by manually specifying the location of sqlfluff in sqlfluff.executablePath. However, this setting currently syncs accross so every time I go to a different computer with a different location for sqlfluff, I need to change it.
Can you please update the sqlfluff plugin to not sync sqlfluff.executablePath

@Snuffy2 You should be able to use the "settingsSync.ignoredSettings": ["sqlfluff.executablePath"], in your settings.json to disable syncing that setting, or any settings you don't want to sync.

Thanks, I'll give that a try. I was not aware that this option existed.
I'd still vote that sqlfluff.executablePath be set to not sync but I can follow your suggestion as well.

I think for those that do not want it to sync then this setting is a good alternative. The other option is to set sqlfluff.executablePath only in workspace settings. I do not want to change the sync behavior for only this setting as that could confuse and would be unexpected.


Added rke.cattle.io client for v2prov validation tests

Issue: https://github.com/rancher/rancher/issues/39260

Validation tests for v2prov certificate rotation
Backport of https://github.com/rancher/rancher/pull/39089

@annablender This is a backport PR, I think refactoring it to that degree is out of the scope of this issue.

@igomez06 @annablender Had to fix merge conflict. I'll make a follow up PR to address some comments after this gets merged.


polymorphism: test mocking

reorganize test code
add mocking demo class
add usage of the mocking class in an additional test

Summary by CodeRabbit

New Features

Introduced a new library polymorphism_lib to enhance project functionality.
Added a new executable test_consume for testing purposes.
Implemented a Mock struct for simulating behavior in tests, including methods to track calls and manage arguments.

Bug Fixes

Enhanced existing tests with additional assertions to ensure proper functionality and side effects.

Tests

Expanded testing framework with new test cases for both modern and mock implementations, improving coverage and validation.

@coderabbitai please propose a code coverage measurement configuration.


Make the button color for '?' and '×' brighter in dark mode

Thanks for @hashomhoot's suggestion feedback via email

 Make the button color for '?' and '×' brighter in dark mode


Bump Apache Support from 3.2.2 to 3.3.0

Expected Behavior
I'd be able to upgrade the Apache dependency within our cookbook from 3.2.2 to 3.3.0.

Current Behavior
Because sous-chef's jumped from 3.2.2 to 3.3.0, our cookbook fails to converge.  The icinga2 cookbook is pessimisticly locking us to Apache's 3.2.X series of cookbooks which are ageing: https://github.com/sous-chefs/apache2/releases
Context
I'm upgrading to Apache 3.3.0 to resolve https://github.com/sous-chefs/apache2/pull/422

Will fix shortly (should be using >=). Thank you for reporting!

Bumping apache2 and other cookbooks to latest version.

Updated apache2 cookbook to v5.0.1.
Closing this issue.
Icingaweb2 is now maintained by another repository https://github.com/Icinga/chef-icingaweb2


Ways without end date not rendering

Bug description
Ways without end dates are not rendering, but they should be.
Objects that exist through to the present should not have an end date.
Repro Steps
Steps to reproduce the behavior:

Create a way for something that should render
Add a start_date = tag
Upload
Wait for it to render
Go to 4.

Screenshots & URL
https://openhistoricalmap.org/way/198307566#map=18/47.60446/-122.33345&layers=OD
The data is there & has the right time range.

Discussing with @batpad and @geohacker, this might be an issue where Tegola is omitting the start_date and end_date keys when they have no value.
In our timeslider code, here https://github.com/OpenHistoricalMap/mbgl-timeslider/blob/master/mbgl-control-timeslider-control.js#L403-L406, we check for the existence of these keys. First step is to make sure the tiles are keeping that key even if it is null.

I wonder if we can change the filter to something like this:
[
    'all',
    // has OSM ID, and also a start and end date defined (even if blank)
    ['has', 'osm_id'],
    ['any',
      ['has', 'start_date'],
      ['has', 'end_date'],
    ],
    // numerical start/end date either absent (beginning/end of time) or else within range
    [ 'any', ['!has', 'start_decdate'], ['<=', 'start_decdate', maxdate] ],
    [ 'any', ['!has', 'end_decdate'], ['>=', 'end_decdate', mindate] ],
]

So it doesn't need the user to add an end_date tag.

Working on the live site with these test buildings:
https://recordit.co/2jQJM94wvP


WIP: Add "see also-sss"

closes https://github.com/mne-tools/mne-python/issues/2289
It's WIP but I made a PR so that people can jump in and make suggestions for related methods :)

thanks @mainakjas for tackling this. This is very useful. Let us know when you're done.

@mainakjas how close are we on this? It would be good to get this merged sooner rather than later since it's going to touch a lot of functions, and could therefore cause rebasing problems with other PRs.

I could do with one round of reviews.

Have you checked to make sure these all look good when built (e.g., comma-separated functions look good / link properly)?

Looks like a good start, but I'd try to make things more consistent. I'd rather have no descriptions, and omit mne. from all names if possible since it's implied.

we should just make sure it renders fine in the doc with cross links.

let's merge after adressing @Eric89GXL 's comments. It's already a big improvement.

I think I'm good here

merged by rebase. Thanks @mainakjas !


[uart/dv] Update tl outstanding number for coverage

Also increase reseed number to cover reset related coverage points
Signed-off-by: Weicai Yang<EMAIL_ADDRESS>

I am merging this in with the hope that UART will hit 100% in tonight's nightly regression run.

I am merging this in with the hope that UART will hit 100% in tonight's nightly regression run.

found couple issues and fixes are still in my local. Will make another PR to hit 100% coverage


chore(Mathlib/Data/Seq/Seq): add cases_eliminators to Sequence.recOn

Add @[elab_as_elim, induction_eliminator, cases_eliminator] to Seq'.recOn.
This can make the proofs more brief.

[ ] depends on: #13241

This PR consists Continued fractions makes an isomorphism between irrationals and baire space
 project

❗This project is temporarily frozen, PRs are all closed for a moment. Detail


Receipt sent to incognito contributors should list the user's name

Describe the bug
Users who contribute as incognito will get a receipt that says "Bill to: Incognito". For the receipt to be useful for proving a user contributed, it should probably list the user's actual name.
To Reproduce
Steps to reproduce the behavior:

Make an incognito contribution to a collective
Receive email with receipt
View receipt

Expected behavior
Receipt should contain user's name
Screenshots

Smartphone (please complete the following information):

Device: iPhone Xs
OS: iOS 13.2.3
Browser: Safari
Version 13

Additional context
The use case for this is showing to other specific folks that I did pay, without revealing my name publicly.
Thanks!

Thanks for reporting this issue, receipts should always include the user name so that's definitely a bug!

From what I can see, I'm not sure you can get the user's actual name through GraphQL because of https://github.com/opencollective/opencollective-api/blob/116cdae0f79240757a54d96c56ee3624896f5607/server/graphql/v1/types.js#L1551

@bolariin that's indeed probably the problem here. However this GraphQL endpoint normally returns the user info if the user is an admin, so we need to check the following:

That req.remoteUser.isAdmin(incognitoProfile.id) returns true when incognitoProfile is the incognito profile of req.remoteUser.
That the invoice service properly send the authorization header to the endpoint when it makes the GraphQL query. This endpoint used to be unauthenticated (because we consider the UUID as private, so that was a way to protect it by default) and it's possible that we didn't update the request in the invoice service properly.

Fixed by @bolariin in https://github.com/opencollective/opencollective-invoices/pull/285. Will be deployed soon


Compile RocksDB only during Benchmark

Feature Request
now when the test is executed, the compilation of rocksdb will be triggered, which will take a very long time, but rocksdb is only used in the benchmark.

I am so sorry I fixed a good first issue just now 😢 https://github.com/tonbo-io/tonbo/commit/67bcd6d34e93c9ed5c6d8b2884713d8b47147b80#diff-2e9d962a08321605940b5a657135052fbcef87b5e360662bb527c96d9a615542R62


fix(Storage): Fix upload files on mobile

What this PR does 📖
This pull request includes changes to the file upload functionality in the src/routes/files/+page.svelte file. The modifications aim to streamline the file selection process and improve the user experience.
Improvements to file upload functionality:

src/routes/files/+page.svelte: Refactored the onFileSelected function to take target directly as an HTMLInputElement parameter instead of extracting it from the event object.
src/routes/files/+page.svelte: Replaced the existing file input element with a dynamically created input element for file uploads. This change includes creating the input element programmatically, setting its attributes, adding an event listener for the change event, and removing the element from the DOM after the file selection is complete.

Which issue(s) this PR fixes 🔨

Resolve #

Special notes for reviewers 🗒
Additional comments 🎤

so i did

git checkout fix-storage-on-mobile
git pull
npm i
npm run build
npx cap sync
npx cap open ios
compiled to a real iPhone
went to files
upload files

mp4 video of 7.5MB - does not appear pdf file of 38MB - does not appear pdf file of 98KB - appears image of 1.9MB - appears
so it seems only gets uploaded if is very very small

I have tested again, no problem with large files.
iOS

Android

@lgmarchi @stavares843 maybe not related to this ticket but what is the file size limit? I get "file size exceeded errors" in web and app for "big files' like greater of 150 mb
https://github.com/user-attachments/assets/af04ee1b-5370-4511-811f-8084a35b7b49

@lgmarchi @stavares843 File size exceeded error shows for the same file in web dev, so I believe file uploads are good. Now, we need to test this PR with files sharing on chats, but friend requests are broken at the moment. Once that friend requests are stable again, we will test this part and if its all good we can approve

@lgmarchi @stavares843 File size exceeded error shows for the same file in web dev, so I believe file uploads are good. Now, we need to test this PR with files sharing on chats, but friend requests are broken at the moment. Once that friend requests are stable again, we will test this part and if its all good we can approve

I will change PR name, but I think is better to open other PR to send files on chats and we approve this one first.


gutter inside the peek widget [feature request]

Hi , i have a request for a productivity feature.
can we show gutter for files icons inside the peek widget  thanks :)

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


⚠ PrivateHD has degraded performance

In e0c270c, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2049 ms

Resolved: PrivateHD performance has improved in 254617d.


enclave-tls: resolve unstable LA quote issue on github test machine

Has been solved. Please see link.


Size array ordering

Due to the way the sizes array is computed any configuration that includes out-of-order numbers will be converted to an ordered array.
So sizes: [300, 500, 150] appears to resolve to [150, 300, 500] when generating the picture output, and in combination with specifying breakpoints will generate different srcset combinations of src image and media queries than expected.
This was while experimenting with a responsive grid of images: a situation where as the viewport widens smaller images are required.
(This might be a use case better served by using sizes of source in combination with width descriptors in srcset but I think only pixel density descriptors and media queries are available here).

Hi @justsee
Thank you very much for the bug report. I just published a new version in which this should be fixed (for both react-optimized-image and next-optimized-images in case you are also using that). Can you test this new version and check if everything is correct? Thanks!

Great - thanks for jumping on that, and can confirm that resolves this issue (and yes I was using this via next-optimized-images).


I can't create app using the cli

Expectations
Create a new app

Reality
TypeError: Cannot read properties of undefined (reading 'ux')

Steps to Reproduce

Running Zcli at ubuntu 22 under wsl using nodejs under asdf

Issue details

Command:  zcli apps:new
Version: @zendesk/zcli/1.0.0-beta.28 wsl-x64 node-v18.12.1
OS: Ubuntu 22.04.1 (WSL)

I'm experiencing the same issue as well

Command: Any zcli command is giving me the same error message
ZCLI Version: @zendesk/zcli/1.0.0-beta.28
Node Version: v18.11.0
OS: Apple M1 Max Ventura 13.1 (22C65)

Same here - was working before this new beta.

Same issue

Same issue too

Just installed it and am getting the same error. I have Node v19 on MacOS v12.6.2

Same issue with node v16.15.1 and v18.13.0

The bug come from :
https://www.npmjs.com/package/@oclif/core?activeTab=versions
The version is not fixed and there is an issue since they published the 2.0.0-beta that npm install as a dependency instead of 1.xxx
Not found a way to force the version for the moment as it's deep in dependencies.

I found a workaround :
Go to your global node_modules folder, replace the package.json of zcli and run npm i :
working package.json : https://github.com/cylwin/zcli/blob/9d7969337a6ed9eb1cf7a913fa27242a4bb06789/package.json
So you can work on your zendesk app during the weekend :-) waiting the zendesk team to fix the issue better.

I found a workaround :
Go to your global node_modules folder, replace the package.json of zcli and run npm i :
working package.json : https://github.com/cylwin/zcli/blob/9d7969337a6ed9eb1cf7a913fa27242a4bb06789/package.json
So you can work on your zendesk app during the weekend :-) waiting the zendesk team to fix the issue better.

I'll try

Thanks for the fix. It looks like the npm package hasn't been updated yet. Is there a way to get this fix right away?

Agreed. So, why was this closed?

@CH-JohnathanEvans npm uninstall @zendesk/zcli -g and reinstall it it will work.


Fix fatal due to wrong symfony/event-dispatcher version detection

The version detection added in #1256 to support symfony 5 relies
only on checking for the presence of a class provided by the
(separate) symfony/event-dispatcher-contracts package.
However, symfony/console 4.2 also requires symfony/contracts
but can coexist with older versions of symfony/event-dispatcher.
Therefore for example if a project has a constraint on updating
symfony/event-dispatcher (e.g. because of using Guzzle 3,
which requires event-dispatcher ~2.1) composer will install:

symfony/console @ 4.2.12 (latest that doesn't advertise a
conflict with newer event-dispatcher)
symfony/contracts @ 1.1.8 (latest that satisfies the console
requirement)
symfony/event-dispatcher @ 2.8.52 (latest that satisfies
guzzle)

In this setup the presence of symfony/contracts incorrectly
signals to behat that we are running symfony 5 and causes a
fatal error when it loads the new implementation of
TestworkEventDispatcher.
The only change between the two versions is that the method
signature for dispatch has changed order and types.
Instead of looking at packages, it is more robust just to use
reflection to detect which version of the dispatch() method
is defined, and then load the extension class that matches
that signature.
Fixes #1277

The appveyor build appeared to fail due to not being able to find php, I'm not sure what that means but I don't think it can be related to my changes.

@ciaranmcnulty ok so.... it turns out the php72 check is required is required because as of symfony/event-dispatcher 4.3 they have been doing magic themselves in the dispatch method to make it accept both old and new arguments so:

On php < 7.1.3 the absolute latest event-dispatcher you can get is 3.4 which has an official signature of public function dispatch($eventName, Event $event = null) here
On php >= 7.1.3 with event-dispatcher 4.4 the signature is public function dispatch($event/*, string $eventName = null*/) note the second arg is actually commented out here which means the Testwork class can add it as an extra optional param without php complaining about incorrect inheritance
On php >= 7.2.5 with event-dispatcher 5.0 the signature is public function dispatch(object $event, string $eventName = null): object

And note those are cumulative - e.g. on php7.2 you could be running any of (2.x/3.x/4.0/4.1/4.2 - all equivalent, 4.3/4.4 or 5.0.
So I still think a PHP_VERSION constraint is useless / misleading albeit putting it back in has made the specific combinations on your travis go green again.
But I think if you actually want to support all of these, we need to sniff for all three variants of the method signature.
I'm also not sure whether it's safe to use the legacy extension class on 4.3/4.4 since it doesn't do any of the symfony magic to guess which order the arguments are in. If it's only called internally within behat it's probably fine, if anything from symfony core is dispatching events then there may be other unpredictable results not yet encountered.

If this was my package, I'd be tempted to revert all the symfony5 support and release a 3.7, then work on a new 4.0 branch requiring symfony5 and php7. Appreciate you may not want to go that way.

If this was my package, I'd be tempted to revert all the symfony5 support and release a 3.7, then work on a new 4.0 branch requiring symfony5 and php7.

Sadly that won't remove the tag of 3.6.0 :/

I agree it sounds like sniffing the three signatures (FFS) is the right approach - I'd like to do some sort of class_alias to separate out the different implementations from the decision logic

@ciaranmcnulty

Sadly that won't remove the tag of 3.6.0 :/

true but we could at least upgrade past it (and I suspect a lot of people don't have 3.6.0 yet so would go straight to 3.7.0 or higher...)
I think I have a robust sniff for the signatures, if travis passes this time. I agree splitting into class_alias is probably the way to go, I need to take a break just now but will try to look later on.

@ciaranmcnulty that's the fix rewritten to use a class_alias based approach as requested.

This looks great, thank you


Test the queries on this page

Some feedback regarding documentation here.
View SQL Server databases - SQL Server | Microsoft Learn
I wanted to take this query to the Graph

Paste, run.

Comments
resources is technically lowercase by autocorrect, add some spaces after the pipes

Hmm “lincensetype” is an issue.

But lincentype work

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

ID: 38bd60d9-ac77-b45b-8d21-68519ba20328
Version Independent ID: 38bd60d9-ac77-b45b-8d21-68519ba20328
Content: View SQL Server databases - SQL Server
Content Source: docs/sql-server/azure-arc/view-databases.md
Service: sql
GitHub Login: @ntakru
Microsoft Alias: nikitatakru

@lucidqdreams -- Lucid, thank you for your well-researched feedback.  I am updating the article.

@WilliamAntonRohm I'll take this one, if you don't mind.

This is great feedback. Let's update the docs article to reflect the changes in terms of the lower case for resources and license type spelling so that the query can be copy and pasted directly in Azure Resource Graph from there.

@lucidqdreams @ntakru @WilliamAntonRohm This issue will automatically close when the internal PR is merged, and the updated article will be published in the next day or so.


Is it possible to have a standalone solution

Is it possible to have a standalone solution?
So that I don't need to install those dependencies, last time I spend one day to install Caffe and only managed to install it on 2/3 of my computers due to OS (ubuntu 12.04/12.10/14.04)  and dependency (boost,opencv,gcc,cuda,hdf5) issues.
I have disk space, and I have hardware, so I don't mind even if a standalone package is 10GB or more.

The web installer that you can download from https://developer.nvidia.com/digits is a standalone solution, but it is only for Ubuntu 14.04. A platform-independent standalone solution is not possible - even many of the required python packages have to get compiled on installation and wouldn't work on another system.

If you're able to use Docker then I have a standalone CPU-only DIGITS image available at https://registry.hub.docker.com/u/kaixhin/digits/. Instructions are included in the readme.
Obviously this is quite slow - if I am able to redistribute the cuDNN archive with my Docker images then I would be able to create one with CUDA and cuDNN.

Nice work, @Kaixhin! I had started down that path of creating a Dockerfile for DIGITS, but I got sidetracked with other things.

Thanks @lukeyeager, it wasn't very difficult with your clearly documented instructions. Given the license for cuDNN is there any way it can be included in a Docker image? Not only would it let me build the optimal DIGITS image, but also plenty of other deep learning libraries.

cuDNN is currently only available to registered developers. If the license or distribution strategy changes, I'll be sure to let you know.

Thanks for the clarification. Nevertheless Docker seems like a decent standalone solution considering VMs would require GPUs with virtualisation capabilities. The only issue is providing different CUDA/driver versions, and I haven't been able to think of a good solution so far.

Thanks @Kaixhin , glade to know docker.

Thanks all.

@StevenLOL does the Docker container meet your requirements? If so @lukeyeager perhaps it is worth documenting on the readme or elsewhere?

Hi @Kaixhin can you tell how to open digits homepage?

@MiguelGarciaCalderon Sorry looks like there was a problem with digits-server, but digits-devserver seems to be working fine. I've updated the Dockerfiles and rebuilt/tested kaixhin/digits successfully. Rebuilding kaixhin/cuda-digits now so those should be ready within a few hours.

@MiguelGarciaCalderon @Kaixhin @StevenLOL
There is now an official Docker solution from NVIDIA which includes prebuilt images for DIGITS:
https://github.com/NVIDIA/nvidia-docker/wiki/DIGITS

Thank you I m going to use it


fix: 4304 - save images as async instead of isolate (fallback)

What

We use isolates when we save images.
As part of #4304, we now try to save images with standard async methods, as a fallback if it failed with isolates.
Not really sure if it makes sense, as for the moment we don't really know what goes wrong when it goes wrong (could be the isolate part, could be the save image part). We'll know better once we get Sentry messages (cf. #4305).

Part of

#4304

Codecov Report

Merging #4323 (2575620) into develop (1cd22a6) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop    #4323      +/-   ##
===========================================
- Coverage    10.80%   10.78%   -0.02%
===========================================
  Files          287      287
  Lines        14202    14218      +16
===========================================
  Hits          1534     1534
- Misses       12668    12684      +16

Impacted Files
Coverage Δ

...mooth_app/lib/helpers/image_compute_container.dart
0.00% <0.00%> (ø)

... and 8 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


get_data for USA subregion georgia pulls data from the country of Georgia.

When pulling data from subregions in the United States, when you get to Georgia, the get_data call will download data from the country of Georgia rather than from the state of Georgia.
import pyrosm

subregions = pyrosm.data.sources.subregions.usa.available
assert subregions[9] == 'georgia'
osm = pyrosm.OSM(pyrosm.get_data(subregions[9]))
pois = osm.get_pois({'amenity':True})

assert len(pois[pois.centroid.x < 0]) == 0

All the POIs that are retrieved will be in the country of Georgia, rather than the USA state of Georgia.

@jspalink Oh my. 😬 Thanks for reporting, will take a look at this! 👍🏻

perhaps get_data could use the "id" of the region provided by the geojson from https://download.geofabrik.de/index-v1.json for download, seems like they have already thought about these conflicts.


Update to Catch Version 3

Is this pull request associated with an issue(s)?
None
Description
Title

🚀 [bumpr] Bumped! New version:v0.0.2 Changes:v0.0.1...v0.0.2


Any VSIX download link can be opened from VS Code Marketplace

Azure Data Studio Version:

Version: 1.8.0 (user setup) Commit: 96e214593190071f0b520439929b218f9fa673de Date: 2019-06-06T01:59:30.203Z VS Code 1.34.0 Electron: 3.1.8 Chrome: 66.0.3359.181 Node.js: 10.2.0 V8: 6.6.346.32 OS: Windows_NT x64 10.0.18362
Steps to Reproduce:
Trying to install a "Atom One Dark Theme" I clicked on "download as a .vsix file" link and nothing happens. Same for all links. Event when I clicked on "Preview on GitHub" button to report this bug :P

@edelCustodio please try the following steps to install an extension from the VS Code marketplace:

Download the extension from the marketplace using the "Download as VSIX" link
Open Azure Data Studio and run the "Install Extension From VSIX Package" from the file menu
The extension is usable once a notification message appears confirm the installation is complete

Please let me know if this does not address your question.

Closing based on previous comment.  Thanks!


The scraper does not show that the announcement job for Back-office Junior Customer Service Agent with German was posted 5 times

ENVIRONMENT:
https://admin.peviitor.ro/
Browser: Chrome
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:
Enter url in browser
In go to "https://admin.peviitor.ro/" and click to jobs for "Allianz"
Click on "Back-office Junior Customer Service Agent with German" tab on "Catre Job "option
EXPECTED RESULTS:
The page shows the full description of the job
ACTUAL RESULTS:
The pages show the same job announcement at accessing the other 4 tabs
See below the screenshot
Attachment:
Page from Validator for Allianz

Jobul nu mai este disponibil, dar din poze par a fi 2 joburi complet diferite. Uită-te atent la link ul jobului ca au id uri unice. Nu sunt duplicate.


cuMemFree failed: initialization error

python tools/tsdf_fusion/generate_gt.py --data_path ~/scannet/tasks/ --save_name all_tsdf_9 --window_size 9
When i ran the command above, it occured
(process_with_single_worker pid=) cuModuleUnload failed: initialization error
(process_with_single_worker pid=) PyCUDA WARNING: a clean-up operation failed (dead context maybe?)
(process_with_single_worker pid=) cuMemFree failed: initialization error

https://github.com/inducer/pycuda/issues/137


[AIRFLOW-2733] Reconcile psutil and subprocess in webserver cli

Make sure you have checked all steps below.
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-2733

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

use psutil.Popen to reconcile psutil and subprocess usage

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #3586 into master will increase coverage by 0.01%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master    #3586      +/-   ##
==========================================
+ Coverage   76.83%   76.84%   +0.01%
==========================================
  Files         204      204
  Lines       15492    15491       -1
==========================================
+ Hits        11903    11904       +1
+ Misses       3589     3587       -2

Impacted Files
Coverage Δ

airflow/bin/cli.py
64.16% <50%> (+0.08%)
:arrow_up:

airflow/models.py
88.72% <0%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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

I would suggest to move all the subprocess calls to psutil :-)
Beside the gunicorn.wait, is there a difference in behaviour?

In the usage of both of these objects, poll is the only method that is not shared between psutil.Process, psutil.Popen, and subprocess.Popen. I cannot find a common function for this purpose between psutil.Process and subprocess.Popen.
psutil.Popen is a convenience wrapper around subprocess.Popen that adds methods provided by psutil.Process.
As an alternative, in this case:
gunicorn_master_proc = psutil.Popen(run_args, close_fds=True)

could have been written as:
gunicorn_master_subproc = psutil.Popen(run_args, close_fds=True)
gunicorn_master_proc = psutil.Process(gunicorn_master_subproc.pid)

but that seems needlessly verbose.
I support the idea of migrating subprocess.Popen calls to psutil.Popen but feel that would be a larger undertaking and would prefer to do that as more of a search/replace separate from functional changes like this poll -> wait change.


Issue with k8s.io/docs/setup/independent/install-kubeadm/

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
No need to explicitly mentioned 'kubectl' , while installing on centOS as kubeadm has dependency on kubeadm it will install automatically.
yum install -y kubelet kubeadm kubectl
Proposed Solution:
yum install -y kubelet kubeadm
Page to Update:
https://kubernetes.io/...

o/p
yum install -y kubeadm-1.9.1-0
Loaded plugins: fastestmirror, langpacks
Loading mirror speeds from cached hostfile

base: centos.mirror.ndchost.com
extras: mirror.tocici.com
updates: pubmirrors.dal.corespace.com
Resolving Dependencies
--> Running transaction check
---> Package kubeadm.x86_64 0:1.9.1-0 will be installed
--> Processing Dependency: kubectl >= 1.6.0 for package: kubeadm-1.9.1-0.x86_64
--> Running transaction check
---> Package kubectl.x86_64 0:1.10.0-0 will be installed
--> Finished Dependency Resolution

Dependencies Resolved
===============================================================================================================
Package                  Arch                    Version                    Repository                   Size
Installing:
kubeadm                  x86_64                  1.9.1-0                    kubernetes                   16 M
Installing for dependencies:
kubectl                  x86_64                  1.10.0-0                   kubernetes                  7.6 M
Transaction Summary
Install  1 Package (+1 Dependent package)
Total download size: 24 M
Installed size: 137 M
Downloading packages:
(1/2): 9ff2e28300e012e2b692e1d4445786f0bed0fd5c13ef650d61369097bfdd0519-kubectl-1.10.0- | 7.6 MB  00:00:01
(2/2): ccc2b7d8ac99c8ead43087e7bb5cc7fb3fe684bfd78241c8240feb945303c40e-kubeadm-1.9.1-0 |  16 MB  00:00:02
Total                                                                          9.2 MB/s |  24 MB  00:00:02
Running transaction check
Running transaction test
Transaction test succeeded
Running transaction
Installing : kubectl-1.10.0-0.x86_64                                                                     1/2
Installing : kubeadm-1.9.1-0.x86_64                                                                      2/2
Verifying  : kubeadm-1.9.1-0.x86_64                                                                      1/2
Verifying  : kubectl-1.10.0-0.x86_64                                                                     2/2
Installed:
kubeadm.x86_64 0:1.9.1-0
Dependency Installed:
kubectl.x86_64 0:1.10.0-0
Complete!

in terms of showing to the user that kubectl will be installed too, i don't see anything wrong with adding the extra package in a yum call unless it throws errors:
yum install -y kubelet kubeadm kubectl

please, re-open if see fit.
/close


Library request on CDNJS

Hi
I am a member of CDNJS. There is a library request cdnjs/cdnjs#10517.
We only host the library running on the browser.
Can you please tell that is this library compatible with browser? Thank you! :)

Hi @extend1994 !
It is designed for node.js to begin with. However, many users make it work for the browser using Browserify that includes the necessary shims or Webpack with adding the shims for some node core libraries manually.
Let me know if you want to know more details.

@analog-nico Thanks for help! I am asking some help from my mentors and will handle the library request as soon as possible.

@analog-nico thanks, I wonder if it's possible to provide officially Browserified release files here? So that we can keep the files' integrity and confidence, thanks.

Sorry, that I didn't bring this up earlier: Using request-promise in the browser is not officially supported, yet. All pending TODOs are listed in issue #93 . Until this issue is closed I cannot provide a browserified release file unfortunately.
Creating a browserified release file is effortless. But providing an official one comes with certain expectations regarding the quality standard. The same is most likely true for uploading one to CDNJS. So I guess we need for issue #93 to get done. And the progress is slow...
Do you see other options to proceed?

I think we can just wait for that, the quality matters 👍

Alright. Let's keep this issue open and I'll notify you here once the browserified release is ready.

Fair enough, thanks @analog-nico

I'd like a browserified release file just for experimentation. Would love to use this library in the browser. Can you just release one as <v1.0 and have v1.0 be the 'quality standard'?

Sorry @JuanAntelo you will have to do the build yourself. As soon as I do any release I have to be ready to support the community by answering questions etc. An experimental version would produce too much support work for me.


Missing WindowsAppSDK push-documentation with azure notification hubs

please share an documentation update on how to configure WindowsAppSDK WNS PushNotifications
(https://learn.microsoft.com/en-us/windows/apps/windows-app-sdk/notifications/push-notifications/push-quickstart)

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

ID: 1cec44d3-da71-5c8e-61c3-e9704db79ab2
Version Independent ID: eb6eceb2-83a9-c28a-2072-909308202498
Content: Configure Windows Push Notification Service in Azure Notification Hubs
Content Source: articles/notification-hubs/configure-windows-push-notification-service.md
Service: notification-hubs
GitHub Login: @sethmanheim
Microsoft Alias: sethm

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

@baaaaif Thanks for sharing feedback! Could you please share more insights on requirement for adding suggested document link ((https://learn.microsoft.com/en-us/windows/apps/windows-app-sdk/notifications/push-notifications/push-quickstart) as this repo is for Azure Notification hubs.

@baaaaif Thanks for sharing feedback! Could you please share more insights on requirement for adding suggested document link ((https://learn.microsoft.com/en-us/windows/apps/windows-app-sdk/notifications/push-notifications/push-quickstart) as this repo is for Azure Notification hubs.

The current documentation only shows how to do a configuration for the "old" UWP apps where you have to specify the package SID.
The UWP successor apps that are created with the WindowsAppSDK now also support push via WNS, but not with the package SID and the PartnerCenter, but with the AAD app registration.
"Push notifications in Windows App SDK use identities from Azure Active Directory (AAD). Azure credentials are required when requesting a WNS Channel URI and when requesting access tokens in order to send push notifications. Note: We do NOT support using Windows App SDK push notifications with Microsoft Partner Center."
How to configure that with the AAD App Registration here in the NotificationHub for WNS is missing

@baaaaif Thanks for the clarification! I have assigned the issue to the content author to review further and update the document as appropriate.

@baaaaif Very sorry for the slow response here. We are aware that the content is out of date and are working with the engineering team to update all of the tutorials. If this is still an issue and you need assistance, please contact Microsoft Support or open a support ticket from the Azure portal. #please-close


[BugFix] Fix transaction stream load LOAD_FINISHED metrics

Fixes #issue
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[x] 3.0
[x] 2.5
[ ] 2.4

[FE Incremental Coverage Report]
:disappointed: fail : 0 / 1 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/http/rest/TransactionLoadAction.java
0
1
00.00%
[175]

[FE Incremental Coverage Report]
:disappointed: fail : 0 / 1 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/http/rest/TransactionLoadAction.java
0
1
00.00%
[175]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:disappointed: fail : 0 / 1 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/http/rest/TransactionLoadAction.java
0
1
00.00%
[175]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:disappointed: fail : 0 / 1 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/http/rest/TransactionLoadAction.java
0
1
00.00%
[175]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:disappointed: fail : 0 / 1 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/http/rest/TransactionLoadAction.java
0
1
00.00%
[175]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:disappointed: fail : 0 / 1 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/http/rest/TransactionLoadAction.java
0
1
00.00%
[175]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:heart_eyes: pass : 1 / 1 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/transaction/GlobalTransactionMgr.java
1
1
100.00%
[]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5


index out of range [-1] (gofakeit.Paragraph)

Similar to https://github.com/brianvoe/gofakeit/issues/175 and https://github.com/brianvoe/gofakeit/issues/155, intermittently seeing the following error when running gofakeit.Paragraph() under load:
panic: runtime error: index out of range [-1]

goroutine 313 [running]:
math/rand.(*rngSource).Uint64(...)
	/usr/local/opt/go/libexec/src/math/rand/rng.go:249
math/rand.(*rngSource).Int63(0xc000908000)
	/usr/local/opt/go/libexec/src/math/rand/rng.go:234 +0x9f
math/rand.(*Rand).Int63(...)
	/usr/local/opt/go/libexec/src/math/rand/rand.go:84
math/rand.(*Rand).Int31(...)
	/usr/local/opt/go/libexec/src/math/rand/rand.go:98
math/rand.(*Rand).Int31n(0xc000194840, 0x1f4)
	/usr/local/opt/go/libexec/src/math/rand/rand.go:133 +0x59
math/rand.(*Rand).Intn(0x1729660, 0xc000194750)
	/usr/local/opt/go/libexec/src/math/rand/rand.go:171 +0x2e
github.com/brianvoe/gofakeit/v6.getRandValue(0x2, {0xc000db5d88, 0xc000904da8, 0x10f0cde})
	go/pkg/mod/github.com/brianvoe/gofakeit/v6@v6.8.0/helpers.go:67 +0xf3
github.com/brianvoe/gofakeit/v6.word(0xc000904e10)
	go/pkg/mod/github.com/brianvoe/gofakeit/v6@v6.8.0/words.go:74 +0xcf
github.com/brianvoe/gofakeit/v6.sentenceGen(0x258, 0x14, 0x186add8)
	go/pkg/mod/github.com/brianvoe/gofakeit/v6@v6.8.0/words.go:111 +0xf8
github.com/brianvoe/gofakeit/v6.sentence(0xc000904f30, 0xc001d0d5f0)
	go/pkg/mod/github.com/brianvoe/gofakeit/v6@v6.8.0/words.go:84 +0x25
github.com/brianvoe/gofakeit/v6.paragraphGen(0x0, {0x1, 0x5, 0x14, {0x1929f88, 0x1}}, 0x186add0)
	go/pkg/mod/github.com/brianvoe/gofakeit/v6@v6.8.0/words.go:139 +0x1bd
github.com/brianvoe/gofakeit/v6.paragraph(...)
	go/pkg/mod/github.com/brianvoe/gofakeit/v6@v6.8.0/words.go:98
github.com/brianvoe/gofakeit/v6.Paragraph(...)
	go/pkg/mod/github.com/brianvoe/gofakeit/v6@v6.8.0/words.go:89

It is not hard to reproduce the issue with large enough number of goroutines:
func TestConcurrencyParagraph(t *testing.T) {
	var setupComplete sync.WaitGroup
	setupComplete.Add(1)

	var wg sync.WaitGroup
	for i := 0; i < 10000; i++ {
		wg.Add(1)
		go func() {
			setupComplete.Wait()
			gofakeit.Paragraph(1, 5, 20, " ")
			wg.Done()
		}()
	}

	setupComplete.Done()
	wg.Wait()
}

The version I used: github.com/brianvoe/gofakeit/v6 v6.8.0

The default rand.Rand source isn't safe for concurrent use. The rand package functions use a singe source protected by a mutex.
Ill update docs to better describe this issue

After a little thought on this. I have had too many people submit this issue of the index being out of range. I understand its an issue with using math rand NewSource(which is what i have to use) but its not worth it anymore even though it will take a performance hit because of it. Ill be making the update today to resolve this.

v6.9.0


Map 9 pois

Code looks for me ok with Pauls advices. But unfortunately I can't see the pois on the map and have no idea why. Did I forget to run something else in addition to the migration?

Yeah, took me some time as well^^ you need to run rails db:seed to add the example POIs to the database


Not working on Windows 10

It's not working with windows 10, the editor start flick

I also have problems with ghosttext in visual studio code on windows 10.
In sublime text works great.

It sounds like this is an issue with the visual studio plugin. Please open an issue in their repo


Starting jellex gives: RuntimeError: no running event loop

I installed jellex via pip. jello runs fine. I get this:

jellex 1.json
Traceback (most recent call last):
File "/home/keymaster/.local/bin/jellex", line 5, in
from jellex.cli import main
File "/home/keymaster/.local/lib/python3.10/site-packages/jellex/cli.py", line 231, in
query.insert_text('_')
File "/home/keymaster/.local/lib/python3.10/site-packages/prompt_toolkit/buffer.py", line 1269, in insert_text
get_app().create_background_task(self._async_completer())
File "/home/keymaster/.local/lib/python3.10/site-packages/prompt_toolkit/application/application.py", line 1057, in create_background_task
loop = self.loop or get_running_loop()
RuntimeError: no running event loop
sys:1: RuntimeWarning: coroutine 'Buffer._create_completer_coroutine..async_completer' was never awaited

$ uname -a
Linux zabbix 5.15.0-47-generic #51-Ubuntu SMP Thu Aug 11 07:51:15 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
$ python --version
Python 3.10.4

Thank you for reporting this! It looks like the latest version of one of the dependent libraries (prompt-toolkit) is causing the issue. You can downgrade prompt-toolkit to the previous version, which seems to work ok:
pip install pip install prompt-toolkit==3.0.36

In the mean-time I'll need to hardcode that version in the setup file.

Downgrading prompt-toolkit works.

I have hardcoded the requirements in jellex v0.5.7


Prepare for release

API change check for Microsoft.Azure.Data.SchemaRegistry.ApacheAvro
API changes are not detected in this pull request for Microsoft.Azure.Data.SchemaRegistry.ApacheAvro


Question: AWS Amplify Graphql S3 Integration

Reading through the documentation I see this line:
"The GraphQL Transform handles creating the relevant input types and will store pointers to S3 objects in Amazon DynamoDB. The AppSync SDKs and Amplify library handle uploading the files to S3 transparently."
From what I understand, this means by running a graphql request with the described S3Object definition I should be able to do a mutation and by running that mutation upload the data to S3. However, that doesn't happen. I'm curious to know if this functionality exists and how to do it or if I am misinterpreting what's being said, thanks!
https://docs.amplify.aws/cli/graphql-transformer/storage#basics

Hi @obrian33 Can you please provide the code snippet that you are trying. It would be great if you can provide the following details to help us debug the issue:

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment(please complete the following information):

Amplify Framework Version: [e.g. 0.10.0]
Dependency Manager: [e.g. Cocoapods, Carthage]
Swift Version : [e.g. 5.0]

Device Information (please complete the following information):

Device: [e.g. iPhone6, Simulator]
iOS Version: [e.g. iOS 11]
Specific to simulators:

Additional context
Add any other context about the problem here.

In case of iOS uploading data to S3 is supported for iOS Appsync SDK, please find the documentation here under Working with complex objects.
iOS Amplify API doesnot support complex objects currently.

Thanks for the response!
Here is the additional info requested:
To Reproduce:
Create a Amazon S3 bucket to hold files via amplify add storage.
Create a user pool in Amazon Cognito User Pools via amplify add auth.
Create a GraphQL API via amplify add api and add the following type definition:
type S3Object {
bucket: String!
region: String!
key: String!
}
type Picture @model @auth(rules: [{allow: owner}]) {
id: ID!
name: String
owner: String
file: S3Object
}
Run this mutation with S3 objects from your client app:
mutation ($input: CreatePictureInput!) {
createPicture(input: $input) {
id
name
visibility
owner
createdAt
file {
region
bucket
key
}
}
}
Expected Behavior:
Uploaded complex S3Object will appear in the specified bucket and region with inputted key.
Environment:

Amplify Framework Version: 1.1.1

Dependency Manager: Cocoapods

Swift Version : 5.0

Device Information:

Device: iPhone 11 Pro Simulator (13.3)

@royjit Your latest comment seems to address my main concern because I am not using the AWS Mobile SDK but strictly the iOS Amplify API. My current workaround is to run a graphql mutation and then use the Amplify.Storage.uploadData function to upload and then the Amplify.Storage.downloadData function during queries to retrieve the complex objects from S3, is there any plans to include complex object support for Amplify.API?

Yes, you are right that would be the current approach. We will take this as a feature request and will update here when we have more info.

Thanks! Excited to see what comes next :)

related issue on the SDK / AppSync SDK https://github.com/aws-amplify/aws-sdk-ios/issues/1740

@royjit Any updates on using complex objects with Amplify?


Scrapers not working

Using the last official build of ES installed on Windows 10. Selected TheGamesDB for scraping.
Runs through the entire game list and returns "No Games scraped." Log file states All games were ignored even with the gamelist.xml present. Went to TheGamesDB discord and they said it wasn't on there end. Any response would be greatly appreciated.

I have the same issue but this issue still being open 3 years later. I think this project is dead looking at the open issues.


[Question] what's different between -DskipTests and -DskipITs?

hi, can someone explain what's different between -DskipTests and -DskipITs ? thanks.

Hi, -DskipTests  means skip Unit Tests and -DskipITs means skip Intergrate Test(mainly in hudi-integ-test module).

Hi, -DskipTests  means skip Unit Tests and -DskipITs means skip Intergrate Test(mainly in hudi-integ-test module).

Ok, thanks.


Clicking a project title takes one to a "Project Details" page

Following up from #172, I'd like this to be a "Project Details" page, with "Project Details: " actually included in the page title preceding the project title, as shown in #247

Hey team! Please add your planning poker estimate with Zenhub @JaymeeH @jrgriffiniii @leefaisonr

@matthewjchandler @hectorcorrea Since there is no Product details page --  as of right now --  I have a "Product Details:" link_text pointing to the Tiger Data homepage. Is that ok for now or should I also setup a boilerplate page for it to link to? Is that necessary since that will be taken care of in issue #264 when the page actually gets created -- and we leave a note to comeback and update the link path?

@JaymeeH The project show page (https://github.com/pulibrary/tiger-data-app/blob/main/app/views/projects/show.html.erb) is the "Product Details"

My concern was mainly about the title that displays on the page for the user. It should be "Project Details: [insert project title]". I believe you all have been calling this the "show" page, where the project details are shown, but I've been calling it the "Project Details" page in my design materials

@JaymeeH Yes, I don't have a need for a link out to a different page in this issue. It's really just about the page title displayed for the user


[stable/sentry] Add Ingress extraPaths to prepend to every host configuration

Add Ingress extra paths to prepend to every host configuration. Useful when configuring custom actions with AWS ALB Ingress Controller.
This helps manage to route the paths using ingress controllers. For example - alb-ingress-controller and chart ingress settings:
ingress:
  enabled: true
  annotations:
    kubernetes.io/ingress.class: "alb"
    alb.ingress.kubernetes.io/scheme: "internet-facing"
    alb.ingress.kubernetes.io/backend-protocol: "HTTP"
    alb.ingress.kubernetes.io/ip-address-type: "ipv4"
    alb.ingress.kubernetes.io/listen-ports: '[{"HTTP": 80}, {"HTTPS": 443}]'
    # Redirect rules - actions.ssl-redirect
    alb.ingress.kubernetes.io/actions.ssl-redirect: '{"Type": "redirect", "RedirectConfig": { "Protocol": "HTTPS", "Port": "443", "StatusCode": "HTTP_301"}}'

And ingress path settings:
ingress:
  enabled: true
  ...
  extraPaths:
    - path: /*
      backend:
        serviceName: ssl-redirect
        servicePort: use-annotation

In this case, all requests from port 80 will be redirected to 443 port. More info you can find here.
Checklist

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test

/lgtm


Remove MutResult

prior pr #1072

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

420 of 447 (93.96%)  changed or added relevant lines in 23 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 98.531%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/src/executor/alter/index.rs
18
19
94.74%

core/src/executor/insert.rs
17
18
94.44%

storages/sled-storage/src/transaction.rs
42
43
97.67%

storages/shared-memory-storage/src/alter_table.rs
12
14
85.71%

storages/sled-storage/src/alter_table.rs
24
26
92.31%

storages/sled-storage/src/index_mut.rs
17
19
89.47%

storages/sled-storage/tests/sled_transaction.rs
5
7
71.43%

core/src/executor/alter/table.rs
90
96
93.75%

core/src/executor/execute.rs
90
100
90.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
37954

Relevant Lines:
38520

💛  - Coveralls


Pull Requests

Hey, @Invertee are you open to pull requests. I've started to use this for a side project and have a couple issues that I want to fix.

Hay @stephenc01, absolutely! I've just merged your changes and I'll publish the new version.

Hay @stephenc01, absolutely! I've just merged your changes and I'll publish the new version.


About Us page

Feature request-
Currently the font colour and background colour of the about page is kind of merging and making it difficult to distinguish between text and page.
the solution I'd like to implement
First of all change in colour shade,font and texture. Adding proper alignment with div and a further requests is to provide some images inorder to add a carousel to make it more interactive.
how this feature will be useful to our readers.
Having right alignment and colour shades always increases the user interactivity and helps to make a impact in the subconscious mind of the user to make them visit the web page again.

level - 1: 3 days
Working under GGSOC'21

Hii Can I work on this issue? I am a gssoc participant , would like to work on this issue

Is this issue still relevant ? Asking because the theme of the website was changed.
If yes, can you add a screenshot highlighting the issue you feel there is 🙂

/revewied


CreateDB on PostgreSQL fails

The problem is when creating the database for a user that doesn't have any other privileges.  To replicate:
Create a new user:
#>psql template1
create role test_user with createdb login password 'test123';
\q

Setup config file as follows:
default: &default
  dialect: postgres
  host: <IP_ADDRESS>
  pool: 5
  user: test_user
  password: test123

development:
  <<: *default
  database: my_testing_db

Then try to create a DB:
#> soda create -a

You'll get an error to the effect of:
Error: couldn't create database my_testing_db: error creating PostgreSQL database my_testing_db: pq: database "test_user" does not exist

The solution is for the client to connect to template1 database before running the create SQL.  This should, also, be the case for dropping a database as well.

The https://github.com/markbates/pop (v3.x) repo has been moved to https://github.com/gobuffalo/pop (v4.x). The markbates/pop repo has been put into READ-ONLY mode and will no longer be updated. You may continue to use this repository, but be aware no more updates will be made on it.
If this is issues is still relevant, please open a new issue on the repository.
For more information on this move see https://github.com/markbates/pop/issues/164
Thank you so much and I apologize in advance for any trouble this may cause you.


Inconsistent virtual text line ordering

Steps to reproduce
When adding virtual text lines, the ordering seems to be inconsistent between Windows and Linux (Debian). I have run the following:
call prop_type_add('test', { 'highlight': 'Normal' })

call prop_add(1, 0, { 'type': 'test', 'text': 'A', 'text_align': 'below' })
call prop_add(1, 0, { 'type': 'test', 'text': 'B', 'text_align': 'below' })
call prop_add(1, 0, { 'type': 'test', 'text': 'C', 'text_align': 'below' })

Result on Linux:
A
B
C

Result on Windows:
C
B
A

I don't have much experience with vimscript so I apologise in advance if I made any wrong assumptions.
Expected behaviour
A
B
C

Version of Vim
9.0.242
Environment
Windows 10 and Debian 11 with same version of gVim.
Logs and stack traces
No response

Steps to reproduce
When adding virtual text lines, the ordering seems to be inconsistent between Windows and Linux (Debian). I have run the following:
call prop_type_add('test', { 'highlight': 'Normal' })

call prop_add(1, 0, { 'type': 'test', 'text': 'A', 'text_align': 'below' })
call prop_add(1, 0, { 'type': 'test', 'text': 'B', 'text_align': 'below' })
call prop_add(1, 0, { 'type': 'test', 'text': 'C', 'text_align': 'below' })

I don't have much experience with vimscript so I apologise in advance if I made any wrong assumptions.
Expected behaviour
Result on Linux:
A
B
C

Result on Windows:
C
B
A

Version of Vim
9.0.242
Environment
Windows 10 and Debian 11 with same version of gVim.

The properties are sorted with qsort(), and we have seen before that in
edge cases it works different from qsort() on Unix.
If you give your properties a different type then you can use the
priority to fix the sorting order.
Hmm, I think we can use the ID itself as a tie breaker, since each
proprty with text has a different ID.
--
The Law of VIM:
For each member b of the possible behaviour space B of program P, there exists
a finite time t before which at least one user u in the total user space U of
program P will request b becomes a member of the allowed behaviour space B'
(B' <= B).
In other words: Sooner or later everyone wants everything as an option.
-- Vince Negri
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Allow payloads of 0 or false

When a Promise resolves with a value of 0 or false, include that value as the payload property of the FULFILLED action.
The existing code omits the payload if it is any “falsy” value, which includes null, undefined, 0, and false.
With this change, null and undefined payloads are still omitted, but 0 and false payloads are passed along with the FULFILLED action.
A slightly more idiomatic check for a null or undefined payload would be newPayload == null, but that is disallowed by the lint configuration.
I’ve added new specs for these cases, but they’re very similar to each other and have a fair bit of duplication with the null spec.  I wasn’t sure what you might want to do to clean that up.

Coverage remained the same at 100.0% when pulling 39137341b0beefe04356d33003df52b3ba970758 on CodingZeal:falsy-payloads into d3c55113a088f1dc7a637ca190586c44096ebebc on pburtchaell:master.

Why is this check here in the first place? Is there a reason why a payload couldn't explicitly be undefined or null?

That's a good question.  I just assumed there was a reason for it when I submitted this PR, but if there's no need for the check, I'd be happy to take it out.

This looks fine to me! Thanks for the PR, @randycoulman.
@mmiller42, as far as why the check is there in the first place: it isn't necessary for the middleware to function and we could remove it. However, since the payload property represents the actual information/message transmitted, I would argue it should only be included if a is value provided by the consumer. In other words, the middleware shouldn't add payload by default. Does that make sense?

@randycoulman Could you rebase this PR please?

@pburtchaell Sort of makes sense to me. I'm just thinking along the lines of, say, a payload that is explicitly null.
I think the simpler the logic is here, the more obvious it is what the middleware does. Take, for instance, a developer who uses promises but is using the middleware for the first time.
From a pure promise-based approach:
let promise = Promise.resolve(null);
promise.then(result => assert.strictEqual(result, null));

A developer using this middleware who sets the payload to be null might implement a check later on, if (action.payload === null) and is confused that, instead, payload doesn't exist at all.
It's splitting hairs, but I think if an explicit null or undefined is given, it's clearer what is going on in the middleware if it's passed along as is.

@pburtchaell Rebase is done!

I think @mmiller42's reasoning makes a lot of sense, and matches what I expected when I read the docs for this project.  That said, I don't think the current implementation would trip me up that much.
I'm happy to leave this PR as-is, or to modify it along the lines that @mmiller42 is suggesting.  Just let me know your preference.

Coverage remained the same at 100.0% when pulling 7a8e80ad8373aa03de6e6707d7a548be00a7edb9 on CodingZeal:falsy-payloads into 58504797c11efde49437d0a56e42c4ccf7330510 on pburtchaell:master.

@mmiller42 I should clarify that my response was to "Why is this check here in the first place?" The payload property shouldn't be included by default, that's all. Realistically, I'm fine with supporting null/undefined payloads, but—as you said—it must be explicitly given by the consumer.
However, I do think you should consider the value of this optimization. I don't see a use case for payloads of null and undefined, which makes me wonder if it will be a valuable change. Actions describe changes in state. Given these two examples, it seems like action A describes the same change in state as action B.
// A
{
  type: 'ACTION`,
  meta: ...
}

// B
{
  type: 'ACTION'
  payload: null,
  meta: ...
}

Let me know what you think?
@randycoulman As for this PR, I'm going to merge as is. I like to keep changes small! If we decide to implement null/undefined payloads, that can be another PR. Thank you for the quick rebase!

Fair enough. You're right. For one, undefined isn't even serializable. I can see the logic behind a null payload being equivalent to an absent payload. It might be worth throwing this in the readme though maybe, just to make sure developers don't run into this case. It happened to me and I spent a good chunk of time trying to figure out what was wrong with my action creator.

@mmiller42 I just added a guide here: https://github.com/pburtchaell/redux-promise-middleware/commit/cecbc62dd56ffcc185565f6bcd4aed7470127f20

if I make a PUT request which will just return 204 with no content if success, then don't I expect it to return null as is?

@cyberhck AFAIA that'll depend on the method or library you're using for making the PUT request and what they return from a 204. It could be null, an empty string or even an error if the lib doesn't check status, content-length or type. This middleware should just do whatever the implementation of the PUT request tells is to do.

I always thought middleware shouldn't do anything, just pass whatever it gets, without caring about anything, that should be upto the user to discard later. If it gets null, pass null, if 0 pass 0.

Yeh that's right.

I wasted quite a bit of time until I found the guide.
{ foo: undefined } is my initial state. I then look for an entity. If it is found, then the state is { foo: entity }. If it could not be found, the state is { foo: null }.
I now have to use a workaround e.g. in the reducer by checking for an absent payload and then explicitly writing it as null into the state.
That's a bummer, as there is no disadvantage in allowing any value as a payload.


[CLI]: Large files not successfully uploading when resuming

Describe the bug

When my run finished training, it accidentally deleted a checkpoint (using wandb.Api()) that I didn't want it to. I always backup the wandb-resume.json file so I can resume training even after calling finish with exit_code=0. So I re-run the training run, and at the start of training it calls wandb.save for all checkpoint files, which should have re-uploaded the checkpoint I deleted via wandb.Api().
The run ends, and calling finish() uploads all the files after a minute or so. But only the small files show up, not the large ones. I checked the files directory and I see the symlinks for all checkpoints, which look correct. I tried changing the policy from 'now' to 'live', and reverted back to Wandb version 0.16.6 (I was on 0.16.7 dev). No luck.
To narrow down the issue I added some other files, a dummy .pth file (with just a line of text), and a different checkpoint. The dummy .pth file uploads fine. Both large checkpoints go through the process of uploading, but they never appear in the web GUI nor when querying the files via wandb.Api().
I also tried using the CLI to call wandb sync and it said it sync'ed but the large checkpoints are still missing.
Additional Files
I've attached my log files. The large checkpoints are best_coco_PQ_epoch_70.pth and best_coco_PQ_epoch_150.pth, they failed to upload (but didn't throw any exceptions / print any errors). The dummy checkpoint is best_test.pth which uploaded fine.
debug-internal.log
debug.log
Environment
WandB version: 0.16.6
OS: Ubuntu 22.04.4 LTS
Python version: 3.11.8
Versions of relevant libraries: Conda env with any packages installed. Let me know if you want the full list.
Additional Context
This issue only seems to happen when resuming. If I delete the wandb-resume.json file it creates a new run, uploads the files, then immediately exists, and everything works. It takes a minute to upload, and I immediately see the files on Wandb.

@collinmccarthy could it be that at some point you deleted these files?
this is the only way i was able to reproduce this behavior.
If so that would explain the behavior your are seeing, it seems that once we delete the run, currently, the UI doesn't show it even when you re-upload it.

Yes, I did delete the files via the Api, accidentally. Now I would like to re-add them to the same run that I deleted them from.

@collinmccarthy looks like currently it is not possible to re-use the same name for delete files, our backend team tries decides what's the best way to solve this.
In the short-term i would suggest to change the name slightly, maybe add some indexing prefix or any other schema that makes in your code.
sorry for the trouble here.

@kptkin Got it, thank you for the update. I'll go with the prefix approach for now, hopefully I won't accidentally delete something again anyway.

Hi @collinmccarthy we have raised a bug request for this and our Engineering team is already looking at this issue. We will get back to you for an update.


Issue with test code

I've created a new project and followed the steps to install / link this library.
The issue that I have is when I replace the code in index.ios.js with the example code:
https://gist.github.com/circuit-monkey/2562e02d4f3fc92e35bce1fa5dcc8df4
Then I get the following errors in the simulator:

changing like 9 to:

var React = require('react');

And line 21 to:

} = require('react-native');

allowed me to run the example code.

@itinance - can you please check if the example code needs any adjustments ?

will do in upcoming days

Nevermind actually - I thought this issue was referring to the samples but it seems it refers to original test code. I will take a look.

This is fixed in 3.2.1 release.


Code reformat, findent

Application of findent's default parameters on present source code, and manual edit to
ease future diffviews and eventual documentation with ford.

Dear @nbehrnd,
Following significant changes in ForImage after your pull request, merging it into the main branch isn't feasible due to conflicts. Since this pull request solely addresses formatting using findent and doesn't involve any code alterations, I intend to close it to maintain project cleanliness.
I appreciate your efforts, and I will be happy for future contributions to the project. Feel free to submit a new pull request!


[request] modrinth release

I know this is like, bearly just got out of the over but can we haz modrinth?

The mod is already on modrinth, just under review

I submitted both mods at the same time, however CurseForge was just quicker reviewing it.

Ah awesome. Thanks 😅


amex cvn (cvv) = 4 digits

Hi excited to try your control.
As I was evaluating noticed the CVV validation was limited to 3 chars and amex cards carry 4 chars.  Sure I can adjust the 3 to 4 in regex but in order to keep user input errors to minimum is this something I can do with a compound object?  I'm not sure, I'll dig in deeper but in meantime if you get this before i come up with answer what is your recommendation?  Is there a flag or something I can evaluate to adjust the field object for the cc number?  ie, isAmex (then I can set set CVV validator to 4)

Hi,
Please take a look at example. CVV validation is made there by APRegexValidator with ^[0-9]{3}$ regex. Regex to support 4 digits will be - ^[0-9]{4}$ . But maybe you would need to adjust this regex every time user changes CC number and you get CC type from APCreditCardValidator.


ensure_process does not have any effect when set as 'running'

There is no effect when setting ensure_process to running on the supervisord::process even if it's not currently running.
So actually, starting a stopped process is not possible.
As seen in the code, there is no behavior described when ensure_process is set to running.

We've hit this as well. I created #77 with our fix for it.

This is fixed by #72 which has just been merged


Broken after the ^0.1.1 and ^0.1.1+1 update

Because no versions of bitsdojo_window_linux match 0.1.1 and no versions of bitsdojo_window_linux match >0.1.1 <0.2.0, bitsdojo_window_linux ^0.1.1 is forbidden.
So, because my_app depends on bitsdojo_window ^0.1.1+1 which depends on bitsdojo_window_linux ^0.1.1, version solving failed.
pub get failed (1; So, because my_app depends on bitsdojo_window ^0.1.1+1 which depends on bitsdojo_window_linux ^0.1.1, version solving failed.)

How does your pubspec.yaml file look like?

Its just the default starter counter app
name: delete_app_2
description: A new Flutter project.

publish_to: 'none'

version: 1.0.0+1

environment:
  sdk: ">=2.12.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter

  cupertino_icons: ^1.0.3
  bitsdojo_window: ^0.1.1+1

dev_dependencies:
  flutter_test:
    sdk: flutter

  flutter_lints: ^1.0.3

flutter:
  uses-material-design: true

flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel dev, 2.3.0-16.0.pre, on Microsoft Windows [Version 10.0.19042.985], locale en-IN)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.6.5)
[√] Android Studio (version 4.2.0)
[√] VS Code (version 1.56.2)
[√] Connected device (3 available)

• No issues found!

I had to revert to ^0.1.0+1 to get it working

Tried with a sample counter app and the same Flutter version but could not reproduce it on my system.
If you have a sample repo that I can clone to reproduce this and further investigate it, feel free to re-open this.


add 1.12.3 to old versions on support/1.14

Attn 1.14 Release Manager: this PR is ready to be merged.  I am ok if you would like to merge it on my behalf (although I'm also quite happy to merge it myself once you have approved it).


Suppress Kraken API warnings

Hi,
Is there an option to suppress the Kraken API warnings? They're messing up my display :/

No, and sincerely I think this should not be done.
Continously get some exchange API warnings / errors is a sinthom that using bot is extremely risky and could end with capital loose

Having a simple option like "--ignore-warnings-at-my-own-peril" wouldn't be so bad...

On your fork or local clone you can change
zenbot/extensions/exchanges/kraken/exchange.js
and comment line 85
console.warn(('\nKraken API warning - unable to call ' + method + ' (' + errorMsg + '), retrying in ' + timeout / 1000 + 's').yellow)

I noticed a bit late that the warning are sent to stderr, so simply redirecting fd 2 to /dev/null solves my issue :)


[Suggestion] Compatibility with modules

Compatible with shield expansion, does not consume endurance when successfully blocked！

Compatibility added! When V0.5.0 is released the mod will have full datapack support. Though, you will have to add in the values for the shields yourself via a datapack, but I included a guide (with a downloadable example) in my new wiki here


[CP][beta channel] Fix crash when developing with DDC

commit(s) to merge: 53895b22707d986776db19d4c953d9d3b0178401
merge instructions: clean merge
What is the issue: DDC had an old undiscovered bug. A certain rare code pattern triggers the issue and in that case the app cannot be loaded for development (the code crashes while loading). The pattern involves an inheritance chain that forms a cycle between classes and type parameters.
Recently a Flutter Web app encountered this pattern and started hitting this issue.
What is the fix: A change in DDC to break the cycle in the generated code.
Why cherrypick: We only know of one affected customer, but the result makes the product unusable in that case.
Risk: low, change is relatively small, we'll get a better signal of risk after the next roll to internal customers tomorrow.
Link to original  issue(s): https://github.com/flutter/flutter/issues/64011
/cc @kevmoo @mit-mit @whesse @athomas @vsmenon @franklinyow @yjbanov @Markzipan @pcsosinski

Merged to beta in 4215dca724fb80de592f51a6cdba51e7638d1723 (2.10.0-110.5.beta).


More README detail please

I am new to Python and to this, could you please provide a more detailed example and readme?
How to init a device, how to get logs, how to get motion state, v.v...
I use the current example and it seems like scanning the BLE device and find for LD2410, but I don't know what to do next to get the logs from LD2410 device

I replace the ADDRESS variable with Mac address of the Ld2410, and the logs for the state show up for about 30s, then the terminal abort, what should I do next?

There is a timeout that calls the "cancel_callback()" after x number of seconds. You can modify "await asyncio.sleep(60)" and replace the number with any arbitrary value. Or alternatively, you could change the await to listen for a keyboard input to exit instead.


pyconfig.h: No such file or directory

Thank you very much and when I use pip to install dagobah in ubuntu 14.04(64),an error has happend,this is the log
building 'lxml.etree' extension
creating build/temp.linux-x86_64-2.7
creating build/temp.linux-x86_64-2.7/src
creating build/temp.linux-x86_64-2.7/src/lxml
x86_64-linux-gnu-gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -fPIC -I/usr/include/libxml2 -I/home/chengchaowang/workspaces/project001/env001/build/lxml/src/lxml/includes -I/usr/include/python2.7 -c src/lxml/lxml.etree.c -o build/temp.linux-x86_64-2.7/src/lxml/lxml.etree.o -w
src/lxml/lxml.etree.c:8:22: fatal error: pyconfig.h: No such file or directory
#include "pyconfig.h"
                  ^

compilation terminated.
error: command 'x86_64-linux-gnu-gcc' failed with exit status 1

How are you building dagobah? This seems like a compilation error for one of dagobah's dependencies, not an issue with dagobah its self. I'm happy to help you figure out what it is though.
Some cursory googling tells me you might need to install the python-dev package from ubuntu: http://stackoverflow.com/questions/24829558/mpi4py-on-ubuntu-pyconfig-h-not-found

thank you very much,finally i solved this problem and installed dagobah according to what you said,
but when i runing dagobahd,an error has happend,here is the log
Traceback (most recent call last):
File "/usr/local/bin/dagobahd", line 9, in
load_entry_point('dagobah==0.3.1', 'console_scripts', 'dagobahd')()
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 351, in load_entry_point
return get_distribution(dist).load_entry_point(group, name)
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2363, in load_entry_point
return ep.load()
File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2088, in load
entry = import(self.module_name, globals(),globals(), ['name'])
File "/usr/local/lib/python2.7/dist-packages/dagobah/daemon/app.py", line 1, in
from .daemon import app, login_manager
File "/usr/local/lib/python2.7/dist-packages/dagobah/daemon/daemon.py", line 249, in
dagobah = init_dagobah()
File "/usr/local/lib/python2.7/dist-packages/dagobah/daemon/daemon.py", line 113, in init_dagobah
backend = get_backend(config)
File "/usr/local/lib/python2.7/dist-packages/dagobah/daemon/daemon.py", line 218, in get_backend
return SQLiteBackend(**backend_kwargs)
File "/usr/local/lib/python2.7/dist-packages/dagobah/backend/sqlite.py", line 38, in init
super(SQLiteBackend, self).init()
File "/usr/local/lib/python2.7/dist-packages/dagobah/backend/base.py", line 26, in init
self.verify_required_packages()
File "/usr/local/lib/python2.7/dist-packages/dagobah/backend/base.py", line 51, in verify_required_packages
if Version(installed_version, partial=True) < Version(spec['version'], partial=True):
File "/usr/local/lib/python2.7/dist-packages/semantic_version/base.py", line 75, in init
major, minor, patch, prerelease, build = self.parse(version_string, partial)
File "/usr/local/lib/python2.7/dist-packages/semantic_version/base.py", line 181, in parse
raise ValueError('Invalid version string: %r' % version_string)
ValueError: Invalid version string: '0.7.5.post2'

@callnothing Whoever's running Alembic just decided to go off the rails and start putting random strings in the versions, which is breaking our semantic version parsing. Please try installing Alembic 0.7.4 as a workaround. I'll create a separate issue to account for this.
pip install alembic==0.7.4

@callnothing please let us know if the workaround did not resolve your issue. For now I am going to close this issue as we've created a separate issue for the bug fix.


Broken UI for Echo framework support

Describe the bug
There seems to be something wrong with UI
To Reproduce
Steps to reproduce the behavior:
Do the exact example of https://github.com/swaggo/echo-swagger/ readme file
Expected behavior
Swagger UI should not be broken
Screenshots

Your swag version
1.3.0
Desktop (please complete the following information):

OS: MacOS
Browser Chrome
Version Latest

Additional context
Add any other context about the problem here.

issue seems to have gone away with latest version


Fixed NullpointerException ...

... when plugin is executed in Maven module without image configuration. This can be the case when using the plugin in a multi module Maven project. I was not able to call docker:build on parent Maven module as there was no image configuration but in sub modules there were.

Thanks ! will do a release soon, probably early next week.


crypto/elliptic: different ecdsa.Verify result between p256 amd64 and generic implementations with a zero hash

What version of Go are you using (go version)?
tip
What operating system and processor architecture are you using (go env)?
GOARCH=386 and GOARCH=amd64
What did you do?
Code pulled from golang-nuts post https://groups.google.com/forum/#!topic/golang-nuts/M7tXCf8a4pk
$ GOARCH=386 go build -o cr32 && ./cr32
2017/05/02 21:10:00 Verification succeded
$ go build -o cr64 && ./cr64
2017/05/02 21:10:00 Failed

What did you expect to see?
I assume the verification should have failed in both cases.
What did you see instead?
Verification passed for the 32 bit binary.
Cause
The difference is in the path taken in ecdsa.Verify:
	var x, y *big.Int
	if opt, ok := c.(combinedMult); ok {
		x, y = opt.CombinedMult(pub.X, pub.Y, u1.Bytes(), u2.Bytes())
	} else {
		x1, y1 := c.ScalarBaseMult(u1.Bytes())
		x2, y2 := c.ScalarMult(pub.X, pub.Y, u2.Bytes())
		x, y = c.Add(x1, y1, x2, y2)
	}

The S390 and AMD64 versions of CombinedMult appear to always set the Z value to 1 regardless of X and Y.
(*curveParams).Add calls zForAffine which returns a Z value of 0 if X or Y are zero.
Modifying zForAffine to match the behavior of the optimized CombinedMults (Z=1 in all cases) causes TestInfinity to fail, but otherwise the generic and optimized CombinedMults give the same output for this  case.

Saving others a click, code is:
https://gist.github.com/ivoras/32b2abd16b5984fa43c006486bfb7e2c

Verified the 386/amd64 difference is the same in Go 1.7, Go 1.8, and tip.

(I believe the verification should have passed in both cases, rather than failed. I.e. the referenced test case appears to be a valid signature of a zero hash. That means that we're rejecting a valid signature rather than accepting an invalid signature, which is less of a problem at least.)

(For my own reference in the future: there's an incomplete fix in https://go-review.googlesource.com/c/42611, but it'll need assembly changes and I'm not going to attempt that today; I'll only mess it up.)

@agl, do you want this in Go 1.8.2? If so, can we wrap up the review? If not, we can roll Go 1.8.2 already with just #20040 fix.

(I believe the verification should have passed in both cases, rather than failed. I.e. the referenced test case appears to be a valid signature of a zero hash. That means that we're rejecting a valid signature rather than accepting an invalid signature, which is less of a problem at least.)

The question here should also be "what should we do if we have a zero hash in ECDSA, knowing that this may partly breaks ECDSA"...
If you have a null hash then the verification process goes as usual with correctly generated signature, excepted that you can trick it into validating your message simply by providing the signature (r,s)=(x,x) where (x,y) are the coordinates of the ECDSA public key !
Actually all dependency upon the secret material is removed when the hash value is congruent to 0 mod n.
A crafted signature would then validate since upon verification you compute:

e = H(m) = 0.
Let z be the Ln (the bit length of the group order n) leftmost bits of e, so it's zero.
w = s-1 = x-1 (mod n) by my choice of s.
u1 = zw ≡ 0 (mod n) and u2 = rw = xw ≡ 1 (mod n).
the curve point computed in the verification becomes:
(x1, y1) = u1⋅ G + u2 ⋅ Q = 0 ⋅ G + 1 ⋅ Q = Q = (x, y)
The signature is valid if x1 ≡ r (mod n), invalid otherwise, but this is the case by choice of the signature's value r = x.

Now, we should ask ourselves what we want to do when confronted to such a zero hash, or such a strange signature... So the zero hash does not permit us to ensure the authenticity of the signature since such forgeries are possible (note that there might be other less trivial forgeries).
So far, I've done a few tests on the topic to see how other are handling it and have found that:

OpenSSL will gladly validate such signatures, both valid and crafted.
mbedTLS answers with an error "MBEDTLS_ERR_ECP_INVALID_KEY" on zero values upon verification, but not upon signature as it generates valid signatures (which are validated on the Playground or by OpenSSL).
CryptoPP will reject the signatures it generated using a zero hash, but while it accepts the crafted signatures or signatures made using Go, its signatures are rejected by other implementations because they are not valid.
Go crypto currently rejects such signature on my computer (amd64) but accept them on the playground as proved by this playground example I just made: https://play.golang.org/p/WTic53IPm6 whose code returns twice false on my computer, but true on the playground. (which is exactly why this issue exists in the first place)

I must say that I'm not really sure what would be the best way around this issue. I believe that we should reject the signatures as soon as the values u1 or u2 are zero, or throw an error maybe. (This is fine since secret material is not involved in the verification process and signing the zero hash value does not leak secret material as long a the k value is secret and well randomly generated.)
I don't know what is opinion on that topic @agl ?