New branchs for testings

Hi everybody
@mariuz what do you think about making an new branch specific for testings and new features?
For example #69 and the new resources for Fb3?
It would help keeping a more stable for the majority of users, and would make easier the deploy of testing code
For now, just an "Testing" branch would be OK, and maybe the new features for FB3 would be a part of

yes you can create a new branch for testing new features


Refactor benchmark setup and enhance CSV generation logging

This refactor splits the benchmark setup into two separate jobs for better handling of individual configurations: one for Docker-based benchmarks and another for PHP-based benchmarks. The jobs now support testing with varying numbers of columns. In addition, the CSV generation process now includes better-riched logs for easier debugging.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2499

Relevant Lines:
2576

💛  - Coveralls


Add contains circle to math.Rectangle

Hi
Added contains method to math.Rectangle to check if a circle is contained in the Rectangle
It might be helpful
Cumps

Just a couple of comments.

I would include equality to the comparisons i.e. <=. I can see it follows the same logic that contains(Rectangle) which doesn't and I would also modify because contains(Vector2) does. I believe this is inconsistent.

It's minor but imo it is clearer to define
xmin = circle.x - circle.radius;
xmax = circle.x + circle.radius;

instead of
xmin = circle.x - circle.radius;
xmax = xmin + 2f * circle.radius;

@obigu Thanks for the feedback.
Agree, its better as you suggested, it's cleaner, and it removes a completely unnecessary multiplication.
I had already noticed that, and i would prefer to have <= comparison on all three, i changed to <= on this one.

This might be better suited in the Intersector class

@xoppa Since all the other contains methods are inside the Rectangle class probably most people are going to look there first for this one as well (and maybe never look for it inside the Intersector class),
Also theoretically it's not an intersection verification, since its only true if the circle is fully contained in the rectangle

@xoppa True, thanks, since xmax, xmin, ymax and ymin were only used once i removed them as well.

Thanks


Tracking issues of RFC-4382: Range based Read

RFC: https://github.com/apache/opendal/pull/4382

Shall be resolved by https://github.com/apache/opendal/pull/4381? And perhaps some follow-ups at #4452.

Yes, should have closed.


jupyter-viewer-xblock

https://github.com/ibleducation/jupyter-viewer-xblock

Thanks!
On Wednesday, April 25, 2018, José Carlos García<EMAIL_ADDRESS>wrote:

Thank you! 🍰
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/quobit/awesome-python-in-education/issues/12#issuecomment-384439086,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADGy9WfjVhMACNLl5qQb4n3AaQTZkAJks5tsOmDgaJpZM4Tj6ar
.


Add MultivariateNormalDiag distribution.

This is an extremely common special case of the MultivariateNormal distribution due to its efficient sampling and log-probability computations.

I see thorough work here, I really appreciate the thorough tests, but I'm unsure about accepting this pr. If, for any reason rand makes a change where we'd need to modify Normal and MultivariateNormal, then we'd have more places to make changes (It makes me think I should not deeply into how we've implemented MultivariateNormal).
As I see it, I believe that your feature set can wrap distribution::Normal so that the only explicit reliance on the fact that this describes uncorrelated, normal or both  would be:

entropy, additive for uncorrelated
variance is defined for each and uncorrelated means there's no covariance to consider,

please let me know if this would not be correct or if I'm missing something important.
But I see both those items being a result of being independently and identically distributed (i.i.d). Perhaps there's an ergonomic solution to cover i.i.d variables from distribution D?
Note: An ergonomic way to support different distributions from uncorrelated variables does not yet seem worthwhile to me, but I would listen to input here as well.

I see thorough work here, I really appreciate the thorough tests, but I'm unsure about accepting this pr. If, for any reason rand makes a change where we'd need to modify Normal and MultivariateNormal, then we'd have more places to make changes (It makes me think I should not deeply into how we've implemented MultivariateNormal).

