Dart analyzer is crashing

Dart analyzer (run via flutter analyze on packages/flutter in the flutter repo) is crashing consistently on my machine:
command
flutter analyze --verbose
exception
AnalysisException: AnalysisException: Cannot compute DART_ERRORS for /usr/local/google/home/amirha/git/flutter/dev/integration_tests/flavors/lib/main.dart
Caused by Unexpected exception while performing ResolveUnitTypeNamesTask for source /usr/local/google/home/amirha/.pub-cache/hosted/pub.dartlang.org/test-0.12.27+1/lib/src/backend/invoker.dart
#0      AnalysisTask._safelyPerform (package:analyzer/task/model.dart:333)
#1      AnalysisTask.perform (package:analyzer/task/model.dart:220)
#2      AnalysisDriver.performWorkItem (package:analyzer/src/task/driver.dart:287)
#3      AnalysisDriver.computeResult (package:analyzer/src/task/driver.dart:112)
#4      AnalysisContextImpl.computeResult (package:analyzer/src/context/context.dart:720)
#5      AnalysisContextImpl.computeErrors (package:analyzer/src/context/context.dart:663)
#6      AnalysisDriver._analyze (package:flutter_tools/src/dart/analysis.dart:87)
#7      AnalysisDriver.analyze (package:flutter_tools/src/dart/analysis.dart:51)
#8      AnalyzeOnce.analyze (package:flutter_tools/src/commands/analyze_once.dart:154)
<asynchronous suspension>
#9      AnalyzeCommand.runCommand (package:flutter_tools/src/commands/analyze.dart:62)
#10     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:253)
<asynchronous suspension>
#11     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:193)
<asynchronous suspension>
#12     CommandRunner.runCommand (package:args/command_runner.dart:194)
<asynchronous suspension>
#13     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:279)
<asynchronous suspension>
#14     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:109)
#15     new Future.sync (dart:async/future.dart:222)
#16     CommandRunner.run (package:args/command_runner.dart:109)
#17     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:162)
#18     run.<anonymous closure> (package:flutter_tools/runner.dart:89)
<asynchronous suspension>
#19     AppContext._run (package:flutter_tools/src/base/context.dart:76)
<asynchronous suspension>
#20     AppContext.runInZone.<anonymous closure> (package:flutter_tools/src/base/context.dart:66)
#21     _rootRun (dart:async/zone.dart:1120)
#22     _CustomZone.run (dart:async/zone.dart:1001)
#23     runZoned (dart:async/zone.dart:1467)
#24     AppContext.runInZone (package:flutter_tools/src/base/context.dart:65)
#25     run (package:flutter_tools/runner.dart:60)
<asynchronous suspension>
#26     main (package:flutter_tools/executable.dart:42)
<asynchronous suspension>
#27     main (file:///usr/local/google/home/amirha/git/flutter/packages/flutter_tools/bin/flutter_tools.dart:16)
#28     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:263)
#29     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:151)

Caused by NoSuchMethodError: The getter 'element' was called on null.
Receiver: null
Tried calling: element
#0      Object._noSuchMethod (dart:core-patch/object_patch.dart:42)
#1      Object.noSuchMethod (dart:core-patch/object_patch.dart:46)
#2      TypeResolverVisitor.visitGenericFunctionType (package:analyzer/src/generated/resolver.dart:9666)
#3      GenericFunctionTypeImpl.accept (package:analyzer/src/dart/ast/ast.dart:5732)
#4      VariableDeclarationListImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:11149)
#5      UnifyingAstVisitor.visitNode (package:analyzer/dart/ast/visitor.dart:3121)
#6      TypeResolverVisitor.visitNode (package:analyzer/src/generated/resolver.dart:9827)
#7      UnifyingAstVisitor.visitVariableDeclarationList (package:analyzer/dart/ast/visitor.dart:3228)
#8      VariableDeclarationListImpl.accept (package:analyzer/src/dart/ast/ast.dart:11144)
#9      FieldDeclarationImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:4369)
#10     UnifyingAstVisitor.visitNode (package:analyzer/dart/ast/visitor.dart:3121)
#11     TypeResolverVisitor.visitNode (package:analyzer/src/generated/resolver.dart:9827)
#12     UnifyingAstVisitor.visitFieldDeclaration (package:analyzer/dart/ast/visitor.dart:3011)
#13     FieldDeclarationImpl.accept (package:analyzer/src/dart/ast/ast.dart:4364)
#14     TypeResolverVisitor.visitClassMembersInScope (package:analyzer/src/generated/resolver.dart:9528)
#15     ScopedVisitor.visitClassDeclaration (package:analyzer/src/generated/resolver.dart:7081)
#16     TypeResolverVisitor.visitClassDeclaration (package:analyzer/src/generated/resolver.dart:9474)
#17     ClassDeclarationImpl.accept (package:analyzer/src/dart/ast/ast.dart:1906)
#18     NodeListImpl.accept (package:analyzer/src/dart/ast/ast.dart:7990)
#19     CompilationUnitImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:2502)
#20     UnifyingAstVisitor.visitNode (package:analyzer/dart/ast/visitor.dart:3121)
#21     TypeResolverVisitor.visitNode (package:analyzer/src/generated/resolver.dart:9827)
#22     UnifyingAstVisitor.visitCompilationUnit (package:analyzer/dart/ast/visitor.dart:2949)
#23     CompilationUnitImpl.accept (package:analyzer/src/dart/ast/ast.dart:2495)
#24     ResolveUnitTypeNamesTask.internalPerform (package:analyzer/src/task/dart.dart:5141)
#25     AnalysisTask._safelyPerform (package:analyzer/task/model.dart:321)
#26     AnalysisTask.perform (package:analyzer/task/model.dart:220)
#27     AnalysisDriver.performWorkItem (package:analyzer/src/task/driver.dart:287)
#28     AnalysisDriver.computeResult (package:analyzer/src/task/driver.dart:112)
#29     AnalysisContextImpl.computeResult (package:analyzer/src/context/context.dart:720)
#30     AnalysisContextImpl.computeErrors (package:analyzer/src/context/context.dart:663)
#31     AnalysisDriver._analyze (package:flutter_tools/src/dart/analysis.dart:87)
#32     AnalysisDriver.analyze (package:flutter_tools/src/dart/analysis.dart:51)
#33     AnalyzeOnce.analyze (package:flutter_tools/src/commands/analyze_once.dart:154)
<asynchronous suspension>
#34     AnalyzeCommand.runCommand (package:flutter_tools/src/commands/analyze.dart:62)
#35     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:253)
<asynchronous suspension>
#36     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:193)
<asynchronous suspension>
#37     CommandRunner.runCommand (package:args/command_runner.dart:194)
<asynchronous suspension>
#38     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:279)
<asynchronous suspension>
#39     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:109)
#40     new Future.sync (dart:async/future.dart:222)
#41     CommandRunner.run (package:args/command_runner.dart:109)
#42     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:162)
#43     run.<anonymous closure> (package:flutter_tools/runner.dart:89)
<asynchronous suspension>
#44     AppContext._run (package:flutter_tools/src/base/context.dart:76)
<asynchronous suspension>
#45     AppContext.runInZone.<anonymous closure> (package:flutter_tools/src/base/context.dart:66)
#46     _rootRun (dart:async/zone.dart:1120)
#47     _CustomZone.run (dart:async/zone.dart:1001)
#48     runZoned (dart:async/zone.dart:1467)
#49     AppContext.runInZone (package:flutter_tools/src/base/context.dart:65)
#50     run (package:flutter_tools/runner.dart:60)
<asynchronous suspension>
#51     main (package:flutter_tools/executable.dart:42)
<asynchronous suspension>
#52     main (file:///usr/local/google/home/amirha/git/flutter/packages/flutter_tools/bin/flutter_tools.dart:16)
#53     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:263)
#54     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:151)

#0      AnalysisContextImpl.computeResult (package:analyzer/src/context/context.dart:725)
#1      AnalysisContextImpl.computeErrors (package:analyzer/src/context/context.dart:663)
#2      AnalysisDriver._analyze (package:flutter_tools/src/dart/analysis.dart:87)
#3      AnalysisDriver.analyze (package:flutter_tools/src/dart/analysis.dart:51)
#4      AnalyzeOnce.analyze (package:flutter_tools/src/commands/analyze_once.dart:154)
<asynchronous suspension>
#5      AnalyzeCommand.runCommand (package:flutter_tools/src/commands/analyze.dart:62)
#6      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:253)
<asynchronous suspension>
#7      FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:193)
<asynchronous suspension>
#8      CommandRunner.runCommand (package:args/command_runner.dart:194)
<asynchronous suspension>
#9      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:279)
<asynchronous suspension>
#10     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:109)
#11     new Future.sync (dart:async/future.dart:222)
#12     CommandRunner.run (package:args/command_runner.dart:109)
#13     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:162)
#14     run.<anonymous closure> (package:flutter_tools/runner.dart:89)
<asynchronous suspension>
#15     AppContext._run (package:flutter_tools/src/base/context.dart:76)
<asynchronous suspension>
#16     AppContext.runInZone.<anonymous closure> (package:flutter_tools/src/base/context.dart:66)
#17     _rootRun (dart:async/zone.dart:1120)
#18     _CustomZone.run (dart:async/zone.dart:1001)
#19     runZoned (dart:async/zone.dart:1467)
#20     AppContext.runInZone (package:flutter_tools/src/base/context.dart:65)
#21     run (package:flutter_tools/runner.dart:60)
<asynchronous suspension>
#22     main (package:flutter_tools/executable.dart:42)
<asynchronous suspension>
#23     main (file:///usr/local/google/home/amirha/git/flutter/packages/flutter_tools/bin/flutter_tools.dart:16)
#24     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:263)
#25     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:151)

flutter doctor
[✓] Flutter (on Linux, locale en_US.UTF-8, channel unknown)
    • Flutter at /usr/local/google/home/amirha/git/flutter
    • Framework revision 98f844ac33 (4 hours ago), 2017-11-10 09:17:52 -0800
    • Engine revision 11d68f63da
    • Tools Dart version 1.25.0-dev.11.0

[✓] Android toolchain - develop for Android devices (Android SDK 26.0.2)
    • Android SDK at /usr/local/google/home/amirha/Android/Sdk
    • Platform android-26, build-tools 26.0.2
    • Java binary at: /usr/local/google/home/amirha/progs/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] Android Studio (version 2.3)
    • Android Studio at /opt/android-studio-with-blaze-2.3
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] Android Studio (version 2.3)
    • Android Studio at /usr/local/google/home/amirha/progs/android-studio
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] IntelliJ IDEA Community Edition (version 2017.2)
    • Flutter plugin version 18.0
    • Dart plugin version 172.4155.35

