[HxGrid] InfiniteScroll + synchronous DataProvider returning empty data => DOMException: Failed to execute 'setStartAfter' on 'Range': the given Node has no parent.

If there is HxGrid with ContentNavigationMode="GridContentNavigationMode.InfiniteScroll" (Virtualize) and the DataProvider delagate returns empty data synchronously, the underlying Virtualize emits JavaScript error to the console:

blazor.webassembly.j…1pV1-b5sQ3Rj11QGI:1
 Uncaught DOMException: Failed to execute 'setStartAfter' on 'Range': the given Node has no parent.
    at https://localhost:44376/_framework/blazor.webassembly.js?v=wC-zAyYHVTNzevCw3SFvHI4jG51pV1-b5sQ3Rj11QGI:1:28726
    at Array.forEach (<anonymous>)
    at IntersectionObserver.<anonymous> (https://localhost:44376/_framework/blazor.webassembly.js?v=wC-zAyYHVTNzevCw3SFvHI4jG51pV1-b5sQ3Rj11QGI:1:28677)
(anonymous)	@	blazor.webassembly.j…1pV1-b5sQ3Rj11QGI:1
(anonymous)	@	blazor.webassembly.j…1pV1-b5sQ3Rj11QGI:1

In InfiniteScroll mode, we set shouldRenderData to true when totalCount == null to render the Virtualize which initiates the DataProvider request. As the DataProvider returns empty data, we re-render with shouldRenderData = false and remove the Virtualize from DOM. It seems it disappears from the DOM too quickly and the supporting JavaScript (MSFT) logs the error to console (not causing the Blazor app to fail!).
Related AB#63587.
Workaround (for the scenarios where the Data is available in advance):
<HxGrid ContentNavigationMode="Data.Any() ? GridContentNavigationMode.InfiniteScroll : GridContentNavigationMode.Pagination" .../>

cc @alexax578 @jirikanda

@hakenr Don't ask me why it helps ;-).

I tried alternative approach and it also works. So no need to use Task.Yield.
BUT there is some issue with CSS, in this case, then I hover the empty template, it showes gray background (while in other scenarios it does not).

@crdo Can you please check the mentioned visual issue in PR? Maybe @Harvey1214?

@hakenr @jirikanda It is set as hover-able table. So I guess it is expected behaviour...

Agree, I missed thad.
Thx.


Update offering-nz-cc-framework-nz.md

REmoved  "Microsoft NZ has worked with the GCIO team to develop a reference architecture for integrating Exchange Online and SEEMail described in the white paper Office 365: SEEMail Integration and Reference Architecture as this is old and inaccurate.
@robmazz  let us know if there is another version of this document so we can replace it.
Thanks
Sri

@JohanFreelancer9  Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/2653

No newer doc - we're advising agencies and partners on an ad-hoc basis. Thanks for taking this down as we've been causing some confusion 🙄


Error: EACCES: permission denied

Hello,
Since the latest update (2.3.17) I got the same access error in both machines (used it as a dependency for node-osm-stream version 0.2.2 on Ubuntu machines):

node-gyp rebuild

gyp ERR! configure error
gyp ERR! stack Error: EACCES: permission denied, mkdir ''

Try
npm install node-expat --verbose --unsafe-perm=true


Add Partial Unbond to UI

I think the image explains it well. Since begin-partial-unbond is here (or coming soon), we should be able to allow for something like this, where the user can enter a specific amount of LP tokens to unbond:
We may be able to eventually have some kind of combined 'Manage LP tokens' or 'Manage bondings' section/prompt for a cleaner interface (like how Sushi has), but for now you should be able to bascially reuse the 'Bond LP tokens' prompt for allowing to specify for partial unbond. Since LP tokens aren't always in user friendly numbers (in terms of readability), perhaps an option to enter a % of bonded tokens and showing $ value of specified amount could also be nice.

Unfortunately, LP bonding positions are still not fungible. Each bonding tx has its unique bonding tx ids (similar to NFTs), therefore I think trying to accommodate a UI for this may be too complex for the average user at this time.
Will re-approach this issue once the bonding positions become fungified.


Testing: Repositories must exist

Require that all repositories defined must exist on the file system.  Thank you! :tada:


Add bluesky to people.json

Add bluesky field to https://github.com/cncf/people/blob/main/people.json

The people.json file is already set up for bluesky profiles. You can see an example here or in the schema here.

Thank you @cjyabraham that's awesome !


Make it clear what the promise will catch

Or just remove any attempt at explaining :)

@freeall Thanks for the suggestion. Although you are technically correct about the final catch block handling failures that might be unrelated to JSON, our examples are provided for simplicity, and we don't plan to cover all the nuances of promise-based exception handling. Therefore, I feel that this example is fine if left unchanged.
I'm open to accepting a separate example that shows how to handle failures coming directly from fetch call separately from handling exceptions that might have happened in any processing of the response afterwards.

@mislav How would you actually do that? Is it possible to check the ex (err?) and see which error it was?

@freeall
fetch(...).then(function(result) {
  // do whatever with result
}, function(err) {
  // this handles network errors
}).then(function() {
  // do whatever with result
}).catch(function(result) {
  // handle all other errors
})


[WIP][UI] Source/target storage

Motivation and context

How has this been tested?

Checklist

[ ] I submit my changes into the develop branch
[ ] I have added a description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

/check

/check

/check

/check

Hi @Marishka17 ! Thanks for the contribution. I was wondering whether all the data is still downloaded from source to container before being exported to target storage ?
S3 source manifest + files -> Local container volume such as /home/django/data -> S3 target manifest + job _annotations.zip
Would be interesting albeit maybe complex to avoid storing locally large chunks of data and directly interact with the S3 target bucket ?


Create a Consumer App

Tutorial URL: https://developers.sap.com/tutorials/cp-cf-rabbitmq-create-consumer.html
Please specify the step you are referring to 7
Formatting appears to be a little incorrect:
applications: - name: consumer random-route: true memory: 128MB disk_quota: 128MB instances: 1 buildpack: nodejs_buildpack command: node app.js

