citra: nightly 1873 -> 1903, canary 2440 -> 2479

Description of changes
Bump versions.
Closes https://github.com/NixOS/nixpkgs/issues/216744 via #224337.
Closes https://github.com/NixOS/nixpkgs/issues/216745 via #224337.
Closes https://github.com/NixOS/nixpkgs/issues/225633 (no cmake stuff).

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 225744 run on x86_64-linux 1

2 packages built:

citra-canary
citra-nightly

Weird thing. I can't compile canary 2475. OK. Now, I can't compile 2474 too (one release behind, successfully built). I keep nightly at latest (successfully built) while leaving canary few releases behind.

Result of nixpkgs-review pr 225744 run on x86_64-linux 1

2 packages failed to build:

citra-canary
citra-nightly

I'm getting a hash mismatch on compat-list.
error: hash mismatch in fixed-output derivation '/nix/store/a2gm2xgfjp874cnlvkvh9rzbxafjv11v-citra-compat-list.drv':
         specified: sha256-qQkFRlIQcXToKShWilwNLR0AcENyTf073hTfvj9RNeQ=
            got:    sha256-J+zqtWde5NgK2QROvGewtXGRAWUTNSKHNMG6iu9m1fU=
error: 1 dependencies of derivation '/nix/store/zvwm0hz1xypwf79d2iycpmrd82lamlih-citra-canary-2478.drv' failed to build
error: 1 dependencies of derivation '/nix/store/cynwnjvvwy6ga8n7rmppan1l63kqyv27-citra-nightly-1902.drv' failed to build
error: 2 dependencies of derivation '/nix/store/ziirff01zbxcqxrhcrbbc46maakrbrzh-review-shell.drv' failed to build

@ofborg build citra-nightly citra-canary

Same compat-list, new hash. Who dis?
trying https://web.archive.org/web/20230428030454/https://api.citra-emu.org/gamedb/
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
remote: Enumerating objects: 2046, done.
remote: Counting objects: 100% (2046/2046), done.
100 13356    0 13356    0     0  36446      0 --:--:-- --:--:-- --:--:-- 36392
error: hash mismatch in fixed-output derivation '/nix/store/7frsxj6gr92mala5zymmzzs5xx5fpacl-citra-compat-list.drv':
         specified: sha256-AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA=
            got:    sha256-iOMzlvQznLTFdo6MqL60N0ON6HlrWZEe1srWtYLi5m4=

@ofborg build citra-nightly citra-canary

Result of nixpkgs-review pr 225744 run on x86_64-linux 1

2 packages failed to build:

citra-canary
citra-nightly

error: hash mismatch in fixed-output derivation '/nix/store/lvd0m2a9r00rgw0nmwzwbgq5ybx3dh4m-citra-compat-list.drv':
         specified: sha256-iOMzlvQznLTFdo6MqL60N0ON6HlrWZEe1srWtYLi5m4=
            got:    sha256-J+zqtWde5NgK2QROvGewtXGRAWUTNSKHNMG6iu9m1fU=
error: 1 dependencies of derivation '/nix/store/968jpqmm9rrnhs1g1ll2hrdaqd7qyz0g-citra-canary-2479.drv' failed to build
error: 1 dependencies of derivation '/nix/store/ppb1pczc2qc13638xaxclq6mk2kw282c-citra-nightly-1903.drv' failed to build
error: 2 dependencies of derivation '/nix/store/hcmqhlmawrjj05f5rmajj6r3v1jq456z-review-shell.drv' failed to build

I'm still getting a hash mismatch, but I got the same hash as my build attempt from yesterday. ofborg's build is getting this hash as well.
How are you building it on your end?

How are you building it on your end?

I follow the documentation. Basically, I go in nixpkgs folder and do nix-build -A <pkg>. I assume it had nothing to do with my end because this is related to compat-list. I try a new snapshot. Never ran into this issue with hash changing before.

@ofborg build citra-nightly citra-canary

Result of nixpkgs-review pr 225744 run on x86_64-linux 1

2 packages built:

citra-canary
citra-nightly

Never ran into this issue with hash changing before.

Same. Strange issue, but it builds successfully now, so I guess all's well that ends well.

This should get reviewed and merged. The next citra versions will need to be looked at again... due to new vulkan backend support (currently in canary and eventually nightly) and possibly other stuff.

Result of nixpkgs-review pr 225744 run on x86_64-linux 1

2 packages built:

citra-canary
citra-nightly

@ofborg build citra-nightly citra-canary

This should get reviewed and merged. The next citra versions will need to be looked at again... due to new vulkan backend support (currently in canary and eventually nightly) and possibly other stuff.

Agreed.
Friendly ping @abbradar @IvarWithoutBones @kira64xyz

Friendly ping @NickCao


bug(@angular/google-maps): @types/googlemaps is deprecated in favor of @types/google.maps

As posted on the npm page @types/googlemaps is deprecated, @angular/google-maps specifies this as a dependency. It should now be @types/google.maps instead
see:
https://www.npmjs.com/package/@types/googlemaps

We can't use the new types package, since we get types declared in both packages.
It would be useful to get this changed

We can't use the new types package, since we get types declared in both packages.

You can likely avoid this issue with skipLibCheck: true in tsconfig https://www.typescriptlang.org/tsconfig#skipLibCheck


Any insight on JSON-RPC services?

