[Bug]: Latest version crashes TLD on startup, no log output

All of the following criteria must be met

[X] All Requirements must be installed.
[X] Changed the title so that it doesn't just says "[Bug]: "
[X] I have searched the GitHub issues for my bug, even in the closed issues.

All of the following are optional to answer

[X] Tried reinstalling the Game.
[X] Tried reinstalling MelonLoader.
[X] Tried restarting PC.
[ ] Was able to see the Start Screen.

Describe the issue.
Melon Loader 0.6.2 in combination with TLD crashes on startup, throwing an "Internal Failure" stating that it "Failed to load original DLL". Prior version (0.5.7) worked fine but unfortunately is not compatible with newer mods. Any help would be appreciated.
Did you attach your log file?
No, I could not find a log file at {Game_Directory}\MelonLoader\Latest.log

known issue, please downgrade to 0.6.1 or use the latest nightly from alpha-development if you're so inclined. Will keep this open until that version releases

Unfortunately both 0.6.0 and 0.6.1 fail with a "System.IO.IOExcpetion: Handle invalid [...]" error. Only 0.5.7 works as expected.

Modern mods with 2.27 require 0.6.1. the I/O exception smells like an antivirus thing

Fixed in v0.6.3


adds more testimonials

Description
Adds additional testimonials to the testimonial page.
By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

@stockholmux some of the logos aren't rendering to similar sizes. let me know if you have suggestions on how we can fix that.

It looks good, however the images need a little work.

wipro is huge at 188kb and 2560x2560. That's 10x file size of Goldman and double the resolution. It should be resized - really no need for the file to be that high res. On screen it's taking up 54 px square. We should over size these images to account for retina displays, but not 47x!
wipro has a ton of padding built into the image which makes it look small when scaled - when I overlay goldman and wipro, you can see the difference.
If you crop the wipro logo closer, they will appear closer in size

Fixed accordingly. Looks better!


Add self deferring functionality on dbt_cloud_job

The logic is a little more complicated than other configurations. Summarized below:
Create Job:
if self_deferring:
  create a POST request to create a job, then send a subsequent POST request to update it with it's own ID

Update Job:
if self_deferring changed:
  if self_deferring == true:
    set deferring_job_id to itself
  else:
    set deferring_job_id to the actual deferring_job_id value in the resource

Reading from remote resource:
if deferring_job_id == job.ID:
  set self_deferring in ResourceSchema to true

Tested with make test (Though I noticed there were no unit tests for Jobs, only acceptance tests, which I will transition over to you)

/ok-to-test sha=4df951de312ef340affd287367460384a1229efb

/ok-to-test sha=7c6e54943d7f4d325caaa8724a9c2119f2fff2d6

/ok-to-test sha=2f9b684d9d1af9832509097c8a6a49c45ac14f3a

/ok-to-test sha=2f9b684d9d1af9832509097c8a6a49c45ac14f3a

/ok-to-test sha=2c13051a7c1bd5788671fb7269914459f4cf3242


Positional argument - SecretsManagerRDSMySQLRotationSingleUser

Hi, I have had the following error takes 1 positional argument but 2 positional arguments (and 5 keyword-only arguments) were given
I have addressed the issue by adding host as a keyword argument to connection string on line 269.
Function: get_connection
Line: 269
Fix: conn = pymysql.connect(host=secret_dict['host'], user=secret_dict['username'], passwd=secret_dict['password'], port=port, db=dbname, connect_timeout=5)
@pj64

I am sorry, there is not enough information here to tell what is going wrong. If you are still experiencing this issue please open a support case with AWS.

I can confirm this is an issue. The pymysql library does not accept non-position arguments from v1 onwards


[Kubernetes STIG v1r11] Add support for ruleset

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #63
Special notes for your reviewer:
Release note:

Diki can now run DISA Kubernetes STIG version `v1r11` ruleset.

Nice catch. And I guess 242435 was a duplicate of 242382.


Is it possible to preview all of the images that are to be transferred?

I found here in github this:

which will preview one image. How can I preview all of the images in $flow.files?

Can you explain, why this is not working?

                    {{$index+1}}

                    {{file.name}}
                    {{file.msg}}

this is also not working:

                    {{$index+1}}

                    {{file.name}}
                    {{file.msg}}

but when i use $flow.files[0] i get a thumbnail.

Can you reproduce this on http://plnkr.co/?

can you explain this
http://plnkr.co/edit/So4LOVZJvwFIyNS3cOqf?p=preview

Ohh, it's working now.

	        {{$index+1}}
	        {{file.name}}
	        {{file.msg}}

use


Is there any plan to open the code? Looking forward to your reply!

Is there any plan to open the code? Looking forward to your reply!

Hi, thanks for stopping by! yes, there is, @aleksandrkim61 will release the code just before the ECCV takes place in-person (Oct. 23th).

Added the current rough code, will clean it up and prep instructions after coming back from ECCV.
Had trouble transferring pre-trained model and data to share - need to figure out a solution for that first, will update the repo in the 1-2 weeks after ECCV

thanks for your wonderful work, is there any instructions for the code?

Away from my laptop these days, will try to find time to add instructions and pre-trained data when I get back this weekend


Added ability to configure number of shards and replicas

A modification that allows a user to configure the number of shards and/or the number of replicas when the index is automatically created by the river plugin.
Also, since this is essentially time series data, an option was added that allows the user the option of having the river automatically create new indexes either daily, weekly, monthly, or yearly (or use a single index) and have an automatic alias combining all the indexes together as a single query point. (i.e. number of indices @ http://www.elasticsearch.org/tutorials/using-elasticsearch-for-logs/)

No move since some months. Closing.
Feel free to reopen if you think my answer was wrong.


delete vendor && optimize the way to generate apis

Signed-off-by: huone1<EMAIL_ADDRESS>

/lgtm
/approve


Can you support "H264" codec?

Expected behaviour
The original opencv version(4.2-dev) can support "H264" codec. however your version(<IP_ADDRESS>) doesn't.
Actual behaviour
write mp4 to file from webcam
Write here what went wrong.
not support H264 format codec
Steps to reproduce
python3:
codec = cv2.VideoWriter_fourcc(*'H264')
out = cv2.VideoWriter("output/11.mp4" , codec, fps, (frame_width, frame_height))

example code
python3:
codec = cv2.VideoWriter_fourcc(*'H264')
out = cv2.VideoWriter("output/11.mp4" , codec, fps, (frame_width, frame_height))
operating system
mac os 10.14
architecture (e.g. x86)
x86
opencv-python version
<IP_ADDRESS>

libx264 that backs H.264 support in FFMpeg is under GPL which is incompatible with opencv-python's MIT license. See https://www.ffmpeg.org/legal.html
If you need it, you can make a custom build of FFmpeg and then of opencv-python, directing it to link against that copy. Looks at the build scripts for how it's done and do the same with any necessary changes.

Thank you.

Thank you.

Hi @pospospos2007 now i use opencv-python 4.4.0
>>> cv2.__version__
'4.4.0'

i found opencv can write h264 video
import cv2

########3# attention, use avc1 instead of x264 or h264
fourcc_type = 'avc1'
# fourcc_type = 'mp4v'

output_path = 'output.mp4'

def main():
    # open camera
    vc = cv2.VideoCapture('/home/zj/test.mp4')
    if not vc.isOpened():
        print('Error: can not opencv camera')
        exit(0)

    ret, frame = vc.read()
    w = int(vc.get(cv2.CAP_PROP_FRAME_WIDTH))
    h = int(vc.get(cv2.CAP_PROP_FRAME_HEIGHT))
    fps = vc.get(cv2.CAP_PROP_FPS)

    fourcc = cv2.VideoWriter_fourcc(*fourcc_type)
    vw = cv2.VideoWriter(output_path, fourcc, fps, (w, h), True)
    while ret:
        vw.write(frame)
        ret, frame = vc.read()

        # cv2.imshow('frame', frame)
        # if cv2.waitKey(int(1 / fps * 1000)) & 0xFF == ord('q'):
        #     break
    # cv2.destroyAllWindows()
    vw.release()

if __name__ == '__main__':
    main()

look into opencv-python/opencv/3rdparty/ffmpeg/readme.txt
  FFMPEG build includes support for H264 encoder based on the OpenH264 library.
  OpenH264 Video Codec provided by Cisco Systems, Inc.
  See https://github.com/cisco/openh264/releases for details and OpenH264 license.
  OpenH264 library should be installed separatelly. Downloaded binary file can be placed into global system path
  (System32 or SysWOW64) or near application binaries (check documentation of "LoadLibrary" Win32 function from MSDN).
  Or you can specify location of binary file via OPENH264_LIBRARY environment variable.

and i find libopenh264 in anaconda
envs/lib$ file libopenh264.*
libopenh264.a:        current ar archive
libopenh264.so:       symbolic link to libopenh264.so.2.1.1
libopenh264.so.2.1.1: ELF 64-bit LSB shared object, x86-64, version 1 (GNU/Linux), dynamically linked, not stripped
libopenh264.so.6:     symbolic link to libopenh264.so.2.1.1

libx264 that backs H.264 support in FFMpeg is under GPL which is incompatible with opencv-python's MIT license. See https://www.ffmpeg.org/legal.html
If you need it, you can make a custom build of FFMpeg and then of opencv-python, directing it to link against that copy. Look at the build scripts for how it's done and do the same with any necessary changes.
(You are responsible for any legal considerations in the course of using the result, of course.)

Hi @pospospos2007  @native-api  now i use opencv-python 4.4.0
>>> cv2.__version__
'4.4.0'

i found opencv can write h264 video
import cv2

########3# attention, use avc1 instead of x264 or h264
fourcc_type = 'avc1'
# fourcc_type = 'mp4v'

output_path = 'output.mp4'

def main():
    # open camera
    vc = cv2.VideoCapture('/home/zj/test.mp4')
    if not vc.isOpened():
        print('Error: can not opencv camera')
        exit(0)

    ret, frame = vc.read()
    w = int(vc.get(cv2.CAP_PROP_FRAME_WIDTH))
    h = int(vc.get(cv2.CAP_PROP_FRAME_HEIGHT))
    fps = vc.get(cv2.CAP_PROP_FPS)

    fourcc = cv2.VideoWriter_fourcc(*fourcc_type)
    vw = cv2.VideoWriter(output_path, fourcc, fps, (w, h), True)
    while ret:
        vw.write(frame)
        ret, frame = vc.read()

        # cv2.imshow('frame', frame)
        # if cv2.waitKey(int(1 / fps * 1000)) & 0xFF == ord('q'):
        #     break
    # cv2.destroyAllWindows()
    vw.release()

if __name__ == '__main__':
    main()

look into opencv-python/opencv/3rdparty/ffmpeg/readme.txt
  FFMPEG build includes support for H264 encoder based on the OpenH264 library.
  OpenH264 Video Codec provided by Cisco Systems, Inc.
  See https://github.com/cisco/openh264/releases for details and OpenH264 license.
  OpenH264 library should be installed separatelly. Downloaded binary file can be placed into global system path
  (System32 or SysWOW64) or near application binaries (check documentation of "LoadLibrary" Win32 function from MSDN).
  Or you can specify location of binary file via OPENH264_LIBRARY environment variable.

and i find libopenh264 in anaconda
envs/lib$ file libopenh264.*
libopenh264.a:        current ar archive
libopenh264.so:       symbolic link to libopenh264.so.2.1.1
libopenh264.so.2.1.1: ELF 64-bit LSB shared object, x86-64, version 1 (GNU/Linux), dynamically linked, not stripped
libopenh264.so.6:     symbolic link to libopenh264.so.2.1.1

Hi @zjykzj
Following your steps, I have tried to install libopenh264 via conda.
conda install -c conda-forge openh264
but I am still getting this error in opencv-python 4.4.0
Could not find encoder for codec id 27: Encoder not found
I have also tried setting the  OPENH264_LIBRARY env variable, but got the same error.
Could you give me more details of how you achieved it?

Hi @zjykzj
Following your steps, I have tried to install libopenh264 via conda.
conda install -c conda-forge openh264
but I am still getting this error in opencv-python 4.4.0
Could not find encoder for codec id 27: Encoder not found
I have also tried setting the  OPENH264_LIBRARY env variable, but got the same error.
Could you give me more details of how you achieved it?

Hi @zjykzj
Following your steps, I have tried to install libopenh264 via conda.
conda install -c conda-forge openh264
but I am still getting this error in opencv-python <IP_ADDRESS>
Could not find encoder for codec id 27: Encoder not found
I have also tried setting the OPENH264_LIBRARY env variable, but got the same error.
Could you give me more details of how you achieved it?

hi  @pospospos2007  @native-api  @mlorenzo-alice , have you solved this problem yet?
i checked my answer again, found there have some questions. Here is my complete solution
my problem
i want to use freetype in opencv, so i need to compile myself using this repo
opencv_contrib_python
here is my solution:
# download
$ git clone --recursive https://github.com/skvark/opencv-python.git
# set freetype
$ export CMAKE_ARGS="-DWITH_FREETYPE=ON"
# enable contrib
$ export ENABLE_CONTRIB=1
# compile
$ pip wheel . --verbose.

After completing the above operations, you can find opencv_contrib_python-<IP_ADDRESS>-cp38-cp38-linux_x86_64.whl
$ pip install opencv_contrib_python-<IP_ADDRESS>-cp38-cp38-linux_x86_64.whl

openh264
i also tried this operation
$ conda install -c conda-forge openh264

but it doesn't work, so i download the openh264-2.1.1-h780b84a_0.tar.bz2 from conda-forge / packages / openh264
use pip to install it
$ pip install --use-local openh264-2.1.1-h780b84a_0.tar.bz2

demo
After completing the above operations, you should be able to find and use h264 in opencv
．．．
．．．

    def get_output_file(self, path, fps=30):
        """
        Return a video writer object.
        Args:
            path (str): path to the output video file.
            fps (int or float): frames per second.
        """
        return cv2.VideoWriter(
            filename=path,
            fourcc=cv2.VideoWriter_fourcc(*"avc1"),
            fps=float(fps),
            frameSize=(self.display_width, self.display_height),
            isColor=True,
        )
．．．
．．．

Hi @zjykzj
Following your steps, I have tried to install libopenh264 via conda.
conda install -c conda-forge openh264
but I am still getting this error in opencv-python <IP_ADDRESS>
Could not find encoder for codec id 27: Encoder not found
I have also tried setting the OPENH264_LIBRARY env variable, but got the same error.
Could you give me more details of how you achieved it?

hi  @pospospos2007  @native-api  @mlorenzo-alice , have you solved this problem yet?
i checked my answer again, found there have some questions. Here is my complete solution
my problem
i want to use freetype in opencv, so i need to compile myself using this repo
opencv_contrib_python
here is my solution:
# download
$ git clone --recursive https://github.com/skvark/opencv-python.git
# set freetype
$ export CMAKE_ARGS="-DWITH_FREETYPE=ON"
# enable contrib
$ export ENABLE_CONTRIB=1
# compile
$ pip wheel . --verbose.

After completing the above operations, you can find opencv_contrib_python-<IP_ADDRESS>-cp38-cp38-linux_x86_64.whl
$ pip install opencv_contrib_python-<IP_ADDRESS>-cp38-cp38-linux_x86_64.whl

openh264
i also tried this operation
$ conda install -c conda-forge openh264

but it doesn't work, so i download the openh264-2.1.1-h780b84a_0.tar.bz2 from conda-forge / packages / openh264
use pip to install it
$ pip install --use-local openh264-2.1.1-h780b84a_0.tar.bz2

demo
After completing the above operations, you should be able to find and use h264 in opencv
．．．
．．．

    def get_output_file(self, path, fps=30):
        """
        Return a video writer object.
        Args:
            path (str): path to the output video file.
            fps (int or float): frames per second.
        """
        return cv2.VideoWriter(
            filename=path,
            fourcc=cv2.VideoWriter_fourcc(*"avc1"),
            fps=float(fps),
            frameSize=(self.display_width, self.display_height),
            isColor=True,
        )
．．．
．．．

Thank you @zjykzj. Those instructions did work!
Note: I just had to replace pip install --use-local openh264-2.1.1-h780b84a_0.tar.bz2 with conda install --use-local openh264-2.1.1-h780b84a_0.tar.bz2)