services: - pubsub-messagingservice`` >The propertyrandom-route` will generate a route that does not conflict with any other application in the same Cloud Foundry instance.

The command cf push reads this file and deploys the application to SAP Cloud Platform. You can find the generated route / URL in the console, once the deployment has finished.

Thanks for reporting this. I just fixed the issue and you should be able to see the updated version shortly


post-installation script subprocess returned error exit status 6

getting an error why trying to install aiy-voicebonnet-soundcard-dkms
sudo apt-get install -y aiy-voicebonnet-soundcard-dkms
any help would be appreciate, I know possibly there have been some changes on the aiy side that may no longer be compatable...also changes to rasbperry OS.  looking like a kernal incompatability
Error! Bad return status for module build on kernel: 5.15.61+ (armv6l)
Consult /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/make.log for more information.
dpkg: error processing package aiy-voicebonnet-soundcard-dkms (--configure):
installed aiy-voicebonnet-soundcard-dkms package post-installation script subprocess returned error exit status 10
Errors were encountered while processing:
aiy-voicebonnet-soundcard-dkms
E: Sub-process /usr/bin/dpkg returned an error code (1)

DKMS make.log for aiy-voicebonnet-soundcard-3.0 for kernel 5.15.61+ (armv6l)
Mon 17 Oct 16:47:58 BST 2022
make: Entering directory '/usr/src/linux-headers-5.15.61+'
CC [M]  /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.o
In file included from /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.c:25:
/var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/rt5645.h:12:10: fatal error: sound/rt5645.h: No such file or directory
12 | #include <sound/rt5645.h>
|          ^~~~~~~~~~~~~~~~
compilation terminated.
make[1]: *** [scripts/Makefile.build:289: /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.o] Error 1
make: *** [Makefile:1898: /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build] Error 2
make: Leaving directory '/usr/src/linux-headers-5.15.61+'

Here you can find up to date drivers: https://github.com/google/aiyprojects-raspbian/issues/697

DKMS make.log for aiy-voicebonnet-soundcard-3.0 for kernel 5.15.61+ (armv6l) Mon 17 Oct 16:47:58 BST 2022 make: Entering directory '/usr/src/linux-headers-5.15.61+' CC [M] /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.o In file included from /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.c:25: /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/rt5645.h:12:10: fatal error: sound/rt5645.h: No such file or directory 12 | #include <sound/rt5645.h> | ^~~~~~~~~~~~~~~~ compilation terminated. make[1]: *** [scripts/Makefile.build:289: /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.o] Error 1 make: *** [Makefile:1898: /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build] Error 2 make: Leaving directory '/usr/src/linux-headers-5.15.61+'

DKMS make.log for aiy-voicebonnet-soundcard-3.0 for kernel 5.15.61+ (armv6l) Mon 17 Oct 16:47:58 BST 2022 make: Entering directory '/usr/src/linux-headers-5.15.61+' CC [M] /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.o In file included from /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.c:25: /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/rt5645.h:12:10: fatal error: sound/rt5645.h: No such file or directory 12 | #include <sound/rt5645.h> | ^~~~~~~~~~~~~~~~ compilation terminated. make[1]: *** [scripts/Makefile.build:289: /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.o] Error 1 make: *** [Makefile:1898: /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build] Error 2 make: Leaving directory '/usr/src/linux-headers-5.15.61+'

DKMS make.log for aiy-voicebonnet-soundcard-3.0 for kernel 5.15.61+ (armv6l) Mon 17 Oct 16:47:58 BST 2022 make: Entering directory '/usr/src/linux-headers-5.15.61+' CC [M] /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.o In file included from /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.c:25: /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/rt5645.h:12:10: fatal error: sound/rt5645.h: No such file or directory 12 | #include <sound/rt5645.h> | ^~~~~~~~~~~~~~~~ compilation terminated. make[1]: *** [scripts/Makefile.build:289: /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build/snd-aiy-voicebonnet.o] Error 1 make: *** [Makefile:1898: /var/lib/dkms/aiy-voicebonnet-soundcard/3.0/build] Error 2 make: Leaving directory '/usr/src/linux-headers-5.15.61+'

I am having the same issue, did @denics solution work for you?


Code optimization

There is a bottleneck for huge git patches (more than 1M lines).
Try to run the added unit test - it requires a lot of time.
It takes a lot less time with the optimization.

PS: tested on linux only. Python 3.9

@cscorley , pls, check whether python3.5 and 3.6 is available for your repo (actions)

Is this approach easily applicable for the other types of patch parsers?

Is this approach easily applicable for the other types of patch parsers?

It seems, a bit different approach   https://github.com/babenek/whatthepatch/tree/tmp. I do not know very well the file types.

Thanks!


document a policy to name variables under var/

a rule to name variables under var has been prefixing __, or double underscores, to avoid name collision. similarly all registers, which are dynamically assigned in roles, start with register_.
this rule should be documented.
related issue:

https://github.com/reallyenglish/ansible-role-example/issues/42

similarly all registers, which are dynamically assigned in roles, start with register_

@trombik couldn't we come up with a bit shorter one?

Some more thoughts on this.
Instead of editors_to_add I prefer editor_packages. If you need to introduce feature for removing editors, you simply add state: absent attribute instead of adding editors_to_remove.
Instead of dict type variable virtual_interface and virtual_interface_to_remove I prefer list of virtual_interface_configs with dict members having state: present by default, state: absent to remove. I am new to ansible but it seems popular practice to use state attribute this way.
In general, list of dict is always superior to dict. List has definite order, but dict not. Sometimes order matters. Also it could be necessary to remove something first then add something, or vise versa. List supports this, dict not.


Wk/taco 231 fix client pool

fixed the client pool by swapping out the channel handler associated with the front-end when reusing a recycled client

There were 2 problems:
1 (found by @khappucino ) clients acquired by the pool were not being attached to the channel handler ctx
2 the client "app handler" (ProxyBackendHandler) was associated with the original front-end channel on pooled clients

Coverage increased (+0.3%) to 43.477% when pulling e10a01864900002af76ad0658f73527bf421dbf5 on wk/TACO-231-fix-client-pool into c6975742920dfc80e10f81c984260a5c2c6ad2dc on master.


treesitter missing tests in summary/status

First of all I appreciate this is a bit of a niche configuration but wanted to raise an issue to see if anyone had any advice for fixing it as I tried looking into it but was unsuccessful.
In a project I work in, we have a series of tests that test different API endpoints and in these test suites we have multiple describe blocks, one of which is returned from a function that conditionally runs tests (see example below). The issue is that as far as I can tell treesitter will only parse the text in the file itself, it won't evaluate any functions in the file to see what they return and whether that can be a test or not which leads to tests being missed in the discovery window / test statuses in the gutter.
Example implementation below:
// foo.test.js
import {functionReturnsTests} from './test-generator.js';

describe('Foo Test', () => {
   describe('Describe One', () => {
        it('runs some test', () => {});
    });

    functionReturnsTests({runTestOne: true, runTestTwo: false});
}

// test-generator.js

function testOne() {
   return  it('runs a test', () => {});
}

function testTwo() {
   return  it('runs a second test', () => {});
}

export function functionReturnsTests({runTestOne, runTestTwo}) {
   return describe('Describe Two', () => {
       if (runTestOne) testOne();
       if (runTestTwo) testTwo();
   });
}

In this example I would expect the discovery panel to look like the below.
- foo.test.js
   - Foo Test
       -  Describe One
          - runs some test
       - Describe Two
          - runs a test

But at the moment I'm only seeing the below.
- foo.test.js
   - Foo Test
       -  Describe One
          - runs some test

As mentioned I did some digging into this to try see what the issue could be and found that the tests are being run by neotest as when I checked the output files in the var directory I could see the results of the tests from the imported function. These tests are also correctly found by the jest CLI which is why I imagine they run correctly through neotest so it appears the issue is just in the project summary / discovery as well as in the pass/fail/in-progress symbols in the gutter.
It's worth noting that these tests are correctly identified and ran with status symbols in VSCode with the Jest extension so I believe this issue is related to neotest.
Someone more experienced with Neovim/treesitter might have a better idea of how to fix this but my original thinking was to somehow build the tests file out into a single file that contains all of the imported code and then run treesitter against that to correctly identify the imported code but I imagine this could get quite time consuming / costly when run at scale which isn't ideal.

Adding beforeAll(async() => {}) allows all tests to show up in my case. It seems Tressitter may be looking for certain tokens.


Added lat long mouse coordinates to map.

It's ok as it is, but I think that it would be better to limit the precision and display the number better so they don't jump around as you move the mouse.  Not to sure what would be the best way to do that?

Ready for review


Removing zap update daemon

When I ran zap init for the first time, I chose to enable automatic updates. Since then, I've decided that I wanted to disable it, however, I'm running into trouble doing so. I've tried running init again and selecting not to auto-update, and I've removed/masked systemctl services that seemedto be associated with zap. However, I still receive regular notifications from zap indicating that each appimage I have installed with zap have been updated.
For reference, I'm running zap on PopOS 22.04 and installed it via the install script. (curl https://raw.githubusercontent.com/srevinsaju/zap/main/install.sh | sudo bash -s)
Any help would be much appreciated.

I think I figured it out. The service file for zapd was located at ~/.config/systemd/user/zapd.service rather than with my other service files at /etc/systemd/. I just deleted zapd.service and I believe the issue is resolved.


port looper to petal

Looks like the README has the two switches reversed (switch 1 controls record/clear and switch 2 controls play/pause)
Again, you'll want to change the LED code to use the DaisyPetal::SetFootswitchLed function that utilizes the led driver.
Other than that, this one's pretty solid.


Segmentation fault in invoking list(zip(,))

The following code triggers a segmentation fault when invoking list(zip(c, c)).
test.py
a = ['1', '2', '3']
b = [1, 2, 3]
c = zip(b, b)
print(list(c))
print(list(zip(c, c)))

Reproduce:
'codon/codon-linux-x86_64/codon-deploy/bin/codon' run -release test.py
Crash message:
[(1, 1), (2, 2), (3, 3)]
Segmentation fault (core dumped)

Behavior on CPython 3.10.8:
[(1, 1), (2, 2), (3, 3)]
[]

Environment:
codon: v0.15.5 on Feb 6
Ubuntu 18.04

Do you know why your second print give a result of [] ? not [((1,1),(1,1)),((2,2),(2,2)),((3,3),(3,3))]

In Python 3, zip() returns an iterator, and we can only use iterators once. After we have used the iterator, it is exhausted and there is nothing left. So the second print gives a result of '[]'.  If you want to print [((1,1),(1,1)),((2,2),(2,2)),((3,3),(3,3))], try "list(zip(zip(b,b), zip(b,b)))".

In Python 3, zip() returns an iterator, and we can only use iterators once. After we have used the iterator, it is exhausted and there is nothing left. So the second print gives a result of '[]'. If you want to print [((1,1),(1,1)),((2,2),(2,2)),((3,3),(3,3))], try "list(zip(zip(b,b), zip(b,b)))".

So I think that's the reason why you got Segmentation fault,you want to store the empty thing to an array.

@xiaxinmeng It works as expected in Codon and in Python.
b = [1, 2, 3]
c = list(zip(b, b))
print(c)

a = list(zip(c, c))
print(len(a))

print(list(zip(c, c)))

prints:
[(1, 1), (2, 2), (3, 3)]
3
[((1, 1), (1, 1)), ((2, 2), (2, 2)), ((3, 3), (3, 3))]

Here is the bug:
b = [1, 2, 3]
c = zip(b, b)
print(list(c))  # list(c) exhausts the "c" generator: [(1, 1), (2, 2), (3, 3)]
print(list(zip(c, c)))  # bug: trying to zip() an exhausted generator!

In Python raises "StopIteration" exception, and in Codon program exists with SIGSEGV: Linux Segmentation Fault | Signal 11. Here is the MRE:
b = [1, 2, 3]
c = zip(b, b)

print(next(c))  # (1, 1)
print(list(c))  # [(2, 2), (3, 3)]
print(next(c))  # Segmentation fault (core dumped)

zip does not like duplicated iterators. Thank you for the report—will check this soon!


Chat messages not sorting properly

It looks like this code is using the same sorting as the firebase web interface which isn't always consistent.
-J4... < -J-...

-J-... < -J0...

Do we need to use - observeEventType:andPreviousSiblingNameWithBlock: to keep the messages in order even if we're not using priority?
In other words, is this a bug in the firechat-ios client or the firechat backend?
Thanks!

Closing since there hasn't been any activity on here in ages, but @klaw23 don't hesitate to send us a PR if you're interested in incorporating your changes into this lib!


msal browserFlow error when using instrumental & middleware in Next.js

Package Name: @azure/identity
Package Version: 4.0.1
Operating system: Windows
[x] nodejs

version: 18.0.18

[ ] browser

name/version:

[x] typescript

version: 5.1.6

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
When using the Next.JS instrumentation feature for getting KeyVault values and using Next.JS' middleware I get the following error:
error -<EMAIL_ADDRESS>(9:0) @ <unknown>
error - Cannot read properties of undefined (reading 'hash')

It appears to be trying to get the URL hash from window, but failing.
This doesn't happen when I'm not using middleware.ts which makes it even more confusing.
To Reproduce
Steps to reproduce the behavior:

Create a Next.JS application with middleware.ts and instrumentation.ts files
Create a SecretClient in instrumentation.ts using DefaultAzureCredential
Get secrets via the client.
Run app

Expected behavior
Secret is returned from the client.
Additional context
I'm using Next.JS version 13.3.3 but it also appears to be happening on newer versions.

👋 Hi @bjamdort - thanks for reaching out about this. I was able to repro this but with slightly different results.
What I did:

ran npx create-next-app@latest
Added a file called middleware.ts to the root level
Used the following bit of code:

export async function middleware(request: NextRequest) {
  const secretClient = new SecretClient(
    "<my vault url>",
    new DefaultAzureCredential({})
  );
  await secretClient.getSecret("my-secret");
  return NextResponse.next();
}

What I got was an error:  DefaultAzureCredential is not supported in the browser. Use InteractiveBrowserCredential instead.
Checkpoint: Did you get that as well?
Then, I switched to using InteractiveBrowserCredential and got the error you described
 ⨯<EMAIL_ADDRESS>(9:0) @ <unknown>
 ⨯ Cannot read properties of undefined (reading 'hash')

I'll keep investigating but I'm not too familiar with next.js so I'll need to learn a bit about where middleware is actually run (server or browser). In the meanwhile, could you confirm that we're looking at the same thing and that you are using InteractiveBrowserCredetial? If you are not, where is your middleware.ts file located?
Under app? src? root level?
Thanks in advance!

Update: looking closely at the issue again, it sounds like instrumentation.ts is where you are trying to hook into Azure and the presence of middleware causes that to stop working is that right?
Looking at Next.JS docs it does look like middleware can only be run on the Edge Runtime. We support NodeJS and browsers today, and while we make a best-effort attempt at supporting other runtimes we cannot guarantee compatibility.
The Edge Runtime does not support all Node APIs, so they recommend using the Node.js runtime if you need to use packages that depend on those APIs. See documentation. This may also explain why adding middleware causes issues to pop up.
Could you try using the Node runtime instead? I'm not sure how the presence of middleware.ts changes nextJS's compilation and environment so I would recommend avoiding that path if possible

It seems like instrumentation.ts and middleware.ts are connected and Next.js will choose the Edge runtime for both when you are using middleware.ts so I can't choose the Node runtime while using middleware.ts.
For our case we won't be using middleware.ts because we need the Node runtime.
Thanks for the quick reaction, it helped us figure out why we couldn't get it to work.
I'll close this issue because I don't think it's an issue with Azure Identity.


Tests folder included in fsutil release

Python version
Python 3.9
Package version
Since release 0.9
Current behavior (bug description)
In my project, we are using python-codicefiscale which has for dependency python-fsutil. When launching our Django app, we load our settings located in tests.[...].settings.
Python is taking the fsutil tests folder, located in my virtual environment, as the folder under which my settings are located.
The bug was introduced in this commit. The testsfolder is not excluded as the wheel is built.
Expected behavior
Update the setup.cfg file to exclude the tests folder.

@sandre35 thank you for reporting this and for already having investigated it.

@fabiocaccamo I tracked down the issue, a PR will appear shortly.

@sandre35 there was an error in setup.cfg here, but even fixing it the release contains the tests/ folder.

@RemiCardona the dist files are not created locally, but I create the release using with this workflow, I don't think there is persistence.

I downloaded the wheel from pypi.org and the tests directory is gone. It all looks fixed to me. Are you looking elsewhere?

I looked the assets in the release on GitHub, tests are still present, but if they are no more on PyPi it's ok.

Oh that zipfile is created by github on the fly, it's just an archive of the files in the git repo under the tag. It's fine to ignore that!
Thanks for the quick release, it's very much appreciated :pray:


Remove unused outstanding fees fields

Closes: #363, closes #361
Notes:

Includes cherry-picked commit by Bernd to fix the tests by hard-coding (temporally fix).
This requires/reacts to the most recent changes in the trustlines-protocol/contracts changes. Unfortunately does the latest trustlines-contracts-bin library version not include them now. There is also no such of a dev version like for NodeJS (relevant for trustlines-protocol/clientlib). To check the changes, an up to date contracts.json must be injected into the virtual environment at ./venv/trustlines-contracts/build.

Status:
This PR can't be merged at the moment, because it is based on the most recent contracts which are not supported yet by the trustlines-contracts-bin library.
Anyways @cducrest asked me to tidy up the history already.

The approval on the actual changes has been given. The waited for update of the dependencies has been done and the tests are fixed. Thereby I will rebase this on master with the new src based directory structure.


Add GLPK support

Hello,
Here is an attempt to close #7 . Comments welcome. :)
Thomas

Thank you so much. It works fine!


Check markdown links

Closes https://github.com/alphagov/govuk-frontend/issues/755
Adds markdown internal link checking using the remark library, and a validator plugin.
I have fixed a bunch of errors after introduce this.
The errors it produces are really useful, giving you the path that it is actually resolving to:

Will try and redo this with jest maybe, no idea why this is failing to be honest...

Trying a rebuild… can't see how those failures would relate to this change.

This might be better done as a probot https://probot.github.io/apps/

So I've done a lot of thinking and poking the code.
The programmatic interface to the remark library doesnt check links between files which is one of the biggest selling points...

Will close this for now, and come back to it when I have some time.


Fixed Javadoc for HystrixThreadPoolProperties.Setter

NetflixOSS » Hystrix » Hystrix-pull-requests #95 FAILURE
Looks like there's a problem with this pull request


[WIP] Make the translog durability configuration

Codecov Report

Merging #5631 into master will decrease coverage by 0.11%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #5631      +/-   ##
============================================
- Coverage        83%   82.89%   -0.12%
+ Complexity    11834    11804      -30
============================================
  Files          1391     1391
  Lines         45508    45518      +10
  Branches       5627     5627
============================================
- Hits          37776    37734      -42
- Misses         5123     5161      +38
- Partials       2609     2623      +14

Impacted Files
Coverage Δ
Complexity Δ

...java/io/crate/metadata/settings/CrateSettings.java
93.49% <ø> (ø)
38 <0> (ø)
:arrow_down:

...java/io/crate/analyze/TablePropertiesAnalyzer.java
91.97% <100%> (+0.08%)
21 <0> (ø)
:arrow_down:

...main/java/io/crate/analyze/TableParameterInfo.java
100% <100%> (ø)
6 <0> (ø)
:arrow_down:

...io/crate/metadata/settings/CrateTableSettings.java
95.65% <100%> (+0.65%)
1 <0> (ø)
:arrow_down:

...eference/information/TablesSettingsExpression.java
100% <100%> (ø)
2 <0> (ø)
:arrow_down:

...ence/partitioned/PartitionsSettingsExpression.java
100% <100%> (ø)
2 <0> (ø)
:arrow_down:

...exceptions/GroupByOnArrayUnsupportedException.java
66.66% <0%> (-33.34%)
1% <0%> (-1%)

...arch/indices/recovery/BlobStartPrefixResponse.java
66.66% <0%> (-33.34%)
3% <0%> (-2%)

...in/java/io/crate/lucene/match/CrateRegexQuery.java
43.47% <0%> (-30.44%)
4% <0%> (-4%)

...rc/main/java/io/crate/analyze/UnboundAnalyzer.java
65% <0%> (-20%)
2% <0%> (ø)

... and 25 more

Continue to review full report at Codecov.

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


Makefile: Fix copy paste error in hypervisor debug

Fix copy paste error introduced during hypervisor debug configuration.
Fixes #696
Signed-off-by: Manohar Castelino<EMAIL_ADDRESS>

lgtm


Added second session to webpage

Uncommented the second session. So that it will appear on the website.

Squash and merge preferably. Fixed an issue with Safari table rendering, thanks @Sanvit-Katrekar for the help

Preview Here

Done :thumbsup:


500 Internal Server Error

Hi all,
I just received the following message: Failed to pull image "quay.io/external_storage/nfs-client-provisioner:latest": rpc error: code = Unknown desc = Error response from daemon: Get https://quay.io/v2/external_storage/nfs-client-provisioner/manifests/latest: received unexpected HTTP status: 500 Internal Server Error.
Does this mean the internal server is down?

It is solved! Thanks!


Is workbook protected?

How can I check whether the excel workbook is protected or not? Can you please give me the method that return boolean for this check?

There is no API for that.


kad: Strategy for removing old MemoryStore records

When the memory store is full and a new record is added, remove expired records.
Extracted from: https://github.com/paritytech/litep2p/pull/78#discussion_r1570937475

+1 for keeping the records sorted and not traversing all the store.


How to use data  form a database?

How to use data from a database instead of reading it from a file like here:
 vega.loader().load('../../sample-data/demovote.tsv').then(function (text) {
        var data = vega.read(text, { type: 'tsv' });
        scatterplotTest.viewer.render(options, data);
    });

Hi @win32nipuh, we use Vega for layout, therefore we support the formats that Vega does: https://vega.github.io/vega/docs/data/#format

The currently supported data formats are json (the default), csv (comma-separated values), tsv (tab-separated values), dsv (delimited text files), and topojson.

Thnx @danmarshall, I see. The question can be redirected to Vega. :-)

By the way Vega developers wrote: a data can be loaded via Ajax request.


Update scalafmt-core to 2.2.0

Updates org.scalameta:scalafmt-core from 2.1.1 to 2.2.0.
Release Notes/Changelog
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: semver-minor

Closed in favor of #268.


add metadatum label 674 - transaction messages/comments

I'd suggest grouping this change with #100 since they are strictly related :+1:

I'd suggest grouping this change with #100 since they are strictly related 👍

thx yes, the change for the registry.json is now directly in #100
so i will close this PR.


Make TimeParsed and RubyTimeParsed package-private

They don't need to be public...

Thanks! Merging.


Supporting proxying to multiple Cloud SQL instances

Question
I'm working on vertically partitioning a Cloud SQL instance to multiple Cloud SQL instances. I'm currently using cloud-sql-proxy to connect to the first instance and wondering if it supports proxying to multiple instances. If not, I was planning on using the Cloud SQL Python connector to connect instead.
Does cloud-sql-proxy support connecting to multiple Cloud SQL instances? If so, what are the pros/cons of using the cloud-sql-proxy over the Python connector?
Thanks in advance!
Code
No response
Additional Details
No response

Hi @digitalnomd, thanks for the great question!
The Cloud SQL Proxy does support connecting to multiple instances! You can do so through the following invocation:
./cloud-sql-proxy <INSTANCE_CONNECTION_NAME_1> <INSTANCE_CONNECTION_NAME_2>

You can read more detailed information about configuring connection to multiple instances in our Basic Usage section of the README.
In terms of pros/cons between the Proxy and a Connector library (Java, Node, Go, Python), it really comes down to preference as both perform the same job and have similar capabilities.
The Proxy is language agnostic since it is a binary vs a Connector which is tied to a specific programming language. So if you are running something locally and prefer using a CLI like psql than maybe the Proxy is best for you. However, if you are comfortable with Python and would rather write code over use a CLI than maybe go with the Python Connector.
Another factor to consider is if you are looking to deploy your application, if you are looking to deploy your code as a Python app say to Cloud Run, Cloud Functions etc than the Python Connector would be easiest as a native integration.
Hope this helps, have a great day!
-- Jack

Closing this as answered, but feel free to re-open if the answer did not clear things up for you.


Max distance (km) for mapping

Hi! I have an issue while recording a larger map using lidarslam.yaml. The desired route is about 6.2km long. So, the mapping stopped somewhere while I was recording it https://drive.google.com/file/d/17_C7tcCxtObeyDzwouvG9QjKfip85lPi/view?usp=sharing
I believe there is a limit in the yaml file right? If it is so, is it can be solved if I use the lidarslam_tukuba.yaml?

As long as this package can store the pcd data in memory, it can be mapped with scan matching.
I have only tried loop close with 2-3km of data.
It seems that NDT scan munging is failing. I think lidarslam_tukuba.yaml also fails because the NDT parameters are the same as lidarslam.yam.
Suggested solutions are as follows
Tune NDT parameters (ndt_resolution between 1~5, vg_size_for_input, vg_size_for_map smaller)
Move the robot in a way that it does not rotate as rapidly as possible and does not vibrate too much during the measurement.
Add more odometry and IMU
Provide rosbag so I can pursue the cause in more detail.

I'm sorry I can't provide you the rosbag at the moment. But, those suggestions are make sense to me. Let say I want to add IMU, so I have to use li_slam instead of lidarslam right?

I have to use li_slam instead of lidarslam right?

Yes.

I'm sorry I can't provide you the rosbag at the moment.

Sad. If you can, give me a video of the rviz of the scan at the time and I might be able to figure out the problem.

I will give you the rosbag soon but not at the moment. I still want to contribute to your work as well as I want the best results for my work.

Hi there,
I have acquired the rosbag for my problem.
https://drive.google.com/file/d/1Vakv6shABeRXbnqPA83424o1bLOZ4fZ2/view?usp=sharing
Tools:

Lidar: Velodyne VLP-16
GNSS/IMU: XSens MTI-G-710

Issues:

Lidarslam crashed in the mid-way.
The elevation of the pointcloud is not accurate. Is the data obtains the x, y, z information?

In my environment, the scan matching failed occasionally and the loop did not close, but I was able to complete the scan matching.
It will take some time, but I will look into a few things to close the loop.
The reason lidarslam crashed on your PC could possibly be that your PC's CPU memory is small and could not hold the point cloud.
Or maybe the low GPU memory made the rviz/PC run slower, which caused the scan matching process to fail. If you are not visualizing point clouds, GPU memory may not be much of an issue.
For reference, my PC has 32GB of CPU memory and 8GB of GPU memory.

Here are some more details about the issue.

It seems to be the same as my environment. Scan matching is failing in places.

I'm sorry. I have been busy lately and it will take some time to get lidarslam_ros2 to close this loop.
If you are in a hurry, please use li_slam_ros2.

That feature was inspired by lego-loam, but its performance isn't optimal. If you want to use IMU, it's better to use li_slam.


Terminal error row[k] is null in recordedit

To reproduce, go to:
https://synapse-dev.isrd.isi.edu/chaise/recordedit/#1/Zebrafish:Synaptic Pair Study/RID=19WJ
Attempt to set the only value available in the "Synaptic Region 1" modal picker and submit. I get a terminal error with the details saying only:
row[k] is null

In the supplied use case it is iterating to the Region 1 URL (which has a value of null supplied) and it’s choking out our conditional check that “assumes” row[k] (the value of Region 1 URL) is defined.
offending line of code:
https://github.com/informatics-isi-edu/chaise/blob/master/common/upload.js#L635

The configuration that caused this involved a combination of nullok: true on a disabled input field that was part of the visible columns list. There are no problems with that configuration, just a small change to the check to make sure it doesn't short circuit again.

@karlcz this has been fixed with this commit to master. Reassigning to you to verify it's fixed and to close this issue.

Fix confirmed.


Links for “County Managed” pages need updating

For the “County Managed” links for Tie Sheet Points we need to update at least Salt Lake and Utah
Salt Lake Co.
https://slco.org/surveyor/apps/surveymonument/map.html
Utah Co.
https://maps.utahcounty.gov/TieSheets/TieSheet.htm
Weber and Davis seem ok. Do you know of others? I thought Carbon but those are ours now.

Isn't this stored in the data?

https://github.com/agrc/PLSS/blob/master/PLSS/src/app/data/existingPlssWebsitesByCounty.js so this needs to be updated?

Checked that database couldn't find anything............
That is what we needed....  let me check those others and see if any more are bad

San Juan remove them, it will be ours
Uintah - http://www.co.uintah.ut.us/surveyor/UCTieSheet.htm
Wasatch - http://wasatch.maps.arcgis.com/apps/webappviewer/index.html?id=103db0251a5342f7bbd1462eb7a47440
these Salt lake and Utah I sent, the others are ok

define({
    'UTAH': 'https://maps.utahcounty.gov/TieSheets/TieSheet.htm',
    'WASATCH': 'https://wasatch.maps.arcgis.com/apps/webappviewer/index.html?id=103db0251a5342f7bbd1462eb7a47440',
    'DAVIS': 'http://www.co.davis.ut.us/surveyor/default.cfm',
    'SALT LAKE': 'https://slco.org/surveyor/apps/surveymonument/map.html',
    'WEBER': 'https://www.co.weber.ut.us/gis/maps/gizmo/',
    'DUCHESNE': 'https://duchesne.utah.gov/surveyor/base&meridian.html',
    'UINTAH': 'https://www.co.uintah.ut.us/surveyor/UCTieSheet.htm',
});

Double check this one more time before I update it please.

That looks good everything checks.

@mheagin it looks like tiesheet_name is null in a lot of the plsspoint_agrc feature class. If I remember correctly, if there was no tiesheet and the tiesheet was managed by one of the 7 counties above we put the county name in that field to tell the app to use the website.
I think a lot of points need that field be updated to make the website feature work properly.
Thoughts?

Well..... Salt Lake and Utah Co are not hitting the url from the table above so they are not working.
'UTAH': 'https://maps.utahcounty.gov/TieSheets/TieSheet.htm',
'SALT LAKE': 'https://slco.org/surveyor/apps/surveymonument/map.html',
And the points are acting weird, in those 7 counties, I am not seeing any of the orange ones that don't have tie sheets, everything goes purple. So I thought they were symbolized by the attribute DISPLAY_GRP, or are they influenced by the TieSheet_Name field?

Did you spell salt lake and utah all caps and two words? They need to match exactly.
I'll have to check the mxd for the symbology. You could also ask Scott. im pretty sure you made it and might have a copy also???

Double checked the county names, they are correct.
The symbology is set to TieSheet_Name, so that is where the issue is.
I'll talk to Sean and see what he want to do.

wait ....so I cleared my cache and now the links work.
not sure why the new ones did work but the old ones didn't or did you change something?
Sean said to leave the point as they were.
So I think we are good now.


Non-fatal errors from deserialization of PSRepositories.xml

Steps to reproduce
The exact conditions which cause the problem to be exhibited are unknown.
PS> Get-InstalledModule PowerShellGet

Many other PowerShellGet cmdlets also exhibit the problem, but Get-InstalledModule is the simplest to demonstrate.
Expected behavior
No exceptions or errors.
Actual behavior
Exception calling "Deserialize" with "1" argument(s): "Current Node type is Text. Expected type is Element. Line 1,
position 81."
At C:\Users\alex.martin\OneDrive - Imagine
One\Documents\WindowsPowerShell\Modules\PowerShellGet\2.2\PSModule.psm1:1132 char:5
+     [System.Management.Automation.PSSerializer]::Deserialize($filecon ...
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : XmlException

The property 'Keys' cannot be found on this object. Verify that the property exists.
At C:\Users\alex.martin\OneDrive - Imagine
One\Documents\WindowsPowerShell\Modules\PowerShellGet\2.2\PSModule.psm1:6460 char:9
+         $script:PSGetModuleSources.Keys | Microsoft.PowerShell.Core\F ...
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], PropertyNotFoundException
    + FullyQualifiedErrorId : PropertyNotFoundStrict

You cannot call a method on a null-valued expression.
At C:\Users\alex.martin\OneDrive - Imagine
One\Documents\WindowsPowerShell\Modules\PowerShellGet\2.2\PSModule.psm1:2749 char:8
+     if($script:PSGetModuleSources.Contains($SourceName))
+        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : InvokeMethodOnNull

Environment data

> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      5.1.18362.145
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0, 5.0, 5.1.18362.145}
BuildVersion                   10.0.18362.145
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

> Get-Module -ListAvailable PowerShellGet,PackageManagement
    Directory: C:\Users\<me>\Documents\WindowsPowerShell\Modules

ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Script     1.4.3      PackageManagement                   {Find-Package, Get-Package, Get-PackageProvider, Get-Packa...
Script     2.2        PowerShellGet                       {Find-Command, Find-DSCResource, Find-Module, Find-RoleCap...

    Directory: C:\Program Files\WindowsPowerShell\Modules

ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Script     1.4.2      PackageManagement                   {Find-Package, Get-Package, Get-PackageProvider, Get-Packa...
Binary     <IP_ADDRESS>    PackageManagement                   {Find-Package, Get-Package, Get-PackageProvider, Get-Packa...
Script     <IP_ADDRESS>    PowerShellGet                       {Install-Module, Find-Module, Save-Module, Update-Module, ...

> Get-PackageProvider -ListAvailable
Name                     Version          DynamicOptions
----                     -------          --------------
msi                      <IP_ADDRESS>          AdditionalArguments
msu                      <IP_ADDRESS>
nuget                    <IP_ADDRESS>
NuGet                    <IP_ADDRESS>          Destination, ExcludeVersion, Scope, SkipDependencies, Headers, FilterOnTag...
PowerShellGet            <IP_ADDRESS>          PackageManagementProvider, Type, Scope, AllowClobber, SkipPublisherCheck, ...
PowerShellGet            <IP_ADDRESS>
Programs                 <IP_ADDRESS>          IncludeWindowsInstaller, IncludeSystemComponent

Resolution attempts
Debugging PowerShellGet using VS Code showed that the error stemmed from attempting to deserialize PSRepositories.xml using [System.Management.Automation.PSSerializer]::Deserialize(), in the function DeSerialize-PSObject. PSRepositories.xml appeared valid to me. I double-checked by deleting it and trying again; the copy that PowerShellGet recreated also exhibited the problem after a PowerShell restart. Replacing DeSerialize-PSObject with an alias to Import-CliXml resolved the problem.
Recommendations
If possible, it might help to detect if Import-CliXml is available and use that instead.

@alexbuzzbee thanks for opening this issue and for taking the time to de-bug the issue yourself...one challenge we are having is re-producing this behavior...have you tried uninstalling and re-installing PowerShellGet...also wondering if you are able to successfully use any PowerShellGet commands and if you hit any errors using PackageManagement i.e. is Get-Package PowerShellGet successful? Also when you hit errors using other PowerShellGet cmdlets is it always the same error message?

I've made a number of attempts to upgrade PowerShellGet and PackageManagement to a newer version, and none of them have fixed the problem. I can try Get-Package PowerShellGet tomorrow when I'm at my normal machine.

@alexbuzzbee has this been happening with different versions of PowerShellGet?  Do you have the System.Management.Automation .dll on your machine?

I should probably note that I have PowerShell 6.2 and 7-preview.2 installed alongside 5.1. This problem does not happen in those versions.
Get-Package PowerShellGet works fine. Cmdlets that do not trigger the problem include Get-PSRepository and Save-Module. The same error messages occur with all problematic commands; additionally, with a Get-InstalledModule of all modules, they occur once for every output object.
I don't know how I can safely fully uninstall PowerShellGet without losing the ability to reinstall it, but I have tried upgrades and per-scope (CurrentUser) reinstalls with no effect.
System.Management.Automation.dll exists in the GAC, in WinSxS, in my copy of the PowerShell Reference Assemblies, and in the install directories for PowerShell 6 and 7. The loaded copy is at C:\WINDOWS\Microsoft.Net\assembly\GAC_MSIL\System.Management.Automation\v4.0_<IP_ADDRESS>__31bf3856ad364e35\System.Management.Automation.dll.
This has been happening for a while now, through several upgrades of PowerShellGet, though I can't tell you for sure what the previous versions were. I think I saw it in 2.1.4 and 2.1.3. The current version I'm using is 2.2 installed in the CurrentUser scope from the PSGallery.

I have this same issue with the 2.2.1 release. Can I provide you with any additional information that can help you address this?

@alexvergilis Compare your system to the information I've put here about mine and list what's the same. Maybe we can narrow down the trigger conditions.

> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.17763.771
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.17763.771
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

> Get-Module -ListAvailable PowerShellGet,PackageManagement

    Directory: C:\Program Files\WindowsPowerShell\Modules

ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Script     1.4.5      PackageManagement                   {Find-Package, Get-Package, Get-PackageProvider, Get-PackageSource...}
Script     1.4.4      PackageManagement                   {Find-Package, Get-Package, Get-PackageProvider, Get-PackageSource...}
Script     1.4.3      PackageManagement                   {Find-Package, Get-Package, Get-PackageProvider, Get-PackageSource...}
Script     1.4.2      PackageManagement                   {Find-Package, Get-Package, Get-PackageProvider, Get-PackageSource...}
Script     1.4.1      PackageManagement                   {Find-Package, Get-Package, Get-PackageProvider, Get-PackageSource...}
Binary     <IP_ADDRESS>    PackageManagement                   {Find-Package, Get-Package, Get-PackageProvider, Get-PackageSource...}
Script     2.2.1      PowerShellGet                       {Find-Command, Find-DSCResource, Find-Module, Find-RoleCapability...}
Script     2.2        PowerShellGet                       {Find-Command, Find-DSCResource, Find-Module, Find-RoleCapability...}
Script     2.1.5      PowerShellGet                       {Find-Command, Find-DSCResource, Find-Module, Find-RoleCapability...}
Script     2.1.4      PowerShellGet                       {Find-Command, Find-DSCResource, Find-Module, Find-RoleCapability...}
Script     <IP_ADDRESS>    PowerShellGet                       {Install-Module, Find-Module, Save-Module, Update-Module...}

> Get-PackageProvider -ListAvailable

Name                     Version          DynamicOptions
----                     -------          --------------
msi                      <IP_ADDRESS>          AdditionalArguments
msu                      <IP_ADDRESS>
nuget                    <IP_ADDRESS>
NuGet                    <IP_ADDRESS>          Destination, ExcludeVersion, Scope, SkipDependencies, Headers, FilterOnTag, Contains, AllowPrereleaseVersions, ConfigFile, SkipValidate
PowerShellGet            <IP_ADDRESS>          PackageManagementProvider, Type, Scope, AllowClobber, SkipPublisherCheck, InstallUpdate, NoPathUpdate, AllowPrereleaseVersions, Filter, Tag, Includes, DscResource, R...
PowerShellGet            <IP_ADDRESS>
PowerShellGet            <IP_ADDRESS>
PowerShellGet            <IP_ADDRESS>
PowerShellGet            <IP_ADDRESS>
Programs                 <IP_ADDRESS>          IncludeWindowsInstaller, IncludeSystemComponent

@alexvergilis @alexbuzzbee could you both let me know which dll's you have loaded when you're encountering this issue ([System.AppDomain]::CurrentDomain.GetAssemblies())

Several previously-unloaded assemblies were loaded when I ran Get-InstalledModule PowerShellGet. I decided that the output from the default table format was not sufficient, so I piped the data to Format-List, which produced too much text to be reasonably included here. As such, I have put the output on Pastebin. If any filtering or further information is desired, I still have the ability to reproduce at will.

My problem actually went away after the following:
sfc /scannow
System restart
Uninstall-Module -Name ModuleNameThatProducedTheErrorBefore -AllVersions -Force
Install-Module -Name ModuleNameThatProducedTheErrorBefore

Interesting. I've run sfc /scannow without seeing any improvement. I'll try the whole set, but I'm not confident that it'll work.

Hi! After a couple years of dealing with this frustrating issue, I've finally found a way to reliably repro this XmlException, PropertyNotFoundStrict, and InvokeMethodOnNull error combo.
The issue (at least in my case) was that--deep within my lengthy profile.ps1, I'd set the preference variable $OFS to ","; the issue seemingly occurs if the value for $OFS includes any non-whitespace characters.
The following code block easy allows for reproducing the error (tested environments are listed at the bottom).
# (Optional) Remove all modules for clean testing.
Remove-Module * -Force

# For easy console differentiation, we'll use verbose output for our $OFS value.
Write-Verbose -Verbose "`$OFS is `'$OFS`'."

