Questions about modifying image in sensor

CARLA version: 0.9.15
Platform/OS: windows11
Problem I have experienced: I want to modify the image from sensor and make some attacks on the vehicle in CARLA, but seems that the attribute of image cannot be changed. So I tried generate a carla.Image object, but it was still failed.
What I expected to happen: I'm trying to do some black-box attack in CARLA to observe the influence on camera sensor, hopefully I can generate a new carla.Image object and replace the orignial one, or I can just modfiy the attribute of the orignial image directly. So that the image perceived by sensor would be changed, and I can implement the black-box attack in CARLA.

I have reviewed your request, and I want to confirm if I understand it correctly: you want to perform image attacks on visual sensors, similar to adversarial examples. The result of the attack is that the perception/control algorithms receiving the image sensor data will behave differently.
Therefore, your requirement is that the visual sensor can generate/transmit images altered by the attack algorithm based on your algorithm. If that is the case, you can implement your algorithm and modify the image data in the file Unreal/CarlaUE4/Plugins/Carla/Source/Carla/Sensor/PixelReader.cpp within the CARLA project path. If you want to achieve the alteration during the rendering process, you can modify the Unreal Engine code.

Of course, you can also add a callback function containing the tampering algorithm in sensor.listen() in the PythonAPI to achieve the tampering effect.
Hope this helpful.


Use CCS Clusters and enhance results persistent storage

Work In Progress
Changes:

Use CCS Clusters during the 'create cluster' test
Added a Dockerfile and Entrypoint to easily run the entire test-suite in a container (docker/podman/k8s)
Added functionality which writes the results to ElasticSearch

This is looking awesome!

Closing this Pull Request as #38 provides a better implementation.


Add sample_new parameter to gendb::incorporate

This is required for GenDB to replicate existing pclean behavior (via sample_new=False) and unblock https://github.com/probcomp/hierarchical-irm/pull/212.

If you want, I can push another commit to this branch that avoids walking the DAG and plumbing the bool arg through.
If it's ok with you I'd like to merge #217 before this (and #212) -- I'm happy to help with merge conflicts, just I'm scared of breaking the Gibbs sampler and that seems like an easier order to do it in.

If you want, I can push another commit to this branch that avoids walking the DAG and plumbing the bool arg through.
If it's ok with you I'd like to merge #217 before this (and #212) -- I'm happy to help with merge conflicts, just I'm scared of breaking the Gibbs sampler and that seems like an easier order to do it in.

My preference would be for you to replace the bool arg plumbing in a separate pull request.
#217 seems like it is still rather far from being checked-in.  My preference would be to check this and #212 in first.

Actually the merge conflicts weren't as bad as I thought so we can check in either first.


Trade - token search

current:
searching a token that does not exist, show blank area.
updated:
https://www.figma.com/file/YpsatcY9jtxJr0SJwlFYNj/Carbon-Screen-Design?node-id=1894-37463&t=UXSWUcYppXN2fNjU-4
IF search by token symbol cannot find a token, suggest searching by address.
IF search by address cannot find a token, suggest creating a strategy -> on click, open create strategy with "base" as the address used by the user.

@ashachaf Can this issue be closed?
"Create a strategy" when selecting a pair with no orders is already suggested in the trade page

Searching by address is now suggested too on pages other than the trade page


Feature/services to hooks

Moved user and scenarios hooks from plain functions calling useQuery to hooks.
I don't think it's needed to do this for the rest as they are only calling UseQuery without any transformation or parameter.
Let me know if they should be changed.
I couldn't test the user function as it is not called yet. Maybe if we have some more time we could do unit tests for the hooks and services.

For now, the queries related to user authentication are not needed. Also, this is not completed in the API. I think everything should be in the same way to keep the consistency because it could be confusing for new developers to understand the criteria.


Poland mobility reports

I tried to follow your code to extract data for Poland but unfortunately the getgraphs.py does not extract graphs from the main svg plot. It works for other countries (i.e. I have checked G20 files that you provided) but not in my case.
To Reproduce
I attached the pdf file along with zipped svg file generated by Affinity Designer 1.6.5.
2020-03-29_PL_Mobility_Report_en.pdf
poland.svg.zip
Desktop (please complete the following information):

OS: MacOS Catalina (10.15.3)

Thank you @BERENZ we will look at this as soon as we can.

@BERENZ when I convert the PDF to SVG it all works ok. It must have changed the default Affinity Designer settings somewhere (I've been using it for a while, so that's not surprising). Below are the screenshots of the loading of the PDF and saving of the SVG. There are no steps in between. Could you compare to your settings so I can isolate the issue and add any non-default settings to the guide please. We also found that it works with the latest Affinity Designer, but the SVG saving option is (for digital). I have also attached my document SVG for reference.
Load

Save

SVG
Poland_dsc.svg.zip

@BERENZ and yes, I originally tried with your SVG, and it didn't work. So it must be something to do with the SVG creation. This is a little temperamental at the moment. We wanted to use an automated method like pdftocairo but the matrix transformations were crazy - and each SVG creator seems to choose it's own change to the stroke-width of the lines, and whether it uses HEX values, RGB % etc.

@mshodge thank you for prompt answer. I'll check the file and Affinity, thanks!. I also tried pdf2svg but it create separate svg for each plot in the pdf.

That may work with our code using -m in getgraphs.py I'll test it out too as Affinity throws up some other bugs like #5 for us at the moment.

@mshodge I confirm that the codes works smoothly, thank you!

No problem, thanks @BERENZ. We aim to make the pipeline better soon with pre-made SVGs to eliminate the errors (I've spent a day working with transformations and it's a killer)


Add Hector SM to Contributors list

Hi  @ZhectorSM, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.

Hi, i think the tutorial is very clear , it explain step by step and the images are very helpful.
I felt so exited about the pull request and merge beacuse I never had done that, I know git ,but this working form is diferent for me.
I will be happy to visit your page. Thank you for your support.

I’d urge you to continue contributing to other open source projects.
We also have a slack team to help/mentor people trying to contribute to open source projects.
You can join the team by going to https://goo.gl/PM1Jdh


machine: raspberrypi-64: Add tar.gz to IMAGE_FSTYPES

This is for NFS boot when LAVA boot test is performed.

Thanks!


SO3_fft_real and SO3_ifft_real do not seem to be inverses of each other?

Hello
I did a simple experiment where I created a random signal x on SO(3) (I tried multiple bandwidths = 20,30,40). Then I computed the FFT using
x_fft = SO3_fft_real.apply(x)
Finally, I computed the IFFT using
x_hat = SO3_ifft_real.apply(x_fft)
I found that x and x_hat are very different from each other and appear to have no patterns that I can identify. For example, one is not a scaled version of the other.
Any help here is appreciated, It's possible I am just using the functions incorrectly.
Thanks
Suhas

A random signal contains a lot of high frequencies but these high frequencies cannot be preserved by the transformation.
What I  usually did to make a test is to take the x_hat that you got and do the experiment again with him

Hi Mario
Thanks for the reply, you were exactly right. Redoing the experiment with x_hat does show that the transformations are invertible.
(I also found that if I start with a signal x with zeros everywhere except one location where it is 1.0, the same problems persist, again explained by the fact that there are too many high frequencies.)
However, I obviously do not observe these with when these signals are defined on the real line (rather than the sphere) and I use the usual 1D FFT and 1D IFFT. I was wondering if you think it is this particular implementation of the FFT on SO(3) that leads to this behavior.
Another thing I noticed which I think may be related to the same problem and could be useful to others as well. (I can post it as a separate issue if you think it is a different topic.)
I took a spherical MNIST image using gen_data.py, x. I created x_hat using S2_fft_real() followed by S2_ifft_real() so as to avoid any problems with the high frequencies. (x_hat is actually quite close to x).
Then I computed the spherical auto-correlation using
x_hat_fft = s2_fft_real.apply(x_hat, 10)
auto_corr_fft = s2_mm(x_hat_fft, x_hat_fft)
auto_corr = SO3_ifft_real(auto_corr_fft)
However, when I look at self_corr, there seem to be some issues:

As the two signals are the same, they are already aligned and we expect the highest correlation to be exactly at alpha = beta = gamma = 0. And this seems to be indeed true or very close to it.

At alpha = beta = gamma = 0, the value of the spherical correlation should be equal to the square of the L2 norm of x_hat (right?). However, the values are 4 orders of magnitude smaller and it is not a constant scale factor that applies to different runs using different x_hat's.

I also tried to correlate x_hat with a Dirac function at 0,0 defined on the sphere. Again, the spherical correlation results are not interpretable as far as I can see.
This has stumped me and I am hoping for some resolution.
Thank you so much again for the great support even now after such a long time since publication
Suhas

It should work... and you share the code please?

This is the test code I am using:
import numpy as np
import torch
import torch.nn as nn
from s2cnn import SO3Convolution
from s2cnn import S2Convolution
from s2cnn import so3_integrate
from s2cnn import so3_near_identity_grid
from s2cnn import s2_near_identity_grid

from s2cnn.soft.s2_fft import S2_fft_real, S2_ifft_real
from s2cnn.soft.so3_fft import SO3_fft_real, SO3_ifft_real
from s2cnn.s2_mm import s2_mm

import gzip
import pickle
import ipdb
import scipy.io

MNIST_PATH = "s2_mnist_train_nr_test_nr.gz"

with gzip.open(MNIST_PATH, 'rb') as f:
	dataset = pickle.load(f)

train_data = torch.from_numpy(dataset["train"]["images"][:, None, :, :].astype(np.float32))

### Get one image for testing and convert to batched version
image = train_data[0,0,:,:].unsqueeze(0).unsqueeze(0)
image = image/255.0

### Removing high frequencies if any (following what Mario Geiger suggested on Github)
image_fft = S2_fft_real.apply(image)
imagehat = S2_ifft_real.apply(image_fft)

### Try auto-correlation with the reconstructed image, auto_corr should be equal to torch.norm(imagehat)**2 at alpha = beta = gamma = 0.0, where it should also attain its maximum

imagehat_fft = S2_fft_real.apply(imagehat,10)
auto_corr_fft = s2_mm(imagehat_fft, imagehat_fft)
auto_corr = SO3_ifft_real.apply(auto_corr_fft)

