iOS Background Location Tracking Stops When Using 'When in Use' Permission

Your Environment

Plugin version:
capacitor background geolocation: 6.0.0
Platform: iOS
Device OS version: 16.5
Device manufacturer / model: iPhne X, iPhone 15
XCode version: 15.4
Capacitor info (npx cap doctor):
@capacitor/cli: 6.1.2, @capacitor/core: 6.1.2, @capacitor/ios: 6.1.2
Plugin config provided to #ready:

await BackgroundGeolocation.ready({
      startOnBoot: false,
      locationAuthorizationRequest: 'WhenInUse'',
      notification: {
        smallIcon: 'drawable/ic_turo_noti_3',
        title: notificationTitle,
        text: notificationText,
        sticky: true
      },
      desiredAccuracy: BackgroundGeolocation.DESIRED_ACCURACY_HIGH,
      stopTimeout: 10,
      stopOnTerminate: false,
      stopOnStationary: false,
      debug: false,
      reset: true,
    });

Expected Behavior
The app tracks and display the user's position path on the map continuously, even when running in the background, while only requesting the "When in Use" location permission
Actual Behavior
When changing the location authorization request from "Always" to "When in Use" and updating the configuration accordingly, the app stops tracking the user's location immediately after closing the app on iOs. This results in only straight lines between the last recorded locations when the app is reopened, instead of a continuous path.
Steps to Reproduce

Set locationAuthorizationRequest to When in Use in the app configuration.
Implement the geolocation plugin to track user positions and draw the path on the map.
Run the app and start tracking a route.
Minimize or close the app.
Reopen the app to observe the location tracking behavior.

Context
I have an app that displays the user's position path on the map over time and calculates the distance traveled. I have set locationAuthorizationRequest to "Always," and the plugin works great.
However, the customer prefers not to prompt users for the "Always" location permission due to privacy concerns. They provided the Strava app as an example, which successfully tracks location in the background with only "When in Use" permission by running a foreground service with a persistent notification. However, replicating this behavior in my app does not yield the same results, especially on iOS, where tracking ceases once the app is closed.
Is there a way to prompt for location permission as "When in Use" and still maintain reliable background tracking of the user's position?

Start by upgrading to the latest version:
https://github.com/transistorsoft/capacitor-background-geolocation/blob/master/CHANGELOG.md

With WhenInUse, the user must manually engage location-tracking while your app is in the foreground, like a “Jogging App” does with some sort of [Start Workout] button.
in that UI button, you will call .changePace(true).  See the wiki “Philosophy of Operation” for more information.
with WhenInUse, an app absolutely cannot automatically engage location tracking in the background when device motion is detected.

This works. Thank you.

This works. Thank you.

The plugin must be enabled: true before calling .changePace(true).  Calling .start() sets State.enabled == true.
onClickStartTracking() {
  await BackgroundGeolocation.start();
  await BackgroundGeolocation.changePace(true);
}

onClickStopTracking() {
  BackgroundGeolocation.stop() // You don't need to call .changePace(false)
}


Added ability to specify zones

Ordering and endorsing organisation roles now have the ability to specify zones
Added a task which check if the number of zones specified doesn't match the number of peers

Untested and found some bugs. Will open a new PR with the changes since this one is so old and dealing with merging will be a pain


Speedup tests

This PR speeds up tests reducing the combination of PublisherSubscriberIntegration test to a minimum but complete set.

Build status:

Linux
Windows

Build status:

Linux
Windows


Crm457 1146/permissions

Description of change
Add permission to updating the application
Link to relevant ticket
CRM457-1145

Closing as we've done this in Ruby instead


Reformat the code after bumping line width to 120

Ticket
#69?
Problem description
After bumping line width to 120, existing code might fail the format check.
What's changed
Reformat the code with the new line width 120.

Thank you @jdh8 !
I merged a different PR with a fix to the workflow and reformat
Closing this.


XXLarge headings

Description
For a project we would like a larger version of the heading. This is useful for e.g hero-sections. Also there exists an xxsmall variant, so why not an xxlarge one? :)
Can contribute a PR if you agree. We would need to decide on sizing first
Additional Information
No response

Ahoi!
Thanks for the request.
We have a typography scale defined in tokens/figma so it would have to be added that way.
We'll have a closer look at this as I assume some might even want bigger headings!

@stianmorsund
Our typography scale is currently defined up to f5 (38px/2.38rem). I added a f6 now (#1374), which will be 44px/2.75rem. Is this enough for your case or should we add even more levels to the scale?

Also, Im not quite sure where to put this size in the semantic set.

Should we add it as a xxl heading? In that case, should xl still be the default H1 size? In cases where those big text are used, will it always be headings? 🤔
Or... should we add a "typography/hero" folder for sizes f6, f7, f8, f9...? 🤔

Correction: Size f6 will be 43px (not 44).
Her you can se more levels added to the scale and their sizes at different viewports:

How large should we go? :D

Here´s an example of an article with
H1 - f5
H2 - f4
H3 - f3
H4 - f2
H5 - f1
H6 - f0

Hi, thanks for the reply! :)
I have talked to our designer and she says that a minimum of 48px would be great for our use case, so f7 would be the closest. Right now we are using the same clamping function with custom sizings like this:
--heading-xxlarge: 500 clamp(1.8rem, calc(1.55vw + 1.8rem), 2.8rem) /1.3 'Inter';

Splitting out into a typography/hero-folder might  be useful to avoid the "coupling" with the article typography, since hero sections sometimes use larger fonts. I don't know how normal that is for designsystems though. Looking at gov.uk they have solved this by providing some helper classes for font-sizing: https://design-system.service.gov.uk/styles/typography/#font-override-classes.

Hi, thanks for the reply! :)
I have talked to our designer and she says that a minimum of 48px would be great for our use case, so f7 would be the closest. Right now we are using the same clamping function with custom sizings like this:
--heading-xxlarge: 500 clamp(1.8rem, calc(1.55vw + 1.8rem), 2.8rem) /1.3 'Inter';

Splitting out into a typography/hero-folder might be useful to avoid the "coupling" with the article typography, since hero sections sometimes use larger fonts. I don't know how normal that is for designsystems though. Looking at gov.uk they have solved this by providing some helper classes for font-sizing: https://design-system.service.gov.uk/styles/typography/#font-override-classes.

Thanks for the feedback! We don't have an issue for it yet, but we have talked about providing more utility/helper classes just like tailwind and ukgov as you mentioned. Not just typography, but also things like borders, border-radius, spacing etc.
We'll have a look again at what makes sense, best would be to find a solution that works now and for future needs also :)

Hi, thanks for the reply! :)
I have talked to our designer and she says that a minimum of 48px would be great for our use case, so f7 would be the closest. Right now we are using the same clamping function with custom sizings like this:
--heading-xxlarge: 500 clamp(1.8rem, calc(1.55vw + 1.8rem), 2.8rem) /1.3 'Inter';

Splitting out into a typography/hero-folder might be useful to avoid the "coupling" with the article typography, since hero sections sometimes use larger fonts. I don't know how normal that is for designsystems though. Looking at gov.uk they have solved this by providing some helper classes for font-sizing: https://design-system.service.gov.uk/styles/typography/#font-override-classes.

Thanks for the feedback! We don't have an issue for it yet, but we have talked about providing more utility/helper classes just like tailwind and ukgov as you mentioned. Not just typography, but also things like borders, border-radius, spacing etc.
There are also some problems using the font shorthand which we already plan to address #957
We'll have a look again at what makes sense, best would be to find a solution that works now and for future needs also :)

We will arrange a meeting where we find a solution for this.

Nice 🎉  Let me know if you need more input from us :)


Issue with examples

I tried compiling "ReadWriteUSB.ino" on a Teensy 4.1 and got an error: 'msController' does not name a type.

I'm using a clean install of Aduino IDE 2.0.0 and teensyduino 1.57. None of the examples related to mass storage devices will compile; they all have the error "'msController' does not name a type". Any ideas what is happening?

Sorry for the slow response. As @KurtE said, you need to use the examples in TD 1.57 USBHost_t36. I will try to adapt some of the examples you posted about and post them on the the forum thread.


Failure to startup with error 'spawn java ENOENT'

Following the readme but I always get this. Didn't have trouble with the sauce labs karma-runner.
09 10 2016 22:47:22.100:INFO [karma]: Karma v1.3.0 server started at http://localhost:9876/
09 10 2016 22:47:22.104:INFO [launcher]: Launching browsers TB_chrome, TB_firefox with unlimited concurrency
09 10 2016 22:47:22.124:INFO [launcher]: Starting browser chrome on TestingBot
Error: spawn java ENOENT
    at exports._errnoException (util.js:1036:11)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
    at onErrorNT (internal/child_process.js:359:16)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)
Shutting down tunnel

Ah, I'm using a docker image that doesn't have java. I'll close this ticket. Perhaps it would be good to note this on the readme. Also as some feedback - I think this is quite a heavy requirement and will not be using testing bot as a result. Thank you for your prompt response!


Add documentation for external users

If I am an external (ie, non Caltech) user, how do I obtain given artifacts? At the moment, the answer to this question is not immediately clear.

ClimaArtifacts helper is indeed in the ClimaArtifacts package, as found at the link.
The Manifest.toml files associated to each artifact should come with this package already. If not, you can dev it with ] dev ClimaArtifactsHelper.jl in the REPL in the main folder of this package.
Note, however that the creation of the artifact in itself with ClimaArtifactsHelper won't be particularly useful to you: if you run all the code before the create_artifact_guided, the script will produce a local artifact.

I am still confused with this package. I overcame the problem with the 'helper', not with the suggestion which didn't work for me, but by cloning ClimaArtifacts not ClimaArtifacts.jl - the latter didn't work and I had attempted to then load the files by hand.
This worked for aerosol_concentrations leading to a successful instaniate and a status with
for the 'helper' having the line
[6ffa2572] ClimaArtifactsHelper v0.0.1 `..\ClimaArtifactsHelper.jl`

The creation of the artifact was also successful.
However, when I next tried earth_orography with eactly the same line in the respective Project.toml line for the 'helper', I was back to the same problem as before.
I am running the following:
Julia Version 1.10.5
Commit 6f3fdf7b36 (2024-08-27 14:19 UTC)
Build Info:
  Official https://julialang.org/ release
Platform Info:
  OS: Windows (x86_64-w64-mingw32)
  CPU: 16 × Intel(R) Core(TM) i7-10700 CPU @ 2.90GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-15.0.7 (ORCJIT, skylake)