[✓] Connected devices
    • Pixel • FA6890301419 • android-arm • Android 8.1.0 (API 27)

@bwilkerson @pq @cbracken

@amirh, Flutter's on an older Dart SDK at this point. Can you re-test w/ a more recent one? If you edit bin/internal/dart-sdk.version to contain 2.0.0-dev.7.0, and run bin/internal/update_dart_sdk.sh, you'll get the latest dev dart sdk.

I had to blow out my local repository as I was unable to work... so don't have the repro anymore, sorry...

Duplicate of #31442 which has more info so I'll close this one.
Thanks for filing!


Add brandLogo API to Upgrade Guide

Add brandLogo API to upgrade guide in documentation.

[ ] Changes have been thoroughly tested to not break existing functionality.
[ ] New functionality has been documented or existing documentation has been updated to reflect changes.
[ ] Visual changes are explained in the PR description using a screenshot/recording of before and after.

Thanks


Make uvloop available to TUI mode

Codecov Report

Merging #255 (7186fbd) into main (dcc3777) will decrease coverage by 0.32%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #255      +/-   ##
==========================================
- Coverage   91.98%   91.65%   -0.33%
==========================================
  Files          10       10
  Lines         848      851       +3
==========================================
  Hits          780      780
- Misses         68       71       +3

Impacted Files
Coverage Δ

