Add proper return codes to /v2.1/{tenant}/servers/{server} GET Method…

… API Call
The function showServerDetails attending calls from /v2.1/{tenant}/servers/{server}
GET Method was returning 500 InternalServerError for all error cases, this
fix covers Unauthorized, NotFound and InternalServer return errors for each
corresponding case. Also returning a more meaningful error message.
Fixes #158
Signed-off-by: Leoswaldo Macias<EMAIL_ADDRESS>

Coverage decreased (-0.03%) to 61.832% when pulling 830833a92f8e945880574d70c4793b661a7863cc on leoswaldo:leoswaldo/topic/returncodes into 9c47545cb6b6466e53ac9724efc8f1273209282e on 01org:master.

The commit message needs to be updated.

The first line should start with the name of the component affected by the commit, e.g., ciao-controller:
The lines are too long.  They should wrap at 72 characters.

The code change looks okay, although I wonder whether it would be nicer to use the json package to generate the json for you rather than embedding json docs as strings in the controller code.  Might be safer to do this going forward, if for example, a future error message was generated from one or more variables.
Of course the problem with using the json package is MarshalIndent returns an error so you'd need to deal with that.  The easiest way to do this would be to pre-create all the error strings on controller startup in an init function, panicking if you can an error from MarshallIndent.  I'm not requesting that you make this change however.  This is more a call for the controller maintainers.  I'm just raising the issue.

Coverage decreased (-0.1%) to 61.751% when pulling 27acade90eeeba4a4cdb4fc50e0510de6225c349 on leoswaldo:leoswaldo/topic/returncodes into 073ce234c4f59ec4faa7a37945069e1b627a3ab1 on 01org:master.

Coverage decreased (-0.09%) to 61.771% when pulling 27acade90eeeba4a4cdb4fc50e0510de6225c349 on leoswaldo:leoswaldo/topic/returncodes into 073ce234c4f59ec4faa7a37945069e1b627a3ab1 on 01org:master.

Coverage decreased (-0.07%) to 61.785% when pulling fb19cb2c7094b1a49c29645fb867b7e876751d13 on leoswaldo:leoswaldo/topic/returncodes into 073ce234c4f59ec4faa7a37945069e1b627a3ab1 on 01org:master.

Coverage decreased (-0.05%) to 61.804% when pulling 5d5c06581526e5b895109a89212d367f8c822a2d on leoswaldo:leoswaldo/topic/returncodes into 073ce234c4f59ec4faa7a37945069e1b627a3ab1 on 01org:master.

It looks good to me now.
@tpepper , @markdryan: This one is ready to merge as far as I'm concerned.

Coverage decreased (-0.08%) to 62.659% when pulling d0ee2004bb14c651a560d08eb2658cc167096e97 on leoswaldo:leoswaldo/topic/returncodes into b5958b7120f84686c6020572143b52fba4bcbcd4 on 01org:master.

Coverage decreased (-0.07%) to 62.669% when pulling be2d08f01350cdeb140a5b8083e551c4a1b41002 on leoswaldo:leoswaldo/topic/returncodes into b5958b7120f84686c6020572143b52fba4bcbcd4 on 01org:master.

Looks good to me.


[BUG] does not download

Describe the bug
does not download
Steps to Reproduce
1 went to the launcher
2 selected the game to download
3 clicked add to library
4 start downloading
Expected behavior
start downloading
Additional information and data

Operating System
windows 10
Hydra Version
v3.0.5 "Skyscraper"
Before opening this Issue

[X] I have searched the issues of this repository and believe that this is not a duplicate.
[X] I am aware that Hydra team does not offer any support or help regarding the downloaded games.
[X] I have read the Frequently Asked Questions (FAQ).

Is there any chance you are dowloading your game in a directory that needs admin permission for write?

Is there any chance you are dowloading your game in a directory that needs admin permission for write?

I didn't understand your question, but how do I download the game as an administrator?

I don't download as administrador. You make sure that the folder does not need admin permissions. The default Windows Downloads folder should work as expected, for example.
Not saying that this is the cause of your issue for sure. But it is a possibility

I checked, the folder where the game is downloaded does not need administrator rights.

