Remove igorw/config-service-provider

This PR removes a dependency on the ConfigServiceProvider we use from https://github.com/igorw/ConfigServiceProvider. The service provider's main feature is its ability to load prefixed configuration from multiple formats given a configuration path to load. It also enabled self-referencing replacements using %replacement% syntax. We currently only support one format, YAML, and only one configuration file is ever loaded at a time. We have not had a reason to use replacements thus far.
We're removing this because the provider is barely incompatible with Silex 2.0 (some typehints need to be changed, basically) and PRs to the repository seem unmanaged.
/**
 * Replace reference to `ChainConfigDriver` with `null` when PR below is merged.
 * Symfony's YAML package deprecated allowing a path to be passed to `parse` method.
 * This was causing our test-suite to fail. When this is merged, we can upgrade and
 * all will be well again.
 *
 * @see https://github.com/igorw/ConfigServiceProvider/pull/46
 */

That PR was opened a year ago. I still plan to reach out to Igor to see if he's interested in at least merging Silex 2 support, but after thinking about it a bit more, it really doesn't make a whole lot of sense for us right now. We can keep it simpler by just loading the format we support and being done with it.
I'm sending this to opencfp:silex-2.x as one of the last remaining things before creating the PR from that branch to master.

[x] Remove dependency on ConfigServiceProvider
[x] Update a few Composer packages that have moved

Follows #403.
/cc @beryllium

Leave this unmerged for a bit to allow folks time to pull in 2f747fc (security update). Currently, 2f747fc will go in without a need for composer install or any other added work. Adding these on top requires more work on end-user and since this isn't really all that urgent, I think it's worth waiting a few more days to make lives easier!


A more usable Sense Output

Hi nodtvedt,
Great example ! Thank you for all the trouble.
In order to make this more useful in Sense interface, it would be cool to have the result in Sense displayed not as the regular text R is returning, but as a table with 5 columns:
Call                                       Intercept           mkt_tf           smb             hml
Im(formula = ....)                   -0.0013338      -0.8139       0.5896         0.612605
(for your example)

Thanks for the feedback.
Indeed, I agree it would be useful to provide a prettier formatting of the result set. It is not trivial to parse the text result from lm() and convert it to a table though, and my initial experiments leads me to believe I'd be better off if I write an R wrapper on the server that returns the result set as a JSON object rather than faffing about with string-parsing.
I'm unlikely to spend time on modifying this proof-of-concept, but I'd be happy to merge any contributed code you may have.

Tx for the fast answer.
Not yet such an advanced R develeoper… I would rather try in Qlik some things…
In case we do something relevant, I will let you know.
Tx again for making the sample and sharing.
Sempre fi,
Cotiso
From: nodtvedt<EMAIL_ADDRESS>Sent: Saturday, July 02, 2016 9:52 PM
To: nodtvedt/qlik-sense-r-regression-extension<EMAIL_ADDRESS>Cc: cotiso<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [nodtvedt/qlik-sense-r-regression-extension] A more usable Sense Output (#1)
Thanks for the feedback.
Indeed, I agree it would be useful to provide a prettier formatting of the result set. It is not trivial to parse the text result from lm() and convert it to a table though, and my initial experiments leads me to believe I'd be better off if I write an R wrapper on the server that returns the result set as a JSON object rather than faffing about with string-parsing.
I'm unlikely to spend time on modifying this proof-of-concept, but I'd be happy to merge any contributed code you may have.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/nodtvedt/qlik-sense-r-regression-extension/issues/1#issuecomment-230116557 , or mute the thread https://github.com/notifications/unsubscribe/AI4IIv5GYo_-7buNECz1AZHEBkmZcP_fks5qRrNPgaJpZM4JDTzp .


bug: faecup and facedown in ios send portrait event

Plugin version:
latest
Platform(s):
ios
Current behavior:
when listen to change horientation and phone is put horizontally on table from landscape mode, it should stay in landscape but instead the faceup event is fired .
And this is transformed to default event portrait
Expected behavior:
No event should be fired when faceup facedown or optionally
Steps to reproduce:
Add this code to one app, and then move the phone into landscape and put it horizontally on table. you will see portrait-primary in the logs
      ScreenOrientation.addListener('screenOrientationChange', async () => {
        console.log(await ScreenOrientation.getCurrentOrientation())
      })

Related code:
The issue come from this line https://github.com/robingenz/capacitor-screen-orientation/blob/a5d08ea371ca14523a56949d7cf3fcdc4d3829e7/ios/Plugin/ScreenOrientation.swift#L89
Other information:
here is the Apple doc

I can propose a fix tomorow

Hi @riderx,
Thank you for your interest in this plugin. I will have a look at it after work.

I have created a fix.
You can give it a try:
<EMAIL_ADDRESS>
I would be happy to receive feedback.

@robingenz super efficient ! i will check !

@robingenz it fix the issue !! perfect !


Use github3?

Pro: Simplify GitHub API calls, or at least make them easier to read.
Con: Introduce third-party dependency.
github3 doc is at https://github3py.readthedocs.io/en/latest/

I would really rather not use github3

Why? License?

there's no clear license, it's not being maintained, and github is getting ready to switch to GraphQL

OK


Update caniuse-lite

Problem

This appears in the build logs:
 WARN  Browserslist: caniuse-lite is outdated. Please run:                                                                               10:29:51
  npx browserslist@latest --update-db
  Why you should do it regularly: https://github.com/browserslist/browserslist#browsers-data-updating

Description

Run the command from the warning and open a PR with the updated files.
Implementation

[ ] 🙋 I would be interested in implementing this feature.

Hey, I just went over the error and the npm command
npx browserslist@latest --update-db

is the perfect solution for this, can I take up the issue?

Sure, @shreyanshmalvya! I'm assigning this issue to you.


Add checked command to set the checked property instead of the  checked attribute (see #33)

Fixes #33

Thanks.


Register memory managers if passed to runtime

Relates to:

Risks
Low. This change introduces a new feature to register memory managers, which should not affect existing functionality.
Background
What does this PR do?
This PR adds functionality to register memory managers if they are passed to the runtime. It iterates through the managers array in the opts object and calls registerMemoryManager for each manager.
What kind of change is this?
Features (non-breaking change which adds functionality)
Documentation changes needed?
My changes do not require a change to the project documentation.
Testing
Where should a reviewer start?
The reviewer should start by examining the AgentRuntime class in the packages/core/src/runtime.ts file, focusing on the new code block that registers memory managers.
Detailed testing steps

Verify that the AgentRuntime class correctly initializes with the new code block.
Test passing an array of memory managers to the runtime options and ensure they are properly registered.
Check that existing functionality remains unaffected by this change.
Verify that the registerMemoryManager method is called for each manager in the array.

Deploy Notes
No special deploy notes are required for this change.
Sources

Good idea


Recommend Extensions only offering ESLint

VSCode Version: 0.10.12-alpha
OS Version: Windows 10

Steps to Reproduce:

F1 -> ext recommend
Shows only ESLint as recommended extensions, without having any extensions installed

If you install ESLint there are no recommendations given even though it is the only installed extension.

@egamma fyi

@vsccarl I think recommendations are based on type of files you were working with. If you didn't work with any projects yet, then there is nothing VS Code can recommend to you.

What @RReverser said above is right.
At the moment the recommendations are based on the type of files you have worked with.


Get blog name without being authenticated

Hi,
Trying to figure out how to get the blogs name and description without having to authenticate via the .settings method. I can get them by doing a separate http request to /wp-json but I was hoping I didn't have to do another request.
Appreciate any help you can provide.
Thanks

Getting them via the wp-json endpoint is the correct approach. The easiest way to do this is to use the .root() method on a WPAPI instance.
import WPAPI from 'wpapi';
const wp = new WPAPI( {
    endpoint: 'https://mysite.com/wp-json',
} );

wp.root().then( response => {
    console.log( `site name: ${ response.name }` );
    console.log( `site description: ${ response.description }` );
} );

Amazing! Thanks @kadamwhite 👍


I can not read itf barcode

Hi guys!
I really liked this framework, we did some internal testing here and it is very rich in terms of settings. We tested various types of barcode and it is serving part of my business.
I would like a help because only one type of barcode is not working and unfortunately it is the default type of payment for accounts here in Brazil, it is the ITF type (i2of5_reader) I could not read even this type of barcode.

![itf-2](https://user-images.githubusercontent.com/8906461/37414630-5ab9640c-2788-11e8-8e96-3853db4754bf.png
I'm testing with a web demo application and the settings are below:

				<button type="button" class="close" data-dismiss="modal" aria-label="Close">
					×
				</button>

Barcode Scanner ITF2

					 Use camera app
					<input type="file" accept="image/*;capture=camera" capture="camera" class="hidden" />

				<button type="button" class="btn btn-primary" data-dismiss="modal">Close</button>

Thank you guys!

Fala @edubritochaves
Conseguiu resolver, ninguém respondeu você aqui.
Comecei a trabalhar nessa ideia e gostaria de saber se você conseguiu algo.

Olá Erik, não consegui e o framework estava  muito instável no mobile, com
isso, partimos para uma solução nativa para IOS e outra para Androide
fizemos um bridge para o javascript acessar as libs.
Att.,
Em qua, 5 de dez de 2018 às 02:55, Erik Figueiredo<EMAIL_ADDRESS>escreveu:

Fala @edubritochaves https://github.com/edubritochaves
Conseguiu resolver, ninguém respondeu você aqui.
Comecei a trabalhar nessa ideia e gostaria de saber se você conseguiu algo.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/serratus/quaggaJS/issues/271#issuecomment-444358689,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIfm3avSdHXS6egyCTnRs4gH8ZjQPUpjks5u11G5gaJpZM4SquOA
.

--
Atenciosamente,
Eduardo Chaves

edited original post to have proper formatting for code

Obrigado pelo feedback

Any news here? :sad:

Is this another type of code, or is it just something that the existing i2of5 reader should be able to handle, but isn't?

Is this another type of code, or is it just something that the existing i2of5 reader should be able to handle, but isn't?

It is something that the existing reader should be able to do. I'm feeding it any type of i2of5  code and it just can't handle them. I would be glad to help anyone who is willing to fix it. Don't no much about barcodoes, tough.

well there are i2of5 barcodes that work in /test/fixtures/i2of5 . . . . i also just discovered that there is a separate 2of5 reader .. i wonder if that's the right one to use with this.

Hey @ericblade, thanks for the feedback. I have also tried the 2of5 reader and it didn't work :/
Here's an example:

@rodrigo-picanco That specific image works fine with the test app using the following settings

https://serratus.github.io/quaggaJS/examples/file_input.html

The original post image also works fine with the following settings


Modify ReadyValidIO noenq to set the data payload to DontCare.

Related issue: None

Type of change: other enhancement

Impact: no functional change (probably)

Development Phase: implementation
Release Notes
Modified ReadyValidIO (also used in DecoupledIO) to set the data payload (.bits) to don't care when a no enqueue action (noenq) is called. This will clean up the frequently used idiom:
c.noenq
when( cond) {
  c.enq( data)
}

This currently needs to be written as:
c.noenq
c.bits := DontCare
when( cond) {
  c.enq( data)
}

It is possible that noenq was used like this in some existing code:
c.bits := 47.U
c.noenq
when( cond) {
  c.valid := true.B
}

If this was the case, then the PR will change the behavior of the circuit.
The comment for noenq is:
    /** Indicate no enqueue occurs. Valid is set to false, and bits are
      * connected to an uninitialized wire
      */

so it wasn't supposed to be used this way.

This is the classic trade-off of "intent" vs. "ease of use".
Without the explicit DontCare connection, FIRRTL will issue a warning if bits isn't connected. There could be complicated logic with nested whens where one of the conditions leaves bits unassigned and this will be correctly flagged as an error by FIRRTL.
With the DontCare connection, this error (a missing connection to bits) will no longer be diagnosed by FIRRTL.

I think that connecting a DontCare for a default noenq actually does more explicitly align with programmer intent.
Also, the problem with not flagging missing connections is universal for all "default assignment" patterns; a default enq at the top would cause the same problem


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_23urj8xddv/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_23urj8xddv/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_23urj8xddv/clone/recipe/meta.yaml:47: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_23urj8xddv/clone/recipe/meta.yaml:47: missing_dev_url: The recipe is missing a dev_url
Errors were found


Can I use glaredb with %%sql magic (jupyter notebook)? or SQLAlchemy?

Description
I want to sql some delta tables in jupyter notebook without spark session.
So, I found glaredb.
So, can I use glaredb with %%sql magic?

Just wanted to clarify, are you specifically looking to be able to use %%sql in a notebook, or are you just looking for a way to glaredb in a notebook?
We have python bindings which we've tested in a variety of notebooks. You're able to install using:
!pip install glaredb

And then in another python cell, you can start using glaredb:
import glaredb

con = glaredb.connect()
con.sql("select 'hello'").show()

You can then use CREATE EXTERNAL TABLE (https://docs.glaredb.com/glaredb/sql-commands/create-external-table/#delta) or delta_scan (https://docs.glaredb.com/glaredb/sq-functions/delta_scan/) to read delta tables.
If you're looking specifically for running sql using %%sql, we would need look into that a bit more.

Thank you for your reply! I want to use %%sql in a notebook. for example:
%%sql glaredb://xxx
select 'hello'

or
%%sql
SELECT * FROM 'https://huggingface.co/datasets/fka/awesome-chatgpt-prompts/raw/main/prompts.csv'

Since https://github.com/catherinedevlin/ipython-sql support %%sql feature, but only in sqlalchemy
So, maybe we need a sqlalchemy dialects for glaredb.


test-azure-win2019-x64-1: jdk_rmi java/rmi/Naming/legalRegistryNames/LegalRegistryNames.java test failure

test-azure-win2019-x64-1: jdk_rmi java/rmi/Naming/legalRegistryNames/LegalRegistryNames.java fails on  test-azure-win2019-x64-1.  It does not fail on  test-aws-win2019-x64-1
It fails on openj9 and hotspot, jdk8  jdk11 and jdk16.
Failing job link: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/80/console
To rerun just the failing test
 https://ci.adoptopenjdk.net/job/Grinder/parambuild/?JDK_VERSION=8&JDK_IMPL=hotspot&JDK_VENDOR=adoptopenjdk&BUILD_LIST=openjdk&PLATFORM=x86-64_windows&CUSTOM_TARGET=jdk/test/java/rmi/Naming/legalRegistryNames/LegalRegistryNames.java&TARGET=jdk_custom_1

Several subtests fail with a java.net.MalformedURLException such as:
17:48:20  testing form: rmi://test-win2019dc-/MyName
17:48:20  java.net.MalformedURLException: invalid authority: rmi://test-win2019dc-/MyName
17:48:20  	at java.rmi.Naming.intParseURL(Naming.java:330)
17:48:20  	at java.rmi.Naming.parseURL(Naming.java:237)
17:48:20  	at java.rmi.Naming.rebind(Naming.java:171)
17:48:20  	at LegalRegistryNames.main(LegalRegistryNames.java:89)
17:48:20  	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
17:48:20  	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
17:48:20  	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
17:48:20  	at java.lang.reflect.Method.invoke(Method.java:498)
17:48:20  	at com.sun.javatest.regtest.agent.MainWrapper$MainThread.run(MainWrapper.java:127)
17:48:20  	at java.lang.Thread.run(Thread.java:748)
17:48:20  	unexpected lookup or unbind exception for form: rmi://test-win2019dc-/MyNameinvalid authority: rmi://test-win2019dc-/MyName

According to https://en.wikipedia.org/wiki/Hostname, hostnames cannot end with a hyphen. On that basis, the test is failing correctly.
The hostname value used in the test is assigned as follows:
localHostName = InetAddress.getLocalHost().getHostName();

So I assume there is something in the configuration of test-azure-win2019-x64-1 which is incorrect - computer name or a registry entry?

Having a look now,
Following the steps listed here, I've changed the hostname to remove the hyphen to test-win2019dc, the machine is restarting now for those changes to take effect

Machine is back up now, and the name has changed as it should. Rerunning grinder: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/91/console

Grinder ran green! Closing issue. Thanks @lumpfish for the comprehensive instructions! :grin:

Do we know how / why the hostname was incorrect? Is it the way the machine was given to us, or did we configure it (manually or via ansible)?  If it was given to us like that, is there something we could put into ansible to check it's not invalid?

AFAIK (and what I could briefly search), we don't set the host-name as part of the playbook (at least, for Windows). I think it's either given to us, or set when the machine is provisioned - however, we could fairly easily add a check in Ansible - It'd just be a case of getting the output of hostname from cmd.exe , and checking it doesn't end in -.

checking it doesn't end in -

"Checking that it's correct" would be better - if we had a notion of what "correct" is.

The machine, as defined in Azure, is called test-win2019dc-1 which suggests that something has truncated it. Our build machines have a similar hostname convention but prefixed bld rather than test which is one fewer characters. There has historically been a limit on the NetBIOS name of Windows systems, so my guess would be that the azure deployments truncate to 15 characters. test-win2016dc-1 has the same problem (The final 1 is not present). Based on @lumpfish's research it would appear that Azure are deploying systems with invalid hostnames in this case.
@Willsparker I'd suggest test-win2019dc1 would be better to allow for a convention with more than one machine, and I'd also suggest making the same change on the equivalent 2016 machine.

Okay, I've renamed test-win2019dc to test-win2019-1, and TEST-WIN2016DC- to TEST-WIN2016-1. Restarted both machines (after checking that nothing was running on them), and they both display the correct names when running hostname in cmd.

"Checking that it's correct" would be better - if we had a notion of what "correct" is.

Well, we could go off the Wikipedia article! From reading it, I gathered the following rules:

Maximum of 253 ASCII characters, 63 per label, a label being each section split by a dot (.)
Each label can only have a-z,A-Z,0-9 and a hyphen in it, but can't end in a hyphen
A label probably shouldn't start with a digit, but definitely shouldn't start with a hyphen

The rules are probably better defined in RFC 1178, but that's a lot of reading.

I think it's fair to say that most of those scenarios are pretty unlikely. If we really want to do a check I'd be ok with just the - thing since that's the only one that's likely to bite us (Famous last words ... I shouldn't have put this in writing ...)

 I'll look into creating an Ansible task :-)


Fix grammar in the README and scripts

This fixes the most prevalent of the issues, but there were a few others that I may have missed.

Thank you.


Implement Invariant EKF Floating Base Estimator

This PR,

Implements InvariantEKFBaseEstimator (the Contact Aided Invariant EKF algorithm) by inheriting from the FloatingBaseEstimator class.  Reference to the original paper is Contact-Aided Invariant Extended Kalman Filtering for Robot State Estimation. This implementation does not contain a full EKF SLAM algorithm using also visual factors. Instead, it's limited to estimating the foot positions of a bipedal robot only along with the base pose and IMU biases. For the original (and full) version, please see https://github.com/RossHartley/invariant-ekf.
adds an InvariantEKFBaseEstimatorTest that tests the estimator algorithm for a simple static configuration of iCubGenova04 robot.

Additionally, this PR also

adds https://github.com/artivis/manif as a dependency
modifies the library structure so as to introduce a clean separation between regular Estimators algorithms and FloatingBaseEstimators algorithm
modifies the FloatingBaseEstimator class slightly to add some required features with respect to the model computations.

Pending from the PR,

[x] Documentation wherever required
[x] implementation of resetEstimator() methods

Fails because CI was not updated to add manif as a dependency.
To avoid any merge conflicts, maybe we can wait for this to be merged.
https://github.com/dic-iit/bipedal-locomotion-framework/pull/82

To avoid any merge conflicts, maybe we can wait for this to be merged.

I added manif to CI in a way that merge conflicts will be avoided.

cc @S-Dafarra @GiulioRomualdi, this PR is ready to be reviewed.
I apologize beforehand for a lengthy PR.

CI is failing due to YCM version. See https://github.com/robotology/whole-body-estimators/pull/83.

Comparison plots for a walking experiment from a Gazebo simulated dataset between a Matlab implementation, bipedal locomotion framework implementation and original implementation against the ground truth is below,

Can you please try to rebase? In this way, the modification relative to the adding of manif should disappear.

Can you please try to rebase? In this way, the modification relative to the adding of manif should disappear.

Done!!
Please note that, this PR also contains a few changes in the parent class FloatingBaseEstimator that were projected upon due to the requirements in InvariantEKFBaseEstimator.  I also reorganized the headers to be separated from Estimators folder into a FloatingBaseEstimators folder.

The macOS CI is failing in the InvariantEKF Test, looks like the estimation results are bad (surprising). This needs to be investigated on a macOS machine.
Instead of the windows, CI is failing due to linking the problem of YarpUtilitiesTest with yarp sig Vector  in the Build step.

On Windows is actually failing because of
2020-09-03T15:58:08.8415226Z D:\a\bipedal-locomotion-framework\bipedal-locomotion-framework\src\Estimators\tests\InvariantEKFBaseEstimatorTest.cpp(29,17): error C2065: 'M_PI': undeclared identifier

Here a related issue: https://github.com/robotology/QA/issues/81
This patch should solve your problem
diff --git a/cmake/AddBipedalLocomotionUnitTest.cmake b/cmake/AddBipedalLocomotionUnitTest.cmake
index 82430fd..4449f25 100644
--- a/cmake/AddBipedalLocomotionUnitTest.cmake
+++ b/cmake/AddBipedalLocomotionUnitTest.cmake
@@ -69,6 +69,7 @@ function(add_bipedal_test)
       target_link_libraries(${targetname} PRIVATE CatchTestMain ${${prefix}_LINKS})
       target_compile_definitions(${targetname} PRIVATE CATCH_CONFIG_FAST_COMPILE CATCH_CONFIG_DISABLE_MATCHERS)
       target_compile_features(${targetname} PUBLIC cxx_std_17)
+      target_compile_definitions(${targetname} PRIVATE -D_USE_MATH_DEFINES)

       add_test(NAME ${targetname} COMMAND ${targetname})

On my laptop, the test (in debug) is still failing. Normalizing the simImuQuat solves the problem.
diff --git a/src/Estimators/tests/InvariantEKFBaseEstimatorTest.cpp b/src/Estimators/tests/InvariantEKFBaseEstimatorTest.cpp
index 9b51ff0..c86684d 100644
--- a/src/Estimators/tests/InvariantEKFBaseEstimatorTest.cpp
+++ b/src/Estimators/tests/InvariantEKFBaseEstimatorTest.cpp
@@ -135,6 +135,7 @@ TEST_CASE("Invariant EKF Base Estimator")
     // ground truth
     Eigen::Vector3d simIMUPos;
     Eigen::Quaterniond simImuQuat = Eigen::Quaterniond(0.3218, -0.6304, -0.6292, 0.3212);
+    simImuQuat.normalize();
     simIMUPos << 0.0296, -0.1439,  0.4915;

     // IMU measures
@@ -222,4 +223,3 @@ TEST_CASE("Invariant EKF Base Estimator")

     REQUIRE( std::abs(newOut.stateStdDev.imuPosition(0) - resetStateStdDev.imuPosition(0)) < 2e-2);
 }
-

On my laptop, the test (in debug) is still failing. Normalizing the simImuQuat solves the problem.

Yes, yet again I had failed to run the tests in Debug. I will keep this in mind. I will make relevant changes. Let's see if that fixes the macOS tests as well.

This patch should solve your problem

I shall open a separate PR for that.

Ciao @kouroshD !! May I ask you a favor of testing this PR by running the InvariantEKFBaseEstimatorTest after compiling on Debug build on your computer?

Ciao @kouroshD !! May I ask you a favor of testing this PR by running the InvariantEKFBaseEstimatorTest after compiling on Debug build on your computer?
You might have to install the dependency "manif" on your computer for testing this PR.

Ciao @kouroshD .. do not worry about this. We managed to procure a Mac workstation in the lab. I shall do the necessary testing myself :)

STrangely, running the InvariantEKFBaseEstimatorTest on macOS Catalina by compiling the repository in Debug, the test passes successfully. But with Release, it fails.
I am installing valgrind according to https://stackoverflow.com/questions/58360093/how-to-install-valgrind-on-macos-catalina-10-15-with-homebrew  by tapping into LouisBrunner/valgrind, since I was not able to directly install it from brew, which threw an error that it is not supported or function as expected in Catalina.

Looks like using auto on a manif to Eigen conversion was causing uninitialized value condition. This was caught by valgrind on macOS Catalina.
I changed this in https://github.com/dic-iit/bipedal-locomotion-framework/pull/85/commits/3020229aa2508e03de7c960e789bf53bed6b4e93 and the tests seem to pass both on Debug and Release.

Looks like using auto on a manif to Eigen conversion was causing uninitialized value condition. This was caught by valgrind on macOS Catalina local workstation.
I changed this in 3020229 and the tests seem to pass both on Debug and Release on the local workstation.

Take home message, never use auto with Eigen types https://eigen.tuxfamily.org/dox/TopicPitfalls.html :grin:

Looks like using auto on a manif to Eigen conversion was causing uninitialized value condition. This was caught by valgrind on macOS Catalina local workstation.
I changed this in 3020229 and the tests seem to pass both on Debug and Release on the local workstation.

Take home message, never use auto with Eigen types https://eigen.tuxfamily.org/dox/TopicPitfalls.html

Changed auto on Eigen types to relevant types in https://github.com/dic-iit/bipedal-locomotion-framework/pull/85/commits/b5deac5a01d8952050adae2e968e71ddb24de463

cc @GiulioRomualdi do you think we can merge this?

@GiulioRomualdi  I've squashed the commits. You could merge it once the CI passes, thank you! :)

Merging


Streaming evaluator returns exit code 0 even if evaluation fails

Steps to reproduce

echo '{ host-a = { ... }: {}; }' > eval-fail.nix
colmena build --evaluator streaming -f $(realpath ./eval-fail.nix)

Expected Behavior
Colmena should return a nonzero exit code to indicate an error, as it does with non-streaming evaluation.
Actual Behavior
Colmena outputs an error:
[INFO ] Using configuration: /home/wanja/src/colmena/eval-fail.nix
[INFO ] Enumerating nodes...
[INFO ] Selected all 1 nodes.
[WARN ] Streaming evaluation is an experimental feature
       ✅ 2s All done!
host-a ✅ 2s Evaluated host-a
host-a ❌ 0s Evaluation failed: No message
[ERROR] Failed to evaluate host-a - Last 7 lines of logs:
[ERROR]  created)
[ERROR]    state) Running
[ERROR]   stderr) error:
[ERROR]   stderr)        Failed assertions:
[ERROR]   stderr)        - The ‘fileSystems’ option does not specify your root file system.
[ERROR]   stderr)        - You must set the option ‘boot.loader.grub.devices’ or 'boot.loader.grub.mirroredBoots' to make the system bootable.
[ERROR]    state) Failed

but returns with exit code 0, indicating success.
Additional Information
This happens on current main (872949504befb9b5540e69f846c6965def7150f9) and stable (4fca657c4e6fca3edb1e04ae645855ec833852ee), the issue is present in both.

Update: I debugged this a little and found out that, in the case from my repro, the Vec of futures in this line is empty. This makes sense, because the error branch that's being taken never pushes to it.
The comment there seems to indicate that some failing future ought to be pushed here, but I'm not sure what that would look like.


New version: SomePythonThings.WingetUIStore version 2.1.1

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

same error as always I suppose

Automatic Validation ended with:

Installation failed with exit code 1

@wingetbot waivers Add Validation-Installation-Error


add zig wrapper

unlike #31, this is mostly done already.
zig doesn't need a wrapper to use the library, but it's a lot more comfortable to use when it takes advantage of zig's type system and uses the same conventions as anything else would.

committed the initial state as of 39d7977. it's enough to make a program with.


erratic behavior if one misses a footnote number in the text

I had 7 it turned it correctly into 6 but moved it to the top

fix in 1.0.1


adding use of valueAccessor to get data

Hello,
I'm using your crossCompare.js lib with a set very complicated dashboards. I'm using multivalues and I am using valueAccessor to get values calculated from other values calculated with custom reduce functions. In order to use the compare functionality provided by CrossCompare i changed the following lines in the crosscompare.cache function:
var i = cache.length;
while (i--) {
	// Set chosen value if chart's data is a crossfilter group with multiple
	// values, otherwise take the only value available
	if (value != 'default')
		cache[i].value = cache[i].value[value];
	else
	{
		cache[i].value = chart.valueAccessor()(cache[i]);
	}

Thanks for sharing your library. I wish this modification would be useful for someone.

Thank you for your addition! I hope the lib is useful to you. Best, Raoul


Image coordinates disappear during enlarging

Duplicate of https://github.com/1j01/jspaint/issues/67


e2e tests to handle dockerhub rate limiting

A bunch of fixes for the e2e tests so that we can avoid getting failures due to dockerhub rate limiting

avoid the use of latest as this as a pull policy of Always.  Use a specific tag for tests that use amazon/aws-cli
remove private-pvc test.  It was using the tag latest somewhere.  Rather than fix it, I opted to remove it because it doesn't work when running in the public cloud.
keep a list of external images that we should push to the kind nodes to avoid going out to dockerhub
updates to push-to-kind.sh so that it accepts the external image list

It's working as expected.


Boolean and the if command

Traditionally Unix shells have no booleans. Instead of taking a boolean, if takes a command (actually a pipeline) and executes the body depending on whether the command has succeeded or not.
Perhaps we could introduce booleans and see how it fares better.
Related to #13.

Boolean type
There is now a bool type and builtin variables $true and $false. All values can be converted to a boolean (internally; there are no builtins for that yet). $false and failure values of type exitus are considered false. All other values are considered true.
if
if takes values instead of closures and is implemented as a builtin function (instead of special form). The builtin = nows puts the result as a bool on the output; hence to test whether $x is xxx one now writes:
if (= $x xxx) {
    ...
}

else if and else remains to be implemented.


when capslock activated, make colored keyboard (illuminated model) and very clear visual indicator on screen

Description of the new feature / enhancement
Hello,
I often miss the capslock state when a write text with my eyes pointing at my keyboard.
So i would like to find an easy way to know this information.
my keyboard can get lights on every key (Roccat Vulcain)
So is it possible to change light color ou light on light off when i switch caps lock ?
Scenario when this would be used?
On every text write
Supporting information
On steam, there is the app "Wallpaper Engine" that can manage light color on that type of keyboard with i beleive generic option.
This may help to not add support to every model in the market.

/dup https://github.com/microsoft/PowerToys/issues/981


[bug] Embedded notes doesn't get exported

This error happens when the embedded notes are not on the same folders as the main document.
Obsidian markdown document:

Exported .odt document:

Thanks for reporting this!
I'll try to fix this by the end of the month (super busy right now lol)

when the embedded notes are not in the same folder as the main document.

Ahhh, that makes sense. I use a giant flat folder for my zettelkasten hence why I never spotted this

Fixed! If you have any issues please let me know.
Apologies it took so long too, I've been very busy as of late

Thank you for your update! Unfortunately, block embeds like ![[verbeekMoralizingTechnologyUnderstanding2011#^f1a73f]] are still not working

Yes, sorry about that! That is still broken and will be fixed separately in #23.


Reopen of #15 (Plot container + Title Bar)

Reopening #15  but on correct branch.
Also improved method of button/slider interaction to be more general so function names can be passed in as a prop.

Title bar with link to home page if you click on the Imp-Vis icon.
A plot container with basic buttons and sliders. This will require an update when other people's sliders and buttons are done, as they can then be replaced with those.
A basic Plotly demonstration component.

Can parse in a list of buttons and sliders to the plot, along with the functions they should call in the plot, and these will be automatically displayed.
Will also need to be updated to include different plot types, when these are developed.
You may need to do "npm install plotly.js --save" to get plotly to work
The title bar component, where the logo in the top-right, redirects to the home page.

The plot container component with the basic demonstration plotly plot and the default buttons and sliders for demonstration.

Hello Robert,
I love the design and functionality of your components. There is nothing major for me to comment but just a few designs that may need to be discussed with the team.

Should clicking the ImpVis icon open a new tab or load on the same tab?
Perhaps its good to demonstrate some axes/graph title labelling on plotly too

One little general comment, it's probably best in future to submit this stuff as multiple pull requests for each feature (a.k.a separate pull request for the title bar and plot container) such that it is easier to review. The way you would probably do this is by having multiple "feature" branches on your forked repo and then submitting PR's for each as you complete features

One little general comment, it's probably best in future to submit this stuff as multiple pull requests for each feature (a.k.a separate pull request for the title bar and plot container) such that it is easier to review. The way you would probably do this is by having multiple "feature" branches on your forked repo and then submitting PR's for each as you complete features

I think there is a bit of miscommunication here -  Robert's request has been there for quite some time now and he is still waiting for Dan's slider and Mark's button. Dan's slider PR is now gone and he has not made a new PR; Mark has not responded to PR comment. To have multiple pull requests, Rob has to have multiple branches. I definitely agree that PRs for separate components do make reviewing easier but I don't see having multiple branches on our forks being beneficial. Plus, this will cause the pull requests to pile up and possibly causing more complicated/sophisticated components that depend on the simple component impossible to make. I think we should have one person having one/two/three similar components in PR and have the team to go through reviewing so that we can push for progress.


Add contact info

Need an email address for people to contact (possibly on a dedicated contact page).

make sure to discuss in the Sunday meeting

maybe use<EMAIL_ADDRESS>for now? or sign up for an outlook account?

I just set up a new email at<EMAIL_ADDRESS>going forward we'll want the contact info to be easy to remember
However, we probably also want an organizational phone number on the site (a lot of grant committees wanted one; I've just been giving them mine), which we should discuss at the board meeting


Fake Client should generate name for resources

The Fake Clientset does not behave as expected when the GenerateName field is defined in a resource.   On creation, the resource does not get a Name with a unique suffix.
When 2 or more resources are created by the Fake ClientSet in testing, an api error occurs because of a Name collision, e.g. "Pod 'default/' already exists" (where Name is an empty string).
This has become a problem for us in testing a custom controller that creates 2 pods sequentially with the GenerateName field defined.
cc @screeley44 @jeffvance

the fake clientset doesn't attempt to duplicate server-side behavior like validation, name generation, uid assignment, etc. if you want to test things like that, you can add reactors to mock that behavior.

/close

If anyone is trying to figure out how to add such reactors, I wrote a detailed explanation here: https://stackoverflow.com/questions/68794562/kubernetes-fake-client-doesnt-handle-generatename-in-objectmeta/68794563#68794563


Add an option to allow disabling ampersand escaping

We've found when launching files with a "&" in their filename this escaping breaks the launch.  (Windows throws a could not find file error).  This PR just adds an option to allow disabling the escaping 👍

Can't we just handle this automatically, so it only escapes on URLs? I prefer no options whenever possible. Every option added is a lot of wasted time for consumer needing to understand the implications and whether they need it.

@sindresorhus I looked and asked around and I can't figure out for sure if it's just URLs that need the "&" escaped.  Are you OK with a basic "if it's an http:// or https:// string, escape, else, don't" style logic?

What about skipping the escaping and wrapping it in double quotes instead?

@MarshallOfSound Can you provide a filename to test?
I create a file name a&b.docx, And I can open it using open('G:\\test-cmd-start\\a&b.docx')

@MarshallOfSound @sindresorhus I had test these targets:

target
escape mode
open result

G:&b.docx
no escape
FAIL

G:&b.docx
escape by ^
SUCCESS

G:\a&b.docx
no escape
FAIL

G:\a&b.docx
escape by ^
SUCCESS

G:\a&b c.docx
no escape
SUCCESS

G:\a&b c.docx
escape by ^
FAIL

G:\a&b c.docx
using " " wraping space only
FAIL

G:\a&b c.docx
using " " wraping target
FAIL

https://bing.com/search?q=a&b^c
no escape
FAIL (open wrong url)

https://bing.com/search?q=a&b^c
escape by ^
SUCCESS  (open url q=a&b^c)

https://bing.com/search?q=a&b^c
using " " wraping target
FAIL

https://bing.com/search?q=a b^c
no escape
SUCCESS (open url q=a%20b^c)

https://bing.com/search?q=a b^c
escape by ^
FAIL (open wrong url)

https://bing.com/search?q=a b^c
using " " wraping target
FAIL

file:///G:/a&b.docx
no escape
FAIL

file:///G:/a&b.docx
escape by ^
SUCCESS

file:///G:/a&b.docx
using " " wraping target
FAIL

file:///G:/a b.docx
no escape
SUCCESS

file:///G:/a b.docx
escape by ^
FAIL

file:///G:/a b.docx
using " " wraping target
FAIL

Finally, I got 3 conclusions:

If contains space, DO NOT escape;
otherwise, DO escape.
No matter target is URL, or file path.

Closing this in favor of #98


[Feature Request] Support for Specifying Homelink Device to Trigger

Summary
The API currently allows you to trigger homelink. However, it is possible to have multiple homelink devices attached to a single vehicle. There is no way to list or select which device to trigger from. It will, therefore, only trigger the first homelink device that was configured. This has been a long-time missing feature and disparity between API and vehicle.
Feature Request

Allow retrieving a list of homelink devices configured for a given vehicle; and
Add a field to the trigger_homelink endpoint wherein a homelink_device_id can be supplied.

Closing as out of scope. Unfortunately I don't have the bandwidth for feature requests that would require vehicle firmware changes.


[BUG] - Error compiling node v1.31.0 on Raspberry pi4B 8Gb

Area
Building the node
Summary
I am getting an error during the build for teh trace-dispatcher when building node 1.31.0 for my raspberry pi relay nodes.
[ 9 of 14] Compiling Cardano.Logging.Tracer.Standard ( src/Cardano/Logging/Tracer/Standard.hs, /home/ubuntu/src/cardano-node/dist-newstyle/build/aarch64-linux/ghc-8.10.7/trace-dispatcher-1.29.0/build/Cardano/Logging/Tracer/Standard.o, /home/ubuntu/src/cardano-node/dist-newstyle/build/aarch64-linux/ghc-8.10.7/trace-dispatcher-1.29.0/build/Cardano/Logging/Tracer/Standard.dyn_o )
src/Cardano/Logging/Tracer/Standard.hs:9:1: error: [-Wdeprecations, -Werror=deprecations]
Module ‘Control.Concurrent.Chan.Unagi.Bounded’:
This library is unlikely to perform well on architectures without a fetch-and-add instruction
|
9 | import           Control.Concurrent.Chan.Unagi.Bounded
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
src/Cardano/Logging/Tracer/Standard.hs:73:26: error: [-Wdeprecations, -Werror=deprecations]
In the use of ‘newChan’
(imported from Control.Concurrent.Chan.Unagi.Bounded):
"This library is unlikely to perform well on architectures without a fetch-and-add instruction"
|
73 |     (inChan, outChan) <- newChan 2048
|                          ^^^^^^^
cabal: Failed to build trace-dispatcher-1.29.0 (which is required by
test:trace-dispatcher-test from trace-dispatcher-1.29.0 and
exe:trace-dispatcher-examples from trace-dispatcher-1.29.0).
Steps to reproduce

Check out the 1.31.1 tag
Configure build as normal
run cabal build all  (or simply cabal build trace-dispatcher)

Expected behavior
This should compile and produce binaries.
System info (please complete the following information):

OS Name: Ubuntu on Raspberry pi 4B 8Gb
OS Version 20.04
Node version 1.31.0

Additional context
I understand that raspberry pi4 architecture is not officially supported, but I also understand like myself there is a large number of SPO's running on raspberry pi4, so if this is indeed a breaking change that requires hardware upgrades this needs to be effectively communicated to anyone who is currently running pi4.

For some reason unagi-chan has
  if !arch(i386) && !arch(x86_64)
    cpp-options: -DNOT_x86

in the cabal file, and then various code snippets like the following:
#ifdef NOT_x86
   {-# WARNING "This library is unlikely to perform well on architectures without a fetch-and-add instruction" #-}
#endif

I can't find actual code doing fetch-and-add instructions directly in unagi-chan, and have to assume this is due to the atomic-primop package, which also doesn't seem to really restrict itself to x86 only either. It has some C code which it claims is only necessary for some fairly old GHC versions, yet keeps including that code for windows.
The whole primop logic (and yes we have fences and cmpxch instructions) feels fairly odd. It appears as if this should be safe to just strip unagi-chan of the warnings and the odd cabal logic.

Updated the ghc-options for the trace-dispatcher to -Wwarn and it compiled, I'm running the new node now on testnet for both relay and core, will see if there's any negative performance

I'm fairly certain this is a bug in unagi-chan.
See https://github.com/jberryman/unagi-chan/issues/42

I'm not sure why this is closed? It does appear that its a bug in the unagi-chan library but there was no PR opened for them in relation to running on pi


Type annotated binders

This is passing all tests locally. Thank you so much @yiming-fang and @leoqiao18 for helping me port this!
Since the changes are quite large, I'm just going to go ahead and format all the files we touched. In particular, I think the typechecker and IR.IR need reformatting since I never did those post-Brittany.

Oh right, @yiming-fang  is still working on getting lambda lifting working with recursive functions. Since that's orthogonal to this PR, maybe you could merge that in on a separate PR?

Oh right, @yiming-fang is still working on getting lambda lifting working with recursive functions (which should also close #111). Since that's orthogonal to this PR, maybe you could merge that in on a separate PR?

Sure, I can open a separate PR for lambda lifting mutually recursive lets.


Conditionally load minified jquery vs regular when in DEBUG mode

Internally, Django minifies jQuery when DEBUG==False (https://github.com/django/django/blob/335c9c94acf263901fb023404408880245b0c4b4/django/contrib/admin/options.py#L629-L641). When two items in the media list are the same, they get merged and included only once. This is what correctly happens with the jQuery included by filer when DEBUG==True.
However, since the current filer code doesn't add .min to jquery when DEBUG==False, there will be two instances of jquery in the media list (jquery.js and jquery.min.js), which means they won't be recognized as the same and jquery will be included twice. This can lead to issues when using packages that install jquery plugins, as the plugins will only be attached to one of the jquery instances and you'll end up with an error of the form:  Uncaught TypeError: $(...).somePlugin is not a function

Codecov Report

Merging #1185 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1185      +/-   ##
==========================================
+ Coverage   71.30%   71.32%   +0.01%
==========================================
  Files          63       63
  Lines        3147     3149       +2
  Branches      441      441
==========================================
+ Hits         2244     2246       +2
  Misses        749      749
  Partials      154      154

Impacted Files
Coverage Δ

filer/fields/file.py
43.33% <100.00%> (+1.28%)
:arrow_up:

Continue to review full report at Codecov.

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

Seems like a new ticket got added for this (https://github.com/divio/django-filer/issues/1188) which lead to the issue being solved in https://github.com/divio/django-filer/pull/1189.

Duplicate of #1189


Switch out CJS module.exports for ES export default

I'm in the unfortunate position of having to support IE11 which doesn't like the arrow function or use of const in the for loop.
This means I'm having to get babel to transpile the package.  I'm using @babel/preset-env with useBuiltIns: "usage", so it inserts import statements into the package.
Mixing these imports with module.exports causes things to break.  I'm working around this by adding sourceType: "unambiguous" to my babel config so that it detects semver-diff as being commonjs and injects require statements instead which will play nicely with module.exports.
Seeing as sourceType: "unambiguous" isn't 100% accurate however, it would be preferable if module.exports was swapped out for the ES module equivalent export default.

Supporting info https://babeljs.io/docs/en/options#sourcetype for sourceType: "unambiguous" not being 100% reliable.

Sorry, but this is a problem with your Babel config. Babel is perfectly capable of handling CommonJS modules.


CTHUB - move vdcd stuff into cthub project

Describe the task
Move the stuff in https://github.com/bcgov/vdcd to https://github.com/bcgov/cthub, subject to considerations.
Purpose
Originally, I thought it would be cleaner to have the vin decoding stuff as a separate project, but users may want, in metabase, to join decoded vin data with other data in CTHUB, so the vin decoder and cthub should be contained within the same application.
Acceptance Criteria

[ ] It was discussed that the decoded vin information should never change; we should remove the "upload of previously uploaded vin triggers a fresh decode" feature in https://github.com/bcgov/vdcd.
[ ] - [ ] The contents (or most of the contents) in https://github.com/bcgov/vdcd should be moved to https://github.com/bcgov/cthub, and should run as before; some modifications/omissions here and there would probably be necessary (e.g. updating paths, excluding certain classes, etc.)

Hey team! Please add your planning poker estimate with Zenhub @emi-hi @JulianForeman @tim738745

Will test when Kuan is finished his dev ops work.

@shayjeff I think you can test this in metabase dev now; you can have a look at the "Vpic Decoded Vin Record" table in the CTHUB database


Added driver for ENC28J60 Ethernet Controller (IDFGH-2272)

I wrote a driver for the ENC28J60 Ethernet Controller.
The ENC28J60 uses SPI (similar to the DM9051) to communicate with the ESP32. The format of driver I wrote is based off of the DM9051 driver.
I also re-wrote the "basic" ethernet example so that this driver can be used.
I have also tested this driver with the MQTT Publish and HTTP request examples without any problems.

@hendog82 Thanks for the contribution.

How do I sign the CLA?
When I try to sign it, it shows "You have signed the CLA for espressif/esp-idf" but when I return to this pull request it shows "Pending — Contributor License Agreement is not signed yet."

Thanks for your contribution again, could please sign the CLA again? Maybe it's caused by network issue?

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.5 out of 13 committers have signed the CLA.:white_check_mark: projectgus:white_check_mark: igrr:white_check_mark: mahavirj:white_check_mark: hendog82:white_check_mark: krzychb:x: weitianhua:x: espxiehang:x: david-cermak:x: koobest:x: jack0c:x: suda-morris:x: hrudaynathdhabe:x: costaudweitianhua seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

I couldn't sign the CLA because I was committing changes as the root user so the commits were not linked to my github. I tried to fix it but it looks like I made a whole mess.
Please use pull request #4435  instead of this one.
Thank you

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 16 committers have signed the CLA.:white_check_mark: suda-morris:white_check_mark: hendog82:x: koobest:x: Wth-Esp:x: espxiehang:x: david-cermak:x: GYC-Espressif:x: mahavirj:x: jack0c:x: krzychb:x: costaud:x: projectgus:x: espressif-bot:x: hrudaynathdhabe:x: igrr:x: 0xjakobYou have signed the CLA already but the status is still pending? Let us recheck it.


Release machinelearningservices microsoft.machine learning services 2023 10 01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM (Control Plane) API Specification Update Pull Request
PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

[1] ARM review queue (for merge queues, see [4])
The PRs are processed by time opened, ascending. Your PR may show up on 2nd or later page.
If you addressed Step 1 from the diagram and your PR is not showing up in the queue, ensure the label ARMChangesRequested
is removed from your PR. This should cause the label WaitForARMFeedback to be added.
[2] https://aka.ms/azsdk/support/specreview-channel
[3] List of SDK breaking changes approvers in pinned Teams announcement
[4] public repo merge queue, private repo merge queue (for ARM review queue, [1])
If you need further help with anything, see Getting help section below.
Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[x] New API version. (Such PR should have been generated with OpenAPI Hub, per this wiki doc.)
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to Step 2, "ARM Review", for this PR.

Breaking changes review (Step 1)

If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
you must follow the breaking changes process.
IMPORTANT This applies even if:

The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
You believe there is no need for you to request breaking change approval, for any reason.
Such claims must be reviewed, and the process is the same.

ARM API changes review (Step 2)

If this PR is in purview of ARM review then automation will add the ARMReview label.
If you want to force ARM review, add the label yourself.
Proceed according to the diagram at the top of this comment.

Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.
Getting help

For general PR approval workflow, see the diagram at the top of this comment.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

            "minimum": 10

nit, I'd suggest don't document min/maximum here, just leave your server the flexibility to set its own policy

Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/stable/2023-10-01/machineLearningServices.json:168 in 5fb8294. [](commit_id = 5fb829494ad9388ad5baba80e9e74d79d377a163, deletion_comment = False)

        "description": "Create or update request is successful.",

this is... not exactly the way I would want to describe it. 200 should always mean the resource was Updated.... without being created. I.e. it was effectively an 'update' request.
201 should mean it was created.

Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/stable/2023-10-01/mfe.json:548 in 5fb8294. [](commit_id = 5fb829494ad9388ad5baba80e9e74d79d377a163, deletion_comment = False)

    }

I wonder if the SDK codegen etc allows this multiple inheritance looking pattern?

Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/stable/2023-10-01/mfe.json:9470 in 5fb8294. [](commit_id = 5fb829494ad9388ad5baba80e9e74d79d377a163, deletion_comment = False)

    }

I don't suppose you actually wanted to replace Datastore as the base type, or insert AzureDatastore in the inheritance sandwich instead?

In reply to:<PHONE_NUMBER>

Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/stable/2023-10-01/mfe.json:9470 in 3ec6f33. [](commit_id = 3ec6f33430feff940b64a261f3694ab33833c40c, deletion_comment = False)

    }

I'm curious if you need these properties - are they not redundant with the resource's id property?

Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/stable/2023-10-01/mfe.json:9578 in 5fb8294. [](commit_id = 5fb829494ad9388ad5baba80e9e74d79d377a163, deletion_comment = False)

        "description": "Success."

Should probably say something more like
"successfully started the long running operation. to get status..."

Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/stable/2023-10-01/machineLearningServices.json:1304 in 5fb8294. [](commit_id = 5fb829494ad9388ad5baba80e9e74d79d377a163, deletion_comment = False)

      },

Should probably say something more like successfully started the long running operation.
[noting  "x-ms-long-running-operation": true,]

Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/stable/2023-10-01/machineLearningServices.json:1217 in 5fb8294. [](commit_id = 5fb829494ad9388ad5baba80e9e74d79d377a163, deletion_comment = False)

      },

Should probably say something more like successfully started the long running operation.
[noting  "x-ms-long-running-operation": true,]

Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/stable/2023-10-01/machineLearningServices.json:1305 in 5fb8294. [](commit_id = 5fb829494ad9388ad5baba80e9e74d79d377a163, deletion_comment = False)

all of the changes in this PR have been previously ARM approved. This PR is a batching of API changes so we can introduce the version in main with all the changes

PRs Currently Merged:
https://github.com/Azure/azure-rest-api-specs/pull/24968
https://github.com/Azure/azure-rest-api-specs/pull/25072
https://github.com/Azure/azure-rest-api-specs/pull/25289
https://github.com/Azure/azure-rest-api-specs/pull/25374
https://github.com/Azure/azure-rest-api-specs/pull/25519
https://github.com/Azure/azure-rest-api-specs/pull/25765
https://github.com/Azure/azure-rest-api-specs/pull/25775
https://github.com/Azure/azure-rest-api-specs/pull/25619
Coming soon/WIP Approval/Merge:
https://github.com/Azure/azure-rest-api-specs/pull/25801
https://github.com/Azure/azure-rest-api-specs/pull/25729
https://github.com/Azure/azure-rest-api-specs/pull/25594

    }

Please respond to this.

In reply to:<PHONE_NUMBER>

Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/stable/2023-10-01/mfe.json:9578 in 5fb8294. [](commit_id = 5fb829494ad9388ad5baba80e9e74d79d377a163, deletion_comment = False)

    }

Please respond to this.
In reply to:<PHONE_NUMBER>
Refers to: specification/machinelearningservices/resource-manager/Microsoft.MachineLearningServices/stable/2023-10-01/mfe.json:9578 in 5fb8294. [](commit_id = 5fb8294, deletion_comment = False)

As I have mentioned before, all of these changes are either legacy or pre-reviewed/approved and are team specific. This PR is just to add them in as a unit.

Please fix circular references as mentioned in other swagger PR also.

I have these this PR's which is approved and will be merged soon: https://github.com/Azure/azure-rest-api-specs/pull/25879
to fix this request

/pr RequestMerge

/pr RequestMerge

merge conflict https://github.com/Azure/azure-rest-api-specs/pull/26085

please resolve the branch conflict and then run /pr RequestMerge to request merge

will merge this PR through proxy/shadow: https://github.com/Azure/azure-rest-api-specs/pull/26104


[BUGFIX] Fix exception rendering on failed request

If a request to elasticsearch fails, for example in case of a
missing request, the error part of the ES response array is set
as the exception message which is then tried to be rendered
as part of the message and raises an exception due to array
string conversion.

:+1: nice one


Notifications not firing

Device: Go Flip 4
OS: KaiOS 3.0
Version: master
When I set an event to give me a notification x minutes before the event it never fires. Could be KaiOS 3.0 specific.
PS: an options to fire a notification at the start of an event would be nice too ;)

pls try the new version

It appears to still not work with master.

I don't see a notification permission in the settings either.

@NuclearKev do you have a way to see the console.log output of the app?

@NuclearKev do you have a way to see the console.log output of the app?

I know that I can't using Firefox/Nightly to view the log but I haven't tried logcat - I can try it sometime (probably not this week, however).

I was able to get logcat running via adb. Here is the output from me opening greg:
12-30 21:00:10.875  1704  1704 E /system/b2g/b2g: Failed to get IAshmemDeviceService.
12-30 21:00:17.766   695   975 W api_daemon::session: Processing 'ProcManagerService request: ProcessServiceCommit' took too long: 12ms
12-30 21:00:17.967   693  4008 E Sandbox : SandboxBroker: denied op=open rflags=2400000 perms=0 path=/data/misc/zoneinfo/current/tzdata for pid=4006
12-30 21:00:17.971   693  4008 E Sandbox : SandboxBroker: denied op=open rflags=2400000 perms=0 path=/data/misc/zoneinfo/current/tzdata for pid=4006
12-30 21:00:18.291   695   975 W api_daemon::session: Processing 'ProcManagerService request: ProcessServiceCommit' took too long: 11ms
12-30 21:00:18.382   693  4008 E Sandbox : SandboxBroker: denied op=open rflags=2500000 perms=0 path=/dev/__properties__/u:object_r:use_memfd_prop:s0 for pid=4006
12-30 21:00:18.382  4006  4006 E libc    : Access denied finding property "sys.use_memfd"
12-30 21:00:18.388   693  4008 E Sandbox : SandboxBroker: denied op=access rflags=6 perms=0 path=/dev/binder for pid=4006
12-30 21:00:18.388  4006  4006 E /system/b2g/b2g: Failed to get IAshmemDeviceService.
12-30 21:00:18.522  4006  4006 E /system/b2g/b2g: Failed to get IAshmemDeviceService.
12-30 21:00:18.566   693   693 E GeckoConsole: [JavaScript Error: "uncaught exception: Object"]
12-30 21:00:22.703   387   387 E cutils-trace: Error opening trace file: No such file or directory (2)
12-30 21:00:22.918   387   387 W Gralloc3: mapper 3.x is not supported

It doesn't appear that any useful information was logged in regards to this issue.

thanks, I think these are system outputswould it be possible to show me the console.log output of the webIDE?
I would be interested in this https://github.com/strukturart/greg/blob/32dc7193f79870c13e18a6bf7c746cd4e7cf1efd/application/index.js#L2

As of now, I cannot connect the GF4 to the webide. It requires root permissions.

okay, i can also write the information in an alert ().

@NuclearKev is it possible to test the alarm api with the KaiOs 3 simulator?


Wrapping Pardiso

Designed for handling real symmetric and complex symmetric matrix.
Requires:
https://github.com/dwfmarchant/pyMKL

Coverage remained the same at 92.519% when pulling f008860e335f0573c1aa8e89965d55d164fa71fb on sgkang:master into 898b75b73d645458d7ae5f3eedd0bb8dd6e4d538 on rowanc1:master.

Coverage remained the same at 92.519% when pulling fbe8dddef936e28b2e108aeda42b5aad7d4fca71 on sgkang:master into 898b75b73d645458d7ae5f3eedd0bb8dd6e4d538 on rowanc1:master.

Could you copy the function you need out of SimPEG to keep the requirements down?

Not sure sp is not recognized ... any idea @lheagy @rowanc1

See #8


file_modified icon is not buffer local

Hey Mr. wind 👋, thank you for building this next level status line plugin 🔥. Also, for the maintenance and solving the last opened issue.
I'm being bold enough to open another one - hoping that's not too bothering 🙈.
The problem the issue title describes reveals itself using the file_modified component in an inactive status line. The component will use the state of the currently active buffer instead of the local buffer. Modifying a buffer will show the modified icon in the inactive buffer even when no modifications happen.
I tried to solve it by mimicking the cache variant of other components like you for example for the file_name and file icon. Unfortunately I wasn't able to solve it that way. Maybe it's something really simple I didn't spot.

i fix that you can use file_modified("+", true) for use it on inactive buffer

Awesome, thank you!

I just came to test it. It seems that adding the true arg always enables it modified icon, no matter the modified state.


CS lime-169, Updating the comments in Creditline.sol

Description
• Parameter _timeElapsed lacks description in calculateInterest.
• State variables protocolFeeFraction and liquidatorRewardFraction have a ambiguous description: it is multiplied by 10**30.
Integrations Checklist

[ ]  Have any function signatures changed? If yes, outline below.
[ ]  Have any features changed or been added? If yes, outline below.
[ ]  Have any events changed or been added? If yes, outline below.
[ ]  Has all documentation been updated?

Changelog

Adding the correct comments in creditline.sol

Event Signature Changes
Function Signature Changes
Features
Events

Codecov Report

Merging #279 (63a4354) into v1-alpha (f3e84c8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           v1-alpha     #279   +/-   ##
=========================================
  Coverage     78.49%   78.49%
=========================================
  Files            16       16
  Lines          1679     1679
  Branches        309      309
=========================================
  Hits           1318     1318
  Misses          361      361

Impacted Files
Coverage Δ

contracts/CreditLine/CreditLine.sol
84.01% <ø> (ø)

contracts/Pool/Extension.sol
84.05% <ø> (ø)

contracts/Pool/PoolFactory.sol
68.96% <ø> (ø)

contracts/SavingsAccount/SavingsAccountUtil.sol
86.04% <ø> (ø)

contracts/yield/NoYield.sol
78.57% <ø> (ø)

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 5b616e7...63a4354. Read the comment docs.


The block didn't update after load a board with same opcode block.

Select arduino uno first then select arduino mega2560, the pin list didn't update at all, still same with the uno.

Fixed in 3727789d48cf647bb596dc53155122e9e0dfe56a.


user: Tutoring Notes and Comments Page

references #14
As a User, I want to access a dedicated page where I can view tutoring notes and comments about specific sessions, so that I can review progress and insights shared during tutoring sessions.
Definition of Done

[x] Ensure that users can navigate to the tutoring notes page from the dashboard seamlessly.
[ ] Implement functionality to retrieve and display session-specific notes and comments dynamically.
[ ] Enable users to filter and search for notes/comments based on session parameters (e.g., date, tutor).

Tasks

[x] Research Modals for Page/Modal decision
[x] Create the component for the new page
[x] Add link to the dashboard for the notes
[x] Add modal/page to the dashboard
[ ] add endpoint to add session notes
[ ] add endpoint to get session notes

Researched proper practices to begin modal production
[t : 120]

Component made
[t : 60]

Basic Skeleton of the Page made
[t: 120]

Basic Modal Completed
[t : 60 ]

About halfway done
[t : 120]

Refactored after unsuccessful testing
[t : 120]

Component complete, db connection still broken
[t : 75]


Please add vshot -- need to change directories where volatility and bulk extractor are located

https://github.com/CrowdStrike/Forensics/blob/master/vshot
need to change directories where volatility and bulk extractor are located -- each under /usr/bin

Added by https://github.com/sans-dfir/sift-files/commit/58818193e894ce30f5f9de92b11adcd18f523e78


New version: Genymobile.scrcpy version 3.0

Pull request has been created with komac v2.7.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

No errors to post.

(Automated response - build 899.)


WIP: Default serializing turtle format

Proposed changes
For creating default policies upon tenant creation, uses a turtle format file
fixes #45
Types of changes
What types of changes does your code introduce to the project: Put
an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing
functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after
creating the PR. If you're unsure about any of them, don't hesitate to
ask. We're here to help! This is simply a reminder of what we are going
to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] I have signed the Contributor License Agreement
[x] I have updated the RELEASE NOTES
[x] I have added tests that prove my fix is effective or that my
feature works
[x] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in
downstream modules

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

the default policy documentation will also need to reflect updates:
https://github.com/orchestracities/anubis/blob/master/docs/admin/default-policies.md

@Cerfoglg https://github.com/orchestracities/anubis/blob/master/docs/admin/default-policies.md
needs to be updated, not sure about: https://github.com/orchestracities/anubis/blob/45-turtle-parser/docs/user/policies.md

@Cerfoglg let's introduce tests for parsing combining different examples, to be sure all works as expected

@chicco785 I did. What other tests do you mean?

@chicco785 I did. What other tests do you mean?

sorry, I missed the new test file, let's merge then


tests: re-enable clang-tidy checks

What problem does this PR solve?
Issue Number: ref https://github.com/pingcap/tiflash/issues/9032
Problem Summary:
clang-tidy is temporary disable to merge the feature branch
https://github.com/pingcap/tiflash/pull/9486#discussion_r1780614126
What is changed and how it works?

re-enable the clang-tidy checks
Check List
Tests

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

Side effects

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

Documentation

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

Release note

None

/hold
I will push a commit to fix arm build later

/unhold


Updating the stats display

Need to update how the Stats are displayed in the stats page
Looking to make them in two columns instead of one. Divided by Offense and Defense
As well as make it more obvious what team you're on. I also want to be able to easily switch teams by a scroll bar at the top.

An attempt was made however it appears that it's going to take a major overhaul of the stats - will most likely be done later -
however:
-defense/offense titles added
-fixed a bug with the size of the player name
-added passing percentage

Was able to get them to appear side by side like previously expected
Stats is cleaner and in a spot I feel comfortable with an alpha release.
moving to review just in case I decide it's still incomplete


Expected known-size vector of Single, got Vector

System Information (please complete the following information):

Model Builder Version (available in Manage Extensions dialog): <IP_ADDRESS>0501
Visual Studio Version: 16.11.7

Describe the bug

On which step of the process did you run into an issue: Training
Clear description of the problem: it Expected known-size vector of Single, but it apparently got Vector
the Parameter name: inputSchema

To Reproduce
Steps to reproduce the behavior:

make an ML model
choose for the scenario "Data classification"
make a file named "db.txt" and put the following content in it:
spam[tab]1
not spam[tab]0
choose to predict col1
train for 60 seconds

Expected behavior
A clear and concise description of what you expected to happen.
it is supposed to train for 60 seconds
Screenshots
If applicable, add screenshots to help explain your problem.
https://prnt.sc/24j8vlp
Additional context
Add any other context about the problem here.

Sorry for the long delay in response here. Can you include your dataset?

Sorry again for the delay. Any chance you can attach the dataset?

@beccamc
I would like to say that I got the same error. I am unable to provide the dataset as at this time as it was a direct connection to a DB. However, when I exported the data I wanted to a CSV file, it was able to read the CSV file fine and do the training. I will try to see if I can somehow sanatize the data and get you a DB backup that causes the error.
I am using
ML.Net Model Builder 2022 Version <IP_ADDRESS>1505
Visual Studio Enterprise 2022 (64-bit) Version 17.1.5
If you want me to create a new bug ticket I can, however, this is the same error I was getting while training.

@sbuchok The dataset or even the schema would be helpful.

@beccamc I will try to have a DB backup with only the table in it and the data by end of day. I will also try to provide screenshots and an STR.

I don't have enough information to reproduce this problem. If you are still seeing it please open a new issue with the dataset. Thank you!

Deeply apologize for the extreme delay, i had issues for a very long time and couldn't reply and just forgot about this, i unfortunately lost my files so i am unable to provide you with the dataset.
Sorry again.

I can duplicate it by using the UI to train with the following CSV:
Score,Client,DepositID
0,5540,GJ9999


suggestions in improving the contributing.md docs

In my earlier attempt in fixing one of the issues, i faced the following challenges which i describe below as a form of updating the contributing.md documentation
line 25:
At this point make sure the Developer Mode is enabled in your user settings of Discord
graph LR
A(Discord Application) --> B(USER SETTINGS)--> C(APP SETTINGS)--> D(Adanced)--> E(Developer Mode = Enable)

