Memory not registered for rdma. Is this iobuf allocated before calling GlobalRdmaInitializeOrDie? Or just forget to call RegisterMemoryForRdma for your own buffer?

Describe the bug (描述bug)
IOBuf cannot append_user_data with address starts after RegisterMemoryForRdma
W20221115 18:15:32.455838 79345 rdma_endpoint.cpp:727] Memory not registered for rdma. Is this iobuf allocated before calling GlobalRdmaInitializeOrDie? Or just forget to call RegisterMemoryForRdma for your own buffer?
W20221115 18:15:32.455842 79345 socket.cpp:1705] Fail to keep-write into Socket{id=2 fd=1034 addr=<IP_ADDRESS>:8002:53496} (0x55b33af574d0): Unknown error 3002 [3002]
E20221115 18:15:32.455933 79247 server2.cpp:95] RPC call failed: [E3002]Fail to keep-write into Socket{id=2 fd=1034 addr=<IP_ADDRESS>:8002:53496} (0x0x55b33af574d0): Memory not registered for RDMA

To Reproduce (复现方法)
// Licensed to the Apache Software Foundation (ASF) under one
// or more contributor license agreements.  See the NOTICE file
// distributed with this work for additional information
// regarding copyright ownership.  The ASF licenses this file
// to you under the Apache License, Version 2.0 (the
// "License"); you may not use this file except in compliance
// with the License.  You may obtain a copy of the License at
//
//   http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing,
// software distributed under the License is distributed on an
// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
// KIND, either express or implied.  See the License for the
// specific language governing permissions and limitations
// under the License.

#include <gflags/gflags.h>
#include "butil/atomicops.h"
#include "butil/logging.h"
#include "butil/time.h"
#include "brpc/channel.h"
#include "brpc/rdma/rdma_helper.h"
#include "brpc/server.h"
#include "bvar/variable.h"
#include "test.pb.h"

DEFINE_int32(port, 8002, "TCP Port of this server");
DEFINE_bool(use_rdma, true, "Use RDMA or not");
namespace brpc{
namespace rdma {
    DECLARE_bool(rdma_trace_verbose);
}
}  // namespace brpc

butil::atomic<uint64_t> g_last_time(0);

namespace test {
class PerfTestServiceImpl : public PerfTestService {
public:
    PerfTestServiceImpl() {}
    ~PerfTestServiceImpl() {};

    void Test(google::protobuf::RpcController* cntl_base,
              const PerfTestRequest* request,
              PerfTestResponse* response,
              google::protobuf::Closure* done) {
        brpc::ClosureGuard done_guard(done);
        uint64_t last = g_last_time.load(butil::memory_order_relaxed);
        uint64_t now = butil::monotonic_time_us();
        if (now > last && now - last > 100000) {
            if (g_last_time.exchange(now, butil::memory_order_relaxed) == last) {
                response->set_cpu_usage(bvar::Variable::describe_exposed("process_cpu_usage"));
            } else {
                response->set_cpu_usage("");
            }
        } else {
            response->set_cpu_usage("");
        }
        brpc::Controller* cntl = static_cast<brpc::Controller*>(cntl_base);
        if (request->echo_attachment()) {
            cntl->response_attachment().append(cntl->request_attachment());
        }
        if (!cntl->request_attachment().empty()) {
          std::string tmp;
          cntl->request_attachment().cutn(&tmp, 5);
          LOG(INFO) << tmp;
        }

    }
};
}

void run_client() {
  brpc::rdma::GlobalRdmaInitializeOrDie();
  brpc::ChannelOptions options;
  options.use_rdma = FLAGS_use_rdma;
  options.protocol = "baidu_std";
  options.connection_type = "single";
  options.timeout_ms = 10000;
  options.max_retry = 0;
  std::string server = "<IP_ADDRESS>:8002";
  brpc::Channel* _channel = new brpc::Channel();
  if (_channel->Init(server.c_str(), &options) != 0) {
    LOG(ERROR) << "Fail to initialize channel";
    return;
  }
  brpc::Controller cntl;
  test::PerfTestResponse response;
  test::PerfTestRequest request;
  char buffer[16] = {'a', 'b', 'c'};
  brpc::rdma::RegisterMemoryForRdma(buffer, 16);
  cntl.request_attachment().append_user_data(buffer + 1, 3, [](void*) {});
  request.set_echo_attachment(false);
  test::PerfTestService_Stub stub(_channel);
  stub.Test(&cntl, &request, &response, NULL);
  if (cntl.Failed()) {
    LOG(ERROR) << "RPC call failed: " << cntl.ErrorText();
  } else {
    LOG(INFO) << "RPC call success";
  }

}

int main(int argc, char* argv[]) {
    brpc::rdma::fLB::FLAGS_rdma_trace_verbose = true;
    GFLAGS_NS::ParseCommandLineFlags(&argc, &argv, true);
    if (FLAGS_use_rdma) {
        LOG(INFO) << "Call GlobalRdmaInitializeOrDie";
        brpc::rdma::GlobalRdmaInitializeOrDie();
    }

    brpc::Server server;
    test::PerfTestServiceImpl perf_test_service_impl;

    if (server.AddService(&perf_test_service_impl,
                          brpc::SERVER_DOESNT_OWN_SERVICE) != 0) {
        LOG(ERROR) << "Fail to add service";
        return -1;
    }
    g_last_time.store(0, butil::memory_order_relaxed);

    brpc::ServerOptions options;
    options.use_rdma = FLAGS_use_rdma;
    if (server.Start(FLAGS_port, &options) != 0) {
        LOG(ERROR) << "Fail to start EchoServer";
        return -1;
    }
    run_client();

    server.RunUntilAskedToQuit();
    return 0;
}

Expected behavior (期望行为)
IOBuf can append_user_data  with data in RegisterMemoryForRdma
Versions (各种版本)
OS: Ubuntu 22.04
Compiler:  GCC 11.2.0
brpc: 3d90221107000133c2d7f3ccc6372680fb6af27d
protobuf:
Additional context/screenshots (更多上下文/截图)

Thanks. This is a bug. The buf to attach is not the exactly same with the base address registered. I will commit a patch to fix this bug later.

@Tuvie How much work is needed? It's grateful if you can shed some light on the relation between user_data and memory registered by ibv_reg_mr

@Tuvie How much work is needed? It's grateful if you can shed some light on the relation between user_data and memory registered by ibv_reg_mr

A PR is committed: https://github.com/apache/incubator-brpc/pull/1999
Pls try again and refer to the update on docs.
By default, if you use memory managed by IOBuf, you don't need to register memory by yourself. If you manage memory outside the IOBuf, you should register memory explicitly as you did. However, since it is not efficient to lookup the lkey for this kind of user_data, we suggest you maintain your own memory pool and register all the pool once.

Would it be better if MAX_USER_MRS be defined by gflags?

Since linear scan is not efficient. Would interval tree be used to query and update in $\mathcal{O}(\log{}n)$ ?

In my scenario, memory pool is not applicable, because the memory are allocated by allocator in the library out of control. I have tested  RegisterMemoryForRdma with size from KBs to MBs and found that RegisterMemoryForRdma always faster than memcpy.

Since linear scan is not efficient. Would interval tree be used to query and update in O(logn) ?

Currently, it is a tradeoff to avoid using lock for lkey lookup (more frequent operation during data transmission). To design a better solution, here is a question for your scenario: how do you use append_user_data in you application? Since you don't manage memory by yourself, why don't you use IOBuf instead?

@Tuvie We use append_user_data in tensorflow's AsyncOpKernel::ComputeAsync. The memory comes from tensor in TensorFlow OP. It's allocated by the framework on each run.
We could use IOBuf::append, but append introduces an extra copy.

@Tuvie We use append_user_data in tensorflow's AsyncOpKernel::ComputeAsync. The memory comes from tensor in TensorFlow OP. It's allocated by the framework on each run. We could use IOBuf::append, but append introduces an extra copy.

So what do you think if we add a new parameter for append_user_data function to specify the lkey directly? In this case we don't need to lookup the lkey in an inefficient way. But you have to manage the lkey as a corresponding buffer id by yourself.

Yes, we can manage LKey. Does adding parameter to append_user_data  breaks backward compatibility?

Yes, we can manage LKey. Does adding parameter to append_user_data  breaks backward compatibility?

Yes. Must try to keep the ABI compatibility. But I think it is a better way to solve this problem. Because application has more information than brpc so that the overall efficiency will be higher.

Maybe another interface append_user_data_ib :)

@372046933 Pls try this PR: https://github.com/apache/incubator-brpc/pull/2005
A new function named append_user_data_with_meta is provided. You should give the lkey returned by RegisterMemoryForRdma as the meta. Pls note that now RegisterMemoryForRdma returns 0 when failed, just contrast to the previous code.

Thanks a million. Is append_user_data_with_meta in the PR? I couldn't find the definition in https://github.com/apache/incubator-brpc/pull/2005/commits/40a24ab3b5f440d11bfb09c8afb2bc6300157688

Thanks a million. Is append_user_data_with_meta in the PR? I couldn't find the definition in 40a24ab

sorry, I repush the PR. Pls try again.

@Tuvie
If IOBuf::append_user_data with size 0. Could we do early return here?
https://github.com/apache/incubator-brpc/blob/4308819cc4cf52fa52027ab9b10a2dbcaa84d3c4/src/butil/iobuf.cpp#L1209-L1213

If IOBuf::append_user_data with size 0. Could we do early return here?

That's OK.

@Tuvie I have tested append_user_data_with_meta. It works as expected. As for the return value of append_user_data when size is 0. I added a pre-check in application and skip the call when size equals 0.

@372046933 Thanks for your feedback. About the return value problem, maybe @wwbmmm can share your opinion.


pam: re-enable pam_lastlog module

Without the chnage login fails at least in VM tests as:
$ nix build --no-link -f. gjs.tests -L
...
vm-test-run-gjs> machine # [    6.331514] lightdm[1069]: PAM unable to dlopen(/nix/store/9p9ya5n7zi0smc3fb95ck2yvjma26dn5-linux-pam-1.5.3/lib/security/pam_lastlog.so): /nix/store/9p9ya5n7zi0smc3fb95ck2yvjma26dn5-linux-pam-1.5.3/lib/security/pam_lastlog.so: cannot open shared object file: No such file or directory
vm-test-run-gjs> machine # [    6.332529] lightdm[1069]: PAM adding faulty module: /nix/store/9p9ya5n7zi0smc3fb95ck2yvjma26dn5-linux-pam-1.5.3/lib/security/pam_lastlog.so

