Imgpkg copy failing to move bundle between same registry, different project

What steps did you take:
Update to v0.26.0 and/or v0.27.0 from v0.25.0 and using imgpkg copy to move a bundle between same registry, different project.
What happened:
After the update, imgpkg copy started to fail for my usage.
v0.25.0 works however.
What did you expect:
I expected the imgpkg copy to continue to work.
Anything else you would like to add:
Environment variables set:
IMGPKG_REGISTRY_HOSTNAME_0=${REGISTRY}/${PROJECT_1}
IMGPKG_REGISTRY_USERNAME_0=<Bot user for above host/project>
IMGPKG_REGISTRY_PASSWORD_0=<Pass>

IMGPKG_REGISTRY_HOSTNAME_1=${REGISTRY}/${PROJECT_2}
IMGPKG_REGISTRY_USERNAME_1=<Bot user for above host/project>
IMGPKG_REGISTRY_PASSWORD_1=<Pass>

Command used:
imgpkg copy -b "${REGISTRY}/${PROJECT_1}/${IMAGE}:${TAG}" --to-repo "${REGISTRY}/${PROJECT_2}/${IMAGE}"

$ imgpkg copy -b <REGISTRY>/<PROJECT_1>/<IMAGE>:1.1.0 --to-repo <REGISTRY>/<PROJECT_2>/<IMAGE>
copy | exporting 3 images...
copy | will export <REGISTRY>/<PROJECT_1>/<IMAGE>@sha256:b473574622c41251fa4a7b62ff7e7cfb25bc4b394eb439bdef69972365e1c89a
copy | will export <REGISTRY>/<PROJECT_1>/<OTHER_IMAGE>@sha256:0fde65bf137374e37cbd983e70a3fb683a12822d4dc4967f4e3d7fdf091025f3
copy | will export <REGISTRY>/<PROJECT_1>/<OTHER_IMAGE_2>@sha256:e6bf93b0460bc4666d88d942468ebe557f0aac365decd70e77ffcea892ab72a8
copy | exported 3 images
copy | importing 3 images...

19.93 MiB / 48.36 MiB [------------------------------------------------------------------->________________________________________________________________________________________________] 41.21% 1.43 MiB p/s
Error uploading images: POST https://<REGISTRY>/v2/<PROJECT_2>/<IMAGE>/blobs/uploads/?from=<PROJECT_1>%2F<IMAGE>&mount=sha256%3A1ed0e4ef744e7e2688d563427f63df35ab43a3056f0b00baa8d239b4d77b52b5: UNAUTHORIZED: unauthorized to access repository: <PROJECT_1>/<IMAGE>, action: pull: unauthorized to access repository: <PROJECT_1>/<IMAGE>, action: pull

19.93 MiB / 48.36 MiB [------------------------------------------------------------------->_______________________________________________________________________________________________] 41.21% 17.92 MiB p/s

copy |
copy | done uploading images

imgpkg: Error: POST https://<REGISTRY>/v2/<PROJECT_2>/<IMAGE>/blobs/uploads/?from=<PROJECT_1>%2F<IMAGE>&mount=sha256%3A1ed0e4ef744e7e2688d563427f63df35ab43a3056f0b00baa8d239b4d77b52b5:
  UNAUTHORIZED: unauthorized to access repository: <PROJECT_1>/<IMAGE>, action: pull: unauthorized to access repository: <PROJECT_1>/<IMAGE>, action: pull

$ imgpkg version
imgpkg version 0.26.0

Succeeded

Environment:

imgpkg version (use imgpkg --version): v0.26.0 and v0.27.0
Docker registry used (e.g. Docker HUB): Harbor
OS (e.g. from /etc/os-release): VMware Photon OS

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

Looks like there is a problem when mounting a blob, where the incorrect credentials are being used. This looks like it was introduced when the transport reuse was added to the code.

with any version ( tried with 26,27,29) of imgpkg greater than v0.25.0 I am hitting the below issue, seems like same behaviour for which above issue is opened.
imgpkg copy --tar /root/foo/bar_latest.tar --to-repo abc-harbor.com/projecttest/testdeploy/tcx/foo-bar-custom --registry-username='testuser' --registry-password='testpass'
copy | importing 3 images...
0 B / 773.52 MiB [_______________________________________________________________________________________________] 0.00% ? p/s
copy |
copy | done uploading images
Error uploading images: POST https://abc-harbor.com/projecttest/testdeploy/tcx/foo-bar-custom/blobs/uploads/?from=tcx%2Fsnapshot%2Ftanzu%2Ffoo0bar&mount=sha256%3Ae16be837292214ac682c9127e1c7ac894913c66dd1ae2d9c8955ab2c8c6c75a5: UNAUTHORIZED: unauthorized to access repository: tcx/snapshot/tanzu/foo-bar, action: pull: unauthorized to access repository: tcx/snapshot/tanzu/foo-bar, action: pull

Hey, @vikundral thanks for opening this issue.
Was the tar created from that same registry? Because one of the checks that we do is to see if the images are in the same registry.


suspicous

https://github.com/motis-project/nigiri/blob/300553573026e2b8d5ef83e74faf5b6399d571f7/include/nigiri/routing/raptor/raptor.h#L255

looks good!
dir is used here:
https://github.com/motis-project/nigiri/blob/300553573026e2b8d5ef83e74faf5b6399d571f7/include/nigiri/routing/raptor/raptor.h#L253


Selecting disable player names will only work on hero units

I'm not particularly bothered however, just as a note, there are several other locations player names will pop up

LootBannerSconce  where they capture a merc camp or destroy a town structure
The end summary screen where it shows MVP and other Awards

And just to be clear, this is probably perfectly fine. But it does not "remove" the player names entirely from the UI

To remove the name from the Banners, the following strings should be edited to use these strings missing their ~ A ~ token:
Param/Expression/lib_Game_3A8C4155=
Param/Expression/lib_Game_B776822C=
Param/Expression/lib_Game_F8D64D74=
Param/Expression/lib_Game_A1A65E62=
Param/Expression/lib_MapM_E914B34B=
Param/Expression/lib_MapM_31CA03E8=
Param/Expression/lib_MapM_97AED412=
Param/Expression/lib_MapM_9A96809B=
Param/Expression/lib_MapM_50C2893A=
Param/Expression/lib_MapM_104CCD72=

And inactivity chat message:
Param/Expression/lib_Game_9D7B4265=A Player has been dropped for inactivity.

These changes cannot be toggled as they edit the content of gamestrings.txt and need to be applied to every localization used.
The MVP screen should have labels that can be dynamically hidden.


Update chunk_overlap in service_context.py

Chunk_overlap should be able to be set to 0.
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of Change

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

[x] I stared at the code and made sure it makes sense

Hmmm, it says this PR has no changes now?

@logan-markewich yeah I fixed the lint errors and it turns out no change. not sure there is any different between original vs @Miroun 's change


search-react: minor search bar analytics refactor

Bit of cleanup as I was poking around at the flaky test

@freben indeed, CC @backstage/discoverability-maintainers 👀


Fix SearchBar

The suggestions in the search bar are overlapped by the carousel below it. Need to add extra padding if suggestions are available.

assign this to me.

I am assigning it to @gautmikhanna . Kindly mention the issue no. In the description of your PR. Thank you

@gaut2003 keep an eye on the repository, We will be adding up new issues soon! Thank you.

I have solved the issue.

Have sent you a PR fixing this bug. @iamartyaa


Missing support for System.Char on auto decoder

I had a char field on one of my bridge messages and it failed to decode. Ended up using string instead, but it would be less problematic with char support out of the box.

I moved this issue to another repository, and further work will take place there.


IOS : Some Indian mobile numbers are shown as "FIXED_LINE_OR_MOBILE"

IOS : Some Indian mobile numbers are shown as "FIXED_LINE_OR_MOBILE"
Hey guys,
Country/region affected : IN
Example number(s) affected :<PHONE_NUMBER>,<PHONE_NUMBER>,<PHONE_NUMBER>
The type of the number(s) : "mobile"
The above three numbers are Indian mobile numbers. (Probably new series added and approved).
The ios library returns "FIXED_LINE_OR_MOBILE" instead of mobile.
Also tried your test page which  returns the same :
https://libphonenumber.appspot.com/phonenumberparser?number=7795705322&country=IN
https://libphonenumber.appspot.com/phonenumberparser?number=7259757766&country=IN
https://libphonenumber.appspot.com/phonenumberparser?number=7977884930&country=IN
Please see the screenshot attached.
Please let us know if this can be fixed soon. We had the same issue in java library, and that got fixed in one of the last upgrades.

Hey guys,
Can someone please look into this issue?
Thanks,
Asithakesh


How to find patients who enter ICU due to specific diseases？

I want to find patients who have entered ICU due to pneumonia, and these patients have no history of sepsis before, How should I do?
I tried use admissiondx table to find patients with pneumonia,But I'm not sure if they were with sepsis when they were diagnosed into admissiondx table.

@alistairewj
Sir, can you please answer this question?, I will be very thankful.

The APACHE admission dx system used does not have many choices for sepsis
but you should be able to find it under the cardiovascular system
diagnoses. This will provide the primary reason for admission for that
specific stay. There isn't a consistent and reliable way to identify a past
history of sepsis.
On Thu, Feb 16, 2023, 20:04 tohokulgq @.***> wrote:

@alistairewj https://github.com/alistairewj
Sir, can you please answer this question?, I will be very thankful.
—
Reply to this email directly, view it on GitHub
https://github.com/MIT-LCP/eicu-code/issues/218#issuecomment-1433949268,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD6USVHAHN5PM35QV2KWNH3WX3FDRANCNFSM6AAAAAAU4QLYKQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

thank you so much for your reply!
So the primary reason for ICU admission is APACHE admission records which in admissiondx, right?
and now I think maybe the diagnosis history could be found in pasthistory table.

Yes but be aware that the past history table is generally not thoroughly
documented so absence of a dx listed does not necessarily mean there is not
a past history of it.
On Fri, Feb 17, 2023, 00:49 tohokulgq @.***> wrote:

thank you so much for your reply!
So the primary reason for ICU admission is APACHE admission records which
in admissiondx, right?
and now I think maybe the diagnosis history could be found in pasthistory
table.
—
Reply to this email directly, view it on GitHub
https://github.com/MIT-LCP/eicu-code/issues/218#issuecomment-1434136011,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD6USVACPFGTZJNRK3D6YJTWX4GPBANCNFSM6AAAAAAU4QLYKQ
.
You are receiving this because you commented.Message ID:
@.***>

Yes but be aware that the past history table is generally not thoroughly documented so absence of a dx listed does not necessarily mean there is not a past history of it.
…
On Fri, Feb 17, 2023, 00:49 tohokulgq @.> wrote: thank you so much for your reply! So the primary reason for ICU admission is APACHE admission records which in admissiondx, right? and now I think maybe the diagnosis history could be found in pasthistory table. — Reply to this email directly, view it on GitHub <#218 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD6USVACPFGTZJNRK3D6YJTWX4GPBANCNFSM6AAAAAAU4QLYKQ . You are receiving this because you commented.Message ID: @.>

