Implement support for configuring the connection pool of inbound clusters via the Sidecar API

Please provide a description of this PR:
Implements support for server-side connection pool configuration via the Sidecar API, based on the API change in this PR.
Salient things here is how this interacts with DestinationRule for configuring inbound clusters. This PR implements the following precedence for picking connection pool config (highest to lowest):

per-port ConnectionPool from the Sidecar
top-level InboundConnectionPool from the Sidecar
per-port TrafficPolicy.ConnectionPool from the DestinationRule
top-level TrafficPolicy.ConnectionPool from the DestinationRule.
(3 and 4 are unchanged from before this PR.)

This PR does not change how client-side connection pools are configured (which remains DestinationRule only).
TODO before this moves from draft to real PR:

[x] update go.mod/go.sum to point to istio/istio after https://github.com/istio/api/pull/2961 is merged
[x] remove the SidecarPort changes after PR (TODO find link) is merged fixing it
[ ] add release notes for this change
[ ] add integration tests (there seem to be a lack of integration tests that use Sidecar?)

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@ZackButcher: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
974858af0852c0ab71f3f0c26bff9ef993102251
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Not sure if the update to istio.io/api 1.20 should happen in a separate PR (IIRC there's a whole ritual for that update that's unfamiliar to me). I just did the simple go get<EMAIL_ADDRESS>to update. I will go add a release note shortly, but the code itself should be ready for review.

@ZackButcher: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
974858af0852c0ab71f3f0c26bff9ef993102251
link
true
/test release-notes

gencheck_istio
974858af0852c0ab71f3f0c26bff9ef993102251
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ZackButcher: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
974858af0852c0ab71f3f0c26bff9ef993102251
link
true
/test release-notes

gencheck_istio
974858af0852c0ab71f3f0c26bff9ef993102251
link
true
/test gencheck

lint_istio
974858af0852c0ab71f3f0c26bff9ef993102251
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ZackButcher: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
974858af0852c0ab71f3f0c26bff9ef993102251
link
true
/test lint

gencheck_istio
f76c4da65ab19d2d5e44fb1b00c33845d1771b33
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[WIP] Jar stamping

Description

Motivation

@ittaiz so far I am lost in a muddle and probably won't have enough time to investigate so am putting this up for you

@ittaiz wondering if you have any easy ideas for this issue, if not I will keep looking.
bazel build --extra_toolchains=//test/toolchains:high_level_transitive_deps_strict_deps_error -- //src/java/io/bazel/rulesscala/specs2:specs2_test_discovery fails with the following
error: Target '@io_bazel_rules_scala_scala_reflect//:io_bazel_rules_scala_scala_reflect' is used but isn't explicitly declared, please add it to the deps.
You can use the following buildozer command:
buildozer 'add deps @io_bazel_rules_scala_scala_reflect//:io_bazel_rules_scala_scala_reflect' //src/java/io/bazel/rulesscala/specs2:specs2_test_discovery
error: Target '@io_bazel_rules_scala_scala_library//:io_bazel_rules_scala_scala_library' is used but isn't explicitly declared, please add it to the deps.
You can use the following buildozer command:
buildozer 'add deps @io_bazel_rules_scala_scala_library//:io_bazel_rules_scala_scala_library' //src/java/io/bazel/rulesscala/specs2:specs2_test_discovery

Looking at specs2_test_discovery_scalac_worker_input and focusing on scala-reflect we see that under Classpath and IndirectJars entries, we have the following

bazel-out/k8-fastbuild/bin/external/io_bazel_rules_scala_scala_reflect/scala-reflect-2.11.12-stamped.jar
bazel-out/host/bin/external/io_bazel_rules_scala_scala_reflect/scala-reflect-2.11.12-stamped.jar