That seems highly unlikely to happen, given the widespread usage of rand.
Usually the libraries reduce the duplication by abstracting away the square root of the covariance matrix (be it a cholesky factor or the diagonal scale) (e.g. MultivariateNormalLinearOperator from TensorFlow Probability and the Covariance from scipy. rv crate provides a few covariance constructors on their MVN although they don't take advantage of the speed benefits. That certainly could be an option here, but it'd have to be pretty adhoc since nalgebra gives us no help here (they don't have a type that subsumes linear operators like this). Something like fn new(mu: DVector<f64>, cov: impl Covariance) could probably be made to work.
A wrapper distribution over independent distributions is a pretty common component of statistics toolkits (e.g. Independent from TensorFlow Probability, ProductDistribution from rv), but I don't think Rust is a flexible-enough language to provide an idiomatic or efficient-enough API for it. The approach taken by rv is particularly unfortunate as it forces the construction of the components onto the user, rather that accepting vector-valued parameters of the component distributions. I don't think it's the right approach if efficiency is at all a goal for statrs.

Your first point alone,

That seems highly unlikely to happen, given the widespread usage of rand.

in tandem - which I'd really not considered - with a significant specialization relative to independent variables is enough motivator to dispel my concern.
I like the ideas you mention about how to allow for specialization of multidimensional distributions. Seems like a good idea for another PR.
I think the only remaining question is if this should go in the 0.17 release or in the 0.18
@henryjac is it odd or "aggressive" in any way for our first release have a breaking change?

Took another look at this, I also think that uncorrelated multivariate normal variables are common enough to be useful, but would it make sense to have a distinct API from existing MVN? It allows a user to opt in/out for optimizations, is this intended?
I'm looking at the scipy implementation that you referenced, and it's close to what I'm thinking. What methods would a Covariance trait have that allow specifying different behavior for the pdf and sampling depending on the representation?

Something like this would be a good start
trait Covariance<V, M>
{
	/// sqrt(cov) @ vec
	fn forward(&self, vec: &V) -> V;

	/// inv(sqrt(cov)) @ vec
	fn inverse(&self, vec: &V) -> V;

	/// diag_part(cov)
	fn diag(&self) -> V;

	/// dense(cov)
	fn dense(&self) -> M;

	/// abs(det(cov))
	fn abs_determinant(&self) -> f64;

	/// ln(abs(det(cov)))
	fn ln_abs_determinant(&self) -> f64;
}

That'll let you implement efficient sampling/log probability and also compute the variance vector and (dense) covariance matrix. Like for the current MVN implement, many of them can be pre-computed upon construction.
For inverse/forward methods, TensorFlow calls them matvec/solve, scipy calls them colorize/whiten.

This looks really good! Would you be willing to implement it to support diagonal and correlations for mvn?


Update scummvm to 2.7.1

This small changes updates the scummvm version from 2.7.0 to 2.7.1, which was recently tagged.

Done

Thanks!


Text with mixed (expanded/contracted) contractions

I'm having a difficulty with this test text: he's got a gun that he is going to use.
If I just dump out the terms (link), I see that he's is represented with 2 terms, which is kind of odd. But I see the expansion property (undocumented?) is populated... is that what that's about, saying those two terms are part of the single word?
If I wanted to, say, count words... how can I tell that terms are part of the same word? Adjacent in the terms array + expansion is populated + ?
Secondly, if I expand the text (link), the two he is's get parsed differently. The first Term, Verb and the second is Person, Verb (tagged as copula). That's surprising because I would have expected in this case for those to come out equivalently. Not saying it's wrong, just that I was surprised.
Third, the expansion of the first he's to he is could be improved to he has due to the following got.

yep, this is a huge error. Thank you for documenting it so clearly.
Yeah, contractions need to be split apart into two terms for proper interpretation of their grammar. If you want a real word-count, you could loop through the terms and not include the ones with a expansion-field, and no text... but maybe that's a good candidate for a helper-method... ya this is probably a good idea I think.
yeah, "apostrophe s" is a really ambiguous thing, not just with possessives, ("spencer's",) but with he has, he is, he was.. etc. I've done a bit of work at pulling these apart, but not enough, as it looks like your example is parsed incorrectly.
Pretty solvable, this stuff, but we're still rounding-third base with the proper, smooth contraction support. It presents problems not just here, but all over the codebase ;)
will keep this open.
thanks!

hey, just going back to this example, it's a doozy.
you can see it's better now but the expand() method is pretty conservative when it comes to ambiguous cases like 'he has'.
closing this for now.
thanks!


Draft: Make AwsClientCustomizer Flexible

:loudspeaker: Type of change

[ ] Bugfix
[ ] New feature
[x] Enhancement
[ ] Refactoring

:scroll: Description
:bulb: Motivation and Context
:green_heart: How did you test it?
:pencil: Checklist

[x] I reviewed submitted code
[x] I added tests to verify changes
[ ] I updated reference documentation to reflect the change
[x] All tests passing
[x] No breaking changes

:crystal_ball: Next steps

I still need to add tests and documentation want to first talk about whether are there ways how we can improve and make customization even simpler.

Closed in favour of #1229


Travis build infrastructure refresh.

Resolves #17163. Ubuntu Trusty containers are now available on Travis, removing the last obstacle stopping us moving to the container based environment. This cuts build times by about 15 seconds.
Regarding the other changes: Ruby 2.3.1 comes preinstalled on the new images, so I have removed the install directive. The npm install directive has been made conditional as only the Node 4.x environment will have npm 2.x. --retry=3 has been added to the bundler command to bring it in line with Travis' default and to provide greater network stability. NPM_CONFIG_PROGRESS="false" has been removed as npm/npm#11283 seems to be fixed. Git --depth has been lowered.
Coming soon in another PR, is switching to using the Travis apt addon to download the Google Cloud SDK (blocked on travis-ci/apt-package-whitelist#3515) and switching to using the Travis sauce_connect addon rather than using grunt-saucelabs (blocked on further investigation).

@cvrebert Wanna take a look at this and let me know what you think? <3

and switching to using the Travis sauce_connect addon rather than using grunt-saucelabs

See #18754. BrowserStack seems like our best option IMHO.

See #18754. BrowserStack seems like our best option IMHO.

Ah. I saw that but I wasn't sure if it was still on the table.


Extended configurations for selenium:

I'm trying to pass custom location of firefox but unable to do that.
Selenium supports 2 ways of adding extra configs:

pass as inline parameter :

java -jar "selenium-server-standalone-3.0.0.jar" -Dwebdriver.firefox.bin="/opt/firefox/firefox-bin"

Use node/hub Json config file:

{
  "capabilities": [
    {
      "browserName": "firefox",
      "maxInstances": 8,
      "version": "49",
      "seleniumProtocol": "WebDriver"
    }
  ],
  "Dwebdriver.firefox.bin=/opt/firefox/firefox-bin": "",
  "Dwebdriver.chrome.driver=/usr/share/chrome-webdriver/chromedriver": "",
  "maxSession": 10,
  "port": 5555,
...
}

Using Json config file does not work , and i'm not sure if selenium grid extra supports extra inline params to be passed to selenium
probably related to #296

@vveliev I am not sure this falls in the line of Selenium Grid Extras (I could be wrong). Selenium Grid Extras created the config file, but after that, it just reads it. So you should be able to add anything to the node config file that you could without selenium grid extras.
There is a known issue where new items added to the config file are being removed, but I think it is only when upgrading from Selenium 2 to Selenium 3 (since the format of the node file changed, I just read the parameters we initially put into the config file, and put them in the new config file in the right format).
Have you tried using the same node config file without selenium grid extras?


Server connect failed

The Growatt Portal is working, but with NodeRed i get since yesterday this Message:
"Error: {"result":-5,"msg":"temporarily unavailable"}"

At my home it works at the moment, the Growatt server responded normally (at my location). The message indicates that there is some error on the Growatt side, not on the node.
So please try again. Also I deprecated this node, as I do not use the Growatt Shine Server system anymore.

Not working for me. Is it possible to change the server in this Node? Is there a INI or config File?
server.growatt.com or server-us.growatt.com
31.7.2023, 09:51:49node: BBS18
msg : error
"Error: {"result":-5,"msg":"temporarily unavailable"}"
Too bad that you don't use the System anymore. It will prefer to get the Data direkt from the Wifi Stick, but there is no way i think......

Get the data directly.. This is exactly the case why I not use the Growatt servers anymore.
https://github.com/otti/Growatt_ShineWiFi-S
With this you can flash your ShineWiFi stick with a new firmware. This firmware will output the data via MQTT and also has a REST endpoint to grab the data.
With this I update my solar data each second :)

Vy cool!!!! This i try directly!


Change "queuing" metrics to "starting"

The term queuing is misleading because the task isn't ready and waiting. It actually reflects the time where underlying infra is starting, pulling container etc. We should use "starting".

https://github.com/awslabs/amazon-omics-tools/pull/61


ENH add diagram to illustrate scikit-learn API

closes #231

I am merging this PR as a first approximation. We can both improve the figure and the text.


Fix Minimum Wage

Description
Trello: https://trello.com/c/IonCW10l/660-investigate-minimum-wage-smart-answer
Zendesk Ticket: https://govuk.zendesk.com/agent/tickets/2231061
Calculate the basic_rate as the average between the normal rate and
the overtime_hourly_rate, instead of just returning the overtime_hourly_rate.
This is stated here: https://www.gov.uk/overtime-your-rights/overview.
This prevents the message "You are not getting the minimum wage" to appear, if
the basic rate is above the minimum rate.
GOV.UK
Preview
Notes for Factcheck
This introduces the calculation of the rate as the average between the normal rate and
the overtime_hourly_rate. Apart from avoiding the result "You are not getting the minimum wage" to be shown, it introduces a change in a lot of outcomes, due to this different calculation. They should be reviewed and approved too.
Some examples of paths with changes:
Before: https://www.gov.uk/am-i-getting-minimum-wage/y/current_payment/apprentice_over_19_first_year/1/16.0/300.0/8.0/10.0/no
After: https://smart-answers-preview-pr-3230.herokuapp.com/am-i-getting-minimum-wage/y/current_payment/apprentice_over_19_first_year/1/16.0/300.0/8.0/10.0/no
Before: https://www.gov.uk/am-i-getting-minimum-wage/y/current_payment/apprentice_over_19_first_year/1/16.0/300.0/8.0/10.0/yes_charged/5.0/5
After: https://smart-answers-preview-pr-3230.herokuapp.com/am-i-getting-minimum-wage/y/current_payment/apprentice_over_19_first_year/1/16.0/300.0/8.0/10.0/yes_charged/5.0/5)
Before: https://www.gov.uk/am-i-getting-minimum-wage/y/current_payment/apprentice_over_19_first_year/1/16.0/300.0/8.0/10.0/yes_charged/50.0/5
After: https://smart-answers-preview-pr-3230.herokuapp.com/am-i-getting-minimum-wage/y/current_payment/apprentice_over_19_first_year/1/16.0/300.0/8.0/10.0/yes_charged/50.0/5
Before

After

@mgrassotti checksums need updating

@chao-xian thanks, it's done

@chao-xian is that ok for you now? thanks!


Upload input button overflows

Informations
Browser version: 62.0.3 (64-bits)
OS: Ubuntu 18.04
Screen resolution: 2560 x 1440
How to reproduce the bug?
Open the Upload dialog

Can't fix without a lot of CSS, so I will not fix that, sorry.

I made changes and "fixed" it... There's no cross browser solution, but I could make it look as good as possible.
I'll try submitting a pull request


Bluetooth: Host: identity resolution causes CCC issues on other connections with the same peer

Describe the bug
The bug assumes that the application uses CONFIG_BT_PRIVACY and CONFIG_BT_SMP configurations. It also assumes that the application exchanges the Identity Address with the connected peer during the pairing phase.
When the Bluetooth GATT layer (gatt.c) gets notified about the identity resolution in the bt_gatt_identity_resolved function during the bonding phase for one Bluetooth identity, it converts the BLE address field (called peer) in all affected CCC descriptors from RPA into the Identity address:
for (size_t i = 0; i < ARRAY_SIZE(ccc->cfg); i++) {
	if (bt_addr_le_eq(&ccc->cfg[i].peer, match->private_addr)) {
		bt_addr_le_copy(&ccc->cfg[i].peer, match->id_addr);
	}
}

If another connection object associated with the second identity is connected simultaneously with the same peer, its peer destination address (conn->le.dst) is not converted from RPA into an Identity address. Once the second identity attempts to send a GATT notification or indication, it fails in the bt_gatt_is_subscribed function as the stack cannot find the CCC descriptor using the RPA address from the conn->le.dst field. The failure happens because of the conversion of the BLE address field in the CCC descriptor that was performed during the bonding phase on the first identity.
To Reproduce
Steps to reproduce the behavior:

Create the Bluetooth sample with two Bluetooth identities and two advertising sets with the Privacy feature enabled.
Connect to the second identity and subscribe to any characteristic.
Connect to the first identity while maintaining the connection with the second identity using the same central device (with the same peer address) as the one used to connect to the first identity (e.g. the same smartphone).
Complete the pairing and bonding procedure with the first identity.
Observe that GATT notifications or indications fail to be sent on the second identity.

Expected behavior
The identity resolution on one connection should not impact the CCC configuration on the second (parallel connections) even if you maintain two different connections with the same peer.
Impact
The Bluetooth stack fails to send GATT notifications or indications on other connection objects associated with the same peer address than the one on which the identity resolution and the pairing happened.
Environment (please complete the following information):

OS: Linux
Commit SHA: the main branch (https://github.com/zephyrproject-rtos/zephyr/commit/f16c4324a7e28c929f7226dd06fef87d951431c9)

You might be able to work around that bug by unsetting CONFIG_BT_GATT_ENFORCE_SUBSCRIPTION


CLOUDSTACK-8582: Modifying tag for test cases which should not be run on simulator

The test case tries to ssh to router through host and take down the services. This is not a right candidate to run on simulator.

LGTM


FwbSideBarItem cannot be used with named route

In case when one has nested pages like: /tool, and /tool/foo in the route definition we have the following:
const routes: RouteRecordRaw[] = [
    {
        component: ...,
        name: 'rool-route',
        path: '/tool',
        children: [
            name: 'tool-foo-name'
            path: 'foo',
            component: ...,
        ]
    }
]

Note that in the children, the path are always relative to their parent component's path.
If one would like to use named routes, the prop of the <RouterLink> expected the following type: { name: string }. So it must be used like:
<RouterLink to:"{ name: 'tool-foo-name' }">

I have created a PR to solve this. That is of course a very ridumentary solution. Only if there is interest in accepting this feature, then I'll put more time into it.


build index 10 minutes after setup and every hour

No review required. This is a simple parameter change.


Need custom function for copying image from clipboard as Pasteboard package does not supports android platform .

/assign

/assign


Node module loading another module sometimes returns package.json instead of main file

I'm trying to parse some HTML from a remote site and use the output to drive my app.  This works great via command-line node:
var text = "HelloWorld!";
var cheerio = require("cheerio");
var $ = cheerio.load(text);

try {
  $("b").remove();
} catch(e) {
  console.log(e.stack);
}

console.log($.html()); //Outputs "World!"

This snippet does not work in Tabris.js
var page = new tabris.Page({
  title: "Cheerio",
  topLevel: true
});

var text = "HelloWorld!";
require("tabris-js-node");
var cheerio = require("cheerio");
var $ = cheerio.load(text);

try {
  $("b").remove();
} catch(e) {
  console.log(e.stack);
}

new tabris.TextView({
  text: $.html() //Shows "HelloWorld!"
}).appendTo(page);

page.open();

Calling remove() (or any other Cheerio function that needs to find an element) throws a TypeError with stack trace:
TypeError: select is not a function
    at exports.find (./node_modules/cheerio/lib/api/traversing.js:33:21)
    at module.exports (./node_modules/cheerio/lib/cheerio.js:81:18)
    at new initialize (./node_modules/cheerio/lib/static.js:26:20)
    at initialize (./node_modules/cheerio/lib/static.js:23:14)
    at ./app.js:17:2
    at Object.defineProperty.get (<anonymous>:1:1413)
    at Object.i.require (<anonymous>:1:1617)
    at g (<anonymous>:1:2993)
    at Channel.fire (./cordova.js:822:23)
    at ./cordova.js:231:49

select is indeed not a function; it's an object.  select is defined 31 lines above:
var select = require('css-select')

I would expect this to be the exported interface that the css-select package exposes in its module.exports, however, it's actually the object representation of node_modules/css-select/package.json.  I thought this might be a problem with Apache, so I switched to http-server and saw the same behavior.
Interestingly, adding DirectoryIndex index.js to Apache config (which makes Apache serve up index.js if it exists instead of an HTML directory index) caused my app to hang at require("cheerio");
--
Here is the full request log output from http-server.
Here is the package.json that I am using for my app:
{
  "main": "app.js",
  "dependencies": {
    "cheerio": "^0.20.0",
    "tabris": "^1.8.0",
    "tabris-js-node": "^0.1.0"
  }
}

As a last resort I tried renaming node_modules/css-select/index.js to index2.js and modifying node_modules/css-select/package.json accordingly but require('css-select') still returns an Object representing its package.json.

Still seeing this issue with nightly JS and nightly Android.

Yeah, some further work on the clients is needed.

It should work now.

Confirmed, thank you!


Untyped variable was accessible globally before bsh-2.0b6 but in master/bsh-2.0b6 untyped variable is only accessible within the local scope.

Script used to test is as follows.I have taken screenshots of the result for the script in all master , bsh-2.0b6.jar and bsh-2.0b1.jar.
As per the documentation,the untyped variable is global scoped but it is not working as expected.

=================================
foo() {
if (true) {
y = 2; // Untyped variable assigned
int x = 1; // Typed variable assigned
print("Inside if block");
print(" y = " + y);
print("x = " + x);
}
print("outside if block");
print("y = " + y);
print("x = " + x);
}
foo();

Any update on this.

We do not support  bsh-2.0b6 please use master.

We have used master to test and it is not working as expected.Please read my comments in the issue created.


Add LUDecomposition triangular factor tests

Bumping coveralls stats 😎

Busy weekend :construction_worker_man:
:+1: :+1:


Having pouchdb-adapter-memory in package.json breaks app, even if not required

Basically as title says, if I install pouchdb-adapter-memory but don't require it anywhere, I still get this error on startup:
Cannot find module 'leveldown/package.json' at<EMAIL_ADDRESS>

Very minor for me, but seems like a legit bug in pundle checking for package.json's.

Are you sure you did not have a local NPM cache issue? because I don't think you can publish to the npm registry without a package.json.
I'd recommend clearing npm cache, removing node modules and reinstalling as error exists outside of pundle

Pundle actually throws the error even though I never require that module
directly, it just was inside my node_modules. I'll try and repro on a clean
repo.
On Thu, Aug 17, 2017 at 8:10 PM Steel Brain<EMAIL_ADDRESS>wrote:

Are you sure you did not have a local NPM cache issue? because I don't
think you can publish to the npm registry without a package.json.
I'd recommend clearing npm cache, removing node modules and reinstalling
as error exists outside of pundle
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/steelbrain/pundle/issues/165#issuecomment-323248068,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAvRKI_I72H63-wvoGSmIiWR9VaWwY5ks5sZQCMgaJpZM4O7AHD
.


adding the HUD controls overlay back in, growing the pause menu to fi…

…t the buttons

I pulled it, tested it, looked good, so merged it.


Suggest setting copilot-enable-predicates to nil for spacemacs

In order for this plugin to work with spacemacs we need to set copilot-enable-predicates to nil. This closes #123

The copilot-enable-predicates contains two functions by default. Can you please confirm which one is causing the plugin to not work with Spacemacs?

I tried setting it to both copilot--buffer-changed and  evil-insert-state-p individually, neither worked.
These were the two functions I found when I did M-x describe-varable copilot-enable-predicates

(setq copilot-enable-predicates '(evil-insert-state-p))
(setq copilot-enable-predicates '(copilot--buffer-changed))
In case you forgot to make it a list.

Ah yes, I didn't make them a list. Thanks, so it's:
(setq copilot-enable-predicates '(evil-insert-state-p))

that is preventing the plugin from working. I wonder if this is because I use spacemacs with emacs keybindings rather than vim ones. That setting sounds very vim-like

Yes, evil-insert-state-p will prevent the plugin from triggering completions if the current VIM state is not insert. However, copilot.el will ignore this function if it's not available, so it should work for users using Emacs keybindings.
I'm curious why (functionp evil-insert-state-p) returns t, even when you're using Emacs keybindings.

I'm not actually able to see functionp when I run M-x, not to sure why. I can describe it though:
evil-insert-state-p is a native-compiled Lisp function in ‘hybrid-mode.el’.

(evil-insert-state-p &optional STATE)

Whether the current state is insert.

So what shall we do with PR? It sounds to me as though you're suggesting it could be an issue with Spacemacs?

Spacemacs offers three editing styles: Vim, Emacs, and Hybrid. However, hybrid-mode is beyond the scope of consideration due to the small number of users.
I recommend that you switch your editing style from hybrid to emacs, or alternatively customize copilot-enable-predicates.

Hello @zerolfx I am using emacs keybindings and I had to set (setq-default copilot-enable-predicates nil) in my .spacemacs config for copilot to work as expected.

I am going to fix this in Spacemacs.


Wrong file path in index.d.ts + TS build issues

https://github.com/vueform/slider/blob/1e491ba0205bb7de5b2d3dea9159b164ff64cf77/src/index.d.ts#L1 references to a wrong file ... it should be ./Slider.
Also, Vue 3 has a problem during build with all vueform components because the types extend from Vue. The current workaround ... shims file in project's src folder:
declare module<EMAIL_ADDRESS>{
   import { DefineComponent } from 'vue'
   const component: DefineComponent
   export default component
}

and import component as
import Slider from<EMAIL_ADDRESS>
The same applies to toggle and multiselect.

Hi @staryja4 ! I'll update the index.d.ts in the next patch - thanks for reporting! Could you submit a PR with proper type definitions for Slider.d.ts that work with Vue 2 & 3?


x/build: use GCP container builder triggers for images?

Work on converting docker images to GCP container builder and nested builds so we don't have to push and update them all the time.
cc @adams-sarah @cybrcodr @bradfitz

I'm a little apprehensive here.
You say "Switch to X" without a reason, so I'm guessing the reason is that your pushes are slow?
My development VM is already on GCP, so my pushes to gcr.io are super fast already.
I'm concerned that this will only slow down my normal hack+push dev cycle, rather than speed it up, though I imagine this would speed up development if my primary dev machine was my laptop pushing to gcr.io from an airplane.
I believe all of our Dockerfiles are designed to make iterative development pretty painless, caching things at the last step to make the final compile quick, e.g.
https://github.com/golang/build/blob/da1460b7c9c9b65383d1336593ed9ad346f6a1c5/cmd/coordinator/Dockerfile.0#L105
So if we switch to GCP container builder, I want to understand why we're doing it and what we gain, and what sort of latency guarantees they have.
Or I want to preserve the ability to do it the current way, which I find to be interactively useful. I don't want a big slow machine between writing code and prod/staging.

It's not that it's slow it's more like "why must I do this when a computer
can do it for me"
On Mon, Jul 31, 2017 at 9:52 PM, Brad Fitzpatrick<EMAIL_ADDRESS>wrote:

I'm a little apprehensive here.
You say "Switch to X" without a reason, so I'm guessing the reason is that
your pushes are slow?
My development VM is already on GCP, so my pushes to gcr.io are super
fast already.
I'm concerned that this will only slow down my normal hack+push dev cycle,
rather than speed it up, though I imagine this would speed up development
if my primary dev machine was my laptop pushing to gcr.io from an
airplane.
I believe all of our Dockerfiles are designed to make iterative
development pretty painless, caching things at the last step to make the
final compile quick, e.g.
https://github.com/golang/build/blob/da1460b7c9c9b65383d1336593ed9a
d346f6a1c5/cmd/coordinator/Dockerfile.0#L105
So if we switch to GCP container builder, I want to understand why we're
doing it and what we gain, and what sort of latency guarantees they have.
Or I want to preserve the ability to do it the current way, which I find
to be interactively useful. I don't want a big slow machine between writing
code and prod/staging.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/21249#issuecomment-319245059, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABYNbPT-MevdvAAIaHVR19DhxRMhQieSks5sToT4gaJpZM4OpHeg
.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

oh ya this was assuming we could have both manual and that

A computer does it for us already. We type make push-prod. It's not like we're assembling a tarball in a hex editor by hand. :-)
If you just want to use new toys, that's fine, as long as it's optional to start with.

lol well I'm sticking it last on my list haha

Just my thoughts.  GCP container builder may be nice to have, but it does come at a cost, so it should only be optional or can be done for "official" builds if we do need such.
I think what may be nice to improve on is switching to multi-stage Dockerfile for the builds, I think that may simplify the Makefile definitions.

multi-stage Docker would be nice. If we do that, though, make sure that for anybody running too-old Docker versions, we fail nicely with a useful error message.

I'm going to close this. We now use multi-stage Dockerfiles and the Makefiles are all super simplified. Things are much cleaner than when this bug was filed.
Even though we still don't use container builder, I'm not sure there's a big advantage.


net-firewall/conntrack-tools: remove obsolete 1.4.5

This change removes an outdated conntrack-tools ebuild - version 1.4.5 - from maintenance branch flatcar-4054. The branch uses 1.4.6-rc1 anyways so the legacy ebuild can safely be deleted.
This ebuild uses EAPI 6 and will cause a transient error message during build. It imports the linux-info eclass which was recently updated to only support EAPI 7 and higher. This will cause emerge to print an error message when considering the 1.4.5 ebuild. While that error message has no impact on the build because 1.4.6-rc1 is more recent and will hence be used, it can be misleading when investigating unrelated issues (which it did to me, hence this patch).
How to use

Check out this branchgit clone https://github.com/flatcar/scripts.git
cd scripts
git checkout t-lo/flatcar-4054-remove-conntrack-1.4.5

Build it./run_sdk_container -t
       ./build_packages
       ./build_image

Testing done
Built the image (see above), built qemu uefi image, ran tests locally.

Please update the Manifest too. I know we don't run QA tools that would check that but...

Please update the Manifest too. I know we don't run QA tools that would check that but...

 Good catch, will update. We should make use of these QA tools one day...

Tests are green except for extra-test.[first_dual].cl.update.oem which is currently broken; tracked in https://github.com/flatcar/Flatcar/issues/1518. Very likely unrelated. Merging.


error in hydration while doing a swap

Description
error in hydration while doing a swap using multix
Steps To Reproduce
Add clear steps on how to reproduce the bug

Go to Multix at 'https://multix.chainsafe.io/'
Connect your wallet
Go to Hydration DEX https://app.hydration.net/trade/swap?assetOut=5
See error below

Expected Behaviour
A clear and concise description of what you expected to happen.
Console Error

Test Host Specifics

OS: Windows 11
Browser Chrome
Version Latest
Wallet Multix

I feel like we had this conversation before, but I can't find it. Did the tx appear on Multix still? If it did, and you signed it, then things would work. I need to find a way to not reject this TX for hydration.

I feel like we had this conversation before, but I can't find it. Did the tx appear on Multix still? If it did, and you signed it, then things would work. I need to find a way to not reject this TX for hydration.

No it's quite different from our earlier conversation. The txns aren't appearing in the wallet at all, previously atleast it was the txns were being initiated.

Sorry for the delay, I've tried to reproduce, but I failed.
Here's what I did:

connect my Multix account with walletConnect. I see this in multix:

go to hydration website, make a swap
the "signing transaction" is loading on Hydration
the extrinsic is ready to be signed on Multix too

https://github.com/user-attachments/assets/c0507f29-00e7-43f3-8f3b-b14f1e70d9eb
Can you try to delete the walletconnect connection, on Multix, and maybe on Hydration, and do it again?

@Tbaut followed the instructions like you mentioned. still there's an issue. attaching the video below
multisig wallet address on hydradx : 5D7tybQTk8v7ULfQkizyVQuaDJpJhfgohHZjk5JoQyiwrfWh
https://hydration.subscan.io/account/5D7tybQTk8v7ULfQkizyVQuaDJpJhfgohHZjk5JoQyiwrfWh
https://github.com/user-attachments/assets/b77bb354-6bf2-4042-9d28-4526bb4c3fdb

I think I get the issue now, most probably because the individual accounts in hydradx don't have the min balance so the error is coming. hmm closing this.. thank you for your support..
#558 waiting for this :)

I was about to say, you could print the logs on hydration and see there, the fact that they don't send the tx over wallet connect made me think there's something wrong there.


basic transforms and two datasets

I created some basic transforms and datasets similar to the torchvision module.
transforms: scale, trim/pad, downmix to mono
datasets: vctk dataset, yesno dataset

looks great! thanks.


Replace the "+" icon to a cart for the "Add to Cart" option.

The icon for the button isn't visually pleasing for the website. I feel it would look better and more practical with the cart icon such as

If you think this issue is relevant, I would be happy to do this job @swaraj-das

@himanchal08 Sorry, the issue has already been assigned to me as I was the one who pointed out the issue. You can go through the project and suggest any possible issues you find with the project.

Could you also assign the level tag for this issue? @swaraj-das


Improve integration testing and automatic unit testing

to go with the extension guidelines
should check conformity with interface design and extension guidelines

run check_estimator on all estimators on master branch

Should test for

raises specific not-fitted error if predict/transform is called before fit (check_is_fitted)
runs fit, predict and score for classifiers/regressors/forecasters
runs fit, transform and inverse_transform (if available) for transformers
raises specific only-univariate-data error on multivariate data
raises error on wrong input types (i.e. not pandas DataFrame for X)

Closed by #246


fix DefaultLitePullConsumerImpl NPE

What is the purpose of the change
fix npe problem
Brief changelog
ProcessQueue processQueue = assignedMessageQueue.getProcessQueue(messageQueue);
//  "processQueue" is nullable here. line 653
if (processQueue == null && processQueue.isDropped()) {
    log.info("The message queue not be able to poll, because it's dropped. group={}, messageQueue={}", defaultLitePullConsumer.getConsumerGroup(), this.messageQueue);
    return;
}

change to
ProcessQueue processQueue = assignedMessageQueue.getProcessQueue(messageQueue);
//  "processQueue" is nullable here. line 653
if (processQueue == null || processQueue.isDropped()) {
    log.info("The message queue not be able to poll, because it's dropped. group={}, messageQueue={}", defaultLitePullConsumer.getConsumerGroup(), this.messageQueue);
    return;
}

Verifying this change
ProcessQueue processQueue = assignedMessageQueue.getProcessQueue(messageQueue);
//  "processQueue" is nullable here. line 653
if (processQueue == null && processQueue.isDropped()) {
    log.info("The message queue not be able to poll, because it's dropped. group={}, messageQueue={}", defaultLitePullConsumer.getConsumerGroup(), this.messageQueue);
    return;
}

change to
ProcessQueue processQueue = assignedMessageQueue.getProcessQueue(messageQueue);
//  "processQueue" is nullable here. line 653
if (processQueue == null || processQueue.isDropped()) {
    log.info("The message queue not be able to poll, because it's dropped. group={}, messageQueue={}", defaultLitePullConsumer.getConsumerGroup(), this.messageQueue);
    return;
}

Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.07%) to 51.003% when pulling c33961ce7688df4acd0aa24c097e04481226ebb8 on lebron374:npe_fix into b3ec283c8d9d11dab95a3f0dfc4ee9789bf53749 on apache:develop.

Codecov Report

Merging #2059 into develop will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             develop    #2059   +/-   ##
==========================================
  Coverage      45.23%   45.23%
+ Complexity      4219     4218    -1
==========================================
  Files            545      545
  Lines          35810    35812    +2
  Branches        4758     4761    +3
==========================================
+ Hits           16197    16201    +4
+ Misses         17566    17561    -5
- Partials        2047     2050    +3

Impacted Files
Coverage Δ
Complexity Δ

...ent/impl/consumer/DefaultLitePullConsumerImpl.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...a/org/apache/rocketmq/filter/util/BloomFilter.java
59.13% <0.00%> (-2.16%)
25.00% <0.00%> (-1.00%)

...ava/org/apache/rocketmq/filter/util/BitsArray.java
58.11% <0.00%> (-1.71%)
29.00% <0.00%> (-1.00%)

...org/apache/rocketmq/common/stats/StatsItemSet.java
38.09% <0.00%> (-1.59%)
12.00% <0.00%> (-1.00%)

...rocketmq/client/impl/factory/MQClientInstance.java
45.93% <0.00%> (-0.45%)
80.00% <0.00%> (ø%)

...ain/java/org/apache/rocketmq/store/MappedFile.java
50.18% <0.00%> (-0.37%)
50.00% <0.00%> (-1.00%)

.../apache/rocketmq/logging/inner/LoggingBuilder.java
63.60% <0.00%> (-0.32%)
3.00% <0.00%> (ø%)

...e/rocketmq/remoting/netty/NettyRemotingClient.java
46.92% <0.00%> (-0.28%)
26.00% <0.00%> (ø%)

...org/apache/rocketmq/store/DefaultMessageStore.java
55.45% <0.00%> (-0.20%)
106.00% <0.00%> (ø%)

...org/apache/rocketmq/common/message/MessageExt.java
68.23% <0.00%> (ø)
34.00% <0.00%> (ø%)

... and 7 more

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 0144b03...c33961c. Read the comment docs.

Hi @lebron374, PR name needs to be followed as [ISSUE #2044] Fix DefaultLitePullConsumerImpl NPE

@lebron374 Could you help to add unit test for here?

It would be better to add ut for this minor bug.

i do not understand why this commit need ut？

It would be better to add ut for this minor bug.

add unittest testPullTaskImpl_ProcessQueueDropped


Monitoring gcePersistentDisk via kubelet_volume_stats_available_bytes ?

Is there a way to see gce disks volume usage via prometheus?
e.g. given sample pod config:
          - mountPath: /exports
            name: gce-test-disk
      volumes:
        - name: gce-test-disk
          gcePersistentDisk:
            pdName: gce-test-disk
            fsType: ext4

Is it possible to see used/free space via kubelet_volume_stats_available_bytes (or any other metric?)
GCE volume plugin does implement Metris interface, and I can see volume stats for regular PVC, but not for gcePersistentDisk
Environment

Prometheus Operator version:

v0.22.0

Kubernetes version information:

Client Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.7", GitCommit:"dd5e1a2978fd0b97d9b78e1564398aeea7e7fe92", GitTreeState:"clean", BuildDate:"2018-04-19T00:05:56Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"10+", GitVersion:"v1.10.7-gke.1", GitCommit:"7221ffe8776f399448d61b4dc9edb5c69232aece", GitTreeState:"clean", BuildDate:"2018-08-29T23:11:12Z", GoVersion:"go1.9.3b4", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes cluster kind:
GKE

That would be an upstream concern, generally I understand the need to monitor this, but I also wonder why you don't provision this via a PVC which has proper support for this?

This should be configurable by the Kubelet, right? I was looking into that myself and I don't see these either. I don't think it has to do with configuration in the Prometheus Operator though.

I just thought I was overlooking something, but PVC will have to do.
Why not PVC right now? Long story short - custom internal tool that creates disks on gce and long running cron jobs on kubernetes, so it was less state to cleanup directly using gcePersistentDisk instead of PVC. But we'll just change that.
thanks again for the awesome tool.


Rh/firefox decimals

closes #884
the issue was that the state of the input could not be changed to something that did not pass parseFloat(). now the state of the input can be erroneous and our onChange() and errors still work the same, the user is just actually allowed to type whatever they want and our parent state will only change to something that passes parseFloat().
in firefox this change lets the user type non-numbers into the input, though they don't pass validation. chrome is nice and limits it to numbers. i do not have edge installed lol
not sure if there's anywhere else that needs a stateful input? @jSylvestre

Edge is Chromium so shouldn't be an issue.
I'll test tomorrow.
There is also an acreage input that has the same issue.


Event to capture brush and zoom range

-Hey
We are using C3 in our BigData Analytics at my company and thank you for creating such a great charting framework!
One thing we do need is the ability to use Crossfilter with C3. While this is already somewhat possible due to C3's nice API, one thing is missing is the ability capture the time range on a zoomed or brushed timeseries chart.
What I've done as a bit of a hack was add a hook in redrawForBrush() to send the brush object to an event I added, __subchart_onbrush(brush);. I realize this isn't ideal (and is only for brushes not zooming), but it would be nice to have this functionality in the framework.

@masayuki0812 i can't find the exampel. the exampel page is not found

@masayuki0812 Please, have you Any bar chart which shows x-axis and y-axis data and filter a third column in the data set


[bugs fix]: soundness bug in powers of challenges; witness gen bug

previous version the powers of challenges are not constrained in circuits. this pr fixes the bug; also does some clean up

commits before
"more accurate rows per sig data" already squashed/merged. So we can rebase the last 4 commits as a new branch to solve the conflicts. (low priority)

commits before "more accurate rows per sig data" already squashed/merged. So we can rebase the last 4 commits as a new branch to solve the conflicts. (low priority)

sounds good. Let me get the parallization feature implemented first before rebasing and merge.

is this still relevant or can be closed?

is this still relevant or can be closed?

No... not relevant. Closed.


Able to resize a panel beyond collapse button

Describe the bug
When using a Panel component which resizable then the user can shrink the side pane to beyond the side pane header's width (including the collapse button).
Setting panelMinSize would solve this but we are using this in a generic way so we'd like to be able to set the min-width of the Panel (or the panelMinSize) dynamically based on the header content.
To Reproduce
Steps to reproduce the behaviour:

Go to the Nova docs
Scroll to the "Enabling Resizability" example
Manually change the header text via Dev tools to a string which is longer than 150px
Click and drag the side pane

Expected behaviour
The side pane prevents the user resizing it below the minimum width of the side header (including the collapse button).
Screenshots
Clicking and dragging pane boundary

Click and drag released

Dev tools showing resizer min-width

Environment (please complete the following information):

OS: MacOS Catalina
Browser Chrome
Browser 89

Additional context
We have currently hit this issue on Nova 9 so a workaround, if available, would be appreciated until we can upgrade to Nova 11 (and Angular 11).

If you can access N-able repos, here is a proposed solution to workaround this issue on Nova 9: https://github.com/logicnow/msp-entity-module/compare/bugfix/RMM-40300_preventSidePaneShrinkingBelowHeaderMinWidth.

https://jira.solarwinds.com/browse/NUI-5580


feat: detect trailing newline character

[ ] Reviewed by @darscan

Coverage remained the same at 100.0% when pulling cba4fe41e4ebb1d22e7b2d50aa7dc37ecff2e928 on feat/trailing-newline-detection into c5253155ad9782abd8f0ce1685d96f922fa09a6f on master.

Coverage remained the same at 100.0% when pulling cba4fe41e4ebb1d22e7b2d50aa7dc37ecff2e928 on feat/trailing-newline-detection into c5253155ad9782abd8f0ce1685d96f922fa09a6f on master.

Coverage remained the same at 100.0% when pulling cba4fe41e4ebb1d22e7b2d50aa7dc37ecff2e928 on feat/trailing-newline-detection into c5253155ad9782abd8f0ce1685d96f922fa09a6f on master.

Coverage remained the same at 100.0% when pulling cba4fe41e4ebb1d22e7b2d50aa7dc37ecff2e928 on feat/trailing-newline-detection into c5253155ad9782abd8f0ce1685d96f922fa09a6f on master.

Coverage remained the same at 100.0% when pulling 1e05e45690ba31e03ef571e0266698308dcf5f6e on feat/trailing-newline-detection into c5253155ad9782abd8f0ce1685d96f922fa09a6f on master.

Coverage remained the same at 100.0% when pulling 075770d0a84e5fa26e1b2bb8912db875c5a48460 on feat/trailing-newline-detection into c5253155ad9782abd8f0ce1685d96f922fa09a6f on master.


HL-560/570: Hide expired tet postings from search

Description :sparkles:
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #1564 (17f21a3) into develop (44b62d5) will decrease coverage by 19.30%.
The diff coverage is 37.50%.

@@             Coverage Diff              @@
##           develop    #1564       +/-   ##
============================================
- Coverage    65.39%   46.08%   -19.31%
============================================
  Files         1041       32     -1009
  Lines        28513      473    -28040
  Branches      2973      157     -2816
============================================
- Hits         18645      218    -18427
+ Misses        9414      245     -9169
+ Partials       454       10      -444

Impacted Files
Coverage Δ

...end/tet/shared/src/__tests__/utils/fake-objects.ts
54.83% <0.00%> (+1.71%)
:arrow_up:

...src/hooks/backend/useEventPostingTransformation.ts
0.00% <0.00%> (ø)

frontend/tet/shared/src/utils/get-ui-end-date.ts
0.00% <0.00%> (ø)

...ontend/tet/shared/src/utils/get-server-end-date.ts
83.33% <83.33%> (ø)

...tend/tet/shared/src/backend-api/transformations.ts
86.11% <100.00%> (-2.47%)
:arrow_down:

...tend/tet/shared/src/constants/furthest-end-date.ts
100.00% <100.00%> (ø)

backend/tet/events/linkedevents.py

backend/tet/events/services.py

...admin/src/components/editor/form/ActionButtons.tsx

... and 1009 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Left some comments, please have a look at them.
backend/tet/events/linkedevents.py: in list_ongoing_events_authenticated function this could be also removed if doing changes to the pull request:
# TODO exclude ended events
# "start": "2021-01-01",
# "end"

as the expired events are not excluded but partitioned into a different group afterwards now.

Removed


Decide back-end platform

Below are all the things I imagine the back-end will need to perform.
Reading

Get list of tables
Get table schema

Field names, types, order
Field validation (ie. email)
For foreign keys, foreign table key and display field names

Get table rows

Paginate

Filter rows
Group/aggregate rows
Full-text search

Writing (also notifications if other users write)

Create new table
Rename table
Remove table
Create new row

Validate values

Modify one or multiple value of a row

Multiple if user copies & pastes multiple values
Validate values

Remove row
Create new field

It will always be created as a generic text type until type is set

Modify field name
Modify field type or validation
Modify field order
Remove field

Relationships
We want users to be able to associate a record in one table with a record in another table. Fieldbook takes the approach of always making this relationship many-to-many. My assumption is because that relationship is easiest for less technical users to understand: records are simply "associated" with one another, so why shouldn't a post have more than one user associated with it?
If many-to-many meets the most use cases then I think we should build for that. The question is whether there's a use case that many-to-many can't accommodate?
As far as how to accomplish this, one way (perhaps the only) is to have a joins table with left and right fields containing the record ids. Although what if you want your posts table to have an author field and an editor field, both linking to the users table? We'd need to associate the joins table records with a particular relationship, requiring a relationships table. Too complex?
Back-end options
PostgresT

Pros: Handles relationships well. Sturdy, flexible API. Allows DB structure to be extremely portable.
Cons: No realtime feed. We'd need to add one like this alongside it. Also if you change a field type in a relational database and one of the rows becomes invalid... what happens? Easier to handle in NoSQL.

Kinto

Pros: Uses a postgres database underneath (though it's still a document store). Handles authentication, realtime, nice API, offline/sync support.
Cons: No concept of relationships (NoSQL). DB structure is less portable since it's documents.

Thoughts? Other options?

postgrest + lapidus gets you realtime
I don't think postgresql will let you change a field type if it will make existing values invalid

I started building on PostgREST but I'm looking over my shoulder now because there's no built-in way to add/alter/drop columns, so we'll have to use stored procedures (plus figure out how to send a HUP signal to the postgrest application) (#13), and that feels a bit like starting to make our own back-end...
On a separate note, I came across a related API generator from IBM called LoopBack. Probably the cool part of IBM.

Things are pretty well underway with postgrest. It's been abstracted to the point where I don't expect it would be too dramatic to change, but I expect we can close this issue for now.


Fix spelling, grammar errors in README.md

Correct spelling, grammar, and capitalization errors.

Thank you :+1:


[feat] Add producer interceptor

Master Issue: #150
Motivation
This is the producer interceptor implementation of #150.
Modifications

Add ProducerInterceptor interface
Add intercept and getInterceptors in ProducerConfiguration.
Triggering the interceptor when beforeSend, onSendAcknowledgement, onPartitionsChange and close.

Verifying this change

[ ] Make sure that the change passes the CI checks.

This change added tests.
Documentation

[x] doc-required
(Your PR needs to update docs and you will update later)

[ ] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

Could you solve the conflicts?

Could you solve the conflicts?

Done. PTAL again. Thanks.

@RobertIndie The main branch is broken due to a recent upgrade of macos-12 runner image. The fix is here: https://github.com/apache/pulsar-client-cpp/pull/199

Hi @RobertIndie, thanks for introducing this feature!
As we discussed just now, for the doc side:
✅ Pulsar feature matrix
I've ticked the boxes of ["Interceptors" for C++ producer and consumer(https://docs.google.com/spreadsheets/d/1YHYTkIXR8-Ql103u-IMI18TXLlGStK8uJjDsOOA0T20/edit#gid=1784579914&range=48:48)
✅ Pulsar client docs
Could you please provide the following tech inputs? TIA!
The following docs are only added to NEXT since this feature is server-independent.

C++ > Producer > Interceptor

C++ > Consumer > Interceptor


az keyvault create sets wrong tenant for the vault

Describe the bug
When we use az keyvault create --subscription=REQUIRED_SUB parameter, the KeyVault is create/updated in the correct REQUIRED_SUB.  However, the TENANT is set to the tenant id of the current default subscription as reported by az account show.
There is a workaround (see below) that is far from ideal as it eliminates the whole purpose and benefits of --subscription parameter which is now thankfully present for all commands.
The impact of this incorrect behaviour is at least a lot of confusion and inability to use KeyVault in Azure Portal due to message "The directory currently selected differs from this key vault's directory. Some actions will be disabled..  Other functionality also may be affected by this but I don't know.
To Reproduce
The following are the snippets and condensed/redacted outputs as run in bash
# Current account
az account show
{
  "isDefault": true,
  "name": "SUB_A",
  "tenantId": "TENANT_A"
}

# Show details of another non-default  account
az account show --subscription REQUIRED_SUB
{
  "isDefault": false,
  "name": "REQUIRED_SUB",
  "tenantId": "REQUIRED_TENANT"
}

# Create KeyVault in REQUIRED_SUB.
# In theory, this should create it in REQUIRED_TENANT.
az keyvault create \
  --subscription=REQUIRED_SUB \
  --resource-group=some_rg \
  --location=northeurope \
  --name=philip-test-kv

# See details of created KeyVault.
# This shows that the vault is in REQUIRED_SUB, but has the
# TENANT_A which is the tenant of SUB_A.
az keyvault show --subscription=REQUIRED_SUB --name philip-test-kv
{
  "id": "/subscriptions/REQUIRED_SUB/resourceGroups/some_rg/providers/Microsoft.KeyVault/vaults/philip-test-kv",
  "location": "northeurope",
  "name": "philip-test-kv",
  "properties": {
    "tenantId": "TENANT_A",
    "vaultUri": "https://philip-test-kv.vault.azure.net/"
  },
  "resourceGroup": "some_rg",
  "type": "Microsoft.KeyVault/vaults"
}

WORKAROUND.
# The only workaround I found is
# to re-run `az vault create` command with the
# default subscription set to the sub in which vault
# is being created. E.g.

# set default sub to REQUIRED_SUB
az account set --subscription=REQUIRED_SUB

# re-run vault create command
az keyvault create \
  --subscription=REQUIRED_SUB \
  --resource-group=some_rg \
  --location=northeurope \
  --name=philip-test-kv

# check details, this now shows correct
az keyvault show --subscription=REQUIRED_SUB --name philip-test-kv
{
  "id": "/subscriptions/REQUIRED_SUB/resourceGroups/some_rg/providers/Microsoft.KeyVault/vaults/philip-test-kv",
  "location": "northeurope",
  "name": "philip-test-kv",
  "properties": {
    "tenantId": "REQUIRED_TENANT",
    "vaultUri": "https://philip-test-kv.vault.azure.net/"
  },
  "resourceGroup": "some_rg",
  "type": "Microsoft.KeyVault/vaults"
}

Expected behavior
The expectation is that az keyvault create --subscription=REQUIRED_SUB should use and set the tenant id of REQUIRED_SUB as supplied by--subscription=REQUIRED_SUB argument.
Environment summary
OS X, installed via brew, latest version as of 2018-10-18
az --version
azure-cli (2.0.48)

@jiasli  add to S162.

Similar to #11871 , mark it as FeatureRequest temporarily.

@bim-msft how come this is a feature request and not a bug when the command argument is not working properly?

Run into the same problem today. In my opinion this should be treated as a bug. This is very annoying when having the command as part of a lager script as it is now not sufficient to just provide subscriptionId to create it at the correct place.


Create postman tests coverage: domestic standing order consent

Description

[ ] Add the proper requests on postman collections to tests the domestic standing orders consent

Acceptance criteria

[ ] All current postman tests pass.
[ ] The Domestic standing orders consent request tests pass.

Hey team! Please add your planning poker estimate with ZenHub @andra-racovita @mariantiris


Using SSL with Flask and eventlet

Flask is not a webserver, so it's not Flask's job to terminate SSL (the dev server is not suitable for production, it's only meant for local development!). This a task for your 'real' webserver (nginx etc.) or even your loadbalancer (if you have one).

Got it. Thanks!


Broken Pipe Error --- Jobs fail when Bam is above ~100Gb

Hi Santiago,
Just a heads up - I've managed to run Telometer on many samples (mostly successfully!) and have noticed that Bam files are above ~100Gb I get a string of BrokenPipeErrors which ultimately produce an out of memory event (I am running on a cluster with 240Gb of RAM). I've documented an instance of this below. For now one can get around this by splitting up their large bam files.
ERROR MESSAGE
Process ForkPoolWorker-43:
Traceback (most recent call last):
File "/users/mkelsey/anaconda/telometer/lib/python3.8/multiprocessing/pool.py", line 131, in worker
put((job, i, result))
File "/users/mkelsey/anaconda/telometer/lib/python3.8/multiprocessing/queues.py", line 368, in put
self._writer.send_bytes(obj)
File "/users/mkelsey/anaconda/telometer/lib/python3.8/multiprocessing/connection.py", line 200, in send_bytes
self._send_bytes(m[offset:offset + size])
File "/users/mkelsey/anaconda/telometer/lib/python3.8/multiprocessing/connection.py", line 404, in _send_bytes
self._send(header)
File "/users/mkelsey/anaconda/telometer/lib/python3.8/multiprocessing/connection.py", line 368, in _send
n = write(self._handle, buf)
BrokenPipeError: [Errno 32] Broken pipe
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/users/mkelsey/anaconda/telometer/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
self.run()
File "/users/mkelsey/anaconda/telometer/lib/python3.8/multiprocessing/process.py", line 108, in run
self._target(*self._args, **self._kwargs)
File "/users/mkelsey/anaconda/telometer/lib/python3.8/multiprocessing/pool.py", line 136, in worker
put((job, i, (False, wrapped)))
File "/users/mkelsey/anaconda/telometer/lib/python3.8/multiprocessing/queues.py", line 368, in put
self._writer.send_bytes(obj)
File "/users/mkelsey/anaconda/telometer/lib/python3.8/multiprocessing/connection.py", line 200, in send_bytes
self._send_bytes(m[offset:offset + size])
File "/users/mkelsey/anaconda/telometer/lib/python3.8/multiprocessing/connection.py", line 404, in _send_bytes
self._send(header)
File "/users/mkelsey/anaconda/telometer/lib/python3.8/multiprocessing/connection.py", line 368, in _send
n = write(self._handle, buf)
BrokenPipeError: [Errno 32] Broken pipe
slurmstepd: error: Detected 1 oom_kill event in StepId=3897771.0. Some of the step tasks have been OOM Killed.
srun: error: node1909: task 0: Out Of Memory

Glad it's been working for you!
Yes, the memory / storage can be an issue and the main page doesn't really comment on it yet.
If you try to pipe minimap2 directly to Samtools view / sort the memory requirement balloon very quickly and on our HPC I have to use a memory partition with >300 GB of RAM for some files.
Splitting it up like in the instructions decreases the memory requirement, but the storage of the BAMs then becomes an issue. Two solutions: reduce the number of supplementary alignments. The script already only keeps the telomere measurement from the alignment with the highest MAPQ in the case of multiple alignments, so reducing -N to 0 should be fine -- I used to be afraid of missing telomeres because a supplementary alignment was closer to the real telomere but the main alignment was not,but I'm not sure this fear is founded. Second option (which is what I do) is to write SAM and unsorted BAM to a /tmp/ directory with very high storage and then delete them after the next processing step is done. Finally, gzip the sorted bam.
For the ease of sharing data for a future publication, I also suggest writing a simple Python script that extracts only the reads associated with telomere measurements (can use the read id column from the telometer output to help you) from the sorted bam into a new bam, sort and index, gzip, and then you have a lightweight bam that can be used for publication. Can go back and do the same for the raw fastq if you concatenate the fastqs as well. If you plan on publishing an experiment, this will save you time later.

Thank you @santiago-es !


ui: Can't use domain name in "Connect peer" modal

Current Behavior
Currently renterd webui suggests that I can use domain name in "Connect peer" modal

but after providing domain name, it says it is incorrect ip, I've used de5a9e88836fb1c4.sia.host:9881

Expected Behavior
x
Steps to Reproduce

Go to "Blockchain node"
Try to add peer using domain name

Version
b052e0c
What operating system did the problem occur on (e.g. Ubuntu 22.04, macOS 12.0, Windows 11)?
Ubuntu 22.04, kubernetes (https://github.com/artur9010/charts/tree/master/renterd)
Autopilot Config
probably not needed
Bus Config
probably not needed
Contract Set Contracts
probably not needed
Anything else?
No response

Fixed by https://github.com/SiaFoundation/coreutils/pull/67


Minor fixes for GitHub changing Markdown parser

- is no longer useful for creating lines it seems.

Thanks a lot for this PR as I could have noticed it months from now!


Compatibility with babel-transform-builtin-classes

This is an incomplete pull request. It just shows how to make the lib compatible with babel-plugin-transform-builtin-classes (and therefore not require webcomponentsjs's adapter).
This would need to be applied anywhere else that a native classes is being used.
What this does is enable transform-builtin-classes to work (it reads the class syntax), so that it will be able to patch the native extension.
End users will still have to configure the plugin to look for HTMLElement, and other native classes. We could also supply an example babelrc in this repo.
I haven't checked, but if all uses of the native class can be supplied from the end user, then it might not be necessary.
So, just opening this here in case it is helpful, but not sure if you actually want to do all these changes.

Also note, Babel 7 will include this transform builtin, but at the moment it doesn't work, or at least, there's no indication of what I have to do to make it work.

I make it work in the following browsers:
Chrome 65.0.3325.181
FF 57.0b12
FF 60.0b10
Safari Version 11.1 (136<IP_ADDRESS>.2)
Edge 16
IE 11 seems it should work but run into another issue: https://github.com/webcomponents/custom-elements/issues/135
.babelrc config:
{
  "presets": [
    [
      "env", {
        "modules": false,
        "debug": true
      }
    ],
    "react",
    "flow"
  ],
  "plugins": [
    "transform-class-properties",
    "transform-object-rest-spread",
    [
      "babel-plugin-transform-builtin-classes", {
        "globals": ["HTMLElement"],
        "logIfPatched": true
      }
    ],
    "syntax-dynamic-import",
    [
      "babel-plugin-styled-components", {
        "minify": false,
        "uglifyPure": false
      }
    ]
  ],
  "env": {}
]

In package.json:
"babel-plugin-transform-builtin-classes": "^0.6.1",

In component source:
 export default define(
   class extends withComponent(class extends HTMLElement {}) {

And including only https://github.com/webcomponents/custom-elements polyfill (w/o any adapters etc).

With custom elements on the horizon, and class support everywhere except IE11, I think I'd prefer to close this for now but it'd be good to document it and link to the PR. Happy to repopen if you disagree and we can discuss. Somethings I'd like if we were to accept this:

Docs on Babel config (v6 and v7)
Doing this everywhere we use HTMLElement

At the moment, I get around this issue by using lowclass at runtime which is an alternative to using babel-plugin-transform-builtin-classes at build time (which adds runtime output that does a similar thing).

Since my last comment, things are now in TypeScript. I'm not sure how they handle this but hope to look into it soon. Definitely prior to releasing.


Wonkwonlee patch 1

Enabled codecov

Added coverage.xml report
Added codecov action to GitHub Actions

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8904387). Click here to learn what that means.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   96.34%
=========================================
  Files             ?        6
  Lines             ?      246
  Branches          ?        0
=========================================
  Hits              ?      237
  Misses            ?        9
  Partials          ?        0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


README can be improved

Additional details such as step by step contribution guide, contributors list, etc. can be added.

@mrcoder991 I want to work on this issue, Kindly assign me.

Assigned to you go ahed..


chore(changelog): update changelog description

Summary

Checklist

[ ] The Pull Request has tests
[x] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

Fix https://konghq.atlassian.net/browse/FTI-6275

Cherrypick in https://github.com/Kong/kong-ee/pull/10531


Allow deploys to happen on PRs created from forked repository

Use Case
https://github.com/dpc-sdp/ripple is an open source repository. As an open source repo third parties are contributing changes back our repo that we want to test against a Lagoon environment.
Issue
Normally PRs against the repository create Lagoon environments as expected and are used for testing. We would like this to happen for PRs created from a forked repository as well. Currently a PR created from a forked repo has the following log message.
[2021-07-06 05:22:38] [info]: Handling github:pull_request:opened for project ripple
[2021-07-06 05:22:38] [info]: lagoon-logs: Send to lagoon-logs: *[ripple]* PR 992 opened. No deploy task created, reason: Head/Base not same repo

Is this something that can be changed or enabled easily?

Is this something that can be changed or enabled easily?

This is not a feature that's supported, so it can't be "turned on" easily. Lagoon needs access to the git repo to build the images and there isn't any guarantee that it will have permission to do a checkout from a repo that's not configured in the Lagoon project.
Supporting this would potentially allow any random user to create environments which directly impacts billing. Imagine checking in a 100GB file and opening 10 PRs, exhausting dev environments and disk space.
Maybe we could put this behind a flag, add some docs/warnings and accept that elevated build errors are a possibility?

To add to @rocketeerbkw's point. Just allowing forked repo PRs as is would be a bad idea. We would need to have more guard rails in Lagoon:

Some sort of crypto miner detection (see https://layerci.com/blog/crypto-miners-are-killing-free-ci/)
Some sort of sensible maximum time a build is allowed to run (e.g. 30 minutes)
Some sort of abuse detection (e.g. no more than X builds per hour per fork)
Better request maximums on build pod requests (e.g. CPU and RAM limits)
Security implications of running untrusted code within your project (drush @production sql-drop anyone)

There are also cost implications to be aware of.
You always can add the forked git repo as it's own Lagoon project, this has several benefits:

provides a gate for the author to pass through (so you need to 'approve' them first)
the non-production env limit is per fork, which helps to address abuse
avoids the above security implications (sandbox++)
you can also deploy them to another cluster entirely, another layer of defence against untrusted code. In AWS you can use lower cost spot instances to power the cluster, to drive the cost down

not planning to implement anything for forked repos


Update tests for the new authentication backend

The new backend is used in the tests. There could be a specialized test case for directly testing the authenticate method and checking it raises the PermissionDenied error.

I would like to contribute if possible.

@Alig1493 feel free to do so. open a PR with your changes when they are ready for review.thanks!

The new test should unit test the authentication class and the methods directly, taking inspiration from Django test things.
https://github.com/jazzband/django-axes/blob/master/axes/backends.py

I see. And will the tests be in a new file. It says update so does that mean existing test cases testing the authentication AxesModelBackend class exists already? And if not will I be writing the tests in a new file or an existing one?

Try a new file


Chat - Unable to send message from newly created account

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Expected Result:
Message should be sent
Actual Result:
Nothing happens after button tapped
Action Performed:

Open url
Login with newly created applause.expensifail account
Create chat
Paste/type text
Tap Send button

Workaround:
No workaround found
Platform:
Where is this issue occurring?
Web and mWeb✔
iOS ✔
Android ✔ (app crashes)
Desktop App ✔
Version Number:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
Video
https://user-images.githubusercontent.com/44479856/113778740-c8377480-96fa-11eb-91f3-91aaccd6e7c4.mp4

Expensify/Expensify Issue URL:

All new applause.expensifail accounts are being affected by this. Not sure if I should add the deploy blocker label since there's another issue in production.
When login in there's a blank page displayed with this JS error message.

CC @roryabraham / @AndrewGable

Oh no! 😱

I'm not sure it should be a deploy blocker since the production bug seems worse 😅

This seems like a duplicate of this issue https://github.com/Expensify/Expensify/issues/158699

Tested on staging branch and master and was not able to reproduce this. Going to update all my repos to be sure the issue doesn't exist someplace else.


How can i set width based on segment title?

In attached screenshot you can see segment title is getting cut from middle due to large title. How can i make sure that it does not get cut by applying segment width based on text?

I'm looking for a way.
First of all, I'm trying to extend the MXSegmentedControl.

I'm looking for a way.
First of all, I'm trying to extend the MXSegmentedControl.

That should be working by default!
If you do not set the segmentControl.segmentWidth property, the width is computed according the content. Do you have a code snippet to share?


DB corruption can happens in some unknown cases

We need to do an investigation

https://github.com/meilisearch/meilisearch/issues/4588

The logs that caused the panic:
updating settings
adding document
index 0: removing item 0
index 0: adding item 0 with vector [0.0, 0.0, 0.0]
index 0: building
deleting embedding
index 0: removing item 0
index 1: removing item 0
doing a search

The bug happens when you forget to .build() a writer and drop it instead.
When you create a reader from this index, later on, it panics.
It should return a user error instead.


Return null if 'val' is falsy

Make sure val is a valid String before using match by returning null on falsy values.
When using productStream I got this error:
TypeError: Cannot read property 'match' of undefined
    at Object.volumeFilter [as filter] (/Users/emilmork/Documents/dev/github/beer-api/node_modules/vinmonopolet/src/filters/volume.js:20:19)
    at /Users/emilmork/Documents/dev/github/beer-api/node_modules/vinmonopolet/src/models/set-props.js:15:38
    at Array.forEach (native)
    at setProps (/Users/emilmork/Documents/dev/github/beer-api/node_modules/vinmonopolet/src/models/set-props.js:6:10)
    at new Product (/Users/emilmork/Documents/dev/github/beer-api/node_modules/vinmonopolet/src/models/product.js:8:5)

Thanks!

:+1:

I made a (MVP) cli for searching products. https://github.com/emilmork/vinmonopolet-cli
Thats the reason i used your package. I have indexed all products in elasticsearch using getProductStream(). Using es makes search and aggregations easy (and very fast)
:)

Sweet! Looks great!

Any chance you'd release the indexing code as well? ☺

Its really not that much code, so did not bother create a repo:
https://gist.github.com/emilmork/42a2fbee8097a7fbeded
The index logic could use some async/stream work. But it work. Takes about ~5-10 seconds to complete :)


UNIFI 5.2.9 error

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version): 29.7
Python release (python3 --version): 3.4
Component/platform: UNIFI WAP
Description of problem:
When starting HASS I now get:
Traceback (most recent call last):
  File "/usr/local/lib/python3.4/site-packages/homeassistant/components/device_tracker/__init__.py", line 137, in setup_platform
    scanner = platform.get_scanner(hass, {DOMAIN: p_config})
  File "/usr/local/lib/python3.4/site-packages/homeassistant/components/device_tracker/unifi.py", line 42, in get_scanner
    ctrl = Controller(host, username, password, port, 'v4', site_id)
  File "/home/pi/.homeassistant/deps/unifi/controller.py", line 92, in __init__
    self._login(version)
  File "/home/pi/.homeassistant/deps/unifi/controller.py", line 163, in _login
    self.opener.open(login_url, params).read()
  File "/usr/local/lib/python3.4/urllib/request.py", line 455, in open
    response = self._open(req, data)
  File "/usr/local/lib/python3.4/urllib/request.py", line 473, in _open
    '_open', req)
  File "/usr/local/lib/python3.4/urllib/request.py", line 433, in _call_chain
    result = func(*args)
  File "/usr/local/lib/python3.4/urllib/request.py", line 1217, in https_open
    context=self._context, check_hostname=self._check_hostname)
  File "/usr/local/lib/python3.4/urllib/request.py", line 1176, in do_open
    raise URLError(err)
urllib.error.URLError: <urlopen error [SSL: TLSV1_ALERT_INTERNAL_ERROR] tlsv1 alert internal error (_ssl.c:600)>

Expected:
No error
Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):

Additional info:

I am on 29.7 and Unifi seems to be working for me.

Hello @frelev,
I'm also running Unifi 5.2.9 with HA 0.29.7 without any problems here.
Did you check the date/time of your system? Are you using a custom SSL certificate with a different or specify cipher?

Hi, date/time is correct on my rPi.
I upgraded Unifi from 4.8 to 5.2.9 if that can matter. 4.8 was working fine.
No special SSL that I know of. The controller is installed on my ReadyNas (intel). I have now also tried to upgrade to HASS 31-dev with the same issue.
HASS uses SSL.
Full log:
16-10-09 11:07:13 homeassistant.components.device_tracker: Error setting up platform unifi
Traceback (most recent call last):
  File "/usr/local/lib/python3.4/urllib/request.py", line 1174, in do_open
    h.request(req.get_method(), req.selector, req.data, headers)
  File "/usr/local/lib/python3.4/http/client.py", line 1090, in request
    self._send_request(method, url, body, headers)
  File "/usr/local/lib/python3.4/http/client.py", line 1128, in _send_request
    self.endheaders(body)
  File "/usr/local/lib/python3.4/http/client.py", line 1086, in endheaders
    self._send_output(message_body)
  File "/usr/local/lib/python3.4/http/client.py", line 924, in _send_output
    self.send(msg)
  File "/usr/local/lib/python3.4/http/client.py", line 859, in send
    self.connect()
  File "/usr/local/lib/python3.4/http/client.py", line 1230, in connect
    server_hostname=sni_hostname)
  File "/usr/local/lib/python3.4/ssl.py", line 364, in wrap_socket
    _context=self)
  File "/usr/local/lib/python3.4/ssl.py", line 578, in __init__
    self.do_handshake()
  File "/usr/local/lib/python3.4/ssl.py", line 805, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLError: [SSL: TLSV1_ALERT_INTERNAL_ERROR] tlsv1 alert internal error (_ssl.c:600)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.4/site-packages/homeassistant/components/device_tracker/__init__.py", line 139, in setup_platform
    scanner = platform.get_scanner(hass, {DOMAIN: p_config})
  File "/usr/local/lib/python3.4/site-packages/homeassistant/components/device_tracker/unifi.py", line 42, in get_scanner
    ctrl = Controller(host, username, password, port, 'v4', site_id)
  File "/home/pi/.homeassistant/deps/unifi/controller.py", line 92, in __init__
    self._login(version)
  File "/home/pi/.homeassistant/deps/unifi/controller.py", line 163, in _login
    self.opener.open(login_url, params).read()
  File "/usr/local/lib/python3.4/urllib/request.py", line 455, in open
    response = self._open(req, data)
  File "/usr/local/lib/python3.4/urllib/request.py", line 473, in _open
    '_open', req)
  File "/usr/local/lib/python3.4/urllib/request.py", line 433, in _call_chain
    result = func(*args)
  File "/usr/local/lib/python3.4/urllib/request.py", line 1217, in https_open
    context=self._context, check_hostname=self._check_hostname)
  File "/usr/local/lib/python3.4/urllib/request.py", line 1176, in do_open
    raise URLError(err)
urllib.error.URLError: <urlopen error [SSL: TLSV1_ALERT_INTERNAL_ERROR] tlsv1 alert internal error (_ssl.c:600)>

Seems like this is maybe not related to HASS.
Running openssl I get:
pi@HomeAssstantPi: openssl s_client -connect <IP_ADDRESS>:8443
CONNECTED(00000003)
3069359312:error:14077438:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert internal error:s23_clnt.c:749:
no peer certificate available
No client certificate CA names sent
SSL handshake has read 7 bytes and written 290 bytes
New, (NONE), Cipher is (NONE)
Secure Renegotiation IS NOT supported
Compression: NONE
Expansion: NONE
Strange....

@frelev  what happens if you try a different protocol? What is the output for the commands below:
# openssl s_client  -connect <IP_ADDRESS>:8443 -tls1_2 -showcerts -msg
# openssl s_client  -connect <IP_ADDRESS>:8443 -tls1_1 -showcerts -msg
# openssl s_client  -connect <IP_ADDRESS>:8443 -ssl3 -showcerts -msg

Which linux distribution are you using? Did you try to update the openssl as well?  Adding the -msg and -showcerts we will be able to see if the server is really being able to respond the ServerHello handshake.
What also happens if you try to access it via curl?
# curl -v https://<IP_ADDRESS>:8443 -o /dev/null

first, thank you @tchellomello for trying to help, much appreciated.
I´m running Rasbian (Wheezy), kernel  4.4.10+.
openssl is OpenSSL 1.0.1e 11 Feb 2013 which seems to be the latest version in apt.
Outputs:
tls1_2:
<<< TLS 1.2 Alert [length 0002], fatal internal_error
    02 50
3069899984:error:14094438:SSL routines:SSL3_READ_BYTES:tlsv1 alert internal error:s3_pkt.c:1261:SSL alert number 80
3069899984:error:1409E0E5:SSL routines:SSL3_WRITE_BYTES:ssl handshake failure:s3_pkt.c:599:

tls1_1:
<<< TLS 1.1 Alert [length 0002], fatal internal_error
    02 50
3069625552:error:14094438:SSL routines:SSL3_READ_BYTES:tlsv1 alert internal error:s3_pkt.c:1261:SSL alert number 80
3069625552:error:1409E0E5:SSL routines:SSL3_WRITE_BYTES:ssl handshake failure:s3_pkt.c:599:

ssl3:
>>> TLS 1.0 Alert [length 0002], fatal handshake_failure
    02 28
3069977808:error:1408F10B:SSL routines:SSL3_GET_RECORD:wrong version number:s3_pkt.c:340:

curl:
* About to connect() to <IP_ADDRESS> port 8443 (#0)
*   Trying <IP_ADDRESS>...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 8443 (#0)
* successfully set certificate verify locations:
*   CAfile: none
  CApath: /etc/ssl/certs
* SSLv3, TLS handshake, Client hello (1):
} [data not shown]
* SSLv3, TLS alert, Server hello (2):
{ [data not shown]
* error:14077438:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert internal error
* Closing connection #0
curl: (35) error:14077438:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert internal error

Since it is working for you I assume it is something with my UniFi installation that is messed up.
Strange it worked with 4.8 though.

Hello @frelev,  you are very welcome.
Based on the error it seems the server is not even able to acknowledge the ServerHello when trying to initiate the SSL handshake.
Looking at the armf for Debian Wheeze it seems that the latest version is  OpenSSL 1.0.1t  3 May 2016. https://packages.debian.org/search?arch=armhf&keywords=openssl
Try running:  apt-get update ; apt-get upgrade on your Pi to see if that fixes the issue.
If you try on your personal computer, are you able to access the Unifi webUI via https?

@tchellomello
Accessing UniFi from Chrome works (https://<IP_ADDRESS>:8443/) but I get the read line over https:.
But isn´t that just because cert isn´t signed? I have that for all sites I run local on LAN with https.
Error is:
Certificate Error
There are issues with the site's certificate chain (net::ERR_CERT_AUTHORITY_INVALID).

Running apt upgrade did indeed update openssl.
Sadly it get the same errors.
Maybe a reinstall of UniFi can help?

@frelev  The message for the certificate is expected for being self-signed certificate.
To be honest I don't think the problem is related with the certificate but on the communication between your PI and ReadyNas.
Did you try, just to test, install Unifi on your Pi to see if openssl s_client will work?  You can find the instructions here: https://help.ubnt.com/hc/en-us/articles/220066768
Once you get it installed, try running the openssl s_client again and let's see if it will work.

@tchellomello Ok, I will try to install UniFi on another server. I tried to reinstall it but no luck
Strange is, when I run the openssl test to ReadyNas on another port it is working.
I have SABNZBD installed on 9090 and handshake is working fine.
So the communication between the rPi seems to work.

@frelev
Yes, you can force the Unifi server to use a specific SSL cipher/protocol. For the other one working, is it using a different SSL cipher or protocol? Have you tried to match the same on Unifi to see if it works?
I did not change it on my setup. This is what I have:
/var/lib/unifi# cat system.properties  | grep -v ^#
data_retention_days=90
download_timeframe=-1
is_default=false
uuid=a2d565fe-<redacted>

This is the output for openssl s_client -connect <IP>:8443
---
SSL handshake has read 1396 bytes and written 495 bytes
---
New, TLSv1/SSLv3, Cipher is ECDHE-RSA-AES128-GCM-SHA256
Server public key is 2048 bit
Secure Renegotiation IS supported
Compression: NONE
Expansion: NONE
SSL-Session:
    Protocol  : TLSv1.2
    Cipher    : ECDHE-RSA-AES128-GCM-SHA256
    Session-ID: 58003D169B87D01F7AAE2C7AE1EFE7C192B8D2ADBB560E544934752D1AE630DA
    Session-ID-ctx:
    Master-Key: B3F2276B77C02E0A846391CB4833AC0CB180D411676231F4D0CD7D61DB2618F200371C6EB0149A5D21C130E3ABC94648
    Key-Arg   : None
    PSK identity: None
    PSK identity hint: None
    SRP username: None
    Start Time:<PHONE_NUMBER>
    Timeout   : 300 (sec)
    Verify return code: 18 (self signed certificate)
---

I've seem this problem already on linux system with LRO or TSO enabled. No sure if that is the case for the ReadyNAS though.

@tchellomello
I´ve not changed it either in config yet.
On 8443 I get:
New, (NONE), Cipher is (NONE)
Secure Renegotiation IS NOT supported
Compression: NONE
Expansion: NONE
SSL-Session:
    Protocol  : TLSv1.2
    Cipher    : 0000
    Session-ID:
    Session-ID-ctx:
    Master-Key:
    Key-Arg   : None
    PSK identity: None
    PSK identity hint: None
    SRP username: None
    Start Time:<PHONE_NUMBER>
    Timeout   : 300 (sec)
    Verify return code: 0 (ok)

On 9090 I have:
New, TLSv1/SSLv3, Cipher is AES256-GCM-SHA384
Server public key is 1024 bit
Secure Renegotiation IS supported
Compression: NONE
Expansion: NONE
SSL-Session:
    Protocol  : TLSv1.2
    Cipher    : AES256-GCM-SHA384

Is it the cipher I need to add? (AES256-GCM-SHA384)

@frelev  the cipher should be auto-negotiated probably but it is failing ecause the message Secure Renegotiation is Not support.
You can force a cipher via openssl command like:
# list all supported ciphers
$ openssl  ciphers
ECDHE-RSA-AES256-GCM-SHA384:ECDHE-ECDSA-AES256-GCM-SHA384:ECDHE-RSA-AES256-SHA384:ECDHE-ECDSA-AES256-SHA384:ECDHE-RSA-AES256-SHA:ECDHE-ECDSA-AES256-SHA:ECDHE-RSA-AES128-GCM-SHA256:ECDHE-ECDSA-AES128-GCM-SHA256:ECDHE-RSA-AES128-SHA256:ECDHE-ECDSA-AES128-SHA256:ECDHE-RSA-AES128-SHA:ECDHE-ECDSA-AES128-SHA:AES256-GCM-SHA384:AES256-SHA256:AES256-SHA:CAMELLIA256-SHA:AES128-GCM-SHA256:AES128-SHA256:AES128-SHA:CAMELLIA128-SHA:DHE-DSS-AES256-GCM-SHA384:DHE-RSA-AES256-GCM-SHA384:DHE-RSA-AES256-SHA256:DHE-DSS-AES256-SHA256:DHE-RSA-AES256-SHA:DHE-DSS-AES256-SHA:DHE-RSA-CAMELLIA256-SHA:DHE-DSS-CAMELLIA256-SHA:DHE-DSS-AES128-GCM-SHA256:DHE-RSA-AES128-GCM-SHA256:DHE-RSA-AES128-SHA256:DHE-DSS-AES128-SHA256:DHE-RSA-AES128-SHA:DHE-DSS-AES128-SHA:DHE-RSA-CAMELLIA128-SHA:DHE-DSS-CAMELLIA128-SHA:PSK-AES256-CBC-SHA:PSK-AES128-CBC-SHA:ECDHE-RSA-DES-CBC3-SHA:ECDHE-ECDSA-DES-CBC3-SHA:DES-CBC3-SHA:EDH-RSA-DES-CBC3-SHA:EDH-DSS-DES-CBC3-SHA:PSK-3DES-EDE-CBC-SHA

# connecting with specific cipher
$ openssl  s_client -connect <FQDN>:443  -showcerts -msg  -tls1_2 -cipher AES256-GCM-SHA384

SSL-Session:
    Protocol  : TLSv1.2
    Cipher    : AES256-GCM-SHA384
    Session-ID: 18FBA61756F0C3A2DD391B9030D19276276A4024112047E2959598099DC68AAA
    Session-ID-ctx:
    Master-Key: 6903D3D02F23F8C1722DB49CC8B59E620D4B61BA81B3AB9B96D5BF54EBDC02E8F9C07BE89DB1634617732C5A348EF5F9
    Key-Arg   : None
    Krb5 Principal: None
    PSK identity: None
    PSK identity hint: None
    TLS session ticket lifetime hint: 300 (seconds)
    TLS session ticket:
    0000 - 0f d7 d6 f3 86 9e f3 19-da 1d 50 bb 5e cf 70 b4   ..........P.^.p.
    0010 - 1e 85 b9 18 36 f9 d6 c7-b4 29 aa 20 48 46 75 3e   ....6....). HFu>
    0020 - 91 f8 ae 78 fe c4 ac 4d-f9 55 ea 9e 11 2c 5c b9   ...x...M.U...,\.
    0030 - 79 c2 50 ba c5 9a cb f0-44 e5 d6 29 a5 aa df 14   y.P.....D..)....
    0040 - 2e 0a 69 be c8 72 3e 44-54 ce 17 c3 66 99 68 33   ..i..r>DT...f.h3
    0050 - 81 24 94 1d 7a ee 70 82-f1 f9 8d e4 69 66 f3 78   .$..z.p.....if.x
    0060 - 1d 84 4a f7 3a 31 81 34-ed 71 47 99 fd 09 82 9f   ..J.:1.4.qG.....
    0070 - 20 d5 bd 70 30 e6 95 1a-a7 d3 ac 72 77 68 ec c6    ..p0......rwh..
    0080 - 3c e4 ef 73 1e bc ba e7-ba 9a 7f b7 5d 12 58 6c   <..s........].Xl
    0090 - 92 dc a0 d8 b9 b3 80 0a-a4 c7 94 66 a7 b6 47 21   ...........f..G!
    00a0 - 03 51 12 0e 3c 28 3c 63-29 24 7d ab 8a f9 b0 4c   .Q..<(<c)$}....L

    Start Time:<PHONE_NUMBER>
    Timeout   : 7200 (sec)
    Verify return code: 0 (ok)