# Search using Find-Module, thus importing PowerShellGet (PSG); this search currently returns 3 items, and will not fail since the default value of $OFS is currently set.
Find-Module Github.G*

# Change the value of $OFS to ', '
$OFS = ', '
Write-Verbose -Verbose "`$OFS is `'$OFS`'."

# Search again using Find-Module; this command also will not fail. Though the value of $OFS has been changed, the exception doesn't occur because PSG is already imported.
Find-Module Github.G*

# Remove the PSG module.
Remove-Module PowerShellGet
Write-Verbose -Verbose "`$OFS is `'$OFS`'."

# Search again using Find-Module, thus (re-)importing PSG. The XmlException, PropertyNotFoundStrict, and InvokeMethodOnNull errors now occur.
Find-Module Github.G*

# To clear the error state, remove the PSG module, change the value of $OFS to empty or whitespace, then search again.
Remove-Module PowerShellGet, PackageManagement; $OFS = " `n`r`t"; Write-Verbose -Verbose "`$OFS is `'$OFS`'."; Find-Module Github.G*

# Personally, I switch error view to "category" for easy viewing on subsequent tests if I already know what the errors are going to be:
$ErrorView = 'CategoryView'

# To test other characters/strings, simply run the above line again, but change the value of $OFS to other characters/strings, e.g.:
Remove-Module PowerShellGet; $OFS = " # ";Write-Verbose -Verbose "`$OFS is `'$OFS`'."; Find-Module Github.G*

