Support running behind a proxy

Take the proxy settings from the repository configuration

Could be nice for corporate environments. Did you mean systemProp.https.* properties or something specific in the repository configuration?

Corporate environment, exactly our problem.
I meant mavenDeployer { repository() { proxy }} as this does not respect systemProp.https.*
But this is just our use case and I don't know gradle good enough to tell if this makes sense for everybody.
In the end, there should be a possibility to configure a proxy, either with or without a sensible default.

The plugin currently uses HTTPBuilder which seems to require defining proxy settings in a specific way. However, I'm in the middle :) of migration to OkHttp, so there is a change it would be easier to implement.

I just ran into this as well. I'm in a corporate environment and outgoing traffic needs to go through an HTTP proxy. Most tools pick up HTTP_PROXY/HTTPS_PROXY, Java has its own -DsystemProp.https.*/-DsystemProp.http.*, but none of these work for this library.
Any way I can help with this? Would it be possible to have the HTTP client (RESTClient) pick up Java's systemProp.http(s) or the system environment variables?


Unboxing is failing for int to Integer conversion

For example:
Integer value = isSomething() ? 7 : getNullableValue();
will throw an exception: $this.nullableValue0 is null
But at the same time it works fine if 7 will be wrapped like new Integer(7) or if type was casted to Integer: (Integer)7

Can you post self-contained example? Which version of TeaVM are you using?


Runtime error with isolated DOM sink emitting undefined

When using an isolation, the code will crash when the dom sink emits undefined. Emitting undefined in a non-isolated sink will just lead to nothing being rendered. So either it's a bug with dom isolation, either we maybe should make it so non-isolated dom also blarf on undefined emit so we would have a consistent behavior.
Code to reproduce the issue:

https://www.webpackbin.com/bins/-Kg0-G3bhJREVL4uMkYj
open dev tools
click on the button
watch the world burn

Expected behavior:
Depending on your opinion, either it should not crash on undefined being emited by isolated dom sink, either non isolated dom sink should throw on undefined emit as well
Actual behavior:
Runtime crash (happens in totalIsolateSink, Cannot read property 'data' of undefined)
Versions of packages used:

@cycle/dom: 16.0.0
@cycle/isolate: 2.1.0
@cycle/run: 3.1.0
xstream: 10.3.0

This is quite a corner case, but given that snabbdom accepts null and undefined as a virtual node to be diffed, I think we should handle null and undefined, specially since non-isolated sink supports it.
Thanks.

Fixed and will be released in dom v17.1

Sorry for the eventual useless and annoying github notification, but it's probably more important to say: thanks! :tada: :heart:


Request

Dear Uzair, Thank you for sharing your work on GitHub. Is there any Publication associated with this so as to enable us to understand the work in more depth (needless to say, integrity of the documentation would be highly respected and it would be properly cited / referred)

@aqureshimy
Yes, we have submitted our initial results to IWCMC 2017 (Spain), which have been accepted for publication. You can view the archived version at https://arxiv.org/abs/1705.04390 .
We are also working on some new results regarding 3D channel modeling, a multi-slope path-loss model and resource optimization in HetNets. I will share the latest results here once the manuscript is ready and has been accepted for publication.


[MINOR] Update parameter description

What is the purpose of the pull request
This PR update parameter description to make parameter description more clear
Brief change log
update parameter description in  HoodieDeltaStreamer.java and HoodieMultiTableDeltaStreamer.java
Verify this pull request
This pull request is a trivial rework / code cleanup without any test coverage.

Not sure if a minor improvement like this need fire a JIRA. will add it if needed.
cc @leesf

@GuoPhilipse Thanks for your contribution, LGTM, merging. PS: you would use the [MINOR] prefix in the title.

Thanks @leesf , have updated the title.


sak-argocd: interface is not accessible immediately after deployment

Describe the bug
I wasn't able to access Argo interface after the deployment on sak-argocd using the correct login / password. The problem is related to this issue and removing the time parameter from Argo secret helped.
A possible fix is to set a correct time for this secret during ArgoCD deployment. (It uses local time and converts it to GMT+0 timezone by default)
Use version
Latest.
To Reproduce
Steps to reproduce the behavior:

Add ArgoCD module
Port-forward to ArgoCD
Open interface and enter login / password

Trick with passwordMtime doesn't  work now.
After some tests I noticed that newly generated password is not the same.
I'm going to remove setting of password from argocd.yaml and set it during creation of helm chart.

fixed with https://github.com/provectus/sak-argocd/pull/14


EIP712 Support

Is your feature request related to a problem? Please describe.
EIP712 support is missing.
Describe the solution you'd like
We should be able to sign messages using EIP712
Additional context
There's an open PR for it in Web3js https://github.com/ethereum/web3.js/pull/3449
https://github.com/0xProject/0x-protocol-specification/issues/11

Relevant:
https://github.com/tomusdrw/rust-web3/issues/169
https://crates.io/crates/eip-712

Hello @gakonst if you have not worked on this issue, I would like to take it up. The plan is to use the above mentioned crate.
One more point to note is that, the Signer trait currently supports a sign_message which eventually hashes the message. I would need to add:
/// Signs the provided message
fn sign_hashed_message<S: AsRef<[u8]>>(&self, message: S) -> Signature;

where the message argument is already hashed, and hence won't be hashed before signing. This requirement is because the hash_structured_data function already hashes the encoded data.

I believe that this wouldn't be compatible with the MIT / Apache-2 license. That said, I think we could re-implement that functionality using a custom Serde visitor instead of the parser they use?

Okay, I will go through Serde's docs and try to implement it :)

Any progress on this?
There is now graphprotocol/eip-712-derive.
The repo contains no license yet, however author hinted that is MIT

Nobody that I know of is working on it, so if you'd like to integrate it, along with an eth sign typed data call I'd be more than happy to merge.

Here's a potentially simpler approach than Graph Protocol's:

Define a trait EIP712
Define a struct e.g. struct Name { field1: U256; field2: u65 }
Have a derive macro which implements the EIP712 trait for that struct

The trait could have a fn typehash() which auto generates the typehash from the datatype, similar to this and a fn input(&self) which would abi encode each field of the struct, same as this
And then you’d have a StructName.eip712_encode() which calculates the full hash by calling:

self.domain_separator()
self.typehash
calculate input
hash them together

We could add an extra param on the derive macro which would set name and version for the domain separator, maybe let verifying_contract and chainid be defined at runtime, since you may want to use the same e.g permit struct for many contracts or many chains (it’d be nice if both could work)
Strawman syntax:
#[derive(Debug, EIP712)]
#[eip712(name = “Uniswap V2”, version = “1”, chainId = 1, verifyingContract = “0xB4e16d0168e52d35CaCD2c6185b44281Ec28C9Dc”)]
struct Permit {
    owner: Address,
    spender: Address,
    value: U256,
    nonce: U256,
    deadline: U256,

let permit = Permit { … };
dbg!(permit.typed_hash())

It’d basically be very abigen-like, and I think I prefer it to the approach that the Graph took.
The trait may not be necessary by default, by I suspect it’d be useful for 3rd party consumers, to be abstract over the data that you want to typehash for (e.g. wallets)

Opened #481 as a draft PR. It's a first iteration and is lacking test coverage against solidity and other libs, such as https://crates.io/crates/eip-712.
Additionally, the macro currently does not recurse all nested Eip712 structs.
I welcome early feedback on the code and any thoughts on improvements. Once test coverage and nested structs are completed, I'll move the PR from draft status to open.

#481 closes this issue. A new issue should be opened to track nested Eip712 structs and another issue for crate restructuring to re-export the macro.

Issues opened, thank you for the hard work on this.

Issues opened, thank you for the hard work on this.

No problem! Will keep an eye on those issues and lend a hand where needed! :)


Typing sometimes continues in the result list

Environment (please complete the following information):

OS: Archlinux
(Neo)Vim version: v0.4.4
vim-clap version: 83cac4972caef144d0e947025ff43af6fd2becac
Have you reproduced with a minimal vimrc: no
Have you updated to the latest plugin version: yes
Have you upgraded to/compiled the latest Rust binary: yes

Describe the bug
Since a few days, when launching clap an immediately starting to type a filter expression (you really have to be fast here), text intended for the top filter windows suddenly goes into the result window with further control characters and the cursor being stuck there instead of the input prompt.
This is how this looks like

You can actually go back to the input prompt, but once you start typing again, the windows will look like:

