Siamese

I am trying to run the Siamese code on MNIST data.
The link of the code is : https://github.com/keras-team/keras/blob/master/examples/mnist_siamese.py
In the code, I used my own training file and splitting the data into train and test as follows:
x_train, x_test, y_train, y_test = train_test_split(desc1, label1, test_size=0.5, random_state=1)
i.e. 50% into training and testing. When I am increasing the test_size from 0.5 to 1, the code is working. But if I reduced this value below 0.5, I am getting this error:
validation_data=([te_pairs[:, 0], te_pairs[:, 1]], te_y))
IndexError: too many indices for array
Please help.

Can you please provide the whole stack traceback along with source code, if possible? It'll be easier to debug.

'''Trains a Siamese MLP on pairs of digits from the MNIST dataset.
It follows Hadsell-et-al.'06 [1] by computing the Euclidean distance on the
output of the shared network and by optimizing the contrastive loss (see paper
for more details).
References

Dimensionality Reduction by Learning an Invariant Mapping
http://yann.lecun.com/exdb/publis/pdf/hadsell-chopra-lecun-06.pdf
Gets to 97.2% test accuracy after 20 epochs.
2 seconds per epoch on a Titan X Maxwell GPU
'''
from future import absolute_import
from future import print_function
import numpy as np

import random
import sys
from keras.datasets import mnist
from keras.models import Model
from keras.layers import Input, Flatten, Dense, Dropout, Lambda
from keras.optimizers import RMSprop
from keras import backend as K
from sklearn.model_selection import train_test_split
from sklearn.preprocessing import LabelEncoder
import pandas as pd
np.set_printoptions(threshold=sys.maxsize)
num_classes = 2
epochs = 1
def euclidean_distance(vects):
x, y = vects
sum_square = K.sum(K.square(x - y), axis=1, keepdims=True)
return K.sqrt(K.maximum(sum_square, K.epsilon()))
def eucl_dist_output_shape(shapes):
shape1, shape2 = shapes
return (shape1[0], 1)
def contrastive_loss(y_true, y_pred):
'''Contrastive loss from Hadsell-et-al.'06
http://yann.lecun.com/exdb/publis/pdf/hadsell-chopra-lecun-06.pdf
'''
margin = 1
square_pred = K.square(y_pred)
margin_square = K.square(K.maximum(margin - y_pred, 0))
return K.mean(y_true * square_pred + (1 - y_true) * margin_square)
def create_pairs(x, digit_indices):
'''Positive and negative pair creation.
Alternates between positive and negative pairs.
'''
pairs = []
labels = []
n = min([len(digit_indices[d]) for d in range(num_classes)]) - 1
print("*****n:",n)
for d in range(num_classes):
for i in range(n):
z1, z2 = digit_indices[d][i], digit_indices[d][i + 1]
pairs += [[x[z1], x[z2]]]
inc = random.randrange(1, num_classes)
dn = (d + inc) % num_classes
z1, z2 = digit_indices[d][i], digit_indices[dn][i]
pairs += [[x[z1], x[z2]]]
labels += [1, 0]
return np.array(pairs), np.array(labels)
def create_base_network(input_shape):
'''Base network to be shared (eq. to feature extraction).
'''
input = Input(shape=input_shape)
x = Flatten()(input)
x = Dense(128, activation='relu')(x)
x = Dropout(0.1)(x)
x = Dense(128, activation='relu')(x)
x = Dropout(0.1)(x)
x = Dense(128, activation='relu')(x)
return Model(input, x)
def compute_accuracy(y_true, y_pred):
'''Compute classification accuracy with a fixed threshold on distances.
'''
pred = y_pred.ravel() < 0.5
return np.mean(pred == y_true)
def accuracy(y_true, y_pred):
'''Compute classification accuracy with a fixed threshold on distances.
'''
return K.mean(K.equal(y_true, K.cast(y_pred < 0.5, y_true.dtype)))
the data, split between train and test sets
#***********************************************************
#data loading
df1=pd.read_csv("class7h.csv") #loads input file from classcsvfiles_headers folder
df1_desc = df1.loc[:, df1.columns != 'label']  # load descriptor as column 1 of input file contains label and rest 2025 columns contain feature vector
df1_label = df1["label"]  # load label
desc1=np.array(df1_desc)  # create descriptor array
label1=np.array(df1_label)  # create label array
#train test split making
x_train, x_test, y_train, y_test = train_test_split(desc1, label1, test_size=0.5, random_state=1) # split into training and test data, 80% training and 20% test
print("******************x_train_shape:", x_train.shape)
x_train = x_train.reshape(x_train.shape[0], 45, 45).astype('float32') # reshape to 45x45 array to form an image for training
x_test = x_test.reshape(x_test.shape[0], 45, 45).astype('float32')   # reshape to 45x45 array to form an image for tesing
x_train = x_train / 255
x_test = x_test / 255
input_shape = x_train.shape[1:]
print("******************x_train_shape:", x_train.shape)
#print("******************x_train:", x_train[0])
print("**************y_train_shape:", y_train.shape)
print("****************y_train:",y_train)
print("******************x_test_shape:", x_test.shape)
#print("******************x_train:", x_train[0])
print("**************y_test_shape:", y_test.shape)
print("****************y_test",y_test)
c=[]
for i in range(num_classes):
c=np.where(y_train == i)[0]
print(c)
create training+test positive and negative pairs
digit_indices = [np.where(y_train == i)[0] for i in range(num_classes)]
tr_pairs, tr_y = create_pairs(x_train, digit_indices)
#print("************tr_pairs", tr_pairs)
print("************tr_pairs shape", tr_pairs.shape)
print("*****************tr_y shape", tr_y.shape)
c=[]
for i in range(num_classes):
c=np.where(y_test== i)[0]
print(c)
digit_indices = [np.where(y_test == i)[0] for i in range(num_classes)]
te_pairs, te_y = create_pairs(x_test, digit_indices)
print("************te_pairs shape", te_pairs.shape)
print("*****************te_y shape", te_y.shape)
network definition
base_network = create_base_network(input_shape)
input_a = Input(shape=input_shape)
input_b = Input(shape=input_shape)
because we re-use the same instance base_network,
the weights of the network
will be shared across the two branches
processed_a = base_network(input_a)
processed_b = base_network(input_b)
distance = Lambda(euclidean_distance,
output_shape=eucl_dist_output_shape)([processed_a, processed_b])
model = Model([input_a, input_b], distance)
train
rms = RMSprop()
model.compile(loss=contrastive_loss, optimizer=rms, metrics=[accuracy])
model.fit([tr_pairs[:, 0], tr_pairs[:, 1]], tr_y,
batch_size=128,
epochs=epochs,
validation_data=([te_pairs[:, 0], te_pairs[:, 1]], te_y))
compute final accuracy on training and test sets
y_pred = model.predict([tr_pairs[:, 0], tr_pairs[:, 1]])
tr_acc = compute_accuracy(tr_y, y_pred)
y_pred = model.predict([te_pairs[:, 0], te_pairs[:, 1]])
te_acc = compute_accuracy(te_y, y_pred)
print('* Accuracy on training set: %0.2f%%' % (100 * tr_acc))
print('* Accuracy on test set: %0.2f%%' % (100 * te_acc))

I meant to ask you to provide the whole stack traceback i.e. the whole error message so that I can look into Keras code and find which line is throwing the error. Also, I read your code and if it is working for test_size>=0.5 and not working for test_size<0.5 then there's a possibility that during random splitting, only 1 type of class is present in train or test set instead of two. I suggest after you get data from create_pairs function, check shapes of data and also check that both classes - 0 & 1 are present in y_train and y_test before proceeding to train.

Thanks Ajay. The error is resolved. I have increased the data so that it may contain both the sets of classes. By doing this, the problem is solved.

Happy to help!


Disable user authentication

Is your feature request related to a problem? Please describe.
I wan't to add NPM on my own applications list, I use goauthentik for my application repository. Unfortunately NPM is using local users and is not able to provide openid auth..
Describe the solution you'd like
I wan't to disable NPM user authentication and management
Describe alternatives you've considered
I think it is possible to use the api to get authentication token, but would be better to fully disable user auth.
Additional context
if there is a workaround that you know, would be great to know

Neither removing authentication nor using a different authentication method is currently possible. Related: https://github.com/jc21/nginx-proxy-manager/issues/437

+1

Issue is now considered stale. If you want to keep it open, please comment 👍

I assume to be used with Authentication service like Authentik; it Port 81 is not exposed and NPM is locked behind a proxy forward authentication?
I assume an OpenID SSO support would be much more useful

Hi,
I'm going to comment this just to raise the interest in it. Please, let us disable the authentication or use an header with the username at least. It should be simpler than implement OAuth

I added this for services with basic HTTP authentication to disable it. I then integrated it into Authentik reverse proxy authentication and it works as expected:
location / {
    proxy_pass http://app_with_basic_http_authentication:port;
    proxy_set_header Authorization "Basic wersdfxcvetc";     #base64 of username:password
    proxy_pass_header Authorization;
}

Is it posible to send the correct headers with the correct values to bypass authentication on NPM admin interface?

This worked with Cockpit. Here I used Authorization Headers to bypass the Login screen and used Authentik + NginX for reverse Proxy Authentication.
I am trying the same Implementation with NPM, but it is not exactly working.


GPU support

Issue #87

Pull Request Test Coverage Report for Build 423

48 of 53 (90.57%) changed or added relevant lines in 4 files are covered.
28 unchanged lines in 14 files lost coverage.
Overall coverage decreased (-0.02%) to 84.178%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

datmo/core/controller/environment/driver/dockerenv.py
40
45
88.89%

Files with Coverage Reduction
New Missed Lines
%

datmo/snapshot.py
1
84.06%

datmo/cli/command/base.py
1
93.02%

datmo/cli/driver/helper.py
2
57.53%

datmo/cli/command/snapshot.py
2
100.0%

datmo/cli/command/datmo_command.py
2
100.0%

datmo/core/controller/file/driver/local.py
2
91.9%

datmo/core/controller/environment/driver/dockerenv.py
2
89.83%

datmo/core/util/i18n.py
2
100.0%

datmo/core/util/json_store.py
2
82.35%

datmo/core/util/misc_functions.py
2
71.13%

Totals

Change from base Build 419:
-0.02%

Covered Lines:
2575

Relevant Lines:
3059

💛  - Coveralls

This PR still needs testing on an nvidia GPU machine

Confirmed this works on datmo-server which has 4x Titan X GPUs.
datmo-team@datmo-server:~/projects/datmo$ pytest ./datmo/core/controller/environment/driver/tests/test_dockerenv.py
================================================================================ test session starts ================================================================================
platform linux2 -- Python 2.7.12, pytest-3.5.1, py-1.5.3, pluggy-0.6.0
rootdir: /home/datmo-team/projects/datmo, inifile: pytest.ini
collected 27 items

datmo/core/controller/environment/driver/tests/test_dockerenv.py ...........................                                                                                [100%]

