introduce Accceptor for representing the conversion of incoming I/Os

Currently, the server API takes the stream of incoming connections as a Stream that returns an asynchronous I/O directly. This means that spawning of the task per connection will be done after the preparation of connection was completed (here, the "preparation" of connection is some asynchronous computation for acquiring a value that implements AsyncRead+AsyncWrite from a "raw" connection, such as TLS handshake). If the preparation of establishing a connection takes many time, the process for successor connections cannot be started.
This PR solves this problem by explicitly handling the preparation process within the implementation of SpawnAll and so on. In order to abstract the preparation process, a trait Acceptor is newly introduced.
Alternatives

does not change. the "parallelization"of preparation tasks are emulated by using FuturesUnordered or else.

in this case, preparations are handled within the same task and hence they cannot be scheduled by the background.

replaces the trait bound of incoming stream with I::Item: Future<Item = impl AsyncRead+AsyncWrite>.

it affects existing code which assumes that I::Item: AsyncRead + AsyncWrite.

Compatibility Notes
It includes some breaking changes in the low-level server API in hyper::server::conn.

add type parameters
the associated type <Connection<I, F, E> as Future>::Error will be changed from F::Error to hyper::Error.

I think I get what this is trying to accomplish. Do you have a small example, like how this would be used for TLS?

Here is an example of using TLS with tokio_tls::TlsAcceptor:
extern crate hyper;
extern crate native_tls;
extern crate tokio_tls;

use hyper::rt::Future;

use native_tls::{Identity, TlsAcceptor as NativeTlsAcceptor};
use tokio_tls::TlsAcceptor;

fn main() {
    let der = std::fs::read("./identity.p12").unwrap();
    let cert = Identity::from_pkcs12(&der, "mypass").unwrap();
    let acceptor = NativeTlsAcceptor::builder(cert)
        .build()
        .map(TlsAcceptor::from)
        .unwrap();

    hyper::rt::run(
        hyper::Server::bind(&"<IP_ADDRESS>:3000".parse().unwrap())
            .acceptor(move |io| {
                acceptor.accept(io)
            })
            .serve(|| {
                hyper::service::service_fn_ok(|_| {
                    hyper::Response::new(hyper::Body::from("Hello World!"))
                })
            })
            .map_err(|e| eprintln!("server error: {}", e));
    );
}

tokio-rustls and tokio-openssl also provide the similar signature of method, so they could be used the same procedure.

I'm closing this PR because the proposed method cannot have a way to access the TLS transports after the handshake process is completed.


youtube-dl - iView no longer copies links to clipboard

Just a note that when using youtube-dl this morning, ABC iView no longer copies the link to your clipboard.  This is when a user goes into iView, seeks out the relevant program, clicks the down arrow to reveal "email this" and :"copy link".  Both result in a generic abc link (https://iview.abc.net.au/show/world#) and there is no program specific id'er at the end of the url.  Looks like people at the ABC have too much time on there hands again. :(

youtube-dl>youtube-dl.exe --verbose https://iview.abc.net.au/show/w1a/series/3/video/ZW1410A001
S00Updated youtube-dl to version 2019.02.18.
The batch file cannot be found.
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://iview.abc.net.au/show/w1a/series/3/video/ZW1410A001S00']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2019.02.18
[debug] Python version 3.4.4 (CPython) - Windows-Vista-6.0.6001-SP1
[debug] exe versions: ffmpeg N-70125-g6c9537b
[debug] Proxy map: {}
[debug] Using fake IP <IP_ADDRESS> (AU) as X-Forwarded-For.
[abc.net.au:iview] ZW1410A001S00: Downloading JSON metadata
[abc.net.au:iview] ZW1410A001S00: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https:
//yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl
with the --verbose flag and include its complete output.
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpw73a_6nx\build\youtube_dl\extractor\common.py", lin
e 605, in _request_webpage
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpw73a_6nx\build\youtube_dl\YoutubeDL.py", line 2225,
in urlopen
File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default

worked two days ago

Just making sure we are on the same page here.  There is nothing wrong with the program itself, Downloading from youtube itself is fine.    ABC have changed their iView system, so users can't no longer obtain program links for downloading specific shows.  Just trying to copy a link from iView into notepad shows the same generic link for every episode.

ABC iView no longer copies the link to your clipboard.

no, there is not change on the website.

Okay, thanks.  Just installed firefox, and the links came up okay, so maybe a google chrome thing. :) :)


The semaphore looks racy

Correct me if I'm wrong, but:
The AWS network does not guarantee the dynamodb lock message won't be delayed or reordered.  It could happen after "cleanfromincomplete".
So combining the above non-guarantee with a one-shot cleanup lambda that doesn't lock out future lock requests, that seems very problematic.  The lock could leak, and all progress would be halted.
I see two ways to fix this:

Read the dynamodb stream and launch a cleanup monitor every time the semaphore is locked.   Thus, if a step function somehow manages to lock it again after cleanup finished, it will still be unlocked by a 2nd cleanup.  The monitor could be an asynchronous express workflow (at-least-once execution of each monitor is fine) that waits until the main step function that locked the semaphore finishes.   Although it wouldn't be efficient, each step function could lock and unlock the semaphore multiple times during normal operation.  Multiple monitors would be queued up, but they only talk to dynamodb after they think the main step function is dead, so shouldn't interfere with "normal" step function execution.

Keep a log in dynamodb of the step function IDs that finished, and those ids aren't allowed to lock the semaphore again (perhaps a transaction is needed).  All executions (100%) need cleanup scheduled by eventbridge.  One concern is storage space in dynamodb, eventually you need to purge this log.  However, each step function should be able to lock and unlock the semaphore multiple times during normal operation, since only cleanup locks them out.

We would really like better concurrency control in AWS, but having a built-in semaphore in step functions seems more reliable, faster, simpler, and cost effective.

I don't understand the statement "the dynamodb lock message won't be delayed or reordered". Can you provide a bit more detail so that I can understand?

This sequence of events:

the updateItem message from AcquireLock is sent
Step Function times out
Event Bridge calls cleanfromincomplete
cleanfromincomplete runs and calls getitem/udpateItem

There is no guarantee that 1 is received or observed before 4.

I see. It is technically possible that the DynamoDB Update call issued by Step Functions could fail to get all the way to the DDB table before SFN fails / aborts / times out, sends the Execution Status Change event to Event Bridge, EB triggers the cleanup state machine, and the cleanup state machine reads the DDB record. It's possible, but highly unlikely.
I think the right solution here is a "sweeper" that runs on a schedule and looks for locks recorded in the DDB record without a corresponding running execution. That way, even if a lock does get leaked (for this or for some other unforeseen reason), it will get picked up. And that state machine can do things like notify via SNS that something unexpected happened and that you should look into it. I had considered this when building the example but decided to leave it out for simplicity, but it would be a good idea to add. I'm not sure when I'll be able to get to it, but I will add that.
Thanks!
Justin


Quick Installation

Expected behavior:
Setup continues after entering correct credentials
Actual behavior:
Setup does not continue and does not provide error message. Just green "slider" in the header is flickering
Reproduce steps:

Linux Hosting
Download Setup Wizard, Make sure all requirements are met
enter correct credentials
if you type in wrong credentials you can see that correct error messages are shown

for better explanation please see this video i made for this: https://screencast-o-matic.com/watch/cYh1Xmgh7C
Version:
latest, i cannot find a version number on setup wizard
Additional Information:
Please ask if you need further information

Use issues to report bugs, please post your questions on the forums or better still search the forums for existing solutions.

@sampoyigi sorry, but i dont understand? I posted a bug and even made a video of it? I dont have a question?
If you cannot understand the bug please ask so i can try to explain better

@sampoyigi if i click continue and nothing happens, not even a error message is shown, even if anything is correct, it is a bug in my understanding? Or am i missing something?

@sampoyigi besides that, i tried to post on the forum. Was not working because registration emails are working. I did send an email about that but didnt get an answer get

Also Check: https://forum.tastyigniter.com/d/1479-can-t-finish-the-installation
This guy did post the same Bug on Forum but unfortunately didnt get an answer.

@sampoyigi besides that, i tried to post on the forum. Was not working because registration emails are working. I did send an email about that but didnt get an answer yet

We had issues with mailgun but emails are back to normal, please try registering on the forum and start a discussion if there isn't one already 👍