Thank you @zjykzj. Those instructions did work!
Note: I just had to replace pip install --use-local openh264-2.1.1-h780b84a_0.tar.bz2 with conda install --use-local openh264-2.1.1-h780b84a_0.tar.bz2)

Thank you @zjykzj. Those instructions did work!
Note: I just had to replace pip install --use-local openh264-2.1.1-h780b84a_0.tar.bz2 with conda install --use-local openh264-2.1.1-h780b84a_0.tar.bz2)

my fault ! thank you for fix it

Thank you @zjykzj. Those instructions did work!
Note: I just had to replace pip install --use-local openh264-2.1.1-h780b84a_0.tar.bz2 with conda install --use-local openh264-2.1.1-h780b84a_0.tar.bz2)

my fault ! thank you for fix it

@zjykzj Can you show the solution for windows. How to solve this problem in windows

@zjykzj Can you show the solution for windows. How to solve this problem in windows

My problem is solved with the following. May be all are not necessary for everyone but I am sharing what I did. I am using anaconda with python 3.8

Opencv 4.5.1 (Its must because for opencv less than 4.4 this codec is not working) using following command
pip install opencv-python
Opencv contrib 4.5.1  using following command
pip install opencv-contrib-python
Install Openh264 using following command
conda install openh264
av for ffmpeg using following command
pip install av

My problem is solved with the following. May be all are not necessary for everyone but I am sharing what I did. I am using anaconda with python 3.8

Opencv 4.5.1 (Its must because for opencv less than 4.4 this codec is not working) using following command
pip install opencv-python
Opencv contrib 4.5.1  using following command
pip install opencv-contrib-python
Install Openh264 using following command
conda install openh264
av for ffmpeg using following command
pip install av

@zjykzj Can you show the solution for windows. How to solve this problem in windows

hi @imihassan, my daily working environment is Ubuntu 18.04, so I can't actually do it in windows
but there are may be two solutions:

use WSL in win10;
the operation of git/pip/conda/anaconda on Linux / Windows platform is consistent, so what you need to pay attention to is how to set environment variables and install the compiled version of windows openh264.

I'm glad to see that you've solved the problem

@zjykzj Can you show the solution for windows. How to solve this problem in windows

hi @imihassan, my daily working environment is Ubuntu 18.04, so I can't actually do it in windows
but there are may be two solutions:

use WSL in win10;
the operation of git/pip/conda/anaconda on Linux / Windows platform is consistent, so what you need to pay attention to is how to set environment variables and install the compiled version of windows openh264.

I'm glad to see that you've solved the problem

Any idea why on windows it doesn't require to rebuild opencv-python https://stackoverflow.com/questions/41972503/could-not-open-codec-libopenh264-unspecified-error, but on linux we need to ?
Also I am trying to rebuild on ubuntu 20, but libavcodec seem to depend on libx264, do you know which dependencies I need to install to build opencv-python without x264 but with libopenh264 ?

Are you sure @zjykzj that your compiled opencv doesn't use x264 instead of openh264 ?

Hello, does anyone still get this problem ?
I'm using anaconda enviroment
I have installed opencv <IP_ADDRESS> from pip and libopenh264 from conda.
~/anaconda3/envs/open/lib$ file libopenh264.*
libopenh264.a:        current ar archive
libopenh264.so:       symbolic link to libopenh264.so.2.1.1
libopenh264.so.2.1.1: ELF 64-bit LSB shared object, x86-64, version 1 (GNU/Linux), dynamically linked, not stripped
libopenh264.so.5:     symbolic link to libopenh264.so.2.1.1
libopenh264.so.6:     symbolic link to libopenh264.so.2.1.1

>>> cv2.__version__
'4.5.1'

but when I attemp to save the video with h264 codec
It still show up the error message.
Could not find encoder for codec id 27: Encoder not found

Is it necessary  build opencv from source code?

Any idea why on windows it doesn't require to rebuild opencv-python https://stackoverflow.com/questions/41972503/could-not-open-codec-libopenh264-unspecified-error, but on linux we need to ?

Afaik, on Linux the codec library must be linked to the OpenCV binary during the build. On Windows, OpenCV has a feature that allows it to load the external shared library during runtime.


unable to load model with custom objects on a Fresh Python Environment!

Hi all, i have an issue in loading a model with custom loss function in a fresh-separate IPython Environment. Following is a sample code i used.
import tensorflow as tf
import tensorflow.keras.backend as K
import numpy as np

# creating the model
x = tf.keras.Input(shape = 10, name = "Input")

dense1 = tf.keras.layers.Dense(units = 10, activation = None)(x)
dense2 = tf.keras.layers.Dense(units = 10, activation = None)(dense1)
dense3 = tf.keras.layers.Dense(units = 10, activation = None)(dense2)
output = tf.keras.layers.Dense(units = 10, activation = None)(dense3)

model = tf.keras.Model(inputs = x, outputs = output)

# custom loss function
def custom_loss_func(y_true, y_pred):
    return K.mean(K.square(y_true - y_pred))

# compiling the model
model.compile(loss = custom_loss_func, optimizer = "sgd")

# sample fit
X = np.linspace(0,1,10).reshape(1,10)
Y = X**2

model.fit(X,Y,epochs = 10, batch_size = 1)

