RangeError when both removing listener and disposing ValueNotifier from listener callback

This code
void main() {
  final vn = ValueNotifier<int>(0);

  void foo() {
    vn.removeListener(foo); // A
    vn.dispose(); // B
  }

  vn.addListener(foo);

  vn.value = 3;
}

results in:
flutter: RangeError (length): Invalid value: Not in inclusive range 0..576460752303423487: -1, #0      new _List (dart:core-patch/array.dart)
#1      ChangeNotifier.notifyListeners (package:flutter/src/foundation/change_notifier.dart:384:50)
#2      ValueNotifier.value= (package:flutter/src/foundation/change_notifier.dart:463:5)

while I would expect it to work. There is no exception when either line A or line B is removed.
Issue can be fixed by setting also _reentrantlyRemovedListeners to zero in dispose here:
https://github.com/flutter/flutter/blob/ea0ddc94ccc63cec77bcac3ef02832806adcd667/packages/flutter/lib/src/foundation/change_notifier.dart#L332
[!] Flutter (Channel unknown, 3.3.2, on macOS 12.5.1 21G83 darwin-arm, locale en-NL)
    ! Flutter version 3.3.2 on channel unknown at /.../flutter
    ! Upstream repository unknown
    • Framework revision e3c29ec00c (6 weeks ago), 2022-09-14 08:46:55 -0500
    • Engine revision a4ff2c53d8
    • Dart version 2.18.1
    • DevTools version 2.15.0

Hi @spkersten, this issue is reproducible on the latest stable and master channels with provided sample code.
The error message is a little different from yours. Labeling this issue for further investigation.