In my tests, failing values included the following characters, both with and without preceding/trailing whitespace: ,;."'][
The space character, 0x20, alone did not fail; other characters that didn't fail, even when used together: `t`r`n
I've successfully repro-ed the issue in the following environments:
• Up-to-date Windows 10 and Windows 11 production environments, using Windows PS 5.1 (including the ISE) and PS 7.2.6 (side-by-side) with PowerShellGet 2.2.5 and PackageManagement <IP_ADDRESS>.
• Ubuntu 22.04 using WSL, PowerShell 7.2.6, and PowerShellGet versions 2.2.5 and 3.0.17 (pre)
• Windows PowerShell 5.1 in Windows (11) Sandbox (install the NuGet package provider prior to running the above code). Interestingly, in the Windows 11 sandbox, the only version of PSG was the default module/script, version 1.0.01.
For now, my workaround is simply leaving the $OFS variable at its default value. If there's anything else I can provide that'll help with this, please let me know.

@Nevember thanks for the detailed repro information!  Were you able to try the repro with 3.0.17-beta17 in Windows 10 and Windows 11?

@alerickson, my pleasure!
I have to apologize for one detail, though: The issue does not occur with 3.0.17-beta17 in Ubuntu. I had forgotten that the command Find-Module has been replaced with Find-PSResource in newer versions of PowerShellGet, so when tested in Ubuntu yesterday, v2.2.5 was imported, not beta17.
I discovered this when testing in Win10 and Win11 (and Server2019) a few minutes ago. When using Find-PSResource from version 3.0.17-beta17, the error didn't occur in any test environment. I tried using comma, semicolon, and pound symbols.
Since the other reporters mentioned Get-InstalledModule, I found that it, too, suffers from the same $OFS issue with PowerShellGet v<IP_ADDRESS> and v2.2.5; however, I don't see an equivalent command in v3.0.17-beta17, unless it's simply Get-Module -ListAvailable from Microsoft.PowerShell.Core, which doesn't seem to suffer from this issue.
By using Set-Alias Find-Module PowerShellGet\Find-PSResource -Force and making sure the only PSG version being imported was 3.0.17-beta17, I was able to successfully run Find-Module Github.G* with $OFS set to ', ' without any issues. 💯