============================================================================ 27 passed in 103.77 seconds ========================================================

Tue May 15 21:03:00 2018
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 390.25                 Driver Version: 390.25                    |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  GeForce GTX TIT...  Off  | 00000000:01:00.0 Off |                  N/A |
| 22%   32C    P8    14W / 250W |      1MiB / 12212MiB |      1%      Default |
+-------------------------------+----------------------+----------------------+
|   1  GeForce GTX TIT...  Off  | 00000000:02:00.0 Off |                  N/A |
| 22%   40C    P8    17W / 250W |      1MiB / 12212MiB |      1%      Default |
+-------------------------------+----------------------+----------------------+
|   2  GeForce GTX TIT...  Off  | 00000000:03:00.0 Off |                  N/A |
| 22%   43C    P8    14W / 250W |      1MiB / 12212MiB |      1%      Default |
+-------------------------------+----------------------+----------------------+
|   3  GeForce GTX TIT...  Off  | 00000000:04:00.0  On |                  N/A |
| 22%   40C    P8    15W / 250W |    293MiB / 12209MiB |      1%      Default |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID   Type   Process name                             Usage      |
|=============================================================================|
+-----------------------------------------------------------------------------+

These directions work.
https://github.com/NVIDIA/nvidia-docker/wiki/Installation-(version-2.0)
Just make sure you have docker-ce installed.
https://docs.docker.com/install/linux/docker-ce/ubuntu/#install-docker-ce-1

the windows build fails because you will need to use the decorator defined in misc_functions.py (pytest_docker_environment_failed_instantiation(filepath)) for the test that is currently failing (which looks like the GPU enabled test) since it requires docker to work. This test will be skipped if a proper docker environment cannot be initialized or built (in the windows case, the linux test container cannot be built ). You can view the test_dockerenv.py file for examples

@pennyfx this is nice that it works on GPUs. Wanted to confirm if it also works on AWS since most would be using it in the cloud?

@pennyfx I don't see a check for Docker >= 1.12 in the instantiation of the Docker Environment Manager. We have a version check for the Git Code Manager as well, so I believe we should enforce this and error if not so the user can fix that, otherwise there will be downstream effects when we try to run certain functions (like the --runtime flag)

docker 1.12 is from 2016, I doubt devs are still using a version that old.   However, I'm not sure if there are some features between 1.12 and the current version that are required for the nvidia runtime to function properly.
https://docs.docker.com/release-notes/docker-engine/#1120-2016-07-28
I'm leaning more on the side of letting it blowup before adding premature error handling that can be too broad.

Still failing on windows. Might be a docker client version issue, since it is passing on master. Going to have to investigate this on a separate branch. For some reason it also seems like CircleCI isn't being triggered to run tests.


Question about updating temporary global variable

Title:  Question about updating temporary global variable
Env
in case
[Taichi] mode=release
[Taichi] version 0.6.37, llvm 10.0.0, commit e0ac1d86, osx, python 3.8.3
[Taichi] Starting on arch=x64
In short
I want to access temporary variable(say init by ti.Vector() ) in taichi scope. However, the results turns out that in taichi scope , program could only capture the initial value of the temporary variable if the variable is not the input of this taichi scope function, but in py scope it could.
To be clear, here we have this script:
import taichi as ti
import math

ti.init(ti.cpu, debug=True)

@ti.data_oriented
class trsfrm:
    def __init__(self, translation=ti.Vector([0.0, 0.0]), orientation=0.0, localscale=1.0):
        '''
        class variable init by ti.Vector
        :param translation:
        :param orientation:
        :param localscale:
        '''
        self.translation = translation
        self.orientation = orientation % (2 * math.pi)
        self.localScale = localscale

    def __repr__(self):
        return '{} ( Trsln : {}, Ornttn: {}, lclScl: {})'.format(
            self.__class__.__name__,
            self.translation,
            self.orientation,
            self.localScale)

@ti.data_oriented
class trsfrm_field:
    def __init__(self):
        '''
        class member init by ti.Vector.field
        '''
        self.translation = ti.Vector.field(2, dtype=ti.f32, shape=[1])

        # self.orientation = orientation % (2 * math.pi)
        # self.localScale = localscale

    def __repr__(self):
        return '{} ( Trsln : {})'.format(
            self.__class__.__name__,
            self.translation,
            )

@ti.kernel
def kern_test_with_input(t_1 : ti.template(), t_2 : ti.template()):
    print("kern with input t1: ", t_1)
    print("kern with input t2: ", t_2[0])

@ti.kernel
def kern_test_without_input():
    print("kern without input t1: ", t1.translation)
    print("kern without input t2: ", t2.translation[0])

@ti.kernel
def kern_add(t:ti.template()):
    t[0] += ti.Vector([2.0, 2.0])
    # print("kern add field", t[0])

t1 = trsfrm()
t2 = trsfrm_field()

while (True):

    t1.translation = t1.translation + ti.Vector([2.0, 2.0])
    print(t1)
    print("py scope print t1: ",t1.translation)

    # increment t2
    kern_add(t2.translation)

    kern_test_without_input()
    # can not capture the incremented t1
    kern_test_with_input(t1.translation, t2.translation)
    print(" ")

What's the program?
Bascially, the program has two forms of class trsfm, one(trsfm) init with ti.Vector, another(trsfrm_field) with ti.Vector.field().
My current implementation is class trsfrm, what I wanna do is   to increment the translation member of class trsfrm and capture the incremented class trsfrm in taichi scope.
However, in kern_test_without_input it could only capture the initial value( keep printing [2.0, 2.0] instead of incrementing itself ). The funny thing to me is that: in py scope the t1 still updates itself . While in  kern_test_input, with t1 as input  it can capture the increasing value.
The truth is, the trsfrm_field no matter without or with input, the taichi scope could always capture the increasement.
The program output is :
trsfrm ( Trsln : [2. 2.], Ornttn: 0.0, lclScl: 1.0)
py scope print t1:  [2. 2.]
[Taichi] materializing...
kern without input t1:  [2.000000, 2.000000]
kern without input t2:  [2.000000, 2.000000]
kern with input t1:  [2.000000, 2.000000]
kern with input t2:  [2.000000, 2.000000]

trsfrm ( Trsln : [4. 4.], Ornttn: 0.0, lclScl: 1.0)
py scope print t1:  [4. 4.]
kern without input t1:  [2.000000, 2.000000]
kern without input t2:  [4.000000, 4.000000]
kern with input t1:  [4.000000, 4.000000]
kern with input t2:  [4.000000, 4.000000]

trsfrm ( Trsln : [6. 6.], Ornttn: 0.0, lclScl: 1.0)
py scope print t1:  [6. 6.]
kern without input t1:  [2.000000, 2.000000]
kern without input t2:  [6.000000, 6.000000]
kern with input t1:  [6.000000, 6.000000]
kern with input t2:  [6.000000, 6.000000]

The kern without input t1 is always the same. Due to some reasons I expect it to increase as well.
Sry about the wording
So The Question:
Is there an elegant way to capture the ti.Vector change in taichi global scope?
Why do I ask this?
I want to implement a transform class for collider in euler based fluid simulation in order to make the collider rotate or translate. Here is the repo link.
The circle(collider) stands still if I do not make the transform.rotation as the function input in rendering function. (currently I access the )
Currently my walkaround is either

take as input
init with ti.field

But personally,  I am not a big fan of capturing the increasement by adding t1 as input because in that way  it may take undefined multiple number of colliders as input. And probably due to this, although the ball can rotate, the fps drops drastically(from 15fps -> 8fps). From the experience, it should not happen since the function did not change that much whether with or without the input.
Also init the transform member with ti.field is not that feasible to me because ti.field cannot be init after materialize or start an kernel function. In this way we somehow cannot trivially add new collider with transform as member to the scene.
So is there any walkaround for this ? 0.0
Hope my statement is clear and clean...

Has been discussed in Taichi forum. Would make the transform variable become ti.field


Dynamic public path

This PR contains a:

[ ] bugfix
[x] new feature
[ ] code refactor
[x] test update
[ ] typo fix
[ ] metadata update

Motivation / Use-Case
We use the file-loader in a next.js plugin to automatically generate asset urls by importing the assets. The asset file names will be hashed, which is good for caching indefinitely. The plugin supports next.js' assetPrefix which allows to provide a different cache server host url for static assets. Next.js uses __webpack_public_path__ by default.
The problem is that in our case the assetPrefix is dynamic at runtime, and will be set based on environment variables which are not present at compile time. The plugin-static gets used during compile time though, and will only use the assetPrefix that is based on environment variables present at compile time. The proper way around this is to make use of __webpack_public_path__. The problem with file-loader is that when setting the publicPath, it is not possible to use the __webpack_public_path__. Therefore a new setting is needed which allows to prefix with __webpack_public_path__.
Breaking Changes
No breaking changes.
Additional Info

That‘s too bad, and I don‘t completely understand it. Could it be that there is a misunderstanding on what the PR does?

With vanilla file-loader, when not setting the publicPath, it will default to __webpack_public_path__ + ${JSON.stringify(outputPath)}, which is basically the same as what‘s possible with this PR.
In this PR, __webpack_public_path__ is not set, it is just returned, so it will be used at runtime to build the public path. Exactly this is necessary so clients can set it according to the documentation you mentioned, and get a path depending on runtime environment variables.
The change of this PR is purely additional. If prefixPublicPathWithWebpackPublicPath is not explicitly set, clients will get the old behavior.

hm, why don't use publicPath as Function?

That is not possible, because the function is executed at buildtime, but I need to modify the publicPath at runtime depending on runtime environment variables that cannot be present at buildtime.

Please provide plugin/app/reproducible repo with problematic and describe what you want to solve in README.md.
As i said above webpack@5 will have built-in file-loader so you solution can't work in next webpack version, we should search way how to solve this in other way.

oh, looks i see your problem, no need link

Ok thx

Let's keep open

I think we need do breaking change again to solve this, publicPath option is broken:

https://github.com/webpack-contrib/file-loader/blob/903aa7544f7c25d524e25a7dc171e9d4d6f80980/src/index.js#L35 we don't use JSON.stringify here, so potentially break code
https://github.com/webpack-contrib/file-loader/blob/903aa7544f7c25d524e25a7dc171e9d4d6f80980/src/index.js#L37 we don't add __webpack_public_path__ here (i think we should this do)

Good catch, anyway publicPath should works as workaround for you:
publicPath(url) {
  return '__webpack_public_path__' + url;
}

Thanks for taking the time to understand the problem!
Until I can switch to webpack 5 it‘s fine for me to use my forked package.

I think we should correct this behavior and migrate all tests to webpack@5 to ensure all works es expected

I think we need do breaking change again to solve this, publicPath option is broken:

publicPath = options.publicPath(url, this.resourcePath, context);
we don't use JSON.stringify here, so potentially break code.

Not true because there is not return. In line 44 it is stringified.