Hey @kbaltrinic, I just wanted to pick your mind about using http-backend-proxy for JSON-RPC calls...
JSON-RPC are solely POST calls with a specified method and params (see http://webob.readthedocs.org/en/latest/jsonrpc-example.html)... so it's pretty hard to differentiate one call from another...
It would be great to see what you think...Thank you!

Yep so this isn't specific to the proxy, this is the same issue even if you are using the http-backend by itself.  I am afraid there is no magic here to help you.
You have two options.  One is to write a response fuction that parses the data argument back to JSON and then evaluates it and does the right thing.  That is probably the best approach, esp. if you want to validate that the right JSON is being passed to the back end.  If you just want to get the back end to return different responses to different tests, and ignore the input sent, then you could use a simpler response function that just returns a value defined on the context object.  Then all you need to do is modify and manually sync the context object between tests.


Document exposed system properties

We have several system properties scattered throughout our codebase that aren't necessarily the most convenient to use since you have to read the code to find them. To help surface these options we can go one of two ways:

In the core module have a class like ConfigConstants.java that is just a bunch of static strings we can use as keys. Then we can add comments to each key to explain what the config does. The benefit of this is the documentation lives with the code making it easier to keep up to date.
Have a top level table defining each property. The benefit of the table is that it will be very visible at the top of the repo and will be formatted in a nice, tabular format. The downside is we will need to make sure we keep this up to date every time we add a new config key.

What I would really like is a Java equivalent of https://github.com/kelseyhightower/envconfig or https://github.com/alecthomas/kong


Landscape support

For now the app has been developed with assumption it would be primarily on portrait mode, we need a design update with list of issues to address if we want the app to be both landscape and portrait mode friendly.

Dupe of #59?


Consider compile target when placing hash.txt

          @maccesch  one last thing i noticed is it doesn't output the hash file in the correct release folder when cross compiling

e.g. with this command
LEPTOS_HASH_FILES=true CARGO_TARGET_X86_64_UNKNOWN_LINUX_GNU_LINKER=x86_64-unknown-linux-gnu-gcc TARGET_CC=x86_64-unknown-linux-gnu-gcc LEPTOS_BIN_TARGET_TRIPLE=x86_64-unknown-linux-gnu cargo leptos build -r
hash.txt gets generated in
target/release/hash.txt
instead of in
target/x86_64-unknown-linux-gnu/release/hash.txt
Originally posted by @robert-king in https://github.com/leptos-rs/cargo-leptos/issues/277#issuecomment-2099469608

@maccesch @benwis  I added a pull request here
https://github.com/leptos-rs/cargo-leptos/pull/282


STEP-22: Create Dockerfiles and "docker-compose.yml"

Create Dockerfiles for all modules and "docker-compose.yml" for project
Acceptance criteria:

Dockerfile for Compensation device service created.
Dockerfile for Full information service created.
Dockerfile for Power transformers service created.
Dockerfile for High volt cables-service created.
Dockerfile for Lighting information service created.
Dockerfile for Protective equipment service created.
Dockerfile for Start information service created.
Dockerfile for Config service created.
Dockerfile for Gateway created.
Dockerfile for Registry  created.
"docker-compose.yml" for project created.

Implements in https://github.com/EllsaG/power-supply-design/pull/59 pull request.


Note not updating immediately

Discord: archer5405
so from the beginning, I noticed that when you toggle the visibility state through your module it does not update the notes in the client save for when you mouse-over them, the only scenario when the change is registered is when you update the module settings, not the note config.
I wanted to force rerendering of the notes after the changes are made, which should be done normally after you change config but for some reason was not.
I wrote a hook to do that on closing the note config, but then again, it only registers the change for the client where the config is changed.
so something is off.

The flag "refreshVisibility" and "refreshText" need to be set in the NoteConfig call - or just set the generic "refresh" request from RENDER_FLAGS
note.renderFlags.set({redraw: true})
Maybe override Note#_onUpdate to also set corresponding flags.

Fixed in 2.4.3


feat(web): cases search by disputeId

Hello, I have fixed the issue of https://github.com/kleros/kleros-v2/issues/689 by adding a fetch request every time a user asks for a particular case by entering the case id
I achieve this by creating a new file called useCasesQueryById which will request the case id. After that, I pass the obtained data from the "useCasesQueryById" hook to the CasesDisplay Component to show the case.
https://user-images.githubusercontent.com/76466509/232862035-4a95527e-a43e-4434-9540-e464a9e6e23f.mp4

It was implemented separately but thanks for the PR anyway.


Update README.md

Thank you for your pull request.
Since I'd like to basically follow the policy of the original repository, please allow me to consider merging this when a contribution guideline is added there.
https://github.com/Stability-AI/stable-audio-tools


Update PyPi package and homepage

I saw the PyPi page for this project (https://pypi.org/project/confluence/) still points to the bitbucket original repo, which has been deleted. A new PyPi release would be great.

Sorry but check https://github.com/pycontribs/confluence/issues/7 - given the number of people willing to takeover maintenance of this project, I would say a new release will never happen. If you are interested into taking over maintenance, fixing CI, this might change. I can provide examples and hints but some legwork is needed.


adding support for bitrig

(cherry picked from commit 970fcfbca5dffda921394172c7298d274eebfd0e)
Conflicts:
configure.ac

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see CONTRIBUTING.md for more information.

oops, nvrmnd, already landed I guess.

@alexcrichton Wait, the change landed in dev/master.  Yes, I do want to cherry pick this.

@alexcrichton I'm trying to get my jemalloc bitrig changes into the current rust build.  Even though we've already merged in my changes in the dev branch, this change needs to land here also.

Thanks!


Write widget and property documentation directly in markdown?

Most of our documentation can be browsed in github. Only the reference documentation for widgets is not available, because it is generated from JSON files at build time. It would be nice to have the entire documentation available in github, without an additional build step.
Moreover, the formal JSON documentation didn't prove as useful as we assumed:

JSON does not allow for longer paragraphs, we'd have to include an extra README file per widget.
Editing description texts in JSON is tedious, it would be easier to edit markdown directly.
The generator is a fairly complex piece of code we could get rid of.
The formal spec describes the API, not the protocol interface–both are quite different. It can neither be used for client implementations nor for TCKs. This narrows the usefulness of the JSON.

On the downside, we could not include derived properties in a manually written documentation.

@ralfstx I would like to close this. A machine-readable description is essential for TypeScript support, and possibly declarative UI. It also allows us to make small changes to the generator with huge effects on the documentation. If anything we could try to create the documentation from JsDoc/TsDoc, like this: https://www.npmjs.com/package/tsdoc

I agree, things have changed and the formal API definition serves us well now.
Now that our code mostly has a class-based structure, JsDoc/TsDoc have become an option.
Another option could be converting the JSON files to YML. That would help to edit longer paragraphs and can probably be done very quickly...

+1 for TsDoc and the editor integration it would allow with DefinelyTyped


When running more than one project, since 1.7.1, need to ddev remove, ddev config, ddev import-db each time

When running more than one project, since 1.7.1, need to ddev remove, ddev config, ddev import-db each time. Previously, could just run ddev start, but now doing that results in the below error.
ddev start
Starting align-platform...
Running Command  Command=ip address show dev docker0
Recreating ddev-align-platform-db ... error

ERROR: for ddev-align-platform-db  Cannot create container for service db: Duplicate mount point: /var/lib/mysql

ERROR: for db  Cannot create container for service db: Duplicate mount point: /var/lib/mysql
Encountered errors while bringing up the project.
Failed to start align-platform: Failed to run docker-compose [-f /home/scott/www/align-platform/.ddev/docker-compose.yaml up -d], err='exit status 1', stdout='', stderr='RRecreating ddev-align-platform-db ... error

ERROR: for ddev-align-platform-db  Cannot create container for service db: Duplicate mount point: /var/lib/mysql

ERROR: for db  Cannot create container for service db: Duplicate mount point: /var/lib/mysql
Encountered errors while bringing up the project.'

Config file
APIVersion: v1.7.1
name: align-platform
type: drupal8
docroot: web
php_version: "7.2"
webserver_type: nginx-fpm
router_http_port: "80"
router_https_port: "443"
xdebug_enabled: false
additional_hostnames: []
additional_fqdns: []
mariadb_version: "10.2"
webcache_enabled: false
nfs_mount_enabled: false
provider: default

OS information:
Ubuntu 18.04
ddev version
bgsync        	drud/ddev-bgsync:v1.7.0
cli           	v1.7.1
commit        	v1.7.1
db            	drud/ddev-dbserver:v1.7.0-10.2
dba           	drud/phpmyadmin:v1.7.0
ddev-ssh-agent	drud/ddev-ssh-agent:v1.7.0
docker        	18.09.6
docker-compose	1.22.0
domain        	ddev.local
router        	drud/ddev-router:v1.7.0
web           	drud/ddev-webserver:v1.7.0

Hope someone is able to help!

Thanks so much for your report. I take it really seriously; somebody else reported similar behavior (apparent shared db volume), but I'm completely unable to recreate it, and know of no recent changes that would affect it.

Is it possible that you have two projects named "align-platform" (looking at the 'name' entry in config.yaml? Names of projects must definitely be unique. Please do provide the config.yaml for the both projects that you're trying to run at the same time.
If you could, please try creating two very simple projects, perhaps named test1 and test2, just ddev config them and try to run them at the same time. I think you'll find they work ok.
Please provide the output of docker volume ls | grep mariadb
Could you please provide (probably as a gist or other link) the output of docker inspect --format='{{json .Mounts}}'  ddev-align-platform-db?  You'll see the mount and the source. If the source is the same between the two projects, we definitely have a problem.

Thanks for your help on this, I sure hope we can nail it.

Oh, are you completely sure that you have to import-db to solve the problem? Perhaps you were just using "start" and "stop" and have the problem described in https://github.com/drud/ddev/issues/1541 ? In that case, ddev rm and ddev start will work just fine, but ddev stop and ddev start (or ddev restart) result in a problem like yours. This behavior is fixed in upcoming v1.8.0.

Ah great! I wrongly assumed ddev rm would remove the database since the container was removed. Will do ddev rm && ddev start when we encounter this issue for now then. Thank you for the quick reply!

I'm also experiencing this with 1.7.1
Wasn't sure what the problem was but suddenly all project started reporting this (I use ddev for multiple projects at once)
ERROR: for 57ebb448590a_PROJECT-db  Cannot create container for service db: Duplicate mount point: /var/lib/mysql

ERROR: for db  Cannot create container for service db: Duplicate mount point: /var/lib/mysql

Before 1.7.1 I wasn't experiencing this.
Will try with v1.8.0-rc1 if this is still the case.

@alesrebec does ddev rm && ddev start fix this for you? I'd sure love to understand what's really going on here.

Will try the next time it happens to me.

Ok, I stayed on 1.7.1 for that reason.
I know how to reproduce it now.
If I stop the project with "ddev stop" and then start it again with "ddev start" I get this error Cannot create container for service db: Duplicate mount point: /var/lib/mysql
Running ddev rm && ddev start does solve it and db data persists.
Probably this is solved already in 1.8.0 @rfay?

That problem was in fact solved in 1.8.0, assuming you have custom mysql settings, https://github.com/drud/ddev/issues/1541; anyway, pretty sure you won't see that any more in 1.8.0

I can confirm.
It's fixed in 1.8.0.

Thanks so much!


There's no sendHubCommand() method. Sync DeviceExecutor with dumped methods.

https://github.com/biocomp/hubitat_ci/blob/74e5109aaedd35f544df76201a3d10e7893dfc64/src/main/groovy/me/biocomp/hubitat_ci/api/device_api/DeviceExecutor.groovy#L336
Methods in DeviceExecutorMethods.txt are up to date, but DeviceExecutor.groovy is not completely in sync with them.

Fixed


[tool_crash] FileSystemException: Cannot copy file to 'C:\Users\estep\Documents\UFMA\Search\Codes\teste2\android\app\src\main\res\mipmap-mdpi\ic_launcher.png', OS Error: Access is denied., errno = 5

Command
flutter create --org br.com.Nascimento teste2

Steps to Reproduce

...
...
...

Logs
FileSystemException: Cannot copy file to 'C:\Users\estep\Documents\UFMA\Search\Codes\teste2\android\app\src\main\res\mipmap-mdpi\ic_launcher.png', OS Error: Access is denied.
, errno = 5
#0      _File.throwIfError (dart:io/file_impl.dart:635:7)
#1      _File.copySync (dart:io/file_impl.dart:340:5)
#2      ForwardingFile.copySync (package:file/src/forwarding/forwarding_file.dart:30:50)
#3      ForwardingFile.copySync (package:file/src/forwarding/forwarding_file.dart:30:50)
#4      Template.render.<anonymous closure> (package:flutter_tools/src/template.dart:267:18)
#5      _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:377:8)
#6      Template.render (package:flutter_tools/src/template.dart:190:24)
#7      CreateCommand._renderTemplate (package:flutter_tools/src/commands/create.dart:799:21)
#8      _rootRunUnary (dart:async/zone.dart:1198:47)
#9      _CustomZone.runUnary (dart:async/zone.dart:1100:19)
#10     _FutureListener.handleValue (dart:async/future_impl.dart:143:18)
#11     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:696:45)
#12     Future._propagateToListeners (dart:async/future_impl.dart:725:32)
#13     Future._completeWithValue (dart:async/future_impl.dart:529:5)
#14     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:40:15)
#15     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:311:13)
#16     Template.fromName (package:flutter_tools/src/template.dart)
#17     _rootRunUnary (dart:async/zone.dart:1198:47)
#18     _CustomZone.runUnary (dart:async/zone.dart:1100:19)
#19     _FutureListener.handleValue (dart:async/future_impl.dart:143:18)
#20     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:696:45)
#21     Future._propagateToListeners (dart:async/future_impl.dart:725:32)
#22     Future._completeWithValue (dart:async/future_impl.dart:529:5)
#23     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:40:15)
#24     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:311:13)

[✓] Flutter (Channel stable, 1.22.3, on Microsoft Windows [Version 10.0.18363.1139], locale
    pt-BR)
    • Flutter version 1.22.3 at C:\src\flutter
    • Framework revision 8874f21e79 (8 days ago), 2020-10-29 14:14:35 -0700
    • Engine revision a1440ca392
    • Dart version 2.10.3

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for
      detailed instructions).
      If the Android SDK has been installed to a custom location, set ANDROID_SDK_ROOT to
      that location.
      You may also want to add it to your PATH environment variable.

[!] Android Studio (not installed)
    • Android Studio not found; download from
      https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for
      detailed instructions).

[✓] VS Code, 64-bit edition (version 1.50.1)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.16.0

[!] Connected device
    ! No devices available

! Doctor found issues in 3 categories.

Flutter Application Metadata
No pubspec in working directory.

Closing as duplicate of https://github.com/flutter/flutter/issues/69116 https://github.com/flutter/flutter/issues/69825


SelectNodes from a node issue

hi
once i try to apply an xpath expression on a node result of a previous query -
it seems as it ignores the hierarchy and it does the search for the entire scope and NOT solely the  node scope as expected
code sample:
var doc = web.Load("someURL");
var nodes = doc.DocumentNode.SelectNodes("//div[@data-item");
foreach (var node in nodes)
var innerNodes = node.SelectNodes("//a[contains(@class, 'someClass')]");

i found a way around that for now:
var doc = web.Load("someURL");
var nodes = doc.DocumentNode.SelectNodes("//div[@data-item");
var documents = nodesRoot.Select(x => {
var temp = new HtmlDocument();
temp.LoadHtml(x.OuterHtml);
return temp;
});
foreach (var document in documents) {
var innerNodes = document.DocumentNode.SelectNodes("//a[contains(@Class, 'someClass')]");
....
}

Hello @RcBuilder ,
This is a command issue.
When using // from an existing node, it searches from the document root and not the current node. Which is exactly the behavior you are getting.
You must use only one slashe:
foreach (var node in nodes)
var innerNodes = node.SelectNodes("/a[contains(@Class, 'someClass')]");

Let me know if that's working with this solution.
Best Regards,
Jonathan

hi, thanks for the quick reply..
your solution fit to cases which you know the full path, i didn't want to provide the full path but skip part of it.
e.g: lets assume that the "item" div is an inner node

1
                2

i want to use the following xpath:
//a
instead of
/div/div/a
thanks, Roby (-:

I believe you only need to use the ./ instead of / operator then.
Something like this:
foreach (var node in nodes)
var innerNodes = node.SelectNodes("./a[contains(@Class, 'someClass')]");

Best Regards,
Jonathan

sure, i'll try that, thanks (-:

it doesn't work.
var d = new HtmlDocument();
d.LoadHtml(
"" +
"" +
"" +
"" +
"1" +
"2" +
"" +
"" +
"" +
"" +
"" +
"" +
"3" +
"4" +
"" +
"" +
"" +
"");
// ns2 is NULL (./a)
var ns1 = d.DocumentNode.SelectNodes("//div[@data-item]");
foreach (var n1 in ns1) {
var ns2 = n1.SelectNodes("./a");
foreach (var n2 in ns2)
Console.WriteLine(n2.InnerText);
}
// WORKS with full path! (div/div/a)
var ns1 = d.DocumentNode.SelectNodes("//div[@data-item]");
foreach (var n1 in ns1) {
var ns2 = n1.SelectNodes("div/div/a");
foreach (var n2 in ns2)
Console.WriteLine(n2.InnerText);
}

We will try to add some tutorial about it soon,
I get sometime lost myself ;(
I tried this code and it's working fine this time:
var ns1 = d.DocumentNode.SelectNodes("//div[@data-item]");
foreach (var n1 in ns1)
{
	var ns2 = n1.SelectNodes(".//a").ToList();

}

Best Regards,
Jonathan

[SOLVED]
works like charm!
many thanks mate, great service!
you can close this ticket

Great ;)


feat: register moonbeam, moonriver, and moonbase etherscan urls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
220 unchanged lines in 9 files lost coverage.
Overall coverage decreased (-5.7%) to 84.263%

Files with Coverage Reduction
New Missed Lines
%

gnosis/eth/typing.py
2
83.33%

gnosis/protocol/gnosis_protocol_api.py
2
88.64%

gnosis/safe/api/relay_service_api.py
2
52.27%

gnosis/safe/proxy_factory.py
3
95.24%

gnosis/util/util.py
5
61.54%

gnosis/eth/multicall.py
9
88.35%

gnosis/eth/oracles/uniswap_v3.py
23
57.81%

gnosis/eth/ethereum_client.py
39
81.66%

gnosis/eth/oracles/oracles.py
135
50.7%

Totals

Change from base Build<PHONE_NUMBER>:
-5.7%

Covered Lines:
3202

Relevant Lines:
3800

💛  - Coveralls

Hi @the-amazing-ultraton , you need to rebase master branch first and fix conflicts.
I'm curious, why are you opening all these PRs for multiple networks?


CTRL-C does not kill cross

Checklist

[X] I've looked through the issues and pull requests for similar reports

Describe your issue
Hitting CTRL-C on my macbook fails to kill a cross build.

[X] macOS
[ ] Windows
[ ] Linux / BSD
[ ] other OS (specify in description)

What architecture is the host?

[ ] x86_64 / AMD64
[ ] arm32
[X] arm64 (including Mac M1)

What container engine is cross using?

[X] docker
[ ] podman
[ ] other container engine (specify in description)

cross version
cross 0.2.4,  cargo 1.66.0-nightly (73ba3f35e 2022-09-18)
Example
cross build --target x86_64-unknown-linux-gnu

Additional information / notes
Everything works correctly when the aarch64-apple-darwin (host) target is used instead

The host target aarch64-apple-darwin falls back to the host cargo, and is not run inside a container, so this wouldn't be cross's domain. The Ctrl+C event not being captured might be my fault....
https://github.com/cross-rs/cross/pull/964

So this still fails if a process is spawned in the background. A quick test is:
$ cross build --target armv7-unknown-linux-gnueabi &
$ jobs
[1]+  Stopped                 cross build --target armv7-unknown-linux-gnueabi
[1]+  Stopped                 cross build --target armv7-unknown-linux-gnueabi
$ fg
cross build --target armv7-unknown-linux-gnueabi
    Finished dev [unoptimized + debuginfo] target(s) in 0.16s
$ jobs

There might be a better way than adding -i for an interactive prompt so we can both kill the docker container if a signal is sent to cross and propagate that to the container. The termination handlers we added might be useful here.

I still encounter this problem when my cross runner opens an ssh session to run remotely. It's really annoying to restart the terminal everytime. I tried to google around on how to pass ctrl+c to the underlying container but cannot figure this out. Anything obvious I could do to pass SIGTERM
./remote-runner
#!/bin/bash

local_path=$1
shift
name=$(basename $local_path)
remote_path="/tmp/$name"
ssh_flags="-o \"UserKnownHostsFile=/dev/null\" -o \"StrictHostKeyChecking no\" -o \"LogLevel ERROR\""

transfer_command="rsync $local_path $CARGO_TARGET_HOSTNAME:$remote_path -e 'ssh $ssh_flags'"
execute_command="ssh -t $ssh_flags $CARGO_TARGET_HOSTNAME 'RUST_BACKTRACE=$RUST_BACKTRACE RUST_LOG=$RUST_LOG $remote_path $@'"

eval $transfer_command && \
eval $execute_command

Cross.toml
[build]
default-target = "aarch64-unknown-linux-gnu"

[target.aarch64-unknown-linux-gnu]
dockerfile = "./dockerfile"

[target.aarch64-unknown-linux-gnu.env]
passthrough = [
    "CARGO_TARGET_AARCH64_UNKNOWN_LINUX_GNU_RUNNER=./remote-runner",
<EMAIL_ADDRESS>    "RUST_BACKTRACE=1",
    "RUST_LOG=trace"
]

If I try to use my runner outside cross/docker container it captures ctrl+c fine

This misbehavior is happening again with cross v0.2.5.
If the user sends SIGINT (e.g. Ctrl+C on macOS) to cross just before cross emits any output, then cross enters a bad program state and stalls indefinitely. The Docker processes continue to spin their wheels as well.
Recommendations for a workaround: Use a portable signal handler like the ctrlc crate; use an AtomicBool to communicate whether the user has requested early termination or not; avoid spawning child processes in the event of early termination; and ensure that all child processes have a wait method applied.
https://crates.io/crates/ctrlc

the pr for the fix is not included in 0.2.5, it's expected that this is still an issue in 0.2.5

Update:
cross now exits mostly cleanly when I send SIGINT / Ctrl+C to my wrapping CLI app. No explicit signal handler involved.
However, two caveats remain.

I have to setup the stdout and stderr cross process streams as piped, rather than the default inherited configuration.
cross still leaves behind Docker containers when cross is killed.


Move Microsoft.WinGet.Client E2E test to Pester framework

This PR finally uses all the refactor to run the Microsoft.WinGet.Client PowerShell module in Pester. The tests are done in the BuildPowerShellModule phase.
The E2E Test Source is built in the build phases, then it will be get published as artifacts under E2ETests\TestLocalIndex'. Other artifacts that are required as published in E2ETests` as well. Having the test source as a build artifacts will make it easier to perform repros without building the source all over again.
The BuildPowerShellModule phase will download the artifacts, move necessary files to around, install the dev package (and dependencies), start the localhost web server and run the Pester tests. Then it will publish the modules as it used to as well as the test results.
It is important to remember the modules published as artifacts target the dev package, so one cannot just download it and use it without it.
I port all the E2E tests to Pester tests except the ones dealing with the server shutdown.
Eventually we will run tests for Microsoft.WinGet.DSC and Microsoft.WinGet.Configuration.
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-cli/pull/3503&drop=dogfoodAlpha

/azp run

/azp run


Rearrange the debian/linux instructions, better guidance

Looking at the current debian/linux instructions, I still get the sense that people will get a second-rate outcome compared to OSX and Windows.
Some of this might be because the instructions start with the suggestion to use Docker, which adds a pile of complexity to just try things out.  Another aspect is that people are required to build libuv.  Also, rather than starting their own projects, people are encouraged to clone the home repository to see some demos.
What might be nicer is:

Show the flow to install to the local system
Add sources to NuGet.config automatically, not manually
Expand on what to do with DNVM once it's installed
Show how to initialize a new, empty MVC project command-line
Encourage people to install Visual Studio Code for extra convenience
Link to an exact starting point in documentation (once available?) for people to continue their learning path

We should try to not get stuck in the flow of thinking that Linux == Docker and move all those instructions to a different page.
Unless Visual Studio Code is planning on integrating support for running all the tooling through a container and managing the filesystem mounts.  Most people are going to need working copies of everything, local on their path.

Sounds very reasonable to me. From #774 it appears that you can submit suggestions and PR's to https://github.com/aspnet/Docs/ and I guess the instructions in this project are to be replaced eventually.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain spinthemoose.com has exceeded the max emails per hour (228/200 (113%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from github-smtp2-ext8.iad.github.net ([<IP_ADDRESS>]:50206 helo=github-smtp2a-ext-cp1-prd.iad.github.net)
by cp1164.blacksun.ca with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.89_1)
(envelope-from<EMAIL_ADDRESS>id 1eVkHc-0001Md-Um
for<EMAIL_ADDRESS>Sun, 31 Dec 2017 14:32:37 -0600
Date: Sun, 31 Dec 2017 12:32:16 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1514752336;
bh=dWugifOe9TQIOfugjRnEtBpzv/i849cT0tjz+YcyIQU=;
h=From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID:
List-Archive:List-Post:List-Unsubscribe:From;
b=V7UUf2F2sSOLre9hj/8B0PEEGbedt3gOMHOoFfMGQemHJsVAvQweW4eIUSwBw2+Gq
xb1OQHM4w5sGsIleRpXSB9hrSOSfalSY0qzRQBzXKDsXBSvTgXhe2ByaIkwLooBBcw
3CuN/xYi+QQatx0FLYHdNhy7N94uhmTdEVtuS5LI=
From: "ASP.NET Hello Bot"<EMAIL_ADDRESS>Reply-To: aspnet/Home<EMAIL_ADDRESS>To: aspnet/Home<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [aspnet/Home] Rearrange the debian/linux instructions, better
guidance (#832)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5a4949508366_15e523f967d01cf2c8959cc";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: aspnet-hello
X-GitHub-Recipient: davidalpert
X-GitHub-Reason: subscribed
List-ID: aspnet/Home <Home.aspnet.github.com>
List-Archive: https://github.com/aspnet/Home
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AACpRNNZXxfnq9pgukxf7cQcBsXGJyk5ks5tF-9QgaJpZM4FrnjQ
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>----==_mimepart_5a4949508366_15e523f967d01cf2c8959cc
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #832.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/aspnet/Home/issues/832#event-1405408678
----==_mimepart_5a4949508366_15e523f967d01cf2c8959cc
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #832.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_5a4949508366_15e523f967d01cf2c8959cc--

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain spinthemoose.com has exceeded the max emails per hour (229/200 (114%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from o4.sgmail.github.com ([<IP_ADDRESS>]:57710)
by cp1164.blacksun.ca with esmtps (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.89_1)
(envelope-from<EMAIL_ADDRESS>id 1eVkHd-0001Me-Cf
for<EMAIL_ADDRESS>Sun, 31 Dec 2017 14:32:37 -0600
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com;
h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=YrHQbYAWdrXSb76YrSfP+kwrOE0=; b=NdOucZLzRk17z8th
4XbBfozU8hYghePPtPCqTpsJwL5cQEG7CHzFZIs0JMXR/PTWbLD6emAWSmxInT2D
DvVixXyvawO++P55f/xRMVZRtdgSB/I0qcYqC0SSMAl4ksZZAjoIN9d/azmJAJ0j
D38c/AqOQTi514WgqpAdpJiY+h8=
Received: by filter0242p1las1.sendgrid.net with SMTP id filter0242p1las1-24548-5A494950-12
2017-12-31 20:32:16.338483014 +0000 UTC
Received: from github-smtp2b-ext-cp1-prd.iad.github.net (github-smtp2b-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0009p1iad1.sendgrid.net (SG) with ESMTP id vZlEkqgmSPC6uRejjNbm4g
for<EMAIL_ADDRESS>Sun, 31 Dec 2017 20:32:16.162 +0000 (UTC)
Date: Sun, 31 Dec 2017 20:32:16 +0000 (UTC)
From: "ASP.NET Hello Bot"<EMAIL_ADDRESS>Reply-To: aspnet/Home<EMAIL_ADDRESS>To: aspnet/Home<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [aspnet/Home] Rearrange the debian/linux instructions, better
guidance (#832)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5a494950c27d_44c73fb62e086f2850569fc";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: aspnet-hello
X-GitHub-Recipient: davidalpert
X-GitHub-Reason: subscribed
List-ID: aspnet/Home <Home.aspnet.github.com>
List-Archive: https://github.com/aspnet/Home
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AACpRNNZXxfnq9pgukxf7cQcBsXGJyk5ks5tF-9QgaJpZM4FrnjQ
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: rlj3t9ahcX0BZUzAOlajQ3VmBXo43xFJc4M1ImIDJhHitWKEiKTn5zE3n/BwppV0HrR8tvljDmbYFd
n2xixyftlqRdbP8RHiKtpuS9QXDgS3YBHyxxJyPrMmxTCA2Lv0kXx/gM801TXaCJvUEJXAx+SDKeud
JsTcSi7i4tqtun7s0QppM8YDte05owkjKJNK7XTZHcm09l0T9N0r0Lurr9A397u8iN53w8AwblqpTL
Y=
----==_mimepart_5a494950c27d_44c73fb62e086f2850569fc
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/aspnet/Home/issues/832#issuecomment-354622604
----==_mimepart_5a494950c27d_44c73fb62e086f2850569fc
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: quoted-printable
This issue is being closed because it has not been updated in 3 months.<=
/p>
We apologize if this causes any inconvenience. We ask that if you are st=
ill encountering this issue, please log a new issue with updated informatio=
n and we will investigate.
&mda=
sh;You are receiving this because you are subscribed to this thread.<=
br />Reply to this email directly, view it on GitHub, or mute the thread.

=
----==_mimepart_5a494950c27d_44c73fb62e086f2850569fc--


chore(dependabot): auto-update lockfiles for monorepo package bumps

SUMMARY

When dependabot bumps packages for monorepo packages/plugins, it doesn't work out. It bumps package.json, but it's not smart enough to know that it needs to run npm install from the root of the superset-frontend folder. This action will keep an eye out for these PRs from dependabot on those paths, and run the installation command, and push the package lock file.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

I approved the change I submitted 😄


db backup mgmt

https://github.com/aptible/app-ui/assets/1940365/e243dbf7-6ae1-45a0-bb54-67d2eb18241d
https://github.com/aptible/app-ui/assets/1940365/c91f80ee-b21c-471b-b33f-42c30bffb05b

nit but the vertical scrolling on tables is a bit weird (what i saw in the video). i know it's not scope of PR but just pointing it out

Style Tweaks


Add showYearAndMonth method to d2l-calendar.

GAUD-6912
This PR adds a new showYearAndMonth(year, month) method to d2l-calendar. This method is a programmatic way to change the year and month like if the user clicked next/previous month buttons. This will be needed by Calendar MVC tool when using the iterator buttons in "Month" view.

Apologies. I've got this working in MVC but it feels super hacky since I'm jumping through hoops to avoid infinite loop. I'm going to put up a new PR for a different approach - what I really want is a property to control the initial year/month in view if provided.


Update ExpanDrive to 5.1.4

Thank you for the contribution. It was merged directly as c9cb428fd9890d7004f082d612eefedaf60becdb to keep commit history cleaner. Your contribution is still credited to you.


Remove ‘plan-b’

The package is deprecated and no longer supported. Please merge when we have zip-1.0.0, older versions of zip depend on ‘plan-b’.
Checklist: not applicable.

We can merge now, plan-b will automatically be pulled in by the current zip until we upgrade its version.


EHR retrieval and query with GenAI models

This application presents a framework including multiple component services to integrate with FHIR service to retrieve patient EHR and a RAG pipeline to generate insights with the use of Embedding and LLM models.

FHIR application. This is a client of FHIR service while at the same time a server to the RAG pipeline for on demand patient EHR retrieval. It authenticates with OAuth2 for bear token, which is used when getting and posting FHIR resources from/to a FHIR server identified by its URL
RAG pipeline. It contains a RAG agent for building vector database with retrieved patient EHR resources, and a query agent answering end users' query. A Web UI is used and supports speech to text as well as text to speech.

The FHIR application can be run multiple ways:

as a HoloHub app container, build and run with a single dev_container CLI command
built and run in a HoloHub Dev container as console app
built and run as console app in dev env without building container
packaged and run as a Holoscan App Package (container), suitable for image distribution and deployment in "production" environment

The RAG pipeline application is only tested for running in HoloHub app container with the help of a start-up shell script, started manually. The whole process can potentially be automated with the dev_container build and run CLI command with the start-up script being the entrypoint of the app container.

Given the scope of these changes, it would be good to have some level of CTest unit test coverage to integrate with nightly dashboards. See Endoscopy Tool Tracking CMakeLists.txt for an example.

There are a few large JSON files in this PR. Would it be possible to reduce those contents to <1MB, provide generation instructions, or even distribute them ourselves rather than adding to commit history?

There are a few large JSON files in this PR. Would it be possible to reduce those contents to <1MB, provide generation instructions, or even distribute them ourselves rather than adding to commit history?

We should put the data on NGC.

There are a few large JSON files in this PR. Would it be possible to reduce those contents to <1MB, provide generation instructions, or even distribute them ourselves rather than adding to commit history?

We should put the data on NGC.
thank you for the suggestion @tbirdso @jjomier - do you think this NGC upload can be done in a subsequent PR or do you feel strongly about making the change in this PR?

There are a few large JSON files in this PR. Would it be possible to reduce those contents to <1MB, provide generation instructions, or even distribute them ourselves rather than adding to commit history?

We should put the data on NGC.
thank you for the suggestion @tbirdso @jjomier - do you think this NGC upload can be done in a subsequent PR or do you feel strongly about making the change in this PR?

It defeats the purpose if we let in the files on first merge, and we had made the decision.
The JSON files used for locally populating vector DB is not really needed (used as a safety net for the GTC demo), so it will be removed from the PR.
The FHIR resource bundle for testing resource sanitization will be downloaded from either NGC or the original source Synthea.
I am handling this.

There are a few large JSON files in this PR. Would it be possible to reduce those contents to <1MB, provide generation instructions, or even distribute them ourselves rather than adding to commit history?

We should put the data on NGC.
thank you for the suggestion @tbirdso @jjomier - do you think this NGC upload can be done in a subsequent PR or do you feel strongly about making the change in this PR?

It defeats the purpose if we let in the files on first merge, and we had made the decision. The JSON files used for locally populating vector DB is not really needed (used as a safety net for the GTC demo), so it will be removed from the PR. The FHIR resource bundle for testing resource sanitization will be downloaded from either NGC or the original source Synthea. I am handling this.

thank you for the reply @MMelQin . I was not aware a decision has been made one way or the other as I saw no comment addressing this issue in the PR nor any other communication regarding this in Slack/e-mail.
RE: "The JSON files used for locally populating vector DB is not really needed (used as a safety net for the GTC demo),"
are you referring to ehr_data.json file?If so, that is not precisely a safety net, that is the result of running the script create_ehr_database_local and have it as part of the PR.

There are a few large JSON files in this PR. Would it be possible to reduce those contents to <1MB, provide generation instructions, or even distribute them ourselves rather than adding to commit history?

We should put the data on NGC.
thank you for the suggestion @tbirdso @jjomier - do you think this NGC upload can be done in a subsequent PR or do you feel strongly about making the change in this PR?

It defeats the purpose if we let in the files on first merge, and we had made the decision. The JSON files used for locally populating vector DB is not really needed (used as a safety net for the GTC demo), so it will be removed from the PR. The FHIR resource bundle for testing resource sanitization will be downloaded from either NGC or the original source Synthea. I am handling this.

thank you for the reply @MMelQin . I was not aware a decision has been made one way or the other as I saw no comment addressing this issue in the PR nor any other communication regarding this in Slack/e-mail.
RE: "The JSON files used for locally populating vector DB is not really needed (used as a safety net for the GTC demo),"
are you referring to ehr_data.json file?If so, that is not precisely a safety net, that is the result of running the script create_ehr_database_local and have it as part of the PR.

Hate to say it in this venue, but to prevent misconception, @cdinea you are wrong here. create_ehr_database_local consumes that pre-staged json file that contains resource that had been retrieved from the FHIR server (by the create_ehr_database at runtime or using client to drive the FHIR client). the local version was there in case there was no internet connection at GTC venue to the FHIR server hosted internally.
Keeping the json files out of the source repo (even though they could be made smaller) was discussed and decided on our weekly horizonal meeting; you might not have attended the meeting then.
Also, I'd strongly suggest use separate channel to discuss and sync up on the details.

thanks @MMelQin . Indeed I reread my comment and would like to rectify, sent it too soon :)  - get_ehr_data is needed in the create_ehr_database() to create the local database
`# Path to the EHR data as input
EHR_DATA_JSON = "/workspace/holohub/applications/ehr_query_llm/lmm/rag/ehr/ehr_data.json"
Path to the downloaded fine tuned model
EHR_FINETUNED_MODEL = "/workspace/volumes/models/bge-large-ehr-finetune"
CACHE_FOLDER = "/workspace/volumes/models"
Persistent storage folder for the Vector DB
PERSISTENT_FOLDER = "/workspace/holohub/applications/ehr_query_llm/lmm/rag/ehr/db"
def get_ehr_data():
with open(EHR_DATA_JSON) as f:
ehr_data = json.load(f)
return ehr_data`

Given the scope of these changes, it would be good to have some level of CTest unit test coverage to integrate with nightly dashboards. See Endoscopy Tool Tracking CMakeLists.txt for an example.
Will consider adding this in the next phase of this example app, at least for the operators.

There are a few large JSON files in this PR. Would it be possible to reduce those contents to <1MB, provide generation instructions, or even distribute them ourselves rather than adding to commit history?

We should put the data on NGC.

As explained and discussed above, the couple large JSON files were removed from this PR as they were indeed unnecessary. Large JSON file for FHIR Sanitizer module testing can be downloaded from public source (added instructions in code with this commit), and the other one used by the EHR Builder Agent was for a edge case for a demo, but the code was erroneously copied over; fixed with this commit.

There are a few large JSON files in this PR. Would it be possible to reduce those contents to <1MB, provide generation instructions, or even distribute them ourselves rather than adding to commit history?

We should put the data on NGC.

As explained and discussed above, the couple large JSON files were removed from this PR as they were indeed unnecessary. Large JSON file for FHIR Sanitizer module testing can be downloaded from public source (added instructions in code with this commit), and the other one used by the EHR Builder Agent was for a edge case for a demo, but the code was erroneously copied over; fixed with this commit.

Just a quick note, as explained in Slack thread, the code was NOT "erroneously copied over", it was intentionally added to the PR for the DeviceTalks conference to use a local ehr db. The content of the db is the same, either by creating it on demand or having a cached json with the data .  @MMelQin please make sure to include explanations form Slack here for a broader context


[Bug]: Increased cpu usage during build on v4.0.0-beta.379

Error Message and Logs
I have a coolify running on Hetzner Cloud on a single instance.
After the last update to v4.0.0-beta.379 the deployment of the new versions started to take significantly longer without anything dramatic changed in the code. (Sveltekit app)
The build eats all the processor resources to the point coolify stops responding and all the running web apps stop responding as well (Cloudflare throws 524).
Steps to Reproduce

Nothing I can add here
Example Repository URL
No response
Coolify Version
v4.0.0-beta.379
Are you using Coolify Cloud?
No (self-hosted)
Operating System and Version (self-hosted)
Ubuntu 22.04
Additional Information
build_log.txt

I as well am running into a probably same situation. After I install Coolify on a freshly created ubuntu machine, It just stops working. I mean i'm unable to even SSH into my server anymore but the server is up and healthy since I can connect to it via VNC and there's not much CPU/RAM usage going on. I don't know what is going wrong since I've been using Coolify for almost a year now on other projects. Would appreciate your help

I have the same issue as @callmearta
Can't find a working solution

@nikolaistarostin I got it working by following manual installation. Apparently there's an issue in the quick install script. I installed it all manually and it didn't crash or freeze the server this time.


Comments on your JOSS paper review

This documents is part of the review of the paper submitted to the Journal of Open Source Software. See https://github.com/openjournals/joss-reviews/issues/2369.
The paper describes the scikit-fem python package which is dedicated to the assembly of the matrices and vectors resulting from the finite element discretization of partial differential equations. More precisely, the library allows the user to focus on the weak form of the differential equation and the choice of the finite element library (several families are provided).
The paper illustrates the library with three examples, grouped in a single figure:

a linear elastic contact problem.
a H(curl)-conforming model problem.
a resolution of the Navier–Stokes equations for different Reynolds numbers using Taylor–Hood element.

General remarks and questions

The paper is easy to read. I don't have particular remarks on the quality of english used.
The paper is very short. I understand that the task is to library is very narrow. However, many things are missing to fully understand the interest and the limits of the library. For example:

The library do provide a way to handle Dirichlet boundary conditions by elimination. This shall be at least mentioned. I imagine that the treatment of more complex conditions, such as multiple points constraints in mechanics, have to be treated by hand after the assembly, isn't it ?
Following the previous remark, it is not clear how the library could be used in this way. For example, is the mesh connectivity clearly described (how the unknowns are numbered) ?
The authors shall describe how the library could be used in non linear cases. In particular, the contact problem given in the first example is indeed non linear.
Can non uniform coefficients be handled ? To make my question clearer, would I be able to treat:

a linear elastic problem with a non uniform Young Modulus.
a heat transfer equation with a non uniform heat source Those non uniform coefficients are dealt with in UFL using the notion of functions spaces. Is there anything equivalent in scikit-fem ?

The authors claims that the assembly, while made in pure python, is  "fast enough" ? This shall be made more explicit. Do the authors mean that the assembly takes a negligible time compared to the resolution of the resulting linear system ?
The authors claims that more than 30 examples illustrate the use of the library. I did not find any page that summarises those examples. It would be very useful for new users. I mean something similar to what the MFEM developers provide: https://mfem.org/examples/
Would it work in MPI-like processes ? Do you plan any extension of the package in that matter or is it out of its scope ?.

Specific comments
Quadrature points
UFL offers functional spaces at quadrature points. This functional spaces are relevant when dealing with non linear constitutive equations in solid mechanics. Is there anything similar in scikit-fem ?
Figures
With which package have the figures been generated ?
Second example
The description of the second example is very sparse.
Dependencies
Some examples depends on external packages which are not installed with scikit-fem. An appropriate list of dependencies would be appreciated.
review.pdf

Thanks for valuable feedback. Some clarifying questions:

Would it work in MPI-like processes ? Do you plan any extension of the package in that matter or is it out of its scope ?.

It's currently out of scope. Do you ask this to be discussed in the paper?

The description of the second example is very sparse.

Do you mean the following example?
https://kinnala.github.io/scikit-fem-docs/example2.html

UFL offers functional spaces at quadrature points. This functional spaces are relevant when dealing with non linear constitutive equations in solid mechanics. Is there anything similar in scikit-fem ?

I'm not sure I recognize the concept but I have used scikit-fem library to solve, e.g., nonlinear (Neohookean) elasticity. Is this (functional spaces at quadrature points) something related to plasticity?

Would it work in MPI-like processes ? Do you plan any extension of the package in that matter or is it out of its scope ?.

It's currently out of scope. Do you ask this to be discussed in the paper?

It would be nice to have a better view of the limitations and perspectives of the package. But that's only a point of discussion among (many) others.

The description of the second example is very sparse.

Do you mean the following example?
https://kinnala.github.io/scikit-fem-docs/example2.html

No. I mean the second example of the paper: "a H(curl)-conforming model problem."

UFL offers functional spaces at quadrature points. This functional spaces are relevant when dealing with non linear constitutive equations in solid mechanics. Is there anything similar in scikit-fem ?

I'm not sure I recognize the concept but I have used scikit-fem library to solve, e.g., nonlinear (Neohookean) elasticity. Is this (functional spaces at quadrature points) something related to plasticity?

In a way, it is related to plasticity. More precisely, I am the main author of MFront (http://tfel.sourceforge.net/) which is a set of domain specific languages dedicated to advanced constitutive equations (such as plasticity). This explains my interest on this topic.
Advanced constitutive equations describes the evolution of internal state variables (elastic strain, dislocation densities, equivalent plastic strain, back-strains, damage, etc....). One the values of the internal state variables are known, the stresss can be deduced, so the knowledge of those values are required at the quadrature points.

Advanced constitutive equations describes the evolution of internal state variables (elastic strain, dislocation densities, equivalent plastic strain, back-strains, damage, etc....). One the values of the internal state variables are known, the stresss can be deduced, so the knowledge of those values are required at the quadrature points.

I think I found a discussion of the concept in Chapter 26 of https://launchpadlibrarian.net/83776282/fenics-book-2011-10-27-final.pdf
I'll try to read it through and see if there is anything comparable in scikit-fem.

There is now a gallery of examples at https://scikit-fem.readthedocs.io/en/latest/listofexamples.html .

Some examples depends on external packages which are not installed with scikit-fem. An appropriate list of dependencies would be appreciated.

There is now an explicit discussion of installation dependencies and test dependencies (all examples are tested) in the README. Do you find it sufficient?

The authors claims that the assembly, while made in pure python, is "fast enough" ? This shall be made more explicit. Do the authors mean that the assembly takes a negligible time compared to the resolution of the resulting linear system ?

There is now also a simple performance benchmark in the beginning of the README. Do you find it sufficient?

Following the previous remark, it is not clear how the library could be used in this way. For example, is the mesh connectivity clearly described (how the unknowns are numbered) ?

We added documentation on the numbering of the DOF's at https://scikit-fem.readthedocs.io/en/latest/bcs.html

There is now also a simple performance benchmark in the beginning of the README. Do you find it sufficient?

That's fine for me. Thanks

I do appreciate our answers to my remarks and the time you spend on it. Closing this issue which is no more meaningful and start finalizing the review.


Feature/levees

This code handles the remaining levee logic with passing tests. The study data tests take a long time to run, I think we should probably comment them out at some point.

I am going to merge this PR and we can create a new branch for messaging and validation

this test is passing on my machine, failing on GitHub.

@Brennan1994 what do you think could be happening here?


[improve][test] remove powermock-reflect dependency

This closes #16912.
Documentation

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

/pulsarbot run-failure-checks

cc @Shoothzj @merlimat @nicoloboschi

Thanks for your reviews! Could You help with merging this patch? @Shoothzj @nicoloboschi

LGTM but tbh it's a very long pull 👀

As long as we use squash and merge😉
We should try to prevent reintroduce the library in further development randomly. And perhaps it's time to convert the "community" page to a contributor guide, like https://inlong.apache.org/community/how-to-contribute.


Memory Leak

When downloading a zip file, Unity holds onto the memory. Try downloading a 200MB file and you'll see that even after you stop the application in the editor, your memory consumption sky-rockets. If you play it again, it cumulatively adds to the memory.

This plugin just holds byte array for downloaded content(see https://github.com/midworld/unity-googledrive/blob/7be81c0ac570a7588d65ed8cf34ab858613b6c78/Assets/GoogleDrive/UnityWebResponse.cs#L46). It just acquires as 'Content-Length'(or using MemoryStream and dispose it). So if you don't retain the byte array anymore, GC will remove it.
Can I get more information with memory dump on debugger or something?

Thank you for the speedy response. As far as the byte array is concerned, I am not even assigning it to a variable. I am simply logging the size and then setting it to null. I am running the "DriveText.unity" scene within the GoogleDrive -> Test folder.
The only difference is that instead of downloading a ".txt" file, I am downloading a large zip file outside of the AppData folder.
Below is my customized code:
`var listFiles = drive.ListFiles(interactiveFolder);
yield return StartCoroutine(listFiles);
var files = GoogleDrive.GetResult<List<GoogleDrive.File>>(listFiles);
		if (files != null)
		{
			foreach (var file in files)
			{
				Debug.Log(file);

				if (file.Title == "WordWall")
				{
                    Debug.Log("Downloading..");
                    IEnumerator downloadCoroutine = drive.DownloadFile(file);
                    yield return StartCoroutine(downloadCoroutine);

                    byte[] data = GoogleDrive.GetResult<byte[]>(downloadCoroutine);

//                        downloadCoroutine.Reset();
Debug.Log(data.Length);
                    data = null;
                    downloadCoroutine = null;
				}
			}
		}
		else
		{
			Debug.LogError(listFiles.Current);
		}`

Your code has no problem(nullify data and downloadCoroutine). I think GC is not executed. Could you please test run following code after you release the byte array?
System.GC.Collect();
System.GC.WaitForPendingFinalizers();

or changing Unity Scene.
Thanks.

Thank you for the suggestion, I modified the above code to:
if (file.Title == "WordWall")
{
      Debug.Log("Downloading..");
       IEnumerator downloadCoroutine = drive.DownloadFile(file);
       yield return StartCoroutine(downloadCoroutine);

       byte[] data = GoogleDrive.GetResult<byte[]>(downloadCoroutine);
       Debug.Log(data.Length);

        data = null;
        downloadCoroutine = null;

        System.GC.Collect();
        System.GC.WaitForPendingFinalizers();

        yield return new WaitForSeconds(5);

        SceneManager.LoadScene(0);
}

Memory Before Starting program in editor:

Memory After initial start:

Memory after Scene was reloaded:

Memory After Stopping the Application, Starting the application again and waiting for the scene to reload:

Unfortunately, it still seems to have the memory issue.

Oh, I don't know how Unity deal its memory(I think Unity doesn't release the allocated space before system has not sufficient memory same as other VM apps). You can check the real used space on Profiler memory tabs. Please check the Profiler. Thanks.

Oh, you are correct. The memory used by the editor does cap out at about 5.5GB. I was afraid that if the memory continued to increase, it would freeze the application, but that is not the case and the Profiler has a steady 22.2MB total allocation.. Thank you.


ESP32 connection lost between cloud

Hi
We are working on a project in which we are developing a bidirectional communication between a device and a cloud through ioT hub. We have downloaded the sample code that you have provided here. After the first start, things go nicely, the system works without any errors, as expected. But as some time passes, we begin to have the following error message and lose the connection. System recovers itself over some time but the time it takes to recover itself is longer than we like. What would you recommend us to do to eliminate this error message.
The error message is given below
Thanks in advance
`-> 06:30:19 PINGREQ
<- 06:30:19 PINGRESP
-> 06:34:10 PINGREQ
<- 06:34:10 PINGRESP
-> 06:38:01 PINGREQ
<- 06:38:01 PINGRESP
-> 06:41:52 PINGREQ
<- 06:41:52 PINGRESP
-> 06:45:43 PINGREQ
<- 06:45:43 PINGRESP
-> 06:49:34 PINGREQ
<- 06:49:34 PINGRESP
-> 06:53:25 PINGREQ
<- 06:53:25 PINGRESP
-> 06:57:16 PINGREQ
<- 06:57:16 PINGRESP
-> 07:01:07 PINGREQ
<- 07:01:07 PINGRESP
-> 07:04:58 PINGREQ
<- 07:04:58 PINGRESP
-> 07:08:49 PINGREQ
<- 07:08:49 PINGRESP
-> 07:12:40 PINGREQ
<- 07:12:40 PINGRESP
Error: Time:Thu Nov 16 07:14:28 2017 File:C:/msys32/home/Kayten/esp/esp-idf/components/azure-
iot/sdk/c-utility/src/xio.c Func:xio_retrieveoptions Line:280 unable to concrete_io_retrieveoptions
-> 07:14:28 DISCONNECT
Error: Time:Thu Nov 16 07:14:28 2017 File:C:/msys32/home/Kayten/esp/esp-
idf/components/azureiot/pal/src/tlsio_openssl_compact.c Func:tlsio_openssl_destroy Line:207 tlsio_openssl_destroy called while not in TLSIO_STATE_CLOSED.
Error: Time:Thu Nov 16 07:18:29 2017 File:C:/msys32/home/Kayten/esp/esp-idf/components/azure-iot/sdk/iothub_client/src/iothubtransport_mqtt_common.c Func:InitializeConnection Line:1800 mqtt_client timed out waiting for CONNACK
Error: Time:Thu Nov 16 07:18:29 2017 File:C:/msys32/home/Kayten/esp/esp-idf/components/azure-iot/sdk/c-utility/src/xio.c Func:xio_retrieveoptions Line:280 unable to concrete_io_retrieveoptions
Error: Time:Thu Nov 16 07:18:29 2017 File:C:/msys32/home/Kayten/esp/esp-idf/components/azure-iot/pal/src/tlsio_openssl_compact.c Func:tlsio_openssl_send_async Line:476 tlsio_openssl_send_async without a prior successful open
Error: Time:Thu Nov 16 07:18:29 2017 File:C:/msys32/home/Kayten/esp/esp-idf/components/azure-iot/sdk/umqtt/src/mqtt_client.c Func:sendPacketItem Line:332 475: Failure sending control packet data
Error: Time:Thu Nov 16 07:18:29 2017 File:C:/msys32/home/Kayten/esp/esp-idf/components/azureiot/sdk/umqtt/src/mqtt_client.c Func:mqtt_client_disconnect Line:1176
Error: mqtt_client_disconnect send failed
-> 07:18:30 CONNECT | VER: 4 | KEEPALIVE: 240 | FLAGS: 192 | USERNAME: DENEMEHUB.azure-devices.net/cihaz1/api-version=2016-11-14&DeviceClientType=iothubclient%2F1.1.25 (openssl_compact) | PWD: XXXX | CLEAN: 0
<- 07:18:30 CONNACK | SESSION_PRESENT: true | RETURN_CODE: 0x0
-> 07:18:30 SUBSCRIBE | PACKET_ID: 6 | TOPIC_NAME: devices/cihaz1/messages/devicebound/# | QOS: 1
<- 07:18:30 SUBACK | PACKET_ID: 6 | RETURN_CODE: 1
-> 07:22:21 PINGREQ
<- 07:22:21 PINGRESP
-> 07:26:12 PINGREQ
<- 07:26:12 PINGRESP
`

Hi @omerfatih
We are also using the ESP32 and have seen the same error, it happens every ~48 minutes when the token is renewed. Would be nice to have this handled in a cleaner way.
Got some other issues tracked here: https://github.com/Azure/azure-iot-sdk-c/issues/269

The ESP32 tlsio is returning NULL from retrieveoptions, which the SDK interprets as an error instead of "no options available", which may be causing this problem. Fixing this is near the top of my priority list. Thanks for the excellent report!

No, its not fixed

Gönderen: jpagliaccio<EMAIL_ADDRESS>Gönderildi: 14 Kasım 2018 Çarşamba 17:53
Kime: Azure/azure-iot-pal-esp32
Bilgi: omerfatih; Mention
Konu: Re: [Azure/azure-iot-pal-esp32] ESP32 connection lost between cloud (#2)
Having the same issue. Was this fixed?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-iot-pal-esp32/issues/2#issuecomment-438689176, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHy5sCp-gI6F_PeeSFSbYE46dW-_4otEks5uvC59gaJpZM4QgE9K.


Use prop-types instead of React.PropTypes

I can confirm with these changes that this module works with React 16

@fabiancook Thanks for the PR. Cheers!

Following up on this, will there be a push to npm with a new version?

Hi, is there any update here? Thanks

@fabiancook just published with the latest changes on npm. Sorry for the delay.

Thanks, would be good to see the distributed version up on GitHub as well for consistency :).


T-DISPLAY S3 PRO ERROR

Is not compatible with T-Display s3 Pro???

Unfortunately no :(

I look forward to the launcher for this model :)

Lilygo sent me this device last week.. probably I'll have it in hands in a few weeks 😜

Oh Yeaaah!

REleased version 2.4.1 with port to this device!

OMG!!!!!!!
About an hour ago I checked to see if there was an update and there wasn't one. Now is there one? I'll install it tomorrow, I don't have a computer at hand right now.
Thanksssss!!!!!!


Support for keyframes in primitive components

I've been playing around rebuilding the create-react-app sample app using styled-components/primitives, which works flawlessly so far, with the exception being the keyframes.
Looking through issues, I found #397 which suggests this code:
import styledWeb from "styled-components";

const WebLogo = styledWeb.img`
    animation: ${rotationBuilder(10)} 4s linear infinite;
    height: 150px;
    width: 150px;
`;

function rotationBuilder(degree) {
  return keyframes`
    0% {
      transform: rotate(0deg);
    }
    25% {
      transform: rotate(-${degree}deg);
    }
    50% {
      transform: rotate(0deg);
      opacity: .9
    }
    75% {
      transform: rotate(${degree}deg);
    }
  `;
}

Full sample here.
This works beautifully for web, as expected.
However, I'm not able to apply the keyframes to the Image primitive, like this:
import styled from "styled-components/primitives";

const Logo = styled.Image`
  animation: ${rotationBuilder(10)} 4s linear infinite;
`

Full sample here.
The primitive works fine without the animation property, but adding it yields this error:
Uncaught TypeError: allStylePropTypes[prop] is not a function (full stacktrace).
Any chance at getting this to work, possibly utilizing react-native-facebook-keyframes?
It would be a rather nice feature to be able to write cross-platform animations using only styled-components.

I'd rather people just use animated as react-primitives suggests.

I don't think we should add support for anything special here, Animated works perfectly fine on react-native and if you're using /primitives you'll likely also want your animations to work cross-platform, so you should use it there too.
Closing!


Getting an error while processing/georeferenceing .all file

Getting this error when trying to process a .all file. Unlike #79 I can import/convert the file but it is the next processing/georefererencing stage which fails:
****Building tx/rx vectors at time of transmit/receive****
Operating on system serial number = 275
using installation params<PHONE_NUMBER>
Traceback (most recent call last):
  File "/snap/pycharm-community/267/plugins/python-ce/helpers/pydev/_pydevd_bundle/pydevd_exec2.py", line 3, in Exec
    exec(exp, global_vars, local_vars)
  File "<input>", line 1, in <module>
  File "/home/david/kluster/HSTB/kluster/fqpr_convenience.py", line 308, in process_multibeam
    fqpr_inst.get_orientation_vectors(initial_interp=orientation_initial_interpolation, subset_time=subset_time)
  File "/home/david/kluster/HSTB/kluster/fqpr_generation.py", line 2208, in get_orientation_vectors
    self.generate_starter_orientation_vectors(prefixes, timestmp)
  File "/home/david/kluster/HSTB/kluster/fqpr_generation.py", line 623, in generate_starter_orientation_vectors
    rx_heading = abs(float(self.multibeam.xyzrph[txrx[1] + '_h'][tstmp]))
KeyError: 'rx_h'

File info:
FQPR: Fully Qualified Ping Record built by Kluster Processing
-------------------------------------------------------------
Contains:
2 sonar heads, 18000 pings, version 0.8.9
Start: Thu Mar 18 14:50:30 2021 UTC
End: Thu Mar 18 14:55:30 2021 UTC
Minimum Latitude:  <omitted> Maximum Latitude: <omitted>
Minimum Longitude:  <omitted> Maximum Longitude:  <omitted>
Minimum Northing: Unknown Maximum Northing: Unknown
Minimum Easting: Unknown Maximum Easting: Unknown
Minimum Depth: Unknown Maximum Depth: Unknown
Current Status: converted complete
Sonar Model Number: em2040_dual_rx
Primary/Secondary System Serial Number: 275/281
Horizontal Datum: 32630
Vertical Datum: waterline
Navigation Source: Unknown
Contains SBETs: False
Sound Velocity Profiles: 1

Inspecting the self.multibeam.xyzrph dict there are rx_port_h and rx_stbd_h instead of rx_h:
...
SortedDict({'1616079030': 0.001}), 'rx_port_h': SortedDict({'1616079030': 180.33}), 'rx_port_p':
({'1616079030': -1.91}), 'rx_port_r': SortedDict({'1616079030': -35.165}), 'rx_port_x': ({'1616079030': 0.003}),
'rx_port_y': SortedDict({'1616079030': -0.336}), 'rx_port_z': SortedDict({'1616079030': -0.162}), 'rx_stbd_h':
({'1616079030': 182.38}), 'rx_stbd_p': SortedDict({'1616079030': -0.28}), 'rx_stbd_r': ({'1616079030': 33.525}),
 'rx_stbd_x': SortedDict({'1616079030': -0.01}), 'rx_stbd_y': SortedDict({'1616079030': 0.338}), 'rx_stbd_z':
...

OK, I see the issue here.  I'll need to update xarray_conversion.return_xyz_prefixes_for_system.  If my code is working correctly, you have a sonar with a single transmitter, dual receivers.  So the self.multibeam.xyzrph dict has a tx_h and rx_port_h rx_stbd_h?
This should be a simple fix i can put in the next release.  I'll go through the conversion code to see if there are any other instances where this configuration will be an issue.

So the self.multibeam.xyzrph dict has a tx_h and rx_port_h rx_stbd_h?

Yes that appears to be the case. I have managed to get it 'working' by changing return_xyz_prefixes_for_systems to the following ( to be honest though I don't know if this is appropriate or whether something else needs modifying):
if 'tx_r' in self.xyzrph and 'rx_r' in self.xyzrph:
    leverarms = [['tx', 'rx']]
elif 'tx_port_r' in self.xyzrph and 'rx_port_r' in self.xyzrph:
    leverarms = [['tx_port', 'rx_port'], ['tx_stbd', 'rx_stbd']]
elif 'tx_r' in self.xyzrph and 'rx_port_r' in self.xyzrph:
    leverarms = [['tx', 'rx_port'], ['tx', 'rx_stbd']]
else:
    self.logger.error('Not supporting this sonartype yet.')
    raise NotImplementedError('Not supporting this sonartype yet.')

@davesteps Thanks!  This is logic that will be in Kluster 0.8.10.  Technically there is also a dual tx, single rx configuration that is possible from the system's perspective, although I don't think in practice that would make sense?  I might put it in there anyway just as a placeholder and print a warning.
I had to make a few other tweaks that had this incorrect assumption built in, still going through the rest of the code.
https://github.com/noaa-ocs-hydrography/kluster/commits/kluster_0_8_10

@davesteps is this issue resolved in the latest version?  Please let me know if I can close.

Hi @ericgyounkin, I have run that survey again with v0.9.0 and it appears to give a slightly different error now. I'll close this a open a new issue.


Feature: add the ability to recover messages when using batching using Retry Interceptors

Affects Version(s): 2.2.1.RELEASE
Enhancement
When using batching in listeners with a List parameter, if an exception is thrown, there's no way to recover from it using a retry interceptor since
https://github.com/spring-projects/spring-amqp/blob/e6e659afeb4906414fb7e3a4b2add902ae928220/spring-rabbit/src/main/java/org/springframework/amqp/rabbit/config/StatelessRetryOperationsInterceptorFactoryBean.java#L59
throws a ClassCastException.
What I think could be done is to have either some other way to use a retry recoverer or detect the proper argument type (i.e. List of messages instead of a single message) and loop over them, although I'm not sure what would be the best approach

Yes, we should fix this, but bear in mind that "recovering" from a failed delivery of batch is difficult because the framework (and hence the recoverer) has no inherent knowledge which record in the batch failed.

Thanks for this, guys!


dtr.discover finds folders with different state files multiple times

When I have a folder with a Treant and a Sim object then I can discover that folder twice. I haven't played around with data in the two yet but I assume they could then both show to the same data-limbs.
I don't know yet if this is potentially problematic. It can for sure lead to some confusion if people use both sims and normal treants.

@kain88-de this isn't a problem technically, but I agree it's a confusing state for a directory to be in. A directory can contain multiple Treant state files and in this case using e.g. dtr.Treant(<dirname>) will fail with an exception saying it's ambiguous which one to use. For the case where a directory has both a Treant and a Sim state file there is no ambiguity with this kind of usage, so it still works.
discover looks for state files, so it's perfectly happy with multiple state files that live in the same place. I don't recommend doing this, though, since 1) it's hard to see a reason for doing it, and 2) it might not work as one expects and is confusing.

This code works fine
import mdsynthesis as mds
import datreant.core as dtr

mds.Sim('.')
dtr.Treant('.')
dtr.Treant('Sim.uuid.json')

According to your comment on of these should fail. Which one should fail in the end?

I don't recommend doing this, though, since 1) it's hard to see a reason for doing it, and 2) it might not work as one expects and is confusing.

I don't think people will do this on purpose but I'm very sure this will happen in the future since we already have 2 packages to create state files already and more might come in the future. Such an accident can happen and error should be thrown. Ideally discover should also handle it.


Export two sided flag where needed (and leave out where not)

Current centurion model does not have the double sided flag for the coat.

@McNopper yes, the model has been imported in Unity and then exporter using the glTF exporter, that set doubleSided to false. It would make sense to add the flag now. Thanks!


_bytesio._BytesIO(...).readinto(array.array("u", ...) broken

From @ironpythonbot on December 9, 2014 17:38
E:\vslrft\Merlin\Main\Languages\IronPython\Tests>26
Python 2.6.2 (r262:71605, Apr 14 2009, 22:40:02) [MSC v.1500 32 bit (Intel)]
on win32
Type "help", "copyright", "credits" or "license" for more information.

> > > import _bytesio
> > > import array
> > > b = _bytesio._BytesIO(bytearray(b'ab'))
> > > a = array.array("u", u"z")
> > > b.readinto(a)
> > > 2
> > > a
> > > array('u', u'\u6261')
> > > ^Z

E:\vslrft\Merlin\Main\Languages\IronPython\Tests>ipyd
IronPython 2.6 Beta 2 DEBUG (<IP_ADDRESS>) on .NET 2.0.50727.3053
Type "help", "copyright", "credits" or "license" for more information.
import _bytesio
import array
b = _bytesio._BytesIO(bytearray(b'ab'))
a = array.array("u", u"z")
b.readinto(a)
1

a
array('u', u'a')

Work Item Details
Original CodePlex Issue: Issue 24303
Status: Active
Reason Closed:  Unassigned
Assigned to:    Unassigned
Reported on:    Aug 13, 2009 at 12:40 AM
Reported by:    dfugate
Updated on: Feb 22, 2013 at 2:11 AM
Updated by: jdhardy
Test:   _bytesio_test.py
Copied from original issue: IronLanguages/main#739

From @ironpythonbot on December 9, 2014 17:38
On 2009-10-22 07:08:40 UTC, dfugate commented:
Regression not re-enabled and still broken:
0.15s testing test_coverage FAIL (<type 'exceptions.AssertionError'>)
Test test_coverage failed throwing <type 'exceptions.AssertionError'> (expected ['a'], but found ['z'])
... run_test in D:\vsl\Merlin\Main\Bin\Debug\Lib\iptest\assert_util.py line 528
... test_coverage in modules_bytesio_test.py line 311
... AreEqual in D:\vsl\Merlin\Main\Bin\Debug\Lib\iptest\assert_util.py line 208
... Assert in D:\vsl\Merlin\Main\Bin\Debug\Lib\iptest\assert_util.py line 198

From @ironpythonbot on December 9, 2014 17:38
On 2009-11-10 03:35:13 UTC, sborde commented:
Its an undocumented method

This works on the latest code.


Prevent "\t" being interpreted as a tab in compat.py

:+1:

maybe it worth using os.path.join https://docs.python.org/2/library/os.path.html#os.path.join to get rid of the if statement

@dimazest -- this is addressed in #1226 :)

Perhaps we can just use os.path.sep which is much concise to me.

@philiptzou -- As @dimazest suggested, it's a bit more concise and considered standard practice to use os.path.join to ensure cross-os support.
ne_chunker = os.path.join("chunkers", "maxent_ne_chunker")
help_tagsets = os.path.join("help", "tagsets")
treebank_tagger = os.path.join("taggers", "maxent_treebank_pos_tagger")
punkt_tok = os.path.join("tokenizers", "punkt")

This removes the need for checking the platform (i.e., if os.path.sep != '/': or if sys.platform.startswith('win'):). This also removes the need for making each path a raw-string because os.path.join will properly escape the string.

:+1:

Actually the if os.path.sep != '/': is just an optimization judgement.
Removing that line (and fix the indent of following lines) won't affect the
result. :)
Michael Recachinas<EMAIL_ADDRESS>周三19:53写道：

@philiptzou https://github.com/philiptzou -- As @dimazest
https://github.com/dimazest suggested, it's a bit more concise and
considered standard practice to use os.path.join to ensure cross-os
support.
ne_chunker = os.path.join("chunkers", "maxent_ne_chunker")
help_tagsets = os.path.join("help", "tagsets")
treebank_tagger = os.path.join("taggers", "maxent_treebank_pos_tagger")
punkt_tok = os.path.join("tokenizers", "punkt")
This removes the need for checking the platform (i.e., if os.path.sep !=
'/': or if sys.platform.startswith('win'):). This also removes the need
for making each path a raw-string because os.path.join will properly
escape the string.
—
Reply to this email directly or view it on GitHub
https://github.com/nltk/nltk/pull/1245#issuecomment-169541338.

@mrecachinas Actually the if os.path.sep != '/': is just an optimization judgement.
Removing that line (and fix the indent of following lines) won't affect the
result. :)
The thing I don't like in #1226's approachment, is the four new useless variables and four new method calls to os.path.join. This is more clean and pythonic:
_PY3_DATA_UPDATES = [('chukers', 'maxent_ne_chunker'),
                     ('help', 'tagsets'),
                     ('taggers', 'maxent_treebank_pos_tagger'),
                     ('tokenizers', 'punkt')]
_PY3_DATA_UPDATES = [os.path.join(*args) for args in _PY3_DATA_UPDATES]

But I still prefer the replacing way since it is faster than concating strings.

@philiptzou --

Actually the if os.path.sep != '/': is just an optimization judgement. Removing that line (and fix the indent of following lines) won't affect the result.

Understood. I'm only suggesting that the more idiomatic solution is to use os.path.join rather than replace all instances of / with os.path.sep (see http://stackoverflow.com/questions/16789714/differences-between-use-of-os-path-join-and-os-sep-concatenation).

But I still prefer the replacing way since it is faster than concating strings.

I agree with the assessment that replace is faster than os.path.join. To compare the times to see exactly how much faster replace is, I wrote a quick script to time both:
from __future__ import division
import os.path
import time

num_runs = 1000000
t1 = 0
t2 = 0
join_tot = 0
for i in xrange(num_runs):
    t1 = time.time()

    DATA_UPDATES = [("chunkers", "maxent_ne_chunker"),
                    ("help", "tagsets"),
                    ("taggers", "maxent_treebank_pos_tagger"),
                    ("tokenizers", "punkt")]

    _PY3_DATA_UPDATES = [os.path.join(*path_list) for path_list in DATA_UPDATES]

    t2 = time.time()
    join_tot += (t2 - t1)

replace_tot = 0
for i in xrange(num_runs):
    t1 = time.time()

    _PY3_DATA_UPDATES = ["chunkers/maxent_ne_chunker",
                         "help/tagsets",
                         "taggers/maxent_treebank_pos_tagger",
                         "tokenizers/punkt"]
    if os.path.sep != '/':
        _PY3_DATA_UPDATES = [s.replace('/', os.path.sep)
                             for s in _PY3_DATA_UPDATES]

    t2 = time.time()
    replace_tot += (t2 - t1)

join_avg = join_tot / num_runs
replace_avg = replace_tot / num_runs

print "{0} {1}".format(join_avg, replace_avg)

The output I got was 4.57689023018e-06 for os.path.join and 3.92367839813e-07 for replace. This means replace is about 91% faster than os.path.join in the case that os.path.sep is /. For the case where os.path.sep is \, I measured 4.4388897419e-06 for os.path.join and 2.15146183968e-06 for replace, which is only a 50% speedup. In a performance critical module, a 50% or 90% speedup would be unbelievably nontrivial; however, in this case, the runtimes are on the order of microseconds and hundreds of nanoseconds (both inconsequential magnitudes of time). Therefore, for the purposes of this module (i.e., a non-time-critical compatibility module), using replace for speed is a mere micro-optimization that ultimately reduces portability and is less Pythonic (and IMO less readable).

The thing I don't like in #1226's approachment, is the four new useless variables and four new method calls to os.path.join. This is more clean and pythonic: ...

Good point! I agree this is more Pythonic than assigning several variables. I've edited #1226 to mirror your suggestion. :)

I strongly agree that readable, Pythonic, maintainable code is the preference.


perf: Port rpc changes from corefx

This is a port of https://github.com/dotnet/corefx/pull/34049 which changes how rpc object are constructed and re-used so avoid repeated creation of new SqlParameter instances where they aren't needed.
This change was not brought across from corefx initially because the introduction of AlwaysEncrypted required additionally changes. Those changes are part of this new PR. manual tests pass in debug mode (with all asserts) so it looks like the AE changes are ok but I'd like some oversight to make sure I've understood what it's doing correctly.
I think I've found a weakness in the batch rpc mode with AE. It assumes that the command will be in sysemParams[0] but this is only true if the batch rpc was constructed as an sql command and not as a stored procedure. This isn't currently causing a problem but might need tightening up.

Hi @Wraith2
Please check and confirm code coverage (as close to 100% as possible) on changes made here. If RPC tests don't exist, please add them to the suite so they cover maximum line changes here.

How do I check coverage? Given that this effects how you do queries it'd be quite surprising if it wasn't covered.

I'm definitely going to need help getting coverage. I've tried getting the manual tests running in visual studio and i'm unable to apply the required environment variables (launchsettings.json doesn't work for some reason) and the tests themselves won't execute because of an nuint serialization exception. So either I need some guidance on how you get it working in visual studio or how to get coverage numbers from msbuild and use them sensibly.

Sure, I'd expect it to be covered too, we just want to verify if we have everything covered here. So maybe try running tests in Visual Studio with coverage plugins?
Also extracted this coverage report for this PR, hope it serves well as a starting point.
20190922.2...13627.coverage.zip

FYI, you can also run dotnet test command with appended --collect:"code coverage" --collect-format:"coverage"  or --collect:"code coverage" --collect-format:"coveragexml" that will give you similar file for your specific test run.

Should tests run from within visual studio? When I try I get this error which stops the entire run:
[25/09/2019 11:13:44.175 PM Informational] Store opened in 0.320 sec.
[25/09/2019 11:13:45.071 PM Informational] ---------- Discovery started ----------
[25/09/2019 11:13:45.285 PM Error] Microsoft.VisualStudio.TestPlatform.ObjectModel.TestPlatformException: Unable to find E:\Programming\csharp7\SqlClient\bin\Windows_NT\Debug.AnyCPU\CoreFx.Private.TestUtilities\netcoreapp2.1\testhost.dll. Please publish your test project and retry.
   at Microsoft.VisualStudio.TestPlatform.CrossPlatEngine.Hosting.DotnetTestHostManager.GetTestHostPath(String runtimeConfigDevPath, String depsFilePath, String sourceDirectory)
   at Microsoft.VisualStudio.TestPlatform.CrossPlatEngine.Hosting.DotnetTestHostManager.GetTestHostProcessStartInfo(IEnumerable`1 sources, IDictionary`2 environmentVariables, TestRunnerConnectionInfo connectionInfo)
   at Microsoft.VisualStudio.TestPlatform.CrossPlatEngine.Client.ProxyOperationManager.SetupChannel(IEnumerable`1 sources)
   at Microsoft.VisualStudio.TestPlatform.CrossPlatEngine.Client.ProxyDiscoveryManager.DiscoverTests(DiscoveryCriteria discoveryCriteria, ITestDiscoveryEventsHandler2 eventHandler)
[25/09/2019 11:13:45.899 PM Informational] [xUnit.net 00:00:00.3081508]   Discovering: Microsoft.Data.SqlClient.Tests
[25/09/2019 11:13:45.970 PM Informational] [xUnit.net 00:00:00.3776758]   Discovering: Microsoft.Data.SqlClient.ManualTesting.Tests
[25/09/2019 11:13:46.350 PM Informational] [xUnit.net 00:00:00.6369007]   Discovered:  Microsoft.Data.SqlClient.ManualTesting.Tests
[25/09/2019 11:13:46.456 PM Informational] [xUnit.net 00:00:00.7167884]   Discovered:  Microsoft.Data.SqlClient.Tests
[25/09/2019 11:13:46.962 PM Warning] No test is available in E:\Programming\csharp7\SqlClient\bin\Windows_NT\Debug.AnyCPU\Microsoft.Data.SqlClient\netfx\Microsoft.Data.SqlClient.dll. Make sure that test discoverer & executors are registered and platform & framework version settings are appropriate and try again.
[25/09/2019 11:13:47.024 PM Informational] ========== Discovery aborted: -1 tests found (0:00:01.9150349) ==========

I can't use the coverage file you provided because I'm only on visual studio pro. Code coverage is an enterprise or ultimate feature and since I buy my own visual studio license rather than getting it through an employer or volume license both editions are way out of my reach. If I could get the test running resharper should provide coverage, but that hits the error above.

Hi @Wraith2
Try running in Release configuration, we're aware of Debug Configuration test issues in Visual Studio.

None of the release mode configurations work for me either. I get an exception from inside xunit extensions. The coverage cli commands suggested aren't recognised at the commandline when appended to the test commands and the dotnet cli docs don't list them even though I can find the feature request for them being added, perhaps post 2.1 dotnet only. using /p:CollectCoverage=true doesn't seem to do anything either.
I've opened a couple of tracking issues for test and coverage. At the moment I can't provide any coverage data.

Now that 1.1.0 is out can this be reviewed please? It's been rebased and is only failing on that AE tests that fail in master as well. This work is needed for the batch implementation.

Hi @Wraith2
Yes, please update your branch with changes from master to trigger a fresh CI build.

Updated.

Thanks @Wraith2
It looks like the Enclave Provider tests are failing for .NET Core, please check logs in the checks (they're public now) and review implementation to find related code changes. It's tricky to get the setup for Enclave Provider for debugging, so I'd recommend going down your changes and understanding failing tests to fix what went wrong.
You'd be the best person to review first and fix the issues being creator of the PR, we'll start reviewing once we have all tests passing and changes are ready to check-in. If you need any help let us know.

So does SQL17W-E mean with enclave? I can guess from stack traces but realistically if I'm (any anyone else) going to need to ensure enclaves work we'll need instructions on how to set up and run those tests correctly, can you provide something?

Sure, you can begin from here: https://docs.microsoft.com/en-us/sql/relational-databases/security/tutorial-getting-started-with-always-encrypted-enclaves?view=sql-server-ver15 for setup, and use following tutorials to setup .NET Applications, same instructions apply for Microsoft.Data.SqlClient, except that Enclave provider is internally available and does not need additional package reference.
We are also just-in-process of putting specific documentation for Microsoft.Data.SqlClient, since now Enclave provider is available in GA 1.1

The changes are done and pass locally on the same tests as master. The SQL12E-E leg is failing with timeouts and flaky tests from what I can see, can you retry that leg?

Hi @Wraith2
Please trigger a new build by pushing a commit. DevOps recently has some issues with Rebuild.

New build triggered and looks like it's been stuck overnight.


Template polymorphism is not implemented

Template polymorphism is not implemented.
For template polymorphism see: refman
The following compiles in Coq:
Definition T := {A : Type & {B : Type & A}}.

While it does not in Template Coq:
Require Import Template.TemplateCoqChecker.
Fail Template Check T.

I am not sure we will implement it because we can use full polymorphism instead but people should be aware of it.
@mattam82 Do you confirm?

Indeed, I’d rather avoid the complications but it’s not so hard to implement

This should be mentionned in the manual/readme


Interface change

Hi Jeremy,
it's me again :)
I'm testing v0.2.0-beta
README needs updating, since struct stacktrace no longer contains a member empty().
Otherwise it's working fine! Thanks!

Hey! Thank you so much, I am immensely grateful for your interest in the library and for catching my mistakes :)

And also very glad to hear it’s working otherwise

empty() was added after the v0.2.0-beta tag I think. The library's interface has been changing quickly as I've been trying to expand functionality. I'll try to do another prerelease soon.

I've bumped the version in the readme and made another prerelease so should be good now


SafeArea for Pixel 3 XL Too Small

I don't know if there are any other phones where this happens, but it seems the container created by the SafeArea Widget is not small enough. The notch still partially bites into the application.
To Reproduce:

Create Emulator Profile for Pixel 3 XL
Use a SafeArea Widget in code (I also filled it with a teal colored Container)
Run the program, and use Flutter Inspector to most easily see it

As you can see in the attached image, roughly half the notch is still in the SafeArea.

I think that's an emulator bug, since the status bar is also incorrect.

I think that's an emulator bug, since the status bar is also incorrect.

I'm on Oreo 8.1 not the latest Android version that might be why the status bar looks different.
If someone has a physical Pixel 3 XL they should try it to see if its an emulator bug.

Hi @jackschedel
This seems to resolved and looks like more like emulator than Flutter issue

Given the test in the screenshot, i feel free to close this issue
if you disagree please write in the comments and I will reopen it
Thank you


Use of image from --allow-enhancement option in OCR-D workflow

Hi
A question.  Is the --allow-enhancement option resultant image intended to be used in an OCR-D workflow?  In some cases, I believe resultant image has dimensions that differ from the dimensions of the input image.  So if it is intended to be used in OCR-D, what is the suggested workflow (and the commands & parameters) that preserves the overlaying of the ocr results with the image?
Thanks!

A question. Is the --allow-enhancement option resultant image intended to be used in an OCR-D workflow?

No, we currently do not use the intermediate results of eynollah, only the resulting PAGE-XML with the assumption that the original image (dimensions) remains unchanged.

Do you think that the enhancement approaches used in eynollah should be replicated outside of it to be used in an OCR workflow, or do you think the OCR-D workflow enhancement steps produce a better quality starting image to do OCR on?

Do you think that the enhancement approaches used in eynollah should be replicated outside of it to be used in an OCR workflow, or do you think the OCR-D workflow enhancement steps produce a better quality starting image to do OCR on?

The impact of enhanced image on OCR is not assessed yet. But I think for documents with low quality the enhanced images may improve  OCR performance.