CupertinoPageRoute throws exception and Navigator is stuck

Steps to Reproduce

Pop a route by pop gesture (a.k.a swipe back)
Push a route immediately

In the sample application below, it is reproduced by the following steps:

Push "Next"
Swipe back
Push "Next" immediately

import 'package:flutter/material.dart';
import 'package:flutter/cupertino.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text("Pop gesture"),
      ),
      body: RaisedButton(
        child: Text("Next"),
        onPressed: () {
          Navigator.push(context, CupertinoPageRoute(
            builder: (BuildContext context) {
              return Scaffold(
                backgroundColor: Colors.yellow,
                appBar: AppBar(title: Text("Second")),
              );
            },
          ));
        },
      ),
    );
  }
}

Logs
I/flutter (30832): ══╡ EXCEPTION CAUGHT BY ANIMATION LIBRARY ╞═════════════════════════════════════════════════════════
I/flutter (30832): The following assertion was thrown while notifying status listeners for AnimationController:
I/flutter (30832): Cannot reuse a CupertinoPageRoute<dynamic> after disposing it.
I/flutter (30832): 'package:flutter/src/widgets/routes.dart': Failed assertion: line 217 pos 12:
I/flutter (30832): '!_transitionCompleter.isCompleted'
I/flutter (30832):
I/flutter (30832): Either the assertion indicates an error in the framework itself, or we should provide substantially
I/flutter (30832): more information in this error message to help you determine and fix the underlying cause.
I/flutter (30832): In either case, please report this assertion by filing a bug on GitHub:
I/flutter (30832):   https://github.com/flutter/flutter/issues/new?template=BUG.md
I/flutter (30832):
I/flutter (30832): When the exception was thrown, this was the stack:
I/flutter (30832): #2      TransitionRoute.didPopNext (package:flutter/src/widgets/routes.dart:217:12)
I/flutter (30832): #3      NavigatorState.pop (package:flutter/src/widgets/navigator.dart:1944:23)
I/flutter (30832): #4      _CupertinoBackGestureController._handleStatusChanged (package:flutter/src/cupertino/route.dart:655:17)
I/flutter (30832): #5      _AnimationController&Animation&AnimationEagerListenerMixin&AnimationLocalListenersMixin&AnimationLocalStatusListenersMixin.notifyStatusListeners (package:flutter/src/animation/listener_helpers.dart:192:19)
I/flutter (30832): #6      AnimationController._checkStatusChanged (package:flutter/src/animation/animation_controller.dart:736:7)
I/flutter (30832): #7      AnimationController._tick (package:flutter/src/animation/animation_controller.dart:752:5)
I/flutter (30832): #8      Ticker._tick (package:flutter/src/scheduler/ticker.dart:228:5)
I/flutter (30832): #9      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:988:15)
I/flutter (30832): #10     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding.handleBeginFrame.<anonymous closure> (package:flutter/src/scheduler/binding.dart:904:11)
I/flutter (30832): #11     __InternalLinkedHashMap&_HashVMBase&MapMixin&_LinkedHashMapMixin.forEach (dart:collection/runtime/libcompact_hash.dart:367:8)
I/flutter (30832): #12     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding.handleBeginFrame (package:flutter/src/scheduler/binding.dart:902:17)
I/flutter (30832): #13     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._handleBeginFrame (package:flutter/src/scheduler/binding.dart:832:5)
I/flutter (30832): #17     _invoke1 (dart:ui/hooks.dart:223:10)
I/flutter (30832): #18     _beginFrame (dart:ui/hooks.dart:162:3)
I/flutter (30832): (elided 5 frames from class _AssertionError and package dart:async)
I/flutter (30832):
I/flutter (30832): The AnimationController notifying status listeners was:
I/flutter (30832):   AnimationController#1892e(⏮ 0.000; paused; DISPOSED; for CupertinoPageRoute<dynamic>(null))
I/flutter (30832): ════════════════════════════════════════════════════════════════════════════════════════════════════

and
I/flutter (30832): Another exception was thrown: 'package:flutter/src/widgets/navigator.dart': Failed assertion: line 1666 pos 12: '!_debugLocked': is not true.