It's been a couple of days but I still haven't been able to solve the problem :(


Swagger: Fix resolve endpoint example and add verify endpoint

Closed via https://github.com/iov-one/iovns/pull/363.


how to make this momentjs driven persian

http://www.daterangepicker.com/#examples
i wnat to make this script persian for my project how to do this i have repalced all functions as your document but not worked

It's not that easy. And not an issue with moment-jalaali. I refer you to my implementation on a utility library used to implement persian support for material-ui-pickers


Add a web-clustering test

An example that uses web-clustering to share session content between 2 pods.
Scenario is:

1 replica
Access the servlet session ID and session content, store sessionID cookie
Scale to 2
Kill the first replica
Access again the endpoint with the stored cookie
Check that content is identical.

@rhusar Please take a look


Mfa modal

added new mfa modal ( Fixes #191 ) to allow profiles such as the following to work with sls dev tools

[profile personal]
region = eu-west-1

[profile personal2]
role_arn = arn:aws:iam::XXXXX:role/admin-role
mfa_serial = arn:aws:iam::XXXX:mfa/admin
source_profile = personal

mfa modal is triggered by the AWS SDK upon detecting that the profile being used references a mfa device

Carried out following testing:

Tested with standard profile which uses aws access key/secret key

verified the mfa modal is not presented and the tool loads as expected

Tested with a profile which references a mfa_serial device

verified if a correct mfa code is entered the tool loads as expected
if an incorrect or timed out mfa code is entered at the moment a console.log is printed to screen (was not sure best how to handle this error - should the tool exit?)

Only area I didn't test was that the region/stackName modals still worked but all I changed is how/when they are triggered not their functionality.

@jnmullen Really nice work James! Thanks again for your contribution!


Weekly New Parts order

I'd like to have a chance to invert the order of the "Weekly New Parts" chart. As I'm more interested in the oldest parts I'd like to see them at the top without the need to scroll all the way down.

Implemented


Add duplicate action package

Name of an extension you are adding
Duplicate Action
Why do you think this extension is awesome?
Being able to duplicate files and folders is super useful.

Thanks!
For more context: https://github.com/Microsoft/vscode/issues/9345


Faucet doesn't work locally

Describe the bug

Clicking the faucet creates errors in the console.
To Reproduce

sign in
click faucet button
see browser console

Expected behavior

faucet works locally / is hidden while on localhost

Screenshots

CC @sabau

This is expected behavior. The faucet doesn't work locally but the button should show so we design around it.

disabled then, maybe?

hahahah ok, disabling is fine for me

disable + tooltip

there is no button anymore.


how to chose the output file frame rate manually

how can I chose the output frame rate manually like to 60fps or 120fps etc instead of the default one >>> 2x frame, I tried -n=4 for example and it didn't work

you need to count the input frames. if you have 200 frames at 30 fps then to get 120 fps you need 200*4 aka -n 800

you need to count the input frames. if you have 200 frames at 30 fps then to get 120 fps you need 200*4 aka -n 800

thanks a lot for your help it worked, I was just hoping there was an easy solution like specify the output directly "frame-rate=60" for example, is there any more detailed reference for using the program?


custom reduction cuda kernel only processing half elements of the input array

Description
I implemented a simple sum reduction cuda kernel on a 1-d array, but if I call the kernel from the cupy.RawKernel, it returns only half of thre correct result. It seems that only the first half of the input elements are taken into consideration.
I think the cuda kernel is correct, I tested it on plain c++. The colab    site of a plain c++ implementation can be found here.
To Reproduce
cuda_kernel = """\
extern "C" __global__
void ShareMemorySumReduction(int* in, int* out) {
  const unsigned int BLOCK_SIZE = 4;
  __shared__ float in_s[BLOCK_SIZE];
  unsigned int t = threadIdx.x;
  in_s[t] = in[t] + in[t+BLOCK_SIZE];
  for (unsigned int stride=blockDim.x/2; stride>=1; stride/=2) {
    __syncthreads();
    if (threadIdx.x < stride) {
      in_s[threadIdx.x] += in_s[threadIdx.x+stride];
    }
  }

  if (threadIdx.x == 0) {
    *out = in_s[0];
  }
}"""

ShareMemorySumReduction = cp.RawKernel(cuda_kernel, "ShareMemorySumReduction")

x = np.random.randint(0, 10, size=(8))
out = cp.zeros(1, dtype=cp.int32)
block_width = x.shape[0]//2

ShareMemorySumReduction((1,), (block_width,),
           (cp.array(x), out))

print(f'input x: {x}')
print(f'output from the cuda kernel is {out[0]}')
print(f'the correc result is {sum(x)}')

Installation
Source (pip install cupy)
Environment
OS                           : Linux-6.1.85+-x86_64-with-glibc2.35
Python Version               : 3.10.12
CuPy Version                 : 12.2.0
CuPy Platform                : NVIDIA CUDA
NumPy Version                : 1.25.2
SciPy Version                : 1.11.4
Cython Build Version         : 0.29.36
Cython Runtime Version       : 3.0.10
CUDA Root                    : /usr/local/cuda
nvcc PATH                    : /usr/local/cuda/bin/nvcc
CUDA Build Version           : 12020
CUDA Driver Version          : 12020
CUDA Runtime Version         : 12020
cuBLAS Version               : (available)
cuFFT Version                : 11008
cuRAND Version               : 10303
cuSOLVER Version             : (11, 5, 2)
cuSPARSE Version             : (available)
NVRTC Version                : (12, 2)
Thrust Version               : 200101
CUB Build Version            : 200101
Jitify Build Version         : <unknown>
cuDNN Build Version          : 8801
cuDNN Version                : 8906
NCCL Build Version           : 21602
NCCL Runtime Version         : 21903
cuTENSOR Version             : None
cuSPARSELt Build Version     : None
Device 0 Name                : Tesla T4
Device 0 Compute Capability  : 75
Device 0 PCI Bus ID          : 0000:00:04.0

Additional Information
No response

x = np.random.randint(0, 10, size=(8))

This generates ndarray of int64 (not int/int32), which does not match with int* in.

x = np.random.randint(0, 10, size=(8))

This generates ndarray of int64 (not int/int32), which does not match with int* in.

Thank you. Exactly as you said, I set it to data type int32, then it works. .astype(np.int32)


Fix link in SUPPORT.md and adding vscode and rider files to gitignore

Fix the link in SUPPORT.md and add vscode and rider files to gitignore file

Updated branch to current code


authenticator: 4.1.6 -> 4.2.0

Description of changes
Authenticator is currently broken, see https://github.com/NixOS/nixpkgs/issues/216770. The fix is to upgrade gtk-rs to >= 0.5.3.
I can't find an easy way to make Authenticator 4.1.6 work with gtk-rs >= 0.5.3. They've released Authenticator 4.2.0, however it relies on the development version of GTK, >= 4.9. That's why I haven't updated Authenticator in nixpkgs, I don't imagine a nixpkgs-wide bump to a development version of GTK4 would be welcomed. So instead I'm overriding GTK4 just for Authenticator.
I'm not sure what the policy is for overrides like this.
4.1.6 -> 4.2.0 changes
Fixes https://github.com/NixOS/nixpkgs/issues/216770.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

1 package built:

authenticator

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/853

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

1 package failed to build:

authenticator

Hmm some change in master since the last build makes it so it can't find the gtk4 from the override.


Test Only: MP systems Modal improperly handles editing and creating data

Steps to Reproduce
Issue 1: Cannot create a systems record after fixing an error

Log into ECMPS
Navigate to MP and check out a config
Select systems as a section
Select create system
Fill out all of the fields in the modal and leave start time blank
Select save
The system prompts the user to enter a value for start time
Enter in a value for start time and select save

The system does not save the changes and continues to display an error message despite the user making updates

The issue has been addressed. Able to create a Systems record successfully:


feat: [M3-8592] - Limits: surface new API errors in Linode and LKE flows

Description 📝
The first milestone of the limits project is mostly a backend one. New errors are being returned by the API when a customer attempts an operation that would "overdraft" their allowed quota. In this case they have to open a support ticket to request additional resources. While we handle the display of API errors already, we have a fresh <ErrorMessage /> component that parses error messages to potentially turn the "contact Support" string into a link to open a ticket with prefilled data depending on the case.
This PR ensures all limit messages for the following endpoints are being surfaced properly, with the support link:

Changes  🔄

Surface support link with <ErrorMessage /> for all needed flows (except upgrade/mutate which is a special complicated case not addressed here)
Improve UI with scrollErrorIntoViewV2 as needed
Add limit for linode and lke to MSW presets

Target release date 🗓
9/30/2024
Preview 📷

Error
Support Ticket

How to test 🧪
Linodes

turn MSW on and select the "Limits > Linode Limits" preset
very error message and support ticket handling for the following actions:

create linode
rebuild linode
resize linode
migrate linode
clone linode

LKE

turn MSW on and select the "Limits > LKE Limits" preset
very error message and support ticket handling for the following actions:

create LKE cluster
add pool
resize pool

As an Author I have considered 🤔
Check all that apply

[x] 👀 Doing a self review
[x] ❔ Our contribution guidelines
[x] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[ ] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[x] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

@mjac0bs i had no clue! sounds like a plan to me, i just assume it had been done this way

@mjac0bs confirming the general form is more appropriate considering the wide array of cases those errors cover

@coliu-akamai good catch! hadn't touched that file but I did now just update to scrollErrorIntoViewV2 from the v1 version and scrolling behavior is consistent now 👍

Confirming e2e failures are flakes


uploading an artifact

Description
This PR adds uploading the contents of .tortellini (excluding .tortellini/in) to an artifact, such that users may look into the generated files.
Refs: #79

[x] I read and I followed contributing guidelines
[x] ☝ Create an issue to discuss what you are going to do

Checklist
This section should be filled in by the creator of this pull request to make sure the pull request is ready to review.

[x] This pull request has a descriptive title
[x] Code is written according to the code quality guidelines
[x] Documentation if applicable is available
[x] Tests run successfully
[x] All checks below this pull request were successful

Instructions to review the pull request
Running the tortellini workflow should result in an artifact being present after the workflow finishes, see e.g. https://github.com/tortellini-tools/action/actions/runs/903430405

Thank you for the review!


acs-engine scale up/down issue with loadbalancer OF SKU: STANDARD

Is this a request for help?:
yes
Is this an ISSUE or FEATURE REQUEST? (choose one):
Issue
What version of acs-engine?:
v0.26.0

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
Kuberenetes 12.1
What happened:
Here's the

Creating a new kubernetes cluster with acs-engine version 0.26.0 with 1 master node (availability set) 2 nodes agent pool (VM scale set).
Create first (nginx controller) service of kind loadbalancer (using helm), the 2 nodes joined the loadbalancer backend pool successfully
Trying to scale up to 3 nodes using acs-engine scale command but getting this error:

FATA[0819] Code="DeploymentFailed" Message="At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-debug for usage details." Details=[{"code":"Conflict","message":"{\r\n  \"status\": \"Failed\",\r\n  \"error\": {\r\n    \"code\": \"ResourceDeploymentFailure\",\r\n    \"message\": \"The resource operation completed with terminal provisioning state 'Failed'.\",\r\n    \"details\": [\r\n      {\r\n        \"code\": \"VMExtensionProvisioningError\",\r\n        \"message\": \"VM has reported a failure when processing extension 'vmssCSE'. Error message: \\\"Enable failed: failed to execute command: command terminated with exit status=50\\n[stdout]\\n\\n[stderr]\\n\\\".\"\r\n      }\r\n    ]\r\n  }\r\n}"}] .
exit status 50 based on the document means: outbound traffic failure.
the new node is created successfully but without kubernetes installed and it didn't join  the loadbalancer backend pool.
Without loadbalancer scaling up and down works perfectly !
What you expected to happen:
Scaling up/down should be succeeded in the existence of the loadbalancer.
How to reproduce it (as minimally and precisely as possible):
As described above
Anything else we need to know:

Just to mention this issue happens only with Loadbalancer of SKU standard:
"loadBalancerSku": "standard"

I have tried this with ACS 0.24.2 and Kubernetes 1.12.1 and did not run into this problem.

I am facing the same issue with aks-engine 0.32.x and kubernetes 1.11.8. Cluster autoscaler v1.3.9
Also using "loadBalancerSku": "standard" and "singlePlacementGroup": false
This is the event I see in a kubernetes service that expects a load balancer to be created:

Warning  CreatingLoadBalancerFailed  4m41s                  service-controller  Error creating load balancer (will retry): failed to ensure load balancer for service mynamespace/traefik: Long running operation terminated with status 'Failed': Code="VMExtensionProvisioningError" Message="VM has reported a failure when processing extension 'vmssCSE'. Error message: "Enable failed: failed to execute command: command terminated with exit status=50\n[stdout]\n\n[stderr]\n"."

I think the issue should be reopened.


How to convert model to TRT

Hello again.
as you mentioned in the another issue, i tried to convert your model to TRT with torch2trt.
here is a part of my code:
dummy_input_to_model_init_encoder     = torch.ones((1, 3, 128, 128)).cuda()
dummy_input_to_model_update_search    = torch.ones((1, 3, 256, 256)).cuda()
model_encoder_update = torch2trt(model, [dummy_input_to_model_update_search, dummy_input_to_model_init_encoder])
after execution i got the following error:
TypeError: forward() takes 2 positional arguments but 3 were given.
how to fix it?

well i finally converted model to trt but gives me this error:
[TensorRT] ERROR: 4: (Unnamed Layer* 325) [Convolution]: count of 2304 weights in kernel, but kernel dimensions (3,3) with 256 input channels, 256 output channels and 1 groups were specified. Expected Weights count is 256 * 3*3 * 256 / 1 = 589824

also after converting model with torch2trt, model's accuracy drops heavily.
why??!!

Hi

[TensorRT] ERROR: 4: (Unnamed Layer* 325) [Convolution]: count of 2304 weights in kernel, but kernel dimensions (3,3) with 256 input channels, 256 output channels and 1 groups were specified. Expected Weights count is 256 * 3*3 * 256 / 1 = 589824

Model runs correctly in PyTorch, so any errors in other runtimes come from the wrong conversion between runtimes. Make sure you didn't make errors in the model while converting it. I haven't tried converting FEAR to TensorRT using torch2trt. Try this TensorRT compiler for PyTorch, I converted FEAR to TensorRT using it a while ago.

also after converting model with torch2trt, model's accuracy drops heavily

It means your model is converted incorrectly. TensorRT should have the same accuracy if you convert all layers properly

How to convert model to ONNX?

@t-martyniuk   When can you release the code?


Train new model

Hii .. I try to train a new Model for my 3D tissue Cellcount, but when I do ctrl+T or go to“Models” menu in the File bar at the top and click “Train new model…” nothing happend. I just have one channel and so the nuclei channel is the first channel and not the CHAN2 is that a problem?
Best,
Luisa

sorry you cannot train a model on 3D images, you need to take 2D slices of the volume and train on those. then you can run your model with the 3D version of cellpose


Cannot access Ghost via host IP after setting url ENV variable

After setting my docker-compose.yml to include
environment:
  - url=https://sub.domain.url

The site then works at the URL but no longer is able to be accessed using the local IP. Is there a way to allow me to access Ghost from both a local IP and a URL?

Hi,
Just to manage your expectation you are in the docker project.
Take a look at: https://forum.ghost.org/
The short answer is yes. You can have a DEV and a PROD config :)

I did not have this issue running outside of a docker container, but that was a long time ago.
Is this an appropriate place to ask how to have a dev/prod config inside my docker container? I'm assuming it's essentially two containers that share a database?
thanks for prompt response!

Again, please use Ghost's forum. It's have been asked many time there :)

Not finding anything relative really in their forums, especially since my issue only affects the Docker Container version. If I won't get help here about the Ghost Docker Container then I'll look elsewhere and close the issue.

Humm: https://forum.ghost.org/search?q=configuration file


Support build with Tensorflow

These commits fix building of Tesseract with Tensorflow.
By default, configure will check automatically whether one of the Tensorflow header files which are needed by the Tesseract code exists. If yes, the build will use Tensorflow.
The configure options --with-tensorflow or --without-tensorflow can be used to explicitly require or refuse Tensorflow.
On Debian, installation of the packages libtensorflow-dev and libprotoc-dev is required. The include path is currently hardcoded for those to /usr/include/tensorflow.

