"zoomBounds" not necessarily present in ChargePoint config json

I have the ha-chargepoint integration installed in my personal Home Assistant setup and it works great! However, I recently had to reconfigure the integration and it's failing to authenticate with ChargePoint.
2023-02-24 22:32:03.626 ERROR (MainThread) [custom_components.chargepoint.config_flow] Attempting to authenticate with chargepoint
File "/config/custom_components/chargepoint/config_flow.py", line 110, in async_step_user
File "/config/custom_components/chargepoint/config_flow.py", line 126, in _update_options
File "/config/custom_components/chargepoint/config_flow.py", line 28, in _test_credentials
File "/usr/local/lib/python3.10/site-packages/python_chargepoint/client.py", line 78, in __init__
File "/usr/local/lib/python3.10/site-packages/python_chargepoint/client.py", line 177, in _get_configuration
config = ChargePointGlobalConfiguration.from_json(response.json())
File "/usr/local/lib/python3.10/site-packages/python_chargepoint/global_config.py", line 116, in from_json
File "/usr/local/lib/python3.10/site-packages/python_chargepoint/global_config.py", line 117, in <listcomp>
ChargePointCountry.from_json(country)
File "/usr/local/lib/python3.10/site-packages/python_chargepoint/global_config.py", line 45, in from_json
zoom_bounds=ChargePointZoomBounds.from_json(json["zoomBounds"]),

I did some debugging and figured out the problem is this "zoomBounds" object isn't necessarily present in every supportedCountries object returned by the ChargePoint API call. Maybe they added some countries recently and some don't have this value?

Canada and Denmark have a "zoomBounds" entry, but Czech Republic doesn't. Maybe the supportedCountries list was updated recently with countries that don't have this object?
The solution here would be to support the possibility that this "zoomBounds" object isn't present. I would submit a pull request with proposed changes, but I'm not familiar enough with the code base to change that to an optional object. Hopefully this information helps make the fix easier to implement!

This is fixed in the latest version of both this module and the HA plugin. 😄 Thanks for the heads up.


[API] Idea dump

I had some time to burn so I refactored the library to roughly how I would do it. Feel free to tell me what you like and don't like.
This is an implementation of the SimRobot idea we talked about a week ago.
The user would grab the robot from the simulation arena and can either pass it throughout their subsystems or statically access it. The user then uses this robot to initialize their drivetrain sim, gamepiece storage, mechanisms/motors and intakes.
A somewhat comprehensive list of changes:

Renamed MapleMotorSim to MechanismSim
SwerveDriveTrainSimulation now has it's own SwerveConfig that extends the base config
Swerve module and gyro now have their own configs
Intakes now are backed by a GamePieceStorage object that can be shared between intakes and other subsystems
GamePiece changes from previous PR
GamePiece now has a user control invariance that only lets user mutating functions work if they should have control over the GamePiece
IntakeSimulation, MechanismSim, GamePieceStorage, and DriveTrainSimulation are all constructed through SimRobot now
GamePieces can be cleared from the arena based on their current state (aka clear all Limbo)
SimulatedArena timing methods are now non-static
Added RuntimeLog for better user feedback on warnings/errors

Most of the code looks impressive me, can you please test the new API in advanced_swerve_example so that we can see how the new API works in real-life code?
Also, please change the target of the PR to
https://github.com/Shenzhen-Robotics-Alliance/maple-sim/tree/new-game-piece-API
thanks a lot for contributing!

As always, really impressed by your work!
a few things though, please check the reviews

The whole library has moved to wpilib units, this means that merging it using git is seemingly impossible.
I'll just merge it to a temporary branch and manually commit changes to dev instead.
Anyways, thanks a lot for all the work!  This is amazing.


adding LHCb geometry

Added LHCb geometry and a macro file with LHCb sensitive detector. Also added the possibility of some angular smearing in the particle gun.

The macro lhcb.mac.in and new gdml geometry files are currently not copied in the bin directory, as the others files in the macros directory, which is convenient since it produces a macro that can be run w/o copying files. Please check how this is done for testEm3 geometry and macro in the CMakeLists.txt

added now

There is still an annoyingly long list of names printed by the example, but it is not the example itself to blame but the G4GDMLRead during the solids section.

The GDML files are missing the license header, you can add it as for teastEM3.gdml in the same folder

The long list of names comes from a forgotten std::cout left in G4GDMLReadSolids::PropertyRead. I will fix it in the next Geant4 patch.


chain(r1, r2, r3, ...) should discard all arguments after the first infinite range

andrei (@andralex) reported this on 2016-04-15T18:10:01Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=15928
CC List

schuetzm

Description
Consider a call chain(r1, r2, r3, ..., rk, ..., rn) where range rk is infinite. Then that should return the same exact result (type and behavior) as chain(r1, r2, r3, ..., rk). This is because chaining effectively stops at the infinite range and simply continues with it forever.

Obviously a good case is chain(r1, r2) which simply returns r1 if infinite.

An alternative would be to refuse compilation - this use may be a bug more often than not. Thoughts?

schuetzm commented on 2016-04-15T18:20:48Z
Not sure about refusing compilation - things like this could appear in otherwise valid generic code...

Anyway, bidi ranges need special treatment - they need to be accessible from the end up to the last infinite range, too.

andrei (@andralex) commented on 2016-04-15T18:28:52Z
@Marc didn't think of that. But then what do you do when you come back and you finish the bidir ranges after the infinite one. So then you have a non-empty range that you can't pop back from anymore. The mind boggles...

I agree about refusal to compile - let it ride!

schuetzm commented on 2016-04-15T19:23:13Z
Ah... I now remember the restriction that infinite ranges cannot be bidirectional. I think this was a conscious decision in order to get a simpler range hierarchy, but there was no fundamental reason to disallow it, right?

This restriction simplifies things for chain(): if any of the individual ranges are infinite, the resulting chain cannot be bidirectional.


Iterbreak

This adds a break block that can be used to break out of for-range, for-each, and while loops.  Executing break terminates execution of the innermost loop.   It is an  error to have a break block that is not contained in a loop.
Note that this pull request does not implement continue, only break.

LGTM

One potential concern I have is that we aren't bumping the blocks language with this change. When we deploy it to ai2-test, we will not want users attempting to bring projects back to ai2 until we update the production system so that the language semantics are the same.

Evan,
Want to take  second look and merge into UCR?
== Hal
Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
On Tue, Feb 13, 2018 at 1:57 PM, Farzeen Harunani<EMAIL_ADDRESS>wrote:

LGTM
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/1142#issuecomment-365367725,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAwLt6_-L8kIIkBwSRb4jKpKdjBLdHFQks5tUdsJgaJpZM4Rl9T8
.

Evan,
Good catch.   We should bump the blocks language.
== Hal
Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
On Tue, Feb 13, 2018 at 2:50 PM, Evan W. Patton<EMAIL_ADDRESS>wrote:

One potential concern I have is that we aren't bumping the blocks language
with this change. When we deploy it to ai2-test, we will not want users
attempting to bring projects back to ai2 until we update the production
system so that the language semantics are the same.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/1142#issuecomment-365383305,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAwLt1BmXUntucMzVhdmmPU8ESUyDwIyks5tUeehgaJpZM4Rl9T8
.

Hi, Is there ETA on when this will be merged? Thanks

Is there a test server and companion for this pull? Thanks

@Amerkashi We can deploy a test server later today.

Thanks.    We'll need to update the documentation when this goes out.
== Hal
Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
On Tue, Mar 6, 2018 at 9:59 AM, Evan W. Patton<EMAIL_ADDRESS>wrote:

@Amerkashi https://github.com/amerkashi We can deploy a test server
later today.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/1142#issuecomment-370808756,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAwLt-9kOxjRwzd0Azutfgf-WIh3Ot8hks5tbqQ6gaJpZM4Rl9T8
.

@halatmit This code won't work with an older Companion. This makes upgrading problematical.

Jeff:  Let's talk about how to handle upgrades.
== Hal
Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
On Tue, Mar 6, 2018 at 10:44 AM, Jeffrey Schiller<EMAIL_ADDRESS>wrote:

@halatmit https://github.com/halatmit This code won't work with an
older Companion. This makes upgrading problematical.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/1142#issuecomment-370824596,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAwLt0NRhCp0NlfyCpzGtEFCGNTlleviks5tbq7MgaJpZM4Rl9T8
.

@Amerkashi This code is now up on ai2-test.appinventor.mit.edu. Be aware, you will need a new Companion, available for download from the site.

@jisqyv did companion source code also had to be updated for this also?
Well check. Thanks

The companion needs to be compiled with the updated code, but there are no
companion specific changes needed.
-Jeff
On Tuesday, March 6, 2018, Amerkashi<EMAIL_ADDRESS>wrote:

@jisqyv https://github.com/jisqyv did companion source code also had to
be updated for this also?
Well check. Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/1142#issuecomment-370883220,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABhdOJwooLyecnc2lROwRa0RJIl1EIUfks5tbtd3gaJpZM4Rl9T8
.

--

Jeffrey I. Schiller
Information Systems and Technology – MIT App Inventor
Massachusetts Institute of Technology
77 Massachusetts Avenue  Room 32G-530
Cambridge, MA 02139-4307
617.910.0259 - Voice
<EMAIL_ADDRESS><EMAIL_ADDRESS>http://jis.qyv.name
http://appinventor.mit.edu

Hello, Please take a look at image below and take the simple aia code. I have zipped the .aia because I couldn't attach. Just unzip it. It's bad coding, but its just to simulate error.

Build and you'll get the error
Drag blocks inside of Screen1.initialize to  outside and build -> no compile error and builds okay!!!

After your investigation, can you provide info why this would cause compile error when
fooForEach.zip

@halatmit @Amerkashi @jisqyv I patched Kawa to address this bug. I put a PR in against Hal's source branch, see https://github.com/halatmit/appinventor-sources/pull/1

@Amerkashi Re. the runtime error, was that after you rolled back the change? That macro is defined in this PR so you should not be getting an unbound location error.

@ewpatton No, I was just providing more info for to show the issue for this update.
Thanks for your patch. Will get started immediately :o) and let you know findings

@ewpatton Just take the updated jar? All other code remains untouched?

@Amerkashi Yes. I was able to compile your test app with the updated Kawa jar.

We are testing with several apps and working great. Still testing, but LGTM.
@ewpatton  Outta curiosity, how did you find out about this and did you patch the library for custom validation handling or downloaded the modified lib?

@Amerkashi I got lucky with this one. The issue had been reported in Kawa back in 2013. I went back in the git history for Kawa and grabbed the commit that fixed the issue (looks like it was fixed in Kawa 1.15). I had to do a couple extra changes to reflect API changes during that time but it was relatively straightforward.