:messages afterwards contains:
Error detected while processing function <lambda>3[1]..clap#highlight#add_fuzzy_async[1]..<SNR>166_apply_add_highlight[10]..48:
line    2:
E803: ID not found: 4

Clap debug
            has cargo: 1
            has maple: /home/languitar/.local/share/nvim/bundle/vim-clap/target/release/maple
           maple info: version 0.1.22 (git v0.22), built for x86_64-unknown-linux-gnu by rustc 1.49.0.
         has +python3: 1
has py dynamic module: 1
            has ctags: ctags with JSON output support
    Current sync impl: Python
     Current FileType:
Third Party Providers: []
       Global Options:
    let g:clap#autoload_dir = '/home/languitar/.local/share/nvim/bundle/vim-clap/autoload'
    let g:clap#provider_alias = {'gfiles': 'git_files', 'hist:': 'command_history', 'hist/': 'search_history'}
    let g:clap_background_shadow_blend = 50
    let g:clap_disable_bottom_top = 0
    let g:clap_disable_matches_indicator = v:false
    let g:clap_disable_optional_async = v:false
    let g:clap_disable_run_rooter = v:false
    let g:clap_enable_background_shadow = v:true
    let g:clap_enable_debug = v:false
    let g:clap_enable_icon = 1
    let g:clap_forerunner_status_sign = {'done': '•', 'running': '!', 'using_cache': '*'}
    let g:clap_insert_mode_only = v:false
    let g:clap_layout = {'relative': 'editor'}
    let g:clap_multi_selection_warning_silent = 0
    let g:clap_no_matches_msg = 'NO MATCHES FOUND'
    let g:clap_open_action = {'ctrl-v': 'vsplit', 'ctrl-x': 'split', 'ctrl-t': 'tab split'}
    let g:clap_popup_border = 'rounded'
    let g:clap_preview_size = 5
    let g:clap_providers_relaunch_code = '@@'
    let g:clap_search_box_border_style = 'curve'
    let g:clap_search_box_border_symbols = {'nil': ['', ''], 'curve': ['', ''], 'arrow': ['', '']}
    let g:clap_theme = 'material_design_dark'
  Provider Variables:
                     []

To Reproduce
This doesn't seem to happen if I remove ntpeters/vim-better-whitespace from my vimrc. However, this also doesn't seem to trigger for a minimal vimrc with only clap and better whitespace. So I was unable to come up with minimal vimrc.
Expected behavior
no corruptions

Can't reproduce locally.

Can't reproduce locally.

You might want to reduce your current vimrc slowly to provide a very short vimrc for the reproduction.

You might want to reduce your current vimrc slowly to provide a very short vimrc for the reproduction.

I did a binary search through my init.vim but couldn't come up with anything consistent that seems to trigger this reliably without the full setup.

I did a binary search through my init.vim but couldn't come up with anything consistent that seems to trigger this reliably without the full setup.

Just in case you want to have a look at the full setup: https://github.com/languitar/config-vim

Just in case you want to have a look at the full setup: https://github.com/languitar/config-vim

Alright, I did some further digging. The problem disappears in my setup once I remove ntpeters/vim-better-whitespace.

Alright, I did some further digging. The problem disappears in my setup once I remove ntpeters/vim-better-whitespace.

If so, you can try to disable this plugin in clap input buffer:
autocmd BufEnter clap_input let b:better_whitespace_enabled = 0

If so, you can try to disable this plugin in clap input buffer:
autocmd BufEnter clap_input let b:better_whitespace_enabled = 0

That doesn't help. After one of these errors I just checked the value for the input windows, which is 0, yet the problem appears.

That doesn't help. After one of these errors I just checked the value for the input windows, which is 0, yet the problem appears.

I personally don't use vim-better-whitespace, I recommend you to file an issue in their repo how to disable it in clap_input buffer.

I personally don't use vim-better-whitespace, I recommend you to file an issue in their repo how to disable it in clap_input buffer.

Or you can see whether this patch https://github.com/liuchengxu/vim-clap/issues/580#issuecomment-761798978 helps. That is intended to make other plugins unable to be aware of the creation of the input buffer. I don't use these plugins and haven't spent time confirming it, not sure if the patch really works.

Or you can see whether this patch https://github.com/liuchengxu/vim-clap/issues/580#issuecomment-761798978 helps. That is intended to make other plugins unable to be aware of the creation of the input buffer. I don't use these plugins and haven't spent time confirming it, not sure if the patch really works.

I could get around this issue by extending g:better_whitespace_filetypes_blacklist with clap_input.

I could get around this issue by extending g:better_whitespace_filetypes_blacklist with clap_input.

Good to see that, closing then.

Good to see that, closing then.

Is there anything special about the implementation in clap that could make it susceptible to such issues?

Is there anything special about the implementation in clap that could make it susceptible to such issues?

Have no idea, these plugins all have the CursorMoved hooks and there are possibly some negative interaction effects.

Have no idea, these plugins all have the CursorMoved hooks and there are possibly some negative interaction effects.

Hm, seems the variable also wasn't the solution. This issues is "back".

Then you should ask why the blacklist option of vim-better-whitespace does not take effect since you have no issue without it.

Or there is still some other plugin causing the same issues.

Yeah, possibly, you'll have to find the culprit using the binary search again (:


Changing step list and header font

Hi,
I can not change the default font for steplist and header ..Is it possible to add ability to change fonts in next version?
In some languages like Arabic and Persian the Tahoma font is more userfriendly than other MS fonts..
Thanks
Originally posted: 2015-03-10T00:58:32

I have remedied this problem in version 2.0.7. StepWizardControl now has a StepListFont property that can be set a design-time. WizardControl has an OverrideThemeFonts methods that can be called after control creation that will allow the fonts to be overridden.
Originally posted: 2015-03-18T16:17:38


Why is ldap not more popular? Advantages of node-ldapjs over auth0? auth.js and ldapjs? passport-ldpauth?

Hello fellow travelers on the wayside
I am curious as to why ldap as a standard is not more commonly used for authentication for webapps. People typically roll their authentication using a database like sql or mongo and hashing using something like passport or auth.js. From my perspective, it's a good idea to stick with the standards and the basics, especially one as old as ldap.
I recently learned about ldap as an authentication method because I need a way to maintain user namespaces (for email) in a way agnostic from the containers that refer to them.
What is the advantage of using node-ldapjs over say auth0? https://auth0.com/docs/authenticate/identity-providers/enterprise-identity-providers/active-directory-ldap
Also I'm slightly confused, there's ldapjs referenced by auth.js https://authjs.dev/guides/other/ldap-auth - this is the same project, right?
There's also passport-lpdauth https://www.passportjs.org/packages/passport-ldapauth/

What is the advantage of using node-ldapjs over say auth0?

They are not mutually exclusive.

Also I'm slightly confused, there's ldapjs referenced by auth.js https://authjs.dev/guides/other/ldap-auth - this is the same project, right?

We are not affiliated with "auth.js". Yes, the ldapjs they suggest using is this module.

To address the overall question: people make their own determinations as to how they will manage their account databases. Some will use an LDAP server, others a RDBMS. We have no opinion on what someone should use. We are not designing their systems and don't know their requirements.


⚠ Telly has degraded performance

In ad67d2a, Telly ($TLY) experienced degraded performance:

HTTP code: 200
Response time: 2144 ms

Resolved: Telly performance has improved in 051174c.


Uploading to specific environment though API

Looking at the https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/administration/dynamics-microsoft-automation-extensionupload-patch it doesn't look it's possible to upload extension to a specific environment (sandbox instead of production) - is this correct? If that's the case - would that be possible in the foreseeable future?

Hi. It is possible.
https://api.businesscentral.dynamics.com/v1.0/sandbox/api/microsoft/automation/v1.0/companies({companyId})/extensionUpload(0)/content
But, I recement using v2.0 of the gateway to use multiple environments. Docs will be updated shortly to reflect newest changes. Heres how you access a named environment with v2.0 of the gateway:
https://api.businesscentral.dynamics.com/v2.0//api/v1.0/customers
With custom namespace, in this case microsoft/automation:
https://api.businesscentral.dynamics.com/v2.0//api/microsoft/automation/v1.0/companies({companyId})/extensionUpload(0)/content
With tenant identifier (need for basic auth or if you're using delegated admin)
https://api.businesscentral.dynamics.com/v2.0///api/microsoft/automation/v1.0/companies({companyId})/extensionUpload(0)/content
I hope it helps.


Converting a keras Sequential CNN to estimator and training it removes all layers, then crashes for want of layers

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): ('v1.12.0-0-ga6d8ffae09', '1.12.0')
Python version: 2.7.15rc1
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): c++ (Ubuntu 7.3.0-27ubuntu1~18.04) 7.3.0
CUDA/cuDNN version: Not installed
GPU model and memory: GeForce GTX 1050 Ti, 4GB