while under DirectJars we have only bazel-out/host/bin/external/io_bazel_rules_scala_scala_reflect/scala-reflect-2.11.12-stamped.jar
Whereas without this change, everyone has just external/io_bazel_rules_scala_scala_reflect/scala-reflect-2.11.12.jar
So the stamping moves the jar into both k8-fastbuild and host
"configurations"? (don't understand this part very well). Which seems undesirable as jar stamping ought to be independent of the config, but maybe that's not how bazel likes it.
I guess probably the direct rule is depending on the host configuration, while it is depending on //external:io_bazel_rules_scala/dependency/specs2/specs2 which is on the k8-fastbuild configuration.
But I guess it's already a bad thing that the classpath contains both jars, right? Because while it might work we'd probably not want to have that in the first place.
(changing all of cfg = "host" to cfg = "target" does not help; it was just something I tried for funsies)

Not sure and indeed a problem.
I have an idea- Wdyt about hacking bazel itself just to see if it will happen also via java_common.stamp_jar? Hack bazel and then build rules_scala with the hacked version.
Also two more thoughts-

Is this the only instance? Maybe we should  print in the plugin (just for now) all classpath entries which have more than one instance
Does this happen only with bind? Maybe we can move this to toolchain. To check maybe just change rules_scala to explicit dependency and not via bind

@ittaiz
This happened with at least scala-library and scala-reflect that I observed.
Removing native.bind did help, though just doing scala-library/reflect/compiler was not enough - also had to do scala-xml, junit, specs2, specs2_junit (not sure which of these last 4 were necessary and which were optional).
In https://github.com/bazelbuild/bazel/issues/1952 it sounds like it is reasonable to try to get rid of native.bind anyways, even if it wasn't causing an issue right now. I am not 100% sure that getting rid of bind will solve all remaining problems or that it won't cause strange issues.
But assuming you are fine with it and there aren't backwards compatibility issues (I couldn't find any docs on native.bind - the linked issue did suggest that people might have wanted to remove it from docs, which may have happened), I'd like to just get rid of rules_scala usage of native.bind just to not have to think about it and getting confused by it when trying to solve this issue.
In my experiment I just replaced all of //external:io_bazel_rules_scala/dependency/scala/scala_compiler with @io_bazel_rules_scala_scala_compiler and similar. Based on your comment it sounds like you want something else, though I don't 100% understand.
Do you mean that we want to add to scala_toolchain e.g.
        "scala_compiler": attr.label(
            default = "@io_bazel_rules_scala_scala_compiler",
        ),```
(or maybe as a provider so we can group all the different libraries together into one attr) (and would we just stick all the libraries on here, including specs2 and jmh and guava and everything else which uses native.bind)

And then access this toolchain everywhere (though I don't think we always have ctx so that's a problem e.g.

```scala_binary(
    name = "scaladoc_generator",
    main_class = "scala.tools.nsc.ScalaDoc",
    visibility = ["//visibility:public"],
    runtime_deps = [
        "//external:io_bazel_rules_scala/dependency/scala/parser_combinators",
        "//external:io_bazel_rules_scala/dependency/scala/scala_library",
        "@io_bazel_rules_scala_scala_compiler",
        "@io_bazel_rules_scala_scala_reflect",
        "@io_bazel_rules_scala_scala_xml",
    ],
)```)

@ittaiz ^ in case the notification was missed

Thanks for the ping.
I didn't miss it per-se but was chewing on the problem and got sidetracked.
The reason we use bind is to offer a layer of indirection where users can decide to use their own deps (different jars, different loaders) without rules_scala needing to change.
The alternative (very roughly speaking) is toolchains in the sense that this is a different mechanism of indirection which allows user configuration.
@liucijus started to dive into binds & toolchains for the cross compilation saga.
Vaidas,
do you have any thoughts on the matter?
To be honest I'm very uncertain on how to play this (since I remember someone poking holes into  the applicability of toolchains, maybe it was because they arrive too late in the lifecycle).
cc @johnynek in case he has some words of wisdom

I don't know the problem with binds deeply, but I think one of the biggest problems is as @Jamie5 described with depending directly on labels that should come via toolchain. I think it prevents rules_scala from getting rid of binds. Maybe there is something more. Grouping Scala libraries with provider is a good abstraction as long as user can override it easily.

Closing this as stale, we already have jar stamping in rules scala


How to create foregn_key with different to_table_name and column_name

If the feature doesn't exist, request the feature

Can you give us some specific examples?

@winebarrel
https://ja.stackoverflow.com/questions/67467/ridgepoleでforeign-keyをto-tableと別名のcolumn-nameで生成する方法/67491#67491
please add document.

Thank you for your report. 😃
The syntax of ridgepole basically follows the Rails migration.
Please refer to the Rails documentation.

@winebarrel OK thanks.
But, This tool has many differences from the rails migration api. ex: ( https://api.rubyonrails.org/classes/ActiveRecord/ConnectionAdapters/Table.html#method-i-foreign_key ), and Readme shows it can be used without rails.
So

The syntax of ridgepole basically follows the Rails migration.
Please refer to the Rails documentation.

This is not obvious.
After all, I thought it would be more user friendly to add text to the Readme.


Support dynamic relocation in libuser

This makes every userland programs position-independent.
This PR also correctly separates every section in its appropriate segment.

I'm curious: Does this fix vi being huge because of the .bss being allocated in the file?

@roblabla yuuup

As I understand (your PR comment wasn't really explicit), this makes kernel builtins illegible for ASLR. But wouldn't relocations for every other userspace programs be handled by the process loader service on behalf of the starting process, instead of having it relocate itself ?
This means the _start will be different for a builtin and a regular process, but they both use libuser. We might want to add some conditional compilation in libuser for that.

Hum ... Travis still hasn't responded yet, and Mergeable isn't even here. I think they didn't like the repository transfer. @roblabla can you do something about your mergeable bot ?

Forcing travis rebuild: closing and reopening PR

Closes #216

Rebased


Handling show requests when membership deleted.

Updated show logic for membership controller.
Added show specs for controller.

[#115812555]

:ship: :it:


DataFramesDataset should take index for timestamp field if available.

Currently, we require that the timestamp field is a column in the passed datasets, but don't support it if it is the index of the DataFrame.
Ideally, it would try to use the index for it by default, raising an error if neither timestamp was passed nor the index can be used for it.

thanks! this should work out nicely with the #1973 stuff


Flaky-test: MLPendingAckStoreTest.testMainProcess

Search before asking

[X] I searched in the issues and found nothing similar.

Example failure
https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706
Exception stacktrace
Error:  Tests run: 16, Failures: 1, Errors: 0, Skipped: 14, Time elapsed: 19.252 s <<< FAILURE! - in org.apache.pulsar.broker.transaction.pendingack.impl.MLPendingAckStoreTest
[807](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:808)
  Error:  testMainProcess(org.apache.pulsar.broker.transaction.pendingack.impl.MLPendingAckStoreTest)  Time elapsed: 0.311 s  <<< FAILURE!
[808](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:809)
  java.util.concurrent.ExecutionException: org.apache.bookkeeper.mledger.ManagedLedgerException$BadVersionException: org.apache.pulsar.metadata.api.MetadataStoreException$BadVersionException: org.apache.zookeeper.KeeperException$BadVersionException: KeeperErrorCode = BadVersion for /managed-ledgers/tnx/ns1/persistent/test-txn-topic-test__transaction_pending_ack/__pending_ack_state
[809](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:810)
  	at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:396)
[810](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:811)
  	at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:2073)
[811](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:812)
  	at org.apache.pulsar.broker.transaction.pendingack.impl.MLPendingAckStoreTest.testMainProcess(MLPendingAckStoreTest.java:259)
[812](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:813)
  	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
[813](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:814)
  	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
[814](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:815)
  	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
[815](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:816)
  	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
[816](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:817)
  	at org.testng.internal.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:132)
[817](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:818)
  	at org.testng.internal.InvokeMethodRunnable.runOne(InvokeMethodRunnable.java:45)
[818](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:819)
  	at org.testng.internal.InvokeMethodRunnable.call(InvokeMethodRunnable.java:73)
[819](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:820)
  	at org.testng.internal.InvokeMethodRunnable.call(InvokeMethodRunnable.java:11)
[820](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:821)
  	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[821](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:822)
  	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
[822](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:823)
  	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
[823](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:824)
  	at java.base/java.lang.Thread.run(Thread.java:833)
[824](https://github.com/apache/pulsar/actions/runs/3338223656/jobs/5525935706#step:10:825)
  Caused by: org.apache.bookkeeper.mledger.ManagedLedgerException$BadVersionException: org.apache.pulsar.metadata.api.MetadataStoreException$BadVersionException: org.apache.zookeeper.KeeperException$BadVersionException: KeeperErrorCode = BadVersion for /managed-ledgers/tnx/ns1/persistent/test-txn-topic-test__transaction_pending_ack/__pending_ack_state

Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

another one: https://github.com/apache/pulsar/actions/runs/3295550972/jobs/5434989599

another one: https://github.com/apache/pulsar/actions/runs/3333737684/jobs/5584052059


Feature: Gear export

Was thinking about exporting gearsets to other useful tools. Like

etro.gg
GearPlanner spreadsheets by "The Balance" (via clipbaord)
I would like to get some feedback on how needed/wanted this would be to plan it:
/polls "Awesome! Need it asap" "Will use it regularly when it is there" "Would use it sometimes" "Don't really need it"

/polls "Awesome! Need it asap" "Will use it regularly when it is there" "Would use it sometimes" "Don't really need it"


BouncyCastle

Hi!
I'd like to decrypt some AES with BouncyCastle as the provider, who used mode "SC". Is there anyway you can show me how it could be done?

Sorry, not getting much time, would have to find time to see, would also need sample code used for encryption to test.


Grid view cover animations

Hi,
can you add animations to game covers i.e. when hovered?
Thanks

Em...I have thought this for a not short time and decide not to have it. I want it lighter and simple.


x86 build

Is it possible to have x86 build to avoid problem like these https://github.com/nvllsvm/freecyngn/issues/19 please ?
Thanks !

Not interested in making a x86 build ?


Fix bug for crashing when positional embedding disabled SongUNet

Modulus Pull Request
Description
SongUNetPosEmbed has the option to disable positional embeding by setting N_grid_channels to 0. This change keeps the forward pass from crashing by adding a check to make sure the positional embedding exists. It also elimates some code duplication between the positional embed and non-positional embed version.
Closes
https://github.com/NVIDIA/modulus/issues/504
Closes
https://github.com/NVIDIA/modulus/issues/529
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[ ] The CHANGELOG.md is up to date with these changes.
[x] An issue is linked to this pull request.

Dependencies

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci


change time only in text box is not consistent when click cancel

To reproduce
http://jsfiddle.net/iamcap/3xyr6ppz/10/

open rangepicker modal
update to-time and click apply
open again rangepicker modal
click cancel

Thanks @iamcap. Can you take a look when you have a min @nonmetalhail? Thanks.

Ok, I just pushed a fix to master with tag v0.2.1. Thanks.


Copy person raises an error in production (HTTP 500)

Does not seem to be the case in development environment. This is also related to #32  we will need to implement custom deep copy operations for database objects.

Fixed.


Add a list of all sub-resources used in the scene

Closes https://github.com/godotengine/godot-proposals/issues/4567

In the proposal I mentioned a dialog, but then realized a popup is enough.
The list shows all sub-resources found in every node in the scene (excluding nested sub-sub-resources). They are grouped by type. If a resource is named, a name is displayed. If not, the owning node's name is displayed instead. Also if a resource is re-used multiple times, number of usages shows too.
This could be probably more fleshed-out, but the original purpose of this feature was to locate built-in scripts in a scene with many of them, and it serves it well.

Thanks!


Can't send message without mrkdwn format because Missing mrkdwn in chat.postMessage.json

Hi,
When I developed slack app with python-slack-sdk, I noticed 'mrkdwn' option is disabled in chat.postMessage.
I tried to post message by tester in api homepage (https://api.slack.com/methods/chat.postMessage), but the result was the same.
This indicates that it is a bug in the slack api itself, not in python-slack-sdk.
And, I checked 'node-slack-sdk' to find api type are generated from java-slack-sdk/json-logs/samples/api/*
(please see node-slack-sdk/packages/web-api/src/response/* or node-slack-sdk/scripts/generate-web-api-types.sh)
In java-slack-sdk/json-logs/samples/api/chat.postMessage.json, I cannot find 'mrkdwn' args.
So, I think this is the cause of the bug.
Please add mrkdwn option to java-slack-sdk/json-logs/samples/api/chat.postMessage.json.
Steps to reproduce:

goto tester (https://api.slack.com/methods/chat.postMessage/test)
set mrkdwn to false and input some text
'mrkdwn' is not in responce json and result is formatting with mrkdwn (Failed to turn off mrkdwn!)

This is my first issue to post on github, so there may be some mistakes, but please help me out.
Thanks,
Yoshi

HI @YoshiInTokyo, thanks for writing in!
I'm not sure about the difficulty that you're having yet. You should be able to use the mrkdwn argument in any of the SDKs. Also, the java-slack-sdk/json-logs/samples/api/chat.postMessage.json is generated from chat.postMessage API response, not its arguments. Thus, if I understand your message here correctly, the JSON file is unrelated.
If a text message using the Slack format does not work for you, can you share examples reproducing your issue?
I hope this helps!

Thank you for your quick response!
I'm sorry that I don't know all the details and that my writing is inadequate.
To reproduce it, in the tester, fill in the bot token and channel, set mrkdwn to false, and put M$_{BH}/M_{odot}$ in the text box.
The message sent to you should not be italicized, but it is in fact italicized.

@YoshiInTokyo Thanks for sharing the details.
I've also observed the same behavior when having text value along with mrkdwn: false. The issue should be resolved on the Slack product side and we'll escalate the issue to the engineering teams.
One workaround that I can suggest is to use blocks for safety. For instance, if you have a plain_text typed text in a block, the outlook of it would be always the one you would like to have. You can use Block Kit Builder to preview how blocks work.

Thanks again for flagging the issue and I hope this was helpful to you.

@seratch
Thank you for your advice!
Now, I try to use this third-party extension to render latex syntax, and this extension is only valid in text field, not blocks.
So, I regret to said that your method is not matched for my purpose.
Anyway, I hope that my issue improves slack api.
Bests,
Yoshi.
P.S. I apologize for sending the issue to this repository, not the help center.

Hi @YoshiInTokyo, thanks for your reply! Another workaround would be to customize the extension project and use pre tags surrounded by ``` instead of giving a raw input to your Python code.
I've escalated the issue to the teams concerned. As we are unable to track the progress of the server-side bug fixes in this SDK issue tracker, let us close this issue now. When you would like to know the progress of the updates on the server-side, I'm sorry to bother you but please contact our customer support agents. Thanks again for your feedback here 🙇


Failed to create catalog manager

