If at least 1 node is set to hidden, useNodesInitialized will never return true

Describe the Bug
useNodesInitialized will never return true if at least one node is hidden. Technically this behaviour is what the documentation describes, as no size will be calculated for the hidden node, but I think it would be more useful if useNodesInitialized only concidered visible nodes.
My use case is that I want to do automatic layout calculations, but simply skip the nodes that are not visible.
Your Example Website or App
https://codesandbox.io/s/romantic-jones-wfwu0x?file=/App.js
Steps to Reproduce the Bug or Issue

Open the Codesandbox
Check the console, it will log {nodesInitialized: false}.
Remove the hidden attribute from the first node
Observe that {nodesInitialized: true} is logged as expected

Expected behavior
useNodesInitialized should only concider visible nodes
Screenshots or Videos
No response
Platform
Version: 11.6.1
Additional context
No response

I agree here. I thought that hidden nodes would not be taken into account here. In the next release, hidden nodes will be ignored by default and we added an option to configure it https://github.com/wbkd/react-flow/blob/next-release/packages/core/src/hooks/useNodesInitialized.ts#L20-L22

available/changed in v11.7.0

I agree here. I thought that hidden nodes would not be taken into account here. In the next release, hidden nodes will be ignored by default and we added an option to configure it https://github.com/wbkd/react-flow/blob/next-release/packages/core/src/hooks/useNodesInitialized.ts#L20-L22

I agree here. I thought that hidden nodes would not be taken into account here. In the next release, hidden nodes will be ignored by default and we added an option to configure it https://github.com/wbkd/react-flow/blob/next-release/packages/core/src/hooks/useNodesInitialized.ts#L20-L22

But useNodesInitialized still giving false  with includeHiddenNodes set to true and atleast one node set to hidden:true


Wrong feedback after command "!!!"

What
After entering a statement with !!! bot gives the feedback: "!!" isn't a registered command.
Problem
Solution
Check if more then one ! is given

https://github.com/Technik-Tueftler/TeTueTwitchBot/pull/8


Information on the Data Source

This blog post offers interesting information on the origin of the data https://www.dailymaverick.co.za/article/2021-12-06-how-i-reconstructed-an-unwritten-ancient-african-language/

It seems that we have all the sources right in this dataset. Is this issue open for a special reason or can we close it for the Lexibank release, @LinguList?

Crazy, I do not remember WHEN I added them, or if it was even me. Magic. I must have pulled it from the XML file.

I just checked, it cannot have been me. Who added the sources in the end?

Sorry, @MuffinLinwist, I just saw that you did this, how cool. Yes, we can close it. Thanks again for your help with that in the past!


Contribution README update

Originally the gem and repo was maintained under salemove organization,
currently opentracing-contrib. Previously the repo was named faraday-tracer,
now more specific ruby-faraday-tracer. The documentation still points
to the previous repo url. The PR updates the outdated information.

Thank you


Add unit tests and a11y test

We should probably add unit tests to the component library.
In addition, there is a storybook plugin for a11y test we can consider using.

Storybook now uses addon-a11y


Diverse Dragons

Diverse Dragons would be amazing in this list

We're currently using Infinite Dragon Variants to spruce up the appearance of dragons,but Diverse Dragons Collection does seem to go a bit beyond just looks, with new attacks/leveling system for these dragons.
It looks great!I recall using an earlier iteration of that mod at some point. I will test it with the load order and see if it works out well with other mods. If so, it will make its way in :) This issue will be updated with the results.

For now at least, I've decided that Dragon War in combination with Infinite Dragon Variants is a very much good amount of Dragon variety in the game. There may come a time when I revisit this, but for now I'll close this.


Fix validation logic and other things

This PR was split into:

#348
#349; as well as:

#370
#373

Another TBD PR with the AlertRulesProvider merge.

We explicitly synthesized a temporary file because the reldata was broken
We should probably test that. I don't remember which charm was creating the double nesting in the first place. Maybe grafana agent, and my dedupe patch squashd it
we [need to] build a list of what different rulesets may look like, because the built-in rules clearly aren't handled the same way, and we should go through a matrix to figure out which cases we need to test.
-- @rbarry82

I'm thinking about these cases for provider's rule files:

one file with one rule per file, and another file with a number of groups and a number of alerts
one rule per file, and the alertname same as filename.
multiple rules per file, and with the same alertname, groupname and filename.

Anything else?

Merging AlertRulesProvider (as well as MetricsAggregator?) into MetricsEndpointProvider can be considered for v1.
Closing for now.


Not work in larger data set

When the data set is large, it will return all NaN in any position!

sorry! I make a mistake . The larger data set is no problem! When the data set has the same position, it will return all NaN in any position. so I will filter the same points myself.


Support Multitask in mmseg

Hello everyone,
I am adding all of you as reviewers so that everyone can understand how we will be modifying the code, creating pull requests, and conducting code reviews.
The template for creating PR was already given by the mmseg developer; I just modified parts of it 😄 .
Motivation
To enable multitask learning in segmentation
Modification
Changes have been made in Dataset, model, Metric and Custom hooks.
A detailed description of all the changes is available at https://github.com/Fernando961226/sea-ice-mmseg/blob/mmseg_multitask/docs_ai4arctic/multitask_readme.md
Also, https://github.com/astokholm/AI4ArcticSeaIceChallenge repo has been added as a folder called AI4ArcticSeaIceChallenge.
Use cases
The template config file to be used can be found here: configs/multi_task_ai4arctic/mae_ai4arctic_ds5_pt_80_ft_20.py
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMDet3D.
The documentation has been modified accordingly, like docstring or example tutorials.

I added AI4ArticSeaIceChallenge as a folder is because it helps differentiate between original and mmseg files. Also the AI4ArticSeaIceChallenge folder is only 4MB.
Changes to demo/image_demo.py is done so that the result is saved. By default save_dir is none, so earlier no file was saved. Reference: https://github.com/Fernando961226/sea-ice-mmseg/blob/f3ba98a493fde6e69be2f43649037e92f9f90d05/mmseg/apis/inference.py#L152

@echonax07 That make sense, however we add additional files to the repo that do nothing it will just make it more confusing for some comming in.

@Fernando961226 True; in that case, feel free to delete any unnecessary files as you are writing a new data loader.

Okay in that case let's just merge and we can delete them later once the dataloader is completed!


add start_stage to run-satellite

This PR adds an optional start_stage param to run-satellite.
The main use is in a "push to talk" feature I am preparing that allows a satellite to be "activated" from the server side. In this case the server will send run-satellite with start_stage = asr to indicate that it wants the satellite to start a pipeline going directly to ASR (skipping vad / wake word / etc). More details will be provided in the corresponding PR in home-assistant/core.
But there could be other uses. run-satellite indicates that the server is ready to run pipelines, it makes sense to be able to specify specific properties of these pipelines.
The param is optional, if None the satellite chooses the starting stage, as it happens now.

Looks good, thank you!


 Quality Gate passed

 Quality Gate passed
Issues
 0 New issues
 0 Accepted issues
Measures
 0 Security Hotspots
 0.0% Coverage on New Code
 0.0% Duplication on New Code
See analysis details on SonarCloud
Originally posted by @sonarcloud in https://github.com/MetaMask/metamask-mobile/pull/10636#issuecomment-2293890914

 Quality Gate passed
Issues
 0 New issues
 0 Accepted issues
Measures
 0 Security Hotspots
 0.0% Coverage on New Code
 0.0% Duplication on New Code
See analysis details on SonarCloud
Originally posted by @sonarcloud in https://github.com/MetaMask/metamask-mobile/pull/10636#issuecomment-2293890914


@ApplicationModuleTest does not work in subclass

Hi, example:
// @SpringBootTest  -- it works
@ApplicationModuleTest abstract class MyBaseTest {}

class MyConcreteTest extends MyBaseTest {

    @Test
    void contextLoads() {
    }
}

Starting MyConcreteTest
A component required a bean of type 'org.springframework.modulith.test.ModuleTestExecution' that could not be found.

This sounds like a duplicate of #173 . Would you mind trying the 0.6 snapshots?

Fixed in 0.6.0-SNAPSHOT

Awesome, happy coding! 🙃


java.lang.NoSuchMethodError: org.apache.spark.scheduler.LiveListenerBus.removeListener

For bugs or installation issues, please provide the following information.
The more information you provide, the more easily we will be able to offer
help and advice.
Environment info
Operating System: MacOSX $ 10.12.6
Compiler: gcc-7
Package used (python/R/jvm/C++): python
xgboost version used: commit b08762066113db81b6872dfcc2c8f253ac092da4
If installing from source, please provide

The commit hash (git rev-parse HEAD)
b08762066113db81b6872dfcc2c8f253ac092da4

Logs will be helpful (If logs are large, please upload as attachment).

add [jvm-packages] in the title to make it quickly be identified

the gcc version and distribution

If you are using python package, please provide

The python version and distribution: python 3.6

The command to install xgboost if you are not installing from source

Steps to reproduce

Compile xgboost from source on MacOSX
Install spark 2.3 standalone
Install https://github.com/apache/incubator-toree
Install https://github.com/jupyter-scala/jupyter-scala
Fire up jupyter-notebook, choose apache toree scala kernel

Observe error below
What have you tried?

Installing apache-toree via pip and compiling it form source

The problem seems to be caused by XGBoost.
ERROR:
numRound: Int = 10
numWorkers: Int = 4
paramMap: scala.collection.immutable.Map[String,Any] = Map(min_child_weight -> 3.0, base_score -> 0.005, colsample_bylevel -> 0.9, silent -> 1, seed -> 49, subsample -> 1.0, max_depth -> 10, objective -> reg:linear, eval_metric -> auc, eta -> 0.023, colsample_bytree -> 0.82)
java.lang.NoSuchMethodError: org.apache.spark.scheduler.LiveListenerBus.removeListener(Ljava/lang/Object;)V
at org.apache.spark.SparkParallelismTracker.safeExecute(SparkParallelismTracker.scala:84)
at org.apache.spark.SparkParallelismTracker.execute(SparkParallelismTracker.scala:107)
at ml.dmlc.xgboost4j.scala.spark.XGBoost$$anonfun$trainDistributed$4.apply(XGBoost.scala:354)
at ml.dmlc.xgboost4j.scala.spark.XGBoost$$anonfun$trainDistributed$4.apply(XGBoost.scala:337)
at scala.collection.TraversableLike$$anonfun$map$1.apply(TraversableLike.scala:234)
at scala.collection.TraversableLike$$anonfun$map$1.apply(TraversableLike.scala:234)
at scala.collection.immutable.List.foreach(List.scala:381)
at scala.collection.TraversableLike$class.map(TraversableLike.scala:234)
at scala.collection.immutable.List.map(List.scala:285)
at ml.dmlc.xgboost4j.scala.spark.XGBoost$.trainDistributed(XGBoost.scala:336)
at ml.dmlc.xgboost4j.scala.spark.XGBoostEstimator.train(XGBoostEstimator.scala:139)
at ml.dmlc.xgboost4j.scala.spark.XGBoostEstimator.train(XGBoostEstimator.scala:36)
at org.apache.spark.ml.Predictor.fit(Predictor.scala:118)
at ml.dmlc.xgboost4j.scala.spark.XGBoost$.trainWithDataFrame(XGBoost.scala:193)

