[Question] Sync partial results

Hello, what is the status of paginated results in sync?
I have messages that belong to a room.
i would need sync last 5 messages per room.
I cant seem to find how to do this.
Is there something im missing or is this not implemented yet? If not implemented, what would be the timeframe?
Thank you!

👋 we've been working the last month on a rebuild of the Electric server over at a temporary repo https://github.com/electric-sql/electric-next/
You can read more about why we made the decision at https://next.electric-sql.com/about
We're really excited about all the new possibilities the new server brings and we hope you'll check it out soon and give us your feedback.
We're now moving the temporary repo back here. As part of that migration we're closing all the old issues and PRs. We really appreciate you taking the time to investigate and report this issue!


Allow Base URL with Path

Cube's REST API by default is located as subpaths of the path /cubejs-api/v1.
However, when users deploy their Cube service behind infrastructure like API gateways, load balancers, reverse proxies, or service meshes, they can configure the service to be exposed under a different path namespace. E.g. /cubejs-api/v1 could become /foo/bar/cubejs-api/v1 to the client.
This change allows the Cube client to accept a Cube service URL that can contain a base path, in addition to the domain and scheme.

seems ok, though I would have thought that this would have already worked, if you accept url.URL in the constructor. Then, you would take the base URL, and append "/load" to it. The issue with the existing code is it was just setting url.Path outright.
If you do this systematically, you don't even need a CubeRESTAPIBasePath, if I'm not mistaken.
However, what you've posted does seem to work so I have no problem forging ahead.

Ah yea; I had gone with that in an earlier commit, but then added the accessor method to help with testing. I've reverted back to the orignal approach now. PTAL?


