Add initial tracker blocking support to Chrome MV3 extension

Initial Chrome MV3 tracker blocking support using the
declarativeNetRequest API. Note that blocked trackers are not yet
reported properly in the popup UI or our devtools panel, though the
necessary ruleId -> tracker domains lookup is being stored ready.

Reviewer: @ladamski
Steps to test this PR:

Build the Chrome MV3 extension: npm run dev-chrome-mv3
Install that in Chrome.
Ensure tracker blocking works (e.g. browse to a news website or our test page and ensure requests are blocked).
Ensure the unit tests and MV3 integration tests pass: npm run test and npm run test-int-mv3

Note: It is expected that blocked trackers are not listed in developer tools panel nor the popup UI yet.
Automated tests:

[x] Unit tests
[x] Integration tests

Reviewer Checklist:

[ ] Ensure the PR solves the problem
[ ] Review every line of code
[ ] Ensure the PR does no harm by testing the changes thoroughly
[ ] Get help if you're uncomfortable with any of the above!
[ ] Determine if there are any quick wins that improve the implementation

PR Author Checklist:

[ ] Get advice or leverage existing code
[ ] Agree on technical approach with reviewer (if the changes are nuanced)
[ ] Ensure that there is a testing strategy (and documented non-automated tests)
[ ] Ensure there is a documented monitoring strategy (if necessary)
[ ] Consider systems implications

So if this is merged, would the default build now use mv3 or its just merged but essentially inert in the release build? Obviously I'm concerned that we don't regress CTL (or anything else)

Right, it's inert unless you go out of your way install a chrome-mv3 build with npm run dev-chrome-mv3, or run the integration tests against MV3 with npm run test-int-mv3. Since the manifest version is 2 otherwise, the tdsStorage.onUpdate() listeners should never be created and the module won't do anything.

@ladamski I've updated this again, so now we're also building a configuration for the extension configuration (so far just the tracker allowlist) and I've increased the ddg2dnr dependency to include the recent improvements. I'm pretty much happy with the changes now.


Update expat dependency

Our version of expat hasn't been updated in over a year, and is once again being flagged by security analysis tools. Based on discussion in https://github.com/flutter/flutter/issues/91384 it sounds like this is probably not an actual security issue the way we're using it (unless something has changed about our usage), but we should still roll it forward again.
(/cc @Hixie @jason-simmons @zanderso who were in involved in the previous roll, but I can drive this if there's not a better candidate.)

Autorolling it would be ideal if there's any chance of that.
Am I wrong in saying we also have libxml2 in our binary somewhere?


Enable multiple file extensions

References #54, detect and load .yml by default.

Nice

What if I have 42.yaml and 42.yml on the same 'level'?

What if I have 42.yaml and 42.yml on the same 'level'?

The use of itertools.product would read them both, in a deterministic order (and hence precedence). Whether that was meant to happen is another question. We can simply state that that's a problem for the user, but maybe we should involve a user in the 'expected behaviour' discussion? :) What would you expect to happen?

I would expect to have an error 😅
It seems like the user has two different configurations that should be treated equally. I would want to know that I'm doing something stupid...

After some discussion, this PR maybe catering to a use case we'd want to discourage and not support like this. Users mixing .yml and .yaml files can load both and combine them in any way they see fit, confidence should maybe not do this out of the box. Agreed? :sweat_smile:

Totally agree, I cannot think of a use case that warrants such iffy behaviour

On second thought, let's not go to PR, 'tis a silly place.
Closes #54


Add support for extracting PathBuf from pathlib.Path

datafusion probably wants this: https://github.com/apache/arrow-datafusion/issues/491


How to set default DM

Hi, I'm use Debian Unstable. I want to set default DM as ly. According to the Debian wiki just need to run dpkg-reconfigure ly command. But it's not working.
Error:

I downloaded and installed the package from this link.

If you installed a .zip file, that's a .deb file, so you didn't install a Debian package, which is why that Debian tool doesn't work. I'm not sure which Debian repos have Ly, but once you find and set up that repo, the install command would then be sudo apt install ly. and then you could use the dpkg-reconfigure command on the package.

sudo systemctl enable ly and compile it from here

Thanks.

Thanks.

Also chmod +x .xinitrc if you have one or else you would just logout


AER load model still fails

@lcwong0928  @sarahmish
Following on closed issue #302
Unfortunately it is not working, I get a new error on loading the model, I pulled from github all changes, here is my code and the error.
train_data = load_signal('Honorable_Interview_00_02_04_EAR_R')

hyperparameters = {
    'mlprimitives.custom.timeseries_preprocessing.time_segments_aggregate#1': {
        'interval': 1
    },
    'orion.primitives.aer.AER#1': {
        'epochs': 3,
        'verbose': True ,
        'input_shape': [200, 1] #,
    }
  }

orion = Orion(
    pipeline='aer',
    hyperparameters=hyperparameters
)

if 1:
  orion.fit(train_data)
  orion.save("AER/temp00AER.pkl")
  with open("tempAER.pkl", "wb") as pickle_file:
      pickle.dump(orion, pickle_file)
else:
   orion = pd.read_pickle("tempAER.pkl")
   orion = Orion( pipeline='aer')
   orion=orion.load("AER/temp00AER.pkl")

(orionGPU3) yaser@yaserubuntu:/media/ydrive/time_series/Orion0.3.1$ !py
python yydebugAER.py
/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/sklearn/utils/validation.py:37: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  LARGE_SPARSE_SUPPORTED = LooseVersion(scipy_version) >= '0.14.0'
Using TensorFlow backend.
WARNING:tensorflow:From /home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/tensorflow/python/ops/init_ops.py:97: calling GlorotUniform.__init__ (from tensorflow.python.ops.init_ops) with dtype is deprecated and will be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
WARNING:tensorflow:From /home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/tensorflow/python/ops/init_ops.py:1251: calling VarianceScaling.__init__ (from tensorflow.python.ops.init_ops) with dtype is deprecated and will be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
WARNING:tensorflow:From /home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/tensorflow/python/ops/init_ops.py:97: calling Orthogonal.__init__ (from tensorflow.python.ops.init_ops) with dtype is deprecated and will be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
WARNING:tensorflow:From /home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/tensorflow/python/ops/init_ops.py:97: calling Zeros.__init__ (from tensorflow.python.ops.init_ops) with dtype is deprecated and will be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
Traceback (most recent call last):
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/pandas/io/pickle.py", line 166, in try_read
    return read_wrapper(lambda f: pkl.load(f))
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/pandas/io/pickle.py", line 148, in read_wrapper
    return func(f)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/pandas/io/pickle.py", line 166, in <lambda>
    return read_wrapper(lambda f: pkl.load(f))
  File "/media/ydrive/time_series/Orion0.3.1/orion/primitives/aer.py", line 76, in __setstate__
    state[network] = tf.keras.models.load_model(fd.name)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/tensorflow/python/keras/saving/save.py", line 146, in load_model
    return hdf5_format.load_model_from_hdf5(filepath, custom_objects, compile)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/tensorflow/python/keras/saving/hdf5_format.py", line 212, in load_model_from_hdf5
    custom_objects=custom_objects)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/tensorflow/python/keras/saving/model_config.py", line 55, in model_from_config
    return deserialize(config, custom_objects=custom_objects)
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/tensorflow/python/keras/layers/serialization.py", line 89, in deserialize
    printable_module_name='layer')
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/tensorflow/python/keras/utils/generic_utils.py", line 192, in deserialize_keras_object
    list(custom_objects.items())))
  File "/home/yaser/miniconda3/envs/orionGPU3/lib/python3.7/site-packages/tensorflow/python/keras/engine/network.py", line 1152, in from_config
    layer_output_tensors = layer._inbound_nodes[node_index].output_tensors
IndexError: list index out of range

During handling of the above exception, another exception occurred:

Thank you @yaseryacoob, I think the issue is happening from the setup. Can you install from source?
git clone git://github.com/sintel-dev/Orion
cd Orion
make install # or pip install -e .

I made a google colab with the installation of the latest version and it seems to work as intended. Let me know if you have any further question.


still printing numbers (in 1.14 at least)

i thought i removed all of the s.o.println(Atum.setSeedAttempts), apparently not

must be another mod, cannot reproduce in dev


Java Troubleshooting Tutorial: error: class X is public, should be declared in a file named X.java

Write a short tutorial on Java's ClassNotFoundException error message, following the "How to troubleshoot Python ZeroDivisionError" example tutorial.
The tutorial should be organized as follows:
---
title: How to troubleshoot Java error: class X is public, should be declared in a file named X.java
author: your name
tags: [tutorial, Java, troubleshooting]
publicationDate: today's date
description: Learn how to error handle the Java error: class X is public, should be declared in a file named X.java
image: https://storage.googleapis.com/sourcegraph-assets/learn/headers/sourcegraph-learn-header.png
imageAlt: Sourcegraph Learn
browserTitle: error: class X is public, should be declared in a file named X.java in Java error handling
type: posts
---

Introduction with the error message

## Reproducing the error

Show how the error can be created.

## Solution 1

Provide a possible solution.

## Solution 2

Provide another solution.

## (optional) Solution 3

Optionally provide a third solution.

## Learn more

Search across open source Java repositories that have the `error: class X is public, should be declared in a file named X.java` to understand the message more.

<SourcegraphSearch query="error: class is public, should be declared in a file named .java lang:java" patternType="literal"/>

Check out more Sourcegraph Learn tutorials on [Java](https://learn.sourcegraph.com/tags/java).

If this seems like a tutorial that you can write effectively for the developer community, please follow our Hacktoberfest 2021 contributor guidelines to proceed.

I think I can do this one! :)


feat: Check script permissions in a GitHub action

Fixes b/325560863

I don't think I can get the workflow to run in a PR from my fork to this one - if you're happy that it looks broadly okay, I suggest we merge as-is, and then I create a new PR which just deliberately introduces a "bad" script... we can check that that breaks the build, and then close the PR.

Note that both this and "check the API catalog" are very small workflows. We could potentially introduce a small "extra checks" workflow containing both of them instead.


tinymce_placeholder is causing issues with views using tinymce editor