you have either incompatible version of Spark/Scala

looking in jvm-xgboost/pom.xml 2.2.1 spark version can be found. Just bump it to 2.3.0 and build I guess.

Still not sure about how to fix it?


[glslang] Provide glslang-default-resource-limits library.

Describe the pull request

What does your PR fix?

This PR makes the glslang-default-resource-limits library publicly available for linking. This is required by third party tools using glslang, in order to get the required resource limits data structure. This struct is internally used by the glslang library and its ABI does change between releases, so it's not possible to maintain this data consistently on the third party side.

Which triplets are supported/not supported? Have you updated the CI baseline?

No change here, in comparison to what was previously available.

Does your PR follow the maintainer guide?

Yes, with the exception of bumping the version field, instead of increasing the port-version, as this was the convention used previously in this port.

LGTM now, thanks for your PR @wolfpld.

LGTM now, thanks for your PR @wolfpld.

/azp run

/azp run

I am not sure how relevant this is, but since this PR relates to glslang-default-resource-limits I was wondering it should have resovled likning the target correctly, but I'm getting this error:
error LNK2001: unresolved external symbol "struct TBuiltInResource const glslang::DefaultTBuiltInResource" (?DefaultTBuiltInResource@glslang@@3UTBuiltInResource@@B)
SDFRay4D.exe : fatal error LNK1120: 1 unresolved externals

If I pull the actual repository I can add this library and link the target correctly, but I am not sure if this package cmake config links the target as in glslang::default-resource-limits which I assume is causing this error.

I am not sure how relevant this is, but since this PR relates to glslang-default-resource-limits I was wondering it should have resovled likning the target correctly, but I'm getting this error:
error LNK2001: unresolved external symbol "struct TBuiltInResource const glslang::DefaultTBuiltInResource" (?DefaultTBuiltInResource@glslang@@3UTBuiltInResource@@B)
....exe : fatal error LNK1120: 1 unresolved externals

If I pull the actual repository I can add this library and link the target correctly, but I am not sure if this package cmake config links the target as in glslang::default-resource-limits which I assume is causing this error.


Create noahmac.py

Mac Gui

Great job Noah, just make sure to update some of the stuff from the windows version into here. The windows one will have some fixes for input and new puzzle.


Release build for apprtc failed for revision 17231

build_apprtc give me this output:

Setting up build environment for Android
Export the base settings of GYP_DEFINES so we can define how we want to build
User has not specified any gyp defines so we proceed with default
ARMv7 with Neon Build
GYP_DEFINES=host_os=linux libjingle_java=1 build_with_libjingle=1 build_with_chromium=0 enable_tracing=1 enable_android_opensl=0 use_sysroot=0 include_tests=0 fastbuild=2 OS=android
Generate projects using GN
Done. Made 2783 targets from 171 files in 6641ms
Build AppRTCMobile in Release (arch: armv7)
Running ninja
ninja: Entering directory out_android_arm/Release' [2/3205] ACTION //base:android_runtime...//build/toolchain/android:android_arm) FAILED: gen/android_runtime_jni_headers/base/jni/Runtime_jni.h python ../../base/android/jni_generator/jni_generator.py --depfile gen/base/android_runtime_jni_headers__jni_Runtime.d --jar_file ../../third_party/android_tools/sdk/platforms/android-24/android.jar --input_file java/lang/Runtime.class --optimize_generation=1 --ptr_type=long --output_dir gen/android_runtime_jni_headers/base/jni --includes ../../../../../../base/android/jni_generator/jni_generator_helper.h --native_exports_optional Traceback (most recent call last): File "../../base/android/jni_generator/jni_generator.py", line 1418, in  sys.exit(main(sys.argv)) File "../../base/android/jni_generator/jni_generator.py", line 1411, in main GenerateJNIHeader(input_file, output_file, options) File "../../base/android/jni_generator/jni_generator.py", line 1314, in GenerateJNIHeader jni_from_javap = JNIFromJavaP.CreateFromClass(input_file, options) File "../../base/android/jni_generator/jni_generator.py", line 663, in CreateFromClass stderr=subprocess.PIPE) File "/usr/lib/python2.7/subprocess.py", line 679, in init errread, errwrite) File "/usr/lib/python2.7/subprocess.py", line 1249, in _execute_child raise child_exception OSError: [Errno 2] No such file or directory [4/3205] CC obj/third_party/boringssl/boringssl/pem_info.o ninja: build stopped: subcommand failed. Release build for apprtc failed for revision 17231 Setting up build environment for Android Export the base settings of GYP_DEFINES so we can define how we want to build User has not specified any gyp defines so we proceed with default ARMv8 with Neon Build GYP_DEFINES=host_os=linux libjingle_java=1 build_with_libjingle=1 build_with_chromium=0 enable_tracing=1 enable_android_opensl=0 use_sysroot=0 include_tests=0 fastbuild=2 OS=android target_arch=arm64 target_subarch=arm64 Generate projects using GN Done. Made 2777 targets from 171 files in 6529ms Build AppRTCMobile in Release (arch: armv8) Running ninja ninja: Entering directoryout_android_arm64/Release'
[6/3098] ACTION //base:android_runtime...build/toolchain/android:android_arm64)
FAILED: gen/android_runtime_jni_headers/base/jni/Runtime_jni.h
python ../../base/android/jni_generator/jni_generator.py --depfile gen/base/android_runtime_jni_headers__jni_Runtime.d --jar_file ../../third_party/android_tools/sdk/platforms/android-24/android.jar --input_file java/lang/Runtime.class --optimize_generation=1 --ptr_type=long --output_dir gen/android_runtime_jni_headers/base/jni --includes ../../../../../../base/android/jni_generator/jni_generator_helper.h --native_exports_optional
Traceback (most recent call last):
File "../../base/android/jni_generator/jni_generator.py", line 1418, in
sys.exit(main(sys.argv))
File "../../base/android/jni_generator/jni_generator.py", line 1411, in main
GenerateJNIHeader(input_file, output_file, options)
File "../../base/android/jni_generator/jni_generator.py", line 1314, in GenerateJNIHeader
jni_from_javap = JNIFromJavaP.CreateFromClass(input_file, options)
File "../../base/android/jni_generator/jni_generator.py", line 663, in CreateFromClass
stderr=subprocess.PIPE)
File "/usr/lib/python2.7/subprocess.py", line 679, in init
errread, errwrite)
File "/usr/lib/python2.7/subprocess.py", line 1249, in _execute_child
raise child_exception
OSError: [Errno 2] No such file or directory
[8/3098] CC obj/third_party/boringssl/boringssl/rsa.o
ninja: build stopped: subcommand failed.
Release build for apprtc failed for revision 17231
Setting up build environment for Android
Export the base settings of GYP_DEFINES so we can define how we want to build
User has not specified any gyp defines so we proceed with default
x86 Build
GYP_DEFINES=host_os=linux libjingle_java=1 build_with_libjingle=1 build_with_chromium=0 enable_tracing=1 enable_android_opensl=0 use_sysroot=0 include_tests=0 fastbuild=2 OS=android target_arch=ia32
Generate projects using GN
Done. Made 2805 targets from 172 files in 6940ms
Build AppRTCMobile in Release (arch: x86)
Running ninja
ninja: Entering directory out_android_x86/Release' [4/3358] ACTION //base:android_runtime...//build/toolchain/android:android_x86) FAILED: gen/android_runtime_jni_headers/base/jni/Runtime_jni.h python ../../base/android/jni_generator/jni_generator.py --depfile gen/base/android_runtime_jni_headers__jni_Runtime.d --jar_file ../../third_party/android_tools/sdk/platforms/android-24/android.jar --input_file java/lang/Runtime.class --optimize_generation=1 --ptr_type=long --output_dir gen/android_runtime_jni_headers/base/jni --includes ../../../../../../base/android/jni_generator/jni_generator_helper.h --native_exports_optional Traceback (most recent call last): File "../../base/android/jni_generator/jni_generator.py", line 1418, in  sys.exit(main(sys.argv)) File "../../base/android/jni_generator/jni_generator.py", line 1411, in main GenerateJNIHeader(input_file, output_file, options) File "../../base/android/jni_generator/jni_generator.py", line 1314, in GenerateJNIHeader jni_from_javap = JNIFromJavaP.CreateFromClass(input_file, options) File "../../base/android/jni_generator/jni_generator.py", line 663, in CreateFromClass stderr=subprocess.PIPE) File "/usr/lib/python2.7/subprocess.py", line 679, in init errread, errwrite) File "/usr/lib/python2.7/subprocess.py", line 1249, in _execute_child raise child_exception OSError: [Errno 2] No such file or directory [6/3358] CC obj/third_party/boringssl/boringssl/ctr.o ninja: build stopped: subcommand failed. Release build for apprtc failed for revision 17231 Setting up build environment for Android Export the base settings of GYP_DEFINES so we can define how we want to build User has not specified any gyp defines so we proceed with default x86_64 Build GYP_DEFINES=host_os=linux libjingle_java=1 build_with_libjingle=1 build_with_chromium=0 enable_tracing=1 enable_android_opensl=0 use_sysroot=0 include_tests=0 fastbuild=2 OS=android target_arch=x64 Generate projects using GN Done. Made 2805 targets from 172 files in 7060ms Build AppRTCMobile in Release (arch: x86_64) Running ninja ninja: Entering directoryout_android_x64/Release'
[4/3352] ACTION //base:android_runtime...//build/toolchain/android:android_x64)
FAILED: gen/android_runtime_jni_headers/base/jni/Runtime_jni.h
python ../../base/android/jni_generator/jni_generator.py --depfile gen/base/android_runtime_jni_headers__jni_Runtime.d --jar_file ../../third_party/android_tools/sdk/platforms/android-24/android.jar --input_file java/lang/Runtime.class --optimize_generation=1 --ptr_type=long --output_dir gen/android_runtime_jni_headers/base/jni --includes ../../../../../../base/android/jni_generator/jni_generator_helper.h --native_exports_optional
Traceback (most recent call last):
File "../../base/android/jni_generator/jni_generator.py", line 1418, in
sys.exit(main(sys.argv))
File "../../base/android/jni_generator/jni_generator.py", line 1411, in main
GenerateJNIHeader(input_file, output_file, options)
File "../../base/android/jni_generator/jni_generator.py", line 1314, in GenerateJNIHeader
jni_from_javap = JNIFromJavaP.CreateFromClass(input_file, options)
File "../../base/android/jni_generator/jni_generator.py", line 663, in CreateFromClass
stderr=subprocess.PIPE)
File "/usr/lib/python2.7/subprocess.py", line 679, in init
errread, errwrite)
File "/usr/lib/python2.7/subprocess.py", line 1249, in _execute_child
raise child_exception
OSError: [Errno 2] No such file or directory
[6/3352] CC obj/third_party/boringssl/boringssl/obj.o
ninja: build stopped: subcommand failed.
Release build for apprtc failed for revision 17231

Am I doing something wrong?