It looks like LSTMTrainer::InitTensorFlowNetwork is currently unused, so training with TensorFlow needs more code. Text recognition with existing traineddata files with type NT_TENSORFLOW might work.

Text recognition with existing traineddata files with type NT_TENSORFLOW might work.

Are there any traineddata files of this type?

Are there any traineddata files of this type?

I don't know. Maybe @theraysmith, @jbreiden or @DanBloomberg know whether there exist TensorFlow models for Tesseract.
I know that other OCR software like for example Calamari uses TensorFlow and hope that the models from that software can be converted to be used by Tesseract.

thanks!

This is not tf but keras, it is not the whole thing, but should help you to do the stuff.
It is kind of messy, including some constants in there and needs some refactoring, but nevertheless i think this is similar what tesseract does.
(at least thats what I think from looking at VGSL specs -same architecture but some different hyperparams-)
(this is for image_height=36)
making data generation and training completely two separate processes would be really good for people, allowing them to introduce their custom data.
`def customreshape(inputs):
inner = inputs[0]
shape = inputs[1]
import tensorflow as tf2
reshaped = tf2.reshape(inner, [shape[0], shape[1], 54] )
return reshaped
def tfshape(inputs):
import tensorflow as tf2
return tf2.shape(inputs)
def tfreshape(inputs):
import tensorflow as tf2
return tf2.reshape(inputs, [-1,18,16])
def build_model(self):
input_shape = (self.img_h, None, 1)
input_data = Input(name='inputs', shape=input_shape, dtype='float32')
    inner = Conv2D(self.conv_filters, self.kernel_size, strides=(1,1), padding='same', activation='relu', kernel_initializer='he_normal', name='convlayer')(input_data)
    inner = MaxPooling2D(pool_size=self.pool_size)(inner)

    inner = Permute((2, 1, 3)) (inner)

    shape = Lambda(lambda x: tfshape(x), output_shape=(4,))(inner)
    inner = Lambda(lambda x: tfreshape(x), output_shape=(18, 16))(inner)

    inner, h_, c_ = CuDNNLSTM(units=54, input_shape=(None, 16), return_sequences=False, return_state=True)(inner)

    inner = Lambda(lambda x : customreshape(x), output_shape=(None, 54))([inner, shape])

    inner = CuDNNLSTM(units=60, return_sequences=True, return_state=False, input_shape=(None, 54))(inner)

    inner = Bidirectional(CuDNNLSTM(units= self.rnn_size, input_shape=(None, 60), return_sequences=True))(inner)

    inner = Dense(self.alphabet_size +1 , kernel_initializer='he_normal')(inner)
    y_pred = Activation('softmax', name='softmax')(inner)
    Model(inputs=input_data, outputs=y_pred)

    labels = Input(name='the_labels', shape=[self.maxImageWidth // self.pool_size[1]], dtype='float32')
    input_length = Input(name='input_length', shape=[1], dtype='int64')
    label_length = Input(name='label_length', shape=[1], dtype='int64')

    loss_out = Lambda(ctc_lambda_func, output_shape=(1,), name='ctc')([y_pred, labels, input_length, label_length])
	..
	..

m = OCRModel(mode ='train', img_h=IMG_H, conv_filters=16,
kernel_size=(3,3), pool_size= (2,2), rnn_size = 128, alphabetDict=dataProvider.alphabetDict,
absolute_max_string_len = dataProvider.maxYLen, maxImageWidth = MAXIMAGEWIDTH)
m.build_model()`


Publish to PyPI

To make it easier to install, we should write a setup.py and publish to pyPI for pip installation.

We need to do #5 before we can do this.

#5 is done so we can do this now :)


The content returned by torch.nn.Sequential.append is inconsistent with the official

According to the documentation, append returns a Sequential. But TorchSharp is void
https://pytorch.org/docs/stable/generated/torch.nn.Sequential.html#torch.nn.Sequential.append

Another question
Why does this place use IList instead of ICollection, the package that my project depends on will output Array
Now, I need toList, and then TorchSharp toArray(). Feels like there's a lot of performance loss in data copying
https://github.com/dotnet/TorchSharp/blob/c55e9f535aa46c4dfa104d839019553b693616c0/src/TorchSharp/Tensor/Tensor.Factories.cs#L140

According to the documentation, append returns a Sequential. But TorchSharp is void

Thanks for pointing that out. It will be fixed in the next version.

Why does this place use IList instead of ICollection, the package that my project depends on will output Array

ICollection seems wrong as the abstract interface here, since it does not imply order, which IList does.

I have a different idea, ICollection has nothing to do with the order, the developer decides which collection to use will affect the order.
Then why not use T[], but use IList.
I think that for machine learning frameworks, performance is the most important, and other limitations should be increased without affecting performance. My input is 500M data, and it will waste a lot of time in processing ToList() and ToArray, and at the same time increase the pressure of GC (Note: ArrayPool can be used in some places)

Ideally there should be an overload taking ReadOnlySpan<T>

If you're starting with an array, I would recommend using tensor instead of as_tensor() -- the array overloads are already available there.

But I can't insert array.

@ChengYen-Tang I can reproduce that with this:
            float[] a = new float[12];
            var t = torch.as_tensor(a);

I'm troubleshooting, but in the meantime, you can replace it with 'frombuffer' or cast the array to IList in the call (for now).
            float[] a = new float[12];
            var t = torch.as_tensor((IList<float>)a);

The reason there's an ambiguity is that there's an implicit operator that converts an array to a Tensor. You should be able to pass the array to the 'forward' directly, unless you need to reshape it or change the element type or device.
For example:
            float[] a = new float[12];
            var x = ReLU();
            var t = x.forward(a);

That still leaves the ambiguity in place, which I should find a way to address. I'll leave this issue open to track.

You can also use torch.tensor() if you want to make sure that a copy is being made. as_tensor() will only do that when necessary because of the dtype or device arguments.

@ChengYen-Tang I can reproduce that with this:
            float[] a = new float[12];
            var t = torch.as_tensor(a);

I'm troubleshooting, but in the meantime, you can replace it with 'frombuffer' or cast the array to IList in the call (for now).
            float[] a = new float[12];
            var t = torch.as_tensor((IList<float>)a);

I don't know, what happened.
this can work

Can anyone tell me why ?😂

This project is a library

存在歧義的原因是有一個隱式運算符將陣列轉換為張量。您應該能夠直接將數位傳遞給“轉發”，除非您需要重塑它或更改元素類型或設備。
例如：
            float[] a = new float[12];
            var x = ReLU();
            var t = x.forward(a);

Since I'm duplicating a python project, I want to preserve features as much as possible

@ChengYen-Tang -- what happened is that you changed the static type from float[] to Array, for which there is a separate overload of 'as_tensor()'

存在歧義的原因是有一個隱式運算符將陣列轉換為張量。您應該能夠直接將數位傳遞給“轉發”，除非您需要重塑它或更改元素類型或設備。

Unfortunately, I am not going to be able to respond to any commentary or questions unless they're in English. Sorry.

Ok, it looks like T[] and Array are not object oriented inheritance.

I'm going to add overloads for 1-D arrays, which are the ones that are affected by the ambiguity. N-D arrays (N > 1) will be caught by the already existing 'Array' overload.

I just released 0.99.1 to NuGet. The array overloads are in that release.


[WIP] Initial implementation

This is WIP of an initial implementation of the CORS handling library for aiohttp.
Implementation uses WIP version of Signals implemented by  @alexsdutton in this PR.
Currently all CORS handling logic is implemented, but tested only superficially.
While more complete tests and documentation will be committed later, right now I interested in feedback about API.
README.rst has Usage section with examples of the current version of the API.
This is a part of https://github.com/KeepSafe/aiohttp/issues/516 fix.

Very exciting PR.
Looks pretty good.
I suggest merging as-is.
Let's discuss next changes after merge.
Say, I doubt using types in annotations make sense.
They are neither checked in runtime as I see nor pep 484 compatible.

Let me fix starting of tests on Travis/Appveyor, and I will merge this PR.

Ok, at least on Travis tests are being run (though not passing yet) - merging.
There is a lot of work to be done, but it's mostly infrastructure: tests, documentation, etc.
I will open appropriate pull requests.

Thanks


Add Pubsub lite write metrics and DLQ support

Added error metrics and dlq routing support for pubsub lite write schema transform. Tested the DoFn functionality with unit test.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

r: @manavgarg, @Dippatel98, @bvolpato

Spotless seems to be failing, can you fix that. LGTM otherwise

r: @johnjcasey

Run Java_GCP_IO_Direct PreCommit

SpannerReadIT.testReadWithDataBoost failing unrelated. merging for now


Angular-Strap Conflict

I am using anuglar-strap and conflict with $modal.
Can you specify the specific directive to use for Dashboard from ui-bootstrap.

The adf uses $modal and collapse.


Autocomplete goodies

[x] Expand file name before trying to execute, Fixes #315
[ ] (#336)
[x] (#305)
[x] (#303)

Hej,
Is this likely to fix issues like we see here [1]?
I.e. parsing errors "leaking" out of the completer and blocking you from doing anything except ^C and start over again?
Note in this particular case we have a -f and a -d <arg> flag both being short aliases.
I understand that -fd is not a correct argument, but in this case i'd expect arguments to -d to be completed here instead. suggesto -f=d in this case is just incorrect i'd say.
[1] https://github.com/flox/flox/issues/1657

I understand that -fd is not a correct argument, but in this case i'd expect arguments to -d to be completed here instead.

Even if I could start completing arguments for -d here - it won't work once you press enter and try to run the whole thing. The goal is to either move to completing the next item -fd<TAB> -> -fd _ or to break them down into separate items: -fd<TAB> -> -f -d _.
zsh with its default completion scripts support doing something smart with adjacent single letter flags, bash does not.
So yeah, I'm planning to fix this problem, but I'm not sure yet what the best solution is. Dealing with a newborn is not helping :)

Well, congratulations :)
I'm fine with it not completing if the result wouldn't be valid anyway..
Mainly hoping to get this to work without it messing up the terminal at least ^^
Is this PR the right place to start if i were to look at it myself?

Well, congratulations :)

:heart:

Mainly hoping to get this to work without it messing up the terminal at least ^^

Should be here, but it's a mess... Might be easier to try to address that issue directly, gimme a sec.


Added basic unittests.