-2 I ran the tests for this change and there are a number of failures that happen due to the macro changes. Might I propose the following:

Return the macro names to foreach, forrange, and while.
Have the macros wrap the versions -with-break using call/cc, but with escapename always as break.
In a future change, once we are enough versions in the future, switch the codegen to use the macro version that takes the escapename for flexibility.

Note the proposal above should also address backward compatibility concerns with older companions because the macro signatures stay the same.

I agree with all of these.  The problems are due to me being overly cute.
== Hal
Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
On Thu, Mar 8, 2018 at 11:19 AM, Evan W. Patton<EMAIL_ADDRESS>wrote:

-2 I ran the tests for this change and there are a number of failures that
happen due to the macro changes. Might I propose the following:

Return the macro names to foreach, forrange, and while.
Have the macros wrap the versions -with-break using call/cc, but
with escapename always as break.
In a future change, once we are enough versions in the future,
switch the codegen to use the macro version that takes the escapename
for flexibility.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/1142#issuecomment-371537990,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAwLt_9dKjJEpgJ018VG1uUoT50JaGggks5tcVoggaJpZM4Rl9T8
.

Oh? What are the failure cases?

@Amerkashi I think in all cases it is due to the change in the macro name producing an error or failure because the test code assumes the old macro names. I do not think there are any behavioral problems but I haven't taken the time to update the names in the tests to verify that's the case.

@ewpatton it's good to know that it might only be test issue.
Can you push update to test server? I like to test a case, not sure if caused by library update, even though the stack trace points to kawa lib. A simple test is to:
Create a global var, use obfuscated text and plug in a text block. Build and install onto phone. Does it crash when opening on device?
/tmp/runtime2311458488692761988.scm:2292:5: unbound location while
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2416)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2476)
at android.app.ActivityThread.-wrap11(ActivityThread.java)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1344)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:148)
at android.app.ActivityThread.main(ActivityThread.java:5417)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)
Caused by: /tmp/runtime2311458488692761988.scm:2292:5: unbound location while
at gnu.mapping.Location.get(Location.java:67)
at com.google.youngandroid.runtime.textDeobfuscate(runtime2311458488692761988.scm:2292)
at com.google.youngandroid.runtime.apply2(runtime2311458488692761988.scm:2290)
at gnu.expr.ModuleBody.applyN(ModuleBody.java:230)
at com.google.youngandroid.runtime.applyN(runtime2311458488692761988.scm:2071)
at gnu.expr.ModuleMethod.applyN(ModuleMethod.java:216)
at gnu.kawa.functions.ApplyToArgs.applyN(ApplyToArgs.java:139)
at gnu.kawa.functions.Apply.applyN(Apply.java:70)
at

@Amerkashi That is a bug in this PR. @halatmit renamed while to while-with-break but the text-deobfuscate code, which relies on while was not updated. If we revert the change to the macro names per my earlier comments the issue should be resolved.

