Feat/atom button

What's the purpose of this pull request?
Considering the context, what is the problem we'll solve? Where in VTEX's big picture our issue fits in? Write a tweet about the context and the problem itself.
How it works?
Tell us the role of the new feature, or component, in its context.
How to test it?
Describe the steps with bullet points. Is there any external link that can be used to better test it or an example?
References
Spread the knowledge: is there any content you used to create this PR that is worth sharing?
Extra tip: adding references to related issues or mentioning people important to this PR may be good for the documentation and reviewing process

for me it's LGTM, however, is it possible to add some guide on how to style the button, doing different states, like disabled etc? I didn't get still 100% of this, thanks!

@tlgimenes Yes, makes sense to me. I will create some guides on how to style the components.

@klzns @tlgimenes what do u think about the spread props or use specific props, like onClick, disabled, etc...?

@klzns @tlgimenes what do u think about the spread props or use specific props, like onClick, disabled, etc...?

I'm not sure. I prefer spreading because it makes our framework more hackeable, however it makes it more difficult to maintain I guess. I'm really not sure, but spreading seems ok to me

for me it's LGTM, however, is it possible to add some guide on how to style the button, doing different states, like disabled etc? I didn't get still 100% of this, thanks!

We can also use the Knobs addon for storybook to test the different behaviors of changing props.
https://storybook.js.org/addons/@storybook/addon-knobs/

@klzns @tlgimenes what do u think about the spread props or use specific props, like onClick, disabled, etc...?

I'm not sure. I prefer spreading because it makes our framework more hackeable, however it makes it more difficult to maintain I guess. I'm really not sure, but spreading seems ok to me

I think it's better to spread props in an atom component to make it as flexible as possible. And since it's a component that is almost an HTML element (), we need this versatility to use this component like a normal HTML element, with a11y attributes, ids, test-ids, roles, autofocus, etc.

If we want to make it extremely flexible and barebones, why not just use the <button /> element itself?
This is more or less a rhetorical question, because it might or might not make sense to create a specific <Button /> element, but I think the answer should help in deciding its API.

If we want to make it extremely flexible and barebones, why not just use the <button /> element itself? This component is basically just that.
This is more or less a rhetorical question, because it might or might not make sense to create a specific <Button /> element, but I think the answer should help in deciding its API. What is its purpose?
Because if it's just an HTML element to have some component semantic/add to a storybook, then yeah, the ...rest is necessary.

It's nice to make a component for button if we're going to make a base style or base config for every button, eventually. We don't have an styleguide for stores yet but we can have a default style for store's buttons, so it's nice to already have this component like this.

It's nice to make a component for button if we're going to make a base style or base config for every button, eventually. We don't have an styleguide for stores yet but we can have a default style for store's buttons, so it's nice to already have this component like this.

🪂  But as far as I know, we won't have styles/themes inside the store-ui. So I agree with @lbebber

It's nice to make a component for button if we're going to make a base style or base config for every button, eventually. We don't have an styleguide for stores yet but we can have a default style for store's buttons, so it's nice to already have this component like this.

🪂 But as far as I know, we won't have styles/themes inside the store-ui. So I agree with @lbebber

I don't like this idea about not having any styles inside the store-ui. Some components simply "doesn't work" without a base style, just imagine the Minicart, our Search or the Login UI with the native HTML style. Also, we need a base style to make our guide with the Storybook anyway.
I prefer to have a default style like storecomponents.store inside store-ui and then overwrite each custom style inside each store. Today we are importing all those default styles inside the store, so it's very confusing to know which one is custom or default.

for me it's LGTM, however, is it possible to add some guide on how to style the button, doing different states, like disabled etc? I didn't get still 100% of this, thanks!

We can use the Knobs addon for storybook to test the different behaviors of changing props.
https://storybook.js.org/addons/@storybook/addon-knobs/

🙏  this is really important I guess


Magnet cannot pull items up from farmland onto hopper. also does not pull 1/2 block away

As seen and reported on livestream at https://www.twitch.tv/darkphan
Case 1
Magnet on top of a hopper.  items from all around coming in (from crops or anything. In this case a harvester)  Items cannot come up off the farmland sometimes, due to height difference.
Case 2
Even if there is no hopper or farmland involved, the items stop about 1 block away or maybe 0.5 block. do not move all the way to magnet center.  Rarely they shoot across to the other side and start going back and forth

Fixed in release 1.6.4


[Addition Request]: PHP, Laravel, React, SvelteKit, Gleam, Nix

Love these icons!
It would be amazing to have the above mentioned added to this aswell.

Hello. I'm glad you like my icons.
The icons you suggested have been added to the project :)
Sorry for the delay

@Nighty3098
Thank you so much they look amazing!!!

As a thank you I wrote up a quick article on my blog, hope we can get many more to appreciate these awesome icons!
https://dev.to/jasper-at-windswept/the-cleanest-icons-for-your-next-readme-1f0h


Will VSCode upload my settings.json file ?

Version 1.23.1
Commit d0182c3417d225529c6d5ad24b7572815d0de9ac
Date 2018-05-10T17:11:17.614Z
Shell 1.7.12
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64
Steps to Reproduce:
1.Will VSCode upload my settings.json file ?
ref: https://code.visualstudio.com/blogs/2018/04/25/bing-settings-search

Nope


EnvoyFilter cannot guarantee idempotency for route configuration

Bug Description
EnvoyFilter supports removing and adding VIRTUAL_HOST. But if you remove and then add the same VIRTUAL_HOST in the same EnvoyFilter, the result is removing. This violates the idempotency.
For example, the initial RDS configuration is as follows, contains the VIRTUAL_HOST "allow_any"
[
    {
        "name": "9090",
        "virtualHosts": [
            {
                "name": "allow_any",
                "domains": [
                    "*"
                ],
                "routes": [
                    {
                        "name": "allow_any",
                        "match": {
                            "prefix": "/"
                        },
                        "route": {
                            "cluster": "PassthroughCluster",
                            "timeout": "0s",
                            "maxGrpcTimeout": "0s"
                        }
                    }
                ],
                "includeRequestAttemptCount": true
            },
            {
                "name": "prometheus.istio-system.svc.cluster.local:9090",
                "domains": [
                    "prometheus.istio-system.svc.cluster.local",
		     // ...
                ],
                "routes": [
                    {
 			// ...
                    }
                ],
                "includeRequestAttemptCount": true
            }
        ],
        "validateClusters": false
    }
]

The following EnvoyFilter will cause the VIRTUAL_HOST "allow_any" to be removed. But actually, I hope it doesn't change.
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: remove-add-test
  namespace: default
spec:
  configPatches:
  - applyTo: VIRTUAL_HOST
    match:
      context: SIDECAR_OUTBOUND
      routeConfiguration:
        name: "9090"
        vhost:
          name: allow_any
    patch:
      operation: REMOVE
  - applyTo: ROUTE_CONFIGURATION
    match:
      context: SIDECAR_OUTBOUND
      routeConfiguration:
        name: "9090"
    patch:
      operation: MERGE
      value:
        virtual_hosts:
        - domains:
          - '*'
          name: allow_any
          routes:
          - match:
              prefix: /
            route:
              cluster: PassthroughCluster

I've looked at the source code and I think the problem is in the processing logic of rc_patch.go.
https://github.com/istio/istio/blob/b55bf1a7debc0ce0081e91ad25694e839737512b/pilot/pkg/networking/core/v1alpha3/envoyfilter/rc_patch.go#L51-L64
In function ApplyRouteConfigurationPatches, merge was executed first, followed by the remove contained in function patchVirtualHosts. This is inconsistent with the order of patch in the above EnvoyFilter.
I think maybe we should execute remove before merge.
Version
hazard@hazard-VirtualBox:~$ istioctl version
client version: 1.13.2
control plane version: 1.13.3
data plane version: 1.13.3 (8 proxies)
hazard@hazard-VirtualBox:~$ kubectl version --short
Client Version: v1.23.6
Server Version: v1.20.7

Additional Information
No response

@howardjohn @ramaraochavali any suggestions?

can you explain the use case more? I think removals are treated differently - if you remove we assume it is removed.

What I want is some specified proxies keep 'allow_any' virtual_hosts, and remove 'allow_any' for other proxies. But the result is removing 'allow_any' for all proxies.
spec:
  configPatches:
  - applyTo: VIRTUAL_HOST
    match:
      context: SIDECAR_OUTBOUND
      routeConfiguration:
        name: "9080"
        vhost:
          name: allow_any
    patch:
      operation: REMOVE
  - applyTo: ROUTE_CONFIGURATION
    match:
      context: SIDECAR_OUTBOUND
      proxy:
        metadata:
          SLIME_APP: LAZYLOAD_GLOBAL_SIDECAR
      routeConfiguration:
        name: "9080"
    patch:
      operation: MERGE
      value:
        virtual_hosts:
        - domains:
          - '*'
          name: allow_any
          routes:
          - match:
              prefix: /
            route:
              cluster: PassthroughCluster

seems no one thinks this is problem?


Showing 2 monitors in RDP for XFCE4 on CentOs7

I have a virtual machine hosted by a vCenter appliance. The machine has 2 monitors specified in it's setting. However when I RDP into the box, the session only sees one monitor. I guess that makes sense since the monitors specified in the vCenter setting would be considered "physical" monitors, so when RDP into the box I'm starting a new X session that has nothing to do with these monitors. When I try setting RDP to use multiple monitors the connection crashes automatically. Any one have experience in this matter? I tried with GNOME, and it appears to set the resolution to span monitors at 3840x1080. Is there a way to force xrdp to show two monitors at 1920x1080 or alternatively a single monitor at 3840x1080?

You're quite correct in that that "physical" monitors connected to your VM are not important.
When you connect to the VM from an RDP client, the client will request a particular screen layout. If you're connecting from the Windows MSTSC.EXE client (and I assume you are), you can either request a single screen, or you can tick the Use all my monitors for the remote session box. This option will cause the client to send a description of all its connected monitors to the VM.
To actually use multiple monitors with the current version of XRDP (0.9.11):-

you will need to select the Xorg backend when you connect
the server will need to have xorgxrdp installed.

If you use the Xvnc backend you will end up with a single spanning monitor, which is possibly what you have already seen.

Closing this - multimon support has been a thing for quite a while.


Rotating the Sky?

Is there a way to rotate the Sky created using this package?  Adjusting the rotation on the Sky object does not seem to effect the horizon/lighting effects.
My scene's camera uses camera.up.set(0, 0, 1); to make the Z axis vertical.
I'm having trouble finding a way to rotate the sky to match.

This zoomed out image shows the rotated sphere (big axes lines) vs the scene axes lines.  This method isn't working. I need to rotate the horizon 90* around the X axis

Hi @jeremylach ! Thanks for reporting, I believe that should go to the Sky.js contributors.