weather_command/main.py
76.66% <0.00%> (-4.04%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


refactor(build-tools): Enforce export maps in the build-tools package

AB#1608
Enforce exports maps in azure-client in order to restrict submodule imports. Details in ticket.

I think this should be a policy handler that enforces that packages have exports, rather than a one-off fix. Here's an example handler that does soemthing similar:
https://github.com/microsoft/FluidFramework/blob/bd5c1c53216802900c8771cc17f1e28c921ef7a3/build-tools/packages/build-tools/src/repoPolicyCheck/handlers/npmPackages.ts#L1024-L1060
In that same file there are other examples, including some that also define "resolvers" that will automate the changes. If you write a handler and resolver, then not only do you get enforcement and protection in CI, you also get to the make the changes across the repo using pnpm policy-check:fix rather than manually.
(One note on resolvers - sometimes you end up having to duplicate logic between the handler and the resolver because the resolver does not run the handler logic.)

Closing in favour of: https://github.com/microsoft/FluidFramework/pull/17824


Considering depreciate migration in 2.2

etcd only supports online/offline migration between one minor version changes.
So we do not expect people to migration from 0.4.x to 2.2 directly.
/cc @philips @kelseyhightower

:+1:

closed by #3207


Change request to use a GET instead of a POST

How can I modify the method used? I want the request to be a GET instead of a POST, which is what makes sense in a RESTful api since we are not changing any record, just getting data.

I believe this is on the list to do according to issue #51

Superb. Closing this.


Replace ChangeNotifier with ValueNotifier in codelab

https://codelabs.developers.google.com/codelabs/flutter-codelab-first#0
https://github.com/flutter/codelabs/tree/main/namer
The lab uses ChangeNotifier to listen for the changes. Nowadays ValueNotifier is lighter and easier option to listen to changes.

@domesticmouse

Hey @filiph is this something you'd like to take on, or should I handle?

Digging into this and I noticed that the ChangeNotifier derived MyAppState object contains two pieces of state:

current of type WordPair
favorites of type <WordPair>[]

While we could munge these two objects together with a record to shove it inside a ValueNotifier, the favorites is a mutable list, which makes this a bad fit which ever way you dice this onion.
Closing this as a bad fit.

First of all, sorry for the late reply! I'm a bit all over the place these days, unfortunately.
Second, thanks @polina-c for the suggestion and @domesticmouse for the reply.
I agree that ValueNotifier would be lighter, but as @domesticmouse says, it loses its simplicity when there are several things to keep track of. Suddenly, we would be telling newcomers to Flutter to create several valuenotifiers, provide them separately, and do things like someState.value = blah (explaining setters in the process). Also, as @domesticmouse says, mutable lists don't work as expected with ValueNotifier — we'd have to introduce something like UnmodifiableListView, which again is something newcomers can easily wait to learn much later.


Color errors in semantic map (UE5)

Describe the bug
I only tested this in UE5. I'm not sure whether this is a bug or I'm doing something wrong, but for some colors, the Plugin will produce erroneous colors in the semantic map and consequently, pixels belonging to the same class will have color values different by one digit.
For example, I have a class to which I assign the RGB color  (1.0, 0.502886, 0.0) using the Color Picker in UE. According to the SemanticClasses.csv, this class gets mapped to (255, 188, 0). However, in the final image, some pixels within the class are incorrectly assigned the value (255, 189, 0). This only happens for some color values.
To Reproduce
Steps to reproduce the behavior:

Build UE5 and EasySynth ue5 branch from source
Create level & open EasySynth Plugin
Manage Semantic Classes, add a class and select a slightly odd color, e.g. RGB = (1.0, 0.502886, 0.0)
Assign the semantic class to some objects
Render (tested with jpg and png)

Expected behavior
All pixels belonging to one semantic class have the exact same
Screenshots
Color in Unreal:

Color after opening & displaying image using Python:

Configuration (if applicable):

OS: Windows 10 Pro
GPU: NVIDIA GeForce GTX 1080 Ti

Hi @cgsaxner,
Semantic colors are stored as integers throughout the plugin and saved as such in SemanticClasses.csv.
It seems that the issue here is with the numerical accuracy of the post-processing material.
Unfortunately, we cannot additionally process images after the rendering, as they are directly saved to the disc. It would require opening, altering, and resaving generated images.
I'll try to reproduce and see what I can do.
The quickest solution is probably to write a simple Python script that would round image pixels to the closest color found in SemanticClasses.csv.

Thanks for the reply & insights. I also can only imagine that during post-processing, there is some small value added somewhere, which causes different rounding when going from float set with the color wheel to integer.
As you suggested, I created a Python script to map the pixel values. Maybe it is useful to anyone else:
from PIL import Image
import numpy as np

# open SemanticClasses.csv and semantic image
classes = pd.read_csv(path.join(root_path, 'SemanticClasses.csv'))
seg_array = np.array(Image.open(seg_path))[:, :, :3]

# get colors from the .csv
colors = []
for cla in classes.iterrows():
    colors.append(tuple((cla[1].loc['255'], cla[1].loc['255.1'], cla[1].loc['255.2'])))

# compute closest color value
distances = np.zeros((seg_array.shape[0], seg_array.shape[1], len(colors)))
for i in range(len(colors)):
    distances[..., i] = np.mean(np.abs(seg_array - colors[i]), axis=2)

mask = np.argmin(distances, axis=2).astype(np.uint8)

# replace the pixels
for color_in_mask in np.unique(mask):
    seg_array[mask == color_in_mask] = colors[color_in_mask]


[FEATURE] Add hints to toolbar

I minor one, hints would be nice on these, before we clicked them:

There are hints if you hover for a bit.


Cannot install using pip when not in login shell

I use a cron job to install packages across several machines. When I try to install emoji-keyboard using pip3 in the cron job, i.e without being in an 'interactive' session, it fails:
Collecting https://github.com/OzymandiasTheGreat/emoji-keyboard/archive/master.zip
  Downloading https://github.com/OzymandiasTheGreat/emoji-keyboard/archive/master.zip
    Complete output from command python setup.py egg_info:
    Unable to init server: Could not connect: Connection refused
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-avx08ieq-build/setup.py", line 5, in <module>
        from lib.emoji_shared import version
      File "/tmp/pip-avx08ieq-build/lib/emoji_shared.py", line 102, in <module>
        wayland = check_wayland()
      File "/tmp/pip-avx08ieq-build/lib/emoji_shared.py", line 82, in check_wayland
        current_user = current_user if current_user != 'root' else os.getlogin()
    OSError: [Errno 25] Inappropriate ioctl for device

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-avx08ieq-build/
Calibre is up-to-date

I had a look at the code, and it seems like the code in check_wayland() and get_keycode() is the problem. This code looks like it should be run at runtime (i.e when the user is starting emoji-keyboard in their desktop session), but there should be no reason to run it at install time during a 'pip install' as far as I understand it.

I had a similar issue, the solution ended up being to run pip3 install command without sudo.

New version out, no installation needed anymore, since it's an AppImage.
Please use that from now on.


LUCENE-10002: replace TopFieldCollector usages in tests with collector manager

This commit replaces some usages of TopFieldCollector in tests with a corresponding collector manager created through TopFieldCollector#createSharedManager.

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[ ] I have given Lucene maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[ ] I have added tests for my changes. (not applicable as it modifies existing tests)

the build seems to have gotten stuck, not sure how to unblock it.


TASK [edgenet-kubernetes : Ensure Kubernetes APT GPG key is present] is failing on Ubuntu 22.04.2

When running the boostrap.sh script, the task installing the GPG key of the kubernetes repo is failing with:
TASK [edgenet-kubernetes : Ensure Kubernetes APT repository is present] ********
fatal: [localhost]: FAILED! => {"changed": false, "msg": "Failed to update apt cache: W:https://download.docker.com/linux/ubuntu/dists/jammy/InRelease: Key is stored in legacy trusted.gpg keyring (/etc/apt/trusted.gpg), see the DEPRECATION section in apt-key(8) for details., W:GPG error: https://packages.cloud.google.com/apt kubernetes-xenial InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY B53DC80D13EDEF05, E:The repository 'http://apt.kubernetes.io kubernetes-xenial InRelease' is not signed."}

Adding the key using the deprecated apt-key and re-running the boostraph.sh script did the trick
curl https://packages.cloud.google.com/apt/doc/apt-key.gpg | sudo apt-key add -

FYI I fixed it in a fork but I am unsure how to bring this in an interoperable way in Ansible.
https://github.com/EdgeNet-project/node/compare/main...mpiraux:edgenet-node:main

Hello @mpiraux  and thank you for opening an issue. Let me look into how we can integrate it with Ansible.

@mpiraux, I am trying to replicate the problem can you give some information about your setup?

I tried running the boostrap.sh script on a Ubuntu 22.04.2 x86_64 machine that was recently provisioned. The thing worth noting that comes to my mind is that I installed podman-docker which might conflict with EdgeNet.

This might me caused by preinstalled binaries. @cscognamiglio

Hello @mpiraux, I checked the implementation and it is working! Thank you for your contribution. I invite you to create a PR for these changes, so that we can merge with main.


Flocker does not support hostname fig directive

Flocker does not support the Fig hostname directive, a single value acting on a container in the same manner as its docker run counterpart.
http://www.fig.sh/yml.html

We are moving our development planning to JIRA. This issue is now being tracked at https://clusterhq.atlassian.net/browse/FLOC-764. You are welcome to file additional issues in GitHub if that's easier for you.


SDK Diff Tests failing to find associated installer builds for 8.0.1xx

The source-build-sdk-diff-tests pipeline is failing for all legs on branch 8.0.1xx with error:
Example build (internal link)
Could not find a build of installer for commit 'installer (commit sha)'.
I believe that this is because the tag has changed from installer-commitsha to installer-installer commitsha. The build is looking for installer-commitsha, but the build is instead tagged as installer-installer commitsha.

This should be fixed by https://github.com/dotnet/source-build/issues/3936


FIX warn on loading of hf2 dataset

that some molecules are aligned around z axis.
In order to avoid surprises.

Great! I was thinking something like this is necessary. However, at some point I will regenerate the dataset to remove such irregularity.


iOS - Expense - Submit button is gone briefly after merchant editor is dismissed on confirm page

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

Version Number: 9.0.62-3
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:  N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Issue found when executing PR https://github.com/Expensify/App/pull/51318
Action Performed:

Launch ND app
Go to FAB > Submit expense
Enter amount > Next
Select a workspace
Tap Merchant
Enter merchant and save it

Expected Result:
Submit button will appear at the bottom as soon as the merchant editor is dismissed
Actual Result:
Submit button is gone briefly after merchant editor is dismissed on confirmation page
Workaround:
Unknown
Platforms:

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

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[x] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/3752ac64-f9ff-49b7-a57c-3aee04cdbe52
View all open jobs on GitHub

I'll try to reproduce in my phone.

The button takes longer to appear in Android native using the staging app, but I guess that is somewhat expected since it is much slower than my mac/web. I'll try to test Android native using the production app to see if there is a difference between staging/production.

Sorry for confusion,
This issue repro on IOS and not repro in Android

Can't reproduce in dev using the IOS emulator, the button appears instantly
https://github.com/user-attachments/assets/d40567ac-e4d7-438e-b840-fb61264956cf
I'll remove the DeployBlockerCash label since I can't reproduce and Jayesh wasn't able to reproduce either: https://expensify.slack.com/archives/C02NK2DQWUX/p1731615988446309?thread_ts=1731615180.163839&cid=C02NK2DQWUX
Maybe the device used was very slow (old?)?

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

I just noticed I should have added retest-weekly together with Needs reproduction

Issue not reproducible during KI retests. (First week)

Closing since it seems like it is not reproducible and I don't think this matters much


Performance of ResNet50?

Thanks for your great work!!
Have you tried to train and tune the ResNet50 model?
I found the performance of ResNet50 is not better than ResNet18 (for the given hyperparameters)

I haven't looked into resnet50. maybe smaller resnet is sufficient for easy dataset as cifar10.


Aux window: unexpected address bar

Testing #196195 on Edge with insiders.vscode.dev
It would be great if there is anyway to hide the address bar, or show the correct address: insiders.vscode.dev

Great idea, but this does not seem possible due to security concerns. I am already instructing the window.open call to open as popup.


Faction buttons in search do not function

I keep getting all factions if I search for something like Jintek ICE. I get all ICE from all factions. Same for Upgrades..etc.

Can you describe what you are doing exactly? What is your locale?

I think I finally figured it out. I had another button in wrong position. I
think.
On Aug 15, 2016 10:56 PM, "Cédric Bertolini"<EMAIL_ADDRESS>wrote:

Can you describe what you are doing exactly? What is your locale?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Alsciende/netrunnerdb/issues/36#issuecomment-240009229,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUCdY4_oJPNigzyLrgUBf7qFk2QL-hwBks5qgVEMgaJpZM4JjunE
.

OK, that did not work. What I am trying to do is get a list of Jinteki ICE.
I am leaving the title, text and flavor text blank. I am selecting the
CORP button (which turns gray) at this point all the faction buttons are
gray as well. I have tried both , clicking on jinteki only (it turns white)
and clicking on all but jinteki (it is the only gray button) and select
"ICE" as type. everything else is left "any" or blank. And the results for
both ways are the same, a list of "all" ICE.
About the only thing I have found that I can almost use is at the bottom
using the "by faction" option to have the entire list sorted by faction.
But it is a random list and the jinteki ice can't be sorted in any manor.
So I'm not sure what I am doing wrong. I'm selecting "corp" I've tried both
clicking only on jinteki and clicking on everything except jinteki on the
faction buttons, and selecting "ice" as type. So I'm not sure if I just
can't figure out how to do this (every other type of search I do works) or
I found a hole in the software. I wanted a list of just jentki ice that I
could sort different ways for comparison. I'm in California.
On Tue, Aug 16, 2016 at 5:17 PM, Robert Thurman <
<EMAIL_ADDRESS>wrote:

I think I finally figured it out. I had another button in wrong position.
I think.
On Aug 15, 2016 10:56 PM, "Cédric Bertolini"<EMAIL_ADDRESS>wrote:

Can you describe what you are doing exactly? What is your locale?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Alsciende/netrunnerdb/issues/36#issuecomment-240009229,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUCdY4_oJPNigzyLrgUBf7qFk2QL-hwBks5qgVEMgaJpZM4JjunE
.

I found the problem. For some reason CHROME can not use the search that
way. It won't work for "text" search either. I switched to IE and
everything is fine. W10, new computer. I'm in California.
On Tue, Aug 16, 2016 at 5:17 PM, Robert Thurman <
<EMAIL_ADDRESS>wrote:

I think I finally figured it out. I had another button in wrong position.
I think.
On Aug 15, 2016 10:56 PM, "Cédric Bertolini"<EMAIL_ADDRESS>wrote:

Can you describe what you are doing exactly? What is your locale?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Alsciende/netrunnerdb/issues/36#issuecomment-240009229,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUCdY4_oJPNigzyLrgUBf7qFk2QL-hwBks5qgVEMgaJpZM4JjunE
.

Ok, thank you for the explanation. It's useful.
Now, can you tell me what happens when you click "Search", after you selected "ICE" and clicked on the buttons to have only "Jinteki" and "Corp" selected?

I assume by your explanation that your form looks like that, right?

Oh, sorry, I misread you. You did say what happens: you get a list of all ICE. Am I right in assuming that the search field at the top contains "t:ICE d:c"?
Now, I have an idea of what can be causing this. When the "Jinteki" button is focussed, it sometimes looks like it's depressed.
Just to do a quick test, can you search for all Jinteki and Weyland ICE? What does the search field contain?

It's midnight here so one last message. I had been using the back arrow of
the browser to back up a page and do another search. This is not a good
idea. What happens is the new search criteria "both,corp,runner and the
factions" seems to be added to the prior criteria. using the back arrow
does not clear out the old faction choice, even if you select a new
faction. I found 2 ways around this.  This seems to clear it out. You can also hit the browser
refresh button to reload page and clear out everything.
Everything in the text boxes stays intact if you use the back arrow, All
the factions are gray as if you had not chosen one. But I think the old
faction choice is still selected, but the faction buttons color is reset,
even though your prior choice is still active. Then when select a new
faction it gets added instead of replacing prior info. So if you keep
searching just runner stuff you get weird results, then if you switch to
corp and run a search.....BONK!
Good hunting .
On Wed, Aug 17, 2016 at 11:31 PM, Cédric Bertolini<EMAIL_ADDRESS>
wrote:

Oh, sorry, I misread you. You did say what happens: you get a list of all
ICE. Am I right in assuming that the search field at the top contains
"t:ICE d:c"?
Now, I have an idea of what can be causing this. When the "Jinteki" button
is focussed, it sometimes looks like it's depressed.
Just to do a quick test, can you search for all Jinteki and Weyland ICE?
What does the search field contain?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Alsciende/netrunnerdb/issues/36#issuecomment-240635758,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUCdY3mZuUFPK0_9uxtoZWv__W8HgGzcks5qg_xbgaJpZM4JjunE
.

In reality I just did not understand how the search engine was set up, and
did not use it correctly. Now that I know to clear search criteria out
manually before next search it will be OK. Thanks for the replys and
checking things out. This database is really a big help.
On Thu, Aug 18, 2016 at 2:06 AM, Cédric Bertolini<EMAIL_ADDRESS>wrote:

Closed #36 https://github.com/Alsciende/netrunnerdb/issues/36 via
a0810ad
https://github.com/Alsciende/netrunnerdb/commit/a0810adfb3312afa93cbaddebccc5242820ec6c5
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Alsciende/netrunnerdb/issues/36#event-759625644, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AUCdY6aDyUiusik9n7elzB6LMEQM4iCeks5qhCCJgaJpZM4JjunE
.


add a TLS multicodec

Alternatively, we could repurpose https (443) as TLS and define HTTPS to be /tls/http (which is, IMO, more correct).
But let's make this decision quickly. I'm fine keeping this as it is and later deprecating /https in favor of /tls/http.

While we are at it, would be good to quickly decide what should be the canonical representation of HTTPS, and ensure it is the default in the libraries we use internally for converting URLs to multiaddrs.

JS: what should we use?  /tls/http vs /https https://github.com/tableflip/uri-to-multiaddr/issues/6


the availablePermits of an invokerSlot may exceed the maxAllowed value when number of controller changed

Environment details:

native CentOS 7.3.1611 with docker 17.05-ce
4 controllers and 9 invokers, with invokerThreShold=16 and numCore * coreShare=20

Steps to reproduce the issue:

invoke a single action in a infinitely loop in every one second

action code is:
function main() {
    sleep(50000)
    return {payload: 'Hello world'};
}

function sleep(d){
  for(var t = Date.now();Date.now() - t <= d;);
}

and the loop script is:
#!/bin/bash
while true
do
  wsk -i action invoke hello --result
  sleep 1s
done

as the hello action's execution time is about 50-51 seconds and it's invoked in every one second, there will be around 50 running containers in the cluster(make sure no other actions are invoked), which will make the home invoker(let's say it's invoker5) overloaded, while the cluster is still capable

stop a controller manually to simulate a failure and terminate the loop script in step 1

wait for all running actions finished, and then execute the loop script in step1 again

Provide the expected results and outputs:
in the home invoker(invoker5), the total running containers should be:
# docker-runc list | grep running | wc -l
19

19 = 16(running containers) + 2(prewarmed containers) + 1(invoker container)
Provide the actual results and outputs:
# docker-runc list | grep running | wc -l
23

23 = 20(running containers) + 2(prewarmed containers) + 1(invoker container)
Explain
the reason is while cluster is changed, the ShardingContainerPoolBalancer will throw away the old invokerSlots and create a new sequence to replace it:
def updateCluster(newSize: Int): Unit = {
  val actualSize = newSize max 1 // if a cluster size < 1 is reported, falls back to a size of 1 (alone)
  if (_clusterSize != actualSize) {
    _clusterSize = actualSize
    val newTreshold = (totalInvokerThreshold / actualSize) max 1 // letting this fall below 1 doesn't make sense
    currentInvokerThreshold = newTreshold
    _invokerSlots = _invokerSlots.map(_ => new ForcableSemaphore(currentInvokerThreshold))

    logging.info(
      this,
      s"loadbalancer cluster size changed to $actualSize active nodes. invokerThreshold = $currentInvokerThreshold")(
      TransactionId.loadbalancer)
  }
}

then the new states of all item in invokerSlots are all set to the max value(5=16/3 in this case), but there are still some invoking permits for some invokers not released(4 in there for the home invoker), when running actions are finished and release its permit, the sync.permits will increase and may exceed the currentInvokerThreshold(actually sync.permits can upto 5+4=9), which may cause the home invoker take more invoking requests than its capacity while some other invokers in idle state

@markusthoemmes


Move the end to the beginning

It was painful and laborious to read this article and then discover the best was at the end beginning with the section Scenarios and configurations.
When you go shopping you walk down aisles and scan. When you see what you are looking for, if you were required to spend many minutes reading technical details before you get to a brief, simple, elegant summary, you'd move onto the next product.
Put the summary on the top so developers can quickly find what they are looking for. They can read the details later. Provide links from the top of the page to the details and code samples.

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

ID: 2639218d-5e54-aba0-ca0a-e53025b06d15
Version Independent ID: fec67d60-26a4-ba4c-45cf-f53c74281659
Content: Service communication with the ASP.NET Core - Azure Service Fabric
Content Source: articles/service-fabric/service-fabric-reliable-services-communication-aspnetcore.md
Service: service-fabric
GitHub Login: @vturecek
Microsoft Alias: vturecek

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. Though the scope of our feedback channel here on GitHub covers specific documentation fixes, we can help redirect you to the right support channel to get an answer to your question: [reference to alternative product/service or community endpoint]. Because this issue does not relate to documentation, we are closing this issue. #please-close


[python] fix failed import Mapping on python 3.10

Signed-off-by: kXuan<EMAIL_ADDRESS>Problem

Fix python controller failed on python 3.10

Traceback (most recent call last):
  File "/data/work/oppo/matter/oppo-lamp/third_party/connectedhomeip/src/app/ota_image_tool.py", line 41, in <module>
    from chip.tlv import TLVReader, TLVWriter, uint  # noqa: E402
  File "/data/work/oppo/matter/oppo-lamp/third_party/connectedhomeip/src/app/../controller/python/chip/tlv/__init__.py", line 32, in <module>
    from collections import Mapping, Sequence, OrderedDict
ImportError: cannot import name 'Mapping' from 'collections' (/usr/lib/python3.10/collections/__init__.py)

Change overview
Python 3.10 has removed some alias from collections module.  Python 3.10 What's New:

Remove deprecated aliases to Collections Abstract Base Classes from the collections module. (Contributed by Victor Stinner in bpo-37324.)

The python controller still use deprecated aliases, so it failed on python 3.10.
This patch fix this problem by import Mapping, Sequence from collections.abc.
Testing
How was this tested? (at least one bullet point required)

Test locally by build ESP32 Ota Requestor demo with a python 3.10 installed system.

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


Phone number is not dialing

The phone number is not being dialed when you press the number. The number is turned into a link but does not dial the number. Is there a solution?
Thanks
T.

Just add href="tel:##########" to a link that you want to call. There is also a magic class, ".selectable," I think, that exposes the text to be interacted with (this is turned off by default as it's closer to native apps... can't "select" text in list items, buttons, toolbars, etc)


Can we define a new keyword for the upgraded keyof to avoid the breaking changes?

@ahejlsberg Such as allkeyof.
related: #23627, #23645

We considered that, but given (a) not much code is affected by the change, (b) it is significantly more complex to reason about two operators and their differences, and (c) all existing documentation only mentions the keyof operator, we feel it is better to just have keyof default to the new (and more desirable) behavior and provide a backwards compability switch (--keyofStringsOnly).
BTW, note that you can easily get the old behavior by using Extract<keyof T, string> where you previously used keyof T.


[gatsby-theme-docz] Using Docz in existing Gatsby site fails

I'm trying to migrate from Storybook to Docz within an existing Gatsby project, and Docz is trying to use my site's config and gatsby-node page generators. Which causes the Docz build to fail and error all over the place.
What's the recommended approach of embedding Docz within an existing Gatsby site, to document its components?

Update: tried using gatsby-theme-docz, but now docz doesn't pull in any of my components etc to ./.docz/, just the child theme and my pages

Okay more digging, figured I can fix the million can't resolve x in y errors by setting up a webpack alias, but the only way to do that is the gatsby-node.js, which will mess up my actual Gatsby build. Is there a custom env context or something Docz uses I could hook into? Or is Docz just not designed to work alongside Gatsby?
I'm also still getting lots of X fragment doesn't exist and cannot query fields x on y errors from Docz trying to interact with my site's data source. I think they're just noise though

Also tried shadowing gatsby-node.js in src/gatsby-theme-docz to no avail

Docz is supposed to work as a theme to gatsby as my understanding source
But I think it is just set up on picking up gatsby-node.js if existent which I think is how most will use it but not ideal in many cases, like yours.
I have also had some trouble with using aliases and I think it's a bit broken or not giving back enough error/info on why it is not working: #1480
Can you explain to me your folder structur och setup a bit clearer as I am having trouble picturing what you have and what you are trying to accomplish.

My folder structure is pretty standard for a Gatsby project:
src
|  components
|  |  component
|  |  |  index.ts
|  |  |  Component.mdx
|  pages
gatsby-config.js
gastby-node.js

I was hoping to use Docz to document the components in my Gatsby project, from .mdx files colocated with each component. The issue is that Docz expects you to provide a gatsby-node and gatsby-config file just for it. This means that it's incompatible with Gatsby projects as far as I understand, because Gastby also needs those files (obviously), and the config for a production site and a Docz setup there will almost always be vastly different.

Did you finally find a solution to your problem?

I've tried to get shadowing to work with gatsby-theme-docz and still cannot figure it out

Yes, I moved back to Storybook 🤷♀

Wow @seaneking, I found that the shadowed theme must be in src/, even though I didn't have an src/ directory prior. That solves my shadowing issue I think?
root
 |- doczrc.js
 |- components
 |- src
     |
     |- gatsby-theme-docz
           |
           |- wrapper.js


How do I record image size and position after scaled?

This is a really amazing stuff for resizing/moving object on mobile. I am trying to use it on setting user's portrait, but I found it uses css matrix to control photo size and position. It there a best way to record and apply photo information after users finish their operation? Thank you.
Sorry about my English.
Paul

$el.panzoom('getMatrix')
Returns matrix.
http://www.w3schools.com/css/css3_2dtransforms.asp
Tells you what the matrix values do. 1st and 4th element give x/y scaling, 5th and 6th give x/y offset.


bumps dependency to work with node 4.0

If you have the lastest version of node, the install will fail. According to this  [comment] (https://github.com/sass/node-sass/issues/1122#issuecomment-143020384) by the package maintainer, bumping to ^3.3.3 will fix the issue.

Pushing out a release shortly.

Published 0.3.6

\o/ thanks for the fast response!


Can't locate web element using Protractor API

Hi,
I am using chrome headless browser in protractor for testing of Angular WebApp. But, the problem is headless browser is not locating the web element and giving below error,
- Failed: No element found using locator: By(css selector, *[id="login_form_login"])
obviously because it doesn't wait to load browser, css elements etc.
How it can be resolved to locate web element for headless browser?

Hi, @bharukaRupesh ! Please provide your config file and test example.

Hi @IgorSasovets ,
Thanks for quick response..!
I have attached 3 files i.e.,
1)config.js file
2)LoginPage.ts   - As I am using page object in this example
3)login_spec.ts
Config_File.txt
LoginPage.txt
login_spec.txt
Please check it and let me know if I am going wrong anywhere.

