Fix LightningCLI signature parameter resolving for some lightning classes

What does this PR do?

Uses a new version of jsonargparse that extended the support for **kwargs to properly find signature parameters where possible by inspecting the source code, see https://github.com/omni-us/jsonargparse/issues/114#issuecomment-1152537388. The relevant documentation is at https://jsonargparse.readthedocs.io/en/latest/#parameter-resolvers.
Fixes #11653
Fixes #12697
Fixes #11574
Fixes #8561
Fixes https://github.com/PyTorchLightning/pytorch-lightning/issues/9887#issuecomment-1018532746
Fixes https://github.com/PyTorchLightning/pytorch-lightning/issues/8008#issuecomment-945540010
Does your PR introduce any breaking changes? If yes, please list them.

None
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

The jsonargparse version is 4.10.0.dev2 because from feedback here it might make sense to change things before a final release. Though the idea is that this pull request is merged only when 4.10.0 is out.


add forceSticky method

Adds a method to force the polyfill to be called -- suggested fix for https://github.com/wilddeer/stickyfill/issues/81
Usage example:
Stickyfill.forceSticky();
Stickyfill.add(element, { forceSticky: true });
Stickyfill.addOne(element, { forceSticky: true });
const sticky = new Stickyfill.Sticky(element, { forceSticky: true });

Also adds some defensive checks for the existence of window so that the package doesn't cause errors when being imported on the server in an SSR app.

@wilddeer mind taking a look and letting me know what you think?

Hey! Thanks for the PR.
There is an issue with your code though: it’s not enough to run the init function, here’s a line that checks for the seppuku variable and it won’t let the stickies to initialize: https://github.com/wilddeer/stickyfill/blob/master/src/stickyfill.js#L93
So, if you want to set forceSticky option per sticky, you would need to save this setting to the Sticky instance and check it in the refresh method. You’d also need to rework these places:
https://github.com/wilddeer/stickyfill/blob/master/src/stickyfill.js#L331-L333
https://github.com/wilddeer/stickyfill/blob/master/src/stickyfill.js#L360-L365
because the same sticky can now be added with different options.
But I would much prefer to have a global option for this. I doubt that anyone would use this option for some stickies and not use for others.
So, this would be something like Stickyfill.forceInit(), which would set seppuku to false, call the init function and call refresh for all existing stickies.
Oh, and please don’t commit the files in the /dist folder. I will do it myself along with the version bump.

Awesome, good call -- I like the idea of having something like forceInit. I'll rework this a bit. Thanks for the quick review!
And whoops didn't notice that -- I'll remove the /dist files as well

@wilddeer updated, PTAL :)

Hey, I will try to review the changes on this weekend, sorry for the delay

Sorry for the delay @wilddeer -- I've been on vacation. Just updated again per your comments, PTAL :)

Ok, I stopped being lazy for a moment and finally reviewed the changes. All is good, merged and released with a new version. Thanks!
I also found an edge case which broke things when the forceSticky method was called while one of the stickies wasn’t in its initial position. Fixed it.


When will a11 be supported?

I don't want a jailbreak I just want a trollstore

Me too

Discord announcement says ARM64 plans have been dropped due to the options of Palera1n and some new tools (such as "Definitely Not a Jailbreak Tool")

...oh no... I Think it is needed since palera1n need a whole reset of ios 16 without passcode and passcode-functions are not supported... so real issues here :( waiting for jailbreak a11 - 16.5

There's a couple others maybe coming soon, keep an eye out


feat(cost-model): migrate adv-stats

In optd, we use MostCommonValues and Distribution as trait bounds. Because of the serialization & deserialization need in optd-persistent, we use concrete types (enum) here instead of traits.

Would be better to have an initial commit to copy-paste everything from the optd repo, and another commit which contains the changes to the original crate. Otherwise I don't know what changes I'll need to review :(

Sure.

Would be better if we can bring all commit history to this repo.

Is there a good way to do so?

Looking at the dependencies, I saw lazy_static being used, which is already a deprecated crate and replaced by std libraries once cell. Probably needs to be fixed?

I've added a TODO in the code.

Regarding enum -- I'm not sure if this is a good approach regarding extensibility but let's merge it as-is.

I've tried to make MostCommonValues and Distribution as traits, but doesn't work out very well. Not sure if there's a better solution.

https://git-scm.com/docs/git-filter-branch + git merge from an orphan branch


Loading Spinner

AS A visitor I WANT to be notified that my request is processing SO THAT i know it worked
GIVEN a user input to request for nutritional information
WHEN I have submitted my request
THEN I am shown a loading spinner to inform me that my request is processing

Loading spinner added with CSS keyframes. Set timeout function added to $ajax GET request to toggle display on/off


crash when trying to commit

Originally discussed in https://github.com/karma-runner/karma/pull/1816 but they suggested I open an issue here.
Whenever I try to commit to the karma repository (or rather my fork of it), validate-commit-msg crashes with the following error:
C:\Git\mramato\karma\node_modules\validate-commit-msg\index.js:103
      return buffer.toString().split('\n').shift();
                   ^

TypeError: Cannot read property 'toString' of undefined
    at firstLineFromBuffer (C:\Git\mramato\karma\node_modules\validate-commit-msg\index.js:103:20)
    at ReadFileContext.callback (C:\Git\mramato\karma\node_modules\validate-commit-msg\index.js:92:15)
    at FSReqWrap.readFileAfterOpen [as oncomplete] (fs.js:303:13)

This happens when the commit message is valid.
Here are my system specs:
Windows 7
Node 4.2.x LTS
Git version 2.6.2.windows.1
Let me know if there's any other information you need or anything I can do to help track down the issue.  Thanks.

I need to give you an apology. For some reason, I want watching this repository so I want getting notifications about issues and pull requests. I am so incredibly sorry. Please forgive me. I really feel awful. I hope my unresponsiveness doesn't hinder you contributing more in the future.
As for your specific pull request, I'll give it a closer look soon.

This should be fixed by #10 Let me know if it's not.

Awesome, thanks.  There's no need to apologize, I know maintaining an OSS project is anything but easy.


collect additional IBM IKS resources

[x] worker pools
[ ] pods

Pods not required for network analysis


Add Ardaas From Sikh Rehat Maryada

Gurmukhi From:
Sikh Rehat Maryada, SGPC, May 2011
English Translation From:
Sikh Rehat Maryada (English), SGPC, August 2004

It is too long in my opinion.

SGPC without dots/full-stops is even shorter and more commonly used/read


Create a top-level group
Using the Mesos API set quota for a role with the same name as your group

$ ## Set Quota
curl -X "POST" "<CLUSTER_URL>/mesos/api/v1?SET_QUOTA" \
     -H 'Authorization: token=<token>' \
     -H 'Accept: application/json' \
     -H 'Content-Type: application/json' \
     -d $'{
  "type": "SET_QUOTA",
  "set_quota": {
    "quota_request": {
      "force": true,
      "guarantee": [
        {
          "scalar": {
            "value": 1
          },
          "name": "cpus",
          "role": "*",
          "type": "SCALAR"
        },
        {
          "scalar": {
            "value": 512
          },
          "name": "mem",
          "role": "*",
          "type": "SCALAR"
        }
      ],
      "role": "<group_name>"
    }
  }
}'

Navigate to Quota tab Services -> Quota tab
Verify the group you created is displayed in the list.

Trade-offs

Dependencies

Screenshots

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


Localfile is trying to get image from an endpoint

What are the steps to reproduce this issue?

Install Kubernetes with kubeadm

Install singularity-cri from master branch

modify kubeadm-flags to point singularity socket
sudo cat /var/lib/kubelet/kubeadm-flags.env
#KUBELET_KUBEADM_ARGS="--cgroup-driver=cgroupfs --network-plugin=cni --pod-infra-container-image=k8s.gcr.io/pause:3.1 --resolv-conf=/run/systemd/resolve/resolv.conf --container-runtime-endpoint=unix:///var/run/singularity.sock --image-service-endpoint=unix:///var/run/singularity.sock"
KUBELET_KUBEADM_ARGS=--container-runtime=remote --container-runtime-endpoint=unix:///var/run/singularity.sock --image-service-endpoint=unix:///var/run/singularity.sock

Run sycri and kubelet

Spin up hello world using following yaml
apiVersion: apps/v1
kind: Deployment
metadata:
name: hello-world-singularity
spec:
replicas: 1
selector:
matchLabels:
app: hello-world-singularity
template:
metadata:
labels:
app: hello-world-singularity
spec:
containers:
- name: hello-world-singularity
image: local.file/home/sanket/hello-world-singularity/hello-world-singularity.sif
ports:
- containerPort: 8080

Describe pot shows following error
Error: ImagePullBackOff
Pulling image "local.file/home/sanket/hello-world-singularity/hello-world-singularity.sif"
Failed to pull image "local.file/home/sanket/hello-world-singularity/hello-world-singularity.sif": rpc error: code = Unknown desc = Error response from daemon: Get https://local.file/v2/: dial tcp: lookup local.file: no such host

What happens?
When I spin up a container using local file, it tries to reach to some hostname local.file and gives message  "local.file: no such host"
What were you expecting to happen?
It should pick up from local file
Any logs, error output, comments, etc?
Failed to pull image "local.file/home/sanket/hello-world-singularity/hello-world-singularity.sif": rpc error: code = Unknown desc = Error response from daemon: Get https://local.file/v2/: dial tcp: lookup local.file: no such host
Environment?
OS distribution and version:: Ubuntu 18.04 Kubernetes master and slave on Virtual box
go version:  go1.12.7 linux/amd64
go env: …
Singularity-CRI version: 1.0.0-beta.4.7.gf95f62d.dirty
Singularity version: 3.3.0-rc.1.455.gd8c768aac
Kubernetes version: v1.15.1

@sanketmlad Can you do kubectl get po -o wide and then kubectl describe no <pod's-node>.
The error looks like pod was allocated on non-singularity node...

We don't have deamon :)

@sashayakovtseva that output is posted in the bug.
But you are right, it might have picked non-singularity node.
How to correct that?

For each singularity node do:
kubectl label no <node-name> runtime=singularity.
Use nodeSelector on pod:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: hello-world-singularity
spec:
  replicas: 1
  selector:
    matchLabels:
      app: hello-world-singularity
  template:
    metadata:
      labels:
        app: hello-world-singularity
    spec:
      nodeSelector:
        runtime: singularity
      containers:
      - name: hello-world-singularity
        image: local.file/home/sanket/hello-world-singularity/hello-world-singularity.sif
        ports:
        - containerPort: 8080

Below is the output for describe command
kubectl describe pod/hello-world-singularity-77f6d7bc46-2pdcrName:           hello-world-singularity-77f6d7bc46-2pdcr
Namespace:      default
Priority:       0
Node:
Labels:         app=hello-world-singularity
pod-template-hash=77f6d7bc46
Annotations:
Status:         Pending
IP:
Controlled By:  ReplicaSet/hello-world-singularity-77f6d7bc46
Containers:
hello-world-singularity:
Image:        local.file/home/sanket/hello-world-singularity/hello-world-singularity.sif
Port:         8080/TCP
Host Port:    0/TCP
Environment:
Mounts:
/var/run/secrets/kubernetes.io/serviceaccount from default-token-wws94 (ro)
Conditions:
Type           Status
PodScheduled   False
Volumes:
default-token-wws94:
Type:        Secret (a volume populated by a Secret)
SecretName:  default-token-wws94
Optional:    false
QoS Class:       BestEffort
Node-Selectors:  runtime=singularity
Tolerations:     node.kubernetes.io/not-ready:NoExecute for 300s
node.kubernetes.io/unreachable:NoExecute for 300s
Events:
Type     Reason            Age                From               Message

Warning  FailedScheduling  23s (x2 over 93s)  default-scheduler  0/2 nodes are available: 1 node(s) didn't match node selector, 1 node(s) had taints that the pod didn't tolerate.

BTW I have not installed singularity on Kubernetes slave machine
Should I install and try again?

Yes, of course :)
чт, 1 авг. 2019 г. в 19:22, sanketmlad<EMAIL_ADDRESS>
BTW I have not installed singularity on Kubernetes slave machine
Should I install and try again?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sylabs/singularity-cri/issues/346?email_source=notifications&email_token=AE7VH4XTB4LQI3BFC46SBSTQCMEUPA5CNFSM4IIR6PJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3LENEY#issuecomment-517359251,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE7VH4UZIHI74SSLWQIIXI3QCMEUPANCNFSM4IIR6PJA
.

