Is shell scripting mode supported?

Hello,
Currently I'm using a couple of scripts on Linux that utilize the shell scripting capability in Nashorn so I'm able to just run the script from the command line.
I'm thinking about this mode:
https://docs.oracle.com/javase/8/docs/technotes/guides/scripting/nashorn/shell.html
The script has a shebang that points to jjs, e.g. #!/usr/bin/jjs -fv --language=es6.
Does graaljs support something similar?
I did a simple test where I edited the shebang to #!/root/graalvm-ce-java11-20.1.0/bin/js --experimental-options --js.nashorn-compat=true and the I tried running my script: ./myscript.js debug, I get the following:
ERROR: Unrecognized argument: '--experimental-options --js.nashorn-compat=true'. Use --help for usage instructions.
Maybe that's the way it's supposed to be?
Thanks!

Hi @idsecurity, thanks for experimenting with Graal.js.
On many systems, shebang does not split the arguments passed to the executable.
In your example, the js launcher receives a single argument ('--experimental-options --js.nashorn-compat=true') rather than two ('--experimental-options, --js.nashorn-compat=true'), and therefore fails to parse them.
Recent versions of env introduced a -S argument to split the arguments:
-S, --split-string=S  process and split S into separate arguments;
                        used to pass multiple arguments on shebang lines

In your case, you would have to add $GRAALVM_HOME/bin to $PATH and have this shebang:
#!/usr/bin/env -S js --experimental-options --js.nashorn-compat=true

If you run with an older env, I suggest you to write a proxy script named, for example, /root/graalvm-ce-java11-20.1.0/bin/graaljs-nashorn that passes the arguments that enable Nashorn compatibility to the js launcher:
#!/bin/bash

exec /root/graalvm-ce-java11-20.1.0/bin/js --experimental-options --js.nashorn-compat=true $*

In myscript.js you could then have:
#!/usr/bin/env graaljs-nashorn

There are probably more elegant solutions, but this should work around the issue.

Hi @idsecurity
thanks for your question.
@ansalond just answered the question around the multiple arguments - that is a problem with your hashbang syntax, not with GraalVM.
For your question itself: we have a --js.scripting option that is simulating the scripting mode also available in Nashorn/JJS.
Finally, you are passing the nashorn-compat flag. Did you check you actually NEED it? This is a legacy feature that you should only use if you really have to; it might lock you out of newer features of newer JavaScript (ECMAScript) features on the long run. See https://github.com/graalvm/graaljs/blob/master/docs/user/NashornMigrationGuide.md if you are using a feature that actually requires strict Nashorn compatibility.
Best,
Christian

Thank you both, I got the script running using the "proxy" solution. Now I get some other errors but at least it is running.
@wirthi , I will test if it runs without nashorn-compat, I'm using for each in my script so I'm unsure if that is a nashorn extension or not.


startDate and endDate are formatted wrong for /Projects

For the /projects end point, the dates for startDate and endDate are formatted incorrectly as yyyy-mm-dd when all other dates are formatted as ISO8601 compliant UTC.

ISO 8601, for representing JUST date (sans time component) is YYYY-MM-DD.
http://en.wikipedia.org/wiki/ISO_8601
Now as for other date representations in jellyfish there is a time component added, for example 2015-04-19T12:59:23Z is the format you might see for when an order was created or updated at.
Presently we don't account for project start time in the creation of a project and given this is the real possibility of the beginning of a project being at a specific time in addition to a date the time component probably shouldn't be added on as the action is encountered by the JF DB (in a similar fashion to that of order placing)
TL;DR  @chriskacerguis, do we care about project start times.  if so I am signing myself up for work.  If not, there really isn't anything to change here, dates are already served as iso8601.

I'm not 100% sure that the 'time' will be important, but you never know.  So, let's say yes we care about the start time.

@jklarfeld - based on the Allen's feedback, start/end_date should return dates not timestamps, is this an issue?
start/end_date should be returning ISO 8601 compliant dates - which it does, it just doesn't have an encoding of the time elapsed on that day like a traditional timestamp.

Ideally, all date/time stamps would return in the same format, for consistency and also to ease parsing.
I can work with the sans-timestamp version, although parsing code will have to be rewritten and setup to detect the second date format.
That is of course provided that these formats are set in stone in their current form and will not change.

@mafernando could we add the time, just to be consistent?

@jklarfeld - start/end_date should now come back as timestamps.
the necessary changes have been merged into master and deployed onto Heroku.


Efficient Hand Recognition in Openpose using ControlNet 1.1

It would be very useful to have in "openpose" the hands used by "ControlNet 1.1". The idea would be to continue using the current hands and when a button is pressed, the hands would appear in "openpose". This way, we would have the hands in "openpose", "depth", "normal", and "canny".

hey,  how did you control those vertex,  did you just use annotator and input an realistic image, or you can position those vertex manualy, useing what software?


TV on Homebridge site but not on iPhone

Plugin work good.
I config one TV and I can use it on Homebridge site.
But on my Home in iPhone there is Bridge but there isn't TV
How do I add the Tv on iPhone?

Plugin Config:
    {
        "debug": false,
        "configVersion": 1,
        "tvs": [
            {
                "ip": "<IP_ADDRESS>",
                "mac": "EE:07:0A:23:85:EE",
                "name": "TV Sala",
                "apiUser": "........",
                "apiPass": "..........."
            }
        ],
        "platform": "PhilipsAndroidTV",
        "_bridge": {
            "username": "FF:C2:6C:BB:BE:FF",
            "port": 12345
        }
    }

Screenshots:

Environment:

Plugin Version: v0.10.12
Homebridge Version: v1.5.1
Node.js Version:  v18.12.1
NPM Version:
Operating System: Raspbian GNU/Linux Buster (10)

see #69


allowInput no longer works with time picker

When using a time-only picker with allowInput, the first field of the time picker is auto-focused whenever the input is clicked so I'm unable to type in the input field.
Demo: https://jsfiddle.net/rtarso4f/

flatpickr version used: 4.4.4
Browser name and version: Google Chrome 65.0.3325.181
OS and version: macOS 10.13.4

What's up with this issue?
I was looking for ways to disable the timepicker from selecting a time automatically when the timepicker is opened.
For calendar and datepicker, no date will be set until user click on one of the date within the picker.
Perhaps an option to set defaultHour and defaultMinute not null and not have anything set when timepicker is displayed?


Update documentation Windows support

In README.rst, there is:
Windows
=======

Windows support is on going and only targets Python 3.5+ with Visual Studio 2017.

Should something be written about what has been done with clang-cl.exe ?

will do ;-)


[Talent V4beta1] Jobs Search and List not giving the same results

Environment details

OS: windows
PHP version: 5.6.4
Package name and version: google/cloud 0.115

Steps to reproduce

batch create jobs
list jobs => 150 results (can change)
empty search on jobs => 140 results (can change)

Code example
$jobServiceClient = new JobServiceClient();
$pagedResponse = $jobServiceClient->searchJobs($formattedParent, $requestMetadata);
            foreach ($pagedResponse->getIterator() as $element) {
                $ids[] = $element->getJob()->getRequisitionId();
            }
I've tried looking for patterns in the jobs that disappear but
a) they are not always the same (though some are disappearing more often)
b) I can't find anything significant anyway, I especially looked at publish time, expire time and visibility, with no success.
it seems like the number of jobs disappearing go up as times goes by but it can go down as well, again, no real patterns.
I would give more but I'm not allowed to publish the data I'm using.
I'm trying to know if this is a known behaviour or if there are things I could look at to solve this problem.
Thanks!

Hi @Harmeko,
Sorry to hear you're having trouble. Unfortunately, I'm not sure that this is the right place to get assistance. We are well-equipped to help solve problems with the client libraries, but it looks like your question is more related to the service itself. For help with that, check out the Support page for the Cloud Talent Solution product, or the google-cloud-talent-solution tag on Stackoverflow.
If I've misunderstood your issue and you believe it is indeed a problem in the client library, please re-open this issue and accept my apologies!
Best of luck.


Causes Chrome to render .txt extensions

Enabling the Skype Links 0.3 extension causes chrome to render code (including javascript - window.location) in files with .txt extension. I would have thought chrome wouldn't pass a .txt file to be rendered.
Disabling the extension returns viewing these files as plain text.
It would be nice to have both functionality.
David

Sorry for the tardy response. I'm unable to reproduce this. If you have a specific link I can look further. Modern versions of Chrome may behave better as well.


Payment percentage fee tax and payment fee amount creditmemo

The first time we credit a order we allready give the full amount back even if we don't credit the full amount. Is there a possibility to adjust this. Also if we do a second credit on the same order he will pay the payment fee an extra time even though we allready refunded the credit amount. This should not be happening i believe.

Hi @leonhelmus,
Thanks for letting us know. I see the issue it should work the same as shipping costs we will have a look how we can change this.
Regards,
Rik
Adyen

ah k let me know when this is fixed we want to implement it this week if possible

is it also possible to give payment fee or percentage back if the creditmemo is fully credited instead of if we make an creditmemo it will allways credit the payment fee or percentage on first creditmemo even though it wasnt fully credited.

Or that we can assign how much we want to refund in payment percentage / payment amount

Has there allready been made any changes to this ticket ? I would still like this functionality or bugfix. That would be very much appriciated

Hi @leonhelmus,
I am looking into it. I have it made available so you can edit the amount. To keep the already refunded payment fee amount subtracted if you are going to do a next partial refund this is not going to be easy and there is no generic way for this as it looks like.
Regards,
Rik
Adyen

Hi @leonhelmus,
The payment fee is now adjustable in the credit memo and support partial refunds.
Regards,
Rik


Refactor codebase to async/await - m365 pp aibuildermodel until m365 pp chatbot, Closes #4990

Refactoring of the commands from m365 pp aibuildermodel until m365 pp chatbot

m365 pp aibuildermodel get
m365 pp aibuildermodel list
m365 pp aibuildermodel remove
m365 pp card clone
m365 pp card get
m365 pp card list
m365 pp card remove
m365 pp chatbot get
m365 pp chatbot list
m365 pp chatbot remove

Closes #4990

Thank you for this enhancement! We'll try to review it ASAP!

ready to merge 🚀

merged manually. Thank you 👏


Add README, touch up notebook text

@RolfHut touched up some formatting, and made non-toc notebooks be included. Otherwise links to these notebooks won't show up.
Did you intend to remove rich prints from the pcrglob notebook? I find the print outputs a bit difficult to read now.

yes, I inteded that, because rich does not work with 'end = "/r"' as argument and therefore prints on new lines, which I did not want and couldn't get right in rich.


implement distribute_size for expand

I think we should somehow support too narrow constraints, and this can be optimized, by starting everyone off with the rounded down values.

Maybe I should add a comment saying that this is the https://en.wikipedia.org/wiki/Huntington–Hill_method


[bug] Clicking a device the second time, removes the discovery doc

Version
Build/Run method

[x] Docker
[ ] PKG
[ ] Manually built (git clone - npm install - npm run build )

Zwave2Mqtt version: 3.1.0
Openzwave Version: 1.6.1115
Describe the bug
Clicking a device twice, will remove the discovery doc from the UI. It seems every second click will show/hide the doc.. Hiding the doc also presents a validation error.
To Reproduce
Steps to reproduce the behavior:

Select a node
Select a device in the devices list
Select the same device again
Observe the discovery doc disappearing, and a validation error is shown

Expected behavior
Clicking a device multiple times will not remove its details.
Additional context
N/A

I just noticed the default of the page is no doc and an error. Maybe the second click is deselecting the device?

Yes that’s an expexted behaviour, the second click is deselecting it, like it is for nodes in nodes table

Daniel - Software Engineer
Support me at:
Github sponsors

On 15 May 2020, at 21:49, Michael Bisbjerg<EMAIL_ADDRESS>wrote:

I just noticed the default of the page is no doc and an error. Maybe the second click is deselecting the device?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Report subprocess error messages

Closes #50. I planned on doing this along with my first PR, but I forgot about it until now.
I appended the error message as-is to the MadoopError. This risks making the error messages a bit cluttered though, so any ideas for a cleaner format are welcome.

Thanks for the contribution!  A quick look seems like this is a reasonable approach and thanks for adding tests.  I'll take a closer look soon.

Pointed out a style choice and added your new messages to the tests. Otherwise, LGTM.


Incorrect address statuses in the address status index

There is a bug in the address status index, it's not clear yet how many addresses are impacted, and can result in clients not fully syncing their histories (which can result in them erroneously and unsuccessfully trying to spend already spent utxos).
For the address bSeWnSKTiQRtQ76jkz4pru2XXnw9wchmqb the history is correct but the precalculated status omits the last transaction when it's hashed, ie it's supposed to be
sha256('3c1b765b412703a9f02d0328886df43fe9516ecfea66e70be61658f79b48d70a:1142400:32ddebefc0620d020d91b0f9de5a93082b636905b5b934675e90dbd26b6f7141:1176287:'.encode()).hex()
'49050a8ffaf514cbae321f21e8ab9649da3743ccaeda102fb107fbde491f7971'

but it actually contains
sha256('3c1b765b412703a9f02d0328886df43fe9516ecfea66e70be61658f79b48d70a:1142400:'.encode()).hex()
'9ad3840619ee1ba015c29ce78cd7bb1b4c89d40c5c682bd2c646e476599003a9'

This needs to be reproduced in a test, the bug is probably in the startup / initial catchup code (or else it would be far more common).

on top of balance issues, this also causes channels or content to become unavialable.

example wth a local wallet I had - had to connect to 16 to get these txes:
2022-08-23 12:06:07,051 INFO     lbry.wallet.ledger:503: finished subscribing to 6 addresses on <IP_ADDRESS>:50001
2022-08-23 12:06:07,159 INFO     lbry.wallet.ledger:135: (lbc_mainnet) on_transaction: address=bU9RraZGaxqnWfKPTTCEYzzN69u5ty3RWp, height=1169309, is_verified=True, tx.id=8786ef5842e329c9049562bbc421ec845696f5913e80d8822ed401e705e4fefc
2022-08-23 12:06:07,159 INFO     lbry.wallet.ledger:560: Sync finished for address bU9RraZGaxqnWfKPTTCEYzzN69u5ty3RWp: 1/1
2022-08-23 12:06:07,224 INFO     lbry.wallet.ledger:135: (lbc_mainnet) on_transaction: address=bUemWWN5Rp6cMgm6wd7JL15noxZsbU9eFW, height=1176360, is_verified=True, tx.id=7d7b48d4e36b552e9e13da6e66e6e63d4b60909df6710203c154be92678d1cd1
2022-08-23 12:06:07,224 INFO     lbry.wallet.ledger:560: Sync finished for address bUemWWN5Rp6cMgm6wd7JL15noxZsbU9eFW: 1/1
2022-08-23 12:06:07,231 INFO     lbry.wallet.ledger:135: (lbc_mainnet) on_transaction: address=baS5DVbBLizs8omuRdpFGTRCHL5r7v3yPj, height=1169352, is_verified=True, tx.id=b271cd2ad690767b577a75b7ad305d3c4dd58da0943b35cd275a53a3c69f00ec
2022-08-23 12:06:07,231 INFO     lbry.wallet.ledger:560: Sync finished for address baS5DVbBLizs8omuRdpFGTRCHL5r7v3yPj: 1/1
2022-08-23 12:06:07,306 INFO     lbry.wallet.ledger:135: (lbc_mainnet) on_transaction: address=bK4ZEF9sy2s5rWgoYfR5ePTE4UvqKvSnZF, height=1169352, is_verified=True, tx.id=b271cd2ad690767b577a75b7ad305d3c4dd58da0943b35cd275a53a3c69f00ec
2022-08-23 12:06:07,307 INFO     lbry.wallet.ledger:135: (lbc_mainnet) on_transaction: address=bK4ZEF9sy2s5rWgoYfR5ePTE4UvqKvSnZF, height=1169309, is_verified=True, tx.id=8786ef5842e329c9049562bbc421ec845696f5913e80d8822ed401e705e4fefc
2022-08-23 12:06:07,307 INFO     lbry.wallet.ledger:560: Sync finished for address bK4ZEF9sy2s5rWgoYfR5ePTE4UvqKvSnZF: 2/2
2022-08-23 12:06:07,900 INFO     lbry.wallet.ledger:933: Loaded account bEApqqArHAgehLzdB8SoezUa4Nu2LnGDtD with 2.864672 LBC, 467 receiving addresses (gap: 20), 472 change addresses (gap: 6), 0 channels, 2 certificates and 433 claims.
2022-08-23 12:06:07,902 INFO     lbry.wallet.ledger:933: Loaded account bH9nRdgbrAFKZLLxKbW1wTCKwdoa1jWvHY with 0.0 LBC, 20 receiving addresses (gap: 20), 6 change addresses (gap: 6), 0 channels, 0 certificates and 0 claims.
2022-08-23 12:06:07,907 INFO     lbry.wallet.ledger:933: Loaded account bHbKbPFVUpq2aqhkFBnqzrMNJqXyhKHTUU with 0.65781 LBC, 140 receiving addresses (gap: 20), 119 change addresses (gap: 6), 8 channels, 9 certificates and 108 claims.


