@azure/storage-blob BlobServiceClient cannot list blobs with SAS token

Package Name: @azure/storage-blob
Package Version: ^12.23.0
Operating system:
[X] nodejs

version: latest

[ ] browser

name/version:

[X] typescript

version: latest

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://learn.microsoft.com

Describe the bug
Listing blobs with SAS token keeps returning "Server failed to authenticate the request. Make sure the value of Authorization header is formed correctly including the signature.". I am guessing it is because the SDK doesn't add "restype=container&comp=list" into the URL
To Reproduce
const client = new BlobServiceClient(`https://${accountName}.blob.core.windows.net?${SASToken}`);
const containerClient = blobServiceClient.getContainerClient(containerName);
const blobs = containerClient.listBlobsFlat();
console.log(blobs);

Expected behavior
Error: Server failed to authenticate the request. Make sure the value of Authorization header is formed correctly including the signature.

Sorry I think I spelled my container name wrong. That was embarassing haha


Fix antmicro-yosys-plugins

Attempt to fix #91

Had to restart the CI after merging - it failed because it didn't find available macos runners, I think


Fire an event after content DOM created, before shown event for tooltip/popover

Hi,
This PR will allow user to know when the tooltip template or popover is added to the DOM.
Ref #16124
/CC @cvrebert

Needs documentation.

A similar unit test needs to be added for Popovers too.

I added a unit test for the popover and documentation

@twbs-savage Retry

Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED
Commit: 1c8612bc68dd3be78ee985182b304dad02bbcf85
Build details: https://travis-ci.org/twbs-savage/bootstrap/builds/55848318
(Please note that this is a fully automated comment.)

I don't know if it's good or not to do that but I haven't got any feedbacks for a while so...
PING :smile:

The inserted event handlers in the unit tests should also assert that the tooltip/popover DOM exists.

Ok thanks for your feedbacks I'll work on it

Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED
Commit: 174a54450d46d0c12deec1c3a6ae59088537588b
Build details: https://travis-ci.org/twbs-savage/bootstrap/builds/57624309
(Please note that this is a fully automated comment.)

Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED
Commit: 3d71eae3011b10daa558baf494b019925fee5f60
Build details: https://travis-ci.org/twbs-savage/bootstrap/builds/57627930
(Please note that this is a fully automated comment.)

Thanks for the revisions; LGTM.
@hnrch02 @XhmikosR LGTY?

@Johann-S Thanks!

You're welcome thanks for your feedbacks


test(delete global draft): adding new test case for deleting global drafts

Summary

Adding a new test case for global drafts, testing delete from global drafts and cancel delete
Is the feature in stable branch or under review?

[] Feature in stable branch (master, main)?
[] Feature still under review? If yes, list all related pull requests for reference.

Test server

Test client

[] Browser
[] Mobile App (version, OS)
[] Desktop App (version, OS)

Merging this now.

New test case saved as MM-T5366.


UHF-9130: Make fresh fixes

How to install

composer require drupal/helfi_drupal_tools:dev-UHF-9130

How to test

Remove dump.sql file from your Git root
Run make fresh
Run mysqlcheck --host=db --user=drupal --password=drupal --all-databases and make sure no errors are reported

Codecov Report

Merging #16 (37ec02b) into main (4b78f15) will not change coverage.
Report is 3 commits behind head on main.
The diff coverage is n/a.

:exclamation: Current head 37ec02b differs from pull request most recent head e6c25eb. Consider uploading reports for the commit e6c25eb to get more accurate results

@@             Coverage Diff             @@
##                main       #16   +/-   ##
===========================================
  Coverage     100.00%   100.00%
  Complexity        37        37
===========================================
  Files              5         5
  Lines             97        97
===========================================
  Hits              97        97

Files
Coverage Δ

src/Update/UpdateOptions.php
100.00% <ø> (ø)

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


Pom madness

dependent on: https://github.com/confluentinc/common/pull/43

Test FAILed.

Test PASSed.

retest this please

Test PASSed.

retest this please

Test PASSed.


Clarify that gradients are interpolated in linear color space

person on lobste.rs wrote:
Gradients are implicitly in sRGB’s pseudo-gamma space, therefore incorrect. See reference rendering.

It should be specified in which color space and color format the interpolation should happen:
http://scanline.ca/gradients/

The math for gradient blending is fairly easy and i've implemented a verification shader on shadertoy.com.
Colors need to be interpolated in linear color space and require gamma application after that:

NOTE: This image must be viewed without any scaling! Otherwise the bayer dithering won't work properly

Fixed.


[flash_ctrl] Connect flash / otp key interface

hey guys, so this one is failing because the req/ack interface to kdi from flash is async.
In this PR, i added some dumb logic to do a 4-phase handshake (the latest one i've not pushed).
But basically, when the full handshake is in, the push/pull interface complains because the ack is still asserted even though the request on the interface has gone away.  This is because the request is still being synchronized internally.  The problem with the version here right now is that the request lingers after it's been granted (since it's still going through sync), and that makes the arbiter assertions unhappy.
I think we probably need to enhance otp_kdi side to handle 4-phase handshakes and also update the push-pull interface to do the same.
We can't realistically route the flash clock into otp, or the otp clock into flash, since long term that's not really a sustainable solution.

discussed with @msfschaffner on #4441 and will go with his recommendation.
So nothing on otp or dv needs to be changed! yay!

Sounds good! Thanks @tjaychen for the update. Yeah if we are baking the prim_sync_2_flop inside OTP, we will need to support async mode in push-pull-agent. If we are using the 4-state proposed by Michael in #4441, then push-pull-agent can get the synced req/ack and should not require any further changes.

Sounds good! Thanks @tjaychen for the update. Yeah if we are baking the prim_sync_2_flop inside OTP, we will need to support async mode in push-pull-agent. If we are using the 4-state proposed by Michael in #4441, then push-pull-agent can get the synced req/ack and should not require any further changes.

yeah if you see @msfschaffner and my exchange on the issue, there are some things neither one of us likes about the req/ack approach, but it's not significant enough to stop progress for now.  We can revisit later when we have more time.
@cindychip / @weicaiyang / @udinator this does remind me though...
if we put the prim_reqack on one side, let's say flash in this case, it does mean OTP is cleaner.. however flash will still be different from block level testing perspective.  Ie the interface is not fully sync with it's own operating clock. That's still something to keep in mind for DV...

Yeah, thanks for the reminder. The IP that contains the prim_reqack would
be good to add a rand clock to check async behavior.
Cindy Chen
On Mon, Dec 7, 2020 at 11:41 AM tjaychen<EMAIL_ADDRESS>wrote:

Sounds good! Thanks @tjaychen https://github.com/tjaychen for the
update. Yeah if we are baking the prim_sync_2_flop inside OTP, we will
need to support async mode in push-pull-agent. If we are using the
4-state proposed by Michael in #4441
https://github.com/lowRISC/opentitan/issues/4441, then push-pull-agent
can get the synced req/ack and should not require any further changes.
yeah if you see @msfschaffner https://github.com/msfschaffner and my
exchange on the issue, there are some things neither one of us likes about
the req/ack approach, but it's not significant enough to stop progress for
now. We can revisit later when we have more time.
@cindychip https://github.com/cindychip / @weicaiyang
https://github.com/weicaiyang / @udinator https://github.com/udinator
this does remind me though...
if we put the prim_reqack on one side, let's say flash in this case, it
does mean OTP is cleaner.. however flash will still be different from block
level testing perspective. Ie the interface is not fully sync with it's own
operating clock. That's still something to keep in mind for DV...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/4442#issuecomment-740137157,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACXPOOLAWSDGUBQSPLUDAELSTUVXHANCNFSM4UOIVS4A
.

@cindychip / @weicaiyang / @udinator this does remind me though...
if we put the prim_reqack on one side, let's say flash in this case, it does mean OTP is cleaner.. however flash will still be different from block level testing perspective. Ie the interface is not fully sync with it's own operating clock. That's still something to keep in mind for DV...

Yeah, thanks for the reminder. The IP that contains the prim_reqack would be good to add a rand clock to check async behavior. Cindy Chen

Agree with Cindy. need to make sure dst/src clocks of prim_reqack are async.

I agree with @weicaiyang and @cindychip also.

hey everyone,
i've updated per our discussion, please have a look.


Add TypeORM MariaDB Integration with Added DB Migration

Added TypeORM MariaDB Integration with Added DB Migration

Summary by CodeRabbit

New Features

Introduced ESLint configurations for improved code quality checks.
Added TypeORM scripts for easier database migration management.
Integrated a new DatabaseModule for enhanced database operations in the API.
Implemented TypeORM configurations using environment variables for MariaDB.

Documentation

Updated dependencies and scripts in package.json for better project maintenance.

Chores

Added .env to .gitignore to prevent environment variables from being tracked.

Closing PR.
Create PR using Template.


Add kid to JsonWebKey definition

This is to address https://github.com/Microsoft/TypeScript/issues/26854

@funkybob you need to commit the generated code in addition to the widl change.

Thanks for jumping in.
This is not in the widl from upstream.
So the correct place would be the addTypes.json

@sandersn I honestly don't know how to make the "generated code"... I ran into this problem when trying to migrate a project to TypeScript, figured I'd trying to solve it, and got this far before I ran out of time to contribute, and worked around it locally.

It's npm run build and then you diff baseline and generated.

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

