New package: WebalexSystem.WinBGP version 1.1.0

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.9 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

Manual Validation ended with:

2024-12-20 10:49:06.971 [FAIL] Installer failed security check. Url: https://github.com/webalexeu/WinBGP/releases/download/v1.1.0/WinBGP-1.1.0-amd64.msi Result: 0x80004005

(Automated response - build 931.)


Return an error if we fail get get the Root cert.

Closes #415
Signed-off-by: Ville Aikas<EMAIL_ADDRESS>
Summary
Return an error if we fail to get the Root Certificate.
Ticket Link

Fixes #415
Release Note

Fix a bug where the client would not get an error on failed RootCert method.

well, that's interesting. UT passes just fine on local machine :)


feat(pegboard): implement v8 isolate runner

Changes

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

#1213  👈
#1212
#1210
main

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

Join @MasterPtato and the rest of your teammates on  Graphite


feat(filterPOAndWriteTranslateSync): ensure the output folder exists

Current coverage is 100% (diff: 100%)

Merging #27 into master will not change coverage

@@           master   #27   diff @@
===================================
  Files           5     5
  Lines          54    57     +3
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
+ Hits           54    57     +3
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 1c99a4d...ab97f19

Thanks! Released v1.0.6 with your changes.


Need brief explain about docker  and nginx

Hi,
I am not sure what you want an explanation.

If you meant generic docker information, please google it.

Hi,
I am not sure what you want an explanation.

any option is there  for install ,without using docker for this pack


icons cause out of memory issues and crashes on Firefox when hardware 3D acceleration is disabled