Describe this problem
Panic with the stacktrace:
2022-10-18 16:01:56.037 ERRO [common_util/src/panic.rs:42] thread 'main' panicked 'Failed to create catalog manager: VisitSysCatalog { source: VisitorOpenTable { source: Unexpected { source: OperateByWriteWorker { space_id: 7, table: "BIZ_643000197_INFLUENCE_DEFAULT", table_id: TableId(7696581394717), source: Channel { source: ReadWal { source: Read { source: "Corruption: block checksum mismatch: expected<PHONE_NUMBER>, got<PHONE_NUMBER>  in /home/admin/data/ceresdbx/wal/263234.sst offset 30618826 size 1260", backtrace: Backtrace(   0: <snafu::backtrace_shim::Backtrace as snafu::GenerateBacktrace>::generate
             at /home/chunshao.rcs/github/CeresDB/ceresdb/wal/src/manager.rs:26:21
      <wal::manager::error::Read as snafu::IntoError<wal::manager::error::Error>>::into_error
             at /home/chunshao.rcs/github/CeresDB/ceresdb/wal/src/manager.rs:26:21
      <core::result::Result<T,E> as snafu::ResultExt<T,E>>::context::{{closure}}
             at /home/chunshao.rcs/.cargo/registry/src/github.com-1ecc6299db9ec823/snafu-0.6.10/src/lib.rs:318:30
      core::result::Result<T,E>::map_err
             at /rustc/d394408fb38c4de61f765a3ed5189d2731a1da91/library/core/src/result.rs:855:27
      <core::result::Result<T,E> as snafu::ResultExt<T,E>>::context
             at /home/chunshao.rcs/.cargo/registry/src/github.com-1ecc6299db9ec823/snafu-0.6.10/src/lib.rs:318:9
      <wal::rocks_impl::manager::RocksLogIterator as wal::manager::BlockingLogIterator>::next_log_entry
             at /home/chunshao.rcs/github/CeresDB/ceresdb/wal/src/rocks_impl/manager.rs:569:25
   1: <wal::manager::BatchLogIteratorAdapter as wal::manager::BatchLogIterator>::next_log_entries::{{closure}}::{{closure}}
             at /home/chunshao.rcs/github/CeresDB/ceresdb/wal/src/manager.rs:320:50
      <tokio::runtime::blocking::task::BlockingT

Steps to reproduce
Have no idea about how to reproduce this panic now.
Expected behavior
Won't fail.
Additional Information
maybe this bug similar to #308

This is caused by the corruption of the RocksDB files. Usually, such error will happen when writing to the disk which is already full.

I guess no need to fix this bug because this is caused by RocksDB. Let's close it.


pip installation has errors when prerequisites are not installed

I am not sure if this is by design, but when you run pip to install subradar, but you don't have numpy installed, the install process fails.  This might happen if you are setting up a fresh virtual environment.

(ve_sds1) freeze.ig.utexas.edu 135% pip3 install -r requirements.txt
Collecting bitstring>=3.1.5
  Using cached bitstring-4.1.4-py3-none-any.whl (60 kB)
Collecting pygdal>=3.2.1
  Using cached pygdal-<IP_ADDRESS>.tar.gz (5.4 kB)
Collecting h5py>=2.9.0
  Using cached h5py-3.10.0-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (4.8 MB)
Collecting matplotlib>=3.3.1
  Using cached matplotlib-3.8.2-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (11.6 MB)
Collecting scipy>=1.5.4
  Downloading scipy-1.12.0-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (38.5 MB)
     |████████████████████████████████| 38.5 MB 9.5 MB/s
Requirement already satisfied: numpy>=1.16.2 in ./ve_sds1/lib/python3.9/site-packages (from -r requirements.txt (line 6)) (1.26.4)
Collecting pandas>=0.24.2
  Using cached pandas-2.2.0-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (13.0 MB)
Collecting pvl>=0.3.0
  Using cached pvl-1.3.2-py2.py3-none-any.whl (66 kB)
Collecting spiceypy>=3.1.1
  Using cached spiceypy-6.0.0-py3-none-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (1.7 MB)
Collecting geopandas>=0.8.1
  Using cached geopandas-0.14.3-py3-none-any.whl (1.1 MB)
Collecting pytest>=6.2.1
  Using cached pytest-8.0.0-py3-none-any.whl (334 kB)
Collecting rsr>=1.0.5
  Using cached rsr-1.0.7-py3-none-any.whl (17 kB)
Collecting subradar>=1.1.0
  Using cached subradar-1.1.2.tar.gz (15 kB)
    ERROR: Command errored out with exit status 1:
     command: /disk/kea/SDS/code/work/ngg/sds_tacc1/ve_sds1/bin/python3 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-dakzlyms/subradar/setup.py'"'"'; __file__='"'"'/tmp/pip-install-dakzlyms/subradar/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-rjbhi5nw
         cwd: /tmp/pip-install-dakzlyms/subradar/
    Complete output (9 lines):
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-dakzlyms/subradar/setup.py", line 6, in <module>
        import subradar
      File "/tmp/pip-install-dakzlyms/subradar/subradar/__init__.py", line 28, in <module>
        from . import iem, invert, roughness, surface, utils, filtering, simulation, fractal
      File "/tmp/pip-install-dakzlyms/subradar/subradar/iem.py", line 10, in <module>
        from scipy.special import factorial
    ModuleNotFoundError: No module named 'scipy'
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
WARNING: You are using pip version 20.2.3; however, version 24.0 is available.
You should consider upgrading via the '/disk/kea/SDS/code/work/ngg/sds_tacc1/ve_sds1/bin/python3 -m pip install --upgrade pip' command.

A workaround is to manually install scipy and numpy first, but not sure if you should do that.
This is the contents of requirements.txt
bitstring>=3.1.5
pygdal>=3.2.1
h5py>=2.9.0
matplotlib>=3.3.1
scipy>=1.5.4
numpy>=1.16.2
pandas>=0.24.2
pvl>=0.3.0
spiceypy>=3.1.1
geopandas>=0.8.1
pytest>=6.2.1

rsr>=1.0.5
subradar>=1.1.0

The RSR circular dependency is fixed in v1.1.4.
That does not solve the dependency issue you are reporting , though. Oddly enough, I have a similar issue (ModuleNotFoundError: No module named 'numpy') when installing v1.1.4 on my laptop, but it does install smoothly on my Ubuntu server.

I started looking at how to make packages, and I see now that rather than the towardsdatascience article you had, there is now an official python tutorial:
https://packaging.python.org/en/latest/tutorials/packaging-projects/
I was looking at your code, and I see that one of the things you do is that you import a module in subradar in order to access the __version__ variable in the python code and use that in setup.
There is a bit of discussion about this maybe not being the best way to do versioning, and rather to place that metadata into the toml metadata file (in the comments under the original post):
https://stackoverflow.com/questions/458550/standard-way-to-embed-version-into-python-package
So removing the version variable and then not importing the code as part of the setup script seems like a better idea to me -- reduces risk of unwanted interactions.  I guess the downside is that people can't query the version number within the code.


Android Basics: lists

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-lists?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-2-pathway-2%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-lists#4
In which task and step of the codelab can this issue be found?
At the "5. Put it all together" section
Describe the problem
At the detailed explanation of the output of the code, the subclass of its parent class(Item)"Noodles" is called "Noodle"
Steps to reproduce?

Go to...
Click on...
See error...

Versions
What version of Android Studio are you using?
Additional information
Include screenshots if they would be useful in clarifying the problem.

If I understand your question right my answer is
Using super keyword in an override function will call the codes in parent , and if you use additional codes in it you can do more but if you do not use super keyword in override function you will ignore the parent codes ( function in parent class ).
example:
if you have this class
open class Property(val name:String) {
	open fun printName(){
        println(name)
    }
}

and you inherit this two classes:
class newProperty(val myName: String): Property(myName){
    override fun printName(){
        super.printName()
        println("this is my overriden function with super keyword")
    }
}

class newProperty2(val myName: String): Property(myName){
    override fun printName(){
        // here we didn't call the supper keyword
        println("this is my overriden function without super keyword!")
    }
}

And if you write this code as your main function
fun main(){
   val n = newProperty("Nilson")
   n.printName()
}

Result will be:
Nilson
this is my overriden function with super keyword
And if you use this code as your main function:
fun main(){
   val n = newProperty2("Nilson")
   n.printName()
}

Result will be:
this is my overriden function without super keyword!
Also I've attached some links for you to get a better understanding of inheritance in Kotlin

https://www.journaldev.com/19705/kotlin-inheritance#:~:text=All classes in Kotlin have,because of famous diamond problem.
https://stackoverflow.com/a/46400277

If you had any further questions I am here to help you 🙂

@hoyanxun
Thank you so much for reporting this issue. This is fixed and will be published soon.


🐛 [firebase_ui_oauth_apple] Displayname is null

Bug report
I am using the new flutterfire version for signing into my app. I have an issue with the apple login. (firebase_ui_oauth_apple: ^1.0.8)
I need to get the user's name after the registration because without that the Appstore is not allowed to upload the new release.
I saw in the source code that the provider contains the necessary scopes, however FirebaseAuth.instance.currentUser.displayname is always null. With the GoogleProvider() everything is working well and I can get the display name.
My code:
Steps to reproduce
Steps to reproduce the behavior:
 return AuthFlowBuilder<OAuthController>(
      provider: AppleProvider(),
        listener: (oldState, state, controller) {
          if (state is SignedIn) {
              //HERE THE state.user.displayName IS EMPTY
              //AFTER THE FirebaseAuth.instance.currentUser.display name IS ALSO EMPTY
        },
        builder: (context, state, controller, _) {
          return ElevatedButton(
              style: AppThemes.BUTTON_FULL_WIDTH_THEME.copyWith(
                  backgroundColor: MaterialStateProperty.all(Colors.white)
              ),
              onPressed: (){
                controller.signIn(Platform.isAndroid ? TargetPlatform.android : TargetPlatform.iOS);
              },
              child: Row(
                children: [
                  Images.image(Images.ICON_LOGIN_APPLE,width: 30),
                  Expanded(
                      child: Center(
                          child: state is SigningIn ?
                          const SizedBox(
                            width: 17,
                            height: 17,
                            child: CircularProgressIndicator(
                              strokeWidth: 2,
                              valueColor: AlwaysStoppedAnimation<Color?>(AppColors.BLUE_DARK),
                            ),
                          )
                        :
                          Text(AppLocalizations.of(context)!.login_apple,style: const TextStyle(color: Colors.black)))),
                ],
              )
          );
        }
    );

Expected behavior
It should contain the display name, because without that I can not upload the app for the appstore

Thanks for the report @kovilevi98
This is an issue with underlying iOS SDK for which you can follow-up in https://github.com/firebase/firebase-ios-sdk/issues/4393 for further updates. Since firebase_ui_oauth_apple or firebase_auth plugins are wrappers around native SDKs, the existing functionality is leveraged and we are not doing anything at client side to display the user details.
Closing from here.

Hello @darshankawar, Is there any work around for getting the displayName of the User ?


portico: Emphasize cover letter requirement for job applications.

Change on https://zulip.com/jobs

Merged, thanks @alya!


pass percentage float?

do you guys care if this is float or int. its a float in database and we have been exporting it as float so far.

float is fine with me. though then you should remove two of the dashes on line 390 :)