Looks like this is not in the WebCrypto spec. Filing an issue there may help. (Reposting from https://github.com/Microsoft/TypeScript/issues/26854#issuecomment-430450762)

Sounds like we should close this here then.


[Issue] No write  access for scope class tenant

Describe the bug
I go through docs to do monitoring extension SAP HANA instance. While terraform apply got error "No write access for scope class tenant"
To Reproduce
Steps to reproduce the behavior:

Go to Manage -> Access Token -> applied all of scopes access token
Copy token to terraform code
terraform init (success)
terraform plan (success)
terraform apply -auto-approve (Got err "No write access for scope class tenant")

Expected behavior
As full access of token should be successfully.
Screenshots

Desktop (please complete the following information):

OS: MacOS
Chorme

resource "dynatrace_hub_extension_config" "com_dynatrace_extension_sql_sap_hana" {
  name = "com.dynatrace.extension.sql-sap-hana"
    value = jsonencode(
    {
      "enabled" : true,
      "description": "hana metrics 2.0 extension",
      "version" : "2.2.0",
      "featureSets" : [
        "Backup Metrics",
        "Savepoint Metrics",
        "Expensive Statements",
        "Unloads Due to Low Memory",
        "Expensive Statement Metrics",
        "Current Alerts",
        "Replication Metrics",
        "Host Infrastructure Metrics"
      ],
      "sqlHanaRemote": {
        "endpoints": [
          {
            "host": "ed51fe82-59da-4a21-8cb5-e100b2f1eea5.hana.prod-us21.hanacloud.ondemand.com",
            "port": 443,
            "authentication": {
              "scheme": "basic",
              "useCredentialVault": null,
              "username": "DBADMIN",
              "password": "xxxxx"
            },
            "ssl": null
          }
        ]
      },
      "scope": "ag_group-default"
    }
  )
}

$ terraform init
$ terraform plan (successfully)
$ terraform apply -auto-approve (Got failure)

Hi @JayChanggithub ,
The ActiveGate group should be set as active_gate_group instead of value.scope. Example below..
I hope that helps!
resource "dynatrace_hub_extension_config" "com_dynatrace_extension_sql_sap_hana" {
  name = "com.dynatrace.extension.sql-sap-hana"
  active_gate_group = "default"
  value = jsonencode(
    {
      "enabled" : true,
      "description": "hana metrics 2.0 extension",
      "version" : "2.2.0",
      "featureSets" : [
        "Backup Metrics",
        "Savepoint Metrics",
        "Expensive Statements",
        "Unloads Due to Low Memory",
        "Expensive Statement Metrics",
        "Current Alerts",
        "Replication Metrics",
        "Host Infrastructure Metrics"
      ],
      "sqlHanaRemote": {
        "endpoints": [
          {
            "host": "ed51fe82-59da-4a21-8cb5-e100b2f1eea5.hana.prod-us21.hanacloud.ondemand.com",
            "port": 443,
            "authentication": {
              "scheme": "basic",
              "useCredentialVault": null,
              "username": "DBADMIN",
              "password": "xxxxx"
            },
            "ssl": null
          }
        ]
      }
    }
  )
}

@kishikawa12
Thanks for your example. Let me try.

@kishikawa12
It's work as your example. Please kindly closed ticket. Thx.

@JayChanggithub Excellent, thanks!


[stable/jenkins] Wait for up to 2 minutes before failing liveness check

Otherwise a plugin installation causes 2 restarts
@lachie83 @viglesiasce

/ok-to-test
Thanks @carlossg!

/lgtm


chore(main): release 0.0.50

:robot: I have created a release beep boop
0.0.50 (2024-07-26)
Other

bump all plugin versions with releases (#355) (d2838e4)
bump network plugin versions (#346) (6ece95b)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.0.50
:sunflower:


Slack/Gitter

Hi, I just thought whether it would be great to have Slack/Gitter so we could simplify communication of developing, improving this library.
Well, I suppose this will involve some new people to help improve library.
Maybe, just give a try. What do you think? Thumbs up/down, please :)

Gitter is also used a lot for open source libraries' communication

https://gitter.im/EZSwiftExtensions/Lobby
here it is


Enable multiple publishers on a single topic, guarded on same topic type str

Closes #110

Looks good, happy to merge. Error types and messages need several overhauls, but good enough for now.


Environment build: MacOS won't take localhost

Q
A

Bug?
No

New Feature?
No

Sulu Version
master

Browser Version
CLI

Actual Behavior
app/console sulu:build prod won't recognise the mysql connection if the dbhost is set to localhost, but does if it's set to <IP_ADDRESS>
Expected Behavior
config.yml can use either localhost or 127 disregarding environment
Steps to Reproduce
MacOS El Capitaine
git clone [master]
composer install
input localhost as host at CLI prompt
attempt app/console sulu:build prod

@SecondeJK is that really a sulu problem? without testing it i would think that this also wont work for other symfony (or doctrine) projects. Do you have tested this? /cc @danrot what do you think?

@wachterjohannes if it's doctrine or symfony then I guess close...not sure when I'd able to test it. I do have another project using symfony core so if this comes up I'll look

thanks maybe i also have time to install symfony demo and test it.

Since we do not modify anything when registering the database settings I would say this issue doesn't belong to us. I'd say we close it for now, and you can reopen it, in case you find out that the issue is really in Sulu.


Aggregation pushdown in SQL Server connector

Follow-up to https://github.com/prestosql/presto/pull/3881
Relates to: https://github.com/prestosql/presto/issues/4138

I would like to work on this story.  Please assign to me.

Please go for it!

I did sign the CLA - but getting "2020-09-16T21:36:40.541Z INFO The contributors @dleach001 have not signed the CLA, adding error status to the pull request".  Think I listed mr-big-spency on the CLA.  Not sure why my company userid is showing up as the contributor name as my fork is at: https://github.com/mr-big-spency/presto/tree/agg-pushdown-sqlserver

Let's move the CLA-related discussion to #dev on our slack. Someone will assist you with this.

@cla-bot check


Update mrtk-packages.md

Link proposal for Issue 415

@nivnar : Thanks for your contribution! The author(s) have been notified to review your proposed change.


np.asarray sometimes doesn't work on List

Python 3.7.6 (default, Jan  8 2020, 13:42:34)
[Clang 4.0.1 (tags/RELEASE_401/final)] :: Anaconda, Inc. on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import numpy as np
>>> from numba import njit
>>> from numba.typed import List
>>> @njit
... def lt(a, b):
...   return np.asarray(a) <= np.asarray(b)
...
>>> l1 = List()
>>> l1.append(4)
>>> l1.append(2)
>>> l1.append(1)
>>>
>>> l2 = List()
>>> l2.append(2)
>>> l2.append(2)
>>> l2.append(2)
>>>
>>> print(lt(l1, l2))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/hameerabbasi/miniconda3/envs/sparse/lib/python3.7/site-packages/numba/dispatcher.py", line 401, in _compile_for_args
    error_rewrite(e, 'typing')
  File "/Users/hameerabbasi/miniconda3/envs/sparse/lib/python3.7/site-packages/numba/dispatcher.py", line 344, in error_rewrite
    reraise(type(e), e, None)
  File "/Users/hameerabbasi/miniconda3/envs/sparse/lib/python3.7/site-packages/numba/six.py", line 668, in reraise
    raise value.with_traceback(tb)
numba.errors.TypingError: Failed in nopython mode pipeline (step: nopython frontend)
Invalid use of Function(<function asarray at 0x7f8df02a0440>) with argument(s) of type(s): (ListType[int64])
 * parameterized
In definition 0:
    All templates rejected with literals.
In definition 1:
    All templates rejected without literals.
This error is usually caused by passing an argument of a type that is unsupported by the named function.
[1] During: resolving callee type: Function(<function asarray at 0x7f8df02a0440>)
[2] During: typing of call at <stdin> (3)

File "<stdin>", line 3:
<source missing, REPL/exec in use?>

>>> import numba
>>> numba.__version__
'0.48.0'

Smaller reproducer:
Python 3.7.6 (default, Jan  8 2020, 13:42:34)
[Clang 4.0.1 (tags/RELEASE_401/final)] :: Anaconda, Inc. on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import numpy as np
>>> import numba as nb
>>>
>>> @nb.njit
... def to_array(x):
...     return np.asarray(x)
...
>>> l = nb.typed.List()
>>> l.append(5)
>>> to_array(l)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/hameerabbasi/miniconda3/envs/sparse/lib/python3.7/site-packages/numba/dispatcher.py", line 401, in _compile_for_args
    error_rewrite(e, 'typing')
  File "/Users/hameerabbasi/miniconda3/envs/sparse/lib/python3.7/site-packages/numba/dispatcher.py", line 344, in error_rewrite
    reraise(type(e), e, None)
  File "/Users/hameerabbasi/miniconda3/envs/sparse/lib/python3.7/site-packages/numba/six.py", line 668, in reraise
    raise value.with_traceback(tb)
numba.errors.TypingError: Failed in nopython mode pipeline (step: nopython frontend)
Invalid use of Function(<function asarray at 0x7fb330038440>) with argument(s) of type(s): (ListType[int64])
 * parameterized
In definition 0:
    All templates rejected with literals.
In definition 1:
    All templates rejected without literals.
This error is usually caused by passing an argument of a type that is unsupported by the named function.
[1] During: resolving callee type: Function(<function asarray at 0x7fb330038440>)
[2] During: typing of call at <stdin> (3)

File "<stdin>", line 3:
<source missing, REPL/exec in use?>

Should be possible to implement.

Thanks for filing the issue! @hameerabbasi
I will try to implement this.

Great, thanks @rohitsanj

@rohitsanj The use of Numba's extension API @overload decorator is strongly recommended for this task. A guide to using @overload is here and API documentation is here.
As a new contributor, some instructions for getting set up for development are here and are a great place to start.
It may be useful to take a look at examples of pull requests adding support for a NumPy function that have already been done if guidance is needed... here's a couple:

np.roll: https://github.com/numba/numba/pull/3563/files
np.roots: https://github.com/numba/numba/pull/2135/files

Thanks for the links! 😄 @stuartarchibald


I can't run the console of a container if Portainer is using https (with reverse proxy)

Hi all,
I'm having a problem with Portainer and https. I'm running P. on Debian 10, I have a container with Nginx configured as a reverse proxy with Letsencrypt installed.
The problem is that I can't run the console of a container from P. It says 'Connecting' but then nothing happens.
If I reach Portainer from the HTTP link it works. I tried it on two vps and with many containers. I'm using the last version (1.24).
Can you guide me in the right direction?
Thank you,
Alessandro

Hi,
I had encountered the same issue and solved.
Have you set WebSocket protocol in the proxy_pass directive?
In my case, I used Apache httpd as a reverse proxy and installed certifications on it.
The connection between Apache and Portainer was not encrypted (used http).
When I got problem, my ProxyPass settings was like below.
ProxyPass /portainer http://localhost:9000/
ProxyPassReverse /portainer http://localhost:9000/

I could access Portainer with HTTPS, but I couldn't use exec console.
After I modified ProxyPass like this, I could connect to the execution console.
ProxyPass /portainer http://localhost:9000/
ProxyPassReverse /portainer http://localhost:9000/
ProxyPass /portainer ws://localhost:9000/
ProxyPassReverse /portainer ws://localhost:9000/

I think the ProxyPass of Apache is almost the same as proxy_pass of Nginx. So please try it!
If you use Google Chrome, please watch Devtool Console tab when you see 'Connecting' message.
It can address some wss request failed. (wss is protocol like https, and ws is like http)

@theRAGEhero I believe this is a proxy issue like is described above. Let me know if this helps you or if you are still having issues

Hi,
thank you both for your advice.
I ended up adding those lines to the location setting on /etc/nginx/nginx.conf
location / {
      proxy_pass      http://portainer:9000;
     proxy_http_version 1.1;
     proxy_set_header   Host               $host:$server_port;
     proxy_set_header   X-Real-IP          $remote_addr;
     proxy_set_header   X-Forwarded-For    $proxy_add_x_forwarded_for;
     proxy_set_header   X-Forwarded-Proto  $scheme;
     proxy_set_header   Upgrade            $http_upgrade;
     proxy_set_header   Connection         "upgrade";
}

Now it works fine!
Thanks again,
Alessandro


figure 14: change in node degree (y-axis) by rank order w.r.t. node degree (x-axis)

color by generation
do stacked column chart so we can see cumulative change in node degree

Method to get : group by molecule (a.k.a. smiles_str), sort by snapshot generation num asc, rolling difference of count_relationships (a.k.a. node degree). Might need to scale the difference in degree; for this single example the degree tripled each generation

First pass at building this one: y axis is node deg diff by molecule from previous generation, x axis is node degree rank (rank of count_relationships)

actually, diff should be represented as line:

first generation has null delta; need to fill with 0

first generation has null delta; need to fill with 0


Add CAP

Pull Request for adding CAP (CAP) Logo ad Contract.
CAP Token Contract
https://etherscan.io/token/0x43044f861ec040db59a7e324c40507addb673142
Landing Page
https://app.cap.uly.dev
App Page
https://cap.eth.link
GitHub
https://github.com/capfina
Blog
https://blog.cap.finance/
Twitter
https://twitter.com/CapDotFinance
Telegram (Official group)
https://t.me/capfin
Uniswap CAP-ETH Pair
https://info.uniswap.org/pair/0xc169f0eb31403c0bcc43dc9feca648a79fafc0f4
CoinGecko
https://www.coingecko.com/en/coins/cap
Thank you!

hi @ape-mario

Can you use an address in EIP-55 version?
Can you check if the quality of your logo is correct?

@ape-mario do you would like to update this PR?


Variables don't work in Body Multipart Form

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
Variables don't work in Body Multipart Form
.bru file to reproduce the bug
No response
Screenshots/Live demo link

Unfortunately this is a duplicate of this old very annoying bug https://github.com/usebruno/bruno/issues/352 😩

This has been fixed and is available in v1.32.0 🎉


Bump docs config to07

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

You might be getting more commits than you want here


Mutations

It would be nice to also be able to run mutations

@chimon2000 what exactly do you have in mind? What do you think package could gain from having mutations?

Similar to other data fetching frameworks like React Query, maybe you want to track the state of a mutation. On load, you might provide some animation or disable the button.  On success/error you might refetch queries, show a dialog/toast, or perform some other action.


"Submitting" form state is replaced with "Valid" straight away

I've just noticed that #218 introduced a line that sends a SetFormState(Valid) action immediately after sending the Submit action. Not sure what the purpose of the change is but it results in no Submitting state. I noticed it because I disable the submit button on submit and it's no longer being disabled. If I log formState on render it stays Valid up until the form is successfully submitted. Is this intentional?
https://github.com/rescriptbr/reform/blob/6c8d6de710896a5f35daf91ab5ea6d7b3801f9f4/reform/src/ReForm.res#L253

Hey @alexeygolev thanks for reporting. It was not intentional and i'ts probably a bug. I'll test the behavior and ping you with a better explanation for this issue.
cc @fakenickels

And sorry for the delay, it has been a busy month.

Hey thanks for bringing this up. It definitively sounds like a bug

It should put the form in Valid state then Submitting. I took a quick look in the code and not sure what can be causing it to not be working normally


fxing long path issue

Bug 8322:Unable to finish the Auto learn item for JAVA based CRM

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 1

Complexity increasing per file
==============================
- Ginger/GingerCoreCommon/Repository/SolutionRepository.cs  7

See the complete overview on Codacy


[zlmediakit] enable mp4/webrtc default

And update zlmediakit to last version.

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

@microsoft-github-policy-service agree


Part 3 - Rename valueType to storedType in ForwardIndexReader

This part 3 of a multi-part set of changes to add support for multi-value noDict column types for immutable and mutable fixed width columns. Support to create segments with multi-value columns with dictionary disabled already existed. On trying to create a multi-value RAW column and querying it, we ran into this issue: https://github.com/apache/pinot/issues/8875
Original PR that adds complete support for querying MV raw columns - https://github.com/apache/pinot/pull/8917. This PR is split part 3 from original PR. Part 1 was merged in https://github.com/apache/pinot/pull/8953. Part 2 was merged in https://github.com/apache/pinot/pull/8993.
This PR renames valueType to storedType in ForwardIndexReader
cc @siddharthteotia @Jackie-Jiang @walterddr

Codecov Report

Merging #9004 (bd33179) into master (db452c5) will increase coverage by 3.30%.
The diff coverage is 24.59%.

@@             Coverage Diff              @@
##             master    #9004      +/-   ##
============================================
+ Coverage     63.46%   66.76%   +3.30%
+ Complexity     4921     4752     -169
============================================
  Files          1776     1365     -411
  Lines         93673    69580   -24093
  Branches      14096    10886    -3210
============================================
- Hits          59449    46457   -12992
+ Misses        29972    19848   -10124
+ Partials       4252     3275     -977

Flag
Coverage Δ

unittests1
66.76% <24.59%> (+<0.01%)
:arrow_up:

unittests2
?

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

Impacted Files
Coverage Δ

...re/common/evaluators/DefaultJsonPathEvaluator.java
29.53% <0.00%> (ø)

...e/operator/dociditerators/MVScanDocIdIterator.java
70.37% <0.00%> (ø)

...e/operator/dociditerators/SVScanDocIdIterator.java
75.51% <0.00%> (ø)

...local/indexsegment/mutable/MutableSegmentImpl.java
54.89% <0.00%> (ø)

...readers/constant/ConstantMVForwardIndexReader.java
0.00% <ø> (ø)

.../readers/forward/FixedBitMVForwardIndexReader.java
73.49% <ø> (ø)

.../readers/forward/FixedBitSVForwardIndexReader.java
0.00% <ø> (ø)

...eaders/forward/FixedBitSVForwardIndexReaderV2.java
100.00% <ø> (ø)

...ocal/segment/readers/PinotSegmentColumnReader.java
70.21% <0.00%> (ø)

...t/segment/spi/index/reader/ForwardIndexReader.java
46.86% <0.00%> (ø)

... and 424 more

Continue to review full report at Codecov.

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


spirv-llvm-translator: cherry-pick a patch to fix build with latest spirv-headers

Description of changes
Renaming public API woo
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Is that patch only required for a specific LLVM version or starting from a version? It doesn't apply when using the llvm 14 branch and through that breaks intel-graphics-compiler and intel-compute-runtime.

It should be required for all versions, but it might need backporting... Is there a reason Intel stuff is stuck on 14? Can we not update it instead?

That is what upstream is using and their code in intel-graphics-compiler is hairy and often enough breaks even in the current branch. I don't think we can lift that to some other major version.


session hangs after repeated queries

Running latest gorethink master and rethinkdb 2.0.1
Essentially the issue happens after a number of rethink queries like below.  This hang doesn't happen every query, sometimes it comes back instantly, and it seems to not present itself if the query is intermixed with other queries (i.e. not repeated).  I've found the most reliable way to reliably get the hang is to trigger the same query multiple times with the same requests.  The debugging I've done in my code shows this query getting called but never returning.
This also causes all future requests to the session to hang.
r.Db(config.DbName).Table(config.TableName).GetAllByIndex("indexname", "value").Run(session)
where session is initialized as:
session, err := r.Connect(r.ConnectOpts{
   Address:  config.RethinkAddress,
   Database: config.DbName,
   MaxIdle:  10,
   MaxOpen:  10,
 })

Right now this call is embedded inside my application but if I'll try to workup a POC to see if I can replicate the behavior.
Here is a screenshot of the cluster performance testing the above query 9 times approx. every 1 sec.  Notice only 8 queries.

Are there any ways to enable some debugging or logging from the gorethink driver side?  (Edit: I found SetVerbose but haven't seen any changes in the log output).
I'm still not convinced I'm not just doing Something Wrong™

With some more testing I found that if I increased the MaxOpen ConnectOpt the number of queries that it took to hang the session rose to the same level.  Setting the Timeout explicitly didn't seem to have any effect.

Hi @bryfry, I am not quite sure what is going on just yet and I haven't got much time to look into this today but I will try to figure it out over the weekend.
As soon as I know more I will get back to you.

After running some quick tests to see if I could replicate the issue (I couldn't unfortunately) I have some follow up questions to help narrow down the cause:

What are you doing with the returned cursor, you may not be closing it which could explain the issue as the connection would be kept open and not returned to the pool?
Are you running the queries in parallel?
Is the issue limited to a single type of query?
What do you mean by "doesnt hang every query"? Do you mean that different ReQL queries succeed/fail or that the exact same ReQL query will randomly fail but sometimes work?

The specific use case I've been using to test with from above is then passed to an err = res.One(&value).  I've never done any closing after a Run, One.  Should I be res.Close()ing?  That makes so much sense if so.
Queries are being run in parallel and sharing the Session
I think all of the Exec and RunWrite queries I do for setup run fine (many in sequence).

I'm definitely going to go try Close on all the things right now.

Which version of the driver are you using? There was an issue before where One was not closing the cursor (it should).

I'm running latest master (did go get update before opening this ticket) with rethinkdb 2.0.0 and 2.0.1 in test.

Ok well at least we have narrowed down the cause of the issue, I will try to fix the bug this evening.

Hi, sorry for the delay I have not been feeling so great over the last week and have been taking some time off.
I have still not managed to track down the issue and as far as I can tell calling One will always close the cursor (https://github.com/dancannon/gorethink/blob/master/cursor.go#L265). Would it be possible to see the code that is hanging?

Im going to close this for now, if anybody still has this issue feel free to reopen it.


Addressing several rollback-related issues

Signed-off-by: Laird Nelson<EMAIL_ADDRESS>

@paulparkinson you are the guy.  Thanks and happy Thanksgiving.

@paulparkinson assuming I haven't screwed up my rebase could you kindly re-approve this?

yep, looks unbroken :)


Cannot open macro deck after not using for a while

Describe the bug
Cannot open macro deck or connect to android
To Reproduce
No way to reproduce
Expected behavior
The client opens upon double click.
Log file
2023-11-26.log

Fixed after removing Media Controls plugin


popup not show immediately when following a getchar()

Below is the test script.
test.vim
function! Test()
  let winid = ['111', '222', '333']
        \ ->popup_create(#{
        \ maxwidth: 10,
        \ minwidth: 10,
        \ maxheight: 5,
        \ minheight: 5,
        \ pos: 'center',
        \ })
  echom winid
  let nr = getchar()
  echom nr
endfunction

run vim -S test.vim --clean
run :call Test()

Then you can see that winid is printed but there is no popup shown and vim wait for me to type keys. Then after the execution of getchar(), the popup is shown.
Environment:

Vim version: VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Oct 24 2020 16:26:37) contain patch 1-2020
OS: Ubuntu 18.04,
Terminal: GNOME Terminal

As a workaround, you can add a :redraw right after invoking popup_create() to force Vim to redraw the screen, and display the popup:
function! Test()
  let winid = ['111', '222', '333']
        \ ->popup_create(#{
        \ maxwidth: 10,
        \ minwidth: 10,
        \ maxheight: 5,
        \ minheight: 5,
        \ pos: 'center',
        \ })
  redraw
  echom winid
  let nr = getchar()
  echom nr
endfunction

closing as duplicate of #7118


Dark overlay on ultrawide

Would it be possible to stretch, or completely remove that dark overlay for ultrawide users?

yeah please!! it's unplayable like that

yeah please!!

Same here, please add option to remove it if is possible!


[Question]:  Execution failed for task ':device_info_plus:compileReleaseJavaWithJavac'. > error: invalid source release: 17

What is your question?
Execution failed for task ':device_info_plus:compileReleaseJavaWithJavac'.

error: invalid source release: 17
getting error when try to take build

Checklist before submitting a question

[X] I searched issues in this repository and couldn't find such bug/problem
[X] I Google'd a solution and I couldn't find it
[X] I searched on StackOverflow for a solution and I couldn't find it
[X] I read the README.md file of the plugin
[X] I am using the latest version of the plugin
[X] All dependencies are up to date with flutter pub upgrade
[X] I did a flutter clean
[X] I tried running the example project

Use the search function or search online, this has been asked before


Site Groups - Tooltip Being Cut Off

Description of bug
Tooltips in the Site Groups extension are being cut off in the Persona Bar by other UI elements,
Steps to reproduce

Open Site Groups from Manage > Site Groups
Hover over the Master Site or Authentication Domain tooltip icons
Notice the tooltip opening but most of it is being covered by another UI element.

Expected behavior
The tooltips should be visible to users.
Screenshots

Affected version

[x ]  9.6.0 latest supported release

Affected browser
Tested on:

[x ] Microsoft Edge Chromium

still an issue in DNN 9.6.2

still an issue as of DNN 9.8 RC1


adjust circles on resolution change

MapCircle objects are created with a radius given in the unit of meters. When the map zoom is changed, the representation of the circle must change accordingly.

fixed, will be in the next release


"camel-jackson - Upgrade to 2.13.x"

CAMEL-17469:camel-jackson - Upgrade to 2.13.x

Both camel-spring-boot and Quarkus using camel-jackson
I run the build with test cases for both

can i update this version also to the 2.13.1

camel-karaf is removed as per this commit
https://github.com/apache/camel/commit/db7b13344b1c2dc4bfdd8de4530d0aaae4f01a93

The json-validator in karaf needed to be updated too - I have done that


Implement numpy.random.shuffle and numpy.random.permutation

Is your feature request related to a problem? Please describe.
As shuffle and permutation are used to reshuffle data before training in deep learning or some stuff, we can introduce the API to Mars.
Describe the solution you'd like
We can implement them with shuffle, in the shuffle map, we generate a random 1-d array in which each element indicates which downstream should sent to, and in the shuffle reduce, random shuffle the data.

We can simply implement these two methods via a fancy index on a Tensor value since fancy index on tensor value is already implemented using shuffle.

We can simply implement these two methods via a fancy index on a Tensor value since fancy index on tensor value is already implemented using shuffle.

Maybe it's a bit hard to generate a index tensor which contains non-repeat numbers?


failed: resolving dependencies

while "bundle install" after setting up the GEMFILE I get following error:
Bundler could not find compatible versions for gem "rails":
In Gemfile:
pageflow (>= 0) ruby depends on
rails (~> 4.0.2) ruby
rails (4.1.6)

as shown, rails 4.1.6 is installed and shoul meet the needed version > 4.0.2
how to solve this?
Regards, Johannes

I am experiencing issues with bundler 1.3.0 and current version is 2. plus. can you help?

Please open a new issue instead of commenting on a closed one, include a link to a gist containing your full Gemfile and Gemfile.lock and include the exact output/error messages you are seeing.


Create ADAPTER_README.md

Hey @natasha-pel,
Closing this pull request, it seems outdated since we already have Virtual Schema for Hive in exasol/virtual-schemas.


Listview background

Hi,
I have used a little bit that library, but I am unable to change the background colour of the listview(ie when it's empty i just have plain white) is there any way doing it, the way it is built with the row_bg.xml layouts for every item?
Thanks

The adapter shows your row layout.
You should make sure that the row layout you provide has the background you want.

Thanks for the answer, however, the background of the row layout, just alters the colour of each row with items, and not the actual background on a list with empty items.
ie:

Thanks

Then set a background on your listview or one of it's parent layouts.
This adapter doesn't touch that. It just wraps individual items in your list.

Thanks a lot , that seemed to work. I was not using a listview layout at all, and I thought it could be implemented otherwise from the rowlayout.


Support for updating repository url if repo file is already exist

Support for updating repository url if repo file is already exist

There's some additional details here: https://github.com/acch/ansible-scale/issues/7

I'm working on this now: https://github.com/acch/ibm-spectrum-scale-install-infra/tree/iss273

I can't seem to be able to reproduce the issue described in acch/ansible-scale#7. Looks like yum_repository will now update existing files if the baseurl changes.
Ansible version: 2.9.14
Python version: 3.8.6
Managed host: RHEL 7.8
This can be verified with a quick test:
- hosts: node1
  tasks:

    - name: test | Clean existing YUM repository
      file:
        path: /etc/yum.repos.d/spectrum-scale-gpfs.repo
        state: absent

    - command: ls -l /etc/yum.repos.d/

    - name: test | Configure new YUM repository
      yum_repository:
        name: spectrum-scale-gpfs
        description: IBM Spectrum Scale (GPFS)
        baseurl: http://infraserv/yum/gpfs/<IP_ADDRESS>/gpfs_rpms
        state: present

    - command: grep baseurl /etc/yum.repos.d/spectrum-scale-gpfs.repo

    - name: test | Modify existing YUM repository
      yum_repository:
        name: spectrum-scale-gpfs
        description: IBM Spectrum Scale (GPFS)
        baseurl: http://infraserv/yum/gpfs/<IP_ADDRESS>/gpfs_rpms
        state: present

    - command: grep baseurl /etc/yum.repos.d/spectrum-scale-gpfs.repo

The output of the first grep is:
baseurl = http://infraserv/yum/gpfs/<IP_ADDRESS>/gpfs_rpms
The output of the second grep is:
baseurl = http://infraserv/yum/gpfs/<IP_ADDRESS>/gpfs_rpms (version number has changed)
Hence, I guess that there were changes to the yum_repository module and it now works as expected (modifies existing repos).
Are you able to reproduce the issue with Ansible 2.9? If so, can you please provide more details as to what's missing?

@rajan-mis Are you still able to reproduce this issue or shall we close it?

@rajan-mis Are you still able to reproduce this issue or shall we close it?


Kubejs issue

Describe the Bug
I followed along the recipe for Kubejs 6.1 but for some reason it does not create it.
event.recipes.createsifter.sifting([
item.of('minecraft:iron_nugget').withChance(0.3),
item.of('create:copper_nugget').withChance(0.4),
item.of('create:zinc_nugget').withChance(0.4)
], ['minecraft:gravel', 'createsifter:string_mesh']).waterlogged()
I tried createsifterShifting still the same
btw codeblock 19 is where the waterlogged() is
Reproduction Steps

install latest kubejs and kubejs create together with create sifter mod
run the code

...
Expected Result
creates a recipe based from the code
Screenshots and Videos

Crash Report or Log
No response
Operating System
windows 10
Mod Version
1.20.1-1.8.1.e-22
Minecraft Version
1.20.1
Forge Version
Neoforge 47.1.106
Other Mods
No response
Additional Context
No response

event.recipes.createsifter.sifting({
type: 'createsifter:sifting',
ingredients: [{item: 'minecraft:gravel'}, {item: 'createsifter:string_mesh'}],
results: [
{item: 'minecraft:iron_nugget', chance: 0.3},
{item: 'create:copper_nugget', chance: 0.4},
{item: 'create:zinc_nugget', chance: 0.4}
], waterlogged: true
})
resorted to using .json welp


Better log messages

The goal of this PR is to provide the original ingestion failure reason in the logs when the initial storing to disk fails.
Motivated by this issue https://github.com/microsoft/ApplicationInsights-Java/issues/3207.

API change check
API changes are not detected in this pull request.


Translate Part 15 into Spanish

Please also make sure that the title of the lesson is also translated.


Added optimistic locking implementation for reactive cosmos repository

Added optimistic locking implementation for reactive cosmos repository

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Clones added
============
- src/main/java/com/microsoft/azure/spring/data/cosmosdb/core/ReactiveCosmosTemplate.java  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Clones added
============
- src/main/java/com/microsoft/azure/spring/data/cosmosdb/core/ReactiveCosmosTemplate.java  1

See the complete overview on Codacy

/azp run

 Here is an overview of what got changed by this pull request:

Clones added
============
- src/main/java/com/microsoft/azure/spring/data/cosmosdb/core/ReactiveCosmosTemplate.java  1

See the complete overview on Codacy


fix: failed connectionless proof request on some case

In demo menu (2a) connectionless proof request, If a mobile agent (for instance Trinsic) does not include Accept header when calling proof request url, the demo would fail.
Signed-off-by: kukgini<EMAIL_ADDRESS>

@WadeBarnes -- can you look at the CircleCI error above  -- "Could not find a usable config.yml, you may have revoked the CircleCI OAuth app."

@WadeBarnes -- can you look at the CircleCI error above -- "Could not find a usable config.yml, you may have revoked the CircleCI OAuth app."

This looks like a very recent change to some config in GitHub.  If you can see the list in this link; https://app.circleci.com/pipelines/github/hyperledger/aries-cloudagent-python.  Run 4466 was the first occurrence triggered by this PR.  Run 4467 triggered CircleCI just fine.  Runs 4468 to 4471 were me attempting to redeliver the trigger from GitHub webhooks to CircleCI.  @ryjones, the error indicates there may have been something revoked on the GitHub side.  Know of anything?  Based on the config in GitHub and CircleCI is should be using the deploy key you setup for it for it back in 2019.

I made this PR using the 'Edit this file' function on github. Could that help find the cause?

This PR is stuck without some solution to the Circle CI issue (which I don't understand :-( ).  I think we need to be able to accept PRs that are submitted via the "Edit This File" capability -- it's very helpful in some situations.

https://support.circleci.com/hc/en-us/articles/360043002793-Troubleshooting-CircleCI-Access-After-Enabling-Github-SSO

Codecov Report

Merging #1933 (de5a61a) into main (4e5ed0f) will decrease coverage by 0.00%.
The diff coverage is 90.00%.

:exclamation: Current head de5a61a differs from pull request most recent head 9f07410. Consider uploading reports for the commit 9f07410 to get more accurate results

@@            Coverage Diff             @@
##             main    #1933      +/-   ##
==========================================
- Coverage   93.59%   93.59%   -0.01%
==========================================
  Files         539      539
  Lines       34262    34270       +8
==========================================
+ Hits        32069    32076       +7
- Misses       2193     2194       +1


Configuring where the cookie.json file is stored

It would be helpful to leave the Auto-GPT folder clean of unversionned artifacts.
Currently, I have to use a script to move cookie.json out of my Auto-GPT folder so that I can pull and rebase.

I just saw the config feature, but the README was misleading.


Wfly 13737

Thanks for submitting your Pull Request!
Please delete this text, and add a link to the Jira issue solved by this PR.
Remember to use the Jira issue ID in the PR title, and any commits.

Hello, Amitninja12345. I'm waiting for one of the admins to verify this patch with /ok-to-test in a comment.

It looks like something has gone badly wrong with your topic branch, it may be an idea to create it again from master and add your commit.
For component upgrades please also include release notes and a link to a diff comparing the two releases so we can see what has changes.

@Amitninja12345 The problem here is your branch is based on WildFly's 20.x branch, not on master. As Darran said the thing to do is create a branch based on master.

I will do it from master branch,


Support External Editors

Yo! Great work - I've been using your dialogue plugin in my series.
Sorry if this has already been suggested, I tried searching but nothing came up.
Is your feature request related to a problem? Please describe.
I use VS code as an external editor, dialogue files are opened in the Godot editor rather than VS code.
Describe the solution you'd like
When I click the dialogue files in Godot they should open in the external editor of choice if I have use external editor selected.

I can probably make it open in the external editor of choice but you'll lose any syntax highlighting and inline error checking.

I can probably make it open in the external editor of choice but you'll lose any syntax highlighting and inline error checking.

Good point, could it be toggled in a setting maybe? Would be curious to try it!

I've added it as an advanced option which only shows up in settings if you've configured an external editor in Editor Settings first.


Add alert for paused machineconfigpools

Checklist

[x] The PR has a meaningful title. It will be used to auto-generate the
changelog.
The PR has a meaningful description that sums up the change. It will be
linked in the changelog.
[x] PR contains a single logical change (to build a better changelog).
[x] Update the documentation.
[x] Categorize the PR by adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog.

for review, the command in the runbook can be checked by looking at all non-paused mcps on a cluster like so:
kubectl get mcp -oyaml --show-managed-fields | yq '.items[] | select(.spec.paused | not) | .metadata | {.name: .managedFields[] | select(.fieldsV1."f:spec"."f:paused")}'


Agility Plugin Bug

not sure if this affects all courses or just the ardy course but if you turn the agility plugin off and on again while not on the course the click boxes for the obstacles on the roofs dont show anymore but the starting obstacle does still show and if you turn the plugin off and on again while on the course the starting obstacle click box doesnt show but the roof obstacles do show
https://i.imgur.com/eGNPuTB.gifv

Sup HSJ, could you edit your comment to use commas and full stops, so it's not a huge run-on sentence? It's a bit of a reading headache right now.


Error: from config: failed to resolve output format "WebAppManifest" from site config

What version of Hugo are you using (hugo version)?

$ hugo version
Hugo Static Site Generator v0.68.3/extended linux/amd64 BuildDate: 2020-03-25T06:15:45Z

Hello, when I run hugo server on this template: https://github.com/wowchemy/starter-hugo-research-group, I have this error: "Error: from config: failed to resolve output format "WebAppManifest" from site config". Is there a solution to resolve this?
I run `hugo mod clean --all' (https://wowchemy.com/docs/hugo-tutorials/troubleshooting/#error-failed-to-resolve-output-format), but the same error happens.

Please ask the Wowchemy team:
https://github.com/wowchemy/wowchemy-hugo-themes/discussions 8
They also have an active Discord channel:
https://discord.com/invite/z8wNYzb 2
In the future, please use the forum (https://discourse.gohugo.io/) for questions and troubleshooting. We prefer to use GitHub for verified bugs and vetted enhancements. Thanks.


dependencies: Update to nx 19.8.4 and angular 18.2.8

general notes:

remove experimental dependency
remove explicit zone.js (included in angular)
run nx migrations

commands run:

npx nx migrate 19
npm install
npx nx migrate --run-migrations

This update seems to work for Linux, but it fails for Windows. So a Windows user needs to change some dependencies that are causing issues. (Waiting could also just solve it, since newer versions could have fixed the bug)

I updated again, but the bug is still not fixed - now it works on Windows but fails in the pipeline :/
Others are experiencing similar problems.
We should probably wait even longer and hope for a fix.


Discord link in README and docs is broken

The link to the discord is given by this line in the README:
https://github.com/KomputeProject/kompute/blame/master/README.md#L67
here is the link given in the above:
https://discord.gg/ywjaBH6v24
Discord tells me this link is broken.
I would really like to get up and running with Kompute, so being able to join the Discord would be super helpful. Thank you!

Thank you very much for the heads up - I hadn't realised the discord link was broken, I have now created a new link which you can use https://discord.gg/MaH5Jv5zwv I will update it in the readme as well


connect refused

Hello, everyone.
When I try to use nmea_navsat_driver to convert my GPS data into /nmea_msgs/Sentence, it does not work well.
My command is rosrun nmea_navsat_driver nmea_topic_serial_reader _port:=/dev/ttyUSB0 _baud:=115200. And it turns out Connection refused. Details are shown before.
File "/opt/ros/noetic/lib/nmea_navsat_driver/nmea_topic_serial_reader", line 38, in
libnmea_navsat_driver.nodes.nmea_topic_serial_reader.main()
File "/opt/ros/noetic/lib/python3/dist-packages/libnmea_navsat_driver/nodes/nmea_topic_serial_reader.py", line 59, in main
rospy.init_node('nmea_topic_serial_reader')
File "/opt/ros/noetic/lib/python3/dist-packages/rospy/client.py", line 326, in init_node
_init_node_params(argv, name)
File "/opt/ros/noetic/lib/python3/dist-packages/rospy/client.py", line 186, in _init_node_params
set_param(rosgraph.names.PRIV_NAME + param_name, param_value)
File "/opt/ros/noetic/lib/python3/dist-packages/rospy/client.py", line 539, in set_param
_param_server[param_name] = param_value #MasterProxy does all the magic for us
File "/opt/ros/noetic/lib/python3/dist-packages/rospy/msproxy.py", line 136, in setitem
self.target.setParam(rospy.names.get_caller_id(), resolved_key, val)
File "/usr/lib/python3.8/xmlrpc/client.py", line 1109, in call
return self.__send(self.__name, args)
File "/opt/ros/noetic/lib/python3/dist-packages/rospy/core.py", line 669, in _ServerProxy__request
return xmlrpcclient.ServerProxy._ServerProxy__request(
File "/usr/lib/python3.8/xmlrpc/client.py", line 1450, in __request
response = self.__transport.request(
File "/usr/lib/python3.8/xmlrpc/client.py", line 1153, in request
return self.single_request(host, handler, request_body, verbose)
File "/usr/lib/python3.8/xmlrpc/client.py", line 1165, in single_request
http_conn = self.send_request(host, handler, request_body, verbose)
File "/usr/lib/python3.8/xmlrpc/client.py", line 1278, in send_request
self.send_content(connection, request_body)
File "/usr/lib/python3.8/xmlrpc/client.py", line 1308, in send_content
connection.endheaders(request_body)
File "/usr/lib/python3.8/http/client.py", line 1251, in endheaders
self._send_output(message_body, encode_chunked=encode_chunked)
File "/usr/lib/python3.8/http/client.py", line 1011, in _send_output
self.send(msg)
File "/usr/lib/python3.8/http/client.py", line 951, in send
self.connect()
File "/usr/lib/python3.8/http/client.py", line 922, in connect
self.sock = self._create_connection(
File "/usr/lib/python3.8/socket.py", line 808, in create_connection
raise err
File "/usr/lib/python3.8/socket.py", line 796, in create_connection
sock.connect(sa)
ConnectionRefusedError: [Errno 111] Connection refused
I hope someone can give me a hand and I am very appreciate for it.
Thank you!

Did you run roscore beforehead?

As @zhangjinwei786 points out, this error message indicates that the node can't connect to a ROS core. You either need to run roscore or launch the node with a launch file and roslaunch, which will automatically start a ROS core if one isn't running.


Customizable json output to reduce file size / Omit raw / body

Please describe your feature request:

Katana already allows customizing the output.
However, this does not apply to the json output format.
Currently, Katana always dumps the whole response into the JSON file both in raw and parsed.
Thereby, the json file is getting very enormous in size quickly.
I propose the following parameters, to reduce the file size:

--omit-raw:  Omit both raw request and raw response from the (json)-output
--omit-body: Omit saving the  parsed response body ({"response": { ... "body": ... }}) from the the (json)-output

Describe the use case of this feature:

The body responses blow up the output most
The above mentioned implemented flags reduce the output file size effectively
Thereby, the meta data (URL, status code, technologies) can be processed after spidering, without dealing with the huge body responses

@maik-s Irregardless of customizable json output, are you able to get any json output at all?? katana -list urls.txt -kf -c 4 -p 4 -jc -d 4 -header "User-Agent: user-agent" -timeout 60 -json -output katana_target_com_results_testing.json -nc -verbose Doing that leads to the json output file not even being created. When JSON output gets fixed, allowing it to be customized is extremely important.

Yes, I get json output, but as described not customisable and therefore it blows up very quickly.
In your case I have the following guess, why you don't retrieve output.
If it doesn't help, please open a seperate issue to keep this thread stick to the original topic.
You specify parameter -kf, which expects a string value (all, sitemap, robotstxt), which you don't supply. Thereby, I think the argument parser is confused for the consecutive arguments and the json option etc is not respected correctly.

Eureka! You are very correct, for some reason I just assumed doing -kf would default to all which seems to be a bit more logical than how it's currently structured but that's okay! Now that is it settled, I wholeheartedly agree with your suggestions as they would be very useful for post processing katana JSON results!

@maik-s thanks for adding this support; now included in the latest release - https://github.com/projectdiscovery/katana/releases/tag/v1.0.2

Thanks for reviewing and merging this so quickly!


Avif support

I'm getting "Internal Server error"  when trying to convert larger images to avif format.
I can understand that the memory might hit the roof at some point, but then I found this in my Lambda logg saying avif is not among the supported filetypes.. Is that just a text update missing or is it not yet fully supported? The Sharp.js library has avif as an output format, so I was counting on it to work.
2021-03-18T07:15:48.654Z	7031ea41-23f3-45ee-b01e-b10b90d39bc3	ERROR	{
  status: 400,
  code: 'RequestTypeError',
  message: 'The type of request you are making could not be processed. Please ensure that your original image is of a supported file type (jpg, png, tiff, webp, svg) and that your image request is provided in the correct syntax. Refer to the documentation for additional guidance on forming image requests.'
}

It works when I convert smaller images like 800x800px but when trying to do 1600x1600 it returns with "Internal server error"
The same operation works fine for larger images with the webp format, so I'm not sure if avif conversion takes alot more memory or if there is something else going on.
Is there anything more I can do about it?

Sorry, I realized that the message was for original image input. But still, the avif conversion is really slow compared to other formats. Can I do anything about that?

Hi @simon-cordts , sorry for the late response. Unfortunately, avif conversion is slow, and also can be resource intensive. This is an issue with the sharp library and with avif conversion in general compared to other formats. The sharp library appears to currently be looking into improving the performance.
https://github.com/lovell/sharp/issues/2597
One option you can try is adding the speed option for avif, this can speed up the conversion, but will also increase CPU and Memory consumption.
https://sharp.pixelplumbing.com/api-output#avif

Ok thanks, I'll follow the progress from there :)


LanguageService.GetEntryAssemblyTargetFramework() only works when the debug target has System.Linq loaded

When the debug target does not use / load the System.Linq assembly, the expression used to get the target framework fails as invalid.
Possibly loading Linq into the target might work?

Can't reproduce - I just created a .net6 console app and all works.
What do your references look like?
There is a reference to System.Linq via Frameworks => Microsoft.NetCoreApp => System.Linq.

    namespace ConsoleApp1
{
    internal class Program
    {
        static void Main(string[] args)
        {
            FileInfo fi = new FileInfo("TestFile");
            Console.WriteLine(fi.FullName);
        }
    }
}`

As simple as it gets. breakpoint on line 8 (Console.Write) and dump fi. fails when evaluating the targetframework.

Interesting, it seems to fail only when using a new instance of the experimental visual studio instance. Guess this might be some caching problem. Once the instance has loaded Linq for an debug target, it works always. Even when the current project does not reference Linq.

Ahh, yes if we are talking about debugging this extension - it fails sometimes when runs a GetEntryAssemblyTargetFramework(). Do you have this issue in non-Debug mode?

I don't know if it's a bug - it appears in extension development mode only. But your example with FileInfo is absolutely brilliant. It totally interrupts the debugging session in the prod version of the extension for C# and VB. Need to investigate the reason for this behavior.

Sometimes less is more, hahahaha

Yup, it's because of stack overflow.
Try something other than fileSystem objects, because they are Composite. Everything has a reference to parent and root.

Hei @Hefaistos68,
Could you recheck the issue with the file info?
I've merged a workaround for FileInfo and DirectoryInfo, also, now recursion depth is limited by 100.

LanguageService.GetEntryAssemblyTargetFramework() doesn't more use System.Linq.


os-smart: Re-add descriptions for cron actions

The "description" fields were removed to not flood the UI with duplicate
items but some of them are actually useful for cron.
Descriptions for actions related to tests were re-added with unique
content to enable periodic SMART tests via cron.

Question: how will you inspect the results of these cron runs?

@fichtner all of these commands do the actual work in the background. Results of the tests can be checked using existing "View Logs" feature of SMART plugin.

@fichtner any more questions or comments?

Merged, thanks!

Merged, thanks!


Rename shell extension Verb Id 2

Workaround Windows issue causing shell extension to not appear in classic context menus on certain systems.
Continuation of #997

@JeromeMartinez I hope is parses numbers before letters. If this does not work then aaaaMediaInfo 😛

https://mediaarea.net/download/snapshots/binary/mediainfo-gui/20241218-2/MediaInfo_GUI_24.12.20241218_Windows.exe
Still works fine on my system as expected.

When right-click a shortcut to a folder, MediaInfo only appears on the classic menu and not the new main menu.

And with the previous version "CMediaInfo" it is not the case?
Crazy...
Do you see how you could change that? At least we have a machine with this "issue" now :).

Feedback for the user:

this one works, both, files & folders... I needed to restart the explorer process manually but after that everthing fine...

It is so crazy... But if it the way to go, we do that.

To be clear, the MediaInfo entry on new menu for shortcuts disappeared when we changed * to the list of file extensions. To be this is not an issue/bug but expected behaviour since we did not list .lnk. However, I just discovered that the classic Windows menu has shortcut resolving feature so the shortcut will be resolved to the target and context menu entry behaves like the actual target is the one right-clicked.

It is so crazy... But if it the way to go, we do that.

My possible explanation that makes this not so crazy is that Windows has a bug that only parses a limited number of shell extensions from 0 - z for the classic menu. When user has many shell extensions installed, the one starting with later alphabets will be missing. So when NanaZip and MediaInfo didn't use "C", it does not work for several people who likely has many extensions starting before "M". When using "C", both apps work for almost all except a very few who maybe has extensions starting with "B" or 0 - A such as "BandiView" as tested by someone in NanaZip thread (NanaZip with "C" prefix stopped appearing as soon as BandiView installed was claimed). As for why Mp3tag and Windows pre-installed apps still work, maybe apps that were already parsed successfully previously are cached somehow. Anyway, I may be completely wrong because who knows how Windows actually works. 😆

I may be completely wrong because who knows how Windows actually works. 😆

For the moment, it works, so we do with that, hoping that Microsoft will fix that at some point, before everyone does "000000000000000000000...." stuff...

Maybe Microsoft wants us to resolve the shortcut and show the MediaInfo menu, doable?

If you want, we need to do shortcut resolving and extension filtering in the shell extension and just put * in manifest so that our shell extension will be called for everything including shortcuts. This way we can open hyperlinks to web media too!
Seems this is what Notepad and possibly other Microsoft apps shell exts does. The manifest has just * and Directory. Notepad entry does not show for directories but shows for all types of files. For shortcuts, invoking the context menu entry will open the target file in Notepad and not the shortcut file itself. Microsoft's and Google's context menu entries seem to take longer to load/appear compared to MediaInfo and Mp3tag so they are doing something there.

The manifest has just * and Directory

I was thinking to only shortcuts so ".url" files, and the shell extension dynamically checks if the target extension is what we want.

This way we can open hyperlinks to web media too!

I am fine with that.

For shortcuts, invoking the context menu entry will open the target file in Notepad and not the shortcut file itself.

It is what I expect. resolving that either in the shell extension or MediaInfo (maybe the later, so drag n drop works? Not currently the case)

My possible explanation that makes this not so crazy is that Windows has a bug...

Another possible explanation is that another shell extension that Windows loads first is somehow causing Windows to stop loading further extensions. This is because it does not really make sense that Windows has a shell extension limit and if so it is unlikely that there are that many shell extensions with verb starting earlier than "C". It is also in line with how a NanaZip user managed to reproduce the issue on a fresh install ("when I finish installing bandiview and hiding the context menu in the settings, it deletes the context menu of nanazip"). I also looked at previous screenshots and noticed that MediaInfo is now higher in the classic menu after the verb rename. As to why the new main menu is not affected, it is a different ordering there (not sure what affects the order) with MediaInfo already at the top. So it could be bugs in other shell extensions but if true, Windows is also at fault for enabling a shell extension to affect other shell extensions.


ESP-IDF离线安装包麻烦更新下 (IDFGH-11982)

Is your feature request related to a problem?
https://dl.espressif.cn/dl/esp-idf/?idf=4.4   IDF离线安装包已经不是最新的了，麻烦打包到最新，谢谢
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
No response

I'm closing this old report.


Ec explain

I guess you also need to add this file to mkdocs.yml

yeah, I forgot to push 2nd commit, mb: https://github.com/Dasharo/docs/pull/904/commits/576812772a1d950b53b6a25ede409a1a03e7f991


Undefined method bubbles?

After adding scss-lint to our project Gemfile test/development group (to be primarily used with Pronto), we got an error during assets precompile:
undefined method invalid_parent_method_name
After adding the necessary methods (pull request here), we now see:
undefined method bubbles? for #31abc2
Any tips on how to proceed? Or can't scss-lint be used like that, because it messes with sass too much?

How would you go about this with guard?
I use guard with nanoc to compile static HTML (and SCSS to CSS). I never require scss_lint, just bundle exec guard. Whenever nanoc compilation kicks in I get:
NoMethodError: undefined method `bubbles?' for 50px:Sass::Script::Tree::Literal

Adding require: false to my scss_lint-related gems (+ checkstyle, guard-scss_lint) doesn't remove the error.

The moment you run require 'scss_lint' in a Ruby process (which is what happens when you bundle exec rake scss_lint...) you're hosed—SCSS-Lint has monkey-patched the parse tree which causes any other Sass library calls to likely fail.
I've opened #750 to track fixing this issue.

@sds Thanks! :+1:


⚠ Mobitopup has degraded performance

In c1bdf24, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3290 ms

Resolved: Mobitopup performance has improved in 6a5678e.


ManagedShowOnStart make the dialog skip.

Hi!
I assume the feature would show the dialog only one time and never again.
In my situation, instead, the dialog isn't ever showed, and also the first time in the log appears "I/Changelog Library: Showing changelog dialog skipped".
How can I fix this?
Thanks

You probably just did not update the app version? It will show the changelog once for an app version only

Reopen if you still have problems


MNT: Added build script for versions

gah, sorry, wasn't paying attention and kept clicking the green button...  harmless change though...


[FM-6956] Removal of unsupported OS from apt

https://github.com/puppetlabs/ci-job-configs/pull/4540

failure is a pre-existing bug.

Merging due to failures being a separate issue


TOML configuration file not reading ckpt_file and vocab_file variables

Checks

[X] This template is only for bug reports, usage problems go with 'Help Wanted'.
[X] I have thoroughly reviewed the project documentation but couldn't find information to solve my problem.
[X] I have searched for existing issues, including closed ones, and couldn't find a solution.
[X] I confirm that I am using English to submit this report in order to facilitate communication.

Environment Details
WSL Debian 12.
Using CLI inference only.
Steps to Reproduce

Create a configuration file like in the example https://github.com/SWivid/F5-TTS/blob/main/src/f5_tts/infer/examples/basic/basic.toml

But add the additional variables ckpt_file and vocab_file
# F5-TTS | E2-TTS
model = "F5-TTS"
ckpt_file = "models/model_1200000.safetensors"
vocab_file = "models/vocab.txt"
ref_audio = "infer/examples/basic/basic_ref_en.wav"
# If an empty "", transcribes the reference audio automatically.
ref_text = "Some call me nature, others call me mother nature."
gen_text = "I don't really care what you call me. I've been a silent spectator, watching species evolve, empires rise and fall. But always remember, I am mighty and enduring."
# File with text to generate. Ignores the text above.
gen_file = ""
remove_silence = false
output_dir = "tests"

Have of course the checkpoint and vocab.txt file in the correct path.

Execute the CLI and notice how its using models from default paths

f5-tts_infer-cli -c basic.toml

The script is using the models loaded from different path instead the ones assigned. However if you pass the same variables as arguments then it does work:
f5-tts_infer-cli -c basic.toml  -p models/model_1200000.safetensors -v models/vocab.txt

I did not tested yet but I believe the same will happen with load_vocoder_from_local and vocoder_name
✔ Expected Behavior
The script should use the variables set in the TOML file
❌ Actual Behavior
The script is ignoring those variables in the TOML, but it works if passed as arguments via CLI

But add the additional variables ckpt_file and vocab_file

The problem is that, these additional variables are not mentioned in guidance cuz it's just not implemented with toml.

But add the additional variables ckpt_file and vocab_file

The problem is that, these additional variables are not mentioned in guidance cuz it's just not implemented with toml.

Then it's not a bug but a missing feature. I was expecting the same variables that can be override from the arguments to be configured from the configuration file.
Thanks for the quick reply.


Read signing configuration from signing.gradle

Instead of manually pasting configuration values into gradle or
studio on each release, this lets the developer keep a
signing.properties file with the necessary information.

I accidentally closed this.

One nit. Otherwise LGTM!

done


Fix ICMPv6 checksums and add address resolution for IPv6

Fix ICMPv6 checksums.
Add Address Resolution for IPv6.
Update the server example to support IPv6.

After this basic on-link usage of IPv6 should work. I updated the server example to work with IPv6.

Fixes: #3

Oh what the hell, this has been fixed for 16 days? I'm sorry. I never got the mail from GitHub somehow.

@m-labs-homu r+

:pushpin: Commit 4a4ed8b has been approved by whitequark

:hourglass: Testing commit 4a4ed8bca76e43c5ee0b8dcaef652da138775ae1 with merge e4b501390754fa6fcb259d4fccf0fbb1ac681858...

:broken_heart: Test failed - status-travis

Neat! I guess unused arithmatic results are part of the unused lint now.

@m-labs-homu retry

:hourglass: Testing commit 4a4ed8bca76e43c5ee0b8dcaef652da138775ae1 with merge 840054cee82793641fef7a0b347216a10a57297a...

Oh what the hell, this has been fixed for 16 days? I'm sorry. I never got the mail from GitHub somehow.

I don't think I mentioned that I updated the PR.
Updated the test case to not have an unused arithmatic op.

@m-labs-homu retry

:hourglass: Testing commit 4a4ed8bca76e43c5ee0b8dcaef652da138775ae1 with merge 6c363762845e69dd316617f7ae0a083b4e534d5b...

:sunny: Test successful - status-travis
Approved by: whitequark
Pushing 6c363762845e69dd316617f7ae0a083b4e534d5b to master...


feat: upgrade niscode package

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


AllTraffic did not pass the ping health check

UnexpectedStatusException: Error hosting endpoint falcon-7b-: Failed. Reason: The primary container for production variant AllTraffic did not pass the ping health check. Please check CloudWatch logs for this endpoint..

@existeundelta Could you please share your code and other reproduce steps?


Develop Resource Page: Leadership: Leadership Principles

This issue is for coordinating around developing the resource page for Leadership: Leadership Principles.
The resource page lives in the following google document for easy editing, commenting, and revision:
https://docs.google.com/document/d/1QyMCck3nnDSAqff46nMhMf-Dt4MpoqC10X_PmnKFjFM/edit
Once the page is in a decent shape, we will put a copy of it in the repository as a markdown file.
To work on this issue, feel free to leave a comment here and start editing the document. For questions, feedback requests, and other coordination tasks, please leave a comment here on the issue.
Please edit the google document directly and add your ideas to it. We can help with cleaning it up later.

@GeorgLink : PR?

Do you agree that the metric ready? Then yes, PR would be great.
Whoever submits a PR, please use the new template that we have been discussing.

In #284, @emmairwin proposed a new version of this metric. We are currently working on the new proposal in this google doc:
https://docs.google.com/document/d/1_GT6HghiYFwGL4KwKZL2lCAdnTc5jBPuIVxLGOmphUw/edit?usp=sharing


Assemblies now formatted and pushed to database. Updated some databas…

Input assemblies are now gzip compressed and renamed to match the input sample name and saved to the BigBacter database. This is in preparation for future development.
Assembly files are included in the database summary function.
"Next steps" are provided when you run BigBacter without '--push true'.

Also caught error in seqtk_seq where the timestamp was being added to the assembly name. This was removed.


support @default jsdoc tag

Mainly to then document default values in option bags. https://jsdoc.app/tags-default.html

Closed with #294


Update to Footer Affiliate link

This pull request updates the footer affiliate class, to link to the what we do page instead of the home page of dta.gov.au

nice mate


#114 Add missing  with the name of the method being called.

Job #159 is now in scope, role is REV

This pull request #159 is assigned to @amihaiemil/z. The budget is 15 minutes, see §4. Please, read §27 and when you decide to accept the changes, inform @yegor256/z (the architect) right in this ticket. If you decide that this PR should not be accepted ever, also inform the architect.

@rok-povsic There are build failures, please fix them. I'll have a look afterwards :)

Codecov Report

Merging #159 into master will increase coverage by 0.16%.
The diff coverage is 92.85%.

@@             Coverage Diff              @@
##             master     #159      +/-   ##
============================================
+ Coverage     74.36%   74.53%   +0.16%
  Complexity      127      127
============================================
  Files            27       27
  Lines           905      911       +6
  Branches         54       55       +1
============================================
+ Hits            673      679       +6
  Misses          210      210
  Partials         22       22

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/org/jpeek/Skeleton.java
93.7% <92.85%> (+0.31%)
16 <0> (ø)
:arrow_down:

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 e42c4de...0862aa2. Read the comment docs.

@amihaiemil Build now passes.

@rok-povsic can you check the conflict?

@skapral ping ;)

@amihaiemil/z this job was assigned to you 5 days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@amihaiemil I've resolved the conflicts with the master branch. My task has been basically reimplemented in the master branch meanwhile the PR has been open. Anyway, the task is done and next puzzles open.

@rok-povsic If the task is already done and the PR doesn't solve anything, then I guess you can close it :-?

@amihaiemil I wouldn't agree with that. It's really not my fault that someone has decided to reimplement the functionality I've written already. He could've just copied the code from my commit (it has been pushed before any master branch changes).
Also, I've added a puzzle and tests that are not in master branch, so it's a step further in any case.

@rok-povsic I don't even understand, how and where has the ticket already been solved. I see you were the only assignee so far, so how has it been solved somewhere else?

@amihaiemil The same approach I've done has also been done here and merged yesterday. See the visitMethodInsn implementation. I've written basically the exact same code. After resolving the conflict with the master branch, it seems like I've done little work, just because the other PR has been merged into master before this PR.
If this PR is rejected, there would be a precedent that even though you do the work, you can still get no reward because someone merged in the same changes before you (or could even take the code you've commited, add it to his own PR, and if gets merged before you, steal your work).
But this PR in any case has added tests + done the next puzzle.

@rok-povsic also, please remove the keyword "fix" from the description. We shouldn't close tickets automatically, they should be closed by their authors always.

@rultor good to merge

@rultor good to merge

@amihaiemil Thanks for your request. @yegor256 Please confirm this.

@amihaiemil Thanks for looking into this, and for the comments which I've fixed (as you've seen). Yes, I also agree this shouldn't happen often.

@yegor256 Ping.

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 14min)

Order was successfully finished: +15 points just awarded to @amihaiemil/z, total is +540

Payment to ARC for a closed pull request, as in §28: +15 points just awarded to @yegor256/z, total is +9634

The job #159 is now out of scope

I think I should've been clearer in my description for #114. At the time I submitted the bug I was more interested in access to Bar.NAME than to the method call Bar.NAME.length().


Update doc structure

broken image? (introduction page)
Copy target - Teams + Enterprise

Looking good other than that!!


Incorrectly selecting the optimal epoch based on the test set

Considering main.py, Training.py, and Models.utils.py together, you have selected the epoch with the best result on the test set as the final result, rather than the best result on the validation set derived from the training set. This use of the test set as the validation set appears to involve information leakage and results in inflated performance metrics.
main.py

val_loader = DataLoader(dataset=test_dataset, ...)
val_evaluator = SupervisedTrainer(model, val_loader, ...)

Training.py

aggr_metrics_val, best_metrics, best_value = validate(val_evaluator, ...)
save_best_acc_model(aggr_metrics_val['accuracy'], epoch, model, ...)

same question


Developer feature: Strip 'Version' automatically from migration name in CLI

Just an idea, but if we'd strip 'Version' from the input argument, it would just be a matter of copying a class name.

This issue has been fixed in #7343


AMQP Support using uriparse.

Here is an implementation that deprecates the previous OpenConnnectionArguments::uri(uri: &str) function. It changes OpenConnectionArguments to accept host and port rather than a uri with only host and port. Chainables for host and port added. Trait TryFrom implemented to support conversion from an ampq(s) uri to an OpenConnectionArguments object. Supports only one query parameter heartbeat with room to support more. I believe there are other query parameters used to help support AMQPS with TLS, but TLS was not hooked up in this PR, but I think this serves as a great basis to implement the AMQPS initialization from an amqps uri.
This is in reference to issue #14, but does not fully address the TLS portion of the issue.

@jacobmiller22 , thanks for your PR. I have added some comments, and also please check the clippy error in check "integration-test/builds"

Hi @gftea, I have addressed the clippy errors in my latest commit. Where are the comments you mentioned? I cannot seem to find them (unless they are the added label and addition to the 0.5 release milestone)

@jacobmiller22 thanks for your great contribution. The PR is merged now!


Feature Request - Pending trades

Hi,
As per my tests, Pending Trades are not being retrieved by the bot, and for what I've read in Y! API documentation, it's possible.
Use case for us: our leagues are set with "Trade review: Commissioner", with the Commissioner been reported about the trade and them he would report to all GMs in a Discord channel. Then all GMs except those involved in the trade vote to accept or reject the trade.
With Pending Trades been reported by the bot, we can discuss and vote about the trade without the need of the Commissioner been reported and reporting back, and possible human failures there.
Is this possible? Again, thanks for this, we're thrilled with this bot!!

I have looked into this and it's only possible to see pending trades for the user who authenticated the bot with the their account.
And I slightly understand why Yahoo did this since others could gain an unfair advantage doing this.
A trade has two states whilst being considered "pending": offer/counter-offer and agreed. If the bot, or anyone else was allowed to see this besides the parties involved, that's where the advantage lies. They could sneak in an offer something before someone else could even see it or discuss it.

Hi,
Sorry, I don't see the advantage. Or I don't see the bot giving more advantage to the Commissioner than he already has.
I feel like I'm been a headache for you, thanks for your patience!

Not a headache at all.
As for what I stated before, let me clarify. The commissioner and others outside the parties involved in the trade can only see the pending trade when it is in the accepted state.
Yahoo's API gives the ability to see that state, as well as see the offer/counter-offer, which would be before the commissioner sees it.
But as stated previously, the API can only be used for the person's account who authenticated with the bot. It cannot see anyone else's pending trades.
That is why the bot can only see finished trades.

Just in case: we're not interested offer/counter-offer, but in agreed. At that point, the Commissioner, i.e. a player with that role, has to approve or reject the trade. So without the bot he already has some advantage. But as they say, "great power comes with great responsability".

We're only interested in trades with state accepted, and we'd use the Commissioner account for this. So the bot would send a message of an already agreed trade to all GMs.

The bot can only see trades that are finished. If they changed their API, I can look at it next season as I don't have time. I am also open to PRs if you can figure it out.
From their documentation:

Once you have the transaction_key for a pending trade that has been proposed to you, which you can get by asking the transactions collection for all pending trades for your team...

Ok, so I think I misunderstood you: when you're saying "the bot can only see..." you're not talking about your piece of code, but about the possibility the API gives you. Copy that now.
As for PRs, I will give it a try. But I'm not programming seriously (that is, apart from scripting) since I started to focus on systems 15 years ago! 😂
Thanks!!

Hi, coming back after exploring and finding how to do it. I have achieved it through this package. After authenticating, these are the code needed to extract accepted transactions:
    game = yfa.Game(session, 'nba')
    league = yfa.League(session, game.league_ids()[0])     # using [0], as I've just one league
    myteam_key = league.team_key()                                 # this gives you the team key in the current context, so myteam...
    myteam = league.to_team(myteam_key)                      # creates myteam object
    pending_trades = myteam.proposed_trades()              # all pending trades, accepted or not
    not_accepted_trades = list(filter (lambda d: d['status'] != 'accepted', pending_trades))

I then just have to manage the content of  not_accepted_trades.
Please use it at your interest. Again, I would gladly help, but kotlin really is out of my understanding.

@curropar have you tried this with another team's key that is not your own?

You actually have to make the request with your own team's key, and the response will contain all the trades with status accepted, including those where your team is not involved.

I forgot something important: pending_trades will contain both accepted trades (including your team or not) AND trades that you have or you have been proposed, and are not accepted or rejected yet! So the last line of code it's important to filter them, as they have to remain private.


ref(tests): Unhardcode integration list

Benefits of unhardcoding integration list and disabling auto integrations:

It becomes possible to successfully run tests in environments where certain extra auto integrations get enabled.
There is no need to update hardcoded list when new default integrations are introduced.

The origin of this change is a patch that I had to apply while packaging for Fedora: https://src.fedoraproject.org/rpms/python-sentry-sdk/blob/30f00540cd30328d578dfb84625226dcc7f28969/f/python-sentry-sdk.spec#_52 (https://src.fedoraproject.org/rpms/python-sentry-sdk/blob/30f00540cd30328d578dfb84625226dcc7f28969/f/0002-tests.test_new_scopes_compat_event-compatibility.patch). The patch was necessary because all possible optional dependencies for optional integrations are pre-installed as system packages, so that as many tests as possible could be run. Unfortunately, the patch no longer applies to a new version of the Sentry SDK, so instead of redoing the patch in Fedora, I am submitting this PR.

General Notes
Thank you for contributing to sentry-python!
Please add tests to validate your changes, and lint your code using tox -e linters.
Running the test suite on your PR might require maintainer approval. Some tests (AWS Lambda) additionally require a maintainer to add a special label to run and will fail if the label is not present.
For maintainers
Sensitive test suites require maintainer review to ensure that tests do not compromise our secrets. This review must be repeated after any code revisions.
Before running sensitive test suites, please carefully check the PR. Then, apply the Trigger: tests using secrets label. The label will be removed after any code changes to enforce our policy requiring maintainers to review all code revisions before running sensitive tests.

GitHub told to update the branch.

I rebased it on master.

@szokeasaurusrex It happened again:

What should I do?

@rominf I can handle getting the PR merged from here


Snocker via snorkel... take 2

Some simple directions and a docker file to create a snorkel installed docker image that can be used to run snorkel installed containers.
This image is also on my docker hub, and can be pulled with the following command:
docker pull drpinkacn/snocker:0.6.2

@ajratner I'm not sure what setting I am missing to add you as a reviewer. If you have any ideas, let me know.

:-/

Hey sorry, didn't mean to make you jump through hoops!  No worries about assigning the reviewer- I'll just look at this PR as soon as I can this week

Is this related to #323?

In order to try Snorkel, I tried this PR and have some small comments at https://github.com/HazyResearch/snorkel/pull/779/files.
Overall, I think that the folder structure is strange.
For example, why don't you put the Dockerfile in the top folder?

@DrPinkACN @vochicong First of all, sorry for my delay in getting to these PRs, and thank you both so much for the contribs!!
After having looked at each other's PRs (this one and #795), any consensus on how to proceed?  I'm less familiar with Docker so discount my own opinion a bit... thoughts?

@ajratner I am happy to make the recommended changes to the structure of this PR made by @vochicong or combine ideas here with #795 . I am not a docker aficionado either, my build was informed by things I knew I needed, or wanted to avoid. the primary merit with the docker image this docker file builds is that phusion more of a fully functional linux kernel and using mini-conda python 2 build makes installing numba a little more headache free. These were not addressed in the comments above, so as long as some piece of those contributions survive, I'll feel like I helped 😄 .
All of this could be for not if we just maintain a working snorkel-docker image at docker hub. Thoughts @vochicong? I'm happy to collaborate on making the best dockerfile/image we can for this purpose.

Thanks @DrPinkACN @ajratner for your input 👍
I have a lot of work experiences with Docker and can help you to confidently build a good, clean Docker. But I'm not familiar with Snorkel and would like to learn more about it.
Basically, #795 can be thought as a refactoring of #779, for a cleaner Dockerfile, more standard structure, and easier usage. Can you @DrPinkACN give it a try?
I can also help to set up Travis CI so that it will automatically build and push up-to-date images to Docker Hub.

We dropped the ball on this one; not sure why it wasn't merged in before. Looks like it's been long enough now that the files may need a few small updates to pass the tests again. If anyone picks this up and updates it, happy to take another look and merge it in.

I've reopened #795, which can be seen as a refactoring of this #779.

Closing—seems like this was updated in https://github.com/HazyResearch/snorkel/pull/795 !