Logs
RangeError: Invalid array length
dart-sdk/lib/_internal/js_dev_runtime/patch/core_patch.dart 557:30                filled
packages/flutter/src/foundation/change_notifier.dart 401:50                       notifyListeners
packages/flutter/src/foundation/change_notifier.dart 489:5                        set value
packages/reproduce_issues/range_error_114224.dart 12:5                            main
web_entrypoint.dart 24:31                                                         <fn>
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 352:37  _checkAndCall
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 357:39  dcall
lib/ui/initialization.dart 77:15                                                  <fn>
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 45:50                <fn>
dart-sdk/lib/async/zone.dart 1660:54                                              runUnary
dart-sdk/lib/async/future_impl.dart 147:18                                        handleValue
dart-sdk/lib/async/future_impl.dart 767:44                                        handleValueCallback
dart-sdk/lib/async/future_impl.dart 796:13                                        _propagateToListeners
dart-sdk/lib/async/future_impl.dart 567:5                                         [_completeWithValue]
dart-sdk/lib/async/future_impl.dart 640:7                                         callback
dart-sdk/lib/async/schedule_microtask.dart 40:11                                  _microtaskLoop
dart-sdk/lib/async/schedule_microtask.dart 49:5                                   _startMicrotaskLoop
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 166:15               <fn>

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.3.6, on macOS 12.6 21G115 darwin-x64, locale en-VN)
    • Flutter version 3.3.6 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 6928314d50 (30 hours ago), 2022-10-25 16:34:41 -0400
    • Engine revision 3ad69d7be3
    • Dart version 2.18.2
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.0.1)
    • Xcode at /Applications/Xcode14.0.1.app/Contents/Developer
    • Build 14A400
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] IntelliJ IDEA Community Edition (version 2022.2.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 70.0.5
    • Dart plugin version 222.4167.21

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

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

[✓] Connected device (3 available)
    • SM T225 (mobile) • R9JT3004VRJ • android-arm64  • Android 12 (API 31)
    • macOS (desktop)  • macos       • darwin-x64     • macOS 12.6 21G115 darwin-x64
    • Chrome (web)     • chrome      • web-javascript • Google Chrome 107.0.5304.62

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[!] Flutter (Channel master, 3.5.0-10.0.pre.166, on macOS 12.6 21G115 darwin-x64, locale en-VN)
    • Flutter version 3.5.0-10.0.pre.166 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4f3f932a68 (5 hours ago), 2022-10-30 17:44:24 -0400
    • Engine revision f91119f84f
    • Dart version 2.19.0 (build 2.19.0-353.0.dev)
    • DevTools version 2.18.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.0.1)
    • Xcode at /Applications/Xcode14.0.1.app/Contents/Developer
    • Build 14A400
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] IntelliJ IDEA Community Edition (version 2022.2.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 70.0.5
    • Dart plugin version 222.4167.21

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.6 21G115 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 107.0.5304.87

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

I think we should disallow calling dispose in the listener callback. https://api.flutter.dev/flutter/foundation/ChangeNotifier/dispose.html
The document suggest only the owner should call this method.

That could be difficult to prevent if a few layers of indirection are present.

If we allows it, it will be hard to manage its lifecycle, and we also need to figure out what would happen after the dispose is called.

the rest of the listeners are still called
This will require delay dispose which is very error prompt and hard to maintain. and we also need to worry about multiple of them may call dispose as well.
the rest of the listeners are skipped.
In this case the invoke order would matter, but we don't guarantee invoke order IIRC. So it would create inconsistent behavior.

I think our current code with your suggestion will be implementing (2).
I can't think of a case that calling dispose in listener is a valid use case. Do you have any use case that this may be more preferred?

I created https://github.com/flutter/flutter/pull/114530, but we should keep the conversation going since this may be a hard breaking change


Allow dataset insert view form field default values to come from URL parameters

Rationale
A new client would like to be able to customize / generate a URL from the study participant page so that a user can insert a new row in a dataset and pull through the ParticipantId as the default / initial value in the dataset insert view form. We currently allow for "default values" to be set on study dataset properties, but that doesn't fit this scenario since they would like to be able to set that participant id based on the referring page.
With this PR, I have added code that will use the InsertView setInitialValues mechanism used for domain default values to also look for URL parameters as a way to set default values into the insert view form. Those URL param default values apply as follows:

only for the insert case initial page load (i.e. not respected on error reshow)
URL default values will be applied after the domain properties default values are set
the URL param key must exactly match the insert view form field key

Here is an example URL that would populate the ParticipantId and Date fields of the dataset insert form based on URL params:
/dataset-insert.view?datasetId=5010&quf_ParticipantId=999320016&quf_Date=2020-06-25
Changes

InsertUpdateAction.java update in getView() to look for URL parameters that match the insert view form field keys to populate initial values

@labkey-matthewb @kevinkrouse @labkey-klum I created this draft PR as a way for you guys to review this idea I was playing with as a way to allow for a client to be able to set default values in a dataset "Insert New Row" form's input fields. Let me know if you guys know of other existing mechanisms that I should explore for this or if you think there are issues with this approach. Thanks.

I think this is a very useful feature.  I don't particularly like the "quf_", but it is useful to have a prefix that is tied to this feature, and does not look like the filter syntax.  Westide used ".", maybe "" by itself would work, or we could be really explicit like "default".

I think this is a very useful feature. I don't particularly like the "quf_", but it is useful to have a prefix that is tied to this feature, and does not look like the filter syntax. Westide used ".", maybe "" by itself would work, or we could be really explicit like "default".

@labkey-matthewb I am only using the "quf_" here because then we get the form value binding for free since that is the prefix used by these dataset insert forms. I could see using a different prefix, but that would mean a bit more code for handling this case. I can make that change if you think it is necessary.

I'd vote for a supporting a different prefix, since it's something we'll be telling others to use in their URLs and trying to maintain backwards compatibility with.

I'd vote for a supporting a different prefix, since it's something we'll be telling others to use in their URLs and trying to maintain backwards compatibility with.

Sounds good. Two votes for a specific URL param prefix works for me. I've updated this PR to use "default." as the URL param prefix for this scenario.

@labkey-adam @labkey-matthewb @labkey-jeckels This PR is ready for re-review, with the change made to use a URL param prefix (i.e. "**default.**ParticipantId", "**default.**Date", etc.).


Integrate Apache Guacamole into Hoster Control Center

Apache Guacamole looks like a much better alternative to noVNC in terms of VM access. It will give users the ability to not only VNC into their VMs, but also use RDP and SSH directly from their browser.
This feature is not a priority right now, but the development can be sped up by user donations or an investment from one of our clients.

Closing down, until better times.


Encountering Issues While Replicating Code

Dear author,
Thank you for your generous sharing.
While attempting to reproduce your work, I encountered some issues. Using the SS-3D-Adv attack method, the victim model is PointNet, and the Target Model includes [PointNet, PointNet++, PointConv, DGCNN]. The attack success rates obtained were [89.65, 3.72, 1.92, 3.45], which differ from the results in your paper [94.9, 10.7, 3.96, 7.70]. The hyper-parameters used are those shown in your paper. What's going on? Is there something wrong on my end?

Dear Hebang,
Our ASR is different from previous work, pls refer to the AOF attack.


setDefaultSchema does not work . All changesets getting executed against public schema.

Environment
Liquibase Version: 4.4.1
Liquibase Integration & Version: spring boot 2.5.2
Liquibase Extension(s) & Version:  N/A
Database Vendor & Version: MySql 8.0
Operating System Type & Version: Windows 10
Description
I am creating Schema-Based Multi Tenant application with spring boot(2.5.2) , mySQL 8.0 and Liquibae 4.4.1.
I expect that all my tenant tables and liquibase internal tables should be created inside a tenant schema which I specify before running the migration.
liquibase.setDefaultSchema(tenantSchema); --sets the schema for changesets
liquibase.setLiquibaseSchema(tenantSchema); -- sets schema for liquibase internal tables
I can see that liquibase internal tables are getting created inside tenant schema correctly .
But liquibase is always executing my changesets against public schema. Looks like setting the deafult schema prior to liquibase migration has no effect. I am using sql-based changesets and I don't want to switch to xml
Steps To Reproduce

create multiple schemas in MySql database.
disable liquibase auto config in spring boot and manually create Spring Liquibase objeect.
create a simple create table sql script .
now try to run this sql against all schemas one by one. Before running the migration , set the target schema lke-liquibase.setDefaultSchema(schema)
You will get 'table already exists ' error.

Actual Behavior
All changesets are getting executed against public schema eve if I specify the target schema before running migration.
Expected/Desired Behavior
All changesets should be executed against the schema which is specified by calling setDefaultSchema on liquibase object.
Screenshots (if appropriate)
If applicable, add screenshots to help explain your problem.
Additional Context
Add any other context about the problem here.

Hi @p-muluk Thanks for creating this issue.  We will add this to the list of issues that we are processing.

If this is a still an issue for you, please try to reopen the issue and we will look into it again. Thanks!


Swagger UI page unresponsive for large swagger json

Hi Team,
I have the swagger json with multiple complex model elements and services collectively sized almost 1.7MB. The swagger generated code is in jersey 2.0.
I am using the latest angular-swagger-ui which is deployed with my server under webapps, but while loading the services from swagger ui the ui page is getting unresponsive, is there any performance improvement can be done for larger swagger json ?
Thanks

Hi !
Do you mind giving an access to your swagger json as i can see what's happening?
Regards

Sure @mathieuales
As this is hosted in a secured proxy server, is there any way i can share this modified json file via email or other options ?

You can attach files to github comments using drag'n'drop

Hi @mathieuales , Sorry for the delay in the response.
Attaching the swagger json we are facing performance issue specifically while loading services having complex models elements like "/complete" , "/rates".
swagger.txt
Thanks in advance..

Hi!
Unfortunately there is not much to do, your models are very huge!!
I tried different things and got it a bit better but nothing acceptable. Generating samples and models takes time and generates large HTML which takes time to render with angularJS.
The result is the same or maybe worse using official swaggerUI as it is unable to render models.
My advise is you should maybe consider redesigning your API as exposing such huge models does not seem to be a good pratice.
Regards


Could not read the traversal from the request sent on DataStax Graph

I am using the latest version and I am encountering errors I am converting the result of my query to case classes.
This is the definition of my case class

import gremlin.scala.{Vertex, id, label, underlying}

@label("tripType")
case class TripType(@id tripTypeId: Option[String], description: Option[String] = Some(""), tripType: Option[String] = Some(""),
                    @underlying vertex: Option[Vertex] = None)

This is how I invoke them
 var r = GrDbConnection.g.V.hasLabel[TripType].toCC[TripType].head()
I always end up receiving this error
[InvalidQueryException: Could not read the traversal from the request sent. Cause: Could not locate method: DefaultGraphTraversal.map([{}])]
This is my definition  my graph

schema.propertyKey("tripTypeId").Uuid().ifNotExists().create();`
schema.vertexLabel("tripType").partitionKey('tripTypeId').properties("tripTypeId","tripType", "description").ifNotExists().create();

Looks like I implemented it the wrong way. It was supposed to be like this.
db.g.V.hasLabel[TripType].toList().foreach(x => println(x.toCC[TripType]))
db.g.V.hasLabel[TripType].toList().foreach(x => println(x.toCC[TripType]))


Add more logging to e2e tests

cleanup rules in the test to make it idempotent
increase verbosity for debug image

Jenkins job manager/presubmit passed


Dev gem's default update behavior seems undesirable.

If I don't specify what the default tasks are, the dev gem calls an update task after the publish task. In the update task dev gem calls nuget update if there are nuget packages in the project. This can cause an unexpected updating of nuget packages versions.
What we found is, the setup's nuget restore correctly restores the nuget packages in the project. But after a successful compile and publish, the update will call a nuget update and which will result in nuget updating to the latest version of a nuget dependency.
When the update of the nuget dependencies happens after a successful build and the project no longer builds it is confusing from the users perspective what happened. It takes some detective work to figure out what happened.
Our work around to solve the issue was to define the default task list.

is there an action you would like me to take?
Lou Parslow
Senior Software Engineer
Musco Sports Lighting
mobile |<PHONE_NUMBER>
<EMAIL_ADDRESS>On Tue, Feb 13, 2018 at 9:21 AM, kevin-marshall<EMAIL_ADDRESS>wrote:

If I don't specify what the default tasks are, the dev gem calls an update
task after the publish task. In the update task dev gem calls nuget update
if there are nuget packages in the project. This can cause an unexpected
updating of nuget packages versions.
What we found is, the setup's nuget restore correctly restores the nuget
packages in the project. But after a successful compile and publish, the
update will call a nuget update and which will result in nuget updating to
the latest version of a nuget dependency.
When the update of the nuget dependencies happens after a successful build
and the project no longer builds it is confusing from the users perspective
what happened. It takes some detective work to figure out what happened.
Our work around to solve the issue was to define the default task list.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lou-parslow/dev.gem/issues/2, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUGonWV2iYNtbkYMuNi87gIgZ8qvv4oks5tUbaFgaJpZM4SEBBk
.


Replace "SOURCE_TYPE_GPS" with "SourceType.GPS"

Updated to HA 2024.3.0 and got this message:
"SOURCE_TYPE_GPS was used from seatconnect, this is a deprecated constant which will be removed in HA Core 2025.1. Use SourceType.GPS instead."
Chris

Duplicate of https://github.com/Farfar/homeassistant-seatconnect/issues/49


ja: Make docs/concepts/services-networking/connect-applications-service.md follow v1.19 of the original text

This is a Feature Request

What would you like to be added
Update docs/concepts/services-networking/connect-applications-service.md to follow v1.19 of the original (English) text.
Why is this needed
content/ja/docs/concepts/services-networking/connect-applications-service.md is outdated.
Comments
/language ja
/good-first-issue
File to update
https://github.com/kubernetes/website/tree/dev-1.19-ja.1/content/ja/docs/concepts/services-networking/connect-applications-service.md
Original
https://github.com/kubernetes/website/tree/release-1.19/content/en/docs/concepts/services-networking/connect-applications-service.md
diff between translated and v1.19
https://gist.github.com/8480dbb9b45996108af9f82e22ee5cd8
Note
Currently, we use dev-1.19-ja.1 branch. Please open a PR targeting the branch.
ref
https://kubernetes.io/ja/docs/contribute/localization/
If you have a question, feel free to ask us at slack.k8s.io #kubernetes-docs-ja channel.

/triage accepted

@inductor @nasa9084
Hello,
I checked the modification in this issue.
In English, this was changed the code of apostrophe of "machine's".
I don't think it has any effect in Japanese and don't need to update.
Please check.
And if there is anything I can help, please contact me.

@s-kawamura-w664
That seems right! Thank you for your review.
/close


Context menu entry for highlighted payments & QR codes

If you right click on a highlighted BOLT-11 payment string / qr code, it would be awesome to have a one-click option to prompt you to pay, rather than having to copy / paste it in.

@wbobeirne I took a stab at implementing this feature in PR #108. Hope you don't mind.


[BUG] Update the doc best practice

Describe the bug
No SUSE SLES described in the best practice, we should correct the OSes Longhorn supported.
https://longhorn.io/docs/1.1.2/best-practices/#software
https://longhorn.io/docs/1.1.2/best-practices/#oses-arent-supported-by-longhorn
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Perform '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Log
If applicable, add the Longhorn managers' log when the issue happens.
You can also attach a Support Bundle here. You can generate a Support Bundle using the link at the footer of the Longhorn UI.
Environment:

Longhorn version:
Installation method (e.g. Rancher Catalog App/Helm/Kubectl):
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version:

Number of management node in the cluster:
Number of worker node in the cluster:

Node config

OS type and version:
CPU per node:
Memory per node:
Disk type(e.g. SSD/NVMe):
Network bandwidth between the nodes:

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal):
Number of Longhorn volumes in the cluster:

Additional context
Add any other context about the problem here.

@jenting feel free to remove the other elements from the issue templates that don't apply when opening an issue :)

Also ref #2694 change of recommended over provisioning percentage in the best practices guide, this needs discussion/approval from @innobead @yasker left some information + reasoning in the issue.

Thanks for the head up @joshimoo . I've concerns over change the default overprovisioning percentage. Let's talk about it when we have time.

The PR addressed it, close this issue.


(role/comcam-fp) add tcp_tweak profile for s3daemon

These are the parameters that KT requested for the lsstcam-dc Nodes.

Why is this different from the existing lhn profile?

just 2 parameters are different (net.ipv4.tcp_congestion_control & net.ipv4.tcp_slow_start_after_idle) you rather I keep the LHN and add another "profile" for these two parameters, or do I add this to the existing LHN profile? @jhoblitt

we should add these 2 parameters to the lhn profile, the rest is the same.

just 2 parameters are different (net.ipv4.tcp_congestion_control & net.ipv4.tcp_slow_start_after_idle) you rather I keep the LHN and add another "profile" for these two parameters, or do I add this to the existing LHN profile? @jhoblitt

I'd rather add those sysctls to the existing LHN profile as support USDF transfers is the intended function of that profile.


Buttons with Dropdowns : Height of dropdown & input don't match in the example in official docs

Using Chrome (Version 46.0.2490.80 (64-bit)) on Ubuntu (14.04).
The height of dropdown & input don't match in the example in docs (Bootstrap v3)

Screenshot (misaligned at bottom edge, click image to enlarge) :

Dropdown height : 19.08

Input height : 20.173

P.S. It's correctly aligned on Chrome + iPad, Chrome + Windows, Firefox + Ubuntu

Please confirm that the page zoom level is 100% (i.e. not zoomed). Page zooming isn't really supported, per http://getbootstrap.com/getting-started/#support-browser-zooming
If zooming isn't the problem, could you please re-test against the latest Bootstrap code in the master branch?
Note that Linux is only unofficially supported, per http://getbootstrap.com/getting-started/#support-browsers

Sorry, my bad. It was indeed a zoom issue that I missed.


Issues connecting to sample app following deployment

Getting the following errors during the deployment, resulting in an error 502 when trying to access the application. In job details, it has a status of: "SUCCEEDED":
null_resource.tomcat_server_config_with_bastion[1] (remote-exec): Errors during downloading metadata for repository 'ol8_ksplice':
null_resource.tomcat_server_config_with_bastion[1] (remote-exec): Error: Failed to download metadata for repo 'ol8_ksplice': Cannot download repomd.xml: Cannot download repodata/repomd.xml: All mirrors were tried
null_resource.tomcat_server_config_with_bastion[0] (remote-exec): Errors during downloading metadata for repository 'ol8_ksplice':
null_resource.tomcat_server_config_with_bastion[0] (remote-exec): Error: Failed to download metadata for repo 'ol8_ksplice': Yum repo downloading error: Downloading error(s): repodata/c777785048c9ca01d1b61db72d4b0ddf9987ca1f-primary.xml.gz - Cannot download, all mirrors were already tried without success; repodata/c8353de4debe349b443b0e8b813444729f3c4b93-filelists.xml.gz - Cannot download, all mirrors were already tried without success
null_resource.tomcat_server_config_with_bastion[1] (remote-exec): Errors during downloading metadata for repository 'ol8_ksplice':
null_resource.tomcat_server_config_with_bastion[1] (remote-exec): Error: Failed to download metadata for repo 'ol8_ksplice': Cannot download repomd.xml: Cannot download repodata/repomd.xml: All mirrors were tried
null_resource.tomcat_server_config_with_bastion[1] (remote-exec): Errors during downloading metadata for repository 'ol8_ksplice':
null_resource.tomcat_server_config_with_bastion[1] (remote-exec): Error: Failed to download metadata for repo 'ol8_ksplice': Cannot download repomd.xml: Cannot download repodata/repomd.xml: All mirrors were tried
null_resource.tomcat_server_config_with_bastion[0] (remote-exec): Errors during downloading metadata for repository 'ol8_ksplice':
null_resource.tomcat_server_config_with_bastion[0] (remote-exec): Error: Failed to download metadata for repo 'ol8_ksplice': Cannot download repomd.xml: Cannot download repodata/repomd.xml: All mirrors were tried
null_resource.tomcat_server_config_with_bastion[0] (remote-exec): Errors during downloading metadata for repository 'ol8_MySQL80':
null_resource.tomcat_server_config_with_bastion[0] (remote-exec): Error: Failed to download metadata for repo 'ol8_MySQL80': Yum repo downloading error: Downloading error(s): repodata/e1bf72614b2807b57db2730f6c0d460702e49482-primary.xml.gz - Cannot download, all mirrors were already tried without success
null_resource.tomcat_server_config_with_bastion[1] (remote-exec): Job for tomcat.service failed because the control process exited with error code.
null_resource.tomcat_server_config_with_bastion[0] (remote-exec): Job for tomcat.service failed because the control process exited with error code.

Looks like temporary YUM repositories were unavailable. I was able to install all elements from the bootstrap script by hand:

I recommend destroying the stack in ORM and redeploying it with apply command.


mirror update-center.json can't identify LTS update

base version LTS  2.190.1, notify to update LTS 2.190.2 with  http://updates.jenkins-ci.org/update-center.json
but if i change to https://jenkins-zh.gitee.io/update-center-mirror/tsinghua/update-center.json,  then notify to update 2.204,

I fixed this issue. You can use a new address which is https://updates.jenkins-zh.cn/update-center.json.
If you want to know more tech details, please feel free to visit here.

Another error info, i do check update after i update the  https://updates.jenkins-zh.cn/update-center.json.
update center: Signature verification failed in update site 'default' （显示详情）java.security.cert.CertPathValidatorException: Path does not chain with any of the trust anchors	at sun.security.provider.certpath.PKIXCertPathValidator.validate(PKIXCertPathValidator.java:153)	at sun.security.provider.certpath.PKIXCertPathValidator.engineValidate(PKIXCertPathValidator.java:79)	at java.security.cert.CertPathValidator.validate(CertPathValidator.java:292)	at org.jvnet.hudson.crypto.CertificateUtil.validatePath(CertificateUtil.java:93)	at jenkins.util.JSONSignatureValidator.verifySignature(JSONSignatureValidator.java:88)	at hudson.model.UpdateSite.verifySignature(UpdateSite.java:254)	at hudson.model.UpdateSite.updateData(UpdateSite.java:218)	at

What's the version of your Jenkins?

Jenkins ver. 2.190.1

I just tested it in a new environment with 2.190.1. It's ok. Please make sure that you follow the right flow:

use the new certificate
change to the new update center site.

I also encountered the same problem mentioned above. What does "use the new certificate" mean? what should I do?

You should put this file into your Jenkins. I believe that you can get more details from here.


[Client] Beautify visuals

Add death animation (Leave a pile of ashes in the color of the player on the death position)
Add bomb dissipation animation (maybe, to be considered)
Add unique color for each character
Rework tileset
Show error messages in the screens like connection errors etc.

Unique color done
Death animation done

Unique color done
Death animation done


Error: Redirect URL must be absolute

Detailed description
I'm trying to setup the auth page callback, and when I try to visit the page, Laravel screams about the Redirect URL has to be absolute. I don't understand what this means, or what I'm doing wrong.
.env

Additional notes
Up to mention,  that I'm using a VirtualHost for this, as you can see in the redirect_url defined in the .env file.
if I go to http://localhost:8000/youtube/callback, then it does works.. but how can I keep the VirtualHost as redirect_url instead localhost? because otherwise I cannot authorize a localhost URL in the Credentials / Verified Domains in Google Console.

Fixed


peanut gallery feedback

i like the code, comprehensive doc and homespun/non sterile language.
i know i'm feeble minded, but I found the fact that the code snippets and live samples skip specifying a custom service url more confusing than helpful.
no need to link to the JSAPI in a section called Requirements and then again in another called Resources directly below
as configurable as everything else is, it'd make the tour substantially more flexible if developers could just pass field names for sequencing and labeling.  at the very least we'd get to see some nonsensical tours by objectid.

Why come I dun not see'd this until now? Thanks for the feedback! Makes good sense.
I avoided URLs because at the time, ArcGIS Online was broken and couldn't create URLs from directions, yet, without manual hacking. I will revisit.
I thought you could just pass fields into the URL to specify field names. Not sure I follow. Let's discuss.

you are my friend.


Simple 5 step installation guide for Protege 5.6.1

5 step installation guide for Protégé

Click here to find latest release of Protégé.
Choose and download the appropriate files for your operating system (Linux, macOS, Windows, platform independent).
Use the PGP signature contained in the appropriate file with an .asc extension to verify the integrity of the downloaded Protégé version.
Decompress the downloaded .zip or .tar.gz file with tools appropriate for your operating system.
Follow the steps as needed by your operating system to install the Protégé application.
For example, on macOS: drag and drop Protégé.app to the Applications folder and replace any older versions of the software.
You may need to right click Protégé.app and then choose Open from the menu to authorise the programme to run on your machine.
Adjust memory settings if necessary.
Memory settings can now be adjusted in a jvm.conf configuration file that can be located either in the .protege/conf directory under your home directory, or in the conf directory within the application bundle itself.
For example, to set the maximum amount of memory available for Protégé to, say, 12GB, put the following in the jvm.conf file:

max_heap_size=12G

On macOS, you can find the file is here:

/Applications/Protégé.app/Contents/conf/jvm.conf

Edit this part:
# Uncomment the line below to set the maximal heap size to 8G
#max_heap_size=8G

@rays22 THANK you the hardest part was to figure out how to give permission on a Mac..
Go to Preferences -> Security -> General
You need to open the little lock, then click "Mac stopped an application from Running (Protege)" -> Open anyways

If I right click on Protege.app in Applications the first time, then it gives a severe warning about security risks, but there is a choice to Open. I click 'Open' and it does open. Subsequently it opens without any warning or problem.
Shall I put this document to obook/docs)/howto/?

I'm working on the GO editors guide; I have a couple useful screenshots here for allowing it to run: https://wiki.geneontology.org/Protege5_6_setup_for_GO_Eds#Download_and_install_Protege


Chips don't include entities of undisclosed area.

Describe the bug

Chips don't include entities of the undisclosed area in the counter.
Chips don't switch entities of the undisclosed area.

Versions

Mushroom-Strategy: main

chips for switches shows all switches including the hidden ones
Hidden or disabled entities should not be included.

I have hidden undisclosed area and view
Hiding a area or view is unrelated to hiding an entity.


Long followpoint

Description
The followpoint should stretch, not use png resolution.

Justification
For aesthetics and convenience (?)

in

This is issue? I mean, a new cursor trail has been added, but the old model is still inside, although he also moved from the main branch of the droid. And it should already be there, probably they suggested it a year ago, they even suggested some solutions, but no one wants to do this.

What do you mean?


hello 0.1.0+85042307

Created via alr publish with alr 2.0-dev+beta2

Closed using alr 2.0-dev+beta2 with reason: 'automated testing'


"failed to put stub in foreground" on FreeBSD

Happens whenever an external command is executed.

Removing the stub and restarting elvish serves as a workaround.

On BSD, the stub died abnormally on startup. Starting from 6a80c3dcef607292800acebf1d5f742eaa7b3692, elvish monitors the state of the stub and turns off job control when it dies. As a consequence, BSD users now will see a "stub has died" message on startup, and elvish will function without job control.

Also reported to occur on Debian.
Should be some race condition at startup.

4d63220e14bfb624fe133d6aa47384157a685bf7 added logging to the stub package, which unfortunately doesn't seem to work.


Can we use nLog for .Net C++ and QT qml?

Hi!
If you have a question that the logging isn't working, check the troubleshooting guide first before posting.
Almost every case is solveble with the internal log
Are you searching for a layout or layout renderer? You could find them here
Stackoverflow is recommend for questions. You could expect a faster response there (but do include all the info below) and the search is better.
For the stackoverfolow the following is recommend:

Use the tag "nlog"
Include versions of NLog/NLog.AspNetCore etc.
Include the platform (.NET 4.5, ASP.NET Core 3 etc)
Include the current config (xml or code)
Explain current result
Explain desired result
Please post full exception details (message, stacktrace, inner exceptions) in case of an error

Hi
This is Ramesh, I am working for Bio-Rad Laboratories in Hercules, CA.
I am investigating to evaluate a log tool for our new product.
We are using C++ with QT and qml.
Please let me know if the nLog logger supports our product.
If yes, please let me know some pointers to evaluate.
Thanks
Ramesh

From: welcome[bot]<EMAIL_ADDRESS>Sent: Monday, August 17, 2020 10:52 AM
To: NLog/NLog<EMAIL_ADDRESS>Cc: Ramesh Yakkali<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [NLog/NLog] Can we use nLog for .Net C++ and QT qml? (#4089)
Hi! Thanks for opening your first issue here! Please make sure to follow the issue template - so we could help you better!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/NLog/NLog/issues/4089#issuecomment-675023589, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJMYIOSSJDLIYUE3SYNUOV3SBFU7LANCNFSM4QB6UTMA.

Only .net is supported.


doc: Document how to leverages packages with extensions.

Motivation for this change
Clarification of /making more explicit something that might be considered "tribal knowledge", AKA my two cents in making this particular cornerstone more "noob friendly".
Things done

[2021-11-07 21:33:07] 0 x10an14@laptop:~/github/nixpkgs/nixos/doc/manual
-> $ ./md-to-db.sh
~/github/nixpkgs/nixos/doc/manual ~/github/nixpkgs/nixos/doc/manual
~/github/nixpkgs/nixos/doc/manual
[2021-11-07 21:33:37] 0 x10an14@laptop:~/github/nixpkgs/nixos/doc/manual
-> $ nix-build ../../release.nix -A manual.x86_64-linux
these derivations will be built:
  /nix/store/bzncn49iijnin90a9kpx3xjfcpjhyksj-nixos-manual-combined.drv
  /nix/store/9vygy8y6rscgs65r5kj3drq7dg4clwqv-manual-olinkdb.drv
  /nix/store/76vwy7jg9hk0w9wlm5fjsih824axpics-nixos-manual-html.drv
building '/nix/store/bzncn49iijnin90a9kpx3xjfcpjhyksj-nixos-manual-combined.drv'...

manual-combined.xml validates

man-pages-combined.xml validates
building '/nix/store/9vygy8y6rscgs65r5kj3drq7dg4clwqv-manual-olinkdb.drv'...
Writing /nix/store/ckpx4b01s8ilkpljmjch4d19d5c7rf36-manual-olinkdb/manual.db for book(book-nixos-manual)
building '/nix/store/76vwy7jg9hk0w9wlm5fjsih824axpics-nixos-manual-html.drv'...
Writing options.html for appendix(ch-options)
Writing release-notes.html for appendix(ch-release-notes)
Writing index.html for book(book-nixos-manual)
/nix/store/wd2zrqplvm36if080wp1q00d7vwbpi7l-nixos-manual-html

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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 wip"
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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.

I'm open for any suggestions on both structure, location, and wording.
I don't feel confident that I have understood where and how this should be documented.

drive-by comments (solicited on matrix): "Examples included (arbitrarily selected)" can be simplified to "Examples included"; the example should be of you using one of the three listed packages; and "These may be leveraged like so" can be simplified to "you can use them like this".

drive-by comments (solicited on matrix): "Examples included (arbitrarily selected)" can be simplified to "Examples included"; the example should be of you using one of the three listed packages; and "These may be leveraged like so" can be simplified to "you can use them like this".

Ahh, thank you =D! I'm sorry I didn't see this (seemed like conversation just continued on in Matrix anyways)...
For future reference, see here: https://matrix.to/#/!KqkRjyTEzAGRiZFBYT:nixos.org/$f98kL86i0z4ZwtiAmwCuIBz3uwarDFRA7q2fzXS5TKM?via=nixos.org&via=matrix.org&via=tchncs.de

Many thanks to @SuperSandro2000, @Artturin, and @enterprisey in no particular order for help =)
Lots of it given over on Matrix

@Artturin: unneeded

Yes, unnecessary for the example at hand, but the intent of this documentation addition is to allow those who don't know Nix/NixOS that well to understand what can be done where in the configuration.
I'm not that comfortable with a stand-alone minimal-working-example documentation's example that only shows pckage extensions.
Can you guys see no way of helping me realize this documentation in a way that doesn't make it (more) difficult for those who don't quite understand the difference between /etc/nixos/configuration.nix and ~/.config/nixpkgs/config.nix?
Those who could benefit from understanding that it's okay to write environment.systemPackages = []; twice, because Nix is smart enough to combine/concatenate lists, and not overwrite?
Most importantly, those who may wonder why there's so many different ways of writing <package> in environment.systemPackages?
The idea behind adding tree and fzf was to show that its in this list (which they most likely already have encountered) they also can write like above.

I may have misunderstood, but I feel like most of the recent feedback I've gotten has been stylization/preference/minimalism.
All of which are fine, but I'd hope for more dialogue when style/preference/minimalism comes at the cost of the intent of the documentation change/PR.
I wrote this PR after pointing out that this was not obvious in the Matrix chat (referenced above). I couldn't find this  documented in any understandable/obvious matter in any of the documentation that I could find - particularly  for someone new to Nix/NixOS.
I don't feel like we're ensuring this anymore now with all of the feedback as it stands now:

I couldn't find this  documented in any understandable/obvious matter in any of the documentation that I could find - particularly  for someone new to Nix/NixOS.

Is this PR still relevant for you? None of the feedback has been addressed yet, do you think should close it?

@drupol: The PR is no longer relevant to me personally, because now I have learned/absorbed this "tribal knowledge".

Feel free to close it if you think it should be.

@drupol I don't think it should be closed. I will admit to being a bit verbose in my comments/dialogue, but I don't see any response from @Artturin (or anyone else) since November 1.5y ago.
I still think and will argue that making the example small enough to only contain python, is a bad thing: I still think it's good/important that a newcomer understands that (1) this syntax can be used in any of multiple environment.systemPackages declared in their config.
And (2) that it's perfectly fine to install more/other packages alongside the withPackage package list.

Ok fair enough, I understand and I respect your decision. In that case, do you think you could at least fix the merge conflict and I will see if we can push this thing forward?

Take your time, just ping me when it's done.

This LGTM, but I will ask someone (@fricklerhandwerk) from the documentation team to review it.

Cool thanks for your responsiveness!


Colspan for Columngroups after add or remove columns in W2UI Grid

I am using columngroups and columns. Columngroups have specified colspan.
But after adding or removing columns, its alignment changes.
Is there any way to make the colspan dynamic?
Thank you in advance.

Hi Vitmalina,
Could you please help me with this? Its really urgent.
Thank you.

You will most likely need to do pull refresh, grid.refresh() should do. It should work in dynamic nature. If it does not help, please create jsFiddle where I can see the problem.

Please check below jsFiddle:
http://jsfiddle.net/api/post/jquery/2.1/
Add new column "Middle name " in the table,It will get added in column group "contact details" but I want to add it under column group "Name Details".
This is happening because colspan for column group "Name details " is 2.
How can I make this colspan dymanic? so that it will be 3 as I add column middle name to it.
Thank you.

span is a property in columnGroups array. You can change it dynamically. The only known problem is that you cannot have hidden columns, you will need to remove (or add them).

Since no further response, closing the issue.


Rewrite reference processing code in preparation for opening/closing from comment references

This PR does the following:

Rearrange the code used for parsing mentions (@user) and cross-references (#123, etc.) in issues, pull requests, comments and commits.
Provide code to properly parse/skip Markdown code in the contents.

Currently reference processing ignores any Markdown code, so references are parsed even in code blocks and other markup that shouldn't; for example:

	This code `mike @mike` should not count as a mention.

Process all references in one place (issues, PRs, comments, commits; not yet for HTML, but I'm preparing for it).
Fix #8259

Although this PR has value on itself, the goal is to set the base for properly processing closing/reopening keywords in a future PR.
Progress (done)

[x] Create a markdown stripper class
[x] Move reference detection from models into modules/references
[x] Detection of closing/reopening keywords in the new references module
[x] Generate content references to facilitate HTML markup insertion
[x] Make markup/html.go use the references module so all reference parsing resides in one place.
[x] Make markup/html.go aware of closes/fixes keywords to render an indication for that.
[x] Support highlight of action keywords like "closes", "reopens", etc.
[x] Should fix --- #7810 but I'm unable to reproduce

In another, future PR

[ ] Close/reopen issues from PR comments.

There's something I don't know how to deal with:
+ make revive
revive -config .revive.toml -exclude=./vendor/... ./... || exit 1
modules/markup/mdstripper.go:71:28: method BlockHtml should be BlockHTML
modules/markup/mdstripper.go:182:28: method RawHtmlTag should be RawHTMLTag
make: *** [Makefile:139: revive] Error 1

I'm just implementing an interface from an external vendor; I have no saying on the name of the method. For golangci-lint, I've simple used the //nolint flag:
func (r *MarkdownStripper) BlockHtml(out *bytes.Buffer, text []byte) { //nolint
    [...]
}

But for the old lint, I don't know how to bypass the test.
Any ideas?

Codecov Report

Merging #8261 into master will increase coverage by 0.06%.
The diff coverage is 69.09%.

@@            Coverage Diff             @@
##           master    #8261      +/-   ##
==========================================
+ Coverage   41.56%   41.63%   +0.06%
==========================================
  Files         489      491       +2
  Lines       65246    65378     +132
==========================================
+ Hits        27122    27220      +98
- Misses      34635    34675      +40
+ Partials     3489     3483       -6

Impacted Files
Coverage Δ

modules/markup/html.go
85.54% <ø> (+0.38%)
:arrow_up:

models/issue_mail.go
53.6% <100%> (ø)
:arrow_up:

models/issue_comment.go
50.46% <100%> (ø)
:arrow_up:

modules/markup/mdstripper.go
53.33% <53.33%> (ø)

models/action.go
56.49% <75.86%> (+2.23%)
:arrow_up:

modules/references/references.go
76.69% <76.69%> (ø)

models/issue_xref.go
56.35% <86.11%> (+2.54%)
:arrow_up:

models/unit.go
62.16% <0%> (-5.41%)
:arrow_down:

modules/log/event.go
64.61% <0%> (-1.03%)
:arrow_down:

models/repo_list.go
73.17% <0%> (-0.98%)
:arrow_down:

... and 7 more

Continue to review full report at Codecov.

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

I know it looks like very little improvement, but getting here was a very though ride:

The idea is that some day (not with this PR!) client javascript will be able to produce:

This is the currently generated HTML:
Closes: #1

So a client-side script can look for links next to "issue-keyword" spans and add a mouseover to check if they are valid issues and display the message.

Ready for your review.

@guillep2k It's a big change. :)


 This started to happen last week when the parsing method changed from indexOf to split.
Expected behavior
Given the input FOO=bar=baz. the value of FOO should be bar=baz.
Observed behavior
The value is bar.
Action YAML
- name: Deploy
  uses<EMAIL_ADDRESS>  with:
    env_vars: CONN=DefaultEndpointsProtocol=https;AccountName=foo;AccountKey=bar

Additional information
If split is preferred over indexOf, the following code should have the desired effect:
let [k, v] = pair.split((/=(.*)/), 2);

Thanks! I think that should solve it for our use case. It looks like it may remove consecutive and trailing equals signs though. This could cause problems with passwords for example, or base64 padding (if there are any decoders that care about that).
The most intuitive behavior to me would be to replicate the gcloud tool, which leaves everything after the first equals sign intact.

@coolstuff-mikael good point - updated the PR to use indexOf.


Implement desktop notifications.

This is another attempt to add desktop notifications based on work of @inikulin in #25.
The main feedback in original PR was about notification content and action we perform on notification click.
With minimal changes, I was able to make it look like so:

Though I'd like to purpose a bit different style:

With all that there are still some questions left:

what do we show if there are more than 1 notification?

list notification?

just say "there are N notifications for you" ? or "there is foo and N-1 other notifications for you"?

we need to somehow track last notification timestamp, so we don't show same thing several times.

on > 1 notification, where should click bring you?

/cc @sindresorhus

Idea: maybe if there > 1 notifications, we can add button like Next so user can cycle through them and Mark all as read to mark all at once?

Though I'd like to purpose a bit different style:

Yeah. I like that style better.

what do we show if there are more than 1 notification?

Shouldn't there be a Chrome notification for each GitHub notification?

Shouldn't there be a Chrome notification for each GitHub notification?

I thought that could be quite annoying for active github users.

@sindresorhus ATM the only problem is to convert GitHub API url to the actual issue url.

For now, I found 2 ways of getting issue url of notification:
1 - We can get if from notification info, but it is hackish and not future proof:
[apiRes.repository.html_url].concat(apiRes.subject.url.split('/').slice(-2)).join('/')
// -> "https://github.com/octocat/Hello-World/issues/123"

2 - We can make a request to GitHub API to get this info:
curl 'https://api.github.com/repos/octokit/octokit.rb/issues/123'
//     ^- this is same as apiRes.subject.url

gives:
{
  "url": "https://api.github.com/repos/octokit/octokit.rb/issues/123",
  "repository_url": "https://api.github.com/repos/octokit/octokit.rb",
  "labels_url": "https://api.github.com/repos/octokit/octokit.rb/issues/123/labels{/name}",
  "comments_url": "https://api.github.com/repos/octokit/octokit.rb/issues/123/comments",
  "events_url": "https://api.github.com/repos/octokit/octokit.rb/issues/123/events",
  "html_url": "https://github.com/octokit/octokit.rb/pull/123",
  "id": 6130680,
  "number": 123,
  "title": "Implement repo assignees",
  // ...
}

and we need html_url from here.
I think it is ok to make this request when user clicks notification, so we send request lazily when needed.

@sindresorhus this is ready for review.

Hello @YuriSolovyov, I'm testing your fork: feedback next week! :-)

Cool!
/cc @fstoerkle @gcochard

@sindresorhus addressed feedback.
I noticed that even though we are setting optional_permissions, this flag is never checked, should I remove it or make it work somehow?

@YuriSolovyov it's a trick to avoid popin each time user refuse the new permission (true and false are !== undefined).

@Narno I guess I should add same thing for notifications?

Yes, but if you find a class solution, that would be cool! ;-)

@Narno I don't quite get how/if user can grant this permission later on in this case...

@YuriSolovyov yep, the only way to do that for the moment is to re-install the extension. I don't know if it's a real problem or not :-)

@Narno
For now I just factored out this to Promise-based API.
We might want to add a setting like "Allow querying my tabs" or something

I'm also thinking about making use of reason field - we can have a map of messages for every type of reason field, so we can add this info to notification like
<message>
<repo>
New comment | You were mentioned | You were assigned | You are subsribed

... and so on
cc @Narno

Now I'm also fixing #70 here

Can anyone try
curl -v -H "Authorization: token <token>" "https://api.github.com/notifications?per_page=1"

and tell me if it returns non-empty ETag header?

@YuriSolovyov curl output: http://pastebin.com/8a3iU4g2

@Matsv please replace <token> with real token, you can generate one in settings, make sure you added notifications permission

@YuriSolovyov Ok, one sec

@YuriSolovyov http://pastebin.com/dWJYnMAM with notications from the public-apis repo

Thanks, @Matsv, seems like I'm not the only one.
Ok, I sent a letter to Github, maybe I missed something.

linter is complaining about snake_case, but since it comes from rails app, it is kind of normal and we can't change it, so we have (at least) 2 options:

Somehow disable the rule atomically, just for 1 function
Replace obj with Map:

// object:
const reasons = {
  subscribed:   'You are watching the repository',
  manual:       'You are subscribed to this thread',
  author:       'You created this thread',
  comment:      'New comment',
  mention:      'You were mentioned',
  team_mention: 'Your team was mentioned',
  state_change: 'Thread status changed',
  assign:       'You were assigned to the issue',
  default:      ''
};

// Map:
const reasons = new Map([
  ['subscribed',   'You are watching the repository'  ]
  ['manual',       'You are subscribed to this thread']
  ['author',       'You created this thread',         ]
  ['comment',      'New comment'                      ]
  ['mention',      'You were mentioned'               ]
  ['team_mention', 'Your team was mentioned'          ]
  ['state_change', 'Thread status changed'            ]
  ['assign',       'You were assigned to the issue'   ]
  ['default':      ''                                 ]
]);

Any ideas?
Also, I'd like to get some feedback about wording of these messages, maybe someone has better ideas, or it is fine for now?

You can always disable it with /* eslint-disable snake_case / before and then / eslint-enable snake_case */ at the end of your handling code.

This is ready for another round of review.

@gcochard you are right about stub changes, works ok with chrome-stub, I just remember there were some problems with it (can't remember what), anyway, I switched back to chrome-stub.

Btw, I don't think we need to stub XMLHttpRequest anymore...

Btw, there is a bit weird implementation detail about reason field in API response that can make things a bit confusing:

Note that the reason is modified on a per-thread basis, and can change, if the reason on a later notification is different.

For example, if you are the author of an issue, subsequent notifications on that issue will have a reason of author. If you're then mentioned on the same issue, the notifications you fetch thereafter will have a reason of mention. The reason remains as mention, regardless of whether you're ever mentioned again.

I think that also applies to the emails github sends out and users should expect that behavior. Once github fixes it, we get the fix for free.
If you do encounter any other issues with chrome-stub, let me know by filling an issue on it. I'm the author and I wrote it primarily for testing this extension so I have an interest in making it work for all of our use cases.

LGTM

I'd like to use it for a couple more days just to be sure

Hello @YuriSolovyov, I'm (re)testing your branch since yesterday: notifications never appears, may be because the extension don't asks me for notifications permissions.

@Narno anything interesting in console?

@Narno can you check following things:

does API response contain Last-Modified header? if not there will be null in lastModifed key in local storage (something is very weird here: I had this header on work, but now it is absent on home PC)
notifications might not some just after upgrade, but at least on 2nd check they should come.
I'm not sure, but it seems like on linux notifications only appear when browser is focused, might be distro-dependent.

nah, appear just fine even without focus

ping @Narno :)

Status update: I was pretty busy over past couple weeks, but I'm going to back to it this weekend, there is indeed a bug here and I'm going to fix it.

@Narno I updated PR, should be better now, I'll use for couple more days and post final go/non-go later.

Nice, I will try the new version this night!

OK, now Chrome asks me the right thing when I checks the "Show desktop notifications" option. Good new!

OK, after several tests, in different contexts, desktop notifications works really fine! \o/

@Narno can you recall if transition from version without notifications to this PR was ok?
Anything you did by hand that should've happen automatically?
Or switched just fine?

Hello @YuriSolovyov, what I have tested:

installing the PR extension in // of the current extension (with the same token): OK
uninstalling the current extension then installing the PR extension (with the same token) : OK
installing the PR extension on a fresh Chrome installation (with a new token) : OK

Done on my laptop computer (Mac OS X Yosemite) and on my desktop computer (Windows Seven).

@sindresorhus I think this is ready.

@sindresorhus now it's your turn :smile:

:+1:

Awesome! Superb work @YuriSolovyov :)
Could you add something to the readme about the desktop notification feature?
I've published a new version. Should be live within an hour.

@sindresorhus sure, I'll add a couple words.


[ISSUE-3064] [WIP] [Improve] Improve streampark-console system package module mapper name base on [3.1 Naming Style]

What changes were proposed in this pull request
Issue Number: see #3064

Brief change log

Improve streampark-console system package module mapper name base on [3.1 Naming Style]

Just query method
Just in mapper

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.
Yes (or)
This change is already covered by existing tests, such as (please describe tests).
No (or)
This change added tests and can be verified as follows:

Does this pull request potentially affect one of the following parts

Dependencies (does it add or upgrade a dependency): No

PTAL~


🍒 [5.5][Concurrent] Warnings/errors should use "sendable" not "concurrent"

Explanation: Some warnings were still using the "non-concurrent" wording about functions and closures. This was wording from before Sendable was a thing. This PR fixes the wording.
Risk: Low.
Reviewed By: @DougGregor
Testing: PR testing and CI on main. Tests adjusted to check for new wording.
Radar: rdar://83416205
Original PR: https://github.com/apple/swift/pull/39406 [Concurrent] Warnings/errors should use "sendable" not "concurrent"

@swift-ci please test

Windows:

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:12 (project)

-- Configuring incomplete, errors occurred!

@swift-ci please nominate

@swift-ci please test Windows

@swift-ci please nominate


Sm/qa pr 418

What does this PR do?
open a vf page by name
What issues does this PR fix or reference?
@W-10514651@
https://github.com/forcedotcom/cli/issues/1386

QA Notes:
✅ opens a page using the .page file
👎🏻  sends user to odd place with message The name can only contain underscores and alphanumeric characters. It must begin with a letter and be unique, and must not include spaces, end with an underscore, or contain two consecutive underscores. when opening the meta-xml
👎🏻 opening a non-existent page (ex: mistyping a filename) sends user to a login page

✅ opens vf meta files
✅ throws when file doesn't exist


In data generation documentation, show the simulator run function

Private methods are now visibile in the API doc


Lw 6

📝 Changed routes:

/launch-week-6

(view full size)

Commit d0566052db431eebbfa8001777a8a91d29c2e8e8 (https://madewithsupabase-j6v5973ad-zernonia.vercel.app).

juicy


Like/Dislike doesn’t work when listening to a radio

Steps to reproduce:
Launch GPMDP and start a radio station (ex: Feel Good Dance/EDM)
Launch VS Code and like the song.
Expected results:
The song is liked
Additional information
Seems to work when playing from a playlist, just not with a single song/radio station.

Doesn't seem to be working for me with playlists either. I can play/pause and skip, but the rating commands seem to do nothing.

After a long delay, this should be fixed in 5141ac90c08443d4adac0c0dfe2385f254ea947d


Add positionStrategy prop

Closes #840.

Any idea why this check started failing? Thought it was an issue with my environment, no idea why it suddenly started happening.
Adding and setting up eslint-import-resolver-typescript solves it. If that's ok, I can push the changes.

Yeah, this was weird, I'm checking your locally. I had to stop because of my work, but I'll check as soon as possible to confirm.
Also, thank you so much for the reports and for the PRs!
Quick question: why you removed the import type (type) from import? - I'm just asking because I don't know the side-effect of using or not using type when importing decorators.
And I'm already building the types, how can I improve this? The types are working on a study project that I have.

why you removed the import type (type) from import?

Oh, this is definitely the issue. Wasn't intentional, I'll revert it.
Basically import type gets completely erased after transpilation, so it doesn't show up at runtime. The issue with eslint is probably because I was trying to just import a d.ts file, but not exactly sure if that's the whole reason.

Oh, this was the issue hahaha
Thanks @gabrieljablonski, merged 😄


Responsive categories card

Describe the bug/feature
Categories card are not responsive
@rockingrohit9639  can i work on this issue

Describe the bug/feature
Categories card are not responsive
@rockingrohit9639 can i work on this issue

those are already responsive. you have to scroll to view more cards.

Ok


docs: add how to use composer

Describe the feature
withNuxt and createConfigForNuxt can be customized using FlatConfigComposer.
(Thanks for making FlatConfigComposer 💚)
However, there is little mention of FlatConfigComposer in the document.
Therefore, it is expected that the following issues and discussions will increase.
・How should I extend Config?
・I want to set xyz as before. What should I do in that case?
・How do I use . prepend?
I think it would be better to include the customization method in the documentation to reduce/avoid responding to this question.
My Opinion
add  Guide > Customizing the Config section and add this page.
This page is write post simple instructions on how to use .append, .override and etc.
In addition, any questions that may be asked in the future (e.g., How should I coexist with Prettier?) will be listed separately in the FAQ.
Additional information

[X] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide.
[X] Check existing discussions and issues.

Hi @antfu , I would appreciate it if you could comment when you have time. 🙏
Thanks for making FlatConfigComposer 💚

We explained here, isn't it? https://eslint.nuxt.com/packages/module#config-customizations
If you think there are any detailed docs needed, they should also go to https://github.com/antfu/eslint-flat-config-utils#composer


Fix PR-comment GitHub Action by using our forked version.

I had to remove a problematic type annotation in the docstring of EnlyzeClient.get_appliances() as well, because that has led to a failure building the docs. For consistency, I've also removed a similar annotation in the same class. Both annotation are not strictly necessary, so this shouldn't be a big deal.

For some context: CI seems to have broken by itself since the last PR some time ago. When I opened https://github.com/enlyze/enlyze-python/pull/35 I found CI failing for reasons unrelated to my actual PR. This PR here fixes all current CI issues.


Authenticate with OAuth

Use OAuth to authenticate with the YNAB API rather than a personal access token to reduce the risk to end users of non-time-bounded API access.

May be able to use this to accomplish this:
https://github.com/int128/oauth2cli


Add type restrictions and docs

Adds type restrictions and some documentation
Minor changes in exception handling

The results from this PR look fine to me, but if @bielim can validate that they are good, I'm fine with merging this!

bors r+

bors r+

Closing since this has gotten too stale


How can i get transactions by block height ?

As the title, what method can I use to get its transaction information by block height ？

It would be great if the implementation of the "https://cardano-mainnet.blockfrost.io/api/v0/blocks/latest/txs " interface could be added to the SDK。

Still not implemented.


Fix sphinx_rtd_theme theme override in embed js

The last refactor wasn't aware of the fact that the js Build class was modifying the actual config object in place. We are fixing this by using a getter method were the value is needed and are not mutating the original config object in place.
Fixes #1542.

I've tested this locally with the mkdocs project (using readthedocs theme for a mkdocs project) and django-floppyforms which is using the sphinx_rtd_theme with Sphinx. I also tested requests project which is Sphinx based an not using sphinx_rtd_theme.


Vine's Expository Dictionary in KJV and Strong Bible

Hi, erykjj, whenever we tap any Strongs Number in Bible KJV, I'd like the Vine's Expository Dictionary to show with the Strongs Dictionary. It would be nice to add.

Also Hebrew in Vine's Expository Dictionary is nice to add too.

Hi @XcXblythe26 ,
Thanks for your suggestions.
Vine's is only a Greek dictionary (of "New Testament" words). It is already cross-linked in Strong's (again, Greek terms only). You'll find a link icon at the end of the definition, which will take you to the corresponding Vine's entry/entries.

I didn't see that, I thought that was just a design. 🤭

Does Vine's has Hebrew?

Does Vine's has Hebrew?

No, it doesn't. There may be other more extensive Hebrew dictionaries out there, but, for now, I will not be adding any. I chose Strong's and Vine's because they are quite often used as authoritative sources.

Thank you!

I

Hello, brother, I saw this on Insight on here:
“Of the noun a·gaʹpe and the verb a·ga·paʹo, Vine’s Expository Dictionary of Old and New Testament Words says: “Love can be known only from the actions it prompts. God’s love is seen in the gift of His Son, I John 4:9, 10. But obviously this is not the love of complacency, or affection, that is, it was not drawn out by any excellency in its objects, Rom. 5:8. It was an exercise of the Divine will in deliberate choice, made without assignable cause save that which lies in the nature of God Himself, cp. Deut. 7:7, 8.”—1981, Vol. 3, p. 21”
Could there be a Hebrew version in another book, couldn't there?

Find me a freely-downloadable source online (XML, JSON or similar) and I'll have a look.

Let me see, brother, I will came back after I see one.
😗🎵 looking for codes in websites
Also, I know that this conversation will be a kind of confidential. Can I contact you in another way?

Currently find a book, I am eager to make an XML file for this, don't worry I use AI to do this work. Can you give me specific format of tags to make an XML.
Example for this: “

Currently find a book, I am eager to make an XML file for this, don't worry I use AI to do this work. Can you give me specific format of tags to make an XML.
Example for this:
“ABOMINATION - (verb) ta'ab (תעב , 8581), “to abhor, treat as abhorrent, cause to be an abomination, act abominably.” This verb occurs 21 times, and the first occurrence is in Deut. 7:26: “Neither shalt thou bring an abomination into thine house.…””

There are only 455 pages of Hebrew dictionary on this book—I found. Also, the Greek is the same on VED. I see also the usage of listing like (A-1, B-1, 1, etc.)

I'll get back to you later.

Okay, I'll wait, brother.


Adopt Microsoft.Data.SqlClient instead of System.Data.SqlClient

Summary of the new feature / enhancement
Maybe we should consider using Microsoft.Data.SqlClient instead of System.Data.SqlClient

https://devblogs.microsoft.com/dotnet/introducing-the-new-microsoftdatasqlclient/

Proposed technical implementation details (optional)
No response

We can not remove System.Data.SqlClient, it will be a huge breaking change. Nevertheless we could add Microsoft.Data.SqlClient.
System.Data.SqlClient.dll is ~1Mb. I guess Microsoft.Data.SqlClient too. We need to see important scenarios to add so large dll in distribution.

Maybe move System.Data.SqlClient to WinCompat and now start migration to Microsoft.Data.SqlClient since we are on new dev cycle (vNext is not LTS).

We would absolutely love to have Microsoft.Data.SqlClient available in Powershell - I'm part of a platform team and want to get rid of SQL Auth. completely in our infrastructure. Not having Microsoft.Data.SqlClient available to us in Powershell is currently blocking us from doing that as we need to authenticate a Powershell process using IAM in our CD pipelines and execute SQL scripts prior to doing database migrations.
Right now, we're forced to stick with SQL Auth which is less than satisfying for us for various reasons - i.e. company compliance policies, just to name one of them

I would love to have Microsoft.Data.SqlClient in Powershell but I don't think that adding this .dll to code is the way to go. It   does not fit with the open-source philosophy since this addition would only bring benefit to SQL Server, a Microsoft product. Since PowerShell was open sourced, we can see a lot of effort to make it more compatible with other non-Microsoft platforms. Like Python, or any other open-source programming language, we have to use libraries for this job and PowerShell has the modules.  The preview v22 version of SqlServer module switches from System.Data.SqlClient to Microsoft.Data.SqlClient. IMHO, that's the way to go.

I would love to have Microsoft.Data.SqlClient in PowerShell but I don't think that adding this .dll to code is the way to go. It would not fit with the open-source philosophy since this addition would only bring benefit to SQL Server, a Microsoft product. Since PowerShell was open sourced, we can see a lot of effort to make it more compatible with other non-Microsoft platforms. Like Python, or any other open-source programming language, we have to use libraries for this job and PowerShell has the modules. The preview v22 version of SqlServer module switches from System.Data.SqlClient to Microsoft.Data.SqlClient and we should use this module for new script developments for SQL Server.

I agree with this sentiment.  The new Assembly is about 1.5MiB on Windows.  These were included to not break existing scenarios.  If the the scenario is new, we should move to a module based solution, and it looks like one already exists.
From a security point of view, eventually, I believe we will eventually have to drop System.Data.SqlClient because of certain issues which cannot be fixe (I'm not predicting what they will be.)

It is a time to consider the breaking change for 7.4. We have a time to get feedback before release.
/cc @SteveL-MSFT @daxian-dbw

@SteveL-MSFT @iSazonov this came up in the PS discord today and already has a breaking issue if someone tries to load SqlGuidCaster

Can you clarify this comment about how it is a huge breaking change? Other than maybe modules relying on the assembly in-built? I didn't see any direct references to the library in PowerShell itself.
https://github.com/PowerShell/PowerShell/issues/16156#issuecomment-927279865

Such issues should be easier to deal with if C# / .NET NuGet packages were natively supported in PowerShell - #6724
Perhaps it's an opportunity to reopen both issues?

@marcinsmialek in this case it wouldn't, because it is natively bundled in PS you'd still have the assembly conflict until it is removed from being bundled in PS.

I have done some investigation packaging the new Microsoft.Data.SqlClient in its own AssemblyLoadContext. It appears to work well as it's own PSGallery module.
I have a proof of concept on github and PSGallery


GFortran?

How involved is it to add a new compiler? It would be great to have GFortran added, many/most flags are shared with gcc.

Pretty easy. The main problem is building all the compilers, which currently I do manually using scripts in the sister project mattgodbolt/compiler-explorer-image. They currently don't build fortran, but I can add it, then update the nodes etc.
Syntax colouring would need to be added too. That's fairly easy to do, but takes time.
This might all work better after #277 is implemented.

I'd love to see this implemented. I would probably be of more use updating mattgodbolt/compiler-explorer-image than I would implementing syntax highlighting...

Amazingly I'm already building the fortran compilers (I forgot I enabled them already...)
ubuntu@ip-172-30-1-211:~$ /opt/compiler-explorer/gcc-7.2.0/bin/gfortran --version
GNU Fortran (GCC-Explorer-Build) 7.2.0

so, this is a step closer than I thought!

Great! Then there's probably not much for me to help with. Thanks!
On Fri, Oct 6, 2017 at 12:11 PM Matt Godbolt<EMAIL_ADDRESS>wrote:

Amazingly I'm already building the fortran compilers (I forgot I enabled
them already...)
ubuntu@ip-172-30-1-211:~$ /opt/compiler-explorer/gcc-7.2.0/bin/gfortran --version
GNU Fortran (GCC-Explorer-Build) 7.2.0
so, this is a step closer than I thought!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mattgodbolt/compiler-explorer/issues/549#issuecomment-334800211,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAREPJJOq3NhhxRjPmdzff2NI8sxk5eMks5splEVgaJpZM4Pts6c
.

@zbeekman  Indeed, one can currently invoke the gfortran frontend by -x f95 for those GCC versions for which gfortran is built. It is a nice feature.

I'd forgotten about -x in this way :). It will still be great to build
Fortran into more of the compilers and to have it officially supported with
syntax highlighting etc too, so will leave this bug open.
On Fri, 13 Oct 2017, 03:51 LadaF<EMAIL_ADDRESS>wrote:

@zbeekman https://github.com/zbeekman Indeed, one can currently invoke
the gfortran frontend by -x f95 for those GCC versions for which gfortran
is built. It is a nice feature.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mattgodbolt/compiler-explorer/issues/549#issuecomment-336391564,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmsdcZezl5o-LOk6aNN3nuQIbmZM7AIks5srySNgaJpZM4Pts6c
.

Since I was also interested in Fortran support for this amazing tool, I've started experimenting with it.
https://github.com/FZJ-JSC/compiler-explorer
I believe it's very basic for now, but it seems to work at least to some level. The build process for compilers is not touched, it rather relies on compilers it finds installed on the machine.
Initial bits on syntax highlighting are added as well, but I don't yet understand how it's supposed to work. I am aware of missing parts here and would also love to see indenting, but it's a start.
Not sure when I'll find the time to extend it further. If it reaches a more finished state I'll make a pull request (unless you feel a PR is worthwhile already).


Importing some APIs throws a 'No such property' error

Plugin version - 1.0.3
Ready!API is not able to import some of the APIs. The following exception is given.
Failed to read API description for[TestAPI] - [No such property: payload for class: io.swagger.models.properties.ObjectProperty] Please contact WSO2 support for assistance

The above was due to a dependency issues. We have fixed this in Plugin version 1.0.4


Compile errors

When cloning the repo or doing the cargo install snoopy, I'm getting some errors about a compilation that happens on the following lines.
https://github.com/kanishkarj/snoopy/blob/master/src/args/capture.rs#L122-L124
https://github.com/kanishkarj/snoopy/blob/master/src/args/capture.rs#L128-L139
It's complaining about Capture<Inactive> not having those methods.
EDIT:
The program is unable to compile on Windows due to missing wpcap.lib.
wpcap has been deprecated and has been replaced with npcap.
A nice library that selects the best library for the packet sniffing, that you could use, is this one: rawsock.

Hi @MGlolenstine , thanks for the suggestion. I shall fix that up right away!


Stub response passed to prepareStream’s callback to avoid a TypeError

Attempting to fix https://github.com/dgreif/ring-alarm/issues/35#issuecomment-504610023
Disclaimer: I haven’t tested this locally, because I don’t have a Ring camera set up.
I inspected the call stack provided by @ifeign, and the cause of the TypeError appears to be the attempt to read response.address.type on line 658 in HAP-NodeJS’ StreamController.
Because ring-alarm-homebrige currently passes an Error as response, and Error does not have a address property, attempting to read type from the non-existent address property results in Cannot read property 'type' of undefined.

Fixed issues that showed up when I ran npm test, amended the commit and force-pushed. When I have tests/lint rules passing, I’m planning to publish this as homebridge-ring-alarm-smockle for ease of testing.

I’m planning to publish this as homebridge-ring-alarm-smockle for ease of testing.
Done: https://www.npmjs.com/package/homebridge-ring-alarm-smockle

Just tested your branch @smockle, this seems to fix the error I was experiencing. Nothing appears in the logs when I select the camera and Homebridge does not crash. Thank you!

@dgreif implemented this differently in https://github.com/dgreif/ring-alarm/commit/3fe39cd8106da178b8003bc55b50141c00e07fe7.
This PR passes a stubbed object to the callback function; that commit omits the call to the callback (replacing it with the comment: // intentionally ignore the request.  This prevents "No Response" overlay for about 20s). Both approaches should avoid the TypeError.
@dgreif feel free to close this PR. Thanks for your work on this! 😊

Just tested your branch @smockle, this seems to fix the error I was experiencing. Nothing appears in the logs when I select the camera and Homebridge does not crash. Thank you!

Also, @ifeign: thank you for taking the time to test this PR and for sharing the results. 😊


ETIMEDOUT Error on First Move of Awning Each Morning

We have just started using the plug-in with our exterior Somfy awning, and generally have no issues. However, after adding some triggered automation the other day, we have started seeing that Homebridge crashes the first time automation triggers the awning to move in the morning. This is the third day in a row where we have seen this same error. Once Homebridge reboots, any new automation triggers or manual adjustments throughout the day call for the awning to move with no issues.
If it helps, I have the Somfy myLink device and the Raspbery Pi running Homebridge each assigned with static IP addresses through my router. I’ve run the normal update scripts and am running Homebridge v1.2.5
[05/02/2021, 08:38:35] [Somfy myLink] Setting position of target CC1103C0.1 (Awning) from 0% to 100%.
[05/02/2021, 08:38:35] Error: write ETIMEDOUT
    at afterWriteDispatched (internal/stream_base_commons.js:156:25)
    at writeGeneric (internal/stream_base_commons.js:147:3)
    at Socket._writeGeneric (net.js:785:11)
    at Socket._write (net.js:797:8)
    at writeOrBuffer (internal/streams/writable.js:358:12)
    at Socket.Writable.write (internal/streams/writable.js:303:10)
    at Socket.wrapper.<computed> [as write] (/usr/local/lib/node_modules/homebridge-mylink/node_modules/socket-pool/lib/utils.js:12:36)
    at /usr/local/lib/node_modules/homebridge-mylink/node_modules/somfy-synergy/src/SomfySynergy.js:33:18
    at new Promise (<anonymous>)
    at /usr/local/lib/node_modules/homebridge-mylink/node_modules/somfy-synergy/src/SomfySynergy.js:31:9
[05/02/2021, 08:38:35] Got SIGTERM, shutting down Homebridge...
[05/02/2021, 08:38:35] [Somfy myLink] Encountered an error setting target position of target CC1103C0.1 (Awning): write ETIMEDOUT
[05/02/2021, 08:38:40] [HB Supervisor] Homebridge Process Ended. Code: 143, Signal: null
[05/02/2021, 08:38:45] [HB Supervisor] Restarting Homebridge...

I’ve had the same issue as well - the myLink may be defective....
Sent from my iPhone

On Feb 5, 2021, at 12:11 PM, mwhite2003<EMAIL_ADDRESS>wrote:

We have just started using the plug-in with our exterior Somfy awning, and generally have no issues. However, after adding some triggered automation the other day, we have started seeing that Homebridge crashes the first time automation triggers the awning to move in the morning. This is the third day in a row where we have seen this same error. Once Homebridge reboots, any new automation triggers or manual adjustments throughout the day call for the awning to move with no issues.
If it helps, I have the Somfy myLink device and the Raspbery Pi running Homebridge each assigned with static IP addresses through my router. I’ve run the normal update scripts and am running Homebridge v1.2.5
[05/02/2021, 08:38:35] [Somfy myLink] Setting position of target CC1103C0.1 (Awning) from 0% to 100%.
[05/02/2021, 08:38:35] Error: write ETIMEDOUT
at afterWriteDispatched (internal/stream_base_commons.js:156:25)
at writeGeneric (internal/stream_base_commons.js:147:3)
at Socket._writeGeneric (net.js:785:11)
at Socket._write (net.js:797:8)
at writeOrBuffer (internal/streams/writable.js:358:12)
at Socket.Writable.write (internal/streams/writable.js:303:10)
at Socket.wrapper. [as write] (/usr/local/lib/node_modules/homebridge-mylink/node_modules/socket-pool/lib/utils.js:12:36)
at /usr/local/lib/node_modules/homebridge-mylink/node_modules/somfy-synergy/src/SomfySynergy.js:33:18
at new Promise ()
at /usr/local/lib/node_modules/homebridge-mylink/node_modules/somfy-synergy/src/SomfySynergy.js:31:9
[05/02/2021, 08:38:35] Got SIGTERM, shutting down Homebridge...
[05/02/2021, 08:38:35] [Somfy myLink] Encountered an error setting target position of target CC1103C0.1 (Awning): write ETIMEDOUT
[05/02/2021, 08:38:40] [HB Supervisor] Homebridge Process Ended. Code: 143, Signal: null
[05/02/2021, 08:38:45] [HB Supervisor] Restarting Homebridge...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Sorry for the long wait, but I finally have an update out that should hopefully resolve a lot of the connectivity issues.   I'll be continuing to tweak it in the coming weeks, but v2.0.0 should hopefully be more reliable.  It also does automatic discovery of channels!  Note, there are some breaking changes (see the CHANGELOG).  Let me know how it works for you!


docs(addon-editor): Editor fix max-height-description (API-page)

PR Checklist
Please check if your PR fulfills the following requirements:

[X] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactoring
[ ] Code style update
[ ] Build or CI related changes
[X] Documentation content changes

What is the current behavior?
Closes #2677
What is the new behavior?
Uses correct hint for maxheight parameter
Does this PR introduce a breaking change?

[ ] Yes
[X] No

Cheers for suggesting, apologies for not clarifying what's required from me earlier! Updated the branch.

@dublinsubway
E2E tests / addon-editor (pull_request)-job failed(
Possibly, it happens bacause of this line:
https://github.com/Tinkoff/taiga-ui/blob/4b971457a4111497684a9123964d7a8eaf4da91d/projects/demo-integrations/cypress/support/editor/helpers.ts#L17


Cannot use TLS on single core servers

Sample config:
config := &atreugo.Config{
	Host:     "localhost",
	Port: 8445,
	TLSEnable: true,
	LogLevel: logger.DEBUG,
	CertFile: "./cert.pem",
	CertKey:  "./privkey.pem",
}

On single core server answer is:
2019/07/27 06:24:03 - atreugo - INFO - Listening on: http://<IP_ADDRESS>:80/

On >1 core server answer is:
2019/07/27 07:05:04 - atreugo - INFO - Updating address config with the new listener address
2019/07/27 07:05:04 - atreugo - INFO - Listening on: https://<IP_ADDRESS>:8445/

I guess this feature is somehow broken with listener_unix.go, where you rewriting existing config with default one, but if we made runtime.NumCPU() > 0 absolutely nothing changes.

Everything magically changes when we start to use more than one core. I really don't know where problem is.

The message 2019/07/27 07:05:04 - atreugo - INFO - Updating address config with the new listener address is showed when you run atreugo with your own listener myserver.Serve(myNetListener) https://github.com/savsgio/atreugo/blob/master/atreugo.go#L96
not with myserver.ListenAndServe(), so it just update the configuration with your listener address. It's independent to use TLS or not.
And the  listener_unix.go check if you have a multicore system and try to use reuseport to increase performance.
So, could you share more code, please? How did you run in a single core?

I run on single core just by using VPS with 1-core CPU. For some reason this code forks fine on 2-VCPU server, but serves only HTTP on 1-VCPU server (ignoring TLSEnable option).

VCPU - Virtual Central Processor Unit

The source code can be just the same as your example server, but with config that i provided in first message.
Sample code:
func main() {
	config := &atreugo.Config{
	Host:     "localhost",
	Port:      8445,
	TLSEnable: true,
	LogLevel:  logger.DEBUG,
	CertFile:  "./cert.pem",
	CertKey:   "./privkey.pem",
        }
	server := atreugo.New(config)

	server.Path("GET", "/", func(ctx *atreugo.RequestCtx) error {
		return ctx.HTTPResponse("
Atreugo
")
	})

	err := server.ListenAndServe()
	if err != nil {
		panic(err)
	}
}

Hope it helps somehow.

Did you get some error?? Or just ignore TLS configuration and run server?
I will try to reproduce this case anyway.

Did you get some error?? Or just ignore TLS configuration and run server?
I will try to reproduce this case anyway.

No errors at all, it just ignores config with TLS and serving default HTTP

Tested it on different server with 1-core and can't reproduce this bug anymore... I guess this is some kind of heisenbug? :D
Here's my test build that wont work properly on my old 1VCPU build: https://cdn.mysh.dev/bug/reference.zip
And github mirror here: reference.zip

I think so you have done the request with https://... prefix in the browser, right?
I have an VPS in with 1 core (1 VCPU), and your example server runs TLS without problems.
It's very strange, Could you fix it?

Nope. Right now i use another server and can't reproduce this bug anymore. So i guess i need to close this issue due i can't reproduce bug anymore (for this time)
If i somehow get this bug once more - i let you know.


Develop a data retention policy for reports and artifacts

Develop a data retention policy so we can auto-delete reports/artifacts after a certain amount of time to prevent the disk from filling.
Motivation
Occasionally, the CyHy reporting disk partition runs out of space. When it runs out of space during a reports generation process, this causes errors and manual work to intervene and rerun the process.
Acceptance criteria

[ ] Determine data retention policy
[ ] Document a data retention policy with specified number of days
[ ] Determine who in addition to Rick/Jason need to sign off on the policy
[ ] Review and sign off on policy by identified stakeholders
[ ] Determine where policy should live or be published
[ ] Post the signed-off data retention policy
[ ] Create follow-on issue for implementing the data retention policy

@hillaryj - I'm thinking keep the past 4 weeks' reports just in case we need to dig the reports out for some reason (e.g. cyhy_reports mailbox gets full and it takes people a couple weeks to realize they need a specific report).  What do you think?  If sounds good to you, I can run by Rick/Jason and anyone else who may need to be checked in with prior to officially documenting/implementing that change.
As mentioned in our meeting in December before the holidays, I'd also like to get a copy of all the DHS parent reports up to this point if it is possible :)

@hillaryj - I'm thinking keep the past 4 weeks' reports just in case we need to dig the reports out for some reason (e.g. cyhy_reports mailbox gets full and it takes people a couple weeks to realize they need a specific report).  What do you think?  If sounds good to you, I can run by Rick/Jason and anyone else who may need to be checked in with prior to officially documenting/implementing that change.
As mentioned in our meeting in December before the holidays, I'd also like to get a copy of all the DHS parent reports up to this point if it is possible :)

That sounds reasonable - data retention policies are often either specified either via number of days or number of artifacts.
Who would know who needs to sign off on the policy before implementation? Where should the policy live?

That sounds reasonable - data retention policies are often either specified either via number of days or number of artifacts.
Who would know who needs to sign off on the policy before implementation? Where should the policy live?

@climber-girl emailed up the chain and received buy-in from Rick Lichtenfels that:

30 days retention is enough
Documentation via FAQ is a good place for now

@climber-girl emailed up the chain and received buy-in from Rick Lichtenfels that:

30 days retention is enough
Documentation via FAQ is a good place for now