Problem with converter in Chrome

The xmlrpc.parseDocument converter seems to have some problems parsing structs in chrome.
The source of this, as far as I can tell, appears to be patchy [reduce] (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/Reduce) support in chrome as used by the struct decode function:
	return $(node)
		.find('> member')
		.toArray()
		.reduce(function(struct, el) {
			var $el = $(el);
			var key = $el.find('> name').text();
			var value = xmlrpc.parseValue($el.find('> value'));

			struct[key] = value;
			return struct;
		}, {});

This works fine in Firefox.
With a quick modification to avoid using reduce, everything worked fine e.g.
var struct = {};
	$(node).find('> member').each(
		function(index, el) {
			var $el = $(el);
			var key = $el.find('> name').text();
			var value = xmlrpc.parseValue($el.find('> value'));

			struct[key] = value;
		});
	return struct;

Thanks for the bug report, @gacarpen. Unfortunately I can not reproduce this error on any Chrome/Chromium version I have access to, and the test suite is all passing.
Could you attach your Chrome/OS version, any stack traces or errors you are getting, and the incorrect output you are getting? Additionally, you can try running the tests. Assuming Linux/OSX:
cd jquery-xmlrpc
npm install -g bower
bower install
python -mSimpleHTTPServer 8000

Then open http://localhost:8000/tests/ in your Chrome version that is showing the error, and see if the tests catch the error.

Thanks for the quick response.
OS: Windows 7/8, OS X, Checked with Chrome Version 43.0.2357.130 most recently.
All of your tests also pass for me, though it may be worth mentioning I'm using the library as part of a tampermonkey script and the way chrome sandboxes things might be confusing the issue.
Get no errors, the reduce function just seems to return something different than it does in firefox (a hideously complicated array of jquery objects from the look of things whereas firefox returns the expected object).
Trying to get equivalent logging form chrome and firefox from:
	var parsed = $(node)
		.find('> member')
		.toArray()
		.reduce(function(struct, el) {
			var $el = $(el);
			var key = $el.find('> name').text();
			var value = xmlrpc.parseValue($el.find('> value'));

			struct[key] = value;
			return struct;
		}, {});
	console.log(parsed)
	return parsed;

gives the following for chrome and firefox respectively (apologies for screenshots):

Logging from withing the reduce function seems to fail silently on chrome, making it difficult to further diagnose the issue.

Thanks for the extra information. I installed Tampermonkey and tried to replicate your issue, using code taken from the tests. Using the attached script:
// ==UserScript==
// @name         jQuery XMLRPC test
// @namespace    http://your.homepage/
// @version      0.1
// @description  enter something useful
// @author       You
// @match        http://example.com/
// @grant        none
// @require      http://code.jquery.com/jquery-1.11.3.min.js
// @require      https://raw.githubusercontent.com/timheap/jquery-xmlrpc/master/jquery.xmlrpc.min.js
// ==/UserScript==

function el(xml) {
    return $(xml)[0];
}

var node = $.xmlrpc.parseNode(el(
    '<struct>' +
    '<member>' +
    '<name>foo</name>' +
    '<value><i4>4</i4></value>' +
    '</member>' +
    '<member>' +
    '<name>bar</name>' +
    '<value><string>Hello</string></value>' +
    '</member>' +
    '</struct>'));

console.log(node);

The correct output appeared in the console:

Could you run the script above in your Chrome environment and see if what result you get? If it works for you, could you attach the script you are working on that causes the error?

Tried your tampermonkey script and it output the object correctly.
Thinking the only remaining logical explanation was that it must be input related, I tried running your script with the exact input from my own, whereupon yours returned correctly and mine continued not doing so.
At this point, all I can think of is that there's some hidden data that's not getting logged tripping up the function in some way but I've no idea what it could be. Sadly, can't attach the script so can't really think of much else that can be done.
Very much appreciate how responsive and helpful you've been, and thanks for the great library.

That is indeed strange. If you get to the bottom of the problem and it does end up being something strange in this library, I am happy for this issue to be reopened. Until then though, I will close this ticket.
You should be able to register your own handler for the struct type that uses the code that works for you. I've never actually tried overriding a defined type, but it should work. Try the following code in your project after you have loaded this library:
$.xmlrpc.makeType('struct', false, function(value, $xml) {
    // This code is the same
    var $struct = $xml('struct');

    $.each(value, function(name, value) {
        var $name = $xml('name').text(name);
        var $value = $xml('value').append(xmlrpc.toXmlRpc(value, $xml));
        $struct.append($xml('member').append($name, $value));
    });

    return $struct;

}, function(node) {
    // Your modification
    var struct = {};
    $(node).find('> member').each(function(index, el) {
        var $el = $(el);
        var key = $el.find('> name').text();
        var value = xmlrpc.parseValue($el.find('> value'));

        struct[key] = value;
    });
    return struct;
});


fix: #9275 fixed shop compiled flag issue

Info
https://github.com/bagisto/bagisto/issues/9275

@devansh-webkul  sir.
I have checked with both fresh installation and also on the existing project, It is working fine. No issue is appearing.  Please check the video for reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/693dce5b-eda7-41bf-a0cb-53a9977d9936


Fix issue #398 condition never met

if (arguments[i] && this.parts[i]) //Condition never met
this.resetParts(); //doens't exist on Part prototype and playNextPart already control the flow
maintainers:
Can we normalize CF vs CLRF line endings using git attributes?
got errors on src\audioWorklet\ringBuffer.js
https://github.com/processing/p5.js-sound/issues/398

Hi @doriclaudino , can you please look into the merge conflicts, if you are busy I can resolve those conflicts and make a new PR around the same :)


Alt text missing using WP Bakery

I'm trying to insert alt text with no luck. The website uses Salient theme and WPBakery. I insert the alt text in the Media section, but is not displayed in the source code:

If I deactivate the plugin, the alt text is displayed.

Same issue here. Is there a fix available to the meantime? If not, what would be a work around in the interim?


”https://github.com/GEUltrasound/GE_CVUS_Loader/releases/“ is unavailable?

”https://github.com/GEUltrasound/GE_CVUS_Loader/releases/“ is unavailable? How to download the GE_CVUS_Loader?

You need to register to GE's developer program. See details in the README.MD file.

You need to register to GE's developer program. See details in the README.MD file.
I found the https://github.com/GEUltrasound and This organization has no public repositories. after registered just now.
But there is no necessary star symbol beside the GitHub Username (Please provide your public GitHub username if you are interested in accessing our AppAPI repository that will enable you to build ultrasound apps that can run directly on-device, as well as real-time network streaming of 2D/3D ultrasound data.), I don't know how to update my github username to GE Ultrasound.

You need to wait until they add you to the organization. Allow at least a few days for them to do this.

Everybody don't forget to add GitHub username to this text box, even though there is no mandatory star symbol besides it, like the picture below.


Android - Chat - Unable to cancel the IOU request while offline

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

Action Performed:

Navigate to staging
Disable the internet connection on the device
Create a money request in a 1:1 conversation
Open the IOU details and CANCEL the IOU request

Expected Result:
Able to CANCEL the IOU request.
Actual Result:
Unable to cancel the IOU request while offline
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/03adc2fd-bdc7-40b5-b7d0-5ffb15508154

Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Apologies, I got assigned these during EC3 and I wasn't able to find time during the conference to triage. I'm now OOO for the rest of the week, so I'm going to re-apply the bug label so that someone that's more available can help move this forward. Thanks!

Lol, of course it was me. The ability to "cancel" a request has been deprecated, and will be replaced with "Delete" which @luacmartins is working on.
I'm going to close this bug report out.


ErrNotSupported when merging structs

When I attempt to merge two structs, I get an ErrNotSupported.  I poked into it a little bit, and ran across this comparison:
https://github.com/imdario/mergo/blob/7fe0c75c13abdee74b09fcacef5ea1c6bba6a874/mergo.go#L60
It looks to me like it is impossible for that condition to ever evaluate as false.
If kind is struct, then it’s false && true
If kind is map, then it’s true && false
If kind is anything else, then it’s true && true
Am I misreading that?

How are your structs? Thanks!
About this comparison, I agree it's kind of strange. I'll get around this soon.


[MB-12602] Modifying the move history sql query to retrieve service member information

Co-authored-by: Katy Chambers<EMAIL_ADDRESS>MB-12602 for this change
Summary
Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make storybook

Terminal 2
Start the UI locally.
make client_run

Terminal 3
Start the Go server locally.
make server_run

Additional steps

Access the milmove app as a service member.
Go through the process of creating a move, all the way through uploading orders, creating the HHG shipment and signing and submitting the move. (Make sure you use the correct GBLOC, Fort McCoy is a duty station that will show up easily for our normal local dev patterns)
Log into the office app as a services counselor and view the move history for the move just created.
You should see the customer information in the Modified By Column including Last name, First name, email and phone number

Verification Steps for Author
These are to be checked by the author.

[ ] Have the Jira acceptance criteria been met for this change?

Screenshots
Before

After

@Mergifyio rebase