Thanks) I'll check it and provide feedback.

Sure

Sure

@bharukaRupesh , I modified it a bit. Please try and provide feedback. Also, if it's Angular app I would recommend you to comment line browser.waitForAngularEnabled(false); in your config file. Which version of Angular used in your application?
Config_File.txt
login_spec.txt
LoginPage.txt

@IgorSasovets , I tried with the latest code modified by you, still I am getting the same error as below,
*- Failed: No element found using locator: By(css selector, [id="login_form_login"])
Basically, the application which I am working on is hybrid i.e. the login page is in "Angular Js" and post login it is in "Angular 5" that's why I have kept browser.waitForAngularEnabled(false); line in my config.js file.

I need to take a look at application. Could you share link to app if it's possible?

@IgorSasovets  The application is still under development and not hosted anywhere.

Ok, but I still need example app to help you with this issue. Do you know some app with similar logic?

I can't share the application since it's under development. I just want to understand, what you need to see in the app. so that I can check it for you and update you.
It would be better if you give me any other pointers to look into.

I want to see integration of Angular5 into AngularJS because I had a lot of issues with Angular5 and synchronization. I used before hooks and worked with synchronization settings using browser.ignoreSynchronization = (false - if I want to work with angularjs | true - angular5).

As I said, the "Login" page is in "AngularJs" so for that I have kept browser.waitForAngularEnabled(false); in my config.js file and once I login to the application it's working fine for Angular 5 as well.
In other way, if I set browser.waitForAngularEnabled(true) to true, I get an error on login page itself that's why I kept it false so that it'll not wait for Angular to load since the login page is in "AngularJs"
Apart from this, I don't have any other integration in my application.

@bharukaRupesh have you tried to use explicit waiter?

@CrispusDH, Yes I have one util package inside which I have written 2 methods with explicit wait( ) code as below,
util.txt

@bharukaRupesh I did not find element with  *[id="login_form_login"] locator in the LoginPage. Could you provide class with this element?

@CrispusDH I am providing you 3 different files as below,

Config.js file
LoginPage.ts
login_spec.ts

Config_File.txt
login_spec.txt
LoginPage.txt

