Plug-able architecture fo FastMQ

Problem
Currently there is no easy way to implement extensions (Plug-ins) to the FastMQ functionality which limits its usefulness for production deployment.
Solution
@voycey team plans to implement integration with Google PubSub and the Google PubSub emulator (for local use), the implementation would look like:

Create an interface for the message broker that includes: /mrv endpoint (publish)
Modify FastMQ to extend this interface and create package message-broker-fastmq and modify package.json to keep default setup
Create new package message-broker-pubsub
Look at possibly implementing this via webpack (https://webpack.js.org/plugins/copy-webpack-plugin/) to allow for easy customisation
PubSubEmulator Docker Container would be created for local development (named message-broker)
Environment variables for production setup to connect to Live PubSub vs PubSubEmulator

This setup would keep the current default setup but provide a plugin architecture to allow for new message queue plugins to be built according to requirements (AWS SQS & Redis for example).
Alternatives
No response

This should be "plugable Archtecture for Message Queue Component" - the idea is to be able to give an easy way to replace FastMQ with whatever queue system is required whilst leaving FastMQ as the default for an install to allow for fast local development.
FastMQ will not be production ready without TLS (even authentication will not be enough as it would be transmitting unencrypted information across networks)

Duplicate of #215

This ticket is also cancelled as we have cancelled #215


Updated Stack Info Sites

Removed nonoperational sites
Added obycode stacks site

 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.


[5.9] Fix Swift SDK bundles not working due to quarantine

When using swift experimental-sdk install subcommand with bundle archives or bundles unpacked from archives downloaded manually in the browser, installed executables are quarantined and trigger an error message when used by swift build
Steps To Reproduce:

Download a destination bundle archive
Unpack and install the bundle with swift experimental-destination install
Confirm that the destination is installed and note its ID with swift experimental-destination list
Build using the newly installed destination with swift build --experimental-destination-selector <destination-id>

Actual result:
Building with this destination triggers an error: "swift-driver cannot be opened because the developer cannot be verified."
rdar://107392863
Modifications:
Running xattr -r -d -s com.apple.quarantine on the installed bundle fixes the error. The reason is that bundles downloaded in the browser get a quarantine attribute assigned and unpacking such bundles preserves it.

Converting to a draft as it's blocked by https://github.com/apple/swift-tools-support-core/pull/419 and https://github.com/apple/swift-tools-support-core/pull/412

@swift-ci smoke test

@swift-ci smoke test

@swift-ci test windows

@swift-ci smoke test

@swift-ci test windows


merge from template-node-app

package.json

chore(deps): add ^11.5.12 remove ^11.5.10 (devDependencies.npm-pkgbuild)

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

npm package (@latest dist-tag)
v6.4.0
GitHub release

Your semantic-release bot :package::rocket:


Not centered on Android

On my android it looks like that... not centered... I am doing something wrong?
Thanks

Hi there,
Is this from the example? If not can you please share the code?
Thanks


Add-on behavior not defined for merging users

@JeffSpies @sloria
When user accounts are merged, on each project, one account is added and the other is removed. This activates the after_remove_contributor callback, which removes add-on credentials from all projects that the removed account had authorized.
I'm not sure what the expected behavior should be—the safest bet would be to inform the user before merging that all projects authorized by the "deleted" account will need to be manually reauthorized. We could instead/additionally define some add-on after_merge_contributor callback to handle this case more specifically, although this is a bit of an edge case to have to define a callback for all current and future add-ons. Thoughts?

I am going to remove 'deferred' on this one -- just in case this "standardization will occur in an upcoming refactor of add-ons" can happen in next Epic.

Self-assigning; this is part of my task list for this epic.

@lyndsysimon -- This taken care of in level 2 merge #2548 - Correct?

Fixed in https://github.com/CenterForOpenScience/osf.io/pull/2548

The pr already goes into production.


Discrepancy in :before content - dot • gets encoded as â€¢

Source on github:

https://github.com/kenwheeler/slick/blob/master/slick/slick-theme.scss#L13 $slick-dot-character: "•" !default;
https://github.com/kenwheeler/slick/blob/master/slick/slick-theme.css#L188 content: '•';

CDN:
https://cdn.jsdelivr.net/jquery.slick/1.4.1/slick-theme.css

+1

same. had to replace these by hand, otherwise my sass wasn't compiling.
$slick-prev-character: "←" !default;
$slick-next-character: "→" !default;
$slick-dot-character: "•" !default;

I think this PR should fix the issue: https://github.com/kenwheeler/slick/pull/1045 ?

+1


Fixes an issue with duplicate gold subscriptions

This PR fixes a bug that could result in a user having multiple gold subscriptions.
I am also going to go back through our gold subscriptions and find any users that have more than 1 subscription as that is likely an error.
Fixes #4595
Fixes #4503

I am also going to go back through our gold subscriptions and find any users that have more than 1 subscription as that is likely an error.

I have cancelled all duplicate subscriptions and refunded Gold Members for the duplicate charges.


Update Assembly.LoadFrom with URI parameter

Taken from https://github.com/dotnet/runtime/issues/38714. We should doc the difference in .NET Core.

Assembly.LoadFrom is supposed to take a URI as its parameter:

The assemblyFile parameter must refer to a URI without escape characters. This method supplies escape characters for all invalid characters in the URI.

On .NET Framework, you can pass in a file:// URI (constructed from the Assembly.CodeBase URI of another assembly, for example) to Assembly.LoadFrom and the assembly will be loaded.

On .NET Core, the assemblyFile parameter is treated as a path. The URI gets passed into Path.GetFullPath which mangles the URI to begin with and continues to treat it as a path from there on. This never works for loading an assembly from any URI.

@steveharter This sounds like a breaking change, can you please the breaking change template? https://github.com/dotnet/docs/issues/new?template=dotnet-breaking-change.md
Thanks!

Re-opened with the breaking change template; it's not really a breaking change because the semantics of .NET Core have not changed -- it's only a doc issue.


test: add sub & unsub benchmark

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
1476

Relevant Lines:
1476

💛  - Coveralls


Markdown tables not rendering

On BeyondBoinc:

https://beyondboinc.net/authors/steempower/bitshares-state-of-the-network-25th-april-2017
On Steemit:

https://steemit.com/bitshares/@steempower/bitshares-state-of-the-network-25th-april-2017

Glasnost has evolved to become a GraphQL app server, HTML pipeline was removed and this issue is no longer relevant.


Remove broadcast in min/max when forwarding to builtins

Resolves issue #535 where CPMpy's min/max is inconsistent with Python's built-in min/max on iterators with single (non-cpmpy) element. Old implementation with broadcast operator doesn't take this edge-case into account and causes an exception. The builtin works perfectly fine with handling the iterator directly, no need for broadcasting.

Thanks, good catch!


Use textwrap.dedent instead of custom replace + strip

Codecov Report

Merging #183 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #183   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          11     11
  Lines        1300   1303    +3
=====================================
+ Hits         1300   1303    +3

Impacted Files
Coverage Δ

ldap2pg/defaults.py
100% <100%> (ø)
:arrow_up:

ldap2pg/config.py
100% <100%> (ø)
:arrow_up:

ldap2pg/role.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Error:Configuration with name 'default' not found.

In android studio I m getting this error after linking react-native-maps. Settings.gradle looks like this
include ':app' include ':react-native-maps' project(':react-native-maps').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-maps/android')

change it to like this
include ':app', ':react-native-maps'
project(':react-native-maps').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-maps/android')
also check your android/app/build.gradle file . Comment the error you are getting.

no same error still exists.no success.

hey riten7 follow this link http://stackoverflow.com/questions/38095524/cannot-evaluate-module-react-native-maps-configuration-with-name-default-n
here you will get the answer i also faced the same. Got my query solved here.

thanks @sayedatif .. it helped me..

Thanks for answering


The overlay prevent me to write to /dev/*

The overlay prevent me to write to /dev/*
I am using overlayroot for a while and I like it. Unfortunately, this was coming with some drawback that I now need to address, and I am not sure how to achieve that the correct way.
I need to be able to access any devices in /dev, and write to them. This seems impossible to do so when overlayroot is enable. (trying to write to /dev/watchdog0)
Is there a built-in way to achieve that? Or, what would be the better way to do this, while keeping the overlayroot in action?
NOTE: I am aware of the rwrootfs command, which make it works for testing, but I need to be able to continuously write to the watchdog, and NOT in the rwrootfs mode.

Are you sure than overlayrootfs is fully installed and working correctly at all?
With it running you can normally write to root anywhere with overlayrootfs running, despite the physical drive being read only. That's what overlayrootfs is meant to do.

As @Lalarian pointed out /dev/* files are not real files and do not come from a real filesystem. They cannot be write-protected in a way a filesystem can be
This is most likely a permissions problem.

Thank you very much, you're right, I was using "sudo" to test, I needed to use "sudo su" to be root first, and then echo 1 > /dev/watchdog0 worked.

Maybe did you try
 sudo echo 1 > /dev/watchdog0

?
This won't work as only echo will be run as sudo.
Use
 echo 1 | sudo tee -a /dev/watchdog0

instead.


It says I’m set up even tho this is a neew

Damn device . I exchanged my devices at apple store for new phone same model I can not set up the app bc it showing still active on some other devices?

what?

I got a new iPhone and I can’t sign in to my Gsuite on business account bc
1 the calls are not coming thru for 2fA and the app says to get a code from
the google auth app how is that even possible I didn’t set up on this phone
yet
On Tue, Apr 9, 2019 at 5:34 AM Thomas Habets<EMAIL_ADDRESS>wrote:

Closed #672 https://github.com/google/google-authenticator/issues/672.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/google-authenticator/issues/672#event-2262233924,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AvJAitsP4-wuaJmGOnDocu0oI5YWU8HUks5vfF65gaJpZM4ckBmx
.

--
null

If you don't have access to your current OTP generator (your old phone) then this is an account recovery problem.
Maybe https://accounts.google.com/signin/recovery can help.
This github repo can't help with account recovery.


Battery drain issue

Thank you for your solution but I have battery issue when I used it for background task.
If I didn't move, my iphone consume battery 4-5% per hour.
But if I moved anywhere, it consumed more than 10% per hour.
I already set desiredAccuracy very low and set terms between location update.
How can I reduce battery drain??

The same issue was occurred for me but after deep investigation I've found frequent network requests are responsible for huge energy consumption.


Update Types

Hey,
Thanks for the awesome package. Could you update the return types from the hooks. Gettin TS errors
e.g.
function createChromeStorageStateHookLocal<S>(key: string, initialValue: S | (() => S)): [S, Dispatch<SetStateAction<S>>, boolean, string];
should be
function createChromeStorageStateHookLocal<S>(key: string, initialValue: S | (() => S)): () => [S, Dispatch<SetStateAction<S>>, boolean, string];

@onikienko cool no worries. Opened a PR


Integration test org/apache/river/test/impl/lookupdiscovery/MulticastRequestDelay.td in unfinished state

Also reported as River-469 https://issues.apache.org/jira/projects/RIVER/issues/RIVER-469
The integration test org/apache/river/test/impl/lookupdiscovery/MulticastRequestDelay.td was never completed and is configured to be skipped by the test suite.
The intent of the test is to check the configuration of initialMulticastRequestDelayRange results in an average delay where the mean delay is halfway between zero and the configured value.
The test records the start time, then starts 100 LookupDiscovery instances, there is a significant start up penalty and a significant amount of time elapses by the time the last LookukpDiscovery instance has started.   Additionally each LookupDiscovery continues to send multicast datagram packets and the test only measures the first 100 datagram packets that arrive from localhost.
The problem is that the test does not determine whether the packet received is the first packet sent by a LookupDiscovery instance.
Also the test should only check whether each received datagram is sent from a local network interface, as the packets aren't sent from localhost in this instance.
Recommend rewriting this test to start one LookupDiscovery at a time, measure the delay time to receive the first multicast datagram from that LookupDiscovery, then shut it down.
Also recommend that the LookupDiscovery delay be modified to account for it's own startup delay, so that the average result is as expected.

•
initialMulticastRequestDelayRange

long

0 milliseconds

With respect to when this utility is started,  this entry controls how long to wait before sending out the first  multicast request. If the value is positive, the first request will be  delayed by a random value between 0 and initialMulticastRequestDelayRange milliseconds. Subsequent request intervals are controlled by the [ multicastRequestInterval
http://river.apache.org/release-doc/current/api/net/jini/discovery/LookupDiscovery.html#multicastRequestInterval]  entry. Note that this entry only has effect when this utility is  initialized. The first multicast request is not delayed if the groups to  discover are subsequently changed.

Test reactivated and passing, refer to test comments.


[Karpenter] Pod Identity Supports

Issue:
Pod Identity Supports Notice
Karpenter now supports using Pod Identity to authenticate AWS SDK to make API requests to AWS services using AWS Identity and Access Management (IAM) permissions.
Description of changes:

Introduce option to enable Pod Identity, this replaces IRSA method to manage credentials for Karpenter controller.
Resources created for using Pod Identity

IAM role with service principle pods.eks.amazonaws.com and allow actions sts:AssumeRole and sts:TagSession
Pod Identity associations
Service account created by Helm chart without eks.amazonaws.com/role-arn annotation

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@shapirov103 Do you know what's wrong with the markdown-link-check ?

@vumdao the md check failure is not related, it is fixed now in main.

/do-e2e-tests


i18next is undefined inside layout

I get this error inside my layout because it can't access the i18next variable. It would be really nice if I could automatically detect the locale in my layout because that's where I put my  tag.

Hey @L1lith, thanks for the issue!
I'm sorry, I'm not sure to understand here. I'm able to import i18next in my layout components using
import i18next from "i18next";. You can see in the BaseTemplate in the example project.
Do you have a minimum reproduction or example?

Hey @L1lith, do you still have this issue? May I close it otherwise?

Hey @L1lith, do you still have this issue? May I close it otherwise?

Yes, sorry I think I made a simple mistake. Sorry it took me so long to respond 😅


Utility.hpp needs to be updated or removed.

The code in model/kernel/utility.hpp is legacy code used to support the testing of some translated kernels.  This has memory allocation using malloc() which should be replaced with more c++ centric memory management, i.e. vectors for arrays of double/float.

I'll take a look at it.
Shengting
On Fri, Apr 10, 2020 at 3:30 PM Nels<EMAIL_ADDRESS>wrote:

The code in model/kernel/utility.hpp is legacy code used to support the
testing of some translated kernels. This has memory allocation using
malloc() which should be replaced with more c++ centric memory management,
i.e. vectors for arrays of double/float.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NOAA-OWP/ngen/issues/49, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACA4SRLFUJ2YDKHXWOE5ALDRL56WXANCNFSM4MFV7FDA
.

The change has been made to use vector instead of malloc(). Please review the most recent utility.hpp.

Utility.h updated by #102 so closing this for now. May decide later to remove.


faq.md: 1.0.0 has been released ;)

@RichiH Could you update per @beorn7's comment?
Per discussion last week, we're not actually following semver to the letter so that wording should be weakened.

This is now sup.rseeded by https://github.com/prometheus/docs/pull/702/


Fix bug in default headers argument in server.call

headers is a mutable type, so its an unsafe default argument if we
modify it (which we now do)

I made this bug, now I'm fixing it.

cc @chrismattmann


add custom dataset training

This piece of code will help in training the current model on a custom dataset.

please do not put 'guided_diffusion' folder under 'script'

Okay sir got it.
On Wed, Jun 7, 2023, 7:04 AM Wu Junde @.***> wrote:

please do not put 'guided_diffusion' folder under 'script'
—
Reply to this email directly, view it on GitHub
https://github.com/WuJunde/MedSegDiff/pull/120#issuecomment-1579715982,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOS6XBSM47SFVBUP7LLHSGDXJ7LA7ANCNFSM6AAAAAAY4N3UTA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thank you for the merge
On Mon, Jun 12, 2023, 5:37 PM Wu Junde @.***> wrote:

Merged #120 https://github.com/WuJunde/MedSegDiff/pull/120 into master.
—
Reply to this email directly, view it on GitHub
https://github.com/WuJunde/MedSegDiff/pull/120#event-9501796632, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOS6XBWV4BOWX2KQMXOWBKTXK4A6TANCNFSM6AAAAAAY4N3UTA
.
You are receiving this because you authored the thread.Message ID:
@.***>


PEP 8 and String formatting

Signed-off-by: vkmrishad<EMAIL_ADDRESS>
PEP 8 fixes
String Formatting

@ksatchit Thank you

lgtm

Can you resolve the conflicts @vkmrishad ?

@ashishranjan738
Conflict fixed.

@shiveshabhishek @ashishranjan738 @ksatchit
PR can be merged.

@ksatchit
Thank you


Add finder option to keep folder always on top

Thanks for this project. This is a useful collection to setup my mac. There is one option I added to my script apps/finder.sh.
# Keep folders on top when sorting by name
defaults write com.apple.finder _FXSortFoldersFirst -bool true

regards,
David

Thanks for the suggestion! Added


poor display of multi-line tabular output

we are using a height on the row, with no vertical padding; this works fine for single-line cell content, but poorly for multi-line cell content

the last PR was no good


Normalize min and max method parameters

The min and max are not normalized at the first place, as they are used as well in comparing dates (min and max date).
Also, there are 7 failing tests.

That's what I noticed in the failing tests. I guess is up to the developer to pass the right config and this PR can be closed.
Still feels weird to use that rule for dates and numbers though

Still feels weird to use that rule for dates and numbers though

The date input has min and max attribute (http://www.w3.org/TR/html-markup/input.date.html) but not every browser has support for it (e.g. IE, Firefox and Safari. See http://caniuse.com/#feat=input-datetime).
Also it's already mentioned in the source code, see src/core.js#L1092-L1101.

Thank you :)

You're welcome :)

@rodrigoaguilera can we also close https://github.com/jzaefferer/jquery-validation/issues/1660 then?

Oh yeah, I forgot


feat: new validator middleware using 3rd-party & current middleware obsolete

This RP includes big changes. If we accept it, we have to merge it into next branch, and it will be released in the next major version, "v3".
In this PR, I introduce new validator middleware supporting the "RPC" feature. And make the current built-in validator obsolete.
Current built-in validator obsolete
Perhaps this means "heartbroken".  I really like the current Built-in Validator, it can easily handle JSON, Forms, etc., and is easy and intuitive to write. But the more we try to make it better, the more we have to do. It is very difficult to make the Validator correct. So I began to think that there was no need for us to make our own validators. I am very sorry to the contributor who worked with, but with this PR, I have removed the current built-in validator completely.
New "generic" validator middleware for using 3rd-party
So I would recommend using 3rd validators, such Zod, TypeBox. They provide type safety as well as value verification.
Therefore, I created a thin Validator middleware that can handle such 3rd-party Validators in a generic way.
New types for RPC
On the other hand, I would like to add an "RPC" feature like tPRC. This means that server-side API definitions are shared as "Types" and can be used by the Client.
https://github.com/honojs/hono/issues/727
So, I added new types and methods app.build() and c.jsonT() to provide "RPC" for 3rd-party validators.
Example
For example, we can write the validator with "Zod" like this:
const v =
  <T extends ZodSchema>(schema: T) =>
  (value: unknown, c: Context) => {
    const parsed = schema.safeParse(value)
    if (!parsed.success) {
      return c.text('Invalid!', 400)
    }
    const data = parsed.data as z.infer<T>
    return data
  }

const api = new Hono()

const postRoute = api
  .post(
    '/author',
    validator(
      'json',
      v(
        z.object({
          name: z.string(),
          age: z.string(),
          nickname: z.string().optional(),
        })
      )
    ),
    (c) => {
      const author = c.req.valid()
      return c.jsonT({
        youare: {
          name: author.name,
          age: author.age,
        },
      })
    }
  )
  .build()

export type AppType = typeof postRoute

This is a screencast for the demo. It's a little different from the implementation in this PR, but it works the same.
https://user-images.githubusercontent.com/10682/209447905-bf1afe60-c0fd-45bb-881a-9cf00a352d08.mov
Make the client
For now, I plan to create an HTTP cline that wraps fetch. We will not include this in the core package, but will distribute it under @hono namespace @hono/client. And will be managed in the monorepo for middleware.
@hono/zod etc.
It is also a good idea to create a module like @hono/zod to write it simpler and shorter.
TODO
If we accept this PR, there are a few things we must do.

Refactor the code.
Deprecate the current built-in validator.
Release the next branch as "pre-release" and test it well.
Create @hono/client.
Try validators other than "Zod".

Makes sense to me. It was fun to work on the validator but you are right, it's a lot of work to make it work well and zod does a lot of the good work for us. I think this makes sense 👍

Multiple Validator support and a large refactoring on Type. All done. Merging.


Strange behaviour of FaceMesh solution with max_num_faces parameter

Hello to everyone,
I noticed a strange behaviour of the python solution "FaceMesh" (mediapipe version <IP_ADDRESS>). When processing some selected images containing faces, no faces are detected when setting up the FaceMesh object with parameter max_num_faces=1, but a single face is detected when I provide 2 or more as parameter.
Img to reproduce:

Code to reproduce:
import cv2
import matplotlib.pyplot as plt
import  mediapipe as mp
import  numpy as np

path = 'img.jpg'

def normalized_landmarks_to_image_coords(landmarks, img_shape):
    x_list = []
    y_list = []
    z_list = []
    for i, p in enumerate(landmarks.landmark):
        x_list.append(min(p.x*img_shape[1], img_shape[1]-1))
        y_list.append(min(p.y*img_shape[0], img_shape[0]-1))
        z_list.append(p.z)
    x_list = np.array(x_list)
    y_list = np.array(y_list)
    z_list = np.array(z_list)
    result = np.stack([x_list, y_list, z_list])
    return result

face_mesh = mp.solutions.face_mesh.FaceMesh(False, max_num_faces=1, min_detection_confidence=0.1)

img = cv2.imread(path)
img = cv2.cvtColor(img, cv2.COLOR_BGR2RGB)

plt.imshow(img)
result = face_mesh.process(img)
if result.multi_face_landmarks is not None:
    print(len(result.multi_face_landmarks), ' faces')
    landmarks = normalized_landmarks_to_image_coords(result.multi_face_landmarks[0], img.shape)
    landmarks = landmarks[:-1, :]
    landmarks = np.transpose(landmarks)
    plt.scatter(landmarks[:, 0], landmarks[:, 1], s=3.2)

else:
    print('No faces')

Output with max_num_faces=1:
No faces

Output with max_num_faces=2 or more
1 faces

It happens just a bunch of times over 25K+ images tested, so it's not a big problem, but this behaviour is still unclear to me and the parameter seems to be counterintuitive

Hi @FedericoCozziVM, Could you please try the above with the latests release. Thanks!

Hi @sgowroji, I tested it with mediapipe pip version <IP_ADDRESS>, but the issue is not fixed yet.
Code:
fig, ax = plt.subplots(1, 3, figsize=(20, 10))
for i in range(3):

    face_mesh = mp.solutions.face_mesh.FaceMesh(False, max_num_faces=i+1, min_detection_confidence=0.1)

    ax[i].imshow(img)
    result = face_mesh.process(img)
    if result.multi_face_landmarks is not None:
        for r in result.multi_face_landmarks:
            if r is not None:
                landmarks = normalized_landmarks_to_image_coords(r, img.shape)
                landmarks = landmarks[:-1, :]
                landmarks = np.transpose(landmarks)
                ax[i].scatter(landmarks[:, 0], landmarks[:, 1], s=3.2)
    ax[i].title.set_text(f'max {i+1} faces')

plt.show()

Result (result with max_num_faces [1, 2, 3]):

Hi @FedericoCozziVM ,
Are you still looking for the resolution on this issue.

Hi @sureshdagooglecom, no I'm not actively looking for a solution, but I just tested on mediapipe 0.8.11 and it's still an issue

Hi @FedericoCozziVM,
Our dataset for training face detection for the front camera is composed of mostly selfie images and we trained with rotation augmentation (+/- 45 degrees). So there are training examples up to 90 degrees rotation.
However, we don't have much flipped face in the dataset and the model is not expected to work in that case. Thank you!

Hi @kuaashish,
thank you for answering. I know, I also read the paper about the training data of the backbone face detector model. The problem is not about not detecting the old man, who is in fact too much rotated. The problem is that if I set the parameter max_num_faces = 1, I would expect to detect the face of the old lady, which is frontal: instead, the face is detected only by setting max_num_faces = 1 or more.

Hi @FedericoCozziVM,
Could you please investigate with several different input images mostly the selfie images or the front facing images and adjust the parameter accordingly to detect the number of faces using the max_num_faces. Please report back if the issue still persists that will be helpful to investigate further. Thank you!

Hi @FedericoCozziVM,
Could you please go through the above comment. Thank you!

Hi @kuaashish and @gkarpiak, thanks for your responses, and sorry if I answered late.
@gkarpiak that's a great idea! I just made some experiments in this colab recycling some code from @kuaashish ones.
mp.solutions.face_detector (blazeface) with model_selection=0 finds 2 faces:

right one (frontal old lady) with confidence 0.68
left one (lateral old man) with confidence 0.57 (actually, that's a noisy detection and obviously bounding box and basic facial landmarks - eyes, mouth, nose -  can't be placed correctly)

So that's even stranger: face detector finds 2 faces and ASSUMING that the face detector model is the same in both mp solutions, as reported in documentation, when setting max_num_faces=1 in FaceMesh it should keep and analyze the face of the old woman that has a higher confidence score... instead FaceMesh continues detecting the right face only when this parameter is 2 or higher.
It would be helpful to pass FaceDetector detections to FaceMesh "externally" or to set different model_selection values to the inner FaceMesh face detector, or again to access FaceMesh confidence score, but I couldn't find a way to do it

Hello @FedericoCozziVM,
We are upgrading the MediaPipe Legacy Solutions to new MediaPipe solutions However, the libraries, documentation, and source code for all the MediapPipe Legacy Solutions will continue to be available in our GitHub repository and through library distribution services, such as Maven and NPM.
You can continue to use those legacy solutions in your applications if you choose. Though, we would request you to check new MediaPipe solutions which can help you more easily build and customize ML solutions for your applications. These new solutions will provide a superset of capabilities available in the legacy solutions.


Add 22.04 to the CI Environment

removed the -g1 optimization due to compiler bug.

@causten any more changes for this?

yes, still need to get past compiler hangs

yes, still need to get past compiler hangs

If hangs only happen on 22.04 debug, then we could just disable that step. We still run the debug version with 20.04.


D20 Intention unclear

D20 states: A JWS may be used for establishing a secure tunnel between BPIs to reduce the number of messages required.
It is unclear how this relates to the rest of the requirement. It's also unclear when or where the tunnel technique is useful here. Finally, it's unclear how this reduces the number of messages required. Are they required for authentication and authorization, or something else? Please help clarify.

@atoulme indeed not clear. It should read:
A JWS may be used for initial authentication and authorization between BPIs.
Good enough?

That clarifies the discussion, thanks.

Closing. Addressed with merged PR https://github.com/eea-oasis/baseline-standard/pull/173


Remove the "View in Inventory" from the RHc settings

Before:

After:

Codecov Report

Merging #29 (5d385aa) into master (515ac5e) will increase coverage by 0.20%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
+ Coverage   11.66%   11.86%   +0.20%
==========================================
  Files          34       34
  Lines         343      337       -6
  Branches       57       51       -6
==========================================
  Hits           40       40
+ Misses        294      288       -6
  Partials        9        9

Impacted Files
Coverage Δ

src/Routes/Dashboard/index.js
0.00% <ø> (ø)

Continue to review full report at Codecov.

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

"Tech preview" removed from the title


hello

Can this warehouse be used in a production environment?
算了，你应该是中国人，这个库能用于生产环境吗

@wellorange sure.

FYI, @wellorange actually I do confirm that it's already in Huawei cloud :)


Undefined offset: 1 error when exception message is empty

An edge case I had with an empty @expectedExceptionMessage annotation
uncovered an unchecked return resulting in the undefined offset.
Fix is to fall-back to empty string which also solves the fixer for
me.

Actually, @expectedExceptionMessage will do nothing, since it only checks if the message is a prefix of the actual message, so the correct implementation should just not check the message at all in the conversion...

Just some more info: If the exception message is set empty to asset for / an empty message is set as expectation Phpunit (in practise at least in one version) is looking for an empty message:
2) Ktomk\Pipelines\Runner\DirectoriesTest::testGetBaseDirectoryThrows
Failed asserting that exception message is empty but is 'Not a base directory: 'XDG_FOO42_HOME'. Known base directories are: "XDG_DATA_HOME", "XDG_CACHE_HOME"'.

Details about the Phpunit version:
PHPUnit 6.5.7 by Sebastian Bergmann and contributors.

Runtime:       PHP 7.2.22-1+ubuntu18.04.1+deb.sury.org+1 with Xdebug 2.7.2

Just for the record.

The discussion between;

$this->setExpectedException(\FooException::class, '');

and

$this->setExpectedException(\FooException::class);

seems semantic to me. This is because '' is the default value for the message (and not null as the PHPDoc incorrectly states).
Changing to:

$this->setExpectedException(\FooException::class, null);

seems more correct as when the annotation @expectedExceptionMessage has no value than the message itself is not checked, like with passing null. However, passing null can be valid, I can't see why someone would omit the value for the annotation.
So fixing to '' seems fine to me, people can update their code to use null if needed or revert the change and remove useless annotation and fix the project again.
Besides all this, it seems setExpectedException is removed from PHPUnit  ....

Thank you @ktomk.

@SpacePossum: You're welcome. I'm glad I could be of help.


WIP: introduce origin system containers for ansible

The job is aligned with test_branch_origin_extended_conformance_install_containerized as much as it can. There are just slight modifications:

install atomic, ostree and runc rpms
atomic pull openshift/origin, openshift/openvswitch and openshift/node into the ostree storage
check the openshift/originwith a proper tag is running by listing containers (viaatomic containers list` command)

The conformance tests failing due to https://github.com/openshift/origin/issues/15210

Do you think you'd be able to split this into two commits - one that first copies the other job into the new file, then a second on top that modifies it?

Sure, I will update the PR once https://github.com/openshift/aos-cd-jobs/pull/441 is merged.

@stevekuznetsov I have separeted the PR into two commits. The first just duplicates the containerized version, the second one alters it so the system containers based deployment is tested. I will open another PR to integrate the new CI into the merge_pull_request_openshift_ansible.yml sjb configuraiton.

@stevekuznetsov so unless there is something else to review, let's merge this. This PR is built on top of https://github.com/openshift/aos-cd-jobs/pull/441. So either let's merge this PR and close the other one. Or merge the other one and rebase this one. Both ways work for me.


Generate randomized links to sign up for instructor training from partner sites

If we're doing instructor training for a partner site, they'll send us names and email addresses of participants. We should send those people a randomized link to the instructor training application page so that:

We can automatically identify which event they're part of.
We can change the wording in the header to make it clear that they're guaranteed a spot in a particular event.

Is this scheduled for v1.6.3 or later?

Later.

Should this be included in v1.7 (in next 4 days) or later?

This one does not need to be in v1.7, but should be in v1.8.

Closing this as we have systems for member sites to sign up for events with a code on Eventbrite.


Types system should be able to handle more complex type definitions

Created out of discussion from #7840
Take a case like keyVault:
resource kv 'Microsoft.KeyVault/vaults@2021-11-01-preview' = {
  ...
  properties: {
    ...
    accessPolicies: [] // required if "enableRbacAuthorization" property is not provided
    enableRbacAuthorization: true // required if "accessPolicies" is not provided
  }
}

Neither accessPolicies nor enableRbacAuthorization are explicitly required, but one of them is required if the other is not included. We need a way to express this complexity in our type system and validate it.

Yes please. oneOf unions are very useful from an API design POV. These are really common in Kubernetes types, so have the capability to validate these in Bicep would be mega-valuable.


Revert "Fix immutable support"

Reverts erikras/redux-form#4471. We can't import from 'immutable' in handleSubmit. We've got to abstract this out into something that gets passed in or lives in structure.
Fixes #4473.

@erikras Can confirm why this is running local with yarn link and in publish it broken? How can we simulate the real publish package?

How can we simulate the real publish package?

Wow. Great question. Seems like the sort of thing someone might have tackled before.

Travis is unhappy, but I'm gonna merge this and fix any problems locally.

The combination of #4470 and #4471 broke the build, but with both reverts merged, it's fine.

Published in v8.2.3.

I'm working on a fix using structure


Debian doesn’t provide apt-transport-https by default

Hi,
On a fresh debian 8 install you will need to install apt-transport-https before adding nodejs repo.

@Tr4sK Merged. Thanks for catching that and contributing.


fix: deprecate images based on 23.04

Changes proposed in this pull request:
Deprecated images based on 23.04.

Ya, sorry about that -- upstream moved on with newer PRs. I rebased the commits. Please take another look.


Revert "2024event"

Reverts Everything-Open-Source/open-source-events#31
@Darkbeast-glitch  can you confirm the changes you made to the events in January 2024? you can check out issue  #33

sure please, on it

There you go i added it 2024event as well as 2023events


HTTP 403 with endpoint /dl/:hash

Hi!
Thanks for the work on this :)
In photoprism/internal/api/download_file.go#L45 requests to download_file need to include a downloadToken, otherwise a status_code: 403 will be returned.
The error will be raised as a KeyError relating to Content-Disposition, but this is actually due to the unhandled 403.

Thanks! I merged your PR#9. I'll leave the issue open until it is packaged and a new version is added to PIP.
We also need to add a test or example for this.


add loss weighting args

@JohnGiorgi can you check this out, works fine on CC running some experiments now

@OsvaldN Closing this unless you are still working on it? If so just re-open.


Weird texture in Jungle section, Mystic Ruins

Weird textures showing up in the Jungle section in Mystic Ruins.

These are the mods I am using (Better SADX Core is just the adx mod plus AVA_TITLE_CMN.pvm and AVA_TITLE_CMN_SMALL.pvm)

I found out a new update for the Modloader was out. It corrected these issues, sorry for pointing it out!

No problem. Otherwise a great quality report!


Draft: Scan text in 64KB chunks

Matches up with https://github.com/ozfive/logrus/pull/1 but this PR is here so CI can run.

Closing as https://github.com/sirupsen/logrus/pull/1376 is the main PR.


Examples do not run on Mac OSX using the M1 chip

Expected Behavior
Examples should run.
Actual Behavior
Examples crash with
diehlpk@Patricks-MacBook-Air build % ./bin/fibonacci

{config}:
Core library:
  HPX_AGAS_LOCAL_CACHE_SIZE=4096
  HPX_HAVE_MALLOC=system
  HPX_PARCEL_MAX_CONNECTIONS=512
  HPX_PARCEL_MAX_CONNECTIONS_PER_LOCALITY=4
  HPX_PREFIX (configured)=
  HPX_PREFIX=

  HPX_FILESYSTEM_WITH_BOOST_FILESYSTEM_COMPATIBILITY=OFF
  HPX_ITERATOR_SUPPORT_WITH_BOOST_ITERATOR_TRAVERSAL_TAG_COMPATIBILITY=OFF
  HPX_PROGRAM_OPTIONS_WITH_BOOST_PROGRAM_OPTIONS_COMPATIBILITY=OFF
  HPX_WITH_AGAS_DUMP_REFCNT_ENTRIES=OFF
  HPX_WITH_APEX=OFF
  HPX_WITH_ASYNC_MPI=OFF
  HPX_WITH_ATTACH_DEBUGGER_ON_TEST_FAILURE=OFF
  HPX_WITH_AUTOMATIC_SERIALIZATION_REGISTRATION=ON
  HPX_WITH_COROUTINE_COUNTERS=OFF
  HPX_WITH_DISTRIBUTED_RUNTIME=ON
  HPX_WITH_DYNAMIC_HPX_MAIN=ON
  HPX_WITH_GENERIC_CONTEXT_COROUTINES=ON
  HPX_WITH_GOOGLE_PERFTOOLS=OFF
  HPX_WITH_INIT_START_OVERLOADS_COMPATIBILITY=ON
  HPX_WITH_IO_POOL=ON
  HPX_WITH_ITTNOTIFY=OFF
  HPX_WITH_LOGGING=ON
  HPX_WITH_NETWORKING=ON
  HPX_WITH_PAPI=OFF
  HPX_WITH_PARALLEL_TESTS_BIND_NONE=OFF
  HPX_WITH_PARCELPORT_ACTION_COUNTERS=OFF
  HPX_WITH_PARCELPORT_LIBFABRIC=OFF
  HPX_WITH_PARCELPORT_MPI=OFF
  HPX_WITH_PARCELPORT_TCP=ON
  HPX_WITH_PARCEL_COALESCING=ON
  HPX_WITH_PARCEL_PROFILING=OFF
  HPX_WITH_REGISTER_THREAD_OVERLOADS_COMPATIBILITY=ON
  HPX_WITH_SANITIZERS=OFF
  HPX_WITH_SCHEDULER_LOCAL_STORAGE=OFF
  HPX_WITH_SPINLOCK_DEADLOCK_DETECTION=OFF
  HPX_WITH_STACKTRACES=ON
  HPX_WITH_TESTS_DEBUG_LOG=OFF
  HPX_WITH_THREAD_BACKTRACE_ON_SUSPENSION=OFF
  HPX_WITH_THREAD_CREATION_AND_CLEANUP_RATES=OFF
  HPX_WITH_THREAD_CUMULATIVE_COUNTS=ON
  HPX_WITH_THREAD_DEBUG_INFO=OFF
  HPX_WITH_THREAD_DESCRIPTION_FULL=OFF
  HPX_WITH_THREAD_GUARD_PAGE=ON
  HPX_WITH_THREAD_IDLE_RATES=OFF
  HPX_WITH_THREAD_LOCAL_STORAGE=OFF
  HPX_WITH_THREAD_MANAGER_IDLE_BACKOFF=ON
  HPX_WITH_THREAD_QUEUE_WAITTIME=OFF
  HPX_WITH_THREAD_STACK_MMAP=ON
  HPX_WITH_THREAD_STEALING_COUNTS=OFF
  HPX_WITH_THREAD_TARGET_ADDRESS=OFF
  HPX_WITH_TIMER_POOL=ON
  HPX_WITH_TUPLE_RVALUE_SWAP=ON
  HPX_WITH_VALGRIND=OFF
  HPX_WITH_VERIFY_LOCKS=OFF
  HPX_WITH_VERIFY_LOCKS_BACKTRACE=OFF
  HPX_WITH_VERIFY_LOCKS_GLOBALLY=OFF

Module coroutines:
  HPX_COROUTINES_WITH_SWAP_CONTEXT_EMULATION=OFF

Module datastructures:
  HPX_DATASTRUCTURES_WITH_ADAPT_STD_TUPLE=ON

Module serialization:
  HPX_SERIALIZATION_WITH_BOOST_TYPES=ON

Module topology:
  HPX_TOPOLOGY_WITH_ADDITIONAL_HWLOC_TESTING=OFF

{version}: V1.7.0-trunk (AGAS: V3.0), Git: bb52868611
{boost}: V1.75.0
{build-type}: release
{date}: May 17 2021 15:56:17
{platform}: Mac OS
{compiler}: Clang version 12.0.5 (clang-1<IP_ADDRESS>)
{stdlib}: libc++ version 12000
{os-thread}: worker-thread#7
{what}: mmap() failed to allocate thread stack: HPX(unhandled_exception)

Steps to Reproduce the Problem
... Please be as specific as possible while describing how to reproduce your problem.

Compile HPX master using #5339
Run any example

Specifications
... Please describe your environment

HPX Version: Master with #5339
Platform (compiler, OS): Big Sur, M1 chip

I just tested with boost 1.78 and hpx master and still not working.

@hkaiser Is there any way I could reproduce that either on ROSTAM or at CCT somehow? Otherwise I could link it to this GSoC project.

@hkaiser Is there any way I could reproduce that either on ROSTAM or at CCT somehow? Otherwise I could link it to this GSoC project.

Unfortunately, we don't have access to an environment that would allow to reproduce the issue.

Looks like this was fixed in https://github.com/STEllAR-GROUP/hpx/pull/5928.


6.0 Feedback:  Pipette offset calibration recommended but I just completed it

Overview

Current Behavior
No response
Expected Behavior
No response
Steps To Reproduce
No response
Operating system
No response
Robot setup or anything else?
No response

@y3rsh It checked pipetteOffsetCalibrations status markedBad (like below).
Could you check your data? If your pipette offset calibrations data, markedBad is false, it's a bug but if not, it's calibration issue.
{
		"id": "P1KSV222021011802&left",
		"pipette": "P1KSV222021011802",
		"mount": "left",
		"offset": [-1.7763568394002505e-15, 0, -5],
		"tiprack": "f979b7278ee3e6c4a13a6336cc6a602688f916ab2962de49263a1af5b513a533",
		"tiprackUri": "opentrons/opentrons_96_tiprack_1000ul/1",
		"lastModified": "2022-03-30T00:10:01.533919+00:00",
		"source": "user",
		"status": {
			"markedBad": false,
			"source": null,
			"markedAt": null
		}
	}

@koji , I am getting markedBad as false, and the banner still persists
{    "data":[       {          "id":"P3HMV212020111108&left",          "pipette":"P3HMV212020111108",          "mount":"left",          "offset":[             -2.0000000000000018,             6.0,             -2.0          ],          "tiprack":"ed323db6ca1ddf197aeb20667c1a7a91c89cfb2f931f45079d483928da056812",          "tiprackUri":"opentrons/opentrons_96_tiprack_300ul/1",          "lastModified":"2022-05-26T21:51:35.419695+00:00",          "source":"user",          "status":{             "markedBad":false,             "source":null,             "markedAt":null          }       },       {          "id":"P20MV212020102303&right",          "pipette":"P20MV212020102303",          "mount":"right",          "offset":[             2.0,             0.0,             -0.3999999999999986          ],          "tiprack":"60e107ba7e6e5d23beefe9f0c888f860f7b923c71906aaa5e7f460ab25c4f38a",          "tiprackUri":"opentrons/opentrons_96_tiprack_20ul/1",          "lastModified":"2022-05-26T21:57:45.125253+00:00",          "source":"user",          "status":{             "markedBad":false,             "source":null,             "markedAt":null          }       }    ],    "links":null }

@reachojhaneha thanks, I will dig into this.

verified closed.


fix: memory issues in counting_sort.cpp

Description of Change

There are several problems with the current implementation of counting_sort.cpp:

memory allocated by creating *Count is uninitialized and not deleted,
https://github.com/TheAlgorithms/C-Plus-Plus/blob/23b133ae1eac4ad7e94e74da140d462180738413/sorting/counting_sort.cpp#L29
memory allocated by creating *Sorted_Arr is not deleted.
https://github.com/TheAlgorithms/C-Plus-Plus/blob/23b133ae1eac4ad7e94e74da140d462180738413/sorting/counting_sort.cpp#L27

This PR fixes that.
Checklist

[x] Added description of change
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.

Notes:
This PR fixes memory issues in counting_sort.cpp.

@Panquesito7, @realstealthninja: please dont-close.


Improve Mouse/Keyboard Camera Control

Current control cannot translate the origin effectively and takes multiple clicks and drags to accomplish large rotations.
Suggest using more standard WASD + mouse control commonly found in first-person video games.

This has been committed to master.


No coverage available for IIS processes

Description
I have a .NETCore 3.1 test project with  Microsoft.NET.Test.Sdk 16.7.1 that is used Azure pipeline.
The agent has Visual Studio 2017 VSTest.
I am getting an exception when running tests using VSTest on the agent machine. We are interested in the coverage of integration tests - we have an API that is hosted in IIS -  hence the configurations <CollectAspDotNet>True</CollectAspDotNet>						<CollectFromChildProcesses>True</CollectFromChildProcesses>
Steps to reproduce
.\vstest.console.exe  /Settings:C:\Projects\CodeCoverage.runsettings  C:\Projects\test\Web.Tests\bin\Debug\Web.Tests.dll --Diag:"C:\Projects\diag.txt"
CodeCoverage.runsettings has the following settings (I've excluded the ModulePaths and Attributes for simplicity).

<?xml version="1.0" encoding="utf-8"?>
<!-- File name extension must be .runsettings -->
<RunSettings>
	<DataCollectionRunSettings>
		<DataCollectors>
			<DataCollector friendlyName="Code Coverage" uri="datacollector://Microsoft/CodeCoverage/2.0" assemblyQualifiedName="Microsoft.VisualStudio.Coverage.DynamicCoverageDataCollector, Microsoft.VisualStudio.TraceCollector, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a">
				<Configuration>
					<CodeCoverage>
						<UseVerifiableInstrumentation>True</UseVerifiableInstrumentation>
						<AllowLowIntegrityProcesses>True</AllowLowIntegrityProcesses>
						<CollectFromChildProcesses>True</CollectFromChildProcesses>
						<CollectAspDotNet>True</CollectAspDotNet>
					</CodeCoverage>
				</Configuration>
			</DataCollector>
		</DataCollectors>
	</DataCollectionRunSettings>
</RunSettings>

Actual behavior
I get the following errors in the diag.txt file:
TpTrace Warning: 0 : 7512, 1, 2020/11/17, 20:30:45.661,<PHONE_NUMBER>6, datacollector.exe, LoaderExceptions: System.IO.FileNotFoundException: Could not load file or assembly 'System.Diagnostics.DiagnosticSource, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51' or one of its dependencies. The system cannot find the file specified.
File name: 'System.Diagnostics.DiagnosticSource, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'
   at System.Reflection.RuntimeAssembly._nLoad(AssemblyName fileName, String codeBase, Evidence assemblySecurity, RuntimeAssembly locationHint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
   at System.Reflection.RuntimeAssembly.InternalLoadAssemblyName(AssemblyName assemblyRef, Evidence assemblySecurity, RuntimeAssembly reqAssembly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
   at System.Reflection.Assembly.Load(AssemblyName assemblyRef)
   at Microsoft.VisualStudio.TestPlatform.Common.ExtensionFramework.TestPluginCache.CurrentDomainAssemblyResolve(Object sender, AssemblyResolveEventArgs args)
   at System.AppDomain.OnAssemblyResolveEvent(RuntimeAssembly assembly, String assemblyFullName)
TpTrace Error: 0 : 7512, 1, 2020/11/17, 20:30:45.679,<PHONE_NUMBER>2, datacollector.exe, The type "Microsoft.VisualStudio.Coverage.DynamicCoverageDataCollector" defined in "Microsoft.VisualStudio.TraceDataCollector.dll" does not have ExtensionUri attribute.
TpTrace Error: 0 : 7512, 1, 2020/11/17, 20:30:45.679,<PHONE_NUMBER>8, datacollector.exe, The type "Microsoft.VisualStudio.TraceCollector.TestImpactDataCollector" defined in "Microsoft.VisualStudio.TraceDataCollector.dll" does not have ExtensionUri attribute.
TpTrace Error: 0 : 7512, 1, 2020/11/17, 20:30:45.679,<PHONE_NUMBER>5, datacollector.exe, The type "Microsoft.VisualStudio.TraceCollector.UnitTestIsolationDataCollector" defined in "Microsoft.VisualStudio.TraceDataCollector.dll" does not have ExtensionUri attribute.

TpTrace Information: 0 : 7512, 8, 2020/11/17, 20:30:52.299,<PHONE_NUMBER>1, datacollector.exe, ProcessDumpUtility.OutputReceivedCallback: Output received from procdump process: [20:30:52] Exception: E06D7363.PEAVEEFileLoadException@@
TpTrace Information: 0 : 7512, 8, 2020/11/17, 20:30:52.299,<PHONE_NUMBER>1, datacollector.exe, ProcessDumpUtility.OutputReceivedCallback: Output received from procdump process: [20:30:52] Exception: E0434352.CLR
TpTrace Information: 0 : 7512, 8, 2020/11/17, 20:30:52.346,<PHONE_NUMBER>7, datacollector.exe, ProcessDumpUtility.OutputReceivedCallback: Output received from procdump process: [20:30:52] Exception: E0434352.CLR
TpTrace Information: 0 : 7512, 8, 2020/11/17, 20:30:52.398,<PHONE_NUMBER>8, datacollector.exe, ProcessDumpUtility.OutputReceivedCallback: Output received from procdump process: [20:30:52] Exception: E0434352.CLR
TpTrace Information: 0 : 7512, 8, 2020/11/17, 20:30:52.406,<PHONE_NUMBER>7, datacollector.exe, ProcessDumpUtility.OutputReceivedCallback: Output received from procdump process: [20:30:52] Exception: E0434352.CLR
TpTrace Information: 0 : 7512, 8, 2020/11/17, 20:30:52.415,<PHONE_NUMBER>0, datacollector.exe, ProcessDumpUtility.OutputReceivedCallback: Output received from procdump process: [20:30:52] Exception: E0434352.CLR
TpTrace Information: 0 : 7512, 8, 2020/11/17, 20:30:52.426,<PHONE_NUMBER>5, datacollector.exe, ProcessDumpUtility.OutputReceivedCallback: Output received from procdump process: [20:30:52] Exception: E0434352.CLR
TpTrace Information: 0 : 7512, 8, 2020/11/17, 20:30:52.427,<PHONE_NUMBER>4, datacollector.exe, ProcessDumpUtility.OutputReceivedCallback: Output received from procdump process: [20:30:52] Exception: E0434352.CLR
TpTrace Information: 0 : 7512, 8, 2020/11/17, 20:30:52.522,<PHONE_NUMBER>5, datacollector.exe, ProcessDumpUtility.OutputReceivedCallback: Output received from procdump process: [20:30:52] Exception: E0434352.CLR
TpTrace Information: 0 : 7512, 8, 2020/11/17, 20:30:52.619,<PHONE_NUMBER>0, datacollector.exe, ProcessDumpUtility.OutputReceivedCallback: Output received from procdump process: [20:30:52] Exception: E0434352.CLR
TpTrace Information: 0 : 7512, 8, 2020/11/17, 20:30:53.100,<PHONE_NUMBER>4, datacollector.exe, ProcessDumpUtility.OutputReceivedCallback: Output received from procdump process: [20:30:53] Exception: E0434352.CLR
TpTrace Information: 0 : 7512, 6, 2020/11/17, 20:30:54.397,<PHONE_NUMBER>1, datacollector.exe, DataCollectionTestCaseEventHandler: Test case starting.
TpTrace Information: 0 : 7512, 6, 2020/11/17, 20:30:54.438,<PHONE_NUMBER>1, datacollector.exe, Blame Collector : Test Case Start
TpTrace Information: 0 : 7512, 6, 2020/11/17, 20:30:54.439,<PHONE_NUMBER>3, datacollector.exe, DataCollectionTestCaseEventHandler: Test case started.
TpTrace Information: 0 : 7512, 8, 2020/11/17, 20:31:29.696,<PHONE_NUMBER>8, datacollector.exe, ProcessDumpUtility.OutputReceivedCallback: Output received from procdump process: [20:31:29] Exception: E0434352.CLR
TpTrace Information: 0 : 7512, 8, 2020/11/17, 20:31:29.818,<PHONE_NUMBER>9, datacollector.exe, ProcessDumpUtility.OutputReceivedCallback: Output received from procdump process: [20:31:29] Exception: E0434352.CLR
TpTrace Information: 0 : 7512, 8, 2020/11/17, 20:31:29.821,<PHONE_NUMBER>6, datacollector.exe, ProcessDumpUtility.OutputReceivedCallback: Output received from procdump process: [20:31:29] Exception: E0434352.CLR
TpTrace Information: 0 : 7512, 6, 2020/11/17, 20:31:37.168,<PHONE_NUMBER>6, datacollector.exe, DataCollectionTestCaseEventHandler : Test case completing.
TpTrace Information: 0 : 7512, 6, 2020/11/17, 20:31:37.172,<PHONE_NUMBER>6, datacollector.exe, Blame Collector : Test Case End
TpTrace Error: 0 : 7512, 6, 2020/11/17, 20:31:42.871,<PHONE_NUMBER>8, datacollector.exe, DataCollectionAttachmentManager.GetAttachments: Fail to get attachments: System.Collections.Generic.KeyNotFoundException: The given key was not present in the dictionary.
   at System.ThrowHelper.ThrowKeyNotFoundException()
   at System.Collections.Generic.Dictionary`2.get_Item(TKey key)
   at Microsoft.VisualStudio.TestPlatform.Common.DataCollector.DataCollectionAttachmentManager.GetAttachments(DataCollectionContext dataCollectionContext)

An empty coverage file is generated.
Not sure where to look next, so any help is appreciated.
Thanks.

After some further investigations I noticed that I can collect coverage if I run manually the powershell commands.
Before test run: CodeCoverage.exe collect /IIS /session:WebTests /output:webtests.coverage   .The output shows that the IIS process is restarted and that all the dll are instrumented properly.
After test run: CodeCoverage.exe shutdown /session:WebTests
I didn't see any documentation if this is necessary or not. I thought that CollectAspDotNet restarts the IIS process and instruments the code but I looked over the https://github.com/microsoft/vstest/blob/2d58f0c2d69a99414e52b114d422559bbb9caa58/src/DataCollectors/TraceDataCollector/VanguardCollector/Vanguard.cs and I didn't see how the IIS parameter is passed to CodeCoverage tool.

It seems this approach is not working. My app - which is a .NET Core 3.1 app - (hosted in IIS) fails to load because of the following error

Application '/LM/W3SVC/2/ROOT' with physical root 'C:\_temp\' hit unexpected managed exception, exception code = '0xe0434352'. First 30KB characters of captured stdout and stderr logs:
Unhandled exception. System.MissingMethodException: Method not found: 'Void Microsoft.VisualStudio.CodeCoverage.Shim.4001414003__coverage_runtime_method__33525_18195__(UInt64)'.
we\_work\1\s\src\Web\Web.Host\Startup\Program.cs:line 52
   at Web.Startup.Program.Main(String[] args) in C:\or-bld-ag-5bf15a79-we\_work\1\s\src\Web\Web.Host\Startup\Program.cs:line 24
   at Web.Startup.Program.<Main>(String[] args)

In the output of test project I have the Microsoft.VisualStudio.CodeCoverage.Shim.dll.
To be honest I don't know where to look next.
Any hint is highly appreciated.

Please try using unverifiable probes. More info: https://github.com/microsoft/vstest/issues/2459#issuecomment-768254976

Please try using unverifiable probes. More info: https://github.com/microsoft/vstest/issues/2459#issuecomment-768254976

After some further investigations I noticed that I can collect coverage if I run manually the powershell commands.
Before test run: CodeCoverage.exe collect /IIS /session:WebTests /output:webtests.coverage  .The output shows that the IIS process is restarted and that all the dll are instrumented properly.
After test run: CodeCoverage.exe shutdown /session:WebTests
I didn't see any documentation if this is necessary or not. I thought that CollectAspDotNet restarts the IIS process and instruments the code but I looked over the https://github.com/microsoft/vstest/blob/2d58f0c2d69a99414e52b114d422559bbb9caa58/src/DataCollectors/TraceDataCollector/VanguardCollector/Vanguard.cs and I didn't see how the IIS parameter is passed to CodeCoverage tool.

@arabelaa Can you try running vstest.console.exe with admin rights? Or Visual Studio with admin rights? And see if you need to do this additional step or not.


"Core API" javadoc

The JavaDoc documentation of ND4J currently lists 1122 classes in 261 packages (https://nd4j.org/doc/) and it seems to describe all the classes of the project, not just the public API. Furthermore it seems to describe many extensions that might not be part of the core library. Examples include classes such as AggregatesTests, AssignImplementationTest, BaseBenchmarkPerformer or the packages org.nd4j.bytebuddy, org.nd4j.jdbc, etc.
Many Java projects, including the JDK, have separate "public API" JavaDocs that do not show the implementation classes. As a newcomer to ND4J it would be nice to be able to access an JavaDoc index that only listed the API classes and nothing else. There could even be a top-level a package-info.html that had a curated list of links to the entry points of main concepts, starting from the classes Nd4j, INDArray and Transforms. Also there could be a JavaDoc description for each package. Currently none of the packages in ND4J have a description.
Looking at the examples (https://github.com/deeplearning4j/dl4j-examples/tree/master/nd4j-examples/src/main/java/org/nd4j/examples), they seem to refer only to a handful of classes in the following packages:
import org.nd4j.linalg.api.buffer
import org.nd4j.linalg.api.memory
import org.nd4j.linalg.api.ndarray
import org.nd4j.linalg.api.ops.impl
import org.nd4j.linalg.dataset
import org.nd4j.linalg.factory
import org.nd4j.linalg.indexing
import org.nd4j.linalg.ops.transforms

I think one issue here is that a lot of the classes are public when they don't need to be.

Also these actions would help:

generate a javadoc that only includes the sub-projects that are considered part of the core API (i.e. exclude sub-projects that are implementation internals or less-used extensions)
do not generate javadoc for test directories (at the moment these are included, which is highly irregular)

This is fair. Thanks for the feedback! I think in general most people really only need the INDArray class and Nd4j.java - most things aren't really relevant beyond maybe the operations till you want to be more advanced. We will look a bit in to this.


Allow GenerateOperationIds method in SwaggerDocument to be overridden

The current implementation of SwaggerDocument.GenerateOperationIds does a poor job of naming operations that are defined by the OData spec. Ideally there should be a way of providing a custom naming method with a more appropriate implementation.
(NB: this method already has a comment saying TODO: Improve this method so this is a known problem.
I imagine that the easiest way of achieving would be to make the method virtual so it can be overridden in a derived class.

Accodring to https://swagger.io/specification/:

Unique string used to identify the operation. The id MUST be unique among all operations described in the API.

So it must be set and must be optional. So in theory, the GenerateOperationIds() method is not even needed... But I agree, we should improve the method. Do you have a replacement implementation? For me, it would be fine, to replace the current implementation (maybe leave the old implementation to use with a legacy setting).

I think we should remove this method and create a new class OperationIdGenerator and a new setting OperationIdGenerator to provide an own implementation...


Remove vpa for vpa-recommender on seeds. Avoids VPA self-scaling…

… on soils.
How to categorize this PR?
/area auto-scaling
/kind bug
What this PR does / why we need it:
No longer deploy a vpa resource for seed recommender, since that would cause the recommender to act on said vpa resource, and attempt to autoscale its own deployment. Self-scaling is not supported by VPA. In place of VPA, use higher fixed request values for the garden/vpa-recommender deployment.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

A the seed vpa-recommender is no longer scaled by VPA. Instead, fixed resource request values are used.

Self-scaling is not supported by VPA.

Can you elaborate on the background? Why is this not supported/how is the vpa-recommender special to any other workload in a Kubernetes cluster when it comes to autoscaling with VPA?

Self-scaling is not supported by VPA.

Can you elaborate on the background? Why is this not supported/how is the vpa-recommender special to any other workload in a Kubernetes cluster when it comes to autoscaling with VPA?

On the Soil we end up with a configuration where the VPA manages itself. A VPA can for sure manage a different VPA like any other workload. However, when the vpa-updater evicts the vpa-recommender and the vpa-recommender should compute a recommendation for itself – this didn't turn out to work very well.

the vpa-recommender should compute a recommendation for itself – this didn't turn out to work very well.

OK, but still, what is not working "very well" here? I'm just trying to get a better understanding about how VPA works and which pitfalls exist, and this seems to be one that is not very obvious (at least to me).

The vpa-updater evicted the vpa-recommender every few minutes because the recommendations changed enough to allow for an eviction. The effect can be seen e.g. here: a restarting recommender uses much less memory than a recommender that is already running for a while.

The presumed causal relations are:

In the absence of checkpoint data, VPA acts on a single data point. p50=p90=p95=SingleDataPoint.
With sample count >1 million, VPA prescribes LowerBound~=Target~=UpperBound
Next VPA loop iteration practically always is outside the Lower-Upper bound range, causing eviction
Memory histogram entries are never added, due to short pod lifetime

The last point is an assumption, the specific VPA execution mechanics yet to be understood.

/assign


Search Score Decreases with More Accurate Query

The following are steps to reproduce an issue I am experiencing. First, create an index and add a two-word String:
var index = lunr(function () {
  this.ref('ref')
  this.field('text')
})
index.add({
  ref: 1,
  text: 'yes funny'
})

This first query is a portion of the first word followed by the complete second word:
console.log(index.search("ye funny"))
[ { ref: '1', score: 0.773957299203321 } ]

This second query also begins with a portion of the first word (can be identical to the term in the previous query, or not) followed by only a portion of the second word:
console.log(index.search("ye fun"))
[ { ref: '1', score: 1 } ]

Issue: Why does the second, less accurate query return a higher score than the first, more accurate query?
Note: For this particular example, the issue occurs only when the stemmer is disabled. See my comment below for better examples.
Thank you very much.

I have the exact same problem.
Unfortunately query terms which are even an exact match are rated lower that partial matches.
E.g.: you have a data set off [foo, foobar] and if you search it for foo lunr will return [foobar, foo]


Sometime the list of REST api is missing some functions

I wrote a service name user that published some rest api like /api/user/register , /api/user/login.
I tried to restart moleculer many times, some time, I got all services correct like this: https://imgur.com/5bMGNpI
But sometime, it miss the /user API like this: https://imgur.com/0QrYTra
What's going on here?

It's my mistake


How is the variance for each class calculated

Although discussed in #2, I am still confused about how variance is computed. In the paper, it says "the variance is related to the object class and scale information", while in the code, sigma is calculated by
https://github.com/yizhou-wang/RODNet/blob/969cad6f08b8957b26bc16f86ac4e835d1294050/rodnet/core/confidence_map/generate.py#L36
As I understand, sigma is the class-specific variance, and confmap_sigma is a hyper-parameter for variance per class. My question is:

Is there a physical meaning or intuition behind np.arctan(confmap_length[class_name] / (2 * range_grid[rng_idx]))
Is confmap_sigmas, confmap_sigmas_interval and confmap_length all hyper-parameters choosed via experiments?

Thanks in advance!

The intuition behind this formula is that sigma is approximating the reflection surface size at different distances/scales. This formula as well as the hyper-parameters you mentioned above is empirically defined.

Thanks!


[Feature Request] : Adding Kadane's Algorithm for Maximum Subarray Sum in an Array |   isuue no.  #115

📥 Pull Request
Description
i have added Kadane's Algorithm
Fixes #115
Type of change

New feature (non-breaking change which adds functionality)

Checklist:

My code follows the style guidelines of this project
I have performed a self-review of my code
I have commented my code, particularly in hard-to-understand areas
I have added tests that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes
Any dependent changes have been merged and published in downstream modules

please review.


Sizes based on largest label/image

I noticed that the control was placing each image/label and then choosing the size of the selection based on a the quotient of number of views and control width.
Instead I figured what the largest size image/label is and base the creation off of that instead.

@jacobvanorder I'm a bit dense: Is the issue here that the selection doesn't work if you have one long and one short string? The design of the original application this is based on assumes that each segment is equally sized.

@jacobvanorder Screenshot of the original crispy flavor:


Oo/oslynn/14247/fix rest transport for ok phd

This PR fixed REST Transport to make it work with OK-PHD.
Test Steps:

clone the PR
./gradlew clean package
./gradlew testIntegration
It should SUCCESS

Changes

Introduced the BearerToken into the Transport message header

Checklist
Testing

[x] Tested locally?
[x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[ ] (For Changes to /frontend-react/...) Ran npm run lint:write?
[ ] Added tests?

Process

[ ] Are there licensing issues with any new dependencies introduced?
[ ] Includes a summary of what a code reviewer should test/verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Linked Issues

Fixes #issue
https://app.zenhub.com/workspaces/engagement-6166edbd409257001e09f1b2/issues/gh/cdcgov/prime-reportstream/14247

To Be Done
Create GitHub issues to track the work remaining, if any

#issue

Specific Security-related subjects a reviewer should pay specific attention to

Does this PR introduce new endpoints?

new endpoint A
new endpoint B

Does this PR include changes in authentication and/or authorization of existing endpoints?
Does this change introduce new dependencies that need vetting?
Does this change require changes to our infrastructure?
Does logging contain sensitive data?
Does this PR include or remove any sensitive information itself?

If you answered 'yes' to any of the questions above, conduct a detailed Review that addresses at least:

What are the potential security threats and mitigations? Please list the STRIDE threats and how they are mitigated

Spoofing (faking authenticity)

Threat T, which could be achieved by A, is mitigated by M

Tampering (influence or sabotage the integrity of information, data, or system)
Repudiation (the ability to dispute the origin or originator of an action)
Information disclosure (data made available to entities who should not have it)
Denial of service (make a resource unavailable)
Elevation of Privilege (reduce restrictions that apply or gain privileges one should not have)

Have you ensured logging does not contain sensitive data?
Have you received any additional approvals needed for this change?

Resent with fixed REST Transport:
(base) judith@A-Ubuntu20Dell7810-DUAL-CPUs:~/workspace/cdc/working/prime-reportstream/prime-router$ curl -X POST -H 'client: simple_report' -H 'Content-Type: text/csv' --data-binary<EMAIL_ADDRESS>http://localhost:7071/api/reports
{
"id" : "516ae6e8-951e-4748-b60d-c9c846d72b7e",
"submissionId" : 213,
"overallStatus" : "Waiting to Deliver",
"timestamp" : "2024-05-01T20:45:15.523Z",
"plannedCompletionAt" : "2024-05-01T20:46:00.000Z",
"actualCompletionAt" : null,
"sender" : "simple_report.default",
"reportItemCount" : 1,
"errorCount" : 0,
"warningCount" : 2,
"httpStatus" : 201,
"destinations" : [ {
"organization" : "Oklahoma Public Health Department",
"organization_id" : "ok-phd",
"service" : "elr",
"itemCount" : 1,
"itemCountBeforeQualityFiltering" : 1,
"sending_at" : "2024-05-01T20:46:00.000Z",
"filteredReportRows" : [ ],
"filteredReportItems" : [ ],
"sentReports" : [ ],
"downloadedReports" : [ ]
} ],
"actionName" : "receive",
"externalName" : null,
"reportId" : "516ae6e8-951e-4748-b60d-c9c846d72b7e",
"topic" : "covid-19",
"bodyFormat" : "",
"errors" : [ ],
"warnings" : [ {
"scope" : "item",
"indices" : [ 1 ],
"trackingIds" : [ "3282519a-e6d3-426c-9b2e-6472113a6bde" ],
"field" : "Equipment_Model_ID (equipment_model_id)",
"message" : "No match found for Equipment_Model_ID (equipment_model_id); please refer to the CDC LIVD table LOINC Mapping spreadsheet for acceptable values.",
"errorCode" : "INVALID_MSG_EQUIPMENT_MAPPING"
}, {
"scope" : "item",
"indices" : [ 1 ],
"trackingIds" : [ "3282519a-e6d3-426c-9b2e-6472113a6bde" ],
"field" : "Patient_preferred_language (patient_preferred_language)",
"message" : "The code '' for field Patient_preferred_language (patient_preferred_language) is invalid. Reformat to HL7 specification.",
"errorCode" : "INVALID_MSG_PARSE_CODE"
} ],
"destinationCount" : 1,
"fileName" : ""
}
From localhost docker log:
[2024-05-01T20:47:00.003Z] Executing 'Functions.batchDecider' (Reason='Timer fired at 2024-05-01T20:47:00.0028189+00:00', Id=d75a4ae4-6f0f-41e0-bad6-cddfd8aff212)
2024-05-01T20:47:00.062210168Z [2024-05-01T20:47:00.061Z] Function "batchDecider" (Id: d75a4ae-6f0f-41e0-bad6-cddfd8aff212) invoked by Java Worker
2024-05-01T20:47:00.062279507Z [2024-05-01T20:47:00.061Z] Executed 'Functions.batchDecider' (Succeeded, Id=d75a4ae4-6f0f-41e0-bad6-cddfd8aff212, Duration=59ms)
2024-05-01T20:47:05.523323588Z [2024-05-01T20:47:05.523Z] HttpClient: RESPONSE: 200 OK
2024-05-01T20:47:05.523359673Z [2024-05-01T20:47:05.523Z] METHOD: HttpMethod(value=POST)
2024-05-01T20:47:05.523367680Z [2024-05-01T20:47:05.523Z] FROM: https://labupload.health.ok.gov/api/document/hl7
2024-05-01T20:47:05.523433299Z [2024-05-01T20:47:05.523Z] COMMON HEADERS
2024-05-01T20:47:05.523483454Z [2024-05-01T20:47:05.523Z] -> Access-Control-Allow-Origin: *
2024-05-01T20:47:05.523545811Z [2024-05-01T20:47:05.523Z] -> Content-Length: 217
2024-05-01T20:47:05.523601854Z [2024-05-01T20:47:05.523Z] -> Content-Type: application/json
2024-05-01T20:47:05.523689694Z [2024-05-01T20:47:05.523Z] -> Date: Wed, 01 May 2024 20:47:05 GMT
2024-05-01T20:47:05.523742598Z [2024-05-01T20:47:05.523Z] -> Server: gunicorn/19.9.0
2024-05-01T20:47:05.523765999Z [2024-05-01T20:47:05.523Z] -> Vary: Accept-Encoding
2024-05-01T20:47:05.523782643Z [2024-05-01T20:47:05.523Z] -> Via: 1.1 google
2024-05-01T20:47:05.523867433Z [2024-05-01T20:47:05.523Z] BODY Content-Type: application/json
2024-05-01T20:47:05.523906630Z [2024-05-01T20:47:05.523Z] BODY START
2024-05-01T20:47:05.523944807Z [2024-05-01T20:47:05.523Z] {"fileName":"CDC_PRIME_REPORTSTREAM_2024-05-01_15:46:05.013414-05:00.hl7","message":"success","status":"File CDC_PRIME_REPORTSTREAM_2024-05-01_15:46:05.013414-05:00.hl7 received successfully. Processing in progress"}
2024-05-01T20:47:05.524006638Z [2024-05-01T20:47:05.523Z] BODY END
2024-05-01T20:47:05.538319428Z [2024-05-01T20:47:05.538Z] {"message":"Starting uploadBlob of none/ok-phd.elr/ok-phd-fb89e58c-1943-4033-9d0c-75e83beb283e-20240501204601.hl7","thread":"pool-2-thread-9","timestamp":"2024-05-01T20:47:05.537Z","level":"INFO","logger":"gov.cdc.prime.router.azure.BlobAccess"}
2024-05-01T20:47:05.538467270Z [2024-05-01T20:47:05.538Z] Message successfully sent!


Feat/test nightly build

added github action that will try to build the application everyday using Rust nightly channel

Shouldn't we move this in the SDK and try to build a sample app ?

Sample app is not yet available at SDK side and I think that it is perfectly relevant to test new nightly with the boilerplate as it will be used by all other external developers when starting a new app. I think that both building and testing shall be performed so I will update the CI script accordingly.

What are the developers supposed to do with this workflow ? Delete it from their app's repo ? It is not something that will be required for deployment...

Sample app is not yet available at SDK side and I think that it is perfectly relevant to test new nightly with the boilerplate as it will be used by all other external developers when starting a new app. I think that both building and testing shall be performed so I will update the CI script accordingly.

What are the developers supposed to do with this workflow ? Delete it from their app's repo ? It is not something that will be required for deployment... @yogh333

Ok missed the point that the same workflow could be triggered from SDK repo. Will move it and update it accordingly.


Refactoring maps code

The MapsController has some code which can be optimized to work more effectively. The complex if statement can be simplified and the gmaps code moved to a helper

This has been pushed live in production and is working successively


example: this.outputPath --> outputPath

I was adding html-minifier transform to my starter, and I noticed that the docs say to use this.inputPath and this.outputPath. However, this.outputPath is undefined, so I assume this should only be used for the inputPath?

Thanks. I'm using 1.x.x. And yes, the caveats with this and arrow functions apply. I tested with a "regular" function declaration, and while I was getting this.inputPath, this.outputPath didn't exist, and I had to use the outputPath from the transform callback function, which has outputPath as the second argument. Is this a bug, or the intended behavior?

Per the docs, I’d say it’s a bug if it’s happening for you in a fresh 1.0.0 install. But curious that I couldn’t repro in a new project using 1.0.0 on macOS 12.2.

Per the docs, I’d say it’s a bug if it’s happening for you in a fresh 1.0.0 install. But curious that I couldn’t repro in a new project using 1.0.0 on macOS 12.2.

Ok, this is really weird. I just tried it again and it's working as expect (this includes both inputPath and outputPath). So I'm going to close this.


add onResizeEvent handler

This adds a handler for the resize event emitted by the window when it's resized. At the moment, all it does is adjust the canvas to match the current innerWidth and innerHeight of the window.
There's a couple of strange side-effects that happen with this - resizing vertically causes the pixel drawing to have a different border, rather than resizing the pixels, and resizing horizontally causes the pixels to break into pieces. Might be the result of an earlier calculation that needs to be redone after resizing the canvas.
Thoughts?

Oh, also adds a generic nodejs .gitignore file.


Expand IValue to all TorchScript Supported Types

Current tch-rs IValue which is used as the input to TorchScript models covers the following input:

Tensor(crate::Tensor)
Int(i64)
Double(f64)
Tuple(Vec<IValue>)

Taking a look at TorchScript supported types we still need to support:

[x] bool | A boolean value: (https://github.com/LaurentMazare/tch-rs/commit/0e4ecff8fbb73e6151ce2bade5a0857a324d8fe5)
[x] str | A string (https://github.com/LaurentMazare/tch-rs/commit/7d3e9cb445a72608789271c0d3735ca43eaf6edf)
[x] List[T] | A list of which all members are type T
[ ] Optional[T] | A value which is either None or type T
[x] Dict[K, V] | A dict with key type K and value type V. Only str, int, and float are allowed as key
types.
[ ] NamedTuple[T0, T1, ...]

Are there plans to add support for these types anytime soon?

Happy to add support for some of the missing ivalue types, I just pushed some changes to support None and bool. We should aim at supporting all of them but let me know if some are more interesting to you and we'll push on these first.

Thanks for the quick response @LaurentMazare! It'd be great if List[T] could be implemented. List[str] is especially useful for dealing with language based models.
FYI we're using this library to implement a production server for TorchScript written in Rust.

There is now some support for monomorphic lists (IntList, DoubleList, TensorList, BoolList) as detailed in this enum. I'm not sure how exactly this relates to List[T], Optional[T] but I think this should cover it.
GenericList should also be supported and can be used to get a list of string. E.g. I also added a new test where a python jit module is used to remove the last letter of each string in a list (python code, rust test).
You can see some other tests using this with the into trait at the end of jit.rs.

Looks great! Thanks for the pointers!

Just to mention that Dict should now be supported. The rust is equivalent is a vector of pairs of IValues for now as IValues cannot be used as a key for maps in a straightforward way (as they include a float case).

Closing this for now as the main types should be supported, feel free to re-open if there are some missing bits that would be useful to you.


Test failures caused by a changed wording of failure messages

Expected Behavior
The build and tests succeed when checking out origin/main
Observed Behavior
The build fails with the following message:
    FAILURE: Build failed with an exception.

    * What went wrong:
    Execution failed for task ':detektMain'.
    > Analysis failed with 3 weighted issues.

    * Try:
    > Run with --stacktrace option to get the stack trace.
    > Run with --info or --debug option to get more log output.
    > Run with --scan to get full insights.
    > Get more help at https://help.gradle.org.

    BUILD FAILED in 29s
    3 actionable tasks: 3 executed
    "
    to contain:
      "failed with 3 issues."
        at io.gitlab.arturbosch.detekt.JvmSpec.Type resolution on JVM(JvmSpec.kt:17)

The full build log is attached here: buildLog.txt
Amongst others, it fails at JvmSpec.kt line 17 which states:
assertThat(result.output).contains("failed with 3 issues.")

The actual output, however, contains "failed with 3 weighted issues.", so, changing the above line to
assertThat(result.output).contains("failed with 3 weighted issues.")

fixes the issue. I will shortly post a PR that contains all places where this is happening.
Steps to Reproduce

Clone the repo
Checkout origin/main  (In particular, I ran this on 8e350ee7a895baacdbeb4480812cfb173ae750a8)
Run ./gradlew build -x dokkaHtml

Context
N/A
Your Environment

Version of detekt used: N/A

Commit affected: 8e350ee7a895baacdbeb4480812cfb173ae750a8

Version of Gradle used (if applicable): 8.5

Gradle scan link (add --scan option when running the gradle task): https://gradle.com/s/bwo67jnwume24

Operating System and version: Windows 10 Version 21H2

Link to your project (if it's a public repository): N/A

I can also reproduce this locally with 7bbdb026e320600c1e97eaf199a266681d83f5e9. Funnily enough, the CI now fails on my PR because of those changes.

Hmm, even when I build detekt on our internal CI (which runs ubuntu), the same test failures happen and are fixed using my PR. It seems as if my CI and your CI use different versions of detekt, even though it is all built from the same commit...

Run ./gradlew publishToMavenLocal before the build. This is unfortunately a necessary step to make sure the functional tests are using artifacts produced by the build. Without that step the artifacts will be pulled from Maven Central and the behaviour won't always align, leading to these failures.
#6415 is stalled but should fix the build dependency issue which avoids this problem.

In that case, Contributing.md should mention that. As a novice to the
project it's otherwise impossible to guess that 🙃
I can make a PR for that tomorrow if I feel motivated 🙈
Matthew Haughton @.***> schrieb am Mo., 11. Dez. 2023,
22:12:

Run ./gradlew publishToMavenLocal before the build. This is unfortunately
a necessary step to make sure the functional tests are using artifacts
produced by the build. Without that step the artifacts will be pulled from
Maven Central and the behaviour won't always align, leading to these
failures.
#6415 https://github.com/detekt/detekt/pull/6415 is stalled but should
fix the build dependency issue which avoids this problem.
—
Reply to this email directly, view it on GitHub
https://github.com/detekt/detekt/issues/6708#issuecomment-1850895850,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGCNGIQTXAM26QMEGWEODDYI5ZM3AVCNFSM6AAAAABAPYCIW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJQHA4TKOBVGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Agreed, this should be covered in that doc. A PR is very welcome, thanks!


Added Battery heater trigger for battery_heater or battery_heater_on.…

This will ensure that both S,X and M,Y models will show when battery heater is active during charging.
Also added option to recognize value "2" on phases as three phases, because it's "2" when the 3-phase connection is active.
Data in the DB stays the same, only Grafana changes the visuals.
I have applied these onto my Grafana and I am happy with results.
Woyteck

I am also inclined to change battery heater value to 10 instead of 1, so it's more visible on the graph, like this:


software bug: when install a k8s in my offline kylin nodes, the installation fail

install k8 in kylin sp2 offline nodes,
version: v0.4.2-rc2
log:
TASK [kubernetes/control-plane : kubeadm | Create kubeadm config] **************
changed: [node1]
Thursday 22 December 2022  03:28:11 +0000 (0:00:07.565)       0:50:37.372 *****
Thursday 22 December 2022  03:28:11 +0000 (0:00:00.060)       0:50:37.433 *****
Thursday 22 December 2022  03:28:11 +0000 (0:00:00.070)       0:50:37.503 *****
Thursday 22 December 2022  03:28:11 +0000 (0:00:00.058)       0:50:37.562 *****
Thursday 22 December 2022  03:28:11 +0000 (0:00:00.059)       0:50:37.621 *****
Thursday 22 December 2022  03:28:11 +0000 (0:00:00.069)       0:50:37.691 *****
Thursday 22 December 2022  03:28:11 +0000 (0:00:00.049)       0:50:37.741 *****
Thursday 22 December 2022  03:28:11 +0000 (0:00:00.065)       0:50:37.806 *****
Thursday 22 December 2022  03:28:11 +0000 (0:00:00.074)       0:50:37.881 *****
FAILED - RETRYING: [node1]: kubeadm | Initialize first master (3 retries left).
FAILED - RETRYING: [node1]: kubeadm | Initialize first master (2 retries left).
FAILED - RETRYING: [node1]: kubeadm | Initialize first master (1 retries left).
TASK [kubernetes/control-plane : kubeadm | Initialize first master] ************
fatal: [node1]: FAILED! => {"attempts": 3, "changed": true, "cmd": ["timeout", "-k", "300s", "300s", "/usr/local/bin/kubeadm", "init", "--config=/etc/kubernetes/kubeadm-config.yaml", "--ignore-preflight-errors=all", "--skip-phases=addon/coredns", "--upload-certs"], "delta": "0:05:00.003571", "end": "2022-12-22 11:48:47.375066", "failed_when_result": true, "msg": "non-zero return code", "rc": 124, "start": "2022-12-22 11:43:47.371495", "stderr": "W1222 11:43:47.401361   49193 utils.go:69] The recommended value for "clusterDNS" in "KubeletConfiguration" is: [<IP_ADDRESS>]; the provided value is: [<IP_ADDRESS>]", "stderr_lines": ["W1222 11:43:47.401361   49193 utils.go:69] The recommended value for "clusterDNS" in "KubeletConfiguration" is: [<IP_ADDRESS>]; the provided value is: [<IP_ADDRESS>]"], "stdout": "[init] Using Kubernetes version: v1.24.7\n[preflight] Running pre-flight checks\n[preflight] Pulling images required for setting up a Kubernetes cluster\n[preflight] This might take a minute or two, depending on the speed of your internet connection\n[preflight] You can also perform this action in beforehand using 'kubeadm config images pull'", "stdout_lines": ["[init] Using Kubernetes version: v1.24.7", "[preflight] Running pre-flight checks", "[preflight] Pulling images required for setting up a Kubernetes cluster", "[preflight] This might take a minute or two, depending on the speed of your internet connection", "[preflight] You can also perform this action in beforehand using 'kubeadm config images pull'"]}

/kind bug

i meeted this phenomenon in also on centos nodes, after retry, it succeeded，but not sure they are same issue.

When I watch the log of kubelet, can only find an error about ca.crt.

node1 kubelet[44516]: Error: failed to construct kubelet dependencies: unable to load client CA file /etc/kubernetes/ssl/ca.crt: open /etc/kubernetes/ssl/ca.crt: no such file or directory

Then I retry kubeadm init manually, it passed.

[bootstrap-token] Configured RBAC rules to allow the csrapprover controller automatically approve CSRs from a Node Bootstrap Token
[bootstrap-token] Configured RBAC rules to allow certificate rotation for all node client certificates in the cluster
[bootstrap-token] Creating the "cluster-info" ConfigMap in the "kube-public" namespace
[kubelet-finalize] Updating "/etc/kubernetes/kubelet.conf" to point to a rotatable kubelet client certificate and key
[addons] Applied essential addon: kube-proxy
Your Kubernetes control-plane has initialized successfully!
To start using your cluster, you need to run the following as a regular user:
mkdir -p $HOME/.kube
sudo cp -i /etc/kubernetes/admin.conf $HOME/.kube/config
sudo chown $(id -u):$(id -g) $HOME/.kube/config
Alternatively, if you are the root user, you can run:
export KUBECONFIG=/etc/kubernetes/admin.conf
You should now deploy a pod network to the cluster.

It seems like kubeadm init be coercive killed by the command timeout -k 300s 300s ..... When I run kubeadm init, it token 10 min about. So, it may a network problem.
I continued to follow up and found that pulling images took too long and occasionally failed.


Macbook Pro 8,2 OSX 10.11.3 - Activated but not working

Installed a compatible card, ran v2.2.3 of CAT on OS X 10.11.3. All reports says Handoff should be working. All settings are correct on iPhone, iPad and Macbook Pro. No bueno with handoff. Let me know if you need anything more than whats below.
CAT DIAGNOSTIC BELOW
--- Initiating system compatiblity check ---
--- Hardware/OS checks ---
Verifying Continuity status...          OK. OS X reports Continuity as active
Verifying Mac model reference...        OK. Known compatible Mac Model detected: MacBookPro8,2
Verifying Mac board-id...               OK. Long board id detected: Mac-94245A3940C91C80
Verifying OS X version...               Warning: This version of Mac OS X (10.11.3) is Experimental! Only partially tested on El Capitan
Verifying Wi-Fi hardware...             OK. A Broadcom AirPort card is active, and is using the Continuity compatible Brcm4360 kext
Verifying AWDL status...                OK. An AWDL interface is up, Wi-Fi is ready for Continuity
Verifying Bluetooth hardware...         OK. The internal Bluetooth card is active
Verifying Bluetooth version...          OK. Bluetooth 4.0 detected
Verifying Bluetooth features...         OK. Bluetooth features are Continuity compliant
Verifying Bluetooth firmware...         OK. Bluetooth firmware version: v151 c5844
Verifying Login Item...                 OK. Login item for Auto Continuity Check is not set.
--- Modifications check ---
Verifying OS kext protection...         OK. Kext developer mode is already active
Verifying SIP...                        Ok. System Integrity Protection is already disabled
Verifying ContinuitySupport...          OK. Already patched.
Verifying kexts readability...          OK. Wi-Fi and Bluetooth kexts were found and could be read
Verifying Wi-Fi whitelist status...     OK. The whitelist is correctly patched with your board-id
Verifying BT4 dongles compatibility...  OK. Compatibility with BT4 USB dongles is not enabled, this tool can fix this if a dongle is plugged in
Verifying old Wi-Fi kext presence...    OK. Legacy Wi-Fi driver Brcm4331 was already removed
Verifying legacy Wi-Fi card patch...    OK. The patch is already done. Old Broadcom Wi-Fi cards may work.
--- Modifications check ---
RELATIVE SYSTEM PROFILER INFO BELOW:
BLUETOOTH
Apple Bluetooth Software Version:	4.4.3f4 16616
Hardware, Features, and Settings:
Name:	Colonel Panic
Address:	7C-C3-A1-44-BE-B9
Bluetooth Low Energy Supported:	Yes
Handoff Supported:	Yes
Instant Hot Spot Supported:	Yes
Manufacturer:	Broadcom
Transport:	USB
Chipset:	20702A3
Firmware Version:	v151 c5844
Bluetooth Power:	On
Discoverable:	Off
Connectable:	Yes
Auto Seek Pointing:	On
Remote wake:	On
Vendor ID:	0x05AC
Product ID:	0x821D
HCI Version:	0x6
HCI Revision:	0x16D4
LMP Version:	0x6
LMP Subversion:	0x2297
Device Type (Major):	Computer
Device Type (Complete):	Mac Portable
Composite Class Of Device:	0x38010C
Device Class (Major):	0x01
Device Class (Minor):	0x03
Service Class:	0x1C0
Auto Seek Keyboard:	On
WIFI
Software Versions:
CoreWLAN:	11.0 (1101.20)
CoreWLANKit:	11.0 (1101.20)
Menu Extra:	11.0 (1110.33)
System Information:	11.0 (1100.1)
IO80211 Family:	11.1 (1110.26)
Diagnostics:	5.1 (510.88)
AirPort Utility:	6.3.6 (636.5)
Interfaces:
en1:
Card Type:	AirPort Extreme  (0x14E4, 0xF5)
Firmware Version:	Broadcom BCM43xx 1.0 (<IP_ADDRESS>.1a1)
MAC Address:	7c:c3:a1:44:be:b8
Locale:	FCC
Country Code:	US
Supported PHY Modes:	802.11 a/b/g/n
Supported Channels:	1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 36, 40, 44, 48, 52, 56, 60, 64, 100, 104, 108, 112, 116, 120, 124, 128, 132, 136, 140, 149, 153, 157, 161, 165
Wake On Wireless:	Supported
AirDrop:	Supported
Status:	Connected

Just an update... magically from the train while tethered Handoff began working. Perhaps it's environmental issues with my home network. Will test from the office network and provide additional feedback.

Hey, I had the same issue as you. Handoff started working when I opened up the Airdrop menu. I noticed you also don't have wireless a/c supported. Let me know if you get that working!

@mhedhliGVIT It sometimes takes some time to sync the data that you now got a Continuity compatible device to the server.
@ErikUri The AirPort card from the 2012 MacBook Pro does not support 802.11ac. It is therefore not possible to enable it.


Nominate Oversight Committee chair

Overview
The primary role of Oversight Committee chair is to facilitate operations and governance of the Oversight Committee.
Chair responsibilities include:

Setting the agenda for meetings
Scheduling and facilitating Oversight Committee meetings
Extending discussion via asynchronous communication to be inclusive of members who cannot attend a specific meeting time
Facilitating the creation and execution of Oversight Committee activities

Eligibility
Those who are eligible to be nominated are current Oversight Committee members listed here.
Nominations
Nominations will be open until July 5th.
Eligible people can nominate themselves.
If you plan to nominate somebody else please ensure that they accept the nomination.
How to nominate
Please include a short nomination comment on this issue with the following details:

Name of the nominee
Motivation for the role
Commitment on how much time you/they can dedicate to the role

Notes
This issue is preceding a formally defined Oversight Committee selection process. Per the governance documentation, this will be defined and finalized by the bootstrap Oversight Committee.
Based on community feedback, this can serve as a example or template for the formally defined process.
Related to #31

@jpower432 Where the votes have to be captured? as part of the comments to this issue?

@vikas-agarwal76 Yes. To nominate someone please add a comment with the following information:

Name of the nominee
Motivation for the role
Commitment on how much time you/they can dedicate to the role

I have a very particular set of skills. Skills I have acquired over a very long career. Skills in creating and cultivating open source communities. I enjoy facilitating, coaching, and growing open source communities. I am lucky enough that my day job not only provides me the opportunity to take on a community commitment like this, it incentivizes me to. I can dedicate 4 or more hours a week to this role.
—Liam Neeson,Taken

Hi, as discussed with Jay, I suggested to have 3 Oversight Committee Chairs following the current practices in TAG and WG Chairs.

Hi, as discussed with Jay, I suggested to have 3 Oversight Committee Chairs following the current practices in TAG and WG Chairs.

@ancatri Thanks for adding this comment. Can you please take a look at the pull request to collaborate on this change I have open (#36) and my response on #31. Once agreed, we can add additional clarity on the ticket and extend the nomination period if need be.

@jpower432 I would like to nominate @mrgadgil as a chair together with @jflowers - I hope to find the right issue/PR :)

You have the right issue. Thanks @ancatri!

@mrgadgil Nomination issue #34 - FYI

Based on the results of this issue and #37, @jflowers and @mrgadgil will be co-chairs. I will submit a pull request updating the maintainers file for review.


Implement DB changes regarding diagnosis and patientsnapshot

See ticket https://app.zenhub.com/workspaces/pdcmfinder-board-5f0f036188f27e0014a7bdfc/issues/pdcmfinder/pdxfinder/851

@zperova changes applied in dev