Correct value of --hirid-data-root?

Hi,
Thank you for providing this public resource! We are attempting to run the data preprocessing and are running into parquet errors. Specifically, after we follow steps 2 + 3 in the downloading data instructions in the readme, and run
python icu_benchmarks/run.py preprocess --hirid-data-root BASE_DIRECTORY_WE_DOWNLOADED_TAR_GZ_FILES_INTO
--work-dir MY_OUTPUT_DIR
--var-ref-path ./preprocessing/resources/varref.tsv
--split-path ./preprocessing/resources/split.tsv
--nr-workers 8
we run into the following error
OSError: Could not open parquet input source 'BASE_DIRECTORY_WE_DOWNLOADED_TAR_GZ_FILES_TO/observation_tables/observation_tables_index.csv': Invalid: Parquet magic bytes not found in footer. Either the file is corrupted or this is not a parquet file.
I assume this is because we are somehow passing in the wrong value of hirid-data-root, since these are indeed CSVs, not parquet files. But we are not sure what path to use instead. Using BASE_DIRECTORY_WE_DOWNLOADED_TAR_GZ_FILES_INTO/observation_tables/ or BASE_DIRECTORY_WE_DOWNLOADED_TAR_GZ_FILES_INTO/observation_tables/parquet/ also yields errors.

This error appears to be caused by the fact that the version of observation_tables we downloaded has a subdirectory, parquet, which the code does not seem to expect. We got this to run by adding parquet to the path in several lines in run.py - if you can confirm this is correct, we can open a PR or something.

Hi @epierson9,
Thanks for using our work and catching this!
Indeed the file organization used to not have a hirid_root/observation_tables/observation_tables_index.csv file but directly all the parquet files corresponding as hirid_root/observation_tables/part-XX.parquet.
I need to check with HiRID authors about the change just to be sure. If it's confirmed a PR would be great thanks!

perfect, let me know! thanks for the quick reply :)

Hi,
Just wanted to check in on this - was my fix correct? Would you like me to issue a PR?
Best,
Emma

Hi Emma,
Sorry for the late reply!
I can confirm the file organization has changed; thus, if you have time a PR would be great!

Great, thanks! Hm, for some reason I'm not allowed to add a branch to the repo, which I think I need to do to issue a PR - sorry if I'm doing something wrong, or maybe it's a permissions issue on your end...?
(default_virtualenv) emmapierson@Emmas-MacBook-Pro-3 icu_benchmarks % git push origin pierson_fixing_paths
ERROR: Permission to ratschlab/HIRID-ICU-Benchmark.git denied to epierson9.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

Hi Emma,
You indeed can't directly add a branch to the public repo.
To open a PR you need to first "fork" the repo, thus have your version, and then submit a PR from your forked repo to this repo.
See:

https://docs.github.com/en/get-started/quickstart/fork-a-repo#propose-changes-to-someone-elses-project
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork

I'll also add a CONTRIBUTING.md file as this was not clear, thanks for raising it!


Chore: Update peer dependencies

Related PR: https://github.com/grafana/scenes/pull/295
What changed?
Updated the @grafana/* peer dependencies to the latest released version (10.2.3). (Please let me know if this shouldn't be done yet for some reason. I came across this while trying to update the scaffolded dependencies in the @grafana/create-plugin tool.)

@leventebalogh seems like main's broken with this PR?


Why CircleCI badge report failure when it is not so?

@obviousrebel , do you know why CircleCI badge on the repo README is red even when https://circleci.com/gh/spacetelescope/notebooks/43 (the latest merge commit at the time of writing) shows green?

It failed the build and check job but passed the deploy job. Both of these jobs are in one workflow so if one fails, the overall job is failing.

Hmm is there a way to force it to re-run the whole thing for a green badge? Or more importantly, did it deploy the right stuff, given the "build and check" one failed?

It shouldn't start the deploy stage when the build fails, I'll have to do some investigating. If it did, it doesn't remove anything. Just adds/overwrites changed files so the gh-pages branch should be fine as it would still have the html files from the previous commit.

Looks like the deploy job 43 was actually for build_and_check job 39. The workflows must have been triggered fairly close together. So the workflow containing the jobs 39 and 43 passed, but the latest workflow containing job 41 failed giving the failed status badge.
tldr:
Everything is working as it should. Deploy job only runs if build_and_check is successful, badge shows latest workflow status for master branch.

How do we make it green again? Is something broken?

I mean, not the deploy process, but is one of the notebooks broken?

It is green currently as the last two workflows have passed. The last failure was because a cell timed out in the align_mosaics notebook on a network call.


Add support for CTRL + ALT + special character

This PR is an attempt to implement/fix #6851 again (hopefully without any regressions):
By removing the accept_event call all shortcuts which are using the exact same keys do still work. I could not notice any regressions on Windows 10, but further testing is required. Note that if a shortcut consists of Ctrl + Alt + [some key used for a special character], the character input is handled AND the command associated to the shortcut is executed.
Testing/feedback is welcome :)

Superseded by #56695.

@akien-mga
Im still getting this issue using the alpha6 version.
Attaching video, of course you cannot see what I'm actually pressing but Im pressing left option key + 2 which in Swedish keyboard layout is @, reproduced in 4.0.alpha6 on a intel 16" MacBook Pro running macOS Monterey 12.3.1.
https://user-images.githubusercontent.com/22000931/162617450-7a1939ff-feb3-43a1-a06a-201de99c04ff.mov


[Feature Request] Add API to specify dialTimeout, readTimeout, writeTimeout

Currently we have to modify the source code to control network operation timeouts.
But It will be great to specify timeouts within beanstalk client itself, so that we could be free from mantaining our patches.

@spacewander do you hage a fork with timeouts implemented?

@tomponline
Yes, we have a private fork of beanstalk.
Instead of just hardcode 10 seconds timeout duration, we add a new API called DialTimeout.
We also introduce other modifications.

@spacewander cool, I'm working on a public fork here, but would be good to see how you've approached it. I've just pushed a read timeout too on reserve-with-timeout command.

@spacewander ive approached this by recording a netConn property inside the main Conn struct that is an interface that supports the necessary timeout functions off net.Conn, but can still be mocked for unit testing without having to implement the entire net.Conn feature set. I could rename this to netConnTimeout or something like that.

@spacewander im pretty happy with this PR now, at least for read timeouts.


Change sender email and name for all emails

Current behavior
The emails (1. registration 2. welcome 3. change password 4. invite user) have sender email and name as follows:
Hikaya<EMAIL_ADDRESS>To Reproduce
Steps to reproduce the behavior:

Registration email: Register a new account
Welcome email: open registration email and confirm email
Change password: request change password
Invite email: Organization settings > people > invite users

Expected behavior
The emails (1. registration 2. welcome 3. change password 4. invite user) have sender email and name as follows:
Hikaya team<EMAIL_ADDRESS>

Hello can I work on this

Hello @ninetteadhikari @andrewtpham can I work on this?

hi @EUGINELETHAL its great to see your interest! Have you been able to setup Activity locally? Please let me know if you have any issues.
In terms of tickets I would suggest this one: https://github.com/hikaya-io/activity/issues/467
It should be simple enough for a start. Please let me know if you have any questions!


'cd' command doesn't work.

[X] This issue is not about OPSEC or bypassing defensive products
[X] I have followed the steps in the Troubleshooting section

OS and version: ubuntu 20.04
Python version: 3
Nim version: 1.6
Using Docker: No

Issue Description
The cd command doesn't work. Directory does not change.

Screenshots

Hey, thanks for yet another report! 🙏
Interesting issue. I recall I've tested this to be working before, so it may have to do with something that changed in the underlying Nim libraries (?). Will definitely investigate for v1.2.
In the meantime, you can use cd \ to switch to the root directory of the current drive, but this may still cause issues with multiple drives.

Hey! This behavior was due to a path normalization bug in the Nim stdlib. I have removed normalization in https://github.com/chvancooten/NimPlant/commit/bef008f9f57c3ca7721d626e416aad15a362e62d, which doesn't seem to affect the cd command at all since it's just one OS anyway.
Note: To switch drives, the correct syntax is cd c:\ (with the trailing backslash) as it is with regular Windows. cd c: does not cause a path change.


Updated ProductID

Updated the MMA ProductID
"Microsoft Monitoring Agent
Product ID is: 6D765BA4-C090-4C41-99AD-9DAF927E53A5"

@jennyhunter-msft,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


registration returns wrong status code

curl -i -H "Accept: application/json" -X POST -d<EMAIL_ADDRESS>http://localhost:7667/api/v1/users
HTTP/1.1 200 OK
X-Powered-By: Express
Access-Control-Allow-Origin: *
Access-Control-Allow-Headers: Origin, X-Requested-With, Content-Type, Accept
Content-Type: application/json; charset=utf-8
Content-Length: 276
ETag: W/"114-a0285961"
Date: Wed, 08 Apr 2015 12:49:18 GMT
Connection: keep-alive

{"message":"User validation failed","name":"ValidationError","errors":{"password":{"properties":{"type":"required","message":"Path `{PATH}` is required.","path":"password"},"message":"Path `password` is required.","name":"ValidatorError","kind":"required","path":"password"}}}

This should definitely be no 200.

Also:
curl -i -H "Accept: application/json" -X POST -d<EMAIL_ADDRESS>http://localhost:7667/api/v1/users
HTTP/1.1 200 OK
X-Powered-By: Express
Access-Control-Allow-Origin: *
Access-Control-Allow-Headers: Origin, X-Requested-With, Content-Type, Accept
Content-Type: application/json; charset=utf-8
Content-Length: 62
ETag: W/"3e-2b7b975d"
Date: Wed, 08 Apr 2015 13:01:15 GMT
Connection: keep-alive

{"message":"registered user","_id":"552526992e59ceff1007045f"}~

And:
HTTP/1.1 200 OK
X-Powered-By: Express
Access-Control-Allow-Origin: *
Access-Control-Allow-Headers: Origin, X-Requested-With, Content-Type, Accept
Content-Type: application/json; charset=utf-8
Content-Length: 81
ETag: W/"51-57742a22"
Date: Wed, 08 Apr 2015 13:07:40 GMT
Connection: keep-alive

{"name":"BadRequestError","message":"User already exists with email<EMAIL_ADDRESS>


VideoMediaReceived does not get called when AudioVideoPlaybackBot joins a meeting

I am using the latest version of AudioVideoPlaybackBot  to join a meeting. In my case I do not see the VideoMediaReceived  getting called. I looked into the code and CallHandler() constructor has this line
this.Call.GetLocalMediaSession().VideoSockets.FirstOrDefault().VideoMediaReceived += this.OnVideoMediaReceived;
However video on my case comes on socket #3, so shouldn't the code be
       foreach (var videoSocket in this.Call.GetLocalMediaSession().VideoSockets)
        {
            videoSocket.VideoMediaReceived += this.OnVideoMediaReceived;
        }

Please let me know if my understanding is correct or not.

Yes I believe it should be the test case. For me the code is something like this

                this.videoSockets = this.mediaSession.VideoSockets?.ToList();
                foreach (var videoSocket in this.videoSockets)
                {
                    videoSocket.VideoMediaReceived += this.OnVideoMediaReceived;
                }


CPP: Fix function pointer/lambda related false positives in 'Resource not released in destructor'

Add some tests and fix several types of false positives in this query related to function pointers and lambdas.  I've also renamed some things in the query as the terminology used had become inconsistent.

This is going to conflict with https://github.com/Semmle/ql/pull/580, when that PR reaches ql-master.  I'm happy to fix that when it comes up.


not picking up specs wrapped in context or additional describe blocks

given the spec file:
RSpec.describe MyModule do
  context 'Some context' do
    describe '.myInstanceMethod' do
      it 'should equal foo' do
        expect(MyModule.myInstanceMethod).to eq 'foo'
      end
    end
  end
  context 'Some other context' do
    describe '.myInstanceMethod' do
      it 'should not equal foo' do
        expect(MyModule.myInstanceMethod).not_to eq 'foo'
      end
    end
  end
end

or even given:
RSpec.describe MyModule do
  context 'Some context' do
    describe MyModule.myInstanceMethod do
      it 'should equal foo' do
        expect(MyModule.myInstanceMethod).to eq 'foo'
      end
    end
  end
  context 'Some other context' do
    describe MyModule.myInstanceMethod do
      it 'should not equal foo' do
        expect(MyModule.myInstanceMethod).not_to eq 'foo'
      end
    end
  end
end

Problem:
The plugin doesn't seem to append the specifications to the docs.
Expectation
The plugin should both find these specs (including the string variants in the first example) and respect the context of each spec in the Specifications outline:
.myInstanceMethod
Specifications:

Some context

should equal foo

Some other context

should not equal foo

Other relevant information:
$ ruby -v
#=> ruby 2.3.0p0 (2015-12-25 revision 53290) [x86_64-darwin16]
$ rspec -v
#=> 3.5.4
$ yard -v
#=> yard 0.9.8

# .yardopts
--exclude lib/task/*
--plugin rspec
-r README.md
-
CODE_OF_CONDUCT.md
LICENSE

My file structure is:

lib/my_module.rb
spec/lib/my_module_spec.rb

and is being invoked via rake task:
YARD::Rake::YardocTask.new do |t|
  t.files         = ['lib/**/*.rb', 'spec/**/*_spec.rb']
  t.stats_options = ['--list-undoc']
