Incorrect kernel parameters with sparse models

I've noticed the following behaviour using sparse GPs that I'd like to understand better. I generate some data to fit as follows
kern=gpflow.kernels.Matern12(1,lengthscales=5.0,variance=1.0)
N=500
X = np.linspace(0, N, N)[:,None]
K = kern.compute_K_symm(X)

Y = np.random.multivariate_normal(np.zeros(N), K, 1).T
Y += (0.1)*np.random.normal(size=Y.shape)

As a sanity check I use a sparse GP with half the data and the same number of inducing points
sgpr = gpflow.models.SGPR(X[::2], Y[::2], kern=gpflow.kernels.Matern12(1,variance=1,lengthscales=5), Z=X[::2].copy())
sgpr.likelihood.variance = 0.01

gpflow.train.ScipyOptimizer().minimize(sgpr)
sgpr.as_pandas_table()

As expected this gives the same results as GPR and I recover the correct parameters:

SGPR/feature/Z 	Parameter 	None 	(none) 	True 	(250, 1) 	True 	[[-3.1508541606040293e-06], [2.004008026951309...
SGPR/kern/lengthscales 	Parameter 	None 	+ve 	True 	() 	True 	5.008733368270208
SGPR/kern/variance 	Parameter 	None 	+ve 	True 	() 	True 	0.9601464571035354
SGPR/likelihood/variance 	Parameter 	None 	+ve 	True 	() 	True 	0.010013976529520336

Next using all the data and half as many inducing points
sgpr = gpflow.models.SGPR(X, Y, kern=gpflow.kernels.Matern12(1,variance=1,lengthscales=5), Z=X[::2].copy())
sgpr.likelihood.variance = 0.01

gpflow.train.ScipyOptimizer().minimize(sgpr,maxiter=2000)
sgpr.as_pandas_table()

this gives

SGPR/feature/Z 	Parameter 	None 	(none) 	True 	(250, 1) 	True 	[[2.499022449377244e-06], [2.004008178124323],...
SGPR/kern/lengthscales 	Parameter 	None 	+ve 	True 	() 	True 	14.086365818290883
SGPR/kern/variance 	Parameter 	None 	+ve 	True 	() 	True 	0.8030125251127183
SGPR/likelihood/variance 	Parameter 	None 	+ve 	True 	() 	True 	0.2255555730664038

So even though I initialised with the known parameters the optimisation moves away. I've tried this with other optimizers (Adam, Adagrad) and with SVGP using natural gradients and get the same thing. I made a plot of the objective function value and the kernel lengthscale during training with Adam for the first case using half the data and the same number of inducing points here:

and the second case where the lengthscale keeps increasing here:

But if I use an RBF kernel I get the correct behaviour in both cases. Is this a numerical stability issue, and there's something about the roughness of the matern12 kernel that leads to this behaviour? Or maybe I'm implementing the sparse GPs incorrectly. Any advice would be appreciated.
I'm using gpflow 1.3, tf 1.10.1

This is to be expected. The problem is that slack in the bound is not uniform over hyperparameter settings. There is no reason that argmax(ELBO) is the same as argmax(true marginal likelihood), where the argmax is wrt model hyperparameters. If the ELBO is used as a proxy for the marginal likelihood then models will be learned that make the sparse approximation better, e.g. long lengthscales.

thanks Hugh - so in general are you saying the sparse methods can't be used to learn model hyperparameters? i.e. only learn hyperparameters on a subset of the data with a full gp?

Sparse methods certainly can be used to learn hyperparameters, it's just that there's no guarantee that they will be the same hyperparameters as the non-sparse model. The bias introduced through optimizing the ELBO rather than the ML will favour models that are well approximated by the sparse posterior, which is likely to be a good thing if computation constraints force the use of a sparse model in the first place. NB subset of data is likely to be worse.


Fix annotation start and length type

Exexample of date annotation returned by the data node.
    {
      "createdAt": "",
      "createdBy": "",
      "format": "",
      "id": 258430,
      "length": "10",
      "noteId": 11583,
      "start": "16",
      "text": "2077-03-31",
      "updatedAt": "",
      "updatedBy": ""
    },

length should be a number
start should be a number

Fixed


[Dataset] Fix CoLA dataset index and add Glue dataset API

Description
Added Glue datasets to the dataset API. Will refactor scripts/bert/dataset.py in the following PR.
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage
[x] Code is well-documented

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

Job PR-682/13 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-682/13/index.html

@Gpwner could you help review the CoLADataset class?


Add some support for atomic and volatile methods on map types

I won't be disappointed if you don't accept this patch; it uses VarHandles, which were introduced in JDK 9, so if support for JDK 8 and earlier is important, you likely don't want this, at least in its present form. If these features are wanted but in a way that does not break JDK 8, one solution might be to use the template system to generate different code for JDK 8 versus 9 and only support these new methods for JDK 9.
My idea is that, because of the design of the index methods such as indexGet and indexReplace, I could extend the design to allow some limited concurrent operations on hash map types, permitting multiple threads to read and write values in parallel with well defined semantics. The main restriction is that while threads are using these new methods, no other thread may perform any action that would change the indexes of values, operations such as put or delete for example.
The new methods are indexGetVolatile, indexSetVolatile, indexReplaceVolatile and indexCompareAndExchange.
Here is an example of use. It follows the common CAS pattern of reading a variable with volatile semantics, preparing assuming its value will not change, then committing with an atomic CAS operation, abandoning and retrying if, due to concurrent operations, the assumption was incorrect.
// Multiple threads can run this concurrently on the same hash map.
// field "elements" is of type LongObjectHashMap<Element>
fun Version.upgradeToWritable(element: Element): Element {
    assertHasSharedLock()
    val id = element.id

    // Index will not be invalidated inside enclosing function because we have ensured that no
    // such invalidating operations will occur by some means, specifically by acquiring a shared
    // lock in this example.
    val idx = elements.indexOf(id)

    while (true) {
        // Check to see if another thread accomplished the upgrade before this thread. Note that if this
        // only called indexGet, the loop could hang indefinitely.
        val existingElement = elements.indexGetVolatile(idx)
        if (existingElement.version === this) {
            return existingElement
        }

        // Speculatively upgrade
        val speculativeElement = element.shallowCopy()
        speculativeElement.version = this

        // Only make the the speculative upgrade visible to other threads if the initial assumption was correct.
        // Otherwise retry.
        if (elements.indexCompareAndExchange(idx, existingElement, speculativeElement) === existingElement) {
            return speculativeElement
        }
    }
}

There are other operations I could have added and can if wanted. The reason I avoided adding indexGetAcquire and indexSetRelease is because they would be redundant. One can get the same effect by using VarHandle.acquireFence and VarHandle.releaseFence combined with the existing indexGet and indexReplace.
Technically, indexGetVolatile and indexSetVolatile are redundant too because one can get the same semantics by combining them with VarHandle.fullFence. However, some processors have special instructions for volatile access that might not be used if the volatile access is not explicit. At least, Doug Lea said so.
I included only the atomic operations that make sense for all types of value. For example, indexGetAndAdd would have been quite useful for integer valued hash maps but does not make sense for reference type valued ones.

It is great to see that people (like you) can build something on top of HPPC and customize it to their needs. Even if the patch doesn't get incorporated, it still has value. And I like the patch - it's elegant and clean. This said, I'm not sure whether we should add them to the baseline HPPC - this seems to extend the code in a direction very few people feel comfortable dealing with (lockless data structures, memory-fences to prevent data corruption in racey conditions). Maybe if you could provide an example of how this can be used then it'd be easier to digest and take advantage of?

It's okay. Would rather not fork but anticipated it might be necessary.


$mod+Return not working

I am on Ubuntu 19.10, $mod+Return does not work.

By default the $mod button is Alt left, are you pressing ALT + Enter or Super + ENTER?

By default the $mod button is Alt left, are you pressing ALT + Enter or Super + ENTER?

Thanks it works.


Add delete-task feature

This should be exposed as a button in the task list, and require a confirmation dialog.
Note: this can fail with multiple kinds of errors (typically due to missing authorization), and the error should be visible to the user.

I'm removing high-priority label, in view of new entries in https://github.com/fractal-analytics-platform/fractal-web/labels/High Priority which are definitely more urgent than this one.


Add naoqi tactile py and launch

I added naoqi_tactile related code for Pepper. This program is almost same as nao_tactile.py, and tactile.launch in nao_apps directory, but I added back bumper feature and hand touch detection.
The node name is /naoqi_tactile.
It publishes  /tactile_touch, /bumper and /hand_touch.
This change requires other pull-requests about HandTouch.msg, Bumper.msg in naoqi_bridge_msgs. ([ add back bumper value for Pepper #3 ] (https://github.com/ros-naoqi/naoqi_bridge_msgs/pull/3), [ add HandTouch.msg for hand touch detection #4 ] (https://github.com/ros-naoqi/naoqi_bridge_msgs/pull/4))
I tried it on Naoqi 2.3 version Pepper, linux 14.04, and ROS indigo.
If there is any problem, please let me know.

ok, I guess we can delete nao_tactile.py then ? Did you remove any functionality ?
So all that looks good but it is the old style to make a module. We created an abstraction (NaoqiNode) that makes things easier to code (like ip parameters and shutdown).
Can you please try to see if you can get that working ? A good example is here: https://github.com/ros-naoqi/naoqi_bridge/blob/master/naoqi_driver_py/nodes/naoqi_joint_states.py#L50
In the end, a lot of code will be removed: the command line arguments, the creation of self.broker, no call to ALModule._init but to NaoqiNode,init, and the memProxy should be created with https://github.com/ros-naoqi/naoqi_bridge/blob/master/naoqi_driver_py/src/naoqi_driver/naoqi_node.py#L128
Thx

Any way you can take the comments into account ? Thx.

@kochigami , please take the comments into account.

I am very sorry for my late replying.
I used getData instead of subscribeToEvent because I don't know how to use a callback function in NaoqiNode, which inherits from Thread class.
I tried my program with
Nao (version <IP_ADDRESS>) and
Pepper (version <IP_ADDRESS>).
My concern is this warning message
[WARN] [WallTime:<PHONE_NUMBER>.174728] Inbound TCP/IP connection failed: connection from sender terminated before handshake header received. 0 bytes were received. Please check sender for additional details.

after I launched tactile.launch and touched sensors (ex. bumper) of a robot many times in a fast speed.
When I ran rostopic echo /bumper (/hand_touch, /tactile_touch), I failed to subscribe some data.
I am very sorry for my trouble again.

I will close this because naoqi_driver package publishes topic related to ALTouch.
I am very sorry to have left this pull-request for a long time.


Add package to pip

It would be great to make this package pip friendly.

That's a very reasonable request. I'll put it in practice later this week.

It should be "easy" to upload the package to pip following those instructions.


What does the first parameter '0'/  '1' (in train.txt ) mean? I notice different samples have different multi-view loss weight ,because of this parameter.

I am trying a different multi-view face dataset. But I don't know how to set this parameter . It seems that the value of this parameter is higher,'2', when the input three images have similar lighting environment. I am not very clear about this. Could you tell me how to set this parameter? @jiaxiangshang ,thanks !

Not all multi-view face data is suitable for multi-view losses, '0' is monocular and "2" is the most suitable one.

I understand. Thanks!


Don't access memory before checking the correct length in e_aes_hmac_sha256.c

Works for master and 1.1.0 but needs some handwork for 1.0.2

Merged. Thanks for report/request!


Lab_3 fixes

I added a commented outline of the code for the lab so it's clearer how to submit in one r chunk, and what sections need to be commented out.

Yep! I will fix that and upload it to blackboard. I will probably submit that PR during my office hours.
Thanks Mike!

From: Michael Dietze @.>
Sent: Friday, September 17, 2021 11:33 AM
To: mdietze/EE375 @.>
Cc: Mccabe, Tempest @.>; Author @.>
Subject: Re: [mdietze/EE375] Lab_3 fixes (#3)
@mdietze requested changes on this pull request.
I'd definitely recommend keeping the ,echo=FALSE in the r code chunk, otherwise you get all the code on the screen too instead of just the app.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/mdietze/EE375/pull/3#pullrequestreview-757601353, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFCH6F3KTEKPBGEEIE2GTTTUCNNT3ANCNFSM5EHSZWMQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


promote numpy

promotes numpy to latest 1 version.
Not sure if this will break anything on your end. If not, the promotion would be greatly appreciated as I have other dependencies  requiring numpy 1.26.4

I think it would also be nice to promote librosa. Will make a suggested change.


refactor(NODE-3157)!: update find and modify interfaces for 4.0

Description
What changed?
BREAKING: Legacy fields option no longer supported, use projection instead.
NODE-3157 Updated public facing option interfaces for find and modify operations: findOneAndDelete(), findOneAndReplace(), findOneAndUpdate(). Standardized option parsing and internal representation of find and modify operations.

Reviewers: please compare https://github.com/mongodb/node-mongodb-native/pull/2796 for the "light" version of these changes


Added dialogs property of AdaptiveDialog to the hidden list

Fixes #5566
Description
Hide the dialogs property of AdaptiveDialog in the Composer UI by adding it to the hidden list in the AdaptiveDialog.uischema file.
Note that as of today this change will not be used by Composer, it is using a local copy of a uischema file that drives it's behavior, so another Issue and PR were created for this in the Composer repo:
AdaptiveDialog has dialogs property added in declarative schema and that property should be hidden.
fix: Added dialogs to hidden list for AdaptiveDialog in index.ts file
The change was also added here so that once Composer stops using the local file this change will already be reflected in the SDK AdaptiveDialog.uischema file.
Specific Changes
Added "dialogs" property to the AdaptiveDialog.uischema file.
Testing
This can't be tested yet today as Composer is not using the properties in the SDK uischema file yet, but the change was done in a consistent manner as the other hidden properties in the AdaptiveDialog.uischema were also done.

Pull Request Test Coverage Report for Build 258839

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

Files with Coverage Reduction
New Missed Lines
%

/libraries/AdaptiveExpressions/BuiltinFunctions/GetNextViableTime.cs
1
90.91%

/libraries/AdaptiveExpressions/BuiltinFunctions/GetPreviousViableTime.cs
1
90.91%

Totals

Change from base Build 258745:
0.0%

Covered Lines:
23118

Relevant Lines:
30370

💛  - Coveralls


Fix selected tab working with RTL

This PR fixes an issue where the Tabs component would render incorrectly when a device is set to RTL mode

LGTM


CDI College

CDI College (Official Website) : https://www.cdicollege.ca/
Students e-mails are : @cdi.college
Old pull request : https://github.com/JetBrains/swot/pull/14993
Note :
College was added in 2021 but e-mail domain has changed since for the students.
College students now use Office 365 Pro e-mails.
Old e-mail domain added was  : lib/domains/ca/collegecdi.txt
Teachers still use this domain but not the students.
Thank you !

@philipto All modifications have been done for a couple days now. Wishing you a good day.

@DeeZeeDeez Would you please provide us with a proof that confirms that

Students e-mails are : @cdi.college

There is no @cdi.college addresses mentioned anywhere at the official website and even Google. Do you have any proof?
Please provide us with a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

@philipto E-mail sent to the college administration. Waiting the official letter from them.
Thank you.

Hi, @philipto !
I have uploaded the college letter as proof.
Have a nice day.
Thank you.

@DeeZeeDeez lib/domains/college/CDI_College_Confirmation.pdf: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@philipto
Deleted the file from the folder.
Pasted a picture of the letter directly.
Thank you!

@DeeZeeDeez Thank you for your patience while waiting for a review and processing your request. Pull request merged.

Thank you @philipto !!
How many time it takes to be synced with the main JetBrains website !?
Just tried to create my student account and it doesn't work.
Have a nice day.


x/tools/gopls: extract-method is slow in eglot (but not VS Code)

It routinely takes eglot 13s to apply an extract-method fix that is almost instant in VS Code. (This happens even when the ApplyFix downcall has been patched to set async: true.) The server's computation of the diff is fast but the client's application of the diff is slow. Clearly something is amiss in eglot. The task of this issue is to identify the cause and file an upstream bug, because whatever it is likely affects a number of other eglot operations as well.
(For the sake of reproducibility, I was extracting the block at L1175 of the https://go-review.git.corp.google.com/c/tools/+/531695/1/internal/refactor/inline/inline.go.)

Until there is evidence that this is a gopls bug, I'm going to put this in the backlog.

Until there is evidence that this is a gopls bug, I'm going to put this in the backlog.

It's not a gopls bug; this issue is just a reminder to follow up with an eglot bug that impairs gopls usability.


Add build flag to disable hlint dependency

Allow hsdev be built without hlint dependency.
Related to #76

Moreover, hlint fails to build due to an ambiguous occurence of escapeHTML at version 2.0.15.  It was fixed in versions greater than 2.1., but the hsdev.cabal prevents newer versions from being used, so cabal install hsdev doesn't work here.

Added in hsdev-<IP_ADDRESS> branch


Front Page Makes No Mention of non-Node Mocha Use

from mochajs/mocha#1387
summary:

Mocha is a great test framework for Node projects, but it's also a great test framework of web projects. However, the front page of the Mocha site makes no mention whatsoever of the web-based usage, which is somewhat confusing for non-Node users:

Installation

Install with npm:

$ npm install -g mocha

If you just took a few lines to say something to the effect of "download mocha.js and mocha.css from the GitHub page here, then include both files on an HTML page before including your test files" I think it would be really helpful to non-Node users.

I think I may have addressed this fully in recent updates, but someone let me know if it's still unclear


autowatch: false with singleRun: false don´t work

I clone karma-browserify example and set singleRun to false but test never start. It stuck in this line:
28 08 2016 00:00:28.400:DEBUG [karma]: A browser has connected on socket /#Eq842Snvk0BAzXC8AAAA
28 08 2016 00:00:28.403:DEBUG [web-server]: upgrade /socket.io/?EIO=3&transport=websocket&sid=Eq842Snvk0BAzXC8AAAA
28 08 2016 00:00:28.439:INFO [PhantomJS 1.9.8 (Mac OS X 0.0.0)]: Connected on socket /#Eq842Snvk0BAzXC8AAAA with id 8306171
28 08 2016 00:00:28.440:DEBUG [launcher]: PhantomJS (id 8306171) captured in 1.667 secs

Setting autowatch to  true works fine, test get passed.

We have got a test case that verifies this works. With v5.1.0 at least.


Adding custom properties

Is it possible to define custom gramm properties, for example in a startup script? I use a few specific calls to gramm that are too long to remember and a bit cumbersome to copy-paste, e.g.
g.stat_fit('fun', @(a,b,g,l,x) g+(1-g-l) * (1./(1+exp(- ( a + b.*x )))), ...
'StartPoint', [0 0.1 0.1 0.1], 'geom', 'line', 'disp_fit', false, 'fullrange', false);
could be shortened to e.g. g.custom_psychometric()
What would be your recommended way of doing this?
Thanks for a great toolbox!

Hi Anne,
Given the way Matlab's object oriented programming works I'm not sure it would work exactly the way you want (the method would need to be declared in gramm.m). But It should be OK to create a function or anonymous function that can rather be called this way custom_psychometric(g).
Have you tried something like that ?
custom_psychometric = @(gramm_obj) gramm_obj.stat_fit('fun', @(a,b,g,l,x) g+(1-g-l) * (1./(1+exp(- ( a + b.*x )))),...
'StartPoint', [0 0.1 0.1 0.1], 'geom', 'line', 'disp_fit', false, 'fullrange', false);

custom_psychometric(g)

That works great, thanks!


Fails To Launch

When I launch Fusion through the launch.sh it asks to check for updates but doesnt launch the app, I've changed the launcher to FusionLauncher.exe but it says this:

and here is the last lines of the terminal

NVidia 1650Super
Ubuntu 23.10
Gnome 45.2

Additional context
Add any other context about the problem here.

I have the same error. I just did a fresh install of Fusion because I somehow killed the old installation. After installing I was greeted with this error. Also, there doesn't seem to be a Fusion360.exe at the install location.

This is probably a duplicate of #411.


Remove --block-gas-limit from README.md

The --block-gas-limit flag doesn't seem to work as a CLI argument for anvil (at least on macOS). The --gas-limit flag works and is already included below. This PR just removes mention of --block-gas-limit.

@all-contributors add @khelmy for maintenance


5147-script-tags-in-entity-description

Closes #5147
Usage:
After creating the viewer, set the viewer.infoBox.allowScripts to true.
Then in entity.description add an HTML with script tag.
Shown in sandcastle example ScriptInInfoBox

@YonatanKra thanks for the pull request!
I noticed that CHANGES.md has not been updated. If this change updates the Cesium API in any way, fixes a bug, or makes any non-trivial update, please add a bullet point to CHANGES.md and bump this pull request so we know it was updated. For more info, see the Pull Request Guidelines.
I am a bot who helps you make Cesium awesome! Thanks again.

Can anyone help - why the build fails?
It fails on tests that aren't related to my code.
Moreover - when I run the failed tests specifically, they pass...

Thanks for the pull request @YonatanKra!  You're right, the test failures coming up on travis don't look related to the code changes you made.

Yea, just before the latest attempt.
https://github.com/AnalyticalGraphicsInc/cesium/pull/5764/commits/33604299252dac27f4e778bee9ee6111067313a6

Even more strange - I've checkout master locally and ran all tests - they failed.

@ggetz do you want to review this?  See if you can figure out what's going on with the test failures.
Thanks!

I look at this test, for example:
it('throws if container string is undefined', function() {
        expect(function() {
            return new HomeButton('testElement', scene);
        }).toThrowDeveloperError();
    });

I can't tell why this code should throw an error.  It would probably fail all the time, since the string itself is defined ('testElement') and it passes the only throwing test in the HomeButton function:
https://github.com/AnalyticalGraphicsInc/cesium/blob/7213d4d25e02deab085448250c5dca966f1dbfcc/Source/Widgets/HomeButton/HomeButton.js#L31
Am I missing something?

I've found the HomeButton issue - I didn't clean up the tests DOM after inserting a 'testElement' and it seems the HomeButtonSpec was relaying on the DOM to not have an element with id="testElement".
I still get errors locally on other things, but travis seems to find them passing, so I guess it is something on my local machine.

Finished everything except the sanitization part. See https://github.com/AnalyticalGraphicsInc/cesium/pull/5764/commits/a3b7f57c71ef357bcd8175e17530343928f3f811

Thanks @YonatanKra!
@hpinkos This looks good to me.

Thanks for the PR @YonatanKra I have some comments so I'm going to do a review now.

This does not appear to work in IE11, even though IE11 supports createContextualFragment.  Any ideas?

It doesn't work in Edge either. (But Firefox and Chrome work).  Are we using the most accepted technique for injecting the scripts?
Can someone try Safari please?

@YonatanKra thanks again for the PR, I think that's all I have for this round of comments.  This will be a great addition once we work out the minor issues.

We messed around with IE11 scripts and they were very temperamental.  It appeared that the frame object had to be created with allow-scripts sandboxing enabled, and it didn't respond well when the sandbox was changed after the frame had been added to the DOM.

Thanks for the thorough review. I'll work on it today or tomorrow.

Regarding the IE issue - I've been wondering - what is the reason the content is inside an iframe?

Originally infoboxes were sanitized using Google Caja sanitizer, which would look at the box contents and clean them of malicious content.
Later that was changed to regular HTML iframes, as support for iframe sandbox properties increased.  The infoboxes are sandboxed to protect against certain malicious attacks which could be executed through the infobox.
See #2512 for the pull request originally implementing iframes.  #2430 also has some discussion of different sanitization strategies, but predates iframes.

Current solution works on Opera as well.  So our issue is IE (as usual...).
How about handling IE separately inside the string2Html method? Is this an acceptable solution?

@mramato @denverpierce @ggetz
I've changed the code according to the spec outlined in: https://www.w3.org/TR/2011/WD-html5-20110525/the-iframe-element.html

Thanks @YonatanKra
@mramato Did you want to take a final look?

@ggetz, yes I will take a look some time this week.

I did a quick review in IE11 and it appeared to work as intended.

I don't think the previous frame gets removed correctly in all cases.
When directly going from one entity to another (ie viewer.selectedEntity = entity1; viewer.selectedEntity = entity2), the second entity's frame contents get appended to the first entity's frame, without the first's being cleared out.
This doesn't happen if you click an entity, click the empty map space, and click the second entity.

@denverpierce I've fixed https://github.com/AnalyticalGraphicsInc/cesium/pull/5764#issuecomment-331568416.  Thanks for noting this one :)
Still can't reproduce https://github.com/AnalyticalGraphicsInc/cesium/pull/5764#pullrequestreview-64709399.

LGTM.  @mramato any chance this could get a checked out for 1.39?

Sorry I didn't get to this in time for yesterday's release.  I have some time carved out this Friday to get this merged.  Thanks.

Thanks Matt!

Thanks again for your contribution @YonatanKra!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.
:earth_africa: :earth_americas: :earth_asia:

Thanks for all the work you put into this @YonatanKra, but I don't think this is a change we want to take.  The settings we have on the InfoBox are for security reasons, and we want to disable them.
Thanks again for the contribution!


Please fix URL in SPEC

I have noticed that SPEC has a wrong URL (example.com). Please fix it.

I have a number of them. I believe this was fixed in https://github.com/poetaster/harbour-bassoontracker/commit/3c1ed9ced311e5200eefa178bcda88ab76d25ff7


[release-4.13] OCPTOOLS-320, OCPTOOLS-357, OCPTOOLS-280: CVE fixes

Fixes the following CVE(s):

CVE-2024-24786
CVE-2024-21626
CVE-2023-48795

Fixes included in this PR are cherry-picked from #1789  #1797 #1788 to resolve merge conflicts altogether.

/assign @sayan-biswas

/lgtm
/label backport-risk-assessed
/label cherry-pick-approved


Where is index.php

Just trying this out, and it won't run. There is no index.php file and htaccess directs nowhere. Am I missing something? I did run composer also.

That's my fault for not providing installation instructions. Your web server should serve the public directory as root. The .htaccess at the root the repo needs to go under public. I forgot to move it when I did the big redesign a while back.

I think there is a way to get htaccess to redirect to the public folder but I would have to search for it. If you know it off the top of your head let me know.

It would be easier to just move .htaccess and serve public. The directory structure is designed so only the public folder is outside facing keeping the rest of the application protected (to an extent).


error with cargo run "rust current package believes it's in a workspace when it's"

i 've this prob' with "windows" && "cargo run --release --example alpha" :
"...rust current package believes it's in a workspace when it's..."
"my" solution, into cargo.toml, add:
[workspace]
members = [ ]

I don't seem to have any problem building for Windows.
I have noticed an issue with rust-analyzer though, if you're working on a file that's not currently part of your project you may see a prompt to configure rust-analyzer. If you click that prompt it'll generate "${workspaceRoot}/.vscode/settings.json" and configure rust-analyzer, which can cause this error message.
Please check if you have a .vscode/settings.json file with any config for rust-analyzer, try remove it and see if you can build.


ContentDialog Text Color Contrast

Describe the bug
The color contrast of hyperlinks and text in a ContentDialog does not meet the color contrast ratio of 4.5:1 for accessibility.
https://github.com/microsoft/terminal/issues/5395 is dependent on a fix for this issue.
Steps to reproduce the bug
This can be seen in Windows Terminal's TerminalPage.xaml , but this just uses the built-in theme dictionaries.
Steps to reproduce the behavior:

Open your app of choice that uses a ContentDialog using HyperlinkButtons
Open Accessibility Insights (download link here) to check the color contrast ratio
Verify whether the color contrast ratio for the SplitButton control meets the 4.5:1 contrast ratio

Expected behavior
In Windows Terminal, the links present in the "About" pop up window and the text to the back ground color in the "OK" button should meet the color contrast ratio 4.5:1
Screenshots

Version Info
NuGet package version:

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop

Mobile

Xbox

Surface Hub

IoT

@carlos-zamora  Is the blue color, the default blue shell accent color ?

@carlos-zamora Is the blue color, the default blue shell accent color ?

Yes. This one was another one submitted by the accessibility testers. Personally, I feel that using the accent color is correct, but I get how that could create a situation like this where the color contrast is not acceptable.
CC'ing @DHowett-MSFT to keep him in the loop.

When choosing an Accent Color, it does warn where there could be visibility problems - so if the user has chosen such a colour - then we can presume they take that risk.
The Windows team should have chosen an accent colour that meets contrast ratio recommendations right?

@chigy can you route this bug ?

We are aware of this issue and will be sure to share this as additional information to the team that owns addressing this issue.

This issue is now fixed in the OS (by internal bug). Corrected accent color will be the default in the next version of Windows.

This issue is now fixed in the OS (by internal bug). Corrected accent color will be the default in the next version of Windows.

So this is the end of
00 78 D4
Looking at the design tokens, it looks like the way the Accent colour is applied to Light and Dark themes will be changing.  Any chance you could share some information about this new formula for picking colours?

@YuliKl ?


Unordered list doesn't render inside a table cell

Steps to Reproduce
Compare the markdown source for this page: https://github.mit.edu/ocw-content-rc/15.s21-january-iap-2014/blame/main/content/pages/readings.md#L57-L59
*   ![This resource may not render correctly in a screen reader.](/images/inacessible.gif)[_Virtual Ink_ Executive Summary (PDF)](http://nutsandbolts.mit.edu/Virtual_Ink/VirtualInk.pdf)
*   _![This resource may not render correctly in a screen reader.](/images/inacessible.gif)_[_Virtual Ink_ Plan: Company and Product Section (PDF)](http://nutsandbolts.mit.edu/Virtual_Ink/VirtualInk_CompanyProduct.pdf)
*   ![This resource may not render correctly in a screen reader.](/images/inacessible.gif)[_Virtual Ink_ Plan: Market Section (PDF)](http://nutsandbolts.mit.edu/Virtual_Ink/VirtualInk_Market.pdf)

with the rendered HTML: https://ocwnext-rc.odl.mit.edu/courses/15-s21-nuts-and-bolts-of-business-plans-january-iap-2014/pages/readings/

Expected Behavior

The rendered HTML should be unordered list, with bullets instead of asterisks

Actual Behavior

The original markdown asterisks are passed through without being rendered.

I thought I was looking at an up-to-date published version of the course, but now I'm not sure.

This particular issue is fixed, although this table is still kind of a mess.


AudioCueEditor issue with JJBA:ASBR v_charasel.acb

When opening the file there are duplicates of an unwanted sound (Dio Brando's first select quote, the first track in the archive) in all of the cues, adding 2 more to the should be 2 total sounds per cue.
After saving and testing in game, all of the new character cues added to the R release go silent, and Will Zeppeli's (1zpl) cue get's shared with Jonathan Joestar's (1jnt) cue.
Here are the two files.

I see whats wrong here.
This ACB has some dummy values that ACE is seeing as additional tracks, and so is adding track 0 to those cues.
This shouldn't be difficult to fix, but it does make me wonder what will happen if those dummy values aren't there. Hopefully it just works normally.

ACE Test.zip
This version reads the tracks correctly, but on save it wont add those dummy values.
If there's still trouble with the tracks being silent then I can try to add them when saving, but if its not needed then I'll just leave it as is.

The extra tracks are indeed gone, however every cue after the "V_charasel_8jsk01_02" cue are still silent in game, unfortunately.
Thank you for the support, I really appreciate it.

ACE Test2.zip
Any difference with this build?

Unfortunately, no difference, still silent in game.

ACE Test3.zip
Well then, I guess its time to try adding those extra values in. I didn't think they mattered but there's nothing else to try right now. I can't see any other issue with the acb.

Still does not seem to work, would the file size bump on the .acb have anything to do with it?
More or less I'm just kind of curious about it, since it goes from 24kb to 37kb after a save with no changes.
Again, I super appreciate the support.

v_charasel.zip
Actually, that might just be it. I've noticed that your ACB lacks some data that ACE always automatically regenerates, so when its saving it creates this data. Which might just explain the silencing.
In the ACB posted above I've removed this data through manual editing, after having saved it with ACE. Let me know if it still has the issue.

Yup, that seem to have done the trick! The characters after 8jsk are no longer silent.

Ah nice. I thought that might do the trick.
Now just to be sure before I close this issue, does this build work correctly?
ACE Test.zip

Ah, I really apologize, I'm pretty sure I imported the wrong file when trying to test the files you gave me, it's still silent..
If there's nothing that can be done at this point I'm okay with giving a shot at trying to take apart the the AWB manually and reassemble it filling in the spaces for the HCAs.

Well that is unfortunate.
But I did notice two more slight differences between the original and saved ACB which are worth testing I think. I've edited them back to the original state in this saved ACB.
v_charasel.zip

Still no luck..

Alright... well there is one last thing to try.
Basically the cue names in ACE are always saved sequentially, but in some ACBs (such as yours) they appear out of order. The cue “V_charasel_8jsk01_02” is where the difference starts, which is the cue you say is the first silent one.
v_charasel.zip

Alright, I made sure to double check I was importing the right things this time, and yes! It does work, nothing is silent.

That's good to hear. I've updated ACE to now properly preserve the original name order, as long as no cues get deleted.
Give it a try and see if it works as well.
ACE Test.zip

https://user-images.githubusercontent.com/92841790/192167518-1113642f-3803-49f5-aba5-2a947c3d03ef.mp4
It works perfectly! Thank you so much for the help, once again.


[Feature] Events, Alarms...

Handle set events, reset events, alarms state update, alarms expiration... From the generated trace of Trampoline.

[ ] GanttPlot method for plotting a set event
[ ] GanttPlot method for plotting a reset event
[ ] Add the corresponding methods to the Axis when a set/reset event is read from the Trace


Remove the backend folder

@mplabs You mentioned that the backend folder is not really needed. Do you mind if I create a branch and delete it?


Filetype plugin disabled on Neovim

Describe the issue
Filetype plugin not recognised on Neovim (v0.9.1), i.e. none of the mappings work.  Continues to work on vim.
To Reproduce

Open groff file using neovim
Enter <leader>o

Neovim will enter insert mode and not open viewer - as if plugin was not there.
Expected behavior
Viewer to open.
Screenshots
N/A
Desktop (please complete the following information):

OS: Manjaro 23.0.0
Neovim v0.9.1

Additional context
This is almost certainly an issue with the disabling option code:
if exists("b:did_ftplugin")
	finish
endif
let b:did_ftplugin = 1

If the variable name is changed then it will work in both vim and neovim.  Does neovim think a plugin for this filetype has already been activated (i.e. b:did_ftplugin= 1) and therefore this plugin is disabled?  However running echo b:did_ftplugin gives a value of 1 on both vim and neovim when running with --clean flag - why therefore is vim not disabling this plugin and neovim is?  Fix is to change the variable name, which does not seem to have any negative consequences, but unsure why neovim is now behaving this way.  May submit an issue with neovim - but need to determine exactly what problem is first.

Fixed in release v2.0.2.

Apparently now fixed in Neovim:
https://github.com/neovim/neovim/issues/24781#issuecomment-1683830915


Download of Data produces Validation Errors (Data Checksum?)

First of all, thank you for your effort in creating a change point detection benchmark. I really appreciate this as the benchmarking of algorithms is not standardized in most publications. Therefore, I wanted to use your dataset to test different algorithms.
I used a virtual environment to install all required packages as described in the Readme. Then I use
python build_tcpd.py -v collect within the activated virtual environment to start the download of data.
After downloading the first files for the Apple dataset (AAPL.csv and apple.json appear in the directory ./datasets/apple). The script throws and Validation error:

Running collect action for dataset: apple ... Traceback (most recent call last):
  File "C:\Users\Lucas\PycharmProjects\TCPD\build_tcpd.py", line 115, in <module>
    main()
  File "C:\Users\Lucas\PycharmProjects\TCPD\build_tcpd.py", line 110, in main
    func(name, script)
  File "C:\Users\Lucas\PycharmProjects\TCPD\build_tcpd.py", line 85, in collect_dataset
    return run_dataset_func(name, script, "collect")
  File "C:\Users\Lucas\PycharmProjects\TCPD\build_tcpd.py", line 81, in run_dataset_func
    func(output_dir=dir_path)
  File "./datasets\apple\get_apple.py", line 213, in collect
    write_patch(json_path, target_path=json_path)
  File "./datasets\apple\get_apple.py", line 68, in wrapper
    raise ValidationError(target)
tcpd.apple.ValidationError: ./datasets\apple\apple.json

If I delete the Apple Dataset from TARGETS in build_tcpd.py, it then proceeds with everything for bee_waggle_6. There it also notes, that the checksum does not match. The script then throws the next Validation error at bitcoin.

Running collect action for dataset: bee_waggle_6 ... Warning: Generated dataset ./datasets\bee_waggle_6\bee_waggle_6.json didn't match a known checksum. This is likely due to rounding differences caused by different system architectures. Minor differences in algorithm performance can occur for this dataset.
ok
Running collect action for dataset: bitcoin ... Traceback (most recent call last):
  File "C:\Users\Lucas\PycharmProjects\TCPD\build_tcpd.py", line 114, in <module>
    main()
  File "C:\Users\Lucas\PycharmProjects\TCPD\build_tcpd.py", line 109, in main
    func(name, script)
  File "C:\Users\Lucas\PycharmProjects\TCPD\build_tcpd.py", line 84, in collect_dataset
    return run_dataset_func(name, script, "collect")
  File "C:\Users\Lucas\PycharmProjects\TCPD\build_tcpd.py", line 80, in run_dataset_func
    func(output_dir=dir_path)
  File "./datasets\bitcoin\get_bitcoin.py", line 131, in collect
    write_json(csv_path, target_path=json_path)
  File "./datasets\bitcoin\get_bitcoin.py", line 66, in wrapper
    raise ValidationError(target)
tcpd.bitcoin.ValidationError: Validating the file './datasets\bitcoin\bitcoin.json' failed.
Please raise an issue on the GitHub page for this project if the error persists.

Could you help me out, what is going wrong here? I would really appreciate it. Maybe this has something to do with the validation checksum computation, which might be different on my PC. The venv is using Python 3.9.13 on a Windows machine.
Best regards
Lucas

I just checked the closed Issues and #3 seems to be similar to mine. So the recommendation is then to use the docker?
My guess is that some windows related character encodings, e.g. EOF, probably mess up the validation check sum.
Might that be right?

Hi Lucas, thanks for opening this issue. I think your conclusion is correct: Windows-related character encodings and line endings might be the root cause of the validation errors. The dataset is regularly checked using builds for docker and ubuntu, and these checks seem to pass. If you're happy to use docker on Windows then I think that's the best solution.


3gstats detects wrong monitor device for Huawei ME909s-120

Hi.
I have a Huawei ME909s-120 in one of my firewalls, that I was able to get to work (after changing an init string to &F, but that's a different issue that I'll post later).
I noticed however that the code that is in OPNsense right now (as of release 18.1.9) seems to assume that the last cua device that belongs to the modem is the one where the status output comes out on. That's not a correct assumption, at least not on my modem.
A workaround to make the statistics almost work is as below:
root@pvz-lab:~ # ps axuw|grep 3gstats
root    11193   0.0  0.5<PHONE_NUMBER>2  -  Is   15:29    0:00.13 /usr/local/bin/php /usr/local/sbin/3gstats.php cuaU0.4 wan
root    29230   0.0  0.1 1080528  2864 u0  S+   15:32    0:00.01 grep 3gstats
root@pvz-lab:~ # kill 11193
root@pvz-lab:~ # /usr/local/bin/php /usr/local/sbin/3gstats.php cuaU0.2 wan
^Z
Suspended
root@pvz-lab:~ # bg
[1]    /usr/local/bin/php /usr/local/sbin/3gstats.php cuaU0.2 wan &

After applying this workaround, /tmp/3gstats.wan seems to be created correctly and is at least updated with a correct RSSI value, but still the statistics I can't see them in the WebUI. Not sure what process polls this tmp file.
root@pvz-lab:/tmp # cat /tmp/3gstats.wan
#seconds,rssi,mode,submode,upstream,downstream,sentbytes,receivedbyts,bwupstream,bwdownstream,simstate,service
0,21,0,0,0,0,0,0,0,0,0,0
root@pvz-lab:/tmp #

The code that seems to determine which monitor device to use is found here:
https://github.com/opnsense/core/blob/ecc15a7680c58a4bdea4bbf3876c8e6aeceab8ee/src/etc/inc/interfaces.inc#L1519
This might also be useful:
root@pvz-lab:/tmp # usbconfig | egrep -ie '(huawei)'
ugen2.3: <Huawei Technologies Co., Ltd. HUAWEI Mobile V7R11> at usbus2, cfg=0 md=HOST spd=FULL (12Mbps) pwr=ON (2mA)

Timeout


ast: Implement eqfilter and path

The communcation between the parser and the resolver is going to be done
using a Abstract Syntax Tree or AST for short, also the idea is to start
representing an expression a filter expression like
routes.running.destination=="<IP_ADDRESS>/0".
This change add the needed types to create a tree to represent that with
the less priority operator at the top, it contains some unit test to
check the yaml annotations.

use k8s.io/yaml


setting path to root not working

i tried path: . as recomended in #8 and also path: / and path:  but i keep getting OSError: [WinError 123] Invalid name: '/tmp\\*'

Hi Tom,
Would you mind providing a reproducible example please

https://github.com/TomW1605/Fusion360_Slicer_Selector_test

Please make sure to include a .spec file which instructs Pyinstaller how to package your application
If you provide a .spec file in the path that you set (ie, .) then the action can run as expected
Let me know if any dramas, check the README for more information about this

thanks i dont know how i missed that. i try it on the weekend when i have time

When I set path to . i have error OSError: [WinError 123] Invalid name: 'Z:\\tmp\\*' (name: Package Application)
  - name: Package Application
      uses: JackMcKew/pyinstaller-action-windows@python3-10-pyinstaller-5-3
      with:
        path: .

    - uses: actions/upload-artifact@v2
      with:
        name: space
        path: src/dist/windows

@EnzoDeg40 Can you please try including the suffix (.spec) to your .yml file please :)

@EnzoDeg40 Can you please try including the suffix (.spec) to your .yml file please :)

I tried that but I'm getting another error
/entrypoint.sh: line 42: cd: .spec: No such file or directory
here is my repo https://github.com/veryredpp/number-system-stuff

sorry for the delay on this, i added the .spec file but now i get other errors TypeError: __init__() got an unexpected keyword argument 'hooksconfig'. should i make a new issue for this one?
https://github.com/TomW1605/Fusion360_Slicer_Selector_test/actions/runs/6624490243/job/17993579038

Check this already raised issue :)
https://github.com/JackMcKew/pyinstaller-action-windows/issues/16

Closing due to stale


[GEP-26] Allow replacing SecretBindingName with CredentialsBindingName in Shoot spec (Part 2)

How to categorize this PR?

/area security ipcei
/kind enhancement
/ipcei workload-identity
What this PR does / why we need it:
This PR is a follow up after https://github.com/gardener/gardener/pull/9853 implementing the remaining functionalities for credentials bindings.
Which issue(s) this PR fixes:
Part https://github.com/gardener/gardener/issues/9586
Special notes for your reviewer:
Kindly asking for a review on this one
/cc @vpnachev @timuthy
Release note:

NONE

/label ipcei/workload-identity

/test pull-gardener-e2e-kind

/assign

/assign

Sorry for the delayed resolution of comments. I enhanced the integration tests + included a fix for proper handling of credentials bindings in the seed authorizer.

/lgtm

/retest-required

/retest

/retest

/lgtm

/retest

/test pull-gardener-e2e-kind-ha-multi-zone


DatePickerElement are not exported from "react-hook-form-mui": "^7.0.0"

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
bundler: "Vite"
Node: "Node 20"
package.json:
"dependencies": {
...
"react-hook-form-mui": "^7.0.0",
"@mui/x-date-pickers": "^7.5.0",
...
}
Expected behavior 🤔
The DatePickerElement to be available having import { DatePickerElement } from 'react-hook-form-mui' of version 7++;
Steps to reproduce 🕹
No response

you are using wrong import please follow https://github.com/dohomi/react-hook-form-mui/issues/252#issue-2230175172
import {DatePickerElement} from 'react-hook-form-mui/date-pickers'


Adapted in latest SDK's interface.

Codecov Report

Merging #94 into dev will increase coverage by 0.09%.
The diff coverage is 1.78%.

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


[Onpremises] Show license expiration date

Related to: https://github.com/CartoDB/onpremises/issues/751

Adds a new field in /me with the license_information when available with a valid date.
Shows the date in the account settings only when cartodb is hosted (on-premises) and the user is org owner or admin.

retest this please

Tested in staging that the account page shows the same as before, and /me now includes "license_expiration": null.


AI Doc Writer For Android Studio

Please provide a plugin for Android Studio soon.

@pawaom working on it and it'll be available in a week! Will keep you posted when that happens

@pawaom we now support Android Studio! You can find the plugin here

Not compatible with the version of your running IDE (Android Studio Flamingo | 2022.2.1 Patch 2)

Not compatible with the version of your running IDE (Android Studio Flamingo | 2022.2.1 Patch 2)

Same here, did you find the solution or any other plugin?


Broken links in docs

https://github.com/zcash/zcash-android-wallet-sdk/blob/preview/docs/-synchronizer/balance.md
Clicking Wallet.WalletBalance 404's, I think it should go to https://github.com/zcash/zcash-android-wallet-sdk/blob/preview/docs/-wallet/-wallet-balance/README.md.

Noted.
There is a documentation task that changes all index.md file names to README.md so that they behave better on Github. Scanning all the documentation and updating links is out of scope.
The current documentation is a stop-gap intended to provide some initial guidance but not intended as a permanent solution. Ultimately, we do not want to host API documentation (e.g. KDocs) inside the actual repository. Instead, we'll host that static content elsewhere.

Closing since this will automatically resolve itself once the API docs are migrated to their permanent home. Right now, this repo is in a "developer preview" status so it's okay if things like this are not finalized.


Fix failures on Windows in hostname resource

Signed-off-by: John McCrae<EMAIL_ADDRESS>Description

Related Issue

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] All commits have been signed-off for the Developer Certificate of Origin.
Signed-off-by: John McCrae<EMAIL_ADDRESS>

totally borked the tree and started a new PR to manage the update for this.


A11y_SQLAzureDataStudio_Notebook_Create new notebook_ScreenReader: There is no announcement of the output cleared.

Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags: #A11y_SQLAzureDataStudioBenchmark;#A11yWCAG2.1;#A11ySev1;#A11yTCS;#SQL Azure Data Studio;#Benchmark;#DesktopApp;#Win32;#NVDA;#JAWS;#A11y_SQlADS_WCAG2.1;#MAS4.1.3;
Environment Details:
Application Name: Azure Data Studio
Version: 1.16.0-insider
OS: Windows_NT x64 10.0.18363
Additional Details:
MAS References:MAS4.1.3
Screen reader: JAWS, NVDA2019.2
Repro Steps:

Launch Azure Data Studio.
Click File then New Notebook.
Launch screen reader as NVDA
Use tab and navigate to clear results button.

Actual
When the code has been executed hit enter on clear results button the data is cleared but no announcement of the same.
Similar fo run cell button
Similar issue with jaws
Expected:
There should be an announcement stating the data has been cleared.
User Impact:
Screen reader users will not know if the data has been cleared or the cell is running.
Attachment link for Reference
A11y_SQLAzureDataStudio_Notebook_Create new notebook_ScreenReader9368.zip

dupe of https://github.com/microsoft/azuredatastudio/issues/9404

GithubTags:#A11y_SQlADS_Win32;#A11yMAS;

Issue was still repro using NVDA the clear results button was not working.
Attachment added for reference

Discussed this in office hours with v-tayazd - for actions which execute immediately such as these we shouldn't be announcing that an action is occurring as that tends to be too verbose.
#closed;

GitHubTags:#SQLADS-Win32-Feb2020;


Script runs but doesn't change the database

Describe the bug
I am trying out grate for the first time with a pre-existing SQL Server database.
My first and only script is Migrations/up/'0001_Add_DrawingLogSheetNos_Id.sql which has the following contents:
ALTER TABLE DrawingLogSheetNos
ADD Id uniqueidentifier NOT NULL DEFAULT NEWID()

I then run with grate -cs "myconnstr" -f "./Migrations".
According to the logs it has executed the script:
Looking for Update scripts in "./Migrations/up". These should be one time only scripts.
--------------------------------------------------------------------------------
  Running '0001_Add_DrawingLogSheetNos_Id.sql'.
--------------------------------------------------------------------------------

However, the column has not been added to the table.
(Manually executing the script, however, does work.)
What am I missing? (Note: I have never used RoundhousE before.)

At first glance that all seems as it should 😕
Would you mind running grate with the -v Trace param, and posting the full output here (with passwords etc redacted as needed)?


GenericBioEngineApp: Align input/output channels in 3d

The rotation for input/output channels should be aligned, i.e. the rotations should be the same.

This would be nice to have, but is not high priority.

@constantinpape  I think you can try to display the image in a single layer. In Python, you can pass an numpy array as a 3D color image to the viewer. The ordering of the dimensions are [Z, Y, X, C].
https://kaibu.org/docs/#/api?id=view_imageimage-options
Example:
# 80x100x100 volume with 5 channels
image = np.random.randint(0, 155, size=[80, 100, 100, 5]).astype('uint8')
await viewer.view_image(image, type="itk-vtk", name="3d-color-image")

Thanks for the code snippet @oeway. Now that we have it integrated we at least have input channels / output channels linked.
But the rotation of input and output is still not linked, and linking them via channels is not an option.
Would be nice to add this feature eventually, but I agree that this is not so important right now.

But the rotation of input and output is still not linked, and linking them via channels is not an option.
Would be nice to add this feature eventually, but I agree that this is not so important right now.

We don't support this for now, I checked it and it looks like a bit hard to synchronise then in 3D. I would suggest to remove the input layer first, and concatenate the input/output channels, and then display them in a single layer. This will only work for models that produce an output with the same spatial size as the inputs. Should we go for this? Alternatively, we can always hide the input when we display the output.

I would suggest to remove the input layer first, and concatenate the input/output channels, and then display them in a single layer. This will only work for models that produce an output with the same spatial size as the inputs. Should we go for this? Alternatively, we can always hide the input when we display the output.

I think both don't sound like such a good solution. Having input and output in the same image in different channels will be quite confusing. And always hiding the input is also not so great, since you want to see how the output looks related to the input.
I think we can leave it as it is right now and maybe come back to this once we have some more time. (I am sure there is in principle some way to sync the rotation if you use the itk viewer backend).


Package File Names don't include Version

Describe the bug
CPack is used to create a .tar.gz and a .zip of each build. This package name is set by the variable CPACK_PACKAGE_FILE_NAME which includes a CMake variable "K4A_VERSION_STR". That CMake variable no longer exists and should now be referred to as "VERSION_STR".
To Reproduce

Run ninja package
Name of package is k4asdk-.Linux

Expected behavior
Name of package should be k4asdk-1.0.1.Linux

This issue is not true for our open source library. I confused this repository and the repository we use to package this. Closing this issue.


Add a host and scheme to Location headers in redirects

Our apache reverse-proxy can't correctly rewrite relative Location headers, so
add the Host from the request into the Location.

LGTM


[azeventgrid] CloudEvent v2 design - ease of use, confusion of types

@jhendrixMSFT brought this up in #21282, and it sparks some other conversations about the type that I think are worth talking about.
Some things I want to look at:

Can we make it so CloudEvent validity checking is  done later so you don't have to have to have error handling code just to generate a CloudEvent? Right now the constructor for CloudEvent can return an error because uuid.New can return an error - not the greatest experience.
Do we need to hide DataBase64? Currently Data is doing double-duty because of it - we're allowing you to pass a []byte (to signal 'binary encoding') and also any other struct/JSON-serializable value.

We've shipped this type officially.


moved nested functions to top level for strict mode

some browsers complained that functions may only be defined on top level in strict mode

I've had a look and I can't replicate browsers complaining. Can you tell me a bit more?

See e.g. 2nd topic in this paragraph:  https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Strict_mode#Paving_the_way_for_future_ECMAScript_versions
I can try to create a simple test project tomorrow.

Here an example: http://plnkr.co/edit/OXJVbkL0BEAGNJKmxXgf?p=preview
Works in Chrome, but gives error in Firefox and Safari:
SyntaxError: in strict mode code, functions may be declared only at top level or immediately within another function

Ah, nice! Thanks for the demo @slintes :)

Updated on npm with 1.0.7


Rework gg push command

Now that gg pull picks up all the refs, I'd like to rethink the gg push command a bit.

Test cases I've considered as of 2019-08-14:

gg push with no arguments:

Pushes all branches & tags to remote
Does not create new branches
Warns for branches not pushed
Does not remove branches
New tags added
No tags removed

gg push with a repository argument, same as the first
gg push with diverging branch, TODO: should others succeed?
gg push --new-branch:

Pushes all branches & tags to remote
Creates new branches
Does not remove branches
New tags added
No tags removed

gg push -r master:

Only pushes master branch
No removals
No tags pushed

gg push -r newbranch:

Should error, no push

gg push -r newbranch --new-branch:

Should create newbranch

gg push -r newtag:

Should create newtag


added artifacts to G-Reg and API-M integration to support the functio…

…nality to unpublish an artifact from API-M
Added the artifacts provided by Chanaka Jayasena with modified .jar file and other supportive artifacts to test the following scenario.

Publish an artifact to API-M publisher via G-Reg
Unpublish the artifact and verify the API-M is updated accordingly. (readme.txt provided)

Hi Pubudu,
Could you kindly do the following changes and send a new PR?

Remove the patch from the PR
Remove the .idea folder and place the jar that we are using here instead


bundle install errors

Hello,
We are trying to build our custom DSC resources (cChoco). The manual states that you should use ruby 1.9.3. We currently have:
ruby 1.9.3p551 (2014-11-13 revision 48407) [x86_64-linux]

Bundler version 1.14.4 is installed.

When we run the command bundle install we are getting the following errors.
Is there a way of working around this issue?
[root@puppetmaster dsc]# pwd
/etc/puppetlabs/code/environments/production/modules/dsc
[root@puppetmaster dsc]# bundle install
Don't run Bundler as root. Bundler can ask for sudo if it is needed, and installing your bundle as root will break this application for all non-root users on this machine.
Fetching https://github.com/puppetlabs/mof.git
Fetching gem metadata from https://rubygems.org/.........
Fetching version metadata from https://rubygems.org/.
Resolving dependencies...................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................
Bundler could not find compatible versions for gem "beaker":
In Gemfile:
beaker (>= 3)
beaker-abs (~> 0.1) was resolved to 0.2.0, which depends on
  beaker (< 4.0, >= 2.9.0)

beaker-module_install_helper was resolved to 0.1.2, which depends on
  beaker (>= 2.0)

beaker-module_install_helper was resolved to 0.1.2, which depends on
  beaker (>= 2.0)

beaker-rspec was resolved to 2.0.1, which depends on
  beaker (~> 1.3)

master_manipulator was resolved to 1.2.6, which depends on
  beaker (>= 2.7.0)

Bundler could not find compatible versions for gem "ruby":
In Gemfile:
ruby
pry was resolved to 0.10.4, which depends on
  coderay (~> 1.1.0) was resolved to 1.1.1, which depends on
    ruby (>= 1.8.6)

rspec-puppet (>= 2.3.2) was resolved to 2.5.0, which depends on
  rspec was resolved to 3.5.0, which depends on
    rspec-mocks (~> 3.5.0) was resolved to 3.5.0, which depends on
      diff-lcs (< 2.0, >= 1.2.0) was resolved to 1.3, which depends on
        ruby (>= 1.8)

beaker (>= 3) was resolved to 3.0.0, which depends on
  google-api-client (~> 0.9) was resolved to 0.10.0, which depends on
    googleauth (~> 0.5) was resolved to 0.5.1, which depends on
      faraday (~> 0.9) was resolved to 0.11.0, which depends on
        ruby (>= 1.9)

beaker (>= 3) was resolved to 3.0.0, which depends on
  google-api-client (~> 0.9) was resolved to 0.10.0, which depends on
    ruby (~> 2.0)

beaker (>= 3) was resolved to 3.0.0, which depends on
  net-ssh (~> 3.2) was resolved to 3.2.0, which depends on
    ruby (>= 2.0)

beaker (>= 3) was resolved to 3.0.0, which depends on
  net-ssh (~> 3.2) was resolved to 3.2.0, which depends on
    ruby (>= 2.0)

beaker (>= 3) was resolved to 3.0.0, which depends on
  net-ssh (~> 3.2) was resolved to 3.2.0, which depends on
    ruby (>= 2.0)

beaker (>= 3) was resolved to 3.0.0, which depends on
  rbvmomi (~> 1.9) was resolved to 1.9.4, which depends on
    nokogiri (~> 1.5) was resolved to <IP_ADDRESS>, which depends on
      ruby (>= 2.1.0)

rspec-puppet-facts was resolved to 1.7.1, which depends on
  puppet was resolved to 4.9.2, which depends on
    ruby (>= 1.9.3)

rspec-puppet-facts was resolved to 1.7.1, which depends on
  puppet was resolved to 4.9.2, which depends on
    ruby (>= 1.9.3)

rspec-puppet-facts was resolved to 1.7.1, which depends on
  puppet was resolved to 4.9.2, which depends on
    ruby (>= 1.9.3)

rspec-puppet-facts was resolved to 1.7.1, which depends on
  puppet was resolved to 4.9.2, which depends on
    ruby (>= 1.9.3)

rspec-puppet-facts was resolved to 1.7.1, which depends on
  puppet was resolved to 4.9.2, which depends on
    ruby (>= 1.9.3)

rspec-puppet-facts was resolved to 1.7.1, which depends on
  puppet was resolved to 4.9.2, which depends on
    ruby (>= 1.9.3)

rspec-puppet-facts was resolved to 1.7.1, which depends on
  puppet was resolved to 4.9.2, which depends on
    ruby (>= 1.9.3)

puppet-blacksmith (>= 3.4.0) was resolved to 3.4.0, which depends on
  rest-client (~> 1.8.0) was resolved to 1.8.0, which depends on
    ruby (>= 1.9.2)

puppet-blacksmith (>= 3.4.0) was resolved to 3.4.0, which depends on
  rest-client (~> 1.8.0) was resolved to 1.8.0, which depends on
    ruby (>= 1.9.2)

puppet-blacksmith (>= 3.4.0) was resolved to 3.4.0, which depends on
  rest-client (~> 1.8.0) was resolved to 1.8.0, which depends on
    ruby (>= 1.9.2)

puppet-blacksmith (>= 3.4.0) was resolved to 3.4.0, which depends on
  rest-client (~> 1.8.0) was resolved to 1.8.0, which depends on
    ruby (>= 1.9.2)

puppet-blacksmith (>= 3.4.0) was resolved to 3.4.0, which depends on
  rest-client (~> 1.8.0) was resolved to 1.8.0, which depends on
    ruby (>= 1.9.2)

beaker (>= 3) was resolved to 3.0.0, which depends on
  google-api-client (~> 0.9) was resolved to 0.10.0, which depends on
    retriable (< 4.0, >= 2.0) was resolved to 3.0.0, which depends on
      ruby (>= 2.0.0)

pry was resolved to 0.10.4, which depends on
  slop (~> 3.4) was resolved to 3.6.0, which depends on
    ruby (>= 1.8.7)

pry was resolved to 0.10.4, which depends on
  slop (~> 3.4) was resolved to 3.6.0, which depends on
    ruby (>= 1.8.7)

pry was resolved to 0.10.4, which depends on
  slop (~> 3.4) was resolved to 3.6.0, which depends on
    ruby (>= 1.8.7)

pry was resolved to 0.10.4, which depends on
  slop (~> 3.4) was resolved to 3.6.0, which depends on
    ruby (>= 1.8.7)

pry was resolved to 0.10.4, which depends on
  slop (~> 3.4) was resolved to 3.6.0, which depends on
    ruby (>= 1.8.7)

pry was resolved to 0.10.4, which depends on
  slop (~> 3.4) was resolved to 3.6.0, which depends on
    ruby (>= 1.8.7)

pry was resolved to 0.10.4, which depends on
  slop (~> 3.4) was resolved to 3.6.0, which depends on
    ruby (>= 1.8.7)

pry was resolved to 0.10.4, which depends on
  slop (~> 3.4) was resolved to 3.6.0, which depends on
    ruby (>= 1.8.7)

pry was resolved to 0.10.4, which depends on
  slop (~> 3.4) was resolved to 3.6.0, which depends on
    ruby (>= 1.8.7)

pry was resolved to 0.10.4, which depends on
  slop (~> 3.4) was resolved to 3.6.0, which depends on
    ruby (>= 1.8.7)

Could not find gem 'ruby (>= 2.0)', which is required by gem 'puppet', in any of the sources.

Try  bundle install --without system_tests instead. That will skip all of the beaker gems which aren't needed when building types

This is resolved as per #271

jpogran, worked. Thank you!


Moving a file between working set and staging should preserve selection

I find it looks bad if every time I move a file between the groups, the blue focus ring appears over the entire list instead of keeping focus on the item I had selected:

This is hard since these are not files, but changes. Staged Geometry.ts is a different element than Unstaged Geometry.ts. List selection preservation works via identity matching and those two elements have different ids, since they are indeed different:

You're actually the first to ask for this, after years of this implementation, so I'm tempted to just close it.

Yeah not a big deal as I typically have the focus color disabled anyway.


chore: add code-coverage-reporting GitHub Workflow

Coverage Report
Status	Category	Percentage	Covered / Total
🔵	Lines	4.19%	7 / 167
🔵	Statements	4.19%	7 / 167
🔵	Functions	0%	0 / 14
🔵	Branches	0%	0 / 14
File Coverage
Generated in workflow #2

Works like a charm. :100:


Not able to disable the telephone number formatting

@Team,
I have a 10 digit mobile number with the country code ex: +17036877205 coming from my backend service, but when I try to bind it to vue-tel-input box its getting formatting getting displayed as<PHONE_NUMBER>, I do not wan to show '(' and  ')' symboles and the spaces between the numbers, I want it to display as<PHONE_NUMBER> with the country flag based on the country code.
I tried by setting the prop value disabledFormatting="true", but still I can able to see the formatting can you please help here?
code snippet
<vue-tel-input
:defaultCountry="phoneCountryCd"
v-model="contactNum"
:key="phoneCountryCd"
data-ut="numberInput"
@keydown="resetMessage"
@mousedown="resetMessage"
@onInput="contactNumEvent"
disabledFormatting="true"
>

Guess you are looking for autoFormat?
<vue-tel-input
....
:autoFormat="false"

Guess you are looking for autoFormat?
<vue-tel-input
....
:autoFormat="false"

autoFormat is available from v.5.0.0

autoFormat is available from v.5.0.0


Falcon theme

Hi, if you are gathering all themes in our repo - here is one more
https://github.com/fenetikm/falcon/blob/master/alacritty/alacritty.yml

Thanks for the link.


Cannot create subscription with GCP provider, payment_method_id is required

Terraform Version
Terraform v0.13.5
Affected Resource(s)
Please list the resources as a list, for example:

rediscloud_subscription

Debug Output
https://gist.github.com/romanlv/fbed36587d45d1625f7d11600d80923d
Expected Behavior
Should be able to create subscription and db without providing payment method
Actual Behavior

Error: strconv.Atoi: parsing "": invalid syntax

I can see that documentation has been updated recently to make payment_method_id but, but actual code uses it as required input
Steps to Reproduce
Please list the steps required to reproduce the issue, for example:

terraform apply

resource "random_password" "password" {
  length = 20
  upper = true
  lower = true
  number = true
  special = false
}

resource "rediscloud_subscription" "example" {

  name = "example"
  // payment_method_id = "0" // data.rediscloud_payment_method.card.id
  memory_storage = "ram"
  persistent_storage_encryption = false

  cloud_provider {
    cloud_account_id = local.rediscloud_account_id
    provider = "GCP"

    region {
      region = "eu-west-1"
      networking_deployment_cidr = "<IP_ADDRESS>/24"
      preferred_availability_zones = ["eu-west-1a"]
    }
  }

  database {
    name = "tf-example-database"
    protocol = "redis"
    memory_limit_in_gb = 1
    data_persistence = "none"
    throughput_measurement_by = "operations-per-second"
    throughput_measurement_value = 10000
    password = random_password.password.result

    alert {
      name = "dataset-size"
      value = 40
    }
  }
}

I tried to comment out payment_method_id in provider code, but started getting
Error: 400 BAD_REQUEST - SUBSCIPTION_GCP_ALLOW_ONLY_INTERNAL: GCP subscription is supported only with a Redis Labs internal resources
Possible that api client needs to be updated too.
Please let me know if it's an easy fix to make, or suggested workaround

Hi @romanlv. Thanks for reporting this. We actually also noticed it and we already have a fix (see #62 and #56). This will be pushed into our next release.


Semantic of container.id and container.name

Motivation
Historically, when a syscall event occurs outside a container, the container.id field is set to host. Our ruleset has consistently followed this pattern: :point_down:
https://github.com/falcosecurity/rules/issues/new?permalink=https%3A%2F%2Fgithub.com%2Ffalcosecurity%2Frules%2Fblob%2Fb6ad37371923b28d4db399cf11bd4817f923c286%2Frules%2Ffalco_rules.yaml%23L226-L227
This behavior is also documented in the official documentation.
Although this design decision is opinionated, it works since a container ID cannot be host.
The container.name field currently follows the same pattern: :point_down:
https://github.com/incertum/libs/blame/master/userspace/libsinsp/filterchecks.cpp#L6232-L6236
However, using container.name = host is unsafe because a container could be named host.
Overall, the current approach could lead to confusion or errors.
Feature
To resolve this issue for non-container cases, we propose two backward-incompatible solutions:

Leave container.name unset (like other container.* fields) and continue using container.id=host.
Leave both container.id and container.name unset. This would make not container.id exists work correctly (assuming the empty value problem will also be fixed).

Alternatives
Doing nothing is not an option, as container.name = host could be misleading.
Additional context
This change would be a major breaking change and should be targeted for Falco 1.0.
Also note the empty value problem (a.k.a. the  issue) is orthogonal to this issue. Still, it should be taken into consideration

Thanks for reporting this one Leo!
Going the full breaking change route, i'd say 2. is the best solution; i love not container.id exists, feels so much better than looking for host instead.

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

/remove-lifecycle stale

Thanks for reporting this one Leo! Going the full breaking change route, i'd say 2. is the best solution; i love not container.id exists, feels so much better than looking for host instead.

@FedeDP will your container plugin implementation go in this direction? Or is it still TBD? :thinking:

Still TBD!


Fixing the order of wrappers in env create function

Necessary to make sure that the state descriptors are also updated when using a fixed initial state with the wrapper

Codecov Report

Merging #128 (e81f13f) into develop (f84f887) will not change coverage.
The diff coverage is 66.66%.

:exclamation: Current head e81f13f differs from pull request most recent head ec6a7d2. Consider uploading reports for the commit ec6a7d2 to get more accurate results

@@           Coverage Diff            @@
##           develop     #128   +/-   ##
========================================
  Coverage    92.31%   92.31%
========================================
  Files          103      103
  Lines         5816     5816
========================================
  Hits          5369     5369
  Misses         447      447

Impacted Files
Coverage Δ

qdax/environments/__init__.py
89.79% <66.66%> (ø)

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


SkiaScroll inside an auto-sized Canvas jumps from the wrong border

If a Canvas has VerticalOptions set to Fill all is fine, in case of VerticalOptions = "Start" (auto-sized) the following weird behavior is seen:
https://github.com/taublast/DrawnUi.Maui/assets/25801194/132fa119-c706-46d0-beaa-aebc2dd8ff4e
When scroll is at the bottom border the viewport suddenly bounces when scrolling back to top. This was watched only on iOS real device.
The current workaround is to set VerticalOptions = "Fill" for Canvas.

Might be fixed already but still need confirmation by testing on ios..

Finally catched and fixed by commit 90854b2be8d3f0a96deca7e41dd877d20555c616


fix: inspect function support resolving shortname

Fix #2524
Currently, there is an error when executing the run command with a short name:

This PR adds the localhost and docker.io prefixes to the shortname.

More info about the shortname, please see: https://github.com/containers/buildah/issues/1034

Thank you for your first contribution!

@fengxsong Is this pr also resolves #2388 ?

Yes~


[#20287][Go SDK] Make base_sdk URN versioned on default docker container.

The Go SDK is fairly stable WRT it's FnAPI behavior, but prior to 2.45.0 the SDK would return errors on benign split failures. This has been determined to be incorrect behavior (the failures, if they don't cause dataloss or corruption, should have returned empty splits).
However, to support variable behavior on these split failures and become intolerant of them in the future, we need to finally have a versioned base SDK capability.
This change builds off of the automatically incrementing SDK version on releases, and makes the default image constant clearer in the core.
Fixes #20287

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.


[TiCDC] support data compression when syncing to MySQL/TiDB

Is your feature request related to a problem?
When deploy multiple TiDB Clusters across multiple regions, TiCDC will be used to replicate data changes between these TiDB Clusters. There are some points need to be considered when doing cross region replication:

The network bandwidth of cross regions is limited.
The cost of data transfer across regions is higher than data transfer inside a region.

Describe the feature you'd like
Support compression when replicate data changes across regions.
MySQL protocol support compression: https://dev.mysql.com/doc/refman/8.0/en/connection-compression-control.html
TiDB also support connection compression: https://github.com/pingcap/tidb/issues/22605
Describe alternatives you've considered

Don't support compression, pay more cost for cross region replication; but it is easier to hit cross region network bandwidth limitation when the data replication throughput is large enough.

Teachability, Documentation, Adoption, Migration Strategy
No response

MySQL go-sql-driver doesn't support compression yet. https://github.com/go-sql-driver/mysql/blob/43e9bef05581335f84d246aba6211af1b5133aae/dsn.go#L468

Another less popular MySQL driver has implemented compression https://github.com/go-mysql-org/go-mysql/pull/787 . Hope it's helpful for the contributor who wants to implement it in go-sql-driver

MySQL go-sql-driver doesn't support compression yet. https://github.com/go-sql-driver/mysql/blob/43e9bef05581335f84d246aba6211af1b5133aae/dsn.go#L468

Indeed: https://github.com/go-sql-driver/mysql/issues/24
Note that if we implement this with either driver we should aim to support both zlib and zstd.


Add Survey paper "Internal Consistency and Self-Feedback in Large Language Models: A Survey"

Hi, I would like to introduce our recent survey paper! Hope you can merge this pull request if you find it useful!
Improving the reasoning ability of large language models (LLMs) and mitigating the hallucination problem are crucial research topics. After extensive thought, we realized that these two issues, "enhancing reasoning" and "alleviating hallucinations," share the same fundamental nature. We approached these issues from the perspective of internal consistency. This perspective allowed us to unify many seemingly unrelated works into a single framework. To improve internal consistency (which in turn enhances reasoning ability and mitigates hallucinations), we identified common elements across various works and summarized them into a Self-Feedback framework.
This framework consists of three components: Self-Evaluation, Internal Consistency Signal, and Self-Update.

Self-Evaluation: Responsible for evaluating the model's internal consistency based on its language expressions, decoding layer probability distributions, and hidden states.
Internal Consistency Signal: Through Self-Evaluation, we can obtain numerical, textual, external, and even comparative signals.
Self-Update: Using these signals, we can update the model's expressions or even the model itself to improve internal consistency.

This summarized framework has the potential to encompass many works, as illustrated in the following diagram.

Additionally, we have derived several important insights through experiments and analysis, such as the "hourglass internal consistency evolution rule," "consistent (almost) equals correct," and the "implicit vs. explicit reasoning paradox."
In summary, we have unified many works from the perspectives of internal consistency and self-feedback, providing inspiration for future researchers and standardizing work in this field.
Relevant links:

Hugging Face: https://huggingface.co/papers/2407.14507
arXiv: https://arxiv.org/abs/2407.14507
PDF: https://arxiv.org/pdf/2407.14507
GitHub: https://github.com/IAAR-Shanghai/ICSFSurvey
Paper List (WIP): https://www.yuque.com/zhiyu-n2wnm/ugzwgf/gmqfkfigd6xw26eg

Great work! Thanks


Allow settings custom git prompt

This gives us ability to use string substitution for changing e.g. how
branch name is displayed.
This was in my local changes for some time. I am using git flow with default feature prefix "feature/". This does not look good with bullet-train prompt. So, this feature gives us ability to change:

to:

This implementation allows user to specify git prompt command. By default we set it to git_prompt_info. As this isn't the actual execution of command, we need to delay variable/function extending, thus \${git_prompt_info}.
So, in case showed on screens this would be
BULLETTRAIN_GIT_PROMPT_CMD=\${\$(git_prompt_info)//\\//\ \ }

Please note, that we still need to delay variable expansion, so we need to escape all weird character - $, ****, <space>, etc.

This is second pull request - I am still learning github and I messed something in previous.
Thanks to @dbkaplun for idea how to give user full flexibility for this.

This was an indeed awesome pull request.
Now we have to show your above example of BULLETTRAIN_GIT_PROMPT_CMD on README, so people can use it.

Great!00
I will prepare proper explanation for README and create PR with it.
Thanks

I just inserted the new variable on README.
Maybe we should do a Tips section with this one.

With screenshot and your custom function.

Sounds good, There is no need for separate file, or wiki page. Did you thought maybe about creating wiki? As README file is really long, and the only way to navigate it with Ctrl-F. Maybe at least TOC?

The problem about creating a wiki page is that we will "decentralize" information, and I have a feeling that nobody uses the wiki on GitHub.


Typo in the docs

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Use a shared ipam namespace by default.

Also switch to the CG-NAT cird <IP_ADDRESS>/10
Set org private_cidr=true to use a non-shared ipam namespace for the org.

blocked on merging design proposal in #998

s/cider/cidr/ in the commit message :)

Ran it through QA a couple of times 🟢
https://github.com/nexodus-io/nexodus/actions/runs/4951380705
https://github.com/nexodus-io/nexodus/actions/runs/4952463061

I posted #1168 for a seg fault I saw in the last e2e test run on here.

LGTM, leaving the ability to spin up a custom CIDR addresses my only hesitation with --cidr <IP_ADDRESS>/16 --cidr-v6 200::/32.

Not sure what you mean. This pull request supports doing that.

LGTM, leaving the ability to spin up a custom CIDR addresses my only hesitation with --cidr <IP_ADDRESS>/16 --cidr-v6 200::/32.

Not sure what you mean. This pull request supports doing that.

I was wondering if the response was just misworded a bit. I interpreted it as "this looks good. Maintaining the ability use custom CIDRs resolved my only hesitation."

LGTM, leaving the ability to spin up a custom CIDR addresses my only hesitation with --cidr <IP_ADDRESS>/16 --cidr-v6 200::/32.

Not sure what you mean. This pull request supports doing that.

@russellb @chirino Sorry, meant my only reservation was addressed with that. I'm a go here!

@chirino looks OK, but unit tests are busted


Control 12.10 and 12.11 does not conform to the CIS benchmark

The controls are written to not allow any SUID or SGID binaries, this is incorrect.
The CIS benchmark for Ubuntu 14.04 states:
"Ensure that no rogue set-UID programs have been introduced into the system. Review the
files returned by the action in the Audit section and confirm the integrity of these binaries."
My suggestion is to allow users to specify some attributes defining a list of allowed binaries along with checksums.
If unlisted binaries are found or if checksums of binaries do not match then the control should fail.

This cookbook has been deprecated. At this time I'm going to close this issue out. I'm sorry that we never brought it to a full resolution prior to the deprecation


Misc cleanup - [merged]

In GitLab by @wtbarnes on Oct 5, 2020, 13:38
Merges misc-cleanup -> master
Some minor refactoring, fixing typos in docs.
Fixes #80

In GitLab by @wtbarnes on Oct 5, 2020, 13:54
added 1 commit
32a6e6d0 - marking more doctests
Compare with previous version

In GitLab by @codecov on Oct 5, 2020, 13:43
Codecov Report

Merging #85 into master will increase coverage by 0.22%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #85      +/-   ##
==========================================
+ Coverage   96.98%   97.20%   +0.22%
==========================================
  Files          14       14
  Lines         431      430       -1
==========================================
  Hits          418      418
+ Misses         13       12       -1

Impacted Files
Coverage Δ

aiapy/calibrate/meta.py
100.00% <100.00%> (ø)

aiapy/calibrate/prep.py
98.03% <100.00%> (+0.08%)
:arrow_up:

aiapy/calibrate/spikes.py
98.33% <100.00%> (-0.03%)
:arrow_down:

aiapy/calibrate/util.py
100.00% <100.00%> (+1.88%)
:arrow_up:

aiapy/response/channel.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

In GitLab by @wtbarnes on Oct 5, 2020, 13:47
added 2 commits
71857813 - add old changelogs976b55fa - mark doctest as remote data
Compare with previous version

In GitLab by @wtbarnes on Oct 5, 2020, 15:53
mentioned in commit a6d35eb5501b9d4b7b59974c2ef9f537066a6e4f


Bundle Loot causing crash

I've been testing this mod to see if I want to add it and I noticed that the Loot Block drops bundles. However when I tried using the bundle I get a crash. I think it might be a conflict with either the Treasure Bag or Gem Bag from More Gems. I can fix it for myself but I just thought I'd let you know so you could remove them from the loot list.

Crash report?

There wasn't any crash report since it did that freeze up thing where I'd need to use task manager. Anyways I found out that what's causing the bundle to crash is actually a different mod I added to change the stack size. So really the only thing you could do on your end would be to remove the bundle from the loot table but idk if you would want to.

Good to know if someone else encounters it.


Return a 5xx if the default Elasticsearch index doesn't exist

For https://github.com/wellcomecollection/platform/issues/5316
The context
In the catalogue API, we let users specify an index on their requests by passing the _index parameter, e.g.
GET /works?_index=my-great-index

This is useful for testing changes in indexes that haven't yet gone out, e.g. I think it's how we select the "tei-on" index for the TEI toggle.
It's possible to select an index that doesn't exist with this parameter, so like a well-behaved HTTP API we return a 404 Not Found if we go to query an index and the index doesn't exist.
The problem
We return a 404 Not Found even if the user didn't specify an index. That is, a request to
GET /works

would return a 404 if the default index is missing. This is wrong – if the default index is missing, that's a problem in the server that we need to fix.
It also means these errors don't get caught by our 5xx alerting. The requests API was serving 404s for almost 24 hours before we noticed, because we'd deleted the underlying Elasticsearch index.
The solution
We return a 500 Internal Server Error if the index is missing unless the user has explicitly told us which index to use.
That means we'll continue to return 404s when somebody asks for a missing index, but if the default index is missing we'll get a 5xx which will be easier to spot in our alerting.

I'd argue we can't tell if the latter is missing, as we can't connect to an index.

We can tell – Elasticsearch returns 404 "index_not_found" exception. We handle it here: https://github.com/wellcomecollection/catalogue-api/blob/b5660c7a15ea43567a446591c307df4061b63282/common/search/src/main/scala/weco/api/search/elasticsearch/ElasticsearchError.scala#L10


[Device Support Request] Ikea 5 button remote (E1810) no longer matches the existing quirks when the firmware is upgraded to 2.3.080

Is your feature request related to a problem? Please describe.
With the release of the 2.3.080 firmware for the Ikea E1810 5 button remote the device signature has changed so the existing quirks no longer match. Support for group binding no longer seems to work but multiple devices can be bound to the remote through Manager Clusters, so the same end result can be achieved.
Describe the solution you'd like
I do not know what functionality the fivebtnremotezha quirks provided, but the device seems to function without it. It certainly would not be benefiting from the battery fixes @MattWestb recently worked on.
Device signature - this can be acquired by removing the device from ZHA and pairing it again from the add devices screen. Be sure to add the entire content of the log panel after pairing the device to a code block below this line.
{
"node_descriptor": "NodeDescriptor(logical_type=<LogicalType.EndDevice: 2>, complex_descriptor_available=0, user_descriptor_available=0, reserved=0, aps_flags=0, frequency_band=<FrequencyBand.Freq2400MHz: 8>, mac_capability_flags=<MACCapabilityFlags.AllocateAddress: 128>, manufacturer_code=4476, maximum_buffer_size=82, maximum_incoming_transfer_size=82, server_mask=11264, maximum_outgoing_transfer_size=82, descriptor_capability_field=<DescriptorCapability.NONE: 0>, *allocate_address=True, *is_alternate_pan_coordinator=False, *is_coordinator=False, *is_end_device=True, *is_full_function_device=False, *is_mains_powered=False, *is_receiver_on_when_idle=False, *is_router=False, *is_security_capable=False)",
"endpoints": {
"1": {
"profile_id": 260,
"device_type": "0x0820",
"in_clusters": [
"0x0000",
"0x0001",
"0x0003",
"0x0020",
"0x1000",
"0xfc7c"
],
"out_clusters": [
"0x0003",
"0x0004",
"0x0005",
"0x0006",
"0x0008",
"0x0019",
"0x1000"
]
}
},
"manufacturer": "IKEA of Sweden",
"model": "TRADFRI remote control",
"class": "zigpy.device.Device"
}
Additional context
Add any other context or screenshots about the feature request here.

Hey thanks that i have missing !!
All my 5 buttons is on the bad firmware and is not getting the new PullControlCluster so i need moving one from my IKEA GW that is still on the old working FW and upgrading it in ZHA (my IKEA GW have upgrading one E1743 to the new but one is not done and the 5 button remote after 3 days).
Perhaps its helping user with battery draining problem then getting the pull control working on the remotes (i was missing that the 5 button was not having it then hunting the battery draining problems).
All IKEA is light controllers and shall working out of the box and being bond to one light group (that looks being gone for the moment but i think IKEA must fixing that) so you is only missing the battery fix (showing half battery % and no extras) and also the device automatons is not working if you are using them.
Also need sniffing the upgraded then Z2M have reporting the scene switching is not working but perhaps  they have doing it different like in Styrbar but is not so urgent then ZHA its not possible setting up Zigbee Scenes ;-(
I making on PR with the new device signature and taking the other things later then we can see how it working with other changes IKEA have doing.

Ops one of the old version have pull control but no scene cluster and the other have scene and no pull control so its one combination of both but perhaps is the scene cluster possible to binding that was not possible before then it was not existing.

@nmeachen I have making one version 3 of the 5 button remote but have not pushed it then i have not tested it.
Can you downloading the updated quirk and installing it as local quirk i ZHA and testing if its loading OK ?
https://github.com/MattWestb/zha-device-handlers/blob/IKEA-5BT-V3/zhaquirks/ikea/fivebtnremotezha.py
Guide for installing local quirk: https://github.com/zigpy/zha-device-handlers/discussions/693#discussioncomment-857274.
Dont forgetting deleting the local quirk then ZHA is having it inside for future updated working OK

Got it working with my E1810.
Added output_cluster "5", and ScenesCluster.cluster_id to signature like this:
class IkeaTradfriRemote(CustomDevice):
    """Custom device representing IKEA of Sweden TRADFRI remote control."""

    signature = {
        # <SimpleDescriptor endpoint=1 profile=260 device_type=2080
        # device_version=1
        # input_clusters=[0, 1, 3, 32, 4096, 64636]
        # output_clusters=[3, 4, 5, 6, 8, 25, 4096]>
        MODELS_INFO: [
            (IKEA, "TRADFRI remote control"),
            ("IKEA of Sweden", "TRADFRI remote control"),
        ],
        ENDPOINTS: {
            1: {
                PROFILE_ID: zha.PROFILE_ID,
                DEVICE_TYPE: zha.DeviceType.NON_COLOR_CONTROLLER,
                INPUT_CLUSTERS: [
                    Basic.cluster_id,
                    PowerConfiguration.cluster_id,
                    Identify.cluster_id,
                    PollControl.cluster_id,
                    LightLink.cluster_id,
                    IKEA_CLUSTER_ID,
                ],
                OUTPUT_CLUSTERS: [
                    Identify.cluster_id,
                    Groups.cluster_id,
                    ScenesCluster.cluster_id,
                    OnOff.cluster_id,
                    LevelControl.cluster_id,
                    Ota.cluster_id,
                    LightLink.cluster_id,
                ],
            }
        },
    }

Updated the quirk so the linking to the "main" IKEA remote is working after renaming it and fixing the wrong mane of one cluster.
Now loading OK and getting the "IKEA scene commands" as events, but is not working then being bounded with one CWS3 GU10 spot ;-((

@MattWestb: As I have 2010.10.6, I had to switch the power configuration back to the DoublingPowerConfigurationCluster as I didn't have PowerConfiguration1CRCluster, but after doing that the custom quirk loaded and the device info for the remote showed the quirk.

Sorry i am also using HA 2021.10 but have patching the IKEA "INIT" with the battery PR for getting it working but must being made in the HA container or is not working with the local quirk.
The easiest is doing like you have done and is working until you is updating HA.

@MattWestb thanks for addressing this so quickly.

You are very welcome !!
Its one updates for Shortcut button but is not so urgent but good.
If you finding what is added in the new firmware ;-)