feat(search): experimental telescope layout (closes #718)

Definitely need some help here I spent wayy too long figuring out TS 😄
But here is the initial look
https://github.com/jackyzha0/quartz/assets/29749331/1df60b6e-7d72-45f8-bdd9-95dec9c19026

oh ye and also the last item and first item need to be only fetched once, and I'm figuring that out atm. So if you have any recommendation that would be great as well.

you might want to mark this as a draft?
also, i feel like that the selection pane (on the left) is a bit too big

Here's the updated styling
https://github.com/jackyzha0/quartz/assets/29749331/d7b0a933-573b-4eb2-ad93-de99d38a48ca

sheesh, i'm pretty packed until fri but I'll take a look as soon as I can, this looks awesome

How this is handled on mobile ?

How this is handled on mobile ?

should just work on mobile afaik

wow this is incredibly cool, very excited! may need some special treatment on mobile, as the space is a bit tight and the ride side panel seems to be blurred, but very cool!

wow this is incredibly cool, very excited! may need some special treatment on mobile, as the space is a bit tight and the ride side panel seems to be blurred, but very cool!

I'm thinking only to show search item on mobile.

Hmm for mobile I just disable preview. lmk what you think about this.

@jackyzha0 I have updated the fetch to memorize the result. Also added flags to control this behaviour. By default it is false, which preserve the previous search behaviour.

https://github.com/jackyzha0/quartz/assets/29749331/51666fa1-bfd8-41c0-b341-720bd294f56c
Final preview for both phone and desktop.

few more add-ons that could be worth taking a stab at for usability

should select/preview the first entry automatically
mouse hover should also preview

i'll do a styling pass tomorrow

few more add-ons that could be worth taking a stab at for usability

should select/preview the first entry automatically
mouse hover should also preview

i'll do a styling pass tomorrow

Now addressed in #774


status quo: Barbara carefully dismisses embedded Future

Rendered
Barbara tries to implement an asynchronous API for embedded systems, and finds that core::future::Future has a very high overhead.

Maybe this report could provide some input on where documentation on how to build and use Futures and executors could be improved?

Building a Future by hand seems easy enough (except for combinators), but I think there is room for better documentation for building executors.
It's been over a year since I ran this experiment, so I'm unsure what the documentation looked like at the time. However, searching for documentation on building executors gives a handful of tutorials, all of which require alloc (even with "embedded" added to my search). A no-alloc executor ends up looking significantly different from the executors in the tutorials.
Ultimately, I realized I had to handle the case where Waker::wake() is called during poll by thinking about what happens in a multithreaded context. I did not find that requirement in any of the documentation I looked at.

One quick note as I read some of the prior comments. Remember that the role of these status quo stories is for people to convey what they experienced. It is explicitly not to propose solutions. Therefore, e.g. when @Matthias247 suggests this:

Maybe this report could provide some input on where documentation on how to build and use Futures and executors could be improved?

I would actually not want that at all. I think the report should focus on what people felt and experienced.
The stories should definitely be technically accurate -- but in some cases, characters thought things that were wrong, or only partially informed. We should be able to capture that, and the FAQ can be especially useful for providing deeper technical info than the character knew at the time.

Thanks so much for writing this up, @jrvanwhy! I'll second @nikomatsakis that I think this can be merged.

One thing that we were discussing on zulip is that some of the comments on this PR might make good FAQs.

One thing that we were discussing on zulip is that some of the comments on this PR might make good FAQs.

I'll send the changes in a separate PR.


{IoT} Fix certificate filename confliction in test

Description

In test_dps_certificate_lifecycle and test_certificate_lifecycle, they use same filenames:
VERIFICATION_FILE = "verify.cer"
CERT_FILE = "testcert.cer"
KEY_FILE = "testkey.pvk"
CHAIN_FILE = "testcert-chain.pem"

These files will be removed after test. When executed in parallel, the FileNotFoundError error sometimes occurs.
Failed test: https://dev.azure.com/azclitools/public/_build/results?buildId=12783&view=logs&j=00feeef1-2e9b-5ce6-e2c5-a2c5a43245ee&t=317d23de-79ee-5aa1-a23a-e6710a3e515f
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

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

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

IoT


Implement Scheduler.isActive

Can probably be implemented with an RxJS subject and a distinct operator.

@jverkoey why is this not blocking next release? :eyebrow: 😕

Everything in a milestone is blocking the next release. Probably going to remove that tag.

🎊 My diff is ready for review at http://codereview.cc/D1300


Can't setup if VS not in default install path

Vintage Story is installed in C:\Program Files for me, but modtools.exe is searching in (x86).
It then asks me for a custom path, which I provide.
I get and error because it reads it as C:\Program.
So I then use double quotes as is standard, and I get nothing either!

Searching for VS in ""C:\Program Files\Vintagestory"..."

Only way this works is if you don't give up just yet and type :

setup "C:\Program Files\Vintagestory"

I guess that would be some escaping issue as Dark pointed out for this repo but in https://github.com/anegostudios/vsmodtools
although the modtools.exe has been deprecated since we can now cover all it needs with the new template
so I will close this here


Asynchronously send the ChatGPT response as the Telegram times out before ChatGPT returns

Seems like Telegram made the timeout a little shorter recently, too short for ChatGPT to generate the response.

It might be only for CallbackQuery that it expects a faster response, testing only using waitUntil for that.


How to chain calls

Hi, I'm trying to chain multiple GET calls, I'm not sure how should it be done correctly (Unity 5.6.6)
        var downloadOps = DownloadTextAsync ("http://www.google.com")
            .Then (text => Debug.Log (text))

          // this does not catch error
          .ContinueWith (op => DownloadTextAsync ("http://www.thisdoesnotexistxxcaas.com").Then (text => Debug.Log (text)))

           // Then doesn't wait
            .ContinueWith (op => DownloadTextAsync ("http://www.thisdoesnotexistxxcaas.com"), AsyncContinuationOptions.ExecuteSynchronously)
            .Then (text => Debug.Log (text))

           .Catch (e => Debug.LogException (e));

Hi, @imtrobin. The intended way of doing what you need is something like this:
var downloadOps = AsyncWww.GetText("http://www.google.com")
	.Then(text => Debug.Log(text))
	.Then(() => AsyncWww.GetText("http://www.thisdoesnotexistxxcaas.com")
	.Then(text => Debug.Log(text)))
	.Then(() => AsyncWww.GetText("http://www.thisdoesnotexistxxcaas.com"))
	.Then(text => Debug.Log(text))
	.Catch(e => Debug.LogException(e));

Please note that github issues is not the right place to ask questions. There is a dedicated forum thread for this library: Unity forums link.

Thanks, noted. Here's a bug, but I think it's Unity 5.6.6 rather than yours. The AsyncWww.GetText is using UnityWebRequest.Get , which .text returns empty even if successful. I've verified this in seperate project and have reported this to Unity


Import Error during using

I am a novice in this field, seek for help...
Operation:
pip install causal-learn(sucessfully)
touch a new file test.py(sucessfully)
copy testPC.py to test.py(sucessfully)
python3 test.py (encounter Error ：
(main) ➜  Test git:(main) ✗ python3 testC.py
Traceback (most recent call last):
  File "testC.py", line 8, in <module>
    from causallearn.utils.cit import chisq, fisherz, gsq, kci, mv_fisherz, d_separation
ImportError: cannot import name 'd_separation' from 'causallearn.utils.cit' (/Users/sheshuaijie/app/miniconda3/envs/main/lib/python3.8/site-packages/causallearn/utils/cit.py)

Any Suggestion will be greatly appreciate!!!!

Hi @Ricardokevins. Which version of causal-learn are you using? This might be due to an outdated version. We'll release a latest one soon and please update to it. Thanks for identifying this :)

Hi, thanks for reaching out. We have just released a version on pypi. Could you please try 'pip install causal-learn==<IP_ADDRESS>' and run your scripts again?

Hi, thanks for reaching out. We have just released a version on pypi. Could you please try 'pip install causal-learn==<IP_ADDRESS>' and run your scripts again?

Thank you for your immediate reply, I just tried and get new Error as follow
Traceback (most recent call last):
  File "ttt.py", line 12, in <module>
    from utils_simulate_data import simulate_discrete_data, simulate_linear_continuous_data
ModuleNotFoundError: No module named 'utils_simulate_data'

Hi @Ricardokevins. Which version of causal-learn are you using? This might be due to an outdated version. We'll release a latest one soon and please update to it. Thanks for identifying this :)

I follow the below instruction and get a new error information

Thank you very much for your quick reply.
Problem fixed with your help


Crashed when output file name is not given

sct_image is supposed to have flag -o as optional, however when not specified, function crashes.
To reproduce:
julien-macbook:~/sct_testing_data/t2 $ sct_image -i t2.nii.gz -setorient RPI

--
Spinal Cord Toolbox (master/9d9b320757fc4a1db5bac3f706a01817c6b351c7)
Running /Users/julien/code/sct/scripts/sct_image.py -i t2.nii.gz -setorient RPI
t2.nii.gz
Traceback (most recent call last):
  File "/Users/julien/code/sct/scripts/sct_image.py", line 623, in <module>
    main()
  File "/Users/julien/code/sct/scripts/sct_image.py", line 243, in main
    im_out = [msct_image.change_orientation(im_in, arguments["-setorient"]).save(fname_out)]
  File "/Users/julien/code/sct/spinalcordtoolbox/image.py", line 422, in save
    raise RuntimeError("Don't know where to save the image (no absolutepath or path parameter)")
RuntimeError: Don't know where to save the image (no absolutepath or path parameter)
Total processing time: 0 min 1 s

Closing as the command in the description no longer crashes. This looks like a fix was implemented incorrectly in #1577 but ultimately fixed by #2371.


Siddhi Plugin Idea implementation

Purpose
This PR contains the implementation for the siddhi idea plugin.
Goals
Providing capability to run and debug siddhi in Intellij IDEA.
Approach
N/A
User stories
N/A
Release note
N/A
Documentation
N/A
Training
N/A
Certification
N/A
Marketing
N/A
Automation tests

Unit tests
N/A
Integration tests
N/A

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Samples
N/A
Related PRs
N/A
Migrations (if applicable)
N/A
Test environment
N/A
Learning
N/A

@Shan1024  FindBugs doesn't allowed to use org.jetbrains.annotations.NotNull instead of javax.annotation.Nonnull.

@RAVEENSR - AFAIS, this is a false negative situation in findbugs. Writing the plugin using jet-brains standards is more important. So suppress the NP_NULL_ON_SOME_PATH rule and use org.jetbrains.annotations.NotNull annotation so the plugin is consistent with jet-brains standards. This false negative issue might get fixed in findbugs in future. IDEA's static analyzing capability will show locations where NP can occur. Use that to identify bugs for now.

@Shan1024 Added NotNull instead of Nonnull in commit d49d467.

@Shan1024 Re-organized the parameters in some method in commit b49a2c2 .


Updating havok to 1.3.10

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15658/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15658/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15658/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15658/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15658/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15658/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15658/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15658/merge#BCU1XR#0

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15658/merge/testResults/webgpuplaywright/index.html

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15658/merge/testResults/webgl2playwright/index.html


Configure required check for existing workflow actions.

Those should be passing for all PRs, and support wildcards for package names.
Validate devcontainer-feature.json files / validate (pull_request) Successful in 3s
CI - Test Features / test-autogenerated (starship, debian:latest) (pull_request) Successful in 33s
CI - Test Features / test-autogenerated (starship, ubuntu:latest) (pull_request) Successful in 33s
CI - Test Features / test-autogenerated (starship, mcr.microsoft.com/devcontainers/base:ubuntu) (pull_request) Successful in 29s
CI - Test Features / test-scenarios (starship) (pull_request) Successful in 43s
CI - Test Features / test-global (pull_request) Successful in 36s

Related to https://github.com/devcontainers/action/issues/180


Help needed: integrating with Capacitor's OAuth2 plugin

Hello,
Capacitor is a new PWA framework that is also going to be integrated into the well known Ionic Framework, which is now based on Cordova (that said, Capacitor will eventually replace Cordova).
I need some suggestions on how the Line Login API could be integrated into Capacitor's OAuth2 plugin. At first, I thought it might look similar to Facebook, but seems Line SDK for Android lacks ready-to-use login activity and callbacks that can be used. I was able to integrate it on iOS, but now struggle with Android. Some examples would be really helpful.
PS: Cordova suggestions would also be helpful as long as they could be adapted to Capacitor, some plugins already exist, but they are based on old SDK versions.

@plateaukao @perthcpe23 @LiYing2010 can anybody help? :)

Close now for having no comments for a while.


Gerrit sync

This commit adds support for gerrit repo indexing.
It supports globs for both including and excluding repos.
It does not support branches or tags (as these are going to be reworked by brendan in the future.
It does not support auth yet (I'd need to spin up my own gerrit instance to test this out) - I can add this later hopefully.
Example URLs for experimentation:
// https://chromium-review.googlesource.com
// https://review.opendev.org
// https://android-review.googlesource.com
Sample config:
{
   "$schema": "./schemas/v2/index.json",
   "repos": [
      {
         "type": "gerrit",
         "url": "https://gerrit-review.googlesource.com/",
         "projects": [
            "apps/**",
            "gerrit**",
            // "modules/**",
            "zuul/**"
         ],
         "exclude": {
            "projects": [
               "gerrit-**"
            ]
         }
      }
   ]
}

https://gerrit.wikimedia.org/
this is another example of a publicly available gerrit

https://gerrit.wikimedia.org/ this is another example of a publicly available gerrit

Hi, I am working for Wikimedia and the current maintainer of that Gerrit instance. Thank you for having added support for Gerrit :100:


Change the way we predicate steps

As-is these aren't disabling Snyk for me with:
    - uses: distroless/actions/vul-scans@main
      with:
        registry: ghcr.io
        username: ${{ github.actor }}
        password: ${{ github.token }}
        image: ${{ matrix.ref }}
        RUN_SNYK: "false"

I'm going to try with this change.

I have a successful run of this downstream on distroless/static (with snyk disabled) here: https://github.com/mattmoor/static/runs/7367313315?check_suite_focus=true#logs

Best me to it. Was going to fix this Monday. Glad you did and tested it.
Lgtm


dynamic filter: inconsistent delete since #7374

Describe the bug
SQL:
create table latest_cart(id int, created_at timestamp, updated_at timestamp);
create materialized view test as select id, created_at, updated_at from latest_cart where created_at > now() - interval '1 minute';

will cause error since #7374:
2023-01-26T10:44:07.581914Z ERROR risingwave_stream::task::stream_manager: actor exit actor=18 error=Executor error: Inconsistent Delete - current: Some(Some(Int64(1674729787580000))), delete: [Some(Int64(1674729786581000))]

To Reproduce
SQL:
create table latest_cart(id int, created_at timestamp, updated_at timestamp);
create materialized view test as select id, created_at, updated_at from latest_cart where created_at > now() - interval '1 minute';

Expected behavior
No response
Additional context
#7374 makes now() operator not single.

https://github.com/risingwavelabs/risingwave/pull/7374 makes now() operator not single.
This is not accurate. The now operator was not single as fragmenter requires manually specifying that fragment is_singleton.

This was not caught, and previously, we would have spawned a now() operator for as many actors as there are downstream.


Update changelog

@JericHunter / @simorenoh  - did we make the version changes separately?
Shouldn't it be part of the release PR - (changelog + version changes).

@kushagraThapar this was the PR generated by the command - I think the version PR with the new changelog entry gets updated after the release is done


stress-ng: 0.16.04 -> 0.17.00

Description of changes
https://github.com/ColinIanKing/stress-ng/blob/master/debian/changelog
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

[x] 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.

Thanks!


Updated version

Thank you very much for your time spending on this project. It really helped me to understand the BTHome protocol.
In tried to use the code as an example and unfortunately it didn't work so I tried with fully respect to update it.
Here you can find an updated version:
https://github.com/Chreece/BTHomeV2-ESP32-example
I tried to make it more user friendly, using a header file with variables for the Object IDs and boolean values.
Also added an auto length calculation.
The only thing that needs to be changed for an implemantation is the device_name and the payload for the entities.
Unfortunately I haven't also understand the encryption documentation and couldn't implement it but I will keep up on it...
I hope you like the new version of your code :)
Keep up!

Thanks Chreece!
My goal of uploading my example code was just to give people a quick starting point for BTHome on ESP32's.
I had hopes of evolving it and adding more examples and functionality, however I have not had time. So I greatly appreciate you picking it up and continuing the work on it.
I have put a link to your project on my main page.
I guess I will keep my project up for old BTHome V1, and yours can focus on V2
Thanks again!

Great attidute man!
I'm not a programmer, just a hobbist. I had in mind to try build a DIY Bluetooth people counter and because of that I stubled upon  your project. It is a start for everyone who wants to build a DIY BT sensor.
In that in mind i try to made the code simplier and just now I upload the V2 with minimal user input.
It just needs the device name, the number of the sensors (entities) that will be send and the states of them. The rest is done under the hood and not in the .ino file that was edited by the user.

The V3 of the example should have also encryption if I will understand it or if chatgtp helps me :)
Keep Up!

Link for BTHome V2 example in Readme


Exporting 'Extract operation history' as JSON

Currently if I want to export the Extract Operation History list of actions, I manually copy paste the JSON and save as a .json file. Would it be possible to add an Export button with a traditional 'save as' dialog?

Hello, I would be interested on working on this!

I have pushed some code! What are the next steps?
Thanks! :)