Threads: 1 default, 0 interactive, 1 GC (on 16 virtual cores)

Here is some more info for my earth_orography attempts:
PS C:\Users\kevin\cc\ccex5\ClimaArtifacts\earth_orography> julia --project
_
_       _ ()_     |  Documentation: https://docs.julialang.org
()     | () ()    |
_ _   | |  __ _   |  Type "?" for help, "]?" for Pkg help.
| | | | | | |/ ` |  |
| | || | | | (| |  |  Version 1.10.5 (2024-08-27)
/ |_'|||_'_|  |  Official https://julialang.org/ release
|__/                   |
(earth_orography) pkg> instantiate
┌ Warning: could not download https://pkg.julialang.org/registries
│   exception = RequestError: HTTP/1.1 301 Australian region -> AU internal redirect trigger (Failed to connect to au.pkg.julialang.org port 443 after 2209 ms: Couldn't connect to server) while requesting https://pkg.julialang.org/registries
└ @ Pkg.Registry C:\Users\Kevin.DESKTOP-SVS3IFD.julia\juliaup\julia-1.10.5+0.x64.w64.mingw32\share\julia\stdlib\v1.10\Pkg\src\Registry\Registry.jl:69
Updating registry at C:\Users\Kevin.DESKTOP-SVS3IFD\.julia\registries\General
Updating git-repo https://github.com/JuliaRegistries/General.git
ERROR: expected package ClimaArtifactsHelper [6ffa2572] to be registered

ClimaArtifactsHelper is distributed with this repo. When you see ERROR: expected package ClimaArtifactsHelper [6ffa2572] to be registered, what you should do is add it to your environment.
Here's the steps:

cd ClimaArtifacts
cd earth_orography
julia --project
In the REPL, press ], this will move you to Pkg mode:
dev ..
instantiate
With this, you should have a working environment.

Thanks. I tried that and quite a few variants including using su mode. No success.  Maybe its the Windows OS ?
PS C:\Users\kab\cc\ccex4\ClimaArtifacts\earth_orography> julia --project
               _
   _       _ _(_)_     |  Documentation: https://docs.julialang.org
  (_)     | (_) (_)    |
   _ _   _| |_  __ _   |  Type "?" for help, "]?" for Pkg help.
  | | | | | | |/ _` |  |
  | | |_| | | | (_| |  |  Version 1.10.5 (2024-08-27)
 _/ |\__'_|_|_|\__'_|  |  Official https://julialang.org/ release
|__/                   |

(earth_orography) pkg> dev ..
ERROR: could not find project file (Project.toml or JuliaProject.toml) in package at `..` maybe `subdir` needs to be specified

(earth_orography) pkg>

Oops, sorry! Indeed, it should have been dev ../ClimaArtifactsHelper.jl

Good, that worked.  Thanks.
One more question for someone new to this part of Julia:
To upload to data.caltech.edu (the Caltech Data Archive), please give me an example command, i.e. what is precisely meant by "the link"? Is it something like
c:\users\kevin\cc\ccex5\ClimaArtifacts\earth_orography\earth_orography_60arcsec_artifact?
Here is what I see on the screen:
Now, upload earth_orography_60arcsec_artifact.tar.gz to the Caltech Data Archive or Box, paste here the link, and press ENTER

Note that the given path is a subdirectory which contains a *.nc file, not a tarball.


[AutoPR mysql/resource-manager] [Don't merge until 06/29/2018] Add mysql replica APIs into spec

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3216

Can one of the admins verify this patch?

Closing PR that is older than 1 month.


fix: select mapping remove if possible

Currently use vmap for compatibility.
But now use select-mode in near latest vim(7.4 or more), side-effect this mapping.
(e.g. snippet plugin select placeholder keyword)
This PR is sunmap if possible(exsits :sunmap).

@tsuyoshicho
I am so grateful for your support!


python310Packages.elasticsearch8: 8.7.0 -> 8.8.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.elasticsearch8 is: Official low-level client for Elasticsearch
meta.homepage for python310Packages.elasticsearch8 is: https://github.com/elasticsearch/elasticsearch-py
meta.changelog for python310Packages.elasticsearch8 is: https://github.com/elastic/elasticsearch-py/releases/tag/v8.8.0
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 8.8.0 with grep in /nix/store/ng5r5sxqakfbwg62wxf7rx34l63kj7aj-python3.10-elasticsearch8-8.8.0
found 8.8.0 in filename of file in /nix/store/ng5r5sxqakfbwg62wxf7rx34l63kj7aj-python3.10-elasticsearch8-8.8.0

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

5 total rebuild path(s)

5 package rebuild(s)

First fifty rebuilds by attrpath
elasticsearch-curator
python310Packages.elasticsearch8
python310Packages.es-client
python311Packages.elasticsearch8
python311Packages.es-client

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ng5r5sxqakfbwg62wxf7rx34l63kj7aj-python3.10-elasticsearch8-8.8.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.elasticsearch8 https://github.com/r-ryantm/nixpkgs/archive/f560a32918e1201dd69eb11b0586c400642c4123.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/ng5r5sxqakfbwg62wxf7rx34l63kj7aj-python3.10-elasticsearch8-8.8.0
ls -la /nix/store/ng5r5sxqakfbwg62wxf7rx34l63kj7aj-python3.10-elasticsearch8-8.8.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

3 packages failed to build:

elasticsearch-curator
python310Packages.es-client
python311Packages.es-client

2 packages built:

python310Packages.elasticsearch8
python311Packages.elasticsearch8

Maintainer pings
cc @fabaff for testing.

@fabaff tentatively closing this since it looks like you covered the same update in c22979269f89c3cc576ce1f8f28fb01aa2fd896c.


fix bug about update app block

Current coverage is 45.92% (diff: 100%)

Merging #141 into master will increase coverage by 2.14%

@@             master       #141   diff @@
==========================================
  Files            16         16
  Lines          8905       8905
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3899       4090   +191
+ Misses         4001       3787   -214
- Partials       1005       1028    +23

Powered by Codecov. Last update 69d07e3...dfb2fdd


Open the reader or the publication dialog when someone clicks on a publication in list mode

fix #497

There is a styling / line formatting issue in the list view, but let's merge this PR and address the presentation bug in a separate issue:


Use one behavior's userdata in another behavior without re-creating them

Hi, I want to use the same userdata for two different behaviors, does it possible to save a behavior's state machine userdata, then load them into another behavior?
I tried to copy the content of the first behavior's create function (for example: _state_machine.userdata.global_planner = "GlobalPlanner"), then past the content into the [MANUAL_CREATE] section of the second behavior's create function. With this operation, while opening the second behavior, I get these userdata in the State Machine Interface section (such as : Input Keys: global_planner), but nothing in the State Machine Userdata section.
I would like to know if it is possible to copy the userdata value as well, to avoid re-create again those userdata.
Thanks in advance.

I want to edit the issue content, I just couldn't.. For using one behavior's userdata into another behavior, a way is to change the behavior's name, so that another behavior will be created, which is the same as this one but with different name.
When using a container, we need to re-write input and output keys that will be used inside the container, and which are already declared in the Behavior Dashboard.  When it concerns many input and output keys, it takes time to re-write them.
Also, sometimes, some output keys of a container should be declared as its input keys as well. Otherwise, warning messages will appear when saving the behavior.
Does it possible to use directly userdata that declared in the Behavior Dashboard inside a container, without re-write them?
Thanks.


renderer: fix clipped clippers

Fast track was applied for clippers even if they
were also clipped. As a result their clips were
omitted.
@Issue: https://github.com/thorvg/thorvg/issues/2777
before:

after:

sample:
        //correct
        {
            auto rect1 = tvg::Shape::gen();
            rect1->appendRect(0,0,800,800);
            rect1->fill(0,0,255);

            auto shape1 = tvg::Shape::gen();
            shape1->moveTo(65, 15);
            shape1->lineTo(165, 15);
            shape1->lineTo(165, 115);
            shape1->lineTo(65, 115);
            shape1->lineTo(65, 15);
            shape1->close();
            shape1->fill(255, 0, 0);

            auto bbox1 = tvg::Shape::gen();
            bbox1->appendRect(70, 20, 20, 100);
            shape1->composite(std::move(bbox1), tvg::CompositeMethod::ClipPath);

            rect1->composite(std::move(shape1), tvg::CompositeMethod::ClipPath);

            canvas->push(std::move(rect1));
        }

        //not correct
        {
            float shift = 100;

            auto rect1 = tvg::Shape::gen();
            rect1->appendRect(0,0,800,800);
            rect1->fill(0,0,255);

            auto shape1 = tvg::Shape::gen();
            shape1->moveTo(65 + shift, 15);
            shape1->lineTo(165 + shift, 15);
            shape1->lineTo(165 + shift, 115);
            shape1->lineTo(65 + shift, 115);
            shape1->close();
            shape1->fill(255, 0, 0);

            auto bbox1 = tvg::Shape::gen();
            bbox1->appendRect(70 + shift, 20, 20, 100);
            shape1->composite(std::move(bbox1), tvg::CompositeMethod::ClipPath);

            rect1->composite(std::move(shape1), tvg::CompositeMethod::ClipPath);

            canvas->push(std::move(rect1));
        }

@mgrudzinska just so you know, this is a "renderer" not a "common"

@mgrudzinska
Thank you very much for your repair, it has been proven to work perfectly! Does this fix work by skipping FastTrack processing when two CILPs are detected?
I also tried to mirror the repair in the built-in thorvg version of LVGL (v0.13.5). Since there are some differences between the code and the current main branch, could you please help me check https://github.com/lvgl/lvgl/pull/6956 whether my modification is correct? Thank you~

@mgrudzinska Thank you very much for your repair, it has been proven to work perfectly! Does this fix work by skipping FastTrack processing when two CILPs are detected? I also tried to mirror the repair in the built-in thorvg version of LVGL (v0.13.5). Since there are some differences between the code and the current main branch, could you please help me check lvgl/lvgl#6956 whether my modification is correct? Thank you~

I saw that you've applied already Hermets suggestion, I'm happy it all worked :)


Switch default_env sync to be before installling Runhouse.

#1024  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rohinb2 and the rest of your teammates on  Graphite


Add --gcov-report flag to spin test

Changes

Added --gcov-report flag to spin test

spin test --help                                                                                1 ↵ ganesh@ganesh-MS-7B86
Usage: spin test [OPTIONS] [PYTEST_ARGS]...
...
  --gcov-report [html|xml|text|sonarqube]
                                  Generate a coverage report for C Code and exit
...

Testing
Happy Path
» spin test --gcov
...
» spin test --gcov-report html                                                                                ganesh@ganesh-MS-7B86
Verifying installed packages for generating coverage reports...
$ ninja -C build -t targets all
Generating html coverage report...
$ ninja -C build coverage-html
Coverage report generated successfully and written to /home/ganesh/os/devpy/example_pkg/build/meson-logs/coveragereport/index.html

Failures

No build found

» git clean -xdf                                                                                ganesh@ganesh-MS-7B86
Removing .spin/__pycache__/
Removing build-install/
Removing build/
» spin test --generate-gcov-report html                                                                                ganesh@ganesh-MS-7B86
Verifying installed packages for generating coverage reports...
Error: `build` folder not found, cannot generate coverage reports. Generate coverage artefacts by running `spin test --gcov`

Gcovr, etc not installed

» spin test --gcov-report html                                                                                ganesh@ganesh-MS-7B86
Verifying installed packages for generating coverage reports...
$ ninja -C build -t targets all
Error: coverage-html is not supported... Ensure the following are installed: Gcovr/GenHTML, lcov and rerun `spin test --gcov`

Notes
ToDo

[x] UT

Misc
resolves https://github.com/scientific-python/spin/issues/158
continues https://github.com/scientific-python/spin/pull/146

Thanks, Ganesh, I've done a brief pass and all looks good. There are a few small tweaks I'd like to make, which I'll do tomorrow morning.
Instead of generate-gcov-report, perhaps the shorter gcov-report? Happy to make that tweak myself, just wanted to make sure you don't object.

Thanks for the review @stefanv , I have renamed the flag to --gcov-report and updated the description.

After looking at this some more, I'm wondering if the gcov-report option is associated with the right command. It seems unrelated to the test options, and rely on the build having been performed with --gcov.
Also, doing spin build, and then spin build --gcov isn't enough. You need spin build --clean --gcov.
So, I'd suggest moving --gcov-report to build, setting html as the default, automatically enabling --gcov when --gcov-report is set, and also detecting when the existing build wasn't run with --gcov (you can check the build/build.ninja file for coverage-* commands).

Also, doing spin build, and then spin build --gcov isn't enough. You need spin build --clean --gcov.

I'll note that this isn't specific to --gcov, it applies to anything that translates to a compile/link flag. --werror and --asan, which SciPy's dev.py has any would also be useful for spin, are good examples.

So, I'd suggest moving --gcov-report to build, setting html as the default, automatically enabling --gcov when --gcov-report is set

I'm not sure I understand that suggestion, or maybe I'm missing your point. You typically build from scratch once, and run tests many times when you're developing. You don't want a coverage report every time you run tests, so it shouldn't be done automatically. But conceptually, you want the coverage report for the tests you've just run I think, through a "give me the gcov coverage report". So it could be integrated with test, or possibly standalone - but it doesn't belong with build.

That was my original understanding, but then I couldn't see where the gcov outputs are generated by pytest. I clearly missed it, so I'll take another look.

That was my original understanding, but then I couldn't see where the gcov outputs are generated by pytest. I clearly missed it, so I'll take another look.

It's a separate command, unrelated to pytest. That command does need to be run after the test run of interest. E.g., see https://github.com/scipy/scipy/blob/maintenance/1.10.x/runtests.py#L510-L533.

@ganesh-k13 Sorry for the delay working on this. I've reworked the logic a bit, as detailed in the last commit message. Take a look, and see if you approve of what I've done?


Fix link in Checking your progress

The text the ones in this commit. should be the ones in these tagged commits per step. and should point to https://github.com/frontity-demos/tutorial-hello-frontity/tags

Check you're on the right track by comparing your changes with the ones in this commit.

That's just an example box to illustrate the type of box that will be encountered at the end of each lesson. The link is deliberately just #.

Oh, I see. In that case, don't you think it would be better to show an image w/ a screenshot of that box? (so the link is not clickable and to avoid a possible confusion)

Good idea 👌 - done!


Grid.ColumnDefinitions?

I'm not experienced with Xamarin so I'm watching a video and trying to apply the lessons. Right off the bat they are using a Grid layout. MBB has a  but it doesn't appear to support the Xamarin syntax for Grid.ColumnDefinitions and Grid.ColumnDefinition.
I don't know what to look for a) is the syntax just different or b) there is limited support for grids at the moment or c) something else :-)
Is the basic concept that the Xamarin controls will be mapped into Blazor or that we just use Blazor components at will?  Thanks again.

I wrote up a doc on some of the syntax as well as examples. They are now live here: https://docs.microsoft.com/mobile-blazor-bindings/ui/grid-layout
The docs show both the old syntax (Preview 4 and earlier) and the new syntax (Preview 5 and later).


Can't show the folder or file of name 'tmp'.

VSCode Version: 1.39
OS Version: Mac Mojave 10.14.6

Steps to Reproduce:
1.Create workspace like this (tmp, tmp1, cantShow.txt are files.):
test-issue/
test-issue/testTmpFile/
test-issue/testTmpFile/tmp
test-issue/testTmpFile/tmp1
test-issue/testTmpFolder/
test-issue/testTmpFolder/tmp/
test-issue/testTmpFolder/tmp/cantShow.txt
2.Use VSCode open this workspace 'test-issue', it only shows:
test-issue/
test-issue/testTmpFile/
test-issue/testTmpFile/tmp1
test-issue/testTmpFolder/

Does this issue occur when all extensions are disabled?: Yes/No

Can you reproduce this with vscode insiders https://code.visualstudio.com/insiders/
Can you include some picttures?

Another question : What do you have in User and Workspace settings for Files: Exclude (files.exclude)?

Another question : What do you have in User and Workspace settings for Files: Exclude (files.exclude)?

U r right. Cause use files.exclude for '**/tmp'.

Can you reproduce this with vscode insiders https://code.visualstudio.com/insiders/
Can you include some picttures?

Remove files.exclude for '**/tmp', file and folder show right. Thanks.


Proper way to include JS and CSS libs?

I would like to make use of for example:
https://github.com/IanLunn/Hover
https://github.com/mbostock/d3
For the first one I would like to make use of the bundled scss files and for the second one only the plain min.js file. Should I add bower to the mix or is this somehow supposed to be taken care of by npm or webpack? Thanks and sorry if the question is too basic ...

For the styles, I would use npm to get the packages locally and them import them like I do bootstrap in app.scss.
As far as the scripts, you should be able to add them to webpack here to have them be compiled with the vendor.js files.

There is also some other tips here for d3: http://fengshuo.co/2015/06/27/use-webpack-with-d3-dashboard-pages/

Thank you, really helpful!
Got both the CSS and JS to work nicely now.


Offer tutorial modal for first timers

We should offer first timer direct links to tutorials

It could be done as notification/toast message

we would need copy for text in toast, maybe @JustLuuuu can drop bites
Consider adding link Tutorials in Navbar
Style could be resued like is in congratulations on first mint and first buy
Check if first timer has minted any nfts, if none, show this tutorial toast/modal

Do we have a testnet of KodaDot on some of the testnet chains?
I though that it might be cool to incentivize people to try and play around with the marketplace before they engage with their own money. Also the process of topping up money into their wallet might be one of the problems they dont engage at all.

this could be done with a testnet (also could have more fictional money to also buy some NFTs etc.)
or I though that we might offer first timers and people who hasnt minted anything some small amount of KSM (enough to create a collection and mint 1 NFT into it) so that they could atleast try to mint something.

This might then incentivize them more with engaging with KodaDot. We would need to think of some restrictions in case of the 2. option so that it wouldnt be targeted and farmed with bots etc. We could also come then come up with some nice and easy on-site tutorial which could help then along the way.
Just a thought :)