---

@tchellomello OK, this is driving me nuts.
I tried defining the chipher, still the handshake fails but works on 9090 (the sabnzbd cert):
pi@HomeAssstantPi` ~ $ openssl  s_client -connect <IP_ADDRESS>:8443  -showcerts -msg  -tls1_2 -cipher AES256-GCM-SHA384
CONNECTED(00000003)
 TLS 1.2 Handshake [length 005f], ClientHello
    01 00 00 5b 03 03 30 72 8d 45 ae f0 b6 a2 e1 3a
    87 2d 1c c2 08 40 10 32 51 0b 9a 52 bc 07 41 82
    82 98 ec 28 dd aa 00 00 04 00 9d 00 ff 02 01 00
    00 2d 00 23 00 00 00 0d 00 20 00 1e 06 01 06 02
    06 03 05 01 05 02 05 03 04 01 04 02 04 03 03 01
    03 02 03 03 02 01 02 02 02 03 00 0f 00 01 01
 TLS 1.2 Alert [length 0002], fatal handshake_failure
    02 28
3069347024:error:14094410:SSL routines:SSL3_READ_BYTES:sslv3 alert handshake failure:s3_pkt.c:1300:SSL alert number 40
3069347024:error:1409E0E5:SSL routines:SSL3_WRITE_BYTES:ssl handshake failure:s3_pkt.c:637:
-
no peer certificate available
-
No client certificate CA names sent
-
SSL handshake has read 7 bytes and written 0 bytes
---
New, (NONE), Cipher is (NONE)
Secure Renegotiation IS NOT supported
Compression: NONE
Expansion: NONE
SSL-Session:
    Protocol  : TLSv1.2
    Cipher    : 0000
    Session-ID:
    Session-ID-ctx:
    Master-Key:
    Key-Arg   : None
    PSK identity: None
    PSK identity hint: None
    SRP username: None
    Start Time:<PHONE_NUMBER>
    Timeout   : 7200 (sec)
    Verify return code: 0 (ok)

`

