Include Organize and Secure methodologies to operating model

When discussing the CAF with customers, I've used the attached images to demonstrate how the Strategy, Plan, Ready, & Adopt methodologies are 'iterative' processes which the organization should perform with each workload migration. The 'continuous' activities are Organize, Govern, Manage, and Secure which the organization will perform regardless of the workload migration process. The second image demonstrates how the CAF methodologies align to the organizational (Business), Platform (Landing Zone), and Workload (Application/Service) domains. This approach is the basis of the new Defense CAF scenario in review for CAF publication.
Highly recommend inclusion of these concepts into the operating model discussion.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: eb736919-3318-849a-f18d-0ebe09936203
Version Independent ID: 7970d251-13ec-f1e1-8102-ffdf9346b88e
Content: Define your cloud operating model - Cloud Adoption Framework
Content Source: docs/operating-model/define.md
Service: cloud-adoption-framework
Sub-service: general
GitHub Login: @BrianBlanchard
Microsoft Alias: brblanch

Thank you for your feedback.  We have created internal ticket 37617 to track and investigate this issue.

#please-close


Media Profiles filtering the Sender's EDID may not be sufficient to cover all use cases

Reviewing this in in engineering, they brought a concern to my attention
When there's

A Monitor with a wide screen timing (not found to the Sender's EDID)
A number of monitor where is one is more sensitive than the others, such that's it's EDID timing should be used as is  (not found to the Sender's EDID)

➡ In this case, we need to pass the raw EDID of the monitor and applying the media-profiles filtering on this "override" EDID.
Additionally we noted, when a new version of EDID/DisplayID is released and is not known to the device, it would be possible to pass this new EDID via an extended API would be an solution to this problem... Though we would also need to update our BCP(s) in that case.

In order to avoid aggressively changing the source, it would be advantageous to expose everything in one API call.

PUT /media-profiles

{
   "edid": "0x129684681...", // null | "auto" return to "initial state" (factory)
   "profiles": [
      {
          // ...
      },
      // ...
   ]
}

Two possible solutions:

Improve Sink schema to express more necessary info from EDID.
Expand Sender with an endpoint which takes raw EDID (hard to describe exact behavior).

I think we all agree ideally we could round trip the EDID but is that achievable? This concept is not new and is used throughout the industry, so what merits does it offer?

1. schema to express more necessary info
Maybe utilize Sink in addition to Receiver Caps

Since the devices covered by IS-11 will almost exclusively be gateway devices (ST2110 <--> HDMI), it does not need to know all the possible settings.
↪ The desired behavior, today in ProAV, is to let the source adjust it's configuration (color, image size, etc) to the monitor. These settings are not properties of the stream, or NMOS, so the device would not need to change its Flow to transmit different content.
Another use case which limits the feasibility of this is proprietary EDID extensions... If a company makes a HDMI player it could pass extra setting within the EDID to the Monitor... If our devices are used as the Gateways, there's no convincible way for spec that in a schema.
The alternative proposed would required BCP-005-01 to change very frequently, by allowing the raw EDID new EDID feature and/or extension blocks could be "added" to the gateway without require an update and new specification and upgraded controllers.
Lastly, the amount of combinations is already an issue, we have https://github.com/AMWA-TV/nmos-edid-to-receiver-caps-mapping/issues/56 which is just a variant of #44 and we saw this in Pebble's demo... At which point does the media profile become to large for low-powered device to solve for?

2. Expand Sender with an endpoint which takes raw EDID
If accept EDID in Sender, let's have a separate endpoint for this.

Having a separate endpoint greatly complexifies the implementation since it does not know when to signal to the source when to change... ⏬ How does the device know when the controller finished configuring it?

Separate endpoint's would require an "activate" command, similar to IS-05 where information can be staged and activated explicitly. IS-08 matches what we have currently and what we'd love to see added.

Having a single endpoint has a few advantages:

single step configure and active
EDID and Media Profiles are always set in tandem (I thinks it's very explicit this way, no "oh I forgot what was there last time" problems)
🆕 DisplayID is an extension and is transported within an EDID such that this mechanic is durable (with current technologies)

When there's
1. A Monitor with a wide screen timing (not found to the Sender's EDID)
2. A number of monitor where is one is more sensitive than the others, such that's it's EDID timing should be used as is  (not found to the Sender's EDID)

Could you explain these cases in detail please? Does item 1 describe >4K monitors?

I think we all agree ideally we could round trip the EDID but is that achievable?

I think we don't need to cover the whole EDID but read User Stories and provide a common and transparent solution which can use EDID or anything similar as underlying component. From my point of view, proprietary EDID extensions are out of scope. If they describe information covered in BCP-005-01, they are closer to the scope and vendor can stay with IS-11 but handle such extensions itself.
Regarding separate endpoints for Media Profiles and EDID, the idea behind it is proposing two alternative ways to configure Sender. PUT /media-profiles -> Sender's configured, then PUT /edid -> Sender lost its Media Profiles and is configured with the EDID.
Passing raw EDID instead of Media Profiles breaks two use cases:

Multicast connections (What gamma, display_size etc. should Sender choose? How to calculate intersection?)
User's intention when selecting Media Profiles (with the current spec it's possible to build a Controller which presents the intersection of Receivers' operating points with checkboxes to let user choose they want the highest resolution or the least bitrate or anything else).

Could you explain these cases in detail please?

As a ProAV user.... based on 1
I have connected a source and monitor which is a 16:10 display. The image looks great! 🖼 I need to connect them over my IPMX network. And setup the following:

The need for media profiles comes in when I have two different sized screens which need to show the same content.
If I buy a 16:10 capable encoder gateway who's EDID by default includes a dozen resolutions for 16:9 (missing the wide 16:10 resolutions) ➡ Because of the limited nature of an EDID means I am not able to describe all the capabilities of the device.
Assuming a perfect decoder gateway, there's a few spots this can go wrong

The Sender rejects the request
The Source begins working in a non-native, unpreferred and lower quality manner than is anticipated
The Receiver or Sink adds black bars to the sides to compensate to the smaller image

The only way for my installation to work is to "override" the encoders EDID with another one (most sensibly that of the Sink)... Having the flexibility in IS-11 would be hugely advantageous

Regarding separate endpoints for Media Profiles and EDID, the idea behind it is proposing two alternative ways to configure Sender. PUT /media-profiles -> Sender's configured, then PUT /edid -> Sender lost its Media Profiles and is configured with the EDID.

Without media profiles, we break the use case of having multicast between different monitors as you point out. The workflow in https://github.com/AMWA-TV/nmos-sink-metadata-processing/issues/32#issuecomment-876638260 is just as important with multicast, we still have the need for profiles even with the raw EDID.
↪ Both the EDID and the Profile covers the largest intersections of use cases

2. User's intention when selecting Media Profiles (with the current spec it's possible to build a Controller which presents the intersection of Receivers' operating points with checkboxes to let user choose they want the highest resolution or the least bitrate or anything else).

Again keeping both is key. The user should (is he so chooses) can select the EDID to which the Media Profiles are applied to. Currently there's an EDID involved between the Source and Sender but the user is completely unaware of what that is.
By exposing it in the API the users could make a better selection in terms of profiles which are relevant.

I have two major points to add to this conversation:

I still think we should leave the /media-profiles endpoint alone and have it as the plug & play way of doing things and add the new EDID file as a separate endpoint. The way I see it you will only use this to adjust particular scenarios when the plug and play behaviour has resulted in sub par results.

I think we need to consider the future viability of having something like raw EDID being passed around. How will this work with DisplayID? Should this new endpoint be a /metadata_file endpoint where we can signal its type via the Content-Type header?

From our meeting notes...

Since Media Profiles may be not enough to get what baseband connection would give, /media-profiles could be renamed and have root properties media_profiles (required) and metadatafile (optional). If both are passed, metadatafile is applied to Sender by vendor and then filtered with the Media Profiles.

We have discussed two options for passing Sink Metadata Binary to Sender:

A separate endpoint with PUT method
/senders/{senderId}/media-profiles expansion

Advantages of option 1 are disadvantages of option 2 and vice versa so let me analyze only one of them.
Pros of the separate endpoint:
a. It separates Sink Metadata Binary and Media Profiles logically and increases flexibility. You can put Profiles and don't want to use the Binary at all. You can put Profiles and adjust video putting Binary then. You can put Binary of a single or main Receiver and then select a desired mode putting Profiles. If error, you know what of them is incorrect even without reading a server response.
b. It lets user PUT Binary without any additional steps (base64 encoding, insertion into JSON) which simplifies implementations and makes processing fail risks lower.
Cons of the separate endpoint:
c. If using Profiles and Binary together is the main use case, then it multiplies work by two.
Also, a question. Does Receiver ID and Sink ID of Binary passed to Sender really matter? Is this bad if Controller puts a Binary from a USB stick as we discussed before?

To counteract some of the cons for having one endpoint, what if we used PATCH method instead

GET Currently active settings
PATCH modify the current setting to match this subset of settings (similar to IS-05)

This permits our two most common scenarios.

Setting both the EDID and Media Profiles we creating new connections
Changing only the Media Profiles when adding/removing Receivers

I do not mean JSON Patch the RFC 6902 and application/json-patch+json.
I mean it in a similar way IS-05 describes it
https://specs.amwa.tv/is-05/releases/v1.1.1/APIs/ConnectionAPI.html#body-13
https://specs.amwa.tv/is-05/releases/v1.1.1/docs/2.2._APIs_-_Server_Side_Implementation.html#staged-requests-precedence
Where properties replace one another
Perhaps in our case we just need a line that says "the array of Media Profiles replaces the existing content in it's entirety" or it's for the all the top level properties... I am sure we can add a test to the test-suite for to ensure interoperability there since it's an implementation detail


bidirectional?

Hi, thanks for your implementation. I wonder if the minGRU will support bidirectional setting?

@Xiaobin-Rong hey Rong, you can easily achieve this with torch.flip
what is your use case?

@lucidrains  Well, I consider sequence modeling cases, such as translation. In translation, the current token is dependent not only on past tokens but also on future ones. Thus, a bidirectional setting will enable the minGRU to predict the current token while considering future information.

@Xiaobin-Rong yea sounds good, make sense
you can just use torch.flip then

@lucidrains  ok, thanks for your response


[nfs-server-provisioner] Can't be installed on Kubernetes 1.16

Describe the bug
stable/nfs-server-provisioner cannot be installed on Kubernetes 1.16.
Version of Helm and Kubernetes:
Helm 2.15.1
Kubernetes 1.16.2
Which chart:
stable/nfs-server-provisioner
What happened:
Tried to install stable/nfs-server-provisioner.
$ helm install stable/nfs-server-provisioner --set persistence.enabled=true,persistence.size=5Gi
Error: validation failed: unable to recognize "": no matches for kind "StatefulSet" in version "apps/v1beta2"

What you expected to happen:
nfs-server-provisioner to be installed.
How to reproduce it (as minimally and precisely as possible):
Run helm install stable/nfs-server-provisioner --set persistence.enabled=true,persistence.size=5Gi against a Kubernetes 1.16 cluster.
Anything else we need to know:
Some of the APIs got moved around/deprecated in Kubernetes 1.16.

Duplicate of #18372

Duplicate of: #18372
There is already a PR for this: #18509


Remove assert.NilError use assert.Assert

I noticed that the old Comparison interface was basically error, so this is an experiment to see if it makes sense to remove NilError and just use Assert directly. The error message is unchanged.

going to close this for now since #55 is merged. We can revisit later.


Get intellij recognize module structure

Hello there,
did someone made intelliJ understand the lib/ folder so that require('submodule') gets code completion?

This should be fixed by adding "lib" as a source folder in your project. I actually tried it and IntelliJ was still not auto-completing. No idea why, to be honest, it should have worked.
Maybe a bug in IntelliJ/WebStorm?


Boost extension

This PR adds a new boost inline macro to create links to a corresponding Boost C++ library. It also includes utility functions to convert strings from and to PascalCase and snake_case. These functions ensure the links are always correct in the https://www.boost.org/libs/my_lib pattern for the URL while the title is always in the canonical Boost.MyLib format used in boost. It also supports target attributes, which replace Boost.MyLib with whatever text is provided as a positional attribute, making it compatible with asciidoc macros that generate regular URL links.
The PR also includes a fix to the scripts that were preventing previews to be generated correctly. For instance, #49 has new content that is not reflected in its preview https://49.site-docs.prtest2.cppalliance.org/index.html because of a previous problem in the interaction between the scripts and the boost extension, that would make antora fetch and render the contents of the develop branch regardless of the contents of the current branch even when developing locally and using the shell scripts. This is why #49 renders the develop branch instead of the content in Peter's PR. The reason this bug might have been unnoticed for such a long time before causing us problems is probably that many people have been using their develop branch for almost everything at these early stages of development.
The commits in this PR are on top of Peter's #49 since it includes content we can use to test the scripts and visualize the result in the preview for this PR. In other words, there should be a History of Boost link in the sidebar, which indicates the fix worked. Peter's commit can be dropped from this PR before merging it to make the PRs independent.

Confirming that this solves #55.


All template document links need to be reviewed

This is a team effort task:

Ensure that lastest version of each template has a document link portal item.
If new version of template was created since last Ops Server release then update existing link item to reflect new template.
Any references to "Military" or "Intelligence" folder in the document link URL needs to be changed to "ArcGISforDefense"

NOTE: All templates that existed in the ArcGISforMilitary or ArcGISforIntelligence folder will be moved into a folder called "ArcGISforDefense" on the Ops Server release USB drive.
NOTE: URL should be in the form of:
https://"server"/Downloads/ArcGISforDefense/"template_name.zip"
where:
"server" is the fully qualified name of our production portal.
"template_name.zip" is the actual name of the template zip file.

If there are any that are missing Document links I'll volunteer to add as many as I can.

Eric will give me access to template folder on server, I will compare list of template ZIPs to document links and see what needs to be created.

the plan:

Starting by getting a list of all template ZIP files on the server into Excel (211 ZIP files)
Filter the list by the latest version (there are multiple versions of the templates listed -- 75 ZIP files )
List to portal search for "type:Document" (ouch .... only 35 matches)
Check each item in step 3 and add the document link to Excel document
Look at the difference between latest version and document links, this will be the working list
Add working list to this Issue

@mfunk Thank you

WORKING LIST
THE FOLLOWING NEED FULL DOCUMENT LINK CREATED:

[ ] CompoundMap5.0.0.zip
[ ] CoordinateConversionArcGISPro1.0.1.zip
[ ] CoordinateConversionArcMap1.0.1.zip
[ ] CoordinateConversionWAB1.0.0.zip
[ ] DistanceAndDirectionArcGISPro1.0.0.zip
[ ] DistanceAndDirectionArcMap1.0.0.zip
[ ] HelicopterLandingZone_4.0.0.zip
[ ] ImageChangeDetection2.0.1.zip
[ ] ImageryBasemap3.0.0.zip
[ ] ImageryCatalog_Web_1.0.0.zip
[ ] IncidentAnalysis5.0.0.zip
[ ] MilitaryAspectsofTerrain_1.0.0.zip
[ ] PatrolDataCapture3.0.2.zip
[ ] PointofOriginDetection5.0.1.zip
[ ] PredictiveAnalysisWeb1.1.0.zip
[ ] RangeCards2.0.0.zip
[ ] ScannedMapsBasemap3.0.0.zip
[ ] Situation_Awareness_1.0.0.zip
[ ] SunPositionAnalysis_1.0.0.zip
[ ] TimeSpanAnalysis2.0.0.zip
[ ] VisibilityArcGISPro1.0.0.zip
[ ] VisibilityArcMap1.0.0.zip

THE FOLLOWING HAVE LINKS BUT NEED TO BE UPDATED:

[ ] BeachLanding4.0.0.zip

This has a document link that  points to another portal item, not the ZIP

[ ] ClearingOperations5.0.1.zip

Current link points to v4.0.0

[ ] DistancetoAssets2.0.1.zip

Current link points to v2.0.0

[ ] GeoMessageSimulator1.0.0Windows-x64.zip

Need to add links (in Description) to other template distributions.
GeoMessageSimulator1.0.0Linux-x64.zip
GeoMessageSimulator1.0.0Linux-x86.zip
GeoMessageSimulator1.0.0Windows-x86.zip

[ ] GeonamesLocator4.1.1.zip

Current link points to v4.0.0

[ ] LodgmentPlanning5.0.0.zip

Current link uses tiny url, not required link from Eric's comments above.

[ ] NationalSecurityBasemaps2.0.1.zip

Current link points to v2.0.0

[ ] PredictiveAnalysisAddIn1.3.1.zip

Current link points to PredictiveAnalysisTools1.3.0.zip

[ ] RangeDomeAnalysis3.0.0.zip

Current link points to v2.0.0

[ ] SolutionsGeoprocessingToolbox2.2.0.zip

Current link points to v2.1.0

[ ] VehicleCommander4.1.1Windows-x64.zip

Need to add links (in Description) to other template distributions.
VehicleCommander4.1.1Linux-x64.zip
VehicleCommander4.1.1Linux-x86.zip
VehicleCommander4.1.1Windows-x86.zip

[ ] VisibilityAndRange5.0.0.zip

Current link points to v4.0.0

HOW TO ADD A NEW LINK (Adapted from deleted page in OneNote 2016):
How to Create a Document Link
To create a document link for the NSEC TEM 2016 RELEASE:

Log into the server

Go to My Content

Select Add Item >> From the web

Select Document in the next dialog

URL: This is the URL to the template on the server (https://"server"/Downloads/ArcGISforDefense/"template_name.zip")
Title: This is the title of the template (Do not use version numbers)
Tags: Control the filter behavior in the gallery.

Select Add Item. This creates the blank Document link page which you will edit to add the remaining items below.

Click thumbnail to change image. below the current thumbnail (probably a world map).
2.  click Browse... and navigate to the thumbnail you want to use. Your thumbnail should be a 286 pixels wide by 190 pixels high (different than the instructions on the Upload Thumbnail prompt) Portable Network Graphic (PNG).
3.  Once you have selected your thumbnail, click Open and OK.

Add an item Summary. This is a short description of the template. In it, you should list the specific technology and version (i.e. "compatible with ArcGIS Pro 1.1").

Add an item Description. This text should be similar to the Overview text on the documentation page. After that, there is required text. They are:

Required Software: The text is "See Online Help for details." Ensure that the words "Online Help" link to the documentation page (link to the Solutions Page for the template).
Download: The text is "Click on the icon above."
Support: The text is:

"If you need support, please contact **Esri Support Services**. If you'd like to get help from other military community members, post your question on the Defense and Intelligence Forum on **GeoNet**. If you have a general question or comment about this solution, feel free to post them in the comments section on this page."
Where "Esri Support Services" links to https://my.esri.com/welcome?source=my-support
and "GeoNet" links to https://geonet.esri.com/community/defense-and-intelligence

Add Access and Use Constraints. This is not data permissions or credits. Contact @karlterrey  for any questions, but the stock verbiage is:

The information presented in this ArcGIS Pro Project template is based on standard Esri Commercial off-the-Shelf (COTS) technologies and commercial data. All scenario data is notional.

Adjust the item properties. This includes:

Item URL - Should already be set, but ensure it links to the tiny URL (http://esriurl.com/…)
Tags - Add additional tags as necessary, but should only correspond to items in the gallery filter (http://solutions.arcgis.com/gallery/#s=0).
Credits - Credit Esri and any additional credits necessary (from data permissions).

Save the item by clicking Save.
Share the Document Link item with Everyone by:

Clicking Share >> Everyone >> OK.

Starting on CompoundMap5.0.0.zip

Starting on ** ImageChangeDetection2.0.1.zip**

Starting on CoordinateConversionArcGISPro1.0.1.zip

Starting on DistanceAndDirectionArcGISPro1.0.0.zip and DistanceAndDirectionArcMap1.0.0.zip.
These will go into ONE Document link with internal links to the separate downloads.

Starting on ** VisibilityArcGISPro1.0.0.zip** and VisibilityArcMap1.0.0.zip.
These will go into ONE Document link with internal links to the separate downloads.

Starting on CrossCountryMobility_3.0.0.zip

Starting on HelicopterLandingZone_4.0.0.zip.

@mfunk Starting on ImageryBasemap3.0.0.zip

Starting on IncidentAnalysis5.0.0.zip.

@mfunk Finished with GeonamesLocator4.1.1.zip

Starting on MilitaryAspectsofTerrain_1.0.0.zip.

Starting on PointofOriginDetection5.0.1.zip.

@mfunk starting on PredictiveAnalysisWeb1.1.0.zip

Starting on RangeCards2.0.0.zip.

@mfunk @dfoll already created PredictiveAnalysisWeb1.1.0, I'm marking it as complete.

Starting on ScannedMapsBasemap3.0.0.zip.

Starting on SunPositionAnalysis_1.0.0.zip.

Starting on TimeSpanAnalysis2.0.0.zip.

@mfunk There are 2 Compound Map doc links. One for previous version 4.0.0 exists id=1ebce832b268418aa2adda47ba766586 - this could be updated for the new version instead of creating an additional item, id=37c6d44fdb9f41c98dad95a0d42fb91e for version 5.0.0. Which one do you want to use/release?

@lfunkhouser @ACueva @elinz
All of the document links that needed to be CREATED are done.
Next is to update existing ones. For that I either need someone to assign them to me or give me permissions to edit them directly.

@lfunkhouser I cannot see 1ebce832b268418aa2adda47ba766586:

@lfunkhouser @ACueva @elinz
I've got permissions and have moved all of my Document links to ReleasedMilitary.

Starting on SolutionsGeoprocessingToolbox2.2.0.zip.

Starting on DistancetoAssets2.0.1.zip.

Starting on ClearingOperations5.0.1.zip.

Starting on BeachLanding4.0.0.zip owned by ReleasedMilitary.
@lfunkhouser can you delete or unshare your Document link to Beach Landing so we don't confuse it with the "fixed" one?

Starting on GeoMessageSimulator1.0.0Windows-x64.zip.

@mfunk starting on LodgmentPlanning5.0.0.zip

Starting on VehicleCommander4.1.1Windows-x64.zip.

Fixed LodgmentPlanning5.0.0.zip There is a document link for this template that is owned by @lfunkhouser, can you remove/hide this?

@mfunk starting on NationalSecurityBasemaps2.0.1.zip

@mfunk starting on RangeDomeAnalysis3.0.0.zip

@mfunk starting on VehicleCommander4.1.1Windows-x64.zip

@Dbarnes1 Negative.... I'm already on Vehicle Commander....

@topowright it seems there is only one Visibility and Range document link. I'm going to update it and send it to ReleasedMilitary.
Starting on VisibilityAndRange5.0.0.zip.

@lfunkhouser Done with updating the document links.

The beach landing template owned by lfunkhouser is not intended to be released. The one owned by ReleasedMilitary should be the item that gets updated if needed.

@mfunk Compound Map points to v4.0.0 not 5.0.0, @elinz do we still have time to fix this?

@lfunkhouser Go ahead and make the change. Let me know when it’s done and also if we can close out this issue.
thanks
From: Lorraine Funkhouser<EMAIL_ADDRESS>Sent: Tuesday, September 13, 2016 11:51 AM
To: Esri/ops-server-config<EMAIL_ADDRESS>Cc: Eric Linz<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Esri/ops-server-config] All template document links need to be reviewed (#1495)
@mfunkhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_mfunk&d=DQMCaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=2ZpT4vzBgD3Uq0Qu1_1lvg&m=TCRa0T_tO6IPnxDPVRujwUqznocp9I0W3c7jnu9oAT8&s=r9fTMMkohLHK8maPbRv3_y2uQr8ZD2P2EAKiz5KdtzI&e= Compound Map points to v4.0.0 not 5.0.0, @elinzhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_elinz&d=DQMCaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=2ZpT4vzBgD3Uq0Qu1_1lvg&m=TCRa0T_tO6IPnxDPVRujwUqznocp9I0W3c7jnu9oAT8&s=T-sYyLw_kuix8JRjxky_6cmyTA3__8dTncKseAfBPuE&e= do we still have time to fix this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_Esri_ops-2Dserver-2Dconfig_issues_1495-23issuecomment-2D246784640&d=DQMCaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=2ZpT4vzBgD3Uq0Qu1_1lvg&m=TCRa0T_tO6IPnxDPVRujwUqznocp9I0W3c7jnu9oAT8&s=_YBxal0rxlN89Yy5iwCSmvIPCAXZys7E7ZY-AsdLTXE&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AEa7Yj3N8eggLBrp3dela1q-5FVqkfVhLCks5qpvCEgaJpZM4J2-2DYS&d=DQMCaQ&c=n6-cguzQvX_tUIrZOS_4Og&r=2ZpT4vzBgD3Uq0Qu1_1lvg&m=TCRa0T_tO6IPnxDPVRujwUqznocp9I0W3c7jnu9oAT8&s=mRHD9-esRugH5G4aBRhbuwGhNWdB085P8W7FgxM3KAQ&e=.

There was one last edit that @lfunkhouser wanted to do. I'm not sure if this was completed, but we are now in the Jenkins build phase so I'm closing this issue.


BUG: semanti ui not reseting form field correctly

Hi,
The .reset function that you can call to a form is a very usefull one cause you can clear/reset all the state of the form to the initial page load state. It's very handy and has been implemented in browsers since early IE 6.0!
So clearly it's a very usefull thing and I use it a lot when user sends a form to avoid having to refresh the page.
This bug manifests itself not reseting the dropdown menu. It resets all other fields excetp dropdown. Here is a simple case where you can test this bug out:
http://quemfazsite.com.br/temp/teste-criar-site44.php
Select any thing in the dropdown menu, type anything in the input text, select anything in the radio/checkbox buttons... then after that click the RESET button. Everything will reset except the dropdown menu!
It's pretty simple to check this bug, hope the demonstration above helps!

Hi @batata004. SUI dropdowns are complex HTML groups, that’s why it uses its own methods to restore values (as well as the form validation component’s reset() and clear() methods). HTMLFormElement.reset() doesn’t work with the dropdown component, because it can’t.

Did you know that with jquery its possible to attach a reset event to the form? Maybe you could attach reset event to clear the dropdown, it would e nice! I think it's possile and would be a nice addition to the library making it more close to what other UI libraries does (at least jquery ui and bootstrap work very well clearing all form fields!)


properly split trainer args and model args in argparser in basic examples

Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

What does this PR do?
Fixes the first bullet point in #1937
TODO: check docs if we have more examples of this type
(hyperparameters.rst, weights_loading.rst)
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

@Borda this is meant to be a fix PR. ok to leave it for later PR?

this argparser is very confusing.. it simply won't add all arguments to the help command, no matter how I compose the parsers with parents etc..
Unless anybody has an idea, I think I will to close this.

this argparser is very confusing.. it simply won't add all arguments to the help command, no matter how I compose the parsers with parents etc.. Unless anybody has an idea, I think I will to close this.

I think that the way around is adding a class method from_args similar to Trainer which would filter arguments based on class init and pass only those allowed...

I'll close this for now. It seems users anyway prefer things like hydra to separate their configuration in smaller parts. For now it is probably better to keep the examples simple even though the module just takes all the trainer args.
We can maybe discuss the api suggested by @Borda, but this would be another PR.


🐞 website: blog header background don't play well with text content

What is the issue?
I get black text on black background in the blog section which makes it hard to read :)
for example:

https://dagger.io/blog/public-launch-announcement

https://dagger.io/blog/series-a

Browser:
Chrome 100.0.4896.60 (Official Build) (64-bit)

Log output
N/A
Steps to reproduce

navigate to a blog post like thru a direct link like https://dagger.io/blog/public-launch-announcement

Interestingly it doesn't happen when navigating interactively from the blog section https://dagger.io/blog/ as a header image gets added, see:

Dagger version
N/A
OS version
Linux proppy 5.15.15-1rodete2-amd64 #1 SMP Debian 5.15.15-1rodete2 (2022-02-23) x86_64 GNU/Linux

Hi @proppy . Thanks for your help.
I can't  reproduce this bug. However, I've changed the way images are displayed.
Could you try those following links and tell me if it's better please ?
https://deploy-preview-165--dagger-io.netlify.app/blog/series-a
https://deploy-preview-165--dagger-io.netlify.app/blog/public-launch-announcement
Thanks :)

yep better!

😕 ... still not good... I'm using object-fit css property.
I saw that you are using a Linux distribution, Linux proppy...?

I am not seeing this problem on my Chrome Browser running on Linux.
I am testing this on:

browser: Google Chrome 99.0.4844.51 (Official Build) (64-bit)
OS: NixOS 21.11
kernel: Linux x22 5.10.103 #1-NixOS SMP Wed Mar 2 10:42:57 UTC 2022 x86_64 GNU/Linux

This is what navigating directly to https://dagger.io/blog/public-launch-announcement looks like for me:

If the image was slow to load, or maybe not load at all, this is what I would see:

The thing which I find more suspicious is that I was expecting the image to be in an img element, but it is in div src currently:

Could this be the problem?
I am curious @proppy, does it happen on a different browser too?

Another test on Firefox 98.0 & NixOS 21.11:

And now side-by-side (Chrome is left, Firefox is right):

thanks @gerhard for your tests. I've actually changed the way this banner is displayed by using a proper  tag. https://github.com/dagger/dagger/issues/1954#issuecomment-1084539107
It looks like it improve experience but still not the same result as yours => https://github.com/dagger/dagger/issues/1954#issuecomment-1084720864

cc @marcosnils :).

:wave:
just tested in the following browsers in linux and don't have any issues:
Mozilla Firefox 98.0.2
Brave Browser <IP_ADDRESS>
Google Chrome 100.0.4896.75

Ok. So we close this issue as we can't reproduce it.


fix retry handling of callbacks for pending batches

This fixes two bugs:

When there was more than one pending batch containing send ops, we were failing to return the on_complete callback for all but the last one.
When all three recv ops are in the same batch, which is seen after we start our own internal recv_trailing_metadata batch, and the completion of that internal batch commits us to a failing call, we failed to return the recv_initial_metadata_ready and recv_message_ready callbacks.

Known issues: #19834 #26595
The "Basic Tests PHP MacOS" failure is an infrastructure timeout.

When there was more than one pending batch containing send ops, we were failing to return the on_complete callback for all but the last one.

Is that even legal?

When there was more than one pending batch containing send ops, we were failing to return the on_complete callback for all but the last one.

Is that even legal?

Yes.  You can't have more than one send_message op pending at once, but you can start one batch with send_initial_metadata, a second batch with send_message, and a third batch with send_trailing_metadata.  See the test for an example.

Known issues: #26595
The "Basic Tests ObjC iOS" test is an infrastructure failure.


Fix eventRegistration cypress test

Due to issue #2271 I have created this PR to fix it. I don't get the width error like @MikeMcC399 gets on the issue, but I get the button error. It was solved updating the date inserted by Cypress.

@brifemu
This test also runs successfully for me.

@MikeMcC399 Thanks for testing!

@dsarkar
Good morning!


Prevent unbounded number of pending flushing tasks; Add PendingFlushT…

…asks metric (CASSANDRA-16261)

@ekaterinadimitrova2 @adelapena Should we add a description of the new metric in metrics.rst? For simple changes, it seems like we should just do it inline with the work itself...

@ekaterinadimitrova2 @adelapena Should we add a description of the new metric in metrics.rst? For simple changes, it seems like we should just do it inline with the work itself...

@ekaterinadimitrova2 @adelapena Should we add a description of the new metric in metrics.rst? For simple changes, it seems like we should just do it inline with the work itself...

@ekaterinadimitrova2 @adelapena Should we add a description of the new metric in metrics.rst? For simple changes, it seems like we should just do it inline with the work itself...

@ekaterinadimitrova2 @adelapena Should we add a description of the new metric in metrics.rst? For simple changes, it seems like we should just do it inline with the work itself...

Indeed we should add the description in metrics.rst (3.11 and trunk).

@ekaterinadimitrova2 @adelapena Should we add a description of the new metric in metrics.rst? For simple changes, it seems like we should just do it inline with the work itself...

Indeed we should add the description in metrics.rst (3.11 and trunk).


mommy moves my promt to the left

For some reason mommy moves my prompt to the left

my .zshrc file
plugins=(git zsh-syntax-highlighting)

source $ZSH/oh-my-zsh.sh

clear
colorscript -e panes
autoload -U colors && colors
PS1="%B%{$fg[red]%}[%{$fg[yellow]%}%n%{$fg[green]%}@%{$fg[blue]%}%M %{$fg[magenta]%}%~%{$fg[red]%}]%{$reset_color%}$%b "
RPS1="$(mommy -1 -s $?)"
HISTSIZE=10000
SAVEHIST=10000
HISTFILE=~/.cache/zsh/history
export PATH=/home/peat/.local/bin:$PATH
autoload -U compinit
zstyle ':completion:*' menu select
zmodload zsh/complist
compinit
_comp_options+=(globdots)               # Include hidden files.
source /usr/share/zsh-syntax-highlighting/zsh-syntax-highlighting.zsh 2>/dev/null

this issue is worked around in the readme (under "shell integration" > "zsh"), but it's easy to miss since i don't actually explicitly mention this issue, and don't explain why it works the way it does.
so, firstly, note that if you do RPS1="$(mommy -1 -s $?)", then RPS1 will be set when .zshrc is loaded, and won't ever update. if you want RPS1 to re-run after every command, then you should replace the double quotes (") with single quotes ('), so now RPS1 is just a literal string, and then you should also add set -o PROMPT_SUBST to your .zshrc, so that zsh knows that that string is a command that should be evaluated again each time.
as for your main issue, zsh is a bit weird with colour codes. mommy uses normal ansi escape codes (like \e[31m, where 31 specifies a red foreground), but when zsh creates the PS1 and RPS1 prompts, it thinks those ansi escape codes actually take up space, and will move the caret accordingly. you can zsh to not do this by wrapping the ansi escape code in %{ ... %}, but there is currently no clean way to do this within mommy's MOMMY_COLOR option. therefore, the best alternative is to set MOMMY_COLOR= (i.e. disable mommy's colour feature), and then put your desired zsh-specific colour codes in MOMMY_PREFIX, and then clear the colour codes in MOMMY_SUFFIX. you can use this snippet from the readme:
MOMMY_COLOR=""
MOMMY_PREFIX="%F{005}/%F{006}"
MOMMY_SUFFIX="~%f"

here, %F{005} is a zsh colour code, and %f resets the colours. using %F{005}/%F{006} means that mommy will randomly choose between colour 5 (pink) and colour 6 (cyan). you can also use this handy xterm color code chart to see which number means what colour.
please let me know if this solves your issue, and tell me if you have more questions~

It worked! Thanks a lot


660 account reskin

On the Transfer page, the text Use this form to send your ETH Tokens, ETH, and REP to another address. should be updated to Use this form to send your ETH or REP to another address.

Also on the Transfer page, Tranfer Funds is misspelled (should be Transfer Funds).

Should probably remove the "Download Login ID" button from the export page, since the whole concept of login ID is about to be deprecated + the point of login ID was to have a string instead of a file.

On the Export page, Use either the QR code or download link save a copy of the key file for import into a wallet. should have the word to between link and save

For some reason, entering an account name on the account page causes Chrome to max out my CPU as I'm typing.  Is something being updated behind the scenes every key press?

The Ethereum address of the account (below the account name, on the account page) should always display the entire address.  I can't think of a situation where someone would want to see 0x94...b2c1 instead of their actual address.

Addresses should be displayed (and verified) using EIP55 (capitals-based) checksums, the same way that Mist, MEW, etc. do it.

[ ] Update Copy on Transfer View
[ ] Fix typo on Transfer View
[ ] Remove Download LoginID
[ ] Update copy on export view
[ ] Check into account name change CPU utilization
[ ] Always Display the entire address in account details component

Ok, all requested changes are now in place.
Re: the name change — I’ve moved that to be housed in local storage.  The slowdown was that since it was stored w/in the loginID, we were re-encoded on change.

Btw — looks like somehow my sig changed…I am signing on my end, let me update things in my account.

OK, signature issue fixed — ended up being a mis-configuration on my end related to user.email.


[connectivity_plus] crash on iOS: Sending a message before the FlutterEngine has been run.

flutter: 2.10.5
connectivity_plus: 2.3.0
Sending a message before the FlutterEngine has been run.
Fatal Exception: NSInternalInconsistencyException
0  CoreFoundation                 0x99d78 __exceptionPreprocess
1  libobjc.A.dylib                0x16734 objc_exception_throw
2  Foundation                     0x1311f0 _userInfoForFileAndLine
3  Flutter                        0x126c4 (缺少 UUID d24b4305a5d8348c8a0909c859505f3a)
4  Runner                         0x3859a4 thunk for @escaping @callee_unowned @convention(block) (@unowned Swift.AnyObject?) -> () (<compiler-generated>)
5  Runner                         0x3b670c partial apply for implicit closure #2 in implicit closure #1 in SwiftConnectivityPlusPlugin.init(connectivityProvider:) + 72 (SwiftConnectivityPlusPlugin.swift:72)
6  Runner                         0x3b4f1c partial apply for implicit closure #2 in implicit closure #1 in PathMonitorConnectivityProvider.ensurePathMonitor() (<compiler-generated>)
7  libswiftNetwork.dylib          0x361c0 closure #1 in NWPathMonitor.init(requiredInterfaceType:)
8  libswiftNetwork.dylib          0x2c80 thunk for @escaping @callee_guaranteed (@guaranteed OS_nw_path) -> ()
9  libnetwork.dylib               0x804ae0 __nw_path_necp_update_evaluator_block_invoke
10 libdispatch.dylib              0x1e68 _dispatch_call_block_and_release
11 libdispatch.dylib              0x3a2c _dispatch_client_callout
12 libdispatch.dylib              0x15280 _dispatch_root_queue_drain
13 libdispatch.dylib              0x1596c _dispatch_worker_thread2
14 libsystem_pthread.dylib        0x1080 _pthread_wqthread
15 libsystem_pthread.dylib        0xe5c start_wqthread

Application Specific Information:
Sending a message before the FlutterEngine has been run.

Thread 37 Crashed:
0   CoreFoundation                  0x32f261d78         __exceptionPreprocess
1   libobjc.A.dylib                 0x360bae730         objc_exception_throw
2   Foundation                      0x3322d51ec         -[NSAssertionHandler handleFailureInMethod:object:file:lineNumber:description:]
3   Flutter                         0x1070de6c4         <redacted>
4   Runner                          0x2009059a4         __cxa_throw
5   Runner                          0x20093670c         __cxa_throw
6   Runner                          0x200934f1c         __cxa_throw
7   libswiftNetwork.dylib           0x411c3a1bc         NWPathMonitor.init
8   libswiftNetwork.dylib           0x411c06c7c         thunk for closure
9   libnetwork.dylib                0x33138aadc         __nw_path_necp_update_evaluator_block_invoke
10  libdispatch.dylib               0x32ebc3e64         _dispatch_call_block_and_release
11  libdispatch.dylib               0x32ebc5a28         _dispatch_client_callout
12  libdispatch.dylib               0x32ebd727c         _dispatch_root_queue_drain
13  libdispatch.dylib               0x32ebd7968         _dispatch_worker_thread2
14  libsystem_pthread.dylib         0x41215b07c         _pthread_wqthread
15  libsystem_pthread.dylib         0x41215ae58         start_wqthread

Same here with Flutter v3.0.0. Already over 10k crash reports by Sentry in last 2 weeks due to this same thing.

Same here, flutter v2.10.1,  connectivity_plus: 2.3.0

Can confirm that also see this issue in one of my projects. Not sure why it started to appear just in last 2 weeks, though.

Same here. We found in Crashlytics that many users experiencing this bug

Also seem related:

https://github.com/flutter/flutter/issues/66647
https://github.com/flutter/flutter/issues/90781
https://github.com/flutter/flutter/issues/96901

There is a comment here looking like a solution: https://github.com/flutter/flutter/issues/96901#issuecomment-1053114107
Why don't you implement it?

Why don't you implement it and submit a PR?

unless people are still seeing this in 2.3.5, this should be able to be closed now

@miquelbeltran can we implement the same logic to sensor_plus?  Looks like the same error here

@followthemoney1 Feel free to submit a PR, tag me in it once you got it, so I can review it quickly

Flutter version: 3.3.3  3.3.10
SDK version: 3.0.2
I have tried many methods, but they still don't work, and I feel that this problem only occurs on my computer：
Exception: Unable to launch com.niuke.flutterApp1 on 4F068ED4-3D84-47C6-9490-B01347FD1B03:
ProcessException: Process exited abnormally:
An error was encountered processing the command (domain=FBSOpenApplicationServiceErrorDomain, code=1):
The request to open "com.niuke.flutterApp1" failed.
The request was denied by service delegate (SBMainWorkspace).
Underlying error (domain=FBProcessExit, code=128):
The operation couldn’t be completed. The process exited.
The process exited.
Command: xcrun simctl launch 4F068ED4-3D84-47C6-9490-B01347FD1B03 com.niuke.flutterApp1 --enable-dart-profiling --enable-checked-mode --verify-entry-points --start-paused --observatory-port=0
Error launching application on iPhone 14 Pro Max.


Why polymer create non-ui element?

why polymer create non-ui element ?
why not inject the function into the javascript obejct?
like <core-ajax> => $.ajax();
poor english, sorry

Elements have some nice properties that we take advantage of when possible:

They live in the DOM tree and can be referenced various ways: this.$, querySelector, childNodes, etc.
They have the custom element lifecycle of created, attached, detached, etc.
They have a natural serialized form in HTML.
They can fire DOM events, which in turn have useful features like bubbling, behaviors at shadow boundaries.
They can directly participate in our data-binding system.
Sometimes it's interesting to be able to render and style what would normally be non-UI

In general we like the idea of building an app out of generally the same "stuff", and for us that is elements. It's not a hard and fast rule that everything be built this way, but we certainly try to embrace this style where we can.

Closing this issue due to age and the release of version 1 of Polymer - please feel free to re-open if this is incorrect.


[Bug] minecat should read pnpm-workspace.yaml to get sub-package info

The whole project is structured like
- package.json
- components
- docs
- pnpm-workspace.yaml

running npx minecat i react react-dom
and resulting in
> npx minecat i react react-dom
Error: ENOENT: no such file or directory, scandir '/root/projects/projj/github.com/BlizzBolts/hoardui/packages'

And the root project must contain minecat property in package.json file to make it work.

nsdd


Reset functionality

Hi,
I am missing some kind of reset function, which would bring the form back to initial state.
The reason for this (at least for me) is, that after AJAX submitting form I call reset method, so required fields gets empty, but they are still green, as if they are valid.
I do not want to call validate, as they will become red (invalid), which would be for user confusing, as he just fill it and submit it.
Also, in traditional approach with reset button, after clicking the reset button errors and success remains and submit button is still enabled.
Please consider this PR. It changes following

reset input is removed from explicit validation (same as submit button)
adds .validator('reset') function
hooks reset event of the form, calling reset function
on destroy it removes also has-success classes

Pavel
PS: there is setTimeout function, as I do not know better way how to hook after reset event.

Thanks for the PR. I think there's a better way to accomplish this though.
We could add an event listener for the reset event on the form in the constructor and call .validator.('destroy').validator(options)

Good find on the missed .has-success cleanup in the destroy method!

I just released a new version of the plugin that implements plugin reinitialization on form reset.
I should have just merged your PR and made the changes I wanted on top of your commits, but it took me a little time to realize the motivation behind some of the bugfixes you included here.
By the end I had pulled in all of your bugfixes one by one, so thanks for bringing them to my attention.

Hi,
.validator.('destroy').validator(options) is not correct way, as it also unbinds events. If used as this
$('#form').validator().submit(function (e) {
  if (!e.isDefaultPrevented()) {
    // AJAX submit form
   $('#form').trigger('reset');
  }
}

It will work for the first time, but not for second time, as the validator event will be called after the submitting code, so e.isDefaultPrevented() will not work and form will get submitted. Thats why I called clearErrors instead of destroy.
Pavel

Hmm, yeah you're right. Maybe what we need to do is to give some of destroy's responsibilities to reset and calling it inside destroy. I don't think just calling clearErrors in reset is sufficient.

I've pushed v0.11.5 which should fix that issue.

I made some test, and everything seems great now. Thank you.


Question: Why does the changes of VirtualService effect the LDS?

When only the resource VirtualServices changes, istiod will trigger LDS push.
Do we really need to push LDS? Or Maybe I miss something.

At least outbound LDS depends on VS, such as the tcp_proxy filter depends on VS.tcp

At least outbound LDS depends on VS, such as the tcp_proxy filter depends on VS.tcp

But, if I only care about VS.http, do we still need to rebuild LDS and trigger push?

There is no such fine grained control whether the modified VS has http or tcp only and then decide to build LDS or not. We will have to look at previous config and current config and decide whether the new one needs LDS push or not.

There is no such fine grained control whether the modified VS has http or tcp only and then decide to build LDS or not. We will have to look at previous config and current config and decide whether the new one needs LDS push or not.

Got it. Thanks.


Upgrade to Java 8

as mentioned in #5 we still require 1.7, so I would prefer it if we didn't move to 1.8 just yet.

sounds good
Dave Rusek
<EMAIL_ADDRESS>@davidjrusek
On Wed, May 6, 2015 at 1:00 PM, Tobias Bieniek<EMAIL_ADDRESS>wrote:

as mentioned in #5 https://github.com/mgodave/esl-client/pull/5 we
still require 1.7, so I would prefer it if we didn't move to 1.8 just yet.
—
Reply to this email directly or view it on GitHub
https://github.com/mgodave/esl-client/issues/6#issuecomment-99574296.


Test-PR-with-netlink-go/nfqueue

Codecov Report

Merging #307 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #307      +/-   ##
==========================================
- Coverage   55.14%   55.12%   -0.03%
==========================================
  Files          46       46
  Lines        4704     4706       +2
==========================================
  Hits         2594     2594
- Misses       1909     1911       +2
  Partials      201      201

Impacted Files
Coverage Δ

enforcer/nfq_linux.go
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Correction of typo introduced in previous commit

Apologies for missing this.

No worries, thank you for the corrections.


Feature: token service and endpoint

Description

This is an implementation of the "/token" endpoint, which is part of the authorized VP flow for resource sharing between EDCs.
Replaces #179
Closes #139

Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review:

[x] DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
[x] Copyright and license header are present on all affected files

@florianrusch-zf I pushed the mentioned fixes.

@DominikPinsel @florianrusch-zf hey, I've pushed a new Validator class for the SecureTokenRequest. Take a look if you're interested in the change, but it's basically the same as before, just in a more modern way.


Composer pins tenant id during publish profile creation; keeps alt account from publishing

Describe the bug
Currently, during publish profile creation, composer caches tenant id after successful login.  Trying then to publish to an account in a different tenant fails as composer tries to use the cached first tenantid.
Version
Version:  1.4.0-nightly.230964.e5e5020
Electron: 8.2.4
Chrome: 80.0.3987.165
NodeJS: 12.13.0
V8: 8.0.426.27-electron.0
Browser

[X] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[X] Windows
[ ] Ubuntu

To Reproduce

Create a bot in composer
Try to create a publish profile with MSIT creds and fail before completion
Try to create a publish profile with non MSIT creds (in my case, personal tenant)

Experience: cannot create publish profile as new ID is not in the tenant cached in step 2. No way also to log out of tenant in step 2 as you're not "logged in".
Expected behavior
I can switch accounts freely, or explicitly "sign out" to clear cached tenant info.
Screenshots

hit https://github.com/microsoft/BotFramework-Composer/issues/6707 during the verification.


Update README, renaming Yields -> Blocks

Just what it says on the tin.

Thanks for opening this, @chriskrycho! I think it's now covered by the changes in #307

EEEEEEEXCELLENNNNNNT


double-beep: Watch theguidestore.com

double-beep requests the watch of the watch_keyword theguidestore\.com. See the MS search here and the Stack Exchange search here.

theguidestore\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


Filters?

Filters for this bot? , I think you can add them using Lavalink.

Filters for this bot? , I think you can add them using Lavalink.

it is very easy
**const player = message.guild.music;
player.node.send({
op: "filters",
guildId:  msg.guild.id || msg.guild,
<Your_filters_here>: value,
});**
for Implementation
https://github.com/freyacodes/Lavalink/blob/63153009235aa578483b42434e5fb496f6092b2f/IMPLEMENTATION.md

i might add filters if i want

i can help in @Allvaa i mean i already have them i will create a pull request and check them if they are okay you can merge.

is there any good filters config? maybe I can use it to implement filters sometime

i might add filters if i want

i can help in @Allvaa i mean i already have them i will create a pull request and check them if they are okay you can merge.

i will add it myself for better filters handling.
is there any good filters config? maybe I can use it to implement filters sometime

you can directly implement filters in music handler i will make a pull request wait.

Maybe an example?
So i have an Idea how to add it?

ded

@TheUsual10k check pull request I added if you want use those

@TheUsual10k check pull request I added some filters using musichandler if you want you can use those.

i tried your new commands in pull request but none of them seem to work , like i dont really notice anything different.

@TheUsual10k check pull request I added some filters using musichandler if you want you can use those.

i tried your new commands in pull request but none of them seem to work , like i dont really notice anything different.

Bruh did u check command handler and are you using Dev branch lavalink


credentials = FetchCredentialsOption.Include doesn't work on client-side

If you try the sample with
requestMessage.Properties[WebAssemblyHttpMessageHandler.FetchArgs] = new
{
    credentials = FetchCredentialsOption.Include
};

The exception
> WASM: System.Net.Http.HttpRequestException: TypeError: Failed to execute 'fetch' on 'Window': The provided value '2' is not a valid enum value of type RequestCredentials.
Is logged in the console
The workarroud is to use
requestMessage.Properties[WebAssemblyHttpMessageHandler.FetchArgs] = new
{
    credentials = "include"
};

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8b5e2e10-ff7b-2b9f-84da-f16028ae2c53
Version Independent ID: c7e59a08-1c60-32c2-75fd-33cb77ff7a5d
Content: Call a web API from ASP.NET Core Blazor
Content Source: aspnetcore/blazor/call-web-api.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Thanks @aguacongas
@mkArtakMSFT @danroth27 I'll need some guidance on this one given that the current content/approach was supplied initially for the topic by engineering and then later checked/approved by engineering for 3.0.
I've prioritized this one for a fast update once I have your guidance on it.

@SteveSandersonMS Is this just a product bug that we should fix?

@SteveSandersonMS, @pranavkm is this a product bug, that we need to move to the AspNetCore repo to track?

This is being tracked via https://github.com/mono/mono/issues/17808.

Am thinking this really is not a mono bug here.  I think the problem is that the AspNetCore is still passing the value of FetchCredentialsOption from their implementation instead of from the WebAssembly Http Message Handler.  Could someone verify that from your side?

Or maybe I am misunderstanding the current functionality.  Could someone explain this or provide a repro with example?

@kjpou1 As far as I can tell, the current version of Mono doesn't provide any per-request way to set a credentials option. The credentials option is always set to the value of the globally static DefaultCredentials property: https://github.com/mono/mono/blob/5d981c267981505f425826ae0a3cc79dd08ecde4/sdks/wasm/framework/src/WebAssembly.Net.Http/WasmHttpMessageHandler.cs#L76. Do you think at some point it will be possible to reintroduce support for setting the credentials option on a per-request basis?
@aguacongas I'm afraid the docs you're pointing to are out of date, as is the code sample you've posted. You're referring to a type WebAssemblyHttpMessageHandler that doesn't exist in current versions of Blazor (or the underlying Mono WebAssembly runtime). It doesn't look to me like your workaround would work with the current framework. Or if I'm missing something about where this type is defined, please let me know!
@guardrex Could we remove the section of the docs that talks about FetchCredentialsOption being put into a Properties dictionary? Currently the only thing in the API that's in this subject area is WebAssemblyHttpMessageHandlerOptions.DefaultCredentials which is still supported and does work, but is not quite the same thing since it's a global static value, not something you can set per-request.
I've added a note about the need to reintroduce an API similar to this: https://github.com/aspnet/AspNetCore/issues/17043#issuecomment-558241317.

... and it looks like setting the referrer here was also dropped at some point since the topic was written. I'll 🔪 that, too.
https://github.com/aspnet/AspNetCore/blob/master/src/Components/test/testassets/BasicTestApp/HttpClientTest/HttpRequestsComponent.razor

@SteveSandersonMS you're right, it's not a workaround anymore since 3.1-preview3. This issue was open before.


Feature Request: CutCornerView rounded corners

CutCornerView
Add an attribute for rounding the cut corners.

can you send me an example ?


How to use the skinning apis ?

Trying to use the skinning setBones api. Even though the uniforms are invalidated, and makeBones invoked. The model doesnt show-up :/
In FRenderableManager::create we make setSkinning as true, and mSkinningBoneCount as 4.
Any other configs which needs to be set ? Are there any samples to refer ?
The model is the simple skin in khronos. https://github.com/KhronosGroup/glTF-Tutorials/blob/master/gltfTutorial/gltfTutorial_019_SimpleSkin.md
Thanks.

HI,
read this and just want to let you know: you do not have to build the whole assimp-lib if you only want to use glTF2. I could prepare a pull request for you to optimize the size of the lib.
Kim

Thanks for the offer. We already don't build all of Assimp, but it still remains a large library.

@sohanjg you might want to check for GL errors by using a debug variant of Filament or by removing lines 34-36 in GLUtils.h
The reason I'm suggesting that is that we see GL errors with skinning with the WebGL platform, you might be running into the same issue on your platform.

The skinning issue we see with WebGL is an INVALID_OPERATION due to an incorrect UBO size, you might be running into the same thing.  Let us know if you see GL errors per my previous comment.

prideout@, sorry was afk. Thanks a lot for the skinning demo.
Regarding the UBO size, we were not getting the INVALID_OP in linux (thats where we are working now, but need to check once we move to mobile drivers).
We were finally able to get something on screen, we found via RenderDoc that mesh_bone_indices and mesh_bone_weights were all 0's. Some how the vectors are not properly setting up the uniform buffer :/

@sohanjg , just to be clear, are you still running into issues with Filament itself after pulling #773?

Yes, the problem still exists. We were not facing the driver invalid op problem in linux.
Our problem seems to be the way assimp loads the bone info into the uniform Buffer.

Ok, looks like we got it working now, may not be 100% yet. Atleast for the simple skin.
With the bone info being initialized into the uniform, the setBones seems to be working all right.
We didnt need the UBO size patch though, atleast for Linux.

@prideout thanks for all your help in this.
But seem like we cant make this work with higher bone count, we are seeing crippled "cesium man" :/
I have gisted our changes (and the output) in MeshAssimp here, https://gist.github.com/sohanjg/567855d3c914efe2f8e22d5d684405bb
Rest of the code is primarily based on http://ogldev.atspace.co.uk/www/tutorial38/tutorial38.html
If you have time, would you please have a look, and let us know where we are going wrong.
We suspect its the matrices, but cant find what can be wrong.
Can, the way we do ProcessNode (position update, bounding box etc.) in MeshAssimp have some influence ?
Thanks.

The gist calls a processSkinning function but I don't see where it's defined.  I wonder if the inverseBindMatrices in the glTF file are being honored?
As a side note, we are actively working on a new lightweight glTF importer that does not use assimp.

"sohanjg" went to a trip, but I try to answer instead. "processSkinning" is very simple function that calls LoadBones (tutorial38) for each mesh. After inspecting the "simpleskin" that you were using in the animation example it seems possible that assimp is loading the inverseBindMatrices somehow wrong.
We get for bone 1 {1,0,0,0,  0,1,0,1,  0,0,1,0,   0,0,0,1} and I think it should be {1,0,0,-0.5,   0,1,0,-1,  0,0,1,0, 0,0,0,1}.
Also sum of some bone weights for "cesium man" is far below 1.
Do you think the inverseBindMatrix above is loaded wrong? I think we need to reprogram this in better way but I'm worried about assimp...


Examples in Docs

Examples as docs above classes.
Example:
https://birchsolutions.slack.com/archives/C02AR20BL7P/p1665039870169539

belongs in generator repos


Add Object Selection Instructions - Download

Add instructions for user that say:
"Please choose one Object Type from the list below"
Time estimate: 1D
Check story #5 for more details/context.

Alonzo completed and tested this.


Basic question: How to play ByteData in flutter

ByteData audioBytes = ttsResponse.audio.buffer.asByteData());
Is there any suggested optimal workaround how to play this bytes in Flutter?
or any library suggestions?
Thank you in advance.

final String filePath = "path/to/file.mp3";
final File audioFile = File(filePath);
await audioFile.writeAsBytes(audioResponse.audio);
// OR
final base64Data = base64.encode(audioResponse.audio);
final dataUri = "data:audio/mpeg;base64,$base64Data";

And use any audio player for Flutter


fix: bump request timeout

This increases the default request timeout to 20s, mirroring geth's rtt estimate:
https://github.com/ethereum/go-ethereum/blob/1c737e8b6da2b14111f8224ef3f385b1fe0cd8b9/p2p/msgrate/msgrate.go#L52

Codecov Report

Merging #695 (82ba4e9) into main (65bc3cb) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #695      +/-   ##
==========================================
- Coverage   73.03%   73.00%   -0.04%
==========================================
  Files         263      263
  Lines       26937    26937
==========================================
- Hits        19674    19665       -9
- Misses       7263     7272       +9

Flag
Coverage Δ

unit-tests
73.00% <ø> (-0.04%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

crates/net/network/src/session/config.rs
83.00% <ø> (ø)

crates/stages/src/stages/bodies.rs
91.68% <0.00%> (-1.71%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Feat: nuxt3 adapter

I'll put this on hold. I realized there's some issues with createContext type. I think an adapter for for @trpc/server has to be created first.

Superceeded by: https://github.com/jlalmes/trpc-openapi/pull/292


added 2 missing variables

Codecov Report

Merging #134 (5a8a418) into develop (0c2faf1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #134   +/-   ##
========================================
  Coverage    48.72%   48.72%
========================================
  Files           21       21
  Lines         1061     1061
========================================
  Hits           517      517
  Misses         544      544

Impacted Files
Coverage Δ

...lunk_connect_for_snmp_poller/manager/hec_sender.py
56.17% <ø> (ø)

..._connect_for_snmp_poller/manager/task_utilities.py
22.38% <ø> (ø)

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 0c2faf1...5a8a418. Read the comment docs.

:tada: This PR is included in version 1.5.0-develop.9 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fill out Material documentation and clarify render_priority and next_pass sorting

Partially address: https://github.com/godotengine/godot/issues/83774 (we still need to add more detail to the manual)

Cherry-picked for 4.1.4. (Including some changes from previous PRs updating the descriptions, since they were causing conflicts and we're invalidating translations anyway.)


Add Ginger to the list of allergens

What
In the list of allergens, ginger isn't listed.
I would rather like it to be.
Why/User story
While Ginger isn't a common allergy, it is one that I suffer from, and it's usually one that's hard to spot in an ingredients list, due to it's short, non-eye-catching name.
It's also commonly used as a flavour agent, especially in milk alternatives, vegan / organic processed foods (like vegan yogurts or sorbets) - things that loved ones will often assumes is free of any possible allergens.

Thank you for the report, I'm transferring the issue to the Server side of Open Food Facts.


Sending 204 With No Body

@timothycrosley is it possible with Hug to send a 204 response with no body? Wasn't able to get it working properly over the weekend. Thx.
This seems to fail every time with a Hug exception:
@delete.urls('/insurance-profiles/{insurance_profile_id}',
                 versions=1)
def delete_insurance_profile(self,
                                 request,
                                 response,
                                 insurance_profile_id: validators.uid(UIDTypes.INSURANCE_PROFILES)
                                 ):
    # resource validation and delete queries go here
    response.status = falcon.HTTP_204
    return

In our endpoint, to get around this, we switched back to sending a 200 status with the full representation of the resource deleted, but it would be nice in the future to allow responses with a null body.

Hi @dreadwestern,
hug should indeed support this! As best as I can tell, it does in the current release so it may be worth checking to see if something else is going on. In my testing for instance, chrome wouldn't respond to or show any indication that it anything happened when trying to reach a null response 204 page, even though the server correctly returned the response (so it's potentially a client issue that you are running into). I've going ahead and added this test case to the hug test suite just to be sure [1], and tried to reproduce locally. Let me know if you are able to reproduce it in the future.
Thanks!
~Timothy
[1] https://github.com/timothycrosley/hug/pull/371/files


python312Packages.qcs-sdk-python: 0.17.10 -> 0.19.0

Description of changes
Diff: https://github.com/rigetti/qcs-sdk-rust/compare/python/v0.17.10...python/v0.19.0
Changelog: https://github.com/rigetti/qcs-sdk-rust/blob/python/v0.19.0/crates/python/CHANGELOG.md

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

8 packages marked as broken and skipped:

python311Packages.cirq
python311Packages.cirq-rigetti
python311Packages.cirq-rigetti.dist
python311Packages.cirq.dist
python312Packages.cirq
python312Packages.cirq-rigetti
python312Packages.cirq-rigetti.dist
python312Packages.cirq.dist

8 packages built:

python311Packages.pyquil
python311Packages.pyquil.dist
python311Packages.qcs-sdk-python
python311Packages.qcs-sdk-python.dist
python312Packages.pyquil
python312Packages.pyquil.dist
python312Packages.qcs-sdk-python
python312Packages.qcs-sdk-python.dist


Add binance exchange or FTX exchange , please

Hello.
This is a great program .
Could be possible you add Binance exchange or FTX exchange ?
Please

Binance maybe, and other platforms. But FTX, not really :)


[20.03] crystal: use latest openssl

(cherry picked from commit e10900b0683d2bdc827a95d0bcf42f2bf8943ce9)

Motivation for this change
backports https://github.com/NixOS/nixpkgs/pull/81272.
See motivation there.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build crystal


Url http://localhost:4822/swagger/docs/v1 is case sensitive but it should not be

when I define
config.SingleApiVersion("V1", "Api");
I got response with url http://localhost:4822/swagger/docs/V1, but there is error for http://localhost:4822/swagger/docs/v1

Technically, as per RFC 7230, for http URLs the path is case sensitive.  Historically the Microsoft stack has ignored this and made paths case-insensitive so the current behaviour although technically correct is violating the principle of least surprise!

I'm happy with the "technically correct" solution. I like being a good citizen of the web :)


fix(@angular-devkit/build-angular): add notice when using bundleDepen…

…dencies: none with Ivy
Closes: https://github.com/angular/universal/issues/1354

Blocked on https://github.com/angular/angular/pull/34225


Socket.sock invalid value

According to nn_socket man page, the function returns file descriptor, so Socket.sock attribute should reflect this.
If we run
import nnpy

sub = nnpy.Socket(nnpy.AF_SP, nnpy.PUB)
sub.bind('tcp://*:5555')
print(sub.sock)

The 0 will be printed.
strace gives the following in its output for above script
$ grep -e ^socket -e ^bind log
socket(AF_INET, SOCK_STREAM|SOCK_CLOEXEC, IPPROTO_IP) = 7
bind(7, {sa_family=AF_INET, sin_port=htons(5555), sin_addr=inet_addr("<IP_ADDRESS>")}, 16) = 0

So Socket.sock should be 7, not 0.

I have just read the nn_socket man page more carefully
Note that file descriptors returned by nn_socket function are not standard file descriptors and will exhibit undefined behaviour when used with system functions. Moreover, it may happen that a system file descriptor and file descriptor of an SP socket will incidentally collide (be equal).

http://nanomsg.org/v1.0.0/nn_socket.3.html
This issue is invalid.


Decouple nested references from verifiers

Describe the solution you'd like
As a cluster operator, I know what types of artifacts will be available as well as the expected basic artifact tree or relationship structure. I want a way to configure this in Ratify independent of individual verifiers. When running in Kubernetes, I would like to be able to maintain this without redeploying Ratify -- ideally via CRD.
Anything else you would like to add:
Just moving the nestedReferences into the otherwise empty Executor section of the JSON config is less than ideal as was discussed regarding #518 . Also, if Ratify is modified to execute all verifiers for a given artifact type as proposed in #448, leaving nested references at the verifier level would be confusing at best. Even now, what if the executed verifier doesn't have nested references configured but the following one does -- should the nested references of the other verifier be considered? Ratify cannot assume user intent either way.
If it made sense, the policy provider could be leveraged to drive whether to check for nested references per artifact type. The current implementation has some holes still ( #351 ) , so if it going to be refactored anyways, it might be a sound investment to put the logic there. Additionally, having a predefined structure gives Ratify control over the depth of a given artifact tree and provides safeguards against unintended DOS attacks.
Environment:

Ratify version: (use the image tag):
Kubernetes version: (use kubectl version):

@binbin-li  have a proposal out at https://hackmd.io/@H7a8_rG4SuaKwzu4NLT-9Q/SkJvCYHeh#Config-Policy-Provider-Redesign, would appreciate any feedback. thanks!

Closing this as it's implemented by https://github.com/deislabs/ratify/pull/798.
Once users enable rego policy, it will decouple nested reference verification from verifiers.


Remove iron-llvm dependency

Unfortunately, the only LLVM bindings which actively maintained and up to date is llvm-sys. There are a few other wrappers on crates.io, such as llvm-sys-wrapper and llvm-wrapped, which may even be updated in time with LLVM updates.
However, llvm-sys seems to be the only dependency which is really keeping up to date with LLVM. Because iron-llvm development time has been a backpressure issue for me in the past, and other libraries not tied closely to llvm-sys are not as far in development (some go as far back as LLVM 3.x when the current LLVM at the time of writing is LLVM 6.0.1).
If we absolutely need to, we can tie the current bindings to LLVM at a specific llvm-sys version and call it a day. Otherwise, we have the freedom to move up LLVM versions as they break backwards compatibility. In the long run, the main compatibility issues will be around the JIT, GC, and coroutine support.
Branch llvm-6 is in development with wraps of llvm-sus in /src/llvm.

Closed in #38.


Option not recognized

Node version (or tell us if you're using electron or some other framework):
v6.4.0
ShellJS version (the most recent version/Github branch you see the bug on):
0.7.5
Operating system:
Mac OS Yosemite
Description of the bug:
Getting option not recognized when running ls -d
Example ShellJS command to reproduce the error:
var shell = require("shelljs");

shell.ls("-d", "./").forEach(function(dir){
    if(dir !== "node_modules"){
      console.log("dir: " + dir);
    }
  });

The option should be recognized. If not, this is a bug.
But why would you have code like that? I don't think it's going to do what you think it will...

I am trying to list the directories in the current directory and check the names of them.
Is there another way I can do this using another command?

Just use ls(). If you need hidden files, use ls('-a'). If you need only directories (no files), then use ls('-d', '*/')

Can't reproduce this on Linux, and I see no reason why it would only repro on Mac OS.
I'll try to repro on Mac OS tomorrow.

@freitagbr can you reproduce this? I won't be able to look at this on my mac.

@nfischer I am able to use the command you sent. ls('-d', '*/')
Unsure why the current directory wouldn't be the second argument instead of the */.
For whatever reason, I am now unable to replicate what was happening before using ls('-d', './').
This was very strange how it just came and went.
Thanks for the help though!

@embrown30 from our docs:

-d: list directories themselves, not their contents

So if you use ls('-d', 'foo'), it means "Don't show me the contents of directory foo, just show me foo itself." Therefore, ls('-d', '.') means "show me . itself, not the stuff inside it." This always returns the array ['.'], which isn't very useful.
When in doubt, unix man pages can also help clarify ambiguities, since we strive for POSIX compliance where it makes sense. Hope this helps!

I tested with ShellJS at 0.7.5 and node at 6.4.0, and I could not reproduce.
Maybe there was a globally installed version being used, instead of a locally installed version. For reference, the '-d' option was first introduced in version 0.7.


Detailed documentation for SageMaker Containers - training

AWS CodeBuild CI Report

Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

also, the build is failing:
The project's long_description has invalid markup which will not be rendered on PyPI. The following syntax errors were detected:
line 154: Error: Error in "code" directive:
maximum 1 argument(s) allowed, 3 supplied.

.. code:: python
   import argparse

   if __name__ == '__main__':
     parser = argparse.ArgumentParser()

     parser.add_argument('--learning-rate', type=int, default=1)
     parser.add_argument('--batch-size', type=int, default=64)
     parser.add_argument('--communicator', type=str)
     parser.add_argument('--frequency', type=int, default=20)

     args = parser.parse_args()
     ...
ERROR: InvocationError for command /codebuild/output/src647328566/src/github.com/aws/sagemaker-containers/.tox/twine/bin/twine check 'dist/*.tar.gz' (exited with code 1)
___________________________________ summary ____________________________________
ERROR:   twine: commands failed

looks like you're missing a newline

AWS CodeBuild CI Report

Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


FWIN-389/Consider-adding-crictl-tool-to-asdf

This PR fixes # A blank Pr to main as forcing main as main branch caused issue with github action
Checklist

[x] I have signed the CLA
[x] I have updated/added any relevant documentation

Description
What's the goal of this PR?
What changes did you make?
What alternative solution should we consider, if any?

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

Nice! Thanks!


Change SignupActivity to an Activity from a WebView

Summary:
SignupActivity is currently a WebView, which is inconsistent with the res of the application and allows users to browse MediaWiki in the WebView without logging in
Expected
SignupActivity should be an Android Activity
Observed behavior:
It is a WebView.
Device and Android version:
Xiaomi (Wingtech) Redmi 2 Prime
Xiaomi Redmi Note 4
Commons app version:
2.6.7-debug

This can be easily implemented using JavaScript injection in a WebView. I'd like to take this up.

Please allow me to work on this :)

Considering the signup process when a user signing up the checking for whether the username exists in the system is done currently after pressing the signup button. It is difficult to going for new user name again and again while retyping the password and responding to recaptcha...... It will be better the user existence test is done during onchange trigger of the username field.

@diddypod Sorry we didn't get back to you about this. Are you still interested in working on it?

Yes, please. May I take it up?

@diddypod Yes feel free :-)

JavaScript injection is a rather hacky implementation. I'll try to make it somewhat better by using the MediaWiki APIs for account creation.

Hello
Really sorry for the inactivity and the lack of updates, I've been having evaluations in college. I've been tinkering away in my free time, though, and while the bulk of the process is working, I'm having a bit of trouble retrieving and displaying the captcha via APIs. Any help or insights?

Hi @diddypod , no worries. :) I am afraid that I can't offer any help with the captcha because that is exactly where I got stuck the last time I tried to implement this. :/ Perhaps someone else can chime in?

Hello
Again, sorry for the delay. I've been working on this for the past couple of days and I have found these tools

https://www.mediawiki.org/wiki/Extension:ConfirmEdit  which will let you generate captchas
https://www.mediawiki.org/wiki/API:Account_creation  to create accounts

API:Account Creation needs a CAPTCHA to work, but I haven't been able to use the Extension using the APIs.
The alternative is to create an Android layout and use JavaScript injection as I said earlier which seems hacky and pointless, since it does not improve or simplify functionality, instead adding more complexity to the current webview implementation.
I'd like to stop working on this issue, and take up some other issues.

@diddypod Thanks for the effort that you put into this. I am inclined to agree with you - as mentioned, I also tried to implement this a couple of years ago and ran into the same roadblocks. An additional concern is futureproofing - if we use the WebView, any changes that WMF makes to the signup process will automatically be incorporated, whereas with our own implementation we would need to keep on top of things ourselves.
Please feel free to work on other issues. Would you like me to close this?

Thank you. Please close this :)


Added Contributing Guidelines

@LokeshKumar-Dev I have added the Contributing Guidelines as CONTRIBUTING.md
Please review it.
And could you please add the GSSOC 23 label to the issue #2

@LokeshKumar-Dev I have added the Contributing Guidelines.
Please review it.

@LokeshKumar-Dev Could you please merge this PR.


:not(:has( and :has(:not( are different

:not(:has( and :has(:not( work differently
actually, only :not(:has( does what is expected, while :has(:not( work exactly as :has(, ignoring the negation introduced by :not(

Can you give an example of each case and show what you expect vs what you get?
I'm trying both cases here: :not(:has) and :has(:not) and they appear OK there.
Source is here if you want to review & debug: https://github.com/jhy/jsoup/blob/970403cfb442d8c835c83e8f85b9384eb2f34390/src/main/java/org/jsoup/select/StructuralEvaluator.java#L129 and https://github.com/jhy/jsoup/blob/970403cfb442d8c835c83e8f85b9384eb2f34390/src/main/java/org/jsoup/select/StructuralEvaluator.java#L53

import `org.jsoup:jsoup:1.17.2`, org.jsoup.Jsoup

object Test {
  def main(args: Array[String]): Unit = {
    val doc1 = Jsoup.parse("""

""")

    println("1:" + doc1.select("""[id="jwplayer-1"]:not(:has([class*="jw-preview"]))"""))
    println("2:" + doc1.select("""[id="jwplayer-1"]:has(:not([class*="jw-preview"]))"""))
  }
}


Remove consul service registry

Per discussions in networking WG, external service registry support will
be done via MCP-over-XDS rather than in-tree
Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

RIP

@howardjohn: PR needs rebase.

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.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
ce04978d14841c140858e4e72032d011d9efb581
link
/test release-notes_istio

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.

@howardjohn: PR needs rebase.

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.

Discussed and approved in networking meeting last week

Isn't is safer to announce the deprecation first? So the users of consul can have more time before migrating to MCP

Isn't is safer to announce the deprecation first? So the users of consul can have more time before migrating to MCP

I believe we did announce this? And if not, this will be in 1.8 so we can announce deprecation in the 1.7 release

Better to have a release note

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
d0764c10eaf7b35d5c6bdbccb920071eb3688f4f
link
/test unit-tests_istio

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.

/test unit-tests_istio


Changes for 6.4 to docs

This change is

Coverage remained the same at 99.321% when pulling 85b9e7a677783389aa82314b785061a3f9fe9dd1 on justin808-6.4-changes-to-docs-1 into dcd2a928a2551efda8e6df974a5c2fa7252d283c on master.


train the lightgbm model with libsvm format dataset

Environment info
Operating System: linux + python3
CPU:
C++/Python/R version:
Error Message:
Traceback (most recent call last):
File "lightgbm_vas.py", line 39, in
clf.fit(train_data,train_label,eval_metric='auc',early_stopping_rounds=200)
File "/home/platform_user/anaconda3/lib/python3.6/site-packages/lightgbm/sklearn.py", line 675, in fit
callbacks=callbacks)
File "/home/platform_user/anaconda3/lib/python3.6/site-packages/lightgbm/sklearn.py", line 414, in fit
X, y = _LGBMCheckXY(X, y, accept_sparse=True, force_all_finite=False, ensure_min_samples=2)
File "/home/platform_user/anaconda3/lib/python3.6/site-packages/sklearn/utils/validation.py", line 521, in check_X_y
ensure_min_features, warn_on_dtype, estimator)
File "/home/platform_user/anaconda3/lib/python3.6/site-packages/sklearn/utils/validation.py", line 416, in check_array
context))
ValueError: Found array with 1 sample(s) (shape=(1, 1)) while a minimum of 2 is required.
Question
i want to train the lightgbm model with libsvm format dataset ,
`train_file = "../../sample/gbdt/train_ctr_2018-05-21_file"
train_set = lgb.Dataset(train_file)
train_set.construct()
train_label = train_set.label
train_data = train_set.data
print(train_set.num_data())
print(train_set.num_feature())
evals_file = "../../sample/gbdt/test_ctr_2018-05-22_file"
evals_set = lgb.Dataset(evals_file)
evals_set.construct()
evals_label = evals_set.label
evals_data = evals_set.data
clf = lgb.LGBMClassifier(
boosting_type='gbdt', num_leaves=31, reg_alpha=0.0, reg_lambda=1,
max_depth=-1, n_estimators=10000, objective='binary',
subsample=0.7, colsample_bytree=0.7, subsample_freq=1,
learning_rate=0.05, min_child_weight=50, random_state=2018, n_jobs=-1
)
clf.fit(train_data,train_label,eval_set =
[(evals_data,evals_label)],eval_metric='auc',early_stopping_rounds=200)
`
but i get error
ValueError: Found array with 1 sample(s) (shape=(1, 1)) while a minimum of 2 is required.

lgb.Dataset only works with lgb.train or lgb.cv, not for sklearn interface.


Need a better way to deal with normalisations in simultaneous XSPEC fits

Paul spoke about adding another multiplicative constant to the fit and allowing that to vary while freezing the normalisation, but I'm not clear on how that's any different to just letting the normalisations of the models vary unlinked?
Ideally I'd like a solution where I get one normalisation value out at the end, so I think I need to talk to him again about it. My ongoing quibble with this is that normalisation is directly linked to physical quantities, yet is different depending which instrument you happened to have measured the spectra with.

Right, I misremembered what Paul said before about how he and Ben did it in the past. After chatting to him again he told me that for each spectrum, a new multiplicative factor is added. For the first spectrum this is set to 1 and frozen, for all others it is free to roam. Then the normalisations for all spectra are linked and that normalisation is free to vary.
As such at the end you have these extra factors which describe the ratio of normalisations of spectra from different observations, and a single normalisation value which can be used for calculations etc.
The XCS LT pipeline does not do this, instead it loads in the spectra in one group and fits like that. I'm going to adopt Paul and Ben's way of doing it.

I have decided that this will be the only way to deal with normalisations in the built in model functions, I will no longer give the choice of linking or not.
When I write the general function (issue #435), then perhaps I will allow it, but not for anything else.

The way I'm planning this it won't require too many changes. I will add a new 'constant' multiplicative component to the tbabs*apec model, I'll remove the option to link norms or not, and the one new feature I'll have to add in the XSPEC script is the ability to freeze components at a individual model level (as the 'first' spectrum's constant needs to be frozen at 1)

As part of this I will also set a default model on get_temperature, most people should just use the single_temp_apec function which will now use the constanttbabsapec model, and they don't know thats the model name to type in

I'm going to add an extra flag that freezes the constant of the first model, rather than allow granular control of each every spectra's model components and whether they should be frozen. This is because I'm concerned about if the first spectrum in the list doesn't make it through the cleaning, in which case a completely general solution would fail.

It seems to be working in the single_temp_apec implementation now, though I need to put it into other models and propagate the fact that I no longer support not linking normalisations, this is what a (sub)set of models for a set of spectra look like in terms of the linking and freezing of parameters now:


HHH-9629 l2 cache key constructed wrong for entity with inheritance when fetching lazy property

Fix to use root entity name instead of plain entity name when creating cache keys for the second level cache.

Applied to both master and 4.3 branches.  Thanks Nicolas!


Don't offer flavours that don't work with our default image

A tech preview customer selected m1.small and the creation fails with a rather generic error Couldn't create node k8s-pool-test-fqt2v: Invalid request due to incorrect syntax or missing required parameters..
Digging through the nova logs I found Flavor's memory is too small for requested image. We should avoid giving the users a selection that is bound to fail.
We need to decide where we want to fix it, the openstack metadata call or the UI.

Hm. Odd. Small used to work

I always use small. So far haven't had an issue with it.

I have fixed the small image issue. Was caused by a external constraint in openstack. You can close this.

We now collect the flavors from Openstack.  If the UI picks the flavors from the metadata API, we could filter the list to only return working flavors.
Would also be good to have for custom flavors. O other Openstacks with different naming schemes...


Use peft_doc as notebook path as doc-builder expects _doc for the Google Colab and AWS Studio links

At the moment, the peft notebooks are generated in a path containing 'peft_docs', with the 's' at the end. This results in broken links in the online documentation.
For example: https://huggingface.co/docs/peft/main/en/task_guides/clm-prompt-tuning : the link to the pytorch version of the google colab notebook is https://colab.research.google.com/github/huggingface/notebooks/blob/main/peft_doc/en/pytorch/clm-prompt-tuning.ipynb , which uses 'peft_doc' without the trailing 's'.
The 'resolv_open_in_colab' method in doc-builder has the path _doc hardcoded at https://github.com/huggingface/doc-builder/blob/3543e0aca2029201051373e3b6ec871ef44d0d93/src/doc_builder/build_doc.py#L52 . This means it always uses a 'peft_doc' path for peft when generating the google colab and aws studio links. There's even a test which checks that the links use _doc for another package at https://github.com/huggingface/doc-builder/blob/main/tests/test_build_doc.py
It's likely the easiest to simply use peft_doc instead of peft_docs so that the automatically generated links are correct.

This would fix https://github.com/huggingface/notebooks/issues/417

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@DriesVerachtert @stevhliu what is the state of this PR?

I'm not sure this PR will fix the issue, and it is being internally investigated by @mishig25


[wizard2] small renamings

Standardized template related mapping components to TEMPLATE_MAPPING_COMPONENT and moved files into mapping_template subfolder
This will be easier to review after #1743 is in

I've synced this past #1743, ptal :)

that was fast! thanks!


In rootProject build.gradle file, the maven repository needs to be above the jcenter one otherwise will error out as time-out/cannot find version

repositories {
    maven { url 'http://dl.bintray.com/kotlin/kotlin-eap' }
    google()
    jcenter()
}

I'm unable to reproduce this error locally or via CI.  Can you try again?  Sometimes the repos have hiccups

You're right, I cleared my cache and still cannot reproduce the error.