Oh wow amazing, thank you! Can’t wait to try this out!

If you literally cannot wait then we have snapshot releases for you: https://github.com/OpenRefine/OpenRefine-snapshot-releases/ (not that the Google integration does not work in them yet).


backport 1274: common: ability to disable handler osd health check

Signed-off-by: Sébastien Han<EMAIL_ADDRESS>

backport of #1274


steam: fix crash at startup with open source radeon drivers

Motivation for this change
This is basically a port of this fix from OpenSUSE.
Steam ships statically linked with a version of libcrypto that conflics with the one dynamically loaded by radeonsi_dri.so.
This provides a compatible one (w/ engine disabled, cpuid fix) and links it into the steam runtime.
The error looks like:
steam.sh: line 713: 7842 Segmentation fault (core dumped)

Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

This broke the game "Wargame: European Escalation" for me.
relocation error: /home/freddy/.local/share/Steam/steamapps/common/Wargame European Escalation/Wargame: symbol ENGINE_load_builtin_engines, version OPENSSL_1.0.0 not defined in file libcrypto.so.1.0.0 with link time reference

@FRidh Could you open an issue for this and tag me on it?
It looks like ubuntu has a version-script.patch for openssl which adds these version symbols.
I can try to build openssl-steam with that patch, but I might need your help to test it.