I can confirm E-Mails are working now. Thank you @sampoyigi
(just to let you know: i found a dead link on tastyigniter.com, shown in picture: tastyigniter.com - support - slack


TS-4555: Make CPP API use a reserved transaction arg.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/602/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/707/ for details.


Failing tests - add here please

Please add failing tests due to timing issues here.
I will try to fix them but not all fails on my fast machine.

NetDaemon.Daemon.Tests.Reactive.RxAppTest.EntityIdsShouldReturnCorrectItems

closing, this seem to be ok now when we refactored the timer tests


fix(entities): fix path from absolute to relative

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[feat] add os shutdown hook

Problem description

Explicit stop is required to properly shutdown the actor system
Failure to explicitly stop the actor system fails to properly shutdown the actor system

Solution description

Implement a coordinated shutdown mechanism to listen to OS signals to properly shutdown the actor system

Alternative solutions

For the meantime always shutdown the actor system properly by implementing a OS signals listener

Additional context
N/A

This is not feasible because syscalls are hooked into the main application where actor system will be running. I will add a note on the readme to let developers handle properly syscalls.


mat: retain type and dimension attributes in encodings

Please take a look.
Closes #373.

PTAL

Failure is real but no longer relevant.

ping

PTAL

They get dismissed by new commit in gonum/gonum, so I need to ask.

I know. I wasn't complaining :) (apologies if it seemed like I was)

No worries.


fix binary released version

This PR:

fixes a typo s/Pypy/PyPi
raises the max version to 3.10 - not sure if there is even higher, but 3.10 is definitely there:

pip install triton -U
Looking in indexes: https://pypi.org/simple, https://pypi.ngc.nvidia.com
Requirement already satisfied: triton in /mnt/nvme0/anaconda3/envs/py310-pt22/lib/python3.10/site-packages (2.2.0)
Collecting triton
  Downloading triton-2.3.0-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (1.4 kB)
Requirement already satisfied: filelock in /mnt/nvme0/anaconda3/envs/py310-pt22/lib/python3.10/site-packages (from triton) (3.13.1)
Downloading triton-2.3.0-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (168.1 MB)
   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 168.1/168.1 MB 2.3 MB/s eta 0:00:00
Installing collected packages: triton
  Attempting uninstall: triton
    Found existing installation: triton 2.2.0
    Uninstalling triton-2.2.0:
      Successfully uninstalled triton-2.2.0
Successfully installed triton-2.3.0

Can you figure out the max PyPi version that's supported, so we don't have to guess at the correct value here?  (I don't know how to do this offhand, but since you're writing the PR I assume you understand more about Python packaging than I do.)

OK, thank you for asking me to investigate this
I guess the confusion came from PyPy which looks like PyPi w/ a typo except there are not the same.
all the different python implementations are nicely listed here https://stackoverflow.com/a/67915291/9201239
And CPython is what most people have:
So to answer your question to tell which PyPi versions are available you go here:
https://pypi.org/project/triton/2.3.0/#files
and then doc is correct 3.8-3.12
For PyPy I can't tell since triton isn't even listed at http://packages.pypy.org/
For posterity, if you need to see which version of python you have, run:
$ python -c "import platform; print(platform.python_implementation())"
CPython

otherwise closing this, apologies for proposing a bogus fix, Justin.

actually, it's 3.7-3.12, and not 3.8-3.12, but I'm not sure what m means CPython 3.7m
which is the last entry at https://pypi.org/project/triton/2.3.0/#files

triton-2.3.0-cp37-cp37m-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (168.1 MB view hashes)
Uploaded Apr 4, 2024 CPython 3.7m manylinux: glibc 2.17+ x86-64


[Discussion] Docs rewrite

Currently, documentation is maintained on a separate branch, gh-pages, and written in Markdown. With all of the type hints that have been incorporated over the last few releases, IDE/LSP's will have better information about what function signatures are available and being used. Python's default way of making documentation available to users is through docstrings.
What I propose is that instead of separately maintaining the documentation, especially the user facing TSC API documentation, separately, that it be maintained through docstrings in the .py files. Then the markdown that is currently being used is converted to reStructuredText. This would open up the ability to use sphinx and its autodoc extension to read those docstrings and produce the web navigable documentation. I believe the docs could still be published to gh-pages, so URLs would not have to change.
.rst is definitely not as simple of a format as .md, but the ability to fetch docstrings and function signatures I think offers a clear advantage over continuing with markdown.

@dzucker-tab, thoughts?
Our tooling would be constrained by having to work with our internal doc writing setup, which I'm not perfectly familiar with.

In the past we've talked about this idea internally as well. One big benefit in my mind is we might have better traction keeping the docs up to date with each PR, whether it's automated docstrings or other doc content that can all come together in one PR.

#902

I had missed the prior conversation about this. Concern over docs updates being tied to package updates is a pretty big one. I know that I have found myself missing adding changes to docs and then doing smaller PRs to fix it later. I guess the question is if having them available in docstrings and in code PRs means the docs revisions are likely to be less frequent?

Docs updates would have to go to /main, but we don't need to bump the package every time we update main. We'd hope not to have a stream of doc updates, but if we need to make some it won't be a real problem.
Dan says 👍 on the branch change and probably doc-strings.

Ok. I'll progressively submit PRs and some conversions to the docs. I think it will take quite a while before the gh-pages branch can be really replaced.


@conda-forge-admin, please add bot automerge

Additional comment:
@conda-forge-admin, please add bot automerge

This is the wrong way.


High level consumers should be able to recover a zk disconnect

This PR is to fix #279

Hi,
I am also experiencing the same issue, I would like to know what is the status and when would it be merged with master?
Thanks a bunch guys!

Hi,
I still having the same issue with version 0.3.2
It's this fix in the version 0.3.2?
I get this trace:
{
"date": "Mon Mar 28 2016 08:56:00 GMT+0000 (UTC)",
"process": {
"pid": 13001,
"uid": 0,
"gid": 0,
"cwd": "/opt/timeline",
"execPath": "/usr/local/bin/node",
"version": "v5.8.0",
"argv": [
"/usr/local/bin/node",
"/opt/timeline/src/index.js"
],
"memoryUsage": {
"rss": 146669568,
"heapTotal": 76627232,
"heapUsed": 32355224
}
},
"os": {
"loadavg": [
1.2470703125,
1.48974609375,
1.39697265625
],
"uptime": 14924063
},
"trace": [
{
"column": 46,
"file": "/opt/timeline/node_modules/node-zookeeper-client/lib/ConnectionManager.js",
"function": "ConnectionManager.cleanupPendingQueue",
"line": 189,
"method": "cleanupPendingQueue",
"native": false
},
{
"column": 10,
"file": "/opt/timeline/node_modules/node-zookeeper-client/lib/ConnectionManager.js",
"function": "ConnectionManager.onSocketClosed",
"line": 274,
"method": "onSocketClosed",
"native": false
},
{
"column": 13,
"file": "events.js",
"function": "emitOne",
"line": 90,
"method": null,
"native": false
},
{
"column": 7,
"file": "events.js",
"function": "Socket.emit",
"line": 182,
"method": "emit",
"native": false
},
{
"column": 12,
"file": "net.js",
"function": "TCP._onclose",
"line": 475,
"method": "_onclose",
"native": false
}
],
"stack": [
"CONNECTION_LOSS: Exception: CONNECTION_LOSS[-4]",
"    at ConnectionManager.cleanupPendingQueue (/opt/timeline/node_modules/node-zookeeper-client/lib/ConnectionManager.js:189:46)",
"    at ConnectionManager.onSocketClosed (/opt/timeline/node_modules/node-zookeeper-client/lib/ConnectionManager.js:274:10)",
"    at emitOne (events.js:90:13)",
"    at Socket.emit (events.js:182:7)",
"    at TCP._onclose (net.js:475:12)"
],
"level": "error",
"message": "uncaughtException: Exception: CONNECTION_LOSS[-4]",
"timestamp": "2016-03-28T08:56:00.113Z"
}
Thanks

@GonzaloGutierrezMartinMaestro +1
events.js:141
throw er; // Unhandled 'error' event
^
FailedToRegisterConsumerError: Exception: CONNECTION_LOSS[-4]

Is it fixed?

yes @GonzaloGutierrezMartinMaestro afaik. Though you should switch to using ConsumerGroup is possible.

@hyperlink , I'm using consumerGroup. Still getting connection lost error. Any suggestions ?

Since ConsumerGroup connects to kafka directly to coordinate it shouldn't be getting any connection_loss rebalance errors.

@hyperlink we experienced this connection loss error using normal Consumers.

Has this issue been fixed also for normal consumers?
If not, is it even advisable to use them or should we switch to consumer groups in production environments?

@respinha-ribeiro I did my best to fix this issue two years ago. Since CONNECTION_LOSS is zookeeper based error so using ConsumeGroup with kafkaHost option will avoid this particular issue.

But this PR also handles Zookeeper abrupt connection closing situations, right?
So, if we want to still use Zookeeper to manage Kafka brokers on a high level consumer / ConsumerGroup in theory we can simply omit the Kafka host, and the client should attempt to reconnect upon error, right?


clean up code - formating

reformat json util codes with style-guide

Those files were generated by JavaCC, so this change is meaningless.


Blynk Mega 2560 + ESP8266 wifi shield 'IPAddress' was not declared in this scope

When uncomment - Blynk.begin(auth, wifi, ssid, pass, IPAddress(192,168,1,100), 8080);
error appears - 'IPAddress' was not declared in this scope
https://examples.blynk.cc/?board=Arduino Mega 2560&shield=ESP8266 WiFi Shield&example=GettingStarted%2FBlynkBlink
and i can't declare this in the scetch
IPAddress local_IP(192, 168, 1, 11);
IPAddress gateway(192, 168, 1, 1);
IPAddress subnet(255, 255, 255, 0);
IPAddress primaryDNS(8, 8, 8, 8);
IPAddress secondaryDNS(8, 8, 4, 4);

Please use https://community.blynk.cc/


Use sql transaction to insert actions in the timing database

I have tested with the replayer, and the timing thread doesn't lag behind the hooks anymore

Looks good, merging👍


Backports 0.19 guix

@Munkybooty do future (non-guix) backports rely on this? Or can we delay this until v19?

I think we should be safe delaying this, all the files are basically isolated and guix seems to be untouched until v20 backports. The only merge conflicts I can see appearing are with qt.mk other than that, it should be fine

superseeded by:

https://github.com/dashpay/dash/pull/5194
https://github.com/dashpay/dash/pull/5237
https://github.com/dashpay/dash/pull/5285


Mention valid module names in the readme

In the JS world naming tests like this <name>.test.js is fairly common.
I followed the readme:

Put tests in a __tests__ directory and use the suffix *test.ml/*test.re.

And created a test module called foo.test.re Like I usually do in JS but nothing got transpiled to JS. It took a bit of figuring out that this notation is not a valid module name in OCaml and the test module names ultimately have to adhere to https://github.com/BuckleScript/bucklescript/blob/master/jscomp/ext/ext_string.ml#L321 (which is [a-z|A-Z|0-9|\-|\\] in JS regex?) so bs picks them up.
I fear that other people who are equally unfamiliar with OCaml modules will get confused the same way when following the readme.
What do you think about amending the readme slightly to make this less likely to happen?

Cool. Thank you!


Can I start notebook just like "iex -S mix"

Then I can import my module to test my code in notebook, how to do this?

Hi @scottming,
if I understand what you're trying to do then I think it's possible. When I debug IElixir and want to achieve something like iex -S mix, I follow these steps:
https://railsware.com/blog/2014/03/26/interconnecting-elixir-nodes/
I set cookie and node's name in start_script.sh by replacing following line:
ELIXIR_ERL_OPTIONS="-smp enable" CONNECTION_FILE=$1 mix run --no-halt

with that line:
ELIXIR_ERL_OPTIONS="-smp enable" CONNECTION_FILE=$1 iex --sname ielixir_node --cookie ielixir_token -S mix run --no-halt

so my node is named from now and I can connect to it by name and using the cookie following steps from above article. I activate Job Control Mode, list all jobs and connect to the one I need with: c [number of the job I want to connect]. Then I have the thing you need.

Hi, @pprzetacznik , I have read the article, but how do I use switch to JCL in notebook? CTRL + G not work. I want to connect the project node in notebook.

Hi, @scottming,
I'm using this approach only for debugging IElixir from external terminal. I run one terminal IElixir having cookies set and the second that I use for connecting to and debugging the IElixir's process. Debugging processes from inside of IElixir is not yet available ;_;


Broken private fields in classes (JS)

Description
Private fields in JS classes use # as part of the identifier name. When beautifying the initial declaration, this library puts it on the wrong line.
Input
The code looked like this before beautification:
class X {
    #__private = null;
}

Expected Output
The code should have looked like this after beautification:
class X {
    #__private = null;
}

Actual Output
The  code actually looked like this after beautification:
class X {
    #
    __private = null;
}

Steps to Reproduce
Environment
OS: Windows
Language: Python
Settings
Example:
{
    "indent_size": 4,
    "indent_char": " ",
    "indent_level": 0,
    "indent_with_tabs": false,
    "preserve_newlines": true,
    "max_preserve_newlines": 2,
    "jslint_happy": false,
    "space_after_anon_function": false,
    "brace_style": "collapse,preserve-inline",
    "keep_array_indentation": false,
    "keep_function_indentation": false,
    "space_before_conditional": true,
    "break_chained_methods": false,
    "eval_code": false,
    "unescape_strings": false,
    "wrap_line_length": 0
}

So what's the solution for this?

PR #1735 fixed it and it is released.


Compilation error

Hi,
I used the following Dockerfile to compile Barretenberg:
FROM alpine:3.13 AS builder
RUN apk update \
    && apk upgrade \
    && apk add --no-cache \
        build-base \
        clang \
        cmake \
        git \
        curl \
        perl
# libomp is not available in alpine by default. Download and build.
RUN git clone -b release/10.x --depth 1 https://github.com/llvm/llvm-project.git \
  && cd llvm-project && mkdir build-openmp && cd build-openmp \
  && cmake ../openmp -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ -DLIBOMP_ENABLE_SHARED=OFF \
  && make -j$(nproc) \
  && make install \
  && cd ../.. && rm -rf llvm-project
WORKDIR /usr/src/barretenberg
COPY . .
# Only build binaries that are needed upstream.
RUN mkdir build && cd build && cmake -DCMAKE_BUILD_TYPE=RelWithAssert -DMULTITHREADING=0 -DTOOLCHAIN=x86_64-linux-clang .. && cmake --build . --parallel

FROM alpine:3.13
RUN apk update && apk add llvm10-libs

COPY --from=builder /usr/src/* /usr/src/

and I got the following error while building:
[ 91%] Built target rollup_proofs_account_tests
CMake Error at /usr/share/cmake/Modules/GoogleTestAddTests.cmake:77 (message):
  Error running test executable.

    Path: '/usr/src/barretenberg/build/bin/stdlib_merkle_tree_tests'
    Result: Illegal instruction
    Output:

Call Stack (most recent call first):
  /usr/share/cmake/Modules/GoogleTestAddTests.cmake:173 (gtest_discover_tests_impl)

make[2]: *** [src/aztec/stdlib/merkle_tree/CMakeFiles/stdlib_merkle_tree_tests.dir/build.make:120: bin/stdlib_merkle_tree_tests] Error 1
make[2]: *** Deleting file 'bin/stdlib_merkle_tree_tests'
make[1]: *** [CMakeFiles/Makefile2:5029: src/aztec/stdlib/merkle_tree/CMakeFiles/stdlib_merkle_tree_tests.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....

The host OS is Ubuntu 20.04.4 LTS and the processor is Intel® Core™ i7-4790 CPU @ 3.60GHz × 8. I didn't get this error on macOS. I directly installed on the macOS host machine (didn't use the docker file).

I'm running on Mac M1, also compilation error: https://pastebin.com/Pgn2ANjW


wakeonlan error 0.74

Home Assistant release with the issue:
0.74, 0.74.1
Last working Home Assistant release (if known):
0.73.0
Operating environment (Hass.io/Docker/Windows/etc.):
Hass.io, HassOS 1.7, Hass.io supervisor 121, rpi3-64, Samsung TV
Component/platform:
wakeonlan
Description of problem:
ElementError executing service <ServiceCall switch.turn_on: entity_id=['switch.bed_room_tv']>

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
Traceback:
Error executing service <ServiceCall switch.turn_on: entity_id=['switch.bed_room_tv']>
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/homeassistant/core.py", line 1038, in _event_to_service_call
    await service_handler.func(service_call)
  File "/usr/local/lib/python3.6/site-packages/homeassistant/components/switch/__init__.py", line 109, in async_handle_switch_service
    await switch.async_turn_on()
  File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 56, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.6/site-packages/homeassistant/components/switch/wake_on_lan.py", line 87, in turn_on
    self._wol.send_magic_packet(self._mac_address)
AttributeError: module 'wakeonlan' has no attribute 'send_magic_packet'

Additional information:

I've got the same issue. Started on 74.0

I found out that there where two libraries in deps/lib/python3.6/site-packages named wakeonlan and wakeonlan-0.2.2-py3.6.egg-info , I renamed these two to .old and restarted the hass.io and after that it seems to work as supposed again. I'm not sure if this affects something else in the installation but the error message disappered and the TV starts via wakeonlan again. Maybe we can get some input regarding this libraries from someone more skilled about this.

I checked homeassistant requirement, we only referenced wakeonlan==1.0.0
It sounds like a user environment issue. Closed issue.
Feel free to reopen it if you still have problem.

How is this a user environment issue when I'm running Hassio.OS? I haven't changed any python dependencies. It stopped working after I moved from Hassio Resin OS to Hass OS.

@Kanishkaz, please open you own issue, you may not have same issue.

Opened a separate ticket: [https://github.com/home-assistant/home-assistant/issues/15756]


git-lfs: 3.4.1 -> cd04b35

Description of changes
Latest git-lfs includes a patch that fixes builtins.fetchGit for lfs-enabled repos.
I think that patch is especially useful for nix users. git-lfs, being a mature project, releases fairly slowly, so I'd like to point nixpkgs at tip of git-lfs main for the time being.
Things done
Bumped git-lfs from v3.4.1 to cd04b35

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Nevermind, PR in question has 19 commits, which would be a pain to patch individually. Using a commit from master would probably be the easier way to go.

Addressed by https://github.com/NixOS/nixpkgs/pull/292479


[Snyk] Security upgrade node from 14-bullseye to 14.20.0-slim

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

Dockerfile

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

Severity
Priority Score / 1000
Issue
Exploit Maturity

500
Release of Invalid Pointer or Reference SNYK-DEBIAN11-AOM-1290331
No Known Exploit

500
Use After Free SNYK-DEBIAN11-AOM-1298721
No Known Exploit

500
Buffer Overflow SNYK-DEBIAN11-AOM-1300249
No Known Exploit

400
Use After Free SNYK-DEBIAN11-MARIADB105-2940588
No Known Exploit

400
CVE-2022-32085 SNYK-DEBIAN11-MARIADB105-2940589
No Known Exploit

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

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

This will be included in the demo overhaul.


Simple release note typo.

Simple typo on 1.3.0 release note.

I initiated pull request based on your suggestion, link


Issue Trackers

What are the issue trackers that you considered for creating the dataset?

When initially creating the data set, we considered jira, sourceforge, google code, and github.
The bug-mining README provides more details.
Best,
René

I am closing this issue; feel free to re-open it if my previous response did not fully answer your question.

Thanks for the help. How do we know which Github projects are using Jira?

Many Github projects use Github's issue tracker. If a project uses something else, then the project description should indicate that. For example, the Apache projects use Jira.
If your question is whether a Git project in Defects4J used Jira, then you can look at the issue-tracker urls in the metadata for that project.


dataset.from_generator error with multiple inputs

System information

no
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 16.04
TensorFlow installed from (source or binary): pip
TensorFlow version (use command below): tensorflow-2.1.0 (cpu)
Python version: 3.7

Describe the current behavior
I use tf.keras.Model to build up a model. It has multiple inputs, say input is like [i_1, i_2, i_3, a_1], output is only one, say y.
I have a generator function
def gen():
    for i in range(data_length):
        ...
        yield [i_1, i_2, i_3, a_1], y

Notice that the inputs have variable sizes. Then I have the dataset built as:
ds = tf.data.dataset.from_generator(gen, output_type=(tf.float32,tf.float32, tf.float32, tf.float32, tf.int8))
when I send the ds to model.fit, it shows the error map_fn() takes from 1 to 3 positional arguments but 5 were given
Then I tried
ds = tf.data.dataset.from_generator(gen, output_type=([tf.float32,tf.float32, tf.float32, tf.float32], tf.int8))
and
ds = tf.data.dataset.from_generator(gen, output_type=((tf.float32,tf.float32, tf.float32, tf.float32), tf.int8))
Both of them show errors. How should make the from_generator take multiple inputs?
Thank you!

@sun-peach
Request you to share colab link or simple standalone code to reproduce the issue in our environment.It helps us in localizing the issue faster.Thanks!


ImportError: cannot import name 'Union' from 'torch.nn.modules.conv' (/data/lby/anaconda3/envs/wenet/lib/python3.8/site-packages/torch/nn/modules/conv.py)

Hello, wenet reported the following errors in the training process , how should I solve
ImportError: cannot import name 'Union' from 'torch.nn.modules.conv' (/data/lby/anaconda3/envs/wenet/lib/python3.8/site-packages/torch/nn/modules/conv.py)

what's your torch version?

have you solved the problem?


bug: correctly use configured postfix mailname

Correctly use configured postfix mailname
Fixes:

Scenario: mailname is configured in [postfix] section in zulip.conf

Current behavior: the option is ignored. /etc/mailname is always equal to machine's hostname.

Expected behavior: the option mailname option should be respect.

In fact, I haven't check the code much because the bug seems to be obvious.
I am using zulip-docker. Currently, instead of rebuilding the image, I monkey patch /etc/mailname for a temporary solution.

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

I was confused by the document about incoming email
https://zulip.readthedocs.io/en/latest/production/email-gateway.html#local-delivery-setup
At step 4, the doc mentions setting mailname.
I successfully setup incoming email integration via docker with this PR merged (via monkey patch).
I haven't tested with the original version (before the PR patched).

I have force-pushed to rewrite the commit message.

Thanks!  I repushed with a longer commit message, and updated documentation.


Can not make pull requests, permission denied

I am attempting to make a minor PR but get:
~/code/htmx-go(update-readme-templ-example) » git push origin update-readme-templ-example

ERROR: Permission to angelofallars/htmx-go.git denied to brainmaniac.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

@brainmaniac Are you pushing to the angelofallars/htmx-go repository instead of your fork?
You should make your fork and push your changes there, instead of to the main repository.


[UI] Remove npm run build warning about NODE_ENV=production

Why are the changes needed?

Before this PR, when we run npm run build to build production dist will meet:
NODE_ENV=production is not supported in the .env file. Only NODE_ENV=development is supported to create a development build of your project. If you need to set process.env.NODE_ENV, you can set it in the Vite config instead.

So, let's remove NODE_ENV=production in .env.production, it's non-used.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Was this patch authored or co-authored using generative AI tooling?

No

FYI cc @zhaohehuhu

Please rebase on the latest master

Codecov Report

Merging #5624 (e4fc8c6) into master (97fd5b7) will decrease coverage by 0.10%.
Report is 2 commits behind head on master.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #5624      +/-   ##
============================================
- Coverage     61.45%   61.35%   -0.10%
  Complexity       23       23
============================================
  Files           603      603
  Lines         35621    35621
  Branches       4872     4872
============================================
- Hits          21892    21857      -35
- Misses        11356    11377      +21
- Partials       2373     2387      +14

see 18 files with indirect coverage changes
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

Thanks, merged to master/1.8


After Windows 11 22H2 update TV goes off too early

After installing 22H2 update couple month ago LGTV Companion suddenly started work bad, it tuns off TV ~30 sec before Windows actually turns off displays.
The issue is not critical of course as it generally works, but not clear why this happens

Hi @samepaul this is not something I've noticed. Can you confirm you mean this issue occurs when shutting down the system?
How do you know when windows actually turns off the displays?
Generally (if we're talking about the system shutting down) the application responds to a SERVICE_CONTROL_PRESHUTDOWN and takes appropriate action. Typically there is no power message (PBT_POWERSETTINGCHANGE) after that to which the app can respond. The app could respond to SERVICE_CONTROL_SHUTDOWN instead but that would leave less time to ensure the TV is powered off.
Can it be that you have some other program delaying your shutdown, which needs to be looked at? If you are shutting down the system then the TV will power off as soon as possible (by design) so if there are some program preventing shutdown, you will not see it (unless you reach for the remote and power it back on)

It is not shutdown. It happens when I don't touch my PC for a while. I have "turn off my screen" in Windows Settings being set to 5 minutes, so after 5 minutes happens what I described - turns off my TV and then 10-30 seconds later my 2 other displays. I have 3 monitors in total, that's how I know. The delay is somewhat random and seems depends on the setting mentioned above. Not sure, didn't have patience to test with longer settings, but for 1 minute all monitors and TV turn off almost simultaneously, for 3 minute the TV was ~10 sec ahead, for 5 minutes almost 20 sec. I am pretty sure Windows has changed some of the power management mechanics under the hood, because before the 22H2 update everything worked perfectly.
here example video: https://youtu.be/_FSsbRoe9z4
If there any technical information/logs/etc I can provide or some experiments to do, please tell me what exactly, I will try my best to collect.

Thanks for the details and the video! Do you by any change use a laptop or stationary with modern standby? Enable logging and paste a log at pastebin or similar and paste the link here and I will have a look

hi @samepaul any inpu wrt the above?

hi, sorry, missed your comment.
No, it is not laptop and it does not support modern standby. Regular desktop PC.
The log was very short, so I guess can paste it here :)
Fri 16:58:40 > LGTV Companion Service started (v 1.8.0) ---------------------------
Fri 16:58:40 > Configuration file successfully read
Fri 16:58:40 > Data path: C:\ProgramData\LGTV Companion
Fri 16:58:40 > Device1, LGwebOSTV.lan, with IP <IP_ADDRESS> initiated (Enabled:yes, WOL:2, PairingKey:*********************, MAC: 58:FD:B1:B1:A2:70 , VerifyHdmiInput:1, SetHdmiInput:off, BlankOnIdle:off)
Fri 16:58:40 > Host IP detected: <IP_ADDRESS>
Fri 16:58:40 > Setting shutdown parameter level 0x3FF
Fri 16:58:40 > ** System requests displays ON.
Fri 16:58:40 > Device1, spawning DisplayPowerOnThread().
Fri 16:58:40 > Device1, repeating WOL broadcast started to MAC: 58:FD:B1:B1:A2:70 using IP address: <IP_ADDRESS>
Fri 16:58:40 > GetBestRoute2() selected interface index 10 LUID<PHONE_NUMBER>518336 route protocol 2
Fri 16:58:40 > Device1, established contact: Display is ON.
Fri 16:58:42 > Device1, repeating WOL broadcast ended
Fri 17:01:42 > ** System requests displays OFF(DIMMED).
Fri 17:01:42 > Device1, spawning DisplayPowerOffThread().
Fri 17:01:42 > Device1, HDMI1 input active.
Fri 17:01:42 > Device1, established contact: Device is OFF.
Fri 17:01:57 > ** System requests displays OFF.
Fri 17:01:57 > Device1, omitted DisplayPowerOffThread().
Fri 17:02:09 > ** System requests displays ON.
Fri 17:02:09 > Device1, spawning DisplayPowerOnThread().
Fri 17:02:09 > Device1, repeating WOL broadcast started to MAC: 58:FD:B1:B1:A2:70 using IP address: <IP_ADDRESS>
Fri 17:02:09 > GetBestRoute2() selected interface index 10 LUID<PHONE_NUMBER>518336 route protocol 2
Fri 17:02:10 > Device1, established contact: Display is ON.
Fri 17:02:11 > Device1, repeating WOL broadcast ended

So there is something interesting in the log. I see that your PC sends out a "display OFF (dimmed)" 15 seconds before "Display off". This is usually something you'd expect to see on devices supporting modern standby (typically laptops) where the screen is dimmed down a little before shutting off. Hence my initial question. The app seems to be working according to design, but the question is why your PC apparently and suddently supports modern standby (at least I think so) and if it is connected to the windows update you mentioned. Can you describe the hardware of your PC a bit more detailed (if you know)? In particular the motherboard and CPU.

Here's some information I found about modern standby, a k a known as S0, which seems relevant. Maybe you would be interested in checking if "powercfg /a" reports that your system is S0 capable?
https://www.minitool.com/news/modern-standby.html

Well, your guess is wrong, modern standby is not supported. I have checked this topic quite long time ago, coz I wanted S0 :) But no, only regular S3-S5.
C:\Users\paul>powercfg /availablesleepstates
The following sleep states are available on this system:
Standby (S3)
Hibernate
Hybrid Sleep
Fast Startup
The following sleep states are not available on this system:
Standby (S1)
The system firmware does not support this standby state.
Standby (S2)
    The system firmware does not support this standby state.

Standby (S0 Low Power Idle)
    The system firmware does not support this standby state.

Yes I know, I've built my PC.
MB: Asus Prime Z690m-plus D4
CPU: i5 12600kf

Thanks, that is interesting. I don't think I have ever seen the dimmed state in all the logs I have reviewed (except when testing it initially on my laptop). I will research it a bit more because honestly I don't remember why I chose to turn the displays off on the "dimmed" message if present, although I'm confident there WAS a reason since I mapped it quite carefully at the time

Yes, it looks like there are 2 "display off" events and what you described about dimming indeed feels familiar to what I've seen on my work laptop. And timing feels matching.
If you had reason for choosing 1st event over 2nd, I won't request completely change behavior, but could you please make this choice configurable? Something like checkbox "ignore dimming event"?

I think I remember (disclaimers apply) that I chose this behaviour because 1) the researched assumptions showed that dimmed state kicks in at the time set in the windows control panel to turn the displays off and 2) a  "display power on"-event will be triggered even after a "display dimmed" state.
But now I'm leaning towards changing this behaviour (if it does not cause ill effects).
Btw, do you have somehintg like this in your power options:

Yeah absolutely. "If it works don't mess with it!" is merely a suggestion ;)
I'll see if I can change it to omit the "dimmed" state without wreaking havoc

Hi, I've looked a bit at this and unfortunately I cannot disregard the dimmed event. For some reason, only microsoft knows, when a normal windows screensaver is running only the dimmed event will be sent when the display should power off (turn off screen after X minutes) . So disregarding the dimmed event will cause the display to not power off when a windows screensaver is running.
I will keep this topic open as I'm still thinking if there is a good way to workaround

can you make an option in configuration? i could try eng build to see if it actually helps

I can't think of a way to add it as an option in the UI so that it wouldn't be confusing to most people. I can however think about adding it as a switch to f o be added manually to config.json (for those who know about it) or, preferrably, I can try to figure out a way to make it work regardless.

What the problem? Add checkbox "Ignore dimming event" and that's it.

Maybe, but I don't think an average user of the app has no idea what a "dimmed" event is. I'll consider it

Average user of your app has not clue about 80% of items in "Additional options" :)
"restart words"?
"user idle mode management"?
Why it is needed? (rhetorical question, no need answer)
yet you have it there. And I think you already found elegant solution with "what's this?" link. Add few more lines into popup of "advanced power options and enhanced burn-in protection" and that should be good enough for average user :)

Yeah that is a fair point. :) Still there is a possibility of a fully automated solution, which I'd prefer.
The only corner case (that I'm aware of) is when the windows screen saver has been triggered. I think the logic can be changed to use the dimmed event in case the screen saver is running, or otherwise the "normal" event

oh sorry missed your question. I cant say i fully understand your proposal.
I don't think that screensaver activation should trigger TV power off. As well as dimming should not. Only proper "display turn off" should do the same to TV.
In any case i can be your guinea pig to check if your solution works or not. :)

Hi @samepaul this should be fixed in v3.1.9. let me know how it goes please and/or feel free to reopen the issue if needed

Sorry for late replay, was on lengthy travel. Just tested it and it works. Especially amazing that it required zero configuration from my side. Awesome work, sir!


Count method performance issue

Tweet.precount(:favorites).map { |t| t.favorites_count }

is much faster than
Tweet.precount(:favorites).map { |t| t.favorites.count }

But I want to use latter one.

It's too slow and I have no workaround to avoid it.
Gave up https://github.com/k0kubun/activerecord-precount/commit/7561958abe9a35a315e1292ef668cecb82945009.


Admin comments should be a plugin

Not everyone needs or wants admin comments, they should really be refactored out into a plugin.

+1

+1
and i think Devise too

+1


[Event Request] table 5900 "Service Header" procedure Validate() field "Customer No."

Please can you add the following code/event to the above procedure...
if "Customer No." = xRec."Customer No." then
    if ShippedServLinesExist() then
        if not ApplicationAreaMgmt.IsSalesTaxEnabled() then begin
            TestField("VAT Bus. Posting Group", xRec."VAT Bus. Posting Group");
            TestField("Gen. Bus. Posting Group", xRec."Gen. Bus. Posting Group");
        end;

Validate("Ship-to Code", Cust."Ship-to Code");

IsHandled := false;                             //NEW CODE
OnValidateCustomerNoOnBeforeValidateBillToCustomerNo(Rec, Cust, IsHandled);      //NEW CODE
if not IsHandled then                           //NEW CODE
    if Cust."Bill-to Customer No." <> '' then
        Validate("Bill-to Customer No.", Cust."Bill-to Customer No.")
    else begin
        if "Bill-to Customer No." = "Customer No." then
            SkipBillToContact := true;
        Validate("Bill-to Customer No.", "Customer No.");
        SkipBillToContact := false;
    end;

The event publisher should be as follows...
[IntegrationEvent(false, false)]
local procedure OnValidateCustomerNoOnBeforeValidateBillToCustomerNo(var ServiceHeader: Record "Service Header"; Cust: Record Customer; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10926.


Update Composer dependencies (2024-01-24-00-10)

Visual regression test passed!
View the visual regression test report


Draft widgets no merge.

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
✔

Tests pass?
✔

Syntax Proposal:
<?php NavBar::begin()
    ->brandLabel($params['app.name'])
    ->brandUrl($params['app.home.url'])
    ->options([
        'class' => 'navbar navbar-dark bg-dark navbar-expand-lg',
    ])
    ->init();

    echo Nav::widget()
        ->items($menuItems)
        ->options([
            'class' => 'navbar-nav float-right ml-auto'
        ]);

NavBar::end(); ?>

Widget::begin() need init() --> Widget::begin()->init()/Widget::end();
Widget::widget need return in run string.

The widgets that are missing are those that have to do with activerecord and cache they are:

ActiveField.php.
ActiveForm.php.
ActiveFormClientScript.php.
FragmentCache.php.
InputWidget.php.
LinkPager.php.
LinkSorter.php.

Let's do active forms separately. These require great deal of rethinking because Yii 3 isn't limited to Active Record.
FragmentCache could be done right now (or separately if you prefer). We have everything for it.
Pager and sorter should be done later when https://github.com/yiisoft/data is ready and will be part of https://github.com/yiisoft/yii-dataview.


Cleanup QuadrigaCX market top level declarations

Changed the top level declarations to use const instead of var
Reordered top level declarations to match other exchanges

@cmroche this is the first part of the changes for the Quadriga market.

I'm all up for these changes, but the branches are currently conflicting. Would you mind rebasing or creating a new branch from an up to date askmike/gekko:develop branch and commiting this on top again?

It might have had to do with me merging another PR that also fixed some codes in QuadrigaCX. Apologies for the delay in responding to PRs.

Closing this due to inactivity :( Feel free to comment

SorryI'm being slow on these changes.  I'll fix it up and resubmit later.

@daflame76 can't wait!
Note though that Gekko has since undergone some major upgrades, if we want to support QuadrigaCX again we need to do more changes:

Everything is now inside GB (Gekko Broker - path: exchange/wrappers)
Wrappers cannot import code from outside GB like the gekko logger and such: GB is also a standalone library
cancelOrder has changed (the objects passed to the callback)
checkOrder has changed  (the objects passed to the callback)
A few new methods are now required (like roundPrice, roundAmount, etc)
capabilities has some new fields, like gekko broker version.
error handling is different now.

See the add an exchange doc for details on above.
I'm swamped with work, so I won't be able to do this anytime soon :(

Thanks for the heads up.  Looking forward to all the improvements.


Detailed Solution: How to debug the API server in dev mode using Visual Code

Solves issue #830
1. Change all file imports (not module) on the API server to their relative location, eg.
    import {mapUrl} from 'utils/url.js'; to
    import {mapUrl} from './utils/url.js';

2. Add retain lines in .babelrc
{
  "presets": ["react", "es2015", "stage-0"],
  "plugins": [
    "transform-runtime",
    "add-module-exports",
    "transform-decorators-legacy",
    "transform-react-display-name"
  ],
  "env": {
    "development": {
      "plugins": [
        "typecheck",
        ["react-transform", {
            "transforms": [{
                "transform": "react-transform-catch-errors",
                "imports": ["react", "redbox-react"]
              }
            ]
        }]
      ]
    }
  },
  "retainLines": true  <-------------------------------- add this line
}

3. Configure APIPort to have else value of 3030:
  apiHost: process.env.APIHOST || 'localhost',
  apiPort: process.env.APIPORT || 3030, <--------------- add "|| 3030"

4. Modify Visual Code launch json:
a. change program path from bin/server.js to bin/api.js
b. change environment from development to production
{
	"version": "0.2.0",
	"configurations": [
		{
			"name": "Launch",
			"type": "node",
			"request": "launch",
			"program": "${workspaceRoot}\\bin\\api.js", <------ a. change this
			"stopOnEntry": false, <----- can opt'ly make true to stop on first line of code hit
			"args": [],
			"cwd": "${workspaceRoot}",
			"preLaunchTask": null,
			"runtimeExecutable": null,
			"runtimeArgs": [
				"--nolazy"
			],
			"env": {
				"NODE_ENV": "production" <--- b. change this
			},
			"externalConsole": false,
			"sourceMaps": false,
			"outDir": null
		},
		{
			"name": "Attach",
			"type": "node",
			"request": "attach",
			"port": 5858,
			"address": "localhost",
			"restart": false,
			"sourceMaps": false,
			"outDir": null,
			"localRoot": "${workspaceRoot}",
			"remoteRoot": null
		}
	]
}

5. Set a break point on updating widget (in api/actions/widget/update)
import load from './load';

export default function update(req) {
  return new Promise((resolve, reject) => {
    .......
        const widgets = load(req);
        const widget = req.body;

6. Run (in Visual Code) debug and let is cycle thru, important it cycles thru completely!!
7. In node command prompt type "npm run dev"
8. Open browser to localhost:3000, click widgets link & modify any widget. On save the breakpoint  will be hit

@JoeGrasso awesome man, thanks for this contribution.
@erikras, can you please turn this into a wiki page so we can close this issue?


⚠ ClamAV has degraded performance

In 72fa9e4, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 87 ms

Resolved: ClamAV performance has improved in bfc6ce7.


docs(options): fix some typos and improve consistency

Summary
Fixes a few typos and consistency issues in the "Settings" documentation:

use "Ruff" consistently in the few places where "ruff" is used
use double quotes in the few places where single quotes are used
add backticks around rule codes where they are currently missing
update a few example values where they are the same as the defaults, for consistency

2nd commit might be controversial, as there are many options mentioned where we don't currently link to the documentation sections, so maybe it's done on purpose, as this will also appear in the JSON schema where it's not desirable? If that's the case, I can easily drop it.
Test Plan
Local testing.

2nd commit might be controversial, as there are many options mentioned where we don't currently link to the documentation sections, so maybe it's done on purpose, as this will also appear in the JSON schema where it's not desirable? If that's the case, I can easily drop it.

I think this is true (that it will appear in the JSON Schema). But I think it's worth it.

Wow, this is excellent -- thank you! I skimmed through all the changes. How did you find these?

By manually skimming through the documentation (yeah, for real 😄).

I'm very impressed, thanks!


Issue loading .xml in Superputty that was created by putty connection manager

What steps will reproduce the problem?
1.Try to import a <file>.xml in Superputty that was created by putty connection
manager
2.
3.

What is the expected output? What do you see instead?
 Expect to load the connecttion database that was saved from putty CM
 Instead I get this error (see attached screenshot)

What version of the product are you using? On what operating system?
 I tried Super putty
  <IP_ADDRESS> and <IP_ADDRESS>

Please provide any additional information below.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 2 Oct 2014 at 4:11
Attachments:

Capture.PNG

Closing this issue but all hope is not lost! We still want to help.
See: https://github.com/jimradford/superputty/wiki/Legacy-Imported-Issues on how to help us to help you!


Left sidebar text is unreadable in light theme

Hey love the theme and your work!
Just wanted to give a heads up that left side bar is unreadable in light theme as attached photo.
Basically can't see "favorite" "recent" or any of the below pages

@jtan7th thanks for posting the issue.  Which logseq version are you on and are you mac or pc?  I do not see the same issue for recent links on 0.8.11 running on Windows.

Sorry @sethfair just saw this - windows 0.8.14

Just to give you an idea that it's still happening

Also was there an update to make the hashtags not appear blue anymore? Wasn't sure if it was a theme or logseq thing - would there be a way to get it back to the original blue color?

Adding a comment b/c I think my issues are similar. Latest version of Logseq and windows 10.
Text in certain fields, such as a code chunk, are unreadable in the light version of the theme.

@snowice0 thank you, will take a look.


GROOVY-9797 bug fix (DRAFT)

https://issues.apache.org/jira/browse/GROOVY-9797
This fixes the above bug by using Float.equals() which also changes -0.0f == 0.0f to false instead of true. This corresponds with the https://docs.groovy-lang.org/docs/latest/html/documentation/#_identity_operator and https://docs.oracle.com/javase/8/docs/api/java/lang/Float.html#equals-java.lang.Object- documentations.
This PR was made by:
@tomikoskinen
@jonspe
@Justsofun
@rhanlol

superseded by merging of #1504


Preview is not active as expected

Behaviours
Preview icon is always active which is not expected.
Reproduce

enable sidebar
open any one existed document
enable preview mode for this doc
click the edit icon on sidebar to create one more new document
Then you will find the preview mode is always active, but the actual mode is editing mode

version 1.13.0

Thanks for the feedback, I never found this bug, I'll fix it in the latest release.

It's been fixed and a release is coming this week, stay tuned for updates!


Fix alignment of minus sign to numbers

This fixes the alignment of the minus sign to negative numbers.
The align_to method was being called with an optional argument alignment_vec that did not do anything.
This is a before shot:

This is an after shot:

Whoops. I only meant to include one commit.  The second commit fixes the position of labels on the y-axis by moving them to the left of the y-axis

Weird, I'm not sure why this hadn't surfaced before.  Or more realistically, what recent change caused it.  In either case, thanks!


gha: ensure that helm values.schema.json is not accidentally backported

79733d2dafe6 ("helm: Introduce values.schema.json and tooling") introduced the helm schema file for Cilium v1.16 and later. Let's add a CI check to prevent accidentally backporting it to v1.15 and earlier, as we always hit a conflict if the upstream commit modifies it, due to this file not being present in the target branch.

/test-backport-1.15


cilium-agent very slow/high cpu to when starting on kernel 5.13.15+, with bpfilter errors in dmesg

Bug report
Hey guys! Further to this Slack thread, I've identified that since Ubuntu mainline kernel 5.13.15, cilium-agent takes 2-3 minutes to "regenerate endpoints" (even if, like me, you only have 2-3 endpoints)
The kernel log continually spews out messages like this:
[16581.895694] bpfilter: Loaded bpfilter_umh pid 672693
[16581.895923] bpfilter: read fail 0
[16582.040666] bpfilter: Loaded bpfilter_umh pid 672717
[16582.040809] bpfilter: read fail 0
[16582.207923] bpfilter: Loaded bpfilter_umh pid 672719
[16582.208105] bpfilter: read fail 0
[16582.381819] bpfilter: Loaded bpfilter_umh pid 672721

Occasionally cilium-agent will exit with error 4 as below:
2021-10-12T22:35:21.868605095Z level=error msg="Command execution failed" cmd="[iptables -w 5 -t nat -A CILIUM_POST_nat -m mark --mark 0x00000f00/0x00000f00 -o cilium_host -m conntrack --ctstate DNAT -m comment --comment hairpin traffic that originated from a local pod -j SNAT --to-source <IP_ADDRESS>]" error="exit status 4" subsys=iptables
2021-10-12T22:35:21.868669514Z level=warning msg="Another app is currently holding the xtables lock. Stopped waiting after 5s." subsys=iptables
2021-10-12T22:35:30.703258495Z level=error msg="Error while initializing daemon" error="exit status 4" subsys=daemon
2021-10-12T22:35:30.703320354Z level=fatal msg="Error while creating daemon" error="exit status 4" subsys=daemon

General Information

Cilium version (run cilium version) : 1.10.4
Kernel version (run uname -a) : 5.13.15 onwards
Orchestration system version in use (e.g. kubectl version, ...) : Kubernetes 1.21

How to reproduce the issue

Use Ubuntu 20.04 nodes
Use https://github.com/pimlie/ubuntu-mainline-kernel.sh to install kernel 5.13.5 (./ubuntu-mainline-kernel.sh -i 5.13.15)
Reboot into kernel 5.13.15, watch kernel logs with dmesg -w, and note how long it takes (if at all) for a cilium pod to regenerate endpoints
Repeat the above on 5.13.14 (./ubuntu-mainline-kernel.sh -i 5.13.14), and note how the fault does not occur.

bpfWaitForELF is higher than on my toy cluster but the rest is similar. That could potentially indicate higher contention on the objectCache lock but I don't see how that would be related to a kernel version :-/
Could you share a complete Cilium sysdump? We should check that templating state looks okay and the agent logs may also have more information on specific endpoint's regenerations.


Document globals argument for HTTP/GraphQL protocols

What is the best way to specify globals in graphql?
In other words, if a resource has a specific access policy, what is the best way to indicate "session data" like the current user ID when interfacing with edgedb over the graphql interface?

You can pass a globals argument to the endpoint, similar to variables.
The keys should be fully qualified, like "default::global_var".
This ought to be documented, of course.

@fritzblue does global even work with your GQL client?
I have added 2 globals currentUserId (scalar) and currentUser (query).
With the current extension, when I visit http://localhost:10704/db/edgedb/graphql/explore
I get the following error:
{
  "errors": [
    {
      "message": "TypeError: Names must match /^[_a-zA-Z][_a-zA-Z0-9]*$/ but 'Filterdefault|currentUser@default|currentUser@global' does not.\n\nNames must match /^[_a-zA-Z][_a-zA-Z0-9]*$/ but 'default|currentUser@default|currentUser@global' does not.\n\nNames must match /^[_a-zA-Z][_a-zA-Z0-9]*$/ but 'default|currentUser@default|currentUser@global' does not.\n\nNames must match /^[_a-zA-Z][_a-zA-Z0-9]*$/ but 'Orderdefault|currentUser@default|currentUser@global' does not.\n\nNames must match /^[_a-zA-Z][_a-zA-Z0-9]*$/ but 'delete_default|currentUser@default|currentUser@global' does not."
    }
  ]
}

@MiroslavPetrik I'm seeing this issue as well (cc @msullivan)

I've also faced with it because it is not documented.


Maintenance: e2e test ignore parameters used to invoke Lambda

Summary
When adding the integration tests for Idempotency (#1442), we modified one of the internal utilities used to invoke the deployed Lambda functions. In doing so, we introduced a bug that causes some unrelated integration tests to fail. We should identify & fix the issue so that all integration tests run successfully.
Why is this needed?
To be able to run the integration tests on all utilities.
Which area does this relate to?
Tests
Solution
No response
Acknowledgment

[X] This request meets Powertools for AWS Lambda (TypeScript) Tenets
[ ] Should this be considered in other Powertools for AWS Lambda languages? i.e. Python, Java, and .NET

Future readers
Please react with 👍 and your use case to help us understand customer demand.

Successful integration test run containing the patch: https://github.com/awslabs/aws-lambda-powertools-typescript/actions/runs/5223831815


[feat] [dialog] How to determine whether to open photos picker or files picker on iOS

using open imported from @tauri-apps/plugin-dialog will open files picker on iOS 17.5 simulator.
Is there a way to determine whether to open photos picker or files picker? I just want to open photos picker by open.
tauri-plugin-dialog: 2.0.0-beta.11

From my testing the following opens the file dialog:
await open({ filters: [{ extensions: ["pdf", "txt"], name: "Documents" }]});

but interestingly the following opens photo chooser:
await open({ filters: [{ extensions: ["pdf"], name: "Documents" }]});

Checking the source code, open method can show multiple dialogs depending on the parsing of the filters attribute

Confirmed.  Adding the second extension to the extensions array opens the Files app.  However, for some reason the target files are greyed out and not selectable.

I wonder if this may be of use:
PLEASE USE CAUTION WITH THE CODE BELOW, I am not a Swift developer.  I was looking to see what could be the issue with parsing the code and so asked Claude AI.  I realize that asking AI may be bad form, but sometimes it proves to be useful.
The code is using kUTTagClassMIMEType to create the UTI, but it's passing in the file extension. Instead, should it use kUTTagClassFilenameExtension?
import MobileCoreServices
import Photos
import PhotosUI
import SwiftRs
import Tauri
import UIKit
import WebKit

// ... (previous code remains the same)

class DialogPlugin: Plugin {
  var filePickerController: FilePickerController!
  var pendingInvoke: Invoke? = nil
  var pendingInvokeArgs: FilePickerOptions? = nil

  override init() {
    super.init()
    filePickerController = FilePickerController(self)
  }

  @objc public func showFilePicker(_ invoke: Invoke) throws {
    let args = try invoke.parseArgs(FilePickerOptions.self)

    let parsedTypes = parseFiltersOption(args.filters ?? [])

    pendingInvoke = invoke
    pendingInvokeArgs = args

    DispatchQueue.main.async {
      if #available(iOS 14, *) {
        var configuration = PHPickerConfiguration(photoLibrary: PHPhotoLibrary.shared())
        configuration.selectionLimit = (args.multiple ?? false) ? 0 : 1

        if !parsedTypes.isEmpty {
          configuration.filter = self.createPHPickerFilter(from: parsedTypes)
        }

        let picker = PHPickerViewController(configuration: configuration)
        picker.delegate = self.filePickerController
        picker.modalPresentationStyle = .fullScreen
        self.presentViewController(picker)
      } else {
        let picker = UIDocumentPickerViewController(documentTypes: parsedTypes, in: .import)
        picker.delegate = self.filePickerController
        picker.allowsMultipleSelection = args.multiple ?? false
        picker.modalPresentationStyle = .fullScreen
        self.presentViewController(picker)
      }
    }
  }

  private func parseFiltersOption(_ filters: [Filter]) -> [String] {
    var parsedTypes: [String] = []
    for filter in filters {
      for ext in filter.extensions ?? [] {
        if let utType = UTTypeCreatePreferredIdentifierForTag(kUTTagClassFilenameExtension, ext as CFString, nil)?.takeRetainedValue() as String? {
          parsedTypes.append(utType)
        } else {
          // If we can't get a UTI for the extension, add a wildcard UTI
          parsedTypes.append("public.item")
        }
      }
    }
    // If no valid types were found, allow all file types
    return parsedTypes.isEmpty ? ["public.item"] : parsedTypes
  }

  @available(iOS 14, *)
  private func createPHPickerFilter(from parsedTypes: [String]) -> PHPickerFilter? {
    let filters = parsedTypes.compactMap { UTType($0) }.map { PHPickerFilter.any(of: [$0]) }
    return filters.isEmpty ? nil : PHPickerFilter.any(of: filters)
  }

  private func presentViewController(_ viewControllerToPresent: UIViewController) {
    self.manager.viewController?.present(viewControllerToPresent, animated: true, completion: nil)
  }

  // ... (rest of the code remains the same)
}

File chooser on a iOS webview asks for chooser type so I think it is best to have different commands/options for choosing the kind of the dialog.

No change when upgrading to RC
tauri-plugin-dialog = "2.0.0-rc.0"
"@tauri-apps/plugin-dialog": "2.0.0-rc.0"

Thanks for all the hard work on getting Tauri 2 ready for release.  Has this issue been resolved yet?  Or is my use of the API incorrect?

Same issue with 2.0.1
Has there been any updates to the API or config?  Do I need to explicitly set permissions on the Simulator device?

I would like to select files with the files app as well

Same problem on iOS.

For me it worked once I set different file types in the extension. This is caused by a bug in the plugin logic.
    extensions: ["image/jpeg", "text/csv"],

Simply setting ["text/csv"] does not work. If the types are not different, it also does not work, such as with ["text/csv", "text/plain"].
@FabianLars This issue is not really a question but a bug.


workshop environment, requirements, and ami updates

Issue #69 and N/A:

Updated AWS AMI to Rancher Manager v2.7.8, Rancher Kubernetes Engine 2 v1.25.14, and Cert-Manager v1.13.1

Tested and verified comparability with the workshop environment on two accounts in us-east-1 and us-west-2

Updated AWS AMI workflow to increase reliability and upgradability for future updates (view the repo)
Updated Workshop Requirements to add clarity for users at AWS Events and Outside AWS Events

Feedback received from users at the most recent AWS Event

Minor updates to AWS CloudFormation template to have the same formatting and configuration

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AMI for us-east-1: https://us-east-1.console.aws.amazon.com/ec2/home?region=us-east-1#ImageDetails:imageId=ami-09d634be7b09ca862
AMI for us-west-2: https://us-west-2.console.aws.amazon.com/ec2/home?region=us-west-2#ImageDetails:imageId=ami-07b6f00cd7f1769df

AMI for us-east-1: https://us-east-1.console.aws.amazon.com/ec2/home?region=us-east-1#ImageDetails:imageId=ami-09d634be7b09ca862
AMI for us-west-2: https://us-west-2.console.aws.amazon.com/ec2/home?region=us-west-2#ImageDetails:imageId=ami-07b6f00cd7f1769df

LGTM! Apologies for all the extra commits.

hey @evdevr, please review and merge (if not changes required)!

Closing due to recent updates that will be better options for the workshop.


Use different font-family for Japanese

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

Closes #XXXXX

I want to change the font-family for Japanese pages.
Background:

Many of the Japanese fonts don't have italic style. They are just displayed in normal style even if they are wrapped in <em> tags or set to font-style: italic; in CSS.
Many freeCodeCamp articles use italic style.
To display those articles properly, I want to use a font that has italic style

Before the change:

After the change:

These themes will no longer be used - like soonish-soon. We should move these changes over to our JAMStack template: https://github.com/freeCodeCamp/news

@raisedadead Thanks for the information. I wasn't aware of that. I will look into the repo and create another PR there.


fix Derrida coefficient normalization

Github is getting weird!
Since the previous PR was merged, there should be only on commit waiting for merge. But it showed two.
Anyway, there are only two lines of difference.

Hi Rion,
Let me clarify two things.

This PR doesn't contain anything involving output format. It only contains patch to Derrida co-efficient per Alex's request. That function returns a float.

Somehow github has problem to show the commits need to merge. It wrongly declared that there are 5 commits to be merged, actually there are only two. I merged all changed from master branch before sending the PR.
If you look at "Filed changed" tab, you will see that actually there are only three lines change, they are on the DC function.

My previous PR (which has been accepted and shouldn't be displayed in this PR any more) is addressing the output problem. GitHub has problem to display that PR too. (On local git, git show only two lines changes but GitHub shows hundreds of lines. Who knows what was happening)
I didn't change the output format from int to str in my previous PR. Somebody else did (I didn't run git blame to check who did it) and his commit broke a lot of old code. Sikander couldn't run the simple ks function and asked me for help. I patched the ks function so that it is compatible to str. It is a dirty quick fix, but at least it allow the master branch to work. (Can't imagine how disastrous it is that people use CANA find out master branch cannot work) Some day we need to dig up who and why change the output from int to str and fix codes relying on that.


feat: Properly store with objects in SqlStorage

Type: feat
Ionic Version: 2.x
Platform: all
Right now if you store objects through SqlStorage, they get stored as a string "[object Object]". I think it would be super helpful and developer-friendly to allow arbitrary json storage. Basically serializing and deserializing it for users. I'm sure there's a lot of considerations to make that practical so maybe you guys already discarded that use case. If that's the case, then you should document that somewhere or even log a warning when a call to .set is passing an object as the value.
    var options = {};
    let storage = new Storage(SqlStorage, options);
    storage.set('myobj', {name:'myname'});
    storage.get('myobj').then((myobj) => {
      console.log(myobj);// this logs "[object Object]"
    });

when logging objects in the console try doing
console.dir(myobj)

Yes, I'm aware, same thing happens (I actually even did debugger to confirm the situation). I think WebSQL simply calls .toString() on objects, which would lead to this. Haven't tested on SQLite.

+1
I have the exact same issue with Local Storage

Hello all! We are currently refactoring the storage module to not use webSQL any more as it is a deprecated technology and is not supported any more in iOS 10 and is soon to be removed from chrome too. IndexedDB is the future of complex web storage and is what we are moving too. Because of all this I am going to be closing this issue for now. Thanks for using Ionic!


Add Menus

@bain3 I don't understand why mypy check doesn't pass😅

Also btw, maybe don't force push... Mistakes are mistakes, no one honestly cares. Best option is to catch them before pushing, but well... ¯\_(ツ)_/¯

Excuse me I will pay attention to my mistakes next time.  And I won't force push anymore

I mean force pushing is a useful tool but it is quite dangerous. Doing it in a PR is fine I guess, on master it would be worse...


[REQUEST] Reducing CPU Memory Footprint during Model and Optimizer Initialization

Is there a method to mitigate the initial memory overhead during the start of training?
Here are my observations while using DeepSpeed stage 2:

Loading the N B model on nGPUs in the DeepSpeed stage-2 process:

Each GPU starts a process that constructs the network in memory, with network parameters occupying a certain amount of memory per process: 4N GB or 2N GB (fp32 or fp16). In the case of fp16, it occupies nGPUs * 2N GB of memory space.

nGPUs divide the entire optimizer state, gradients, and fp32 parameter copies, utilizing DeepSpeed to initialize optimizer states (first-order, second-order), gradient, and fp32 parameter copies in memory. This requires a total of (4 + 4 + 2 + 4) * N = 14N GB of memory space, where each card or process occupies 14/nGPUs GB of memory space.

Distribute the 2*N GB parameters and 14N/nGPUs GB optimizer components to different GPUs.

Hence, the memory requirement is at least nGPUs * 2N GB + 14N GB of space.
For example, when training a 13B model using 8 GPUs,

each card occupies 26GB of space in memory to store fp16 parameters, totaling 208 GB for all 8 cards.
The "optimizer state, gradients, and fp32 parameter copies" on the 8 cards together occupy 14 * 13 = 182 GB of memory space.

Therefore, the total memory space required is 208 GB + 182 GB = 390 GB.

@Line290 Actually it's not DeepSpeed bug!
i'm faced exactly the same question when using Zero-stage2, when loading Llama-2-7b takes 220+gb RAM and Llama-2-13b going up to OOM (my device only have 250gb RAM)
in my case, The questions is LlamaForCausalLM.from_pretrained(model_name_or_path) loading  shards on cpu by default, adding the parameter device_map=auto will resolve it
moreover, the loading weights is defaulted on fp32, u need set torch_dtype=torch.float16 see at https://discuss.huggingface.co/t/llama-7b-gpu-memory-requirement/34323
Additional, i also set low_cpu_mem_usage=True
Last, device_map=auto and low_cpu_mem_usage=True mismatch with Zero-stage3
Hope it works for you😛

Thanks @Honesty-of-the-Cavernous-Tissue for your input. Closing as resolved.

@Honesty-of-the-Cavernous-Tissue I tried it but failed.


Update nodes.md

--api-servers has been deprecated and is no longer in 1.7+.

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.8 Features: set Milestone to 1.8 and Base Branch to release-1.8
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: Please check the “Allow edits from maintainers” box (see image below) to
allow reviewers to fix problems on your patch and speed up the review process.
Please delete this note before submitting the pull request.

This change is

@zacharysarah
should he change his base branch to release 1.8?

Hi @zhangxiaoyu-zidif, it looks like this has been gone since 1.6. here is the current man page:
https://kubernetes.io/docs/admin/kubelet/


Fix for seed data test travis failure

Description and Motivation
...
Change type

[x] Bug fix
[ ] New feature
[ ] Enhancement
[ ] Breaking change (breaks backward compatibility)

Checklist:

[x] All tests passed
[x] The code follows the code style and conventions of this project
[ ] The change requires a change to the documentation
[ ] I have updated the documentation accordingly

Please provide a description for the PR.  I don't understand why the change is required, why is 'docker rmi' a problem?

Closing this PR since this is merged and updated in https://github.com/Azure/azure-iot-pcs-remote-monitoring-dotnet-tests/pull/15


CakePHP WebSocket Plugin Implementation in 2.6.10

I follow all steps as you defined but when It display fetal error like : Unsupported operand types
My Steps as below:

Clone this plugin to app/Plugin/WebSocket
Load the plugin in your app/config/bootstrap.php file :
define('SERVER_NAME', $_SERVER['SERVER_NAME']);
CakePlugin::load('WebSocket');
3)To configure one of your model to use the Publishable Behavior.

I put it in my app/Model/User.php
public $actsAs = array('WebSocket.Publishable' => array('fields' => array('name', 'status_date', 'status_code', 'status_progress')));
4) To configure one of your controller to use a socket.
I put it in my app/Controller/UsersController.php
'WebSocket.php'));
$websocket = new WebSocket(array('port' => 8080, 'scheme'=>'ws'));

class UsersController extends AppController {

5) Setup a Node WebSocket server using Socket.io like:

I put it in my app/View/Users/index.ctp

6) Implement this custom Exception

// in bootstrap.php
require(APP . 'Lib' . DS . 'MyExceptions.php');

// in app/Lib/MyExceptions.php
class ServiceUnavailableException extends CakeException {

```
protected $_messageTemplate = 'Test';
```

}

7) Quick example:

I put it in my app/Controller/UsersController -> index()

websocket = new WebSocket(array('port' => 8080, 'scheme'=>'ws'));

if($websocket->connect()) {

```
$someData = array('notify' => false, 'foo' => $bar);
$websocket->emit('my other event', $someData);
```

}

so please please guide me where I miss some steps or whatever wrong in this steps.

Thanks & Regards,
Brijesh

brijshivam1987 Have you got any solution of this , because i am also facing same issue


Feature Request: Folder name with no _index should not direct to empty page. Show subpages instead

Feature Description
Currently when a folder has no _index.md and you click on its name in the sidebar, you get redirected to a blank article page.
Problem/Solution
Either only toggle the collapse, or direct to the first item of the list, or automatically list the subpages. I like the latter the most.
Alternatives Considered
Adding an _index.md to all folders. However, they need to be maintained and kept up to date when their purpose is to link to the subpages.

Now that I think about it: It might be better UX if the folder name only toggles the list and is not a link to a site itself, as this can be unexpected and makes it harder to find the data you are looking for (or clicking on it just list the subpages)

Currently when a folder has no _index.md and you click on its name in the sidebar, you get redirected to a blank article page.

Hi, can you provide more details on how to reproduce this?

This is weird. I cloned the Hextra template, removed the _index.md from folder and it works as expected.
Whereas with the files from https://github.com/visual-dawg/hyprland-wiki it creates odd behaviour. Not sure why though.
I will clone its contents into the starter repo and see if maybe my setup was simply wrong

I created a reproducible repo using the start template here: https://github.com/Visual-Dawg/hextra-bug-report/tree/main

thanks, will look into it 👍

Thank you! Also a huge thanks for creating this theme. It is by far the best looking theme for Hugo out there

Hello
I am also interested in this. I tried to create a custom view by adding a list.html to the layouts/blog folder and copied the content from the hextra\layouts\blog\list.html file. However, this causes the page to render incorrectly. all the content is displayed one another. Instead of "sidebar | content | toc", it shows
"Sidebar
toc
content"

@AndyRaphael

I tried to create a custom view by adding a list.html to the layouts/blog folder and copied the content from the hextra\layouts\blog\list.html file. However, this causes the page to render incorrectly. all the content is displayed one another. Instead of "sidebar | content | toc", it shows
"Sidebar
toc
content"

regarding this, can you open a separate discussion?

Thank you for your great looking template. Looking forward to this update to show folder content automatically.

@Visual-Dawg

I created a reproducible repo using the start template here: https://github.com/Visual-Dawg/hextra-bug-report/tree/main

This should be fixed by #325
but I guess you've added _index.md to every sub-directories already :)

Nice, thank you! Yes, we did, but good to know for the future :D


Revert "Android : implement InputEventMagnifyGesture and InputEventPanGesture"

This reverts commit 9cc66495cfb6393102ce5ad4e82f2ccb716b9b33 / #25474.
This caused regressions with the handling of screen drag events.
Fixes #33428.
Fixes #33459.
Fixes #33470.
It was my mistake for merging such a PR so close to the 3.2 beta, it should have been reviewed and merged earlier in the release process to be able to iron out regressions.
The PR can be redone (with regressions handled) and merged after 3.2 is released, possibly cherry-picked for 3.2.1.

I have the fix, it's in #25474, I'm checking it right now.

I have the fix, it's in #25474, I'm checking it right now.

Any update? I want to release 3.2 beta 2 today or tomorrow, so I need a fix for the regressions.

Maybe simply revert it?
Issue: https://github.com/godotengine/godot/issues/33459
The continue case happens too many times because this is set?
https://github.com/godotengine/godot/pull/33536/files#diff-7c53cc557c747dfb35393fd94faeef8dL420
'continue case'
https://github.com/godotengine/godot/pull/33536/files#diff-7c53cc557c747dfb35393fd94faeef8dL454
Issue: https://github.com/godotengine/godot/issues/33428
Removing this enables kind of smooth movement but it looks too precise which is why PAN_GESTURE_MIN_DELTA was porbably added in the first place
https://github.com/godotengine/godot/pull/33536/files#diff-7c53cc557c747dfb35393fd94faeef8dL414
I think the d variable has to be a class variable, currently it always resets which means it will only fire if a fast/big move happens.
https://github.com/godotengine/godot/pull/33536/files#diff-7c53cc557c747dfb35393fd94faeef8dL413
 d += (p_points[0].pos - touch.write[0].pos);
I never used these events but is there a reason why this is not used https://developer.android.com/reference/android/view/GestureDetector.OnGestureListener.html#onScroll(android.view.MotionEvent, android.view.MotionEvent, float, float)

Yes, I'll go with a revert for now. @jeremyz seems to have a possible fix but more testing is needed and probably more iteration on the API, so it's better to revert and work on a better implementation for 4.0 (together with implementations for other platforms with touch support to have a more consistent API).

the problem still exist on version 3.2.1 from https://godotengine.org


Dynamically move mask to device in SemanticSegmentationTarget

Fixes issue #545
Avoids pre-moving the self.mask tensor to a device and appropriately does so when the class is called. This prevents a RuntimeError from occurring when the user attempts to call the model with a device that is not the pre-determined MPS/CUDA device.

Thank you so much for the contribution!


Animation helper when animating from / to visible

I often find myself in a situation where I have a layer with two states (hidden and visible), usually handling the transition by fading the opacity. When the layer is hidden I want to be able to pass events to layers behind it, so I want to set its visible property to false. I'd like to set it up like so:
modal.states.add
	hidden:
		opacity: 0
		visible: false
	visible:
		opacity: 1
		visible: true
modal.states.switch("visible")

However, when I actually run the transition, the visible property doesn't "do the right thing" by turning on immediately before the transitions in both directions. Would it be possible to special-case the transition of this property so it helps reduce code complexity?

Yeah I was thinking about this the other day.
How would you determine to throw the visible setting at the front?
On Tue, Jan 17, 2017 at 1:08 PM Nefaur Khandker<EMAIL_ADDRESS>wrote:

I often find myself in a situation where I have a layer with two states
(hidden and visible), usually handling the transition by fading the
opacity. When the layer is hidden I want to be able to pass events to
layers behind it, so I want to set its visible property to false. I'd
like to set it up like so:
modal.states.add
hidden:
opacity: 0
visible: false
visible:
opacity: 1
visible: true
modal.states.switch("visible")
However, when I actually run the transition, the visible properly doesn't
"do the right thing" by turning on immediately before the transitions in
both directions. Would it be possible to special-case the transition of
this property so it helps reduce code complexity?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/koenbok/Framer/issues/463, or mute the thread
https://github.com/notifications/unsubscribe-auth/AACbYlVQDgyLlwYPzOWwPyAIRdA7PuoFks5rTS2ugaJpZM4LmKWY
.

Good question! I think it'd be safe to assume that if you're ever animating between visible == true to visible == false (in either direction) you'd immediately set visible to true at the beginning of the animation and add in an opacity animation (if one doesn't already exist). At the end of the animation you'd set visible to whatever the destination value is.
This wouldn't apply if that transition weren't animated.


Some explanation of how it was done?

The project looks fantastic.
It would be cool if there were some WIKI page explaining how it was done :).

I am about to do that since I found the README.md is bloating...


Warning: Accessing PropTypes via the main React package is deprecated. Use the prop-types package from npm instead.

Summary:
In react 15.5, instead of accessing PropTypes from the main React object, install the prop-types package and import them from there:

Ditto, getting this

Fixed in v1.7.4

run in your shell "npm update"

Strangely, I'm getting this error
Warning: Accessing PropTypes via the main React package is deprecated. Use the prop-types package from npm instead
even after having:
import PropTypes from 'prop-types';

and
Home.propTypes = {
  increment: PropTypes.func,
  count: PropTypes.number
};

Note: I'm having only 1 component in my project. So, that's not isssue.


Docs: Fix Typo

Fix Typo. TabPane Props is Tab Props

Note:
I am not sure but i think something is wrong in Tab Props/TabPane Props. I dont see any usage of the prop closeable. I briefly watched the source code, but dont have any time to dig deeper into it.


Size-optimize displacement map

The displacement map, g, is currently an array of uint32_t. This is almost always a waste of space as displacement values are usually less than 16 bits, and often less than 8 bits. When generating or loading the hash function, chose the type of g based on the base-2 log of d_max.

Compaction capability added with 2e32183b20fceffbeffef50592c1ab725eebcc87. Lua phf.new compacts by default with fe29c1b2d221803070b1ba92e78217aae205e4f0 refactoring.


demo user email is incorrect

At https://braindump.pw/auth/register instructions say to use<EMAIL_ADDRESS>when it should be<EMAIL_ADDRESS>

Thanks sorry about that :)
Sent from my iPhone

On Dec 6, 2015, at 20:22, Jeff Cressman<EMAIL_ADDRESS>wrote:
At https://braindump.pw/auth/register instructions say to use<EMAIL_ADDRESS>when it should be<EMAIL_ADDRESS>—
Reply to this email directly or view it on GitHub.


Add Rules and alerting capabilities doc

https://issues.redhat.com/browse/MON-2231
I think some of this will change soon, specially after the implementation of obsctl - but for now I wrote with the current setup we have

@ianbillett thanks for the review! Makes a lot of sense to add a section about token-refresher/auth, will do this next :)

@ianbillett I've added your suggestions, now is ready for another round of review :)
I'll wait for the Alertmanager configuration changes to get merged in production and once they are done, I'll update this doc to refer to prod environment instead - what do you think?


Record & show silver/gold numbers in project_stats

We'll delay adding showing these stats until #838 is implemented.

Issue #838 has been implemented and is in the production tier.  Now we can go back and add this (sensibly).


Add no-loc for Visual Studio

Fixes https://github.com/dotnet/docs.pt-br/issues/193

That's a good question and I asked Jason and he didn't have an answer except to say the fix was no-loc. 😄


containeranalysis: fix path

Containeranalysis in google-cloud-go recently got a new path - see https://code-review.googlesource.com/c/gocloud/+/31890

Sweet. Thanks! Can one of y'all merge this? I don't have powers :(

Can't get the package:
package cloud.google.com/go/containeranalysis/apiv1beta1: unrecognized import path "cloud.google.com/go/containeranalysis/apiv1beta1" (parse https://cloud.google.com/go/containeranalysis/apiv1beta1?go-get=1: no go-import meta tags ())
Fetching https://cloud.google.com/go/pubsub?go-get=1

Might need to wait a few minutes?

Master is also broken by this. :stuck_out_tongue:
package cloud.google.com/go/devtools/containeranalysis/apiv1beta1: cannot find package "cloud.google.com/go/devtools/containeranalysis/apiv1beta1" in any of:
	/usr/local/go/src/cloud.google.com/go/devtools/containeranalysis/apiv1beta1 (from $GOROOT)
	/tmpfs/src/gopath/src/cloud.google.com/go/devtools/containeranalysis/apiv1beta1 (from $GOPATH)


Simpler/straightforward examples - I couldn't get any of it to work.

I just argued with this thing for 30 minutes and could not get anything inside the ReactGridLayout object to ever render. I just gave up. If I can't get something very simple to work "out of the box" with some decent error reporting to explain WHY it isn't rendering anything, it is not worth any more of my time.

Thank you for your valuable contribution to this not-for-profit project that is run by volunteers in their free time.


distro-base: add ability to be build dependent images

Issue #, if available:
Description of changes:
When working locally i have wanted the ability to build an image along with the base image(s) it depends on. What i usually do is make minimal-images-base minimal-images-base-glibc minimal-images-base-iptables, which is annoying. I also think we could use this for presubmits to break them up a bit for this job, esp as we add more images.
Getting the order right for the various targets has always been an issue with all the wildcard rules and ive never really understood why. When introducing the dep building support, it was easier to go ahead and define most targets as actual targets, using variables since they are generated, to make the targets more clearly defined.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/lgtm
/approve


WIP LSTM layer

This would solve issue #644 by adding a LSTM layer to stax

@joaogui1 This looked exciting, why close without merging?


Leaderboard and Scoreboard should be mobile compatible

maybe use this:
https://stackoverflow.com/a/60811141


won't load on 1.8.7?

Successfully installed optimism-3.0.2
ruby-1.8.7-p352 :003 > require 'optimism'
SyntaxError: /home/rochkind/.rvm/gems/ruby-1.8.7-p352@umlaut2/gems/optimism-3.0.2/lib/optimism/parser.rb:184:     undefined (?...) sequence: /
           (?<brace>
           (\A|\n)[^\n]+<<-OPTIMISM_EOF[0-9]+
              (
                 (?!<<-OPTIMISM_EOF|OPTIMISM_EOF).
                |
                  \g<brace>
              )*
           OPTIMISM_EOF[0-9]+
          )/
        from /home/rochkind/.rvm/rubies/ruby-1.8.7-p352/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:36:in `gem_original_require'
    from /home/rochkind/.rvm/rubies/ruby-1.8.7-p352/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:36:in `require'
    from /home/rochkind/.rvm/gems/ruby-1.8.7-p352@umlaut2/gems/optimism-3.0.2/lib/optimism.rb:11
    from /home/rochkind/.rvm/gems/ruby-1.8.7-p352@umlaut2/gems/optimism-3.0.2/lib/optimism.rb:8:in `each'
    from /home/rochkind/.rvm/gems/ruby-1.8.7-p352@umlaut2/gems/optimism-3.0.2/lib/optimism.rb:8
    from /home/rochkind/.rvm/rubies/ruby-1.8.7-p352/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:58:in `gem_original_require'
    from /home/rochkind/.rvm/rubies/ruby-1.8.7-p352/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:58:in `require'
    from (irb):3

What version of ruby do you actually write for? I like your design of Optimism, but it seems to frequently stop working!

yes. It's only compatibility with Ruby 1.9. btw. they have different regexp engine in 1.9 and 1.8. and that's the error. When I'm writing the library, I use some some new features from 1.9. And I don't have time to migrate it to 1.8, I'm very glade if someone would like to help this out.
I'll add a note about it in REAME.
thanks

Okay, thanks. I think there's also a way to express the dependency on
1.9 in the gemspec so rubygems will refuse to install on 1.8, but I'm
not sure.

yes. I've add a require_ruby_version in .gemspec.
thanks.


Does this also affect NPC's?

Does this also affect NPC's? Otherwise, i like this idea.

You edited after I made the comment but would you rather them not affect NPC's? Just want to make sure I understand what you're saying

No, i actually would like them to be affected as well. (I should have used "anyway" instead of otherwise)