Describe the current behavior
This issue was also mentioned in #21778, but I believe this warrants an issue of its own.
I ran into this problem trying tho make a CNN using the fashion_mnist dataset. Instead of training the model, invoking train on the model seems to drop all the layers of the original model, and then crash because there are no layers.
The example below has this output:
WARNING:tensorflow:Using temporary folder as model directory: /tmp/tmp_l6v51cf
[<tensorflow.python.keras.layers.core.Reshape object at 0x7fcb25244a90>, <tensorflow.python.keras.layers.convolutional.Conv2D object at 0x7fcb07f670f0>, <tensorflow.python.keras.layers.core.Flatten object at 0x7fcb07f7fa20>, <tensorflow.python.keras.layers.core.Dense object at 0x7fcb07f7fb00>]
WARNING:tensorflow:From /home/heikki/Koodaus/python_sketches/venv/lib/python3.6/site-packages/tensorflow/python/estimator/inputs/queues/feeding_queue_runner.py:62: QueueRunner.__init__ (from tensorflow.python.training.queue_runner_impl) is deprecated and will be removed in a future version.
Instructions for updating:
To construct input pipelines, use the `tf.data` module.
WARNING:tensorflow:From /home/heikki/Koodaus/python_sketches/venv/lib/python3.6/site-packages/tensorflow/python/estimator/inputs/queues/feeding_functions.py:500: add_queue_runner (from tensorflow.python.training.queue_runner_impl) is deprecated and will be removed in a future version.
Instructions for updating:
To construct input pipelines, use the `tf.data` module.
[]

Describe the expected behavior
I would expect train to train the model without emptying the layers-array. Also, the deprecation warning probably shouldn't be there either.
Code to reproduce the issue
import numpy as np
import tensorflow as tf
from tensorflow import keras

network = keras.Sequential([
            keras.layers.Reshape((10, 10, 1)),
            keras.layers.Conv2D(filters=8, kernel_size=(4, 4), activation=tf.nn.elu),
            keras.layers.Flatten(),
            keras.layers.Dense(1, activation=tf.nn.relu)
        ])

network.compile(optimizer=tf.keras.optimizers.SGD(lr=0.0001, momentum=0.9),
                loss='categorical_crossentropy',
                metric='accuracy')

estimator = tf.keras.estimator.model_to_estimator(keras_model=network)

train_input_fn = tf.estimator.inputs.numpy_input_fn(
        x=np.random.rand(10, 100),
        y=np.random.rand(10, 1),
        num_epochs=1,
        shuffle=False)

print(network.layers)

try:
    estimator.train(input_fn=train_input_fn, steps=10)
except:
    print(network.layers)

Other info / logs
Without the try/except, the error thrown is
Traceback (most recent call last):
  File "/home/heikki/Koodaus/python_sketches/Estimator.py", line 26, in <module>
    estimator.train(input_fn=train_input_fn, steps=10)
  File "/home/heikki/Koodaus/python_sketches/venv/lib/python3.6/site-packages/tensorflow/python/estimator/estimator.py", line 354, in train
    loss = self._train_model(input_fn, hooks, saving_listeners)
  File "/home/heikki/Koodaus/python_sketches/venv/lib/python3.6/site-packages/tensorflow/python/estimator/estimator.py", line 1207, in _train_model
    return self._train_model_default(input_fn, hooks, saving_listeners)
  File "/home/heikki/Koodaus/python_sketches/venv/lib/python3.6/site-packages/tensorflow/python/estimator/estimator.py", line 1237, in _train_model_default
    features, labels, model_fn_lib.ModeKeys.TRAIN, self.config)
  File "/home/heikki/Koodaus/python_sketches/venv/lib/python3.6/site-packages/tensorflow/python/estimator/estimator.py", line 1195, in _call_model_fn
    model_fn_results = self._model_fn(features=features, **kwargs)
  File "/home/heikki/Koodaus/python_sketches/venv/lib/python3.6/site-packages/tensorflow/python/estimator/keras.py", line 278, in model_fn
    labels)
  File "/home/heikki/Koodaus/python_sketches/venv/lib/python3.6/site-packages/tensorflow/python/estimator/keras.py", line 201, in _clone_and_build_model
    optimizer_iterations=global_step)
  File "/home/heikki/Koodaus/python_sketches/venv/lib/python3.6/site-packages/tensorflow/python/keras/models.py", line 448, in clone_and_build_model
    _in_place_subclassed_model_reset(clone)
  File "/home/heikki/Koodaus/python_sketches/venv/lib/python3.6/site-packages/tensorflow/python/keras/models.py", line 318, in _in_place_subclassed_model_reset
    name = layers_to_names[layer]
KeyError: <tensorflow.python.keras.layers.core.Reshape object at 0x7fde28e5ca90>

I was able to reproduce your error message in TF 1.12.0
However using TF 1.13.rc0 gave more clear error message as below:
ERROR:tensorflow:Model diverged with loss = NaN.
By solving this error message I was able to use train to train the model.

How does one solve that error? I tried changing the loss function to tf.losses.mean_squared_error, but that didn't change the result on 1.12.0.