Description of changes

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

[ ] 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
[ ] 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.

Shall we do
--- a/nixos/modules/security/pam.nix
+++ b/nixos/modules/security/pam.nix
@@ -813,9 +813,6 @@ let
             skel = config.security.pam.makeHomeDir.skelDirectory;
             inherit (config.security.pam.makeHomeDir) umask;
           }; }
-          { name = "lastlog"; enable = cfg.updateWtmp; control = "required"; modulePath = "${pkgs.pam}/lib/security/pam_lastlog.so"; settings = {
-            silent = true;
-          }; }
           { name = "ecryptfs"; enable = config.security.pam.enableEcryptfs; control = "optional"; modulePath = "${pkgs.ecryptfs}/lib/security/pam_ecryptfs.so"; }
           # Work around https://github.com/systemd/systemd/issues/8598
           # Skips the pam_fscrypt module for systemd-user sessions which do not have a password

I suggest doing it separately in master. It's a functional change. And we might as well try https://github.com/thkukuk/lastlog2 as a replacement.


Pydantic 2.4 support

Description

Brings in support for pydantic 2.4
Loosens dependencies a bit, making it easier to install along with other libraries
Adds pylint and mypy sections to poetry configuration (pyproject.toml)
Disable apache-airflow (needs to be manually side loaded as poetry is unable to) - this because apache-airflow supports pydantic 2.*, but poetry's dependency resolced doesn't understand how to resolve it correctly
no new tests needed
Black configuration modernized (120 character width vs 88)

Related Issue
N/A
Motivation and Context
Want to bring in support for pydantic 2.4
How Has This Been Tested?
Unit tests and some integration tests with locally available projects that run against Brickflow
Screenshots (if appropriate):
N/A
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

updates:

[x] reversed the line length change for black (left pylint alone to avoid having to update a bunch of docstrings)
[x] fixed apache-airflow dependency

Please let me know when/if we can get a release around this PR. :)

Please let me know when/if we can get a release around this PR. :)

One more PR pending from @stikkireddy , as it gets fixed we can do 1.0.0. We can target next week


Add WalletSyncApi

The sync account API doesn't really belong to the quorum driver (quorum driver will be stateless). Adding a new json rpc api class for this function.

@lxfind @patrickkuo it looks like this PR breaks the wallet cli <> gateway.
Repro

./sui start
./rpc-server
./wallet switch --gateway http://<IP_ADDRESS>:5001
./wallet and observe error RPC call failed: ErrorObject { code: MethodNotFound, message: "Method not found", data: None }


return Result for Client::from_service_account_key_file

This PR is to make Client::from_service_account_key_file return Result, same as Client::from_service_account_key. It will close #39.

LGTM


How to work with Offscreen canvas?

I have offscreen canvas transferred from main thread to worker, how to use it with Speedy2D?

#101


Installing a plugin from a private PyPI repository

I am trying to integrate a pybuilder plugin that is deployed in a private PyPI repository, but it seems that the link to the repository is being ignored by the plugin downloader. It appears that the plugin is loaded right when the use_plugin() statement is encountered, but the project properties are not set at that point, leading to those 3 parameters being None
@init
def initialize(project):
    # ....
    project.set_property('install_dependencies_extra_index_url', 'url goes here')

After some digging it seems that things are done a bit out of order - first the project module is loaded, which implies loading all plugins and the initializers are executed much later.
As a workaround one can access the project object from the Reactor instance and set the properties required for loading the plugin there:
from pybuilder.core import init, use_plugin
from pybuilder.reactor import Reactor

# loading other plugins...

# required to access a private PyPI repository
project = Reactor.current_instance().project
project.set_property('install_dependencies_extra_index_url', 'pypi url')
project.set_property('"install_dependencies_trusted_host', 'hostname')

use_plugin('pypi:pybuilder_private_pypi_plugin')

@init
def initialize(project):
    # the usual stuff

Hi, @kvalev
Great workaround.

Unfortunately, workaround doesn't work for me - I need to pass install_dependencies_extra_index_url from command line. But PyBuilder applies properties from command line later.

Created #515 which provide global properties from command line to properties immediately.

Did corresponding changes into pypi_server plugin
Now it sets properties on use_plugin step. So usage is:
use_plugin('...')
use_plugin('...')
....
use_plugin('pypi:pybuilder_pypi_server')
use_plugin('pypi:pybuilder_private_pypi_plugin')

@pavan-kumar222 What kind of authentication are you looking for in the private repository?

You can see here the way private repository authentication can be configured outside of PyB: https://pip.pypa.io/en/stable/topics/authentication/

@arcivanov  I will use either username/pwd based authentication or a user token. We will try to use Nexus to host Pypi type private repo.
However my question was in similar lines as the first comment in this issue, Should I use the workaround mentioned here to load the plugin from my private repo or is there a better way, where a plugin is first searched in public pypi and also private repo automatically.

I'll need to create/find a test harness that allows PIP503-compliant repository to test your use case. A lot of things changed since 2017, including internal pip logic.

I've done some debugging, and while the workaround above does works for me - the url for my private PIP repo does indeed get added when PIP is being executed - I'm getting an SSL error, because my repo is using a SSL certificate not trusted by default by PIP. The workaround would work for my, if the --cert option for PIP was exposed I guess.


Trouble incorperating sass-json-vars with compass