I ran

install-build-deps-android.sh

located in src/build/ and build_apprtc worked


Translation Delivery

Translation Delivery

This pull request is for translation delivery. Please review and approve them.

"COVID 19" should be written as "COVID-19" (with a hyphen).
Refer to:

https://www.coronawarn.app/en/
https://www.zusammengegencorona.de/en/basic-knowledge-about-coronavirus/

This pull request is for translation delivery. Please review and approve them.

@MikeMcC399 thank you again for the thorough review. With https://github.com/corona-warn-app/cwa-app-ccl/pull/112/commits/4c0fb856125e034a599944d5b05a9b77811e6233, all occurrences should be fixed 🚀


Support orderBy in query structure

Needed for library-data-platform/ui-ldp#12

OK, so this is done as far as possible. But there appears to be no way, using com.healthmarketscience.sqlbuilder.SelectQuery, to specify whether null values should sort to the start or end of the list. So for now, we just ignore those.

Can you use something like:
ORDER BY <boolean_expression> (ASC | DESC) NULLS (FIRST | LAST)

No doubt I could, if I were just gluing a string together by hand. But the code uses the com.healthmarketscience.sqlbuilder.SelectQuery package to build the query string, and that doesn't provide a method for doing this.

And there are only two open issues in the "feature requests" tracker at https://sourceforge.net/p/openhms/feature-requests/ — neither of them relevant.

I filed a ticket: https://sourceforge.net/p/openhms/feature-requests/28/

Good news! It turns out that you can do this after all, you just need to use a different API call. See https://github.com/library-data-platform/mod-ldp/commit/9d8685c6381434cbc53893aff58aa226bc337f12


Schema.xml not accessible

The link to schema.xml is not publicly visible. Please upload the schema.xml to the documentation homepage and change the link.
http://api-dev.freme-project.eu/doc/knowledge-base/freme-for-api-users/freme-ner.html

fixed.


Cross-browser compatibility (OSX/Safari)

The default bootstrap select & input form elements are broken in Safari (8/9 OSX Yosemite / El Capitan).
Screenshots:
Safari 8/9 (OSX Yosemite / El Capitan:)

Windows10 Chrome:

Please support  pure-css cross-browser compatiblity!

I do recall reading something about Safari (and iOS iirc) forcing their own browser-side styling, which is what you're seeing. You also didn't say which version of Bootstrap you are referring to, however the v4-alpha has different styling of selects, so I'm assuming you're talking about 3.x.x.

Boostrap 3.3.6, but this not work any other versions, too

@progcode I don't see what breakage you're referring to. Please describe the breakage in words.

@cvrebert i suspect the "breakage" is "it doesn't look the same". which is to be expected, as we don't fake non-native selects, but just rely on the platform's own select implementation ... so I think this is not a bug, so a won't-fix

Yeah, platform-native form controls are a feature, not a bug.
If you want absolute uniformity rather than nativeness, use http://v4-alpha.getbootstrap.com/components/forms/#custom-forms or http://wtfforms.com or similar.
If we've misunderstood you somehow, we'll be happy to reopen this.


Error on npm run dev

Node.js version:
v10.15.0
npm version:
6.9.1-next.0
Strapi version:
Operating system:
Windows 7 Ultimate x64
Which example is causing problem?
nuxt-strapi-deliveroo-clone-tutorial
Do you want to request a feature or report a bug?
bug
What is the current behavior?
x error Error: Plugin not found: D:[GitLab]\strapi-examples\nuxt-strapi-deliveroo-clone-tutorial\client.nuxt\bootstrap-vue.js
at Promise.all.plugins.map (D:[GitLab]\strapi-examples\nuxt-strapi-deliveroo-clone-tutorial\client\node_modules\nuxt\dist\nuxt.js:4011:15)
if I remove "modules: ['bootstrap-vue/nuxt']" in nuxt.config.js, above error is gone but
I will get another error which is.
nuxt-strapi-deliveroo-clone-tutorial\client\node_modules\snapdragon\lib\parser.js:473
throw new Error('no parsers registered for: "' + self.input.slice(0, 5) + '"');
If the current behavior is a bug, please provide the steps to reproduce the problem
I just follow the step given.
cd nuxt-strapi-deliveroo-clone-tutorial
cd client
npm install
npm run dev

Hello! I just pulled and followed same instruction as you did and it works on my side.


using two (or more) accessories in one combined accesory