end

this also doesn't work if you use describe in place of context in either of my examples as well.

This plugin doesn't support nested definitions. When this was written, that feature was not available in RSpec.

ah, ok so less of a bug more of a feature request.

@aaronmallen PRs accepted, I don't really update this repository anymore. If you want to take over maintainership of yard-rspec, I'm all for it.

@Isegal not to pass the buck, but it definitely seems that offer would be better extended to:

@ogeidix
@zetaben
@Jesus
@nikhgupta

They all seem to have done a lot of great work on top of yours already and might be a little more qualified for the task.

I'd be interested in doing some updates to this plugin, I'd take the offer to be a contributor.

I'm sorry to bother again, but ping @lsegal 😅


bump dwn-sdk-js, dwn-sql-store and @web5/dids

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.43%. Comparing base (3436b21) to head (ce59147).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #119   +/-   ##
=======================================
  Coverage   94.43%   94.43%
=======================================
  Files          25       25
  Lines        2247     2247
  Branches      299      299
=======================================
  Hits         2122     2122
  Misses        124      124
  Partials        1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Unbound: Info text for field ""Register DHCP Static Mappings" is outdated (due to UI changes)

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
In the OPNsense admin UI (in 24.7.1) the info text for the input field "Register DHCP Static Mappings" under "Services: Unbound DNS: General" is outdated. The domain is no longer set in "System:General Setup", but in "System: Settings: General". (Luckily the link still works, so this is essentially a cosmetic isssue.)
To Reproduce
Steps to reproduce the behavior:

In the OPNsense admin UI go to 'Services: Unbound DNS: General'
Click on the "info" icon next to field 'Register DHCP Static Mappings' to show the info text for that field.
See outdated text

Expected behavior
Replace "System:General Setup" with "System: Settings: General" in the info text.
Describe alternatives you considered
n/a
Screenshots

Relevant log files
n/a
Additional context
n/a
Environment
OPNsense 24.7.1-amd64

@bernhardkohlhaas thanks, found a couple of others too, renamed in https://github.com/opnsense/core/commit/e37e67ebcde65abd35b9e33c2256168ce838ce38


Fix typo found by codespell

Hi @DimitriPapadopoulos , thanks for your PR, could you re-format your commit message by following conventional commits please ? This is required to pass the CI, I'll add this to the contributing guidelines soon.

Ok I'm not familiar with the GH interface, but it stills create a fork in your personal space, so I guess you could amend your commit and force push

Note that commitlint itself has a bug, it reports two errors where there can be only one (probably the latter):

heavy_multiplication_x   subject may not be empty [subject-empty]
heavy_multiplication_x   type may not be empty [type-empty]

Indeed, I guess it is not able to detect the type/subject as long as there is no ':' in the commit message.

All good, thanks for your contribution!


[Bug] Hearts 2 Recipe

Describe the bug
There is no recipe for Hearts 2
In NEI (JEI) There is no recipe for Hearts 2. It does show some information but nothing on how to create the plasmid for hearts 2.

The recipe existed, it just didn't show in EMI. Fixed now, ty


Update distributing process

add the right way to copy necessary dlls from mingw*/
add the right way to copy share/ and lib/ from mingw*/
add a minor remind about libadwaita

Related to:

#57

Looks good. Thanks again for figuring all of this out!


Align color to depth

|---------------------------------|------------------------------------------- |
| Camera Model                       | { Realsense D435 depth camera} |
| Operating System & Version |   {Win 10}  |
| Platform                                 | PC |
| Language                            | {matlab, python } |
I have some pairs of depth and color images aquiered from a d435 Realsense depth camera , and I want to align the color image to the depth image.
I have the intrinsics and extrinsics values of the camera i used.
Do you know how can I acomplish this?

If you want to align color with depth using Python, a guide published by Intel may be useful to you.
https://github.com/IntelRealSense/librealsense/blob/jupyter/notebooks/distance_to_object.ipynb

[Realsense Customer Engineering Team Comment]
Hi @alt01,
To clarify:
You have your own extrinsic between your own color camera (not D435 color camera) and D435 depth camera?

No, I am using the D435 depth and color camera, and I have the intrinsics and extrinsics values:
Depth intrinsics: width: 424, height: 240, ppx: 214.063, ppy: 120.214, fx: 213.177, fy: 213.177, model: Brown Conrady, coeffs: [0, 0, 0, 0, 0]
Color intrinsics: width: 424, height: 240, ppx: 214.031, ppy: 117.083, fx: 307.266, fy: 307.39,model: Brown Conrady, coeffs: [0, 0, 0, 0, 0]
depth to color extrinsics: rotation: [0.999966, 0.00585115, -0.00579111, -0.00586071, 0.999982, -0.00163449, 0.00578144, 0.00166838, 0.999982]
translation: [0.0146822, 0.0002266, -3.92719e-05]

color to depth extrinsics: rotation: [0.999966, -0.00586071, 0.00578144, 0.00585115, 0.999982, 0.00166838, -0.00579111, -0.00163449, 0.999982]
translation: [-0.0146833, -0.000140612, -4.59912e-05]

depth_scale: 0.0010000000475.
I have a depth and color images aquired by the camera, and I want to allign the depth image to the color image. The code Im using is:
depth_point = rs.rs2_deproject_pixel_to_point(depth_intrin, depth_pixel, depth_scale)
color_point = rs.rs2_transform_point_to_point(depth_to_color_extrin, depth_point)
color_pixel = rs.rs2_project_point_to_pixel(color_intrin, color_point)
The code obtains the point cloud of the scene with the depth camera coordinates ("rs.rs2_deproject_pixel_to_point"). Then I change from the depth camera coordinates to the color camera coordinates (with the "rs.rs2_transform_point_to_point"). Finally, I project the last poincloud using the color camera intrinsic values. At the end I should have a 424x240 depth image alligned to the color image but instead I have a 521x292 image.

[Realsense Customer Engineering Team Comment]
Hi @alt01,
If you like to re-project depth to color pixel, need to modify as follows.
depth_point = rs.rs2_deproject_pixel_to_point(depth_intrin, depth_pixel, depth value * depth_scale)
Did you also try align sample?

[Realsense Customer Engineering Team Comment]
Hi @alt01,
Did you try "align.m" in matlab to see how it works and fulfill your requirement?

I dont know how to use the function specified on align.m, it seems to be a method from a class.
I already have the color and depth streams on matlab, but im having troubles with the align function.

[Realsense Customer Engineering Team Comment]
Hi @alt01,
See #2710 to fix the error in align method. Then, you can try the process function like below code to align depth to color.
align_to = realsense.stream.color;
alignedFs = realsense.align(align_to);
.....
 fs = pipe.wait_for_frames();
 aligned_frames = alignedFs.process(fs);

Is it possible to align a depth and color images that i have stored on my computer?
This is because I took some samples (depth and color images) with a D435 depth camera, and I need to work with this samples.
I have the intrinsic and exrinsic values of the camera.

@alt01  This discussion linked to below may be useful to you:
https://github.com/IntelRealSense/librealsense/issues/2534

[Realsense Customer Engineering Team Comment]
Hi @alt01,
See #2710 to fix the error in align method. Then, you can try the process function like below code to align depth to color.
align_to = realsense.stream.color;
alignedFs = realsense.align(align_to);
.....
 fs = pipe.wait_for_frames();
 aligned_frames = alignedFs.process(fs);

This is exactly what i'm looking for, however I get the following error message when executing the lines above:
"Error using realsense.align/process (line 17)
Expected input number 2, frame, to be a scalar.
Error in realsense.depth_example_stream_testingalign (line 41)
aligned_frames = alignedFs.process(fs);
"
Just like alt01 I'm on D435 and have the depth and color streams running. Any idea what can be the issue? I was expecting align/process to be happy with a frameset...

[Realsense Customer Engineering Team Comment]
Hi @alt01,
See #2710 to fix the error in align method. Then, you can try the process function like below code to align depth to color.
align_to = realsense.stream.color;
alignedFs = realsense.align(align_to);
.....
 fs = pipe.wait_for_frames();
 aligned_frames = alignedFs.process(fs);

