blueberry: missing library for autostart desktop file

Describe the bug
I get an error message when trying to exectute the autostart dektop file blueberry-obex-agent.desktop.
Steps To Reproduce
Steps to reproduce the behavior:

nix build 'nixpkgs#blueberry'
nix-shell -p dex
dex result/etc/xdg/autostart/blueberry-obex-agent.desktop

results
[nix-shell:/tmp]$ dex result/etc/xdg/autostart/blueberry-obex-agent.desktop

[nix-shell:/tmp]$ Traceback (most recent call last):
  File "/nix/store/0dwdzfh0zdxlarsqf95zw6a5j3q060kl-blueberry-1.4.8/lib/blueberry/.blueberry-obex-agent.py-wrapped", line 29, in <module>
    gi.require_version('Notify', '0.7')
  File "/nix/store/d8as2swdgm78cxzh4zddd0861w6pxn9i-python3.10-pygobject-3.44.1/lib/python3.10/site-packages/gi/__init__.py", line 126, in require_version
    raise ValueError('Namespace %s not available' % namespace)
ValueError: Namespace Notify not available

Expected behavior
Desktop file should be runnable without any problem.
Additional context
I ran into this because I wanted to autostart all my applications that follow the xdg-autostart specification. For this, I used the program dex. When it tried to autostart all the applications following xdg-autostart, it got this error when it tried to start everything.
Additional thoughts
Should the two autostart files really be enabled by default, without any option to turn them off? Maybe add a NixOS module with the option to enable the two desktop file separately and default to have them disabled? Because I do not think installing a package should automatically autostart a program, just because the system is following xdg-autostart.
Notify maintainers

@bobby285271 @romildo
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
/tmp .................................................................took 4m5s
❯ nix-shell -p nix-info --run "nix-info -m"
this path will be fetched (0.00 MiB download, 0.00 MiB unpacked):
  /nix/store/x6yjkzgmdppwi95y4gvm4lxi6q27936s-nix-info
copying path '/nix/store/x6yjkzgmdppwi95y4gvm4lxi6q27936s-nix-info' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.29, NixOS, 23.11 (Tapir), 23.11.20230524.f91ee30`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(root): `"home-manager-22.11.tar.gz, nixos-22.11"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos````

Should have been fixed by https://github.com/NixOS/nixpkgs/pull/234935


Cannot grasp with my created robotic hand

Hello Jennifer,
First, Thank you very much for your work.
I am working on ubuntu 14, ros indigo and gazebo 2.2.6. I am using the simulation of the Kuka robot proved by: https://github.com/IFL-CAMP/iiwa_stack and a robotic hand that I created to simulate the robotic hand from Righthand takktle2 which is a robot with 3 fingers (maybe the issue is comming frome here).
It is a dummy question.. because I think I forgot something somewhere.. So excuse me for this question.
I installed your package in my catkin_ws and everything is built and worked.
I ran my robot and try to grasp the object but the object is not caught but the contacts are done.
Here is a picture what I did to have an idea of the issue.
And this one, when it was trying to grasp.

I didn't take the picture of the right moment but there is a contact of both side of the object and I sat 170degrees for the force angle tolerance. The other setting are :
<forces_angle_tolerance>170</forces_angle_tolerance>       <update_rate>100</update_rate>       <grip_count_threshold>2</grip_count_threshold>       <max_grip_count>10</max_grip_count>       <release_tolerance>0.05</release_tolerance>       <disable_collisions_on_attach>true</disable_collisions_on_attach>       <contact_topic>__default_topic__</contact_topic>
My question is: do I have to set somewhere the name of the object ? Here it is only a rectangle. Or what did I miss ?
Edit:
I think there is no collision between the object and the hand. I have some collisions sensors and those one work.
If you need more info, I would answer it with pleasure.
Warm regards,
Jonathan

