torch.mm hung

🐛 Describe the bug
The following code some times hung and sometimes get Native API failed. Native API returns: -2 (PI_ERROR_DEVICE_NOT_AVAILABLE) -2
Reproduce:
stock Pytorch main branch
dirvier LTS 803.29
Run the following script:
import torch
from torch import device, empty_strided

if __name__ == "__main__":
    from torch._dynamo.testing import rand_strided

    s0 = 16777472

    # if use a smaller s0, no problem happends.
    # s0 = 167772

    mat1 = rand_strided((2*s0, 4), (4, 1), device='xpu', dtype=torch.float32)
    mat2 = rand_strided((2, 4), (4, 1), device='xpu', dtype=torch.float32)
    mat2_r = torch.as_strided(mat2, (4, 2), (1, 4), 0)
    mat2_o = rand_strided((4, 2), (2, 1), device='xpu', dtype=torch.float32)

    out = empty_strided((2*s0, 2), (2, 1), device='xpu', dtype=torch.float32)
    # This line will hung or get Native API failed. Native API returns: -2 (PI_ERROR_DEVICE_NOT_AVAILABLE) -2 (PI_ERROR_DEVICE_NOT_AVAILABLE)
    torch.mm(mat1, mat2_r, out=out)

    # This can pass even with s0 = 16777472
    # torch.mm(mat1, mat2_o, out=out)

Versions
stock Pytorch main branch
dirvier LTS 803.29,803.61

Has reported the issue to oneDNN team


this disable button click event

Windows 10, .net4.7.2.
C# WPF with Visual Studio 2019.
below XMAL file:

<control:HexEditor x:Name="HexEdit" Width="Auto" Height="Auto"/>

no click event on the button.
remove HexEditor controller, button works OK.

On file ?? in sample projet ?

I create a new project, XAML file is:

<control:HexEditor x:Name="HexEdit" Width="Auto" Height="Auto"/>

and codes is:
filebtn.Click += (s, e) => {
var fileDialog = new OpenFileDialog();
            if (fileDialog.ShowDialog() != null && File.Exists(fileDialog.FileName))
            {
                Application.Current.MainWindow.Cursor = Cursors.Wait;

                HexEdit.FileName = fileDialog.FileName;

                Application.Current.MainWindow.Cursor = null;
            }
        };
    }

but when i click the button, nothing happened. if i remove HexEditor, then it works on button.

Do you have taken the latest code on github ?

it always remove the button line code when i submit this issue, again below is the XAML codes:

<control:HexEditor x:Name="HexEdit" Width="Auto" Height="Auto"/>

yes, i just cloned the latest codes from here.

do you have check the sample project ?
i will try your code right now 😄

sure.
i build the sample project and seems that it works ok.
but i load file from menu, i just want to have a try through a button.

it's work fine:
using Microsoft.Win32;
using System.IO;
using System.Windows;
using System.Windows.Input;
namespace WpfApp1
{
public partial class MainWindow : Window
{
    public MainWindow()
    {
        InitializeComponent();

        filebtn.Click += (s, e) =>
        {
            var fileDialog = new OpenFileDialog();

            if (fileDialog.ShowDialog() != null && File.Exists(fileDialog.FileName))
            {
                Application.Current.MainWindow.Cursor = Cursors.Wait;

                HexEdit.FileName = fileDialog.FileName;

                Application.Current.MainWindow.Cursor = null;
            }
        };
    }
}

}

XAML:

<Grid Margin="0,40,0,0">
    <control:HexEditor
        x:Name="HexEdit"
        Width="Auto"
        Height="Auto" />
    <Button
        x:Name="filebtn"
        Width="75"
        Margin="11,-34,0,0"
        HorizontalAlignment="Left"
        VerticalAlignment="Top"
        Content="Button" />
</Grid>

please try below codes:

<control:HexEditor
x:Name="HexEdit"
Width="Auto"
Height="Auto" />
    </StackPanel>
    <StackPanel>
        <Button
    x:Name="filebtn"
    Width="75"
    Margin="11,-34,0,0"
    HorizontalAlignment="Left"
    VerticalAlignment="Top"
    Content="Button" />
    </StackPanel>
</Grid>

yes, it works ok on your pasted codes. but it doesn't work within stackpanel.

Try this 👍

If you like my project hit the ⭐ button please 😄

thanks a lot.


Would really appreciate a dx 11 version

I got this request multiple times now. And it shouldn't be a big deal. So yes, I will work on one soon.

Done. If it doesn't work, re-open please.


Godot doesn't crop 8-pixel overscan area when playing Ogg Theora video

Godot version
3.4.4, 3.5.rc5, 4.0.alpha11
System information
Fedora 36, GeForce GTX 1080 (NVIDIA 510.68.02)
Issue description
Godot doesn't crop the 8-pixel overscan area at the bottom when playing an Ogg Theora video in a VideoStreamPlayer node (VideoPlayer in 3.x). This is most noticeable when the video plays, rather than when it's paused, so I recommend trying out the MRP locally if you can't notice the issue on the screenshots.
Notice how the last 8 rows of pixels are duplicated in Godot, but not in mpv:
Screenshot from Godot (bad)
Cropped to show only the video area (896×512).

Screenshot from mpv (good)
896×504.

This happens even if I re-encode the video using FFmpeg using ffmpeg -i Big_Buck_Bunny_medium.ogv bb.ogv.
ffprobe information (on original video file)
[ogg @ 0x55c00d5965c0] Broken file, keyframe not correctly marked.
Input #0, ogg, from '/tmp/test_ogg_theora/Big_Buck_Bunny_medium.ogv':
  Duration: 00:09:56.47, start: 0.000000, bitrate: 1317 kb/s
  Stream #0:0: Data: none
  Stream #0:1: Video: theora, yuv420p, 896x504, 24 tbr, 24 tbn
    Metadata:
      ENCODER         : ffmpeg2theora-0.25
      SOURCE_OSHASH   : cc9e38e85baf7573
  Stream #0:2: Audio: vorbis, 48000 Hz, stereo, fltp, 96 kb/s
    Metadata:
      ENCODER         : ffmpeg2theora-0.25
      SOURCE_OSHASH   : cc9e38e85baf7573
Unsupported codec with id 0 for input stream 0

Steps to reproduce

Add VideoStreamPlayer node and specify an Ogg Theora video file.
Disable Expand on the VideoStreamPlayer.
Run the project and look at the video's bottom pixels.
Play the video in another video player such as mpv or VLC, and compare the visual output.

Minimal reproduction project
https://0x0.st/olc3.zip (link expires in July 2023)

i believe i ran into this issue in 4.2.dev5, with 12 extra pixels on the bottom and right sides.
Cropping the video to its original resolution using a parent Control node with Clip Contents enabled seems to be a good workaround though.

I also encountered similar problems. My godot version is 4.1.1. The right and bottom sides of my video in godot are blurry.Here are my ffmpeg command: ffmpeg -i xx.mp4 -q:v 10 -q:a 10 xx.ogv. I have recorded the video using mkv format, and then using ffmpeg: ffmpeg -i xx.mkv -q:v 10 -q:a 10 xx.ogv.Amazingly, there are no issues with the ogv video anymore.

Same here in 4.2.1.
Used ffmpeg -i input.mp4 -c:v libtheora -q:v 7 output.ogv

This got me when playing a fullscreen video @ 1920x1080 in 4.2.2, and the VideoStreamPlayer and parent AspectRationContainer take a forced transform of size: 1920x1088.


Embed info object into a path

Consider the following situation:
# schema.graphql
Query {
  getA: A
}

type A {
  id: ID!
}

# our query
query {
  getA {
    id
  }
}

# remote.graphql
Query {
  getB: B
}

type B {
  a: A
}

type A {
  id: ID!
}

// resolver
getA(parent, args, ctx, info) {
  // this doesn't work, because we return `B` but query for `A`
  return ctx.remote.query.getB({ }, info)

  // this works, but doesn't use `info`. for more complicated queries, this becomes unmaintainable.
  const b = await ctx.remote.query.getB({ }, `{ a { id } }`)
  const a = b.a
  return { ...a }

  // ideally, we could "embed" the `info` object into a query
  return ctx.remote.query.getB({ }, embed(info, `{ a { $info } }`)
},

Visual explanation.
We want to get from
.
└── getA
    ├── id
    └── anotherField

to
.
└── getB
    └── a
        ├── id
        └── anotherField

There are a couple of GraphQL primitives to consider before going for this approach:
Fragments
Inline Fragments
If the original query contains inline fragments, they need to be properly copied over to the new resulting info object.
Global Fragment definitions + fragment spread
Not just the normal selection set must be taken care of, but also globally defined fragments on types (and all related fragments) + the fragment spread that is being used in the relevant sub-selection.
Variables
The resulting query must include the incoming variables, but still support extending the variables with the bindings.
Arguments
The arguments and variables need to be properly propagated to the resulting query document.
A big challenge here is how to provide arguments for sub fields.
Aliases
As soon as aliases are involved, the response cannot be storngly typed anymore, as a field could have any name.
Name clashes
Both in the delegation as in variables/args, there could be name clashes.
We need a clear definition, which API gets precedence over the other.

Thank you for reporting.
In the last few months, since the transition of many libraries under The Guild's leadership, We've reviewed and released many improvements and versions to graphql-cli, graphql-config and graphql-import.
We've reviewed graphql-binding, had many meetings with current users and engaged the community also through the roadmap issue.
What we've found is that the new GraphQL Mesh library is covering not only all the current capabilities of GraphQL Binding, but also the future ideas that were introduced in the original GraphQL Binding blog post and haven't come to life yet.
And the best thing - GraphQL Mesh gives you all those capabilities, even if your source is not a GraphQL service at all!
it can be GraphQL, OpenAPI/Swagger, gRPC, SQL or any other source!
And of course you can even merge all those sources into a single SDK.
Just like GraphQL Binding, you get a fully typed SDK (thanks to the protocols SDKs and the GraphQL Code Generator), but from any source, and that SDK can run anywhere, as a connector or as a full blown gateway.
And you can share your own "Mesh Modules" (which you would probably call "your own binding") and our community already created many of those!
Also, we decided to simply expose regular GraphQL, so you can choose how to consume it using all the awesome fluent client SDKs out there.
If you think that we've missed anything from GraphQL Binding that is not supported in a better way in GraphQL Mesh, please let us know!

In the context of that particular issue - I would love to hear your thought on our transform API and the currently available transforms to see how your query would look like with GraphQL Mesh.
Please feel free to open a new issue on the GraphQL Mesh repo.
We're looking forward for your feedback of how we can make your experience even better!


Memory Leaks Cause Server OOMs (CPU, TF2/ONNX)

Description
Using latest Triton v22.01 with ONNXBackend (self-built) or with the TensorFlow Backend (image from Nvidia Container Registry) and performing the CPU inference there seems to be a memory leak: memory usage increases with each request, finally leading to an OOM for the Triton Inference Server instance.
Triton Information
What version of Triton are you using? v22.01
Are you using the Triton container or did you build it yourself? Both the Triton container version and custom ONNX build are affected.
To Reproduce
Steps to reproduce the behavior.

Run the Inference server to host the CRAFT model either in a model.savedmodel format (for TensorFlow backend) or model.onnx format.
Perform multiple inferences using tritonclient[all]==2.18.0.
Watch the memory usage of Triton Inference Server to increase (locally using docker stats).

Model configuration file:
name: "craft"
backend: "onnxruntime"
max_batch_size: 1
input [
  {
    name: "input_2"
    data_type: TYPE_FP32
    dims: [-1, -1, 3]
  }
]
output [
  {
    name: "conv_cls.8"
    data_type: TYPE_FP32
    dims: [-1, -1, 2]
  }
]
instance_group {
  kind: KIND_CPU
  count: 1
}
model_warmup {
    name: "CRAFT Warmup"
    batch_size: 1
    inputs: {
        key: "input_2"
        value: {
            data_type: TYPE_FP32
            dims: [1024, 1024, 3]
            zero_data: false
        }
    }
}

Example tritonclient usage (Python 3.10):
        import tritonclient.grpc as grpcclient
        from tritonclient.grpc import InferenceServerClient, InferenceServerException

        client = InferenceServerClient(
            url=TRITON_GRPC_SERVICE,
        )
        TRITON_CRAFT_MODEL_NAME = "craft"
        TRITON_CRAFT_MODEL_VERSION = "1"
        TRITON_CRAFT_MODEL_INPUT_NAME = "input_2"
        TRITON_CRAFT_MODEL_OUTPUT_NAME = "conv_cls.8"

        input = grpcclient.InferInput(
            name=TRITON_CRAFT_MODEL_INPUT_NAME,
            shape=[1, input_image.shape[0], input_image.shape[1], input_image.shape[2]],
            datatype="FP32",
        )
        output = grpcclient.InferRequestedOutput(TRITON_CRAFT_MODEL_OUTPUT_NAME)
        input.set_data_from_numpy(np.array([input_image]))

         response = self.client.infer(
                model_name=TRITON_CRAFT_MODEL_NAME, inputs=[input], outputs=[output],
                client_timeout=TRITON_GRPC_TIMEOUT
            )
        result = response.as_numpy(name=TRITON_CRAFT_MODEL_OUTPUT_NAME)

Example output of Triton Inference Server with --log-verbose=1 flag:
I0217 15:49:51.941706 1 grpc_server.cc:3206] New request handler for ModelInferHandler, 89
I0217 15:49:51.941713 1 model_repository_manager.cc:590] GetModel() 'craft' version -1
I0217 15:49:51.941723 1 model_repository_manager.cc:590] GetModel() 'craft' version -1
I0217 15:49:51.941744 1 infer_request.cc:566] prepared: [0x0x7f43c401c490] request id: , model: craft, requested version: -1, actual version: 1, flags: 0x0, correlation id: 0, batch size: 1, priority: 0, timeout (us): 0
original inputs:
[0x0x7f43c401c748] input: input_2, type: FP32, original shape: [1,1024,1024,3], batch + shape: [1,1024,1024,3], shape: [1024,1024,3]
override inputs:
inputs:
[0x0x7f43c401c748] input: input_2, type: FP32, original shape: [1,1024,1024,3], batch + shape: [1,1024,1024,3], shape: [1024,1024,3]
original requested outputs:
conv_cls.8
requested outputs:
conv_cls.8

I0217 15:49:51.941788 1 onnxruntime.cc:2427] model craft, instance craft_0, executing 1 requests
I0217 15:49:51.941796 1 onnxruntime.cc:1334] TRITONBACKEND_ModelExecute: Running craft_0 with 1 requests
2022-02-17 15:49:51.941856332 [I:onnxruntime:log, bfc_arena.cc:306 AllocateRawInternal] Extending BFCArena for Cpu. bin_num:20 (requested) num_bytes: 549453824 (actual) rounded_bytes:549453824
2022-02-17 15:49:51.941888464 [I:onnxruntime:log, bfc_arena.cc:186 Extend] Extended allocation by<PHONE_NUMBER> bytes.
2022-02-17 15:49:51.941898397 [I:onnxruntime:log, bfc_arena.cc:189 Extend] Total allocated bytes:<PHONE_NUMBER>
2022-02-17 15:49:51.941904942 [I:onnxruntime:log, bfc_arena.cc:192 Extend] Allocated memory at 0x7f435bffe040 to 0x7f439bffe040
2022-02-17 15:49:51.959883920 [I:onnxruntime:, sequential_executor.cc:155 Execute] Begin execution

The last part is especially important, since it seems that the ONNX Runtime (in this case) allocates an additional 1 GB+ of memory:
2022-02-17 15:49:51.941888464 [I:onnxruntime:log, bfc_arena.cc:186 Extend] Extended allocation by<PHONE_NUMBER> bytes.
2022-02-17 15:49:51.941898397 [I:onnxruntime:log, bfc_arena.cc:189 Extend] Total allocated bytes:<PHONE_NUMBER>
2022-02-17 15:49:51.941904942 [I:onnxruntime:log, bfc_arena.cc:192 Extend] Allocated memory at 0x7f435bffe040 to 0x7f439bffe040
2022-02-17 15:49:51.959883920 [I:onnxruntime:, sequential_executor.cc:155 Execute] Begin execution

OOM happens after the last log.
Expected behavior
Nvidia Triton Inference Server should not allow for the OOM. Memory should be freed after the inference request is performed.
How can I fix this issue?

@askhade  Can we expose above options in onnxruntime backend? Do you have any other solutions?

There are some examples in the https://github.com/triton-inference-server/server/tree/main/deploy you can check them out.

@Tabrizian thank you! 🙏
I can also confirm that using the fixed-size inputs for the model helped to achieve stable memory usage in my tests on development and production environments.
Still, the configuration option of memory usage strategy for ONNXRuntime on CPU builds would be beneficial.
Right now, I observe the following:

After model warmup (1 request, fixed-size input, batch size 1, fullconfiguration posted here) our Triton instance uses ~600 MB of memory
After another request (same size of input, but non-random data) memory usage increases to ~1.2 GB
After several more requests, memory usage stabilizes at ~1.5 GB

Thus, I presume that replacing the default arena_extend_strategy=kNextPowerOfTwo with a less aggressive one for ONNXRuntime would result in a reduction of memory usage of Triton, which could be really beneficial.
I'll be looking forward to your thoughts on that. Also, thank you for being so helpful and responsive - kudos 🎉

@Tabrizian @tanmayv25 @askhade do you happen to have any updates regarding the reported issue or any feedback on the observations I've made? 🙏
While the fixed-size inputs have stabilized the memory usage of my Triton instance with ONNXRuntime, sadly it increased the average inference time of our CRAFT model - since I've found no other way to fix input size of CNN than to scale up/down images and add padding so that the input image is always a rectangle.
Do you happen to have any other proposals on fixing input size that will not negatively affect the inference time? I must say, that I am a bit bummed out about the current state of my Triton-based project, any help would be appreciated 💯

I've not reproduced this yet. I'll try to do that today and report back if my findings differ. As for the configs, we've 3 options:

Disable the arena altogether for CPU. This should be easy to expose in Triton via 'parameters'. There're some obvious downsides to disabling the arena, but it really depends on your workload.
Expose the arena config options. This can be done in 2 ways.

ORT doesn't allow per session arena configs, hence they cannot be exposed in 'parameters'. ORT allows the user to configure and register a CPU allocator with custom arena cfgs, but this allocator is then registered at the env level (with the option to use/not use the env registered allocator at a per session level). Since the env is created only once in the process lifecycle the arena cfg options will have to be exposed as cmd line arguments like this in addition to exposing this via 'parameters' to allow users to actually use this customized allocator.
Another option is for ORT to allow users to configure arena cfgs at the session level after which these can be exposed in Triton via 'parameters'. This requires some work in both ORT and Triton ORT backend.

Option #1 is the fastest and something you can try right now by changing onnxruntime.cc and adding this line of code THROW_IF_BACKEND_MODEL_ORT_ERROR(ort_api->DisableCpuMemArena(soptions)); here.

Option #1 is merged to master. This should definitely help. We'll look into seeing how other arena params can be exposed after this.

@pranavsharma thank you for tremendous help 🎉
I will experiment with these parameters and let you know if they help us fix the memory management issues for model inference on CPUs.

@pranavsharma I can now confirm that setting the enable_mem_area and enable_mem_pattern flags to 0 based on changes introduced in https://github.com/triton-inference-server/onnxruntime_backend/pull/106 has successfully resolved our memory management issues, as confirmed by my internal tests on various pre-prod and prod environments.
This chart shows the memory usage before and after the addition of flags you've introduced to the model's configuration:

I'm anxious to see whether onnxruntime==1.11.0 will introduce any further memory management-related improvements. However, it seems that the issue can be resolved when latest onnxruntime-backend is released with the config options exposed (I've prepared my own build for tests).
Thank you so much for your help! 💯 You guys are amazing!

ORT 1.11 does have some memory management related improvements. See here https://github.com/microsoft/onnxruntime/releases/tag/v1.11.0.

@pranavsharma Thank you! Is it safe to use ORT 1.11 with onnxbackend r22.04, or should I wait for a newer release of the latter? The compatibility matrix states that I should use ORT 1.10.

There are some issues with r22.04 and ORT 1.11 combination. I would wait for a latter release of Triton. I fixed a bug yesterday that was preventing ORT 1.11 upgrade in Triton.

@pranavsharma I've tested thoroughly ORT 1.10 with tritonserver 22.03 and triton-onnxbackend 22.04 and, by using the new memory flags, I have successfully resolved a memory leak.
I've now upgraded by image to tritonserver 22.06, triton-onnxbackend 22.06 and onnxruntime 1.11 and I do not see any regression.


start_http_server fails when given an IPv6 address to bind

It would be good not to force an IPv4 fallback.

What was the exact code you used?

% python                                                                                [master]
Python 3.5.0 (default, Sep 14 2015, 02:37:27)
[GCC 4.2.1 Compatible Apple LLVM 6.1.0 (clang-602.0.53)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import prometheus_client
>>> prometheus_client.start_http_server(port=9097, addr='::')
>>> Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/local/Cellar/python3/3.5.0/Frameworks/Python.framework/Versions/3.5/lib/python3.5/threading.py", line 923, in _bootstrap_inner
    self.run()
  File "/Users/alynn/Thread/hamon/venv/lib/python3.5/site-packages/prometheus_client/exposition.py", line 64, in run
    httpd = HTTPServer((addr, port), MetricsHandler)
  File "/usr/local/Cellar/python3/3.5.0/Frameworks/Python.framework/Versions/3.5/lib/python3.5/socketserver.py", line 443, in __init__
    self.server_bind()
  File "/usr/local/Cellar/python3/3.5.0/Frameworks/Python.framework/Versions/3.5/lib/python3.5/http/server.py", line 138, in server_bind
    socketserver.TCPServer.server_bind(self)
  File "/usr/local/Cellar/python3/3.5.0/Frameworks/Python.framework/Versions/3.5/lib/python3.5/socketserver.py", line 457, in server_bind
    self.socket.bind(self.server_address)
socket.gaierror: [Errno 8] nodename nor servname provided, or not known

That looks more like an issue down in cPython, we're passing the addr straight down.

Vote to close.


"help wanted" I cannot get the TensorBoard working, I am following the given tutorial

**I am having trouble writing the log file to be able to visualize it in TensorBoard. Can somebody point me in the right direction? I have followed the code given in the tutorial. When run I get the error: **

"expected %s got %s." % (cls.name, type(msg).name))
TypeError: Parameter to MergeFrom() must be instance of same class: expected GraphDef got Graph. for field Event.graph_def*

The error is said to be on line:
summary_writer = tf.train.SummaryWriter('/Users/Cristian/Desktop/log', sess.graph)

My code:
import input_data
import tensorflow as tf

print "Start program"
mnist = input_data.read_data_sets("MNIST_data/", one_hot=True)
print "Done loading images"

x = tf.placeholder("float", [None, 784])
W = tf.Variable(tf.zeros([784,10]))
b = tf.Variable(tf.zeros([10]))

y = tf.nn.softmax(tf.matmul(x,W) + b)

y_ = tf.placeholder("float", [None,10])

print "Done init"

cross_entropy = -tf.reduce_sum(y_*tf.log(y))
train_step = tf.train.GradientDescentOptimizer(0.01).minimize(cross_entropy)

print "Done loading vars"

init = tf.initialize_all_variables()
print "Done init vars"

sess = tf.Session()
sess.run(init)

print '------'
print type(sess.graph_def)

correct_prediction = tf.equal(tf.argmax(y,1), tf.argmax(y_,1))
accuracy = tf.reduce_mean(tf.cast(correct_prediction, "float"))

acc = tf.scalar_summary("Accuracy:", accuracy)

merged_summary_op = tf.merge_all_summaries()
summary_writer = tf.train.SummaryWriter('/Users/Cristian/Desktop/log', sess.graph)

print "Session started"

for i in range(2000):
  batch_xs, batch_ys = mnist.train.next_batch(100)
  sess.run(train_step, feed_dict={x: batch_xs, y_: batch_ys})
  if i % 100 == 0:
    print sess.run(accuracy, feed_dict={x: mnist.test.images, y_: mnist.test.labels})
    summary_str = sess.run(merged_summary_op)
    summary_writer.add_summary(summary_str, i)

print "Done training"

correct_prediction = tf.equal(tf.argmax(y,1), tf.argmax(y_,1))

accuracy = tf.reduce_mean(tf.cast(correct_prediction, "float"))

print sess.run(accuracy, feed_dict={x: mnist.test.images, y_: mnist.test.labels})

sess.graph is a python tf.Graph object, whereas SummaryWriter takes a tf.GraphDef (proto).  I think you can get further if you use sess.graph_def rather than sess.graph.  Let us know if that helps!

Also, consider using StackOverflow for these kinds of help questions -- you'll probably get better community support there.
(See https://github.com/tensorflow/tensorflow/blob/master/tensorflow/g3doc/resources/index.md#help--support--how-do-i- )


Cannot extract custom headers

I was wondering why I cannot extract headers when calling an api endpoint via cross domain. XHR package wasn't spitting out any errors on the console. I was only able to figure out the problem or show the error on the console when I used native XMLHttpRequest. Is this a bug in XHR package?

Please include browser name and version.
Would you be so kind to share some code too to let me reproduce?
I think it could be a cross-domain request issue or just a misunderstanding or documentation issue.
It's been always working for me (except IE8/9 where you need extra options) but if it's not, I want to fix it asap.

It is a cross domain issue. But I was just wondering why XHR package wasn't spitting out the error. If it spitted the error, I could have fixed it quickly. Sure, here is the code
The browser is Chrome and version is Version 55.0.2883.95 (64-bit) on Mac OSX.
https://gist.github.com/v1rt/3c4d8c4680638127ed930637ceac7b95

I'll look into it and see what I can do and if it can be considered a bug.
Access-Control-Expose-Headers is a must and as far as I remember, XMLHttpRequest was not calling the right callbacks for errors when trying to access headers failed for a cross-domain request.
If xhr is to blame, it could be swallowing the exception on reading headers, but that's surprising... I remember testing that between old IEs and newer browsers.
I have some time reserved for opensource on Fridays, so I should be able to get to it before the weekend.

Thanks a lot!

Ok, I looked into it and the thing is - if you use XMLHttpRequest straight and call the function that gets a particular header, you trigger an error log in the console to let you know you weren't allowed.
But that's not what response.headers[name] is doing. response.headers is a plain array that we get from listing all available headers. I'd probably have to rewrite to proxy objects to be able to call a function when an array is requested. Not enough browser support for that anyway.
Avoiding errors and returning undefined is correct behavior, xhr is not swallowing any errors, it's just aiding the user in not triggering the error in the first place.
I know it's not helpful if you forget to expose a header, but that's the compromise.
In your example, if you tried response.rawRequest.getResponseHeader('x-name') you would see the same error. xhr is using getAllResponseHeaders to create an array of everything that's available.
TL;DR: not a bug

Ok, I'll try this response.rawRequest.getResponseHeader('x-name') and I'll let you know. Thanks!


採点

初期状態
================================================================
Congratulations! All Scoring Process Successfully Done!!
    Score:                  68

    Total requests:        898
      Success:              50
      Fail:                  0
      Timeout:             848
================================================================

画像縮小後
    ================================================================
        Congratulations! All Scoring Process Successfully Done!!

        Score:                 192

        Total requests:        928
          Success:             111
          Fail:                  0
          Timeout:             817
    ================================================================
    ```

画像のbase64への変換処理を効率化
あまり変わらかなっかた・・・
================================================================
    Congratulations! All Scoring Process Successfully Done!!

    Score:                 202

    Total requests:        926
      Success:             112
      Fail:                  0
      Timeout:             814
================================================================

ログイン時のsha256の計算が非効率 #5
変わらず
================================================================
Congratulations! All Scoring Process Successfully Done!!
    Score:                 217

    Total requests:        927
      Success:             109
      Fail:                  0
      Timeout:             818
================================================================

git pullして再実行
================================================================
    Congratulations! All Scoring Process Successfully Done!!

    Score:                 373

    Total requests:       1001
      Success:             229
      Fail:                  0
      Timeout:             772
================================================================

        Congratulations! All Scoring Process Successfully Done!!

        Score:                 196

        Total requests:        950
          Success:             151
          Fail:                  0
          Timeout:             799
    ================================================================

    ================================================================
        Congratulations! All Scoring Process Successfully Done!!

        Score:                 379

        Total requests:       1008
          Success:             244
          Fail:                  0
          Timeout:             764
    ================================================================

session高速化 #9 + match-groups高速化
    Congratulations! All Scoring Process Successfully Done!!

    Score:                 353

    Total requests:       1005
      Success:             236
      Fail:                  0
      Timeout:             769
================================================================

indexをちゃんと反映させた結果です。
================================================================
    Congratulations! All Scoring Process Successfully Done!!

    Score:                1237

    Total requests:       1203
      Success:             589
      Fail:                  0
      Timeout:             614
================================================================

update: randomのソートクエリを速度改善

https://github.com/yunixuma/42HoursTuning2023/commit/fad6643776d59a99d0fad6346795ac5e1ca57bb9
Congratulations! All Scoring Process Successfully Done!!

Score:                1873

Total requests:       1314
  Success:             793
  Fail:                  0
  Timeout:             521

================================================================

getUsersByUserIds 最適化
================================================================
    Congratulations! All Scoring Process Successfully Done!!

    Score:                1954

    Total requests:       1325
      Success:             802
      Fail:                  0
      Timeout:             523
================================================================

ルール変更後

    ================================================================
        Congratulations! All Scoring Process Successfully Done!!

        Score:                1934

        Total requests:       1271
          Success:             746
          Fail:                  0
          Timeout:             525
    ================================================================
    ```

Docker再起動後
    ================================================================
        Congratulations! All Scoring Process Successfully Done!!

        Score:                1942

        Total requests:       1270
          Success:             745
          Fail:                  0
          Timeout:             525
    ================================================================```

インデックスを戻す
================================================================
Congratulations! All Scoring Process Successfully Done!!
    Score:                1910

    Total requests:       1266
      Success:             740
      Fail:                  0
      Timeout:             526
================================================================

マッチング最適化　部署、オフィス
================================================================
Congratulations! All Scoring Process Successfully Done!!
    Score:                2048

    Total requests:       1280
      Success:             761
      Fail:                  0
      Timeout:             519
================================================================

マッチングの部署指定時のバグ修正
================================================================
    Congratulations! All Scoring Process Successfully Done!!

    Score:                2088

    Total requests:       1297
      Success:             774
      Fail:                  0
      Timeout:             523
================================================================

スキル指定最適化
下がった・・・
================================================================
    Congratulations! All Scoring Process Successfully Done!!

    Score:                1899

    Total requests:       1262
      Success:             738
      Fail:                  0
      Timeout:             524
================================================================

マッチングget最適化
================================================================
    Congratulations! All Scoring Process Successfully Done!!

    Score:                3575

    Total requests:       1450
      Success:             947
      Fail:                  0
      Timeout:             503
================================================================

search最適化
================================================================
    Congratulations! All Scoring Process Successfully Done!!

    Score:               21308

    Total requests:       8539
      Success:            8537
      Fail:                  0
      Timeout:               2
================================================================

icon似非キャッシュ
================================================================
    Congratulations! All Scoring Process Successfully Done!!

    Score:               36710

    Total requests:      10917
      Success:           10916
      Fail:                  0
      Timeout:               1
================================================================

マッチングスキル指定バグ修正
================================================================
    Congratulations! All Scoring Process Successfully Done!!

    Score:               30222

    Total requests:      12852
      Success:           12852
      Fail:                  0
      Timeout:               0
================================================================

nginx設定
Results per API:
- POST /api/v1/session
  ✓ requests: 1941, success: 1941, fail: 0, timeout: 0
  ✓ rps: 32.35

- GET /api/v1/users/user-icon/{userIconId}
  ✓ requests: 3094, success: 3094, fail: 0, timeout: 0
  ✓ rps: 51.57

- GET /api/v1/users
  ✓ requests: 3949, success: 3949, fail: 0, timeout: 0
  ✓ rps: 65.82

- GET /api/v1/users/search
  ✓ requests: 3936, success: 3936, fail: 0, timeout: 0
  ✓ rps: 65.6

- POST /api/v1/match-groups
  ✓ requests: 196, success: 196, fail: 0, timeout: 0
  ✓ rps: 3.27

- GET /api/v1/match-groups/members/{userId}
  ✓ requests: 948, success: 948, fail: 0, timeout: 0
  ✓ rps: 15.8

================================================================
    Congratulations! All Scoring Process Successfully Done!!

    Score:               33297

    Total requests:      14064
      Success:           14064
      Fail:                  0
      Timeout:               0
================================================================

mizuki mysql.conf設定コピー
Results per API:
- POST /api/v1/session
  ✓ requests: 2376, success: 2376, fail: 0, timeout: 0
  ✓ rps: 39.6

- GET /api/v1/users/user-icon/{userIconId}
  ✓ requests: 3441, success: 3441, fail: 0, timeout: 0
  ✓ rps: 57.35

- GET /api/v1/users
  ✓ requests: 4622, success: 4622, fail: 0, timeout: 0
  ✓ rps: 77.03

- GET /api/v1/users/search
  ✓ requests: 4618, success: 4618, fail: 0, timeout: 0
  ✓ rps: 76.97

- POST /api/v1/match-groups
  ✓ requests: 233, success: 233, fail: 0, timeout: 0
  ✓ rps: 3.88

- GET /api/v1/match-groups/members/{userId}
  ✓ requests: 1099, success: 1099, fail: 0, timeout: 0
  ✓ rps: 18.32

================================================================
    Congratulations! All Scoring Process Successfully Done!!

    Score:               39870

    Total requests:      16389
      Success:           16389
      Fail:                  0
      Timeout:               0
================================================================


Add cilium network policies.

When switching to cilium, we lost support for defining network policies that define CIDRs as targets, such as the one we use in this app to allow egress to the API server.
This PR adds a ciliumnetworkpolicy (disabled by default, enabled by cluster-operator in vintage clusters) to overcome this cilium limitation.
See https://github.com/giantswarm/giantswarm/issues/23014#issuecomment-1476207008

@QuantumEnigmaa @QuentinBisson can I merge?

If you tested if sure


Bump OTel to v0.87.0

Description of change
Checklist

[ ] Tested in playground or other setup
[ ] Screenshot (Grafana) from playground added to PR for 15+ minute run


Fix "method_complexity" issue in config.py

Function init_environment has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
https://codeclimate.com/github/sendgrid/open-source-library-data-collector/config.py#issue_59f69b9f7d682f0001000025

Hi, Is there a way to test the cognitive complexity locally ?

I created a pull request with a small refactor that I think will get around the warning.
#115


How do I change DNS without reinstalling everything?

Hello,
This might be a simple question, but here it goes.
When I installed the VPN using the script I setup a DNS provider. However, I want to change it.
I'm not sure if I just need to modify /etc/resolv.conf (CentOS) to take effect because I don't know what option / where does the script save the information I add about the DNS server I want to use.
Thank you for letting me know.
Regards.

You can modify /etc/openvpn/server.conf and then restart the openvpn server with systemctl restart openvpn@server


"Cannot use import statement outside a module", potentially missing Nextjs support

Hey Chris, I am getting "Cannot use import statement outside a module" when running the standard globe:
import React from 'react';
import ReactGlobe from 'react-globe';

function SimpleGlobe() {
  return <ReactGlobe />
}

I think it's because this repository is missing Nextjs support?
19:38:34.674 | Unhandled error during request: /vercel/21055b9b/node_modules/three/examples/jsm/controls/OrbitControls.js:1
19:38:34.674 | import {
19:38:34.674 | ^^^^^^
19:38:34.674 | SyntaxError: Cannot use import statement outside a module
19:38:34.674 | at wrapSafe (internal/modules/cjs/loader.js:1053:16)
19:38:34.674 | at Module._compile (internal/modules/cjs/loader.js:1101:27)
19:38:34.674 | at Object.Module._extensions..js (internal/modules/cjs/loader.js:1157:10)
19:38:34.674 | at Module.load (internal/modules/cjs/loader.js:985:32)
19:38:34.674 | at Function.Module._load (internal/modules/cjs/loader.js:878:14)
19:38:34.674 | at Module.require (internal/modules/cjs/loader.js:1025:19)
19:38:34.674 | at require (internal/modules/cjs/helpers.js:72:18)
19:38:34.674 | at Object.<anonymous> (/vercel/21055b9b/node_modules/react-globe/dist/index.js:1:237)
19:38:34.674 | at Module._compile (internal/modules/cjs/loader.js:1137:30)
19:38:34.674 | at Object.Module._extensions..js (internal/modules/cjs/loader.js:1157:10)
19:38:34.675 | Error occurred prerendering page "/networks". Read more: https://err.sh/next.js/prerender-error
19:38:34.675 | /vercel/21055b9b/node_modules/three/examples/jsm/controls/OrbitControls.js:1
19:38:34.675 | import {
19:38:34.675 | ^^^^^^
19:38:34.675 | SyntaxError: Cannot use import statement outside a module
19:38:34.675 | at wrapSafe (internal/modules/cjs/loader.js:1053:16)
19:38:34.675 | at Module._compile (internal/modules/cjs/loader.js:1101:27)
19:38:34.675 | at Object.Module._extensions..js (internal/modules/cjs/loader.js:1157:10)
19:38:34.675 | at Module.load (internal/modules/cjs/loader.js:985:32)
19:38:34.675 | at Function.Module._load (internal/modules/cjs/loader.js:878:14)
19:38:34.675 | at Module.require (internal/modules/cjs/loader.js:1025:19)
19:38:34.675 | at require (internal/modules/cjs/helpers.js:72:18)
19:38:34.675 | at Object.<anonymous> (/vercel/21055b9b/node_modules/react-globe/dist/index.js:1:237)
19:38:34.675 | at Module._compile (internal/modules/cjs/loader.js:1137:30)
19:38:34.675 | at Object.Module._extensions..js (internal/modules/cjs/loader.js:1157:10)
19:38:34.675 | info  - Generating static pages (12/13)
19:38:34.675 | info  - Generating static pages (13/13)
19:38:34.675 | > Build error occurred
19:38:34.677 | Error: Export encountered errors on following paths:
19:38:34.677 | /networks
19:38:34.677 | at exportApp (/vercel/21055b9b/node_modules/next/dist/export/index.js:24:1103)
19:38:34.677 | at runMicrotasks (<anonymous>)
19:38:34.677 | at processTicksAndRejections (internal/process/task_queues.js:97:5)
19:38:34.677 | at async build (/vercel/21055b9b/node_modules/next/dist/build/index.js:37:212)
19:38:34.730 | error Command failed with exit code 1.
19:38:34.730 | info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
19:38:34.739 | Error: Command "yarn run build" exited with 1
19:38:36.782

fixed it with
const ReactGlobe = dynamic(() => import('react-globe'), {
  ssr: false
});

Hi @PeerRich, I'm not very familiar with the bundling/build tools (this library uses microbundle).  If you are able to find more information on how this can be fixed in the library, a PR is very welcome and I'm interested to know how/where the fix is applied!

@PeerRich i tryed your little trick but it seems it still doesnt want to work for me
`Unhandled Runtime Error
TypeError: Class constructor EventDispatcher cannot be invoked without 'new'
Call Stack
InteractionManager
../node_modules/three.interaction/build/three.interaction.module.js (711:0)
new Interaction
../node_modules/three.interaction/build/three.interaction.module.js (4466:0)
scene.camera
../node_modules/react-globe/dist/index.module.js (1:4769)
new e
../node_modules/react-globe/dist/index.module.js (1:4872)
eval
../node_modules/react-globe/dist/index.module.js (1:12432)
invokePassiveEffectCreate
../node_modules/react-dom/cjs/react-dom.development.js (23487:0)
HTMLUnknownElement.callCallback
../node_modules/react-dom/cjs/react-dom.development.js (3945:0)
Object.invokeGuardedCallbackDev
../node_modules/react-dom/cjs/react-dom.development.js (3994:0)
invokeGuardedCallback
../node_modules/react-dom/cjs/react-dom.development.js (4056:0)
flushPassiveEffectsImpl
../node_modules/react-dom/cjs/react-dom.development.js (23574:0)`
Did you run in something similar?
@chrisrzhou it would be really cool to have next support as it is one of the most used react tools at the time.

Some solution?

@intensr did you find the solution of the error?


Target PR 676

IMPORTANT: Do Not Merge - To be merged by Docs Team Only
Version(s): 1.3,1.4

Issue: RHIDP-4827

Link to docs preview:

Reviews:

[ ] SME: @ mention assignee
[ ] QE: @ mention assignee
[ ] Docs review: @Gerry-Forde
[ ] Additional review: @mention assignee (by writer)

Additional information:

Can you provide a more undersnatdable title for the pull request?


TwinLiteNetv2 release date

When do you plan to release V2 models?
Thank you!

When do you plan to release V2 models?
Thank you!


Aaronmgdr/about below

Description
Implements the below the fold portion of the new about page design
Tested

Other changes
Describe any minor or "drive-by" changes here.
Related issues

Fixes #1052

waiting on text/copy from Vanesa


fix get_permalink response struct

👋 hey @abdolence, quick fix for get_permalink. According to the docs, I think the existing response struct is malformed

Thanks for the contributions, @noxasaxon   I'll do some clean ups in another PR - I see that clippy is complaining to not related places since it was updated, and I may use some types.

Released with v0.30.0


npm overrides

Is there a way of overriding the "main" key of a npm package during compilation
I am trying to npm install jquery-ui-dist to use in npm modules. However. it seems as if the distribution of jquery-ui-dist has an incorrect main file
warn: main file for module 'jquery-ui-dist' does not exist (node_modules/jquery-ui-dist/ui/widget.js), assuming index.js

The correct file is actually jquery-ui.js
I tried adding the following to the npm section
overrides": {
          "jquery-ui-dist": {
            "main": "jquery-ui.js"
          }
        },

but it didn't work - neither did putting overrides in the main section
Is there a way of overriding npm information in brunch-config.js ?

I guess you could use require('jquery-ui-dist/whatever/file-you-want') instead of require('jquery-ui') in your code (as a workaround?).

Yes, indeed I can - discovered that by accident ! It works perfectly, thanks.

sorry, hit the wrong button

I think this issue should be re-opened. By overriding main in 2.9.1, the module will be bundled in, but you can't just do import "module" - which I feel like is the expected behaviour - but have to do import "module/main". I spent a good hour on figuring out what is the exact problem here.


Crash on Startup after Win 11 update

Describe the bug
OotD crashes on startup with RPC failure, also says Outlook not running (it is).
To Reproduce
On startup every time including after restart.
Expected behavior
start as always :). BTW: love this product, hope to get it back soon!
Screenshots

Windows Version
Edition	Windows 11 Enterprise
Version	21H2
Installed on	7/29/2021
OS build	22000.376
Experience	Windows Feature Experience Pack 1000.22000.376.0
Additional context and log file
Add any other context about the problem here
Also upload the most recent log file located in C:\Program Files (x86)\Outlook on the Desktop\logs
2021-12-15.log

After a couple days and restarts it started working again. :)


java.lang.NoSuchMethodError: edu.wpi.first.wpilibj.communication.FRCNetworkCommunicationsLibrary.HALSendError

It looks like this is getting thrown whenever wpilib attempts to print out
PWM Channel N ... Output not updated often enough.
in gradle simulation or verify
it can be reproduced (on my machine at least) with this code: https://github.com/Retro3223/robot2016/tree/sim_uhoh

though I suppose forcing me to feed my things could be considered a feature..


False positive on step failure:  ##[error]SettingsUpdation.Run : Exception occurred during the updation of run settings

Note
Issues in this repo are for tracking bugs, feature requests and questions for the tasks in this repo
For a list:
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
If you have an issue or request for the Azure Pipelines service, use developer community instead:
https://developercommunity.visualstudio.com/spaces/21/index.html )
Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: VSTest@2
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure Pipelines or TFS on-premises?
Azure Pipelines

If using TFS on-premises, provide the version:

If using Azure Pipelines, provide the account name, team project name

, build definition name/build number:
Pipeline
Release Pipeline: Select-Gauntlet

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name:

If using private agent, provide the OS of the machine running the agent and the agent version:

Issue Description
The VSTest step had the error shown below, which is fine, but the step showed a "Succeeded", which made the overall pipeline show as successful.  This is obviously not good since none of our tests ran.
Task logs
A complete debug log can be provided privately.
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
2019-07-23T14:50:59.2572189Z ##[warning]Unable to parse the override run parameters string:
2019-07-23T14:50:59.2572527Z ##[debug]Processed: ##vso[task.logissue type=warning;]Unable to parse the override run parameters string:
2019-07-23T14:50:59.2573444Z ##[error]SettingsUpdation.Run : Exception occurred during the updation of run settings: System.FormatException: Error encountered while overriding test run parameters. Please check the test run parameteres provided.

@splateaux Can you please share the test run parameters you are giving ?

The test run parameters were absolutely not properly formed.  They were effectively something like -parameterOne steve -parameterTwo some value with spaces but no quotes
It's not that my parameters were wrong and that the test step gave an error... it should have, it's that the step showed as "Succeeded" when it went failed so badly.

@splateaux we have fixed this and this should get rolled out in around 2 weeks

@splateaux this is fixed in task version 2.159.x
this version has been rolled out to around half the accounts and the other half should get it in a week.
If you have version 2.159.x or higher and are still facing the issue please re-open this ticket.

I have the same error , one of my variables has a space in it and the total length is 586 for the variable 2020-02-10T21:30:42.6519039Z ##[error]SetupPhase.Run : Exception occurred during the updation of run settings: System.FormatException: Error encountered while overriding test run parameters. Please check the test run parameteres provided.
2020-02-10T21:30:42.6519586Z    at Microsoft.VisualStudio.TestService.SettingsManager.OverrideParamsSettingsProcessor.GetOverrideParameters(String overrdeParametersString)
2020-02-10T21:30:42.6519842Z    at Microsoft.VisualStudio.TestService.SettingsManager.OverrideParamsSettingsProcessor.UpdateSettingsWithParameters(XDocument settings)
2020-02-10T21:30:42.6520043Z    at Microsoft.VisualStudio.TestService.SettingsManager.CommonSettingsManager.UpdateCommonSettings(InputDataContract inputDataContract, SettingsModifier settingsModifier)
2020-02-10T21:30:42.6520214Z    at Microsoft.VisualStudio.TestService.SettingsManager.SettingsManager.UpdateSettingsAsRequired(InputDataContract inputDataContract)
2020-02-10T21:30:42.6520401Z    at MS.VS.TestService.VstestConsoleAdapter.SetupPhase.Run(VstestConsoleRunContext testRunContext, CancellationToken cancellationToken)


bugfix: 31DA not using two's complement for negative temperatures

2022-11-20T08:32:06.904058 063 RP --- 32:134446 37:171685 --:------ 31DA 030 00EF007FFF2F1B0226069A07EEFFE4F8000038988F0000EFEF1F2420FC00

The above payload parses like so: 'outdoor_temperature': 655.08. However, temperatures such as FEE4 should be -0.28 °C.
The solution is to use temp_from_hex('FEE4') instead of double_from_hex(...).
The Fan is an Orcon HRC 425
[edited by dev]

@tomkooij thanks for your PR - looks like an easy win.
For posterity, please indicate the make/model of FAN.
The reason we need to know make/model because some of the fields in 31DA vary by make (manufacturer), although the temperature fields are not likely to be so affected.
I will add this to the test suite, too.

@tomkooij thanks for your PR - looks like an easy win.
For posterity, please indicate the make/model of FAN.
The reason we need to know make/model because some of the fields in 31DA vary by make (manufacturer), although the temperature fields are not likely to be so affected.

iI added Orcon HRC 425

And we're done. Thanks.


A question about the adaptation for different frame size e.g. 10ms 22band

Thank you for the fantastic work.
In denoise.c, line 193, should I change the 22 to be NB_BANDS for different NB_BANDS?
out[i] = sum*sqrt(2./NB_BANDS);
My configuration is 10ms per frame, and 16kHz.
Other modification I have done：

in the denoise.c

#define FRAME_SIZE_SHIFT 1
#define FRAME_SIZE (80)
#define WINDOW_SIZE (2*FRAME_SIZE)
#define FREQ_SIZE (FRAME_SIZE + 1)

#define PITCH_MIN_PERIOD 20
#define PITCH_MAX_PERIOD 256
#define PITCH_FRAME_SIZE 320
static const opus_int16 eband5ms[] = {
  0,  1,  2,  3,  4,  5,  6,  7,  8, 10, 12, 14, 17, 20, 24, 28, 33, 39, 47,57, 68, 80
}

features[NB_BANDS+3*NB_DELTA_CEPS] = .01*(pitch_index-100);

in the rnnoise_demo.c
#define FRAME_SIZE 160

I suppose many pepole want to change the sampling frequency, frame size and NB_BANDS. I hope we can share the ideas in this issue. I'd appreciate any help.

There are many open issues regarding the sampling frequency. Please use the search function!
The short answer is: This is not sufficient. There is some pitch filtering/correlation done, which wont work, if you change the sampling frequency.

@Zadagu
I also want  training sample's sample rate is 16000, how to change pitch_downsample,pitch_search,remove_doubling those
funtions?  I have searched all issues  but didn't find the answer，Can you give some guidance?
Thanks！

I'm also very interested in a 10ms window variant. I made an issue about this upstream: https://gitlab.xiph.org/xiph/rnnoise/-/issues/9. I'm fine with the stock 48kHz sample rate and band count, just the window size is a big deal for me. If you have any advice on how to go about this, I'd love to hear it!


Nested branch support in backend compiler

TL;DR
Pulls in latest FlytePropeller/compiler which contains the Nested branch support. related to - https://github.com/flyteorg/flytepropeller/pull/264
For all nested branches to work, this has to be enabled on the flyte cluster.
Type

[x] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[x] Smoke tested
[x] Unit tests added
[x] Code documentation added
[x] Any pending items have an associated Issue

Tracking Issue
https://github.com/flyteorg/flyte/issues/1052

update pkg/manager/impl/launch_plan_manager_test.go too?

update pkg/manager/impl/launch_plan_manager_test.go too?

just did it


Check if entry is password protected

Hi,
I was wondering how I could check if a rar entry is password protected, thanks!

#r "nuget: SharpCompress, 0.32.2"

open SharpCompress.Archives

let archive = ArchiveFactory.Open("secret.zip")

for e in archive.Entries do
  printfn $"IsEncrypted %b{e.IsEncrypted}"


HTTP request method

During development of the PGP prototype, we ran into an issue related to CORS and querying the key server with a POST request.  To simplify things we used a GET request to add keys to the key server.
This breaks convention and would be confusing for anyone new to the code base.  We need to change these requests to use POST instead, which will involve using CORS.  This will need to be a coordinated update in both the privly-applications repo and the privly-keyserver repo.

Since Persona isn't actively being developed by Mozilla and the current PGP app is tightly coupled to Persona, I am closing this issue.  When/If the PGP app is resurrected this issue should be reopened.


Add withJWTProps HOC

Hej.
We found JWT decoding operation repetitive so "here is a HOC for that" ©:
withJWTProps(<client token prop name>, <token fields to props mapping>)(Target)

// spread all fields from decoded token as props
withJWTProps('clientToken')(Target)

// provide `map_url` token field as a `baseURL` prop
withJWTProps('clientToken', { base_url: 'baseURL' })(Target)

Also onJWTError callback will be invoked if something went wrong.

This happened fast 🚤  🚄
Thanks!


Migration of contracts instantiated with WasmMsg::Instantiate fail

When a CW contract is instantiated from command-line, the migratable flag can be set to true. However, when the contract is instantiated programmatically using WasmMsg::Instantiate, there isn't a way to set migratable flag, so migration of such contracts fail.

This is a good point and oversight when I added WasmMsg::Migrate.
I would propose adding admin: Option<String> to WasmMsg::Instantiate. The contract could set admin: Some(env.contract.address) and then be able to migrate the sub-contract

Do we have an example for deserializing the JSON of Option<String> to a Go struct? The JSON schema is this:
{
  "admin": {
    "type": [
      "string",
      "null"
    ]
  }
}

Try for yourself: https://play.golang.org/p/W-VB5AXQNFN
{"admin":""} and {"admin":null} and {"admin":""} all unmarshal to the empty string. Play around there to try other cases

Good, thanks. *string is a good mirror type that wrks both ways as it is serialized to null or a string.


licensing

Would you consider providing an alternate license of pyapi-gitlab so that it can be linked with non-free software?
I would honestly prefer something like MIT, but if you want to keep copyleft, the MPL is a popular license that is a source-level copyleft, thus requiring changes to be provided back to the original author, but still allows you to link to non-free software.

Hey @jzelinskie , why would be the reasoning for this change?
I been thinking about this for a long time actually. At first I went with the GPL3 because it was ok, but the more I went on the innards of it the more I feel like its not the rigth choice for any project to grow.
Would something like the Apache license work for whatever your needs are?

Apache sounds great -- we'd like to use this library to add GitLab support to Quay.io.

Great! Im checking to do a hotfix for #125 so Ill probably do the change of the License at the same time a release a new version to pypi.

Awesome! Thanks a bunch.


fix typo in poly sample constructor initialisations

Signed-off-by: mHash1m<EMAIL_ADDRESS>

Hi @mHash1m , good spot!


Move the package to SublimeText org

Hi @jbrooksuk,
I'd like to move this package to the SublimeText org with your approval. The last notable changes to it were made 10 years ago and it's basically been unmaintained since then (but it also still works just fine and I use it regurlarly). However, in order to prevent it from potentially being lost in the aether of unmaintained stuff and in case you yourself lose interest in it, I'd like to move it to the org where we have serveral admins to handle maintainership changes et al.
The trigger for this was that I noticed the package's homepage pointing to https://james-brooks.uk/, which is defunct, and I as a contributor (the highest non-owner permission level for account-owned repositories) cannot change this level of setting despite being a co-author of the package.
Please let me know if you're in support of this. You could either transfer the repo to me or I add you to the org (if you're not already a member) and you can transfer it there yourself, whatever you prefer. I'll handle the rest, i.e. updating the Package Control entry.

@FichteFoll yeah, I'm happy to do that. Let's update the package's homepage to the GitHub repo.
I'll transfer this to the SublimeText org now.

This is now done 👍🏻

Changes have also been made on the PC channel repo.


Switch to a declarative way of using numbers in the tests

The current approach of passing numbers in the tests is janky: there are lots of constants that have to be imported.
An easier way, which would be more declarative, would be to use a function like fp. A nice side-effect of doing this would be that the size of the code base would be (slightly?) reduced.

I should use my recently published evm-fp here.

Done in 188a837.


Breaking bastions is a little resource intensive

Probably room for improvement regarding resource usage of Bastion plugin when players are breaking bastions. At times it can be resource heavy and cause lag issues, especially if multiple bastions are being broken by multiple players.
https://timings.aikar.co/dev/?id=48ab57eaccbd48268a5648436dcbc30d
https://spark.lucko.me/NE8Xx049Wo

I'm closing this out as something that we won't fix. It could be revisited in the future, but it's not within our capacity to work on right now.


Jira407

HWKAGENT-407  when unable to connect to the endpoint, set avails to down

for remote monitoring this should solve the issue, if you can't connect
to the endpoint we should assume it's down, and that it's associated
resources are down.
for local/embedded dmr this shouldn't happen, because if the server
goes down so will the agent

@jmazzitelli please review.  We may want to wait on any merge for the fix needed to run the itest

Let's wait for https://issues.jboss.org/browse/HWKMETRICS-407 to be fixed so we an enable the test. If that isn't going to happen in a while, then we can merge this but first write a JIRA to remind us to re-enable this test once that metrics bug is fixed.

@jshaughn so that metrics JIRA isn't really a bug. Can we re-enable that test and then merge this when it goes green?

@jmazzitelli You're right and the new test is already enabled in this PR.  The entire -Pitests run has passed twice for me locally. I restarted travis again now, I have no idea why it fails but it seems travis-related.  This PR is ready for merge imho.


Multi-page HTML output

See asciidoctor/asciidoctor#626

Going to close this issue as it's under described and @rhattersley is no longer active.


Pagination: Is enabled, but I couldn't find any pagination page.

Using jekyll 3.6.2 I followed the instructions and I still got the following message when doing bundle exec jekyll serve:

Pagination: Is enabled, but I couldn't find any pagination page. Skipping pagination. Pages must have 'pagination: enabled: true' in their front-matter for pagination to work.

My _config.yml file contains the following:
# Pagination Settings
pagination:
  enabled: true
  per_page: 3
  permalink: '/page/:num/'
  title: ':title - page :num of :max'
  limit: 0
  sort_field: 'date'
  sort_reverse: true

And the layout home where I want the pagination for my posts looks like this:
---
layout: default
pagination:
  enabled: true
---

Posts
 {{ content }}

        {% for post in paginator.posts %}

            {{ post.date | date: "%b %-d, %Y" }}

          {{ post.title | escape }}

{{ post.excerpt | strip_html | truncatewords:20 }}

{% endfor %}

What am I doing wrong? Why paginator.posts seems to be empty?
It is not showing any post.
My Gemfile contains:
# If you have any plugins, put them here!
group :jekyll_plugins do
   gem "jekyll-feed", "~> 0.6"
   gem "jekyll-paginate-v2"
end

Solved by adding
pagination:
  enabled: true

In index.html outside the _layouts folder.

For anyone else who comes across this, I had the same issue. I realized that jekyll-paginate-v2 will ignore all pages that appear in directories that start with an underscore (internal to Jekyll), it only sees pagination pages if they appear inside public directories.
More details in #84.


fix: dclexperience: Octopus NPC dialog scale is huge

What does this PR change?
Fix #705
How to test the changes?

Launch the explorer,
Go to dclexperience.dcl.eth world (command: /world dclexperience.dcl.eth).
Talk with the octopus.
Observe the conversation UI doesn't change its size depending on the resolution.

IMPORTANT NOTE TO TEST THIS FIX: Lately, the dclexperience.dcl.eth world is not being loaded well sometimes, so please BE SURE that the dialog with the octopus advances when you click, if not, it does mean that the scene was not correctly loaded and you will have to go back to the point 2 and load the world again auntil it works correctly.
Our Code Review Standards
https://github.com/decentraland/unity-renderer/blob/master/docs/code-review-standards.md

Finally, it was enough to change the Scale Mode of the UIToolkit RootCanvas Settings to make the UI's resize orderer by the scene works correctly! 👍

I realized that I was not able to test the fix well because most of the times the dclexperience.dcl.eth world was not being loaded correctly! ...and it was provoking an STOP on the scene flow, so none of the UI dialogs were receiving any resize from the scene.


(feature): Add ignores_warnings property

[Closes #28]

We're not the most ruby savvy team so let us know if there's anything we can do to improve this PR.
Looks like we've got a little whitespace fixing to do at the very least
lib/xcode_summary/plugin.rb:62:1: C: Layout/TrailingWhitespace: Trailing whitespace detected.
lib/xcode_summary/plugin.rb:95:1: C: Layout/TrailingWhitespace: Trailing whitespace detected.
lib/xcode_summary/plugin.rb:164:1: C: Layout/TrailingWhitespace: Trailing whitespace detected.

Thanks @bitwit , I just fixed the warnings, added some tests and documentation.

Thanks for the extra legwork and merge @diogot ! Was a pleasure to contribute


Use consistent errors package

Signed-off-by: Lei Jitang<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master     #792   +/-   ##
=======================================
  Coverage   59.26%   59.26%
=======================================
  Files           5        5
  Lines         739      739
=======================================
  Hits          438      438
  Misses        198      198
  Partials      103      103

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 92af03f...75b889d. Read the comment docs.

LGTM

LGTM


Pass group-by names in ReadHints

This PR adds a list of group-by fields to SelectParams and ReadHints. For example for the query sum by (url, app)(my_gauge) that list would be [url, app].
This is needed in order for our (sysdig) remote read backend to allow grouping by fields that are derived metadata and that would not otherwise be returned. There may be other uses for it in other contexts.

to allow grouping by fields that are derived metadata and that would not otherwise be returned.

This sounds a bit off semantically. Hints should never change the end results that a user sees, they should only enable optimisations.

This sounds a bit off semantically. Hints should never change the end results that a user sees, they should only enable optimisations.

There's already a precedent where SelectParams do change the results (in Thanos).
What's the difference between ReadHints and SelectParams in that respect? (I should probably have asked that question when the name came up
in https://github.com/prometheus/prometheus/pull/4122#pullrequestreview-116263721 ...)

There's already a precedent where SelectParams do change the results (in Thanos).

Not as far as I'm aware, the end user result doesn't change.

What's the difference between ReadHints and SelectParams in that respect?

SelectParams is the actual selector being sent down which you are responding to. ReadHints gives you some additional context to allow you to process it more efficiently. For example if someone is asking for data with a 1m step, there's no need to send second granularity data back. You can instead prune the samples that would have been discarded.

Depending on the value of SelectParams.Func, you get different results back e.g here https://github.com/improbable-eng/thanos/blob/master/pkg/query/querier.go#L144

Prometheus gets back different results for remote read, the user does not. These are optimisations that don't affect the query result (beyond the fundamental changes due to downsampling).

ReadHints is nothing more than the "forwarding on" of SelectParams to be sent over with a remote read query.

Ah wait, I got confused. You are right. In both cases the result the end-user gets should not change based on ReadHints/SelectParams though.

Prometheus gets back different results for remote read, the user does not. These are optimisations that don't affect the query result (beyond the fundamental changes due to downsampling).

With downsampling, the user absolutely will get different results back based on the value of ReadHints.Func. In this case it's not just an optimization hint: its presence and value is going to affect the results of the promql query.
But regardless, if the goal is that additional SelectParams/ReadHints should not carry result-modifying information, then PR can probably be closed.

Closing as per discussion above.

@henridf @brian-brazil
In my use case, i do this by：
sum(abc{HINTS="{'by':'url|app'}"}) by (url, app)


Flutter Shortcuts physical key is pressed on a different logical key

Steps to Reproduce

Execute flutter run on the code sample
...
...

Expected results: IR remote works smoothly
Actual results: I build flutter app for android TV, but I have issue with the IR remote when I press OK key

Code sample
 shortcuts: <LogicalKeySet, Intent>{
    LogicalKeySet(LogicalKeyboardKey.select,): const ActivateIntent(),
  },

Logs
======== Exception caught by services library ======================================================
The following assertion was thrown during a platform message callback:
Attempted to send a key down event when no keys are in keysPressed. This state can occur if the key event being sent doesn't properly set its modifier flags.  RawKeyDownEvent || _keysPressed.isNotEmpty

Hello @MahmmoudSabri. Thank you for filing this issue. Can you please provide the following information?

A complete and minimal, reproducible example. Ideally, a single main.dart file that can be run directly.
The output of flutter doctor -v
The output of flutter run -v

Please provide the information in the form of text. See how to make collapsible sections with Markdown here.


#643 feat(templates): add privileged flag to templates

Closes #643

The docs here have to be update too :
http://portainer.readthedocs.io/en/latest/templates.html


Fix failing test in travis

https://github.com/tensorflow/tfjs-core/pull/1379 added a unit test that failed in travis due to numerical precision: https://travis-ci.org/tensorflow/tfjs-core/jobs/451654174#L801
This PR:

fixes that test
Also fixes test-travis.sh to make sure we don't forward webgl --> cpu except for the last browser run -- probably caused during conflict resolving of https://github.com/tensorflow/tfjs-core/pull/1371 and https://github.com/tensorflow/tfjs-core/pull/1372 (https://reviewable.io/reviews/tensorflow/tfjs-core/1371#-LQ_a2y92YYLRMiYWqhC)

For repository owners only:
Please remember to apply all applicable tags to your pull request.
Tags: FEATURE, BREAKING, BUG, PERF, DEV, DOC, SECURITY
For more info see: https://github.com/tensorflow/tfjs/blob/master/DEVELOPMENT.md

This change is

Looks like those bash comments are screwing things up, my guess is because of the \ happening before the comment.

Out of curiosity: Why is the Safari JSVM behaving differently? The results seem to be way of. Even if summed up naively:
Array.from({ length: 10*1000 }, () => Math.fround(0.1**2) )
     .reduce( (x,y) => Math.fround(x+y), /*initVal=*/0.0 )

The result on my machine is<PHONE_NUMBER>756836. But if I am not mistaken the CPU matrix multiplication in TFJS is blocked up. Inside the blocks, a float64 value is used to sum the values up. So the result should be something along the lines of:
const blockSize = 50;
Array.from(
  { length: 10*1000/blockSize },
  () => Math.fround(
    Array.from({ length: blockSize }, () => Math.fround(0.1**2) )
         .reduce( (x,y) => x+y, /*initVal=*/0.0 )
  )
).reduce( (x,y) => Math.fround(x+y), /*initVal=*/0.0 )

That result even perfectly rounds to 100 on my machine.
In the Travis log the result seems to be 99.29019927978516 which would be way off.

I noticed the same - Didn't give much thought since that test was not failing on my local laptop (Macbook Pro). The only diff between my laptop and Travis was the Safari version: mine is 12.0.1, Travis used 11.1.2

I too had some issued with the Safari tests on Travis. My guess was that it was a WebGL thing, but maybe there is an issue with Float32Array in that Webkit/Safari version?


Convert HTML to Markdown (EnumResourceNamesA)

Also, update link to ENUMRESNAMEPROC.

@ratijas : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks for the contribution. I've merged your changes and they should be live soon.


Budgets: az resource create - deployment error message and cannot see budgets

Describe the bug
I'm trying to create budgets with az resource create and encounter two problems:

command execution results in error message: Deployment failed. Correlation ID: <id>. Request is missing a certificate. According to the --debug output, the following things happen internally:
** GET request to Microsoft.Consumption provider --> succeeds (status 200)
** GET request to the resource group --> succeeds (status 200)
** PUT request to create budget --> succeeds (status 201, containing the budget data in the body)
** GET request to the new budget --> fails (status 401 and Request is missing a certificate error)
In Azure Portal and via az consumption budget list, I do not see the new budget. But if I try to create the same budget again, I get a There are changes made to Budget <name>... error message. And I receive correct email notifications from such a budget, if I add an e-mail notification. Seems like I am creating zombies...

To Reproduce
az resource create -g testgroup -n "testbudget2" --resource-type "Microsoft.Consumption/Budgets" --properties "{\"amount\" : 1000, \"category\" : \"Cost\", \"timeGrain\" : \"Monthly\", \"name\" : \"testbudget2\", \"timePeriod\" : {\"startDate\" : \"2019-07-01T00:00:00Z\", \"endDate\" : \"2019-12-31T00:00:00Z\"}}"
Expected behavior

budget creation succeeds without error message
budget is visible in Azure portal and via az consumption budget list

Environment summary
azure-cli                         2.0.69

command-modules-nspkg               2.0.3
core                              2.0.69
nspkg                              3.0.4
telemetry                          1.0.3

Extensions:
azure-cli-iot-ext                  0.7.1

Python (Linux) 3.6.5 (default, Jul 11 2019, 08:41:35)
[GCC 7.4.0]

Ubuntu 18.04

Additional context
I think, there is something happening in the background. On Monday this week, the behavior of the az resource create command changed. Until then, things were like this:

I saw the missing certificate error
I saw the Budgets after creating them
No e-mail notifications were thrown, when budget limits were exceeded

Another weird thing is that if I first create a budget this way, then delete the whole resource group with this budget inside, then create a new resource group with the same name as the old one and create the same budget inside again, I also get the There are changes made to Budget <name>... error. So obviously, the zombies don't get killed properly when destroying their resource group.
If it helps, I can also provide some debug output. Thanks for your help!

@awoSYS I will check this issue

I managed to reproduce this issue myself, and I have some more observations.
PUT /subscriptions/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx/resourcegroups/rg1/providers/Microsoft.Consumption/Budgets/testbudget2?api-version=2019-05-01
Returns with code 201.
Response headers contain following header:
'Location': 'https://consumption.azure.com/subscriptions/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx/resourcegroups/rg1/providers/Microsoft.Consumption/Budgets/testbudget2?api-version=2019-05-01'
But also body is returned:
Response content:
msrest.http_logger : {"id":"subscriptions/172a3774-5a9d-4160-8dab-a4fe8d35c676/resourceGroups/rg1/providers/Microsoft.Consumption/budgets/testbudget2","name":"testbudget2","type":"Microsoft.Consumption/budgets","eTag":""1d54223db133924"","properties":{"timePeriod":{"startDate":"2019-07-01T00:00:00Z","endDate":"2019-12-31T00:00:00Z"},"timeGrain":"Monthly","amount":1000.0,"currentSpend":null,"category":"Cost","notifications":{},"filter":{}}}

The response may contain URI in Location header field in HTTP headers list, which can have reference to the newly created resource. Also, response payload also may include an entity containing a list of resource characteristics and location(s) from which the user or user agent can choose the one most appropriate. The entity format is specified by the media type given in the Content-Type header field.

This means we could just use response body, or submit another GET to retrieve content from specified location. In this case former happens.
Pls note, that new location has different host value consumption.azure.com.

I have tried:
az resource show -g rg1 -n "testbudget2" --resource-type "Microsoft.Consumption/Budgets"
and it works correctly.

@awoSYS i have one question. what is the reason that you don't use az consumption budget create? I guess it's because it doesn't provide some parameters that are available via REST API?
I think there are 2 problems here to be fixed:
(1) az resource shouldn't attempt to use Location header if body is available.
(2) We should upgrade az consumption to use latest API.

(3) check why budget created via rest api is not visible on the portal

@zikalino Thanks for testing! Exact same behavior for me.
concerning az consumption budget create: You're right, there is no possibility to define contact mails and contact groups (see this issue), which I need.

there's something weird going on, I have tried older api version;
# az resource create -g rg1 -n "testbudget2" --resource-type "Microsoft.Consumption/Budgets" --properties "{\"amount\" : 1000, \"category\" : \"Cost\", \"timeGrain\" : \"Monthly\", \"name\" : \"testbudget2\", \"timePeriod\" : {\"startDate\" : \"2019-07-01T00:00:00Z\", \"endDate\" : \"2019-12-31T00:00:00Z\"}}" --debug --api-version 2018-01-31

And I got following error:
Subscription 172a3774-5a9d-4160-8dab-a4fe8d35c676 is not associated with EA offer type

This is also already a known bug: https://github.com/Azure/azure-cli/issues/9867

@awoSYS thanks for linking this. Seems like @axelwogawa created budget sing the same command and could at least see it in the portal. Perhaps that have changed as well.

@zikalino Actually, @axelwogawa is another account of mine (sorry for confusing). What I described there was last week's observation. Because now the budgets became invisible, I decided to file a new issue for this.

@awoSYS now I see similarity in acc names :-)
anyway, are you able to create budget via portal? I can't really find it anywhere...

anyway, are you able to create budget via portal? I can't really find it anywhere...

Yes, I am (Resource group -> Cost management -> Budgets -> Add): Works fine!

@zikalino Your suggestion from #9867

Btw, you could also try az rest command, i think it doesn't have the same problem.

I did
az rest --method put --uri "https://management.azure.com/subscriptions/<ID>/resourcegroups/<group>/providers/Microsoft.Consumption/Budgets/testbudget4?api-version=2019-05-01" --body '{"location": "westeurope", "properties": {"category": "Cost", "timeGrain": "Monthly", "amount": 10, "timePeriod": {"startDate": "2019-07-01T00:00:00Z", "endDate": "2020-12-31T00:00:00Z"}, "notifications": {"testNotification100": {"operator": "GreaterThanOrEqualTo", "threshold": 100, "contactEmails": ["<mail>"], "enabled": true, "contactGroups": ["<action group path>"]}}}}'

and got the same result as by using az resource create:

invisible in portal
az consumption budget list and az consumption budget show don't find it
az resource show finds it

(Of course, there is no 401 error now, because I don't do any explicit GET)

@awoSYS I have to discuss this with service team.
In the Azure CLI:
(1) we can fix az resource ( @yugangw-msft  do you agree?)
(2) upgrade az consumption to use latest API
and we will have some consistency here at least.
But... it almost seems like we are dealing with two separate services here, and it also seems portal was not updated....

Any updates on this issue?

The information in this issue is relatively large. After sorting it out, there are mainly the following problems:

Solved problems
The budget created by az resource create budgets can be seen in az consumption budget list/show, az resource show and portal (need to refresh).

Know issue
Subscription <subscription id> is not associated with EA offer type (Request ID: <request id>) problem with command az consumption budget create. #9867
It can be resolved by upgrading the api version called by the az consumption budget create command

The issues need to be solved by Service team:

The error Request is missing a certificate on first budget creation:

The first time to create a budget with PUT request:

PUT /subscriptions/0b1f6471-1bf0-4dda-aec3-cb9272f09590/resourcegroups/zhoxing-test/providers/Microsoft.Consumption/Budgets/zhoxing-test?api-version=2019-10-01

Service returns the redirect URL

https://consumption.azure.com/subscriptions/0b1f6471-1bf0-4dda-aec3-cb9272f09590/resourcegroups/zhoxing-test/providers/Microsoft.Consumption/Budgets/zhoxing-test?api-version=2019-10-01'

Request the URL of the asynchronous result

GET https://consumption.azure.com/subscriptions/0b1f6471-1bf0-4dda-aec3-cb9272f09590/resourcegroups/zhoxing-test/providers/Microsoft.Consumption/Budgets/zhoxing-test?api-version=2019-10-01

The Service return {"error":{"code":"401","message":"Request is missing a certificate"}}
When the PUT request is called for the second time, the status of 200 and the data of corresponding budget are returned without error.

If adding the budget for a resource group, deleting the resource group will not cascade to delete the budget. If the resource group with the same name is recreated later, the budget bound to the deleted old resource group will still be bound to the new resource group.
Is this in line with expectations?

After adding the budget through CLI, if creating a budget with the same name on the potal without first refreshing the budget list on the list page. The duplicate check will ignore the budget just created by CLI and create a new budget with the same name to overwrite the one created by cli. (I found it by accident)

Team is investigating.

Hi @zhoxing-ms , please find the responses to your questions

The issue where the Location Header was returning you a 401 has been fixed. We will be rolling this out in the coming weeks after holidays.
Budgets are sticky to the resource groups, so recreating the resource group will bring back the budget. Also, budgets do not get deleted automatically on resource group deletion. This is expected.
The behavior in the portal is expected because last write wins.

@theelderwand Thank you for your reply. Please let we know when you rolling this fix out and we will test it again~

@zhoxing-ms Fix for the location header issue has been deployed now. You can test it again.

@zhoxing-ms Fix for the location header issue has been deployed now. You can test it again.

@JaikishanRupani Thank you for your notice. I have verified it and this issue has been solved

@JaikishanRupani Thank you for your notice. I have verified it and this issue has been solved


Allow lun disks to be mapped to DataVolume sources

What this PR does / why we need it:
This pull request modifies the vmi admission webhook to allow using DataVolume sources for Lun Disks, instead of allowing only PVCs.
Which issue(s) this PR fixes:
Fixes # https://bugzilla.redhat.com/show_bug.cgi?id=2190171
Release note:

Bugfix: Allow lun disks to be mapped to DataVolume sources

Opening a new PR since the previous one wasn't being updated https://github.com/kubevirt/kubevirt/pull/9863

/test pull-kubevirt-e2e-k8s-1.26-sig-compute

/test pull-kubevirt-e2e-k8s-1.26-sig-compute

/lgtm
/approve

/cherry-pick release-0.59

Should this have been in release-1.0 as well? trying to cherry pick a test fix but it looks like this thing is only in 1.1/0.59

Should this have been in release-1.0 as well? trying to cherry pick a test fix but it looks like this thing is only in 1.1/0.59

Thanks for noticing @akalenyu, yeah, this should've been in 1.0.  I probably assumed main was linked to 1.0 back then. I'll open a backport.

/cherry-pick release-1.0


Update code train

Please release training code !

Please release training code !

Okay, I've been a little busy recently. The complete code will be updated in the next week or two

Thanks your reply.

Thanks your reply.

you are welcome.

The semantic segmentation process code is available now, the other tasks will be released later


Add amplification limit test to interop quic testing

Because this ci job only runs from the master branch, we need to add the test here to validate that our server respects amplification limits in our ci runs.
Checklist

[x] tests are added or updated

This pull request is ready to merge

This was merged.


[ApolloPagination] Load All Pages

What are you trying to accomplish?

[!IMPORTANT]
This pull request relies on #80, but I cannot point this branch to that branch and attempt to open it up as pull request in this repository (as I'm an unaffiliated contributor to the repository). As such, the overwhelming majority of commits on this branch are not part of this pull request; only the final 4 commits are the contents of this pull request.

I want consumers of the GraphQLPager to be able to load all pages at once, should they desire to do so.

What approach did you choose and why?
First, I modified the fetch function to follow async/await syntax. That's so that we can await its return prior to beginning to loop over loadMore.
Then, I implemented the function using a while loop.
Finally, I made additive changes to the MockGraphQLServer to support subsequent loads of this nature.

Anything you want to highlight for special attention from reviewers?
Yes: This doesn't work for ReversePagination -- and that's okay. ReversePagination is due for some re-working as part of https://github.com/apollographql/apollo-ios/issues/3265

If something goes wrong, what are the mitigation strategies?

Rollback - This change can only be disabled by reverting this pull request or commit.

🤖 Generated by Copilot at 12d13bc
Summary
🧪🔄🔒

This pull request adds a new feature to the GraphQLQueryPager protocol and class, which allows loading all the pages of a paginated GraphQL query in one call. It also updates the tests and the mock GraphQL server to use the new feature and the async/await syntax. Additionally, it removes some unnecessary public modifiers from internal extensions.

Sing, O Muse, of the swift and skillful coder
Who added to the GraphQLQueryPager a wondrous feature
To load all pages of a query with one call, like a mighty river
That flows from the mountains to the sea, unstoppable and eager.

Walkthrough

Add loadAll methods to AnyGraphQLPager and GraphQLQueryPager protocols and classes to allow loading all pages of a paginated query in one call
Refactor GraphQLQueryPager.Actor class to use async/await syntax, initialFetchTask property, and isLoadingAll flag for pagination logic and cancellation handling
Add test cases for AnyGraphQLQueryPager and ForwardPaginationTests using mock GraphQL server and expectations
Add subscript operator to MockGraphQLServer class to access and set RequestExpectation objects for different operation types
Modify FriendsQuery extension methods to use specific query instances with expected variables instead of generic operation type
Remove unnecessary public modifiers from GraphQLQueryPager.Actor extensions in AnyGraphQLPager.swift and Pager+Convenience.swift files
Move deinitializer after initializer and remove public modifier from initializer in GraphQLQueryPager class
Add missing import statement to FriendsQuery+TestHelpers.swift file

Closing in favor of #115


MODULES-2983: IPv6 for ismaster fact and sslAllowInvalidHostnames feature

This Pull request will add the following feature:
class {'::mongodb::server':
ssl_invalid_hostnames => true
}
If SSL is enabled and the certs have the FQDN as DN, some mongo calls from this module will fail.
It is not possible to refactor the calls to use the FQDN, cause if AUTH is enabled the admin user will be created via <IP_ADDRESS> which will connect as admin by default without credentials.
Now it is possible to ignore such cert FQDN warnings.
The pull request #348 is also included for the ismaster fact and modified for the right selection of mongo parameters.

@tphoney Can you remove the needs rebase label, i rebased it to actual master.

Hi,
will this be merged soon?


upgrade to OpenNLP 2.3.0

Description
Building upon the #448 pull request upgrading to 1.9.4 -- this could be follow-up or alternative.
#11277

FYI 2.3.1 was just released.

Hi @cpoerschke - apologies for the delay. This looks ok to me.  I guess this is unavoidable if somebody wants to use it though.
It would be good migrate the model regeneration to gradle - probably a good follow-up issue on its own though.

What's probably missing is ... a note on migration?

Yes, indeed. It being a 1.x to 2.x dependency upgrade, I'm not yet familiar enough with OpenNLP to gauge what's involved in migrating. Perhaps @jzonthemtn and/or @epugh would have insights to share on that? Thanks!

I don't know about the migration side specifically, but based on https://opennlp.apache.org/news/release-200.html and some of the other release notes, here is a first stab.
OpenNLP 2x opens the door to accessing various models via the ONNX runtime.  To migrate you will need to update any deprecated OpenNLP methods that you may be using and be running on Java 17.

FYI 2.3.1 was just released.

2.3.2 just released -- https://opennlp.apache.org/news/release-232.html -- I'll update this PR

... perhaps fire an email to legal at apache, asking ...

Good idea, will do. And thanks for reviewing also with this in mind!

Hi @cpoerschke,
the backport to 9.x produces errors, as opennlp seems to require Java 17:
/home/jenkins/workspace/Lucene-9.x-Linux/lucene/analysis/opennlp/src/java/org/apache/lucene/analysis/opennlp/tools/NLPChunkerOp.java:21: error: cannot access ChunkerME
import opennlp.tools.chunker.ChunkerME;
                            ^
  bad class file: /home/jenkins/.gradle/caches/modules-2/files-2.1/org.apache.opennlp/opennlp-tools/2.3.2/d739edba1e729691ed5ab80e1ccf330555a02ea7/opennlp-tools-2.3.2.jar(/opennlp/tools/chunker/ChunkerME.class)
    class file has wrong version 61.0, should be 55.0
    Please remove or make sure it appears in the correct subdirectory of the classpath.

I think we have to revert the upgrade in Lucene 9.x and only apply it to main (Lucene 10). It should not take too long until we release 10.0, so I think that's a fine compromise.
If you revert the commit on 9.x, please move the CHANGES.txt entry in main branch.
Uwe

Here is the confirmation: OpenNLP 2.3 raised minimum Java version to 17: https://opennlp.apache.org/news/release-230.html
So we need to revert this on 9.x branch.

@cpoerschke I think we need to revert this commit from 9x because the failed builds keep polluting the mailing list. I know it's the weekend, so I'll do this without waiting for your permission - hope you're ok with this while we're looking into alternatives.

OpenNLP 2.2 would be an option, which ist still compatible to Java 11.
But I don't think we should spend too much time with this. Let's release 10.0 at some point.

This PR also added a migration guide entry. Maybe move it, too.

I have, see commit 7d99e6aafbdb32e1a6c94e10e41d58a1f790630a

Thanks @uschindler and @dweiss for reverting on branch_9x here!
minor: pushed https://github.com/apache/lucene/commit/8bf10130e95c7b1b3da16606be0bf872c28fb6ed to remove the empty 9.9 to 9.10 migrate section.

Thanks everyone involved in this!


[Feature]: Add new topic "Global State Management"

Contact Details
No response
Feature Type
New react topic
Description
Add resources and content of Global State Management from already added react experts

I am interested in this feature

I am interested in this feature

@misbahafzal, Raise a PR for it by following our contribution guidelines


Blender 2.81 release

[x] Update Release notes: https://wiki.blender.org/wiki/Reference/Release_Notes/2.81/Add-ons
[ ] Update doc
[x] Check T69857 and remove workaround if fixed

For info, here are current milestones :
Release date for 2.81 is currently set to 14 November 2019
Cut-off date (major changes and features in by) 13 September 2019
Stabilizing branch creation 1 October 2019
Master branch will be open from then on, i.o.w. no freeze while 2.81 is being stabilized

Added a check on status of blender bug https://developer.blender.org/T69857
Remove commit 51ee88f1af6753abccf89f0a6c83482bdde7d70a if blender bug is solved

Release note are up to date (sept. 19)

Is someone here have time to update documentation?
@donmccurdy @emackey @pjoe @scurest

What's needed on doc?  New screenshots of texture transform nodes, I know.  Anything else?

Merging animation for example.
Here is the current release note, listing all features/fixes (all of these don't need doc update) : https://wiki.blender.org/wiki/Reference/Release_Notes/2.81/Add-ons

Umm, I tried writing a little bit on the Animation -> NLA Strips option. Is that kind of what you wanted?
Diff
Rendered

Seems we are ready for 2.81 :)


[FEATURE] add video section

Description
please add a section called "Video".
It should look like the one in the screenshot below
Screenshots

please call it "Videos"

Please assign this to my name @FrancescoXX , will add new page

i can


Atom on Chromebook?

Have you talked about bringing Atom to the world of Chromebooks as a packaged app or something? Any thoughts on that?

Hi, there is a discussion about ChromeOS!
https://discuss.atom.io/t/lets-talk-chromeos/8644

Uh nice. Hope that it will be having some fruits in the near future..

OK, so I read at least a majority of the thread above.  I've got atom working in a chroot running Ubuntu.  What I really want to understand is what exactly need to be done to get atom to run on ChromeOS as a packaged app, or in a browser without Crouton or Ubutu.


Modify model_preprocessing.py to keep the original shapes of torch parameters

torch.nn.Linear weight needs to change from [1, 1, K, N] to [K, N]
torch.nn.Linear bias needs to change from [1, 1, 32, N] to [N]
torch.nn.LayerNorm parameters needs to change from [1, 1, 32, N] to [N]
torch.nn.Embedding weight needs to change from [1, 1, vocab_size, hidden_size] to [vocab_size, hidden_size]

Updated torch.nn.Linear weight and torch.nn.Embedding weight to have the same shapes as torch.
Updated the test to be [1, N] (cannot make the shapes 1D because of the tile layout)