model.save("custom_model.h5")

# test code to load along with the model. and it works
#  new_model = tf.keras.models.load_model("custom_model.h5", custom_objects = {"custom_loss_func":custom_loss_func})

in the last line, if i uncomment and run the script means, it works fine as the function is already defined on top.
# test code to load along with the model. and it works
new_model = tf.keras.models.load_model("custom_model.h5", custom_objects = {"custom_loss_func":custom_loss_func})

But if i run above line on a separate fresh IPython environment which hasn't seen this code yet, it throws error as below.
In [34]: new_model = tf.keras.models.load_model("custom_model.h5", custom_objects = {"custom_loss_func":custom_loss_func})
---------------------------------------------------------------------------
NameError                                 Traceback (most recent call last)
<ipython-input-34-77825814faa4> in <module>
----> 1 new_model = tf.keras.models.load_model("custom_model.h5", custom_objects = {"custom_loss_func":custom_loss_func})

NameError: name 'custom_loss_func' is not defined

Its correct, because that function object isn't defined in the new environment.
I tried by defining a dummy function in the same name and try to load model, it didn't work as well, the errors are about mismatched input parameters, then mismatched return object etc..
Is there any working way to load the model with custom objects back on a fresh environment?

@Ramkumar47
I have tried in colab with TF version 2.2 , and i am not seeing any issue after defining the custom loss function and loading the model.Please, find the gist here.Thanks!

Now i understood, there is no way to load a model with custom objects, without pre-defining them in the new environment.. Thank you @ravikyram


[Feature][Cherry-pick][Branch-2.5] Support sort key be able to nullable. (#15641)

Support sort key be able to nullable.

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)


[Q]  Run specific hyperparameteer combinations in a sweep.

Hi WandB,
I want to use WandB as an experiment runner on a cluster.  I have a set of hyperparameters and some values, but i only want to run a very specific* and small set of the possible combinations of these hyperparameters (as opposed to all combinations).
This custom sweep seems to have been discussed before (e.g. here).  There are a lot of complex ideas about logic etc, but I think a sensible first step would be just allowing the user to supply an explicit "manifest".  What I'm imagining is instead of a standard config, you can specify something like:
sweep = {
    "method": "hardcoded",  # "hardcoded" would indicate "parameter_sets" will just be explicitly iterated over
    "name":   "sweep",
    "metric": "reward",
    "parameter_sets": {
         "exp1": {
              "rnn_cell_type": "gru",
              "penalty_term": 0.0,
         },
         "exp2": {
              "rnn_cell_type": "gru",
              "penalty_term": 1.0,
         },
         "exp3": {
              "rnn_cell_type": "vanilla",
              "penalty_term": 1.0,
         },
    },
}

e.g. i don't need to run the "vanilla" with a reward penalty of zero.  And then WandB does its magic behind the scenes generating the string to call each config as per usual.  In this, each config could even have a variable number of parameters, eg:
sweep = {
    "method": "hardcoded",  # "hardcoded" would indicate "parameter_sets" will just be explicitly iterated over
    "name":   "sweep",
    "metric": "reward",
    "parameter_sets": {
         "exp1": {
              "rnn_cell_type": "gru",
              "penalty_term": 0.0,
         },
         "exp2": {
              "rnn_cell_type": "gru",
         },
         "exp3": {
              "rnn_cell_type": "vanilla",
         },
    },
}

e.g. if the penalty_term  defaults to 1.0.
Thanks,
A
*A lot of the hyperparameters correspond to physiological conditions, and i only want to test specific combinations out of the many possible combinations.

In reference to the thread references above (here), that could be solved as:
sweep = {
    "method": "hardcoded",  # "hardcoded" would indicate "parameter_sets" will just be explicitly iterated over
    "name":   "sweep",
    "metric": "reward",
    "parameter_sets": {
         "exp1": {
              "approach": a,
              "sub_approach": a1,
         },
         "exp2": {
              "approach": a,
              "sub_approach": a2,
         },
         "exp3": {
              "approach": b,
              "sub_approach": b1,
         },
         "exp4": {
              "approach": b,
              "sub_approach": b2,
         },
    },
}

Hello @andrewwarrington !
Thanks for the suggestion! I can certainly make a feature request for you and see what our Engineers can do about this.

Does Hydra allow you to do this?

This is possible, although not the cleanest since Hydra does not use subclass the primitive Python dictionaries.
It can work (as seen here) and wandb does integrate with hydra (example report) but a better workaround would to use wandb with Optuna with the WeightsAndBiasesCallback since that would integrate better.


Is there a color picker for formly?

[ ] bug report => search github for a similar issue or PR before submitting
[x] feature request
[ ] support request

Is there a color picker for Formly? If not, any guide on how to go forward and build one?
Thanks

Create a custom template.
Use this guide:
https://ngx-formly.github.io/ngx-formly/guide/custom-formly-field
Use a library like:
https://www.npmjs.com/package/ngx-color-picker
Here is a stackblitz example of both put together.
https://stackblitz.com/edit/ngx-formly-custom-template-zhho1m?file=app/formly-field-input.ts
Someone might have a better way to bind, this was just my first stab.

Thanks @thorgod What's a custom template in general? Is it different from custom fields?

Well its just wording, in terms of ngx-formly it just uses the name custom template to describe a custom field.  So no its not different from a custom field.

here is another one for material https://stackblitz.com/angular/xlrjknqxmvr

here is another one for material https://stackblitz.com/angular/xlrjknqxmvr

Is there a color picker there? I can only see auto-complete control. Thanks @aitboudad

Ooh I've missed the save button :cry:, it was the mix of the following packages:

https://scttcper.github.io/ngx-color/
https://github.com/ncstate-sat/popover
material input

I'll try to re-create another later.

https://stackblitz.com/edit/angular-arsdhq


Where is the last layer (1280 channels) in MobileNetV2 implementation?

In the model.py, I didn't find the last layer (1280 channels) after No.16 inverted_res_block. Why? It exists in MobileNetV2 paper and tensorflow.deepLab implementation.

Did you find the answer?


⚠ Processors has degraded performance

In 9704acc, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 46 ms

Resolved: Processors performance has improved in 20bad44.


[Suggestion] Improve accessibility for reading

Suggested change

Improve the readability of text within the application.
Make a distinction between the tooling and the GitHub repo redirect.

Why

The black font on a dark gray background is quite difficult to read and raises accessibility concerns.

Comments

This happens elsewhere in the app but for now this PR makes minor changes to the tool list on the main page.
If this suggestion is helpful, I'll be happy to look into other sections of the app - maybe even introduce a theme picker🌈

Preview of changes
Before

After

Opened PR: https://github.com/YourAverageTechBro/DevToolboxWeb/pull/78


feat: add timeout support

Changelog

Add s!timeout.
Add events.timeout configuration key, for logging manual actions.

Private changelog
#2595 introduced a few bugs... all fixed in this PR.

Added action hooks for the moderation logger.
Fixed double completion, causing timed entries to duplicate the logs.
Fixed ModerationActionTimeout not setting the data correctly.

TODO

[ ] Figure out why manual actions sometimes doesn't fire the guildMemberUpdate event.


Fix logical plan optimization will execute twice in SQL mode

Which issue does this PR close?
Closes https://github.com/apache/arrow-datafusion/issues/1182
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

A summary of what the PR will do.

I think it might be ok to optimize the plan twice (in other words, perhaps we can close the ticket as "working as expected"?). Do we have any examples of the double optimization causing problems (or taking overly long)?
I think @Dandandan was also hinting at this point in his comment at #1183 (comment)
Adding a lot of additional code to LogicalPlan (is_optimized on every variant) seems like it will be a very large change without much benefit, though perhaps I am overlooking potential benefits

OK, No code is best code😄

A summary of what the PR will do.

I think it might be ok to optimize the plan twice (in other words, perhaps we can close the ticket as "working as expected"?). Do we have any examples of the double optimization causing problems (or taking overly long)?
I think @Dandandan  was also hinting at this point in his comment at https://github.com/apache/arrow-datafusion/pull/1183#discussion_r739082628
Adding a lot of additional code to LogicalPlan (is_optimized on every variant) seems like it will be a very large change without much benefit, though perhaps I am overlooking potential benefits

Yes I was hinting at that.
If we are able to show otherwise (examples of logical plan optimization takes very long) then we can see if we can optimize for this case. Otherwise I agree "working as expected" should be the conclusion.

Thanks again for looking into this @xudong963


Update analyse_en.json

Fixed infobox "Received warnings".

@HelTob
Would you also take a look at issue https://github.com/corona-warn-app/cwa-website/issues/2084 to make it clearer that the count of Received Warnings is underestimated because it only counts users who have subscribed to Data Donation?

@HelTob

We're on issue #2084 and will change the description in the near future.

I'm happy to hear that! Thank you.


It's difficult to free-draw an angle

See the vid. It's so close to being right, but there's always this wiggle that spoils the angle. Ideally I'd like it to produce a proper rounded corner when I make a sharp turn.
https://user-images.githubusercontent.com/810438/108633758-eeb48f80-746d-11eb-90a5-e79cd76f3ce4.mov

https://user-images.githubusercontent.com/125676/108860857-49c5be00-75f7-11eb-9b17-d2a79f40547f.mp4

https://excalidraw.com/#json=5647123399835648,q194OOTCGKy2qdSmzmKBjg

Well, fwiw I do want them to be rounded. It's just that the algorithm that calculates the rounding doesn't seem to do very well with this particular path.

FWIW, implementation-wise, it's just a bezier curve without explicit control points. We had a discussion to add support for edit the control points, but concluded it's out of scope at that point. For limited use cases, we added rounded rectangles (rational behind it was we wanted to add sharp-edged lines.) Also, even if we edit control points, roughjs doesn't support continuous lines (yet). https://github.com/rough-stuff/rough/issues/168


Instructions in step 2 are invalid. Populating ID and Key info in .env and config files does nothing once the application is built and run

See step 2 in: https://github.com/hashgraph/guardian#installation
Step 2 instructions are not valid, therefore confusing/unnecessary.
Repro steps:
I added my testnet credentials to the files, once the app is built and run the credentials have to be entered again or generated randomly.
On that same note, when adding the credentials, if I add the public key instead of the private key of the account I get useful errors for the Installer accounts (something along the lines of "Invalid key length"). The error message for the RootAuthority is not helpful

As of today, @Pyatakov needs to provide @danielnorkin witih Text and he will verify the instructions. Alex is working on this one.

@danielnorkin As of today, we need to discuss this with you. Assigning this to you.