How to use PSDMatrices.PSDMatrix as a covariance parameter

In my code I rely on PSDMatrices.jl to represent covariace parameters of Gaussian distributions. I would also like to use these as covariance parameters for MarkovKernels.Normal. When I try this it works, but the covariance parameter gets transformed into a Symmetric:
2×2 Symmetric{Float64, PSDMatrix{Float64, Matrix{Float64}}}

But by construction, PSDMatrix is already symmetric. I tried overloading Symmetric in PSDMatrices.jl with
Symmetric(M::PSDMatrix) = M

but then I get stack overflow issues as the Normal constructor repeatedly tries to call Symmetric. What do you think would be the best way to fix this and allow for PSDMatrices?
One proposal would be to write a new constructor for Normal that does not attempt to transform a PSDMatrix into Symmetric, e.g. with
function Normal(μ::AbstractVector, Σ::PSDMatrix)
    T = promote_type(eltype(μ), eltype(Σ))
    return Normal{T}(convert(AbstractVector{T}, μ), convert(PSDMatrix{T}, Σ))
end

This could then be a package extension. But maybe a different approach makes more sense so let me know what you think.

An alternative would be to remove the constructor with signature Normal(::AbstractVector, ::AbstractMatrix)

I would be fine with that too.

Let's do that then.


can you just offer an example of using nak's DBSCAN in java code?

As the title above, because I never used scala, and I just running java task on spark cluster, I want using nak's DBSCAN in my java task, thank you!

Hi @zhaoxp0905
Thanks for your request. This repository has not been maintained in years. Make sure you are aware of that if you plan to use this library in production code.
Depending on the Java Version you are using the code is very similar to https://github.com/scalanlp/nak/blob/master/src/test/scala/nak/cluster/GDBSCANTest.scala
import nak.cluster.GDBSCAN._
val gdbscan = new GDBSCAN(
   DBSCAN.getNeighbours(epsilon = 1, distance = Kmeans.euclideanDistance),
   DBSCAN.isCorePoint(minPoints = 2)
)

Add more type infos, replace val with the actual type, remove the named parameters.
import nak.cluster.GDBSCAN.*
GDBSCAN gdbscan = new GDBSCAN(
   DBSCAN.getNeighbours(1, Kmeans.euclideanDistance),
   DBSCAN.isCorePoint(2)
)

I have no idea if this works, but it should be close. Don't know if breeze makes use of implicits. If so, then you will have no fun with using this in java.


Create identity service: "An unexpected error prevented the server from fulfilling your request. (HTTP 500) (Request-ID: req-ed70ceef-c225-4f2e-a3cd-d7850f864df3)"

When trying to install the identity service from liberty installation docs:
http://docs.openstack.org/liberty/install-guide-ubuntu/keystone-services.html
openstack service create --name keystone --description "OpenStack Identity" identity

An unexpected error prevented the server from fulfilling your request. (HTTP 500) (Request-ID: req-ed70ceef-c225-4f2e-a3cd-d7850f864df3)

Log file gives:
tail /var/log/keystone/keystone-wsgi-admin.log

2016-11-09 00:41:05.712 2662 ERROR keystone.common.wsgi   File "/usr/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 1337, in _handle_dbapi_exception
2016-11-09 00:41:05.712 2662 ERROR keystone.common.wsgi     util.raise_from_cause(newraise, exc_info)
2016-11-09 00:41:05.712 2662 ERROR keystone.common.wsgi   File "/usr/lib/python2.7/dist-packages/sqlalchemy/util/compat.py", line 199, in raise_from_cause
2016-11-09 00:41:05.712 2662 ERROR keystone.common.wsgi     reraise(type(exception), exception, tb=exc_tb)
2016-11-09 00:41:05.712 2662 ERROR keystone.common.wsgi   File "/usr/lib/python2.7/dist-packages/sqlalchemy/engine/base.py", line 1139, in _execute_context
2016-11-09 00:41:05.712 2662 ERROR keystone.common.wsgi     context)
2016-11-09 00:41:05.712 2662 ERROR keystone.common.wsgi   File "/usr/lib/python2.7/dist-packages/sqlalchemy/engine/default.py", line 450, in do_execute
2016-11-09 00:41:05.712 2662 ERROR keystone.common.wsgi     cursor.execute(statement, parameters)
2016-11-09 00:41:05.712 2662 ERROR keystone.common.wsgi OperationalError: (sqlite3.OperationalError) no such table: service [SQL: u'INSERT INTO service (id, type, enabled, extra) VALUES (?, ?, ?, ?)'] [parameters: ('7f702fafd28848458ec5940433a6df47', u'identity', 1, '{"description": "OpenStack Identity", "name": "keystone"}')]
2016-11-09 00:41:05.712 2662 ERROR keystone.common.wsgi

didn't comment reference to sqlite in /etc/keystone/keystone.conf

connection = sqlite:////var/lib/keystone/keystone.db

Another issue with the same symptom:
tail /var/log/keystone/keystone-wsgi-admin.log
2016-11-09 00:53:31.757 2992 ERROR keystone.common.wsgi     engine_args, maker_args)
2016-11-09 00:53:31.757 2992 ERROR keystone.common.wsgi   File "/usr/lib/python2.7/dist-packages/oslo_db/sqlalchemy/enginefacade.py", line 362, in _setup_for_connection
2016-11-09 00:53:31.757 2992 ERROR keystone.common.wsgi     sql_connection=sql_connection, **engine_kwargs)
2016-11-09 00:53:31.757 2992 ERROR keystone.common.wsgi   File "/usr/lib/python2.7/dist-packages/oslo_db/sqlalchemy/engines.py", line 152, in create_engine
2016-11-09 00:53:31.757 2992 ERROR keystone.common.wsgi     test_conn = _test_connection(engine, max_retries, retry_interval)
2016-11-09 00:53:31.757 2992 ERROR keystone.common.wsgi   File "/usr/lib/python2.7/dist-packages/oslo_db/sqlalchemy/engines.py", line 334, in _test_connection
2016-11-09 00:53:31.757 2992 ERROR keystone.common.wsgi     six.reraise(type(de_ref), de_ref)
2016-11-09 00:53:31.757 2992 ERROR keystone.common.wsgi   File "<string>", line 2, in reraise
2016-11-09 00:53:31.757 2992 ERROR keystone.common.wsgi DBConnectionError: (pymysql.err.OperationalError) (2003, "Can't connect to MySQL server on 'controller' ([Errno -2] Name or service not known)")
2016-11-09 00:53:31.757 2992 ERROR keystone.common.wsgi

Wrong controller hostname used in /etc/keystone/keystone.conf

[database]
connection = mysql+pymysql://keystone:openstack@controller/keystone

Mine is libcontroller

tail /var/log/keystone/keystone-wsgi-admin.log

2016-11-09 01:01:54.514 2990 ERROR keystone.common.wsgi   File "/usr/lib/python2.7/dist-packages/pymysql/connections.py", line 826, in _read_packet
2016-11-09 01:01:54.514 2990 ERROR keystone.common.wsgi     packet.check_error()
2016-11-09 01:01:54.514 2990 ERROR keystone.common.wsgi   File "/usr/lib/python2.7/dist-packages/pymysql/connections.py", line 370, in check_error
2016-11-09 01:01:54.514 2990 ERROR keystone.common.wsgi     raise_mysql_exception(self._data)
2016-11-09 01:01:54.514 2990 ERROR keystone.common.wsgi   File "/usr/lib/python2.7/dist-packages/pymysql/err.py", line 116, in raise_mysql_exception
2016-11-09 01:01:54.514 2990 ERROR keystone.common.wsgi     _check_mysql_exception(errinfo)
2016-11-09 01:01:54.514 2990 ERROR keystone.common.wsgi   File "/usr/lib/python2.7/dist-packages/pymysql/err.py", line 109, in _check_mysql_exception
2016-11-09 01:01:54.514 2990 ERROR keystone.common.wsgi     raise errorclass(errno,errorvalue)
2016-11-09 01:01:54.514 2990 ERROR keystone.common.wsgi ProgrammingError: (pymysql.err.ProgrammingError) (1146, u"Table 'keystone.service' doesn't exist") [SQL: u'INSERT INTO service (id, type, enabled, extra) VALUES (%s, %s, %s, %s)'] [parameters: ('37fd60101e904ba8a8d5861d964a375a', u'identity', 1, '{"description": "OpenStack Identity", "name": "keystone"}')]
2016-11-09 01:01:54.514 2990 ERROR keystone.common.wsgi

Re-populate the identity service database after correcting the controller hostname in /etc/keystone/keystone.conf
su -s /bin/sh -c "keystone-manage db_sync" keystone

Another issue with the same symptom:
tail /var/log/apache2/keystone.log