yy, that is similar to this issue

#4877

Yep thats basically it, not sure whether 20 mints isnt too much but this could be reduced if needed.


⚠ ClamAV has degraded performance

In 0dc8063, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 53 ms

Resolved: ClamAV performance has improved in cce44c6.


Support for FIFO streaming readout

Hi. I would like to continuously sample the BMA423 accelerometer. And for the samples to be well synchronized in time. From my testing so far, doing polling based from MicroPython introduces occasional stutters of several milliseconds in the worst case.
The best way to get stable and performant readouts is using the FIFO functionality in the accelerometer. The BMA423 has 1 kB of FIFO space, so it can hold data for many milliseconds, which also enables the host microcontroller to sleep in between. This is an "advanced" feature, which many drivers unfortunately do not implement.
Are you interested in such functionality, if I implement it? If so, do you have any guidelines as to implementation and/or API?
I am thinking to let the BMA423 stay simple methods for poking the device registers. And then the whole flow of FIFO based data streaming can be in an example, like the others you already have.

Sketch for how a usage example could look like:
i2c = SoftI2C(scl=11,sda=10)
sensor = BMA423(i2c)

fifo_samples = 100
buffer = bytearray(6*fifo_samples)

def sensor_callback(data):

    sensor.read_fifo_info(buffer)

    # do something with the samples found in "buffer"
    # including converting into normalized readings etc
    # sensor.self.convert_to_int12()
    # sensor.normalize_reading()
    # Or push the data onto a queue for later processing

interrupt_pin = Pin(14, Pin.IN)

sensor.set_accelerometer_freq(100)
sensor.enable_fifo(watermark=fifo_samples, mode='overwrites_oldest')
sensor.enable_interrupt(1, interrupt_pin, sensor_callback, ["fifo-wm"])

Come to think of it, doing it pull based is probably easier - at least when fetching the samples can be done every 100+ milliseconds.
New sketch.
import machine
from machine import SoftI2C

i2c = SoftI2C(scl=11,sda=10)
sensor = BMA423(i2c)

samplerate = 100
wait_seconds = 0.25
fifo_samples = math.ceil(wait_seconds*samplerate)
buffer = bytearray(6*fifo_samples)

sensor.set_accelerometer_freq(samplerate)
sensor.set_advanced_power_save(adp=False) # avoid ADP, to enable FIFO readouts at any time
sensor.enable_fifo(watermark=fifo_samples, mode='overwrites_oldest')

while True:
    bytes_read = sensor.read_fifo_info(buffer)

    # do something with the samples found in "buffer"
    # including converting into normalized readings etc
    # sensor.self.convert_to_int12()
    # sensor.normalize_reading()

    machine.lightsleep(1000*wait_seconds)


Can't build ROS2 Humble Image

I have an NVIDIA Jetson Orin NX.
I am trying to build an image with the following configuration:

ROS2 Humble

I am going to be working in python and cpp.
FYI this is a continuation of Can't build CUDA=11.8 ROS2 Humble Image #530
We are using Jetpack 5.1 and CUDA 11.4:
orin_nx@ubuntu:~$ sudo apt-cache show nvidia-jetpack
[sudo] password for orin_nx:
Package: nvidia-jetpack
Version: 5.1.1-b56
Architecture: arm64
Maintainer: NVIDIA Corporation
Installed-Size: 194
Depends: nvidia-jetpack-runtime (= 5.1.1-b56), nvidia-jetpack-dev (= 5.1.1-b56)
Homepage: http://developer.nvidia.com/jetson
Priority: standard
Section: metapackages

I run the following command:
jetson-containers build --name=ai-dev ros:humble-desktop

I get the following Error:
-- Testing container ai-dev:l4t-r35.3.1 (opencv:4.5.0-deb/test.py)

docker run -t --rm --runtime=nvidia --network=host \
--volume /home/orin_nx/Desktop/software/jetson-containers/packages/opencv:/test \
--volume /home/orin_nx/Desktop/software/jetson-containers/data:/data \
--workdir /test \
ai-dev:l4t-r35.3.1 \
/bin/bash -c 'python3 test.py' \
2>&1 | tee /home/orin_nx/Desktop/software/jetson-containers/logs/20240523_094951/test/ai-dev_l4t-r35.3.1_test.py.txt; exit ${PIPESTATUS[0]}