Hey everyone I am a dev pretty new to sass and compass, but I cannot for the life of me be able to get this module of yours to work with compass. I have tried to add this line of code to my config.rb file:
gem 'sass-json-vars'  But it had no effect and I still get this error when I am trying to compile my sass:
error scss/custom.scss (Line 7: File to import not found or unreadable: jsonfile.json yet I have made sure my json file is in the same scss folder as everything else. Any help will be great.
P.S. is there any way you guys could add support for compass? That way I can just use a standerd 'require' statment in my config.rb file.

Thanks for getting back to me nhunzaker. I will try that out tomorrow afternoon. Sorry for the late reply. I have just had allot of work as of late. Cheers

I finally got to trying out your suggestion today. It worked very well and so I am glad to say that I am now closing this issue.I think that perhaps I had some error in my .json as well because only after changing the json to that of the example code was I able to get it to work. But either way it works and I am pleased. Cheers

Great to hear!


Add n-way table unions

n-way table unions should be more performant than repeated 2-way unions

This is a WIP, so not reviewable just yet

Should be ready for review now! Note that this PR is built on top of #486.


Add support for type-templated test cases

Original motivation for this was for writing tests for primitive lists/results, but it also turned out to be useful for refactoring some tests to both be shorter and involve fewer macros.

It looks like this needs some adjustments to work with MSVC.


Grading Critera

Technical Acceptance Criteria: 40%

Satisfies all of the preceding acceptance criteria plus the following code improvements:
[ ] - Application's links all function correctly.
[ ] - Application's CSS selectors and properties are consolidated and organized to follow semantic structure.
[ ] - Application's CSS file is properly commented.

Deployment: 32%

Application deployed at live URL.
Application loads with no errors.
Application GitHub URL submitted.
GitHub repository that contains application code.

Application Quality: 15%

Application resembles (at least 90%) screenshots provided in challenge instructions.

Repository Quality: 13%

Repository has a unique name.
Repository follows best practices for file structure and naming conventions.
Repository follows best practices for class/id naming conventions, indentation, quality comments, etc.
Repository contains multiple descriptive commit messages.
Repository contains a quality README file with description, screenshot, and link to deployed application.

Issue resolved


[ISSUE-173][FOLLOWUP] The size of single buffer flush should reach rss.server.flush.cold.storage.threshold.size

###What changes were proposed in this pull request?
follow  #173, when single buffer flush is triggered, the buffer size should reach rss.server.flush.cold.storage.threshold.size,  we keep the same logic as MultiStorageManager#selectStorageManager
###Why are the changes needed?
we keep the same logic as MultiStorageManager#selectStorageManager
###Does this PR introduce any user-facing change?
No
###How was this patch tested?
Already added

Thank you very much @jerqi

UT failed, I revert this patch first.

flushSingleBufferTest failed. You should modify it when we use this patch.

I recreated a pr  #180  @jerqi


indoor cameras stopped updating after installing 0.9.1 add-on

Hello Everyone!
it looks like after installing 0.9.1 add-on, my indoor cameras stopped updating status... whenever I arm/disarm, status is not changing in home assistant.
if I opened the eufy app on my iPhone, status will update.
if I restart HA, status will update.
if I restart the add-on and reload integration, status will also update.
if I restart the whole OS, like restart HA alone, status will update.
in all case when status has been updated after a restart, it will stop updating thereafter.
it looks like there's no "push" update.
there's no error in the add-on or supervisor log.
all was working fine before... not sure why it stopped working now. may be linked to 0.9.1 I just installed since it was working fine before.

Did you update the integration too?

On May 25, 2022, at 5:00 AM, zadigre @.***> wrote:

Hello Everyone!
it looks like after installing 0.9.1 add-on, my indoor cameras stopped updating status... whenever I arm/disarm, status is not changing in home assistant.
if I opened the eufy app on my iPhone, status will update.
if I restart HA, status will update.
if I restart the add-on and reload integration, status will also update.
if I restart the whole OS, like restart HA alone, status will update.
in all case when status has been updated after a restart, it will stop updating thereafter.
it looks like there's no "push" update.
there's no error in the add-on or supervisor log.
all was working fine before... not sure why it stopped working now. may be linked to 0.9.1 I just installed since it was working fine before.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

yes both

that said, I believe it started after installing the add-on... but I'm not 100% sure.
I remember installing an update, either the integration or add-on. next day, the other one was available.
but I do remember after installing the first update, it was still working fine.

i got the same issue, and when i roll back the back up its working fine.
However i cant force it to change status like you I tried restarts of app /integration/addon but status just stays the same while in the eufyapp the changes are made.

just to add more info on this...
the arm/disarm services are still working fine even if status is not updated.
I also have Eufycam 2 Pro... those works as usual (status is updated immediately).
I've found that as soon as I receive a notification on my Eufucam 2 Pro, no matter what is the notification, status on my indoor cameras will also be updated at the same time.
For example, the mailman just came to my door. I received a motion notification from my Eufycam 2 Pro. on my indoor cameras, which was still showing as disarmed since 8am this morning, switched to armed away status after I received this notification from eufycam 2 pro.

also fyi, the eufy security force sync does absolutely nothing to "fix" this issue.

I am using `indoor camera pan&tilt" and I can see that sync is working fine in two ways;

if I change the guard mode on HA, it is synced into eufy app after couple of seconds
if I change the guard mode on eufy app, it is synced to HA after couple of seconds
All these communications are handled by push notifications,

Can you retest with latest add-on version?


Add project API changes required for config schema generation

Purpose
Add project API changes required for config JSON schema generation.
Fixes part of  https://github.com/ballerina-platform/ballerina-lang/issues/32585
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@Dilhasha let's say in the existing config.toml file there is an unused configurable. When you merge generated configs, are you planning to throw a compilation/runtime error or you will issue a WARN during the build?

@Dilhasha let's say in the existing config.toml file there is an unused configurable. When you merge generated configs, are you planning to throw a compilation/runtime error or you will issue a WARN during the build?

We will be generating a JSON schema not the Config.toml
With Beta 4, only this API will be available to get the configurable details as a map. If needed users need to add the compiler plugin to distribution to get the JSON schema generated.
AFAIK there is a change planned to issue an error instead of warning for unused configurable in toml.


Docker-ized MNE not detecting PyVista and Mayavi backends

Steps to reproduce:
Install MNE inside a docker image:
mamba install -c conda-forge -vy jupyterlab ipywidgets ipympl nodejs pytorch torchvision mne mne-nirs

conda list shows tht Mayavi and PyVista both are installed.    Provides the same results in REPL as well as Jupyter Lab Notebook.

echo -e "import mne; mne.sys_info()" | exec python3
Platform:       Linux-5.10.76-linuxkit-x86_64-with-glibc2.31
Python:         3.9.9 | packaged by conda-forge | (main, Dec 20 2021, 02:41:03)  [GCC 9.4.0]

mne:            0.24.1
numpy:          1.22.0 {blas=NO_ATLAS_INFO, lapack=lapack}
scipy:          1.7.3
matplotlib:     3.5.1 {backend=agg}
sklearn:        1.0.2
numba:          0.53.1
nibabel:        3.2.1
nilearn:        0.8.1
dipy:           1.4.1
cupy:           Not found
pandas:         1.3.5
mayavi:         Not found
pyvista:        Not found
pyvistaqt:      Not found
ipyvtklink:     0.2.1
vtk:            Not found
PyQt5:          5.12.3
ipympl:         0.8.5
mne_qt_browser: 0.1.7
pooch:          v1.5.2

Trying to set "Notebook" (or any other) 3D backend:
Provides the same results in REPL as well as Jupyter Lab Notebook.
echo -e "import mne; mne.viz.set_3d_backend(\"notebook\")" | exec python3
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<decorator-gen-182>", line 24, in set_3d_backend
  File "/lib/python3.9/site-packages/mne/viz/backends/renderer.py", line 129, in set_3d_backend
    _reload_backend(backend_name)
  File "/lib/python3.9/site-packages/mne/viz/backends/renderer.py", line 34, in _reload_backend
    backend = importlib.import_module(name=_backend_name_map[backend_name],
  File "/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 850, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/lib/python3.9/site-packages/mne/viz/backends/_notebook.py", line 10, in <module>
    import pyvista
  File "/lib/python3.9/site-packages/pyvista/__init__.py", line 12, in <module>
    from pyvista.plotting import *
  File "/lib/python3.9/site-packages/pyvista/plotting/__init__.py", line 4, in <module>
    from .charts import Chart2D, ChartMPL, ChartBox, ChartPie
  File "/lib/python3.9/site-packages/pyvista/plotting/charts.py", line 13, in <module>
    from pyvista import _vtk
  File "/lib/python3.9/site-packages/pyvista/_vtk.py", line 369, in <module>
    import vtkmodules.vtkRenderingContextOpenGL2  # Necessary for displaying charts, otherwise crashes on rendering
ImportError: libGL.so.1: cannot open shared object file: No such file or directory

Installing OpenGL libraries inside Docker:
I installed all OpenGL libraries inside Docker (libgl1-mesa-glx, ffmpeg, libsm6, libxext6, python3-opencv), but the result remains the same.  MNE sys info does not detect these backends.

Until python -c "import vtk" works, MNE/PyVista will not work. If you do glxinfo | grep profile in your dockerized environment, does it work? You might need to sudo apt install mesa-utils to get it.

... looking at our CircleCI config:
https://github.com/mne-tools/mne-python/blob/main/.circleci/config.yml#L95
You probably need at least libosmesa6 libglx-mesa0 libopengl0 libglx0 -- I assume some of these were not installed by libgl1-mesa-glx...?

Might be worth looking at https://github.com/mne-tools/mne-docker
that has docker files that are close to what I'm actively using.

Closing since I think there are solutions in here that should work, but let's reopen if none of them do


Pytorch 1.6.0 seem to leak memory in conv2d

I'm using Pytorch 1.6.0, and there is only one conv, my codes are as the follows:
inputs = torch.randn(1, 3, 512, 512).cuda()
conv = torch.nn.Conv2d(3, 64, (7, 7), stride=2, padding=3, bias=False).cuda()
output = conv(inputs)
Before execute the conv operation, the GPU memory usage from the nvidia-smi is 1019MB, after the conv operation the GPU memory usage is 1429, and the conv operation consume about 410MB, I know the im2col may consume a huge memory when input size is large. What i can't understand is that after the conv operation the GPU usage not become low. I think there is maybe memory leak in conv2d or there is something wrong in my experiment?

repo wrong


Resolve security questions around ZIP 321 and zcash: handler registration by malicious apps

See https://github.com/Electric-Coin-Company/zashi-ios/issues/1093 and https://github.com/Electric-Coin-Company/zashi-ios/issues/1094 .

User must willfully select the malicious app to handle the intent? Doesn't seem like a realistic or practical attack from my understanding.

Wallet-specific URIs wouldn't be useful or convenient. Seems like a bad idea.

No, it's not correct that a user must willfully select the malicious app. Here's a write-up that covers that misperception.
Background on potential attacks
URI handler interception
It is not possible, on either Android or iOS, for a wallet app to reliably ensure that only it (or only other genuine wallet apps) can register as handlers for a given URI scheme.
On iOS, the behaviour is particularly unhelpful: one of the apps that registers for that scheme will be selected, but there is no user input, no reliable or documented specification of how the selection is made, and nothing in the UI that would indicate to the user that other apps are registered for that scheme.
On Android, the user is presented with a list of apps that have registered for the scheme, but there is no way to prevent a malicious app from having a confusingly similar name and icon to the correct app.
Activating the wrong app can have both privacy implications (the intercepting app gets the URI, and can potentially use the address, amount, etc. for social engineering attacks or in chain analysis), and integrity implications (the intercepting app could pass a modified URI, perhaps paying to the attacker's own address, to the real wallet app that has the user's spending keys).
Clipboard interception
The clipboard is not secure on either iOS or Android. Arbitrary apps can intercept and modify the clipboard contents. There is no way for the user to detect attacks other than checking every character, and no practical workaround.
Proposed solution
It is safe to parse zcash: URIs from a wallet's own QR code scanner. We also propose to register a zcash: handler that only displays a message and then exits.
The message should convey to the user that they need to manually open the wallet they intend to use, and scan zcash: URIs using that wallet's scanning UI.
Hypothesis: this will be more convincing if the security problem is explained to the user.
Proposed wording
You have scanned or followed a zcash: link representing a payment request.
On $OS there is no way to ensure that this way of using zcash: links will reach your intended wallet app.
If a fake wallet app were installed on your device, it could:

gain information about the payment, which could then be used in social engineering attacks or to link this use of Zcash with other transactions;
modify the link and then pass it on to a real wallet app, which could result in you sending money to the wrong address.

These attack possibilities apply when scanning a QR code, following a link, or copying and pasting a link from the clipboard.
To maintain your privacy and security, please instead manually open the Zcash wallet app that you intend to pay with, and use its scanning feature to scan QR codes. If you do this in future then it will be faster as well as more secure: you won't need to go through this screen again, and you will be safer from any malicious apps.
For Zashi the scanner is [brief description of how to get to it].
We can't automatically open Zashi for you because this screen could also be faked by a malicious app. Thanks for reading, and sorry for the inconvenience.
         [Close this message]

It is definitely possible to adapt the behaviour to both follow the human interface guidelines and meet the security requirements. The warning screen tells the user to tap the Home button and go into the app again. The app detects the resulting UIApplication.willResignActiveNotification event and navigates to the app's main screen. This teaches the user the safe way to act on a payment request QR code.

If the app instead just takes the user into the scanning UI in Zashi, how is that any worse? A malicious app that emulates this behavior can't gain any additional capabilities that it didn't already have. I think that the UX cost of what you're proposing here is too high for too little gain, vs. educating the user and then taking them to the camera to re-scan.

That UI doesn't solve the "URI leaks information to malicious app" problem. You have to train the user not to use the system scanner for things that might be payment links in order to solve that.

I think we can close this one, right, guys? @daira @nuttycom

To summarize what was decided:

When scanning a ZIP 321 URI using the system scanner, the user will be redirected to a page in the application that directs them to re-scan using the Zashi scanner.


How to install and what are server requirements, in order to run this app

How to install and what are server requirements, in order to run this app

Hi, PHP 5.6+ MySQL (MariaDB) for run installation.


Adding more branch coverage in client.py.

Rename _RequireCryptoOrDie() to _require_crypto_or_die()
Testing un-traversed GAE and GCE branches in
GoogleCredentials.get_application_default()
Missing env. var. branch in _get_environment_variable_file
Missing APPDATA branch (on Windows) for _get_well_known_file
Renaming some of the long ADC test names
Adding test for GoogleCredentials.get_application_default()
that actually uses the well-known file
Branch in GoogleCredentials.from_stream() when filename is
False-y
Testing _get_application_default_credential_GAE and
_get_application_default_credential_GCE (these just wrap
circular imports at run time)
Testing _require_crypto_or_die

LGTM


Prompting and Evaluation-Query writing and SQL langchain tutorial learning

Continuing to work on writing SQL queries while the model continues to be hosted.

Now that the model is running, come up with some example questions to determine whether or not the model needs to be fine tuned. Come up with a way to evaluate whether or not the model is sufficiently answering the question (LLM evaluating LLM)

Learning SQL langchain tutorial done and query writing done.Prepare example question by each member and tested.


Add examples for publish:bitbucketServer scaffolder action & improve related tests

Hey, I just made a Pull Request!
Contribution of publish:bitbucketServer examples as part of https://github.com/backstage/backstage/issues/18832

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

HI @benjdlambert, @tudi2d
Can you please review this PR.


Multiple GPU support?

We find we must set CUDA_VISIBLE_DEVICES in order to run without error. Setting this to more than one device however we get a file error:
FileNotFoundError: [Errno 2] No such file or directory: 'exp/simsiam2d3d/test_sample/model_20.pth'
Does MiloPYP support more then one GPU?

sorry currently it doesn't support multi-gpu yet. but the code should be able to work with a single gpu.


fix: EnqueueStrategy.All erroring with links using unsupported protocols

This changes EnqueueStrategy.All to filter out non-http and non-https URLs (mailto: links were causing the crawler to error).
Let me know if there's a better fix or if you want me to change something.
Thanks!
Request failed and reached maximum retries. Error: Received one or more errors
    at _ArrayValidator.handle (/path/to/project/node_modules/@sapphire/shapeshift/src/validators/ArrayValidator.ts:102:17)
    at _ArrayValidator.parse (/path/to/project/node_modules/@sapphire/shapeshift/src/validators/BaseValidator.ts:103:2)
    at RequestQueueClient.batchAddRequests (/path/to/project/node_modules/@crawlee/src/resource-clients/request-queue.ts:340:36)
    at RequestQueue.addRequests (/path/to/project/node_modules/@crawlee/src/storages/request_provider.ts:238:46)
    at RequestQueue.addRequests (/path/to/project/node_modules/@crawlee/src/storages/request_queue.ts:304:22)
    at attemptToAddToQueueAndAddAnyUnprocessed (/path/to/project/node_modules/@crawlee/src/storages/request_provider.ts:302:42)
    at RequestQueue.addRequestsBatched (/path/to/project/node_modules/@crawlee/src/storages/request_provider.ts:319:37)
    at RequestQueue.addRequestsBatched (/path/to/project/node_modules/@crawlee/src/storages/request_queue.ts:309:22)
    at enqueueLinks (/path/to/project/node_modules/@crawlee/src/enqueue_links/enqueue_links.ts:384:2)
    at browserCrawlerEnqueueLinks (/path/to/project/node_modules/@crawlee/src/internals/browser-crawler.ts:777:21)

@stefansundin do you plan to finish this? I'd rather not merge such change without any added tests

Hi @B4nan. I started writing a test but I had some more important work come up that took priority.
I may be able to finish it next week.
If you prefer then we can close this PR and open an issue instead.


Remove extraneous % sign from UI strings

Coverage decreased (-20.3%) to 12.039% when pulling a73037674e84fb6c7ea987f9a4f9be9add9d4b6e on mzazrivec:remove_extraneous_percent_signs_from_ui_strings into 74bb352867a7556841ba704ca6b87c9b2ca4b986 on ManageIQ:master.


Begin TypeScript Migration?

Summary description
Migrate project to TypeScript.
Implementation
Looks like an easy win as TS can be adopted incrementally. To add support in this project simply:
yarn add --dev typescript && touch tsconfig.json && dc down && dc up -d

Next.js will populate the tsconfig.json file with some sane defaults as the container comes up and you can start renaming source files to end with ts and tsx. (Note at least one TypeScript file is required in the project before the tsconfig.json file will be autopopulated with defaults.)
Step two would be to get rid of the prop-types dependency. If all goes well, start porting other parts of the framework. Consider pairing this work with https://github.com/reactioncommerce/example-storefront/issues/643.
Motivation
Type safety. IntelliSense. Better static code analysis resuling in fewer regressions.
Notes
Doesn't seem to work on the v3.1.0 tag but worked for me on trunk. Probably a Next.js update which allows the tsconfig.json file to auto-generate.

Incremental Migration to Typescript indeed is really simple with nextjs.
However, I think this is a general decision that needs to be made across the stack.
On the api, migration to typescript could come with other benefits and help generalizing schemas.

:tada: This issue has been resolved in version 4.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Type text/plain not valid in HTML validator.

Error: A script element with a src attribute must not have a type attribute whose value is anything other than the empty string, a JavaScript MIME type, or module.
<script type="text/plain" data-cookiecategory="analytics" async src="https://www.googletagmanager.com/gtag/js?id=

Any help, please?

Hi @Arnoost,
the type="text/plain" attribute is required, as this is what effectively disables the script. Using type="text/plain" is common practice for cookie consent managers.
You should pass validation errors using one of the following methods:

Remove the async attribute and change src to data-src
Load gtag using the plugin's loadScript() method (example):
<script type="text/plain" data-cookiecategory="analytics">
cookieconsent.loadScript('https://www.googletagmanager.com/gtag/js?id=GA_MEASUREMENT_ID', function(){
    // Script loaded, do something

    window.dataLayer = window.dataLayer || [];
    function gtag(){window.dataLayer.push(arguments);}
    gtag('js', new Date());

    gtag('config', 'GA_MEASUREMENT_ID');
});
</script>

Load gtm instead of gtag (you might need to change your configuration), since gtm can be loaded using the inline way (without the src attribute)
Same as the second option, but this time use the .loadScript() method inside the onAccept/onChange callbacks (might be  difficult if you don't know much about javascript or how the plugin works)

The logic of all these options is simple: if you care about validation, avoid using the src attribute together with type="text/plain" attribute.

Thanks a lot


Android: Alert cancelled make focus lost

Ionic version:  (check one with "x")
2.3.0
I'm submitting a ...  (check one with "x")
[x ] bug report
Current behavior:
The focus is lost when user cancel an alert
Expected behavior:
The focus must stay
Steps to reproduce:
1/ User click on button 1 which launch an alert
2/ User cancel the alert (by clicking somewhere else or on a cancel button)
3/ User want to click again on button 1
4/ The button is not focus at click
Related code:
I prepared a little example to expose what I am saying.
So there is a list of element clickable with checkbox
When I click on the button start it launches the alert. Then I cancel it (by clicking somewhere else for example)
Then I want to click on it again. However instead of clicking on it the screen go back to the top and I am clicking on addElem.
ts file:
@Component({
  selector: 'page-pageOne',
  templateUrl: 'pageOne.html'
})
export class PageOne {

  listElement = [
    {
      label : "One",
      checked : false
    },
    {
      label : "Two",
      checked : false
    },
    {
      label : "Three",
      checked : false
    },
    {
      label : "Foor",
      checked : false
    },
    {
      label : "Five",
      checked : false
    },
    {
      label : "Six",
      checked : false
    },
    {
      label : "Seven",
      checked : false
    },
    {
      label : "Eigth",
      checked : false
    },
    {
      label : "Nine",
      checked : false
    },
    {
      label : "Ten",
      checked : false
    }
  ];

  constructor(private nav : NavController,
              private alertCtrl : AlertController,) {
  }

    addElem(){
      this.nav.push(OtherPage);
    }

    start(){
        let alert = this.alertCtrl.create();
        alert.setTitle("Title");
        alert.setMode("md");

          alert.addInput({
          type: 'radio',
          label: "Do something",
          value: 'SOMETHING',
          checked: true
        });

          alert.addInput({
          type: 'radio',
          label: "Do something else",
          value: 'SOMETHING_ELSE'
        });
        alert.addButton({
          text: 'OK',
          handler: data => {
            console.log("Ok clicked: " + data);
          }
        });
        alert.addButton({
          text: 'Cancel',
          handler: () => {
            console.log("Cancel click");
          }
        });
        alert.present();
    }
}

html file:
<ion-header>

  <ion-navbar color="primary">
    <ion-title>Title</ion-title>
  </ion-navbar>

</ion-header>

<ion-content padding>
  <ion-list>
    <ion-item *ngFor="let elem of listElement" text-wrap>
      <ion-label>

          {{elem.label}}

      </ion-label>
      <ion-checkbox color="primary" checked="false" [(ngModel)]="elem.checked"></ion-checkbox>
    </ion-item>
    <button ion-item (click)="addElem()" class="aprimary"> Select more
      <ion-icon name="add-circle" item-left></ion-icon>
    </button>
  </ion-list>

    <button mode="ios" ion-button color="primary" (click)="start()"> Start  </button>

</ion-content>

css file:
page-pageOne{
.center {
  margin-left: auto;
  margin-right: auto;
  display: block;
  text-align: center;
}
}

Other information:
It seems that there is a margin which is added when the alert is presented (I have more scroll when I cancel it)

Hello! Thank you for opening an issue with us!
I attempted to duplicate your issue, but everything appears to be working fine for me. Here is my Ionic Info output so you know which version of stuff this was tested with:
cli packages: (/usr/local/lib/node_modules)

    @ionic/cli-utils  : 1.9.2
    ionic (Ionic CLI) : 3.9.2

global packages:

    Cordova CLI : 7.0.1

local packages:

    @ionic/app-scripts : 2.1.3
    Cordova Platforms  : android 6.2.3 ios 4.4.0
    Ionic Framework    : ionic-angular 3.6.0

System:

    ios-deploy : 1.9.1
    ios-sim    : 6.0.0
    Node       : v8.3.0
    npm        : 5.3.0
    OS         : macOS Sierra
    Xcode      : Xcode 8.3.3 Build version 8E3004b

Can you provide the output of running ionic info for your project?
Can you provide a GitHub repo of a simple sample application that exhibits the issue you are having?
Thank you for using Ionic

Hello.
Thanks for your fast reply. I will do it tomorrow morning (french hour) so you can reproduce the issue.
Regards

Hello,
So I was able to reproduce the issue explained by my user with a starter project.
The link is here:
fail2Scroll
Here are my ionic info (I tried with latest version of ionic angular as well and it is reproducible):
`
Your system information:
ordova CLI: 6.4.0
Ionic Framework Version: 2.3.0
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.1.4
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 7
Node Version: v4.6.0
Xcode version: Not installed
`
When I click on the alert a div class button-effect is created and added in the button.
`

`
Width and height always have the same value
All the properties depend on the button size.
If you have a button with a single word like "OK" or "Start" the width and height will be short enougth to not have this losting of focus
However if the text of the button is too big the width will be bigger as well as the height so the focus is lost. I attached a picture of the issue.
Here are two example with two different texts ( first is "start" and second is "Start customization")
`

Thank you for your sample application and for your detailed explanation. I will have a look at this later today.

I have tried to duplicate this issue using your repo, but after a few attempts in a couple of different browsers, I cannot. If I understand correctly, the issue you have is:

start with long text in button (Start customization)
click that button
dismiss the alert by touching outside of alert
hit button again
??? something other than alert happens ???

For me, once I get to item #5, I get the alert again, so it appears to be working fine for me. I tried this on Chome, Firefox, Safari, and the Android emulator.
Is my understanding of your issue correct?

Hello
Thanks for your investigation. So basically nothing appears really (we cannot view it).
I only viewed it when I highligthed it with chrome debugger.
The issue does not occures on emulator and on browser unfortunately. I was just able to reproduce on mobile.
Galaxy s6, s7 and lg g5.
In fact as I said in the button a new div is created when alert is launched ( the one I showed in the image)
I made a little video to explain the steps.
1/ I Click on next page
2/ I scroll to the really bottom of the page (I cannot go further)
3/ I click on the alert and cancel it (by clicking somewhere else)
4/ I continue scrolling (at that moment I can go further)
5/ I restart to click on the button but as you can see the check box is selected (while I tried to click on the button)
Hope it will help.
The solution would be to remove this div if the alert is cancelled
2017_08_23_20_33_20.zip

Hello all! @speedfl so I also pulled down your repo and tested it on my devices. One is a Nexus 6P running Android 8.0 and the other is a Nexus 6 running Android 7.1.1 and unfortunately, on both of those devices, I was not able to reproduce this issue. Because of this I would recommend taking this issue to our forum or our slack channel. Thanks for using Ionic!


[Bug] Log Errors for State during Startup

Describe the bug
This is not something that prevents operation
This error shows up in logs after startup:
"Received invalid light is_on state: unavailable. Expected: on, off, true, false"
Appears to be associated with the warning about using states here https://www.home-assistant.io/docs/configuration/templating/#home-assistant-template-extensions
To Reproduce
Steps to reproduce the behavior:

Restart HA
Observe errors in log

Expected behavior
Hoping for fewer errors in log.
System configuration
System: Proxmox VM
HA Version: 2021.3.4
SengledApi Version: 0.2
home-assistant.log
2021-03-30 16:28:21 ERROR (MainThread) [homeassistant.components.template.light] Received invalid light is_on state: unavailable. Expected: on, off, true, false

I will add this to the list, but it could take me awhile. In the meantime
you roll back you version of HA.
On Tue, Mar 30, 2021 at 6:58 PM Michael D Reynolds @.***>
wrote:

Describe the bug
This is not something that prevents operation
This error shows up in logs after startup:
"Received invalid light is_on state: unavailable. Expected: on, off, true,
false"
Appears to be associated with the warning about using states here
https://www.home-assistant.io/docs/configuration/templating/#home-assistant-template-extensions
To Reproduce
Steps to reproduce the behavior:

Restart HA
Observe errors in log

Expected behavior
Hoping for fewer errors in log.
System configuration
System: Proxmox VM
HA Version: 2021.3.4
SengledApi Version: 0.2
home-assistant.log
2021-03-30 16:28:21 ERROR (MainThread) [homeassistant.components.template.light] Received invalid light is_on state: unavailable.
Reply to this email directly, view it on GitHub
https://github.com/jfarmer08/ha-sengledapi/issues/44, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEKHFB3AZGCJAPHU2JLU3ITTGJQT7ANCNFSM42DH4ISQ
.

--
Thanks Allen :)