@bharukaRupesh yep, I took a look at those files. But I don't see the element, that's why I don't see how you implement action with this element. Maybe you didn't use explicit waiter.

Hey @CrispusDH , I am using Page Object Model and I have given you the "LoginPage" file in which I have used "element( )" to locate the UI Elements.
Yesterday also I had provided you one "util" file in which I have implemented explicit wait. Still, I getting an issue.

@bharukaRupesh I see element with locator *[id="login_form_login"] that was root of issue in the your error message - Failed: No element found using locator: By(css selector, *[id="login_form_login"]).

'f_user' does not equal to "login_form_login"
'f_pass' does not equal to "login_form_login"
'login_form_submit' does not equal to "login_form_login"

So, I ask you where I can take a look on your class where element with this locator "login_form_login" is located?

Firstly, let's try to find the root of problem.
this it:
    it('should login to the application', function(){
        console.log("Login to application");

        loginPage.enterUsername(browser.params.userName);
        loginPage.enterPassword(browser.params.password);
        loginPage.clickLoginButton();
    });

change to:
    it('should login to the application', function(){
        console.log("Login to application");
        browser.sleep(10000);

        loginPage.enterUsername(browser.params.userName);
        loginPage.enterPassword(browser.params.password);
        loginPage.clickLoginButton();
    });

It will wait 10 sec until further actions will start. In this way we will understand wether the problem with wait or with something else.

I have tried by using browser.sleep(10000); but still getting the same error message.
So, what I think is problem is not with wait but it's with something else.

I can't share the application since it's under development.
It would be better if I get to know what you want to check in it, so that I can check it for you and let you know.

The first thing is, we have id's in the DOM that is for sure.
Secondly we are not using iframe.

So, the issue in the headless chrome. I'm sure will be better to create a question on stack overflow :). I'm not familiar with specific issues that could appear on headless chrome.

Hey,
I have tried using headless firefox  and it has worked perfectly. Just did the following configurations,
{
  browserName: 'firefox',
  firefoxOptions: {
    args: ['--headless']
  },
  'moz:firefoxOptions': {
    args: [ '--headless' ]
  }
}
]```

That means, now we can say there might be an issue in ```headless chrome.```

@bharukaRupesh , you can do another one PoC. Just add custom wait that would be enough on your mind for this element to appear and execute next command:
const res = await browser.executeScript('return document.querySelector(<elementSelector>).offsetParent;', <selector>);.
If res not equal to null and you don't get any errors then element is visible on page.

@IgorSasovets, Thanks for the additional pointer. I'll surely try this and will update you.

Did you get a solution to this problem. We are getting a similar issue, but it is behaving differently in different machines, which is even stranger.


feat: add github actions

This commit adds GitHub actions support on each commit or new pull request.

Thanks for your time @eloycoto !
I'm using xtask so I don't have to remember how to compile things :D . Would it be interesting to also use 'cargo xtask...' in CI so we do the same at home and in the cloud?
Should we add a fmt check to the CI? (it would also be nice as a git hook...)

@punkto done,
The fmt check I'll explore a custom action which add comments in the PR code + clippy.

Thanks, @eloycoto . I'm merging it.


DefaultMembershipRebootDatabase doesn't like the connectionString

var db = new DefaultMembershipRebootDatabase(this.connString);
I am keep getting this network-related error:
A network-related or instance-specific error occurred while establishing a connection to SQL Server. The server was not found or was not accessible. Verify that the instance name is correct and that SQL Server is configured to allow remote connections. (provider: SQL Network Interfaces, error: 26 - Error Locating Server/Instance Specified)
but the instance name is correct, remote connection is enabled, the same connection string is used in another project and it works fine. Why would this happen, did I miss something?

That's EF giving the error, since MR is just passing along the connection string.

Thank you very much, Brock!
On Tue, Feb 23, 2016 at 7:55 AM, Brock Allen<EMAIL_ADDRESS>wrote:

Closed #586
https://github.com/brockallen/BrockAllen.MembershipReboot/issues/586.
—
Reply to this email directly or view it on GitHub
https://github.com/brockallen/BrockAllen.MembershipReboot/issues/586#event-560379146
.


Small Buttons

First off I just want to say thank you for your beautiful CSS Framework. Well Done!
I have a suggestion regarding buttons. In my opinion the default buttons are the perfect size for a everyday button. It would however be nice to be able to have a option for a smaller button as well.

My Button

.siimple-btn.siimple-btn-small {
padding: 0px 10px;
font-size: 12px;
line-height: 20px;
}

Added new class siimple-btn--small to change the button size to small:

Will be available with the new release of siimple. Thank you :smile:


pull request for e2e test at 12/18/2016 21:45:17

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix getting instance availability zone from Nova

What this PR does / why we need it:
Fix the code to set the correct availability zone for the instance, otherwide the pvc cannot bind to the pod because the zone doesn't match(using ignore-volume-az=false).
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.10

/lgtm

This pull request fixes #191?

This pull request fixes #191?

didn't work for me

@mogaika yes, that's what this PR is trying to fix
@strigazi You still didn't see the az info in the node label during the CCM initialization? Please make sure the node has correct taint before you start CCM.


Named routes not working in standalone mode

Hello
Your framework looks promising, but I had some trouble getting it to work. Routes with params in them did not work correctly
So something like this
 *          @Drest\Route(
 *              name="get_brand",
 *              routePattern="/brands/:id",
 *              routeConditions={"id": "\d+"},
 *              verbs={"GET"}
 *          ),

and using a bootstrap like this
<?php
// bootstrap.php
use Doctrine\ORM\EntityManager;
use Drest\Configuration;
use Drest\Event;

require_once "vendor/autoload.php";

// Create a simple "default" Doctrine ORM configuration for Annotations
$isDevMode = true;

// Create an example doctrine application
$ormConfig = new \Doctrine\ORM\Configuration();
$pathToEntities = array(__DIR__ . '/src/');
$ORMDriver = $ormConfig->newDefaultAnnotationDriver($pathToEntities, false);
$ormConfig->setMetadataDriverImpl($ORMDriver);

// Other various configuration options..
$ormConfig->setProxyDir(__DIR__ . '/Entities/Proxies');
$ormConfig->setProxyNamespace('Entities\Proxies');
$ormConfig->setAutoGenerateProxyClasses(true);

// the connection configuration
$dbParams = array(
    'driver'   => 'pdo_mysql',
    'user'     => 'foo',
    'password' => 'foo',
    'dbname'   => 'phpapi',
);

$em = EntityManager::create($dbParams, $ormConfig);

$drestConfig = new Configuration();
$drestConfig->setDetectContentOptions(array(
    Configuration::DETECT_CONTENT_HEADER => 'Accept',
    Configuration::DETECT_CONTENT_EXTENSION => true,
    Configuration::DETECT_CONTENT_PARAM => 'format'
));
$drestConfig->setExposureDepth(3);
$drestConfig->setExposeRequestOption(Configuration::EXPOSE_REQUEST_PARAM_GET, 'expose');
$drestConfig->setMetadataCacheImpl(new \Doctrine\Common\Cache\ArrayCache());
$drestConfig->setDebugMode(true);
$drestConfig->addPathsToConfigFiles($pathToEntities);
// Set up event manager
$evm = new Event\Manager();
//$evm->addEventListener(array('preServiceAction', 'postServiceAction', 'preRouting', 'postRouting', 'preDispatch', 'postDispatch'), new MyEvents\MyEvent());
$emr = \Drest\EntityManagerRegistry::getSimpleManagerRegistry($em);
$drestManager = \Drest\Manager::create($emr, $drestConfig, $evm);
echo $drestManager->dispatch();

(so this uses the default HttpFoundation lib, which is installed)
a call to /brands/1 would never see the param. Strangely, calling
echo $drestManager->dispatch(null, null, "Brands::get_brand", array('id'=>1));

works
To fix this, I had to add the following line to MappingRouteMetaData.php -> matcher method
$this->route_params = $matcher->getRouteParams();

Bests

Hi @wirtsi,
So the following:
echo $drestManager->dispatch(null, null, "Brands::get_brand", array('id'=>1));

actually bypasses the internal router. You're explicitly stating what route you want to call, and passing in the relevant parameters. This implementation:
echo $drestManager->dispatch();

Uses the internal router, and which should then provide (as long as your web server is forwarding all your requests to this file ie apache RewriteRule) all matched parameters to the param_names variable on the  relevant Drest\Mapping\RouteMetaData object should be written to the request object. Can you try retrieving the value like so:
$drestManager->getRequest()->getRouteParam('id')

This should give you what you need. The param_names variable on the manager object should really be internal. Did you see any issues with the request being routed properly?
Regards,
Lee

Hi Lee ... thanks for your response.
It indeed was the built in PHP server I'm using. You need to pass the bootstrap script to it, then everything works fine
php -S localhost:8000 bootstrap.php

Bests
wirtsi
btw, I would greatly appreciate if you could shed some light on https://github.com/leedavis81/drest-client/issues/1 ;)


Version 2.0.0: generation of a library with default implementation

It has been suggested in #570 to generate a dynamic library that contains the compiled version of a default implementation (e.g. Model, Data and all the default algorithms).
This issue is opened to discuss this implementation and the technical details.

You do not need C++ 11 to do that. As an example, see here.
template class se3::SE3Tpl <double>;

Do you know what extern keyword adds in this case ?

I checked and your version is not necessarily handled by all the compilers.
Here, the extern mentions that the template has been generated in another .o file.
Your version will generate the template only for the current .o file, not for the others.
So if you set your example in several .cpp files, they will also generate the template and duplicate the code.

So in one cpp file, you will set:
template class se3::SE3Tpl<double>;

and in the orders, you just need to say:
extern template class se3::SE3Tpl<double>;

But, yes, this has only been checked with gcc. I didn't try with other compilers.

This means that we need to really split declaration from definition in our classes, which is a tedious work.

I agree. I check how FCL or PCL do. I know understand the piece of code which achieves this.
They add at the beginning of each definition file the line you mentioned (extern template class Foo<0>;). I wonder what happen if you compile a file that contains this line and you do not link to the library that provides its implementation: Will the compiler compile Foo<0> or fail with a linking error ?
If it compiles Foo<0>, we merely add the extern declaration for all class and play with CMake and pkg-config ! Otherwise, we must surround the extern declaration with a #ifdef PINOCCHIO_LINK_TO_LIBRARY ... #endif.

Indeed, I do not see any alternative to that: even if you manage to
pre-compile a specialized version of the template, you still need to
have access to the declaration in the 3rd-party code, is it not?
Re-organizing would have the advantage to also enable faster compilation
in 3rd parties, that would then only include declaration in their
header, and some definitions in their sources.
But this is a huge work, both in the definition of the methodology and
in the execution of it. I suggest to decouple it from 2.0.0. What we
could do is to anticipate in the way headers should be included in
2.0.0, so that changing the lib organization afterward does not change
the API. But even that seems a big package of work.
Are we clear about how this problem is tackle in Eigen? I suppose we
have a lot to learn from the Eigen team.
On 10/30/2018 10:34 AM, Justin Carpentier wrote:

This means that we need to really split declaration from definition in
our classes, which is a tedious work.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stack-of-tasks/pinocchio/issues/571#issuecomment-434231341,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAb-CI2FBhbNfKk01q0RiSZh48TCwvrLks5uqB0WgaJpZM4X_2gm.

But this is a huge work, both in the definition of the methodology and in the execution of it. I suggest to decouple it from 2.0.0. What we could do is to anticipate in the way headers should be included in 2.0.0, so that changing the lib organization afterward does not change the API. But even that seems a big package of work.

For the purpose of the library, I do not think you need a clear separation of the declaration and definition. I think it isn't that hard to achieve.
For decreasing the compilation time and provide declaration-only headers, I agree the work is long and tedious. This is not for v2.0.

So then let's skip the code generation part for the current release.
Anyway, by using the keyword 'extern template', we have to rely on C++11 (which is not bad).
Of course, if people still need to compile only with C++98 standard, they will be able to do it through the header-only part.

I agree. I check how FCL or PCL do. I know understand the piece of code which achieves this.
They add at the beginning of each definition file the line you mentioned (extern template class Foo<0>;). I wonder what happen if you compile a file that contains this line and you do not link to the library that provides its implementation: Will the compiler compile Foo<0> or fail with a linking error ?
If it compiles Foo<0>, we merely add the extern declaration for all class and play with CMake and pkg-config ! Otherwise, we must surround the extern declaration with a #ifdef PINOCCHIO_LINK_TO_LIBRARY ... #endif.

It will depend if you a partial or a complete definition of your class I guess.

Is this going to be in v2 ?

It can be, but it requires some refactoring work.
The question is first: do we want to impose C++11 for extern templatekeywords use?

What about a incomplete but already better solution of encapsulating all #include <foo.hxx> by some #ifndef PINOCCHIO_USE_LIBRARY ... #endif` and adding the flags to pkg-config file ?

I think we should keep .hxx files and just add between an #ifdef the extern template keywords.
In this way, the compiler won't compile the template if we refer them through keywords.
In particular, I think only Model, Data and all the algorithms and parsers should be externalized. It should be sufficient for HPP and other frameworks.
@jmirabel What do you think?

Fine, go ahead for this.

Are you sure you can keep the compatibility with C++ 98?

Yes. Just people using C++98 won't have access to this feature, as it requires C++11.

I will provide a solution for this issue soon.

Most of the teams using pinocchio are just in the middle of the merge toward pinocchio v2. Please be very careful not to damage the current API in any mean.

Yes, it won't be the case. It will just use the C++11 feature.
If C++98 is used, then this feature won't be activated and all the code will be compiled as it is the case now.

I gave a quick trial to template instantiation for C++ >= 11: dbffed2
With this commit, compiling file lib.cc:
#include <pinocchio/algorithm/kinematics.hpp>

using namespace pinocchio;

Eigen::MatrixXd foo()
{
  Model model;
  Data data(model);
  Eigen::MatrixXd q;
}

with the CMakeLists.txt:
cmake_minimum_required(VERSION 3.10)

project(test LANGUAGES CXX)

find_package(pinocchio REQUIRED)

add_library(lib_linked SHARED lib.cc)
target_link_libraries(lib_linked pinocchio::pinocchio)

add_library(lib_not_linked SHARED lib.cc)
target_link_libraries(lib_not_linked pinocchio::pinocchio)
target_compile_definitions(lib_not_linked PRIVATE "-DPINOCCHIO_ENABLE_TEMPLATE_INSTANTIATION=0")

@jmirabel wrote:
Then, time make lib_linked (using template instantiation) gives:

real	0m6,208s
user	0m5,707s
sys	0m0,482s

and time make lib_not_linked (using template instantiation) gives:

real	0m8,290s
user	0m7,591s
sys	0m0,570s

The benefits in terms of compilation time is obvious.

Thanks a lot @jmirabel for this attempt and I would say this result was expected.
We just need time to handle correctly and completely this task.
I think we should wait for #911 which will impose some major modifications, and then handle this one. @jmirabel Do you agree?

Done in pinocchio-3x which should be released soon.


How to add an error after processing the server in ValidationObserver?

There is a form that consists of ValidationObserver and ValidationProvider:
<ValidationObserver @submit.prevent="validate().then(signUp())" v-slot="{ valid, validate }" ref="signUpForm" tag="form" novalidate>
  <ValidationProvider name="email" rules="required|email" v-slot="{ errors }" class="mb-12" tag="div">
    <tw-text-input
      v-model="formEmail"
      :error-messages="errors"
      name="email"
      type="email"
      placeholder="E-mail">
    </tw-text-input>
  </ValidationProvider>

  <!-- ... -->

When I submit the form data to the server, the data that is wrapped in Promise comes back. As a result, in the component, I have the following code:
signUp() {
  this.send().then(() => {
    this.signIn()
  }).catch((response) => {
    console.log(
      response
    )
  })
}

There are errors in the response - this may be for one or more fields. Below is my code in which I was desperately trying to iterate over a lot of options:
signUp() {
  const hash = {
    field: 'email',
    msg: 'test error'
  }

  this.$refs.signUpForm.errors.add(hash)
  this.$refs.signUpForm.$validator.errors.add(hash)
  this.$validator.errors.add(hash)
  this.errors.add(hash)
}

It didn’t work with Promise, so I immediately deleted it and left a simple code.
Tell me, please, why am I unable to add an error?

this.$validator is only available if you use the v-validate directive. With the provider/observer, it is a little different. You can find it documented here
in v3 it will be much easier.

@logaretm

Note that v3 was released recently so some links are now down.
here you go:
http://vee-validate.logaretm.com/v2/guide/components/validation-provider.html#adding-errors-manually


QuickJs methods should not be synchronized

We really should make it the caller’s responsibility to do all synchronization? Like ArrayList ?
(This might not be feasible depending on what existing users expect.)

I am very supportive of this.


Upstream changes

Upstream changes

Auto-generated using create-pull-request.
Do not edit: just merge it via Create a merge commit.

Let me test the new workflow :)


Add TCP accept metric which tracks the total number of TCP connections accepted.

proxy.process.net.tcp.accept
This is useful to track the rate of inbound connections accepted. This can be compared to transactions, open connections, or other OS network metrics to better identify performance issues.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1457/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1563/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/128/ for details.

Why are there tsconfig changes with this?

Bad merge, I'll fix the TSConfig stuff.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/6/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1573/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1467/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/138/ for details.

Hmmm, how is this different from proxy.process.http.total_incoming_connections ?

The http stat measures only HTTP 1.X connections and only those that are successfully created. If, for example, there is a connection that fails the TLS handshake, this stat will record it but http.total_incoming_connections will not. Tunnels will also be tracked by tcp_accept and not by the http stat.
It also looks like HTTP 2 connections are not tracked by that stat either, which is also distinct. It might be reasonable to either make the existing stat track both or track them independently and sum.

It seems like we should have some sort of layered approach, where we keep track of tcp (l4) http (L5?) and TLS (L7). We have metrics for some of this it sounds, but IIRC they are a bit patchy.
So I'm in favor of a raw "tcp accepted", and IMO the http incoming_connections should probably be 1/2 agnostic (if we want an http_stream_accepted-- we can do that too).

Yes, if there are subtle differences like this, we should break them out accordingly, and then make an aggregated "sum" metrics in the Lua metrics.config.
It almost sounds like a bug / misfeature that the H2 connection is not included in the proxy.process.http.total_incoming_connections. And it's very inconsistent with other metrics; Whereas these two connection metrics works as you describe (independent), the request metrics are not. E.g.
Sending 4 HTTP/2 request, I see this:
./bin/traffic_ctl metric get proxy.process.http2.total_client_streams proxy.process.https.incoming_requests proxy.process.http.incoming_requests
proxy.process.http2.total_client_streams 4
proxy.process.https.incoming_requests 4
proxy.process.http.incoming_requests 4