--
Sasha Yakovtseva
Software Engineer, Sylabs, Inc.
www.sylabs.io

That did not affect the error :(

I suspect that the prefix local.file is still not being considered, and it is trying to get to web.
"Error response from daemon: Get https://local.file/v2/: dial tcp: lookup local.file: no such host"
It is assuming local.file is an endpoint

@sanketmlad Still the error output looks like it is docker, not Singularity.
On your node where you want to run SIF, do:

Install Singularity
Install and launch Singularity-CRI (refer to docs)
Label node with runtime=singularity (as described above), make sure that label doesn't appear on other nodes ( that where not set up with steps 1-2 )
Launch pod with kubectl apply -f pod.yaml

LMK if that works.

Hello @sanketmlad,
Did you manage to make it work so far?

I tried it and I get the same error. Doesn't seem to work as described in the documentation.


Classify '^' as Other, should classify it as Marks

https://github.com/74th/vscode-vim/blob/1db62fd74b5dce48e12a732e001472ff6bdec5a4/src/Utils.ts#L84
should be
    if (charCode <= 0x5E) {
    ...

Thank! I fixed it and release v0.5.5.


Add support for InfluxDB 2

Description
This PR add support for InfluxDB 2 authentication token (see documentation).
The InfluxdbBackendListenerClient has new optional property: influxdbToken that is used as an Authorization header in Http request.
Motivation and Context
Supporting InfluxDB 2.0
How Has This Been Tested?
It was tested by InfluxDB_Listener.jmx.zip with InfluxDB 2 instance and also I added unit tests - HttpMetricsSenderTest.java.
Screenshots (if appropriate):

Types of changes

New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.

Thank you for this contribution.
I have added two minor comments, if you can handle them, I'll be happy to merge the PR after that.
Thank you
Regards

Hi @pmouawad
I fixed your comments.
Thanks for review
Regards

Codecov Report

Merging #487 into master will increase coverage by 0.1%.
The diff coverage is 57.14%.

@@             Coverage Diff             @@
##             master     #487     +/-   ##
===========================================
+ Coverage     54.26%   54.36%   +0.1%
- Complexity     9634     9642      +8
===========================================
  Files          1007     1007
  Lines         62411    62415      +4
  Branches       7048     7049      +1
===========================================
+ Hits          33868    33935     +67
+ Misses        26129    26060     -69
- Partials       2414     2420      +6

Impacted Files
Coverage Δ
Complexity Δ

...visualizers/backend/influxdb/UdpMetricsSender.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...lizers/backend/influxdb/InfluxdbMetricsSender.java
100% <ø> (+100%)
0 <0> (ø)
:arrow_down:

...ackend/influxdb/InfluxdbBackendListenerClient.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...isualizers/backend/influxdb/HttpMetricsSender.java
67.03% <80%> (+67.03%)
8 <0> (+8)
:arrow_up:

...n/java/org/apache/jmeter/reporters/Summariser.java
84.73% <0%> (-0.77%)
17% <0%> (-1%)

...ackend/influxdb/AbstractInfluxdbMetricsSender.java
33.33% <0%> (+33.33%)
1% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


Mobile docs view icon width

On mobile the docs will show images with 100% width, which is not the best for the few icons, that are used.

If it's annoying for you, feel free to replace it by the warning sign character or emoji (but I'm not sure emoji are supported by readthedocs).

Closing due to inactivity.


[Feature Request]  Add support for generating LCS encoded structs in a Rust IDL

🚀 Feature Request
Currently, the existing YAML files in https://github.com/libra/libra/tree/master/testsuite/generate-format/tests/staged help authors check to see whether LCS-encoded types change and detect breakage.  As we work toward defining specifications, we would define these LCS-encoded types match those in the code.  We need to generate Rust IDL specifications directly from the existing YAML- e.g. similar to
~/.cargo/bin/serdegen --language Rust tests/staged/consensus.yaml
in order to do debugging without having to install i.e. cargo install serde-generate.  There is already a tool to generate the yaml with cargo run -p generate-format -- --corpus consensus network for example.  It would be nice to have it support another flag like --language Rust
Additionally, we should build tooling to verify that these definitions are identical to those in a markdown file and can update that markdown file.  For instance if that markdown file is something like:
Block
Block extends block_data with a unique identifier and a signature for proposed blocks.
struct Block {
    id: HashValue,
    block_data: BlockData,
    signature: Option<Ed25519Signature>,
}

We should be able to invoke something like cargo run -p update-doc -- --file file.md and get the types to be updated within the  ```rust...``` block.

Working on a draft PR: https://github.com/libra/libra/pull/4830

Improvement: https://github.com/facebookincubator/serde-reflection/pull/22


Assertion failure in +[XCTContext currentContextInThread:], XCTContext.m

Getting an assertion failure when snapshotting a URLRequest as curl.
2021-11-29 10:41:56.327719-0300 xctest[18855:204415] *** Assertion failure in +[XCTContext currentContextInThread:], XCTContext.m:339
2021-11-29 10:41:56.330131-0300 xctest[18855:204415] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Current context must not be nil'

Hi @grsouza, sorry for the late replay! I'm cleaning up old issues that don't contain enough information for us to debug/reproduce. If you still believe this is a bug with the library, please file an issue here with enough information to reproduce:
https://github.com/pointfreeco/swift-snapshot-testing/issues/new?assignees=&labels=&template=bug_report.md&title=
Thanks!


Read SBOL3 and coordinate discussion

Read and discuss the following paper to implement SBOL3
https://www.frontiersin.org/articles/10.3389/fbioe.2020.01009/full

@CarolusVitalis do you want to do this? I've already read the paper and most of the specification.


Unusuals: 🟦_OVERSTOCKED while stock < max

Describe the bug
Bot holds correct trade offers for review with reason 🟦_OVERSTOCKED, while current stock < max.
This only happens with Unusuals, and only when there are several of the same item with different effects in the backpack.
For example, when:

I am buying Massed Flies Dread Knot for 6 keys
My current stock for this item is 0
My max stock for this item is 2
I currently own > 2 Unusual Dread Knots with different effects

This bug only happens on received trade offers.
Everything works as intended when seller trades via chat message.
To Reproduce
Steps to reproduce the behavior:

Create buy order for Unusual item with effect [X] and set maximum stock to [N]. (User can choose X & N)
Have less than [N] of said item with effect [X] available in backpack.
Have more than [N] of said item with different effects available in backpack.
Wait for seller to send a correct trade offer to the bot containing Unusual item with effect [X].
Bot will mark offer as 🟦_OVERSTOCKED and hold the trade for manual review.

Expected behavior
Bot accepts correct trade offers containing Unusual items when stock < max.
Screenshots and logs
Trade below shows current stock = 0 and max stock = 2 for the Infernal Smoke Bad Pipes,
yet the offer is marked as 🟦_OVERSTOCKED.

Bad Pipes with different effects available in the backpack.

Version:
3.6.5

Thanks for the report. We will look into it.

Thanks for the details I can replicate the issue.

I am pretty such this case is the same as the one my bot experiencing during v3 development. I don't buy any unusual right now so I don't know this thing still exist.
the log files I sent in the #v3-beta-tester and #contributors channel might help.
#contributors:
https://discord.com/channels/664971400678998016/745410459212972173/797355434263838760
https://discord.com/channels/664971400678998016/745410459212972173/798713423130525776
https://discord.com/channels/664971400678998016/745410459212972173/800620712741175316
#v3-beta-tester
https://discord.com/channels/664971400678998016/801610351245787156/802344313748193350


sam_e70_xplained:running samples/net/sockets/civetweb/ failed

Describe the bug
running samples/net/sockets/civetweb/, and the result just keep waiting interface until time out.
To Reproduce
Steps to reproduce the behavior:
west build -b sam_e70_xplained samples/net/sockets/civetweb/
west flash
see error
*** Booting Zephyr OS build v2.4.0-rc1-11-g8c3efc7be5fe  ***
Hello World! sam_e70_xplained
[00:00:00.000,000]  soc: CIDR mismatch: chip = 0xa1020e01 vs HAL = 0xa1020e00
[00:00:00.000,000]  eth_sam: MAC: 00:00:00:00:00:00
[00:00:00.000,000]  eth_sam: Queue 0 activated
[00:00:00.000,000]  eth_sam: Queue 1 set to idle
[00:00:00.000,000]  eth_sam: Queue 2 set to idle
[00:00:00.000,000]  eth_sam_phy: Soft Reset of ETH PHY
*** Booting Zephyr OS build v2.4.0-rc1-11-g8c3efc7be5fe  ***
[00:00:00.090,000]  eth_sam_phy: PHYID: 0x221561 at addr: 0
[00:00:00.095,000]  net_config: Initializing network
[00:00:00.095,000]  net_config: Waiting interface 1 (0x20420a70) to be up...
[IMPLEMENTATION MISSING : sscanf]
[00:01:33.098,000]  net_config: Timeout while waiting network interface
[00:01:33.098,000]  net_config: Network initialization failed (-62)
Environment (please complete the following information):
OS: Fedora28
Toolchain: Zephyr-sdk-0.11.3
Commit ID: g8c3efc7be5fe

I just flashed some other sample to this board and noticed that the MAC address was all zeros. This caused issues when trying to get IP address. The error I saw was not Network initialization failed so the issue described here might be something else.
Anyway, I needed to set CONFIG_ETH_SAM_GMAC_MAC_I2C_EEPROM=y in order the network interface work properly.

Hi ,how's it going and what to do next step?


Add ec_op builtin

Motivation:
Add the ec_op builtin to the vm
Changes:
Add EcOpBuiltinRunner, a struct that implements BuiltinRunner
Add deduce_memory_cell for EcOpBuiltinRunner (in progress)
Add builtin to deduce_memory_cell to VirtualMachine (next target)
Add unit tests on new and affected methods (next target)
Missing:
Integration tests (full program execution)
Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

Codecov Report

Merging #102 (dbb4016) into bitwise-builtin (b74c3af) will decrease coverage by 0.93%.
The diff coverage is 10.25%.

@@                 Coverage Diff                 @@
##           bitwise-builtin     #102      +/-   ##
===================================================
- Coverage            96.76%   95.83%   -0.94%
===================================================
  Files                   15       15
  Lines                 3589     3626      +37
===================================================
+ Hits                  3473     3475       +2
- Misses                 116      151      +35

Impacted Files
Coverage Δ

src/vm/runners/builtin_runner.rs
82.14% <5.88%> (-7.92%)
:arrow_down:

src/vm/runners/cairo_runner.rs
98.63% <40.00%> (-0.27%)
:arrow_down:

Continue to review full report at Codecov.

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


Fix for page.url when paginating

When paginating, currently each page gets a PageContext instance differing only in the Paginator and OutputPath properties. Especially all PageContexts share the same Bag dictionary.
However, if you want to use page.url in your template, you always get the url of the first page, even on the successive pages. This makes it impossible to include a canonical url in the head of your HTML page.
This PR changes the copy c'tor of PageContext so that it receives a clone of the original PageContext.Bag, and when paginating each PageContext's bag will have its correct url property set.

Sorry for the silence, I was out of town for a few time.
Thanks a lot for your PR! :smiley:


Missing Notifications / Missing Writes

My application uses two BLE Characteristics as RX/TX channels to implement a serial communication. For every write to the TX Characteristic I expect to receive an answer via notifying RX characteristic.
Generally this is working great with this library, however occaisionally (between 1000-10000) write/notify pairs I am missing a notification on the RX characteristic. I see three possibilites: The device is not answering my request, or my request is not reaching the device, or the devices answer is not reaching my application.
As I cannot (yet) debug the device I'm communication with I'd like to make sure that the error is not on my side.
Is there a way I can confirm that my flutter side writing to the TX characteristic as well as the receiving of the RX notification are not being lost on their way to/from the android/ios host?

Hi! You can try setting log level to verbose and then running it from the native project (xcode or Android Studio), as flutter run unfortunately does not display native logs. If your command reached radio and failed or a notification was not received, but should've been, it should be in the logs.

Thansk for the reply.
Is it enough to run the native app in Android studio and select "verbose" in the logcat menu? The output doesnt seem to be different so far. Do I need to start the app via commandline with a flag? I'm probably searching wrong but I'm not finding a guide on how to enable verbose logging anywhere.

Hi!
You need to use https://pub.dev/documentation/flutter_ble_lib/latest/flutter_ble_lib/BleManager/setLogLevel.html and pass it LogLeve.verbose and there should be logs from Android's Bluetooth stack.


Change display of Part Label of Resources in Detail

Solve how not to make the line uneven. Probably: if that R entity has "Part Label" filled in, then display it in an uneditable column, rather than breaking the height of the line as it is done now.

@davidzbiral do you think that showing the "Part label" in the value tooltip + showing it as a placeholder of suggested (when no entity is assigned) would be enough here?

@adammertel Yes, good solution. By placeholder, you mean that in the empty suggester, there would be this as gray text? Yes, that's fine, good solution.


How to implement global attention in PointCept?

As explained in the ReadMe, PointCept consolidates the batch_size dimension and the num_point dimension when reading data, necessitating the use of an offset parameter to differentiate between different point cloud instances within the same batch.
Therefore, if we were to compute global attention, we would need to use the offset to distinguish between different point cloud instances in a batch when implementing the multiplication of the query and key matrices. However, as far as I'm aware, Pointcept does not offer such an operator.
I noticed that SPFormer faced the same issue. Their solution was to individually extract each point cloud from the batch data and calculate the global attention for each point cloud using a loop with the size equals to batch_size. Clearly, this serial method is not efficient. If I wish to implement an efficient  global attention operator in Pointcept, could you recommend any example code for reference?

Yes, of course, use PTv2 as an example,

https://github.com/Pointcept/Pointcept/blob/main/pointcept/models/point_transformer_v2/point_transformer_v2m2_base.py#L207, make "referece_index" index to all points in the same point cloud instead of the neighborhood (need to assume all point cloud share a same shape);
Or just reshape the coord, feat to [b, n, c] as 2D attention (Also need to assume all point cloud share the same shape, more efficient for Multi-head Scalar Attention)

Thank you very much for your guidance! Your first solution seems more feasible, and I plan to start experimenting along these lines.
However, while the second solution may seem simple, it appears to be less applicable in practice: from my understanding, the number of point clouds in Pointcept is variable (unless they exceed point_max). Therefore, the assumption that "all point clouds share the same shape" may not hold true in Pointcept. Do you have any simple ideas to make each point cloud have the same number of points?

Maybe you can add a Padding augmentation in Transformer and use it with SphereCrop (limit max point number). And usually for dataset like ModelNet, the number of points for each point cloud is the same, you can assume all point clouds share the same shape for a dataset like that.

OK, thank you very much!


Bloodline List

Basic bloodline list implemented.
Completes basics of #16

Changes will be made and pushed.


Add CI step to detect disallowed changes to go.mod

Makes sure that people don't accidentally make commits with changes in the go.mod around the code to replace components-contrib
Catches issues such as #5809 too

Good change.


installation: Invenio hosting of jQuery plugins

FIX Amends installation procedures to download necessary jQuery
plugins from Invenio-hosted web site, fixing problems with
non-existing Google Code pages. (closes #3620)

:shipit:


Doc for deployment.template.json

It would be useful to link to https://docs.microsoft.com/en-us/azure/iot-edge/module-composition?view=iotedge-2020-11 when the deployment.template.json document is introduced.

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

ID: ed1fefda-8454-a8a8-23ae-2b0cb317d2e2
Version Independent ID: cc13a227-6544-b472-c335-f6ee9afdb40e
Content: Continuous integration and continuous deployment to Azure IoT Edge devices - Azure IoT Edge
Content Source: articles/iot-edge/how-to-continuous-integration-continuous-deployment.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

Hello @algattik
Our SME will review your feedback and get back soon.

Hello @algattik. I'm interested to learn more about your experience going through this article. What would be useful about having a link to the conceptual article about deployment templates?
I assume that when you said "when the deployment.template.json document is introduced" you mean step 7 of creating the build pipeline? Did you not have a deployment template created at that point, or did you have one but weren't aware of where it was?
One of the prerequisites for this article is "An IoT Edge solution committed and pushed to your repository" and any solution made via Visual Studio or Visual Studio Code automatically has a deployment.template.json file. Did you see that prerequisite but didn't connect it to the deployment.template.json mentioned in the article, or did you miss that prerequisite?
Thank you.

@algattik Since I didn't hear back from you, I went ahead and updated the prerequisites with a specific callout that you'll need to provide the path to the deployment.template.json file, and included a link to the conceptual page about deployment manifests. When the pull request with these changes is merged, I'll close this issue.
If you have any further questions or comments on this topic, feel free to comment here again and we can reopen the issue.
Thank you again for your feedback.


 What version are you on? Do any tests failed when running cargo test? Some of the tests in gen.rs should fail if a number goes outside of the range specified.
I tried to recreate using:
for _ in 0..1_000_000 {
        let x  = rng.generate_range(1..=3);
        if x == 0 {
            break
        }
    }

But a 0 never occured.

I encounter the same problem. It occurs only with unsigned integer types though.
use nanorand::{tls_rng, Rng};

assert_eq!(tls_rng().generate_range(1..2 as u32), 1); // -> succeeds
assert_eq!(tls_rng().generate_range(1..2 as i32), 1); // -> panicks

thread 'main' panicked at src/main.rs:4:1:
assertion `left == right` failed
  left: 0
 right: 1

I confirmed it happening on platforms x86_64-unknown-linux-gnu and wasm32-unknown-unknown.
I enabled these features in Cargo.toml:
nanorand = { version = "0.7", default-features = false, features = ["tls", "getrandom"] }
getrandom = { version = "*", features = ["js"] }

I also observed this and similar snippets (where rng is a WyRand) hitting the unreachable case:
let value = match self.rng.generate_range(0..4) {
    0 => 0,
    1 => 1,
    2 => self.rng.generate_range(0..10),
    3 => self.rng.generate(),
    _ => unreachable!(),
};

I have also noticed this issue on 0.7.0 but not 0.6.x.
However, I experience it with signed types only.
My PoC.
use nanorand::{Rng, WyRand};

fn main() {
    let mut test_seed = 0;
    let mi = 50;
    let ma = 100;
    'outer: loop {
        let mut rng = WyRand::new_seed(test_seed);
        for j in 0..128 {
            let v = rng.generate_range(mi..ma);
            if v < mi || v >= ma {
                println!("{} {} {}", test_seed, j, v);
                // break 'outer;
            }
        }
        test_seed += 1;
    }
}


New version: CrystalDewWorld.CrystalDiskInfo version 9.4.4

Created by 🥟 Dumplings in workflow run #7860.

Log
Updated: 9.4.3 → 9.4.4
No ReleaseNotes (en-US) for version 9.4.4
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Update README.md to add my translation contribution

👍🏻 Keep up the good work with Catima

Thanks :)