Sorry to bother you again.
I want to know the meaning of 'activetx' in apachepredvar table.
Because i am doing the machine learning research right now and 'activetx'is the most important variable after the analysis of the machine learning model, but there are less explanation of this character.
Your answer will help me a lot! !


cilium/cmd: Cosmetic code clean-ups

This PR makes the code a little more consistent in terms of formatting and structure. There is no functional change.

test-me-please

Given that these are cosmetic changes, I've marked this PR to be backported for 1.9 in order to avoid merge conflicts in potential future 1.9 backports.

retest-gke

All team reviews have been addressed and CI is passing. Merging.


Switch to requirements.txt

@minrk I would like to go ahead and swap this docs build to use requirements.txt instead of conda. The last PR broke the build on RTD. I did a bunch of troubleshooting yesterday re: conda and rtd, and I ran into a ton of build errors, timeouts, etc. Switching to requirements.txt, it all built fine and much faster.

Makes sense. I think the main reason to use conda for docs is the few things that pip doesn't package well:

qt (not packaged at all)
large compiled dependencies (numpy, matplotlib)


The following error occurred when executing step six!

Traceback (most recent call last):
File "D:\AIGC\videotalk\video-retalking\utils\inference_utils.py", line 118, in face_detect
predictions.extend(detector.get_detections_for_batch(np.array(images[i:i + batch_size])))
File "D:\AIGC\videotalk\video-retalking\third_part\face_detection\api.py", line 66, in get_detections_for_batch
detected_faces = self.face_detector.detect_from_batch(images.copy())
File "D:\AIGC\videotalk\video-retalking\third_part\face_detection\detection\sfd\sfd_detector.py", line 42, in detect_from_batch
bboxlists = batch_detect(self.face_detector, images, device=self.device)
File "D:\AIGC\videotalk\video-retalking\third_part\face_detection\detection\sfd\detect.py", line 69, in batch_detect
olist = net(imgs)
File "C:\Users\menka.conda\envs\video_retalking\lib\site-packages\torch\nn\modules\module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "D:\AIGC\videotalk\video-retalking\third_part\face_detection\detection\sfd\net_s3fd.py", line 71, in forward
h = F.relu(self.conv1_1(x))
File "C:\Users\menka.conda\envs\video_retalking\lib\site-packages\torch\nn\functional.py", line 1298, in relu
result = torch.relu(input)
RuntimeError: CUDA out of memory. Tried to allocate 508.00 MiB (GPU 0; 4.00 GiB total capacity; 2.69 GiB already allocated; 0 bytes free; 3.11 GiB reserved in total by PyTorch)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "inference.py", line 342, in
main()
File "inference.py", line 211, in main
for i, (img_batch, mel_batch, frames, coords, img_original, f_frames) in enumerate(tqdm(gen, desc='[Step 6] Lip Synthesis:', total=int(np.ceil(float(len(mel_chunks)) / args.LNet_batch_size)))):
File "C:\Users\menka.conda\envs\video_retalking\lib\site-packages\tqdm\std.py", line 1178, in iter
for obj in iterable:
File "inference.py", line 292, in datagen
face_det_results = face_detect(full_frames, args, jaw_correction=True)
File "D:\AIGC\videotalk\video-retalking\utils\inference_utils.py", line 121, in face_detect
raise RuntimeError('Image too big to run face detection on GPU. Please use the --resize_factor argument')
RuntimeError: Image too big to run face detection on GPU. Please use the --resize_factor argument

RuntimeError: CUDA out of memory；you can try cpu mode or try let batchsize down

How to enable CPU operations? Can you share the commands?

RuntimeError: CUDA out of memory；you can try cpu mode or try let batchsize down

How to enable CPU operations? Can you share the commands?


MSID: 96852 Version: 1 DOI: 10.1101/2<IP_ADDRESS>5731

MSID: 96852
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>5731
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=96852
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>5731

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/96852v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>5731
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/96852/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!


Uncaught ReferenceError: blueimp is not defined

I get this error in console:
GET http://localhost/webapplication/frontend/web/assets/5aaa8e05/css/blueimp-gallery.min.css
1:462 GET http://localhost/webapplication/frontend/web/assets/5aaa8e05/js/blueimp-gallery.min.js 404 (Not Found)
1:468 Uncaught ReferenceError: blueimp is not defined

@creocoder will handle it.

@hdushku make sure the bower plugin was installed properly.


Build failure: citrix_workspace

Steps To Reproduce
Steps to reproduce the behavior:

use citrix_workspace (i.e. the "main"/"latest" one)
update channel nixos-unstable to rev a3c0b3b21515f74fd2665903d4ce6bc4dc81c77c
sudo nixos-rebuild build --flake ~/mynix

Build log
nix log  /nix/store/8h191qnqkib7zdmp26q5lhyghf2lxq3x-citrix-workspace-<IP_ADDRESS>.drv
https://gist.github.com/GsakuL/4d67b55bc23428899aa9f08ec5575c64
error at the bottom:
...

auto-patchelf: 1 dependencies could not be satisfied
error: auto-patchelf could not satisfy dependency libogg.so.0 wanted by /nix/store/ms9j3ik2yfvdbxw1hqzii9a7azvnlg6d-citrix-workspace-<IP_ADDRESS>/opt/citrix-icaclient/VORBIS.DLL
auto-patchelf failed to find all the required dependencies.
Add the missing dependencies to --libs or use `--ignore-missing="foo.so.1 bar.so etc.so"`.

Additional context
maybe see also #348866
also, is it normal, that the Citrix version reports itself as <IP_ADDRESS> even tho in nixpkgs it is <IP_ADDRESS>?
Notify maintainers

@michaeladler
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.54, NixOS, 24.11 (Vicuna), 24.11.20241006.c31898a`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.8`
 - nixpkgs: `/nix/store/rs4fjbnw4qx7ns2hzzrz2iz52va7vs5z-source`

Add a :+1: reaction to issues you find important.

FYI, pkgs.citrix_workspace_23_11_0 on current unstable is also broken.

Yeah, I think commit 04bf3a5a24294b530a985b95ec8aa984c7d339ae by @doronbehar broke it (switching to cmake-based build).

Adding -DBUILD_SHARED_LIBS=ON to libogg's cmakeFlags fixes things (by making the .so libs build again.
This should probably be fixed in libogg - do you agree @ehmry?
Here is an overlay that makes things compile again:
final: prev: {
  # https://github.com/NixOS/nixpkgs/issues/348868
  citrix_workspace = prev.citrix_workspace.override {
    libvorbis = final.libvorbis.override {
      libogg = final.libogg.overrideAttrs (prevAttrs: {
        cmakeFlags = (if prevAttrs ? cmakeFlags then prevAttrs.cmakeFlags else []) ++ [
          (final.lib.cmakeBool "BUILD_SHARED_LIBS" true)
        ];
      });
    };
  };
}

if prevAttrs ? cmakeFlags then prevAttrs.cmakeFlags else []

By the way, you can write this more simply as prevAttrs.cmakeFlags or []
https://nix.dev/manual/nix/2.18/language/operators#attribute-selection
But yeah I agree that fixing libogg itself is the way to move forward

Adding -DBUILD_SHARED_LIBS=ON to libogg's cmakeFlags fixes things (by making the .so libs build again.
This should probably be fixed in libogg - do you agree @ehmry?

Yes I agree. Sorry about this regression.

Will create a PR targetting staging shortly.

https://github.com/NixOS/nixpkgs/pull/349503

Here is an overlay that makes things compile again:
final: prev: {
  […]
}

I got an infinite recursion error when using that (libvorbis = final.libvorbis seems to be the issue), but this worked:
{
  nixpkgs.overlays = [
    (final: prev: {
      libvorbis = prev.libvorbis.override {
        libogg = final.libogg.overrideAttrs (prevAttrs: {
          cmakeFlags = (prevAttrs.cmakeFlags or [ ]) ++ [ (final.lib.cmakeBool "BUILD_SHARED_LIBS" true) ];
        });
      };
    })
  ];
}

(Well, it's still building, but it hasn't failed yet :grin:.)