Ouch. If this code is already pushed and then macros are renamed, will it break functionality and cause issues? :(

FYI, This appears to be non-issue in companion. It only happens when installing on device

#1180 is an evolution of this PR, so I am closing this one. Check out #1180 for more information.


remote builds

closes: #85
Building all the use images in #69 takes much more disk space. The hacks to make that work broke the image merging job that had worked previously.
After exploring many options, this PR adopts Depot.dev to build remotely. Their concurrent multiplatform bake greatly simplifies the CI. It removes the need to self-host our ARM runner and the large persistent build cache greatly speeds up our builds (e.g. a JS change builds in minutes to resolve the image caches instead of over an hour to re-run all the chain operations.)
There are some other changes that should be evident from the commit messages.
Test plan

[x] Green with manually triggered run (https://github.com/Agoric/agoric-3-proposals/actions/runs/7763083187)
[ ] yarn test works locally and includes unpassed proposals

Please mention me again when the tests are passing.  Thanks!

Please mention me again when the tests are passing.  Thanks!

@michaelfig please provide your perspective on:

"build" command only for downstream repos
reliance on "depot" and how it's factored
documentation requirements

I don't expect any of those to change in the course of getting the tests back to green. (They were green when I first requested review)
Happy to sync if more context is needed.

Sorry for not reviewing this earlier, and I'm not even sure I would have caught it, but synthetic-chain and base image changes cannot be coupled as there may be existing consumers of a specific version of the synthetic-chain package that use the latest a3p images also produced by this repo. In this case agoric-sdk.


Mentor Registration Page

We need to add a Mentor registration page since we don't need the same information as we do for students. One option is to make the registration form more dynamic and only show the relevant fields based on who is registering.

This will probably help a bit


added shopping_app file

A shopping app UI with headphones and speakers.

Screenshot of the UI

Looks Good,
Thanks For Contribution.
Leave a Star & Share With Friends.


Skewt refactor

Refactors our skew-T lesson to:

Try to write better exercises that are not copy/paste, but not impossible
Change the solution paradigm back to load magics so we can teach in jupyter lab
Combine skew-T, hodograph, and layout back into a single notebook that we can get through as much as we want.

Potential things to change:

Maybe use a different/better case?
IA State gives a hodograph with lots of holes in it, but it's only for one exercise. Maybe that isn't the desired behavior?

NCEI is 503 causing the travis failures


Default grid to size of size of window and spacing between parameters

Description
Please default the grid spacing so that windows, inputs, output, lines, and reroute nodes align perfectly
As you can see here it's not quite aligned properly and the default grid size wasn't working either.

Edit:  think the offset is wrong too.
Implementation ideas
No response

Hi, this would be something to suggest upstream in the https://github.com/godotengine/godot repository, as this is all default widgets provided by the engine itself.


render_notifications weird behavior

render_notifications is rendering an empty div.  render_notification will generate a notification.
Both of them however, are giving me an error if I pass a renderer:
=render_notification notification, 'feed'
I get:
wrong number of arguments (given 2, expected 1)
Rails 5.2
notification-renderer 1.2.6
Thanks!

Thanks for your report. Can you provide an example to reproduce the Problem? This could be a public repository or a number of steps one can follow to get to the Problem. A stack trace for the error would also be helpful.

Closing because of inactivity.


filter based on detection date without setting the hour value

if I do not set the hour value just the date the reports will not be filtered based on that day

#2047 resolves this issue too


Doesn't work with spaces in path

lol idk why

Wait no it works wtf

Does it work on Windows? This sounds like an issue Windows would have

Not really sure, it works sometimes but sometimes it doesn't

I think we can confirm this fixed

When was it confirmed?

STOP. FUCKING. TIMING. ME. OUT.

 Just clearing a few old issues up


fatal errors are not logged on startup

When the configuration is invalid (see below) and arangod refuses to start, you don't see the error in the log /var/lib/arangodb3/arangod.log, but only when you start the daemon by hand, you can see the cause, e.g.:
content of 'ENGINE' file '/var/lib/arangodb3/ENGINE' and command-line/configuration option

I'm running ArangoDB 3.2.5 in a docker container (Docker version 17.05.0-ce, build 89658be) with Ubuntu 14.04 and /var/lib is a host mounted volume, this could be an issue with docker caching a write but I doubt it.
The error can re reproduced by supplying an invalid configuration, e.g. by writing a different storage engine into the '/var/lib/arangodb3/ENGINE' file.

when you use docker, you should be able to see docker logs <containerid> which is the stdout of the process - thats the way docker intends it.

@grexe did the above help you?

thanks @dothebart that helped, but it shows the real cause of the issue: ArangoDB logs this error only to STDOUT but not to the log file in /var/lib/arangodb3.

you can add commandline options to the arangod started arangodb docker container:
docker run -e ARANGO_RANDOM_ROOT_PASSWORD=1 -p 8529:8529 -d arangodb arangod --log.file /var/log/arangodb3/arangod.log

However, we currently have an issue with not enough logging showing up in the foreground, so docker logs won't tell you the full picture.


[Configuration] Delete VM

Support deleting a VM

Just in case you or someone is not aware, I believe that you can go into the files app->on my iPad -> UTM, and delete the vms from there. Maybe this is not the official way to do it, but it seems to work. I apologize if I’m wrong, but just trying to help.

does completely delete all files and storage associated with vm? im not clear on that and its not mentioned in docs


Fix for not working user login & creation

I had added:

missing body for user creation request
changed URI creation for login, because previous one didn't worked. Full url was setted as a path
this commit also include changes from this pull request #5

Fixed


#3610 broke dependencies

On #3610 you changed the package name from ms-vscode.csharp to ms-dotnettools.csharp. This is causing all the packages that depend on this to fail.
Is this change here to stay or are you changing your mind again soon?
In the mean time if you're having this issue, uninstall the one you're using and get this one: https://github.com/OmniSharp/omnisharp-vscode/releases/tag/v1.21.12

Thanks for the report. The extension name change is final, and we're very sorry for the issues caused with various upstream dependencies.


Only first fold is preserved when updating foldings.

When making a FoldingManager.UpdateFoldings call that has multiple NewFoldings with DefaultClosed set to true, only the first resulting folding's state is preserved. The following failing test reproduces this issue:
        [Fact]
        public void Test_Only_First_Fold_Preserved()
        {
            // Arrange.
            var doc = new TextDocument(@"
                This is some test text.
                It needs to span multiple
                lines in order to demonstrate
                folding.
            ");

            var manager = new FoldingManager(doc);

            // Act.
            manager.UpdateFoldings(new[]
            {
                new NewFolding(17, 57) { DefaultClosed = true },
                new NewFolding(101, 148) { DefaultClosed = true}
            }, -1);

            // Assert.
            Assert.True(manager.AllFoldings.All(f => f.IsFolded));
        }

I suspect this is related to this change: 21f8a38

Fixed in #110


How to compute the self-supervised acc?

Here it is: https://github.com/TengdaHan/DPC/blob/ac25b1b8fe86bb5e2c5547cd4bc33a8b8f2a5424/dpc/main.py#L217


Add TTL for sliding window stats_key

Issue https://github.com/cristipufu/aspnetcore-redis-rate-limiting/issues/197

@cristipufu please take a look.


[desktop application] Next/Previous image with keyboard

Hi,
I read the doc and I don't manage to find a way to move to the next ou previous image from a keyboard key on a desktop application. Did I miss something ? Is there any current way to do it ?
Thanks.

I used the keymap flutter library to impelment this, using this code:
KeyboardWidget(
bindings: [
KeyAction(
LogicalKeyboardKey.arrowRight,
'go to next photo',
() {
gotoNextPage();
},
),
KeyAction(
LogicalKeyboardKey.arrowLeft,
'go to previous photo',
() {
gotoPrevPage();
},
),
KeyAction(
LogicalKeyboardKey.backspace,
'go back',
() {
Navigator.of(context).pop();
},
),
],

I used the keymap flutter library to impelment this, using this code: KeyboardWidget( bindings: [ KeyAction( LogicalKeyboardKey.arrowRight, 'go to next photo', () { gotoNextPage(); }, ), KeyAction( LogicalKeyboardKey.arrowLeft, 'go to previous photo', () { gotoPrevPage(); }, ), KeyAction( LogicalKeyboardKey.backspace, 'go back', () { Navigator.of(context).pop(); }, ), ],

Format code please.


[Plugin design] Remove global NM.Client

Created libnmstate.nm.NetworkManagerPlugin to hold the NM.Client
and pass it to needed functions in libnmstate.nm folder:

The libnmstate.nm.NetworkManagerPlugin.refresh_content() is used
to refresh the content on verification retry.

Changed the checkpoint to use the MainContext of NM.Client for
refreshing the timeout. Because without Mainloop running after
checkpoint rollback or destroy, we cannot ask Mainloop to release
the reference of NM.Client used for refreshing timeout.

Created internal public function
libnmstate.nmstate.show_with_plugin() allowing apply() function to
query current status without creating new instance of
NetworkManagerPlugin.

Created internal public function
libnmstate.nmstate.plugin_context() allowing properly unload the
plugin.

The design principle is move NM dedicate code to NetworkManagerPlugin
and shareable between plugins code should be in libnmstate.
Hence, remove the checkpoint autodestroy argument and contextmanager
code as these should be handled by libnmstate top level.

Waiting PR #806

@ffmancera @tyll @EdDev Please review this giant PR which is the fist step to the plugin support.

Coverage remained the same at ?% when pulling a81c61d6444652a6ba0d37ea1a612c221e2e4ee1 on cathay4t:remove_global_nmclient into e4a7a0fd94a35e6b864d0936751acdd3e1bc9d6e on nmstate:master.

Coverage remained the same at ?% when pulling a81c61d6444652a6ba0d37ea1a612c221e2e4ee1 on cathay4t:remove_global_nmclient into e4a7a0fd94a35e6b864d0936751acdd3e1bc9d6e on nmstate:master.

I will do the rebase after the whole VLAN filtering stuff merged.

New patch uploaded with fixed to recommendations from @EdDev

I agree with the changes. I still approving this. Thank you for the hard work @cathay4t :-)


Update Character “muspella-stumpton-annabelle-lee”

Automatically generated by Netlify CMS

👷 Deploy Preview for dynamic-cheesecake-5fbd8b processing.

Name
Link

🔨 Latest commit
fc41c15ef902852c97d120c4d2cd41935e21ea27

🔍 Latest deploy log
https://app.netlify.com/sites/dynamic-cheesecake-5fbd8b/deploys/62ec00360e71a90009780e93


fix file permission error when overwriting directory containing .git …

…file with init command

Codecov Report
Patch coverage has no change and project coverage change: -0.06 :warning:

Comparison is base (30f483a) 78.85% compared to head (518466b) 78.80%.

: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
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2599      +/-   ##
==========================================
- Coverage   78.85%   78.80%   -0.06%
==========================================
  Files         628      631       +3
  Lines       51279    51618     +339
==========================================
+ Hits        40436    40677     +241
- Misses       8854     8930      +76
- Partials     1989     2011      +22

Flag
Coverage Δ

go-1.15-386
78.83% <ø> (+<0.01%)
:arrow_up:

go-1.15-amd64
?

go-1.16-386
?

go-1.16-amd64
78.84% <ø> (+0.01%)
:arrow_up:

go-1.17-386
78.81% <ø> (?)

go-1.18-386
78.77% <ø> (?)

go-1.18-amd64
78.76% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 5 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Is there a simple way to get a list of items by a set of ids?

** Which Category is your question related to? **
API
** What AWS Services are you utilizing? **
AppSync, Amplify
** Provide additional details e.g. code snippets **
Is there a simple way to write a resolver to get a list of items from a table by passing in a set of ids?
I'm currently solving this problem by building a filter expression string rather manually (this is a function for a pipeline resolver):
#set( $expValues = {} )
#set( $expression = "#id IN (" )
#set( $ids = $ctx.stash.get("ids"))

#set( $index = 0 )

#foreach( $id in $ids )
    #set( $index = $index + 1 )
    #if( $ids.size() == $index )
        #set( $expression = "${expression} :id${index})" )
    #else
        #set( $expression = "${expression} :id${index}, " )
    #end
    $util.qr( $expValues.put(":id${index}", { "S" : "${id}" }) )
#end

{
  "operation" : "Scan",
  "filter" : {
      "expression" : "${expression}",
      "expressionNames": {
        "#id" : "id"
      },
      "expressionValues" : $util.toJson($expValues)
  }
}

Is this the best way? For example, I noticed there was a utility, $util.transform.toDynamoDBFilterExpression and tried implementing it as suggested from @mikeparisstuff 's answer on StackOverflow: https://stackoverflow.com/questions/52046495/util-transform-todynamodbfilterexpression
But I kept getting errors in the response about $util and expecting 'null', 'true', or 'false'. So I had to resort to the above approach.
It would be nice to have a better way to build complex queries as DynamoDB-JSON is a bit painful to write. I was hoping to be able to use similar filters (with OR: [{ id: "123" }, {id: "124"}] -type constructs that AppSync/Amplify already provides at the API level, but at the resolver level.

To get a set of values based on id, you should use the BatchGetItem resolver operation as outlined here https://docs.aws.amazon.com/appsync/latest/devguide/tutorial-dynamodb-batch.html. This is currently a bit tough to use when using multiple environments and ideally would be encapsulated as a transformer such as @batchGet. We have plans to open up the CLI such that you can run custom transformers and that would be the best way to get this working in a reproducible manner.

@mikeparisstuff, have there been any recent developments into making this work with multiple environments?

+1, is there any way to do this more easily now?

The batchGet is actually the best shot at doing this, although you will be limited to 100 items (https://docs.aws.amazon.com/amazondynamodb/latest/APIReference/API_BatchGetItem.html). The sole problem at the moment, however, is that as far as I know, we are not able to inject any environment variables such as the table name - which is required for the BatchGetItem - into the custom resolver.
This is supposedly tracked in #1774.
Maybe somebody in the near future will find some time to dig deep into it and find a way to make this work.
Other than that, I think, you can pretty much only run the expensive scan with the filter expression.

Oh, #1543 ist also talking about this. I don't know, what the pricing implications are, but if you really need a batchGet you can of course always lean back to a lambda for the resolver. Lambdas can be added with amplify add function and detailed configuration which other resources access is required to.

any updates on this?


Split CI descriptor in smaller scripts

Change content and motivation
Split CI descriptor in smaller scripts. Apart from being cleaner this allows to shellcheck the CI process.
GitHub cross-links

Fixes:

Docs:

Gets messy with file referencing... closing for now...

Yes it might look a little bit messy, but still not very long and easy maintainable.


More deploy options

It is intended that this project offer other deploy options than only rsync - such as Github Pages and Amazon S3.

I've already implemented the deploy to Github Pages option and a I think that Amazon S3 is not required by now.


Bug on applying lambda expression

Suppose :
Vector (t : Type) (n : Nat) : Type
Vector.empty <t> : (Vector t Nat.zero)
Vector.cons <t> (n : Nat) (v : t) (pred : (Vector t n)) : (Vector t (Nat.succ n))

Vector.length (t : Type) (n : Nat) (vec : (Vector t n)) : Nat
Vector.length t Nat.zero (Vector.empty t) = Nat.zero
Vector.length t (Nat.succ k) (Vector.cons t k value pred) = (Nat.succ (Vector.length t k pred))

Vector.index_equal_length (t : Type) (n : Nat) (vec : (Vector t n)) : (Equal Nat (Vector.length t n vec) n)
Vector.index_equal_length t n (Vector.empty t) = (Refl Nat Nat.zero)
Vector.index_equal_length t (Nat.succ k) (Vector.cons t k p pred) =
  let rec = (Vector.index_equal_length t k pred)
  let func = (@e (Nat.succ e)) :: Nat -> Nat
  (Equal.apply Nat Nat (Vector.length t k pred) k (@e (Nat.succ e)) rec)````

It gives a panic error like this :
thread 'main' panicked at 'Invalid output: 1 (String.cons 27 (String.cons 91 ...)), src/main.rs:229:5
And if tried to use the let as the lambda the normalizer tells me that:
- Expected: (Equal Nat (Nat.succ (Vector.length t k pred)) (Nat.succ k))
- Detected: (Equal Nat (func (Vector.length t k pred)) (func k))

But I do not know if let expression suffer from reduction, so it could not be a bug actually.

This was a bug with NbE getting confused by the usage of different variables with the same name. We could either rename, or show an error. I think showing an error is better, since it makes it more clear, to the user, that there is no such a thing as equally named variables (as in Idris).
Can you update and let me know if the issue is solved?

Can you check this, I am getting a loop? :
Vector.test (t : Type) (n : Nat) (vec : (Vector t n)) : Nat
Vector.test t z g = z

Vector.test2 (t : Type) (n : Nat) (vec : (Vector t n)) : (Equal Nat (Vector.test t n vec) n)
Vector.test2 t z g = (Refl Nat z)

Am I doing something wrong?

Thanks, that loop was actually caused by a bug on HVM's Rust-mode interpreter. Reporting it helped the project significantly.


log4j fix for zipkin-elasticsearch7

Elasticsearch 7.10.1 is vulnerable to CVE-2021-44228

These are test images. Not sure if they represent and actual risk.


Settings for PX4

What are the setting for PX4? @cmeldas only mentioned for arducopter

I am sorry, there was mistake, by "PX4" i mean hardware Pixhawk 4.
I never use PX4 firmware, if you find settings for LightWare SF10,, use them, should work.

Hi,
I use  sensor HC-SR04 and send it to Pixhawk on Arduino mega serial success
but I try  pixhawk connect to missionplanner change LTE wireless transmission  although connect to successsa,missionplanne not display HC-SR04  value
what should I do?
Thanks & Regards.


Calendar button - add popup title

David  via Slack
It's a little thing, @RoamHacker. But could you add a tool tip to the awesomely useful calendar function you added to the toolbar?

Twitter @gottalead
feature added


Temporary submission prior to more rigorous error catching mec…

…hanism.
Work-in-progress. Awaiting response from DQ.

Remain in testing mode.


Fix threshold for PInvoke path in Buffer.Memmove

Threshold for PInvoke path in Buffer.Memmove was way too low. I suspect it was tuned based on the debug builds. I have updated it to be
at the lower end of the break-even point range, based on measurements on several different types of hardware. The best improvements are
on i7-3520M: 64-byte aligned memory blocks - 35% faster, 128 byte aligned memory blocks - 17% faster.
Also:

Delete misleading comment about PInvoke overhead
Add comment related to #430

@AlexGhiondea PTLA

Forgot to mention - the perf. difference between .NET Native for UWP and CoreCLR/RT is neglible.

LGTM!


rework containerd config merging

Issue #, if available:
Description of changes:
It was recently realized that containerd's native import merging strategy is not one that behaves as most users would like. This is because it replaces the entire body under keyed maps such as Plugins. This conflicts with the use case to override singular parameters under existing sections. (see xref).
This PR implements the merging in an unstructured way so that users can continue providing whatever inline containerd config fields they would like overridden.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Testing Done

See this guide for recommended testing for PRs. Some tests may not apply. Completing tests and providing additional validation steps are not required, but it is recommended and may reduce review time and time to merge.

/ci
+workflow:os_distros al2023

/ci
+workflow:os_distros al2023


coredump while resolve trace

I use gapid to start a trace on android device and capture a file named .perfetto;
I tried to resolve it in cmd :
out/android/trace_to_text text
But it seems to be error:
_[858.045] trace_to_text.cc:187    Skipping invalid packet
[858.045] utils.cc:112            PERFETTO_CHECK(preamble == 0x0a) (errno: 0, Success)
------------------ BEGINNING OF CRASH ------------------
Signal: Illegal instruction (possibly unaligned access)
Fault addr:<PHONE_NUMBER>521F11
Backtrace:
#00  0x00007F9B9328B390
#01  perfetto::trace_to_text::ForEachPacketBlobInTrace(std::__1::basic_istream<char, std::__1::char_traits >, std::__1::function<void (std::__1::unique_ptr<char [], std::__1::default_delete<char []> >, unsigned long)> const&)
../../tools/trace_to_text/utils.cc:112
#02  perfetto::trace_to_text::TraceToText(std::__1::basic_istream<char, std::_1::char_traits >, std::__1::basic_ostream<char, std::__1::char_traits >)
../../tools/trace_to_text/trace_to_text.cc:182
#03  perfetto::trace_to_text::(anonymous namespace)::Main(int, char**)
../../tools/trace_to_text/main.cc:178
#04  main
../../tools/trace_to_text/main.cc:196
#05  0x00007F9B92AC4830
#06  0x00000000004FA389
------------------ END OF CRASH ------------------
Illegal instruction (core dumped)
Then I try to parse it in web:
_https://ui.perfetto.dev/#!/viewer_
And it can‘t load the file successfully and stop here:
loading trace 93% - 246M/s
Maybe there are some errors in my trace file?
How can I check it?

Looks like the trace is malformed.
Is there any chance you can shared the trace (here or in some drive link)?
Does that trace load in Gapid (I assume no).
@pmuetschard FYI

Sorry, I can't upload the trace file and I try to use gapid to open it and failed:
Error resolving capture
Cause: Failed to convert proto from *capture.Record to capture.wrapper
Cause: Not a recognized capture format.
I think the file is broken, but I don't have idea how it happened.

Well in this case there is nothing I can do to help, other than tell you the trace file is malformed.


Move the opening of PDF file attachments into the DownloadManager-implementations

Note how the PDFAttachmentViewer handles PDF file attachments specially, by opening them in a new window/tab, rather than forcing them to be downloaded. This is done to improve the overall UX, since browsers in general are able to handle PDF files internally.
However, for file annotations we're currently not attempting to do the same thing and are instead just downloading them directly. In order to unify the behaviour, without having to duplicate a lot of code, the opening of PDF file attachments is thus moved into a new DownloadManager.openOrDownloadData method.

/botio unittest

/botio integrationtest

/botio-linux preview

Thank you for improving this!


Remove  wrapping requirement

This is awesome. I don't have a current use case for it, but I know that when I do I'm not going to want to wrap everything in a <tour> is there any way that requirement can be lifted? The thing is, I don't have a page so I can have a tour. Integrating the tour should be as lightweight as possible I think, and by requiring everything to be wrapped in a specific element it's imposing itself on what really matters (the actual page markup). Thoughts?

I'm facing the same problem. I had to wrap everything inside the <tour> element which causes maintainability issues.

@DaftMonk did anything ever come of this? I really like @kentcdodds's suggestion and very much dislike the idea of wrapping everything in .

You can use virtual steps to get around this


F64 RCP/RSQRT Eltwise test are failing

With: https://github.com/libxsmm/libxsmm/commit/051ac124629124a3c5d87f78351c996874c07ee9
the check-norm calculation changed which leads to major issues with reciprocals (with are only 28bit instead of 54bit for FP43). I have added the old norms:
diff --git a/samples/eltwise/eltwise_unary_simple.c b/samples/eltwise/eltwise_unary_simple.c
index 2ec3b7dfa..d24c76160 100644
--- a/samples/eltwise/eltwise_unary_simple.c
+++ b/samples/eltwise/eltwise_unary_simple.c
@@ -605,6 +605,7 @@ int test_unary_op( const libxsmm_blasint M, const libxsmm_blasint N, const libxs
   printf("L2 rel.error  : %.24f\n", norms_out.l2_rel);
   printf("Linf abs.error: %.24f\n", norms_out.linf_abs);
   printf("Linf rel.error: %.24f\n", norms_out.linf_rel);
+  printf("Check-norm (o): %.24f\n", norms_out.normf_rel);
   check_norm = libxsmm_matdiff_epsilon(&norms_out);
   printf("Check-norm    : %.24f\n\n", check_norm);

and here is one of the pathological cases:
Check-norm (o): 0.000026310733587602566858
Check-norm    : 0.003054283766758906018546

(i see even from time to time 0.1).
We cannot crank up the error bar to these levels. What is the math in the new norm which leads to so off the rails results?

Indeed, libxsmm_matdiff_epsilon is practically normf_rel, but contains/abstracts this norm instead of hard-coding it. We have cases with relative and absolute errors being small (and potentially below threshold/margin), but totally different distribution of respective noise. The normf_rel is a relative error-measure which is already not very obvious, and we discussed this (and then agreed on) years ago. Best case for normf_rel is linear algebra or particularly BLAS related work which is somewhat settled with the Froebenius (matrix-)norm. To "fix" aforementioned case about small relative and small absolute errors for input with small magnitude, I am essentially dividing normf_rel by the coefficient of determination or "R squared" (rsq). Imagine a two matrices A and B (can degenerate to vectors), with noise below margin, then rsq may disagree (0) for A and B being equal aka different distribution of below-threshold-values. The rsq is practically 1 for two matrices (... degenerated) with similar value-distribution. The rsq helps us to disagree on "gold" vs "test" if all-small values are always in different locations.

but was is the fix, it sounds to me we are using the wrong norm.

but was is the fix, it sounds to me we are using the wrong norm.

I mean dividing by non-zero rsq should be fine and just emphasize/increase a small normf_rel. It is not too easy to judge the case about "A and B are different" if A and B are matrices (consist of MxN values) based on a single number.

Beside of our sophisticated error measures (and actually using Kahan-compensated reductions), a lot of issues can stem from generating artificial input. If one normalizes the input values, they may still carry vastly different orders of magnitudes. The latter can screw up any reduction especially non-compensated reductions.

My point is the following: we were using Frobenius-Norm before and the inputs are the same as before. Now we change the norm and they are normally the same, but the new norm has spikes where the old one is smooth. Why is the new norm better than Frobenius?


chore(deps): Bump opentelemetry-instrumentation to 2.7.0

I'll need to sign CLA and check up on any other contributor guidelines, but after hitting a dependency conflict related to otel instrumentation I decided to take a stab at upgrading.
This will constitute a breaking change, so will need to be targeted appropriately, hence branching from 7.0.x
Passing tests locally, but I have not yet built and used it locally.

 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.

Ping @sdelamo
Are there wider thoughts on how to manage the otel instrumentation 2.x release yet?

I think it should go in Micronaut 5.


move under pytest-dev team?

(Original issue 5 created by bubenkoff on 2015-03-01T15:04:57.679285+00:00)
Please consider moving under shiny new pytest-dev team

(Original issue was assigned to t2y)
(Original issue 5 last updated on 2015-03-01T17:56:42.454568+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by t2y on 2015-03-01T16:09:46.295133+00:00)
Of course. That's good idea. How do I do?

(Original comment by bubenkoff on 2015-03-01T16:14:14.030957+00:00)
go to the settings of the repo and transfer it to pytest-dev team(i've added you there)

(Original comment by bubenkoff on 2015-03-01T16:15:15.612796+00:00)
go to the settings of the repo and transfer it to pytest-dev team(i've
added you there)

(Original comment by t2y on 2015-03-01T16:19:04.003265+00:00)
OK, done.

(Original comment by bubenkoff on 2015-03-01T16:20:27.642988+00:00)
pls update all the links to new location as bitbucket doesnt do any redirects from old locations like github does

(Original comment by bubenkoff on 2015-03-01T16:27:17.011606+00:00)
nice to have it automatically tested with drone.io as well (and have badges in the readme)

(Original comment by t2y on 2015-03-01T16:27:40.401407+00:00)
I see. I'll release what the link information would be updated. Thanks.

(Original comment by t2y on 2015-03-01T17:22:57.861566+00:00)
I've released new version 0.8.2 and now it automatically would be tested. Thank you for good opportunity.

https://pypi.python.org/pypi/pytest-quickcheck
https://drone.io/bitbucket.org/pytest-dev/pytest-quickcheck/latest

(Original comment by bubenkoff on 2015-03-01T17:56:42.449263+00:00)
great! thanks and welcome to pytest-dev team!


Cannot start the very first session (install request) on 23+ devices in non-auto sesion mode.

Cannot start the very first session (install request) on 23+ devices in non-auto sesion mode.
In registerAppInit
    if (intentState_ != INTENT_STATE.READY) {

request.addProcessWaitLock(ServerRequest.PROCESS_WAIT_LOCK.INTENT_PENDING_WAIT_LOCK);
}

In 23+ devices, due to the delayed intent handling code, the state is pending and this lock isn't released unless you are in the auto session mode.
The same works fine on pre 23 devices as the state is initialised to ready.
To reproduce:

No Branch.getAutoInstance() call
Branch branch = Branch.getInstance(SomeActivity.this); // Not the application context, hence no activity life cycle events
branch.initSession(...)

@deepak-aruba Sorry that you are facing this  issue. From Branch SDK v2.0 onwards we have removed the support for Non auto session. The delayed onNewIntent()  is handled in version after 2.0 and so it is supported in auto session management mode.
Is there ant reason for not using the auto session management?

@sojanpr getAutoInstance() or getInstance() with the application context are registering for the activity lifecycle events and thus posting the app open/close session tracking events and there doesn't seem to be a way to turn them off.
For some use cases like using BUO to create links from the app or initialising a session to get the referring params only on the first launch, full session tracking seemed unnecessary.
Please feel free to close the issue.

@getdeepak  Understand the use case but currently most of our partners want session tracking. That's why SDK doesn't provide a way for initialization at any point in application


Log spam when The One Probe is not installed.

McJtyLib spams the log on world/server join, when "The One Probe" is not installed.
Outtake of the log: https://gist.github.com/GirafiStudios/a6ab0dfa1f3c1c6f8639675104493999

That's not McJtyLib doing that. I don't output anything like that. Also notes that it talks about chiseling. Most likelty that's from the Chisel mod


works of xbox home but not games

is it only supposed to work on xbox home because i want to use it on black ops three and it won't let me?

If you can move around on the xbox dashboard, you should be-able to move it around inside games. I have not tested any backwards compatibility games but they should all work the same.
The application will essentially connect a virtual usb to the computer and state that this device is a xbox controller and then the Xbox App will pick it up and use it as a controller for games. So it should work if you can move around in the xbox dashboard using kb/m.

It's because he has his controller already plugged in or hit the guide button on the streaming app when he was using another controller.  IIRC, the controller that presses start at the beginning of a game is the input controller


Seed recipes not changing, when using mystical customization (1.21.1)

I am currently in the process of opening a minecraft server on 1.21.1 NeoForge, at version 21.1.61.
When attempting to update the crafting recipes of the seeds and the tiers, I have so far been unsuccesful. I am using Visual Studio Code to do this and can't see any issues with the code and have also tried to just copy the code I had used on my 1.20 server, which also isn't working.
Im unsure if this is some form of bug or possibly a mod conflict, any help would be appreciated.
Code being used to change the tier crafting item:
{     "mysticalagriculture:elemental": {       "essence": "mysticalagriculture:inferium_block"     } }
Code being used to change the individual seed:
{     "mysticalagriculture:coal": {       "ingredient": {         "item": "minecraft:coal_block"       }     } }
These are just the start but would be extended to cover a number of seeds and all tiers, once I've managed to get it fixed.
Versions (please complete the following information):

Minecraft: 1.21.1
NeoForge: 21.1.61
Cucumber: 8.0.6
Mystical Agriculture: 8.0.7
Mystical Customization: 6.0.0
Singleplayer or Multiplayer: Currently Single Player but will be changed to a Server once working.

I'm going to close this due to inactivity. If this is still an issue on the latest version you can reply here.


Phone format

When I have a phone number like<PHONE_NUMBER>, can we set this into the textfield to format<PHONE_NUMBER>? Also when type the number can this be format into a format mentioned?

up


Precise could not be downloaded(

Thanks for making this cool project.  In the logs of the stable version mycroft-voice:stable I see the logs say:
Precise could not be downloaded(ValueError('Invalid MD5 url: https://github.com/MycroftAI/mycroft-precise/releases/download/v0.2.0/precise-engine_0.2.0_armv7l.tar.gz.md5'))

Hi @vallard

Thanks for making this cool project
You're welcome :+1:

This error is not related to the Docker images itself, it's more related to Mycroft which is trying to download the md5sum file.
I tried to download the file manually and it worked, I compared the checksum between the the tar.gz and the the md5sum file and they are identical.
A weird thing is that it tries to download an older version of Precise Engine (0.2.0) which should be 0.3.0. I poked some guys on Mycroft chat, I'll keep you posted with update.

After some discussion, 0.3.0 is the default for aarch64 architecture.
Does your RPi still have access to Internet? If you try to run this command from your RPi, does it work?
$ wget https://github.com/MycroftAI/mycroft-precise/releases/download/v0.2.0/precise-engine_0.2.0_armv7l.tar.gz.md5

Yes, that works fine from the command line on the RPI.
On Mon, Feb 22, 2021 at 4:46 PM Gaëtan Trellu<EMAIL_ADDRESS>wrote:

After some discussion, 0.3.0 is the default for aarch64 architecture.
Does your RPi still have access to Internet? If you try to run this
command from your RPi, does it work?
$ wget https://github.com/MycroftAI/mycroft-precise/releases/download/v0.2.0/precise-engine_0.2.0_armv7l.tar.gz.md5
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/smartgic/docker-mycroft/issues/3#issuecomment-783785340,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJKTO2UV7JVVG7UBHSJZ3TAL3IDANCNFSM4X73GKVA
.

logging into the container I get the following:
$ docker exec -it dfb /bin/bash
$ curl -O https://github.com/MycroftAI/mycroft-precise/releases/download/v0.2.0/precise-engine_0.2.0_armv7l.tar.gz.md5
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (60) SSL certificate problem: unable to get local issuer certificate
More details here: https://curl.haxx.se/docs/sslcerts.html

curl failed to verify the legitimacy of the server and therefore could not
establish a secure connection to it. To learn more about this situation and
how to fix it, please visit the web page mentioned above.

Try the -k option with curl.

Yeah that works but still doesn’t solve the issue that the code is trying to download it and not able to.

On Feb 22, 2021, at 6:05 PM, Gaëtan Trellu<EMAIL_ADDRESS>wrote:

Try the -k option with curl.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Did you try the c_rehash as root within the container?

Hmmm. I’m not sure what that is.  I haven’t modified the container images at all. Was just trying to make it all run. Maybe there are other problems.  Thanks for your help though.

On Feb 22, 2021, at 6:37 PM, Gaëtan Trellu<EMAIL_ADDRESS>wrote:

Did you try the c_rehash as root within the container?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

You will have nothing to install, c_rehash is part of the openssl package already install within the image.
http://manpages.ubuntu.com/manpages/trusty/man1/c_rehash.1ssl.html

mycroft@mycroft_voice:~/core$ c_rehash
Skipping /usr/lib/ssl/certs, can't write
mycroft@mycroft_voice:~/core$ sudo c_rehash
bash: sudo: command not found

Is this the problem?
On Mon, Feb 22, 2021 at 6:44 PM Gaëtan Trellu<EMAIL_ADDRESS>wrote:

You will have nothing to install, c_rehash is part of the openssl package
already install within the image.
http://manpages.ubuntu.com/manpages/trusty/man1/c_rehash.1ssl.html
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/smartgic/docker-mycroft/issues/3#issuecomment-783838570,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJKTKVLBJG6SPNJTOCMN3TAMJA3ANCNFSM4X73GKVA
.

To connect as root inside a container, you don't need sudo.
$ sudo docker exec -ti -u 0 mycroft_voice bash
root@mycroft_voice:/home/mycroft/core# c_rehash

Output:
Doing /usr/lib/ssl/certs
WARNING: Skipping duplicate certificate ca-certificates.crt
WARNING: Skipping duplicate certificate ca-certificates.crt

Then restart the container.

I think that may have worked!  Thank you, seeing this now, but wake word is being detected.  Prob something else I have wrong on my end.
2021-02-23 05:33:40.873 | ERROR    |     1 | mycroft.client.speech.hotword_factory:update_precise:240 | Precise could not be downloaded(PermissionError(13, 'Permission denied'))
2021-02-23 05:33:43.998 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:33:44.733 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:33:45.368 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:33:45.872 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:34:14.002 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:34:14.736 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:34:15.381 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:34:15.877 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:34:41.368 | INFO     |     1 | mycroft.client.speech.hotword_factory:on_download:289 | Downloading Precise executable...
2021-02-23 05:34:41.373 | ERROR    |     1 | mycroft.client.speech.hotword_factory:update_precise:240 | Precise could not be downloaded(PermissionError(13, 'Permission denied'))

I think that may have worked!
Great, I'll add this to the Dockerfile.

2021-02-23 05:33:40.873 | ERROR    |     1 | mycroft.client.speech.hotword_factory:update_precise:240 | Precise could not be downloaded(PermissionError(13, 'Permission denied'))
2021-02-23 05:33:43.998 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:33:44.733 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:33:45.368 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:33:45.872 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:34:14.002 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:34:14.736 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:34:15.381 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:34:15.877 | INFO     |     1 | mycroft.client.speech.hotword_factory:during_download:301 | Still downloading executable...
2021-02-23 05:34:41.368 | INFO     |     1 | mycroft.client.speech.hotword_factory:on_download:289 | Downloading Precise executable...
2021-02-23 05:34:41.373 | ERROR    |     1 | mycroft.client.speech.hotword_factory:update_precise:240 | Precise could not be downloaded(PermissionError(13, 'Permission denied'))

I think you have created the mycroft-config as root which means that mycroft_voice which is running as mycroft user cannot right into it.
Adjust the permissions on this directory to make it writable within the container.

I pushed two commits, one for the c_rehash command and the other about the Permission denied.
https://github.com/smartgic/docker-mycroft/commit/3ee8fd1cb364e14625ae0399dcc68dcb9f624732
https://github.com/smartgic/docker-mycroft/commit/c0099cad44feda2435bd1d54dd9280b42c46b495

I just build new images with the fix for the stable tag.

@vallard does everything is fixed for you?

Yes, fixing the directory permissions helped like you said and the system registered.  Thank you so much for your help!

Yes, fixing the directory permissions helped like you said and the system registered. Thank you so much for your help!

Yeah!
Thanks for reporting the issue :+1:


Fix grouper closing in GroupByMergingQueryRunnerV2.

The grouperHolder should be closed on failure, not the grouper.

👍


Update Composer dependencies (2020-10-12-00-08)

Visual regression test passed!
View the visual regression test report


Catch every exception bubbled up by the CLI, to prevent showing stack traces

Is your feature request related to a specific problem?
Our users get really ugly stack traces from the CLI whenever there is an error we haven't caught.
Describe the solution you'd like
We should implement a "catch all" that handles any unexpected error, that then prints a generic An error has occurred message. We can then focus on gracefully handling all of the different errors our users encounter. Additionally, when dev_mode is enabled it should let the ugly errors and stack traces go through
Describe alternatives you've considered, if any
Not sure what the other options are here reasonably. There will always be places we throw errors where we didn't expect to, so trying to catch every one pre-emptively seems unrealistic

@sanders41 considering we use SystemExit in a lot of places to have pretty, clean error output, I'm thinking the easiest way to do this is basically catch everything that isn't a SystemExit?
Any other clever ideas here? Effectively we never want a user to see a stack trace form the CLI

I think you are right about everything except the SystemExit, but I don't think there is an except not SystemExit type option to catch everything except that (at least I haven't seen one). The only way I can think to be sure we catch everything is to put an except Exception at the entry point of the script.
On a broader scope, doing this makes it next to impossible to debug user issues or for them to debug themselves because you never know what the real error is. Really, I don't have a good solution to this though, its kind of a no win situation, users don't like to see stack traces, and they don't like it when we can't help debug because we don't have a stack trace. I also realize I'm not saying anything you don't already know with this :smile:

I agree, it's the equivalent of a 500 error from a server, basically letting the user know they're SOL and to file a bug report but like you said, we're between a rock and a hard place here, although I do come down on the side of hiding the stack traces/implementation details (hence the issue haha)
Right, so we can do an except Exception and then do nothing if the type is SystemExit or something then


Help Please :(

Hello! I have never done a but of modding in my life, but I want to make a modpack. I just can't seem to figure out how to link Solder to the TechnicPlatform. I am very confused and have no idea what is going on. Help is greatly appreciated. :D

First of all welcome to TechnicSolder.
The best way to start is working through the docs: http://docs.solder.io/
There you will find everything you need to know to get started.
Here you will find information on how to connect Solder to your TechnicPlatform account: http://docs.solder.io/v0.7/docs/linking-solder-to-the-technic-platform
But please be aware that solder is not your "double click .exe setup" and should be handled with caution! Excerpt from the solder docs getting started page:

Have fun and let us know if you need additional information that could not be found in the docs.

Thank you for the help! I guess I'll just have to wait until I understand this stuff more so I don't ruin my computer, but thank you again.

If you don't feel up to using solder, but still want to play around with making your own modpack, you can still do it using the zip method.
For that all you need to do is make a zip file that the contents will be placed in the base folder of the modpack. So normally it would look like this:
file.zip
bin
modpack.jar  (this file is just the forge version you want to use with the name changed to modpack)
config
any configs you would like to include/modify
mods
modA.jar
modB.jar
resourcepacks   (optional, you do not have to include folder you aren't going to use)
Unity-Complete-1.2.0.zip (example of resource pack)
options.txt  (these 2 are not in any folders, just in the base zip file)
server.dat
Here is a picture of my base zip folder to help make more sense:

The only folders you really actually need in here is the bin folder with forge inside of it, and your mods. anything that isn't in there will be generated to be default like the options, or all the mod configs. But if you want to edit them, then you need to include the edited configs etc..
After you have your zip file set up, you will need to host the file somewhere that the file download link doesn't change and is a direct download.
(I use dropbox, if you want to use dropbox too, let me know and I will post how I get my direct download links if you want to go that route, but there are other places)
If you can paste your link in the address bar, press enter, and its starts downloading the file right away, then that is the link you need.
once you have that set up and ready, you can now go to http://www.technicpack.net/modpack/create
and start setting it up! Choose a name, select the minecraft version you are using, select hidden pack if you probably haven't gotten any permissions to use the mods yet. You can change it later from hidden as long as you have all the permissions to have the mods in a public pack. But for now, it should be a private pack while you are testing, and getting your permissions page setup etc..
Then you need a small description of the pack, you can change it later so don't worry about it too much.
If you agree to the terms, then check the box and create your modpack page.
Once your modpack page is ready and made, you can click the edit modpack button, and under "Modpack Location" you want to paste the direct download link from earlier there.
You can also go to the "Versions" tab and put in a version number and have your changelog there.
Explore the pages, and fill out any other information you want.
But that is basically the easy way of getting a modpack setup without using solder.

Thank you so much! I got to the point where I can see my pack in the launcher, but whenever I try to install it, it says there was an error unzipping the file and that the file did not exist. I have the zip in my dropbox and linked as Archive.zip. Do I need to change the name of it for it to work?

Paste your drop box link

Ok, i got to the point where I can install it, thanks to the link, but there's something wrong with the files. When I hit play, it just pauses for a second then goes back to the launcher. Here's the link - https://www.dropbox.com/s/4qoq04y61vh2qd6/Archive.zip?dl=0
That's not the edited link, just the one dropbox gave me. The modpack has the modifed one in it.

its as simple as you didn't put code chicken core in your modpack..

I put codechickencore in the mods folder, but it still isn't working. Sorry for being a noob at this, im still really bad at modding. https://dl.dropboxusercontent.com/s/emtlcagc3oa5t4u/Archive 2.zip?dl=0

I ran the modpack you have packaged, and that is what pops up. Means it works, but it tells you that the minimu version of codechickencore is 1.0.4. Simple
here is a link to the mod on curse. http://minecraft.curseforge.com/mc-mods/222213-codechickencore/files
Grab the latest for 1.7.10.
I would also update NEI http://minecraft.curseforge.com/mc-mods/222211-notenoughitems/files

Thank you for all your help! Sorry I haven't responded sooner, I got slammed with schoolwork. I did what you said, but it still isn't working for some reason. I think it might be my launcher, but the topic has gone a ways away from Solder, so I'll go ahead and close this. Thank you for your help! :)


from_timecode doesn't treat rate properly

@darbyjohnston reports
I was debugging an issue in my code and noticed something interesting about RationalTime::from_timecode(), however I'm not an expert in timecode so I don't know if this is expected behavior?
If I compare the result of RationalTime::from_timecode("01:00:00:00", 24.0) and RationalTime::from_timecode("01:00:00:00", 23.976), they both have the same value, 86400.0, but with the respective 24.0 and 23.976 rates. I guess I was expecting the second value to be 86,313.6 (23.976 * 3600.0)?
It looks like this is caused by RationalTime::from_timecode() running std::ceil() on the rate before calculating the result.
@meshula analyzes
I notice test_opentime.py, and the cpp, lack tests for equality between rates. For example, a test comparing two identical timecodes at different rates, converted to seconds (by dividing out the rate), should return the same result.
nominal_fps is just supposed to be used to check if the integer frame count in the time code string is greater than the number of frames that can possibly be valid indicated in the string.
The conversion to frames at the end of the function should be using rate, not nominal_fps.
Fix for this issue should include unit tests at the cpp level, optionally reported in the py bindings.

When OpenTime implements time travel, I'm going back and explaining, emphatically, to the NTSC that their clever yet beknighted idea of wodging colorburst information between audio and the black and white in the transmission is going to screw up the computation of time, until the end of time. The last analog broadcast in North America was switched off in 2009, but we are still stuck with the analog hack for black and white/color compatibility!

The conversion to frames at the end of the function should be using rate, not nominal_fps.

Should a similar change also be made in RationalTime::to_timecode() so the values round trip? Also what about this test in test_opentime.py:
def test_timecode_23976_fps(self):
        # This should behave exactly like 24 fps
        timecode = "00:00:01:00"
        t = otio.opentime.RationalTime(value=24, rate=23.976)
        self.assertEqual(t, otio.opentime.from_timecode(timecode, 23.976))

Is this test irrelevant if 24 and 23.976 fps behave differently, or should the comment be removed and the test values updated to work?

When is one second not a second? When NTSC has had a go.
A more meaningful test to replace that one would be to generate rational times for "00:00:01:00" at 24 and 23.976, and then assert that that the difference between them is as expected due to framerate difference.

Related: support for NTSC rates in timecode conversion was improved in this PR: https://github.com/AcademySoftwareFoundation/OpenTimelineIO/pull/1477

I think it's addressed, thanks.


PBclust should not have a default number of clusters

This may be controversial, but I do not think PBclust should have a default number of clusters. While I understand that it is convenient for beginners to have a minimum of options to care about, I think the number of clusters should be chosen based on the user knowledge of the system and the problem. Having a default value for the number of clusters hides the importance of this choice.
In addition, a default value should make sense in most cases. Yet, in most cases, 5 is not a pertinent number of clusters.
Therefore, I think the clusters argument of PBclust should be mandatory, and should not have a default value.

I agree.

I agree with the PBxplore's gods.
Default is good.

As it had been tested ?


feat: force open retrieved file

This PR makes it so that when a file is retrieved using Term.retrieve(), the retrieved file is forcibly opened (:e!) after retrieval (see #231)
It does so by calling util.try_open_file as a callback, and edits the cmd in try_open_file to force the edit. This method is currently called only when:

Creating new components, where adding the ! would make no difference
Retrieving an Apex class using SF md list, in which case, if you have the Apex class opened, it makes sense to use !, since it works as calling retrieve on the opened file
The new feature, which is what we want

Lovely! You almost own the entire next release notes now :)! 💯


win: implements uv__udp_try_send

following win: implements uv__udp_try_send by @gemabarni implement the required fixe by @saghul

I think we can land the first three commits with @gemabarni as the author. Then, add the last commit with @supercairos as the author, as the commit really just changes two aborts to returning UV_EINVAL.
@saghul you had outstanding comments in #1210 that should be addressed now.

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/302/

CI again: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/303/

No related new fails ... so ready to merge?

Landed in https://github.com/libuv/libuv/commit/c33fe9874a4982f47644706ef72b8333e1650232 and  https://github.com/libuv/libuv/commit/500a1f96ebc2b26055518af6f30a12be036f1ab3. Thanks!


Install error

Greeting,
This is Fu. I got an error when I wanted to install this project using M1-max chip. I failed to install mini_racer, and I have check many approaches but it always failed. My configurations are:

gcc-12 (or clang arm64-apple-darwin22.2.0)
Bundler version 2.4.2
gem version 3.4.2

Error is:
...
/Users/yinfu/gems/gems/bundler-2.4.2/lib/bundler/worker.rb:90:in `block (2
levels) in create_threads'

An error occurred while installing mini_racer (0.6.3), and Bundler
cannot continue.

In Gemfile:
  mini_racer

Best,
Fu

mini_racer might not be compatible with Apple chips. Would suggest you to create an issue in the upstream repository: https://github.com/rubyjs/mini_racer
In the mean time you can try using docker to compile your site. Docker instructions.


4D to 3D image conversion?

Is there a way to "unpack" a 4D image into a series of 3D images within ANTsR?
I have managed to do it with the following code but it doesn't look the best way to do it:
> seg
antsImage
  Pixel Type          : float
  Components Per Pixel: 1
  Dimensions          : 196x237x194x5
  Voxel Spacing       : 0.699999988079071x0.699999988079071x0.699999988079071x1
  Origin              : 68.2 97.3 -65.7 0
  Direction           : -1 0 0 0 0 -1 0 0 0 0 1 0 0 0 0 1

mask <- as.antsImage(array( 1 , dim(seg)[1:3] ))
mat = timeseries2matrix(seg, mask)
newseg = matrixToImages(mat,mask)

> newseg[[1]]
antsImage
  Pixel Type          : float
  Components Per Pixel: 1
  Dimensions          : 196x237x194
  Voxel Spacing       : 1x1x1
  Origin              : 0 0 0
  Direction           : 1 0 0 0 1 0 0 0 1

you might want to write a function for this that uses :
antsSetDirection  antsSetOrigin     antsSetSpacing

to set the values as you wish.
so you get the header information correct.

I use this in my extrantsr package, where you would use ind = 1:3 in your case:
setMethod("subset_4d", "antsImage", function(img, ind, ...) {

  arr = .subset_4d(img, ind)
  newimg = as.antsImage(arr)
  get_ind = seq(length(dim(newimg)))

  origin = as.numeric(antsGetOrigin(img))
  origin = origin[get_ind]
  antsSetOrigin(newimg, origin)

  adir = antsGetDirection(img)
  adir = adir[get_ind, get_ind]
  antsSetDirection(newimg, adir)

  spacing = as.numeric(antsGetSpacing(img))
  spacing = spacing[get_ind]
  antsSetSpacing(newimg, spacing)

  # newimg = antsCopyImageInfo(img,
  #                            newimg,
  #                            ...)
  return(newimg)
})

#' @rdname subset_4d-methods
.subset_4d = function(img, ind) {
  dimg = dim(img)
  if (length(dimg) != 4) {
    stop("not a 4D image")
  }
  arr = as.array(img)[,,,ind]
  return(arr)
}

Hi @muschellij2
Your solution may be more memory efficient than the function I created:
antsImage4Dto3D = function(img) {
  # check input is good
  if ( class(img) != 'antsImage' ) stop('Input is not antsImage.')
  if ( length(dim(img)) != 4 ) stop(paste('Input image is not 4D.'))

  # put everything to a matrix
  mask <- as.antsImage(array( 1 , dim(img)[1:3] ))
  mat = timeseries2matrix(img, mask)

  # put matrix to a list
  newimg = matrixToImages(mat,mask)
  rm(mat)
  invisible(gc)

  # copy headers to new images
  direction = antsGetDirection(img)[-4,-4]
  spacing = antsGetSpacing(img)[-4]
  origin = antsGetOrigin(img)[-4]
  for (im in newimg) {
    antsSetDirection(im, direction)
    antsSetSpacing(im, spacing)
    antsSetOrigin(im, origin)
  }
  return(newimg)
}

I guess it needs compiling though, I cannot run it simply by sourcing your code:
source('/data/jag/dpustina/Code/HCP/subset_4d.R')
Creating a generic function from function subset_4d in the global environment
Error in rematchDefinition(definition, fdef, mnames, fnames, signature) :
  methods can add arguments to the generic subset_4d only if '...' is an argument to the generic

@stnava do you have thoughts what is good to go in ANTsR? My function uses an intermediate matrix which may require a lot of memory for large 4D image, but gives a list of images. John's method gives only a single image (if my I am understanding the code correctly).

Added the above function from #128 . Closing issue. If 'subset_4D' method is to be integrated this can be done again later.

Here's one where you can actually source it.
library(ANTsR)
library(pbapply)
subset_4d = function(img, ind, ...) {
  arr = .subset_4d(img, ind)
  newimg = as.antsImage(arr)
  get_ind = seq(length(dim(newimg)))

  origin = as.numeric(antsGetOrigin(img))
  origin = origin[get_ind]
  antsSetOrigin(newimg, origin)

  adir = antsGetDirection(img)
  adir = adir[get_ind, get_ind]
  antsSetDirection(newimg, adir)

  spacing = as.numeric(antsGetSpacing(img))
  spacing = spacing[get_ind]
  antsSetSpacing(newimg, spacing)
  return(newimg)
}

#' @rdname subset_4d-methods
.subset_4d = function(img, ind) {
  dimg = dim(img)
  if (length(dimg) != 4) {
    stop("not a 4D image")
  }
  arr = as.array(img)[,,,ind]
  return(arr)
}
#
# img = antsImag
# ts_list = pblapply(1:dim(img)[4], function(x) {
#   subset_4d(img, ind = x)
#   })

I am a bit confused, I thought we merged a function for 4D to 3D conversion before at https://github.com/stnava/ANTsR/pull/128/commits but it's not in the stnava/master
@stnava can you have a look if that function (or the solution of jmuschelli) can be added to provide 3D conversion. It might also be useful to have a function that loads directly loads 4D images as 3D list without conversion. Loading a 4D and converting to 3D requires double the memory and may fail with 200+ images easily.
Note also that I have consistently found a memory leak with antsImages in linux (i.e., our CFN cluster). The problem is alleviated only by adding invisible(gc()) to functions or for loops that do manipulations of images.

https://github.com/stnava/ANTsR/blob/master/man/mergeListToNDImage.Rd
This is a general solution.
regarding memory usage there is a good discussion here
http://adv-r.had.co.nz/memory.html
And Tools are also provided that might help characterize what you are
talking about.
On Sat, May 13, 2017 at 12:10 PM dorianps<EMAIL_ADDRESS>wrote:

Reopened #126 https://github.com/stnava/ANTsR/issues/126.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTsR/issues/126#event-1081031543, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AATyfkpCPf-sHZOsuUOGmE4OpHzOUxDqks5r5dXogaJpZM4LUkJ_
.

--
brian


add remote storage policy config for create table properties

Proposed changes
Issue Number: close #9669
Problem Summary:
add remote storage policy config for create table properties
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

Please add document for this


User Logout

in the logout feature when user click on the logout button after login as a admin in the top right corner he gets redirects to the login page but if we refresh it shows the dashboard page .
Bug Severity: Critical

fixed the bug 15 minutes


Cypress: Initial implementation of MCUBoot for 062 - BSP less

This PR is created to substitute existing one https://github.com/JuulLabs-OSS/mcuboot/pull/605.
This implementation is not dependent on BSP asset, as previous one. All the dependencies used as submodules are publicly available.
Features:

All the code is contained in boot/cypress folder and do not change any MCUBoot source code
Makefile based build support implemented
Dependencies are provided as submodules
image signature and hash validation implemented using software mbedtls
BlinkyApp - test application provided to quickly test MCUBootApp which is a bootloader.
Information regarding bootloader and test application build can be found in corresponding Readme.md files.

Please rebase on master.

@romanjoe This still adds a mbedtls submodule from cypress, can't we use the one we already have at ext/mbedtls?

@romanjoe This still adds a mbedtls submodule from cypress, can't we use the one we already have at ext/mbedtls?

Looks like it is the same instance, as we have - from base https://github.com/ARMmbed/mbedtls, so i suppose we can substitute. When we first started this work, there were some subset of mbedtls in upstream, that is why we added full repo to our folders.

fixed code per Fabio comments
rebased this branch against Fabios PR with MbedTLS version update
got rid of cypress instance of MbedTLS, using ext/mbedtls now
updated makefiles accordingly

I think this looks ok, as most of the change is within the Cypress directory. I'll be curious if the extra submodules end up being challenging for people on other targets.
I still think we should try to get Cypress support through the various RTOS's. Effectively, this is adding another RTOS, that just happens to be a lot thinner. But, this isn't really going to scale for adding more targets this way.

Could you please squash f0e95a02e7d8e50f65c41d382cb97ebb9bd82f2a, 0a4a53c9c9de9d119ae8f9ea38a4a5739919f507 and 8a44b8d275702e5e64e9e41bebd13bc072288525?


Renaming of a project doesn't work

I've just created new project locally. E.g. running:
 ./run gui watch --skip-wasm-opt

and
sbt runProjectManagerDistribution

and connecting to localhost 8080 via Chrome browser.
The newly created project got New_Project_22 name. As soon as it opened, I returned back to dashboard and stopped the project. Then I renamed it to SDG and opened it again. Everything is broken, nothing works:

Even the name of the project is still New_Project_22! The project is still in New_Project_22 directory on disk, its package.yaml contains new name:
name: SDG
namespace: local
version: 0.0.1
license: ''
authors: []
maintainers: []
edition: 0.0.0-dev
prefer-local-libraries: true

After closing my browser, the project folder on disk renamed to SDG. Connecting again to localhost 8080 and opening the project after writing this bug report seems to get things into normal mode of operations.

Renaming fails because the dashboard expects the engine version field in the response. Then the IDE is started with an incorrect (old) project name.

@JaroslavTulach i'm having issues running the old gui for some reason, so i'm not 100% sure it solves the issue, but a PR with a fix has been opened:

#7975

PR fixes the issue for me

Fixed by #7975


Refactor slicing code

Add dropdims overload for grids
Use MVectors for minvals and maxvals

Codecov Report

Merging #16 (5c67849) into master (06b9422) will decrease coverage by 5.22%.
The diff coverage is 44.44%.

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
- Coverage   61.20%   55.98%   -5.23%
==========================================
  Files          13       12       -1
  Lines         232      259      +27
==========================================
+ Hits          142      145       +3
- Misses         90      114      +24

Impacted Files
Coverage Δ

src/recipes.jl
0.00% <0.00%> (ø)

src/show.jl
0.00% <0.00%> (ø)

src/subset.jl
45.31% <20.00%> (-15.16%)
:arrow_down:

src/units.jl
48.48% <25.00%> (-2.95%)
:arrow_down:

src/grids.jl
58.82% <40.00%> (-26.90%)
:arrow_down:

src/abstractarray.jl
84.00% <90.00%> (ø)

src/algebra.jl
100.00% <100.00%> (ø)

src/utils.jl
33.33% <100.00%> (+6.06%)
:arrow_up:

src/vector.jl
89.58% <100.00%> (-0.42%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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


Follow up bugs after progressive wizard PR

Opening a separate issue to follow up after merging #1361 so we can start testing it end to end, since the main use case works.
Issues observed when testing the new wizard changes and comparing them to the mockups:

[x] Crashes when trying to edit existing plan (see https://github.com/konveyor/mig-ui/pull/1361#issuecomment-988232093)

[ ] General step

[ ] "All fields are required" text is missing
[x] Fields are out of order, source cluster should be before migration type
[x] Too much space between the fields?
[x] "Repository" field should be titled "Replication repository"

[ ] Namespaces step

[ ] In storage class conversion mode, there are editable row controls but nothing to edit
[ ] Non-PF icons/cursor in use for editable rows

[ ] PVs step

[ ] migration type column should be disabled until a PV is selected (when selection is a thing, state+full only?)
[x] Target storage class field is missing from here, should be here instead of on copy options step when in storage class conversion mode (need to check with @vconzola)
[x] PV migration type is still here when in storage class conversion mode, it was removed in the mockups for that mode

[ ] Copy options step

[ ] Target storage class field should not be editable if the row is not in edit mode? check mockups
[ ] Non-PF icons/cursor in use for editable rows

Additional implementation needed:

[ ] Plans table

[ ] Type column (Full, state only, storage class conversion) (WIP in #1366)
[ ] Rearrange table to fit everything? (compact variant? expandable w/ source + target under the fold?) (WIP in #1366)
[x] Remove/rename actions in the dropdown as appropriate for each migration type
[x] Disable the Rollback action when it is not applicable (need to discuss with @eriknelson)

https://github.com/konveyor/mig-ui/pull/1363 implements everything here except for:

Plans table

Type column (Full, state only, storage class conversion)
Rearrange table to fit everything? (compact variant? expandable w/ source + target under the fold?)
Disable the Rollback action when it is not applicable (need to discuss with @eriknelson)

Per conversation with @vconzola and @pranavgaikwad, we will actually want to:

[ ] add Stage and Cutover actions on all types of plans and not just full migrations.

[ ] For SCC/State migration plans, indirectImageMigration and indirectVolumeMigration must be set to false.
[ ] State migration can be performed even when one or more PVCs have move option selected
[ ] For state migrations, stage should be set to False whenever migrateState is set to True.

Addressing stage/cutover action changes above in https://github.com/konveyor/mig-ui/pull/1368 (WIP).

Remaining tweaks above will be addressed in https://github.com/konveyor/mig-ui/pull/1369


Add /go/linux-credentials/ redirect

relates to https://github.com/docker/docker.github.io/issues/15229
relates to https://github.com/docker/docker.github.io/pull/15231
relates to https://github.com/docker/docker.github.io/pull/15219

Docker Desktop for Linux currently has a link to;
https://docs.docker.com/desktop/linux/#credentials-management
which has moved, and didn't have a redirect, causing the link to break;
unfortunately we can't fix existing versions of Docker Desktop; redirects
are being added (but won't be able to redirect to the correct anchor).
This patch adds a https://docs.docker.com/go/linux-credentials/ redirect,
so that we can continue using that URL even if content moves around.

Thanks! Let me get this one in to unblock the PR in docker desktop itself


Support to use PEFT settings in BootStrapFineTune

I am trying to fine tune Mistral 7B model and want to use PEFT setting. but getting following error :
File ~/.conda/envs/env/lib/python3.10/site-packages/dspy/teleprompt/finetune.py:166, in BootstrapFinetune.compile(self, student, teacher, trainset, valset, target, bsize, accumsteps, lr, epochs, bf16, int8, peft, path_prefix)
164 compiler_config_ = dict(compiler_config)
165 compiler_config_["save"] = compiler_config["save"] + "." + name
--> 166 best_ckpt_path = finetune_hf(training_data_path, target, compiler_config_)
168 print(f"#> Best checkpoint path: {best_ckpt_path} for {name}")
169 finetune_models[name] = dsp.HFModel(model=target, checkpoint=best_ckpt_path)  # best_ckpt_path
File ~/.conda/envs/env/lib/python3.10/site-packages/dsp/modules/finetuning/finetune_hf.py:342, in finetune_hf(data_path, target, config)
340 if config['peft']:
341     model = AutoModelClass.from_pretrained(target, device_map='auto')
--> 342     task_type = TaskType.SEQ_2_SEQ_LM if encoder_decoder_model else TaskType.CAUSAL_LM
343     peft_config = LoraConfig(task_type=task_type, inference_mode=False, r=8, lora_alpha=32, lora_dropout=0.1)
344     model = get_peft_model(model, peft_config)
NameError: name 'TaskType' is not defined
Here is my code:
peft_settings = {
     'r' : 16,
    'target_modules' : ["q_proj", "k_proj", "v_proj", "o_proj",
                      "gate_proj", "up_proj", "down_proj",],
    'lora_alpha' : 16,
    'lora_dropout' : 0,
    'bias' : "none",
    'use_gradient_checkpointing' : True,
    'random_state' : 3407,
    'use_rslora' : False,
    'loftq_config' : None,
}

teleprompter = BootstrapFinetune()
cot_compiled = teleprompter.compile(CoT(),
                                    target='mistralai/Mistral-7B-Instruct-v0.3' ,
                                    trainset=sampled_example,
                                    peft = True, bsize=1,accumsteps=4,lr=1e-4, epochs=5,bf16=True)

I tried using both options peft=True and peft=peft_settings
Any way to solve this?

Hi @ShwetaliShimangaud , ah it's because the import was left commented out . LMK if uncommenting resolves the issue for you, and feel free to push a PR after testing!

Hi @arnavsinghvi11 why is it commented?

have the same issue and was asking why is it commented


[Question] Can we know how many user viewed a particular Instagram story ??

Can we track or know how many users viewed a story of a public account.

At the moment, I don't see any way to get this informerion.


HDDS-10512. Refactor initialization of ScmTopologyClient in OM

What changes were proposed in this pull request?

As part of a review comment on PR: #5391, it has been observed that since KeyManager is dependent on ScmTopologyClient, we should start the client before the KeyManager in OzoneManager.
Cleanup unused code.
KeyManager is already being stopped prior to the client.

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-10512
How was this patch tested?
Existing tests should cover the changes, Green Git CI: https://github.com/tanvipenumudy/ozone/actions/runs/8261507655/

@kerneltime could you please review the changes, thanks!


change of page size from 100 -> 10 in workbench causes error

When trying to page with rdf4j 2.1.2 ; if I change the page size from 100 -> 10
The following error is logged:
21-Nov-2016 20:39:07.145 INFO [http-nio-8080-exec-8] org.apache.coyote.http11.Ht
tp11Processor.service Error parsing HTTP request header
java.lang.IllegalArgumentException: Invalid character found in the request targ et. The valid characters are defined in RFC 7230 and RFC 3986
This was on Tomcat 8.5.8

I can not immediately reproduce this. Can you show me the exact SPARQL query you do when you experienced this? Also: which internet browser are you using?

I had to redact this some
I went to the query panel in the workbench then ran  :
select ?s ?p ?o where {?s ?p ?o}
I then navigated to a record :
http://localhost:8080/rdf4j-workbench/repositories/MyVirt9/explore?resource=<http%3A%2F%2Fontologies.example.com%2FETL%2Fentity%23104>
Then navigated again and got the error:
http://localhost:8080/rdf4j-workbench/repositories/MyVirt9/explore?resource=<http%3A%2F%2Fontologies.example.com%2FETL%2Fentity%23104>&limit_explore=10&know_total=23&query=select ?s ?p ?o where {?s ?p ?o}&ref=
I just read something about so , not sure if that is what it's about.
"Neither '{' nor '}' are permitted characters in a URI and that includes
the query string."

Although I still can not reproduce the exact error you're getting, I believe I have found the cause. If I produce a patched build of the workbench, could you test it locally and see if it fixes the problem?

Patched version of workbench: rdf4j-workbench-2.2p659.war.zip
Simply rename to rdf4j-workbench.war (you don't need to unzip first, the .zip extension was only added because github does not allow upload of .war files) and deploy. Since the fix involves clientside Javascript you may need to clean your browser cache as well.

Confirmed!!! This patch corrected the issue!

Awesome, thanks for confirming. I'll reopen because I haven't actually integrated this patch in codebase yet.


Removed react-native-router-flux and added custom navigation logic

The basic way this works now is that we have the navigation section of the redux store that's controlled with standard actions and a reducer. Right now, it only uses simple actions that push, pop, and reset the stack, but it has support for anything that NavigationExperimental can handle.
Overview of directory changes:

src/actions/navigation - For simple navigation actions that can be reused. It might make sense to move these into view actions in the future since while they seem reusable now, they'll become more view specific once we tighten up the app navigation fow
src/navigation

.../router.js - This is what uses the navigation state to render the whole app using NavigationExperimental. It also renders things like the navigation bar and transitions.
.../routes.js - These are plain objects representing each route in the app. It can also store properties of that route such as whether it has a title, back button, how it transitions, etc
-src/scenes - This used to be named src/routes, but I think it was a good idea to separate the definition of a route (which usually involves how you get there) from a scene (which is the actual contents of the route) and to make it more clear that this navigation isn't using a router with a path like we had before.
.../index.js - This provides a function mapping route keys to the scene that they use. This could've just been a field in src/navigation/routes.js, but I wanted to keep them separate to simplify dependencies because the actions and reducer for navigation both import routes.js

@hmhealey where are you with this changes? I would love to merge this tomorrow morning


Testing with Left data/class keep failing.

Hi, Dartz community,
I have another Issue when testing left data from either.
test('should return server failure when remote data is unsuccessful',
          () async {
        // arrange
        final ServerFailure sv = ServerFailure();
        when(mockNetworkInfo.isConnected).thenAnswer((_) async => true);
        when(mockRemoteDataSource.getConcreteNumberTrivia(any))
           .thenThrow(ServerException());
        // act
        final result = await repository.getConcreteNumberTrivia(tNumber);

        result.fold((l) {
          verify(mockRemoteDataSource.getConcreteNumberTrivia(tNumber));
          verifyZeroInteractions(mockLocalDataSource);
          expect(l, ServerFailure());
        }, (r) {
          fail("shouldn't returning right data");
        });

this is my repository code.
@override
  Future<Either<Failure, NumberTrivia>> getConcreteNumberTrivia(
      int? number) async {
    networkInfo.isConnected;
    try {
      final remoteTrivia =
      await remoteDataSource.getConcreteNumberTrivia(number);
      localDataSource.cacheNumberTrivia(remoteTrivia);
      return Right(remoteTrivia);
    } on ServerException {
      return Left(ServerFailure());
    }
  }

That test would be resulting in this error
Expected: <Instance of 'ServerFailure'>
Actual: <Instance of 'ServerFailure'>
I've made sure that ServerFailure has come from the same file and class.
Thanks.

Hi @yusufa445! Does ServerFailure() == ServerFailure() evaluate to true? That is, have you overridden operator == to implement structural equality? If not, the behaviour you describe is expected, since Dart defaults to reference equality. You might want to try expect(l is ServerFailure, true); or something similar instead?
Good luck!


🐛 [Bug]:添加 append-to-body后，点击弹窗内部弹窗也会立即消失

Vue Version
3
Link to minimal reproduction

Step to reproduce

What is expected
No response
What is actually happening
No response
What is your project name
1
Any additional comments (optional)
No response

Could you please offer a minimal reproduction?

试了上面的例子，不能复现 点击内部，弹窗会消失。
你可以再贴一下准确的截图。