Short time message displayed on Workshops when workshop was created

Environment: https://oos.dmytrominochkin.cloud/#/
**Reproducible: ** always
Preconditions

Follow the link https://oos.dmytrominochkin.cloud/#/
Log into OoS website as SP
Select "Мої гуртки" in the menu bar in the top right corner.

Steps to reproduce

Click on the “Додати гурток” button and fill in all mandatory fields in each section.
Fill in all mandatory fields on the step "Про гурток"
Fill in all mandatory fields on the step "Опис"
Fill in all mandatory fields on the step "Контакти"
Fill in all mandatory fields on the step "Викладачі"
Click on the button "Зберегти"
The "Гуртки" page is opened with the new workshop created; 'Дякуємо! Новий гурток успішно доданий.' message is displayed at the top of the page

Actual result
The message "Дякуємо! Новий гурток успішно доданий"  displayed for 2 seconds
Expected result
The message displayed for 10 seconds
User story and test case links
https://jira.softserve.academy/browse/OS-365
Labels to be added
"Bug", Priority:low, Severity: trivial, Type: Functional

@val-ukr fixed!

@litvinets  I have the same mistake
email<EMAIL_ADDRESS>password Asdf!6789

wrong password or email
On Wed, Jan 18, 2023 at 1:29 PM Valeriia @.***> wrote:

@litvinets https://github.com/litvinets I have the same mistake
email: @.***
password Asdf!6789
—
Reply to this email directly, view it on GitHub
https://github.com/ita-social-projects/OoS-Frontend/issues/1951#issuecomment-1386904848,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGRYMR2DB5I4YQXJZEIBDFLWS7H2XANCNFSM6AAAAAATRIGJ4M
.
You are receiving this because you were mentioned.Message ID:
@.***>

@val-ukr  wrong password or email

@litvinets  sorry, one more:
<EMAIL_ADDRESS>Qwer1234@

Hi, the bag was fixed


[WIP][PE1-1340] Set managed S3 CORS bucket origin access policy default

Description

Changes included:

Allow overriding default request policies from configuration
Set default request policy for bucket origins to managed S3 CORS
Renamed awscloudfront.DistributionConfig type variables to distCfg
Updated docs

Motivation and Context

https://gympass.atlassian.net/browse/PE1-1340
How has this been tested?

Automated tests
Checklist

[x] I have implemented automated tests for the changes.
[x] I have updated the documentation accordingly.

@go-faustino could you please update the .env.example file within new environment variables?

@caiofralmeida done, thanks!


add support for client authentication with certificates

one proposed approach is JSON web tokens so that it works through SSL proxies


ncbi-genome-download post

@meren,
Please take a look and if you like it feel free to merge this pull request.
The content of this blog was originally included in the refinement blogpost, but now to make it cleaner, I plan to mention this separated blogpost within our refinement blogpost for people who want to see a good way of getting genomes that are similar to their genomes for pangenomics or phylogenomics.
(yes, I know I'm not supposed to be writing blogposts all day (I have no idea where I learned this habit of writing blogposts instead of writing papers....))

This is excellent Alon!! I was just talking to Dan and his colleagues at Forsyth, telling them I will need to write a blog post on this exact topic (instead of writing a paper, indeed), and here you have done it! :)
Thank you very much :)


License?

The license paragraph is rather cryptic - it's just a link to your website. How do I deduce license terms from that?

I added License to repository. For more information open up the "LICENSE" file.
Thank you.


Exclude librairies from restriction

Hi,
Thanks for this solution, it works greatly !
I have a request tho, is it possible to exclude a library from the time limit ?
A have a music librairy I'd like to let accessible even if time limit is on.
Thanks !
MasterBOX

Hi,
I've made a quick change to keep selected libraries from disabling. Please take a look (no_limit_folders in config file) and let me know if that works as expected.
Thank you for a great idea! ...and for the coffee :)

Great, I'll take a look !
Le lun. 30 déc. 2024 à 13:29, Joker @.***> a écrit :

Hi,
I've made a quick change to keep selected libraries from disabling. Please
take a look (no_limit_folders in config file) and let me know if that
works as expected.
Thank you for a great idea! ...and for the coffee :)
—
Reply to this email directly, view it on GitHub
https://github.com/Joker-KP/jelly-watch-wise/issues/7#issuecomment-2566005696,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEVXW4JS6KUQMDNVJYWEO32IHJOZAVCNFSM6AAAAABULDKUX2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRWGAYDKNRZGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


join slack failed

When I join in the slack, I got the error  "Failed! token_revoked" after click "join".

same as #44


Please invite me to the GitHub Community Organization

Name
Muhammad Hassan
Discord Username (if applicable)
Muhammad Hassan#4498
Additional Context
Hey @EddieHubCommunity , Please Add me here for contributions.

User name Mhassanbughio


Batched ForwardDiff pushforward

Codecov Report
Attention: Patch coverage is 0% with 92 lines in your changes missing coverage. Please review.

Project coverage is 5.70%. Comparing base (4390675) to head (e91f16a).

Files
Patch %
Lines

...t/DifferentiationInterfaceForwardDiffExt/onearg.jl
0.00%
45 Missing :warning:

...xt/DifferentiationInterfaceForwardDiffExt/utils.jl
0.00%
32 Missing :warning:

...t/DifferentiationInterfaceForwardDiffExt/twoarg.jl
0.00%
15 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (4390675) and HEAD (e91f16a). Click for more details.
HEAD has 47 uploads less than BASE
| Flag | BASE (4390675) | HEAD (e91f16a) |
|------|------|------|
||49|2|

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #328       +/-   ##
==========================================
- Coverage   94.93%   5.70%   -89.23%
==========================================
  Files          95      63       -32
  Lines        4557    2489     -2068
==========================================
- Hits         4326     142     -4184
- Misses        231    2347     +2116

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


Issues with the top bar when using a theme other then the default (dark and sepia)

Bug Report
Description
When on mobile, and the theme is sepia or dark and you scroll up from the top the bar at the top disappears which makes the UI hard to read and interact with
Steps to Reproduce
Steps to reproduce the behavior:

Go to any page
Click on the big A (I have no clue what it is called)
Switch theme from the default to a different one
Scroll down
The bar at the top disappears

Expected behavior
From what I can tell there is supposed to be be a bar behind the language selector and the A (I have no clue what it is called)
Screenshots
This first screen shot is of the default theme with what seams to be the default and correct behavior

Next is the sepia and dark themes in that order which have issue

Environment

OS: iOS
OS Version: 15.0.2
Browser: Brave, Firefox, Firefox focus and Safari
Browser Version: brave: 1.32.3, Firefox: 40.2

Additional context
I am on IOS and don't have access to Android so I could not test it in those devices; I also haven't had the chance to test on desktop. Also I would like to note that this is my first issue on this project so I am sorry if I made a mistake when typing this up.
For Algorithm Archive Developers

[x] The bug can be reproduced
[ ] The bug can be fixed
[ ] There is a timeline to fix the bug
[ ] The bug has been fixed (Please link the PR)