publicPath = `${

we don't add webpack_public_path here (i think we should do this)

Also not true because there is no return. Line 46 will be executed in either way.

Good catch, anyway publicPath should works as workaround for you:
publicPath(url) {
  return '__webpack_public_path__' + url;
}

No, this does not work, because of the JSON.stringify in line 44. This will add quote around the whole string like this: "__webpack_public_path__ + passed_url" which cannot work.

No, this does not work, because of the JSON.stringify in line 44. This will add quote around the whole string like this: "webpack_public_path + passed_url" which cannot work.

Do you try this? We don't add anything when publicPath is function https://github.com/webpack-contrib/file-loader/blob/903aa7544f7c25d524e25a7dc171e9d4d6f80980/src/index.js#L34

Do you try this? We don't add anything when publicPath is function

Not true. Line 44 is also executed in this case afterwards, because there is no return in line 35. And yes, I tried it 😉

Oh, yes sorry, anyway you catch a bug and we fix it in next release (wrkaround: use forked repo right now)

/cc @fabb sorry for delay, let's use better name (ideas?), also let's do this as String|Function (default string is __webpack_public_path__)

Sorry, no better ideas for naming 🙈
String|Function is a good idea, i guess you mean for the publicPath option? Then we don‘t need to find a better name for prefixPublicPathWithWebpackPublicPath, because we don‘t need it at all, right?

Maybe just postTransformPublicPath because you can add any own global variable before publicPath not only __webpack_public_path__ when you use Function?

Ah, good idea. I just saw that publicPath can already be a function, but the result still gets stringified, so for backwards compatibility, a new option is needed anyways (or overload the return value of the publicPath function, but that gets too complicated.
Ok will change the PR soon.

Feel free to ping me

@evilebottnawi I've implemented the change. I fear it might be a bit confusing for users on how to use the option correctly. Please take a look at the unit tests and tell me what you think.

ok done! any additional remarks? should we add resourcePath and context as parameters as well for flexibility?

@fabb let's rebase

done!

Good work!

@evilebottnawi can we merge the PR?

@fabb yep, at this week we merge and release, in my todo

I remember you was against it, @evilebottnawi?
Quotes are yours:

Because it is fix only logic for file-loader, but other loader also can emit assets, i think we need solve this on webpack side, so i request minimum reproducible test repo

Does this PR fix this on webpack side? Where's the "minimum reproducible test repo"?

We don't want add new option to file-loader, because in webpack@5 it was built-in loader, so we need solve this problem on webpack level to avoid same problems in future for other developers

You refused to merge my PR even if it was not adding any new options, but merged this one with an additional option?

@prontiol A similar feature was implemented on webpack side and several people requested a similar solution for file-loader.
Sometimes I'm mistaken, I should have heard you better, sometimes it’s difficult because of the influx of many ideas, most of which look terrible, sorry


Add reverse operation for XorLinkedList

One of the nicest things about XOR linked lists is the O(1) reverse.  This PR adds that functionality.

Thanks!


atlmfc:x64-windows-static failed : BUILD_FAILED

Vcpkg version: 2018.11.23-nohash

The ATL/MFC component must be installed using the Visual Studio installer.
After you install them vcpkg install atlmfc:x64-windows-static should succeed.
@koudic can you confirm whether you have ATL/MFC components installed?

@vicroms, thanks for the tip, only ATL was installed, after installing the MFC component everything worked out with the installation.

Hi @koudic, thank you for posting the issue, this issue has been resolved, so I am closing this issue for now.


upgrade dependency

fix https://github.com/gliderlabs/logspout/issues/449

looks like the bump in version requires use of a new API

looks like the bump in version requires use of a new API
router/pump.go:160:20: SA1019: p.client.InspectContainer is deprecated: Use InspectContainerWithOptions instead.  (staticcheck)
	container, err := p.client.InspectContainer(event.ID)
	                  ^
router/pump.go:206:20: SA1019: p.client.InspectContainer is deprecated: Use InspectContainerWithOptions instead.  (staticcheck)
	container, err := p.client.InspectContainer(id)
	                  ^
router/pump.go:273:22: SA1019: p.client.InspectContainer is deprecated: Use InspectContainerWithOptions instead.  (staticcheck)
			container, err := p.client.InspectContainer(id)

In fact I have successfully built and deployed logspout to our server with the Rate unmarshal issue solved. I'll look into this.

In fact I have successfully built and deployed logspout to our server with the Rate unmarshal issue solved. I'll look into this.

this was caught by golangci-lint. the CI process requires this to pass regardless of the code compiling

Thanks for reminding me. I'll fix this soon.
On Tue, Feb 9, 2021 at 12:00 AM Michael Hobbs<EMAIL_ADDRESS>wrote:

this was caught by golangci-lint. the CI process requires this to pass
regardless of the code compiling
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gliderlabs/logspout/pull/501#issuecomment-775250542,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABAA7JYNXRKBZDAGV4MUZJLS6ADBTANCNFSM4XJHIZBA
.

The deprecated API has been removed, according to its implementation here https://github.com/fsouza/go-dockerclient/blob/v1.7.0/container_inspect.go#L14


Ipfs package

Changes
HTTP plugin
I have refactored the HTTP plugin to support sending body as form data (this is necessary as IPFS expects files to come in this way). Currently, I implemented this by defining Body type to be:
type Body {
  stringBody: String
  rawBody: Bytes
  formDataBody: FormData!
}

This allows users to define what kind of request body (payload) is going to be sent. I am not entirely pleased with this solution...it would be for sure better to define a union type for the body that would include all defined types (body: String | Bytes | FormData) but we are not going to support this so I had to find a workaround. Another idea is to have multiple post functions defined within HTTP plugin (postWithStringPayload, postWithFormDataPayload,...).
@dOrgJelli I would like to hear what you think about the current solution.
IPFS package
Implemented catFile, catToString and addFile ipfs functions + e2e tests.
Other
I expanded query command of CLI to support defining paths toward files in format file:path/to/file/example.txt. This will read file content and convert it to Uint8Array. This is used when testing ipfs function addFile, but I think functionality itself could be useful in multiple usecases.
TODO
I can't figure out why, but when inside schema definition of HTTP plugin I set property formDataBody of body to be nullable then generated code has a problem on the compilation. For some reason in generated function writeHttp_Body even though there is a check if (type.formDataBody) before calling write, the type is being considered to be Http_FormData | null, and because of that compilation fails.
Here is the error message:
Compile Web3APIERROR TS2322: Type 'src/mutation/w3/imported/Http_FormData/index/Http_FormData | null' is not assignable to type 'src/mutation/w3/imported/Http_FormData/index/Http_FormData'.

     Types.Http_FormData.write(writer, type.formDataBody);
                                       ~~~~~~~~~~~~~~~~~
 in src/mutation/w3/imported/Http_Body/serialization.ts(29,39)

And here is a snip from generated code where an error occurs:
export function writeHttp_Body(writer: Write, type: Http_Body): void {
  writer.writeMapLength(3);
  writer.writeString("stringBody");
  writer.writeNullableString(type.stringBody);
  writer.writeString("rawBody");
  writer.writeNullableBytes(type.rawBody);
  writer.writeString("formDataBody");
  if (type.formDataBody) {
    Types.Http_FormData.write(writer, type.formDataBody);
  } else {
    writer.writeNil();
  }
}

Issues
Closes #188
Closes #247

I also changed the CI timeout from 15 to 20 minutes as CI was failing because of this.

Diving into this now, my apologies for lagging on getting to this @MakMuftic, looks fantastic at first look through!

In reply to the lack of union type support in GraphQL & WASM (AssemblyScript), I think we'll hit this problem over and over again, and lacking unions will surely be a problem when porting SDKs to Web3API (especially those written in dynamic languages like JavaScript).
In order to get around this, I think we can simply support enums within any WASM compatible language by create union specific variant objects. For example:
union Body = String | Bytes | FormData
Would turn into a simple variant class. In languages that support manual memory allocation and points (for ex C/C++), we can be space efficient (psuedo code):
enum BodyType {
  String,
  Bytes,
  FormData
}

class Body {
  public const BodyType m_type;
  private void* _instance;

  public constructor(BodyType type, MsgPackBuffer* input = 0)
  {
    this.m_type = type;

    switch (this.m_type) {
      case BodyType::String:
        this._instance = input ? deserializeString(input) : new String();
        break;
      case BodyType::Bytes:
        ...
    }
  }

  public getString() String {
    if (this.m_type != BodyType::String) {
      throw new Error(
        `Incorrect Body instance access. Tried to get "String", when instance is of type "${this.m_type}".`
      );
    }
    return String(this._instance);
  }

  ...
}

In languages that don't support dynamic allocations (like AssemblyScript), and also don't have support for union types, we can use a less memory efficient pattern like so:
enum BodyType {
  String,
  Bytes,
  FormData
}

class Body {
  public readonly type: BodyType;
  private _string: String | null;
  private _bytes: Bytes | null;
  private _formData: FormData | null;

  constructor(type: BodyType, input?: MsgPackBuffer) {
    this.type = type;

    switch (this.type) {
      case BodyType::String:
        this._string = input ? deseralizeString(input) : new String();
        break;
      case BodyType::Bytes:
        ...
    }
  }

  public getString(): String {
    if (this.type != BodyType::String) {
      throw new Error(
        `Incorrect Body instance access. Tried to get "String", when instance is of type "${this.type}".`
      );
    }
    return this._string;
  }

  ...
}

Created this issue: https://github.com/Web3-API/monorepo/issues/246

Also @MakMuftic I found a solution to the bug you found with the AssemblyScript compiler not respecting the null check in the generated code. Pushing a fix now.

^^^ this will allow for formDataBody to be nullable as you've described. This will resolve some of my comments I added to the review.

hey @MakMuftic I am trying to build this locally but I am getting an error:

Probably I am missing something, do you have any idea?

Sorry for the late reply, I will investigate what happened and try to resolve this problem asap

Thank you so much for your work on this @MakMuftic, it's wonderful!
We're migrating this wrapper to the integrations repo in this PR: https://github.com/polywrap/integrations/pull/101
Going to close this.


Perform scale up if no candidate leases are present

What this PR does / why we need it:
This PR changes the code to performs scale up if no candidate leases are present
Which issue(s) this PR fixes:
Fixes #111
Special notes for your reviewer:
Release note:

Perform scale up if no candidate leases are present

/cherry-pick rel-1.2.0

/kind enhancement


+setHidden properties to empty array on fillForm record

Please update the fillForm() in Filament\Resources\Pages\EditRecord + Filament\Resources\Pages\ViewRecord as the password field wasn't available as a value passed on the form submit due to it being in the $hidden attribute array in the User model.

Hey, can you see if my new changes instead still fix the issue you were facing?

I assume it did :)


superagent should not escape apostrophes, quotes, etc. in the URL

As stated in http://stackoverflow.com/a/18251730/556678, quotes, apostrophes, and few others should not be escaped.
request(app)
	.get("/alerts?filter=title:invalid'title")
	.expect(200, function (err, res) {

	});

The URL that is sent becomes /alerts?filter=type:invalid%27type.
Calling encodeURIComponent("invalid'title") produces the proper result invalid'title where apostrophe is not escaped.

We use encodeURIComponent  https://github.com/visionmedia/superagent/blob/d4fb001df6ce6862e3dbdb4999c1ff82b54a4bad/lib/client.js#L99
but XHR in the browser does not allow unencoded '. xhr.open('GET', "/'") will send GET /%27 HTTP/1.1 on the wire.

Currently I have to do:
('/alerts?filter=' + encodeURIComponent(filter).replace("'", '%27'));


Doesnt work in valorant?

Hey,
so I tried the script had the gui popup clicked the enable button and it said quickswitch mode enabled and it registered D and A key presses in the terminal but when I tried it out in valorant (the range) it did not seem to do anything when i held D and pressed A it just slowed me down but I kept going right. Perhaps im using it wrong? Idk just didnt work for me. anyways thank you for reading!

Code has been fixed


Migrate support libs to AndroidX

More info here: https://developer.android.com/topic/libraries/support-library/androidx-rn

Also lib KTX is deprecated


ospd-openvas: 22.5.0 -> 22.5.1

Diff: https://github.com/greenbone/ospd-openvas/compare/refs/tags/v22.5.0...v22.5.1
Changelog: https://github.com/greenbone/ospd-openvas/blob/22.5.1/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 232597 run on x86_64-linux 1

2 packages built:

ospd-openvas
ospd-openvas.dist


State Tool

Implement the state tool:

Click the icon to activate the tool:

icon graphically shows to be active
mouse turns into a small state with a even small plus sign (+) at the lower right corner

Click as often as you need into the plan editor field:

creates a state with member variables set to some default each time

Click the icon to deactivate the tool:

icon gets deactivated
mouse turns back to normal

works


Loading Commands Dynamically

Right now all commands are loaded manually.
https://github.com/valkyrienyanko/Unity-Transport-Layer-Boilerplate/blob/2b0450020b623bd31b0f9852c84d6e652afe7432/Server/Assets/Scripts/Console.cs#L44-L52
If the commands were loaded dynamically somehow, this would save the time of having to manually type out each new command in the future.
https://github.com/valkyrienyanko/Unity-Transport-Layer-Boilerplate/blob/2b0450020b623bd31b0f9852c84d6e652afe7432/Server/Assets/Scripts/Console.cs#L33-L42

Can't be solved without ugly reflection.

Commands = typeof(Command) // Gets the type Command
    .Assembly // Gets the assembly (.DLL) where Command is located
    .GetTypes() // Gets all the types of the assembly where Command is located
    .Where(x => typeof(Command).IsAssignableFrom(x) && // Filters by types that inherits Command
    !x.IsAbstract) // Excluding abstract types
    .Select(Activator.CreateInstance) // Creates an instance of each one
    .Cast<Command>() // Cast them all to Command
    .ToDictionary(x => x.GetType().Name.ToLower(), x => x); // creates the dictionary


update for sending small tx below 0.1 NYAN

I have built and this does indeed fix bug when sending using electrumx. Merged.


Add the oauth2 keys setup in scripts/installer.js

What is the motivation for adding / enhancing this feature?

Currently, we do have a fast/simple integration path with Magento2: https://github.com/DivanteLtd/vue-storefront/blob/docs/docs/guide/installation/magento.md#fast-integration
Adding the questions for the vue-storefront-api/config/local.json:magento2:api section and putting the config there in the https://github.com/DivanteLtd/vue-storefront/blob/035f3a7bc70d5669fc431fddae1caf6369682e2e/core/scripts/installer.js#L225
will be awesome!
What are the acceptance criteria

the installer will ask the user for "url", "consumerKey", "consumerSecret", "accessToken", "accessTokenSecret"
the installer will update the vue-storefront-api/config/local.json

This section - on m2 integration - shouldn't be required. It should ask You "would You like to perform initial data import from Magento2 instance?"


Test updated to use the actual IRI

@kasei I have created permanent identifiers for the two datatypes, for the SPARQL functions, and for the spec. With this, the IRI prefix that our usual cdt: prefix name maps to is
https://w3id.org/awslabs/neptune/SPARQL-CDTs/
Hence,

cdt:List is https://w3id.org/awslabs/neptune/SPARQL-CDTs/List
cdt:Map is https://w3id.org/awslabs/neptune/SPARQL-CDTs/Map
cdt:contains is https://w3id.org/awslabs/neptune/SPARQL-CDTs/contains
etc.

I have changed the spec accordingly, and updated all the tests (see commit https://github.com/awslabs/SPARQL-CDTs/commit/2a9f6e988a2795fb530230593a7a33bc91802ef8).
You can use this issue as a reminder to update your implementation. Once you have updated it and it can run the updated tests, you can close this issue.

I've updated the namespace prefix in my implementation. I still have a few tests that are not passing, but unrelated to this change. I'm happy to see this closed, but I don't think I have permissions to do that in the awslabs' repo.

Thanks Greg! I close it then.


GPX path not displaying since v1.9.5

Describe the bug
I just updated my docker container to v1.9.5 and noticed that the GPX path on the maps are not displayed anymore
Server logs (optional)
No log from the server apart from this during compression :
8/25/2023, 2:34:14 PM[WARN_] GPXCompressionJob  Could not compress gpx at: /app/data/foobar.gpx
Environment (please complete the following information):
Debian bullseye. Problem seen on Brave and Firefox
Used app version:

release v1.9.5

Please try it with nightly to see if it still an issue
-- Sorry for being brief, sent from my phone.
On Fri, 25 Aug 2023, 16:44 lythandas, @.***> wrote:

Describe the bug
I just updated my docker container to v1.9.5 and noticed that the GPX path
on the maps are not displayed anymore
Server logs (optional)
No log from the server apart from this during compression :
8/25/2023, 2:34:14 PM[WARN_] GPXCompressionJob Could not compress gpx at:
/app/data/foobar.gpx
Environment (please complete the following information):
Debian bullseye. Problem seen on Brave and Firefox
Used app version:

release v1.9.5

—
Reply to this email directly, view it on GitHub
https://github.com/bpatrik/pigallery2/issues/701, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZKA5WXE6Y6CB6BPHXTF2TXXC25XANCNFSM6AAAAAA36TTS24
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hey bpatrik, thanks for that very quick reply.
Unfortunately, nightly has this error on startup :
TypeError: Preview.Sorting should be an Enum from values: 10,11,20,21,30,31,40,41,100,ascName,descName,ascDate,descDate,ascRating,descRating,ascPersonCount,descPersonCount,random, got: 6
    at SubConfigClassType.__validateType (/app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:384:23)
    at SubConfigClassType.__validate (/app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:254:29)
    at SubConfigClassType.__validateType (/app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:343:51)
    at SubConfigClassType.__validate (/app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:254:29)
    at SubConfigClassType.set [as Sorting] (/app/node_modules/typeconfig/src/decorators/property/ConfigPropoerty.js:61:62)
    at set (/app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:158:35)
    at /app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:174:25
    at Array.forEach (<anonymous>)
    at SubConfigClassType.__loadJSONObject (/app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:149:39)
    at /app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:179:41

Node.js v18.17.1
/app/node_modules/typeconfig/src/decorators/class/ConfigClass.js:151
                            throw e;
                            ^

Just delete the config.json
-- Sorry for being brief, sent from my phone.
On Fri, 25 Aug 2023, 17:20 lythandas, @.***> wrote:

Hey bpatrik, thanks for that very quick reply.
Unfortunately, nightly has this error on startup :
TypeError: Preview.Sorting should be an Enum from values: 10,11,20,21,30,31,40,41,100,ascName,descName,ascDate,descDate,ascRating,descRating,ascPersonCount,descPersonCount,random, got: 6
at SubConfigClassType.__validateType (/app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:384:23)
at SubConfigClassType.__validate (/app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:254:29)
at SubConfigClassType.__validateType (/app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:343:51)
at SubConfigClassType.__validate (/app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:254:29)
at SubConfigClassType.set [as Sorting] (/app/node_modules/typeconfig/src/decorators/property/ConfigPropoerty.js:61:62)
at set (/app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:158:35)
at /app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:174:25
at Array.forEach ()
at SubConfigClassType.__loadJSONObject (/app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:149:39)
at /app/node_modules/typeconfig/src/decorators/class/base/ConfigClassBase.js:179:41
Node.js v18.17.1
/app/node_modules/typeconfig/src/decorators/class/ConfigClass.js:151
throw e;
^
—
Reply to this email directly, view it on GitHub
https://github.com/bpatrik/pigallery2/issues/701#issuecomment-1693532728,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZKA5TYM2AIHQO5CXUJZCTXXC7ETANCNFSM6AAAAAA36TTS24
.
You are receiving this because you commented.Message ID:
@.***>

Alright, now I may have some interesting piece of information .
It's still not working but now I can see this in the logs :
/app/src/backend/model/fileprocessing/GPXProcessing.js:82
            const timeDelta = (Date.parse(list[i].time[0]) - Date.parse(list[i - 1].time[0])); // mill sec.
                                                      ^

TypeError: Cannot read properties of undefined (reading '0')
    at gpxEntryFilter (/app/src/backend/model/fileprocessing/GPXProcessing.js:82:55)
    at Array.filter (<anonymous>)
    at GPXProcessing.compressGPX (/app/src/backend/model/fileprocessing/GPXProcessing.js:87:51)
    at async compressGPX (/app/src/backend/middlewares/MetaFileMWs.js:24:30)

Node.js v18.13.0

Indeed it's more interesting:)
Can you share a broken gpx with me?
-- Sorry for being brief, sent from my phone.
On Fri, 25 Aug 2023, 18:33 lythandas, @.***> wrote:

Alright, now I may have some interesting piece of information .
It's still not working but now I can see this in the logs :
/app/src/backend/model/fileprocessing/GPXProcessing.js:82
const timeDelta = (Date.parse(list[i].time[0]) - Date.parse(list[i - 1].time[0])); // mill sec.
^
TypeError: Cannot read properties of undefined (reading '0')
at gpxEntryFilter (/app/src/backend/model/fileprocessing/GPXProcessing.js:82:55)
at Array.filter ()
at GPXProcessing.compressGPX (/app/src/backend/model/fileprocessing/GPXProcessing.js:87:51)
at async compressGPX (/app/src/backend/middlewares/MetaFileMWs.js:24:30)
Node.js v18.13.0
—
Reply to this email directly, view it on GitHub
https://github.com/bpatrik/pigallery2/issues/701#issuecomment-1693628666,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZKA5T4WUAPZCA5UTCU36LXXDHWZANCNFSM6AAAAAA36TTS24
.
You are receiving this because you commented.Message ID:
@.***>

islande.zip
Sure, here's a gpx which is a concatenation of several paths, but it was displaying on the map before v1.9.5 (there were some issues with the display however but that's another story, I'll open another issue after we have settle this one)

The latest commit should fix it.

OK ! It's working just as it used to now, but it's back with the other issue. I shall open a new issue.
Many thanks for the fix !


README Tweaks

Coverage remained the same at 99.635% when pulling 3cdaefca1f23053098740c58db4aa71ad1dde897 on FredericJacobs-patch-1 into a52bf5fff1c6d14011c907564b6a1d88742a8bb2 on master.


Pass pomodoro data to statistics

CS2103T Individual Assignment for Sir Hardoh
Problem statement: We want to keep track of data in the past n (7) days while accounting for usage across different days. (eg. Use 1 day, never touch for 2 days, or 10 days, and use again)
Tentative solution: statistics.json will store and track the DayData (Date [LocalDate], PomDurationData [Integer], TasksDoneData [Integer]) for the past n days. Note that you construct Date, PomDurationData and TasksDoneData using Strings.
Statistics will create a list of empty DayData objects for the past 7 days (including current day) upon initialisation in statistics.json. I have defined several helper methods in Statistics for your usage.
fetch() // reinitialises dayDataList to current day while retaining stored data.
updatesDayData(DayData dayData) // Auto-checks and replaces a new DayData object at the same Date
getDayDataFromDate(Date date) // Gets DayData object at current date.

Proposed Implementation:
Check if current date exist in list using getDayDataFromDate.
LEVEL 1
If date exist, simply add data to the DayData obtained.
Then update the list using updatesDayData.
LEVEL 2
If date doesn't exist, this means list is outdated, and should be reinitialised using fetch to update list to the new day, which also deletes the outdated days in the list.
Repeat LEVEL 1 for new day.
If this is the event where day crosses to the next day, remember to repeat LEVEL 1 for the previous day too.
Please feel free to propose/make any changes to this implementation.


benefits of using macros

https://github.com/rust-lang/rust/issues/63775

We don't want macros

Don't close this issue

We don't want macros

Who is "we"? Everyone knows macros are very powerful. That's why they require the "!", so they stand out. You can not use them, nobody forces you to.


Allow menu props to define layout delegate for customisation

Hi @salim-lachdhaf
Currently there is a straightforward logic of popup positioning
To have more flexibility on that matter it would be nice to add a prop pass through to the MenuProps

I will check the this weekend just after releasing the v7.0.0-pre which has a lot of modifications ...


Function declaration's closing bracket highlighted as error

When any but the last destructuring key is surrounded by quotes, the error ocurs:

When any but the last destructuring key is NOT surrounded by quotes, the highlight is ok:

I've managed to research a little bit the issue and this is what I found:

This happens in a .js file.
This happens to the closing bracket of a function declaration. Doesn't matter if it's exported or not.
This happens only when inside that function there's a nested object destructuring statement AND any of the keys but the last one is surrounded by quotes, either single, double, or backticks (see images above).
The result of SyntaxAttr for that closing bracket is group: jsParensError->Error guifg=255(255) guibg=203(203), which is declared in the syntax/javascript.vim file.
I'm using Vim Polyglot with no additional configuration for javascript files.

I'm not very savvy in regular expressions or syntax highlight debugging so I really don't know where to start looking for a fix. Hope someone can have an idea about fixing this.

Closing this bug and creating it in vim-javascript. I think I have a solution for that.

@andresrodco You can use yuezk/vim-js as the alternative to vim-javascript. I believe it won't have this problem and I'm more familiar with that plugin.

Thanks @yuezk That plugin worked as expected!


week number calculation is wrong in some cases

There was an issue with dates during the period that daylight saving was observed and years where 1 Jan was Friday. Fixed by using all UTC methods.
https://stackoverflow.com/questions/6117814/get-week-of-year-in-javascript-like-in-php

fixed in 0.5.1


Meta: Clarify feature names

Resolves #4992

conversation-filters → more-conversation-filters
quick-pr-diff-options → quick-diff-options
quick-review-buttons → review-type-buttons

conversation-filters → more-conversation-filters

👍

quick-pr-diff-options → quick-diff-options

one-click-diff-options?

quick-review-buttons → review-type-buttons

one-click-review-submission?

Actually, most quick-* features could be renamed one-click-*.

Actually, most quick-* features could be renamed one-click-*.

Let's keep this in mind for the future. I'm still wary of renaming features because it's harder to find them in discussions etc.

We don't need to go for one-click-. Some people don't/can't use mouse. The current way is more inclusive.


Update rust-skia to support aarch64

What kind of change does this PR introduce?

[x] Fix
[ ] Feature
[ ] Codestyle
[ ] Refactor
[ ] Other

Did this PR introduce a breaking change?
A breaking change includes anything that breaks backwards compatibility either at compile or run time.

[ ] Yes, please list breaking changes
[x] No

Tested locally on a chromebook machine, seems to work fine. Fixes https://github.com/Kethku/neovide/issues/766. I would just double check with @Shatur before merging
Side note: I think quite a few of the dependencies are out of date now. Is it possible to add dependabots for rust? if not, I might have to manually take a look at them one day

I'm inclined to merge this regardless as it probably doesn't break anything.

@Kethku @shaunsingh Can't build this. The error is exactly the same. I guess we should update the version inside the .toml file too.

@Shatur Whoops, forgot about that. Fixed in https://github.com/Kethku/neovide/blob/main/Cargo.toml, I think it should work now

@shaunsingh @Kethku Now the build is broken for all platforms, you forgot to update the checksums. See #798.


"Authentication failed" after upgrade to v<IP_ADDRESS> while logging into admin panel

I just upgraded my Rainloop Mail to v<IP_ADDRESS> using the option in the Admin Panel. After the upgrade I lost access to the admin panel and started receiving "Authrntication Failed" message using the same credentials.
I modified data/.../appliction.ini to the following
; Login and password for web admin panel
admin_login = "admin"
admin_password = "pass123!@#"

But when I try to use this combination to login to the admin panel I get the following

I'm not facing problems with normal email accounts. Only admin panel seems to have a problem. I encountered this only after the upgrade.
Any help would be great. Thanks!

Sorry for that, but admin password should be hashed (for security reason).
There is only one exception: password 12345 (for backward compatibility).
This means that you need to set "12345" as your password in application.ini.
After that you can change it to your real password through admin panel.

That totally fixed the issue. Thanks. :+1:

Hi,
i've got the same problem but in my application.ini, password is already set to "12345".
Login failed all the time.
Have you any idea ?
Thanks in advance.
Regards


Multiple corrections in the guide (and my previous PR)

-Updated information to disable "Dump MLC" in NAND Dumper as the MLC part isn't needed for unbricking.
-> Since the NAND Backup is relatively small in size without the MLC part, I removed the "You'll need a 64/16GB SD card"
-Changed the SD-size recommendations to say the same as the main page (with some additional information because of my change in the NAND Backup secion)
-I've seen a few people having trouble finding the payload.zip file, so I added additional information on which download includes the payload

so after speaking with plai, there are a few different things in play here.
can you split the corrections and the MLC/NAND backup stuff.
there are revisions that need to be done on the NAND backup/MLC sections that would encompass some of what you requested here.

Will close this because I'll open a new PR (without updated NAND Dumping information)


support gatsbyjs-v3

mainly modify the issuer property.
documentation on Webpack5 issuer property.
https://webpack.js.org/configuration/module/#ruleissuer

Thanks for jumping on this! Is there a way to maintain support for pre-3.0 Gatsby with the same codebase?

The versions of other GatsbyJS plugin packages have been upgraded from 2.x.x to 3.x.x, I think changing the package version number to 3.x.x can clearly indicate support for GatsbyJS V3

Came looking for this. Great work @piecego!

I've published this as v3.0.0-beta.0 for the time being so anybody on Gatsby v3 can give this a whirl without blocking on a solution that will allow us to maintain compatibility with Gatsby v2. This is something that I'd consider a very high priority since v2 has been out in the wild for over 2 years and likely has millions of installs, and the tendency with JS packages is to simply yarn add x instead of specifying a particular version range per-package, especially for plumbing like gatsby-plugin-svgr.

Hey @piecego, I'm going to re-open this since it hasn't been merged in yet. 😄

Will this be merged?


Error: Expression: oldDMN != nullptr since a few days suddenly.

This issue tracker is only for technical issues related to Raptoreum Core.
General Raptoreum questions and/or support requests and are best directed to the Raptoreum Discord.
To report critical security issues, DM the Core or Developer Teams instead of creating Github issues.
Describe the issue
Can you reliably reproduce the issue? yes
If so, please list the steps to reproduce below:
a few days ago this error suddenly appeared after trying to start the Raptoreum Core App. System was not changed since. All Windows-Updates done.
I deinstalled Core App <IP_ADDRESS> - and reinstalled same version - same error
After that I installed newer version <IP_ADDRESS> - same error
After that I installed newest beta <IP_ADDRESS> form here - same error
I deinstalled all Visual C++ components from Windows an reinstalled the latest VC++ All-in-One package - same error
Expected behaviour
App should start and load the database
Screenshots.
If the issue is related to the GUI, screenshots can be added to this issue via drag & drop.
What version o

f Raptoreum Core are you using?
List the version number/commit ID, and if it is an official binary, self compiled or a distribution package such as PPA.
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS> beta
Machine specs:

OS: Win 11 22H2 build 22621.2134
CPU: Ryzen 7 5825U with Radeon Graphics
RAM: 32GB DDR4-3200 Kingston SK-Hynix
Disk size: 2TB Kingston SKC3000D2048G
Disk Type (HD/SDD): SSD

Any extra information that might be useful in the debugging process.
This is normally the contents of a debug.log or config.log file. Raw text or a link to a pastebin type site are preferred.

For running on mainnet you will want to run, <IP_ADDRESS> and the beta version <IP_ADDRESS> if want to run the new assets code on testnet. To fix the issue you will need to apply the bootstrap. The bootstrap can be found here https://bootstrap.raptoreum.com
There is a video guide on how to bootstrap here: https://youtu.be/m41ue-xjpMU
Bootstrap doc is here: https://docs.raptoreum.com/raptoreum-docs/raptoreumcore-other/bootstrap/

Closed,
The fix for mainnet is as @Blueskys379  pointed out to use the correct wallet for mainnet and bootstrap it.


Azure Data Explorer Insights - Add "Commands and Queries" tab (log-based)

Add "Commands and Queries" tab (log-based monitoring)

@gardnerjr, yes, it is intended that 2 titles are visible when the purple bar is visible, so users can see what 2 tiles/what section is based on the log-data. I can revalidate it with our UX expert, but as for now, this is the spec until the next round (will be soon).
Once there are logs, the tiles have a title. The "text steps" are there because the "real" title is longer. See image below:

As I said - I'll revalidate this issue for the next PR (we will edit some of the KQL queries for LA)


Perf: Reduce FontPicker impact on text selection latency

Context
I noticed large element selection latency during the bug bash, and with React Profiler it turned out that FontPicker was one of the most expensive renders during text selection.

Summary
Luckily, it turned out that almost all of that cost was from inefficient generation of fontMap!
Using Array.reduce to generate a map is about 1000x slower, probably due to O(N) object creation (we should probably take a look at other places that use this pattern).

Then I realized that we probably can bypass most instances fontMap generation since recentFonts and curatedFonts are just subsets of fonts (so it only needs to be run once, ever).
Looks like this might shave a quarter second or more off of text selection latency:

Relevant Technical Choices
N/A
To-do
N/A
User-facing changes
Selecting text elements should be more responsive.
Testing Instructions

QA

This PR can be tested by following these steps:

Select a text element in the editor
The editor should update slightly faster (display selection box & update text panel)
The font picker/dropdown should continue to work as expected

UAT

[X] UAT should use the same steps as above.

Reviews
Does this PR have a security-related impact?
No.
Does this PR change what data or activity we track or use?
No.
Does this PR have a legal-related impact?
No.
Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[ ] I have tested this code to the best of my abilities
[ ] I have verified accessibility to the best of my abilities (docs)
[ ] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[ ] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[ ] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Partial for #8259

Nice find!


feat(plugin): add twikoo comment plugin support

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


start_sync failing

I am upgrading from PG12 and I initially was trying to upgrade to PG15.
The error message I am seeing:
ERROR: CREATE SUBSCRIPTION ... WITH (create_slot = true) cannot run inside a transaction block

I enabled extra logging with the DEBUG=1 environment setting, and I can see that there we are not running the Query with a transaction.
Things I tried
I tried upgrading to PG 14 and PG 13 to see if there was an issue with postgres 15 creating the slot on postgres 12.
I tried using the deferred slot approach from the postgres docs:

Change CREATE SUBSCRIPTION to have connect = false.
Run pg_create_logical_replication_slot on the source db.
Run the ALTER SUBSCRIPTION ... ENABLE.
Run the `ALTER SUBSCRIPTION .. REFRESH PUBLICATION.

