Add setEventListener method to HardwareDriver

@btelman96 I tried to test this in the remo.tv app by building an aar and by using a Gradle composite build, but both of those options gave me trouble. Could you publish a beta release?

@NoahAndrews 0.14.0-beta1
https://bintray.com/btelman96/maven/ControlSDK/0.14.0-beta1

@NoahAndrews 0.14.0-beta1
https://bintray.com/btelman96/maven/ControlSDK/0.14.0-beta1

yeah I just find it easier to create a beta build to test things, but would be nice to find a better flow for Remo. Goal of this was to use with multiple apps, but only really has use with Remo right now.

yeah I just find it easier to create a beta build to test things, but would be nice to find a better flow for Remo. Goal of this was to use with multiple apps, but only really has use with Remo right now.

Alright, it works as intended! I successfully sent a chat message from RemoBroadcaster.

Alright, it works as intended! I successfully sent a chat message from RemoBroadcaster.

@NoahAndrews https://bintray.com/btelman96/maven/ControlSDK/0.14.0

@NoahAndrews https://bintray.com/btelman96/maven/ControlSDK/0.14.0


[NUXT, Vue] => export 'default' (imported as 'LightweightCharts') was not found in 'lightweight-charts'

Lightweight Charts Version:  3.2.0
Introduction
Thank  you at first for the great library, it seems I found probably a tiny bug using it in combination with VueJS and Nuxt in server side rendering context. I already saw the other issue but I could not solve the using it in combination with VueJS and Nuxt.
Steps/code to reproduce:
I installed the library using npm with this command npm install --save lightweight-charts
Then I created plugin file for Nuxt which might be the way to go
import Vue from 'vue'
import LightweightCharts from 'lightweight-charts'

Vue.use(LightweightCharts)

Reproduction
npm run dev

Steps to do
When I save and run this, I got the following error. It seems it has sth. to do with the window object...
But to be honest I have no glue I'm trying this already now for two days and I cant figure it out.
The expected behavior would be that I can use the methods without any import error obviously.
Any help with is highly appreciated, thanks in advance.

I got the following error. It seems it has sth. to do with the window object...

What error?
Can you please create repro for this issue? Also, isn't it related to #543?

Hi @timocov,
exactly this relates #543
will close this issue.


TFLite/GPU tfl.strided_slice not supported - docs claim it is.