@frelev  interesting.. Check it out this 2 links
http://security.stackexchange.com/questions/101965/ssl3-error-when-requesting-connection-using-tls-1-2
http://security.stackexchange.com/questions/101511/determine-if-a-server-is-asking-for-a-client-certificate-using-openssl-s-client
Did it work when installing on the PI?

@tchellomello
Reading you links I see that tls1.2 is just the maximum protocol, it doesn´t have to be the actual one used.
When comparing the certs on 8443 and 9090 in Chrome I can see that 8443 says v3 and 9090 v1.
I installed UniFi on my Windows HTPC and from there everything works.
Might run it on that server instead even if it´s a bit annoying...

@frelev  interesting..  I don't have a readyNAS to test it but I'm glad that you were able to remedy the situation.
It may be some special configuration on the ReadyNas that may be causing this issue.  Glad that at least it was not a problem on HA.
Thanks for the feedback.

@tchellomello  I´ll close the issue and thank you for taking your time with this even if it wasn't a HA problem.

@frelev  you are welcome


⚠ Samsung has degraded performance

In ff719ee, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2429 ms

Resolved: Samsung performance has improved in 2af179a.


Add table kubernetes_version. Closes #140

The table kubernetes_version will provide the information related to the version of the Kubernetes cluster's API server.
For example:
+-----------+-------------+-------+-------+------------------------------------------+----------------+---------------------------+------------+----------+-------------+----------------------------------+
| component | git_version | major | minor | git_commit                               | git_tree_state | build_date                | go_version | compiler | platform    | _ctx                             |
+-----------+-------------+-------+-------+------------------------------------------+----------------+---------------------------+------------+----------+-------------+----------------------------------+
| server    | v1.26.3     | 1     | 26    | 9e644106593f3f4aa98f8a84b23db5fa378900bd | clean          | 2023-03-15T19:03:12+05:30 | go1.19.7   | gc       | linux/arm64 | {"connection_name":"kubernetes"} |
+-----------+-------------+-------+-------+------------------------------------------+----------------+---------------------------+------------+----------+-------------+----------------------------------+