The REFRESH PUBLICATION failed with the same error message.
Questions:

Is this an issue that my source database version is too old? If so, what version of postgres is supported at minimum?
Is this an issue of a long outstanding transaction on the source database? I don't think so since I would expect an issue about locks expiring if so...
Any other ideas to consider or try?

Interesting. Are you manually modifying the create subscription somehow? Curious about create_slot = true, since start_sync doesn't take additional params also.

Closing due to inacitivity, please feel free to reopen with any additional thoughts or feedback, when you have a chance too.


Need to write context-path, when call to other microservice

When try get from send-sms service
restTemplate.getForObject("http://send-sms/send-sms/Test", Object.class);
It's seems that the service-discovery not share the context-path.
send-sms.properties
spring.application.name=send-sms
server.servlet.context-path=/send-sms
server.port: 8060
eureka.client.serviceUrl.defaultZone=http://localhost:8761/eureka/
eureka.client.registerWithEureka = true
eureka.client.fetchRegistry = true

Service discovery doesn't know about context path. This is a duplicate. I'll find the issue later

Service discovery know a little.
Whenever I added
server.contextPath=/send-sms   (in addition to server.servlet.context-path=/send-sms)
and click on the send-sms service in Service discovery panel,
a new tab opens with url = http://pc.mshome.net:8060/send-sms/actuator/info