Click to expand!
Issue Type
Documentation Bug
Source
binary
Tensorflow Version
v2.8.0-0-g3f878cff5b6 2.8.0
Custom Code
Yes
OS Platform and Distribution
Google Colab
Mobile device
No response
Python version
3.7.13 (Google Colab)
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
According to the [documentation](https://www.tensorflow.org/lite/performance/gpu_advanced#supported_ops), a `strided_slice` operation should be supported when running Tensorflow Lite with a GPU Delegate.

Running sanity check with the [Tensorflow Lite Authoring Tool](https://www.tensorflow.org/lite/guide/authoring#checking_gpu_compatibility) raises Exception. How should one use this op, to benefit from GPU acceleration on Android?

Standalone code to reproduce the issue
# Colab: https://colab.research.google.com/drive/1ih1I3m1j4ZNnc7RKp7AzkLsM-1Cf9TUn?usp=sharing

import tensorflow as tf

# Declare dummy model with strided_slice op:
@tf.function(input_signature = [tf.TensorSpec(shape=(1, 2), dtype=tf.float32)])
def apply(x):
    return x[0, 0]

# Verify works
apply(tf.constant([[1., 2.]]))

# Fails on the GPU with authoring tool check
target_spec = tf.lite.TargetSpec()
target_spec.experimental_supported_backends = ["GPU"]
func_to_test = tf.lite.experimental.authoring.compatible(
    apply, converter_target_spec=target_spec, raise_exception=True
)
func_to_test(tf.ones(shape=(1,2)))

Relevant log output
'tfl.strided_slice' op is not GPU compatible: Slice does not support shrink_axis_mask parameter.
  - /usr/local/lib/python3.7/dist-packages/tensorflow/python/util/dispatch.py:1082
  - /usr/local/lib/python3.7/dist-packages/tensorflow/python/util/traceback_utils.py:150
  - <ipython-input-32-9f16c67cb20c>:6

COMPATIBILITY WARNING: op 'tfl.strided_slice' aren't compatible with TensorFlow Lite GPU delegate. https://www.tensorflow.org/lite/performance/gpu

Hi @sachinprasadhs ! Could you please look at this issue . It is replicating in 2.8, 2.9  and nightly . Thanks!

TFLite GPU is not 100% compatible with all the ops & options.  As the error message says:

Slice does not support shrink_axis_mask parameter.

You can look for that string in //tensorflow/lite/delegates/gpu/common/model_builder.cc.

@impjdi Are there plans to support this feature? in 2.3.0 such model works correctly in Tensorflow Lite + GPU.

Not being in the TFLite team myself, I have no idea what the process is and how the priorities are set.
re:

in 2.3.0 such model works correctly in Tensorflow Lite + GPU.

I'm not sure why it would.  Maybe tflite convert didn't populate those and instead produced different network that didn't require that.  Is it possible for you to be explicit with reshapes to eliminate the axis thingy?

@impjdi Thank you for the response

Is it possible for you to be explicit with reshapes to eliminate the axis thingy?

I need to access the last N (e. g. 5) elements of a tensor while preserving other dimensions:
x = inputs[:, :, :5]
is there a way to achieve this so that it works with Tensorflow Lite GPU Delegate?


Documentation: connection to Lustre

Hi! I watched your talk at Haskell Symposium 2020 -- good job! You mentioned Lustre several times. Especially it seemed crucial for the answer to the question about handling temperature hysteresis ("if you can solve it in Lustre, you can solve it in Haski"). Is it the file system? Would it make sense to mention it in README? If not, feel free to close the issue.

@ulysses4ever I believe the authors were referring to Lustre the programming language - http://www-verimag.imag.fr/The-Lustre-Programming-Language-and and not the file system.

@Abhiroop thanks for the clarification! I hope this could be added to the documentation.

Yes I agree, this is important for the documentation.

That's right, the reference is to the programming language Lustre. Interestingly, this comment was also given on the paper by one of the reviewers! I'll make a note of this in the README to avoid this confusion.


Moving elasticbeanstalk(2), imagebuilder(1), route53(1) resources to v1beta1 version

Moving elasticbeanstalk 2 resources to v1beta1 version

[ ] aws_elastic_beanstalk_application_version
[ ] aws_elastic_beanstalk_environment

Moving imagebuilder 1 resource to v1beta1 version

[ ] aws_imagebuilder_component

Moving route53  1 resource to v1beta1 version

[x] aws_route53_resolver_config

elasticbeanstalk - removed due these resources already tested.


esp_matter_attribute_utils::val_print() - missing data type (CON-434)

Source code: esp_matter_attrubite_utils.cpp
Function name: val_print
Issue: missing conditional statement for ESP_MATTER_VAL_TYPE_INT8
example:
I (301759) esp_matter_attribute: ********** W : Endpoint 0x0002's Cluster 0x00000202's Attribute 0x00000000 is <invalid type: 7> **********

The example log shared is the FanMode attribute (0x0000) of Fan Control Cluster(0x0202)and it is of the type enum8. The check for  ESP_MATTER_VAL_TYPE_ENUM8 is present in the val_print() function.
The missing conditional check for ESP_MATTER_VAL_TYPE_INT8 has been addressed in the PR 342.

@shripad621git OK, I got it. I'll wait for next commit. Thanks.


Update 8_cleanup.md

fixed section number and capitalization
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thank you for the fix.


could not install pathme, and I found a solution

Possibly, pathme (and viewer) cannot be installed with python 3.8 or greater. Perhaps due to 'compath'.
I tried in many ways and did not find a solution.
Looking at the docker example, I realize that it looks for python 3.65
Therefore, I did the following code that worked fine,
conda create -n net36_env python=3.6
conda activate net36_env
conda install -y -c anaconda numpy scipy jupyter
pip install --upgrade pip
python3 -m pip install pathme
python3 -m pip install pathme-viewer

Thank you for posting this!


Incorrect overlap on upper limit and displayed value

When the slider is mover to the lower range, it's displayed value overlaps nicely with the lower limit of the slider.  This does not occur on the upper limit, seemingly because the upper limit value is positioned towards the inside of the slider. A simple but non-elegant fix may be to position the upper limit to the right of the end of the slider.  Hope this helps, thanks for the software.

yes indeed see what can be done, thx

Still thinking about it ...  perhaps the current value could swap sides after it passes the middle of the slider.  that might be a cleaner solution.  regards

I see where it comes from but could you send me one or too screenshots, thx

regards

opps wrong button

will be fixed today normally for both horizontal or vertical sliders, I keep you in touch


MINOR: [C++] Name RunEnd[C]Type consistently

Minor change: renaming template parameters so it's clear whether arrow or C types are expected as argument.

@zeroshade


Check if SIGNATURE_FILE_NAME exists and use it as output_filename

Hi,
I've already discussed this PR with @psifertex on Slack.
Long story short: I'm working on a script to automatically create signatures from .LIB files (more info here: https://github.com/Vector35/sigkit/issues/28) and this PR basically allows me to set the filename for the resulting .SIG directly from my script.
Thanks,
Luca

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

lgtm

@psifertex do you know when it will be possible to download the new version containing this PR from the official plugin repository? Thanks a lot.

It should be live now.


Trouble sending object to a single field for multipart/form-data

Hi! I'm new to coding and am experiencing some issue with sending the following type of data:
const user = [{
            first_name: 'foo',
            last_name: 'bar',
            age: 23,
}];

My code:
RNFetchBlob.fetch('POST', `url here`, {
                // Accept: 'application/json',
                'Content-Type': 'multipart/form-data',
                Authorization: userToken
            },
            [
                // ...
                { name: 'user', data: JSON.stringify(user) },
                //...
            ]
        )
        .then((response) => {...

Structure from returned POST:
user:Array(1)
      0: Object
         first_name: ""
         last_name: ""
         age: ""

I'm basing it on the given note:
RNFetchBlob.fetch('POST', 'http://www.example.com/upload-form', {
    Authorization : "Bearer access-token",
    otherHeader : "foo",
    'Content-Type' : 'multipart/form-data',
  }, [
   //...
    { name : 'info', data : JSON.stringify({
      mail<EMAIL_ADDRESS>      tel : '12345678'
    })},
  ]).then((resp) => {
    // ...
  }).catch((err) => {
    // ...
  })

I tried using Postman and the data enters just fine. May I know what could be the possible issue?
I tried the following format and all returns empty:

[{first_name: 'foo',last_name: 'bar',age: 23}] (in an array)
{first_name: 'foo',last_name: 'bar',age: 23} (without an array)
with JSON.stringify()
without JSON.stringify()

Any help would be greatly appreciated. Thanks!

Support is hard to get here, since issues are for bugs, you may be better off going to https://stackoverflow.com


feat(3280): navigation banner at fair show page

Jira: FX-3280
Description
As a collector, I should be able to go back to the Fairs 'Exhibitors A-Z' tab after clicking through to view a Fairs 'Show'. Currently, there is no navigational banner making it explicit and easy for the collector to go back to the Fair page to continue browsing exhibitors within a Fair.
Figma
Still waiting Jonathan answer about focused border but pr is ready for review
Changes

Added navigation banner
Small refactor of fair components

Demo
https://user-images.githubusercontent.com/56556580/134356999-ce1b8cd1-9361-4b25-94d1-428fba855752.mov
https://user-images.githubusercontent.com/56556580/134358180-52d99e9a-77dd-4da2-888e-221a7d3611f1.mov

Design related question: we have this: https://github.com/artsy/force/blob/master/src/v2/Components/TopContextBar.tsx
Which is used in a few places and looks like:

Should we be using that?

@dzucconi what about TopContextBar, we have not a separator below a link, so I think we should not use it

@tr-ann yeah you're right. I guess I'm hinting at a larger discussion about whether or not we have a consistent pattern for this. Like there's a difference between an entity being 'part of' something larger and calling that out with TopContextBar vs a sub-page which has a back button.

@dzucconi yes, you are right. Should we discuss it with Jonathan?

I think it's sufficient to go with this PR and maybe just leave a comment in the Figma. Can always circle back. Show pages are another one which have this same back button pattern: https://www.artsy.net/show/moskowitz-gallery-transitive-horizon/info so it's consistent with that.


Change RPM build sync destination for AMD and ARM architecture

Issue #, if available:
Description of changes: Explanation As the part of the change, in this PR only the Makefile sync path will be changed. The download_golang.sh script pulls from the S3 bucket, thus if the Makefile change and the download_golang.sh scripts are changed in the same PR, the download_golang.sh script will try to pull from a path which doesn't exist yet.
Also, in this PR a new target clean is introduced to free space before the local-images target is run.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/test golang-1-18-tooling-presubmit
/test golang-1-19-tooling-presubmit

/test golang-1-18-tooling-presubmit

/approve


update win32 dependence (#275)

Fix #275.
Replace out of date pypiwin32 dependency with pywin32.
Removed specific version pin in appveyor as well.

Coverage remained the same at 97.186% when pulling dcd723c9b3a31e2a0f6f5afec4056f911ead95e3 on drewarnett:master into 62a7c89076a3b1d03cdce5f29824f3d051ce510e on peterbrittain:master.


Update raphael-koellner.yml

Learn Build status updates of commit 323d759:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

rd/raphael-koellner.yml
:x:Error

Details

rd/index.yml
:warning:Warning

Details

rd/toc.yml
:warning:Warning

Details

rd/raphael-koellner.yml

Line 2, Column 1: [Error: missing-attribute] Missing required attribute: 'connect'.
Line 23, Column 1: [Error: unknown-field] Could not find member 'Connect' on object of type 'Array'.

rd/index.yml

Line 1388, Column 8: [Warning: xref-not-found] Cross reference not found: 'rd.raphael-kollner'.

rd/toc.yml

Line 283, Column 10: [Warning: xref-not-found] Cross reference not found: 'rd.raphael-kollner'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 7c7b985:
:warning: Validation status: warnings

File
Status
Preview URL
Details

rd/index.yml
:warning:Warning
View
Details

rd/toc.yml
:warning:Warning
View
Details

rd/raphael-koellner.yml
:white_check_mark:Succeeded
View

rd/index.yml

Line 1388, Column 8: [Warning: xref-not-found] Cross reference not found: 'rd.raphael-kollner'.

rd/toc.yml

Line 283, Column 10: [Warning: xref-not-found] Cross reference not found: 'rd.raphael-kollner'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Docker Launch Error

Summary
Docker encountered exception error upon being launched
Description
I have checked that my Windows laptop fulfills all the prerequisites of using Docker, such as turning on the Virtual setting at BIOS and OS versions. However, I noticed that it might be the problem of installing 'Linux kernel update package' (which Docker prompted me to install upon the first launch). My laptop does not show any successful installations of such distributions in PowerShell by using    wsl --list command, even after I manually installed the update package through an exe. file. On another machine, once the update package was installed through the exe. file, Docker can run, and I can see the installed distributions on its PowerShell.
The full error code message
====
System.InvalidOperationException:
Failed to set version to docker-desktop: exit code: -1
stdout: The attempted operation is not supported for the type of object referenced.
stderr:
at Docker.ApiServices.WSL2.WslShortLivedCommandResult.LogAndThrowIfUnexpectedExitCode(String prefix, ILogger log, Int32 expectedExitCode) in C:\workspaces\main-merges\src[github.com](http://github.com/)\docker\pinata\win\src\Docker.ApiServices\WSL2\WslCommand.cs:line 160
at Docker.Engines.WSL2.WSL2Provisioning.d__8.MoveNext() in C:\workspaces\main-merges\src[github.com](http://github.com/)\docker\pinata\win\src\Docker.Engines\WSL2\WSL2Provisioning.cs:line 81
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Docker.Engines.WSL2.LinuxWSL2Engine.d__28.MoveNext() in C:\workspaces\main-merges\src[github.com](http://github.com/)\docker\pinata\win\src\Docker.Engines\WSL2\LinuxWSL2Engine.cs:line 177
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Docker.ApiServices.StateMachines.TaskExtensions.d__0.MoveNext() in C:\workspaces\main-merges\src[github.com](http://github.com/)\docker\pinata\win\src\Docker.ApiServices\StateMachines\TaskExtensions.cs:line 29
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Docker.ApiServices.StateMachines.StartTransition.d__5.MoveNext() in C:\workspaces\main-merges\src[github.com](http://github.com/)\docker\pinata\win\src\Docker.ApiServices\StateMachines\StartTransition.cs:line 67
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at Docker.ApiServices.StateMachines.StartTransition.d__5.MoveNext() in C:\workspaces\main-merges\src[github.com](http://github.com/)\docker\pinata\win\src\Docker.ApiServices\StateMachines\StartTransition.cs:line 92
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Docker.ApiServices.StateMachines.EngineStateMachine.d__14.MoveNext() in C:\workspaces\main-merges\src[github.com](http://github.com/)\docker\pinata\win\src\Docker.ApiServices\StateMachines\EngineStateMachine.cs:line 69
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Docker.Engines.Engines.d__22.MoveNext() in C:\workspaces\main-merges\src[github.com](http://github.com/)\docker\pinata\win\src\Docker.Engines\Engines.cs:line 108
Screenshots (if any):

After troubleshooting, this seems like a WSL2 issue. WSL2 wasn't able to start properly due to an issue with Winsock module. The solution to fix this is linked here:
https://github.com/microsoft/WSL/issues/4177
As a result, since docker engine was using WSL2 backend, the docker engine also failed to start. Once WSL2 is fixed, docker engine started normally.

For students who are on Windows machine, there is an option in the settings of Docker Desktop to turn off WSL2 backend. If you have already installed WSL2 and it is working fine, then it is ok to keep using the WSL2 backend. Otherwise, you may find using the HyperV backend to be easier since you don't have to install WSL2 backend.


It's wrong for 'N=C(O)CC'

It's wrong for 'N=C(O)CC'

do you mean ReferenceError: highlight_atoms is not defined?

@goldwind-ting
the right structure for "N=C(O)CC" should be:

while smileDrawer produce a structrue like this:

This is a style preference, not a bug (I too prefer the fuller structures).  You can get the output you want by disabling compactDrawing when you create your Drawer:
drawer = new SmilesDrawer.Drawer({
    // other stuff...
    compactDrawing: false
  })

@xavierholt Thanks a lot.


Add initial support for custom bar-id

Sets a default bar-id on all bars. This can be changed with the id sub-command (once implemented).

I have moved numlen() to common/util.c and merged it with sway/util.c which only had one function.
I have also updated the bar-id stuff to use numlen.
Hope it's ok to have all this in a single PR.

It's fine for this to all be in one PR. Thanks for taking care of that.


Beacon verification with non deterministic tx_power

Problem description
The oracle validation calculates the expected tx_power of a beacon based on the max_eirp for the region and the asserted gain of the gateway. This validation can fail because the gateway-rs code will report the beacon with the actual tx_power that it received from the packet forwarder in the tx_ack packet.
If the gateway-rs code requests a transmission with a power level the packet forwarder does not support the packet forwarder will attempt to find a lower power level that it does support. Due to a bug in the packet forwarder code it will pick a higher power level if no lower power level exists.
This change tries to solve several problems that are dependent on each other and require changes over several subsystems. The oracles should allow tx_power levels lower or equal to validate. The gateway-rs code should stop using MIN_CONDUCTED_POWER. The packet forwarder code should stop sending at a higher power level than requested. If the packet forwarder stops transmitting packets with too much power the low power regions such as EU868 and AS923 will not be able to transmit if their asserted gain is larger than 4dBi. To solve this the packet forwarder code must be changed to allow more than 16 power lookup table entries so that they can support the lower power levels while still supporting the higher power levels. Finally the packet forwarder configurations must be changed to include a baseline for the lower power levels.
oracles

[X] make beacon validation accept beacons with a lower or equal tx_power (https://github.com/helium/oracles/pull/381)

gateway-rs

[X] update beacon crate to allow checking for lower or equal tx power (https://github.com/helium/gateway-rs/pull/372)
[X] remove MIN_CONDUCTED_POWER (https://github.com/helium/gateway-rs/pull/373)

sx1301 packet forwarder

[ ] allow packet forwarder to have more 32 LUT entries (https://github.com/helium/lora_gateway/pull/4)
[ ] fix lut entry selection to only select power levels lower or equal to the requested power level
[ ] add configuration for the remaining power levels 0dBm - 12dBm

sx1302/sx1302 packet forwarder

[ ] allow packet forwarder to have more 32 LUT entries (https://github.com/helium/sx1302_hal/pull/34)
[ ] fix lut entry selection to only select power levels lower or equal to the requested power level (https://github.com/helium/sx1302_hal/pull/35)
[ ] add configuration for the remaining power levels 0dBm - 12dBm

comms

[ ] contact makers to update their packet forwarders to match the new behavior

This was addressed as part of https://github.com/helium/oracles/pull/381.  @mawdegroot if you consider the issue still to unresolved, please reopen the issue


Evaluation environments aren't being closed, processes accumulate

That's surprising. I expected it to be handled properly in the baselines VecEnv code.
For the moment, I would suggest a simple hack. I would get a list of envs and close them manually. It should be relatively easy to unwrap them.
I have an example here:
https://github.com/ikostrikov/pytorch-a2c-ppo-acktr/blob/master/enjoy.py#L31
I will be able to work on it on Monday (working on a deadline right now).

Oh.
Sorry, didn't notice PR :)


Update to jenkins version 2.138.1?

Is there any reason why master is still using jenkins-war-2.121.1.war when the LTS release of Jenkins is 2.138.1?

In Dockerfile there is just a default value. Release flows set it properly


Adding of a guide for bitcoin compiling on WSL

While installing the wsl the classic way, we could not be able to build bitcoin due to a lack of space in the C:\ drive.
That would also make impossible to run tests (units, regression and functional).
That's why it may be important to install WSL on a non-system drive in order to compile bitcoin core.

Good job @nourou4them
and thank you for your contribution, very useful 🔥


Propagate the original index in the export

David identified a problem in the export : the id being slugified, there is no identifying key left to join the data after the export.
The solution would be to add the original key in the export

Done


Looking for maintainers

Hi 👋  @mikestead
Is this project looking for maintainers? You don't have free time for this project so far 🙁

I think it's no longer necessary to maintain the extension because you can add the config below to your JSON settings file.
"files.associations": {
  "*.env": "properties"
}


Handle storage initializers

This is blocked by two things:

JSON ABI spec for how storage initializers should show up: https://github.com/FuelLabs/fuel-specs/issues/302
Compile-time constant evaluation: https://github.com/FuelLabs/sway/issues/407

What is the right approach here? Does adding a new array to the JSON file like this work?
[
   {
      "type":"storage_initializer",
      "location":"0x0101010101010101010101010101010101010101010101010101010101010101",
      "initial_value":"0x1234123412341234123412341234123412341234123412341234123412341234",
   },
   {
      "type":"storage_initializer",
      "location":"0x0101010101010101010101010101010101010101010101010101010101010102",
      "initial_value":"0x1234123412341234123412341234123412341234123412341234123412340000",
   }
]
[

   {
      "type":"function",
      "inputs":[
         {
            "name":"arg",
            "type":"u64"
         }
      ],
      "name":"entry_one",
      "outputs":[

      ]
   }
]

Note that constant evaluation isn't needed for allow initial storage for primitive types.

Yup, clarified the description.

The spec for that is now ready: https://github.com/FuelLabs/fuel-specs/blob/master/specs/protocol/storage_initialization.md. This can now be implemented in the compiler.


tf.image.extract_patches fails to JIT-compile inside keras model

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
Yes
Source
binary
Tensorflow Version
2.11.0
Custom Code
Yes
OS Platform and Distribution
Ubuntu 20.04, Mac OS X 12.6
Mobile device
No response
Python version
3.8
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
When trying to set jit_compile=True at model with Dense (Conv2D) layer followed by tf.image.extract_patches an error occured.

Standalone code to reproduce the issue
https://colab.research.google.com/drive/1-lXiMNp8NSe-c47-NAruypplraDId3O3?usp=sharing

Relevant log output
Detected unsupported operations when trying to compile graph __inference_run_step_742[] on XLA_GPU_JIT: SparseTensorDenseMatMul (No registered 'SparseTensorDenseMatMul' OpKernel for XLA_GPU_JIT devices compatible with node {{node gradient_tape/model/temp/SparseTensorDenseMatMul/SparseTensorDenseMatMul}}){{node gradient_tape/model/temp/SparseTensorDenseMatMul/SparseTensorDenseMatMul}}

</details>

Hi @shkarupa-alex !
I was able to replicate this bug with tf.function style and work-around is to use auto-clustering.
@SuryanarayanaY !
Could you look at this issue. Attached gist in 2.9, 2.10, 2.11 and nightly for reference.
Thank you!

The function tf.image.extract_patches is not supported on cloud TPU. Check out this link for more information https://cloud.google.com/tpu/docs/tensorflow-ops
@mohantym In you auto-clustering code, the code fails if we try to set jit_compile=True in model.compile function. Can you elaborate more about the auto-clustering code?

Hi @rajivhockey97 ,
XLA have some limitations.At present all the Ops are not compilable with jit_compile=True.There are also some known errors as documented here. The error might be probably due to either the Op is not compatible with Jit_compile or the shapes might be dynamic which is not supported by XLA.
If you choose auto_clustering then it will automatically chooses from CPU to GPU based on implementation which is not possible with explicit setting in model.compile(jit_compile=True) which forces to use only jit_compile and hence throwing the error. Please refer to autoclustering tutorial here. However autoclustering too have limitations at present like below as per source.

Note: In TF2, only the code inside tf.function will be clustered.

Note: Auto-clustering support on CPU and on multi-GPU environments is experimental.

It seems the shape of the Op tf.image.extract_patches is not inferrable at the time of compilation i.e. the output shape of the Op output can't be inferred without actual computation which might happen only during model.fit() in this case.Could you cross check and confirm?

@SuryanarayanaY Thanks for the explanation. Now it makes sense as to why the error shows up when jit_compile is set to true. However i am not sure how to identify whether the shape of the Op tf.image.extract_patches is inferrable at the time of the compilation or not.

Hi @rajivhockey97 ,
If the shape is not inferrable without some computation that may happens during model.fit() or say during training step then it is not available for compilation.
In our case we can check it by model.summary() before model.compile().If we don't get the proper shape(of course batch size might be neglected here since we pass it during training only) may be we can't use jit_compile there. Please refer the demo gist for our case.Hope it helps for you and comeback if still needs clarity.
Thankyou!

Hi @rajivhockey97 ,
If the shape is not inferrable without some computation that may happens during model.fit() or say during training step then it is not available for compilation. In our case we can check it by model.summary() before model.compile().If we don't get the proper shape(of course batch size might be neglected here since we pass it during training only) may be we can't use jit_compile there. Please refer the demo gist for our case.Hope it helps for you and comeback if still needs clarity.
Thankyou!

Even if we set constant shape for input layer in this gist (and all shapes in model summary become defined), it still fails with jit_compile=True

As far as i understand, this issue along with https://github.com/tensorflow/tensorflow/issues/59061 come from gradient implementation here https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/ops/array_grad.py#L1021

@shkarupa-alex ,
Even if the input shapes are known but if the function output shapes depends upon computation that happening inside the function then jit_compile=True may not work. For example let take the sample code of yours below:
    def call(self, inputs):
        outputs = self.d1(inputs)
        outputs = tf.image.extract_patches(outputs, [1, 16, 16, 1], [1, 8, 8, 1], [1] * 4, padding='SAME')
        outputs = self.d2(outputs)

        return outputs

The shape of the variable outputs might depends upon the parameters ([1, 16, 16, 1], [1, 8, 8, 1], [1] * 4) passed in to the Op tf.image.extract_patches . Here the shape only inferrable after calling with some input data which do some computation.This computation won't happen during Jit compilation. Hence you may get error.
If you can build the model such a way that the output shapes independent of actual computation then jit_compile should work.Please review once again and comeback if anything missing or needs clarification here.
Thankyou!

@SuryanarayanaY i figured out that issue comes from gradient computation.
Here is a fix https://github.com/tensorflow/tensorflow/pull/59185

I have validated my comment above.I tried to pass the Inputs with clear input shape (16, 16, 8) and model.summary() prints the output shape as (None, 2, 2, 1) which means all the shapes are inferrable at the compilation time itself.Please refer attached revised gist.
Model: "model_4"
_________________________________________________________________
 Layer (type)                Output Shape              Param #
=================================================================
 input_5 (InputLayer)        [(None, 16, 16, 8)]       0

 temp_4 (Temp)               (None, 2, 2, 1)           8481

=================================================================
Total params: 8,481
Trainable params: 8,481
Non-trainable params: 0

Hence this problem is not related to shape inference as suspected earlier by me.May be this is valid bug which needs fix.Hoping the above commit by @shkarupa-alex fixes this issue.Thanks!


Should there be SegmentedBuf::with_capacity?

Similarly, reserve and maybe clear (which, I assume, is x.advance(x.remaining())).
Workaround: into_inner() + From<VecDeque>::from, except of the cached length.

Hello
I'm not entirely sure if it's very useful (the use case is, as described by the docs, that one doesn't really know the sizes upfront). Furthermore, the name with_capacity might be a bit ambiguous ‒ would it be capacity for X individual (arbitrary large) buffers or would it be for X bytes total (internally, only the former would be possible, but casual reader of the code could assume the latter). Similar naming ambiguity is with reserve.
That being said, if someone was to submit a pull request with clear and equivalent of the methods with some less ambiguous names, I'm not opposed to having them there.


Container not starting on AWS Ubuntu

Description
vfarcic/docker-flow-proxy:latest not starting
Steps to reproduce the issue:

docker swarm init (only using a single note atm)
docker network create --driver overlay proxy
docker stack deploy -c dfproxy.yml proxy (where dfproxy.yml is the same as https://github.com/vfarcic/docker-flow-proxy/blob/master/docker-compose-stack.yml but only using 1 replica as I only have one node)

version: "3"

services:

  proxy:
    image: vfarcic/docker-flow-proxy
    ports:
      - 80:80
      - 443:443
    networks:
      - proxy
    environment:
      - LISTENER_ADDRESS=swarm-listener
      - MODE=swarm
    deploy:
      replicas: 1

  swarm-listener:
    image: vfarcic/docker-flow-swarm-listener
    networks:
      - proxy
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    environment:
      - DF_NOTIFY_CREATE_SERVICE_URL=http://proxy:8080/v1/docker-flow-proxy/reconfigure
      - DF_NOTIFY_REMOVE_SERVICE_URL=http://proxy:8080/v1/docker-flow-proxy/remove
    deploy:
      placement:
        constraints: [node.role == manager]

networks:
  proxy:
    external: true

Describe the results you received:
vfarcic/docker-flow-proxy:latest dies after approx 15 seconds. repeated restarts.
CONTAINER ID        IMAGE                                       COMMAND                  CREATED             STATUS                            PORTS                       NAMES
fb28cdccc74c        vfarcic/docker-flow-proxy:latest            "/sbin/tini -g -- do…"   11 seconds ago      Up 4 seconds (health: starting)   80/tcp, 443/tcp, 8080/tcp   proxy_proxy.1.j0xax9cfseahgmbz8b78186qs
1f0c6d440c0e        vfarcic/docker-flow-swarm-listener:latest   "docker-flow-swarm-l…"   6 minutes ago       Up 5 minutes (healthy)            8080/tcp                    proxy_swarm-listener.1.un37vvwowk8mxuswbd6sft4gu
ubuntu@ip-10-0-0-125:~$ docker ps
CONTAINER ID        IMAGE                                       COMMAND                  CREATED             STATUS                            PORTS                       NAMES
fb28cdccc74c        vfarcic/docker-flow-proxy:latest            "/sbin/tini -g -- do…"   16 seconds ago      Up 9 seconds (health: starting)   80/tcp, 443/tcp, 8080/tcp   proxy_proxy.1.j0xax9cfseahgmbz8b78186qs
1f0c6d440c0e        vfarcic/docker-flow-swarm-listener:latest   "docker-flow-swarm-l…"   6 minutes ago       Up 6 minutes (healthy)            8080/tcp                    proxy_swarm-listener.1.un37vvwowk8mxuswbd6sft4gu
ubuntu@ip-10-0-0-125:~$ docker ps
CONTAINER ID        IMAGE                                       COMMAND                  CREATED             STATUS                             PORTS                       NAMES
fb28cdccc74c        vfarcic/docker-flow-proxy:latest            "/sbin/tini -g -- do…"   21 seconds ago      Up 14 seconds (health: starting)   80/tcp, 443/tcp, 8080/tcp   proxy_proxy.1.j0xax9cfseahgmbz8b78186qs
1f0c6d440c0e        vfarcic/docker-flow-swarm-listener:latest   "docker-flow-swarm-l…"   6 minutes ago       Up 6 minutes (healthy)             8080/tcp                    proxy_swarm-listener.1.un37vvwowk8mxuswbd6sft4gu
ubuntu@ip-10-0-0-125:~$ docker ps
CONTAINER ID        IMAGE                                       COMMAND                  CREATED             STATUS                      PORTS                       NAMES
fb28cdccc74c        vfarcic/docker-flow-proxy:latest            "/sbin/tini -g -- do…"   24 seconds ago      Up 17 seconds (unhealthy)   80/tcp, 443/tcp, 8080/tcp   proxy_proxy.1.j0xax9cfseahgmbz8b78186qs
1f0c6d440c0e        vfarcic/docker-flow-swarm-listener:latest   "docker-flow-swarm-l…"   6 minutes ago       Up 6 minutes (healthy)      8080/tcp                    proxy_swarm-listener.1.un37vvwowk8mxuswbd6sft4gu

service shows 0 replicas for proxy_proxy:
ubuntu@ip-10-0-0-125:~$ docker service ls
ID                  NAME                   MODE                REPLICAS            IMAGE                                       PORTS
5rx22hus0gyr        proxy_proxy            replicated          0/1                 vfarcic/docker-flow-proxy:latest            *:80->80/tcp, *:443->443/tcp
n0o5z1acpxw6        proxy_swarm-listener   replicated          1/1                 vfarcic/docker-flow-swarm-listener:latest

service logs don't have anything particularly useful
ubuntu@ip-10-0-0-125:~$ docker service logs proxy_proxy
proxy_proxy.1.tyfqo5rw7tuy@ip-10-0-0-125    | 2018/03/20 11:08:05 Starting HAProxy
proxy_proxy.1.mcnynftz808r@ip-10-0-0-125    | 2018/03/20 11:09:46 Starting HAProxy
ubuntu@ip-10-0-0-125:~$ docker service logs proxy_proxy
proxy_proxy.1.i0pifcakpnvv@ip-10-0-0-125    | 2018/03/20 11:10:11 Starting HAProxy
proxy_proxy.1.tyfqo5rw7tuy@ip-10-0-0-125    | 2018/03/20 11:08:05 Starting HAProxy
ubuntu@ip-10-0-0-125:~$ docker service logs proxy_proxy
proxy_proxy.1.tyfqo5rw7tuy@ip-10-0-0-125    | 2018/03/20 11:08:05 Starting HAProxy
proxy_proxy.1.i0pifcakpnvv@ip-10-0-0-125    | 2018/03/20 11:10:11 Starting HAProxy
ubuntu@ip-10-0-0-125:~$ docker service logs proxy_proxy
proxy_proxy.1.tyfqo5rw7tuy@ip-10-0-0-125    | 2018/03/20 11:08:05 Starting HAProxy
proxy_proxy.1.lzk35lbem1x0@ip-10-0-0-125    | 2018/03/20 11:10:36 Starting HAProxy

Describe the results you expected:
2 containers running in the service
Additional information you deem important (e.g. issue happens only occasionally):
tried a couple of arbitrary versions going back several months. works using vfarcic/docker-flow-proxy:17.10.12-41 (maybe later versions also, just glad I've got a working proxy now :))
Additional environment details (AWS, VirtualBox, physical, etc.):

using Ubuntu on AWS 16.04.4 LTS (GNU/Linux 4.4.0-1052-aws x86_64)
brand new install of docker using convenience script as of yesterday
using docker swarm

bit extra info: all containers are stopping with error "task: non-zero exit (143): dockerexec: unhealthy container":
ID                          NAME                IMAGE                                                                                NODE                DESIRED STATE       CURRENT STATE               ERROR                                                          PORTS
lqq3eqo1kudz786dqf3leejvl   proxy_proxy.1       vfarcic/docker-flow-proxy:latest@sha256:47fb1b619478da4745cd6c0b422f9186abc902b61b6f17332da79bd6804c469d   ip-10-0-0-110       Running             Starting 18 seconds ago
zmzvahdtvf5lncqbp87toanwm    \_ proxy_proxy.1   vfarcic/docker-flow-proxy:latest@sha256:47fb1b619478da4745cd6c0b422f9186abc902b61b6f17332da79bd6804c469d   ip-10-0-0-110       Shutdown            Failed about a minute ago   "task: non-zero exit (143): dockerexec: unhealthy container"
z1schwkyrtjia6skk8qplyvrl    \_ proxy_proxy.1   vfarcic/docker-flow-proxy:latest@sha256:47fb1b619478da4745cd6c0b422f9186abc902b61b6f17332da79bd6804c469d   ip-10-0-0-110       Shutdown            Failed 2 minutes ago        "task: non-zero exit (143): dockerexec: unhealthy container"
v9ugtdriiotplobgtgtxtgweq    \_ proxy_proxy.1   vfarcic/docker-flow-proxy:latest@sha256:47fb1b619478da4745cd6c0b422f9186abc902b61b6f17332da79bd6804c469d   ip-10-0-0-110       Shutdown            Failed 2 minutes ago        "task: non-zero exit (143): dockerexec: unhealthy container"
x48rtz5bkovdw5yqp2xx4dvn1    \_ proxy_proxy.1   vfarcic/docker-flow-proxy:latest@sha256:47fb1b619478da4745cd6c0b422f9186abc902b61b6f17332da79bd6804c469d   ip-10-0-0-110       Shutdown            Failed 5 minutes ago        "task: non-zero exit (143): dockerexec: unhealthy container"

I was having seemingly unrelated trouble with @hamburml's docker flow lets encrypt companion which was unable to connect to the acme server. turned out to be a dns resolution issue from within the container which was resolved using instructions here.
After figuring that one out, thought I'd give :latest another go and it works fine :)
Guessing somewhere between version 17.10.12-41 and now, dfproxy started needing internet access from within the container or it'd fail.

I think I'm having the same exact problem as you were, can you explain what settings you changed on your EC2 box? @unclebob418

I followed the instructions here:
https://aws.amazon.com/premiumsupport/knowledge-center/ec2-static-dns-ubuntu-debian/

I read that, I meant what settings did you define for DNS. Did you just set it to Google's DNS or did you have to set your domain? My current EC2s /etc/resolv.conf look like this:
root@ip-10-0-57-186:~# cat /etc/resolv.conf
# Dynamic resolv.conf(5) file for glibc resolver(3) generated by resolvconf(8)
#     DO NOT EDIT THIS FILE BY HAND -- YOUR CHANGES WILL BE OVERWRITTEN
nameserver <IP_ADDRESS>
search ec2.internal

When I go into the DF container I can't do an apk update or ping google.com so I also don't think they have internet access.

I used Google's DNS - not sure the details of how exactly I did it sorry - this was over a year ago remember.


Any way to print the version number

Is there a way to print CoreLocationCLI's version number?
Something like CoreLocationCLI -version would be nice.
Currently I'm using the file's CRC-32 value to compare a bundled CoreLocationCLI with the version the user has installed.

Good idea. This is in project scope.

Here is what I'm talking about with the crc32: https://github.com/JayBrown/Cuckoo-Menu/blob/master/cckm.6h.sh#L110-L126

Is there a slick way in Swift to get that from the package.swift?


Container/reporter for reporting host utilization statistics

Is your feature request related to a problem? Please describe.
Reporting host utilization statistics (CPU, RAM, etc.).
It would be nice to see some quick overall system load info on the main overview page, that way you can get a feel for if everything is running ok on the underlying host.
Describe the solution you'd like
Exhibit such information e.g., in ARTEMIS page footer (overview).
Even a simple txt line after the Module status with the normal 5/10/15 load avg info and maybe memory.
To do this:

use an available or custom reporting container, instructing it to report stats let's say every second
connect it with rabbitmq
instruct it to send messages to the db
the db will store them in a "host_stats" table
the frontend will use graphql to access these statistics

Ideally we would need:
CPU, RAM and DISK for the host machine, and CPU and RAM per container (e.g., backend, postgres) and per module (e.g., backend monitor, etc.).
Per-module stats could be placed in the overview table upper right, host-wide stats at the bottom of the page, and per-container stats are optional (maybe they need their own page).
We need a lightweight solution!
Describe alternatives you've considered
https://github.com/google/cadvisor (BUT: high CPU usage!)
https://github.com/stefanprodan/dockprom (BUT: too much info, heavy)
https://github.com/pitrho/docker-host-stats (sth along these veins...)
Additional context
https://github.com/pitrho/docker-host-stats
if you run:
docker run -v=/proc:/proc_host:ro pitrho/docker-host-stats -cdmn -f 1

you get every second:
{"message": "Reporting stats using cAdvisor: False", "host-stats": {"disk": {"/": {"status": "OK", "total": 486.62, "percent": 86.2, "free": 42.46, "used": 419.42}}, "network": {"status": "OK", "interfaces": {"allnic": {"packets_sent": 9, "packets_recv": 11, "bytes_recv": 1012, "dropout": 0, "bytes_sent": 572, "errout": 0, "errin": 0, "dropin": 0}}}, "hostname": "ac001afb8312", "cpu": {"status": "OK", "percent_combined": 28.4, "percent_per_cpu": [52.4, 15.8, 15.8, 15.8]}, "memory": {"available": 7.7, "status": "OK", "used": 15.17, "percent": 53.9, "free": 1.54, "total": 16.7}}}

This dictionary can be sent to the db via rabbitmq and update a table row every second.

This functionality is provided by other monitoring tools which are supposed to co-exist with ARTEMIS. (kubernetes, nagios, etc.)

After internal discussion we decided that the proposed functionality can be provided by other monitoring tools which are supposed to co-exist with ARTEMIS. (kubernetes, nagios, etc.)


scene viewer improvements: animation reset

Objective

Changing animation mid animation can leave the model not in its original position
The movement speed is fixed, no matter the size of the model

Solution

when changing animation, set it to its initial state and wait for one frame before changing the animation
when settings the camera controller, use the camera transform to know how far it is from the origin and use the distance for the speed

Tested this out, I like the changes! The movement speed here is slower than before on the flight helmet model, but I'm a fan of that change.

bors r+

bors r+


[Feature Request] Prefer to use configured Tracker's manga name to get recommendation.

Why/User Benefit/User Problem
As the current recommendation works, Recommendation feature is just pass through the title name in the library(Manga.title).
But, If the source provides a translated title(non-english, non-romaji). The recommendation feature failed because it won't match any of title. so it fails.
Even I configured the tracker hand by hand too. Because I mentioned, It just pass through.
What/Requirements
So, Firstly, Look up the currently configured tracker (side of each manga), If matches AniList or MyAnimeList, See the recommendation of it firstly.
If nothing is registered on tracker, then uses current way.

Recs work for any manga regardless of tracker, as tracker only works for manga in library. Also, the recs are chosen in current way because MAL has more manga suggestions than Anilist and Anilist has more+better webtoon suggestions than MAL. Tracker is completely unrelated to it.

I don't know what recs is. (When I looked up is, It uses AniList API.)
But It's not just English title. If manga uses Non-English(Maybe some asian language.), The recommendation is broken as it is not matched anything with their dbs.

@she11sh0cked Can you have a look into this bug?

Added support for using the tracked manga id for the recs


Add df.ww.to_parquet method to serialize to a single file

Add df.ww.to_parquet method to serialize to a single file

Closes #1223
Closes #1225

Adds a new accessor method to serialize a Woodwork dataframe to the specified parquet file. Saves all Woodwork typing information and other metadata in the parquet file metadata. Also adds a new method read_parquet to deserialize the saved information back into a Woodwork initialized dataframe.

Why treat this is as a separate serialization method instead of updating the parquet approach of the to_disk method?

We could do that, but I think that pretty much means a ground-up re-write of to_disk and all the other utility functions that are used along the way (both read and write) to handle this type of flow. Currently to_disk does not support users specifying the output filename, and I think that is something we want here. Handling parquet files in to_disk like they are here would mean that in all the methods and functions we would need unique paths for parquet files because they would work differently than the other formats.
As a bit of a tangent, I'm not sure having a monolithic to_disk method is a great approach long term. Because of all the logic branches, it does make it a bit difficult to follow what happens for a given file format, compared to having individual to_format methods. I guess that is probably a conversation for a different day though...

Closing this MR as we have decided to refactor the serialization approach in Woodwork. That refactor will have a significant impact on the changes needed to serialize to a single parquet file, and will be better to handle in a new PR after the serialization refactor is complete.


Standalone should use crc32c by default as well

Motivation
Fixed the default for digest type to match what we're already using in regular broker.

run cpp tests
run java8 tests
run integration tests


MouseyPad

Ready to make a PR? Great! Make sure your directory structure looks like this:
├── CAD
│   ├── case.stl
│   └── plate.dxf
├── firmware
│   └── rp2040.bin
└── PCB
├── macropad.pro
└── macropad.sch
(The file names do not matter, just the folders)

Great job! You can request your Hackpad here: https://forms.hackclub.com/hackpad :)


-X still doesn't seem to work

Okay so this is what I am trying to do
" java -jar muda-2.0.0-DEV1.jar -t 8209 -b s -X 4 -N asd135 -d "C:\pathtothefiles" "
but when I hit enter it just post one file instead of 4. Also tried it with 5.
according to cmd I am using java version "1.8.0_71"

Hey there, thanks for the report.
I never got around to implementing -X unfortunately. 8chan's instability has made any kind of testing on this feature remarkably difficult. Someone else reported the same thing in #9.

Damn, I really wish it would have worked. What about that muda-ruby thing? I couldn't get it to work, but if it worked, would it be able to post several pics in one post?

Generally that means that the site burped. That message comes up if the site returns a 404 for the thread. In a sane world, you'd only see that if the thread was really nuked, but in unstable-8chan-land, it comes up pretty often.
The ruby version does have working maxfile support, but it hasn't been tested in ages and I think it's somehow less tolerant of errors than this one. If you want to download it and try, I can probably walk you through setting it up.

Does the ruby version only work on linux? I can't seem to get it to work. Problem is I don't even know how to execute it.
./muda.rb -t 8830 -b s -d "C:\My Files\Pictures" does obviously not work, but I don't know what else to try either.
Also, isn't there a way to force it to continue? Like if there is a 404 error that it just ignores it and continues with the next file? Or sometimes it happens that it uploads a image, but it thinks it didn't upload it, then the script stops there because it can't continue. Is there a way to make it continue where it stopped the last time?

Yes on both counts. The first thing you need is a copy of ruby if you don't have one already - if you do, then the ./muda.rb that you mentioned should Just Work assuming you did the bundle install stuff in that version's instructions.
As far as forcing it to continue, yes. Note that every post will have a number associated with it. If you run the exact same command again with --resume (that number where it stopped), it should pick up where it left off. There's also --maxfailures that you could increase to a large number to work around it.
I can probably give you better instructions if you tell me exactly what error you're getting back from the script, though!

I don't quite understand the -R command. What number do I have to write after it?
I put this into cmd
java -jar muda-2.0.0-DEV1.jar -t 5648607 -b b -D 1 -x 500 -E 5 -d "G:\My Files\Pictures\2clovers.com\AquaBlanka-Part1-2clovers.com-2013"
script stops at 5/95 and says that the thread doesn't exist. Now if I want to continue what do I write? -R and then 6 because it stopeed at 5? Do I write the post id of the last post like 5670784?

Okay, so it stopped at image 5 - reuse the exact same command you used before, but add -R 5 on it.

When I write -resume it says unknown option. and when I try -R 5 it just starts at the first image like always. Does resume only work on ruby version?


Add pmon to list

Hello, your token doesnt currently meet the liquidity or volume requirements to be listed. Please build up liquidity in the pool on TJ and target healthy/sustained volumes

Please reopen or resubmit at a later date when your token meets the tokenlisting criteria


runYarn fails on Windows 10

I have been able to reproduce this on two Windows 10 systems so far attempting to build the same project. The project works on a mac.
When running the build the following error occurs:
error An unexpected error occurred: "https://registry.yarnpkg.com/babel-runtime/-/babel-runtime-6.26.0.tgz: ENOENT: no such file or directory, utime 'C:\\users\\admin\\_bazel_admin\\wjqdalcp\\execroot\\services_core\\node_modules_cache\\v1\\npm-babel-runtime-6.26.0-965c7058668e82b55d7bfe04ff2337bc8b5647fe\\core-js\\reflect\\get-metadata-keys.js'".
child_process.js:644
    throw err;
    ^

Error: Command failed: yarn --cwd bazel-out/x64_windows-fastbuild/bin/external/bazel_javascript/internal/packages_installed_dir --ignore-scripts --cache-folder ./node_modules_cache --frozen-lockfile
    at checkExecSyncError (child_process.js:601:13)
    at Object.execSync (child_process.js:641:13)
    at runYarn (C:\users\admin\_bazel_admin\wjqdalcp\external\bazel_javascript\internal\common\run_yarn.js:7:17)
    at Object.<anonymous> (C:\users\admin\_bazel_admin\wjqdalcp\external\bazel_javascript\internal\npm_packages\install.js:30:1)
    at Module._compile (module.js:635:30)
    at Object.Module._extensions..js (module.js:646:10)
    at Module.load (module.js:554:32)
    at tryModuleLoad (module.js:497:12)
    at Function.Module._load (module.js:489:3)
    at Function.Module.runMain (module.js:676:10)

I checked manually and C:\\users\\admin\\_bazel_admin\\wjqdalcp\\execroot\\services_core\\node_modules_cache\\v1\\npm-babel-runtime-6.26.0-965c7058668e82b55d7bfe04ff2337bc8b5647fe\\core-js\\reflect\\get-metadata-keys.js does exist.
Versions
Windows 10 Pro: 10.0.17134 Build 17134
bazel: 0.16.1 (current chocolatey package)
bazel-javascript: 0.0.23
build_bazel_rules_nodejs:  0.10.0
path nodejs: 10.9.0 (current chocolatey package)
path yarn: 1.9.4

I am going to keep trying to figure out how to fix this, but any pointers are appreciated.

Do you think that this should be using @nodejs://:yarn rather than the one installed on the path? Or is the one on the path being used so that it uses the same version that the developer currently is?

It should be using @nodejs://:yarn, the priority is consistency across
developers of a team. I didn't realise it wasn't the case!
On Thu., 30 Aug. 2018, 14:01 Rich Adams<EMAIL_ADDRESS>wrote:

Do you think that this should be using @nodejs://:yarn rather than the
one installed on the path? Or is the one on the path being used so that it
uses the same version that the developer currently is?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zenclabs/bazel-javascript/issues/20#issuecomment-417182760,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAvJ2j_OSKqB-na0dGtmYY7XW8yJJ3z_ks5uV2ObgaJpZM4WSnDr
.

I just tried it out and I think that I was wrong.  It seems to be using @nodejs//:yarn even though it isn't getting passed its location. I think that I don't fully understand if bazel modifies the PATH of actions created through actions.run or if @nodejs//:node modifies its the PATH somehow.

I was able to get past the caching issue by resolving the cache path (see this commit).
But now am running into this error:
server_lib_transpilation_src failed (Exit 255): node.cmd failed: error executing command
  cd C:/users/admin/_bazel_admin/wjqdalcp/execroot/services_core
  SET NODE_PATH=bazel-out/x64_windows-fastbuild/bin/external/bazel_javascript/internal/packages_installed_dir/node_modules
  external/nodejs/bin/node.cmd external/bazel_javascript/internal/ts_library/create_full_src.js //services/products/server:server_lib bazel-out/x64_windows-fastbuild/bin/packages_installed_dir tsconfig.json  services/products/server/BUILD.bazel|services/products/server/createDb.ts|services/products/server/products.ts|services/products/server/server.ts bazel-out/x64_windows-fastbuild/bin/services/products/server/server_lib_transpilation_src
'services' is not recognized as an internal or external command,
operable program or batch file.

I think Node and/or Windows' shell might be confused by the "//services/products/server:server_lib" you have in the second argument to node.cmd?

Is that supposed to be expanded somehow before going to create_full_src.js?

No, it's put into a targetLabel variable that is only used for error reporting when compiling, e.g. the following in js_library/create_full_src.js:
// Copy source code.
for (const src of srcs) {
  if (!src) {
    continue;
  }
  if (!fs.existsSync(src)) {
    console.error(`
Missing file ${src} required by ${targetLabel}.
`);
    process.exit(1);
  }
  const destinationFilePath = path.join(destinationDir, src);
  safeSymlink(src, destinationFilePath);
}

I think that this is a problem with the sources that are joined with | on windows if I run:
echo Hi|there

I get a similar error:
'there' is not recognized as an internal or external command,
operable program or batch file.

Nice find. Let's use another separator?

I'm working on wrapping the arguments with quotes. If I do that windows will leave it alone.
The scripts need to be changed a little though, because in windows the shell doesn't consume the quotes, it passes them through.

@fwouts So I probably pulled more threads than I should have on this one. But in researching actions.run it seems like we should have a param file. So to facilitate that I started creating a JsContext provider to wrap a rule context similar to how rules_go does it. Do you think this is a good way to go?

What do you mean by a "param file"?
Either way, the JsContext provider seems like a good idea! It sounds like it would be useful on its own as a first PR to clean up the existing code?

The Args.use_param_file() option allows args objects created by ctx.actions.args() to overflow into a param file if they will be larger than the systems params limits. I think they are also more performant because they allow you to add depsets as parameters.
And yeah, I should probably break this work into a set of smaller PRs.


Refactor and harmonizing DID method options

Overview
Refactored according to offline conversation.
Description
This PR

Removes the option of passing updateKeyJwk and recoveryKeyJwk.
Allows for passing verification methods that can be generated by the keyManager inside ion.
CommonOptions contains the common options across different operations.
Renamed to use Did<Method>Api pattern.
Renamed Did in favor of StatefulDid.
Harmonized usage across different did method implementations.
Updated documentation.

How Has This Been Tested?
Unit tests still pass. More tests were added.

I think i walked away from our discussion earlier this week with something a bit different in mind. trying to figure out the best way to convey

alright @andresuribe87 lmk if i'm off the mark here:
I think both of us have different use-cases in mind and i think both are valid! I'm confident we can find a way to support both
My desired Api surface. Use-case: services creating and primarily using a single DID ->
// creation
val did = DidIon.create(km, opts)

// resolution - 2 paths
val resolutionResult = did.resolve()
DidIon.resolve("did:ion:whatever")

// updating - 2 paths
did.update(opts)
DidIon.update("did:ion:whatever", km, opts)

// recovering - single path.
DidIon.recover("did:ion:whatever", km, opts)

maybe two paths are possible? though it seems weird to be able to call `did.recover()`

// deleting - 2 paths
did.delete(opts)
DidIon.delete("did:ion:whatever", km, opts)

// instantiating pre-existing did
val did = DidIon("did:ion:whatever", km, opts)

the instance methods simply use the static methods under the hood.
Now, I believe what this is missing that you would like to have is primarily two features:

being able to set the host once for all subsequent static DidIon.* fn calls
being able to set a keymanager once for all subsequent static DidIon.* fn calls

if i understand correctly, the motivation behind wanting the two missing features is that you're thinking about usage scenarios where multiple dids have to be managed (e.g. building an SSI Service)
are my assumptions correct? and more importantly, if we figure out how to support the two missing features without changing the api surface i've described, can we satisfy both use-cases?

@mistermoe I think we're close to something that's going to be very robust! 🙌

alright @andresuribe87 lmk if i'm off the mark here:
I think both of us have different use-cases in mind and i think both are valid! I'm confident we can find a way to support both
My desired Api surface. Use-case: services creating and primarily using a single DID ->
// creation
val did = DidIon.create(km, opts)

// resolution - 2 paths
val resolutionResult = did.resolve()
DidIon.resolve("did:ion:whatever")

// updating - 2 paths
did.update(opts)

Does updateKeyAlias live inside the opts object? I view this as an important detail because I don't know of a way of determining what the update key (even the public key) is by only looking at the DID Document (do you know if it's possible?). I have similar concerns with other operations below.

DidIon.update("did:ion:whatever", km, opts)

// recovering - single path.
DidIon.recover("did:ion:whatever", km, opts)

maybe two paths are possible? though it seems weird to be able to call `did.recover()`

Out of curiousity, where do you see the differences between recover and update (i.e. what exactly is weird about it)?

// deleting - 2 paths
did.delete(opts)
DidIon.delete("did:ion:whatever", km, opts)

// instantiating pre-existing did
val did = DidIon("did:ion:whatever", km, opts)

the instance methods simply use the static methods under the hood.

Whether methods are static or not, I think that's an implementation detail. I would suggest focusing more on the syntax that we want to support.

Now, I believe what this is missing that you would like to have is primarily two features:

being able to set the host once for all subsequent static DidIon.* fn calls

Yes! And also the httpClient engine (or the client itself). For a long lived service which is doing resolution, there is a benefit of not creating a ktor HttpClient per request (see here). I've seen this pattern in many languages, so I assume the same is true if we were to use OkHttp.

being able to set a keymanager once for all subsequent static DidIon.* fn calls

Almost! I agree with the spirit, but I don't think we need to impose whether some methods are static or not. As I mentioned above, I would want to support the specific syntax.
Note that I didn't address that in this PR as we decided to punt in our last sync.

if i understand correctly, the motivation behind wanting the two missing features is that you're thinking about usage scenarios where multiple dids have to be managed (e.g. building an SSI Service)

Yes! I think a wallet app would also want to manage multiple dids?

are my assumptions correct? and more importantly, if we figure out how to support the two missing features without changing the api surface i've described (can certainly be additive), can we satisfy both use-cases?

Most of the assumptions are correct, thanks for putting together this nice summary!
I have an additional desire to use names that reduce future confusion for the abstractions we're creating.
Finally, a totally separate concern is that I think we should be fostering the explicit practice of using separate KeyManager instances to store the recovery vs other keys. I don't think that should be optional. I didn't actually address that in this PR.
Would love to hear any alternatives to handle the above!

Does updateKeyAlias live inside the opts object? I view this as an important detail because I don't know of a way of determining what the update key (even the public key) is by only looking at the DID Document (do you know if it's possible?). I have similar concerns with other operations below.

yeah ideally as optional. just spitballing here: it seems like our ion implementation can just default to a predetermined alias if one isn't provided e.g. ${shortFormDid}:update given that there's only 1 update key at any given time. same pattern for recovery.
sounds like a detail people shouldn't have to worry about unless they explicitly want to. do you have any reservations or hesitations with that approach?

Out of curiousity, where do you see the differences between recover and update (i.e. what exactly is weird about it)?

oh this was just out of ignorance actually. i hadn't looked into what recover actually meant in the context of ion. but it makes sense now. prior to reading the sidetree spec, i thought "hmmm how would i ever have an instance of the did if i am trying to recover it? doesn't it mean i lost it?." so we can ignore that comment. 2 paths there works just as well

Yes! And also the httpClient engine (or the client itself). For a long lived service which is doing resolution, there is a benefit of not creating a ktor HttpClient per request (see here). I've seen this pattern in many languages, so I assume the same is true if we were to use OkHttp.

yeah totally. i'd hope we don't instantiate an http client per request in any scenario. small nuance here is: creating one once and using it for every request doesn't necessitate that we allow devs to provide their own http clients. if we see a good reason for allowing people to provide their own client then happy to support it.

Does updateKeyAlias live inside the opts object? I view this as an important detail because I don't know of a way of determining what the update key (even the public key) is by only looking at the DID Document (do you know if it's possible?). I have similar concerns with other operations below.

yeah ideally as optional. just spitballing here: it seems like our ion implementation can just default to a predetermined alias if one isn't provided e.g. ${shortFormDid}:update given that there's only 1 update key at any given time. same pattern for recovery.
sounds like a detail people shouldn't have to worry about unless they explicitly want to. do you have any reservations or hesitations with that approach?

I agree that they shouldn't worry about that. I tried to design it in that way, but ended up having really strong coupling.
I think that can work as long as we're ok with overwriting a key in the key manager, and as long as we have the ability to provide a custom key id when generating a new key within the manager. Do you think that the AwsKeyManager would be able to implement those two features?

I think that can work as long as we're ok with overwriting a key in the key manager, and as long as we have the ability to provide a custom key id when generating a new key within the manager. Do you think that the AwsKeyManager would be able to implement those two features?

yep for sure. something that Tom and I had previously discussed and punted until there was a need which it seems like we have now!

Finally, a totally separate concern is that I think we should be fostering the explicit practice of using separate KeyManager instances to store the recovery vs other keys. I don't think that should be optional. I didn't actually address that in this PR.

yep totally on board with encouraging this. only question for you here is: is providing a separate key manager a MUST for you or a CAN/SHOULD? where CAN/SHOULD implies that the key manager used for all other keys will also be used for recovery unless a recovery specific key manager is supplied

Finally, a totally separate concern is that I think we should be fostering the explicit practice of using separate KeyManager instances to store the recovery vs other keys. I don't think that should be optional. I didn't actually address that in this PR.

yep totally on board with encouraging this. only question for you here is: is providing a separate key manager a MUST for you or a CAN/SHOULD? where CAN/SHOULD implies that the key manager used for all other keys will also be used for recovery unless a recovery specific key manager is supplied

For me it's a MUST. And if people know what they're doing , they can override to use the same :)

I think that can work as long as we're ok with overwriting a key in the key manager, and as long as we have the ability to provide a custom key id when generating a new key within the manager. Do you think that the AwsKeyManager would be able to implement those two features?

yep for sure. something that Tom and I had previously discussed and punted until there was a need which it seems like we have now!

Is there a way we could support a latest tag for a versioned key? That would be my preference. That way there is a nice trail.

Yes! And also the httpClient engine (or the client itself). For a long lived service which is doing resolution, there is a benefit of not creating a ktor HttpClient per request (see here). I've seen this pattern in many languages, so I assume the same is true if we were to use OkHttp.

yeah totally. i'd hope we don't instantiate an http client per request in any scenario. small nuance here is: creating one once and using it for every request doesn't necessitate that we allow devs to provide their own http clients. if we see a good reason for allowing people to provide their own client then happy to support it.

 They're separate concerns. I would like to allow for that flexibility though, because I can see users of the SDK wanting to override that. An example would be if they want to do custom logging and/or tracing.

This is great @andresuribe87 . let's merge this right after hack week


Drugs with a blackbox warning flag and no additional information about the warning

The profile page for ABARELIX - CHEMBL1252 displays the black box warning flag for the drug, but there is no corresponding information in the data table.

The same view is also seen on the CHEMBL profile page for ABARAELIX.
The original package insert for the drug notes "immediate-onset systemic allergic reactions, some resulting in hypotension and syncope".
Should this drug have a black box warning for vascular toxicity (MedDRA 10027433), similar to METFORMIN - CHEMBL1431?
And as a related question, are there other examples of drugs found in the Platform drug index where there is no information about the warning, other than the flag?

@ireneisdoomed has some background on this I believe

Looking at CHEMBL1252 what is displayed on the Platform is all that we're getting from ChEMBL:
+--------------+------------------+-------------------------+-------------+---------------+-------------------+----------+------------+-----------------+------------+
|_metadata     |molecule_chembl_id|parent_molecule_chembl_id|warning_class|warning_country|warning_description|warning_id|warning_refs|warning_type     |warning_year|
+--------------+------------------+-------------------------+-------------+---------------+-------------------+----------+------------+-----------------+------------+
|[[CHEMBL1252]]|CHEMBL1252        |CHEMBL1252               |null         |null           |null               |5         |[]          |Black Box Warning|null        |
+--------------+------------------+-------------------------+-------------+---------------+-------------------+----------+------------+-----------------+------------+

Ticket closed as ChEMBL have been notified of the issue and we expect it to be resolved with the release of CHEMBL29. For more information, see https://github.com/opentargets/platform/issues/1616#issuecomment-869625869


throw new TypeError('barcodeValueTest must be a regular expression');

version 1.5.2 throws new TypeError('barcodeValueTest must be a regular expression') on load. This does not happen in version 1.5.1.

Hi @jnssnmrcs! Thanks for reporting.
Oops, it looks like 1.5.2 should never have been published, as it is incomplete. We bumped the underlying barcode-scan-listener module which this Angular module wraps, but the interface to the underlying module now requires a barcodeValueTest param.
Feel free to just ignore 1.5.2 and we'll try to fix this soon.


MudTable Search on text input

Feature request type
Enhance component
Component name
MudTable
Is your feature request related to a problem?
No response
Describe the solution you'd like
Hi,
i use the Mud Table with search and filter  for showing users to select them.
How can i search the users by typing the searchstring? Pressing the enter button to search is not handy and quick.
If more than 3 characters were typed, the filter function could be triggered. Is this possible?
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
I am actually looking for a peaople picker. Maybe an alternative?
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

https://mudblazor.com/components/textfield#immediate-vs-debounced

@Garderoben how is this marked as completed? The link you provided was to a textfield component but @cuban8IO was asking about the table component. Has the textfield's functionality been implemented on the table component?

@mwasson74 , because when filtering in a table, the search string is bound to MudTextField.
<ToolBarContent>         <MudText Typo="Typo.h6">Periodic Elements</MudText>         <MudSpacer />         <MudTextField @bind-Value="searchString" Placeholder="Search" Adornment="Adornment.Start"<EMAIL_ADDRESS>IconSize="Size.Medium" Class="mt-0"></MudTextField>     </ToolBarContent>
Snippet from example


Cannot clear storage account on LCS-hosted devbox

I'm trying to re-enabling the scale unit, when I try to clear the azure storage, I get an error
Scale unit storage to clean up: 2
Unhandled exception. System.ArgumentNullException: Value cannot be null. (Parameter 'connectionString')
at Azure.Storage.StorageConnectionString.Parse(String connectionString)
at Azure.Storage.Blobs.BlobServiceClient..ctor(String connectionString, BlobClientOptions options)
at ScaleUnitManagement.ScaleUnitFeatureManager.Common.StorageAccountCleaner.DeleteBlobContainers() in /home/runner/work/SCMScaleUnitDevTools/SCMScaleUnitDevTools/src/ScaleUnitManagement/ScaleUnitFeatureManager/Common/StorageAccountCleaner.cs:line 29

Hi @ABOZH, are you setting up two LCS-hosted dev boxes? Or using the UseSingleOneBox parameter to configure a single machine as Hub and Spoke both?
If the former; @Omaaarz / @Antonstockmarr: on LCS-hosted dev boxes a storage account is created per environment, so users need not add it as a parameter to UserConfig.xml - we should grab the value from web.config in that case.

Hi @AndreasHassing, i am using 2 separate LCS hosted devbox.

I can clean storage when decrypt web.config copy connection string value in userconfig.xml
Thank you for support
also (maybe this is important) in the step of removing the wordload from the hub, I was able to do this after changing the user from the default

to this

@Omaaarz do we have a tracking bug for user rights for workload deletion?


Setup coveralls

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@5fff647). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #7   +/-   ##
=========================================
  Coverage          ?   96.57%
=========================================
  Files             ?        7
  Lines             ?      380
  Branches          ?        0
=========================================
  Hits              ?      367
  Misses            ?       13
  Partials          ?        0

Continue to review full report at Codecov.

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


Add crane to determine digest of the container image

Signed-off-by: Kemal Akkoyun<EMAIL_ADDRESS>xref: https://github.com/parca-dev/parca/pull/1041

@cpanato quay seems failing I'm trying to debug it https://github.com/parca-dev/parca-agent/runs/6440132726?check_suite_focus=true
It's an authorization error but it has the write access. Have you seen any similar errors before?
[39](https://github.com/parca-dev/parca-agent/runs/6440132726?check_suite_focus=true#step:10:39)
main.go:52: error during command execution: POST https://quay.io/v2/parca/parca/blobs/uploads/: UNAUTHORIZED: access to the requested resource is not authorized; map[]
[40](https://github.com/parca-dev/parca-agent/runs/6440132726?check_suite_focus=true#step:10:40)
make: *** [Makefile:237: push-signed-quay-container] Error 1```

@cpanato quay seems failing I'm trying to debug it https://github.com/parca-dev/parca-agent/runs/6440132726?check_suite_focus=true
It's an authorization error but it has the write access. Have you seen any similar errors before?
[39](https://github.com/parca-dev/parca-agent/runs/6440132726?check_suite_focus=true#step:10:39)
main.go:52: error during command execution: POST https://quay.io/v2/parca/parca/blobs/uploads/: UNAUTHORIZED: access to the requested resource is not authorized; map[]
[40](https://github.com/parca-dev/parca-agent/runs/6440132726?check_suite_focus=true#step:10:40)
make: *** [Makefile:237: push-signed-quay-container] Error 1```

humm, looks like it is not getting the login we did in a step before.

@kakkoyun i will try to reproduce that in a test project and check it, 🐻 with me

able to reproduce https://github.com/cpanato/testing-ci-providers/runs/6442136152?check_suite_focus=true, checking a fix

the issue looks like is related to the podman login if i use docker login instead that works https://github.com/cpanato/testing-ci-providers/runs/6442149954?check_suite_focus=true
for the ci job, i think that is okay to replace to docker login because we don't use podman for anything related with quay
cc @kakkoyun


DISCUSS: Remove the Lightbend telemetry docs?

When converting docs from akka -> pekko referencees, the telemetry docs discuss integrating Lightbend's Telemetry product into a system that is using Lightbend Akka.
These docs are unique because they refer to the integration of another Lightbend tool, Lightbend Telemetry to gather metrics about Akka usage.
https://developer.lightbend.com/docs/akka-guide/telemetry/index.html
Question: Do we keep this documentation in our guide?
Users might want to use this tool to connect with pekko. However I have the following doubts:

The tool may not work with Pekko APIs.
In platform-guide our docs on telemetry may become out of date and therefore continuously require copy from the akka-guide docs.
Telemetry is a paid service for Lightbend subscription. It is really up to customers whether they choose to take telemetry or another metrics service. I don't really think Pekko should be pushing that?
Currently there is a Gitlab workflow to test this validate-telemetry-sample.yml . This requires a lightbend subscription and is failing. I've made it manual-trigger only in PR #21 while we figure this out

My vote: Remove telemetry stuff, take care not to delete anything that is useful. There's no point having it if it won't even work with Pekko.

Nothing that makes no sense should be kept. No Lightbend tools support Pekko

I'll remove the telemetry docs

We should completely remove all refs to the akka-cloud-platform guide too.

We are not offering a Pekko-cloud-platform
This has already been removed from the akka-guide ( https://github.com/akka/akka-guide/commit/b38970a3d6f5bcc9cfefe7181a454e0a7ae1b2fe).

Please don't link to latest Akka code or PRs that were started after the BSL license change. It's ok to point to the public docs.

I'm proceeding by creating a PR for removing Akka Cloud Platform docs from this guide as it is not needed.

I think this is done - reopen if necessary


9766 - Only email active users on SMS token change.

Only send emails to active users or admins when an SMS token is changed for a mission.

Terrific!


Make installation easier with optional developer config file

Would it be an idea to have an optional developer config file that can override "Team" and "Bundle identifier" settings, much like NetNewsWire does it.
This way, you only need to configure the app once, and not each time you want to install it.
I tried this myself, but I'm not an XCode warrior (more python guy), but I think it's still a good idea to implement.

Great idea! I will look into this. The dev team changes are a pain, I agree.


Any Plans To Add text/xml Support?

Deno DOM is primarily focused on HTML parsing, there are no definite plans for XML parsing. This is partially due to the instability of the xml5ever crate, and uncertainty regarding whether XML parsing should be strict or lenient (which is prevailing design of the xml5ever crate, but maybe undesired by XML consumers). If there's any precedence for strict parsing, then I would need to find a separate compliant parser for XML which also adds some complexity to the codebase.

Ok thank you for the fast response :)


QCEngine/NWChem interface minor update

Description
A minor update to make it so that the keywords "noautosym" and "nocenter" can be passed to NWChem
Changelog description
Added a few lines in "build_input" in "runner.py" which can take some input keywords and change the NWChem input file to add them.
Status

[x] Code base linted
[x] Ready to go

I've tested these changes on a personal installation of QCEngine/NWChem (for preparation for NWChemEx...)

Have you tried nocom noreorient symmetry c1 if you're building the molecule from string https://github.com/MolSSI/QCEngine/blob/master/qcengine/programs/tests/test_ghost.py#L20-L21 . Or fix_com=True, fix_orientation=True, symmetry=c1 https://github.com/MolSSI/QCElemental/blob/master/qcelemental/models/molecule.py#L263-L286 if building molecule from schema?
I think that should do what you intend. qcng is supposed to return program native orientation if fix_com/_orient=F or return in AtomicInput input Cartesian orientation if fix_com/_orient=T.

Did the alternate way of getting gradients back in input orientation end up working for you?


Unexpected behavior with a user-defined list generator

I was writing a procedure that would take a string and output a list where all instances of param rep was replaced by param sub
Full code here
testing the code using the coimmand using the [repl](http://www.biwascheme.org./demo/repl.html] from biwascheme.org)
(replace-split "aeaeaeur" #\e #\a)
returned
(#\r #\a #\r #\a #\r #\a #\r #\r)
I tested this using other lists, and the result was the same, all of the characters not replaced with param sub got replaced with the last item on the string. I got the same results with numbers, letters and spaces.
I checked by manually going through the instructions in the code and applying them to the string example above, and got the expected result. I also verified this by running the same code using the repl at JsScheme, and at Gambit-js and all returned the expected results as described in the procedure.

Thanks for reporting. I've confirmed the bug happens (and does not happen with Gauche). I'll investigate.

BiwaScheme seems to have a bug related to internal define. The following behaves the same with BiwaScheme and Gauche.
(define (replace-split str rep sub)
  (let ((ls (string->list str)))   ; Changed from internal define
    ; check for EOL
    (if (null? (cdr ls))
      ls
      (cond
        ((eqv? (car ls) #\space)
         (replace-split (list->string (cdr ls)) rep sub))
        ((eqv? (car ls) rep)
         (cons
           sub
           (replace-split (list->string (cdr ls)) rep sub)))
        (else
          (cons
            (car ls)
            (replace-split (list->string (cdr ls)) rep sub)))))))

(print (replace-split "aeae" #\e #\a))


Custom console

It might just be me, but switching to Pry is the first thing I do when I make a new gem so I thought this would be helpful to others as well.
It's done in a little strange of a manner to allow any console library to work—a new console shouldn't have to come to us for their users to be able to use it, that would be unfair.

Closing as a duplicate of https://github.com/bundler/bundler/pull/3334.


Allow Layer ComboBox to take a new field (new layer)

There is no way to create a new Layer on the fly. The layer has to exist in the current drawing for it to work.

No longer needed


include instance profile module with zts server binary image

Description
addresses #2818
Contribution Checklist:

[x] The pull request does not introduce any breaking changes
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Attach Screenshots (Optional)

@havetisyan This is just out of curiosity, I think the issue is with the configuration when building the docker image. If we fix this, will it also resolve the problem with the docker image?

@ysknkd Yes, there is nothing wrong with the docker image configuration. If you look at how the docker image is built, it just takes the contents of the zts server tar file that is built as part of the assembly job.
https://github.com/AthenZ/athenz/blob/master/docker/util/athenz-builder/Dockerfile#L35

@havetisyan I understand now. Thank you for your explanation.


Define the convention of the transformation matrix properly

The definition of the transformation matrix should be stated somewhere. It is different for row and column lattice matrices, here we use the column lattice matrix convention so
M c = C

Where c and C are matrices made of column lattice vectors.
This means for row vector matrices we have:
c^TM^T = C^T

Note： both ase and pymatgen stores the lattice as row vectors.

Notes added in #44 to highlight the difference caused by row and column vectors.


Empty name not allowed and the catalog view is not working right

Empty names
Leaving the name all empty and just using the subject causes the server to not accept the post with the error "invalid name".
Instead of checking for is_null($name), checking for is_null($name)&&is_null($email)&&is_null($subject) in sp.php line 26 would fix this.
Catalog sync
And for the catalog sync to work, the thread parameter in qp.php has multible threads comma separated like https://nsredux.com/namesync/qp.php?b=b&t=878719219,878729635,878731553,878731419,878731460
which causes an Error 406.
You could allow a , additionally to any number and check when its present. Then have a different query that is ORed together.
But thats a bit more work.
Minor things
One could flag an anon post later on with some name and troll by giving every anon a name.
The server side could deliver the timestamp of the post so the client could verify this with the thread data. If its within a few seconds, all is fine. With a timestamp this could be done on the client side but this inflates the data a bit.
Another thing would be to check the Origin header from the browser to be https://boards.4chan.org/ in sp.php
And the X-Requested-With header to have NameSync at position 0 present in sp.php to make sure its from the extension.

Fixed empty names, added some checks for Origin and X-Requested-With headers. I already have checks in place from before to combat name trolling. I parse the 4chan board API and keep track of previous posts, if your post # is behind the one I already have logged, it will fail. If you mean for posts that have yet to be made, they would have to guess the post number (and hope its in the same thread), and they only get 4 posts a minute to try before getting limited.
Catalog sync should really be its own file, it shouldn't be mashed into qp.php (unless stock namesync does this, but I think it's new to frensync).

Perfect. Thats done then.
Having a post behind another is still possible if one has a slow network and theres a high activity. There have been atleast two posts in the same milisecond as well and some clients have a clock offset of some seconds. Thats why i'm a bit careful locking that down without throughly testing it.
I ment that a troll could have a script running that checks every few seconds for a new post and instantly flags this with a name.
But thats details for when it gets larger.
Catalog sync is still stock NS code and done in the same file because the query doesn't differentiate on which page it syncs.

For example in a paged view there could be 10 different threads with 3 posts each being shown - each could have a name.
The catalog query just has all threads from all pages with just the first post being interesting.
Or if Appchan is fancy, you could have weird things like two open threads at the same time on one page.
For compatibility having it in qp.php would be great but i agree. It depends how you set it up - like RISC or CISC have one query functon per file or one file for all queries.
Maybe an cq.php with the board, thread numbers and the limit of 1 or 3 posts per thread would be more elegant but that breaks compatibility for now. Let me test first whether that can be split up easily in the client or whether that breaks things.
But also having a single post in a thread on a board as a query would be nice. There is that ancient bug where 4chanX loads in a quoted post from another thread and the name for that one is not synced. And thats also a bit difficult on the client side if theres no distinct event for loading that external post.

I'm getting HTTP 405 again on the options request. The data itself is correct if you ignore CORS.
Seeing that you added a $ recently, there might be some bug present.

Yep, that would be because I didn't test before pushing (bad me, bad). Ill work on catalog syncing when i'm not to busy.
For caching, honestly all of this could just be converted to a flat-file DB, it would be extremely cache friendly and fast.

I also did that for simple things. A bit down time is not too bad since its all redundant.
Thanks for working on it in the first place.
If you want to test this, change line 137 in frensync to var onload_proxy =function(t){console.log(this,t);return this.status&&(t=this),t.target?onload(t.target):((null!=onload)?onload(t):false)}; and you get the result dumped in the catalog for all three servers. And have the GM_API XHR unchecked in the options if you want to use the networking tab.
I als need to work on it a bit but my approach was to let the database deal with it and have a large query with all ids:
SELECT distinct(thread) FROM sync where FIND_IN_SET(thread,{$t}) and board={$b}; where $t = "1,2,3" and $b ="b";

Had an hour, catalog query is in and tested.


The default source build command script should be eng/common/build.*

Right now the default is to call build.cmd/sh at the root of the repo. Most of these scripts add a couple standard params (e.g. restore) and call the eng/common build script. Source build generally wants to invoke the root script with standard args that include restore and pack and such. This is fine on Linux because switches passed twice do not cause issues. In powershell this causes binding issues.
Instead, we should have the default be eng/common, and set the build command explicitly to the root cmd in cases where required.

This already got implemented in the Windows PoC. Keeping the issue open until we merge it back into main.

Fixed by https://github.com/dotnet/installer/pull/18214


Path Traversal Bug in HTTP::StaticFileHandler

In Windows both \ and / are accepted as path delimiters, but HTTP::StaticFileHandler isn't considering \ as such.

I checked what some other static web servers do on windows. I used the default server implementations in Ruby (httpd) and Python (http.server) and apache with default configuration.
Ruby seems to be entirely disallow backslashes in the request path. It results in a Bad Request  error.
Python seems to have some buggy support for file names including backslashes on Windows. If you request back\slash.txt, you get a redirect to back\slash.txt/ and that shows the directory.
Apache seems to treat backslashes as directory separators.
Using the replacement character directly (or rather URI-escaped) seems to always work: /back%EF%81%9Cslash.txt.
Also worth noting: Browsers typically convert backslashes into forward slashes in the URL.

https://github.com/crystal-lang/crystal/pull/11847 should fix this issue by mapping backslashes in a request path to a replacement character, that Windows also recognizes. Thus requesting a path such as /foo\bar will have the same semantics on Unix-like systems as well as Windows.
After that we're just missing specs for this.


pkg/chann(ticdc): remove close method of DrainableChann

What problem does this PR solve?

Issue Number: ref #8255
What is changed and how it works?
remove close method of DrainableChann
Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`.

/merge

/merge
Label it only requires one LGTM. Because it is a quite simple change.

/run-engine-integration-test

/run-integration-test

/run-engine-integration-test


martin ui POC - WIP

This is a work in progress for a possible ui for martin. You currently can run it by running the martin server on port 3000 and then running the ui by going to martin-ui folder, and running
npm ci
npm run dev

@tomeronen, thanks so  much for working on this! I tried it, but I couldn't actually get it to show the data from the selected layers - i used just start and just run in addition to the above commands (to run local test infrastructure).  Note that this requires docker, rust compiler, and the just utility installed locally. I do everything on Linux, so other OS were not tested as thoroughly.
I started #1120 to capture the "grand vision" for the UI.
P.S.  Just a suggestion - whenever creating a PR for any project, create a new branch on your fork, don't use main branch for that.

going to open a series of small PR's to be more organised and so they will be reviewable


Add low precision optimization tool

Add low precision optimization tool.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.wanchen.swc seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.