[4.0] Breadcrumb alignment on rtl theme

Bug report
When "rtl" is chosen for html_direction in config, the breadcrumb alignment is still like "lrt" setting.
What I did:
I changed justify_content_end class to justify_content_start in inspector and it worked.
What I expected to happen:
I think adding a condition on breadcrumb view file can fix it.
What I've already tried to fix it:
I am working on it and I think it takes 2 hours.
Backpack, Laravel, PHP, DB version:
Backpack 4.0
Laravel 7.0

Fixed by #2691


Dialog renderer and css not registered when using custom configuration

I'm submitting a bug report

Library Version:
1.0.0-rc.1.0.3

Please tell us about your environment:

Operating System:
Windows 10

Node Version:
6.9.1

NPM Version:
4.5.0

Browser:
all

Language:
TypeScript 2.X

Current behavior:
I'm overriding the header template with my own. This is my configuration:
  aurelia.use
    .standardConfiguration()
    .plugin('aurelia-dialog', (config: DialogConfiguration) => {
      config
        .useResource('ux-dialog')
        .useResource('ux-dialog-body')
        .useResource('ux-dialog-footer')
        .useResource('attach-focus');
    })

Unfortunately with this configuration I'm getting 'DialogRenderer must implement getDialogContainer().' because the default renderer is not registered in custom mode. Looking at the source code of DialogConfiguration, it is not possible anymore to register only custom resources keeping the default resources (ie: css and renderer). This could be by design (and is a breaking change) but unfortunately the default dialog renderer is not exported so I'm unable to register it manually (ie: config.useRenderer()). The same is true for the css styles as well.