2016-11-09 11:49:44.485447 2016-11-09 11:49:44.483 2992 ERROR keystone.common.wsgi     engine_args, maker_args)
2016-11-09 11:49:44.485451 2016-11-09 11:49:44.483 2992 ERROR keystone.common.wsgi   File "/usr/lib/python2.7/dist-packages/oslo_db/sqlalchemy/enginefacade.py", line 362, in _setup_for_connection
2016-11-09 11:49:44.485455 2016-11-09 11:49:44.483 2992 ERROR keystone.common.wsgi     sql_connection=sql_connection, **engine_kwargs)
2016-11-09 11:49:44.485459 2016-11-09 11:49:44.483 2992 ERROR keystone.common.wsgi   File "/usr/lib/python2.7/dist-packages/oslo_db/sqlalchemy/engines.py", line 152, in create_engine
2016-11-09 11:49:44.485464 2016-11-09 11:49:44.483 2992 ERROR keystone.common.wsgi     test_conn = _test_connection(engine, max_retries, retry_interval)
2016-11-09 11:49:44.485469 2016-11-09 11:49:44.483 2992 ERROR keystone.common.wsgi   File "/usr/lib/python2.7/dist-packages/oslo_db/sqlalchemy/engines.py", line 334, in _test_connection
2016-11-09 11:49:44.485473 2016-11-09 11:49:44.483 2992 ERROR keystone.common.wsgi     six.reraise(type(de_ref), de_ref)
2016-11-09 11:49:44.485476 2016-11-09 11:49:44.483 2992 ERROR keystone.common.wsgi   File "", line 2, in reraise
2016-11-09 11:49:44.485481 2016-11-09 11:49:44.483 2992 ERROR keystone.common.wsgi DBConnectionError: (pymysql.err.OperationalError) (2003, "Can't connect to MySQL server on 'controller' ([Errno -2] Name or service not known)")
2016-11-09 11:49:44.485486 2016-11-09 11:49:44.483 2992 ERROR keystone.common.wsgi

i thought this time i'm gonna follow the Newton installation guide step by step but i got the same, gonna go ahead and comment that sqlite line in keystone.conf
(strange how one step is to delete the sqlite db file and never mention this step)
what about compute / neutron...et.conf?
you think they should be commented as well?

Hi @eriksh7 , sorry for the late reply.
I think it doesn't harm to comment that line, as it refer to a not used db backend which apparently can create confusion.

hi i have the same error but servername is coreect there is no line for sqlite connection in database field so dont need for comment and i did every thing step by step on openstack piko installition but i just get thiss error help pleaseeee !!! @AJNOURI

connection =<EMAIL_ADDRESS>(example)
change ip according to your config


Doc for hot reloading

This doc should be integrated here I think ?
https://github.com/cozy/cozy-template/edit/master/docs/hot-reloading.md

Fixed by #812


Update object-detection.md

Change the curl example to {{ .Page.Params.model.name }} this seems inline with what all the other code-examples are doing
Closes #14913

TY for the catch and fix, @Nebhay


workflows: Add test for sigstore-go

This is part of #79. Fixes #91.

I'm not super happy about making the custom-test.yml longer and more complicated but we can try this
I don't know what the correct method of installing go apps is but this seems to work?

I wonder if we should set test.yml to trigger in PRs if any workflows change?

changed to @latest for sigstore-go, fixed some whitespace issues


Update AutoSSL.md

Added description for which ports to open when behind a firewall/router.

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

I agree, this was drafted a bit prematurely. As long as I'm the only one expending this it should not go in the docs.

@drasbeck I think we figured it out in the #ubuntu-snap channel.  I think we can safely drop that and get the others merged.  Thanks :+1:

I guess we can close this :)

if we re-open and remove port 53 this might still be a useful note to add

I agree, the first part contains valuable information but we should remove the part with port 53, since I needed to open this port due to another problem entirely, and thus it does not apply to everyone.

Hi, I just removed the part with port 53.

@geekgonecrazy Can you review this one again, please?


Manually roll Flutter Engine from cbb9d0b06 to cef721963

https://github.com/flutter/engine/compare/37d8342ccac159892d93008f64c9a3ac0ed2f9e1...cef721963336c56e88127d82db6400ff1d247b26