print(torch.norm(imagehat)**2)
print(auto_corr[0,0,0,0,0])```

For the particular image I am testing I get torch.norm(imagehat)**2 = 241.2491 and auto_corr[0,0,0,0,0] = 0.0377. As an aside, [0,0,0] is indeed very close to where auto_corr attains its maximum.

I don't have s2cnn installed anymore so I cannot check myself but everything sounds good so I don't clearly understand why it does not work. Did you check that the shape of the tensor provided to s2_mm is good? Reading again it's docstring made me wonder if the two argument are really symmetrical (it's a pity that it was written in that non symmetric way).

Also try to compute imagehat also using b_out=10, maybe you still have some high frequencies that makes the mismatch

Another think, you asked why Fourier transform are exact on the plane but not on the sphere. I think it's because there are no regular tiling of the sphere.

Hi Mario
Thanks for the suggestions.

Yes, the inputs sizes for the s2_mm seem fine as per the documentation (there are no syntax errors either, obviously).
Could you explain by what you mean by s2_mm is not symmetric?
Do you have any test codes that you used to test correctness of the SO(3) and S2 convolutions that you can share?

Nothing else I do has helped so far.
Thank you again
Suhas

It would also be great if you could point me to a clear formula for what s2_mm is accomplishing. I can try to modify it if that is the one causing trouble.
The equivariance plots provided in the repository, provided as an example, suggest that everything should work as expected. Just not the code I am trying to run, which is really weird.

https://github.com/jonas-koehler/s2cnn/blob/master/s2cnn/s2_mm.py#L30-L58
in pseudo code, it does the following
x and y are lists of tensors indexed by L from 0 to Lmax

for x_L, y_L for zip(x, y):
    # x_L is a tensor of shape [2L+1, batch, i]
    # y_L is a tensor of shape [2L+1, i, j]

    output.append( make a (2L+1)x(2L+1) matrix and contract the i index )

return output

"However, I obviously do not observe these with when these signals are defined on the real line (rather than the sphere) and I use the usual 1D FFT and 1D IFFT. I was wondering if you think it is this particular implementation of the FFT on SO(3) that leads to this behavior."
This is ultimately due to the fact that we don't have a regular grid on the sphere or SO(3). The S2 and SO3 FFT/IFFT maps are only one-sided inverses. However IIRC there should be the option to compute the FFT up to a higher order, in which case you will get an increasingly accurate reconstruction. This page has some good info: https://www-user.tu-chemnitz.de/~potts/nfft/. Have a look at the papers by that group as well.
"At alpha = beta = gamma = 0, the value of the auto_corr should be equal to the square of the L2 norm of x_hat (right?). However, the values are 4 orders of magnitude smaller and it is not a constant scale factor that applies to different runs using different x_hat's."
This depends on how you normalize the basis functions. You could make them unit norm (int_G f_ij(g)^2 dg = 1), or make the matrices f(g) orthogonal, or something else. I don't quite remember what we choose to do, but it's probably something that makes gradients flow nicely and avoids numerical instabilities. It might even be that this scaling depends on frequency l. Not entirely sure what else could explain the lack of a single scale factor. Could you check if s2_mm is bilinear and fft / ifft are linear?
"It would also be great if you could point me to a clear formula for what s2_mm is accomplishing."
It is doing a batched block-wise matrix multiplication, with the blocks of dim 2l+1 for l=0...N, being flattened and stacked into a big vector for the input and output. Not sure if a formula would be simpler than the code itself: https://github.com/jonas-koehler/s2cnn/blob/master/s2cnn/s2_mm.py. It is made for applying a bunch of filters to a minibatch of data, so only one of the inputs has a batch dim.

Hi Taco
Thank you very much for these clarifications.
Yes, s2_mm is bilinear, and both S2_fft  and SO3_ifft are linear.
I did try to play with the normalization factors a little, but could not get it to work. It appears that for the FFT functions, S3.quadrature weights are multiplied to the Wigner matrices. For the IFFT functions, the Wigner d-matrices are multiplied by 2L+1. I tried removing these weights, but that does not help.
I don't fully follow why these weights are present, I feel that is one possible reason why the results are not as expected.
Suhas


Currently tested stable environment still up to date?

Hi,
is the currently tested stable setup still as stated in the README?

This code was tested on Ubuntu 18.04 LTS using TensorFlow 2.3.1 and Python 3.6.9.

Thanks.  ktrain  should work on newer versions of Python and Tensorflow on Linux and Windows.  If you have trouble with a particular OS or Python/Tensorflow version, please do open an issue and let us know.


Correctly use launch profile name from model annotations

Fixes https://github.com/dotnet/aspire/issues/1000

One will need DCP 0.1.45 (just added into the workload) for this code to take effect, but one can run with this change and DCP 0.1.44 without any adverse effects.


Make buffer module stand-alone

Many other projects might benefit from the buffer API in Netty.  To better enable this, the buffer module should be made as lightweight as possible, preferably stand-alone.
We'll need to investigate whether this is feasible and what would need to change in order to facilitate the change.
@normanmaurer @Scottmitch  FYI

:+1: Although I thought it is standalone (no deps)

as long as "no deps" equals "depends on netty-internal" I think it's a realistic goal

Just looked at the size of buffer and common modules ... they total nearly 1MB (355,607B, and 580,552B respectively as of 4.0.1.Beta7).  It's probably worth breaking things down a bit so that other projects can depend only on the parts they need.  One obvious way to start down this path would be to define a buffer-api module.  I'll start putting some thought into this.

Not sure this is really worth doing ATM ... closing for now.

+1

Am 27.01.2016 um 17:57 schrieb Nathan Mittler<EMAIL_ADDRESS>Not sure this is really worth doing ATM ... closing for now.
—
Reply to this email directly or view it on GitHub.


SelectionMerge / BranchConditional / Branch

Hi,
I have been playing around with optimizing SPIR-V code and stumbled upon the following problem:
My SPIR-V:
					...
              11:    10(bool) ConstantFalse
              14:      6(int) Constant 2
              16:      6(int) Constant 1
					...
      20(result):     19(ptr) Variable Output
	      25:             TypePointer Function 6(int)
					...
         4(main):           2 Function None 3
               5:             Label
        28(temp):     25(ptr) Variable Function
                              SelectionMerge 31 None
                              BranchConditional 11 30 31
              30:               Label
                                Store 28(temp) 14
                                Branch 29
              31:             Label
                              Store 28(temp) 16
                              Branch 29
              29:             Label
              21:      6(int) Load 28(temp)
                              Store 20(result) 21
                              Return
                              FunctionEnd

SPIRV-Cross compiles this to:
out int result;
void main()
{
    int temp;
    if (true)
    {
        temp = 2;
        int _21 = temp;
        result = _21;
    }
    temp = 1;
    int _21 = temp;
    result = _21;
}

Note that the SPIR-V in Block 30 branches to Block 29 and NOT to Block 31. (This is valid as far as I know, please correct me if I am wrong)
So what I would expect:
out int result;
void main()
{
    int temp;
    if (true)
    {
        temp = 2;
        int _21 = temp;
        result = _21;
	return;
    }
    temp = 1;
    int _21 = temp;
    result = _21;
}

cheers,
Reveillios
FYI:
I am working with SPIR-V for my master's thesis and my setup looks like this:
GLSL ⇨ [glslang] ⇨ SPIR-V ⇨ [SPIRV-Tools(optimizer, validator)] ⇨ "opt SPIR-V" ⇨ [SPIRV-Cross] ⇨ GLSL

This looks a bit odd ... Do you have the original .spv file?

I attached the .spv
Note that I did an Inline Function Pass on the SPIRV module to get to this "odd" branch.
test.zip

Ok ... this is indeed a bit tricky.
The problem here is that block 30 is branching beyond the merge target of 31, so the code-gen will not think this "closes" the if bracket. It can't, because this is effectively a goto. :\
The equivalent codegen would have to be:
if (true)
{
    tmp = blah;
    goto %29
}
31:
   tmp = blah;
29:
  result = tmp;
  return;

We now get two independent streams which both will "inline" the code for block 29.
if (true)
{
    tmp = blah;

    // 29 is duplicated
    result = tmp;
    return;
}
   tmp = blah;
   // 29 is duplicated
  result = tmp;
  return;

The problem is that the final "return" is optimized away if the return block dominates the exit from the function, but here that's not quite right, because if anything can just branch into 29 at will it will be context-dependent when we can perform this redundant "return" removal.

Meanwhile I think how to solve this properly, you can try this patch.
diff --git a/spirv_glsl.cpp b/spirv_glsl.cpp
index 7601028..58fce85 100644
--- a/spirv_glsl.cpp
+++ b/spirv_glsl.cpp
@@ -6892,8 +6892,9 @@ void CompilerGLSL::emit_block_chain(SPIRBlock &block)
                // we do not need an explicit return which looks out of place. Just end the function here.
                // In the very weird case of for(;;) { return; } executing return is unconditional,
                // but we actually need a return here ...
-               else if (!block_is_outside_flow_control_from_block(get<SPIRBlock>(current_function->entry_block), block) ||
-                        block.loop_dominator != SPIRBlock::NoDominator)
+               //else if (!block_is_outside_flow_control_from_block(get<SPIRBlock>(current_function->entry_block), block) ||
+               //         block.loop_dominator != SPIRBlock::NoDominator)
+               else
                        statement("return;");
                break;

For reference, do you have the original GLSL file and .spv file you used to pass to spirv-opt?

I attached all the files.
I used spirv-opt, but I implemented the inlining pass myself.
input_ouput.zip

This code has been declared illegal by the working group, so closing this as not a bug. It breaks structured control flow.


Registration link Embeded on website

Here is the code, I put it on github too.

                JavaScript is disabled in your browser. To access our ticket shop without JavaScript, please click here.

Done in acbcf32a1e279dc1b2791819d3995dfff7a2880a


Repeats "Unzipping Index" infinitely when on the download tab of the Geode menu

Geode Issue

[X] I confirm that this bug is NOT related to a mod but directly to Geode Loader itself.

Platform
Windows
SDK commit
No response
Geode Version
v2.0.0-beta.7
Mods Installed
No response
Expected Behavior
Expected to view available mods.
Actual Behavior
Infinitely loops through these messages:
"
Checking Status
Beginning Download
Downloading
Unzipping Index
"
Steps to Reproduce

Click on the Geode icon on the main menu
Go into the download tab

Additional Information
Not sure if I'm just being too impatient but I've waited around 5 minutes and it keeps doing this.

cant reproduce, also do you still get this in beta 9

Technically it's not happening anymore, but in it's place it's producing an error message/popup/window

i did it somehow, it was working fine at first, then mods stopped loading when i customized an OpenHack keybind and OpenHack started crashing, so i uninstalled it, when i got back in geometry dash, it was forever updating local cache instead of unzipping index


Prepare for Vercel deployment

Added new UI

hey possible to see ?

look like we are doing the same thing at the same time

Module not found: Can't resolve '@openai/realtime-api-beta'


Feature ideas and / or requests - Colours in log and SSH port config

Hey,
just finished my mailserver and was looking for a backup solution. Found your script and I'm happy with it.
Two things tho, maybe you can make the use of colour in the logfile optional? It would make parsing and looking at it with different tools more easily. (I just removed the command -v tput...)
Also, my remote server is not listening to the default ssh port 22 due to secrity reasons, so I modified line 512 to use a different port.
export BORG_RSH="ssh -p 0815 -i ${borgSSHKey}"
Would be nice to have this as a configuration option also
Otherwise thanks for your script.

Glad it's working out for you! I've actually been thinking about those exact suggestions for a little while now and was going to put them into the next version. Here are my thoughts at the moment:

command-line switch to disable logfile colouring
SSH-port in the configuration details file

I should be able to make the changes this weekend, I'll keep you posted. Thanks for the suggestions! I very much appreciate it.

Implemented as noted above with commits.
Both suggestions implemented in Version 3.5
Feel free to re-open if I missed something. Thanks again for the great suggestions!


Qna datagen

Data Generation and Upserting into Qdrant Vector DB Completed.

Will raise a pr once the bash script for data generating is also included.


Add Lost (original) by Crim3s

Song Name
Lost (original)
Artist Name
Crim3s
Source
Youtube
Youtube Link
cpEkXk6u_b4
Direct File Link
No response
Song ID
482872
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 15050533
Downloaded: Yes

Yes


Current state upgradability

Will Elemental, in its current state, be upgradable to future versions ? And if not, are there any estimates as to when it will be ?

Well, not in a strict sense. As we're following cloud-native principles, there's no upgrade, only replace/redeploy 😉
But maybe I misunderstood your question 🤔

I don't have much knowledge in the repository structure and methodology being used.
What I mean is, Elemental is capable of upgrading to (or rather being replaced by, if I understand correctly), future versions of the operating system. Is Elemental currently capable of such ? Or is the operating system permanently immutable (which I doubt) ?
Maybe rewording it would help; if I install Elemental right now, in its current release, with rancherd and all, will it be, for the lack of better words, future-proof ? Will there be an clear upgrade path for, for instance, the replacement of rancherd by system-agent ?

The "rancherd" variant is/was experimental and there won't be a seamless switch to its successor, system-agent. You will need to adapt machineRegistration (and probably need to re-create machineInventory).
Once we do an official release of the stack, we will ensure upgradability to later versions.

Any idea when we could expect an official release, by any chance ?

By end of month 🤞🏻


fix(glyphs): fix order glyph

Just a little fix to order.svg
Before

After

Current coverage is 100% (diff: 100%)

Merging #163 into master will not change coverage

@@           master   #163   diff @@
====================================
  Files          19     19
  Lines         327    327
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
  Hits          327    327
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update 0b1f194...ddf719a


Allow choosing google default credentials in the C++ interop clients

Allows running the C++ standard interop clients with bins/dbg/interop_client --custom_credentials_type=google_default_credentials ...
Useful for testing google default creds specifically

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

for vm2prod with google default creds: this should be possible but is slightly outside of scope of my primary goal here (my main intent was to allow using google default creds in a different interop suite).
vm2vm is not valid a valid usage for google default creds?


Added a flag to include an additional listen metrics url

There is an issue in establishing communication between etcd endpoints and monitoring prometheus to do queries to the /metrics endpoint.
So, this change adds/exposes an additional url to be used as url for metrics queries.
This is related to this issue:
rackerlabs/kaas#1886
and to this PR:
rackerlabs/kaas#1985

Whoops. We need to make the PR against rackspace-v2.10.0 branch as well.

Ok


Add tests for MSC3787, I think

Disclaimer: I have no idea what I'm doing, but tried to reduce code duplication as much as possible.
See diff.

I don't feel it's fair to ask a contributor to also write the gomatrixserverlib changes for something like this, so sending up for review as-is.

I think this just needs https://github.com/matrix-org/synapse/pull/12623 and we can merge it?

I reran CI and it's still unhappy. I think we might be missing something on the Synapse side?
2022-05-18 17:37:29,895 - synapse.federation.federation_server - 612 - WARNING - GET-6 - Room version org.matrix.msc3787 not in ['4', '9', 'org.matrix.msc3667', '2', '6', '7', '1', '3', '5', '8']
2022-05-18 17:37:29,895 - synapse.http.server - 167 - INFO - GET-6 - <SynapseRequest at 0x7f41d0a32730 method='GET' uri='/_matrix/federation/v1/make_join/%21QMVXEbNUbesHiVJWag:hs1/@charlie:host.docker.internal:39159?ver=4&ver=9&ver=org.matrix.msc3667&ver=2&ver=6&ver=7&ver=1&ver=3&ver=5&ver=8' clientproto='HTTP/1.1' site='8448'> SynapseError: 400 - Your homeserver does not support the features required to interact with this room

I have no idea what I'm doing either.
In this CI run by Sean. There were three failures:
❌ TestCannotSendKnockViaSendKnockInMSC3787Room (8.63s)
      federation_room_join_test.go:356: Deploy times: 8.495259ms blueprints, 4.268363051s containers
      client.go:544: POST hs1/_matrix/client/r0/createRoom => 200 OK (242.262475ms)
      federation_room_join_test.go:370: MustJoinRoom: make_join failed: contents=[123 34 101 114 114 99 111 100 101 34 58 34 77 95 73 78 67 79 77 80 65 84 73 66 76 69 95 82 79 79 77 95 86 69 82 83 73 79 78 34 44 34 101 114 114 111 114 34 58 34 89 111 117 114 32 104 111 109 101 115 101 114 118 101 114 32 100 111 101 115 32 110 111 116 32 115 117 112 112 111 114 116 32 116 104 101 32 102 101 97 116 117 114 101 115 32 114 101 113 117 105 114 101 100 32 116 111 32 105 110 116 101 114 97 99 116 32 119 105 116 104 32 116 104 105 115 32 114 111 111 109 34 44 34 114 111 111 109 95 118 101 114 115 105 111 110 34 58 34 111 114 103 46 109 97 116 114 105 120 46 109 115 99 51 55 56 55 34 125] msg=Failed to GET JSON (hostname "hs1" path "/_matrix/federation/v1/make_join/!QMVXEbNUbesHiVJWag:hs1/@charlie:host.docker.internal:39159") code=400 wrapped=M_INCOMPATIBLE_ROOM_VERSION: Your homeserver does not support the features required to interact with this room

❌ TestKnockRoomsInPublicRoomsDirectoryInMSC3787Room (4.92s)
      knocking_test.go:379: Deploy times: 8.771865ms blueprints, 4.261340298s containers
      client.go:544: POST hs1/_matrix/client/r0/createRoom => 200 OK (268.73146ms)
      client.go:544: PUT hs1/_matrix/client/r0/rooms/!QDdSjnCQiLVsmtUips:hs1/state/m.room.join_rules/ => 200 OK (48.796638ms)
      knocking_test.go:397: SendEventSynced waiting for event ID $c0hKZ7VvtUewPdBpX4GOARpiqGoyXP5tmdcCY1pqxSc
      client.go:544: GET hs1/_matrix/client/r0/sync => 200 OK (55.937387ms)
      client.go:544: PUT hs1/_matrix/client/r0/directory/list/room/!QDdSjnCQiLVsmtUips:hs1 => 200 OK (15.593207ms)
      client.go:544: GET hs1/_matrix/client/r0/publicRooms => 200 OK (7.796654ms)
      knocking_test.go:467: Room was not present in public room directory response
  2022/05/18 17:37:21 ============================================

❌ TestKnockingInMSC3787Room (11.97s)
      knocking_test.go:39: Deploy times: 8.00577ms blueprints, 6.083955172s containers
      client.go:544: POST hs1/_matrix/client/r0/createRoom => 200 OK (318.012575ms)
      server.go:101: Server.UnexpectedRequestsAreErrors=false received unexpected request to server: GET /_matrix/federation/v1/query/profile - sending 404 which may cause the HS to backoff from Complement
  2022/05/18 17:37:16 complement: Unable to unmarshal incoming /invite request: gomatrixserverlib: unsupported room version 'org.matrix.msc3787'
      client.go:544: POST hs1/_matrix/client/r0/rooms/!OepnCEOfNUHyQTojDf:hs1/invite => 400 Bad Request (64.751684ms)
      knocking_test.go:76: CSAPI.MustDo POST http://localhost:49338/_matrix/client/r0/rooms/%21OepnCEOfNUHyQTojDf:hs1/invite returned HTTP 400 : {"errcode":"M_UNKNOWN","error":"Bad Request","message":"gomatrixserverlib: unsupported room version 'org.matrix.msc3787'"}

In the most recent CI run, two of those failures now pass. The middle one remains.
❌ TestKnockRoomsInPublicRoomsDirectoryInMSC3787Room (3.41s)
      knocking_test.go:379: Deploy times: 7.369706ms blueprints, 2.903263873s containers
      client.go:544: POST hs1/_matrix/client/r0/createRoom => 200 OK (180.2697ms)
      client.go:544: PUT hs1/_matrix/client/r0/rooms/!MoNVCTnYmGfxBaSjjk:hs1/state/m.room.join_rules/ => 200 OK (25.66037ms)
      knocking_test.go:397: SendEventSynced waiting for event ID $nF2cmnFVEYT5WIa54_b1MIPjtVK3mlA7yw65O-nKwXM
      client.go:544: GET hs1/_matrix/client/r0/sync => 200 OK (40.19948ms)
      client.go:544: PUT hs1/_matrix/client/r0/directory/list/room/!MoNVCTnYmGfxBaSjjk:hs1 => 200 OK (7.753812ms)
      client.go:544: GET hs1/_matrix/client/r0/publicRooms => 200 OK (3.987357ms)
      knocking_test.go:467: Room was not present in public room directory response
  2022/05/19 18:03:32 ============================================

In the most recent CI run, two of those failures now pass. The middle one remains.
❌ TestKnockRoomsInPublicRoomsDirectoryInMSC3787Room (3.41s)
      knocking_test.go:379: Deploy times: 7.369706ms blueprints, 2.903263873s containers
      client.go:544: POST hs1/_matrix/client/r0/createRoom => 200 OK (180.2697ms)
      client.go:544: PUT hs1/_matrix/client/r0/rooms/!MoNVCTnYmGfxBaSjjk:hs1/state/m.room.join_rules/ => 200 OK (25.66037ms)
      knocking_test.go:397: SendEventSynced waiting for event ID $nF2cmnFVEYT5WIa54_b1MIPjtVK3mlA7yw65O-nKwXM
      client.go:544: GET hs1/_matrix/client/r0/sync => 200 OK (40.19948ms)
      client.go:544: PUT hs1/_matrix/client/r0/directory/list/room/!MoNVCTnYmGfxBaSjjk:hs1 => 200 OK (7.753812ms)
      client.go:544: GET hs1/_matrix/client/r0/publicRooms => 200 OK (3.987357ms)
      knocking_test.go:467: Room was not present in public room directory response
  2022/05/19 18:03:32 ============================================

Nice job getting it working! I think you've found a legitimate test failure. I found three places in Synapse where we consider the knock join rule, but not knock_restricted:
https://github.com/matrix-org/synapse/blob/942c30b16b86cb05d2109b13bc2c1dc9ac2fea70/synapse/handlers/room_summary.py#L664-L668
https://github.com/matrix-org/synapse/blob/320186319ac4f1d16f8f964d92db8921a4b1073e/synapse/storage/databases/main/room.py#L244-L253
https://github.com/matrix-org/synapse/blob/320186319ac4f1d16f8f964d92db8921a4b1073e/synapse/storage/databases/main/room.py#L381-L394
The next step probably be to figure out what those three places should do and fix Synapse?

if this is accepted, I don't know if go.mod/sum ought to be updated to point to the main branch of gomatrixserverlib rather than my one-off branch

Yes please — since we squash commits into GMSL, it's best to pin commit IDs from the main branch where possible.


Implement ES6 destructuring assignment

To swap variables:
var a=6,b=3;
[a,b]=[b,a];
console.log(a); //3
console.log(b); //6

Array destructuring:
var array=[5,3,9,8,3,2];
var [a,b,c,...d]=array;
console.log(a); //5
console.log(b); //3
console.log(c); //9
console.log(d); //[8,3,2]

Create variables with values from an object:
var object={a:5,b:"+",c:false};
var {c:bool,a}=object; //Creates the variables `bool` and `a` with values from `object`.
console.log(bool); //false
console.log(a); //5

More information here:
Mozilla's documentation
Exploring ES6

Duplicate to #366, closing.

Hmm, didn't see it when I searched it yesterday.


Extension issue

Issue Type: Bug
Extension Name: LiveServer
Extension Version: 5.6.1
OS Version: Windows_NT x64 10.0.19042
VS Code version: 1.57.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:

Issue Type: Bug
Extension Name: LiveServer
Extension Version: 5.6.1
OS Version: Windows_NT x64 10.0.19042
VS Code version: 1.57.1

Issue Type: Bug
Extension Name: LiveServer
Extension Version: 5.6.1
OS Version: Windows_NT x64 10.0.19042
VS Code version: 1.57.1


npm start error on eject

Hi,
i followed these instructions:
npx create-inferno-app inferno-app
cd inferno-app
npm start

npm start runs fine,
then when i wanted to eject it doesn't work
then i wanted to eject
npm run eject
npm start

><EMAIL_ADDRESS>start
> node scripts/start.js

node:internal/modules/esm/resolve:263
    throw new ERR_MODULE_NOT_FOUND(
          ^

Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/Users/liturribarria/Sites/evolve/temp/inferno-app/scripts/utils/process-env-develop.js' imported from /Users/liturribarria/Sites/evolve/temp/inferno-app/scripts/start.js
    at finalizeResolution (node:internal/modules/esm/resolve:263:11)
    at moduleResolve (node:internal/modules/esm/resolve:908:10)
    at defaultResolve (node:internal/modules/esm/resolve:1131:11)
    at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:390:12)
    at ModuleLoader.resolve (node:internal/modules/esm/loader:359:25)
    at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:234:38)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:85:39)
    at link (node:internal/modules/esm/module_job:84:36) {
  code: 'ERR_MODULE_NOT_FOUND',
  url: 'file:///Users/liturribarria/Sites/evolve/temp/inferno-app/scripts/utils/process-env-develop.js'
}

Node.js v21.6.1

Thanks for reporting this issue, I will look into it

This is now fixed in the latest version.

thank you @Havunen !!!
and just a question for you... any thoughts on updating so we don't have any warnings?
> npm install
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-terser
npm WARN deprecated<EMAIL_ADDRESS>Use your platform's native atob() and btoa() methods instead
npm WARN deprecated<EMAIL_ADDRESS>Use your platform's native DOMException instead
npm WARN deprecated<EMAIL_ADDRESS>Please use @jridgewell/sourcemap-codec instead

added 1448 packages, and audited 1449 packages in 3s

312 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

Yeah all the npm packages were updated to latest avalaible versions but I might have missed some packages if they have changed their names. Can you open a new issue about that please

sure, thank you so much @Havunen


add tests for boolean, number and string editing

One of the todo's of #86

fix input type for strings
replace some == with ===
add tests for boolean, number and string editing

RFC: strip :data-testid bindings on production builds. Eventually might help with perf (fewer bindings to evaluate)... not sure it's necessary yet, thoughts?

RFC: strip :data-testid bindings on production builds. Eventually might help with perf (fewer bindings to evaluate)... not sure it's necessary yet, thoughts?

You already added this, right? Better to include then, and probably better to think performance early on anyway.

RFC: strip :data-testid bindings on production builds. Eventually might help with perf (fewer bindings to evaluate)... not sure it's necessary yet, thoughts?

You already added this, right? Better to include then, and probably better to think performance early on anyway.

Yep it's done with a regex which isn't ideal but will work for now.


Enable connection state listener by default

Task: https://github.com/Azure/azure-sdk-for-java/issues/28268
Description:
Enable endpointRediscovery(aka. ConnectionStateListener) by default.

API change check for com.azure:azure-cosmos
API changes are not detected in this pull request for com.azure:azure-cosmos

API change check
API changes are not detected in this pull request.

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

Azure cosmos pipeline: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1576752&view=results
Run the following tests locally:
getSessionTokensInRequests
splitQueryContinuationToken
orderByQueryForLargeCollection
before_MultiOrderByQueryTests


Cleanup and prepare Openshift CI

Reorg Makefile:

add build target
add lint target: runs all except build and unit test
replace fmt with goimports (it's a superset)
remove vendor related targets (doesn't make sense for a library)

Update github workflow for updated Make targets
Delete vendor dir
Add build root image config file for OpenShift CI

/lgtm
NIT: We can also update the tools in this PR of a follow up

GOIMPORTS_VERSION ?= v0.11.0
SORT_IMPORTS_VERSION = v0.2.1


architecture for migrating spring boot apps to azure spring apps

[Enter feedback here]
I am looking for architecture diagram on the infra services that are used in the app consolidation and migration of spring boot apps to azure spring apps. i have gone through the azure spring apps standard and enterprise architecture already.

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

ID: 09d9c752-ab9b-2ec1-5ad8-5730e9068762
Version Independent ID: 09d9c752-ab9b-2ec1-5ad8-5730e9068762
Content: Migrate Spring Boot applications to run on Azure Container Apps
Content Source: articles/java/migration/migrate-spring-boot-to-azure-container-apps.md
Service: azure-java
GitHub Login: @KarlErickson
Microsoft Alias: manriem

Hi @hemas2022 - I'm not aware of any ASA architecture diagrams besides the ones you've already seen. You might be able to find something on the Apps on Azure blog, or you can ask for help on the Microsoft Q&A site. I hope this helps!


New Term Request: SPN

For new term requests, please provide the following information:
Preferred term label
'serotonergic projection neuron' but it's a specific pair that are activated by lack of Dnc so maybe needs more?
Synonyms
SPN
Textual definition
FBrf0244287 but original description in FBrf0238754
The SPN is located in the gnathal ganglia (GNG) and projects to central brain compartments, where it surrounds the MB peduncle.
Suggested parent term
projection neuron?
Attribution
If you would like a nanoattribution, please indicate your ORCID id

'serotonergic projection neuron' but it's a specific pair that are activated by lack of Dnc so maybe needs more?

Agree, “serotonergic projection neuron” can be used as a synonym but for the label we’ll need to mint a more precise name – unless it is clear that this pair are the only projection neurons that are serotonergic, which seems unlikely.


Search conflicting with Ransack Gem (used by active_admin)

I'm not able to perform a search due to: https://github.com/elastic/elasticsearch-rails/issues/96

but how is it possible? Chewy doesn't use elasticsearch-rails or elasticsearch-model gems at all

@pyromaniac - Hitting the same issue. When I remove active_admin, the error goes away.

@ACPK what if you will try to use User.ransack for searching instead of User.search ?


local development: command "pnpm i" causes dependency error in nextra-theme-docs 2.0.1

Just did a fresh "deploy" from https://nextra.site/docs/docs-theme/start
As documented in README.md

when I run pnpm i below error is caused.
dependencies:
+ next 13.0.6
+ nextra 2.0.1
+ nextra-theme-docs 2.0.1
+ react 18.2.0
+ react-dom 18.2.0

 WARN  Issues with peer dependencies found
.
└─┬ nextra-theme-docs 2.0.1
  └─┬ @reach/skip-nav 0.17.0
    ├── ✕ unmet peer<EMAIL_ADDRESS>|| 17.x": found 18.2.0
    ├── ✕ unmet peer<EMAIL_ADDRESS>|| 17.x": found 18.2.0
    └─┬ @reach/utils 0.17.0
      ├── ✕ unmet peer<EMAIL_ADDRESS>|| 17.x": found 18.2.0
      └── ✕ unmet peer<EMAIL_ADDRESS>|| 17.x": found 18.2.0

pnpm-lock.yaml generated in my local repository has the highlighted differences compared to same file in https://github.com/shuding/nextra-docs-template
Any suggestions for a workaround?

root cause seems https://github.com/reach/reach-ui/issues/925

root cause seems reach/reach-ui#925

Considering that Reach UI is not maintained anymore it might worth considering getting rid of that library.

@phenomen thanks for hint
@shuding radix  as used in https://github.com/UMAprotocol/voter-dapp-v2/pull/118/commits/527b1e0f72ad2521d4090591d42c652f06539e06 looks like a quick and easy approach to fixing it. What is your thought?

Makes sense to move away from Reach to Radix! Happy to accept a PR or I can do it next week 👍

reach/skip-nav is used only in examples/swr-site/pages/_document.js
would replacement with  as in https://www.w3.org/WAI/ARIA/apg/example-index/landmarks/navigation.html be a proper replacement?

I don't think that does the same job. skip-nav provides this "skip to content" link when using tab to navigate the site:

Unfortunately I have no idea how to fix this issue :(

update:
after I removed the mdx file using "callout", build succeeded but it seems to indicate another error in nextra-theme-docs
./pages/99_HowToEditSite/2 Advanced formatting in Markdown.mdx
Module not found: Package path ./callout is not exported from package /vercel/path0/node_modules/nextra-theme-docs (see exports field in /vercel/path0/node_modules/nextra-theme-docs/package.json)

Solution proposed in https://github.com/shuding/nextra/pull/1051


How could we assert that values are up-to-date?

Hello,
While @fighou was working on the RuPay card (#34), the card was updated on the Wikipedia's page. You can view the diff.

Issuing network
IIN ranges
Active
Length
Validation

RuPay
60, 65, 81, 82, 508
Yes
16
Luhn algorithm

(RuPay-JCB co-branded)
353, 356
Yes
16
Luhn algorithm

So, what could we do to assert that values are up-to-date? Would it be possible to have a bot who check for wiki edit on the Payment card number. Is such a bot do exist or should we create one? How could we tackle that?

I build a simple zap to get an issue whenever the wikipedia page's RSS is updated: https://zapier.com/shared/dc960411048cab84fb11b5e8b834080cb3dc52b5
Let's see if it works as expected.
Related to #42


Bug with procedure calls in if/cond statements

Example:
(if ((baz? 10)) 5 10)
The if statement conditional should not require extra parentheses to call a procedure. Need to add procCall rule to condCond and ifCond.

Scratch that - we just need to remove the open paren and close paren for the if statement in the grammar. It should just be if expr expr expr or if expr expr.

Scratch that - we just need to remove the open paren and close paren for the if statement in the grammar. It should just be if expr expr expr or if expr expr.


Not Able To install Home-brew Formula's

I would like you to guide me on how I can install home-brew formulas as if I add it to casks.nix it doesn't gets installed as your casks.nix searches only brew casks not formulas.
Please provide a solution to this or give me a way to install all those from nix-store with the file location and where to write it

Please see my answer here:
https://github.com/dustinlyons/nixos-config/issues/108#issuecomment-2315998019


When I use "typescript 4.8.4" to package the production environment, I get an error and switch to "typescript 4.6.4" and everything works fine.

Vue version
3.2.25
Link to minimal reproduction
none
Steps to reproduce
 "vite": "^2.7.10",
 "vue": "^3.2.25",
 "typescript": "^4.8.4",

When I use "typescript 4.8.4" to package the production environment, I get an error and switch to "typescript 4.6.4" and everything works fine.
Components are defined using the "defineComponent setup" method。Export functions through "return"
What is expected?
working.
What is actually happening?

System Info
No response
Any additional comments?
No response

Repeoduction missing.


cordova-plugin-whitelist does not support cordova-android 10.x

Feature Request
Motivation Behind Feature
Now that cordova-android  10.0.0  is out .   This plugin should support cordova-android 10+
Feature Description
Please update this plugin to support cordova-android 10+
Alternatives or Workarounds
no workaround

the readme  mentioned  cordova android 10 does not need this plugin .

The plugin is deprecated. See https://github.com/apache/cordova-plugin-whitelist#deprecation-notice
Cordova-android has the same implementation (renamed to allow list) built in now.


feat: Add graphite-cli aliases file with out of store symlink

feat: Add graphite-cli aliases file with out of store symlink
Setup home-manager backup file extension
When a file is in the way, hm renames it withthis extension and replaces
it with the new one
refactor: Separate options into options.nix
Move options from default.nix to options.nix, import in default.nix

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#12
#11  👈 (View in Graphite)
#10
#9
#7
#5 : 1 other dependent PR (#6 )
#4
#2 : 1 other dependent PR (#3 )
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


ARROW-3347: [Rust] Implement PrimitiveArrayBuilder

Adds builder for PrimitiveArray.
@sunchao has mentioned that it's unfortunate that we have to rely on macros to define the impl block for types implementing ArrowPrimitiveType.  When specialization lands in stable we can remove much/all of this but for now we have to rely on macros.
This implementation mostly focuses on being correct.  However, maybe we should add push_value_raw and push_null_raw and allow the caller to handle updating the bitmap (i.e. avoid checking if the bitmap is_some on every push)?  If so, I can add this as a separate PR (along with other optimizations).

cc @kszucs @crepererum @sunchao @andygrove

The underlying BufferBuilder is already wrong here, but I think capacity and len should be usize (or at least u64 if you like to overflow on 32bit systems). Maybe this could be fixed in a follow-up PR.

@crepererum I'm not sure about this, the spec says that lengths should be i32, I believe that this is to ensure compatibility with different implementations, in particular, with Java.
But different implementations can provide support up to i64, for instance, C++ seems to use i64.  I think that we may need to add assertions to guard against overflows though (I recall seeing this in the C++ implementation but I can't find it now)?
@xhochy or @wesm could you comment on this?

Implementations are free to support larger than i32 lengths as long as they stay inside their scope. But you need to be able to split these things up when you pass Arrow data to other implementations.  You can pass these i64 length buffers to C++. But in the case that you deal with "any" Arrow implementation, you should only pass i32 sized buffers along.

@crepererum my thinking is that as BufferBuilder is generic over T so len and capacity are measuring array slots and so i64 is used.  The resulting Buffer is type-less and so it's len is usize.
Does this make sense to you or do you still see an issue?

I still don't see how "array slots" can be negative though. Same goes for everything that @xhochy said. How can a length be negative?
Not matter how we decide, we have basically two options:
a) use signed integers but assert that the are not negative (if nobody comes up with a good reason why this should be the case)
b) use unsigned integer but check that they don't exceed std::i32::MAX, although I'm kinda unhappy about the fact that we only support 2G elements.

These lengths cannot be negative. We use signed types due to some weird language behaviours in Java and C++. Sometimes, e.g. for the null count, we use -1 as a placeholder value that we need to compute this value first.
Thus feel free in the Rust implementation to use one of the things @crepererum proposed.

At first I was going to agree but some versions of Java have no unsigned ints so data transferred from the Java impl to the Rust impl could be interpreted the wrong way at the byte level, I guess this must be the reason for this choice.
Maybe you should ask this question on the mailing list, as it has wider implications for the other implementations and is much larger in scope than this PR.

Thanks @xhochy.  @crepererum the jira is here.
Is this PR ready to merge then?

Great. Thanks @paddyhoran for the work, patience and issue creation. Thanks @xhochy for the clarifications. IMHO the PR is now ready to merge.

Rebased on top of https://github.com/apache/arrow/pull/2868

But you need to be able to split these things up when you pass Arrow data to other implementations.

@xhochy Is there any existing example on this? I assume if we choose usize on a 64-bit platform, we have to handle this conversion somehow in order to be truly compatible with the standard, right?

Thanks for the PR and the reviews!

@sunchao We should add rust to the integration testing suite: https://github.com/apache/arrow/tree/master/integration

@sunchao We should add rust to the integration testing suite: https://github.com/apache/arrow/tree/master/integration

Sure. Filed ARROW-3688.


feat: Include ntl

📑 Description
Added ntl dependency.
✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update embedded copy of protoc to 3.10 in grpc-tools rubygem

What version of gRPC and what language are you using?
1.25.0
What operating system (Linux, Windows,...) and version?
GNU/Linux Debian unstable/sid
What runtime / compiler are you using (e.g. python version or version of gcc)
ruby 2.5
What did you do?
We noticed embedded copy of protoc inside grpc-tools rubygems is still 3.8.0
What did you expect to see?
It should be updated to latest version of protoc 3.10.1
What did you see instead?
It was using protoc 3.8.0
Anything else we should know about your project / environment?
We'd like to use protoc 3.10.1 in debian to build gitaly but it fails with protoc 3.8.0 with following error.
> grpc_tools_ruby_protoc --version
libprotoc 3.8.0
-> grpc_tools_ruby_protoc -I ./proto --ruby_out=ruby/proto/gitaly --grpc_out=ruby/proto/gitaly proto/blob.proto proto/cleanup.proto proto/commit.proto proto/conflicts.proto proto/diff.proto proto/hook.proto proto/namespace.proto proto/objectpool.proto proto/operations.proto proto/ref.proto proto/remote.proto proto/repository-service.proto proto/server.proto proto/shared.proto proto/smarthttp.proto proto/ssh.proto proto/wiki.proto
[libprotobuf WARNING google/protobuf/compiler/ruby/ruby_generator.cc:509] Omitting proto2 dependency 'google/protobuf/descriptor.proto' from proto3 output file 'shared_pb.rb' because we don't support proto2 and no proto2 types from that file are being used.
--ruby_out: shared.proto: Extensions are not yet supported for proto2 .proto files.

See https://gitlab.com/gitlab-org/gitaly/issues/2164#note_245423255 for context and discussion

It looks like the reason for this is just that our protobuf submodule dependency is a bit old, and hasn't been updated since https://github.com/grpc/grpc/pull/19182
It may be worth an update.
@jtattermusch @nicolasnoble, do you have any extra context here?

3.11 release of protobuf will be out soon, we should make sure we'll update to that one (there were some C# issues in 3.9 and 3.10 and we never updated, it should be fine with 3.11)

@jtattermusch protobuf 3.11 is now released, I hope we can have a new release soon with this version.

+1, I'm currently blocked by this as well.
It would be fantastic if the grpc-tools rubygem could be updated to protobuf 3.11! :)

@apolcyn we are blocked on this for updating grpc-tools/protoc in gitlab and debian, it'd be great if this can be fixed soon.


FunctionCallSignature works incorrectly

Describe the bug
Seems PEAR.Functions.FunctionCallSignature.Indent sniff and fix works incorrectly with some nested structures.
The problem appears when there is indentation issue together with another issue, like PEAR.WhiteSpace.ScopeIndent.Incorrect.
The problem does not appear when correctly formatted and correct code is processed by phpcbf.
However, wrong number of indentation character (space) is not reported anymore.
Sample code
public function store()
{
  $fails  = $this->req(array(
    'device_id'=> array (
      'default' =>$this->app->param('device_id'),
      'min' => 0,
      'optional' => true
    ),
    )
    );
}

Phpcbf formatting
public function store()
{
    $fails  = $this->req(
        array(
        'device_id'=> array (
        'default' =>$this->app->param('device_id'),
        'min' => 0,
        'optional' => true
        ),
        )
    );
}

Another Phpcs wrong indent not caught:
public function store()
{
    $fails  = $this->req(
        array(
        'device_id'=> array ( // missing indent
                'default' =>$this->app->param('device_id'),
                'min' => 0,
                'optional' => true
            ), // this line only has 2 spaces
        )
    );
}

Sample from PEAR standard
From: https://pear.php.net/manual/en/standards.funcalls.php
$this->someObject->subObject->callThisFunctionWithALongName(
    $this->someOtherFunc(
        $this->someEvenOtherFunc(
            'Help me!',
            array(
                'foo'  => 'bar',
                'spam' => 'eggs',
            ),
            23
        ),
        $this->someEvenOtherFunc()
    ),
    $this->wowowowowow(12)
);

Custom ruleset
No custom ruleset
To reproduce
Steps to reproduce the behavior:

Create a file called test.php with the code sample above...
Run phpcs test.php ...
See error message displayed
Run phpcbf test.php
Check indentation
phpcs test.php no longer reports indentation error.

PHPCS output here
-------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 9 ERRORS AFFECTING 6 LINES
-------------------------------------------------------------------------------------------------------------------------------------------------------
  2 | ERROR | [ ] Missing file doc comment (PEAR.Commenting.FileComment.Missing)
  2 | ERROR | [ ] Missing doc comment for function store() (PEAR.Commenting.FunctionComment.Missing)
  4 | ERROR | [x] Line indented incorrectly; expected at least 4 spaces, found 2 (PEAR.WhiteSpace.ScopeIndent.Incorrect)
  4 | ERROR | [x] Opening statement of multi-line function call not indented correctly; expected 0 spaces but found 2
    |       |     (PEAR.Functions.FunctionCallSignature.OpeningIndent)
  4 | ERROR | [x] Opening parenthesis of a multi-line function call must be the last content on the line
    |       |     (PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket)
  5 | ERROR | [x] Multi-line function call not indented correctly; expected 6 spaces but found 4 (PEAR.Functions.FunctionCallSignature.Indent)
  9 | ERROR | [x] Multi-line function call not indented correctly; expected 6 spaces but found 4 (PEAR.Functions.FunctionCallSignature.Indent)
 10 | ERROR | [x] Multi-line function call not indented correctly; expected 6 spaces but found 4 (PEAR.Functions.FunctionCallSignature.Indent)
 11 | ERROR | [x] Multi-line function call not indented correctly; expected 2 spaces but found 4 (PEAR.Functions.FunctionCallSignature.Indent)
-------------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 7 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------------------

Time: 91ms; Memory: 6MB

Expected behavior
Phpcbf: follow standard while fixing.
Phpcs: catch wrong indentation.
Versions (please complete the following information)

Operating System
Debian 10

PHP version
7.4

PHP_CodeSniffer version
PHP_CodeSniffer version 3.8.0 (stable) by Squiz (https://www.squiz.net)

Standard
PEAR

Install type
git clone

Please confirm

[ x] I have searched the issue list and am not opening a duplicate issue.
[ ] I confirm that this bug is a bug in PHP_CodeSniffer and not in one of the external standards.
[ x] I have verified the issue still exists in the master branch of PHP_CodeSniffer.

Thank you!

Hi,
I edited the issue and added more details with better sample code.

My guess is sniffer would need recursive processing of multi line function calls. Or from inside to outside.
I tried to look into it, but it was too much...


Emails

Includes 3 small updates related to emails.

Default from email was changed to<EMAIL_ADDRESS>I tested sending to my Gmail account and it was received within a minute
Grouped error messages so they should only be sent when the count for a particular error is 1, 2, 4, 8, 16, 32, 64, 128, ..., (2**n)
In looking at the email form, it already requires patrons to login. It looks like we storing their JHU email so Catalyst will now pre-populate the form with the saved email to simplify the user experience

@fackrellj I'm glad that the emails are coming through now! Does this get rid of the question mark in Gmail's mobile app?

Yes it does. I deployed this to stage if you want to test it.


Missing writer for extension "gif"

Hi, i discover this error this morning
Uncaught PHP Exception Endroid\QrCode\Exception\UnsupportedExtensionException: "Missing writer for extension "gif""

i dont see where this come form. It's working very well and still, but some time i get this error.
Thank for helping
Sf3.3.16, QrcodeBundle 1.4

Hi @xorgxx are you sure you are using 1.4? The exception class you mention does not exist in that version.

Sorry version was endroid/qrcode": "2.4.0"

Ah ok. I see that version does not have a GIF writer, see https://github.com/endroid/qr-code/tree/2.4.0/src/Writer. So maybe you should use the .png extension in the URL you use to generate the image.

generate url in controller by            $url        = $this->generateUrl( 'nfc_secure_index',array('machash' => $yneTag->getMachash()), UrlGeneratorInterface::ABSOLUTE_URL );
generate Qrcode in twig by : <img src="{{ qrcode_url( url ) }}"/>

I am not familiar with the 'nfc_secure_index' route, it is not part of this library. What URL does it generate? If it generates a URL ending with .gif you have the source of the issue.

this url pass to qrcode_url() : https://www.xxxx.com/fr/qrcode/https://www.xxxx.com/fr/xxxx/nfctag/secure/7f3a37740896b0a9062ee1a8c8f4d204

Hi @xorgxx when I search the code there is no presence of the string gif and when I check out the 2.4.0 version and use the qrcode_url twig call to render your URL, I get a normal PNG. In other words unfortunately I am not able to reproduce the issue you are having...

Maybe you could try to use a more recent version of the library or generate the code programmatically instead of via the Twig extension since it seems you're the only one having this issue.

This is why i was asking same help. I have look before. more over, it can be that for some raison the url to pass to qrcode_url()  suppose to have no extension  having extension.gif? then in my code i have some bug's. it das not always bug. I even think it das apen only same time with Iphone :/.

i also see now that this error come main ly with googlebot(at)googlebot.com!!


Database Layer

Build initial database tables used for room configuration and system setup.

   System
      |
      |
     / \
    /   \
   /     \
Rooms   Devices


Good New Page

This is simple, comprehensive, and useful - an improvement over last version docs.

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

ID: cca95f5e-0fe4-bbd2-fd23-13562e02f888
Version Independent ID: c3fb9673-4da6-5eeb-08bb-6f8e7bc8e0c6
Content: Part 8, add a new field to an ASP.NET Core MVC app
Content Source: aspnetcore/tutorials/first-mvc-app/new-field.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @wadepickett
Microsoft Alias: wpickett

@tomth1, thanks so much for the feedback, great to hear!


Correct unchecked assignment in AutoValue implementation

Fixes unchecked assignment error and a syntax error in AutoValue example implementation

Thanks!


Enforce Spatial Raster Type

Following up on my comment I think it may actually make more sense to just remove this type part of the schema altogether during the schema upgrade.  Do you see any value to either leaving it as-is without this PR or enforcing the type?  The information seems redundant so removing it would make things cleaner.

I also agree that it is redundant right now, since we currently can get the value from isBitmask of raster.json. However, I could imagine in the future, maybe there will be other file types which do not have this option, and then it will need to be defined in the coordination object like this, so I think this PR is a good idea.


Balance x is less than y (including fee)

First of all, fantastic port of the bit library, things seem to be coming along nicely. This may fit into the dynamic fees problem, but I wanted to see if there was anything I could do to fix it in the meantime.
I have an app that receives transactions and processes the balances to result in changes for the application. The balance is then marked as processed and is (eventually) moved to a cold storage wallet. Often this is the entire balance of that wallet. My code looks something like:
processed_balance = get_processed_balance()  # This is often the full balance of the wallet
if processed_balance > 0:
    key = wif_to_key(get_priv_key())
    print(key.get_unspents())
    outputs = [
        (
		settings.COLD_STORAGE_WALLET,
		processed_balance,
		'bch'
	),
    ]
    key.send(outputs, fee=1)

And the output:
# Debug printing the unspents
[Unspent(amount=199322, confirmations=82, script='someScriptKey', txid='sometxid', txindex=1)]
# The resulting error
Traceback (most recent call last):
  File "actions.py", line 62, in move_to_cold_storage
    key.send(outputs, fee=1)
  File "/venv/lib/python3.5/site-packages/bitcash/wallet.py", line 291, in send
    outputs, fee=fee, leftover=leftover, combine=combine, message=message, unspents=unspents
  File "/venv/lib/python3.5/site-packages/bitcash/wallet.py", line 249, in create_transaction
    compressed=self.is_compressed()
  File "/venv/lib/python3.5/site-packages/bitcash/transaction.py", line 155, in sanitize_tx_data
    'fee).'.format(total_in, total_out))
bitcash.exceptions.InsufficientFunds: Balance 199322 is less than 199548 (including fee).

Should I be getting the total fee for the transaction beforehand and subtracting it from the total to send? What is the best way to go about doing that?
Thanks in advance!

Hi!
I'm glad you've been finding it useful. Thanks for writing in.
For sweeps, I've just been doing this: https://github.com/teran-mckinney/walkingliberty-python/blob/master/walkingliberty/init.py#L92
Sounds like you aren't always sweeping, though. So your outputs less have to be less than the balance of the wallet. The fees don't get taken from the outputs. The leftover_address is where the remainder should be sent after fees. You can't have outputs of the full balance, there's no logic in the library to remove fees from the outputs. So if you want to sweep the whole thing, you have to calculate the fees in advance, or you can just do a sweep like that.
All of the inputs have their amount accounted for. Whatever isn't an output in that is a fee. A change address is usually where you want the remainder to go.
Take a look here at the fees and outputs: https://github.com/sporestack/bitcash/blob/master/bitcash/wallet.py#L501
Been a while since I worked on the library last so hopefully am explaining it right. Let me know if this makes sense or not.

Thanks for getting back so quickly! That makes good sense. I'll probably end up doing sweeps the same way you are.
Thanks for the clarification and pointers. I'll close this for now since it was really just a question, feel free to reopen as needed!


SAN migration function request

Request from Devon Carr @OCTeksperts via Twitter -
Basically, the migration script would connect to Pure SAN A, get disks with name like "SERVER1" and dump that output to an array, then connect to SAN B and create those disks with same name and size and protection group of the same name. Can you help me?

This should be fairly simple to implement.


Incorporate miscellaneous Roy feedback # 1-3, 5,12

Acronyms. Suggest a “hard Scrub” on acronyms and always “spell out” on first use. A very quick read identified a number of acronyms not yet included in section 1.5. (GFE, SDLC, CPARS, WBS, CI/CD, MID (Note I believe this is a postal acronym for “Mailer Identification number,” SOW, Q/A, and UMGC and ITEC (note the last two may not be recognized by a contract support team after hand of to the Postal Service.
GFE - government furnished equipment
SDLC - Software Development Life Cycle
CPARS – Contractor Performance Assessment Reporting System
WBS – Work Breakdown Structure
MID - Mailer Identification number
CI - Continuous Integration (CI)
CD - Continuous Delivery (CD)
SOW – statement of work
QA – Quality Assurance
UMGC – University of Maryland Global Campus
SWEN - Software Engineering Project (SWEN)
Misc.
Section 1 – Project Information – first sentence currently reads “United States Post Office (USPS)…  Should read “United States Postal Service (USPS)
Section 1.2 Purpose – second paragraph, last sentence reads: “The PMP will be revisited for possible revision or updates when any of the following condition occur:” I assume a list of conditions will be added.
Section 1.4 Stakeholders. Please change my (Roy Gordon) email address to<EMAIL_ADDRESS>Section 2.1 Scope. Misspelled word in third paragraph “Vale”

Section 1 – Project Information – first sentence currently reads “United States Post Office (USPS)… Should read “United States Postal Service (USPS)
Updated
Section 1.2 Purpose – second paragraph, last sentence reads: “The PMP will be revisited for possible revision or updates when any of the following condition occur:” I assume a list of conditions will be added.
List was already there. Not sure if he just didn't see it or it printed weird
Section 1.4 Stakeholders. Please change my (Roy Gordon) email address to<EMAIL_ADDRESS>done
Section 2.1 Scope. Misspelled word in third paragraph “Vale
I don't see this

Add Team B Misc items

Consider adding a “Documentation Information” table to page 2. Include: Document ID, Document Owner, Issue Date, Last Date Document Saved, File Name.

Suggest removing Approval Signatures for Robert Dixon, Dr. Mir Assadullah, and Roy Gordon.

Suggest removing Section 1.1 Purpose, second paragraph, first sentence: “Information provided in this PMP is determined from the Statement of Work (SOW) to be provided by the selected contractor……….”  I’m not familiar with a commitment to provide an SOW artifact and OCIO policies do not need to be cited.

Acronyms. Suggest a “hard Scrub” on acronyms and always “spell out” on first use. A very quick read identified a number of acronyms not yet included in Appendix A. (e.g. IDMA, OCR, HRHS) Also, it appears that Appendix A – Acronyms contains acronyms that do not appear in the document. (e.g. BAD, NFR, and others)

Misc.
a.	Section 1.4, Table 1.1. Please change my (Roy Gordon) email address to<EMAIL_ADDRESS>b.	Section 3.5 first sentence.  Reference made to “Table XXXX.”
c.	Section 8 Risk Management, Figure x-0.  Recommend citing source of diagram if not original art.

Add Team B Misc items
Consider adding a “Documentation Information” table to page 2. Include: Document ID, Document Owner, Issue Date, Last Date Document Saved, File Name. - Done
Suggest removing Approval Signatures for Robert Dixon, Dr. Mir Assadullah, and Roy Gordon. - Done
Suggest removing Section 1.1 Purpose, second paragraph, first sentence: “Information provided in this PMP is determined from the Statement of Work (SOW) to be provided by the selected contractor……….” I’m not familiar with a commitment to provide an SOW artifact and OCIO policies do not need to be cited. - Done
Acronyms. Suggest a “hard Scrub” on acronyms and always “spell out” on first use. A very quick read identified a number of acronyms not yet included in Appendix A. (e.g. IDMA, OCR, HRHS) Also, it appears that Appendix A – Acronyms contains acronyms that do not appear in the document. (e.g. BAD, NFR, and others) - Done
Misc.
a. Section 1.4, Table 1.1. Please change my (Roy Gordon) email address to<EMAIL_ADDRESS>- Already Done
b. Section 3.5 first sentence. Reference made to “Table XXXX.” - Done
c. Section 8 Risk Management, Figure x-0. Recommend citing source of diagram if not original art. - Done


bump nix to 0.14 to fix libc issues

~Waiting on: https://github.com/nix-rust/nix/issues/1060#issuecomment-494627579~
See also: https://github.com/rust-embedded/linux-embedded-hal/pull/17
Patching up to git source tested to be working

r? @posborne
(rust_highfive has picked a reviewer for you, use r? to override)

FYI: solution suggested here may not work.
This should fix the issue, until this pr is not merged
[patch.crates-io]
i2cdev = { git = "https://github.com/ryankurte/rust-i2cdev" }

@posborne  nix 0.14.x has landed and IC seems happy ^_^

Should the version number also be bumped to 0.4.2?

This project hasn't been updated to declare and test against a MSRV formally (https://github.com/rust-embedded/wg/blob/master/ops/msrv.md).  I believe from 0.11 -> 0.14 of nix certainly would change the implicit MSRV which would warrant a "major" version bump (0.4 -> 0.5) as I understand it.
It certainly seems annoying to have a major bump for this but this crate does have decent fan-in from other published crates and I am hesitant to possibly break the MSRV of other crates and projects so I believe the correct change here is to go to 0.5 with an MSRV for the new version of 1.24.1 based on the nix MSRV @ 0.14.

I'm willing to go with the MSRV from sysfs-gpio of 1.26.0 as the "implied" MSRV right now.  The MSRV for nix is 1.24.1 for 0.14 so I think this is OK.

bors r+


Fix job detail run history integration tests

Adjust the default sort behaviour and fix the job run history integration tests.

Reviewer: @jfurrow

🚢  LGTM!

If the fixture change does not bother you, 🚢


Add option to change worker placement_tenancy.

PR o'clock
Description
Support launching worker node with non default tenancy.
Some companies have compliance requirements,  ec2 instances must be dedicated instances.
Checklist

[x] terraform fmt and terraform validate both work from the root and examples/eks_test_fixture directories (look in CI for an example)
[ ] Tests for the changes have been added and passing (for bug fixes/features)
[ ] Test results are pasted in this PR (in lieu of CI)
[ ] Docs have been updated using terraform-docs per README.md instructions
[ ] I've added my change to CHANGELOG.md
[ ] Any breaking changes are highlighted above

Thanks for the PR @monsterxx03 ! Can you please review the checklist and do some more of those items? e.g. at least put your change in the change log.


feat: Updated ferry map

Summary of changes
Asana Ticket: Update source URL for ferry map
[[Please include a brief description of what was changed]](feat: Updated ferry map)

Before getting review, please check the following:

[x] Does frontend functionality render and work correctly in IE?
[x] Have we load-tested any new pages or internal API endpoints that will receive significant traffic? See load testing docs
[x] Are interactive elements accessible to screen readers?
[x] Have you checked for tech debt you can address in the area you're working in?
[x] If this change involves routes, does it work correctly with pertinent "unusual" routes such as the combined Green Line, Silver Line, Foxboro commuter rail, and single-direction bus routes like the 170?
[x] Are the changes organized into self-contained commits with descriptive and well-formatted commit messages?

@thecristen It was way worse than hard-coding the result in the test: the test was calling implementation code for the "expectation". But in CI it was referencing an environment that hadn't been released to you. I adjusted all those tests to just test the type of the expected result.

@thecristen Thanks! Missed that there were two of them.


Shared Worker

I tried looking at the code but somehow I'm not sure how to revise the implementation a bit with support for SharedWorker. There is some material here that might help.
https://github.com/GoogleChromeLabs/comlink/issues/485#issuecomment-720108900
Can this be tweaked to also allow instantiating a SharedWorker on runtime?

any updates?

I've made some modification which allow to spawn Shared Worker, currently testing it. But I don't know if it possible to integrate to comlink-loader or not.
https://github.com/socheatsok78/comlink-loader

any updates?


(#120) TCC

as per #120

Job #182 is now in scope, role is REV

This pull request #182 is assigned to @amihaiemil/z, here is why. The budget is 15 minutes, see §4. Please, read §27 and when you decide to accept the changes, inform @yegor256/z (the architect) right in this ticket. If you decide that this PR should not be accepted ever, also inform the architect.

weird... let me check the test failure in the build

@amihaiemil please review

Codecov Report

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

@@            Coverage Diff            @@
##             master     #182   +/-   ##
=========================================
  Coverage     74.79%   74.79%
  Complexity      133      133
=========================================
  Files            27       27
  Lines           964      964
  Branches         67       67
=========================================
  Hits            721      721
  Misses          213      213
  Partials         30       30

Continue to review full report at Codecov.

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

@amihaiemil I'm actually a bit confused by this puzzle. It said that we needed to take into account when two methods are related because they both call a third one. So I implemented that.
But through a cursory look at the PDF bieman95.pdf I found this:

The direct connectivity between methods is determined
from the class abstraction. If there exists one or more
common instance variables between two method abstractions then the two corresponding methods are directly
connected.

So it seems like TCC was already implemented? WDYT?

@amihaiemil I've read the paper and produced some notes. I'll be making some changes and let you know. Thank you for your patience

@amihaiemil please review

@llorllale

I'm actually a bit confused by this puzzle

Sorry, but I can't help you with details related to the metric itself. I don't know them. Here, I am paying more attention to the style, size, Java code in the PR etc and the puzzles.
Basically, since these are sandbox projects, the goal is for the new guys to learn how the model works. If there's a bug in a metric (which is highly expected, since nobody is an expert in theoretical software metrics here), it's not the end of the world. Plus, we have the ARCH, who is supposed to look at the things with more substance :)

@llorllale From my point of view, it looks good. Maybe, after the new puzzle is created by 0pdd, go there and link the notes that you've made here :)

@rultor merge

@rultor merge

@amihaiemil Thanks for your request. @yegor256 Please confirm this.

@amihaiemil my confusion stemmed from blindly following through with the puzzle's instructions. After reading the PDF for myself I realized that the instructions did not apply to TCC. So I wasted quite a lot of time on this one :)

@yegor256 please merge

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 10min)

@ypshenychka/z please review this job, as in §30

@ypshenychka Can you give the verdict here?

@amihaiemil Don't worry. I will review as soon as the issue https://github.com/zerocracy/farm/issues/592 is resolved.

@amihaiemil/z this job was assigned to you 5 days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@llorllale According to our QA Rules:

Pull request title explains the problem it is fixing.
Pull request description explains the solution proposed and contains a link to the original ticket it is related to.

Please provide more informative title and description.

@ypshenychka would the following suffice?

Title: (#120) TCC: advancing implementation
Description:

TCC(C) = NDC(C) / NP(C), where C is the class, NP(C) is a
maximal possible number of direct or indirect connections - N * (N - 1) / 2,
NDC(C) is a number of direct connections. Value of the metric is in range [0, 1],
greater is better.
See the bieman95.pdf paper for more details.

@llorllale

llorllale commented 5 days ago
weird... let me check the test failure in the build

According to our QA Rules:

Messages in a job always start with a name of a user they are addressed to.

Please correct your message by indicating an addressee in the beginning.

@llorllale Sure. Please go ahead and edit title\description.

@ypshenychka fixed title, description, and that unaddressed message you pointed out

@amihaiemil According to our QA Rules:

The code reviewer found at least three problems in the code.
Comments were mostly about design problems, not cosmetic issues.

No issues were found during code review. Please confirm that you'll try to find at least three problems while future reviews.

@llorllale Great. Thanks.

@ypshenychka right, I'll try to find more bugs in future tasks. But since these are sandbox projects, I am paying more attention to the methodology :)

@amihaiemil thank you

@0crat quality acceptable

QA review completed: +15 points just awarded to @ypshenychka/z, total is +240

Order was finished, quality was "acceptable": +15 points just awarded to @amihaiemil/z, total is +730

Payment to ARC for a closed pull request, as in §28: +15 points just awarded to @yegor256/z, total is +10274


Fixed logic on Winner page play again button & style issues

This PR fixes the broken play again button on winner page, brings in the uniform button styling from the sibling pages so that all buttons match and addresses issues of responsiveness and element bunching within winner-container.
This pull request is atomic in addressing only the issues of the winner page. Only two pages were adjusted.
PlayAgainButtonAnimation.js
REMOVED   element as wrapper to button fixing the warning message
ADDED className to button element and changed onclick to onClick fixing the navigation logic
WinnerPage.less
Multiple styling fixes to fix container size, padding, and Y axis adjustments on text and button
Play Again button now works sending user to /child/play-again

Good job fixing the button!


Post Install Question

GEE Team,
After running the two scons commands for the building of google earth enterprise, is there anything else to do or scripts to run inside the NATIVE-OPT-x86_64/ directory for building GEE?  After executing scons -j8 optimize=1 there were no GEE services created in init.d or anything created in /opt

@CLEAOSBORNE You then run the install scripts, which are in src/install. The server one isn't finished yet (see #106 if you want a mostly working version), but install_fusion.sh works pretty well.

@bradh Thanks Brad

Does this resolve your problem? If so, can you close this ticket?

unfortunately no.....just ran the commands and no dice.........see error below.  I ran scons -j3 installdir=/tmp/fusion_os_install install at src

Did you do the third_party and base build steps first? Those were only the install steps (after the build steps).

yes....I'm rerunning the script and it seems more promising.........I edited the command and following it now.   ran : scons -j3 installdir=/tmp/fusion_os_install

Hello @CLEAOSBORNE
I hope the latest response helped point you in the right direction. If you'd like to continue discussing this or any other issue I invite you to join the [Slack Chatroom] (http://slack.opengee.org). Since we have not heard back in a week this issue will be closed out but if you encounter further issues please bring it up in the slack room.


Draw chart from the right side

Hi everyone,
I've investigated documentation, but did't find a way how to draw something like these charts.
I know that question more for stackoverflow, but I believe community here can help me

It can be drawn externally and then add the drawn dom to the chart through createHtml.


Sidecar: Added matchers support

Fixes task 2: For Prometheus store (sidecar), add matchers to the LabelValues query of #3779
Changes
Implemented API /api/v1/status/buildinfo which returns the buildinfo of prometheus. Calling this API once when the sidecar starts and using it in pkg/store/prometheus.go labelValues method, if the version is > "2.24" used labelValues query otherwise fallback to series query.
Verification
Added unit tests

As suggested, removed the separate goroutine for fetching the Prometheus version and moved the code block to an existing goroutine.
The failing circleci test seems unrelated to me.

Need another look for this pr to merge. Maybe @yashrsharma44 @bwplotka ?

PHAL! @yeya24 @yashrsharma44

cc @yashrsharma44 @bwplotka for a final review. If there is no objection, I can merge it tomorrow.


Adapt the "Style guidelines" section of the style guide

Adapt and improve the Style guidelines section to work for the user and infrastructure documentation.

This issue relates to actions for the Nimbus project that are no longer needed.


External DependentResource with Updater cannot check if desired and actual match

If I have an dependentResource that extends from AbstractExternalDependentResource, but also from Updater, then the update will never be called when the Primary resource is updated. This is caused by a problem with the match method.
For example this dependent resource:
public class ExternalUserDependentResource
        extends PerResourcePollingDependentResource<ExternalUser, MyPrimary>
        implements Creator<ExternalUser, MyPrimary>,
                             Deleter<MyPrimary>,
                             Updater<ExternalUser, MyPrimary> {
    // ...
    @Override
    protected ExternalUser desired(MyPrimary primary, Context<MyPrimary> context) {
        return new ExternalUser(primary.getSpec().username());
    }

    @Override
    public ExternalUser create(ExternalUser desired, MyPrimary primary, Context<Prim> context) {
        externalSystem.create(desired.username());
        return desired;
    }

    @Override
    public ExternalUser update(ExternalUser actual, ExternalUser desired, MyPrimary primary, Context<MyPrimary> context) {
        externalSystem.update(desired.username());
        return desired;
    }

    @Override
    public Set<ExternalUser> fetchResources(MyPrimary primary) {
        return externalSystem
               .findUsers(primary.getSpec().username())
               .map(u -> new ExternalUser(u.username())
               .collect(toSet());
    }
}

This is caused by a method call to match that keeps calling itself:

AbstractDependentResource::match is called:

public Result<R> match(R resource, P primary, Context<P> context) {
    return updater.match(resource, primary, context);
}

It calls the match method on the field updater, but this is the same dependent resource (ExternalUserDependentResource in my example):

protected AbstractDependentResource() {
    creator = creatable ? (Creator<R, P>) this : null;
    updater = updatable ? (Updater<R, P>) this : null;

Stuck in a loop

Workaround:
Override match in the dependent resource and do the check yourself.

@BramMeerten I added a PR that might make this a bit more user friendly. There is no good solution for this IMO, the intention was that this is indeed overwritten. That is what the matcher method  in the Updater should indicate. But this is complicated with the Bulk resources, where the matcher has different footprint and you want that to be defaulted that case. Pls check the PR if that makes sense, it provides a default not recursive implementation.

@csviri Looks good to me
Having to override matcher is no problem for me, the main problem that I had was that it wasn't working without any error that indicated what was wrong.
Somehow there was no stackoverflowexception because of infinite recursion, the recursion just stopped after a while.
And this PR fixes the infinite recursion, thx!


feat(playground): vue-router template

I used vue-router to improve the Playground project, and I believe it is necessary.

 That show the great power of community !!
Thanks a lot for your contribution ~


Use CARGO_CFG_TARGET_OS where needed

Fixes #2435.

Coverage remained the same at 81.967% when pulling b404989485a36f6c8fdd59a11a74993f79a709ee on rust-av:unbreak-cross-compiling-on-macos into ed4d6781416a0a51db7eb54e555515abdf40e38c on xiph:master.


feat: add newrelic agent to web service

Summary:
Resolves #1372
Expected behavior:
Explain and/or show screenshots for how you expect the pull request to work in your testing (in case other devices exhibit different behavior).
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Run the unit tests with gradle test to make sure you didn't break anything
[x] Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
[x] Linked all relevant issues
[ ] Include screenshot(s) showing how this pull request works and fixes the issue(s)

@KClough I see this is still a draft, what's the update? This is the last issue before we publish a release 🚀


chore(deps): upgrade Web3 to v1.5.2

Hoping that this will fix a good chunk of the outstanding
security vulnerabilities that we have.
Signed-off-by: Peter Somogyvari<EMAIL_ADDRESS>

Codecov Report

Merging #1323 (94cc262) into main (a01ce63) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1323      +/-   ##
==========================================
- Coverage   71.49%   71.47%   -0.02%
==========================================
  Files         298      298
  Lines       10743    10743
  Branches     1321     1321
==========================================
- Hits         7681     7679       -2
- Misses       2364     2366       +2
  Partials      698      698

Impacted Files
Coverage Δ

...main/typescript/rustc-container/rustc-container.ts
66.29% <0.00%> (-2.25%)
:arrow_down:

Continue to review full report at Codecov.

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


Permission ID showing group description

Current behavior

Individual permission cards are showing the description of the whole group. So all permission in the admin section are showing the description of the admin security group.
Expected behavior

Should show the description of the permission id no the group

Verified and working fine in release v1.5.2.
https://jam.dev/c/d4e98322-6a20-46c6-9440-0056f3aa48de


[FEATURE] Multi Login Backend

It would be nice to implement multiple, parallel login backends. That you can use LDAP und DB together.
We need to use LDAP for our internal use and DB-Users to deliver container to external persons.

@arogarth it sounds very similar to #801 (which we will provide for 2.4). Is that what you mean? If so, we can close this as a duplicate :wink:

I'm currently working on #801 and I now I see that this issue has a slight twist that might be useful to implement in the near future. I'll keep it as an improvement from my work on #801 that should come in 2.5. Thanks for the feedback :+1:


How to tell a component to inject it's css at the end

I have created a component that inject all my CSS utilities, this component wrap my application like this :
    <Provider store={store}>
      <LanguageProvider messages={messages}>
        <BootstrapProvider
          utils={{
            align: true,
            background: true,
            border: true,
            clearfix: true,
            display: true,
            flex: true,
            float: true,
            position: true,
            screenreaders: true,
            sizing: true,
            spacing: true,
            text: true,
            visibility: true,
          }}
        >
          <Router
            history={history}
            routes={rootRoute}
            onUpdate={hashLinkScroll}
            render={
              applyRouterMiddleware(useScroll())
            }
          />
        </BootstrapProvider>
      </LanguageProvider>
    </Provider>,

I only use injectGlobal for settings the html and body for the font.
I have noticed that the css of this component is injected before all overs in the html.
This make my utility less prioritized. I want to add them at the end, is there a chance to see this in a future version ?
The only fix I have for the CSS is to use !important which is not recommended.

@kopax see https://github.com/styled-components/styled-components/issues/293, https://github.com/react-boilerplate/react-boilerplate/issues/1589 and https://github.com/webpack/webpack/issues/215
As you can see in the linked issues, the problem is with webpack not being able to ensure the order of CSS inclusion. I think the recommended way for now is to statically add any vendor CSS in your HTML to ensure it's included before styled components. I realize this doesn't play well with your current solution, but AFAIK, there is no other way atm. (except the usage of !important or any other method that makes the specificity higher).

The only fix I have for the CSS is to use !important which is not recommended.

It's perfectly fine for utilties. (See https://csswizardry.com/2016/05/the-importance-of-important/#forcing-immutability-with-important)

@diondiondion I rather use !important on the last layer of css.
@mxstbr would it be possible to see this feature one day, or does it purely depend on webpack ?

It's not really related to webpack. Styles are injected when the component is mounted. Also, if you have dynamic styles,
styled.div`
  color: ${props => props.color};
