Pageserver didn't shutdown on signal

Backtrace:
(gdb) thread apply all bt

Thread 2 (Thread 0x7f30ba50a700 (LWP 5877)):
#0  0x00007f30be991ddf in accept4 (fd=5, addr=..., addr_len=0x7f30ba5098ec, flags=524288) at ../sysdeps/unix/sysv/linux/accept4.c:32
#1  0x000055d66eba3439 in std::sys::unix::net::Socket::accept::{{closure}} () at library/std/src/sys/unix/net.rs:225
#2  std::sys::unix::cvt_r () at library/std/src/sys/unix/mod.rs:212
#3  std::sys::unix::net::Socket::accept () at library/std/src/sys/unix/net.rs:225
#4  std::sys_common::net::TcpListener::accept () at library/std/src/sys_common/net.rs:407
#5  std::net::tcp::TcpListener::accept () at library/std/src/net/tcp.rs:810
#6  0x000055d66e4ea563 in pageserver::page_service::thread_main (conf=0x55d670441aa0, auth=..., listener=..., auth_type=zenith_utils::postgres_backend::AuthType::Trust)
    at pageserver/src/page_service.rs:194
#7  0x000055d66e3925db in pageserver::start_pageserver::{{closure}} () at pageserver/src/bin/pageserver.rs:603
#8  std::sys_common::backtrace::__rust_begin_short_backtrace (f=...) at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/std/src/sys_common/backtrace.rs:125
#9  0x000055d66e392fd8 in std::thread::Builder::spawn_unchecked::{{closure}}::{{closure}} () at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/std/src/thread/mod.rs:481
#10 <core::panic::unwind_safe::AssertUnwindSafe<F> as core::ops::function::FnOnce<()>>::call_once (self=..., _args=<optimized out>)
    at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/core/src/panic/unwind_safe.rs:271
#11 std::panicking::try::do_call (data=<optimized out>) at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/std/src/panicking.rs:403
#12 std::panicking::try (f=...) at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/std/src/panicking.rs:367
#13 std::panic::catch_unwind (f=...) at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/std/src/panic.rs:129
#14 std::thread::Builder::spawn_unchecked::{{closure}} () at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/std/src/thread/mod.rs:480
#15 core::ops::function::FnOnce::call_once{{vtable-shim}} () at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/core/src/ops/function.rs:227
#16 0x000055d66ebb1633 in <alloc::boxed::Box<F,A> as core::ops::function::FnOnce<Args>>::call_once () at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/alloc/src/boxed.rs:1636
#17 <alloc::boxed::Box<F,A> as core::ops::function::FnOnce<Args>>::call_once () at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/alloc/src/boxed.rs:1636
#18 std::sys::unix::thread::Thread::new::thread_start () at library/std/src/sys/unix/thread.rs:106
#19 0x00007f30bebe7fa3 in start_thread (arg=<optimized out>) at pthread_create.c:486
#20 0x00007f30be9904cf in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95

Thread 1 (Thread 0x7f30be892a40 (LWP 5871)):
#0  0x00007f30bebe9495 in __GI___pthread_timedjoin_ex (threadid=139847261005568, thread_return=0x0, abstime=0x0, block=<optimized out>) at pthread_join_common.c:89
#1  0x000055d66ebb170d in std::sys::unix::thread::Thread::join () at library/std/src/sys/unix/thread.rs:242
#2  0x000055d66e3927d2 in std::thread::JoinInner<T>::join (self=0x7fff03273ab8) at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/std/src/thread/mod.rs:1271
#3  std::thread::JoinHandle<T>::join (self=...) at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/std/src/thread/mod.rs:1403
#4  0x000055d66e39ea0c in pageserver::start_pageserver::{{closure}} (signal=<optimized out>) at pageserver/src/bin/pageserver.rs:627
#5  zenith_utils::signals::ShutdownSignals::handle (self=..., handler=...) at /zenith/zenith_utils/src/signals.rs:54
#6  0x000055d66e3af553 in pageserver::start_pageserver (conf=0x55d670441aa0) at pageserver/src/bin/pageserver.rs:607
#7  pageserver::main () at pageserver/src/bin/pageserver.rs:511
(gdb)

According to the dump, it's stuck on

https://github.com/zenithdb/zenith/pull/787 attempts to address this and has a few comments about feasible alternatives.

I've chased my ideas from https://github.com/zenithdb/zenith/pull/787#issuecomment-993355999 and checked the other crate. It turned to be a nice rust api around low level API that we already use in our repo:
https://github.com/zenithdb/zenith/blob/b5324707928502d7a99407a71f587d8141e5b81b/zenith_utils/src/tcp_listener.rs#L13
First, I've tried setting timeouts (both read and write), it was possible via "low-level" way and the other lib way, but neither helped to fix the "sigint single pageserver" blocking issue.
There's an interesting method in the other lib, that I've not found among the setsockopt options:
https://docs.rs/socket2/0.4.2/socket2/struct.Socket.html#method.set_nonblocking
If I use that, when I sigint the pageserver, it stops immediately, but it panics:
thread 'main' panicked at 'thread exited with an error: Resource temporarily unavailable (os error 35)', pageserver/src/bin/pageserver.rs:630:22
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I'll check the panic more in detail, but feels like it could work for us, if we could suppress it?
I've pushed a branch to see how our tests react on the nonblocking sockets: https://github.com/zenithdb/zenith/compare/kb/timed-accept?expand=1

I've also decided to try out a simple experiment with tokio: I assumed, that we'll be stuck on some transition case where some of the listeners would be needed to be async, and some (such as pg connection) will stay sync for a while.
So, I've composed a small PR that takes zenith_utils::tcp_listener::bind and uses tokio::net::TcpListener to bind with into_std() to convert it back to std::net::TcpListener after binding.
https://github.com/zenithdb/zenith/compare/kb/async-accept?expand=1
That replicates similar behavior to my previous comment:

"sigint single pageserver" test interrupts normally now
tests are broken with Connection refused (os error 61)

So, it feels that either we have to move everything to async at once, or we'll have issues with whatever std::net primitives we get out of tokio.

Fixed by https://github.com/zenithdb/zenith/commit/dab30c27b665ec8c8368bed402a9ea3aded7ce4d.


Added conda channel command

The command for creating channels was added as a defined type.
Documented channel and updated readme
Added a new parameter for env (exec_timeout) as I was experiencing exec timeouts with the default 300 seconds of the exec puppet resource.

Those look great!  Thanks for your changes.


Add README to otel github org

I thought it would be nice to have a organization README that is presented on github.com/open-telemetry (as outlined here: https://docs.github.com/en/organizations/collaborating-with-groups-in-organizations/customizing-your-organizations-profile)
This is inspired by a few other READMEs that already exist, especially https://github.com/nodejs

I missed a requirement that this file needs to be put in the "profile" folder: #7


WPF using Python Machine Learning Libraries

Hi, I was just wondering if I am in the right place?
Currently, I am working on a project to create a GUI by using WPF, we want to utilize the majority of Python Machine Leaning Libraries (Scikit-Learn, TensorFlow, Pandas, Numpy). So my question is: can I take advantage from LEAN?

Hi @redcican,
Actually, this question should be sent over to QuantConnect Support.
We have been working on a GUI using Electron that it will emulate QuantConnect Algorithm Lab.
It is not clear how the WPF app will work. Do you want to use it to connect to QuantConnect.com or will it run Lean locally?

Hi, actually it should run locally. Since I have been looking for an appropriate way to solve my question for a long time. I thought that LEAN may help me.

Please contact<EMAIL_ADDRESS>with more details. I don't know how we can help you with the information you gave us so far.


Update event-hubs-for-kafka-ecosystem-overview.md

for jaas example, semicolon works fine, but not for the OAUTHBEARER, so suggested to fix as I've filed also a PR to the samples repo.

@ernani : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Test dev action on pull request branch

This is a pull request to test github action

Actions on pull request working


Adds command capitalisation.

Work on #618.
Changes proposed in this PR:

use snake case in commands' names

I updated most commands to snake case and superseding #506. Much easier now that we have tests.
I think the only two pending are nftinfo and getrate. Should I convert those too? And, if I do convert nftinfo to nft_info, should I also change the csv filenames?


[Feature][Manager][Agent][DataProxy] Support full link tracking to improve the observability of the project

Description
Inlong lacks observability, which is not conducive to link tracking and error location.
InLong Component
InLong Manager, InLong Agent, InLong DataProxy
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@ZhaoNiuniu Please add more design documents to this issue description.


Export Options type from the entry point

This is a speculative pull request addressing https://github.com/snabbdom/snabbdom/pull/975#issuecomment-997180772, so please feel free to close this if you don't need this.

Exporting the Options type, so that other libraries in the ecosystem (e.g. @cycle/dom) can depend on it.

Thanks!


Fixed Issue Java Medium

Added the add, subtract, multiply, divide methods to the Calculator class

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Fixes #351, doesn't mark path as 'FILE' unless it exists

I didn't add an additional test for this because:

npm run test-regen is failing locally
Tests are failing in CI

So, it sounds like there's existing test issues

This repository hasn't been updated in almost a year, and there's a lot of PRs in purgatory 😢
However, if you're reviewing this PR as a maintainer, let me know! I just don't want to be spending time polishing a PR if it's for a dead project :smile:


Can't create a Language class

Why is the LanguageFactory class marked as internal? Documentation states that the Language class is instantiated via LanguageFactory, but I can't access that class.
I am using 1.5.0 via Nuget.

Sorry. I made the internals visible to the client project (Nuve.Gui) for some reason. That's why my code compiled and I couldn't see that. Now I remove the [assembly: InternalsVisibleTo("Nuve.Gui")] not to repeat the same mistake. I also pushed a new version 1.5.1 to Nuget.

Thanks.

The commit that fixes the issue: https://github.com/hrzafer/nuve/commit/edb51baed854ab1cac0647cad4c5f594ce0fd4dc


Use the new memo function to Memoize functions where it is safe to do so, such as Colour.make()

At a glance, this should be doable for

The Colour functions
The Math functions
Number toString function
String functions

Hey I don't think we should do this for a couple of reasons: I'd like to keep the different 'libraries' independent of each other, so that they don't use each other.
I think memo would speed up Colour.make, but I want to rewrite that anyway to be a much smaller + simpler function: #30
I don't think that memo would speed up those other examples you mentioned, but even if it did, I wouldn't want to impose it onto the user
I hope that kinda makes sense? :)
I know it's hard when I haven't written up goals/guidelines or anything so sorry about that. Yesterday I started preparing more of the redesign on the simplified-build branch, and I'll finish it off, and set out some goals+stuff when I'm back from holiday in a week or too :D


Trivy: Install Python packages

For the Trivy to evaluate Python, it needs to find a site-packages directory.