Personally, I don't like it being a float.
With integers it's clear that the whole number represents a percentage.
As a float, it's a little ambiguous between whether 75% is 75 or .75. It also adds floating point arithmetic complications for (IMHO) unnecessary levels of precision.
My current opinion is: Let's change the export to cast to integers OR do a brief investigation to see if anyone is using sub-percent precision and change the underlying schema.
In either case, we may want to explicitly say the number must be between 0-100 in this documentation.

My 2.0 cents - leave it as a float to allow for greater precision (5 9's ?), but add documentation to make the range clear

I'm w/ @jgan on this one -- unnecessary level of precision that won't be
used and potentially confusing.
Thx
Enis Konuk
Google Cloud Learning
<EMAIL_ADDRESS>+1<PHONE_NUMBER>
On Mon, Jul 20, 2020 at 4:50 PM davetchen<EMAIL_ADDRESS>wrote:

My 2.0 cents - leave it as a float to allow for greater precision (5 9's
?), but add documentation to make the range clear
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/CloudVLab/qwiklabs-content-bundle-spec/pull/62#issuecomment-661325994,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMERA3QX42X7PBSW4SB3NDR4SUZ3ANCNFSM4O47LVLA
.

I went with integer everywhere: json schema, db schema, code etc. no more headache from this hopefully


Clean up modules.  app.modules should be injected into main brewLabApp module.  Also, change name to brewLabApp.controllers, etc to indicate they are children of the main app.

The app.modules are already injected, bozo.


[REQUEST] Autoformat code based on major mode

A feature to autoformat code (e.g. under SPC c f) based on the major mode would be nice. That is, use clang-format for c++-mode, autopep8 for python-mode, etc.

I intend to add this eventually; possibly by 2.1.2. My eye is on https://github.com/lassik/emacs-format-all-the-code. I'll let you know when there's movement on this matter.

could use prettier for js and friends
https://github.com/prettier/prettier

@ar1a The emacs-format-all-the-code plugin uses prettier for javascript et all.

 Look into a universal "code formatting" solution

This is now in develop, as part of the new :editor format module. It's still experimental and needs more testing, but I'll consider this issue resolved.


Table | Does not support footers

User story:

As a user of the Table component I want to be able to specify a footer through the API, separate from the data array.

Dev hint

This could be done in the column prop (adding a footerproperty to the column object to define what string/number will be displayed there) or in a separate prop from column or data simply called footer. The latter would mean the column definitions could be more static and only data and footer would be likely to depend on dynamic data.

Requirements:

Accept data for a footer row which has the same typing as the normal cell rows. It should accept new overrides for the new subcomponents.
New footer data should go into the tfoot DOM element to keep it as accessible as possible.
It should conditionally render the tfoot depending on if any data is being rendered in it.
It should adhere to the responsiveness that exists in the rest of the Table component.
Tests should be updated if necessary to cover the new code additions.

Design:
The FooterRow and FooterCell can be restyled Row/Cell styled components, but should use grayXLight for the background and set the font-weight to 700.

This shouldn't be a breaking change.

Started 8/5/21


Examples

It would be nice if you could put the examples in github, and link to them from the main project.

The demo website (http://wjs.wexample.com/) is now available into https://github.com/weeger/wjs/tree/demo


Timeline to deprecate Bond serializer

Before opening a feature request against this repo, consider whether the feature is generic enough for the SDK, or it should be a specific plugin/extension to the SDK.
Is your feature request related to a problem?
Bond is not actively maintained for a few years, we should have a timeline to deprecated it and move to an alternative.
Describe the solution you'd like.
Move to other serializer like JSON or probobuf.
Describe alternatives you've considered.
N/A.
Additional context.
Add any other context about the feature request here.

I've made changes to support JSON serializer in AzMon module, so replacement to JSON shouldn't be complex, just provide different implementations

You need to check on this with collector team and Diagnostic Data Viewer team:

Bond is fully supported by Collector at the moment, for 1DS client, 1DS server, and 1DS JS SDK.
1DS uses its own "bondlite" code generator / serializer, that isn't affected by existing bond bugs, esp. since all input is controlled by app locally, not arbitrary random input from remote end.
Diagnostic Data Viewer most likely has a dependency on Bondi serialization in order to decode and show outgoing events.

This is a bigger undertaking that would require orchestration with other interested parties.

@pbgodwin - you can use json.hpp to place a complex structure to any property as a string:
props.SetProperty("object", "{ \"some_nested\": \"property\" }");

Your object containing JSON string lands in data property bag.
Subsequently, depending on your consumption model ... assuming you route your data to Kusto, you can extract from there as:
ClientEvents
| project object=todynamic(todynamic(original_message).data.object)

Which returns you a structured JSON object back, i.e. Bond protocol isn't a limitation here. You'd also need to plan on how you route your "flat" properties thru systems like ADLS aka Cosmos, since in many cases these systems require flat 2D datasets. It gets complicated when you pass an array.

Looking at how Visual Studio does this with the JSON uploader, they'd still express the values as "flat" lists, flattening on their end... Screenshot removing EUPI:

See how every prop gets the full tree node name expanded on it... This is JSON payload, actually, not Bond. Which means, it's not about Bond vs JSON at a transport layer, it's about having things expressible as "relatively flat" lists. For the downstream processors to more efficiently handle the stream without having to deserialize anything. You'd artificially flatten your JSON at source. This is not very efficient from payload size (though compressed). And not very efficient from storage size. But much more efficient from the pipeline processing - queries size. Like, you can run queries much faster on these datasets, either with Kusto or Azure Data Lake.
Ultimately if you really-really want to keep your data as a tree structure, then you can do so with manual JSON serialization. Other teams appear to be not wanting their data to remain tree-structured. And artificially flattening it at source - even on the JSON protocol uploader path. JSON is readily supported by Collector today, it's just less efficient and more costly, since collector has to do more parsing.
Closest analog of Bond in terms of efficiency is Protobuf. Collector doesn't support Protobuf. This discussion has to start with collector team, as to what protocols they are willing to support; and if it is acceptable to use JSON instead of Bond. A few years ago it was considered unnecessarily expensive from cogs cost perspective. Hope that makes sense.

I would like to bump this feature request. Microsoft's Component Governance detected the presence of Bond (unsupported since 2018) and declared it to be a high severity issue.

I would like to bump this feature request. Microsoft's Component Governance detected the presence of Bond (unsupported since 2018) and declared it to be a high severity issue.

In your ADO build pipeline, right after the source checkout is done - delete the two projects:

.\tools\fiddler\OneDSInspector\OneDSInspector.csproj
.\tools\server\server.csproj

C# Bond is not used anywhere in C++ codebase. This is false-positive.


mise panics if prefix: is used on certain plugins

Describe the bug
When using prefix: on certain plugins (bun, deno, java, ruby,..) the installation panics with following error.
To Reproduce
mise i java@prefix:sapmachine-17
Expected behavior
Installation is successful with latest version matching the prefix.
Additional context
Most likely related to following code snipped used in the mentioned plugins.
 fn install_version_impl(&self, ctx: &InstallContext) -> Result<()> {
 ...
    assert!(matches!(
        &ctx.tv.request,
        ToolVersionRequest::Version { .. }
    ));
...

@jdx Shouldn't be possible to use scopes via version suffix as well or was this only intended for configuration files?

I am not sure why that check is there. I think maybe I expect that the version was resolved at that point or something so if it was ToolVersionRequest::Prefix in the past it should at this point be resolved to ToolVersionRequest::Version. I would try adding ToolVersionRequest::Prefix and seeing if everything works, if so we can add that to the assertion

Other core plugins like go, node, python don't have this assertion. Adding Prefix to the assertion works, question is if other scopes also would need to be supported and removal of the assertion would make more sense.


Stuck busy looping

First please take whit a grain of salt, and assume I might be doing something wrong, or the data is corrupted.
I am doing long running rdedup stressttest I caught it hanged, using 100% of one of the CPUs. I was able to attach gdb and turns out it looping on a drop/finish in this code of mine:
  8     fn decompress(&self, buf: SGData) -> io::Result<SGData> {
  7         let mut decompressor =
  6             bzip2::write::BzDecoder::new(Vec::with_capacity(buf.len()));
  5
  4         for sg_part in buf.as_parts() {
  3             decompressor.write_all(&sg_part)?;
  2         }
  1         Ok(SGData::from_single(decompressor.finish()?))
114     }

Note that 114 might be off, as I might have added/removed a line or two. I'm guessing it is finish() call.
#0  bzip2::write::{{impl}}::write<collections::vec::Vec<u8>> (self=0x7fff68365488, data=...)
    at /home/dpc/.cargo/registry/src/github.com-1ecc6299db9ec823/bzip2-0.3.2/src/write.rs:254
#1  0x00005567a4127d20 in core::ptr::drop_in_place<bzip2::write::BzDecoder<collections::vec::Vec<u8>>> ()
    at /home/dpc/.cargo/registry/src/github.com-1ecc6299db9ec823/bzip2-0.3.2/src/write.rs:219
#2  0x00005567a4127952 in rdedup_lib::compression::{{impl}}::decompress (self=<optimized out>, buf=...)
    at lib/src/compression.rs:114
#3  0x00005567a4153913 in rdedup_lib::{{impl}}::read_chunk_into (self=0x7fff6837fd38, digest=...,
    chunk_type=<optimized out>, data_type=rdedup_lib::DataType::Data, writer=...) at lib/src/lib.rs:352
#4  0x00005567a4151807 in rdedup_lib::ReadContext::read_recursively (self=0x7fff68367aa0, accessor=..., digest=...)
    at lib/src/lib.rs:262

If I keep doing next in my gdb, I get:
core::ptr::drop_in_place<bzip2::write::BzDecoder<collections::vec::Vec<u8>>> ()
    at /home/dpc/lab/rust/rdedup/lib/<try macros>:3
3       /home/dpc/lab/rust/rdedup/lib/<try macros>: No such file or directory.
(gdb)
218             while !self.done {
(gdb)
219                 try!(self.write(&[]));
(gdb)
3       /home/dpc/lab/rust/rdedup/lib/<try macros>: No such file or directory.
(gdb)
218             while !self.done {
(gdb)
219                 try!(self.write(&[]));
(gdb) n
3       /home/dpc/lab/rust/rdedup/lib/<try macros>: No such file or directory.
(gdb)
218             while !self.done {
(gdb)
219                 try!(self.write(&[]));
(gdb)
3       /home/dpc/lab/rust/rdedup/lib/<try macros>: No such file or directory.
(gdb)
218             while !self.done {
(gdb)
219                 try!(self.write(&[]));
(gdb)
3       /home/dpc/lab/rust/rdedup/lib/<try macros>: No such file or directory.

If I do step, I get:
3       /home/dpc/lab/rust/rdedup/lib/<try macros>: No such file or directory.
(gdb) s
218             while !self.done {
(gdb)
219                 try!(self.write(&[]));
(gdb)
bzip2::write::{{impl}}::write<collections::vec::Vec<u8>> (self=0x7fff68365488, data=...)
    at /home/dpc/.cargo/registry/src/github.com-1ecc6299db9ec823/bzip2-0.3.2/src/write.rs:254
254             if self.done {
(gdb)
258                 try!(self.dump());
(gdb)
258                 try!(self.dump());
(gdb)
bzip2::write::BzDecoder<collections::vec::Vec<u8>>::dump<collections::vec::Vec<u8>> (self=0x7fff68365488)
    at /home/dpc/.cargo/registry/src/github.com-1ecc6299db9ec823/bzip2-0.3.2/src/write.rs:200
200             while self.buf.len() > 0 {
(gdb)
204             Ok(())
(gdb)
205         }
(gdb)
bzip2::write::{{impl}}::write<collections::vec::Vec<u8>> (self=0x7fff68365488, data=...)
    at /home/dpc/lab/rust/rdedup/lib/<try macros>:3
3       /home/dpc/lab/rust/rdedup/lib/<try macros>: No such file or directory.
(gdb)
267             (self.raw.total_in_lo32 as u64) |
(gdb)
268             ((self.raw.total_in_hi32 as u64) << 32)
(gdb)
261                 let res = self.data.decompress_vec(data, &mut self.buf);
(gdb)
bzip2::mem::Decompress::decompress_vec (self=0x7fff68365488, input=..., output=0x7fff683654a8)
    at /home/dpc/.cargo/registry/src/github.com-1ecc6299db9ec823/bzip2-0.3.2/src/mem.rs:243
243                 let before = self.total_out();
(gdb)
bzip2::mem::Decompress::total_out (self=<optimized out>)
    at /home/dpc/.cargo/registry/src/github.com-1ecc6299db9ec823/bzip2-0.3.2/src/mem.rs:243
243                 let before = self.total_out();
(gdb)
bzip2::mem::Stream<bzip2::mem::DirDecompress>::total_out<bzip2::mem::DirDecompress> (self=<optimized out>)
    at /home/dpc/.cargo/registry/src/github.com-1ecc6299db9ec823/bzip2-0.3.2/src/mem.rs:272
272             (self.raw.total_out_lo32 as u64) |
(gdb)
273             ((self.raw.total_out_hi32 as u64) << 32)
(gdb)
bzip2::mem::Decompress::decompress_vec (self=0x7fff68365488, input=..., output=0x7fff683654a8)
    at /home/dpc/.cargo/registry/src/github.com-1ecc6299db9ec823/bzip2-0.3.2/src/mem.rs:245
245                     let ptr = output.as_mut_ptr().offset(len as isize);
(gdb)
core::ptr::{{impl}}::offset<u8> (count=<optimized out>, self=<optimized out>) at /checkout/src/libcore/ptr.rs:617
617     /checkout/src/libcore/ptr.rs: No such file or directory.
(gdb)
(...)

Nah I'd definitely believe it, this has happened before!
Would you happen to have the data oh hand that caused this to reproduce?

Unfortunately, my tests are a higher level: generates random data, deduplicates it in rdedup, recovers it back, and compares the checksum. Since I have no good tracing and/or logging facilities, I don't have any nice, easy to digest information yet. I plan to add it but it will take a while.
If you can tell me what exactly would help (I am guessing the exact sequence of input pieces that lead to it), I could try to quick-hack something like that.

Also, I might be wrong about it, but I've seen data corruption problems before (that's what I am actually looking for), and since identifying this problem, I've blacklisted bzip2 compression from my suite, and so far they are gone, so there might be more of them.
The only time where I was able to catch any issues were with the following rdedup settings:
target/release/rdedup init --chunking gear --chunk-size 2K --compression bzip2 --encryption curve25519 --hashing blake2b --nesting 0
target/release/rdedup init --chunking fastcdc --chunk-size 2K --compression bzip2 --encryption none --hashing blake2b --nesting 0

So I know that encryption is irrelevant, and I'm quite sure the problem reapears only (or with much higher probability) if I deal with a lot of small pieces of data (2K is the smallest chunking setting from my suite). It might take hours to catch one like these with rdedup, and it could be a combination of problems etc. so nothing here is conclusive, but I would advice to write a simple stresstester, that feeds bzip encoder small, random pieces of data in a stream, and then decrypts them back (again, in small random pieces if need be). This is pretty much what rdedup is doing, just with a lot of overhead of other pieces of deduplication.
I'll definitely keep you in loop when looking into it, but I will be busy in next two weeks.

The script has been running for 5 hours now, with bzip2 disabled and all issues are gone, so I'm quite confident there are some other issues lurking in there.

I think it's a great opportunity for fuzzers and quickcheck. :)

I have finally got back from vacation and had some time to dive into it. The problem was entirely in rdedup and it must have been just some bad luck that bzip2 was failing. After fixing actual rdedup issues, I wasn't able to catch any issues, even witth bzip2, so I'm closing.

Thanks for investigating!

So, I suspect, the busy looping part might be a problem with bzip2. The bugs I've fixed in rdedup were about data corruption, but I don't see how could they cause bzip2 to loop like described.

Oh oops! Is there still sometimes a loop in bzip2?

I wasn't able to catch it but the original issue I reported was real so I am guessing it might still be there. If I catch it again I'll reopen.

Ok, thanks!


Migrate from track1 to track2

fixes: https://github.com/Azure/azure-sdk-for-go/issues/18182

Please take a look at the code to fix the issue from the screenshot. We need a clean execution here and it looks like the same protocol is being passed twice after the changes.

I also failed to create on azure portal，


gs1.1 libp2p param

Replace gossipsub peerId, registrar, connectionManager, etc. params with a Libp2p object.
Fix all tests to suit

minor test cleanup where needed
ws connections in node, relayed connections in browser see https://github.com/ChainSafe/js-libp2p-gossipsub/pull/99#issuecomment-652511107

Codecov Report

Merging #101 into gsv1.1 will increase coverage by 9.84%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           gsv1.1     #101      +/-   ##
==========================================
+ Coverage   81.81%   91.66%   +9.84%
==========================================
  Files           1        2       +1
  Lines          11       24      +13
==========================================
+ Hits            9       22      +13
  Misses          2        2

Impacted Files
Coverage Δ

test/utils/index.js
81.81% <ø> (ø)

.aegir.js
100.00% <100.00%> (ø)

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 e60483d...4beb8b2. Read the comment docs.


sudo pip command not found ECS base image

Hi,
I've been trying to get this module running on a base AMI of the ECS image with packer:
source_ami_filter:
      filters:
        virtualization-type: "hvm"
        architecture: "x86_64"
        name: "*-amazon-ecs-optimized"
        root-device-type: "ebs"
      owners: ["amazon"]
      most_recent: true

However when i get to install-consul it errors with sudo pip command not found
To overcome that I installed pip -  - "sudo easy_install pip"
I can run pip -V and I get a version out but sudo pip -V and I get sudo pip command not found
I can't tell if this an issue with the base ECS image or if the code in this module shouldn't call sudo on pip
Any pointers gratefully received 😄
UPDATE - I have this working but the workaround is a bit ugly - - "sudo ln -s /usr/local/bin/pip /usr/bin/pip"

Ah, yes, the code assumes pip is available on root's PATH, but that won't be the case for all distros, as it may be installed in /usr/local/bin, which is no on root's PATH. Perhaps a workaround is to run pip_path=$(command -v pip) (without sudo) and then sudo "$pip_path" install aws?

Even modifying the $PATH didn't fix it. I could have altered the sudoers secure property but the symlink fixed it but not that elegant

Modifying PATH is not something this script should do, but looking up the absolute path to pip and using that should work fine.


Access-Control-Allow-Origin -- issue

I tried to hit your api from my application, but its through the error:
Failed to load http://horoscope-api.herokuapp.com/horoscope/today/aries: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'localhost//4200' (OR website address) is therefore not allowed access.
I think you need to add CROS file to your root folder and  set 'Access-Control-Allow-Origin' : true;

+1

This isn't sorted out till now. :(


Restarting program after patching

Julian, at first thank you very much for helping me with this problem. (It is working for 3 hour now ;-) I bought a used MacBook pro and I think the seller knew about the problem, but didn't tell me ...
What I noticed ... after patching the MacBook Pro and restarting your program, there was no message appearing that this MacBook Pro IS already patched.
Is that right?
Best wishes, Joern

Yup everything is normal, the program does not check if the macbook is already patched or not.


Prepare the repo to support tsp-client

tsp-client is the way we generated Azure TypeSpec SDK. It requires an initial setup on the repo, which is quite simple, but need knowledge from Go people :)
You just need to have this file, but with the go dependency:
https://github.com/Azure/azure-sdk-for-python/blob/main/eng/emitter-package.json
You may also commit the lock file as necessary:
https://github.com/Azure/azure-sdk-for-python/blob/main/eng/emitter-package-lock.json

We already have the above files.
https://github.com/Azure/azure-sdk-for-go/blob/main/eng/emitter-package.json
https://github.com/Azure/azure-sdk-for-go/blob/main/eng/emitter-package-lock.json
Please let me know if there are any other ask to support tsp-client.


FUSETOOLS2-1428 - Activate JMX over http

by simply adding Jolokia 2 as a dependency
Writing a test is complicated, might be easier to provide an end to end test at VS code level

(internal video for Red Hat available here https://redhat-internal.slack.com/archives/C06CYJGDKH6/p1721748800328089?thread_ts=1720184285.902599&cid=C06CYJGDKH6 do not worry, i plan to publish a nicer video when everything is merged and published)


Implement functions which are necessary to create a simple calculator

Implement

AddExpr
MulExpr
UpArrowExpr
BracketExpr
TextExpr
ParameterExpr
ReadStatement

And also implement a compile system.
After this PR is merged, you can create an executable from Kaprino.


luci-app-freifunk-policyrouting: service tab does not appear

The tab for configuration does not appear in services tab.

freifunk packages have been removed from this repo. Please raise an issue here: https://github.com/freifunk/openwrt-packages


Add cell_id field to metadata of app crashed events

[#155871993]
Signed-off-by: Edwin Xie<EMAIL_ADDRESS>Thanks for contributing to cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

A short explanation of the proposed change:

As a CF app developer, I expect to see the Diego cell ID in the app crash event metadata so that I can more easily report the cell on which an app instance crashed

An explanation of the use cases your change solves

see this story in Diego's backlog: https://www.pivotaltracker.com/story/show/155871993

[ x] I have reviewed the contributing guide

[ x] I have viewed, signed, and submitted the Contributor License Agreement

[x ] I have made this pull request to the master branch

[x ] I have run all the unit tests using bundle exec rake

[ x] I have run CF Acceptance Tests

Thank you! 🐐 ⛰ 🥇


Fix for jspm

#82
jspm load non js files with special loaders. For sass, it uses plugin-sass.
These special loaders are invoked by the presence of the ! character, like this : import './style.scss!';.
Without this, jspm expects a simple .js file, therefore, it raises exceptions when it fails to import style.js.
To resolve this issue, I simply added a style.js file that load the corresponding scss file in each components directory.
As the files holds the scss extension, jspm would look for a scss module, not a sass module.
To correct this, sass needs to be specified at the end of the import, like this :
import './style.scss!sass';

These files are then compiled and put directly into the lib folder.
And apparently it works :)

Does this work for webpack as well? I think there should be a way to set a context. Via env variable or whatever to add this customizations. We can think further about it during the weekend, I'm going to PR something that makes tooltox potentially usable without webpack

Closin! I think this is not going to be needed with the new style approach, thanks :)

Hi, I'm just wondering if react-toolbox is finally working with jspm? (I've tried quickly and is not working). Many thanks!

It should!


lastFetchTime property's 'readwrite' attribute needs to be marked 'readOnly'

Tracked internally in b/144366943


Add dnf support for rsync install

PR for issue #1051
Add support for dnf package manager to install rsync on guest

Downside would be the removal of auto-healing (e.g. if rsync is removed by accident in the test). So if we value that (and expect to happen) then try/except would be better approach

This actually happens during the installability test of the rsync component and lead to the introduction of the _check_rsync() method. So I believe we need to cover this one anyway not to regress.

Thanks much for looking into this! Looks good in general. On the other hand I think we are already doing too many extra checks on the guest and this change is adding one more command before each guest.push() or guest.pull().
I was thinking about improving this by trying rsync first and only if it fails checking the guest. It could look like this:
try:
    self.run(["rsync", ...])
except tmt.utils.RunError:
    try:
        self._check_rsync()
        self.run(["rsync", ...])
    except tmt.utils.RunError:
        self.fail(...)

@lukaszachy, @FrNecas what do you think?

Was trying to make it reusable as possible :)
I have the same thoughts as @lukaszachy said,

We've discussed the approach below on the hacking session today:
try:
    self.run(["rsync", ...])
except tmt.utils.RunError:
    try:
        self._check_rsync()
        self.run(["rsync", ...])
    except tmt.utils.RunError:
        self.fail(...)

The outcome was that this seems to be the best way how to address the issue for now. In the future we might enhance the final part of the provision step as well.
@BeeGrech, are you willing to adjust your changes based on this? The _check_rsync() method can nicely handle the new check for dnf versus yum as well.

So just to confirm I am understanding correctly, modify the push and pull methods, with this Try Except clause, then fold the new pkg_mgr method into _check_rsync method?

Yes, exactly. The method could start with a rsync --version check and bail out immediately if rsync is already installed (which would mean a real synchronization failure appeared) so that the original error could be re-reraised from push() or pull().

We've renamed the default branch to main. I've rebased on the latest main which should fix the problem.

/packit test


[enh] Implement new resource for Cargo Package

Implement a new plugin to support cargo packages
This PR allows using Cargo Package as a source and a condition.
By default, it will use crates.io index repository, but a private repository can be used as well
Test
To test this pull request, you can run the following commands:
cd pkg/plugins/resources/cargopackage
go test

Additional Information
Tradeoff
Potential improvement

Waow nice, I am glad to see you manage to create a new resource without reaching out for help :D
I'll find some time later today or this week to review your PR

No problem, I've remove all the git handling, I had not understood the scm concept fully. I'm adapting the description as well to remove the tradeoff

Coud I also ask you to add an manifest to e2e/updatecli.d/success.d/crate.yaml?
The manifest cannot return any error or warning message as define on https://github.com/updatecli/updatecli/blob/main/e2e/venom.d/test_diff.yaml

I think it's missing the part where you dowload the index from the crate.io repository.
I just tested the following manifest
sources:
  test:
    kind: cargopackage
    spec:
      package: syn

and I get
SOURCES
=======

test1
-----
ERROR: something went wrong while opening the package file "open /home/olblak/Projects/Updatecli/updatecli/3/s/syn: no such file or directory"
ERROR: ✗ open /home/olblak/Projects/Updatecli/updatecli/3/s/syn: no such file or directory
Pipeline "CRATE.YAML" failed
Skipping due to:
	sources stage:	"open /home/olblak/Projects/Updatecli/updatecli/3/s/syn: no such file or directory"

This makes me think you may want to download the repository in the temporary directory created by Updatecli
https://github.com/updatecli/updatecli/blob/main/pkg/core/tmp/main.go#L25

You mean that we should default to use and clone the crates.io repository, if no IndexUrl or scmId is defined?

Excepted a few minor remark, I think this pullrequest is really great.
Once the default "clone" operation uses the Updatecli scm snippet so it clones the repository in the Updatecli temporary location and we ensure that we do not modify the spec data, I think it can be merged
Not in this pull request but a follow up effort would be to improve the git clone operation . I think some benchmarking is needed to speed up. I never realize how slow it could be for large repositories

Super excited to see this pullrequest.
I did some testing and it works great.
I am waiting for the tests to pass before merging


Fixes and Enhancements Apr 8, 2016

fix: options not passed to  L.Autocomplete on instantiation
enhancement: remove population sort from search to improve performance
enhancement: show all search criteria in suggestion list
enhancement: 	increase max result count to 15

Thanks Tim,
This looks good but can you put the screenshots back?  Removing them now won't make the git clone any smaller, and I like to have a history of what the site looked like.
-braddock


Modernize facilities related to hpx::sync

@hkaiser could you please summarize the changes? Is it ::type to _t, ::value to _v, typedef to using, and a few new helper type traits? Something else?

@hkaiser could you please summarize the changes? Is it ::type to _t, ::value to _v, typedef to using, and a few new helper type traits? Something else?

Yes, this is mostly _t/_v changes and adding missing implementations for those. I have also replaced some explicitly calculated return types with a simple auto.

@hkaiser I've only formatted a file. Was this otherwise ready to go now?

@hkaiser I've only formatted a file. Was this otherwise ready to go now?

Yes, thanks!

Thanks for taking care of this @msimberg !


OpenAPI 3 never handling seems incorrect

Right now never properties are omitted, but should probably be emitted as we do in JSON Schema: { not: { } }.

is this widely supported in OpenAPI3 processors?


Pixel 5 Spoofing now breaks safetynet

Spoofing your device to a Pixel 5 to enable special functions such as, Adaptive Sound, New Generation Assistant, etc. Makes ctsProfile check fail, I have tested this in a CustomROM, DotOS, spoofing to my correct fingerprint made it pass again, but features are now shorter, tested with Pixelify 1.71, I will try it with 1.7 and see if it happens.

Safety-net with Pixelify 1.7 is working correctly!!! I hope this helps somehow

!

Safety-net with Pixelify 1.7 is working correctly!!! I hope this helps somehow  !

Working here, well my device is very old. Safetynet always pass. I will what I can doo


Update README badges

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Thanks @ian4hu . LGTM.


Method on() doesn't override connection name for relations.

Laravel Version: 5.6.26
PHP Version: 7.2.3
Database Driver & Version: doesn't matter.

Description:
Method on() doesn't override a connection name for relations in case if related model has explicitly defined connection.
According to this comment and a comment of on() it looks like connection should be inherited from parent model if on() is used.
So, I'm ready to prepare a patch but I need to understand what's the most correct way — either prohibit retrieving relations in case of using method on() or override connection name of all relations (introduce property protected $forcedConnection and use it as an overriding connection name). I'd prefer to follow on()'s description and implement second option. Waiting for your opinions.
This issue was met in a real application using 3 connections and explicit connection definition helps a lot, but overriding ability of on() would introduce much more flexibility and prevent creation «base» models with absence of protected $connection.
Steps To Reproduce:
Create 2 db connections: conn1 and conn2. Create two models related each other: Model1 and Model2. Define protected $connection = 'conn1'; for both of them. Create relation of any type. Create at least one entry for Model1 and save related Model2. Try next code:
Model::on('conn2')->with('relationName')->get();

@SCIF The comment on the method on() might be a little incomplete as I believe it was written after  the PR #16103.  It only uses the parent connection if the related model has not specified it.
There's also a few solutions or workarounds offered in #9355, where the exact issue is discussed but overall it seems that it's supposed to work as it is.

Hi @mihkelallorg, method  on() is useless if the same result could be reached by using setConnection(). Using null for related models solves an issue but it's very ugly to have «base» models with empty $connection to be able to request related models. And one more time — since null is specified for related model — on() is useless. Thus, description seems pretty logically for me.

This is indeed not possible with the current on method. I think we should update the comment in the on method to something like as well as being set on each relationship we retrieve which does not have an explicit connection defined. to make things more clear.
I believe we could provide some way to indeed some way to force the connection of the parent model. If this can be solved with a non-breaking change this could target the current release, otherwise the next release is more appropriate.

My PR on improving the comment has been merged.


Spike data support for OpenEphysBinaryRawIO

Hey,
I was looking for a neo IO class which supports a) lazy loading / memory efficient loading, and b) reading AND writing neo blocks. The openephysbinaryrawio class comes close, but is still missing support for spiking data (according to this line), which I definitely need. Is there any existing progress on this front? Would this issue count as a "user request" for the feature? 😄

Noice, thanks Julia! I'll test and expand on your fork when I find some time.

Hey @JuliaSprenger and @samuelgarcia, I have been neck deep in experiments that leave me with barely any time to code. This will be the case at least until May, so that's probably the earliest that I can make any significant contributions on this front :(


Enhancement: Update teams team add command to use Microsoft Graph v1.0 API

The teams team add command currently uses the beta version of the Microsoft Graph API, as the teams endpoint is now available in v1.0 API we should update the command accordingly.
Create team - https://docs.microsoft.com/en-us/graph/api/team-post?view=graph-rest-1.0&tabs=http

Can I work on this?

Absolutely, all yours @nanddeepn thanks for the help 👍🏻


oxigraph: 0.3.11 -> 0.3.13

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/oxigraph/oxigraph/releases.
meta.description for oxigraph is: SPARQL graph database
meta.homepage for oxigraph is: https://github.com/oxigraph/oxigraph
Updates performed

Rust version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.3.13 with grep in /nix/store/yhs2lqwx4gvadr9fdb3b486gly8vrhjx-oxigraph-0.3.13
found 0.3.13 in filename of file in /nix/store/yhs2lqwx4gvadr9fdb3b486gly8vrhjx-oxigraph-0.3.13

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
oxigraph

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/yhs2lqwx4gvadr9fdb3b486gly8vrhjx-oxigraph-0.3.13 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A oxigraph https://github.com/r-ryantm/nixpkgs/archive/0693400844080344d2bdcf911d3d086db249b416.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/yhs2lqwx4gvadr9fdb3b486gly8vrhjx-oxigraph-0.3.13
ls -la /nix/store/yhs2lqwx4gvadr9fdb3b486gly8vrhjx-oxigraph-0.3.13/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

oxigraph

Maintainer pings
cc @astro for testing.

Result of nixpkgs-review pr 219014 run on aarch64-darwin 1

1 package built:

oxigraph


Add option for showing the distance.

Formatting idea: X x Y (D)
cc @letterror

A quick idea on how to implement:
subscriber.py

add extensionKeyStub + "showDistance" : False to the defaults dict.
make the measurePoints function return (width, height, distance).
the distance should be None if > 2 points are selected.
in getPersistentPointMeasurements trim linkedPointMeasurements to linkedPointMeasurements[:2] so that the distance doesn't break the existing code.
make a similar changes in hudAddNamedValueCallback and findSelectionNames.
make a new formatWidthHeightDistanceString from formatWidthHeightString. it should only use the distance if it is not None.
use the new function in updateText if the user pref is to show distances.

settingsWindow.py
This should be pretty straightforward.

I think #38 is related to this.


Simply adding BSSID when listing nearby networks.

This information is useful in some case, like mine.

Looks good! Thanks for the request.


Run(): don't include any default environment variables

When copying environment variables from the image's configuration to the spec that we'll pass to the runtime, clear out any defaults that the generate package might be supplying.  Currently, that's "$TERM".

Change LGTM, but unhappy test results.

bot, retest this please

Added a variation on how exec.LookPath() resolves commands to do so using the PATH we're going to set and the rootfs that we've just mounted.

bot, retest this please

LGTM
@TomSweeneyRedHat @umohnani8 @giuseppe @pixdrift PTAL

@rh-atomic-bot r+


Go-style Methods/Function Receivers

edited
We have two options.
func (thing *Thing) do() {}
OR
func (*Thing) do() {}
The difference is, in the first you define the receiver name yourself. In the second, the receiver name is automatically defined as self.

How do we do static methods in things with this syntax?

@kiljacken No idea.

What about this:
Method
func (thing *Thing) do() {}
Static method
func (Thing) do() {}
It's not great, but the best I can think of.

:+1:

@MovingtoMars These are done now I think? And me and Liam fixed the bug that stopped mac-ark from working.

Just a few more things, and static methods, which I'll do later today.

Okay nice

Ok, just static methods to go.
Which syntax should we use:
Thing::dostatic()
or
Thing.dostatic()

I like :: more for static methods.

:+1: on :: for static

@MovingtoMars updates on this??

Completed.

@MovingtoMars Brillaint, gj :+1:


Remove global variables

This PR removes the remaining global variables. The standalone version works now, but I still need to work on the CMSSW side. I'll mark it as ready for review once everything works and there's a companion PR on the CMSSW repo.

This PR will also remove the sizes defined in Constants.h right? Now that these are no longer globals. https://github.com/SegmentLinking/TrackLooper/issues/380

This PR will also remove the sizes defined in Constants.h right? Now that these are no longer globals. https://github.com/SegmentLinking/TrackLooper/issues/380

Oh yeah, I wasn't planning on it, but we should do that on this PR too

Oh yeah, I wasn't planning on it, but we should do that on this PR too

That turned out to be much more convoluted than I anticipated. I'll try again once this PR is more complete, but I might also defer it to a future PR.

I think it should all be working now. I'll test it with the CI while I check if there's things that could be cleaned up a bit. Also, I'll check to see how much it needs to be changed to fix the issue @GNiendorf mentioned.
/run standalone
/run cmssw 26

After thinking about it a bit more, I think that a way to make this cleaner is to have the load functions construct the shared pointers instead of only filling them, which would make #380 much easier to address. Also, I should probably move the LSTESData.h header that I added in https://github.com/SegmentLinking/cmssw/pull/26 to here so we can pass around all this data more cleanly. I'll ask for opinions during the meeting today and I'll have it ready within the next couple of days.

This is PR is close to being ready for review. I did end up fixing #380, but it needed quite a few changes. I made things as const as possible. Also, I found out the out of bounds issue I mentioned above, which we should look into. I'll run the CI to make sure that things look fine. I still have to work on the CMSSW side.
/run standalone

Thanks @GNiendorf for pointing out that dxdy_slope_ was a map, not a vector.
/run standalone

a few classes and methods will now appear as identical visible symbols in all sdl_ library backend builds.
Is there some first principles justification that this is safe?

For now, should we keep using templated classes even for things that don't use any alpaka buffers or should we figure out how to move them into a non-alpaka part now? As far as I can tell, CMSSW only loads one library at a time, so there probably wouldn't be symbol conflicts, but I'm not sure if in more general scenarios there could be issues.

I think for now it doesn't matter if we have potential symbol conflicts. When we move to CMSSW (hopefully pretty soon) it will be a lot easier to separate non-alpaka parts by moving them out of the alpaka directory.

/run standalone
@ariostas will the previous run cmssw actually complete or does the request/job get killed by the PR update?

will the previous run cmssw actually complete or does the request/job get killed by the PR update?

It will complete, but let's run a new one to make sure that the new changes don't break anything.
/run cmssw 26

Here is a timing comparison:

pLS column changed from 543 to 498 (a similar reduction as in a test on May 14 , but not on May 17); I'm curious if a rerun (after the more technical const update) will confirm the reduction.

There seems to be some issue with CUDA. I'll look into it.

I moved some functions back into an unnamed namespace. I didn't realize that there was a reason for it. Now it also works with CUDA.
I ran a timing comparison on cgpu-1, but the pLS time is too small to notice a difference.


Technical Debt

There is some very messy python here at the moment. Needs cleaning.

This includes removing side-effects from functions and adding some unit-tests for them

Agreed!


[MM-19756] Remove role='presentation' for post timestamps

Summary

This PR removes the role='presentation' from post timestamp thus stopping errors raised by axe.
Ticket Link

Resolves https://github.com/mattermost/mattermost-server/issues/13009

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-4570.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Test server destroyed

/update-branch

Thanks for the PR @bolariin :+1:


Can't find project

What do I do when Zapier doesn't see any projects, but it does see repositories and branches?  Where would I look to find the GUID of my project to manually enter it?

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

ID: 229dda3c-d750-fc3a-e626-e0695d47e884
Version Independent ID: 0367b931-2282-0a33-6f4c-6f941c3efbf3
Content: Zapier with VSTS
Content Source: docs/service-hooks/services/zapier.md
Service: unspecified
GitHub Login: @erickson-doug
Microsoft Alias: elbatk

Assigning this to the product team to help you try to get this sorted out @magius96

Hi @magius96 , I'm going to close this as a docs issue and suggest that you post to the Developer Community which is actively monitored by the product team. Best of luck!


Multiple NIC support

/kind feature
What this PR does / why we need it:
This PR adds support for attaching multiple network interfaces to both AzureMachines as well as AzureMachinePools (VMSS), it also allows to specify one or more IPConfigs for those interfaces to pre-warm them with multiple IPs on each interface.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #2327
Special notes for your reviewer:

[x ] squashed commits
[x ] includes documentation
[ x] adds unit tests

Release note:
Adds a new optional field for a slice of NetworkInterfaces for AzureMachines and AzureMachinePools to specify a list of NetworkInterfaces, the SubnetName each should be in, and a slice of optional IPConfigs for each interface if desire.

Example below create a machine template with two interfaces in different subnets, and five IPs on the node-subnet interface:

apiVersion: infrastructure.cluster.x-k8s.io/v1beta1
kind: AzureMachineTemplate
metadata:
  name: capi-quickstart-control-plane
  namespace: default
spec:
  template:
    spec:
       networkInterfaces:
           - subnetName: control-plane-subnet
             acceleratedNetworking: false
           - subnetName: node-subnet
             acceleratedNetworking: true
             ipConfigs:
              - {}
              - {}
              - {}
              - {}
              - {}
      dataDisks:
      - diskSizeGB: 256
        lun: 0
        nameSuffix: etcddisk
      osDisk:
        diskSizeGB: 128
        osType: Linux
      sshPublicKey: ""
      vmSize: Standard_D2s_v3

This also needs a make generate-manifests run and committed. There are changes to the azuremachinepools CRD yaml that need to be generated.
diff --git a/config/crd/bases/infrastructure.cluster.x-k8s.io_azuremachinepools.yaml b/config/crd/bases/infrastructure.cluster.x-k8s.io_azuremachinepools.yaml
index a206f603..0b5229f1 100644
--- a/config/crd/bases/infrastructure.cluster.x-k8s.io_azuremachinepools.yaml
+++ b/config/crd/bases/infrastructure.cluster.x-k8s.io_azuremachinepools.yaml
@@ -1647,11 +1647,16 @@ spec:
                     description: Network Interfaces to attach to the to a virtual
                       machine
                     items:
+                      description: Network Interfaces to attach to each VM
                       properties:
                         acceleratedNetworking:
                           type: boolean
+                        id:
+                          type: string
                         ipConfigs:
                           items:
+                            description: IP Configuration defines options to confiure
+                              a network interface
                             properties:
                               privateIP:
                                 type: string


[Awake] idea: add date/time as method to end

Provide a description of the new feature / enhancement
@dend Splitting of the idea as mentioned in this https://github.com/microsoft/PowerToys/issues/11148#issuecomment-839908997 because it's out of scope right now.
Currently there are two settings for "Keep awake":

until manually disabled
until timer has expired

and I think it would be nice to also have

until time/date

(suggest to use a  <DatePicker> and <TimePicker>)

As per #12154 I would like to see a start/end as well, but even better would be a scheduling system.  I'd like to be able to say 'enable Awake  from XX:XX to YY:YY' with an optional date or recurrence option like 'every [day|M|T|W|R|F|S|S]', like setting a meeting.

Maybe schedule can be done via Task Schedular?

The task scheduler doesn't have an option to terminate at a specific time,
only to kill the process after X time so I'd have to create a second task
to kill it at the appropriate time. Still, I've set up a task to see what
happens.
Cheers,
Ben
On Sat, Aug 7, 2021 at 4:15 AM Jay @.***> wrote:

Maybe schedule can be done via Task Schedular?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/11233#issuecomment-894635125,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMVY6R5WBAW6RGLTPHX75BTT3UBVFANCNFSM44Y6K5WA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Awake did start (as expected) using the Task Scheduler.  Left this open on my screen:

which provided the insight I should be able to add --time-limit to the Action in order to have it run only for the duration I'd like it to.  This does provide a workable solution for me, though I don't know what percentage of users would a) want this b) know how to do it through the task scheduler.