@ed-marquez - Thanks for the ticket. We have updated the Readme in step 2 of the usage guide to address this. "Please keep in mind that this Hedera Operator ID and Operator Key is used for this reference implementation as a placeholder until there is a wallet integration. There will be other steps in the Demo Usage Guide that will require the generation of Operator IDs and Operator Keys. It is important to mention that the Operator IDs and Operator Keys in the .env will be used to generate demo accounts."
We understand that this is confusing, but it's a placeholder until we integrate a wallet to make the creation of demo accounts more simple.
Please see the commit that addresses the change in the readme. Also, please note that this is in the Develop branch. And will be doing a release next week, which will push all code from Develop to main. https://github.com/hashgraph/guardian/commit/2e21f84589754a8b8e957bc6d24e42987e7a7f5f


Include only files that match pattern

Hi, nice project, thanks for building!
I was working on an iOS project yesterday, where there are lots of random JSON files for assets and plist files and whatnot in the project directory, but I'd only like to include .swift files in the prompt.
I think an option like --include-only or maybe just --only would be nice. It could be made either or with the ignore options so there can't be rule clashing.

I just found myself wanting exactly the same thing - I so often want to say "just include the .py and .md` files", for example.
I'm going with -e py -e md - where -e is short for --extension.

fd is an example of a command that uses -e for this: https://github.com/sharkdp/fd/blob/master/README.md#searching-for-a-particular-file-extension


Cyclone surge mapping

Hi @noorhasan1596 thanks for weighing in. Can you describe what you have in mind?


wire in SqlExceptionHelper which is now useful in Vert.x 4.4.0

Note that this work incorporates an upgrade to Vert.x 4.4.1.

It fails for Sql Server because of this: https://github.com/hibernate/hibernate-orm/pull/6323

Superseeded by https://github.com/hibernate/hibernate-reactive/pull/1603


Allow using import tools from command line

Operating system or device - Godot version:
Godot 2.1.3+
Issue description:
Allow using import tools from command line so Godot users can create scripts and automatize assets import.
For example:

I could develop some scripts to help me refresh a group of fonts given a JSON configuration file, instead of manually re-importing one by one.
I could batch import multiple images as textures, from multiple directories, and use the same script over and over as I update the source images.
Have an script to monitor original images and watch for changes, automatically importing the assets as textures.

Steps to reproduce:
n/a
Link to minimal example project:

Related/dupe of #533, I guess.

Indeed.


Creating Linked-Lists folder in cpp and adding fundamental algorithms

Hey @gantavyamalviya , I want to add link list algorithms for reversing , removing loop and merging.

Hey @gantavyamalviya , I want to add many leetcode questions on Linked Lists.

Hey @gantavyamalviya , I would like to add some Important LinkedList Algorithms .


Two level toolkit design with show more/less

Maybe separate PR:

fading background as per design for active one (just add to the effect)
activation for Explore, no need yet but it's coming in search

See #376

Visually, I still don't like "SHOW MORE" but I think we should probably merge without worrying about that and then consider options (mostly because this is holding up other PRs).

Spotted a spacing regression in Reference.
New:

Old:

Preview URL is:
https://review-python-editor-next.microbit.org/two-level-design-2/

There's an issue open about an issue with audio which might be related: #317

Scrolling to reference documentation from the link in the signature help is a little inconsistent. It always scrolls, but not necessarily to the correct place (e.g. for display.show(), I can see a 'show less' at the top of the scroll area).

I think this is #325 in "show more" form. We don't uniquely identify the overload of scroll so we expand both which can cause the offset issue. Can you reproduce this with a method that doesn't have an overload?

The link in signature help for anything in audio only takes you to references docs level 1, and not into the audio section for some reason.

I'll look at this with #317.

If not already inside the relevant second level of the reference toolkit, clicking the link in the signature help a second time after scrolling has finished, appears to cause a flash/glitch in the toolkit. E.g., display.set_pixel()

New issue:
URLs are utterly messed up! E.g. ends with %5Bobject%20Object%5D. We need to unwrap Anchor.

Scrolling to reference documentation from the link in the signature help is a little inconsistent. It always scrolls, but not necessarily to the correct place (e.g. for display.show(), I can see a 'show less' at the top of the scroll area).

I think this is #325 in "show more" form. We don't uniquely identify the overload of scroll so we expand both which can cause the offset issue. Can you reproduce this with a method that doesn't have an overload?

We could and I've fixed it with a delay so the final height of the container accounts for the show-more expand before the scroll.

The link in signature help for anything in audio only takes you to references docs level 1, and not into the audio section for some reason.

I'll look at this with #317.

Still leaving that to look at separately.

If not already inside the relevant second level of the reference toolkit, clicking the link in the signature help a second time after scrolling has finished, appears to cause a flash/glitch in the toolkit. E.g., display.set_pixel()

I can't make this happen.

We reproduced this and I've finally fixed it in Slide.tsx (unintentional tree structure change causing component recreation).

New issue:
URLs are utterly messed up! E.g. ends with %5Bobject%20Object%5D. We need to unwrap Anchor.

This is also fixed.
I think this PR is good to go. I'll address the fading background separately.

Spotted a spacing regression in Reference. New:  Old:

This is also fixed.

One issue to look at after merge:

If there's a CM in the Collapse then it seems to fade out but not slide up, e.g. display.scroll. Clearer if you remove the other content in the Collapse. What's going on?

If there's a CM in the Collapse then it seems to fade out but not slide up, e.g. display.scroll. Clearer if you remove the other content in the Collapse

Now #407


 Thanks in advance

please reopen the issue, since it still exist, and any one please have a look on this??

Hey @Madhu-cherukuri, this seems like a configuration problem. We have end-to-end tests to check these type futures and we haven't experienced any regressions. You might need to provide more information.
And also you could ask your questions on CNCF Slack #thanos channel you would get more help for these type of issues.


ActiveGate state check fails for running ActiveGate

See: https://github.com/dynatrace-oss/dynatrace-azure-log-forwarder/blob/a45dafe8ea40582ab85b128702c26df92dab66c6/deployment/dynatrace-azure-logs.sh#L103
ActiveGate state check checks if response from ${TARGET_URL}/rest/health equals RUNNING string, but mentioned ActiveGate endpoint returns "RUNNING", so check should expect \"RUNNING\" string.
Fixed code:
check_activegate_state() {
  if ACTIVE_GATE_STATE=$(curl -ksS "${TARGET_URL}/rest/health" --connect-timeout 20); then
    if [[ "$ACTIVE_GATE_STATE" != "\"RUNNING\"" ]]; then
      echo -e ""
      echo -e "\e[91mERROR: \e[37mActiveGate endpoint is not reporting RUNNING state. Please verify provided values for parameters: --target-url (${TARGET_URL})."
      exit 1
    fi
  else
      echo -e "\e[93mWARNING: \e[37mFailed to connect with provided ActiveGate url ($TARGET_URL) to check state. It can be ignored if ActiveGate does not allow public access."
  fi
}

@kzzalews could you tell me what was the setup and/or bash you were running this against? I will probably add a check for both options for now, but still would like to know what was the scenario when it equals ""RUNNING"" - for me it's still "RUNNING" (both locally and in Azure CloudShell) and the script works only before the "fix"

Temp fix for different responses from DoK and normal Dt clusters - checking both versions
https://github.com/dynatrace-oss/dynatrace-azure-log-forwarder/pull/55


fixfix bug that Read garbled files caused be crash.

Proposed changes
Issue Number: close #xxx
fix bug that read garbled files caused be crash.

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall


For now only facial recognition api's work

So, you know that the URLs of API's has a version number. For now, this version number seems to be hardcoded as "v3". As all of the facial recognition services have this version number, this never had been an issue until I tried to use other APIs like Body Recognition and Image Recognition.
And since face swap API is included in the Image Recognition Category. This API is not working either.
As for the solution I am currently planning to create a list of versions including all the API versions and query the versions from this list. But the problem with this solution is that every time a change happens in these values. List has to be updated. That's why maybe also a list updater or something like that will be needed. List updater will get all the documentation from the face++ documentation page and parse all the values.

or maybe we can go even further and hardcode all possible API's but for the sake of back-compatibility still, keep the old method.


No gulp test in prepublish

Fixes https://github.com/redfin/react-server/issues/6.
Currently redundantly testing during npm install in Travis.
Our actual publishing to npm also frequently times out and has to retry.
Can we just publish when travis is green or we've manually run gulp test ourselves?
Do we need a publish script that wraps testing and publishing as recommended
in https://github.com/npm/npm/issues/3059#issuecomment-12282746?

I think lerna has a way to specify a per-project prepublish file, or something - maybe that could run gulp test. I have no idea if that's better than what's going on here, but it seems like it might be, because it wouldn't be running during npm's prepublish step.

Yes, lerna supports per-project prepublish scripts, but it doesn't look like they check the output at all.  So if our tests failed, we'd still proceed to publish.
Is running our tests (and linter) in CI enough?

seems reasonable to me -- I don't think we want anyone other than the CI bot to publish.

I don't think we want anyone other than the CI bot to publish.

Uhh... we've been publishing by hand so far. :grin:

ed: ideally, no-one other than the CI bot would publish.

Merging this.  Publishing fails too often.


PipTest: Update the expected results, again

This change is

Actually, the itsdangerous dependency was not downgraded, but upgraded again.

I've added a commit to also fix NpmVersionUrlTest.


[v1.14] stm32: k_sleep() actual sleep times are different than its input

Describe the bug
k_sleep() generally sleeps more than the intentional time. Starting from 1 millisecond and for exactly every 10 values there is a new fixed time that the system sleeps. I've used the Analog Discovery 2 to contact the measurements on a GPIO pin. The effect seems consistent as you can see in the graph I've made:

Here you can see the spreadsheet with the measurements.
This is obvious for small sleep times but, it is not so observable for greater sleep times.
I haven't tried to diagnose the issue. However, on master (f5d2b3271a), the behavior is the expected one.
To Reproduce
I've tried the below code on nucleo_l496zg on top of v1.14-branch (3132a41600348a6a546c55c35c8f46822973c459):
#include <zephyr.h>
#include <device.h>
#include <gpio.h>

#define PIN_CONTROLLER	"GPIOF"
#define PIN	13

struct device *gpiof;

void test_sleep(int msec)
{
    gpio_pin_write(gpiof, PIN, 1);
    k_sleep(msec);
    gpio_pin_write(gpiof, PIN, 0);
}

void main(void)
{
    gpiof = device_get_binding(PIN_CONTROLLER);
    gpio_pin_configure(gpiof, PIN, GPIO_DIR_OUT);
    gpio_pin_write(gpiof, PIN, 0);

    /* You can change the range as you please */
    for (u16_t i = 1; i <= 20; i++) {
        test_sleep(i);
        k_busy_wait(100);
    }
}

Expected behavior
I expected the sleep times that I put as input to k_sleep().
Impact
Really weird and frustrating behaviors on the final application.
Environment (please complete the following information):

OS: GNU/Linux
Toolchain Zephyr SDK

@galazari

close as stale