Fixed ymls

Details
What does this PR have in it? Screenshots are worth 1000 words 😄
Checklist

[ ] Validate name matches <platform>_<mitre att&ck technique>_<short description> nomenclature
[ ] CI/CD jobs passed ✔
[ ] Validated SPL logic.
[ ] Validated tags, description, and how to implement.
[ ] Verified references match analytic.

playbook name consistent with the file name. LGTM!

Minor change looks good! :thank you:


Failed to connect to to raw.github.com

Using
wget https://git.io/vpn -O
Get an error below
--2024-09-21 09:22:42--  https://git.io/vpn   Resolving git.io (git.io)... <IP_ADDRESS>                                                                    Connecting to git.io (git.io)|<IP_ADDRESS>|:443... connected.                                                HTTP request sent, awaiting response... 301 Moved Permanently                                                 Location: https://raw.github.com/Nyr/openvpn-install/master/openvpn-install.sh [following]                    --2024-09-21 09:22:43--  https://raw.github.com/Nyr/openvpn-install/master/openvpn-install.sh                 Resolving raw.github.com (raw.github.com)... 2606:50c0:8003::154, 2606:50c0:8001::154, 2606:50c0:8002::154, . ..                                                                                Connecting to raw.github.com (raw.github.com)|2606:50c0:8003::154|:443... failed: Connection timed out.

This is an issue in your end. Try later, or transfer the file in a different way.


vm/adb: add console impl for remote Android virtual device

Add console string to the device struct in Adb Config
Add console implementation of Cuttlefish, using ssh when it's in remote instance

I've test it locally but not when Cuttlefish is remote, please instruct if this solution makes sense or there is better way to handle it.

Can you please give a bit more context to this changeset? So that it could be reviewed by someone without big expertise in Android.

I see you've implemented a direct ssh connection to the emulator. Why can't we use adb shell in the case of remote Cuttlefish instances? Would it be much slower or just impossible?
In the case of such Android emulators, do we actually need adb at all? Can remote Cuttlefish instances be treated just like other Linux VMs?

This PR adds console implementation of Cuttlefish (Android virtual device). The current Android console solution in Syzkaller works for physical Android phones that use cereal cable for debugging. But Cuttlefish doesn't have physical cable, it creates the console under cuttlefish_runtime.1/console that links to /dev/pts/. It also provides kernel.log under the same folder.
Cuttlefish also has two scenarios:

Cuttlefish runs locally on the same host where Syzkaller runs, this is easy to handle, we can pass in the console path in config and then reuse OpenConsole function to open it.

Cuttlefish runs remotely on a docker/vm, in such case Syzkaller can't directly open its console by syscall.Open(). So I implemented a direct ssh connection to it, first I tried using ssh -t vsoc-01@ip screen console_path but it only worked when I ran it on terminal, not in program. So I turned to use ssh to read the kernel.log instead, but tail -f kernel.log seems not fast enough so it doesn't return enough kernel logs back. Any suggestions?

I see you've implemented a direct ssh connection to the emulator. Why can't we use adb shell in the case of remote Cuttlefish instances? Would it be much slower or just impossible?
Can you explain more about how to use adb shell? Today Syzkaller uses 'adb shell dmesg -w' to get logs but adb could lose connection to the device when kernel crash happens.

In the case of such Android emulators, do we actually need adb at all? Can remote Cuttlefish instances be treated just like other Linux VMs?
If we can ssh connect to the console in remote Cuttlefish instance, probably doesn't need adb anymore. But I couldn't find a way to make it work, ssh -t vsoc-01@ip screen console_path worked when I ran it on terminal but returned error in code. Please advise if there is different ways to connect to remote console.

but it only worked when I ran it on terminal, not in program

Did it return any specific error messages? Or do you have a reason to exclude this approach at all? If it just fails for some (yet) unknown reason, I would rather investigate it to find the actual problem.

Today Syzkaller uses 'adb shell dmesg -w' to get logs but adb could lose connection to the device when kernel crash happens.