`

you'll generate new styles every time the color prop changes. Every time you inject styles, they're injected at the end. There's no real way to control the order in styled-components.
If !important on your utility classes fixes it for you, don't feel bad! Bootstrap and CSS Wizardry (to name a few) all think this is the way to go—you’re using them to override default styles!

Yeah to control that would mean to break styled-components, we sadly cannot allow you to do that I'm afraid 😕 Because you're rendering BootstrapProvider as the first styled component in the entire app, we inject its CSS first, which is why its at the top.
If we removed that behaviour there'd be a lot of ordering bugs based on the fact that your CSS code will change based on where and when you render your components, which is incredibly frustrating and hard to debug, so there's no way we'll allow that in the future. Sorry!

@mxstbr from tech to tech, how hard would it bo to implement it on a fork ?

@mxstbr I am not asking to remove that behavior, but if it's possible to add it as an extended features for some component. If it's used wisely, this can be a powerful feature.
For example using
const MyWrapper = styled.div`
   & .fixed-top { position: fixed; top: 0; }
`
MyWrapper.styleOrdering = 0;

This of course doesn't have to be enabled by default but will allow developers to customize the dependency injection of their components.
I don't even know if it's possible and how hard would it be, would you mind sharing a more technical explanation.


Trailing slash matching

https://github.com/unjs/radix3/blob/83724b4ee821fb43463598744f05052fdf8eaf97/tests/lookup.test.ts#L95

This does not work when the route is something like:
/editor/:site_id/:page_id
when I visit /editor, my expectation is that it will match with site_id and page_id filled with empty string.

@rizrmd Can you please make a new issue with reproduction so i not forget to check? 🙏🏼


Icon is too Big compared to other ones (macOS)

Screenshots

Desktop (please complete the following information):

OS: MacOS

A new release solving this issue has just been published!


Set form definition for binding parameters.

Form definition for bind parameters were not being set, rendering input fields out of order set by brokers.

 (8) Bind Parameters

@jeff-phillips-18 FYI, we should also use the form definition when we show bind parameters


SourceMapDevToolPlugin does not work - ignored

After https://github.com/zeit/next-plugins/pull/350 (using optimize-css-assets-webpack-plugin to minify css files), webpack.SourceMapDevToolPlugin is ignored (https://github.com/NMFR/optimize-css-assets-webpack-plugin/issues/99), that means that is not possible to create custom sourcemaps for CSS files

workaround:
in webpack config:
    const filteredMinimizers = config.optimization.minimizer.filter(
      (plugin) => plugin.constructor.name !== 'OptimizeCssAssetsWebpackPlugin'
    );
    config.optimization.minimizer = filteredMinimizers;

And use postcss to minify css files.

Hi, thanks for creating an issue. We currently recommend using https://nextjs.org/docs/basic-features/built-in-css-support as the plugins have been deprecated in favor of the built-in support.