sourcing   /opt/ros/humble/install/setup.bash
ROS_DISTRO humble
ROS_ROOT   /opt/ros/humble
testing OpenCV...
/usr/lib/python3/dist-packages/requests/__init__.py:89: RequestsDependencyWarning: urllib3 (2.2.1) or chardet (3.0.4) doesn't match a supported version!
  warnings.warn("urllib3 ({}) or chardet ({}) doesn't match a supported "
/usr/lib/python3/dist-packages/requests/__init__.py:89: RequestsDependencyWarning: urllib3 (2.2.1) or chardet (3.0.4) doesn't match a supported version!
  warnings.warn("urllib3 ({}) or chardet ({}) doesn't match a supported "
Traceback (most recent call last):
  File "test.py", line 6, in <module>
    import requests
  File "/usr/lib/python3/dist-packages/requests/__init__.py", line 95, in <module>
    from urllib3.contrib import pyopenssl
  File "/usr/local/lib/python3.8/dist-packages/urllib3/contrib/pyopenssl.py", line 43, in <module>
    import OpenSSL.SSL  # type: ignore[import-untyped]
  File "/usr/lib/python3/dist-packages/OpenSSL/__init__.py", line 8, in <module>
    from OpenSSL import crypto, SSL
  File "/usr/lib/python3/dist-packages/OpenSSL/crypto.py", line 1553, in <module>
    class X509StoreFlags(object):
  File "/usr/lib/python3/dist-packages/OpenSSL/crypto.py", line 1571, in X509StoreFlags
    NOTIFY_POLICY = _lib.X509_V_FLAG_NOTIFY_POLICY
AttributeError: module 'lib' has no attribute 'X509_V_FLAG_NOTIFY_POLICY'
Error in sys.excepthook:
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/apport_python_hook.py", line 72, in apport_excepthook
    from apport.fileutils import likely_packaged, get_recent_crashes
  File "/usr/lib/python3/dist-packages/apport/__init__.py", line 5, in <module>
    from apport.report import Report
  File "/usr/lib/python3/dist-packages/apport/report.py", line 32, in <module>
    import apport.fileutils
  File "/usr/lib/python3/dist-packages/apport/fileutils.py", line 12, in <module>
    import os, glob, subprocess, os.path, time, pwd, sys, requests_unixsocket
  File "/usr/lib/python3/dist-packages/requests_unixsocket/__init__.py", line 1, in <module>
    import requests
  File "/usr/lib/python3/dist-packages/requests/__init__.py", line 95, in <module>
    from urllib3.contrib import pyopenssl
  File "/usr/local/lib/python3.8/dist-packages/urllib3/contrib/pyopenssl.py", line 43, in <module>
    import OpenSSL.SSL  # type: ignore[import-untyped]
  File "/usr/lib/python3/dist-packages/OpenSSL/__init__.py", line 8, in <module>
    from OpenSSL import crypto, SSL
  File "/usr/lib/python3/dist-packages/OpenSSL/crypto.py", line 1553, in <module>
    class X509StoreFlags(object):
  File "/usr/lib/python3/dist-packages/OpenSSL/crypto.py", line 1571, in X509StoreFlags
    NOTIFY_POLICY = _lib.X509_V_FLAG_NOTIFY_POLICY
AttributeError: module 'lib' has no attribute 'X509_V_FLAG_NOTIFY_POLICY'

Original exception was:
Traceback (most recent call last):
  File "test.py", line 6, in <module>
    import requests
  File "/usr/lib/python3/dist-packages/requests/__init__.py", line 95, in <module>
    from urllib3.contrib import pyopenssl
  File "/usr/local/lib/python3.8/dist-packages/urllib3/contrib/pyopenssl.py", line 43, in <module>
    import OpenSSL.SSL  # type: ignore[import-untyped]
  File "/usr/lib/python3/dist-packages/OpenSSL/__init__.py", line 8, in <module>
    from OpenSSL import crypto, SSL
  File "/usr/lib/python3/dist-packages/OpenSSL/crypto.py", line 1553, in <module>
    class X509StoreFlags(object):
  File "/usr/lib/python3/dist-packages/OpenSSL/crypto.py", line 1571, in X509StoreFlags
    NOTIFY_POLICY = _lib.X509_V_FLAG_NOTIFY_POLICY
AttributeError: module 'lib' has no attribute 'X509_V_FLAG_NOTIFY_POLICY'
Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/orin_nx/Desktop/software/jetson-containers/jetson_containers/build.py", line 103, in <module>
    build_container(args.name, args.packages, args.base, args.build_flags, args.simulate, args.skip_tests, args.test_only, args.push, args.no_github_api)
  File "/home/orin_nx/Desktop/software/jetson-containers/jetson_containers/container.py", line 162, in build_container
    test_container(name, package, simulate)
  File "/home/orin_nx/Desktop/software/jetson-containers/jetson_containers/container.py", line 322, in test_container
    status = subprocess.run(cmd.replace(_NEWLINE_, ' '), executable='/bin/bash', shell=True, check=True)
  File "/usr/lib/python3.8/subprocess.py", line 516, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'docker run -t --rm --runtime=nvidia --network=host --volume /home/orin_nx/Desktop/software/jetson-containers/packages/opencv:/test --volume /home/orin_nx/Desktop/software/3.1 /bin/bash -c 'python3 test.py' 2>&1 | tee /home/orin_nx/Desktop/software/jetson-containers/logs/20240523_094951/test/ai-dev_l4t-r35.3.1_test.py.txt; exit ${PIPESTATUS[0]}' returned non-zero exit status 1.

Full log:
output_log_jetson_containers .txt
Current images:
orin_nx@ubuntu:~$ sudo docker image ls
REPOSITORY                   TAG                              IMAGE ID       CREATED         SIZE
ai-dev                       l4t-r35.3.1                      3bfa7c837499   19 hours ago    13.5GB
ai-dev                       l4t-r35.3.1-ros_humble-desktop   3bfa7c837499   19 hours ago    13.5GB
ai-dev                       l4t-r35.3.1-cmake                6167f3fb39c4   22 hours ago    11.2GB
ai-dev                       l4t-r35.3.1-opencv_deb           fc57b4aa0abb   22 hours ago    11.1GB
ai-dev                       l4t-r35.3.1-numpy                7414089c021f   22 hours ago    10.3GB
ai-dev                       l4t-r35.3.1-python               1b53c3a061da   22 hours ago    10.2GB
ai-dev                       l4t-r35.3.1-tensorrt             1b53c3a061da   22 hours ago    10.2GB
ai-dev                       l4t-r35.3.1-cuda                 80709df5b4bf   22 hours ago    10.1GB
ai-dev                       l4t-r35.3.1-cudnn                80709df5b4bf   22 hours ago    10.1GB
ai-dev                       l4t-r35.3.1-build-essential      f47b003624f1   23 hours ago    10.1GB
<none>                       <none>                           72f3fcd97195   8 days ago      2.59GB
<none>                       <none>                           55c6334ea5c9   9 months ago    3.84GB
nvcr.io/nvidia/l4t-jetpack   r35.3.1                          ff2dd43d5687   13 months ago   9.77GB

I'm unsure what the problem is it looks like some test script failed maybe?

@mswanson24 I recently had to add this patch to torchvision on 20.04 for the same error, pinning pyopenssl==24.0.0 (presumably due to some package update on PyPi) - https://github.com/dusty-nv/jetson-containers/commit/004d74c2e3670f6e0a23671ab231eceb08ab5d2b
Can you try adding this same block to the bottom of jetson-containers/packages/opencv/install.sh  and try building again?
if [ $(lsb_release --codename --short) = "focal" ]; then
    # https://github.com/conda/conda/issues/13619
    pip3 install --no-cache-dir pyopenssl==24.0.0
fi

@mswanson24 I recently had to add this patch to torchvision on 20.04 for the same error, pinning pyopenssl==24.0.0 (presumably due to some package update on PyPi) - 004d74c
Can you try adding this same block to the bottom of jetson-containers/packages/opencv/install.sh  and try building again?
if [ $(lsb_release --codename --short) = "focal" ]; then
    # https://github.com/conda/conda/issues/13619
    pip3 install --no-cache-dir pyopenssl==24.0.0
fi

I will try this and let you know how it goes. Thank you!

Hi @dusty-nv,
I was able to successfully build the image:
orin_nx@ubuntu:/media$ docker image ls
REPOSITORY                   TAG                              IMAGE ID       CREATED         SIZE
ai-dev                       l4t-r35.3.1                      c3534d6c6b42   3 days ago      13.5GB
ai-dev                       l4t-r35.3.1-ros_humble-desktop   c3534d6c6b42   3 days ago      13.5GB
ai-dev                       l4t-r35.3.1-cmake                03b5506805e1   3 days ago      11.2GB
ai-dev                       l4t-r35.3.1-opencv_deb           88089638a4de   3 days ago      11.1GB
ai-dev                       l4t-r35.3.1-numpy                753134c78196   3 days ago      10.3GB

One thing i noticed in relation to the issue :https://github.com/dusty-nv/jetson-containers/issues/530#issue-2302275837
Is that I am building with r35.3.1 and they are building with r35.4.1.
Could that be causing complications?


CIV-13301 Fix Failing test

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-13301
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No


UPD address-api-client to check expected status code

@lulf shouldn't status code on delete operation(s) be 204 instead of 200? because response on delete operation is empty body
 DELETE-address: path /apis/enmasse.io/v1alpha1/namespaces/enmasseci/addressspaces/brokered-shared-0/addresses/brokered-shared-0.brokeredqueuea 11:14:49,604 [vert.x-eventloop-thread-0] ERROR io.enmasse.systemtest.apiclients.ApiClient - expected-code: 204, response-code: 200, body: null


Added new custom token function

Added function for create custom token as per given list of custom token.
So If anyone wants some custom token they can create by just giving list else remain same.

@GharudxD please make the case for this functionality being added to NLTK. There needs to be some existing demand otherwise we cannot justify adding to the codebase.

@GharudxD please make the case for this functionality being added to NLTK. There needs to be some existing demand otherwise we cannot justify adding to the codebase.

Yap, you are right we can't add all things in codebase but I thought sometimes when we tokenize sometimes all word didn't tokenize based on space. Some words treated as token like full forms organisation name etc, for this reasons I created this function.I thought if I add this feature in nltk peoples haven't do this manually.

Sorry @GharudxD, this is too obscure. A better work-around is to pre-process the problematic input.


[1pt] Create basic structure for Missions - Setup

Create a route and a view component. Use <NavLink /> for the page navigation links and style active class to indicate which section/page the user is currently on (underline active navigation link).
Create a directory for all Redux state slice files.

https://github.com/RiyaBulia12/Space-Hub/pull/28


add COMMON_ISSUES as file

This PR moves the list of common issues from issue #567 to a dedicated file inside the repository. Allowing to modify it via PRs. Additionally the issue template is adjusted and inside the issue template picker common issues are highlighed.
You can preview this in https://github.com/Shegox/cla-assistant/issues/new/choose

fixes #567

Coverage remained the same at 95.098% when pulling ba056073248dcc3cf2f762973f81bc2ac464f9a7 on Shegox:master into 2e8d9903a2e38b098d14984df7b6d56e21d327dc on cla-assistant:master.

78f08eb720ec5ea21f0d16247cb6207ba97c56cf


added blog

@Vaish-922 Please go through the comments and make the changes accordingly. Also, resolve the merge conflict.

@Vaish-922 The content of the blog lgtm
The naming of the commits is off. I would suggest

stash all commits into a single commit with proper naming or,
make a new PR if the former is difficult to do

@Vaish-922 Also look into why the tests jekyll tests are failing


Fix README formatting (rebased)

Successor to #10

@thomasrockhu sorry for the late reply but I've fixed this PR accordingly.

Thanks @superusercode !


updata rippleview

last time you update it for listview,maybe you forgot it also has girdview.I update it for all views extend abslistview.

I can't merge your PR please merge with master and solve conflicts ;)


Doc: Setting DNS for Docker daemon using systemd drop-in

User asked on Stackoverflow.
http://stackoverflow.com/questions/33784295/setting-dns-for-docker-daemon-using-systemd-drop-in

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Was more like configuration question. And we have docs about that.


https://github.com/tarikbouari/Kanban-Board

We are 2 members in this group: @tarikbouari  and @summyalena

STATUS: INVALID CODE REVIEW REQUEST 🛑 🛑
Hi @tarikbouari,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
In your issue you should provide the link to your kanban board
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. And please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

STATUS: CHANGES REQUIRED ♻ ♻
Hi @tarikbouari,
Good job so far!
✔ Kanban columns are weel set up
However, there are some issues that you still need to work on to go to the next project but you are almost there!
Required Changes ♻

[ ] Your tasks are not properly assigned. Kindly add the other student to this repo and choose who is Student A and B and assign the tasks according. Different group tasks can be assigned to more than one person or just one person, but they all must be assigned.

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the Issue thread if something is not 100% clear. And please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Issue for re-reviews. You should use the same Issue submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hi @tarikbouari @summyalena,
APPROVED 🟢
Highlights
✔ Descriptive PR
✔ Passing linters
✔ Good branch name
Great implementation of all the requirements in this milestone. 👏
You have done an amazing job 👍
Your project is complete!  There is nothing else to say other than... it's time to merge it. :shipit:
Congratulations! 🎉
CHEERS AND HAPPY CODING!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Fix version inconsistency between artifacts upload and download

According to download-artifact:

Downloading artifacts that were created from action/upload-artifact@v3 and below are not supported.

And since upload-artifact@v4 did not support uploading to the same named Artifact multiple times, we should degrade the version of download-artifact from v4 to v3.

Thank you for the contribution, please could you merge master into your branch to fix the CI failure?


Whisper ASR is broken

This plugin cannot work with the Whisper ASR Webservice anymore. It sends audio file to the docker, but cannot pick up the response properly.

Has the Whisper ASR response format changed? If so, can submit a PR or use the ASR docker image versioned before the format change

I might be seeing the same issue:
Error with URL: http://localhost:9000/asr?output=json&word_timestamps=true&language=en TypeError: Cannot read properties of undefined (reading 'map')

I've replicated the above error on Whisper ASR versions 1.3.0 (latest) and also 1.2.4 and 1.1.1. It seems to be failing here. It looks like the code above expects rawResponse.segments to be an array of arrays, whereas I'm seeing it as an array of Segment[], where Segment is like:
interface Segment {
  avg_logprob: number;
  compression_ratio: number;
  end: number;
  id: number;
  no_speech_prob: number;
  seek: number;
  start: number;
  temperature: number;
  text: string;
  tokens: number[];
}

I'm happy to revert to a previous version of the docker image -- does anyone know what the latest functional version of whisper-asr-webservice is?

I might be seeing the same issue:
Error with URL: http://localhost:9000/asr?output=json&word_timestamps=true&language=en TypeError: Cannot read properties of undefined (reading 'map')

I've replicated the above error on Whisper ASR versions 1.3.0 (latest) and also 1.2.4 and 1.1.1. It seems to be failing here. It looks like the code above expects rawResponse.segments to be an array of arrays, whereas I'm seeing it as Segment[], where Segment is like:
interface Segment {
  avg_logprob: number;
  compression_ratio: number;
  end: number;
  id: number;
  no_speech_prob: number;
  seek: number;
  start: number;
  temperature: number;
  text: string;
  tokens: number[];
}

I'm happy to revert to a previous version of the docker image -- does anyone know what the latest functional version of whisper-asr-webservice is?

Hey! Were you able to solve the problem?

Ah in that case this might be related to https://github.com/djmango/obsidian-transcription/pull/50 @bscholer

Thanks for investigating @dahifi - i will corroborate your findings and release an update later today accordingly

I found the line here: https://github.com/djmango/obsidian-transcription/pull/50/files#diff-0f4208f0163c212f445df35fc43b99ceb09432700a33b38b514883c99c7d6169R131
I'm going to grab lunch then would like to do the PR myself, kind ser.

I haven't looked close at the code yet, but this does seem like it's due to my PR #50. Will leave for @dahifi to patch unless you'd like me to!

Gotcha, much appreciated

Actually got it done before my sandwich cooked, this should do the trick. Can we get a release quickly so I don't have to downgrade my Notebook plugin? @djmango

Please disregard that, I should probably take care to validate the response. Probably rushing because I was hungry. Give me an hour to load up my Obsidian dev vault and test everything.

Ok, so I have no idea which version I was using before, and there's a lot that seems different here. WhisperASR expects the file to be a multipart form, e.g.
curl -X 'POST' \
  'http://<IP_ADDRESS>:9000/asr?task=transcribe&encode=true&output=json&diarize=false' \
  -H 'accept: application/json' \
  -H 'Content-Type: multipart/form-data' \
  -F 'audio_file=@Recording<PHONE_NUMBER>3030.webm;type=video/webm'

Where it looks like you're using octet-streams in the request body?
I've had bad luck trying to get things working in my dev notebook. I went back through all the various commits that modified that function and finally wound up on the last contribution I did back in October, which should be v.3.1.6.
For now I recommend ASR users load up that version, I'm not sure I want to spend more time on it. DJ may want to reconsider supporting it since Swiftlink has diverged from it so much now. Might just want to make a note in the readme and leave it at that.

It looks like this issue does not occur when using ASR_ENGINE=faster_whisper. Seems like there is an inconsistency between the engines in https://github.com/ahmetoner/whisper-asr-webservice

Doing further investigation, it looks like ASR_ENGINE=openai_whisper returns a structure that looks like this (notice the named fields in the segments array):

ASR_ENGINE=faster_whisper instead returns a structure that looks like this (notice the lack of named fields). Although I hate this format, this is what I wrote the code to expect.

I'll open up an issue on https://github.com/ahmetoner/whisper-asr-webservice to try to standardize this, but for now, I'll just tweak the code to handle both so we can get this issue fixed.

Update released


Please update to Firefox 66.0.4 (master and release branches) to fix broken addons

Issue description
As many already know, an issue with expired certificates made all Firefox addons suddenly "untrusted" by the browser:
https://blog.mozilla.org/addons/2019/05/04/update-regarding-add-ons-in-firefox/
Apparently a new release of Firefox with updated certificates has been published:
https://ftp.mozilla.org/pub/firefox/releases/66.0.4/
Would it be possible to update master and appropriate release branches please?
See also #60916
Thanks

The master and 19.03 branches have Firefox 66.0.4 now. The channels should update shortly.


IC-1529: Move sesson feedback into sessionDelivery directory

NOTE: this PR depends on commits in #240 so you can ignore everything but the last commit on this branch (the others will be removed once that PR has been merged)
What does this pull request do?
Moves session feedback into a sessionDelivery/postSessionFeedback directory.
What is the intent behind these changes?
We can re-use the check your answers page in the PP journey, which uses the presenters in this directory. I'd quite like us to re-structure the directories in this project based on section of the journey e.g. session delivery, intervention progress, rather than everything in one big SP/PP-focused directory, as there are a few places we can re-use components across the two sides.
Next steps
This is a starting point, and if we think this is a good route to go down, we can continue restructuring this. If people think this is a good approach, I'd like to move the following into this directory:

Creating an action plan
Assigning referrals
Adding/editing/viewing sessions

Potential proposed structure, based on intervention progress screen
routes/interventionProgress/
--/initialAssessment/
--/actionPlan/
--/sessionDelivery/
--/endOfServiceReport/
-- /serviceProvider/interventionProgress.njk
-- /probationPractitioner/interventionProgress.njk

I'm not 100% sure about this proposed restructure, it would be good to discuss as a team

I took a look at the changes here, and I think that there's a sensible idea behind it, but I'm having trouble imagining how the sharing might look in practice. Perhaps with an example of something being shared between the PP and SP it would be clearer?

We can re-use the check your answers page in the PP journey

How would this reuse work? Is there a "submit session feedback" journey as part of the PP service? (I can't remember something like that). Or do you mean that there's some general logic common to all "check your answers" pages that could be shared?

How would this reuse work? Is there a "submit session feedback" journey as part of the PP service? (I can't remember something like that). Or do you mean that there's some general logic common to all "check your answers" pages that could be shared?

@lawrence-forooghian At the moment, the check your answers page doesn't have any change links, and displays the answers selected by the PP, so essentially it looks like an "intervention details" style page (although there won't be a "confirm" button). I feel like the only thing that might change in the near future is adding in some change links, which we could possibly do via a conditional, based on an argument or sub-class style method of passing in the user type. I'm not 100% convinced by this approach, but let's chat about it at some point.

Closing this, as it's not needed right now, and more discussion might be needed in future.


Build Github explorer

Describe the project
fetch details of users by just entering their name and by single click we can move to their profile
Issue Type:

[ ] Bug Report
[ ] Feature Request
[ ] Documentation
[✓ ] New Project

Additional context
Add any other context or screenshots about the project here.
Screenshots

If applicable, add screenshots to help explain your problem.
@TusharKesarwani @himanshu1221 @Abhishantpadam @Asrk03 @Ayush9951 #763

Hey @MrinalCom, are you working on it?

yes


CSS for light theme

Hello. At first I want to say your project is amazing. I like it very much.
Right now I am using the light theme. I don't like some default settings. I want to change them. I found there are many .css files. I don't know which one is for light theme. Can you tell me?

By the way. Can you tell me how to remove the title
HUGO
The world’s fastest framework for building websites
I don't see it in params.toml and config.toml files


Implement redux form

Hey @rjmk would it be possible to exclude the membership_type object from the MembersController accountInfo function?  Ideally we would have membership_type String which comes from the members table but i think it is getting overwritten.

@JMurphyWeb It is certainly possible. It's probably there because we're likely to need the amount of the subscription payment. You can get the value off the object if you want the name of the type.
Alternatively, you could change populateAll() on line 22 to populate('payments')

@rjmk this pull request is not ready to be merged in, however if you have some time could you have a look, see if we are going in right direction? #299 has what we are working on atm.


feat: add SdkVer and initial prereleasing support

As discussed, SdkVer is an EXPERIMENTAL feature, and as a result, we will not opt for:

Adding the new configuration parameters in the documentation
Provide support on this feature
Consider the current implementation as final (i.e. the API might change without marking it as a breaking change).

Rebased and squashed away the fixups.


Cptx node

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

49 of 51 (96.08%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 64.209%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

topo/node/cptx/cptx.go
49
51
96.08%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
1376

Relevant Lines:
2143

💛  - Coveralls


Adding logic to Dockerfile to build ipmitool into the controller

Description
Adding steps to install ipmitool binary on the controller image. These changes are based on PBNJ's Dockerfile.
Why is this needed
ipmitool binary is required by bmclib during runtime. The ipmitool provider on bmclib uses ipmitool binary for actions.
Fixes: #
How Has This Been Tested?
make docker-build

Codecov Report

Merging #8 (a8fd76d) into main (13eef9a) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head a8fd76d differs from pull request most recent head e5d0cad. Consider uploading reports for the commit e5d0cad to get more accurate results

@@           Coverage Diff           @@
##             main       #8   +/-   ##
=======================================
  Coverage   58.49%   58.49%
=======================================
  Files           4        4
  Lines         106      106
=======================================
  Hits           62       62
  Misses         37       37
  Partials        7        7

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 13eef9a...e5d0cad. Read the comment docs.


Bug: Can't see button to install drivers, popup too small

Disclaimer:

[x] I have checked for a similar issue and sure it hasn't been reported before.

Describe the bug
Cannot click on "install driver" as the popup is too small, and can't be resized or scrolled.
Can't clic kon anything, keyboard has no effect either. Completely stuck on that screen.
Steps to Reproduce
Install eqMac
Get that screen

Expected behaviour
See the entire screen, be able to scroll or resize. Be able to press ESC or ENTER.
Setup information:

macOS Version: 13.5.2 (22G91)
eqMac UI Version : can't see it, but I just reinstalled it with latest version as of time of posting.

Additional information
I have been using eqMac pro for a while. I don't know why I'm getting this message to install drivers again, and get stuck on that page.

There have been some fixes to UI, do you still experience same issue with v1.8.7?

I think it's fine now, yes


WIP: Add LLC 3D region data

@dhruvbalwada starting the catalog page here. If you have time it would be great if you could check the preview.
Marking this as WIP because that bucket is currently not public (requested a change already).

pre-commit.ci autofix

@andersy005 It seems like the final deployment (for previewing is not working here). Any chance you could take a look?

@andersy005 It seems like the final deployment (for previewing is not working here). Any chance you could take a look?

@jbusecke, this is a bug and i was able to track it down and fixed it as part of

https://github.com/carbonplan/leap-data-catalog/pull/82

in the meantime, you can use this preview link

Thanks @andersy005! Ill wait for https://github.com/leap-stc/LLC4320_12h_3D-regions_feedstock/issues/1 until moving ahead here!

@andersy005 I think this is ready to go. Should we wait for the preview or just go ahead and merge?

pre-commit.ci autofix


fix: Change Authorization to DG-Auth

Make SlashChannel cloud compatible

Note: This is required because C# does validation on the format of the Authorization header.


[RFE] Include Pool Member Service Port in conversion / AS3 file generation

Is your feature request related to a problem? Please describe.
Currently, when AS3 Conversions are run, the service Port (port associated with a pool member) is not created in the AS3 file.
This seems to be the situation for all conversions, so not specific to any NS configuration, always occurs.
This generally should be available in the source NS.conf file, and when not found for a given pool member I would expect this could be assumed to be the same as the virtual port used (LB Port used).
It is worth noting that in the VS Code Generator Screen for 'Fast Template' the Service port does show as present, so perhaps this is a FAST template issue?
Describe the solution you'd like
It would be good to have NS.conf parsed and input into AS3 file with this information.
Not sure of current cause, but assume it is one of:

Failed/Lack of parsing
Failed/Lack of mapping
Limitation of FAST template

Describe alternatives you've considered
For now, workarounds are to manually verify and add, or use external scripting tools to parse and add.
Additional context
Source NS.conf file has information such as:
add lb vserver lb_example SSL <IP_ADDRESS> 443 -persistenceType SOURCEIP -cltTimeout 180
bind lb vserver lb_example lb_example.svc
bind serviceGroup lb_example.svc SERVER22 443
add server SERVER22 <IP_ADDRESS> -comment "EXAMPLE BACKEND SERVER"

An AS3 conversion ends up with pool section looking something like:
    "pool_lb_example": {
      "members": [
        {
          "serverAddresses": [
            "<IP_ADDRESS>"
          ],
          "shareNodes": true
        }
      ],
      "class": "Pool",
      "loadBalancingMode": "least-connections-member",
      "monitors": [
        "https"
      ]
    }

When it should look more like this (note servicePort):
    "pool_lb_example": {
      "members": [
        {
          "serverAddresses": [
            "<IP_ADDRESS>"
          ],
          "shareNodes": true,
          "servicePort": 443
        }
      ],
      "class": "Pool",
      "loadBalancingMode": "least-connections-member",
      "monitors": [
        "https"
      ]
    }

Hey @samualblair, I think we have this in the latest round of fast templates.  I will be working on the next release to include these soon

I confirmed that we are parsing the details from the ns configs and through the ns2fast mutations.  Final step is to update fast templates to read those new params, which as indicated in the last comment, will be updated for the next release


Change organization / group ID

[x] adapt URLs to point to new Travis and GitHub projects
[x] add myself as a developer
[ ] rename package
[ ] adapt instructions in README

Fixes #7.

@lunaryorn I am still trying to figure out what to do with the sbt-headers settings. I would want to keep your copyright stanza intact, and just add a second one to the file. Is that possible?
Alternatively, I would probably simply change the start year to 2019 and put in myself. WDYT?

@avdv I think sbt-headers supports custom copyright headers, which you could use for whatever copyright lines you'd like, but really, I don't care.  Do whatever you like, you may just as well remove my copyright line; it has no legal significance whatsoever anyway :)

@avdv I think sbt-headers supports custom copyright headers, which you could use for whatever copyright lines you'd like, but really, I don't care. Do whatever you like, you may just as well remove my copyright line; it has no legal significance whatsoever anyway :)

I say credit where credit is due. :-) But, OK I won't over-complicate things here...


Expose API port & and upgrade to v1.4.3

This adds the ability to expose ports, such as for the API, on the Docker host. This is useful/required if the crowdsec bouncer is external to the Docker host running Home Assistant.

Sorry for the many commits - I was testing things out on the main branch after I created the PR :)