It might be in the metadata but the load balancer will not add it by default

There is some way the load balancer can get this information(contextPath of the service)?

You can use the DiscoveryClient to get the metadata


⬆  Bump Uvicorn max range to 0.17.x

I believe this is important to be done due to:
https://github.com/encode/uvicorn/pull/1400
https://github.com/MagicStack/httptools/issues/76

New versions of Uvicorn are not compatible with FastAPI, doing a few tests websockets stop working all together.

@dynalz Can you give more info on this? I've just run the three examples in https://fastapi.tiangolo.com/advanced/websockets/, using the following environment and they seem to be working:

New versions of Uvicorn are not compatible with FastAPI, doing a few tests websockets stop working all together.

An issue on uvicorn on how to reproduce it is appreciated...

Same here, installed Uvicorn (standard) separately from FastAPI and it's doing fine, currently in 0.17.6

Thanks @pedropozzobon! And thanks for the conversation everyone.
This was included as part of https://github.com/tiangolo/fastapi/pull/4803, and it will be available in FastAPI 0.75.2 in a couple of hours, so I'll close this one now.
But thanks for the effort @pedropozzobon! :cake:


Installation Error: Cloning into 'third_party/eigen' failed.

While installing from source, an error occurred.
$ git clone --recursive https://github.com/onnx/onnx-coreml.git