Revert "[Windows] Move to Visual Studio 2019" (https://github.com/flutter/engine/pull/36606)

Issue: https://github.com/flutter/flutter/issues/112894
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Push an empty commit to this PR after the builds on engine CI have finished.


Non media files cannot be overwrited

Hello, thank you for including the ability to create Manifest Urls for forms. I created one last week using this feature and it went fine. Now I have changed some things within the compendia and wanted to update everything, manifest url and all included.
While C2M was creating the Module I was notified on screen "You may not upload non-media files which would overwrite an existing file on the server".
In fact then it happened that opening the Manifest URL from the browser, the text inside module.json still bore the previous module version numbers.
I don't know if it's an issue of C2M itself, but I wanted to report this, since I don't have direct access to the raspberry that hosts me and having to edit the files by hand would make everything cumbersome, but I'm aware that if the problem is only related to my condition, I can't expect the problem to be solved through the module.

That very much sounds like a security feature of Foundry, I shall investigate.

So yes, I can confirm it. The good thing: The new zip file gets saved properly and is updated. Unfortunately the module.json does not get updated, staying like it is. I have an idea how I can circumvent it, but maybe I can think of a better solution that doesn't feel quite as hacky :)

So yes, I can confirm it. The good thing: The new zip file gets saved properly and is updated. Unfortunately the module.json does not get updated, staying like it is. I have an idea how I can circumvent it, but maybe I can think of a better solution that doesn't feel quite as hacky :)

Thank you.


jedi autocomplete don't work.

Issue Type: Bug
once python interpreter is selected by vscode, editor don't give no any auto complete feature, that is to say, no matter type what in editor, there is no autocomplete window show.
And if disable JEDI from python user settings, the inner "Microsoft Python Analysis Engine" can work.
Extension version: 2020.1.58038
VS Code version: Code 1.41.1 (26076a4de974ead31f97692a0d32f90d735645c0, 2019-12-18T14:58:56.166Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8350U CPU @ 1.70GHz (8 x 1896)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.84GB (6.13GB free)

Process Argv
A:\py\p4p.py

Screen Reader
no

VM
0%

Please fill the issue template including the steps to reproduce (file content, settings etc.) We need more info to pin point the issue you're facing.

Closing as no further info was provided.


Improve article 1906.05274

Exact location of issue
equation 9 of [1906.05274] Efficient Exploration via State Marginal Matching
Problem details
This equation display is weird on chrome

It works correctly on firefox

Desktop (please complete the following information)

OS: Windows
Browser: chrome

High-chances it will auto-resolve in January when we get native MathML support in Chrome.
(Though other issues may need attention)
Will keep the issue open until then and revisit.

I think the display of such equations will be better now that we have Chrome 109 and native MathML Core support. We still have some issues to iron out first.
This article has been updated to the latest version, and I think this equations has been reworked. If I am not mistaken, it is now equation 3 in the PDF, and - strangely enough - equation 7 in ar5iv.
There seems to be a numbering bug we need to look into, the equation numbers go 1,2,7,8,... in this article.


Create Umbra class and test file

Creates a helper class called Umbra.ts. This class is intended to abstract away contract interactions, random number handling, and cryptographic operations from the frontend.
To test

Go to https://ropsten.etherscan.io/ and copy the latest block number
Paste the value in the startBlock fields of contractInfo in line 43 of Umbra.ts
Run yarn test

The above is necessary because if the block is too old, the ganache fork errors with could not detect network when scanning for events. Presumably this is because the ganache fork needs to scan for events emitted by transactions on the real Rospten, and our local fork. If we could start ganache with the deterministic flag we could run these tests on localhost, but it seems OpenZeppelin's test-environment doesn't support that, so this is the workaround for now. When we switch to hardhat we can run these tests locally instead of against Ropsten
Notes

Withdraw functionality and tests are not yet implemented since it's dependent on #72. These can be implemented in a separate PR since #72 requires a redeploy to Ropsten before we can test against it

Potential Improvements

Regarding the tsconfig.json and typechain setup, not sure if this is optimal for the monorepo. Specifically, when running yarn build from the root, it should compile the contracts directory first, then build the umbra-js files to ensure Typechain is run with the latest contract version. This will be impacted by @wildmolasses updating the contracts folder, so we can leave this as-is for now

hey @mds1, I'm unable to get this to build. At the top level, I run yarn install and get the following error:
$ lerna run prepare
lerna notice cli v3.22.1
lerna info versioning independent
lerna info Executing command in 1 package: "yarn run prepare"
@umbra/umbra-js: yarn run v1.22.10
@umbra/umbra-js: $ yarn build
@umbra/umbra-js: $ tsc --build && yarn build:types
@umbra/umbra-js: src/classes/Umbra.ts(15,47): error TS2307: Cannot find module '../../types/contracts' or its corresponding type declarations.
@umbra/umbra-js: error Command failed with exit code 1.
@umbra/umbra-js: info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
@umbra/umbra-js: error Command failed with exit code 1.
@umbra/umbra-js: info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
lerna ERR! yarn run prepare exited 1 in '@umbra/umbra-js'
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

I tried blowing away node_modules to no avail. Any recommendations?

hey @mds1, I'm unable to get this to build. At the top level, I run yarn install and get the following error:
$ lerna run prepare
lerna notice cli v3.22.1
lerna info versioning independent
lerna info Executing command in 1 package: "yarn run prepare"
@umbra/umbra-js: yarn run v1.22.10
@umbra/umbra-js: $ yarn build
@umbra/umbra-js: $ tsc --build && yarn build:types
@umbra/umbra-js: src/classes/Umbra.ts(15,47): error TS2307: Cannot find module '../../types/contracts' or its corresponding type declarations.
@umbra/umbra-js: error Command failed with exit code 1.
@umbra/umbra-js: info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
@umbra/umbra-js: error Command failed with exit code 1.
@umbra/umbra-js: info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
lerna ERR! yarn run prepare exited 1 in '@umbra/umbra-js'
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

I tried blowing away node_modules to no avail. Any recommendations?

Go to https://ropsten.etherscan.io/ and copy the latest block number
Paste the value in the startBlock fields of contractInfo in line 43 of Umbra.ts
Run yarn test

I know this is intended to be temporary until we get Hardhat working, but I wonder if it isn't a bit of code smell in itself. Instead of having only hardcoded values by chainId, maybe we should enable the user to optionally pass the chain configuration data into the constructor. After all, they could be using a version they deployed, or using an older version of umbra-js with a chain we recently deployed to (e.g. Matic or xDai or whatever).
Off the top of my head, not tested to be correct, this could be something like this:
type ChainId = number;

interface ChainConfigData {
  chainId: ChainId;
  umbraAddress: string;
  umbraPaymasterAddress: string;
  startBlock: number;
}

type ChainConfig = ChainId | ChainConfigData;

constructor(readonly provider: EthersProvider, chainConfig: ChainConfig, signer: JsonRpcSigner | null) {
  // If chainId is a `number` pull from hardcoded mapping, otherwise use the data provided
}

Go to https://ropsten.etherscan.io/ and copy the latest block number
Paste the value in the startBlock fields of contractInfo in line 43 of Umbra.ts
Run yarn test

I know this is intended to be temporary until we get Hardhat working, but I wonder if it isn't a bit of code smell in itself. Instead of having only hardcoded values by chainId, maybe we should enable the user to optionally pass the chain configuration data into the constructor. After all, they could be using a version they deployed, or using an older version of umbra-js with a chain we recently deployed to (e.g. Matic or xDai or whatever).
Off the top of my head, not tested to be correct, this could be something like this:
type ChainId = number;

interface ChainConfigData {
  chainId: ChainId;
  umbraAddress: string;
  umbraPaymasterAddress: string;
  startBlock: number;
}

type ChainConfig = ChainId | ChainConfigData;

constructor(readonly provider: EthersProvider, chainConfig: ChainConfig, signer: JsonRpcSigner | null) {
  // If chainId is a `number` pull from hardcoded mapping, otherwise use the data provided
}

The remaining related TODOs have been pulled out into #75, #78, and #80, so I'm going to merge this PR

The remaining related TODOs have been pulled out into #75, #78, and #80, so I'm going to merge this PR


[Storage] Support rewrite existing blob

Description

Feature spec: https://microsoft-my.sharepoint.com/:w:/p/rerdmann/Ecc1Q0_C7uJLs-ysOuv1sigBnC9DSpOmy8BTx0UAXvSdLg?e=OpNhoS&CID=6F46B2B5-765F-43C9-8F05-DD5E10DC0632&wdLOR=c8DFAEA4E-4577-435C-ABA7-D3D0DD327AB1
Need new SDK release  https://github.com/Azure/azure-multiapi-storage-python/pull/43
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

/azp run


Usage with react-navigation: Proper use of the module

What kind of Issue is this?
Discussion
Hi @timomeh ,
I wanted to try the new version of the module, but I'm having some doubts about how to use it in the right way.
I have three screens, in which every one of them has to refer to the tab, but I have to use react-navigation.
So which is the right use?
I tried createMaterialBottomTabNavigator, but it does not graphically reflect the module I would like to use as graphics, I prefer this.
Could you post some code examples?
Actual behavior

Is it happening on Android, iOS or both? Android

Environment

software
version

react-native-material-bottom-navigation
1.0.0

react-native or expo
0.55.4, latest version

node
latest version

npm or yarn
latest version

Hi! You maybe already read the section in the README about React Navigation Support, see here.
I don't think there is "a right way" to use react-navigation together with this Bottom Navigation. You could build your own Navigation View (read more), or you could simply use the Bottom Navigation on the Top Level (read more).
I don't have any working code examples at hand since I'm currently not working on any project which uses react-navigation. If I would use react-navigation together with the Bottom Navigation, I'd use it in the Top Level Component (here) together with a Stack Navigator, and onTabPress would dispatch a StackActions.reset Event.
⚠ Warning, dangerous pseudo code, just for demonstration purposes:
import { StackActions, NavigationActions } from 'react-navigation'

const AppNavigator = createStackNavigator(SomeAppRouteConfigs)

class App extends React.Component {
  handleTabPress = newTab => {
    this.navigator && this.navigator.dispatch(
      StackActions.reset({
        index: 0,
        actions: [NavigationActions.navigate({ routeName: newTab.key })]
      })
    )
  }

  render() {
    return (
      <View>
        <AppNavigator ref={nav => { this.navigator = nav }} />
        <BottomNavigation
          activeTab={this.navigator.state.routeName}
          tabs={[ /* ... */ ]}
          onTabPress={this.handleTabPress}
          // ...
        />
      </View>
    )
  }
}

If you implemented it in some way, feel free to post your solution so other people can learn from it. Then I could also use your solution and put it in the docs, or maybe you want to contribute directly and add it to the docs. Thanks!
Note: I'm going to close this issue since it's not an active bug or something similar. You can still comment below, and I'll still answer.

@timomeh : I'm trying to create an example on expo, but there's something wrong.
link: https://snack.expo.io/HJ0lsgOGm
Let me know where I'm wrong.

Also keep in mind that I currently have no deep experience in working with react-navigation. I don't know the current state of it, and how you would do different things. For more details on react-navigation integration I might be the wrong person to ask.


MRG: Additional checking of fNIRS data before processing

Reference issue
Further preparation for #9141
What does this implement/fix?
Now that we are going to have people creating their own data structures it will become more important to thoroughly check the ordering and encoding of fNIRS metadata.
Additional information
I will add checking for hbo/hbr type next, but wanted to thoroughly test the existing function before I expand it.

Render: https://26465-1301584-gh.circle-artifacts.com/0/dev/auto_tutorials/io/plot_30_reading_fnirs_data.html#storing-of-optode-locations

I think this is good to go (assuming CIs come back green, they do locally). @larsoner @agramfort @drammock could you please review.

Done. This PR is now only additional back end checks for fNIRS data. @larsoner could you please review again.

Thanks @larsoner Ill merge if all green.


Ability to let user type time

Is there a way to also let user type in the time either in the input or the popup box?

Ok, so I removed this on line 201 $('.wickedpicker__controls__control--hours').focus(); . Temp solution but it needs to be an option in the plugin.

Handling user input isn't something I want to add to the wickedpicker, but feel free to fork the project and add the feature.

I ended up using an input mask.


Armin 251 assets view

closes #251

Show currently chosen network and navigate to network selection page …
Show current account as an icon, open account settings on tap

showing net: nctr-gsl-dev is definitively very ugly. but let's leave it for now as we need to define what to show instead

You can use this mobx-computed value to get the connection state: https://github.com/encointer/encointer-wallet-flutter/blob/b72bc73bc796df6a0912b37ac1c2943e5d0c6f91/lib/store/settings.dart#L148

Review comments are implemented, ready for second review!


'title' property no longer does anything

I used to pass title='' to avoid having TimeAgo set a title attribute in the datetime tag it creates, since I'm doing my own tooltip — but now:

If I pass an empty string, the test here will be falsy, and it'll show an ISO string
If I pass another string, the test will be truthy and it'll show a timestamp

I think it might just be an operator precedence problem? You mean title || (date is string ? timestamp : ISO string), but the code behave as (title || date is string) ? timestamp : iso string

By the way, even if you add parenthesis, the problem won't be solved — an empty title is falsy, but imho should allow clearing the title attribute altogether. So the test really should be something like:
const passDownTitle = (typeof title === 'undefined'
     ? (typeof date === 'string'
      ? date
      : (new Date(date)).toISOString().substr(0, 16).replace('T', ' ')
     : title

but that's a little verbose, a helper method or two wouldn't hurt ;)

Fixed in my fork so we don't have to wait for upstream fix, will delete once resolved! https://github.com/fasterthanlime/react-timeago-titlefix

@fasterthanlime Pull Requests Welcome!
Since you've fixed it, might as well make a PR?


[enhancement] image tint/coloring

it would be pretty cool if we were able to color or tint images, like an extra argument

This actually already exists fortunately! You can set the color on any image element. If you are working with images in FormattedText elements there is a tag "ic" that allows you to specify the color.
This was added in version <IP_ADDRESS>.A2. All of the pre-release builds after also have it.

oh damn i'll have to check that out then


NIFI-3243: Rendering the selected Process Group in the header if possible

NIFI-3243:

Rendering the selected Process Group in the header if possible.

Reviewing....

Thanks @mcgilman this has been merged to master.


color sliders not working right

Color slider color options gets overwritten when colors picked. If you slide to black or white you lose all colors.

@ayhanyildiz This has always been a grievance of mine about the tinycolor library.

I hear you. If I can fix, i will send you PR

I have finally fixed this one in commit 92bf25b and released in v3.4.5

great thanks!


TV: Hitbox Thumbnails are missing (was Twitch thumbnails are missing, but it's hitbox)

I blame Twitch for this. So note to self: set some minimum dimensions.

Ah, this is actually caused by hitbox.

Hitbox is dead.


VSCode does not break at 120 characters on formatting

Description:
Formatting of a Ballerina VSCode file does not break at 120 characters while the intellij plugin does.
Related Issues (optional):
https://github.com/ballerina-platform/ballerina-lang/issues/13862

This will be implemented as a diagnostic provided by the Ballerina Linter. So instead of breaking from 120 characters, we will be providing a warning to the user that line is exceeding the max line length. I will attach this issue to the linter as a tracking point but we won't be breaking lines which are exceeding 120 lines with the formatting tool. We are planning to provide a code action to incorporate quick fixes to the linter issues and breaking the lines also will be a part of that.
I'm closing this issue as this will be related to the Linter implementation.


Preset profiles: Immersive and Shushudaa

I just installed DynamicCam 2 10.0 (fresh install - I didn't have a previous profile or used any previous version of the addon and it's a fresh wow install).
It's giving me this error when trying to load either Immersive and Shushudaa presets:
Error: Hearth/Teleport (200), condition, runtime
[string "for k,v in pairs(this.spells) do..."]:1: bad argument #1 to 'pairs' (table expected, got nil)
When selecting "Fix manually". it leads me to this page:

Yeah, please do not use the profiles in Beta. These are not finished. Sorry, I should have marked that more clearly.


Iterators

Create iterators for Z3 expressions to allow looping through their terms.

Fixed by #13.

Fixed by #13.


QR Code support

Hello, I couldn't quite find the mention of these APIs:
https://www.ups.com/uel/api/GetBarcode
and
https://iship.com/Root/Enterprises/Mobile Edop/Mobile Dropoff/shipping locations/Movement/dropoffs
We're using these to generate QR codes. They're not the standard UPS Webservices, but we just wanted to make sure, will these keep working past the June '24 deadline?

Hi, are you using access keys for both of these APIs?

Hi, the first one has no auth, the second one is using a different set of credentials.


fix(driver/bpf): accessing loginuid depends upon a config flag in the kernel

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug
Any specific area of the project related to this PR?
/area build
/area driver-bpf
What this PR does / why we need it:
The config flag is either CONFIG_AUDITSYSCALL for kernels < 5.1.0, or CONFIG_AUDIT for newer ones.
See https://github.com/torvalds/linux/blob/master/include/linux/sched.h#L1104.
I caught this build issue while testing an archlinux builder for driverkit using a cross-build to arm through qemu, building kernel 4.17.0.
Possibly, the archlinux arm folks did not enable those flags in their kernel configs.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

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

Done @Andreagit97 !

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

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


lncli connect does not understand ipv6 with port

 lncli connect 024ad958ec9accc5c81facd2d0144759a1e113bae1d33094d260c16ff65ca8dac1:2a02:ed0:4367:7000:781f:faf4:4eaa:f1cd:9735
[lncli] target address expected in format: pubkey@host:port

Hey @shtirlic! I believe this error is raised when the address provided is missing the @ sign https://github.com/lightningnetwork/lnd/blob/master/cmd/lncli/commands.go#L259. The command pasted seems to also be missing it, is this an exact copy of the command you used?

Hi, yes it was missing the @, after I added it errors with this:
lncli connect 024ad958ec9accc5c81facd2d0944759a1e113bae1d33094d260c16ff65ca8dac1@2a02:ed0:42fd:c700:cc27:dc8f:ce9b:3082:9735
[lncli] rpc error: code = Unknown desc = lookup 2a02:ed0:42fd:c700:cc27:dc8f:ce9b:3082:9735: no such host

ipv6 addresses need to be wrapped in square brackets. Does this work for you?
lncli connect 024ad958ec9accc5c81facd2d0944759a1e113bae1d33094d260c16ff65ca8dac1@[2a02:ed0:42fd:c700:cc27:dc8f:ce9b:3082]:9735


Restructure directories

The way the directories are layed out is a result of the intial VS Node.js template.

Move all the directories up a level
Put directives in more contained locations
Put controllers next to partials within their own folders

Completed in https://github.com/drewtorg/wramtas/commit/062b9317a543f01d96c640aeb40980e62e245f21


How do I get the title in the text？

I was looking for a good rich text editor and I found Quill. I need a title in the text to be displayed outside the editor, and click the title to scroll the content in the text, but I can't find it in the document, how can I implement it?

I was looking for a good rich text editor and I found Quill. I need a title in the text to be displayed outside the editor, and click the title to scroll the content in the text, but I can't find it in the document, how can I implement it?


How to start webpack

After cloning and installation with "npm start" I can't start project with "npm start".

There is no npm start script: https://github.com/reactjs/react-router-redux/blob/master/package.json


”Cormaia” is listed in the drop-down menu

Description:
"Cormaia" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Cormaia" in the search bar.

Expected results
The location is listed in the drop-down menu as "Cormaia, Bistrița-Năsăud (Sângeorz-Băi)".


training code

Hi! Thanks for your code! Is there any training related code? It looks like only a pre-trained model is loaded...

Finetuning / training from scratch pipeline is in the works, will probably be a while before it gets pushed though.


tasks.json TypeScript build produces different errors than tsc on command line

VSCode 1.10, Win10.
This is my task definition & the output of its run:

This is what I see when I execute tsc manually:

How can they be different?

I've tried changing something in tsconfig.json and now, the Build action in VSCode doesn't report any errors while tsc and WebStorm do.
My guess at the moment is that there is something funny going around tsconfig.json changes.

@borekb There is likely a mismatch between the version of TypeScript you are using in the editor and the tsc you are running. To see what may be causing this, please:

share your tsconfig.json.
Are you using a workspace version of TypeScript? What version? Do you have a typescript.tsdk setting?
What does tsc --version return?

Both versions are 2.2.1, both are invoked via tsc. I don't see a difference.

@borekb Please provide the other information as well

I cannot reproduce it so will close this issue. Just out of curiosity, @mjbvz, does it matter which version of TypeScript is configured for VSCode? I thought that these two lines in tsconfig.json:
{
    "command": "tsc",
    "args": ["-p", "."]
}

mean that tsc -p . will be run on a command line literally. So that will use my system-wide tsc, right?


Enh#102

Introduces mothur option to remove OTU with abundance lower/equal to a specific treshold.
Fixes test and their results accordingly.
Closes #102

Close due to problems with python 3.4

I see Python3.4 check failed during the preliminary steps. Restarted it for you.

Actually, I am tired of Python3.4 errors and 10 minutes runs @TravisCI just because of pandas compilation.  If this will not go fine this time we will abandon Python3.4 support.

Thanks :) I removed 3.4 as requested. It was really annoying.


fix: prevent horizontal scrolling in text areas and containers

Updated index.css, style.css, and output.css to improve responsiveness.

Set overflow-x: hidden on text areas and containers to eliminate horizontal scrollbars.

Ensured all text areas adapt to container width for better mobile experience.

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
The changes introduced in this PR are classified as a bug fix. The updates enhance the responsiveness of the application by preventing horizontal scrolling in text areas and containers, thereby improving the user experience, especially on mobile devices.

Why was this change needed? (You can also link to an open issue here)
This change was needed to enhance the user experience on mobile devices by preventing unnecessary horizontal scrolling in text areas and containers. Users were experiencing layout issues, as reported in [issue #1218], which made the application less user-friendly. By addressing this bug, we improve responsiveness and overall usability.

@ManishMadan2882 sure.


Install main.py as a command called bibliopixel in setup.py

I didn't call main.py "bibliopixel" because of course there would be conflicts...

No worries... gotta remember how to make this work on windows...
On Thu, Mar 2, 2017 at 1:50 PM, Tom Swirly<EMAIL_ADDRESS>wrote:

Assigned #123 https://github.com/ManiacalLabs/BiblioPixel/issues/123 to
@adammhaile https://github.com/adammhaile.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/123#event-984187370,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6vs8oUZnImSP6Uko919VefoUbKMMks5rhw9ZgaJpZM4MRYOQ
.

Exactly why I assigned it to you! :-)

Complete.


Loading multiple postscribe calls on same page in parallel

Hi,
We're trying to use postscribe in our script, however there's another script on the webpage which uses postscribe and there are run-time conflicts happening between the first & the second postscribe call.
Any help would be appreciated.
Thanks in advance

Similar-ish issue here.
For some specific ad formats, our adserver requires to implement the adserver calls simulataneously and not one after each other.
Is there a way with postscribe to write all the placements in the div, delay until all the calls are prepared and then do the calls simultaneously?
Thanks !

Can I get a place where I can reproduce this? @jguerine pls file a separate ticket I don't think your concern is related (but, I don't know much about either issue..)

@neeagl For current version of postscribe, you can capture the value. The next version will support more ways to load. Capturing would look something like (assuming you have a build step):
var myCopy = (function() {
  var oldPostscribe = window.postscribe;
  delete window.postscribe;
  // original postscribe code
  var myPostscribe = window.postscribe;
  window.postscribe = oldPostscribe;
  return myPostscribe;
})();

We believe this is fixed in 2.0.0.  Please retest against 2.0.0 release and open a new issue following Issue Reporting guidelines in 2.0.0 if you believe this is still an issue.


fix: Run rigetti tests in the time window

Issue #, if available:
Description of changes:
fix: Run rigetti tests in the time window
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Confirmed with Kshitij, don't merge until we have fix working for tests.

Remove Jobs Notebooks from EXCLUDED_NOTEBOOKS


Upgrade to nom 5 and replace all macros!

Not yet finished.
So far it's a 1:1 translation of the old code to the new way of doing things.
I think there's now a chance to simplify and clean it up a bit.

Ah heck, I merge as is and we can work on cleaning it up in follow-ups.


Check if the pk of the model is type uuid and convert it to a str

closes #154

Sorry forgot to import UUID


When compiling on the server side, pass my keystore password in the form of parameters. Is there a good way?

When compiling locally, my keystore password will be exposed. I want to use mirakle to compile on the server. When compiling on the server side, pass my keystore password in the form of parameters. Is there a good way?

The keystore password is stored on the server and only I know it


error when react-native run-android

:react-native-device-info:processReleaseResourcesC:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_icon_dark.xml:5:27-80 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_icon_dark_disabled').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_icon_dark.xml:8:27-79 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_icon_dark_pressed').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_icon_dark.xml:13:27-78 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_icon_dark_normal').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_icon_dark_focused.xml:14:29-80 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_icon_dark_normal').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_icon_light.xml:5:27-81 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_icon_light_disabled').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_icon_light.xml:8:27-80 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_icon_light_pressed').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_icon_light.xml:13:27-79 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_icon_light_normal').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_icon_light_focused.xml:14:29-81 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_icon_light_normal').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_text_dark.xml:7:27-80 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_text_dark_disabled').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_text_dark.xml:10:27-79 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_text_dark_pressed').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_text_dark.xml:15:27-78 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_text_dark_normal').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_text_dark_focused.xml:14:29-80 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_text_dark_normal').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_text_light.xml:5:27-81 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_text_light_disabled').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_text_light.xml:8:27-80 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_text_light_pressed').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_text_light.xml:13:27-79 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_text_light_normal').
C:\Users\lfarfan\Desktop\projects\ReactNative\react-native-starter-app\node_modules\react-native-device-info\android\build\intermediates\exploded-aar\com.google.android.gms\play-services-base\10.0.1\res\drawable\common_google_signin_btn_text_light_focused.xml:14:29-81 : No resource found that matches the given name (at 'drawable' with value '@drawable/common_google_signin_btn_text_light_normal').
FAILED
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':react-native-device-info:processReleaseResources'.