I just checked on desktop and it has the same issues with the themes (I would link my version but I don't know how to check the version of Firefox on desktop).

Thanks for the report!
You can check thr version on dedktop in Help > About firefox I think.

Thank you for the help with the menus, that means that the images for desktop(https://github.com/algorithm-archivists/algorithm-archive/issues/978#issuecomment-1000866088) are taken from Firefox 95.0.2

I can reproduce this on Vivaldi browser version 5.0.2497.32, installed on my Ubuntu 20.04 machine.
This seems to be intentional, is in the light theme bar is opaque, while the Dark and Sepia themes have a transparent bar.
This will probably a simple CSS change (if JS doesn't mess with it), if someone can do this or point me to the themes.

question, should I mark The bug can be fixed as checked on the main post at the top or should I not since from what I can tell at this point it would be possible to do so.

I think this can be solved by adding these lines to styles/website.css
.book.color-theme-1 .book-header {
    color: #afa70;
    background: #f3eacb;
}

.book.color-theme-2 .book-header {
    color: #7e888b;
    background: #1c1f2b;
}

It worked on Firefox and  Chromium on my Arch Linux machine.

This should probably be solved using CSS variables, but this should be good enough, I'll look at it when I have some time

I think we use the default gitbook themes, which makes it hard to change them.
We seem to be using those plugins, which doesn't seem to include the plugin you linked:

mathjax@https://github.com/algorithm-archivists/plugin-mathjax
bibtex-cite
wordcount
api-language-selector@https://github.com/algorithm-archivists/gitbook-plugin-api-language-selector.git
include-codeblock
bulk-redirect
prism
-highlight

So from what I can tell this is fixable, am I wrong here, does this mean that I can mark it as can be fixed on the top post

I think we got caught up in the fact that the theme api plugin does not technically have a license attached to it, so we were uncomfortable modifying it. The fonts plugin is apache, so it's fine to fork and modify that.
Maybe the best course of action is to fork that one, make the changes and see if setting it as a dependency fixes the issue.


[PROPOSAL] Replace contact information in code-of-conduct, potentially replace the CoC itself.

What/Why
What are you proposing?
The OpenSearch Project had adopted the Amazon Open Source Code of Conduct in https://github.com/opensearch-project/.github/blob/main/CODE_OF_CONDUCT.md. The contact information listed at the bottom of that document ([Contact us](mailto:opensource-codeofconduct@amazon.com).) needs to change.
What users have asked for this feature?
Coming from #216.
What problems are you trying to solve?
Amazon should not be on the hook to enforce CoC in the OpenSearch Software Foundation.
What is the user experience going to be?
Clear explanation of who to contact for CoC violations.

This might help:This file might fix it
https://mega.nz/file/1vkGlJTL#yU-SJLlIilF0mx5qwpPOd3BM8iKZqOJLEQUStR76rrQ
I put the necessary dlls in the archive

#63 contains some previous discussion related to this topic as well.
Also that issue mentions another email that likely needs updated - https://github.com/search?q=org%3Aopensearch-project aws-security%40amazon.com&type=code. Not sure if we want to combine these updates in this issue or create a new ticket.
[aws-security<EMAIL_ADDRESS>

@bryanlb Good callout - I think that #219 captures be focused on the security changes - if they get updated in the same PR its OK with me.

Use this email for the for CoC reports -<EMAIL_ADDRESS>


Micromatch dependency - Security advisory vite-plugin-watch-and-run

Describe the bug
When installing vite-plugin-watch-and-run we get a security vulnerability advisory.
# npm audit report

micromatch  <4.0.8
Severity: moderate
Regular Expression Denial of Service (ReDoS) in micromatch - https://github.com/advisories/GHSA-952p-6rrq-rcjv
No fix available
node_modules/micromatch
node_modules/vite-plugin-watch-and-run/node_modules/micromatch
  vite-plugin-watch-and-run  *
  Depends on vulnerable versions of micromatch
  node_modules/vite-plugin-watch-and-run

2 moderate severity vulnerabilities

Some issues need review, and may require choosing
a different dependency.

Severity
annoyance
Steps to Reproduce the Bug

npm install vite-plugin-watch-and-run
Observe npm audit warning 2 moderate severity vulnerabilities

Reproduction
No response

https://github.com/jycouet/kitql/blob/53804e44e6ae3ba8e0de54a2de87cae4f4e32437/packages/vite-plugin-watch-and-run/package.json#L47
Updating this line to 4.0.8 should fix the issue.
See https://github.com/advisories/GHSA-952p-6rrq-rcjv

Thx for reporting.
I will do this today.

Thank you


Telemetry for some webview commands

As we've moved commands out of the context menu and into webviews, we've lost a lot of granularity from our telemetry. For example, we no longer know which kubectl or Inspektor Gadget commands are being run, or whether users are invoking cluster operations from webviews. Instead we can only see that a webview has been launched, even though users may not have taken any action from it.
It would be really useful to log telemetry about actions taken from webviews. This can be determined from the messages sent from webviews to VS Code.
However... We want to be very careful about what data we're collecting here. For privacy reasons we cannot simply collect the contents of each message sent to VS Code. Defining what gets collected needs to be clear, explicit, and opt-in only, so that we can easily see and review what is being collected.
Proposal: We extend the PanelDataProvider interface so that each implementor (one per webview) has to implement a function that specifies what telemetry is collected (if any) for each message to VS Code in that webview, e.g.:
export interface PanelDataProvider<TContent extends ContentId> {
    getTitle(): string;
    getInitialState(): InitialState<TContent>;
+    getTelemetryDefinition(): TelemetryDefinition<TContent>;
    getMessageHandler(webview: ToWebviewMessageSink<TContent>): ToVsCodeMessageHandler<TContent>;
}

An implementation of this might look like (for 'Create Cluster'):
    getTelemetryDefinition(): TelemetryDefinition<"createCluster"> {
        return {
            getResourceGroupsRequest: false,  // No telemetry for this message
            getLocationsRequest: false,  // No telemetry for this message
            cancelDeploymentRequest: true,  // Log that the command was run, but no other data
            createClusterRequest: (args) => ({
                location: args.location,  // Illustration purposes only: in practice we probably wouldn't log `location`
                preset: args.preset,
            }),
        };
    }

[Edited above to allow boolean values to indicate whether to log whether a command is run, with no other metadata]

It seems like we can't specify any properties that are "unknown". Only command seems to be permitted. The others get stripped out:
"DataHandlingTags": unknown_props;dropped_unclassified_props;ip2city;dropped_null_props;delinked_macaddresshash;dropped_suppressed_props;,

This means it's easy to see, say, an Inspektor Gadget command was run:
"command": "gadget.runStreamingTraceRequest"

...but the other properties which tell us which gadget category (e.g. trace) or resource (e.g. dns) get removed.
@Tatsinnit: Any idea where the allowed properties are configured?


rx.color does not work in markdown component map

Describe the bug
A clear and concise description of what the bug is.
markdown_style = {
    "code": lambda text: rx.code(text, color_scheme="gray"),
    "codeblock": lambda text, **props: rx.code_block(
        text, **props, margin_y="1em", background_color=rx.color("gray", 1)
    ),
    "a": lambda text, **props: rx.link(
        text,
        **props,
        font_weight="bold",
        text_decoration="underline",
        text_decoration_color=accent_text_color,
    ),
}

  return rx.markdown(content, component_map=styles.markdown_style)

Gets the following error:
    super().__init__(*args, **kwargs)
  File "pydantic/main.py", line 341, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 1 validation error for CodeBlock
custom_style -> backgroundColor
  str type expected (type=type_error.str)

I have worked on this a little bit. I have a solution that works, and everything seems to work as intended (same test outputs; reflex application loads with the correct rx.color values).
Before:

After:

There are other minor changes as well, such as the replacing
isinstance(value, str) and isinstance(value, Color)
with
isinstance(value, str).
Before I make a pull request, is this the kind of behavior we want from rx.color? Unlike before, where it returned Var or Color, it now returns str, which may not be what we want.

what about something like this
diff --git a/reflex/components/datadisplay/code.py b/reflex/components/datadisplay/code.py
index c108a304..1d220a82 100644
--- a/reflex/components/datadisplay/code.py
+++ b/reflex/components/datadisplay/code.py
@@ -2,6 +2,7 @@
 import re
 from typing import Dict, Literal, Optional, Union

+from reflex.constants.colors import Color
 from reflex.components.chakra.forms import Button
 from reflex.components.chakra.layout import Box
 from reflex.components.chakra.media import Icon
@@ -373,7 +374,7 @@ class CodeBlock(Component):
     wrap_long_lines: Var[bool]

     # A custom style for the code block.
-    custom_style: Dict[str, str] = {}
+    custom_style: Dict[str, Union[str, Var, Color]] = {}

     # Props passed down to the code tag.
     code_tag_props: Var[Dict[str, str]]

We already have all the serializers in place to make it work, i think pydantic was just upset about the type annotation.

I see. That looks a lot better!


Multi-gun firerate nerf

About the PR

🕷warpath
Why / Balance

Holding multiple guns straight up doubles your firerate. that is insanium. Especially for future guns with slower firerates like rifles and snipers, it makes sense to have a period where you get the gun into a firing position wherein you're unable to shoot it.
Technical details

literally just what melee weapons do.
Media

i had a gif but it was so awful i removed it.

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

tweak: Swapping between guns now has a small delay before you can fire.

what a nice balance PR, i'm sure it's not preparing for anything at all :clueless:

im going to rebalance the game with a steel bat


Cocoapods code not updated

The code in the Github project and the one on cocoa pods is not the same.

Updated cocoa pod code here: https://github.com/RishabhTayal/SLParallaxController. Use it as pod RTParallaxController


capstone project Week 5

project setup on repo
Set up the repository and tools.
added unique content
chose global warming as a topic.
Included images, text, icons, fonts.
made the site mobile first.
created home and about page
included desktop version
Adapted the 2 pages to desktop.
added necessary interactions with Js
added dynamic page
Added documentation (redme file)
code refactor

i have made the changes sir
On Sat, 28 May 2022 at 12:00, Makungong Collins @.***>
wrote:

@.**** requested changes on this pull request.
Hi @MicroverseNelson https://github.com/MicroverseNelson,
Good job so far!
There are some issues that you still need to work on to prepare your
project for the final evaluation but you are almost there!
Highlights 🚀 ✔ ✔

The repo looks professional.✔
Git/GitHub flow best practices have been followed✔
There are no linter errors✔
You have descriptive PR✔

Suggested changes
Check the comments under the review.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
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.
In index.html
https://github.com/MicroverseNelson/global-warming/pull/2#discussion_r884100606
:

@@ -40,7 +40,7 @@

Please kindly fix your header width to fit the project requirement
https://www.behance.net/gallery/29845175/CC-Global-Summit-2015

Required Design
[image: header]
https://user-images.githubusercontent.com/55596970/170817117-8ac4984f-7baa-41fb-b180-e36910bea8a2.png
Your Design
[image: fakeHeader]
https://user-images.githubusercontent.com/55596970/170817162-c570a5e2-531c-4d42-b20a-db279493df49.png
In index.html
https://github.com/MicroverseNelson/global-warming/pull/2#discussion_r884101237
:

@@ -77,31 +77,31 @@ Main Program

Please do your best to fix the Main Program section to match the required
design https://www.behance.net/gallery/29845175/CC-Global-Summit-2015

Required Design
[image: main]
https://user-images.githubusercontent.com/55596970/170817498-58feba77-7d0b-4920-aeb9-21fecbaa33df.png
Your Design
[image: mainsec]
https://user-images.githubusercontent.com/55596970/170817685-82e68e2f-3db5-4236-9299-8c60a3153d55.png
In index.html
https://github.com/MicroverseNelson/global-warming/pull/2#discussion_r884103363
:

@@ -133,24 +132,26 @@ Featured Speakers

More

More

Please center the speaker section to match the required design
https://www.behance.net/gallery/29845175/CC-Global-Summit-2015
[image: speakers]
https://user-images.githubusercontent.com/55596970/170818105-add274b3-2f6e-4d40-be38-2d07633a5faa.png

In index.html
https://github.com/MicroverseNelson/global-warming/pull/2#discussion_r884104381
:

Please add the Partner and Footers sections to your project as per the project
requirement
https://github.com/microverseinc/curriculum-html-css/blob/main/capstone/html_capstone.md
[image: footers]
https://user-images.githubusercontent.com/55596970/170818377-dc10686d-b1ab-4a11-bdcb-e7f4f0aab870.png
—
Reply to this email directly, view it on GitHub
https://github.com/MicroverseNelson/global-warming/pull/2#pullrequestreview-988420197,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYAFWSFGQD6LOJIWFOUXMGLVMHODXANCNFSM5XGFFMVQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Can support oceanbase? is tidb supported?

Feature Description
need distributed sql. what is the best option so far for use with gitea?
Screenshots
No response

TiDB has been supported.

I'd point you towards tidb, I've confirmed that Gitea works with their hosted offering. Not sure about oceanbase, if you try it out with them, please let us know. I'll close this issue now.


hover tool for spikes with datetime x

import pandas as pd
import numpy as np
import holoviews as hv
hv.extension('bokeh')

dts = pd.date_range('2017-01-01', '2017-01-05')
y = np.arange(len(dts))

hv.Bars((dts, y)).options(tools=['hover'])

Bars automatically convert the time

Spikes do not automatically convert the time

Thanks for reporting this! Definitely looks like a bug.


Missing moveit_planning_execution.launch

The file was added in cee59d494bf7afe5eda331fbf9cae00a4b4b792f, but I can't find it any more in master.
Was it devoured by some git merge?

Hi @marcoesposito1988 Yep, I see it was out in #18, but I don't think on purpose, too many changes in that PR, feel free to add it again, and my apologies !

I thought so :rofl: no problem, I will make a pull request to put it back

BTW: in the current RViz configuration, the planning request marker size is set to zero. This means, that you can't see the marker to command the robot. This can be EXTREMELY confusing for a newbie just trying out the package (I think I spent half an hour figuring this out myself, including recompiling RViz!)
Would you mind if I add another RViz config file to be loaded by moveit_planning_execution.launch? This way your setup would not change, but new users would be not as confused.

Fixed by #88


Gradle should remove transforms-2/files-2.1/.../unzipped-distribution

Expected Behavior
It looks like Gradle does not really use .gradle/caches/transforms-2/files-2.1/.../unzipped-distribution/..., so it should remove the files earlier.
Current Behavior
.gradle/caches/transforms-2/files-2.1 contains several unzipped distributions (e.g. 6.6, 6.7-rc-5).

Extra files in .gradle/caches results in extra space usage at CI servers
IntelliJ IDEA indexes unzipped-distribution/... files for some reason

Context
I'm analyzing .gradle contents, and extra files make the comparison harder.

Gradle creates that directory for IDE support. It contains the sources of the distribution: https://github.com/gradle/gradle/blob/e0dd1c8d5f7e686b303ffbfbf0700b7e80dfc88a/subprojects/kotlin-dsl/src/main/kotlin/org/gradle/kotlin/dsl/resolver/SourceDistributionProvider.kt#L38-L53
I think this transform should not run on CI, and IDEA would cause the transform to run. Did you find the directory on CI as well?

Thanks for the clarification. I think I see the directory on my local machine only.


Include ca cert in service cert secret

Libvirt/OVN need the specific CA cert, not a bundle

@olliewalsh
Ok - this isn't originally what I had in mind - I figured one would basically label the secret containing the libvirt cert and use this as the bundle - but I suppose we can make it work.
Part of the reason for this was to reduce the size of the secrets that are mounted in the ansibleEE container.  Recall that these secrets must contain all the certs, keys and now cacerts for all nodes in the nodeset.  But we have a PR up to handle the size of this - so it should be fine.  See https://github.com/openstack-k8s-operators/dataplane-operator/pull/718

We'll have to adjust that - as well as the docs in https://github.com/openstack-k8s-operators/dataplane-operator/blob/main/docs/assemblies/tls.adoc

@olliewalsh
Ok - this isn't originally what I had in mind - I figured one would basically label the secret containing the libvirt cert and use this as the bundle - but I suppose we can make it work.
Part of the reason for this was to reduce the size of the secrets that are mounted in the ansibleEE container. Recall that these secrets must contain all the certs, keys and now cacerts for all nodes in the nodeset. But we have a PR up to handle the size of this - so it should be fine. See #718

ack, can change if you want. This was really just a quick fix for libvirt as what was in place was not secure.

Option 1 isn't an option. The CA bundle(s) must not be used to validate libvirt client certs, it would give everyone with a signed cert from one of the trusted public CAs admin access to all of the hypervisors.

If libvirt needs to trust a CA it should be added to the system-wide trusted CAs and mounted in the container

Option 1 isn't an option. The CA bundle(s) must not be used to validate libvirt client certs, it would give everyone with a signed cert from one of the trusted public CAs admin access to all of the hypervisors (unless sasl auth has been implemented in the edpm role, not sure if that has yet).

sasl auth has not been implemented and its not currently planed for ga.
we could implement it for FP1 but for GA we need to go with option 2 based on what is currently implemented in the role.

OK - I can see the benefits of going with option 2 in this case.  We can revisit later as needed, but for now, we'll adjust https://github.com/openstack-k8s-operators/dataplane-operator/pull/718 accordingly.

cool, don't think there is any other option for ovn as it doesn't support any auth so all we have is client certs and the ca cert to control access


Init all tables

Init not only the filter table but also nat mangle and raw tables.

I appreciate your suggested changes, however:
Have you experienced the NAT and RAW tables not being initialized? 'iptables -L' only lists the tables, which is a work around to allow the openwrt firewall to restart. It doesn't actually initialized the different tables (perhaps my comment should be changed).
As to the last part, your patch no longer checks that there is an WAN Zone which requires masquerading. The reason that code is there, is in the event the router is running in a IPv6-only environment, there is no need to insert a MASQ rule.

After the proposed changes, the openwrt firewall itself creates all NAT rules, including masquerading.


CanvasLayerRenderer.prototype.getDataAtPixel - better performance result

This is my second task about this feature. Please check it.
You should improve your existing CanvasLayerRenderer.prototype.getDataAtPixel function.
In the function you get image data by whole context. This solution decrease performance due to scanning of all pixels.
CanvasLayerRenderer.prototype.getDataAtPixel = function (pixel, frameState, hitTolerance) {
        var renderPixel = applyTransform(this.inversePixelTransform, pixel.slice());
        var context = this.context;
        var data;
        try {
            data = context.getImageData(Math.round(renderPixel[0]), Math.round(renderPixel[1]), 1, 1).data;
        }
        catch (err) {
            if (err.name === 'SecurityError') {
                // tainted canvas, we assume there is data at the given pixel (although there might not be)
                return new Uint8Array();
            }
            return data;
        }
        if (data[3] === 0) {
            return null;
        }
        return data;
    };

To achieve a better performance, first you should crop canvas to 1px image (drawing new 1px image) and finally get that pixel using getDataImage:
CanvasLayerRenderer.prototype.getDataAtPixel = function (pixel, frameState, hitTolerance) {
        var renderPixel = applyTransform(this.inversePixelTransform, pixel.slice());
        var context = this.context;
        var data;
        try {
            var x = Math.round(renderPixel[0]);
            var y = Math.round(renderPixel[1]);
            var newCanvas = document.createElement("canvas");
            var newContext = newCanvas.getContext('2d');
            newCanvas.width = 1;
            newCanvas.height = 1;
            newContext.clearRect(0, 0, 1, 1);
            newContext.drawImage(context.canvas, x, y, 1, 1, 0, 0, 1, 1);
            data = newContext.getImageData(0, 0, 1, 1).data;
            //data = context.getImageData(Math.round(renderPixel[0]), Math.round(renderPixel[1]), 1, 1).data;
        }
        catch (err) {
            if (err.name === 'SecurityError') {
                // tainted canvas, we assume there is data at the given pixel (although there might not be)
                return new Uint8Array();
            }
            return data;
        }
        if (data[3] === 0) {
            return null;
        }
        return data;
    };

Please, improve this function. Performance result is much better.
Regards.

Thanks for your attempts to fix this. Would you be able to provide a pull request? That way, you'll be recognized as author of this change, and it will be easier for project maintainers to incorporate your fix.

Can you give me an access to making pull request? Actually i can't making pr because of authentication.

You'll want to create a fork of the repository. In that fork, you can create a branch, make changes, and submit the branch as pull request. See https://github.com/openlayers/openlayers/blob/main/DEVELOPING.md for more details.


docs: revise install instructions, provide CLI process

Closes #4502
Update CLI (and API) instructions for users who want end-to-end automated process without UI interaction.
Still WIP, but closer.
Add link to influxd recovery CLI

Closes #
Describe your proposed changes here.

[ ] Signed the InfluxData CLA
(if necessary)
[ ] Rebased/mergeable

@kelseiv Still in draft, WIP. Thinking we should add 'create more tokens' as an optional step and remove it from the note.


Add Adam to .zenodo.json

@symmy596 needs to be added to the .zenodo.json file.

closed with https://github.com/pythoninchemistry/intro_python_chemists/commit/ee065e929d5de7b797e71bfb7cf09d4ba171e6ef


Added styles to increase text contrast for blockquotes

Summary
Updated the color of text when using blockquotes to adhere to accessible contrast ratios of text to background color.
Resolves #2188

@alex-bezek These seem to be the same code changes from https://github.com/cerner/terra-core/pull/2209. We should close this PR and make the changes on the original so we can track any PR change requests / comments.

Have been working on it all day and some on Monday. Should have something to show Brett for tomorrow :)
Get Outlook for iOShttps://aka.ms/o0ukef

From: Emily Rohrbough<EMAIL_ADDRESS>Sent: Wednesday, February 20, 2019 4:22 PM
To: cerner/terra-core
Cc: Brisimitzakis,Alex Philip; Mention
Subject: Re: [cerner/terra-core] Added styles to increase text contrast for blockquotes (#2215)
@emilyrohrbough commented on this pull request.

In packages/terra-markdown/src/Markdown.scsshttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcerner%2Fterra-core%2Fpull%2F2215%23discussion_r258702625&data=02|01|alex.brisimitzakis%40cerner.com|a05529de84c441230b0808d69781e0c8|fbc493a80d244454a815f4ca58e8c09d|0|0|636862981411881616&sdata=m%2B6oYCr18rJaDLVegNShGAAR3xa8QdYR0WK1mPODwSo%3D&reserved=0:

@@ -5,3 +5,7 @@
background-color: #ebebef;
}
}

+.markdown-body blockquote {

color: #24292e;
+}

@AlexBrizihttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAlexBrizi&data=02|01|alex.brisimitzakis%40cerner.com|a05529de84c441230b0808d69781e0c8|fbc493a80d244454a815f4ca58e8c09d|0|0|636862981411881616&sdata=xZUwXNaSJ1MCH1wG45FJIt6yJKNzoosHkyTcEJHD4C4%3D&reserved=0 any progress?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcerner%2Fterra-core%2Fpull%2F2215%23discussion_r258702625&data=02|01|alex.brisimitzakis%40cerner.com|a05529de84c441230b0808d69781e0c8|fbc493a80d244454a815f4ca58e8c09d|0|0|636862981411891624&sdata=RYwesnYe2XECIl2D2NmLTnLupvVo1jHpW6WtOh7SmA8%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAoBppHNWfoeXVyyiWqrDr2U6qs8thqgkks5vPcqbgaJpZM4aloMz&data=02|01|alex.brisimitzakis%40cerner.com|a05529de84c441230b0808d69781e0c8|fbc493a80d244454a815f4ca58e8c09d|0|0|636862981411901632&sdata=U%2Bte9wdKMCR4EIbmhwx14uUbh3z8YZgddUjvIFebVD4%3D&reserved=0.
CONFIDENTIALITY NOTICE This message and any included attachments are from Cerner Corporation and are intended only for the addressee. The information contained in this message is confidential and may constitute inside or non-public information under international, federal, or state securities laws. Unauthorized forwarding, printing, copying, distribution, or use of such information is strictly prohibited and may be unlawful. If you are not the addressee, please promptly delete this message and notify the sender of the delivery error by e-mail or you may call Cerner's corporate offices in Kansas City, Missouri, U.S.A at (+1)<PHONE_NUMBER>.

are these the expected style changes?
before

after

@emilyrohrbough Left a comment about the hashing of the class name. With hashing it through CSS modules, it results in unintending styling. Left a comment on how to get it to apply the styles as intended.

@AlexBrizi The terra-markdown wdio screenshots need to be updated now that the CSS is mapping correctly.


net/textproto: ability to skip conversion for canonical format

What version of Go are you using (go version)?

go version go1.9.4 linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/projects/go"
GORACE=""
GOROOT="/usr/lib/go-1.9"
GOTOOLDIR="/usr/lib/go-1.9/pkg/tool/linux_amd64"
GCCGO="/usr/bin/gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build449725707=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"

What did you do?
I want to use ReadMIMEHeader() from net/textproto/reader.go for reading headers on ICAP protocol. So I have a trouble with key := canonicalMIMEHeaderKey(kv[:endKey]) (line: 496), because ICAP protocol have a case sensitive headers (for ex: ISTag). Please can you add a new flag-variable to ReadMIMEHeader function for skip canonicalMIMEHeaderKey conversion.
What did you expect to see?
Something like this:
func (r *Reader) ReadMIMEHeader(toCanonical bool) (MIMEHeader, error)
and
if toCanonical == true {       key := canonicalMIMEHeaderKey(kv[:endKey]) }
What did you see instead?
func (r *Reader) ReadMIMEHeader() (MIMEHeader, error)
and
key := canonicalMIMEHeaderKey(kv[:endKey])

It seems like the decision here was not to add the feature, and we're not really waiting for any additional information even if this is labelled as waiting for info. I'm closing this, but feel free to comment if I'm wrong.


Safari doesn't work when Stealth mode is enabled

Settings - Privacy - Advanced tracking protection is checked
Open Safari and go to google.com or bing.com

Actual result:
https://uploads.adguard.com/Bing_2018-10-26_13-18-01.png

Most likely this is caused by the invalid stealth script configuration as navigator.userAgent returns "null" even though "hide user-agent" is unchecked in the settings.

@zebrum Can you please check with <IP_ADDRESS>1 nightly?

Thanks!

core/pull-requests/797


perf(travis): skip deploy on stage level

Pull Request Test Coverage Report for Build 1529

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 78.03%

Totals

Change from base Build 1528:
0.0%

Covered Lines:
93

Relevant Lines:
112

💛  - Coveralls


Video Screensaver

If some game as any video preview, play it?
Video resolution might not be enough for a full screen rendering..
Investigate.

This should help. It has already been implemented. Cheers
https://retropie.org.uk/forum/topic/8734/please-test-random-game-selection-launch-via-video-screensaver

I wasn't planning to work on this any time soon..
but if their implementation works smoothly I'll consider to borrow it.
Thanks ;)

@alextinti83 would you consider contributing your time towards Retropie-ES development? I see that you are actively developing ES. May I know if this is for personal use or for something else

Hey @hex007, I decided to implement myself  the features I needed since I was seeying very little progress on the RetroPie repo - ES side; they have other priorities AFAIU.
I doubt they're going to merge any of my branches or features implemented though,
I had to do several refoctors to the original code base to get them working as I wanted.
Moreover I couldn't really test them to be 100% sure I didn't break some other functionality..
I any case I'll try to back merge their changes time to time, here https://github.com/alextinti83/RetroStation/tree/retropie/sync_to_master

https://github.com/RetroPie/EmulationStation/pull/153 was merged to retropie.

Merged: 9a35746ec2e82acc2e4c0d1237e60eef99fb879d


File change detection does not work for BPMN and CMMN (DMN works)

Camunda Modeler Version: 1.8.0
Also changes are ignored when re-opening closed tab.

Works on Linux.
It seems however that the general file change detection is somewhat broken.

We check for changed files on app re-open
We check for file change on tab re-open

In both cases, I'd expect the new version of the diagram to just get opened.

Related to #563

Should be fixed via #563?

Cannot reproduce anymore.


tertretete

Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Describe alternatives you've considered

Additional context

? :D

? :D

Testing the auto workflows 😂


No media rules for devices under 576px

It seems there are rules missing for devices with widths under 576px.
For example:

The page will maintain a two column layout from 768px and upwards, however it will only maintain a single column layout between 576px to 768px. Sub 576px it will go back to a two column layout. I understand that "xs" was removed, but most mobile devices are under 576px in width, so something is seriously wrong here.

It's amazing how fast you can realize your own mistakes, after you type up the issue. Sorry. I understand now:


Unable to run danger-swift on Travis CI or locally

I'm just getting started with Danger, setting it up for an open source project that I manage that uses Travis CI. I've setup my Dangerfile.swift and .travis.yml based on the Danger in Swift guide. Here is the relevant PR that introduces Danger into my project.
Unfortunately, Travis CI fails my build due to an Unhandled 'error' event error that occurs in events.js.
I tried to run Danger locally as well, using the danger-swift pr https://github.com/BottleRocketStudios/iOS-Hyperspace/pull/74 command, but that fails as well (with a different error). Here is my console output from running that command:
danger-swift pr https://github.com/BottleRocketStudios/iOS-Hyperspace/pull/74
Running: /Users/tyler.milner/.nvm/versions/node/v8.12.0/bin/danger pr --process danger-swift https://github.com/BottleRocketStudios/iOS-Hyperspace/pull/74
Starting Danger PR on BottleRocketStudios/iOS-Hyperspace#74
You don't have a DANGER_GITHUB_API_TOKEN set up, this is optional, but TBH, you want to do this
Check out: http://danger.systems/js/guides/the_dangerfile.html#working-on-your-dangerfile
Dangerfile.swift:2:8: error: missing required module 'ShellOut'
import Danger

  ^

Created a temporary file for the Dangerfile DSL at: /var/folders/00/2_dwjx0x2k12p75w98nh_z5c00012l/T/danger-dsl.json
Running Dangerfile at: Dangerfile.swift
Running: /usr/bin/swiftc --driver-mode=swift -L /usr/local/lib/danger -I /usr/local/lib/danger -lDanger -L /usr/local/lib/danger -I /usr/local/lib/danger Dangerfile.swift /var/folders/00/2_dwjx0x2k12p75w98nh_z5c00012l/T/danger-dsl.json /var/folders/00/2_dwjx0x2k12p75w98nh_z5c00012l/T/danger-response.json
Completed evaluation
ERROR: Dangerfile eval failed at Dangerfile.swift
ERROR: Could not get the results JSON file at /var/folders/00/2_dwjx0x2k12p75w98nh_z5c00012l/T/danger-response.json

Danger: ⅹ Failing the build, there is 1 fail.
## Failures
danger-swift` failed.
## Markdowns
### Log

```sh
Dangerfile.swift:2:8: error: missing required module 'ShellOut'
import Danger
       ^
Created a temporary file for the Dangerfile DSL at: /var/folders/00/2_dwjx0x2k12p75w98nh_z5c00012l/T/danger-dsl.json
Running Dangerfile at: Dangerfile.swift
Running: /usr/bin/swiftc --driver-mode=swift -L /usr/local/lib/danger -I /usr/local/lib/danger -lDanger -L /usr/local/lib/danger -I /usr/local/lib/danger Dangerfile.swift /var/folders/00/2_dwjx0x2k12p75w98nh_z5c00012l/T/danger-dsl.json /var/folders/00/2_dwjx0x2k12p75w98nh_z5c00012l/T/danger-response.json
Completed evaluation
ERROR: Dangerfile eval failed at Dangerfile.swift
ERROR: Could not get the results JSON file at /var/folders/00/2_dwjx0x2k12p75w98nh_z5c00012l/T/danger-response.json

```

Am I doing something horribly wrong? I'm using the latest version 0.5.1 if that helps. Thanks for your support.

Interesting, maybe we're doing dependencies wrong in danger. I might make a release of the current build and we'll see if that fixes it

alright, released danger swift 0.6.0 - can you give that a try?

Thanks, @orta. Version 0.6.0 works locally now with the danger-swift pr https://github.com/BottleRocketStudios/iOS-Hyperspace/pull/74 command.
However, Travis CI is still having build issues (same Unhandled 'error' event error).

@orta re: Doing dependencies wrong.
We were just not shipping the ShellOut's swiftmodule and swiftdoc.
It was fixed here https://github.com/danger/swift/pull/91

Hi @tylermilner,
We just released the version 1.0.0
Can you please try again to see if this is still an issue please?

@f-meloni this is working for me now - both locally using the danger-swift pr https://github.com/BottleRocketStudios/iOS-Hyperspace/pull/74 command and on Travis-CI using the danger-swift ci command!
I was still having issues the past couple of days trying to test things out (even after switching danger process danger-swift to danger-swift ci), but I think it was mainly due to something else screwy with my .travis.yml configuration. Unfortunately, I made several changes at once so I'm not sure exactly what fixed it, but here's a summary of how I reworked my Travis-CI configuration:

Removed npm install -g danger from the before_install portion of my .travis.yml.
No longer trying to install danger-swift in the before_install portion of my .travis.yml. Instead, I'm using the Travis-CI homebrew addon - like so.
Switched danger process danger-swift to danger-swift ci as recommended.
Updated to the latest xcode10.1 Travis-CI osx_image.

The only thing that has me slightly concerned is that one of the jobs for my build seems to have timed out on the danger-swift ci command. This might be due to me running danger-swift ci for each job, rather than just once per build like I do for my pod lib lint command. I'll make this change in my Travis-CI configuration and keep an eye on it, but I think we can close this issue now. Thanks for your hard work, everyone!

@tylermilner thank you for the detailed steps, that will maybe help someone else that will end up in your same situation.
Given it looks solved, I will close this for now.

FWIW: The timed out is likely https://github.com/danger/swift/issues/97#issuecomment-447640815 - if you're using the swiftlint plugin, you can remove that as it's now shipping with Danger Swift


Cert-manager

Hello, we would like to start using cert-manager to request certificates. Is there more information available somewhere on how we should configure cert-manager?

Hello. To use cert-manager with the TameMyCerts REST API, you would have to write a connector for cert-manager (an Issuer in cert-manager terminology).


First draft of OpenStack-Ceph config

Good spots thanks Will - I had wanted a non-default name but according to Kolla documentation the Ceph config appears to be fixed to the default name of ceph.


resolve 2/3 security advisories

This doesn't fix the kind-of issue but does fix the other two alerts attached to this repo. Kind of is a lot messier so I think we can hold out a few more days to see if big maintainers like storybook and FB will fix their deps. It cleans up yarn audit output a ton.
minimist
security advisory: Prototype pollution in minimist
came from gonzales-pe pkg via postcss-sass

fixed in tonyganch/gonzales-pe#304 (=> 4.3.0)
incorporated in AleshaOleg/postcss-sass#143 (=> 0.4.4)

I could have fixed by patching yarn.lock alone, but used the force upgrade method for explictness here. See my work in b8e6805d, c0f0a829, and 791b1e27.
acorn
security advisory: Regular Expression Denial of Service in Acorn
TBH, came from bad yarn logic. This is a long-time bug in yarn which happens from time to time: our dependency tree had no reason to have both 6.4.0 and 6.4.1, but yarn isn't great at cleaning these dupes. So I manually patched yarn.lock in fac6225b.
reviewer notes
opened this based on yarn audit alone, so please check to ensure I didn't break the build? :)
yarn audit output-- before my changes (394 vulnerabilities)

yarn audit output-- after my changes (1 vulnerability)
this last vulnerability isn't from kind of. a closed issue on storybook claims this is already fixed, but upgrading to 5.3.18 did not resolve this. I am still bored so may open a PR to upgrade marksy & then can follow up to upgrade storybook if nobody else catches it.

I don't feel comfortable committing manual fixes directly to the yarn.lock file -- IMO they will be too easily lost and it's not clear enough what versions we're requiring and why. I found this yarn feature that sounds like a good enough alternative: https://classic.yarnpkg.com/en/docs/selective-version-resolutions/

@suzubara TIL! I will add those to package.json 👍


Update nom to 7.0

Thanks!
That's an interesting diff, why does it take the constructor function itself now?

The signature of fold_many0 was changed to take a function that returns the initial value, instead of taking the initial value directly. So we could write || BTreeMap::new(), which is equivalent to BTreeMap::new. I don't exactly know why they changed that, but I assume there is a case in which that function doesn't get called.


chore: import repository git@github.com:phimichel/sample-jx3-k3s-vault.git

this commit will trigger a pipeline to generate the CI/CD configuration which will create a second commit on this Pull Request before it auto merges

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


FBI 2.4.9 3dsx format error

fbi gives an error when i trying to  extract a save from an installed title currently using luma3ds and launching FBI with rosalina HB exploit

Are you trying to access the save of a NAND title, and if not, have you made sure to open the title in question and let it create save data?

@Steveice10 yes and every installed title (including system titles) gives me this error no matter if i create a save file or not

NAND/System titles cannot have save data, hence the error.

@Steveice10 but im talking about installed titles too like games or warez content (cias)

They are installed to the SD card and have existing save data?

yes

Could you provide a screenshot of the information of such a title shown on the top screen in FBI?

yes give me a second

I'm unable to reproduce this problem on my N3DS except for with titles that do not have save data created. I can only assume you don't actually have any save data for these titles or that there is some other issue with your setup.


SSLError raised when path to CA_BUNDLE is not valid could be more meaningful

Hi there,
First of all thanks for the requests library, it's just awesome!
In one of my project I was pulling 'verify' from a config file and passing it along every requests as an argument. Instead of receiving a boolean, it turned out that I was actually passing verify='true' to the requests methods, which makes requests think that I was providing a path to a ca bundle.
Although I was obviously misusing the api, it took me some time to debug it and I believe that the error message could have been more meaningful.
In fact the following:
requests.get('https://github.com/', verify='true')

raises
/.../requests/requests/adapters.py in send(self, request, stream, timeout, verify, cert, proxies)
    495         except (_SSLError, _HTTPError) as e:
    496             if isinstance(e, _SSLError):
--> 497                 raise SSLError(e, request=request)
    498             elif isinstance(e, ReadTimeoutError):
    499                 raise ReadTimeout(e, request=request)

SSLError: [Errno 2] No such file or directory

I'll do a PR and make it raise an IOError (with the path being printed) in case the verify param does not point to a valid file/dir. As minor as it is, I hope you'll find it valuable.

So, before you leap into this (which is a good idea), be aware that you need to be able to distinguish this case from when either of the paths given to the cert parameter are bad.

Thanks for the quick feedback, I'll take that into account


remove events we don't want right now

Pull Request Test Coverage Report for Build 49136

0 of 0   changed or added relevant lines in 0 files are covered.
13 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 74.454%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Builder/ActivityHandler.cs
13
73.97%

Totals

Change from base Build 49109:
-0.1%

Covered Lines:
3958

Relevant Lines:
5316

💛  - Coveralls


The link to the tutorial doesn't work

When I click the link to pen the tutorial it brings me to my games.

Good catch, it's linking to sprig.hackclub.dev rather than sprig.hackclub.com. I'll do a full repo search for old links that I somehow missed (I thought I did this before!)


Update investigating-partially-indexed-items-in-ediscovery.md

if a partially indexed item has metadata (metadata is indexed by definition) that matched the search condition(date range) then it will be returned with the regular Indexed items.
See https://docs.microsoft.com/en-us/microsoft-365/compliance/partially-indexed-items-in-content-search?view=o365-worldwide#messages-and-documents-with-partially-indexed-file-types-can-be-returned-in-search-results  👍
"Similarly, messages with partially indexed file attachments and documents of a partially indexed file type are included in search results when other message or document properties, which are indexed and searchable, meet the search criteria. Message properties that are indexed for search include sent and received dates, sender and recipient, the file name of an attachment, and text in the message body. Document properties indexed for search include created and modified dates. So even though a message attachment may be a partially indexed item, the message will be included in the regular search results if the value of other message or document properties matches the search criteria."

@Sokarpi PR merged. Thanks for making this update.


Missing Python package for MySQL

Description:
The mysql-connector-python package is not included in the requirements.txt file, causing issues with MySQL connectivity. Please add the package to ensure proper installation and functionality.
Expected Outcome:
Successful installation of the mysql-connector-python package when running pip install -r requirements.txt.

The issue has been resolved after implementing the changes mentioned above and closing the issue


Allocation of 67108864 exceeds 10% of system memory

Hello, what is the minimum computer configuration required to train the network?

You're using CPU to train this network, try to install tensorflow-gpu==1.13 with CUDA 10.0 then this issue would be gone

I installed TensorFlow-gpu ,but still, the code is running on CPU. I get the system memory error! Help please!

I installed TensorFlow-gpu ,but still, the code is running on CPU. I get the system memory error! Help please!
I also installed tensorflow-gpu==1.15, but it still remind me 'exceeds 10% of system memory'. Did tensorflow need some code to load the model into GPU such as 'to(params.device)' in the Pytorch?


Fix trivial typo

What is this PR for?
Fixes a trivial typo in travis_check.py.
The semicolon does nothing in travis_check.py.
Python use semicolon when several statements on the same line, here is python doc.
Questions:

Does the licenses files need update? NO
Is there breaking changes for older versions? NO
Does this needs documentation? NO

LGTM! 👍

LGTM merge if no more discussion.


AttributeError: 'NoneType' object has no attribute 'group'

Hello, I am working with Python 3.5.2 on macOS using Pyenv. When I tried the following example:
import matplotlib.pyplot as plt
plt.plot([1, 23, 2, 4])
plt.ylabel('some numbers')
viz.matplot(plt)

I encountered the following warning:

File "", line 1, in
File "/Users/MufeiLi/.pyenv/versions/3.5.2/lib/python3.5/site-packages/visdom-<IP_ADDRESS>-py3.5.egg/visdom/init.py", line 482, in matplot
opts['height'] = 1.4 * int(re.search('height="([0-9]*)pt"', svg).group(1))
AttributeError: 'NoneType' object has no attribute 'group'

Hi @mufeili - We're still working on getting our matplotlib integration down fully, and this bug seems to come out of one of the ways we were getting things to work. I'll investigate a fix!

It appears that not all versions of matplotlib produce the same SVG (given the same input).
@mufeili To help me debug this, could you add a print(svg) in visdom/init.py:481 and send me a Gist containing the output of that print?

Hi @lvdmaaten , you may find the result here result of print(svg). The result is from Python 3.6 installed by Anaconda rather than what I described initially due to my attempts to fix it myself. Thanks.

Ah @lvdmaaten we're missing catching a decimal in the regex!

Yep :)


doc: autogenerate python interpreter table

This serves as a practical example on generating documentation by inspection of the evaluated nixpkgs tree.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/2024-05-21-documentation-team-meeting-notes-129/45796/1

@fricklerhandwerk @alejandrosame What do you two think of the following?
Can be checked with nix eval -f <file> --raw.
I simplified the sorting by doing it when I get the list of interpreters.
Getting the "cute" name and aliases are turned into functions that only need knowledge of the interpreter name.
let

  pkgs = import ./. {
    config = { };
    overlays = [ ];
  };
  lib = pkgs.lib;

  inherit (lib.attrsets) attrNames filterAttrs;
  inherit (lib.lists) naturalSort;
  inherit (lib.strings) concatStringsSep;

  isPythonInterpreter = name: (lib.strings.match "(pypy|python)([[:digit:]]*)" name) != null;
  interpreters = naturalSort (
    builtins.filter isPythonInterpreter (builtins.attrNames pkgs.pythonInterpreters)
  );

  getCuteName =
    interpreter:
    let
      cuteName = {
        cpython = "CPython";
        pypy = "PyPy";
      };
      pythonInterpreter = pkgs.${interpreter};
    in
    "${cuteName.${pythonInterpreter.implementation}} ${pythonInterpreter.pythonVersion}";

  getAliases =
    interpreter:
    builtins.attrNames (
      filterAttrs (
        name: value:
        if isPythonInterpreter name then
          value.outPath == pkgs.${interpreter}.outPath && name != interpreter
        else
          false
      ) pkgs
    );

  result = builtins.map (interpreter: {
    inherit interpreter;
    cuteName = getCuteName interpreter;
    aliases = getAliases interpreter;
  }) interpreters;

  toMarkdown =
    data:
    concatStringsSep "" (
      map (interpreter: ''
        | ${interpreter.interpreter} | ${
          concatStringsSep ", " interpreter.aliases or [ ]
        } | ${interpreter.cuteName} |
      '') data
    );
in
''
  | Package | Aliases | Interpeter |
  |---------|---------|------------|
  ${toMarkdown result}
''

I get this markdown output
| Package | Aliases | Interpeter |
|---------|---------|------------|
| pypy27 | pypy, pypy2 | PyPy 2.7 |
| pypy39 | pypy3 | PyPy 3.9 |
| pypy310 |  | PyPy 3.10 |
| python27 | python, python2 | CPython 2.7 |
| python39 |  | CPython 3.9 |
| python310 |  | CPython 3.10 |
| python311 | python3 | CPython 3.11 |
| python312 |  | CPython 3.12 |
| python313 |  | CPython 3.13 |

Rendered

Package
Aliases
Interpeter

pypy27
pypy, pypy2
PyPy 2.7

pypy39
pypy3
PyPy 3.9

pypy310

PyPy 3.10

python27
python, python2
CPython 2.7

python39

CPython 3.9

python310

CPython 3.10

python311
python3
CPython 3.11

python312

CPython 3.12

python313

CPython 3.13

@Djacu yeah nice!

@djacu Yeah, and checking for outPath also simplifies the code and makes the alias collection more generic

@djacu wait, how did you get past the Python 2.7 EOL error? We have to evaluate derivations if we want to access outPath, and this is where the allowInsecure assertion will hit.

@djacu wait, how did you get past the Python 2.7 EOL error? We have to evaluate derivations if we want to access outPath, and this is where the allowInsecure assertion will hit.

Meh, so still there's the point that going through outPath is "unreliable": not guaranteed  to work at the level we care here, identifying aliases from an evaluated tree.
Is parsing the nixpkgs AST the only truly general solution to discover aliases? (To be clear, it would be out of scope of this PR, but in the end this is the underlying issue that surfaces when traversing pkgs to get aliases).

@djacu wait, how did you get past the Python 2.7 EOL error? We have to evaluate derivations if we want to access outPath, and this is where the allowInsecure assertion will hit.

export NIXPKGS_ALLOW_INSECURE=1 :P

outPath was the one thing I was suspicious of and forgot to test the script without export NIXPKGS_ALLOW_INSECURE=1. If you run it without the value.outPath == pkgs.${interpreter}.outPath expression, you don't need the insecure flag, but it obviously gets the wrong answer. I was mostly trying to figure out if I could simplify the logic. If we can resolve the insecure/outPath issue, I think we have something really nice.

Wow, this got a lot shorter


NatGen-base.commons-jxpath.mid-996.idx-12519.2.buggy

Added proper indentation

Line 125 should be an 8-space indentation since you are continuing the line! (See line 144-145) :)

Done
Thanks

Please check This branch cannot be rebased due to conflicts. Sorry if we accepted some PR before yours.

Closing due to conflicts. Sorry if we accepted some PR before yours.

Reopening to delete the LICENSE file.


Legend backend API

For #9584, we need an API in order to get, edit and update all legend data. Until now, only one legend per layer was allowed, but from now on, we can have up to two legends per layer. It would be nice to make it ready for more than two though.
We have five types of layers: html, category, bubble, choropleth and custom. The type of data we are going to save is:

For all types, title: it could be optional
For all types, two fields for raw html. The user can add custom html before or/and after the html of the legend, that it can't be changed by the user.
For html type, one field with html raw. This html type is created to migrate the old legends to the builder.
For category and custom, a collection of pairs of title and color or color and icon.
For bubble, a fill color
For gradient, a prefix and suffix (text)

At vizjson level, every layer node will have a legends node of type array.
If you need access to mockups in order to check the UI, ping me.

Related to https://github.com/CartoDB/cartodb.js/pull/1401 for details in the vizJSON.

In order to make more understandable the part of the custom html before and after the legend's code, I made a quick demo for the UI: http://codepen.io/nobuti/pen/BLoVzz. In it you will see the user can add code before and after some line marked as readonly.
👆 @alonsogarciapablo @piensaenpixel

@nobuti nice! Thanks for the demo!

Hey Buti, thanks for the ticket, it's very complete!
Some questions:
a) In categories, do you mean cloropeth instead choropleth?
b) How do you call the pre and post html in front? Let's have a consistente naming. Same goes for the others, if you've picked out names, map them out!
Thanks again 💋

@gfiorav here https://github.com/CartoDB/cartodb/pull/9738/files#r78395473 you can see the naming, it follows cartodb.js' convention

Closed via https://github.com/CartoDB/cartodb/pull/9809