Hi!
I'm looking for a way to use already written accessory plugin inside another accessory plugin.
My usecase is such: I have homebridge-eq3ble plugin which can show the target temperature on thermostat. However, the device doesn't provide way to get current temperature.
for current temperature I have accurite sensors which report to mqtt and then feed to homebridge with help of homebridge-mqtt-temperature.
So, I want to bridge those two accessories in one thermostat.
However, I want to make it universal and give the ability to connect any temperature sensors registered in homebridge and this eq-3 thermostat.
so, the question is:
a) is there possibility to programmatically get all registered accessories from inside another accessoryA (or serviceA), feed the accessoryA with values from chosen accessoryB (how to subscribe and get values - I know)  and hide the chosen accessoryB from homebridge advertised accessories set?
or:
b) is there a way given name and configuration parameters of accessoryA initialize it inside other accessoryB (universally, without knowing the constructor and other methods of chosen accessoryA) ? (and then I know the way to subscribe to the values of abstract TemperatureAccessory and feed it's values to accessoryB)

As far as I know this isn't possible.
But have a look at:
http://nodered.org/ (Node-RED is a tool for wiring together hardware devices, APIs and online services.)
https://flows.nodered.org/node/node-red-contrib-eq3-bluetooth
and
https://github.com/cflurin/homebridge-mqtt
Node-red is preinstalled on Rpi (jessie) and a mqtt-broker like mosquitto is easy to install on the same Rpi running node-red and homebridge.

yeah, that was understanding as well (on topic of homebridge).
I think I will do just consuming temperature from mqtt topic - as this is general enough.
(I just didn't want to add more dependencies like node-red for such simple task.
thank you


Update docker-registry to v0.9.1

I updated our private support tweaks to docker-registry against v0.9.1 of docker-registry. This required fixing the tests and some code to match new validation and behavior. Then I fixed our docker checksum calculation to match the behavior in docker-registry's current tests, purged the TarSum class we don't use now, had deis-registry build from the new-repository-import-v091 branch, and updated builder's docker to 1.4.1.
Closes #3079.

Code LGTM! It'll be nice to keep the Docker versions in-sync for #3048 - I'll rebase that once this is merged.

Jenkins passed - nice work on this, @mboersma

I want @bacongobbler to review this before we merge it, since these are his code changes.

Updated to use "docker/1.0.0" for User-Agent.

Still LGTM

thanks! :heart:
:shipit:

Using this PR I found this issue:
26/Feb/2015:17:16:01 +0000 DEBUG: [resolve_repository_name] image_name=https://dev-registry.cl:5000/app-frontend-rest
26/Feb/2015:17:16:01 +0000 DEBUG: [validate_repository_name] repository_name=app-frontend-rest
26/Feb/2015:17:16:01 +0000 INFO: Starting new HTTPS connection (1): dev-registry.cl
26/Feb/2015:17:16:01 +0000 DEBUG: api_error: __init__() got an unexpected keyword argument 'server_hostname'
......
Feb 26 17:16:01 nodo-2 sh[23342]: <IP_ADDRESS> - - [26/Feb/2015:17:16:01 +0000] "POST /v1/repositories/app-frontend-rest/tags HTTP/1.1" 500 88 "-" "python-requests/2.4.3 CPython/2.7.9 Linux/3.18.6"

and that's it. Even in debug mode :)

@aledbf a git grep shows nothing on server_hostname... You sure you're not running a modified version of the registry?
><> cd docker-registry/
><> git co new-repository-import-v091
Branch new-repository-import-v091 set up to track remote branch new-repository-import-v091 from origin.
Switched to a new branch 'new-repository-import-v091'
><> git grep server_hostname

@bacongobbler it's a new image from master


Refactor KeyManager

Partially addresses https://github.com/TBD54566975/web5-rs/issues/158
Context
The KeyManager and KeyStore traits are fuzzy abstractions -- the conceptual boundary between a "store" and a "manager" is not clear to me. KeyManager currently seems to be a wrapper around KeyStore with little-to-no added functionality. Also, the traits themselves have methods export_private_keys() and import_private_keys() which ought not to be coupled to the notion of "management" which entails storing private keys, signing, and getting public keys. See https://github.com/TBD54566975/web5-rs/issues/158 for more.
Design

Remove the KeyStore trait entirely. Move logic from struct InMemoryKeyStore into LocalKeyManager.
Pare down KeyManager to just three methods: generate_private_key(), sign(), and get_public_key().
Add trait KeyImporter with one required method import_with_alias() and one provided method import() which generates its own alias.

This PR does NOT add the KeyExporter trait described in https://github.com/TBD54566975/web5-rs/issues/158 because I do not see an immediate use case in web5-rs. Exporting private keys is risky stuff. Until we have a clear need for a standard interface for exporting, we ought to omit it.
Open questions

What is the purpose of the .alias() method in the Key trait? Is it a unique digest? Whatever it is, we should probably rename the method.
What is the expected behavior of KeyImporter when a key alias is overwritten with a new private key? Should that be an error? Is it up to each individual impl of KeyImporter? I lean towards the former.

Alright a handful of things...

This is amazing work. I've ✅'d but I propose we let it sit for a couple days at least to give people time to chime in if they choose.
I respect taking the discretion to remove the KeyStore and I approve. We have a guiding principal here in rust core to "prefer modularity over convenience" but in this case, we don't have a concrete use case (that we can even think of) where decoupling the abstractions is necessary, so yeah I think it's wise to remove.
I appreciate the named functions import_with_alias and import rather than using a single function with Option param. I think this is the ideal pattern here in our rust initiative -- it's slightly more verbose than what we're accustomed to with other languages, but it'll pay dividends IMO as we start bindings things.
💯 on not implementing the KeyExporter trait. I think what I'll probably do is eventually close #158 and create a new ticket for the export functionality, which hopefully just sits there forever. I'll handle that.

What is the purpose of the .alias() method in the Key trait? Is it a unique digest? Whatever it is, we should probably rename the method.

It's a good point... the way I think about it is, the key alias is akin to a primary key in a db, so it's self referential with respect to a key. It's slightly confusing though because we enable callers of key manager methods to override the key alias, so it's a matter which is conceptualized at the key manager level, but at the same time we implement the alias for the Jwk which is the thumbprint. 🤔 created https://github.com/TBD54566975/web5-rs/issues/206

What is the expected behavior of KeyImporter when a key alias is overwritten with a new private key? Should that be an error? Is it up to each individual impl of KeyImporter? I lean towards the former.

Yeah that should be an error. So the requirement is: key's cannot be imported which have a key alias which pre-exist in the key manager. Created https://github.com/TBD54566975/web5-rs/issues/207 @diehuxx feel free to go ahead and do that here or not and we can follow up, your call.

How is a portable did created without a key exporter?

How is a portable did created without a key exporter?

It can't. But, TBD. This is a hot topic, so perhaps I'll open a ticket to discuss the full merits. I know this has been discussed at length a while back, but it's appropriate to take a bit of time to reopen the matter with the rust initiative.


"Pinning" message to the bottom should be less frequent for gambas/polls to avoid so much flashing

Fixed as part of rewrite.


Feature/devise pwned password protection

For new user creation, however we do not enable registrations on muffi so this is only relevant when creating users via console.
Update:
Added after sign in flash warning

:+1:


Use the new charter palette

Description
This improvement addresses the whole documentation.
Taipy has a new charter palette:
Primary colors: #ff371a / #0b0b0e / #ffffff
Secondary colors: #55c1f6
Expected change
The colors have to be all changed in the documentation. Replacing the :
Primary colors: #ff462b / #191622 / #283282 / #44505c / #f7f8fc / #f0faff
Secondary colors: #8e1574 / #22aed1 / #7a3db8 / #ffc15e / #bfbac4 /

Today, 7 colors (5 colors + black + white) are used.
Since the previous charter has more colors than the new one (4 only), the issue is not just a simple replacement. We need either to redesign the doc pages, considering just one secondary color, or use more non-official colors.
Various locations:

[ ] Nav bar
[ ] User manual
[ ] Visual elements
[ ] Gallery

@RymMichaut
The style sheet used in the doc set can be found in
[taipy-doc]/taipy_theme/assets/stylesheets/avaiga.css.
There a quite a few colors and font sizes and stuff at the very beginning of that stylesheet.
You or you designer team can pick it up and change that!

Hi, can i work on this? @RymMichaut

@codermal7 this is not opened to Hacktoberfest.

no issues, I still want to work on it.

I let it to @jrobinAV and @FabienLelaquais to decide because it's a quite big issue

The issue is straightforward.
@RymMichaut We are just missing a mapping between today's palette and the new one you proposed.
You proposed 4 colors, while we are using 11 colors today. Please provide a complete mapping so anyone can work on this issue.

@jrobinAV , That's the thing, the new palette hasn't got more than 4 colors. We wanted the new website and designs to be as sober as possible

/taipy_theme/assets/stylesheets/avaiga.css

@FabienLelaquais, did yo mean: https://docs.taipy.io/en/latest/taipy_theme/assets/stylesheets/avaiga.css

@Rym Michaut @.***> No what I mean is: the stylesheet is
in the repo, where I refer to it.
On Tue, Oct 8, 2024 at 5:22 PM Rym @.***> wrote:

/taipy_theme/assets/stylesheets/avaiga.css
@FabienLelaquais https://github.com/FabienLelaquais, did yo mean:
https://docs.taipy.io/en/latest/taipy_theme/assets/stylesheets/avaiga.css
—
Reply to this email directly, view it on GitHub
https://github.com/Avaiga/taipy-doc/issues/1118#issuecomment-2400154352,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUUUKBY4C5UQHTOWAXPD5S3Z2P2DVAVCNFSM6AAAAABN42FGX6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBQGE2TIMZVGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

As @jrobinAV said, the old palette appears in :

the menu

the blocks

Either the contributor has the skills to change everything to get something like below or we wait for a designer to redo the whole doc UI/UX:

the menu as the website's

the blocks with minimalist design

What do you think @jrobinAV and @FabienLelaquais ?

We need a UX resource to implement the new charter on the documentation site.

Coexia?
Regards,
PS: I don't work on Wednesdays. If it's urgent, mention it in the mail.
Le ven. 13 déc. 2024 à 14:27, Jean-Robin @.***> a
écrit :

We need a UX resource to implement the new charter on the documentation
site.
—
Reply to this email directly, view it on GitHub
https://github.com/Avaiga/taipy-doc/issues/1118#issuecomment-2541463486,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHP2YASSSQITDAWYE23DABT2FLOCLAVCNFSM6AAAAABN42FGX6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBRGQ3DGNBYGY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add Capella to RestBuilderClientTest spec context

PR Description
Just extended the test to support the Capella data structures.
Fixed Issue(s)
N/A
Documentation

[X] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[X] I thought about adding a changelog entry, and added one if I deemed necessary.

LGTM. the only things that concerns me is that the values (especially roots) in the resources should be unique to check that we are not exchanging a value with another.

Agree. Changed them with more random ones.


abyssal lantern

Description:
Added Abyssal lantern to buyables, recent update ingame
Changes:
Added Abyssal lantern to buyables, recent update ingame
Other checks:

[x] I have tested all my changes thoroughly.

Boosts?


etcd returns 500 internal server error on ubuntu which causes postgres to crash.

Hi,
Postgres server crashes as a node tries to write a value in etcd and gets 500.
self.put_client_path("/members/%s" % member, {"value": connection_string, "ttl": self.ttl})

File "/home/i074560/shashank/governor/helpers/etcd.py", line 40, in put_client_path
opener.open(request)
File "/usr/lib/python2.7/urllib2.py", line 406, in open
response = meth(req, response)
File "/usr/lib/python2.7/urllib2.py", line 519, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python2.7/urllib2.py", line 444, in error
return self._call_chain(*args)
File "/usr/lib/python2.7/urllib2.py", line 378, in _call_chain
result = func(*args)
File "/usr/lib/python2.7/urllib2.py", line 527, in http_error_default
raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)
urllib2.HTTPError: HTTP Error 500: Internal Server Error
LOG:  received fast shutdown request

What is the impact if we just catch this exception and retry for few times instead of just letting postgres crash on first attempt?


Move async to dependencies

Is used by the src/ of the module, should be installed when running npm install ipfs-block. Hopefully fixes https://github.com/ipfs/js-ipfs-block/pull/22

@mmalmi could you check if this fixes the issue for you?

@dignifiedquire This fixes it.


Feature Request: Set a custom frame order or loop for AnimatedSprite

It would be cool if you could add a field to AnimatedSprite.FrameObject that designates the next frame to play after the current one.
I would be willing to submit a PR if you are open to this feature.

Lets open it: https://github.com/pixijs/pixi.js/blob/dev/src/extras/AnimatedSprite.js#L30
Now, what do you want me to add there?

If you wouldn't mind, could you please explain your proposal more and give a concrete example of how it would be used.

@ivanpopelyshev see #4348
@bigtimebuddy For example, if I had multiple animations for walking in a specific direction:
new PIXI.extras.AnimatedSprite(
    {texture: PIXI.Texture.from('walkUpLeftFoot.png', time: 0.2},
    {texture: PIXI.Texture.from('walkUpRightFoot.png', time: 0.2, next: 0},
    {texture: PIXI.Texture.from('walkDownLeftFoot.png', time: 0.2},
    {texture: PIXI.Texture.from('walkDownRightFoot.png', time: 0.2, next: 2},
    {texture: PIXI.Texture.from('walkLeftLeftFoot.png', time: 0.2},
    {texture: PIXI.Texture.from('walkLeftRightFoot.png', time: 0.2, next: 4},
    {texture: PIXI.Texture.from('walkRightLeftFoot.png', time: 0.2},
    {texture: PIXI.Texture.from('walkRightRightFoot.png', time: 0.2, next: 6}
)


[ast] Fix Lint, Latches syntax, & Registers numbers (32->39)

This Draft PR includes:

Fix Lint issues, including latches syntax.
Fix JEN decode from strict to loose
Update AST register's nuber from 32 to 39.

Signed-off-by: Jacob Levy<EMAIL_ADDRESS>

@tjaychen Please review and push it as part of PR #12570.

@tjaychen  @rswarbrick The FPGA runs fails! Not sure why can you guys take a look..

let me push my PR into this one and see what happens.

@Jacob-Levy
here's the fpga errors
2022-05-13T19:50:26.2334837Z ERROR: [Synth 8-327] inferring latch for variable 'sys_clk_byp_sel_reg' [/azp/agent/_work/1/a/build-out/hw/src/lowrisc_systems_ast_0.1/rtl/ast_clks_byp.sv:396]
2022-05-13T19:50:26.2335720Z ERROR: [Synth 8-327] inferring latch for variable 'io_clk_byp_sel_reg' [/azp/agent/_work/1/a/build-out/hw/src/lowrisc_systems_ast_0.1/rtl/ast_clks_byp.sv:397]
2022-05-13T19:50:26.2336747Z ERROR: [Synth 8-327] inferring latch for variable 'usb_clk_byp_sel_reg' [/azp/agent/_work/1/a/build-out/hw/src/lowrisc_systems_ast_0.1/rtl/ast_clks_byp.sv:398]
2022-05-13T19:50:26.2337602Z ERROR: [Synth 8-327] inferring latch for variable 'aon_clk_byp_sel_reg' [/azp/agent/_work/1/a/build-out/hw/src/lowrisc_systems_ast_0.1/rtl/ast_clks_byp.sv:399]
2022-05-13T19:50:26.2338522Z ERROR: [Synth 8-327] inferring latch for variable 'ext_freq_is_96m_reg' [/azp/agent/_work/1/a/build-out/hw/src/lowrisc_systems_ast_0.1/rtl/ast_clks_byp.sv:117]

@tjaychen In the real design the latch is needed, but I can drop it for the OS.
Will do the fix, rebase & rerun the draft.
Did the CI(private) issue cleared?

yeah the private CI issues appear to be cleared.
On Fri, May 13, 2022 at 2:35 PM Jacob @.***> wrote:

@tjaychen https://github.com/tjaychen In the real design the latch is
needed, but I can drop it for the OS.
Will do the fix, rebase & rerun the draft.
Did the CI(private) issue cleared?
—
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/12640#issuecomment-1126526447,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSUVJKITDBCBAXTIIIDVJ3DJLANCNFSM5V2OJ5NA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@tjaychen  It looks that all other check paased. The CI(private) fail since I an missing your PR #12570.
What is the CI (Execute tests on ChipWhisperer CW310 FPGA board) check? I
All the othere FPGA check passed.  t fails also on your PR...

@tjaychen It looks that all other check paased. The CI(private) fail since I an missing your PR #12570. What is the CI (Execute tests on ChipWhisperer CW310 FPGA board) check? I All the othere FPGA check passed. t fails also on your PR...

the failure last week appeared to be an fpga setup issue, all the fpga CI's last friday were just being directly cancelled.
I need to pull your update for the latch fix in OS (i just hacked in something on my end).

Merge as part of PR #12570


Make the ? suffix for empty attributes optional

This was necessary back when braces were optional, and so a bare identifier (e.g. title) could mean either an attribute or an element.
But with #137 making braces mandatory, there's no need for the ? anymore – it is perfectly unambiguous without it.
The old syntax is kept around to avoid breaking existing users.

TODO:

[ ] Update changelog
[ ] Update docs


MalDuino not working

My new MalDuino just came in the mail yesterday and I followed the guide on setting up the device. When I flashed the sketch onto the MalDuino errors suddenly started popping up in the Arduino IDE, I was stupid and didn't screenshot or copy the error messages but I will try to see if there are any log files and post them here later.
If it matters what PC I used to flash it, I was using a Raspberry Pi 4B 4GB, that might have something to do with it.
I thought that I would try to setup the MalDuino again today but first noticed that there isn't any lights flashing, didn't think much of it and redid the same process as yesterday but on another PC, a Windows 10 laptop this time. I first really noticed the problem when I was going to select the MalDuino in the Tools>Ports where it didn't even show up, I unplugged it and restarted the entire IDE to check if anything had changed. I did that multiple times and it still doesn't show up. I am currently guessing I somehow bricked the MalDuino, might help a bit if I find the logs but this is all I have currently.
EDIT: Forgot to mention it doesn't show up in Device Manager as well.

Sent a mail to Maltronics with this error and they replied with this link
https://www.youtube.com/watch?v=dFQHXm1y5Io
This fixed it for me.


feat: Include custom labels for NEW_RELIC_LABELS and also apply podlabel "k8s_agents_operator_version"

Description
https://new-relic.atlassian.net/browse/NR-311790

        env:
         - name: NEW_RELIC_LABELS
           value: "team:k8s"

Given this user provided label, the final value of NEW_RELIC_LABELS will be "team:k8s,operator:auto-injection"

Add pod label "k8s_agents_operator_version:{appVersionNo}" to every mutated pod.

Type of change

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

Checklist:

[ ] Documentation has been updated
[ ] This change requires changes in testing:

[ ] unit tests
[ ] E2E tests

@ramkrishankumarN I'm not sure it should be a comma. I think agents are using ; as the delimiter ex: https://docs.newrelic.com/docs/apm/agents/java-agent/configuration/java-agent-configuration-config-file/#labels


bad example in docs

Describe the bug
In the docs, there is a link to a broken CodeSandbox.
Screenshots
!(error in codesandbox)[https://sh-drop.s3.us-east-1.amazonaws.com/cs/Screenshot 2019-03-20 13.49.53.png)

The sandbox needs to have the loop protection removed. The example is fine, but codesandbox tries to detect that something will infinite loop if it exceeds like 10,000 loop iterations.

If you would like to, you could PR a sandbox.config.js file to that example folder located in the repo archives. Somewhere in there you can hard code that flag to turn off the infinite loop protection.


[Bug]: Trait method 'activities' will not be applied because it collides with 'LogsActivity'

What happened?
Trait method 'activities' will not be applied because it collides with 'LogsActivity' when adding the trait to user model
How to reproduce the bug
add the trait to user model
Package Version
2
PHP Version
8.2
Laravel Version
10
Which operating systems does with happen with?
macOS
Notes
https://github.com/edwink75/filament-user-activity/issues

Can you give me a little more details about the conflicting plugin you're using? And more details about the error?

z3d0x/filament-logger

Thanks for reporting. I added the package to the conflict section of composer.

Added to composer


prevent non existing repository

Revert "Fix the repo for CaaSP"
This reverts commit 45dc75ece636f7b64bb34aa0879d6d644694c771.
Fixes #35

Many states depend on /etc/zypp/repos.d/containers.repo, so that repo must be present, even when it is not used at all. It is not important that the repo does not make sense as CaaSP does not use repos.

@inercia still http://download.opensuse.org/repositories/Virtualization:/containers/openSUSE_Leap_1.0/ on casp needs to be fixed


spiceypy v2.1.2 (take 2)

Supersedes #7

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


New indoor classifications

Developments on archeryutils are preparing for indoor classification updates.
Here we update the tooling to make use of these new routines:

[X] Update databases to hold new indoor classifications
[X] Update tables routine to return new classifications and numbers
[X] Update calculator to return new classifications
[x] Handle higher than maximum score (may be done in archeryutils)

Will close #31

I think this is now all ready to go.
Handling of maximum scores is, indeed, done by archeryutils.
Holding off merging until process is complete and changes announced.

Updated event tables and added provisional warnings.
Open a new branch to remove warnings once changes are officially announced.


Provide new interactive window experience for editing notebooks directly

Reusing the interactive window, when opening an ipynb file, just load it into an interactive window, but with these differences:

Cells are editable
There's a convert button
Cells are reorderable (P2)

Start of this is here:
https://github.com/microsoft/vscode-python/tree/rchiodo/edit-notebooks

Will this feature make python interactive window more jupyter-alike?

Sort of. It will change the way you edit cells at least. They'll be inline like a jupyter notebook.

Sort of. It will change the way you edit cells at least. They'll be inline like a jupyter notebook.

How soon will this update be released? It won't be long, right?

Maybe next release, or the one after that. It's the next big thing we're working on.
Here's a gif of what I currently have. Comments welcome

Maybe next release, or the one after that. It's the next big thing we're working on.
Here's a gif of what I currently have. Comments welcome

Damn Brilliant!!  Will this feature be compatible with VIM extension? After this release, we could code on ipynb rather than .py file ?

Will this feature be compatible with VIM extension?

Likely not a first. Not sure how the VIM extension changes editing, so we'd have to look into it. Extensions that modify the editor would have to be reapplied to this window.

After this release, we could code on ipynb rather than .py file ?

Yep. That's the idea. Or go between them if you like.

Yep. That's the idea. Or go between them if you like.

Or the PyCharm2019 pro's jupyter is another solution

Sorry are you suggesting people should use pycharm, or that we should have a pycharm like experience?

Likely not a first. Not sure how the VIM extension changes editing, so we'd have to look into it. Extensions that modify the editor would have to be reapplied to this window.

Jupyter notebook or Jupyter Lab has its own VIM extension too. And thier VIM can move the curse between Cells. It ma

Sorry are you suggesting people should use pycharm, or that we should have a pycharm like experience?

Nope, I didn't suggest PyCharm. I don't use PyCharm actually. I mean PyCharm's jupyter style is another solution for VS Code's python interactive window. Sorry for the comment above, I didn't realise it will cause misundstanding.

I mean PyCharm's jupyter style is another solution for VS Code's python interactive window.

That's what I was hoping you meant :). This is also in our plans but will likely happen after the idea I showed here.

That's what I was hoping you meant :). This is also in our plans but will likely happen after the idea I showed here.

Keep the current python interactive window on .py file but make the display of output like the PyCharm Jupyter or Atom Hydrogen where input each cell has its corresponded output cell. Is that right?

Yes the idea is to have 3 ways to do jupyter like work

Current interactive window where it just appends
ipynb editor that behaves like jupyter (everything inline)
Interactive window where cells don't append unless you run a new cell

Going on vacation. Work is here:
https://github.com/microsoft/vscode-python/tree/rchiodo/edit-notebooks
Here's the minimum list of stuff left to do:

Functional tests
Arrow down not working on last line if wrapped
Keyboard shortcuts missing
? dd - delete
? insert above/below
? others?
Export to pdf
Interactive window hover cell buttons instead
Interactive window doesn't need refresh to use a textArea, just always use a monacoEditor
Cleanup UI buttons and dropdowns
Double check different themes
Weirdness around startup of server
? Timeout doesn't work anymore, just fails. May have broken the jupyter session retry logic
Change markdown editor to CodeMirror as it supports markdown previewing inline
? Monaco markdown editing isn't as good as CodeMirror
Startup server in background
? Make an option
Select multiple cells
? Make menus work on all the cells at once
? Support merge commands

Maybe also investigate https://github.com/microsoft/azuredatastudio to see how they handle the monaco editor

Will jupyter extension be compatible with new interactive window? It seems to be a jupyter notebook inside VSCode?
And debug Cell?

https://github.com/VSCodeVim/Vim/issues/3973#issuecomment-522954761
Is this implemented with a WebView?

@AeneasZhu. This new notebook editing experience is basically another way of interacting with the Interactive Window experience. In the interactive window your .py file is being used as a base for the source code and the interactive window is just to display a history of what you have executed.
This new editing experience uses the same output elements as the interactive windows, but instead of entering code from a .py file it uses a more jupyter like interface where the code elements are all inline with their output.
It shares much of its code with the existing interactive window, so yes it is also implemented as a WebView and will not initially support VS extensions like VSCodeVim (sadly for me as I use that as well).

Creating work list from Rich's last comment

[ ] Functional tests
[ ] Arrow down not working on last line if wrapped
[ ] Keyboard shortcuts missing
[ ] dd - delete
[ ] insert above/below
others?
[ ] Export to pdf
[ ] Interactive cell buttons only on mouse hover
[ ] Cleanup UI buttons and dropdowns
[ ] Double check different themes
[ ] Weirdness around startup of server
[ ] Timeout doesn't work anymore, just fails. May have broken the jupyter session retry logic
[ ] Change markdown editor to CodeMirror as it supports markdown previewing inline
[ ] Monaco markdown editing isn't as good as CodeMirror
[ ] Startup server in background
[ ] Make an option
Select multiple cells
Make menus work on all the cells at once
Support merge commands

@IanMatthewHuff  Will this feature be added in Sep update?

This is an unbelievably strong addition which I feel should be very actively promoted on the blog or in some sort of announcement. The current state of Jupyter support is acceptable, but the flexibility proposed and implemented by @rchiodo just elevates VSCODE as a whole to another level for working data scientists. I do a lot of my development in Jupyter in the browser but I switch to VSCODE for production code or for FOSS contributions. Being able to shift my entire workflow away from the browser while retaining the same sort of user interface (especially regarding fonts, compare #5256) is fantastic!
Also from an ergonomic sense, working on an ultrawide display and not having to shift the focus of the eyes from left (code/ #%% cells) to the right (Jupyter-like output) will be a huge improvement (compare #727).
Congratulations to all involved, I will try to contribute ASAP as well, just currently completely drowning in work. In that spirit, happy vacation @rchiodo !

Echoing comments above, I'm really excited for this feature to come to the Python VS Code extension! My machine learning class (and several other linear algebra/ML classes at my college) actually requires us to use Jupyter Notebooks (converted to PDF) for homework submissions. The homeworks involve submitting notebooks with code, plots, and significant amounts of markdown with LaTeX equations (markdown/LaTeX is about 70% of the notebook), so I've defaulted to the browser workflow in the interest of directly editing and re-executing markdown cells in place. But I'd love to have the direct editing experience in VS Code—I miss the IntelliSense, Git integration for versioning/collaborating on my notebooks, and not being browser-bound. Can't wait for this to ship!!

Yes the idea is to have 3 ways to do jupyter like work

Current interactive window where it just appends
ipynb editor that behaves like jupyter (everything inline)
Interactive window where cells don't append unless you run a new cell

I try vscode notebook  these days, and I still prefer the old style of python interactive windows, for the following points:

Unable to debug cell
Intellisense is not intelligent and doesn't make sense
No support for vim extension
No support for intellicode

Will the python interactive window feature the  following gif in the feature:
blob:https://www.jetbrains.com/da511e7e-d5ed-4b91-833d-e6fbc7f40847

@AeneasZhu Intellisense is being tracked by a bug:#7316, although if you have more specific feedback, that would great. It should be partially working.
Debugging cells is something we're going to release later. It's on our todo list.
VIM support is longer term. We probably need support from VS code to support loading extensions into the editor we're hosting.
However, intellicode should be working. What results did you get when you tried it?

Yes the idea is to have 3 ways to do jupyter like work

Current interactive window where it just appends
ipynb editor that behaves like jupyter (everything inline)
Interactive window where cells don't append unless you run a new cell

https://user-images.githubusercontent.com/44701446/65811893-13adc000-e1f2-11e9-8b94-0e626455f03d.gif
Is this style of 'Interactive window where cells don't append unless you run a new cell' still on the todo list?

Yes. This issue here is tracking it:
https://github.com/microsoft/vscode-python/issues/6604
Feel free to upvote it.

@rchiodo omg, I just tried the new ipynb editor out, it's UNBELIEVABLE! Many congrats to all involved!

@rchiodo the customized color theme like italic seems no longer working and also not autocompletion for () and []. Are those known issues?

Sorry @StevenLi-DS but not sure what you mean by customized color theme not working. Can you enter new issues for each thing you're seeing?
Theme support and intellisense support should be identical to before the notebook editor was created. If you're seeing a regression please log an issue.

Is there a way to enable nbextensions?

No sorry nbextensions are not currently supported. Please enter an issue, if you'd like them to be, so other people can up vote the issue.

table of contents feature for notebooks would be great

@zouhx11, please feel free to enter a suggestion for a table of contents. This way others can up vote it.

When opening an ipynb file it goes right away to the interactive mode. Is there a way to look at the raw json file as well? Sometimes you want to look at the metadata for example.

The compare changes in notebooks is a bug:
https://github.com/microsoft/vscode-python/issues/7905
To look at the actual JSON, you have to turn this setting off:
"python.dataScience.useNotebookEditor": "true".
With the setting turned off, ipynb files don't automatically open in the new editor. You can still open them though with a right click menu.

Hello,
i found about this functionality today - and it is great! It is making me consider switching my current IDE.
I have few minor GUI related issues however, which I haven't seen mentioned around here:

CTLR + Enter does not work to "run" markdown cell (while Shift + Enter does)
Navigating cells (using arrows) scrolls "viewpoint" extensively and unnecessarily. Every time I switch focus from one cell to another, whole editor scrolls up or down, even if it is not necessary for highlighting the cell (ie it is already on the screen).
Selecting the ipynb file in Explorer sidebar runs the whole process of opening json, switching to editor and so on - even if the file is the one that is already currently open in the editor. This has some side effects, like not being able to rename the file by selecting it in sidebar with mouse and pressing F2 - the sidebar loses focus, which is different behavior than for other file types.

Still, I am very thankful for this functionality and look forward to subsequent releases.
OS: Windows 10
Version: 1.39.2


Creating new angular 2 app w/cli

Please provide us with the following information:

OS?  Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)

Windows 8.1
2. Versions. Please run ng --version. If there's nothing outputted, please
run in a Terminal:
node --version
And paste the result here.
angular-cli: 1.0.0-beta.5
node. 4.2.4
os: win32 x64

Repro steps. Was this an app that wasn't created using the CLI? What change did you
do on your code? etc.
Installed angular2 cli as per website instructions.  Creating demo app:  ng new my-dream-app.

The log given by the failure. Normally this include a stack trace and some
more information.

Did not see any log files.
See attached screen dump.

Mention any other details that might be useful.
I tried running ng serve and got the error:  TypeError: cli is not a function.

The initial angular-cli appeared to install properly.  I believe the only error message I saw referred to a deprecated reference to to lodash.

Thanks! We'll be in touch soon.

ECONNRESET is usually a network issue. Can you try again and tell me if it still happens?

FIRST TRY: Just tried it again and the same error came up.  It took about 30 seconds from the time “Installing packages for tooling via npm” displayed and the error in events.js:141 with the unhandled ‘error’ event message came up.  I’m not really a full-fledged front end developer in the open source sense (more of a .NET person) so I just followed the instructions given on the angular cli website for creating a new angular 2 project (ng new my-dream-app) and hoped for the best.
SECOND TRY:  I thought I’d play around a bit and upgraded my version of node.js from v4.2.4 to 4.4.5 and it seemed make things better – no error.  It took a long time to download all the files and there does seem to be a lot of files.  When I checked the folder holding my-dream-app it had 41,462 files and 7,367 folders.  Seems like it had everything and the kitchen sink.  I did run ng serve and I got the “my-dream-app-works” in my browser.  Do the number of files/folders seem about right?
From: Filipe Silva<EMAIL_ADDRESS>Sent: Tuesday, May 31, 2016 7:04 PM
To: angular/angular-cli<EMAIL_ADDRESS>Cc: oncalldev<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [angular/angular-cli] Creating new angular 2 app w/cli (#960)
ECONNRESET is usually a network issue. Can you try again and tell me if it still happens?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/angular/angular-cli/issues/960#issuecomment-222856880 , or mute the thread https://github.com/notifications/unsubscribe/ADtR-j7i0howbERXe3Ao-6cKGS_FZylpks5qHMxggaJpZM4Ip3CJ .  https://github.com/notifications/beacon/ADtR-lT5v6-JgGUuvGnSQry0xo7teMeGks5qHMxggaJpZM4Ip3CJ.gif

Yeah the insane amount of folders/files is the norm in the node world... In this specific case it's worsened by the fact that CLI projects include angular-cli itself as a dependency for backwards compatibility.
Regarding the node version, I should have noticed that earlier - we recommend node 5 and up and that's how we do our CI tests. Regardless, I'm glad that it now works for you!


ForkOptions is now abstract in Gradle 8.0

Expected Behavior
ClojureNRepl should start on Gradle 8+
Current Behavior
It fails with the following:
* What went wrong:
An exception occurred applying plugin request [id: 'dev.clojurephant.clojure', version: '0.7.0']
> Failed to apply plugin class 'dev.clojurephant.plugin.common.internal.ClojureCommonPlugin'.
   > Could not create task ':clojureRepl'.
      > Could not create task of type 'ClojureNRepl'.
         > org.gradle.api.tasks.compile.ForkOptions

Caused by: org.gradle.api.tasks.TaskInstantiationException: Could not create task of type 'ClojureNRepl'.
        ... 239 more
Caused by: java.lang.InstantiationError: org.gradle.api.tasks.compile.ForkOptions
        at dev.clojurephant.plugin.clojure.tasks.ClojureNRepl.<init>(ClojureNRepl.java:30)

It looks like the offending call is here - ForkOptions is an abstract class now.
Looking at the 8.0 release notes, there's a section on some classes being made abstract, if that helps.
Steps to reproduce

Clone sample-clojurephant-clj-app
./gradlew wrapper --gradle-version 8
./gradlew clojureRepl

Environment

Build scan URL: N/A
clojurephant version: 0.7.0
Gradle version: 8.0
Java version: java-17-openjdk
OS version: Manjaro

Cheers @ajoberstar!

Thanks for reporting this!

I still get this issue on both 0.8.0 beta 1 and 2 with gradle 8.3 (also on 8.1, but tried updating gradle as well in the process) and Java 17. Am I perhaps using a wrong version?

The same problem in gradle 8.4


Add leading hex zero to addresses

@aviateur17 Hey so this is something that has stumped us both before, but your irrigation code gave me an idea. Here's the proof of concept I wrote:
  Serial.begin(115200);
  uint8_t x = 2;
  Serial.println("It's 0x" + String((x < 16) ? "0" : "") + String(x, HEX));

I applied it to the gateway's ESP-NOW functions and it seems to work. However, when I started expanding it further I realized there are a lot of spots where we'd use this, especially in LoRa. I think we can probably transform it into a reusable function to keep things neat. What do you think?

That looks excellent.  We could use a macro like the ones at the top of fdrs_lora.h
#define LZ(a) (" 0x" + String(a < 16) ? "0" : "") + String(a, HEX))

Something like that and then just insert LZ(5) or whatever in the code
Great idea!


Improve definition task and global argument prediction

These are improvements to the model definitions and global argument prediction:

(Optional) unit normalize embeddings.  I do this correctly by adding it as a constraint to the Embedding.  If I just normalize the embeddings when I use them, then the layer normalization brings the embeddings to 0.0 and eventually I start dividing by zero.  This keeps them in place.
(Optional) Add cosine similarity as source of logits for global argument prediction.  This includes normalizing the key and query and also dividing the cosine similarity by a learned temperature parameter which is common.  Note, I chose to normalize the embeddings since we may want to try cosine similarity with embeddings which are not normalized.  I guess a not too difficult future optimization would be when creating the LogitsFromEmbedding object to add a parameter stating if the embeddings are already normalized.
Add local and global arg heads.  This way the local and globals get there own keys.  They are just dense layers that branch off a common argument head.
Change definition loss to use norm squared instead of mean squared error.  It is equivalent, but isn't effected so much by hidden layer size.
Change default final reductions to mean instead of max.  Changing the final reduction from max to mean makes a huge improvement.  Max seems to make the definition task and tactic task compete against each other.  As one gets better, the other gets worse.  It eventually overcomes this competition, but it takes a long time.  It seems better with mean.
I also changed the reduction after each convolution from max to mean.  I don't know if this is better, and I'm still experimenting a bit, but in principle we have some graphs with multi-edges, and max can't distinguish a multi-edge from a single edge.
Increase definition task to 500.  It previously was 100.0 using mean squared error, which would be 100/128 = 0.78 using the norm squared loss.  So we are increasing it by over 500x.  This is made possible with the above improvements.

There also are a few internal changes to the predict code:

I moved a lot more of the functions related to embeddings (like enlarging the embedding table) into the embedding class itself. The reason was that as I made changes to the embedding object it was hard to keep track of where else it was used in the code and if I needed to make changes there.  It will also help facilitate future experiments where I replace the embeddings in the code between epochs.
Similarly, I moved some of the functionality of the LogitsFromEmbedding class back into the class.

More notes:

I haven't merged the previous PR this is built on top of so this PR is a mix of both.  It would be easiest to just look at the most recent commit.
I still need to test that the predict server works correctly.
I also want to do a full scale run of this model.

I walked Mirek through the code and he gave his approval as long as I checked that it still works in the server.  I fixed a bug which prevented the saving all the weights to checkpoints.  I also used prints to ensure that the embedding table was still being updated correctly in the server.  I also added an initializer to the embeddings so even without training they are on the unit sphere.  It's ready to merge.


Implement privileged database schema user.

For assigning and closing issues different user has to be used in order to ensure in order to defend end users from themeselves by restricting their access to sensitive data.

Developer decided to generate database users dynamically for each party along with permissions.


🐞 [Bug]: TF Robot(dashboard applications) docs redirecting to tf robot sdk guide

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
Dashboard
What happened?
TF robot docs from the dashboard redirects to the sdk (how to install and configuration )
Steps To Reproduce
No response
which network/s did you face the problem on?
Dev
version
2.5
Twin ID/s
No response
Node ID/s
No response
Farm ID/s
No response
Contract ID/s
No response
Relevant screenshots/screen records

it should redirect to another docs for using the machine or the deployment in the dashboard section not the developers section
Relevant log output
.

may this docs also containt tfcmd , tf-grid-cli ,tf robot which are also installed on the vm

@khaledyoussef24 please open an issue in info_grid too

Blocked on https://github.com/threefoldtech/info_grid/issues/563

@zaelgohary
Good to go on the manual side.
Amazing addition to the Dashboard guys! Will clearly explore that one.

@zaelgohary Good to go on the manual side.

Will update the URL once the PR is merged.

@zaelgohary is it updated ?

@zaelgohary it is not updated yet

Done.


Backend - Saving and loading

Add saving and loading of the game states. Could simply modify JSON data but that's impractical for large data sets so perhaps store a JSON file of changes instead? Main data would be loaded as normal and then the save data used to overwrite the necessary variables to their new states.

Done, player stats and area modifications are now saved and loaded to player specific files. See commit 55c1ced2dfed7fe0e66624a0494f21a0079c430d.


Don't include tag if not not needed

https://github.com/chafey/dicomParser/issues/43
{
untilTag: {
tag: '',
include: false
}
}
untilTag option now takes object like above or string like before

Is c55297f necessary? You can already tell if a byteArrayParser is big or little endian by comparing it to the dicomParser.bigEndianByteArrayParser and dicomParser.littleEndianByteArrayParser objects directly.

Probably didn't need to push that to this pull request, however, if the dicomWriter that I wrote is to understand whether data is to be handled with little endian or big endian without having to look at the data itself, then it's a really nice helper. The dicomWriter, then, doesn't need to require the dicomParser, so that it can compare the parsers.


Various Coverity warnings

Hi,
I analyzed a project that uses TLSe using Coverity's free service (Scan), and it found a number of issues in TLSe that I thought I'd pass on. (I can invite you to the project if you wish to have a look directly in the system yourself; the error messages are somewhat more detailde.)
The ones that I found immediately relevant (line numbers are at bdbf5ecc):

tlse.c:9685: setsockopt() to set TCP_ULP to "tls" doesn't check the return value.
tlse.c:1733: Various _done() functions are assigned to err (indicating they can actually return an error?), but never actually checked.
tlse.c:5237: “key_size = strlen(default_dhe_g);” is unreachable code; it cannot be reached whether default_dhe_g is nullptr or not. I assume the || should have been && earlier?
tls_build_certificate(): This function returns early if !all_certificate_size, yet checks it again immediately below (causing more dead code).
__private_tls_prefer_ktls(): context is dereferenced, and only then NULL-checked later. Either the order should be switched around, or the NULL check is redundant.

In all, there are 20 reported issues, of which I would assume there's one or two issues that could potentially be real user-facing bugs. Whether the dead code is important to you or not is a different story, of course :-) And then, there are probably some straight-out false positives.

Hello,
Thanks for reporting this. I've fixed the 5 issues.
Thanks,
E.

Thanks. I'll have a look and see if I can send a pull request for the rest.

I created a PR fixin all GCC 7 warnings, and most Coverity warnings; feel free to grab what you want from it. There are some remaining issues reported by Coverity that I believe are false positives.
The main issue remaining is that _private_tls_update_hash() doesn't check the return value of the hash functions (like sha384_process()). It probably should, since the rest of the code does, but it doesn't seem to have a way of returning errors.

Thanks for the PR. I've just replaced if (err) break; with TLS_ERROR(err, break). Looking at tomcrypt's *_init function, I see that always returns CRYPT_OK (0). I think that checking every *_init is a little defensive, but I think is the clean way to do it.
Thanks for contributing!


Error message properties added to swift kit.

👍

I find the name UIError is a bit misleading, since the errors are not related to creating/managing the UI


Unicode and the telegram token

I am not sure if this is exactly an issue but I wanted to point it out, since I saw related problems in old issues and took me some time.
When builiding a new telepot bot, if you imported the token from, lets say, JSON, the bot will be instanced normally, but every call will cause it to break. So, if the token is unicode, the bot will break in every request to the telegram API because it gets concatenated to strings. Using python 2.7.

Using Python 2.7, this works for me:
>>> import telepot
>>> bot = telepot.Bot(u'12345678:TOKEN')
>>> bot.sendMessage(999999999, 'hello')

I explicitly made the token unicode, and sendMessage works fine.

This is what I did:
>>> import telepot
>>> bot = telepot.Bot(u'12345678:TOKEN')
>>> bot.sendMessage(10809372, "¡hello!")
UnicodeDecodeError: 'ascii' codec can't decode byte 0xad in position 83: ordinal not
in range(128)
>>> bot.sendMessage(10809372, u"¡hello!")
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc2 in position 83: ordinal not
in range(128)

If the message has non-ascii characters it doesn't work, and even if the sendMessage string is set to unicode, still not working.
However if the token string is a normal byte string, it does work, with unicode and byte strings (however normal bytes strings have to be properly encoded or so).

Im trying to sendMessage()
msg = 'é' msg = unicode(msg, 'utf-8') telegramBot.sendMessage('@testBot', msg)
but always get this error:
UnicodeDecodeError: 'utf8' codec can't decode byte 0xe9 in position 49: invalid continuation byte
am I doing something wrong ?

Well, these all work for me, both on Python 2.7 and 3.5, on Raspbian Jessie:
bot.sendMessage(111111111, u'香港')
bot.sendMessage(111111111, u'¡hello!')
bot.sendMessage(111111111, u'é')

I really have no idea why those decode errors occur :thinking:

I fixed it forcing the default encode in this way:
# encoding setup reload(sys) sys.setdefaultencoding('utf-8')

I had the same problem,
I solved it by saving the script itself as an utf-8 file and not as standard as ascii

I had the same problem (Python 2.7). Bot instantiates OK and telepot methods like sendMessage() and getMe() work fine, but sendPhoto() runs into the encoding errors when the bot token is read from json e.g. "Error when sending the photo is: UnicodeDecodeError: 'ascii' codec can't decode byte 0xff in position 445: ordinal not in range(128)".
The solution was to use the str() method on the token when instantiating the bot e.g. bot = telepot.Bot(str(token)). Thereafter, the sendPhoto() method works with file-like objects as obtained by open().

This may not be relevant. I am writing this in case someone find it useful.
If you use sendPhoto() or similar methods to send files and you obtain the file-like object using open(), remember to give the rb flag, e.g. open(path, 'rb'). Forgetting the b may result in some kind of encoding error, similar to the ones discussed in this thread.


Change from iconUrl to className

Use css (which is relative to css) for
images. This will stop icons from being missing
and won't require a absolute path since everything
is in a relative path to the css file.
This should fix the issue #14.

love it, thank you and sorry for the delay


How to use functions correctly?

Hi, your project looks very interesting, but unfortunately I don't understand how to use it. Every time calling the function from your API (e.g. client.light.setColor()) do I need to use LIFXClient.connect()? Or maybe just use the connect function only once?
LIFXClient.connect(host: .ipv4(IPv4Address("<IP_ADDRESS>")!)).then { client in     return client.light.setColor(color: .orange) }
I also don't quite know how to get for example client.light.getPower() properties.
Sorry if the questions are trivial, but I would like to understand how to use your library.
Thank you for help.

Hi, yeap you'll need to connect to the light first before sending messages to it.
It might be useful to get familiar with PromiseKit if you have not used it before.
These operations can be chained together, or you can make the connection first and then send messages to it later.
I've just open-sourced the mac app I've been building, perhaps have a read and see if it makes sense: https://github.com/dclelland/3LED/blob/master/3LED/AppDelegate.swift

In this app, the AppDelegate fetches a list of connections and then stores them in the NSMenuItem.representedObject properties. When a menu item is selected, the connection is retrieved from the menu item and passed to a window, which can then call client.light.setColor etc without first having to create a new connection.
Hope this helps... let me know if you have any more questions.

Thank you so much, I managed to deal with it! Your library helps me a lot in my iOS project :).


Move selected line up and down in linux default conf

'ctrl-shift-up': 'editor:move-line-up'
'ctrl-shift-down': 'editor:move-line-down'
For some reason this is default for OSX but not for Linux, while Sublime's default configuration for this keymap in Linux and Windows is indeed moving the line up or down

Did you close this by accident? Is this something you'd still like to see merged in?

Yes I did, sorry I created a new one


feat: run validation & lint automatically on create

Description
This removes the duplicated checks between the schema and validate, and runs error linting on create. This introduces a new type packageError to track errors and warnings through packager. This type replaces packager.warnings, so this PR touches many files, however besides lint and create behavior is the same.
Related Issue
Relates to #2485
Checklist before merging

[ ] Test, docs, adr added or updated as needed
[ ] Contributor Guide Steps followed

This will change how the errors are presented to users to look like what happens during zarf dev lint rather than a normal error message

I'll pass through this again in the morning, but lots of questions ✨

Broke this out into #2658 and #2660


Fix module path in go.mod

Installing it via go install now gets:
go install  github.com/jameswoolfenden/ghat@latest

go: downloading github.com/jameswoolfenden/ghat v0.1.5
go: github.com/jameswoolfenden/ghat@latest: version constraints conflict:
<EMAIL_ADDRESS>parsing go.mod:
	module declares its path as: ghat
	        but was required as: github.com/jameswoolfenden/ghat

Thanks for taking the time to fix it!


netrc should use atomic file replacement

Currently the library writes directly to the file that was opened. To avoid corruption issues, it should write to a temp file close to the real file and use File.rename to atomically replace the old file.
This relates to #1; the file opened with Netrc.read should probably not be replaced until it's verified the contents will be valid.

Yeap fixed that in the pull request.


Feature Request: Change font size, style, etc. in the composer

Can I change font size for my email with style?
My colleague got my email and suprised "letters are too big."
I just request as the regular HTML email composing option.
Glad if I could see the preview when it's arrived.

Hey @F7  - we'll be making the font size a bit smaller in the next release. It seems it was often just a bit too big. Stay tuned!

@bengotow Cool. Looking forward to the release!

Does this mean we will be able to determine the typeface and size ourselves soon?

@chriscostuna Actually my request was not to decorate my text sent, but just make my text sent smaller when it's shown to reader.
As confirming that achived in the latest build, now I can use this again and start sending emails to other people.
@bengotow Thanks for the update. Can I close this issue because I confirmed that my request has already been supported?

Merging into #1087!

Hi, I just stumbled here while I was also searching for solution. I am in windows and I did try other method of scaling workspace in config.json but whenever I click emails the font would resize to orginal.Luckily for me I use my monitor on 125% dpi. On windows 10 you can force nylas.exe in compatibility tab of its executable Properties to override the dpi scaling for the application, finally worked like a charm.


Finer control of parameters

Please consider to add other GUI elements that allow a finer tuning of parameters. Using the finger sometimes causes large jumps between the values which is quite tedious in some cases in particular for filters which are quite sensitive.

You can tap the value and enter it by keyboard 👀

OK, I didn't see that. Thx.


JoinPaths: Prepend ${prefix} when dir is inside the prefix.

Note I started off with the copy merged in jsoncpp.
https://github.com/open-source-parsers/jsoncpp/commit/5be07bdc5e2d5b7715ecbc73749af3e625674dcb
At least on unix systems there are three cases for generating a pkgconfig file with cmake.

CMAKE_INSTALL_<dir> is relative and should have ${prefix} prepended.
CMAKE_INSTALL_<dir> is absolute and not inside CMAKE_INSTALL_PREFIX, it should be used as is.
CMAKE_INSTALL_<dir> is absolute, but it is still inside of CMAKE_INSTALL_PREFIX, it should have ${prefix} prepended.

This fixes the third case.
Also there is a currently open PR for FAudio with this.
https://github.com/FNA-XNA/FAudio/pull/228

I missed one case where CMAKE_INSTALL_PREFIX is equal to CMAKE_INSTALL_<dir> and updated the commit to fix it.

This makes sense for pkg-config-specific use cases but it is not suitable for a generic join_paths function like this one – imagine someone actually trying to join something to install-prefix.

Yea, I failed to take in account anything other than pkgconfig files in this PR...
I updated this PR again to rename the function join_paths_for_pkgconfig and left the original function alone. Would you prefer them to be in separate files or is this fine? It seems to make it more complicated if the user requires two functions to set the pkgconfig file paths.

I think having them in a single file is okay, as is the name.
Thinking about the API, I do not think it makes much sense to strip the prefix from the third and successive arguments. Maybe we could just have the prototype function(join_paths_for_pkgconfig joined_path first_path_segment second_path_segment) that would strip prefix from second_path_segment and then return join_paths(${first_path_segment} ${second_path_segment_possibly_stripped} ${ARGN}).
We could even make what is stripped depend on the ${first_path_segment}, for \${prefix} it would be ${CMAKE_INSTALL_PREFIX}, for libdir ${CMAKE_INSTALL_FULL_LIBDIR}…
Lastly, once we finish designing this, it would be nice to add at least a single sentence comment explaining what it does. Something like

Joins path segments passed as arguments into a single path returned as the first argument. When an absolute path is encountered, the previously built path is discarded and building a new path begins based on that argument. When the first input argument is a pkg-config variable reference and the following one is absolute path starting with a GNU directory path corresponding to that variable, they will be joined together as if the absolute path was relative to the GNU directory path.
Examples:
join_paths_for_pkg_config(out "${prefix}" "${CMAKE_INSTALL_PREFIX}/foo/bar") → "${prefix}/foo/bar"
join_paths_for_pkg_config(out "${libdir}" "${CMAKE_INSTALL_FULL_LIBDIR}/foo" "bar") → "${libdir}/foo/bar"


GH-1444: Observablility Documentation, Polish

Refactor for latest snapshots; add documentation generation. Use NOOP registry and supplier for context.
Also fix class tangles.
Resolves https://github.com/spring-projects/spring-amqp/issues/1507
Disable auto doc generation and polish manually.

Green finally; will address the logback and web client issues after merge.


Access Violation

librealsense
2.53.1 RELEASE

OS
Windows

Intel RealSense Viewer / Depth Quality Tool has crashed with the following error message:
Unhandled exception escaping from a worker thread!
Error type: Access Violation!

Hi @yiying66   Please first check whether permission to access the camera hardware and permission for applications to access the camera are both set to 'On' in the Windows camera privacy settings.  This can be done using the instructions at https://github.com/IntelRealSense/librealsense/issues/3381#issuecomment-503424789
If you find that both settings are already On then please try completely uninstalling and reinstalling the two RealSense camera drivers (Depth and RGB) in the Windows Device Manager interface using the instructions at the link below.
https://support.intelrealsense.com/hc/en-us/community/posts/4419989666323/comments/4431239847443
In Step 4 of the instructions, if there is not a tick-box to select then skip to Step 5.

Hi @yiying66  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


Message type optimizations

I think wayland_commons::wire::Message is a bit sub-optimal right now.
Currently Vec is allocated for every message, while judging by wayland.xml max number of arguments is 8. To remove unnecessary allocations we can use SmallVec for args, though we will have to decide the array size for SmallVec, e.g. 3, 4, 6, or 8.
Argument always heap allocates for Array and String types, not only it causes needless allocations and data copies, but also makes Argument bigger (right now it uses 32 bytes). Instead of copying data we could point to the origin buffer instead, i.e. we will add a lifetime Argument<'a> bound to the origin buffer (and consequently to Message as well). The simplest option will be to use &'a [u8] and &'a CStr, but it will bring Argument size to only 24 bytes. Instead with a bit of unsafe code we can store *const u8 and phantom data (to track lifetime) inside two opaque types which will dereference to &'a [u8] and &'a CStr respectively. It will bring Argument size to 16 bytes on 64-bit machines. I think adding lifetime to Message shouldn't be a big problem, as IIUC they are almost immediately consumed inside event loop.

Currently Vec is allocated for every message, while judging by wayland.xml max number of arguments is 8. To remove unnecessary allocations we can use SmallVec for args, though we will have to decide the array size for SmallVec, e.g. 3, 4, 6, or 8.

Agreed, most messages have less than 4 arguments in general, though 8 is not some hard limit, other protocol extensions can definitely have messages with more arguments, nothing forbids that. And using SmallVec here seems to be a simple enough change.

I think adding lifetime to Message shouldn't be a big problem, as IIUC they are almost immediately consumed inside event loop.

Well, yes and no. Server-side this is mostly true, but client side it isn't. Messages are first stored in temporary buffers for each event queue, and there is no guarantee that the messages will be consumed from these event queues.
In a more general view, I'm not very clear about what would be gained from such optimizations. In contrast, the C library also does a lot of copying around and allocations, and it does not appear to be such a bottleneck that anyone has complained about it. So I'm not willing to significantly increase the complexity of the implementation for small hypothetical performance gains.

Messages are first stored in temporary buffers for each event queue

Hm, then one way to solve this will be to introduce in addition to the current owning Message a borrowing MessageRef<'a>, with an ability to easily get one from another. But if you'll judge it's too much trouble for too little gain, feel free to ignore this proposal. It's just that I found such copies and allocations a bit wasteful, that is all. :) Also making Argument twice smaller will make SmallVec smaller as well. Maybe having statistics for passed messages (to determine number of strings and arrays flying around) for common usage patterns and applications could be useful.

Reducing the allocations using SmallVec will likely be a significant gain indeed, as there can be a relatively high frequency of exchanged messages at times.
Regarding copies though, in practice messages with String or Array arguments are rather rare in proportion to be honest. A few of them are typically exchanged during the initialization of a client, and occasionally during the run (for example when a window gains focus), but that's about it. So I don't think there is much to be gained here honestly.
Though I'll seriously consider the issue if somebody ever reports how this is a bottleneck for their specific app, but other than that I prefer to keep the implementation as simple as possible (which is already not very simple :confused: ).


Trojan detected : wacatac.b ml

Defender block the download cause trojan detection for "wacatac.b ml".
Would you mind to tell more ?
(I know turning off the anti-virus is the solution)

https://www.microsoft.com/en-us/wdsi/threats/malware-encyclopedia-description?Name=BrowserModifier:Win32/Wacatac.B!ml&ThreatID=291283
As I understand it's due because I can "modify your browser settings".
I think it's because my extension need the storage permission to save the configuration ?

You can see all the permissions I need here https://github.com/yoannchb-pro/MoodleGPT/blob/main/extension/manifest.json.

Top I tested it on MAC and works perfectly you are a genius !
However it didn't work on a mock up protocred moodle exam. Do you if there is a link with proctorio ?
Furthermore, considering that you have managed to solve the problem of "how to communicate with GPT without it being noticeable," would you be able, based on this extension, to make it read the question, answer it in the background (as is already the case), and ensure that the correct answer to the Moodle quiz appears on a non-suspicious third-party website?
The exam is open book but proctored, so we can access any website. Could you make sure that the answers are hidden among other content on a random website or any other non-suspicious means where we would just need to know where to look to find the correct answer?
By omitting the hover select step, keeping the hidden GPT reasoning, and (new) displaying the answers somewhere (website, Word document, text, or hidden on a fake Google search results page)?

As I understood you want the extension to be able to display the good answer on other pages for example (just a little bit hidden in some kind of text or somewhere else )?

Exactly ! And maybe a way to hide the extension from the chrome list :)

Actually you can hide the extension by your self by clicking on the eye icon (can make it directly in the code). What I can do if you want is a new option that make the response always go in the clipboard so you should able to paste it on any page you want ?
(it's an other account)

Would be perfect ! Do you have a link for donations ? If it works with proctorio, let's go !

Btw what is protorio ? And sure in the next update I will notify you want and add a support link. I will also may be post it directly on the chrome webstore (just need to pay and create an account and it will be done).

Thank you very much!
Can you estimate when you will be able to post the update ?
Maybe putting on chrome would make it very popular and by the way reduce the life of the extension with incoming patchs or school blacklist. Proctorio is a proctor system to record screen during exams.
Yes, let me know, me and my friends totally agreed to pay you for your work (the 2 next weeks are our exams weeks).

I also have exam comming. I’m gonna try to work on it this week but I have some problem with my api key due to too much request xD. I’ll notify you when it’s done. You are not obligate to give money haha I love when people contribute by giving ideas :). I’m gonna try to publish it on chrome store and to add a mode for autocomplete, clipboard and question to answer (the question will be replaced by the answer)

Ok it should be done. I published the update and I'm waiting for chrome to review my extension and it should be available on the chrome webstore.

 Thank's for your work !
Any informations about if it works during proctored exams ?

Too complicate to know if I can’t try it on my own side :/. If you try and it work tell me. But it should normally work.

https://chrome.google.com/webstore/detail/moodlegpt/fgiepdkoifhpcgdhbiikpgdapjdoemko?hl=fr


Add deployment_id parameter for azure

This PR adds support for a deployment_id parameter for azure. The "deployment_id" parameter overloads the engine parameter. (Similar to as "model" overloads the "engine" parameter for openai)
Parameter checks and exceptions are refactored and moved into the EngineApiResource base class.

@t-asutedjo is this ready for review?

@t-asutedjo is this ready for review?

Yes

@hallacy This seems complete, could you please have a final look and merge it? Thanks

Nice!  Thanks for pulling the common error message out to the base class!


IMAP session

Write class that establish connection via imap protocol and provides interface to interact with it.

Implement after #7

Solved


refactor(Label): update props to the latest specs

Shorthand
Currently, the Label allows shorthand props icon, image, and text along with children.  These should be disallow()ed via propTypes and early returns put in place to render one or the other.
text
This prop should be renamed to content.  See #391.
link
This prop renders as an a tag.  It should go away as we now have augmentation  instead.
Docs & Tests
Docs and tests should then be updated for all usages.

Somewhat related - I'm having an issue with this line which adds the empty class: https://github.com/TechnologyAdvice/stardust/blob/master/src/elements/Label/Label.js#L41.
I'm trying to set a circular label with the text shorthand prop and it's getting the empty class. This is because that logic is only based on children:

// Currently

11

// To make it non-empty, needs children
{count}

11

I could see empty being set when there's no content of any kind, although I think I'd prefer to just set it manually:
// Better (but still not completely intuitive):

// Ideal:

Yep, that's bad.  See https://github.com/TechnologyAdvice/stardust/issues/485

Actually, let's just do this all at once, updated description above.

Fixed in #486


condense

Im gonna need a little more detail on what there is to condense

@meckleunn please help :(


note for Apollo

Just done it in https://github.com/dotansimha/graphql-code-generator/commit/9f0662a798f9cb1bb28c96623d0347c212968913#diff-2e4db0927cea96d01402e7d186a1451aR134 :) thanks!