@l0b0 Of course as written that would cause infrec, writing something like a = final.a.override... would inherently be infrec. The infrec was because of not including the previous line:
citrix_workspace = prev.citrix_workspace.override {

@0-a-9-5-6 Being a relatively new NixOS user, how would I use your code above?  In the same module where I include the citrix_workspace, just add it?
Thanks!

Replace pkgs.citrix_workspace with (pkgs.citrix_workspace.override ...). (I'll not rewrite it out the ... again.)

Thank you for taking time out to help me.  I was able to copy your code, and it is working for me now.  Thanks, again!

The PR that fixed this has reached nixos-unstable:
https://nixpk.gs/pr-tracker.html?pr=349503


Custom CSS Theme

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched existing issues to ensure the feature has not already been requested

🚀 Feature Proposal
Good time of day. I tried to find an opportunity to connect a custom CSS file (a theme for Swagger), but unfortunately I didn't find it. Is there such a possibility? If not, is it planned?
Motivation
No response
Example
No response

I would really love support for this too.

@mcollina the fact is that I am the developer of the "swagger-theme" library. And now the question has arisen about the support of Fastify. But unfortunately, I was unable to connect the library.


RemoveUnusedImports removes used imports and adds duplicated ones

What version of OpenRewrite are you using?
I am using

Maven plugin v4.46.0

How are you running OpenRewrite?
I am using the Maven plugin, and my project is a single module project
What is the smallest, simplest way to reproduce the problem?
https://github.com/pzrep/openrewrite-unused-import-record
What did you expect to see?
I expected to see no changes.
What did you see instead?
https://github.com/pzrep/openrewrite-unused-import-record/actions/runs/5100022041/jobs/9168054354
Depends on number of imports done with * I guess:

star-import is replaced with explicit imports, including duplicated [Record] one:

diff --git a/src/main/java/pzrep/p2/Client1.java b/src/main/java/pzrep/p2/Client1.java
index 2533533..edfecd5 100644
--- a/src/main/java/pzrep/p2/Client1.java
+++ b/src/main/java/pzrep/p2/Client1.java
@@ -1,7 +1,10 @@
 package pzrep.p2;

 import pzrep.p1.Record;
-import pzrep.p1.*;
+import pzrep.p1.A;
+import pzrep.p1.B0;
+import pzrep.p1.B1;
+import pzrep.p1.Record;

 class Client1 {
     void f(Record r) {

This is unfortunate but allows to compile modified class.

required import gets removed

diff --git a/src/main/java/pzrep/p2/Client2.java b/src/main/java/pzrep/p2/Client2.java
index 2bb52f5..50aea66 100644
--- a/src/main/java/pzrep/p2/Client2.java
+++ b/src/main/java/pzrep/p2/Client2.java
@@ -1,6 +1,5 @@
 package pzrep.p2;

-import pzrep.p1.Record;
 import pzrep.p1.*;

 class Client2 {

This makes modified source to be uncompilable. (That's the original issue I observed with https://www.jooq.org/javadoc/latest/org.jooq/org/jooq/Record.html)

I see that duplicated Record import (p. 1 above) is no longer inserted (looks that something in maven plugin 5.0.0->5.2.1 helps :smile:)

this was very kindly fixed by @tclayton-newr in #3397; you can already try this in our snapshot versions, or with the next release in a week or so

Still having my Record removed in

https://github.com/pzrep/openrewrite-unused-import-record/pull/8

Still having my Record removed in

https://github.com/pzrep/openrewrite-unused-import-record/pull/9

Could this be reopened (or cloned to new one if needed) by maintainers, please?


Sweep Requests

The Requests table should be periodically "swept" to manage memory usage.
Considering #38, if the configuration lives on the endpoint, then we will need to decide when to start the sweep. Here are my ideas:

Start the sweep after the first request.
Add an endpoints: [atom(), ...] option to the config :phoenix_profiler and pull the configs when the Requests process starts.

For 1)
Pro: Zero config for endpoint discovery. No unnecessary sweeps.
Cons: Config validation also has to wait for the first request. Special logging required for bad configs?
For 2)
Pros: Config validation happens on Application start.
Cons: Additional config for endpoint discovery. Split configuration for the profiler.
I am leaning towards 1) but I open to other thoughts as well :)
As for the configuration, wherever it lives, I was thinking of this:
  requests_expire_after: :timer.hours(24), # how long until requests expire (in milliseconds).
  requests_sweep_interval: :timer.seconds(600) # how often to sweep in milliseconds.

I'm 👍🏻 for option 1, more explicit and less install steps to do.

Special logging required for bad configs?

We could leverage https://github.com/dashbitco/nimble_options although pulling an extra lib seems a bit overkill at this moment.

I think I am really asking what should happen at runtime if the profiler config on the endpoint is invalid? 🤔

I think I am really asking what should happen at runtime if the profiler config on the endpoint is invalid? 🤔

Disable it and display a warning? I mean, it shouldn't affect the host app and it's a tool made for developers so it should be straightforward to spot it's not working just by looking at logs/console and then fix it (the config is just a few keys to check/change). In other words, don't get in the way if something is wrong.
Unless you see a scenario where it would break or something.


Simplify Debian & Ubuntu Docker Engine installation guide

Proposed changes
Simplify Debian & Ubuntu Docker Engine installation guide

Thanks @StefanScherer and @usha-mandya , I found this doc can be simplified during the progress of dealing with https://github.com/docker/scan-cli-plugin/issues/164, and I found that there are so many issues here, so I opened up another one - https://github.com/docker/docker.github.io/issues/13638, would like to see if it can help resolve some of the issues from the help of Hacktoberfest participant!


Close the "0" loophole

Before this PR:

PkgA = "0" is considered to be upper-bounded
PkgA = ">= 1.2.3" is considered to NOT be upper-bounded

After this PR:

PkgA = "0" is considered to NOT be upper-bounded
PkgA = ">= 1.2.3" is considered to NOT be upper-bounded

This is a breaking change.

Motivation
The compat entry PkgA = "0" represents [0.0.0, 1.0.0). So technically, it has an upper-bound. But because 0.x.0 to 0.(x+1).0 is allowed to be a breaking change, the range [0.0.0, 1.0.0) contains infinitely many breaking releases. So we really shouldn't consider [0.0.0, 1.0.0) to be upper-bounded.

bors merge

bors merge

bors merge-

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge


Recursive calls not working

The following (dumb/incorrect) recursive code doesn't compile, but should:
Facility T;
    Oper Main();
        Procedure
            Main();
    end Main;
end T;

we aren't finding the operation corresponding to Main() (probably because we haven't added it to mains scope yet).
EDIT:
the code should (I think) actually be:
Facility T;
    Oper Main();
        Recursive Procedure
            Main();
    end Main;
end T;

Turns out we only add a pointer to the name of the operation to its scope if the operation has a return value. Clearly we now need to do so regardless for the sake of recursion (recursion that modifies the params). We just need to make sure we appropriately check for the following now:
Operation F(eval x : Int); Procedure F := 3; end F;

F doesn't declare a return value, but if the user types this in, as per our expected change above, a reference to F will now be in scope allowing users to say this erroneous code. Before, by not adding F to scope by default, F wouldn't have be in there so this wouldn't even be a possibility (but it would've given a pretty shitty, indirectly related error message to indicate as much).

OK, sparing some details -- trying to add an OperationSymbol into scope for an OperationProcedureDecl causes scope problems. Notably, if the OperationProcedureDecl has a return type, then we already add a variable w/ a name matching the operation into scope prior to population of the requires and ensures clauses.
So what? Why does that matter for this? Well, upon attempting to add an OperationSymbol to handle lookups of any recursive calls, we'll inadvertently trigger a duplicate symbol exception  (since we've already added a variable of the op's name, and now we're trying to add another one -- an operation). In other words, we've got a messy situation where we have two invisible things (one: a variable, and the other: an operation) with the same name that need to occupy the same scope -- but can't for obvious reasons. Here's some things I thought of to 'fix' this:

Allow found OperationSymbols to be coerced into ProgVariableSyms.
Yes, I suppose you could simply make it so that operation symbols could be coerced into programming variables, but the ability to coerce at this point doesn't matter because we wouldn't be able to get it into scope anyhow without colliding with the potentially present variable. 2 comes up with a solution to this:

Build an inners scope to house variableDecls* stmt* and simply add the dummy OperationSymbol there.. This seems to work, except (even with coercion) implemented, I was running into some issues where some of the math stuff was still somehow tripping an UnexpectedSymbolException.

Overall I got tired of fighting with this and simply now use the symbol Resolve.OperationProcedureDecl node itself to check if a program param exp is in fact a recursive call (I think this works fine in general) and also allows us to give the user the following warning when they make a recursive call without declaring the procedure recursive:
warning(39): T.resolve:4:13: recursive call Main() to unmarked recursive procedure 'Main'; correct definition is: Oper Main(..); Recursive Procedure' ... end Main;

Anyways, this will be in the next merge after some further tests.

Need to add the warning for ProcedureDecls as well...

Warnings added -- this is performed in the SanityCheckingListener as of merge: #41


Nested SVG layout is wrong

nested_svg.svg -> nested_svg.pdf
Expected to see 4 rectangles at all the edges like with other converters.
https://github.com/typst/svg2pdf#supported-features
Supported features explicitly include nested SVGs but I couldn't find any input example in tests.

This is a bug in the resvg library that svg2pdf uses under the hood, see here.

As I can see, the root cause is fixed and merged into resvg master branch so this issue can be closed as soon as this project is updated to use the latest version of resvg (usvg). 👍

I can confirm that usvg 0.30.0 fixes this issue, thanks to your contribution: https://github.com/RazrFalcon/resvg/commit/00005084c2544179a7e094d4efa979df1ec04e1b


License informtion inconsistent with other docs

In the license and role requirements at https://learn.microsoft.com/en-us/dynamics365/sales/understand-the-up-next-widget#license-and-role-requirements it states that Sales Premium is required to use the Up next widget.
However, elsewhere in the documentation for Adding the Up next widget to a custom form is it says that Sales Premium or Sales Enterprise are allowed.
https://learn.microsoft.com/en-us/dynamics365/sales/add-upnext-widget-form#license-and-role-requirements
I think one of the pages must be wrong as they should match ?

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

ID: d3ae5a63-9b84-b426-a1b4-e58f9aaa4e14
Version Independent ID: d3ae5a63-9b84-b426-a1b4-e58f9aaa4e14
Content: Understand the up next widget on records
Content Source: ce/sales/understand-the-up-next-widget.md
Service: dynamics-365-sales
GitHub Login: @udaykirang
Microsoft Alias: udag

Thanks for your inputs. update the section accordingly.


Fix startup flags logic (--genesis_block & --overwrite_ledger)

Signed-off-by: Igor Egorov<EMAIL_ADDRESS>Description of the Change
This is a better implementation of #1776.
The problem is that CI needs to be able to launch irohad that will restore the existing state with the same set of flags as that was passed at the first launch.
The new way of launch parameters handling is implemented. All the possible cases is covered.
The new truth table is below:

For a better understanding of changes made in code please take a look at two execution flowcharts (old and new one).
Old execution flow

New execution flow

Benefits
Nothing is changed in already exposed API.
No unexpected behavior is added.
All the combinations of three parameters (presence of blockstore, presence of new genesis_block, presence of overwrite_ledger flag) are covered.
Possible Drawbacks
None?
Usage Examples or Tests
cmake -H. -Bbuild
cd build
make irohad_test
test_bin/irohad_test

Can we throw a warning when blockstore is present and overwrite ledger flag is set?

Can we throw a warning when blockstore is present and overwrite ledger flag is set?

That is already implemented


chore: use a different syntax kind for suffices-by

There are other ways to address this issue, but this seems like the simplest fix. The problem that appears in mathport is that we want to replace by exact $t -> $t, but when this appears as a subterm of a suffices, like suffices Nat by exact 1, we get the malformed expression suffices Nat 1, because suffices takes only a term of the form by $tacs, not any term, even though the syntax kind of this by is the same as the usual one.
The fix implemented here is to have a separate syntax kind byTactic' for use in show and suffices, so that we cannot confuse (byTactic'| by exact $t) with (term| by exact $t).

I think the error is because this needs a bootstrap update. Help!

I think the error is because this needs a bootstrap update. Help!

Indeed, you just need to update stage0.  It's because the show-elaborator is implemented using a syntax matcher; and the syntax matcher uses the syntax as parsed by the stage0 compiler (since show is built-in notation).

Merged manually because of update stage0 conflicts.

@digama0 Thanks for adding the comment.


[BUG] Unable to build recent commits related to osx deployment

