Documentation: es6 Map and Set in the state are not documented

I did not find in the documentation how one should or should not use things like es6 Map and Set with Redux.
It seems like a new map should be created every time when we 'modify' it in the reducer, but I'm not sure that it is correct.

I don’t think it’s a good idea to use them in Redux because they are optimized for mutability. If you want to have something efficient with similar semantics, consider using Immutable.js.

Can this be mentioned somewhere in the docs, that it is not a good idea?

Yes, I'm not able to use ES6 map it seems it just deal with Objects, and array only.

You can use Maps and Sets as state values, but it's not recommended due to serializability concerns.  In addition, the built-in combineReducers utility expects to work with plain JS objects, so if you use a Map as part of your state tree, you would need to use other reducer handling instead of combineReducers.

FYI, I have implemented generic map and set reducers using plain JS objects in redux-data-structures:
https://github.com/adrienjt/redux-data-structures/blob/master/src/reducers/map.js
https://github.com/adrienjt/redux-data-structures/blob/master/src/reducers/set.js

@adrienjt can you add a usage example gist for the map type above?

Not sure if this is what @markerikson means by "serializability concerns", but for example if you try to do things like export/import your entire state with json, any maps in your reducer state will not properly be initialized.  The basic issue is outlined here: http://2ality.com/2015/08/es6-map-json.html

@markerikson Maps are to "heavy" for me. Too much code needed to handle uniquenes and as you said serializability concerns... I am reflecting on the idea of using Sets to help maintain my states. In this example https://redux.js.org/usage/structuring-reducers/normalizing-state-shape allIds for example would be a Set.

@spedy Actually, Redux Toolkit's createEntityAdapter maintains that "IDs list" as an array, and already does all the work of updating it for you :)  Use that:
https://redux.js.org/tutorials/essentials/part-6-performance-normalization#managing-normalized-state-with-createentityadapter

I don’t think it’s a good idea to use them in Redux because they are optimized for mutability. If you want to have something efficient with similar semantics, consider using Immutable.js.

serify-deserify reversibly transforms unserializable values into serializable ones, and back. It natively supports BigInt, Date, Map, and Set, and you can easily add support for custom types.
The package includes a Redux middleware that applies the serify transformation on the way into the store. Wrap retrieval functions in the deserify function in order to transform the results to the appropriate type.
Full disclosure: I am the package creator!


electron app alerts user that it is an unsupported browser

electron app alerts user that it is an unsupported browser
https://github.com/cryptonomex/graphene-ui/blob/e76a9915e3295af76fd1171883df9e8c461ce187/web/app/App.jsx#L115

My bad, always forget about the light client!


Description popup opens even if option in Enter/Leave Giveaway Button is disabled.

Not much to explain, neither are there many steps to reproduce: 2.11 is enabled and all its sub-options are disabled as they have always been for me. But the popup pops up.
This is the same issue as issue 381
Screenshots

ESGST Version: 8.3.12
Browser + Version: Opera 60.0.3255.84

I'm sorry. Why was this labeled invalid and closed outright without any questions or anything? What is invalid about it? The issue still persists for me.

The option no longer exists. You can use filters to prevent certain descriptions from showing.


Rev:r is 'ones-based"

Using Rev:r you cannot get a version 1.0.0
It will always start at 1.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a57f8545-bb15-3a71-1876-3a9ec1a59b93
Version Independent ID: 28c87c8d-c28d-7493-0c7c-8c38b04fbcd7
Content: Run (build) number - Azure Pipelines
Content Source: docs/pipelines/process/run-number.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

It looks like you've encountered an issue with the product itself, rather than an issue with the documentation. To make sure it gets in front of the right people, please submit your bug here.


Fix: Improve build speed

If a code change is made, permissions on helpers is set each time. For some reason, this is slow and takes 3-5 seconds (on Gitpod workspace)
Doing this trick in Dockerfile ensures that helper setup is only made when there is a change to helper files.
Before:
https://github.com/open-runtimes/open-runtimes/assets/19310830/b659effb-c62b-4ce9-bbd8-0535c3e1a973
After:
https://github.com/open-runtimes/open-runtimes/assets/19310830/4d4fba3d-7978-4321-9cc6-4c05d21b4564

Dont merge. It might be causing this.


Poor results in verification

I'm trying to verify faces using compare.py.
I've trained a model on a custom dataset that is consisting of frames from selfie videos. Here is its training loss:

However, when I use the trained model in compare.py and get a new image(with the different situation from the dataset images) the distance between the training images and the given image seems weird.
Images from the same person have L2 distance more that 1.1 and images from different persons have the distance less than 1.1.
Is it because of the overfitting in training? Or what is happening?
Thanks.

Do you trained the model using softmax loss ? I got a poor result too. Now I wonder if the softmax loss can't give a same performance as triplet loss on verification by L2 distance.

@heiwushi Yes, I trained the softmax loss.
Have you tried the triplet loss on verification? Any improvements on results?

@modanesh No, I haven't.

@heiwushi the pretrained model requires permission, how did you get that
20180402-114759

@whSwitching give some information regarding "20180402-114759".
What is the "20180402-114759" model? What it's been trained on?

@modanesh  home page  pre-trained-models
https://github.com/davidsandberg/facenet#pre-trained-models

@modanesh How did you train the network by using you custom datasets? did you have use finetuing?

@rain2008204 Yes I finetuned a model which was trained on MS-Celeb.

@modanesh， how did you finetune on your dataset?? I have been recently using this pretrained model to train my few-shot dataset of 43 persons, but i always got the error of classes not matching. maybe because my dataset size is 43 while MS-Celeb size is 10575. I'm not familiar with the saving and restoring models in TF, I want to restore part of the parameters and to finetune on my small dataset. So would you please give me some instructions about your finetune step??
Thank you~~~

@Victoria2333 Look it up here #139

@modanesh Many thanks, It worked, But when I changed the code to this:
all_vars = tf.trainable_variables()
var_to_restore = [v for v in all_vars if not v.name.startswith('Mixed_8b')] in inception-renet-v1
saver = tf.train.Saver(var_to_restore)
it happened:
invalid argument 2: size '[16x8x8x8]' is invalid for input with 2048 elements at /pytorch/aten/src/TH/THStorage.c:41.
And i found that 'Mixed_8b' corresponds 128 tensor, but 'Logits' corresponds 512 tensor. I don't know why the tensor size is different.
Could you please help me with this?Thanks

@modanesh，@rain2008204 But the accuracy finetuning on my own dataset is very low, i don't know why.
I run:
python src/train_softmax.py --data_dir /home/han/facenet/train_face_160 --image_size 160 __embedding_size 512 /home/han/facenet/src/pretrained_models/20180408-102900/model-2018408-102900.ckpt-90
Here is my code:
all_vars = tf.trainable_variables()
var_to_restore = [v for v in all_vars if not v.name.startswith('Mixed_8b')] in inception-renet-v1
saver = tf.train.Saver(var_to_restore)
And I freeze the created ckpt file to pb file which is loaded in my_classifier.py to train and test but the training accuracy is 1 while test accuracy is about 0.4...
Could you please help me with this?Thanks

@Victoria2333 I guess your model is overfitted.

I think it too, but i also doubt the finetune method, cause the variables shouldn't be dropped ,instead just the weights need to be finetuned rather than initialized??

Did you try it with a new identity that's different from LFW? Since it's trained in softmax, it can only recognise the identities found in its training set (LFW).

Did you try it with a new identity that's different from LFW? Since it's trained in softmax, it can only recognise the identities found in its training set (LFW).

i think the softmax loss helps network learn how to extract features of different people, so if the model is pre-trained well , it can also extract new identity's feature instead of recognising identities in dataSet

Did you try it with a new identity that's different from LFW? Since it's trained in softmax, it can only recognise the identities found in its training set (LFW).

i think the softmax loss helps network learn how to extract features of different people, so if the model is pre-trained well , it can also extract new identity's feature instead of recognising identities in dataSet
Hi tangchen, can you elaborate on this? Because if you train it on softmax, the embeddings of different features won't have any guarantees on how they are different. For example, imagine training a dog-cat classifier with softmax with a 128D embedding before the final layer.

Giving it an image of a hyena or a penguin will probably give it a different embedding than a dog say, but comparing the two embeddings by L1 or L2 probably will not give any meaningful interpretation.


Add License terms

What is the license? MIT?

+1

what would you like it to be?

MIT lisence is fine. https://opensource.org/licenses/MIT


ci: only generate indexes for stacks being built

Checklist:

[x] Read the Code of Conduct and Contributing Guidelines.

[x] Followed the commit message guidelines.

Makes prefetch optional via an environment variable (default is off) - On for travis builds
Generate indexes for only built stacks i.e. no need to generate experimental index when only incubator stacks are being built
Only add repos (for appsody repo add) for stacks being built.

Related Issues:
Fixes: #354
Related to: #355

There should be no TRAVIS specific logic in package.sh. That needs to be corrected.

Closing this PR as this might is solved in a different way by appsody stack package for stack developers.  We can revisit this when the ci has been updated to use those.


Finish the router re-write complete with tests.

The router was the first part of gophr that was ever built. It has started to show its age. It is time to bring it up to code with the rest of the codebase while correcting some issues that go get was having with gophr.pm urls.

Question. Now that we've faced our first real issues with router + sub versioner in prod I think it might be time to address how to reproduce go get properly in dev. Currently we don't properly sub-version all packages in a repo. Meaning it starts at the root package you version and then stops. In other words it doesn't mimic the recursive nature of a go get which is the functionality I think might be breaking in prod.
Would fixing the problem with the import statements in fix theses issues in dev?

I think the design now is fine, and will work with go-get in dev as soon as I correct the router. What is:

the problem with import statements

Awesome thats really good news then. The issues are one in the same I mean't  no go-import meta tags issues


[xs] Added variables DATA_PACKAGE_URL and DATA_PACKAGE to dataset_sho…

…w in  _snippets.html -refs #254
Dpr-js need DATA_PACKAGE_URL and DATA_PACKAGE from dpr-api to render graph.
Added these two variable in _snippets.html

Closing as we don't want to set DATA_PACKAGE variable.


Visibility in RecommenderService

Describe your question
In the recommendVideosForUser method, should the recommended video filter out the video that is not visible to the user?

yes


[ROCm] Fix for the broken ROCm CSB - 191203 - eigen