Save addresses in device.txt

I have problems with saving found LE-devices in the file 'device.txt'. First I do a scan of LE-devices with btferret. First I do a LE-scan with btferret:
'> b'
.... This is my device:

3 FOUND 00:1B:10:4F:FFFFF - Fixed
    Flags = 06
    Manuf specific = 50 44 43 53 CB 54 77 64 00 00 B8 AD 4F 10 1B 00
    Tx Power Level = 0A
    No name so set = LE node 1003
    New device LE node 1003
....

Then connect to node 1003 and all the rest works correctly.
But if I try to save the device in 'devices.txt' as
DEVICE = Chessnut  TYPE=LE node=1 ADDRESS =  00:1B:10:4F:FFFFF

I get an format error:
Device data from devices.txt file
DEVICE = Chessnut  TYPE=LE node=1 ADDRESS =  00:1B:10:4F:FFFFF
Hex format error - odd number of digits
   ERROR **** Device address must be 6 bytes
************ initblue() FAILED ************

So I have to scan for LE-devices each time at program start, which is a little bit annoying🙁.
Can you tell me what would be the correct entry for this?

This looks like a difference between the Raspberry Pi C compiler/library and the one you are using because I cannot reproduce it on a Pi with gcc. The address should be 6 bytes separated by 5 colons - 11:22:33:44:55:66. The FFFF value appears because the compiler is treating that number as a signed byte rather than unsigned. Try this fix. If it works, let me know and I'll update the code to account for other compilers.
In btlib.c - change char to unsigned char as follows:

Line 363
char *baddstr(unsigned char *badd,int dirn);

Line 2003
char *baddstr(unsigned char *badd,int dirn)

And if that does not work:

Line 2014
sprintf(s+3*n,"%02X:",(unsigned char)badd[k]);

I modified only line 2014 in btlib.c.  Now it seems to work with my gcc : gcc version 9.4.0 (Ubuntu 9.4.0-1ubuntu1~20.04.1)
This is the new output of LE-scan:
> b
Scanning for LE devices - 10 seconds..
...
2 FOUND 00:1B:10:4F:AD:B8 - Fixed
    Flags = 06
    Manuf specific = 50 44 43 53 CB 54 77 64 00 00 B8 AD 4F 10 1B 00
    Tx Power Level = 0A
    No name so set = LE node 1002
    New device LE node 1002
....

After adding the address  00:1B:10:4F:AD:B8 to devices.txt all works perfectly. Thanks again!

OK, thanks. Seems like inconsistent sprintf behaviour.


PP-5950 Release new version of product page

See https://github.com/alphagov/pay-product-page/commit/112535e4ffb6305e0a0547398067530a094da41b

Well done, so proud

lmao


Documentation for defining scopes seems to be out of date

I hope this is the right place to report this, apologies if not (I tried updating the doc myself but clicking on the "edit on github" link resulted in a 404.
On https://dbsphinxtest.readthedocs.io/en/latest/start/scopes.html it says:

For this document we gonna use the in-memory version of the scope store. You can wire up the in-memory store in ConfigureServices via the AddInMemoryScopes extensions method.

which seems to be incorrect.  I believe the new way is via AddInMemoryApiResources

Nevermind, I just noticed the url was not from the official documentation.


Lose test location after a compile

Describe the bug
The test explorer items are rebuilt after a compile, and we the location of test items is lost

Fixed in #462