Cloning into 'onnx-coreml'...
remote: Counting objects: 1717, done.
remote: Compressing objects: 100% (47/47), done.
remote: Total 1717 (delta 15), reused 34 (delta 1), pack-reused 1669
Receiving objects: 100% (1717/1717), 326.54 KiB | 411.00 KiB/s, done.
Resolving deltas: 100% (877/877), done.
Checking connectivity... done.
......
Cloning into 'third_party/eigen'...
Username for 'https://github.com': naruya
Password for<EMAIL_ADDRESS>
remote: Repository not found.
fatal: repository 'https://github.com/RLovelett/eigen.git/' not found
fatal: clone of 'https://github.com/RLovelett/eigen.git' into submodule path 'third_party/eigen' failed
Failed to recurse into submodule path 'third_party/pytorch'

Certainly the eigen repository does not exist.
How do I avoid this error...?

Go into the third_party folder and do
git clone https://github.com/libigl/eigen.git
then run
git submodule update --init --recursive
That should install all the third party repositories

Thank you!


Reproduce of SUNRGBD with 30 and 37 classes

Hi,
Thanks for your great work! I am trying to reproduce the results on perspective (30 classes) and total (37 classes) SUNRGBD with provided checkpoints and configs. I use v3 for all the experiments.
I can reproduce the results of the votenet (10 classes) SUNRGBD with mAP 41.0.
bash tools/dist_test.sh configs/imvoxelnet/imvoxelnet_sunrgbd_fast.py pretrained_ckpt/20211007_105255.pth  8 --eval mAP
+-------------+---------+---------+---------+---------+
| classes     | AP_0.25 | AR_0.25 | AP_0.50 | AR_0.50 |
+-------------+---------+---------+---------+---------+
| bed         | 0.7176  | 0.8893  | 0.3710  | 0.4660  |
| table       | 0.3914  | 0.8313  | 0.0808  | 0.2351  |
| sofa        | 0.5149  | 0.8852  | 0.1373  | 0.3142  |
| chair       | 0.5366  | 0.8058  | 0.1637  | 0.3169  |
| toilet      | 0.7676  | 0.9172  | 0.3645  | 0.4828  |
| desk        | 0.2184  | 0.7806  | 0.0169  | 0.1562  |
| dresser     | 0.2120  | 0.7982  | 0.0248  | 0.1697  |
| night_stand | 0.3452  | 0.8392  | 0.0714  | 0.2745  |
| bookshelf   | 0.0565  | 0.4539  | 0.0009  | 0.0213  |
| bathtub     | 0.3405  | 0.8367  | 0.0562  | 0.1633  |
+-------------+---------+---------+---------+---------+
| Overall     | 0.4101  | 0.8037  | 0.1287  | 0.2600  |
+-------------+---------+---------+---------+---------+
However, the results of perspective (30 classes) and total (37 classes) SUNRGBD  are very low:
bash tools/dist_test.sh configs/imvoxelnet/imvoxelnet_perspective_sunrgbd_fast.py pretrained_ckpt/20211007_105254.pth 8 --eval mAP
+--------------+---------+---------+
| classes      | AP_0.15 | AR_0.15 |
+--------------+---------+---------+
| dresser      | 0.2233  | 0.8119  |
| painting     | 0.0053  | 0.1971  |
| bookshelf    | 0.2164  | 0.7766  |
| night_stand  | 0.4626  | 0.8275  |
| drawer       | 0.0608  | 0.7162  |
| cabinet      | 0.0789  | 0.6658  |
| shelf        | 0.0402  | 0.4317  |
| lamp         | 0.1762  | 0.6657  |
| garbage_bin  | 0.3334  | 0.7308  |
| box          | 0.0330  | 0.2742  |
| bed          | 0.7786  | 0.9320  |
| sofa         | 0.6390  | 0.9250  |
| sofa_chair   | 0.5459  | 0.9034  |
| pillow       | 0.1468  | 0.5900  |
| desk         | 0.3189  | 0.8730  |
| chair        | 0.6593  | 0.8881  |
| paper        | 0.0000  | 0.0000  |
| stool        | 0.1037  | 0.6804  |
| whiteboard   | 0.0099  | 0.2667  |
| picture      | 0.0083  | 0.2180  |
| endtable     | 0.2493  | 0.8241  |
| sink         | 0.3995  | 0.7962  |                                                                                | computer     | 0.0271  | 0.6526  |                                                                                | table        | 0.5120  | 0.9127  |
| toilet       | 0.8454  | 0.9517  |                                                                                | coffee_table | 0.3649  | 0.9320  |                                                                                | recycle_bin  | 0.5227  | 0.9059  |
| monitor      | 0.1986  | 0.6497  |                                                                                | keyboard     | 0.0001  | 0.0157  |                                                                                | cpu          | 0.0530  | 0.4492  |
+--------------+---------+---------+                                                                                | Overall      | 0.2671  | 0.6488  |                                                                                +--------------+---------+---------+
bash tools/dist_test.sh configs/imvoxelnet/imvoxelnet_total_sunrgbd_fast.py pretrained_ckpt/20211007_105247.pth  8 --eval mAP
+----------------+---------+---------+
| classes        | AP_0.15 | AR_0.15 |
+----------------+---------+---------+
| dresser        | 0.3536  | 0.9486  |
| cabinet        | 0.1910  | 0.8230  |
| night_stand    | 0.3855  | 0.8935  |
| lamp           | 0.1319  | 0.7016  |
| bookshelf      | 0.1635  | 0.8675  |                                                                                | shelves        | 0.0546  | 0.5224  |                                                                                | sink           | 0.4448  | 0.7989  |
| chair          | 0.6299  | 0.8869  |                                                                                | bed            | 0.7911  | 0.9541  |                                                                                | fridge         | 0.2329  | 0.9497  |
| sofa           | 0.6059  | 0.9624  |                                                                                | tv             | 0.1302  | 0.6500  |                                                                                | box            | 0.0276  | 0.2483  |
| mirror         | 0.0023  | 0.2533  |                                                                                | pillow         | 0.1110  | 0.5153  |                                                                                | table          | 0.5116  | 0.9310  |
| desk           | 0.3116  | 0.9118  |                                                                                | door           | 0.0075  | 0.4900  |                                                                                | clothes        | 0.0133  | 0.5200  |
| towel          | 0.0190  | 0.2989  |                                                                                | books          | 0.0013  | 0.1720  |                                                                                | bag            | 0.0053  | 0.3810  |
| counter        | 0.1479  | 0.8614  |                                                                                | window         | 0.0018  | 0.6786  |
| person         | 0.4273  | 0.8571  |
| curtain        | 0.0350  | 0.5959  |
| toilet         | 0.7656  | 0.9533  |
| picture        | 0.0014  | 0.1379  |
| whiteboard     | 0.0116  | 0.2819  |
| blinds         | 0.0026  | 0.3182  |
| bathtub        | 0.4356  | 0.9038  |
| paper          | 0.0000  | 0.0282  |
| shower_curtain | 0.0000  | 1.0000  |
+----------------+---------+---------+
| Overall        | 0.2107  | 0.6454  |
+----------------+---------+---------+
Could you please have a look at your convenience? Thank you!