The error arises due to the assignment of x y```` args to tf.estimator.inputs.numpy_input_fn``` . If you rerun the code snippet as is multiple times it should pass at next try. I see that currently it is random assignment to x, y which leads to model diverging in some runs.

I see. I tried to change the code to this:
fashion_mnist = keras.datasets.fashion_mnist
(train_images, train_labels), (test_images, test_labels) = fashion_mnist.load_data()

input_height = test_images.shape[1]
input_width = test_images.shape[2]
network = keras.Sequential([
    keras.layers.Reshape((input_height, input_width, 1)),
    keras.layers.Conv2D(filters=8, kernel_size=(4, 4), activation=tf.nn.elu),
            keras.layers.Flatten(),
            keras.layers.Dense(10, activation=tf.nn.relu)
        ])

network.compile(optimizer=tf.keras.optimizers.SGD(lr=0.00001, momentum=0.9),
                loss=tf.losses.mean_squared_error,
                metric='accuracy')

estimator = tf.keras.estimator.model_to_estimator(keras_model=network)

train_input_fn = tf.estimator.inputs.numpy_input_fn(
        x=test_images,
        y=test_labels,
        num_epochs=1,
        shuffle=False)

print(network.layers)

try:
    estimator.train(input_fn=train_input_fn, steps=10)
except:
    print(network.layers)

But the behavior repeats, including emptying the network's layers.

Apologies for the delay in response. The new version of code snippet you provided throws a different error in TF 1.13.1 Can you please confirm? Thanks!

The error I'm getting now is this:
Traceback (most recent call last):
  File "Estimator.py", line 31, in <module>
    estimator.train(input_fn=train_input_fn, steps=10)
  File "/home/heikki/.local/lib/python2.7/site-packages/tensorflow_estimator/python/estimator/estimator.py", line 358, in train
    loss = self._train_model(input_fn, hooks, saving_listeners)
  File "/home/heikki/.local/lib/python2.7/site-packages/tensorflow_estimator/python/estimator/estimator.py", line 1124, in _train_model
    return self._train_model_default(input_fn, hooks, saving_listeners)
  File "/home/heikki/.local/lib/python2.7/site-packages/tensorflow_estimator/python/estimator/estimator.py", line 1154, in _train_model_default
    features, labels, model_fn_lib.ModeKeys.TRAIN, self.config)
  File "/home/heikki/.local/lib/python2.7/site-packages/tensorflow_estimator/python/estimator/estimator.py", line 1112, in _call_model_fn
    model_fn_results = self._model_fn(features=features, **kwargs)
  File "/home/heikki/.local/lib/python2.7/site-packages/tensorflow_estimator/python/estimator/keras.py", line 278, in model_fn
    labels)
  File "/home/heikki/.local/lib/python2.7/site-packages/tensorflow_estimator/python/estimator/keras.py", line 201, in _clone_and_build_model
    optimizer_iterations=global_step)
  File "/home/heikki/.local/lib/python2.7/site-packages/tensorflow/python/keras/models.py", line 511, in clone_and_build_model
    target_tensors=target_tensors)
  File "/home/heikki/.local/lib/python2.7/site-packages/tensorflow/python/training/checkpointable/base.py", line 442, in _method_wrapper
    method(self, *args, **kwargs)
  File "/home/heikki/.local/lib/python2.7/site-packages/tensorflow/python/keras/engine/training.py", line 449, in compile
    output_loss = weighted_loss(y_true, y_pred, sample_weight, mask)
  File "/home/heikki/.local/lib/python2.7/site-packages/tensorflow/python/keras/engine/training_utils.py", line 647, in weighted
    score_array = fn(y_true, y_pred)
  File "/home/heikki/.local/lib/python2.7/site-packages/tensorflow/python/ops/losses/losses_impl.py", line 669, in mean_squared_error
    predictions.get_shape().assert_is_compatible_with(labels.get_shape())
  File "/home/heikki/.local/lib/python2.7/site-packages/tensorflow/python/framework/tensor_shape.py", line 1023, in assert_is_compatible_with
    raise ValueError("Shapes %s and %s are incompatible" % (self, other))
ValueError: Shapes (?, 10) and (?,) are incompatible

The layers, however, seem to be intact now, so I'm willing to call this fixed.

Closing this issue since the original error has now been resolved. Feel free to post a new issue if have further problem. Thanks!


.NET 6 Isolated Functions: Unable to edit App Files with Web Deploy

I'm unable to edit app files for a .NET 6 Isolated Azure Function.

Configurations
App

Function

General

This is by design. As the message suggests, please edit your files in your local machine using your favorite editor. Once you are done, publish the output of your app and deploy that artifacts. Remember, you are not deploying the source code, but the publish output of that (result of dotnet publish).


Benchmarks: add use of binaryen wasm-opt

Derek Schuff encouraged me to use binaryen's wasm-opt optimizer on the wasm emitted by llvm - this is one additional pass that Emscripten performs, and it apparently makes a big difference in performance.

The current stable version of wasm-opt segfaults with our benchmarking suite.
I'm currently trying to see if this is a recent regression, and what exact optimization pass is doing this.


General Comments

Below are the comments I have compiled from reading through the paper. Note that I have very limited knowledge of the subject so many of the conceptual concerns may be due to a lack of technical knowledge. Nevertheless here are a my comments as a layperson:
p.4 - difference between cloze and prefix prompting, concepts of hard vs soft prompts, and idea of what constitutes task agnostic techniques not entirely clear to me
P.5 - MMLU benchmarking criteria is introduced with very little explanation, perhaps that would good for the reader to have bit of context (it is included in references) also note that references and acknowledgements sections are not included in table of contents
2.1.1 - meaning of masked frame / window
2.1.1 - text only mentions arXiv, yet graphic (and preceding paragraph) include that records are pulled from SS and ACL as well
I found figure 2.3 very helpful for understanding the parameters and characteristics of exemplars, perhaps make it more of a focal point
Figure 2.5 - 'letter' should be 'letters'
p.10 - difference between few shot prompting and few shot learning is a bit confusing
2.2.2 - paragraph wording
<IP_ADDRESS> - 'present' should be 'presents'
2.4 - technique -> techniques
2.4 - concept of 'frozen LLM' a bit elusive to me
2.4 - in the RLprompt paragraph, the phrasing 'grammatical gibberish' might have a better alternative
2.4 - DP20 paragraph wording
2.5 - should be "variance response format"
3.2 - one case of the word "technique" needs to be made plural
4.0 - section 4 first paragraph, second sentence needs to be fixed
4.1.3 - what is meant by "toy environments"
5.1.1 - definition of prompt injection, are original developer instruction in the prompt of being overridden with the use of the prompt?
5.1.3 - technique -> techniques
6.1 - technique -> techniques
6.2.1 - signal -> signals
6.2.3 - explain concepts of precision and recall.
Overall I found it very informative  and thorough but a bit challenging as a non-expert. There were many instances of words that were incorrectly not pluralized, specifically the word technique. My other general advice would be to give context for some more specific phrases and technical concepts e.g. MMLU, backprop, decoder-only models, or even more basic things like what an exemplar label is. some of these concepts get introduced somewhat abruptly and could discombobulate non technical readers. Food for thought if this paper is intended to be read by the lay-person

Ty!

[ ] p.4 - difference between cloze and prefix prompting, concepts of hard vs soft prompts, and idea of what constitutes task agnostic techniques not entirely clear to me
[ ] P.5 - MMLU benchmarking criteria is introduced with very little explanation, perhaps that would good for the reader to have bit of context (it is included in references) also note that references and acknowledgements sections are not included in table of contents
[ ] 2.1.1 - meaning of masked frame / window
[ ] 2.1.1 - text only mentions arXiv, yet graphic (and preceding paragraph) include that records are pulled from SS and ACL as well
[ ] I found figure 2.3 very helpful for understanding the parameters and characteristics of exemplars, perhaps make it more of a focal point
[ ] Figure 2.5 - 'letter' should be 'letters'
[ ] p.10 - difference between few shot prompting and few shot learning is a bit confusing
[ ] 2.2.2 - paragraph wording
[ ] <IP_ADDRESS> - 'present' should be 'presents'
[ ] 2.4 - technique -> techniques
[ ] 2.4 - concept of 'frozen LLM' a bit elusive to me
[ ] 2.4 - in the RLprompt paragraph, the phrasing 'grammatical gibberish' might have a better alternative
[ ] 2.4 - DP20 paragraph wording
[ ] 2.5 - should be "variance response format"
[ ] 3.2 - one case of the word "technique" needs to be made plural
[ ] 4.0 - section 4 first paragraph, second sentence needs to be fixed
[ ] 4.1.3 - what is meant by "toy environments"
[ ] 5.1.1 - definition of prompt injection, are original developer instruction in the prompt of being overridden with the use of the prompt?
[ ] 5.1.3 - technique -> techniques
[ ] 6.1 - technique -> techniques
[ ] 6.2.1 - signal -> signals
[ ] 6.2.3 - explain concepts of precision and recall


agent: do not call wg.Done before wg.Add

fixes race seen in https://github.com/coreos/fleet/pull/1287

weird that this didn't come up earlier. LGTM


Error on local host in windows

Skipping legacy plugin 'FCNPC'; It requires memory hacking to run and is therefore broken on open.mp. There should be a replacement component supported by open.mp

Yes, open.mp isn't supported, use sa-mp.

No collision data found.
[FCNPC] Warning: Unable to create NPCs. The maxnpc limit in server.cfg is 0.

The maxnpc limit in server.cfg is 0.

https://cdn.discordapp.com/attachments/1084514051540910120/1089231870287429694/image.png

?

@ziggi

?

Done
Tnx


update interpret-community to interpret-core v0.3.2

update interpret-community to interpret-core v0.3.2

Codecov Report
Patch coverage has no change and project coverage change: -0.27 :warning:

Comparison is base (2fdac32) 87.66% compared to head (c0975d0) 87.39%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #556      +/-   ##
==========================================
- Coverage   87.66%   87.39%   -0.27%
==========================================
  Files          62       62
  Lines        4094     4007      -87
==========================================
- Hits         3589     3502      -87
  Misses        505      505

Flag
Coverage Δ

unittests
87.39% <ø> (-0.27%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 23 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


how to close tab?

What versions are you running?
$ go list -m github.com/chromedp/chromedp
github.com/chromedp/chromedp v0.7.6
$ google-chrome --version
Google Chrome 95.0.4638.54
$ go version
go version go1.17.2 linux/amd64

What did you do? Include clear steps.
how to close tab?
tried below code, it works but it blocks execution forever.
ctx, _ := chromedp.NewContext(parent, chromedp.WithTargetID(targetToBeClosed))
chromedp.Run(ctx, page.Close()) // the page is closed, but execution stuck here

i know the function page.close is marked as experimental, that's why i'm asking if there's an api to close a tab, or only this one.
thank you

sorry i'm late to tell that i run it in callback function that registered to chromedp.ListenBrowser.
my goal is, when a tab has a url that match something, the tab will be closed automatically.
to reproduce the issue, run below code:
package main

import (
	"github.com/chromedp/chromedp"
	"github.com/chromedp/cdproto/page"
	"github.com/chromedp/cdproto/target"
	"context"
	"strings"
	"log"
)

func main() {
	opts := append(chromedp.DefaultExecAllocatorOptions[:],
		chromedp.Flag("headless", false),
	)
	allocCtx, cancelAlloc := chromedp.NewExecAllocator(context.Background(), opts...)
	ctx, cancelCtx := chromedp.NewContext(allocCtx)
	defer cancelAlloc()
	defer cancelCtx()

	if err := chromedp.Run(ctx, chromedp.Navigate("about:blank")); err != nil {
		panic(err)
	}

	chromedp.ListenBrowser(ctx, func(ev interface{}) {
		switch ev := ev.(type) {
		case *target.EventTargetInfoChanged:
			// if tab url match, then close the tab
			if strings.HasSuffix(ev.TargetInfo.URL, "httpbin.org/anything") {
				targetCtx, _ := chromedp.NewContext(ctx, chromedp.WithTargetID(ev.TargetInfo.TargetID))

				log.Println("cancelling")
				chromedp.Run(targetCtx, page.Close()) // here, cancelling context didn't work either
				log.Println("cancelled")
			}
		}
	})

	<-chromedp.FromContext(ctx).Browser.LostConnection
}

when chrome window opens, try navigating to http://httpbin.org/anything:

https://github.com/chromedp/chromedp/blob/master/chromedp.go#L674-L678


impossible to add more than 200 members

it works great up to 200 group members, then it's impossible to add more, I always get the
Some Other error in adding
error.
Any way to overcome this?

use multiple accounts

do you face any error like indexError:list out of range

no the only error is the one above.
I guess it's the standard telegram limit.
I wrongly assumed this script would have bypassed it.. The script is nice, but imo you should underline this limit in your readme.
It's not so handy having to create multiple accounts, each one with its phone number and api codes..


don't allocate/update hessians when calculate_hessian = false

When the background intensity is computed, for example, its gradient and Hessian should not be computed. Same for when computing active pixels. Same for when evaluating the rust region model quality.

Done in #436


Recalculate health checks on host removal

https://github.com/rancher/rancher/issues/3763
@cjellick could you please review?

LGTM


python: add support for python2/python3

@sbinet apologies for the late reply.  Can you explain what this PR does a bit further ?  Is it syntax changes to support Python 3?

this PR modifies (slightly) the python code to be both python2(.7) and python3 compliant.

@sbinet wow, I dropped the ball on this and just now noticed.  I will merge.


Added iOS official client

I use this client for almost 1 month on ios 11 and 12.
It working fine

Hello! Thanks for the PR.
In the last days, the script has changed a lot, and I implemented OpenVPN 2.4 features. I remember the iOS app not supporting those a while ago.
If possible, could you try with a freshly installed OpenVPN server with the latest version of the script and default parameters? If not, I will wait for someone else's feedback.

Hey J'ai le Script de euh Lundi je crois et ça marche ici :)

@MartinPC62 Thanks for the feedback. 👍
However I added most of OpenVPN 2.4's features yesterday.

okay, I reinstall openvpn and try ~~

Working fine ~~ on iOS 12 in IphoneSE
Though, I got this error message while I add user on Debian9 vultr VPS

Thanks 👍

I can't reproduce your errors though, but that's another subject

This pull request can now be closed.


Customizations Not Applied in Single Ragapp Deployment

I'm encountering an issue where customizations are not being applied when deploying Ragapp using the single configuration setup. Specifically, the customizations that are available in the project ('patch' folder) are not reflected in the application when using the single deployment mode, and I'm unsure how to apply them and if this is the right way
Thank you

I created first the Ragapp customizated image and later created the Ragapp single. don´t know if it is the best way but it is working great.
Thank you


allow future IntelliJ versions

great thanks - but how does this work exactly?

Is there a reason for this not being merged?
I would love to help out!

Hi - sorry I haven't worked on this for a few years and don't plan on doing any more work to it - however if someone wants to take it over I can give access

@kingsleyh I would love to take over and continue on your great work!

This patch is not needed anymore.


[WIP] Regression Benchmark

This feature has been supported in https://github.com/open-mmlab/mmgeneration/tree/dev-1.x/.dev_scripts
I think @LeoXing1996 can help to improve this pull request so that it can be merged into master branch.


Calling MockDatabaseReference.update with null values adds null values to database instead of deleting the entry

Description:
when setting a value of DatabaseReference to null using update, FirebaseDatabase removes this value from the database.
MockDatabaseReference.update inserts a null value instead
How  to reproduce:
test('MockDatabaseReference removes value when updating with null', () async {
  DatabaseReference reference = MockFirebaseDatabase.instance.ref('path'); // test works with FirebaseDatabase.instance.ref('path')

  await reference.update({'x':'value'});

  var snapshot = await reference.get();
  expect(snapshot.value, {'x':'value'});

  await reference.update({'x':null}); // should remove the entry

  snapshot = await reference.get();
  expect(snapshot.value, null); // this fails, actual: {'x':null}
});

Thanks for reporting this, I will work on it as soon as I get some free time. PRs are also welcome :)


Autocomplete API routes

Describe the problem
I would love to see autocompleted API routes.
Describe the proposed solution
One great example of this is how NuxtJS 3 shows you autocompleted API routes when you use their useFetch hook, and shows you a list of API routes you have at hand. This makes sure that you don't have a URL that goes no where, and can warn the developer before an insight is overlooked, or a typo happen and spending hours on 1 typo.
Alternatives considered
No response
Importance
would make my life easier
Additional Information
No response

Closed as duplicate of https://github.com/sveltejs/kit/issues/647


[363] Redact access request controller

Context

This trello ticket

Changes proposed in this pull request

Redact V2 controller

Guidance to review
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Visit code debt on V2


Add support in multiple ports per instance

Currently an amalgam8 instance has only one endpoint.
Both kubernetes and eureka allows multiple endpoints per service/instance. It sounds reasonable to add support in multiple endpoints to amalgam8.
Adding multiple endpoints requires API changes.
My suggestion is to add a new field of type array, called endpoints, and document the current endpoint field as deprecated. Users can continue to use endpoint, and the new code will continue to set this field (as well as the new endpoints field).
Later, we can decide to drop the endpoint field and stop supporting the old API version.
Current API:
type ServiceInstance struct {
ID                   string                   json:"id,omitempty"
ServiceName   string                   json:"service_name,omitempty"
Endpoint        *InstanceAddress  json:"endpoint,omitempty"
...
}
New API format:
type ServiceInstance struct {
ID                   string                   json:"id,omitempty"
ServiceName   string                   json:"service_name,omitempty"
// Endpoint is deprecated. The Endpoints array replaces this field.
Endpoint        *InstanceAddress  json:"endpoint,omitempty"
Endpoints     []*InstanceAddress json:"endpoints,omitempty"
...
}

Are you referring to multiple ports per instance ? If there are different IP and port pairs for the service instance, there needs to be an additional metadata to identify the endpoint type (we have one already) and an additional field to prioritize which endpoint in the list should the caller prefer.
I am assuming that you simply want to support multiple ports per instance, where endpoint type field is different for each endpoint.
The sidecar can then choose the https endpoint by default.
I would rename the issue as multiple ports per instance.

@rshriram yes, the intent is to support multiple ports per instance. Using an endpoints array (with each endpoint having a type and a value, as is today) is one way of doing so, with the added flexibility of specifying multiple ip:port pairs for a service instance. If that doesn't make sense, then we can probably just add a ports array (named ports) to the existing endpoint field.

Adding sidecar tag as this feature should affect the sidecar as well.
Particularly, the sidecar should be aware of multiple endpoints/ports per instance.
Load balancing should be done across service instances (normally, nodes) and not endpoints, whereas routing rules + sensible defaults should handle multiple endpoints per instance.

I can understand what it means to have an instance that implements multiple endpoints/ports, but I don't understand what it means to have multiple endpoints/ports on a single service, and how it would be exposed in the rules API. I would have thought that if an instance has two endpoints (for example: port 1234 and port 5678) those two endpoints are different services, not both implementing the same service. Wouldn't it be better to handle this by allowing instances to register themselves as implementing two different service (e.g., SERVICE=foo_1234 and SERVICE=foo_5678)?

@frankbu the discussion here was in the context of multiple endpoints/ports for the a single service. For multiple services, it probably makes more sense to just register separate service instances.
The canonical (and only?) example for multiple ports for a single service is HTTP vs HTTPS.
An example for multiple IPs for the same service instance is when an instance provides the service over multiple network interfaces. Not sure it makes much sense in a containers world, maybe with VMs.


Multi party Relin key generation

I am using SEAL to implement multi party Relin key following the following protocol in [1]:

I wonder if I can use the same decomposition basis w that SEAL generated in
uint64_t factor = barrett_reduce_64(key_modulus.back().value(), get<1>(I));
to compute sw (s_iw in the protocol). s is provided by modifying
generate_kswitch_keys(secret_key + 1, count, static_cast<KSwitchKeys &>(relin_keys), save_seed);
to
generate_kswitch_keys(secret_key, count, static_cast<KSwitchKeys &>(relin_keys), save_seed);
Reference:
[1] Mouchet C, Troncoso-Pastoriza J, Bossuat J P, et al. Multiparty homomorphic encryption from ring-learning-with-errors[J]. Cryptology ePrint Archive, 2020.

You cannot use generate_kswitch_keys function to implement that algorithm. Fundamentally, generate_kswitch_keys encrypts a new secret key located at secret_key + 1 with the old secret key located at secret_key, where all randomness are generated during the encryption. What the algorithm describes requires a as an input that cannot be fed into generate_kswitch_keys. Also the algorithm is more complicated than Microsoft SEAL's relinearization key generation. You might want to ask the authors whether they know of any implementation based on Microsoft SEAL.

@WeiDaiWD Thanks for the information, I modified code as the protocol described (given common random polynomial a, designing a function in rlwe.cpp to output $-u_i a+ e_{0,i}, s_i a +e_{1,i}$)
 SEAL_ITERATE(iter(new_key, key_modulus, destination, size_t(0)), decomp_mod_count, [&](auto I) {             SEAL_ALLOCATE_GET_COEFF_ITER(temp, coeff_count, pool_);             encrypt_zero_symmetric_crp_round_one(                 secret_key_, context_, u_, key_context_data.parms_id(), true, save_seed, get<2>(I).data());             uint64_t factor = barrett_reduce_64(key_modulus.back().value(), get<1>(I));             multiply_poly_scalar_coeffmod(get<0>(I), coeff_count, factor, get<1>(I), temp);             CoeffIter destination_iter = (*iter(get<2>(I).data()))[get<3>(I)];             add_poly_coeffmod(destination_iter, temp, coeff_count, get<1>(I), destination_iter);         });
Function encrypt_zero_symmetric_crp_round_one outputs $-u_i a+ e_{0,i}, s_i a +e_{1,i}$. My question is that using factor generated by
uint64_t factor = barrett_reduce_64(key_modulus.back().value(), get<1>(I));
and old secret key
generate_kswitch_keys(secret_key, count, static_cast<KSwitchKeys &>(relin_keys), save_seed);
can I compute the correct $s_i w$ then compute $-u_i a+ s_i w + e_{0,i}, s_i a +e_{1,i}$?
Thank you!

I see. Then it is just about the value of factor. The decomposition basis used in Microsoft SEAL is described in this paper in section B.2.1. This basis equals to 1 modulo get<1>(I) and therefore disappears from the formula that calculates factor. Microsoft SEAL adopts a special prime for noise reduction, described in section B.1.2, which is key_modulus.back().value(). If you only want to have a new basis w, you should choose uint64_t factor = barrett_reduce_64(w[j], get<1>(I)), where j iterates all elements in the basis w. If you further want to use the same noise reduction adopted by Microsoft SEA, you should have key_modulus.back().value() multiplied to factor; you need to modify the algorithm to make it correct.

Thank you so much. I have completed the algorithm following your suggestion and successfully generated the multiparty version relin key.


Update README.md (grammmatical errors)

This pull request will fix some possible grammatical errors, 3 I spotted.

Resolves grammatical errors (the ones I saw)

Thanks
@swift-ci smoke test and merge


[FEATURE] Support Version Decoupling in Maps Dashboards Plugin

Is your feature request related to a problem?
OpenSearch plugins currently tightly couple their versions with the underlying OpenSearch/OpenSearch Dashboards version. This tight coupling can lead to challenges in maintaining and upgrading plugins independently. To enable supporting of the decoupling of the plugin versions from the OpenSearch versions will allow for more flexibility in managing and updating plugins.
More details is available at RFC Plugins Version Decoupling - https://github.com/opensearch-project/OpenSearch-Dashboards/issues/5877
What solution would you like?
Support version decoupling in the maps dashboards plugin, where end users would be able to consume the latest version of maps dashboards plugin with a range of supported OS versions instead of just one version
What alternatives have you considered?
More as detailed in Plugins Version Decoupling
Do you have any additional context?
More as detailed in Plugins Version Decoupling

Tagging @vamshin to prioritize this in next release


implement interval_us functionality

Implemented ability to sleep before sending response as intended
by the ResponseParameters proto.

LGTM
waiting for green

Test this please

@yang-g Do these failures look related? I don't think they are but I'll defer to you.

Test this please


OrchestrationClient has been renamed

The OrchestrationClient binding was renamed to DurableClient in v2.0.0-beta2 and the example should reflect this. This version was released 31 Aug 2019. What is the process for updating docs and examples after releasing new versions?

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

ID: 4bda2389-aeb2-b28f-a646-082979122f88
Version Independent ID: 4fa3fdc7-4f59-ff5e-6e42-bc1ed2b2e306
Content: Task hubs in Durable Functions - Azure
Content Source: articles/azure-functions/durable/durable-functions-task-hubs.md
Service: azure-functions
GitHub Login: @cgillum
Microsoft Alias: azfuncdf

@estensen Thank you for your feedback! We will review and update as appropriate.
You can always contribute to the Microsoft docs. You can leverage this article: https://docs.microsoft.com/en-us/contribute/

@estensen Thanks for reporting this item! I have updated the doc and it will be published soon. We appreciate the feedback!


onyo_tree() uses weird tuple for dirs

The onyo_tree function accepts a tuple (str, Path) for dirs.
This is in contrast to how other commands accept paths. The str is to show the user the directories they called, in the format they called it.
This avoids loads of complexities with relative vs absolute paths, and CWD that is not repo-root.
Questions are (non-exclusive of each other):

should onyo_tree print always relative to repo root
what other commands are candidates for this tuple-approach
is there a better solution than these tuples

@bpoldrack @TobiasKadelka Not sure if I captured your suggestions/thoughts well here. Please add them to help clarify what you two see.

Registering the idea of PathParameter class, that allows to capture that sort of annotation (currently description). onyo_XXX could potentially accept both Path or PathParameter. Constructor would take a Path and an optional annotation (implementing a default ofc). This class could behave like a Path and only overwrite its __str__ to make use of the annotation. So, for any actual path operation code doesn't need to change and only when calling str() on it there would be an effect.

I experimented with subclassing Path (which, as an aside, was not possible prior to 3.12) and simply overriding its __str__ donder to return the string that it received when instantiated.
I now believe this approach is not a good one.
My approach was something like:
class PathParameter(Path):
    def __init__(self, p: str, *args, **kwargs) -> None:
        self._original = Path(p, *args, **kwargs)
        super().__init__(p, *args, **kwargs)

    def __str__(self) -> str:
        return str(self._original)

And this mostly works. The self._original = Path(p, *args, **kwargs) is where things get fun.
Imagine the following code:
print(path / self.ANCHOR_FILE_NAME)

The self.ANCHOR_FILE_NAME is passed as an arg to path, and it bubbles up all the way to __init__, creating a new object for print.  So capturing those args is important. Check.
Except lets consider the following:
p = PathParameter(d).resolve()

This does not return Path() that is .resolve()ed. Rather .resolve() returns a new PathParameter that was instantiated with the resolved path. Now we have lost d, which was the entire point. After reading the pathlib source, I do not see a trivial way to wedge our desired functionality into resolve() (not to mention other functions that behave similarly).
Simply adding an attribute is the next logical solution. However, it fails the path.resolve() scenario for the exact same reasons.
Even if it is possible to override this cleanly, I have objections:

loads of functions will need their type annotations adjusted
all the new objects created will carry this thoroughly irrelevant information

the point here isn't waste, it's rather: that these objects are used for tons of path traversal. Onyo will check Path(/) at some point, and it would still carry this inane self.original. It's so irrelevant that it's dangerous.

I explored using PathParameter at only the CLI level and into onyo_tree(). From there on, everything would be converted into Paths. Perhaps a bit wasteful, but really not that bad given all the object creation going on anyway. Except path.resolve() rears its head again. We resolve all paths at the CLI layer, before passing into the onyo_*() functions. And I strongly believe this is the right thing to do.
The only two viable paths I see are:

keep using the tuple
create a non-pathlike object that carries the "description" and the resolved path object. onyo_tree() will accept either Path or this bespoke type.

Uff, thanks for your work, this seems to have been a tough quest.
Reading through this, I think the best course of action is 1., and to just continue to live with the situation of the tuple.

Thanks for the research! I should have realized that resolve does return a new object, ofc, since it needs to make it a concrete path and resolve against actual FS.
However, I still think this is solvable.

My approach was something like:
class PathParameter(Path):
  def __init__(self, p: str, *args, **kwargs) -> None:
     self._original = Path(p, *args, **kwargs)
       super().__init__(p, *args, **kwargs)

   def __str__(self) -> str:
       return str(self._original)

I would now try to do this:
class PathParameter(Path):
    def __init__(self, p: str, *args, **kwargs) -> None:
        self._original = Path(p, *args, **kwargs)
        super().__init__(p, *args, **kwargs)

    def __str__(self) -> str:
        return str(self._original)
    def __getattr__(self, item) -> Any:
        result = super().__getattr__(item)
        if isinstance(result, Path):
            # we got a new path-like back
            new_path = PathParameter(result)
            new_path._original = self._original
            return new_path
        return result

So, whenever a method does return a new Path (or subclass) we wrap that thing before returning it and maintain original. Otherwise we are transparent.
The second best option is a simple (data-) class to replace the tuple, and that's only intended to transport that information between CLI and command implementations (seems to be 2.).
In any case: I would be fine with post-poning the resolution of this issue - it's not urgent at all. That is: 1.) but don't close this issue.

This is functionally the same thing I was doing before, and will fail in the following situation:
print(path / self.ANCHOR_FILE_NAME)

When a new object is created, it makes sure to retain _original. And __str__() returns _original. Thus only path is printed.
The problem comes down to:

we want to retain this information
if we do so, all future objects will retain either irrelevant or (even worse) incorrect information

These two are at odds with each other.
I agree that the tuple is weird, but creating an entire dataclass just to avoid a tuple also feels silly. onyo_tree() is unlikely to be used by anyone outside of cli/tree.py, but telling consumers that they must import a bespoke dataclass in order to pass in a string as a sidecar seems even sillier than saying "pass in an awkward tuple".
I will add a note in the source code, and then close this issue.
Ben knows that DataLad was dealing with similar problems, and he is interested in seeing how they addressed them. Perhaps their solution may be of relevance to us. At which point, this issue can be reopened or a new issue created.

Hah! And an idea strikes once the issue is closed. Reopening...


Crash when outputting from Industrial smelter to a condenser

crash-2020-09-01_21.54.36-server.txt
crash-2020-09-01_21.53.52-server.txt

This seems to be an Astromine issue, I've already talked with one of the devs and I believe a fix is on the way. If the issue is actually on Industrial Revolution, reopen the issue and I'll fix it. Thanks for the report!


Receive failed should be just on one line

Received failed over 2 lines on Zooko's phone, should be just 1 line.

Fixed


closes #2175

Checklist:

[x] Updated extVersionCode value in build.gradle for individual extensions
[x] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[x] Added the isNsfw = true flag in build.gradle when appropriate
[x] Have not changed source names
[x] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio

The title should be a very short description of what you did, and "closes #..." should be in the body, not title
pull requests must be tested or else they won't be merged


Check links against the SHA commit

Current Verify-Links.ps1 throw exception if the link not exist on master. This should be valid link after code checked in. We currently have workaround to comment out the link in first PR and add it back in a following PR.
This PR is to replace the link with SHA and check if the link valid under SHA, and throw exception only if the replaced link is also invalid.
Test code:https://github.com/Azure/azure-sdk-for-android/pull/289/checks?check_run_id=982955662
Test pipeline: https://dev.azure.com/azure-sdk/public/_build/results?buildId=495914&view=logs&jobId=b70e5e73-bbb6-5567-0939-8415943fadb9

New test pipeline: https://dev.azure.com/azure-sdk/public/_build/results?buildId=496973&view=logs&s=6884a131-87da-5381-61f3-d7acc3b91d76

https://dev.azure.com/azure-sdk/public/_build/results?buildId=496994&view=logs&j=b70e5e73-bbb6-5567-0939-8415943fadb9&t=e75da802-0139-5977-bd1e-91175bec9059

/azp run azure-sdk-tools - sync - eng-common


Isolate example crashes when the isolate completes

Steps to Reproduce

Run these commands:

cd flutter/examples/layers/
flutter run -t services/isolate.dart

Click the Start button

=> App crashes when the progress reaches 100%
F/libc    (16109): Fatal signal 11 (SIGSEGV), code 1, fault addr 0x5c in tid 16132 (ui_thread)

@a-siva but we discussed in email; can you assign it to the engineer that is looking at this?

Created pull request https://github.com/flutter/engine/pull/4218

Landed https://github.com/flutter/engine/pull/4218 to fix this issue.

Thanks!

Do we have a test for his?

examples/layers

@a-siva looks like isolate_test.dart just runs the main() method? I think we need to extend that test with a tap on the 'Start' button?

@mit-mit @a-siva Not sure it is related, but I tested the isolate.dart example and now it crashes as soon as I press the start button, instead of at 100% of the progress as it was in alpha 0.0.16. Using the debugger, this is the last Dart line it manages to reach:
Isolate.spawn(_calculate, message).then<Null>((Isolate isolate) {
It also seems that it doesn't enter the _calculate function from the other isolate.
When I run it from console/intellij I just see a Lost connection to device. error.
If I try to run it from the xcode project, I receive this:
2017-11-09 01:49:29.252669+0100 Runner[23488:362570] Observatory listening on http://<IP_ADDRESS>:60411/
2017-11-09 01:49:34.046035+0100 Runner[23488:362585] [VERBOSE-3:dart_init.cc(350)] Check failed: Dart_IsolateMakeRunnable(isolate). ```

