add KEEP_PACKAGE_SOURCES option to hunter_config()

PR for initial review/discussion:

append KEEP_PACKAGE_SOURCES option in hunter_calculate_config_sha1.cmake if specified
introduce _hunter_keep_package_sources in hunter_download.cmake as logical OR of global HUNTER_KEEP_PACKAGE_SOURCES and local HUNTER_${h_name}_KEEP_PACKAGE_SOURCES

I've pushed a modified version up for discussion.  I'll prepare a minimal test to reproduce the issue I'm seeing.

Released: https://github.com/ruslo/hunter/releases/tag/v0.19.165


 This breaks the absolute paths in the service files.

Workaround implemented in e4fc333f4ce04ada712e66a0d39350baf4add6f3, dbus config still not working. It might be installed in /usr/local/share/dbus-1/, but dbus requires it to be in /usr/share/dbus-1/.

As documented in the appendix of debian's python-policy for packaging, debian uses modified versions of distutils and setuptools that set the default installation dir to /usr/local/lib/pythonX.Y/dist-packages.
By running setup.py install with the --install-layout=deb flag, this is reverted to /usr/lib/pythonX.Y/dist-packages.
When using pip to install (according to the pip docs), this can be achieved using the --install-option argument.
To make sure the configs are installed in the right directory, maybe also look at the python3 documentation for custom installation schemes.

Indeed!
On debian-based systems the installation to the right prefix can be achieved with:
pip3 install --extra-index-url https://pip.alx-g.de/ --no-binary bsrv --install-option='--install-layout=deb' bsrv
The problem is, that pip will complain during uninstalling:
Not uninstalling bsrv at /usr/lib/python3/dist-packages, outside environment /usr
and apparently, there is currently no way to uninstall this package, ever.

Installation will be done with system packet managers (pacman, apt-get) in the future.


Pagination

Hello,
Seems like the CSS doesn't work for pagination. I have imported  but I don't get the default buttons for pagination.

what's thing did you import, could you please give more detail information :)

yes sorry, I imported the CSS (react-bootstrap-table-all.min.css) the scroll bar works and it only shows me the 10 first values; however, the buttons that I am supposed to get at the bottom are missing, I don't get buttons I only get a list.

did you enable search={true}? and any error on browser console?

no errors on console.
<BootstrapTable data={this.state.data} pagination={true} search={true}>       <TableHeaderColumn dataField="name" isKey={true}>Name</TableHeaderColumn>       <TableHeaderColumn dataField="name">Name</TableHeaderColumn>       </BootstrapTable>
Everything works fine, the only thing is the CSS for the pagination buttons. Do you know what am I missing? Do I have to import something else?

I do not want to search it's just the default pagination sample. Sorry for any misunderstandings

@arrb, so you means you did not see any the pagination list or page button? it's wired, what's bootstrap version and react-bootstrap-table did you used?

https://cloud.githubusercontent.com/assets/9982963/10610837/e00a8c68-7749-11e5-92da-0bd443989903.png
I can't see that (click on click above) I downloaded react-bootstrap-table today. I have these versions:
"react-bootstrap": "^0.28.3",
"react-bootstrap-table": "^2.0.0",

I know it seems odd to me too, I really can't see where my error is.

@arrb, did you have any repo or fully example which can reproduce this issue? btw, react-bootstrap-table have no any dependency on react-bootstrap :)

Okay I am just trying to think of every single possibility.
I don't have the repo right now, but everything works perfectly. I get all the data from a server, even sort works it's just the styling for pagination... Well it's okay thanks for helping anyways

@arrb, really? I think there must some thing stuck and maybe is a bug. however please tell me how did you solve. If you can't solve it, feel free to give your repo or examples. so I will close this issues until this issue could be confirm.
Thanks, feel free to ask anything :+1:

Hi,
I "fixed it" just by adding my own css, but I still would like to know what's wrong. Here is a link to my repo: https://github.com/arrb/reduxSample

by adding my own css

is this?

yes, my question is just if you know why the default css doesn't work for pagination? is there something I did wrong?

ok, I will spend some time to figure out :) Thanks your feedbacks

Same here, default css doesn't work in pagination part. I also import css and version I use is React-bootstrap-table 4.0.2

Same issue here pagination CSS looks weird and does not work properly.


png to webp (not-lossless) option ?

ngx_pagespeed can convert png to webp lossless. But is there an option to save to webp not-lossless and specify a quality % ?
I did some tests at https://community.centminmod.com/posts/23323/ and ngx_pagespeed auto converts png to webp lossless and ends up at 424KB size same as if i used irfanview app to manually save to webp lossless. But if i use irfanview to save as webp with quality = 75%, the webp image is reduced to between 83-88KB and I can't perceive a difference between it and 424KB webp lossless version.
So would be nice if ngx_pagespeed had a filter for same png to webp that is not lossless :)
cheers
George

From the docs for convert_jpeg_to_webp: https://developers.google.com/speed/pagespeed/module/filter-image-optimize#convert_jpeg_to_webp
"With this option enabled, a gif or png image can also be converted to a lossily-encoded webp if the configuration allows it to be converted to a jpeg"
Which works with WebpRecompressionQuality: https://developers.google.com/speed/pagespeed/module/filter-image-optimize#WebpRecompressionQuality
Does that look like what you want?

@oschaaf cheers totally overlooked that was a separate option !


Refactor code to improve performance

Changed the status line function to just get a buffer variable for the
status line and delegate the update of this variable to the lsp_handlers

Thanks for this! I think this is a good direction to take, but there are a few issues with the current implementation. First, we would ideally not call the (relatively expensive) get_lsp_statusline function on every message update. I'm playing around with an alternative to this. Second, this seems to break messages with other language servers for me - for example, with Sumneko's Lua LSP, the messages just freeze.
I'll try to fix things up this weekend and ping you for review. Thanks again!

I'm happy to help, could you elaborate on the issue? I found that the last message on some servers are not updated with ✔ until you change function and so issue another update.

I've added the timer to defer the get_lsp_status call every 100ms. It uses the vim buffer variables as a global state to know when there is an update. Which probably isn't good but it's working.
Also I noticed the statusline text creates an array and then concatenates all the elements together, which is more work than concatenating each element to a sting.

Thanks again for this. I think the issue I was seeing is what you mention, that the last message is not updated until triggering another event.

I've added the timer to defer the get_lsp_status call every 100ms. It uses the vim buffer variables as a global state to know when there is an update. Which probably isn't good but it's working.

Thanks, I'll take another look at the design.

Also I noticed the statusline text creates an array and then concatenates all the elements together, which is more work than concatenating each element to a sting.

