The lack of documentation is a real hinderance

Hi, this code seems to have taken quite some effort, which is why I'm surprised that it's almost completely undocumented. By that I mean only source code is provided, even the official documentation is just source code with a search bar, no doc comments.
As a new user, this makes it extremely hard to use: things that could otherwise take a minute can take upwards of an hour (not to mention the possibility of something not even being supported).
I know this is supposed to be a thin wrapper around Box2D, however the difference is still large enough to cause confusion and necessitate documenting the edge cases.
Of course, for example, the step function or the idea of fixtures don't have to be documented. But how UserDataTypes are used and hundreds of other such things induced by the language difference totally require explanation.

Hi,
Thanks for the feedback, I am happy to add documentation where users would find it useful (I also welcome contributions).
Do you have specific features, other than user data, that you wish were explained?

@corazza I will add documentation for the user data soonish, let me know if you have other things in mind.

The master documentation upload seems to be broken, but I added some documentation for user data:
https://github.com/Bastacyclop/rust_box2d/compare/7b504a426f6e8f67ca5c17137102e0e524f20664..ebdc4da1197b9f5c9dd80afaa0cdc03b974c2cf6

Thank you for this response, and for your time! And sorry for not getting back to you, I'm on a pause from this project until the semester is over, and rarely check my GH notifications.
Anyways, I wish I'd documented my own path through using this crate, but as it turns out I don't remember the exact stuff that I wished was better documented. I'll try to backtrace a bit through my project to maybe come up with examples, but in a month or so when I'll have time.
What would be most useful, however, are examples for frequently-used functions, and some overall examples on how to achieve simple tasks that a user might want. Maybe when I catch some time, I'll do a writeup on my blog, and examples of this kind could be extracted from there.
I'll try to be more helpful and provide examples later, as I said.

I'll close for now since there are no concrete plans on the way.


Op.not operator shows a warning

Issue
Using Op.not operator in any of the queries, Eg: findAll, findOne.. shows a warning in my code editor (VS Code).
But the code compiles and runs without any error.
where: {
  [Op.not]: true
}

Error:
Type '{ [Op.not]: true; }' is not assignable to type 'WhereOptions<any>'.
  Types of property '[Op.not]' are incompatible.
    Type 'true' is not assignable to type 'undefined'.ts(2322)
model.d.ts(850, 3): The expected type comes from property 'where' which is declared here on type 'UpdateOptions<any>'

Versions

sequelize: 6.6.5
sequelize-typescript: 2.1.0
typescript: 4.2.3

Issue type

[x] bug report
[ ] feature request

Actual behavior

Expected behavior

Steps to reproduce

Related code

It's most likely issue of sequelize types rather than sequelize-typescript as it doesn't modify these types, just extends couple of model related ones. Would recommend opening issue in sequelize repo if not already opened.

We have recently fixed this in Sequelize 7, but it hasn't been backported to Sequelize 6 yet https://github.com/sequelize/sequelize/pull/14022


when field validator message is displayed and clicking New button will cause infinite loop

Hi, this is an edge case which will cause the browser to hang.
Repro steps:

Run the react-bootstrap-table example project locally
Navigate to the "Advanced data edit & Insert" option under Examples
Scroll to the last example
click the job name cell. Remove some characters so validator will display validation msg
before the msg disappear, click the "+new" button.

Result
browser will hang

@konghoiwai, thanks you, this issue has been solved on v3.0.0-dev, check #497 for details.  I we release v3.0.0 as soon as possible

The v3.0.0-beta.7 already released and fixed on it, check this out, it's certainly stable for developer :)


Auto Commenting when a PR is merged

This feature aims to solve the problem of inconsistent and delayed communication following the merging of PRs. By automating the commenting process, it ensures that contributors receive timely feedback and acknowledgments. It also reduces the administrative burden on maintainers, allowing them to focus more on code reviews and other critical tasks.
Could you please assign me this issue under GSSOC'24.

Could you elaborate more on this issue and how exactly are you planning to implement it?

I will adding a feature such that whenever a new pull request is merged, automatically a comment will be added on it. To do this, I will be adding a YAML file in the .github directory to define this workflow of auto commenting on the trigger of a pull request being merged.

I will adding a feature such that whenever a new pull request is merged, automatically a comment will be added on it. To do this, I will be adding a YAML file in the .github directory to define this workflow of auto commenting on the trigger of a pull request being merged.

What would be in the comment? because an AI code reviewer is configured along with a first commit that that wait for the reviewers to get through this. If you are planning to add a time based comment passing system so that it can ask for the contributors about their status then go ahead...


Do not try to close unused fp in advance

This breaks internal Pillow's workflow.

Codecov Report

Merging #18 (611d05a) into main (fc79c1f) will increase coverage by 0.25%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #18      +/-   ##
==========================================
+ Coverage   99.49%   99.74%   +0.25%
==========================================
  Files           6        6
  Lines         398      398
==========================================
+ Hits          396      397       +1
+ Misses          2        1       -1

Impacted Files
Coverage Δ

HeifImagePlugin.py
99.26% <ø> (+0.70%)
:arrow_up:

tests/test_reading.py
100.00% <100.00%> (ø)

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 fc79c1f...611d05a. Read the comment docs.


Enable icmp error replies with enable-pmtu-discovery flag

ICMP error replies (packet size too big) are sent
in dsr_reply_icmp4 and dsr_reply_icmp6 functions.
These replies are sent only when cilium is enabled
in LBOnly mode and NodePortAcceleration is set to true.
This change allows icmp error replies to be sent to the
client in cilium CNI mode when enable-pmtu-discovery
flag is set to true. By default, this flag is false.
Fixes: #21795
Signed-off-by: Nishant Burte<EMAIL_ADDRESS>

@jibi I have added the new flag as to whether to send icmp error replies or not.
Also, I have tested the icmp replies with tc hook using tcpdump.
For ipv4, tcpdump captured the icmp reply packet as
19:20:04.808758 IP <IP_ADDRESS>.2000 > <IP_ADDRESS>.3331: UDP, length 1465
19:20:04.808795 IP <IP_ADDRESS> > <IP_ADDRESS>: ICMP <IP_ADDRESS> unreachable - need to frag (mtu 1492), length 36

For ipv6, tcpdump captured the reply packet as
17:47:27.885842 IP6 2620:0:1000:2631:3:10:3:1d0 > 2620:0:1000:2631:250:56ff:fe82:28d6: ICMP6, packet too big, mtu 1476, length 112

/test

@jibi Could you please review this?

Looking good!
One last thing is that this new option should also be exported in the Helm chart, by defining a new value and then using it in the configmap template. One example of that:

https://github.com/cilium/cilium/blob/b4a4b62296e8fed33c669fd1dd885f7645c0a16d/install/kubernetes/cilium/values.yaml#L321-L325
https://github.com/cilium/cilium/blob/b4a4b62296e8fed33c669fd1dd885f7645c0a16d/install/kubernetes/cilium/templates/cilium-configmap.yaml#L863-L867

(after that remember to run make -C Documentation update-helm-values to update the docs).
I think it should also be mentioned in the upgrade document, under the New Options paragraph

@jibi Thank you. I have addressed your comment.

/test

/test-1.23-5.4

/test-1.25-net-next

@nnbu could you try rebasing this PR please as that may help with the failing tests?

@nnbu could you try rebasing this PR please as that may help with the failing tests?

@jibi done

@jibi 3 tests failed.

External workloads (ci-external-workloads)  and Multicluster / Cluster mesh (ci-multicluster) failed in Create GKE cluster step.

ERROR: (gcloud.container.clusters.create) ResponseError: code=400, message=Request contains an invalid argument.
The arguments of cluster create are not related to this change.

Cilium Datapath (ci-datapath) failure is this

[2022-10-25T18:32:24Z] ❌ 1/31 tests failed (1/237 actions), 0 tests skipped, 0 scenarios skipped:
[2022-10-25T18:32:24Z] Test [client-egress-to-echo-service-account-deny]:
[2022-10-25T18:32:24Z]   ❌ client-egress-to-echo-service-account-deny/pod-to-pod/curl-1: cilium-test/client2-bf46d8f5d-w5bkr (<IP_ADDRESS>) -> cilium-test/echo-same-node-dd6764db8-w459g (<IP_ADDRESS>:8080)

I am not sure how this could be related to my change. Probably, flaky test?

@nnbu could you try rebasing this PR please as that may help with the failing tests?

@jibi done

Great, that did the trick :+1:
One last thing I just notice: can you update the commit message as we are not actually enabling this by default? Then we should be good to go

One last thing I just notice: can you update the commit message as we are not actually enabling this by default? Then we should be good to go

Done

ct-external-workloads is failing to provision the cluster (https://github.com/cilium/cilium/pull/21885 should fix it), all other required tests are green, marking as ready

@jibi I do see ready-to-merge label.  So I thought, PR would get merged automatically. So just checking if anything is pending from my side on this one.

@nnbu nothing pending from your side :slightly_smiling_face: we use that label as an indication for the person who is on top hat duties to merge the PR

@nnbu Thanks for your contribution :)

@nnbu Can it work in hostport mode and kubernetes service clusterip and more ?


Fix for OWLS-82511 - OPERATOR NOT RECOVERING GRACEFULLY FROM LACK OF …

Fix for OWLS-82511 - OPERATOR NOT RECOVERING GRACEFULLY FROM LACK OF CRD.
In replace CRD Step, in case of failure due to missing permissions, continue to onSuccess case. okhttp3 client throws StreamResetException when operator doesn't have update/patch permissions for CRD. It checks for StreamResetException in onFailure and continues to onSuccess.

This PR is superseded.


Bugfix: Log format output type does not match. v5.0.171, v6.0.66

A segmentation fault occurred on arm https://github.com/ossrs/srs/issues/3714
Please describe the summary for this PR.
The 64-bit variable crashed after using the %d output on the 32-bit arm

Thanks @terrencetang2023


Bulk uploader static site guidance (content)

Guidance on static site (bulk uploader overview)
Include how it works overview for the guide (this should include how to send to multiple STLTs - one spreadsheet)

PR #391 adds results uploader guidance to the static site.


Update build pack, dependencies and update Python to 3.6.6

Issue number: n/a
Description of change
Updates to the latest build pack and Python 3.6.6 and also updates a few dependencies. Change logs:

https://docs.python.org/3.6/whatsnew/changelog.html#python-3-6-6-final
http://docs.celeryproject.org/en/v4.2.1/changelog.html
https://github.com/django-extensions/django-extensions/blob/master/CHANGELOG.md
https://github.com/boto/boto3/blob/develop/CHANGELOG.rst
http://www.gevent.org/changelog.html

Checklist

[x] Have any relevant search models been updated?
[x] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[x] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[x] Has the admin site been updated (for new models, fields etc.)?
[x] Has the README been updated (if needed)?

Codecov Report

Merging #1030 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1030   +/-   ##
========================================
  Coverage    96.47%   96.47%
========================================
  Files          211      211
  Lines         5848     5848
  Branches       559      559
========================================
  Hits          5642     5642
- Misses         141      142    +1
+ Partials        65       64    -1

Impacted Files
Coverage Δ

datahub/core/admin.py
82.6% <0%> (ø)
: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 57e0bce...9b73497. Read the comment docs.