This is exactly what i'm looking for, however I get the following error message when executing the lines above:
"Error using realsense.align/process (line 17)
Expected input number 2, frame, to be a scalar.
Error in realsense.depth_example_stream_testingalign (line 41)
aligned_frames = alignedFs.process(fs);
"
Just like alt01 I'm on D435 and have the depth and color streams running. Any idea what can be the issue? I was expecting align/process to be happy with a frameset...

Hello apachebalou!
Did you solve the problem?
I have the same problem

[Realsense Customer Engineering Team Comment]
Hi @alt01,
See #2710 to fix the error in align method. Then, you can try the process function like below code to align depth to color.
align_to = realsense.stream.color;
alignedFs = realsense.align(align_to);
.....
 fs = pipe.wait_for_frames();
 aligned_frames = alignedFs.process(fs);

This is exactly what i'm looking for, however I get the following error message when executing the lines above:
"Error using realsense.align/process (line 17)
Expected input number 2, frame, to be a scalar.
Error in realsense.depth_example_stream_testingalign (line 41)
aligned_frames = alignedFs.process(fs);
"
Just like alt01 I'm on D435 and have the depth and color streams running. Any idea what can be the issue? I was expecting align/process to be happy with a frameset...

Hello apachebalou!
Did you solve the problem?
I have the same problem

Hi sebawski,
Actually I just commented away validateattributes check in align.m and it worked. I dont understand the root cause, but seems to work at least.

[Realsense Customer Engineering Team Comment]
Hi @alt01,
See #2710 to fix the error in align method. Then, you can try the process function like below code to align depth to color.
align_to = realsense.stream.color;
alignedFs = realsense.align(align_to);
.....
 fs = pipe.wait_for_frames();
 aligned_frames = alignedFs.process(fs);

This is exactly what i'm looking for, however I get the following error message when executing the lines above:
"Error using realsense.align/process (line 17)
Expected input number 2, frame, to be a scalar.
Error in realsense.depth_example_stream_testingalign (line 41)
aligned_frames = alignedFs.process(fs);
"
Just like alt01 I'm on D435 and have the depth and color streams running. Any idea what can be the issue? I was expecting align/process to be happy with a frameset...

Hello apachebalou!
Did you solve the problem?
I have the same problem

Hi sebawski,
Actually I just commented away validateattributes check in align.m and it worked. I dont understand the root cause, but seems to work at least.

Thank you! finally the code worked

Hello,
I am trying to understand and apply your code but I'm really struggling with this.
I manage to have both rgb and depth streaming, now I 'd like to align RGB to this depth data, but I don't know how to process your discussion.
How do you display the expected result : a colorized pointcloud ?
Can you please share a bit more of your code ?

Hi
Does anyone can help me to use built-in alignment feature between. color and depth image in VB.NET? I have seen many examples in python and C but I really haven't seen any VB.NET sample code for image alignment... Any help will be very appreciated!
Thanks.

I am getting the error:
Error using realsense.align
Expected input to be a scalar.
I am passing realsense.align a vector uint8 variable which is the data from the color frame set. I can only pass a number or a realsense.stream into the class so cf is not an option due to it being a video_frame variable. I cannot find any examples of using the align class when not using the stream function.
 % Obtain frames from a streaming device
        fs = pipe.wait_for_frames();

        % Produce pointcloud
        if (depth.logical() && cf.logical())

        % Align depth frame to Color
            cf = fs.get_color_frame();
            dd = cf.get_data();

            align = realsense.align(dd);
        end


Fix dependency install

no non-extra dependencies where installed since merging #50  (so auto builds were failing - generating empty builds for AWS layer for example https://github.com/kensuio-oss/kensu-py/actions ), my quick guess is that required= was a wrongly named keyword.
seem to fix the thingy locally, let's test on AWS layer build.

oh really 🤨... K although weird it worked before...

indeed weird, but I'd expect all what was used in tests been also transitive deps of .[all], and tests didn't actually use HTTP, so these deps were ignored.


Add make and convenience functions for async iterators

This adds a make function and value + done convenience functions for async iterators.
cc @cometkim

Curious about feedback from @cometkim

I expect the producer and consumer of an iterator to work cooperatively.
However, in this example, only the producer can decide to stop, or it must expose its internal state to the outside world. This negates most of the benefits of the abstraction and is also of little use in FFI.
This is why @JonoPrest and I extended the topic in Retreat to include break/continue and generators.
https://github.com/JonoPrest/iterator-rescript/blob/retreat-note/RETREAT_NOTE.md

@cometkim sure, and when that lands that'll be a much better solution. This a little something up until the point that it does land.

I'd like to know your motivation and use case for adding this. I'm not sure if this would be useful without supporting break/continue.
But it's easy to support if we add a small runtime for it.

I'd like to know the motivation and use case for adding this. I'm not sure if this would be useful without break/continue.
But it's easy to support if we add a small runtime for it. e.g. https://github.com/JonoPrest/iterator-rescript/blob/retreat-note/src/Demo.res

There are places where you need to return an actual AsyncIterator. One example is graphql-js subscriptions.

Yes. The binding to iterator types and supporting a factory will be useful for FFIs.

Yes. The binding to iterator types and supporting a factory will be useful for FFIs. I had thought that forEach was newly added here.
I think adding a factory is ok, If the AsyncIterator.t type privides a proper forward compatibility.

I guess it would, no?

@cometkim in your eyes, can we merge this or are there still outstanding questions you want answers to?

Yes. The binding to iterator types and supporting a factory will be useful for FFIs. I had thought that forEach was newly added here.
I think adding a factory is ok, If the AsyncIterator.t type privides a proper forward compatibility.

I guess it would, no?

I think it would, or it can be easily migrated.

forEach doesn't seem useful, but that's not related to this PR. Right?

forEach doesn't seem useful, but that's not related to this PR. Right?

Not related to this PR, no. Why is it not useful though? I use it myself in a few places.

forEach doesn't seem useful, but that's not related to this PR. Right?

Not related to this PR, no. Why is it not useful though? I use it myself in a few places.

As already mentioned, if the iterable is not array-like, its usability is limited without control flow support.
Or it can be (more) useful when combined with additional utilities like take(limit).
https://github.com/tc39/proposal-iterator-helpers/?tab=readme-ov-file#takelimit

Right, got it!


Remaining paging work

The paging PR is merged. Most work of paging is finished. There are two remaining issues.

@jianghaolu suggested add only path parameters to parameter list. Parameters can appear in many places in HTTP request. They can be in path, query, header, etc. Current behavior is only required parameters will be exposed to users. I believe other languages face same problems. Maybe we should align the design across languages.
@jianghaolu, @srnagar, @weidongxu-microsoft, @lmazuel How do you think?