Maybe the developers (like @dend) can work this out. A start with task scheduler and an end with an internal time-keeping function?

Yeah, I've added the --time-limit to my scheduled task. Aside from the cmd window this should meet my specific needs, though not average user friendly.
Your original request is easier to implement since it can be accomplished with a date/time entry and just calculating the seconds between start time and desired end time and passing that to the existing --time-limit. My scheduler request is far more complicated.

I see that this issue is being listed as the original for a handful of other requests that specifically ask for a scheduled 'enable' functionality.
I think this is either incorrect, or the title of this issue needs to change.
This issue started as looking for a date/time picker as another option for the 'end' event. But this still requires a user to manually initiate the start event.
True hands-off scheduling was only raised as a comment on this issue, and is a separate work item in my opinion.

Here's my settings. I'm not 100% sure they're working the way I want though.  I'll watch and see. The terminal window isn't there anymore, Awake is running, the History shows Task Scheduler did run it.  But in the system tray Awake says 'Keep screen on' 'Indefinitely'.

I'll try to remember to make sure Awake is disabled tonight and see what state it's in tomorrow morning.  No promises. 🤞

Yeah. I think it is starting but it's ignoring the options I'm passing in b/c it's set to indefinite when I log in.

Currently, I use the "Caffeine" tool to keep my PC awake. If Awake had a shortcut key and the ability to set a time limit on how long to keep it active, I will switch over to using Awake.