CVE-2016-2177 and CVE-2016-2178

We got some new openssl CVEs which have been fixed in Alpine (fixes aren't officially released upstream just yet) -- would you mind kicking off a rebuild of at least 3.3 and 3.4 (which is where packages were fixed)? :smile: :pray:

alpine 3.1 and 3.2 are also fixed now.


Using did:key as a way to express keys

RFC 0360: did:key Usage describes the usage of did:key DIDs as a replacement for base58 encoded ed25519 keys in several protocols.
The RFC mentions:

The use of a DID to represent a public key is seen as odd by some in the community. Should a representation be found that is has better properties than a plain public key but is constrained to being "just a key", then we will consider changing from the did:key representation.

And I think I'm on the side of the community that finds this quite odd. It feels like using did:key to identify keys is misusing a did method that seems to work for this use case. DIDs are not the same as keys and therefore I don't think we should use dids to identify a single key.
If we were to remove the did:key prefix from the entries, I think we're exactly expressing what we want to express. A key encoded in such a manner that we can determine the encoding and the key type. If you already support did:key this is trivial to support because it's the same as extracting a key from a did:key.
The did core is also mentioning the use of multi base public key (as publicKeyMultibase).
I'm not fully up to date on the discussions around this topic, so if I'm missing something please let me know, but my current thought would just be to remove the did:key prefix.
So instead of this:
{
    "@id": "123456781",
    "@type": "<baseuri>/mediate-grant",
    "endpoint": "http://mediators-r-us.com",
    "routing_keys": ["did:key:z6Mkfriq1MqLBoPWecGoDLjguo1sB9brj6wT3qZ5BxkKpuP6"]
}

We'll encode keys like this:
{
    "@id": "123456781",
    "@type": "<baseuri>/mediate-grant",
    "endpoint": "http://mediators-r-us.com",
    "routing_keys": ["z6Mkfriq1MqLBoPWecGoDLjguo1sB9brj6wT3qZ5BxkKpuP6"]
}

Or to be more explicit:
{
    "@id": "123456781",
    "@type": "<baseuri>/mediate-grant",
    "endpoint": "http://mediators-r-us.com",
    "routing_keys_multibase": ["z6Mkfriq1MqLBoPWecGoDLjguo1sB9brj6wT3qZ5BxkKpuP6"]
}

I don't think it is worthwhile to change this now (the change to did:key is already making it harder to support both AIP 1.0 and 2.0), but I'd be interested to hear some thoughts on this.

Discussed in WG Call. DIDComm v2 will allows DIDs for routing keys (as it allows more flexibility to let a mediator share a DID instead of a specific key).
Current spec is just an in between of only allowing did:key dids, while DIDComm v2 will allow. There's still the question of whether we want to use did:keys when only wanting to express keys.


Gaussian blur is broken

Hello!
I tried to use the GaussianBlur augmenter, but it seems to be broken. You can see my results in this image:

The code that used:
imgaug.seed(random.randint(0,1000000))`
imgaug_sequence = imgaug.augmenters.Sequential([ imgaug.augmenters.GaussianBlur(sigma=(0,1)) ], random_order=True)
img = imgaug_sequence.augment_images(img)

img.shape is (640, 368, 3), dtype is uint8
imgaug version is 0.2.6, I installed it using pip3.
Am I doing something wrong?

If i change the sigma parameter to "sigma=10" I get the following:

Its only working on one axis and somehow gets black and white.

Found the problem:
It only works if I pass a 4D array/list (as described in the docs).
Please add a warning/exception to the augment_images function if it receives an incorrect format.

If you want to augment a single image, you have to use either augment_image(image) or augment_images([image])[0] or augment_images(np.array([image]))[0].
An exception currently cannot be raised in augment_images() upon 3-dimensional inputs, because the library also accepts grayscale images of shape (height, width) (i.e. no channel axis) and hence the input can be of shape (N, H, W).
However, a warning is currently raised if the input is 3-dimensional and the last axis has value 1 or 3, which should catch most false inputs and should have appeared in your case. The warning might not be in the pypi version of the library yet -- not sure.


[5.0] Allow jsonb field key notation for postgres

Connected with #8240 issue. After this commit it's possible to do this
$content = \App\Content::where('id', '=', 1)->select('id', "title->'en' AS title")->get();

Please fix the cs.

Coding style - what did I miss?

Braces are in the wrong place, and you need correct the placement of spaces.

You can use DB::raw for this.


Restrict browsing based on app certificates

It's sometimes useful to restrict browsing of content. The best way to do this is by package name and signing certificates. This code existed in v1 and should be brought over to the new version.

Added in commit 41db5f9


Unicode Characters not Supported as Keywords.

Keywords containing unicode characters do not display correctly as titles. Specifically, they display as '??'.

If a cookie is set in which there is a keyword containing unicode characters, the website displays and works fine. The problem must be present before the cookie is generated.


feat(MeasureTheory/Constructions/Polish): generalize topological assumptions to measurable ones.

Several theorems in MeasureTheory/Constructions/Polish.lean about measurable sets, functions, etc. assume that a space is a BorelSpace or OpensMeasurableSpace for some nice topology. This PR removes the topological data from such theorems when possible, replacing them with sufficient and natural assumptions on MeasurableSpace's.
The old versions of the theorems still work automatically thanks to TC inference.
TODO: Add CountablySeparated typeclass abbreviating HasCountableSeparatingOn _ MeasurableSet univ.

bors r+


Update cats-effect to 3.3.5

Updates org.typelevel:cats-effect from 3.3.4 to 3.3.5.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "cats-effect" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #143.


WalletSendTransactionError: Blob.encode[recentBlockhash] requires (length 32) Uint8Array as src

Describe the bug
I have a code which is on NextsJs React code and runs when I run it with npm run dev, But when I build it with npm run build followed by pm2 restart rps I get this error:
WalletSendTransactionError: Blob.encode[recentBlockhash] requires (length 32) Uint8Array as src
    at el.sendTransaction (665.1ad8f2fdfae3f382.js:1:42460)
    at async 665.1ad8f2fdfae3f382.js:1:48390 el {_events: n, _eventsCount: 4, signTransaction: ƒ, signAllTransactions: ƒ, signMessage: ƒ}
(

My code is :
    const { connected, publicKey, sendTransaction } = useWallet();

    ....

    const connection = new Connection(clusterApiUrl("devnet"), "confirmed");
    const dmlcoWallet = new PublicKey(<MyWalletPublicKey>);

      const transaction = new Transaction().add(
        SystemProgram.transfer({
          fromPubkey: publicKey,
          toPubkey: dmlcoWallet,
          lamports: LAMPORTS_PER_SOL * data.pay,
        })
      );
      transaction.feePayer = publicKey ?? profile.value.walletPublicKey;

      let recentHash = await connection.getLatestBlockhash('confirmed');
      transaction.recentBlockhash = recentHash.blockhash;

      const signature = await sendTransaction(transaction, connection,{preflightCommitment: 'confirmed',});

      recentHash = await connection.getLatestBlockhash('confirmed');
      // transaction.recentBlockhash = recentHash.blockhash;

      await connection.confirmTransaction({
        blockhash: recentHash.blockhash,
        lastValidBlockHeight: recentHash.lastValidBlockHeight,
        signature: signature,
      });

The code executes to this ponint : sendTransaction(...) I run both debug & production on devnet.
To Reproduce
Just use the code and build the NextJs app for production and run It.
Expected behavior
It should work without bug in production like when I run it on dev mode.

This is not an error thrown by Wallet Adapter. What wallet are you connected with when calling sendTransaction?

Hello, any update on this? I got the same problem...


⚠ Bofrost.at has degraded performance

In 1da82b0, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1731 ms

Resolved: Bofrost.at performance has improved in beb9015 after 36 minutes.


Consider giving more horizontal space to API references docs

Would it make sense to make the docs site use the full width of larger screens? In the example below, only about 1/3 of my screen's width is used for the API reference that I want to see, with many code blocks requiring to scroll horizontally.

I understand this may not be applicable to all pages on the website, like blog posts, but maybe we could experiment with API reference only.

Another option that may make these docs seem less narrow would be to lint our code samples for line length, since the width of the page today conforms to UX readability guidelines.


netlrts compilation fails with ft builds

Original author: Nikhil Jain
Original issue: https://charm.cs.illinois.edu/redmine/issues/470

No body.

Original date: 2014-04-28 00:32:37

Tested to compile, link, and run.


10-backend-server-transform-fn-Olus

Description
transforms data to fit a predefined model, model will be updated to reflect new db schema
Related to
#10
Checklist:

[x] I have carefully reviewed my own code
[x] I have commented my code

thanks @OlushoreOdude. Great job. Reviewed and merged.


[WIP] adding --replace flag to restart webconsole pod

What type of PR is this?
Feature
What this PR does / Why we need it?
Addresses the issue with the web console pod failing to start if it is already running in a different terminal session
Which Jira/Github issue(s) does this PR fix?
https://issues.redhat.com/browse/OSD-23459
Implemented as suggested in https://redhat-internal.slack.com/archives/C016S65RNG5/p1715785884588929
Unit Test Coverage
Before writing the unit test, I would like to see if the functionality fits the purpose

/test images

/retest

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

Project coverage is 45.42%. Comparing base (c090c68) to head (e979566).

Files with missing lines
Patch %
Lines

cmd/ocm-backplane/console/console.go
50.00%
7 Missing :warning:

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #583      +/-   ##
==========================================
+ Coverage   45.39%   45.42%   +0.02%
==========================================
  Files          87       87
  Lines        6759     6770      +11
==========================================
+ Hits         3068     3075       +7
- Misses       3311     3315       +4
  Partials      380      380

Files with missing lines
Coverage Δ

cmd/ocm-backplane/console/console.go
37.58% <50.00%> (+0.37%)
:arrow_up:

Closing this PR in favor of https://github.com/openshift/backplane-cli/pull/584 as the initial suggested way by passing --replace flag is not feasible due to container order.


Mac OS generated code issue

I cannot use this plugin on Mac OS:
Target of URI doesn't exist: 'package:file_saver/file_saver_web.dart'.
Try creating the file referenced by the URI, or Try using a URI for a file that does exist.dart(uri_does_not_exist)

Please run flutter clean and flutter pub upgrade with latest version of plugin (0.0.10)

The issue is fixed.


Adjusting the threshold to decrease the false postive

I am using the Total Segmentator while it seem have some false postive results. Could I adjust the threshold of the softmax results to get less false postive results?

Theoretically you could do that, but this will result in missing a lot true positives. The default threshold of 0.5 typically works best and adjusting it does not help in most cases. Therefore TotalSegmentator does not contain an option to adjust the threshold.


12 Related Safe Addresses

Related Safe Addresses
0xfd917CB00c96ae6E346E6325dFc65251Cf12f1E3
0x714f325150edEf2c507133dbe5950B0BdC6623a5
0x21D28D75e9e9809BcF1fde68370beC44477D65cC
0xDf11763F5EfeF4Cd997644beEf7ecE2d1C1B3d68
0xdA9EF9667dd95653De0479D798BB8716A06A2374
0x80359dA842Bfaa8995f4aDAC9f96310acA88E35f
0xC1C8e9E0BFD0ed35adcc14C428f923Ba907B6058
0x8EFdea0f3496Ab47B3Abcbe364eFc86404f78E57
0xcC0c41CD1129b6425415465bBbD6195BC6080B94
0xa02eE579B28C4f568913eB64967A6DF3B25f039F
0x737B8949CD26d39529C9dD565252Fd931DF18018
0xA14dd663920c973EF229F52365d67D4cBafAf167
Reasoning
0x497c20fed24d61c7506ef2500065e4fd662f3779 created 52 Gnosis Safes on Dec 14, 2021. Out of those 52 only 12 are eligible for the airdrop.
The 12 eligible Gnosis Safes were all created between Dec-14-2021 09:48:53 AM +UTC and Dec-14-2021 01:23:55 PM +UTC. So within less than 4 hours all those were created by the same address, which didn't create another Gnosis Safe at any other point in time.
The pure magnitude of Safe creations in such a short period of time is not explainable by anything other than the desire to farm a potential Gnosis Safe airdrop.
Methodology
The reported addresses were discovered by checking for irregularities in the Safe creations with the Safe: Proxy Factory 1.3.0 contract. On Dec 14, 2021 I identified a quite large number of Safe creations that were made by a single address and then decided to check those for further information:

Safe Address
Creator
Safe Date Created
Tokens Allocated

0xfd917CB00c96ae6E346E6325dFc65251Cf12f1E3
0x497c20fed24d61c7506ef2500065e4fd662f3779
Dec-14-2021 09:48:53 AM +UTC
411.2889196892567

0x714f325150edEf2c507133dbe5950B0BdC6623a5
0x497c20fed24d61c7506ef2500065e4fd662f3779
Dec-14-2021 09:53:34 AM +UTC
408.1229261423787

0x21D28D75e9e9809BcF1fde68370beC44477D65cC
0x497c20fed24d61c7506ef2500065e4fd662f3779
Dec-14-2021 09:54:48 AM +UTC
320.69256874799527

0xDf11763F5EfeF4Cd997644beEf7ecE2d1C1B3d68
0x497c20fed24d61c7506ef2500065e4fd662f3779
Dec-14-2021 09:56:50 AM +UTC
341.439156362875

0xdA9EF9667dd95653De0479D798BB8716A06A2374
0x497c20fed24d61c7506ef2500065e4fd662f3779
Dec-14-2021 09:58:17 AM +UTC
285.41356920498333

0x80359dA842Bfaa8995f4aDAC9f96310acA88E35f
0x497c20fed24d61c7506ef2500065e4fd662f3779
Dec-14-2021 12:58:03 PM +UTC
307.0794360852565

0xC1C8e9E0BFD0ed35adcc14C428f923Ba907B6058
0x497c20fed24d61c7506ef2500065e4fd662f3779
Dec-14-2021 01:01:50 PM +UTC
256.62792622598965

0x8EFdea0f3496Ab47B3Abcbe364eFc86404f78E57
0x497c20fed24d61c7506ef2500065e4fd662f3779
Dec-14-2021 01:04:02 PM +UTC
195.9539799186103

0xcC0c41CD1129b6425415465bBbD6195BC6080B94
0x497c20fed24d61c7506ef2500065e4fd662f3779
Dec-14-2021 01:07:13 PM +UTC
194.69121502609536

0xa02eE579B28C4f568913eB64967A6DF3B25f039F
0x497c20fed24d61c7506ef2500065e4fd662f3779
Dec-14-2021 01:12:43 PM +UTC
230.70563627958202

0x737B8949CD26d39529C9dD565252Fd931DF18018
0x497c20fed24d61c7506ef2500065e4fd662f3779
Dec-14-2021 01:15:52 PM +UTC
211.90060820487372

0xA14dd663920c973EF229F52365d67D4cBafAf167
0x497c20fed24d61c7506ef2500065e4fd662f3779
Dec-14-2021 01:23:55 PM +UTC
247.40730096858064

Safe Address
0x81058ff64a2D765E73fC04c6a19E051701D101C8

Furthermore, all 12 related safe addresses have 5 Safe Owners, out of which 4 (co)-owners are always the same:
0xA41, 0xa17, 0xb81, 0xbC8
1 2 3 4 5 6 7 8 9 10 11 12

Thanks for the report.

Furthermore, all 12 related safe addresses have 5 Safe Owners, out of which 4 (co)-owners are always the same:

Owning several Safes is quite common.
Looking at the Safes, it seems like they all own some ERC20 called UNION. No idea what this is, but the txs look to me like it's distributing the token to team members. e.g. this tx could be that. Hence there is a non-zero chance this would eliminate legitimate usage.


feat: Remove unencryptedDataKeyLength

It is a feature that is not being used.
This was created to let callers confirm that the unencrypted data key matched the algorithm suite,
but this checking is done directly into the cryptographic material now.
The hasUnencryptedDataKey is the best way to check for existence,
and the materials themselves will ensure that algorithm suite specifications are meet.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Check any applicable:

[ ] Were any files moved? Moving files changes their URL, which breaks all hyperlinks to the files.

NOTE: This breaking change is ONLY done because the project is in beta.
Otherwise there would be a major version bump.


RuntimeError: Tried to trace  but it is not part of the active trace. Modules that are called during a trace must be registered as submodules of the thing being traced.

🐛 Bug

To Reproduce
Steps to reproduce the behavior:
#%%
# Copyright (c) Facebook, Inc. and its affiliates. All Rights Reserved.
from __future__ import division
import os

import numpy
from io import BytesIO
from matplotlib import pyplot

import requests
import torch

from PIL import Image
from maskrcnn_benchmark.config import cfg
from predictor import COCODemo
from maskrcnn_benchmark.structures.image_list import ImageList

#%%
# load config from file and command-line arguments

project_dir = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
cfg.merge_from_file(
    os.path.join(project_dir,
                    "configs/caffe2/e2e_mask_rcnn_R_50_FPN_1x_caffe2.yaml"))
cfg.merge_from_list(["MODEL.DEVICE", "cpu"])
cfg.freeze()

# prepare object that handles inference plus adds predictions on top of image
coco_demo = COCODemo(
    cfg,
    confidence_threshold=0.7,
    show_mask_heatmaps=False,
    masks_per_dim=2,
    min_image_size=480,
)

#%%
def transform_image(image):
    image = image.float() / 255.0
    image = image.permute(2, 0, 1)
    # we are loading images with OpenCV, so we don't need to convert them
    # to BGR, they are already! So all we need to do is to normalize
    # by 255 if we want to convert to BGR255 format, or flip the channels
    # if we want it to be in RGB in [0-1] range.
    if cfg.INPUT.TO_BGR255:
        image = image * 255
    else:
        image = image[[2, 1, 0]]

    # we absolutely want fixed size (int) here (or we run into a tracing error (or bug?)
    # or we might later decide to make things work with variable size...

    # Normalize transform
    image = image - torch.tensor(cfg.INPUT.PIXEL_MEAN)[:, None, None]

    # should also do variance...
    image_list = ImageList(image.unsqueeze(0), [(int(image.size(-2)), int(image.size(-1)))])

    return image_list

def single_image_to_top_predictions(image):
    image_list = transform_image(image)

    result, = coco_demo.model(image_list)
    scores = result.get_field("scores")
    keep = (scores >= coco_demo.confidence_threshold)
    result = (result.bbox[keep],
              result.get_field("labels")[keep],
              result.get_field("mask")[keep],
              scores[keep])
    return result

@torch.jit.script
def add_annotations(image, labels, scores, bboxes, class_names=','.join(coco_demo.CATEGORIES), color=torch.tensor([255, 255, 255], dtype=torch.long)):
    # type: (Tensor, Tensor, Tensor, Tensor, str, Tensor) -> Tensor
    result_image = torch.ops.maskrcnn_benchmark.add_annotations(image, labels, scores, bboxes, class_names, color)
    return result_image

def process_image_with_traced_model(image):
    original_image = image

    if coco_demo.cfg.DATALOADER.SIZE_DIVISIBILITY:
        assert (image.size(0) % coco_demo.cfg.DATALOADER.SIZE_DIVISIBILITY == 0
                and image.size(1) % coco_demo.cfg.DATALOADER.SIZE_DIVISIBILITY == 0)

    boxes, labels, masks, scores = traced_model(image)

    # todo: make this in one large thing
    result_image = combine_masks(original_image, labels, masks, scores, boxes, 0.5, 1, rectangle=True)
    return result_image

#%%
pil_image = Image.open("./test.jpg").convert("RGB")

# convert to BGR format
image = torch.from_numpy(numpy.array(pil_image)[:, :, [2, 1, 0]])
original_image = image

if coco_demo.cfg.DATALOADER.SIZE_DIVISIBILITY:
    assert (image.size(0) % coco_demo.cfg.DATALOADER.SIZE_DIVISIBILITY == 0
            and image.size(1) % coco_demo.cfg.DATALOADER.SIZE_DIVISIBILITY == 0)

for p in coco_demo.model.parameters():
    p.requires_grad_(False)
traced_model = torch.jit.trace(single_image_to_top_predictions, (image,))

#%%
@torch.jit.script
def my_paste_mask(mask, bbox, height, width, threshold=0.5, padding=1, contour=True, rectangle=False):
    # type: (Tensor, Tensor, int, int, float, int, bool, bool) -> Tensor
    padded_mask = torch.constant_pad_nd(mask, (padding, padding, padding, padding))
    scale = 1.0 + 2.0 * float(padding) / float(mask.size(-1))
    center_x = (bbox[2] + bbox[0]) * 0.5
    center_y = (bbox[3] + bbox[1]) * 0.5
    w_2 = (bbox[2] - bbox[0]) * 0.5 * scale
    h_2 = (bbox[3] - bbox[1]) * 0.5 * scale  # should have two scales?
    bbox_scaled = torch.stack([center_x - w_2, center_y - h_2,
                               center_x + w_2, center_y + h_2], 0)

    TO_REMOVE = 1
    w = (bbox_scaled[2] - bbox_scaled[0] + TO_REMOVE).clamp(min=1).long()
    h = (bbox_scaled[3] - bbox_scaled[1] + TO_REMOVE).clamp(min=1).long()

    scaled_mask = torch.ops.maskrcnn_benchmark.upsample_bilinear(padded_mask.float(), h, w)

    x0 = bbox_scaled[0].long()
    y0 = bbox_scaled[1].long()
    x = x0.clamp(min=0)
    y = y0.clamp(min=0)
    leftcrop = x - x0
    topcrop = y - y0
    w = torch.min(w - leftcrop, width - x)
    h = torch.min(h - topcrop, height - y)

    # mask = torch.zeros((height, width), dtype=torch.uint8)
    # mask[y:y + h, x:x + w] = (scaled_mask[topcrop:topcrop + h,  leftcrop:leftcrop + w] > threshold)
    mask = torch.constant_pad_nd((scaled_mask[topcrop:topcrop + h, leftcrop:leftcrop + w] > threshold),
                                 (int(x), int(width - x - w), int(y), int(height - y - h)))   # int for the script compiler

    if contour:
        mask = mask.float()
        # poor person's contour finding by comparing to smoothed
        mask = (mask - torch.nn.functional.conv2d(mask.unsqueeze(0).unsqueeze(0),
                                                  torch.full((1, 1, 3, 3), 1.0 / 9.0), padding=1)[0, 0]).abs() > 0.001
    if rectangle:
        x = torch.arange(width, dtype=torch.long).unsqueeze(0)
        y = torch.arange(height, dtype=torch.long).unsqueeze(1)
        r = bbox.long()
        # work around script not liking bitwise ops
        rectangle_mask = ((((x == r[0]) + (x == r[2])) * (y >= r[1]) * (y <= r[3]))
                          + (((y == r[1]) + (y == r[3])) * (x >= r[0]) * (x <= r[2])))
        mask = (mask + rectangle_mask).clamp(max=1)
    return mask
<!-- If you have a code sample, error messages, stack traces, please provide it here as well -->

Expected behavior
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
~/Workspace/scripting/t-vi/maskrcnn-benchmark/demo/trace_model.py in
     11 for p in coco_demo.model.parameters():
     12     p.requires_grad_(False)
---> 13 traced_model = torch.jit.trace(single_image_to_top_predictions, (image,))

~/.pyenv/versions/maskrcnn-tracing-t-vi/lib/python3.7/site-packages/torch/jit/__init__.py in trace(func, example_inputs, optimize, check_trace, check_inputs, check_tolerance, _force_outplace, _module_class, _compilation_unit)
    881     traced = torch._C._create_function_from_trace(name, func, example_inputs,
    882                                                   var_lookup_fn,
--> 883                                                   _force_outplace)
    884
    885     # Check the trace against new traces created from user-specified inputs

~/Workspace/scripting/t-vi/maskrcnn-benchmark/demo/trace_model.py in single_image_to_top_predictions(image)
     25     image_list = transform_image(image)
     26
---> 27     result, = coco_demo.model(image_list)
     28     scores = result.get_field("scores")
     29     keep = (scores >= coco_demo.confidence_threshold)

~/.pyenv/versions/maskrcnn-tracing-t-vi/lib/python3.7/site-packages/torch/nn/modules/module.py in __call__(self, *input, **kwargs)
    537                 input = result
    538         if torch._C._get_tracing_state():
--> 539             result = self._slow_forward(*input, **kwargs)
    540         else:
    541             result = self.forward(*input, **kwargs)

~/.pyenv/versions/maskrcnn-tracing-t-vi/lib/python3.7/site-packages/torch/nn/modules/module.py in _slow_forward(self, *input, **kwargs)
    523         tracing_state._traced_module_stack.append(self)
    524         try:
--> 525             result = self.forward(*input, **kwargs)
    526         finally:
    527             tracing_state.pop_scope()

~/Workspace/scripting/t-vi/maskrcnn-benchmark/maskrcnn_benchmark/modeling/detector/generalized_rcnn.py in forward(self, images, targets)
     49         proposals, proposal_losses = self.rpn(images, features, targets)
     50         if self.roi_heads:
---> 51             x, result, detector_losses = self.roi_heads(features, proposals, targets)
     52         else:
     53             # RPN-only models don't have roi_heads

~/.pyenv/versions/maskrcnn-tracing-t-vi/lib/python3.7/site-packages/torch/nn/modules/module.py in __call__(self, *input, **kwargs)
    537                 input = result
    538         if torch._C._get_tracing_state():
--> 539             result = self._slow_forward(*input, **kwargs)
    540         else:
    541             result = self.forward(*input, **kwargs)

~/.pyenv/versions/maskrcnn-tracing-t-vi/lib/python3.7/site-packages/torch/nn/modules/module.py in _slow_forward(self, *input, **kwargs)
    523         tracing_state._traced_module_stack.append(self)
    524         try:
--> 525             result = self.forward(*input, **kwargs)
    526         finally:
    527             tracing_state.pop_scope()

~/Workspace/scripting/t-vi/maskrcnn-benchmark/maskrcnn_benchmark/modeling/roi_heads/roi_heads.py in forward(self, features, proposals, targets)
     24         losses = {}
     25         # TODO rename x to roi_box_features, if it doesn't increase memory consumption
---> 26         x, detections, loss_box = self.box(features, proposals, targets)
     27         losses.update(loss_box)
     28         if self.cfg.MODEL.MASK_ON:

~/.pyenv/versions/maskrcnn-tracing-t-vi/lib/python3.7/site-packages/torch/nn/modules/module.py in __call__(self, *input, **kwargs)
    537                 input = result
    538         if torch._C._get_tracing_state():
--> 539             result = self._slow_forward(*input, **kwargs)
    540         else:
    541             result = self.forward(*input, **kwargs)

~/.pyenv/versions/maskrcnn-tracing-t-vi/lib/python3.7/site-packages/torch/nn/modules/module.py in _slow_forward(self, *input, **kwargs)
    523         tracing_state._traced_module_stack.append(self)
    524         try:
--> 525             result = self.forward(*input, **kwargs)
    526         finally:
    527             tracing_state.pop_scope()

~/Workspace/scripting/t-vi/maskrcnn-benchmark/maskrcnn_benchmark/modeling/roi_heads/box_head/box_head.py in forward(self, features, proposals, targets)
     49
     50         if not self.training:
---> 51             result = self.post_processor((class_logits, box_regression), proposals)
     52             return x, result, {}
     53

~/.pyenv/versions/maskrcnn-tracing-t-vi/lib/python3.7/site-packages/torch/nn/modules/module.py in __call__(self, *input, **kwargs)
    537                 input = result
    538         if torch._C._get_tracing_state():
--> 539             result = self._slow_forward(*input, **kwargs)
    540         else:
    541             result = self.forward(*input, **kwargs)

~/.pyenv/versions/maskrcnn-tracing-t-vi/lib/python3.7/site-packages/torch/nn/modules/module.py in _slow_forward(self, *input, **kwargs)
    523         tracing_state._traced_module_stack.append(self)
    524         try:
--> 525             result = self.forward(*input, **kwargs)
    526         finally:
    527             tracing_state.pop_scope()

~/Workspace/scripting/t-vi/maskrcnn-benchmark/maskrcnn_benchmark/modeling/roi_heads/box_head/inference.py in forward(self, x, boxes)
     98             boxlist = boxlist.clip_to_image(remove_empty=False)
     99             if not self.bbox_aug_enabled:  # If bbox aug is enabled, we will do it later
--> 100                 boxlist = self.filter_results(boxlist, num_classes)
    101             results.append(boxlist)
    102         return results

~/Workspace/scripting/t-vi/maskrcnn-benchmark/maskrcnn_benchmark/modeling/roi_heads/box_head/inference.py in filter_results(self, boxlist, num_classes)
    152         result = cat_boxlist(result)
    153         scores = result.get_field("scores")
--> 154         keep = self.detections_to_keep(scores)
    155         result = result[keep]
    156         return result

RuntimeError: Tried to trace <__torch__.maskrcnn_benchmark.modeling.roi_heads.box_head.inference.PostProcessor object at 0x562da85bc0b0> but it is not part of the active trace. Modules that are called during a trace must be registered as submodules of the thing being traced.

Environment
Please copy and paste the output from the
environment collection script from PyTorch
(or fill out the checklist below manually).
You can get the script and run it with:
wget https://raw.githubusercontent.com/pytorch/pytorch/master/torch/utils/collect_env.py
# For security purposes, please check the contents of collect_env.py before running it.
python collect_env.py

PyTorch Version (e.g., 1.0): 1.3.1
OS (e.g., Linux): Ubuntu 18.04
How you installed PyTorch (conda, pip, source): pip
Python version: 3.7.6
CUDA/cuDNN version: 10.1/7.5
GPU models and configuration: RTX 2080Ti

Reference https://github.com/pytorch/pytorch/issues/25682

Tracing is done successfully. Buy detection result got worse.
detection result using original model

detection result using traced model


Custom content types

Hello John!
Some services put additional information in Content-Type header.
Instead of application/json we can see application/some.description.here+json and so on.
It's still json content type but it hard to check with inclusion.
At the first time I've seen it working with Artifactory's REST API.
Here I add check for that situation with regexp and fix integration test waiting for patch:)
Best regards, Nick.

Thanks!  Sorry for the long delay in getting to this, included in v1.1.3.


Undo global context

Due to #403, I think the solution to #374, is to:

Undo: #376
Use fix in PROJ: https://github.com/OSGeo/PROJ/pull/1566

The problem with this proposed solution is that it depends on a future release of PROJ, I think? (meaning, problem to solve it now. I don't know when a next PROJ release would occur, and that also would bump the minimal required version?)

The version bump will be necessary for adding support for:
https://github.com/OSGeo/PROJ/pull/1547
If this fix is merged in soon, the next PROJ release is scheduled for September 1.

I haven't been able to to come up with a clean solution without the change in PROJ. But, if you think of one, I would be happy to give it a go.

The version bump will be necessary for adding support for:
OSGeo/PROJ#1547

But that could be optional? Or is there a reason you need this in pyproj?

I haven't been able to to come up with a clean solution without the change in PROJ.

Short term, it seems like choosing between segfaults on windows or crashes with many CRS objects? (although I didn't follow both closely) In the last release you fixed the second which caused the first, correct? It could also be chosen the other way around (but again, don't know which of the two affects most end-users)

But that could be optional? Or is there a reason you need this in pyproj?

Provide pathway to address #401 and #389

It could also be chosen the other way around

Yeah, I went ahead with it as is for 2.3.0 so there is at least one version with a fix in it for the second problem.  Next step in the plan is to  undo global context in 2.3.1 (to have fix for first problem) and then in 2.4.0 using the PROJ fix to resolve the second problem.
Side note: Looks like 2.3.0 works fine in windows with conda-forge. Just wheels have issues.

One thought I had was to add an optional kwarg context when creating the Proj, CRS, Transformer objects. Then, a user could pass in their own ContextManager. This could allow for a semi-global context when many of the objects are being created. But, if it is not passed in, a new ContextManager is created in its stead.
This could be accomplished by making the ContextManager public:
https://github.com/pyproj4/pyproj/blob/master/pyproj/_datadir.pyx
This would provide a solution for both use cases and would not require pinning to the new version of PROJ yet. That being said, I am still thinking I might pin to the new version soon due to the new features included.


add support for canonical domain exceptions

Allow to set CANONICAL_DOMAIN_EXCEPTIONS list of complete domains on which is no redirection to the canonical domain.
This replaces #2 with simpler design of domain exceptions.
Note: I am still a bit fond of the regexp matches. It would make the use-case of this option much wider - e.g. excluding whole domains or certain subdomain on all domains. I cloud add secondary settings CANONICAL_DOMAIN_EXCEPTIONS_REGEX for that if there is interest.

Well I thought that the SecurityMiddleware would perform the redirect in such case. But I will look into it a bit more.

Yes it does, but it redirects to SECURE_SSL_HOST when it is set and when it does a redirect at all. Which means, that the exception list is useless.

@matthiask OK, thanks, that would be what I was missing. Now I added the redirect.
btw: There already is SECURE_REDIRECT_EXEMPT setting in Django which uses regexps. Shouldn't we maybe copy that behavior?

btw: There already is SECURE_REDIRECT_EXEMPT setting in Django which uses regexps. Shouldn't we maybe copy that behavior?

I didn't even know this setting :-) Yes, that would be a welcome addition if it doesn't make the code too complex. Let's finish the exceptions list first!

Note: I am still a bit fond of the regexp matches. It would make the use-case of this option much wider - e.g. excluding whole domains or certain subdomain on all domains. I cloud add secondary settings CANONICAL_DOMAIN_EXCEPTIONS_REGEX for that if there is interest.

Yes, I still somewhat agree, and I sort of do not understand the reluctance to use regexes. I now wonder if it would be better to use (more advanced compared to lists) regexes for the advanced use case of adding exceptions, especially when we add support for SECURE_REDIRECT_EXEMPT which brings us to regex land anyway.

@matthiask Django uses only the settings with regexps. Wouldn't be best to make only the EXEMPT setting to match Django's behavior closely as possible?

@PetrDlouhy You mean using SECURE_REDIRECT_EXEMPT to match both hosts and paths? Wouldn't this change the meaning of Django's own settings? I'd like to avoid that.
If we did something like that we should probably add our own CANONICAL_DOMAIN_EXEMPT setting and make that one handle hosts and paths?

Something like the following to combine both examples maybe? That could work, but maybe it's a bit unexpected that one regex should cover almost full URLs?
CANONICAL_DOMAIN_EXEMPT = r"^api\.example\.com|.*/no-ssl/"

Yeah, I meant adding new settings like CANONICAL_DOMAIN_EXEMPT.
But I thought, that Django's SECURE_REDIRECT_EXEMPT would match whole path including domain. I tested it and it is not the case. So maybe we could leave it with the exceptions for now.

I try to recapitulate:

Replace the CANONICAL_DOMAIN_EXCEPTIONS list with a CANONICAL_DOMAIN_EXEMPT regex
Add support for SECURE_REDIRECT_EXEMPT

I think that could and would be nice.

OK, I will do that.
To be clear:

The CANONICAL_DOMAIN_EXEMPT will be list (to match Django's behavior) of regexps matching just host (e.g. api.example.com).
If SECURE_REDIRECT_EXEMPT matches the path, we will do nothing (return get_response(request)).

Yes, makes sense. I agree that SECURE_REDIRECT_EXEMPT should take precedence.


cgroups: add support for blkio.weight_device

Signed-off-by: Qiang Huang<EMAIL_ADDRESS>

Hi,
Please note that this PR is not ready for merge, systemd part is not working right now, hope I can get some help here. (Sorry if this is not the recommended way, but google group is really not active :( )
I tested this on RHEL 7, which has systemd 208, from what I know, BlockIODeviceWeight is already supported from systemd 207, I have confirmed it form systemd's code log, and also we can see it from: http://man7.org/linux/man-pages/man5/systemd.cgroup.5.html
Fs part is working well (I added some necessary code in docker side for test), but if I use systemd, it keeps telling:
./docker run -ti --rm -m 300m --blkio-weight=400 --blkio-weight-device="/dev/sda 500" ubuntu:14.04 /bin/bash
FATA[0002] Error response from daemon: Cannot start container 5c4366c010b525db7f485fef8721cb0125cff890b8ab5da59b2c6c1104eae78d: Invalid argument
I think the command should be right, --device="/dev/sda" is not necessary, and I also tried --blkio-weight-device="8:0 500", but according to my understanding of systemd code, the device should be name as "/dev/sda", not "8:0". And of course I have confirmed sda on my host is using cfq,
cat /sys/block/sda/queue/scheduler
noop deadline [cfq]
So for now, I've no idea why this is not working, confusing me for several days, I'm not familiar with systemd or dbus, can anyone see something wrong here, or is this possible to be a systemd bug?
Any help is appreciated, thank in advance!

Thanks @hqhq. I can look into it tomorrow. I was in fact looking into systemd code for blkio controller today and saw that it expects device path instead of major minor numbers. It converts the device path into major minor number. http://cgit.freedesktop.org/systemd/systemd/tree/src/core/cgroup.c#n352

I did some debugging. When I try to write to that cgroup directly it fails. However, it works for blkio.throttle.write_bps_device. Using systemctl segfaults for that property but works for BlockIOWriteBandwidhth
[root@localhost docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope]#  systemctl set-property --runtime docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope "BlockIOWriteBandwidth=/dev/mapper/docker-252:3-386434-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f 10M"
[root@localhost docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope]# cat blkio.throttle.write_bps_device
253:2 10000000
[root@localhost docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope]# echo "253:2 20000000" > blkio.throttle.write_bps_device
[root@localhost docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope]#
[root@localhost docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope]# cat blkio.throttle.write_bps_device
253:2 20000000
[root@localhost docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope]# echo "/dev/mapper/docker-252:3-386434-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f 500" > blkio.weight_device
-bash: echo: write error: Invalid argument
[root@localhost docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope]# echo /dev/mapper/docker-252:3-386434-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f 500 > blkio.weight_device
-bash: echo: write error: Invalid argument
[root@localhost docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope]# systemctl set-property --runtime docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope "BlockIODeviceWeight=/dev/mapper/docker-252:3-386434-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f 500"
Segmentation fault (core dumped)
[root@localhost docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope]#

Getting late here. I will continue debugging tomorrow.

@mrunalp If you echo to blkio.weight_device, you should use major minor numbers, it should work.
[root@localhost docker-0e8fb784b93e5e6b2c4f9be0037c1075009c1b4a1bea27481d27b022c6b5e712.scope]# echo "8:0 500" > blkio.weight_device
[root@localhost docker-0e8fb784b93e5e6b2c4f9be0037c1075009c1b4a1bea27481d27b022c6b5e712.scope]# cat blkio.weight_device
8:0 500

And I tried use systemctl directly as well, it works for me:
[root@localhost docker-0e8fb784b93e5e6b2c4f9be0037c1075009c1b4a1bea27481d27b022c6b5e712.scope]# systemctl set-property --runtime docker-0e8fb784b93e5e6b2c4f9be0037c1075009c1b4a1bea27481d27b022c6b5e712.scope "BlockIODeviceWeight=/dev/sda 600"
[root@localhost docker-0e8fb784b93e5e6b2c4f9be0037c1075009c1b4a1bea27481d27b022c6b5e712.scope]#
[root@localhost docker-0e8fb784b93e5e6b2c4f9be0037c1075009c1b4a1bea27481d27b022c6b5e712.scope]# systemctl show docker-0e8fb784b93e5e6b2c4f9be0037c1075009c1b4a1bea27481d27b022c6b5e712.scope  | grep Block
DropInPaths=/run/systemd/system/docker-0e8fb784b93e5e6b2c4f9be0037c1075009c1b4a1bea27481d27b022c6b5e712.scope.d/90-BlockIOAccounting.conf /run/systemd/system/docker-0e8fb784b93e5e6b2c4f9be0037c1075009c1b4a1bea27481d27b022c6b5e712.scope.d/90-BlockIOWeight.conf /run/systemd/system/docker-0e8fb784b93e5e6b2c4f9be0037c1075009c1b4a1bea27481d27b022c6b5e712.scope.d/90-CPUAccounting.conf /run/systemd/system/docker-0e8fb784b93e5e6b2c4f9be0037c1075009c1b4a1bea27481d27b022c6b5e712.scope.d/90-Description.conf /run/systemd/system/docker-0e8fb784b93e5e6b2c4f9be0037c1075009c1b4a1bea27481d27b022c6b5e712.scope.d/90-MemoryAccounting.conf /run/systemd/system/docker-0e8fb784b93e5e6b2c4f9be0037c1075009c1b4a1bea27481d27b022c6b5e712.scope.d/90-MemoryLimit.conf /run/systemd/system/docker-0e8fb784b93e5e6b2c4f9be0037c1075009c1b4a1bea27481d27b022c6b5e712.scope.d/90-Slice.conf
BlockIOAccounting=yes
BlockIOWeight=400
BlockIODeviceWeight=/dev/sda 600

Sorry missed the lines where I had used device numbers. I am on Fedora 21.
What distro are you on?
[root@localhost docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope]# echo "253:2 500" > blkio.weight_device
-bash: echo: write error: Invalid argument
[root@localhost docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope]# echo "253:2 20000000" > blkio.throttle.write_bps_device
[root@localhost docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope]# cat blkio.weight_device
[root@localhost docker-585cbac66f8f10d7566b5448959b40663fd90c943718957f3956b55d004c142f.scope]# cat blkio.throttle.write_bps_device
253:2 20000000

I am guessing in your case it is some intermediate library/binding then.

I'm on RHEL 7, is 253:2 on your box using cfq as IO scheduler? Only devices using cfq can be written to blkio.weight_device, and blkio.throttle.write_bps_device don't have this limit.
You can  cat /sys/block/xxx/queue/scheduler to see the scheduler.

Hi,
I added read/write bps device support, met the same error on systemd:
[root@localhost home]# ./docker_bps run -ti --rm -m 300M --blkio-read-bps-device="/dev/sda 50M" --blkio-write-bps-device="/dev/sda 20M" ubuntu:14.04 /bin/bash
FATA[0003] Error response from daemon: Cannot start container 1e554f3f7fff1459c674d966a88563799807806e85f12d6d799416b2b1eec486: Invalid argument

Fs part is working well.

@hqhq you are right. I was using a device which doesn't have cfq support. I think it should be fairly easy to try the write bps support and see if I can reproduce the issue you are seeing.

Just got back to this. I see no failures but nothing is written for write bps device.  I will debug it further tomorrow.

@hqhq Should we implement this using fs workaround in systemd?

@mrunalp Yes, I think that'll be the way to move on, I'll update.

Updated.

LGTM

@vmarmol @rjnagal ping

LGTM.
An added niceness would be to  add another check to verify the scheduler being used by the device. It will help in returning a meaningful error.


Draft for reading files in chunks

TODO:
From Zack:

The tricky part is keeping track of timestamps as we iterate through, there can be drift
If we just do every 30s we’re going to be missing some text because the model only transcribes up to a good breaking point, and assumes we will seek to that point and run the next 30s
Yea we just seek to them, which is where you could insert your method, instead of seeking, you load the next chunk and start at 0 time again

Putting on hold in favour of #125


Things to be Added - Indicator Template

Hey 👋
Indicator Templates refer to multiple set of Indicators which work in a sync.
We have deployed all such Technical Indicators in this repository.
Hence, There is no point developing Indicator templates.
If you need help in developing a trade system do hit me up.
Regards,
@ChintzRuparel
Founder, TWODS Capital


@cap-js/sqlite: search with multiple words leads to error

customer reports error SQLite only supports single value arguments for $search.
fine on postgres.
should be avoided by concatenating multiple search clauses with OR

Can you add an example?

https://github.com/cap-js/cds-dbs/pull/132

Thanks. We intentionally decided to provide only limited support for that, at least in SQLite, as the complexity and performance overhead would be significant, and the value rather low (most end users don't do advanced search expressions)

Hello.
Can someone please clarify if this issue/missing functionality plan to be released soon?
We might require to disable main Search functionality in our Fiori application which is running on @cap-js/posgres exactly due to this issue.

Hi @ArtyomAD,
I would think that it would be better if you check if this issue also occurs when you use sqlite as then you could contribute the SQL statement that is created when you issue the query. Or you create a new issue which mentions the postgres driver.

Hello @gregorwolf.
We face identical error in both cases with sqlite and postgres.
SQLite:

Trace without $search:

Trace with $search:

Postgres in BTP:

Confirmation from BTP as it's running with postgres:

I can confirm that the issue also occurs with SQLite. The search input from Fiori Elements is passed directly to the search query - if the user enters a search string with whitespaces, search produces an error.

This issue also breaks Fiori Elements apps, when users try to enter text with whitespaces in value help fields.


remove unused --recursive from collect-sources.sh

I am running a full test now. Please do not merge until done. I will comment here.

It works. Ready to merge.


solo-python: 0.0.31 -> 0.1.1

Description of changes
this also renames solo-python to solo1-cli
Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

[x] (Package updates) Added a release notes entry if the change is major or breaking
[x] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module
[x] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@Lassulus Do you mind taking a look whether this is merge-worthy?
@GrahamcOfBorg build solo1-cli

@GrahamcOfBorg build python3Packages.solo1-cli

would merge, but there is a merge conflict, so needs a rebase

follow-up version upgrade without the nice rename, but marking it as broken due to fido2 version 1.1: https://github.com/NixOS/nixpkgs/pull/196586


128/130b encoding support

Adding in 128/130b encoding support. The addition will allow for all packets to be used by the application layer (meaning none reserved for internal use). 128/130 encoder will be embedded in S-Link so no need to have one externally in the PHY.
Expect a week or two for these changes. Will detail more after implementation.

128b/130b encoding is now supported. A future update will allow the encoding and certain components to be bypassed.


1)Added script to extract private key and cert from PKCS#12 File

2)Added test keypair for ECC256/384/521
By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---
CONTRIBUTING.md also tells you what to expect in the PR process.
Description
Provide the information we need to review your PR. What problem does the pull request solve? "Bug fix" is not a good description.
Related Issue
If you opened an issue before creating the PR, point to it here.
Context
What do we need to know about your development environment, tools, target, and so on. Screenshots are always helpful if there is a UI element to this PR.

Feature combined with the following PR.
PR #35


Data download/MediaReview: remove extraneous data

The columns/fields highlighted in yellow in the schema table are extraneous and we'd like to remove them from the JSON/CSV download.

I was double-checking this info and made a list of the fields for my own benefit, so I'm including it here for handy reference:
author.image
author.sameAs
itemReviewed.creator.@type
itemReviewed.creator.name
itemReviewed.creator.url
itemReviewed.interpretedAsClaim.@type
itemReviewed.interpretedAsClaim.description
itemReviewed.mediaItemAppearance.1.accessedOnUrl
itemReviewed.mediaItemAppearance

itemReviewed.mediaItemAppearance
So since this is populated in some of the items, but not others I'm going to make this an empty array if there's nothing.

This is read to review

Looks good, I think my only question is tied to my comment on on #571 and whether or not we want to adjust our FactStream output to match the Google DataFeed output. Will need @ericaryan input on that.

done


Early access user enters email address and password to login. They are granted a user role.

An early access user should be able to enter their email address and password to login to Navigator. Once logged in, they will be granted a “User” role.

Hey team! Please add your planning poker estimate with ZenHub @eurolife @ingemar-cpr @kdutia @opyate


shelljs throw issue

Hi, i'm trying to use protocol-registry on vscode extension .
shelljs.exec on your lib returns null..
I debug it and config.execPath in shelljs set 'null' .
as i know vscode is electron base .. and shelljs has issue about it.
https://github.com/shelljs/shelljs/wiki/Electron-compatibility
Any update for this ?

can yo share any code snippet

it gets thrown in require .
const ProtocolRegistry = require("protocol-registry");
protocol-registry looks using shelljs only in linux, macos.
so, i do register protocol manually(not using your lib) in linux, macos cases .
Thanks.

Ok, that's great to hear that you problem has been solved.
But I think if shelljs is creating problems then lets replace it with child process. Thus will be opening a new issue for removing shelljs.
Contribution is highly encouraged.

Fixed with : cb27941fe3321edf9a802fab7c2f8c51700ba953


Overflow when step is a double type

I encountered the problem that when I set step = 0.1, the value will overflow sometimes.
For example, 0.1 + 0.1 = 0.20000000001. Is there any way to round off the value shown in the UI?

Hi Alan @yykk7711 , Thank you for submitting the issue.
Floating-point arithmetic can sometimes lead to precision errors due to the way computers represent real numbers. This issue is not specific to Dart; it's a common challenge in many programming languages.
This problem is on my radar, I will look for the best solution.
I found a third-party plugin called decimal  on pub.dev,
but I want this package to be less dependent on additional plugins. so I will try to add a method to handle this

@yykk7711  please try to use v2.2.0
This 0.1 + 0.1 = 0.20000000001 should not happen again.
in current version:   0.1 + 0.1 = 0.2
and I also added new props decimalPlaces. So you can specify the maximum decimal number that can be used in the widget.
InputQty(
   decimalPlaces: 3, // this will limit decimal, maximum 3
   initVal: 0.0,
   minVal: -100.0,
   maxVal: 100.0,
   steps: 0.1,
),

Thanks @pmatatias for the very prompt upgrade, I have tested v2.2.0 and it is well solved!


Add a zoomable image source for coil3

Addresses https://github.com/saket/telephoto/issues/103

[ ] Resolve TODOs
[ ] Generate screenshots and compare them with coil2

Closing this because I accidentally merged the branch locally.


Relicense under dual MIT/Apache-2.0

Why?
The MIT license requires reproducing countless copies of the same copyright
header with different names in the copyright field, for every MIT library in
use. The Apache license does not have this drawback, and has protections from
patent trolls and an explicit contribution licensing clause. However, the
Apache license is incompatible with GPLv2. This is why Rust is dual-licensed as
MIT/Apache (the "primary" license being Apache, MIT only for GPLv2 compat), and
doing so would be wise for this project. This also makes this crate suitable
for inclusion in the Rust standard distribution and other project using dual
MIT/Apache.
How?
To do this, get explicit approval from each contributor of copyrightable work
(as not all contributions qualify for copyright) and then add the following to
your README:
## License

Licensed under either of
 * Apache License, Version 2.0 ([LICENSE-APACHE](LICENSE-APACHE) or http://www.apache.org/licenses/LICENSE-2.0)
 * MIT license ([LICENSE-MIT](LICENSE-MIT) or http://opensource.org/licenses/MIT)
at your option.

### Contribution

Unless you explicitly state otherwise, any contribution intentionally submitted
for inclusion in the work by you shall be dual licensed as above, without any
additional terms or conditions.

and in your license headers, use the following boilerplate (based on that used in Rust):
// Copyright (c) 2015 t developers
// Licensed under the Apache License, Version 2.0
// <LICENSE-APACHE or
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT
// license <LICENSE-MIT or http://opensource.org/licenses/MIT>,
// at your option. All files in the project carrying such
// notice may not be copied, modified, or distributed except
// according to those terms.

And don't forget to update the license metadata in your Cargo.toml!
Contributor checkoff

[x] @neon64

Thanks for the detailed explanation. Is this Cargo trawling through all crates for issues or something?
I didn't even consider the implications of the MIT license, I just picked it when Cargo prompted for one.
With the license metadata in Cargo.toml is this the right way?
license = "Apache-2.0/MIT"

No problem! The metadata for Cargo.toml is typically "MIT/Apache-2.0" (I don't know why) but the order doesn't matter and what you have written is fine. Yeah, I wrote some scripts to help handle this - a lot of people don't know the ramifications of software licensing, and having the as much of the rust ecosystem as possible be dual MIT/Apache-2.0 has potential benefits.


Added Fyrenest v3

Fyrenest v3 - Lore Update
This update re-works most of the Hollow Knight lore! This is the first snapshot of what is to come in version 3! So far, Fyrenest v3 remaps the Ancient Basin, The Hive, The Abyss, The Resting Grounds, and Crystal Peak! It also rewrites dialogue for most of the NPCs like The Seer, The Last Stag, Elderbug, Cornifer (now called Conifer), Iselda (now called Pine), and more! Version 3 of Fyrenest also adds new charms, modifies some old ones, and changes the ways to get some!
Technically, this is a pre-release of the finished version 3 of Fyrenest, But since this is already such a big update, I think it should have its own release.
Major Changes

Added lots of room transitions.
Added various game objects to different rooms.
Deleted various game objects in different rooms.
Replaced the location of lots of items.
Rewrote & revised all already replaced dialogue.
Replaced tons more dialogue.
Split the Hive area.
Mirrored half of Ancient Basin.
Mirrored half of The Hive.

Minor Changes
These are minor changes from the last update that aren't already listed.

Made some modifications to the language replacement system.
Made all charms inherit the itemchanger charm class.
Re-worked some code. (So minor it isn't all gonna be said)
Added more comments and summaries so code is easier to understand.
Starting on adding charms around the map. (Like with VoidSoul)
Made all charms publicly accessible. (In terms of the code)
Updated where the charm images are stored.
Made tiny tweaks to some of the charm images.
Started doing something for the README.md (Not that it effects the gameplay)
Added SlowJump.
Added QuickJump.
Modified SlowFall.
Modified QuickFall.
Added image for SlowJump.
Added image for QuickJump.
Added new ways to get some of the charms.

Bug Fixes:

Fixed Elderstone charm, now gravity changing works.
Fixed some menu bugs.
Fixed some bugs with various charms.
Fixed a bug with the abilities. (With object preloads and stuff)

Known Bugs:

SlowJump is broken. (Does nothing)
SlowFall is broken. (Does nothing)
QuickJump is broken. (Does nothing)
QuickFall is broken. (Does nothing)
Errors with some room transitions when continuing from past saves.
A few bugs with the Aspid room in the Forgotten Crossroads.
Some bugs with the hot springs in the Forgotten Crossroads.
Sometimes the knight cannot move. (To fix this, use noclip from the DebugMod to enter a room transition. You can then move as normal.)
SPOILER COMING UP: VoidSoul can still be obtained without getting to the area where Joni's Blessing is found.

tt?


Xenon not running in hardware accelerated mode

For some reason Xenon is no longer running in hardware accelerated mode, which really slows down some operations, like workview resizing.

As a workaround, try adding -Dprism.forceGPU=true to the command line.

This appears to only be a problem on Sapphire where I'm running an AMD Navi 14 card. It does not appear to be a Java or Java FX issue as the same binary works correctly on other machines. I suspect this is a recent driver update issue.


Deprecate this role in favor of the new geerlingguy.mac collection

I have consolidated my macOS-related roles into the new geerlingguy.mac collection.
This collection contains the exact same copy of this role at version 3.2.0 in its 1.1.1 release.
All future work will go into the collection role.
Migration steps:

Make sure you're running at least Ansible 2.9 (Ansible Core 2.10 or Ansible 3.0.0 or later recommended).
Install the collection: ansible-galaxy collection install geerlingguy.mac (you can also specify collections: inside your requirements.yml file separately from roles:).
Change all references to the geerlingguy.mas role to geerlingguy.mac.mas.

If you have any migration issues, please consider opening an issue on the geerlingguy.mac collection repository, a this repository will be archived, and the role on Galaxy will be marked as 'deprecated'.
TODO:

[x] Add a deprecation note to the README of this role and release a new tag so it appears on Galaxy.
[x] Add a deprecation note to the role's repository description.
[ ] Archive the repository.

Archiving next!


New option: use double reset button to enter configuration mode

Hi, I have already begun to use this great library.
I want to know if it is possible to implement another option.
It is to use RESET button instead falsh button.
That is why Wemos Mini D1 has to use D3 pin for that buttton, but it has a reset button.
Please, see this library: https://github.com/datacute/DoubleResetDetector
Regards,

You could relatively easy implement this yourself using the library you mentioned and the built-in functions in Homie. I haven't done this myself but I can't see any obvious showstoppers.
I believe the reset button on the Wemos is a "hardware" reset. The reset option in Homie is a software reset with added functionality (clearing the config file).

@ortegafernando unfortunately, as @bertmelis stated, the reset button actually reset the hardware, so we cannot hook to it.


Creating a nuget package

I've never created a nuget package before so I'm not really sure how to go about it. If you have any experience I'd love some help. (Yes, I have been pretty inactive recently but I'm getting back into the project now)

This details how to do it:
https://docs.nuget.org/create/creating-and-publishing-a-package.
Although I still don't reckon that the engine is in a releasable state just yet.

I agree, but its always nice to plan ahead.


(AzureCXP) Adding log size info

The sign-in events should be about 4kb of data storage per sign-in event. per git issue : https://github.com/MicrosoftDocs/azure-docs/issues/33264
And https://stackoverflow.microsoft.com/questions/152833/azurecxp-size-of-sign-in-audit-event-logs

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


keep "<" and ">" special char in columns names when properly quoted

First of all, wanna compliment you for this helpful tool!
I came into a (possibly?) minor bug when trying to parse an hql file containing column names with special characters "<" or ">" properly quoted like:
something<2% float
I found out this is due to "parser.pre_process_line" that insert spaces around "<" and ">" if "(" not in line.
Possible solution is not to insert spaces if "<" ">" are inbetween quotes.

@gtanzi , hi! Thanks for warm words and reporting the issue. I will be back from vacation on 8 of November and will take a look in the big :) thank you!

@gtanzi about spaces - I added them because of some issues with parsing HQL structures like this - ARRAYstructcolx:string,coly:string. I will work on more normal workaround and will try to remove it.

@gtanzi sorry that it took so much time - worked on another issues, I fixed this problem in 0.22.5. https://github.com/xnuinside/simple-ddl-parser/blob/main/tests/test_hql_output_mode.py#L1888 - test added. Feel free to open the new issue if needed something. Thank you!


Create a Pool Connections for LMDB database

┆Issue is synchronized with this ClickUp task by Unito

Closing this issue as it is moved to https://github.com/minos-framework/minos-python/issues/32


Fix sass filter to change directory as needed

Currently the cwd argument to _apply_sass is being ignored, this was
accidentally dropped in refactoring in commit fbe1b8b.
Restore the previous behaviour by adding a cwd argument for the
ExternalTool#subprocess method.
Fixes #466.

Thanks.


sql statement in getJourneyPatterns

Hi,
I tried the same thing with the AVV importer and realized that the second sql statement in getJourneyPatterns  (dino.py) takes ridiculously long, doesn't it ?
Is there something wrong in my setup or did you go with it?

Tell me if you dont manage to do it. I could help you out

Thanks for answering and thanks for the offer.
Thanks!
Well, after setting up the data in a more reasonable way it ran through.
I've already seen that the stops are little bit messed up but i don't have an overview yet.
Is your restulting gtfs data fine or did you do some postprocessing ?

Hi,
I did not do much postprocessing. What do you think is wrong.
I did a manual review and things looked fine. In my opinion.
Why do you think stops are messed up for you?

@cabanana ?

@cabanana ?

Sorry, I'm not working on it right now.
It was the splitting of the stop locations that got me confused.
However, using the bliksemlabs script seem to work.
Thanks for your comments!


Plural / Singular units

I really do not get how your Plural / Singular units System is working. I want for singular "Minute" and for plural "Minuten" and tried:
Minute{{MinutenS}}
Minuten{{MinuteS}}
Minute{{MinuteN}}
but none of them seems to work. What could i do to get this working? The syntax is very confusing.
Maybe it would be easier if you would use a syntax like so [minute|minutes]. This would ne inspired by http://laravel.com/docs/5.0/localization#pluralization
http://siddii.github.io/angular-timer/index.html#/clock-timer-i18n just don't work!

Hi,
Have you tried ?
<timer interval="1000" language="de" >{{yearUnit}}</timer>

It is written here : http://siddii.github.io/angular-timer/index.html#/clock-timer-i18n

Yes this works but not for my usecase.

Does this work? I haven't had a chance to test it:
<timer interval="1000" language="de">{{minutes}} minute{{minutesS}}</timer>

Is there any way to use i18n version, but have days, hours and minutes in seperate containers?
I need to style each element seperately, and whay I get if I use this expample is:
<timer interval="1000" start-time="1357020000000" language="de" >{{yearUnit}}</timer>

Result:
3 Jahre, 18 Tage, 9 Stunden, 14 Minuten, 44 Sekunden

Desired result would be similar to:

  3 Jahre
  18 Tage
  9 Stunden
  14 Minuten

Hi @keligijus :smile:
I guess we could split this value $scope.yearUnit https://github.com/siddii/angular-timer/blob/master/app/js/_timer.js#L236 in :

$scope.dayI18n
$scope.hourI18n
...

$scope.yearUnit  is built there : https://github.com/siddii/angular-timer/blob/master/app/js/i18nService.js#L53 and generated with https://github.com/EvanHahn/HumanizeDuration.js
Do you think you could provide a PR ?

Hi @polomarcus :)
I'll try to do my best. If I can figure it out and make it work, will definitely provide a PR


Document type of HTML elements in .html files

Suggestion

🔍 Search Terms

dom html types
✅ Viability Checklist

My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion

📃 Motivating Example

I want to make a custom JSX transformer that just returns strings.
I want to export the types the JSX module needs.
I want to map over the attribute types you see in .html files.
But I can't find these things.
They are mysterious and hidden from me.
Can you document them?
And export them if needed?
💻 Use Cases

See above.

More details:
I have created these types:
export type IntrinsicElements = {
  [Tag in keyof HTMLElementTagNameMap]: (
    { [Attr in keyof Omit<HTMLElementTagNameMap[Tag], 'children' | 'toString'>]?: string }
    & { children?: any, class?: string }
  );
};

export type Element = string;

However, this is not exactly the same type as what .html files offer. There are two obvious differences:

className is offered instead of class so I have to manually add this in here
style is a Style object instead of a string; which also means I don't get VS Code's super great auto-completion inside the style-string

It seems everyone is confused, because how would you actually use it?

class is a reserved word in JS, so everywhere you refer to it in JS, it's className. It's the same for JSX and the native Element.className.
How would you get auto-complete if you are in a string anyways?

@Josh-Cena (a) it's just a key in attrs; (b) VS Code already does this in HTML mode

What VS Code does in HTML files has nothing to do with TypeScript.  The types you seem to be looking for aren't hidden from you; they just don't exist.

You would really need to make some kind of codegen tool to make this work ergonomically.
For CSS string completions, I would recommend using a VS Code or TypeScript language service plugin. We're not in a position to be providing that kind of data.

Thanks @RyanCavanaugh. This is one of the few missing things that would make the new TypeScript web framework Novo Cantico better, since right now it has nothing close to VS Code's .html convenient support for HTML attributes. My plan based on your advice is to somehow pre-generate a .d.ts file to include into the framework itself.


added fix for shutdown race (#6384)

Pull request opened by the merge tool on behalf of #6384

Merged into branches iso6 in f1a439353019fbfec5c1c564753bf5a1984066ac


Command line option to specify default graphics quality

Reported by karl smith on 22 May 2012 19:16 UTC
Idea from Patrick Brockmann: add a command-line option to specify the default graphics setting when starting pyferret.
At first seemed like a good idea.  But thinking about it more, I am still trying to think of use cases where you would not just have a SET GRAPHICS command in $HOME/.ferret or in the Ferret script itself.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1943

Comment by steven.c.hankin on 23 May 2012 15:26 UTC
Agree.  This seems sufficient.
Could also address Patrick's concerns by adding pre-defined aliases to make it very quick for a user to switch between graphics qualities.  say, "GRHI", "GRLO"

Comment by karl.smith on 30 May 2012 19:01 UTC
Deleting this issue since could not come up with a good use-case for it.


No Clipper query frontend found

Am trying to deply a model - I get this error
"No Clipper query frontend found to attach model container to"
Please let me know how to resolve this.

@sudham :
Did you start Clipper before deploying your model?

I was able to resolve it - it seems as if there was not enough disk space. Thank you.
Am closing this issue.


Document how the Design System is deployed

You can preview this change here:
Built with commit 0eb6ea4bfb6c7e3a1c4cdf40c5a8175a7e3b8e90
https://deploy-preview-408--govuk-design-system-preview.netlify.com


Support for VideoJS 5 Responsive / Fluid Classes

Thank you guys for bringing 5.x support to this plugin!
Any chance we can bring in support for the new responsive options (http://blog.videojs.com/post/130143694631/video-js-5)?
e.g. vjs-16-9, vjs-4-3 classes or setting fluid = true?
Currently, the plugin requires that you hard-code the player size and if you add in those classes, the ads appear below the player.

I'll consider this a feature request and we'll definitely look into it. The IMA SDK that we use under the hood ultimately requires pixel dimensions to initialize and resize ads, so we'll need to see how we can get that info from the video player when it's using responsive CSS classes rather than hard coded pixel dimensions.

Do you have a specific device and browser where you saw this problem? I did a quick test in Chrome on Linux and I do see the ads where they're supposed to be with the responsive classes, so it may be platform-specific.

I face the same problem. Latest videojs' player.width() and player.height() are returning 0 if player is not hardcoded with a specific size.
I am thinking about circumventing the issue until videojs fixes this with following code:
player.ima.width = function() {
  return player.el().offsetWidth
}
player.ima.height = function() {
  return player.el().offsetHeight
}

adContainerDiv.style.width = player.ima.width() + 'px';
adContainerDiv.style.height = player.ima.height() + 'px';

Could wrap this in a PR if you like.

@shawnbuso I am using Chrome (latest version) on a Mac. VideoJS 5.3 and using your code from the sample. Do you need me to put up a Fiddle?

Even better, here it's live on our staging site on a sample video happening. It looks like the ad is now appearing behind the video instead of below...
http://staging.foodfanatic.com/videos/how-to-make-truffles/iframe/

@hc2p While we're waiting for this feature to be release, would you mind sharing a gist with me so I can see where that code goes for a workaround?
Should we try and make an issue / PR at videojs?

i created a PR for this issue https://github.com/googleads/videojs-ima/pull/161 sorry for the delay, got sidetracked.
@ehoch PR on videojs is not needed. read https://github.com/videojs/video.js/issues/2974 and https://github.com/videojs/video.js/issues/2282

Can you folks confirm that the issues you were seeing are resolved with #161? If so we can go ahead and close this issue. Thanks!

@shawnbuso @hc2p Unfortunately for me, when running things in fluid or with the vjs-16-9, the ad appears in the correct resolution, but below the player:

i have the same issue. Totally forgot that i worked around it with this css:
 #ima-ad-container {     position: absolute;       top: 0;     }

@hc2p That mostly solves the issue, however, if a user resizes the fluid player, the player will adjust but the ad does not leading to this:

yes, that's a problem with the ima-plugin which hardcodes the width/height
of the ad-player.
On Tue, Feb 2, 2016 at 4:50 PM, Eric Hochberger<EMAIL_ADDRESS>wrote:

@hc2p https://github.com/hc2p That mostly solves the issue, however, if
a user resizes the fluid player, the player will adjust but the ad does not
leading to this:
[image: screen shot 2016-02-02 at 10 48 53 am]
https://cloud.githubusercontent.com/assets/852746/12754801/b7099e1e-c99a-11e5-9ca3-a93a38397118.png
—
Reply to this email directly or view it on GitHub
https://github.com/googleads/videojs-ima/issues/143#issuecomment-178649561
.

@shawnbuso Is this a fixable problem ever with the way Google IMA handles ads?

Does the player fire any kind of event to let you know that it's resizing / has resized? If so we can respond to that event with a call to adsManager.resize() with the new player dimensions.

FYI it looks like the issue with the ad being bumped below the player is that playerStyles.height is being set to 0, so there's no space for the ad inside the ad container. We might have to somehow delay ad display container creation until the video player is initialized - it looks like right now we're creating it before the video player exists and the height is being reported as 0. The width is being accurately reported for some reason though...

Looking into this more, the height of 0 is breaking the controls as well. We'll only be able to make this work if we can get the actual DOM height and width of the video player when we initialize the ad container and ads manager. The code from pull request #161 is returning a height of 0 when we create the ad display container. Setting position:absolute and top:0 fixes the ad video rendering, but all the controls rendering is still broken.

I think I might have something here - take a look at https://github.com/shawnbuso/videojs-ima/tree/fluid_support and let me know what you think.

This mostly works for me (noted a few typos in the checkin comments), but for the life of me I can't get the player.one('play', setUpPlayerIntervals_) callback to fire. The fix works if I just do that setInterval outside of the player.one('play') callback though...

Think I narrowed it down to us calling the ima plugin via a play handler on our player.
Moved the setUpPlayerIntervals_ logic to player.on('loadedmetadata') callback and moved all the callbacks into the player.ready block and all's well.
:+1: for the fix

This works for me in terms of solving the fluid issue, but I'm running into a different problem with this branch (that might be related to what @jgubman is seeing).
We use this for a click-to-play player so I don't directly call player.play(), I let VideoJS handle that when a user clicks (except on mobile where we follow your iPhone example). The regular branch works, but the fluid branch does not do anything when you click the player, only when you call player.play().

@shawnbuso are you sure you want to read out width/height in an interval?https://github.com/googleads/videojs-ima/commit/5106b675aa3066f6949c945d53fa974d8c63d42d#diff-56b40006e5136dbf9868a615013d9e03R773 It looks like it would force a lot of additional layouts (see https://gist.github.com/paulirish/5d52fb081b3570c81e3a#getcomputedstyle).
What about exposing an resize() method on the plugin (videojs.ima.resize()). It would be up to the integration code to call it on the right time. One could listen on specific player-size affecting media-query changes (see https://developer.mozilla.org/en-US/docs/Web/API/MediaQueryList) and call it only when needed.

There is also this library https://github.com/marcj/css-element-queries. Which easily could be done by the user and doesn't need to be part of the plugin.

@ehoch How does your click to play player work outside of this branch? Looking at the workflow for the contrib-ads plugin that we use, I can't find a path that ets you to ad playback without calling player.play()
@hc2p I could expose just a resize method, but that would leave more for the integrator to do and potentially do incorrectly, which would lead to more support cases for us. (Not that you would do it incorrectly, but potentially the hundreds of other people that use the plugin). Currently I only actually resize the adsmanager if the numbers I get back are different than the current size, so we shouldn't have extraneous calls to adsManager.resize.
I do like the look of that library, but I'd want to thoroughly test it with video.js to make sure it responds the way we think.

@shawnbuso In the main branch, on desktop it works by simply calling:
player.ima(options);
player.ima.requestAds();

Then when the user was clicking play on the player it just seemed to be working on desktop. Was I supposed to be hijacking their click and calling player.play() myself?
I thought that iPhone thumbnail hack was just because VideoJS broke the tap event and wasn't needed for desktop..

Hmm interesting, I wonder if there is an internal call to play() that triggers ad playback. In all of our examples we are calling player.play() after requesting ads. See [https://github.com/googleads/videojs-ima/blob/master/examples/advanced/ads.js#L84](here in the advanced sample).

Thanks @shawnbuso. I'll update my code to do exactly that. Any idea when VideoJS will be fixing that regression with the tap event so we can go back to using the simple code rather than the thumbnail hack?

I've filed an issue for it here: https://github.com/videojs/video.js/issues/2958. It looks like it might be fixed with a more recent version of video.js but I haven't had a chance to test yet.

@shawnbuso Well for now, using the thumbnail hack for mobile devices and tracking the click on desktop and the fluid branch works beautifully. Thank you for this. Can we get it merged?

Sure thing, just opened a pull request.


eip.tf generated with non-existent key

Generating the eip.tf file results in an invalid file when there's an eip without attached instance.
Without an instance to specify, terraforming specifies the network interface ID.
It uses network_interface_id as the key for this value. This, however, is not a valid key in terraform.
According to the documentation this key is supposed to be "network_interface", not "network_interface_id".
Seems like a small fix is required, it does load the correct value, just uses the wrong notation for the key itself. Sadly I'm not able to fix this myself as I have never worked with Ruby. I guess the problem is in this file.

Thank you for reporting ❗ and sorry for late response 🙇
I reproduced this on my account and I found it's just a typo. I'll fix this soon.


Compile PipeCNN Alexnet openCV demo for DE10-Nano

Need to get cross compiler working with opencv

https://docs.opencv.org/2.4/doc/tutorials/introduction/crosscompilation/arm_crosscompile_with_cmake.html
# done on ubuntu 16.04 24 vCPU instance
sudo apt install cmake libv4l-dev libopencv-dev libjpeg-dev libjasper-dev libavcodec-dev libavformat-dev x264 v4l-utils ffmpeg
cd ~
git clone https://github.com/opencv/opencv.git
cd opencv
git tag
# checkout the latest 2.4 release
# https://github.com/opencv/opencv/releases
git checkout -b <IP_ADDRESS> tags/<IP_ADDRESS>
mkdir build
cmake -DCMAKE_BUILD_TYPE=RELEASE -DWITH_V4L=ON -D-DENABLE_NEON=ON -DCMAKE_TOOLCHAIN_FILE=~/opencv/platforms/linux/arm-gnueabi.toolchain.cmake ~/opencv
make -j24

This builds opencv on the server, but now I'm not sure what to do... Link the main.cpp to this?

We can just compile on the FPGA board. The YOLO webcam + opencv + pipecnn is working! 671ms per inference right now


Use TabNavigator instead of StackNavigator

Use TabNavigator instead of StackNavigator  #10

Will test this locally and merge today

Looks Cool! Thanks for the PR. 👍  Merging it now

Thank you for your quick reply. 💯


Add support for direct connection to device

As asked in https://github.com/rkosegi/jdownloader-cli/issues/51#issuecomment-2472410498, evaluate what it takes to add support for direct connection between CLI and device (without involvement of my JD). This will probably need update in client library

For what it's worth, JD has a deprecated API available in the Advanced Settings flags. Under normal circumstances I think you connect via MyJD and then after authenticating the session it attempts to connect directly to local instances of JD and falls back to the MyJD relay if this fails, though how it does this is a black box and isn't really documented.
The deprecated API needs to be manually enabled via a flag (followed by a JD restart) and listens on port 3128. By default it only listens on localhost but another flag will allow LAN connections. It allows unauthenticated API calls to be made (same endpoints as the modern MyJD API), I think this is how that Python library attempts to connect to JD directly.


Implement View Statistics

Currently the button does nothing. Let's make it show:

Total of people welcomed by User (All time)
Total of people welcomed by User (This week)
Total of people welcomed by User (This month)
Total of people welcomed all time
Total of people welcomed this week
Total of people welcomed this month

I'd love to give this a try!


cardano-tracer: Resolve MVar issue in benchmark suite.

Description
The benchmarking suite would get blocked indefinitely on MVar:
cardano-tracer-bench: ExceptionInLinkedThread (ThreadId 8) thread blocked indefinitely in an MVar operation
Benchmark cardano-tracer-bench: ERROR

The solution was to wrap each test in beforeProgramStops. It also makes minor changes:

Adds two "full" profiles trace-full and trace-full-rtview which run longer (45 blocks synced).

Adds common language extensions, like BlockArguments, ImportQualifiedPost (and update some code accordingly).

Make Criterion as verbose: This can be overridden by --verbosity={0,1,2}:
cabal bench cardano-tracer --benchmark-options=--verbosity=0

Checklist

[ ] Commit sequence broadly makes sense and commits have useful messages
[ ] New tests are added if needed and existing tests are updated.  These may include:

golden tests
property tests
roundtrip tests
integration tests
See Runnings tests for more details

[ ] Any changes are noted in the CHANGELOG.md for affected package
[ ] The version bounds in .cabal files are updated
[ ] CI passes. See note on CI.  The following CI checks are required:

[ ] Code is linted with hlint.  See .github/workflows/check-hlint.yml to get the hlint version
[ ] Code is formatted with stylish-haskell.  See .github/workflows/stylish-haskell.yml to get the stylish-haskell version
[ ] Code builds on Linux, MacOS and Windows for ghc-8.10.7 and ghc-9.2.7

[ ] Self-reviewed the diff

Note on CI
If your PR is from a fork, the necessary CI jobs won't trigger automatically for security reasons.
You will need to get someone with write privileges.  Please contact IOG node developers to do this
for you.

Superseded by #5554 .


AirPods Pro 2 - USB C on 16.4.1

Hi there!
I tested using PreRelease 0.4.0 ~ 1 and it causes crash of Bluetooth when using anything other than AirPods, such as cad stereo etc.
so I’ve had to remove for now. But it’s a bad experience without PodsGrant because no battery info, Siri announcements etc.
please help! I can provide assistance in debug or whatever is needed

Attaching my Bluetoothd file here
bluetoothd.zip

Could you provide any logs related to the crashes? You might be able to get them by searching bluetoothd or sharingd at Settings > Privacy > Analytics & Improvements > Analytics Data.

bluetoothd-2024-12-18-230002.txt
Same issue with bluetoothd crashing, also using AirPods Pro 2 USB-C on 16.4.1, attached my log.

Seems iOS 16.4.1 has a different bluetoothd binary than iOS 16.4, please check prerelase 3.

Tried the new pre-release, working very well so far. Annouce Notifications and other stuff is now working. Best!


Production plan: reducing resources deletes driving assertions.

In the production plan, the number of resources (vehicle & drivers) can be edited. As soon as a allocated resource is deleted, e.g. 20 -> 19 resources, the application will delete one resource (driving pool).
However, any driving assertions allocated to such a driving pool should not be deleted, but remain in existence and no longer allocated to any driving pool.

Fixed, see the following, where the needed resources have been reduced from 20 to 4: