Exception on IOS

🐛 Bug Report
I'm using the stream feature for real time location updates and I ran into an issue. When my position is fixed in the simulator, the update works normally but when I set the location to moving (simulator->debug->location->city run), an exception is thrown:
[VERBOSE-2:shell.cc(184)] Dart Error: Unhandled exception:
PlatformException(ERROR_UPDATING_LOCATION, The operation couldn’t be completed. (kCLErrorDomain error 0.), null)
#0      _rootHandleUncaughtError.<anonymous closure> (dart:async/zone.dart:1112:29)
#1      _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#2      _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)

Expected behavior
The location updates without throwing an exception
Reproduction steps

var locationOptions =
      LocationOptions(accuracy: LocationAccuracy.high, distanceFilter: 10);

geolocator.getPositionStream(locationOptions).listen((Position position) {
      print("New pos update: $position");
      if (position != null)
        setState(() {
          _position = position;
        });
    });

Configuration
Version: 2.1.1
Platform:

[x] :iphone: iOS
[ ] :robot: Android

I have a similar issue - Using the stream for real time location updates and I ran into an issue. When my position is fixed in the simulator, the update works normally. Update the position from debug/location a couple of times and this error occurs
[VERBOSE-2:ui_dart_state.cc(148)] Unhandled Exception: PlatformException(ERROR_UPDATING_LOCATION, The operation couldn’t be completed. (kCLErrorDomain error 0.), null)
Config
Using geolocator: "3.0.0"
iOS Simulator and device
I am investigating this further, will update soon.

Works correctly on iOS simulator 10.1, 11.1
using - simulator->debug->location->freeway drive
Fails on 12.1
using - simulator->debug->location->freeway drive
PlatformException(ERROR_UPDATING_LOCATION, The operation couldn’t be completed.

geolocator example Fails on 12.1

I'm having the same problem

True. It happens with the provided example also.

Hi, I am seeing the same issue, but for me I tried "city bicycle ride" first, and that worked! But then it failed when switching over tio "city run" with this error.
So City Bicycle Ride might be a profile you can usein the meantime.

I'm seeing this as well. I did this as a work around:
  StreamSubscription<Position> _positionStream;

  void _streamLocation() {
    // TODO: initialize _positionStream
    _positionStream.onError((e) {
      print("error streaming position: $e");
      _disposePositionStream();
      _streamLocation();
    });
  }

  void _disposePositionStream() {
    if (_positionStream != null) {
      _positionStream.cancel();
      _positionStream = null;
    }
  }

Restarting the stream seems to make it work now.
NOTE: This is not the optimal way to handle this with nulling the _positionStream while it is still being used by onError.

Still getting this error using the example app
Single seems to work, although occasionally reports null, stream fails after a few updates.
PlatformException(ERROR_UPDATING_LOCATION, The operation couldn’t be completed. (kCLErrorDomain error 0.), null)
Config (Now)
Using geolocator: "5.1.1+1"
iOS Simulator (12.4) and device (10.x.x)

By way of a Hack test..
LocationTask.m

(void)locationManager:(CLLocationManager *)manager didFailWithError:(NSError *)error {
NSLog(@"%s", sel_getName(_cmd));
//    [[self context] resultHandler]([FlutterError errorWithCode:@"*ERROR_UPDATING_LOCATION" message:error.localizedDescription details:nil]);
[self stopTask];
[self startTask]; // Forgive me...
}

This works reliably on the Simulator 12.4 and 10.3
Remember the simulator Debug, Location sometimes has to be kicked a few times before it changes.
Thoughts?

Having the same problem after updating library to latest version. After some tests we found that exceptions kicked when WiFi is on, if enable 3g it works correcty. Any solutions for this?

Having a similar problem. I'm trying to complete a course from londonappbrewery. I'm getting the following exceptions! Don't know how to fix it :/
Code:

As I press the button it shows:
Launching lib/main.dart on iPhone Xʀ in debug mode...
Running Xcode build...
Xcode build done.                                           24.1s
Syncing files to device iPhone Xʀ...
[VERBOSE-2:ui_dart_state.cc(148)] Unhandled Exception: PlatformException(ERROR_UPDATING_LOCATION, The operation couldn’t be completed. (kCLErrorDomain error 0.), null)
#0      StandardMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:569:7)
#1      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:316:33)

#2      Geolocator.getCurrentPosition (package:geolocator/geolocator.dart:103:32)

#3      _LoadingScreenState.getLocation (package:clima/screens/loading_screen.dart:12:10)

#4      _LoadingScreenState.build. (package:clima/screens/loading_screen.dart:22:13)
#5      _InkResponseState._handleTap (package:flutter/src/material/ink_well.dart:654:14)
#6      _InkResponseState.build. (package:flutter/src/material/ink_well.dart:729:32)
#7      GestureRecognizer.invokeCallback (package:flutter/src/gestures/recognizer.dart:182:24)
#8      <…>

#252

I faced same issue, my solution is:
Debug -> Location ->

When I checked here, neither 'None' nor 'Custom Location' were selected, I selected an option as you see and it worked.

Also seeing this for iOS devices in production on Sentry. I'm not using realtime location updates, just calls to getCurrentPosition/getLastKnownPosition
PlatformException: PlatformException(ERROR_UPDATING_LOCATION, The operation couldn’t be completed. (kCLErrorDomain error 0.), null)
  File "message_codecs.dart", line 569, in StandardMethodCodec.decodeEnvelope
  File "framework.dart", line 3919, in ComponentElement.mount
  File "framework.dart", line 3101, in Element.inflateWidget
  File "framework.dart", line 2904, in Element.updateChild
  File "framework.dart", line 5127, in SingleChildRenderObjectElement.mount
  File "framework.dart", line 3101, in Element.inflateWidget
  File "framework.dart", line 2904, in Element.updateChild
  File "framework.dart", line 3961, in ComponentElement.performRebuild
  File "framework.dart", line 3738, in Element.rebuild
  File "framework.dart", line 3924, in ComponentElement._firstBuild
  File "framework.dart", line 4088, in StatefulElement._firstBuild
  File "framework.dart", line 3919, in ComponentElement.mount
.
.
.

Ideas?

The following code creates the same error for me:
import 'package:flutter/material.dart';
import 'package:geolocator/geolocator.dart';

class LoadingScreen extends StatefulWidget {
  @override
  _LoadingScreenState createState() => _LoadingScreenState();
}

class _LoadingScreenState extends State<LoadingScreen> {

  void getLocation() async {
    Position position = await Geolocator()
      .getCurrentPosition(desiredAccuracy: LocationAccuracy.high);
    print(position);
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: RaisedButton(
          onPressed: () {
            getLocation();
          },
          child: Text('Get Location'),
        ),
      ),
    );
  }
}

Versions
Android Studio 3.5.3
Build #AI-191.8<IP_ADDRESS>10548, built on November 15, 2019
JRE: 1.8.0_202-release-1483-b49-5587405 x86_64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
macOS 10.15.2
geolocator-5.1.5
location_permissions-2.0.3
flutter version: v1.13.5
Running on similator: Iphone 11 Pro Max 13.3

Having a similar problem. I'm trying to complete a course from londonappbrewery. I'm getting the following exceptions! Don't know how to fix it :/
Code:

As I press the button it shows:
Launching lib/main.dart on iPhone Xʀ in debug mode...
Running Xcode build...
Xcode build done. 24.1s
Syncing files to device iPhone Xʀ...
[VERBOSE-2:ui_dart_state.cc(148)] Unhandled Exception: PlatformException(ERROR_UPDATING_LOCATION, The operation couldn’t be completed. (kCLErrorDomain error 0.), null)
#0 StandardMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:569:7)
#1 MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:316:33)
#2 Geolocator.getCurrentPosition (package:geolocator/geolocator.dart:103:32)
#3 _LoadingScreenState.getLocation (package:clima/screens/loading_screen.dart:12:10)
#4 _LoadingScreenState.build. (package:clima/screens/loading_screen.dart:22:13)
#5 _InkResponseState._handleTap (package:flutter/src/material/ink_well.dart:654:14)
#6 _InkResponseState.build. (package:flutter/src/material/ink_well.dart:729:32)
#7 GestureRecognizer.invokeCallback (package:flutter/src/gestures/recognizer.dart:182:24)
#8 <…>
I've also tried to run it on the android simulator (Pixel 3a API 28). It worked but unfortunately, it's providing incorrect coordinates. The difference in coordinates is huge. Like, I'm trying to get a location from Pakistan and its showing location somewhere in the USA.

fixed it by following the iOS Simulator -> Features -> Location -> Custom Location...

You have to set you custom location on the simulator under features>location>custom location

locationManager:didUpdateLocations:
[CurrentLocationTask dealloc]
[LocationTask dealloc]
I followed iOS Simulator -> Features -> Location -> Custom Location and now I am getting the above message. What is this?

I faced same issue, my solution is:
Debug -> Location ->

When I checked here, neither 'None' nor 'Custom Location' were selected, I selected an option as you see and it worked.

I've had to set this option a couple of times now. For some reason every now and again the setting resets itself. Anyone know why this is?

Updated, this is now in Features -> Location -> Select an option

Solved in version 6.0.0

@mvanbeusekom Obviously, this issue still appears when no location is set in the iPhone Simulator as others have pointed out. This is alright, but how to catch it properly?
try {
  permissionGranted = await location.requestPermission();
  if (permissionGranted == LocationPlatformInterface.PermissionStatus.granted) {
    currentPosition = await Geolocator.getCurrentPosition(
    desiredAccuracy: desiredAccuracy ?? LocationAccuracy.best);
  } else {
    currentPosition = null;
  }
} catch (e) {
  print(e);
  currentPosition = null;
}

This code will not catch the issue and it's hard to pin-point, as the message only appears in the Xcode logs.
Also, do you know if this issue can occur on real devices if I check for permissions? I'm concerned.
Thanks in advance.

Geolocator.getPositionStream().listen((Position position) {
_currentPosition = position;
}, onError: (error) {
  print('Error receiving location update: $error');
});

//Adding onError fixed it for me.


Minor changes in to update the signup process

I have made a minor change to reflect the signup process for Microsoft Store for Business.
Problem: https://github.com/MicrosoftDocs/windows-itpro-docs/issues/3631

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@Dansimp The metadata for this article points to TrudyHa as the author, who we know to have been retired. Can you point out to whom shall we contact for this PR? Thank you.


python310Packages.bash_kernel: 0.7.2 -> 0.9.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.bash_kernel is: Bash Kernel for Jupyter
meta.homepage for python310Packages.bash_kernel is: https://github.com/takluyver/bash_kernel
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.9.0 with grep in /nix/store/q6k4b4x94zi1754pc1pnxhsk0jnixxnp-python3.10-bash_kernel-0.9.0
found 0.9.0 in filename of file in /nix/store/q6k4b4x94zi1754pc1pnxhsk0jnixxnp-python3.10-bash_kernel-0.9.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.bash_kernel
python311Packages.bash_kernel

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/q6k4b4x94zi1754pc1pnxhsk0jnixxnp-python3.10-bash_kernel-0.9.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.bash_kernel https://github.com/r-ryantm/nixpkgs/archive/18ccf94aa758b3e463e6a82698e9144fc6269f42.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/q6k4b4x94zi1754pc1pnxhsk0jnixxnp-python3.10-bash_kernel-0.9.0
ls -la /nix/store/q6k4b4x94zi1754pc1pnxhsk0jnixxnp-python3.10-bash_kernel-0.9.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python310Packages.bash_kernel
python311Packages.bash_kernel

Maintainer pings
cc @zimbatm for testing.

Result of nixpkgs-review pr 219630 run on aarch64-darwin 1

2 packages built:

python310Packages.bash_kernel
python311Packages.bash_kernel


chore(mempool_node): remove redundant component type from config

Stack:

#1111
#1110
#1109
#1103
#1102 ⬅
#1101

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is

✓ Commit merged in pull request #1110


Use getRawOriginal instead of getOriginal

Fixes #680
From the Laravel 7 upgrade guide

The $model->getOriginal() method will now respect any casts defined on the model. Previously, this method returned the uncast, raw attributes. If you would like to continue retrieving the raw, uncast values, you may use the getRawOriginal method instead.

I used the same test from #681 But this fix seemed easier.

Sorry, I forgot about the Laravel 6 support. :sweat_smile:  So probably this is not the best way.

released: https://github.com/spatie/laravel-activitylog/releases/tag/3.11.2

@Gummibeer Thank you!


fix svc name to not include period

/rerun all

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_uperf
Pass
0
00:23:54


FlinkProcessor source utils method generates random source table name to avoid conflict

What is the purpose of the change
FlinkProcessor source utils method generates a random table name for the temporary source table to avoid conflict.
This bug causes the nightly build to fail lately.
Brief change log

Update datagen_utils.py and file_system_utils.py to generate a random table name for the temporary source table

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@lindong28 Could you review this PR?

@lindong28 Thanks for your comment! I updated the PR. Could you take another look?

Thanks for the update! LGTM.


Hardware Passthrough Support

Is your feature request related to a problem? Please describe.
I have tried migrating my Proxmox workloads to Harvester and I was faced with a lack of support for configuring hardware passthrough from the GUI. Among those workloads are things such as:

virtualizing a TrueNAS instance -> passing through an HBA
virtualizing a pfSense instance -> passing through multiple NICs
virtualizing a workstation -> passing through GPUs and USB controllers

While I'd have no problem configuring these from the command line if needed, i also couldn't find proper documentation on how to implement these things.
Describe the solution you'd like
Ideally, after configuring the devices to use the vfio drivers, a drop down should be present allowing us to pick available devices to pass through, just like proxmox does it.
Describe alternatives you've considered
Alternatively, providing some documentation and examples of configuring such setups by hand would be a great first step, and much quicker than the "full" solution.
Additional context
N/A

follow #1029

Hello,
I need to be able to add storage LUNs presented to harvester nodes via Fiber Channel Qlogic HBA card. I can see the LUNs on OS level using "cat /proc/partitions", however I don't see them when I try to add disks.
Appreciate help getting them added.
Thanks

This feature is now present as an addon in release 1.1.0. I have a video walkthrough of how to use the feature here.

@tlehman thanks for that! I will take a look tomorrow and set it up in my homelab.

Hello,
Does it support Qlogic NetXtreame 10Gb SPF NIC?
Kind regards,
Andrei


[SPARK-27180][BUILD][YARN] Fix testing issues with yarn module in Hadoop-3

What changes were proposed in this pull request?
Fix testing issues with yarn module in Hadoop-3 by downgrade jetty to 9.3.24.v20180605, upgrade jersey-1 to 1.19 and add a test class.
How was this patch tested?
manual tests:
build/sbt  yarn/test -Phadoop-3.1 -Pyarn

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8972/
Test PASSed.

Test build #103561 has started for PR 24115 at commit 757a09a.

Test build #103561 has finished for PR 24115 at commit 757a09a.

This patch fails build dependency tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class ServerSocketUtil

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103561/
Test FAILed.

Test build #104104 has started for PR 24115 at commit 3c9bb7b.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9450/
Test PASSed.

Test build #104104 has finished for PR 24115 at commit 3c9bb7b.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class SessionHandler extends ScopedHandler

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104104/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9454/
Test PASSed.

Test build #104108 has started for PR 24115 at commit 3c9bb7b.

Test build #104108 has finished for PR 24115 at commit 3c9bb7b.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class SessionHandler extends ScopedHandler

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104108/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9469/
Test PASSed.

Test build #104123 has started for PR 24115 at commit a33fcef.

Test build #104123 has finished for PR 24115 at commit a33fcef.

This patch fails RAT tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104123/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9470/
Test PASSed.

Test build #104124 has started for PR 24115 at commit 6a00f19.

Test build #104124 has finished for PR 24115 at commit 6a00f19.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104124/
Test FAILed.

retest this please

Test build #104136 has started for PR 24115 at commit 6a00f19.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9479/
Test PASSed.

Test build #104136 has finished for PR 24115 at commit 6a00f19.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104136/
Test PASSed.

Test build #104149 has started for PR 24115 at commit 49f0d8a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9492/
Test PASSed.

Test build #104149 has finished for PR 24115 at commit 49f0d8a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104149/
Test PASSed.

Merged to master


I can't see entire error just this RSpec::Core::MultipleExceptionError

Issues:
Using it as command line tool has brought so many issues.

After setting this I can't use rspec spec/example_test_spec.rb command anymore. It forces me to use with prefix bundle exec
After executing command I get the .xml then I need to convert it, for that I have to be in "allure-ci" java project directory.
To run command one more time, I need to stop the server by pressing Ctrl+c
At last I have the report but it doesn't the entire log. At least tell me at which line error is.

Summon @smecsia

@pushprajsingh05 Do you use RSpec exceptions aggregation feature? If so, there is an issue in allure-rspec adaptor (allure-rspec#58) and this is probably the duplicate.

@pushprajsingh05 btw thanks for suggestion.

After executing command I get the .xml then I need to convert it

Yep, you are definitely need to run allure-commandline to generate the report for xml results. The reason - xml is just a raw representation of your test results. Each test has its own result (In general you can run your tests in different environments). So then you need to create the report from such result. You can use allure-commandline or Jenkins Plugin or whatever you need to generate the report (an example you can integrate Allure to your own build server as we did).

that I have to be in "allure-ci" java project directory.

Could you please explain what are you mean?

To run command one more time, I need to stop the server by pressing Ctrl+c

In case you generate the report at localhost you need to start web server to serve the report cause Chrome security. Or you can run chrome with --allow-file-access-from-files flag. In case of Firefox it is not necessarily. Do you have any suggestions how to avoid of Ctrl+c to stop the server? When we should stop it? I can propose you allure report open --no-web-server command in case Firefox is your default browser.

Closed as no response. Feel free to reopen.

I am still facing this issue. Is there a solution to this ?


Update dependencies for Kitura 2.3 release

Update to appropriate dependencies for the Kitura 2.3 release
Relax minor constraints as these are no longer necessary

Note that this will no longer require Kitura-Sample to be updated when a new minor of Kitura is released.

Codecov Report

Merging #85 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #85   +/-   ##
=======================================
  Coverage   87.83%   87.83%
=======================================
  Files           1        1
  Lines         189      189
=======================================
  Hits          166      166
  Misses         23       23

Flag
Coverage Δ

#KituraSampleRouter
87.83% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fortran 2008 Complex Function

TAN, ATAN, and so on handle complex data structures.
We don't generate them in OvO for now.

fixed by #de0ebe818fa2aba828ab091aa39791a918aca492


Started feed_me

Started the feed_me project!

I request changes Also, please help me to set this up and edit in VS

If I have done my job writing the README.md properly, you should not need assistance. I added a bit more to it so that you could see it.


Tox: run the current pylint on Python 3.8

@cclauss When you're doing PRs that aren't ready for review yet, please choose "create as draft" so we wouldn't get multiple notifications for them before they're ready.


Scaffolder with v1beta2 schema template fails to create correct folder structure for existing cookiecutter templates

Expected Behavior
Using an existing Cookiecutter template repo with the scaffolder on v1beta2 schema should produce the correct repo structure:
.
├── components
├── package.json
└── pages

Current Behavior
The templated repo has an extra folder in the root, under which all the expected contents (that should be in the root) are nested. Also the hooks folder and cookiecutter.json are copied over from the template:
.
└── my-service
    ├── components
    ├── cookiecutter.json
    ├── hooks
    ├── package.json
    └── pages

Possible Solution
https://github.com/backstage/backstage/blob/f094bf78238ad2109aec9debf2aab9f1cb53cbc2/plugins/scaffolder-backend/src/scaffolder/actions/builtin/fetch/cookiecutter.ts
It looks like the logic here is incorrect for existing cookiecutter templates. I'm not exactly sure how the bug would be fixed, but the way the directory structure is generated in this action seems to cause the problem.
Steps to Reproduce

Create a template.yaml following the v1beta2 schema and pointing at an existing cookiecutter template repo that uses the nested {{cookiecutter.component_id}} structure. For example: https://github.com/spotify/cookiecutter-golang
Update your backstage config to use this new template.
Create a repo with the template.
Observe that the files are NOT under the root of the repo, and the hooks didn't run because Cookiecutter was pointing at the wrong directory for the template.

Context
We are upgrading our templates to v1beta2 to take advantage of the new custom actions, but are blocked because of this bug.
Your Environment

NodeJS Version (v12): v14
Operating System and Version (e.g. Ubuntu 14.04): Alpine Linux
Browser Information: Chrome

@jessebye the layout of the alpha1 and beta2 template contents aren't directly compatible. In alpha1 the template dir is the root of the cookiecutter directory, which usually would contain the cookiecutter.json, hooks, and template dir. In beta2 templates instead point straight at the template dir.
You should be able to either rename the dir and point to it, or set the template path to include '{{cookiecutter.component_id}}'

@Rugvip will hooks still function for beta2?

@jessebye no, although it could be worked in. The intention is really that additional steps with custom actions are used instead.

@jessebye I'd recommend renaming the folder, as we prolly want to avoid using '{{cookiecutter.component_id}}' in the template schema as it could be confusing if it gets templated or not with the other template variables in the v2 template.
@ruvip how would we manage the hooks? Have a seperate hooks folder input to the fetch:cookiecutter action?

@benjdlambert tbh I don't think the pattern of executing shell scripts on hosts is great, neither in docker but especially not in the same environment as the backend. Any hook that was just there to e.g. curl some other endpoint would be simple to replace with an action that simply does the same, could even be something we want to provide as a built-in one potentially. Scripts that modified the templates is a bit trickier, but tbh it should always be possible to just have the template in the correct shape from the beginning afaik?

I mean I fully agree that it would be best to move them into Typescript actions anyway, that way they are re-usable across templates and not bound to cookiecutter. What do your hooks do @jessebye? Could you move them into custom typescript actions instead?

@benjdlambert @Rugvip We can make this work, but our templates will be a bit more convoluted as a result. We were using hooks to do things like delete a few folders based on user selections. For instance, if the user does not need a database, we delete two folders from the template with the hook. We can do this with templating but you end up with really ugly folder nested folders like this:
└── {{\ "database"\ if\ "Database"\ in\ cookiecutter.service_reqs\ else\ ""\ }}
    └── {{\ "migrations"\ if\ "Database"\ in\ cookiecutter.service_reqs\ else\ ""\ }}
        └── {{\ "1_create_table.up.sql"\ if\ "Database"\ in\ cookiecutter.service_reqs\ else\ ""\ }}

The if statements have to follow the folders all the way down, and also have to be included for every filename inside the folders.
While we could create a TypeScript action to do this just like the hook did, it gets a bit convoluted because each template has different set of files that might be deleted.

I realized that we can simplify our structure a bit. We can have the conditional for just the filename, not the directories too - since empty directories will not be committed to git.

Hi team. Wondering what the status is around hooks currently. I have started using backstage and my cookiecutter post generate hooks do not appear to execute as expected when running fetch:template with cookiecutterCompat mode. Have hooks or some alternative to them been worked in yet?

@osiris-dev - we've purposefully left hooks out of the cookiecutter action and the cookiecutterCompat mode for security reasons by default. We'd recommend that you convert your hooks to custom actions instead, or you can build your own action that can run cookiecutter templates with the hooks instead of course, but then there's the added security risk that those hooks will just run arbitrary bits of code that you have no control or review over which we'd advise against 👍

@benjdlambert thanks for confirming. Are the hooks explicitly disabled somewhere in the code? Looking through https://github.com/backstage/backstage/blob/master/plugins/scaffolder-backend-module-cookiecutter/src/actions/fetch/cookiecutter.ts

Hey @benjdlambert, @Rugvip,
How can I modify fetch:cookiecutter into custom action that executes hooks? Are they  disabled in some cross-dependent code, could you point us to it?
PS. Cookiecutter CLI that fetch:cookiecutter uses can disable hooks execution with "--accept-hooks" flag.


Name your proposed file change ex: Fixed typo

@itzmashy : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Will angular-oauth2-oidc support Angular 9?

I'm using "Angular-oauth2-oidc" in angular 7, I'm planning to upgrade angular 9, i just want to know whether Angular-oauth2-oidc will support for angular 9?

Hey hey! You could check out #717 and the related PR #718 and see if that would work for your application's upgrade.
Whenever the maintainer gets some time, we might get a new version to support Angular 9. I recommend keeping an eye on the repo, linked issue, and possibly the announcement label.
On a side note, the 8.x version of the library might also work in Angular 9 applications, though I haven't tried it yet myself.


Fix: Unset the default [Server].Address option for PPM

as this can cause issues when serving from a domain name

What issues does this cause specifically?


why the repo of dockerhub is docker.io not hub.docker.com

why the repo of dockerhub is docker.io not hub.docker.com?
I know the official website is hub.docker.com, but in the fact ,  the repo is docker.io.  why?
e.g.
[root@node1 ~]# docker login -u xxx -p xxx docker.io
WARNING! Using --password via the CLI is insecure. Use --password-stdin.
WARNING! Your password will be stored unencrypted in /root/.docker/config.json.
Configure a credential helper to remove this warning. See
https://docs.docker.com/engine/reference/commandline/login/#credentials-store

Login Succeeded

[root@node1 ~]# docker login -u xxx -p xxx hub.docker.com
WARNING! Using --password via the CLI is insecure. Use --password-stdin.
Error response from daemon: login attempt to https://hub.docker.com/v2/ failed with status: 404 Not Found

In another words, there is a scene:
First, when I google the dockerhub, then I got the hub.docker.io. And then, I can see all my repos.So now I may think the repo is this url.But , in the fact , it is not.So I got confused. Yes, I know the right one is docker.io.But to the primers, they may easily met with the same doubt. The main problem is that ,  I cannot find any docs about this message, that told me the registry address is docker.io.

Hi, you don't need to specify the registry name for Docker Hub, so you don't generally need to know this. This is very historical, but difficult for us to change now.

See also https://docs.docker.com/registry/introduction/#understanding-image-naming
@justincormack About registry-1.docker.io, I think it's only available as an "edge" registry right?

Yes, in the general, I needn't to know specify it . But in some scenes, I do need, such as If I build some private repos and provide the offers to customers, then I should have to specify it explicitlly. To change the address is a difficult thing, but maybe that  offer some message to notify users is a good way.


Wishlist: be able to ignore SSL errors

Many sites under development use https for their API but do not install a proper certificate. It would be great to be able to ignore SSL errors in Resty by calling a method on the Resty instance.

added some half-baked support on a global basis: Call Resty.ignoreAllCerts which will install a very trusting TrustManager i.e. will trust all certificates.

Great, thank you very much! I was missing Resty for some personal projects :)


I have signed a CLA but it's always in pending status

https://github.com/iliakan/javascript-tutorial-cn/pull/15
I have signed CLA for this project but it's always in pending status after several "recheck". could you please kindly take a look at this?
account: arproio

Did @deshengxu also sign the cla? He/she as a committer should sign it.


Correct name of envvar in documentation

The buildkite agent does not appear to recognise the buildkite_meta_data
env var and it is referred differently above

Oh, thanks for catching this @kcollasarundell!


Bug: render error for markdown

> Well, I have learnt more...........own go-cqhttp without the adorable Koishi.

I actually think one of us (probably it's me) doesn't completely understand the source code. Let's take a look at this part.

\`\`\`go
	if len(byteKey) == 0 {
		log.Infof("密码加密已启用, 请输入Key对密码进行解密以继续: (Enter 提交)")
		cancel := make(chan struct{}, 1)
		state, _ := term.GetState(int(os.Stdin.Fd()))
		go func() {
			select {
			case <-cancel:
				return
			case <-time.After(time.Second * 45):
				log.Infof("解密key输入超时")
				time.Sleep(3 * time.Second)
				_ = term.Restore(int(os.Stdin.Fd()), state)
				os.Exit(0)
			}
		}()
		byteKey, _ = term.ReadPassword(int(os.Stdin.Fd()))
		cancel <- struct{}{}
	} else {
		log.Infof("密码加密已启用, 使用运行时传递的参数进行解密，按 Ctrl+C 取消.")
	}

	encrypt, _ := os.ReadFile("password.encrypt")
	ph, err := PasswordHashDecrypt(string(encrypt), byteKey)
	if err != nil {
		log.Fatalf("加密存储的密码损坏，请尝试重新配置密码")
	}
	copy(base.PasswordHash[:], ph)
\`\`\`

In my understanding, it would first check if there's an argument that provides the 'password' to decrypt the login password. If so, use it to decrypt the password, else wise, ask the user to input the password in the terminal. This would solve the problem that Masnn mentioned: the 'password' to decrypt the login password is not stored and is memorized by the admin, so only if the user inputs the password in the terminal or provide it through arguments, gocqhttp would be able to decrypt the password. Elsewise, neither gocqhttp nor other software would be able to decrypt it.

-----------

Also, I'd like to share some situations when we need it:
1. It would be helpful if Koishi backend has some critical vulnerability for arbitrary file reading.
2. I've seen many users directly send their `config.yml` out when they are asking for help. This would prevent accidental password leaks.
3. When the server was cracked, this would stop the hacker from stealing the account. Ofc this won't work if the hacker get permission of reading memory, but it still works when the hacker cannot get root permission.
4. It protects the user who doesn't use `auth` plugin.

---------------

The workflow can be easily simplified with the password args. We just need to (fake code):
\`\`\`js
pwd_encrypt = ask the user for the password.
if( file "password.encrypt" not exists ){
    pwd = ask the user for qq password
    write pwd to config.yml and enable encrypt option
    start process 'gocqhttp'
    pass the password to standard input
    wait until it exits
}
remove the password in config.yml, save it.
start process 'gocqhttp' with arg `--pwd=${pwd_encrypt}`
\`\`\`

---------

Anyway, I think that's a good feature and I'd like to leave the issue here. You contributors take the final determination if implement this feature or not.

----------

![(5@ DXGO16ACC_EC{MO2FSN](https://user-images.githubusercontent.com/66859419/215329111-69757232-2161-4dca-968a-feb863c6f469.jpeg)

Sincerely,
MicroBlock

this markdown string would cause a render error in the message like this:

dont know if this is intentional or not

我猜这个修了，你可以再试试能否复现（）


fix: support DD_SITE env var

What was changed
Added support for the DD_SITE environmental variable.
Why?
Not all users are part of the default datadog site.
https://docs.datadoghq.com/getting_started/site/

Closes

How was this tested:

[ ] Building and deploying on our own infrastructure
[ ] WIll add a unit test to ensure the value is set on the context.

Any docs updates needed?

[ ] update the readme to include details about the DD_SITE config option and add it to the helm chart.

 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.


polygon click not working in v2.1.0?

It seems that the click event does not work for polygons in v2.1.0. Popups don't show anymore when clicking on a polygon in index.html. It's working fine for points and lines.
Any ideas what is going on?

Is it fixed ? I'm still experiencing the same issue here.

I noticed this last night.  I'll have to look asap.
On Thu, Jun 27, 2019 at 7:47 AM Philéas Condemine<EMAIL_ADDRESS>wrote:

Is it fixed ? I'm still experiencing the same issue here.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/robertleeplummerjr/Leaflet.glify/issues/21?email_source=notifications&email_token=AAFFZO7GGSU6XXSNVCOHGD3P4SSDNA5CNFSM4GSJYZV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYW3RYQ#issuecomment-506312930,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFFZO7OCT3XSOG6AWEQ5F3P4SSDNANCNFSM4GSJYZVQ
.

--
Robert Plummer

This seems solved now. Thanks!


[enhancement]: Azure App Service Deployment: ARM - .NET 8 Support

Task name
Azure App Service Deployment: ARM - .NET 8 Support
Describe your feature request here
Currently it seems .NET 7 is the highest available version for the runtime stack.

.NET 8 is already released and should be supported.

I was upgrading our clients to .NET 8 just to find out the release pipelines don't support it. Kind of crazy considering this is maintained by Microsoft...?

Runtime Stack is an editable field and is of type string. In the meantime you can enter DOTNETCORE|8.0 and it will work.

Hi @v-mohithgc, I'm happy to work on this if you can assign this to me.

@amitjaura thanks, that works!
Is it "DOTNET-ISOLATED|8.0" for azure functions then?

@evaldas-raisutis, I left if blank and it worked fine.

Manually editing the field and changing the 7's to 8's actually unsets the configured stack in the Azure App Service configuration after a release, don't do it!

I was trying to use .NET 8 isolated, and thought I'd be ok given this statement in the docs:

If there is a newer version of a framework available in the supported runtime versions you can specify it even if it is not in the list.

During deployment, I got errors about a bad request (.NET 7 isolated works fine). I guess .NET 8 isolated needs to also be supported by the task, not just Azure Functions.

Please fix this ASAP!

Any idea when this bug will be fixed?

@microsoft/azure-app-service-pipelines-tasks FYI

@v-mohithgc  just checking to see if you have any updates on this

Looks like this task uses this API

https://learn.microsoft.com/en-us/rest/api/appservice/provider/get-available-stacks?view=rest-appservice-2023-12-01

to get the list of runtimestack, at the moment it doesn't have the Dotnet core 8 version support. I've reached out to some contacts to see if we can get an updated api. I'll keep this chat updated if i get any updated

How this is not a high priority ?!! Please fix..

For app services, setting the Runtime Stack manually works:


test-branch/execute-job/quoted-authorization/45a8e793-04f7-42dc-ae4e-6fd9c20af616

PR for testing 30c2efeacc537918e1df32a96609300407b6d4ef

/canonical/self-hosted-runners/run-workflows 30c2efeacc537918e1df32a96609300407b6d4ef


[CssBasline/Migration] Change in default fontSize

This is an issue with documentation.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Text 😯
One of the v5 breaking changes is https://github.com/mui-org/material-ui/pull/24018 and the suggested fix is:
[CssBaseline] The body font size has changed from theme.typography.body2 (0.875rem) to theme.typography.body1 (1rem).
To return to the previous size, you can override it in the theme:

const theme = createMuiTheme({
  typography: {
    body1: {
      fontSize: '0.875rem',
    },
  },
});

(Note that this will also affect use of the Typography component with the default body1 variant).

Suggested Text 🤔
It would make more sense to address this with an override to CssBaseline itself. As noted, the fix above will change body1 to be the same size as body2 throughout the app. Better to just change the fontSize on body back to the old default:
const theme = createMuiTheme({
  components: {
    MuiCssBaseline: {
      styleOverrides: {
        body: {
          fontSize: '0.875rem',
        },
      },
    },
  },
})

Possibly, to perfectly mimic the old behaviour, it should also include lineHeight and letterSpacing.

Would it be possible to add this as part of AdaptV4Theme and thus part of the codemod?

@webarnes I agree, the approach to old behavior should be done via CSSBaseline not body1 variant.

@webarnes would you like to create a PR for updating the migration guide? :)


Merge pull request #1 from twbs/master

Snap (Jan. 29, 2016).

Hi @dsotingco!
Thanks for wanting to contribute to Bootstrap by sending this pull request!
Unfortunately, your pull request seems to have some problems:

The title of this pull request strongly suggests that you were using Bootstrap to test out Git or GitHub, rather than proposing a legitimate change. If that's accurate, please DON'T do that again! Instead, use your own personal repositories to experiment with how to use Git or GitHub. Using the repos of public projects (such as Bootstrap) for such experiments wastes the time of those projects' maintainers and is thus considered rude.

You'll need to fix these mistakes and revise your pull request before we can proceed further.
Once you've fixed these problems, you can either ask the maintainers to re-open this pull request, or you can create a new pull request.
Thanks!
(Please note that this is a fully automated comment.)


1.1RC Missing argument 1 for loadPMLabels()

Hi,
update 1.1 beta 4 to 1.1RC
personal messages page error
Cache cleared forum maintenance done,  reupload PersonalMessage.controller.php the problem continues

Missing argument 1 for loadPMLabels(), called in /sources/controllers/PersonalMessage.controller.php on line 91 and defined

hi,
Until RC 2 comes out
do you have the code to solve the problem?
thanks

This block:
		// Load the label data.
		if ($user_settings['new_pm'] || ($context['labels'] = cache_get_data('labelCounts:' . $user_info['id'], 720)) === null)
		{
			$this->_loadLabels();

			loadPMLabels();
		}

		// Now we have the labels, and assuming we have unsorted mail, apply our rules!
		if ($user_settings['new_pm'])
		{
			// Apply our rules to the new PM's
			applyRules();

			require_once(SUBSDIR . '/Members.subs.php');
			updateMemberData($user_info['id'], array('new_pm' => 0));

			// Turn the new PM's status off, for the popup alert, since they have entered the PM area
			toggleNewPM($user_info['id']);
		}

		// Load the label counts data.
		if ($user_settings['new_pm'] || !Cache::instance()->getVar($context['labels'], 'labelCounts:' . $user_info['id'], 720))
		{
			$this->_loadLabels();

			// Get the message count for each label
			$context['labels'] = loadPMLabels($context['labels']);
		}

should become:

		// Load the label counts data.
		if ($user_settings['new_pm'] || !Cache::instance()->getVar($context['labels'], 'labelCounts:' . $user_info['id'], 720))
		{
			$this->_loadLabels();

			// Get the message count for each label
			$context['labels'] = loadPMLabels($context['labels']);
		}

		// Now we have the labels, and assuming we have unsorted mail, apply our rules!
		if ($user_settings['new_pm'])
		{
			// Apply our rules to the new PM's
			applyRules();

			require_once(SUBSDIR . '/Members.subs.php');
			updateMemberData($user_info['id'], array('new_pm' => 0));

			// Turn the new PM's status off, for the popup alert, since they have entered the PM area
			toggleNewPM($user_info['id']);
		}


CI/CD conventions for metrics

Area(s)
area:cicd
Is your change request related to a problem? Please describe.
This issue is to discuss attributes specific to metrics and as part of the CI/CD Working Group and Semantic Conventions WG.
Also a challenge specific to metrics can the time series cardinality when CICD observes metrics for individual builds.
Describe the solution you'd like
Following https://github.com/open-telemetry/semantic-conventions/pull/1075 (by adjusting the vocabulary here below to align with #1075) we should define metric attributes for

duration of pipelineRuns (by status, pipeline)
count of pipelineRuns (by status, pipeline)
count of agents
queue length of pending pipelineRuns
duration for how long a pipelineRun is in the queue before starting execution

Additionally it should be possible to opt-in to metrics specific to a particular pipelineRun.
These could be metrics about the agent which executes a pipelineRun, the OS, network, jvm, the number of failed/total tests …
We need to specify the attribute which should link these metrics to the pipelineRun, eg. pipeline.run.id
Metrics specific to a pipelineRun are of high cardinality. We should document this as a warning and give guidance how these metrics can be efficiently encoded in the OTel protocol, ie by using resource attributes instead of metric attributes wherever possible.
Describe alternatives you've considered
Span metrics could be used for duration and count of pipelineRuns, however this relies on the pipelineRuns having completed.
This is due to limitations inherent in using traces to represent pipelineRuns, a span can only be sent when complete.
Due to this limitation it could be preferable for the CICD system to expose metrics directly about the duration, count and status of pipelineRuns. These pipelineRuns could account also for in progress builds.
Additional context
CICD metrics were discussed at KubeCon March 2024 SemConv users meeting.
High cardinality was highlighted as an issue for per build metrics.
Notes on how to deal with cardinality were:

Could we use Exemplars? We could link to the build trace from some metrics.
This added information might make it easier to identify pipelineRuns that need investigation.
Using the resource attribute for the build ID is fine for the OTel protocol,
but backends (eg. Prometheus) would still have the cardinality issue when storing the time series
(metric / resource attributes would be flattened into time series).

We can use label area:cicd instead of area:new.

Currently have a pull request open to change the metrics in the Git Provider Receiver component within the OTEL Collector to better match the new conventions set in the registry. I think this can help provide contextual implementation details as part of this conversation.

@adrielp Can we use #1184 for point 3 "metrics related to individual builds (high cardinality issue)", perhaps renaming the issue or should this be a separate issue?


useMediaMatches

A hook which is simlar to the useMedia hook, but uses listeners on the media query so that it can listen to any type of media change and not only screen size.
import json2mq from 'json2mq';
import { useEffect, useState } from 'react';

function useMediaMatches(query, defaultMatches = false) {
  const [matches, setMatches] = useState(defaultMatches);
  const queryString = typeof query !== 'string' ? json2mq(query) : query;
  useEffect(
    () => {
      const update = event => setMatches(event.matches);
      const mediaQueryList = window.matchMedia(queryString);
      setMatches(mediaQueryList.matches);
      mediaQueryList.addListener(update);
      return () => mediaQueryList.removeListener(update);
    },
    [queryString]
  );
  return matches;
}

json2mq is useful if you want to write your media queries as objects, but is otherwise optional.
import { useEffect, useState } from 'react';

function useMediaMatches(query, defaultMatches = false) {
  const [matches, setMatches] = useState(defaultMatches);
  useEffect(
    () => {
      const update = event => setMatches(event.matches);
      const mediaQueryList = window.matchMedia(query);
      setMatches(mediaQueryList.matches);
      mediaQueryList.addListener(update);
      return () => mediaQueryList.removeListener(update);
    },
    [query]
  );
  return matches;
}

Thank you! Do you see any downsides (performance or otherwise) to this method over a listener on the resize event? At first glance your method seems all around better. If that's the case I'll just go ahead and update the original useMedia post.

I've gone ahead and updated the origin useMedia post. Thanks for the suggestions!


Xiaomi device with batteries reporting less than 2.8V cause negative battery % remaining

Describe the bug
Xiaomi device with batteries reporting less than 2.8V cause negative battery % remaining
Reported in https://github.com/home-assistant/core/issues/44578#issuecomment-751768537
To Reproduce
Steps to reproduce the behavior:

Get xiaomi to report a battery voltage less than 2.8V
Get negative % remaining

Expected behavior
Should not report negative %. Probably reporting 0% is acceptable
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Reported in https://github.com/home-assistant/core/issues/44578#issuecomment-751768537

After this fix for negative battery % my aqara sensors only report 0, 50 or 100%

After this fix for negative battery % my aqara sensors only report 0, 50 or 100%

Debug logs

Debug logs

Sure!
What is needed?
These?
homeassistant.components.zha: debug
bellows.zigbee.application: debug
bellows.ezsp: debug
zigpy: debug
zhaquirks: debug
And send the home-assistant.log after some logging?

Sure!
What is needed?
These?
homeassistant.components.zha: debug
bellows.zigbee.application: debug
bellows.ezsp: debug
zigpy: debug
zhaquirks: debug
And send the home-assistant.log after some logging?

Yes. Send the logs for the device reporting it's battery level.

Yes. Send the logs for the device reporting it's battery level.

2021-01-10 15:58:45 DEBUG (MainThread) [bellows.zigbee.application] EZSP Counters: COUNTER_MAC_RX_BROADCAST: 304,COUNTER_MAC_TX_BROADCAST: 52,COUNTER_MAC_RX_UNICAST: 128,COUNTER_MAC_TX_UNICAST_SUCCESS: 67,COUNTER_MAC_TX_UNICAST_ RETRY: 34,COUNTER_MAC_TX_UNICAST_FAILED: 2,COUNTER_APS_DATA_RX_BROADCAST: 1,COUNTER_APS_DATA_TX_BROADCAST: 1,COUNTER_APS_DATA_RX_UNICAST: 70,COUNTER_APS_DATA_TX_UNICAST_SUCCESS: 34,COUNTER_APS_DATA_TX_UNICAST_RETRY: 5,COUNTER_AP S_DATA_TX_UNICAST_FAILED: 2,COUNTER_ROUTE_DISCOVERY_INITIATED: 7,COUNTER_NEIGHBOR_ADDED: 12,COUNTER_NEIGHBOR_REMOVED: 0,COUNTER_NEIGHBOR_STALE: 0,COUNTER_JOIN_INDICATION: 0,COUNTER_CHILD_REMOVED: 0,COUNTER_ASH_OVERFLOW_ERROR: 0, COUNTER_ASH_FRAMING_ERROR: 0,COUNTER_ASH_OVERRUN_ERROR: 0,COUNTER_NWK_FRAME_COUNTER_FAILURE: 0,COUNTER_APS_FRAME_COUNTER_FAILURE: 0,COUNTER_UTILITY: 0,COUNTER_APS_LINK_KEY_NOT_AUTHORIZED: 0,COUNTER_NWK_DECRYPTION_FAILURE: 0,COUN TER_APS_DECRYPTION_FAILURE: 0,COUNTER_ALLOCATE_PACKET_BUFFER_FAILURE: 0,COUNTER_RELAYED_UNICAST: 0,COUNTER_PHY_TO_MAC_QUEUE_LIMIT_REACHED: 0,COUNTER_PACKET_VALIDATE_LIBRARY_DROPPED_COUNT: 0,COUNTER_TYPE_NWK_RETRY_OVERFLOW: 0,COU NTER_PHY_CCA_FAIL_COUNT: 0,COUNTER_BROADCAST_TABLE_FULL: 0,COUNTER_PTA_LO_PRI_REQUESTED: 0,COUNTER_PTA_HI_PRI_REQUESTED: 0,COUNTER_PTA_LO_PRI_DENIED: 0,COUNTER_PTA_HI_PRI_DENIED: 0,COUNTER_PTA_LO_PRI_TX_ABORTED: 0,COUNTER_PTA_HI _PRI_TX_ABORTED: 0,COUNTER_ADDRESS_CONFLICT_SENT: 0 2021-01-10 15:58:50 DEBUG (MainThread) [bellows.ezsp.protocol] Application frame 69 (incomingMessageHandler) received: b'00040100000101000100006484bdaa8bffff261c5f114b0a01ff421d0121a90b0328150421a81305212300062401000000000a21001 5641000' 2021-01-10 15:58:50 DEBUG (MainThread) [bellows.zigbee.application] Received incomingMessageHandler frame with [<EmberIncomingMessageType.INCOMING_UNICAST: 0>, EmberApsFrame(profileId=260, clusterId=0, sourceEndpoint=1, destinat ionEndpoint=1, options=<EmberApsOption.APS_OPTION_ENABLE_ROUTE_DISCOVERY: 256>, groupId=0, sequence=100), 132, -67, 0x8baa, 255, 255, b'\x1c_\x11K\n\x01\xffB\x1d\x01!\xa9\x0b\x03(\x15\x04!\xa8\x13\x05!#\x00\x06$\x01\x00\x00\x00\ x00\n!\x00\x15d\x10\x00'] 2021-01-10 15:58:50 DEBUG (MainThread) [zigpy.zcl] [0x8baa:1:0x0000] ZCL deserialize: <ZCLHeader frame_control=<FrameControl frame_type=GLOBAL_COMMAND manufacturer_specific=True is_reply=False disable_default_response=True> manu facturer=4447 tsn=75 command_id=Command.Report_Attributes> 2021-01-10 15:58:50 DEBUG (MainThread) [zigpy.zcl] [0x8baa:1:0x0000] ZCL request 0x000a: [[Attribute(attrid=65281, value=<TypeValue type=CharacterString, value=A!�KC(UD!�SE!#>)]] 2021-01-10 15:58:50 DEBUG (MainThread) [zigpy.zcl] [0x8baa:1:0x0000] Attribute report received: 65281=A!�KC(UD!�SE!# 2021-01-10 15:58:50 DEBUG (MainThread) [zhaquirks.xiaomi] 00:15:8d:00:03:2b:89:c9 - Attribute report. attribute_id: [65281] value: [{'battery_voltage_mV': 2985, 'temperature': 21, 'X-attrib-4': 5032, 'X-attrib-5': 35, 'X-attrib- 6': 1, 'path': 5376, '0xff01-100': <Bool.false: 0>}] 2021-01-10 15:58:50 DEBUG (MainThread) [zhaquirks] LUMI lumi.sensor_magnet.aq2, Voltage [RAW]:30 [Max]:3.0 [Min]:2.8, Battery Percent: 100.0 2021-01-10 15:58:55 DEBUG (MainThread) [bellows.ezsp.protocol] Send command readCounters: () 2021-01-10 15:58:55 DEBUG (MainThread) [bellows.ezsp.protocol] Application frame 241 (readCounters) received: b'38013500810043002200020001000100470022000500020007000c00000001000000000000000000000000000000000000000000000000000000

2021-01-10 15:58:45 DEBUG (MainThread) [bellows.zigbee.application] EZSP Counters: COUNTER_MAC_RX_BROADCAST: 304,COUNTER_MAC_TX_BROADCAST: 52,COUNTER_MAC_RX_UNICAST: 128,COUNTER_MAC_TX_UNICAST_SUCCESS: 67,COUNTER_MAC_TX_UNICAST_ RETRY: 34,COUNTER_MAC_TX_UNICAST_FAILED: 2,COUNTER_APS_DATA_RX_BROADCAST: 1,COUNTER_APS_DATA_TX_BROADCAST: 1,COUNTER_APS_DATA_RX_UNICAST: 70,COUNTER_APS_DATA_TX_UNICAST_SUCCESS: 34,COUNTER_APS_DATA_TX_UNICAST_RETRY: 5,COUNTER_AP S_DATA_TX_UNICAST_FAILED: 2,COUNTER_ROUTE_DISCOVERY_INITIATED: 7,COUNTER_NEIGHBOR_ADDED: 12,COUNTER_NEIGHBOR_REMOVED: 0,COUNTER_NEIGHBOR_STALE: 0,COUNTER_JOIN_INDICATION: 0,COUNTER_CHILD_REMOVED: 0,COUNTER_ASH_OVERFLOW_ERROR: 0, COUNTER_ASH_FRAMING_ERROR: 0,COUNTER_ASH_OVERRUN_ERROR: 0,COUNTER_NWK_FRAME_COUNTER_FAILURE: 0,COUNTER_APS_FRAME_COUNTER_FAILURE: 0,COUNTER_UTILITY: 0,COUNTER_APS_LINK_KEY_NOT_AUTHORIZED: 0,COUNTER_NWK_DECRYPTION_FAILURE: 0,COUN TER_APS_DECRYPTION_FAILURE: 0,COUNTER_ALLOCATE_PACKET_BUFFER_FAILURE: 0,COUNTER_RELAYED_UNICAST: 0,COUNTER_PHY_TO_MAC_QUEUE_LIMIT_REACHED: 0,COUNTER_PACKET_VALIDATE_LIBRARY_DROPPED_COUNT: 0,COUNTER_TYPE_NWK_RETRY_OVERFLOW: 0,COU NTER_PHY_CCA_FAIL_COUNT: 0,COUNTER_BROADCAST_TABLE_FULL: 0,COUNTER_PTA_LO_PRI_REQUESTED: 0,COUNTER_PTA_HI_PRI_REQUESTED: 0,COUNTER_PTA_LO_PRI_DENIED: 0,COUNTER_PTA_HI_PRI_DENIED: 0,COUNTER_PTA_LO_PRI_TX_ABORTED: 0,COUNTER_PTA_HI _PRI_TX_ABORTED: 0,COUNTER_ADDRESS_CONFLICT_SENT: 0 2021-01-10 15:58:50 DEBUG (MainThread) [bellows.ezsp.protocol] Application frame 69 (incomingMessageHandler) received: b'00040100000101000100006484bdaa8bffff261c5f114b0a01ff421d0121a90b0328150421a81305212300062401000000000a21001 5641000' 2021-01-10 15:58:50 DEBUG (MainThread) [bellows.zigbee.application] Received incomingMessageHandler frame with [<EmberIncomingMessageType.INCOMING_UNICAST: 0>, EmberApsFrame(profileId=260, clusterId=0, sourceEndpoint=1, destinat ionEndpoint=1, options=<EmberApsOption.APS_OPTION_ENABLE_ROUTE_DISCOVERY: 256>, groupId=0, sequence=100), 132, -67, 0x8baa, 255, 255, b'\x1c_\x11K\n\x01\xffB\x1d\x01!\xa9\x0b\x03(\x15\x04!\xa8\x13\x05!#\x00\x06$\x01\x00\x00\x00\ x00\n!\x00\x15d\x10\x00'] 2021-01-10 15:58:50 DEBUG (MainThread) [zigpy.zcl] [0x8baa:1:0x0000] ZCL deserialize: <ZCLHeader frame_control=<FrameControl frame_type=GLOBAL_COMMAND manufacturer_specific=True is_reply=False disable_default_response=True> manu facturer=4447 tsn=75 command_id=Command.Report_Attributes> 2021-01-10 15:58:50 DEBUG (MainThread) [zigpy.zcl] [0x8baa:1:0x0000] ZCL request 0x000a: [[Attribute(attrid=65281, value=<TypeValue type=CharacterString, value=A!�KC(UD!�SE!#>)]] 2021-01-10 15:58:50 DEBUG (MainThread) [zigpy.zcl] [0x8baa:1:0x0000] Attribute report received: 65281=A!�KC(UD!�SE!# 2021-01-10 15:58:50 DEBUG (MainThread) [zhaquirks.xiaomi] 00:15:8d:00:03:2b:89:c9 - Attribute report. attribute_id: [65281] value: [{'battery_voltage_mV': 2985, 'temperature': 21, 'X-attrib-4': 5032, 'X-attrib-5': 35, 'X-attrib- 6': 1, 'path': 5376, '0xff01-100': <Bool.false: 0>}] 2021-01-10 15:58:50 DEBUG (MainThread) [zhaquirks] LUMI lumi.sensor_magnet.aq2, Voltage [RAW]:30 [Max]:3.0 [Min]:2.8, Battery Percent: 100.0 2021-01-10 15:58:55 DEBUG (MainThread) [bellows.ezsp.protocol] Send command readCounters: () 2021-01-10 15:58:55 DEBUG (MainThread) [bellows.ezsp.protocol] Application frame 241 (readCounters) received: b'38013500810043002200020001000100470022000500020007000c00000001000000000000000000000000000000000000000000000000000000

Please let me know if you need more. Thanks!

Please let me know if you need more. Thanks!

battery_voltage_mV': 2985
Rounded , that's 2.99v. that's 100% per thresholds I've described earlier.

battery_voltage_mV': 2985
Rounded , that's 2.99v. that's 100% per thresholds I've described earlier.

Sorry, bad example. I will wait some more to get an other voltage update.

Sorry, bad example. I will wait some more to get an other voltage update.

≤2.8 is 0%
2.9 is 50%

3.0 is 100%

Right?
Should not 2.99 be 95% ?
And I have a sensor here reporting 2965mV.
Should not 2.99 be 95% ?
And I have a sensor here reporting 2965mV.  Should be 25% if rounded to 2.85V
Perhaps you are rounding to one decimal instead of two? Maybe that's why we only get 0,50 or 100% ?
2021-01-11 08:16:15 DEBUG (MainThread) [zhaquirks.xiaomi] 00:15:8d:00:04:86:86:0d - Attribute report. attribute_id: [65281] value: [{'battery_voltage_mV': 2845, 'X-attrib-4': 17320, 'X-attrib-5': 7, 'X-attrib-6': 1, 'temperature_measurement': -1787, 'humidity_measurement': 4714, 'pressure_measurement': 100534, 'path': 35172}]
2021-01-11 08:16:15 DEBUG (MainThread) [zhaquirks] LUMI lumi.weather, Voltage [RAW]:28 [Max]:3.0 [Min]:2.8, Battery Percent: 0.0

2845mV gives 0% now. Should be 25% if rounded to 2.85V
Perhaps you are rounding to one decimal instead of two? Maybe that's why we only get 0,50 or 100% ?
2021-01-11 08:16:15 DEBUG (MainThread) [zhaquirks.xiaomi] 00:15:8d:00:04:86:86:0d - Attribute report. attribute_id: [65281] value: [{'battery_voltage_mV': 2845, 'X-attrib-4': 17320, 'X-attrib-5': 7, 'X-attrib-6': 1, 'temperature_measurement': -1787, 'humidity_measurement': 4714, 'pressure_measurement': 100534, 'path': 35172}]
2021-01-11 08:16:15 DEBUG (MainThread) [zhaquirks] LUMI lumi.weather, Voltage [RAW]:28 [Max]:3.0 [Min]:2.8, Battery Percent: 0.0

yeah, right now it operates in 1/10th of the V and thus has only 100%, 50% and 0%. And it doesn't go negative anymore.
2.99v is a bad example, cause that gets rounded to 3.0V

yeah, right now it operates in 1/10th of the V and thus has only 100%, 50% and 0%. And it doesn't go negative anymore.
2.99v is a bad example, cause that gets rounded to 3.0V

Possible to increase the resolution?

Possible to increase the resolution?

See https://github.com/zigpy/zha-device-handlers/pull/720

See https://github.com/zigpy/zha-device-handlers/pull/720

Excellent! Thank you! 👍 🥇

Excellent! Thank you! 👍 🥇


ARROW-6428: [Python][CI] Use PR branch for turbodbc integration test

Fix integration build requiring https://github.com/blue-yonder/turbodbc/pull/226

@ursabot crossbow submit docker-turbodbc-integration


Add global model in round parameter

I think this is OK for now given all the limitations of our implementation, but on the long run we'll have to change this. Model is the "internal" representation of a model. When serializing it to send it to the participants, we actually want to convert it to primitive numeric types i32, i64, f32 or f64 depending on the type of model. This is currently not possible though, so sending the Model is all we can do.


better trl parser with yaml config

correctly overrides yaml config with command line arguments
do this by changing the defaults to values in the yaml so we get dataclass default < yaml config < command line arguments

adds return_remaining_strings

when return_remaining_strings is False, raises error if yaml contains extra args that are not in the dataclasses

removes the need to have a config field in one of your dataclasses (I can revert this if this is preferred)

simpler and cleaner than previous yaml parsing without the need for merging dataclasses

addresses #1733
@younesbelkada let me know if you have comments or suggestions!

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


[3.x] GDScript: Prevent native class shadowing

Fixes #57496.

In 4.x, an attempt to define a member that shadows a native class results in an error. In 3.x, this is done for nested classes, but not for constants, which can also be used as type hints.
Proper implementation of shadowing is too difficult and risky for the LTS branch, especially since it's not implemented even in 4.x. Showing error is the easiest option. Let me know if you think a warning is better here.

Thanks!


test: update genesis wasm contract

Description

adds the compiled tendermint wasm contract bytes and updates genesis.json to have this contract already hardcoded
Commit Message / Changelog Entry
type: commit message

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

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

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

I think we don't need this anymore, this PR can probably be closed @charleenfei

I think we don't need this anymore, this PR can probably be closed @charleenfei

Yeah, I think you're right. We checked in the contract to the testing repo :)
Will close


Setting download from video # untill video # doesn't work as intended.

When you put anything under download to video # as soon as you close setting tab, the number becomes 0 and instead number download from video # becomes the number you put.

can confirm

Should be fixed now, update the program

Yup works. Thanks for quick fix.


Make psycopg2 to optional

Following error occurred when import from django_sharding_library.decorators import shard_storage_config.
I'm not use psycopg2 :(
django.core.exceptions.ImproperlyConfigured: Error loading psycopg2 module: No module named 'psycopg2'

Coverage decreased (-0.3%) to 89.487% when pulling 4811f84bd1e6c397a6f7b50893d2368527241201 on uncovertruth:master into eb6d9ab35163832a62c750f6f89c3b75e2cc38a5 on JBKahn:master.

This looks fine I'll look to merging it in this week

Since it's not actually an import error I'll fix it in a seperate PR.


The file share snapshot doesn't auto display when creating a snapshot under the snapshots view

Storage Explorer Version: 1.25.0-dev
Build Number: 20220609.2
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Monterey 12.4 (Apple M1 Pro)
Architecture: ia32\x64
How Found: From running test cases
Regression From: Previous release (1.24.2)
Steps to Reproduce

Expand one storage account -> File Shares.
Create a file share -> Click 'View Share Snapshots'.
Right click the file share -> Click 'Create Snapshot'.
Check whether the snapshot auto displays.

Expected Experience
The snapshot auto displays.
Actual Experience
The snapshot doesn't auto display.

Verified this issue on the build main/20220610.4. Fixed. So close it.


Upgrade Markdown to latest version

Upgrading Markdown from 3.0.1 to 3.1 results in the following error.
https://travis-ci.org/zulip/zulip/builds/523579779

@zulipbot label "area: dependencies"

@brockwhittaker @PentaHiggs FYI

Reading that error, I suspect the issue is that the new release isn't installable on Ubuntu Trusty.  Does it pass tests on other platforms?

@aero31aero @PentaHiggs this could be a good project to pick up, now that Ubuntu Trusty is removed as a supported platform.


Tab title slots

Is there any chance you could implement tab titles as optional slots (keeping the "title" prop for backward compatibility) please?
I often find myself needing to put dynamic content into the tab titles (e.g. counts, warning icons etc, ...) and the only way to do this is to have (sometimes complex) computed values returning HTML for the titles.
Thank you!

Pls try title slot in next version.


Filing divorce papers for languages and countries.

EXPLOSIVE CHANGES
:boom: :boom: :boom: :boom: :boom: :boom: :boom:
This PR is removing the connection between languages and countries in the code. For further global expansion this is a must. Example:
If a Canadian user visits the site, do we serve them french or english?
Our current logic wouldn't know, and just return the first lang listed under the country. Many distinctions and assumptions are made throughout the code that one language always relates to one country (except global). For 1.0 this is ok, but going beyond that we need to update this.
To solve it, I'm relying on a header value called "Accept-Language" that's basically guaranteed to be sent by the browser. You can see the reasoning I chose to use this in the issue for this PR here (and more details on how it works) https://github.com/DoSomething/phoenix/issues/5664#issuecomment-153436293
In the event a Accept-Language header is not sent by the browser, there is a fallback. First, for authenticated users it will try your account language. If that is not set, or you're an anonymous user, it will return global english.
Here is where things get complicated, for testing this you now need to change your browser language in addition to using Torgaurd. Yes, this makes testing more annoying, but it's also the most accurate simulation we can make next to actually being in the country.
After this PR is merged I'll be able to begin the large global documentation readme & diagramming that explains which functions should be used in what cases, in the event parts of this are still unclear.
EXPLOSIVE CHANGES
:boom: :boom: :boom: :boom: :boom: :boom: :boom:
Do not merge yet, still testing and developing

@deadlybutter Thor deployments only work from the Phoenix repo.

gotcha, when its ready for testing I'll make another branch here than real quick

Basic testing on Thor really quick seems to be fine. Will do more later

sena im assigning you to this so we remember to investigate this later, gonna hit pause on this for now and get some of the bugs out of the way
cc @mikefantini @angaither

@deadlybutter Fantastic that you're getting to this issue. It warms my Canadian heart.
If I understand the description correctly this makes the assumption that the browser language setting takes precedence over the DoSomething user account (session) language preference? This seems problematic if the user logs into different computers with different language preferences at the browser level (a possibility for a typical Canadian student in a shared school computer lab).
Wouldn't the user specific language preference first, then browser setting second not be a more accurate choice?

No with this change the user language still should always take precedence. This mostly impacts anonymous users.
Also not sure when / if this will be merged, had complications and then we had higher priority things to take care of.


Add clean shutdown of channels

The idea is to perform a clean shutdown on channels once the program is existing.
More information here

I'm closing this issue because the issue relates to an old version (v0.8.0) of the rust coordinator that is no longer maintained.
If there is a similar issue in the upcoming version, please open a new ticket instead of opening it again.


Show calling function in warning

Current code makes warning show a line of dotenv source code, rather than the code that calls dotenv.read_env. The stacklevel param fixes that.

@merwok awesome, thanks!


Add a last modified by

Add a "last modified by" feature to the history system

I added a LastModified function to the the History.py server file in the Lib folder.
I also added a LastModifiedArray function to the History.py server file

I created routes in the Business.py and Human.py files called /Business/LastModified and /Human/LastModified.
Still need to add this to the other parent routes

Still need to add this to the front end.

I think this is resolved.


Problem with users

I want to perform the scraping of three Twitter users, but the problem is that I don't get all their tweets, only about 2500 tweets. I don't know what the problem might be.
I'm using the following command:
twint -u username -o file.csv --csv
For these three users:

AytoGuadix

AytoCoin

losbarriosmejor

I only get these tweets for each user:

AytoGuadix --> 4626 tweets (last date: 2016-08-08)

AytoCoin  --> 2594 tweets (last date: 2017-01-09)

losbarriosmejor --> 2344 tweets (last date: 2016-05-01)

Someone would know why it doesn't allow me to get more tweets?

Would you do a manual search in the browser to see if the latest tweet that you find is the latest tweet that you scrape?

They are not the last tweets. They have older tweets. For that reason I do not understand why it does not scraped more.

Twitter usually blocks IPs that make too many requests, this happens even when I scroll the feed with the native app


Replace Overview page with blank slate pattern on Recipes

The Overview page is very empty and is mostly just tailored for
usability testing with pre-existing recipes. For real deployments with
no pre-existing recipes it's a more common workflow to put the
instructions as a "blank slate pattern" on the actual recipes page.
This works towards avoiding the navigation menu completely (together
with dropping the Users page).
Rename home.index.spec.js unit test to index.norecipes.spec.js and
verify that blank slate is generated with zero recipes.

As discussed in yesterday's meeting. Currently the empty recipes page looks like this:

Wit this PR, it looks like this, and the separate "Overview" menu enty is gone:

We can/will surely adjust the text in the box further (that requires some designing), but this at least works towards cleaning up the navigation, together with PR #147.

Codecov Report

Merging #148 into master will increase coverage by 0.64%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #148      +/-   ##
==========================================
+ Coverage   41.07%   41.72%   +0.64%
==========================================
  Files          78       77       -1
  Lines        2325     2284      -41
  Branches      301      298       -3
==========================================
- Hits          955      953       -2
+ Misses       1168     1135      -33
+ Partials      202      196       -6

Impacted Files
Coverage Δ

routes.json
56.52% <ø> (+2.35%)
:arrow_up:

components/Layout/Navigation.js
50% <ø> (+7.14%)
:arrow_up:

pages/recipes/index.js
66.66% <ø> (+7.57%)
:arrow_up:

core/selectors.js
92.85% <0%> (+2.15%)
:arrow_up:

data/MetadataApi.js
37.28% <0%> (+2.36%)
:arrow_up:

data/RecipeApi.js
56.98% <0%> (+3.98%)
:arrow_up:

core/actions/modals.js
95.45% <0%> (+4.15%)
:arrow_up:

components/Modal/CreateRecipe.js
84.12% <0%> (+5.02%)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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

Thanks for the update! I'm good with finalizing the empty state message in another PR.


Automate Deposits Sweeps CLI

The changes here are a step on the road to the fully automated sweeps (https://github.com/keep-network/keep-core/issues/3614).
We update the propose-deposits-sweep command initially implemented in https://github.com/keep-network/keep-core/pull/3549.
The main change to the command is that if deposits are not provided as arguments, they will be automatically fetched from the chain.
The command will find the oldest unswept deposits for the oldest unswept wallet and construct a sweep proposal for them.
Number of deposits for the proposal is based on the depositSweepMaxSize defined in the WalletCoordinator contract. The value can be overwritten with --deposit-sweep-max-size <number>.
If --wallet <wallet-public-key-hash> is provided the command will find unswept deposits for the particular wallet.
If --fee <number> is not provided the command will estimate a fee for the sweep transaction.
Sample Command
To execute the command in dry-run mode:
# Sweep deposits from the oldest unswept wallet:
LOG_LEVEL=debug go run . coordinator --testnet propose-deposits-sweep --dry-run

# Sweep deposits from a specific wallet:
LOG_LEVEL=debug go run . coordinator --testnet propose-deposits-sweep --wallet 0x7670343fc00ccc2d0cd65360e6ad400697ea0fed --dry-run

Thanks for the review @pdyraga. I consider the work for this PR completed. It's ready for the next round.


Add () and restrict near-matching in subqueries

This adds toggle-able negation, fixes 0-score normalization, SameEvoTree tokens, and adds single-subquery pruning
Closes #357
Closes #1773
Closes #1774

Tests this fixes:


chore(deps): remove delay and cpy-cli

Fixes #520  🦕

Closing because improvements are trivial. This repo's dependencies are already quite lean.


default to async?

TinCanJS supports synchronous calls and the basic usage example in the documentation is synchronous.
Synchronous XMLHttpRequests are now deprecated in Chrome and Firefox.
We should update the documentation example to only show asynchronous examples and perhaps consider deprecating synchronous calls in TinCanJS in favour of making all calls asynchronous by default. This could be a breaking change for some implementers, but they will break in some future version of Chrome and Firefox anyway.

I've been tracking a bit of an internal roadmap for TinCanJS and this was very much on it. Removing synchronous capability I imagine would be a huge backwards compatibility issue which is why I've not gone too far down the path of doing it, but yes I expect we'll get there at some point. I'm going to open a "roadmap" issue tracker in case anyone wants to comment. There is an example asynchronous call on the home page currently: http://rusticisoftware.github.io/TinCanJS/ though I can't say whether people understand that all of the other (than query) calls have the same capability. They would if they read the API doc, that's a big if :-).

@garemoko see #110.

Should I now close this issue?

Up to you.


Show proper variable names on breakpoints

Currently, when debugging we can see:

x$1 is a lambda parameter and should actually be called definition
This is caused by the fact that Java Debug Server does not understand Scala symbols. We might be able to somehow override the default IVariableFormatter and maybe use scalap to overcome this?

My progress on this issue so far.
Reproducing
In Scala 2.13, I came up with this little piece of code to reproduce the described behavior:

First breakpoint: method body

All names are correct:

str is a method parameter
greet is a local variable

Second breakpoint: first lambda

Here variable names are synthesized:

greet$1 is a captured variable in the lambda closure => should appear as greet
x$1 is the anonymous lambda parameter => it could appear as _
MODULE$ is a reference to Main

Third breakpoint: second lambda

Again some variable names are synthesized but the lambda parameter is correctly called name.
Comparing with IntelliJ
IntelliJ shows this:

Implementation
I don't think we can use scalap because it does not handle local variables, but only class fields.
Alternatively we can try:

custom variable formatter: truncate the variable name when it contains $
explore the debug info with asm to see if we can find the position of the original identifier in the source code.

I think we do a similar thing in Metals under completionItemResolve  using the presentation compiler, I wonder if we should somehow handle it in Metals? We do have that kind of information and I am not sure if this will not be more effort in here. :thinking:


Mehsana and Mahesana is same(in Gujarat state)

Describe the bug
A clear and concise description of what the bug is.
Included new city with wrong spelling name.
To Reproduce
Steps to reproduce the behavior:

Go to 'www.covid19India.org'
Click on 'gujarat'
Scroll down to 'Mehsana'
See error

Expected behavior
Removing Mahesana and changing count of mehsana in Gujarat state.
Screenshots
Added the screen shot from your website and State portals website
https://gujcovid19.gujarat.gov.in/

Data Issue resolved


Restore memory of last_flow in TVIntegrator::GetFlow() + simplify volume update

Fix a bug seen while adding unit tests to the sensors, while I am not 100% satisfied with the tests, this fix cannot really wait.
Per reviewers demand, finally added the tests I was working on.

CC: @jlebar

@jkff is this ok to land?


Bug: Gene Datatable hanging on "processing" step

When accessing the browser via a PC computer in lab, I found that the datatable for gene selection stalls when dealing with large datasets.
<20 seconds later> Turns out the problem is with setting the default number of values to display at 25 - that's just too many. Reduce to 5 or 10. Do this for all the datatables, just for safety.

Hopefully fixing this with https://github.com/astrasb/Strongyloides_RNAseq_Browser/commit/64fbfdce9452bc8113353961b76cd37413dd44e3
Reduced default page length to 10. Also, expanded number of columns in the gene expression data table that are cut off after 20 characters. Now includes Interpro column.


SAK-50309 wysiwyg Upgrade to CKEditor 5

https://sakaiproject.atlassian.net/browse/SAK-50309

I've submitted this PR as a draft so that people can start to take a look if they want. You need to build webcomponents and restart tomcat. Then use the Conversations tool and add a topic. The description field there is using the ck5 mode of sakai-editor. Just kick the tires, if you have the time. Try dark mode. One big thing still missing is elfinder, which I'll start looking at when I've attended to some of my other responsibilities.

Awesome! Will check this out soon. Looks like at least elfinder has a pattern for this that you may have seen. https://github.com/Studio-42/elFinder/wiki/Integration-with-CKEditor-5
I added a comment to the SAK with some other plugins we'll have to address and others we can either drop or will have to re-do from scratch.


App name

What should the app be named, Smoothie?

@stephanegigandet We can also discuss this in slack or something like that.
I don't find Jungle so suitable, I understand your thoughts behind it, but for someone who sees the app in the Play / App store it is rather confusing.


Correct README.md syntax error

Shoot, I was just about to comment on this. I think maybe the add was unneeded? the rest of the docs show new Collection([items])
It works as-is idk if we care to update it again.

You're right. I think it wouldn't be bad to be consistent. New PR incoming.


Set the schema for the result column to be the same as the ground truth column if it exists.

In the original Sklearn prediction operator, all prediction results were cast to the String type to maintain consistency. We are now introducing an option for users to specify whether they have a ground truth column. If a ground truth column exists, the data type of the prediction column will be aligned with the data type of the ground truth column. In this PR, we add a feature that ensures if a ground truth column is present, the prediction column will have the same data type as the ground truth column. If no ground truth column exists, the prediction result will be cast to a String.

@aglinxinyuan please have a pass