[✓] Flutter (Channel unknown, v1.2.1, on Mac OS X 10.14.2 18C54, locale ja-JP)
    • Flutter version 1.2.1 at /Applications/flutter
    • Framework revision 8661d8aecd (3 weeks ago), 2019-02-14 19:19:53 -0800
    • Engine revision 3757390fa4
    • Dart version 2.1.2 (build 2.1.2-dev.0.0 0a7dcf17eb)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/najeira/Library/Android/sdk
    • Android NDK at /Users/najeira/Library/Android/sdk/ndk-bundle
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = /Users/najeira/Library/Android/sdk
    • ANDROID_SDK_ROOT = /Users/najeira/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.0, Build version 10A255
    • ios-deploy 2.0.0
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 33.0.1
    • Dart plugin version 182.5215
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[✓] IntelliJ IDEA Ultimate Edition (version 2018.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 31.1.1
    • Dart plugin version 181.4203.498

[✓] VS Code (version 1.31.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 2.24.0

[✓] Connected device (2 available)
    • Android SDK built for x86 • emulator-5554    • android-x86   • Android 8.0.0 (API 26) (emulator)

This looks like a bug fixed just yesterday - https://github.com/flutter/flutter/pull/28756

I confirmed that this was fixed with #28756
However, I found another problem, I will reproduce and open issue.


Get integration testing working again

Unity 5.5 broke compatibility with the unity test tools we were using, so they have been removed and disabled for now.  We need to figure out the right way to do our testing, whether thats fix the test tools package, find another one, or make a lightweight version ourselves.

Possibly switch to using something like this? https://www.assetstore.unity3d.com/en/#!/content/27388

@Amarcolina We're using NUnit now, so we're good on this aren't we?

We are good yes!


1115 Updating CEP18 Tutorials

What does this PR fix/introduce?
This PR completely revamps the CEP18 tutorials.
Links have been fixed, code snippets updated and flow corrected so that it makes sense. I will open an additional ticket to update the Testing Framework tutorials, as the previous content was no longer accurate in any way.
Additional context
Update CEP18 Tutorials #1115
Checklist
(Delete any that aren't relevant)

[x] Docs are successfully building - yarn install && yarn run build.
[x] For new internal links I used relative file paths (with .md extension) - e.g. ../../faq/faq-general.md - instead of introducing absolute file path, or relative/absolute URL.
[x] All external links have been verified with yarn run check:externals.
[x] My changes follow the Casper docs style guidelines.
[x] All technical procedures have been tested (if you want help with this, mention it in Reviewers).

Reviewers
Writing and Flow: @ipopescu / Tech: @deuszex

@ipopescu I'm planning to open another ticket to do the Testing Framework section. The recent updates made the existing tutorial completely incorrect, unfortunately.


V3 - Add Default SMS Sender Logic

We want to ensure V3 has feature parity with V2. Clients can send an SMS request with or without an sms_sender_id. If they send without, then we must look up the default sender assigned to the service and send the SMS from that sender.
User Story - Business Need

[x] Ticket is understood, and QA has been contacted (if the ticket has a QA label).

User Story(ies)
As a VA Notify Client
I want to send SMS notification to Veterans
So that I can deliver timely, actionable information
Additional Info and Resources

There was already work completed to deliver sms notifications with a specified sms_sender_id - #1361

Engineering Checklist
Changes should be made here.  The existing v2 logic is here.

[x] Reach out to @kalbfled for a little intro sync since he's the SME on this project
[x] Add logic to check for a default sms sender on the service if sms_sender_id is not provided (basically a copy-and-paste)
[x] Persist the sms_sender_id associated with the default sender in the database
[x] Write new unit tests here

Acceptance Criteria

Givens

A default SMS sender is assigned to the service
There is another SMS sender assigned to the service which is not the default

Cases

[x] Send an sms without specifying a sender

Send an sms request through postman to /v3/notifications/sms and do not include sms_sender_id in the request body
You should receive the sms from the default number assigned to the service

[x] Send an sms with a sender that is the default

Send an sms request through postman to /v3/notifications/sms and include a valid sms_sender_id which is the default in the request body
You should receive the sms from the sender associated with the id provided

[x] Send an sms with a sender that is not the default

Send an sms request through postman to /v3/notifications/sms and include a valid sms_sender_id which is not the default in the request body
You should receive the sms from the sender associated with the id provided

Givens
There is not a default sms sender assigned to the service
Cases

[x] Missing default sender

Send an sms request through postman to /v3/notifications/sms and do not include sms_sender_id in the request body
Then upon using GET notification you should see a validation error stating no default sender was found

QA Considerations
For QA to populate. Leave blank if QA is not applicable on this ticket.

[ ] Validate above cases in Perf once deployed there
[x] Automate or create ticket to automate

Out of Scope
There is a separate ticket to restore multiple provider logic #1505

Question here @mjones-oddball @kalbfled @nikolai-efimov
I was playing around with this on DEV since Nikolai had deployed it there and checked all the boxes for the AC and discovered....
In the GET v2/notifications/:notification_id route for a an sms sent through the v2 POST route, I see the following in the response: "sms_sender_id": "17060187-ad48-4391-a465-aacd048b4ae0",
However, in the GET v2/notifications/:notification_id route for a an sms sent through the v3 POST route, that property is set to null; "sms_sender_id": null,
Is this desired? The ticket actually doesn't mention what to do with that field. I would think we still want to return that value? It matters for automation, since I would want to verify that when sent using X, then X actually sent the message.

@cris-oddball , let's discuss that in the stand-up meeting.

Code and tests are updated and ready to be tested. https://github.com/department-of-veterans-affairs/notification-api/pull/1532

Code approved, merged and deployed to Perf, testing in progress.
Created new ticket to automate this feature.

PASSES QA Testing in Perf

TC01 PASSED Send an sms without specifying a sender (should use default sms sender id beginning with 17adae4d), expect 202 on POST v3 sms and 200 on GET v2 notifications/:notification_id that includes the service's default sms_sender and a status of delivered.

payload
{
    "template_id": "{{sms-template-id}}",
    "phone_number": "<redacted>"
}

202 response:
{
    "id": "20dce0b5-0b60-4e38-a80d-add4970ed71e"
}

GET response properties that validate this works as intended:
    "sms_sender_id": "fccd9e82-<snip>",
    "status": "    "status": "delivered",

TC02 PASSED Send an sms by specifying the default sms sender (beginning with 17adae4d), expect 202 on POST v3 sms and 200 on GET v2 notifications/:notification_id that includes the sms_sender and a status of delivered.

payload
{
    "template_id": "{{sms-template-id}}",
    "phone_number": "<redacted>",
    "sms_sender_id": "{{sms-sender-id}}"
}

202 response
{
    "id": "fbb331bc-501d-4b18-89d8-a588fc112b19"
}

GET response properties that validate this works as intended:
    "sms_sender_id": "fccd9e82-<snip>",
    "status": "delivered",

TC03 Send an sms by specifying an alternate sms sender (beginning with b6288085), expect 202 on POST v3 sms and 200 on GET v2 notifications/:notification_id that includes the sms_sender and a status of delivered.

payload
{
    "template_id": "{{sms-template-id}}9",
    "phone_number": "<redacted>",
    "sms_sender_id": "b6288085-<snip>"
}

202 response
{
    "id": "217a5be1-3d8d-45e6-975a-468419985915"
}

GET response properties that validate this works as intended:
    "sms_sender_id": "b6288085-<snip>",
    "status": "delivered",

TC04 Send an sms from a service that does not have a default sms sender assigned, expect 202 on POST v3 sms and 200 on GET v2 notifications/:notification_id that includes a status of technical-failure and the reason for that failure.
Created service in Perf via Portal (so, no default sender is set) and added an sms template
Service is
payload

202 response
GET response properties that validate this works as intended:

Regression Results in Perf PASSED
172 passed, 4 xfailed in 257.79s (0:04:17)


Moved test files to src/tests.

Abandon *.spec.js file naming, since eslint currently does not provide a way to declare separate lint rules for src files contra test files, when test files reside along src files.

Coverage remained the same at 100.0% when pulling 049b6bb8d2dda1e9035f7638d976f3303c966685 on moveTestFilesToSeparateDir into<PHONE_NUMBER>b801d33333b7f60b22f1ecdd19f172 on master.


Bump inmanta-dev-dependencies[async,extension,pytest] from 2.53.0 to 2.54.0 (#364)

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

Merged in b3d0ef5c0de0afe9c0c5927373b058c894716b22


test_lazy_zarr_array is failing with latest zarr-python v3

From https://github.com/cubed-dev/cubed/actions/runs/11855798688/job/33040762855?pr=617:
    def test_lazy_zarr_array(tmp_path):
        zarr_path = tmp_path / "lazy.zarr"
        arr = lazy_zarr_array(zarr_path, shape=(3, 3), dtype=int, chunks=(2, 2))

        assert not zarr_path.exists()
>       with pytest.raises((FileNotFoundError, TypeError, ValueError)):
E       Failed: DID NOT RAISE (<class 'FileNotFoundError'>, <class 'TypeError'>, <class 'ValueError'>)

Running git bisect shows that this change in Zarr is the cause: https://github.com/zarr-developers/zarr-python/pull/2442

Opened https://github.com/zarr-developers/zarr-python/issues/2491


adding hero article for azure devops

this should go under Get started here https://azure.microsoft.com/en-us/documentation/articles/azure-portal-keyboard-shortcuts/

Hi @mlearned, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

@mlearned : Thanks for your contribution to the Azure documentation! The author, @mlearned, has been notified to review your proposed change.

@mlearned
Thanks for your contribution! It looks like you're a Microsoft employee, and a writer. You should be contributing to our private repository, Azure/azure-content-pr. Our guidance wiki has the instructions for getting permissions, etc. The advantages for you are much more validation and error checking before publishing, and content staging so that you can see your changes just by opening a pull request.
Guidance: https://github.com/Azure/azure-content/blob/master/contributor-guide/tools-and-setup.md#permissions-in-github
also @Flanakin, Please review and approve this article as well. Michael is asking that this be added to the left-nav for the Azure Portal documentation (open "Azure portal keyboard shortcuts" and that's the left-nav I mean) under Getting Started. If you approve of the article and the navigation request, I (Curtis Love) can help you with the mechanical part of that if you need me to.

Yea sorry about that, I'm submitting to private repo. I'm making some updates and will do that this weekend.


QnA Maker sharing procedure do not work

I followed the procedure but the person I shared with do not see the shared QnA Maker service.
Please note that this has also been reported as a comment in the ticket #8219 but I think it's better to have a separated one as this issue is more important.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 937c1738-727e-794f-36c4-86694e41a69d
Version Independent ID: eda35f4a-a5f4-cf47-d5d4-12c93722ddd9
Content: Collaborating on your knowledge base - Microsoft Cognitive Services - Azure
Content Source: articles/cognitive-services/QnAMaker/How-To/collaborate-knowledge-base.md
Service: cognitive-services
GitHub Login: @nstulasi
Microsoft Alias: saneppal

@apeyrat Thanks for the feedback! We are currently investigating and will update you shortly.

@apeyrat Hi, can you please help me make sure 1) Did the person you sent the invitation receive the email and click 'Get start' in it? 2) Is the person you invited are in the same subscription with you?

@apeyrat We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

I can also not get this to work. In particular, I'm trying to share a KB with a guest user. I have added this user in Azure, but they still do not see any available services when logging into the QnA Maker portal.


Here is how you get added to cloudflare

https://publicsuffix.org/

https://github.com/publicsuffix/list/pull/1672 We've already had one since December of last year.


Add completions for cargo remove

What does this PR try to resolve?
This PR continues the deferred work of #11099 by adding bash and zsh completions for the cargo remove subcommand.
How should we test and review this PR?
There doesn't seem to be much in the way of testing for these completions automatically, so I would suggest verifying that they work in practice and sufficiently cover the subcommand's surface area.
Additional Information
I will also soon post a PR for cargo remove's documentation.

r? @ehuss
(rust-highfive has picked a reviewer for you, use r? to override)

Added missing rm alias as of dfbf3e6.

For whom wants to test this completion
zsh:
# under cargo project directory
fpath+=$PWD/src/etc
compinit

# test it
cargo rem<tab>

bash
# install any bash completion from your distro and source their init script
# e.g. brew install bash-completion && . "/path/to/pbash_completion.sh"

# under cargo project directory
. ./src/etc/cargo.bashcomp.sh

# test it
cargo rem<tab>

@bors r+

:pushpin: Commit 3b6d2c4c8c78f553ef33c2e03e1a751272413315 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 3b6d2c4c8c78f553ef33c2e03e1a751272413315 with merge e691e18ef8c68b40a8d26302ec4b67bb869a2f01...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing e691e18ef8c68b40a8d26302ec4b67bb869a2f01 to master...


Many to many

Hi,
I am submitting this pull request with an example for Ecto's many_to_many function docs on self-referencing many_to_many associations.
Thank you ♥
-Mark

This looks great, thank you! I have added just one comment!

This looks great, thank you! I have added just one comment!

Okay, I separated it into another guide and gave it a #Self-referencing many to many title with ##People relationships subheading. 😊

Okay, I separated it into another guide and gave it a #Self-referencing many to many title with ##People relationships subheading. 😊

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Add back tranmutex

Need a RWMutex to protect nil client usage during reconnect. Also includes a workaround for https://bugzilla.redhat.com/show_bug.cgi?id=2000375

@girishmg @hzhou8 @tssurya @dcbw

Hi @trozet, the problem looks clear but I wonder if this is the right way to fix. There can be different reasons for a transaction to fail - it can be temporary. Are we going to blindly reconnect regardless of the reason?


dtype padding in surface_area

TauFactor version: 0.1.3

Description
In surface_area, padding with -1 can be a problem if the dtype of the img is np.uint8 and one of the phases is 255 (-1)

Can you explain this more? What is the issue? Is the surface_area calculated wrong? Does it not run? I can't recreate this issue

It is calculated wrong.
The padding should be with a number that is different from all phases.
When padding with -1, it is the same as padding with 255, but 255 is already one of the phases..


Fixed a bug with svg plugins

If a plugin has a top-level svg element, it's className property is
not a string and that was causing a js error which in turn would break
structureboard actions, like editing plugins etc.
Thanks to @jmelett for report.

Coverage remained the same at 82.483% when pulling ee92bf955a6a60df9ea1277893082e92bd11c975 on vxsx:bugfix/svg-plugins into ef4247f38a4d8b0b22d0e8b01b9b6e54190eadb4 on divio:release/3.4.x.


KAFKA-18026: KIP-1112, ProcessorWrapper API with PAPI and partial DSL implementation

This PR includes the API for KIP-1112 and a partial implementation, which wraps any processors added through the PAPI and the DSL processors that are written to the topology through the ProcessorParameters#addProcessorTo method.
Further PRs will complete the implementation by converting the remaining DSL operators to using the #addProcessorTo method, and future-proof the processor writing mechanism to prevent new DSL operators from being implemented incorrectly/without the wrapper

@ableegoldman May I ask why you merged this PR without any approval from a committer excluding yourself?
At least, I was not able to find the approval.


Fix typo in README.md of Strong Baselines

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Fix typo in README.md of Strong Baselines.
Modification
Fixed large-scale "jittor" to large-scale jitter.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMCls.
The documentation has been modified accordingly, like docstring or example tutorials.

 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.


Trig functions math/numpy

PR Summary
Trig functions and update Contributor's Guide

sin
cosine
tangent
arcsine
arccosine
arctangent

PR Checklist
General

[x] PR includes a summary of changes
[x] Link relevant issues, make one if none exist

Looks great. Thanks for the updates to the Contributing guide as well. It seems to be missing from the homepage though?

Just a few initial thoughts, I'll give this a more thorough look tomorrow 👍

Sorry, the review notebook app didn't let me leave a top-level review comment.
I'm thinking we should either refocus this notebook to be a receipt, or at least a trig focused ncl entry if there's anything that a receipt notebook produces that's significant enough to warrant adding an ncl notebook.

Ok, based on suggestions, I expanded this to cover the trig functions (sin, cos, tan, asin, acos, atan) since they are all very similar in execution. The notebook is now broken into:

Python package options
"Grab and Go" sections for each function (with hyperlinks)
Overview of differences from NCL
Additional Python resources

I removed the changes made to the contributor's guide and put them into the relevant PR #52 to clean up this PR

Since the trig functions are using the same notebook, I don't think it will be able to populated by NCL Applications side bar since that is populated by separate notebooks


Ensure bag version is provided in callback

Also in the /ingests endpoint, where we don't present it either.

The notifier gets an instance of Ingest, but that doesn't have a notion of version (yet) – that’s passed among the payloads, but not to the ingests app.
We could change that, and track version in the ingests DB.


Fix UTType.bim_iModel to be let instead of var

@toddsouthenbentley  Note: future versions of Swift won't be happy with this being a var due to it introducing an apparent concurrency race condition.


I don't have an answer.   Probably, it should be discussed in whisper.


Refactoring Port Manager

After optimization, the performance data for creating a single port is as follows:
Samples: 4877
Average: 10
Media: 10
90% Line: 12 (90% of the samples took no more than 12)
95% Line: 13 (93% of the samples took no more than 13)
99% Line: 17 (99% of the samples took no more than 13)
Min: 7
Maximum: 221
Error %: 0
Throughput: 81.3/sec
Received KB/sec: 95.60
Sent KB/sec: 42.31
Port configuration:
{
"port": {
"network_id": "5687abe4-9ae3-446a-979d-04d7e486646c",
"tenant_id": "3d53801c-32ce-4e97-9572-bb966f476ec",
"allowed_address_pairs": [{
"ip_address": "<IP_ADDRESS>",
"mac_address": "00:01:6C:06:A6:29"
}]
}
}

Codecov Report

Merging #301 into master will decrease coverage by 3.81%.
The diff coverage is 77.04%.

@@             Coverage Diff              @@
##             master     #301      +/-   ##
============================================
- Coverage     38.58%   34.77%   -3.82%
- Complexity      725      825     +100
============================================
  Files           309      347      +38
  Lines          7257     8392    +1135
  Branches        822      945     +123
============================================
+ Hits           2800     2918     +118
- Misses         4094     5130    +1036
+ Partials        363      344      -19

Impacted Files
Coverage Δ
Complexity Δ

...turewei/alcor/portmanager/repo/PortRepository.java
86.08% <0.00%> (-3.92%)
28.00 <0.00> (ø)

...portmanager/service/implement/PortServiceImpl.java
0.00% <0.00%> (-78.98%)
0.00 <0.00> (-60.00)

.../privateipmanager/controller/IpAddrController.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...wei/alcor/privateipmanager/entity/IpAddrRange.java
0.00% <0.00%> (-44.57%)
0.00 <0.00> (-15.00)

...i/alcor/privateipmanager/repo/IpAddrRangeRepo.java
0.00% <0.00%> (-48.72%)
0.00 <0.00> (-18.00)

...ipmanager/service/implement/IpAddrServiceImpl.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...urewei/alcor/route/controller/RouteController.java
52.87% <ø> (ø)
7.00 <0.00> (ø)

...r/route/service/Impl/RouteDatabaseServiceImpl.java
15.38% <ø> (ø)
1.00 <0.00> (ø)

...service/Impl/RouteWithSubnetMapperServiceImpl.java
3.27% <ø> (ø)
1.00 <0.00> (ø)

...te/service/Impl/RouteWithVpcMapperServiceImpl.java
3.27% <ø> (ø)
1.00 <0.00> (ø)

... and 131 more

Continue to review full report at Codecov.

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

@chenpiaoping Some code conflict due to PR #304 that got just merged.

After optimization, the performance data for creating a single port is as follows:
Samples: 4877
Average: 10
Media: 10
90% Line: 12 (90% of the samples took no more than 12)
95% Line: 13 (93% of the samples took no more than 13)
99% Line: 17 (99% of the samples took no more than 17)
Min: 7
Maximum: 221
Error %: 0
Throughput: 81.3/sec
Received KB/sec: 95.60
Sent KB/sec: 42.31
Port configuration:
{
"port": {
"network_id": "5687abe4-9ae3-446a-979d-04d7e486646c",
"tenant_id": "3d53801c-32ce-4e97-9572-bb966f476ec",
"allowed_address_pairs": [{
"ip_address": "<IP_ADDRESS>",
"mac_address": "00:01:6C:06:A6:29"
}]
}
}

@chenpiaoping The result is outstanding. I will expedite the PR review and let us try to get the PR part of the release.
Couple of questions:

When we generate the test payload, do we automatically generate different configurations (uuid, ip address, mac address etc.) for different ports? This would impact performance as if we use the same configuration, we always hit the same record.
The size of samples is good to show some initial result. Can we increase to 10,000, 50,000, 100,000, and 500,000 to see if the latency and throughput curve of a single-node controller? Basically we want to find the breaking point.
Ignite: do we use one instance of DB, or multiple instances? Maybe that could be part of the metrics too.

Cheers!
@xieus

When we generate the test payload, do we automatically generate different configurations (uuid, ip address, mac address etc.) for different ports? This would impact performance as if we use the same configuration, we always hit the same record.
> PM assigns random ip and mac address to ports that do not have ip and mac address
The size of samples is good to show some initial result. Can we increase to 10,000, 50,000, 100,000, and 500,000 to see if the latency and throughput curve of a single-node controller? Basically we want to find the breaking point.
>From the test result , we find that the QPS is about 81.3/sec. that is to say, it is about 5000 / min. If the rate of the sender exceeds this value, the average processing time of each request will increase.
Ignite: do we use one instance of DB, or multiple instances? Maybe that could be part of the metrics too.
> I deployed an ignite instance on my laptop.  the only meaning of this test result is that it is much better than the previous test results in the same environment (my laptop).

@xieus
The problem of ignite deadlock, which I mentioned at the last regular meeting, appeared in the K8s environment:
"http-nio-8080-exec-6" #22 daemon prio=5 os_prio=0 cpu=70.00ms elapsed=635.62s tid=0x00007fddfe53d800 nid=0x1b runnable  [0x00007fddb87a6000]
java.lang.Thread.State: RUNNABLE
at<EMAIL_ADDRESS>Method)
at java.net.SocketInputStream.socketRead(java.base@11.0.5/SocketInputStream.java:115)
at java.net.SocketInputStream.read(java.base@11.0.5/SocketInputStream.java:168)
at java.net.SocketInputStream.read(java.base@11.0.5/SocketInputStream.java:140)
at org.apache.ignite.internal.client.thin.TcpClientChannel$ByteCountingDataInput.read(TcpClientChannel.java:535)
at org.apache.ignite.internal.client.thin.TcpClientChannel$ByteCountingDataInput.readInt(TcpClientChannel.java:572)
at org.apache.ignite.internal.client.thin.TcpClientChannel.processNextResponse(TcpClientChannel.java:272)
at org.apache.ignite.internal.client.thin.TcpClientChannel.receive(TcpClientChannel.java:234)
at org.apache.ignite.internal.client.thin.TcpClientChannel.service(TcpClientChannel.java:171)
at org.apache.ignite.internal.client.thin.ReliableChannel.service(ReliableChannel.java:180)
at org.apache.ignite.internal.client.thin.GenericQueryPager.next(GenericQueryPager.java:71)
at org.apache.ignite.internal.client.thin.ClientQueryCursor$1.nextPage(ClientQueryCursor.java:93)
at org.apache.ignite.internal.client.thin.ClientQueryCursor$1.hasNext(ClientQueryCursor.java:76)
at org.apache.ignite.internal.client.thin.ClientQueryCursor.getAll(ClientQueryCursor.java:47)
at com.futurewei.alcor.common.db.ignite.IgniteClientDbCache.getAll(IgniteClientDbCache.java:182)
at com.futurewei.alcor.common.db.ignite.IgniteClientDbCache.getAll(IgniteClientDbCache.java:175)

@xieus This PR is ready to be reviewed.

The time cost of calling CreatePortBulk to create 50 port(Body contains 50 port) is as follows:

@chenpiaoping Hey Piaoping, the contract between Port Mgr and Node Mgr has been recently changed in the PR https://github.com/futurewei-cloud/alcor/pull/326. Basically instead of using node id, we call the List Node API and query by name (of node).
Please make the change accordingly.

@chenpiaoping Hey Piaoping, the contract between Port Mgr and Node Mgr has been recently changed in the PR #326. Basically instead of using node id, we call the List Node API and query by name (of node).
Please make the change accordingly.

ok

@xieus  I tested it in my local environment, and the problem without neighbor information did not reappear, Here are the results of my test:
NetworkConfiguration(rsType=null, opType=null, portEntities=[InternalPortEntity(routes=[RouteEntity(destination=<IP_ADDRESS>/24, target=Local, priority=0, associatedType=MAIN, associatedTableId=937588ce-9c85-4066-b914-44ac03ce1f13)], neighborInfos=null, bindingHostIP=<IP_ADDRESS>, internalNeighborInfo1=null, subnetEntities=null, vpcEntities=null)], vpcs=[VpcEntity(cidr=, routeEntities=[RouteEntity(destination=null, target=Local, priority=0, associatedType=MAIN, associatedTableId=bab7f7f1-842c-4e31-abdd-09f1821d2cbe)], adminStateUp=true, dnsDomain=, mtu=1450, portSecurityEnabled=true, networkType=vxlan, physicalNetwork=, segmentationId=14860288, routerExternal=false, segments=[], shared=false, vlanTransparent=false, isDefault=false, availabilityZoneHints=[], availabilityZones=[Nova], qosPolicyId=[], revisionNumber=1, status=ACTIVE, tags=[], created_at=2020-08-05 06:24:52, updated_at=2020-08-05 06:24:52, ipv4AddressScope=, ipv6AddressScope=, l2Adjacency=, subnets=[5186ff87-6531-487a-94e8-ef6f4b9e528b])], subnets=[InternalSubnetEntity(tunnelId=null)], securityGroups=[CustomerResource{projectId='ede9a56fd64243bdbca77182e330e3a1', id='0836c5f5-3e6d-48a9-82db-3de5521248d7', name='default', description=''}])
NetworkConfiguration(rsType=null, opType=null, portEntities=[InternalPortEntity(routes=[RouteEntity(destination=<IP_ADDRESS>/24, target=Local, priority=0, associatedType=MAIN, associatedTableId=937588ce-9c85-4066-b914-44ac03ce1f13)], neighborInfos=[NeighborInfo(hostIp=<IP_ADDRESS>, hostId=cn-jswl-jsy-yuanwei-01, portId=69234789-c16a-4a6f-857c-e588548d4391, portMac=AA-BB-CC-EF-5F-3E, portIp=<IP_ADDRESS>)], bindingHostIP=<IP_ADDRESS>, internalNeighborInfo1=null, subnetEntities=null, vpcEntities=null)], vpcs=[VpcEntity(cidr=, routeEntities=[RouteEntity(destination=null, target=Local, priority=0, associatedType=MAIN, associatedTableId=bab7f7f1-842c-4e31-abdd-09f1821d2cbe)], adminStateUp=true, dnsDomain=, mtu=1450, portSecurityEnabled=true, networkType=vxlan, physicalNetwork=, segmentationId=14860288, routerExternal=false, segments=[], shared=false, vlanTransparent=false, isDefault=false, availabilityZoneHints=[], availabilityZones=[Nova], qosPolicyId=[], revisionNumber=1, status=ACTIVE, tags=[], created_at=2020-08-05 06:24:52, updated_at=2020-08-05 06:24:52, ipv4AddressScope=, ipv6AddressScope=, l2Adjacency=, subnets=[5186ff87-6531-487a-94e8-ef6f4b9e528b])], subnets=[InternalSubnetEntity(tunnelId=null)], securityGroups=[CustomerResource{projectId='ede9a56fd64243bdbca77182e330e3a1', id='0836c5f5-3e6d-48a9-82db-3de5521248d7', name='default', description=''}])


Fix override ally drivers issue

As you may know, the LinkedIn driver is not working for API v2 (#69). I'm trying to implement a new driver for Linkedin which is compatible with API V2. Here's the code I use to extend Ally in hooks.js:
ioc.extend('Adonis/Addons/Ally', 'linkedin', () => {
   return new linkedin(Config);
});

The main problem is that you can extend Ally using a new key but can't override existing drivers. To solve this issue I gave priority to extended drivers in AllyManager.

Coverage remained the same at 67.775% when pulling 99afbbb68e0096747a88b410b731db677d366a4e on amirzaeiyan:develop into 3c8697f7e0281f5ebe2903426a20c8bb8d24cace on adonisjs:develop.

This may some other effects too (not 100% sure). Why not you name yours linkedv2?

Thanks for your reply. Yes, you are right but the current version of the LinkedIn driver is deprecated and can't be used anymore and for some reason, I need to use the same key for LinkedIn.


[Component] Create Button component

Description
As a user of the Ouds library
I want to be able to use the component
So that I can add an Ouds Button component in my own projects
As a user of the library
I want to be able to use the Button component
So that I have access to the documentation on how to use it
As a user of the Demo App
I want to access the Button Component
So that I can understand how to use it:

while using the demo app I can interact with the available options and observe the changes on the component itself
I can see the code associated to the component customization
I can copy and paste the code from the app based on the selected customization
or when looking into the code of the app itself, I would also understand how to use it

Context
The component is described in the Design tools and allows to describe the various states and variants.
Variants
TBD
Demo App
TBD

"On hold" as things are not defined in the description, and prototypes or tests shuld be done on how we can customize focus states (double borders / inline / outline / colors-


parser: fix crash in ParsedExpr::equals() for ParsedExpr::JaktArray and add a test

A typo / missing parenthesis when comparing the size of both arrays caused an out-of-bounds array access during element-wise comparison.

Hello!
One or more of the commit messages in this PR do not match the Jakt code submission policy, please check the lint_commits CI job for more details on which commits were flagged and why.
Please do not close this PR and open another, instead modify your commit message(s) with git commit --amend and force push those changes to update this PR.


Could you please make sure that older versions of conduit still work after an update of the framework

Because right now my blockchain doesn't launch anymore if the framework is updated and i have to copy everything to the new updated blank starter template.

I think the best option I can provide is a migration tool. Some of the time conduit updates correspond with dart updates.


[Feature]: Option to embed bags into bag frame

Describe the feature
I think it would be nice to have a toggle either in options or in the bag menu to have your bags embedded into the frame.
I think it would be useful for quickly changing bags without having to see all of you bags unsorted and having to find your bags one by one if you don't mind having your bag slightly larger if at all depending on implementation.
Describe the implementation
The option could be separately in bags and bank categories, or having it in the menu, but I think the settings makes more sense as for one you could have it separate for bank and bags as well as the menu already has many options and adding such a minor toggle would just bloat the list.
When turned on, it depends on implementation, but in my head it would look like the search bar would be moved to the left and to the right next to the search bar, there would be your bags in a row. I made a quick visualization, not of good quality as my skills are lacking, but it should get you my idea, the bags would need to be more to the right and probably either scaled down or background extended to the top, or mix of both.

Alternatively, if it is not something you would want to do, then a simple alternative would be a setting that would just make the Show Bags option only show the bag frame with no changes to the actual bag.

Adding the bags directly to the frame like that or some other form is very unlikely to happen. It is a design goal to keep the number of controls on the bag window to a minimum.
A request is pending to have the full 'bag bar' show instead of the single bag icon - maybe this would suffice?

(default bar)

The bags could also be added at the end after the Free Space, all in all it is only optional, not a default behavior, there are multiple ways of approaching this idea. But well it is only an idea, if the design is against it then its fine.


[WIP] Add tests for integer-gmp-1 changes

refs #3

@cartazio I'll ping you when this is done.

Great! Many thanks.  I'll be more than happy to code review it once you
indicate it's ready.
Arithmoi definitely needs some testing and benchmarking of public Apis if
we're to evolve it in a meaningful way going forward!
On Tuesday, June 9, 2015, Shea Levy<EMAIL_ADDRESS>wrote:

@cartazio https://github.com/cartazio I'll ping you when this is done.
—
Reply to this email directly or view it on GitHub
https://github.com/cartazio/arithmoi/pull/8#issuecomment-110347759.

@cartazio ready.

thanks
ill cut a release now

(well, after travis shows all green)


Can not use sc-uws on Ubuntu 18.04, node 10.12.0

I tried to use sc-uws engine on both, socketcluster boiler app and in sc-crud-sample. In both cases it failed with folowing ( i get it from build_log.txt in /node_modules/sc-uws/built-log.txt):
ws.target.mk:106: recipe for target 'Release/obj.target/uws/src/addon.o' failed
make: *** [Release/obj.target/uws/src/addon.o] Error 1
make: Leaving directory '/home/dev/Code/sc-crud-sample/node_modules/sc-uws/build'
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/home/dev/.nvm/versions/node/v10.12.0/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:262:23)
gyp ERR! stack     at ChildProcess.emit (events.js:182:13)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:240:12)
gyp ERR! System Linux 4.15.0-38-generic
Are there any dependencies or something ?

I know this doesn't directly answer your question but I had to do build and run on Docker to get it to work. Here is my Dockerfile
FROM node:8.12.0-onbuild

RUN apt update -y
RUN apt install build-essential -y

WORKDIR /app
COPY ./package.json ./
RUN npm install
COPY . .

RUN npm remove sc-uws --save
RUN npm install sc-uws --save

CMD ["npm","run","start"]

Very weird that I had to remove it from the package.json and then add it again. That was the only way I could get it to work....

I'll give it a try, thank you for your answer :)

According to this comment https://github.com/SocketCluster/socketcluster/issues/497#issuecomment-503404420 sc-uws only compiles with nodejs 8.


Add BuildArgs for Docker build.

Version bump for the AdoptOpenJDK image to bring it to the latest released version and...
The processing of the --buildArgs option in docker.gradle seems to have been broken for a long time.  I've fixed that problem to give the option of using an ARG to specify a docker repository for AdoptOpenJDK downloads.  Why?  I've been putting together a repository of multi-architecture images that may be ahead of the core repository (including an Alpine image if anyone's interested).
This change should be transparent to most users because the ARG in question defaults to the docker.io library.

Whoops.  Looks like Travis is still using an old Docker.  Will look into what it'll take to fix that.


Only orgs with certain level receives harvest

Signed-off-by: Ruben Vargas<EMAIL_ADDRESS>Fixes #251

For deployment:
Contracts that need to be deployed:

[ ] Harvest

Other considerations:

[ ] Settings need to be updated


Grunt: Factor out new docs-github task

This factors out a new docs-github task for building the hosted docs, since they're occasionally patched outside of the long release cycle and since simply using Jekyll directly isn't sufficient anymore due to the HTML minification (which makes the diffs a pain to read, but that's a story for another time).
CC: @XhmikosR

OK, I guess.


Show dimensions of data variables in structure report

I think this would be useful.
N.B. we have chosen so far to show only dimension names, not numbers.
But I think it would still make sense to list dimensions of data variables, as we show the location dimensions in meshes.
For example, referencing this file (though it does also have some problems) ...
$ python -m ugrid_checks -seq unstructured_grid/lfric_ngvat_2D_1t_face_half_levels_main_conv_rain.nc

File mesh structure
-------------------
Meshes
    "Mesh2d_half_levels"
        node("nMesh2d_half_levels_node")
            coordinates : "Mesh2d_half_levels_node_x", "Mesh2d_half_levels_node_y"
        edge("nMesh2d_half_levels_edge")
            edge_node_connectivity : "Mesh2d_half_levels_edge_nodes"
            coordinates : "Mesh2d_half_levels_edge_x", "Mesh2d_half_levels_edge_y"
        face("nMesh2d_half_levels_face")
            face_node_connectivity : "Mesh2d_half_levels_face_nodes"
            coordinates : "Mesh2d_half_levels_face_x", "Mesh2d_half_levels_face_y"

?? Connectivities with no mesh ??
    "Mesh2d_half_levels_face_edges" ("nMesh2d_half_levels_face", "nMesh2d_half_levels_vertex")
        cf_role = "face_edge_connectivity"

Mesh Data Variables
    "conv_rain"
        mesh : "Mesh2d_half_levels"
        location : "face"

That could easily (and probably usefully) say  :
Mesh Data Variables
    "conv_rain" ("time_counter", "nMesh2d_half_levels_face")
        mesh : "Mesh2d_half_levels"
        location : "face"

NOTE: It's not probably not worth addressing this until #50 is sorted out, as this will presumably (eventually) include re-writing the structure report in terms of the structure-analysis object.


docs(firebase_messaging): Updated Firebase_messaging documentation

Description
Changed the registerWith function in Firebase_messaging documentation, suggested as this.
Related Issues
(#4396)
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

:wave what version of FlutterFire messaging did you test this on? The docs website is for the v8 dev release which has a completely different Android package name/structure so this change won't work;

:wave what version of FlutterFire messaging did you test this on? The docs website is for the v8 dev release which has a completely different Android package name/structure so this change won't work;


Make externalsPresets.node obey externalsType option

What problem does this feature solve?
Right now node_target_plugin has node-commonjs value hardcoded as externals type (same as Webpack), but for better ESM compatibility it would be nice if it inherited value of externalsType configuration option. This way all nodejs builtins can be easily excluded from bundling and will be imported at runtime natively via ESMs import syntax
What does the proposed API of configuration look like?
externalsPresets: { node: true },
externalsType: 'module'

I wonder if this is a change @TheLarkInn and @alexander-akait will also be making as part of the ESM support in webpack being tracked in webpack/17121

Do we really need to wait until it lands in Webpack before something like this can be implemented?

Do we really need to wait until it lands in Webpack before something like this can be implemented?

if it's confirmed as a bug we're willing to fix it both in rspack and webpack, but we may need to discuss with webpack team about the api design context and make sure it's a bug before fixing it.

@blinkzz technically you are right, but not all externalsType values supported for Node.js, I think we should use logic:
new ExternalsPlugin(isModule ? "module" : "node-commonjs", builtins).apply(compiler);

Just intresting - why you want to use another value? I would just like to understand that there are no bugs or something else

@alexander-akait, just for consistency, since externalsType already controls user-defined packages in externals, so why not use it for presets too, it’s very unlikely that somebody will make a mistake here by choosing wrong type for their environment

@blinkzz

it’s very unlikely that somebody will make a mistake here by choosing wrong type for their environment

You can take my word for it, they did it :smile: so I think it's better to limit "module" and "node-commonjs"`, both can be used.
Here is another problem, there is a situation - some Node.js modules doesn't work with ESM. Example:
import zlib from 'node:zlib';

console.log(zlib)

and
const zlib = require('node:zlib');

console.log(zlib)

You can found more them, and to avoid this ploblem we are using node-commonjs, because it always works. Yes, we can check a type of dependency and generate output based on it (i.e. use import when import was used), but not sure it is really worth prioritizing...

So changing it right now can break build for some developers... it is a little bit tricky to implement this right, we need:

wait until Node.js will have support for all built-in modules in ESM (if it will be not, we need to have two lists)
maybe we will need browserslist because some modules were unavaliable in ESM and now they are avaliable
analize original dependency (how developer imported it and it should be import ... from ...;)

@blinkzz

it’s very unlikely that somebody will make a mistake here by choosing wrong type for their environment

You can take my word for it, they did it 😄 so I think it's better to limit "module" and "node-commonjs"`, both can be used.
Here is another problem, there is a situation - some Node.js modules don't work with ESM. Example:
import zlib from 'node:zlib';

console.log(zlib)

and
const zlib = require('node:zlib');

console.log(zlib)

that is a good catch，nodejs changes esm behaviour a lot in different version，which makes esm harder to work on all supported nodejs version

@hardfist Yeah, therefore, purely theoretically, this is a solvable problem, but it will take time and you will have to maintain a map of modules and versions

close as it's same as webpack


cc_auto script

Anyone get the cc_auto script to work? I've had nothing but troubles with it this tale.

I have fixed this. When I get home from work I will submit the fixes for cc_auto and cc_assist.

Tried CC/auto out just now. I simply does nothing :=)

That is because I have not submitted the fix yet. Be patient hehe.

Fixed in


hugo not auto-recompiling scss on ubuntu

Somehow, I need to stop the server and start it again, so my scss would be recompiled on ubuntu. My friend working on the same project does not have the issue on his Windows machine.
Did anyone encounter such problem?
$ hugo version
Hugo Static Site Generator v0.56.3/extended linux/amd64 BuildDate: unknown

Did anyone encounter such problem?

You may find a broader audience to ask at https://discourse.gohugo.io/
I would guess an editor issue, but I thought we had ironed out those issues. What text editor do you use? Does it make any difference if you edit it with nano? I don't use Ubuntu on a regular basis, so I cannot test this easily. Maybe others will chime in.

Thank you for your suggestions. I just tried Vim and gedit but unfortunately it does not make a difference.

I don't use Hugo extend so I don't know anything about this. However, if I was you, I would separate the compiling SCSS task away Hugo. I usually let Hugo do the rendering layout task, meaning while the compiling theme task will be handled by Gulp auto built script. Because you are not always change SCSS so separating them would help to speed your site up.

With the new update (0.58.3) it's not an issue anymore and now seems to works fine.


Bug: Submodule with separate TRANSLOCO_SCOPE does not load translations from lazy loaded parent.

Current behavior
When a sub-module is imported into another module (lazy loaded?) with it's own TRANSLOCO_SCOPE, the sub-modules TRANSLOCO_SCOPE does not load the translation.
Expected behavior
Expectation is that each Module can have it's own scope and will load properly whether the sub-module is loaded directly or lazily.
Minimal reproduction of the problem with instructions
Navigate first to this link: https://ngneat-transloco-ho6spo.stackblitz.io/lazy
Note that h-alias.TITLE is not translated.
Navigate to the HOME link and note that h-alias.TITLE is now translated (module w/ separate scope is loaded in App Module).
Now navigate back to Lazy and notice that h-alias.TITLE is properly translated.
Stackblitz code: https://stackblitz.com/edit/ngneat-transloco-ho6spo?devtoolsheight=33&file=src/app/helper/myhelper/myhelper.component.html
Browser:

[x ] Chrome (desktop) version XX
[ ] Chrome (Android) version XX
[ ] Chrome (iOS) version XX
[ ] Firefox version XX
[ ] Safari (desktop) version XX
[ ] Safari (iOS) version XX
[ ] IE version XX
[ x] Edge version XX

Additional context
Add any other context about the problem here.
Contribution
I would like to make a pull request for this feature:
[ ] Yes! 🚀
[x ] Maybe next time

@NaviKnow please read this guide carefully, you might find your answer there


x/crypto/curve25519: pure Go implementation mismatch with BoringSSL

I believe there is a bug in the pure Go implementation of curve25519.  Comparison between the pure Go and assembly implementations demonstrated a mismatch. Further testing showed that the pure Go version fails test vectors generated with BoringSSL. Full details are in the mmcloughlin/bug25519 repository, but the salient points are below.
Problem
BoringSSL test vectors pass on the amd64 architecture (assembly implementation):
$ go version
go version go1.11 darwin/amd64
$ go test -v -run Current
=== RUN   TestTestVectorsCurrent
--- PASS: TestTestVectorsCurrent (0.00s)
    testvectors_test.go:47: failed 0 of 32
PASS
ok  	github.com/mmcloughlin/bug25519	0.006s

However the pure Go version does not (induced with the appengine tag)
$ go test -v -run Current -tags appengine | head
=== RUN   TestTestVectorsCurrent
--- FAIL: TestTestVectorsCurrent (0.01s)
    testvectors_test.go:39:     in = 668fb9f76ad971c81ac900071a1560bce2ca00cac7e67af99348913761434014
    testvectors_test.go:40:   base = db5f32b7f841e7a1a00968effded12735fc47a3eb13b579aacadeae80939a7dd
    testvectors_test.go:41:    got = 78202e24db99e237f2a14f9ec61b051814ec8fd23a5e8e68add48d66fd09fc12
    testvectors_test.go:42: expect = 090d85e599ea8e2beeb61304d37be10ec5c905f9927d32f42a9a0afb3e0b4074
    testvectors_test.go:39:     in = 203161c3159a876a2beaec29d2427fb0c7c30d382cd013d27cc3d393db0daf6f
    testvectors_test.go:40:   base = 6ab95d1abe68c09b005c3db9042cc91ac849f7e94a2a4a9b893678970b7b95bf
    testvectors_test.go:41:    got = 2ec45ca394a3febc6d63b8995ae63b38c7ba909bafed2a039dd54973f2b5be73
    testvectors_test.go:42: expect = 11edaedc95ff78f563a1c8f15591c071dea092b4d7ecaac8e0387b5a160c4e5d

Approximately half of the test vectors fail.
Fix
Comparison of the individual fe*() functions against the ref10 implementation suggests the problem is in feFromBytes(). By eye we see the Go implementation is missing a mask. The following patch appears to fix the problem.
diff --git a/curve25519/curve25519.go b/curve25519/curve25519.go
index cb8fbc5..75f24ba 100644
--- a/curve25519/curve25519.go
+++ b/curve25519/curve25519.go
@@ -86,7 +86,7 @@ func feFromBytes(dst *fieldElement, src *[32]byte) {
        h6 := load3(src[20:]) << 7
        h7 := load3(src[23:]) << 5
        h8 := load3(src[26:]) << 4
-       h9 := load3(src[29:]) << 2
+       h9 := (load3(src[29:]) & 0x7fffff) << 2

        var carry [10]int64
        carry[9] = (h9 + 1<<24) >> 25

See the corresponding line in the reference implementation. Note also that RFC 7748 specifies:
   The u-coordinates are elements of the underlying field GF(2^255 - 19)
   or GF(2^448 - 2^224 - 1) and are encoded as an array of bytes, u, in
   little-endian order such that u[0] + 256*u[1] + 256^2*u[2] + ... +
   256^(n-1)*u[n-1] is congruent to the value modulo p and u[n-1] is
   minimal.  When receiving such an array, implementations of X25519
   (but not X448) MUST mask the most significant bit in the final byte.
   This is done to preserve compatibility with point formats that
   reserve the sign bit for use in other protocols and to increase
   resistance to implementation fingerprinting.

@FiloSottile

Re-opening this as a reminder to update the version vendored into the standard library. @FiloSottile is it ok to wait for 1.13 for that or does it need to happen in 1.12?

Let's do it in 1.13, there seems to be no security implication.

This got imported in Go 1.13.
https://github.com/golang/go/blob/09c9bced825593aedfd79af5c35916392f43113c/src/vendor/golang.org/x/crypto/curve25519/curve25519.go#L89


WP Super Cache config doesn't pass query strings to REST API

Steps to reproduce:

Create site with --wpsc command.
Install WP REST API (version 2) plugin (currently 2.0-beta4, v1 is a separate plugin).
Go to URL for simple posts retrieving, e.g. http://example.com/wp-json/wp/v2/posts.
Now add some query strings to it, e.g. http://example.com/wp-json/wp/v2/posts?page=2 or http://example.com/wp-json/wp/v2/posts?filter[s]=test. Same results.
Compare that to normal REST API installation (e.g. --wpfc on EE will work).

Problem is on server level because even if you enter print_r( $_GET ); in index.php you get empty array.
I think that EE should become compatible with REST API because parts of this plugin are being merged in next WordPress version, while rest is being merged in the next WP version (see article), so it means it is here to stay.

I think if issue happens with wp-super-cache only, may be it is issue with wp-super-cache.
We are using REST API and excited about it's inclusion in WordPress core. :-)

I forgot to mention that even if you deactivate WP Super Cache same thing happens.

I did a post about that in the past at RTCamp Community Forum and they answer is something about the compatibility with WooCommerce
In wpsc.conf add/change:
Use cached or actual file if they exists, Otherwise pass request to WordPress
location / {
# try_files /wp-content/cache/supercache/$http_host/$cache_uri/index.html $uri $uri/ /index.php$args;
# try_files /wp-content/cache/supercache/$http_host/$cache_uri/index.html $uri $uri/ /index.php?$query_string;
try_files /wp-content/cache/supercache/$http_host/$cache_uri/index.html $uri $uri/ /index.php?$args;
}

Sorry for a delayed reply. I hope your issue is fixed by now.
We will be dropping support for wp super cache anyway.
If you are still facing the issue, please use http://community.rtcamp.com/c/easyengine for further support.
We are cleaning up issue tracker and closing this issue because we won't be able to contribute to this discussion further.


docs(secure-internal-postgres): add available ssl mode options

add available options to specify sslmode for uaa jdbc database connection
Closes #240

Thanks for the PR!
Can you add the sign-off line to your commit? It's quite common for OSS projects to require it.
Then update the branch by force pushing:
git push --force

@taylorsilva @xtremerui I apologize i never actually got a notification on this one, I've fixed it up for you, i had done that commit from a customer provided computer and didn't have all my settings :)

Thank you for the PR!


PAML building Components as child classes?

I found this in a serialization:
<https://bbn.com/scratch/iGEM_LUDOX_OD_calibration_2018/PrimitiveExecutable5/LocalValuePin2/Component4> a sbol:Component ;
    sbol:displayId "Component4" ;
    sbol:name "LUDOX(R) CL-X colloidal silica, 45 wt. % suspension in H2O" ;
    sbol:type <https://identifiers.org/pubchem.substance:24866361> .

This seems to indicate that the OPIL library being used by PAML has built an sbol:Component as a child object rather than a TopLevel. This was produced by https://github.com/SD2E/paml/blob/main/scratch.py

The assignment in this line https://github.com/SD2E/paml/blob/main/scratch.py#L91 is where the parent-child relationship of this object is created.  I believe this happens because the material property is defined as a compositional property here https://github.com/SD2E/paml/blob/main/paml/paml.ttl#L301

This has been fixed.


cnspec: 9.6.1 -> 9.8.0

Diff: https://github.com/mondoohq/cnspec/compare/refs/tags/v9.6.1...v9.8.0
Changelog: https://github.com/mondoohq/cnspec/releases/tag/v9.8.0
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 269495 run on x86_64-linux 1

1 package built:

cnspec


Make ain_getTransactionByHash() return tx execution result

Change summary:

Make ain_getTransactionByHash() return tx execution result
Add IN_BLOCK transaction state
Update the states of the transactions in blocks when they're executed

Related issues:

https://github.com/ainblockchain/ain-blockchain/issues/1202

Thanks for the review. @shyun-comcom @woomurf Now merging..


No such service: -d

When i set compose options up to something, like nginx, when i start a service it errors: 'nginx mysql redis workspace', if i clean the compose option up to blank, it then works...
Im using laradock (https://laradock.io/getting-started/)

Running on OSX Catalina. Docker desktop <IP_ADDRESS>, Compose 1.25.4

I have the same problem too. Any way to solve?


Cant leave Key Lock when in Hidden Menue

Thanks for the great work you're doing! I've found a small bug:
When you turn off the radio with KeyLock on, and go into the hidden menue, there is no way to leave the KeyLck. (Long Pressing F doesnt work)
After turning on the radio in normal mode and turning off keylock, everything works fine again.
Suggestion: Either make the F button work if you go into the hidden menu, or just disable keylock when hidden menu gets activated
Or is this a wontfix for memory reasons?

Hi,
Thank you for this issue. It's interesting.
The KEYLOCK feature is specifically designed to prevent changes. In my opinion, it's normal for the KEYLOCK feature to persist after a restart, even if you restart your pocket to access the hidden menus.
However, you're right: in hidden menu mode, there's no way to UNLOCK the pocket, as you go straight to the menu. This could be a (small) problem...
So I'll try to improve that, if I have enough memory.
Regards,
Armel F4HWN.

Hi,
Thank you for this issue. It's interesting.
The KEYLOCK feature is specifically designed to prevent changes. In my opinion, it's normal for the KEYLOCK feature to persist after a restart, even if you restart your pocket to access the hidden menus.
However, you're right: in hidden menu mode, there's no way to UNLOCK the pocket, as you go straight to the menu. This could be a (small) problem...
So I'll try to improve that, if I have enough memory.
Regards,
Armel F4HWN.

Oh and one small thing... It took me a moment to understand what you mean by "pocket". Perhaps "radio" would be a better term?
Dont take this the wrong way, most of us arent native english speakers (me included), and Im trying to say this in the nicest way possible... :upside_down_face:
And thanks again for the great work youre doing!

Ah ah ok 😂 🤭 Yes, sorry, "pocket" means "pocket radio" / "radio" / "transceiver" 😌

Fixed in v2.7

Thank you!


feat(extend): optionally deep-extend when last parameter is true

Same as #10507 but with my review comments addressed and jscs fixed.
Someone quickly review this so we can land it :<

I'll tweak the tests and land this today.


alerting on performance dashboard - if perf tests start going 10% slower

also, let's make the performance test that runs in CI quite a bit chunkier (more nodes, more jobs)
e.g. 100 nodes, 10000 jobs concurrency 10

we should also monitor memory usage


Fix broken links

./docs/about/FAQ.md

[ ] https://www.sciencedaily.com/releases/2016/11/161110154942.htm => 405

See also: https://travis-ci.org/sonarwhal/sonar/jobs/250225258

Duplicate of #360


feat: add onclick to connect buttons and refacto agentconnect

🚀 Thank you!

Hi @lsagetlethias,
Unfortunately the preload doesn't work for some reason.
https://github.com/codegouvfr/react-dsfr/assets/6702424/31439ba0-730d-4cdf-845a-802d4f9f612c

Fixed by this commit


nit: Adjust NextSequenceReceive to work with a potential ORDERED_ALLOW_TIMEOUT channel.

Description
Addressing comment made here: https://github.com/cosmos/ibc-go/pull/3357#discussion_r1152041510
closes: #XXXX
Commit Message / Changelog Entry
nit: Adjust `NextSequenceReceive` to work with a potential `ORDERED_ALLOW_TIMEOUT` channel.

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[x] Provide a commit message to be used for the changelog entry in the PR description for review.
[x] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Codecov Report

Merging #3378 (08b5dc4) into main (15cb750) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3378   +/-   ##
=======================================
  Coverage   78.67%   78.67%
=======================================
  Files         181      181
  Lines       12514    12514
=======================================
  Hits         9845     9845
  Misses       2238     2238
  Partials      431      431

Impacted Files
Coverage Δ

modules/core/04-channel/keeper/grpc_query.go
88.25% <100.00%> (ø)


Add support for async event handlers in Synchronizer class.

Context

Fixes some weird crosshairs behaviour when synchronizers where used (tmtv mode)
Fixes this issue https://github.com/OHIF/Viewers/issues/3758

Changes & Results

Testing

Checklist
PR

[] My Pull Request title is descriptive, accurate and follows the
semantic-release format and guidelines.

Code

[] My code has been well-documented (function documentation, inline comments,
etc.)

[] I have run the yarn build:update-api to update the API documentation, and have
committed the changes to this PR. (Read more here https://www.cornerstonejs.org/docs/contribute/update-api)

Public Documentation Updates

[] The documentation page has been updated as necessary for any public API
additions or removals.

Tested Environment

[] "OS:
[] "Node version:
[] "Browser:

I tested this on my end as well and it solved a lot of issues with the crosshairs.
Two issues were resolved:

No crosshair gap

Doubling crosshair lines during rotation

Here's a gif of me running the same manual test with the changes to the synchronizers, looks all perfect to me:


AMP: Accelerated Mobile Pages

https://support.google.com/webmasters/answer/6340290

@agungsb you should pick this task


Coverage and bug fixes

In tonight's extended edition, we try to move branch coverage to 80%.   During this process we discover a number of bugs that need correcting.

GIL segfaults on exit
C++ exception information not being written out
uint64 transform to float/double

This PR makes some use of GCOVR_EXCL statements as there are many parts of our code that is purely defensive and cannot be executed in normal operation or is restricted to shutdown and prestart conditions.
JPPlatform headers were moved to a source file as the level of code in the header was larger than I felt comfortable as a header.   A number of other header functions were moved to the source files so that it better represented a division between inline code and implementations.
This pull request is based on #619 thus it can't be included until the base PR is merged.  Also this likely conflicts with #617 and #618, so all three should be included first.
Fixes #620

Codecov Report

Merging #621 into master will increase coverage by 28.33%.
The diff coverage is 82.75%.

@@             Coverage Diff             @@
##           master     #621       +/-   ##
===========================================
+ Coverage   57.45%   85.79%   +28.33%
===========================================
  Files         102      100        -2
  Lines        8017     7784      -233
  Branches     3589     3459      -130
===========================================
+ Hits         4606     6678     +2072
+ Misses        914      596      -318
+ Partials     2497      510     -1987

Impacted Files
Coverage Δ

native/python/include/jp_pythontypes.h
100% <ø> (+10%)
:arrow_up:

native/common/jp_primitivetype.cpp
100% <ø> (+50%)
:arrow_up:

native/common/include/jp_booleantype.h
100% <ø> (+33.33%)
:arrow_up:

native/common/include/jp_voidtype.h
100% <ø> (+75%)
:arrow_up:

native/python/pyjp_proxy.cpp
93.33% <ø> (+40.87%)
:arrow_up:

native/common/include/jpype.h
100% <ø> (ø)
:arrow_up:

native/common/jp_value.cpp
50% <ø> (+16.66%)
:arrow_up:

native/common/include/jp_primitivetype.h
100% <ø> (+33.33%)
:arrow_up:

native/common/include/jp_context.h
60% <ø> (ø)
:arrow_up:

native/common/include/jp_platform.h
100% <ø> (ø)

... and 88 more

Continue to review full report at Codecov.

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

Due to COVID-19 unfortunately my business has shutdown by public health order.  Still waiting on details.  Perhaps that means I can plow through to JPype-0.8.

some conflicting files need to be resolved.

Thanks again for this incredible testing coverage increase. And you squashed so many bugs along this path :+1:

holy smokes what an increase! tremendous job!

@originell well 2/3 of the increase was a cheat.   The coverage system was counting all the exception paths as partials because there was a possibility that Python would pass in a null pointer into a reference handle and it would throw which would only happen if something really bad happened.  As the first thing we did when handling a Python object was put it into a reference handle so we could throw a C++ without losing a reference in many places, it was considering all the code as having an untested branch.   This was basically doubling our branch count.
Adding the flag --exclude-thrown-branches which the documentation indicates would produce more "sensible" branch coverage (not my word, but theirs) fixed most of our bad coverage problems.


Remove unnecessary Sync

Hi @mvniekerk
Please ref BoxFuture
Please review it. thank you.

Hi @vkill
Thank you for the contribution. If you don't mind, I'm going to hold off on this PR up until 0.6 beta 2 is released and chat with you about it then?

Hi @vkill
0.6 beta 2 has been updated. Can you kindly update your PR please?

Hi @vkill 0.6 beta 2 has been updated. Can you kindly update your PR please?

But I cannot test it now.
   Compiling tokio-cron-scheduler v0.6.0-beta.2 (/path/tokio-cron-scheduler)
error[E0601]: `main` function not found in crate `lib`
   --> examples/lib.rs:1:1
    |
1   | / use std::time::Duration;
2   | | use tokio_cron_scheduler::{Job, JobScheduler};
3   | | use tracing::{error, info};
4   | |
...   |
141 | |     sched.shutdown();
142 | | }
    | |_^ consider adding a `main` function to `examples/lib.rs`

And I have updated based on 0.6 beta 2.

Thank you @vkill


Efficient batch construction/insert for rle_str

It would be useful (I think) to have a way to make a rle_str from a vector<int>, or to insert a whole vector<int> into an rle_str at some position.
Would breaking those operations out as batch operations make them any more efficient than just inserting each value one at a time? I don't know much about the internals, but I suspect a workload of inserts all at the very end, or all one after the other, might produce tree balancing problems.

Thanks for the suggestion Adam!
In bitvectors batch inserts would take just O(log n + k) time, where k is the number of bits inserted. Tree balancing is not a problem because we would need to rebalance the tree (log n time) only every log n inserted bits (i.e. leaf size), so this cost would be amortized over the k insertions. In wavelet-tree strings this would be multiplied by log sigma. In rle_str this could probably also be done efficiently, but it's more tricky because we need to do multiple batch inserts on all the gap-encoded bitvectors corresponding to characters in the inserted vector.. I'm not sure about the complexity of doing this. I'll think about it.
While writing the library I was thinking about doing this, but since my main goal was to implement dynamic RLE FM-indexes I abandoned the idea (because in this case inserts are done at random positions and not in batches). I'll add it to the TODO list.

@nicolaprezza are you still working on RLE FM-indexes? That sounds really cool!

Thanks Erik! yes, the RLE FM-index in this library is fully operative and I have already used it to implement a couple of algorithms to compute LZ77 in compressed space. If you are interested in RLE indexes, take a look also to my github.com/nicolaprezza/lz-rlbwt repository: this one is a RLE FM-index combined with LZ77 to achieve compression also of the suffix-array sampling.


[Azure Load Testing] Updating ARM endpoint based on cloud

Changes made in this PR -

Adding support for "az load test" in AzureUSGovernment cloud by setting the ARM endpoint based on the cloud.

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

General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[x] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Azure Load Testing


Impossible to use App on macOS

What happens?
I've no idea how this is happening since I don't have anything in the app, but new Workspaces somehow can't do clean requests.
Simplest new workspace test to https://google.com
https://github.com/user-attachments/assets/950bef09-56ae-4c58-b5a2-d4882e9cee9c

What did you expect to happen?
Request should return https://google.com response, not some other workspace/cached/deleted/idk where taken response
OpenAPI Document
no need

hello there @vladshcherbin do you have anything set as the proxy?  hit settings in the bottom left

Then you should be able to see if anything is set there. Seems like its proxying through a vercel app

I don't have any settings for proxy or environment

Anyway I've moved back to Insomnia, can't stand the amount of bugs I find on daily basis. Closing since I won't be able to test.
There's also a bug where watch mode doesn't work the expected way. If you stop it - it will still do schema requests every ~5 seconds. Won't open another issue for it, just a note.

hey @vladshcherbin , we really appreciate you opening this issue and giving us context.
we also apologize for the bugs you've been hitting! i'm looking into this one right now.
stability is something we are moving towards, there was a lot of sweeping changes we had to make in the last 3 months that were a necessary evil that unfortunately made stability a bit hard to maintain.
no excuses, just wanted to give context. we are working hard on stability now.
i will update this thread once this is resolved, and also hope in the future you can try out scalar again :)
appreciate your time.


A hero section with jumbotron

A hero section
    - A jumbotron featuring your picture, your name, and any other information you'd like to include.

HTML done
Styling remaining

complete now. closing


Skip another mention of deprecated crate

Please see previous commit by Evan Mesterhazy for details

Will merge shortly, thanks a lot for your contribution


Dark.mode

About my code.
I made obj-c category on UIColor, which works very much like class extension in Swift, and added "adaptive" to all color declarations inside framework.
Also I added a universal converter for all colors, it reduces brightness of color.
But I also added another method
-(UIColor* _Nonnull)adaptiveWithDark:(UIColor* _Nonnull)color;

Use it to explicitly define color variant for dark mode if universal converter is not good for specific color. You can find example if its usage in demo, see CTApperanceViewController, line 62.

Hi Patrick, I merged latest fixes, please revise PR.


Create FIFO buffer for talking to Trinamic board to stabilize communication timing

This will likely help solve #152 because different threads are likely command that collide. By having a single file FIFO buffer, there will only be one thread in charge of sending and receiving messages to the board at a time.
There still could be collisions if more than one program is talking to the board.

fixed a year ago.


list object not callabble

Description
What steps will reproduce the problem?

I get "list object is not callable" for these np.array entries
points and p
my email is<EMAIL_ADDRESS>import numpy as np
#Knn nearest neighbor locate points closest to target point
points = np.array([[1,1],[1,2],[1,3],[2,1],[2,2],[2,3],[3,1],[3,2],[3,3]])
p=np.array([2.5, 2])
import matplotlib.pyplot as plt
plt.plot(points[:,0], points[:,1], "ro")
plt.plot(p[0],p[1],"bo")
Versions

Spyder version: 4.1.4
Python version: 3.8.3
Qt version: 5.9.7
PyQt5 version: 5.9.2
Operating System: Windows 10

Dependencies

# Mandatory:
atomicwrites >=1.2.0           :  1.4.0 (OK)
chardet >=2.0.0                :  3.0.4 (OK)
cloudpickle >=0.5.0            :  1.5.0 (OK)
diff_match_patch >=20181111    :  20200713 (OK)
intervaltree                   :  None (OK)
IPython >=4.0                  :  7.16.1 (OK)
jedi =0.17.1                   :  0.17.1 (OK)
nbconvert >=4.0                :  5.6.1 (OK)
numpydoc >=0.6.0               :  1.1.0 (OK)
paramiko >=2.4.0               :  2.7.1 (OK)
parso =0.7.0                   :  0.7.0 (OK)
pexpect >=4.4.0                :  4.8.0 (OK)
pickleshare >=0.4              :  0.7.5 (OK)
psutil >=5.3                   :  5.7.0 (OK)
pygments >=2.0                 :  2.6.1 (OK)
pylint >=1.0                   :  2.5.3 (OK)
pyls >=0.34.0;<1.0.0           :  0.34.1 (OK)
qdarkstyle >=2.8               :  2.8.1 (OK)
qtawesome >=0.5.7              :  0.7.2 (OK)
qtconsole >=4.6.0              :  4.7.5 (OK)
qtpy >=1.5.0                   :  1.9.0 (OK)
rtree >=0.8.3                  :  0.9.4 (OK)
sphinx >=0.6.6                 :  3.1.2 (OK)
spyder_kernels >=1.9.2;<1.10.0 :  1.9.2 (OK)
watchdog                       :  None (OK)
zmq >=17                       :  19.0.1 (OK)

# Optional:
cython >=0.21                  :  0.29.21 (OK)
matplotlib >=2.0.0             :  3.2.2 (OK)
numpy >=1.7                    :  1.18.5 (OK)
pandas >=0.13.1                :  1.0.5 (OK)
scipy >=0.17.0                 :  1.5.0 (OK)
sympy >=0.7.3                  :  1.6.1 (OK)

Hi @artsarno checking the code you provided I was able to run it without any issue:

At first glance seems like this isn't related with Spyder. Maybe in some place of your code instead of square brackets you are using parenthesis.
I'm closing this one but hope the info above helps :)

Thank you i believe it may be a Spyder version issue as the code runs on another IDE

From: Daniel Althviz Moré<EMAIL_ADDRESS>Sent: Wednesday, August 19, 2020 5:27:54 PM
To: spyder-ide/spyder<EMAIL_ADDRESS>Cc: artsarno<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [spyder-ide/spyder] list object not callabble (#13576)
Closed #13576https://github.com/spyder-ide/spyder/issues/13576.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/spyder-ide/spyder/issues/13576#event-3673049670, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQBN4BGRF34MECY5LR63PHTSBRUYVANCNFSM4QFOLJKQ.

Thank you again but I am using Spyder Python 3.8
There is no other code in this program
I really think this is a bug in the release
For numpy arrays

From: Art Sarno<EMAIL_ADDRESS>Sent: Wednesday, August 19, 2020 6:13:56 PM
To: spyder-ide/spyder<EMAIL_ADDRESS>spyder-ide/spyder<EMAIL_ADDRESS>Cc: Mention<EMAIL_ADDRESS>Subject: Re: [spyder-ide/spyder] list object not callabble (#13576)
Thank you i believe it may be a Spyder version issue as the code runs on another IDE

From: Daniel Althviz Moré<EMAIL_ADDRESS>Sent: Wednesday, August 19, 2020 5:27:54 PM
To: spyder-ide/spyder<EMAIL_ADDRESS>Cc: artsarno<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [spyder-ide/spyder] list object not callabble (#13576)
Closed #13576https://github.com/spyder-ide/spyder/issues/13576.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/spyder-ide/spyder/issues/13576#event-3673049670, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQBN4BGRF34MECY5LR63PHTSBRUYVANCNFSM4QFOLJKQ.

Could you try to submit a GIF displaying the way you are running your code? You can use for that something like LICECap (you probably will need to submit the GIF using Github directly instead of responding by email).
Without more info to replicate the behavior your are experiencing we can't help you.
Also, just in case, bellow is the env used for the test where the example code runs without issue (using Python 3.8):

# packages in environment at C:\Users\Daniel\Anaconda3\envs\py38:
#
# Name                    Version                   Build  Channel
alabaster                 0.7.12                     py_0
argh                      0.26.2                   py38_0
astroid                   2.4.2                    py38_0
atomicwrites              1.4.0                      py_0
attrs                     19.3.0                     py_0
autopep8                  1.5.4                      py_0
babel                     2.8.0                      py_0
backcall                  0.2.0                      py_0
bcrypt                    3.1.7            py38he774522_1
blas                      1.0                         mkl
bleach                    3.1.5                      py_0
brotlipy                  0.7.0           py38he774522_1000
ca-certificates           2020.6.24                     0
certifi                   2020.6.20                py38_0
cffi                      1.14.0           py38h7a1dbc1_0
chardet                   3.0.4                 py38_1003
cloudpickle               1.5.0                      py_0
colorama                  0.4.3                      py_0
cryptography              2.9.2            py38h7a1dbc1_0
cycler                    0.10.0                   py38_0
decorator                 4.4.2                      py_0
defusedxml                0.6.0                      py_0
diff-match-patch          20200713                   py_0
docutils                  0.16                     py38_1
entrypoints               0.3                      py38_0
flake8                    3.8.3                      py_0
freetype                  2.10.2               hd328e21_0
future                    0.18.2                   py38_1
icu                       58.2                 ha925a31_3
idna                      2.10                       py_0
imagesize                 1.2.0                      py_0
importlib-metadata        1.7.0                    py38_0
importlib_metadata        1.7.0                         0
intel-openmp              2020.1                      216
intervaltree              3.0.2                      py_1
ipykernel                 5.3.4            py38h5ca1d4c_0
ipython                   7.16.1           py38h5ca1d4c_0
ipython_genutils          0.2.0                    py38_0
isort                     4.3.21                   py38_0
jedi                      0.17.1                   py38_0
jinja2                    2.11.2                     py_0
jpeg                      9b                   hb83a4c4_2
jsonschema                3.2.0                    py38_0
jupyter_client            6.1.6                      py_0
jupyter_core              4.6.3                    py38_0
keyring                   21.2.1                   py38_0
kiwisolver                1.2.0            py38h74a9793_0
lazy-object-proxy         1.4.3            py38he774522_0
libpng                    1.6.37               h2a8f88b_0
libsodium                 1.0.18               h62dcd97_0
libspatialindex           1.9.3                h33f27b4_0
markupsafe                1.1.1            py38he774522_0
matplotlib                3.2.2                         0
matplotlib-base           3.2.2            py38h64f37c6_0
mccabe                    0.6.1                    py38_1
mistune                   0.8.4           py38he774522_1000
mkl                       2020.1                      216
mkl-service               2.3.0            py38hb782905_0
mkl_fft                   1.1.0            py38h45dec08_0
mkl_random                1.1.1            py38h47e9c7a_0
nbconvert                 5.6.1                    py38_0
nbformat                  5.0.7                      py_0
numpy                     1.19.1           py38h5510c5b_0
numpy-base                1.19.1           py38ha3acd2a_0
numpydoc                  1.1.0                      py_0
openssl                   1.1.1g               he774522_1
packaging                 20.4                       py_0
pandas                    1.1.0            py38ha925a31_0
pandoc                    2.10.1                        0
pandocfilters             1.4.2                    py38_1
paramiko                  2.7.1                      py_0
parso                     0.7.0                      py_0
pathtools                 0.1.2                      py_1
pexpect                   4.8.0                    py38_0
pickleshare               0.7.5                 py38_1000
pip                       20.2.2                   py38_0
pluggy                    0.13.1                   py38_0
prompt-toolkit            3.0.5                      py_0
psutil                    5.7.0            py38he774522_0
pycodestyle               2.6.0                      py_0
pycparser                 2.20                       py_2
pydocstyle                5.0.2                      py_0
pyflakes                  2.2.0                      py_0
pygments                  2.6.1                      py_0
pylint                    2.5.3                    py38_0
pynacl                    1.4.0            py38h62dcd97_1
pyopenssl                 19.1.0                     py_1
pyparsing                 2.4.7                      py_0
pyqt                      5.9.2            py38ha925a31_4
pyrsistent                0.16.0           py38he774522_0
pysocks                   1.7.1                    py38_0
python                    3.8.5                he1778fa_0
python-dateutil           2.8.1                      py_0
python-jsonrpc-server     0.3.4                      py_1
python-language-server    0.34.1                   py38_0
pytz                      2020.1                     py_0
pywin32                   227              py38he774522_1
pywin32-ctypes            0.2.0                 py38_1000
pyyaml                    5.3.1            py38he774522_1
pyzmq                     19.0.1           py38ha925a31_1
qdarkstyle                2.8.1                      py_0
qt                        5.9.7            vc14h73c81de_0
qtawesome                 0.7.2                      py_0
qtconsole                 4.7.5                      py_0
qtpy                      1.9.0                      py_0
requests                  2.24.0                     py_0
rope                      0.17.0                     py_0
rtree                     0.9.4            py38h21ff451_1
setuptools                49.6.0                   py38_0
sip                       4.19.13          py38ha925a31_0
six                       1.15.0                     py_0
snowballstemmer           2.0.0                      py_0
sortedcontainers          2.2.2                      py_0
sphinx                    3.2.1                      py_0
sphinxcontrib-applehelp   1.0.2                      py_0
sphinxcontrib-devhelp     1.0.2                      py_0
sphinxcontrib-htmlhelp    1.0.3                      py_0
sphinxcontrib-jsmath      1.0.1                      py_0
sphinxcontrib-qthelp      1.0.3                      py_0
sphinxcontrib-serializinghtml 1.1.4                      py_0
spyder                    4.1.4                    py38_0
spyder-kernels            1.9.2                    py38_0
sqlite                    3.32.3               h2a8f88b_0
testpath                  0.4.4                      py_0
toml                      0.10.1                     py_0
tornado                   6.0.4            py38he774522_1
traitlets                 4.3.3                    py38_0
ujson                     1.35             py38he774522_0
urllib3                   1.25.10                    py_0
vc                        14.1                 h0510ff6_4
vs2015_runtime            14.16.27012          hf0eaf9b_3
watchdog                  0.10.3                   py38_0
wcwidth                   0.2.5                      py_0
webencodings              0.5.1                    py38_1
wheel                     0.34.2                   py38_0
win_inet_pton             1.1.0                    py38_0
wincertstore              0.2                      py38_0
wrapt                     1.11.2           py38he774522_0
yaml                      0.2.5                he774522_0
yapf                      0.30.0                     py_0
zeromq                    4.3.2                ha925a31_2
zipp                      3.1.0                      py_0
zlib                      1.2.11               h62dcd97_4

Let us know if you have more info in order to replicate the issue :)

Dear Daniel
Thank you so much for your assistance.
This morning I reset% the console and now the code is working!
I Was not aware that by running the program using the ‘Run’ menu option that the console code would have an adverse impact.
Best Regards,
Art Sarno

On Aug 19, 2020, at 9:08 PM, Daniel Althviz Moré<EMAIL_ADDRESS>wrote:

Could you try to submit a GIF displaying the way you are running your code? You can use for that something like LICECap (you probably will need to submit the GIF using Github directly instead of responding by email).
Without more info to replicate the behavior your are experiencing we can't help you.
Also, just in case, bellow is the env used for the test where the example code runs without issue (using Python 3.8):
Let us know if you have more info in order to replicate the issue :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Add an onboard LED?

Would it be possible to get an onboard LED on the breakout board? It would allow for some useful debug "output" and status indication in testing.
And, as a follow-up thought, if it can function as a drop-in for the equivalent RGB onboard LED on the Feather boards, that would make the File->New project experience equivalent too.

Good idea


⚠ Bofrost.nl has degraded performance

In e436246, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 940 ms

Resolved: Bofrost.nl performance has improved in 1140e39.


IPS not working

It could be me just doing something wrong. I did follow the guide to block facebook from https://docs.opnsense.org/manual/how-tos/ips-sslfingerprint.html
I then cleared the cache and everything in firefox using CTRL+SHIFT+DEL and then when to facebook. It did not generate an alert.
I am doing this to test to make sure the rules that I have enabled that come with OPNsense are working correctly.

@L1ghtn1ng I've seen the same behaviour before when using firefox, it looks like flushing the cache doesn't always forget the state of the ssl session.
I just tried the same for our website (www.deciso.com) using the corresponding fingerprint (AA:C4:99:2B:A0:23:E8:3D:61:DB:E4:21:C1:07:4E:FB:F3:1A:98:C1) same issue when using firefox, internet explorer did trigger an alert, firefox didn't (only after a full reset, https://support.mozilla.org/en-US/kb/refresh-firefox-reset-add-ons-and-settings).

Does chrome/chromium work correctly?

I am also trying to work out as well if the rules that are in  Opnsense by default without me creating a rule are working correctly as I have not seen any blocked alerts at all and I would of thought I would have

@L1ghtn1ng Maybe we should add an easy to test ruleset to our set, I've heard other people as well having issues with testing their setup, most rules in there are meant for specific (real) threads.
Can we change the issue to something like "IPS: add easy setup test ruleset"?

Yeah go for it

added a simple test using the eicar test virus (http://www.eicar.org/download/eicar.com.txt)
just apply the test ruleset and try to download the virus, it should block and log the event.
Maybe we should add additional tests at some point, but I don't have better ideas at the moment for them.

How about a simple website that could be setup to also test SSL
blocking as well?


删除combo里的nodo后报错

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link

Steps to reproduce
删除节点后回退

Environment
Info

g6
4.3.5

System
win10

Browser
Chrome

移动combo后回退也有问题，如何能一步回退到初始拖动位置？

下个小版本会修复


Only consider relevant entities for HmIP-RGBW

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (b2bee57) 90.46% compared to head (27b8ac7) 90.39%.

Files
Patch %
Lines

hahomematic/platforms/custom/light.py
50.00%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #1361      +/-   ##
==========================================
- Coverage   90.46%   90.39%   -0.07%
==========================================
  Files          49       49
  Lines        5976     5989      +13
==========================================
+ Hits         5406     5414       +8
- Misses        570      575       +5

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.