@Nevember thank you for this feedback, this is great to hear that this is not reproducing in v3.0.17-beta17, yay! The equivalent command for Get-InstalledModule is Get-PSResource in PowerShellGet V3. If the $OFS issue occurs with the Get-PSResource cmdlet please let us know, thank you!
Closing this issue as we're not working on PowerShellGetV2. If you're still facing it while using PowerShellGetV3 please feel free to comment and we'll re-open.

This issue is not fixed by PowerShellGet v3 until it is generally available. While PowerShellGet v3 is still in preview, PowerShellGet v2 remains the production version.


PHP: Accept \wsl$<distro>\path\to\php as php.validate.executablePath

This request looks like #22391 : Is it possible to use wsl bash php for php.validate.executablePath and the suggested workaround using a *.bat file works indeed.

Since Windows 10 build 1903 now supports accessing files in WSL using \\wsl$\<distro>path\to\file, it would be nice to use this reference for php.validate.executablePath

I think that would be a linux binary that we can't run in a windows context.
See some details about developing in WSL here: https://code.visualstudio.com/docs/remote/wsl


could not init cloud provider "vsphere": unknown escape sequence

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Begin creating a vSphere new cluster
Edit YAML to include configuration for vSphere provider that uses a backslash in the username:

cloud_provider:
  name: vsphere
  vsphereCloudProvider:
    global:
      insecure-flag: true
    virtual_center:
      vcenter.tld.com:
        user: "DOMAIN\\user"
        password: "password"
        datacenters: "DC1"

Create the cluster

Result:

...failed to run Kubelet: could not init cloud provider "vsphere": 10:21 unknown escape sequence
Other details that may be helpful:

I did try several values for the username, such as "DOMAIN\user". "DOMAIN\\user", "DOMAIN\\\\user", "DOMAIN/user".
We were able to use<EMAIL_ADDRESS>rather than DOMAIN\user to get past this error.

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.1.0
RancherOS version: v1.4.2
Installation option (single install/HA): HA

Why is your username DOMAIN\\user, which authentication do you use?

I will reproduce it when I am ready for the VCenter environment.

Sorry for the delayed response. I reached out to my IT department to verify that the authentication method is just active directory.

Hello I have the same issue. Im using the same credentials for cluster provisioning and storage provisioning on vcenter.
Does anyone can help on this please ?

@jalbou it has been a minute since I thought about this problem, and I have since moved on to a different job. If I remember correctly, though, I think we were able to resolve the problem on our end by removing a dash from the username? Are you attempting to use a username that includes a dash, by chance?

@jalbou it has been a minute since I thought about this problem, and I have since moved on to a different job. If I remember correctly, though, I think we were able to resolve the problem on our end by removing a dash from the username. Are you attempting to use a username that includes a dash, by chance?

@codekoala Yes they was one, I changed for another one no longer the message but still not able to create a persistant disk on my pods


fix-52: added 404.html

use most things from old page. Fix #52

Can you revert the change to h1 font size and weight and instead add "opacity: 0.8;"? This would be more like what we see on desktop.
Also let's change to the page title to "Page Not Found | elementary OS" instead of 404.
Make sure you're using "row" and not "grid-margin" as this has changed in master

Port to the templating system, please. :)


fix(.npmignore): Ignore MongoDB data directory

mikro-orm is currently published with 920 MB of test data left over from run-rs. Adding /data to .npmignore excludes this test data from the published package and greatly reduces the installation footprint.
I'm unsure how to add a test for this, if it's even reasonably possible. I saw tests requested as part of the pull request guidelines, so let me know if there's anything I ought to do in that regard.
I've only been working with mikro-orm for a short while now, and I'm really liking it so far. Keep up the good work, and I'm happy to have the opportunity to contribute!