Won't this also affect the current approach with tail -f kernel.log? If the kernel crashes, I'm not sure if it's going to pipe any data through tail and then ssh.
Syzkaller supports a so called "isolated" mode, where it fuzzes an existing Linux machine that is accessible via ssh. (You probably know, but just in case: here's a link to the documentation). And there is acually the same problem - we might get e.g. warnings (if the kernel is not configured to panic on a warning), but not some severe crashes.
In that case, the pstore mechanism is used to preserve the information about a crash. And then that data is retrieved through this Diagnose method:
https://github.com/google/syzkaller/blob/fcdb12ba70875c410749932abf39160d19c753d9/vm/isolated/isolated.go#L379
I fear something similar must also be done in this case. Maybe Android/Cuttlefish has some of its own specifics that change everything, but it doesn't seem likely to me. Please correct me if I'm wrong.
By the way, could the isolated type of machine be applied to Cuttlefish instances just as it is?
--
PS I see there are also some other (open-ended) questions, but personally I don't have direct answers to them.

Notes from an offline discussion:

By ssh'ing into vsoc-01@ip, we access not the Android VM itself, but its host, who stores all serial output. So in case of crashes nothing should be lost.
isolated mode is not acceptable here, because of the reason above + we need adb to establish root ssh access.

And also please fix the ci/smoke presubmit test. Otherwise, the changes look good to me.

could you please offer a implementation about how to fuzzing android kernel in vm?


Locally funded channels

Fund channels locally from on chain wallet.
closes https://github.com/synonymdev/react-native-ldk/issues/30
Step 1: Create channel
const res = await lm.createChannel({
  counterPartyNodeId: 'abc123',
  channelValueSats: 20000,
  pushSats: 5000,
});

const { value_satoshis, output_script, temp_channel_id } = res.value;

Step 2: Create funding tx using on chain wallet (DO NOT BROADCAST)

Convert output_script to address
bitcoin-cli createrawtransaction '[{"txid":"<tx-id>","vout":<index>}]' '{"<address>":<btc>}'
bitcoin-cli signrawtransactionwithwallet <raw-transaction-hex>

Step 2: Fund channel
const res = await ldk.fundChannel({
  temporaryChannelId,
  counterPartyNodeId: peers.lnd.pubKey,
  fundingTransaction: signedTx,
 });

@Jasonvdb @coreyphillips code looks ok by my judgement, I was eager to test it but it won't run the JS in the example app (android app starts, then shows red error screen)
now I did notice after looking at the failed tests that it is the same error I got:

Class private methods are not enabled. Please add @babel/plugin-proposal-private-methods to your configuration.


⚠ Bofrost.lu has degraded performance

In ea75d4a, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1149 ms

Resolved: Bofrost.lu performance has improved in 5a720f9 after 17 minutes.


Add new cli command to update runtime with graphql depth limit

Why make this change?

Closes #2243, #2249

Currently CLI doesn't support updating runtime settings.
Need a flexible way to add/update graphQL depth limit

What is this change?

This PR contains changes related to CLI.
Updated the draft schema to add the new property in the runtime section.
A new command is added dab configure, which will allow updates for non-entity based config settings.
We have added an optional flag --runtime.graphql.depth-limit, which will be used to specify the depth limit for GraphQL queries and mutation.
By Default, No limit is imposed.
values supported are only integer values [1, Int32.MaxValue].
example: dab configure --runtime.graphql.depth-limit 20.

NOTE:
if limit is not already specified, it will add this property to config, else update it.

Additionally, users can use -1 to remove limit.
dab configure --runtime.graphql.depth-limit -1

How was this tested?

[X] Unit Tests

Sample Request(s)
initial config
"graphql": {
    "path": "/graphql",
    "enabled": true,
    "allow-introspection": true
}

command: dab configure --runtime.graphql.depth-limit 20
"graphql": {
    "path": "/graphql",
    "enabled": true,
    "allow-introspection": true,
    "depth-limit": 20
}

command: dab configure --runtime.graphql.depth-limit 8
"graphql": {
    "path": "/graphql",
    "enabled": true,
    "allow-introspection": true,
    "depth-limit": 8
}

command: dab configure --runtime.graphql.depth-limit -1
"graphql": {
    "path": "/graphql",
    "enabled": true,
    "allow-introspection": true,
    "depth-limit": -1
}

Special Case:
if value of "depth-limit" is null in the config. it will stay as is, if not updated.
"graphql": {
    "path": "/graphql",
    "enabled": true,
    "allow-introspection": true,
    "depth-limit": null
}

command: dab configure
No change:
"graphql": {
    "path": "/graphql",
    "enabled": true,
    "allow-introspection": true,
    "depth-limit": null
}

/azp run

/azp run

/azp run


added django examples

This repository is a collection of  opensource projects and examples using the Django WebFramework

Hmm, good idea. Doesn't seem like it has mass adoption yet so holding off for now on adding.


DOC: Make it clear notebooks are Python 3 only?

Should we make it clear in README that notebooks only run in Python 3 and not Python 2?

I think that's a good idea - some users who are not familiar with Python may not realize that as a possible issue and having it in the README would be a good first step to helping them troubleshoot that issue.


twister: runner: add tfm_merged.hex in post process

tfm_merged.hex is the final hex used in tfm
accroding to https://docs.zephyrproject.org/latest/samples/tfm_integration/tfm_ipc/README.html

right, seems platfrom with sysbuild configure right, can have the right file, and if not, this file is not there


Invalid argument

EXCEPTION: N5boost16exception_detail10clone_implINS0_19error_info_injectorINS_16bad_lexical_castEEEEE
bad lexical cast: source type value could not be interpreted as target
C:\ESBC\esbcoin-qt.exe in Runaway exception
2019-04-26 17:24:51 CDBEnv::EnvShutdown : Error 22 shutting down database environment: Invalid argument

Check data in the masternode.conf file, note how the transaction index is specified.
And in the next release I will add more checks on the correctness of the source data.

@SkaZun yeah, i found a "bug", it was not commented mn line in masternode.conf but why this crash start wallet i donno =(


Figure out Monorepo strategy

Description
We are currently 4 major version behind the latest release of lerna. We should consider either upgrading to the latest version or switching to a new monorepo framework such as turborepo, nx or pnpm.

IF it's easy enough, let's do it. Otherwise it's ok.

I would be this at a medium. There are always annoying configuration issues that pop up and can become time consuming.


Getting BLS group generators

What is the best way to get the generators from the groups associated with bls12-377?
I am looking at gnark/std/algebra/native/sw_bls12377/g1.go but do not see any way to get the generators.
I had thought that something like the following would work, but it creates a nil pointer dereference:
import bls12377 "github.com/consensys/gnark/std/algebra/native/sw_bls12377"

var g bls12377.G1Affine
var a big.Int
a.SetString("1", 10)
g.ScalarMulBase(api, a)

I think that I have gotten things working as follows. Please advise on whether this is a sensible method.
import(
bls12377 "github.com/consensys/gnark/std/algebra/native/sw_bls12377"
"github.com/consensys/gnark-crypto/ecc/bw6-761/fr"
)

var gen bls12377.G1Affine
	var genX big.Int
	genX.SetString("81937999373150964239938255573465948239988671502647976594219695644855304257327692006745978603320413799295628339695", 10)
	var genY big.Int
	genY.SetString("241266749859715473739788878240585681733927191168601896383759122102112907357779751001206799952863815012735208165030", 10)

	xCoord := new(fr.Element)
	xCoord.SetBigInt(&genX)

	yCoord := new(fr.Element)
	yCoord.SetBigInt(&genY)

	gen.X = *xCoord
	gen.Y = *yCoord

This should work. There is a shorthand for assigning the values https://github.com/Consensys/gnark/blob/master/std/algebra/native/sw_bls12377/g1.go#L410.
You can also get dynamically the generator by calling https://github.com/Consensys/gnark-crypto/blob/master/ecc/bls12-377/bls12-377.go#L145. Then you won't have to hardcode the base point in the circuit.


improve version command output

What this PR does / why we need it:

always output git sha (plus dirty or clean)
if we're building a tag, show that
remove build timestamp

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged):
Fixes #1227
Fixes #1228
Special notes for your reviewer:
Let me know if there's any additional changes you all would like in these areas.

This change is

@sgoings,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Example outputs:
Clean commit (no changes in working directory)
./bin/acs-engine version
INFO[0000] ACS-Engine Version: canary (6e4b760b-clean)

Dirty working directory
./bin/acs-engine version
INFO[0000] ACS-Engine Version: canary (6e4b760b-dirty)

Commit == Tag (made-up-tag)
./bin/acs-engine version
INFO[0000] ACS-Engine Version: made-up-tag (6e4b760b-clean)

👍 once this gets rebased

Rebased


[request]: Allow passing in aws_access_key_id and secret (or document if already possible)

Tell us about your request
I can't seem to find a way to pass in an arbitrary aws_access_key_id and aws_secret_access_key when configuring a client. Please provide a way, or provide docs/examples of how it can be done currently.
Tell us about the problem you're trying to solve.
My use case is that I need to be able to supply an arbitrary key/secret per request, and use that for the client.
I'd like to be able to do something like the following, but with .credentials(key, secret) or similar.
let shared_config = aws_config::from_env().region(region_provider).load().await;

I looked into how to build a CredentialsProvider, but that looks way too complicated for such a simple thing, and I figured I was going down the wrong road.
Are you currently working around this issue?
    std::env::set_var("AWS_ACCESS_KEY_ID", creds.0);
    std::env::set_var("AWS_SECRET_ACCESS_KEY", creds.1);

    let shared_config = aws_config::from_env().region(region_provider).load().await;

    let client = Client::new(&shared_config);

   [... do stuff...]

    std::env::remove_var("AWS_ACCESS_KEY_ID");
    std::env::remove_var("AWS_SECRET_ACCESS_KEY");

I suspect this introduces a race condition between concurrent runs, and is insecure to boot, but it's just a stopgap until I figure out how to do it properly so I can write the rest of my logic in the meantime.
Additional context
No response

https://docs.rs/aws-types/0.3.0/aws_types/credentials/struct.Credentials.html#method.from_keys
Note that you'll need to enable the hardcoded-credentials feature.


error execution phase upload-config/kubelet: Error writing Crisocket information for the control-plane node: timed out waiting for the condition

What happened:
An error is reported when executing the command
 sealer run kubernetes:v1.19.9 --masters <IP_ADDRESS> --passwd 8ql6,qqq

But when I log in to the node(<IP_ADDRESS>) execute this command, it is successful
kubeadm init --config=/var/lib/sealer/data/my-cluster/kubeadm-config.yaml --upload-certs -v 0 --ignore-preflight-errors=SystemVerification

[apiclient] All control plane components are healthy after 88.505715 seconds
[upload-config] Storing the configuration used in ConfigMap "kubeadm-config" in the "kube-system" Namespace
[kubelet] Creating a ConfigMap "kubelet-config-1.19" in namespace kube-system with the configuration for the kubelets in the cluster
error execution phase upload-config/kubelet: Error writing Crisocket information for the control-plane node: timed out waiting for the condition
To see the stack trace of this error execute with --v=5 or higher
2021-05-27 16:54:26 [EROR] init master0 failed, error: [ssh][<IP_ADDRESS>]run command failed [kubeadm init --config=/var/lib/sealer/data/my-cluster/kubeadm-config.yaml --upload-certs -v 0 --ignore-preflight-errors=SystemVerification]. Please clean and reinstall

What you expected to happen:
Successful installation
How to reproduce it (as minimally and precisely as possible):
sealer run kubernetes:v1.19.9 --masters <IP_ADDRESS> --passwd 8ql6,qqq --user root
Anything else we need to know?:
Environment:

sealer version (use sealer version):

{"gitVersion":"latest","buildDate":"1970-01-01T00:00:00Z","goVersion":"go1.16.3","compiler":"gc","platform":"linux/amd64"}

Cloud provider or hardware configuration:

OS (e.g: cat /etc/os-release):

NAME="CentOS Linux"
VERSION="7 (Core)"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="7"
PRETTY_NAME="CentOS Linux 7 (Core)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:7"
HOME_URL="https://www.centos.org/"
BUG_REPORT_URL="https://bugs.centos.org/"

CENTOS_MANTISBT_PROJECT="CentOS-7"
CENTOS_MANTISBT_PROJECT_VERSION="7"
REDHAT_SUPPORT_PRODUCT="centos"
REDHAT_SUPPORT_PRODUCT_VERSION="7"

Kernel (e.g. uname -a):

Linux  3.10.0-957.27.2.el7.x86_64 #1 SMP Mon Jul 29 17:46:05 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Install tools:
Others:

If run the command not on master0, will has this issue.


Non-Windows install instructions?

Describe the feature you'd like
I was looking forward to trying this until I got to the part in the README that says:

Run Install.bat from Windows Explorer as a regular, non-administrator user.

Request: Would love to see installation instructions for non-Windows users.
Reason:  I'm on a Mac. I also have access to a Linux machine.  No Windows.

Hi @drscotthawley, Thanks for raising the issue.
In case if you are using Mac, you can use shell script like install.sh.
You can run install.sh and then run start-webui.sh to run the WebUI after installation.
I should have added this to the ReadME, so I've added it now.
But I'm not sure if the shell script would work because I haven't tested it on the Mac.
If encounter any problem during the process, please let me know.

Hello. I'm currently using a Mac. Looking at the Git commit history, it seems that the shell script was deleted yesterday. Do you have any plans to restore it in the future?

Hi @kwongiho, the project has a Dockerfile but if you're unable to use Docker,
It would definitely be better to have shell script.
I removed the shell script because just it contained errors, and I don't have the environment to test them.
Any PR about it would be very appreciated.
For now, please install and run it manually by :
For installation :
python -m venv venv
source venv/bin/activate
pip install -r requirements.txt

To run :
source venv/bin/activate
python app.py

Shell scripts are now added in #125, so this is closed.
Please feel free to reopen if you find any errors!

你好@drscotthawley，感谢您提出这个问题。 如果您使用的是 Mac，则可以使用类似 的 shell 脚本install.sh。 您可以在安装后运行install.sh然后运行start-webui.sh以运行 WebUI。 我应该将其添加到 ReadME 中，所以我现在添加了它。
但我不确定该 shell 脚本是否可行，因为我还没有在 Mac 上测试过。
如果您在此过程中遇到任何问题，请告诉我。

@jhj0517 可以使用，最好加入到安装说明下，还有很多macbook是python3 /
pip3 所以可以在安装说明下确定当前的macos环境python的相关环境


Board Review: Management Plane Namespace Review Microsoft.AzureLargeInstance

Thank you for submitting this review request. Thorough review of your management library namespaces ensures that your library names are consistent with the guidelines and the consumers of your management library have a consistently good experience when using Azure.
To ensure consistency, all language library names will generally be reviewed together.
Before submitting, ensure you adjust the title of the issue appropriately.
Note that the required material must be included before a meeting can be scheduled.
Contacts and Timeline

Responsible service team: Bare Metal Infrastructure
Main contacts<EMAIL_ADDRESS><EMAIL_ADDRESS>/ @alexandraavendano<EMAIL_ADDRESS>/ @pabowers<EMAIL_ADDRESS>Expected code complete date: Implementation is complete
Expected release date: By end of this month

About the Service (required)
(I'm adding a new Resource type and namespace to an existing RP - Bare Metal Infrastructure)

Link to documentation introducing/describing the service: https://learn.microsoft.com/en-us/azure/baremetal-infrastructure/concepts-baremetal-infrastructure-overview
Link to the service REST APIs Specs: https://github.com/Azure/azure-rest-api-specs/pull/25221
Link to GitHub issue for previous review sessions, if applicable:

Namespace Proposals (required per language)
In the examples below please replace every occurrence of [ResourceProviderName] with your the service resource provider name. Be sure to keep the casing shown with [ResourceProviderName] when replacing it with the service resource provider name.

.NET: Azure.ResourceManager.LargeInstance
Java: azure-resourcemanager-largeinstance (com.azure.resourcemanager.largeinstance)
Go/Golang: sdk/resourcemanager/largeinstance/armlargeinstance
JavaScript: @azure/arm-largeinstance
Python: azure-mgmt-largeinstance

Thank you!

According to the description of the service, my suggestion is:
.NET: Azure.ResourceManager.BareMetalInstance
Java: azure-resourcemanager-baremetalinstance(com.azure.resourcemanager.baremetalinstance)
Go/Golang: sdk/resourcemanager/baremetalinstance/armbaremetalinstance
JavaScript: @azure/arm-baremetalinstance
Python: azure-mgmt-baremetalinstance

@ArthurMa1978 Thanks for the suggestion!
Since this is also a new namespace and there are already sdks for bare metal infrastructure, wouldn't a separate name as in AzureLargeInstance be okay for the sdk names?

Yup Azure Large Instance is the new name we use to refer to our service for new customers. I'm not sure I understand your question, in the API spec this same name is used?

@ArthurMa1978,
Can you explain why you think the libraries should use the BareMetalInstance name? It looks like Azure Large Instance is the naming used in the API Spec.
@8Gitbrix,
One note on your proposed namespaces. It doesn't look like you kept the casing from the samples. Only .NET uses upper case letters. All other languages should be all lower case letters. Also, we don't reuse the Azure name in the libraries. For example, in .NET, instead of Azure.ResourceManager.AzureLargeInstance it should just be Azure.ResourceManager.LargeInstance.

@ronniegeraghty Thanks for catching that. Would it be okay for us to use Azure in the name? As a part of the rebranding from BMI, our team specifically wants Azure in our name to make it clear to customers that this is an Azure cloud product instead of a separate one.

@ronniegeraghty Thanks for catching that. Would it be okay for us to use Azure in the name? As a part of the rebranding from BMI, our team specifically wants Azure in our name to make it clear to customers that this is an Azure cloud product instead of a separate one.

The Azure at the top level of the namespaces should accomplish this right? We find repeating the Azure name again in the lowest level of the namespace is repetitive and makes the namespaces unnecessarily longer.

I see. We'd still like Azure in the sdk name. Scott Guthrie and our marketing team have been pushing for this

@ArthurMa1978, Can you explain why you think the libraries should use the BareMetalInstance name? It looks like Azure Large Instance is the naming used in the API Spec.
@8Gitbrix, One note on your proposed namespaces. It doesn't look like you kept the casing from the samples. Only .NET uses upper case letters. All other languages should be all lower case letters. Also, we don't reuse the Azure name in the libraries. For example, in .NET, instead of Azure.ResourceManager.AzureLargeInstance it should just be Azure.ResourceManager.LargeInstance.

My suggestion is based on the introduction of the service https://learn.microsoft.com/en-us/azure/baremetal-infrastructure/concepts-baremetal-infrastructure-overview, and BareMetal is a well-known name in the industry, but I'm ok for LargeInstance:
.NET: Azure.ResourceManager.LargeInstance
Java: azure-resourcemanager-largeinstance(com.azure.resourcemanager.largeinstance)
Go/Golang: sdk/resourcemanager/largeinstance/armlargeinstance
JavaScript: @azure/arm-largeinstance
Python: azure-mgmt-largeinstance

To clarify will we still be able to use Azure in the namespace name? As a part of the rebranding from BMI to ALI, Scott Guthrie and out team specifically want Azure to be in the namespace, as in AzureLargeInstance

@8Gitbrix,
I'm going to start an email thread with our architect team where we can discuss further and the architects can share their concerns.

@ronniegeraghty Given the low customer impact and upon discussion with my team we decided to go ahead with the suggestion to use LargeInstance. I have updated the description.

@8Gitbrix, alright then I will start the second phase of the review process. The new names will be shown to the Azure SDK Architects, and they will have 1 week to make any objections to the names. I don't foresee there being any objections since most of them suggested we use these names. If there are no objections or all objections are handled by EoB on 10/27 then the names will be considered approved, and I will add a comment to this issue stating such.

The 1 week is up and there have been no objections from our architects on the names in the description of this issue. Please consider the proposed namespaces approved.


Cannot use more than one table per page

When i use two of these on one page the second doesn't render properly.

Hmm, seems to be working for me in the demos and in here: http://jsbin.com/yekaru/edit?html,output
Can you modify the jsbin so that the issue reproduces?

true, I wasn't able to get the jsbin to do it, it must be something else that is causing the second table to appear when i remove the first. I'll re-report if I can get some more specifics


Basic file logger

The built-in Screen logger is fantastic for outputting log messages to the console but what if the user wants to output them to a file instead? Yes, they can use MonoLog but that requires typing stuff into composer and waiting. I think we can provide alot of value by way of user experience by including a basic PSR-3 compatible file logger with the library. The logger should be able to create a new file if not present and provide the option to either append the log from the end or to start a new log each time. In addition, it should provide a timestamp and an indication of what type of message (INFO, ERROR, WARNING, etc.) with each line.
Are there edge cases we need to worry about? Should we lock the file used for writing to?

Decided this is not a good idea at this time

Decided this is not a good idea at this time


Allow to specify the detection_threshold in the prediction method of ObjectDetector

🚀 Feature
Allow in the method predict of the ObjectDetector to specify the parameter detection_threshold that is used to filter the bounding box in base of the confidence of the model
Motivation
In Icevision is possible to specify the detection_threshold that is used to filter the bounding box predicted by the model. If it's not specified the value is set to default at 0.5. Instead using ObjectDetector is not allow to specify this parameter and then the prediction are always filtered using the default threshold
Pitch
Allow to specify a custom value for the threshold used to filter the bounding boxes
Alternatives
Additional context
This is a screenshot of the predict method used in Icevision. If you agree that is can be useful to add this functionality I can work on it and open a PR

Hi @Actis92 Yes I think this would be a very welcome addition! Here is the line that calls predict_from_dl, you could pass the additional kwarg there (which I think should be a property of the model set on init, maybe with a setter / getter in case people want to change it): https://github.com/PyTorchLightning/lightning-flash/blob/a0c97a39f2083b5344a08d248ccab7e5bfa91df4/flash/core/integrations/icevision/adapter.py#L201
Let me know if you have any issues implementing it :smiley:

Hi @ethanwharris I have done the changes, I think in the same way as you have suggested. I have opened a pull request, so let me know if you think it's ok or must be done in a different way :)


Descew multiple page .tif image file.  (x64 bit machine)

How to Deskew multiple page .tif image file? The below-given example Deskew first page only.  (x64 bit machine)
    public void DescewTest()
    {
        var sourcePixPath = TestFilePath(@"processing\multi-page.tif");
        using (var sourcePix = Pix.LoadFromFile(sourcePixPath))
        {
            Scew scew;
            using (var descewedImage = sourcePix.Deskew(new ScewSweep(range: 45), Pix.DefaultBinarySearchReduction, Pix.DefaultBinaryThreshold, out scew))
            {
                SaveResult(descewedImage, "descewedImage.tif");
            }
        }
    }

Thanks in advance.
Sample .tif image file attached.
multi-page.zip

Try using PixArray to load the multi page tiff and you should be able to
descew pages individually. I haven't tried it from memory though.
On Sun, 17 Jul 2022, 16:43 Pulkitsoft LLP, @.***> wrote:

How to Deskew multiple page .tif image file? The below-given example
Deskew first page only.
public void DescewTest()
{
    var sourcePixPath = TestFilePath(@"processing\multi-page.tif");
    using (var sourcePix = Pix.LoadFromFile(sourcePixPath))
    {
        Scew scew;
        using (var descewedImage = sourcePix.Deskew(new ScewSweep(range: 45), Pix.DefaultBinarySearchReduction, Pix.DefaultBinaryThreshold, out scew))
        {
            SaveResult(descewedImage, "descewedImage.png");
        }
    }
}

Thanks in advance.
Sample .tif image file attached.
multi-page.zip
https://github.com/charlesw/tesseract/files/9127141/multi-page.zip
—
Reply to this email directly, view it on GitHub
https://github.com/charlesw/tesseract/issues/613, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB7HSCRUCNAEFWU4A3STG3VUOTPHANCNFSM53ZIDOSA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@charlesw Thanks for your quick response, I am not a C# expert, can you please edit this code that works for multiple .tif image files.
public void DescewTest()
{
var sourcePixPath = TestFilePath(@"processing\multi-page.tif");
using (var sourcePix = Pix.LoadFromFile(sourcePixPath))
{
Scew scew;
using (var descewedImage = sourcePix.Deskew(new ScewSweep(range: 45), Pix.DefaultBinarySearchReduction, Pix.DefaultBinaryThreshold, out scew))
{
SaveResult(descewedImage, "descewedImage.png");
}
}
}
Thanks in advance.

Here's an example from the tests:
[Test]
public void CanParseMultipageTif()
{
using (var engine = CreateEngine()) {
using (var pixA =
PixArray.LoadMultiPageTiffFromFile(TestFilePath("./processing/multi-page.tif")))
{
int i = 1;
foreach (var pix in pixA) {
using (var page = engine.Process(pix)) {
var text = page.GetText().Trim();
string expectedText = String.Format("Page {0}", i);
Assert.That(text, Is.EqualTo(expectedText));
}
i++;
}
}
}
}
Sorry about formatting, on phone 🙂 Anyway you should be able to swap out
the OCR code with descew opp instead
On Mon, 18 Jul 2022, 01:44 Pulkitsoft LLP, @.***> wrote:

@charlesw https://github.com/charlesw Thanks for your quick response, I
am not a C# expert, can you please edit this code that works for
multiple .tif image files.
public void DescewTest()
{
var sourcePixPath = TestFilePath(@"processing\multi-page.tif");
using (var sourcePix = Pix.LoadFromFile(sourcePixPath))
{
Scew scew;
using (var descewedImage = sourcePix.Deskew(new ScewSweep(range: 45),
Pix.DefaultBinarySearchReduction, Pix.DefaultBinaryThreshold, out scew))
{
SaveResult(descewedImage, "descewedImage.png");
}
}
}
Thanks in advance.
—
Reply to this email directly, view it on GitHub
https://github.com/charlesw/tesseract/issues/613#issuecomment-1186550115,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB7HSC2D6V23YVJWLTMZULVUQS4FANCNFSM53ZIDOSA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Unsupported Lat/Lon

I am getting Unsupported Lat/Lon error even though I am in Australia -34 138

just updated to latest problem solved

just updated to latest problem solved


rac dropzone fix file upload dialog

Fixes file upload dialog that was found during testing. It should no longer tab through the input twice.
Also removes some comments and unused stuff
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:
Test FileTrigger Button, FileTrigger Link
you should only be able to tab to either the button/link and not the input

🧢 Your Project:

Possibly pre-existing but we have an aria violation I just noticed in https://reactspectrum.blob.core.windows.net/reactspectrum/f3c94103d23eabd8f3a2d66de7220e7c9d397829/storybook/index.html?path=/story/react-aria-components--dropzone-example-with-file-trigger-link&providerSwitcher-express=false
the hidden button is aria-labelledby a non-existent element

Possibly pre-existing but we have an aria violation I just noticed in https://reactspectrum.blob.core.windows.net/reactspectrum/f3c94103d23eabd8f3a2d66de7220e7c9d397829/storybook/index.html?path=/story/react-aria-components--dropzone-example-with-file-trigger-link&providerSwitcher-express=false the hidden button is aria-labelledby a non-existent element

ahh yeah i'm pretty sure that's pre-existing. i think we might need to rethink some of the labeling stuff within dropzone


feat: Automatic assignment tracking

Summary
Added auto-assignment tracking feature
Checklist

[X] Does your PR title have the correct title format?
Does your PR have a breaking change?:  No

Should we support multi-instance for the server-side SDK? @justin-fiedler I found we support multi-instance in Java but not Python or Go. Maybe it's ok to leave it for now?


Automatically file a JIRA ticket for AWS events.

You need a package-info.java with ExportPackage annotation (see other packages in this module for example).


Fix Issue 7006 - std.math.pow (integral, integral) crashes on negative exponents

I improved the docs too, including the public unittest.

I do not understand, why this always fails on ci/circleci. First I thought it is unrelated, but meanwhile the error was reproduced with each run: std/range/package.test fails. When I run that test locally it succeeds (but needs about 6 minutes). What does ci/circleci actually do?

Circle is green now, but mir/algorithm still fails consistently.

Circle is green now, but mir/algorithm still fails consistently.

Found (and fixed) the problem: m <= -1 is not a good idea, when m is unsigned. I added also a unittest (assert(pow(0,uint.max) == 0);) to guard for that problem. (There are two more expressions I put into static if, but that is mere for extra security; IMHO they could not break and therefore I could not add a guarding unittest for them.)


tag file managment from UI

Would be useful if admins could manage tag files from the UI (aka, see what tag files are loaded, when they were loaded, what version they are, etc)

If talking about WISE then #548 if talking about the old tags table that is gone since 1.x


html homepage with h1 and css with h1 style

This req is about H1 on the html and the styles added on the css

Thanks For Your Response
I have updated the ReadMe.md with my info.
Sincerely
On Tue, Feb 8, 2022, 8:04 PM Kerolous Samy Grace @.***>
wrote:

@.**** requested changes on this pull request.
Hi @sentayhu19 https://github.com/sentayhu19,
Good job so far!
There are some issues that you still need to work on to go to the next
project but you are almost there!
Required Changes ♻
Check the comments under the review.
https://camo.githubusercontent.com/6ac905aa3a075ef45de55758c20c6e6efb1dfdbbb0d9fd03d4718d26c5c96cde/68747470733a2f2f6d65646961322e67697068792e636f6d2f6d656469612f4a713832345239334a734c775a436169534c2f67697068792e676966
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop
the approval of this PR. However, I strongly recommend you to take them
into account as they can make your code better.
You can also consider:

[OPTIONAL] You can make every change in a separate commit.

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In README.md
https://github.com/sentayhu19/Hello-Microverse/pull/1#discussion_r801858744
:

+# Project Name

+> Description the project.

+## Built With

+- Major languages
+- Frameworks
+- Technologies used
Please here you can add your project name and a description and the
technologies that you used in this project.
In README.md
https://github.com/sentayhu19/Hello-Microverse/pull/1#discussion_r801860324
:

+## Authors

+👤 Author1

+- GitHub: @.***(https://github.com/githubhandle)
+- Twitter: @.***(https://twitter.com/twitterhandle)
+- LinkedIn: LinkedIn

+👤 Author2

+- GitHub: @.***(https://github.com/githubhandle)
+- Twitter: @.***(https://twitter.com/twitterhandle)
+- LinkedIn: LinkedIn
And here please you can add your links and delete the other author.
—
Reply to this email directly, view it on GitHub
https://github.com/sentayhu19/Hello-Microverse/pull/1#pullrequestreview-876356180,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASRE4335VS5X7EGTOFII4ATU2FEKNANCNFSM5N3AXDYQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

@keroloussamy
Thanks For Your Response
I have updated the ReadMe.md with my info.
Sincerely
On Tue, Feb 8, 2022 at 9:08 PM Sentayhu Berhanu @.***>
wrote:

Thanks For Your Response
I have updated the ReadMe.md with my info.
Sincerely
On Tue, Feb 8, 2022, 8:04 PM Kerolous Samy Grace @.***>
wrote:

@.**** requested changes on this pull request.
Hi @sentayhu19 https://github.com/sentayhu19,
Good job so far!
There are some issues that you still need to work on to go to the next
project but you are almost there!
Required Changes ♻
Check the comments under the review.
https://camo.githubusercontent.com/6ac905aa3a075ef45de55758c20c6e6efb1dfdbbb0d9fd03d4718d26c5c96cde/68747470733a2f2f6d65646961322e67697068792e636f6d2f6d656469612f4a713832345239334a734c775a436169534c2f67697068792e676966
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop
the approval of this PR. However, I strongly recommend you to take them
into account as they can make your code better.
You can also consider:

[OPTIONAL] You can make every change in a separate commit.

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if
something is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
Please, do not open a new Pull Request for re-reviews. You should use
the same Pull Request submitted for the first review, either valid or
invalid unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In README.md
https://github.com/sentayhu19/Hello-Microverse/pull/1#discussion_r801858744
:

+# Project Name

+> Description the project.

+## Built With

+- Major languages
+- Frameworks
+- Technologies used
Please here you can add your project name and a description and the
technologies that you used in this project.
In README.md
https://github.com/sentayhu19/Hello-Microverse/pull/1#discussion_r801860324
:

+## Authors

+👤 Author1

+- GitHub: @.***(https://github.com/githubhandle)
+- Twitter: @.***(https://twitter.com/twitterhandle)
+- LinkedIn: LinkedIn

+👤 Author2

+- GitHub: @.***(https://github.com/githubhandle)
+- Twitter: @.***(https://twitter.com/twitterhandle)
+- LinkedIn: LinkedIn
And here please you can add your links and delete the other author.
—
Reply to this email directly, view it on GitHub
https://github.com/sentayhu19/Hello-Microverse/pull/1#pullrequestreview-876356180,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASRE4335VS5X7EGTOFII4ATU2FEKNANCNFSM5N3AXDYQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hello @sentayhu19, when you change the requirements submit the same pull request again in your dashboard so any reviewer will review it for you.


One Css file per font

Hi,
thanks for this lib. I have a suggestion. If you use tools like Webpack or Browserify you may want to only get one type of font and not all of them.
With webpack if you require('roboto-fontface') you will automatically get all font files (kinda big).
I suggest you separate every type of font in different css files then we will be able to only require what we need like this:  require('roboto-fontface/css/Roboto-ThinItalic.css')

That is true. But I would still like to keep the all-in-one files. Can you create a PR that adds the indiviual CSS files?

Do not really have time for that sorry, just keep it in mind ;)

+1 here. :)

I've created PR.

Now it is one file per weight, that includes both straight and italic, but if you think they should be separate, too, let me know.

Thx @mspanc for implementing this! Have released a new version 0.4.4 to bower and npm.

And 0.4.5, because I forgot to update bower.json in 0.4.4 :)


feat: poc

Resolves #1

Coverage remained the same at 100.0% when pulling feda861b3de2f46cb1cc6f4901dd8d1b3150a9c1 on feat/poc into ac31418b502284f7bd533eb985f02670dadb5565 on master.


Add Client::invoke_service_http

Add ability to specify:

HTTP method
Content-Type
Queries

It is currently a very stringly-typed API but I am not sure if there's even an attempt to enumerate all the possible Content-Type.
Other than that, I am not sure if passing the queries directly as a String is correct here, because
we want to utilize stronger correctness with something like Iterator<Item=(String,String)> instead.
But this does give the freedom to the user to decide how they want to allocate the string.
Closes #71
Signed-off-by: Hanif Ariffin<EMAIL_ADDRESS>

@hbina I think we're trying to encourage users use native HTTP clients for service invocation. As a user from a Rust perspective, would you feel more comfortable using an HTTP client or the new method you're adding here?


Extreme lag from GeyserModelEngine

https://spark.lucko.me/OjrT4Tu4oU

try update

try update

https://spark.lucko.me/dSNquUNUmD no luck

huh strange, how many meg entities are spawned? did you try disable GeyserModelEngine?

try update again

try update again

Hey i wont be able to test as my dev was able to make a version of it fixed for me and I do not want to have to deal with it again. Hopefully someone else can test.

I decided to test the latest update. No lag so far - however, no bedrock players can see any models anymore.

update geyserutils-geyser

update geyserutils-geyser

Still bedrock can see no models.

update GeyserModelEngine again?

wait

GeyserModelEngine.zip
try this

Seems good!


Null Hypothesis Not Rejected as Expected

Working with this example, with lines 24-25 uncommented so that Weight is defined as an ordinal variable.
When using this hypothesis:
results = tea.hypothesize(['Sport', 'Weight'], ['Sport:Wrestling > Swimming'])
the null hypothesis is rejected as expected, with the following output:
The median of Weight for Sport = Wrestling is significantly greater than the median for Sport = Swimming.
We expect that flipping the inequality would also result in rejection of the null hypothesis:
results = tea.hypothesize(['Sport', 'Weight'], ['Sport:Wrestling < Swimming'])
but instead it produces the following output:
There is no difference in medians between Sport = Swimming and Sport = Wrestling on Weight.
Note: The values of test statistic, p value, alpha, dof, effect size, etc. are the same in the output from both hypotheses.

I have debugged the code and this is what I found:
The line below calcualted 1-pvalue for 'less than' prediction but Mann–Whitney U test returns identical p-value for both examples.
https://github.com/emjun/tea-lang/blob/a93dc6ad249d9519c6b99b8af034b3e319e216a2/tea/runtimeDataStructures/testResult.py#L184
Note that we are doing two-sided MW test
https://github.com/emjun/tea-lang/blob/a93dc6ad249d9519c6b99b8af034b3e319e216a2/tea/helpers/evaluateHelperMethods.py#L484

Looking into this further, I don't think this is a bug.
We would not expect that both those inequalities would reject the null hypothesis because we are testing two different hypotheses.
By expressing a one-sided hypothesis, we are not looking for effects in the other direction.

If our hypothesis is Sport:Wrestling > Swimming we are only sensitive to the median of Wrestling being greater than the median of Swimming. We do not "look for" the opposite.
If our hypothesis is Sport:Wrestling < Swimmingwe are only sensitive to the median of Wresting being less than the median of Swimming. We do not "look for" the opposite.

What is confusing is that the null hypothesis is the same for one-sided (greater or lesser) and two-sided hypotheses. The null hypothesis is that there is no difference in medians.
I find this page helpful in providing general background and a similar example: https://stats.idre.ucla.edu/other/mult-pkg/faq/general/faq-what-are-the-differences-between-one-tailed-and-two-tailed-tests/

I think this is a great example of limitations of current output for Tea and many other tools. @meli365


How to get the file name when detect on potential vulnerable files as well?

When we found "potential vulnerable files", I found the output only gives the number of found, but no file name and its path.

I do get file list from "vulnerable files", but not from "Potential vulnerable files"
Any options to enable this feature?

this is the test output
sh-4.2$ ./log4j2-scan --trace --debug /home
Logpresso CVE-2021-44228 Vulnerability Scanner 1.6.3 (2021-12-16)
Scanning directory: /home
Scanning directory: /home
Scanning directory: /home/ec2-user
Scanning directory: /home/ssm-user
Skipping file: /home/ssm-user/.bash_logout
Skipping file: /home/ssm-user/.bash_profile
Skipping file: /home/ssm-user/.bashrc
Scanning directory: /home/ssm-user/test1
Skipping file: /home/ssm-user/test1/log4j2-scan.zip
Skipping file: /home/ssm-user/test1/log4j2-scan.exe
Skipping file: /home/ssm-user/test1/a.tar.gz
Scanning directory: /home/ssm-user/test1/log4j-2.15
Scanning file: /home/ssm-user/test1/log4j-2.15/log4j-core-2.15.0.jar
[*] Found CVE-2021-45046 vulnerability in /home/ssm-user/test1/log4j-2.15/log4j-core-2.15.0.jar, log4j 2.15.0
Scanning directory: /home/ssm-user/test1/log4j-1.2
Scanning file: /home/ssm-user/test1/log4j-1.2/log4j-1.2.15.jar
Scanning file: /home/ssm-user/test1/log4j-1.2/log4j-1.2.15-noJMS.jar
Scanning directory: /home/ssm-user/test1/log4j-2.10
Scanning file: /home/ssm-user/test1/log4j-2.10/log4j2-demo-1.0.0-12.jar
[*] Found CVE-2021-44228 vulnerability in /home/ssm-user/test1/log4j-2.10/log4j2-demo-1.0.0-12.jar (BOOT-INF/lib/log4j-core-2.10.0.jar), log4j 2.10.0
Skipping file: /home/ssm-user/test1/._log4j2-scan
Skipping file: /home/ssm-user/test1/logpresso-log4j2-scan-1.6.3-linux-customized.tar.gz
Skipping file: /home/ssm-user/test1/log4j2-scan

Scanned 7 directories and 13 files
Found 2 vulnerable files
Found 2 potentially vulnerable files
Found 0 mitigated files
Completed in 0.57 seconds

so from the output, there is no print out for potentially vulnerable files.
But in your code:
https://github.com/logpresso/CVE-2021-44228-Scanner/blob/v1.7.0/src/main/java/com/logpresso/scanner/Log4j2Scanner.java#L944-L962
it sets different output style
if vulnerable files, use [*]
if potential files, use [?]
Since there is no file to be list when it is potential files, I didn't get any [?] Found  records

@ozbillwang Would you test latest version (v2.3.4)? I want to know your problem still exists.

@xeraph
run with latest version 2.3.6, still no output to potential files
sh-4.2$ ls -l ~/test1
total 0
drwxr-xr-x 2 ssm-user ssm-user 60 Dec 17 14:14 log4j-1.2
drwxr-xr-x 2 ssm-user ssm-user 38 Dec 17 14:14 log4j-2.10
drwxr-xr-x 2 ssm-user ssm-user 35 Dec 17 14:14 log4j-2.15
sh-4.2$ ./log4j2-scan /
Logpresso CVE-2021-44228 Vulnerability Scanner 2.3.6 (2021-12-20)
Scanning directory: / (without devtmpfs, tmpfs)
[*] Found CVE-2021-45046 (log4j 2.x) vulnerability in /home/ssm-user/test1/log4j-2.15/log4j-core-2.15.0.jar, log4j 2.15.0
[*] Found CVE-2021-44228 (log4j 2.x) vulnerability in /home/ssm-user/test1/log4j-2.10/log4j2-demo-1.0.0-12.jar (BOOT-INF/lib/log4j-core-2.10.0.jar), log4j 2.10.0

Scanned 5427 directories and 46112 files
Found 2 vulnerable files
Found 0 potentially vulnerable files
Found 0 mitigated files
Completed in 0.91 seconds

It works. Thanks.

@ozbillwang Thank you for test report!


Declaring a variable in a for loop

This code produces a syntax error:
for (int i = 0; i<42; i++) {
    /* ... */
}

Such replacement works, but it's definitely uncomfortable to do manually:
int i;
for (i = 0; i<42; i++) {
    /* ... */
}

This program is meant to handle the code found in the Go compiler (a C program). It is not a general C program. There's no intent to handle narrowly scoped variables well, for example.


Request to add tagged Releases

Hi @ciaranrobrien,
Thanks so much for creating a great masonry view. It's working really well for us!
Would you be up for adding a tag with a version number in GitHub? There's a nice article documenting it here: https://appracatappra.com/2021/07/releasing-and-versioning-your-swift-package/#Versioning-your-Swift-Package-on-GitHub. It should be pretty simple. The advantage is that we can then point to a specific version and get to chose to update at a time that suits us rather than whatever's the latest.
Thanks again!
Harry

For anyone coming across this in the future, the closest alternative is probably using the commit dependency rule:


added single filtering

Signed-off-by: Aditya Chatterjee<EMAIL_ADDRESS>Description
Added single field filtering, haven't completed doing all pages yet still working on this PR.
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

From some reason the radio buttons are functioning, but not showing as unchecked.
https://user-images.githubusercontent.com/7570704/148150150-88604fe7-b038-46c5-90be-c227fde61756.mov

Yes @leecalcote I'm working on fixing that.

@leecalcote could you please take another look?

Let's merge for now @leecalcote we can again work on refactoring later on.


Update install-unifi.sh

update unifi to 5.13.29
Update python2 to python37 (resolve new install)
Update mongod34 to mongod36 (resolve new install)

Thank you


Multiline Textbox Word Wrap

I'm sure this is low priority, but it would be nice if Multi-line Textboxes had working word wrap functionality. as it currently stands it'll just keep letting you type past the end of the box. All text is still there though so no real bug other than you can't see it.

I think we'll definitely want to support this.  It just complicates some of the textinput logic, so might take a bit to come up with the right solution.  👍


MINOR: Set JVM parameters for the Gradle Test executor processes

We suspect that the test suite hangs we have been seeing are
due to PermGen exhaustion. It is a common reason for
hard JVM lock-ups.

Review by @hachikuji.

@hachikuji I updated the PR to only add MaxPermSize if the JDK version is lower than Java 8.

LGTM. Merging to trunk, 0.10.1 and 0.10.0.


Can we use multi select with async MultiSelectorItems fetched asynchronously based on what user typed in the input

Hello :)
Can we use the multi-select component with MultiSelectorItems fetched asynchronously based on what is typed in the input?
I tried, but the component doesn't display the SelectorItem.
I added a prop to MultiSelectorInput to retrieve the string typed in the input in the parent component. I use this string to make an asynchronous request, and I set the result in a useState which is then used for my MultiSelectorItems, but it's not working.
This is my code :
export const AdsFiltersGameCity = () => {
  const [data, setData] = useState<string[]>([]);
  const [inputValue, setInputValue] = useState<string>("");

  const { search } = useCitySearch();

  const onCitiesChanged = (newValues: string[]) => { };

  const [_, cancel] = useDebounce(
    async () => {
      const res = await search(inputValue);
      setData(res);
    },
    400,
    [inputValue]
  );

  return (
      <MultiSelector
        values={cities}
        onValuesChange={onCitiesChanged}
        loop={false}
      >
        <MultiSelectorTrigger formatLabel={formatLabel}>
          <MultiSelectorInput setInputValueExternal={setInputValue} />
        </MultiSelectorTrigger>
        <MultiSelectorContent>
          <MultiSelectorList>
            {data.map((option, i) => (
              <MultiSelectorItem key={option} value={option}>
                {option}
              </MultiSelectorItem>
            ))}
          </MultiSelectorList>
        </MultiSelectorContent>
      </MultiSelector>
  );
};

Best regards,
Mickaël

Hi @MickaelMesnage.
Responding to your question, I think it is possible to implement search behaviour on the multi select, u just need to give the onValueChange prop the function that will handle the behaviour that you want to do after u make the search.
An example on this is what we're doing for the command search on the user end side on shadcn extension, you can find it here.
Let us know how it goes.
Best regards,
Belkacem

I'm facing the same issue this is my code

const [searchTerm, setSearchTerm] = useState<string>("");
  const debouncedSearchTerm = useDebounce(searchTerm, 500);

  const searchedTagsData = useTypedSelector(selectFilteredUsersFromTags);

useEffect(() => {
    // Trigger new query when debounced search term changes
    setQuery((prevQuery) => ({
      ...prevQuery,
      page: 1, // Reset to first page
      fullName: debouncedSearchTerm, // Set search term
    }));
    appDispatch(setUserSearchFromContactsData(debouncedSearchTerm));
  }, [debouncedSearchTerm])

<FormField
              control={form.control}
              name="users"
              render={({ field }) => (
                <FormItem className="w-[30rem]">
                  <MultiSelector
                    onValuesChange={field.onChange}
                    values={field.value}
                  >
                    <MultiSelectorTrigger
                      data={searchedTagsData}
                      // @ts-ignore
                      onChange={(event) => setSearchTerm(event.target.value)}
                    >
                      <MultiSelectorInput placeholder="Select the contacts"  onValueChange={setSearchTerm}/>
                      <ChevronDown className="text-slate-400" />
                    </MultiSelectorTrigger>
                    <MultiSelectorContent>
                      <MultiSelectorList className="scrollbar scrollbar-thumb-primary">
                        {searchedTagsData.map((contact) => (
                          <MultiSelectorItem
                            key={contact.value}
                            value={contact.value}
                          >
                            {contact.label}
                          </MultiSelectorItem>
                        ))}

                      </MultiSelectorList>
                    </MultiSelectorContent>
                    <FormMessage className="text-center" />
                  </MultiSelector>
                </FormItem>
              )}
            />

I can confirm the results are coming but the list doesn't render it's showing no results found. if i clear the input then type again the same search term i got the results


[FEATURE REQUEST]

how do i make this program able to output response body html and write to html like google.com.html
tiktok.com.html
i need it to view source code of web page,
is there any solution?

to download the source, you'll need another tool. I prefer using https://github.com/tomnomnom/fff
feroxbuster -u https://bitdiscovery.com -w tests/extra-words --silent | fff -s 200 -o source-files

how about i edit your source code,
i really like this program , can you tell me which file i should edit to get source code output.html

i'd accept a PR, but it would be quite a bit more effort than a single file edit. below is a list of things that go into adding a new flag/option to feroxbuster. if you still want to work on this, i can help you figure out where best to implement the feature as well.
- Add new command line option checklist
    - config/container.rs
        - add value to `Configuration` struct
        - add value to `Configuration::default`
        - update `Configuration` docstring
        - add value to `Configuration::parse_cli_args`
        - add value to `Configuration::merge_config`
    - config/tests.rs
        - update `Configuration::tests::setup_config_test` with new value
        - update `Configuration::tests::default_configuration` with new value
        - add test to check config parsing
    - ferox-config.toml.example
        - add new value
    - banner/container.rs
        - add new `BannerEntry` to `Banner` struct
        - update `Banner::new` with a `BannerEntry`
        - update `print_to` with whether to print or not
    - tests/test_banner.rs
        - add new test so that the new `BannerEntry` is displayed and examined under test
    - Cargo.toml
        - bump version - minor release +1 - patch release = 0
    - README.md
        - update default config section (if needed)
        - update example config file section
        - update help output section
        - consider adding to an example (new or existing)
        - update comparison table
    - parser.rs
        - add the option
    - scan_manager/tests.rs
        - add serialized version of new option to the feroxstates_feroxserialize_implementation test
    - wherever.rs
        - implement the new feature

``Thank you for your help, hopefully useful ,I will do it ,


Use the new side effect analysis in the performance inliner

This is the first part of replacing the old escape- and side-effect analysis. The second part is here: https://github.com/apple/swift/pull/61715
The inliner uses side-effect analysis to check if a function is a pure function (used for the inlining heuristics).
This PR also contains a few bug fixes for the ComputeSideEffects pass.

@swift-ci test

@swift-ci benchmark


slow download speeds

Before submitting the issue

[ ] I have checked that the bug is not a duplicate or, if it is, it needs to be reopened
[ ] I have checked that I am running the latest version of WingetUI
[ ] I have tried resetting and reinstalling WingetUI and the issue persisted or happened again
[ ] I have checked the FAQ section for solutions

Describe the bug
I tried to download the intellij IDE from wingetUI and the website. the download speed when downloaded from website was around 500mbps but when I tried to install from wingetUI, its only about 80 to 120mbps. Is there any fix for this?
To Reproduce
Steps to reproduce the behaviour:

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

Expected behaviour
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
The Log
Please paste here the log (Go to the three dots menu on the top-right corner of WingetUI -> Winget Log.

If the issue can be reproduced, please send the log AFTER reproducing the issue (1. Launch WingetUI, 2. Reproduce the issue, 3. Paste the log)

Additional context
Add any other context about the problem here.

No really, this is an issue with Winget-cli.
Maybe open an issue there?


fix: App canvas is stuck in loading state ( Desktop Layout )

this PR fixes the following bug:

This app is stuck in a loading state and I'm unable to edit the app. The app can be viewed in view mode though

Fixes #14381
Type of change

Bug fix

How Has This Been Tested?

Manually tested

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

/ok-to-test sha=e794bea


The language switch on the frontend doesn't work

Description
When I try to add the ability to change the language to the page, after saving the changes, the following error appears on the store.

I change the language in the admin panel using the tutorial available on youtube: https://www.youtube.com/watch?v=PJ1YnNgJeaw

Hey @joanna-gil,
could you try to save the domain for the language with a trailing slash probably http://localhost:8000/de/.
Does that solve the problem for you?
However we are looking into this.

Sorry for the problem, I found a mistake in my side - I didn't change the security level from https to http, therefore it couldn't find the right location


Blit Shader is null post-build

I'm using the KlakSpout v2.0.3, and it's installed via Scoped Registries, alongside Klak NDI.
Unity Version 2020.3.33f1
Target: Windows x64
I'm receieving the following error when I build my project:
NullReferenceException: Object reference not set to an instance of an object   at Klak.Spout.Blitter.GetMaterial (Klak.Spout.SpoutResources resrc) [0x00011] in F:\1_Unity_Projects\Toolkit Workpsace\Library\PackageCache\jp.keijiro.klak.spout@2.0.3\Runtime\Internal\Utility.cs:45    at Klak.Spout.Blitter.Blit (Klak.Spout.SpoutResources resrc, UnityEngine.Texture src, UnityEngine.RenderTexture dst, System.Boolean alpha) [0x00000] in F:\1_Unity_Projects\Toolkit Workpsace\Library\PackageCache\jp.keijiro.klak.spout@2.0.3\Runtime\Internal\Utility.cs:25    at Klak.Spout.SpoutSender.Update () [0x00099] in F:\1_Unity_Projects\Toolkit Workpsace\Library\PackageCache\jp.keijiro.klak.spout@2.0.3\Runtime\SpoutSender.cs:112
I've added the Spout Blit shader to the Built In Shader's section, which I thought was the fix when similar errors have occured before, however it still doesn't function.

Is there something I'm missing? did something not install correctly?

You have to give SpoutResources to the component. Please check the "Scripting interface" section in README.


how to write the file of .params ever 10 epoch when using model.fit()

I use the default solver "model.fit()" ,and  in it I set up the save parameter "epoch_end_callback=mx.callback.do_checkpoint(prefix)" .By default ,it writes out the file of .params for every epoch. However ,I want  it to write out the file of .params every 10 epoch or evey 20 epoch.And how to modify the parameters?
Thank you so much.

The following code is from mxnet/callback.py
You can see that there is a period argument.
def do_checkpoint(prefix, period=1):
    """Callback to checkpoint the model to prefix every epoch.

    Parameters
    ----------
    prefix : str
        The file prefix to checkpoint to
    period : int
    	How many epochs to wait before checkpointing. Default is 1.

    Returns
    -------
    callback : function
        The callback function that can be passed as iter_end_callback to fit.
    """
    period = int(max(1, period))
    def _callback(iter_no, sym, arg, aux):
        """The checkpoint function."""
        if (iter_no + 1) % period == 0:
            save_checkpoint(prefix, iter_no + 1, sym, arg, aux)
    return _callback

@magic282  thank you .But when I wrote the the second argument,I got an error:
=========================================

prefix='aa'
p=10
cp=mx.callback.do_checkpoint(prefix,p)
Traceback (most recent call last):
File "", line 1, in
TypeError: do_checkpoint() takes exactly 1 argument (2 given)
========================================

@magic282 thank you .But when I wrote the the second argument,I got an error:
prefix='aa'
p=10
cp=mx.callback.do_checkpoint(prefix,p)
Traceback (most recent call last):
File "", line 1, in <module>
TypeError: do_checkpoint() takes exactly 1 argument (2 given)

@lugui2009 https://github.com/dmlc/mxnet/blob/master/python/mxnet/callback.py
Are you using the latest version?

@magic282 Oh,I see.I will update my version of mxnet .Thank you so much

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


altair: 5.0.17 -> 5.0.19

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/altair/versions.
meta.description for altair is: A feature-rich GraphQL Client IDE
meta.homepage for altair is: https://github.com/imolorhe/altair
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.0.19 with grep in /nix/store/mvzj6l13krnr6k3zkh9g9bvd9g1p3wgl-altair-5.0.19
found 5.0.19 in filename of file in /nix/store/mvzj6l13krnr6k3zkh9g9bvd9g1p3wgl-altair-5.0.19

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
altair

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mvzj6l13krnr6k3zkh9g9bvd9g1p3wgl-altair-5.0.19 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A altair https://github.com/r-ryantm/nixpkgs/archive/eb7fd31ca062c57e20564380eac5167c3da9ad1b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/mvzj6l13krnr6k3zkh9g9bvd9g1p3wgl-altair-5.0.19
ls -la /nix/store/mvzj6l13krnr6k3zkh9g9bvd9g1p3wgl-altair-5.0.19/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

altair

Maintainer pings
cc @evalexpr for testing.

Result of nixpkgs-review pr 223187 run on x86_64-linux 1

1 package built:

altair


Support default dtype in vae example

Rel. #6168.
This PR fixes the VAE example to support the default dtype feature.

Jenkins, test this please.

Jenkins CI test (for commit 4df3f1638fc77f3fdf4b8798f468392375b70ac4, target branch master) failed with status FAILURE.

LGTM. Merging.


New package: AcroSoftwareInc.CutePDFWriter version 4.0

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Not sure what evidence I need to provide, but here is the link to download the software. You can see at the bottom right of the package the .zip installer.

--
Get Zipped Setup

Try now  [Get Zipped Setup](https://www.cutepdf.com/download/CuteWriter.zip)

@wingetbot waivers Add Validation-Domain

@stephengillie This program is designed to be a PDF document printer, essentially it's a replacement for Microsoft PDF Printer. If you install it, open a web browser, print the page, and select CutePDFWriter as the printer it should work.

@stephengillie This program is designed to be a PDF document printer, essentially it's a replacement for Microsoft PDF Printer. If you install it, open a web browser, print the page, and select CutePDFWriter as the printer it should work.

Thanks for helping me to understand how to use this application.


WIP: HTTPlug Symfony bundle integration doc

I dug on a lot of brainfucking before figuring out how to integrate your custom HTTPClient with the official bundle.
I made a quick documentation that I think, may help a lot for other Symfony users.
Regards.

Please wait https://github.com/php-http/HttplugBundle/issues/210 solving and this PR update before merge.

Hey, thanks for the PR, this is something that would be nice to have in the 2.0 version, can you rebase this work on 2.0 branch ?

Yeah I'll do it when I'll have time for it. :wink:

Wait a minute, this is not related to https://github.com/docker-php/docker-php/issues/255.
This is only documentation about how to integrate it with http-plug.
But a bridge and some modification must be done and yes, may be added on 2.0.
Though?

Thanks for this, will you have time to test if this is still acurate for 2.0 ?

@joelwurtz For now I can't test because of https://github.com/php-http/logger-plugin/issues/3


Submission thread seems to block on unrelated tags

I'm working in some of the future_tag code, and I'm seeing (https://github.com/boto/s3transfer/blob/develop/s3transfer/futures.py#L328)

        with self._lock:
            # Wait if there is a maximum reached
            self._wait_for_futures_to_complete_if_needed(tag)
            # Submit the task to the executor
            future = self._executor.submit(fn, *args, **kwargs)
            # Add the recently added future to the currently running futures
            self._add_future_to_currently_running(future, tag)
        return future

So suppose I have a max tags of {"A": 1, "B": 1} and sequence such as:
T1: submit tag A --> noblock
T1: submit tag A --> block on waiting futures
T2: submit tag B --> block on lock acquire

Where T1/T2 are threads and A/B are tags.  This means that even though T2 is submitting tag B which has no futures, the submission on tag B will still block on the with self._lock until a future for tag A completes.

Fixed with this: https://github.com/boto/s3transfer/pull/31. Closing.


Newer node versions (>=6) failing to compile script via Babel during execution

Description
I upgraded to RN 0.39.2 today, but had problems every time I ran the packager (via yarn start and react-native bundle [...]. The error I got was:
$ yarn start
yarn start v0.18.1
$ cd node_modules/react-native && npm start -- --reset-cache

><EMAIL_ADDRESS>start /Users/joshua/dev/work/Taxfix/taxfix-ios/TaxfixReactNative/node_modules/react-native
> /usr/bin/env bash -c './packager/packager.sh "$@" || true' -- "--reset-cache"

/Users/joshua/dev/work/Taxfix/taxfix-ios/TaxfixReactNative/node_modules/react-native/packager/react-packager/src/Logger/index.js:85
printFields? = [])
           ^
SyntaxError: Unexpected token ?
    at Object.exports.runInThisContext (vm.js:76:16)
    at Module._compile (module.js:542:28)
    at loader (/Users/joshua/dev/work/Taxfix/taxfix-ios/TaxfixReactNative/node_modules/babel-register/lib/node.js:144:5)
    at Object.require.extensions.(anonymous function) [as .js] (/Users/joshua/dev/work/Taxfix/taxfix-ios/TaxfixReactNative/node_modules/babel-register/lib/node.js:154:7)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/joshua/dev/work/Taxfix/taxfix-ios/TaxfixReactNative/node_modules/react-native/packager/react-packager/index.js:14:16)
✨  Done in 1.67s.

Reproduction
Run the packager via a recent Node version:
$ node --version
v6.9.2
$ node_modules/react-native/./packager/packager.sh --reset-cache
/Users/joshua/dev/work/Taxfix/taxfix-ios/TaxfixReactNative/node_modules/react-native/packager/react-packager/src/Logger/index.js:85
printFields? = [])
           ^
SyntaxError: Unexpected token ?
    at Object.exports.runInThisContext (vm.js:76:16)
    at Module._compile (module.js:542:28)
    at loader (/Users/joshua/dev/work/Taxfix/taxfix-ios/TaxfixReactNative/node_modules/babel-register/lib/node.js:144:5)
    at Object.require.extensions.(anonymous function) [as .js] (/Users/joshua/dev/work/Taxfix/taxfix-ios/TaxfixReactNative/node_modules/babel-register/lib/node.js:154:7)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/joshua/dev/work/Taxfix/taxfix-ios/TaxfixReactNative/node_modules/react-native/packager/react-packager/index.js:14:16)

Solution
Solution 1, in my control: Downgrade to Node 5.x:
$ node --version
v5.12.0
$ node_modules/react-native/./packager/packager.sh --reset-cache
Scanning 1086 folders for symlinks in /Users/joshua/dev/work/Taxfix/taxfix-ios/TaxfixReactNative/node_modules (14ms)
 ┌────────────────────────────────────────────────────────────────────────────┐
 │  Running packager on port 8081.                                            │
 │                                                                            │
 │  Keep this packager running while developing on any JS projects. Feel      │
 │  free to close this tab and run your own packager instance if you          │
 │  prefer.                                                                   │
 │                                                                            │
 │  https://github.com/facebook/react-native                                  │
 │                                                                            │
 └────────────────────────────────────────────────────────────────────────────┘
Looking for JS files in
[...]

Solution 2, requiring a patch on your side:
Update babelRegisterOnly.js to explicitly include the transform-es2015-parameters plugin. Proof:
# Without transform-es2015-parameters
$ ./node_modules/.bin/babel --no-babelrc --plugins transform-flow-strip-types,transform-object-rest-spread,transform-react-jsx node_modules/react-native/packager/react-packager/src/Logger/index.js | grep "function print"
function print(logEntry, printFields? = []) {

# Same as above, with transform-es2015-parameters
$ ./node_modules/.bin/babel --no-babelrc --plugins transform-flow-strip-types,transform-object-rest-spread,transform-react-jsx,transform-es2015-parameters node_modules/react-native/packager/react-packager/src/Logger/index.js | grep "function print" -A 1
function print(logEntry) {
  let printFields = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : [];

However, it seems like this should be unnecessary. I'd imagine the underlying issue is with transform-flow-strip-types not stripping the ? correctly, and that transform-es2015-parameters conveniently solves the issue. I'll look into that further, but at least these are two concrete workarounds.
Additional Information

React Native version: 0.39.2
Platform: iOS
Operating System: macOS

AFAIK the offending code has been removed in the last few days (via ede04abf8f4463155c196c6cfb8ea47684e8c6b8), but there's no reason it won't appear again and cause problems!

Okay, this looks like it's recently been addressed by the Flow team (!). PR here: https://github.com/babel/babel/pull/4872, issue here: https://github.com/babel/babel/issues/4863.
So I suppose the actual solution here is to bump the requirement for babel-plugin-transform-flow-strip-types in package.json to ^6.21.0, and verify it works.
(FWIW, it's working for me via npm install; npm start, but fails via yarn install; yarn start. I'm too tired to work out why on earth that'd happen, despite even manually overwriting the seemingly relevant packages!)

I am hitting this issue as well

same here

Me too 💯

@notjosh it's working for me too but it's the opposite here. Not working with NPM but working with Yarn.

Yarn is installing whatever version that was locked in your yarn.lock file. Npm installs whatever the latest version parameterized by your package.json file.
Sent from my iPhone

On Dec 28, 2016, at 4:26 PM, Lori Karikari<EMAIL_ADDRESS>wrote:
@notjosh it's working for me too but it's the opposite here. Not working with NPM but working with Yarn.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Solution 2 is working fine for me.
Put following plugins :
plugins: [
'transform-flow-strip-types',
'syntax-trailing-function-commas',
'transform-object-rest-spread',
'transform-es2015-parameters'
]
in babelRegisterOnly.js file.

So the only valid solution for Node >=6 is to manually edit the packager/babelRegisterOnly.js file?
Any suggestions how to wrap it up in a nicer way, so npm i won't override this? :)
Iv'e tried to update babel-plugin-transform-flow-strip-types in my package.json to 6.21.0 but it didn't help...

@notjosh
do u know of a way to do the equivalent of react-native start with the cli patch u mentioned? something like:
$ react-native start --no-babelrc --plugins transform-flow-strip-types,transform-object-rest-spread,transform-react-jsx,transform-es2015-parameters

@goldylucks no idea, sorry. I updated our dependency via yarn.lock, so it's not been a problem for us since.

@goldylucks no idea, sorry. I updated our dependency via yarn.lock, so it's not been a problem for us since.

@notjosh Is the solution to add babel dependencies the package.json of my project? If yes, which one exactly? Or do I need to upgrade my react-native version?

@wcandillon I don't know how best to do it with package.json w/ npm only. We're using Yarn, so I manually updated yarn.lock to point babel-plugin-transform-flow-strip-types to 6.21.0. For us, that block looks like:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>  version "6.21.0"
  resolved "https://registry.yarnpkg.com/babel-plugin-transform-flow-strip-types/-/babel-plugin-transform-flow-strip-types-6.21.0.tgz#2eea3f8b5bb234339b47283feac155cfb237b948"
  dependencies:
    babel-plugin-syntax-flow "^6.18.0"
    babel-runtime "^6.0.0"

Whether or not that is a Good Idea™ or not is another discussion ;)

@notjosh thank you for these details :)

@wcandillon did that work for u?
I've locked npm to the same versions as @notjosh posted and still hitting the same error :(

@notjosh @goldylucks Didn't work for me either.

@goldylucks @wcandillon I've run into this exact issue - the problem was that even when you manually fix the versions, the wrongly compiled file was already in the babel cache so it never got recompiled.

Try running BABEL_DISABLE_CACHE=1 react-native run-ios
This should now run packager successfully since we're not using the cache any more. If it still fails, something else might be wrong (AFAIK, you need to make sure that both babel-generator and babel-plugin-transform-flow-strip-types are on at least 6.21.0 (lower might work but I did not test that)).

If this works, then the next step would be to clear the babel cache.
The default location for that seems to be ~/.babel.json, so you could just delete that file. Alternatively, you could put a console.log(FILENAME) somewhere after it gets set in your /node_modules/babel-register/lib/cache.js file and find where the cache is located that way.

Finally, I initially ran into this issue after adding a manual dependency on babel-register which meant that the latest babel-generator was being used alongside an outdated babel-plugin-transform-flow-strip-types.

I just ran into this issue. I was able to solve it using the yarn.lock solution from @notjosh and then clearing the babel cache by removing ~/.babel.json as @nikaspran suggested 👍

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with Canny! Check out its new home: https://react-native.canny.io/feature-requests/p/newer-node-versions-6-failing-to-compile-script-via-babel-during-execution


[Technical Question] "IotHubRegistryManager" exception ErrorCodes.UnauthorizedAccess: Token or Certificate is invalid.

Hello everyone, when I send clould-to-device message with "registry_manager.send_c2d_message(DEVICE_NAME, data, properties=props)" sometimes i got exception: UnauthorizedAccess: Token or Certificate is invalid. Have anyone had seen this problem before. Please help, thank you very much.

The authentication I am using is symmetric key. This is the shared access key:E/uKH6XK6G2NLCYWQxu26+mrNBKmecm0I4WU2A1FfrY=. I am sending C2D messages from my local machine (Ubuntu 18.04 OS, python 3.7.5 and IoT-Hub sdk installed through pip install) to response a Jetson Nano device requests through D2C messages. I have checked carefully the time of my local machine also. Thank you for your response!

@khoathecoolest
A fix for this issue has been released as part of the latest version (2.6.0) of the azure-iot-hub package.
Please let us know if this addresses the issue.


feat: added codecov, golangci-lint, security scan

https://issues.redhat.com/browse/HACBS-1055

the gosec complaints are "won't fix"as I see it, as what @stuartwdouglas is doing is not encrypting data, but rather just encoding
the test result page seemed to provide a way of indicating that, though I'm curious of that selection persists across PRs @psturc

the gosec complaints are "won't fix"as I see it, as what @stuartwdouglas is doing is not encrypting data, but rather just encoding
the test result page seemed to provide a way of indicating that, though I'm curious of that selection persists across PRs @psturc

I think we can remove it by annotating those pieces of code that we know that aren't a security issue

the gosec complaints are "won't fix"as I see it, as what @stuartwdouglas is doing is not encrypting data, but rather just encoding
the test result page seemed to provide a way of indicating that, though I'm curious of that selection persists across PRs @psturc

I think we can remove it by annotating those pieces of code that we know that aren't a security issue

cool ... so as I read https://github.com/redhat-appstudio/jvm-build-service/pull/200/checks?check_run_id=8093189957:

for dependencybuild.go, that is currently line 5, 155, 593
for artifacdtbuild.go, that is currently line 5, 6, 376, 499
can you handle those as well with this PR @psturc ?  or would you rather defer to @stuartwdouglas or I for that ?

the gosec complaints are "won't fix"as I see it, as what @stuartwdouglas is doing is not encrypting data, but rather just encoding
the test result page seemed to provide a way of indicating that, though I'm curious of that selection persists across PRs @psturc

I think we can remove it by annotating those pieces of code that we know that aren't a security issue

cool ... so as I read https://github.com/redhat-appstudio/jvm-build-service/pull/200/checks?check_run_id=8093189957:

for dependencybuild.go, that is currently line 5, 155, 593
for artifacdtbuild.go, that is currently line 5, 6, 376, 499
can you handle those as well with this PR @psturc ?  or would you rather defer to @stuartwdouglas or I for that ?

I'm on it - will add a comment here once it's ready for a review

issues with CI
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 500 Internal Server Error
2022-08-30 15:18:16 ERROR 500: Internal Server Error.

/test jvm-build-service-e2e
/test jvm-build-service-in-repo-with-pruning-e2e
/test jvm-build-service-in-repo-e2e

all tests passed 😮
@stuartwdouglas @gabemontero  ready for another round of reviews