Hi @Jeff-LiangF ,
Thanks for your interest in our research. If I remember correctly, for these 2 benchmarks we train on 30 and 37 classes, but evaluate on 10 and 10 following the original papers. So you can calculate mAP for only selected categories by yourself.

Hi @filaPro ,
I really appreciate your prompt response. I recalculate the mAP of the selected 10 votenet classes. The number is now correct! Thanks!

Btw, these sets of 10 classes are not the same in VoteNet, PerspectiveNet and Total3dUnderstanding.

Oops... will check! Thanks for the reminder.


BUG: mne watershed_bem relative paths

I recently ran mne watershed_bem which created the right files as subject/bem/..., but with absolute-path linking to /whatever/subject/bem/watershed/... rather than relative to watershed/.... This unnecessarily limits the share-ability of datasets.

This has been fixed


Small Typo on bot-builder-primitive-prompts.md

Javascript file extension incorrectly labelled as C# file extension

Fixing in private repo

Fixing in private repo


Update header.html

set the landmark of navigation_logo to nav;
set the value of navigation_logo to {{ settings.base.SiteSettings.title_suffix }}.

Closing as invalid


Check FUSE installation and display proper message

Is your feature request related to a problem? Please describe.
Check error when FUSE is not installed
Describe the solution you'd like
Check if FUSE is available on system.
Display proper message to user where to find installation of FUSE for system.
Describe alternatives you've considered
No response
Additional context
No response


Addon Export Incorrect on Reforging

Describe the bug
I used the addon to get my gear into the simulator and there may be other mistakes but it messed up the reforge on my pants. Here is the text:
{"talents":"0300000000000000000-2302000000000000000-23203003023022121311","glyphs":{"minor":[{"spellID":57903},{"spellID":57866},{"spellID":57870}],"prime":[{"spellID":63067},{"spellID":63066},{"spellID":56841}],"major":[{"spellID":56833},{"spellID":56857},{"spellID":56829}]},"class":"hunter","unit":"player","race":"Orc","name":"Muddshotts","gear":{"items":[{"id":64710,"gems":[41398,40112],"reforging":166},{"id":57931,"gems":[]},{"id":64712,"gems":[40112],"reforging":154,"enchant":4205},{"id":56315,"gems":[],"reforging":144},{"id":56281,"gems":[52082,52102]},{"id":63479,"gems":[],"reforging":151},{"id":58125,"gems":[52175]},{"id":57916,"gems":[40112]},{"id":58122,"gems":[52142,52102],"reforging":154},{"id":62385,"gems":[52082],"reforging":166,"enchant":4076},{"id":62348,"gems":[],"reforging":166},{"id":56310,"gems":[],"reforging":167},{"id":52199,"gems":[],"reforging":137},{"id":63840,"gems":[],"reforging":166},{"id":65168,"gems":[],"reforging":167,"enchant":4227},null,{"id":56376,"gems":[],"reforging":130,"enchant":4176}]},"professions":[{"name":"Engineering","level":525},{"name":"Jewelcrafting","level":476}],"level":85,"spec":"survival","realm":"Whitemane"}
Pants are haste to hit but come through as haste to mastery

To Reproduce

Export my gear from game
Import to sim

Expected behavior
Expect it to copy my gear in correctly with the right reforges
Sim Links and Screenshots
(https://wowsims.github.io/cata/hunter/survival/#eJzlVF1oFFcUnnPu7O7dE3VvrkkdL6EmY2vXxdiZ2Q1sg7hjnlL74qO+KWiQqpBmXvQt/iAxVAgqjU1fNIotBaUtFCVtoYjE1FY2RFArhZaiRIpiCaVGpNB7Z3YxNg/+PfYww577ne9859xz7yy9yUFACNuhFwYAjgN8BvA9wE2AuwD9CAMIXyB0ZTl0JxRpqeMNIyy9YeeWPdv6eEag+gaoQYw/Yi5aVv4MkC1+vMRooZh4xJz7rS5Y+RGgjHh8nuWHgBaIn88zNz3zETs4zIjEwwfMUIZB+4fHmWtXh5lFXHx7KRaknBjUaPrcsOEbpZz4cpo595a5TGuYehnx4XTN+VvX+NQ405p7yDi/zcQhEkP/MGd/m4laenXhAnMmW/P7wF3I0WvegBtxc2oIoHAWaRas/6lJ70UzOq0u7IVB8Gg1LRfLVBNHAWUWQjfbznabCzME2JZw/6x40Hj1J2P3KmO4xit6/7X2oOgF87HAUHVEP4Ef+EXfv4hv8NN3UJy6g86RKrYeq2L+aBW9T6pYvjGF4ZUp7L4+hRNoT2LmFlq/40TKAZckJ9QXwLyLKaNSxPhfO1xOabLbg45Ig2llEzqNbpYylGr3V5ciaiauNIFf/tWeC9cF7vfUBHwvIlkD/+CuJhJrLz7JPzzKamApckC+pppIFBbxlMROyxAI/ajTknm1glqUotQVRA7UWMhxW+J6S8W5ukvDFSfHmXQ4aI0G01Rgio7l4trXmmLtefhYTpJKWpk+weQBUHuBIvkB8cBgX59gtKpQ4GnJ36+tVctXuJSDzH6OdYRIb9bX57IrouwIzE20nyexQye6sU92h78rkl0qJF++TargcKYT6pIqHnVg+PEMUpIdsWtzKpX6Gig7WqfK9aqb1qo1tMAoP7UZ9rybUXWtleotel22xEeTjucelyxHxEfBjPFWi0p+pDkl01p6TmtlL47eXiLfU+9SKNfOY5S8WNt+SntJoVmT5uw+DhSjulj94PQdvItb97LkkyqHJ2vepnCQWe4vmI1X/T1hY4I3hUtHPjb2Q+WdBJmstPXEdr2yrv41BqewdaOTKgzgK/1/PMvyh8KXSevfevml8l7FkmF99yBcl6xnK5vhX27H80k=)
Desktop (please complete the following information):

OS: Windows
Browser: Edge
Version: Unknown

Additional context
N/A

Shoulders are wrong too:

Duplicate of #530
You can track the progress of the fix for this with the linked issue.


[LoongArch64] Fix the issue of incorrect printing information in the gcroot command

Adjust the offset values of SP and IP as well as the contextSize value in the EnumerateSackTrace function.

@shushanhf @mikem8361 Could you please review this PR？


use https for git submodules

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


Proposal for a Docker and Linux-based Weak Network Simulation Feature

@mhils
Problem Description
I have been working on a feature to simulate weak network conditions. Implementing a native weak network simulation that is cross-platform compatible has proven challenging. The feature is currently only viable in Linux environments or via Docker for non-Linux systems.
Proposal
I've developed a solution that functions well within a Docker or Linux environment. While it may not be the ideal cross-platform solution, it does provide a functioning weak network simulation for non-Linux platforms through Docker.
Benefits
The current weak network simulation combined with the WireGuard, Sock5 features can simulate weak network conditions for all types of traffic, including video calls, chats, as well as common TCP and UDP requests, etc. This is highly user-friendly for testers and developers, and it is applicable to all modes of mitmproxy.
Alternatives
I have considered finding a more universal method to simulate weak network conditions across various platforms. However, the technical challenges and potential inconsistencies might outweigh the benefits.
Additional context
This feature could prove particularly useful for testing how applications handle weak network conditions. However, its current limitation to Docker or Linux environments may restrict its wide applicability.
I would appreciate your feedback. Would you still be interested in this PR?
Best,

Hi there!
If you have a draft PR ready, please just send it. It's easier to talk about concrete code (even if not finished/polished) than hypothentical changes.
Generally speaking, mitmproxy's current design does not lend itself very well to simulating bad network conditions. Most proxy modes are based on regular sockets, so it's hard to drop or delay individual packets. With WireGuard mode it's maybe a bit different, but I feel that could/should maybe be a completely separate project? Again, if you have a simple proof of concept let's look at that and take it from there. 😃

Awesome, thanks. Feel free to just dump non-working stuff in a PR that illustrates what you are doing, I don't need a polished PR to evaluate. :)


[Feature] ENAButton activity indicator for submission flow

This PR implements an activity indicator for the ENAButton. The indicator is applied to the submission flow loading states.
The PR incorporates commits from the original PR #439 and builds on #752.

When clicking on the "Aktualisieren" button in the pending Test Result screen, the activity indicator slides in from the left into the button. Is this intended behavior? it seems quite odd.


Possible to document structs and enums?

We'd like to have structs as part of the generated output.  Is this possible?

Should be possible. Take a look at this test template: https://github.com/OpenZeppelin/solidity-docgen/blob/master/fixtures/struct-enum/templates/contract.hbs

Closing as this is currently supported, shown in the link above.


Add Customizable Parameters to Charts With Dropdown

Currently, the first two charts in the non-profit view are only able to display one set of variables. Text above the chart details those variables.

Replace text above the chart with a drop-down menu where users can select from a list of variables. Switching a variable re renders the chart with new data.
Example:

What variables do you what them to be able to select from?

How about starting with revenue, expenses, assets, and liabilities?


Metrics endpoint should return 404 when there are none

Right now the metrics endpoint returns a 500 if there are no metrics. It should return a 404.

@Vafilor What about returning a "204 No Content" instead? The API url is valid, just no data.

@aleksandrmelnikov I was thinking about that.
The issue is we might generate metrics at the end. Does it make more sense that you have no metrics or that they are not found, and later you do have metrics.
@rushtehrani Thoughts?

If they can be at the end, I think it makes sense to use 204 in the middle.

404 means, to me, the url is invalid. Which is not the case if the metrics are in the middle and not yet generate.
204 means yes, valid, but no content (yet)


Observers, computed, decorators and node.style do not get along

Was doing some investigating about all the weirdness that happens using the model suggestion from #2166 (I think these two bugs are related so I'm putting them in one).

http://plnkr.co/edit/DsE1vzrjTUL078IzQ97s?p=preview

Hit the "move" button then hit the  "delete" button. Notice that the "three" box is now slowly animating when it should never have gotten an "x" value. This doesn't happen for the "direct" approach. I think Ractive is recycling the DOM node? If you run app.findAllComponents()[1].get() the model object still has a name of "two" (it didn't get deleted?), but if you run app.findAllComponents()[1].get('model.name')it returns "three". :confused:

http://plnkr.co/edit/N4GEj1KpvKFv3NDcPlyb?p=preview
Same plunk, but the only difference is that I've removed the entire style="left:{{x}}px" attribute on the component div and added an observer to change the color green when x changes.
In this case all of the computed nodes turn green when only the second item should be changing. It works correctly for the direct nodes.

This doesn't happen for the "direct" approach. I think Ractive is recycling the DOM node?

Correct. When a new array is "set" Ractive reuses the dom nodes. When the array is "sliced` (or any equivalent) the corresponding DOM nodes are removed.
So the "direct" case gets sliced and those (more minimal) changes are propagted selectively (remove node TWO). But the computed property is more coarse as Ractive treats the entire array as having changed.
Rative could call merge on computed properties of type array that change, but there's no way to specify merge comparator options.
Generally, I think best bet is to move filtering logic into the template, rather than returning a whole separate array via computed properties
Alternatively You can use an observer instead and then use merge: http://plnkr.co/edit/DAE68GKRZc7OhGR64dLZ?p=preview
I'm removing the bug label in favor of better docs, because I don't see anything not working in a way that we would change it, though arguably it's not as expected.

Generally, I think best bet is to move filtering logic into the template, rather than returning a whole separate array via computed properties

That's the whole point of computed properties for me though: to move that logic outside the template because it's too complicated. I usually end up with things like filtering, sorting returning default values and a slew of other things in a computed prop's function and a simple {{# items}} in the template. But ractive does not play nice with computed arrays in these cases.

              this.observe('comps', function(comps) {
                  this.merge('items', comps.slice());
              });

Totally would never have thought to do it that way, but it seems to work much better than computed. Now I'm going to have a bunch of observers instead of nice clean computed props :confounded:

Now I'm going to have a bunch of observers instead of nice clean computed props

I've been thinking about the array case as well, that's why I mentioned:

Rative could call merge on computed properties of type array that change, but there's no way to specify merge comparator options.

Could be added to the property same way we do get and set:
    computed: {
            items: {
                base: 'comps',
                filter: function(comps){
                    return comps.filter(...);
                }
            }
     },

Just a strawman API, but could also potentially allow binding to individual items of array.

Would there be no way to correctly diff the computed array from its previous value?
My perception has been that any time an array changes, ractive would be able to figure out the differences and smartly update the dom as needed. That's probably why I've literally never used merge before, since I just assumed that ractive did some black magic to figure out the differences after a set.

0.9 seems to be behaving much better. The only weird part is the green decorator sticking around on the second element, but that tracks with what is supported right now. I'm going to punt the discussion of better sorting to #2599 and #2947


React Virtualized requires mouse movement

I have tried applying react-sortable-hoc to a react virtualized grid.  When I drag something and then pause, all of the cells take up the area left by the old cell and no longer move.
jfrolich's comment here: https://github.com/clauderic/react-sortable-hoc/pull/326
demonstrates the same behavior

I did some basic analysis why this happens, but couldn't get to the root cause quickly. Would be great to get some pointers from someone more familiar with this library. It's probably a minor thing because initially it works as expected.


[FR] hide preview #6825

This commit adds functionality to hide and show the preview like requested in #6855

Grunt validate fails with two tests, but I can't figure out why.

@sakulstra I know why this isn't working. There's a unit test for the gh-editor component that needs your new gh-preview-toggle component to be able to run correctly. I'd suggest switching that unit test (core/client/tests/unit/components/gh-editor-test.js) to an integration test (would go in core/client/tests/integration/components/gh-editor-test.js)

Rebased and ported this to the new repo https://github.com/TryGhost/Ghost-Admin/pull/5 , thanks for your help!


[ISSUE-27] Fix EFS Storage Class Mismatch with Name Prefix

This change fixes a template issue when using AWS EFS as the storage for self-hosted deployments

LGTM! This shouldn't affect functionality, but it does keep consistent naming across Deepgram resources.
Could you add a line to the CHANGELOG.md under a new header called "Unreleased" describing the change in the specified format there?

I'll get that added.  This was impacting functionality in our cluster as we are using Karpenter Autoscaler; the name mismatch prevented karpenter from scheduling the engine pod, so it wouldn't spin up a new node for it.

That's good to know! This hasn't been explicitly tested with Karpenter, so I wasn't aware of that. Thanks for the heads up!
In that case, if you have autoscaling enabled to include the HPA manifests, make sure to explicitly disable the cluster-autoscaler dependency.

Thank you! I'll cut a release for this later today.
Thanks for your patience on the Changelog request. The other open PR in this report will include a few more CI checks to help guide PR submissions to include that 😀


Out of bounds Int8 when recognizing chords.

macOS Version(s) Used to Build
macOS 13 Ventura
Xcode Version(s)
Xcode 14
Description
When recognizing a chord as shown below:
/// INSIDE MY PROJECT

func recognizeChord() -> Chord? {
    var currentPitches: PitchSet = PitchSet()
    for (keyID, pressed) in midiHelper.MIDIKeysDown {
        if pressed {
            currentPitches.add(Pitch(intValue: keyID.clamped(to: 0..<128)))
        }
    }
    let rankings = Chord.getRankedChords(from: currentPitches, allowTheoreticalChords: true)
    return rankings.first
}

The program hard crashes with an out of range exception for the Int8
/// INSIDE TONIC

public var noteNumber: Int8 {
    let octaveBounds = ((octave + 1) * 12) ... ((octave + 2) * 12)
    var note = Int(noteClass.letter.baseNote) + Int(noteClass.accidental.rawValue)
    if noteClass.letter == .B && noteClass.accidental.rawValue > 0 {
        note -= 12
    }
    if noteClass.letter == .C && noteClass.accidental.rawValue < 0 {
        note += 12
    }
    while !octaveBounds.contains(note) {
        note += 12
    }
    return Int8(note) // <--- Error Here
}

The error occurs above D7 (MIDI Key 98).
Crash Logs, Screenshots or Other Attachments (if applicable)

We've overhauled the chord stuff in V2. Hopefully that addresses this issue. If it does not, please write a test that shows the problem.


It's consoled too many logs.How to shut down it?

👋 friend. Welcome to styled-jsx and thanks for contributing!
⚠ IMPORTANT ⚠
If you need help or have a question about styled-jsx please ask it on Spectrum https://spectrum.chat/styled-jsx or join our Slack community at https://zeit.chat and ask it in the #next channel.
Before you open a new issue please take a look at our Frequent Asked Questions and open issues.
Remember, often time asking in chat or looking at the FAQ/issues can be faster than waiting for us to reply to a new issue*.
If you are here to report a bug or request a feature please remove this introductory section and fill out the information below!

Do you want to request a feature or report a bug?
What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce and possibly a minimal demo or testcase in the form of a Next.js app, CodeSandbox URL or similar

What is the expected behavior?
Environment (include versions)

OS:
Browser:
styled-jsx (version):

Did this work in previous versions?

duplicate of #480


Location Based on County

As a platform user, I want to enter the county, along with the state to get a more accurate search results based on my location(county).

Investigation: Size 1

Ready for PO Review

County is the same as zip code. I entered "Kunia" for county, changed location, and no resources showed up.

@KellyeDC , The same reason as in #971. Could you please provide valid data for the topic entities, so we can retrieve that based on County? Thank you.

As discussed the location should be mapped to resources not topics.  Please let me know if you require further clarity.
Kellye Bennett
Legal Navigator Project Manager
Legal Services Corporation

Office of Information Technology
3333 K Street, NW
Washington, DC 20007
P:<PHONE_NUMBER>
M:<PHONE_NUMBER>
E<EMAIL_ADDRESS>
From: yevhenobertas<EMAIL_ADDRESS>Sent: Thursday, October 10, 2019 9:47:17 AM
To: LSCOIT/LegalNavigator<EMAIL_ADDRESS>Cc: Kellye Bennett<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [LSCOIT/LegalNavigator] Location Based on County (#970)
@KellyeDChttps://github.com/KellyeDC , The same reason as in #971https://github.com/LSCOIT/LegalNavigator/issues/971. Could you please provide valid data for the topic entities, so we can retrieve that based on County? Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/LSCOIT/LegalNavigator/issues/970?email_source=notifications&email_token=AK5GT5DA5Y5DJIDFJHKJY2TQN4W6LA5CNFSM4IJN3SC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEA4MNUY#issuecomment-540591827, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AK5GT5CACW7DGCGNUGQ7X63QN4W6LANCNFSM4IJN3SCQ.
[X]
Caution External Email Alert!
This message was sent from someone outside of LSC. Please do not click on links or open attachments unless you recognize the source of this email and know that the content is safe. If you believe that this email is a phishing attempt or contains malicious files or offensive content please contact the OIT HelpDesk.
OITHELPDESK mailto:oithelpdesk.lsc.gov

@KellyeDC , Fixed in the development environment


Return supported instance types as list

The get_supported_instance_types function has been changed in commit #e717cb36 to return a set but this breaks integration tests with awsbatch. Fixed to return a list as before.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Actually the change happened in commit #5a6250f.


Renamed files and variables to a common standard (Completed)

Renamed files and variables to a common standard (completed for samples: group-scaling-v1, more-complex-app, nested-group, single-group-group-scaling, single-group-v1, single-group-v2, single-group-v3)

Thanks @dinithis for your contribution, I see following conflicts in this pull request:
CONFLICT (content): Merge conflict in samples/applications/single-group-v2/artifacts/mock/deployment-policy.json
CONFLICT (content): Merge conflict in samples/applications/nested-group/artifacts/openstack/deployment-policy.json
CONFLICT (content): Merge conflict in samples/applications/nested-group/artifacts/kubernetes/deployment-policy.json
CONFLICT (content): Merge conflict in samples/applications/nested-group/artifacts/ec2/deployment-policy.json

I'll look into this.
On Fri, Feb 6, 2015 at 11:39 PM, Imesh Guaratne<EMAIL_ADDRESS>wrote:

Thanks @dinithis https://github.com/dinithis for your contribution, I
see following conflicts in this pull request:
CONFLICT (content): Merge conflict in samples/applications/single-group-v2/artifacts/mock/deployment-policy.json
CONFLICT (content): Merge conflict in samples/applications/nested-group/artifacts/openstack/deployment-policy.json
CONFLICT (content): Merge conflict in samples/applications/nested-group/artifacts/kubernetes/deployment-policy.json
CONFLICT (content): Merge conflict in samples/applications/nested-group/artifacts/ec2/deployment-policy.json
—
Reply to this email directly or view it on GitHub
https://github.com/apache/stratos/pull/216#issuecomment-73284624.

--
Dinithi De Silva
Associate Software Engineer, WSO2 Inc.
m:+94716667655 |<EMAIL_ADDRESS>| w: www.wso2.com
| a: #20, Palm Grove, Colombo 03