The following commit breaks the --config=rocm build
https://github.com/tensorflow/tensorflow/commit/3bc9dd3b776c26fbd8d4eba7917fe873a5596e40
The above commit moves the eigen repo pointer to a commit that breaks the ROCm build. The fix for the ROCm errors has been merged into the Eigen repo ( https://bitbucket.org/eigen/eigen/commits/177c403a361be18ab9ce76594b0ece232f1c8da4 ). Until the eigen repo pointer in workspace.bzl is moved to a commit that includes the above fix, we need to apply the same fix via a patch file to make the ROCm build functional again.
/cc @rmlarsen @whchung

Closing this PR as it is no longer required. TF eigen pointer has moved past the commit which contains the fix being applied by this patch.


Remove data from preparers

Once this library PR goes into main, this PR handles the breaking change update to preparer parameters.

thanks! much appreciated! we could get used to this 😅


The new "Stop auto play/buffering" implementation works perfectly only in current tab.

The new implementation is the most accurate/immediate  in stopping videos from  [playing/buffering]
However, they don't work at all if the video is opened in a background tab.
My settings:
SPF: Enabled, Chrome: latest Version 64-bit, YTC developer version 459.

it also doesn't work right with html5. it activates too late, sometimes several seconds of video play before it's simply paused.
the title image never appears.


[Bug]: [graphql] improve error messages

Is there an existing issue for this?

[X] I have searched the existing issues

Description
graphql query, wrong authentication settings, I get this error:

it looks unformatted
Steps To Reproduce
here are the steps I followed:

create a graphql datasource, url https://api.github.com/graphql

give some rubbish authentication

create a query as:
query myquery{  repository(owner:"appsmithorg", name:"appsmith"){  issues(last:20){    edges{      node{        title      }    }  } } }

see error

Public Sample App
No response
Version
Cloud

trying to attach again


Feature/python docstrings

I know this is broken for now, but wanted to make PR to facilitate discussion and have code to talk over.
This PR intends to enable the process of:
Public Header Doxygen Comments --> Python Docstring String-Literals --> Macro Expansion --> PyBind11 Python Bindings (with DocStrings)
The goal is that if you help(foo) in Python you get full docstrings, that are equivalent to the same C++ comments. This also facilitates IDE hints in VSCode etc...
It also allows docstrings to exist in one and only one place. The public C++ headers.

I looked at the CI failure. Linux and Windows are failing on configure because our VMs/images don't have LATEX installed. Is the LATEX dependency just for the Sphinx PDF target? If so what are your thoughts on removing that target and dependency. RTD can generate a PDF from the HTML docs, correct?
For macOS, which probably has LATEX installed already, the failure is due to the missing clang Python package, which mkdoc depends on. To add this dependency via CMake (with support for OCIO_INSTALL_EXT_PACKAGES) you can use this CMake macro, which acts like a Find<package>.cmake module for Python packages. It will check for the existence of the package by trying to import it into Python, and then fallback to installing it the ext/ dir with pip.
https://github.com/AcademySoftwareFoundation/OpenColorIO/blob/master/share/cmake/macros/FindPythonPackage.cmake
The find command should be put in this file, under the OCIO_BUILD_DOCS section (you'll see the find Sphinx command there):
https://github.com/AcademySoftwareFoundation/OpenColorIO/blob/master/share/cmake/modules/FindExtPackages.cmake
Finding a package in that module makes it globally accessible to the rest of the project, so you would add something like this:
include(FindPythonPackage)

find_python_package(clang, <min version>)

Each needed Python package can have a similar line.

I can't find any specific information in RTD docs about how they generate the PDF format, but that suggests that they simply let Sphinx generate it from the documentation source files (not the HTML output) on their side.

I can't find any specific information in RTD docs about how they generate the PDF format, but that suggests that they simply let Sphinx generate it from the documentation source files (not the HTML output) on their side.

Yeah it is generated from the source and from scratch by default:
python /home/docs/checkouts/readthedocs.org/user_builds/colour/envs/develop/bin/sphinx-build -b latex -D language=en -d _build/doctrees . _build/latex

Purely adding the above macro to find additional pypi packages doesn't appear to work, I get the following error:
CMake Error at share/cmake/modules/FindExtPackages.cmake:54 (find_package):
  By not providing "Findbreathe.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "breathe", but
  CMake did not find one.

  Could not find a package configuration file provided by "breathe"
  (requested version 4.19.0) with any of the following names:

    breatheConfig.cmake
    breathe-config.cmake

  Add the installation prefix of "breathe" to CMAKE_PREFIX_PATH or set
  "breathe_DIR" to a directory containing one of the above files.  If
  "breathe" provides a separate development package or SDK, be sure it has
  been installed.
Call Stack (most recent call first):
  CMakeLists.txt:260 (include)

-- Configuring incomplete, errors occurred!
See also "/media/fawlty/Alpha/Dev/github/OpenColorIO/build/CMakeFiles/CMakeOutput.log".

The Sphinx package find/download appears to only work because the additional pypi download is specified in it's FindSphinx.cmake module.

@scoopxyz what are the nest steps with this work? Let me know if you want some help sorting out the API docs.

Will have a new PR coming soon to piggy back on this work. Going to close this. Thanks @scoopxyz !


rm layout.json v1

Removes /api/v1/dlm/layout.json, which was NPE-ing in vanilla master for me.
Adds stub documentation for the /api/v4-3/dlm/layout.json that presumably one uses instead when finding that the v1 web service is gone.

Removing a web service API is a big, breaking change -- but then uPortal 4->5 is a big, breaking-changes release.
cf. uportal-dev@ list thread
Checklist

[x] the individual contributor license agreement is signed
[x] documentation is changed or added

Coverage increased (+0.01%) to 18.076% when pulling e5e87802bcf13f4a6ba832f16d8457813009e878 on rm-layoutjson-v1 into f0805a00465795102796d3c7ffef93628887bf83 on master.

This change will cause issues for uMobile app clients from OU. I was told by @bpowell that using the layout v4-3 is not technically possible. Brandon will need to voice his concerns and help with technical direction moving forward on this change and what is needed to help support mobile app clients.

I'm currently writing an email to respond back to that uportal-dev thread about this now.

Cool. Flagged as WIP. Prompting the soul-searching about how much we need to support layout.json v1 for how long is a great outcome for this PR; I fully expect this PR will end up closed un-merged.

@apetro is the plan still to close this out?
Or was there an offline conversation that found a way forward for this PR?

Closing PR for now, it can be re-opened if a path forward is found.


Admin middleware,admin route and controller.

Add admin actions to every possible scenario.

Added main admin routes and controllers, also added some changes to Users and Posts models.


HTTP Header Parsing is wrong in 0.11.1

in boost/network/protocol/http/client/connection/sync_base.hpp
in the
void read_headers(...)
function it is assumed that the HTTP header field content starts 1 byte after the colon.
(colon_offset + 2)
response_ << header(name, header_line.substr(colon_offset + 2));
From the HTTP/1.1 Spec (http://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html#sec4.2)
[...]
The field value MAY be preceded by any amount of LWS, though a single SP is preferred.
[...]

Interesting. I agree this seems broken. Can you send in a pull request to
fix this?
On Tue, Jan 20, 2015, 07:14 Patrick Heeb<EMAIL_ADDRESS>wrote:

in boost/network/protocol/http/client/connection/sync_base.hpp
in the
void read_headers(...)
function it is assumed that the HTTP header field content starts 1 byte
after the colon.
(colon_offset + 2)
response_ << header(name, header_line.substr(colon_offset + 2));
From the HTTP/1.1 Spec (
http://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html#sec4.2)
[...]
The field value MAY be preceded by any amount of LWS, though a single SP
is preferred.
[...]
—
Reply to this email directly or view it on GitHub
https://github.com/cpp-netlib/cpp-netlib/issues/487.

sure, I'm working on a testcase for it.

Actually this looks like it's a real bug. re-opening.

I already sent a pull request. But I'm struggling with getting the test to work with travis.

Maybe it[s something we can help with? What is the issue you're having with Travis?

in pull request #490 I have a testcase. executing it local on my machine succeeds. but travis hung on executing it.

Right. So as I mentioned in #490 and #495 this is being done on the synchronous client, which is no longer in 0.12. In the asynchronous client, this has to be done in the incremental parser instead, which is testable in isolation. Do you want to try and do that instead @speedpat?

Sure, I'll do it. @deanberris

Any update on this @speedpat -- I've just tagged an rc1 for 0.12.0 and would love to get this fixed in the next RC.


Remove some deprecation warnings

Description:
Remove some deprecation warnings
Type of change:

[x] Bug fix.
[ ] New feature.
[ ] Documentation update.

Checklist for the reviewer:

This checklist should be used as a help for the reviewer.

[ ] Is the change limited to one issue?
[ ] Does this PR close the issue?
[ ] Is the code easy to read and understand?
[ ] Do all new feature have an accompanying new test?
[ ] Has the documentation been updated as necessary?

Codecov Report
Base: 67.52% // Head: 67.25% // Decreases project coverage by -0.27% :warning:

Coverage data is based on head (718843b) compared to base (68a30ed).
Patch coverage: 100.00% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #83      +/-   ##
==========================================
- Coverage   67.52%   67.25%   -0.27%
==========================================
  Files          14       14
  Lines        1124     1124
==========================================
- Hits          759      756       -3
- Misses        365      368       +3

Impacted Files
Coverage Δ

tripper/triplestore.py
75.70% <100.00%> (-1.06%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


rgb color, how to?

it's not clear how to use your own rgb color values for the themes/prompt. could someone please point me in the right direction? thanks

Sorry to revive this, but I just don't understand how to change the colors using RGB.
zork.theme.bash...
I used this:
my_ps_host="$(color rgb 195 232 141)\h${normal}";
But the color is a pinkish orange, not green.
Looked at colors.theme.bash:
function __color_rgb {
r=$1 && g=$2 && b=$3
[[ r == g && g == b ]] && echo $(( $r / 11 + 232 )) && return # gray range above 232
echo "8;5;$(( ($r * 36  + $b * 6 + $g) / 51 + 16 ))"
}
This makes 0 sense to me :(
I've spent about an hour reading and following links to no avail.  Any help with the syntax?

Sorry, can't help you with that. A search in the code showed that the __color_rgb function is not used anywhere.
Not sure who added that function and what it does, sorry...

it was added a while back.
https://github.com/Bash-it/bash-it/pull/99#issue-3133339
on this file https://github.com/Bash-it/bash-it/blob/569a7a902e133963bd897eee1a1a8173a5a3f66a/themes/colors.theme.bash#L110
Maybe shooting in the dark... thanks at any rate

Perfect! Not what I was thinking but it did the trick. Thanks


List only CI-run Rubies as supported

Tell only of tested support for exact versions

I don't know... This change, when I look at it, it's not so fun.
Think of it as an opener: do you want to add required_ruby_version to the gemspec? In v3.0.0?

It actually does not yet work with jruby-head (you're welcome to contribute). I allow failures for ruby-head and jruby-head on Travis. But if you update the list to reflect that, I'm happy to merge the pull request.

Coverage remained the same at 91.76% when pulling 4d92381346b2454b07472e1dd2fbf6a94f5ec041 on olleolleolle:patch-2 into bfb1f4b73f070c28f41d7d6f40a455e304b8e8e1 on rudionrails:master.

Coverage remained the same at 91.76% when pulling 32f3e918382082e48a69b3921324e36ac7090748 on olleolleolle:patch-2 into bfb1f4b73f070c28f41d7d6f40a455e304b8e8e1 on rudionrails:master.

@rudionrails I changed the build to run on jruby-head, as well. Now the README change reflects what is.


perf: conditional keyring event listeners

Description
This is both a performance optimization and a bug fix.
Recent performance improvements exposed a bug, that can actually be fixed by yet more performance improvements.
The bug (and perf issue): the KeepKey event handler would subscribe to events no matter what type of wallet was connected.
In one edge case it would cause MetaMask's connection event to fire KeepKey's 'handleConnect' function, which would update the wallet state's type value, which would break the connected wallet routing.
The fix is two-fold:

Only subscribe to KeepKey events if a KeepKey is connected (additionally, I applied the same logic to the Native event hook for good measure).

Decouple the "connected wallet" state from the "model flow type" (names are hard). The second bug that this fixes is, for e.g., if you clicked "Switch Wallet" and selected "KeepKey", the state would update such that the connected wallet was now KeepKey before the connection took place. This meant that if you pressed "back" you'd be in a really wierd place.

A final note, I don't really know why the latest performace issues make all of this more apparent, because as far as I can tell this code was always logically flawed.
Pull Request Type

[x] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
N/A, production issue.
Risk
Medium - impacts wallet events (especially Native and KeepKey), switching wallet logic, and connected wallet functionality.
Testing
Test:

Switching between wallets, especially Native, KeepKey, and MetaMask
Events from Native (password entry) and KeepKey (PIN entry, changing passwork, signing messages etc)

Engineering
☝
Operations
☝
Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #4999   👈

This comment was auto-generated by Graphite.


skip ReadonlyMap and ReadonlySet types when not available

This should fix #624 .
Instead of my initial idea of providing interfaces with the same shape as ReadonlyMap, ReadonlySet, WeakMap and WeakSet (which would also have worked, but probably required a lot more duplicate code), this is now following a different approach:
It checks if the types are any (which they would fall back to if they were not present) or a keyless interface (which at least ReadonlyMap and ReadonlySet would be if the node types are installed), in which case they fall back to void, which does not trigger the conditional.
The check for any is done by true | false extends (T extends never ? true : false) -  every other type will either be true or false, but since any takes both paths, it becomes true | false.
The check for the empty interface is done by keyof T extends never - everything except the empty interface has some keys, including primitives like string and number (which will have the keys of their prototype).

Thanks for the PR! Looks fine at first sight and clever trick :). However, if it turns out type checking becomes more expensive or there is other fallout, the changes might get referred.

:tada: This PR is included in version 7.0.11 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/214 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit ad0baa9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Prints babel 5 deprecation

Relates to https://github.com/ember-cli/ember-cli/issues/7642.
Based on feedback from @Turbo87 we are printing a deprecation message every time this addon runs. This will help any consumers of a ^5.x version receive a deprecation message.
(Collaborating with @cah-danmonroe)

tests will pass once https://github.com/babel/ember-cli-babel/pull/208 merges.

Can you rebase?

Rebased commits on top of upstream/v5.x. Please let me know if you want me to squash as well.

Thank you!


Fix for CORE-3643: Change the accessor methods of validateX properties of the ConstraintConfig

This should take care of the issues raised in CORE-3643.  I tried to keep the changes to the minimum needed to fix the Groovy DSL parser.  There is still some parts of the code that use shouldValidate instead of getValidate that I could fix for code consistency, but we can leave that for another day.
All of the unit tests pass, but there are 28 integrations tests that fail for one reason or another.  The same 28 integration tests were failing before I made any changes.

Hi @stevesaliman
Thanks for your pull request!
Here’s what happens next:
A member of the Liquibase team will take a look at your contribution and may suggest:

Changes based on guidelines for PR submission
Additional tests (unit and/or integration)
Provide other feedback, like requesting changes.

The PR will be prioritized according to our internal development and testing capacity.
We’ll let you know when it’s ready to move to the next step or if any changes are needed.

I sized this as Risk Med as:

It is not a bug, but a nice to have (an admittedly useful nice to have)
Some test but will defer to internal team on testing is sufficient (the previously failing int. tests should be addressed).

Closing as we merged this as part of #1177


[21.05] element-desktop: apply patch to run on Wayland

Motivation for this change
See upstream PR#261[1] for further reference. Previously, the
enable-features-setting was entirely discarded due to an earlier
regression resulting in an attempt to start element-desktop in
Wayland-mode without all necessary components.
Closes #137377
[1] https://github.com/vector-im/element-desktop/pull/261
(cherry picked from commit 5a0d0ec1cf90aae4354a97a2afaff079605ffc82)
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 137672 at 4af67928 run on x86_64-linux 1

1 package built successfully:

element-desktop

4 suggestions:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/tools/yarn2nix-moretea/yarn2nix/default.nix:332:7:
    |
332 |       installPhase = attrs.installPhase or ''
    |       ^

warning: build-tools-in-build-inputs
rsync is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/tools/yarn2nix-moretea/yarn2nix/default.nix:292:7:
    |
292 |       buildInputs = [ yarn nodejs rsync ] ++ extraBuildInputs;
    |       ^

warning: build-tools-in-build-inputs
yarn is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/tools/yarn2nix-moretea/yarn2nix/default.nix:292:7:
    |
292 |       buildInputs = [ yarn nodejs rsync ] ++ extraBuildInputs;
    |       ^

warning: missing-patch-comment
Consider adding a comment explaining the purpose of this patch on the line preceeding.
Near pkgs/applications/networking/instant-messengers/element/element-desktop.nix:23:5:
   |
23 |     (fetchpatch {
   |     ^

Result of nixpkgs-review pr 137672 run on x86_64-linux 1

1 package built:

element-desktop

Looks good! :+1:


Prometheus scraping doesn't seem to work

Prometheus shows:
Service Discovery
envoy-stats (28/128 active targets)
galley (1/28 active targets)
istio-mesh (1/28 active targets)
istio-telemetry (1/28 active targets)
kubernetes-apiservers (1/1 active targets)
kubernetes-cadvisor (1/1 active targets)
kubernetes-nodes (1/1 active targets)
kubernetes-pods (0/128 active targets)
kubernetes-pods-istio-secure (2/124 active targets)
kubernetes-service-endpoints (3/145 active targets)
pilot (1/28 active targets)
But there is no data for istio_requests_total
There are some stats like envoy_cluster_upstream_rq_total but doesn't seem to pull in everything, since I hit bookinfo page a lot and it shows 2 requests total
From telemetry logs:
2019-04-30T19:04:09.052960Z     info    Publishing 24 events
2019-04-30T19:04:09.058654Z     info    Built new config.Snapshot: id='6'
2019-04-30T19:04:09.058726Z     info    adapters        getting kubeconfig from: ""     {"adapter": "kubernetesenv.istio-control"}
2019-04-30T19:04:09.058743Z     warn    Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
2019-04-30T19:04:09.059781Z     info    adapters        Waiting for kubernetes cache sync...    {"adapter": "kubernetesenv.istio-control"}
2019-04-30T19:04:09.159879Z     info    adapters        Cache sync successful.  {"adapter": "kubernetesenv.istio-control"}
2019-04-30T19:04:09.159904Z     info    adapters        getting kubeconfig from: ""     {"adapter": "kubernetesenv.istio-control"}
2019-04-30T19:04:09.159908Z     warn    Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
2019-04-30T19:04:09.160621Z     info    Setting up event handlers
2019-04-30T19:04:09.161880Z     warn    No destination sets found for the default namespace 'istio-control'.
2019-04-30T19:04:09.161896Z     info    Cleaning up handler table, with config ID:5
2019-04-30T19:04:09.161906Z     info    adapters        deleted remote controller       {"adapter": "kubernetesenv.istio-control"}
2019-04-30T19:04:09.161922Z     info    adapters        adapter closed all scheduled daemons and workers        {"adapter": "prometheus.istio-control"}
2019-04-30T19:04:09.161930Z     info    Starting Secrets controller
2019-04-30T19:04:09.161945Z     info    Waiting for informer caches to sync
2019-04-30T19:04:09.162063Z     info    adapters        shutting down daemon... {"adapter": "kubernetesenv.istio-control"}
2019-04-30T19:04:09.162081Z     info    adapters        shutting down daemon... {"adapter": "kubernetesenv.istio-control"}
2019-04-30T19:04:09.162090Z     info    adapters        shutting down daemon... {"adapter": "kubernetesenv.istio-control"}
2019-04-30T19:04:09.162098Z     info    adapters        shutting down daemon... {"adapter": "kubernetesenv.istio-control"}
2019-04-30T19:04:09.162105Z     info    adapters        adapter closed all scheduled daemons and workers        {"adapter": "kubernetesenv.istio-control"}

istio-mesh (1/1 up)

Endpoint
State
Labels
Last Scrape
Error

http://<IP_ADDRESS>:42422/metrics
UP
instance="<IP_ADDRESS>:42422"
11.776s ago

So the telemetry target is read and scraped properly, but when I curl http://<IP_ADDRESS>:42422/metrics I get a 200 OK but with an empty body

Have you looked through: https://istio.io/help/ops/telemetry/missing-metrics/ ?


Add ARM64 Dockerfile

Currently, the arm64 image bundles x86 binaries, which results in exec errors.
Added a Dockerfile to build correctly for ARM64
fixes https://github.com/rancher/cis-operator/issues/151

The Dockerfile.arm64 might not be needed because we can parse ARCH from dapper environment.
You can specify following commands in Dockerfile.dapper
ARG DAPPER_HOST_ARCH="amd64"
ENV ARCH=${DAPPER_HOST_ARCH}

And put the ARCH variable into the docker build build-args in scripts/package script.

@dgiebert  Just copy the current Dockerfile (that has your changes) to Dockerfile-arm64 and make the ARCH  to arm64 in the arguments.
Then update the scripts/package like below

scripts/package
#!/bin/bash
set -e

source $(dirname $0)/version

ARCH=${ARCH:-"amd64"}
if [ "$ARCH" = "amd64" ]; then
  SUFFIX="-amd64"
elif [ "$ARCH" = "arm64" ]; then
  SUFFIX="-arm64"
else
  echo "Unsupported architecture: $ARCH"
  exit 1
fi

cd $(dirname $0)/..

TAG=${TAG:-${VERSION}${SUFFIX}}
REPO=${REPO:-rancher}

if echo $TAG | grep -q dirty; then
    TAG=dev
fi

mkdir -p dist/artifacts

IMAGE=${REPO}/security-scan:${TAG}
DOCKERFILE=package/Dockerfile
if [ -e ${DOCKERFILE}${SUFFIX} ]; then
    DOCKERFILE=${DOCKERFILE}${SUFFIX}
fi

docker build -f ${DOCKERFILE} -t ${IMAGE} .
echo Built ${IMAGE}

The changes looks good. I build two images, one on amd64 (rayandas/security-scan:amd1) and another one on arm64 (rayandas/security-scan:arm2) machines and tested the images on ARM (Single node all role) cluster and AMD (Single node all role) cluster and the scans are passing.
Scan report on AMD64 Cluster:

Scan report on ARM64 Cluster:

@prachidamle Need your approval here before merging.

@andypitcher @JonCrowther Adding you folks as a reviewer.

These changes are no longer needed as #112 adds the use of Dapper's ARCH env var. That PR introduced E2E tests for both amd64 and arm64.

I'm closing this PR as per @pjbgf 's comment.


Adds Python 3.7 testing

Adds Python 3.7 to the Travis matrix (Python 3.8 is due out soon). Unlikely to cause feature conflicts due to your upgrade cycle, but they do occasional make tweaks to concurrent.futures.

This fails on 3.6 (not the added 3.7) with an MPI install error that seems unrelated to the PR... this happens sometimes because we install so much stuff each time.


update to current branch

I assume this was opened by accident.

That's right; it was an accident. Thanks!
On Nov 22, 2016 1:17 AM, "Georgios Kalpakas"<EMAIL_ADDRESS>wrote:

Closed #386 https://github.com/angular/angular-seed/pull/386.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angular/angular-seed/pull/386#event-867866363, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGT-vw07-r44oOzAxa0o9wb5O67eRp2ks5rArMtgaJpZM4K5AMY
.


Accordions with other open content areas isn't working

Test case: https://codepen.io/cferdinandi/pen/NEBvew

Fixed with https://github.com/cferdinandi/houdini/releases/tag/v10.0.1


Any ideas?

I appreciate this isn't a support forum, but had an issue when deploying a new version of our clients site. So decided to roll back to last known commit but now throwing this error:
Fatal error: Class 'PHPRouter\Router' not found in
******************/wp-content/plugins/timber-library/lib/timber-routes.php on line 30
Can anyone throw any light on the issue?! (URGENTLY if possible! :))
Assumed it was something to do with the Github version VS Wordpress.org but I've used the WP.org version...

This was the first error we got:
Fatal error: Class 'Routes' not found in ......

Looks like something might have gone wrong with your rollback - I think PHPRouter is the old router that Timber used to use, before it got switched out for AltoRouter.
So - check on the server in /wp-content-plugins/timber-library/vendor/ to see if you've got an altorouter folder. If so, that's probably the issue. You can download older plugin versions here: https://wordpress.org/plugins/timber-library/developers/ - you just need to get the one you were using previously.

Amazing @anugupta, thanks for helping out with this! Typical, meant to be going live with the new version and all this explodes! I've managed to get the site working by completely deleting the plugin and re-uploading via FTP...I guess there may have been something conflicting in Codebase that was throwing the error...going to have to trial and error to see if I can resolve the issue!
Thanks again!

You're welcome - glad that helped :)


Resource Level Control - Authentication & Throttling

Hi,
We are planning to integrate Kong into our architechture and after some playing around with Kond admin apis, We have the following questions.
1. Resource level Authentication
I could see that the Admin API for adding an API contains request_path with the help of which I can add resources to my API. Will I be able to control the Authentication at Resource and HTTP Method of the resource level.
For Example.
Lets say My Api is Pradeep and I have 2 resources within that
/todo and /user
Scenario 1:
now I want to control the Oauth 2 authentication in such a way that
POST to /user should not be checked for authentication since , Signup need not require any token
Other resources should be authenticated.
Scenario 2:
With the OAuth in place I need the flexibility to use different token types like Client_Credetials (Application Token) , Password or Auth Code (User). This is required for the above same reason
/user resource any HTTP method can be accessed only with the user context. So I should be able to say something like
/user POST, GET, PUT should be allowed only for token types (password, Auth code)
2. Resource level Throttling
same as the above I need to set the request limit for any Resource and HTTP method combination.
Kindly let us know the feasibility of the above with kong and its plugins.
If the above scenarios are in kong's road map please let us know. We are also open to desgin and contribute the same back to Mashape if you guys think it will be really a good addition.
Pradeep.

Currently incoming requests are NOT matched based on the method (get/post/etc). It's a highly requested feature that we're planning to add.
As for throttling, please checkout the rate-limiting plugin.

Considering this answered, thanks.


Reverible diode prerouted

when diode is used in smd, and reversible configuration, we can now enable include_traces_vias which shorts both the front and back layers of the pcb, which will simplify routing.

@im-AMS thank you for your contribution! Check out what has also been added to the reversible diode (thru-hole SMD pads) for an alternative to routing with via / traces.


Feature/navbar

Reworked the navbar and all responsiveness.
  That CSS should just be removed from the library completely.. I hate it.


Pytest Warnings

There are currently 27 warnings when running pytest.
These need to be investigated and, ideally, removed.
../../Library/Caches/pypoetry/virtualenvs/swole-v2-XOUsbHQA-py3.12/lib/python3.12/site-packages/passlib/utils/__init__.py:854
../../Library/Caches/pypoetry/virtualenvs/swole-v2-XOUsbHQA-py3.12/lib/python3.12/site-packages/passlib/utils/__init__.py:854
../../Library/Caches/pypoetry/virtualenvs/swole-v2-XOUsbHQA-py3.12/lib/python3.12/site-packages/passlib/utils/__init__.py:854
../../Library/Caches/pypoetry/virtualenvs/swole-v2-XOUsbHQA-py3.12/lib/python3.12/site-packages/passlib/utils/__init__.py:854
../../Library/Caches/pypoetry/virtualenvs/swole-v2-XOUsbHQA-py3.12/lib/python3.12/site-packages/passlib/utils/__init__.py:854
  /Users/timothy/Library/Caches/pypoetry/virtualenvs/swole-v2-XOUsbHQA-py3.12/lib/python3.12/site-packages/passlib/utils/__init__.py:854: DeprecationWarning: 'crypt' is deprecated and slated for removal in Python 3.13
    from crypt import crypt as _crypt

tests/unit_tests/test_validators.py::test_check_valid_uuid
tests/api_tests/test_workouts.py::TestWorkouts::test_workout_update_fails_with_params[Test update with incorrectly formatted date fails]
tests/api_tests/test_workouts.py::TestWorkouts::test_add_exercise_fails_when_adding_exercise_from_other_user
tests/api_tests/test_users.py::TestUsers::test_user_create_multiple_succeeds
  /Users/timothy/Library/Caches/pypoetry/virtualenvs/swole-v2-XOUsbHQA-py3.12/lib/python3.12/site-packages/pytest_asyncio/plugin.py:654: DeprecationWarning: There is no current event loop
    old_loop = asyncio.get_event_loop()

tests/api_tests/test_workouts.py::TestWorkouts::test_workout_update_fails_with_params[Test update with incorrectly formatted date fails]
tests/api_tests/test_workouts.py::TestWorkouts::test_add_exercise_fails_when_adding_exercise_from_other_user
tests/api_tests/test_users.py::TestUsers::test_login_with_invalid_creds_fails
tests/api_tests/test_users.py::TestUsers::test_user_create_multiple_succeeds
  /Users/timothy/Library/Caches/pypoetry/virtualenvs/swole-v2-XOUsbHQA-py3.12/lib/python3.12/site-packages/uvloop/__init__.py:31: DeprecationWarning: uvloop.install() is deprecated in favor of uvloop.run() starting with Python 3.12.
    _warnings.warn(

tests/api_tests/test_workouts.py::TestWorkouts::test_workout_update_fails_with_params[Test update with incorrectly formatted date fails]
tests/api_tests/test_workouts.py::TestWorkouts::test_add_exercise_fails_when_adding_exercise_from_other_user
tests/api_tests/test_users.py::TestUsers::test_login_with_invalid_creds_fails
tests/api_tests/test_users.py::TestUsers::test_user_create_multiple_succeeds
  /Users/timothy/Library/Caches/pypoetry/virtualenvs/swole-v2-XOUsbHQA-py3.12/lib/python3.12/site-packages/pytest_asyncio/plugin.py:749: DeprecationWarning: The event_loop fixture provided by pytest-asyncio has been redefined in
  /Users/timothy/Personal/Swole-V2/tests/conftest.py:50
  Replacing the event_loop fixture with a custom implementation is deprecated
  and will lead to errors in the future.
  If you want to request an asyncio event loop with a scope other than function
  scope, use the "scope" argument to the asyncio mark when marking the tests.
  If you want to return different types of event loops, use the event_loop_policy
  fixture.

    warnings.warn(

tests/api_tests/test_users.py::TestUsers::test_login_succeeds
tests/unit_tests/test_auth.py::TestAuth::test_get_current_user_succeeds
tests/unit_tests/test_auth.py::TestAuth::test_get_current_user_fails_with_incorrect_secret
tests/unit_tests/test_auth.py::TestAuth::test_get_current_user_fails_with_non_existing_user
tests/unit_tests/test_auth.py::TestAuth::test_inactive_user_fails
tests/unit_tests/test_auth.py::TestAuth::test_get_current_user_fails_with_invalid_token
  /Users/timothy/Personal/Swole-V2/src/swole_v2/database/repositories/users.py:100: DeprecationWarning: datetime.datetime.utcnow() is deprecated and scheduled for removal in a future version. Use timezone-aware objects to represent datetimes in UTC: datetime.datetime.now(datetime.UTC).
    to_encode.update({"exp": datetime.utcnow() + timedelta(minutes=self.settings.TOKEN_EXPIRE)})

tests/unit_tests/test_auth.py::TestAuth::test_get_current_user_succeeds
tests/unit_tests/test_auth.py::TestAuth::test_get_current_user_fails_with_non_existing_user
tests/unit_tests/test_auth.py::TestAuth::test_inactive_user_fails
tests/unit_tests/test_auth.py::TestAuth::test_get_current_user_fails_with_invalid_token
  /Users/timothy/Library/Caches/pypoetry/virtualenvs/swole-v2-XOUsbHQA-py3.12/lib/python3.12/site-packages/jose/jwt.py:311: DeprecationWarning: datetime.datetime.utcnow() is deprecated and scheduled for removal in a future version. Use timezone-aware objects to represent datetimes in UTC: datetime.datetime.now(datetime.UTC).
    now = timegm(datetime.utcnow().utctimetuple())

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html

The passlib library using crypt is a known issue. However, there doesn't seem to be much activity in the passlib repo.
https://foss.heptapod.net/python-libs/passlib/-/issues/148


Streamline setting up gitto

As mentioned in the commit fa8066df9c7f967248991f95a1e1f67665dc13a1, it is expected that the gitto is cloned at ../gitto and built. This is cumbersome and we need to find a better way to handle this.

Option 1:
Run
$ go install github.com/pipalacademy/gitto@latest

That installs gitto at ~/go/bin.
Add ~/go/bin to PATH from the Makefile so that Honcho can find it when starting it.

@nikochiko what do you think?
The go install could be part of a make rule.

Yeah this is good enough.

Added in https://github.com/pipalacademy/capstone/commit/d9c98d8b50ccd94018a63aaa9ae2073766514e8e


[help wanted] Web application data privacy and error rules IDs misalignment between provider version 1.18.1 and older version

Describe the bug
I created web applications as well as their data privacy rules and error rules using provider version 1.14.1. While creating them, the IDs for data privacy and error rules have the prefixes of "data-privacy-" or "error-rules-". However, when I tried to upgrade the provider version to 1.18.1 and plan the changes (which will fetch the configurations of the states), the errors came in. The IDs now have suffixes rather than prefixes, so there are errors when using the old ID to GET the information.
The problem might come from the state itself. Is there a way to cope without destroying the resource and recreating it?
To Reproduce
Steps to reproduce the behavior:

create a web application, its data privacy and error rules using provider version 1.14.1
upgrade the provider version to 1.18.1
terraform init -upgrade and terraform plan

Expected behavior
Maybe can refresh the states...and plan successfully
Screenshots

Version
Terraform version: 1.1.3
Dynatrace cluster version: 1.254
Additional context
Any help over here will be great. Thanks.

Hello @cyang1001
Our code has undergone a major refactoring with 1.18.1 - and in this specific case we apparently didn't pay enough attention. Apologies for the troubles.
On short notice: Do you have access to the terraform.tfstate file? You could modify the IDs in there to match the new syntax.
I will however also find a way to support both, the old and the new IDs in parallel until our next release (planned for Monday).

Hello @cyang1001
Our code has undergone a major refactoring with 1.18.1 - and in this specific case we apparently didn't pay enough attention. Apologies for the troubles.
On short notice: Do you have access to the terraform.tfstate file? You could modify the IDs in there to match the new syntax.
I will however also find a way to support both, the old and the new IDs in parallel until our next release (planned for Monday).

Hi @Dynatrace-Reinhard-Pilz , thank you for the information! I will try to edit the tfstate file.


use i2c scanner issue..

// i2c_scanner
//
// Version 1
//    This program (or code that looks like it)
//    can be found in many places.
//    For example on the Arduino.cc forum.
//    The original author is not know.
// Version 2, Juni 2012, Using Arduino 1.0.1
//     Adapted to be as simple as possible by Arduino.cc user Krodal
// Version 3, Feb 26  2013
//    V3 by louarnold
// Version 4, March 3, 2013, Using Arduino 1.0.3
//    by Arduino.cc user Krodal.
//    Changes by louarnold removed.
//    Scanning addresses changed from 0...127 to 1...119,
//    according to the i2c scanner by Nick Gammon
//    https://www.gammon.com.au/forum/?id=10896
// Version 5, March 28, 2013
//    As version 4, but address scans now to 127.
//    A sensor seems to use address 120.
// Version 6, November 27, 2015.
//    Added waiting for the Leonardo serial communication.
//
//
// This sketch tests the standard 7-bit addresses
// Devices with higher bit address might not be seen properly.
//

#include <Wire.h>

void setup()
{
  Wire.begin();

  Serial.begin(115200);
  while (!Serial);             // Leonardo: wait for serial monitor
  Serial.println("\nI2C Scanner");
}

void loop()
{
  byte error, address;
  int nDevices;

  Serial.println("Scanning...");

  nDevices = 0;
  for(address = 1; address < 127; address++ )
  {
    // The i2c_scanner uses the return value of
    // the Write.endTransmisstion to see if
    // a device did acknowledge to the address.
    Wire.beginTransmission(address);
    error = Wire.endTransmission();

    if (error == 0)
    {
      Serial.print("I2C device found at address 0x");
      if (address<16)
        Serial.print("0");
      Serial.print(address,HEX);
      Serial.println("  !");

      nDevices++;
    }
    else if (error==4)
    {
      Serial.print("Unknown error at address 0x");
      if (address<16)
        Serial.print("0");
      Serial.println(address,HEX);
    }
  }
  if (nDevices == 0)
    Serial.println("No I2C devices found\n");
  else
    Serial.println("done\n");

  delay(5000);           // wait 5 seconds for next scan
}

I used Arduino IDE 2.2 to run a simple i2c scanner code, but I didn't get the correct response. Can you check the result of this code execution?

I have also used the 'wire' from the Ameba2 mini examples, and verified with an oscilloscope that there is no output change in every GPIOs pin.

Dear @Oliver0804 ,
Thanks for reporting this issue.
I have verified that AMB82-MINI does not support examples such as "I2C - Scan I2C devices" nor "I2C - Display Data on LCD_Screen" at this point of time, we will update you again upon deploying the bug fix.

keep

keep again...

Keep

Hi @Oliver0804 ，
Example will be provided on v4.0.6 early release (dev branch) today, lower level including HAL and ROM code have been modified, thus we need to update the archive libraries as well.
The glance of the code can be found below:
Open the example in “File” -> “Examples” -> “AmebaWire” -> “I2C_Scanner”.

When the I2C bus detect any I2C device, the serial monitor will show the address of the I2C device as shown below:

When there is no I2C device connected to the board, the Arduino IDE serial monitor will show the message below:

Thanks

It has been tested and the problem still exists. When no i2c device is mounted, a pile of addresses will still be scanned

I used amb82 for testing

Thank you. After I updated the library functions at the same time, the problem was resolved

@Tomi-ren,
Glad that your issue has been solved.

@Tomi-ren, Glad that your issue has been solved.

But I tried multiple i2C devices, but the amb82mini couldn't scan their addresses and kept saying "No I2C devices found". On the contrary, using Arduino Uno to run the scan example made it easy to scan the devices, which I find difficult to understand

Reminder everyone, if using pin 12.13, please change Wire1. begin() to Wire. begin(), and you will find that it is working properly

@Tomi-ren ,
Sorry that since it's the dev branch, while I was doing the commit I didn't change from Wire1 to Wire that might causing potential confusing. But anyway, glad that you have solved your problem, and I will proceed to close this issue.

Hi @Oliver0804 ，
Example will be provided on v4.0.6 early release (dev branch) V4.0.6-QC-V01, lower level including HAL and ROM code have been modified, thus we need to update the archive libraries as well.
The glance of the code can be found below: Open the example in “File” -> “Examples” -> “AmebaWire” -> “I2C_Scanner”.

When the I2C bus detect any I2C device, the serial monitor will show the address of the I2C device as shown below:

When there is no I2C device connected to the board, the Arduino IDE serial monitor will show the message below:

Thanks

I will wait for it to be merged into the release version before I proceed with testing.

Hi @Oliver0804 ，
Example will be provided on v4.0.6 early release (dev branch) V4.0.6-QC-V01, lower level including HAL and ROM code have been modified, thus we need to update the archive libraries as well.

This is very good, i2c support has been a real issue with using this board for any projects.


Ticket 466011 - Widget 'LoginForm' option to present email specific keyboard

Feature request: We use email addresses as user ID's, so would it be possible for the LoginForm widget to have an option to trigger the email specific keyboard on tablets/phones so that common characters user in emails (e.g. '@' or '_' ) are readily available to the user.

Feature request - notify the user if / when they have failed to login 3 times, and thus their account is locked for 5 minutes (per Mendix's standard settings).

Fixed in 3.5.0


Fleshing out appmetadata()

The appmetadata() method always does the same as long as __init__() or some other method sets the metadata instance variable. To simplify wrapping we could just define it on ClamsApp:
def appmetadata(self):
    return json.dumps(self.metadata)

8d25e3a implements this behavior, but as it changes the ABS API spec, it breaks all existing apps (we're updating them anyway so I think it doesn't matter).

closed via #22


add all cordex high res domains (0.11)

we should update the cordex tables with the 0.11 resolution according to EUR-11.

this file needs updates: https://github.com/euro-cordex/cordex/blob/develop/src/cordex/tables/cordex-high-res.csv


Fix connectivity 898

From the last implementation, there were two connectivity components 1 for the connected numbers which are located on the expansion panel header, and the second one for connectivity itself (as an angular component for connectivity was moved to its own module), which was not the correct approach I guess. As the connectivity component is an external web component we need to run it anyways and get the number, so instead of run 1 connectivity web component on the nehuba container and second on the angular "component" itself we have expanding panel to get the connection number first and then closing the panel and use half resources. Maybe in the future when we will fully integrate Brainscapes API, if we will get connection number for the region without fetching full connectivity we could be able to change current approach but for now, I think it is the most efficient and performant way.

I suspect this has something to do with getting connectivity data? I suspect if you implement #889 for connectivity data, this issue may be resolved automatically?

Yes, maybe after Brainscapes integration the ability to use lazy loading will be more realistic.

The third approach is that we can change the connectivity web component so that it will be able to be used only to get the number of connections and not the diagram but I do not know if it is the best approach. Maybe for the time being the current one is the best?

As the connectivity component is an external web component we need to run it anyways and get the number,

at the time, when brainscape didn't exist, this might have been the approach we went for. But with brainscape unifityin the api into a single end point, it is far better to make the component dumb, and let angular do the fetching.

As the connectivity component is an external web component we need to run it anyways and get the number,

at the time, when brainscape didn't exist, this might have been the approach we went for. But with brainscape unifityin the api into a single end point, it is far better to make the component dumb, and let angular do the fetching.

I agree. So if we need to have a fix ASAP "before we integrate Brainscapes" we can merge current PR (and I will add "ToDo" comments to remove it after integration). If we can wait some time I will close the PR.

you are working to integrate brainscapes api any way, right?

you are working to integrate brainscapes api any way, right?

Yes. Currently, I'm still on quick view but from tomorrow I hope I will back to it.

Fixed with: https://github.com/FZJ-INM1-BDA/siibra-explorer/pull/949


(PDB-1784) Target PC1 repos in ezbake config

This commit saves release from having to type an extra flag when pushing
the PuppetDB packages up for a release.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1731/


Add Transform hover to the Footer Links

What would you like to share?
Hey @punyakrit
I would like to add a smooth transform hover on the footer links to zoom a little to look more visually appealing

Like this
https://github.com/punyakrit/social-share/assets/101971980/782b0cfa-19a6-4077-85e8-8db10d36f446
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@punyakrit could you please assign me this issue?


Added realpath

Oh, we can't check in raw binaries. Who knows what's in them. Can it be built from source? (A simple postinstall command?)

As a sidenote, the ocaml Core Filename module seems to offer a realpath function. https://ocaml.janestreet.com/ocaml-core/109.07.00/doc/core/Filename.html

If it does in fact work on Windows we could make a tiny program that depends only on OCaml and it would work on all OS's.


Functional laplace

What is still missing for FunctionalLaplace? From what I can tell, it seems like only marginal likelihood is missing, right?

indeed, the only thing still missing is marginal likelihood for the classification case:
def _log_marginal_likelihood(self):
    self.fit(self.train_loader)
    if self.likelihood == 'classification':
        raise NotImplementedError
    elif self.likelihood == 'regression':
         return - 0.5 * (self.log_det_K + self.scatter_lml + self.M * self.n_outputs * np.log(2 * np.pi))

I got a bit stuck here since the algorithm from R&W 2006 does not scale linearly in nr. of classes (C) based on my understanding - I am 95% sure that there is an error (same error that you already pointed out to me). I will double check it again in the following days.
What I can do is add a support for the case where L_diagonal=True (L is a Hessian of log likelihood w.r.t. f, see L_{NN} in A.2.1 chapter in your Local Linearization paper). This is anyways what we also do for a posterior in the classification case, since also for a posterior the algorithm from R&W 2006 can not be used based on my understanding. Let me know if that makes sense or if you have any other proposals here.

What you say makes sense. In any case the marginal likelihood computation should be synchronized with that of the posterior predictive. So when L_diagonal=True, this should also be used for the marginal likelihood.

Asked about the issue found in R&W 2006 on CrossValidated StackExchange: https://stats.stackexchange.com/questions/555183/gaussian-processes-multi-class-laplace-approximation

Added marginal likelihood. In case of classification we approximate L with a diagonal matrix both in posterior as well as in marginal likelihood.

Addressed change requests. However, when merging latest changes from main branch there are quite some conflicts that will take me some time to resolve. I will resolve them over the weekend, but if you want to make a release before just go ahead 🙂

actually managed to resolve conflicts sooner than expected. let me know if there's anything else that I should do in case we are to include FunctionalLaplace into the upcoming release:

should I run black formatter locally to adhere to standards mentioned in #63 ?
is FunctionalLaplace too visible in the documentation (see pic below)? I guess ParametricLaplace and FunctionalLaplace have all public methods listed in the menu on the left because they are first-level children of BaseLaplace, is there an easy way to change that?

Regarding code style, I think you followed most conventions except in some places. For now, we use 'string' types of strings, I think you also did. Further, we try to stay below 100 linewidth but I am not sure if it's even consistent atm. We might do that automatically in #63 soon anyway so you don't need to worry so much as long as it's just linewidth and minor differences.

Main changes since last time:

addressed underfitting of FunctionalLaplace in regression_example.py, fixed the issue with the marginal likelihood: https://github.com/AlexImmer/Laplace/pull/55#discussion_r833521031
added calibration_gp_example.py where FunctionalLaplace is used on a pre-trained FMNIST classifier (same experiment as in linearised Laplace paper)

I also benchmarked the code with BNN-predictions-repo in terms of speed and the code here is 2-3x slower. After investigating, I believe the main reason is that code here allows one to only use smaller batch sizes before running into memory issues when doing GP inference (b=128 here compared to e.g. b=512 in the BNN-predictions-repo).
When it comes to memory, the bottleneck seem to be torch.einsum functions used in the kernel methods in FunctionalLaplace. To get around this, I also tested implementation without torch.einsum. However, then the memory issues started to pop up in the backpack backend.
Further benchmarking and faster implementation can be left for future PRs imo.

Replaced by #192


Response schemas have query parameter as properties

The following properties should not be included in the response features as they are not included in the OMSF Observation feature types:

place
latlon
lonlat

It's ok to still have these as query parameters.

They are not included by the backend. Sampo ?

I think this is not (purely) a backend issue, collection must have those properties set in order to be used as query parameters. Sampo ?

Ok, I was not aware of the core limitation on collection specific query parameters (=must match property names). @sampov2 is working on a change in this in the core

I've released version 0.4.0 of the core (sofp-lib version 0.3.2) where each collection can set up additional query parameters (https://github.com/vaisala-oss/sofp-lib/blob/master/src/collection.ts#L34). When these parameters are in the data query string, core produces an AdditionalParameterFilters for the backend (https://github.com/vaisala-oss/sofp-core/blob/master/src/server/filters/additional_parameter_filter.ts). This design separates collection properties that need to have a one-to-one equivalency with query string parameters and the data model and these cases where we want a query string that is used by the API but is not necessarily connected to the data model.
Please note that the default implementation for the AdditionalParameterFilter is to throw an exception - the server cannot know what to do with them. This mandates the backend to take care of the filter and not pass it back to the core.

backend now uses AdditionalParameterFilters


Improve 2Fa workflow

Fixes #ISSUEID
Add description here.

It has a mess of commits, closing it on the behalf of #2865


Check token

Hi,
I get the message with the Ecosia script, that I should check the entered token. I installed the Ecosia script with Widget Hub. This was working well. Then log into Ecosia with my standard browser. Used the shortcut to extract the token and put it into the widget parameter. But no mater if I use this way or do it manually or try it with vpn I always get the same error message. That error is the message from Line 42 in the script.

@FulytheFox thanks for the pull request. I will accept it in the next minutes.


Extend workload template to support sub-op templates in a more uniform way

In order to allow for compound operations, known as batch statements in CQL and under other names in other protocols, the workload template format needs to be extended.
This will allow for the separate scenarios single operations and compound operations to be expressed distinctly, checked for structural coherence, and mapped directly to the correct mapper interface.
The current supported data structures only recognize "op" as a specific field within the ParsedOp. This needs to be extended to support multiple ops.
Example syntax:
blocks:
 block1:
  ops:
   op1:
    sharedfield: "{shared}"
    bindings:
     shared: NumberNameToString();
    ops:
     part1: "asdf"
     part2: "zxcv"

In this case, an additional occurence of "ops" is allowed under the op template. This should be desugared to the equivalent:
blocks:
 block1:
  ops:
   op1:
    sharedfield: "{shared}"
    bindings:
     shared: NumberNameToString();
    op:
     ops:
      part1: "asdf"
      part2: "zxcv"

... the differnce being that the second form adheres to the extant requirements that all op fields are located underneath "op" or "params" when specified, but that all other op template level fields are either reserved words or unreserved words which are automatically moved into one of these properties.
In both cases, it is implied that the op template level bindings, tags, params and so on are cascaded to each of the interior op templates just as they would be at higher levels in the doc.

This has been added using some conventional patterns. The ParsedOp API has been extended to make it easy to iterate over sub-template definitions. See the batch support the CQL driver for an example.


[Containers]Launching a Node.js Application on a Kubernetes Cluster

Topic Suggestion
Proposed title of article
Launching a Node.js Application on a Kubernetes Cluster
Proposed article introduction
Kubernetes is a portable, adaptable, open-source platform for managing containerized workloads and services that enables declarative configuration and automation, according to the description. It is, therefore, essentially a container orchestration tool.
In this tutorial, we'll use **kubeadm to construct a Kubernetes cluster, then create a Hello World Node.js application and dockerize it. After that, we'll tell Kubernetes to generate two replicas and a load balancer and deploy it to our cluster.
Key takeaways

What is the purpose of Kubernetes?
Kubernetes Components
Kubernetes installation
Creating the application
docker hub ID
created a node.js application in our local system and then containerized it using docker.

References
N/A
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

@LinusMuema
In my article, I will use kubeadm to construct a Kubernetes cluster, then create a  Node.js application and dockerize it.
I will also use orchesterisation.


username not present in the on_client_gone and on_client_offline webhooks

The webhook bodies for on_client_gone and on_client_offline do not contain username field. I would like some context around why this decision was taken and if adding it would cause some unwanted effects

@dumbpreacher mainly because the ClientID is guaranteed to be unique not the Username. When multiple MQTT clients are using the same Username, the information isn't that useful.

@ioolkos thanks a lot for the quick response. But I have a doubt around why some webhooks contain the username and some don't (specifically the ones that are called when the client disconnects). Is this intentional? If I add the username in the on_client_offline or on_client_gone for my specific usecase, will it cause any side effects in the desired behavious?

@dumbpreacher decisions like that are a balance between what you can do and should do. We could obviously add everything to hooks with the philosophy that someone might have a use case. OTOH all the other use cases will see increased payload sizes, traffic etc.
For your specific question: why not just test it instead of me having to think and "guarantee" that there will be no side-effects :)

Thanks @ioolkos, will test this and get back.


test: Port tests in references.rs to sqllogictest

Which issue does this PR close?
Closes #8211
Rationale for this change
Port SQL tests from Rust to sqllogictest
What changes are included in this PR?
Remaining unit tests ported
Are these changes tested?
Yes
Are there any user-facing changes?
No

Hi, @simicd , there is a conflict in this PR.

Hi @jackwener, thanks for the hint - fixed the merge conflict and pushed an update, currently CI is running


Adding Scroll feature to cookie box

Added scroll feature to cookie box and reduced the font size for better access
Related Issue
adding scroll feature in cookie and reducing font size
Fixes:  #3487
Description
The scroll feature in the cookie consent box allows users to view all the content within the box without it taking up too much space on the screen. If the content exceeds a specified height, a scrollbar appears, enabling users to scroll through the text easily. This ensures that the cookie consent remains compact and unobtrusive while maintaining access to all necessary information.
@anuragverma108  please assign my pr the labels for GSSOC'24EXTD, hacktoberfest'24 and level labels

#3487
Type of PR

[ ] Feature enhancement

Screenshots / videos (if applicable)
https://github.com/user-attachments/assets/5cf88f2d-f91b-46b2-857c-77c6d78e0320
Checklist:

[ ] I have made this change from my own.
[ ] My code follows the style guidelines of this project.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] I have tested the changes thoroughly before submitting this pull request.
[ ] I have provided relevant issue numbers and screenshots after making the changes.

work with updated branch @Chitrakadyan2005


Bundling typebox?

dist/bun/index.js seems to include a full copy of @sinclair/typebox. It breaks TypeSystem.Type, TypeSystem.Format etc since elysia uses an internal typebox instance. elysia/type-system exports the global typebox instance so there is no way to access the elysia typebox.

For me its breaking my builds, I'm getting a lot of TypeBoxError: Duplicate type kind 'Files' detected. I don't know if this was the best idea

Elysia now excludes TypeBox from bundling in 0.8.7.
Feels free to reopen the discussion if need.

Reopen because when TypeBox is excluded from Bun bundle causing an error (see 0.8.7)
If added back (only change in 0.8.8), it seems to be working again.
Not sure why this happen but seems like we can't find a way to do that just yet.


utils/export: add MP4 / AV1 4:2:0 profile

Should we also add yuv444 ?

Yeah please add 444

Yeah please add 444

Done, also adjust the comment regarding the 4:2:0 compat with hw decoders.

Yeah please add 444

Done, also adjusted the comment regarding the 4:2:0 compat with hw decoders.

I removed the yuv444p profile, it is not supported by svtav1.


Add and update configurations for Microsoft.Data.Sqlite.Docs

Delete folder msdata;
Update docfx configurations;
Update publish configurations.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

docfx.json
:white_check_mark:Succeeded

msdata/TOC.md
:white_check_mark:Succeeded

msdata/docfx.json
:white_check_mark:Succeeded

msdata/index.md
:white_check_mark:Succeeded

msdn.4.5.2.zip
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.gitignore
:white_check_mark:Succeeded

.openpublishing.publish.config.json
:white_check_mark:Succeeded

TOC.md
:white_check_mark:Succeeded

docfx.json
:white_check_mark:Succeeded

index.md
:white_check_mark:Succeeded

msdata/TOC.md
:white_check_mark:Succeeded

msdata/docfx.json
:white_check_mark:Succeeded

msdata/index.md
:white_check_mark:Succeeded

msdn.4.5.2.zip
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fixes null @ currentBatch due flush() calls @ LightPipeline.js

This PR: Fixes a bug
Test scenario:

Mesh gameobject -> .setPipeline('Light2D')
Applying a rotation to the mesh
The test might reveal an exception/crash due null values at currentBatch property of LightPipeline.
Crash flow: MeshWebGLRenderer -> setGameObject -> setNormalMapRotation -> flush

Rationale: setGameObject @ MeshWebGLRenderer invoking a flush @ setNormalMapRotation and making currentBatch=null.
The crash happens @ MeshWebGLRenderer  once reading a property on currentBatch (null).

@greptileai

@urueda I am not able to reproduce the bug mentioned. Here is a link to the test code:
https://labs.phaser.io/view.html?src=src\bugs\6822 mesh null currentbatch.js
It would be great if you could provide a link that demonstrates the bug you mentioned so that we can investigate further. Thanks.

@zekeatchan I suggest:

a test code over Phaser version 3.70.0 (bug was found there)
increasing the rendering complexity of the scene

For the bug to reveal you would need a batch flush to happen when rendering the mesh.
Perhaps, you could repeat the track-race mesh many times to increase the chances for the crash to happen (reading a property on null batch).

We don't test against 3.70, as 3.80 is the current version.
The Mesh we tested with has 10,136 faces, which is about the limit of the number of faces a Mesh should have in Phaser 3. The batch size is 4096 by default. I forced it down to 1024 in the config it still doesn't cause a crash, even though you can clearly see it flushing multiple times during the single draw in Spector.gl.

As it stands, unless you can provide us with a reproducible test case against v3.80 then we can't move ahead with this issue.

Please, find next a test code revealing the bug: https://phaser.io/sandbox/xv7EdA2P

The sandbox is what should have been provided at the very start, really. You don't need to call setInterval. You don't even need to create a complex batch. All you need is a single object on the display list, followed by a mesh (of any size) and then immediately call setRotation on it. Like this:
class Example extends Phaser.Scene
{
    constructor ()
    {
        super();
    }

    preload ()
    {
        this.load.setPath('https://labs.phaser.io/assets/obj/racing/');
        this.load.obj('roadStart', 'roadStartPositions.obj', 'roadStartPositions.mtl');
    }

    create ()
    {
        this.add.rectangle(50, 50, 50, 50, 0xff0000, 0.25).setPipeline('Light2D');

        const track = this.add.mesh(400, 300);

        track.setPipeline('Light2D');

        const rot90 = Phaser.Math.DegToRad(90);
        const rot180 = Phaser.Math.DegToRad(180);

        track.addVerticesFromObj('roadStart', 1, 0, 0, 0, rot90, rot180);

        track.setRotation(Math.random());
    }
}

const config = {
    type: Phaser.AUTO,
    width: 800,
    height: 600,
    backgroundColor: '#0a440a',
    parent: 'phaser-example',
    scene: Example
};

let game = new Phaser.Game(config);

setRotation is important. Using one of the rotate properties, like rotateX doesn't cause the crash.
Comment out the rectangle before it, and it doesn't crash. Remove the Light2D pipeline from the rectangle and it doesn't crash. Add a sprite or text or anything not using Light2D before the Mesh, and it doesn't crash.
The combination is very specific: Something using Light2D immediately before a Mesh that uses Light2D, and you have to then immediately invalidate the Mesh to trip the bug.

Thanks for opening this issue, and for submitting a PR to fix it. We have soft-merged your PR into the master branch and attributed the work to you in the Change Log. If you need to tweak the code for whatever reason please submit a new PR.


why is dispatch_barrier_sync instead of dispatch_barrier_async in addProgressCallback:andCompletedBlock:forURL:createCallback?

Could anyone help me explain why the author use dispatch_barrier_sync instead of dispatch_barrier_async in the line 186 of SDWebImageDownloader.m (addProgressCallback:andCompletedBlock:forURL:createCallback) ?
Because actually these codes are running on main thread as a fact of system optimization and there are not other codes wait to execute and variables to return.

I haven't written that code, but I assume it's because we need to return token which is created on the barrierQueue. Mutating URLOperations is not thread safe, so we are synchronizing the that using the barrierQueue.

Thank you for the reply, but I think that's not the function I was asking for. In latest code of "addProgressCallback: completedBlock: forURL: createCallback", there is really a token need to be returned. but I was asking "addProgressCallback:andCompletedBlock:forURL:createCallback", which maybe has been deleted. I paste the following code in version 3.7.1, there was no token to return or code wait to execute.

(void)addProgressCallback:(SDWebImageDownloaderProgressBlock)progressBlock andCompletedBlock:(SDWebImageDownloaderCompletedBlock)completedBlock forURL:(NSURL *)url createCallback:(SDWebImageNoParamsBlock)createCallback {
// The URL will be used as the key to the callbacks dictionary so it cannot be nil. If it is nil immediately call the completed block with no image or data.
if (url == nil) {
if (completedBlock != nil) {
completedBlock(nil, nil, nil, NO);
}
return;
}
dispatch_barrier_sync(self.barrierQueue, ^{
BOOL first = NO;
if (!self.URLCallbacks[url]) {
self.URLCallbacks[url] = [NSMutableArray new];
first = YES;
}
  // Handle single download of simultaneous download request for the same URL
  NSMutableArray *callbacksForURL = self.URLCallbacks[url];
  NSMutableDictionary *callbacks = [NSMutableDictionary new];
  if (progressBlock) callbacks[kProgressCallbackKey] = [progressBlock copy];
  if (completedBlock) callbacks[kCompletedCallbackKey] = [completedBlock copy];
  [callbacksForURL addObject:callbacks];
  self.URLCallbacks[url] = callbacksForURL;

  if (first) {
      createCallback();
  }

});
}

I don't know the details of that code in particular, but it's probably synchronizing the whole operation.


Integrate kbatch

Fixes | Closes | Resolves #

Please remove anything marked as optional that you don't need to fill in.
Choose one of the keywords preceding to refer to the issue this PR solves, followed by the issue number (e.g Fixes # 666).
If there is no issue, remove the line. Remove this note after reading.

Changes introduced in this PR:

Types of changes
What types of changes does your PR introduce?
Put an x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features to not work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing
Requires testing

[x] Yes
[ ] No

In case you checked yes, did you write tests?

[ ] Yes
[ ] No

Documentation
Does your contribution include breaking changes or deprecations?
If so have you updated the documentation?

[ ] Yes, docstrings
[ ] Yes, main documentation
[ ] Yes, deprecation notices

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

[x] Finish integrating dask-gateway
[ ] Enable kbatch to run in insecure mode (no SSL cert)
[ ] Add overrides mapping
[ ] Add documentation

cc @Adam-D-Lewis @costrouc

@costrouc @Adam-D-Lewis I think this PR is ready for a final review :)


<EMAIL_ADDRESS>decompress error

ERROR: Cannot create symbolic link : A required privilege is not held by the client. : C:\Users***\scoop\apps\twilio\5.21.1\twilio\node_modules\acorn-walk\node_modules.bin\acorn

I guess I figured this out myself. Re-open Powershell as an administrator, then execute the install.


Allow showing bitmap or text overlay and other effects  for specific duration

[REQUIRED] Use case description
Allow showing bitmap or text overlay and other effects   for specific duration Say for example we have a 50 second long video and we want to show a bitmap or text overlay from 10th second to 20th second , this feature should be supported
This feature should be available for

bitmap overlay
Text overlay
Video (PIP) overlay
Effects using OpenGL
Music overlay

Proposed solution
Similar muxer solutions are already available

https://github.com/linkedin/LiTr
https://github.com/natario1/Transcoder
The google Youtube Create app  already provides such solutions
https://play.google.com/store/search?q=youtube create&c=apps

Alternatives considered
the above mentioned muxer can be used but inbuild solution will be helpful .
This  missing features limit the option to develop a better video editor app compared to existing competition.

Thank you for the request. These features are on our roadmap. Please follow this issue for more updates.

A Time Frame of development  will be helpful , please include these important features at the earliest, after studying various codes on github it seems to be a muxer related issue
these might be useful
https://www.sisik.eu/blog/android/media/mix-audio-into-video
https://www.sisik.eu/blog/android/media/add-text-to-video
specially handling duration etc

We are unable to share a specific deadline for this feature. We will provide you with a more specific timeline as soon as we can.

I read the
OverlaySettings.java
in the main branch compared to 1.1.1 release and it provides various options like
useHdr
alphaScale
backgroundFrameAnchor
overlayFrameAnchor
scale
rotationDegrees
If they can add  further options to manage these changes with time  based on

how much time (in terms of seconds ) we can use the rotation degrees or scale , alpha etc
based on background time at what time we start and end showing the overlay.

It will be really helpful which can help us to show animations of the overlay with these features,


can I specify that the image resolution ratio is 720x1440?

This is for bugs only
Did you already ask in the discord?
Yes/No
You verified that this is a bug and not a feature request or question by asking in the discord?
Yes/No
Describe the bug

yes, no problem. Go inside ./ai-toolkit-main/toolkit/buckets.py file, edit the resolutions_1024 list and you can simply customize your resolution.


ds18b20 error compilation

When I compiled example for DS18B20 from OneWireNg in this spring (for Attiny3216), everything was fine. Now it ends up compiling with an error. Can you figure out why? Some bug in the library?
C:\Users\radek\AppData\Local\Temp\cccalydd.ltrans0.ltrans.o: In function UartClass::begin(unsigned long, unsigned int)': <artificial>:(.text+0x8ee): undefined reference to operator delete(void*, unsigned int)'
collect2.exe: error: ld returned 1 exit status
exit status 1
Nastala chyba při kompilaci u desky ATtiny3216/1616/1606/816/806/416/406 w/Optiboot.

UartClass::begin() is your code? How it looks like?

Please post also core name and version eventually link to core and sketch with error.

I did a clean install of the Arduino IDE 1.8.16. Using the "json" link I installed the current megaTinyCore from SpenceKonde.In the sketch folder  I copied the unzipped current version of OneWireNg.  In its Examples folder is the DallasTemperature sample project. I opened that and did a compilation for attiny3216/Optiboot (it doesn't work in general for any circuit). The compilation ended with errors. In the spring I also tried it for attiny3216, everything was fine then. I don't know where the problem is. Maybe in the core, maybe in the library...

Some insight into this issue is here. What is the json link you use to install the core?

This is an issue in megaTinyCore core. I compiler it by adding this lines at the end of megaTinyCore/hardware/megaavr/2.4.2/cores/megatinycore/new.cpp:
void operator delete(void *ptr, unsigned int) {
  free(ptr);
}

void operator delete[](void *ptr, unsigned int) {
  free(ptr);
}

also add these declarations to new.h (in the same dir):
void operator delete(void *ptr, unsigned int);
void operator delete[](void *ptr, unsigned int);

After this the example compiles successfully. Please report this as a bug to megaTinyCore and provide link to this issue here.

That's great. It's working. I followed your advice and modified the core. Then I compiled the project without errors and Dallas DS18b20 is working. :-) Thanks a lot.

This is an issue in megaTinyCore core. I compiled it successfully by adding these lines at the end of megaTinyCore/hardware/megaavr/2.4.2/cores/megatinycore/new.cpp...............

You are welcome. Finally @SpenceKonde will need to look at this and fix it in the core.

Hm, we already have
void operator delete(void * ptr);
void operator delete[](void * ptr);

Are those correct and we also want the versions that also take an unsigned int? What does the unsigned int do?  It looks like a dummy argument than will never be used.... doesn't that generate compiler warnings if we don't give the second argument the unused attribute, or am I missing something here? That the core compiles without warnings is a requirement for all releases.
This is the second time that issues with "missing" lines have been reported, so I'd like to make sure that file has everything it ought to and nothing it oughtn't.. Where is the reference for what should be defined here., and  the documentation for it which explains what these operators do? I will freely admit that I am not particularly competent regarding most of the stuff that's in  cpp but not c. I mostly write pure C at a very low level, and am comfortable with inline assembly, but as soon as classes or malloc get involved, things seem to get a lot more confusing....

I didnt go into details, but it looks like a new gcc changed something in its includes and it ends up by compilation error in the core. This is the information I got from the link mentioned above. It also contains a reference to C++ spec. regarding this extra arg.
If you would need some more detailed insight into this issue I may look at this topic more thoroughly but in the next week.

Thank you, the cppreference article there has definitely granted some insight on why this became a problem when it previously wasn't.

Root cause of the issue is described here. Library fix provided by #21.


parse error with UPPER field

We are using golang spansql 1.17.0
if we  parse the following sql (that is generate by google)
CREATE TABLE resourceWorkschedule (
resourceWorkscheduleId STRING(MAX) NOT NULL,
labelUpper STRING(MAX) NOT NULL AS (upper(label)) STORED,
send_userRoleId STRING(MAX) NOT NULL,
send_version TIMESTAMP NOT NULL OPTIONS (
allow_commit_timestamp = true
),
) PRIMARY KEY(resourceWorkscheduleId);
we got following error / got "(" while expecting ")"
the problem is related by  AS (upper(label)) STORED , I believe
is this possible to solve this ?

i am sorry, this is not correct


deployed documentation is not working

documentation in the readme.md pointing to https://sphinx-wagtail-theme.netlify.app/ is HTTP 404

We recently migrated to Vercel for hosting, I'm guessing you are referring to the GitHub link which now probably put of date.
I'm not sure on the correct URL but this preview URL was generated on the PR that adopted Vercel.
https://sphinx-wagtail-theme-wtojdzah7-wagtail.vercel.app/
https://github.com/wagtail/sphinx_wagtail_theme/pull/240

Build Failed
Command "npm run build && python3 -msphinx -M html docs/ _build" exited with 2
...
Could not import extension sphinx.builders.linkcheck (exception: urllib3 v2.0 only supports OpenSSL 1.1.1+, currently the 'ssl' module is compiled with 'OpenSSL 1.0.2k-fips  26 Jan 2017'. See: https://github.com/urllib3/urllib3/issues/2168)

Vercel build is still failing.

Closing in favour of https://github.com/wagtail/sphinx-wagtail-theme/issues/265, as the original issue is about the Netlify -> Vercel migration, and as far as I can tell preview builds were working fine from November 2022 till May 2023.


Enable TypeScript compilation for PRs

This will enable TypeScript compilation for PRs.
Signed-off-by: SaeHie Park<EMAIL_ADDRESS>

For #51


flutter: Install Android variant artifacts in platform subdirectories

Description of changes
This fixes the installation location of build-variant-specific Android engine artifacts in the Flutter SDK.
Closes #231247.
(CC: @NixOS/flutter)
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2196


Unable to load DLL 'OpenPoseDotNetNative'

Hi takuya-takeuchi,
I have been able to build and run the original OpenPose C++ on my new laptop. I have since been trying to get the OpenPoseDotNet to work. I have taken some code from one of the examples, built it, but when I run it I get the dreaded "Unable to load DLL" or one if its dependencies. I located a "Dependencies" tool and loaded the DLL into it. Here's the link for the tool https://lucasg.github.io/Dependencies/ The old depends.exe does not work. So I'm seeing some DLL's that I don't have on my system... 'api-ms-win-core-wow64-11-1-1.dll'...and others. Since I've been communicating with you, I've been seeing this problem, but thought I was not building correctly.
My research indicates that these DLL's have been eliminated in Windows 10, but I don't understand why it builds! I'm still researching.
I assume you are using Windows 10 1909 release? All my systems are on this release.

The dependencies view...

Thanks,
Bruce

I have no clue why this error is happening. I'm trying to take some code from one of the examples and load it into my .NET application using your library. Basically I'm trying to do what OpenPose demos does (the one from CMU labs).

@bruce987
Very sorry late reply. We world are just focusing to beat threat of COVID-19. Thank you for your patient.

I assume you are using Windows 10 1909 release? All my systems are on this release.

Yes. My dev machine is latest windows.
The following is result of dependencies for OpenPoseDotNet.Native.dll.

But there is no dependency issue.
This weird issues is reported in https://github.com/lucasg/Dependencies/issues/102
If you can, you can upload your build OpenPoseDotNet.Native.dll and I can check dependencies and work on my machine.
Thanks.

@takuya-takeuchi
Yes I'm on Windows 10 1909 release. My investigation indicates that the -wow- dll's are no longer installed on Windows latest release, which I verify is accurate.
I had to change the extension to txt, dll is not supported download type.
OpenPoseDotNetNative.dll.txt

@bruce987
This is your built library.

However, there is no issue.
Uhh.. My dev machine is not installed as clear install.
It may mean that -wow- dll's remains from old version windows.
So I should create new 1909 machine as Hyper-V and test it.

If you open the first link and drill down you might see an error. The interesting thing is the OpenPose library shows the same issue, but runs without issue... that’s what has me baffled!
Sent from my iPhone

On Apr 11, 2020, at 9:41 PM, Takuya Takeuchi<EMAIL_ADDRESS>wrote:

@bruce987
This is your built library.
However, there is no issue.
Uhh.. My dev machine is not installed as clear install.
It may mean that -wow- dll's remains from old version windows.
So I should create new 1909 machine as Hyper-V and test it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@takuya-takeuchi ,
Since I was able to load and run your 'examples\TutorialApiCpp\01_BodyFromImageDefault'; I took that build libraries and put them into my application that I'm building. This runs basically without issues.
...but I have another question which I will open a new issue.
-Bruce


Forced location

Another Widget should no longer require the location permission for displaying weather, since you added a way for manually selecting the location.

Fixed in version 2.0.1.


Update skeleton to Bootstrap 3.0.0; 2. Add new html file into views t..

Update skeleton to Bootstrap 3.0.0  and update related html's CSS.
Update jQuery to 1.10.2
Add nav.html, sidebar.html and skeleton.html into app/views directory, to minimize html repetition by including skeleton.html.
Reformat all html to generate neat html source code.

We'll update bootstrap and jquery in a new PR and target the correct branch.