This is in #24183.


Do not create namespace in helm chart

Have you tested this works?

Have you tested this works?

It works manually fine. I have not tested with the CLI

/hold
Waiting for CLI changes

/lgtm
/approve


Cant find any downloaded files

progress bar completes every time , but no files in destination folder , even shows the download speed .

This usually happens when you try to save the files in a location that requires administrator permissions. Try right click the program and then run "Run as administrator", or alternatively you can try saving it in a different drive (like, instead of saving it in drive C save it drive D)
Let me know if the issue continues after that


💬 RFC: consistent format for recipes

🔖 Need, Context, and Problem Statement
I would like to choose a format for recipes so we can edit, lint, and view them consistently.
Some requirements from me (open to more)

template must render the ingredient list first, this will make it easier when shopping.
template must render into GItHub Pages

🎉 Proposal
Choose one of the following (see below).  I'm sure more options exist
My current preference is Cooklang, this just seems cool/fun to try.  Rendering seems pretty straight forward in a GitHub Action.

Cooklang – Recipe Markup Language

〽 Alternatives

Paprika
DevCookbook
Jekyll Recipes
Unbelievable Flavour Recipes

Need a domain to host it also. Hungrynerds?

i assume
T = table
t = tea
???????? Is that a common format idea?

i assume
T = table t = tea ???????? Is that a common format idea?