refactor: Replace LRU Dependency With Custom Implementation

Fixes #1368 🦕

do not merge: Want to add some tests before moving forward.


MSI Creation not embedding files correctly

Good morning
created an officeproplus.msi fine last night with the files embedded as you can see in the picture below the size is correct.
today I have selected only visio and to embed the files but the MSI is not correctly generating
please see below the visioproplus.msi is only small with the office folder separate?

The office folder is being output separately from the MSI, can you advise why this behaviour has changed please?
Thanks

@keane2017 This behavior is working as designed.  If the contents of the files to be imbedded exceed 2GB, they are pushed into a folder that is to be kept in the same directory as the MSI during deployment.  The issue is that the generated files are only 32 bit, and any file size generated over 2GB will crash and fail.

Hi
I am only selecting Visio?
I have created 3 or 4 this week that have all worked ok?
I'm not sure what you mean? it wasn't working like this the other day?
Thanks

if you notice, a new version was deployed overnight it looks like, 16.0.7369.2038.  The contents of this folder are larger than the contents of the prior version.  embedding these files will cause the msi to meet or exceed the file size limitations of a 32 bit file for Microsoft Windows, so instead of creating a file too large for a 32 bit windows application, we copy the files to the "office" folder that you see created in the same directory as your generated MSI.  This is to prevent the application from crashing when launched.  Just make sure that "office" folder is kept in the same folder as the MSI when you launch it on a client machine.  The MSI will copy the contents to the correct location on the client when it's run.  If you're still uncertain about why this there is a file size limitation, I suggest performing a google search with the key terms "32 bit file size limit", this will explain to you the issues we were running into when we were first writing and deploying this toolkit.  Thanks.