tests/tests_feed.py Does work on my main account, most likely not working on my alt testing account due to a ratelimit.

This PR was supposed to be to development, not main.


update to cache method providers

Fixes https://github.com/Azure/autorest.csharp/issues/4288

API change check
APIView has identified API level changes in this PR and created following API reviews.
Microsoft.Generator.CSharp


Support searching in project files and swift packages.

First of all this is an awesome plugin.
Second: Is it possible to search for files/text inside the sub swift packages? It would be great to have a seamless integrated code navigation in neovim.
I wonder if there's any peculiarity with local packages and remote ones inside derived data.
I have no experience with making neovim plugins, so I'm unable to help. But I believe this feature would be great for people using this plug in.

Hi and thank you!
Could you provide more detail on what do you mean by "sub swift packages"? What's your project structure? Do you have xcodeproj and you add SPM packages as dependencies? Are they local or remote?
I've got the project modularized with SPM and it's possible to navigate between packages. I'm also able to navigate to external package - but it depends on how it's created, some packages may include xcframework instead of source code

Hi @wojciech-kulik sorry for the long time to respond. Currently it is possible to navigate to packages inside the project if you go to definition or find references.
What I mean was to have something more similar to what we have in Xcode when we search for something In Project and Package Dependencies. This way when we look for code we're looking at the whole picture and  not just the project scope. The packages are in the derived data stuff, so maybe adding their folders to the fuzzy finder would solve the problem.

LSP features are implemented in sourcekit-lsp, the plugin itself doesn't do anything in this area. I don't think there is any option to modify that behavior.
If you are just looking for a better text search, you could probably configure Telescope to search in a specific directory instead of the whole cwd.

That would work. But I think it would be a good experience if xcodebuild could already link the folders added as dependencies, either being in the derived data (remote packages) or custom directories(local packages).
Or maybe xcodebuild could also have some way to list these directories for easier navigation, similar to what xcode does for us to see the code in the file tree explorer.
That's just an idea though. Thanks for hearing me out.

That would be quite a challenging change. To make it usable it would require reinventing a file tree plugin from scratch.
Why do you need it? If you have to check an implementation, you can quickly jump to it using LSP. If you heavily browse those packages you can always navigate to the root of all packages and open another Neovim instance.
 Maybe you could try playing with symlinks/hard links/plugin config to make packages appear in your file tree. Basically, all packages that are used are in: ~/Library/Developer/Xcode/DerivedData/<project>/SourcePackages/checkouts.
At this point, I don't see how we could easily integrate it unless you've got some ideas.


Sticky top-level comments.

Moderatable.distinguish now accepts key-word argument sticky.  If object is a top-level Comment, the sticky parameter is appended to the json request.  Also added a small test to ensure basic functionality works (however, I'm not sure I'm using betamax correctly).

Travis job failed; as far as I understand because I forgot to commit a "test_distinguish_and_sticky.json" cassette. If I recall correctly, I just copied another cassette used for commenting and renamed it.
How do I proceed? Add the file and submit another pull-request?

The PR looks great, and you've gotten the betamax tests all figured out so nice work. Sorry if I got too caught up in the parameter thing.
There's only one minor change I'd recommend before we merge, and that would be replacing the following:
self.assertEqual(comment.stickied, True)
self.assertEqual(comment.stickied, False)

with:
self.assertTrue(comment.stickied)
self.assertFalse(comment.stickied)

just to make the best use of the tools provided by the unittest framework. I was going to suggest replacing self.assertEqual(comment.distinguished, None) with self.assertIsNone(comment.distinguished) but there are several other tests using assertEqual for that so maybe it's a compatibility thing. Don't worry about it for now.
Thanks!

assertIsNone wasn't added until python 2.7. I believe we removed python 2.6 support so using assertIsNone is fine: https://docs.python.org/2/library/unittest.html#unittest.TestCase.assertIsNot

With no other comments, it looks like we're ready to merge. Thanks for tying this up!


actions: add spack

Add actions for Spack to install and run a software stack. The actions will be placed in https://github.com/popperized/spack

@ivotron Any assistance required from my side regarding this action? I would like to pick it up, I saw there is some work already done, can you tell me what else is needed here?

thanks @arshul! Yes, there are a couple of things that need to be done in order to finish with this action:

move the contents of actions/spack/* to the root. This repo contains only one single action, so it can be directly on the root.
add a README that documents the usage and environment variables
move the test workflow to a ci/ subfolder

Sure, Please add me to contributors.


[stable/traefik] Set TestFramework Toggle to Default to False

What this PR does / why we need it:
Sets the testFramework default to false so that effort is only required to turn it on.
Special notes for your reviewer:
PR based on discussion here #18783 with @dtomcej
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @dtomcej

/ok-to-test

/lgtm

Thanks! @dtomcej


[Python] Better pytest parametrization for different compression codecs

Follow-up for https://github.com/apache/arrow/pull/10717
Certain codecs are optional, we should have global pytest fixtures automatically applying the right pytest markers.
Reporter: Krisztian Szucs / @kszucs
Note: This issue was originally created as ARROW-13380. Please see the migration documentation for further details.

take


'go get' in module mode is deprecated

╰─ make check
Error: No such object: k8s-guide-control-plane
Error: No such network: kind
Error: No such object: k8s-guide-control-plane
Error: No such network: kind
make[1]: *** [.mk/kind.mk:22: kind-test] Error 1
Error: No such object: k8s-guide-control-plane
Error: No such network: kind
go get: installing executables with 'go get' in module mode is deprecated.
Use 'go install pkg@version' instead.
For more information, see https://golang.org/doc/go-get-install-deprecation
or run 'go help get' or 'go help install'.
Error: No such object: k8s-guide-control-plane
Error: No such network: kind
Error: No such object: k8s-guide-control-plane
Error: No such network: kind
Error: No such object: k8s-guide-control-plane
Error: No such network: kind
make[1]: *** [.mk/kind.mk:22: kind-test] Error 1
check failed. see logs above.

+1

+1

Kind is dependency. Please add it to prerequisites.
https://kind.sigs.k8s.io/docs/user/quick-start/


Remove firewalld actions on libvirt

On JeOS images firewall is disable. We need to make sure to only
support this images, otherwise firewall may also be enabled and
cloud-init is not flexible enough to do proper checks to see
if firewalld exists. So lets fully disable it and only recommend
using JeOS images.
Fixes SUSE/avant-garde#1347
Why is this PR needed?
Does it fix an issue? addresses a business case?
add a description and link to the issue if one exists.
Fixes #
Reminder: Add the "fixes bsc#XXXX" to the title of the commit so that it will
appear in the changelog.
What does this PR do?
please include a brief "management" technical overview (details are in the code)
Anything else a reviewer needs to know?
Special test cases, manual steps, links to resources or anything else that could be helpful to the reviewer.
Info for QA
This is info for QA so that they can validate this. This is mandatory if this PR fixes a bug.
If this is a new feature, a good description in "What does this PR do" may be enough.
Related info
Info that can be relevant for QA:

link to other PRs that should be merged together
link to packages that should be released together
upstream issues

Status BEFORE applying the patch
How can we reproduce the issue? How can we see this issue? Please provide the steps and the prove
this issue is not fixed.
Status AFTER applying the patch
How can we validate this issue is fixed? Please provide the steps and the prove this issue is fixed.
Docs
If docs need to be updated, please add a link to a PR to https://github.com/SUSE/doc-caasp.
At the time of creating the issue, this PR can be work in progress (set its title to [WIP]),
but the documentation needs to be finalized before the PR can be merged.
Merge restrictions
(Please do not edit this)
We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:
What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

Code is good, commit message isn't: We don't need to only recommend JeOS, as skuba now disables the firewalld anyway on node bootstrap/join

This is about the load balancer, which on CI/maybe smaller deployments can be a machine. That machine needs to be run on JeOS, otherwise it can have any sort of firewalld and that will block the deployment. Skuba does not run on the load balancer because...well, it should be a load balancer, not a machine :D

Oh yeah, this also fixes libvirt deployment as confirmed broken from @suseclee and @davidcassany and that removing the firewall stuff fixed it.
Somebody probably broke this....


[FFM-1622] Feature fetch operation error

There is a problem integrating SDK with ReduxToolkit because RTK use Immer and Immer is auto freezing state tree after produce method call.

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


Concatenate global and service specific extraMounts

The list of global and service specific extraMounts are now concatenated, instead of the previous behavior where service specific extraMounts would supersede/override the global list.
The services that support their own list of extraMounts is unchanged. Those services are cinder, glance, manila and neutron.

/test openstack-operator-build-deploy-kuttl

/test images

build fails with the following:
[ERROR - 2024-03-12T08:31:54Z]
  * could not run steps: step openstack-operator-ci-build-deploy failed: error occurred handling build openstack-operator-ci-build-deploy-amd64: build not successful after 5 attempts: [the build openstack-operator-ci-build-deploy-amd64 failed after 3m19s with reason PushImageToRegistryFailed: Failed to push the image to the registry., the build openstack-operator-ci-build-deploy-amd64 failed after 3m26s with reason PushImageToRegistryFailed: Failed to push the image to the registry., the build openstack-operator-ci-build-deploy-amd64 failed after 3m13s with reason PushImageToRegistryFailed: Failed to push the image to the registry., the build openstack-operator-ci-build-deploy-amd64 failed after 5m54s with reason PushImageToRegistryFailed: Failed to push the image to the registry., the build openstack-operator-ci-build-deploy-amd64 failed after 3m9s with reason PushImageToRegistryFailed: Failed to push the image to the registry.]
[INFO - 2024-03-12T08:31:54Z] Reporting job state 'failed' with reason 'executing_graph:step_failed:building_project_image'

/retest


Resolve HIP tester failures

The HIP-3.5 CI tests fail in what seems random manner. It is always ArborX_LinearBVH_Test test that may fail. This really slows down the pull request processing, and requires many retest this please. I think it's time to do one of the following:

actually figure out what's going on there
update to 3.7 to see if it helps
disable HIP testing

How frequent are these failures?  I consider "do nothing" to be a reasonable option.  I would object to disabling but I'm fine with the ROCm upgrade even though I don't see how this would help.

The HIP CI test is not required and we can merge pull requests without it.

Documenting HIP failures:
<snip>
/var/jenkins/workspace/ArborX_PR-369/test/tstLinearBVH.cpp(834): error: in "structured_grid<Kokkos__Device<Kokkos__Experimental__HIP_ Kokkos__Experimental__HIPSpace>>": check indices_host(i) == i has failed [1059 != 1056]
/var/jenkins/workspace/ArborX_PR-369/test/tstLinearBVH.cpp(834): error: in "structured_grid<Kokkos__Device<Kokkos__Experimental__HIP_ Kokkos__Experimental__HIPSpace>>": check indices_host(i) == i has failed [1060 != 1057]
/var/jenkins/workspace/ArborX_PR-369/test/tstLinearBVH.cpp(835): error: in "structured_grid<Kokkos__Device<Kokkos__Experimental__HIP_ Kokkos__Experimental__HIPSpace>>": check offset_host(i) == i has failed [1056 != 1057]
<snip>

Are we sure it's not a bug in Kokkos HIP implementation? If it is intermittent failure, and the failure is slightly off indices, does not that indicate that there is some kind of race condition (in parallel_scan from what it seems) that may produce wrong answer?

The HIP CI test is not required and we can merge pull requests without it.

@masterleinad While true, failing HIP test also bypasses the style checks, which are required.

What I meant is that we can currently merge pull requests even if CI is failing. Of course, bypassing the style checks is problematic then.
Locally the test is passing with Kokkos 3.2 and HIP 3.6. for me. Let's see if simply updating Kokkos helps.

The problem is not getting any better. Do we expect anything to change anytime soon, or do we just continue merging PRs with failing HIP in foreseeable future?

The problem is not getting any better. Do we expect anything to change anytime soon, or do we just continue merging PRs with failing HIP in foreseeable future?

 So it might be worth trying that. Unfortunately, I can not reproduce the issue manually on other machines so we don't have a good test case. Still, I think it's important to keep testing with HIP to see if more issues come up.

Closing with the merge of #392. Reopen as needed.


Note mode enters setting by deafult

When entering note mode, I have to press FN to close the setting ui, which doesn't need adjusting every time.
It would be nice to enter note directly without the setting ui, making it smoother.

This is indeed the correct behavior by design at the moment. I will do some survey and see if it need to be changed.

This will be changed in V2.5

Changed in 99fcae391545f5ede6a1f04d5e6402cc1f91b7c1


time-picker's now action timestamp(seconds) error when using minutes props

TuSimple/naive-ui version (版本)
2.34.4
Vue version (Vue 版本)
3.2.36
Browser and its version (浏览器及其版本)
Chrome(113.0.5672.126)
System and its version (系统及其版本)
MacOS(13.3.1)
Node version (Node 版本)
Reappearance link (重现链接)
https://codesandbox.io/s/time-picker-error-v4vxw4?file=/src/Demo.vue
Reappearance steps (重现步骤)
When use props minutes, Click now button will have seconds error
First click:
timestamp:<PHONE_NUMBER>106
LocaleTimeString: 上午10:30:08
Second click:
timestamp:<PHONE_NUMBER>106
LocaleTimeString: 上午10:30:17
Expected results (期望的结果)
Expected two clicks have a same return timestamp, and LocaleTimeString should be 上午10:30:00
Actual results (实际的结果)
Not reset seconds, even seconds props = 0，
If not reset seconds，after click now actions and select any time options will inaccurately
如果沒有重置秒數，則按下 now 按鈕後任何選擇時間的操作將失準
Remarks (补充说明)
Currently I am changing update-value event to reset seconds
const handleUpdateValue = (timestamp: number) => {
  const date = new Date(timestamp);
  date.setSeconds(0, 0);
  const updatedTimestamp = date.getTime();
  emits('update-value', updatedTimestamp);
};

You shouldn't set seconds to 0. I think you may need to set seconds to [0].


Pt/#146312989 typeahead dialog select

https://www.pivotaltracker.com/story/show/146312989
@serenamarie125 @Loicavenel @chriskacerguis
Heres the action shot! (shots?)

🌮 💃

For your review @ManageIQ/core-ux
/cc @Loicavenel

@jennyhaines can you take a look at this screenshot?  Should the dropdown be a bit closer to one another?

BZ - https://bugzilla.redhat.com/show_bug.cgi?id=1502316


Following a Previously Read Series After Re-installing Doesn't Correctly Set the Up-Next Part

The logic (PartDao.getLatestFinishedPart()) relies on both all parts' Progress and Part being loaded into the database.
While all Progress data is loaded in on sign-in, Part data is only loaded in on-demand.
Workaround: In the explorer view, navigate to the latest volume in the series with a read part before following it.

Fixing this would require the loading in of all available parts in a series (potentially a lot!) upon follow... :/


SPEC as backend solver

It has been suggested that a back-end SPEC solver for hklpy could be available.

The SPEC server mode would provide the communications.


Make status messaging uniform again

Matching with the kubes stack of charms where messages are fragments of
a sentence with full stop.

+1 for these messages.


chore: ARC pullPolicy and kubectl version

What happens when your PR merges?
Setting the github ARC runner to always pull latest image.
Explicitly setting the kubectl version in the installation.
Provide some background on the changes
Prerequisite for private eks

ingress	nginx    	2       	2024-02-12 19:08:42.93215444 +0000 UTC	deployed	nginx-ingress-1.1.2	3.4.2

github-arc-ss-staging	github-arc-controller	3       	2024-03-12 15:04:29.49139 -0400 -0400	deployed	gha-runner-scale-set-0.8.2	0.8.2

Comparing release=ingress, chart=charts/nginx-ingress
Comparing release=secrets-store-csi-driver, chart=secrets-store-csi-driver/secrets-store-csi-driver
Comparing release=aws-secrets-provider, chart=aws-secrets-manager/secrets-store-csi-driver-provider-aws
Comparing release=blazer, chart=stakater/application
Comparing release=github-arc, chart=charts/gha-runner-scale-set-controller
Comparing release=github-arc-ss-staging, chart=/tmp/helmfile1338758386/github-arc-controller/staging/github-arc-ss-staging/gha-runner-scale-set/0.8.2/gha-runner-scale-set

This shows no changes because I patched the staging deployment locally to make sure this worked. Didn't want to waste the PR.


Explain/PnL doesn’t include fees paid

Scope:
Must have:
Make it clear how it is presented to the user  includes/does not include fees
(show as default net of fees, then show with fees too) - is this something supplied by data-node?
Nice to have (subject to a new ticket based on effort and what data-node has):
take data / aggregate data (does this exist / data-node work)

I'd suggest showing this in a tooltip on the P&L column header, ideally with the ability to toggle fees:

My hunch is that it would be more useful to include fees by default if possible and then allow the user to exclude them.

I'd suggest showing this in a tooltip on the P&L column header, ideally with the ability to toggle fees:

My hunch is that it would be more useful to include fees by default if possible and then allow the user to exclude them.


standby Displays brightness

Hi,
could you modify these two standby displays for lower brightnesses? There is not possible to adjust it.
Thank you.
Tomas K.

Did it today. Now you can ajust it with the brightness knob in the overhead panel.


Only infos are logged

Testing #161786

Have an extension that logs some infos and some debugs.
Open the extensions log and see the infos ✅
"Developer: Set Log Level..." and choose "Debug"
Do something in the extension that causes the debug logs to happen.
Look at the log again and see that the debug logs are not there.

Error and trace also don't log.


CH02: Use train_test_split instead of StratifiedShuffleSplit

In CH02, the book uses StratifiedShuffleSplit to split data according to income category, maybe it's more user-friendly to use train_test_split
code from master:
from sklearn.model_selection import StratifiedShuffleSplit

split = StratifiedShuffleSplit(n_splits=1, test_size=0.2, random_state=42)
for train_index, test_index in split.split(housing, housing["income_cat"]):
    strat_train_set = housing.loc[train_index]
    strat_test_set = housing.loc[test_index]

updated version:
strat_train_set, strat_test_set = train_test_split(
    housing, test_size=0.2, random_state=42, stratify=housing["income_cat"])

Thanks for the awesome book!

Hi @qinhanmin2014 ,
Great suggestion, thanks a lot! Apparently, this parameter was added in Scikit-Learn 0.18, it wasn't there when I started writing the book, I wasn't aware of it.
As soon as I have a minute, I'll update the book and the notebooks!
Thanks again

Strongly agree with this. When I read the book, the StratifiedShuffledSplit code makes me confused. In my understanding, the stratified sampling already did by generate the income_cat, not sure about whether anything further did by StratifiedShuffledSplit. After use train_test_split instead, I think it's clear. Thanks!

Hello, I was following the code mentioned in the book but it isn't working and gives the following error:
TypeError                                 Traceback (most recent call last)
 in
2
3 split = StratifiedShuffleSplit(n_splits=1, test_size=0.2, random_state=42)
----> 4 for train_index, test_index in split.split(housing, housing["income_cat"]):
5     strat_train_set = housing.loc[train_index]
6     strat_test_set = housing.loc[test_index]
~\Anaconda3_2\lib\site-packages\sklearn\model_selection_split.py in split(self, X, y, groups)
1771         to an integer.
1772         """
-> 1773         y = check_array(y, ensure_2d=False, dtype=None)
1774         return super(StratifiedShuffleSplit, self).split(X, y, groups)
1775
~\Anaconda3_2\lib\site-packages\sklearn\utils\validation.py in check_array(array, accept_sparse, accept_large_sparse, dtype, order, copy, force_all_finite, ensure_2d, allow_nd, ensure_min_samples, ensure_min_features, warn_on_dtype, estimator)
478     # DataFrame), and store them. If not, store None.
479     dtypes_orig = None
--> 480     if hasattr(array, "dtypes") and len(array.dtypes):
481         dtypes_orig = np.array(array.dtypes)
482
TypeError: object of type 'CategoricalDtype' has no len()
I also tried the code mentioned here(without the stratified shuffle split) but it gave the same error. Please help me rectify this. Thank you so much!

@raool8 This is a bug in scikit-learn 0.20.0 and 0.20.1 (resolved in https://github.com/scikit-learn/scikit-learn/pull/12706), please update your scikit-learn.

thanks a lot


[all] F5 always reauthenticate

Bug fixed on version 2.X.X is back in 3.X.X
The correct version to take is
https://github.com/AxaGuilDEv/react-oidc/blob/65bc3a6e0375c1181401eadd8c049bb0710071a4/packages/redux/src/authenticate.js
I will fixe it on monday

In the same case : when token is expired the silent signin is not launched.

It is not the same case at all Olivier :p


Use “Keep A Changelog” format

As specified by http://keepachangelog.com (github repo: olivierlacan/keep-a-changelog).
This is about 90% of the way there; a few small edits remain, but first I want to make sure that even this level of change is okay with the repo owners.

First of all: Thank you for the effort you've added to this. 😊
Secondly, I honestly don't think these kinds of changelogs will have a long life here at ZEIT. I will do my best to push the org into the direction of simply using GitHub releases. They're much cleaner and just made for proper release notes.
But until then: Sure, let's merge this!

Secondly, I honestly don't think these kinds of changelogs will have a long life here at ZEIT. I will do my best to push the org into the direction of simply using GitHub releases. They're much cleaner and just made for proper release notes.

Cool.  I like GitHub Releases!
These changes are not mutually exclusive, though.  Grouping the “Added”, “Changed”, “Removed”, etc. lists can be useful in the markdown descriptions of each GitHub Release, too. :)


[BUG] Invalid version string on rr get-binary

@rustatian
I'm getting an error when trying to do rr get-binary.
The version string referenced is the latest pre-release in the repo.
I tried this code:
/var/www/vendor/bin/rr get-binary

I expected to see this happen: a normal download of the binary
Instead, this happened:
#25 1.324
#25 1.324  Environment:
#25 1.324    - Version:          2.*
#25 1.326    - Stability:        stable
#25 1.329    - Operating System: linux
#25 1.329    - Architecture:     amd64
#25 1.329
#25 1.993
#25 2.017 In VersionParser.php line 185:
#25 2.018
#25 2.018   Invalid version string "Plugins separate repo experiment"
#25 2.018
#25 2.018
#25 2.018 get-binary [-o|--os [OS]] [-a|--arch [ARCH]] [-f|--filter [FILTER]] [-l|--location [LOCATION]] [-s|--stability [STABILITY]]
#25 2.018

The version of RR used: 2.*
My .rr.yaml configuration is: can supply if needed

Hey @nil-nil , this is https://github.com/spiral/roadrunner-cli bug. Let me transfer the issue.


Monthly Call with IDSA Technology & Architecture Team

Hi all,
I am contacting you on behalf of @ssteinbuss, CTO of IDSA.
As Technology and Architecture team of IDSA, we would like to organize a monthly call (60 mins) with the maintainers of IDSA repositories to discuss about the work and rules (license, code of conduct, issue labels, how to contribute file) for each repo.
As maintainers of this repo, can you please provide us your email address? So that we can create a contact list and organize our kick-off meeting?
(you can either reply here or send a direct e-mail to me at<EMAIL_ADDRESS>Thank you,
Best regards,

This is not an issue and already addressed.


ui-router loop forever

reference issue:
https://github.com/beeman/loopback-angular-admin/issues/67

not sure if its the appropriate solution, but this worked out for me just fine.

Hi @silverbux, I've just pushed a different fix of the boot-loop problem to master as this did not seem to fix it.
Could you please check master to see if this PR is still needed? :)


Beamer headline not shown

On beamer slides, headline are not shown.

This seems to be linked with a special beamer template. With Warsaw, everything seems to work fine.

With Warsaw, the frametitle seems still

This issue was moved to DragonRock/slideshow.io-clean#1


ENH: PARSE_ARGS_INT_ONLY added to ParseArgsAndCallDoIt()

If an application supports only integer pixel types, it is
now possible to use ParseArgsAndCallDoIt() and the templated
function DoIt() to handle all integer types
(such as 'int', 'short',...) without supporting float types.

Looks good to me!


Add and use BenchmarkResults class

Closes #112 and #113 by adding a new BenchmarkResults dataclass.
Motivation:
As we modularize the conbench ecosystem, {benchmarks} will need to be able to output consistent json that can be picked up by another tool which will send it all to conbench. That json can be generated in a consistent fashion by calling dataclasses.asdict() on an instance of BenchmarkResults (or our own wrapper which adjusts structure, names, etc. a bit if we like).
Design choices:
For now, uses the builtin dataclasses.dataclass instead of pydantic in order to start minimal. We can escalate to pydantic later if we like, especially as we start to harden this schema.
Also I chose to put this code here for now instead of directly in {conbench}. We may want to move some of this to the .benchmark() and .record() methods there if we plan on keeping them there.
Notable changes:

Adds simple new BenchmarkResults dataclass and adjusts the runners to return an instance of it instead of result, output, where result was a json blob and output was a string of stdout.
Changes run methods to return new class
Adjusts a lot of asserts in tests to handle the new structure
Stops trying to send stuff to conbench (on localhost) when a DRY_RUN env var is set so running the tests is less of a mess
Removes a bunch of copy.deepcopy() calls that weren't doing anything

@ElenaHenderson @austin3dickey Tagging you both because this changes outputs, and I'm not totally sure if these changes will impact anything in the larger ecosystem. I don't think so because most i/o should be through sending stuff to conbench, but please check me on that (and feel free to review the rest in as much depth as you like).

Conbench runner expects benchmark() and record() methods of Benchmark classes to return benchmark dict and output. Returning BenchmarkResult object only will not work.
I recommend testing PR changes by running actual benchmarks in this repo locally using conbench.

So I did, and it...doesn't fail? It doesn't print anything even if I set --show-result and --show_output, but the return code is always 0 and it seems to be taking a reasonable amount of runtime for various runs. I don't have a local conbench server set up yet, so I can't really see what it's doing, though.
More strangely, I get exactly the same behavior on main. Looking at cli.py it looks like this branch should error out and main should work, but that's not what's happening. I need to dig deeper to see what's actually happening.
Taking a step back, does conbench do anything with the returns aside from the CLI [maybe] printing them? Everything sent to the server should still look the same. In terms of next steps, we could
a. move the class usage to only be internal to the overwritten methods so eventually JSON can be generated as a side-effect
b. make light wrappers for .benchmark() and .record() that return a BenchmarkResult for non-conbench use
c. make this PR to conbench instead and make accompanying PRs everywhere else that impacts
d. wait for the benchmark orchestration piece to get moved out of conbench and adjust it all then
I'm leaning towards (c) since we can likely just move the code over whenever we get to (d), but I'll have to go chase down every other repo it affects. Maybe there's a simpler option here?

@alistaire47 How do you run benchmarks?
I tried this now for this PR's branch and benchmarks defined in this repo fail:
cd ~/benchmarks

conda create -y -n ursa-benchmarks -c conda-forge --file requirements-build.txt python=3.8

conda activate ursa-benchmarks

pip install conbench

python setup.py develop

cd benchmarks

conbench list

conbench csv-read ALL --iterations=1 --all=true

Traceback (most recent call last):
  File "/Users/elena/miniconda3/envs/ursa-benchmarks/bin/conbench", line 8, in <module>
    sys.exit(conbench())
  File "/Users/elena/miniconda3/envs/ursa-benchmarks/lib/python3.8/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/Users/elena/miniconda3/envs/ursa-benchmarks/lib/python3.8/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/Users/elena/miniconda3/envs/ursa-benchmarks/lib/python3.8/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/elena/miniconda3/envs/ursa-benchmarks/lib/python3.8/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/elena/miniconda3/envs/ursa-benchmarks/lib/python3.8/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/Users/elena/miniconda3/envs/ursa-benchmarks/lib/python3.8/site-packages/conbench/cli.py", line 149, in _benchmark
    for result, output in benchmark().run(**kwargs):
TypeError: cannot unpack non-iterable BenchmarkResult object

Yep, sorry, got it failing properly now; I think the combinations of params I was trying were getting ignored

@alistaire47 @jonkeane Note that I don't have a strong opinion on using BenchmarkResult class in conbench or benchmarks repo.
I am kind of just watching you to see where it will go.


[Feature Request]: Enhanced Feedback Cards

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Responsive feedback cards featuring smooth transitions, interactive elements, and clear layouts. Showcase customer testimonials effectively while providing an engaging and visually appealing user experience tailored to your brand.
Use Case
Utilize feedback cards to display customer testimonials, enhance user engagement, and gather insights while maintaining a visually appealing interface.
Benefits
Boosts Engagement: Captivates users with interactive features.
Responsive Design: Optimizes for all devices.
Clear Testimonials: Builds trust with potential customers.
Brand Customization: Aligns with brand identity.
Valuable Insights: Gathers user feedback for improvement.
Add ScreenShots

This cards are just examples for the feature I want to update the feedback cards
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC-EXT contributor
[X] I'm a HACKTOBERFEST contributor
[X] I have starred the repository

@aditiverma-21, please go ahead

@GAVINESHWAR Thank you
I'll add PR soon.


Unable to apply MF transforms properly

Describe the bug
Because of https://github.com/manifestival/manifestival/issues/33 image override transforms cannot be applied properly
Expected behavior
apiVersion: operator.knative.dev/v1alpha1
kind: KnativeEventing
metadata:
  name: knative-eventing
  namespace: knative-eventing
spec:
  registry:
    override:
      eventing-controller: docker.io/foo/bar

would result in evening controller deployment's container to use docker.io/foo/bar
To Reproduce
Apply
apiVersion: operator.knative.dev/v1alpha1
kind: KnativeEventing
metadata:
  name: knative-eventing
  namespace: knative-eventing
spec:
  registry:
    override:
      eventing-controller: docker.io/foo/bar

Check evening controller deployment's container. It doesn't use docker.io/foo/bar
Knative release version
Operator version 0.13
Additional context
Need to upgrade to Manifestival 0.4.0 as in serving here: https://github.com/knative/serving-operator/pull/338

/assign @aliok

oh, this is already done with https://github.com/knative/eventing-operator/pull/142/files
I was apparently checking the 0.13 branch. Fixing it on master is fine.


Hi, i have 2 questions

hi, does this:

unrealengine plugin can driven metahuman?
does it has python script sending detection result?

hi yes it can drive a metahuman.  We have a walkthrough in the info section of our website www.poseai.co.uk.
we have a brief example python script here which shows how to stream JSON packets with all the animation data from the app to python.  This does not involve the unreal engine.  With the JSON you could read the data and connect with other applications


ReadToken errors in multi-gpu training

hi alls,
In multi-gpu training, I encounter strange errs, sometimes in "tr.iter*.1.log", sometimes in "cv.iter*.1.log", "*" means a rand iterator number, following is the err info：

cv.iter3.1.log:ERROR (train-ctc-parallel:ReadToken():io-funcs.cc:155) ReadToken, failed to read token at file position -1
cv.iter3.1.log:ERROR (train-ctc-parallel:ReadToken():io-funcs.cc:155) ReadToken, failed to read token at file position -1

or sometimes:

tr.iter2.1.log:ERROR (train-ctc-parallel:ReadToken():io-funcs.cc:155) ReadToken, failed to read token at file position -1
tr.iter2.1.log:ERROR (train-ctc-parallel:ReadToken():io-funcs.cc:155) ReadToken, failed to read token at file position -1

what's the possible reasons for this?

Some stability problem with file access? Are these local files, or being accessed via NFS or some other shared file system? Always for process 1, i.e. the master thread?

yes, these files being accessed via NFS, I am running ctc training in a cluster, with 1 master node and 9 other common node. By looking the source code and printing extra debug info, I found that sometimes the master thread(maybe job1 more exact) begin to read file(egg . "nnet/nnet.iter1.cv.done.job20") that has not been written over yet by other job（job20）, this cause the error!

hi @fmetze ,  I make a check in the code that if the file to be read do have not been written finished, let the master job wait for that!  This seems works for me, so I will close this issue!  thank you for your reply!

if the fix worked, would you mind submitting it? thanks!

ok, I will submit it in a few days!

@mixcoder, I am now encountering the same error in a multi-GPU setting, probably for the same reason, i.e. the master file attempting to read a file that has not been written yet.
Mind telling me which file you made your fix in, or even better your fix code?
I could make the pull request for you if that's a hassle.

I wrote a quick hack to pinpoint the problem.
In net/communicator.h, adding a sleep(1) (1 second) before it reads from the log file fixes the issue but adds a 1 second latency before every file read (l. 150, before ReadToken())
I'll work on a cleaner fix and submit it as a pull request in the next few days.

A colleague and I wrote a cleaner fix. The master job attempts to read from the subjob output file only if the file exists AND is not empty.
in src/base/kaldi-utils.cc, adding the following function:
bool FileNotEmpty(const char *file_name) {
std::ifstream infile(file_name, std::ifstream::ate | std::ifstream::binary);
return (infile.tellg()==0);
}
in src/net/communicator.h, reading from subjob output file only if file exists AND is not empty:
if (FileExist(subjob_done_filename.c_str())) {
if (FileNotEmpty(subjob_done_filename.c_str())) {
// read the file
}
}
@fmetze @riebling , does this seem like a good fix? Any other place where that race condition might occur in a multi-GPU setting? If satisfactory I will make a pull request.

Has all of this been tried over NFS file systems, or local discs? I have not worked with this code in a long time, but I think the best way to make these errors go away is to recommend to run the training on a single node, with a local file system, rather than a shared networked filesystem? Of course that may limit the numbers of GPUs that you can access - not sure if this is a factor?

I do use NFS, I am storing the data on the cloud on AWS EBS
https://aws.amazon.com/fr/ebs/pricing/.
Storing the data locally would imply uploading all the data to my cloud
instance every time (unless I'm missing something).
On 11 August 2017 at 15:48, Florian Metze<EMAIL_ADDRESS>wrote:

Has all of this been tried over NFS file systems, or local discs? I have
not worked with this code in a long time, but I think the best way to make
these errors go away is to recommend to run the training on a single node,
with a local file system, rather than a shared networked filesystem? Of
course that may limit the numbers of GPUs that you can access - not sure if
this is a factor?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/srvk/eesen/issues/134#issuecomment-321816632, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEXQ_PKD6zhmeSOCjejYoK0l8k7oXK9sks5sXFuQgaJpZM4NaWWi
.

--
Eric Bolo
CTO
tel: 06 29 72 19 80 http://06 58 79 25 37/

Batvoice Technologies
10, rue Coquillière - 75001 Paris
www.batvoice.com

It is typically a good strategy to copy all data that you are going to need onto the VM anyway at the beginning of the training process, because access will be much faster to local discs. Traffic into the VM is free, and you won’t need to retrieve it.

On Aug 11, 2017, at 8:19 PM, ericbolo<EMAIL_ADDRESS>wrote:
I do use NFS, I am storing the data on the cloud on AWS EBS
https://aws.amazon.com/fr/ebs/pricing/.
Storing the data locally would imply uploading all the data to my cloud
instance every time (unless I'm missing something).
On 11 August 2017 at 15:48, Florian Metze<EMAIL_ADDRESS>wrote:

Has all of this been tried over NFS file systems, or local discs? I have
not worked with this code in a long time, but I think the best way to make
these errors go away is to recommend to run the training on a single node,
with a local file system, rather than a shared networked filesystem? Of
course that may limit the numbers of GPUs that you can access - not sure if
this is a factor?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/srvk/eesen/issues/134#issuecomment-321816632, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEXQ_PKD6zhmeSOCjejYoK0l8k7oXK9sks5sXFuQgaJpZM4NaWWi
.

--
Eric Bolo
CTO
tel: 06 29 72 19 80 http://06 58 79 25 37/

Batvoice Technologies
10, rue Coquillière - 75001 Paris
www.batvoice.com
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/srvk/eesen/issues/134#issuecomment-321944406, or mute the thread https://github.com/notifications/unsubscribe-auth/AEnA8fOnV-zvmD5Ct2_ZXTrwg84ImUjnks5sXO-JgaJpZM4NaWWi.

Since this issue for NFS-only, I won't submit the pull request.
However, if someone is on an NFS system (for instance, an AWS EC2), I noticed a mistake in the code I wrote in my previous command, to check for empty file, the code should be "infile.tellg()>0" NOT "infile.tellg()==0".

Since this issue for NFS-only, I won't submit the pull request.
However, if someone is on an NFS system (for instance, an AWS EC2), I noticed a mistake in the code I wrote in my previous command, to check for empty file, the code should be "infile.tellg()>0" NOT "infile.tellg()==0".


sfy-drifter: sometimes goes crazy and sends thousands of messages

https://discuss.blues.io/t/track-qo-which-one-is-position-time/995/12

Check if the modem is badly connected
Check if the missing super-caps is the problem

Presumably this is the reason.


Code Folding shortcuts in spyder IDE

Hello,
is it possible to create a shortcut to folder all code up to some level ?
Reason is we often work with very large files...
Stackoverflow:
https://stackoverflow.com/questions/59943300/keyboard-shortcut-for-code-folding-in-spyder
2018 open issue:
https://github.com/spyder-ide/spyder/issues/6810
https://github.com/spyder-ide/spyder/issues/2430
Thanks !

Hi @uprokevin
I am also interested in this code-folding feature. Is it already possible, Carlos @ccordoba12 ?
I often have multiple cells (#%%) and want to close all of them, but I don't know how.
I have been a donator for two years and will continue to be. This feature would be a nice addition.
(On a separate note, improving remote development would be amazing too. I just had to switch to vs code for remote development in a container because even though I could setup spyder kernels there, most features like debugging didn't work)

2018 open issue:
https://github.com/spyder-ide/spyder/issues/6810

Hi there, so as mentioned this is a duplicate of https://github.com/spyder-ide/spyder/issues/6810

(On a separate note, improving remote development would be amazing too. I just had to switch to vs code for remote development in a container because even though I could setup spyder kernels there, most features like debugging didn't work)

There is some ongoing work to improve remote development capabilities inside Spyder. Not totally sure about remote debugging although it seems there are some issues open about that which we will try to tackle in future releases (issue like https://github.com/spyder-ide/spyder/issues/21996 https://github.com/spyder-ide/spyder/issues/10946) 🤔
Since seems like a round of questions and answer related to the code folding shortcuts idea has been done over https://github.com/spyder-ide/spyder/issues/21009 I will be closing this. However, any further feedback related with the shortcuts for the code folding is appreciated over #6810 Thank you!

I have been a donator for two years and will continue to be. This feature would be a nice addition.

@jmoraispk, sorry for the late reply. I agree with what you said and we'll try to implement it next year (the problem is that right now we are understaffed for all the things we'd like to do).


Suggestion: Refactor init.py into separate modules

I believe most if not all of the classes in __init__.py should be in their own files to make the code easier to work with. I would be willing to take on this refactor but obviously won't without a greenlight from @JakeCover

For sure, if you can do it and still make the package work. I know it's possible, but I don't have enough experience myself. If you can do it, go ahead!

sure thing i'll set to working on it this weekend.

Ok, the Refactor is done as of #5 being committed. Great job!

With your agreement, I'll close this issue

Actually, can you write a blurb about how the new code structure works, and what now lives in each file? This would be really useful to make sure we know what to add to each file. IMO a docstring at the top of each new file should work great, just a short paragraph is fine.

Sure I can bang that out right now.

Closing this since the refactor is complete and pushed to master.


HAAR-889: list of roles current user

List of roles for current user

Closing this PR as this logic can reside in 'manage user api' service


WIP: feat(metrics): add actix-web metrics exporter

For a starter, we can only include the basics metrics from the actix web prom crate.
as discussed in #773

this looks fine, and I read your discussion in nlopes/actix-web-prom#72 - but still not certain I understood the resolution of the problem -- would it make sense for the middleware to have an extra API to override path param to convert from /srcA,srcB/1/1/1 to srcA,srcB (but at the same time ignore the tilejson request /srcA,srcB? Should this be some magical attribute?

I don't really what you are saying, the issue is with the variance of /srcA,srcB, could also be /srcB,srcA (I suppose) So probably the best thing to do is to have a magical callback that is defined on the maplibre side, that take the request parameter and request path and return the path for metrics. So the business logic stay downstream and not defined in the lib. But, I don't recall that we settled on a solution yet.

Are you sure its a big deal to collapse /srcA,srcB and /srcB,srcA into one? I wouldn't worry too much about this tbh... Worst case - can later add some sort of a collapser, e.g. /srcA+ would mean everything with srcA as the first layer, plus any number of other layers mixed in.  The collapser could even be configurable to how many sources to keep, e.g. 0 -- /+ means all multi-sourced requests, or 2 would create /srcA,srcB and /srcA,srcB+ metrics

this looks fine, and I read your discussion in nlopes/actix-web-prom#72 - but still not certain I understood the resolution of the problem -- would it make sense for the middleware to have an extra API to override path param to convert from /srcA,srcB/1/1/1 to srcA,srcB (but at the same time ignore the tilejson request /srcA,srcB? Should this be some magical attribute?

I don't really what you are saying, the issue is with the variance of /srcA,srcB, could also be /srcB,srcA (I suppose) So probably the best thing to do is to have a magical callback that is defined on the maplibre side, that take the request parameter and request path and return the path for metrics. So the business logic stay downstream and not defined in the lib. But, I don't recall that we settled on a solution yet.

Are you sure its a big deal to collapse /srcA,srcB and /srcB,srcA into one? I wouldn't worry too much about this tbh... Worst case - can later add some sort of a collapser, e.g. /srcA+ would mean everything with srcA as the first layer, plus any number of other layers mixed in. The collapser could even be configurable to how many sources to keep, e.g. 0 -- /+ means all multi-sourced requests, or 2 would create /srcA,srcB and /srcA,srcB+ metrics

yes, you right, for now, let's not worry about this issue, will deal with that later.

@lefuturiste i added metrics to reserved in #802 , let me know if you need any help with this PR

@lefuturiste any updates? thx

@lefuturiste any updates? thx

Hi!
I've updated upstream PR for this, because I really want to have path cardinality (idk really why an obsession may be 😄 ), https://github.com/nlopes/actix-web-prom/pull/73

while i'm awaiting for a response from the actix-web-prom developer I will explore the instrumentation of postgresql connexion.

The PR at https://github.com/nlopes/actix-web-prom/pull/73 was merged today. So we can go forward from there.

update : I'm still working on it,
I'm waiting for another PR upstream to be merged
#87
It's ok, I'm patient. In 1 year may be this will be merged x)


Event "call service" "see" is not fired or logged on the device tracker platform

Home Assistant release with the issue:

arch
x86_64

dev
false

docker
true

hassio
false

os_name
Linux

python_version
3.7.3

version
0.92.2

virtualenv
false

Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):
Docker
Component/platform:
platform: device_tracker
components. fritz, nmap and possibly other device tracker platforms
Description of problem:
I have an automation that should be executed every time a device is “seen” .  However, the automation is never executed.
When manually firing the device_tracker.see service with mock data in the home assitant interface the automation works fine.
Otherwise the device tracker works well and reliable, for presence detection.
No errors occur.
Furthermore when analyzing the home assistant.db it shows that the service is never logged, except, when called manually. However, the service is registered multiple times.
Excerpt of the db:
Service registered entry:
"98503"	"service_registered"	"{""domain"": ""device_tracker"", ""service"": ""see""}"	"LOCAL"	"2019-05-03 05:32:43.082224"	"2019-05-03 05:33:02.843096"	"fcde4b5ce5e1462383bdbbd56d1809ca"
Manually triggered event in web-ui
Mock data: {"dev_id":"device", "location_name":"home"}
Corresponding database entry:
"226942"	"call_service"	"{""domain"": ""device_tracker"", ""service"": ""see"", ""service_data"": {""dev_id"": ""redmi_note_5"", ""location_name"": ""home""}}"	"LOCAL"	"2019-05-09 12:13:10.476427"	"2019-05-09 12:13:10.497045"	"a5ebf98b12824269a6a4f0318c97a223"	"ef49cf62bc7b44f0ae88b9bf8c3ad62c"
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
- alias: Unknown Network device connected
  trigger:
    platform: event
    event_type: call_service
    event_data:
      domain: device_tracker
      service: see
  action:
    - service: system_log.write
      data_template:
        level: info
        message: >-
          {% set x = trigger.event.as_dict()['data'] %}
          DATA: {{ x }}, ENTITY_ID: {{ x.service_data.dev_id }}

Traceback (if applicable):

Additional information:
This behavior has been verified on a second local home assistant instance using the nmap and fritz component.
As a further remark: Testing the call service event using the identical automation with a different platform
      event_data:
      domain: light
      service: turn_on

works flawlessly.

This is normal.
Device tracker platforms work directly with the device tracker component and do not call the device_tracker.see service. Consequently, when they "see" a device, the service is not fired, and hence you won't see a call_service event.
If you want an automation to fire when a device is seen/updated, then you should probably be using a state trigger.

Thank you


Support lists in text boxes

Coverage remained the same at 100.0% when pulling 8ea7eab4666cb8ea5c1886b1c036b5d1c2aa19a5 on dsager:list-in-text-box into 60083cf5581a59ecd133fdd1244612f584aa57b3 on Devex:master.

+1

+1

+1

+1


Timer design changes for compact mode

[x] Make the font smaller - the same size as the listing font.
[ ] Duration and start/stop button fonts should have the same size as other text
[ ] Start button should be smaller in width. Maybe the width of two header icons instead of three.
[ ] Switch the locations of project and description field. Description is always on top.
[x] Hide the 00:00 (duration) field altogether when the timer is not running

Mostly implemented, any further details should be discussed, but are easily added/changed.


command not found : complete when using bash completion

Hello,
I have added the code for completion in my .bash_profile file and when I source it, I have the following error :
rvm/gems/ruby-2.0.0-p598/gems/lunchy-0.10.1/lib/../extras/lunchy-completion.bash:18: command not found: complete
Looks like the command used there https://github.com/eddiezane/lunchy/blob/5640b8a47720adeb085924994aaabe07c80ee212/extras/lunchy-completion.bash#L18 isn't available on my system (OSX 10.10.4).
Thanks for your help.

  If not, this completion script won't help and you should probably leave it out of your profile/bashrc.
If you are using bash, do you have bash_completion installed?
If not, or you can try installing it via homebrew.
You can check to see if bash_completion is available (assuming you have homebrew).
brew list
you should see bash_completion in the list.  If not then install it
brew install bash_completion
Then restart your terminal and see if that works.
If you get this far and things are still not working, let us know and we'll try to get it figured out.  Please include the version of bash, the list of brew packages.

My bad, I'm using zsh :-( Thanks for your quick answer.

i don't know how zsh completion works, but it seems like there must be a mechanism.  If you'd like to write a completion script, we'd be happy to roll it in.

zsh have a bash completion script compatibility mode, you need to run :
autoload bashcompinit
bashcompinit

before sourcing the bash completion file. I have tested it and it works partially (listing the lunchy command is ok but not listing of the plist files).
I'll investigate and try to find  a solution to have a working completion for ZSH.

Proposal in #59

Closed via #59

that is cool!

Is there a workaround for ZSH yet?

Have you tried the latest?  It looks like that has been solved by #3236 (referenced above).  Try installing the latest version, follow the zsh completion instructions and see if that works.

autoload bashcompinit
bashcompinit
I tried this, but it didn't work for me.

For those who were not able to make it work, just paste those two lines in your ~/.zshrc file right before the source filethathascompletion.bash line

At the end of the zshrc ^^. Putting it on the beginning does not fix the issue

autoload bashcompinit
bashcompinit

On top of .zshrc works! 👍

this is the only thing that worked for me.
got it from here https://github.com/creationix/nvm/pull/1780/files
added this to the top of my .zshrc
autoload -U +X bashcompinit && bashcompinit
autoload -U +X compinit && compinit

this is the only thing that worked for me.
got it from here https://github.com/creationix/nvm/pull/1780/files
added this to the top of my .zshrc
autoload -U +X bashcompinit && bashcompinit
autoload -U +X compinit && compinit

thx bro! finally works~

this is the only thing that worked for me.
got it from here https://github.com/creationix/nvm/pull/1780/files
added this to the top of my .zshrc
autoload -U +X bashcompinit && bashcompinit
autoload -U +X compinit && compinit

This for sure worked!

autoload bashcompinit && bashcompinit
The same principle, this method will be more concise

this is the only thing that worked for me.
got it from here https://github.com/creationix/nvm/pull/1780/files
added this to the top of my .zshrc
autoload -U +X bashcompinit && bashcompinit
autoload -U +X compinit && compinit

This for sure worked!

This doesn't work for me on macOS 14.2. Neither at the top, nor at the bottom of .zshrc.


write port label to certain layer

@meghna09 We are adding constraints in this pull request to specify port label layer.
The constraint format is:
{             "ports": [                 {                     "port_name": "vbiasnd",                     "layer": ["M4", "M5", "M3"]                 },                 {                     "port_name": "vss",                     "layer": ["M5", "M4", "M3"]                 }             ],             "const_name": "LabelLayer"         }

@meghna09 As this constraint is not supported by constraint translator yet, we cannot using this constraint in the flow directly. One temporary solution is to git cherry-pick af28a70 into your old ALIGN flow, and use the constraint format in the above comment. However, this solution is not a long-term solution.
So I created a issue in #705 .  We want to directly copy any new constraint from users into the PnR input folder.
@soneryaldiz , could please help make the new constraint available to the PnR input folder? Thank you.

@meghna09 As this constraint is not supported by constraint translator yet, we cannot using this constraint in the flow directly. One temporary solution is to git cherry-pick af28a70 into your old ALIGN flow, and use the constraint format in the above comment. However, this solution is not a long-term solution.
So I created a issue in #705 . We want to directly copy any new constraint from users into the PnR input folder.
@soneryaldiz , could please help make the new constraint available to the PnR input folder? Thank you.

@854768750 , could you please elaborate what this constraint means? Based on you example above:

vbiasnd should be drawn up to M5 even if M4 and M5 were not used by signal router?
if vbiasnd ever used M3, label the M3 wire segments as pins?

vbiasnd should be labeled at M4 first. If M4 does not exist, then choose M5, then M3.

Alternative to this constraint (till we polish this further), could we:

for signals, automatically label all top metal wires as ports
for power, automatically label the top metal layer as ports
in any case put the label in the center of a wire

Sure, we can put label at the top metal layer

Multiple labels are supported, so we can have a label on each metal when multiple parallel connections on the highest metal layer are present.

@soneryaldiz @meghna09 to discuss and close this out

Alternative to this constraint (till we polish this further), could we:

for signals, automatically label all top metal wires as ports
for power, automatically label the top metal layer as ports
in any case put the label in the center of a wire

We decided to follow this strategy for now
We might need to explicitly tag pin locations in the future but that will need some concept of location & layer.
Closing this PR for now.