Thanks!


Why is mining a pubkey maybe cryptographically dubious?

I had trouble finding where the note to this effect was originally added to the documentation. I was hoping for some kind of explanation of just how dubious.

On Wed, Sep 13, 2023 at 03:29:19PM -0700, Glenn Willen wrote:

I had trouble finding where the note to this effect was originally added to the documentation. I was hoping for some kind of explanation of just how dubious.

I'm not sure if it is or not. It is just a statement of ignorance.

Well, I was specifically wondering what would make it more dubious than just regular key generation. After reading the source, I'm assuming it's about the fast trick for generating the next candidate, at the bottom of the loop?

On Fri, Sep 15, 2023 at 01:08:52PM -0700, Glenn Willen wrote:

Well, I was specifically wondering what would make it more dubious than just regular key generation. After reading the source, I'm assuming it's about the fast trick for generating the next candidate, at the bottom of the loop?

I think I was wondering if grinding a pubkey with a certain number of
leading zeros would reduce the uniformity of the keyspace or something,
and if that opened it up to some kind of attacks. I didn't think it
would be an issue but I'm not sure becase I'm not cryptographer.
The xor thing seemed fine? I hope? =/

I am also not a cryptographer but I have worked with a few. I'm pretty sure grinding itself is fine, I'm pretty sure I had cryptographically-knowledgeable ex-coworkers who used bitcoin vanity addresses (which are ground like this.)
The xor thing is kind of sus to me, in the way of "I'm pretty sure it's fine; I couldn't come up with anything wrong with it in 5 minutes of thinking, which was enough for me to use it; but I would run it by a cryptographer to be sure."
I can try to get one of of my Blockstream ex-coworkers to look at it if you like.

On Sat, Sep 16, 2023 at 02:39:12PM -0700, Glenn Willen wrote:

I am also not a cryptographer but I have worked with a few. I'm pretty sure grinding itself is fine, I'm pretty sure I had cryptographically-knowledgeable ex-coworkers who used bitcoin vanity addresses (which are ground like this.)
The xor thing is kind of sus to me, in the way of "I'm pretty sure it's fine; I couldn't come up with anything wrong with it in 5 minutes of thinking, which was enough for me to use it; but I would run it by a cryptographer to be sure."
I can try to get one of of my Blockstream ex-coworkers to look at it if you like.

sure! thanks!

There is no problem grinding a secret key to get some property of the public key, as long as the only thing you're filtering for is a pubkey property. (If this were a problem, then an attacker could e.g. scan the Bitcoin blockchain for public keys that naturally had a lot of 0s, and then apply whatever attack your grinding supposedly enabled, and all of pubkey cryptography.)
But this xor thing does seem super weird, and this tactic of repeatedly generating fresh independent keys is extremely slow. You should be able to get a multiple-hundred-times speedup by using e.g. https://github.com/bitcoin-core/secp256k1/pull/507

Because it is not otherwise linked here, the grinding code is at https://github.com/jb55/nostril/blob/8726ad3b15f10e5a09864bbef61f7a35d1d2f3d7/nostril.c#L255

On Sun, Sep 17, 2023 at 06:36:26AM -0700, Andrew Poelstra wrote:

There is no problem grinding a secret key to get some property of the public key, as long as the only thing you're filtering for is a pubkey property. (If this were a problem, then an attacker could e.g. scan the Bitcoin blockchain for public keys that naturally had a lot of 0s, and then apply whatever attack your grinding supposedly enabled, and all of pubkey cryptography.)
But this xor thing does seem super weird, and this tactic of repeatedly generating fresh independent keys is extremely slow. You should be able to get a multiple-hundred-times speedup by using e.g. https://github.com/bitcoin-core/secp256k1/pull/507

happy to drop it, I'll look into applying this patch locally if it hasn't bitrotted.

@jb55 even if the patch does not apply, you can get a speedup by using the public API to repeatedly call pubkey_tweak_add and seckey_tweak_add with the same fixed value (say, 1). This should still give you a massive speedup, though it will still leave a lot of performance on the table.

On Sun, Sep 17, 2023 at 08:01:29AM -0700, Andrew Poelstra wrote:

@jb55 even if the patch does not apply, you can get a speedup by using the public API to repeatedly call pubkey_tweak_add and seckey_tweak_add with the same fixed value (say, 1). This should still give you a massive speedup, though it will still leave a lot of performance on the table.

oh! ok that makes sense. If someone reading this wants to PR that I would appreciate it.


Region Of Interest

I am using datalab V0.9.2. I would like to add/modify/delete ROIs to an image already loaded with a macro.
There is no example with ROIs in the tutorials.
I know how to acces the ROIs of my image:
image = proxy.get_object(2)
data = np.array (image.data, copy=True)
and the ROIs are in image.roi.
I have tried :
proxy.add_object(image)
It saves the image with a new number, but without the ROIs.
Could you help ?
charlesreve

Hi,
Thanks for the feedback.
This is a limitation of current implementation of the add_object method which is only passing data, title and labels (not metadata).
But, this has just been taken care of: we just added full support for signal and image objects!
Here's the proof:

Would you be able to wait a few days for this feature?
This will be available in forthcoming v0.18.1.
Otherwise, you may also try the latest version using the main branch which is generally quite stable.
Cheers,
Pierre

Thank you !
I’ll wait for v0.18.1.
De : Pierre Raybaut @.>
Envoyé : jeudi 5 décembre 2024 11:50
À : DataLab-Platform/DataLab @.>
Cc : REVERDIN Charles 087700 @.>; Author @.>
Objet : Re: [DataLab-Platform/DataLab] Region Of Interest (Issue #111)
Hi,
Thanks for the feedback.
This is a limitation of current implementation of the add_object method which is only passing data, title and labels (not metadata).
But, this has just been taken care of: we just added full support for signal and image objects!
Here's the proof:
image.png (view on web)https://github.com/user-attachments/assets/4909942f-346e-4a38-8aeb-07dec81a43ce
Would you be able to wait a few days for this feature?
This will be available in forthcoming v0.18.1.
Otherwise, you may also try the latest version using the main branch which is generally quite stable.
Cheers,
Pierre
—
Reply to this email directly, view it on GitHubhttps://github.com/DataLab-Platform/DataLab/issues/111#issuecomment-2519956899, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BNOKSYKUPNQRBJJ2ARZ4RN32EAVVDAVCNFSM6AAAAABTANWEXWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJZHE2TMOBZHE.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

v0.18.1 has just been released


Fix CLI and some refactoring

fix CLI commands set-ports and toggle-ports
refactor _execute_try -> remove lambda. Use stronger type and better error message

@stephrdev  What about removing the check and raise here? It should be already ok with the type
class RequestFrame:
    def __init__(self, command: CommandCodes, address: int = 0, data: int = 0) -> None:
        if command not in CommandCodes:
            raise RelayCardError(f"Wrong command{command}. Not found in {CommandCodes}")
        self.command = command

@stephrdev What about removing the check and raise here? It should be already ok with the type

Agreed.

Some stuff I noticed:

[ ] Card is initialized even if no card-command like get-ports/set-ports is used (for example running without any command at all)
[ ] Default serial port is defined in two places. Couldn't we remove the default from the Card class and only have it in the cli?

@stephrdev What about removing the check and raise here? It should be already ok with the type

Agreed.

Done

Some stuff I noticed:
* [x]  Card is initialized even if no card-command like get-ports/set-ports is used (for example running without any command at all)

* [x]  Default serial port is defined in two places. Couldn't we remove the default from the Card class and only have it in the cli?

I think the "action" commands could be made into a exclusic group and put some args as required

Anything else missing for this PR?

No, i’m on vacation and had no chance to check an merge. Will do asap


remove New and Init

As the zero value is OK to use, the New method is unnecessary.
Likewise, the Init method isn't necessary (*d = deque.Deque{} suffices).
Note that the existing Init method didn't actually clear all the fields (it didn't initialize lastTailPosition).

Codecov Report

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

@@          Coverage Diff          @@
##           master     #3   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines         128    118   -10
=====================================
- Hits          128    118   -10

Impacted Files
Coverage Δ

deque.go
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

I agree there's really no need to have New and Init. Either using the deque zero value or the new keyword suffices. However, we'd love to get the deque into Go's std lib and so it's important for the deque to resemble the std lib container packages as much as possible. list and ring, for instance, provide a New method.
But mostly important, removing the New and Init would be a breaking change to the API. Although there's probably not many people using the deque right now, I want to make sure we follow the deque principles everywhere and at every step of the way. Breaking changes causes a lot of trouble for the users and it's our goal to treat the deque users like kings. They are the reason the deque exist and everything we do, should be aimed at making the deque users happier. Breaking changes is not a good way to start. :-)
Regarding the issue with Init not initializing lastTailPosition, I truly appreciate the insight and the recommendation to use "*d = deque.Deque{}" instead of clearing all fields one by one. I have commit this change here and it was released with v1.0.1.

I'll keep this PR open to collect feedback from other users. If you strongly feel like Init and New should be removed, please let us know. We love to get feedback and am very responsive and appreciative of them.

With regard to API stability, the API hasn't been generally reviewed yet, so I think it's probably too early.
That said, you could always publish v2.
The list and ring packages require a New function because with neither of them is the zero value OK to use. Other types in the Go stdlib do not supply a New function unless it has an argument.
I don't know any examples of Init used as a method to zero a value. Reset is the usual standard, and that's used (for example by time.Timer) when we explicitly do not want to zero an object. I would support a Deque.Reset method that removed all items from the queue but kept around some allocated slices.

I agree it's probably too early to enforce API stability and it's definitely better to have a simpler API. Regarding the Deque.Reset suggestion, sounds very useful to me. I vote we do it. Feel free to get started with it. Otherwise let me know and I should be able to work on it this Friday (busy tomorrow). The changes looks good. 👍

We've been thinking about removing the Init and New methods and although these methods are really not required, they make using the deque a bit easier and cleaner. Using "d := deque.New()" not only looks better, but also saves a few keystrokes when compared to "d := new(deque.Deque)". Besides, we'll likely need a New method on the deque's safe for concurrent use future version, so both versions would be API compatible.
Also on top of that, it's a terrible inconvenience to have the users to perform code changes when they upgrade deque to a new version. Although apparently deque still has very few users right now given its infancy, even if a single user is using deque, that user deserves the MOST respect, and not the least. So I reverted the changes. We'll keep the API unchanged and won't do any breaking changes on this package. No v2 releases as well as this also requires code changes if the users want to upgrade to a v2 version.
This change have been reverted.

Using "d := deque.New()" not only looks better, but also saves a few keystrokes when compared to "d := new(deque.Deque)".

With other similar data structures, it's just as common to do:
var d deque.Deque

which is just as nice IMHO. By including New and stating that the zero value is OK to use, you impose cognitive overhead because it's not entirely clear which is best to use.