Hi
I understand the limits
My point was it's not over the threshold like you say it's the same size as the officeproplus msi I have.
Thanks for letting me know though I don't think the limit is the issue here.
Thanks for your help

It's no problem.  It may not be over the limit, but if it's close enough to be questionable, then the cut off is made, and we put the source files in another folder just to be on the safe side.

Makes sense :)
Thanks for the quick responses it's appreciated

Does the msi work differently when compiled like this?
The source files don't appear in the temp folder as before when they are separate to the msi?
Only the setup and config files do?

Thank you for pointing out this issue, I will work on fixing it.  In the meantime, I suggest you use MSI without embedding the files, or use the exe option if you plan on embedding.

The exe was doing the same.
I have deployed both fine today but wasn't sure where the files were ?
It must be picking up from the internet?
I'll await the fix to ensure it's deploying properly.
Thanks

Hi
is there any update on this please?
Thanks

Issue has been resolved today.  thanks


Order of drawing

Rewrite the order of drawing so that the GPU handles the Z-position instead of relying on the order of draw functions in the script.

 Order of drawing


Not possible to compile latest master branch

For some reason, in GCC for ARM, it is not possible to compile the lib.
Errors like "CO_SDO_t" or CO_storae_entry_t errors are appearing in the core lib.
Why?

Živijo,
Missing include directories?
Can you try:
cd CANopenNode/example
make

Note also, latest master branch switched to v4.0 recently, it is quite an upgrade (new object dictionary, new SDO server, etc.), see https://github.com/CANopenNode/CANopenNode/issues/298

Janez, many thanks for your feedback. Seems you are Slovenian, or at least speak Slovenian :) Hvala za odgovor.
Currently I have a working solution with PIC old examples and will continue with this one.
I would appreciate to get your email for contribution and example on STM32 platform.
Let me know if you are interested, or contact me at<EMAIL_ADDRESS>


Not working with Raspberry Pi 3 (on-board WiFi)

Everything installs fine and the Access Point "ResinAP" shows up on other devices. But once you try to connect to it, it never succeeds and the capture portal never appears. I tested with Pi2 and USB dongle and it worked fine.
The internet suggests that the on-board wifi chip supports SoftAP, e.g. https://frillip.com/using-your-raspberry-pi-3-as-a-wifi-access-point-with-hostapd/

Thanks for reporting @tlangmo - we hadn't yet tested on an RPi 3.
Might be that we have to use hostapd like we did on the edison branch.
Or it could also work with connman if we manually add wlan0 to the bridge using brctl (I've seen sometimes connman fails to add the interface to the bridge, which causes that behaviour where the AP appears but you can't connect to it).
In any case, I'll have to do some tinkering to figure it out. (And you're more than welcome to give it a try if you want ;)

@tlangmo I've prepared a branch that uses hostapd basing from the edison branch with a few changes.
Haven't tested it, but wanna give it a try?
It's rpi3-hostapd

@pcarranzav I've tested it successfully on Rpi3 with your hostapd version. The AP comes up and I can connect to it.
I have noticed a few things though:
*Sometimes, a hotspot gets opened although the network can be joined and the credentials are correct. I have not really figured out how to replicate this. Can I set a timeout somewhere?
*Unfortunately, systemd (ENV INITSYSTEM on) does not work with resin-wifi-connect. This is a bummer since systemd is quite important. Whenever it is active, I get this error message:
Unhandled rejection Error: No WiFi networks found
at /usr/src/app/node_modules/connman-simplified/lib/WiFi.js:509:17
at mgr.GetServices.finish (/usr/src/app/node_modules/connman-simplified/node_modules/connman-api/lib/connman.js:160:5)
at /usr/src/app/node_modules/connman-simplified/node_modules/connman-api/node_modules/dbus/lib/interface.js:49:10
Not sure yet whether this is Rpi3 related or generally a problem with the wifi-connect  app

@tlangmo the systemd thing is a known issue, that has a workaround that was implemented  on the master branch but not the rpi3 version. It requires changing this line to:
DBUS_SYSTEM_BUS_ADDRESS=unix:path=/host_run/dbus/system_bus_socket npm start &> wifi.log

As for the other issue, there might be a bug in how we save the credentials after a container restart (also fixed in master  but not on this branch). I'll clean up and fix as soon as possible.

@pcarranzav I think there are several issues at play which cause problems with  resin-wifi-connect on Rpi3. Here is what I noticed so far in my tests:

Hotspot via connman tethering with on-board broadcom chip does not work. However hostapd works
Even with correct credentials, connman-simplified-resin cannot join a WPA network. This causes the hotspot to be opened on every reboot
Unhandled rejection Error: Joining network failed (wrong password?) at onChange (/usr/src/app/node_modules/connman-simplifiedresin/lib/WiFi.js:256:23) at emit (events.js:98:17) at null.<anonymous> (/usr/src/app/node_modules/connman-simplified-resin/node_modules/connman-api-resin/lib/service.js:87:12) at emit (events.js:117:20) at null.<anonymous> (/usr/src/app/node_modules/connman-simplified-resin/node_modules/connman-api-resin/node_modules/dbus/lib/bus.js:43:22) at emit (events.js:106:17)at /usr/src/app/node_modules/connman-simplified-resin/node_modules/connman-api-resin/node_modules/dbus/lib/dbus.js:70:42
Standalone connmanctl using the same SSID/password works fine though! Maybe some agent problem?
Systemd and connman have some issues on Rpi3 which I have not noticed on Rp2. For example, if connman service is running with INITSYSTEM on, the SSH web terminal does not work anymore.

@tlangmo about the systemd problem, yes, it's because this branch doesn't have this fix:
https://github.com/resin-io/resin-wifi-connect/commit/ce4da53f5961020a3d1e558daa8b626c2524bccc
(basically the systemctl disable connman on the Dockerfile avoids a connman daemon starting on the container and conflicting with the host connman)
Also the connman-simplified-resin version is outdated (master has 0.1.5), so probably the join error is fixed there (I recall seeing that in the past).
I'll PR fixes for this as soon as I can.

@tlangmo think could you try the branch rpi3-rebased to see if it fixes those issues?
I interactively rebased from master so that the newest fixes are applied, but haven't been able to test it yet.

@pcarranzav I did more testing with rpi3-rebased and found s.th. interesting. The good news, it works in principle, but not right away.
After the AP is up, and you enter the credentials the first time, it will enter the AP mode again on next boot. If you then enter the same credentials a second time, it will work from there on.
Looking closer and /data/connections.json, you can see that the second entry got appended to the list. While the first network join fails, subsequent will work. Here is a print out the /data/connections.json file and the corresponding node error msgs. I added a non existing entry manually for testing.
[{"ssid":"Motesque 2.4Ghz","passphrase":"P@ssw0rd"},{"ssid": "noexist", "passphrase" : "whoknows"}, {"ssid":"Motesque 2.4Ghz","passphrase":"P@ssw0rd"}]
Trying to join previously known networks

Trying to join Motesque 2.4Ghz pwd:P@ssw0rd
Error joining network { [Error: Joining network failed (wrong password?)]
  cause: [Error: Joining network failed (wrong password?)],
  isOperational: true } Error: Joining network failed (wrong password?)
    at onChange (/usr/src/app/node_modules/connman-simplified-resin/lib/WiFi.js:256:23)
    at emit (events.js:98:17)
    at null.<anonymous> (/usr/src/app/node_modules/connman-simplified-resin/node_modules/connman-api-resin/lib/service.js:87:12)
    at emit (events.js:117:20)
    at null.<anonymous> (/usr/src/app/node_modules/connman-simplified-resin/node_modules/connman-api-resin/node_modules/dbus/lib/bus.js:43:22)
    at emit (events.js:106:17)
    at /usr/src/app/node_modules/connman-simplified-resin/node_modules/connman-api-resin/node_modules/dbus/lib/dbus.js:70:42
No success Error: Joining network failed (wrong password?)

Trying to join noexist pwd:whoknows
Error joining network { [Error: Service not found] cause: [Error: Service not found], isOperational: true } Error: Service not found
    at /usr/src/app/node_modules/connman-simplified-resin/node_modules/connman-api-resin/lib/connman.js:228:23
    at mgr.GetServices.finish (/usr/src/app/node_modules/connman-simplified-resin/node_modules/connman-api-resin/lib/connman.js:160:5)
    at /usr/src/app/node_modules/connman-simplified-resin/node_modules/connman-api-resin/node_modules/dbus/lib/interface.js:49:10
No success Error: Service not found

Trying to join Motesque 2.4Ghz pwd:P@ssw0rd
Joined! Exiting.

@pcarranzav Actually, I had instances now  where also the second time it did not work with the same No success Error: Joining network failed (wrong password?)  error message. So bottom line, something works but not reliable.

@tlangmo maybe you've thought of this, but is it possible that the signal is just not so strong? Could you try moving closing to the router and seeing if it works more consistently?
It might be worth just adding a few retries to the join...

@pcarranzav I tried a lot already... Signal is not the issue, I tried two different location (home & office), also very close to router. I also cross checked with Rpi2, everything works fine on that device...

@tlangmo sorry for the delay. I'll be working on debugging this throughout next week - I'll keep you updated!