Did a bit of research - it looks like if the variable VIRTUAL_ENV is set to the path of the virtual environment, it would look for site packages there (see https://github.com/aquasecurity/trivy/blob/672e886aed152ae0f09a16941706746f3053ca94/pkg/fanal/analyzer/language/python/pip/pip.go#L135).
However, trying to set that did not seem to work.
There is also the problem we'd have to set this and re-run trivy every time.
Perhaps a better approach would be to run a different tool to generate an SBOM then use the SBOM analyzer in Trivy.

Note solution proposed in https://github.com/aquasecurity/trivy/discussions/7734#discussioncomment-10957841

The proposed solution seems to work, but we'll need to change the invocation of Trivy to be executed from the shell to enable the $VIRTUAL_ENV to be picked up.

This isn't really needed since we're using Parlay


[release-1.9] Upgrade to latest dependencies

bumping pkg -dprotaso
/cc we don't use Prow
/assign we don't use Prow
Produced by: knative-sandbox/knobots/actions/update-deps

Codecov Report

Merging #514 (2426d77) into release-1.9 (990f67b) will increase coverage by 0.14%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           release-1.9     #514      +/-   ##
===============================================
+ Coverage        66.08%   66.23%   +0.14%
===============================================
  Files               38       38
  Lines             2079     2079
===============================================
+ Hits              1374     1377       +3
+ Misses             660      657       -3
  Partials            45       45

Impacted Files
Coverage Δ

pkg/vsphere/adapter.go
64.73% <0.00%> (+1.57%)
:arrow_up:

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


isValid hangs regardless timeout

Driver version
Driver 7.2.2-jre8
SQL Server version
SQL 2016 Enterprise with multi-subnet HA
Client Operating System
Centos 7.6
JAVA/JVM version
Oracle JDK 1.8.192
Table schema
N/A
Problem description

Expected behaviour:
Invoking isValid operation on connection with the timeout parameter should not take longer than the timeout configured

Actual behaviour:
If DB failover happens, sometimes we are getting TCP socket read hanging forever in JDBC Driver (see stacktrace below).
This causes our connection pool to completely hang, as write lock acquired on connection pool object is never released during testing if connections in pool are healthy.
Currently as per implementation isValid timeout is set as query timeout, but if I understand the code correctly, this is server-side timeout, while our problem happens on the client side.
Therefore the current implementation of isValid timeout is not handling correctly client-side problems.

Error message/stack trace:
N/A as everything hangs

Any other details that can be helpful:
I believe this is similar problem to that one: https://github.com/microsoft/mssql-jdbc/issues/764
Except we are getting it almost every time.

I think the solution might be to temporarily change TCP Socket timeout from the one set by connection parameter, to the one set in isValid parameter to make sure TCP timeout equals to isValid timeout, then after successful validation change TCP timeout to original value.
JDBC trace logs
Thread dump:

java.lang.Thread.State: RUNNABLE
at java.net.SocketInputStream.socketRead0(Native Method)
at java.net.SocketInputStream.socketRead(SocketInputStream.java:116)
at java.net.SocketInputStream.read(SocketInputStream.java:171)
at java.net.SocketInputStream.read(SocketInputStream.java:141)
at com.microsoft.sqlserver.jdbc.TDSChannel.read(IOBuffer.java:2023)
at com.microsoft.sqlserver.jdbc.TDSReader.readPacket(IOBuffer.java:6418)
- locked <0x00000007266a5fd0> (a com.microsoft.sqlserver.jdbc.TDSReader)
at com.microsoft.sqlserver.jdbc.TDSCommand.startResponse(IOBuffer.java:7579)
at com.microsoft.sqlserver.jdbc.SQLServerStatement.doExecuteStatement(SQLServerStatement.java:866)
at com.microsoft.sqlserver.jdbc.SQLServerStatement$StmtExecCmd.doExecute(SQLServerStatement.java:768)
at com.microsoft.sqlserver.jdbc.TDSCommand.execute(IOBuffer.java:7194)
at com.microsoft.sqlserver.jdbc.SQLServerConnection.executeCommand(SQLServerConnection.java:2935)
- locked <0x00000007266a62b0> (a java.lang.Object)
at com.microsoft.sqlserver.jdbc.SQLServerStatement.executeCommand(SQLServerStatement.java:248)
at com.microsoft.sqlserver.jdbc.SQLServerStatement.executeStatement(SQLServerStatement.java:223)
at com.microsoft.sqlserver.jdbc.SQLServerStatement.executeQueryInternal(SQLServerStatement.java:700)
at com.microsoft.sqlserver.jdbc.SQLServerConnection.isValid(SQLServerConnection.java:5591)
at com.microsoft.sqlserver.jdbc.SQLServerConnectionPoolProxy.isValid(SQLServerConnectionPoolProxy.java:480)
at org.apache.aries.transaction.jdbc.internal.ValidatingDelegatingManagedConnectionFactory.isValidConnection(ValidatingDelegatingManagedConnectionFactory.java:59)
at org.apache.aries.transaction.jdbc.internal.ValidatingDelegatingManagedConnectionFactory.getInvalidConnections(ValidatingDelegatingManagedConnectionFactory.java:84)
at org.apache.aries.transaction.jdbc.internal.ValidatingGenericConnectionManager$ValidatingTask.run(ValidatingGenericConnectionManager.java:155)
at java.util.TimerThread.mainLoop(Timer.java:555)
at java.util.TimerThread.run(Timer.java:505)

Reproduction code

Establish approx 100 connections to DB
Failover multisubnet HA installation.
During failover, keep invoking isValid on all connections

Hi @PiotrKlimczak ,
Thanks for creating the issue, we will investigate and get back to you soon.

Just to update our findings.
We have implemented the fix ourselves in a copy of your 7.2.2 tag and it works correctly now.
Our implementation of is valid:
public boolean isValid(int timeout) throws SQLException {
    boolean isValid = false;

    Boolean disableIsValidTimeoutFix = Boolean.valueOf(System.getProperty("mssql.jdbc.isValid.timeout.fix.disable", "false"));
    int defaultTimeout = Integer.valueOf(System.getProperty("mssql.jdbc.isValid.default.timeout", "500")); //Why would it take any longer?

    int originalNetworkTimeout = 0;

    if (!disableIsValidTimeoutFix) {
        originalNetworkTimeout = getNetworkTimeout();
    }

    loggerExternal.entering(getClassNameLogging(), "isValid", timeout);

    // Throw an exception if the timeout is invalid
    if (timeout < 0) {
        MessageFormat form = new MessageFormat(SQLServerException.getErrString("R_invalidQueryTimeOutValue"));
        Object[] msgArgs = {timeout};
        SQLServerException.makeFromDriverError(this, this, form.format(msgArgs), null, true);
    } else if (timeout == 0) {
        timeout = defaultTimeout;
    }

    // Return false if the connection is closed
    if (isSessionUnAvailable())
        return false;

    try {
        SQLServerStatement stmt = new SQLServerStatement(this, ResultSet.TYPE_FORWARD_ONLY,
                ResultSet.CONCUR_READ_ONLY, SQLServerStatementColumnEncryptionSetting.UseConnectionSetting);

        // If asked, limit the time to wait for the query to complete.
        if (0 != timeout)
            stmt.setQueryTimeout(timeout);

        if (!disableIsValidTimeoutFix) {
            setNetworkTimeout(null, timeout);
        }

        // Try to execute the query. If this succeeds, then the connection is valid.
        // If it fails (throws an exception), then the connection is not valid.
        // If a timeout was provided, execution throws an "query timed out" exception
        // if the query fails to execute in that time.
        stmt.executeQueryInternal("SELECT 1");
        stmt.close();
        isValid = true;
    } catch (SQLException e) {
        // Do not propagate SQLExceptions from query execution or statement closure.
        // The connection is considered to be invalid if the statement fails to close,
        // even though query execution succeeded.
        connectionlogger.fine(toString() + " Exception checking connection validity: " + e.getMessage());
    }

    try {
        if (!disableIsValidTimeoutFix) {
            setNetworkTimeout(null, originalNetworkTimeout);
        }
    } catch (Exception e) {
        // no-op
    }

    loggerExternal.exiting(getClassNameLogging(), "isValid", isValid);
    return isValid;
}

Hi @PiotrKlimczak ,
Thanks for the update.
Would you also try your application with 7.0.0 version of the driver and let me know if the issue still occurs? We made changes to query timeouts in 7.2.2, I am wondering if is related.

I have a similar problem where the JDBC connection hangs after emitting a SELECT 1 query, which I can see on the SQL Server Profiler.
I'm using Java 11. I first found the error while using version 7.2.2.jre11 just like @PiotrKlimczak (except for the different Java version).
I have changed it to version 7.0.0.jre8 and the issue is still present. I will try the suggested code change.

@ulvii @PiotrKlimczak Was this ever fixed?  I am experiencing the exact same issue above on version 8.4.1 of the MSSQL JDBC driver when connecting to a Azure managed DB.  Intermittent network dropouts will cause the isValid checks in the connection pool manager to hang and it brings down the entire server due to open locks in the connection pool manager.

Hi @bdw429s ,
Thanks for reporting the details and confirming that the issue happens with newer versions of the driver.  Is using socketTimeout connection property an option for you? This can prevent the "hang" and throw an exception immediately if the connection is invalid.

Hi @ulvii I can't directly control the socket timeout in use, and I'm currently in discussion with the developers of the app server I use to confirm what the socket time out is that's being used.
https://luceeserver.atlassian.net/browse/LDEV-3124

hi @ionutioio, sorry but there is no timeline for this fix at the moment.


Colab - Cuda OOM

After trying Depth-to-Image model a 4 or 5 times, I got this error.
CUDA out of memory. Tried to allocate 6.78 GiB (GPU 0; 14.76 GiB total capacity; 6.31 GiB already allocated; 5.71 GiB free; 7.80 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation. See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

I'm not sure if something wrong happens with the D2I model. Just letting you know!

This was on a Free Colab

Thanks for the info, I'll look into that

I'll close it. It looks like it has to do with image size. Whenever the image size crosses 1000px, this happens.


Support customizing existing themes

Current tailwindcss-extra works well when using builtin themes or creating custom themes. However, it does not support modifying builtin themes using the instructions in How to customize an existing theme? because that source path (daisyui/src/theming/themes) is not included in the bundle and a MODULE_NOT_FOUND is thrown. Would it be possible to include it in the bundle?

Hi! I got it working. Give me a day or two, and I'll push the change and will create a new release.

Awesome, that's great! looking forward to it :)

@elfjes, please check if the latest binary resolves the issue for you.

@dobicinaitis perfect, thanks! 😁


Rebranding fmp

Rebranding suggestion for fmp:

Two plugins for OpenShift & Kubernetes
Using a neutral repository name (tbd, idea welcome !)
Use a “openshift-maven-plugin” (or “okd-maven-plugin” ?) with a “okd:” prefix
“kubernetes-maven-plugin” with “k8s:” prefix
Let kubernetes-maven-plugin create only a kubernetes.yaml but openshift-maven-plugin include kubernetes.yml and openshift.yml
Skip all fabric8 naming

Okay, so we're considering to shift to this repository : https://github.com/jshiftio
So in order to begin movement/rebranding, I see these two approaches:

Move fabric8 maven plugin repository to jshift (transfer ownership) and then refactor in order to abstract openshift-maven-plugin, jshift-kit etc out of it afterwards.
Maintain fabric8-maven-plugin here in this org, and start these projects from scratch(kubernetes-maven-plugin, openshift-maven-plugin, jshift-kit) and give a link in readme that fmp is gonna get depracated soon and use these new plugins. This would require having a new clean start :-)

@rhuss @lordofthejars @dev-gaur Please share your thoughts?

I opt for the second options as this allows for a clean and fresh start. Moving bits and pieces individually might be more work, but also an occasion to get rid of legacy.
Also it makes it more evident that this is a clear cut.


Update PLATFORM.md

From @satlxq on October 29, 2014 9:9
修改了几个错别字
Copied from original issue: NetEaseWD/NEJ/pull/6

thx


File numbers returned by partitioning tool

The partitioning tool implemented in #101 behaves as follows:
from roms_tools.utils import partition
file_numbers, partitioned_datasets = partition(ds, nx=1, ny=5)

returns the following file_numbers
[0, 5, 10, 15, 20]

Is this intentional @TomNicholas? Don't we want the file_numbers to be
[0, 1, 2, 3, 4]

in this case? My understanding was that the file_numbers help with naming the files to write via xarray.save_mfdataset(), and ROMS expects  .0.nc, .1.nc, .2.nc, ...

That's probably a bug. Looks like I forgot to explicitly check the returned file numbers.

Okay, I'm working on a PR that integrates partition with the .save() methods. I will fix the bug as part of that PR, and ask you to review.

Closed via #109.


feat: inject remark-frontmatter if missing

Depends on https://github.com/brillout/vite-plugin-mdx/pull/11

In configResolved hook, check if remark-frontmatter is installed by project

If it is, do nothing.

Otherwise, find a plugin named vite-plugin-mdx that has a mdxOptions property

If no plugin is found, warn the user to upgrade or install vite-plugin-mdx

Otherwise, inject remark-frontmatter into the mdxOptions.remarkPlugins array

@csr632<EMAIL_ADDRESS>is published. Ready to merge this?


fix break in node 8

ok node 8 intrudes a new method on writable streams that breaks this module, embaresing as I was the author of that change in node and this module, woops, this fixes it

Ready for release,  can't do it myself atm


Include an external configuration file when building a war

The UAA documentation states
There is a default file in the WAR that you should not touch. Overrides and additions can come from an external location, the most convenient way to specify that is through an environment variable (or system property in the JVM) named CLOUD_FOUNDRY_CONFIG_PATH.

Turns out that CLOUD_FOUNDRY_CONFIG_PATH, UAA_CONFIG_URL, UAA_CONFIG_PATH etc are all runtime configurations, i.e. if you want to build a war file to deploy the UAA standalone to cloudfoundry, there is no way to include an external configuration file.
The current workaround is to modify the uaa.yml before building the war, explicitly warned against in the current documentation.
It would be nice to be able to include an external configuration file that replaces the included uaa.yml file when building the war.

We have addressed by being able to pass in complete Yaml configuration through the CF application manifest
https://github.com/cloudfoundry/uaa/commit/f55f2b4d55c0b14dc76a4aa4d096595203770f37#diff-04c6e90faac2675aa89e2176d2eec7d8R207

Work around for now is to either set individual environment variables or modify the war file to add in your own uaa.yml file.


pocketbase-vue-starter Template

Hey there,
Wanted to throw my repo in the hat.
I have created a pocketbase-vue-starter template that uses PocketBase as a framework with Go migrations.
The default first time admin setup has redirected to use the vue-client admin setup page.
This repo can be used with PocketBase running somewhere else or by how it's setup in the repo. I have set it up for PocketBase to host the static files of the vue-client, and this can be run on a single Docker container for ease (hence the redirection options for PocketBase)
I created this last night, and have been updating today. There's still more I would like to add to it, but here's the repo: https://github.com/MRSessions/pocketbase-vue-starter
Title: pocketbase-vue-starter
Description: A starter template for using Vue/Vuetify with Pocketbase as a backend.
Thank you!

@MRSessions sounds great! Would you please create a PR to add your contribution? Thank you so much!

Absolutely. I'll try and get one up this afternoon/evening.

Hey there,
Wanted to throw my repo in the hat.
I have created a pocketbase-vue-starter template that uses PocketBase as a framework with Go migrations.
The default first time admin setup has redirected to use the vue-client admin setup page.
This repo can be used with PocketBase running somewhere else or by how it's setup in the repo. I have set it up for PocketBase to host the static files of the vue-client, and this can be run on a single Docker container for ease (hence the redirection options for PocketBase)
I created this last night, and have been updating today. There's still more I would like to add to it, but here's the repo: https://github.com/MRSessions/pocketbase-vue-starter
Title: pocketbase-vue-starter Description: A starter template for using Vue/Vuetify with Pocketbase as a backend.
Thank you!

Nuxt3 starter pack on it's way 😃? or should I do it? 🤨


Move template files from docs/ to root

Ref: https://github.com/kubernetes/steering/issues/28 and https://github.com/kubernetes/community/issues/3053
The template files need to be at the root of the repo. The content is as per https://github.com/kubernetes/kubernetes-template-project.
/assign @andrewtandoc @bigkraig @M00nF1sh

@andrewtandoc you are listed as an approver for this repo. You can get lgtm + approve powers if you apply for a kubernetes-sigs member. :)
Instructions for it are here:

https://github.com/kubernetes/community/blob/master/community-membership.md#member
https://github.com/kubernetes/org/issues/new?template=membership.md

Also, I'm not sure why the lint test fails here. Any suggestions?

/lgtm

i'll fix the lint

/re-test

/retest

@M00nF1sh Thanks for fixing the lint :)


Add outstanding MSK_CLASSI_B00 file from processing baseline 04.00

These changes are required to enable the fetched products to support BioPhysical processing in SNAP.

I agree it makes sense to add this constant which can then be used by users. Otherwise the current code for creating the .SAFE structure already includes qi/MSK_CLASSI_B00, it is just generated on-the-fly.
Merging this PR to develop and it will be released in the next package version 3.6.0. Thanks for the contribution!


Update DataGridCellInfo.cs

Proposed solution for #3297. Make DataGridCellInfo struct independent from row virtualization processes.

Since this is in your repro I don't really mind, but I do want to point out that what you propose is a breaking change in public API and deviates from documented behavior.
Also note that apart from other issues, your proposed solution will return equality for an array of the same items while the current implementation does not.

You are right. It's not good solution. I'm closing request.


how can I set the date picker format?

Hi,
is it possible to set a custom date format in the date/time picker? I think all the pickers available have an option, but I'm not sure how to set it when using react querybuilder
thanks.

Hi, @jeacott1. Are you looking to modify the date format for the ValueEditor component from one of the compatibility packages, or the default ValueEditor?
The compatibility components have their own requirements and limitations, but if you're just using the default components with inputType: "date"/"time"/"datetime-local" on a field, that format is determined by the HTML5 spec. (You can see the actual values that are stored in the rule value property for the various value editor configurations on this page.)
You'd need to set up a custom value editor to use a different format. This page has an example that uses react-datepicker which can accept custom date formats.

@jakeboone02 thank you! I'm just looking to handle different input formats for different regions so I think the default editor is plenty. thank you for the refs.


kfctl fails because directory not empty if run on kfctl apply

There is a bug in kfctl and currently if you do the following
mkdir ${KFAPP}
cd ${KFAPP}
curl -L -o kfctl_gcp_iap.yaml https://raw.githubusercontent.com/kubeflow/manifests/master/kfdef/kfctl_gcp_iap.yaml
yq w -i kfctl_gcp_iap.yaml spec.plugins[0].spec.project ${PROJECT}
yq w -i kfctl_gcp_iap.yaml spec.plugins[0].spec.zone ${ZONE}
yq w -i kfctl_gcp_iap.yaml metadata.name ${NAME}
kfctl apply all -V -f kfctl_gcp_iap.yaml

It will fail because the directory ${KFAPP} is non empty. Similarly if you try to rerun kfctl apply from ${KFAPP} it will fail because the directory isn't empty.
The problem is in the implementation of how kfctl gets the appDir. The desired semantics of
kfctl apply -V -f ${KFDEF}

are

If  the value of  ${KFDEF} is a local file path; then we should use the directory of that file as the KFAPP directory regardless of whether it is empty

If its not empty we assume its the output of a previous run of kfctl aply

If the value of ${KFDEF} is a remote URI then we use the current working directory and check that it is empty.

This should be fixed by kubeflow/kubeflow#4115

Workaround
To work around this put ${KFDEF} in a different directory then ${KFAPP} e.g.
mkdir ${KFAPP}
cd ${KFAPP}
curl -L -o /tmp/kfctl_gcp_iap.yaml https://raw.githubusercontent.com/kubeflow/manifests/master/kfdef/kfctl_gcp_iap.yaml
yq w -i /tmp/kfctl_gcp_iap.yaml spec.plugins[0].spec.project ${PROJECT}
yq w -i /tmp/kfctl_gcp_iap.yaml spec.plugins[0].spec.zone ${ZONE}
yq w -i /tmp/kfctl_gcp_iap.yaml metadata.name ${NAME}
kfctl apply all -V -f kfctl_gcp_iap.yaml

/assign @zhenghuiwang

@zhenghuiwang I think @richardsliu PR kubeflow/kubeflow#4115 fixed this. I think we just need to build and verify that.

@jlewi I tried to reproduce/verify the issue. I think https://github.com/kubeflow/kubeflow/pull/4115 fixes the dir checking problem mentioned. I think another problem is caused by some checks that spec.project and spec.zone shouldn't be empty.
I set the following (but not in the plugin[0].spec)
yq w -i kfctl_gcp_iap.yaml spec.project ${PROJECT}
yq w -i kfctl_gcp_iap.yaml spec.zone ${ZONE}