Besides, we'll likely need a New method on the deque's safe for concurrent use future version, so both versions would be API compatible.

None of Deque's anologs in the standard library are safe for concurrent use, so why should Deque itself become so? And why would you need a New method even in the concurrent-safe version, given that mutexes are OK to use when they're zero?
Also, note that it would be a breaking change to require New, given that you've stated that the zero value is OK to use.
One kind of New method that I'd consider fine is one that actually had an argument. For example, how about:
 // New returns a Deque instance pre-initialized with the given capacity.
 // Although the number of items in the queue may rise higher, there
 // will always be space available for at least cap items without allocation
 // even when there are no items in the queue.
 func New(cap int) *Deque

Second thing: you argued in favour of New, but why keep Init?

Excellent points! The idea of a New(cap) where deque would use a single, fixed size slice as its underlying data structure sounds useful. If the user knows for sure how many items they will push into the deque, deque could potentially perform a bit faster. The only problem with this change is the fact it would make deque not only a unbounded, but also a bounded data structure (according to how its used). I like the idea and I think its worth exploring as this seems to be a pretty common scenario (knowing the number of items beforehand).
In any case, there's no right or wrong answer whether New(with or without params) and Init should be there or not. Let's let deque's current and future users have a say about it. I'll file issues proposing both changes: Remove New and Init; Have New with a cap.

The idea of a New(cap) where deque would use a single, fixed size slice as its underlying data structure sounds useful.

To be honest, I wasn't thinking that it would make a single slice for all the elements, as that could have significant disadvantages if the size of the queue rises above the initial capacity and then drops again. I'd just allocate (cap + (maxInternalSliceSize-1))/maxInternalSizeSize nodes, each with maxInternalSliceSize capacity, and then just use them as normal. Then the queue would perform exactly as normal when the capacity rises above the initial capacity.

Sounds interesting. Can you please post an issue about it?
I created one to remove New and Init here. Please leave your comments there (copy and paste from here).

Quick note: deque's future safe for concurrent use version will not be based on mutexes. Mutexes degrade performance a bit too dramatically. Deque's safe for concurrent use will use mixed techniques to achieve safe for concurrency use. I'm expecting much better performance when compared to using only mutexes.


Fix negative amount #168

Issues
https://github.com/SushiChain/SushiChain/issues/168

Great work!! another option might have been to use Math abs - but this is probably better.


feat(css minimize): add CSS minification in production build for all templates.

As discussed on #844, this PR allows webpack to compress all CSS using the minimize property from the css-loader plugin. It only applies when --env.prod is provided.
Please note this will also compress any css passed through webpack, therefore you should expect a small reduction in size on files other than vendor.css. Out of curiosity, I'm posting a comparison for each template, showing all the affected files their corresponding sizes before and after this change.
Finally, I also would like to highlight the following passage from the css-loader project:

In some cases the minification is destructive to the css, so you can provide some options to it....

Although I couldn't find any errors while testing the templates, I encourage everyone to read the css-loader documentation in case of a css compression error.

Angular2Spa Template

File
css-loader
css-loader?minimize
Diff (%)

vendor.css
315kB
286kB
29kB (9.21%)

main-client.js
12.2kB
11.1kB
1.1kB (9.02%)

main-server.js
150kB
146kB
4kB (2.67%)

Total
477.2kB
443.1kB
34.1kB (7.15%)

AureliaSpa Template

File
css-loader
css-loader?minimize
Diff (%)

vendor.css
315kB
286kB
29kB (9.21%)

app.js
26.4kB
26.1kB
0.3kB (1.14%)

Total
341.4kB
312.1kB
29.3kB (8.58%)

KnockoutSpa Template

File
css-loader
css-loader?minimize
Diff (%)

vendor.css
315kB
286kB
29kB (9.21%)

0.js
1.17kB
1.12kB
0.05kB (4.27%)

1.js
2.01kB
1.98kB
0.03kB (1.49%)

2.js
0.577kB
0.563kB
0.014kB (2.43%)

main.js
5.99kB
5.90kB
0.9kB (1.5%)

site.css
1.52kB
0.729kB
0.791kB (52%)

Total
326.2kB
296.2kB
29.9kB (9.19%)

ReactReduxSpa Template

File
css-loader
css-loader?minimize
Diff (%)

vendor.css
315kB
286kB
29kB (9.21%)

main-server.js
94.2kB
93kB
1.2kB (1.27%)

site.css
1.52kB
0.729kB
0.791kB (52%)

Total
410.7kB
379.7kB
30.9kB (7.55%)

ReactSpa Template

File
css-loader
css-loader?minimize
Diff (%)

vendor.css
315kB
286kB
29kB (9.21%)

site.css
1.52kB
0.729kB
0.791kB (52%)

Total
316.5kB
286.7kB
29.7kB (9.41%)

VueSpa Template

File
css-loader
css-loader?minimize
Diff (%)

vendor.css
315kB
286kB
29kB (9.21%)

main.js
21.8kB
21.6kB
0.2kB (0.9%)

Total
336.8kB
307.6kB
29.2kB (8.67%)

Please, feel free to recommend/perform any necessary modifications.

@etimesg,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

@etimesg, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot

@etimesg In case you got a notification with some other review comments from me, you can ignore it, because I realised my comments were wrong so I deleted them.


Create CallAgent has typos in Android Quickstart

Typos in the code snippet for creating the CallAgent which cause errors.  It says DeviceManage (no 'r') when it should be DeviceManager and there is an extra parenthesis in the method to create the CallAgent.
Currently the code snippet shows:
String userToken = '<user token>'; CallClient callClient = new CallClient(); CommunicationTokenCredential tokenCredential = new CommunicationTokenCredential(userToken); android.content.Context appContext = this.getApplicationContext(); // From within an Activity for instance CallAgent callAgent = callClient.createCallAgent**(**(appContext, tokenCredential).get(); **DeviceManage** deviceManager = callClient.getDeviceManager().get();
The code should be:
  callClient = new CallClient();             CommunicationTokenCredential tokenCredential = new CommunicationTokenCredential(userToken);             android.content.Context appContext = this.getApplicationContext(); // From within an Activity for instance             callAgent = callClient.createCallAgent(appContext, tokenCredential).get();             DeviceManager deviceManager = callClient.getDeviceManager().get();

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

ID: f4267f1d-40a1-bd0e-3954-0986657f3930
Version Independent ID: ce985f80-0851-4464-d553-4bfb0a2c5424
Content: Quickstart - Add voice calling to your app - An Azure Communication Services quickstart
Content Source: articles/communication-services/quickstarts/voice-video-calling/getting-started-with-calling.md
Service: azure-communication-services
GitHub Login: @GrantMeStrength
Microsoft Alias: jken

@v-jameelaesa Thanks for the feedback! We are currently investigating and will update you shortly.

@v-jameelaesa Thank you for the feedback. After review, we agree that this is a doc bug and needs to be fixed. We submitted and merged a PR to update the doc. The changes should be public within 24 hours for this document.
We appreciate you raising this up to us so we can prevent others from being impacted by the same issue.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Toggle theme doesn't do anything

In Sketch 3.4.2, this feature doesn't do anything... Thanks

I got it~

Cool thank you!


More variations of Pipe and Compose

Added variations of Pipe and Compose for 5 to 16 functions.
The functions were generated by python code that was added as a comment

Maybe the python code should be added to a new generators folder and be used with go generate

Thanks @Force4760 !!
Couple of comments:
1 - I really like the idea of increasing the number of variations for pipe and compose but on the tests I would like to be more consistent in the testing style. I see pro and cons for both solutions (old tests and new ones). What I would like to suggest is to make them consistent. I think a good idea could be having couple of helper functions in the test file and name them add1 and double that can be reused in combination in all tests as you suggested in your MR. This means we should refactor a bit also the old tests.
2 - I would remove the generators comments from within the tests files as I hope won't be used again.
What do you think?

I like the add1 and double functions and unifying the tests, will work on it asap
As far as the comments go, I think we shouldn't remove them since they can be useful if there's a need to make a change to either the tests or the functions later on.

@Force4760 we still need to remove the comments in the tests file after adding the generators folder with the py script


kicad-nightly-lite: Add version r20311.50e22de3ba