I'm experiencing this same issue in alpha 0.0.17, and I'm experiencing it also in my own code every time I try to spawn an Isolate.

Are you on master or alpha? I also still get a crash in alpha, but have it working on master.

I just tried to run it with a completely new master directory and I'm able to reproduce it consistently. Here is my flutter doctor:
    • Flutter at /Users/sroddy/flutter/flutter
    • Framework revision 436aa93086 (2 hours ago), 2017-11-09 07:12:36 +0100
    • Engine revision bc2acf7bdd
    • Tools Dart version 1.25.0-dev.11.0

[✓] Android toolchain - develop for Android devices (Android SDK 26.0.0)
    • Android SDK at /Users/sroddy/Library/Android/sdk
    • Platform android-26, build-tools 26.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] iOS toolchain - develop for iOS devices (Xcode 9.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.0.1, Build version 9A1004
    • ios-deploy 1.9.2
    • CocoaPods version 1.3.1

[✓] Android Studio (version 3.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] IntelliJ IDEA Community Edition (version 2017.2.5)
    • Flutter plugin version 18.4
    • Dart plugin version 172.4343.25

[✓] Connected devices
    • iPhone SE • BA3ECA04-6D6A-41DE-9AB7-02CDCD6D56F1 • ios • iOS 11.0 (simulator)```

I can repro now! It does work on Android Emulator, and physical iOS devices, but if I run an iPhone SE in the Simulator I also get the "Lost connection to device." error when I click the button.
@sroddy would you mind opening a new bug for this?

yes, no prob.


Emmet not working any more - instead Js Types are shown in Intellisens

VSCode Version: Code 1.15.1 (41abd21afdf7424c89319ee7cb0445cc6f376959, 2017-08-16T18:07:25.676Z)
OS Version: Windows_NT x64 10.0.15063
Extensions: the listing length exceeds browsers' URL characters limit

Steps to Reproduce:

Reproduces without extensions: Yes/No

@HansS

If you want to expand Emmet abbreviations and snippets using tab, set emmet.triggerExpansionOnTab to true
If you are referring to js like this.someproperty triggering emmet suggestion like <this class="some property"></this>, please follow #32647

There have been quite a few changes to Emmet in the July release.
Please read the Release Notes on Emmet or Emmet 2.0 blog and the Emmet docs
If the above still doesn't help, then please provide more details include screenshots/gifs and repro steps


Code requires address and private key

Hi
Executing the following command results in TypeError: init() missing 2 required positional arguments: 'address' and 'private_key'
from uniswap import Uniswap
uniswap_wrapper = Uniswap()
eth = "0x0000000000000000000000000000000000000000"
bat = "0x0D8775F648430679A709E98d2b0Cb6250d2887EF"
dai = "0x89d24A6b4CcB1B6fAA2625fE562bDD9a23260359"

uniswap_wrapper.get_eth_balance(bat)

Could not find the clarification on how to set the address/private key in ReadMe file... Would appreciate help on the issue.
Thanks!

@alex33d No, that is not possible. For that, you'd need an archive node to parse the event logs, which is way beyond the scope of this library.
Duplicate of https://github.com/shanefontaine/uniswap-python/issues/16, closing.

This is now mentioned in the README: https://github.com/shanefontaine/uniswap-python/commit/b5f21fcbb18a537bf99a044e84c4b7fa31938fe7


Add version number to drawer menu

Summary | Résumé
Adds the app version number to the drawer menu.
This will help with support when asking folks what version of the app they are using.

Test instructions | Instructions pour tester la modification
Open the drawer menu
Scroll to the bottom
Note the version #

@amazingphilippe please see screen above and let me know if that works okay.

@amazingphilippe please see screen above and let me know if that works okay.

Looks good to me!

Looks good to me!


DHT11Plugin merge

First merge of the DHT-11 plugin. I hope everything is right - I'm new to the whole github/pull/branch/merge part - my last contribution to a project some time ago was still a patch file ;-)
The code compiles, however it's untested as of now, as the pixelix has already been given away :-)

Trying to link to the corresponding Issue: #28

Make it sense that you rework it first, with the knowledge of the last review?

Yes, definitly, I'll do that.


I am working on a Awesome-Python-HTMX, seeking your feedback

A few of us at PyCon this year got together and brainstormed a new Web Stack that we are calling PyHAT (Python, htmx, ASGI, TailwindCSS). The first thing we set out to do is create awesome-python-htmx; a collection of active tools/libraries/projects in that space. Your project seems like an obvious thing to include. In fact, I had never heard of it before, but seemingly you have already beaten us to the punch on building out an opinionated template for the PyHAT stack :sweat_smile: . I have included it under a "templates" section.
If you would participate in PyHAT-stack/awesome-python-htmx#1 that would be greatly appreciated!

I'm also going to include https://oluwatobi.dev/blog/posts/htmx-support-in-pycharm, super cool!

Wow, great project and initiative, I love the spirit and the stack and the name is definitely better than what I have for my own project right now.
Two of the best django htmx resources I know of that seem to be missing on the repo right now are

https://github.com/spookylukey/django-htmx-patterns
https://youtube.com/@bugbytes3923

I'll make sure to keep PyHAT in mind when I release any new htmx and Django projects / content in the future, and I'll be sure to contribute them to the awesome-python-htmx collection. Thanks for including my project, and I'm excited to see how PyHAT evolves!

@Tobi-De thanks for your feedback, it's very appreciated! I have added both of resources you mentioned, you are correct they are extremely good, I will definitely need to take the time to go through them all.
As you have expressed interest in getting more involved, would you mind introducing yourself on PyHAT-stack/awesome-python-htmx#2 so we can start getting everyone talking in the same space?


[improve][pip] PIP-321  Split the responsibilities of namespace replication-clusters

Motivation
Because the configurations of clusters at the namespace level are unclear, some flexible topic policies can not work as expected, e.g. geo-replication at the topic level. see more information about the bug at https://github.com/apache/pulsar/pull/21564.
Modifications
Use replication-clusters, allowed-clusters, and topic-policy-synchronized-clusters to replace a single replication-clusters originally in the namespace policy.
See more explanation in the proposal.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

@Technoboy- Your comments will highly appreciated in the discussion of this PIP in the mailing list.

@Technoboy- Your comments will highly appreciated in the discussion of this PIP in the mailing list.

Replied


Drivers linked to dead link

You cant download the drivers. The links just take you to the microsoft home page
[Enter feedback here]

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

ID: 3cded3d0-9d0a-ce94-1c91-21de62e3ddaf
Version Independent ID: c14c4066-056b-faea-0a28-5eba2c544a43
Content: Download - JDBC Driver for SQL Server
Content Source: docs/connect/jdbc/download-microsoft-jdbc-driver-for-sql-server.md
Service: sql
Sub-service: connectivity
GitHub Login: @David-Engel
Microsoft Alias: v-davidengel

Thanks for the heads up! This is fixed now.
Regards,
David
#please-close


child.type.update is not a function

Please, push the new version to npm, because in npm it has this code:
// reducers.js
function childMonitorState(props, state, action) {
  var child = props.children;
  return child.type.update(child.props, state, action);
}

but master branch is:
// reducers.js
function childMonitorState(props, state, action) {
  var child = props.children;
  return child.type.reducer(child.props, state, action);
}

and it's don't wanna work :worried:

It's actually the other way around—I forgot to push the local commits.
child.type.update is the final API.

It still does not work if the latest realise is 1.0.1.
Just installed it and got the same error.
Simply change update to reducer and it's working.

What monitor do you have inside of DockMonitor?
Whatever it is, it's not compatible with DevTools 3.0.
Maybe you're using<EMAIL_ADDRESS>instead of 1.0.2?
Or some other monitor that hasn't been updated for DevTools 3.0 final.

You can see that this version of package is used both in Redux DevTools examples and in Real World example in Redux repo. So it's not correct to say "it does not work". It does not work in some specific circumstances, but without clarifying them I can't help you.

I'm sorry, yes, I did use log-monitor beta-3 insead.
Thank you :)


all contributors

@all-contributors please add @JoachimGoedhart @mgalland

Was missing the bot. Should work now.

@all-contributors please add @JoachimGoedhart @mgalland

@all-contributors please add @JoachimGoedhart for code, bug barbecue, examples, tool, test and much more!

@all-contributors please add @mgalland for ideas, project management and tests.

@all-contributors please add @mgalland for ideas, project management and tests.

@all-contributors please add @ebrando for data, example, ideas, project management, research, user testing and much more!

@all-contributors please add @ebrando for data, example, ideas, project management, research, user testing and much more!

@all-contributors please add @ebrando for data, example, ideas, project management, research, user testing and much more!

@all-contributors please add @mgalland for ideas, project management, testing and research.


chore: release v1.5.0

bump version
add changelog

Codecov Report

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

@@            Coverage Diff            @@
##             v1.5.0    #1360   +/-   ##
=========================================
  Coverage          ?   29.48%
=========================================
  Files             ?       81
  Lines             ?    10163
  Branches          ?        0
=========================================
  Hits              ?     2997
  Misses            ?     6839
  Partials          ?      327

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Hi, please have a look at the failed CI 🙂


Fix post media preview

Follow this checklist for submitting a pull request (PR):

[X] Your branch is up-to-date with the main branch.
[X] You are submitting your PR against the main branch of the repository you are contributing to from your fork.
[X] You have linked your GitHub issue at the Resolves Issue section if it exists.
[X] You have provided a brief description of the issue.

Description
Quick description of what this pull request does.

This pull request resolves the video upload issue, yet, i am not sure if it can break something or if there are any security problems, ill explain why i did everything so it can help to find remanescent bug faster

Resolves Issue: proudly closes #203

Thank you, I had to weed out the dependa bots pull request

@Ronnie018  are you able to resolve these conflicts

@Ronnie018 are you able to resolve these conflicts

I never had to, so im a litte lost, but ima getting there

I'll close it for now

You just click on resolve issues and delete the section that is no longer needed. It will show you the 2 sections it is comparing. You can write part of the code from one into the other if you only need part of it.

one code actually overwrites the other in this case I am rewriting my code over his.

okay, that makes since. Thank you Ronnie, really seriously thank you. This sickness is worse then when I had covid. I'm sorry for not being on the ball today. Hopefully I'll snap out of it within a few days but thank you.

okay, that makes since. Thank you Ronnie, really seriously thank you. This sickness is worse then when I had covid. I'm sorry for not being on the ball today. Hopefully I'll snap out of it within a few days but thank you.

I didn't know you were sick, I hope you get better.

Yes I've been feeling crappy for a few days and have not got much work done because of it.