com.android.ide.common.process.ProcessException: org.gradle.process.internal.ExecException: Process 'command 'C:\Users\lfarfan\AppData\Local\Android\sdk\build-tools\23.0.1\aapt.exe'' finished with non-zero exit value 1

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED
Total time: 1 mins 54.606 secs
Could not install the app on the device, read the error above for details.
Make sure you have an Android emulator running or a device connected and have
set up your Android development environment:
https://facebook.github.io/react-native/docs/android-setup.html
(node:11424) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): TypeError: Cannot read property 'message' of undefined
(node:11424) DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Hi luisfarfan,
I already same kind of issue when run-android. In my case i fixed by changing directory of source like I initially got the build issue in C drive desktop folder, then i changed into D drive it works without any issue.
Sorry for bad english. Hope this helps to you.

This helped me out if you dont have Build Tools 23.0.1 installed http://stackoverflow.com/questions/36683726/failed-to-find-build-tools-revision-23-0-1/39828902


Add Model::matches as special-cased proxy to _.matches

Discussion → #3070

This method should be rewritten on something which not produces proxy function. You not using it in Collection#where which is proof of my words, it's bad method for loop callbacks. Code is elegant but performance is just not good enough

@redexp That could be accomplished with https://github.com/jashkenas/underscore/issues/1985.