Update Java doc. Current generated code is below. The response body schema does not contain meaningful content. This issue is not specific to paging. It is a common issue in LLC.

    /**
     * A paging operation that must return result of the default 'value' node.
     *
     *
Response Body Schema
     *
     *
{@code
     * BinaryData
     * }

     *
     * @return a DynamicRequest where customizations can be made before sent to the service.
     */
    @ServiceMethod(returns = ReturnType.COLLECTION)
    public PagedFlux<BinaryData> getNoItemNamePagesAsync(RequestOptions requestOptions) {

For (1), I might slightly prefer the option of adding only path parameters (which gives best protection again swagger change). The rest required/optional parameter can go to the javadoc (similar as payload and response model).
Agree that we can just go consistent across all languages.
For (2), I believe Jianghao have plan and code for a improved javadoc on payload/response model, that you will be able to reuse later.


Seems to sync rooms it's not supposed to?

I'm using this to implement a slack<->gitter bridge by using a bot account in Slack and a dedicated GitHub account for Gitter.
So far it works great for syncing two channels together (one on each), but to sync two more channels I'm noticing that the bot will misdirect the message and send it when it's not supposed to.
Here's the setup:

Initially: slackbot in 1 slack channel, A1. gitterbot in 1 gitter channel, A2.
Want to sync another channel (on each network), so I invite the slackbot to another slack channel, B1, and join (as the gitterbot) the other gitter channel, B2 (under the same organization).
I type a message (as my own user) in the new Slack channel (B1), and even though I haven't restarted matterbridge or updated it configuration to recognize either of the B channels, that message gets relayed into A2.

In fact, two messages got relayed into A2 that shouldn't have been:
<gitter> <@U2LRT44TD|gitter> has joined the channel
<greg> test

The first is a notification that I invited the Slack @gitter bot to the new channel B1, and it's sent to A2. The second is the message from B1 sent to A2.
Neither of those messages should have been relayed, as the .toml wasn't even setup to support the new B channels.
Here's the end of my .toml, and I'm using the latest 0.7.0-dev build from master at commit db0e4ba:
[[gateway]]
#OPTIONAL (not used for now)
name="gateway1"
#Enable enables this gateway
##OPTIONAL (default false)
enable=true

    #[[gateway.in]] specifies the account and channels we will receive messages from.
    #The following example bridges between mattermost and irc
    [[gateway.in]]

    #account specified above
    #REQUIRED
    account="slack.groupincome"
    #channel to connect on that account
    #REQUIRED
    channel="groupincome"

    [[gateway.in]]
    account="gitter.groupincome"
    channel="okTurtles/group-income"

    [[gateway.out]]
    account="slack.groupincome"
    channel="groupincome"

    [[gateway.out]]
    account="gitter.groupincome"
    channel="okTurtles/group-income"

Note that the above shows the configuration for A1 and A2.

If it helps, for slack I'm using useAPI=true.

Would it be possible to run with -debug and to reproduce this again? (and paste the debug log)

@42wim I'll give it a shot as soon as I get a moment.

Should be fixed in master, reopen if still an issue.


Autocomplete should not show fields that we are already using

When we are using the auto-complete we should not display id when it's already used, neither should we propose a fragment-spread when it has already been spread.

I can take this one!

Assigned it to you so folks have visibility into it, thank you!

@JoviDeCroock After some debugging, I think this functionality should actually be built it into the graphql-language-service package. The TS language server already handles this and only provides unused fields in their recommendations.
Gonna hack on graphql-language-service to get a PR over to the graphiql repo with that, otherwise the other alternative is to vendor/modify those functions in this repo!

Currently it looks like GraphiQL does not take in account the surrounding fields neither does the function hint at doing so 😅 it mostly seems to take the context of the current typed token in account (or the absence thereof). We might need to filter it ourselves based on the selection-set we are on, or maybe the LSP is smart enough to de-duplicate based on some insertText/... would need to look into that.

@JoviDeCroock opened a PR with a fix for this!


Updated Mongo replica set docs for failover walkthrough

CLA signed

/cc @foxish

@crimsonfaith91 PTAL

@billpratt looks good, thank you. Will merge after comments are addressed.

/ok-to-test

@billpratt Marking a stale. Please update within one week.

@unguiculus  I addressed the code review feedback in 1425ffe40c1c85744e6e4d52c3c7d4b7c9c84161


feat: Enable ProductDescription override in ProductDetails section

What's the purpose of this pull request?
This PR intends to enable override for ProductDescription component and props in ProductDetails section.
How it works?
You can override ProductDescription using __experimentalProductDescription.
How to test it?
In the @faststore/core package:
In packages/core/src/customizations/src/components/overrides/ProductDetails.tsx
change the following code:
const override: SectionOverride = {
  section: SECTION,
  components: {
    __experimentalProductDescription: {
      props: { descriptionData: [{ title: 'Title', content: 'Some content' }] },
    },
  },
}

In the starter.store:

Follow this doc: https://developers.vtex.com/docs/guides/faststore/overrides-component-props;
Check the Overrides migration from V1 to V2 to ensure you are creating the override in the correct way.

Starters Deploy Preview
https://github.com/vtex-sites/starter.store/pull/446
Using the changes of the PR above, you can test by running the store locally (yarn dev) then go to http://localhost:3000/api/preview?contentType=pdp&documentId=4675d165-170d-11ef-8452-12a5e84da56d&versionId=eb329256-7113-4f74-9087-6e0838e77aaa, that's the version using the draft changes I made.

@lucasfp13 can you pls check the error why codesandbox build is failing? It would be great if we could test in the starter too.

hey, @lucasfp13 does descriptionData  support a TSX component, in the Title and Content array?
const override: SectionOverride = {
  section: SECTION,
  components: {
    __experimentalProductDescription: {
      props: { descriptionData: [{ title: 'Title', content: 'Some content' }] },
    },
  },
}

hey, @lucasfp13 does descriptionData support a TSX component, in the Title and Content array?
const override: SectionOverride = {
  section: SECTION,
  components: {
    __experimentalProductDescription: {
      props: { descriptionData: [{ title: 'Title', content: 'Some content' }] },
    },
  },
}

@victors1682 as of right now, the answer appears to be no.
https://github.com/vtex/faststore/blob/09cdd6fbdeacc796d24738e232d660153dade68d/packages/core/src/components/ui/ProductDescription/ProductDescription.tsx#L10-L13
https://github.com/vtex/faststore/blob/09cdd6fbdeacc796d24738e232d660153dade68d/packages/core/src/components/ui/ProductDescription/ProductDescription.tsx#L70


FIRECERT-2084: [FCS] FC report issue fix - Part 1

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Kummithi, Guru Eswar seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Additional Information heading should not show if no additional info is configured [BUG]

Describe the bug
There are no additional options configured on my event but the Additional Information heading is still displaying.
To Reproduce
Steps to reproduce the behavior:

Setup and event with a couple of standard questions
Make sure no additional information (e.g. questions) are setup.  Note that I did have them setup and removed them so the issue may be there if you try to reproduce with a fresh event and cannot reproduce.
Save the event, create an event instance, view the registration page.

Expected behavior
When no additional options configured on an event  the Additional Information heading should not display.
Screenshots

Unable to reproduce and believed that this was resolved earlier.  Closing due to inability to reproduce on current branch.

Was able to recareate

Was able to recreate and found on line 157 on the SummiEventRegisterController:
 if (                     !String.isBlank(eventPage.Add_Info_Question_Type_1__c) ||                             !String.isBlank(eventPage.Add_Info_Question_Type_1__c) ||                             !String.isBlank(eventPage.Add_Info_Question_Type_1__c) ||                             !String.isBlank(eventPage.Add_Info_Question_Type_1__c) ||                             !String.isBlank(eventPage.Add_Info_Question_Type_1__c) ||                             eventPage.Include_Time_frame_List__c ||                             eventPage.Allow_Other_Attendees__c ||                             (eventPage.Ask_Gender__c != 'Do not ask' && eventPage.Ask_Gender__c != null) ||                             (eventPage.Ask_Registrant_Program_Of_Interest__c != 'Do not ask' && eventPage.Ask_Registrant_Program_Of_Interest__c != null)                     ) {                 additionalQuestions = true;
It should read:
            if (                     !String.isBlank(eventPage.Add_Info_Question_Type_1__c) ||                             !String.isBlank(eventPage.Add_Info_Question_Type_2__c) ||                             !String.isBlank(eventPage.Add_Info_Question_Type_3__c) ||                             !String.isBlank(eventPage.Add_Info_Question_Type_4__c) ||                             !String.isBlank(eventPage.Add_Info_Question_Type_5__c) ||                             eventPage.Include_Time_frame_List__c ||                             eventPage.Allow_Other_Attendees__c ||                             (eventPage.Ask_Gender__c != 'Do not ask' && eventPage.Ask_Gender__c != null) ||                             (eventPage.Ask_Registrant_Program_Of_Interest__c != 'Do not ask' && eventPage.Ask_Registrant_Program_Of_Interest__c != null)                     ) {                 additionalQuestions = true;

This is happening particularly when the Question Type has a value, but the Question Text is blank.  When the Question Type has a value, but the text is blank, the question doesn't appear at all.
@tcdahlberg may want to consider updating the login here to reflect the Add_Info_Question_Text.  Otherwise it'll continue, but now with the other questions should there be a value and no Question Text.

Included in beta release <IP_ADDRESS>

Included in production release <IP_ADDRESS>


Bump to Swift 5.9 (by April 2024)

Apple will require Xcode 15/Swift 5.9 to deploy apps to the App Store by April 2024. So at some point then I'll bump min Swift version to 5.9. This is just an issue to remind me of it.
Branch: https://github.com/davdroman/swiftui-navigation-transitions/tree/swift-5.9

I'm late to this but will do by next week.


Combine six landing page sections into a single Lander component and …

Combine six landing page sections into a single landing page and route "/" to it.
Set "About" in the footer to route "/about" page

At this time, the "PROJECTS" nav and "FIND PROJECTS" button (in the original landing-a) are not linking to any page (as the original landing-a). Let me know whether they should be set to "/projects" page or leave the task for the next coming new stories.
Thanks,
Haiyan Huang


re: config merge when input same files on load_program.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

re: #153

2. What is the scope of this PR (e.g. component or file name):

kclvm/sema/src/pre_process/config.rs

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

test_config_merge at kclvm/sema/src/pre_process/tests.rs

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

43 of 52 (82.69%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 59.736%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

kclvm/sema/src/pre_process/config.rs
19
20
95.0%

kclvm/sema/src/pre_process/tests.rs
24
32
75.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
22484

Relevant Lines:
37639

💛  - Coveralls


test-branch/execute-job/quoted-authorization/aa1011d2-ce34-42f7-803e-47c79749c3be

PR for testing 87b84cdc23e55526d46813833fd2d26fb77d6187

/canonical/self-hosted-runners/run-workflows 87b84cdc23e55526d46813833fd2d26fb77d6187


When change left position, dragger isn't accurate

If I make the sliders position to absolute (or the sliders wrapper), and change the left amount, and you start dragging the dragger, it goes to the wrong position. Here's a JSFiddle showing what I mean:
https://jsfiddle.net/rxo8ae86/

Use padding-left → https://jsfiddle.net/tovic/rxo8ae86/1
I don’t have any plan to make this supports the absolute position and such. I will let another developers to do that for us.

http://jsbin.com/jugecupawu/edit?html,output fixed :smile:


Test s3 feature and commands

Reference #48

#50. Merge this before testing.

Testing completed! S3 upload and download works!


toc.insert throws error if  present inside code block

let toc = require('markdown-toc');

sample = `
- old toc 1
- old toc 2
- old toc 3

## abc
This is a b c.
\`\`\`\`

\`\`\`\`

## xyz
This is x y z.
`

console.log(toc.insert(sample).json);

Throws Error: markdown-toc only supports one Table of Contents per file.  Expectation is that the fenced code block would act as an escape.

Hi, I stumbled across this issue (because I had a similar problem): you may use

<!-- toc -->

if supported and syntax highlighting does not matter.

...and it does not throw the error if applied to a markdown file containing only a single TOC start comment and another one somewhere in a fenced code block.
In that case the insert() function will expect the second TOC start comment to be the TOC stop and replace everything between these comments with the generated TOC.


Update macos terminal command UI

Open this page & update UI
https://artbindu.github.io/essential_command/#macos

Done basic update


handle warning in newer pandas version

We currently get a warning with newer pandas versions:
grblogtools\helpers.py:47: FutureWarning: Not prepending group keys to the result index of transform-like apply. In the future, the group keys will be included in the index, regardless of whether the applied function returns a like-indexed object.
To preserve the previous behavior, use

	>>> .groupby(..., group_keys=False)

To adopt the future behavior and silence this warning, use

	>>> .groupby(..., group_keys=True)
  return parameters.groupby("Version").apply(fill_for_version_nosuffix)

I added the suggested group_keys=False to the two affected calls of groupby() to avoid this warning.

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

@simonbowly, do you have any idea why those checks are failing?

They're failing on master as well. It looks like poetry has an issue with isort, and we need to upgrade it: https://github.com/PyCQA/isort/issues/2077

Merging #31 should fix the problem here

I merged #31 and rebased this. All good now


Login in error routes are using the next-auth defaults

For example, if you put in a wrong login code, you get redirected to: https://plutomi.com/api/auth/error?error=Invalid Code
This should be handled by an alert on the same page instead of redirecting.

Added error handling in 14962c173cf528bb3ec78403f9c8631690b62bd5, no custom messages or anything just the error message in an alert()


Ar/add user id to stream object

What does this pull request do? Explain your changes. (required)
Adds a userId field to the stream object, and populates it when user is logged through gmail.
How did you test each of these updates (required)
Added tests + local testing
Does this pull request close any open issues?
https://github.com/livepeer/livepeerjs/issues/488
Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.

@iameli sounds like a great addition. We have two options:

Make the id field on user be the same as the id on the token object
Autogenerate an id on user object, and copy to additional userId field on token object

Which did you have in mind?

@angyangie 2 - we need the ability to revoke access tokens while leaving the user object intact.


SwaggerSchema ReadOnly doesn't work for object properties

The SwaggerSchema readOnly property doesn't look to work correctly with custom objects.
As example here my entity:
public class Brand : BaseEntity
{
    public string Name { get; set; }

    [SwaggerSchema(ReadOnly = true)]
    public Guid LogoId { get; set; }

    [SwaggerSchema(ReadOnly = true)]
    public virtual Blob Logo { get; set; }
}

This is the expected result:
"Brand": {
    "type": "object",
    "properties": {
        "name": {
            "type": "string",
            "nullable": true
        },
        "logoId": {
            "type": "string",
            "format": "uuid",
            "readOnly": true
        },
        "logo": {
            "$ref": "#/components/schemas/Blob",
            "readOnly": true
        }
    },
    "additionalProperties": false
}

This is the actual result:
"Brand": {
    "type": "object",
    "properties": {
        "name": {
            "type": "string",
            "nullable": true
        },
        "logoId": {
            "type": "string",
            "format": "uuid",
            "readOnly": true
        },
        "logo": {
            "$ref": "#/components/schemas/Blob"
        }
    },
    "additionalProperties": false
}

The "readOnly": true is missing in the "logo" field

Your "expected" result is actually incorrect Swagger JSON. This is because the $ref keyword SHOULD NOT be paired with other schema members (nullable, readOnly etc.). See here for more details on how the $ref keyword should be used - the following line being the most relevant here:

Any members other than $ref in a JSON Reference object are ignored

So, with that said, if you want to add "contextual" metadata such as the readOnly flag then you need to "extend" the referenced schema using the allOf keyword. In other words, your "expected" JSON for the logo property in your example, should in fact look like this:
"logo": {
  "allOf": [
    { "$ref": "#/components/schemas/Blob" }
  ],
  "readOnly": true
}

Now, to have Swashbuckle generate this type of Swagger JSON, simply apply the UseAllOfToExtendReferencedSchema setting.

It work great. Really thanks


[BUG] Light/Dark mode for new pages

Description
Light/Dark mode has not been added to new pages

manage repos pages/manage/repos.js
admin section pages/admin/*
profile repo list components/user/UserRepos.js

Screenshots
No response
Additional information
No response

Can you assign this to me?

@sreyaarc We would appreciate that you want to work on this issue but please check the assignees tab to see if someone is already assigned or not.

@ChinmayMhatre Are you working on it? Do you mind assigning this to me?

@sreyaarc it was just assigned less than 1 day ago, please look for another issue that is unassigned as mentioned by @sital002 ! You can learn more on how to contribute in our projects contributing guide https://github.com/EddieHubCommunity/LinkFree/blob/main/CONTRIBUTING.md


Request to Add pkl (Configuration as Code Language) Package to NixOS

Issue Description
Problem
The pkl package, a Configuration as Code language developed by Apple, is not currently available in the NixOS packages.
Desired Outcome
Include the pkl package in NixOS packages.
Package Information

Package Name: pkl
GitHub Repository: apple/pkl

Additional Context
pkl is a Configuration as Code language with rich validation and tooling. Adding it to NixOS packages would benefit users involved in configuration management and development workflows.
Environment Information

NixOS Version: 23.11 or current
Nixpkgs Channel: unstable

Reference Links

NixOS Website
NixOS Package Search - pkl

Thank you for considering this request.

To make it work in NixOS, I followed the steps below (using jpkl).

Downloaded jpkl:
curl -L -o jpkl https://repo1.maven.org/maven2/org/pkl-lang/pkl-cli-java/0.25.2/pkl-cli-java-0.25.2.jar

Made the script executable:
chmod +x jpkl

Checked file type (confirmed as POSIX shell script executable):
file jpkl
jpkl: POSIX shell script executable (binary data)

Checked dynamic dependencies:
ldd ./jpkl
not a dynamic executable

Set up a Nix shell with the appropriate JDK:
nix-shell -I nixpkgs=https://github.com/NixOS/nixpkgs/archive/nixos-unstable.tar.gz -p jdk

Verified installation:
./jpkl --version
Pkl 0.25.2 (Linux 6.1.69, Java 19.0.2)

output
~/jpkl eval bird.pkl

name = "Swallow"
job {
  title = "Sr. Nest Maker"
  company = "Nests R Us"
  yearsOfExperience = 2
}

~/jpkl eval -f json bird.pkl
{
  "name": "Swallow",
  "job": {
    "title": "Sr. Nest Maker",
    "company": "Nests R Us",
    "yearsOfExperience": 2
  }
}

~/jpkl eval -f plist bird.pkl
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
  <key>name</key>
  <string>Swallow</string>
  <key>job</key>
  <dict>
    <key>title</key>
    <string>Sr. Nest Maker</string>
    <key>company</key>
    <string>Nests R Us</string>
    <key>yearsOfExperience</key>
    <integer>2</integer>
  </dict>
</dict>
</plist>

I've written a brief blog post on setting up Pkl using Nix flakes on NixOS.
Details available at https://byteshiva.medium.com/simplifying-configuration-with-pkl-cli-on-nixos-08790fbe4e5c

There's a request (https://github.com/NixOS/nixpkgs/issues/286104) with a WIP PR for packaging pkl on the nixpkgs repo: https://github.com/NixOS/nixpkgs/pull/286658
There's also another flake (https://github.com/capslock/pkl-flake) which I've been using for the last few days. I think it just repackages the official native binary releases, so it doesn't require pulling in the entire JDK.

I've been trying to package it for NixOS. The ideal way would be to build from source and not download the binary, the latter is only done for proprietary programs, in which case it is the only option. However, I've been having some issues, as described in the PR (https://github.com/NixOS/nixpkgs/pull/286658), and any help would be appreciated.
However, it is not hard to build pkl yourself in nix, if you desire. To build the Java version, you just need gradle_7 and jdk17 and run gradle build. To build the native package it is a bit harder... You need graalvm for JDK 11, that got removed and is only available in 23.05 (github:nixos/nixpkgs/23.05#graalvm11-ce). You then need to edit pkl-cli.gradle.kts, removing the dependsOn(":installGraalVm") from Exec.configureExecutable and in the same function, change executable into just native-image, this way gradle will use the graalvm downloaded from nix instead of trying to download it itself, what will not work. Then gradle nativeBuild will work.

According to https://github.com/NixOS/nixpkgs/pull/286658#issuecomment-2229226963 the PR NixOS/nixpkgs#272380 (gradle: add setup hook) got merged, which might unblock the pkl work.


fix mispronounce in ind.tsv

@googlebot I signed it!

@googlebot I signed it!

Thanks, just checked with another informant and they confirmed these corrections, we apppreciate it. I'll merge shortly.


How can reture HTTP status message

Hi Team,
I need to return specific HTTP status message instead of default ok.
E.g - 1. 409 - Conflict
2. 400 Bad Request
i tried wth below code snippet but is returning 'ok' instead of 'conflict' while calling getStatusText() operation,.
httpClientMock.onPost("testUlr").doReturn(409, "some message");
Could you please suggest any way to set status text as well?

I got one alternative for this problem to use Action. It is working fine for me.
Recomendation - I must recommend you to provide one doReturn  override method which should accept status text as well.

Hi @abhikt48
You are right currently the only way to set statusText is by using "Action".
To make that easier I will extend the method withStatus to accept reason parameter.


Generate Server Code

Hi Team ,
I am getting error when trying to generate server specifiction using attached swagger file-
swagger.zip
C:\Harish\Go\workspace\timeofday>swagger generate server -f swagger.yml
2019/04/26 13:34:28 validating spec C:\Harish\Go\workspace\timeofday\swagger.yml
2019/04/26 13:34:29 preprocessing spec with option:  minimal flattening
2019/04/26 13:34:29 building a plan for generation
2019/04/26 13:34:29 FindFirstFile go: The system cannot find the file specified.

Could you please provide us with your spec, so we may attempt to reproduce your issue? Thank you.

Tried to reproduce this with the attached spec on both linux and windows.
go 12.5.1, go-swagger: master
Cannot reproduce: generates server just fine.
Maybe are you trying to generate outside GOPATH?

Hi team,
This is working fine for me too. Original issue was for GOPATH only.
On Sat, 11 May 2019 at 8:31 PM, fredbi<EMAIL_ADDRESS>wrote:

Tried to reproduce this with the attached spec on both linux and windows.
go 12.5.1, go-swagger: master
Cannot reproduce: generates server just fine.
Maybe are you trying to generate outside GOPATH?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/go-swagger/go-swagger/issues/1933#issuecomment-491518293,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABHB47KJQOQQHJD6OHP2JZDPU3NUXANCNFSM4HITNOWQ
.


Invalid app name

I'm trying to create a root app for my domain mydomain.com. In Dokku, I would do this using (on my local machine):
git remote add dokku dokku@mydomain.com:mydomain.com
git push remote master

This would create the app mydomain.com in Dokku, but I instead get this message:
Invalid app name
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

How would I create a root domain when I can't specify a domain as the app name?

What is wrong with using periods in the app name? I realize Heroku doesn't allow this, but that doesn't feel like a good reason to prevent usage of periods in names.

The original dokku allows periods. If they are present, application name becomes its fully qualified domain name, saving an extra configuration step. Would be nice to have this feature in dokku-alt as well.

+1 I don't see the advantage of adding an extra configuration step.


Key Error When Defining Meta Without 'allow_inheritance'

In Base.py, there's a line:
if meta and not meta['allow_inheritance'] and not self.field:

If you have a meta dictionary defined, but "allow_inheritance" is not defined as a key in the document, you end up with a key error.  It should read something like:
if meta and 'allow_inheritance' in meta and meta['allow_inheritance'] is False and not self.field:

https://github.com/chwilliam/mongoengine/commit/3be571beaa73356354445256103ec497d1d8e338

https://github.com/hmarr/mongoengine/issues/430


Desync between client and server items

Holding map specific custom items will cause them to flicker and sometimes do random reload animations
https://user-images.githubusercontent.com/83467229/203416064-16247251-0882-4087-b2f9-ac6530bbf132.mp4

If they don't exist / have invalid data inside the game_items.txt then this tends to happen.
I'll keep this open for now if anything from our side needs fixing.


Add Yelp_Review Dataset

I added yelp_review dataset including Yelp_review_full and yelp_review_polarity,  which is proposed by Xiang Zhang in Paper Character-level Convolutional Networks for Text Classification .

I signed it!

My Request did not pass the check of travis-ci.
I have checked job log and wondered if it is possible that there is something wrong with travis-ci.

My Request did not pass the check of travis-ci.
I have checked job log and wondered if it is possible that there is something wrong with travis-ci.

Travis is broken by other tests failing, adding a new dataset should be fine, I'll doubllt check internally.
PS: Can you also add this to https://github.com/tensorflow/datasets ?

Thanks a lot for this @ksboy -- we'll fix a few nits on our end, thanks a lot for contributing!

My Request did not pass the check of travis-ci.
I have checked job log and wondered if it is possible that there is something wrong with travis-ci.

Travis is broken by other tests failing, adding a new dataset should be fine, I'll doubllt check internally.
PS: Can you also add this to https://github.com/tensorflow/datasets ?

I will when I am free

Many thanks for adding it in the first place!
On Sat, Apr 27, 2019, 11:22 PM ksboy<EMAIL_ADDRESS>wrote:

[image: image]
https://user-images.githubusercontent.com/16255580/56454689-766b7780-6387-11e9-8580-abf903b58d24.png
My Request did not pass the check of travis-ci.
I have checked job log and wondered if it is possible that there is
something wrong with travis-ci.
Travis is broken by other tests failing, adding a new dataset should be
fine, I'll doubllt check internally.
PS: Can you also add this to https://github.com/tensorflow/datasets ?
I will when I am free
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensor2tensor/pull/1551#issuecomment-487348730,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIUEFS23IYD4Y3RTXZHDP3PSU7C7ANCNFSM4HHJDVMQ
.


CTRL+V hotkey setup adds additional characters

I've changed the CTRL+Shift+V paste hot key to CTRL+V, but it produces extra unwanted characters. There are many consistency issues with paste. Examples:
Copy String: test-string:

Right click, paste output: test-string
CTRL+Shift+V output: test-string
CTRL+V output: 0~test-string1~
Middle click output: 0~test-string1~test-string

This is still a problem.

For me the cmd+V add additional characters, cmd+shift+V doesn't paste anything. When i right click and paste also with additional characters. But When i go to menu and click Paste item it is working correctly
Mac 10.15.2
Terminus 1.0.97

Having this issue as well on Windows. It is extremely annoying.
Using 'test-string' as the OP did, pasting ends up showing this:
[200~test-string[201~

@MostHated this means your shell doesn't support bracketed paste - disable it under Settings -> Terminal

i have the same issue

check here


CR026: localization needed for sign-up pop-up

steps to find it:

sign out
open a text
open Resources
click on "add to sheet"

note: to be localized on Figma board

"Sign-up" button still in English


Add document for Gateway Threat Protectors

Purpose
Add document for gateway threat protectors.
Hide Configuring API Labels(free/paid) docs

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


Sonar Cloud Bug: Monitoring Plan

[ ] Remediate sonar cloud bug in monitoring plan
https://sonarcloud.io/organizations/us-epa-camd/projects

[ ] Resolving promises reference:
promises.push( new Promise((resolves, rejects) => {}) ) -> Needs to become promises.push(async function signature)

@shahbazkhan1999 will do peer review

@ergjustin please check comment on PR

Private Zenhub Image
Sonar Cloud Bug: Monitoring Plan#5657 failed

SONAR IMAGE
This ticket was completed in September last (3 weeks ago) that time it is fully clear and code coverage is above 70%. After completed this, codes were added in this repo and not added unit test, so that reason code coverage reduced.


Add some simple e2e tests

Adds a library to help manage k8s resources in an e2e testing workflow, and a few simple e2e test cases to cover common scenarios.
We can expand this later, but for now it at least provides a fast sanity check while developing the codebase.

I'll probably ask you to teach me how to run this when you've finished putting everything together


Style Page

Build the page structure using css3 and bootstrap 5.

The choice between styling with pure CSS3, Bootstrap 5 only, or a combination of both depends on your specific preferences and needs. I will briefly explain the options to help you decide:
Pure CSS3: Styling with pure CSS3 gives you full control over your website's style. You can create custom styles, define layouts, animations, and transitions precisely. However, it may require more time and effort to create styles from scratch.
Bootstrap 5: Bootstrap 5 is a popular framework that provides a comprehensive set of pre-defined styles and components. It is designed to facilitate the development of responsive and visually appealing websites. Bootstrap 5 can speed up the styling process as you can use the classes and components provided by the framework. However, your website may end up having a similar appearance to other websites that also use Bootstrap.
Combination of both: You can also combine the use of pure CSS3 and Bootstrap 5 to get the best of both worlds. You can leverage Bootstrap's classes to quickly build a responsive framework and then customize it with pure CSS3 to add your unique touch. This allows for greater control over the style while taking advantage of Bootstrap's ease of use.
My suggestion is to start with Bootstrap 5 to streamline the styling process and ensure a responsive layout. Then, you can customize the style with pure CSS3 to make the website unique and meet your preferences.
Remember to consult the Bootstrap 5 documentation to make the most of the available classes and components. Try different combinations of styles, colors, and layouts to achieve the desired look.

Update style on tag header

Update style on tag header.
Fixed problema in the resolution 280 x 653px

We added a media query rule for devices with a width smaller than 480px.
Within this rule, we adjusted the styles of the  to improve the visualization on smaller screens:
We reduced the padding of the  to padding: 10px; to save space.
We changed the direction of the  list to flex-direction: column; to display it vertically.
We added a bottom margin to the  list items with `margin-bottom: 10px.


Request: pihole/adguard/NAT compatible (HOST format) list.

Hi, thank you for your work! Like the title says it would be nice to have  HOST format lists for pihole/adguard and also NAT blocking.
Greetings!

Will look into it!

So, I've just created a hosts file. I currently don't have a spare computer/raspberry pi with me atm, but I tested it with my OS hosts file and it seems to work fine. Test it out and tell me how it goes! (hopefully fine!)

Thank you! So far i can see my adguard accepted the list without any issue and have generated 1798 rules. I will test later on a bunch of addresses before introducing further blocking on my android phone (for mobile data).

Work for now very well, closing this issue :)


chore: bump core dependencies

Update all dependencies from the core repo before release.

@dyladan either update yrself or allow to modify it so I can update and merge

@dyladan after you merge, are you going to prepare a release candidate if so, is it going this week then (as stated on Wednesday meeting) ?

@obecny yes I'm planning on releasing this week.
I have the allow modification box checked but because my branch is on an org not a personal account only org members can click "Update Branch" button. I tried to ask if I can use my personal account to fix this but it is company policy.


MRG, VIZ: fix shortcuts and interactivity for raw.plot() with macosx backend

closes #8687

scalebar toggle now works correctly with macosx backend (previously it showed, but did not hide, scalebars immediately)
zen mode now works correctly with macosx backend (previously it would show/hide the scrollbars but not use the freed space)
help dialog corrected: macOS uses fn+→ not ⌘+→ (similar for other 3 arrow keys)
help dialog no longer shows F11 as a fullscreen key for macOS (it didn't work, and fullscreen is built into the OS, so we just don't show a fullscreen key now)

Thanks @drammock, this is 💯! Works like a charm!

Can you add a changelog entry?

Thanks @drammock!

Shall we backport ?


Formula error in excel inputs file

On the 'Resource use and costs' sheet of the model inputs Excel files (ID6184_RCC_model inputs FAD version [ACIC redacted, cPAS redacted and CIC redacted].xlsm and ID6184_RCC_model inputs FAD version [UK RWE unredacted, ACIC redacted, cPAS redacted].xlsm), cells BH193:BI195 have incorrect formulae. The referenced cells should be offset down by one cell.

@DawnLeeHE - Could you verify this? Thanks!


uninit intrinsic is gone

Finally. :D
However, to land this we'll need to fix the intptrcast issues that block https://github.com/rust-lang/miri/pull/803.

@bors r+

:pushpin: Commit fa290f1a481b0f98ed1de06206e643af8e04acd5 has been approved by RalfJung

:hourglass: Testing commit fa290f1a481b0f98ed1de06206e643af8e04acd5 with merge e4a98ea8d81fec630d4f7d3793bd0c550049edaa...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing e4a98ea8d81fec630d4f7d3793bd0c550049edaa to master...


as_shared_dtype converts scalars to 0d numpy arrays if chunked cupy is involved

I tried to run where with chunked cupy arrays:
In [1]: import xarray as xr
   ...: import cupy
   ...: import dask.array as da
   ...:
   ...: arr = xr.DataArray(cupy.arange(4), dims="x")
   ...: mask = xr.DataArray(cupy.array([False, True, True, False]), dims="x")

this works:
In [2]: arr.where(mask)
Out[2]:
<xarray.DataArray (x: 4)>
array([nan,  1.,  2., nan])
Dimensions without coordinates: x

this fails:
In [4]: arr.chunk().where(mask).compute()
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In[4], line 1
----> 1 arr.chunk().where(mask).compute()

File ~/repos/xarray/xarray/core/dataarray.py:1095, in DataArray.compute(self, **kwargs)
   1076 """Manually trigger loading of this array's data from disk or a
   1077 remote source into memory and return a new array. The original is
   1078 left unaltered.
   (...)
   1092 dask.compute
   1093 """
   1094 new = self.copy(deep=False)
-> 1095 return new.load(**kwargs)

File ~/repos/xarray/xarray/core/dataarray.py:1069, in DataArray.load(self, **kwargs)
   1051 def load(self: T_DataArray, **kwargs) -> T_DataArray:
   1052     """Manually trigger loading of this array's data from disk or a
   1053     remote source into memory and return this array.
   1054
   (...)
   1067     dask.compute
   1068     """
-> 1069     ds = self._to_temp_dataset().load(**kwargs)
   1070     new = self._from_temp_dataset(ds)
   1071     self._variable = new._variable

File ~/repos/xarray/xarray/core/dataset.py:752, in Dataset.load(self, **kwargs)
    749 import dask.array as da
    751 # evaluate all the dask arrays simultaneously
--> 752 evaluated_data = da.compute(*lazy_data.values(), **kwargs)
    754 for k, data in zip(lazy_data, evaluated_data):
    755     self.variables[k].data = data

File ~/.local/opt/mambaforge/envs/xarray/lib/python3.10/site-packages/dask/base.py:600, in compute(traverse, optimize_graph, scheduler, get, *args, **kwargs)
    597     keys.append(x.__dask_keys__())
    598     postcomputes.append(x.__dask_postcompute__())
--> 600 results = schedule(dsk, keys, **kwargs)
    601 return repack([f(r, *a) for r, (f, a) in zip(results, postcomputes)])

File ~/.local/opt/mambaforge/envs/xarray/lib/python3.10/site-packages/dask/threaded.py:89, in get(dsk, keys, cache, num_workers, pool, **kwargs)
     86     elif isinstance(pool, multiprocessing.pool.Pool):
     87         pool = MultiprocessingPoolExecutor(pool)
---> 89 results = get_async(
     90     pool.submit,
     91     pool._max_workers,
     92     dsk,
     93     keys,
     94     cache=cache,
     95     get_id=_thread_get_id,
     96     pack_exception=pack_exception,
     97     **kwargs,
     98 )
    100 # Cleanup pools associated to dead threads
    101 with pools_lock:

File ~/.local/opt/mambaforge/envs/xarray/lib/python3.10/site-packages/dask/local.py:511, in get_async(submit, num_workers, dsk, result, cache, get_id, rerun_exceptions_locally, pack_exception, raise_exception, callbacks, dumps, loads, chunksize, **kwargs)
    509         _execute_task(task, data)  # Re-execute locally
    510     else:
--> 511         raise_exception(exc, tb)
    512 res, worker_id = loads(res_info)
    513 state["cache"][key] = res

File ~/.local/opt/mambaforge/envs/xarray/lib/python3.10/site-packages/dask/local.py:319, in reraise(exc, tb)
    317 if exc.__traceback__ is not tb:
    318     raise exc.with_traceback(tb)
--> 319 raise exc

File ~/.local/opt/mambaforge/envs/xarray/lib/python3.10/site-packages/dask/local.py:224, in execute_task(key, task_info, dumps, loads, get_id, pack_exception)
    222 try:
    223     task, data = loads(task_info)
--> 224     result = _execute_task(task, data)
    225     id = get_id()
    226     result = dumps((result, id))

File ~/.local/opt/mambaforge/envs/xarray/lib/python3.10/site-packages/dask/core.py:119, in _execute_task(arg, cache, dsk)
    115     func, args = arg[0], arg[1:]
    116     # Note: Don't assign the subtask results to a variable. numpy detects
    117     # temporaries by their reference count and can execute certain
    118     # operations in-place.
--> 119     return func(*(_execute_task(a, cache) for a in args))
    120 elif not ishashable(arg):
    121     return arg

File ~/.local/opt/mambaforge/envs/xarray/lib/python3.10/site-packages/dask/optimization.py:990, in SubgraphCallable.__call__(self, *args)
    988 if not len(args) == len(self.inkeys):
    989     raise ValueError("Expected %d args, got %d" % (len(self.inkeys), len(args)))
--> 990 return core.get(self.dsk, self.outkey, dict(zip(self.inkeys, args)))

File ~/.local/opt/mambaforge/envs/xarray/lib/python3.10/site-packages/dask/core.py:149, in get(dsk, out, cache)
    147 for key in toposort(dsk):
    148     task = dsk[key]
--> 149     result = _execute_task(task, cache)
    150     cache[key] = result
    151 result = _execute_task(out, cache)

File ~/.local/opt/mambaforge/envs/xarray/lib/python3.10/site-packages/dask/core.py:119, in _execute_task(arg, cache, dsk)
    115     func, args = arg[0], arg[1:]
    116     # Note: Don't assign the subtask results to a variable. numpy detects
    117     # temporaries by their reference count and can execute certain
    118     # operations in-place.
--> 119     return func(*(_execute_task(a, cache) for a in args))
    120 elif not ishashable(arg):
    121     return arg

File <__array_function__ internals>:180, in where(*args, **kwargs)

File cupy/_core/core.pyx:1723, in cupy._core.core._ndarray_base.__array_function__()

File ~/.local/opt/mambaforge/envs/xarray/lib/python3.10/site-packages/cupy/_sorting/search.py:211, in where(condition, x, y)
    209 if fusion._is_fusing():
    210     return fusion._call_ufunc(_where_ufunc, condition, x, y)
--> 211 return _where_ufunc(condition.astype('?'), x, y)

File cupy/_core/_kernel.pyx:1287, in cupy._core._kernel.ufunc.__call__()

File cupy/_core/_kernel.pyx:160, in cupy._core._kernel._preprocess_args()

File cupy/_core/_kernel.pyx:146, in cupy._core._kernel._preprocess_arg()

TypeError: Unsupported type <class 'numpy.ndarray'>

this works again:
In [7]: arr.chunk().where(mask.chunk(), cupy.array(cupy.nan)).compute()
Out[7]:
<xarray.DataArray (x: 4)>
array([nan,  1.,  2., nan])
Dimensions without coordinates: x

And other methods like fillna show similar behavior.
I think the reason is that this: https://github.com/pydata/xarray/blob/d4db16699f30ad1dc3e6861601247abf4ac96567/xarray/core/duck_array_ops.py#L195 is not sufficient to detect cupy beneath other layers of duckarrays (most commonly dask, pint, or both). In this specific case we could extend the condition to also match chunked cupy arrays (like arr.cupy.is_cupy does, but using is_duck_dask_array), but this will still break for other duckarray layers or if dask is not involved, and we're also in the process of moving away from special-casing dask. So short of asking cupy to treat 0d arrays like scalars I'm not sure how to fix this.
cc @jacobtomlinson

Ping @leofang in case you have thoughts?

Sorry that I missed the ping, Jacob, but I'd need more context for making any suggestions/answers 😅 Is the question about why CuPy wouldn't return scalars?

The issue is that here: https://github.com/pydata/xarray/blob/d4db16699f30ad1dc3e6861601247abf4ac96567/xarray/core/duck_array_ops.py#L193-L206 we try to convert everything to the same dtype, casting numpy and python scalars to an array. The latter is important, because e.g. numpy.array_api.where only accepts arrays as input.
However, detecting cupy beneath (multiple) layers of duckarrays is not easy, which means that for example passing a pint(dask(cupy)) array together with scalars will currently cast the scalars to 0-d numpy arrays, while passing a cupy array instead will result in 0-d cupy arrays.
My naive suggestion was to treat np.int64(0) and np.array(0, dtype="int64") the same, where at the moment the latter would fail for the same reason as np.array([0], dtype="int64").

Thanks, Justus, for expanding on this. It sounds to me the question is "how do we cast dtypes when multiple array libraries are participating in the same computation?" and I am not sure I am knowledgable enough to make any comment.
From the array API point of view, long long ago we decided that this is UB (undefined behavior), meaning it's completely up to each library to decide what to do. You can raise or come up with a special rule that you can make sense of.
It sounds like Xarray has some machinery to deal with this situation, but you'd rather prefer to not keep special-casing for a certain array library? Am I understanding it right?

there's two things that happen in as_shared_dtype (which may not be good API design, and we should probably consider splitting it into as_shared_dtype and as_compatible_arrays or something): first, we cast everything to an array, then decide on a common dtype and cast everything to that.
The latter could easily be done by using numpy scalars, which as far as I can tell would be supported by most array libraries, including cupy. However, the reason we need to cast to arrays is that the array API (i.e. __array_namespace__) does not allow using scalars of any type, e.g. np.array_api.where (this is important for libraries that don't implement __array_ufunc__ / __array_function__). To clarify, what we're trying to support is something like
import numpy.array_api as np
np.where(cond, cupy_array, python_scalar)

which (intentionally?) does not work.
At the moment, as_shared_dtype (or, really, the hypothetical as_compatible_arrays) correctly casts python_scalar to a 0-d cupy.array for the example above, but if we were to replace cupy_array with chunked_cupy_array, the special casing for cupy stops to work and scalars will be cast to 0-d numpy.array. Conceptually, I tend to think of 0-d arrays as equivalent to scalars, hence the suggestion to have cupy treat numpy scalars and 0-d numpy.array the same way (I don't follow the array api closely enough to know whether that was already discussed and rejected).
So really, my question is: how do we support python scalars for libraries that only implement __array_namespace__, given that stopping to do so would be a major breaking change?
Of course, I would prefer removing the special casing for specific libraries, but I wouldn't be opposed to keeping the existing one. I guess as a short-term fix we could just pull _meta out of duck dask arrays and determine the common array type for that (the downside is that we'd add another special case for dask, which in another PR we're actually trying to remove).
As a long-term fix I guess we'd need to revive the stalled nested duck array discussion.

So really, my question is: how do we support python scalars for libraries that only implement __array_namespace__, given that stopping to do so would be a major breaking change?

I was considering this question for SciPy (xref scipy#18286) this week, and I think I'm happy with this strategy:

Cast all "array-like" inputs like Python scalars, lists/sequences, and generators, to numpy.ndarray.
Require "same array type" input, forbid mixing numpy-cupy, numpy-pytorch, cupy-pytorch, etc. - this will raise an exception
As a result, cupy-pyscalar and pytorch-pyscalar will also raise an exception.

What that results in is an API that's backwards-compatible for numpy and array-like usage, and much stricter when using other array libraries. That strictness to me is a good thing, because:

that's what CuPy, PyTorch & co themselves do, and it works well there
it avoids the complexity raised by arbitrary mixing, which results in questions like the one raised in this issue.

in case you do need to use a scalar from within a function inside your own library, just convert it explicitly to the desired array type with xp.asarray(a_scalar) giving you a 0-D array of the correct type (add dtype=x.dtype to make sure dtypes match if that matters)

Would this be an acceptable, if temporary, fix for #9195? Modified code in as_shared_dtype:
     array_type_cupy = array_type("cupy")
     # temporary fix
     import nested_duck_arrays.dask

     def _maybe_cupy(seq):
         return any(isinstance(x, seq) or
             is_duck_dask_array(x) and x.__duck_arrays__()[-1].__module__ == 'cupy'
             for x in seq)

     # if any(isinstance(x, array_type_cupy) for x in scalars_or_arrays):
     if _maybe_cupy(scalars_or_arrays):
         # end of fix
         import cupy as cp

I'd go with something like
import nested_duck_arrays.dask
import nested_duck_arrays
...

if any(nested_duck_arrays.first_layer(x) is array_type_cupy for x in scalars_or_arrays):
    import cupy as cp

and add nested_duck_arrays.first_layer which would have a fallback of returning a 1-tuple containing type of x in case x is not a duck array (I'd be happy to relatively quickly release that to PyPI / conda-forge).
We'll need to think about what to do if nested_duck_arrays is not installed, though.


Refactor code

Description

Refactoring code and optimize imgs
Checklist

[] - All code changes have been tested, if any
[] - The documentation has been updated accordingly for any code changes
[] - This PR fixes an already existing issue
[] - This PR adds a new feature (new functions, methods, classes, parameters, etc.)
[] - This PR fixes a bug
[x] - This PR changes the main DShell cog
[] - This PR is a breaking change (removes/renames functions, parameters etc.)
[] - This PR is NOT a code change (changes Documentation files, README.md, CONTRIBUTING.md etc.)

Additional context

Will test later

Looks great to me, I'll test it from my side too. If all goes well, I'll merge it

OK cool

Tested it, it works. Thanks for contributing :>

Np


Nvss 622 mother weight height

Summary

Added Mother weight/height for fetal death
moved weight at birth to birth record (it is birth specific)
encountered some warnings to include xml comments for protected methods. I tried adding some but the warnings would not go away

The code changes look good but there are some conflicts and failing tests to address before merging


Functionality lost on cast reconnect

I am experiencing an issue where the hooks lose functionality and data (eg. deviceName), if you disconnect from the receiver device, and reconnect again without reloading the page. The useCast hook will show both initialized: true and connected: true, but will not be able to see the deviceName and/or control the device using useCastPlayer hook. The issue seems only to be resolved by reloading the page.
This can be replicated on the demo page https://antewall.github.io/react-cast-sender/ as well.
Browser:
Chrome Version 80.0.3987.122 (Official Build) (64-bit)
OS:
MacOS Mojave
10.14.6

#8


Recovery phrases generated by SDK do not comply with BIP-39

Ref: https://github.com/infincia/bip39-rs/pull/17

Fix is up, just waiting for it to be merged through to a release. I could switch it to point to my fork again for now.


PHP Warning on Component createField() method

Me and my work colleges we were having an issue with ansible installing a shopping worlds plugin.
The problem is that the PHP function max() raises a warning when you pass as an argument an empty array, I'm aware that Shopware is expecting it because of this line:

$this->maxFieldPositionValue = max($positions) ? : 0;
but the problem is that PHP Warnings are unnecessary. Our suggestion  is not raising warnings.

Thanks

@bcremer

Hi @jorgemurta,
thanks for the contribution, for me it looks good. The GitHub tickets are also part of the core development's sprints, so it might take a short while sometimes. If @bcremer didn't nit-pick, yet, it is usually a good sign :)

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/#/issues/SW-14594.
Please use this issue to track the state of your pull request.

Merged with commit https://github.com/shopware/shopware/commit/3536b3f3c9cfd29727d689098066ca84648e0ab0.


First pass at REUSE and SPDX licensing

At present the aim is to demonstrate to developers which licenses apply. The tool is not enforcing compilance of checking file contents.
The intention is to build on this to ensure compilance of all files in the repository.

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


Enable support for footnotes in markdown files to match jekyll behavior

I noticed that my blog that the footnotes I had in my jekyll blog were no longer being generated from the markdown, and tracked it down to the flags that were being sent to blackfriday didn't enable footnotes. The footnote HTML generated by blackfriday doesn't 100% match what jekyll generates, but it's close enough that you can make it match with a single CSS rule.
Checklist

[x] I have read the contribution guidelines.
[x] make test passes.
[x] make lint passes.
[x] New and changed code is covered by tests.
[x] Performance improvements include benchmarks.
[x] Changes match the documented (not just the implemented) behavior of Jekyll.

Thanks!


Pressing key to toggle zoom

One of my favorite features of Konduit was that the client's zoom could be toggled on and off with Ctrl+MiddleClick, effectively giving two levels of zoom toggled by pressing a button instead of scrolling (switching between the in-game and Konduit's zoom).
I recognize that RuneLite's zoom is not a client-specific one, but rather depending on the in-game zoom, however it would be great if there were some way to make a similar feature. An alternative could be an option to accelerate the zoom amount (like make it twice as fast), the problem in the first place is that I feel the scrolling is too sluggish.

Duplicate of #919


do assignments maybe need attachments?

so, in the old app, a perform activity (check it at the old app by logging in as student 860f629b and then choosing a perform activity, i.e., *melody or * bassline) had a recorded Sample Audioand also Accompaniment. Probably this is an important thing to keep around... Where to put it? is it an attachment? if so, to what? Seems lke it shoudl not be to an assignment since an assignment exists for every user, but the activities are unaware of the piece and so cannot have the audio attached /-:

Accompaniment should go on Piece (but possible Accompaniment should be its own model with foreign key to Piece so that there could be many accompaniments and teacher could pick which one they want?)
Sample Audio should go on Part since there is a Part for melody and bass for each Piece and that seems to be how many sample audios there are in the old site. (Same question here about allowing many and teacher can pick which one they like?)
Probably can leave the many audios for v2, should be a reasonable data migration to split these off

Sorry I'm not sure I got it?
I guess right now we think we have 1 accompaniment per piece and 1 sample per part (with all existing pieces having 1 part per piece [melody and bassline], but the suspicion that we may wish to have a combo part as soon as the current version) .
This seems reasonable that e.g., the accompaniment would be the same for both the bassline and the melody since eventually the melody and bassline are from the same piece (so it's reasonable that there'd remain with new pieces we add 1 accompaniment even though there'd likely be 2 parts), but that the sample audio should vary differ/correspond since it's specifically for the part.
So i think you're saying that we just add:

for the accompaniment, to piece, the 1 audio/file field (similar to what we use on submission attachments' file just a varying character fieldthat holds the s3 object name?)
for the sample audio, to the part, 1 such field

for the accompaniment, to piece, the 1 audio/file field (similar to what we use on submission attachments' file just a varying character fieldthat holds the s3 object name?)
for the sample audio, to the part, 1 such field

Exactly

awesome, sounds great to me

Deployed via 47d099c8ac9c228f0e48124dd68edcdda3915271
Thoughts on how these files will be uploaded in prod? Probably through Django admin pages? If so, that will require an nginx config change because it's giving a 413
https://stackoverflow.com/questions/66929579/django-nginx-413-request-entity-too-large

Hmm yeah, I think in django admin is great for v1.


get_threshold enhancement

get_threshold() is the function that should find the optimal value for the subsequent use of cv2.threshold(). Currently it considers the lowest section of the document (since that part is usually white), and applies cv2.threshold() with descending values from 200, stopping as soon as the threshold is low enough that 99.9% of the pixels in the section are set to white. If the brightness of the picture is sufficiently uniform, the result is pretty good, otherwise some parts of the document become unreadable or completely black.
Also, if that lowest section isn't supposed to be white, the function can't work.
If you have any ideas, please write a comment or submit a pull request, and if you would like to contribute but you don't understand some parts of the code, don't hesitate to ask.
EDIT:
Right now I'm combinig my custom function get_threshold with OpenCV's cv2.adaptiveThreshold using a cv2.bitwise_or between the two, so that if a pixel is made white by either of the functions it will be white in the result. This improves performances significantly, but there are still problems with pictures that have non-uniform illumination.

i would like to work on this. I will give u an approach on it soon

hello @aniketmdinde, I'm glad to see you interested! I will assign you. If you take a look at scanner.py, you'll notice that get_threshold isn't currently used, as the job is done by cv2.adaptiveThreshold. Neither of the two approaches is perfect, and in the future I'd like for the user to be able to specify which one should be used. If you're interested in that too let me know and I'll open another issue.


How to import new urdf models in mujoco?

Hello,
I have a urdf file of an inverted pendulum which I want to import in mujoco but I am not able to find any details regarding that. In other platforms, I was able to have functions to do the importing but I don't have anything in mujoco. Please help me out.

This thread from the old forum might prove useful: https://roboti.us/forum/index.php?threads/meshes-ignored-when-converting-urdf-to-mjcf.3433/

Hello,
I tried to do as you said but this error is showing up whenever I am dragging and dropping the urdf file in the simulate app.
Error: could not open STL file '/home/soumyajit/Downloads/Inverted/urdf/base_link.STL'
Object name = base_link, id = 0
Also, I have attached the Urdf file for your reference.
Inverted.zip

I too face the same problem while using the outputs of solidworks2urdf or fusion2urdf scripts.
I am not sure how the file is parsed but it is clear that there is no mesh file at the said path: /home/soumyajit/Downloads/Inverted/urdf
I just copy all the meshes (.stl files) to the /urdf directory and it works fine. (It is working for your example too).
It would be great if someone throws light on the details of why this happens.

This thread from the old forum might prove useful: https://roboti.us/forum/index.php?threads/meshes-ignored-when-converting-urdf-to-mjcf.3433/

(a) Unfortunately, examples in this forum cannot be downloaded anymore. Can we do so, from somewhere else?
(b) The solutions include adding Mujoco tag (and specifying the mesh path) in the urdf file. Can we do so without adding the Mujoco tag?

@v-r-a Why can't you add a <mujoco> tag? If you don't add that tag, how do you expect to tell MuJoCo where the meshes are? And it looks like already have the workaround of just copying all the meshes to the same directory, so you can avoid using the <mujoco> tag if you really want. So basically it's not clear to me what the issue is.
@soumy97 please add a <mujoco> tag, as documented here.

@v-r-a Why can't you add a <mujoco> tag? If you don't add that tag, how do you expect to tell MuJoCo where the meshes are? And it looks like already have the workaround of just copying all the meshes to the same directory, so you can avoid using the <mujoco> tag if you really want. So basically it's not clear to me what the issue is.
@soumy97 please add a <mujoco> tag, as documented here.

(1) I was just wondering if the path that is specified already in urdf could be used. For example:

(2) This is a query for another day, but are there any plans to implement inverse kinematics in MuJoCo? I was trying to avoid   tag to maintain compatibility with a library that can do inverse kinematics, e.g., https://github.com/rbdl/rbdl .

This is certainly possible. You are welcome to submit a pull request 😄.
No such plans. There are many ways to implement IK and implementing just one of them doesn't make sense, there will always be someone unhappy with the implementation. However, given analytic Jacobians (which MuJoCo computes very efficiently) any one of these implementations is rather easy to write. Here is an example from DeepMind.

@wangcongrobot Unsure about why compile isn't part of the binaries anymore but you can load the XML file in the simulate app and it will compile it for you.


zip can not uncompress

The zip file can not be uncompressed.

Please read the README:

Note: try 7zip if decompression errors. It might be an unsupported compression issue.

duplicate of #5


Add --log-file to gvproxy

Add ability specify a log-file for log messages.  This PR redirects are logged messages from the logrus logger to a given file.  It is intended to help debug gvproxy failures and usage.

@mheon ptal

LGTM. We're doing this on the Podman side for gvproxy (setting os.Stdout/Stderr to a file, instead of passing this as an argument) but this is a good utility add

@mheon long term, i'm wondering if podman is --log-level debug ... maybe cann gvproxy with this file too and always store in rundir or the like (you know, another memory leak :) )

/lgtm


I'm trying to log in, but I can't log in due to a network error.

I'm going to study Firebase using "Friendly Eats" sample source.
https://firebase.google.com/codelabs/firestore-android#2
I'm proceeding with the explanation number 5(Run the app).
The local emulator ran well.
You also created a account on the Authentication tab at http://localhost:4000/.
When logging in with the email generated from the login screen, the login process is not progressing due to a network error.
What should I do?

@hansungbin what is the network error? Please provide detailed logs so we can help you debug.

Generally, I didn't test it with a local emulator because I operated it on my phone.
When I first wrote the question, I only did it on my phone, but it didn't work, so I asked the question.
I accidentally operated on a local emulator and it works well.

But it still doesn't work well on my phone, what should I do?
Other applications work well without any problems.

Is there a way to work well on my phone?

@hansungbin when you're using the Firebase Emulator Suite you can't use a real device, you need to use an Android emulator so that they can communicate over localhost.
If you want to use a real device you'll need to use a real Firebase project, which is covered in the final step:
https://firebase.google.com/codelabs/firestore-android#11

I am going to close this issue, but I'll also make sure to add a warning to the codelab to make this clearer.