it seems that icons could cause out of memory issues on firefox when direct rendering is disabled;
the issue has been verifyied on Firefox 46 and on the latest Tor browser that is based on Firefox 46.
What happens is that icons are not loaded and the browser show a character representing the encoding of the font; sometimes instead the browser simply crash completel.
the issue is debugged on the GlobaLeaks issue tracker (https://github.com/globaleaks/GlobaLeaks/issues/1680) and the reason is still not known.

FWIW, Glyphicons have already been removed from Bootstrap v4.
Glyphicons is a separate project (http://glyphicons.com ), we just vendor a copy of it in Bootstrap v3.
If the problem lies in the font files themselves (rather than the CSS), you'll need to bring it up with the Glyphicons' maintainer.
We're not likely to spend time investigating this problem ourselves (since it concerns an uncommon legacy case), but if there's a specific fix you'd like to propose, we'd duly consider it.

perfectly understood @cvrebert; thank you so much.
let's keep the ticket a bit open so that in case i will find a possible solution i will surely suggest it to you and maybe others could suggest hints.

Hey there!
We're automatically closing this issue since the original poster (or another commenter) hasn't yet responded to the question or request made to them 14 days ago. We therefore assume that the user has lost interest or resolved the problem on their own. Closed issues that remain inactive for a long period may get automatically locked.
Don't worry though; if this is in error, let us know with a comment and we'll be happy to reopen the issue.
Thanks!
(Please note that this is an automated comment.)

Closing this due to inactivity. If you come up with some concrete suggestion, feel free to open a new issue.


Feature/issue 194

Description
Fixes #194 .

TODOs
Read the Gruntwork contribution guidelines.

[x] Update the docs.
[x] Run the relevant tests successfully, including pre-commit checks.
[x] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
[x] Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Added / Removed / Updated [X].
Updated dependencies for following CVEs reported.
https://security-tracker.debian.org/tracker/CVE-2022-1996
https://github.com/advisories/GHSA-cg3q-j54f-5p7p
Migration Guide

Hi @zackproser
Thanks. I updated the test and got it working in circleci.

@pras111gw Nice, thank you :raised_hands:


feat(rpc): add getunclaimedgas RPC method

fix https://github.com/CityOfZion/neon-js/issues/593

Needs a unit test to ensure the data parsing works.
Needs an integration test for smoke testing.

Can you please point me in a direction. I'm still trying to find my way around TS and the neon-js library itself. Thanks

neon-core/integration/clients/NeoServerRpcCLient.ts should be the place for integration test. We probably just wanna issue the call and ensure that we get a non-zero value (aka nothing blows up).
The unit tests is a little more tricky lets skip it for now.

I have added a unit test with the mocks just to make sure that the parsing of the numbers work.
I have adjusted the integration test such that it will work with the integration container that we run against during normal tests. Your test will fail as the endpoint returns an error "invalid address".

Thanks!

On 10 Oct 2020, at 09:44, Yak Jun Xiang<EMAIL_ADDRESS>wrote:

I have added a unit test with the mocks just to make sure that the parsing of the numbers work.
I have adjusted the integration test such that it will work with the integration container that we run against during normal tests. Your test will fail as the endpoint returns an error "invalid address".
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Only write to the hook file if necessary

Should we customize the updateToDate so that the task reports as not being run? I realize it's somewhat superficial but if a frequent task such as check is dependent, it might give a clearer build result output.
Maybe it would be a quick check if the file had ##### KOTLINTER <version number> HOOK START #####?
Thank you for your patience. Normally I try to get feedback to contributors same week, but a lot going on last couple of weeks!

Good idea on actually complying with Gradle's expectations re up-to-date checks.
Version in the hook is a good idea—we could either use the plugin version if there's a way to get that, or just have a manually-incremented version in InstallHookTask.  Thoughts?

Yeah I think the plugin version works fine. It'll get rewritten each revision whether it needs it or not, but that should be ok.

Do you have ideas for how to get the version from within the plugin code itself?

I don't know if there is an easy programmatic way, reference: https://stackoverflow.com/questions/41689977/how-can-i-determine-the-android-gradle-plugin-version-in-use-from-within-my-own
I'm fine with introducing a const into the plugin code and keeping it in sync with the build plugin version.

@jeremymailen Ended up writing the version from the build.gradle.kts to a resource file at build time and reading that in the hook task code.  Bit of a change, split diff will be clearest.

Thank you! Made some adjustments and incorporated: https://github.com/jeremymailen/kotlinter-gradle/pull/158
Will push a release.


fix github link

Currently, the GitHub icon either doesn't work, or brings the user to about:blank.

Thanks!


Update init.py

Changed a few silly things.

I signed it!

I do not think this change is necessary. If you have a compelling reason for doing this, please let me know.


Update sbt-mima-plugin to 0.8.1

Updates com.typesafe:sbt-mima-plugin from 0.3.0 to 0.8.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.typesafe", artifactId = "sbt-mima-plugin" } ]

labels: sbt-plugin-update, semver-minor

Superseded by #132.


Can't resolve '../../client' error after swizzling SearchBar

Description
When I try to swizzle the docusaurus-theme-search-typesense SearchBar component it fails to compile docusaurus with an error.
Steps to reproduce

Run yarn run swizzle docusaurus-theme-search-typesense SearchBar
Select the 'Eject' option
Run yarn start

Expected Behavior
Local Docusaurus will start without error with a copied version of the typesense searchbar as it was previously
Actual Behavior
Docusaurus launches with an error:
Compiled with problems:X

ERROR in ./src/theme/SearchBar/index.js 7:143-199

Module not found: Error: Can't resolve '../../client' in '<redacted>\content\src\theme\SearchBar'

Metadata
docusaurus/core: 2.2.0
docusaurus-theme-search-typesense: 0.8.0

I have the same issue.
My Metadata is the same, however I use npm instead of yarn and I swizzled the SearchBar component using the Typescript flag.
The issue seems to the relative path to the client module in the following line (inside src/theme/SearchBar/index.tsx - directory created by running the swizzling command)
import {useTypesenseContextualFilters} from '../../client';

I tried to manually change the path to the client module by replacing the previous import statement as such:
import { useTypesenseContextualFilters } from "docusaurus-theme-search-typesense/lib/client/index";

But got
Module not found: Error: Package path ./lib/client/index is not exported from package <redacted>/node_modules/docusaurus-theme-search-typesense (see exports field in <redacted>/node_modules/docusaurus-theme-search-typesense/package.json)
client (webpack 5.75.0) compiled with 1 error

And also as such
import { useTypesenseContextualFilters } from "docusaurus-theme-search-typesense/src/client/index";

But got again
Module not found: Error: Package path ./lib/client/index is not exported from package <redacted>/node_modules/docusaurus-theme-search-typesense (see exports field in <redacted>/node_modules/docusaurus-theme-search-typesense/package.json)
client (webpack 5.75.0) compiled with 1 error

Just noting here that Ejecting SearchPage also gives errors:
Module not found: Error: Can't resolve '@theme/SearchPage' in 'C:\Users\DavidL\source\repos\hilltop-docs\content\.docusaurus'
Module not found: Error: Can't resolve '@theme/SearchPage' in 'C:\Users\DavidL\source\repos\hilltop-docs\content\.docusaurus'


fix(table): bug on late loading tables not paging

Summary

If you late loaded a table, subsequent attempts to page through its data would fail.  This is because the totalItems bit wasn't being stored on the pagination widget
Added a fix to pass along the totalItem count every time we called TABLE_REGISTER

Acceptance Test (how to verify the PR)

Hard to verify locally unfortunately, but we saw this error in our dashbaord

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0-beta.9 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.71.5 :tada:
The release is available on npm package (@release-1.x dist-tag)
Your semantic-release bot :package::rocket:


fix(calendar): fixed styles

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 80.307%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7851

Relevant Lines:
8870

💛  - Coveralls


use @Nonnull/@Nullable from javax.annotations.* instead of org.jetbrains.annotations.*

What does this implement/fix? Explain your changes.
AB#1927267: use @Nonnull/@Nullable from javax.annotations.* instead of org.jetbrains.annotations.*
Does this close any currently open issues?
N/P
Any relevant logs, screenshots, error output, etc.?
N/P
Any other comments?
N/P
Has this been tested?

[ ] Tested

@wangmingliang-ms Do we also need to remove the reference for jetbrains annotation library?
Refers https://github.com/microsoft/azure-maven-plugins/pull/1957/files#diff-0eef806b122cba737ed4c1217ece49ef9f3a95e640806184a1804c1f1f46ecf8R110


!!!!!!

I can't clone this code snippet. Can you help me?

Thank you for reporting this issue. To better assist you, could you please provide more details about the error you're encountering while trying to clone the code? Specifically, it would be helpful if you could share the following information:
1. Your environment (operating system, Node.js version, npm or yarn version).
2. The exact error message you're receiving.
3. The steps you took to clone the repository and set up the project.
4. Any relevant extensions or tools you’re using, if applicable.
With more detailed information, we’ll do our best to assist you as soon as possible.

Thank you very much for your enthusiastic support . Can i send a tip as a token of my appreciation ? Best regard .


Short tutorial video on using scqubits

We ask that you make a short (4-6 minutes) tutorial video about some aspects of using scqubits. This video could include going through and explaining one of the tutorial notebooks that have already been written (see here for examples), or alternatively creating a custom notebook and explaining what it does.
Some examples of what such a video would look like are shown here or here.
Good candidates for notebooks to discuss would be:
demo_qutip_fluxoniumreset.ipyn
demo_qutip_fluxoniumcz.ipynb
scqubits_paper_source_code.ipynb
demo_customcircuit.ipynb
Alternatively putting together a "custom" notebook that includes some new discussion that we don't yet cover, or even some combination of what's already been done would also be great. An example might be a notebook that explores performing a gate operation on a noisy transmon qubit (such a notebook would include parts of content presented in here and here).
Specific requirements:

Choose a scqubits tutorial notebook from here or alternatively write your own - it could be some combination of content from ones that already exist.

Reply with a comment to this bounty that includes a few sentences (could be in point-form, i.e., a sentence per section in the notebook you want discuss) that outlines what your tutorial would cover and what which aspects of the code and relevant physics you aim to highlight.

Put together a 4-6 minute video

Either upload the video to youtube and provide a link to it. Alternatively provide a link to where the video could be downloaded from (e.g. dropbox). In the latter case, please explicitly write that you agree that we upload your video to the the @scqubits youtube channel.

Dear Peter,
I want to work on this video. My idea to cover is the following:

Explain the circuit quantization procedure.

Give the procedure to quantize any circuit (step by step given the in the Quantum Information & Quantum Optics by Juan Jose Ripoll). Starting with the differential equation, find the Lagrangian and the Hamiltonian, and finally, give the common examples from the LC oscillator and the transmon (C+JJ).

Show the ability to scqubits to calculate and give any custom_circuit quantization. Show the same equation given theoretically.

Show & explain the scqubits GUI to finally encourage people to try the library.

My idea is to do a custom notebook that explains and covers this topic step by step, explaining how to design in scqubits. Let me know if this is enough or if you have more ideas to include.
Time constraints? I think the video will be longer than the 4-6 min (4-6 min explaining the theory and 4-6 min explaining scqubits). Is there a problem with having a longer video?
Best,
Kevin J.

Hi Kevin... what you propose sounds great!
As far as the length of the video - I would propose splitting the video into two parts... each about 4-6 mins long - that seems to work better with people's short attention spans (just a reality of things these days  ;-), but also might make them easier to make.
If you feel rather strongly about a single longer one, however, that would also be fine.
The rest of what you propose reads great - picking custom circuits is a perfect candidate for tutorial videos - good choice!
Please let us know if you have any other questions.

Hi Kevin,
Can I also work on the videos? I am also interested in making tutorial videos.

Hi Kevin,
Can I also work on the videos? I am also interested in making tutorial videos.

Hi Alireza, just to help clarify, are you interested in collaborating with Kevin to make tutorial videos or you would like work on video(s) by yourself?

We can work together.

Hi Alireza, thanks for your interest in collaborating with scqubits and me!
I already have the two presentations about quantum circuit quantization, and I am planning to record them this weekend. If you already have a video in mind I will absolutely be happy to work with you if not we can come up with another idea using scqubtis; more is better! You can reach me at https://www.linkedin.com/in/kevin-joven/

Hi Kevin,
I added you on Linkedin. Here is mine https://www.linkedin.com/in/ali-ghasemi-46404a87/
We can discuss this more on LinkedIn.

Hi Kevin,
I added you on Linkedin. Here is mine https://www.linkedin.com/in/ali-ghasemi-46404a87/
We can discuss this more on LinkedIn.

Hi I'd also like to work on this! Do let me know!

Dear Peter,
I already upload the videos as "unlisted". Please check the videos and let me know what you think to put the videos to the public. Let me know if there are things that need to be corrected, I will do it as quickly as possible. Also, I would like to read the comments from Tianpu Zhao & Alireza if possible.
Thanks!
Quantum Circuit Quantization Part 1 - Theory: https://youtu.be/ChefdJWovYc
Quantum Circuit Quantization Part 2 - scQubits: https://youtu.be/Tv3nLKb66OE
Best,
Kevin J.

Thanks @KevinJoven11.
Will get back to you in a day or two (but from a quick glance, they look great!)

btw, @KevinJoven11  will you want to have these uploaded to the scqubits channel (https://www.youtube.com/@scqubits4175/videos ), or to your own? either would be fine.

Dear Peter,
I am okay with both. Just let me know what you prefer!
Best,
Kevin J.

Hi Kevin,
Your videos look really nice. Thank you very much for your effort! I like the idea to start with studying the quantum version of the LC resonator in the first video, then shows how to model LC resonator and transmon with the Circuit module. I think these videos can certainly be further polished and improved, but your work might already be qualified for claiming the bounty (it's up to Jens/Peter's decision).
Here are things I think could be improved. Maybe for some of these points you might have your own considerations/justifications, I would really love to hear your thoughts as well!
For the first video:

The title "quantum circuit quantization": "quantum circuit" is ambiguous in the field of quantum computing - quantum circuit often refers to quantum computation models (those wires and boxes diagram you often see in quantum algorithms); for circuits made of superconducting metals, "superconducting circuit" is more frequently used.
I think the derivation for the quantum LC oscillator can be streamlined for a better clarity. In your presentation, you started with expressing energies stored in a capacitor and an inductor, and wrote H(I, V) = CV^2/2 + LI^2/2, then you expressed I and V in terms of Phi and Phi dot and wrote H(Phi, Phi dot). Both are not the correct form of Hamiltonian, since Hamiltonian should be written in terms of pairs of conjugate variables (n and Phi in our case). I saw you later presented the general procedure (starting from differential equation, write down Lagrangian, then perform Legendre transformation to obtain Hamiltonian, then promote conjugate variables to quantum operators); why don't you start with using these steps to build the LC circuit Hamiltonian at the beginning?
As far as I know, the common practice for circuit quantization starts from writing down the Lagrangian, rather than writing down the differential equation.
You also have a slides titled "working at ~mK" in the middle of your LC oscillator example. I feel like it'd be better discussed at the beginning, since I think these points are motivations for us to use superconducting circuits at ~ mK as qubits. Also the point "no resistance" can help answer why we usually only model circuits with L, C and Josephson junctions.
You have a slide talking about Josephson junctions. I feel like the motivation for discussing Josephson junctions is missing there. I noticed that in the second video you did talk about anharmonicity is needed for obtaining a qubit and therefore we need nonlinear elements. Maybe it would be better to briefly mention in this slide that nonlinear inductance is crucial for making qubits.
Inductor symbol: I usually saw inductor symbols are curly lines (and there are many variants of them). Is the square lines in your slides another commonly used inductor symbol?

For the second video:

The content generally looks great!
The screen capture seems quite blurred and I can barely read - would be much better if the video has higher resolution.

Thanks again @KevinJoven11 for working on the videos, and thanks @ZhaoTianPu for your comments.
@KevinJoven11, if you have it in you to address some of the comments from @ZhaoTianPu, that would of course be great, but since we know that these sorts of videos take a long time to make, we are also fine accepting them as-is, as far as the bounty goes.
As far as publishing them, what might be best, is if you could make them publically available on your channel, and maybe leave a link somewhere where the source files could be downloaded from (so that down the road, if it makes sense, Jens could also put them up on the @scqubits channel).
Agh.. and one suggestion: at least for the second video, maybe make sure that "scqubits" is mentioned somewhere in the title, so that the video is easier to find via youtube's search machinery.
thanks

Dear Peter,
Thanks for your feedback and to Tianpu Zhao as well!
I already made the video for public and put in the title scQubits and anyone can download the repo from the link in the video below.
I am planing to improve the videos as well.
Thank you so much,
Kevin J.

Thanks @KevinJoven11
We can consider this bounty as closed then.

Hi Alireza, thank you so much for your contributions. Your video looks really nice! I will make time today to watch it and provide my thoughts/suggestions.
I am not sure if you can get the bounty at this time (I hope you will be able to, based on what you've worked)! This would be better answered by @petergthatsme.

Thanks for your work on the video @Algh89110377   - it's very nice. Please feel free to make it public.
A couple of minor points:

could you please update the title of the video so that it contains "scqubits"? that will make it easier for people to find.
could you please update the video description with a couple of sentences that describe what the video is about, and ideally, with a link to the very notebook you're going over

I asked the powers that be at the Unitary Fund, and it looks like they will be able to award both you and @KevinJoven11   with the rewards - which is great (thanks Unitary Fund!).

@KevinJoven11 after discussing with colleagues I hope to correct myself: it is ok to talk first about equations of motion, then the Lagrangian and Hamiltonian. As long as the Euler-Lagrange equation obtained from Lagrangian agrees with the equations of motion, then it is fine.

Dear @ZhaoTianPu ,
Thank you so much for your feedback! In the future, I will explain this tutorial more deeply as soon as I have more experience working with superconducting circuits!
Best,
Kevin


[Master Story] Patient Data Ingest

As an Organization Administrator, I need to populate Patient Data in our organization's Data Input data store. To do this, I want to upload a CSV file that I exported from my organization's system of record.
This issue is the core summary of features for this story, and includes sub tasks (that aren't obviously subtasks) such as:
#11 - Define Prototype CSV structure for Ingest
#13 - Define Data Input Data Storage Schema

From 10/06 MVP meeting
Current needs

CSV import

Create template for import
Record management done in CSV
Patients have a uniqe ID from a system of record (ex: student ID number)
Records can be exported
Records are updated via CSV export/edit/re-import

Records can be deduped
Imported records are displayed in an index

Important concerns

HHS compliance
PII PHI

Parking lot

Manual patient entry form
Editing records within the app
Patient self entry

w/code/ID generation (QR code, or similar)

My overall notes from the meeting, a bit chaotic:
Should start with Neil & Shelby clarifying what they want out of this meeting so we stay on target for them, I'm assuming it will be something like...
define the various "users" for the mocks
    clarify the workflows tied to those users (who does what)
    work through page by page to clarify what is happening
    discuss alternatives due to eng limitations
    try to get enough detail so we can create some user stories and tasks to flesh this out
Mockups:
https://primedatainput.invisionapp.com/console/PDI-10-6-2020-KSA-ckfxxtc1h0071012aewwdcpfl
Data Input Board:
https://github.com/CDCgov/prime-central/projects/1
Neil:
Would be helpful for the product as a whole for everyone to get on the same page about what is being worked on. Has started to implement some stuff but the mockups are changing.
Also would like to know what the designers need.
Would like to use the project boards on github.
Left a dozen or more comments on the mocks and have a dozen or more to add.
Katie: I have questions too, everyone probably has questions.
Katie: in terms of feedback we're looking for, feedback on feasibility, a couple things we're just throwing in there.
Q where to capture comments decisions discussions?
Drew: Maybe github issues?
Katie: Is there a certain type of feedback or issue better suited to capturing in github vs comments.
some discussion about that
Neil: In turns of tracking design to-dos, what are the preferred ways for design issues?
Drew: I personally quite like github issues and it would make sense to have it in one place. Maybe use tags? Design, engineering, etc.
Katie: like the idea of everything in one place
okay let's try rolling in github for a couple sprints, on to mockup review!
Test Queue view: The person administering a test or checking in a patient at a POC?
We've been using the word "test administrator" as the person administering the test, but it might mean something else?
Alicia: an area to do more research on, but there's two options on how this could be used:

the person who checks in the patient and adds you to the queue
the person who is using the queue to do the test

"think about it like you come in for the test and the administrator is the person who checks you in AND records the result"
Rick possibly using the word "nurse"
option to scan QR code (button)
or search by name, DOB, or alpha ID
is the QR code good for MVP?
QR codes are new sexy
search bar: autocomplete is dangerous for PHI? possibly dangerous at some point
for MVP maybe just an alphanumeric id?
but people might forget it, still need a way to look them up
maybe just not build autocomplete, search pattern needs to protect privacy
probably something to ask lawyers and privacy people about
lots of discussion about autocomplete or not
if you're a bad actor and want to look up random people it shouldn't be easy
Shelby: I want to better understand how patient records get into this system to begin with? For the MVP is it just the staff entering it manually? do we have to have a page where patients fill out first?
Alicia: CSV upload is number one feature people are excited about.
Administrator adding people in.
Health department wanted to make sure the names are correct.
Shelby: If we assume CSV upload, could be 10k or 5 records...  what is a useful search feature? Probably want ot have ways to identify patient based off the ways they'd be record in other systems, such as student id number, external identifiers like email, probably not a code that we generate.
Presumption: did we agree MVP would be just CSV upload?
Alicia - assume a person's record is tied to a single location.
Neil: Assume this is at a university, when you say location you mean the "university" as a location or test sites in a university of which they can be many?
Alicia: Location = University
Rick: There's testing organization and actual testing facilities. Could have many CLIA numbers maybe?
Alicia: haven't seen that yet but something we should probably ask about
Neil: thinking there might be multiple test sites, part of the flow might include which sites, but that feels like metadata for patient registration. what shelby and I were thinking was a "nurse" is associated with an organization.
define organization: multiple test sites, but all patient data shared between all test sites. not shared with other orgs
shouldn't be data shared across organizations
CSV as source of truth for us - other system that generates CSV is the system of record
CSV upload needs deduplication
pre-test survey: patient fills this out in advance to get a QR code
workflow to this point:
upload CSV
that includes id
search by id
uploading dupes get deduped
Shelby does agree we should discuss if/when we can be a system of record.
Neil suggests storing changes separately in separate fields.
2-4 weeks, get by with CSV upload, download, etc.
as soon as possible should be building add/edit patients information
maybe it's easier to skip CSV? pete and shelby shaking their heads
how do typos/etc get fixed
Neil: taking it back a little bit, editing feels like something that can cause problems later [something to think about later]
If this information doesn't go anywhere after we change it, or if changing it in our site creates problems, I don't see it as possible or feasible for an MVP.
Alicia wants to make alternate possible:

still upload by CSV to add data
add new CSV or add more data to CSV
way to disable/hide records -

Katie: Bulk upload route, only getting information is system of record.
Next steps:
Neil:
async on a million questions about the queue
set up time on Thursday
Shelby:
this changes how I've been thinking about the API resource design

Break out into individual tickets, Parking lot becomes icebox

These have now been broken out into individual tickets by Alicia, including stuff like #55 #54 #53 #56 etc.


Unable to save favorites in order to get quick access in the future

When using DataAll, it is not possible to save a favorite view or chart using the URL and saving it to your desktop etc.
This makes the navigating to the most used views very time consuming.
It also means you cannot create a specific drill down view to monitor a particular issue and save it.
Added to the complication of short timeout, makes navigating around DataAll unworkable at times.

More information on this.
The ability to set filters and parameters on a dashboard and then save them, within the embedded dashboard in DA is not possible, we would like the functionality of saving the state brought into the embedded DA view.

You are talking about Quicksight right? I think the issue goes back to the type of user that data.all creates in Quicksight. It is explained in detail in #281

Closing stale issue. If needed it will be reopened


added solution for 4 Sum in C++

Please add the solution according to the pre-defined code in the Leetcode.

updated it

Looks good. Don't forget to star this repository and share on your social media about this repo for hacktoberfest. Thank you for the contribution.


add opencensus metrics to MCP client

This largely follows the same structure as the server metrics. Part of #3311.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers: ayj, kyessenov
If they are not already assigned, you can assign the PR to them by writing /assign @ayj @kyessenov in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS
mixer/OWNERS
pilot/OWNERS
pkg/mcp/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ayj
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: kyessenov
If they are not already assigned, you can assign the PR to them by writing /assign @kyessenov in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS [ayj]
mixer/OWNERS
pilot/OWNERS [ayj]
pkg/mcp/OWNERS [ayj]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @kyessenov

/lgtm
(for mixer one liner)

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ayj, kyessenov
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS [ayj]
mixer/OWNERS [kyessenov]
pilot/OWNERS [ayj,kyessenov]
pkg/mcp/OWNERS [ayj]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

@tangiel: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
3523e23afef9e12b74c7480396d787e4c0bb8fcc
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tangiel: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
3523e23afef9e12b74c7480396d787e4c0bb8fcc
link
/test istio-unit-tests

prow/istio-presubmit.sh
3523e23afef9e12b74c7480396d787e4c0bb8fcc
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

New changes are detected. LGTM label has been removed.

Rebased in hopes of tests auto-fixing.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ayj, kyessenov, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS [ayj,ozevren]
mixer/OWNERS [kyessenov]
pilot/OWNERS [ayj,kyessenov]
pkg/mcp/OWNERS [ayj,ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tangiel: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
4ce7e2ff1bdd0bd27d6aeb357c09129c442daee8
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tangiel: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
4ce7e2ff1bdd0bd27d6aeb357c09129c442daee8
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
4ce7e2ff1bdd0bd27d6aeb357c09129c442daee8
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tangiel: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
4ce7e2ff1bdd0bd27d6aeb357c09129c442daee8
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
4ce7e2ff1bdd0bd27d6aeb357c09129c442daee8
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
4ce7e2ff1bdd0bd27d6aeb357c09129c442daee8
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

New changes are detected. LGTM label has been removed.

Looks like all tests pass on rebase and random rerun, so please lgtm again.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ayj, kyessenov, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS [ayj,ozevren]
mixer/OWNERS [kyessenov]
pilot/OWNERS [ayj,kyessenov]
pkg/mcp/OWNERS [ayj,ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tangiel: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
592bbc5b05c2a2c54c05fb22d67efcba18739104
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Broken Parameters

Hey love the progress made on the plugin, I'm running the latest release and am running into issues with random parameters.
Not only am I not able to get Integer, Header, and XY parameters working (for things like setting the resolution), but I also have random floats not working. For example, in the file I've attached, the Particle Count and Particle Size parameters are not working in Resolume. Not sure what I'm doing wrong on my end.
Thanks so much!
O.X
PARTICLES - Seaweed.zip

I’ll have a look to see what may be the issue. Can you please send me your resolute log as well.

Sure thing, where can I find it?

I'm seeing that header breaks everything, and that pulse comes in as toggles, momentary comes in as pulses. Something must not be parsed correctly when converting.
When this gets stable it will be such a game changer, you're doing a huge service to the communty. The last major super important thing is figuring out how to get floats ranged properly, there absolutely has to be a way to get normMin normMax parameters represented properly as floats with a range bigger than 0-1

Resolume Arena log.txt
Here you are.

I found the source of this issue with the header parameters. It will take a bit of a rewrite to be able to solve it, but I know how to.

Ok so there is now a version with a fix when header parameters are included. https://github.com/medcelerate/FFGLTouchEngine/releases/tag/v1.1.1

I'm seeing that header breaks everything, and that pulse comes in as toggles, momentary comes in as pulses. Something must not be parsed correctly when converting.
When this gets stable it will be such a game changer, you're doing a huge service to the communty. The last major super important thing is figuring out how to get floats ranged properly, there absolutely has to be a way to get normMin normMax parameters represented properly as floats with a range bigger than 0-1

So there is a way to do it under the hood, but in the UI it will only ever be between 0 and 1. Unfortunately this is due to limits in the resolume FFGL SDK that I do not have control over.

Hey I am able to get my .tox working well now. Super good to know that the parameters will always be between 0 and 1, I can plan my files accordingly. Might be worth putting that information in the description.
The last issue I'm running into now is that XY parameters don't load to defaults. I'm using it to set my resolution, but no matter what I try it always loads to 0 as a default. Is it possible to have all parameters load set to their defaults as set in TouchDesigner?
Thanks for the continued support!

Would it be possible to add functionality for saving the default values of the XY parameters? I'm not able to use intergers to set the resolution of files for some reason. I've only had success with the XY type parameter, but everytime it loads in resolume to 0 when it should default to 1.
Ideally it would be nice to be able to use an integer parameter, but even that has a max value of 1000, when 10000 would be ideal to allow for large stage resolutions (Max I've experienced is 16000 pixels wide)
PARTICLES - Seaweed.zip
You can see in my attached tox, where the Render TOP is setting the resolution.


feat: search agent API

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

hey @yuye-aws , what does this PR do?

This PR is implements an API to search agent: ML_BASE_URI + "/agents/_search". As this PR is missing unit tests, it is still a draft, and I will implement unit tests later.

Thanks! Would you mind renaming the PR to match?

Ok

Hi @HenryL27 ! I have just committed some unit cases. This PR is ready for code review now! Thanks for reminding me of renaming this PR.

@ylwu-amzn This PR is ready for review now.

Don't we need a search action: https://github.com/opensearch-project/ml-commons/blob/main/common/src/main/java/org/opensearch/ml/common/transport/connector/MLConnectorSearchAction.java ?

Don't we need a search action: https://github.com/opensearch-project/ml-commons/blob/main/common/src/main/java/org/opensearch/ml/common/transport/connector/MLConnectorSearchAction.java ?

Is this what you need: common/src/main/java/org/opensearch/ml/common/transport/agent/MLSearchAgentAction.java


entrypoint: exec cloud-api-adaptor process

also us exec form in Dockerfile
Fixes: #289
Signed-off-by: Snir Sheriber<EMAIL_ADDRESS>

@caavery could you please confirm it fixes the issue
images for testing:
quay.io/snir/cloud-api-adaptor-vsphere:sig_fix
quay.io/snir/cloud-api-adaptor-libvirt:sig_fix

hi @snir911 ! This is going to ensure that SIGTERM is cascaded to the cloud-api-adaptor-PROVIDER process. Okay, my question is: is there anything else that we should be doing on cloud-api-adaptor-PROVIDER to kindly terminate the process?
Other than that, your change looks good to me.

@wainersm I'm not sure i'm getting your point, AFAIU it seems it terminates kindly when it gets SIGTERM, so it should be enough


Logout / invalidation not working as expected

Subject of the issue
Logout doesn't appear to work. Or am I missing something. I would assume after logout, an attempt to retrieve the user with the same token would result in an "unauthorized" response. But I can still get the user with the same token after logging out (hence, valid token).
Your environment

Q
A

Bug?
yes

New Feature?
no / yes

Framework
Laravel

Framework version
8.12

Package version
1.1

PHP version
8

Steps to reproduce
Login, then logout (see attached code)
Expected behaviour
The token sould be invalid after logging out
Actual behaviour
The token is still valid.
`<?php
namespace App\Http\Controllers\Auth;
use App\Http\Requests\Auth\LoginRequest;
use App\Models\User;
use Illuminate\Http\JsonResponse;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use PHPOpenSourceSaver\JWTAuth\Exceptions\JWTException;
use PHPOpenSourceSaver\JWTAuth\Exceptions\TokenExpiredException;
use PHPOpenSourceSaver\JWTAuth\Exceptions\TokenInvalidException;
use PHPOpenSourceSaver\JWTAuth\JWTAuth;
use stdClass;
class AuthController
{
private JWTAuth $auth;

public function __construct(JWTAuth $auth)
{
    $this->auth = $auth;
}

public function login(LoginRequest $request)
{
    $email = $request->email;
    $credentials = ['email' => $email, 'password' => $request->password];
    $isAuthorized = Auth::attempt($credentials);

    if (!$isAuthorized) {
        return response('Not Authorized', 401);
    }

    $user = User::where('email', '=', $email)->first();
    $token = $this->auth->fromUser($user);

    $frontEndUser = new stdClass();
    $frontEndUser->id = $user->id;
    $frontEndUser->name = $user->name;
    $frontEndUser->email = $user->email;

    if ($token) {
        return response()->json(['token' => $token, 'user' => $frontEndUser]);
    }

    return response('Something went wrong', 500);
}

public function user(): JsonResponse
{
    $user = auth('api')->user();

    if (!$user) {
        return response()->json('not authenticated', 401);
    }

    $frontEndUser = new stdClass();
    $frontEndUser->id = $user->id;
    $frontEndUser->name = $user->name;
    $frontEndUser->email = $user->email;

    return response()->json(['user' => $frontEndUser]);
}

public function logout( Request $request )
{
    try {
        $this->auth->parseToken()->invalidate();

        return response()->json( [
            'error'   => false,
            'message' => trans( 'auth.logged_out' )
        ] );
    } catch ( TokenExpiredException $exception ) {
        return response()->json( [
            'error'   => true,
            'message' => trans( 'auth.token.expired' )

        ], 401 );
    } catch ( TokenInvalidException $exception ) {
        return response()->json( [
            'error'   => true,
            'message' => trans( 'auth.token.invalid' )
        ], 401 );

    } catch ( JWTException $exception ) {
        return response()->json( [
            'error'   => true,
            'message' => trans( 'auth.token.missing' )
        ], 500 );
    }
}

}
`

Please try the new release 1.2.0 and set the variable JWT_SHOW_BLACKLIST_EXCEPTION to true.

@rohrig this issue persist?
If in a week we didn't hear back from you we'll close it and mark it as solved.

Due to inactivity, we'll close this issue.


Added imports option for entities

As mentioned in https://github.com/kean/CreateAPI/issues/18#issuecomment-1031208978, this PR adds the imports option to the Entities.
Why?
Similar to the paths.imports,  this pretty much allows the user the import any modules in the generated entities files

Great, thanks!


[Bug report] docs:dev and docs:build run successfully; serving production build throws router(?) errors and removes the app from the page

Description
I linked to a PR on our beta branch with a working deploy preview where you can view the actual errors.
In the reproduction PR, I am upgrading our VuePress beta version (and related dependencies) from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>🟢  When I run docs:dev the local VuePress site spins up successfully with no issues
🟢  When I run docs:build the docs directory is properly built with zero errors or messages
🔴  When the built docs/.vuepress/dist files are served (either locally using yarn docs:preview in the repo, or on the Netlify deploy preview) the site loads successfully for a split second; however, then starts throwing the errors shown below:

When I inspect the page, I can see that the errors that were thrown have cleared out the entire contents of the VuePress app

If I remove the sidebar entry from the theme: defaultTheme({ /* ... */ }) config, the site will properly render (without clearing the content); however, navigation is still completely broken, which leads me to believe this bug is somehow related to the router.
I have tried disabling VuePress plugins, and have noticed that if I disable any of the plugins that utilize router.afterEach the number of errors is reduced

Reproduction
https://github.com/Kong/kongponents/pull/656
Used Package Manager
yarn
System Info

Click to expand system info
System:
    OS: macOS 12.4
    CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
    Memory: 5.41 GB / 32.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.15.1 - /var/folders/mg/3ttphynn3f13__q7wph27bbm0000gq/T/yarn--1655049285146-0.9745900856344969/node
    Yarn: 1.22.17 - /var/folders/mg/3ttphynn3f13__q7wph27bbm0000gq/T/yarn--1655049285146-0.9745900856344969/yarn
    npm: 8.6.0 - ~/Development/kongponents/node_modules/.bin/npm
  Utilities:
    Git: 2.32.1 - /usr/bin/git
  Browsers:
    Chrome: 102.0.5005.115
    Edge: Not Found
    Firefox: Not Found
    Safari: 15.5
  npmPackages:
    @vuepress/bundler-vite:  2.0.0-beta.48
    @vuepress/bundler-webpack: Not Found
    @vuepress/cli:  2.0.0-beta.48
    @vuepress/client:  2.0.0-beta.48
    @vuepress/core:  2.0.0-beta.48
    @vuepress/markdown:  2.0.0-beta.48
    @vuepress/plugin-active-header-links:  2.0.0-beta.48
    @vuepress/plugin-back-to-top: ^2.0.0-beta.48 => 2.0.0-beta.48
    @vuepress/plugin-container:  2.0.0-beta.48
    @vuepress/plugin-docsearch: Not Found
    @vuepress/plugin-external-link-icon:  2.0.0-beta.48
    @vuepress/plugin-git:  2.0.0-beta.48
    @vuepress/plugin-google-analytics: Not Found
    @vuepress/plugin-medium-zoom:  2.0.0-beta.48
    @vuepress/plugin-nprogress:  2.0.0-beta.48
    @vuepress/plugin-palette:  2.0.0-beta.48
    @vuepress/plugin-prismjs:  2.0.0-beta.48
    @vuepress/plugin-pwa: Not Found
    @vuepress/plugin-pwa-popup: Not Found
    @vuepress/plugin-register-components: Not Found
    @vuepress/plugin-search: ^2.0.0-beta.48 => 2.0.0-beta.48
    @vuepress/plugin-shiki: Not Found
    @vuepress/plugin-theme-data:  2.0.0-beta.48
    @vuepress/plugin-toc: Not Found
    @vuepress/shared:  2.0.0-beta.48
    @vuepress/theme-default:  2.0.0-beta.48
    @vuepress/utils:  2.0.0-beta.48
    vue: ^3.2.37 => 3.2.37
    vue-loader: Not Found
    vue-router: 4.0.12 => 4.0.12 (4.0.16)
    vuepress: ^2.0.0-beta.48 => 2.0.0-beta.48
    vuepress-vite:  2.0.0-beta.48
    vuepress-webpack: Not Found

👎Try not to waste others time.


Update about page and readme to include people who have contributed

I thought it would be good to see if we could list everyone who has helped contribute to the site and show their avatar and name.
Something likes this:

Would be great to see it on the readme and on the about page for the site.

I'll take this one.

Hey @willywos! Im participating in the Hacktoberfest and will do this today!
Best,
Daniel

Welp I merged it thinking that it would render in github differently then what it was showing me in the pull request but it looks a little broken. Could you take a look at the project readme again and submit another pull request? Thanks @inqii for working on this.

Oh snap! I coded it using hackmd.io and there it looked fine. I fixed it now, the problem was that I changed the layout to three columns but used a row break with five columns. Should look fine now!

Yay looks good!


[Task]: Replace usage of TenantRepository.save to TenantService.save in EE

SubTasks
Description

Reduce the reliance on the TenantRepository class, instead rely on the service classes.
Using the TenantService class for the save method will call the cache eviction process for the tenant, which will help in maintaining data consistency between redis and the database. (The addition of tenant information to redis is done in this PR).

Clone EE Issue: https://github.com/appsmithorg/appsmith-ee/issues/4173
Note: This issue in CE is only for tracking.

Closing as the EE PR has been merged to release.
https://github.com/appsmithorg/appsmith-ee/pull/4174


Missing dlls: VCRUNTIME140D.dll, MSVCP140D.dll & ucrtbased.dll

Hey @Jaysmito101!
Very interesting work! I saw the video and wanted to try it on my Windows 10 machine, but when I extracted the zip and started the executable, I only got a cascade of WFCForge.exe - System Errors complaining about missing stuff:

The code execution cannot proceed because VCRUNTIME140D.dll was not found. Reinstalling the program may fix this problem.
The code execution cannot proceed because MSVCP140D.dll was not found. Reinstalling the program may fix this problem.
The code execution cannot proceed because ucrtbased.dll was not found. Reinstalling the program may fix this problem.

I googled the first and apparently the VS redistributable for 2015 is missing. However, I can't install that on my machine, as a newer one is already installed. Any ideas?
System info

WFCForgeversion
v0.1

OS Name
Microsoft Windows 10 Pro

Version
10.0.19044 Build 19044

OS Manufacturer
Microsoft Corporation

System Manufacturer
LENOVO

System Model
20BE00B8MZ

System Type
x64-based PC

Processor
Intel(R) Core(TM) i7-4710MQ CPU @ 2.50GHz, 2494 Mhz, 4 Core(s), 8 Logical Processor(s)

Installed Physical Memory (RAM)
16,0 GB

I'm currently trying to build it on Linux. If you can give me access rights to create a branch I'll create a PR with some details on the requirements to get the development set up.

@LordFckHelmchen hey. Actually the issue is the release exe file 8s the debug version so the debug dlls are missing. I will update the exe soon.
I don't think you need any access to create a pr. Just fork the repository create the branch and create pr with that.

@Jaysmito101 Hey. Did  so. But I'm not sure if it can be done on Linux. So far I get a lot of errors about not implemented functions and windows-specific defines. I'll wait for the running exe.,


Let bindings are not scoped

(Apologies for flooding the Issues)
let x = (
  let y = 4
  in y)
in y

=> y : Natural

Oh wow, that is definitely a bug. Yeah, that should definitely not happen and the intention is that Grace is lexically scoped.


Update Outline icons for dark theme

Fixes #54712
Icons for Class,Field, and Interface should now look correct in the dark theme:

Thanks


Read trailing data on apply

This was removed by mistake during the direct unpack PR refactoring.
Test needed, but this fixes a pull regression
Closes #3569

Codecov Report

Merging #3570 into master will decrease coverage by 5.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3570      +/-   ##
==========================================
- Coverage    42.3%   37.26%   -5.04%
==========================================
  Files         126       84      -42
  Lines       13869    11554    -2315
==========================================
- Hits         5867     4306    -1561
+ Misses       7116     6648     -468
+ Partials      886      600     -286

Flag
Coverage Δ

#linux
?

#windows
37.26% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

archive/tar_opts.go
11.76% <0%> (-47.06%)
:arrow_down:

cio/io.go
1.4% <0%> (-45.08%)
:arrow_down:

snapshots/native/native.go
1.79% <0%> (-41.26%)
:arrow_down:

archive/tar.go
19.18% <0%> (-28.78%)
:arrow_down:

metadata/snapshot.go
23.86% <0%> (-24.05%)
:arrow_down:

content/local/writer.go
57.69% <0%> (-0.97%)
:arrow_down:

gc/scheduler/scheduler.go
66.34% <0%> (-0.97%)
:arrow_down:

oci/spec_opts.go
28.96% <0%> (-0.24%)
:arrow_down:

mount/temp_unix.go

sys/reaper_linux.go

... and 39 more

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 4924bcb...550518a. Read the comment docs.

LGTM

Thanks!


Is it possible to jump to the next sibling from normal mode?

I love jumping around using this plugin with vn but sometimes it selects a node that is longer than the screen, and I can't see the top of it, which is where the definition is. Also, I would prefer to be in normal mode after finishing jumping so it's easier to go and edit things.
Is this something in the cards, or is it out of scope?

It is in the cards. Back then when I was making this plugin I didn't think about jumping around in normal mode. I should be able to add this feature when I get back from my break.

That sounds great, really looking forward to it.
Really appreciate your work :)

Hi @elmehalawi , sorry for the delay :)
Now you can use these commands: STSJumpToStartOFCurrentNode and STSJumpToEndOFCurrentNode.
Also if you use vn, and it jumps you to the end of the node, you can press o to jump to the beginning of the selection.

@ziontee113 Thank you so much for this :)


Platform On Walker Is Glitching

The platform for the walker is glitching pretty severely.
I am working on modifying it to the latest ML-Agents version. Do you know what might be the issue?

I think my platform setup with configurable joints is unnecessarily complicated and fidgety. Apparently I didn't read the Unity docs carefully enough and missed better ways to control the tiles.

Hey mbaske,
There was no issue with your platform it was because of the modifications I was doing on it to have it run on the latest ML-agents.  Everything is working so far and currently training the walking since we can not use the .bytes files anymore. I am planning on creating a pull request once I have modified it to run on the latest ml-agents.
I will also fix the robot-ants pull request once I get time.
I might need some help with the navigation brain and the shoot brain once I get to that stage.
What are the observation spaces for the navigation and shoot brains and how many actions each output?
This will save me some time when I get to that stage.
Keep up the great work.

Hi DroneMesh, Thanks for taking the time to look into this.
The navigator brain only observes visual input and outputs a single continuous value for the walk direction in 2D/XZ space.
The shooter brain observes a proximity value and has one discrete output for shooting / not shooting.
As far as I can tell, the robot-ants repo is already compatible with the current ml-agents release. Only the trained models aren't running on the barracuda inference engine.

Hi mbaske,
Just saw your latest video cant wait to play around with that  new environment.
I have just trained Angry AI to walk and navigatation is currently in progress. I had to convert the bytes to pb then import to tensorboard to see the output and input nodes.. 7 hours later and all worked out great ;) ..
However I am noticing something the shooting brain is not doing a great a job. I have a feeling I should train it after navigation brain matures a bit.
Once I have trained all brains to mimic your video's results I will send a pull request with the updates.
You can close this issue..
P.S Keep up the great work

The original idea was training the brain to aim and shoot based on visual observations. However, I couldn't get that working, especially with an independent brain controlling the walking motion.
Please feel free to make any changes that would yield better training results. Thanks!


Remove unused IdxMap type parameter from ShapedAxis

As I was reviewing the subtypes of AbstractAxis to try to get a better understanding of how ComponentArrays work, I couldn't figure out the point of the IdxMap type parameter in ShapedAxis, so I decided to remove it and see what breaks....
But all the tests pass without this type parameter, so maybe we can just remove it? It seems to me that the index mapping is tracked at the level of the axis containing the ShapedAxis. For example, in
julia> typeof(ComponentArray(a=7, b=rand(2,2)))
ComponentVector{Float64, Vector{Float64}, Tuple{Axis{(a = 1, b = ViewAxis(2:5, ShapedAxis((2, 2), NamedTuple())))}}}

the indices of the matrix are tracked by the ViewAxis, not the ShapedAxis.
I changed the supertype from AbstractAxis{NamedTuple()} to AbstractAxis{nothing} because nothing seems like a better indicator of "there's no index information available".
(I also accidentally removed some trailing whitespace, happy to revert that NFC change if desired)

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (35221bd) 73.92% compared to head (0a18438) 73.21%.

Files
Patch %
Lines

src/show.jl
25.00%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #244      +/-   ##
==========================================
- Coverage   73.92%   73.21%   -0.71%
==========================================
  Files          23       23
  Lines         744      743       -1
==========================================
- Hits          550      544       -6
- Misses        194      199       +5

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Oh wow. I didn't realize we weren't testing this. That's supposed to exist for components that are multi-dimensional arrays of componentarrays. Which, maybe that's also just a thing that nobody needs.

In addition to being untested, I think that behavior has a broken constructor. Specifically, components that are multidimensional arrays of component arrays are automatically flattened to single dimensional arrays of component arrays.
A = fill(ComponentArray(x = 3), 2, 2);
B = fill(4, 2, 2)
C = ComponentArray(A = A, B = B);
(size(A), size(B), size(C.A), size(C.B)) === ((2, 2), (2, 2), (4,), (2, 2))
(eltype(A), eltype(B), eltype(C.A), eltype(C.B)) === (
    ComponentVector{Int64, Vector{Int64}, Tuple{Axis{(x = 1,)}}},
    Int64,
    Any,
    Int64)

Ah. Okay well if it's been broken this whole time and nobody's noticed it, then yeah, let's just get rid of it.


Issues with CanvasNode after recent merge of 0.2 into master

I noticed some new issues with CanvasNode after the recent merge of 0.2 into master.
I first noticed the issue in Molecules and Light.  WindowFrameNode, which extends CanvasNode, no longer appears in the sim.  There are similar cases in Capacitor Lab; objects which extend CanvasNode do not appear.  No further errors or hints for these cases at this time.
There is a new bug in Color Vision with the SingleBulbPhotonBeamNode, which extends CanvasNode.  To reproduce the issue, open the sim, select "Single Bulb" screen, and use the 'Photon Beam' button.  Any further interaction with the sim will result in
"TypeError: Cannot read property 'nextBlock' of null" (GreedyStitcher.js: 708) and
"TypeError: Cannot read property 'toString' of null" (Display.js: 904)
or "Uncaught TypeError: Cannot read property 'trail' of null" (SVGGroup.js: 42)
The HeadNode can get clipped or moved about the screen:

For the Molecules and Light-related issue, I've added an assertion to make sure canvasBounds are non-empty. It looks like at least once the canvasBounds are empty on the CanvasNode.

Reproduced Color Vision error, upgrading to high priority.

Sorry about that, and sorry I missed that in the documentation of CanvasNode.  Bounds have been set, and the issue is resolved for Molecules and Light and Capacitor Lab.

Cannot reproduce as of 1/27/15. @jessegreenberg can you please verify whether this is resolved? If it's not, change back to "high-priority".

The Color Vision error appears to be resolved.  Closing.


Authorization adapters

Add sugar to the moleculer-web, the authorize option in the settings, example:
authorize: {
  type: "bearer", // add social and other adapters
  checkToken: "user.resolveToken"
}

authorize: {
  type: "query", // from url query params or header
  param: "token",
  checkToken: "user.resolveToken"
}

and through the function who needs a custom solution, example:
authorize: (ctx, route, req) => {},

Thus, it will be possible to make a semblance of an authorization adapter and use the modules.
Write your wishes in this issue! :speech_balloon:

good idea 👍


Partial, formal, and total charges are reported with inconsistent units

The current version of the toolkit reports partial, formal, and total charges with inconsistent units:
>> from openforcefield.topology import Molecule
>>> molecule = Molecule.from_smiles('c1ccccc1')
>>> molecule.total_charge
0
>>> molecule.atoms[0].formal_charge
0
>>> molecule.atoms[0].partial_charge
Quantity(value=0.0, unit=elementary charge)
>>> molecule.partial_charges
Quantity(value=array([0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0., 0.]), unit=elementary charge)

Instead, we should presumably adopt a consistent output format (with or without units) for all charge-related quantities.

Good catch. #471 will standardize to report partial and formal charges in units of elementary charge.

I'm also making the setter for Atom.formal_charge accept int or Quantity, since there's a good argument for both. All getters will return Quantity, and all other setters will require Quantity.


[Feature Request] Add an "each" advanced options function

Just like you have the "before" and "after",  I'd also like to request an "each" function which is run each time you iterate over the array of images.   Useful if you want to change the template for every third image or something like that.
I hacked this idea locally for a project.  I put the following snippet right above line #161
if ((this.options.each != null) && typeof this.options.each === 'function') {
              this.options.each.call(this);
 }

Putting it above the imageString area allows me to also changed the template on the fly from within my "each" function
I'm using a grid system where every 3 images are wrapped in a row and I wanted to do that in the template.   This is how I'm using it for now and it's an example of what I'd use the "each" function for:
var count = 0;
var feed = new Instafeed({
    get: 'user',
    userId: ########,
    accessToken: 'XXXXX',
    resolution : 'standard_resolution',
    limit : 9,
    each: function(){
      count++;
      if (/[1,4,7]/.test(count)){
        templateToUse = templateRowStart + myTemplate;
      } else if (/[3,6,9]/.test(count)) {
        templateToUse = myTemplate + templateRowEnd;
      } else {
        templateToUse = myTemplate;
      }
      this.options.template = templateToUse;
    },
    after: function() {
        // disable button if no more results to load
        if (!this.hasNext()) {
            loadButton.setAttribute('disabled', 'disabled');
        }
    },
    template: templateRowStart + myTemplate
});

I'd be onboard with this idea! Seems like something prime for a user-submitted contribution :)
@FranciscoG Are you interested in submitting a pull-request for this idea?

This worked for me using bootstrap... I'm not using the next button, so I took it out.
thanks to @FranciscoG
`    jQuery(document).ready (function() {
    var templateRowStart = '
';
    var templateRowEnd = '
';
    var myTemplate = '

';
    var templateToUse = '';

    var count = 0;
    var feed = new Instafeed({
        get: 'user',
        userId: XXX,
        clientId: 'ZZZZZ',
        limit: 9,
        resolution: 'standard_resolution',

        each: function(){
          count++;
          if (/[1,4,7]/.test(count)){
            templateToUse = templateRowStart + myTemplate;
          } else if (/[3,6,9]/.test(count)) {
            templateToUse = myTemplate + templateRowEnd;
          } else {
            templateToUse = myTemplate;
          }
          this.options.template = templateToUse;
        },
        template: templateToUse
    });
    feed.run();
});`

@stevenschobert
Sure I'll make a pull request at some point today.
@leowebdev
I switched from using the regex test to simple modulus checking.  That way it's agnostic to how many rows it can handle.  For my purpose I was outputting the instagram feed in a 3x3 grid so that's why I was originally only testing using numbers but I think this way is better.
Also be warned that if you're doing columns of 3 that the API only returns a max of 20 which will leave your last row with only 2 so you'll have a
 without a closing
.  You might not notice in Chrome or FF but it'll definitely show in older IE.
// var count =  0
...
         each: function(){
              // moved the count++ below
              if (count % 3 === 0){
                templateToUse = templateRowStart + myTemplate;
              } else if (count % 3 === 2) {
               // everything in the third column (2,5,8,etc) will always have a modulus 3 remainder of 2
                templateToUse = myTemplate + templateRowEnd;
              } else {
                templateToUse = myTemplate;
              }
              this.options.template = templateToUse;
              count++; // increment count here instead of above so we can start at 0
            },

thanks @FranciscoG
I have my function encapsulated as a shortcode for visual composer, so the modulus check is better is perfect as users have the possibility to set the number of images/rows.

@leowebdev
if you want to make number of columns more dynamic for users to choose, you can do something like this
count = 0
columns = 4  // or whatever number you want
if (count % columns === 0) {
  // add opening div class row
} else if (count % columns = columns - 1) {
  // add closing div
} else {
  // middle column, do nothing, drink a beer
}
count++

might want to run a few tests on that to make sure I got that right but I think it should work
@stevenschobert
I'm working on that pull request, unit test keeps failing but I know it works because when I test it in the browser I can see it working.

@leowebdev Cool I like the direction. If you want put up what you have so far in a pull request, I'll help look at the specs.
I'd also recommend we pass the item, index, and full array as parameters to the each option, that way users are not required to track that information themselves.

pull request submitted
https://github.com/stevenschobert/instafeed.js/pull/386


Initial work for setting up opentelemetry package

This PR introduces the opentelemetry package [azure-core-tracing-opentelemetry]  which uses opentelemetry-api for tracing.
TODO's

SpanData wouldn't reflect if it had a remote parent. Issue, Fixed
Opentelemetry packages Jaeger exporters throwing an NPE in some HTTP call scenarios. Issue, Fixed
JaegerExporter not showing HTTP status code or description. Issue, Investigating

Fixes #6114

Can't put a comment on it, did you mean to add the .vscode/settings.json

Can't put a comment on it, did you mean to add the .vscode/settings.json

removed

//CC @weshaggard @mitchdenny
@samvaity there are a couple of things wrong in this PR.

We should not be taking dependencies on SNAPSHOT versions, there are two in here io.opentelemetry:opentelemetry-api and io.opentelemetry:opentelemetry-sdk. We could not release with these dependencies. Further, I believe that unless azure-core-tracing-opentelemetry's version was also -SNAPSHOT, that Maven would reject the package
We should not be adding  tags to the individual component files. This overrides the repository settings defined further up in the hierarchy and bypasses our cache.

There's also one other big issue with the SNAPSHOT version dependencies  is that Maven is going to try and query to pull these every time which will cause transient errors. I'm actually running into a transient error on my local machine because it this which built earlier and is failing now.


Calls to classify end point seem to use 1 API call per string not per list of strings

If I understand the "classify" API documentation correctly, I should be able to pass multiple text strings in a single call and only use 1 API call, correct? I'm basing this on

This endpoint allows you to perform the classification of many text samples using only one request to a custom or public module.

Assuming my understanding is accurate, I am seeing different behavior. Instead of a single API call for a list of text strings, I see one API call per text string.
This is my sample code
from monkeylearn import MonkeyLearn
import user_settings

MONKEYLEARN_API_KEY = user_settings.KEY
MODULE_ID = user_settings.MODULE_ID

ml = MonkeyLearn(MONKEYLEARN_API_KEY)
res = ml.classifiers.detail(MODULE_ID)
samples = ['this is a string',
           'a dog barks on tuesday',
           'i like purple muffins',
           'apples are funny fruit',
           'the house is up a mountain',
           'cats do not like trees',
           'computers will take over',
           'you sank my battleship',
           'duck, duck, goose',
           'yellow baby buggy bumpers']

res = ml.classifiers.classify(MODULE_ID, samples, sandbox=True, debug=False)
print("Quota Remaining")
print("-"*20)
pprint.pprint(res.query_limit_remaining, indent=4)

Two runs, back to back, of this print the following:

RUN 1:
Quota Remaining
--------------------
'99940'

RUN 2:
Quota Remaining
--------------------
'99930'

Notice that the remaining quota is 10 apart between runs. This is the same number of sample text strings. If I comment out 5 of them and run it again, I get this output:

Quota Remaining
--------------------
'99915'

To me, this is confirming that I am using 1 API call to the classify endpoint per string, not per group of strings.
Is this the expected behavior, or should my list of 10 strings only be costing 1 API call?

Hi! This is the expected behaviour. Each classification costs 1 query. The difference between sending a list of texts instead of sending one text at a time is that less HTTP requests are performed in the former and the classification works way faster.

Thank you for the clarification.


AMP Review 2

[x] Update titles and links in README.md section on "Deploying on CML"
[x] Apply nb_black code formatter

Note - I had to also !pip3 install ipython --upgrade for this to work
I also deleted the %load_ext nb_black cell from the notebook after applying formatter

[x] Saved the notebook with all cell output visible

I selectively cleared some of the noisy outputs though (ex. tqdm progress bars)
This is how we did it with spaCy AMP... any objections @cjwallace ?)

[x] Update H1 title in notebook
[x] Change final repo name to "CML_AMP_Explainability_LIME_SHAP" (@cjwallace feel free to push back if you feel strongly otherwise?)

@victordibia Thanks for the great content in this notebook, super helpful for anyone wanting to learn about LIME or SHAP!
@cjwallace With the approval of these changes, I declare this AMP reviewed. I'll leave it for you to actually merge pending any changes you might want to add.
Thanks!

@victordibia Thanks for the great content in this notebook, super helpful for anyone wanting to learn about LIME or SHAP!
@cjwallace With the approval of these changes, I declare this AMP reviewed. I'll leave it for you to actually merge pending any changes you might want to add.
Thanks!

I ran into a problem on CDP public where a dependency couldn't be installed, but re-running the cell worked. I think it was a temporary resource error (full cluster, I guess 🤷 ) so I don't think we need to worry about it. Apart from that, looks good, approved!

I ran into a problem on CDP public where a dependency couldn't be installed, but re-running the cell worked. I think it was a temporary resource error (full cluster, I guess 🤷 ) so I don't think we need to worry about it. Apart from that, looks good, approved!

Thanks all! Much appreciated!

Thanks all! Much appreciated!


[FlutterEngine] Android BroadcastReceiverControlSurface#attachToBroadcastReceiver declares useless parameter, Lifecycle

BroadcastReceiverControlSurface declares a method: void attachToBroadcastReceiver(       @NonNull BroadcastReceiver broadcastReceiver, @NonNull Lifecycle lifecycle);
FlutterEngineConnectionRegistry implements it, but unfortunately it doesn't use lifecycle at all:
  @Override
  public void attachToBroadcastReceiver(
      @NonNull BroadcastReceiver broadcastReceiver, @NonNull Lifecycle lifecycle) {
    Log.v(TAG, "Attaching to BroadcastReceiver: " + broadcastReceiver);
    // If we were already attached to an Android component, detach from it.
    detachFromAppComponent();

    this.broadcastReceiver = broadcastReceiver;
    this.broadcastReceiverPluginBinding =
        new FlutterEngineBroadcastReceiverPluginBinding(broadcastReceiver);
    // TODO(mattcarroll): resolve possibility of different lifecycles between this and engine
    // attachment

    // Notify all BroadcastReceiverAware plugins that they are now attached to a new
    // BroadcastReceiver.
    for (BroadcastReceiverAware broadcastReceiverAware : broadcastReceiverAwarePlugins.values()) {
      broadcastReceiverAware.onAttachedToBroadcastReceiver(broadcastReceiverPluginBinding);
    }
  }

@matthew-carroll, you added a TODO saying: resolve possibility of different lifecycles between this and engine attachment. BroadcastReceiver's aren't LifecycleOwners, there is no way to get a lifecycle, unless I create a my own, which would be pretty useless for Flutter IMHO. This was your PR: https://github.com/flutter/engine/pull/8962 (code)
Can you confirm this was a mistake. You may have copy and pasted the same comment from another implementation: attachToContentProvider:
  public void attachToContentProvider(
      @NonNull ContentProvider contentProvider, @NonNull Lifecycle lifecycle) {
    Log.v(TAG, "Attaching to ContentProvider: " + contentProvider);
    // If we were already attached to an Android component, detach from it.
    detachFromAppComponent();

    this.contentProvider = contentProvider;
    this.contentProviderPluginBinding =
        new FlutterEngineContentProviderPluginBinding(contentProvider);
    // TODO(mattcarroll): resolve possibility of different lifecycles between this and engine
    // attachment

Solution
I think we should create a function overload with only 1 parameter: BroadcastReceiver, and deprecate this existing method. Or even just remove it completely, because it never made sense for BroadcastReceivers.
I'm happy to create a PR for this.
More information about Lifecycle

Defines an object that has an Android Lifecycle. Fragment and FragmentActivity classes implement LifecycleOwner interface which has the getLifecycle method to access the Lifecycle. You can also implement LifecycleOwner in your own classes.

cc @blasten

@matthew-carroll has replied to me on Discord, and I think it's useful to add his reply here:

I don't remember why I did that. It may have been a mistake, or it may have been intentional for reasons that we'll never know. At this point, whatever the Flutter org wants to do with that code, they should do. Just make sure you take into account any interactions that are unique to Flutter. Flutter has to track all these components to make them available to plugin authors.

@ben-xD what are you trying to do? why is the lifecycle parameter an issue in your case?
If there's no use case for this parameter, then it's technical debt, and should be removed following the Flutter deprecation guidelines.

Since Flutter provides a way for plugins to listen to broadcast receivers attachment/ detachment, I want to make my app a good citizen in Flutter by using BroadcastReceiverControlSurface#attachToBroadcastReceiver. This allows the plugins to be notified when a broadcast receiver attaches/ detaches if they implement BroadcastReceiverAware.


Add page_after_footer block

The block is surrounded by a div element because it should have a
clear: both style property to avoid overlaping with navigation anchors.
This should fix #17 .

thanks!


Create an SAP Fiori App Using SAP Business Application Studio

Tutorials: https://developers.sap.com/tutorials/appstudio-fioriapps-create.html
The overall project template is not working as step 4 and further in "https://developers.sap.com/tutorials/appstudio-fioriapps-mta-build-deploy.html"
Ponit 1 :
Step 4.4 : cf html5-list -di FioriDemo-destination-service -u   This command doesnot return the project url  ,steps to create project template is not correct . Due to this we cannot continue with next stpes
We are blocked for the validation .

Labels to add:
High-Prio
SCPTest-2109A
SCPTest-fbizapp
SCPTest-trial1
SCPTest-trial2
SCPTest-trial3
Thanks and Regards,
Vanita

Hi @VanitaDhanagar ,
Thanks for sharing.
A fix was deployed.
Be aware that a major update for this tutorial group is planned to be published next week,
Regards - Raz


Option for setting instanceId.

We have the need to run multiple instances of the same service on the same host machine.
This is not possible then the InstanceId field is automatically defined using the hostname.
I would like the option to set the instanceId field in the configuration array for the Eureka client.

Does it sound reasonable to set it under instance.id? Then we would pick the instanceId by first checking if that is defined, before defaulting to the hostname.
Looping in @jquatier for eureka knowledge.

Yeah, I think the question would be whether this is in a AWS environment or not, and what should we do in that scenario. Currently, if its AWS it will pull the instance ID from the AWS metadata. I guess to cover both AWS an non-AWS you would want to do something like this:
get instanceId() {
    if(this.config.instance.id) {
      return this.config.instance.id;
    } else if (this.amazonDataCenter) {
      return this.config.instance.dataCenterInfo.metadata['instance-id'];
    }
    return this.config.instance.hostName;
  }

Yes. That that would solve the problems we have. :+1:

I'll write up a PR later today for this.
On Sat, Mar 5, 2016 at 1:41 PM Frederik Mogensen<EMAIL_ADDRESS>wrote:

Yes. That that would solve the problems we have. [image: :+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/jquatier/eureka-js-client/issues/42#issuecomment-192747242
.

The option is available in v2.3.0 of the module. Update your package.json and try it out. Thanks @kesne for getting this in.


feat: added ELU metrics

Motivation
Add extra metrics related to Event Loop utilization
Description
Relies on node:perf_hooks to track Event Loop utilization metrics: Adds the following histograms:

utilization akin to CPU utilization, but for ELU; between 0 and 1
idle how long the ELU was idle since last poll (every 5s by default)
active how long the ELU was active since last poll (every 5s by default)

Those metrics are tracked on main thread and currently monitored worker threads.

I think it's not as useful to track this as an average over time or at least I don't see which questions this helps to answer.
What we might be interested in is the event loop utilization (ELU) during a certain time frame of the slot, e.g. 1-4, 4-8, 8-12, or even on a per second basis. Should be able to achieve this via labels on metrics + running an interval on a clock timer.
This could help us analyze what parts of the slot we are overloaded, and if a change improves the ELU for a specific slot frame or not.

can be further discussed in https://github.com/ChainSafe/lodestar/issues/7147


While reading long number in excel it is converting to exponential format

While entering long number in excel it is converting to exponential format (it is fine, behind data is still long number).
But, while reading data in typescript. it is just reading exponential number, and data after some digit are lost and zero is appended.

The cell v property has the original numeric value and the w is the formatted text.  Share some code and we can advise on how to get the values

I have -<PHONE_NUMBER>1234 in excel.
var workbook = XLSX.read(inputBuffer, { type:"binary", cellDates: true });
var worksheet = workbook.Sheets[workbook.SheetNames[0]];
var array = XLSX.utils.sheet_to_json(worksheet, {raw: true});
particular cell is read as (worksheet) :
{ t : 'n', w:'1.23457E+13' v:12345700000000 }

That is definitely unexpected, can you share the file?  If you can't do it publicly, you can email<EMAIL_ADDRESS>

Shared details via mail.

these is some issue in the excel. while uploading the fresh excel with data. then data is getting read properly.
please share the excel If you can with dev.

It happened when I was using .csv file, but it was okay for .xlsx file.

if ur using sheet_to_json method or function use in body raw: true :
this.importData= XLSXSample.utils.sheet_to_json(wb.Sheets[sheet_name_list[0]], { raw: true,});


Support for Intel Quicksync improve encoding by 33% (on supported hardware)

When adding the Intel Quicksync encoders to 'encoder.py' (i.e. the list of allowed encoders), rendering was sped up by 33% according to a test done by a user in the discord server who asked for this and I added it to my fork. He processed a 5 minute video, without this code it took 300 seconds. With this code it took 200, as reported by him. I added the following lines to the 'encoder.py' file.
    "hevc_qsv": {
        "pix_fmt": {
            "nv12",
            "p010le",
            "yuyv422",
            "y210le",
            "qsv",
            "bgra",
            "x2rgb10le",
        },
    },
    "h264_qsv": {
        "pix_fmt": {
            "nv12",
            "p010le",
            "qsv",
        },

I have attached a copy of this updated file.
encoder.zip
.

I have added those encoders. In the future, you should make a pull request so I can apply these changes with one click.


Mergin - version tracking for Peace 2024

Initial Mergin tracking for Peace 2024 (version 125-136) can be found here https://github.com/NewGraphEnvironment/fish_passage_peace_2023_reporting/issues/46
Moving forwards, version tracking will now be tracked in this issue.

v138 - updating the site UTMs after some sites were moved as per https://github.com/NewGraphEnvironment/fish_passage_template_reporting/issues/48#issuecomment-2549271107


fix(uiSelectCtrl): open dropdown when clicking on caret

Closes #1797, #1862

Thanks for the pr will have a look at it and provide feedback if needed.

Thanksfor the PR @talitagoulart


Images for README

Hi,
I have tried to integrate this library . So do you have a sample project which installed this library or good documentation ?
Thanks


New rule: Ban functions that throw?

new URL("asdf") (see https://github.com/danielnixon/readonly-types/blob/master/src/index.ts#L75)
decodeURIComponent('%')
"".normalize("asdf")
others? https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects

new URL("asdf") is covered by the new no-partial-url-constructor rule.

Throws if passed a "lone surrogate" (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String#utf-16_characters_unicode_codepoints_and_grapheme_clusters):
> encodeURI('\ud83d')
Uncaught URIError: URI malformed

> encodeURIComponent('\ud83d')
Uncaught URIError: URI malformed

Debatable:
> parseInt('asdf')
NaN

> new Intl.Locale("asdf")
Uncaught RangeError: Incorrect locale information provided

JSON.stringify({a: 1n})
Uncaught TypeError: Do not know how to serialize a BigInt

> JSON.parse("");
Uncaught SyntaxError: Unexpected end of JSON input

> const foo = new RegExp("[asdf");
Uncaught SyntaxError: Invalid regular expression: /[asdf/: Unterminated character class

although this one is covered by https://eslint.org/docs/latest/rules/no-invalid-regexp

> escape('\ud')
escape('\ud')

Uncaught SyntaxError: Invalid Unicode escape sequence


Error: Building package sdl2:x86-windows failed with: BUILD_FAILED

Package: sdl2:x86-windows
Vcpkg version: 2019.06.21-nohash
Just trying to do the basic things from readme. It's probably some issue with my configuration.
Log:
https://gist.github.com/Predelnik/8e4ebb8f6855e9749fdd9c9998b856f7
Basically cmake tests fail with

LINK : fatal error LNK1104: cannot open file 'ucrtd.lib'

I have tried updating cmake, setting DCMAKE_SYSTEM_VERSION manually in triplet, installing specific SDK version which I found in some other issue as supposed solution, nothing seems to have any effect.
Running cmake standalone, it passes tests for msvc 2019 just fine. What am I missing here?

CMAKE_SYSTEM_VERSION stopped working, apparently something changed
But  VCPKG_CMAKE_SYSTEM_VERSION doesn't work the same way
https://github.com/microsoft/vcpkg/blob/master/docs/users/triplets.md
SDK is selected for some reason that last installed

@Voskrese gave me an idea. I noticed that I have "windows driver kit" installed  which has higher version than rest of normal kits for some reason in my case it was 10.0.18362.0 and my latest currently installed sdk is 10.0.17763.0.  This driver kit seems to miss ucrt stuff. After uninstalling it everything seems to work. Maybe this is a rare case but I still think that this is valid issue which other people might also face it in some cases and find puzzling, feel free to close this but I think that proper solution for configurations such as this also might need to be implemented.

I hit this error and found that that it was using a different version of CMake.  Could you start a clean cmd and allow vcpkg to download its own CMake.

I definteily had separate cmake install but I'm not sure if vcpkg was using or not, it was one of the latest vesions anyway.
The problem is that I already fixed this as I described in the comment so I can't reproduce it now, but I strongly believe that removing windows driver kit made this issue solved. Maybe someone should try to install it and see if it is reproduced.

I tried but I can't repro this issue, the ucrt library installed correctly for all window sdk revision for me.


update.py:23: Перенести классы в отдельные файлы 10

The puzzle 899-c5b76c1c from #899 has to be resolved:
https://github.com/blablatdinov/tg-quranbot/blob/3c83a562066717543817037d6f58540869e1060f/src/app_types/update.py#L23-L23
The puzzle was created by Ilaletdinov Almaz on 20-Jun-24.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 899-c5b76c1c has disappeared from the source code, that's why I closed this issue.


Implement --prompt / --force flags for adding new servers

Servers can be added via the iml CLI with a command like:
iml server add --hosts mds[1,2].local,oss[1,2].local --profile base_managed_patchless

However, if the command fails, the user has no recourse and must deploy servers via the GUI instead.
Instead of that, we can add two new mutually exclusive flags to the iml server command.
--prompt
This flag will ask for confirmation to continue deployment if a command returns with an error during server deployment. The user must type y to continue, or n to abort. The command failure should be displayed so the user know what they are agreeing to.
--force
This flag will move to the next step in the deployment process regardless of the Command failure. Even though the command has failed, we should still print what the error was before continuing.
The code for the server CLI lives here:
https://github.com/whamcloud/integrated-manager-for-lustre/blob/25587229ee99e419fd8083469ab78436641c773c/iml-manager-cli/src/server.rs

Build failed:
   Compiling termion v1.5.4
error[E0433]: failed to resolve: maybe a missing crate `sys`?
  --> C:\Users\Basil\.cargo\registry\src\github.com-1ecc6299db9ec823\termion-1.5.4\src\lib.rs:24:9
   |
24 | pub use sys::size::terminal_size;
   |         ^^^ maybe a missing crate `sys`?

Adding sys = "*" to Cargo.toml doesn't help.
If this is the known trouble, please tell the solution.

Which command errors should be ignored?
I see at least four points of potential failure:

preflight checks
wait_for_cmds(commands).await? after "agents..."
wait_till_agent_starts(&hosts, &profile.name).await?;
wait_for_cmds(commands).await? after "host profiles..."

Is it possible to run server cli under Windows?

I don't think so.

If not, can you give me an access to Unix machine where I can build and run that cli?

Local Linux VM would be much easier I think.

If you just want to build locally, you can run the following command inside the integrated-manager-for-lustre dir (with docker installed / running):
docker run -it -e SPEC=“rust-iml.spec” -e LOCAL_ONLY=“True” -e CARGO_HOME=/build/.cargo -v $(pwd):/build:rw imlteam/copr-rust:stable
This will build / produce an RPM within the _topdir directory structure.
In order to test your changes you can use https://github.com/whamcloud/Vagrantfiles (this brings up a cluster of VMs, which can be used to build as well).

As Docker itself can be difficult on Windows, I recommend installing CentOS 7 or Fedora 31 on VirtualBox or VMWare and developing there. Then docker can be run from inside the VM to check the full build if you want that, although the same happens in PR CI so you'll be able to see build failures anyway.
So developing and running CLI in Linux VM is probably enough for the task.

Good, thank you. I have built it under Windows by commenting termion lines, so build is not a problem.
But I still need a guidance which error should be ignored.
The issue tells "move to the next step in the deployment process regardless of the Command failure". What is the Command - the whole command "iml server add"? Some part of add command, like
preflight checks
wait_for_cmds(commands).await? after "agents..."
wait_till_agent_starts(&hosts, &profile.name).await?;
wait_for_cmds(commands).await? after "host profiles..."
or something else?

@Basil22 There are two spots:

If any of the preflight checks have failed:

https://github.com/whamcloud/integrated-manager-for-lustre/blob/00ec637c3c54de10d26700e13fab5bc2ad7350a9/iml-manager-cli/src/server.rs#L302-L306

if any of the host profiles are invalid (this is currently being filtered out):

https://github.com/whamcloud/integrated-manager-for-lustre/blob/00ec637c3c54de10d26700e13fab5bc2ad7350a9/iml-manager-cli/src/server.rs#L349-L363
I also have a patch in flight that removes termion here:
https://github.com/whamcloud/integrated-manager-for-lustre/pull/1426
I think this should allow you to cargo check the iml-manager-cli crate individually if you base your changes on it.

Great! Thank you for the patch. CLI compiles under Windows now. And also thanks for spots, I'll continue work on issue.