That's one way.  I've also seen tbsp and tsp
from wikipedia

In recipes, an abbreviation like tbsp. is usually used to refer to a tablespoon, to differentiate it from the smaller teaspoon (tsp.). Some authors additionally capitalize the abbreviation, as Tbsp., while leaving tsp. in lower case, to emphasize that the larger tablespoon, rather than the smaller teaspoon, is wanted. The tablespoon abbreviation is sometimes further abbreviated to Tb. or T.

I don't think it really matters as long as it's consistent?


MSID: 89045 Version: 1 DOI: 2<IP_ADDRESS>3432

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v1/by-publisher/elife/get-by-manuscript-id?manuscript_id=89045)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/89045/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews
Who can help: Editorial team

[ ] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>3432)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/89045/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/89045)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/89045

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/89045

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/89045/v1/89045-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/89045)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

June 19: authors have asked to correct author information in the preprint: waiting for new version

June 20: V1 should use preprint V2: https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>3432v2


Fix Deprecation warning of mongoose.Promise

I noticed deprecation warning below when I run test and app debugging console.

(node:34180) DeprecationWarning: Mongoose: mpromise (mongoose's default promise library) is deprecated, plug in your own promise library instead: http://mongoosejs.com/docs/promises.html

To fix this warning I override mongoose's internal promise by node.js global promise function.

@starcharles Are you using Node.js 4.x?

@sahat No, I'm using v6.3.0

I am not seeing this issue on my machine, but after looking around, seems like many other people are getting the same warning with Mongoose in certain cases. Doesn't hurt to add it. Thanks!


roll back documentation changes

This reverts the changes I made to the README.md as part of a larger PR: https://github.com/RaRe-Technologies/smart_open/pull/237

revert changes according to https://github.com/RaRe-Technologies/smart_open/pull/237#issuecomment-425805309