Note how each H2 stream is counted as an HTTPS request, as well as an HTTP request (I'm guessing because of how H2 uses HTTP internally). This seems rather inconsistent to me, but does make some sense I guess

http2 connections are tracked by proxy.process.http2.current_client_sessions and proxy.process.http2.total_client_connections.
All successfully negotiated SSL/TCP connections would be the sum of proxy.process.http2.total_client_connections and proxy.process.http.total_client_connections

We should have a metric that tracks the total number of accepts at the TCP layer.  I recommend that the name be inline with the current metrics proxy.process.net.total_accepts

@bryancall - do you mean you want proxy.process.net.tcp.total_accepts instead of proxy.process.net.tcp.accepts?

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/279/

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/283/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/297/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/1870/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/409/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/1978/

clang-analyzer build failed! https://ci.trafficserver.apache.org/job/clang-analyzer-github/542/


Vscode terminal command line string overlaps with output

Steps to Reproduce:

Debug and run the C file

When the command string is executed under powershell, it will be displayed together with the printf content. When the terminal window size is changed, it will return to normal:

In addition, I tried turning "terminal.integrated.gpuAcceleration" on or off, but it didn't work.
When switching the terminal configuration to "Command Prompt" or "Windows PowerShell" it looks normal (if you ignore the ^C and blank lines):

settings.json:
{
    "workbench.colorTheme": "Default Dark+",
    "editor.fontSize": 18,
    "terminal.integrated.gpuAcceleration": "auto",
}

tasks.json
{
    "tasks": [
        {
            "type": "cppbuild",
            "label": "C/C++: gcc.exe 生成活动文件",
            "command": "C:\\msys64\\ucrt64\\bin\\gcc.exe",
            "args": [
                "-fdiagnostics-color=always",
                "-g",
                "${file}",
                "-o",
                "${fileDirname}\\${fileBasenameNoExtension}.exe"
            ],
            "options": {
                "cwd": "${fileDirname}"
            },
            "problemMatcher": [
                "$gcc"
            ],
            "group": {
                "kind": "build",
                "isDefault": true
            },
            "detail": "调试器生成的任务。"
        }
    ],
    "version": "2.0.0"
}

VS Code version: Code 1.96.0 (138f619c86f1199955d53b4166bef66ef252935c, 2024-12-11T02:29:09.626Z)
OS version: Windows_NT x64 10.0.19044
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 5 2600 Six-Core Processor             (12 x 3394)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
undefined

Memory (System)
15.95GB (8.39GB free)

Process Argv
--crash-reporter-id 8e4bb2b1-a6b6-4f0f-82a0-457aaf97359a

Screen Reader
no

VM
0%

Extensions (6)

Extension
Author (truncated)
Version

vscode-language-pack-zh-hans
MS-
1.96.2024121109

debugpy
ms-
2024.14.0

python
ms-
2024.22.0

vscode-pylance
ms-
2024.12.1

cpptools
ms-
1.22.11

cpptools-extension-pack
ms-
1.3.0

(1 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vscod805:30301674
binariesv615:30325510
vsaa593:30376534
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
vscrpc:30673769
962ge761:30959799
pythonnoceb:30805159
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
dvdeprecation:31068756
dwnewjupytercf:31046870
nativerepl1:31139838
pythonrstrctxt:31112756
nativeloc2:31192216
cf971741:31144450
iacca1:31171482
notype1:31157159
5fd0e150:31155592
dwcopilot:31170013
stablechunks:31184530
6074i472:31201624

This issue seems to originate from an upstream component and is not something VSCode can fix by making a change to our code. As such, I will go ahead and close it. Please try to reproduce in Windows Terminal and report to microsoft/terminal with repro steps.
If you cannot reproduce in Windows Terminal, chances are it's fixed in a later version of Windows. You can read more about how we consume conpty here. Thanks for understanding.
/label upstream terminal-conpty windows


Document how to create a new mapping commons using cookiecutter

@hrshdhgd would you be able to add a documentation page here https://github.com/mapping-commons/sssom/tree/master/src/docs called create-mapping-commons.md and fill it with some initial instructions and

how to create a new mapping commons from zero to hero and
how to update one

We have a new team member (@ehartley) who will help us building the template up.

Do you mean using https://github.com/mapping-commons/mapping-commons-cookiecutter ?

Sorry I meant to share that link! Yes exactly!

Started a stub: https://github.com/mapping-commons/sssom-py/pull/374
I don't know what the end product is supposed to look like.

Completely messed that up. Sorry! https://github.com/mapping-commons/sssom/pull/277


problem when using conditional constraints on portfolio optimization

hi! I am using cvxpy to do portfolio optimization and I got some problem.
n = len(factor_data) #the number of stock is nearly 3300
weight = cvx.Variable(n)
exposure = alpha.T * (weight - w_bench) #factor exposure

objective = exposure
turnover = cvx.abs(weight - w_last)
f_style = X_style.T * (weight - w_bench)  #style exposure
f_ind = X_ind.T * (weight - w_bench) #industry exposure

###tracking error
cov_matrix = np.dot(np.dot(exposure_matrix,factor_covariance_matrix),exposure_matrix.T) + spec_ret
error = cvx.quad_form(weight-w_bench, cov_matrix)
contraints = [cvx.sum(turnover) <= 1, cvx.sum(weight) == 1, weight >= 0, weight <= 0.05, f_style >= -0.01,f_ind >= -0.01, f_style <= 0.01, f_ind <= 0.01, error <= 0.07 * 0.07 / 250]
prob = cvx.Problem(cvx.Maximize(objective), constraints=contraints)
prob.solve(solver=cvx.MOSEK)
print(prob.status)

when i run this code, it can get the optimal result, but the number of none zero weight is nearly 1600, it means we need to hold half of the stock in the market, which is hard to achieve. So I want to add constraints of the stock number we hold.
stock_number = cvx.sum([0 if weight[i] <= 0 else 1 for i in range(n)])
contraints = [stock_number <= 100,....]

and I got the error
Exception: Cannot evaluate the truth value of a constraint or chain constraints, e.g., 1 >= x >= 0.
I have no idea how to add this constraints and make it work.
Hope you can help me solve this problem.
Thank you in advance.

This is a modeling question rather than a cvxpy issue. Since you are using Mosek you can look at https://docs.mosek.com/modeling-cookbook/mio.html#avoiding-small-trades or https://docs.mosek.com/9.2/pythonfusion/case-studies-portfolio.html#doc-card-constraints, they both contain examples of how to use binary variables to achieve your cardinality bounds and similar other penalties.


AMPLIFY PUSH FAILING IN RECENT DAYS - IDENTICAL CONFIG PREVIOUSLY WORKED

Describe the bug
My Cloudformation Stack starts with status "UPDATE_COMPLETE" (Green) before I initiate amplify push. The code I push is correct - last week it correctly pushed and updated on the cloud. But now this same code is failing with the following error
✖ An error occurred when pushing the resources to the cloud
Resource is not in the state stackUpdateComplete
An error occured during the push operation: Resource is not in the state stackUpdateComplete
I have reproduced this in two separate accounts and get identical results.
If I look at the Cloudformation Events for this stack I see that the earliest error event was this...
Embedded stack arn:aws:cloudformation:XXXX was not successfully updated. Currently in UPDATE_ROLLBACK_IN_PROGRESS with reason: The following resource(s) failed to update: [CustomResourcesjson].
So I manually do "Continue update rollback" from the Stack Actions menu on the Cloudformation console. This fails a couple of times, but eventually I get back to status "UPDATE_ROLLBACK_COMPLETE".
Amplify CLI Version
Amplify 3.17.0
To Reproduce
I'm not sure how to reproduce outside of my environment.
However, I have the following steps work around the problem.

I perform the PUSH as defined above (amplify push --no-gql-override --yes)
The push fails as described above
Go to CloudFormation and Continue Update Rollback until the Stack gets to "UPDATE_ROLLBACK_COMPLETE" status.
I perform PUSH again and after it starts to deploy I terminate  (CTRL-C) which causes the CloudFormation stack to terminate the update and returns to the "UPDATE_COMPLETE" status (Green).
I peform PUSH again and it successfully deploys.

If I follow that workflow every time in every account is successfully deploys - but it always fails first as described.
Expected behavior
Successfully push changes to the cloud without having to follow this workflow. Last week the same code was deploying successfully - this week it is not.
Desktop (please complete the following information):

I am using Cloud9 to run Amplify
I am using Node 10.16.0

Additional context
I have a large database with 44 Tables in DynamoDB. I have progressively pushed this database to the cloud in order to work around the GSI limitations. I have identical instances in DEV, TEST, PROD in completely separate accounts and the issue is being experienced in all accounts.

Stacks that appear to be consistently causing issues are [SearchableStack, ConnectionStack, ConnectionStack2]. I use transform.conf.json to overflow resources from ConnectionStack to ConnectionStack2 - I have over 300 resources defined across these 2 stacks.

@sacrampton Ddi you upgrade your CLI from last week to this week?

I didn't upgrade the CLI - the problem occurred on a version that was previously working (3.15.0).  However, in an effort to see if a newer version of the CLI fixed the problem I upgraded to 3.17.0

If I keep doing the manual go to CloudFormation and Continue Update Rollback until the Stack gets to "UPDATE_ROLLBACK_COMPLETE" status then amplify push it will eventually succeed - but it can take many times of pushing, failing, rollback before it actually does succeed. Again, the fact that it eventually succeeds means that my schema.graphql and the rest of the build is valid.

@sacrampton can you reproduce it outside of Cloud9, locally with your environment? Do you've custom content in CustomResources.json?

I have not tried to reproduce outside of Cloud9 (its a non-trivial exercise for me to do that - I use Cloud9 because I've set it up to be many times faster than my local environment). However, the issue is identical on 3 separate Cloud9 instances in 3 separate AWS accounts. I do have a lot of custom content in CustomResources.json - however there have been no edits to this file since 18th October (exactly 3 weeks ago - and this problem only started in recent days).
I am eventually able to get a successful push every time - but it might take me 4 or 5 attempts to do so and having to manually roll back in CloudFormation each time.

Whatever was done on the Cloudformation side of things seems to have resolved itself.


Events TypedDict typing

Description
Documentation
Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.
[ ] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

Are the dsync.group.user_added and dsync.group.user_removed events coming in a follow PR?

Does this require changes to the WorkOS Docs?

Since we have not changed the API resource types, not yet. But when we do, then yes I think we will need to.

i hadn't done them all, just cause i wanted to make sure how i'm typing it is correct. let me switch this to a draft to make that clear

i'll add the membership stuff in a commit

I'm totally cool with the membership stuff being part of this PR or a later one. It's on your radar. That's all that matters.


Fix a broken web-app port

Hi!
Thank you for a great project!
I'm very interested in it.
I guess you changed a web-app port to the Angular CLI default one ...and forget to replace all places where it is used ....it is confusing a little bit ...so i fixed it ...please consider to apply my changes.

Thanks for the PR. I find it to be a good idea to change the default port, so experimental Angular apps that I may run side-by-side with default port 4200 can start/stop without errors. In addition, in order to change the default port in this setup, docker-compose must also be updated.


Update iphoto-library-manager to 4.2.7

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as e5718a3fd5660cd0acdeb2c23cc64e335fd69417. Your contribution is still credited to you.


reexport checkCohorts

When omopgenerics new release

This function is now deprecated in omopgenerics hahah should we reexport the validateCohortArgument function? Altough it might be confusing to have a validate argument function
https://darwin-eu-dev.github.io/omopgenerics/reference/checkCohortRequirements.html

yeah seems weird to reexport validateCohortArgument so let's close this (if we see we need something like this in the future CohortConstructor could have it's own function like checkCohorts)


MacOS Cmd + k stopped working with the tmux plugin

This keystroke is intended for clearing terminal buffer on MacOS.
MacOS 10.15.5
tmux 3.1b
neovim 0.4.3
Alacritty 0.4.3

The plugin should not have any effect on keybindings using Cmd unless you're using a non-standard setup. Also, just to confirm, the default configuration of the plugin does interfere with Ctrl-l for clear screen, but we have notes on restoring that functionality. Just wanted to highlight in case this is what you're referring to.
Otherwise, some additional questions to help figure this out:

Are you a new user of the plugin and experiencing this with your first setup, or have you used the plugin before with success?
Did this breakage come with an upgrade of any of the listed OS, tmux, or vim versions?
Does this just "not work", or does it behave in an unexpected way?

My setup is standard with no customizations related to the plugin. I refer specifically to Cmd + k because this is the main way to clear terminal on MacOS when you working with it.

I experienced this issue every time I tried to use the plugin. There were a few times I tried it.
No, I don't think so.
Yes, the keystroke in tmux just stop working when I enable the plugin.

By the way, this may be important, I use the Alacritty 0.4.3 terminal emulator.

Cmd+K still switches splits instead of clearing terminal. Any plans to fix that?

By the way, this may be important, I use the Alacritty 0.4.3 terminal emulator.

Hi @osenvosem I have noticed many things not working out-of-the-box myself with Alacritty; most noticeably this plugin. Although I'm not a Mac user, I believe issues like alacritty/alacritty#2204 are relevant to me and you. I've not gone to investigate much further that this.

I briefly investigated this issue on my Mac, I think the issue lies with
bind-key -n C-l   if-shell  "$is_vim"  "send-keys C-l"   "select-pane -R"

When you press cmd-k the terminal emulator remaps it to Ctrl+l command to the terminal. But Tmux with the config above captures it and interprets the command as select-pane to the right.
I have rebound my keyboard with Karabiner and use alacritty as my terminal emulator. Ideally alacritty could send a clear screen command another way instead of the rebinding, but I suspect that terminals take Ctrl+l to clear screen by default.
For now I'm content typing clear. Would love to hear other people's thoughts/work-arounds on this!

So I have found a work around which isn't perfect like Ctrl+l. If you're using alacritty you can add a keyboard binding to send clear\n whenever you press cmd+k. Here is my alacritty.toml config snippet:
[keyboard]
bindings = [
{ key = "k", mods = "Command", chars = "clear -x\n" },
]

The -x flag ensures the history of window is still accessible if you scroll up [ref].
This work around leaves a blank line at the top which is a little unflattering, but functionally it's good enough for me.


Css is not working while routing

Version
Please replace this sentence with the React Router version that you are using.
Test Case
https://codesandbox.io/s/7zwjlm2j3q
Steps to reproduce
Expected Behavior
Actual Behavior

This is a bug tracker, not a support system. For usage questions, please use Stack Overflow or Reactiflux where there are a lot more people ready to help you out. Thanks!


Advertissement on home assistant

Description
SUPPORT_TONES was used from tapo_control, this is a deprecated constant which will be removed in HA Core 2025.1. Use SirenEntityFeature.TONES instead, please create a bug report at https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/issues
SUPPORT_TURN_OFF was used from tapo_control, this is a deprecated constant which will be removed in HA Core 2025.1. Use SirenEntityFeature.TURN_OFF instead, please create a bug report at https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/issues
SUPPORT_TURN_ON was used from tapo_control, this is a deprecated constant which will be removed in HA Core 2025.1. Use SirenEntityFeature.TURN_ON instead, please create a bug report at https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/issues
SUPPORT_DURATION was used from tapo_control, this is a deprecated constant which will be removed in HA Core 2025.1. Use SirenEntityFeature.DURATION instead, please create a bug report at https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/issues
Reproduction Steps
go to system logs
Expected behavior
don't have this advertissement
If applicable, add error logs.
No response
Device Firmware
1.3.11
Integration Version
5.4.12
Using stream component
Yes
Does camera work via official integrations?
Yes
Camera has all attributes filled out in developer tools
Yes
HASS Environment
pi
Search for similar issues
Yes
Additional information
No response

Not sure what you mean by 'advertissement' but it's a warning.
Please note this is related to Home Assistant version 2024.1.0b0!
(Please mention this next time @Nemesis24)
Furthermore these deprecations are also present:

[ ] SUPPORT_ON_OFF was used from tapo_control, this is a deprecated constant which will be removed in HA Core 2025.1. Use CameraEntityFeature.ON_OFF instead, please create a bug report at https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/issues
[ ] SUPPORT_STREAM was used from tapo_control, this is a deprecated constant which will be removed in HA Core 2025.1. Use CameraEntityFeature.STREAM instead, please create a bug report at https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/issues

nearly all "SUPPORT_" features for different devices (Climate, Camera, Cover, Vaccuum, ....) will be depricated with 2024.1.
With one the first beta releases, an Error was logged, but this was due to a missing dependency - since 0b1 or 0b2 only a deprication warning will be shown.
After a short look into the code, I guess, the following lines needs to be changed:
from homeassistant.components.camera import (
    SUPPORT_ON_OFF,
    SUPPORT_STREAM,
    Camera,
)

to:
from homeassistant.components.camera import  CameraEntityFeature, Camera

    def supported_features(self):
        if self._enable_stream:
            return SUPPORT_STREAM | SUPPORT_ON_OFF
        else:
            return SUPPORT_ON_OFF

to:
    def supported_features(self):
        if self._enable_stream:
            return CameraEntityFeature.SUPPORT_STREAM | CameraEntityFeature.SUPPORT_ON_OFF
        else:
            return CameraEntityFeature.SUPPORT_ON_OFF

<--- these are just examples, because I do not have TAPO installed and can't test any change.
Also, I am not sure, if other files might be affected, too...
here's the blog post about the deprication:
https://developers.home-assistant.io/blog/2022/04/02/support-constants-deprecation
https://developers.home-assistant.io/blog/2023/12/28/support-feature-magic-numbers-deprecation


slightly odd behaviour with variables

Hello,
I just started testing the comparison between docker desktop and rancher desktop on mac and I noticed something odd with an environment variable not being printed as it would via docker, I might be missing something obvious here but I thought of mentioning it:
$ docker run -e value=testvar --rm packages.dns.ad.zopa.com:5000/busybox:latest sh -c 'echo $value'
testvar
$ nerdctl run -e value=testvar --rm packages.dns.ad.zopa.com:5000/busybox:latest sh -c 'echo $value'

$ nerdctl run -e value=testvar --rm packages.dns.ad.zopa.com:5000/busybox:latest sh -c 'env'
SHLVL=1
HOME=/root
value=testvar
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
PWD=/
$ nerdctl --version
nerdctl version 0.12.1

Thanks,
Max

Hey @macsmax
Cannot reproduce with current main:
$ nerdctl run -e value=testvar --rm busybox sh -c 'echo $value'
testvar

Could you confirm that it is working for you now?


Lab needs to be updated. It will fail starting at Task 04 Step 03.

Module: 14
Lab/Demo: 01
Task: 04
Step: 03
Description of issue
Been trying to reproduce the lab, Ansible with Azure, a couple of times now and it failed to be reproduced.
What I've seen, is that if you install ansible using the apt command, subsequent tasks will fail. One, because version 2.5 is installed and it will fail here: "ansible-galaxy collection install azure.azcollection", and if ansible 2.9 is installed via apt command, it will fail at Task 8, Step 10, here: "ansible-playbook ~/PartsUnlimitedMRP/Labfiles/AZ-400T05-ImplemntgAppInfra/Labfiles/ansible/new_vm_web.yml --extra-vars "vmname=<VM_name> resgrp=az400m14l03arg vnet=az400m1403vm1VNET subnet=az400m1403vm1Subnet"".
Also I've changed the default version of python that is used and it worked; but I had to change the version in the commands where python2.7 string was found and replace it with 3.6
It was more troubleshooting than following the steps, but it was an experience.
Thanks.

Hopefully the course owner here can take the helm and identify the authoritative fix soon - I, too, have been unable to make this lab work with the steps given despite multiple run-throughs.
In case cosminlof is still watching this issue report:  do you have any additional notes regarding how you got by the apparent Python 2.7 vs. 3.6 compatibility issues?
While I was able to get by the error in the 'ansible-galaxy collection install azure.azcollection' step, also by forcing Ansible 2.9, I cannot get any progress past the error when trying to run 'ansible-playbook ~/PartsUnlimitedMRP/Labfiles/AZ-400T05-ImplemntgAppInfra/Labfiles/ansible/new_vm_web.yml --extra-vars "vmname=<VM_name> resgrp=az400m14l03arg vnet=az400m1403vm1VNET subnet=az400m1403vm1Subnet'
It errors differently depending on whether I try and get dependencies installed via either pip, or pip3, but in either case I run the ship hard aground on the reef of this error:

Is there an update on this issue? As-is, the instructions for Mod 14 are non-functional.

This lab has been updated and the issues fixed.

I had run this lab in the past without any problem but now I have the following error:
TASK [Create storage account] **********************************************************************************************************************************************
fatal: [localhost]: FAILED! => {"changed": false, "msg": "Error retrieving resource group az400m14l03arg - Could not retrieve token from local cache. Please run 'az login'."}
PLAY RECAP *****************************************************************************************************************************************************************
localhost


fontawesome does not display properly

NOTE: Before you start, the following should be completed.

Read tutorial to understand the usage and the correct effect of functional design.
Make sure no similar issue(including closed ones) exists.
Try to find the answer on Jekyll Forum and StackOverflow.

Description

Operations you have already tried

Logs/Screenshots

Hi, jwcen. I checked your blog repo and found it rendered properly. It may be caused by the unstable network in mainland China.


Follow PyPI redirects. Fixes #497

@richardowen Thanks for the PR! Looks good but would you mind writing a quick test for this in the pip_spec? We already have some stubbed responses going on in there so you would just need to follow suit.

@xtreme-shane-lattanzio Apologies for the delay but have added a test for this now. Verified that the test fails before my change and passes after.

@richardowen Thanks. Looks good!


Add information on what companies have the most influence on PHP.

Something like a pie chart that displays the % influence each company has on PHP trough it's voters.

Done


Fixed nettle soup recipe

About the PR
Fixes #10656
Screenshots

Changelog

:cl:

tweak: Nettle soup recipe works now

who puts egg in nettle soup anyways


update readme to reflect markov.parse_string

I was playing with this and noticed that the parse_string was missing from the documentation, thought I'd add it in.
n

Ah, thank you very much! :+1:


HELM repository to provide the list of available charts:version in the status of HelmRepository CRD

Presently Flux source controller provides a way to register a particular HELM repository and provide the resync interval.
This resync activity checks for any updates in the index.yaml present in the repository.
Feature request :
Can the flux source controller add the list of Helm charts available in the index.yaml in the HelmRepository CR status.
OR
All the charts in the repository can be created as a HelmChart CR

We can’t do this as the etcd limit is 5MB, there are repos like Bitnami and many others that have hundreds of charts with thousands of versions, the index file is usually way bigger than the max limit of a Kubernetes resource.

Flux already has a HelmChart CRD and it does create HelmCharts based on the version or the semver range specified in a HelmRelease. Creating thousands of HelmCharts just because they are in the index, means filling out etcd and DDOS-ing Kubernetes API, if you need to list all chart versions, better to do it outside the cluster, etcd IMO is not the place to put such data.

Thank you for the detailed repose again. It did give the additional clarity I was missing.
I will be closing this issue.