Currently, the table doesn't provide the Client version, as we are getting invalid versions while trying out the method that the Kubectl uses.
We have created an issue to track the above case: https://github.com/kubernetes/client-go/issues/1274

Closing this PR for now as we're blocked by https://github.com/kubernetes/client-go/issues/1274, will re-open once we find a workaround or that issue is resolved.


Multistep Workflows currently broken?

In the public Arbor instance, "Workflow two-step" in Arbor collection breaks during the InputsView initialization phase (failure at line 55, referredInput is null).
"Workflow one-step", also in Arbor collection,  works OK.
My private instance behaves the same as the public instance.

I'm not sure if this type of workflow ever worked. The problem is that the "columnname" input is looking for an input named "inputTable" to use to populate its drop-down menu of columns. But you do not expose "inputTable" in the workflow, instead it's being set from the "Aggregate table by average" output.
If you change the name of the input of "Filter by attribute" to "table" instead of "inputTable" and rebuild the workflow, it should work. This is a hacky workaround for now.
What we really need is a way to let the user update the domain of the "columnname" input to point to another table when building the workflow (an actual workflow input), and warn when the referenced table input will not be found. I think this is possible in the Romanesco spec, but right now we just blindly copy the sub-task input into the workflow input and don't let the user edit it's properties (including its domain). Note that there is no way to know that "Aggregate table by average" will produce a table with the same columns, so there is no way to have this "just work".

Oh, I see, this situation involves dynamic resolution of column names, etc.  I didn’t realize.  Agreed about the short term solution.  Thanks.
On Apr 29, 2015, at 8:01 AM, Jeffrey Baumes<EMAIL_ADDRESS>wrote:

I'm not sure if this type of workflow ever worked. The problem is that the "columnname" input is looking for an input named "inputTable" to use to populate its drop-down menu of columns. But you do not expose "inputTable" in the workflow, instead it's being set from the "Aggregate table by average" output.
If you change the name of the input of "Filter by attribute" to "table" instead of "inputTable" and rebuild the workflow, it should work. This is a hacky workaround for now.
What we really need is a way to let the user update the domain of the "columnname" input to point to another table when building the workflow (an actual workflow input), and warn when the referenced table input will not be found. I think this is possible in the Romanesco spec, but right now we just blindly copy the sub-task input into the workflow input and don't let the user edit it's properties (including its domain). Note that there is no way to know that "Aggregate table by average" will produce a table with the same columns, so there is no way to have this "just work".
—
Reply to this email directly or view it on GitHub.