using components in templates

Is it possible to use for example tooltip from vue-strap in template?
<tooltip placement='bottom' content='test'></tooltip>
works like charm outside vue-tables.
But when I use it in template:
'Proces': "<tooltip placement='bottom' content='dupa dupa dupa'></tooltip>"
I get: build.js:9218 [Vue warn]: Unknown custom element:  - did you register the component correctly? For recursive components, make sure to provide the "name" option.

You probably registered the component locally on your instance, and so the table instance, which is a child, cannot access it. You need to register it globally.


Bugs with (Up+)Ctrl+Click shortcut

Item duplication?

With ctrl+click involved, this could be related to Not Enough Items being installed.

Insufficent report, works fine for me. Plus, it might be fixed by another shortcut patch.


UADP NetworkMessage sytntax

Is your feature request related to a problem? Please describe.
I need the definition of the Opc.Ua.PubSub.NetworkMessage syntax for the current implementation published on the master branch.

Hi Mariusz,
You can find the Pub/Sub message encoding logic in this file: https://github.com/Azure/Industrial-IoT/blob/master/components/opc-ua/src/Microsoft.Azure.IIoT.OpcUa.Protocol/src/Stack/PubSub/NetworkMessage.cs
We don't have the schema documented since this implementation is rather specific to our solution and I believe it is not completely spec compliant. We are working right now on making it compliant so I would kindly ask for your patience until it's done. I'll get back to you as soon as we have a compliant preview.

Hi @cristipogacean,
Thanks for your answer. I know where it is, but my concern is if there is any compliant implementation you know. My implementation also needs to be upgraded mpostol/OPC-UA-OOI#208 and it looks similar to my implementation compliant with a draft proposal of the specification. Now I am working on a road map and priorities of work on my side so any information that I can use for harmonization of our work is welcome.

I believe our friends at Softing and Unified Automation have compliant implementations in their SDKs and trial/demo apps. I only briefly tested Softings PubSub in the .net toolkit. The issue with UADP is that there are a lot of flags and knobs to setup the messages and some require state-full decoding. It's pretty hard to test and support everything.