alternatively we could say this at the beginning of the page on line 3  :
To develop this project locally, you will need to be running [NodeJS v20](https://nodejs.org/en/) , [pnpm v8](https://pnpm.io/). and have enabled Developer Mode in Discord
line 40 could be updated to pnpm i --frozen-lockfile  or pnpm install
npm ci refused to work for me showing the following as error
The npm ci command can only install with an existing package-lock.json or npm ERR! npm-shrinkwrap.json with lockfileVersion >= 1. Run an install with npm@5 or npm ERR! later to generate a package-lock.json file, then try again.
line 46 could be updated to pnpm run build.
line 73 could be updated to pnpm start.
line 83 could be updated to pnpm run lint.
Desktop (please complete the following information):

OS: [MacOS 13.4.1]
Browser: [Chrome]
Version: [122.0.6261.94

Additional context
I am happy to work on this issue once the correct approach and wording is decided.

This sounds good.
I think the developer mode comment should go directly before

Right-click on your Discord server icon (the one in which you'll be running the bot) and select Copy ID. This goes in your .env file as HOME_GUILD.

We could say something like If you haven't already, enable Developer Mode in your Discord client. Do this by opening your user/account settings, selecting Advanced, then toggling Developer Mode` on.

That's brilliant, Thank you.
are you ok with me updating the commands on lines 40, 46, 73 and 83?


Make swiftmailer reconnect to support daemonized queues

this bug with suggested fix was noted here
https://github.com/symfony/SwiftmailerBundle/issues/39
fix was tested and works locally

Nice idea.

Tweaked this a bit and fixed tests.


Update generative_network.py

Summary
Just a little fix for a better performance.  Nothing more.
Details and comments

 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.

Could you please add a description to the PR: what the goal of this PR is and why the suggested code is better? If it is better, I would rather explore the code to replace other such places.

Could you please add a description to the PR: what the goal of this PR is and why the suggested code is better? If it is better, I would rather explore the code to replace other such places.

Edited. Will be happy if the PR passed. Thanks

Could you please update this line as well?
https://github.com/Qiskit/qiskit-machine-learning/blob/main/qiskit_machine_learning/algorithms/distribution_learners/qgan/discriminative_network.py#L36

Could you please update this line as well?
https://github.com/Qiskit/qiskit-machine-learning/blob/main/qiskit_machine_learning/algorithms/distribution_learners/qgan/discriminative_network.py#L36

Done. I don't have time to test it all to make it smoother. It's like a slight modification of the above. Thank you


Error handling for custom params

Currently implementation removes the potentially stored file in case of an upload error, but does not respond to errors passed by the user in the call backs. Need to respond to those as well in some way.

your cloudinary storage does not have enough information

We remove the file if errors occur in user callbacks now


Request: cancel out of set-mark mode when copying

This is a feature request: if I select a region with ctrl-space followed by navigation commands, and then copy it with cmd-c, the editor remains in set-mark mode. So if I then navigate to somewhere else, I first have to unset marking mode with ctrl-g. Is it possible to automatically exit out of this set-mark mode when using the copy command?

How about using alt-w?
The current behavior is actually the built-in Atom function. Like emacs, atomic-emacs, doesn't do anything to the command key. This way also feels more in line with other applications: e.g. in the notes app cmd-c doesn't deselect.

Fair enough! I just rebound cmd-c in my personal keymap.cson, works great:
'atom-workspace atom-text-editor':
  'cmd-c': 'atomic-emacs:copy'


Update readme

Signed-off-by: guxiaoyang_s20_307<EMAIL_ADDRESS>Fixes : [#133 ]
Description :
fixed #133 .
Inform user to modified  GithubAuthClientID, GithubAuthClientSecret in config files.

Screenshot of changes :

Please ensure Checkmark the following :

[x] - Tested Localy
[x] - Checked in IE11 and Firefox
[ ] - Checked in mobile
[ ] - Backend work
[ ] - Frontend work
[ ] - Bug Fixes

@hsluoyz Please review!


jco gen command

For generating types and stub JS code for componentization, it could be nice to have a simple gen command.
Perhaps something like:
jco gen --wit file.wit

to generate the source code for componentizing a world, outputting world.js, with stub interfaces and functions and imports.
TypeScript generation is in theory a subset of the above, so a jco gen --wit file.wit --types might be useful.
Options should otherwise be similar to transpile, supporting different wit inputs, and world selection, naming and possibly even mapping.

Superseded by https://github.com/bytecodealliance/jco/issues/152.


Refactor localization configurations & models

This fixes a part of issue #545:

Rename GlobalConfiguration.SimpleCultures to GlobalConfiguration.Cultures
Rid off SimpleCulture and replace it with the Culture model
Rid off  ResourceString and use Resource model
Move the localization models to SimplCommerce.Infrastructure.Localization

Maybe it's not a good idea to move the Culture and Resource class from Localization module to Infrastructure. I think Infrastructure should not be concerned about data persistent or business domain.
Actually the Localization should be changed to EfLocalization so that maybe we can have others localization provider.

Agree but if we look closely the Culture & Resources are core piece for any localization source provider: JSON, Database, PO .. etc. All the implementation details are in localization Module

Further more I did an experiment for EfRequestCultureProvider so such this will be in the localization module not the localization infrastructure .. hope you got my point

FYI you can have a look to OrchardCore they did something similar here they construct a CultureDictionary & CultureDictionaryRecord as a primitives for localization, but the implementation details is up to the provider creator

ok, I think just move the mapping from attributes [Table("Localization_Culture")] and [Table("Localization_Resource")] to LocalizationCustomModelBuilder would be fine

That's what I wanna asked you about 😄


Improve the release process

Releasing new versions of the source is currently an entirely manual process. cargo-release could help, but has several odd interactions with workspaces, such as missing interpolation of version numbers in automated commits. This might be addressed by a small shell script which amends the commit history, or we might prefer to be more explicit about how the version numbers of the crates are bumped in relation to one another.
This issue tracks:

Determining how the versions of the crates in the workspace are bumped in relation to one another
To the extent possible, using an automated or semi-automated process for adding and pushing commits and publishing the new version(s) on crates.io.

cargo-release is not exactly worth its weight at this point. It requires a lot of configuration, and it doesn't play that well with the git history. I have a local branch with a shell script based off this from the Tokio project, which seems to provide the right amount of structure. But in any case, I'm putting this on hold for the moment, to work on other things.


Add ensure_connected method to client_transport

Why
Describe what prompted you to make this change, link relevant resources: Linear issues, Slack discussions, etc.
What changed
Describe what changed to a level of detail that someone with no context with your PR could be able to review it
Test plan
Describe what you did to test this change to a level of detail that allows your reviewer to test it

Current dependencies on/for this PR:

main:

PR #84

PR #85

PR #86 👈

This comment was autogenerated by Freephite.

Yeah, I think that's the best/most pythonic way to do it. I'm just trying to fit it in the existing structure we have, it's probably worth fixing down the line. I'm trying to get connection errors to bubble up as early as possible and we don't have a way to do that right now (other than triggering an RPC)


LXD: Disable idmapped mounts if LXD_SHIFTFS_DISABLE=true

This is a tentative fix for https://github.com/lxc/lxd/issues/10011

I guess we should've disabled this at daemon startup as that's where we also disable shiftfs? But works too where you did it.

I guess we should've disabled this at daemon startup as that's where we also disable shiftfs? But works too where you did it.

Are you thinking something like a d.state.OS.IdmappedMounts field?


Sylpheed.Sylpheed version 2.2.7a

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @KarbitsCode,
This manifest adds a DisplayVersion to the AppsAndFeaturesEntries that isn't present in previous manifest versions. This entry should be added to the previous versions, or removed from this version.
(Automated response - build 896.)

Hi @KarbitsCode,
This manifest adds a DisplayVersion to the AppsAndFeaturesEntries that isn't present in previous manifest versions. This entry should be added to the previous versions, or removed from this version.
(Automated response - build 896.)

https://github.com/microsoft/winget-pkgs/pull/176071#issuecomment-2386254197


Insert into statement table using postgres

Hi all,
I have my chatterbot connected to postgres database. The bot is trained initially.
Now, I am trying to insert into the statement table using the command

text = "Can you cook"
cur.execute('INSERT INTO statement (text) VALUES (%s)', text)

But throws an error

TypeError: not all arguments converted during string formatting

Is there a way to insert a row into the statement table (from python script) using postgres commands.
I am able to update and delete the rows though.
Many thanks

Hi @gunthercox
This issue is resolved and can be closed.
Thank  you.


ci: run tests on more Android versions

📜 Description

💡 Motivation and Context

📢 Changelog

JS

iOS

Android

🤔 How Has This Been Tested?

📸 Screenshots (if appropriate):

📝 Checklist

[ ] CI successfully passed
[ ] I added new mocks and corresponding unit-tests if library API was changed

maybe prepare e2e and then run e2e tests (using matrix strategy)?


iOS 11.3 WebApp shows blank video

Tried this on iOS 11.3 WebApp (safari under the hood).  It "works" in so far as it does not sleep but it pops up a blank square on a black background covering the WebApp which I am guessing is the video.  Tried basic things like hidden:true and display:none without success.  If I home and return it goes away and the app is still humming along.  Used same JS on android WebApp and all good so high degree of confidence that implementation is correct.

Adding a bump because I'm seeing this on the firefox focus browser on iOS 11.4.1.
 I can dismiss it, but very strange UX.

Thumbs up.
From: Hartley Brody<EMAIL_ADDRESS>Sent: Monday, August 6, 2018 11:26 PM
To: richtr/NoSleep.js<EMAIL_ADDRESS>Cc: jh<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [richtr/NoSleep.js] iOS 11.3 WebApp shows blank video (#51)
Adding a bump because I'm seeing this on the firefox focus browser on iOS 11.4.1.
 I can dismiss it, but very strange UX.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/richtr/NoSleep.js/issues/51#issuecomment-410921273, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AlNNhIRdtMv-mudKsYlfMNf-8owT8Tg5ks5uOQjIgaJpZM4Tzx62.

Yeah same issue here. Also trying to identify WebView through UA or the like, to not inclde nosleep in those scenarios, has been impossible so far...

I failed to disclose that I was running a web app on top of a thin native client.  You need to add the following snippet to your view controller in order to prevent it from popping the noSleep.js video to full screen.
            var config = new WKWebViewConfiguration

           // override Fullscreen mode for video and autoplay

            config.Preferences.JavaScriptEnabled = true;
            config.MediaPlaybackRequiresUserAction = false;
            config.AllowsInlineMediaPlayback = true;

That might work for those running a native mobile app.
Any ideas for those running regular ol' javascript websites in a mobile browser?

Haven't had a problem otherwise (tested on chrome and FF and Safari non-native on PC, Android and iPhone without issue) but the way I discovered the problem was a little experiment where I substituted my own conspicuous video to watch what was happening and when I did, it showed me my video in the requested dimensions and when I went to no sleep it popped up my video to full screen (whether I liked it or not - i.e. no manner of CSS styling affected this).  In this particular case since the video supplied in noSleep.js is "tiny" (perhaps not visible to the naked eye the the full screen pops up a white box but what it really is doing is making the video full screen - unfortunately is it doesn't show you the video controls (which would have been a useful tip-off).
This led me to conclude that it was not nosleep.js per se  but something in the hosting mechanism that was overriding any settings that I applied to my test video.  I would look at the inline settings that are being employed in noSleep.js and see if whatever is hosting has some other settings that override it and force the video to full screen.  Look for some "inline" setting that may be missing.
Hope that helps....