Fixed #35901 -- conf: Protected users from accidental debug mode for security.

Trac ticket number

ticket-35901
Branch description
We're helping users that enable/disable debug mode based on e.g. environment variables for good, but gone wrong:

good: DEBUG = os.environ.get("DEBUG", "False") == "True"
(bad: DEBUG = os.environ.get("DEBUG", "True") != "False" … but not the point here)
very bad: DEBUG = os.environ.get("DEBUG", False) — will end up True for any non-empty string including "False" and "0"

I have seen this mistake happen in real life at https://github.com/climateconnect/climateconnect/pull/1331 with very unfortunate consequences. Please help me help the next user not end up in the same situation.
I'm happy to extend docs and tests when I have confirmation on the general direction. Thank you!
Checklist

[x] This PR targets the main branch.
[x] The commit message is written in past tense, mentions the ticket number, and ends with a period.
[x] I have checked the "Has patch" ticket flag in the Trac system.
[ ] I have added or updated relevant tests.
[ ] I have added or updated relevant docs, including release notes if applicable.
[ ] I have attached screenshots in both light and dark modes for any UI changes.

Closing per ticket.


formatSelectionTooBig missing in 4.0?

Is there a ways to override the default text when a user has reached their max selectable options for a multi-select box?

Noted in the release notes, the formatters were moved to the language option.
{
  language: {
    maximumSelected: function () {
      return 'You can't select any more.';
    }
  }
}


Trainer should overwrite max_epoch when max_time is specified

🚀 Feature
Currently, the Trainer terminates at max_epoch (1000) even if the max_time is specified. Even if min_epoch is set to be greater than 1000, the Trainer should either overwrite the max_epoch value, or the documentation should state that min_epoch is only applicable if early_stopping is enabled. It could also throw warning when min_epoch is set while early_stopping is disabled.
Motivation
To continue training for X amount of time (e.g 7 days) when you don't know the maximum number of epochs and no early_stopping/criteria is specified.
Pitch
I think overwriting max_epoch with values of min_epoch or max_time should be reasonable when they are larger than max_epoch. I wanted to train a big model for 2 days, but after 1 day if was stopped despite setting the max_time and min_epoch parameters.
If you are concerned with the duration of training, or you are interested in the asymptotic behavior of your model in infinite time horizon,   it makes sense to stop training when it reaches a max_time regardless of max_epoch.
Alternatives
The alternative is to set a very large max_epoch to ensure the training won't stop until max_time is reached.

@awaelchli - should we pass max_time to the fit loop constructor and skip setting the default of max_epochs if max_time is not None?
https://github.com/PyTorchLightning/pytorch-lightning/blob/4af8eff0a13309b2e030c423db91156b1e869fac/pytorch_lightning/loops/fit_loop.py#L51
This also raises what the default should be if no stopping condition is passed to the trainer (e.g. no max steps / max epochs / max time) - should this be an infinite while loop and require an external signal to kill the job? Is it conceivable someone wants to train forever, or for an absurdly long amount of time where they neither know the number of epochs or time it takes for convergence? Should we allow this only if there's an early stopping callback attached to the trainer?

Even if min_epoch is set to be greater than 1000, the Trainer should either overwrite the max_epoch value, or the documentation should state that min_epoch is only applicable if early_stopping is enabled. It could also throw warning when min_epoch is set while early_stopping is disabled.

Would this be needed if we addressed the max time issue? Overwriting the arguments to me doesn't feel right. If a user is specifying min_epochs > max_epochs, we should raise a misconfiguration exception.


Xcode 11.4.1

Xcode 11.4.1 has just been released. Could the Catalina images be upgraded?

I saw the 11.4.1 image being commit, so gave it a go. It seems the OS X instance won't start yet. Reverting to 11.4...

11.4.1 images are available now. Feel free to try them out!

Something is not right with the 11.4.1 images. I get 3 (out of 8) "instance failed to start" messages with it.

But the ones stated worked OK?

It actually seems to have "recovered", so may not have been related to the 11.4.1 version after all. I restarted the failed jobs and all went through.
Thank you, case closed.


failed transfer issue

HI, My transfer from coinbase pro to bittrex account failed, however I dont see the fund in my account,
all data items are correct, but the transaction failed
this is the transaction hash
https://stellar.expert/explorer/public/tx/56e6403d122ca53b67e0fbd517b738b9fe275229121885b25cf91e4d4701cd3b
Thanks.

Hello,
Judging from the transaction result, the payment failed because you didn't have enough funds on the account balance at the moment when the transaction has been submitted. It failed, so funds shouldn't be deducted from your balance.


Another track ID in annotation and on opened task

Hello everyone :)
I uploaded own model to CVAT, I made task with video and I used auto annotation.
In my case I have video with 126 frames, and model recognized total 925 boxes.
When I use dump annotation "CVAT XML 1.1 for images", everything looks ok and logically.
But when I use "CVAT XML 1.1 for video", I don't understand how works the naming of track Id in dumped xml file, because in opened task in CVAT's window I see another numbers. Total number of tracks is the same like in the CVAT's task window (925).
I supposed that was a problem with incorrect dumping annotation, but I made a test. I added new task with the same video and uploaded annotation. Boxes are on correct places so it means that CVAT understand this naming.
Numbers of track are from 0 to 924, but numbers in task's window ale from 1 to 925. I see that numbers of tracks are totally disagree with numbers of instances in task's window. They aren't even in close proximity to each other. Track with id = 916 on frame 49, can have BB on instance id = 123 on frame 49. Only frame is the same.
How it works?

Hi @sbanori
all tracks are indexed during dump and track IDs in the dumped file are not correspond to IDs in the CVAT's task window, it's not a bug. We have 'track id' attribute for 'CVAT for video' format by historical reason and for backward compatibility.
https://github.com/opencv/cvat/blob/develop/cvat/apps/annotation/cvat.py#L309:L312

I'll close the issue for now, don't hesitate ask your questions on gitter.


Callouts and other small formatting

Section: Tasks > Configure Pods and Containers

This change is

@chenopis Should I add links to staged page so it's easier for reviewers to see the changes? I'm going to be spamming a lot of these.


Custom-Function, unclear Exception-Message

My custom-function returned in function execute(..) null instead of a string-instance.
Got following exception-message
FAILURE: Caused by: NoSuchFunctionException: Function: ….  could not be found
I think this exception-message is not correct. Maybe an Message like “function  did return NULL” would be more helpful.