I have collected the results of my investigation here, I suppose no one is compliant.
The syntax defined by the spec is really very complex and the native language text-based. I have worked out a file that contains Augmented BNF definition](https://github.com/mpostol/OPC-UA-OOI/blob/master/Networking/SemanticData/MessageHandling/NetworkMessage.abnf) of the NetworkMessage.  I hope we could use it to automatically generate a parser.

closing for now, please reopen if needed


DASHBUILDE-111: Some ELS tests fails on different time zones.

Hi @dgutierr , same bug fixings for 0.3.x. Thx!

There are 3 ELS tests failing. I forced a jenkins rebuild and failed again.
@romartin Can you double check please?


How to change the color of text in a table ?

I use day and night themes. Help me change the color of the text in the table.
Thanks.

Hi @Vovusik,
if you are using the SimpleTableDataAdapter just call the setTextColor() method. You just have to care to put the color value and not the color resource.
Best regards,
Ingo

Thank you very much for the help. But I need to use the color from the resource.
Example from the library:
Final SimpleTableHeaderAdapter simpleTableHeaderAdapter = new SimpleTableHeaderAdapter (context,
                 R.string ....);
SimpleTableHeaderAdapter.setTextColor (ContextCompat.getColor (context, R.color.colorWhite));
SetHeaderAdapter (simpleTableHeaderAdapter);
But using the SimpleTableDataAdapter

It is the same for the data adapter.
SimpleTableDataAdapter.setTextColor(ContextCompat.getColor(context, R.color.colorWhite));

I can not overcome this.
If you have the opportunity, help me, please.
Thank you very much.
SimpleTableDataAdapter simpleTableDataAdapter = new SimpleTableDataAdapter(context, (List<String[]>) getContext());
simpleTableDataAdapter.setTextColor(ContextCompat.getColor(context, R.color.colorAccent));
setDataAdapter(simpleTableDataAdapter);
This example does not work.

I have checked if there is some regression, but the feature is working as expected. The code I used you can see here:
final String[][] data = new String[][]{
            {"BMW", "3er", "120PS", "20.000€"},
            {"BMW", "4er", "130PS", "30.000€"}};

final SimpleTableDataAdapter dataAdapter = new SimpleTableDataAdapter(this, data);
dataAdapter.setTextColor(ContextCompat.getColor(this, R.color.table_data_text));

final TableView<String[]> tableView = (TableView) findViewById(R.id.tableView);
tableView.setDataAdapter(dataAdapter);

If this code is not working for you, the issue seems to be in another part of your application.


[FLOC-2993] Document containerized installer

Note that as a change to the labs docs this only needs to be checked to see that it doesn't break the Flocker build.
Recent changes to the unofficial Flocker tools installer mean that:

The installer itself can be run inside a container
The installer can use Terraform to automatically provision nodes on AWS
The installer now works a lot more reliably, in particular on Ubuntu 14.04

This PR documents these changes.
This PR also includes changes as agreed with @alicesowerby and @robhaswell to run an experiment where the main installation pages offer the chance for people to try the Labs installer.

This includes a bunch of random arbitrary changes that are not just for labs docs.

@itamarst I've removed the unrelated changes. Sorry about that. Everything in this PR should be as per the description now.

Merging since Madhuri took a look over this and gave approval to merge in JIRA. Thanks!
As follow-up, I will make a code change in UFT itself to warn users about keys not existing, giving the user-facing not the container-facing path.

Docs build is passing, since this is a docs-only change, other failing tests must be unrelated, so merging.


[mle] handle rx key seq update based on MLE message class

This commit implements new mechanism in Mle related to key sequence
update when an MLE message is received with a larger key sequence
compared to the one being used by KeyManager. The MLE messages are
categorized into classes of Authoritative, Tentative, or Peer
depending on the MLE command type and included TLVs. The class
determines different actions: ~Tentative is used where the sender of
MLE message may not be sure that its key seq is in sync with the rest
of the Thread network, so the receiver will not adopt the key seq.~
Authoritative indicates sender is confident that its key seq is in
sync, so receiver will adopt the larger key seq. Peer class is used
when both sender and receiver think they are in sync. In this case,
if the MLE message is from a known neighbor, receiver will adopt the
larger key seq if the difference is one, otherwise it will try to
re-establish link with the neighbor using Authoritative message
exchanges (e.g. sending Link Request or Child Update Request).

This is related to SPEC-1025 and Thread spec sections 7.3.12

Here are how different MLE message are categorized into different calsses:
Advertisement                  Peer
DataRequest                    Peer
DataResponse                   Peer
ParentRequest                  Peer
ParentResponse                 Authoritative      (always includes the ResponseTlv)
ChildIdRequest                 Authoritative      (always includes the ResponseTlv)
ChildIdResponse                Peer
Announce                       Peer
ChildUpdateRequest             Peer
ChildUpdateResponse            Authoritative if has ResponseTlv, Peer otherwise.
LinkRequest                    Peer
LinkAccept                     Authoritative     (only if has ResponseTlv)
LinkAcceptAndRequest           Authoritative     (only if has ResponseTlv)
LinkMetricsManagementRequest   Peer
LinkMetricsManagementResponse  Peer
LinkProbe                      Peer
TimeSync                       Peer              (OT only for Time Sync feature).

High-level algorithm on when to adopt a higher rx key seq:
A) If rx key seq > dev key seq
   ~Tentative: Do not adopt the rx key seq~
   Authoritative: Do adopt the rx key seq
   Peer:
           If from a known neighbor
               If delta > 1, do not adopt, try to re-establish link
               if delta = 1, do adopt
           else
               do not adopt

B) If rx key seq == dev key seq
    No changes

C) If rx key seq < dev key seq
    Do not adopt (still process the message)

The "re-establish link" mechasnim:
If device is child and neighbor (from which MLE msg is rcved) is parent -> Send Child Update Request with Challenge TLV
If device is FTD:
    If neighbor is a router -> Send Link Request (include challenge TLV)
    If neighbor is a child of device -> Send Child Update Request (with Challenge TLV).

Thanks @EskoDijk for reviewing this PR.

To save code, I think all of the assignments "aRxInfo.mClass = RxInfo::kPeerMessage;" could be deleted and replaced by a single one of this assignment on top where the aRxInfo instance is initialized. Since all messages not explicitly defined otherwise should be 'peer'. In case of an unknown/invalid message type this assignment can always be set again to kUnknown in the exit code section. This should work since all of the Handle...() methods for particular MLE message types are private members and all the Handle..() methods are only called from HandleUdpReceive().

I considered this model at first, but I think this will increase code size even more.
The idea in this PR is that we set mClass only after we validate the MLE message (i.e., it is well-formed, contains proper well-formed TLVs, and all required TLVs, etc). We use the fact that mClass is still kUnknown when we want to decide whether to adopt the received key seq or not,.
Other options are:

To have all Handle{MleMesssage}() methods track and return error which is a bigger change (and may increase code size more).
Or  in Handle{MleMessage}() during parsing if any error happens to set mClass to kUnknown (which I think will impact code size more than just setting mClass at the end of Handle{MleMessage}() method.

We can also consider to not process (handle) any Peer MLE messages with peerKSC > myKSC+1, see SPEC-1025 discussion.

I am not sure if this is a good idea and/or adds much value.
I see this classification into tentative/peer/authoritative as whether we want to adopt a higher key sequence or not and not tied to wether we want to process or ignore the message.
It makes sense to keep the logic for processing or ignoring the received message simple and tied to whether it passes security checks and frame counters check, etc. We do process a message if key seq is lower (no matter by how much).

As a general review remark: the 'peer' type MLE messages are all processed normally when delta-tKSC > 1 even though the spec text says it should not. Any risk identified in this? I tried to think of any; in worst case an attacker might succeed to construct based on captured traffic a fake MLE message with delta-tKSC > 1 and let this be accepted by the receiver but it would take a long attack time. And then nothing bad really happens, no secrets are revealed. The receiver is triggered to redo the link with its peer.

In the new push, changed the PR to remove "tentative" class and use "peer" in its place.

As a general review remark: the 'peer' type MLE messages are all processed normally when delta-tKSC > 1 even though the spec text says it should not. Any risk identified in this? I tried to think of any; in worst case an attacker might succeed to construct based on captured traffic a fake MLE message with delta-tKSC > 1 and let this be accepted by the receiver but it would take a long attack time. And then nothing bad really happens, no secrets are revealed. The receiver is triggered to redo the link with its peer.

I cannot think of any. One example situation is for "Parent Request" from an unknown child with delta-tKSC > 1. I think we should still process it on parent and respond to the child (though we won't adopt the higher tSKc until we exchange "authorative msg" with the child). One situation I can think of for this to happen is a BR/router type device which was off for some long time and then user powers it up again (device will be using old key seq while other nodes in network may have moved forward).

One example situation is for "Parent Request" from an unknown child with delta-tKSC > 1. I think we should still process it on parent and respond to the child

Yes, that's essential. In the former 'tentative' class this was described in the spec as always "partially processed" but now for the new 'peer' class we define it as "processed, but don't change any state based on it" if delta-tKSC > 1. So we've essentially merged tentative into the peer class and that seems okay with slightly different behavior in some (corner) cases. Slightly improved behavior, I would also say.
The changes now look good to me.


Add makeAvailable claims to ScenarioPlanningUnitsInclusionProcessor

Updating the  ScenarioPlanningUnitsInclusionProcessor to handle makeAvailable claims
Overview
This PR is to be merged to #1332 (changing DTOs to accept makeAvailable claims) and updates the ScenarioPlanningUnitsInclusionProcessor for it to receive PUs via geoJSONs and ids that will change their status to available and lock_status_set_by_user to true
Designs
NA
Testing instructions
The existing test for changing PU status has been expanded with a use case when selected PUs are first updated as excluded and in the following step are updated as available, lock_status_set_by_user is also tested to be true
Feature relevant tickets
MRXN23-78
MRXN23-81
Checklist before submitting

[ ] Meaningful commits and code rebased on develop.
[ ] If this PR adds feature that should be tested for regressions when
deploying to staging/production, please add brief testing instructions
to the deploy checklist (docs/deployment-checklist.md)
[ ] Update CHANGELOG file

thanks @yulia-bel!
a quick recap of the changes we discussed:

remove the .todo tests as they would not apply anymore in any case
test on contrasting claims: add overlapping makeAvailable claims (from geo)
existing test "when changing lock status via GeoJSON with multiple features per feature collection -> it marks relevant pu in desired state": leave as is, and create a new test that checks what you did in the current version of the pr, that is, that after initial calculation of includes and excludes, when user supplies a new set of claims that include makeAvailable, these are applied and override the previous ones; also renaming things to avoid ambiguity between "make available" and "leave untouched" and avoiding to repurpose planningUnitsToBeExcluded
new test where a protected area is set, with very low threshold (so that even barely intersecting PUs are considered protected by default, and are locked-in by default), checking that the relevant PUs are locked in, and then applying a user geometry for makeAvailable claims that partly overlap the protected/locked-in PUs, and asserting that the features made available by the user are really available

It's quite a lot, but we knew that tests would be the real core of this task 😬
thanks!


CI: sanity check for req. pkgs

What does this PR do?
it shall prevent situations like #11811 when we were silently skipping some tests which shall be running
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

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

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

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

Did you have fun?
Make sure you had fun coding 🙃
cc @borda @carmocca @akihironitta

As #11811 is merged into master, all running jobs here should be green now 👀


ExpansionPanel List Styling Options Limited to cardColor

I have been trying to use flutter's ExpansionPanelList widget, however, styling options are quite limited. The widget automatically uses cardColor attribute of the current theme and from what I see, the only option to apply custom styles/widgets is to override the _ExpansionPanelListState, which I can't because it is private. As per this S.O comment, currently the only solution is to copy the entire expansion_panel.dart file and override the widgets's build method.

@shihaohong I would like to work on this issue, the color for the ExpansionPanel is decided inside the MergableMaterial where we can first add a new property to change the color, and later we can expose that in ExpansionPanelList.
I found one more issue in the implementation, that it completely ignores the CardTheme.color which should be the first priority, and it only focuses on ThemeData.cardColor.  So, in the first PR to add a new property to MergableMaterial I would also fix that. WDYT?

Sounds good! I would first add the feature to modify the color by hand. In terms of fixing the color to first reference CardTheme.color, make that a separate change but first write a design doc, since it'd be a breaking change. It might also make sense to create a new issue to track that since it's not related to the original issue.

Also something to consider is being able to modify every card's color separately instead of having the entire list specified. For example, maybe I'd want ExpansionPanelList.tileBackgroundColor to be yellow, but maybe I want one or two of them to be green or pink, so make sure to take that into consideration as well.

Sounds good! I would first add the feature to modify the color by hand. In terms of fixing the color to first reference CardTheme.color, make that a separate change but first write a design doc, since it'd be a breaking change. It might also make sense to create a new issue to track that since it's not related to the original issue.

Sure, I'll create a new issue to keep track of it.

Also something to consider is being able to modify every card's color separately instead of having the entire list specified. For example, maybe I'd want ExpansionPanelList.tileBackgroundColor to be yellow, but maybe I want one or two of them to be green or pink, so make sure to take that into consideration as well. I'm not sure if that's easy or not to do, but would be nice to add along with your intended/mentioned behavior.

That's a good suggestion. I had a look at the source code of MergeableMaterial, and these are my findings. Instead of having color property defined inside MergeableMaterial it would make more sense to declare it inside the MaterialSlice. Instead of adding color here and here it would make more sense to add color here when the MaterialSlices are being added to slices which are then added to ListBody.
Since MaterialGaps cannot be the first in the MergeableMaterialItem list, take a look here  the color will always be added to the slices.
Once, we have added the above property, we can have color property inside ExpansionPanel and when we are creating MergeableMaterialItem list in ExpansionPanelList, we can add the color to MaterialSlice which will be taken from ExpansionPanelList[index]. Take a look here

@shihaohong I have submitted the PR for the first part. Looking forward to your reviews :)


Update closure-linter version

work in progress

Closing due to lack of activity. @fredj, please reopen when you get to working on this again.


Trouble with AngularJS lifecycle.

I'm trying to expose some of my directive's methods to $scope, so those functions could be called from the controller. This directive is being used as a wrapper for my form directives, and it has a controller so that other directives can communicate with it. Also, this directive is the "official event emitter" for my form. The directive looks like this:
function MyForm() {
    return {
        restrict: 'A',
        scope: false,
        require: 'form',
        transclude: false,
        controller: ['$scope','$element','$attrs','$timeout', function($scope, $element, $attrs, $timeout) {
            let ctrl = this;

            $scope.obj = {}
            $scope.obj.fun1 = fun1;
            $scope.obj.fun2 = fun2;
            $scope.obj.fun3 = fun3;
            $scope.obj.fun4 =fun4;

            function fun1 (par1, par2){
                // code
            }

            function fun2(){
                // code
            }

            function fun3 (par1, par2, par3, par4){
                // code
            }

            function fun4(){
                // code
            }

        }]
    }
}

And on the controller, i'm trying to acess this  obj and its functions like this:
function MainController($scope,$timeout){
    $scope.obj.fun1('name', 'foo');
   //Cannot read property 'fun1' of undefined
}

i'm trying to figure it out the problem, and when i use $timeout around the function on $scope, i can call the function. How do i solve it?

If you use the ngController directive on the element, It runs before your directive runs due to the priority (ngController priority is 500 and yours 0).
To make your directive run before the ng-controller directive simply add a greater priority to it than 500.
Check the $compile api for reference https://docs.angularjs.org/api/ng/service/$compile.

Hello @ozsay, and thanks!
Even if i add a greater priority value at the directive, it still doesn't work because i'm not using ng-controller, i'm using ui.state. :disappointed:

do you mean ui.router ? is MainController what you attach to the state ? see if https://github.com/angular-ui/ui-router/wiki#resolve can help you

Yeah, sorry. I was looking for a way to just make this value available from the directive itself.

This looks like a usage question. Please use one of these channels (especially important if you use third-party modules): https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#question Thanks!


VM tests with OCS disks

What this PR does / why we need it:
This PR introduces VM start/stop/delete and serial console connection with OCS disks, if Ceph-RBD is present within the cluster.
Release note:

Enhance the functional tests with running basic VM operations using OCS disks.

/cc @slintes @stu-gott

/retest

/test pull-kubevirt-e2e-k8s-1.15.1-ceph

@slintes it looks good - the OCS tests are executed within pull-kubevirt-e2e-k8s-1.15.1-ceph and skipped in other lanes

/hold
missing:

lifecycle+console tests with volumeMod = File
remove data volume on test-cleanup
wait for data volumes to be imported in lifecycle+console test

/test pull-kubevirt-e2e-k8s-1.15.1-ceph

/hold cancel

/test pull-kubevirt-e2e-k8s-1.15.1-ceph

/test pull-kubevirt-e2e-k8s-1.15.1-ceph

/test pull-kubevirt-e2e-k8s-1.15.1-ceph

/lgtm
/approve
/retest

I didn't see this failure before, appeared on both os-3.11.0 and os-3.11.0-crio.
VM Live Migration Starting a VirtualMachineInstance with a shared ISCSI Filesystem PVC should migrate a vmi with UNSAFE_MIGRATION flag set
Also k8s 1.11 has some more failures than usual. Let's retry...
/retest

New changes are detected. LGTM label has been removed.

Just resolved conflicts in vm_test.go :)

/test pull-kubevirt-e2e-k8s-1.15.1-ceph

/test pull-kubevirt-e2e-k8s-1.15.1-ceph

extracted the migration stuff yesterday as the failures seemed related to migration tests but apparently they still fail but specifically on k8s 1.11.0 which I didn't check locally, checking now

/retest

/test pull-kubevirt-e2e-k8s-1.15.1-ceph

/retest

Seems like the issue in the tests was fixed - I accidentally removed PV creation that is needed in one of the tests...

/lgtm
/retest

/cherrypick release-0.20


Unable to create a NextJS project

Describe the bug
I'm using this simple prompt to create a NextJS app:
Create a todo app with NextJS and Shadcn

Then I get this response:

And there it stands for ever. Spinning on the first command. Looking an the JavaScript console shows the following:

So I assume that the command to create the NextJS app is still waiting for a user feedback (if he wants to use Turbopack yes or no). I could be handled without user interaction,
We could make the command non-interactive by adding the --yes or -y parameter. Here's the complete command:
npx create-next-app@latest todo-app --typescript --eslint --tailwind --app --src-dir --use-npm --yes

Alternatively, we could also set all other parameters as defaults by prefixing them with --no- if you don't want them. For example:
npx create-next-app@latest todo-app --typescript --eslint --tailwind --app --src-dir --use-npm --no-import-alias --yes

The most important parameters are:

--yes or -y: Skips all prompts and uses defaults
--typescript: TypeScript setup
--tailwind: TailwindCSS installation
--eslint: ESLint setup
--app: Uses the App Router (instead of Pages Router)
--src-dir: Creates a src/ directory
--use-npm: Uses npm instead of yarn
--import-alias: Sets the import alias (default is @/*)

Link to the Bolt URL that caused the error
http://localhost:5173/chat/nextjs-shadcn-todo-app
Steps to reproduce
I installed the fork locally on my mac and run it with pnpm run dev.
And then simply use the aforementioned prompt:
Create a todo app with NextJS and Shadcn

Expected behavior
It should create a NextJS app and doesn't wait for user interactions that I can't do.
Screen Recording / Screenshot
No response
Platform

OS: macOS
Browser: Google Chrome Canary
Version: 132.0.6826.0

Additional context
No response

I would also like to note that I have no "Bolt" terminal, like at bolt.new where I can see what he does.
Here is the official version:

And here how it looks like when I run this fork locally:

next also not working for me. I also see it is not available within bolt, even my powershell says it is there:

I'm getting a similar issue
It is stuck on the initial setup

Same when trying Nuxt project.

Same issue here!