FSDP tests fail due to ImportError _state_dict_type_to_config  (has been moved inside of set_state_dict_type method

System Info
- `Accelerate` version: 0.15.0.dev0
- Platform: Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.31
- Python version: 3.9.13
- Numpy version: 1.23.4
- PyTorch version (GPU?): 1.14.0.dev20221107 (True)
- `Accelerate` default config:
        Not found

Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[X] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[ ] My own task or dataset (give details below)

Reproduction
pytest tests/fsdp/test_fsdp.py
results in
FAILED tests/fsdp/test_fsdp.py::FSDPPluginIntegration::test_auto_wrap_policy - ImportError: cannot import name '_state_dict_type_to_config' from 'torch.distributed.fsdp.fully_sharded_data_parall...
FAILED tests/fsdp/test_fsdp.py::FSDPPluginIntegration::test_backward_prefetch - ImportError: cannot import name '_state_dict_type_to_config' from 'torch.distributed.fsdp.fully_sharded_data_parall...
FAILED tests/fsdp/test_fsdp.py::FSDPPluginIntegration::test_cpu_offload - ImportError: cannot import name '_state_dict_type_to_config' from 'torch.distributed.fsdp.fully_sharded_data_parall...
FAILED tests/fsdp/test_fsdp.py::FSDPPluginIntegration::test_mixed_precision - ImportError: cannot import name '_state_dict_type_to_config' from 'torch.distributed.fsdp.fully_sharded_data_parall...
FAILED tests/fsdp/test_fsdp.py::FSDPPluginIntegration::test_sharding_strategy - ImportError: cannot import name '_state_dict_type_to_config' from 'torch.distributed.fsdp.fully_sharded_data_parall...
FAILED tests/fsdp/test_fsdp.py::FSDPPluginIntegration::test_state_dict_type - ImportError: cannot import name '_state_dict_type_to_config' from 'torch.distributed.fsdp.fully_sharded_data_parall...

Expected behavior
The tests to pass.

Note that the fail is due to a change in recent pytorch, the variable _state_dict_type_to_config has been moved into the static method set_state_dict_type , so isn't available for direct import from fully_sharded_data_parallel.py

cc @pacman100

Hello @Thomas-MMJ, Thank you for bringing this to the notice. Please use PyTorch 1.13.0 stable version for time-being. We will resolve all the issue once PyTorch version 1.14.0 has a stable relelase.


changing deployment_name references to project/venue

Purpose

The management console sets deployment_name as 'default', which is causing name collisions/errors deploying parts of the apigateway. This PR changes those deployment_name references to the (more) unique project/venue pairs we use elsewhere.

Proposed Changes

CHANGE ${deployment_name} references to ${project}-${venue}

Issues

no relevant issues, ran into this while deploying a personal management console

errors prompting this change:
Error: creating CloudWatch Logs Log Group (/aws/lambda/default-unity-apigateway-unity-cs-common-lambda-authorizer): operation error CloudWatch Logs: CreateLogGroup, https response error StatusCode: 400, RequestID: 83e27bc9-31e5-45cf-81d5-ff89c6d91a1d, ResourceAlreadyExistsException: The specified log group already exists

  with module.default-unity-apigateway-gOnqt.aws_cloudwatch_log_group.cs_common_lambda_auth_log_group,
  on .terraform/modules/default-unity-apigateway-gOnqt/main.tf line 50, in resource "aws_cloudwatch_log_group" "cs_common_lambda_auth_log_group":
  50: resource "aws_cloudwatch_log_group" "cs_common_lambda_auth_log_group" {

Error: creating IAM Role (default-unity-apigateway-iam_for_lambda_auth): operation error IAM: CreateRole, https response error StatusCode: 409, RequestID: 3168368d-9c64-4eb3-828d-67e64dd30cb9, EntityAlreadyExists: Role with name default-unity-apigateway-iam_for_lambda_auth already exists.

  with module.default-unity-apigateway-gOnqt.aws_iam_role.iam_for_lambda_auth,
  on .terraform/modules/default-unity-apigateway-gOnqt/main.tf line 116, in resource "aws_iam_role" "iam_for_lambda_auth":
 116: resource "aws_iam_role" "iam_for_lambda_auth" {


README update

The JSON object in the README.md isn't JSON formatted correctly. The usages for commandline POST could be useful.  @BondAnthony Can we make this a Hacktoberfest issue? #

Absolutely! Please feel to contribute in any manner if that be documentations or adding new features. Create the issues and I'll add some labels.


Is it possible to add quota per device ?

Hello
I'm looking to limit my kids data quota per day on my WIFI router
Is evillimiter providing that ?

Good idea, I will write that down for future versions of EvilLimiter.


Shader on a tilemap stops editing

Godot version
3.3 stable
System information
linux mint
Issue description
I hope I'm not being silly here, but I add a shader to a tilemap material and on doing so editing is disabled.
With shader: cannot edit (no grid)

With no shader: can edit

If this is a bug, is there a workaround other than clearing my shader whenever I want to edit my tilemap?
Steps to reproduce

add a shader
try to edit

Minimal reproduction project
No response

Duplicate of https://github.com/godotengine/godot/issues/32002. As a workaround, save the shader to an external .shader file and edit it using an external text editor.

Thanks. I did search for 'tilemap shader' and other variants :)
btw, why has this been pushed to 4.0, is it just to make use of the new tilemap system? People may not want to use 4.0 for a long time.


Mites & The Warden

I'm not sure if this is an intentional feature or not, but I've noticed mites will attack and bury into Wardens. I find this a bit odd as I was under the impression that they'd be on the same team, so to speak.

There are currently no plans to make the warden and the sculk horde not fight each other. This would require modifying vanilla code, which I am not certain I want to do.


MailTransport remote code execution

I think the MailTransport is vulnerable to this issue.
In SwiftMailer, the parameter passed to setFrom() is sprintf'd into -f%s. If there is attacker-controlled setFrom() calls (e.g. hosted remailer or hosted SaaS product), then by setting the email address to a parameter containing -X the shell parameter injection causes remote code execution.
As far as i can see, calls to setFrom() are not validated for correct RFC email format. It's a little difficult to say for sure, though, via five layers of indirection and interfaces.
Prior disclosure of this issue:

http://securitysucks.info/exploit-phps-mail-to-get-remote-code-execution/
http://www.reddit.com/r/netsec/comments/2fc0zy/exploit_phps_mail_to_get_remote_code_execution_cc/

Reverted in #829. It was wrong as mail() in PHP already escapes this argument.


Maximum a posteriori inference

I am considering using pyro for my problem, but I don't quite grok the library yet.  Part of my confusion comes from SVI being the main well documented inference engine in pyro, and I am a beginner to variational inference.
For me, the simplest "inference" methods to understand are maximum a posteriori (MAP) and maximum likelihood, and it would be nice if the pyro had some examples of these. I assume pyro has the optimization tools necessary to handle MAP, but the documentation is not very clear about how to do it.

Great question!  You can do MAP inference by using SVI with Delta distributions in your guide:
def model():
    mu = pyro.sample("mu", dist.normal, Variable(torch.zeros(1)), Variable(torch.ones(1)))
    return pyro.sample("x", dist.normal, mu, Variable(torch.ones(1)))

def guide():
    mu_param = pyro.param("mu_param", Variable(torch.randn(1), requires_grad=True))
    return pyro.sample("mu", dist.delta, mu_param)

For maximum likelihood, you can either put very wide Uniform priors on your latent variables in the model and proceed with deltas in the guide as above, or you can change the sample statements in your model into param statements and use an empty guide:
def model():
    mu = pyro.param("mu", Variable(torch.randn(1), requires_grad=True)))
    return pyro.sample("x", dist.normal, mu, Variable(torch.ones(1)))

def guide():
    return None

Thanks so much for that description. I am still a little confused about how variational inference with delta functions as the approximating distribution reduces to MAP.  This is something that edward also glosses over in its MAP documentation. I guess the ELBO equals the log of joint probability in the case when q(z) is a delta function. Would it be worth adding an inference class for this special case?

:+1: for a helper along the lines of
def MAP(model, *args, **kwargs):
    guide = poutine.lower(model)  # poutine.lower is opposite of poutine.lift
    return SVI(model, guide, loss="ELBO", *args, **kwargs)

I guess the ELBO equals the log of joint probability in the case when q(z) is a delta function

Yep, this is correct.

Would it be worth adding an inference class for this special case?

We could do something like what @fritzo suggested but I'm partial to the manual Delta guide method because it doesn't require executing the model twice.

At least for beginners, I think it would be nice to have a MAP convenience function which someone who knows nothing about variational inference can use.

Great use case of poutine.lower!

@nbren12 I believe #573 should work for you. The purpose of SVI and guides (over simple MAP) is to be able to deal with latent variables whose value you want to integrate over, rather than optimize. The purpose of the guide is to provide a good random guess for each latent variable.

@eb8680

In your MAP and MLE examples, where does the data come into play? In particular, you didn't 'observe' anything. If data is not involved, how can pyro learn the values?
Pyro documentation insists that the guide 'samples' the same latents as the model. However,  this is not the case in any of your examples. Is there a bug in the examples?
It would be great if you could provide an actual minimal running example -- e.g. learn the mean of an actual dataset of 100 numbers sampled from a Gaussian distribution.

Please help - I'm also trying to learn pyro starting from these basic cases and am finding it very difficult -- which is a shame, there seems to be great potential here!

@kjchalup This should MAP estimate the mean mu_est of numbers packed into data:
def model(data):
    pyro.sample('mu', normal, Variable(torch.zeros(1)), Variable(torch.ones(1)))
    for i, x in enumerate(data):
        pyro.sample('obs_{}'.format(i), normal, mu, Variable(torch.ones(1)), obs=x)

def guide(data):
    mu_map = pyro.param('mu_map', Variable(torch.randn(1), requires_grad=True))
    pyro.sample('mu', delta, mu_map)

@kjchalup are your questions specifically about MAP, or about Pyro more generally?  If it's the latter, I recommend our examples and tutorials, especially the introductory inference tutorial and the stochastic variational inference (SVI) tutorial.  If the former, here is a more complete version of my snippet above:
def model():
    mu = pyro.sample("mu", dist.normal, Variable(torch.zeros(1)), Variable(torch.ones(1)))
    return pyro.sample("x", dist.normal, mu, Variable(torch.ones(1)))

def guide():
    mu_param = pyro.param("mu_param", Variable(torch.randn(1), requires_grad=True))
    return pyro.sample("mu", dist.delta, mu_param)

# constrain model sample site "x" to be equal to some fixed observed data
conditioned_model = pyro.condition(model, data={"x": data})

svi = pyro.infer.SVI(model=conditioned_model,
                     guide=guide,
                     optim=pyro.optim.SGD({"lr": 0.001}),
                     loss="ELBO")

losses = []
for t in range(1000):
    losses.append(svi.step())

# to get the value of mu, just call guide
mu_final = guide()


[Win] RH IP address can not be defined after import management or later

sudo subutai management_network or sudo subutai info ipaddrcommand returns null:
it can happen at installation time or later
if happens at installation time - installation fails
but it can happen later, so we need to find a reason

GUI Network
Adapter #1

Adapter #2

Adapter #3

Ifconfig :
Part #1

Part#2

sudo subutai management_network detect command output :

Windows/CMD - ipconfig/all  :
Part #1

Part #2

Part #3

Part #4

one of the reasons - second default route
it should be deleted with command under Admin account
route delete <IP_ADDRESS> mask <IP_ADDRESS> <x.x.x.x>

How to reproduce - turn off internet, ifconfig down does not result to "null" in sudo subutai info ipaddr

In process of testing


backups: check for file existance before uploading (bsc#1025309)

@rsalevsky  @skazi0 needs a re-review for the added tests :D


Use the latest configured FC in the Upcoming tab

What does this PR do and why?
This PR addresses confusion around funding cycle reconfigurations and the Upcoming funding cycle tab.
Currently, we only show the queued funding cycle. If a user reconfigures, this reconfiguration doesn't appear in the Upcoming tab.
This PR improves the Upcoming funding cycle tab in the following ways:

uses latestConfiguredOf to get the latest configured funding cycle.
If the latest configured FC is equal to the current FC (i.e., theres been no reconfiguration), we use the queued funding cycle for the upcoming tab
If the latest configured FC is different to the current FC (i.e. there's been a reconfiguration), we use the latest configured funding cycle for the upcoming tab. We also show the ballot state (Pending or Approved) as a badge in the card header.

Closes https://github.com/jbx-protocol/juice-interface/issues/1106
Screenshots or screen recordings

status
dark
light

pending

failed

With Tooltip

Acceptance checklist

[x] I have evaluated the Approval Guidelines for this PR.
[x] I have tested this PR in all supported browsers.
[x] I have tested this PR in dark mode and light mode (if applicable).

ping @johnnyd-eth

Eggsellent work @aeolianeth. Code looks great and is working perfectly for me - awesome work with the ballot states.
My only thought is, since we've now added this check for upcoming FC being different to the current, if it's worth adding something about that in the UI? Perhaps as a risk indicator next to the 'Upcoming' tab title.

@aeolianeth this should take into account splits reconfiguration too, as they are/might be changed in reconf (with the same ballot trick etc), sorry for being late to the party':)
Cheat code:


Solve annotation problem of decorator

This PR adds typing stuff to make decorated class "understandable" by type checkers.
The screenshot below is how test code looks like in my vscode.

LGTM!!


memory

hallow，could you please tell me how much memory of cuda i should to prepare training

Hi, for training 256x256 videos with 16 frames, we were able to use batch size=3 on a single A100 80G gpu. It takes about 71G memory during training for this setting.


DOC: Add GitHub Repository Stats to README.md

What's wrong with the existing documentation
Description:
The issue involves adding a table to display repository statistics, such as the number of stars, forks, issues, and pull requests, directly in the README.md file. This will help contributors and users track the activity and contributions in the repository at a glance.
Steps to Implement:

Create a table in the README.md file displaying the following repository statistics:

Stars
Forks
Open Issues
Open Pull Requests
Closed Pull Requests

Use markdown syntax to include badges or icons for each statistic, along with dynamic links to the corresponding sections of the repository.

Expected Behavior:
The README.md file will contain a neatly formatted table that displays real-time repository statistics, making the repository's activity more visible and engaging for contributors and users.
Additional Information:

This enhancement will make the repository more user-friendly by providing visibility into its overall activity.
The table can be dynamically updated by using GitHub shields or other badge services to fetch real-time stats.

Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue

@Ganeshmoorthii Any updates?

Dear #andoriyaprashant ,
I hope this message finds you well. I kindly request you to review and merge my pull request at your earliest convenience. Additionally, if the contribution meets your standards, please provide the hacktoberfest-accepted label.
Thank you for your time and consideration.
Best regards,
#Ganeshmoorthi


Add number-of-awakenings modifiers

Add number-of-awakenings modifiers
yay max wanted to use it last night so now it's on our roadmap
e.g. 7-sb for Valeria
Question 1: Should Valeria ALSO have 6-sb and 5-sb and 4-sb etc? (imo yes)
Super awakenings
Gorkaos should have 4-sb as well as 6-sa-sb
Question 2: Should Gorkaos also have 5-sa-sb? (imo yes) Question 3: what about 4-sa-sb, 3-sa-sb, etc? imo no.
"Plus" versions of awakenings (te++, sb++, unbindable++)
Question 4: Option 1 or option 2? (imo 2)
Option 1
Literally don't bother to consider them at all, they are JUST two of the normal version.

2-unbindable (.......i guess...........)
2-sb
2-te

Option 2
Both count them as 2x of the normal version as above, AND ALSO count them as one of the bigger version. (For unbindable this would mean you have 1-unbindable and also 2-unbindable unless we special-case-out that because of how we nicknamed it.)

We will possibly strim this yay

We will possibly strim this yay


Refactor session feedback layout

Reduce over 100 rows of activity_session_feedback.xml.

@hotchemi Refactored. Could you review it?

Nice!


Huge data-set On GPU

is there a way to run your code on GPU? because my data set is over 1000000 images and takes forever to run on CPU

@nnowaker The code is default run on GPU if available. Maybe you didn't configure keras and tensorflow correctly? Make sure you have installed tensorflow by pip install tensorflow-gpu instead of pip install tensorflow.

thanks a lot problem solved , Now each epoch is only 3 secs insted of 35 secs !


ARROW-1375: [C++] Remove dependency on msvc version for Snappy build

it seems the version number of Visual Studio 2017 increased on Appveyor to the latest  and this caused Snappy build failure.


CONTRIBUTING.md

It may be worth adding a CONTRIBUTING.md file to the repo, even if it's just with a few loose guidelines on how to prepare to put in a pull request.
I know that a barrier to contributing for some devs is not being 100% versed in the language/app, and despite knowing how to fix a bug, they avoid it because they're worried about not doing it to the right standard, etc. Getting a few points down might help encourage people to chip in if they know what the minimum requirements are.
Added bonus is Github recognises the file and highlights it to potential collaborators.  https://github.com/blog/1184-contributing-guidelines
Is it worth me putting together a working draft, and you guys can pick it apart till we get something everyone's happy with?

@chrismou :+1:, let me know what you come up with,

@chrismou Absolutely, that would be wonderful!

Sounds like a great idea.

@chrismou Any update on this draft?

@jadengore Been looking into this https://github.com/radiant-player/radiant-player-mac/issues/388#issuecomment-151875245 -- I'll aim to get a draft up in the next day or so

First draft is up: https://github.com/radiant-player/radiant-player-mac/pull/401
I'll close this issue now - feel free to post any feedback you have on the PR :-)
@jadengore @jacobwgillespie @kbhomes @motokochan @benburton


Need to support validating arrays of objects

Support validating items in an array as well as more complex objects that are array properties.

putting this in the deep object graph validation bucket: https://github.com/aurelia/validation/issues/283


React hooks dependencies and react-easy-state

Hi,
currently when you use global state values from react-easy-state in hook dependencies, it produces warning.
import React, { useEffect } from "react";
import { store, view } from "react-easy-state";

const state = store({
  value: 1
})

const Component = view(() => {
  useEffect(() => {
    console.log(state.value)
  }, [state.value]);

  return
{state.value}

})

React Hook useEffect has unnecessary dependencies: 'state.value'. Either exclude them or remove the dependency array. Outer scope values like 'state.value' aren't valid dependencies because mutating them doesn't re-render the component  react-hooks/exhaustive-deps
I think this is good time to reopen some ticket that I read earlier about introducing hooks api to solve this problem and have more unified interface.
eg instead of
const Component = view(() => {
  return
{state.value}

})

to have
const Component = () => {
  const {value} = useStore(state);
  return
{value}

}

Looks like a valid concern, I will look into it in the afternoon.

I think this is only an issue for useEffect. Please check the new autoEffect function for this use case (in the newly released @risingstack/react-easy-state v6.2.0).

What about dependencies for useCallback/useMemo?
The base problem is, that react-easy-state is always mutating the state object. This means, all referential equality checks will match (even if data changed). This become a problem as soon as you need to call into any code that is not built around react-easy-state

Referential equality checks will match but it won't cause an issue. Callbacks won't have to be recreated since they will see the latest data without recreation (because the data is stored in mutable objects instead of closures). I really don't like hooks, I think they confuse even the smartest programmers.
We will take a pretty bold approach regarding hooks and try to get rid of their reliance on closures instead of doing a traditional integration. If everything goes right a new blog post will be released tomorrow here that explains the broad idea and it will be followed by an 'Idiomatic React Easy State' docs that explain the edge cases soon.
The current hook integration situation is a mess but I think we can clean it up with zero API changes and some new patterns and docs.

Wow this sounds really interesting! Looking forward to the blog post!
One very simple example would be a library component, that implements pure component. Passend state objects will not trigger a rerender k side of this component, because the identity didn't change. Same with a library component that uses useEffect. The effect will not rerun, when the state object is mutated.

Passend state objects will not trigger a rerender k side of this component, because the identity didn't change.

I would argue with this. This is why we suggest people to wrap ALL components with view - even pure ones. If you wrap pure components they will re-render when the passed down store parts get mutated (they don't have to change by reference). The same stands for useEffect (if I understood correctly).
There are pretty bad known edge cases though which have to be addressed. You are right about that.

I think I finally got what you meant (:
I guess you mean the case when you don't have control over the mentioned pure component and can not wrap it with view (it's a third party component). There is a "Passing nested data to third party components." part in the docs that deals with this case, it is pretty rough though. We could make a more sophisticated helper for this in the future.

Thanks a lot for your patient responses! I really appreciate it.

No worries, you are raising very valid issues.


[BUG] PQ implementation collapses all subspaces

It's possible that I'm missing something due to unfamiliarity with the codebase, but it looks to me like vpq_dataset and train_pq are collapsing all the subspaces into a single codebook.  E.g. if you have pq_n_centers=256, pq_len=8, and dim=128, a classic PQ implementation would result in a codebook of 256*8*32, with centers for each 8-dim subspace trained separately.  Instead, cuvs is training just 256*8 which probably works okay for extremely symmetrical datasets but will diminish accuracy unnecessarily with others.

You're right. vpq_dataset and train_pq are parts of the CAGRA implementation. In contrast to IVF-PQ, the implementation of compression in CAGRA uses just one codebook for all subspaces, by design.
As far as I understand, this greatly reduces the GPU shared memory / cache requirements and, through that, improves the performance. The product quantization in vpq_dataset is applied on the residuals of the vector quantization (k-means cluster centers); I assume the drop in recall is partially mitigated by increasing the number of vectors (parameter vq_n_centers).

Thanks!  I had not noticed that cagra_q_dataset_descriptor_t::set_smem_ptr assumes that the entire codebook needs to fit into shared memory.  Given that constraint, the "collapsing" here makes sense.


Publish to respective package managers

npm run package builds distribution artifacts for NPM and PyPI, so we should add some scripting that publishes those build artifacts.
Open questions:

What account would be used for publishing to each package manager?
When/how would publishing happen?

Seems like having it run as part of CI would be preferable
Only publish if the version number has changed?

This seems interesting: https://github.com/marketplace/actions/jsii-publish

Currently working on this in 6/publish-packages.
My dev notes so far:
Versioning

Ideas:

Each package should be versioned separately.  No need to up the version (and thus have consumers upgrade) when the package hasn't changed.
Can we have CI manage versions?  Or should this be a required part of manual edits made to each component?

It seems that Lerna has tooling to support managing versions via lerna version

If we use the conventional commit flag which includes logic to determine the correct version number based on the commit message (read more)

Questions:

In a world where we have multiple versions of packages alongside one another, does that exclude of from registering releases via a Github Release? I think so.

Suggestions:

To keep us moving, we may want to go with manual control over versioning right now and approach auto-versioning via CI in the future.

Publish

Notes:

Lerna supports publishing via lerna publish. I'm not sure how well this plays with the distributables created by jsii-pacmak.  Being that we will want to be publishing to multiple package managers, I'm leaning towards the idea that we will want to use a custom script that iterates over the directories in /packages and publishes each to the appropriate package managers.
While https://github.com/marketplace/actions/jsii-publish is cool, it is built to handle repos that represent a single construct.  To handle publishing multiple constructs, we will likely want to use that action as an example but will need to build our own logic.
In an ideal world, we'll publish packages via CI and only those packages have changed (using Lerna to manage versions can help with this, I believe) when the code has been pushed to main.  Someday, it may even be possible to use commit messages to instruct CI to publish alpha pre-release versions for testing.


Problems with S2_L1C Images

Hi!
I'm experiencing  some troubles trying to run the script with a S2_L1C image, particularly this one: https://scihub.copernicus.eu/dhus/odata/v1/Products('ea4dfeee-1d28-47c5-a6a8-c86ada64768f')/$value
It seems to be some sorts of recursive problem, not sure exactly what.
This is the console output:

(base) Macbook-Pro-2020-de-*******:WaterDetect ******$ python runWaterDetect.py  -i ./Input/ -o ./Output/ -s ../../../Downloads/LaJara/LaJara.shp --product S2_L1C
Loading configuration file WaterDetect.ini
File WaterDetect.ini verified.
Folder ./Input/ verified.
File ../../../Downloads/LaJara/LaJara.shp verified.
Folder ./Output/ verified.
Starting WaterDetection version: 1.5.10
Opening image in folder Input/S2A_MSIL1C_20210207T174511_N0209_R098_T13SDA_20210207T212818.SAFE
Retrieving bands for product S2_L1C
****** WARNING ********
[Errno 2] No such file or directory: 'Input/S2A_MSIL1C_20210207T174511_N0209_R098_T13SDA_20210207T212818.SAFE/GRANULE/Input/S2A_MSIL1C_20210207T174511_N0209_R098_T13SDA_20210207T212818.SAFE/GRANULE/L1C_T13SDA_A029410_20210207T175338/IMG_DATA'. Skipping to the next image

I was not able to find any other image source for that particular place.
I'm on MacOS Big Sur, and the latest version of the script.

Hi @fede1608 ,
I did a minor correction on the InputOutput.py, based on the comments of another user that had a similar issue as you had. As I am using windows and this problem don't occur to me, I cannot test it correctly at the moment.
If you can clone the project and test it again... I will upload a new version to PyPI if it works.
For your image, here is an example of the output. Be aware that, as you are working with L1C images, there is no layer with pre-defined masks, so you need to take care of the snow pixels beforehand (passing an external mask) or change the .ini to try to clip these values according to the reflectance on the visible.

Regards,
Mauricio

Thanks! It worked fine!


Nominate fossedihelm for sig-compute approver

What this PR does
Nominating @fossedihelm for SIG compute approver
Federico has been contributing to Kubevirt for 3 years, dating first contribution back to https://github.com/kubevirt/kubevirt/pull/6681 5.11.2021
He contributed  200+ PRs https://github.com/kubevirt/kubevirt/pulls?q=is%3Aclosed+is%3Apr+author%3Afossedihelm+, raging from big features(KSM, free pages reporting, Improved IO errors handling, generated clients) to small iterative improvements of the code base, bug fixes (such as memory leaks in our c-bindings) or tests improvements(Fake client, speedups of unit tests, conformance tests).
He reviewed almost 600 PRs https://github.com/kubevirt/kubevirt/pulls?q=is%3Apr+commenter%3Afossedihelm+is%3Aclosed+-author%3Afossedihelm+ . Most notably PRs around hotplug, volume migration, rollout, console logging and many more.
As shown above the numbers are impressive on itself and when you go through the contributions and reviews you will see that Federico has touched huge part of the code base that requires substantiation knowledge. I believe this nomination will bring mutual benefits, both to project and Federico.

Fixes #
Special notes for your reviewer

Release note

NONE

/approve
Very much deserved, @fossedihelm thank you for all your contributions so far!

@xpivarc, nit: please use is:merged in the github query instead of is:closed, since it's more accurate.

I definitely support this.
@fossedihelm's contributions are significant and valuable. He showed great talent in both implementation and reviews. Thank you for all of your contributions!!
/lgtm
As usual, let's hold this for some time to let others chime in.
/hold

/override pull-kubevirt-e2e-k8s-1.29-sig-compute

/override pull-kubevirt-fuzz

/approve

/approve

Thanks @xpivarc! I really like the way the nomination PR is written (with supporting data and details)
Good luck @fossedihelm !

/cc @davidvossel @rmohr @stu-gott @fabiand @mhenriks @vasiliy-ul

@dhiller off-topic: I remember having some rules defined to situations where we can skip running the test lanes. I think this is one of them :)

The PR history shows less than 1 minute execution time for all the lanes: https://prow.ci.kubevirt.io/pr-history/?org=kubevirt&repo=kubevirt&pr=12909
Note that this is based on the poor mans skip we implemented here 1, so they still show up as being run.

/lgtm

/lgtm

/lgtm

/hold cancel
A week has passed and we got almost all to explicitly agree. In case somebody didn't have time to comment, please contact me.


Move return statement of of if statement

https://github.com/yahya-abdul-majeed/CodeRefactoring2023/blob/13fd912b300904b6ff8bfb51da2be73ad1126bca/CLabManager-Web/Areas/Admin/Controllers/UserRoleController.cs#L49-L59

Why is this code bad? What principles does it violate? What can the existence of such code lead to? 0,3


Update Makefile for docker-compose.yaml

File is mistakenly pointing at docker-compose.yml instead of the file that exists with an a in the file extension.

LGTM


Moved creating UserInfoPopup in one place.

This PR fixes this issue: https://github.com/fourtf/chatterino2/issues/759
And a bit refactors code.

Works great 👍


bug: axios V1.0.0 support

Describe the bug
A few types are broken when using with axios v1.0.0 due to the being no overlap between axios and axios cache instance.
To Reproduce
Bump axios to v1.0.0 and use a axios instance as a cache interceptor type.
Expected behavior
Be able to compile the ts without any error.
Additional context

Axios: 1.0.0
Axios Cache Interceptor: latest
Error thrown:

Error: src/functions/NitrApi.ts(25,21): error TS2352: Conversion of type 'AxiosInstance' to type 'AxiosCacheInstance' may be a mistake because neither type sufficiently overlaps with the other. If this was intentional, convert the expression to 'unknown' first.
123
  Type 'AxiosInstance' is missing the following properties from type 'AxiosCacheInstance': storage, generateKey, waiting, headerInterpreter, and 3 more.
124
Error: src/functions/NitrApi.ts(35,33): error TS2345: Argument of type 'AxiosCacheInstance' is not assignable to parameter of type 'AxiosInstance'.
125
  Types of property 'defaults' are incompatible.
126
    Type 'AxiosDefaults<any> & { cache: CacheProperties<unknown, unknown>; }' is not assignable to type 'Omit<AxiosDefaults<any>, "headers"> & { headers: HeadersDefaults & { [key: string]: AxiosHeaderValue; }; }'.
127
      Type 'AxiosDefaults<any> & { cache: CacheProperties<unknown, unknown>; }' is not assignable to type '{ headers: HeadersDefaults & { [key: string]: AxiosHeaderValue; }; }'.
128
        Types of property 'headers' are incompatible.
129
          Type 'HeadersDefaults' is not assignable to type 'HeadersDefaults & { [key: string]: AxiosHeaderValue; }'.
130
            Type 'HeadersDefaults' is not assignable to type '{ [key: string]: AxiosHeaderValue; }'.
131
              Index signature for type 'string' is missing in type 'HeadersDefaults'.

See #365.

@arthurfiorette  If you have a task list that you need to work through on the the pr id be happy to have a look to help contribute to push towards support for axios v1.0.0

Migrated everything (and created a task list) at #368. Thanks!


Kirby, Kirbyup & Tailwind class order issue

I'm not sure this necessarily belongs here, but I've just spent an hour debugging it, and I thought it should be noted somewhere, and since kirbyup + tailwind is one of the official examples, I thought you might want to include a warning or a note:
If you're using kirbyup with tailwind in two separate plugins, and the latter uses the same class as the one loaded earlier, it might happen that your responsive tailwind classes will stop working, since kirby just smushes the generated CSS together into one CSS file.
Example:
/* first-plugin/index.css */
.p-2 {}
.lg\:p-4 {}

/* second-plugin/index.css */
.p-2 {}

Now the element with p-2 lg:p4 from first plugin will stay at p-2, since the second plugin redefines the p-2 class much later in the resulting index.css file.
Solutions:

either configure the first plugin to use important by default, which is stupid
prefix the latter (second) plugin with a custom tailwind prefix, so the css classes don't colide

Good point for plugin devs. Looks like a nice addition to the README of the Tailwind example. Will add that.


chore: Focus search input box when widgets library is opened

Description
Auto-focus search input box when widgets library is open in explorer
Fixes #13929
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:white_circle: Total coverage has not changed
// Code coverage diff between base branch:release and head branch: chore/focus-widget-search-input

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
app/client/src/pages/Editor/WidgetSidebar.tsx
66.67 (3.03)
38.89 (22.22)
37.5 (8.93)
63.64 (2.35)

:red_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.71 (-0.23)
40.83 (-0.84)
36.21 (0)
56.74 (-0.25)

/ok-to-test sha=d43e3c9


Italian locale

Hi,
I've used the extension, Italian locale was missing, so I added it.

Thanks @nvignola !


Merge release-0.12 into master

This PR merges the release-0.12 branch into master.
cc @thanos-io/thanos-maintainers

Thanks!


RA2(steam)+cnc-ddraw+AMD RSR= pc reboot

I play RA2(steam) is ok(win10), play RA2(steam) +AMD RSR is ok, play RA2(steam) +cnc-ddraw is ok.
but if use cnc-ddraw+AMD RSR both, open RA2 pc will reboot. (3times, 100%)
So may I ask if these two things are then inherently incompatible in principle?
(use AMD RSR is to make game graphics from 1080P to 4k)
(PC other things all normol, as long as i don't turn on both features at the same time, the computer won't reboot automatically)

I haven't heard about any issues related to RSR yet. Maybe it just doesn't support Direct3D9? Did you try to change your renderer to opengl in cnc-ddraw config.exe?

I haven't heard about any issues related to RSR yet. Maybe it just doesn't support Direct3D9? Did you try to change your renderer to opengl in cnc-ddraw config.exe?

no, also reboot with select opengl.(just try 1 time,100%)
(i dont mean cnc-ddraw have problem, maybe AMD drivers RSR problemm, just for a feedback record)

Try if the same happens if you would disable RSR and enable supersampling instead. And if you have any "overlays" enabled, try to disable them too, before (re)trying RSR or trying supersampling.

Try if the same happens if you would disable RSR and enable supersampling instead. And if you have any "overlays" enabled, try to disable them too, before (re)trying RSR or trying supersampling.

i dont know what u said " supersampling".
i use AMD Radeon Image Sharpening is ok.
(RSR and RIS cant work together)

Not sure if it's useful for you, but I added support for FSR directly into cnc-ddraw. This way you may not need RSR anymore. Only problem is that the menus will not upscale though (in-game works fine)
Here is the latest build: cnc-ddraw.zip
BTW, do you have some other games installed that work with cnc-ddraw? I'm wondering if they are working with RSR? Maybe the problem is related to the RA2 menus

Not sure if it's useful for you, but I added support for FSR directly into cnc-ddraw (set shader to fsr.glsl, presentation to borderelss or fullscreen upscaled). This way you may not need RSR anymore. Only problem is that the menus will not upscale though (in-game works fine)
Here is the latest build: cnc-ddraw.zip
BTW, do you have some other games installed that work with cnc-ddraw? I'm wondering if they are working with RSR? Maybe the problem is related to the RA2 menus

nice! I will try in this weekend.
I have RA2、RA2Yuri's Revenge and  Zuma Deluxe, i will try Zuma Deluxe(steam).

new problem(maybe ago i met, but forgot solution ). Just back to steam if i open RA2 in my pc.
i try several renderer or shader, just back steam(desktop) again and again.
unless no cnc-ddraw.
i shoose dont play RA2, or dont modify ddraw in my pc,maybe it didnt match RA2(STEAM version)

Not sure if it's useful for you, but I added support for FSR directly into cnc-ddraw (set shader to fsr.glsl, presentation to borderelss or fullscreen upscaled). This way you may not need RSR anymore. Only problem is that the menus will not upscale though (in-game works fine)
Here is the latest build: cnc-ddraw.zip
BTW, do you have some other games installed that work with cnc-ddraw? I'm wondering if they are working with RSR? Maybe the problem is related to the RA2 menus

new problem(maybe oldproblem, but i forgot solution ). Just back to steam if i open RA2 in my pc(if use cnc-ddraw).
i try several renderer or shader, just back steam(desktop) again and again.
unless no cnc-ddraw.
i shoose dont play RA2, or dont modify ddraw in my pc,maybe it didnt match RA2(STEAM version)


LNURL withdraw link uses msat amount as default instead of dividing by 1000

Describe the bug
LNURL withdraw link uses msat amount as default instead of dividing by 1000.
To Reproduce
Try using an LNURLw link
Expected behavior
Default is sats
Information About Alby

Alby Version: 1.10.0
Alby installed through browser store
Wallet connected with LNbits

Screenshots (If Any)
Add screenshots to help explain your problem.
Device Information [optional]:
left blank
Additional context
This was an attempted withdrawal on stacker.news
Are you working on this issue? (Yes/No)
No

Could you provide a screenshot? I can't reproduce the problem.


Add tutorial

After some user testing I realized that editing accounts doesn't seem to be as easy to understand as I hoped it would be so I want to add a tutorial which guides the user through the first time. Maybe changing the UX would be a better idea but I'm still not sure if I know a better way to do this.

Move to later version, I'll add a basic tutorial to the FAQ for now.

Would be interesting to see if this is still needed.

Closed since it's probably not needed anymore


Trying to get in touch regarding a security issue

Hey there!
I'd like to report a security issue but cannot find contact instructions on your repository.
If not a hassle, might you kindly add a SECURITY.md file with an email, or another contact method? GitHub recommends this best practice to ensure security issues are responsibly disclosed, and it would serve as a simple instruction for security researchers in the future.
Thank you for your consideration, and I look forward to hearing from you!
(cc @huntr-helper)

Hi @JamieSlome as indicated in our readme as well as our contributing guidelines and documentation site, please report security issues at<EMAIL_ADDRESS>

See https://github.com/area17/twill#security-vulnerabilities, https://github.com/area17/twill/blob/2.x/CONTRIBUTING.md#security-vulnerabilities, and https://twill.io/docs/#security-vulnerabilities.

@JamieSlome a SECURITY.md file was added to the repository.


Make edited hv.Table values overwrite values in the source dataframe

Hi
I am trying to make efficient use of holoviews to edit some data. I had a look at one example in http://holoviews.org/reference/streams/bokeh/BoundsX.html#bokeh-gallery-boundsx
I did the changes marked in bold in order to show all the table rows (commenting the describe()..) for the corresponding points selected and added editable=True in the options builder. What I really want is to be able to edit the table values in the hv.Table and write these edits back to df with edits stored. I would be really interested in this ability to store information back through holoviews.
import pandas as pd
import numpy as np
import holoviews as hv
from holoviews import streams
hv.extension('bokeh')
n=200
xs = np.linspace(0, 1, n)
ys = np.cumsum(np.random.randn(n))
df = pd.DataFrame({'x': xs, 'y': ys})
curve = hv.Curve(df)
def make_from_boundsx(boundsx):
sub = df.set_index('x').loc[boundsx[0]:boundsx[1]]
return hv.Table(sub).opts(editable=True)  # .describe().reset_index().values, 'stat', 'value')
dmap = hv.DynamicMap(make_from_boundsx, streams=[streams.BoundsX(source=curve, boundsx=(0,0))])
curve.opts(tools=['xbox_select']) + dmap

This would break a lot of assumptions in HoloViews, if you need this you can use hv.streams.CDSStream in HoloViews to get the data or you will be able to use the DataFrame widget in Panel once it is merged (see https://github.com/pyviz/panel/pull/560).

Sorry, this was not ment as a pull request to change anything in the library. Rather I was interested in examples on how one might approach the type of challenge described. Thank you for the tip on using a CDSStream. I will be happy to see more examples on how to go about changing data through holoviews. I am thinking of things like interactively tagging time series data and write back as an attribute to the input data and simultaneously change coloring of time series plots. I might come back with examples if I manage some of this.

We should add an example like that to the gallery.


get_session different exception messages

If no active_session - get reason from database

!test

Refer to this link for build results (access rights to CI server needed):
http://jenkins.2gis.local/job/vmmaster.staging.regression/344/
Test PASSed.

!test

Refer to this link for build results (access rights to CI server needed):
http://jenkins.2gis.local/job/vmmaster.staging.regression/345/
Test FAILed.

!test

Refer to this link for build results (access rights to CI server needed):
http://jenkins.2gis.local/job/vmmaster.staging.regression/346/
Test FAILed.

!test

Refer to this link for build results (access rights to CI server needed):
http://jenkins.2gis.local/job/vmmaster.staging.regression/347/
Test PASSed.

!test

Refer to this link for build results (access rights to CI server needed):
http://jenkins.2gis.local/job/vmmaster.staging.regression/351/
Test PASSed.

@nwlunatic @sh0ked поправил замечания.

:+1:

:

!test

!test

Refer to this link for build results (access rights to CI server needed):
http://jenkins.2gis.local/job/vmmaster.staging.regression/362/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.2gis.local/job/vmmaster.staging.regression/363/
Test FAILed.

!test

Refer to this link for build results (access rights to CI server needed):
http://jenkins.2gis.local/job/vmmaster.staging.regression/364/
Test FAILed.

!test

Refer to this link for build results (access rights to CI server needed):
http://jenkins.2gis.local/job/vmmaster.staging.regression/365/
Test FAILed.

!test

Refer to this link for build results (access rights to CI server needed):
http://jenkins.2gis.local/job/vmmaster.staging.regression/366/
Test PASSed.


[autocompletion] at wrong place

I am reporting two bugs regarding autocompletion function in Jupyter when I was using SOS.
The first one, when the hinterland function is disabled in nbextension.
As shown in the following image

basically, when I tried to use the autocomplete, the first time it works fine (as shown in step 1 and 2). However, for the second time, the autocomplete inserted the code at the wrong place (as highlighted by the red rectangle)
The second is still related to autocompletion function, but this time, I turned on hinterland in nbextension. As shown in the following image, the autocomplete function adds an addition letter to the end of the code (instead of ggplot, the code becomes ggplotp)

I notice that you have python3 instead of Python3 in the language dropdown box. This means the sos-python language module has not been installed. This does not mean this bug report is invalid but the bug might manifest in a different way if sos-python is installed.
Will look further into this later.

I have just pushed a patch. Please let me know if the bug has been fixed.

Thanks for the patch. I tried it on my local machine, now the autocompletion works perfectly.


Encode struct fields with foo[bar]

Whilst I understand why foo.bar may be preferable, it would be useful to be able to configure the encoder to output struct fields as foo[bar] is this is the format that lots of other systems expect (e.g. ruby).
Happy to submit a PR if you can point me in the right direction.

Hi @danielchatfield
I definitely want to keep the default working as is, but am open to different output modes; I'll need a little more information though, like:

Does or how does ruby handle or need it formatted for nested structs?
How does ruby handle arrays and maps?

Any other information would be helpful :smile:

This should now be closed by #56


Ftr closable

I added a close() method to the OsmParser Trait and implemented it in the OsmXmlParser and the OsmPbfParser.
The OsmPbfParser will forward this call to it's DataInputStream.
And the OsmXmlParser will forward it to its Source that will forward it to its FileInputStream as can be seen in the source code: https://github.com/scala/scala/blob/v2.11.7/src/library/scala/io/Source.scala#L97
Demonstration. This code won't explode anymore:
~/geow$ sbt console
scala> import io.plasmap.parser._

scala>(0 to 4000).foreach( x => {val p = OsmParser("./duesseldorf-regbez-latest.osm.pbf"); p.close()} )

Thanks a million!


Unable to locate element

Errors:

Message: no such element: Unable to locate element: {"method":"xpath","selector":"//*[@id="idSIButton9"]"}
(Session info: chrome=108.0.5359.124)

Traceback (most recent call last):
File "/BingRewards/main.py", line 1145, in update_searches
PC = driver.find_element(By.XPATH, value='//[@id="userPointsBreakdown"]/div/div[2]/div/div[1]/div/div[2]/mee-rewards-user-points-details/div/div/div/div/p[2]').text.replace(" ", "").split("/")
File "/usr/local/lib/python3.9/dist-packages/selenium/webdriver/remote/webdriver.py", line 861, in find_element
return self.execute(Command.FIND_ELEMENT, {"using": by, "value": value})["value"]
File "/usr/local/lib/python3.9/dist-packages/selenium/webdriver/remote/webdriver.py", line 444, in execute
self.error_handler.check_response(response)
File "/usr/local/lib/python3.9/dist-packages/selenium/webdriver/remote/errorhandler.py", line 249, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.NoSuchElementException: Message: no such element: Unable to locate element: {"method":"xpath","selector":"//[@id="userPointsBreakdown"]/div/div[2]/div/div[1]/div/div[2]/mee-rewards-user-points-details/div/div/div/div/p[2]"}
(Session info: chrome=108.0.5359.124)

Message: no such element: Unable to locate element: {"method":"xpath","selector":"//*[@id="daily-sets"]/mee-card-group[1]/div/mee-card[1]/div/card-content/mee-rewards-daily-set-item-content/div/a/mee-rewards-points/div/div/span[1]"}
(Session info: chrome=108.0.5359.124)

These type of error occurs frequently, and some quizzes or Daily Sets can not be completed.

Any news on how to fix this?


WIP: Sneak peek into provisioning provider refactor

This is an early preview of refactoring required to support multiple provisioning providers.

Closing this in favor of a more stable integration path.  New PR coming soon.


Failing find_with_related with column_name attribute

Description
Defining a has_many relationship when the primary key is a column with a column_name breaks .find_with_related.
Steps to Reproduce
See MWE repository.
Expected Behavior
When I run the query, I expect to retrieve the elements from the database.
Actual Behavior
The query fails with the following message:
Error: Query Error: error returned from database: 1054 (42S22): Unknown column 'container.rust_id' in 'order clause'

Reproduces How Often
It reproduces every-time.
Versions
I am working with MySQL on a debian (not sure if it is relevant for this bug):
$ mysql --version
mysql  Ver 8.0.23-3+b1 for Linux on x86_64 ((Debian))

$ cargo tree | grep sea-
sea-orm-column-name-bug v0.1.0 (/home/bleuse/tmpfiles/work/sea-orm-column-name-bug)
├── sea-orm v0.7.1
│   ├── sea-orm-macros v0.7.0 (proc-macro)
│   ├── sea-query v0.23.0
│   │   ├── sea-query-derive v0.2.0 (proc-macro)
│   ├── sea-strum v0.23.0
│   │   └── sea-strum_macros v0.23.0 (proc-macro)

Additional Information
The query generated to handle a .find_with_related call does not respect the column_name attribute set in the Model struct: it uses the name derived from the struct field name.
The generated query is the following:
SELECT
  `container`.`db_id` AS `A_db_id`,
  `content`.`id` AS `B_id`,
  `content`.`container_id` AS `B_container_id`
FROM
  `container`
LEFT JOIN
  `content` ON `container`.`db_id` = `content`.`container_id`
ORDER BY
  `container`.`rust_id` ASC

The ORDER BY clause is wrongly generated as we should not get rust_id but db_id. The rest of the query is correctly generated.
See https://github.com/bleuse/sea-orm-column-name-bug for a MWE.

Hey @bleuse, thanks for catching this! I'm working on a patch for this


Add kube-slack chart

This PR adds an incubator chart for kube-slack.
I tried to incorporate best practices as far as possible. I did not enter any maintainers in the Chart.yaml yet. Should I enter myself there?
If all requirements are met, I can also move it to stable.
Thanks for considering this PR!
/cc @whs (author of kube-slack)

@foxish anything else I should do about this chart?

Would love to see this merged 👍

/hold

/assign

@jstriebel Are you able to make these changes, or can I pick them up?

Sorry for the late reply, I was unavailable the last week. @rawkode, I would add the changes the next days, but feel free to pick it up.

@unguiculus Thank you for the feedback, I changed the chart accordingly.
As I would still like to use this chart with kubernetes 1.7, I used semverCompare. The linter had a bug about the usage of , which is already fixed, only the testing needs to be upgraded to the newest helm version.
To support kubernetes 1.7, I used apps/v1beta1 as the apiVersion for the deployment, I hope that's fine. I already use the explicit selector which is needed in future versions.
If you think that is to much clutter for the compatibility, I would change the rbac and deployment versions accordingly.

@unguiculus Thanks for the feedback! I incorporated it and moved the chart to stable. I also renamed the rbac.apiVersion helper to kube-slack.rbac.apiVersion. Is anything else missing?
PS: Sorry for the delay.

/lgtm

/hold cancel


_send should use logging and not print

As a library, it is expected from users to configure logging from the library using logging module or something similar.
But, as it is using print function, it's difficult to remove this sometimes unwanted logging without copying the code or forking the code, thus defeating the purpose of having a library.
Would it be possible to do this change?
Thank you!

Yes, I will change it to support a better logging mechanism recently.

I've fixed it, check v1.0.1 update.


fix: remove redundant zeroing of memory allocated by bump alloc

TLDR memory is zeroed by default, so no need to manually zero the memory allocated https://doc.rust-lang.org/src/core/alloc/global.rs.html#201.
Ideally there would be a way to also test that this doesn't require paging in the memory allocated until it's actually written, but that seems like a hard variant to test. Can revisit if we want to be sure of this.

I recently added a test to MultiTest and now the AllocZeroed test case is failing. Leading me to think that zeroing of memory isn't actually gauranteed. I'll try to be more conclusive about this, but we may need to revert this.

Follow up, looks like this was actually related to using inout vs inlateout in our assembly instructions when invoking a syscall. When using inout there were situations where the compiler would emit a lui instruction (which loads the upper bits of an immediate value into a register) but would sometimes elide the corresponding add to ensure the lower 12-bits are also loaded into the register. It's still not clear to me why inlateout fixed it or what the issue with inout is.


Error on values reporting during plugin load

Describe The Bug:
Receive an error message when plugin loads.
To Reproduce:
resetart the HB plugin
Expected behavior:
No reported errors
Logs:
[28/04/2024, 15:49:51] [homebridge-rinnai-controlr] This plugin generated a warning from the characteristic 'Target Temperature': characteristic     was supplied illegal value: number 45.888888888888886 exceeded minimum of 48.888888888888886. See https://homebridge.io/w/JtMGR for more info.
Plugin Config:
{
"username"<EMAIL_ADDRESS>"password": "thisismypassword",
"recirculationDuration": 15,
"temperatureUnits": "F",
"minimumTemperature": 120,
"maximumTemperature": 140,
"platform": "RinnaiControlR"
}
Screenshots:
Screenshots:

Environment:
Plugin Version: v1.0.26
Homebridge Version: v1.8.1
Node.js Version: v20.12.2
NPM Version: unknown
Operating System: Raspbian

I got this return when my homebridge "UI Settings" we set to Celsius and my Rinnai was configured to Fahrenheit.
My guess this not a bug.

I got this return when my homebridge "UI Settings" we set to Celsius and my Rinnai was configured to Fahrenheit.
My guess this not a bug.

With your comment on F/C mismatch, I went into the ControlR app to check.
The unit is displaying in F.
The HB is displaying in F
I do wonder if the conversion value is being mishandled somewhere...
The value reported is 114.6 F; and my unit is set for 115.
The Min allowed value reported is 120.0 F
It seems the Plugin places a required minimum that is not in line with the ControlR App values. I did attempt to scroll down as far as possible on the app - it went to 98 F  and then gave a message 98 is an invalid setting (the unit had started running before I attempted) I was then able to move it up to 120, which is higher than needed, After a restart of HB, no error in the logs. I then set ot down to 110 and restarted HB again - still no error. This all seems very odd that this error repeated several times at the moring hour, then seems to have cleared. I am wondering where this value is really coming from, how is is comparred to this 114.8-120 valuation.
Also, seems the original report did not ask for the ControlR app and system values
App version 2.5.7
Firmware 238 (Also says "Latest Build"
No options to upgrade either one are available in the app.


zookeeper link fix

Change kafka-manager zookeeper link to zookeeper:zk and change ZK_HOSTS to match

Whoops


Type improvements and related logic fixes

The lack of local feedback during development has resulted in an accumulation of type errors. I've fixed the existing type errors in the browse module proper, and reinstated a version of the old test script.
As previously discussed, we decided to not put an emphasis on types in test code, but we have nearly 150 errors there - I've capped it arbitrarily at 150 in the test script. However, any errors are still spit out.
We can discuss how to best get feeback by perhaps changing the MyPyTest class to suit our needs.

There seems to be an issue with the travis system finding a new version of the hypothesis module, can revert if necessary.


always disable bracketed paste after read_line

According to @sholderbach 's comment: https://github.com/nushell/nushell/pull/9399#issuecomment-1585692641
It's good to disable bracketed paste in reedline rather than nushell

The fact that this was merged makes me seriously reconsider using this project.

I think the problem is that in your previous PR you were re-setting the bracketed paste. Now it's just being disabled indefinitely.


Why sharedServices return id? I can not use dot-notation syntax.

I have to do like:
((SystemServices *)[SystemServices sharedServices]).allSystemInformation

Is this intentionally design?

Hey @VincentSit,
Good catch!
That was not intentional.  It should be "instancetype".  I'll make sure that gets fixed.
Thanks,
Shmoopi

Thank you!


Add Asset Allocation Experiment code

I do not intend to merge this branch, this pull request has been created so that I can document what changes I have made and the thinking behind them. I intend to keep master close to the original simulation.
This experiment was to see what happens if...  Rather than rebalancing each year, we 'rebalance' to avoid selling equities during downturns and keep a reserve of 'safe' assets the rest of the time.
So far, it appears that it does provide a very minor benefit, but this may be down to coding error or many other factors. Further investigation is still required.

It turns out that this strategy is very similar to OmegaNot, only this one allows the bond allocation to be replenished. Perhaps it should be called NotOmegaNot?


[MM-11986 & MM-11987] Add bottom margin of 10 to "show more/less button"

Summary
Add bottom margin of 10 to "show more/less button"
Ticket Link
Jira ticket: MM-11986 & MM-11987
Checklist

[x] Added or updated unit tests (required for all new features)
[x] Has UI changes

Device Information
This PR was tested on: [iPhone and Android simulator/emulator]
Screenshots
iPhone:

Android:

@asaadmahmood Please see screenshot if those spacing are acceptable.  Thanks!


build-marist-rhel77-s390x-2 Out Of Space

https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk15/job/jdk15-linux-s390x-hotspot/3/consoleFull
15:29:54  zip I/O error: No space left on device

Hmm, I'm wondering if we're going to get squeezed on space again because of 8+11+14+15+16

Almost certainly - was something I mentioned in slack

Now recovered and back online. 83Gb free


fix fact tests dont build without closure

PR Summary

I discovered while debugging some issues for @ktsai7 that the tests do not build with -DSINGULARITY_BUILD_CLOSURE=OFF because the closure code is used in singularity_eos.cpp, which is called in the tests.
Long term, @dholladay00 I think decoupling closure code from the C/fortran API is probably a good idea. We can tackle that when the xrage-specific stuff moves out of singularity-eos. Short term, I have simply fixed the issue by adding appropriate include guards.
I also add a test in github to check that a minimal build configuration can still construct unit tests.
PR Checklist

[x] Adds a test for any bugs fixed. Adds tests for new features.
[x] Format your changes by using the make format command after configuring with cmake.
[x] Document any new features, update documentation for changes made.
[x] Make sure the copyright notice on any files you modified is up to date.
[x] After creating a pull request, note it in the CHANGELOG.md file

I also add a test in github to check that a minimal build configuration can still construct unit tests.

So the "Tests Minimal" is just a subset of the "Tests Full", right? Am I correct than running both is basically to help debugging CI issues?

So the "Tests Minimal" is just a subset of the "Tests Full", right?

Yes, but the code is built differently, so "Tests Minimal" captures a different code-path in the cmake.

Am I correct than running both is basically to help debugging CI issues?

It's partly CI issues, but also partly that we need to continue support building the code without closures if, for example, we want to build with Kokkos but without kokkos-kernels. This new test ensures that that build path continues to work.

It's partly CI issues, but also partly that we need to continue support building the code without closures if, for example, we want to build with Kokkos but without kokkos-kernels. This new test ensures that that build path continues to work.

Yeah I think my confusion was that turning off options is not necessarily a perfect subset of the code compared to having those options on; turning off options has different code associated with it in some instances. In other words, testing with more options doesn't guarantee that the code works when those options are off.


server and client names are incorrect

Thrift.Test.UserService.Servers.Binary.Framed should be Thrift.Test.UserService.Binary.Framed.Server
Thrift.Test.UserService.Clients.Binary.Framed should be Thrift.Test.UserService.Binary.Framed.Client

(in the docs)


fix: Fix the python release workflow

Getting failure to reupload here which means it worked: https://github.com/extism/extism/actions/runs/3465513773/jobs/5788329117


Discussion on referring to numbers of bytes

@Rick-Anderson @wadepickett
I raised this❓a couple of times over the years about removing the confusion on KB, MB, and GB. Devs get confused, for example, knowing if 1 KB is 1,000 bytes or 1,024 bytes. My remarks on this go back as far as the old issue  (2017! 👴) ...
Beefing up my (mis)understandings on contribution guidelines
https://github.com/dotnet/AspNetCore.Docs/issues/2617
... and that brings up a few fun memories 😄.
The style manual only hints that they want to avoid embracing the standard that's been out for like 20 years now 😢 ...
https://learn.microsoft.com/en-us/style-guide/a-z-word-list-term-collections/term-collections/bits-bytes-terms
Technically per IEC and SI, we should be using "KiB" (kibibyte), "MiB" (mebibyte), and "GiB" (gibibyte) to eliminate the confusion ...
https://en.wikipedia.org/wiki/IEEE_1541-2002
Since they won't permit that approach, I've been hardening docs for several years using either of the following techniques ...

Place the exact number of bytes in parentheses after the mention ... "The default is 32 K (32,768 bytes)."
Show the calculated number of bytes ... "The default is 32 K (32 * 1024)."

I would prefer to standardize on one of those and not drop the clarification entirely. I think it makes it sooooo much plainer what we mean ... it's perfectly clear in light of avoiding the standard.
I don't think you care if I continue using a technique in Blazor docs, but what do you think about those two approaches? Do you like one better than the other?

I prefer The default is 32 K. I think most folks know that means 32,768 bytes, but even if they don't, when is it that important? cc @tdykstra

I don't think you care if I continue using a technique in Blazor docs

Correct.

I suppose another way to go is just for me to briefly remark on this in the Blazor Fundamentals overview and go with your suggestion everywhere (i.e., drop my convention). I agree that most devs know what MS means because MS has always taken the non-standards approach.
Still tho ... my recommendation for all companies in this industry is to adopt the IEC/SI standard so that now and in another 20 years people still aren't confused about what these figures mean.

How about using the KiB notation and make KiB a link to our readme or some other doc that defines it. I don't like Wikipedia links.  Or find a good link like https://web.stanford.edu/class/cs101/bits-gigabytes.html

I don't want to break management's guidance tho. Maybe I'm giving them too much credit, but I think they're preserved their guidance on purpose. What I've been doing isn't going to ruffle any feathers because it adopts their implicit approach and merely clarifies it for our readers. If I go with "KiB"/"MiB"/"GiB", they might sharpen their beaks and claws 🦅 for me 😨😄.
I think I've talked myself out of my convention. Tanay just told me that he thinks almost all of the devs out there know what MS means in our docs. You said that independently. I think so ... I think that's right.
I'm going to recast this issue for resolving the Blazor docs on a short bit in the Fundamentals overview and just using "KB"/"MB"/"GB" everywhere.


Boost SEO

Content on section pages, content on front page.  Meta data tags, google webmaster tools. Indexing. etc.
Add the slogan to the front page/decide on a real slogan:

Compiled and garbage-collected news for you
The best of HackerNews
HackerNews Easy Mode
Shortcut/Hack


chore: fork

Temporary fork that should address:

flutter_quill #1889
flutter_keyboard_visibility #163
flutter_keyboard_visibility #150

See README.md of flutter_keyboard_visibility_temp_fork, published in pub.dev and still compatible with flutter_keyboard_visibility_platform_interface

It seems that a exception is thrown as warning in one of the tests, which was before the fork:
00:05 +6: KeyboardDismissOnTap It removes focus when tapped

Warning: A call to tap() with finder "Found 1 widget with key [<'box'>]: [
  SizedBox-[<'box'>](width: 100.0, height: 100.0, renderObject: RenderConstrainedBox#9412b relayoutBoundary=up1),
]" derived an Offset (Offset(400.0, 50.0)) that would not hit test on the specified widget.
Maybe the widget is actually off-screen, or another widget is obscuring it, or the widget cannot receive pointer events.
The finder corresponds to this RenderBox: RenderConstrainedBox#9412b relayoutBoundary=up1
The hit test result at that offset is: HitTestResult(_RenderInkFeatures#3649f@Offset(400.0, 50.0), RenderPhysicalModel#2a6f8@Offset(400.0, 50.0), RenderPointerListener#bd145@Offset(400.0, 50.0), RenderSemanticsGestureHandler#ca674@Offset(400.0, 50.0), RenderSemanticsAnnotations#19c62@Offset(400.0, 50.0), RenderRepaintBoundary#25249@Offset(400.0, 50.0), RenderIgnorePointer#bf3ff@Offset(400.0, 50.0), _RenderSnapshotWidget#1e25a@Offset(400.0, 50.0), _RenderSnapshotWidget#95139@Offset(400.0, 50.0), _RenderSnapshotWidget#b118d@Offset(400.0, 50.0), _RenderSnapshotWidget#ff261@Offset(400.0, 50.0), RenderRepaintBoundary#b6aa3@Offset(400.0, 50.0), RenderSemanticsAnnotations#f02ad@Offset(400.0, 50.0), RenderOffstage#5a944@Offset(400.0, 50.0), RenderSemanticsAnnotations#3f025@Offset(400.0, 50.0), _RenderTheater#e2770@Offset(400.0, 50.0), RenderAbsorbPointer#63519@Offset(400.0, 50.0), RenderPointerListener#68ed4@Offset(400.0, 50.0), RenderSemanticsAnnotations#ed5df@Offset(400.0, 50.0), RenderCustomPaint#ba033@Offset(400.0, 50.0), RenderSemanticsAnnotations#65bb4@Offset(400.0, 50.0), RenderSemanticsAnnotations#adaa2@Offset(400.0, 50.0), RenderTapRegionSurface#f0efb@Offset(400.0, 50.0), RenderSemanticsAnnotations#63b8a@Offset(400.0, 50.0), RenderSemanticsAnnotations#51f2d@Offset(400.0, 50.0), RenderSemanticsAnnotations#dadc2@Offset(400.0, 50.0), HitTestEntry<HitTestTarget>#65d04(_ReusableRenderView#1423b), HitTestEntry<HitTestTarget>#fb738(<AutomatedTestWidgetsFlutterBinding>))
#0      WidgetController._getElementPoint (package:flutter_test/src/controller.dart:1941:25)
#1      WidgetController.getCenter (package:flutter_test/src/controller.dart:1792:12)
#2      WidgetController.tap (package:flutter_test/src/controller.dart:1040:18)
#3      main.<anonymous closure>.<anonymous closure> (file:///Users/ellet/Developer/playground/framework_based/flutter/flutter_keyboard_visibility_temp_fork/flutter_keyboard_visibility_temp_fork/test/flutter_keyboard_visibility_test.dart:243:20)
<asynchronous suspension>
#4      testWidgets.<anonymous closure>.<anonymous closure> (package:flutter_test/src/widget_tester.dart:189:15)
<asynchronous suspension>
#5      TestWidgetsFlutterBinding._runTestBody (package:flutter_test/src/binding.dart:1032:5)
<asynchronous suspension>
#6      StackZoneSpecification._registerCallback.<anonymous closure> (package:stack_trace/src/stack_zone_specification.dart:114:42)
<asynchronous suspension>
To silence this warning, pass "warnIfMissed: false" to "tap()".
To make this warning fatal, set WidgetController.hitTestWarningShouldBeFatal to true.

00:05 +9: All tests passed!

I'm considering about removing the web package from the dependencies since the package doesn't support Flutter web and will always return false.


Fix guava library version in build.xml

@rizard guava library version in build.xml is out-of-date, it was V13.0.1, and it should be V20.0.
Update Google guava library version to the correct one "guava-20.0.jar", so that at the runtime it will be loaded correctly.

Good catch, @QingWang0909


Wfly 4560

https://issues.jboss.org/browse/WFLY-4560
v10 is fine for this. making sure it merges OK ready for when master opens

Linux Build 6391 is now running using a merge of 5a9848f341787d29ee5f088d96ab7edab4292128

Windows Build 1470 is now running using a merge of 5a9848f341787d29ee5f088d96ab7edab4292128

Linux with security manager Build 1318 is now running using a merge of 5a9848f341787d29ee5f088d96ab7edab4292128

Linux with security manager Build 1318 outcome was SUCCESS using a merge of 5a9848f341787d29ee5f088d96ab7edab4292128
Summary: Tests passed: 825, ignored: 317 Build time: 0:05:21

Windows Build 1470 outcome was SUCCESS using a merge of 5a9848f341787d29ee5f088d96ab7edab4292128
Summary: Tests passed: 2857, ignored: 373 Build time: 0:46:48

Linux Build 6391 outcome was FAILURE using a merge of 5a9848f341787d29ee5f088d96ab7edab4292128
Summary: Tests failed: 2 (2 new), passed: 2852, ignored: 373, muted: 1 Build time: 0:48:27
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ejb.security.jbossappxml.JBossAppXMLSecurityTestCase.testEJBRunAs: <no details avaliable>

org.jboss.as.test.integration.jca.moduledeployment.PartialObjectActivationJarTestCase: <no details avaliable>

I'm assuming this is for 10.0 as we've already released 9.0.0.CR1. Please advise if that's incorrect. We'd probably need to arrange special testing.

sorry, I should have said 10 is fine nps. apologies if I missed a notification that I shouldn't be PRing at this stage

No worries; PRs for 10 are fine. There are just a few where it's uncertain if it needs to go in 9.x, and it's no trouble for me to ask. :)

I am not sure how to request a mute @ctomc but http://brontes.lab.eng.brq.redhat.com/project.html?projectId=WF_PullRequest&testNameId=4790235807401694128&tab=testDetails fails all the time it seems.
In terms of JBossAppXMLSecurityTestCase.testEJBRunAs I have ran this several times and do not see a failure in any of my environments. I realise it is the first time it failed but looking at the build log I have zero idea why my change would make any difference.

retest this please

Windows Build 1508 is now running using a merge of 5a9848f341787d29ee5f088d96ab7edab4292128

Linux Build 6429 is now running using a merge of 5a9848f341787d29ee5f088d96ab7edab4292128

Linux Build 6429 outcome was SUCCESS using a merge of 5a9848f341787d29ee5f088d96ab7edab4292128
Summary: Tests passed: 2867, ignored: 373, muted: 1 Build time: 0:48:54

Windows Build 1508 outcome was FAILURE using a merge of 5a9848f341787d29ee5f088d96ab7edab4292128
Summary: Tests failed: 1 (1 new), passed: 2867, ignored: 373 Build time: 0:52:18
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.clustering.cluster.cdi.CdiFailoverTestCase(SYNC-tcp).testGracefulSimpleFailover: <no details avaliable>


Respect use case where multiple wallets and one is not on blockchain

Operation "transaction add-inputs" fails with "Wallet is not attached to any blockchain" with multiple wallets if wallet "previous" to wallet containing the UTxO is not attached to blockchain.
./target/debug/cardano-cli transaction add-input 3hrnAF 60540befd34211b88888dd216410ec8772d25645ad7826e69cb20662411e8a2a 1
Wallet is not attached to any blockchain

It is correct that UTxO inputs can come from different wallets, but should skip non-attached wallets.
Recreate: Make a wallet AAA, do not attach. Make a wallet BBB attach mainnet. Send coin to BBB. Create new transaction on mainnet, attempt to add-input from BBB. Will exit because it looks at AAA first, sees it is not attached and quits.
The function is load_attached_blockchain in /wallet/utils.rs which will process::exit if called with unattached wallet. Called by create_wallet_state_from_logs, which is called from multiple places. Where called for a wallet /wallet/commands.rs it is ok to exit I think because it is only called for one wallet. However, function is called from places in /transaction/commands.rs for ALL wallets, in this case should not exit, but move on to next wallet.
Eg. add-inputs, function find_input_in_all_utxos (probably same issue with sign() func, maybe others). So for add-inputs could fix in find_input_in_all_utxos with a continue maybe.
    for (_, wallet) in Wallets::load(root_dir.clone()).unwrap() {

        let blockchainname = wallet.config.attached_blockchain().unwrap();
        if !blockchainname.is_some() {
            // wallet is not attached to a blockchain
            continue;
        }

I have started a branch to fix this.
Making create_wallet_state_from_logs non-panicky is less trivial than the type of changes done in #52, as it needs a unified error type for AddressLookup and all its impls.

File under #39


attributes() method always return an Object{undefined: undefined}

Today , I upgrade to beta.8, and I tried to using attributes() , but found that it always return Object{undefined: undefined}.

This will be released in beta.9


Passive entity mappings

Signed-off-by: liach<EMAIL_ADDRESS>

Closing in favor of #1231


Cloud Watch Metrics not Cleaned up When Stack Deleted

Amazon CloudWatch Metrics are automatically deleted after 14 days. There is no delete API call and customers have up to 10 free custom metrics.
http://aws.amazon.com/cloudwatch/faqs/
http://aws.amazon.com/cloudwatch/pricing/


Add React Hot Reload

Closes #125
Description
Adds React Hot Reload functionality to the development environment. No modifications are made to production builds. This simply improves the feedback loop of seeing changes
Steps to Test

Run yarn run hot
Load the extension in Chrome
Click on the toolbar icon to open the popup
Click on the Invoice button to open the drawer
Type any number in the Amount field
In your code editor, open the InvoiceForm.tsx file
Change the Amount label to Payment Amount then Save the file
Observe in the browser that the label updates but the state of the component remained unchanged

Notes
This is not 100% ready to be merged, but I wanted to put it out there for now. I need to test browsers other than Chrome to make sure nothing is broken.

Confirmed working in Chrome, Firefox & Brave. This PR is ready, awaiting feedback.

Sorry for the delay, have had some computer issues that have kept me away from Joule for the last week. I'll be able to test & merge this tomorrow.


[Improvement]: Prioratize return over retry in code completions

Description
$subject since writing a return statement is the most common scenario.

Closing the issue as this a limitation according to https://github.com/ballerina-platform/ballerina-lang/issues/35017#issuecomment-1074823742


Allow invoke IDs to be resolvable at type-level

closes https://github.com/statelyai/xstate/issues/4605
TODO:

[ ] tests

Doing some cleanup 🧹 we may revisit this.


Sent requests via HTTP proxy

It would be nice to be able to use the google-api-nodejs-client via a HTTP proxy.  Request appears to support proxy configuration.
Possibly via the withOpts:
discover('youtube', 'v3').withOpts({ proxy: 'http://mylocalproxy.com' });

Yep, process.env.HTTPS_PROXY = 'http://:'; worked for me. Thanks @aguegu


Use html loader

Remove the need for html!
Allow control over html minify options

TP: https://rockabox.tpondemand.com/entity/12240
Requires:

https://github.com/rockabox/rbx_ui_components/pull/460
https://github.com/rockabox/rig_create/pull/136
https://github.com/rockabox/rbx_campaign_management/pull/403

:+1: ready for QA :+1:

Can be merged for QA


Add custom extension builder to setup.py

Fixes #358
In order to provide a more stable installtion accross different packages
managers, the deduction of the build_lib directory have been removed and
replaced by the usage of a custom extension builder which is guaranteed
to know the exact path to the temporary build directories.

Codecov Report

Merging #359 (aa0e2bc) into master (987862c) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head aa0e2bc differs from pull request most recent head 29d42fa. Consider uploading reports for the commit 29d42fa to get more accurate results

@@           Coverage Diff           @@
##           master     #359   +/-   ##
=======================================
  Coverage   98.45%   98.45%
=======================================
  Files          93       93
  Lines        2982     2982
=======================================
  Hits         2936     2936
  Misses         46       46

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Fix typo "tranform-fn" to "transform-fn"

Fixes a typo in the following sentence:

⚙ Transform
When writing your own viewer, the first extension point you should reach for is :tranform-fn.

@vezwork thanks!