This is addition to https://github.com/lukesampson/scoop-extras/pull/5529, but nightly version this time.
Probably already outdated version (I don't have windows machine nearby at this moment). I rely on auto updates by Excavator.

/verify

/verify

Now I see that. i686 and x86_64 builds have different version because commits happen quite often and thet occur even between builds.
https://kicad-downloads.s3.cern.ch/index.html?prefix=windows/nightly/
Any suggestion?
The only solution I got is separate manifest fo 32bit and 64bit variants....

Now I see that. i686 and x86_64 builds have different version because commits happen quite often and thet occur even between builds.
https://kicad-downloads.s3.cern.ch/index.html?prefix=windows/nightly/
Any suggestion?
The only solution I got is separate manifest fo 32bit and 64bit variants....

Can you update this to the latest version?

i686 and x86_64 builds have different version because commits happen quite often and they occur even between builds.

That's fine. We can keep same version for both, and the manifest will only update itself when both 32 and 64bit builds are at same versions.

Issue is no longer present. I'll update manifest next week.

Here's an updated manifest - there are no longer 32bit binaries available: https://downloads.kicad.org/kicad/windows/explore/nightlies
{
    "version": "r27499.f14dc9e305",
    "description": "Electronics Design Automation Suite, nightly build without libraries",
    "homepage": "https://kicad.org",
    "license": "GPL-3.0-only",
    "architecture": {
        "64bit": {
            "url": "https://kicad-downloads.s3.cern.ch/windows/nightly/kicad-nightly-r27499.f14dc9e305-x86_64-lite.exe#/dl.7z",
            "hash": "6ec3214d982c3a61259ce65a71a3846f74ca2e885ea722d8d0211f2aa9db0e3d"
        }
    },
    "pre_install": "Remove-Item \"$dir\\`$*\", \"$dir\\Uninst*\" -Recurse",
    "bin": [
        [
            "bin\\kicad.exe",
            "kicad-nightly"
        ]
    ],
    "shortcuts": [
        [
            "bin\\kicad.exe",
            "KiCad-nightly"
        ]
    ],
    "notes": "You should configure environment paths KICAD6_3DMODEL_DIR, KICAD6_FOOTPRINT_DIR, KICAD6_SYMBOL_DIR, KICAD6_TEMPLATE_DIR, KICAD_USER_TEMPLATE_DIR, otherwise they'll default to $dir\\share\\kicad",
    "checkver": {
        "url": "https://kicad-downloads.s3.cern.ch/?delimiter=/&prefix=windows/nightly/",
        "regex": "<Key>windows/nightly/kicad-nightly-(r\\d+\\.[a-z0-9]+)-x86_64-lite\\.exe</Key>",
        "reverse": true
    },
    "autoupdate": {
        "architecture": {
            "64bit": {
                "url": "https://kicad-downloads.s3.cern.ch/windows/nightly/kicad-nightly-$version-x86_64-lite.exe#/dl.7z"
            }
        }
    }
}

/verify

For some reason the file was last updated today according to this page: https://downloads.kicad.org/kicad/windows/explore/nightlies
Should checkver look at the <LastModified> tag too?

Sure.
Let's handle this in a new PR.

Superceded by #512


ProcessTomography analysis gives UnboundLocalError: local variable 'num_cond' referenced before assignment

Informations

Qiskit Experiments version: 0ce3f85d7609a81febf745bea84d9d000f8533b4
Python version: 3.10.11
Operating system: Arch Linux

What is the current behavior?
Steps to reproduce the problem

Run a ProcessTomography experiment and save the data to the database service

from qiskit import QuantumCircuit, QuantumRegister
from qiskit_experiments.library import ProcessTomography
from qiskit_ibm_provider import IBMProvider

hub = 'ibm-q-internal'
group = 'deployed'
project ='default'

provider = IBMProvider(instance=f"{hub}/{group}/{project}")
backend = provider.get_backend("ibm_sherbrooke")

register = QuantumRegister(2, name="q")
target_circuit = QuantumCircuit(register)
a, b = register
target_circuit.h(a)
target_circuit.cnot(a, b)

experiment = ProcessTomography(target_circuit)
device_data = experiment.run(backend, shots=10000)
device_data.save()

Load the data from the service and attempt to produce analysis results

from qiskit_experiments.framework.experiment_data import ExperimentData

service = ExperimentData.get_service_from_backend(backend)
data = ExperimentData.load("4f939335-6fe0-4711-b47e-e7c6a14afb0c", service)
experiment = ProcessTomography(target_circuit)
data = experiment.analysis.run(data)
data.block_for_results()

Observe error

Analysis callback failed [Experiment ID: 1ccd621b-cdb8-4c16-b575-db350cbf99d2][Analysis Callback ID: d313440656894aecb8c2338ac6f23ecd]:
Traceback (most recent call last):
  File "/home/kjs/projects/qiskit-experiments/qiskit_experiments/framework/experiment_data.py", line 992, in _run_analysis_callback
    callback(self, **kwargs)
  File "/home/kjs/projects/qiskit-experiments/qiskit_experiments/framework/base_analysis.py", line 169, in run_analysis
    results, figures = analysis._run_analysis(expdata)
  File "/home/kjs/projects/qiskit-experiments/qiskit_experiments/library/tomography/tomography_analysis.py", line 192, in _run_analysis
    outcome_data, shot_data, meas_data, prep_data = tomography_fitter_data(
  File "/home/kjs/projects/qiskit-experiments/qiskit_experiments/library/tomography/fitters/fitter_data.py", line 122, in tomography_fitter_data
    if num_cond:
UnboundLocalError: local variable 'num_cond' referenced before assignment

What is the expected behavior?
Should work.
Suggested solutions
Weird thing I noticed is that if I add the following line, which should be unnecessary, then there is no error, but also no analysis results are produced (data.analysis_results() is an empty list)
data.add_jobs([provider.retrieve_job(job_id) for job_id in data.job_ids])

Ok specifying the provider fixes the issue. The error message should be made more helpful though.
from qiskit_experiments.framework.experiment_data import ExperimentData

service = ExperimentData.get_service_from_backend(backend)
data = ExperimentData.load("4f939335-6fe0-4711-b47e-e7c6a14afb0c", service, provider=provider)
experiment = ProcessTomography(target_circuit)
data = experiment.analysis.run(data)
data.block_for_results()

the data is retrieved using _retrieve_data()  in ExperimentData class. If  provider is not passed when loading the data, the method  _retrieve_data()  return nothing.
I will add a meaningful message so the user will know.


Update NumberInputLazy to select the text automatically

When a user selects the numbers in these fields, it is typically to overwrite the values, rather than append to it. It would be nice if these fields automatically selected the whole text so a user just has to type to overwrite, rather than select, delete, then start typing

I can take a look! Been working with NumberInputLazy for a bit so shouldn't be too hard.


Direct extraction of SRS data

Direct extraction, as described in the LDP Admin Guide, is needed for SRS data because they are too large to extract via the SRS API.

A different approach to data extraction is now planned for 1.2, making this issue no longer relevant.


It doesn't update in all items in a flat grid or flat item in a scroll view, this happens in android only.

I am getting the following error for items that are not currently visible in a flat list.
Possible Unhandled Promise Rejection (id: 0):
Error: Failed to findViewById using handle 0x32f.
Error: Failed to findViewById using handle 0x32f.
at createErrorFromErrorData (/Users/parkal/Documents/StutiSadhanam-Client/.vscode/.react/index.bundle:2114:17)
at /Users/parkal/Documents/StutiSadhanam-Client/.vscode/.react/index.bundle:2066:27
at MessageQueue.__invokeCallback (/Users/parkal/Documents/StutiSadhanam-Client/.vscode/.react/index.bundle:2633:18)
at /Users/parkal/Documents/StutiSadhanam-Client/.vscode/.react/index.bundle:2364:18
at MessageQueue.__guard (/Users/parkal/Documents/StutiSadhanam-Client/.vscode/.react/index.bundle:2537:13)
at MessageQueue.invokeCallbackAndReturnFlushedQueue (/Users/parkal/Documents/StutiSadhanam-Client/.vscode/.react/index.bundle:2363:14)
at /Users/parkal/Documents/StutiSadhanam-Client/.vscode/.react/debuggerWorker.js:187:58
at process. (/Users/parkal/Documents/StutiSadhanam-Client/.vscode/.react/debuggerWorker.js:60:9)
at process.emit (events.js:189:13)
at emit (internal/child_process.js:820:12)

Likely because the views have been unmounted whilst offscreen. Would it be possible to try remounting the component when fonts have been changed?


optimize: relayout algorithm bug

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


Added TaskMuncher to Showcase

[x] I have followed (at least) the PR section of the contributing guide.

@johnvincentio Well done with the project. Thanks for sharing.


JKNS-384: Update Jenkins base images to RHEL9

update ubi7 to ubi9

Signed-off-by: Avinal Kumar<EMAIL_ADDRESS>

/test all

I think the references to RHEL9 should be updated as well in this PR. Please let me know if that should be a different PR.

We should aim to use RHEL 9/UBI9 in as many spots as possible. Note that there is a lot in this repo that appears to be way out of date, and isn't used either in OpenShift CI or our downstream build systems.
One thing I'd advise is to copy the Dockerfile.rhel8 files to Dockerfile.rhel9, where you then update the base UBI images. That should help make the transition a bit smoother.

/remove-lifecycle rotten

/cherry-pick release-4.17 release-4.16

/lgtm


add Type + card link function

Works as expected locally.

Hi, @paulMery. I'm reviewing this right now.


WalletContextState is not getting updated, so cannot get any value like publicKey or connected

I successfully connect to the wallet:

Code is below, but publicKey is not getting updated after I am connected so useEffects is not triggered aka I am not getting anything from useWallet:
import {
  WalletError,
} from "@solana/wallet-adapter-base";
import {
  ConnectionProvider,
  useLocalStorage,
  useWallet,
  WalletProvider,
} from "@solana/wallet-adapter-react";
import {
  getBloctoWallet,
  getLedgerWallet,
  getPhantomWallet,
  getSlopeWallet,
  getSolflareWallet,
  getSolletExtensionWallet,
  getSolletWallet,
} from "@solana/wallet-adapter-wallets";
import { useSnackbar } from "notistack";
import { useCallback, useEffect, useMemo } from "react";
import { useRecoilState, useRecoilValue } from "recoil";
import {
  atomClusterApiUrl,
  atomIsWalletConnected,
  atomNetwork,
} from "../../recoil/atoms";

function WalletSolana() {
  const network = useRecoilValue(atomNetwork);
  const clusterUrl = useRecoilValue(atomClusterApiUrl);

  const { publicKey, connected } = useWallet();

  const [isWalletConnected, setWalletConnected] = useRecoilState(
    atomIsWalletConnected
  );

  const [autoConnect, setAutoConnect] = useLocalStorage("autoConnect", false);

  const wallets = useMemo(
    () => [
      getPhantomWallet(),
      getSolflareWallet(),
      getSlopeWallet(),
      getLedgerWallet(),
      getBloctoWallet(),
      getSolletWallet(),
      getSolletExtensionWallet(),
    ],
    [network]
  );

  const { enqueueSnackbar } = useSnackbar();

  const onError = useCallback(
    (error: WalletError) => {
      enqueueSnackbar(
        error.message ? `${error.name}: ${error.message}` : error.name,
        { variant: "error" }
      );
      console.error(error);
    },
    [enqueueSnackbar]
  );

  useEffect(() => {
    if (publicKey) {
      console.log("got public key " + publicKey);
    }
  }, [publicKey]);

  return (
    <div>
      <ConnectionProvider endpoint={clusterUrl}>
        <WalletProvider
          wallets={wallets}
          onError={onError}
          autoConnect={autoConnect}
        >
          <ReactUIWalletModalProvider>
            <ReactUIWalletMultiButton />
          </ReactUIWalletModalProvider>
        </WalletProvider>
      </ConnectionProvider>
    </div>
  );
}

export default WalletSolana;

This has been resolved by wrapping the app with context.

Yeah, you need to wrap the app -- or more specifically, any component subtree you expect to call the useWallet hook from -- with the context. Glad you figured out out!

Thanks for the questions and answers.  I was briefly stuck on this.

can somebody give me an example on how should I wrap my component with context?

@justinbalaguer See https://github.com/solana-labs/wallet-adapter/blob/master/packages/starter/example/components/ContextProvider.tsx
In the example Next.js app, this is used to wrap the entire app: https://github.com/solana-labs/wallet-adapter/blob/master/packages/starter/example/pages/_app.tsx


Various documentation fixes.

Greetings Pietro,
I've had a run through the docs correcting a number of spelling and grammatical errors, as well as a few areas where the docs referenced deprecated API usage.
Cheers,
Brad

There is just that little thing to be fixed, but thank you so much for this PR!
While I was reading the diff, I felt embarassed multiple times for some of the errors...

Alright, I think I've made this bit about component names more clear.

While I was reading the diff, I felt embarrassed multiple times for some of the errors...

Ha, no worries. There's a lot to be documented, and it's not at all as interesting as working on or with the code :) Since you have more pressing decisions to make about the code, and seen you've released it to PyPI, I thought I'd have a go through the docs.
Brad

Again, thank you! Will be live with botogram 0.2.

And it's live on the development docs!


Layout issue on Add Exchange Rate page.

Framework Version: 0.1.1
Test Steps:

Click on Add Exchange Rate.
Leave Rate field blank and click on Save Exchange Rate.

Framework Version: 0.1.3
Fixed


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/73013

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/95482

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/108777

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/119102