Are you sure that, your custom function is really executed. First check your configuration and look in citrus-context.xml if the custom function is configured correctly there, and if yes put a break point in your custom function and check if it is really executed.
Best regards
Tamer
From: krguse<EMAIL_ADDRESS>Sent: Monday, March 14, 2016 15:33
To: christophd/citrus<EMAIL_ADDRESS>Subject: [citrus] Custom-Function, unclear Exception-Message (#84)
My custom-function returned in function execute(..) null instead of a string-instance.
Got following exception-message
FAILURE: Caused by: NoSuchFunctionException: Function: …. could not be found
I think this exception-message is not correct. Maybe an Message like “function did return NULL” would be more helpful.
—
Reply to this email directly or view it on GitHubhttps://github.com/christophd/citrus/issues/84.

Hi Tamer,
thanks for your quick response.
And yes I am sure that everything is correctly configured.
Changing  my return-value from
return null;
to
return “”;
let the exception disappear and everything is running fine.
The exception “NoSuchFunctionException” sounds a little bit like reflection. I assume that the returned value is analyzed and because null is not a valid string-instance (not the correct class) the analyzer is “confused” .
Please try to reproduce it.
Kind Regards
Kai
Von: vdsrd<EMAIL_ADDRESS>Gesendet: Montag, 14. März 2016 3:39
An: christophd/citrus
Cc: Guse, Kai-Rasmus
Betreff: Re: [citrus] Custom-Function, unclear Exception-Message (#84)
Are you sure that, your custom function is really executed. First check your configuration and look in citrus-context.xml if the custom function is configured correctly there, and if yes put a break point in your custom function and check if it is really executed.
Best regards
Tamer
From: krguse<EMAIL_ADDRESS>Sent: Monday, March 14, 2016 15:33
To: christophd/citrus <citrus<EMAIL_ADDRESS>Subject: [citrus] Custom-Function, unclear Exception-Message (#84)
My custom-function returned in function execute(..) null instead of a string-instance.
Got following exception-message
FAILURE: Caused by: NoSuchFunctionException: Function: …. could not be found
I think this exception-message is not correct. Maybe an Message like “function did return NULL” would be more helpful.
—
Reply to this email directly or view it on GitHubhttps://github.com/christophd/citrus/issues/84.
—
Reply to this email directly or view it on GitHubhttps://github.com/christophd/citrus/issues/84#issuecomment-196340195.

OK, I could also reproduce the issue. Yes you are right, the exception is misleading.
Kind regards
Tamer
From: krguse<EMAIL_ADDRESS>Sent: Monday, March 14, 2016 15:47
To: christophd/citrus<EMAIL_ADDRESS>Cc: vdsrd<EMAIL_ADDRESS>Subject: Re: [citrus] Custom-Function, unclear Exception-Message (#84)
Hi Tamer,
thanks for your quick response.
And yes I am sure that everything is correctly configured.
Changing my return-value from
return null;
to
return “”;
let the exception disappear and everything is running fine.
The exception “NoSuchFunctionException” sounds a little bit like reflection. I assume that the returned value is analyzed and because null is not a valid string-instance (not the correct class) the analyzer is “confused” .
Please try to reproduce it.
Kind Regards
Kai
Von: vdsrd<EMAIL_ADDRESS>Gesendet: Montag, 14. März 2016 3:39
An: christophd/citrus
Cc: Guse, Kai-Rasmus
Betreff: Re: [citrus] Custom-Function, unclear Exception-Message (#84)
Are you sure that, your custom function is really executed. First check your configuration and look in citrus-context.xml if the custom function is configured correctly there, and if yes put a break point in your custom function and check if it is really executed.
Best regards
Tamer
From: krguse<EMAIL_ADDRESS>Sent: Monday, March 14, 2016 15:33
To: christophd/citrus <citrus@noreply.github.com<mailto:citrus@noreply.github.commailto:citrus<EMAIL_ADDRESS>Subject: [citrus] Custom-Function, unclear Exception-Message (#84)
My custom-function returned in function execute(..) null instead of a string-instance.
Got following exception-message
FAILURE: Caused by: NoSuchFunctionException: Function: …. could not be found
I think this exception-message is not correct. Maybe an Message like “function did return NULL” would be more helpful.
—
Reply to this email directly or view it on GitHubhttps://github.com/christophd/citrus/issues/84.
—
Reply to this email directly or view it on GitHubhttps://github.com/christophd/citrus/issues/84#issuecomment-196340195.
—
Reply to this email directly or view it on GitHubhttps://github.com/christophd/citrus/issues/84#issuecomment-196345270.

Hi Tamer,
Thanks ;-)
Kind Regards
Kai
Von: vdsrd<EMAIL_ADDRESS>Gesendet: Montag, 14. März 2016 3:57
An: christophd/citrus
Cc: Guse, Kai-Rasmus
Betreff: Re: [citrus] Custom-Function, unclear Exception-Message (#84)
OK, I could also reproduce the issue. Yes you are right, the exception is misleading.
Kind regards
Tamer
From: krguse<EMAIL_ADDRESS>Sent: Monday, March 14, 2016 15:47
To: christophd/citrus <citrus<EMAIL_ADDRESS>Cc: vdsrd <vdsrd<EMAIL_ADDRESS>Subject: Re: [citrus] Custom-Function, unclear Exception-Message (#84)
Hi Tamer,
thanks for your quick response.
And yes I am sure that everything is correctly configured.
Changing my return-value from
return null;
to
return “”;
let the exception disappear and everything is running fine.
The exception “NoSuchFunctionException” sounds a little bit like reflection. I assume that the returned value is analyzed and because null is not a valid string-instance (not the correct class) the analyzer is “confused” .
Please try to reproduce it.
Kind Regards
Kai
Von: vdsrd<EMAIL_ADDRESS>Gesendet: Montag, 14. März 2016 3:39
An: christophd/citrus
Cc: Guse, Kai-Rasmus
Betreff: Re: [citrus] Custom-Function, unclear Exception-Message (#84)
Are you sure that, your custom function is really executed. First check your configuration and look in citrus-context.xml if the custom function is configured correctly there, and if yes put a break point in your custom function and check if it is really executed.
Best regards
Tamer
From: krguse<EMAIL_ADDRESS>Sent: Monday, March 14, 2016 15:33
To: christophd/citrus <citrus@noreply.github.com<mailto:citrus@noreply.github.com<mailto:citrus@noreply.github.com%3cmailto:citrus@noreply.github.commailto:citrus@noreply.github.com<mailto:citrus@noreply.github.com<mailto:citrus<EMAIL_ADDRESS>Subject: [citrus] Custom-Function, unclear Exception-Message (#84)
My custom-function returned in function execute(..) null instead of a string-instance.
Got following exception-message
FAILURE: Caused by: NoSuchFunctionException: Function: …. could not be found
I think this exception-message is not correct. Maybe an Message like “function did return NULL” would be more helpful.
—
Reply to this email directly or view it on GitHubhttps://github.com/christophd/citrus/issues/84.
—
Reply to this email directly or view it on GitHubhttps://github.com/christophd/citrus/issues/84#issuecomment-196340195.
—
Reply to this email directly or view it on GitHubhttps://github.com/christophd/citrus/issues/84#issuecomment-196345270.
—
Reply to this email directly or view it on GitHubhttps://github.com/christophd/citrus/issues/84#issuecomment-196351059.


When timer is up, API call for previous round's voting is made and user votes are saved in order to move to next round

https://github.com/sopra-fs23-group-15/envisage-server/commit/4c3a4d460d679edb82adbad93b992f1513df2a08


add data from 2 cells

Hi friend, good afternoon, I'm trying to amount data from 2 cells but it tells me that it is not defined
{ Header: "Aciertos",  accessor: "point", id: "point",   Cell: ({ state, column, props, value }) => (value)},

While possible, this is an implementation question and should be asked in the #react-table Slack channel (badge at the top of the doco).
In any case, the mix of parameters in the Cell function also contain the visible row and the original row (row, original) - that way you can access other data in the row.

I add correctly, however there are fields that do not exist because the user does not believe it, as it could be valid if it exists to add it otherwise it omit it, use conditional but it does not workTypeError: row.questForo is undefined

@gary-menzel hello you cant help me please ?

Not here - please join the #react-table channel on Slack. The github issues aren't for general help. Thanks.


Request to host package - google_maps

This issue was originally filed by @a14n

Please can you host my package "google_maps" on pub.dartlang.org
You have to use "pub" branch :
git clone https://github.com/a14n/dart-google-maps.git -b pub
Thanks

This issue has been moved to dart-lang/pub#237.


Wrong event triggered when push notification is opened on iOS

I am trying to handle the event when a user taps on the push notification in his device.
To do this, I am using the OnNotificationOpened event.
In an Android device it works as expected.
However, in an iOS device, in the same situation, this event is not triggered: the OnNotificationReceived is triggered instead.
Bug Information
Version Number of Plugin: 3.0.3
Device Tested On: iPhone 6s iOS 13.3
Simulator Tested On: iPhone 11 iOS 13.3
Version of VS: Visual Studio for Mac 2019 v8.4
Version of Xamarin: Xamarin.Forms v<IP_ADDRESS>1477
Versions of other things you are using:
Steps to reproduce the Behavior

Send a push notification to an iOS device from firebase
Open the notification on the iOS device

Expected Behavior
In the code, I expect the "OnNotificationOpened" event to be triggered (this is the behaviour when I run the same code on Android).
Actual Behavior
The "OnNotificationReceived" event is triggered (this event should be triggered when the notification is received on the device, not when it is opened).
Code snippet
  CrossFirebasePushNotification.Current.OnNotificationReceived += (s, p) =>
  {
    System.Diagnostics.Debug.WriteLine("Received");
    // On iOS this event is triggered when the notification is opened
    if (Device.RuntimePlatform == Device.iOS)
    {
      HandleOpenedNotification(p.Data);
    }
  };

  CrossFirebasePushNotification.Current.OnNotificationOpened += (s, p) =>
  {
    System.Diagnostics.Debug.WriteLine("Opened");
    HandleOpenedNotification(p.Data);
  };

Screenshots

same here

I am also facing the exact issue. Any workaround?


Make 'clear' button work as expected

I just pushed something up that might fix this.

Yep, looks like it. Unless we want it to clear all fields.

Maybe Reset is a better description?

Changed label to reset.


why Sampler doesn't support request params

i want all request
if (request.getHeader("uid").equals("123")) {
    send to zipkin
}else {
    use PercentageBasedSampler
}

Sampler should According to the user's request.

You can. Just create sth similar to https://github.com/spring-cloud/spring-cloud-sleuth/blob/master/spring-cloud-sleuth-core/src/main/java/org/springframework/cloud/sleuth/instrument/web/SleuthHttpSampler.java#L30

@marcingrzejszczak
version 2.0+ feture?

Yes. Please read the documentation before submitting questions - https://cloud.spring.io/spring-cloud-static/Finchley.SR1/single/spring-cloud.html#_customizations

i use  Edgware.SR1 and spring-cloud-sleuth 1.3.1.RELEASE, Do you have any suggestions?

I suggest you upgrade to FInchley.SR1. If that's not possible, I suggest that you upgrade to Edgware.SR4 and try to work this around by applying a filter before the TraceFilter that registers what's there in the request, puts the request contents to ThreadLocal, then create your own Sampler that will retrieve that value, and basing on its contents perform sampling.
We migrated to Brave for a reason so my suggestion is to upgrade to Finchley.


feat(vim): Initial ':vimcompatible' setting implementation

Issue: For users coming from Vim, Onivim's set of defaults are not optimal and there are a bunch of features that may not be desirable. It'd be better to be able to have a quick way to set vim-like defaults, and then opt-in to additional functionality if desired.
Fix: Introduce a vimcompatible setting - this defaults to off, but if :set vimcompatible is run, it will default to using layout-tabs, positioning them on top, and turning off some features like smooth scroll and minimap.
This is much like how Vim exposes the compatible setting for vi-like defaults.

It'll be a work-in-progress to pick the right set of defaults - for example, we might also want to hide the sidebar with vimcompatible. Open to feedback and ideas!

In the spirit of #1551 I would prefer, if Onivim behaves more like Vim by default.
I don’t mind the minimap and the smooth scrolling. Those are actually nice additions, which don’t hurt the Vim experience, but especially the buffer / window / tab management is a core Vim features, which should work right out of the box. This is also something, which other non-Vim based editors can’t deliver and which Onivim should use as its advantage.

We could consider going down the path we looked at with Oni1, where the option is used to gate a notification.
I.e. if its not set, we pop up and ask so that a user who first launches would be asked. Then once you answer and its set to yes/no somewhere, we can just leave you to it, and its easy to change or get the prompt back if you really wanted it for some reason. That makes the default less meaningful since the user is instantly prompted.
There is a balancing act there, the first launch of the app we don't want to set the wrong impression and give the user a bunch of config options, part of the reasoning behind Oni2 is to help avoid some of the awkward/confusing options that Vim has for new users.
We do also want to set the right expectations as the default we choose will most likely be the one most people use I expect, so the expectations that the default sets are important. Some users will expect Code with Vim and some will expect Vim with Code, with different assumptions around that.
This is probably around the right time for that conversation, with vim plugins or config nearer in the pipeline, and depending how we want to deal with them we could end up with different implementations (Auto load config? Prompt about config? Separate config? etcetc), and that could help the decision here as well.
There is also the fact Oni2, whilst only in Alpha and change should be expected by everyone, is in use by people etc, and turning something like this on by default does change the editor in a big way between updates. Not that means we shouldn't do it, but we should consider the options!
I will say, and the above comment I think agrees with this, that some way of specifying "this is what vim compatible means to me" would be nice. So maybe we default it to [tabs, smoothscroll, minimap] but a user who doesn't mind the smooth scroll can remove it and slowly add stuff at their own pace. This does also add a new config option after I've said we don't want a million of them so...take from that what you will.

Thanks @mabuchner and @CrossR for the feedback, much appreciated!
Ya, one of the most challenging aspects is figuring the right balance (and which side to optimize) - we're in the precarious position of being caught in the middle of Code and Vim. This is a screenshot of a slide I'm getting ready for Vim conf...

But I think it's very valid. We essentially have two options:

Start with Vim-style defaults (tab layout), and make it easy to switch to Code style defaults if someone prefers. Could even be a toggle on the Welcome screen. I do like the idea of vim-style defaults for tab layout, but keeping smooth scroll, minimap, etc.

Start with Code-style defaults (buffer tabs), and make it relatively easy to switch Vim-style defaults (the idea behind the :set vimcompatible).

We have users coming in from both directions. But I think it's fair, given the overwhelming upvotes for #1551 to consider option 1.
I'll do the following:

I'll run a twitter poll and see what default is preferred, and whatever the result is, I'll update the defaults. I'll do this early next week - I'm a bit squeezed for time ATM. Agree that the vim-style tabs are a unique feature, and it's cool to showcase those.
I'll hold off on merging this until we get the results of the poll. Maybe can do away with this, if we just go with option 1.

I.e. if its not set, we pop up and ask so that a user who first launches would be asked. Then once you answer and its set to yes/no somewhere, we can just leave you to it, and its easy to change or get the prompt back if you really wanted it for some reason. That makes the default less meaningful since the user is instantly prompted.

Yep, I like this idea - in either option 1 or option 2, we can make it easy to toggle on the welcome screen.
Let me know if you have other ideas / feedback.

Personal preference would definitely for me would be a vim default, but I don't think that's the right one, even if you may have more users voting for it (initially at least).  I think users that have never experienced vim would find the vim defaults very confusing, whereas it's unlikely a vim user would find a vscode/standard editor default confusing. Together with an obvious way to switch to a vim default on the welcome screen (as you suggested) is probably the route that would provide the best overall user experience. (Probably a good thing to test and see though)

As a developer I would clearly not add a vimcompatible option (for now) and completely take out the non-layout tab solution. Having to support both layout strategies is a maintenance burden and makes future improvements (e.g. session management) more difficult.
I don't see a problem with an editor called "Onivim 2" behaving like Vim and I would see Onivim 2 much closer to Vim, than to VS Code in the Editor Spectrum.
Even, if Onivim 2 behaves more like VS Code by default, naive VS Code user will get annoyed as soon as they try to edit a file. That's why I would try to keep it simple and not consider those users for now.

I also think, the better approach to onboard VS Code users is to give them some help on how to use a Vim-like editor (spacemacs/spacevim/which-key) and to make some Vim features accessible via a GUI.

I always thought for vscode style users they would start with (and permanetly be in) a pseudo insert mode, with something like esc disabled so they just stay in that mode, like a normal editor.
Although a lot of us love the vim part, there us probably a lot of non-vim style users who would just like a really fast vscode (particularly the start up time which enables use-cases that vscode just can't).

I always thought for vscode style users they would start with (and permanetly be in) a pseudo insert mode, with something like esc disabled so they just stay in that mode, like a normal editor.

That's a valid point and I haven't thought about that. If it's actually doable, then it voids my "naive VSCode users will get annoyed as soon as they try to edit a file" argument.


Toast message for favorites

Added Toast message for favorite button #25
fixex #25
Description
used react-hot-toast for toast notifications to notify book is added to favorites
Changes in
installed react-hot-toast - package.json
imports and notify function for notifications - Book.js

@Arshad-ashuu there is an update in UI of the page. Please update your branch

Ok I'll check that

Added Toast message for favorite in new UI  #25
fixex #25
Description
used react-hot-toast for toast notifications to notify book is added to favorites
Changes in
installed react-hot-toast - package.json
imports and notify function for notifications - Book.js


Update calendarauto-function-dax.md

Change contiguous to continuous

@EnzoPavez - please let me know if you're ready to move this to publishing. Thanks for the contribution!

@jeroenterheerdt - can you please validate this change? I will move to publishing with your approval

@kfollis the original is fine by me. contiguous is the word that describes the requirements, the same as the continuous by the way. I see no need to make changes.

#please-close

[like]  Kate Follis reacted to your message:

From: Jeroen ter Heerdt @.>
Sent: Thursday, August 8, 2024 7:56:24 PM
To: MicrosoftDocs/query-docs @.>
Cc: Kate Follis @.>; Mention @.>
Subject: Re: [MicrosoftDocs/query-docs] Update calendarauto-function-dax.md (PR #225)
@kfollishttps://github.com/kfollis the original is fine by me. contiguous is the word that describes the requirements, the same as the continuous by the way. I see no need to make changes.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/query-docs/pull/225#issuecomment-2276550989, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AG3PD4OSKIPXYEULOAYPR7LZQPEORAVCNFSM6AAAAABMG7DRRKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZWGU2TAOJYHE.
You are receiving this because you were mentioned.Message ID: @.***>


DNM: test removal of CR update

What this PR does / why we need it:
This is just a test for now. Do not merge.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note:

NONE

/retest

/retest

/retest

/retest

@jean-edouard  Anything I can help with?

@jean-edouard Anything I can help with?

If this PR is of interest I could revive it.
Upgrading KubeVirt using the CR won't work if the target RBAC has permissions that the source doesn't, which is true in most cases, so the feature doesn't make much sense IMO.
This PR was to figure out how many tests would break without the feature, and the answer seems to be none, since the e2e lanes passed...

@jean-edouard Right, I think there are more caveats with updating just the images (Cr) without the operator and quit frankly I am convinced it complicates code. I would like to write down all the problems and present it to community for feedback. How did you stumble upon this?

How did you stumble upon this?

Funny story, the big functional test for this feature is broken (I don't remember how exactly unfortunately), but after fixing it it stopped passing! That's when I investigated and found this.


[admin] Publish

Docs Build status updates of commit 984939e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/outlook/autolaunch.md
:white_check_mark:Succeeded
View

docs/testing/debug-desktop-using-edge-chromium.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Feature/qx private sku

这个 PR 实现什么功能/修复什么问题:
私有云套餐纳管
是否需要 backport 到之前的 release 分支:
None

Codecov Report

Merging #1363 into master will decrease coverage by <.01%.
The diff coverage is 0.64%.

@@            Coverage Diff            @@
##           master   #1363      +/-   ##
=========================================
- Coverage    7.73%   7.73%   -0.01%
=========================================
  Files         374     374
  Lines       58866   58869       +3
=========================================
  Hits         4555    4555
- Misses      53909   53912       +3
  Partials      402     402

Flag
Coverage Δ

#aFlag
7.73% <0.64%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/util/aws/region.go
0% <ø> (ø)
:arrow_up:

pkg/compute/models/skustools.go
2.63% <0%> (-0.05%)
:arrow_down:

pkg/compute/models/cloudsync.go
0% <0%> (ø)
:arrow_up:

pkg/util/esxi/fakeregion.go
0% <0%> (ø)
:arrow_up:

pkg/compute/models/cloudproviders.go
1.26% <0%> (-0.02%)
:arrow_down:

pkg/compute/models/skus.go
1.5% <0.72%> (+0.01%)
:arrow_up:

pkg/util/seclib2/seclib.go
91.48% <0%> (-4.26%)
:arrow_down:

pkg/appsrv/response.go
69.23% <0%> (+3.07%)
:arrow_up:

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 7e45338...990eda0. Read the comment docs.


Evaluation Annotations (analyze_results)

Is it possible (preferably without modifications to the code) to generate evaluation results (test set) where the predictions are written to a coco style annotation file instead (how I currently understand it and got it working) annotating the images and storing the images?
If yes, how do I achieve this?

You only need to specify --format-only in tools/test.py

Great! Thank you. This seems like the option I was looking for.

How does one read/load the results.pkl that tools/test.py creates?

I have found the relevant documentation I think. One needs to provide, in addition to --format-only a jsonfile_prefix through the eval-options.
Full command I used:
mim test mmdet results/train/config.py \
    --checkpoint results/train/best_model.pth \
    --work-dir results/test \
    --format-only \
    --eval-options jsonfile_prefix=results/test/annotations

This results in two files being created in the results/test folder. annotations.bbox.json and annotations.segm.json. The bbox json file seems fine. However, the segmentation file looks strange to me, especially the "counts" field. I attached an example here:
  {
      "image_id": 75,
      "bbox": [
          13.571070671081543,
          20.626428604125977,
<PHONE_NUMBER>281372,
<PHONE_NUMBER>974365
      ],
      "score": 0.9921395182609558,
      "category_id": 1,
      "segmentation": {
          "size": [
              960,
              1280
          ],
          "counts": "j\\a0`0P1H8O^:M]E:R81bG;J;D7MBk0Ad:c1kLh4XHkIi:i1dKa5[IlHn:Q2oJb5lIbHR;h2lIW6Q6XJRIZ6k6TJZH\\6b7T8L5K4L4L3M3M3M3N2N2N2N1O2O0O101O0000000O10000000000O100000001O0000000000000000000000000000000000000000000000000000000000O1000000000000000000000000000000000000000010N10O1000000000000000000000000000000000000000000000000000000000000001O00O100000000000000000000000000000000000000000000000001O00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000O100001O000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000001O000000000000000000000000000000000000000000000O100000001O00000000000000000000001O0O2O001O001N101O1O0O2S^OWFVOLn>n9]AQGkNJ\\?]9UAfIe=i6^AkI]>Q;K4K3N1N2N5K8H9G7G6I7I8G8H:F:G6K6I7K5PFR]Oe8Wc0lFU]On8Td0F:[Of0YOg0\\HYZOW6Ug0^Oc0\\Od0ZKYWOU3`j0QMhUO2XZog0"
      }
  },

Is this normal?


Support instrumenting unary grpc

Issue #, if available:
#110
Description of changes:
Continue @sawadashota's effort (#264) to implement grpc support.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Also it could be great reference to look through the OpenTelemetry instrumentation for good patterns re: naming, etc.
https://github.com/open-telemetry/opentelemetry-go-contrib/blob/main/instrumentation/google.golang.org/grpc/otelgrpc/interceptor.go
They don't seem to have separate packages for unarystream / clientserver so I'm thinking it's relatively scalable.

Ah I don't think we need to be redundant with e.g. ServerInterceptor. Just UnaryServerInterceptor(WithContext(ctx), WithSegmentNamer(...)) can be done I think, there's no difference between server / client versions of these options right? If we can simplify these names I think we're good to go

Merged Server/ClientInterceptorOption -> GrpcOption.
I also add a refactor that changes the API a little bit:
Changed the option WithContext -> withRecorder because the option was used to provide a custom xray.Config to the server handling context. We only propagate the xray.Config value of the given context. We're not propagating other values associated with the context. This should make the option's intent more clear. (Also, it seems to be a code smell to store a context inside a struct)


Roll Flutter Engine from 33faf88f6ecc to c481a7aded75 (1 revision)

https://github.com/flutter/engine/compare/33faf88f6ecc...c481a7aded75
2022-09-09<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from nHBOdXRdrHSKPhszb... to 64AKvws9D_pywDW-f... (flutter/engine#36028)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from nHBOdXRdrHSK to 64AKvws9D_py
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Services Pages: Our toolbox

For the toolbox on the design and Develop page, bring the outside images closer to the center. ideally, they should be within the margins that are used for the nav bar.

images are now placed correctly, closing ticket


UnexpectedElasticsearchClientException: The request was canceled

NEST/Elasticsearch.Net version: 6.0.0
Elasticsearch version: 6.1.3
Description of the problem including expected versus actual behavior:
When using NEST from a System.Threading.Tasks.Dataflow pipeline to pull records from a database, convert them to a type marked with the ElasticsearchType attribute, and (synchronously) index them in bulk, we are seeing an UnexpectedElasticsearchClientException wrapping a System.Net.WebException with message: "The request was aborted: The request was canceled." Full trace:
Type: Elasticsearch.Net.UnexpectedElasticsearchClientException
Message: The request was aborted: The request was canceled.
Source: Elasticsearch.Net
TargetSite: TResponse Request[TResponse](Elasticsearch.Net.HttpMethod, System.String, Elasticsearch.Net.PostData, Elasticsearch.Net.IRequestParameters)
DataflowMessageValue: Document[]
StackTrace:    at Elasticsearch.Net.Transport`1.Request[TResponse](HttpMethod method, String path, PostData data, IRequestParameters requestParameters)
   at Elasticsearch.Net.ElasticLowLevelClient.Bulk[TResponse](PostData body, BulkRequestParameters requestParameters)
   at Nest.LowLevelDispatch.BulkDispatch[TResponse](IRequest`1 p, SerializableData`1 body)
   at Nest.ElasticClient.Nest.IHighLevelToLowLevelDispatcher.Dispatch[TRequest,TQueryString,TResponse](TRequest request, Func`3 responseGenerator, Func`3 dispatch)
   at ElasticDocumentFacade`1.BulkWrite(IBulkDocumentWriteRequest`1 request) in <our build server>
   at IndexService.<>c__DisplayClass5_0.<Build>b__0(IEnumerable`1 batch) in <our build server>
   at System.Threading.Tasks.Dataflow.ActionBlock`1.ProcessMessage(Action`1 action, KeyValuePair`2 messageWithId)
   at System.Threading.Tasks.Dataflow.ActionBlock`1.<>c__DisplayClass5.<.ctor>b__0(KeyValuePair`2 messageWithId)
   at System.Threading.Tasks.Dataflow.Internal.TargetCore`1.ProcessMessagesLoopCore()
Type: System.Net.WebException
Message: The request was aborted: The request was canceled.
Source: System
TargetSite: Int32 Read(Byte[], Int32, Int32)
StackTrace:    at System.Net.ConnectStream.Read(Byte[] buffer, Int32 offset, Int32 size)
   at System.IO.StreamReader.ReadBuffer(Char[] userBuffer, Int32 userOffset, Int32 desiredChars, Boolean& readToUserBuffer)
   at System.IO.StreamReader.Read(Char[] buffer, Int32 index, Int32 count)
   at Nest.Json.JsonTextReader.ReadData(Boolean append, Int32 charsRequired)
   at Nest.Json.JsonTextReader.ReadStringIntoBuffer(Char quote)
   at Nest.Json.JsonTextReader.ParseProperty()
   at Nest.Json.Serialization.JsonSerializerInternalReader.PopulateObject(Object newObject, JsonReader reader, JsonObjectContract contract, JsonProperty member, String id)
   at Nest.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Nest.Json.Serialization.JsonSerializerInternalReader.SetPropertyValue(JsonProperty property, JsonConverter propertyConverter, JsonContainerContract containerContract, JsonProperty containerProperty, JsonReader reader, Object target)
   at Nest.Json.Serialization.JsonSerializerInternalReader.PopulateObject(Object newObject, JsonReader reader, JsonObjectContract contract, JsonProperty member, String id)
   at Nest.Json.Serialization.JsonSerializerInternalReader.Populate(JsonReader reader, Object target)
   at Nest.BulkResponseItemJsonConverter.ReadJson(JsonReader reader, Type objectType, Object existingValue, JsonSerializer serializer)
   at Nest.Json.Serialization.JsonSerializerInternalReader.DeserializeConvertable(JsonConverter converter, JsonReader reader, Type objectType, Object existingValue)
   at Nest.Json.Serialization.JsonSerializerInternalReader.PopulateList(IList list, JsonReader reader, JsonArrayContract contract, JsonProperty containerProperty, String id)
   at Nest.Json.Serialization.JsonSerializerInternalReader.CreateList(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, Object existingValue, String id)
   at Nest.Json.Serialization.JsonSerializerInternalReader.SetPropertyValue(JsonProperty property, JsonConverter propertyConverter, JsonContainerContract containerContract, JsonProperty containerProperty, JsonReader reader, Object target)
   at Nest.Json.Serialization.JsonSerializerInternalReader.PopulateObject(Object newObject, JsonReader reader, JsonObjectContract contract, JsonProperty member, String id)
   at Nest.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Nest.Json.Serialization.JsonSerializerInternalReader.Deserialize(JsonReader reader, Type objectType, Boolean checkAdditionalContent)
   at Nest.Json.JsonSerializer.DeserializeInternal(JsonReader reader, Type objectType)
   at Nest.InternalSerializer.Deserialize(Type type, Stream stream)
   at Nest.InternalSerializer.Deserialize[T](Stream stream)
   at Elasticsearch.Net.ResponseBuilder.SetBody[TResponse](ApiCallDetails details, RequestData requestData, Stream responseStream, String mimeType)
   at Elasticsearch.Net.ResponseBuilder.ToResponse[TResponse](RequestData requestData, Exception ex, Nullable`1 statusCode, IEnumerable`1 warnings, Stream responseStream, String mimeType)
   at Elasticsearch.Net.HttpConnection.Request[TResponse](RequestData requestData)
   at Elasticsearch.Net.RequestPipeline.CallElasticsearch[TResponse](RequestData requestData)
   at Elasticsearch.Net.Transport`1.Request[TResponse](HttpMethod method, String path, PostData data, IRequestParameters requestParameters)

We are using an IElasticClient registered with our DI container as a singleton, with no custom ConnectionSettings save for the node Uri. This is occurring in a development environment with a single ES node running on the same machine. Oddly, it only seems to happen intermittently, and only after the pipeline runs for a while (>5-10 minutes, usually >5 million documents).
Steps to reproduce:
Run bulk index operations long enough...? Happy to provide more details to help with reproducing this if you need.

Hi @w-bonelli are you running on .NET core? Are you using Windows/Linux?

Thanks @Mpdreamz for your response. We are on Windows, .NET framework 4.5.1.

Thanks that opens up the possibility to diagnose with network tracing:
e.g as described in this blogpost: http://mikehadlow.blogspot.nl/2012/07/tracing-systemnet-to-debug-http-clients.html
Informational might yield too much though lets set it to Error only for now: https://docs.microsoft.com/en-us/dotnet/framework/network-programming/how-to-configure-network-tracing
Before doing that though , with what kind of concurrency are you hitting your local elasticsearch instance. If you start seeing requests taking longer and longer you might be overloading your single node with too many requests at once.
Another point of interest would be the output of:
GET /_nodes/stats/http
After an import run.
You'll have to restart ES before doing the import though to reset the counters.

We've tried the Dataflow block calling IElasticClient.Bulk() with max concurrency between 2-8.  GET /_nodes/stats/http : current_open never tops 9 (thinking the 9th could be Kibana?). The ES node is configured with 16gb heap size, with request times climbing a bit at first then leveling off. ES logs look ok too, so I don't see an indication of overloading. I should also note we've tried bulk sizes between 1,000-100,000 and continue to see this.
If I can get to it later today I will try network tracing. Thanks for the suggestion.

Hey @w-bonelli i'd also be interested in total_opened from the http stats. This should also be a low number if the only thing happening on the cluster is the bulk import (given a freshly started node).

@Mpdreamz Sorry for the silence. total_opened increases at what seems like a constant rate as bulk indexing keeps running. E.g. single digits per minute. Should it be staying constant instead? Yes, bulk import is the only thing happening.
I'm hoping to reproduce the error with network tracing later today and will update if/when I can.

Hey @Mpdreamz I managed to reproduce this. Network tracing gives Exception in HttpWebRequest#<>:: - The operation has timed out. Immediately prior to seeing the exception we saw response times jump from ~0.5 seconds to 20-40 seconds. It seems like overloading as you suggested.

It does appear so. Are you seeing any 427's from elasticsearch?

What does 427 indicate? Not seeing any of those in ES logs or in the network trace. Do you mean HTTP 429's?

I fixed it with Elasticsearch config elasticsearch.yml
https://www.elastic.co/guide/en/elasticsearch/reference/current/modules-http.html
http.max_header_size: 1mb
http.max_initial_line_length: 1mb

http.max_content_length: 500mb

@w-bonelli closing this due to its age. I dropped the ball on this completely as the ticket flew off my radar for which I apologize.
Be keen to hear back on whether you were able to stabilize your bulk import!


Always check res.ok and content-type header

Description
I have noticed that I forgot some response assertions in #1516.
I also noticed that it makes more sense to always check the content-type header if we're going to call res.json().
Therefore, if we know the response might include error details in JSON, we first check the header. If we know this isn't the case, we first check if the response was ok before we check the header. If we would always check first if the response is ok, we might miss error details.
Additional Context
https://stacker.news/items/773703
Checklist
Are your changes backwards compatible? Please answer below:
yes
On a scale of 1-10 how well and how have you QA'd this change and any features it might affect? Please answer below:
7. I tested successful withdrawals with:

LNbits
NWC
~CLN~ local certificate needs an update

For frontend changes: Tested on mobile, light and dark mode? Please answer below:
n/a
Did you introduce any new environment variables? If so, call them out explicitly here:
no

You should be able to test CLN without a cert. I think I made that change to test against polar.

You should be able to test CLN without a cert. I think I made that change to test against polar.

Ah ok, it works with the onion but only with the cert, else I get
ERROR	failed to create test invoice: request to http://hmsykvji73qkrzxnjxmnzas3uvync7sbnubrmijclr3tu46r3ve2nuyd.onion:3010/v1/invoice failed, reason: socket hang up


The type of flake.apps.<name>.<name>.program makes it impossible to refer to a single-file derivation output

This type will match strings that are store paths as types.package, it therefore tries to run getExe on it, which fails.
It feels like it should be possible to fix this but keep the wanted behavior by replacing lib.types.package with a type that only checks isDerivation and not isStorePath.
https://github.com/NixOS/nixpkgs/blob/f1dd761ae8e02f8182832a73bee1c9d508c308bd/lib/types.nix#L380
https://github.com/NixOS/nixpkgs/blob/f1dd761ae8e02f8182832a73bee1c9d508c308bd/lib/attrsets.nix#L427
https://github.com/NixOS/nixpkgs/blob/f1dd761ae8e02f8182832a73bee1c9d508c308bd/lib/strings.nix#L740-L746

I couldn't reproduce the problem. Here's my attempt:
$ cd flake-parts
$ nix repl
nix-repl> let self = builtins.getFlake ("git+file://" + toString ./.); in (self.lib.mkFlake { inherit self; } { systems = ["x86_64-linux"]; perSystem = { pkgs, ... }: { apps.default.program = "${pkgs.hello}/bin/how-low"; }; }).apps.x86_64-linux.default
{ program = "/nix/store/4fy9jnx6s5n0j9yyg38ldv2z8i9cmrix-hello-2.12/bin/how-low"; type = "app"; }

Do you have an example where it fails?

That was a bit more subtle than I expected.
Thank you for the fix!