System tests might be deleting all datasets inside our test project

https://github.com/googleapis/nodejs-bigquery/blob/960d84e3e320685d3766508e6d8a3988c1b6397e/system-test/bigquery.ts#L1672-L1680
There's no reference to prefix option to list datasets:
https://cloud.google.com/bigquery/docs/reference/rest/v2/datasets/list
And we're suspecting it is causing a test dataset in nodejs-dlp to be deleted, causing its nightly to fail:
https://github.com/googleapis/nodejs-dlp/issues/52#issuecomment-415837211

filter with label: https://cloud.google.com/bigquery/docs/labeling-datasets#filtering_datasets_using_labels

Elevating to p0 since it is blocking other repositories from passing system tests and releasing.

P0PR: https://github.com/googleapis/nodejs-bigquery/pull/177/files


Feature/keyboard kyria

Using Kyria layout as provided here: https://docs.splitkb.com/hc/en-us/articles/360010818820-Is-there-a-Keyboard-Layout-Editor-template-available-for-the-Kyria-

@daanhorn
LGTM👍 Thanks!

@daanhorn
LGTM👍 Thanks!


Added license meta file

This fixes installing via the git URL, which throws an error at the moment like has no meta file, but it's in an immutable folder. The asset will be ignored.

Thanks!


Rare Unix exception during SMT solving

I've very rarely run into the following error when running the SMT solver (stacktrace included on the offchance it's helpful:
Fatal error: exception Unix.Unix_error(Unix.EMFILE, "pipe", "")
Raised by primitive operation at file "src/lib/smt/SolverUtil.ml", line 26, characters 28-58
Called from file "src/lib/smt/Smt.ml", line 148, characters 15-34
Called from file "src/lib/Main_defs.ml", line 94, characters 10-60
Called from file "src/lib/utils/Profile.ml", line 10, characters 12-16
Called from file "src/lib/Main_defs.ml", line 139, characters 8-146
Called from file "src/exe/Main.ml", line 21, characters 12-37
Called from file "src/lib/utils/Profile.ml", line 10, characters 12-16
Called from file "src/exe/Main.ml", line 31, characters 2-67
I've only seen it twice and so far it hasn't been repeatable. The most recent time was during network slicing: it failed on slice number 125 (out of 500). All the slices should be isomorphic. It's rare, but it completely crashes NV, so it's a pain if it happens during a really long execution (like it did above). I don't have a great grasp of the system calls happening here; anyone else have any ideas?

The exception EMFILE means that the process has opened too many files, in this case pipes to a z3 process. I don't think I've placed proper cleanup code for old pipes (whatever old means) as it used to be the case that you will run only 1 instance of z3. Once NV is done the pipes are closed automatically. This is probably only observable in the context of slicing where depending on how you implemented things you fork a new z3 process and a new pipe for each slice.
Fixes should be:

You don't fork a new z3 process for every slice, instead you reuse the existing z3 process. Maybe fork N processes if you do parallelism. But in general there is an overhead to forking new processes so there should be reuse.
Implement proper cleanup of open channels.


Add base branch matching

Allow the specifying of the base branch along with the matching of a
head branch. This is a quite a simple change as we use the same code for
branch matching, but tailor it for base branches.
This is great for my usage, as I can then label new PRs dependant on
where they're going to land, helping priortize and categorize incoming
changes to a repo.

thanks @SimonRichardson! I should be able to review this today


support UDTF in FE

What type of PR is this：

[ ] bug
[x] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：
For #3074
Problem Summary(Required) ：
Now we use CREATE TABLES FUNCTION ... to create UDTF. because CREATE TABLE may has conflict with create table name (). later we should fix

This PR may cause compatibility problems, as older versions of FE do not recognise TableFunction, but we can provide some compatibility patches to allow older versions of FE to be skipped.

You should add UT cover it.

It's hard to add UT. I'll add a Functional Test


Scrolling error: cannot unmarshal number into Go struct field SearchHits.hits.total of type elastic.TotalHits

I am using version v7.0.10 of this library along with Elastic 7.5.1 which is the latest available in Elastic Cloud.
Here is the code (trimmed for the sake of this example):
scroll := elasticsearch.Client.Scroll("my_index").
    Query(query).
    Size(100).
    IgnoreUnavailable(true)
for {
    results, err := scroll.Do(context.Background())
    if err == io.EOF {
        return nil
    }
    if err != nil {
        log.Printf("Error while scrolling: %s", err.Error()) // <--- HERE
        continue
    }
}

This is triggering the error:

Error while scrolling: json: cannot unmarshal number into Go struct field SearchHits.hits.total of type elastic.TotalHits

Is this because v7.0.10 of this library is different from the one I use in Elastic Cloud (Even though it's still ES7)? Anything else I am doing wrong?
Note: This seems to be only happening while scrolling. Other Search queries are working correctly.

The error indicates that you're using a combination of Elasticsearch 6 and elastic v7 or vice versa. The reason for this error is that ES 7.x changed the response structure to return the total hits.
See e.g. https://github.com/olivere/elastic/issues/1007 for a detailed discussion.

@olivere As mentioned, I am using v7.0.10 with Elasticsearch 7.5.1

This is the breaking change in ES 7.x, and elastic v7 will parse that just fine. There is a test for it (also for Scroll API), and we're using it in production for months now.
I don't know why it is wrong in your combination.

@olivere This is the content of my go.mod file:
module my/module

go 1.13

require (
        ...
	github.com/olivere/elastic/v7 v7.0.10
        ...
)

And a screenshot of our Elastic Cloud cluster.

This is very confusing, but I'll keep digging. Thank you.


ActivityStreams 2.0 support

Recently the draft for ActivityStreams 2.0 was published, which

...builds upon that initial foundation by incorporating lessons learned through extensive implementation, community feedback and related work being performed in other standards development communities.

In the near future it would most likely be beneficial to implement this newer spec, seeing as Pump is the image of a successful AS project right now.

This will go into the release after next, along with ActivityPub (probably).

What's the difference between #1483 and #851?

#851 is an issue and #1483 is a pull request intended to resolve that issue.

Will Activitystreams and Activitypub get implement at the same time?

Well, AS2 landed on master a week and a half ago. I believe Evan is still hacking away on the AP implementation. It's likely that it'll ship at the same time but we're not sure.


Await watch call

Add missing await to calls to watch(...) so subsequent inserts get caught by the watcher and furthermore to allow stopping the watcher properly.
Fixes #9.

Thanks for noticing the issue, and for your quick website fix :)
It's probably also a good idea if you were to update the JSDoc comments in watch.ts as well, they also don't include the await keyword yet.


Update git action go versions to 1.21.13, 1.22.6, 1.23.0

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.06%. Comparing base (edf9e7c) to head (c90dbe7).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #19   +/-   ##
=======================================
  Coverage   99.06%   99.06%
=======================================
  Files           3        3
  Lines         213      213
=======================================
  Hits          211      211
  Misses          1        1
  Partials        1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add integrations page to ironicbaremetal.org

There are numerous places where ironic is used or can
be used by an infrastucture operator. This page is intended
to help reveal some of these cases so people can recognize
the ecosystem is larger than just one project.

@jimmytipit So the nav bar looks a little over to the right. At least with my browser on my desktop. Thoughts?

I think Wes did that when he added the Home button and Pixie Boots in to the top bar to keep things a little less crowded.
On May 3 2021, at 1:50 pm, Julia Kreger @.***> wrote:

@jimmytipit (https://github.com/jimmytipit) So the nav bar looks a little over to the right. At least with my browser on my desktop. Thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/OpenStackweb/ironic-website/pull/29#issuecomment-831459917), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AARD32K6HXFLJAIP2EZDDJLTL3WBHANCNFSM44BKCT6Q).

I guess we could nuke the explicit home link since it is redundant.
On Mon, May 3, 2021 at 12:10 PM Jimmy McArthur @.***>
wrote:

I think Wes did that when he added the Home button and Pixie Boots in to
the top bar to keep things a little less crowded.
On May 3 2021, at 1:50 pm, Julia Kreger @.***> wrote:

@jimmytipit (https://github.com/jimmytipit) So the nav bar looks a
little over to the right. At least with my browser on my desktop. Thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (
https://github.com/OpenStackweb/ironic-website/pull/29#issuecomment-831459917),
or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AARD32K6HXFLJAIP2EZDDJLTL3WBHANCNFSM44BKCT6Q
).

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OpenStackweb/ironic-website/pull/29#issuecomment-831471419,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMXQZB6YWYIVQPJRSDSQITTL3YLTANCNFSM44BKCT6Q
.

Yes, I made the header nav updates when the logo was added. I support removing "Home" from the main nav since the logo is present now


API docs not updated

Using v 2.0.0, looks like documentation / readme is not updated, trying the example from readme
dhan.place_order(security_id='52175',           # Nifty PE
exchange_segment=dhan.NSE_FNO,
transaction_type=dhan.BUY,
quantity=550,
order_type=dhan.MARKET,
product_type=dhan.INTRA,
price=0)

Gives error:
[<ipython-input-21-ffa5fc03369e>](https://8563w6di12g-496ff2e9c6d22116-0-colab.googleusercontent.com/outputframe.html?vrz=colab_20241030-060124_RC00_691361479#) in <cell line: 1>()
      1 dhan.place_order(security_id='52175',           # Nifty PE
----> 2     exchange_segment=dhan.NSE_FNO,
      3     transaction_type=dhan.BUY,
      4     quantity=550,
      5     order_type=dhan.MARKET,

AttributeError: 'dhanhq' object has no attribute 'NSE_FNO'

Looking into the source code made below change:
dhan.place_order(security_id='52175',           # Nifty PE
    exchange_segment=dhan.FNO,
    transaction_type=dhan.BUY,
    quantity=550,
    order_type=dhan.MARKET,
    product_type=dhan.INTRA,
    price=0)

But it gives below error from API:
{'status': 'failure',
 'remarks': {'error_code': 'DH-905',
  'error_type': 'Input_Exception',
  'error_message': 'Missing required fields, bad values for parameters etc.'},
 'data': {'errorType': 'Input_Exception',
  'errorCode': 'DH-905',
  'errorMessage': 'Missing required fields, bad values for parameters etc.'}}

@dhan-oss  Any update on this issue ?

Hello @mithunmanohar
Two things here. First, the declared constant NSE_FNO was missing in 2.0.0, which has been added in 2.0.1. On the error that you are getting for using dhan.FNO and still not able to place order, can you check Security ID list and ensure that the lot size and security ID are correct, as these keep on changing.


Segmentation fault and corrupted output during inference with large 3D U-Net

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): YES
OS Platform and Distribution: Linux CLE 7.0UP00
TensorFlow installed from (source or binary): binary
TensorFlow version: 2.0.0
Python version: 3.7.4

Describe the current behavior
I have a 3D U-Net model that I am trying to use for very large inputs (arrays as large as 1024^3). Obviously, I cannot fit this in GPU memory so I am running the network on CPU compute nodes which have a very large amount of RAM available (~350 GB). I have a "bare-bones" script which performs only the forward pass using the 3D convolution methods from tf.nn, but I am only able to process arrays of size  <= 640^3 without issues. If I go for the next feasible size (which is 704^3 since I can only increase spatial dimensions of the input by multiples of 64), the following behavior occurs:

If using tensorflow with the Intel MKL library, which is optimized for running on CPUs on my system, the network runs without crashing but the generated output is corrupted (see attached images). The corruption seems to only occur along the outermost spatial axis of the output array.

If using tensorflow-gpu or regular tensorflow (without Intel MKL), the program crashes with a segmentation fault

Though the input sizes are very large, I would expect tensorflow to be able to complete the computations without issue since I have enough RAM available. The difference in behavior with or without Intel MKL is also perplexing.
Code to reproduce the issue
Below is a script that performs the U-Net computation on random data and reproduces the segfault/bug:
import tensorflow as tf
import numpy as np

size=704
dtype = np.float32
sample = np.random.normal(size=(1,size,size,size,4)).astype(dtype)
print(sample.shape)

convs = [(4, 4, 4, 4, 64),
         (4, 4, 4, 64, 128),
         (4, 4, 4, 128, 256),
         (4, 4, 4, 256, 512),
         (4, 4, 4, 512, 512),
         (4, 4, 4, 512, 512)]
Tconvs = [(4, 4, 4, 512, 512),
          (4, 4, 4, 512, 1024),
          (4, 4, 4, 256, 1024),
          (4, 4, 4, 128, 512),
          (4, 4, 4, 64, 256),
          (4, 4, 4, 5, 128)]

x = tf.constant(sample)
skips = []
shapes = []

# Downsampling
for kernelshape in convs:
    bias = np.zeros(shape=(kernelshape[-1]))
    kernel = np.random.normal(loc=0.0, scale=0.02, size=kernelshape)
    x = tf.nn.conv3d(x, filters=kernel.astype(dtype), strides=[1,2,2,2,1], padding='SAME')
    x = tf.nn.bias_add(x, bias.astype(dtype))
    x = tf.nn.leaky_relu(x, alpha=0.3)
    skips.append(x)
    shapes.append(x.shape)
    print(x.shape)

# Upsampling with skip connections
skips = list(reversed(skips[:-1]))
for kernelshape, skip in zip(Tconvs, skips):
    bias = np.zeros(shape=(kernelshape[3]))
    kernel = np.random.normal(loc=0.0, scale=0.02, size=kernelshape)
    out_shape = np.array(x.shape.as_list())
    out_shape[1:4] = 2*out_shape[1:4]
    out_shape[-1] = kernel.shape[3]
    x = tf.nn.conv3d_transpose(x, filters=kernel.astype(dtype), strides=2, padding='SAME', output_shape=out_shape)
    x = tf.nn.bias_add(x, bias.astype(dtype))
    x = tf.nn.relu(x)
    x = tf.concat([x, skip], axis=-1)
    print(x.shape)

# Final upsampling conv
kernelshape = Tconvs[-1]
bias = np.zeros(shape=(kernelshape[3]))
kernel = np.random.normal(loc=0.0, scale=0.02, size=kernelshape)
out_shape = np.array(x.shape.as_list())
out_shape[1:4] = 2*out_shape[1:4]
out_shape[-1] = kernel.shape[3]
x = tf.nn.conv3d_transpose(x, filters=kernel.astype(dtype), strides=2, padding='SAME', output_shape=out_shape)
x = tf.nn.bias_add(x, bias.astype(dtype))
x = tf.math.tanh(x)

print(x.shape)
x = x.numpy().astype(np.float32)

Other info / logs
The attached images show the output of the U-Net (running on random noise) for the problematic size (704^3), via 2D slices along the x-y and x-z planes:

The images should look like uniform noise, but instead have a sharp change occur along the x-axis after index~520.
On my actual dataset, the corruption looks like:

Thank you for reporting the issue!
I tried this on an Ubuntu 16.04 VM + Python 3.7.4 + 360GB memory. I could reproduce the corrupted image issue with both TensorFlow-MKL (Intel-optmized TensorFlow) and regular TensorFlow. (Regular TensorFlow didn't crash for me.) Will try to fix the corrupted image issue first and then we can see if regular TensorFlow still crash for you.

@pzharrington @penpornk  !
The session is crashing at below line in Colab .
sample = np.random.normal(size=(1,size,size,size,4)).astype(dtype)
before going to other steps in 2.8 version onwards.
Attached gist 2.10.0 for reference.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Add pixel aspect support

This is useful for older DVD-era videos which were usually encoded with non-square pixels.

Thanks!


This commit damages the coverage report by removing tools/base and should be re-worked before submission to master.

Could you review the changes https://github.com/OPAE/opae-sdk/pull/1230 which fixed the coverage issue?

Coverage decreased (-0.7%) to 80.764% when pulling db1d4c5a69e8d8f302cbb2251309b806e77f3b1e on tswhison/rev_0cdd49f into 0cdd49f254452c87bfa85776d6b419ca0e54f939 on master.

Coverage issue corrected by #1230


Find Support: Disable mouse zoom of Google map

Since the Google map takes up a fair portion of the main Find Support page, I often inadvertently place my mouse in that area when attempting to scroll down the page to see the listed resources.  From the perspective of this user only at this point, I would prefer if the mouse zoom in/out function be disabled and the user would need to use the + & - menu option to do so.  Clicking on numbered circles and such should still zoom as usual.

Also noted on slide 84 in the 09/17/2019 section of the feedback deck:
Whenever you’re scroll down a page that has the map, it often accidentally zooms in and out on the map which can be a little annoying. Not sure if there would be anything to help that.

Some info on how to tell agm to ignore the scroll gesture: https://github.com/SebastianM/angular-google-maps/issues/1411


about mosei raw data

The link to the original CMU-MOSEI dataset http://immortal.multicomp.cs.cmu.edu/raw_datasets/CMU_MOSEI.zip is no longer available, could you please share it?

Sorry for the inconvenience. Immortal servers are currently being moved so are temporarily unavailable. CMU-MOSEI can be downloaded here alternatively: https://github.com/matsuolab/CMU-MultimodalSDK/tree/master/mmsdk/mmdatasdk/dataset/standard_datasets/CMU_MOSEI


Expose libdeflate's crc32 algorithm, closes #11

Thanks for the PR! I'll get this and what looks to be some incoming tests wrapped up into a new release sometime this week.


Broken link to: path

The link to more info on 'path' is broken.

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

ID: a87a82b5-133d-b6df-fc51-98a2f77a0d50
Version Independent ID: d9b312de-c845-cb25-ff25-9c36a5d307dd
Content: StorageProviderSyncRootManager.Register(StorageProviderSyncRootInfo) Method (Windows.Storage.Provider) - Windows UWP applications
Content Source: winrt-api-build/xml/Windows.Storage.Provider/StorageProviderSyncRootManager.xml
Product: uwp
Technology: winrt-reference
GitHub Login: @lastnameholiu
Microsoft Alias: alholiu

Thanks @jandrewb for pointing this out. The link has been fixed and the changes should be live within 24 hours.


Refactor LinearExpression and Constraint using composition

Since there are unsafe methods in xr.Dataset that do not correctly pass on the LinearExpression or Constraint behaviour, we switch the two classes to a composition instead of the current inheritance pattern.
This is meant to address @lumbric's comment https://github.com/PyPSA/linopy/issues/43#issuecomment-1283949895 that another design pattern would be saner.
Note that several methods that are probably safe are not forwarded through to the new LinearExpression.
Variable is still inheritance based and should also be converted similarly.
Several docstrings are still missing.

@coroa this is great stuff! Stable and backwards-compatible (with a tiny fix, the pypsa optimization already runs through). If you want I'd propose the implementation for the Variable class to properly understand everything. Otherwise I'd wait till you say this is ready for review. What do you think?

Great, thank you! I am not familiar with the code-base, but LGTM and seems to fully resolve the issues addressed in #43.
Note that to_dataset() is deprecated, but still used.

I continued a bit further and tried to convert the Variable class as well, but hit a road-block with xarray eagerly left multiplying into Variables, ie.
c * x

if c is an xarray of coefficients and x is the non-xarray inherited Variable class, then it does not evaluate the __rmul__ but falls into xarray trying to __mul__ a Variable that has been converted to an ndim=0 numpy array, like
np.array(Variable(...)). I don't see a way around this.
Stepping through xarray's left multiplication, I see that they had a special case for inherited xarrays, which was hit before. Not sure, whether this can be solved.

To give you a head-start to dig into the problem, consider the following test case (which is the setup of test_sum_drop_zeros at https://github.com/PyPSA/linopy/blob/e10c096bea6b30f15633c69b70532ba06afa97b8/test/test_linear_expression.py#L234):
import pandas as pd
import xarray as xr
from linopy import Model

m = Model()
z = m.add_variables(0, pd.DataFrame([[1, 2], [3, 4], [5, 6]]).T, name="z")

coeff = xr.zeros_like(z.labels)
coeff[1, 0] = 3
coeff[0, 2] = 5
expr = coeff * z

will fail with
Traceback (most recent call last):
  Input In [9] in <module>
    expr = coeff * z
  File ~/.local/conda/envs/pypsa/lib/python3.10/site-packages/xarray/core/_typed_ops.py:212 in __mul__
    return self._binary_op(other, operator.mul)
  File ~/.local/conda/envs/pypsa/lib/python3.10/site-packages/xarray/core/dataarray.py:3098 in _binary_op
    f(self.variable, other_variable)
  File ~/.local/conda/envs/pypsa/lib/python3.10/site-packages/xarray/core/_typed_ops.py:402 in __mul__
    return self._binary_op(other, operator.mul)
  File ~/.local/conda/envs/pypsa/lib/python3.10/site-packages/xarray/core/variable.py:2469 in _binary_op
    result = Variable(dims, new_data, attrs=attrs)
  File ~/.local/conda/envs/pypsa/lib/python3.10/site-packages/xarray/core/variable.py:305 in __init__
    self._dims = self._parse_dimensions(dims)
  File ~/.local/conda/envs/pypsa/lib/python3.10/site-packages/xarray/core/variable.py:573 in _parse_dimensions
    raise ValueError(
ValueError: dimensions ('dim_0', 'dim_1') must have the same length as the number of data dimensions, ndim=0

Indeed, that is a bit tricky. I'm thinking about overloading the xarray Dataset and DataArray mul function. But still not sure how secure this would turn out and where to actually do that.
xarray uses a _binary_op function to assign arithmetic operations with type distinction, resulting in differentiation for the  __mul__ operator. I have to do some more research on what is going on there.

overloading the xarray Dataset and DataArray

nah, that is not a valid option, since you don't want to control the types of coeff.

Indeed, this makes sense. However, I am still lingering with the idea of supplementing the mul function. Something like:
import linopy
import pandas as pd
import xarray as xr

def __mul__(da, other):
    if isinstance(other, linopy.Variable):
        return other * da
    else:
        return da * other

xr.DataArray.__mul__ = __mul__

m = linopy.Model()

coords = [pd.RangeIndex(10, name='a')]
x = m.add_variables(coords=coords, name='x')

c = xr.DataArray(range(10), coords=coords)

c * x

which works fine.

Indeed, this makes sense. However, I am still lingering with the idea of supplementing the mul function. Something like:

NO! Really, no. That is what you would typically type piracy.

We could ask in an issue on xarray's tracker if they can add some sort of hooking mechanism? Or what they would suggest for our use case.

We could ask in an issue on xarray's tracker if they can add some sort of hooking mechanism? Or what they would suggest for our use case.

Yes, let's do that. In general, I think it is important to further support the coefficient-variable (in that order) multiplication pattern.

Hmm, i thought about this once more and I think it is an acceptable approach if we limit it to the above and
maybe warn about it in the documentation.
Maybe adding some syntactic sugar around it:
@monkey_patch(xr.DataArray, pass_unpatched_method=True)
def __mul__(da, other, unpatched_method):
    if isinstance(other, linopy.Variable):
        return NotImplemented
    return unpatched_method(da, other)

where we define monkey_patch like:
from functools import update_wrapper, partial
def monkey_patch(cls, pass_unpatched_method=False):
    def deco(func):
        wrapped = getattr(cls, func.__name__)
        if pass_unpatched_method:
            func = partial(func, unpatched_method=wrapped)
        update_wrapper(func, wrapped)
        setattr(cls, func.__name__, func)
        return func
    return deco

After some clean-up including several doc-strings. This might be getting somewhere.
It still hinges on whether we want to allow the monkey patching? (which now has utility in
common.py and is triggered from variables.py, maybe this should move into a dedicated
monkey_patch_xarray.py module to make it super obvious).
I think we can ask whether xarray could adhere to the scheme set out by numpy that
if __array_ufunc__ is set to None and other implements __rmul__ then it returns
NotImplemented (see at https://numpy.org/doc/stable/reference/arrays.classes.html#numpy.class.array_ufunc ).

No idea what the proper solution would be and what to change in xarray. I am slightly confused how __rmul__ is supposed to be used and don't have better suggestions than the approach suggested by @coroa.
Wouldn't be the only sane approach for the implementation for __mul__ in xarray (and numpy and everywhere else) to raise NotImplemented in case other is of unknown type? In that case there would be absolutely no problem or need for any kind of other hooking mechanism. However, I just checked a couple of types (numpy arrays, str, int, dict, ...) and all of them raise a TypeError for types they can't handle. This makes sense semantically because it is a type error in most cases and not a missing implementation, but it prevents the usage of __rmul__ if types are not inherited as we see here.

However, I just checked a couple of types (numpy arrays, str, int, dict, ...) and all of them raise a TypeError for types they can't handle.

What do you mean by checked? You tried multiplying them? The logic for a * b is. Python tries in order:

check if issubclass(type(b), type(a))
if it is:
a. call b.__rmul__(a)
b. if it (returned NotImplemented or it is not defined) call a.__mul__(b)
if it isn't
a. call a.__mul__(b)
b. if it (returned NotImplemented or it is not defined), call b.__rmul__(a)
if we get NotImplemented again or it is not defined, then a TypeError is raised.

Uh sry, seems as if I got some things wrong when writing my previous comment. Let me correct:
I have the impression that the only sane way to implement a.__mul__(b) is to return a reasonable value for a * b or to return NotImplemented if the class type(a) does not know how to handle a multiplication with b. Standard data types like int, str, float and dict seem to follow this pattern, but numpy, xarray and pandas do not. I don't understand why (my guess is that it makes implementation of __mul__ easier).
This example demonstrates what I mean:
class MyType:
    def __mul__(self, other):
        return 42

    def __rmul__(self, other):
        return 43

MyType() * other is always 42 as one can expect. other * MyType() has some surprising results. In my opinion, there is no reasonable result other than 43 for other * MyType(), which means the implementation of __mul__ in numpy, pandas and xarray should be changed.
The good part:
>>> 3 * MyType()
43
>>> MyType() * 3
42
>>> 'string' * MyType()
43
>>> MyType() * 'string'
42
>>> 3. * MyType()
43
>>> MyType() * 3.
42
>>> object() * MyType()
43
>>> MyType() * object()
42

Numpy, xarray and pandas do some really surprising things, but all of them follow the same pattern:
>>> np.array(3.) * MyType()
43
>>> MyType() * np.array(3.)
42
>>> np.array([3., 4.]) * MyType()
array([43, 43], dtype=object)
>>> MyType() * np.array([3., 4.])
42
>>> pd.Series([1, 2, 3]) * MyType()
0    43
1    43
2    43
>>> MyType() * pd.Series([1, 2, 3])
42
>>> xr.DataArray(3) * MyType()
array(43)
>>> MyType() * xr.DataArray(3)
42

Thanks @coroa for this nice change :)


Changing fingerprint to a working one

I have tested and now the lib works for me

FINGER PRINT IS NOT WORKING

Hello JeniferMArshall,
What is the exert error that you are getting?
Regards Kamau Solomon
On Sun, 7 Nov 2021 at 03:51, JeniferMarshall @.***>
wrote:

FINGER PRINT IS NOT WORKING
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FirebaseExtended/firebase-arduino/pull/532#issuecomment-962531437,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRYJPYFG276RTJ72REVL6LUKXEQRANCNFSM5G375BMQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Also try changing the fingerprint to
"6E:F3:55:47:6D:F1:0E:5B:2B:04:F2:F8:C0:8B:E8:05:65:0F:1A:C1"
This worked for me. Let me know if any issues or -ve feedback
Regards Kamau Solomon
On Sun, 7 Nov 2021 at 06:49, Solomon Kamau @.***> wrote:

Hello JeniferMArshall,
What is the exert error that you are getting?
Regards Kamau Solomon
On Sun, 7 Nov 2021 at 03:51, JeniferMarshall @.***>
wrote:

FINGER PRINT IS NOT WORKING
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FirebaseExtended/firebase-arduino/pull/532#issuecomment-962531437,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRYJPYFG276RTJ72REVL6LUKXEQRANCNFSM5G375BMQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@googlebot I signed it!


The color of the button obscures the text content

Gitea version (or commit ref): docker image id  9058d7d9f1a1
Git version: 2.28.0
Operating system: macOS 11.0.1

Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No

Description
This button does not display the master text by default, and the master text can only be displayed when I select the button text

Fix for this added on https://github.com/go-gitea/gitea/pull/13733.

@silverwind
Great, I am using a white theme, how can I change it to dark?

User menu, setting, account.

@silverwind
Hi, I found another interface problem. The two buttons below are pasted together. This looks strange. I hope these two buttons can be separated.

Addressed in https://github.com/go-gitea/gitea/pull/13744.


Porting ga('set', ... ) calls

I am porting some ga code to use segment. There are the ga('set', .... ) calls for which I dont see a corresponding version in analytics. It is not possible to pass in the custom dimensions as properties because they were being set in a different part of the page, and the track/page calls happen elsewhere.
What do you suggest we do to solve this?

hey @ferozed - your issues sounds specific to the Google Analytics integration. Could you open a ticket the Google Analytics integration repository instead?
I might also shoot a request via http://segment.com/contact to make sure this gets seen by the right team.

Well, it is google related but it is applicable outside of google as well.
What I am proposing is, we add a Analytics.prototype.set(props) method. The props would be buffered internally. And when users call track or page or identify, the properties passed in to the api calls would be merged with those passed in with the set call.

I don't think we should add this to analytics.js unless we have a examples of multiple tools needing this functionality.
For now I think this logic should either live in the GA integration, or you can call into GA directly.


How can i get the Leetcode API access?

Hi I understand, due to DMCA notice you can't keep this repository online, but can you please help me, with where i can find Leetcode api so that i can modify the code locally and use it.
Thanks :)

If i would share it here, there would be no point of me shutting this extension down :), sorry i cant

Ooh right, so can we connect to some other more secure platform (like discord telegram)?


wrong value using Switch or TextField(!) in combination with updateOn: 'blur'

The first toggle seems to get lost. After that the value is 'true' if the switch is off and 'false' if the switch is on
This is the repo to reproduce this issue:
https://github.com/gms1/issues/tree/master/nativescript_angular/switch-updateOnBlur

Hi @gms1,
Thank you for the sample project.
I tested this case while using the Switch component with '@angular/forms'. Indeed the valueChanges method will not be executed on first Switch value change. For further info, please keep track on the issue.

Hi @tsonevn,
regarding Switch and other controls, I wondered how an "updateOn: blur" could work, as long as there are no focus/blur events from the underlying framework for these controls.
However, I am getting a similar behavior using Textfield, although focus/blur events are fully supported for this control.
In the repo there is now a new folder 'testfield-updateOnBlur' with which the problem with TextField can be reproduced.
Should I open a new issue for TextField?
For Switch / Slider, ... this feature may not be that important, but with TextField we are now forced to unnecessarily validate all typos, or to implement a workaround every time that bypasses the standard validation

added TextField to the title
however, it is possible that the TextField-issue already has a solution:
https://github.com/NativeScript/nativescript-angular/pull/1230

verified: #1230 solves the TextField-part of this issue!


ValueError: buffer size must be a multiple of element size

Hi, I tried to use Radar data only from Nuscenes to do some experiments.
I created my custom dataset.py according to provided nuscenes converter and dataset .py file to generate pkl files that contain radar info only.
The error occurred when create_groundtruth_database was called.
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/path/tools/data_converter/create_gt_database.py", line 263, in create_groundtruth_database
    example = dataset.pipeline(input_dict)
  File "/path/anaconda3/envs/SST/lib/python3.7/site-packages/mmdet/datasets/pipelines/compose.py", line 40, in __call__
    data = t(data)
  File "/path/mmdet3d/datasets/pipelines/loading.py", line 417, in __call__
    points = self._load_points(pts_filename)
  File "path/mmdet3d/datasets/pipelines/loading.py", line 393, in _load_points
    points = np.frombuffer(pts_bytes, dtype=np.float32)
ValueError: buffer size must be a multiple of element size.

my running env is
Python: 3.7.12 | packaged by conda-forge | (default, Oct 26 2021, 06:08:21) [GCC 9.4.0]
CUDA available: True
GPU 0,1: Quadro RTX 8000
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 10.2, V10.2.89
GCC: gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
PyTorch: 1.10.1
PyTorch compiling details: PyTorch built with:
  - GCC 7.3
  - C++ Version: 201402
  - Intel(R) Math Kernel Library Version 2020.0.4 Product Build 20200917 for Intel(R) 64 architecture applications
  - Intel(R) MKL-DNN v2.2.3 (Git Hash 7336ca9f055cf1bfa13efb658fe15dc9b41f0740)
  - OpenMP 201511 (a.k.a. OpenMP 4.5)
  - LAPACK is enabled (usually provided by MKL)
  - NNPACK is enabled
  - CPU capability usage: AVX512
  - CUDA Runtime 10.2
  - NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_37,code=compute_37
  - CuDNN 7.6.5
  - Magma 2.5.2
  - Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=10.2, CUDNN_VERSION=7.6.5, CXX_COMPILER=/opt/rh/devtoolset-7/root/usr/bin/c++, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -fopenmp -DNDEBUG -DUSE_KINETO -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -DEDGE_PROFILER_USE_KINETO -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-unused-local-typedefs -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_VERSION=1.10.1, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON,

TorchVision: 0.11.2
OpenCV: 4.5.5
MMCV: 1.4.0
MMCV Compiler: GCC 7.3
MMCV CUDA Compiler: 10.2
MMDetection: 2.14.0
MMSegmentation: 0.14.1
MMDetection3D: 0.15.0+068d40e

Hi, I found radar pts_file ends with pcd and lidar pts_file ends with pcd.bin. I guess only bin file is supported?

Yes, we do not support loading radar files currently. Maybe you need a more compatible implementation for reading pcd files.

thanks for the reply, I found a way to read radar pcd files.

@chyohoo Hi, I am also working on loading radar data from nuscences. Could you please help me with this?


Ignore builds on other branches

closes: #147
If there are two builds on different branches but with the same hash, GitHub reports these at least twice!  (Once per branch.)  This makes it so that we only consider build status changes on the actual relevant testing branch.  There's a detailed explanation here.

[x] parse the branches list sent by GitHub
[x] update send-webhook to include the list of branches
[x] ignore other branches on BuildStatusChanged
[x] add specific test with repeated build status update for a sha but different branches.  The status for the other branch should be ignore
[x] mark for review
[x] amend if needed
[ ] merge

@FPtje This is a fix for the multiple-comment issue you and Bert reported yesterday.  Can you please review when you have the time?
GitHub's documentation isn't super clear of what's the criteria for including the branch in the branches list: https://docs.github.com/en/developers/webhooks-and-events/webhooks/webhook-events-and-payloads#status.  Here's what it says:

"An array of branch objects containing the status' SHA. Each branch contains the given SHA, but the SHA may or may not be the head of the branch. The array includes a maximum of 10 branches."

If that's true in the most straightforward interpretation, there is no way to actually identify the "canonical" build for a sha in a branch!   In this PR I assume the list only includes the branches for which the build refers to (this would make more sense).  In the case of Semaphore, there can be multiple builds for the same commit sha if they are on different branches.
My plan is to see in practice what happens after we deploy this change:

if it only includes the relevant branch, the issue is fixed;
if it includes all branches containing the sha (even those that do not refer to the build, the issue will still be present, but Hoff will continue working "as-is".

In case of "2.", I will have to prepare a second PR where different URLs for the same SHA are simply ignored.  This will make Hoff "internally inconsistent" but at least we will not get duplicate comments.

I'm a little short on time. @frank-channable could you review? Thanks!

@frank-channable Thanks for the review.
@OpsBotPrime mergeee

Pull request approved for merge by @rudymatela, rebasing now.

Rebased as 88a0ec601b8d1008e08ff3a6ea50815daba67d31, waiting for CI …

CI job started.

@OpsBotPrime mergeee

@OpsBotPrime You do accept trailing alphabetic characters... Interesting...

CI job started.

@OpsBotPrime test.

That was not a valid command.

CI job started.

CI job started.

This makes sense.  The reply to the merge command when it is in progress is the current status: "CI job started".  :+1:

Message from the future: this merge has been reverted in faf04c96486cb9dc30b6f133f6574ceab3401955.


Laggy TextInput while typing

I think there's some issue with the component's text input field - while typing a query, I think because of the async autosuggestions, it eats a few letters from the query.
For example, searching for "iPhone X" turned into "iphoex", "Apple iPhone" into "Ape pow" etc.
I noticed this in your example code as well. Any help will be greatly appreciated!

+1 This only happens when async suggestions are enabled, otherwise it works just fine. More severe on Android, compared to iOS.

Hmm.. weird. Ok, I will take a look at the lag issue. I never notice it before while testing.

Ok found and fixed the issue. I moved the text input related state update outside of the async suggestion call. Shouldn't be in there in the first place. Thanks for spotting the bug. I will push out new version later today.

Works great on 0.2.9. Thanks for the prompt fix!


ENH: Add Index.fillna

Closes #10089.

value can only accept scalar.
MultiIndex.fillna raises NotImplementedError because isnull is not defined for MI.
Moved hasnans and related properties to base Index.

Thanks, did the suggested changes.
A test fails in Series.fillna which uses DatetimeIndex.putmask. Looks #11329 fixes the same part, I'm leaving it now.

Rebased, and now green.

Thanks, simplified the logic based on the comments.

Thanks, updated based on the comments.

_convert_for_op converts value for scalar op

assert_can_do_op check whether the value is acceptable for scalar op

timezone check is split to different method. Because it is used in list-likes such as equals.

Some changes related to output detailed error messages / keep current messages

@sinhrks looks really good!
just a minor comment / question

Reabsed, and now green.

lgtm
merge when ready
thanks!
might want to add s short example in the Index docs (iirc where we go over Index set ops)

Thanks. Added brief doc to indexing.rst. Merging within few days if there is no further comments.
CC: @jorisvandenbossche

@sinhrks Looks good! One more remark, doc-wise: I see there is a downcast keyword, but there is no example given of its usage in docstring or rst docs. That would be nice to include to clarify its usage I think.

@sinhrks thanks!
always gr8 PR's from you!

I don't see an entry in API.rst can you add?

@jreback Ah, let me add it with #11537.


Mutators harness

Mutator's harness and first mutators
Mutators to include/included:

[x] replace a function's body with unreachable
[x] replace a function's body with a default return value
[x] rename an export
[x] remove an export
[ ] swap the left and right operands of a commutative operation (using temporary variables)
[ ] turn a load offset=n into a an iconst n; iadd; load offset=0
[ ] undo strength reduction by turning x << 1 into x * 2
[ ] take the complement of an if/else's condition and swap the consequent and alternative blocks

Left a bunch of comments but I am going to merge this into the wasm-mutate branch now and we can address the comments in follow up PRs, as discussed.
Thanks @Jacarte!

Notes taken on the comments :). I will address them in the next iteration


Cloud Run Deploy Action

Wondering if Terraform could be suitable for deploy 🤔  https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/cloud_run_service

Wondering if Terraform could be suitable for deploy 🤔  https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/cloud_run_service

Solved using Google Cloud Build.

Solved using Google Cloud Build.


Avoid shared coroutines in Observable.fromCoroutine;

Currently, if a function is passed to Observable.fromCoroutine, a
coroutine is created and it is shared among all Observers.  Instead,
use a fresh coroutine for each Observer so all Observers receive the
same values.
When creating an Observable using an existing coroutine, this isn't
possible because the Observable is stateful (calling coroutine.resume
won't always yield the same values).

Fixes #15


Added sample filter tool.

Addresses issue #296.

@davidbenjamin please review.

@vruano comments are all trivial, will take single-digit minutes.

Ok thanks will fix and merge.


Added (simpler) Groovy sample and reviewed the Java client

2 streams of changes here:

on the Java client, the way the query is passed as a FileAttachment is way too convoluted and also incurs extra I/O due to going to disk when in fact everything can be done in memory.
Since I know there are some of us in Netflix using Groovy, and due to the fact that Groovy has a lot of syntactical sugar around initialization and properties setting, I've put together a Groovy client sample too which is much more concise I feel than the Java one and as such might make it easier to understand the client (as well as help folks who rely on Groovy).

NetflixOSS » genie » genie-pull-requests #7 SUCCESS
This pull request looks good

NetflixOSS » genie » genie-pull-requests #8 SUCCESS
This pull request looks good

Merged. Thanks @liviutudor!


Document the doc attribute

Currently we just refer to the book for the doc attribute, we should fully document it.

Can we close this as per the previous comment's statement?

Closing as per previous comment.


abstract methods are allways too long

public abstract ILoader LoadData(object argument);
-> Method too long

Fixed in 5.03, available in Extension Manager.


Allow to set type for optional values

Our use-case is as follows:
In the chart we have various optional values, e.g. defined with an empty value in the values.yaml:
# -- replica count
replicaCount:

In the Helm chart template:
  {{- if $values.replicaCount }}
  replicas: {{ $values.replicaCount }}
  {{- end }}

The requirement is to have the value listed in the values.yaml, so that we can document it with helm-docs, but we want the value to be optional and not set per default.
This now generates the following schema:
        "replicaCount": {
            "type": "null"
        }

This results in an Invalid type error.
Maybe it would be possible to specify the type as a comment on the property similar to the requirement in https://github.com/losisin/helm-values-schema-json/issues/21
Do you have any plans to implement something like this?
BTW: Nice plugin 👍

@anessi I just tried your use-case and it seems to work:
My values.yaml file:
replicaCount:

Command used:
helm schema -input values.yaml

Generated JSON schema:
{
    "$schema": "https://json-schema.org/draft/2020-12/schema",
    "type": "object",
    "properties": {
        "replicaCount": {
            "type": "null"
        }
    }
}

The error you are getting can happen when when you actually declare value to the property and then run helm commands: lint, template, install or upgrade.  To work around it you should regenerate the schema manually, with pre-commit hook or in CI/CD pipeline. Check the Integrations section from README.md https://github.com/losisin/helm-values-schema-json#integrations.
If I missed something can you please provide the whole output from the error?

@losisin : thanks for the feedback. If I understood correctly you would either remove the field in a post processing step or fix the type later. IMO it would be better to be able to use a different approach e.g. with annotation/types in comments like helm-docs does which I think is more sustainable.
helm-docs actually allows to set the type like documented in this section for nil values:
controller:
  # -- (int) Number of nginx-ingress pods to load balance between
  replicas:

  # controller.image -- (string) Number of nginx-ingress pods to load balance between
  image:

If your plugin could process these comments as well, then there would be a valid type in the json schema.
The example from this issue would look like this:
# -- (int) replica count
replicaCount:

Does this make sense?

In addition it would be nice to mark a fields as required like it is documented in the json schema specification. Per default all are optional which is fine for the start I guess 😄 .
BTW: I tested if it's possible to add additional annotations in helm-docs similar to default values and ignoring values but helm-docs will render this into the output comment.
Example:
# -- (int) replica count
# @default -- some default value
# @required
replicaCount:

is rendered as:
| replicaCount | int | some default value | replica count @required |

so we would need to find a way to ignore the additional annotations or check with the helm-docs maintainers how to change this. Of course there could also be other ways like adding a comment after the property:
# -- (int) replica count
# @default -- some default value
replicaCount: # @required

which works fine helm-docs currently (the full @required comment is ignored).
Anyway, something for a future issue I guess.

I'm already working on reading annotations from comments but can't say for sure when it will be done. So far, the plan is to use HeadComment similar to helm-docs which means they have to play nice together :) :
# @schema:"xxxx,xxxxx,xxx"
This plugin and what helm-docs does serve different purpose. I can and will add support to skip a property but if you declare it at some time will defy the purpose of the schema which is to check input types. It will be valid and helm would be happy to lint, template, install or upgrade. It's the same effect as adding completely new properties in your values file but never updating your schema.
In addition to pre-commit hook or github action, you can also generate the schema from multiple values files. This plugin only cares about value type not the value itself. For example:
values.yaml:
replicaCount:

production.yaml:
replicaCount: 2

command:
helm schema -input values.yaml,production.yaml

schema will be:
{
    "$schema": "https://json-schema.org/draft/2020-12/schema",
    "type": "object",
    "properties": {
        "replicaCount": {
            "type": "integer"
        }
    }
}

This is specific to replicaCount  because I had similar request and here is what I used:
values.yaml:
replicaCount: 2
autoscaling:
  enabled: false

production.yaml:
autoscaling:
  enabled: true
  minReplicas: 2
  maxReplicas: 10
  targetCPUUtilizationPercentage: 80

deployment.yaml:
  {{- if not .Values.autoscaling.enabled }}
  replicas: {{ .Values.replicaCount }}
  {{- end }}

my schema:
{
    "$schema": "https://json-schema.org/draft/2020-12/schema",
    "type": "object",
    "properties": {
        "autoscaling": {
            "type": "object",
            "properties": {
                "enabled": {
                    "type": "boolean"
                },
                "maxReplicas": {
                    "type": "integer"
                },
                "minReplicas": {
                    "type": "integer"
                },
                "targetCPUUtilizationPercentage": {
                    "type": "integer"
                }
            }
        },
        "replicaCount": {
            "type": "integer"
        }
    }
}

Please keep in mind if you have null as value in some defaults, but override it in real usage with integer in environment specific value file or as command line argument like --set  is exactly what json schema is trying to prevent.

So far, the plan is to use HeadComment similar to helm-docs which means they have to play nice together

I fully agree that this is a very important point :)
IMO generating the schema file is something that should be done when developing the helm chart. So the use-case you have mentioned (e.g. helm schema -input values.yaml,production.yaml) does not apply for us at least. Because we would not re-generate the schema with real values, but only with the default values file.
=> The schema delivered with the helm chart needs to be valid for all possible combinations of values.
Going back to the described use-case we would expect the following schema for replicaCount:
        "replicaCount": {
            "type": ["null", "integer"]
        },

Regarding the enabled approach: we use this very often as well and it makes totally sense, especially if you want to enable/disable whole sections or have other logic depending on some enabled/disabled flag. However, for a single property the null-value approach is simpler as you don't need to have an additional enabled property for each property you want to enable/disable.

Please keep in mind if you have null as value in some defaults, but override it in real usage with integer in environment specific value file or as command line argument like --set is exactly what json schema is trying to prevent.

I don't think I understand this. From my point of view I want to specify all valid properties and types in the schema. It's as simple as that. For the example above this includes null and integer. Our main use-case is to help the person that writes or updates the values file and allow him to easily detect errors in his IDE. We can't enforce the result in the end as he can use various other tools (e.g. like kustomize) to tweak the generated result. The final result is enforce by the API server in the end.
Please also see the approach described here: https://argo-cd.readthedocs.io/en/stable/user-guide/best_practices/#leaving-room-for-imperativeness

@anessi I agree! New version supporting what we've discussed here is available in main branch: https://github.com/losisin/helm-values-schema-json/tree/main/docs#multiple-types
Thanks for the feedback!


Check ast

closes #438

Codecov Report

Merging #440 (77c42e1) into master (1e59e13) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #440   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           12        12
  Lines          638       638
=========================================
  Hits           638       638

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 1e59e13...77c42e1. Read the comment docs.

Maybe add it to the changelog e.g.:
pre-commit-hook: New hook nbqa-check-ast

sure - I'll write the changelog when I make the next release


Update Node to 4.3.2 and 5.7.1

Thank you @MariusRumpf - I have to make you a collaborator next time to keep up the speed :-)

No problem, glad to help :-)


Create Azure Batch Pool Without Public IP in EastUS2 and CentralUS

I have two questions,

Even though the document didn't mention eastus2 and cetnralus, but I'm able to create pool without pip in these two regions
Can you please confirm the feature is available?

The document still says pool without pip is preview feature, can we confirm if the document is accurate and when the feature will be GA?

Thanks!
[Enter feedback here]

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

ID: 6642341a-eaaa-1d80-550b-0594bb91072e
Version Independent ID: ea38ea4e-08c6-a747-51af-d49bdffcb7fe
Content: Create an Azure Batch pool without public IP addresses - Azure Batch
Content Source: articles/batch/batch-pool-no-public-ip-address.md
Service: batch
GitHub Login: @pkshultz
Microsoft Alias: peshultz

Thanks for the feedback! We are currently investigating and will update you shortly.

Hi @RonnieQuan - thank you for the question. We've confirmed that support for pools without public IP addresses in Azure Batch is still in public preview, and this applies to all regions except China East, China East 2, China North, and China North 2. The document will be updated later today and I will close this issue after that happens, but wanted to let you know now. Thanks again for submitting your feedback/question!

Forgot to close this out but the document has been updated with the current region support.
#please-close


[RUNTIME][OPENCL] Make OpenCL runtime Compatible with OpenCL2.0

Build opencl fails with:
tvm/src/runtime/opencl/opencl_device_api.cc:278:9: error: ‘_cl_command_queue* clCreateCommandQueue(cl_context, cl_device_id, cl_command_queue_properties, cl_int*)’ is deprecated [-Werror=deprecated-declarations]
The clCreateCommandQueue API is deprecated in OpenCL2.0.
Possible solutions:

Switch to clCreateCommandQueueWithProperties

Define  either we need to:
#define CL_USE_DEPRECATED_OPENCL_1_2_APIS  prior to the inclusion of cl.h

cc @kazum , likely we can detect the OpenCL version in macro and use the new API(it is not yet available in old version of the API).

I think we cannot detect the OpenCL version statically - we need to query it with the OpenCL APIs.  Maybe, it's possible to detect the version in cmake and define an appropriate macro.
Anyway, there are still many OpenCL platforms which doesn't support OpenCL 2.0.  Defining CL_USE_DEPRECATED_OPENCL_1_2_APIS looks good for now.

By search around i have found __OPENCL_VERSION__ , not sure if it is part of standnard

Is __OPENCL_VERSION__ only available in OpenCL kernel code?  https://www.khronos.org/registry/OpenCL/sdk/2.1/docs/man/xhtml/preprocessorDirectives.html
I searched the macro in the official OpenCL headers (https://github.com/KhronosGroup/OpenCL-Headers), but couldn't find it.  Even if it is defined there, the supported OpenCL version is up to the target device, IIUC.

OK, perhaps we can define the backward compatible flag for now then

Resolved by #2950


Cluster-autoscaler: bump version to 0.5.0-alpha1 in preparation for the next release

cc: @fgrzadkowski @jszczepkowski @MaciekPytel

/lgtm

This change is


Tile Rotation

Adds quarter rotations for tiles to TileMaps by composing flips with new transpose transform.
Adds extra tool button for transpose as well as dedicated buttons for each quarter rotation.
As this touches a lot of files, I ask others do some testing before it's merged to make sure I haven't broken anything.
(Duplicate of #1330. Just moving this out of my master branch.)

Hey surt, here's a feature many people asked for the tilemap, maybe it would not be that difficult for you to add.
Basically it's the ability to do isometric tilemaps that Y-Sort themselves and children nodes. It's probably not very efficient but it should not be that difficult to do either.  The option would do this:
1- Force octant size to 1 (so each tile is it's own canvasitem)
2- Make sure the canvasitem (octant) origin is near the lower corner of the tile when drawn (isometric must always be sorted by the lowest Y point)
3- Enable Y-Sorting on the TileMap node itself, this is done with a function call to VisualServer (check the YSort node as reference).
With that, you could make 3D isometric rather easily, as each "layer" or "floor" can be made with a tilemap, and if you want to move a children node between layers, just move it between nodes as child.

The advantage of this method is that all children of the tilemap will be sorted, so you can place your own scenes as children of it and they will be sorted together with the tiles transparently. With the imminent arrival of the new 2D Light/Shadow system i'm working on (that I hope to finish this weekend), isometric games will look awesome! :)


Different default icons for each control

It'd be quite nice to have different default icons for each control, as this makes them easier to tell apart in entity lists, and when having a set of dashboard controls together. I suggest these:

Keep mdi:theme-light-dark for master control
mdi:brightness-4 for brightness
mdi:sun-thermometer for colour
mdi:sleep for sleep mode

Looks good!


adds type definitions to index.js

This pull request adds type definitions to main file of the package allowing for seamless usage with typescript.

iconSrc can be false as well (returns only the icon string in that case). Gonna approve and merge after that change, thank you!


[FEATURE] Extensible Auto-calibration strategies

Description
Added an autocalibration strategy extra that uses a list of commonly blacklisted files to detect & filter blacklisted responses
Fixes: #689

Hi, sorry for taking some time to get back on this. There are two linter issues (unused struct members), but other than that it looks good to merge!


Option to disable color output

Output log looks terrible on heroku.
Yes, you can install heroku-cli for colorful logs but I don't want to see colorful logs, I just want to see some readable text.

We currently have a PR in to fix it

I suggest just checking the color support:
https://github.com/composer/xdebug-handler/blob/c602b6e6d08d23c7d89be3b3d203045289714bf7/src/Process.php#L101
This also fixes the problem with cmd.

@xPaw without checking of ANSICON or ConEmuANSI, the colors will not work in Cmder on Windows 7-8.


Need YAML support for test cases that run longer than 90 seconds

Summary:
As per test plan of TC-OO-2.3 needs wait time more than 90 sec we are facing timeout error when we execute, need YAML framework support for test cases that run longer than 90 seconds
Execution screenshot:

You can normally add a timeout parameter in the config section.
For example:
config:
   cluster: "Test Cluster"
   endpoint: 1
   timeout: 120 # or any number. It also generates a command line optional argument in case you need to customise that on the fly


Configure the PaaS  authentication services

Configure the PaaS level resources, e.g., authentication services, etc.
This a tosca template to send to SDIA

Is integrated with keycloack


hdel on non existent key returns error

Here's redis:
<IP_ADDRESS>:6379> hdel foo bar
(integer) 0

Here's ardb:
<IP_ADDRESS>:16379> hdel foo bar
(error) WRONGTYPE Operation against a key holding the wrong kind of value.

In both cases, the key didn't exist in the database. Shouldn't ARDB return 0 in that case rather than error?

it's fixed by latest commit.

thx. looks good.


Make MariaDBContainer.MARIADB_PORT constant public

Similar to other JDBC containers (e.g. MySQL) it would be helpful if the MARIADB_PORT constant would be public instead of protected.
https://github.com/testcontainers/testcontainers-java/blob/dabc99c73eb9832ed2971c42412dd56bbd4d15be/modules/mariadb/src/main/java/org/testcontainers/containers/MariaDBContainer.java#L26

Hi,
In the future those internal properties will be package-private or private. Feel free to reopen the issue and elaborate more on your need.


Dashboard cleanup

Currently we are missing a lot of escrow factories that we previously ran. Let's track down  All of the Escrow Factories that Intuition Machines has ever run (ethereum ropsten/polygon dev net) and make sure they are on the dashboard

This is covered in https://github.com/humanprotocol/human-protocol/issues/54


No more need for byte masking (val & 0xff...) ???

Because Groovy uses autoboxing behind the scenes for all of its primitive types, we don't fall into the trap in vanilla Java where going from a small primitive to a bigger one fills in the bits w/ 1s, eg:
byte b = 127; // 0xff
int i = b; // Doesn't stay as 127, but instead 0xffffffff

But in Groovy:
byte b = 127; // 0xff
int i = b; // Stays as 127, becomes 0x000000ff

This should make all the bit/byte operations that I had to mask in Java unnecessary! 🎉   Man Groovy is the best 😁

OK, I've found there are some tricks/caveats with this.  If I'm still dealing with hard primitives, like those coming out of other Java code and not my own Groovy code, then it's still subject to the usual Java behaviour.  So I can't just go removing all the bit masks and call it a day - gotta either test or make sure if doesn't break anything.

Done in all the places I thought it safe!


Fix 404 on /releases/feed

Handlebars removed the obj[0] notation in 0.25: https://github.com/sunng87/handlebars-rust/commit/779df9868ebb713300bab3744137a5cb58c529a3#diff-e0934bb6f70a4f8c3baf22a773c14ec7L35
Closes https://github.com/rust-lang/docs.rs/issues/782
r? @Kixiron
This also uncovered that we handle failure pretty badly in impl Handler for CratesfyiHandler, we always return the first error encountered even if it was a 404, ignoring all future errors. This should be fixed, but I want to get this fix in first.

Whoops, let me remove the dbg


[YUNIKORN-2908] unregister its metrics when a queue is removed

What is this PR for?
Delete all metrics so a deleted queue will not continue to report 'guaranteed resource' 'max resource'
What type of PR is it?

[x] - Bug Fix
[ ] - Improvement
[ ] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-2908
How should this be tested?
Screenshots (if appropriate)
Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

@hguo25 is this chang ready for review? I can only see two "wip" commits. If it's not ready, please make it draft by clicking on "Convert to Draft".

Also, when ready, squash the commits into a single one and change the commit message to "[YUNIKORN-nnn] JIRA title" format. Thanks.

@pbacsko yes it's ready. Squashed commits as you asked. Please review, thanks

thanks @pbacsko @craigcondit I have added test

@pbacsko thanks for the review. The e2e failed in gang scheduling, which doesn't seem to be related to my change?

We need a follow up on YUNIKORN-2926 to fix the gang scheduling e2e test. The changes I made there are not enough to fix the whole issue. Logged new jira for that.


dimension should be via join unit

currently dimensions result in a KeyError as the join dimension has a different key than the provider dimension

Looks good. Thanks for the fix.


merge package template from Kronos-Tools/npm-package-template

package.json

chore(devDependencies): update<EMAIL_ADDRESS>from template
chore(devDependencies): update<EMAIL_ADDRESS>from template

Coverage remained the same at 95.122% when pulling 6b70e35f19bb4c4080c8552267b236727836a1d6 on template-sync-1 into 1ad6b262d1bf6e54c30a0f9fd89473a41f69711b on master.

Codecov Report

Merging #196 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #196   +/-   ##
=======================================
  Coverage   98.05%   98.05%
=======================================
  Files           1        1
  Lines         103      103
=======================================
  Hits          101      101
  Misses          2        2

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 1ad6b26...6b70e35. Read the comment docs.


Add F.as_strided

This PR is for #5036 and #5796.
This feature (F.as_strided) strides array like numpy.lib.stride_tricks.as_strided, but this function is itemsize-agnostic (i.e. stride size is given in unit of steps, not bytes).
This is a transported code from pytorch. The algorithm and API is the same.
This feature will be helpful in the implementation of F.stft.
If this feature would be accepted, I will create a PR of F.stft.

How do I fix the travis's error "No module named cupy"?


Difference between container and breakpoints.

Could you please explain the difference between container and breakpoints in the configuration?

container are defined witdh for <Grid /> and breakpoints are used for media queries generation, used by <Row />


Issue building the library with Gatsby - "window" is not available during server side rendering.

Hi :)
First of all, very nice job! The library is amazing.
I am trying to build it with gatsby, which uses server-side rendering and I've found the following error:
failed Building static HTML for pages - 17.921s

 ERROR #95312

"window" is not available during server side rendering.

See our docs page for more info on this error: https://gatsby.dev/debug-html

   8 |          for(var i in a) (typeof exports === 'object' ? exports : root)[i] = a[i];
   9 |  }
> 10 | })(window, function() {
     |  ^
  11 | return /******/ (function(modules) { // webpackBootstrap
  12 | /******/         // The module cache
  13 | /******/         var installedModules = {};

  WebpackError: ReferenceError: window is not defined

  - browser.js:10
    node_modules/swissqrbill/lib/browser.js:10:2

This "window" variable is nested within the *js bundle. Any thoughts on this?

Hi
Thank you for your feedback. I am not familiar with gatsby, but shouldn't gatsby include the file node.js instead of browser.js if it uses server-side rendering?
The code which uses window is generated by webpack and only browser.js gets bundled by webpack. You may need to import swissqrbill/lib/node directly.

Hi! Thanks for your answer :)
I found a workaround - importing the library using async dynamic imports. This worked for me 👍
  useEffect(() => {
    async function preparePDF() {
      const instance = await import("swissqrbill/lib/browser");
      const SwissQRBill = instance.default;
      if (isLoadingPDF) {
        const data: any = {
          currency: "CHF",
          amount: Number(amount.join("")),
          additionalInformation: reason.toString(),
          creditor: {
            name: "name",
            address: "address",
            zip: 8952,
            city: "City",
            account: "CHXX 0000 0000 0000 0000 0",
            country: "CH"
          },
          debtor: {
            name: address.name,
            address: address.address,
            zip: Number(address.zip),
            city: address.city,
            country: "CH"
          }
        };

        try {
          if (validateFields()) {
            const stream = SwissQRBill.blobStream();
            const pdf = new SwissQRBill.PDF(data, stream);
            pdf.on("finish", () => {
              setBlobLink(stream.toBlobURL("application/pdf"));
            });
          }
        } catch (e) {
          alert(e.message);
        }
      }
    }
    preparePDF();
  }, [
    address.address,
    address.city,
    address.name,
    address.zip,
    amount,
    dispatch,
    isLoadingPDF,
    reason,
    validateFields
  ]);


FAB's icon is not centered

I want to replace support FABs with cFABs, but icon is not centered :(

What is the parent layout?. You don't need to set a size. The Floating action button has two sizes. Normal (56dp) and small (48dp). The size you are setting will be averriten to default sizes. To have custom size use floating action layout instead

@robertlevonyan I know I don't need. But using wrap_content cause icon issues like on the screenshot.

have you added a text?

please check the latest version, I made some changes (2.0.5)

This issue still exists in 2.0.5.

Here's the layout:
<android.support.constraint.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:id="@+id/dashboard_layout"
    android:layout_width="match_parent"
    android:layout_height="match_parent">

    <fragment
        android:id="@+id/map"
        class="com.google.android.gms.maps.SupportMapFragment"
        android:layout_width="0dp"
        android:layout_height="0dp"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintHorizontal_bias="1.0"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent"
        app:layout_constraintVertical_bias="1.0" />

    <com.robertlevonyan.views.customfloatingactionbutton.FloatingActionButton
        android:id="@+id/location_fab"
        android:layout_width="56dp"
        android:layout_height="56dp"
        android:layout_marginBottom="8dp"
        android:layout_marginEnd="8dp"
        android:layout_marginStart="8dp"
        app:fabColor="?attr/colorBackgroundFloating"
        app:fabIcon="@drawable/ic_location_fab"
        app:fabIconColor="@android:color/holo_blue_light"
        app:fabIconPosition="start"
        app:fabSizes="mini"
        app:fabType="roundedSquare"
        app:layout_constraintBottom_toTopOf="@+id/traffic_fab"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent" />

    <com.robertlevonyan.views.customfloatingactionbutton.FloatingActionButton
        android:id="@+id/traffic_fab"
        android:layout_width="56dp"
        android:layout_height="56dp"
        android:layout_marginBottom="8dp"
        android:layout_marginEnd="8dp"
        android:layout_marginStart="8dp"
        app:fabColor="?attr/colorBackgroundFloating"
        app:fabIcon="@drawable/ic_traffic_fab"
        app:fabIconColor="@android:color/black"
        app:fabSizes="mini"
        app:fabType="roundedSquare"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent" />

</android.support.constraint.ConstraintLayout>

Is this something wrong with my layout?

@robertlevonyan any news on this? I've been looking at the code but I haven't been able to find anything that could be causing it. Do you have a culprit in mind?

Nope. But I think this issue may be caused because of Constraint layout. I've seen previously some strange behavior of other views inside it. Can you try using another parent for the CFABs?

No go. I tried relative, linear, grid and nothing worked, I even tried in a whole new project and still the same problem. Can you provide the code for the preview layouts in the readme? It works fine in the sample project but for some reason when I include it in any other project it gives me this weird result.

I encountered the same issue:
the icon is off center with wrap_content, I specify the size to 56dp as ash0x0 mentioned, then the icon is centered OK.

guys check the latest version 2.1.0


Single Header

Would be nice to get this library as a single header with no lib dependency.

In the next major version (v3), there will be no compiled dependencies, as the reliance on simdutf is removed. FastFloat (which is header-only) is still needed, and will be for the foreseeable future, as I don't intend to intend to implement that myself, and there's no way I could do it any better.
As for the single-header question, v3 will also consist of less files, but there will still be a clear split between the public interface and the private implementation. I don't intend to add support for a header-only mode in the short-term future, and it's not really a practice I would want to encourage.  The public interface will only be a single header file, though.


Migrating identities for projects with small numbers of identities causes a 504

e.g. I just migrated a project with only 10 identities and got a 504 error when the request timed out after 30s.

Screen recording: https://photos.app.goo.gl/CrejL5VPYLuN2Bzs9

Going to move to ecs task processor


Community 14.04.21

🍰 Pullrequest
Let mysql fill received value for transactions instead of php
received is part of gradido protocol and with blockchain-type mysql
mysql should decide this things (sequence-number and receiving date)
correct auto-increment basis for id in transactions
With blockchain type = db the mysql table transactions act mainly as ordering service,
the sequence number will be created by mysql auto-increment id.
But if an error occure in code and mysql rollback was called, the next call to transactions insert deliver
a higher id so a gap is created in blockchain.
To prevent this, I added code which set auto-increment to the count of actually transaction entrys in db after rollback.

#8a6acd67f7236e767a7a90ea6bbd497d6967bd6d
Fixtures a used to automatic filling test db
Now I have added a function (inside a new base class for all fixtures) which translate mysql export format into fixture records format.
So I can easly export test data from my db, change the ( to [ and ) to ] call the function and finish :)

#4690192a71f8b720d463fc7c29b6dd84d8145add
config/bootstrap.php:
check if $_SERVER['REQUEST_METHOD'] isset because in tests this is not set.
I have added
if ($_SERVER['REQUEST_METHOD'] == 'OPTIONS') {
    exit(0);
}

because it was needed for frontend working. Maybe it wasn't the right location, you can move/update it if you find a better solution.
config/routes.php:
Whitelist empty clientIP for ajax calls because if test framework is called, clientIP is empty.

add my test data set to Fixtures for updateBalance test in next step

commited to many files, sorry

"Furthermore we need a way to seed data. But I will make an issue for that."
For cakephp this is done via the FixtureClasses in tests/Fixture folder.


Better documentation on how to start a new branch off another branch

This could be down to me not being able to find the this in the manual, but here is my problem...
By experiment I have found out that I can do:
gl branch -c new-branch -dp origin/some-branch
gl switch new-branch
gl branch -su origin/some-branch

And then we are in business!
If this is already in the documentation please direct me to it.
If not, please tell me how I can submit PRs on the gitless.com documentation.
My first attempt at fixing a branch went through git show-ref branch-name to get the commit hasd and then do a gl branch -sh commit. Not a pleasant experience...

I'm not sure that -dp origin/some-branch is correct.
The documentation is available at gitless.com.
What do you want to do ? If you simply want to start a branch rooted at HEAD of the current branch, with remote tracking on origin, all you need to do is
gl branch -c new-branch
gl switch new-branch
gl branch -su origin/some-branch

The later fails because the remote branch doesn't exist.
The weird thing is that if I try to switch back to my previous branch I get an error
gl branch -su origin/test
✘ Branch "test" doesn't exist in remote "origin"
X meessen@marc ~/Sources/go/src/FileSync/core [test * >3]
12:40 $ gl switch refactor-fileToSend
Traceback (most recent call last):
  File "gl.py", line 14, in <module>
  File "gitless/cli/gl.py", line 87, in main
  File "gitless/cli/pprint.py", line 80, in err
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 97: ordinal not in range(128)
Failed to execute script gl

If I try your first instruction I also get an error
gl branch -c test2 -dp origin/test2
✘ Some internal error occurred
  ➜ If you want to help, see http://gitless.com for info on how to report bugs and include the following information:

  0.8.4

  Traceback (most recent call last):
    File "gitless/cli/gl.py", line 73, in main
    File "gitless/cli/gl_branch.py", line 60, in main
    File "gitless/cli/gl_branch.py", line 105, in _do_create
    File "gitless/core.py", line 118, in revparse_single
  AttributeError: 'NoneType' object has no attribute 'head'

I don't think that Python is the right language to program gitless.

The right way to do what you wanted is as following
$ gl branch -c test3 origin/test3
$ gl switch test3
$ gl branch -su origin/test3

You can create more than one branch with gl branch -c. So you create the local branch and the remote branch on origin. They may not have the same branch name.

I was probably not clear enough in what I wanted to do.
I want to create a new branch off the tip of an existing branch on one of my remotes.
I know that the example I provided above works for suitable existing origin/some-branch.
My problem is that the documentation on gitless.com does not provide an example that shows that the divergent-point can be an existing remote branch.
For me this is a very common use case and I think it should be documented. I cannot see anything on divergent-point that explains that a branch name can be used as DP. Hence my offer to help out with the documentation.

8b5384e90a7b641ff1a5153da13ba895e0669f35,<PHONE_NUMBER>48bac81fe138ae014eb4ea84543e91


The reproduction of NSFF in the hypernerf_vrig dataset

Hello author, thank you very much for such a good job. I have a request to ask you. One of my papers was reworked, and a reviewer asked me to add the results on hypernerf_vrig, but this was a bit difficult for me. So, I was wondering if you could send me your repro code on NSFF. Thank you very much.

Thank you for your attention. We have not been able to reproduce the results of NSFF. The results of NSFF in our paper on hypernerf_vrig are all based on the HyperNeRF paper.

你好，作者，非常感谢你如此出色的工作。我有一个请求要问你。我的一篇论文被重新修改了，一位审稿人要求我在 hypernerf_vrig 上添加结果，但这对我来说有点困难。所以，我想知道你是否可以把你在 NSFF 上的重现代码发给我。非常感谢。
Did you solve it? The data loading for Hyper is really hard to modify.


Selection of parameter in Rllib training for SAC agent in MetaDriveEnv and SafeMetaDriveEnv

What is the proper buffer size / batch size / entropy coefficient for SAC to reproduce the results? I find it hard to reproduce results in SafeMetaDriveEnv.
In https://arxiv.org/pdf/2109.12674.pdf, does the reported success rate of SAÇ in Table1 refer to the training success rate (and no collision, i.e. safe_rl_env=True)?

Hi Henry,
The hyper-parameter could be found in the appendix.
And the results reported in Table 1 is test success rate, which is the average success rate on test maps. Note that in safe exploration experiment the number of maps in the training set and test set is 100 and 50 respectively.
If you feel it is difficult to reproduce the results with your own algorithms, you can actually find our training scripts (based on ray/rllib) in https://github.com/decisionforce/EGPO or https://github.com/decisionforce/HACO, where SAC/PPO/SAC-Lag/PPO-Lag are trained on SafeMetaDriveEnv.
Quanyi

According to the MetaDrive white paper, the parameters for SAC is:

As you can see in SAC baseline of EGPO paper: Code most parameters are following RLLib default. Combining default in RLLib and our specification in our code, the final config of SAC is:
train_batch_size = 256,
learning_starts = 10000,
optimization=dict(actor_learning_rate=1e-4, critic_learning_rate=1e-4, entropy_learning_rate=1e-4),
buffer_size=1M

Note that the entropy coefficient is update automatically by RLLib and it is not a pre-defined value.

The test success rate is recorded in the test environment, which is defined by test environment config as in here
evaluation_config = dict(env_config=dict(
    vehicle_config=dict(use_saver=False, overtake_stat=False),
    safe_rl_env=True,
    start_seed=500,
    environment_num=50,
    horizon=1000,
))

If possible, could you please share the learning curves and your learning code / config? So that we can help to diagnose what's the problem. Thanks!

Thanks Zhenghao and Quanyi for your timely reply, I follow most of the config of SAC in EGPO and rllib default one. I think the previous issue is mainly due to I put safe_rl_env=False during evaluation, which takes collision into consideration in testing time. I retrain the parameter today, think now the result of SAC seems to be quite consistent with the paper.
Also for SAC_Lag, is the penalty learning rate actually the k_p for the update on Lagragian multiplier \lambda? I notice the SAC_Lag baseline has different settings of PID controller in HACO and EPGO.

Hi @HenryLHH,
In our practice, we find that using sole integral control to update /lambda makes the training unstable. So our SAC-Lag integrates a PID controller to update the Lagrangian multiplier, so the k_p,  k_i, k_d are 3 hyperparameters to tune the PID controller.
Since SAC-Lag is not very hyper-parameter-sensitive, the slight hyperparameter difference between HACO version and EGPO version is acceptable.
Quanyi
Quanyi


Rich Preview for Link is not working for all websites

Issue Summary
Hi Team,
I just noticed that the rich preview link (copy paste link and it changes to beautiful container / box) is not working for all websites.
Appreciate if you can help into following.
Thanks!
Steps to Reproduce
** Steps to Reproduce **

Go to ghost.org
Login to website
Open Post
Try to copy paste following: https://www.axs.com/uk

Expected Result:
Rich Preview link container
Actual Result:
Just link was pasted
Other Information
Test Browsers: Safari on iPad, Safari on macOS and Chrome Canary 127 on macOS
Refererence Screenshot
It looks like following:

Ghost Version
Online ghost.io free account
Node.js Version
Online ghost.io free account
How did you install Ghost?
Not Applicable
Database type
Other
Browser & OS version
Safari 17.5 on macOS, Safari 17.5 on iPadOS and Chrome Canary 127 on macOS
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

I believe https://www.axs.com/ is blocking requests that aren't coming from a browser. All requests from a node process at least are getting 403 responses. If you'v got other sites that you think are problematic, please open a new issue and we can look in to those.


Arm64 Images support

Please do not submit issues here. Best place for self-hosting issues, questions and ideas is the community forum here on GitHub. Thanks!
Support for Arm64 architecture

It's already supported :)

If you are referring to the mail service, then it's going to be removed in v2.1.0.