Temporary hack:
      config.useDefaults();
      let resources = <Array<string>>(<any>config).resources;
      resources.splice(resources.indexOf('ux-dialog-header'), 1);


PyTorch implementation is way slower than Tensorflow

Hi!
First of all, thanks for the great work!
I have compared both implementations (on CPU) and Torch is much slower.
I unfortunately have to use Torch so if you have any idea on how to make it faster?
Thanks

Torch is not optimized for fast inference. Use ONNX or similar for optimized inference.


Module file structure

It would be really great if Este had some modular per-app file structure.
Like:
--common
--settings
--helpers
--apps
--dashboard
--users
--companies
--whatever

Did I overlooked something? It has.


Buggy in official OpenWrt 22.03.3

Module: Phicomm K3
Hardware: BCM5301X
CPU: ARM_Cortex-A9 [ARMv7 Processor rev 0 (v7l)]
Version: OpenWrt 22.03.3 r20028-43d71ad93e / LuCI openwrt-22.03 branch git-22.361.69894-438c598
Kernel: 5.10.161
Bugs:

Cannot download SpeedTest-CLI. Download button always in progress, and still show "Not Installed" after refresh. But there is speedtest & speedtest-cli in /usr/bin/

root@OpenWrt:/usr/bin# ls speed* -al
-rwxr-xr-x    1 root     root           980 Sep 19  2021 speedtest
-rwxr-xr-x    1 root     root           988 Sep 19  2021 speedtest-cli
root@OpenWrt:/usr/bin# ./speedtest-cli --version
speedtest-cli 2.1.3
Python 3.10.7 (main, Jan 25 2023, 15:45:14) [GCC 11.2.0]

Web shows "No related RPC reply" in "Ookla SpeedTest" page, and always in progress, no more results.
Screenshot here:

In default, the speedtest use package python3-speedtest-cli, the efficiency of this package is relatively low, and the single test time will be as long as 5+ minutes (RPC timeout is 3 minutes).
The usual solution is to refresh the page after waiting for more than 5 minutes
Ookla SpeedTest may fail to download due to network problems. (unstable connection, ip is recognized as a robot...)
You can manually download the corresponding architecture version from https://www.speedtest.net/apps/cli, put it to /usr/libexec/netspeedtest/speedtest
and grant execute permission chmod +x /usr/libexec/netspeedtest/speedtest

In default, the speedtest use package python3-speedtest-cli.
While it is compatible with the most platforms, but efficiency is relatively low. the single test time will be as long as 5+ minutes (RPC timeout is 3 minutes).
The usual solution is to refresh the page after waiting for more than 5 minutes, then click test again if there is no result.

I've waited for hours, which still shows "Testing in progress..."
Then I found the file /tmp/speedtest_result exists for days.
root@OpenWrt:/tmp# cat /tmp/speedtest_result
Testing

And the python3-speedtest-cli works well in commandline mode :
root@OpenWrt:/tmp# /usr/bin/speedtest
Retrieving speedtest.net configuration...
Testing from xxxxxxxxxxxxxxxxxxx...
Retrieving speedtest.net server list...
Selecting best server based on ping...
Hosted by xxxxxxxxxxxxxxxxx [77.57 km]: 11.911 ms
Testing download speed................................................................................
Download: 56.37 Mbit/s
Testing upload speed..................................................................................
Upload: 23.34 Mbit/s

Ookla SpeedTest may fail to download due to network problems. (unstable connection, ip is recognized as a robot...)
You can manually download the corresponding architecture version from https://www.speedtest.net/apps/cli, put it to /usr/libexec/netspeedtest/speedtest
and grant execute permission chmod +x /usr/libexec/netspeedtest/speedtest

Tried the official ookla speedtest biniary, same as python3-speedtest-cli :
page shows "Testing in progress..." for hours and run biniary in ssh is ok.
root@OpenWrt:/tmp# /usr/libexec/netspeedtest/speedtest

   Speedtest by Ookla

      Server: xxxxxxxxxxxxxx
         ISP: xxxxxxxxxxxxxx
Idle Latency:    17.32 ms   (jitter: 0.15ms, low: 17.25ms, high: 17.58ms)
    Download:   155.93 Mbps (data used: 220.8 MB)
                 19.21 ms   (jitter: 3.54ms, low: 17.33ms, high: 269.10ms)
      Upload:    23.15 Mbps (data used: 35.1 MB)
                 19.26 ms   (jitter: 26.23ms, low: 14.93ms, high: 269.44ms)
 Packet Loss:     0.0%
  Result URL: https://www.speedtest.net/result/c/xxxxxxxxxxxxxxxxxxxxxx

I've waited for hours, which still shows "Testing in progress..."
Then I found the file /tmp/speedtest_result exists for days.

This seems to be that the init.d program did not exit properly, it should show 'Test failed' normally
You can delete /tmp/speedtest_result or execute the test again
And you can also execute /etc/init.d/netspeedtest speedtest to display the results on the page
I will update the detection logic later


Swag ignores package path if it starts with vendor

Describe the bug
I was running swag in my package -> vendor-service. But due to a vendor prefix check, all the files are being ignored and swagger docs are not generated.
To Reproduce
Steps to reproduce the behavior:

Create a package with starts with "vendor"
Add an API and swagger annotation to that API
Run swag init
Check docs/swagger.json does not have the swagger documentation data

Expected behavior
Swagger documentation should get generated similar to packages that don't start with "vendor"
Your swag version
v1.16.3
Your go version
1.21.6

As info.PackagePath will always have the prefix of the package where swag was run, do we need this check?
If we do can we restrict it to vendor/ paths?
Would be happy to raise a PR to get this fixed.

swag init  --parseVendor

@sdghchj I had tried this config but it didn't help with my issue. From what I could tell, the parseVendor config is used only in the getAllGoFileInfo function for this check:
			if !parseVendor && f.Name() == "vendor" || // ignore "vendor"
				f.Name() == "docs" || // exclude docs
				len(f.Name()) > 1 && f.Name()[0] == '.' && f.Name() != ".." { // exclude all hidden folder
				return filepath.SkipDir
			}

But if the file name starts with "vendor" like in my case ("vendor-service"), the skip dir doesn't happen with or without the parseVendor config. In my case, issue is in the parser.packages.RangeFiles function which runs after the getAllGoFileInfo fn, where file is skipped if it starts with "vendor", and it doesn't use the parseVendor config to skip this check:

for _, info := range pkgDefs.files {
		// ignore package path prefix with 'vendor' or $GOROOT,
		// because the router info of api will not be included these files.
		if strings.HasPrefix(info.PackagePath, "vendor") || strings.HasPrefix(info.Path, runtime.GOROOT()) {
			continue
		}
		sortedFiles = append(sortedFiles, info)
	}

Hi @sdghchj, just following up on this PR. Let me know if there's anything else needed from my side. Thanks for your time!


Get the different device names in the backend

It would be nice to see the names of the used iOS devices in the analytics. Currently it only says iPhone for all iPhone models including the simulator.

The current information displayed in the Piwik Backend is read from the UserAgent. Setting more specific information (e.g. the specific device model) is not a normal thing for the user agent and, as far as I know, not supported by the Piwik Backend.
But, there is a way to get more information about the devices your users are using. Imho the best way would be to use Custom Dimensions. Custom Dimensions are currently in development. Once it's implemented and merged the way to use would be similar to this.


Add Assert URL  for codimd

ATM,
we can set simple string for codimd

we need to force url  in API using assert ( https://symfony.com/doc/current/reference/constraints/Url.html )
there : https://github.com/CMEPW/Smersh/blob/master/api/src/Entity/Mission.php#L59

https://github.com/CMEPW/Smersh/pull/30  i did it there


Engine: add basic proptest for titan

What is changed and how it works?

Issue Number: Ref #16259

What's Changed:
Add property test for titan's basic operations: get/put/delete/scan/delete_range

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

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

Side effects

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

Release note

None

/merge


Service cover.set_current_position not available

ESPSomfy-RTS-HA version
v2.4.1
What happened? What did you expect to happen?
I want to use the "set_current_position" service to correct for an incorrect curtain position due to delay's caused by the command lenght and the curtain to respond. However the "set_current_position" service is not available.
How to reproduce it (step by step)
1. give command

service: cover.set_current_position
data:
  entity_id: cover.woonkamer_klein_raam
  position: 18

2. receive error

Logs
N/A

@nl0pvm did you get this resolved?

did not have time yet :)

Tested; it just does not matter what YAML i use ..... Both in homeassistant.log (and the GUI) the following error is thrown:
 Error handling message: Service cover.set_current_position not found.

I am sorry I didn't play close enough attention to your yaml above you need to be using the espsomfy_rts domain for the service.  The HA default cover does not contain a service for setting the current position.
service: espsomfy_rts.set_current_position
data:
  position: 93
target:
  entity_id: cover.woonkamer_klein_raam

Thanks, tested, Works.
Maybe to add this example to  to the documentation at https://github.com/rstrouse/ESPSomfy-RTS-HA/wiki/Services as without you never think of using the espsomfy_rts service instead of cover. :)

Thanks that wiki needs some love.  I'll update it when I find some time.


New version: SoftwareFreedomConservancy.QEMUGuestAgent version 102.10.0

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.2 schema?

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

Microsoft Reviewers: Open in CodeFlow

The installer URLs use the same domain and path structure as the previous version. It also downloads fine with no server blocks in place (as far as I can tell). Looking at the previous version's PR (#37032), it seems this same error occurred. Can we just continue merging anyway?


rebrand MDE and add proper channel and event data

Hi @olafhartong , Thank you very much for the PR. I see that these changes follow the same schema from your previous PR (Log provider, log channel and event id). I will merge the PR.
Best,
Jose


Some of the example programs delete a void * render object

See compile errors on Raspberry Pi.

Fixed in eaeb99394a829f5f12009d6db9cd1e5d5a0e3a9d


[Snyk] Security upgrade golang from latest to 1.18-bullseye

This PR was automatically created by Snyk using the credentials of a real user.As this is a private repository, Snyk-bot does not have access. Therefore, this PR has been created automatically, but appears to have been created by a real user.
Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

vendor/github.com/hpcloud/tail/Dockerfile

We recommend upgrading to golang:1.18-bullseye, as this image has only 103 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

261
Uncontrolled Search Path Element SNYK-DEBIAN11-GIT-2635965
No Known Exploit

261
Uncontrolled Search Path Element SNYK-DEBIAN11-GIT-2635965
No Known Exploit

364
CVE-2022-1271 SNYK-DEBIAN11-GZIP-2444256
No Known Exploit

221
CVE-2022-1587 SNYK-DEBIAN11-PCRE2-2808697
No Known Exploit

221
CVE-2022-1586 SNYK-DEBIAN11-PCRE2-2808704
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

That is 'vendor' code - we don't make changes in that directly.  We would need to update the version of the dependency.


wordArray Updated

Fixes #126
Updated wordArray
Added a text to indicate the purpose of the checkbox in the header of the popup, updated wordArray in background.js to include the new and missing headers. Additionally centered some poorly formatted headings in the popup for uniformity.
Types of changes

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

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

SWOC Participation

Name: Madhav Ram
Discord Username: Madhav#7781
[x] I have included the issue number in my PR

I have updated the wordArray along with previous changes.
I closed the previous PR as I changed the branch name. Sorry about this, it's my first time working with merge conflicts.


AnkiConnect failed to fetch

I added https://killergerbah.github.io to ankiconnect but it doesn't work. Do you know how to fix it?

@yona19 Some questions:
What browser are you using? Do you have any ad blockers enabled?
Can you paste the logs that appear when trying the AnkiConnect URL in asbplayer?
Go to Chrome menu -> Other tools -> Developer console to see logs.

I changed the browser to Chrome and it worked. Thank you I really appreciate it

Glad it works now. Curious what browser you were using? I know with Brave you have to add the asbplayer site to a whitelist.

Sorry for the late response. I was using brave.


net: ipv4: Fix ARP probe check in address conflict detection

The second condition needs to check ARP probes only

Could you also add text from your issue to the commit message, at least this could be put there
The ACD is not properly implemented as described in RFC5227 ch. 2.1.1
The implementation incorrectly detects an IP conflict, if an ARP request is received for the target IP.
The reason is that the current implementation checks for ARP requests instead of ARP probes.

changed commit message and elminated local variable


Unknown props warnings on div since React 15.X

Passing props should be filtered on div to avoid development warnings with React 15.X:
Warning: Unknown prop `defaultDate` on <div> tag. Remove this prop from the element. For details, see https://fb.me/react-unknown-prop

Why was this closed ? Is it fixed in a new release?


SetPagesVisibility and EnsureScrollBars issue

Following commit makes PdfPageItemsControl unstable: SetPagesVisibility and EnsureScrollBars do not behave properly in AOT dotnet8 (behaviour is not consistent over different builds...)
26dd7d52b1fd93ad42701176d518598f1c2e7e36
The attempt to fix the problem in
6bb558f719a76a4b561ebd963308fcf9fcaf072d
    private void OnLayoutUpdated(object? sender, EventArgs e)
    {
        LayoutUpdated -= OnLayoutUpdated;

        Dispatcher.UIThread.Post(() =>
        {
            EnsureScrollBars();

            // Ensure the pages visibility is set when OnApplyTemplate()
            // is not called, i.e. when a new document is opened but the
            // page has exactly the same dimension of the visible page
            SetPagesVisibility();
        }, DispatcherPriority.Loaded);
    }

is still unstable.
Switching to dotnet9 looks to fix the problem (difficult to assess as problem is not consistent)

Hack for scroll bars is linked to issue in avalonia, should be fixed when the following is merged
https://github.com/AvaloniaUI/Avalonia/pull/17461


A problem about data preprocessing

Excuse me,I am just a beginner for ACE Event Extraction,Thank you for sharing your code.But I have some  problems about prepareDataSet_joint.py as follows:

what should I put in "dataDir",in your example,the directory is "../ni_data/EngMix/dev(train/test)/ ",according to the dataset of ACE2005,which type of file should I put in?
When I run your code,I find some problems about in the function "obtainAllEvents",in details,is "cPickle",I get the error message saying "cPickle.UnpicklingError invalid load key",does it has something to do with the input files?

Thank you ! and please forgive my poor English.

Thanks for attention.

data files in ../ni_data/EngMix/dev/ include all *sgm and *apf.xml of dev files. For example, CNN_IP_20030409.1600.02.sgm, CNN_IP_20030409.1600.02.apf.xml

you should get *.ee files first with eventExtFromRaw.py.  For example, CNN_IP_20030409.1600.02.ee

Good luck.

Thank you so much,I will have a try.

Sorry,Can I ask you for the ACE2005 original dataset？I just cannot find the corpus on the net.

Sorry about this. We could not release original dataset due to copyright issue.
".sgm" may be same with us. I am not sure about ".sgm.apf.xml". You may try change ".apf.xml" to ".sgm.apf.xml" in the code.
By the way, mine dataset is also slightly different with dataset provided by some collaborators. Not sure how many version there are..

Thank you ! The dataset on the LDC really costs too much.


Update in the DMG theme

Adding correct drop cap letters colors, and correct table colors when inside a note.

Thanks @5e-Cleric ! 💯 👍 😎


User is not able sign aggregate bonded transaction