Yes, this is more expensive than a fixed, known number of concatenations, but is more convenient and more efficient than concatenating in a loop (ref: https://gitspartv.github.io/LuaJIT-Benchmarks/#test18).

Oh, minor nitpick: can I request that you rename this branch to refactor/throttle-messaging? refactor is a bit vague, and - while I think I could change the branch name on my end - I think it'll be easier from yours. Thanks!

One suggestion, though I haven't gone through the changes in depth yet: what if, instead of a constantly running timer, we test if the time since last update is within one interval-length of the current time, and issue a redrawstatus! if it is not? That would simplify the flow of the code a bit, and potentially save on computation (though probably not beyond the level of a micro-optimization).


$createUser promise not returning user object

I'm trying to create a new user, then grab the uid so that I can save additional user data during registration. I'm finding that there is no user object being returning as per the doco in https://github.com/firebase/angularfire/blob/master/src/FirebaseAuth.js.
Here is my Authentication service...
myApp.factory('Authentication',
  function($firebaseAuth, FIREBASE_URL, $location) {

  var ref = new Firebase(FIREBASE_URL);
  var authObj = $firebaseAuth(ref);

  var myObj = {
    register: function(user) {
      return authObj.$createUser(user.email, user.password)
        .then(function(authData) {

          console.log(authData); // returns undefined

        });
    },

    login: function(user) {
      return authObj.$authWithPassword({
        email: user.email,
        password: user.password
      });
    },

    logout: function() {
      return authObj.$unauth();
    }
  }

  return myObj;

});

This seems to affect both, legacy Simple Login and the new built-in Firebase Auth.
Hopefully this is going to be fixed in the next few days. This feature is rather useless without user object being returned.

Seems fixed now.

Yes, as of AngularFire 0.9.1 (which was released yesterday), the $createUser() method now returns a user object containing the uid of the created user. Please upgrade to AngularFire 0.9.1 or later if you need this functionality.

I have still not been able to get the userData back from the $createUser method.  I am using the cdn  to import code into my project and I have Firebase version 2.0.4 and AngularFire 0.9.1.  I have a simple controller which has the example code from the docs pretty much exactly copied and pasted.  After successfully creating a user console log reports "cannot read property 'uid' of undefined"

The user data was not exposed from the Firebase client until version 2.0.5 (here is the full changelog for the client). Just upgrade your Firebase version and you'll be all set. The latest version (which is 2.1.1 as of this comment) is fully compatible AngularFire 0.9.1 so I'd suggest upgrading to that.

That was it, thanks jwngr!


Add novnc subfilter

Description
This PR fixes novnc deployment issues with the new nginx version

/deploy-staging

/rebase

/merge

/merge


README Updated (typos)

PR Branch
#ADD LINK TO THE PR BRANCH
Motivation and Context
Checklist:

[ ] Unit tests added

[ ] Integration tests added

[ ] Test names are meaningful

[ ] Feature manually tested

[ ] Branch build passed

[ ] No 'package.*' in the imports

[ ] Relevant Wiki page updated with clear instruction for the end user

[x] Not applicable. This was only a refactor change, no functional or behaviour changes were introduced

[ ] Http test added to http-testing module(if applicable) ?

[x] Not applicable. The changes did not affect HTTP automation flow

[ ] Kafka test added to kafka-testing module(if applicable) ?

[x] Not applicable. The changes did not affect Kafka automation flow

😀 thanks, Have a great journey ahead 🤗
On Fri, Oct 2, 2020, 3:59 AM authorjapps<EMAIL_ADDRESS>wrote:

@authorjapps approved this pull request.
👍
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/authorjapps/zerocode/pull/439#pullrequestreview-500760602,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANUATRNSUSEPZ32PS2MDQKLSIT7F5ANCNFSM4R72N44Q
.


Grafana secrets expect a key named password

Probably a copy/paste mistake from kiali as it uses passphrase there.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: gottfrois
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: gyliu513
If they are not already assigned, you can assign the PR to them by writing /assign @gyliu513 in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

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

Hi @gottfrois. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

I signed it!


Other matrix

Ist it possible to use it with an other matrix as an 4x16
I have an older board with an 8x8 matrix but it's not really fast by calculating the points.
thats why I would upgrade

While this is totally possible I did code the matrix part for a löwen dart spare part matrix. Unfortunately without the matrix at hand I cannot convert the C program to support the matrix.


I have weird terminal in my right click menu and I don't know how to remove it!

Unfortunately, that's not us. But if you search around for this registry path you should be able to find some information on how to clean those up: HKEY_CLASSES_ROOT\Directory\shell
Be careful modifying your registry though, good luck.


Enhanced ElasticSearch Sink (TLS, Auth, KeyValue Schema, INSERT/UPDATE/DELETE,  "_id" management, multi index support)

Motivation
This patch (based on original work by @vroyer at #11263)  is a set of enhancements to the ElasticSearch Sink:

support TLS
support authentication
support KeyValue schema and other Pulsar Schemas
support Bulk writes
support "errors action" (FAIL, IGNORE...)
support AVRO records (Avro to JSON conversion)
support for using the "key" of the record as record "_id" in ES (supporting both Message key and KeyValue encoding)
support for creating the "_id" in ES using a selection of values from the Message (in case of AVRO/JSON message)
support DELETING documents in case of non empty key and null value
support multiple ES indexes (with automatic mapping between Topic Name and Index Name)

By default the Sink behaves the same way it did with Pulsar 2.8.0, that is basically reading from the Topic and writing to ElasticSearch.
Schema related features are to be enabled explicitly by configuring "schemaEnable: true" in the configuration.
This patch also re-enables ElasticSearch integration tests, that were disabled in 2.8.0.
Apart from the security related features the most important feature of this patch is that we are not supporting KeyValue schema and we are supporting the concept of "_id" (key) of ElasticSearch.
So when a record comes to the Sink we can:

INSERT the record if it is missing in ElasticSearch
UPDATE the record if it is present
DELETE the record if it is present and the value in Pulsar is NULL

This is very powerful because we can integrate ElasticSearch with other CDC Sources (especially Debezium) and with the Kafka Source in order to use Pulsar as a bridge to connect any Database to ElasticSearch
Verifying this change

this change add unit tests that cover the changes and the new features
this change re-enables ES integration tests (disabled in 2.8.0?)

We (at DataStax/Kesque) have also tested manually the compatibility  with the previous version of the sink.
It is hard to setup a backward compatibility test and therefore it will add cost on CI.
Documentation
Once this patch is merged I will be happy to enhance the documentation, because we have many new features.

@nlu90 @freeznet Can you review this pull request?

@eolivelli thanks for taking the doc into consideration, feel free to ping me if you need a doc review.

Integration tests are failing due to  this bad problem with the classpath, basically the problem is that the AVRO library is not usable and you cannot manipulate the result of GenericRecord.getNativeObject()
https://github.com/apache/pulsar/issues/11274

@Anonymitaet I will follow up with a PR for the docs. Stay tuned

Doc: https://github.com/apache/pulsar/pull/11602


Refactor default vhost implementation

This is intended to fix #33 as well as stay compatible with the fix for #8.
The AMQP URI specification states that a missing vhost (i.e. no path on the URI) should be treated as not provided, whereas a trailing / should be treated as an empty vhost.
This PR will adjust this behaviour by using the default vhost if not provided, and trimming the / if it is provided (thus making it empty). Previously this was broken as url.path() will always return / (even if it was not provided in the input); in the case it wasn't provided, the vhost was being set to "" due to the previous clean_vhost implementation. This was coincidentally the same as the default, but this shouldn't be relied upon (read on...).
In order to resolve #33, I took the liberty of setting the default vhost to "/" as it's the default for RabbitMQ, which is widely regarded as the most popular AMQP implementation. Having "" as the default is unintuitive (see #33) for RabbitMQ users as you need to supply amqp://host// to connect successfully.
Tests still pass after accounting for the change to the default, so it shouldn't really break anything; anyone still using amqp://host// will still function correctly, as well as anyone specifying the vhost. The only behaviour change is the default vhost, which can technically be seen as a bug anyway :p

I think this looks alright.
This vhost story is really confusing, hopefully this should clear up some confusion.
Thanks for the PR!


[Java] Fix failed test

InferRequestTests > testGetPerformanceCounts FAILED
    java.lang.AssertionError: Map size expected:<10> but was:<22>
        at org.junit.Assert.fail(Assert.java:89)
        at org.junit.Assert.failNotEquals(Assert.java:835)
        at org.junit.Assert.assertEquals(Assert.java:647)
        at InferRequestTests.testGetPerformanceCounts(InferRequestTests.java:57)

source: https://dev.azure.com/openvinoci/dldt/_build/results?buildId=87114&view=logs&j=0609cd82-f9f7-5e70-371c-89fca78031ab&t=0971f6e8-539b-5170-a800-5a6fa6f3c74b

Layers were changed:

Name before
Name now

19/Fused_Add_
Content Cell

21
Content Cell

21
Content Cell

21
Content Cell

21
Content Cell

21
Content Cell

("19/Fused_Add_");
layer_name.add("21");
layer_name.add("22");
layer_name.add("23");
layer_name.add("24/Fused_Add_");
layer_name.add("26");
layer_name.add("27");
layer_name.add("29");
layer_name.add("fc_out");
layer_name.add("out_fc_out");

@dkurt  @likholat are we going to align the test with the modified network? Or other actions are required here?

For now test is disabled. Well, it's better to check only API but relax the requirements to pass the test.

@dkurt is it supposed that @likholat will change the test? Or somebody else needs to update it?

@likholat, please enable the test with the relaxed check


Remove outdated prefixes

Stylis includes a prefixer module:
import { prefixer } from 'stylis';

This is very helpful as there are CSS properties that need vendor prefixes. For example, user-select: none; still requires it in browsers that have an important global usage rate.
However, I have noticed a couple of prefixes that I would argue are more harmful than beneficial. Mainly, the flexbox properties. The global need for the prefix seems to be 0.48% (0.01 + 0.14 + 0.05 + 0.01 + 0.02 + 0.03 + 0.01 + 0.1 + 0.11). Do we need to keep it, at this point?
By removing it, we would gain:

A better DX when debugging CSS. When this module is used with emotion, you get this kind of output by default

Before

After is so much better:

A smaller bundle.

Actually, this might simply be the root cause of a different problem: What's the browser support target? Do we have something like this https://github.com/kripod/style-vendorizer#browser-support?

A fun read https://css-tricks.com/is-vendor-prefixing-dead/#aa-prefixing-in-2021
A related thread https://github.com/styled-components/styled-components/issues/285
I have reopened #283 as I didn't understand why it was closed.

yeah, prefixes are dead, let's have them optional pls

cc @brijeshb42 adding this in your radar. It fits well under the "improving the style engine of MUI" north start objective of your team (so far 1 person but hopefully we can grow this).


[WIP] Add Jenkinsfile and Dockerfile for AIO Jobs

This Jenkinsfile will be called by CIT Jenkins for PRs and Branch builds.
It will in turn call the RPC-AIO job to test swift/ceph/xenial/upgrades.
The results of the AIO builds will be added to the PR, with direct links.
Lint Jobs are executed in Docker, to isolate them from the CIT slave.
The Lint jobs are executed before AIOs to prevent wasting resources on
AIO builds when lint fails.
Connects rcbops/u-suk-dev#1313

recheck_ceph

CIT ceph and xenial jobs failed on downloading ruby gems, retrying.

skip ci

recheck_ceph recheck_xenial


Fix/iclr setup

use tqdm for all the iterations.
fix authorids field description.
make the pdf field mandatory for the revision invitation

Does dev have the current webfield code?

Thanks @zbialecki , I just deployed the changes in the dev site, could you take another look? I want to make sure the approach of reloading the whole tab panel is correct.

I think it's working correctly, the only thing I noticed is that it's missing the referrer banner


[Idea] Support dual sided sticker

As a user I would like to follow the pattern of Zach Poff and fold the sticker over the edge of the bag

Idea
It would be nice if there were an option to mirror the output so it could be displayed on both sides of the bag. Here is an example:

Constraints
Understandably not all users may want this feature, so it would be ideal to make a feature flag or environment variable to enable it.

Agree :)
I feel like this is getting out of hand for a simple Python Script, I am seriously considering to rewrite the entire thing in Rust, with a proper command line and yaml configuration.
That would have the advantage that you can simply download the released binary instead of having to clone the repository and deal with git. I hear that for many people downloading things from github isn't intuitive at all; I simply don't see this point of view because I work with git/github/gitlab/gerrit etc since >10 years, so it feels very intuitive to me :D
What's your opinion on that matter?

As a user I would like to follow the pattern of Zach Poff and fold the sticker over the edge of the bag

I think the intend of Zach Poff was to have it only on one side, as seen in his original pdf.
I agree with your request to make it configurable, though.

I've created a rough draft which accomplishes what I'm hoping for here: https://github.com/Finomnis/ResistorLabels/pull/12
I've left it as a 'draft' PR because it still has 1 major bug I haven't been able to solve yet. As long as the user is printing full pages, my branch works as intended.

Closed by https://github.com/Finomnis/ResistorLabels/pull/12


Automatic PR from RestAPI-PR1647

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/1647

Codecov Report

Merging #1424 into master will decrease coverage by 0.01%.
The diff coverage is 4%.

@@            Coverage Diff             @@
##           master    #1424      +/-   ##
==========================================
- Coverage   52.61%   52.59%   -0.02%
==========================================
  Files        3344     3344
  Lines       85048    85073      +25
==========================================
+ Hits        44746    44747       +1
- Misses      40302    40326      +24

Impacted Files
Coverage Δ

.../network/v2017_08_01/models/load_balancing_rule.py
100% <ø> (ø)
:arrow_up:

...gmt/network/v2017_08_01/models/inbound_nat_rule.py
100% <ø> (ø)
:arrow_up:

...gmt/network/v2017_06_01/models/inbound_nat_rule.py
31.25% <ø> (ø)
:arrow_up:

.../network/v2017_06_01/models/load_balancing_rule.py
27.77% <ø> (ø)
:arrow_up:

.../operations/virtual_network_gateways_operations.py
16.83% <4%> (-1.16%)
:arrow_down:

Continue to review full report at Codecov.

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

Network handled manually


More intelligent transaction requests

UPDATE: new plan described in https://github.com/input-output-hk/ouroboros-network/issues/1161#issuecomment-594423279
Old plan:

Currently the txSubmissionInbound protocol handler does not look at the Mempool to see if txs are already there before requesting them. So in practice, it will needlessly download txs it already has. More confusingly, we can observe that node A sends a tx to B, which then sends it right back to A.
There are multiple options to improve this situation:

Look at the Mempool and don't request txs already in there.
Maintain a shared (across all txSubmissionInbound protocol handlers) set of TxIds that were recently in the Mempool. They should also expire from this set, so perhaps a PSQ would be a good fit. This can subsume (1).
Keep track of which TxIds we're asking from which peers and don't ask for the same one from multiple peers unless that would otherwise cause things to stall, in which case we should make another request for the same TxId.

Because of the more complex concurrency of (3), we'll keep that for later. Getting (1) working will be easy. Our current aim is to achieve (2).

Moving to 1.1 as it's inessential and we need to cut the release soon.

@dcoutts has also informed me that this PSQ should be shared between all the tx submission handlers, remote and local.

While working on #1514, @nfrisby discovered that unlike duplicate txs, duplicate votes are not rejected. Without the improvements described in this issue, the nodes end up resubmitting the votes to each other so that blocks end up containing the same votes many, many times. (I'll use tx to mean generalised transaction, which includes votes)
The approach taken in #1476 (option 2 from the description of this issue) would mitigate this, but not entirely. It helps avoid requesting duplicate txs from other nodes, but it doesn't avoid receiving duplicate txs from the (local) wallet.
A proper solution would be to track in the Mempool which txids it contains and reject incoming txs/votes that are already in the Mempool.
However, this would largely duplicate the txids cache of RecentTxIds.
Therefore, @edsko and I came up with a new approach:

Remove RecentTxIds.
Add MempoolSnapshot.hasTx :: GenTxId blk -> Bool, implement it by keeping a Set (GenTxId blk) in the Mempool. (Let it return Maybe (GenTx blk) and keep a Map (GenTxId blk) (GenTx blk)? Would that be useful? YAGNI?)
Instead of using RecentTxIds to check which txs to request/acknowledge in TxSubmission.Inbound, use MempoolSnapshot.hasTx to do this.
In Mempool.tryAddTxs, reject txs already in the Mempool. I suppose we'll need to introduce a sum type that includes ApplyTxErr blk and the new case, i.e., DuplicateTx or TxAlreadyInMempool.

We believe this approach is simpler than the one described in (2) (reason about "recent" and time is more complicated), it fixes the duplicate votes problem @nfrisby discovered, it avoids complex interactions in #1496.
The only scenario we believe in which it is worse than option (2):

We receive or forge a block containing txs
We remove these txs from our mempool
Another node receives some of those txs
That node informs us about these txs
We request them, even though we have requested them in the past, which we could have avoided with RecentTxIds
We receive the txs, but the Mempool rejects them, as they are part of our ledger already (except for votes).

@dcoutts What do you think about the new direction (let's call it option 1+) for this issue and #1743?

It's worth noting that we could mitigate this duplicate votes issue with the RecentTxIds approach by performing a lookup before attempting to add a tx received by the local tx submission server to the mempool. However, I'd say that the new approach is worth pursuing for the reasons mentioned by @mrBliss.

Note that we have to be very cautious: the wallet will monitor the blockchain to verify whether or not transactions that it has submitted are eventually included, and if they aren't, resubmit them. We have to be sure that such resubmitted transactions aren't ignored.


Add "why this package" to readme

From the readme, I don't get the purpose of this package. What's the difference to rosbridge_server? Is there any advantage in using this over rosbridge_server in conjunction with Studio?

Fixed with #56


"Reflective" Particle construction

This is the same as #5140, but with a cleaner git history to make copybara easier.

Waiting on #5243 to fix flaky storage proxy test.


Missing parameter for sklearn.ensemble.HistGradientBoostingRegressor

Since scikit-learn 1.4 HistGradientBoostingRegressor has had parameter max_features: https://github.com/scikit-learn/scikit-learn/blob/6e9039160/sklearn/ensemble/_hist_gradient_boosting/gradient_boosting.py#L1499
This appears to be missing from the stubs: https://github.com/microsoft/python-type-stubs/blob/main/stubs/sklearn/ensemble/_hist_gradient_boosting/gradient_boosting.pyi#L90

Thanks for the issue. We would welcome a PR with a fix.

@debonte, I've given it a shot in #323. Let me know if I need to do anything further!

@ara-incom, thanks. I won't be able to look at this until Monday. Just wanted to let you know that I saw it and an not ignoring it.


igGrid AOT sample is throwing an error

Description
igGrid AOT sample is throwing an error
Steps to reproduce

Go to http://igniteui.github.io/igniteui-angular-wrappers/samples/igGrid-AoT/igGrid-AoT.html
Observe browser console

Result
There is an exception:
Error
ZoneDelegate.invoke @ zone.js?main=browser:192
Zone.run @ zone.js?main=browser:85
(anonymous) @ zone.js?main=browser:451
ZoneDelegate.invokeTask @ zone.js?main=browser:225
Zone.runTask @ zone.js?main=browser:125
drainMicroTaskQueue @ zone.js?main=browser:357
ZoneTask.invoke @ zone.js?main=browser:297
Expected result
There should be no exception.

We don't have that sample any more, ng serve --aot will run all of the sample in AOT


Add filter diagnostics (flags for when each filter is active)

[ ] https://github.com/CliMA/ClimaAtmos.jl/pull/1344
[ ] https://github.com/CliMA/ClimaAtmos.jl/pull/1329

Done


Why does sum() not return NumberPath ?

table column is Integer,but sum(Integer)  return  NumberPath<Integer>  .
but  sum(Integer) has the possibility of exceeding the length;

Most operations in ANSQI SQL and JPA return the type of their parameters.
If you need bigger numbers, you should cast the parameters. Or if your DBMS doesn't need it, feel free to cast the result instead.


Added Ashmi Chheda

Added 2 women contributors

@ashmichheda I squashed your commits and merged. Please squash such commits always.

I think your comment got deleted by me.

@tapaswenipathak Thanks for squashing. I will do so henceforth.
Thanks!


Should egui::Buttons change the cursor to PointingHand on hover?

At the moment, mousing over an egui::Button does nothing, but I think it should change to PointingHand to be consistent with most other UIs.
For example, mousing over any button on this page, or in vscode, etc. will turn the cursor into PointingHand.
This could be achieved with ctx.output_mut(|out| { out.cursor_icon = CursorIcon::PointingHand }) inside the .hovered() call for a button.
It would be painful to implement this everywhere yourself, so maybe it should just be part of the component itself?

this is a web browser thing that absolutely does not belong on native controls! leave this up to the user please and do NOT use the pointing hand cursor by default


Implement the following flags for buildah run

Implement the following flags for buildah run
    --cpu-period
    --cpu-quota
    --cpu-shares
    --cpuset-cpus
    --cpuset-mems
    --memory
    --memory-swap
    --ulimit

Signed-off-by: umohnani8<EMAIL_ADDRESS>

@rhatdan @TomSweeneyRedHat PTAL

LGTM, but would like an eyeball from @rhatdan or @nalind

Reworking this whole PR.Closing for now.


Path error workaround

Looks like theres is a bug in Xcode 7b5... BuildaUtils won't build because of stringByAppendingPathComponent - claiming that this method is available only for NSString but we're trying to call it on String... 😐
Of course, other ppl are facing the same issue (https://forums.developer.apple.com/thread/13580), so for now we'll have to do what I've done in this PR.

Looks like DVR is facing the same kind of issues...
Created a PR with hot-fix: https://github.com/venmo/DVR/pull/18

Sorry man but I won't be at my Mac till tomorrow night so I can't fix the spec till then :/

Nevermind, we still have issues with DVR.

Thanks! New spec released - 0.0.9.


missing instructions for ff/burp config

somewhere around line 36, we need some instructions like:
Tell Firefox to use the Burp Proxy:

Preferences > General > Network Settings

Select "Manual proxy configuration"

HTTP Proxy : <IP_ADDRESS>
Port: 8080

Click "OK"

Which file is this needed to be updated?

Apologies.  This one:
documentation/cloud-compute/application-misconfigurations.md

Added. Check and close this issue, if the explanation is okay.


Chef::Exceptions::EnclosingDirectoryDoesNotExist: Parent directory /etc/profile.d does not exist.

I am trying to run this on a mac. chef detects the machine to be mac. but still this error pops. I even tried to change the directory as /etc/profile which is specific to mac. but still I get the same issue.
Detailed log :
Synchronizing Cookbooks:

ark (3.0.0)
dmg (3.1.1)
maven (5.0.0)
sublime-text (0.1.5)
build-essential (8.0.1)
windows (3.0.5)
seven_zip (2.0.2)
java (1.49.0)
apt (6.1.0)
mingw (2.0.0)
ohai (5.0.3)
homebrew (4.1.0)
Installing Cookbook Gems:
Compiling Cookbooks...
Converging 8 resources
Recipe: maven::default

ark[maven] action install

directory[/usr/local/maven-3.5.0] action create (up to date)

remote_file[/Users/arun/PurpleSlate/.chef/file_cache_path/maven-3.5.0.tar.gz] action create (up to date)

execute[unpack /Users/arun/PurpleSlate/.chef/file_cache_path/maven-3.5.0.tar.gz] action nothing (skipped due to action :nothing)

execute[set owner on /usr/local/maven-3.5.0] action nothing (skipped due to action :nothing)

link[/usr/local/maven] action create (up to date)

template[/etc/profile/maven.sh] action create

Parent directory /etc/profile does not exist.
================================================================================
Error executing action create on resource 'template[/etc/profile/maven.sh]'
================================================================================

Chef::Exceptions::EnclosingDirectoryDoesNotExist
Parent directory /etc/profile does not exist.
Resource Declaration:
In /Users/arun/PurpleSlate/.chef/file_cache_path/cookbooks/ark/resources/default.rb
101:     template "/etc/profile/#{new_resource.name}.sh" do
102:       cookbook 'ark'
103:       source 'add_to_path.sh.erb'
104:       owner 'root'
105:       group node['root_group']
106:       mode '0755'
107:       cookbook 'ark'
108:       variables(directory: "#{new_resource.path}/bin")
109:       only_if { new_resource.append_env_path }
110:     end
111:   end
Compiled Resource:
Declared in /Users/arun/PurpleSlate/.chef/file_cache_path/cookbooks/ark/resources/default.rb:101:in `block in class_from_file'
template("/etc/profile/maven.sh") do
action [:create]
retries 0
retry_delay 2
default_guard_interpreter :default
source "add_to_path.sh.erb"
cookbook "ark"
variables {:directory=>"/usr/local/maven-3.5.0/bin"}
declared_type :template
cookbook_name "maven"
mode "0755"
owner "root"
group "wheel"
path "/etc/profile/maven.sh"
only_if { #code block }
end
Platform:
x86_64-darwin14

================================================================================
Error executing action install on resource 'ark[maven]'
Chef::Exceptions::EnclosingDirectoryDoesNotExist
template[/etc/profile/maven.sh] (/Users/arun/PurpleSlate/.chef/file_cache_path/cookbooks/ark/resources/default.rb line 101) had an error: Chef::Exceptions::EnclosingDirectoryDoesNotExist: Parent directory /etc/profile does not exist.
Resource Declaration:
In /Users/arun/PurpleSlate/.chef/file_cache_path/cookbooks/maven/recipes/default.rb
26: ark 'maven' do
27:   version node['maven']['version']
28:   url node['maven']['url']
29:   checksum node['maven']['checksum']
30:   home_dir node['maven']['m2_home']
31:   win_install_dir node['maven']['m2_home']
32:   append_env_path node['maven']['setup_bin']
33: end
34:
Compiled Resource:
Declared in /Users/arun/PurpleSlate/.chef/file_cache_path/cookbooks/maven/recipes/default.rb:26:in `from_file'
ark("maven") do
action [:install]
retries 0
retry_delay 2
default_guard_interpreter :default
declared_type :ark
cookbook_name "maven"
recipe_name "default"
version "3.5.0"
url "http://apache.mirrors.tds.net/maven/maven-3/3.5.0/binaries/apache-maven-3.5.0-bin.tar.gz"
checksum "beb91419245395bd69a4a6edad5ca3ec1a8b64e41457672dc687c173a495f034"
home_dir "/usr/local/maven"
win_install_dir "/usr/local/maven"
append_env_path true
extension "tar.gz"
prefix_bin "/usr/local/bin"
prefix_root "/usr/local"
owner "root"
path "/usr/local/maven-3.5.0"
release_file "/Users/arun/PurpleSlate/.chef/file_cache_path/maven-3.5.0.tar.gz"
end
Platform:
x86_64-darwin14

Running handlers:
Running handlers complete
Chef Client failed. 0 resources updated in 14 seconds

It looks like you are trying to use the maven cookbook: https://github.com/chef-cookbooks/maven which does not have mac os x as a supported platform. The problem that you are sharing doesn't appear to be ark specific.
On the mac platform /etc/profile is a file and not a directory. In your error message " template "/etc/profile/#{new_resource.name}.sh" do" this is attempting to set up a file there which will fail because /etc/profile is not a directory.
When you look at the cookbook, check the metadata.rb for the supports keywords that will let you know what OS's are supported.

Please reopen with additional details if this is still a problem.


tikv panicked in parse_proc_cgroup_v2

Bug Report

What version of TiKV are you using?
 5.3.0
What operating system and CPU are you using?

Ubuntu 20.04 in systemd-nspawn container (host kernel 5.15.2)
x86_64
Steps to reproduce

tiup cluster deploy tidb-test v5.3.0 ./topology.yaml --user root -i .ssh/id_ed25519
(using config-templates/simple-mini.yaml)
What did you expect?
Deploy successfully
What did happened?
thread 'main' panicked at 'assertion failed: `(left == right)`
  left: `2`,
 right: `1`', components/tikv_util/src/sys/cgroup.rs:167:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

tiup log:
{"task": "UserSSH: user=tidb, host=<IP_ADDRESS>\nMkdir: host=<IP_ADDRESS>, directories='/tidb-deploy/tikv-20160','/tidb-deploy/tikv-20160/log','/tidb-deploy/tikv-20160/bin','/tidb-deploy/tikv-20160/conf','/tidb-deploy/tikv-20160/scripts'\nMkdir: host=<IP_ADDRESS>, directories='/tidb-data/tikv-20160'\nCopyComponent: component=tikv, version=v5.3.0, remote=<IP_ADDRESS>:/tidb-deploy/tikv-20160 os=linux, arch=amd64\nInitConfig: cluster=tidb-test, user=tidb, host=<IP_ADDRESS>, path=/root/.tiup/storage/cluster/clusters/tidb-test/config-cache/tikv-20160.service, deploy_dir=/tidb-deploy/tikv-20160, data_dir=[/tidb-data/tikv-20160], log_dir=/tidb-deploy/tikv-20160/log, cache_dir=/root/.tiup/storage/cluster/clusters/tidb-test/config-cache", "error": "init config failed: <IP_ADDRESS>:20160: executor.ssh.execute_failed: Failed to execute command over SSH for 'tidb@<IP_ADDRESS>:22' {ssh_stderr: thread 'main' panicked at 'assertion failed: `(left == right)`\n  left: `2`,\n right: `1`', components/tikv_util/src/sys/cgroup.rs:167:5\nnote: run with `RUST_BACKTRACE=1` environment variable to display a backtrace\n, ssh_stdout: , ssh_command: export LANG=C; PATH=$PATH:/bin:/sbin:/usr/bin:/usr/sbin /tidb-deploy/tikv-20160/bin/tikv-server --config-check --config=/tidb-deploy/tikv-20160/conf/tikv.toml --pd \"\" --data-dir \"/tidb-data/tikv-20160\"}, cause: Process exited with status 101: check config failed", "errorVerbose": "check config failed\nexecutor.ssh.execute_failed: Failed to execute command over SSH for 'tidb@<IP_ADDRESS>:22' {ssh_stderr: thread 'main' panicked at 'assertion failed: `(left == right)`\n  left: `2`,\n right: `1`', components/tikv_util/src/sys/cgroup.rs:167:5\nnote: run with `RUST_BACKTRACE=1` environment variable to display a backtrace\n, ssh_stdout: , ssh_command: export LANG=C; PATH=$PATH:/bin:/sbin:/usr/bin:/usr/sbin /tidb-deploy/tikv-20160/bin/tikv-server --config-check --config=/tidb-deploy/tikv-20160/conf/tikv.toml --pd \"\" --data-dir \"/tidb-data/tikv-20160\"}, cause: Process exited with status 101\ngithub.com/pingcap/tiup/pkg/cluster/spec.checkConfig\n\tgithub.com/pingcap/tiup/pkg/cluster/spec/server_config.go:308\ngithub.com/pingcap/tiup/pkg/cluster/spec.(*TiKVInstance).InitConfig\n\tgithub.com/pingcap/tiup/pkg/cluster/spec/tikv.go:284\ngithub.com/pingcap/tiup/pkg/cluster/task.(*InitConfig).Execute\n\tgithub.com/pingcap/tiup/pkg/cluster/task/init_config.go:51\ngithub.com/pingcap/tiup/pkg/cluster/task.(*Serial).Execute\n\tgithub.com/pingcap/tiup/pkg/cluster/task/task.go:85\ngithub.com/pingcap/tiup/pkg/cluster/task.(*StepDisplay).Execute\n\tgithub.com/pingcap/tiup/pkg/cluster/task/step.go:111\ngithub.com/pingcap/tiup/pkg/cluster/task.(*Parallel).Execute.func1\n\tgithub.com/pingcap/tiup/pkg/cluster/task/task.go:142\nruntime.goexit\n\truntime/asm_amd64.s:1581\ninit config failed: <IP_ADDRESS>:20160"}

I noticed my /proc/self/cgroup has 2 lines:
root@focal:~# cat /proc/self/cgroup
1:name=systemd:/
0::/user.slice/user-0.slice/session-13.scope

Edit: It seems the strange 2-line /proc/self/cgroup is a result of running centos 7 in nspawn container (Image from https://hub.nspawn.org/images/). It restored 1-line after a reboot.

From the cgroup v2 doc:

"/proc/$PID/cgroup" lists a process's cgroup membership.  If legacy cgroup is in use in the system, this file may contain multiple lines, one for each hierarchy.  The entry for cgroup v2 is always in the format "0::$PATH"::

This two-line cgroup file is definitely not conformant with the spec.
I'm guessing some systemd bug in the centos 7 (which contains a rather old systemd) breaks the unified hierachy of the cgroup v2...

/type bug

/severity major

I think this should only happen with operating system bugs. Lower severity to minor.


Fix bug for deseri type error that cause data error

What problem does this PR solve?
Issue Number: close #4851
Problem Summary:
[2022/05/10 15:18:28.483 +08:00] [TRACE] [PageDirectoryFactory.cpp:131] ["PageDirectoryFactory:restoring record [type=1] [page_id=192.352] [ver=<3405,0>] [entry=PageEntry{file: 0, offset: 0x0, size: 0, checksum: 0x0}]"] [thread_id=1]
[2022/05/10 15:18:28.483 +08:00] [TRACE] [PageDirectoryFactory.cpp:131] ["PageDirectoryFactory:restoring record [type=8] [page_id=192.352] [ver=<3407,0>] [entry=PageEntry{file: 0, offset: 0x0, size: 0, checksum: 0x0}]"] [thread_id=1]
[2022/05/10 15:18:28.483 +08:00] [TRACE] [PageDirectoryFactory.cpp:131] ["PageDirectoryFactory:restoring record [type=6] [page_id=192.357] [ver=<3421,0>] [entry=PageEntry{file: 0, offset: 0x0, size: 0, checksum: 0x0}]"] [thread_id=1]

The page_id=192.352 type is PUT_EXTERNAL(1), but it should be VAR_EXTERNAL(7). Because this record has been dumped into log_5_1
Then after restored, PageDirectoryFactory::loadEdit will deal this edit as PUT_EXTERNAL(1) following by DEL(8), it will make that this page_id  being_ref_count be 0. MVCC will clean it in memory.

The main reason why "page_id=192.352 type is 1" is that we should not decode VAR_EXTERNAL(7)  to PUT_EXTERNAL(1) in deserializePutExternalFrom.
What is changed and how it works?

Changed deserializePutExternalFrom
Added a log to check the similar situations.

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/rebuild

/run-all-tests

/merge


Add "apt-get update" for "apt_packages" option

By running "apt-get update" before installing the listed packages, we make sure that the package sources are up-to-date.
This addresses #57.
Please note that I am not too familiar with Python. Perhaps there is a more concise way to write the changes. Please feel free to let me know how it could be improved!

Awesome, thanks @marcluque ! I've made a new release (v0.10.1) so you can start using this straight away


Connections page takes unusually long to load

Julian said it might be the results count taking a bit, which sounded like the likely cause, but I found out later that it's also about the same when doing a search with only a few results so it might be something else. I'm not sure what indexes the table has so maybe the long search time is a separate thing

It's getting the result count having to do a full table scan over a 5-million row table.

Julian said it might be the results count taking a bit, which sounded like the likely cause, but I found out later that it's also about the same when doing a search with only a few results so it might be something else.

The reason fetching result count is expensive is that, in Postgres, fetching result count requires a full table scan. The only index on the table is for user ID, so searching anything else also has to do a full table scan.

You can also use the count estimate postgres has stored in the meta table to get at least a rough estimation of the result count when not filtering.

Realistically we should also just be more serious about removing old connection logs. And we should fix it so people don't spam connections if the server is full, bloating the table.


No heart rate data in list of streams

Why there is no heart rate data from the list of streams you have mentioned in line 41 in empaticae4.py file.
I can see from the official empatica website that the heartrate data is also available in real time.

Hi @UtharaKeerthan, this library was primarily developed for cometrics which has a live heart rate monitor from the E4.  You can see the connection code here.  You can see here that the HR data is displayed on the user interface.  Additionally, you can see here that pyEmpatica does handle and expect HR data.
I believe that the HR data is subscribed to from either the IBI or BVP subscriptions.  Is there Empatica documentation saying you need to explicitly subscribe to that stream? Have you had issues receiving that data stream with pyEmpatica? I noticed that the HR data only updates when it has a good reading, so it is more intermittent than the other data streams.

Thanks for the response. I can compute the HR from the IBI intervals as mentioned in the empatica documentation.


Adding $package_ensure parameter

To be consistent with other puppetlabs puppet modules, I've added a $package_ensure parameter to the class, in lieu of using ensure_packages.

@lieutdan13 thanks for the contribution! I ended up merging something that conflicted with this, so I rebased it and added docs/tests in #58.

Thanks! I should have known better to add the docs and test myself anyway.


Bunch of fixes with admin

Fix some minor Admin/Assessor issues, integration tests.

All good for me :+1:


ci(docker): Test sbom & provenance

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


Range<String.Index> for Token without Trivia

Description
I need a property for TokenSyntax that gives the Range<String.Index> for this token without Trivia.
My goal is to get all tokens ranges when override func visit(_ token: TokenSyntax) -> TokenSyntax is called
I iterate a long string, so the speed is critical.
My code to get it:
//First code (slow)
let tokenStart = code.utf8.index(code.utf8.startIndex, offsetBy: token.positionAfterSkippingLeadingTrivia.utf8Offset)
let tokenEnd = code.utf8.index(code.utf8.startIndex, offsetBy: token.endPositionBeforeTrailingTrivia.utf8Offset)
let tokenRange: Range<String.Index> = tokenStart..<tokenEnd

//Second code (faster but still slow)
let location = token.positionAfterSkippingLeadingTrivia.utf8Offset
let length = token.byteSizeAfterTrimmingTrivia
let start = code.utf8.index(code.utf8.startIndex, offsetBy: location)
let end = code.utf8.index(start, offsetBy: length)
let tokenRange = start..<end

rdar://106505510

I don’t think that’s an API SwiftSyntax should offer by default because SwiftSyntax internal doesn’t actually work on strings but on byte buffers so it can also represent invalid UTF-8, which might occur in a source file that is being parsed. Thus, talking about string indices into a String doesn’t make sense for SwiftSyntax.
What you could try is keeping track of a running index indoor visitor instead of always computing the start and end index based on the source code’s start. Something like the following (written in GitHub so the APIs might look a little different if I’m misremembering them)
var index = sourceCode.utf8.startIndex

override func visit(_ token: TokenSyntax) -> TokenSyntax {
  if token.presence != .present {
    return
  }
  index = sourceCode.utf8.index(index, offsetBy: token.leadingTriviaLength)
  let start = index
  index = sourceCode.utf8.index(index, offsetBy: token.textLength)
  let end = index
  index = sourceCode.utf8.index(index, offsetBy: token.trailingTriviaLength)
  let tokenRange = start..<index
}

Thanks!
Polished code:
var index = sourceCode.utf8.startIndex

override func visit(_ token: TokenSyntax) -> TokenSyntax {
	if token.presence != .present {
		return
	}
	index = sourceCode.utf8.index(index, offsetBy: token.leadingTriviaLength.utf8Length)
	let start = index
	index = sourceCode.utf8.index(index, offsetBy: token.contentLength.utf8Length)
	let end = index
	index = sourceCode.utf8.index(index, offsetBy: token.trailingTriviaLength.utf8Length)
	let tokenRange = start..<end
}

It is faster but it still slow for a long string.
thefunc index(_ i: String.UTF8View.Index, offsetBy n: Int) -> String.UTF8View.Index complexity is:
O(1) if the collection conforms to RandomAccessCollection; otherwise, O(k), where k is the absolute value of distance.
I hope Range<String.Index> will be included in some way.
Since we can get the token text via: token.text, so we supposed to get its Range<String.Index>

It's faster but still slow for a long string.
thefunc index(_ i: String.UTF8View.Index, offsetBy n: Int) -> String.UTF8View.Index complexity is: O(1) if the collection conforms to RandomAccessCollection; otherwise, O(k), where k is the absolute value of distance.

I don’t exactly understand why it’s slow. If index(…) is O(distance) then the entire walk should be in O(length of source buffer) since the addition of all the distances should result in the length of the source string. Maybe there’s a performance problem lurking somewhere else.

I hope Range<String.Index> will be included in some way.
Since we can get the token text via: token.text, so we supposed to get its Range<String.Index>

As I mentioned above, adding Range<String.Index> to the syntax nodes will not be possible since the parser could be started on an arbitrary source buffer consisting of bytes that cannot be represented as a String if it contains invalid UTF-8
https://github.com/apple/swift-syntax/blob/f92f133ddc17f028b9ee4ff6ea67fb2ed3654a35/Sources/SwiftParser/Parser.swift#L147
If you deeply care about performance, an alternative you could try is to @_spi(RawSyntax) import SwiftSyntax and access token.raw.syntaxText. This gives you an unmanaged substring of the original source buffer. Be aware though that SyntaxText is only alive as long as the syntax tree is alive so you need to make sure that the syntax tree from which you retrieved the SytnaxText outlives the SyntaxText.

Thanks!
I can't get the syntaxText.

Ah, sorry, I remembered incorrectly. It’s token.tokenView.rawText.

Thanks!
Can you give me a code example to know how can I use it to get the Range<String.Index> ?

Oh, you can’t, SyntaxText is a byte-based string representation that works on byte offsets whereas String.Index is based on Unicode Characters, so converting between the two has the performance implications you already saw. I just wanted to mention that this is another performant way to get the contents of a token in a syntax tree.

Oh okay, I'm so grateful for your help.

If you think this answers your questions, could you close the issue?


Fix StreamResponse output_length typo.

Just small typo fix

@hellysmile thanks!
Can you add small test for this? E.g. similar to this one.

@rutsky done!

Thanks! LGTM.

Thaks! Merged.


Update bio benchmarks with comparisson

This will add a final WP plot at the end of each of the Bio benchmarks files.
The idea is that for the Catalyst paper, we can just download the figures from the SciMLBenchmakrs docs directly (since they only compare within Catalyst, unlike the main figure where we compare between tools). So I am basically adding proposed paper supplementary figures at the end here.
For each summary plot, I basically pick the best benchmarks across the docs. In some cases I also try and pick the best from each set (unless where a set only have really bad ones).

Seems a commit "Merge pull request #4 from SciML/master" snuck in. I think that was when I tried to ensure that my version of SciMLBecnhmarks (TorkelE/SciMLBenchmarks) was up-to-date with the main repo. Might this become a problem?

Before finalizing this I will need to figure out https://github.com/SciML/DiffEqDevTools.jl/issues/103 as well.
(I know that different markers are currently auto generated. But for clarity I have the same markers for the different solvers in the paper, so would be nice to use the same marker for e.g. QNDF in the other figures as in the WP diagrams)

I think that was when I tried to ensure that my version of SciMLBecnhmarks (TorkelE/SciMLBenchmarks) was up-to-date with the main repo. Might this become a problem?

Yes, this is a problem. You're ignoring my code changes. Please rebase

Thanks, closing this while I try to figure out.


remove unnecessary link from prerequisites

name: :memo: Documentation Fix
about: Fixing a problem in an existing docs page
Checklist

[ ] Is there an existing issue for this PR?

link issue here

[ ] Have the files been linted and formatted?

What docs page needs to be fixed?

Section: Redux Fundamentals, Part 2: Concepts and Data Flow
Page:  this

What is the problem?
What changes does this PR make to fix the problem?

Hi! Appreciate all the links work you've been doing :)  But, this one is important.  The "Fundamentals" tutorial uses hand-written immutable updates with object spreads, so this is still an important prerequisite concept to know.


[UAT] CasperDash version 1.5.0

Is your feature request related to a problem? Please describe.
Audit Fixes: https://docs.google.com/document/d/15bDsEJkEtvNhNBjuR_uDfppQAwKKoHkydKor8S_jSOU/edit

[ ] Replaced the download button with a copy button

[ ] [Bug] Cannot import account. I tried both cases with and w/o

-----BEGIN EC PRIVATE KEY-----
-----END EC PRIVATE KEY-----

[ ] UI BUG
https://github.com/CasperDash/casperdash-client/assets/4486806/6630583e-f8d4-485a-bd4b-53597a2bf9bb


Custom radio, click between text and radio not working

Hi,
On bootrstap 4 custom radio or checkbox, there is 0.5rem of space between the custom control and the text, this lead to a problem when you click on that space, it's not enabling the control.
The .custom-control div wrapping the input and the label has a left padding of 1.5rem.
A better solution would be to set a left padding of 1rem to the wrapping div and add a 0.5rem left padding on the label element itself.
By doing this, the space would still be present between the radio and the text but it would be clickable.
Here you can see a jsfiddle demonstrating it : http://jsfiddle.net/y4up67gk/
Thank you

I've noticed this as well for custom radio, checkbox and the new switch style checkboxes.

This one confused me. I'm unsure why they designed it in this way. To me, it makes more sense to remove the padding-left from the containing .custom-control, and add it to .custom-control-label. Then the svg sits within the padding, and the entire thing is clickable.

We use padding-left here so that all children elements of .custom-control (like the label, form validation message, additional help text, etc) is all easily aligned with the label text. See <
https://jsfiddle.net/pz3xh0uq/> for an example of what I mean.
Without this, we’d have to manually set the padding-left or margin-left of these children elements, or require them to be part of the <label>, which seems unnecessary and, in some cases, invalid. At the very least, it’s problematic for screen reader technologies that would announce everything in the <label>.
I’ve been playing with a new approach to checks/radios for v5 that would merge the native and custom form controls. See https://codepen.io/anon/pen/xmOEvx. This example was intended to explore a few things:

Can we easily customize the check/radio without custom HTML or ::before/::after? Turns out we perhaps can! With the help of appearance: none, we can theoretically style the input as if it were another element.

Can we use flexbox to build the checks/radios? Answer I’ve found is that no, we cannot. Not without adding more markup as we need a row direction for the input and label, but then we need a column direction with the label, validation, help text, etc. This isn’t an issue if no one does this, but these are all common patterns.

Can we make the tap target larger? As mentioned above, not without some trade offs. Moreover, there’s a reason we don’t do a block-level label as the wrapper—the negative space next to any label text would also be clickable. As such, I don’t think we should even be optimizing for this—clicking empty space should be clicking empty space.

Can we get away from position: absolute;? Probably! This demo uses a floated input with some margin-top to vertically align itself with the label’s text baseline.

Could we swap for em units to enable sizing? Yup, just haven’t done it in the demo yet. Once we move to this, we can use font-size utilities or inheritance to change their size.  Thoughts appreciated!

Feels like we’ll never get this 100% right for everyone’s expectations, and that’s okay. We’re trying to build the most flexible, bulletproof, and customizable components. Let me know what you think of the above example and thinking.

Hi,
Thank you for your explanations it makes sense.
To keep the padding on the .custom-control, putting a negative left margin of 0.5rem and a left padding of 0.5rem to the label can do the job : http://jsfiddle.net/2uxbtd1p/
For the click on whitespace, when you have very small label like for example a number selection, it's harder to enable the radio/checkbox, that's why I think it would be a great improvement to do this.
Best regards

The explanation makes sense to me too. In fact, I don't think we should even be talking about this 'problem' anymore, because there isn't one. The checkbox and the label are different elements, the whitespace is just that - whitespace. I now feel that if you need the whitespace clickable due to small labels, then you can simply add the 5 or so lines to your CSS to get it to work.
I'm happy with how it works currently, and I think the appearance method, with absolute position for alignment, is probably the best way to go.
Thanks again @mdo

Closing out per my comment above. Probably not going to do the padding/margin hack as that feels largely unnecessary and potentially complicates (customizing) gutter spacing.


declare the supported k8s version

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


Background transfer sample doesn't work on HP Elitepad 1000 G2

Works fine on my Windows 10 x64 (build 10240) laptop.
But when running on HP ElitePad1000 (upgrated to windows 10), download hangs on following line:
await download.StartAsync().AsTask(cts.Token, progressCallback);
in file Scenario1_Download.xaml.cs

This issue is not actionable as we cannot reproduce. Also, this is not an issue with the sample per se.
If this issue is continuing to happen, you can open a support case with our Developer Support team:

Visit http://aka.ms/storesupport and sign-in with your Microsoft account
Under the “App Development” section choose "Windows 10 universal app development", then choose the appropriate problem type and category to get engaged through a formal support channel.


Automatically determine if a player's skin is slim ("ALEX") or classic ("STEVE")

Could use mojang API, see UUID -> Profile + Skin/Cape (wiki.vg) (includes whether or not a player's skin is slim).
Might also be useful to pull the skin from there instead of crafatar (as we do right now), although it would limit player info retrieval to once per player per minute.

This should do:
import java.util.UUID;

public boolean isSlim(String uuid) {
  return (UUID.fromString(uuid).hashCode() & 1) != 0;
}

This should do:
import java.util.UUID;

public boolean isSlim(String uuid) {
  return (UUID.fromString(uuid).hashCode() & 1) != 0;
}


Overseas place of birth policy

For discussion (to be read in conjunction with current instructions): abbreviate all as OVF?  Continue OVB? TWYS (search using emendation list so Gibraltar finds Gib., USA finds United States of America, etc.?  a reason for abandoning OVB: https://en.wikipedia.org/wiki/History_of_British_nationality_law#Early_English_and_British_nationality_law shows it is very difficult to judge whether someone is, or isn't a British subject.

We also have to take into account that some enumerators do actually enumerate entries as British Subject irrespective of where they were born.
We also get “Naturalised British Subject” as well, and they could have been anywhere in the world, Timbuctoo’ for example.
So not all those that say they were born in France, USA, Germany etc., are given an OVF as the Chapman Code, if the enumerator has added the codicil “British Subject” or  “Naturalised British Subject” etc.
Parents could have been working there, visiting etc., but registered the ‘Child’ in the UK on their return!
So it is not safe to assume that all those born “outside of the Commonwealth Countries list” are classed as OVF.
They could therefore be a “British Subject” or  “Naturalised British Subject” and therefore be OVB irrespective of where they were born!
A lot more to think about!
From: PatReynolds<EMAIL_ADDRESS>Sent: 21 June 2018 14:48
To: FreeUKGen/FreeCENMigration<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [FreeUKGen/FreeCENMigration] Overseas place of birth policy (#489)
For discussion: abbreviate all as OVF? Continue OVB? TWYS (search using emendation list so Gibraltar finds Gib., USA finds United States of America, etc.?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/489 , or mute the thread https://github.com/notifications/unsubscribe-auth/ANC91haVEsYQsWl87YCKFiKxvKQxN6njks5t-6QpgaJpZM4UyGzR .

Yes, irrespective of the quality of the information, we still need a clear way to indicate that the person has been described as British, in addition to just stating their place of birth. Doing genealogy involves all kinds of judgement calls by the researcher.

Comments posted on a different story, moved here:
Pat 11/7/2018 : Possibly remove OVB except for where person is stated to be British? Possibly remove OVB except for where country is known to be a British colony/dominion at the time of birth - or leave this for a Validation task?
Brenda replied same day: I would say that 90%+ of OVB are transcribed because the Enumerator added
the words British Subject.
Therefore the Transcriber would transcribe it AS IS.
The remainder would/should be picked up by the Validator.
Removing the OVB option would go against what we tell them to Transcribe AS
IS.
Leave it for the Validation Process is my preference.

We have the policy now - closing.


[WIP] Move docs to ReadTheDocs

Coverage remained the same at 73.802% when pulling 9a30d7251f4fad6f43a255df1abf6d61c9576bf6 on docs-to-readthedocs into ff5f295fffb52dfabc8d86de6cd66aeb5e8f2efb on master.

Coverage remained the same at 73.802% when pulling 9491047ddb2d6e6605b195d46bb85173d5b7b211 on docs-to-readthedocs into ff5f295fffb52dfabc8d86de6cd66aeb5e8f2efb on master.

Coverage remained the same at 73.802% when pulling 9491047ddb2d6e6605b195d46bb85173d5b7b211 on docs-to-readthedocs into ff5f295fffb52dfabc8d86de6cd66aeb5e8f2efb on master.

Coverage remained the same at 73.802% when pulling 9491047ddb2d6e6605b195d46bb85173d5b7b211 on docs-to-readthedocs into ff5f295fffb52dfabc8d86de6cd66aeb5e8f2efb on master.

Coverage remained the same at 73.802% when pulling 9491047ddb2d6e6605b195d46bb85173d5b7b211 on docs-to-readthedocs into ff5f295fffb52dfabc8d86de6cd66aeb5e8f2efb on master.

Coverage remained the same at 73.802% when pulling 9491047ddb2d6e6605b195d46bb85173d5b7b211 on docs-to-readthedocs into ff5f295fffb52dfabc8d86de6cd66aeb5e8f2efb on master.

Jason notes that a good place to put the remaining section would be a warning in the install docs.

Coverage increased (+0.02%) to 73.817% when pulling b0f226c1c41cb967ae835a1de82d1b40d77d6c8c on docs-to-readthedocs into ff5f295fffb52dfabc8d86de6cd66aeb5e8f2efb on master.

Coverage decreased (-0.1%) to 73.701% when pulling 18d07576eef8140768ff4e77ea39090729b28c0a on docs-to-readthedocs into ff5f295fffb52dfabc8d86de6cd66aeb5e8f2efb on master.

Ref #1276


DDDify persistency provider abstraction and Couchbase implementation

Thank you.
Closed with https://github.com/CanerPatir/DDDify/commit/7b96d3033b9f9d1044bc4a5900b0c660006fb741


output: do nothing in output_set_image if backend has no renderer

This is useful when using the noop backend for example.

Thanks!


Changelog and version number process

While a good idea in theory, our current process with changelog entries and semantic versioning doesn't really hold up to the reality of our development methodology.
For example, I currently have four open pull requests (#140, #141, #142, and #143), and cannot include changelog or version information in any of them, since they would conflict with each other.
Come up with a better way of doing things.

It's probably only the maintainer who can make release decisions, so external contributors should not include such in their pull requests.
When merging #142, i included ChangeLog and version.rb update in the merge commit. Does this seem like a reasonable way to manage these things?

Hmm. I've never seen Git do that, but it's obvious that it should be possible. Did you have to do anything weird to accomplish that? Any changes to git-ghpr?
I'm not sure if it's reasonable, but it may very well be the most reasonable. Having the version bump in its own commit makes it visible in a way that embedding it in a merge doesn't. On the other hand, setting a version tag will mitigate that somewhat, and in a philosophical sense having the functionality merge and version bump atomically in the same commit is nice.
I say we try it for now. We probably want the submitter to state in the pull request what kind of version bump is required, and a suggested changelog message. Could you change the README?

All I did was git add the edited files before committing the merge, but I probably should've added a line about the version bump to the commit message to make it look nicer.


project_history promble

I added the project path to the project_history, but after saving, it was reset, and the added path disappeared. How can I solve this problem

Hi there,
The suggestion is to use the default cache folder instead of a customized path, as the customized path seemed to have caused the "project_history" file to become blank after saving.
Going to editor, run command " :echo stdpath("cache") " with the output as a default path of cache. So now just edit the project_history inside this path. That works for me.


Have TQDMProgressBar work in evaluate mode as well

Describe the feature and the current behavior/state.
Right now, the TQDMProgressBar callback can only be used when training a model.
When evaluating nothing happens. And even if I try to "cheat" by manually setting the on_test_batch_(begin|end) methods, it fails with an error about mode.
Relevant information

Are you willing to contribute it (yes/no): don't know
Are you willing to maintain it going forward? (yes/no): no
Is there a relevant academic paper? (if so, where): no
Is there already an implementation in another framework? (if so, where): no
Was it part of tf.contrib? (if so, where): no

Which API type would this fall under (layer, metric, optimizer, etc.)
callbacks
Who will benefit with this feature?
everyone using tqdm to evaluate in a notebook: I want something fancier than the default evaluate progressbar
Any other info.
It failed in both tf2.2 and tf2.1
You can find the colab gist here.

I don't think it would be hard to do because keras exposes methods for that: e.g. https://www.tensorflow.org/api_docs/python/tf/keras/callbacks/Callback#on_test_batch_begin . Contributions welcome!

@gabrieldemarmiesse , please review PR #38447

As a small note, I believe the TQDM callback should have a cleanup as we removed the "samples" mode and a refactoring would be welcome before a PR to add this feature. There is definitly some dead code/renaming to do.

Yep. Got it. Let me do that too.

@gabrieldemarmiesse , I have created PR for adding methods on_test_batch_begin() and on_test_batch_end(). I have also looked at removing unused code related to samples mode. I have found one clue here. But since, I am not aware about past implementation and changes, I couldn't do on my own. Please give me hints or point me the direction where I can find one.

ops, sorry I have forgot about evaluate() when I wrote it and thanks @ashutosh1919 so much for the contribution!

@zaccharieramzi and @gabrieldemarmiesse , please review PR. Creates as below.


[AIRFLOW-2809] Fix security issue regarding Flask SECRET_KEY

JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-2809
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a JIRA issue.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

Background
Currently there is a configuration item secret_key in the configuration .cfg file, with a default value "temporary_key".
Issue
Most admins would ignore it and just use the default value "temporary_key". However, this may be very dangerous. User may modify the cookie if they try the default SECRET_KEY while the admin didn't change it.
**In Flask documentation, it's suggested to have a SECRET_KEY which is as random as possible (http://flask.pocoo.org/docs/1.0/quickstart/ ). **
My Proposal
If Admin explicitly changed the SECRET_KEY in .cfg file, we use this SECRET_KEY given by Admin.
If the default SECRET_KEY is not changed in .cfg file, randomly generate SECRET_KEY. Meanwhile, print INFO to remind that a randomly generated SECRET_KEY is used.
This solution will not affect user experience at all.
Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

@XD-DENG Can you prepend the squash the commits and prepend the message with the Jira ticket [AIRFLOW-2809] ...
This looks like a good idea to me, maybe check what the other committers think about this.
cc @kaxil @bolkedebruin @jgao54

Thanks @Fokko . Have squashed the commits and prepended the JIRA ticket label as suggested.

Codecov Report

Merging #3651 into master will increase coverage by <.01%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master    #3651      +/-   ##
==========================================
+ Coverage    77.5%   77.51%   +<.01%
==========================================
  Files         205      205
  Lines       15747    15751       +4
==========================================
+ Hits        12205    12209       +4
  Misses       3542     3542

Impacted Files
Coverage Δ

airflow/www/app.py
99.01% <83.33%> (-0.99%)
:arrow_down:

airflow/models.py
88.58% <0%> (+0.04%)
: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 449a7fd...c3f231b. Read the comment docs.


Offset jump step size

Hi,
Is it possible with current syntax to search for a string in a loop at specific offset and then to jump given number of bytes and search there again.
strings:

	$a = { 50 45 } // PE

condition:

	$a at 128
	or $a at 136
	or $a at 144
	or $a at 152

and so on. If I could indicate that jump is 8 bytes I could specify to jump 20 times (1 .. 20) and search at those offsets only.
With big files this could save a lot of time.
Something like this
for any i in (1 .. 20):
	$a at 128
	jump + 8

Thanks!

I tried to do it but always getting syntax error.
for any i in (1 .. 20):
    $a in (128 .. 128 + i * 8)

can i be multipled is the question?

In order to do what you want to achieve, you can do the following:
	condition:
		for any i in (1 .. 20) : (
			$a at (128 + i * 8)
		)

Got it, thanks @r0ny123
So brackets were needed after all.

No, you can do that without brackets.

This won't work
for any i in (1 .. 20):
    $a in (128 .. 128 + i * 8)

but this will
for any i in (1 .. 20):
    ($a in (128 .. 128 + i * 8))

Same is with at condition

This won't work
for any i in (1 .. 20):
    $a in (128 .. 128 + i * 8)

but this will
for any i in (1 .. 20):
    ($a in (128 .. 128 + i * 8))

Same is with at condition

With your rule, yes. It won't work without brackets, because that is not a valid yara syntax. I was talking about mine.

Got it, thanks 👍


Search the multivalue control

Hi there,
This control have autocomplete?
It's possible to search a value in the list of values?
The problem is, i'm trying to use this component to add people, and i would like to filter does values.
Thanks

@mohitbagra  and @ostreifel  Thoughts?

We can use something like Fabric's tag picker. It supports multiple values and search functionality.
https://developer.microsoft.com/en-us/fabric#/components/pickers

The only downside is that this control doesnt have a "open dropdown" icon to see all available values. Users will see a dropdown with filtered resutls when they type something.

I think we should do as the contributions graph does for repos

This would alleviate #56 as well

Would also fix #35

These changes are now live. Please file additional issues if you encounter any problems.


fix(content): Tutorial island giant rat stats

I'm pretty sure giant rats on tutorial island had 3 HP and lower stats. The stats that were listed were from regular level 3 giant rats, not tutorial island ones
Similar time proof: https://www.youtube.com/watch?v=Quwc_TfPW8Q
https://oldschool.runescape.wiki/w/Giant_rat_(Tutorial_Island)

Those stats seem to match up better for their era accurate vislevel too


Add "Create Swift Project" to explorer view when no project loaded

The Java extensions does this

You can do it using viewsWelcome contribution point
https://code.visualstudio.com/api/references/contribution-points#contributes.viewsWelcome

I can take this one.


Plugins using a platform dependency can't be published

Expected Behavior
It should be possible to deploy a gradle plugin to the Gradle Plugin Portal, which uses a platform dependency
Current Behavior
The publishing fails with the following exception:
> Task :okhttp-plugin:publishPlugins FAILED
Could not reuse POM from pluginMaven publication because mavenCoordinates() was used

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':okhttp-plugin:publishPlugins'.
> No version found for com.squareup.okhttp3:okhttp on pom generation.

* Try:
Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Exception is:
org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':okhttp-plugin:publishPlugins'.
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.lambda$executeIfValid$1(ExecuteActionsTaskExecuter.java:205)
        at org.gradle.internal.Try$Failure.ifSuccessfulOrElse(Try.java:263)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeIfValid(ExecuteActionsTaskExecuter.java:203)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.execute(ExecuteActionsTaskExecuter.java:184)
        at org.gradle.api.internal.tasks.execution.CleanupStaleOutputsExecuter.execute(CleanupStaleOutputsExecuter.java:114)
        at org.gradle.api.internal.tasks.execution.FinalizePropertiesTaskExecuter.execute(FinalizePropertiesTaskExecuter.java:46)
        at org.gradle.api.internal.tasks.execution.ResolveTaskExecutionModeExecuter.execute(ResolveTaskExecutionModeExecuter.java:62)
        at org.gradle.api.internal.tasks.execution.SkipTaskWithNoActionsExecuter.execute(SkipTaskWithNoActionsExecuter.java:57)
        at org.gradle.api.internal.tasks.execution.SkipOnlyIfTaskExecuter.execute(SkipOnlyIfTaskExecuter.java:56)
        at org.gradle.api.internal.tasks.execution.CatchExceptionTaskExecuter.execute(CatchExceptionTaskExecuter.java:36)
        at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.executeTask(EventFiringTaskExecuter.java:77)
        at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.call(EventFiringTaskExecuter.java:55)
        at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter$1.call(EventFiringTaskExecuter.java:52)
        at org.gradle.internal.operations.DefaultBuildOperationExecutor$CallableBuildOperationWorker.execute(DefaultBuildOperationExecutor.java:416)
        at org.gradle.internal.operations.DefaultBuildOperationExecutor$CallableBuildOperationWorker.execute(DefaultBuildOperationExecutor.java:406)
        at org.gradle.internal.operations.DefaultBuildOperationExecutor$1.execute(DefaultBuildOperationExecutor.java:165)
        at org.gradle.internal.operations.DefaultBuildOperationExecutor.execute(DefaultBuildOperationExecutor.java:250)
        at org.gradle.internal.operations.DefaultBuildOperationExecutor.execute(DefaultBuildOperationExecutor.java:158)
        at org.gradle.internal.operations.DefaultBuildOperationExecutor.call(DefaultBuildOperationExecutor.java:102)
        at org.gradle.internal.operations.DelegatingBuildOperationExecutor.call(DelegatingBuildOperationExecutor.java:36)
        at org.gradle.api.internal.tasks.execution.EventFiringTaskExecuter.execute(EventFiringTaskExecuter.java:52)
        at org.gradle.execution.plan.LocalTaskNodeExecutor.execute(LocalTaskNodeExecutor.java:41)
        at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$InvokeNodeExecutorsAction.execute(DefaultTaskExecutionGraph.java:372)
        at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$InvokeNodeExecutorsAction.execute(DefaultTaskExecutionGraph.java:359)
        at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$BuildOperationAwareExecutionAction.execute(DefaultTaskExecutionGraph.java:352)
        at org.gradle.execution.taskgraph.DefaultTaskExecutionGraph$BuildOperationAwareExecutionAction.execute(DefaultTaskExecutionGraph.java:338)
        at org.gradle.execution.plan.DefaultPlanExecutor$ExecutorWorker.lambda$run$0(DefaultPlanExecutor.java:127)
        at org.gradle.execution.plan.DefaultPlanExecutor$ExecutorWorker.execute(DefaultPlanExecutor.java:191)
        at org.gradle.execution.plan.DefaultPlanExecutor$ExecutorWorker.executeNextNode(DefaultPlanExecutor.java:182)
        at org.gradle.execution.plan.DefaultPlanExecutor$ExecutorWorker.run(DefaultPlanExecutor.java:124)
        at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:64)
        at org.gradle.internal.concurrent.ManagedExecutorImpl$1.run(ManagedExecutorImpl.java:48)
        at org.gradle.internal.concurrent.ThreadFactoryImpl$ManagedThreadRunnable.run(ThreadFactoryImpl.java:56)
Caused by: com.gradle.publish.MissingDependencyVersionException: No version found for com.squareup.okhttp3:okhttp on pom generation.
        at com.gradle.publish.PomWriter.addDependency(PomWriter.java:58)
        at com.gradle.publish.PomWriter.createPomDocument(PomWriter.java:44)
        at com.gradle.publish.PomWriter.writePom(PomWriter.java:30)
        at com.gradle.publish.PublishTask.generatePom(PublishTask.java:168)
        at com.gradle.publish.PublishTask.ensurePomIsAvailable(PublishTask.java:154)
        at com.gradle.publish.PublishTask.publish(PublishTask.java:70)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at org.gradle.internal.reflect.JavaMethod.invoke(JavaMethod.java:104)
        at org.gradle.api.internal.project.taskfactory.StandardTaskAction.doExecute(StandardTaskAction.java:49)
        at org.gradle.api.internal.project.taskfactory.StandardTaskAction.execute(StandardTaskAction.java:42)
        at org.gradle.api.internal.project.taskfactory.StandardTaskAction.execute(StandardTaskAction.java:28)
        at org.gradle.api.internal.AbstractTask$TaskActionWrapper.execute(AbstractTask.java:727)
        at org.gradle.api.internal.AbstractTask$TaskActionWrapper.execute(AbstractTask.java:694)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter$3.run(ExecuteActionsTaskExecuter.java:568)
        at org.gradle.internal.operations.DefaultBuildOperationExecutor$RunnableBuildOperationWorker.execute(DefaultBuildOperationExecutor.java:402)
        at org.gradle.internal.operations.DefaultBuildOperationExecutor$RunnableBuildOperationWorker.execute(DefaultBuildOperationExecutor.java:394)
        at org.gradle.internal.operations.DefaultBuildOperationExecutor$1.execute(DefaultBuildOperationExecutor.java:165)
        at org.gradle.internal.operations.DefaultBuildOperationExecutor.execute(DefaultBuildOperationExecutor.java:250)
        at org.gradle.internal.operations.DefaultBuildOperationExecutor.execute(DefaultBuildOperationExecutor.java:158)
        at org.gradle.internal.operations.DefaultBuildOperationExecutor.run(DefaultBuildOperationExecutor.java:92)
        at org.gradle.internal.operations.DelegatingBuildOperationExecutor.run(DelegatingBuildOperationExecutor.java:31)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeAction(ExecuteActionsTaskExecuter.java:553)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeActions(ExecuteActionsTaskExecuter.java:536)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.access$300(ExecuteActionsTaskExecuter.java:109)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter$TaskExecution.executeWithPreviousOutputFiles(ExecuteActionsTaskExecuter.java:276)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter$TaskExecution.execute(ExecuteActionsTaskExecuter.java:265)
        at org.gradle.internal.execution.steps.ExecuteStep.lambda$execute$1(ExecuteStep.java:33)
        at org.gradle.internal.execution.steps.ExecuteStep.execute(ExecuteStep.java:33)
        at org.gradle.internal.execution.steps.ExecuteStep.execute(ExecuteStep.java:26)
        at org.gradle.internal.execution.steps.CleanupOutputsStep.execute(CleanupOutputsStep.java:67)
        at org.gradle.internal.execution.steps.CleanupOutputsStep.execute(CleanupOutputsStep.java:36)
        at org.gradle.internal.execution.steps.ResolveInputChangesStep.execute(ResolveInputChangesStep.java:49)
        at org.gradle.internal.execution.steps.ResolveInputChangesStep.execute(ResolveInputChangesStep.java:34)
        at org.gradle.internal.execution.steps.CancelExecutionStep.execute(CancelExecutionStep.java:43)
        at org.gradle.internal.execution.steps.TimeoutStep.executeWithoutTimeout(TimeoutStep.java:73)
        at org.gradle.internal.execution.steps.TimeoutStep.execute(TimeoutStep.java:54)
        at org.gradle.internal.execution.steps.CatchExceptionStep.execute(CatchExceptionStep.java:34)
        at org.gradle.internal.execution.steps.CreateOutputsStep.execute(CreateOutputsStep.java:44)
        at org.gradle.internal.execution.steps.SnapshotOutputsStep.execute(SnapshotOutputsStep.java:54)
        at org.gradle.internal.execution.steps.SnapshotOutputsStep.execute(SnapshotOutputsStep.java:38)
        at org.gradle.internal.execution.steps.BroadcastChangingOutputsStep.execute(BroadcastChangingOutputsStep.java:49)
        at org.gradle.internal.execution.steps.CacheStep.executeWithoutCache(CacheStep.java:153)
        at org.gradle.internal.execution.steps.CacheStep.execute(CacheStep.java:67)
        at org.gradle.internal.execution.steps.CacheStep.execute(CacheStep.java:41)
        at org.gradle.internal.execution.steps.StoreExecutionStateStep.execute(StoreExecutionStateStep.java:44)
        at org.gradle.internal.execution.steps.StoreExecutionStateStep.execute(StoreExecutionStateStep.java:33)
        at org.gradle.internal.execution.steps.RecordOutputsStep.execute(RecordOutputsStep.java:38)
        at org.gradle.internal.execution.steps.RecordOutputsStep.execute(RecordOutputsStep.java:24)
        at org.gradle.internal.execution.steps.SkipUpToDateStep.executeBecause(SkipUpToDateStep.java:92)
        at org.gradle.internal.execution.steps.SkipUpToDateStep.lambda$execute$0(SkipUpToDateStep.java:85)
        at org.gradle.internal.execution.steps.SkipUpToDateStep.execute(SkipUpToDateStep.java:55)
        at org.gradle.internal.execution.steps.SkipUpToDateStep.execute(SkipUpToDateStep.java:39)
        at org.gradle.internal.execution.steps.ResolveChangesStep.execute(ResolveChangesStep.java:76)
        at org.gradle.internal.execution.steps.ResolveChangesStep.execute(ResolveChangesStep.java:37)
        at org.gradle.internal.execution.steps.legacy.MarkSnapshottingInputsFinishedStep.execute(MarkSnapshottingInputsFinishedStep.java:36)
        at org.gradle.internal.execution.steps.legacy.MarkSnapshottingInputsFinishedStep.execute(MarkSnapshottingInputsFinishedStep.java:26)
        at org.gradle.internal.execution.steps.ResolveCachingStateStep.execute(ResolveCachingStateStep.java:94)
        at org.gradle.internal.execution.steps.ResolveCachingStateStep.execute(ResolveCachingStateStep.java:49)
        at org.gradle.internal.execution.steps.CaptureStateBeforeExecutionStep.execute(CaptureStateBeforeExecutionStep.java:79)
        at org.gradle.internal.execution.steps.CaptureStateBeforeExecutionStep.execute(CaptureStateBeforeExecutionStep.java:53)
        at org.gradle.internal.execution.steps.ValidateStep.execute(ValidateStep.java:74)
        at org.gradle.internal.execution.steps.SkipEmptyWorkStep.lambda$execute$2(SkipEmptyWorkStep.java:78)
        at org.gradle.internal.execution.steps.SkipEmptyWorkStep.execute(SkipEmptyWorkStep.java:78)
        at org.gradle.internal.execution.steps.SkipEmptyWorkStep.execute(SkipEmptyWorkStep.java:34)
        at org.gradle.internal.execution.steps.legacy.MarkSnapshottingInputsStartedStep.execute(MarkSnapshottingInputsStartedStep.java:39)
        at org.gradle.internal.execution.steps.LoadExecutionStateStep.execute(LoadExecutionStateStep.java:40)
        at org.gradle.internal.execution.steps.LoadExecutionStateStep.execute(LoadExecutionStateStep.java:28)
        at org.gradle.internal.execution.impl.DefaultWorkExecutor.execute(DefaultWorkExecutor.java:33)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeIfValid(ExecuteActionsTaskExecuter.java:192)
        ... 30 more

Context
I've updated the dependencies section from one of my gradle plugins to use a platform dependency, so I can omit some version numbers in my other dependency declaration.
With this change, it is not possible anymore to deploy the plugin to the Gradle Plugin Portal
https://travis-ci.org/freefair/gradle-plugins/jobs/654152950
Steps to Reproduce
Execute ./gradlew :okhttp-plugin:publishPlugins in https://github.com/freefair/gradle-plugins/tree/5.0.0-rc4
Your Environment

Build scan URL: https://scans.gradle.com/s/tgdf2ncqcfy3i

Thanks for bringing this up @larsgrefer.
Unfortunately 😞, publishing Gradle Module Metadata is not yet supported for plugins. For full platform dependency support, that would be required.  We will work on that soon and then also get back to this bug report.
But you can still use platforms today.
First, you should remove this from your build script (you don't need it, project.group is used by default):
mavenCoordinates {
  groupId = project.group
}

Then the warning you are seeing about mavenCoordinates will go away and the POM generated by the maven-publish plugin will be used.
Then you can publish with resolved versions by adding the following to your publications.withType(MavenPublication) { } block:
versionMapping {
  usage('java-api') {
    fromResolutionResult()
  }
  usage('java-runtime') {
    fromResolutionResult()
  }
}

Then the pom should have the versions, without the need to publish the platform. The POM will contain an (invalid) dependencyManagement entry to the platform. But since this is not used by any tool automatically, it should not hurt.
To make it completely right you can follow this pattern to put the platform on a separate Configuration that is not published. Then you won't get the dependencyManagement entry:
https://github.com/gradle/gradle/issues/10861#issuecomment-576562961

@jjohannes Thanks for the fast and detailed response.
Just removing
mavenCoordinates {
  groupId = project.group
}

fixed the upload issue.

Closing, as this is not a bug but rather badly documented.
Improving documentation around plugin development is tracked separately (https://github.com/gradle/gradle-private/issues/688,  https://github.com/gradle/gradle-private/issues/913)

Unfortunately 😞, publishing Gradle Module Metadata is not yet supported for plugins.

Is there any ticket for this?

Good point @pmaedel. We only tracked this internally, but we should have a public issue. I created one: https://github.com/gradle/gradle/issues/15992
The current plan is to tackle this during the Gradle 7.0 release which we are working on right now.
Also to revise my earlier comment: It's really a bit weird, but if you publish the plugin to a "normal" repository (like Maven Central), using the publishing mechanism of "maven-publish", it already works. (see also: https://docs.gradle.org/current/userguide/publishing_gradle_plugins.html#custom-plugin-repositories)
Of course that is not a good solution long term, because only the portal is added as source for plugins by default in Gradle. Hope we'll have this fixed really soon.

this still happens with 7.0
implementation(platform("org.http4k:http4k-bom:<IP_ADDRESS>"))
implementation("org.http4k:http4k-core")
implementation("org.http4k:http4k-server-netty")
implementation("org.http4k:http4k-client-apache")

If you use the latest version of the plugin - id("com.gradle.plugin-publish") version "0.14.0" - module metadata is published that should including platform dependencies.
What exactly is not working for you @elect86?

That implementations I quoted earlier will trigger the same error whenever I try to publish it

Execution failed for task ':plugin:publishPlugins'.
No version found for org.http4k:http4k-core on pom generation.

If I manually set the version for each of them, it runs fine
//    implementation(platform("org.http4k:http4k-bom:<IP_ADDRESS>"))
implementation("org.http4k:http4k-core:<IP_ADDRESS>")
implementation("org.http4k:http4k-server-netty:<IP_ADDRESS>")
implementation("org.http4k:http4k-client-apache:<IP_ADDRESS>")

@elect86 the (legacy) code that throws the error should not run. Please check:

Apply the maven-publish plugin (in addition to com.gradle.plugin-publish)
Do not use mavenCoordinates { ... } (inside pluginBundle { ...
Glad to hear it works.

I'd prefer the latter
my 2c


Fix two issues caused by the escape fix

For #14212, #14216
Debugger problem was caused by text still being an array.
Latex problem was us forcing the mime type to text/html when there's actually a better (text/latex in this case) to display. It also seems the nteract text transform handles escaping itself so we don't need to do the escaping for text/plain

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[x] Appropriate comments and documentation strings in the code.
[x] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #14228 into main will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main   #14228      +/-   ##
==========================================
- Coverage   59.95%   59.93%   -0.03%
==========================================
  Files         706      706
  Lines       39185    39185
  Branches     5683     5683
==========================================
- Hits        23495    23486       -9
- Misses      14453    14459       +6
- Partials     1237     1240       +3

Impacted Files
Coverage Δ

...very/locators/services/virtualenvwrapperLocator.ts
76.92% <0.00%> (-15.39%)
:arrow_down:

src/client/common/utils/platform.ts
60.00% <0.00%> (-12.00%)
:arrow_down:

.../pythonEnvironments/common/externalDependencies.ts
57.57% <0.00%> (-3.04%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0.00%> (-0.73%)
:arrow_down:

Continue to review full report at Codecov.

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


Add CloudCannon CMS support

Is your feature request related to a problem? Please describe.
Currently Hinode supports direct editing of markdown content only. Although this is adequate for most developers, it is less friendly for less tech-oriented users.
Describe the solution you'd like
Visual Content Managements Systems (CMSes) provide a more intuitive, user-friendly approach for content management. In general, so-called headless CMSs are either Git-based or API-driven. In a Git-based approach, the content still resides within the same repository and does not require an additional back-end or database.
CloudCannon is one of the more popular and robust Git-based CMSes. Ideally, Hinode would offer out-of-the-box integration with the CMS. The integration should include a basic configuration, documentation, and support for custom shortcodes.
Describe alternatives you've considered
Two other popular Git-based CMSes compatible with Hugo are Decap CMS and Tina CMS. At the time of writing, Decap CMS is being transferred to a new owner and still requires significant code changes (the latest release is from April 2022). Although promising, Tina CMS is still a relatively new product that appears less robust compared to CloudCannon. However, additional support for these alternative CMSes could be considered later.

That would be really interesting. The only reason I see to adding Tina or Decap next to CloudCannon is that they are free or have a free plan :)

Yes, I agree - it's a pity CloudCannon does not have a free plan (for public repositories).