I'm using chatter version 0.2.13
tinymce_placeholder is causing issues with views using tinymce editor, see image below.
I think the extra , after the trans call and before the . is causing this issue in chatter/src/Lang/en/messages.php
'tinymce_placeholder' => 'Type Your '.trans('chatter::intro.titles.discussion'),.' Here...',

This is fixed now in 0.2.14

Thank you. Just tested with the new 0.2.14 update and it now working. Thanks for the quick response.


Consider interacting with the network off the calling thread

When retrofit added suspend support, its suspending functions ran the actual network interaction off the calling thread. While it's fairly easy to wrap connect-kotlin generated suspend calls in a withContext dispatch, having that thread switch happen automatically seems less error prone. Since we use both buf and retrofit in our codebase, this is a relatively common source of confusion that often results in NetworkOnMainThreadExceptions.
Square's implementation uses their own executor, but using the built-in Dispatchers.IO might work too? I'm unsure which would be preferable.
https://github.com/square/retrofit/pull/2886

@jzbrooks, what version are you using? As of v0.5.0, you can set a property on the ProtocolClientConfig so that this dispatch happens automatically. The change was made here: #218.

Hey @jhump, that's exactly what I was looking for. I missed that note on the release. Thanks!

Any reason why this isn't enabled by default?

Any reason why this isn't enabled by default?

@erawhctim, at the time I added it, I was concerned about it being surprising behavior. Also it's not backwards compatible: if you have application code that did not configure this new property and explicitly dispatched to a particular context, the library under the hood would re-dispatch back to its default context, which may be surprising or problematic.
I'm certainly open to changing the default behavior though before we get to a v1.0.0 release. But I was thinking of making that change in the same release as other more significant (and backwards-incompatible) API changes, so that it's less likely to be overlooked when someone upgrades.

very wise; that makes a lot of sense 👍 Setting that as the default for v1.0 sounds ideal.


Added CellDecoratorSettings to give ability to change default behavior of cell_without_validator function

I was challenging to define the setting everywhere in my code when using this to turn off cache or always do flatten.
This way user will be able to change the default behavior of cell_without_validator by just updating the CellDecoratorSettings class variables.
For example: If the use do not want to use cache for some reason then (My use case is related to that)
This way cache can be turned off defaultly everywhere just by:
from gdsfactory.cell import CellDecoratorSettings
CellDecoratorSettings.cache = False

Instead of adding cache=False in every argument or clearing cache with gf.clear_cache() everywhere.

in principle, I'm not opposed, but if we do this, you should probably mimic the pattern we used for GDS and OASIS write settings... Add the CellDecoratorSettings to the PDK, and get the settings of the active PDK by default. The way you have it, getting the values from the class, rather than an instance of the class will be difficult to customize reliably
See
https://github.com/gdsfactory/gdsfactory/blob/main/gdsfactory/component.py#LL1770C39-L1770C39

Curious how this would be used if you don;t add to the pdk and plan on using it in multiple files, do you have to import and set cache to false ever single time...
Why not just add args to the cell_without_validator, and add cell settings there.
-Skandan

@drinwater, In my opinion

With the args in _cell: already present
With the args in cell_without_validator: we can also add this as well.
For the entire project: PDK will be a good way to change the default setting

The priority order for these setting can be in the order:  _cell > cell_without_validator > PDK

Looks good to me,
if you don't use the cache, don't you end up with lots of duplicated cells?
as far as I understand klayout will have issues opening the file and Calibre DRC won't run on your devices


Disqus Support

Hello, i was wondering if is there any support for Disqus in the post page? It would be neat to have it :)

That's sounds good.
This hugo theme has enough room to be customize by users.
If you have a disqus account, only you have to do is to put folloing code in single.html.

Please enable JavaScript to view the comments powered by Disqus.


Implement ability to generate TSMT proofs

This PR implements the ability to generate TSMT proofs which can be used to verify membership of a given key-value pair in the TSMT.

Huh, I didn't know that GH automatically changed the base branch:

I always opened multiple PRs with the same commits and add comments ... now I understand why you suggest to make a PR on top of another 😅


pr#1458

// Describe your PR here; eg. Fixes #issueNo
Enhance shenyu-sync-data-websocket module unit test，make its coverage rate is more than 80% pr#1458

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Hi， please check ci ,  ,welcome join!

I have been add the licenses and remove the empty line

@realcalu
Error:  src/test/java/org/apache/shenyu/plugin/sync/data/websocket/handler/WebsocketDataHandlerTest.java:[1] (misc) NewlineAtEndOfFile: File does not end with a newline.
Error:  src/test/java/org/apache/shenyu/plugin/sync/data/websocket/handler/WebsocketDataHandlerTest.java:[17] (whitespace) EmptyLineSeparator: 'package' should be separated from previous statement.
Error:  src/test/java/org/apache/shenyu/plugin/sync/data/websocket/handler/WebsocketDataHandlerTest.java:[37] (whitespace) EmptyLineSeparator: 'CLASS_DEF' has more than 1 empty lines before.
please  checked it

sorry! I haven been run "mvn clean install -Dmaven.javadoc.skip=true" to check the CI and I have been fixed the code format style.

Codecov Report

Merging #1557 (5170d74) into master (34f0997) will decrease coverage by 0.58%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1557      +/-   ##
============================================
- Coverage     63.96%   63.38%   -0.59%
+ Complexity     2235     2229       -6
============================================
  Files           449      449
  Lines          9402     9402
  Branches        949      949
============================================
- Hits           6014     5959      -55
- Misses         2916     2968      +52
- Partials        472      475       +3

Impacted Files
Coverage Δ

...enyu/client/tars/TarsServiceBeanPostProcessor.java
34.66% <0.00%> (-56.00%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
66.66% <0.00%> (-27.78%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
52.63% <0.00%> (-11.58%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
63.10% <0.00%> (-8.74%)
:arrow_down:

...in/alibaba/dubbo/cache/ApplicationConfigCache.java
75.86% <0.00%> (-4.60%)
:arrow_down:

...he/shenyu/plugin/grpc/client/ShenyuGrpcClient.java
9.09% <0.00%> (-4.55%)
:arrow_down:

...henyu/plugin/httpclient/NettyHttpClientPlugin.java
57.14% <0.00%> (-2.05%)
:arrow_down:

...nc/data/websocket/handler/AbstractDataHandler.java
83.33% <0.00%> (+75.00%)
:arrow_up:

...c/data/websocket/handler/WebsocketDataHandler.java
100.00% <0.00%> (+100.00%)
:arrow_up:

Continue to review full report at Codecov.

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

Hi welcome join,  please add  wechat  xixy199195


To Staging: Change to cloud-hosted-guide-containerize-podman

Updated by github actions, triggered by the guide-containerize-podman repo.

close

SNL not support podman yet


[Backport release-23.11] {tor,mullvad}-browser: 13.0.12 -> 13.0.13

Description of changes
Backport of #298196
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

Not sure why the Build Nixpkgs manual step failed, can we restart it somehow?

Superseded by #298330


compile-error on handler generation for Dictionary

I'm trying to generate the handler code for the org.bluez.LEAdvertisement1 interface containing some dictionary properties.
This is a snipped of the generated code trying to access Key and Value from the ManufacturerData dictionary instance:
case "ManufacturerData":
{
    MessageWriter writer = context.CreateReplyWriter("v");
    writer.WriteDBusVariant(new DBusVariantItem("a{qv}", new DBusDictEntryItem(new DBusUInt16Item(BackingProperties.ManufacturerData.Key), new DBusVariantItem(BackingProperties.ManufacturerData.Value))));
    context.Reply(writer.CreateMessage());
    writer.Dispose();
    break;
}

Introspection LEAdvertisement1
<?xml version="1.0" encoding="utf-8" ?>
<node>
	<interface name="org.bluez.LEAdvertisement1">
		<method name="Release"/>
		<property name="Type" type="s" access="read"/>
		<property name="ServiceUUIDs" type="as" access="read"/>
		<property name="ManufacturerData" type="a{qv}" access="read"/>
		<property name="SolicitUUIDs" type="as" access="read"/>
		<property name="ServiceData" type="a{sv}" access="read"/>
		<property name="Includes" type="as" access="read"/>
		<property name="LocalName" type="s" access="read"/>
		<property name="Appearance" type="q" access="read"/>
		<property name="Duration" type="q" access="read"/>
		<property name="Timeout" type="q" access="read"/>
	</interface>
</node>

Btw: Very nice library! ❤

Thank you ^^
Do you mind testing with the latest CI build ?
Seems like I messed up the parsing of dictionaries...

I tried using v0.0.15 from github artifacts but it produced the same output.

MSBuild can be pretty aggressive with caching, so perhaps the new version wasn't used.
I bumped the version which should prevent this, I'm sorry to ask again, but please try out this new version.

It works!
Sorry for keeping you waiting so long, I'm very busy at the moment.
I hadn't realized that v0.0.15 was also the latest version on Nuget, so msbuild fetched the version from Nuget instead of from my local directory.


Separate comment on page /03-fode-1.ipynb

On the solve_ivp section, the cell with "??f  # we defined this before" does not work properly because the comment is in the same line as the command.

thanks.


Forcing 2d arrays in robustlq.jl

@spencerlyon2 Testing the plots in this PR I couldn't run robustness/robust_monopolist.jl because of the following issue:
in lqcontrol.jl I added an internal function getZ to obtain a conformable element (matrix or number) Z to be passed to the function cond(). In order for robust_monopolist.jl to work following this change, we should then make C a Vector in this line https://github.com/QuantEcon/QuantEcon.applications/blob/master/robustness/robust_monopolist.jl#L61 (it is now a 2d array). However, when passing a 1d array, RBLQ here forces a 2d array. Therefore I don't see a way to make robust_monopolist.jl work unless we i) change getZ again ii) change RBLQ.
What would you suggest we do?

Hey @albep I would vote for adding one more method to getZ. The main reason is that I would like the "hard work" with the type system to be done inside the library so users don't have to think about it.
I believe the target audience for users of QuantEcon.jl is similar to the audience for the lectures website. That being the case, we don't want to make them think about the type system when we could handle it ourselves .
@jstac any comments here?

That sounds sensible to me.

@spencerlyon2 I definitely agree with you. Before I implement it, let's check how getZ would then look like. It now has 3 methods:
""

BB::Float64 : result of B' * B
gamma::Float64 : parameter in the Riccati equation solver
R::Float64

Returns

::Float64 : element Z in the Riccati equation solver
""

""

BB::Array{Float64, 1} : result of B' * B
gamma::Float64 : parameter in the Riccati equation solver
R::Float64

Returns

::Float64 : element Z in the Riccati equation solver
""

""

BB::Matrix : result of B' * B
gamma::Float64 : parameter in the Riccati equation solver
R::Matrix

Returns

::Matrix : element Z in the Riccati equation solver
""

I guess what you're saying is we should add another method which goes: if you're passing me R::Float64 and  BB::Array{Float64, 2}, I do the same thing as if you passed me a  BB::Array{Float64, 1}, i.e. R + gamma * BB[1]

Hmm that's a bit tricky. Do we know that this Matrix will only have one element?

If R is a Float, BB[1] must have only 1 element

Ok sounds good. I think that sounds good then. We can probably do this by just relaxing the function signature on the second method to be
getZ(R::Float64, gamma::Float64, BB::Union{Vector,Matrix})

instead of
getZ(R::Float64, gamma::Float64, BB::Array{Float64, 1})

Thanks @spencerlyon2, I implemented this in PR #135

Excellent, if this is now resolved will you please close the issue?

Sure, thanks!


RuntimeError: Device 0 does not have CUDA compute capability 3.0 or greater

While training I am getting this (CUDA 8, Ubuntu 16.4):
(.venv2) gamer@gamer:~/neon/simple_dqn$ ./train.sh Breakout-v0 --environment gym
2017-10-25 12:51:47,305 PyCUDA Runtime error: cuInit failed: no CUDA-capable device is detected
No handlers could be found for logger "gym.envs.registration"
Traceback (most recent call last):
  File "src/main.py", line 104, in <module>
    net = DeepQNetwork(env.numActions(), args)
  File "/home/gamer/neon/simple_dqn/src/deepqnetwork.py", line 34, in __init__
    stochastic_round = args.stochastic_round)
  File "/home/gamer/neon/neon/backends/__init__.py", line 113, in gen_backend
    deterministic=deterministic)
  File "/home/gamer/neon/neon/backends/backend.py", line 468, in allocate_backend
    return Backend.backends[name](**kargs)
  File "/home/gamer/neon/neon/backends/nervanagpu.py", line 732, in __init__
    check_gpu.ensure_gpu_capability(device_id)
  File "/home/gamer/neon/neon/backends/util/check_gpu.py", line 73, in ensure_gpu_capability
    "capability 3.0 or greater")