@pcarranzav I checked the wifi connection mechanism using the python test scripts of connman (https://sources.debian.net/src/connman/1.21-1.2/test/) itself. That worked without problem meaning on every start up, the wifi is joined with known credentials using dbus messages. I ran the agent in a separate process.
So my guess is the problem lies somewhere in one of the node.js abstractions.

@tlangmo thanks for digging in. I've pushed a version that adds some retries - doesn't solve the core issue but might get you going while I keep on researching.

@pcarranzav tried it out. The SSID's never populate. Any hints as to where the issue may be?

+1 did not work for me either.

@pcarranzav this PR seems to address the issue https://github.com/pcarranzav/connman-simplified/pull/2

this is stuck until #42 is fixed

rpi3-rebased branch works for me after recent changes.  THANK YOU SO MUCH!!

RaspberryPi 3 support is now merged to master


lxd: Prevent internal cluster migration of instances with backups

Fixes #7959

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

Open to ideas about testing. test/suites/clustering.sh seems like it could work but AFAICT lxc export deletes the backup after the tar.gz is generated.

You can use lxc query -X POST /1.0/instances/NAME/backups to create a non-expiring backup, that should work.
Also, your commit message needs the Signed-off-by: Firstname Lastname <email> line.

I was having trouble getting these tests to run in my environment but hoping they will pass here.

jenkins: test this please

jenkins: test this please

@stgraber the jenkins: test this please does not seem to trigger  a test suite run on this PR, is it a permissions issue? Thanks

Possibly, I'll have to check how that stuff is configured in Jenkins again...

jenkins: test this please

Looks good, thanks!


Should remove app.kubernetes.io/version label from label selectors

Report
When upgrade from KEDA v2.0.0 to v2.2.0, we encountered the following issue
Deployment.apps "keda-operator" is invalid: spec.selector: Invalid value: v1.LabelSelector{MatchLabels:map[string]string{"app":"keda-operator", "app.kubernetes.io/part-of":"keda-operator", "app.kubernetes.io/version":"2.2.0"}, MatchExpressions:[]v1.LabelSelectorRequirement(nil)}: field is immutable
Expected Behavior
We should be able to upgrade KEDA without any issue
Actual Behavior
Deployment.apps "keda-operator" is invalid: spec.selector: Invalid value: v1.LabelSelector{MatchLabels:map[string]string{"app":"keda-operator", "app.kubernetes.io/part-of":"keda-operator", "app.kubernetes.io/version":"2.2.0"}, MatchExpressions:[]v1.LabelSelectorRequirement(nil)}: field is immutable
Steps to Reproduce the Problem

Install KEDA by https://github.com/kedacore/keda/releases/download/v2.0.0/keda-2.0.0.yaml
upgrade KEDA with https://github.com/kedacore/keda/releases/download/v2.2.0/keda-2.2.0.yaml

Logs from KEDA operator
example

KEDA Version
2.1.0
Kubernetes Version
1.18
Platform
Any
Scaler Details
No response

Relevant kustomize issue: https://github.com/kubernetes-sigs/kustomize/issues/1009


Solutions to ch15

Hi Aurelien,
I know you are busy. If time permits can you upload the solutions to chapter 15. Thanks. Pardon me to ask this, did you include any new materials to version 2 of this book Hands-On Machine Learning with Scikit-Learn & TensorFlow.

Hi @s1782662 ,
Yes, I'm sorry there are still a few missing exercise solutions in the book, I never had time to do them. I'm  writing the 2nd edition of the book right now, so that's my priority, but when it's over I'll have time to work on the exercise solutions. In the 2nd edition:

chapters 1 to 8 (introduction to Machine Learning using Scikit-Learn) are mostly the same, except I updated them to the latest versions of all libraries (e.g., using Scikit-Learn 0.20's ColumnTransformer class rather than the current mix of FeatureUnion, Pipeline and custom transformers).
I added a new chapter on Unsupervised Learning techniques (clustering, novelty and anomaly detection, mixture models...).
chapters 9 (intro to tensorflow) is completely rewritten for TensorFlow 2.0. I also moved it to be after the new tf.data chapter.
chapters 10 (intro to neural nets) is about 50% rewritten to introduce the Keras API and TensorFlow 2.0's implementation (tf.keras) and use it to build the first neural nets.
chapter 11 (techniques to build deep nets) is about 50% rewritten to use high-level TF code (i.e., tf.keras) instead of low-level code.
chapter 12 (distributed TensorFlow) is completely rewritten and pushed to the end of the book. It also contains more info on deployment (to GCP, mobile...).
There's a new chapter on loading & preprocessing data with tf.data and TFRecords.
Chapters 13, 14, 15 and 16 (CNNs, RNNs, Autoencoders and Reinforcement Learning) will probably be about 50% rewritten to use TensorFlow 2.0. I also plan to add more examples and more content (e.g., on object detection, semantic segmentation, GANs, etc.)

In short, the first part has changed little expect for a new chapter on unsupervised learning, but the second part has huge changes, due to the fact that (1) TensorFlow 2.0 is a big change, (2) Deep Learning is a fast-moving field.
Hope this helps


feat!: implement config cat web provider

This PR

Implements ConfigCat Web Provider as discussed here #816.
It mainly moves the context and result mapping logic to a shared package and implements the usage of snapshots for the web-sdk as discussed with @adams85.

@adams85 it took some time to find time to implement, but here it is, as we discussed in #816.
I could not add you as a reviewer, because you are not part of the org. If you like to join the org, I can add you :)
Would be happy about a review.

This is my only concern.

Hey @adams85, took me quite some time to get to it but I just change the two things you proposed :)
Would be happy about a re-review!
I changed the mapping to support your new typings for the User object and only allow AutoPollfor the web sdk.
Marking this as breaking is the User mapping changed.

Thanks for the review @adams85!
I like your proposals and implemented them.
Would be happy if you could have a look again!
I left one question regarding handling the client getting back to ready status after an error.


fix(ffe-form-react): Tooltip handles aria-controls better

Fixes two minor bugs in Tooltip concerning aria-controls:

If children is provided to Tooltip, keep them in the DOM when not expanded or aria-controls
will refer to a non-existing element
If children is not provided to Tooltip, support letting the consumer provide one and use
it on the toggle-button

Fixes #633

Coverage decreased (-0.03%) to 82.023% when pulling 4d9a6becebaf535487c82b312a858ac2f48df4d5 on ivarni:633-aria-controls-tooltip into eacfd0a7ca74ed53e00c5e3bf6a268b24ca0f9a6 on SpareBank1:develop.


Update database fields through the needs list/ supply quotations

[ ] For needs list, update: Item Name, Item type, Unit into the database on upload of the needs list instead of doing a manual upload.
[ ] For the supply quotations side, update: Supplier Name into the database on upload of the supply quotations instead of doing a manual upload.

@manuranga @jayasanka-sack we have organized a meeting with Arosha for Tues 9.45AM to discuss this further.

Update Supply quotation sheet : unit (tablet, capsule), item type (medicine, device) @akbandara
For @manuranga @AmilaJ
Solution post MVP:

We move out of using the spreadsheets altogether. Design and implement a UI: Search items list, new API for the items table, new API for the supplier entity.
Week of 22nd, @akbandara can work on wireframes/design
Post MVP implementation.

@ayeshLK
As discussed: https://docs.google.com/spreadsheets/d/1qRcb_oSRGlNrPL9UsHbeQ1dDkSLWlTtE80m374fQZLg/edit?pli=1#gid=1776238512
Medical Needs Google doc:

Download Medical Needs from Master Needs tab
Download Supply Quotation from Supply Quotation tab

Enclosing the CSV files for dev testing:
SupplyQuotation success dedup.csv
Needs success.csv
Prod testing:
Medical Needs .csv
SupplyQuotation.csv

@nshamit I hv added the PR to create the non-existing medical-items when uploading medical-needs csv file. But, to create a medical-item, we need to have a type for the medical item (Medicine/Device etc.). But this information is not in the uploaded csv file. Hence, I hv added the default value for this as Medicine.
Could you please verify whether this is okay ?

@manuranga @akbandara please provide more info here.
The type information is located in the Master Needs tab (google sheet below) but the information is not brought into the system.
https://docs.google.com/spreadsheets/d/1qRcb_oSRGlNrPL9UsHbeQ1dDkSLWlTtE80m374fQZLg/edit?pli=1#gid=1776238512
Is there a way to define the columns that download in the CSV and include column G from master needs?

We can't specify which columns of the spreadsheet get downloaded in the CSV - all the columns are included in the download. Here are the latest versions
Medical Needs - Master Needs.csv
of the CSVs to use when testing
Medical Needs - SupplyQuotation.csv

@ayeshLK, @manuranga - @akbandara provided the following feedback about this issue fix in dev:
I have managed to test the needs upload and it is working now.
However, the supplier quotation upload has an issue due to missing data in linked tables - MEDICAL_ITEM and SUPPLIER.
It would be good to discuss the best way to add/update entries in the MEDICAL_ITEM and SUPPLIER table.
A suggestion for getting this functionality in the short term is to have CSV upload capabilities for MEDICAL_ITEM and SUPPLIER tables - with the same type of behaviour as the other CSV upload functions.


help

sorry its not issue just wanted to ask with theme is this picture is because I can't find it

which**

/question

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


Extends 'teams user app list' command to expand teamsApp. Closes #4128

Extends teams user app list command to expand teamsApp relationship to the existing $expand query parameter and add that to the output. Closes #4128

Thank you @nanddeepn
Will review ASAP!

@nanddeepn could you also update the response in the docs please? Thank you!

Merged manually, thank you very much!


Finding an IL grandmaster

I need the IL for
    Steam.x(ValidateAuthTicketResponse_t x)
        {
            if (x.m_eAuthSessionResponse == EAuthSessionResponse.k_EAuthSessionResponseOK)
            {
                SteamPending item = null;
                int num = 0;
                while (num < Steam.p.Count)

to include
    if(RocketPermissionsManager.CheckValid(x.CSteamID)){
        return;
    }

in the first line and finally look like:
    Steam.x(ValidateAuthTicketResponse_t x)
        {
            if(RocketPermissionsManager.CheckValid(x.CSteamID)){
                return;
            }
            if (x.m_eAuthSessionResponse == EAuthSessionResponse.k_EAuthSessionResponseOK)
            {
                SteamPending item = null;
                int num = 0;
                while (num < Steam.p.Count)

We need this for:
#1 Canceling sending and receiving packages in SteamChannel.Send & SteamChannel.Receive
#2 Intercept DamageTool.damage()
#3 Properly implement Whitelist & Protected Slots in the example above

I am my IL grandmaster ;)

I guess the IL code I gave you from |-o-| Positive didn't help then?


mcaselector: init at 2.2.2

Description of changes
https://github.com/Querz/mcaselector
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

argh, I should stop working on stuff early in the morning :sweat_smile:.
I must admit that I based my changes on the derivation of amidst and I didn't catch these myself. Thanks for the heads-up either way :D


Command to update alex

It will be convenient if we can update alex with alex command, not ugly pub global activate alex.
Also may be worth to implement some sort of "new version available" notice.

https://pub.dev/packages/pub_updater


IOS - I don't want a white screen to appear when the screen is all the way down.

I don't want a white screen to appear when the screen is all the way down.
'disallowOverScroll: true'
I gave the above option, but it doesn't work.
Is there another way?

Same here