AllNodes is initializing aggregate bonded multisig modification. For example (It's Ledger but same issues on non-Ledger wallets):

User is not able to sign it. Password is provided, pop-up closes but transaction never get confirmed.
After deadline it's of course is pruned:
http://explorer.symbolblockchain.io/transactions/C1CEC7611AAD7CDD0FCACFB7BC41443D19C2EA70702ACE3284322D30454DF868

I could not reproduce. Asked AllNodes user if he can.

AllNodes told that nobody asked about that issue anymore. I'm closing.


rawgit is ending! move rawgit to jsdelivr.

Trello
rawgit is ending! read at the home page: https://rawgit.com/ we need to change all the rawgit path to an alternative.

Thank you @Ouyang-Zhaoxing to repost it!

Try using githack:
https://raw.githack.com/jeromeetienne/AR.js/1.7.0/aframe/build/aframe-ar.js

Someone is will to make a PR for that? Using githack seems good
it should not be a big thing, just a find-and-replace and a quick test

https://github.com/jeromeetienne/AR.js/pull/538

Well done guys!


Current Behavior

Expected Behavior
wandb 创建launch报错 Variable "templateVariables" has invalid value null. Expected type "JSONString!", found null.
Steps To Reproduce
No response
Screenshots
No response
Environment
OS:
Browsers:
Version:
Additional Context
No response

Hello @xiaodongmei !
I am not completely following what error occurred. Did you get the message Expected type "JSONString!", found null. in the UI or within the launch job? Also, in your screenshot, I am unable to see "templateVariables". Where was this input into?


Add getClient function stub

For Go

Current coverage is 81.20% (diff: 100%)

Merging #471 into disco_head will not change coverage

@@           disco_head       #471   diff @@
============================================
  Files             183        183
  Lines            6969       6969
  Methods             0          0
  Messages            0          0
  Branches          908        908
============================================
  Hits             5659       5659
  Misses           1037       1037
  Partials          273        273

Powered by Codecov. Last update 9ecf88b...2f86b9a

LGTM
Amend PR name to clarify this is for Discovery


[FEAT] New option: "After reviewing a note, postpone all of the note's flashcards till after today"

Is your feature request related to a problem? Please describe.
There is an existing option in the "Flashcard" section "Bury sibling cards until the next day". It currently is used to control two different things:

During flashcard review, where there are multiple cards associated with a single question
During note review, to "bury" all flashcards within the note (this is very unintuitive), as discussed in #788.

Describe the solution you'd like
A new option should be added called "After reviewing a note, postpone all of the note's flashcards till after today"
It defaults to the current value of the "Bury sibling cards..." setting
The existing "Bury sibling cards..." option would only pertain to function (1) above
The new option would control function (2) above.
Additional context
I think that the new option should be in the "Notes" section of settings, rather than "Flashcards"

Hi  @LennyPhoenix I've just added this feature request.
Could you please review in terms of solving #788, before I implement.
Cheers
Ronny

Yep this should sort it!

Hi @LennyPhoenix
Beta release is now available.
There is a file called main.js in this zip file
main_791A.zip
You will need to copy that to the obsidian plug-in folder, e.g.
D:\Obsidian\Obsidian\.obsidian\plugins\obsidian-spaced-repetition
Then reload the plug-in, or restart obsidian. Hopefully solves #788.
Any feedback appreciated.
Ronny


PWM to drive over 16 channels

Is your feature request related to a problem? Please describe.
if the need is over driving 16 channels,  it means we must to use another object for 16 additionnal channels if I m not wrong?
juste to be sure how to process on that case.
So what happen if non other hardware TIMER still available?? (usually One is for millis(), one will be for ISR_Timer interupt generating this PWM, another one can be keep for core operations (in my case NMRA DCC signal management).... so few solution available!
Worst case on megatiny CPU with less TIMER ( A0 B0 B1 only)
Describe the solution you'd like
May it would be possible to limit frequency ( ex between 50hz-120hz range  and add PWM channels on the 16 ones on the initial range as a way? (may be not at all!)
Describe alternatives you've considered
using biggest hardware on the family to have enough TCB Timer ressources is the intial possibility if available!
other is to get capacity or possibility to move millis() on a TIMER A type? (not sure it would be well managed by dx core or megacorex and megatinycore?)
May require some additionnal settings according optional links between TIMER A type and B type.
using millis on same timer as the one managing PWM? (stupid way? uncompatible?)
Additional context
optimisation of selected hardware according usages needs specially for limited TIMER numbers.

Hi @LaurentR59
The current 16-channel limitation is artificial and arbitrary decided to avoid overloading the board. I'll create a new version for this feature.
Anyway, I'll make it configurable in users' code, and will be max 48.   But I think for slow PWM (low PWM frequency), the max number of channels can be very large.
Is that value 48 reasonably large enough ?
You can also use many Timers, if available, by adding the Timers and similar code for 1 Timer.

other is to get capacity or possibility to move millis() on a TIMER A type? (not sure it would be well managed by dx core or megacorex and megatinycore?)

With DxCore, you can move millis() to other Timers, such as TCA, TCD

Hi
48 is really large number but will be helpfull a lot and d ne perfect for slow timings and could cover a large range of usages with many cpu pin version.(8 to 48 or 64!)
Usually it s required  to avoid led flickering so. 50/60hz or 100/120hz is good and enough! ( specially for video) Should be adapted for code facility around these values or possibilities.
Probably as an extension il d helpful to have such extent capacity for all of your others pwm_slow libs (rp2040, atmega, avrmega,… ) on same kind of build if possible. (Easiest way to build code project by #if defined(xxxcpu model xxx) and including header files and some variables for modular approach with similar core code between the different lib usages
Sorry to requiring so much change!
Anyway sincerely really appreciate!
Regards
Laurent
Envoyé de mon iPhone

Le 25 août 2022 à 17:41, Khoi Hoang @.***> a écrit :

Hi @LaurentR59
The current 16-channel limitation is artificial and arbitrary decided to avoid overloading the board. I'll create a new version for this feature.
Anyway, I'll make it configurable in users' code, and will be max 48. But I think for slow PWM (low PWM frequency), the max number of channels can be very large.
Is that value 48 reasonably large enough ?
You can also use many Timers, if available, by adding the Timers and similar code for 1 Timer.
other is to get capacity or possibility to move millis() on a TIMER A type? (not sure it would be well managed by dx core or megacorex and megatinycore?)
With DxCore, you can move millis() to other Timers, such as TCA, TCD
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

OK, hear you, I'll set max number of channels to 64 and configurable in your code.
Will modify those slow PWM-related libraries similar way. The next release will be out within today.
Please test then, with max 64 channels, if possible.

Hi @LaurentR59
The new Dx_Slow_PWM releases v1.0.1 has just been published. Your contribution is noted in Contributions and Thanks
Looking forward to receiving more contributions from you.
Best Regards,

Release v1.0.1

Make MAX_NUMBER_CHANNELS configurable to max 64 PWM channels
Remove debug codes possibly causing hang
Improve debug to use Serialx port automatically according to boards

Our messages cross temselves!
Configurable value up to 64 is just perfect!! ( cover all case hardware)
And can be set to requested level
Thanks
Laurent
Envoyé de mon iPhone

Le 25 août 2022 à 20:50, Khoi Hoang @.***> a écrit :

Hi @LaurentR59
The new Dx_Slow_PWM releases v1.0.1 has just been published. Your contribution is noted in Contributions and Thanks
Looking forward to receiving more contributions from you.
Best Regards,
Release v1.0.1
Make MAX_NUMBER_CHANNELS configurable to max 64 PWM channels
Remove debug codes possibly causing hang
Improve debug to use Serialx port automatically according to boards
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

You now just declare in your code (any number up to 64). Will be trimmed to 64 if bigger than 64.
https://github.com/khoih-prog/Dx_Slow_PWM/blob/fd340604528a7073d8d38792d2f4cbee02e5056f/examples/ISR_8_PWMs_Array_Complex/ISR_8_PWMs_Array_Complex.ino#L39-L40

Hi
64 channels means that 64 I/O pins existing and can be used if they’re  aivalable.
48 pins CPU Like AVR Dx Ofer only 41 I/O
64 pins version propose 55 I/O
32 pins CPU match with 27 I/O
Same on atmega or megatiny.
So 64 is over physical capacity according to one pin per channel rule. ( or I might wrong !?)
64 is may be « pleasant for factor 2 » settings?
On rp2040 only 30 I/O
So may be compromised around  32 ( or 48 as max but I think 64 is too much and without target usage for existing cpu although it could cover largest 64 pins cpu)
With only curiosity nano avr128da48 board I could not make test over the limit explained earlier.
But I will for more Than 16 like 24 on my labo board.
Laurent
Envoyé de mon iPhone

Le 25 août 2022 à 20:14, Khoi Hoang @.***> a écrit :

OK, hear you, I'll set max number of channels to 64 and configurable in your code.
Will modify those slow PWM-related libraries similar way. The next release will be out within today.
Please test then, with max 64 channels, if possible.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.


[ADP-2504] add Publish Docs to bors status dependencies

[x] add 'Publish Docs' dependency to bors

Issue Number
ADP-2504

bors try

bors try

bors try-

bors try


Require the connection lock earlier in iterate

Iterate requires starting a transaction which can error
with "operation already in progress" if another query
is being executed by a different task.
Iterate did acquire a lock before, this commit simply
moves it to before the transaction is started.

If we want to predictable behaviour when sharing a connection, I'm assuming we do given we've setup the lock system, there's a potential problem regarding transactions we may want to address.
If two tasks share the connection and one of them starts a transaction, all changes made by the other task will affect the transaction which can create conflicts. We may want to add further locking in the transaction to prevent that.


What certificates are required for docker?

The readme mentions two certificates that we must provide for local development:

In order to run EFGS locally two certificates (in addition to your personal authentication and signing certificate) need to be created: TrustAnchor Certificate and Outbound Callback Certificate. To create them just execute the create_trustanchor and create_callback_client_certificate script from tools directory. These scripts will generate certifcates and pack them into a Java Key Store.

What are the specifications of these certificates? Can I just generate two X.509 certificates with openssl and use them?

Hi Ryan,
sorry for the late response. Do you still need some support regarding this topic or can we close this issue?

Hi Ryan,
sorry for the late response. Do you still need some support regarding this topic or can we close this issue?

Closing this because of inactivity.


webview: TypeError: Cannot set property 'scrollTop' of null

My Setup

Operating System: Windows 10 (1809)
App Version: 2.15.3
Installation type: exe

[x] I have tested with the latest version
[x] I can simulate the issue easily

Description
Upon upgrading RocketChat server, the last displayed conversation goes blank. All other conversations are fine. The developer tools window displays the following:
I guessed it had something to do with cache, but I did "Reload ignoring cache" and it did not help. This does not happen on the web-based access to the rocketchat endpoint, only on the Electron client.
webview: TypeError: Cannot set property 'scrollTop' of null
    at https://myserver.domain.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:281:282689
    at t._compute (https://myserver.domain.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:43:2321)
    at new t (https://myserver.domain.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:43:1452)
    at Object.s.autorun (https://myserver.domain.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:43:3998)
    at L (https://myserver.domain.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:281:281867)
    at g.action [as _action] (https://myserver.domain.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:281:207847)
    at g.callAction (https://myserver.domain.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:161:10091)
    at https://myserver.domain.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:161:6845
    at Object.s.nonreactive (https://myserver.domain.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:43:4179)
    at https://myserver.domain.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:161:6692

Current Behavior
Screen goes blank on last conversation.
Expected Behavior
The last conversation should be properly loaded.

This happened while upgrading the server to develop commits 5e60a8a and 482ae2d (latest develop).
Client does not seem to have this issue with commit 309335e

Seems there are more errors:
webview: Exception from Tracker recompute function:
webview.js:52 webview: Error: Expected template rendered with Blaze.render
    at Object.f.remove (https://kaiwa.epginsurance.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:87:24512)
    at f.close (https://kaiwa.epginsurance.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:281:260133)
    at Array.from.map.t (https://kaiwa.epginsurance.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:281:260398)
    at Array.map (<anonymous>)
    at f.closeOlderRooms (https://kaiwa.epginsurance.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:281:260386)
    at f.open (https://kaiwa.epginsurance.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:281:260630)
    at https://kaiwa.epginsurance.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:281:282018
    at t._compute (https://kaiwa.epginsurance.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:43:2321)
    at new t (https://kaiwa.epginsurance.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:43:1452)
    at Object.s.autorun (https://kaiwa.epginsurance.com/9b216696b264cec2c24a4e68ee99e13b19a294ba.js?meteor_js_resource=true:43:3998)

@tassoevan is there any kind of process/migration script I would have to run to correct this? I just find it odd that the web client has no issues.

@bkraul Could you please check which values are listed on emoji.recent variable (Application > Local storage)? I've seen a similar issue after EmojiOne change to JoyPixels.

Here it is:
ohmy,triumph,scream,fingers_crossed,thumbsup,nauseated_face,rubber_duck,gumo,ocasio,pelosi,aoc,sekiro_death,poop,pbear,amiwrong,dizzy,trump_eyeroll,middle_finger,ayaya,rolling_on_the_floor_laughing,filter,jobs,trump,+1_tone3,arrow_upper_right,cry,sunglasses,musical_note,crying_cat_face,smirk,maga,klansman,takei,laughing,pedobear,kkk,scream_cat,mueller,smile,cat,frowning2,rainbow_flag,wink,rolling_eyes,dog2,smile_cat

I have a few custom emoji in this list. Would I need to be disabling some of those?

I have updated to develop build based on commit 53093e0. Everything seems to be working with no further blank pages. Just for kicks I tried to go back to 482ae2d, but apparently there is a migration version change in between, and could no longer go back. (which means phew...thank God it works now, as I have a new base).

No. Still does not work on 2.16.2 (with 2.3.0-develop server).


Strange marker behavior when scrolling with soft wrap enabled

Just noticed the following behavior on latest master (1.4.0-dev-2696f49) with spellcheck lines and find-and-replace selections:

Notice how the markers somehow translate from line to line as the top edge of the editor scrolls over the line with the marker (I'm only guessing this is marker related since it can be reproduced with both spellcheck lines and find-and-replace selections, I haven't investigated this further).
I was able to reproduce this with a bunch of lorem ipsum snippets wrapped to 80 chars (notice the top line which has double spellcheck lines):

cc @nathansobo @as-cii for more :eyes: Also, not sure if this is related, but https://github.com/atom/atom/issues/8007 has similar keywords as this.

Taking a look now. @izuzak have you been able to reproduce on beta at all?

@nathansobo No reproduction on beta for me, only on master.

No reproduction on beta for me, only on master.

@abe Hmm, strange -- I've been able to reproduce this on beta as well, but not on stable. @nathansobo -- the easiest way to reproduce this is to open a new tab, enable soft wrapping, paste a bunch of lorem ipsum text (so that you can scroll), and then slowly scroll down. You'll see how the spellcheck lines duplicate on the line which the top edge of the screen is over.

@izuzak Oh s***, my beta branch was out of date, rebuilding now.

Right, I can reproduce that on beta as well. My bad...

@nathansobo I can still reproduce this on latest master. Are you sure this was fixed by 8da44c4?

@izuzak Ugh, it got lost in the merge of beta into master. Fixed it again, hopefully for good this time.

Yep, looks good now -- double :zap: :zap: for fixing this two times, @nathansobo!


Breaks Night Vision Effect

As the title says, the latest version of this mod breaks the night vision effect, regardless of whether or not you have night vision in-game dark spots will still be dark. I'd imagine this has something to do with avoiding updating the lightmaps

will fix in 1.6.0

fixed, but it will be a few hours or days till it is released


CHANGELOG.md: Undo version <IP_ADDRESS>

Pull Request (PR) description
Removes version <IP_ADDRESS> from CHANGELOG.md
This Pull Request (PR) fixes the following issues
None
Task list

[ ] Added an entry under the Unreleased section of the change log in
CHANGELOG.md. Entry should say what was changed, and how that affects
users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as
appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See
DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See
DSC Resource Testing Guidelines.
[ ] New/changed code adheres to
DSC Resource Style Guidelines
and
Best Practices.

This change is

Codecov Report

Merging #524 into dev will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##            dev   #524   +/-   ##
===================================
  Coverage    74%    74%
===================================
  Files        27     27
  Lines      4031   4031
  Branches      4      4
===================================
  Hits       3005   3005
  Misses     1022   1022
  Partials      4      4

Done. Thanks @mhendric


Add IBM provider metadata to secret labels and/or annotations

Problem Statement
We would like the ESO to add a certain metadata while creating or updating a Kubernetes secret to be reflected in its labels or annotations. The metadata could be any of the fields above that are supported and returned in the response by the IBM Cloud provider.
For a secret created in IBM Cloud Secrets Manager, we have the metadata that can be associated with a secret, like secret-group-id, secret-type, secret-id etc for cross-referencing. This will help in below:

Make the secret self describing enabling it to be cross-referenced easier for debugging in case something goes wrong in the cluster
Update timestamp in the secret for easier debugging and knowing if a secret update caused pod errors
And other details which could help in the long run.
We also have integrations that are based on the metadata available in the Kubernetes secret labels.

Related Issue
Fixes (https://github.com/external-secrets/external-secrets/issues/2369)
Proposed Changes
The existing templating concept provided by ESO can be leveraged to supplement the labels, where the metadata can be added, along with the data fields. Refer below yaml file.
In addition to this, a new field includeSecretMetadata is introduced under spec.dataFrom.extract.
Below are the changes from different perspectives:
i. User:
In order for the user to opt-in to adding metadata to secret, a new optional field is introduced under spec.dataFrom.extract.includeSecretMetadata. This is an optional, whose value defaults to false. The required metadata can then be specified under template.metadata.labels or template.metadata.annotations, same as existing. The required secret data can be specified under template.data. While the secret is being reconciled, it will have the secret data along with the required labels.
In the below example, secret_id and updated_at are the metadata of a secret in the Secrets Manager.
apiVersion: external-secrets.io/v1beta1
kind: ExternalSecret
metadata:
  name: sample-external-secret
  namespace: external-secrets
spec:
  dataFrom:
  - extract:
      key: username_password/test-secret
      includeSecretMetadata: true           # additional optional parameter, defaults to false
    secretKey: secret
  secretStoreRef:
    kind: SecretStore
    name: sm-secret-store
  target:
    name: sample-secret
    template:
      engineVersion: v2
      data:
        secret: "{{ .password }}"
      metadata:
        labels:
          secret_id: "{{ .id }}"
          updated_at: "{{ .updated_at }}"

ii. Provider:
The GetSecretMap() in provider.go under ibm package returns a map[string][]byte holding the value of map of secret values fetched.
If includeSecretMetadata is set to true, metadata is also included along with the secret value as map in its return value.
iii. Controller applying template:
While applying the template, the controller will de-serialize the content of this dataMap and extract the required label/annotation keys to build the label/annotation key-values in the secret along with the secret data.
Below is the example of the secret after reconciliation:
apiVersion: v1
# ESO adds data
data:
  secret: OHE0MFV5MGhQb2FmRjZTOGVva3dPQjRMeVZXeXpWSDlrSWgyR1BiVDZTMyc=
# End: ESO adds data

immutable: false
kind: Secret
metadata:
  annotations:
    reconcile.external-secrets.io/data-hash: 02217008d13ed228e75cf6d26fe74324
  creationTimestamp: "2023-05-04T08:41:24Z"

# ESO adds labels
  labels:
    secret_id: 1234
    update_at: 2023-05-04T08:57:19Z
# End: ESO adds labels

  name: sample-secret
  namespace: external-secrets
  ownerReferences:
  - apiVersion: external-secrets.io/v1beta1
    blockOwnerDeletion: true
    controller: true
    kind: ExternalSecret
    name: sample-external-secret
    uid: c2a018e7-1ac3-421b-bd3b-d9497204f843
  resourceVersion: "1803567"
  uid: f5dff604-611b-4d41-9d65-b860c61a0b8d
type: Opaque

Though above example refers to containing the metadata in labels, the same can be extended to annotations as per user requirement.
Note:
The code changes are limited to ibm provider package and apis/externalsecrets/v1beta1/externalsecret_types.go

[x] I have read the contribution guidelines
[x] All commits are signed with git commit --signoff
[x] My changes have reasonable test coverage
[x] All tests pass with make test
[x] I ensured my PR is ready for review with make reviewable

Hi @vsroy ! How does the proposed flag differs from MetadataPolicy? IMO they are the same , and I rather not have an additional field that does slightly the same thing :/

@gusfcarvalho Thanks for looking into this. I have removed the dependency from the initially proposed flag and set the functionality to be dependent on the existing MetadataPolicy flag being set to Fetch or None. Please review.

@moolen Can I have an approval on this please.

code-wise LGTM. Though linting fails, could you please fix that? :pray:
I hope you did an integration test against a live API. Because we don't have that covered yet and i don't have an IBM account.

@moolen Thank you for reviewing the code. The linting issues have been fixed.
Also the changes have been tested against a live API in a k8s cluster.


Documentation on coordinate system when using RawImage and dart:ui.

I'm using the 0->1 system as per Texture2D in GL fragment shaders. The documentation is not clear on the drawRawAtlas call in this regard. The color argument is also not explained. My assumption is an over color mask, and why paint when a blendmode is specified?

cc @liyuqian


ODP presentations load without background

ODP presentations show plain. No background and text colors are gone.

not like urs . ODP presentations show nothing , just download source file XX.odt , hope to get answer!

Are you using the proper syntax as described here? In that case you should be seeing a grey screen, and downloading should not be possible.
http://viewerjs.org/instructions/

@socialweb: in what (version of what) application was the presentation created? If the images used are wrapped in some proprietary wrapper, the browser (or in fact other applications) might not understand. A quick fix with problematic documents is to convert to PDF, and use that instead - viewerJS understands both ;)
You can also check your document with http://validator.opendocumentformat.org.

/ViewerJS/#../demo/11.odt
syntax is right
perform well when src=PDF file , but fail to present ODP file =. =


Update from source, release prep

As discussed via mail. I'll add more things here for the release preparations. Anything else I should check/can help with?

Looks good -- the tests seem to be failing because they're based on py3.6 which is deprecated. Can you update these thanks?

wonderful, thanks @chrzyki !