Describe the bug
Getting an error while trying to build any commits beyond 073076ac7dce7cc36d4753d8cbb7bdbdf17a0f68 (i.e., commits beginning with 9d7fa6b - Use osx deployment target to pick Metal version related to #1595 and #1601)
To Reproduce
% git checkout master
% git pull
% pip install -U  .

Expected behavior
A build without error
Desktop (please complete the following information):

OS Version: MacOS 15.1

Additional context
The error is:
% pip install -U  .
Processing /path/to/mlx
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Building wheels for collected packages: mlx
  Building wheel for mlx (pyproject.toml) ... error
  error: subprocess-exited-with-error

  × Building wheel for mlx (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [157 lines of output]
      running bdist_wheel
      running build
      running build_py
      copying python/mlx/_os_warning.py -> build/lib.macosx-10.9-universal2-cpython-311/mlx
      [..snip..]
      running build_ext
      CMake Warning at /private/var/folders/_7/1_8y8v7x0y94v6y52ryvrrhm0000gn/T/pip-build-env-dldgc7s1/overlay/lib/python3.11/site-packages/cmake/data/share/cmake-3.31/Modules/Platform/Darwin-Initialize.cmake:311 (message):
        Ignoring CMAKE_OSX_SYSROOT value:

         /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX15.0.sdk

        because the directory does not exist.
      Call Stack (most recent call first):
        /private/var/folders/_7/1_8y8v7x0y94v6y52ryvrrhm0000gn/T/pip-build-env-dldgc7s1/overlay/lib/python3.11/site-packages/cmake/data/share/cmake-3.31/Modules/CMakeSystemSpecificInitialize.cmake:34 (include)
        CMakeLists.txt:3 (project)

      -- The C compiler identification is AppleClang <IP_ADDRESS>00026
      -- The CXX compiler identification is AppleClang <IP_ADDRESS>00026
      -- Detecting C compiler ABI info
      -- Detecting C compiler ABI info - failed
      -- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc
      -- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc - broken
      CMake Error at /private/var/folders/_7/1_8y8v7x0y94v6y52ryvrrhm0000gn/T/pip-build-env-dldgc7s1/overlay/lib/python3.11/site-packages/cmake/data/share/cmake-3.31/Modules/CMakeTestCCompiler.cmake:67 (message):
        The C compiler

          "/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc"

        is not able to compile a simple test program.

        It fails with the following output:

          Change Dir: '/path/to/mlx/build/temp.macosx-10.9-universal2-cpython-311/mlx.core/CMakeFiles/CMakeScratch/TryCompile-uYqxst'

          Run Build Command(s): /private/var/folders/_7/1_8y8v7x0y94v6y52ryvrrhm0000gn/T/pip-build-env-dldgc7s1/overlay/lib/python3.11/site-packages/cmake/data/bin/cmake -E env VERBOSE=1 /usr/bin/make -f Makefile cmTC_61bb6/fast
          /Applications/Xcode.app/Contents/Developer/usr/bin/make  -f CMakeFiles/cmTC_61bb6.dir/build.make CMakeFiles/cmTC_61bb6.dir/build
          Building C object CMakeFiles/cmTC_61bb6.dir/testCCompiler.c.o
          /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc   -arch arm64 -MD -MT CMakeFiles/cmTC_61bb6.dir/testCCompiler.c.o -MF CMakeFiles/cmTC_61bb6.dir/testCCompiler.c.o.d -o CMakeFiles/cmTC_61bb6.dir/testCCompiler.c.o -c /path/to/mlx/build/temp.macosx-10.9-universal2-cpython-311/mlx.core/CMakeFiles/CMakeScratch/TryCompile-uYqxst/testCCompiler.c
          Linking C executable cmTC_61bb6
          /private/var/folders/_7/1_8y8v7x0y94v6y52ryvrrhm0000gn/T/pip-build-env-dldgc7s1/overlay/lib/python3.11/site-packages/cmake/data/bin/cmake -E cmake_link_script CMakeFiles/cmTC_61bb6.dir/link.txt --verbose=1
          ld: library 'System' not found
          cc: error: linker command failed with exit code 1 (use -v to see invocation)
          /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc  -arch arm64 -Wl,-search_paths_first -Wl,-headerpad_max_install_names CMakeFiles/cmTC_61bb6.dir/testCCompiler.c.o -o cmTC_61bb6
          make[1]: *** [cmTC_61bb6] Error 1
          make: *** [cmTC_61bb6/fast] Error 2

        CMake will not be able to correctly generate this project.
      Call Stack (most recent call first):
        CMakeLists.txt:3 (project)

      -- Configuring incomplete, errors occurred!
      Traceback (most recent call last):
        File "/path/to/venv/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        [..snip..]
        File "/private/var/folders/_7/1_8y8v7x0y94v6y52ryvrrhm0000gn/T/pip-build-env-dldgc7s1/overlay/lib/python3.11/site-packages/setuptools/_distutils/command/build_ext.py", line 502, in _build_extensions_serial
          self.build_extension(ext)
        File "<string>", line 98, in build_extension
        File "/path/to/sytem/wide/Python/Versions/3.11/lib/python3.11/subprocess.py", line 571, in run
          raise CalledProcessError(retcode, process.args,
      subprocess.CalledProcessError: Command '['cmake', '/path/to/mlx', '-DCMAKE_INSTALL_PREFIX=/path/to/mlx/build/lib.macosx-10.9-universal2-cpython-311/mlx/', '-DCMAKE_BUILD_TYPE=Release', '-DBUILD_SHARED_LIBS=ON', '-DMLX_BUILD_PYTHON_BINDINGS=ON', '-DMLX_BUILD_TESTS=OFF', '-DMLX_BUILD_BENCHMARKS=OFF', '-DMLX_BUILD_EXAMPLES=OFF', '-DMLX_PYTHON_BINDINGS_OUTPUT_DIRECTORY=/path/to/mlx/build/lib.macosx-10.9-universal2-cpython-311/mlx/', '-DMLX_VERSION=0.20.0.dev20241119+2419edd5']' returned non-zero exit status 1.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for mlx
Failed to build mlx

Thanks for flagging, I believe #1601 closes this


New technique suggestion: Spectral Segmentation

Hi,
Could you add "Spectral Segmentation" as a technique?
Here is the link with more info
https://github.com/Leveltlab/SpectralSegmentation

Thanks for raising the issue. @lzehl what do you think? We have "anatomical segmentation technique". To me it sounds correct to add "spectral segmentation technique" as well. What do you think?

@UlrikeS91 theoretically this is can be considered a subtype of an anatomical segmentation technique (since they use it to define masks for neuron cell bodies). Nonetheless it could be used for non-anatomical segmentation problems as well therefore we should discuss it as a separate segmentation technique term.
For registration I would probably call this "cross-spectral power-based image segmentation" OR "spectral power image segmentation". (not sure if we need "image" in the term)
@tgbugs : I'd like your opinion here from the ontology perspective.

@tgbugs and @ehennestad suggest to add also the super class for this technique which would be "auto-segmentation"

With auto-segmentation this becomes a subclass of auto-segmentation that uses a power spectrum over the time series for individual pixels or voxels in the image to generate the segmentation.

so we suggest to register as well "spectral power auto-segmentation"

@chrisvdt what is your opinion here? could you give us inside as develper of the tool https://github.com/Leveltlab/SpectralSegmentation ?

Interesting discussion.  @tgbugs comment describes the software correctly. Spectral power based auto-segmentation would be a correct term. The technique is only for image segmentation in image sequences (movies), not for static images or other type of data, because it depends on functional changes in pixel fluorescence traces.
hope this clarifies things.

closed via https://github.com/HumanBrainProject/openMINDS_controlledTerms/pull/373


[Nox.Ui] Create Flows and Entity Diagrams for Cryptocash Phase1

Initially agreed to build entities for:

Customer
Employee
VendingMachine
Booking
VendingMachineOrder
ExchangeRate
Commission

see Cryptocash design V1.ppt

Still require design to demonstrate example of Nox Relationships


Basis operator Quat() is not exported to gdnative

Godot version:
ec110076c
Issue description:
Tried to convert a Basis to a Quat in godot-python, discovered it can't be done.
Steps to reproduce:
gdscript has a Quat constructor from a Basis; Basis itself has an operator Quat that does the proper conversion. But that functionality is not exported to gdnative, so godot-python can't use it.
Discussion from IRC:
<darkstarsystems> I'm trying to get a quaternion from a basis in python. In gdscript you can do Quat(basis), and the godot code has basis::operator Quat() to do it. But I can't see where that's exposed in gdnative.
<karroffel> darkstarsystems: oh, seems like you found a hole in the API
<karroffel> In C++ most of the math types are custom implemented anyway, but you are correct that they are missing
<darkstarsystems> karroffel, I see -- I'll file an issue. File it against godot itself?
<karroffel> Yes exactly
<darkstarsystems> I'd be happy to look at fixing it if there's a good sample to follow
<karroffel> The fix itself is simple
<karroffel> the hard thing is to make it work in a non breaking way
<karroffel> Because GDNative uses "layers" of structs with function pointers to keep compatibility, now the API is fixed and to fix it you would have to add in a new GDNative core extension, which has never been done yet, only for NativeScript so far

Also just discovered with karroffel that there's no Quat(euler) constructor in gdnative.

Also missing from GDNative is a Transform(Quat) constructor, which certainly exists in GDScript but isn't exported, so also not usable from any of the current bindings.

I'll add get_quat and set_quat to Basis. They're supposed to be there, just incompleteness. There're rot-scale constructors which set both at once are also missing and need to be added

#20757 This should be added as well

Also the recent Dictionary change (#21012) should be considered


check if cdr is destroyed

Bug fix checking if cdr is destoryed

Hi @zolekm
Thank you for PR. I cannot reproduce the issue on my side and you changes look promising but the CI build was failed with the following error because the ChangeDetectorRef doesn't have the destroyed property:
Error at /home/travis/build/akserg/ng2-dnd/src/abstract.component.ts:196:28: Property 'destroyed' does not exist on type 'ChangeDetectorRef'.

Can you fix that issue yourself or provide the code snippet to reproduce it on my side?

Has there been any further progress on this?  and will it be merged into master soon?
Thanks!

Hi @zolekm,
I have fixed the issue in your code.
Thank you for PR.


Share sheet helper (and possibly other shortURL methods) intermittently returns large, non-short URLs

I have successfully integrated the Branch iOS SDK and I'm using the share sheet helper to create and share links. The problem I am facing is that I am occasionally getting very long share URLs (large number of query parameters) when I should be receiving the short URL. If I restart my app and try sharing again it usually works and I get given a correct short URL. Is this a known issue?

Hey @mwkas - How frequently are you seeing this? This should occur on a very rare occurrence. We fallback to generating a long URL with all relevant parameters when there are connectivity issues. Are you testing on a solid connection?
One option is to up the retryCount https://github.com/BranchMetrics/ios-branch-deep-linking/blob/master/Branch-SDK/Branch-SDK/Branch.h#L407 or networkTimeout https://github.com/BranchMetrics/ios-branch-deep-linking/blob/master/Branch-SDK/Branch-SDK/Branch.h#L414.
Please share your thoughts.

Yeah I assumed it was a safe fallback if the shortening request fails. During the couple of days of integration and testing I'd seen it about 4ish times. All on a very good connection.

Where are you based if you don't mind me asking? Also, try bumping up those counts I linked to above and let me know if you still observe it.

We're based in London. I'll up the counts and see if I still experience the issue. Thanks!

Going to close for the time being.

@aaustin This exact problem is still occuring with me. Before initializing in the App Delegate, I am setting branch.networkTimeout to 50.0 and branch.setMaxRetries to 50. I am also on a very good connection. The weird thing is, This bug is only happening in one spot in our application where we are setting 30+ Metadata keys in our BranchUniversalObject.

Hey @JPROFFICIAL - Thanks so much for the note. I have a hypothesis that it's related to one of your keys. Is it possible that you call 1. setDebug before initializing to turn on logging, 2. attempt to create this link and 3. send us the logs (you can start a new ticket at integrations@branch.io).
One hypothesis I have is that our request to one of your URLs could be timing out. Our link creation process will attempt to scrape your $desktop_url, $fallback_url or $canonical_url to pre-fill some open graph tags. If this URL times out when we crawl it, it will significantly delay (and maybe time out) the link creation process. We can check if this is it once you send us an example link POST body.


refactor!: Rename TimeAnalysisPlot to TimeSeriesLinePlot.

The new name better describes the plot.
BREAKING CHANGE: The name of edvart.report_sections.timeseries_analysis.TimeAnalysisPlot was changed to edvart.report_sections.timeseries_analysis.TimeSeriesLinePlot.
Resolves #52.

Please add BREAKING CHANGE footer.

Done


UI: Current profile progress screen

Would it be possible, similar to how it is done on the original BrewPi interface, to add a screen that shows how far along in a profile the brew is? Going to the profile page gives me dates as if I was starting the profile at that point in time, but I would be interested in seeing how close to the next setpoint I am at any given time, to aid in monitoring ("Oh look, the temp should start increasing tomorrow morning")
Sorry for the issue flood :)

Have we had any progress on this issue? Any chance to include in the upcoming release?

I'll add on to this from the perspective of waiting to start raising temperature until the fermentation starts to slow- I know its "around" 7 days, but I wait until I see it, then edit the profile. Knowing where in the profile it is (in terms of day 1, day 4, day 7, etc) helps make it easier.

You’re going to make me do actual work, aren’t you.😳
I’ll see if I can get these going.


Use of delete in calibratesensor.cpp/.h

All occurences of the scalar delete in the calibrated sensor code should be replaced by delete[] (vector delete). The bahaviour is otherwise undefined since the arrays were allocated by new float[..].
/Chris

Thanks for reporting it!

Sorry, for not submitting it as a PR, but my code simply differs too much from the original version.
/Chris


scrollbar-gutter not effective on body

https://github.com/argyleink/open-props/blob/main/src/extra/normalize.css#L42
probably should just remove it. need to investigate further.

@argyleink Not sure if this fits your use case, but thought I'd mention that scrollbar-gutter needs to be applied to :root (i.e. html)


network performance - update the model file urls to amazon s3

@Honry I am trying to improve the model loading speed, so updated the model file urls to Amazone S3, please help to merge.
It spent near 80s to download full mobilenet v2 weight and bias files from github.io in my network environment.

@BruceDai, we can also replace the urls in webnn-polyfill and webnn-native to improve the performance.

@Honry  @BruceDai please do not replace webnn-polyfill and webnn-native untill notifying me.


Change u32assert2 error code type to u32

Currently u32assert2 operation uses Felt as a type of its error code. It should be changed to u32 since error code could be represented only by u32 value.

Closed by #1382.


Migrate configuration files motion control v4 to v5 erzelli02

I followed the steps from 1 to 4 in the document that @julijenv sent me to migrate the configuration files.
In the commit messages there are the sed commands (credits @drdanz).
13b2c7c was not in the guide, @randaz81 should I remove it?
I skipped the step 5 and 6, which problem can I encounter?
Please review code.

We successfully run the yarprobotinterface, we will open another PR for the PID tuning.
Merging.


uncaught error: utxos.map is not a function

Heya, getting the following error. It worked for a while, then started throwing these.
Also curious: I managed to mint one block successfully back in the beginning, but everything since has been failures to win seeking the block, like dozens. Is that normal? No idea :)
Anyway, the error:
~/.git/fortuna$ deno task cli mine
Task cli deno run --allow-all miner/main.ts "mine"
error: Uncaught (in promise) TypeError: utxos.map is not a function
    return Promise.all((utxos as any).map(async (utxo: any) => {
                                      ^
    at Kupmios.kupmiosUtxosToUtxos (https://deno.land/x/lucid@0.10.1/src/provider/kupmios.ts:224:39)
    at Kupmios.getUtxos (https://deno.land/x/lucid@0.10.1/src/provider/kupmios.ts:92:17)
    at eventLoopTick (ext:core/01_core.js:183:11)
    at async Command.actionHandler (file:///home/mei/.git/fortuna/miner/main.ts:61:28)
    at async Command.execute (https://deno.land/x/cliffy@v1.0.0-rc.3/command/command.ts:1948:7)
    at async Command.parseCommand (https://deno.land/x/cliffy@v1.0.0-rc.3/command/command.ts:1780:14)
    at async file:///home/mei/.git/fortuna/miner/main.ts:401:1
I tried restarting the instance but to no avail.

update: it seems to have resolved itself. Maybe a bug in kumios I guess

I'm still having this error sporadically:
error: Uncaught (in promise) TypeError: utxos.map is not a function
return Promise.all((utxos as any).map(async (utxo: any) => {
^
at Kupmios.kupmiosUtxosToUtxos (https://deno.land/x/lucid@0.10.1/src/provider/kupmios.ts:224:39)
at Kupmios.getUtxos (https://deno.land/x/lucid@0.10.1/src/provider/kupmios.ts:92:17)
at eventLoopTick (ext:core/01_core.js:183:11)
at async Command.actionHandler (file:///C:/Users/xx/Downloads/fortuna-main/fortuna-main/miner/main.ts:121:28)
at async Command.execute (https://deno.land/x/cliffy@v1.0.0-rc.3/command/command.ts:1948:7)
at async Command.parseCommand (https://deno.land/x/cliffy@v1.0.0-rc.3/command/command.ts:1780:14)
at async file:///C:/Users/xx/Downloads/fortuna-main/fortuna-main/miner/main.ts:417:1

Hello,
I having the same error too:
eturn Promise.all((utxos as any).map(async (utxo: any) => {
at Kupmios.kupmiosUtxosToUtxos (https://deno.land/x/lucid@0.10.1/src/provider/kupmios.ts:224:39)
at Kupmios.getUtxos (https://deno.land/x/lucid@0.10.1/src/provider/kupmios.ts:92:17)
at eventLoopTick (ext:core/01_core.js:183:11)
at async Command.actionHandler (file:///home/x/fortuna/miner/main.ts:105:28)
at async Command.execute (https://deno.land/x/cliffy@v1.0.0-rc.3/command/command.ts:1948:7)
at async Command.parseCommand (https://deno.land/x/cliffy@v1.0.0-rc.3/command/command.ts:1780:14)
at async file:///home/x/fortuna/miner/main.ts:401:1

Talked to a few and they all (including me) have the error now constantly.

Me too....
On Wed, Aug 30, 2023, 12:00 PM adanorthpool @.***>
wrote:

Talked to a few and they all (including me) have the error now constantly.
—
Reply to this email directly, view it on GitHub
https://github.com/aiken-lang/fortuna/issues/16#issuecomment-1699540098,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE5EXEE2WO2LOHEAWA2FYSDXX5WR3ANCNFSM6AAAAAA4DWNADA
.
You are receiving this because you commented.Message ID:
@.***>

Is this related to Kupo perhaps?

Same issue here, constantly.

this happens when kupo is not responding correctly on demeter. Some people have started running their own instance. You can always run a private instance on demeter or host it locally.


Add email_document_supertype > announcements > world_news_story

The subscriber lists in email-alert-api have an
email_document_supertype of announcements, but
we're sending this through as 'other' for these
document types. This means we don't match those
subscriber lists and some people wouldn't get
notified if we switched to sending through
email-alert-api instead of govuk-delivery. This
fixes that.
For example:
$ rake email_topics:check[5f101654-7631-11e4-a3cb-005056011aef]

{ ..., :email_document_supertype=>"other", ... }

additional govuk-delivery topics:
UKGOVUK_12298
UKGOVUK_2332
UKGOVUK_3055

$ govuk_app_console email-alert-api
> SubscriberList.find_by(gov_delivery_id: "UKGOVUK_3055")

#<SubscriberList id: 3797, ..., email_document_supertype: "announcements", ...>

@gpeng updated.


Proposal for the improvement of  element

Example: Aurelia-KendoUI plugin uses this form:
<require from="kendo-ui/styles/web/css/kendo.materialblack.min.css"></require>

If I could change the value of the from attribute at runtime, that would allow me to change the application's "look and feel" at a click of a say radio-button. Of course, it is developer's responsibility to use this feature in a consistent fashion.

We aren't going to make this dynamic because require in html is designed to work like import in JavaScript, in that it is statically analyzable.
For your scenario, a better solution would be to create a custom element, perhaps called kendo-theme and give it a theme property. Then it can simply use the Aurelia loader to download the specified theme whenever the property changes and inject it into the head. It can also remove the previous theme, if present. Something like this:
import {Loader, bindable} from 'aurelia-framework';

@inject(Loader)
export class KendoTheme {
  @bindable theme;

  constructor(loader) {
    this.loader = loader;
  }

  themeChanged(newValue) {
    this.loader.loadText(newValue).then(text => {
      //remove old style element from head
      //add new style to head, using the text for the css
    });
  }
}

This code is not tests, but you should get the general idea. Because you are loading the css as text through the loader, it also means that the kendo css can be bundled, if desired.
The only caveat to this method is that there may be a flash of unstyled content while the theme downloads. We are planning a way to queue up async composition requests in the near future that would hold back rendering until the async work is complete. This could be used to solve that problem, by ensuring that the styles are downloaded before the composition is complete.

Need just a bit more guidance as my use case is:
File app.html
<template>
    <require from="bootstrap/css/bootstrap.css"></require>
    <require from="kendo-ui/styles/web/css/kendo.common.min.css"></require>
    <require from="kendo-ui/styles/web/css/kendo.materialblack.min.css"></require>

    <require from="./nav-bar"></require>

  	<nav-bar router.bind="router"></nav-bar>

  	<div class="page-host">
        <router-view></router-view>
	</div>
</template>

As the actions take place withing the <template> element I do not see how to do the part
      //remove old style element from head
      //add new style to head, using the text for the css

to achieve the equivalent of replacing the value of the from attribute on the third <require> element above.

First, you wouldn't include the third css require at all. If you include that, that tells Aurelia to add it to the head  automatically, which is not what you want.
To implement the body of the method above, you can use our DOM helper.
let styleElement = DOM.injectStyles(styleText);

That will create a style element with the text you provide, add it to the head of the document and then return the created style element back to you.

Perfect - thank you Rob!


alternative methods to improve code readablity

Hey, I hope I'm not offending. I was going through your code and felt like your usage of indexOf could be replaced with more explicit and appropriate methods like 'includes' and 'startsWith'. Let me know if you are actually using the index. It took me a while to understand you weren't using the index so thought to make it more explicit and use better alternatives.

startsWith is a new feature of ES2015, so it doesn't have a good compatibility.

Ah, I didn't think of that.

That's ok. There is a polyfill in MDN
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/startsWith#Polyfill
But this could bring more code. And I want this library smaller.


Migrate to Kotlin DSL and Versions Catalog

This is an enhancement PR, which does the following:

Migrate build files from groovy to kts
Use versions catalog to setup dependencies
Introduce dokka, ktlint, spotless and detekt plugins
Updates gradle wrapper version
Doesn't update the dependencies versions (might need to do that in a different PR)

Good read to understand the changes on this PR - https://proandroiddev.com/using-version-catalog-on-android-projects-82d88d2f79e5

Good read to understand the changes on this PR - https://proandroiddev.com/using-version-catalog-on-android-projects-82d88d2f79e5

Need to figure out a fix for this error
Failed to transform kotlin-compiler-embeddable-1.7.10.jar

I also need to migrate the signing configurations here as well 😄

Good read to understand the changes on this PR - https://proandroiddev.com/using-version-catalog-on-android-projects-82d88d2f79e5

Need to figure out a fix for this error
Failed to transform kotlin-compiler-embeddable-1.7.10.jar

Fixed this build issue (needed me to update ktlint plugin version)


Release 0.6.0 of simtools

Lot of changes have been implemented since the last release (0.5.1, in April 2023 !!) and we will make quite some changes in the next weeks with the introduction of the new simulation model names.
I suggest therefore the release 0.6.0, which is simply an intermediate release with no specific major milestones (unless someone wants to make the effort and list all major points - don't think it is necessary).
You should be able to see the release draft in https://github.com/gammasim/simtools/releases - let me know if you want to change anything. I would otherwise like to release tomorrow.

Released.


Failed to compile python 2.7 32-bit on Linux Ubuntu 64-bit

I can compile the 64-bit version of python without error, but I am still having trouble compiling the 32-bit version
CMAKE_C_FLAGS=-m32
CMAKE_EXE_LINKER_FLAGS=-m32
Cmd
cmake -DCMAKE_C_FLAGS=-m32 -DCMAKE_EXE_LINKER_FLAGS=-m32 -DCMAKE_INSTALL_PREFIX:PATH=${HOME}/python2.7/python-install ../python-cmake-buildsystem

I do not understand what the problem, any help ?
-- SRC_DIR: /home/user/python2.7/Python-2.7.11
-- PY_VERSION: 2.7.11
-- The system name is Linux
-- The system processor is x86_64
-- The system version is 3.16.0-46-generic
-- Could NOT find BZip2 (missing:  BZIP2_LIBRARIES) (found version "1.0.6")
-- Could NOT find Curses (missing:  CURSES_LIBRARY CURSES_INCLUDE_PATH)
-- Could NOT find EXPAT (missing:  EXPAT_LIBRARY) (found version "2.1.0")
-- Could NOT find OpenSSL, try to set the path to OpenSSL root folder in the system variable OPENSSL_ROOT_DIR (missing:  OPENSSL_LIBRARIES OPENSSL_INCLUDE_DIR) (Required is at least version "0.9.7")
-- Could NOT find TCL (missing:  TCL_LIBRARY TCL_INCLUDE_PATH)
-- Could NOT find TCLTK (missing:  TCL_LIBRARY TCL_INCLUDE_PATH TK_LIBRARY TK_INCLUDE_PATH)
-- Could NOT find TK (missing:  TK_LIBRARY TK_INCLUDE_PATH)
-- Could NOT find ZLIB (missing:  ZLIB_LIBRARY ZLIB_INCLUDE_DIR)
-- ptmx: //dev/ptmx ptc: HAVE_DEV_PTC-NOTFOUND
-- Checking for XOPEN_SOURCE
-- Checking for XOPEN_SOURCE - yes
-- Checking for Large File Support
-- Checking for Large File Support - yes
-- Using unsigned short for unicode [Py_UNICODE_SIZE: 2]
--
-- The following extensions will NOT be built:
--
--     cmath (not set: HAVE_LIBM)
--     crypt (not set: HAVE_LIBCRYPT)
--     datetime (not set: HAVE_LIBM)
--     time (not set: HAVE_LIBM)
--     atexit (not set: IS_PY3)
--     _codecs (not set: IS_PY3)
--     faulthandler (not set: IS_PY3)
--     _opcode (not set: IS_PY3)
--     _operator (not set: IS_PY3)
--     _pickle (not set: IS_PY3)
--     _sre (not set: IS_PY3)
--     _stat (not set: IS_PY3)
--     _symtable (not set: IS_PY3)
--     _testbuffer (not set: IS_PY3)
--     _testimportmultiple (not set: IS_PY3)
--     _testmultiphase (not set: IS_PY3)
--     _tracemalloc (not set: IS_PY3)
--     _weakref (not set: IS_PY3)
--     xxlimited (not set: IS_PY3)
--     xxsubtype (not set: IS_PY3)
--     zipimport (not set: IS_PY3)
--     nis (not set: HAVE_LIBNSL)
--     errno (not set: IS_PY3)
--     _posixsubprocess (not set: IS_PY3)
--     _scproxy (not set: APPLE HAVE_LIBSYSTEMCONFIGURATION)
--     _msi (not set: WIN32)
--     msvcrt (not set: MSVC)
--     nt (not set: WIN32)
--     _subprocess (not set: WIN32)
--     _winreg (not set: WIN32)
--     overlapped (not set: WIN32 IS_PY3)
--     _winapi (not set: WIN32 IS_PY3)
--     _socket (not set: HAVE_LIBM)
--     _bsddb (not set: DB_INCLUDE_PATH DB_LIBRARIES)
--     bz2 (not set: BZIP2_LIBRARIES)
--     _curses_panel (not set: CURSES_LIBRARIES PANEL_LIBRARIES HAVE_PANEL_H OR HAVE_NCURSES_PANEL_H)
--     _curses (not set: CURSES_LIBRARIES)
--     dbm (not set: NDBM_TAG GDBM_LIBRARY GDBM_COMPAT_LIBRARY)
--     gdbm (not set: GDBM_INCLUDE_PATH GDBM_LIBRARY GDBM_COMPAT_LIBRARY)
--     _hashlib (not set: OPENSSL_INCLUDE_DIR OPENSSL_LIBRARIES)
--     readline (not set: READLINE_INCLUDE_PATH READLINE_LIBRARY CURSES_LIBRARIES HAVE_READLINE_READLINE_H)
--     _sqlite3 (not set: SQLITE3_LIBRARY)
--     _ssl (not set: OPENSSL_INCLUDE_DIR OPENSSL_LIBRARIES)
--     _tkinter (not set: TCL_LIBRARY TK_LIBRARY TCL_INCLUDE_PATH TK_INCLUDE_PATH X11_INCLUDE_DIR X11_LIBRARIES)
--     zlib (not set: ZLIB_LIBRARY ZLIB_INCLUDE_DIR)
--
CMake Error: The following variables are used in this project, but they are set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake files:
HAVE_LIBM
    linked by target "libpython-static" in directory /home/alex/python2.7/python-cmake-buildsystem/cmake/libpython
HAVE_LIBUTIL
    linked by target "libpython-static" in directory /home/alex/python2.7/python-cmake-buildsystem/cmake/libpython

-- Configuring incomplete, errors occurred!
See also '/python2.7/python-build/CMakeFiles/CMakeOutput.log".
See also "/python2.7/python-build/CMakeFiles/CMakeError.log".

Thanks for the report.
You should consider setting CMAKE_CXX_COMPILER_ARG1 and CMAKE_C_COMPILER_ARG1
(( Note: I just realized there are not documented in CMake ... I will work on that ))
Otherwise there is a CMake linux32-bit toolchain available here: https://github.com/thewtex/cross-compilers/blob/master/linux-x86/Toolchain.cmake
For reference, the build of Python 2.7.11 and 3.5.1 using various toolchain are tested here:

See section "CircleCI [Latest]" on https://open.cdash.org/index.php?project=CPython
For linux Python 2.7 on 32-bit and look for platform 2.7.11-i686-linux-gnu_#130_a2606f9

Have you solved your problem?
I ran into the same problem when compiling slicer
Can you share your solution?
thank you

I ran into the same problem when compiling slicer

I suggest you post questions related to Slicer on the forum, see http://discourse.slicer.org/

Have you solved your problem?

Python can easily be cross compiled to 32-bit using the python-cmake-buildsystem.
You need to make sure to have the 32-bit system libraries installed on your system.
Here is an example using the dockcross linux-x86 image:
Configure
$ dockcross-linux-x86 cmake -Bbuild -H. -GNinja
-- The C compiler identification is GNU 4.9.2
-- The ASM compiler identification is GNU
-- Found assembler: /usr/i686-linux-gnu/bin/i686-linux-gnu-gcc
-- Check for working C compiler: /usr/i686-linux-gnu/bin/i686-linux-gnu-gcc
-- Check for working C compiler: /usr/i686-linux-gnu/bin/i686-linux-gnu-gcc -- works
[...]
-- PY_VERSION: 3.5.2
[...]
-- The system name is Linux
-- The system processor is i686
-- The system version is 1
[...]
-- Checking ABIFLAGS - m
-- Checking SOABI - cpython-35m-i386-linux-gnu
[...]
-- Configuring done
-- Generating done
-- Build files have been written to: /work/build

Build
$ dockcross-linux-x86 ninja -Cbuild
ninja: Entering directory `build'
[436/436] Relocate _sysconfigdata.py and update pybuilddir.txt

Check
file ./build/bin/python
./build/bin/python: ELF 32-bit LSB executable, Intel 80386, version 1 (SYSV), dynamically linked, interpreter /lib/ld-linux.so.2, for GNU/Linux 2.6.32, BuildID[sha1]=6a881f38302c0629178c77aed6becfbc4325fecb, not stripped


Tuto quick start command line for java invalid

What

Some discrepancies between tuto java options and actual jdk java option (also update version)

Why

This should not puzzled experienced users but some newbie with java could fill this as a showstopper

Refs

https://developers.rsk.co/quick-start/step1-install-rsk-local-node/

Merged two change in next because I found why RSKj was not launching websocket server

New complete PR is #591


Implementation of the Lego Spike Prime/Robot Inventor (Big Hub) in the connector program

The Lego Spike Prime/Robot Inventor (Big Hub) should be implemented in the openroberta-connector program, so that it is possible to connect the robot with the openroberta-lab and upload programs from the openroberta-lab onto the robot.

[x] connector program discovers robot
[x] openroberta-lab connection works
[x] program upload works

released


Rename ambientWeatherHandler.ts

Since it handles more than just ambientWeather :D Maybe call it weatherDataHandler.

Done as part of another PR.


Rendering overlay issue

https://mclo.gs/9zGlXb3
how do I fix this?

Not a Geckolib issue.
net.minecraftforge.fml.config.ConfigFileTypeHandler$ConfigLoadingException: Failed loading config file infernalexp-client.toml of type CLIENT for modid infernalexp


migrate to janusgraph service

this pattern uses the now deprecated graph service, it needs to be upgraded to use compose for janusgraph https://www.ibm.com/blogs/bluemix/2017/10/retirement-of-ibm-graph/

this is done: https://github.com/IBM/procurement-analysis-with-wks/commit/1bfe7d86bced9d8d410a0649fc87cd34141dfab3


Update doctest library to version 2.3.5

Relates to this PR

Thanks again for keeping us up-to-date

No problem :)


sync: Added support for syncing notary/cosign signatures, closes #261

Signed-off-by: Petu Eusebiu<EMAIL_ADDRESS>
What type of PR is this?
feature

Which issue does this PR fix:
#261
What does this PR do / Why do we need it:
Added support for syncing notary and cosign signatures
If an issue # is not available please add repro steps and logs from IPAMD/CNI showing the issue:
Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades. Has updating a running cluster been tested?:
no
Does this change require updates to the CNI daemonset config files to work?:

Does this PR introduce any user-facing change?:

no

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

will be updated with tests, just wanted to see code coverage


[improve][broker] Support topic-level migration from blue cluster to green cluster

PIP: #xyz

Motivation
Currently, Pulsar supports blue-green cluster migration at both the cluster level and the namespace level. However, sometimes there are many topics under a single namespace, and we do not want to migrate all topics in that namespace to the green cluster at the same time, as this may introduce risks. Therefore, we would like to introduce topic-level migration functionality to have more granular control over the migration process.
Modifications
Add an admin API and command for enabling migration at the topic level.
Verifying this change
Added test in: org.apache.pulsar.broker.service.ClusterMigrationTest#testTopicMigration
Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[x] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/hrzzzz/pulsar/pull/12

I will add the relevant pip later


Stop working close/minimize/maximize buttons in window control in Opera

After 4.2.35 update

Stop working close/minimize/maximize buttons in window control.

at Opera browser 56.0.3051.99
and last Opera browser 58.0.3135.53
tested.
Go desktop sample and try it.

In onDragStop(pos, el) windows callback
el === undefined

in all browsers

Thanks. Problem found in draggable component.

OK. the metro great!
I tryed connect you by Skype some days ago.
Please, go online.

I will fix it soon. For now, you can specify data-drag-element = ".window-caption .icon, .window-caption .title"

and add next style to improve:
.window-caption .icon {
    width: 36px;
    height: 36px;
    text-align: center;
    flex-shrink: 0;
}

.window-caption .icon * {
    line-height: 36px;
    max-height: initial;
}

.window-caption .title {
    width: 100%;
}


State loading issue

Hi thank you for sharing your work,
While trying to replicate your results i faced following problem:
in train_rcnn.py, i uncommented eval() function and trained for 50 epochs (both frcnn and mrcnn), however while trying to load states, i got the errors below.

For your reference below are the links for the latest checkpoints:
https://drive.google.com/file/d/128HDnr5P3CvzX_HBKWdSG-6kMpS4DwjA/view (frcnn)
https://drive.google.com/file/d/12BKIH_zLeP69t4RHGZk7NbMgykLlVDH8/view (mrcnn)
Can you please help,
Thank you for your time

import torchvision
from torchvision.models.detection.faster_rcnn import FastRCNNPredictor
num_classes = 2
model = torchvision.models.detection.fasterrcnn_resnet50_fpn(pretrained=True)
in_features = model.roi_heads.box_predictor.cls_score.in_features
model.roi_heads.box_predictor = FastRCNNPredictor(in_features, num_classes)
model.to(device)
checkpoint = torch.load('path to check point', map_location='cpu')
model.load_state_dict(checkpoint['model'])
model.eval()
clear_output()

I solved this by model.load_state_dict(torch.load(model_path))


Added style-loader npm dependency

style-loader library is not part of webpack anymore

Thx for your contribution.
The style-loader was never part of webpack core and is not required to use this library. You could also import the CSS as JS string and do something else with it. Especially when using the sass-loader in conjunction with the extract-text-webpack-plugin, the style-loader must not be in your loader chain.


Allow for Other Logger Features to supply level

From @martindsouza on July 27, 2013 1:10
The "[Other Logger Functions]https://github.com/tmuth/Logger---A-PL-SQL-Logging-Utility/wiki/Logger-API#wiki-other-logger-procedures) don't allow for the level to log them  at.
This can be an issue in certain situations. For example log_apex_items will only be logged when the current level is set to "APEX" (or 128). Developers may want this functionality when an error occurs in production (which usually runs using the "ERROR" logger level).
Implementation ideas: Not exactly sure how to implement this but odds are a new parameter, p_level (optional) will allow developers to call with a custom level. If defined, it will ignore the default settings and log based on the level.
Copied from original issue: tmuth/Logger---A-PL-SQL-Logging-Utility#29

Part of APEX and timer re-writes


[stable/percona-xtradb-cluster] Fixes #23207

Signed-off-by: Rafael Rivero<EMAIL_ADDRESS>Is this a new chart
No, it's a fix for issue #23207 for chart stable/percona-xtradb-cluster.
What this PR does / why we need it:
Adds support for serviceAccount and securityContext parameters, to allow the chart to run on Openshift.
Which issue this PR fixes
Issue #23207
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @maver1ck

/ok-to-test
This looks great! Ideally we'd follow the pattern of optionally creating the service account if it doesn't already exist ( .Values.serviceAccout.create: true|false )  but this is defintely a useful fix as is.

/lgtm


Unable to run in product due to regeneratorRuntim

In order to get this running in production I had to add @babel/polyfill to the projects dev dependencies alongside adding import '@babel/polyfill'; to the top of `src/index.js'. As guided by https://stackoverflow.com/questions/33527653/babel-6-regeneratorruntime-is-not-defined
Has anyone else come across this? Is this a viable solution and should I submit a PR?

We're not supporting async/await yet - and it's not a bug. It's just how it was configured :) I see no reason why not to add the babel settings to support it. So You can create a PR - sure :)


content min-height

Is there any reason why on markup.scss there is a min-height of 600px for the content ID ?
And how to remove that on child theme ?

Yes, that's because if you have a site with just line of content then the footer will stuck in the middle of the screen.
You can simply override this in your child:
#content {
  min-height: auto;
}

A user created a sticky footer plugin: https://github.com/norico/BtScore-Footer-Adjust
This plugin pushes the footer always to the bottom of the browser window even you have just one line of content and have disabled the min-height. So, maybe this is what you are looking for.
Does this help?

An alternative for this, is a height 100% on html and body, and use flex to push the footer down. But that is something we could consider for the theme itself.

Thanks for the help !

I reopened this issue because as @justinkruit mentioned, we can add this to the theme. It's better than the min-height.
https://github.com/bootscore/bootscore/issues/29 @norico do you want to do that?


A simple command-line interface to show quotes

A simple command-line interface to show quotes in the terminal.

Moved off this repository as the scope of the project became larger than initially anticipated. It now resides in its own repo here https://github.com/Shresht7/quotes-cli.
P.S. had some great (read terrifying) adventures with git filter repo when I was moving this branch over trying to retain the git history.


VSCode does not surface errors resulting from SetDataBreakpoints requests.

VSCode Version: 1.38
OS Version: Ubuntu 18.04

At the moment, VSCode seems to assume that a successful return from DataBreakpointInfoRequest automatically means that setting a data breakpoint on that variable will succeed.   It ignores the verified flag returned in SetDataBreakpointsResponse, nor does it show the attached message.
This is very unfortunate, because in native code, there is a multitude of reasons why data breakpoint creation may fail, and it isn't really possible to predict the outcome in advance.
Would be nice to have some way of surfacing failures in SetDataBreakpoints  to the user.

Please try it out with VSCode insiders tomorrow and let us know how it behaves.
Thanks!

Confirmed working in 2019-09-23 insider build.

Thanks for letting us know, adding verified label.


Error

It says "link is not valid"
How i can fix it?

You need to enter url of page/website you want to redirect in index.js


Upgrade to hyper-rustls 0.27/rustls 0.23

Sticks with ring as the default crypto provider for rustls for now (see #2136).
This is also a prerequisite for re-enabling a Quinn-based H3 backend, since Quinn is skipping rustls 0.22.

Thanks so much! I do have some questions, to help plan how we could allow using the different crypto providers.

If no provider is enabled, what happens? I tried removing "ring" from the dependencies in this PR locally, and it still builds...

Oh I see, I just ran the simple example, it panics are runtime.

I suppose we could introduce rustls-tls-ring and rustls-tls-awslc features eventually.

Thanks so much! I do have some questions, to help plan how we could allow using the different crypto providers.
* If no provider is enabled, what happens? I tried removing `"ring"` from the dependencies in this PR locally, and it still builds...

  * Oh I see, I just ran the simple example, it panics at runtime.

Yes. To be safe at compile time, we could use the builder_with_provider() API and pass rustls::crypto::ring::default_provider() explicitly instead.

* I suppose we could introduce `rustls-tls-ring` and `rustls-tls-awslc` features eventually. We don't need to do so yet, but is there anything to take into consideration?

Those features would be non-additive, because rustls will panic if both are enabled. Also note that people might want to use a third-party crypto provider -- this is of course already possible via the preconfigured_tls() path.

Those features would be non-additive, because rustls will panic if both are enabled.

Oh, so reqwest would be picking the crypto provider for the application as a whole? If any other library depends on rustls and doesn't disable the default features, will the application start to see panics when they upgrade reqwest?

The ClientConfig::builder() indirectly calls into here:
https://github.com/rustls/rustls/blob/main/rustls/src/crypto/mod.rs#L254
This will:

If a process-wide crypto provider has already been installed, use it
If no provider has been installed and only one has been enabled via Cargo features, use it and install it as the default
Otherwise, panic

As such, I think the idea is that reqwest should rely on this and if people run into a run-time panic they should call CryptoProvider::install_default() first before setting up reqwest's Client/ClientBuilder.
See more discussion in https://github.com/rustls/rustls/issues/1877.

Won't it be more reasonable, to prefer one crypto to another instead of panicking at runtime?
Or maybe a compile_error! is also better than runtime panic.

Feel free to contribute to the discussion in the upstream issue.

I would like to suggest a change to be able to setup a custom provider without having to compile both ring and the custom provider.

do not activate the ring feature on hyper-rustls, rustls and tokio-rustls
activate the ring feature only if rustls-tls feature is enabled. rustls-tls = ["rustls-tls-webpki-roots", "rustls/ring"]
optionally, add another feature that uses rustls but don't activate the ring feature. rustls-tls-providerless = ["rustls-tls-webpki-roots"]. This is optional as users could use the rustls-tls-webpki-roots directly

FWIW, this PR was just intended to do the rustls upgrade while making as few other changes as possible. Obviously there could be a number of new Cargo features that help control the crypto provider used by rustls. But I think reqwest will want to pick a default crypto provider to make adopting rustls as easy as possible.

My suggested changes still pick up ring as a default provider without adding any new feature, and leaving the possibility to control  the crypto provider used by rustls. That's why I think it's interesting to consider this now. This can also be, of course, in a follow-up PR as you suggest.

Any chance this can be merged & new reqwest version sliced out? We're a bit dependent on that now since opted for Rustls 0.23 for a new project. I guess I'll have to fork for now.

I do want this to merge. As to the status of this upgrade: I'm hesitant to merge and publish since anyone upgrading that was already using rustls v0.23 may suddenly start seeing a panic, because reqwest enables the alternative backend. (And I don't really want to "fix" that by switching to aws-lc.)
My feeling is that I'd want to wait for a rustls version that doesn't have that issue. Discussion is in https://github.com/rustls/rustls/issues/1877.

@seanmonstar I think we can work around that issue by using ClientConfig::builder_with_provider(), explicitly passing whichever provider we have selected into the config builder explicitly. This should never panic, independent of which Cargo features other libraries have selected.

I've revised this to explicitly select a ring (using the existing Cargo features) or aws-lc-rs (using a parallel set of Cargo features) and use the selected crypto provider explicitly when constructing a rustls ClientConfig. This setup should also enable preventing a dependency on either ring or aws-lc-rs by using a new rustls-base feature (renamed from __rustls) -- the latest release depends on ring (via rustls) unconditionally so even though one could wire up a preconfigured rustls ClientConfig using aws-lc-rs or another crypto provider, there would still be a ring dependency.
This is motivated in part by rustup concerns for making rustls the default TLS implementation, since some scenarios need P521 cipher suites which only aws-lc-rs supports.

So the MSRV build fails with this error:
djc-2021 rustls-0.23 reqwest $ cargo +1.63 c
error: Package `reqwest v0.12.4 (/Users/djc/src/reqwest)` does not have feature `rustls`. It has an optional dependency with that name, but that dependency uses the "dep:" syntax in the features table, so it does not have an implicit feature with that name.

Not entirely sure what's up with that? From looking at the state of Cargo.toml after these changes I don't think there are any references to a rustls feature left?

fwiw just running cargo c on this branch with rust 1.63 results in the same error (ubuntu 24.04 container)

hyper-rustls 0.27.1 also bumps msrv from 1.63 to 1.64

From my look into this PR, I think TlsBackend can't have a Default impl if only rustls-base is enabled, since the TlsBackend::Rustls variant indicates that either aws-lc-rs or ring has been selected, and requires the call to default_provider() for one of the backends.
In that scenario it is required that a user pass in a a rustls-compatible clientconfig so that the BuildRustls variant is set on the client.
Would it make sense to set UnknownPreconfigured as the default in that case? That could lead to confusing error messages, though, so maybe a new variant UnsetRustls or something could be created in order to tell users that rustls was enabled but no backend was configured

I think these compile issues could have been caught by CI if the new features were added to the github actions workflow, also

Alright, I've returned to this. I do want to get an upgrade out. I think the recent review comments do catch some typos, which would be good to fix.
As I understand the code now, this will explicitly use the crypto provider enabled with features, and will not ask for the "default" from rustls. The downsides that still exist for releasing this upgrade are:

Users who have an app already depending on rustls 0.23, default features, and are using ClientConfig::builder() or ServerConfig::builder() will start to see a panic when pull this in. (It seems like those constructors are mostly a bad idea to use, in general.) It might appear like reqwest's "fault", but there's not much we can do there.
It looks like the features within reqwest are not additive. If someone enables the rustls-tls-aws-lc-webpki-roots feature, and another dependency has enabled rustls-tls (which still implies ring), all reqwest::Clients will use the ring provider. Would we need to provide builder methods to pick at runtime?

I see that a simple PR with rustls 0.23 has been merged. I would still be interested in more configurability here. When upgrading to rustls 0.23 i've migrated my other dependencies from ring to aws-lc-rs, and I will be providing an aws-lc-rs ClientConfig to reqwest in my application, so I would like to avoid compiling ring if possible

As mentioned, a subset of this has been merged in #2299.
You can open an issue about aws-lc features, if you'd like to discuss. One thing that's important to me in reqwest is that features be additive, and we maintain backwards compatibility.

Note that with the merged PR you can still use aws-lc-rs by installing a default provider before initializing the Client.


[Bug] Long pressing library button restarts phone

Device information

Tachiyomi version: 2.0.13
Android phone & version: Honor 6X, Android 7.0

Steps to reproduce

Long press library twice

Expected behavior
Open up search in library tab
Actual behavior
Restarts phone immediately without any prior warning
Other details
Additional details and attachments.

The library bottom button?

The library bottom button?

Yup, that's the one

Will look into the bug but either way that doesn't open the library search

I see, thank you!
On Sun, May 31, 2020, 6:45 PM Carlos<EMAIL_ADDRESS>wrote:

Will look into the bug but either way that doesn't open the library search
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CarlosEsco/Neko/issues/225#issuecomment-636541620,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APZHJI7YNSQZKGDQEVEAGWLRULMXNANCNFSM4NPJLD5Q
.

i am unable to replicate this.  if you can replicate it please reopen and attach a video


Should we have a general class for Mushroom Body Output Neuron (MBON)

see Aso2014 FBrf0227178

FBbt:00047953

It is FB_2019_01. There has been a bit of a gap between FBbt releases, as FB_2018_06 version was early in the cycle (early October) and FB_2019_01 version was late in the cycle (mid January).


Mode selection on homekit

If it's possible i would like to have the option to select the Mode of the dehumidifier in Homekit. In Homekit under the dehumidifier target there is already a wheel with some options that i thing in some way they might be changed to the different modes, right now the options are automatic(only turns on), humidifier(does nothing), dehumidifier(same as automatic), and deactivate, stops  the dehumidifier.
I also seen in the logs that if i change the mode in the app it reads the value in homebridge, so the only thing missing I think it's creating this link somehow. I also have seen that the configuration already has the mode, defined.
If I have experience with this I will try to do it my self, but I don't know where to start, rather than copying one of the sliders and linking the positions to the modes, might not be the best approach.
Could some one implement that

The short answer is no.
The longer answer... I consider this a bug in the Apple Home user interface... it is presenting auto and humidify options to the user when it should not, because the user interface is not paying attention to the validValues for target state.  The HomeKit API has concept of current/target state, this can be humidify, dehumidify or auto.  Because this is a dehumidifier the only valid current/target state is dehumidifying or off/idle.  It is not valid for either to be humidifier.  To prevent errors being logged if Apple Home attempts to request the device to start humidifying, the plugin sets the only valid value as being dehumidifier.
Okay, so the user interface is ignoring that setting.  But under the covers it is not.  If you select auto or humidify then the UI accepts it, but immediately adjusts it to dehumidify and the plugin receives a request to turn on (or off) the dehumidifier.  The only values coming through to the plugin are 1 or 0 representing on or off.  The plugin is not receiving requests to set target state to auto/humidify/dehumidify because it can only ever be dehumidify, thus there is no way to know if the user selected auto, humidify or dehumidify.
Could this be changed.  Maybe.  But it would likely open up a can of worms.  The plugin would have to register as both a humidifier and dehumidifier... changing what are considered validValues for current/target state.  In fact during development I had that, but then changed it because it caused other issues that caused error messages to log because the device is only a dehumidifier.
If you have a homebridge development environment and want to have a go...

Make it user configurable by adding a setting to device specific config.
When selected, change the valid values for TargetHumidifierDehumidifierState here
Update the target state handler for set
Not sure what you should do with the target state handler for get, this is the can-of-worms.  I don't know how HomeKit will behave if this always returns dehumidifying (as it should).
More can-of-worms likely in current state handler for get, as it probably does need to pretend to be humidifying when it is not if you want the Apple Home user interface to show the state as humidifying (when it is not).

I think you are trying to fit a square peg in a round hole and in my opinion it is not worth trying.

I recommend closing this issue as not planned.  iOS 17.2 fixes the bug so auto and humidify options will not show any more.


CPPKI: Complete Control-Plane PKI

The control plane PKI needs to be completed. Currently, TRCs are trusted as there are no cross-signatures yet. Also, certificate revocation needs to be closely looked at (currently we rely on auto-expiration).
This is a larger issue and needs to be broken down in smaller chunks as we get to it.

Some previous notes regarding the CPPKI, linked from https://github.com/scionproto/scion/issues/1500. They are quite outdated by now, but might still be somewhat useful:

TRC Design by @Oncilla: https://docs.google.com/document/d/1T5UMI6IZldaFXMpPYCmCgVqGWGc4O3jYewWHYejE6eA/edit?usp=sharing
Trust Verification Specification by @scrye: https://docs.google.com/document/d/1Nfw8RuF73x18fMefKwoPRHzenqbZDH0BlhV-p2xZNhY/edit?usp=sharing
Notes on validation by @scrye: https://docs.google.com/document/d/1X_mIfuXqF3fx8_2BT0-Lyu-eS33JOFa6irWtj_BDGTw/edit?usp=sharing

As the PR route of collaborating on the design doc is a bit cumbersome we move the discussion and collaboration
from #2464
to stackedit.io.
Open https://drive.google.com/file/d/1LNB20SxY3hHt_sTSLmj8mHcd-T_al8kW/view?usp=sharing using stackedit.io to collaborate

We have come a long way since the creation of this issue.
The new CPPKI design was announced in #3784


vimPlugins.project-nvim

Project description
The superior project management solution for neovim.
Metadata

homepage URL: https://github.com/ahmedkhalf/project.nvim
source URL: https://github.com/ahmedkhalf/project.nvim
license: apache 2.0
platforms: all

Isn't this already in the repository?

Oh yeah, how weird I coldn't find it the other day.


Fix usage of custom EVP_CIPHER objects

If a custom EVP_CIPHER object has been passed to EVP_CipherInit() then it
should be used in preference to a fetched cipher.
We also fix a possible NULL pointer deref in the same code for digests.
If the custom cipher passed to EVP_CipherInit() happens to use NID_undef
(which should be a discouraged practice), then in the previous
implementation this could result in the NULL cipher being fetched and
hence NULL encryption being unexpectedly used.
CVE-2022-3358
Fixes #18970
Note that this has been assessed as a low severity security issue. As per our policy, since it is low, it is being fixed in public and will be included in the next release.
We also fix a bug where the cleanup function for an EVP_CIPHER was not always being called when it should, which was highlighted by writing the test for the original issue.

Feedback addressed.

Yes nice please share

This pull request is ready to merge

Needs an indentation fix before merge.

Pushed to master and 3.0. I fixed the indentation issue during merge. There was a trivial conflict cherry-picking to 3.0 that I also resolved during the merge.


fix: suspicious logic to update triggerValue

Describe your changes
This PR includes a fix to a suspicious logic I found in event service.
https://github.com/rodekruis/IBF-system/blob/c377b55f4e4e3074a4031bc13a96627128a676cd/services/API-service/src/api/event/event.service.ts#L861-L894
The function checks for an updated triggerValue and assigns it the eventArea object.
But it does not push this to the database UPDATE query.
Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] I have added tests wherever relevant
[ ] I have made sure that all automated checks pass before requesting a review

Notes for the reviewer
I have not found the symptoms of this error. I found the logic suspicious so made the fix. If it's difficult to identify any symptoms I understand it will be difficult to review. I think it's better to have the fix and deal with those potentially unintended consequences rather than ignore this.

> test:api:all
> node --expose-gc node_modules/.bin/jest --config=jest.api.config.js --runInBand --detectOpenHandles --logHeapUsage

ts-jest[versions] (WARN) Version 4.9.5 of typescript installed has not been tested with ts-jest. If you're experiencing issues, consider using a supported version (>=2.7.0 <4.0.0). Please do not report issues in ts-jest if you are using unsupported versions.
 PASS  test/email/floods/email-uga-floods.test.ts (33.836s, 79 MB heap size)
  Should send an email for uga floods
    ✓ default (8205ms)
    ✓ warning (8680ms)
    ✓ warning-to-trigger (7948ms)
    ✓ no-trigger (8264ms)

 PASS  test/email/flash-flood/email-mwi-flash-flood.test.ts (28.872s, 80 MB heap size)
  Should send an email for mwi flash flood
    ✓ default (7937ms)
    ✓ no-trigger (20792ms)

 PASS  test/email/drought/email-uga-drought.test.ts (14.417s, 80 MB heap size)
  Should send an email for uga drought
    ✓ triggered in january (6092ms)
    ✓ non triggered any month (8243ms)

 PASS  test/email/floods/email-ssd-floods.test.ts (15.398s, 81 MB heap size)
  Should send an email for ssd floods
    ✓ default (7953ms)
    ✓ no-trigger (7351ms)

 PASS  test/email/dengue/email-phl-dengue.test.ts (16.898s, 81 MB heap size)
  Should send an email for phl dengue
    ✓ default (8476ms)
    ✓ no-trigger (8332ms)

 PASS  test/email/malaria/email-eth-malaria.test.ts (19.629s, 79 MB heap size)
  Should send an email for eth malaria
    ✓ default (10298ms)
    ✓ no-trigger (9238ms)

 PASS  test/email/typhoon/email-phl-typhoon.test.ts (10.016s, 88 MB heap size)
  Should send an email for phl typhoon
    ✓ default (9778ms)

Test Suites: 7 passed, 7 total
Tests:       15 passed, 15 total
Snapshots:   0 total
Time:        139.545s, estimated 147s
Ran all test suites.