Hi Jonathan,
can you post the output on the command line please?
Also, try to visualize the collision shapes (I think it's in the "View" menu) and see if contacts happen with the collision shapes, or only with the visual shapes you are seeing on your screen.
You can also try to reduce the forces_angle_tolerance and increase the grip_count_threshold.
Best regards,
Jennifer

Sorry I didn't think to check the output on your screenshot. I can't see any collision output from the grasp plugin. Try to visualize the collision shapes and let me know if the object and the hand are actually colliding.

I have reduced the forces_angle_tolerance and I have also increased the grip_count_threshold. But nothing change.
I tried as you recommand to check the collision, so here this two pictures.
This one seems that it doesn't have any contact with the object...

But in this one, it seems to have a contact. It is a weird behavior... Why one finger will be like a fantom and the 2 others work ? I checked the urdf but everything seem to be good..

So maybe the issue comes from my side.. :/

Yes on the two screenshots below I can see there are contacts with one side of the gripper, but I can't see if there are also contacts from the opposing finger (the side where it has only one finger?).. Can you send a screenshot where you can see both sides? It looks to me that the finger is kind of inside the object...

Hello, I have fixed the issue! It came from my reinforcement learning algorithms.
So to everyone, this was the issue...:
self.reset_simulation_proxy = rospy.ServiceProxy('/gazebo/reset_simulation', Empty)
        self.reset_world_proxy = rospy.ServiceProxy('/gazebo/reset_world', Empty)

Be careful when you reset the environment of gazebo.
And to answer at my own question, we don't need to specify the name of the object that we want to catch.
Thank you very much for trying to help me and good job with what you have done with this patch.

Glad you solved it, thanks for posting the solution!


downgrade surefire and failsafe

zipkin core tests are not being run due to maven plugin conflicts
https://travis-ci.org/openzipkin/zipkin/jobs/589012904#L1708
and
https://springframework.guru/why-your-junit-5-tests-are-not-running-under-maven/
downgrading to 2.21.0 made the tests run again, the rest of the modules did not seem to care either way.

Thanks Jorge. I think we need to look closer.. at least fail safe is
working else build wouldnt take so long. I would prefer to cite an actual
issue vs a blog as not always blogs are correct and at least issues can go
somewhere
cc @anuraaga
On Wed, Sep 25, 2019, 4:42 AM Jorg Heymans<EMAIL_ADDRESS>wrote:

zipkin core tests are not being run due to maven plugin conflicts
https://travis-ci.org/openzipkin/zipkin/jobs/589012904#L1708
and
https://springframework.guru/why-your-junit-5-tests-are-not-running-under-maven/
downgrading to 2.21.0 made the tests run again, the rest of the modules
did not seem to care either way.
You can view, comment on, or merge this pull request online at:
https://github.com/openzipkin/zipkin/pull/2822
Commit Summary

downgrade surefire and failsafe

File Changes

M pom.xml
https://github.com/openzipkin/zipkin/pull/2822/files#diff-0 (6)

Patch Links:

https://github.com/openzipkin/zipkin/pull/2822.patch
https://github.com/openzipkin/zipkin/pull/2822.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin/pull/2822?email_source=notifications&email_token=AAAPVV7ZKXERWRVGG5G6ACTQLJ3SXA5CNFSM4I2ETTPKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HNNR3JA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAPVV6VSORUO3WYWJNOGKDQLJ3SXANCNFSM4I2ETTPA
.

I think there's something else going on. For example, stackdriver has mixed junit classic and jupiter tests in one module, and I notice they run: ./mvnw clean verify -pl  :zipkin-autoconfigure-storage-stackdriver. Maybe there's something @saturnism fixed we need to carry here?
https://github.com/openzipkin/zipkin-gcp/commit/32c479d16f209f49e51e63f6acb32a1107ed5ee3

It's entirely possible something else is going on but i'm pretty sure zipkin-core tests were being run in Travis at some point, and most likely an upgrade of either maven or a plugin made it crash. zipkin-core is really trivial in terms of tests, yet surefire is unable to run them. I'll raise an issue to track, you can either downgrade and let this work out itself - or get out the flashlight and enter the rabbit hole ;-)

let me take this on on. thanks @jorgheymans!

ps I didn't get to this. if someone wants to yank please do

fixed by @anuraaga #2828


Add handles for manually resizing terminal

https://github.com/STRML/react-resizable

On second thought, this doesn't seem super useful... closing for now.


add engineOpts and dockerVersion fields in machine

@cjellick

@wlan0 do we ever expect those fields to be set by the user? If so, we should add the permissions to user-auth.

These fields are required for allowing a user to specify a docker version, and other docker engine options when performing the 'Add host' action for a cloud provider.

@alena1108 Set 'create' permissions for "project" user on these fields. The "project" user will create these fields.

LGTM


Error load

When creating a DataFrame from a JSON object, the length of each entry must the same and must contain the same keys.


Add metrics

What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-build comment=true

/run-build-release comment=true

Can put the grafana screenshot in PR description.

/run-integration-test

/hold

/run-build-release comment=true

/run-build-release comment=true

/run-build-release comment=true

/run-build-release comment=true

/run-build-release comment=true

/run-build-release comment=true


Add Recommendation to Debug mode

This PR fixes #12
What are changed:

saves both patientData and convertedData (FSH format)
fixes the issue that sending convertedData to CDS engine
displays recommendations from CDS in debug mode

Resolves Issue https://github.com/ccsm-cds-tools/ccsm-cds-dashboard/issues/12


Discussion

Here's my initial thoughts about this language:

Conversion check should be nominal for simplicity
Only case-tree instantiation need to be supported
Surface syntax should be considerate of parsing ease
Simple (co)inductive types (not indexed) with an identity type as described in Jesper's paper

It's not actually (co)inductive -- there's no termination or productivity checks yet

Definition by pattern matching according to Jesper's paper
Coverage check + case-tree generation described in Jesper's paper
Prefix (applying on projection) and postfix (projecting from data) projection (or maybe not?)

... and I plan to enhance everything in the next language after Narc, including (but not limited to):

Pattern instantiation, to see if we can prove things easier
Structural conversion check (or a partial one, like in mlang)
Totality check: termination/productivity
... more?

... and even next language in the future:

Indexed data families, remove the built-in identity type
IDE mode like agda2-mode, but I'll go for both Code (primary) and Emacs (secondary)
De-morgan cubical primitives (Interval, Path, hcomp, transport, Glue)
... more?

@AD1024 @zyh3838438zyh :wink:

Typecheck ok gives 🐮🍺, and qutting REPL gives 🍵.

Typecheck ok gives 🐮🍺, and qutting REPL gives 🍵.

I'm not doing this right now -- 🍵 is the output for scope-check failure.
I tried 🌶🐔, but both Windows Terminal & VSCode console cannot display this properly.

May I know is there a reason that you remove the row polymorphism?
I saw it from the Changelog 0.0.2 #1 but I can't find any documentation expressing the reason.

May I know is there a reason that you remove the row polymorphism?

narc-rs is the language that implements core features from Agda, and voile-rs is the one that experiments with row polymorphism and dependent types.

I see, I thought narc-rs would be superseding the voile-rs so I am expecting the row polymorphism feature. Got it!

I see, I thought narc-rs would be superseding the voile-rs so I am expecting the row polymorphism feature. Got it!

I did some minor research in the first year and I've found row polymorphism to be unpleasant in a dependent type setting.
Btw, the effort has been moved to the Aya prover, you can take a look if you're interested. Currently we're working on some UX features and it's going to be published once the module, library, and releasing systems are ready


Update Arcan and associated packages. Temporarily disable libunwind support added in new Arcan/Tracy versions.

Description of changes
Updated arcan and related software to latest versions. Did not enable new libunwind backtraces for Tracy because I am having difficulty getting tracy/arcan to link to libunwind during the build.
https://github.com/letoram/arcan/blob/master/CHANGELOG.md
I don't have arcan working on bare metal yet, so I cannot assess whether there is any regression in bare metal support. I tested the console and durden appls briefly in SDL mode to confirm no obvious regressions there.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I see there is already https://github.com/NixOS/nixpkgs/pull/310831. Feel free to close this in favor of the existing PR.


Derive Debug, PartialEq, Eq on Fdstat

Similarly to #34, it would nice for Fdstat to autoderive a couple of traits such as Debug, PartialEq, and Eq. This would lend itself nicely to asserts on Fdstat structs:
assert_eq!(fdstat1, fdstat2, "should be equal);

Also, it seems that Fdstat in the previous version of wasi did implement those traits which may be considered a regression for any user of this crate migrating from 0.7 to 0.9.

Debug is good. Eq/PartialEq -- fdstat isn't problematic in the way filestat is, but I'd still be curious how normal code might use these, other than tests.

Similarly to #34, I suggest we initially add Debug which seems fundamental, and Eq and PartialEq can be left out for now until we get more reports from the users.


drivers: flash: w25qxxdv: Avoid locking when not threaded

When using CONFIG_MULTITHREADING=n, the semaphore primitives are
non-functional and useless. Remove their usage when this option is
enabled.
Signed-off-by: Johannes Hutter<EMAIL_ADDRESS>

Similar to the changes in #8430. Related to the discussion in #8368

Codecov Report

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

@@           Coverage Diff           @@
##           master    #8450   +/-   ##
=======================================
  Coverage   64.61%   64.61%
=======================================
  Files         421      421
  Lines       40296    40296
  Branches     6803     6803
=======================================
  Hits        26037    26037
  Misses      11126    11126
  Partials     3133     3133

Continue to review full report at Codecov.

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


pytest fixtures trigger pylint redefined-outer-name

Standard usage of pytest triggers the pylint warning redefined-outer-name. This issues is discussed here.

This has been temporarily fixed by silencing this pylint warning in test_memberdatabase.py, ce91a087c5349bab262f4bf008b6bc82a8c3c806.


Adds repo:update_owners task

Introduces UpdateRepoOwner service
Adds repo:update_owners rake task, which allows the backfilling of repos.owner_id introduced in https://github.com/thoughtbot/hound/pull/586

https://trello.com/c/zQPwrdf6/475-backfill-repos-owner-id

Closing until https://github.com/thoughtbot/hound/pull/586 lands.


fix: fixing mysql error message

SUMMARY
The previous regex for custom database errors for mySQL had an extra period, this fixes that. This add that period.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
After

TEST PLAN

ADDITIONAL INFORMATION

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

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

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

@AAfghahi I just realized, can you update the unit tests as well? If though they should pass, it would be nice to have the error message there consistent with what MySQL returns.

@AAfghahi I just realized, can you update the unit tests as well? If though they should pass, it would be nice to have the error message there consistent with what MySQL returns.

Oh yes, good point.

@betodealmeida fixed the tests


Fix broken stdout

I'm getting reports that my changes might have broken stdout might have broken out putting to stdout. I can't think of anything I did that would have caused that, but I did not test it after my changes. As this is used by QElectrotech when calling the program instead of just from the cli, it's quite important.

So this isn't actually broken, but you just need the correct flags. It needs both the -i (to turn off the info) and the -v (to turn of creating the .elmt) to return the results to stdout.
so you need to execute as: dxf2elmt -i -v -f myfile.dxf
This choice of flags and combinations seems a bit odd, and I'm not sure why a "verbose" flag would stop creation of the elmt file. So I might need to do some cli flag cleanup here at some point. But for now I'm closing this as printing to stdout does work.


DIGITS 5.0 requires torch file

Hello
I'm trying to run a torch model from new digits 5.0 and the error points to a file /home/gonzalo/digits/digits/tools/torch/struct.lua
I looked up and it's not there nor in the Git.
Is it a DIGITS problem or something could be wrong with my installation?

Is this a Torch model of yours?

Nop, I'm trying the text-classification example.

We have Torch install instructions there.

Thank you. I had to reinstall the following packages
luarocks install nn
luarocks install cunn
luarocks install cunn 1.0-0
luarocks install torch
luarocks install cutorch
luarocks install cutorch 1.0-0
luarocks install cudnn
Still, DIGITS have some bug, I'm seeing this:

^ That's a separate issue. Will you please close this and open a new one? It helps to keep our issues properly sorted and organized so that others with the same issue can search and find the solution.

Sorry for this much questions
So that was fixed but when I use cudnn, it encounters an illegal memory access:
2016-11-10 13:08:20 [20161110-130814-08b5] [WARNING] Train Torch Model unrecognized output: THCudaCheck FAIL file=/tmp/luarocks_cutorch-scm-1-4974/cutorch/lib/THC/generic/THCStorage.c line=182 error=77 : an illegal memory access was encountered
2016-11-10 13:08:20 [20161110-130814-08b5] [ERROR] Train Torch Model: /home/gonzalo/.luarocks/share/lua/5.1/nn/Container.lua:67:
2016-11-10 13:08:20 [20161110-130814-08b5] [WARNING] Train Torch Model unrecognized output: In 2 module of nn.Sequential:
2016-11-10 13:08:20 [20161110-130814-08b5] [WARNING] Train Torch Model unrecognized output: /home/gonzalo/.luarocks/share/lua/5.1/dpnn/OneHot.lua:33: cuda runtime error (77) : an illegal memory access was encountered at /tmp/luarocks_cutorch-scm-1-4974/cutorch/lib/THC/generic/THCStorage.c:182

Thank you for your help!

Did you remember to set mean subtraction to None when creating the model?

haa
That was it. It's working!!
Thank you Greg


Crash from Sentinel issue on arm 32-bit

In what area(s)?

/area runtime

What version of Dapr?

1.2.2

Expected Behavior
No crash at startup.
Actual Behavior
dapr v1.2.2 crashes at startup on arm 32-bit due to an integer overflow issue in Sentinel v1.0.2, as I reported upstream, including a stack trace. What's odd is that I have not configured dapr to actually use Sentinel -- no component or configuration yaml.
I have found two ways to workaround the error. First, comment out references to Sentinel in cmd/daprd/main.go. Second, use a version of Sentinel patched for the overflow. See Sentinel patch and use.
Steps to Reproduce the Problem
You should be able to reproduce just by running an app with dapr v1.2.2 on an arm 32-bit platform, like a Raspberry Pi 3.
Release Note
RELEASE NOTE: UPDATE Runtime dependency.

Do you have interested to fix it?

Do you have interested to fix it?

I would prefer not to work with the upstream Sentinel repository because I have no experience with it and don't need to use it. This work would take time that I really don't have ATM.
It does seem odd that just compiling dapr with the Sentinel plugin present but not configured would cause this issue in dapr. Once again though I don't need to use Sentinel and have a working solution, and unfortunately no time to pursue it.

/cc @artursouza @wcs1only


Refactor via open metastore function

Summary of changes (Bug fix, feature, docs update, ...)

I created a function open_metastore in a new _metastore module in openghg.store,
which accepts a bucket and a data type and returns an instance of the child of BaseStore
corresponding to that data type.  (This is slightly different from load_metastore, which requires
a key, rather than a data type; this is so that the metastore can be accessed without knowing internal
details of the metastore.)
Any place in the code where children of BaseStore were being used to access the metastore
was changed to use open_metastore.
The point of this change is to

clearly mark all points in the code where we're accessing the metastore
allow us to easily change how we access the metastore.

Note: this builds on PR #760

Please check if the PR fulfills these requirements

[x] Partial progress towards #766
[x] Tests added and passed if fixing a bug or adding a new feature
[x] All code checks passing - black --line-length 110 run over code, mypy and flake8 not showing any errors
[x] Documentation and tutorials updated/added
[x] Added an entry in the latest CHANGELOG.md file if fixing a bug or adding a new feature
[x] Added any new requirements to requirements.txt and recipes/meta.yaml

Codecov Report

Merging #767 (533a640) into devel (5e6a8d0) will decrease coverage by 0.50%.
Report is 2 commits behind head on devel.
The diff coverage is 63.21%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##            devel     #767      +/-   ##
==========================================
- Coverage   33.72%   33.23%   -0.50%
==========================================
  Files         175      178       +3
  Lines       13935    13903      -32
==========================================
- Hits         4700     4620      -80
- Misses       9235     9283      +48

Files Changed
Coverage Δ

openghg/dataobjects/_datamanager.py
0.00% <0.00%> (ø)

openghg/objectstore/_integrity.py
0.00% <0.00%> (ø)

openghg/retrieve/_search.py
0.00% <0.00%> (ø)

openghg/standardise/__init__.py
0.00% <ø> (ø)

openghg/standardise/_standardise.py
0.00% <0.00%> (ø)

openghg/store/_boundary_conditions.py
0.00% <0.00%> (ø)

openghg/store/_emissions.py
0.00% <0.00%> (ø)

openghg/store/_eulerian_model.py
0.00% <0.00%> (ø)

openghg/store/_footprints.py
0.00% <0.00%> (ø)

openghg/store/_obscolumn.py
0.00% <0.00%> (ø)

... and 21 more

@gareth-j I should have made this clearer in the description, but all of the stuff to do with standardising is in PR #760. The new changes in this PR are around open_metastore.
I will apply your suggestions to PR #760 and then merge them into this branch.
I couldn't figure out how to get github to show a diff from this PR to PR #760, but it should only be changes involving open_metastore.

This is the diff from PR #760: https://github.com/openghg/openghg/compare/refactor_tests_to_use_standardise...refactor_via_open_metastore_function

If this PR is built upon the code within PR #760, I would suggest getting that PR completed first and merged into devel and then your diff would only contain the additional changes on this PR. With chained PRs like this it is easiest if these are done in order and it does make it easier to review.

If this PR is built upon the code within PR #760, I would suggest getting that PR completed first and merged into devel and then your diff would only contain the additional changes on this PR. With chained PRs like this it is easiest if these are done in order and it does make it easier to review.

I think Gareth meant to put these comments on PR #760. Do you suggest I don't request reviews on one until the other is finished? (I guess I expected Github to just show the diff from 760 since this is a branch off of that branch.)

I implemented Gareth's suggestions in the part 1 PR, since the args/Args etc. problems were all in code in that PR.

I mentioned this in the review because I can see at least one place which use "args" etc. e.g. in your new open_metastore function this includes "args" and "returns" so would be worth double checking changes related to Gareth's suggestions have been implemented.

I implemented Gareth's suggestions in the part 1 PR, since the args/Args etc. problems were all in code in that PR.

I mentioned this in the review because I can see at least one place which use "args" etc. e.g. in your new open_metastore function this includes "args" and "returns" so would be worth double checking changes related to Gareth's suggestions have been implemented.

Oh okay, sorry. I did fix everything Gareth pointed out. FWIW it's fixed in PR 3 because I re-wrote the docstring.

I'm closing this and requesting a review for PR #771, which incorporates these changes and completes the change of how open_metastore works.


Event not showing in the SDK INTEGRATION TESTS

Report
Plugin Version
6.12.11
On what Platform are you having the issue?
Android
What did you do?
I'm using the following block of code
appsFlyer.initSdk({
  devKey: ***,
  isDebug: __DEV__,
  appId: ***,
  onInstallConversionDataListener: true,
  onDeepLinkListener: true,
  timeToWaitForATTUserAuthorization: 10,
});

appsFlyer.logEvent(
  "select_station",
  { ["station_id"]: 1, ["station_name"]: "station" },
  () => console.log("success"),
  (error) => console.log("failed", error)
);

What did you expect to happen?
Event to be logged in the dashboard
What happened instead?

The event is not showing in the dashboard
Success callback is called
No error returned

Please provide any other relevant information.
2023-08-21 18:08:28.210 27804-27973/*** I/AppsFlyer_6.12.1: sendWithEvent from activity: ***.MainApplication
2023-08-21 18:08:28.212 27804-27973/*** I/AppsFlyer_6.12.1: Trying to fetch GAID..
2023-08-21 18:08:28.277 27804-27973/*** I/AppsFlyer_6.12.1: ******* sendTrackingWithEvent: select_station
2023-08-21 18:08:28.290 27804-27973/*** I/AppsFlyer_6.12.1: IMEI was not collected.
2023-08-21 18:08:28.290 27804-27973/*** I/AppsFlyer_6.12.1: Android ID was not collected.
2023-08-21 18:08:28.312 27804-27973/*** I/AppsFlyer_6.12.1: AppsFlyer: first launch date: 2023-08-21_150723+0000
2023-08-21 18:08:28.329 27804-27973/*** I/AppsFlyer_6.12.1: AppsFlyerLib.sendWithEvent
2023-08-21 18:08:28.352 27804-27963/*** I/AppsFlyer_6.12.1: INAPP-13: preparing data: ************
2023-08-21 18:08:29.027 27804-27963/*** I/AppsFlyer_6.12.1: CACHE: caching request with URL: https://bv1p0o-inapps.appsflyersdk.com/api/v6.12/androidevent?app_id=***&buildnumber=6.12.1
2023-08-21 18:08:29.030 27804-27963/*** I/AppsFlyer_6.12.1: CACHE: done, cacheKey:<PHONE_NUMBER>027
2023-08-21 18:08:29.451 27804-27963/*** I/AppsFlyer_6.12.1: CACHE: Deleting<PHONE_NUMBER>027 from cache

Hi- this is exactly what I'm experiencing also. Even tried downgrading the package to 6.10.3 but no luck.

I am also facing the same issue. Is there any update here?

Any luck here?

i got this trouble also, i just do all step to test non-organic installs but then i open the app not thing show on test page. Please help me ?

I am still facing this issue. any solutions ??

Hi all,
Please ensure that the device being tested is first added as a test device in your AppsFlyer dashboard.
Afterward, follow these steps:
1.	Start with the non-organic test.
2.	Proceed with the events tests.
For detailed instructions, refer to this link: Testing the SDK Integration for Marketers.
Additionally, please note that we now have a Live Events Tests feature available on the Integration Tests page.
If additional assistance is needed, please don’t hesitate to contact support for further help.


Launcher is not started with specified ServiceAccountName

When specifying a MPI Job, a service account can be provided for the Launcher:
apiVersion: kubeflow.org/v1
kind: MPIJob
metadata:
  name: fe83205596f05414384b-n0-0
  namespace: dev
spec:
  mpiReplicaSpecs:
    Launcher:
      replicas: 1
      restartPolicy: Never
      template:
        metadata: {}
        spec:
          affinity: {}
          containers:
              image: someimage
              imagePullPolicy: IfNotPresent
              name: mpi
              resources:
                limits:
                  cpu: "2"
                  memory: 4000Mi
                requests:
                  cpu: "1"
                  memory: 1000Mi
          serviceAccount: default
          serviceAccountName: default

But the serviceAccountName is not being respected when creating the Launcher pod. The service account is named: mpiJob.Name + launcherSuffix. Also see pkg/controller.v1/mpi/mpijob_controller.go and function newLauncher.
This is a problem if permissions are bound to a certain service account and the launcher e.g. needs to access a resource in order to start a distributed task. Then it is crucial, that the specified service account name is being used.
So it turns out to be a showstopper for certain scenarios.
The following change should fix the problem:
Index: pkg/controller.v1/mpi/mpijob_controller.go
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/pkg/controller.v1/mpi/mpijob_controller.go b/pkg/controller.v1/mpi/mpijob_controller.go
--- a/pkg/controller.v1/mpi/mpijob_controller.go	(revision 3107ab73bb5334e8736b88320e97069b1bc40f3e)
+++ b/pkg/controller.v1/mpi/mpijob_controller.go	(date<PHONE_NUMBER>099)
@@ -1035,7 +1035,6 @@
 		jc.PodGroupControl.DecoratePodTemplateSpec(podSpec, mpiJob, rt)
 	}

-	podSpec.Spec.ServiceAccountName = launcherName
 	podSpec.Spec.InitContainers = append(podSpec.Spec.InitContainers, corev1.Container{
 		Name:            kubectlDeliveryName,
 		Image:           kubectlDeliveryImage,

Thanks for the issue. can you create a PR


Cannot install with pip

Could not find a version that satisfies the requirement python-gdcm (from versions: )
No matching distribution found for python-gdcm

What is the Python version?

@tfmoraes Python 3.8.3

What is the OS and architecture of your system? ARM? x86-64?

Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.6 LTS
Release:	16.04
Codename:	xenial
Architecture:          x86_64
CPU op-mode(s):        32-bit, 64-bit
Byte Order:            Little Endian
CPU(s):                16
On-line CPU(s) list:   0-15
Thread(s) per core:    2
Core(s) per socket:    8
Socket(s):             1
NUMA node(s):          1
Vendor ID:             AuthenticAMD
CPU family:            23
Model:                 1
Model name:            AMD Ryzen Threadripper 1900X 8-Core Processor
Stepping:              1
CPU MHz:               1997.335
CPU max MHz:           3800.0000
CPU min MHz:           2200.0000
BogoMIPS:              7585.07
Virtualization:        AMD-V
L1d cache:             32K
L1i cache:             64K
L2 cache:              512K
L3 cache:              8192K
NUMA node0 CPU(s):     0-15

I think your Python is not recognized as a manylinux2014. The wheels I submitted to Pypi are manylinux2014. Since I forgot to submit the tar.gz to Pypi your pip couldn't install it. Now that I submitted the tar.gz your system will build and install gdcm. You just need to install cmake, ninja-build and swig before the pip install using the apt.


Region support for all countries

I found that when parsing JHU data, we just throw away all region data by summing them up. Then we built a complex workaround for managing US states data.
Instead, we could treat all countries equally and have all country regions, supported by JHU, for cheap. It requires some refactoring but not a dramatic one. (We already have working code for Germany/RKI)

That would be a cleaner solution. The current situation is there for historic reasons (initially, there wasn't much subnational data available), and not ideal.

I looked into this more deeply and found that the main JHU dataset is not that detailed as I initially expected. Relatively easily I can add regional support for only a few countries: Canada, Australia and China. Therefore I would decrease the priority of this task for now.


nextjs examples code is not good

https://github.com/mui-org/material-ui/blob/next/examples/nextjs-with-typescript/pages/_document.tsx#L16
the nextjs official document : https://github.com/vercel/next.js/blob/master/errors/no-document-viewport-meta.md

Thanks for the concern, however no viewport are in the _document.


fix #12

It's usefull when you call docker run instand of  docker exec, as described in the issue.

Fixed in b3137e447c256a6ab345241986919ec132520723 and 930d0bbf7521301e2fe48b732c95bd569ebe30df


Create RKE2 clusters from Rancher

EPIC for RKE2 provisioning from Rancher
Expected result:
It should be possible to provision nodes for and create RKE2 clusters from Rancher.
Other details that may be helpful:
Depending on the details of the design, it should not be difficult to also provision nodes for and create k3s clusters from Rancher with the same implementation.
Todo:

[ ] rancher-machine https://github.com/rancher/rancher/issues/30597 (To Test)
[ ] ACE https://github.com/rancher/rancher/issues/31604 (To Test)
[ ] RKE2 Config https://github.com/rancher/rancher/issues/32174
[ ] Remove rancher-operator from feature charts https://github.com/rancher/rancher/issues/32179 (To Test)
[ ] Add support to pass files during machine provisioning https://github.com/rancher/rancher/issues/32231
[ ] Air-gap support https://github.com/rancher/rancher/issues/32403

I'm going to close this parent issue as it will be tracked in a different way.


3D UI goals / discussion

Overview
This is a meta issue to capture our goals for the UI for 3D interactivity.
Features

ability to select layers by clicking (#3)
an overlay with a local coordinate system that the user can interact with to translate/rotate the object. The objects may be the volume itself in the case of an Image layer or a plane that is used for something else (e.g., slicing). In Blender this is called a "manipulator"

An 3D axis tied to the canvas that rotates with the view. In blender this is the "gizmo"

H @kevinyamauchi , I see that this is quite an old issue but I have been wondering whether it's possible to bring a smaller version of this to napari-threedee? In particular, I would find a tool very helpful that simply indicates the Cartesian axes anywhere in the viewer. Would be happy to try myself at a PR with a pointer in the right direction :)

Hey @jo-mueller !

I would find a tool very helpful that simply indicates the Cartesian axes anywhere in the viewer.

I am not sure I am following. Do you mean a tool that adds the cartesian axis under the mouse in the scene?

I was thinking about something like an axis direction indicator that could be present somewher ein the viewer similar to how it is done in Blender:

When working with pointclouds/vectors/surfaces, I find the lack of direction indication sometimes a bit hindering. I typically solve it by adding three vectors layers that show the cartesian axes, but maybe this could be helpful in a more generalized way.

Hey @jo-mueller - we have 'Manipulator' objects which move in the scene, are you proposing a general purpose object which, like the gizmo in blender, would be fixed in screen space?

Hey @alisterburt , I think the gizmo would be pretty much what I'd have in mind but having seen the updates to come in napari 0.5.0 I think that may already be underway 🤔

oh weird! I'm disconnected from the napari-sphere right now so am not aware
of what's going on but cool if it's in the pipeline!
On Thu, 9 Nov 2023 at 07:26, Johannes Soltwedel @.***>
wrote:

Hey @alisterburt https://github.com/alisterburt , I think the gizmo
would be pretty much what I'd have in mind but having seen the updates to
come in napari 0.5.0 I think that may already be underway 🤔
—
Reply to this email directly, view it on GitHub
https://github.com/napari-threedee/napari-threedee/issues/4#issuecomment-1804044268,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABXYBYD7RY4SXIRCGZJVOKDYDTY2JAVCNFSM5IMGWFJKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOBQGQYDINBSGY4A
.
You are receiving this because you were mentioned.Message ID:
@.***>


CON-2727 added msrp to productlist and grid

Coverage decreased (-0.0006%) to 1.711% when pulling 11239377bf83372f882e9e1ab84435a34207ebfd on CON-2727-missing-msrp into cbf5979a7fb147ee0acaf19b2a6e51006d928104 on master.


TEST - [Portal Generated] Review request for Microsoft.Batch to add version 2018-29-11

If you are a MSFT employee you can view your work branch via this link.
Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-node

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-go

Automation for azure-sdk-for-java

Can one of the admins verify this patch?


better summary of symbols

@homunkulus r+

:pushpin: Commit 5c3b694 has been approved by japaric

:hourglass: Testing commit 5c3b694 with merge 5c3b694...

:sunny: Test successful - travis


Kas registry should allow naming

Blocked by https://github.com/opentdf/platform/issues/153

Closed by https://github.com/opentdf/otdfctl/pull/429


Fix RNN scoping situation

There is a long lasting problem of scoping which was introduced in original python wrappers early in H1. Basically each RNNCell implemented has to manually scope outputs of each of the operators. If somebody forgets, then there could be weird bugs with layers etc.
Approach is the following. User has to explicitly specify current scope when using  apply_over_sequence function and others if the function is going to be called several times (like for stacking layers). This way we use Caffe2 native scoping approach instead of inventing one extra API people have to use (i.e. passing scope name as an argument to the RNNCell constructor).

@caffe2bot retest this please


Update to version 3.20.1

Checklist

[x] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy
[x] Ensured the license file is being packaged.

Re-render was done locally.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


[2pt] Add unit tests for all implemented methods

Add unit tests for all implemented methods:

add_item method in the Genre class
can_be_archived?() in the MusicAlbum class

Finish


Support for more properties?

We wish to use editorconfig for a new project in our company. Some of our developers use Eclipse(STS) and some use NetBeans, but issue is this plugin doesn't support trim_trailing_whitespace and insert_final_newline, which are a must-have for us(the NetBeans plugin does support them).
Are there any timelines for support of these properties?

Hi @chaitanyathengdi thanks for your interest in this project. I did not plan to add support for "trim tailing whitespace" and "insert final newline" properties. I disagree with your view that these are "must haves";  in my experience, unlike indentation rules, these editor settings are not likely to change between files or projects so it's not a significant burden to manage them manually in Eclipse global configuration.
I expect it would not be difficult to implement basic support for these rules. I can provide some guidance with development if you are interested in contributing toward the feature.

I'd love to add my voice to this issue. We also use both these options in our projects (trim_trailing_whitespace: false, insert_final_newline: true) but would like our team to have different settings for their personal projects outside work.
(I can understand if you don't want to spend the work on it yourself, I mainly wanted to add that more people have this wish.)

@lode personally I'm avoiding Eclipse now so there is no chance I'm going to do any more work on this project. My recommendation is to use Intellij for Java and if that's not an option then find a job where Eclipse is not mandatory. I stuck with Eclipse for a long time just because it was so familiar but the features are really lagging behing Inteillij now.
A big chunk of my work in the last 12 months has been with Node and Vue and for that I'm just using Vim which is an absolute joy. I recommend taking on Spacemacs and Vim Adventures to anyone looking to get serious about productivity with their editor .

Thanks for the heads up! I personally don't use it either, just that in our team people use different editors. Familiarity is probably a big thing :)


Add support for downstream CodeFlare operator installation into QE tooling

Add support for installing the RHODS CodeFlare operator in a similar way how the RHODS operator is installed in the downstream QE tooling. The goal is to be able to install the downstream CodeFlare operator as a prerequisite for the RHODS operator in the downstream Jenkins job.

Moved to https://github.com/red-hat-data-services/distributed-workloads/issues/3


Cmd+click in the VSCode plugin should jump to source

Background & motivation
When reviewing HugoByte's code, I've very often wanted to cmd-click to jump to the source. Unfortunately, this isn't supported
Desired behaviour
Cmd+click jumps to the source of the function in VSCode if it's within the same package, and opens Github to the function if not
How important is this to you?
Painful; the lack of this feature makes using Kurtosis frictionful.

According to PK there's some complexities. Need to be properly scoped.
To be moved to backburner

this is a very good feature to have


rspec-rails 2.8.1 rake tasks unnecessarily called

Using Rails 3.1.3 and rspec-rails 2.8.1. Any call to rake appears to invoke "spec".

I can't reproduce this with a fresh rails 3.1.3 app. Did you upgrade from an older version? Also, what version of Ruby and what OS?

Incidentally if anybody else has this issue, be sure to include the command in quotes, i.e.
zsh$ bundle exec "rake db:migrate"


match the text in joint optimization diagram default view

It would be reduce cognitive load if the default selection for the joint optimization diagram matched the selection described in the text.
The text says black and white + mosaic.   The diagram could show this black and white + mosaic view by default.
suggested

current

Thanks Micah, that does make more sense. :-)


frozen

Describe the bug
{
"name": "dalle-image-generator",
"private": true,
"version": "1.0.0",
"type": "module",
"scripts": {
"dev": "vite",
"build": "vite build",
"preview": "vite preview"
},
"dependencies": {
"openai": "^4.20.1",
"dotenv": "^16.3.1"
},
"devDependencies": {
"vite": "^5.0.0"
}
}
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-uulmrm
Steps to reproduce
dall-e 3 api deploy ment failed to netlify
Expected behavior
??
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 56855504,
  "usedJSHeapSize": 52390168,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = GrayJediMaster
Chat ID = 1c417c98fff7

Additional context
No response

Hi @GrayJediMaster,
Are you saying the app failed to deploy to Netlify when you clicked the Deploy button. Did you receive a specific error message?

Yes! I would've spent maybe five hours trying to get it to deploy and when
it did there were initially issues because even though I had put the API
key in the .ENV section in Netlify - it was not able to retrieve it.
Then we tried to deploy it on a different platform by modifying front and
back codes and adding api in wix secret  - which also failed.
Then we went back to Netlify and when it did publish and I have the API key
in Netlify, it did not Deploy properly.
Meaning the preview inside of Molt worked but inside of Netlify it didn't.
I also noticed that when I tried to restore to prior versions that were
working - they didn't work any more.
I also downloaded the entire Project stack of code onto my hard drive and
uploaded back because those are the working versions and it still didn't
work when I did.
It seems to edit the code randomly even when not prompted, same way that if
you use ChatGPT or Claude they do hallucinate so maybe there's a
hallucination issues when it restores. There are other issues but I can't
remember.
I got many project files and can share them if need be.
Dany
On Tue, Nov 12, 2024 at 8:10 AM Brian @.***> wrote:

Hi @GrayJediMaster https://github.com/GrayJediMaster,
Are you saying the app failed to deploy to Netlify when you clicked the
Deploy button? Did you receive a specific error message?
—
Reply to this email directly, view it on GitHub
https://github.com/stackblitz/bolt.new/issues/1951#issuecomment-2469170584,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7IRTKGYWSGXYNZTZEGTZVL2AETOJAVCNFSM6AAAAABRQXYKBKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRZGE3TANJYGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Create button widget for showing "views" of the entity list (e.g. Exact Match/Related). #618.

Ticket
Closes #618.
Reviewers
@NoopDog.
Changes

Added button widget ToggleButtonGroup to facilitate switching "views" of the entity list.
Added placeholder config in explorer/site-config/ncpi-catalog-dug/dev/config.ts for display of widget on DUG.

Definition of Done (from ticket)

Added button widget to facilitate switching of "views" of the entity list.

Looks great thx!


Allow RTCP packets for keepalive also in state PLAY (#185)

Implement changes suggested in https://github.com/aler9/gortsplib/issues/185#issuecomment-1425653499

Codecov Report

Merging #186 (f1657bc) into main (6deba3e) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head f1657bc differs from pull request most recent head 5a47272. Consider uploading reports for the commit 5a47272 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #186   +/-   ##
=======================================
  Coverage   86.70%   86.71%
=======================================
  Files         101      101
  Lines       10729    10738    +9
=======================================
+ Hits         9303     9311    +8
  Misses       1096     1096
- Partials      330      331    +1

Impacted Files
Coverage Δ

server_session.go
77.93% <100.00%> (+0.15%)
:arrow_up:

server_session_media.go
93.68% <100.00%> (+0.09%)
:arrow_up:

client.go
74.34% <0.00%> (-0.28%)
:arrow_down:

server_conn.go
91.04% <0.00%> (+0.51%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

merged, thanks


split up long text block for database

User story

As a software developer
I need to split up long text blocks for storage in a database
So that the database performance and scalability are improved, and the searchability process is more precise.

Acceptance criteria

Develop a mechanism that splits the long text blocks into smaller segments.
Verify that the splitting mechanism correctly identifies and sets appropriate boundaries for the segments.
Assess the performance of the database with the segmented text segments, considering factors such as query response times & storage requirements,
Document the splitting mechanism.

Definition of done (DoD)

Acceptance criteria are met.
Work products are uploaded to the Github repository.
A pull request is created for each related branch.
The work products in the pull requests are reviewed.
The corresponding branches are merged and closed.
The bill of materials section of the planning documents is updated.
All defined conventions are complied with.

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

Stuff to do:

[ ] Annotate message with username
[ ] Change id to channel plus message id
[ ] Think about Chunk ids in database
[ ] Where to chunk

I see good progress :)
there is only one minor thing that came to attention while doing another issue (read confluence data).
without transform_text_to_chunks it looks like this:

with transform_text_to_chunks it looks like this:

Hope this helps to fix the issue :)

Thanks for the quick fix :)


[WIP] symbolic expression for symbolic pulse equating

Summary
Qiskit-Terra PR #9002, converted the library symbolic pulses from a complex amp representation to a real (float) amp,angle representation. To overcome the non-unique nature of the new representation (particularly, when negative amp is allowed), this PR adds an optional attribute params_comp_exp to store symbolic expression\s which are used to equate pulses.
Details and comments
Qiskit-Terra PR #9002, converted the library symbolic pulses from a complex amp representation to a real (float) amp,angle representation. To better support several experiments in Qiskit-Experiments, the amp parameter is allowed to take both positive and negative values. This decision creates a non-unique representation for the pulses, where a pi shift of angle and a sign flip of amp have the same effect (the issue obviously also exists for 2*pi shift of angle). For example, the following two pulses
from qiskit.pulse.library import Gaussian
import numpy as np

g1 = Gaussian(duration=100, sigma=50, amp=-1, angle=0)
g2 = Gaussian(duration=100, sigma=50, amp=1, angle=np.pi)

are not the same when equated (because their parameters are different):
g1 == g2 # False

but they represent the same waveform:
np.all(g1.get_waveform().samples == g2.get_waveform().samples) # True

This PR sets to correct this situation, by introducing a new attribute to the SymbolicPulse class - param_comp_exp. This new optional argument stores symbolic expression\s which are used when equating two pulses. Parameters appearing in these expressions are not equated individually, but only via the value of the total expression. For custom pulses, one can use any expression, but for the library pulses the expression in use is amp*sym.exp(sym.I*angle) which removes the problem. Returning to the example above, the new PR yields:
g1 == g2 # True

Discussion
The new addition is flexible and allows for a single expression or a list of expressions to be used. However, it is not obvious that such flexibility is needed. I can't think of an example where one would want two or more such expressions for a pulse.
Additionally, performance seems to be an issue. The new test added to test\python\pulse\test_pulse_lib.py is by far the slowest of the entire batch - despite testing a rather modest scenario. Furthermore, the new attribute will increase instances' size, for example, when saved to QPY.

[ ] Correct QPY loader for backwards compatibility.

@nkanazawa1989
Thanks, Naoki, for the insightful comments.
Correct me if I'm wrong, but it seems like your proposal hard codes the amp\angle conversion, thus assuming a role for these parameters even for custom built pulses? This is what we tried to avoid with PR #9002. To avoid role assumption, one has to somehow store in the specific instance the conversion mechanism from the "parameters" to the "canonical parameters". If we don't want it to be a symbolic expression, we need to find a different solution. (Storing a callable?)
I know we have only recently dropped the sub-classes (PR #8278), but it would seem like the easiest solution is to create subclasses where you can hard-code things whatever way you like (Note that the inability to differentiate library pulses from custom built ones also gave us a hard time in #9002). However, I am sure there was a good reason to drop out the subclasses.


How to add an additional step to chat agent?

I initialize the agent with create_llama_chat_agent and then call its run method.
I'm currently using llama_index version 0.5.27. But I can upgrade to branch 0.6 if needed.
I'm trying to solve this problem: After the agent has found the answer to the given question with or without the index, I would like to add an additional check for completeness of information in the given query in relation to the context found in the index and the answer received from the model. And in case the query wasn't complete enough (for example, it was too abstract and can be interpreted differently), the model should be forced to generate a new question instead of a response, asking for more specific information. Then, this request and the answer received from the user will be used as the context of the dialog, and the following answer will be more accurate, as the clarifying request complements the original one.
Unfortunately, I haven't found the right way to do this. Could someone give me a hint?

I would first try using customized prompts doing exactly what you described.

I tried to overwrite the refine_template by adding the line "If the original question is too general (for example "Will it work in my case?"), then, instead of answering, ask a new clarifying question."
This worked in some cases. But only sometimes. I would like to hear more variants.


Feat/file digest

Need tests to check the contents in the file are not missing.

I have modified two parts:

The hash option is now optional, i.e., similar to fileLength. The user does not add this field when calling the API, but if so, he/she need to make sure it is correct; this also will not infect the old APIs.
Instead of using promise, read the stream twice (first count the bytes, second upload) so that it has the same style as another issue.

The file contents check is already done here:
https://github.com/ideafast/ideafast-portal/blob/bbdb488db7b5088ae004dad3adf3a84e9d328c4c/packages/itmat-interface/test/serverTests/file.test.ts#L355-L361
@fguitton


Use Senzing SDK for Python packaging

See https://github.com/Senzing/knowledge-base/blob/master/proposals/python-packaging.md
See also https://github.com/Senzing/stream-loader/issues/246

Duplicate of https://github.com/Senzing/g2command/issues/32


How to define "additionalProperties" one as string other as array under the same object

I am looking for the json structure of the below kind using "additionalProperties" under metadata tag.
{
  "ingestRequest": {
    "document": {
      "metadata": {
        "additionalProp1": "string",
        "additionalProp2": "string",
        "additionalProp3": "string",
        "additionalProp1": [
          {
            "additionalProp1": "string",
            "additionalProp2": "string",
            "additionalProp3": "string"
          }
        ],
        "additionalProp2": [
          {
            "additionalProp1": "string",
            "additionalProp2": "string",
            "additionalProp3": "string"
          }
        ],
        "additionalProp3": [
          {
            "additionalProp1": "string",
            "additionalProp2": "string",
            "additionalProp3": "string"
          }
        ]
    }
    }
  }
}

using of "additonalProperties" more than once is not working as usual. Any workaround is helpful.

Do you really have additionalProp1 appearing more than once in your JSON instance, once as a string and once as an array? In this case the behaviour of JSON parsers will be undefined or 'unpredictable'.
If you mean something else (e.g. that additionalProp1 may be either a string or an array, or additionalProp1 represents two different property names, could you amend your example?
As you note, use of additionalProperties more than once in a JSON schema will usually fail, for exactly the same reason. additionalProperties defines a single sub-schema which constrains all of the additional properties not already covered by the schema definition.
This is also more of a generic JSON schema question, rather than one which (as yet) relates to the OpenAPI specification.

Thanks Mike. I mean to declare name:value pairs as string type and followed by array type using the additionalParameters option in the yml.
something like below :
{
"ingestRequest": {
"document": {
"metadata": {
"additionalProp1": "string",
"additionalProp2": "string",
"additionalProp3": "string",
"CustomerDetails": [
{
"CustomerNr": "12345",
"FName": "ABC",
"LName": "XYZ"
}
],
"CustomerDetails": [
{
"additionalProp1": "string",
"additionalProp2": "string",
"additionalProp3": "string"
}
],
"AddressDetails": [
{
"address1": "xyz",
"address2": "pqr",
"postcode": "12234"
}
]
}
}
}
}
Structure :

ingestDocumentMetadataRequest
document
metadata
<>: <> (key/value pair)
<> (array object)
<>: <> (key/value pair)

@kjapati in OAS 3.0, you can do something like:
{
    "additionalProperties": {
        "oneOf": [
            {
                "type": "string"
            },
            {
                "type": "array",
                "items": {
                    ...
                }
            }
        ]
    }
}

If this answers your question, please close out the issue.  You are also welcome to bring JSON Schema questions to the JSON Schema slack workspace

Ping @kjapati

It's been more than a year, closing.


Bunch 'o fixes

Make "make serve" actually work.

Eliminate superfluous "scroll left nav into view" button on screens that don't have a sidebar
(community & search result). Only matters on mobile.

Fix unpleasant wrapping in the footer when on mobile.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [geeknoid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


custom icon can not show on background mode

Expected Behaviour
show custom notification icon correctly
Actual Behaviour
notification icon does not show up on every devices' status bar and neither in the notification list
when app on background (custom icon work fine when app on foreground)
Reproduce Scenario (including but not limited to)
Steps to Reproduce
config.xml
<resource-file src="icon.png" target="res/drawable/icon.png" />
put icon.png into project root directory,
www/js/index.js
var push = PushNotification.init({
        android: {
            sound: true,
            vibrate: true,
            forceShow: true,
            icon: 'icon',
            iconColor: 'grey'
        },
        ios: {
            alert: true,
            badge: true,
            sound: true
        },
        windows: {}
    });

put app to background
send push from firebase console
got only white square icon
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Android 6.0.1
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Sony Z3 , Android Emulator
Cordova CLI version and cordova platform version
cordova --version                                    # 7.0.1
cordova platform version android                     # 6.2.3

Plugin version
cordova plugin version | grep phonegap-plugin-push   # 2.0.0-rc5 "PushPlugin"

Sample Push Data Payload
send notifications from https://console.firebase.google.com/
Sample Code that illustrates the problem
Logs taken while reproducing problem

@yelban sounds like your icon has no transparent pixels in it, which would account for the white square. https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/PAYLOAD.md#images

@macdonst my icon do transparent pixels, the icon display normally when app on foreground mode, the white square only happens app on background mode
icon work fine when app on foreground mode

icon become white square when app on background

the icon do transparent pixels

@yelban okay, in that case I think you are running into this https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/PAYLOAD.md#notification-vs-data-payloads
When you send via the Firebase console I believe it put everything in notification, not data.

@macdonst Thank you for giving me good advice.
Switch to use Firebase Admin SDK to send payload a as "PAYLOAD.md#notification-vs-data-payloads" mentions, icon problem resolved !
Thanks,
Yelban.


ARROW-13185: [MATLAB] Create a single MEX gateway function which delegates to specific C++ functions

Overview
This pull request implements a more scalable and sustainable approach to organizing the C++ functionality that needs to be exposed to MATLAB. It adds a new singular MEX gateway function, mexfcn('<cpp-function-name>', <cpp-function-argument-1>, ..., <cpp-function-argument-N>), which delegates to specific C++ functions. To make use of mexfcn, the directory containing mexfcn.<mex-extension> must be added to the MATLAB path.
Advantages to this approach:

Organization

All C++ functions that are exposed to MATLAB are registered in one location
Reduce the complexity of managing and linking many MEX files and ensuring that they are added to the MATLAB path
Reduce cognitive load for adding new functions

Avoid polluting the source tree with build artifacts
Reduce build times, building a single MEX function is faster than building potentially hundreds
Reduce binary bloat caused by creating a separate MEX file for every MEX function
Enable flexibility in terms of where C++ implementation files live

Implementation

One MEX function, mexfcn, defined in matlab/src/cpp/arrow/matlab/mex/mex_util.h, dispatches to individual C++ implementation files.

For example, to invoke featherread functionality from MATLAB, that is implemented in C++:
>> [variables, metadata] = mexfcn('featherread', 'featherfile.feather');

Functionality implemented in C++ that we want to expose in MATLAB is registered in a function map in the file matlab/src/cpp/arrow/matlab/mex/mex_functions.h.
Restructured source tree layout and performed general code clean up in preparation for feature implementation work:
3.1. Split source code to matlab/src/matlab and matlab/src/cpp
3.2 Make packages, namespaces, and directories consistent in terms of naming and hierarchy for simplifying navigation and header inclusion.
3.3 Renamed the MATLAB package name mlarrow to arrow as the ml is superfluous.
Refactored matlab/CMakeLists.txt:
4.1. Build shared library, arrow_matlab, that contains C++ functionality for the interface.
4.2. macOS: explicitly add the path of arrow to the rpath of arrow_matlab, as paths of libraries output by imported targets are not automatically included.
4.3. Windows:

Copy shared library, arrow.dll to the directory where the MEX function lives.
Add the path to MATLAB and GTest shared libraries to the ctest ENVIRONMENT when building tests.
Specify the release version of MSVC Runtime Libraries for all targets created in the CMake file.

Testing
Qualified CMakeLists.txt changes by building and running all tests:

On Windows 10 (Ninja and Visual Studio), macOS 11.5 (Make and Ninja), and Debian 10 (Make and Ninja)
Configurations: build both Arrow and GTest, use provided ARROW_HOME, use provided GTEST_ROOT, use both ARROW_HOME and GTEST_ROOT.

Future Directions

Currently, users on macOS can use Arrow and GTest binaries that were built independently, but they cannot relocate the Arrow and GTest binaries built via the MATLAB Interface build system and reuse them. On macOS, shared libraries contain the paths of linked libraries, therefore, relocating them invalidates the libraries' rpath entries. On Windows and Linux, a user can:

Build Arrow and GTest binaries in the MATLAB Interface build system
Move those binaries to a separate location outside of the build tree
Use those binaries for subsequent builds, pointing to them using ARROW_HOME and GTEST_ROOT

Investigate why the default CMake behavior does not link the test executables against the correct MSVC Runtime libraries (ie. ucrtbase.dll versus ucrtbased.dll) when building with Ninja on Windows.
Add support for specifying function names and arguments as MATLAB strings to mexfcn. Currently, only character vectors are supported.
Refactor mexfcn to use MATLAB Data Arrays (MDAs) and C++ Mex.
For multi-configuration generators, the binaries are built into a subdirectory indicating the build configuration. In CMakeLists.txt, create a text file, is_multi_config.txt, that lists the configuration used. Then, use this configuration text file in the MATLAB tests to ensure they are able to find the MEX file.

Notes

Thank you for all of your help on this pull request, @sgilmore10 and @kevingurney!

I apologize for the size of this pull request. We found it tricky to partition our changes into separate pull requests as they involved modifying the build system and refactoring the structure of the source files. However, in the future, our goal is to submit smaller, more incremental pull requests to reflect more isolated changes.

Thank you for your feedback, @kou!
At this time, [matlab-actions/setup-matlab](https://github.com/matlab-actions/setup-matlab) only supports Linux-based VMs in GitHub Actions. I have created two Jira tickets to track adding GitHub Actions jobs on macOS and Windows once the two platforms are supported:

macOS: ARROW-15195
Windows: ARROW-15196

Thank you for checking in about ARROW-13564, it does not need to block this pull request. Because each of the co-authors pushed commits to this pull request, I think everyone will be represented in the final commit after the pull request is merged.
I have addressed your code review feedback, please refer to the individual threads for the code specific comments.

Simply out of curiousity, mexfcn does not supports C++ functions that take FunctionOptions in addition to the function arguments?

@edponce, mexfcn only receives input arguments from MATLAB. End users will interact with the MATLAB APIs, which will make use of mexfcn. mexfcn will dispatch to individual C++ functions that implement the necessary function specific behavior.
Depending on the needs of a particular C++ function, developers can pass options in the way that they deem appropriate, in order to get that information to the C++ layer. mexfcn accepts a variable number of input arguments, of any type, from MATLAB. So a developer could pass an options struct to mexfcn in order to propagate any options to the C++ layer. The translation from the options struct to C++ data structures would need to be taken care of by the specific C++ functions.

Thank you for code reviewing this change and accepting the pull request, @Kou!
I am able to see the commit that merged this change into the main branch, however, I see a message on this conversation thread named "Closed with unmerged commits". Do you know what causes this message and are there any action items for me to resolve it?

@lafiona Sorry for confusing you by our merge method. You don't need extra action for this pull request.
We use our merge script https://github.com/apache/arrow/blob/master/dev/merge_arrow_pr.py to merge a pull request. It squashes commits in the pull request, pushes the squashed commit to master and updates the correspondent JIRA issue. We don't use GitHub's "merge button". So  "Closed with unmerged commits" is seen on GitHub UI. But you don't need extra action because we have changes in this pull request on master.

@Kou, no worries, thank you for explaining why this occurs! Good to know that it is expected behavior. Thanks again for your time reviewing this change!


I am struggling to understand how to configure caddy with coraza.

It took me a while and I managed to first install caddy as a reverse proxy.
Then after this worked for a week I tried to add coraza to the picture.
I eventually managed to build caddy with coraza support but now I am struggling to understand how to configure it with the CRS.
I looked at:
https://coraza.io/docs/tutorials/coreruleset/
and the files sits at: /opt/src/coreruleset ie:
/opt/src/coreruleset# ls -la  /opt/src/coreruleset/
total 28
drwxr-xr-x  3 root root 4096 Dec  8 23:58 .
drwxr-xr-x  8 root root 4096 Dec  8 23:57 ..
-rw-r--r--  1 root root 9443 Dec  8 23:57 coraza.conf
drwxr-xr-x 10 root root 4096 Dec  8 23:59 coreruleset
-rw-r--r--  1 root root   46 Dec  8 23:57 links

# ls -la  /opt/src/coreruleset/coreruleset/
total 292
drwxr-xr-x 10 root root   4096 Dec  8 23:59 .
drwxr-xr-x  3 root root   4096 Dec  8 23:58 ..
-rw-r--r--  1 root root 114584 Dec  8 23:57 CHANGES.md
-rw-r--r--  1 root root  27903 Dec  8 23:57 CONTRIBUTING.md
-rw-r--r--  1 root root   6085 Dec  8 23:57 CONTRIBUTORS.md
-rw-r--r--  1 root root  31922 Dec  8 23:57 crs-setup.conf.example
drwxr-xr-x  3 root root   4096 Dec  8 23:57 docs
-rw-r--r--  1 root root    519 Dec  8 23:57 .editorconfig
drwxr-xr-x  8 root root   4096 Dec  8 23:57 .git
drwxr-xr-x  5 root root   4096 Dec  8 23:57 .github
-rw-r--r--  1 root root    662 Dec  8 23:57 .gitignore
-rw-r--r--  1 root root    151 Dec  8 23:57 .gitmodules
-rw-r--r--  1 root root  13515 Dec  8 23:57 INSTALL
-rw-r--r--  1 root root   2776 Dec  8 23:57 KNOWN_BUGS.md
-rw-r--r--  1 root root  11351 Dec  8 23:57 LICENSE
-rw-r--r--  1 root root    287 Dec  8 23:57 .linelint.yml
drwxr-xr-x  2 root root   4096 Dec  8 23:57 plugins
-rw-r--r--  1 root root    438 Dec  8 23:57 .pre-commit-config.yaml
-rw-r--r--  1 root root   3011 Dec  8 23:57 README.md
drwxr-xr-x  4 root root   4096 Dec  8 23:57 regex-assembly
drwxr-xr-x  2 root root   4096 Dec  8 23:57 rules
-rw-r--r--  1 root root   4511 Dec  8 23:57 SECURITY.md
-rw-r--r--  1 root root    102 Dec  8 23:57 SPONSORS.md
drwxr-xr-x  5 root root   4096 Dec  8 23:57 tests
drwxr-xr-x 17 root root   4096 Dec  8 23:57 util
-rw-r--r--  1 root root    751 Dec  8 23:57 .yamllint.yml

I have never used CRS or modsecurity so I will readlly appreciate any help.
I will give an example caddy file which can be used to set the example up:
{
        email<EMAIL_ADDRESS>        order coraza_waf first
}

https://example.com {
        coraza_waf {

        }

        header * x-request-id "{http.transaction_id}"
        reverse_proxy https://<IP_ADDRESS>:443 {
                header_up X-Forwarded-Proto "https"
                transport http {
                        tls_insecure_skip_verify
                }
        }
}

I will appreciate any help to understand how to do the most basic configuration, From there I assume it will be pretty simple since it's basic CRS configurations.
Thanks,
Eliezer

Hey! The easiest way to load the CRS is the one described by Using OWASP Core Ruleset section of the Readme of this repo. Also, this repo comes with an out of the box example. Its CaddyFile is under example/CaddyFile, here you can see some back configuration overrides, custom rules etc.
If you wish to point to a local version of the CRS, It should be enough to customize the Include directives( E.g. Include @coraza.conf-recommended) pointing to your files instead of relying on the aliases loaded thanks to load_owasp_crs


[macOS] Fix open file limits

Description
Workaround for limit applying thru SIP restrictions.
Related issue:
https://github.com/actions/runner-images-internal/issues/5314
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run macos11,macos12


Smelter Part

Tracker for the 'Smelter' part.  See diagram in #2
Discussion:

[ ] Form Factor
[x] Amount of Kerbals it can hold
[ ] Cost
[ ] Mass

Needs:

[ ] Model
[ ] Description
[ ] Config File

I believe this is a 0-Kerbal part.

Part Config: https://github.com/DanStaal/DStaal_UKS_KPBS_Addons/blob/Community_Dev/New_Parts/Smelter.cfg


Instructors' Question View

Clicking on a question on #32 will lead here. Detailed view of question along with answer options and  ability to start polling.
Wireframe:

For more context, the instructor would spend most of their time during class on this page. Whenever they'd like to ask a question, they would click on the given question from the list of questions.

It might also be worthwhile for an instructor to create a question on the fly. They would click a "new question" button that would start a question without any text, and the default A-E options.

@sathp I don't think the picture matches this issue

Hi, I would like to take this issue! Could you assign it to me?


Develop New UI

Coverage remained the same at 100.0% when pulling 2a8fe50ba91f0cf230819c86d576cef7e82fc1c3 on develop into da82e2d21d9067bfaefe6fbfec385c62d8054414 on master.

Coverage remained the same at 100.0% when pulling 2a8fe50ba91f0cf230819c86d576cef7e82fc1c3 on develop into da82e2d21d9067bfaefe6fbfec385c62d8054414 on master.

Coverage remained the same at 100.0% when pulling 2a8fe50ba91f0cf230819c86d576cef7e82fc1c3 on develop into da82e2d21d9067bfaefe6fbfec385c62d8054414 on master.


modify: allow to init only for selected element

More flexibility if you don't want to enable it for whole document or might init modules inside of HTML loaded via Ajax.

:+1: regarding the limited scope - this may help with products that already have some frameworks used and someone's migrating slowly to Carpet. I like it.
Regarding the dynamically added content to the DOM Tree, how's that going to work with already initialised modules? Is it going to affect them in any way or keep untouched?

It should not affect them, but it needs to be checked properly.

I like the idea. It should work the same as reinitiating carpet (double use of Carpet.init()).
Please resolve conflicts and write unit tests for this change. If you need any help or have questions don't hesitate to write here.

I'll sort it out next week.
By the way, I would remove init() in fond of riseOfTheClonesModules, just because it's cooler:

@mateuszgachowski Test added.

This build fails at Travis-CI because of some weird Travis local env issues :/

@sobstel This fails, because you just forgott to use the new element variable as context for querySelectorAll ;)

@yckart :flushed:
Fixed. (it did not help with travis though: https://travis-ci.org/mateuszgachowski/Carpet.js/builds/114298923)

@sobstel I moved those changes with solved conflicts to a feature branch and soon will merge this to master.
Ill try to solve Travis problem soon.


openphoto Amazon AMI repeatedly refreshes all pages

I tried to install the software using an Amazon EC2 instance and the openphoto pre-configurated image. When the software came up, the setup page was constantly refreshing.

Closing because development has ceased on this project.


Update sbt-jmh to 0.4.6

About this PR
📦 Updates pl.project13.scala:sbt-jmh from 0.4.5 to 0.4.6
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "pl.project13.scala", artifactId = "sbt-jmh" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "pl.project13.scala", artifactId = "sbt-jmh" }
}]

labels: sbt-plugin-update, early-semver-minor, semver-spec-patch, commit-count:1

Codecov Report
Patch has no changes to coverable lines.
:loudspeaker: Thoughts on this report? Let us know!.


Fix broken link in storage/file documentation

The file.createResumableUpload documentation says to refer to storage.createWriteStream, but this method doesn't exist. I assume this should actually refer to file.createWriteStream, which appears to exist.

Coverage remained the same at 100.0% when pulling d8c4a9e52e3a0c70a9a87752c9088298ea91a48e on jergason:patch-1 into 9e78eba27177a891742f156899a7689b089cb39e on GoogleCloudPlatform:master.

Thanks! We have a code freeze on Storage right now, but we will address this eventually :)

Coverage remained the same at 100.0% when pulling 2a1bb95777bff46ab037c45a5b1a934bfdd521a4 on jergason:patch-1 into 9e78eba27177a891742f156899a7689b089cb39e on GoogleCloudPlatform:master.

I will get this moved over really quick.

Fixed in https://github.com/googleapis/nodejs-storage/pull/29. Thanks, @jergason!


API Use mysql-safe table namespace separator

The idea behind the original behaviour is that pre-namespaced tables in 3.x would continue to work.
However, namespaced dataobjects were not feasible in 3.x. They were theoretically possible, but ran into several edge cases which made it impossible in practice. I don't believe it's worth trying to soft-support automatic upgrades for these classes in this case.
As a result of this change, all new namespaced classes (without any table_name) will be buildable in mysql.
Note: Please leave this PR open for a few days for public approval. There are a few opinions that need to be considered on the matter. :)

I'm more concerned about relation conflicts as they are both defined outside the framework and could cause issues.
The fact we hand build DB identifiers rather than use Convert::symbol2sql makes using a - much harder than it should be 😞


self.t = -1

@Naereen
I have an issue with BasePolicy.init
We have self.t = -1 and self.t =0 in BasePolicy.startGame
Should not it be self.t =0 for both ?
https://github.com/SMPyBandits/SMPyBandits/blob/master/SMPyBandits/Policies/BasePolicy.py

PS : SWUCB and DiscountedUCB do not work with current time initialization.
see https://github.com/SMPyBandits/SMPyBandits/pull/214

I'll fix this myself in another comment, there is about a dozen of files to fix for this.


Apollo 2.5 Hardware Installation Guide

Apollo 2.5 Hardware Installation Guide

 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.


Source object is ignored if it has a type other than a regular "object"

Bug Report
I would expect the resulting merged object to retain its type, like the "merge" function does in the "lodash" library. This issue makes this package not as useful as lodash, and not a suitable replacement since any unit testing of objects will result in a type mismatch.
Expected behavior
Merge should be successful if the source object (first argument) has a type.
Actual behavior
Source object is ignored if it has a type other than a regular object.
Steps to reproduce
import { merge } from "lodash";
import { deepmerge } from "deepmerge-ts";

class Object1 {
  value1: string | undefined;
  value2: string | undefined;
}

let object = new Object1();
object.value1 = "val1";
console.log(object);

let mergedLodash1 = merge({ value1: "val1" }, { value2: "val2" });
console.log("mergedLodash1", mergedLodash1);
// mergedLodash1 {value1: "val1", value2: "val2"}

let mergedLodash2 = merge(object, { value2: "val2" });
console.log("mergedLodash2", mergedLodash2);
// Object1 {value1: "val1", value2: "val2", constructor: Object}

let mergedDeepmerge1 = deepmerge({ value1: "val1" }, { value2: "val2" });
console.log("mergedDeepmerge1", mergedDeepmerge1);
// {value1: "val1", value2: "val2"}

let mergedDeepmerge2 = deepmerge(object, { value2: "val2" });
console.log("mergedDeepmerge2", mergedDeepmerge2);
// {value2: "val2"}

Proposed changes

This is actually intended behavior, and is considered a feature of this library.
This is because in general, we have no idea how to merge non-record objects. If you want to handle such objects, you need to tell the library how it should merge them by defining a custom merger with deepmergeCustom.
If you want to simple treat all objects as records, you can use the following:
import { deepmergeCustom, type DeepMergeLeaf, type DeepMergeMergeFunctionsURIs } from "deepmerge-ts";

declare module "deepmerge-ts" {
  interface DeepMergeMergeFunctionURItoKind<
    Ts extends ReadonlyArray<unknown>,
    MF extends DeepMergeMergeFunctionsURIs,
    M
  > {
    readonly DeepMergeLeafTreatAllObjectsAsReocrdsURI: Ts[number] extends object
      ? DeepMergeRecordsDefaultHKT<{ [I in keyof Ts]: { [J in keyof Ts[I]]: Ts[I][J]; }; }, MF, M >
      : DeepMergeLeaf<Ts>;
  }
}

export const deepmergeObjectsAsRecords = deepmergeCustom<{
  DeepMergeOthersURI: "DeepMergeLeafTreatAllObjectsAsReocrdsURI";
}>({
  mergeOthers: (values, utils, meta) => {
    let allObjects = true;

    // Convert all objects to records.
    const asRecords = values.map((v) => {
      if (typeof v === "object" && v !== null) {
        return { ...v };
      }
      allObjects = false;
      return false;
    });

    // If only merging objects, merge them as records.
    if (allObjects) {
      return utils.mergeFunctions.mergeRecords(asRecords, utils, meta);
    }

    // Otherwise, do the normal merge.
    return utils.actions.defaultMerge;
  },
});


Create buildspec.yml

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

done

merge please


Initial state

This PR is a placeholder to keep track of the changes I made for https://github.com/MetaMask/MetaMask/compare/redux-persist?expand=1

Codecov Report

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

@@          Coverage Diff          @@
##           master    #16   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          14     14
  Lines         601    601
  Branches       72     72
=====================================
  Hits          601    601

Continue to review full report at Codecov.

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


test: set up basic test infrastructure (1.x)

See also #191

@kptdobe I assume your approval on #191 is good here, too.


Add Artwork page to share logos

See https://trello.com/c/FpxHM0Aw/70-artwork
Shall I link to the svg files in this repo for people to download? Or do we want to upload the logos to downloads.openmicroscopy.org? Do we want to include another format and have a zip bundle?

Staged at https://snoopycrimecop.github.io/www.openmicroscopy.org/artwork/ and https://snoopycrimecop.github.io/www.openmicroscopy.org/site-map/ - looks like I'll need to do something about the display size, although my local build looked fine, they are displaying bigger and in need of some whitespace borders on the snoopy staging site.

Now I realise we'll obviously need at least one zip bundle because we have one link for all the product logos.

https://snoopycrimecop.github.io/www.openmicroscopy.org/artwork/ updated

Minor suggestions:

"Download Products Logos & Marks"

the problem with adding all the product marks is that it messes up the layout of the page - I tried playing with it but couldn't get it to work nicely so this may require some further styling from @lunson

https://snoopycrimecop.github.io/www.openmicroscopy.org/artwork/ updated with a rather hacked layout - it looks okay to me but I won't be offended if you want to restyle the page @lunson ;)

I certainly prefer this to the previous. I'd be happy to go with it, and we can add improvements.

Great job, @hflynn!

I assume the next step following live deployment might be to add a redirect for http://www-legacy.openmicroscopy.org/site/art as well?

Yes, I'll open a PR for that now @sbesson


Switch to using the swag library for pointer manipulation.

And also handle the roothash.
Followup from #718
Signed-off-by: Dan Lorenc<EMAIL_ADDRESS>
Summary

Ticket Link

Fixes
Release Note

Codecov Report

Merging #719 (a236c8c) into main (5e5aa9b) will increase coverage by 0.09%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #719      +/-   ##
==========================================
+ Coverage   47.50%   47.60%   +0.09%
==========================================
  Files          65       65
  Lines        5675     5670       -5
==========================================
+ Hits         2696     2699       +3
+ Misses       2688     2681       -7
+ Partials      291      290       -1

Impacted Files
Coverage Δ

cmd/rekor-cli/app/log_info.go
2.97% <0.00%> (+0.14%)
:arrow_up:

pkg/types/alpine/v0.0.1/entry.go
62.01% <0.00%> (+1.16%)
:arrow_up:

Continue to review full report at Codecov.

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


Add support for solc path

Added support for solc path, which can be passed to solcVersion in waffle.json.

Document the change


[Bug] Unable to switch MPR series

Describe the Bug
The first series of MPR operation, the page can be rendered normally, double-click the second series, the page will be black and can not render, and if you return to the first series will also be black

The browser reports an error message

Steps to Reproduce
1.According to dicomjson data source,
2.the MPR of the first series is normal,
3.double-click the second series when building the first series,
4 the page will be black, and then the corresponding image cannot be rendered on the canvas
The current behavior
Stuck, black screen, page not rendered
The expected behavior
Double-click on different series to build different series of MPR
OS
win11
Node version
16.20.2
Browser
chrome latest

Can you try with latest and if not working re open this issue please


Some issues about Evaluation

Thanks for your interesting work!
I have met some problems when I tried to evaluate your model. I have downloaded the Data, Annotations, and Checkpoints following your guidance in README.md. However, when I run the evaluation code, I got all 0.000~0.001 in Bleu, METEOR, Rouge, CIDEr score. Maybe it is not correct?
I even tried training the second stage on my computer, but it also didn't work. Since I followed your guidance and your code step by step, I didn't know where I got wrong. What do you think about that? I am looking forward to your reply.

I will fix it as soon as possible.
On Tue, Apr 23, 2024 at 1:22 PM Fugtemypt123 @.***>
wrote:

Thanks for your interesting work!
I have met some problems when I tried to evaluate your model. I have
downloaded the Data, Annotations, and Checkpoints following your guidance
in README.md. However, when I run the evaluation code, I got all
0.000~0.001 in Bleu, METEOR, Rouge, CIDEr score. Maybe it is not correct?
I even tried training the second stage on my computer, but it also didn't
work. Since I followed your guidance and your code step by step, I didn't
know where I got wrong. What do you think about that? I am looking forward
to your reply.
—
Reply to this email directly, view it on GitHub
https://github.com/Pointcept/GPT4Point/issues/18, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWVKZMPU6LK2VJUB5R6VE6LY6XVYFAVCNFSM6AAAAABGUCSM7WVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI2TOOJXGEYDSOI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I will fix it as soon as possible.
…
On Tue, Apr 23, 2024 at 1:22 PM Fugtemypt123 @.> wrote: Thanks for your interesting work! I have met some problems when I tried to evaluate your model. I have downloaded the Data, Annotations, and Checkpoints following your guidance in README.md. However, when I run the evaluation code, I got all 0.000~0.001 in Bleu, METEOR, Rouge, CIDEr score. Maybe it is not correct? I even tried training the second stage on my computer, but it also didn't work. Since I followed your guidance and your code step by step, I didn't know where I got wrong. What do you think about that? I am looking forward to your reply. — Reply to this email directly, view it on GitHub <#18>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AWVKZMPU6LK2VJUB5R6VE6LY6XVYFAVCNFSM6AAAAABGUCSM7WVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI2TOOJXGEYDSOI . You are receiving this because you are subscribed to this thread.Message ID: @.>

Thanks!

I have corrected the error and now you can run the eval code directly to get the result.