RuntimeError: Device 0 does not have CUDA compute capability 3.0 or greater
Exception AttributeError: AttributeError("'NervanaGPU' object has no attribute 'ctx'",) in <bound method NervanaGPU.__del__ of <neon.backends.nervanagpu.NervanaGPU object at 0x7fddda953e50>> ignored

This https://github.com/tambetm/simple_dqn/issues/3 does not help.
check_gpu outputs:

DISPLAY:neon:PyCUDA Runtime error: cuInit failed: no CUDA-capable device is detected
DISPLAY:neon:0

CUDA compute capability is different thing than CUDA version. You can see CUDA compute capabilities of different cards here: https://en.wikipedia.org/wiki/CUDA#GPUs_supported.

I have 1070gfx, which is rel. new and seems to be supported as "GTX 1070" (https://en.wikipedia.org/wiki/CUDA#GPUs_supported)

Can you run any of the Neon examples (https://github.com/NervanaSystems/neon/tree/master/examples)? If you get the same error, you should post an issue to Neon repo.


Incremental Java compilation is broken with Android 3.5.1 and Gradle 6.0

See https://issuetracker.google.com/issues/144011039
Changes to sources added by the Android Gradle plugin to AndroidJavaCompile.getSources() are ignored for incremental Java compilation in Gradle 6.0. When changing a source file, the output is something like this:
> Task :mylibrary1:compileReleaseJavaWithJavac UP-TO-DATE
Task :mylibrary1:compileReleaseJavaWithJavac in app Starting
Build cache key for task ':mylibrary1:compileReleaseJavaWithJavac' is 669111c814f7b237087e6a217a4ff93a
Task ':mylibrary1:compileReleaseJavaWithJavac' is not up-to-date because:
  Input property 'sources' file /home/user/test/mylibrary1/src/main/java/com/test/mylibrary1/LibClass.java has changed.

Note that there isn't a mention of the input property stableSources, which is actually tracked for changes. Not the property source of the SourceTask is mentioned.

cc @gradle/build-cache

See https://github.com/gradle/gradle/pull/11359 for the smoke tests.

Fixed via #11331.


Dynamic Host selection from Application Service Entry

(This is used to request new product features, please visit https://github.com/istio/istio/discussions for questions on using Istio)
We have a setup, where we have multiple application namespaces defined. As part of platforms team, we define egress rules(service entry, destination rule, virtual service) for every Azure resource in the application namespace, as well as in the istio-system namespace. But this way of working is repetitive and we want to avoid that. We have bunch of Azure resources application teams connect to from the AKS cluster.
It would be good to have something that will automatically detect the host name from the istio-proxy side car container and route traffic through egress gateway using the hostname, rather than manually setting up service registry and all in istio-system namespace. This way, we will have control as to what is cleared for connection from application namespace level and at the same time, we don't have to fiddle istio-system namespace changes.
We can have a common hostname defined in the istio-system namespace service entries, virtual services and destination rules, where the hostname values will be replaced from the istio-proxy side car host name in the run time.
We have outboundPolicy ALLOW_ANY. But that will skip istio-proxy side car altogether and it won't go through egress gateway it self. We don't want that option.
I tried to experiment on this idea, where as I defined the dts-demo-dev-kv.vault.azure.net KV in the service entry, destination-rule and virtualservice in application namespace. Whereas, I have defined host: www.vault.azure.net in istio-system namespace.
From istio-proxy, the connection is forwarded to the egressgateway in istio-system namespace. When it tries to resolve the host name to the IP, it resolves the hostname of www.vault.azure.net. But he IP thus received is not the IP for KV: dts-demo-dev-kv.vault.azure.net and hence it results in KV not found issue
istio-proxy:
[2024-03-24T21:54:46.155Z] "- - -" 0 - - - "-" 517 0 6 - "-" "-" "-" "-" "<IP_ADDRESS>:8443" outbound|443|kv|istio-egressgateway.istio-system.svc.cluster.local <IP_ADDRESS>:32978 <IP_ADDRESS>:443 <IP_ADDRESS>:55040 demo-dev-kv.vault.azure.net
egress-gateway
[2024-03-24T21:54:47.111Z] "- - -" 0 - - - "-" 1157 7333 1761 - "-" "-" "-" "-" "<IP_ADDRESS>:443" outbound|443||www.vault.azure.net <IP_ADDRESS>:40670 <IP_ADDRESS>:8443 <IP_ADDRESS>:43700 demo-dev-kv.vault.azure.net -
We are using Istio version 1.19.6 and Kubernetes Version: 1.27.7
Affected product area (please put an X in all that apply)
[ ] Ambient
[ ] Docs
[ ] Dual Stack
[ ] Installation
[x ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context

When it tries to resolve the host name to the IP, it resolves the hostname of www.vault.azure.net. But he IP thus received is not the IP for KV: dts-demo-dev-kv.vault.azure.net and hence it results in KV not found issue

Not quite understand, how do you access external service, what's your SE look like?

Hi @hzxuzhonghu
This is the service entry that we have in istio-system:
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: kv
  namespace: istio-system
spec:
  hosts:
  - "www.vault.azure.net"
  exportTo: ["."]
  ports:
  - number: 443
    name: tls
    protocol: TLS
  resolution: NONE
  location: MESH_EXTERNAL

This service entry is the one, which we have in application namespace:
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: kv
  namespace: demo
spec:
  hosts:
  - "demo-dev-kv.vault.azure.net"
  exportTo: ["."]
  ports:
  - number: 443
    name: tls
    protocol: TLS
  resolution: DNS
  location: MESH_EXTERNAL

https://istio.io/latest/docs/tasks/traffic-management/egress/egress-gateway/ Give a good example how to capture external traffic to egress gateway, you can set the VS host to *.azure.net
For the gateway, you can use tls passthrough mode
https://istio.io/latest/docs/tasks/traffic-management/egress/egress-gateway/#egress-gateway-for-https-traffic


Add initial testing foundation

This PR adds the initial testing foundation:

pytest
paramiko based mock SSH server

Closes #14

Additional note: I do not agree with all formatting rules applied. The still marked "formatting error" for the ternary operator does not make sense at all and mixes condition with return value for one but not both possibilities.

I don't really know, which rule was applied on that line, but I think, probably the line was to long with the else statement in the first line, so it did the line break before else?

Please also use type hints for method arguments and return values


[cert_manager] Migrate from integrations-extras

What does this PR do?
Migrates the cert-manager integration from integrations-extras.
I have not changed the code. All credit goes to @arapulido who created the integration and all the other contributors.
I just adapted a few things like the install instructions in the README, the maintainer email, etc.
This is my first time migrating an integration from the integrations-extras repo, so I have some questions:

I have set 1.0.0 as the version number and reset the CHANGELOG. Is that OK or should we continue from the version in integrations-extras?
I've changed the author to Datadog because I've seen that's what we use in the integrations of this repo. Is that OK or should we keep the original in this case?

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Can you add container integrations as codeowners for this?

@sarah-witt Thanks for the review.  I've addressed your comments:

Deleted requirements.in.
Deleted MANIFEST.in.
Updated setup.py with some things that were missing from the original one. I wasn't sure about what's no longer needed.
Removed the pip install -r requirements.in line from tox.ini.
Updated the datadog-checks-base dependency requirement to >=25.1.0. I don't know which version we should use exactly, so I chose this one which is the one that most integrations are using.
Added the spec in manifest.json.
Regenerated the example spec. My understading is that this integration should use openmetrics_legacy because I've seen that it accepts prometheus_url as a config param.

I've added each thing in a different commit.

Reference

Has this been addressed? I think we should still bump the version to generate a new release from core, but we should definetly preserve prior history

@hithwen I added a new commit that brings the changelog from integrations-extras and continues with the same version number.

The CI is failing because there's a check that detects that there's already an integration with the same name, which is expected because we know that it exists in the integrations-extras repo.
Apart from that, I pushed 3 new commits that were added after the last approval:

Migrate to OpenMetrics V2 (as suggested by @ofek ).
Adapt to changes in tooling.
Migrate manifest to v2.0.0 (it's now required to make the tests pass).


PDFDocument object conflicts with PDFKit.PDFDocument

When I embed UXMPDFKit into my project, even without importing the framework in a particulat file, PDFDocument object inside library cause Xcode fails to import new PDFKit.PDFDocument and mark it as unavailable. I suggest renaming it to UXMPDFDocument to avoid ambiguity.
Your Environment

iOS Version: 11.0
Xcode Version: 9.0
Swift Version: 4.0

@amosavian due to the introduction of PDFKit on iOS there is going to need to be a full re-write of the code to both not conflict, remove redundant code and support features previously not implemented. That is something that is going to take quite a bit of time. I went ahead and namespaced the files in the swift-4.0 branch for the time being however.

Thanks a lot ❤


ROMFS multicopter airframes don't call other airframes

Fixes https://github.com/PX4/Firmware/issues/10980

We need testing in the CI system needs to run through all airframes and verify basic functionality.


[FEATURE] Add more Backend Languages and resources

Add more Backend Languages
Currently, there are only 2 languages added in the backend roadmap's page. More needs to be added.
Languages to be added

[ ] PHP
[ ] Java
[ ] C#
[ ] Ruby
[ ] C++
[ ] C
[ ] Go
[ ] .NET

Task

create sections for each languages in the BackendLangs.js file .
add relevant resources to the subsections for each languages.

Things to Note

Read our  contributing guidelines before making your contribution.
only one language per person per pull request.
if you don't have resources to all the subsections in the language section that is fine, add the ones you know.
be as descriptive as possible when naming your links and do not forget to include the author or creator of the content.
Eg. 'Build a food recipe app with fetch API and vanilla JavaScript. ~ GeetProbin'.
do not duplicate resources or sections.
follow the code format or template used.
include the link to the section in the Backend sidebar component file.

Yep, I will add one language per pull request and , I will start with C programming language

That sounds great. Thank you @Goutham-erram


moviedb.org doesn't find any movie, no pictures and covered settings windows

Version: 3.0-SNAPSHOT
Build: 2018-02-18 01:21
OS: Linux 4.13.0-32-generic
JDK: 1.8.0_161 amd64 Oracle Corporation
What is the actual behaviour?
The Scraper doesn't find any movies in german. The other scrappers sometimes but only english Descriptions. Also i can't downloading any picture. No search result
The second problem is. I want to change my settings to add a mac address for my Kodi. For this i must change with Alt-Tab to my Browser to copy the address. When i change back the open window with the settings is covered by the settings window and i can't save or exit tiny.
Steps to reproduce:
Additional
Have you attached the logfile from the day it happened?

i can explain that i find the solution for the moviedb.org error. I find in the logfile some hint's about java ssl certifacte error. It's a simple access problem with tiny and java. On my Linux Mint tiny starts in non root syntax. change it and run !! Happy.
But the other Problem with the covered windows still present.
On every windows i open a subwindows it was covered by the root windows.
Is this a bug? or another Problem with my Linux ?
Thx

well, you should never run a program as admin (unless it really needs the rights). You may need to install/reinstall ca-certificates-java on your machine!
well alt-tabbing out of tmm and back in works in linux... I'll check this in v3 since v2 is kinda deprecated

as far as I could see that is already fixed in v3, but I've added this fix also to v2 (2.9.9)


SIL verifier: Fix polarity of consuming check through (unchecked_addr_cast (open_existential_addr)).

This was looking for a single non-consuming use instead of checking for the absence of any consuming uses, leading to false-positives when there were multiple non-consuming uses. Fixes rdar://problem/44116763.

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha -<PHONE_NUMBER>36f75730540f730720556aac7574b8

@swift-ci Please test Linux

Build failed
Swift Test OS X Platform
Git Sha -<PHONE_NUMBER>36f75730540f730720556aac7574b8

@swift-ci Please test OS X

Compatibility suite failures look like unrelated known failures.


Common area followup

Add leader and member cobra commands
Avoid lot of if/else checks and create two commands
one for leader and one for member.

Also split the clusterset controller since most logic
is disjoint.

NOTE:
This patch is based on common-area branch which is covered
in PR: https://github.com/antrea-io/antrea/pull/3041

Kindly only review the top patch

Codecov Report

Merging #3066 (4b87f33) into feature/multi-cluster (f55dbad) will decrease coverage by 0.34%.
The diff coverage is 31.21%.

@@                    Coverage Diff                    @@
##           feature/multi-cluster    #3066      +/-   ##
=========================================================
- Coverage                  40.18%   39.84%   -0.35%
=========================================================
  Files                        166      179      +13
  Lines                      20763    21385     +622
=========================================================
+ Hits                        8344     8521     +177
- Misses                     11603    12030     +427
- Partials                     816      834      +18

Flag
Coverage Δ

unit-tests
39.84% <31.21%> (-0.35%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

multicluster/cmd/multicluster-controller/leader.go
0.00% <0.00%> (ø)

multicluster/cmd/multicluster-controller/main.go
0.00% <0.00%> (ø)

multicluster/cmd/multicluster-controller/member.go
0.00% <0.00%> (ø)

...ulticluster/cmd/multicluster-controller/options.go
8.00% <0.00%> (ø)

...controllers/multicluster/core/local_common_area.go
0.00% <0.00%> (ø)

...ticluster/core/remote_resourceimport_controller.go
0.00% <0.00%> (ø)

...icluster/cmd/multicluster-controller/controller.go
4.68% <2.56%> (ø)

...lticluster/core/mock_remote_common_area_manager.go
23.80% <23.80%> (ø)

...ontrollers/multicluster/core/remote_common_area.go
29.45% <29.45%> (ø)

...llers/multicluster/core/mock_remote_common_area.go
31.70% <31.70%> (ø)

... and 8 more

/test-all


Plugins: Run in User's Context-option missing

When using the CrmPluginRegistrationAttribute-class, there doesn't seem to be any way to pass the a name or ID of a systemuser to adjust the option 'Run in User's Context' that is provided by the PluginRegistration tool. Could you add this please?
Thanks in advance.

Duplicate of #180


Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined

Summary
Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined
Reproducible sample code
import MapView, {PROVIDER_GOOGLE} from 'react-native-maps';

Steps to reproduce
open screen in whic using app got app crash  with error
Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
Expected result
map lad
Actual result
Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
React Native Maps Version
1.0.0
What platforms are you seeing the problem on?
Android, iOS (Google Maps)
React Native Version
0.68.2
What version of Expo are you using?
Not using Expo
Device(s)
iphone 12,android 11
Additional information
No response

Hi @ravin-amplework 👋. You will have to provide more than your import statements as Reproducible sample code for anyone to be able to track this down :)

import MapView, {PROVIDER_GOOGLE} from 'react-native-maps';
<MapView
ref={mapRef}
provider={PROVIDER_GOOGLE}
// onPress={(e) => setMk(e.nativeEvent.coordinate)}
style={styles.map}>
<MapView.Marker
coordinate={{latitude: item?.lat || 0, longitude: item?.lng || 0}}
title={item?.name}
/>

this code perfectly working  on  version 0.30.2.

New renderer is also enable no main App.js file
import { enableLatestRenderer } from 'react-native-maps';
enableLatestRenderer();

Can you provide a repo or snack that experiences this?


Mind pointing to the changelog you want to be cleaned up?

Oh well, I actually didn't put a changelog in this repo. And it seems like someone else sniped the PR


Fix unset environment variable in harbor-trivy image

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

/approve


[DO NOT MERGE] SDK-1136 DFP unit tests

Linear Ticket: SDK-1136
Changes:

This encompasses a bit of a refactor to make it possible/easier to test the DFP/CAPTCHA flows (without, y'know, actually testing  reCAPTCHA or the webview injection)

Notes:

Putting this PR up to see if tests pass on GH. Having some weird issues locally, and wondering if it's because I upgraded my XCode 😬

Checklist:

[ ] I have verified that this change works in the relevant demo app, or N/A
[ ] I have added or updated any tests relevant to this change, or N/A
[ ] I have updated any relevant README files for this change, or N/A

Ok, there's definitely something up with XCode 15, since these are all passing here. Not gonna spend too much time on it right now, but that's annoying.
Before I submit this before review, I wanna play it safe and do another round of manual E2E testing in the morning, just to be super cautious, since this was a not-insignificant refactor.


Support for Sparse Arrays

Hey there, great project, thank you for all the hard work on this.
I am wondering if there is any planned support for sparse array data? We are currently in the planning stages of implementing mleap in Rust and need to support both sparse and dense data.
Thank you again!

The work I know about is https://github.com/vbarrielle/sprs I don't look like I can commit to working on sparse arrays at this point.
I'm very open to working with ndarray and consumers of it, in the way of extending it in terms of integration features (access to raw details of the data structure) or features closely related to the current feature set.

Hello @hollinwilkins, I'm the author of the sprs library @bluss mentionned. I'd be glad if the library can help, but it's still in its early days. I try to make the library compatible with ndarray (eg sparse/dense products). Feel free to open issues in the sprs repo if you have any issue/question.

hi @vbarrielle. I'm about to release ndarray 0.8 unfortunately, for serde 0.9.

No problem, I'm close to releasing a breaking change to sprs too, so it's a nice occasion for me to syncrhonize these changes.

great, then I'll speed up rather than drag the feet for that. :)

@vbarrielle Does your library support broadcasting sparse/dense multiplies and dot products?

@hollinwilkins if you mean eg multiplying a sparse matrix by a 1-dim ndarray vector, then yes that works. However apart from this there is no support for broadcasting in sprs. But that could change. What kind of operations would you need?

Ok, keep https://github.com/bluss/rust-ndarray/issues/273#issuecomment-283743192 in mind and please report well scoped issues on this topic.


Test: Selection menu

#15156

[x] Windows @weinand
[x] macOS @bpasero
[x] Linux  @aeschli

Complexity: 2

Check the new Selection menu makes sense
Check the entries run the corresponding commands
Check that simple keybindings are shown after the label (complex ones do not)
Do the above for the 2 Edit > Toggle (Block) Comment and 2 Edit > Emmet commands

Closing as the assigned testers are done testing


Any plans to upgrade to v12?

Just that, are there plans to maintain this module in v12?
Thanks!

i also would like this to be updated to V12, 330 D&D 3.3.1


Infrastructure details

Hi,
Thanks a lot for sharing the code. On web demo, model is providing the responses with in 10-15 seconds. Can you please share brief details of  infrastructure used for the demo.
Thanks once again for the paper and code.
Regards,

The backend uses a single 1080ti GPU with both models pre-loaded into the GPU memory.

Thank you


Stateful/non 'static handlers?

Consider a fairly trivial use-case for an API:
struct Handler {
    db: database::Client
}

impl Handler {
    async fn user_book_handler(&self, req: Request<Body>) -> Result<Response<Body>, io::Error> {
        let user_name = req.param("userName").unwrap();
        let book_name = req.param("bookName").unwrap();

        let user_id = self.db.get(user_name).unwrap();

        Ok(Response::new(Body::from(format!("User ID: {}", user_id))))
    }

    pub fn router(&self) -> Router<Body, io::Error> {
        Router::builder()
            .get("/users/:id", |request| self.load_user(request))
            .build()
            .unwrap()
    }
}

fn main() {
    let handler = Handler { db: database::Client::new("<IP_ADDRESS>:6379") };
    let service = RouterService::new(handler.router()).unwrap();
    let addr = SocketAddr::from(([127, 0, 0, 1], 3001));
    let server = Server::bind(&addr).serve(service);
    if let Err(err) = server.await {
        eprintln!("Server error: {}", err);
    }
}

This is not possible, given this compiler error:
error[E0495]: cannot infer an appropriate lifetime due to conflicting requirements
  --> src/main.rs:24:36
   |
24 |                 .get("/users/:id", |request| self.load_user(request))
   |                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
note: first, the lifetime cannot outlive the anonymous lifetime #1 defined on the method body at 22:9...
  --> src/main.rs:22:9
   |
22 | /         pub fn router(&self) -> Router<Body, io::Error> {
23 | |             Router::builder()
24 | |                 .get("/users/:id", |request| self.load_user(request))
25 | |                 .build()
26 | |                 .unwrap()
27 | |         }
   | |_________^
note: ...so that the types are compatible
  --> src/main.rs:24:36
   |
24 |                 .get("/users/:id", |request| self.load_user(request))
   |                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   = note: expected  `&&users::Handler`
              found  `&&users::Handler`
   = note: but, the lifetime must be valid for the static lifetime...
note: ...so that the type `[closure@src/main.rs:24:36: 24:69 self:&&users::Handler]` will meet its required lifetime bounds
  --> src/main.rs:24:18
   |
24 |                 .get("/users/:id", |request| self.load_user(request))
   |                  ^^^

Given this limitation, I don't understand how I can do anything useful with this crate. Am I missing something obvious? I would really appreciate the help.

Thanks for the solution, @rousan. Would there be any solutions besides making the state global? I have several different modules that will be creating sub-routers and added under scopes within a main router, and sharing this global static state across several modules seems like it will get messy quickly.

let me clarify, are you using a single state across all the modules or allocating a state for each module?
Case-1: You could use lazy_static crate to declare a central state in one place globally and use it across modules.
Define the state:
lazy_static! {
   pub static ref STATE: State= State::new();
}

Case-2: if you're trying to manage state per module, then just declare them as private global variable as follow:
lazy_static! {
   static ref DB_CLIENT: database::Client = database::Client::new();
}

My state is initialized at runtime, so it cannot be global static state. From my original example, The database client will be initialized with arguments from the command-line, which means your example with database::Client::new() does not match my use-case.

I'm looking for built-in support from routerify to accomplish something like the following to support accessing run-time initialized state from the handlers.
async fn main() {
    let addr = SocketAddr::from(([0,0,0,0], 3000));

    // My global state
    let state = Arc::new(RequestState { foo: 321 });

    // How can I make some sanity this onion1, onion2, onion3, onion4? Is it possible?
    let make_svc = make_service_fn(|_| {
        let onion1 = Arc::clone(&state);
        async move {
            Ok::<_, Infallible>(service_fn(move |req: Request<Body>| {
                let onion2 = Arc::clone(&onion1);
                async move {
                    handle_request(&*onion2, req).await
                }
            }))
        }
    });
    let server = Server::bind(&addr).serve(make_svc);
    if let Err(e) = server.await {
        eprintln!("server error: {}", e);
    }
}

Source: https://users.rust-lang.org/t/hyper-passing-a-state-reference-to-async-request-handler/36295/3

See this example from a different router which makes application state available in the handlers: https://github.com/kardeiz/reset-router/blob/master/reset-router/src/lib.rs#L232

Good point, I will add this support very soon.

Thank you! If at all possible, I'd like to help since this feature is blocking my ability to use this crate.

Will make a new release with the support today

Thanks again for the quick support on this! I can test it today

I just made a released v1.1.3 by adding the above feature.
Doc: Data and State Sharing.


Secrets providers API, initial TextFile and EnvironmentVariable provider implementations

Fixes: #

Add model documentation for the Secret class
Add SecretsProvider abstract base class defining the interface for retrieving a specific secret from a specific secrets provider
Add EnvironmentVariableSecretsProvider and TextFileSecretsProvider concrete implementations
Secret.value property now retrieves the value from the specified SecretsProvider with the specified parameters
Add ability for plugins to register additional SecretsProvider subclasses

Add plugins documentation of this feature
Dummy plugin now includes a simple (insecure!) example ConstantValueSecretsProvider

Improved Secret edit view

"provider" selection is now a select menu listing available secrets providers, instead of a free-text field
Provider parameters can still be inputted as raw JSON, but specific SecretsProvider classes can now provide a ParametersForm that will be displayed as an alternative to the JSON text field

Added a view endpoint and custom jquery code to retrieve the appropriate ParametersForm when selecting or changing providers
Added JS to keep the JSON field and parameters form in sync with one another

TODO:

[ ] Identify what if any hard-coded and/or configurable restrictions we want to place on EnvironmentVariableSecretsProvider and TextFileSecretsProvider to reduce potential security risks of allowing a user to access environment variables or filesystem files that they shouldn't actually be permitted to access.

The form does not appear to be presenting on create of a new secret, no matter the . If I create a new environment-variable new secret, the form doesn't show, but if I use JSON then go to edit the form, I am able to then use the form to update the value.

Good catch! Probably on the last N iterations of working on this I was just working with existing secrets and forgot to test the new-secret flow with the latest code. Will fix.

Presumably this is in support of #541, which did initially state that:

Nautobot needs a supported set of UI and APIs for working with secrets (Git tokens, device credentials, OAuth2 parameters, SSoT integration credentials, etc.).

In support of the highlighted "device credentials", I was expecting there would be some kind of context and/or accommodations for devices, where a given device would be presented as a configurable parameter as obj. I can totally understand if it has been descoped, but I am strong advocate to include this feature. If it is de-scoped, would be great to know either way though.

In support of the highlighted "device credentials", I was expecting there would be some kind of context and/or accommodations for devices, where a given device would be presented as a configurable parameter as obj. I can totally understand if it has been descoped, but I am strong advocate to include this feature. If it is de-scoped, would be great to know either way though.

The current idea is that rather than having a dynamic parameter in the Secret definition itself, we will provide the ability to associate a specific Secret with a single device or a set of devices (using the config-context pattern). This does mean that if you have a different set of credentials for each device, you will need to create a distinct Secret record per device (rather than a single Secret definition that somehow resolves to a different secret value per device), but in the (hopefully more common) case where you have a shared service account for a whole group of devices, you will be able to create a single Secret and share it across that set of devices. Does this adequately address your concern?

You are addressing the more common, however, there is certainly examples of per device, I actually had a customer today tell us that the there is a new requirement for the user/pass to be per device. I am not a fan of the config-context pattern, as is a bit of magic that will not be obvious to users. I also think that with the obj context, one can still have the same capability by example.

Good points,  thanks Ken. We should discuss this some more. Need to make sure that any solution we come up with isn't too tightly coupled with Devices specifically (as that was a key shortcoming of the old feature). I'm also hesitant about the usability/painfulness of potentially needing something like a mixture of Jinja2 templating with JSON if we need to have dynamic parameters to the lookup.

Merging this into the gfm-secrets branch in its current state, will continue to iterate and refine before the final code drop.


[GHSA-69ch-w2m2-3vjp] An attacker may cause a denial of service by crafting an...

Updates

Affected products
Source code location
Summary

Comments
Add commit link

Hey there @rbeuque74 👋
Is there a package associated with this? We don't maintain advisories on the go standard library which it seems like this might be a part of.

Hey @darakian !
Yes, there is: https://pkg.go.dev/golang.org/x/text/language
Github repository is : https://github.com/golang/text
You can find the Go vulnerability report here: https://pkg.go.dev/vuln/GO-2022-1059


Add .env.template

This PR adds an empty .env.template file for people to use as a starter. It also adds notes in the README in order to be able to use it properly.
Fixes https://github.com/vercel/commerce/issues/30, and potentially helps on https://github.com/vercel/commerce/issues/29, https://github.com/vercel/commerce/issues/26, https://github.com/vercel/commerce/issues/24, https://github.com/vercel/commerce/issues/17 and https://github.com/vercel/commerce/issues/14.

@FredvanRijswijk thanks for the review! I think that could be nice to add to the README, but with some kind of indication that it only applies if you are using Vercel and have an app already set on it. For any other use case, manually setting the .env.local file is the way to go.


Ability to render Table with pre selected rows

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the Modus style guidelines

Proposal
The old Data Table had the option to load the table with some rows already selected. The Table component should have the same behaviour
Motivation and context
This is important for use cases that enable editing a selection of elements from a list. If the table can't load with the previous selected elements, the user needs to select everything again when editing a selection
What is the issue regarding ?
@trimble-oss/modus-web-components
Priority
High
What product/project are you using Modus Components for ?
Trimble Inspections
What is your team/division name ?
Camaleão/Transportation
Are you willing to contribute ?
Yes
Are you using Modus Web Components in production ?
no

Hello @leosole-trimble / @msankaran0712  Is this done already?

@rthanga1 yes


add some patterns for lazy loading

just some things (I just write those here because I think they don't exist yet and are quite nice)
you can consider which (if any) are worth implementing
groups for keys https://github.com/wbthomason/packer.nvim/pull/1063
patterns for commands like NvimTree* or NvimTree{Toggle,Open,Close}
ig for autocommands this should work anyway (haven't looked at the code yet though)

I'm planning to make some improvements to the keys handlers.
Patterns for commands is not something I'm interested in. Thank you for the suggestions


Fixed browser opening issue

There is an i-donno-what issue when spotipy.uitl.prompt_for_user_token() tries to open the browser using the subprocess.call. #29 issue might be related to this.
This pull request intends to fix this by utilizing webbrowser.open instead. It also works around the unnecessary verbose output of webbrowser.open and programmatically redirects it to the /dev/null.
The code is tested on both OSX and Linux.

Why this hasn't been merged? it appears to be working on all platforms (I also tested it on Windows) while subprocess.call(["open", auth_url]) does not.

Has this project died?  Having the above fix would be much appreciated!

thanks for the PR - the functionality has been incorporated via another PR, so this one has been closed.


⚠ Slack has degraded performance

In f254241, Slack (https://status.slack.com/api/v2.0.0/current) experienced degraded performance:

HTTP code: 200
Response time: 72 ms

Resolved: Slack performance has improved in c5caaa7.


refactor external client to make single getAWSConfig call per connect

Description of your changes
Refactors getAWSConfig calls in the external client code, to make single call per connect. getAWSconfig calls involve resolving credentials and executing AWS STS calls when there are relevant AssumeRole or AssumeRoleWithWebIdentity  operations needed. This was being called multiple times per connect, causing superfluous invocations.
This PR will improve the situation reported at #997
I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Is it possible to implement credential caching? Currently, it seems we make a single call per connect. However, when managing multiple resources, each resource triggers a separate call. Considering that IRSA credentials are valid for 15 minutes, it would be more efficient to perform the call only once every 15 minutes, even when dealing with numerous managed resources - what do you think ?

@haarchri This PR is an initial improvement for addressing the low-hanging fruit. Our plan is to make further improvements in iterations. This is especially motivated by the fact that those parts of the provider are currently fractured because we have a hybrid architecture for that provider (both the CLI-based & SDK-based upjet architectures are in use at the moment).
The plan is once we completely switch to the nofork arch for the remaining few resources , we will be able to simplify the provider setup logic there. And as we simplify provider setup logic, it will be easier for us to make further optimizations like credential caching, that is definitely in our scope.

/test-examples="examples/iam/role.yaml"


ARROW-6508: [C++] Add Tensor and SparseTensor factory function with validations

I'd like to add Make factory functions in Tensor and SparseTensor.  These functions validate the given parameters.
The following validations run in Tensor::Make:

Data type (non-null and is_tensor_supported)
data pointer
the size of shape
elements of shape are all positive
the consistency of shape and strides
the consistency of strides and the size of data
the consistency of the size of dim_names and shape

The following validations run in SparseTensor::Make:

Data type
the consistency of the sparse index and shape
the consistency of the size of dim_names and shape

TODO:

[x] Use Result<T> as return types.

A couple of high level comments before I review the details:

I might suggest calling these MakeSafe or similar since the Make constructors on other classes cannot fail (see Table::Make) (cc @fsaintjacques @pitrou for thoughts)
Since you're adding new APIs, are these good candidates to use Result<T> instead of Status?

I might suggest calling these MakeSafe or similar since the Make constructors on other classes cannot fail (see Table::Make)

OK, I understand. Renaming to MakeSafe is good for me, but I'm concerning that this name is a new name in C++ API.

Since you're adding new APIs, are these good candidates to use Result<T> instead of Status?

Oh, I forgot to rewrite this to use Result<T>.  I will do it.
Thanks!

I finished rewriting by using Result<T>, renaming to MakeSafe, and rebasing it onto the latest master branch.
I search uses of Result<T> and I found that StructArray::Make performs validations and returns Result<T>.
It may be better to rename it to StructArray::MakeSafe likewise.

I would rather have all those methods named Make personally. The fact that they return a Result hint that there may be some validation going on.

The other (unsafe) Make functions also return Status and use an output parameter. The difference is whether they will logically validate (via Validate) the inputs.
We had this discussion previously, not sure if on the ML or in a PR review: Make|MakeUnsafe versus Make|MakeSafe.

I guess it is enough to see the return type to distinguish whether a Make function may perform validations and can be failed, too.
But we have to use the different names if we want to introduce two factory functions, which have the same argument types and the different return types, and the one performs validations and the other does not.

Well, why do we need the unsafe APIs here? You're only checking the shape and strides, so it's gonna be fast (unless you have 1000-dimensional tensors?).

We don't need unsafe APIs in tensor and sparse tensor.
I just mentioned my anxiety about the potential issue of the naming convention that uses only Make.  For example, we can consider adding a safe version of Table::Make:
Result<std::shared_ptr<Table>> Make(
    const std::shared_ptr<Schema>& schema,
    const std::vector<std::shared_ptr<ChunkedArray>>& columns,
    int64_t num_rows = -1
);

But we cannot add this because this function differs only the return type from the existing Table::Make.

If we want to add a safe version of Table::Make, we can rename the unsafe version to Table::MakeUnsafe.

OK, I'll revert the commit to rename Make to MakeSafe.
I found the previous discussion thread in ML.  The discussion hasn't been settled yet.

@pitrou I finished fixes for all of your comments.  Could you please review this again?


qq: init at 2.0.3-543

Description of changes
Added https://aur.archlinux.org/packages/linuxqq-new. Still need to install a desktop file and the binary file, but the output runs fine.

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, 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Having read the comments in the link, I got the impression that this version of QQ is not available to all users. Is this still the case?
Also, I would prefer that aarch64-linux support be added to the package.

I got the impression that this version of QQ is not available to all users. Is this still the case?

Login is still restricted to beta testers, but should be opening up soon as a page for it already went public.


fix: add missing sections colours

:tada: This PR is included in version 0.55.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Ideas for the Industrial Doors (Yes, plural)

Describe the feature you'd like
1: So we have the really cool industrial door that looks great. However, the only door we get is one that can only be opened with redstone, like the iron doors and so on.
It would be nice if we also had a door that can be opened as usual, with the right click. No need for buttons or levers etc.
2: The other doors in the game can be combined from a 1x2 door to a 2x2 door by placing them side by side. The door from Oritech remains a single door, which also causes the gray door-frame to overlap eachother and glitch.
I would love if you could convert that into a 2x2 door as well when placed next to another door, so that the door-frame expands around both doors, with the swirly lock bolt ending up in the middle or something.
3: Trapdoors! both redstone locked and regular right click unlocked!
4: And something I really miss from Ad Astra.. the 3x3 doors. They had locked and unlocked 3x3 doors, they also had airlock doors, and blast doors, different designs on them, and some could be opened with right click, some had to be unlocked with redstone.
Describe alternatives you've considered
No response
Additional context
No response

Oh in addition. There is a mod called Iron Door Key, which is a key that can open iron doors and it's likes with a right click, instead of buttons and levers.
You need to hold the key in your hand and click the door.
It defaults to the vanilla iron doors and similar, but via a tag in a datapack you can add more doors that it will work with. For instance the Obsidian Reinforced Door from Blockus, and so on.
But that doesn't work with the Industrial Door for some reason, so I guess it's made in a different way from the regular iron doors and trapdoors.


New TapHold handler instead of legacy Tap

Fix #6980, fix #5425 (close #5670 PR as obsolete), fix #6705 (close #6788 PR as obsolete), fix #6916, fix #7255 (and numerous similar, see discussion).
Also lets avoid numerous issues with Leaflet.draw.
Use lower delay value: 600 (instead of 1000), close #6959.

Test page: https://gist.githack.com/johnd0e/a3647bebc3a65440d2f7ea470998816b/raw/test.html

[ ] May be change name? Like tapContext or touchContext
[ ] Should we leave (deprecated) tap alias for compatibility?
[ ] Is common map Handler really best way to implement this feature?
Or better do that similar to DomEvent.DoubleTap.js (to handle every individual target separately)?
[ ] May be expose tapHoldDelay option.

This is still WIP.
Currently major blocker is #7058.

@IvanSanchez
You see last commit marked as "temporary" (because of https://github.com/Leaflet/Leaflet/pull/7026#issuecomment-606575241).
Of course we can revert it later...
And we should decide what to do with old tap. Currently it is just marked as "Deprecated".
Do you think that it still can be useful (under some conditions, or for test purposes)?
I doubt that we really need it for compatibility.
So may be better to remove it completely?
And one more consideration: currently both tap and tapHold add map option with same name - tapTolerance.
I suppose that it can lead to issues with documentation.

[ ] Cleanup _simulated checks

[ ] Cleanup _simulated checks

لطفا ب من در بازیابی جیمیل های نام برده و گذرواژه هام و اطلاعاتم کمک کنید

@johnd0e do you want to change something? I would force this PR so that the bug is fixed in the master

do you want to change something?

Yes

@mourner if you can review this PR we would solve a lot of issues and get a big step further

Fixes #7499 too

to make sure there's no delay

Do you still observe delay in recent browsers (without meta tag)?
In either case, we can add touch-action: manipulation later on request.


3.0 Documentation

PR Open for development of TwitchIO Version 3.0 Documentation.
This PR should be used to contribute and discuss changes to documentation on TwitchIO Version 3.
Main Points:

Consistent Docstrings
Easy navigation of the Documentation
Guides and Examples
Full Coverage of TwitchIO and Extensions
Design Discussion (CSS/JS etc)
Clear and Up to Date Changelog

Opinions on guide topics.
I have created an initial one for commands extensions

Command Parsing


Do not add suspended contracts to a capitation report

Check for contract status, if "Suspended"==true do not add contract to report

Done


Fail to import some .blend files (with shader composed of Principled BSDF only) from Blender (ERR_FILE_CANT_OPEN) [ArchLinux]

Godot version
4.1.1.stable.arch_linux from AUR
System information
Godot v4.1.1.stable unknown - Arch Linux #1 SMP PREEMPT_DYNAMIC Thu, 21 Sep 2023 11:06:39 +0000 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 2060 SUPER (nvidia) - AMD Ryzen 7 5800X 8-Core Processor (16 Threads)
Issue description
I'm having issued with Blender's (3.6.4) and Godot's (4.1.1.stable.arch_linux) latest versions available on ArchLinux.
Most .blend files "behave as expected" (they are correctly imported, can be displayed, made local, etc), but today I was modelling a cute burger (yes it is.) and it's .blend file could not be loaded into godot as I usually do it (saving the .blend file in a subfolder of the root of my Godot project called "Blender Meshes").
Instead, it keeps on failing to import said file (marking it with a red cross on it's icon in the resource panel) and retrying to import it.
Trying to put the result of a .escn created by exporting the .blend file with the Escn Exporter plugin from Blender works as expected (though my materials are missing), same goes if I just export them as .gltf.
I don't have a lot of logs sadly, but the minimal recreation project I created to post the issue logs as follow :
Godot Engine v4.1.1.stable.arch_linux (c) 2007-present Juan Linietsky, Ariel Manzur & Godot Contributors.
--- Debug adapter server started ---
--- GDScript language server started ---
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true. Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.
  modules/gltf/gltf_document.cpp:7432 - Condition "err != OK" is true.
[...]
Returning: ERR_FILE_CANT_OPEN
  Error importing 'res://Burger.blend'.

I'm a bit at loss as to what to do. I already tried chmod and other POSIX permissions-related things, but none worked. As I said, other files work perfectly well. Changing the name of the file, or even putting it directly at the root of my project, won't change anything. Same goes for deleting materials/objects from the Blender scene.
Cute burger .blend file included in minimal example.
Steps to reproduce

Be on ArchLinux
Have Blender 3.6.4 installed
Have Godot 4.1.1.stable.arch_linux
Setup Blender Path in Godot's editor settings (/usr/bin in my case)
Create a Godot project (I used Forward +)
Build a mesh
Save it as a .blend file in the created Godot project
Go back to the Godot Window, which should show errors in the console and cross out the .blend file n the FileSystem panel.

Minimal reproduction project
BlendImportIssueMin.zip

Thanks for the report! COnsolidating in https://github.com/godotengine/godot/issues/80530.


Multiple offset encodings warning when used with clangd

In C++ files when clangd is attached and copilot.lua is loaded, a repeated warning about Multiple offset encodings occurs. This seems to be a common issue as well when using null-ls at the same time as clangd but the fixes used for that do not seem to work with copilot.lua.

To reproduce, have copilot.lua and let clangd attach itself to a buffer. The below warning then repeatedly pops up for any cursor change or edit whether inside Insert mode or outside.

To reproduce, load copilot.lua and let clangd attach itself to a buffer. The below warning then repeatedly pops up for any cursor change or edit whether inside Insert mode or outside. When I do :Copilot disable, this stops occurring.

Got the same warning!

https://github.com/jose-elias-alvarez/null-ls.nvim/issues/428
The problem ariese because two clients copilot and clangd using different offset_encoding, one of the solution proposed in the thread is to set copilot to use utf-32 offset encoding (which is same as clangd).

same problem

This isn't an copilot.lua issue.
jose-elias-alvarez/null-ls.nvim#428
The problem ariese because two clients copilot and clangd using different offset_encoding attached to same buffer, solution proposed in the thread is to either change clangd's or in this case copilot offset encoding. By default, clangd uses utf-32 offset encoding.

by any chance, do you know how do change the offset for copilot?


Add tracing on the graphite plugin interface

Coverage increased (+0.03%) to 81.918% when pulling 8e18e4984778356504d76b70d8d8a236637036c5 on geobeau:improve-tracing2 into 5312748b52762538ae968d471e97c11c5b4daaa8 on criteo:master.


New Endpoint for full data transfer

See https://github.com/swissfintechinnovations/ca-mortgage/issues/49

@rolfwagner Thanks for your feedback - I have renamed the enpoints.

@olegkovalov Ok for MP? Scope rls 3. We will discuss it at the next mortgage meeting.

@rolfwagner So I guess we're ready to merge? 😃


Improve allocation filter handling

Currently only one allocation filter strategy is supported
The filter should be outlined to a base class, and different strategies inhereit from a common interface (see feat/sc20):

[x] Base class, for a common interface
[x] Runtime selectable with argument
[x] Whole-program call-graph filter

closed by pr #26


dircolors module does not work with oh-my-zsh module

Issue description
The dircolors module does not play well with the oh-my-zsh module.
The dircolors module uses initExtra to inject its $HOME/.zshrc code, but this code is injected after loading oh-my-zsh. The issue is that oh-my-zsh requires the LS_COLORS environment variable to be present before setting up the shell aliases that enable colors by default.
A working solution is to use the initExtraBeforeCompInit setting of zsh instead of initExtra to inject the dircolors code as early as possible.
I just started using home-manager a couple days ago, so I though I would open a bug report before trying to submit a PR.
Meta
Maintainer CC
@JustinLovinger
Technical details
❯ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-darwin"`
 - host os: `Darwin 19.4.0, macOS 10.15.4`
 - multi-user?: `no`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.3.4`
 - channels(caleb): `"home-manager, nixpkgs-20.09pre226586.571212eb839"`
 - nixpkgs: `/Users/caleb/.nix-defexpr/channels/nixpkgs`

@caleb Makes sense. I don't see any reason dircolors shouldn't be run before compinit. I'll make a pull request with the changes.
In the meantime, you can disabledircolors.enableZshIntegration and source it yourself.

This issue should be fixed and safe to close.

Thanks! It works over here.


Update: Fix Error spatie/laravel-missing-page-redirector

Since update from Laravel 9.32 to 9.33 the package spatie/laravel-missing-page-redirector did not work anymore.
This should fix it.

Sent PR https://github.com/spatie/laravel-missing-page-redirector/pull/78 to spatie/laravel-missing-page-redirector as an alternative to address this same issue.
@timacdonald you may want to look on the reasoning on that PR.
For short, as PR #44339 introduced the Illuminate/Routing/Contracts/CallableDispatcher, empty containers -- such as the default to Illuminate\Routing\Router when no container instance is provided to its constructor -- would fail to dispatch closure based routes as they won't know how to build this interface.
I believe this is kind of a breaking change, as maybe other packages using private Router instances, for similar reasons as the package linked above, could start failing to dispatch closure based routes.
As this ship has been sailed already, maybe we could consider changing this line on Illuminate\Routing\Router's constructor:
https://github.com/laravel/framework/blob/a1611c76ffd9221978378f361af00fac2fe0b4aa/src/Illuminate/Routing/Router.php#L138
To
$this->container = $container ?: Container::getInstance();

So at least when no container is provided it uses the global one instead, and avoid this issue. What do you think?

Or the alternative would be dropping the new Illuminate/Routing/Contracts/CallableDispatcher interface, as the introduced Illuminate\Foundation\Routing\PrecognitionCallableDispatcher is a sub-class for this new interface only subclass Illuminate\Routing\CallableDispatcher
Instead of binding to the interface, and dispatching an interface that might be unbounded on the container, we ask the container for the concrete class, and PrecognitionServiceProvider can alias/rebind its subclass when needed.

Looks like an alternative was merged to the package itself.


Cannot install a previous version when another extension depends on it

I tried installing a previous version of the Python extension and I got this:

Checking the Developer Tools console I got a more detailed message of the reason why:

Could this be displayed in the prompt as well so we have more information of why the extension installation has failed?

@luabud This error message should never appear. Installing a previous version should work even though another extension depends on it. Because you are not uninstalling it.
Fixed it.
To verify:

Install Azure Cosmos DB extension which installs Azure account extension
Try to install previous version of Azure account extension
Make sure there is no error

I guess you could end up with an incompatible version of the dependency, because we don't specify extension versions, because we expect people to use the latest versions normally. I guess if you are picking versions manually, you know what you're doing.


Does not work with apps that block screenshots

Apps such as Bitwarden and Silence block screenshots. Attemtping to take a screenshot on the device results in this error.

Using scrcpy, the app functions normally on the device, but scrcpy displays a black background with the status bar visible. No notification or log output indicates anything abnormal. The screenshot notifciation does not appear.

Strangely enough, LineageOS's built-in screen recording app captures these applications without problems. The device is not rooted.
Tested on a OnePlus 3T running LineageOS 15.1 20180305.

You can disable it in Silence, in Silence settings → Privacy → Screen security.

Strangely enough, LineageOS's built-in screen recording app captures these applications without problems.

They probably modified the feature in the ROM, because on AOSP, even screenrecord do not display the screen content.

Since I added an entry in the FAQ, I'm closing.

Very great, thanks for fixing! I now can finally see the unlock pattern of my phone with broken screen :heart:

I really enjoy using scrcpy. I think that it's a great product. Unfortunately I am having the black screen issue mentioned above on some specific apps. I am new to coding so I apologize. I was wondering if someone could explain more in details how you  fixed the issue so that I can do it myself.
Thanks for the help!

The apps which remain black could not be mirrored with scrcpy (some drm prevent it to be captured).

Thank you. Great app.

@rom1v Do you know if an app is able to detect the case when an app such as scrcpy is pretending to be a secure display even though it's not?
The problem I have is that scrcpy allows taking screenshots and screen recordings, even though it pretends to be a secure screen, which creates a security vulnerability.
Furthermore, even if scrcpy filtered out protected content from screenshots and recordings, it would be possible to record the whole screen on the computer, unless scrcpy would filter that too.
But even then, anybody could build scrcpy from source as it is currently and use it to take screenshots of content that was supposed to be protected. So do you know if it's possible to detect "fake" secure displays?
(I know that it's always possible to take a photo of the phone with an ordinary camera, but I don't need to worry about that, I just need to prevent taking screenshots)

I don't know.
FWIW, I think that the users should always be able to take screenshots or videos of ther screen (it's their own device). It's ok that app developers provide a hint that some screen should not be captured by default, but device users should always have the possibility to easily bypass this setting. (I know that this is not the current trend from Google or others.)

I am having the same issue again with some banking app, which prevents taking screenshots too. I am using android 12. Tried the latest dev branch.

any update onhow to bypass that? some Bank app are black..even on v 1.23
thanks for help

Any updates on this issue? Is there any possibility to bypass it?

Not since Android 12 #2129.

Any updates on this issue? Is there any possibility to bypass it?

You can bypass it by decompiling the APK, disabling FLAG_SECURE for the main view, and recompiling the APK back.
It is very easy to do using "APKLab" extension for Visual Studio Code.

Open APK using APKLab, it decompiles it into a folder structure with smali files
in the resulting folder structure search for RegExp android/view/Window;->(set|add)Flag
Ignore all occurrences in androidx/* or any third-party namespaces
find (usually only one per app) occurrence that is preceded by setting register to 0x2000 like this:

    .line 143
    invoke-virtual {p0}, Landroid/app/Activity;->getWindow()Landroid/view/Window;

    move-result-object v0

    const/16 v1, 0x2000

    # invoke-virtual {v0, v1, v1}, Landroid/view/Window;->setFlags(II)V

comment out whole line calling addFlag/setFlags (as shown above)
rebuild&sign APK back by right clicking on the "apktool.yml" file and clicking "APKLab: Rebuild the APK"
install the APK on your device
enjoy

This approach has been tested on both RSA Authenticator and MS Athenticator apps running on Android 12.

Addendum to @CoolSpot's comment for Duo Mobile
From grepping android/view/Window;->(set|add)Flag I got five hits. None of them were prefixed with 0x2000. They were prefixed with other register addresses.
I just commented out all of them and it worked like a charm.

Hello,
Rooted users can bypass FLAG_SECURE using this LSPosed plugin.
Just tested it successfully.
Thanks

Root users may also use this fork which simply runs the scrcpy-server as the system user: https://github.com/Genymobile/scrcpy/issues/3049#issuecomment-1603612740

Root users may also use this fork which simply runs the scrcpy-server as the system user: #3049 (comment)

Is there a non root version for some apps which do not run on rooted devices such as banking apps.

In order to make a banking app work on a rooted device, you just have to add it to Magisk's DenyList.


Fix #45: Output Streams & Error Code

Configure ora to send output to stdout
Write validation output to stderr using console.error()
Set the exit code to 1 when validation fails

@westim Thank you for your contribution!


Import ics file, last day missing

I try to import some .ics files to google calendar.
When I import an event that goes from 2017/12/22 to 2018/01/05, on the calendar I see it going from 2017/12/22 to 2018/01/04. The last day is ever missing
Is there something wrong with the ics file I'm importing?
An example of my .ics is this:
BEGIN:VCALENDAR
VERSION:2.0
CALSCALE:GREGORIAN
METHOD:REQUEST
BEGIN:VEVENT
DTSTART;VALUE=DATE:20171227T000000Z
DTEND;VALUE=DATE:20180105T000000Z
DTSTAMP:20171130T132000Z
<EMAIL_ADDRESS>CREATED:20171130T132000Z
DESCRIPTION: HOLIDAYS
CREATED:20171130T132000Z
SEQUENCE:0
STATUS:ACCEPTED
SUMMARY:HOLIDAYS
END:VEVENT
END:VCALENDAR

For import i use this command:
~# gcalcli import myfile.ics --calendar 'test_calendar'
Thanks for your help

I think that's working as intended. Your event ended at midnight 01/05 meaning it has 0% overlap with the day of 01/05, so it lasts all day through 01/04 but doesn't cover the following day.
AFAICT the ICS format doesn't have explicit support for explicitly specifying "all day" events and it's slightly ambiguous how to handle cases like this, but especially when you're specifying a midnight datetime like 20180105T000000Z it seems correct to me to say that doesn't represent an "all day event including all of 01/05". I did see you can specify a date without a time instead, like 20180105, but that seems to work the same in practice...
LMK if there's some aspect I'm missing and it still seems wrong for whatever reason.


TextAlignment support?

Does this plugin support textAlignment, specifically Center?

I am using the default material tooltip and it has the alignment "hardcoded" to left, afaik. I wish for a nicer tooltip that can center text.

@adrianvintu
You can now update to v2.
Just use the Text textAlign property


Added Capability to rename a Dashboard

You can now rename a dashboard you own , Admin users of Hygieia can rename any dashboard
Renaming a dashboard owned by a user

Admin user renaming Dashboard

@amitmawkin Cool video and helpful feature.

LGTM

LGTM


Add a way to clear data stored on the device by k9

Checklist

[X] I have used the search function to see if someone else has already submitted the same feature request.
[X] I will describe the problem with as much detail as possible.
[X] This issue only contains a request for one single feature, not multiple (related) features.

App version
6.509
Problem you are trying to solve
Day after days, by using k9 and searching in some of my folders by loading e-mails, the app database size growing. k9 is now using 147Mo of user data.
The folder com.fsck.k9/databases uses 135.7Mo.
There is no way to unload a folder or all folders.
Suggested solution
It can be interesting to have a way to see and clear local data (without deleting it from the server) :

in the folder view, display the space used by the folder after its name
add a way to remove the cached data for each folder with a long click on the folder's name
in the k9 parameters settings, add an option to clear all cached messages

Screenshots / Drawings / Technical details
No response

Go to Manage folders, open the folder settings screen of a folder, then select Clear local messages from the menu.

Thanks, I did not know !
It can be useful to have the local size of all folder in this view to see which one is using all the storage space

Go to Manage folders, open the folder settings screen of a folder, then select Clear local messages from the menu.

Is this also available as a global action for the individual account?
And for data security reasons I wonder: Local messages deleted from a folder using "Clear local messages" are then safely deleted from the device or not visually retrievable until they are downloaded again via IMAP?

Is this also available as a global action for the individual account?

No.

And for data security reasons I wonder: Local messages deleted from a folder using "Clear local messages" are then safely deleted from the device or not visually retrievable until they are downloaded again via IMAP?

Removing local messages will never be a security feature. If you require sensitive messages to not be stored on the device, please use an email client that only downloads messages on demand and doesn't store a local copy.


Fix tests for faraday 2.1

Coverage: 100.0%. Remained the same when pulling 35cc774571013ba55a394950856f9974c6fd1b37 on fix_tests_for_faraday_2.1 into 49bdc1423ea6b21126241d2ec442b081dcea6800 on master.


Feature request: Rename without re-rendering?

Two requests in one day, apologies. :)
I do a lot of module renaming (mostly prefixes) in order to impose some organization on the module list. Since each rename triggers a new render of the module list (in order to correctly sort, I presume), renaming multiple modules at a time is kind of a drag because each one requires:

Re-filtering the list
Scrolling back to where I was in the list
Waiting for all the buttons to render etc. in the modules above where I'm at in the list so that the context menu is accessible.

What would be killer is if (optionally, probably?) renaming didn't trigger a re-render, meaning I could rename a bunch in a row efficiently, living with the bad sort until the next time the application re-rendered for another purpose.
Thanks for considering!

Could I ask why and what modules you are renaming? I am not sure I like the idea of not rerendering the screen after a rename as this seems like more of an edge case then the common case where the idea of renaming a module is to better the random ones that don't end up quite where they should.
If you are renaming a lot of modules, maybe I should look into what your renaming and why you are renaming them and improve my smart label feature to auto rename them for you.

Yeah, mostly I'm substituting for a lack of sort/filter/functionality tagging by using prefixes covering primarily two dimensions:
1, Type: Is this a Content module that only provides content packs of some kind, or does it add some kind of functionality?

Category: If it is a content module, what kind of content (adventure, scenes, sounds, animations, etc). If it is a module that adds functionality, what is the primary area of functionality added?

So my module names end up looking like "Type - Category - Module Name", ie "Content - Adventure - Grim Harvest" or "Func - Chat - Tabbed Chatlog" etc. That way if I want to scroll to all the modules that primarily add chat features or enable a few modules containing music to look for a good track it is easy to find them all grouped.
In the case of modules that either don't function on the current foundry version or conflict with others that I'm using I'll sometimes replace the above with a prefix that indicates there is a problem rather than uninstalling so that I still have it if it gets updated but I don't have to re-ask myself "Wait, why isn't X enabled?" all the time.
The 'real' solve for this set of issues is filter/sort/arbitrary tagging, but that is a much more substantial ask.


Added memcache sample

@tmatsuo @lesv
Please review

Sure. Which page is the code for?

https://cloud.google.com/appengine/docs/java/memcache/

Also added a threadsafe updater as sample code for this section: https://cloud.google.com/appengine/docs/java/memcache/#Java_Safely_handling_concurrent_memcache_updates

Overall, I feel like the snippets in that doc don't need to be a perfect class. They can be just static methods, showing necessary method calls in order.
Thoughts?

Also, I would like you to write a unit test, at lease it make sure it compiles, ideally testing the snippet correctness.

Yeah I'm working on writing an App Engine integrated tester for these
service libraries
On Thu, Apr 2, 2015 at 3:48 PM Takashi Matsuo<EMAIL_ADDRESS>wrote:

Also, I would like you to write a unit test, at lease it make sure it
compiles, ideally testing the snippet correctness.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/appengine-java-snippets/pull/4#issuecomment-89067938
.

closing... no longer like this


Set default windows workflow root for presubmits

When running the presubmit, we do not want to have to pass in a workflow_root, and instead want the default workflow_root to work.
This sets the default value correctly.

/cc dorileo
/cc bkatyl