and find things worked. (Also I guess if you pickup my changes on setting default project and zone, it would work too)
One issue I find is some error logging missing (should be fixed in https://github.com/kubeflow/kubeflow/pull/4279). With that, it is clear to see why kfApp is nil.
Here is the error log with https://github.com/kubeflow/kubeflow/pull/4279 produced when I follow your exact listed steps:
$ kfctl apply all -V -f ~/kfctl-test/kfctl_gcp_iap.yaml

INFO[0000] Downloading /Users/zhenghui/kfctl-test/kfctl_gcp_iap.yaml to /var/folders/1k/m148sdcd2szf5j5zdnbzq0c4006c2s/T/289183164/tmp.yaml  filename="utils/k8utils.go:133"
INFO[0000] Downloading /Users/zhenghui/kfctl-test/kfctl_gcp_iap.yaml to /var/folders/1k/m148sdcd2szf5j5zdnbzq0c4006c2s/T/750196715/app.yaml  filename="v1alpha1/application_types.go:349"
WARN[0000] you need to set the environment variable `ZONE` to the GCP zone you want to use  filename="coordinator/coordinator.go:452"
INFO[0000] App directory /Users/zhenghui/kfctl-test already exists  filename="coordinator/coordinator.go:511"
INFO[0000] Writing KfDef to /Users/zhenghui/kfctl-test/app.yaml  filename="coordinator/coordinator.go:521"
INFO[0000] Writing stripped KfDef to /Users/zhenghui/kfctl-test/app.yaml  filename="v1alpha1/application_types.go:660"
INFO[0000] Downloading /Users/zhenghui/kfctl-test/app.yaml to /var/folders/1k/m148sdcd2szf5j5zdnbzq0c4006c2s/T/322867790/app.yaml  filename="v1alpha1/application_types.go:349"
INFO[0000] Writing stripped KfDef to /Users/zhenghui/kfctl-test/app.yaml  filename="v1alpha1/application_types.go:660"
INFO[0000] Creating directory /Users/zhenghui/kfctl-test/.cache  filename="v1alpha1/application_types.go:440"
INFO[0000] Fetching https://github.com/kubeflow/manifests/archive/master.tar.gz to /Users/zhenghui/kfctl-test/.cache/manifests  filename="v1alpha1/application_types.go:480"
INFO[0004] Fetch succeeded; LocalPath /Users/zhenghui/kfctl-test/.cache/manifests/manifests-master  filename="v1alpha1/application_types.go:506"
INFO[0004] /Users/zhenghui/kfctl-test/.cache/manifests exists; not resyncing   filename="v1alpha1/application_types.go:455"
Error: failed to build kfApp from URI /Users/zhenghui/kfctl-test/kfctl_gcp_iap.yaml: couldn't generate KfApp:  (kubeflow.error): Code 500 with message: coordinator Generate failed for gcp:  (kubeflow.error): Code 400 with message: GCP Zone is not set, plea
se set it in KFDef.

I think @richardsliu 's PR partially fixed this but we also need kubeflow/kubeflow#4275


Problem with ZXing (Xamarin cross-platform)

Hi everyone,
I have a Xamarin solution with Android, iOS and UWP projects. I am trying to use ZXing in order to read some barcodes but i found some problems.
This is the way I use it:
try
{
  this.scanner = new ZXing.Mobile.MobileBarcodeScanner();
  this.result = await scanner.Scan();
  this.text = this.result.Text;
}

catch (Exception ex)
{
  this.text = "No code";
}

This is what i've done for each platform:
Android:

Install ZXing.Net.Mobile version 2.1.47 and Zxing.Net.Mobile.Forms version 2.1.47 packages.
Edit AndroidManifest.xml by adding permissions for camera and flashlight:

<uses-permission android:name="android.permission.CAMERA" />
<uses-permission android:name="android.permission.FLASHLIGHT" />

Edit MainActivity.cs by adding the following lines:

protected override void OnCreate(Bundle bundle)
{
 .
 .
 .
  MobileBarcodeScanner.Initialize(Application);
 .
 .
 .
}

public override void OnRequestPermissionsResult(int requestCode, string[] permissions, Permission[] grantResults)
{
 global::ZXing.Net.Mobile.Forms.Android.PermissionsHandler.OnRequestPermissionsResult(requestCode, permissions, grantResults);
}

This works fine but, at some random? scanning, the screen gets black before the scanning starts, the application doesn't work any more and it has to be manually killed. As I kill the application manually, I don't have access to the camera any more until I reboot the device (I think that's because the camera it is never released by the application instance killed manually).
iOS:

Install ZXing.Net.Mobile version 2.1.47 and Zxing.Net.Mobile.Forms version 2.1.47 packages.

App ends before the scanning starts and I get:
Got a SIGABRT while executing native code. This usually indicates
a fatal error in the mono runtime or one of the native libraries
used by your application.
UWP:

Install ZXing.Net.Mobile version 2.1.47 and Zxing.Net.Mobile.Forms version 2.1.47 packages.
Add to Package.appxmanifest Capabilities Microphone (doesn't make sense but i've read in some places that it has to be enabled) and Webcam.

App ends just after the scanning and I get:
Access is denied. (Exception from HRESULT: 0x80070005 (E_ACCESSDENIED))
Extra info:
Xamarin.Forms version <IP_ADDRESS>
Am I missing something?
Thanks

I'm having the same issue on iOS. I have not tested on any other platforms. Is there any update on this?

I Have the same problem !! Is there any solution ??   pleaseeee!!!

Hi! I have the same issue, but only on iOs versions 10+
On 8.4.1 it works like a charm

I had a similar issue on Android. Although the app was never killed before the scanner page was closed the camera wasn't released properly at some point. Therefore after another start of the app the camera service couldn't be accessed.

Have you tried "ZXing.Net.Mobile for Forms"? It's easer.
Folow the instructions in this link:
https://components.xamarin.com/gettingstarted/zxing.net.mobile.forms

Have you tried "ZXing.Net.Mobile for Forms"? It's easer.
Folow the instructions in this link:
https://components.xamarin.com/gettingstarted/zxing.net.mobile.forms

Thanks for reporting this issue!  Unforunately it took me way too long to respond 😭.  Sorry, I apologize!  Recently the source code for this project was completely refactored to modernize it.  Many PR's were included in this effort, and many bugs were hopefully fixed.  Please try out the latest 3.x series of NuGet packages (currently in prerelease).  To try and make the project more maintainable in my spare time going forward, I've decided to close all existing issues to start with a clean slate.  If you're still experiencing this issue on the newest version, please open a new issue with as much detail as possible.  Thank you for your patience and understanding!  Happy scanning!


Update build scripts

None of these appear to be used; some are redundant.
The rename/relocation of the "create XBlock" script serves a few purposes..
Following *nix convention, you can add the bin directory to your ${PATH} to make the script directly invokable. Given this, the workbench- prefix is important  as it removes ambiguity from the command name; startnew.py is a poor name when exposed "globally".
If all new scripts (yes, expect additional PRs shortly) follow this convention, all workbench commands are "grouped"; auto/tab-completion FTW.

Thanks for the pull request, @stvstnfrd! I've created OSPR-855 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.
Feel free to add as much of the following information to the ticket:

supporting documentation
edx-code email threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Instead of removing the quality script can we make Travis run it?

and I guess we don't need coverage w/ coveralls

@sarina I added a new target make quality which runs PEP8 and Pylint. This is included as part of Travis' build. Consequently, a number of quality fixes are now included as separate commits.

Disclaimer: Coveralls is reporting a minor (0.02%) drop in coverage.
Upon inspection, this appears to be an "accounting mistake"; some of the PEP8/Pylint refactoring resulted in a slightly higher line count amongst untested code paths. So the logic and its testing are unchanged, but technically there are more untested lines. I'd like to consider this a non-issue and ignore during merge consideration.

Yeah that makes sense to me. I can take a review pass this week; @singingwolfboy can you review too?

@sarina @singingwolfboy Thanks for the quick review! I have added clarifying comments where requested and made the following significant changes:

removed ignoring of missing-docstring and just documented them
removed test_requires changes; deferring until a later date.

Let me know if you need anything else; otherwise this is ready to merge on my end.

:+1:
I'll let @singingwolfboy do final review & merge, but this is good by me. DB - the coverage slight drop is expected, and I assert we should accept.


indexList without title property may cause some problem

A feature or reporting a bug?
Bug
What is the current behavior?
IndexList component without title property may cause the position of uppercase bar unpredictable,also when tapping the fast navigation bar , for example, I should tap below the letter C to choose C.
What is the expected behavior?
Running as the demo shows to me.
How to reproduce the current behavior (if this is a bug)?
Not declaring the title property
Provide the test code and the tested page URL (if applicable)
Tested page URL:
Test code
<cube-index-list :data="cityData" @select="selectItem" ></cube-index-list>

Specify your

vue version: 2.5.2
cube-ui version: 1.1.0

commit: get actual title hight will fix this issue. Please wait our next version, published tomorrow perhaps ~


Fix warning with torch.meshgrid

Motivation
torch.meshgrid() has started raising a warning when not called with an explicit indexing parameter:
https://pytorch.org/docs/stable/generated/torch.meshgrid.html
Modification
Provide indexing='ij' to all calls to torch.meshgrid that don't already specify indexing.
Checklist
Before PR:

[x] Pre-commit or other linting tools are used to fix the potential lint issues.
[ ] Bug fixes are fully covered by unit tests, the case that causes the bug should be added in the unit tests.
[ ] The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[ ] The documentation has been modified accordingly, like docstring or example tutorials.

After PR:

[ ] If the modification has potential influence on downstream or other related projects, this PR should be tested with those projects, like MMDet or MMSeg.
[ ] CLA has been signed and all committers have signed the CLA in this PR.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


fix(scripts): Add hashbang to the index script.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Use containerd importer

Fixes dmcgowan/docker#48
Signed-off-by: Sam Whited<EMAIL_ADDRESS>

Still digging through tests since I can't run them right now to see if this all works or if I'm misunderstanding anything about the image and content stores. Will advise.

I think it is worth considering changing the name of ImportImage as it is very confusing. Image and layers used to be the same thing, this is a left over from that. Really this is creating a new image from a layer. So it takes in a layer stream and image metadata.

I see at least three different types of "Image" in containerd, and have found them used in different places in other parts of this package. What's the difference between containerd.Image, images.Image, and image.Image and which do you mean when you say to create an image config?
They all appear to have the same documentation (more or less) and the first two don't seem to implement the interface that's the last one.

I did a rebase on the underlying branch if you want to update.
As for the different image types. The image.Image from github.com/docker/docker/image is deprecated and should not longer be used (that whole package). In contained, containerd.Image is an image representing a specific platform and has some helper functions to perform operations in context of all the services. images.Image on the other hand is primitive which mirrors the API and what gets stored in an image store. It has no context around it, it doesn't even necessarily need to present a container image.


Lost data on any event other than proper disconnect. Synchronize all every now and then.

As i see in code synchronization happens only on player disconnect. If server crashed or stopped data could be lost.

This might be a good idea

https://github.com/MrNavaStar/InvSync/pull/26

Fixed by #35


Auto Width and Height

Add functionality auto width and height when :resizable="false" and :autosize="true"

Unfortunately it's not so easy! It works only when the component is not constrained in parent, because  width and height are required for the calculations. Honestly I don't know if it's possible to have both features.


chore: Add a comment over 2 lines in the new snippet format to prevent copy.

Let me know if the text needs a change.

@saranshdhingra now that #1384 is merged, those comments should be added there as well!


An extra '…' shows next to the buttons under query mode when setting 'Toolbar Size' to 'Regular / Small'

Storage Explorer Version: 1.29.0-dev
Build Number: 20230302.7
Branch: feature/table-explorer-react
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Ventura 13.2.1 (Apple M1 Pro)
Architecture: ia32/x64
How Found: From running test cases
Regression From: Previous release (1.28.0)
Steps to Reproduce

Launch storage explorer -> Open 'Settings' -> Change the setting 'Toolbar Size' to 'Small'.
Expand one storage account -> Tables.
Right click one table -> Click '[2] Open'.
Click 'Query' -> Observe the buttons under the query mode.
Check whether no extra '...' shows.

Expected Experience
No extra '...' shows.

Actual Experience
An extra '...' shows.

Additional Context
The '...' disappears after manually refreshing.

@JasonYeMSFT I believe this is very similar to the problem I see with the main toolbar, where the toolbar fails to collapse extra buttons into the More button when it's first displayed. Can you investigate this?

I debugged the code. The toolbar resize logic is executed before the query panel elements get rendered. As a result, the code that are meant to hide the more button cannot do anything and at a later stage the query panel gets rendered which includes the more button.

Verified this issue on the build 20230317.1. Fixed. So, we are going to close it.


Datepicker add a option to disable days from previous/next month

Hello, currently the datepicker has no option to disable the dates from the previous and the next month, it will always print the 30., 31. or so from the date before and the 1., 2.
Another option would be that the date-disabled function has the current date so we could build a global option like that: http://stackoverflow.com/questions/22544118/angular-ui-datepicker-limiting-days-to-one-month

I think, with a bit of tinkering, that you can achieve this by implementing a custom template for the datepicker.

yea rvanbaalen is right.. i did this and it worked great..
I also modified the more of the custom template to get the look I wanted too.
attached is what i was able to get..

@sandman45 : Can you please share the template you used to achieve this?
Thanks

+1 for the template @sandman45

@maverick09 Ok so this isn't a great way to solve it, but it worked for me:
The issue is that the activeDate property on the Datepicker's controller isn't easily accessible without going through a decorator to modify the directive. But the activeDate property is available in the template using `{{datepicker.activeDate}}.
In my custom template for the day picker I but an ngIf on the <button> element:
<button type="button" class="uib-daypicker__date"
             uib-is-class="'btn-info' for selectedDt,'active' for activeDt on dt"
             ng-click="select(dt.date)"
             ng-disabled="::dt.disabled"
             ng-if="datepicker.activeDate.getMonth() === dt.date.getMonth()"
             tabindex="-1">
             {{::dt.label}}
</button>

I didn't put the ngIf on the  because that would collapse the space for each day that wouldn't be rendered and make the days and dates not line up. To make sure these elements were ignored I slapped a ng-aria-hidden="{{datepicker.activeDate.getMonth() !== dt.date.getMonth()}}" on the  just to hide the empty cells from screen readers.
It works but it's kind of dirty :/ If you're not going to go the decorator route this will work for you. Just make sure you create your own custom template.
Hope that helps!

In my front of view the best way to do this it's put this setting into directive "template-url="yourWayToDatepickerTpl". Just like this

And then in "datepickerCustomTmpls/index.html" :

Then you can control daypicker or monthpicker or yearpicker in your templates "datepickerCustomTmpls/day.html", "datepickerCustomTmpls/month.html", "datepickerCustomTmpls/year.html". If you dont want use custom tmpl for year (for example), just delete template-url setting in uib-yearpicker

Code of day, month and year tmpls for @maverick09 :
Day, i'm already added @jonwinton solution (ng-if)

            <button type="button" class="btn btn-default btn-sm pull-left uib-left" ng-click="move(-1)" tabindex="-1"></button>

            <button id="{{::uniqueId}}-title" role="heading" aria-live="assertive" aria-atomic="true" type="button"
                    class="btn btn-default btn-sm uib-title" ng-click="toggleMode()"
                    ng-disabled="datepickerMode === maxMode" tabindex="-1">{{title}}</button>

            <button type="button" class="btn btn-default btn-sm pull-right uib-right" ng-click="move(1)" tabindex="-1">
                </button>

            {{::label.abbr}}

        {{ weekNumbers[$index] }}

            <button type="button" class="btn btn-default btn-sm"
                    uib-is-class="
                        'btn-info' for selectedDt,
                            'active' for activeDt
                               on dt"
                    ng-click="select(dt.date)"
                    ng-disabled="::dt.disabled"
                    ng-if="datepicker.activeDate.getMonth() === dt.date.getMonth()"
                    tabindex="-1">{{::dt.label}}
            </button>

Month:

            <button type="button" class="btn btn-default btn-sm pull-left uib-left" ng-click="move(-1)" tabindex="-1"></button>

            <button id="{{::uniqueId}}-title" role="heading" aria-live="assertive" aria-atomic="true" type="button"
                    class="btn btn-default btn-sm uib-title" ng-click="toggleMode()"
                    ng-disabled="datepickerMode === maxMode" tabindex="-1">{{title}}</button>

            <button type="button" class="btn btn-default btn-sm pull-right uib-right" ng-click="move(1)" tabindex="-1">
                </button>

            <button type="button" class="btn btn-default"
                    uib-is-class="
              'btn-info' for selectedDt,
              'active' for activeDt
              on dt"
                    ng-click="select(dt.date)"
                    ng-disabled="::dt.disabled"
                    tabindex="-1">{{::dt.label}}</button>

Year:

          <button type="button" class="btn btn-default btn-sm pull-left uib-left" ng-click="move(-1)" tabindex="-1"></button>
          <button id="{{::uniqueId}}-title" role="heading" aria-live="assertive" aria-atomic="true" type="button" class="btn btn-default btn-sm uib-title" ng-click="toggleMode()" ng-disabled="datepickerMode === maxMode" tabindex="-1">{{title}}</button>
          <button type="button" class="btn btn-default btn-sm pull-right uib-right" ng-click="move(1)" tabindex="-1"></button>

           <button type="button" class="btn btn-default"
             uib-is-class="
               'btn-info' for selectedDt,
               'active' for activeDt
               on dt"
             ng-click="select(dt.date)"
             ng-disabled="::dt.disabled"
             tabindex="-1">{{::dt.label}}</button>

It's my first answer on github) Dont kick me)

@jonwinton
How do you access the activeDate property with a decorator?  I am trying to hook my own functionality onto the change of month like is discussed here: http://stackoverflow.com/questions/26928861/angular-ui-bootstrap-datepicker-is-there-a-way-of-detecting-when-the-month
function DatepickerDirectiveDecorator($delegate) {
    var directive = $delegate[0];
    var link = directive.link;

    directive.compile = function() {
      return function(scope, element, attrs, ctrl) {
        link.apply(this, arguments);

        scope.$watch(function() {
          return ctrl[0].activeDate.getTime();
        }, function() {
          console.log('switched')
        });
      }
    };

    return $delegate;
  }

However activeDate like I'm using it here always seems to be undefined.  How do you access it?

Hi sir.
I have issue occur in Angularjs DatePicker. when i am Edit DateTime they can't show the date on UI but they are available in backend. kindly see the attachment you will understand.

hi sir.
I am using angularj Bootrap DatePicker. i have issue occur . when i am edit date. the can't show on UI but they can avilible on Backend. this Url that i am using DatePicker
kindly see the Attachment

Cannot say what exactly you are looking but might be this link can help you:
http://www.advancesharp.com/blog/1224/angularjs-jquery-ui-datepicker-for-any-condition

No need to use a custom template. Use the custom class option.

then in your controller\component
getCustomClass (date, mode) {
  let customClass = '';
  if (mode === 'day') {
    let monthToCheck = date.getMonth();

    // 'this' refers to the datepicker's scope so you can get access to all it's goodies
    let activeMonth = this.datepicker.activeDate.getMonth();

    if (monthToCheck === activeMonth) {
       customClass = 'datepicker-day-current-month';
    }
  }

  return customClass;
}

And now in your app's css you can do something like this:
.uib-datepicker .uib-day:not(.datepicker-day-current-month) {
  visibility: hidden;
}

I created this revision so that I could 'hear' the active date change. Hope it works for you. My change


⚠ Thumbnails API Endpoint has degraded performance

In e7b45ce, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 4748 ms

Resolved: Thumbnails API Endpoint performance has improved in abe7336 after 15 minutes.


BLE Split Link Layer Improvements

This issue lists all the improvements we want to include that are not part of bringing the Split Link Layer up to parity with the existing one (#12681).

[x] Add support for Controller (SW deferred)-based Privacy (#14621)
[ ] Change the role dependent event-done handling from type switch-case to callback concept.
[x] Move the LLL event prepare/resume queue/handling into LLL. (@cvinayak )
[x] Move the LLL-to-ULL Tx ack queue API to ULL domain.
[ ] Add documentation for ticker @stko-ot
[ ] Include the diagrams done by @pskhansen in the documentation
[ ] Robustness fix (nice to have): #13389
[x] Add tests in nrf52_bsim to run on upstream CI for (at least basic) controller functionality
[ ] Style fixes (#22048)
[ ] Create a porting guide to help users port to new radio hardware

"- [ ] Use atomic_cas when supported by the arch without disabling interrupts (see for more info)"
was removed due to https://github.com/zephyrproject-rtos/zephyr/pull/13471 being closed

"- [ ] Add support for (current) Mesh Extensions (#14678)"
removed as there was no real interest for that, and the ticket was morphed into adding full mesh extensions to the controller.

Ooooo - I like this! It might also be a good idea to add a basic / minimum recommendation and documentation for the radio API. Naturally, most chips will have a different radio, but we are mostly using the same interface so far. However, that interface is largely undocumented. Some state diagrams would be nice as well.

Ooooo - I like this! It might also be a good idea to add a basic / minimum recommendation and documentation for the radio API. Naturally, most chips will have a different radio, but we are mostly using the same interface so far. However, that interface is largely undocumented. Some state diagrams would be nice as well.

Just added a point at the end, thanks!

Add tests in nrf52_bsim to run on upstream CI for (at least basic) controller functionality

Isn't this done?

@carlescufi we close this issue since realistically no work will be done on this

realistically no work will be done on this

Will the split LL ever be "official", or should we use the old system?

realistically no work will be done on this

Will the split LL ever be "official", or should we use the old system?

Split Link Layer is the default upstream since Zephyr v2.0.0-rc1 , see https://github.com/zephyrproject-rtos/zephyr/commit/6cf26b8fe63afdb83817ad551a3f03c40f744a72

I see, thanks!


Minor tweak in description of make-link command

Notes for Reviewers
This PR fixes sub-task #6255
Signed commits

[X] Yes, I signed my commits.

🚀 Preview for commit 93c326131f1ad2e20df9b0860440474c7e0addb1 at: https://6447a4a6d8fc5b39b154d486--meshery-docs-preview.netlify.app


Add -z now to default Linker flags + add -z lazy

BTW, I'm open to at some point changing the default to not be -z now to match the other linkers, however I'm not sure what outstanding bugs there are in the lazy binding code, so those would need to be fixed first.


Change misleading message of Style/EmptyLinesAroundAccessModifier:

If we have file test.rb with code:
module A
  private
  def a
  end
end

rubocop show warning: test.rb:2:3: C: Keep a blank line before and after private.
Ok, manually add blank line before and after private:
module A

  private

  def a
  end
end

No good: test.rb:2:1: C: Extra empty line detected at module body beginning.
So I changed message to
Keep a blank line after %s. And before, if it is not first line of module.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests are passing.
[x] The new code doesn't generate RuboCop offenses.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.

I don't think this is the way to go. There should simply be two different messages for the two different scenarios.

@bbatsov I don't think I have enouth knowladge to implement two different messages

Just file this as a bug then and me or someone else will fix it eventually.

Added issue for this https://github.com/bbatsov/rubocop/issues/3503


de-duplicate provider-gitlab resource names to match gitlab api

Signed-off-by: Jan Willies<EMAIL_ADDRESS>Description of your changes
This PR renames a couple of things in order to stay close to the gitlab api:

rename projecthook.projects.gitlab.crossplane.io to hook.projects.gitlab.crossplane.io
rename projectmember.projects.gitlab.crossplane.io to member.projects.gitlab.crossplane.io
rename groupmember.groups.gitlab.crossplane.io to member.groups.gitlab.crossplane.io

If you are using this provider already, note that this is a breaking change.
Fixes #36
I have:

[x] Read and followed Crossplane's [contribution process].
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
unit tests

I checked it works. LGTM

Thanks for checking @Stanislav2017! I'll take a look what's missing for the CI.
Since this a breaking change (and we want to get it in before releasing a first version), I'd like someone else to have a look too. Maybe @benagricola?


Double check our source link support

We support sourcelink for a long time now but I haven't double checked if we package this correctly  for a similar long while either.
Creating this ticket to remember to investigate our support for this feature is still good.

Hello, I would say that it's probably not packaged or shipped correctly.
I don't know if I should post this here or create a separate issue, let me know if I need to change it please.
I've tried yesterday to load PDB for Nest and it failed on me.
The Symbol Load log looks like this:

SYMSRV:  BYINDEX: 0x16B
<local_cache_path>*https://msdl.microsoft.com/download/symbols
Nest.pdb
3F54E208B90547C5BE259DA1B4429D7C1
SYMSRV:  UNC: <local_cache_path>\Nest.pdb\3F54E208B90547C5BE259DA1B4429D7C1\Nest.pdb - path not found
SYMSRV:  UNC: <local_cache_path>\Nest.pdb\3F54E208B90547C5BE259DA1B4429D7C1\Nest.pd_ - path not found
SYMSRV:  UNC: <local_cache_path>\Nest.pdb\3F54E208B90547C5BE259DA1B4429D7C1\file.ptr - path not found
SYMSRV:  HTTPGET: /download/symbols/Nest.pdb/3F54E208B90547C5BE259DA1B4429D7C1/Nest.pdb
SYMSRV:  HttpQueryInfo: 80190194 - HTTP_STATUS_NOT_FOUND
SYMSRV:  HTTPGET: /download/symbols/Nest.pdb/3F54E208B90547C5BE259DA1B4429D7C1/Nest.pd_
SYMSRV:  HttpQueryInfo: 80190194 - HTTP_STATUS_NOT_FOUND
SYMSRV:  HTTPGET: /download/symbols/Nest.pdb/3F54E208B90547C5BE259DA1B4429D7C1/file.ptr
SYMSRV:  HttpQueryInfo: 80190194 - HTTP_STATUS_NOT_FOUND
SYMSRV:  RESULT: 0x80190194
https://msdl.microsoft.com/download/symbols: Symbols not found on symbol server.
SYMSRV:  BYINDEX: 0x16C
<local_cache_path>*https://symbols.nuget.org/download/symbols
Nest.pdb
3F54E208B90547C5BE259DA1B4429D7Cffffffff
SYMSRV:  UNC: <local_cache_path>\Nest.pdb\3F54E208B90547C5BE259DA1B4429D7Cffffffff\Nest.pdb - path not found
SYMSRV:  UNC: <local_cache_path>\Nest.pdb\3F54E208B90547C5BE259DA1B4429D7Cffffffff\Nest.pd_ - path not found
SYMSRV:  UNC: <local_cache_path>\Nest.pdb\3F54E208B90547C5BE259DA1B4429D7Cffffffff\file.ptr - path not found
SYMSRV:  HTTPGET: /download/symbols/Nest.pdb/3F54E208B90547C5BE259DA1B4429D7Cffffffff/Nest.pdb
SYMSRV:  HttpQueryInfo: 80190194 - HTTP_STATUS_NOT_FOUND
SYMSRV:  HTTPGET: /download/symbols/Nest.pdb/3F54E208B90547C5BE259DA1B4429D7Cffffffff/Nest.pd_
SYMSRV:  HttpQueryInfo: 80190194 - HTTP_STATUS_NOT_FOUND
SYMSRV:  HTTPGET: /download/symbols/Nest.pdb/3F54E208B90547C5BE259DA1B4429D7Cffffffff/file.ptr
SYMSRV:  HttpQueryInfo: 80190194 - HTTP_STATUS_NOT_FOUND
SYMSRV:  RESULT: 0x80190194
SYMSRV:  BYINDEX: 0x16D
<local_cache_path>*https://symbols.nuget.org/download/symbols
Nest.pdb
3F54E208B90547C5BE259DA1B4429D7C1
SYMSRV:  UNC: <local_cache_path>\Nest.pdb\3F54E208B90547C5BE259DA1B4429D7C1\Nest.pdb - path not found
SYMSRV:  UNC: <local_cache_path>\Nest.pdb\3F54E208B90547C5BE259DA1B4429D7C1\Nest.pd_ - path not found
SYMSRV:  UNC: <local_cache_path>\Nest.pdb\3F54E208B90547C5BE259DA1B4429D7C1\file.ptr - path not found
SYMSRV:  HTTPGET: /download/symbols/Nest.pdb/3F54E208B90547C5BE259DA1B4429D7C1/Nest.pdb
SYMSRV:  HttpQueryInfo: 80190193 - HTTP_STATUS_FORBIDDEN
SYMSRV:  RESULT: 0x80190193

Also, according to this post, there should be a Download symbols link on NEST's nuget page on the right under Download package, but it's missing.

It looks like since SourceLink was originally implemented, some required pieces that are needed for it to work were lost in refactoring of build components and the move to newer SDK projects. I've reinstated these pieces and I've opened #4224 to address. I've checked the PR in both Visual Studio 2019 and JetBrains Rider, and both are functioning correctly.


Push to Heroku

We should be able to to this now.

Guide to Deploy with Heroku and MongoDB Atlas on the Full-Stack Blog

Heroku deployed


Cannot uninstall python 3.6

Steps to Reproduce (for bugs)

Try to uninstall an existing python 3.6 install using chocolatey

C:\\ProgramData\\chocolatey\\bin\\choco.exe uninstall python3 --yes --no-progress --limit-output --package-params /InstallDir:C:\\Python36 --timeout 2700 --allow-multiple --version 3.6.8

act:
python3;/InstallDir:C:\\Python36

python3 v3.6.8
Directory 'C:\ProgramData\chocolatey\lib\python<IP_ADDRESS>.3.6.8' does not exist.
python3 uninstall not successful.
Package install not found:'C:\ProgramData\chocolatey\lib\python<IP_ADDRESS>.3.6.8'
python3 not uninstalled. An error occurred during uninstall:
 python3 uninstall not successful.

Chocolatey uninstalled 0/0 packages.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).
Second path fragment must not be a drive or UNC name.
Parameter name: path2

exp: Uninstalls python 3.6
Note: The path that exists is C:\ProgramData\chocolatey\lib\python3 and not C:\ProgramData\chocolatey\lib\python<IP_ADDRESS>.3.6.8
Your Environment

Operating System and version: win 10
Chocolatey version: 0.10.15

Not sure where the problems that you are seeing are. However, I can unfortunately say it isn't a package related problem.
The messages you are seeing are from the chocolatey executable itself, saying it can't find the path and not something we can control in the package (the package itself don't even have an uninstall script).
It looks like the problem may be because you are using --allow-multiple when calling chocolatey uninstall, which I guess makes Chocolatey look for a package that has been side-loaded.
Anyhow, as this problem is more related to the chocolatey executable, the issue needs to be opened up over at https://github.com/chocolatey/choco (if a similar issue for the problem don't already exist).

Thanks, it works when ommiting the --allow-multiple but then I still get an error saying
        "Chocolatey uninstalled 0/0 packages. ",
        " See the log for details (C:\\ProgramData\\chocolatey\\logs\\chocolatey.log).",
        "Second path fragment must not be a drive or UNC name.",
        "Parameter name: path2"

But I will put that on top of the ~ 700 other issues you mentioned ;)


Update available v1.16.7

Update to MakeMKV v1.16.7 (where is v1.16.6 tho?) is available. An update would be awesome, if it's not too much to ask for.
I anyway hope you do have update scripts, so it's not timeconsuming to update.
Thanks in advance!

Thanks for reporting, I will update the image.

Might I also ask why QT v5.9.9 is getting used while there are way more up to date versions?
Is there a compatibility break, or breaking changes?
Newer version:
v5.15.2
v6.2.3
Maybe it works with newer versions too? If yes, are there any benefits using newer version? Just asking out of curiosity.

Initially, this QT version was used because more recent ones were using the statx system call.  At that time, this system call was not allowed by default by Docker.
At some point I might try to upgrade the QT version, since today statx system call should be allowed by default on most installations.  However, since QT is compiled manually (to significantly reduce the size of the image), this requires more time than just using an existing package...

New Docker image available.


Change OBS_ID keyword from integer to string

The event list extension has an OBS_ID keyword that is currently defined as integer. This comes from the original event list format specification from Karl. It appears however that OGIP has in the meantime switched to a string, see for example https://heasarc.gsfc.nasa.gov/docs/heasarc/ofwg/docs/general/ogip_94_001/ogip_94_001.html which says:

OBSERVATION ID: Sequence numbers are commonly used by missions to identify 'uniquely' a particular dataset. The definition of the sequence number is not unique across mission and typically depend upon the different way a dataset is identified within that mission. Although sequence number suggests a numerical value only, past mission had given the sequence number as a mixture of numerical and character values. To avoid a proliferation of keywords to store the sequence number value, the OGIP recommend to use OBS_ID. The keywords value is a string, to allow back compatability with the already archived mission. The OBS_ID keyword value can be used by database software.

So I would propose to change the OBS_ID keyword from integer to string.

@jknodlseder - A few questions:

What are current high-energy missions (Fermi, Chandra, XMM, ...) using?
Integer or string for OBS_ID?
In tools like Gammalib and Gammapy, would we have to support both integer and string for OBS_ID for a while or forever?
For FITS header keywords, there's no difference, right? If the value can be parsed as an int, tools like astropy.io.fits return it as an int. So we're only talking about table column dtypes for EVENTS and OBS_INDEX or other tables that have OBS_ID as a column?

Just to give the counter-argument why we might want to keep int:

This is a breaking change. Current files use int and science tools only support int:
https://github.com/gammapy/gammapy/search?utf8=✓&q=OBS_ID
https://github.com/gammalib/gammalib/search?utf8=✓&q=OBS_ID
The quote you cite says that int would be nicer, and this is only allowed as string because older missions had string.
One use case I could think of for int OBS_ID that I've used myself is to merge events from multiple observations (all for HESS 1) into one event list and then join that with event lists from other recos and cuts to study event reconstruction differences and cut differences. Of course it can be done differently.

My impression is that int is simpler and more convenient and is in place, so I don't see the advantage of doing the change.
@jknodlseder - if you're available I'll put this on the agenda for next Tuesday's IACT DL3 telcon:
https://github.com/open-gamma-ray-astro/2016-04_IACT_DL3_Meeting/blob/master/notes/2016-09-06-IACT_DL3_Telcon.md

@jknodlseder - Do you still think CTA / IACT DL3 data should change OBS_ID to be a string instead of an int?
It would be quite a big breaking change, at least in Gammapy we work with OBS_ID as integer in several places, and presumably in ctools you do as well?
@kosack - Is there a plan for this in CTA already? Should discussions continue here on the DL3 spec, or is there a different spec / process now in CTA?


chore(localize): updated translation from Crowdin

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


fix: get pull request only once

/hold
Waiting test it locally.

Codecov Report

Merging #165 (53e9d18) into master (cdc207e) will increase coverage by 0.17%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##           master     #165      +/-   ##
==========================================
+ Coverage   85.72%   85.90%   +0.17%
==========================================
  Files          11       11
  Lines         946      972      +26
==========================================
+ Hits          811      835      +24
- Misses         76       77       +1
- Partials       59       60       +1

Flag
Coverage Δ

unittests
85.90% <87.50%> (+0.17%)
:arrow_up:

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

Impacted Files
Coverage Δ

internal/pkg/externalplugins/tars/tars.go
79.19% <87.50%> (+2.77%)
:arrow_up:

Continue to review full report at Codecov.

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

/hold cancel
/assign @Mini256
PTAL~ Thanks!

/cc @whileTrue0x90

/lgtm

@Mini256  Could you please help me merge this?

/merge


training log

Hi, noahzn
Could you provide training logs?
When I was training, I didn't know why there was a problem with the predicted rgb:

Can you give me some help? Thank you very much!

In the case above where I changed the camera intrinsics to real intrinsics , when I used the parameters provided in the code, it became like this, I wonder why they have this difference,

I seem like to meet the same problem with your first four pictures, could you share me with the problem that you fould about it？

I seem like to meet the same problem with your first four pictures, could you share me with the problem that you fould about it？
Hey, my problem is that the intrinsic parameters are set incorrectly. You can check your intrinsic parameters.


discussion: esp32 firmware?

anyone reckon if this implementation is lightweight enough to be turned into custom firmware for esp32 devices?

It has already been done.
You can look into the projects I've started, there's one doing just that.

It has already been done.
You can look into the projects I've started, there's one doing just that.

ah cool. mind pointing me to the repo/path? i only see a fork of esphome, but not sure what that would have to do with diying homekey specifically?

https://github.com/rednblkx/HK-HomeKit-Lib


Multiple problems when Guild Wars 2 client has never been run

While debugging mutex code I started the application under a user account that has never used the client before. Multiple exceptions ensued. Better handling of first ever run should be on the list of things to do.
(I can likely handle this eventually but I'm looking for existing mutex problems that will effect most people)

I closed issue #82 because I could not reproduce but I would not at ALL be surprised if these two issues are related as this was the state my user account was in when hitting the issue where 0 handles were found for GW2.exe when trying to kill the mutex.
I will create some new user accounts and try to reproduce #82.
@TheCheatsrichter - Is there an easy way to get rid of ANY/ALL LB user/machine data such that LB will reinitialize all configuration files / registry values or is it as simple as just removing the AppData for the user?

@ShiftLeftLogical
There are 3 files/folders to delete for a fresh start:
%userprofile%\appdata\local\TheCheatsrichter
%appdata%\Guild Wars 2\Launchbuddy.bin
%appdata%\Gw2 Launchbuddy\


Fallback to TCP for DNS Truncation

We are utilizing redisson 3.21.0 to connect an AWS Lambda on the Java 11 Runtime to a MemoryDB cluster within a seperate AWS VPC. We recently ran into an issue where we were getting truncated DNS responses due to a change made related to a CVE. Our lambda began failing. Based on the CVE, it seems that in case of a truncated UDP DNS response, it should automatically fallback to TCP. However, we are not seeing this happening and we are getting out right failures. We have confirmed that on the Netty end, the fallback to TCP has been implemented as of Netty 4.1.37 and we have confirmed we are on Netty 4.1.68. We are wondering if there is a config we can set to be able to fallback to TCP when receiving a truncated DNS response.
References:

CVE for truncating DNS responses: https://nvd.nist.gov/vuln/detail/CVE-2023-28450
Netty change to support fallback to TCP: https://github.com/netty/netty/issues/9127

It's handled automatically https://github.com/netty/netty/blob/13e143c2aeec7cc38fa54095189ed4bd762a2975/resolver-dns/src/main/java/io/netty/resolver/dns/DnsNameResolver.java#L1389

Just a heads-up: Ive opened an issue with netty about the possible misinterpretation of their tcp fallback behavior. If my understanding is correct, then redisson might need to make some change as well.
https://github.com/netty/netty/issues/13616

@yomr let me check it.

Done.

alpine 3.18.3 , redisson 3.24.0 and netty 4.1.100 also can reproduce


Wrong address returned

Describe the bug
When you click on a location that doesn't have an address, it returns the address of the last location that had an address.
To Reproduce
Steps to reproduce the behavior:
Click on a random spot in the sea, the latitude and longitude are returned correct, but the address will be the address on your location.
Expected behavior
Return null (or "Invalid Address")
Info Required

Which version of the library do you actually use?
6.1.0

Do you have the localization permission granted?
Yes

Are you sending parameters to the activity through the bundle?
Uhm, I don't know.

Android monitor shows any log related to the library when the error is shown?
It's not an error

Any other thing that could help me to reproduce the error?
It's a simple problem.

Smartphone:

Device: Emulator Nexus 5X

Hi @Lars-
I'll check it ASAP. Many thanks for reporting it.

@Lars- new release version 6.1.1 is now available for download. It solves this issue.
Many thanks for reporting it. 🍷 😉

Awesome, thanks man!


edgenet.kubernetes: Fix Docker installation

{
   "cache_update_time":1614770720,
   "cache_updated":false,
   "changed":false,
   "msg":"'/usr/bin/apt-get -y -o \"Dpkg::Options::=--force-confdef\" -o \"Dpkg::Options::=--force-confold\"      install 'docker-ce=5:19.03*'' failed: E: Sub-process /usr/bin/dpkg returned an error code (1)\n",
   "rc":100,
   "stderr":"E: Sub-process /usr/bin/dpkg returned an error code (1)\n",
   "stderr_lines":[
      "E: Sub-process /usr/bin/dpkg returned an error code (1)"
   ],
   "stdout":"Reading package lists...\nBuilding dependency tree...\nReading state information...\ndocker-ce is already the newest version (5:19.03.15~3-0~ubuntu-focal).\nThe following packages were automatically installed and are no longer required:\n  libfprint-2-tod1 libllvm9 nvidia-kernel-common-440\nUse 'sudo apt autoremove' to remove them.\n0 upgraded, 0 newly installed, 0 to remove and 42 not upgraded.\n1 not fully installed or removed.\nAfter this operation, 0 B of additional disk space will be used.\nSetting up docker-ce (5:19.03.15~3-0~ubuntu-focal) ...\r\nJob for docker.service failed because the control process exited with error code.\r\nSee \"systemctl status docker.service\" and \"journalctl -xe\" for details.\r\ninvoke-rc.d: initscript docker, action \"start\" failed.\r\n* docker.service - Docker Application Container Engine\r\n     Loaded: loaded (\u001b]8;;file://for1452/lib/systemd/system/docker.service\u0007/lib/systemd/system/docker.service\u001b]8;;\u0007; enabled; vendor preset: enabled)\r\n     Active: activating (auto-restart) (Result: exit-code) since Wed 2021-03-03 12:35:33 CET; 5ms ago\r\nTriggeredBy: * docker.socket\r\n       Docs: \u001b]8;;https://docs.docker.com\u0007https://docs.docker.com\u001b]8;;\u0007\r\n    Process: 279658 ExecStart=/usr/bin/dockerd -H fd:// --containerd=/run/containerd/containerd.sock \u001b[0;1;31m(code=exited, status=1/FAILURE)\u001b[0m\r\n   Main PID: 279658 (code=exited, status=1/FAILURE)\r\ndpkg: error processing package docker-ce (--configure):\r\n installed docker-ce package post-installation script subprocess returned error exit status 1\r\nErrors were encountered while processing:\r\n docker-ce\r\n",
   "stdout_lines":[
      "Reading package lists...",
      "Building dependency tree...",
      "Reading state information...",
      "docker-ce is already the newest version (5:19.03.15~3-0~ubuntu-focal).",
      "The following packages were automatically installed and are no longer required:",
      "  libfprint-2-tod1 libllvm9 nvidia-kernel-common-440",
      "Use 'sudo apt autoremove' to remove them.",
      "0 upgraded, 0 newly installed, 0 to remove and 42 not upgraded.",
      "1 not fully installed or removed.",
      "After this operation, 0 B of additional disk space will be used.",
      "Setting up docker-ce (5:19.03.15~3-0~ubuntu-focal) ...",
      "Job for docker.service failed because the control process exited with error code.",
      "See \"systemctl status docker.service\" and \"journalctl -xe\" for details.",
      "invoke-rc.d: initscript docker, action \"start\" failed.",
      "* docker.service - Docker Application Container Engine",
      "     Loaded: loaded (\u001b]8;;file://for1452/lib/systemd/system/docker.service\u0007/lib/systemd/system/docker.service\u001b]8;;\u0007; enabled; vendor preset: enabled)",
      "     Active: activating (auto-restart) (Result: exit-code) since Wed 2021-03-03 12:35:33 CET; 5ms ago",
      "TriggeredBy: * docker.socket",
      "       Docs: \u001b]8;;https://docs.docker.com\u0007https://docs.docker.com\u001b]8;;\u0007",
      "    Process: 279658 ExecStart=/usr/bin/dockerd -H fd:// --containerd=/run/containerd/containerd.sock \u001b[0;1;31m(code=exited, status=1/FAILURE)\u001b[0m",
      "   Main PID: 279658 (code=exited, status=1/FAILURE)",
      "dpkg: error processing package docker-ce (--configure):",
      " installed docker-ce package post-installation script subprocess returned error exit status 1",
      "Errors were encountered while processing:",
      " docker-ce"
   ]
}

TASK [edgenet.kubernetes : Enable EdgeNet service] ********************************************************************************************************************************
fatal: [localhost]: FAILED! => {"changed": false, "msg": "Unable to start service edgenet: Job for edgenet.service failed because the control process exited with error code.\nSee \"systemctl status edgenet.service\" and \"journalctl -xe\" for details.\n"}


Update troubleshoot-aspnet-core-localization.md

Fixes #28674

cc @hishamco

I don't think we need this because already we describe everything in troubleshooting section


DynUNet needs updating for deep_supervision

Describe the bug
Following this PR to DynUNet, the corresponding notebook needs updating.
I tried simply removing deep_supervision = True from the constructor, but this wasn't enough.
The subsequent error is attached here: dynunet.txt.
@Nic-Ma I think you might best know what needs to be changed here.

Dear @Nic-Ma @rijobro et al,
The dynunet returns list (containing the output tensor). Had it returned a tensor in validation mode, it would have been easier to use.
So I changed a bit in the sliding window inference code to make it work. What do you think?
  #seg_prob = predictor(window_data, *args, **kwargs).to(device)  # batched patch segmentation
  seg_problist2tensor = predictor(window_data, *args, **kwargs)
  seg_prob = seg_problist2tensor[0] if isinstance(seg_problist2tensor, list) else seg_problist2tensor
  seg_prob = seg_prob.to(device)

Dear @Nic-Ma @rijobro et al,
The dynunet returns list (containing the output tensor). Had it returned a tensor in validation mode, it would have been easier to use.
So I changed a bit in the sliding window inference code to make it work. What do you think?
  #seg_prob = predictor(window_data, *args, **kwargs).to(device)  # batched patch segmentation
  seg_problist2tensor = predictor(window_data, *args, **kwargs)
  seg_prob = seg_problist2tensor[0] if isinstance(seg_problist2tensor, list) else seg_problist2tensor
  seg_prob = seg_prob.to(device)

I will have a try to run the notebook tomorrow, thanks for your issue report.
Thanks.

I will have a try to run the notebook tomorrow, thanks for your issue report.
Thanks.

Possibly best to make any changes in this PR, as it will already have PEP8 modifications (and git merge is horrible wit jupyter notebooks)

Possibly best to make any changes in this PR, as it will already have PEP8 modifications (and git merge is horrible wit jupyter notebooks)

Hi @saruarlive and @rijobro ,
I just submitted a commit to fix this issue in that branch: https://github.com/Project-MONAI/tutorials/pull/107/commits/c61cfd1cd02a2a2dea223d06ac2eed2d9ad1a929
@wyli , should we try to merge the notebook format PR ASAP? Otherwise, it's hard to develop new PRs and resolve conflicts.
Thanks.

Hi @saruarlive and @rijobro ,
I just submitted a commit to fix this issue in that branch: https://github.com/Project-MONAI/tutorials/pull/107/commits/c61cfd1cd02a2a2dea223d06ac2eed2d9ad1a929
@wyli , should we try to merge the notebook format PR ASAP? Otherwise, it's hard to develop new PRs and resolve conflicts.
Thanks.

Hi @saruarlive and @rijobro ,
I just submitted a commit to fix this issue in that branch: https://github.com/Project-MONAI/tutorials/pull/107/commits/c61cfd1cd02a2a2dea223d06ac2eed2d9ad1a929
@wyli , should we try to merge the notebook format PR ASAP? Otherwise, it's hard to develop new PRs and resolve conflicts.
Thanks.

sure, could you please help review the changes? @Nic-Ma @yiheng-wang-nv @ericspod

Hi @saruarlive and @rijobro ,
I just submitted a commit to fix this issue in that branch: https://github.com/Project-MONAI/tutorials/pull/107/commits/c61cfd1cd02a2a2dea223d06ac2eed2d9ad1a929
@wyli , should we try to merge the notebook format PR ASAP? Otherwise, it's hard to develop new PRs and resolve conflicts.
Thanks.

sure, could you please help review the changes? @Nic-Ma @yiheng-wang-nv @ericspod

Dear @Nic-Ma @rijobro et al,
The dynunet returns list (containing the output tensor). Had it returned a tensor in validation mode, it would have been easier to use.
So I changed a bit in the sliding window inference code to make it work. What do you think?
  #seg_prob = predictor(window_data, *args, **kwargs).to(device)  # batched patch segmentation
  seg_problist2tensor = predictor(window_data, *args, **kwargs)
  seg_prob = seg_problist2tensor[0] if isinstance(seg_problist2tensor, list) else seg_problist2tensor
  seg_prob = seg_prob.to(device)

@saruarlive This was helpful for me. I was finally able to get DynUNet working for the specified tutorial.

Dear @Nic-Ma @rijobro et al,
The dynunet returns list (containing the output tensor). Had it returned a tensor in validation mode, it would have been easier to use.
So I changed a bit in the sliding window inference code to make it work. What do you think?
  #seg_prob = predictor(window_data, *args, **kwargs).to(device)  # batched patch segmentation
  seg_problist2tensor = predictor(window_data, *args, **kwargs)
  seg_prob = seg_problist2tensor[0] if isinstance(seg_problist2tensor, list) else seg_problist2tensor
  seg_prob = seg_prob.to(device)

@saruarlive This was helpful for me. I was finally able to get DynUNet working for the specified tutorial.

Hi all, I just submitted a PR to update the DynUNet tutorial to meet the changes here. Now, the forward function will only return a single tensor (for both training and eval mode, with or without deep supervision).


there should draft up once without watching for changes

I'd like to be able to build and deploy my app as one off, without having to use more then one shell, e.g. something like draft up --single-shot  or draft up --no-watch or draft up --once or even draft once.

You can disable the watch functionality by flipping watch to false in draft.toml. That will give you the single-shot functionality. It's just enabled by default.

See the User Guide for more context on the parameters available in draft.toml.

Ok, but I don't think editing config file is good UX, flag would be more convenient.

I intentionally left out feature flags for the pre-production release so we don't have two conflicting places to set feature flags, but point noted.


Target netstandard 2.0 with our analyzer

Can anyone think of a reason not to target netstd 2.0?

I believe the reason for netstandard1.3 is to support oldner MSBuild (ie. older VS versions).

I think that's a valid reason to keep it where it's at.  I have pretty much eliminated anything lower than VS2019 on any of my environments, but I imagine I'm an outlier.  Closing this issue for now.


ci(release-please): enable option to trigger manual release creation

Offers the option to trigger a release manually.

How would one decide which package to release? Currently we only have one, but in future would this trigger a new release for every package?

How would one decide which package to release? Currently we only have one, but in future would this trigger a new release for every package?

True. Thanks for pointing this out.


Feature request: non-interactive deletion of ports collections, jails etc.

Trying to manage Poudriere through Puppet. My module is progressing nicely, but removal of ports collections etc. is impossible due to the command throwing a Y/n prompt which Puppet can't handle since it's non-interactive. It'd be nice to have a command-line flag to always assume 'Y' so that all commands can be executed by non-interactive tools.

What command? pkgclean?
    -n          -- Do not actually remove anything, just show what would be
                   removed
    -y          -- Assume yes when deleting and do not confirm

What I'm looking for, is a way to do something like:
poudriere ports -d -p local
..but without needing to answer Y to a prompt. Maybe I'm missing something here, in that case I'll submit a PR to the docs ;-)

ports -d already only prompts if stdin is a tty, otherwise it assumes yes. There is also always yes | poudriere.

Can you confirm whether or not puppet itself got stuck on the prompt, or just your testing outside of puppet?

I tested actual Puppet runs and those do work. I was thrown off by the prompts I get in a TTY and didn't expect different behaviour. So if anything, this difference might be addressed in the docs a little more explicitly.

So should we add a -y to poudriere ports just like I added to poudriere jail in ae7bc3723f29e862c89b51a97ef6c89a12453c5d, or is that not needed?

I guess it makes sense for consistency. Go for it if you want. I don't think it is needed as it only prompts in a TTY and yes is a thing, I meant to close this when #649 went in so closing now.

/usr/local/bin/poudriere ports -k -d                       -p $PORTS
prompts even if it is in a script ,
this is minor ,
but annoying.


Tablite component sorting on drag n drop

The tablite component is sorting on drag n drop, so the ordering of items get mangled up.
http://adaptv.github.io/adapt-strap/##tablelite

Drag any item on the list
The list shuffles and item gets lost.

@kashjs should we sorting here https://github.com/Adaptv/adapt-strap/blob/master/src/utils/utils.js#L303.
Because the  loadPage  function gets called on a drag and drop, once there is a drop and sortKey is true the list reorders.

yah this is bug....

https://github.com/Adaptv/adapt-strap/pull/204

seems like this is fixed?? if so can you close it.


New link added in practice for beginners

Summary of your changes
Description
Added new link in the section practice for beginners

Checklist

[x] My change follows the Contributing Guidelines
[x] I have added only one new link to the list.
[x] I have checked that the link that I added does NOT exist in the project already.
[x] I have sorted the link alphabetically under the related section.

@sdmg15 I added leetcode coding platform link to when you get stuck section. Please review it.


Optimize print preview

In the print preview are some not ideal representations to print a drawing

done in v2, but not in v3 (this repo)

Still not done, close/hide all bars/buttons in print preview

In addition, the image of the map should be displayed on the entire page. We currently have too much empty space.


[Doc][Wiki] Restrict scope of ServiceAccount

Hi
I just made a change on https://github.com/wpCloud/wp-stateless/wiki/Manual-Setup
This change will restrict scope of the serviceAccount to the current bucket instead of the whole project.
Thanks !


Upgrade to Java 17

PR Description
Upgrade source compatibility and CI to use Java 17
Fixed Issue(s)

https://github.com/ConsenSys/teku/issues/7047
Documentation

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

Changelog

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

We need to update our README as well:
https://github.com/ConsenSys/teku/blob/master/README.md#install-prerequisites

Think it would be good to upgrade all the besu libraries to the latest ones as part of this PR as well

Might be worth waiting until just before merging this as likely to be more besu releases in between.

@siladu when you are free, can you rebase this? The current CHANGELOG.md in master is fine, so no need to have changes there.

@siladu when you are free, can you rebase this? The current CHANGELOG.md in master is fine, so no need to have changes there.

Don't worry. Already rebased it.

@siladu getting Caused by: org.apache.logging.log4j.LoggingException: log4j-slf4j-impl cannot be present with log4j-to-slf4j when updating the besu libraries. (you can see build) Any idea? I tried fixing it by excluding this dependency in the evm module, but it doesn't work.

Upgrading<EMAIL_ADDRESS>to<EMAIL_ADDRESS>breaks 4 tests on windowsBuild consistently:
onDeneb_shouldStoreBlobSidecarsAlongWithBlock()
onDeneb_shouldStoreEarliestBlobSidecarSlotCorrectlyWhenThereIsGap()
onDeneb_shouldStoreEarliestBlobSidecarSlotCorrectlyWhenItsDenebGenesis()
retrieveBlobSidecars_shouldReturnBlobSidecarsWhenExists()

These tests actually fail on my Windows laptop as well consistently with same assertion error, so can investigate.
Upgrading besu libraries from 22.10.4 to 23.4.3 break a lot of tests with an exception: org.apache.logging.log4j.LoggingException: log4j-slf4j-impl cannot be present with log4j-to-slf4j
Probably should exclude some dependency?

I've pushed the fix https://github.com/Consensys/teku/pull/7107/commits/0889bebf4060c0e2a868dc23050bfd5ab8c435af
Detailed explanation is available in log-captor library https://github.com/Hakky54/log-captor#using-log-captor-alongside-with-other-logging-libraries
My little worry about it is that it could affect logging of some tests, but I ran few and didn't notice any changes.
As an alternative we could introduce new task/configuration dedicated to the tests with usage of the log-captor, it will look similar to those we have for integration or acceptance tests. So it will run as a separate task here in CI and require separated sourceSet, dedicated directories for these tests, dedicated dependencies set etc.

Wonder if we can enable CapellaUpgradeAcceptanceTest with upgraded besu libraries?

@StefanBratanov it's not related, we already had latest docker there. We need a feature to toggle Capella not on time condition like it was done for mainnet. As far as I know it's not implemented in Besu yet

@StefanBratanov it's not related, we already had latest docker there. We need a feature to toggle Capella not on time condition like it was done for mainnet. As far as I know it's not implemented in Besu yet

Yeah got it. All good. Not related yeah.

@siladu getting Caused by: org.apache.logging.log4j.LoggingException: log4j-slf4j-impl cannot be present with log4j-to-slf4j when updating the besu libraries. (you can see build) Any idea? I tried fixing it by excluding this dependency in the evm module, but it doesn't work.

Sorry @StefanBratanov I missed this. Thanks @zilm13 for the fix. Let me know if there's any other besu questions, maybe better to ping me on Slack.

@siladu all good, thank you for the help! Sorry, we've not touched it for more than a month, just started testing.

Removing the doc-change-required label because the docs already mention Java 17+ is required:
https://docs.teku.consensys.net/get-started/install/build-from-source


Hi Sajiyah-salat, I have been exploring this repo and I saw it got so complex after several contribution

I mean I am not able to understand what is the starting point for this project.
So if you like I can help you with that by arranging all sorts of things as I understand

Yeah, after so many contributions, it's kinda messed up. Please help me do that...

Where can I discuss some doubts with you, @Sajiyah-Salat

We can discuss on LinkedIn