Command syntax error - Register-CustomAdfs -CustomAdfsName Contoso -CustomADFSFederationMetadataFileContent $using:federationMetadataFileContent

Hi Team,
In the document, it is suggested to connect to PEP and then register-CustomAdfs :
+++
$federationMetadataFileContent = get-content c:\metadata.xml
$creds=Get-Credential
Enter-PSSession -ComputerName  -ConfigurationName PrivilegedEndpoint -Credential $creds
Register-CustomAdfs -CustomAdfsName Contoso -CustomADFSFederationMetadataFileContent $using:federationMetadataFileContent
+++
However, using variable can be used only with invoke-command, so we will get error:

Could you update the command as "Register-CustomAdfs -CustomAdfsName Contoso -CustomADFSFederationMetadataFileContent $federationMetadataFileContent" or not enter-pssession, juct invoke the session to run the command please? Thanks!
Regards,
Hailian Li

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

ID: d295c322-0813-e159-e6f8-86f04b759c48
Version Independent ID: 02b87d3f-125b-d674-b750-79dbaee5e497
Content: Integrate AD FS identity with your Azure Stack Hub datacenter - Azure Stack Hub
Content Source: azure-stack/operator/azure-stack-integrate-identity.md
Service: azure-stack
Sub-service: azure-stack-hub
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@Lihailian
Thanks for your feedback! We will investigate and update as appropriate.

@Lihailian
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


bug(List): List title handling

List should raise a warning if the given props titleKey do not match on an given list object property.

This issue was moved to Talend/ui#40


Impossible to open Preferences on Linux and Windows

rn the window looks like this:

I have not jet found a way to open the settings menu. Light theme hurts :smile: :radioactive:
I could imagine something like psst-gui --theme=dark.
psst-gui --help doesn't seam to be a thing yet since it start the gui normally.
Environment:
Debian 10 (installed via .deb package)
Version: 562f9df (Mar 10, 2021)

I have not yet found a way to open the settings menu.

Haha, we should definitely fix this, at the very least the shortcuts should work! Sorry, there isn't any command line handling yet, correct. In the meantime, you can switch the theme in the JSON config file. The path should get printed in the logs.
BTW, most of GTK apps on Linux don't use an application menu anymore, and the global (Mac-like) menu from the old Ubuntu days isn't used either, correct? We could add a hamburger icon to the top-right corner with a context menu for Linux and Windows.

Application menu is available in Gnome, but it gets complicated/"i don't know" how when there is a custom desktop environment like dwm envolved.
Hamburger sounds good (food and menu!).

Hamburger sounds good (food and menu!).

Oh and while we are at it, the tofu □ characters in place of emojis should get solved soon in Druid (by switching to Pango text rendering.)

Fixed in 0bb5258, I've added just a "Show Preferences" button instead of a full hamburger, as we don't have anything else to put there now :)

The icon(s?) looks overlayed on windows, not shure if intentional.


Fixed "type: string" to "type: String"

Actually, it should probably be Number, as validator and default value are for number.


Manual->Tutorial[s] vs. AQL->Tutorial - please harmonize.

chose one.

This will be considered as part of a larger restructuring effort.


Windows Easy Transfer in Windows 10

In the docs for the user state migration tool, it says that for a small number of computers the Windows Easy Transfer wizard can be used, but the WETW is not available for Windows 10.

Thanks for catching this! I've updated the topic to indicate that PCmover Express should be used intead, per https://support.microsoft.com/en-us/instantanswers/d4872cf1-5854-a363-2e14-2497f56f1505/windows-easy-transfer-is-not-available-in-windows-10
The changes will be live tomorrow, 2/3/2017

Thanks. I actually ended up using the User State Migration feature of our SCCM 2012 R2 system. It worked perfectly. I wish I had looked into that feature years ago!


Custom properties via api

I'm sending a exception using @simple_error and the events api. How do I add custom properties like a hostname. It's not in any of the examples.
This is for python for which I understand there currently isn't an SDK. Tia matt

Hey,
You'd just need to put it in the data dictionary and we'll just pick it up. If you have any questions please let us know. Also, We'd be more than happy to accept a pr or a gist for a python client/wrapper :)
Just a heads up, we may change how we index custom properties in the very near future (probably going to need to map them in configuration somehow, we have yet to discus, but sending them below wouldn't change).
{
  "data": {
     "HostName": "test"
  }
}


New Release

Version 5.0.0 does not work due to InfluxDB signing key URL changing:
TASK [boutetnico.influxdb : Manage InfluxData APT repositories] ****************
failed: [monitoring-instance] (item=deb https://repos.influxdata.com/ubuntu focal stable) => {"ansible_loop_var": "item", "changed": false, "item": "deb https://repos.influxdata.com/ubuntu focal stable", "msg": "Failed to update apt cache: W:GPG error: https://repos.influxdata.com/ubuntu focal InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY D8FF8E1F7DF8B07E, E:The repository 'https://repos.influxdata.com/ubuntu focal InRelease' is not signed."}

This was fixed in https://github.com/boutetnico/ansible-role-influxdb/commit/8fb28b3b5dfad932845b3e479f66aa28e23d8443 but a new release has not been created yet. Is is possible for a new release and version on ansible-galaxy be published? Thanks.
Current fix is to use role directly from github.

Thanks @JMuff22 for bringing this up, new release 5.0.1 is out.


fix(test): move integration test out from unit tests

High Level Overview of Change
There was an integration test that was in the unit test section. This PR moves that over.
Context of Change
The test was added in #520. I noticed that unit tests were suddenly taking much longer to run.
Type of Change

[x] Bug fix (non-breaking change which fixes an issue)
[x] Tests (You added tests for code that already exists, or your new feature included in this PR)

Test Plan
CI passes.

@mvadari the diff seems weird?

@mvadari These test are written to specific networks to get a specific network id response. Therefore we cannot use the IntegrationTestCase and must use an active network. This also means we cannot use WALLET because we need a wallet active on that specific network.
So I would like your opinion on what we should do?
@pdp2121 If you can elaborate at all please share. I tried to implement IntegrationTestCase without success.


Lisp nesting exceeds 'max-lisp-eval-depth'

After update nlinum-hl from MELPA right now I get the message above.
Also minibuffer shows: Variable binding depth exceeds max-specpdl-size. Backtrace is empty.

Well, with v1.0.3 the problem is gone. Nice.


build: use shared browserslist config

Removes custom browserslist configuration in favor of using a shared configuration.

This change reduces the resultant asset bundle size.

done in #212. closing this


Fix an ApiInformation.IsTypePresent check in Acrylic material doc

Checking if the Windows.UI.Xaml.Media.XamlCompositionBrushBase type is present and using Windows.UI.Xaml.Media.AcrylicBrush will cause application crash on W10 Creators Update; as XamlCompositionBrushBase was introduced with Creators Update and AcrylicBrush with Fall Creators Update.

@Sylvain2703 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@jwmsft - can you take a look at this?

#sign-off

#sign-off


Streaming reads of samples with read_eval_log_samples() function

This PR introduces a read_eval_log_samples() function that provides a generator over all of the samples in an eval log.
Note that accessing samples using a generator will result in lower memory usage only when using the .eval log file format (as it supports random access to individual samples whereas the .json format still needs to read all samples into memory).  If you have existing .json files you can easily batch convert them to .eval using the inspect log convert command.
There are a number of ways to incrementally read from a .eval log file: (1) Read just the header of eval logs with the header_only option; (2) Read individual samples using read_eval_log_sample(); or (3) Read samples iteratively using read_eval_log_samples() (which will ensure that only one sample at a time is read into memory):
# read only the header
log = read_eval_log(log_file, header_only = True)

# read a single sample
sample = read_eval_log_sample(log_file, id = 42)

# read all samples using a generator
for sample in read_eval_log_samples(log_file):
    ...

This looks great! Are in-progress runs supported or only completed runs?

This looks great! If I set all_samples_required=False, can I use this read_eval_log_samples to stream samples from in-progress runs?

You can, with two qualifications:

You can only stream completed samples (as we currently don't write samples until they are complete). Note that our upcoming realtime UI will enable you to "stream" (i.e. view) in progress samples.

We currently buffer samples (10 is the default). You can change this using the --log-buffer option. I'd expect that for local filesystem writes that a log buffer of 1 would be fine (unless your log is hundreds of megabytes). For remote filesystems with uncertain performance it could be a bit much to take the buffer down to 1 (again, especially for very large log files).


Changes robbyrussell/oh-my-zsh to new official repo ohmyzsh/ohmyzsh

Updating all references to legacy oh-my-zsh repository once the official repository moved from robbyrussell/oh-my-zsh to ohmyzsh/ohmyzsh on GitHub. 🙂

Updates?

@b4b4r07 updates?

@b4b4r07 could you merge this PR? this issue are blocking us from installing oh-my-zsh repo, thanks


Update README.md

Added info about the fork

Sorry, wrong button! I wanted to merge it in my fork but GitHub prioritized collaboration.


test: behaviors-message_on_lock_failure (2024-02-26 23:05:51.871328+00:00)

automatic test cleanup


AvionDb Roadmap April-May-Beyond

Roadmap
Within Next two months

[ ] Exposing pubsub channel to orbit-db-store interface

[ ] Secure, authenticated pubsub (Aviondb specific, or orbitdb specific?)
[ ] Non Identity based authentication. Tokens, symmetric keys, 3rd party oauth etc. Goes along with custom access controller.

[ ] Plugins. Non essential additions for authentication, data manipulation, secondary-higher level Peer-To-Peer communications.
[ ] Custom access controller. (Track Progress here)
[ ] Server daemon
[ ] CLI, import/export other operations if possible

Two months and beyond

[ ] Store (Collection) Sharding
[ ] File Management Support (#5). Start of tmfs standard.
[ ] Full scale mongodb TCP, HTTP API for 3rd party applications.

See supporting issues
AvionDb feature list
Orbit-db feature requirements

Is there a release date available for an HTTP REST API?
Would love to test them for some of the apps where Mongo is being used.

@0zAND1z I can pick this up after 3 July. I'm not sure about the release date though.

Is there a release date available for an HTTP REST API?
Would love to test them for some of the apps where Mongo is being used.
Cc @vasa-develop

I am making that sorta thing


chore(deps): update to stimulus 2.0.0

As a stepping-stone to 3.0.0, enabling this provides Console warnings
for deprecated values.
A blocker to getting to 3.x is some interplay between babel, gulp, jest, and ES6 targets. I spent some time poking at this unsuccessfully, and considered it would be wise to get this shipped for now, and iterate on getting to 3.x
Signed-off-by: Mike Fiedler<EMAIL_ADDRESS>

While working through this, I entertained adding cypress for UI testing, since I used it to load some of these pages and inspect the console.log for warnings stemming from Stimulus, as 2.0.0 surfaces the deprecation warnings prior to 3.x/
If there's interest in creating a UI Test suite for warehouse, let me know! There's a possibility to obtain an OSS license, surfacing feedback.


Use SPIR-V headers from the KhronosGroup/SPIRV-Headers repo.

In principle, this is good.
Don't we need a corresponding change in google/shaderc to react to the rename of the .inc file?
Also, we need to make an AOSP repository for SPIRV-Headers, and update the manifest for the NDK to include it.
-1  Can't include this change until the above are staged as well.

@dneto0: we don't need to do anything in google/shaderc for .inc files, but we do need to add build steps to pull SPIRV-Headers there.
See https://github.com/google/shaderc/pull/203.

LGTM, though still gated on AOSP work...

LGTM.
Question for @dekimir : Why is this gated on AOSP work?  The AOSP build only takes in new sources when we explicitly merge into its master.

@dneto0: at the time of my comment, the manifest branch still hadn't included SPIRV-Headers, so #includes would have failed.

@antiagainst : please rebase

@dneto0: done

Sorry, needs another rebase.

@dneto0: done

+2

Rebased and pushed into master as 10dba91781


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20220311163702215 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20220311163702215 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
14959

Relevant Lines:
29405

💛  - Coveralls


⚠ ClamAV has degraded performance

In 654ce7d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 716 ms

Resolved: ClamAV performance has improved in 4b24c99.


fix(flow) - flow fix me md imports

sets up config so that flow does not complain about .md imports

extracted from https://github.com/uber-web/baseui/pull/588

closing this in favor of #617


New function: Backup-DbaServiceMasterKey

name: Add Backup-DbaServiceMasterKey 🔐

Summary of new feature
Add a new Backup-DbaServiceMasterKey command, similar to the existing Backup-DbaDbMasterKey.
Proposed technical details (if applicable)
SMO exposes the same Export command used in Backup-DbaDbMasterKey on the instance's SMK:
$server = Connect-DbaInstance ServerA

$server.ServiceMasterKey.Export

OverloadDefinitions
-------------------
void Export(string path, string password)

T-SQL Command reference: https://docs.microsoft.com/en-us/sql/t-sql/statements/backup-service-master-key-transact-sql?view=sql-server-2017
Latest version of dbatools as of writing
0.9.809

🎉


readme is outdated

The readme and package godoc has c := qt.New in its example code, but New doesn't exist in this v2. :)

Good catch! The README now no longer mentions qt.New. I believe the C type can potentially be reintroduced here, but that's a conversation that we can have at https://github.com/go-quicktest/qt/issues/7


Does operator-sdk support additional printer columns?

Type of question
Are you asking about community best practices, how to implement a specific feature, or about general context and help around the operator-sdk?
Question
What did you do?
I want to add additional printer columns in my crds. I saw kubebuilder supports this feature, https://book.kubebuilder.io/reference/generating-crd.html#additional-printer-columns.
I am wondering how to set this in operator-sdk? Currently the only way I find is to add these configurations in the crd file.

@yeya24 Operator SDK uses the same underlying tool as kubebuilder to generate CRDs (controller-tools). Have you tried using the kubebuilder annotations for additional printer columns and then running the operator-sdk generate crds subcommand?

It does support additional printer columns, we use it in our CRDs (we're on operator-sdk 0.13.0 but it worked in earlier versions). For example we have the following Go struct:
// CoherenceRole is the Schema for the coherenceroles API
// +k8s:openapi-gen=true
// +kubebuilder:subresource:status
// +kubebuilder:subresource:scale:specpath=.spec.replicas,statuspath=.status.replicas,selectorpath=.status.selector
// +kubebuilder:categories=coherence
// +kubebuilder:resource:shortName=cohrole
// +kubebuilder:printcolumn:name="Role",type="string",JSONPath=".spec.role",description="The name of this role"
// +kubebuilder:printcolumn:name="Cluster",type="string",JSONPath=".metadata.labels.coherenceCluster",description="The name of the Coherence cluster that this role belongs to"
// +kubebuilder:printcolumn:name="Replicas",type="integer",JSONPath=".spec.replicas",description="The number of Coherence Pods for this role"
// +kubebuilder:printcolumn:name="Ready",type="integer",JSONPath=".status.readyReplicas",description="The number of ready Coherence Pods for this role"
// +kubebuilder:printcolumn:name="Status",type="string",JSONPath=".status.status",description="The status of this role"
type CoherenceRole struct {
	metav1.TypeMeta   `json:",inline"`
	metav1.ObjectMeta `json:"metadata,omitempty"`

	Spec   CoherenceRoleSpec   `json:"spec,omitempty"`
	Status CoherenceRoleStatus `json:"status,omitempty"`
}

Which generates a CRD with this in it:
apiVersion: apiextensions.k8s.io/v1beta1
kind: CustomResourceDefinition
metadata:
  name: coherenceroles.coherence.oracle.com
spec:
  additionalPrinterColumns:
  - JSONPath: .spec.role
    description: The name of this role
    name: Role
    type: string
  - JSONPath: .metadata.labels.coherenceCluster
    description: The name of the Coherence cluster that this role belongs to
    name: Cluster
    type: string
  - JSONPath: .spec.replicas
    description: The number of Coherence Pods for this role
    name: Replicas
    type: integer
  - JSONPath: .status.readyReplicas
    description: The number of ready Coherence Pods for this role
    name: Ready
    type: integer
  - JSONPath: .status.status
    description: The status of this role
    name: Status
    type: string
  group: coherence.oracle.com
  names:
    kind: CoherenceRole
    listKind: CoherenceRoleList
    plural: coherenceroles
    shortNames:
    - cohrole
    singular: coherencerole
  scope: Namespaced

Really thank you for your great collaboration, @thegridman. 🥇
Hi @yeya24,
Shows that all is sorted out. So, I am closing this one. However,  please, feel free to re-open if you think that it still required.


Handbrake wheels don't get added to passenger cars

I may have gone down quite the rabbit hole to find my missing handbrake wheel!
The wheelPositionsMap is populated before the passenger wheel position is added to the wheelPositions List. Just a simple order of operations mistake.
https://github.com/mspielberg/dv-handbrake/blob/68bbdf84a6bf81ab55b71d58c4f4dbd1547709ab/HandBrakeWheel.cs#L48-L57

I see this has been fixed in 0.3.7


New settings editor - hidden selection color

Issue Type: Bug
The Search settings field inside the new settings editor has no text selection color, or it is the same as the fields background.
When text is selected, one cannot see it.
Also, when pasting something in the middle of the text, the cursor goes all to the right.
Here is a screen recording of the problem

I'm pressing Ctrl+X and Ctrl+V to cut/paste.
This issue is related to this comment: https://github.com/Microsoft/vscode/issues/50249#issuecomment-415794630. Initially I was thinking that selection did not work. It works, but no color appears.
VS Code version: Code - Insiders 1.27.0-insider (47d0584301d16d4efd6618f6f33ef14099c92561, 2018-08-29T05:18:47.523Z)
OS version: Windows_NT x64 10.0.15063

This is probably from your theme, which theme are you using?

Maybe same as https://github.com/Microsoft/vscode/issues/57281

The two issues are probably the same.
The extensions search bar has the same problem.
Looks like it is the theme indeed. I am using molokai.
Should I open an issue in the theme's repository instead?

No, I think we are not respecting the theme color correctly.


Field name in lower case

I have several stored procedures in mysql, which is returning data from table. In this tables all fields have naming convenience as proper case (eg. UserID, UserName, FirstName, LastName ). It is ok when I have retrieve this values using select statement in mysql workbeanch, But If I retrieve using node-mysql, then I want it in lower case. (only column/field name should be convert in lower case). Is it possible or How do I do it?
I don't want to write field name with alias in select statement.
thanks.

Sounds like this was a shut case several years back, but I do want to provide a reason why something like this might be desired. It mainly has to do with javascript convention to use camelCasing for variable names, other than cases for Constructors which are PascalCased. I do know that SQL convention is PascalCasing.
Since mysqljs lib sits in between SQL and JS land, I think it can be reasonable to have this as a feature. I worked using the TediousJS lib (MS-SQL) in the past, and that library had that support built-in (http://tediousjs.github.io/tedious/api-connection.html look for options.camelCaseColumns) so I don't think it's a completely unreasonable idea.
With all that being said, in the meantime, is there an official "filter" process that @sidorares mentioned? Something a la middleware pipeline, or was that just a general reference to adding a function to the callback? Also, is there a way one can add a global query callback that will fire on all queries?


Add a License?

Hi, I am interested in using part of qwertigraphy in another project. I was wondering if you could add a license to the repository to state the rights others have over the source code. The github docs may be a helpful resource for choosing and adding a license.

Hello chanicpanic. This was a great question, and I had to think about what I want out of this project. In the end, it's basically nothing but for people to get to try it. I chose MIT, and I wish you the best of luck. I'd love to hear/see what you are building, and hope you have the best of success.

Thank you! I'm glad you are willing to share your great project. I am planning to use it in Grascii which contains a tool for searching reverse Gregg Shorthand dictionaries. At the moment it has a pre-anniversary dictionary, and I am hoping to expand by converting some of qwertigraph's anniversary dictionaries into Grascii's format.


PR Template requires some fixes

Description
insted of ### Solves Issue #  it should be Closes #
 The title should be short and relevant.

Updated.


Flexibile icon

Describe the requested icon
An arrow that splits into three arrows, indicating the ability to adapt or branch out into different directions
Screenshots

Would arrows-split or git-branch suit your needs?

Also graph, tree-structure


is it possible to do inpaint conditioned on another image?

Instead on conditioning on a text prompt, can we somehow use controlnet to condition on another image and replace an masked area with this new image?
Thanks

Yes, and there are many ways to achieve that.
First, you can turn off the text prompt by either using an empty string "" or by setting the parameter of guidance_scale=0.0 (as explained in the documentation, this corresponds to the strength of text guidance).
For mixing two images, you can potentially do the following:

Combine the two images using the mask, yielding a composite image
Extract ControlNet guidance, such as canny edge (as shown in the example notebook), from the composite image
To the ControlNetInpaint pipeline, feed as input the composite, the mask, and the ControlNet guidance

I haven't tested this approach and there are many more ways to do that. If there is more interest in this use case, I could potentially include it in the notebook and propose some solutions!

Hey @mikonvergence ,
I'm also trying to combine the images. I'm trying to insert an object into another image instead of the text prompt.
Can you include in the notebook or maybe guide us on what are the ways to achieve that?
Thanks

Hi @gersonvneto! Yes, definitely! I can include an example for that in the notebook (or perhaps a separate notebook in the repo) soon.
Can you tell me whether you're more interested in a scenario where

(a) the pasted object remains exactly as is, and the network harmonises transition between the pasted object and background
or (b) the pasted object undergoes some changes in colour and lightning to make it work with the rest of the image?

If anyone wants to, they can provide the source images for this operation I will use as the example. Otherwise, I'll try to find something of my own and hope it's relatable to other users!

The first scenario (a) is the one that I'm trying to achieve. I believe that the object could have a more realistic look if it fits the image it was inserted into, as is.
The method you proposed had the object pasted looking different, maybe somewhat similar to (b).

These are two generic pictures, but my goal was to insert the lamp somewhere in the living room

I am trying to achieve a similar goal :)

I'm still trying to figure out what we would like the model to do.
In the lamp example, there isn't much to do in terms of the mask border, so scenario (a) doesn't seem to be useful in this case. I feel like the changes that would be desired here are adjusted perspective of the object (especially the stand) and the color/lighting changes (so closer to scenario (b)). While the color and lighting can probably taken care of, fixing object perspective is probably something that requires further research.

Yes, or sometimes the background of the object is blurry. In my case would be nice to do both things. Incorporate an object in a scene and change it perspective

Yep, that probably was not the best example.
Maybe the exemple below it is a better one. Where the picture have a background and a different lighting.

ControlNet does not really do masking of the source image, this should be taken care of before using the tool.
Once you have a masked object you would like to include in the correct location, you could follow the process I described in the first reply.
At some point, I might look into implementing an app that helps with that process if you're interested.

I tried folowing your suggestion but I got a bad result. I will organize on a notebook and send it here. Thanks

here is the notebook. https://colab.research.google.com/drive/1Egx6VCiIqfOlpzNW40ziwlEV7xwM7_3m#scrollTo=sAEKqQl8ykq-
maybe my masking was not good?

Thanks @themrzmaster - can you also share the images you used in the notebook?

![mesa](https://user-images.githubusercontent.com/852747/228023225-d170a865-9256-4619-b47c-7dff4b248bc2.jpeg)
![a5cba229dd730cb2042d68775de6d1e2](https://user-images.githubusercontent.com/852747/228023253-cd699104-ce9e-45ef-87fb-3a026b6f9ebf.jpeg)
yes! thanks

Okay, there are a few issues with the use of the pipeline shown in the notebook:

The mask image needs to be a binary value that indicates which area must be inpainted
Your control image seems to be inconsistent with the composite image - notice the edge of the carpet in your control image that overlaps with the table (so some fraction of the edge should actually be invisible in the output image)
The table needs to be inserted into the process by either (a) masking out only the white background of the pasted rectangle, or by (b) combining the control image of the room with the control image of the table while masking out the entire table region

The tool has not been used correctly, and you will get better results if you use it as intended. However, it also seems like you're working on a more complex problem (related to control-based inpainted, but not exactly the same) and might need to design a solution tailored to your problem.

this might be a good way to do it https://arxiv.org/pdf/2303.17546.pdf

Yeah, I remember looking through that paper - it's quite neat! It does look like it can bring you closer to the solution to your problem than ControlNetInpaint.
Feel free to post some results in this thread if you manage to get some interesting outputs! :rocket:


Update the get_teams endpoint's query set

Set the query set of the get_teams endpoint to only return teams from the currently ongoing jam.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 3 (66.67%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 80.923%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

api/routers/teams.py
2
3
66.67%

Files with Coverage Reduction
New Missed Lines
%

api/routers/teams.py
1
47.27%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
666

Relevant Lines:
823

💛  - Coveralls


Arg_NotSupportedException when calling executor.Reset()

title

Code: internal void Step()
{
try
{
ExecutionEnded = !executor.MoveNext();
if(ExecutionEnded)
{
executor.Reset();
executor.MoveNext();
}
}
catch(Exception ex)
{
SYS32.KernelPanic(ex, "Coroutine");
}
}

@ascpixi @zarlo

It looks like you're inheriting a class from Coroutine. You shouldn't really do that - what are you trying to achieve?

i'm trying to make repetative tasks

What do you mean by "repetitive tasks"? If you simply want to start multiple coroutines that run in infinite loops, please refer to the README file. Here's an example:
var coroutine = new Coroutine(MyCoroutine1());
coroutine.Start(); // will run the coroutine on the main pool; to run it in another, use CoroutinePool.AddCoroutine

// ...

IEnumerator<CoroutineControlPoint> MyCoroutine1()
{
    while(true) {
        Console.WriteLine("This prints every second.");
        yield return WaitFor.Seconds(1);
    }
}

More details are in the README file. Please be sure to yield return in order to return control back to the scheduler. You shouldn't inherit any class other than the CoroutineControlPoint class.

oh thanks


AttributeError: 'Worker' object has no attribute '_global_rank'

is the lstm example working ? thanks

if I remove this line i get :
'Worker' object has no attribute '_multinode'

I got this error, too. Can anybody help? I installed every dependency that platoon need. But  still got this error.


Flutterfire apps no longer show a confusing log message about configuring default app

Fixes flutter/flutter#16534
Note that this doesn't completely fix the issue, it just changes the error message from the previous, really confusing:
"4.8.1 - [Firebase/Core][I-COR000003] The default Firebase app has not yet been configured.
Add `[FIRApp configure];` (`FirebaseApp.configure()` in Swift) to your application
initialization.

New log message:
[Firebase/Core][I-COR000004] App with name __FIRAPP_DEFAULT does not exist.

I switched to using patch versions, will land this once the bots are happy

test?

test?

I don't think that our current infrastructure can test this change, because the only observable impact is in the iOS device logs.


adding extract option for pdftotext

Adding an option for author to add extract option like -table or -layout.
Refactored main.py, invoice_template.py and pdftotext.py to PEP-8

Solves #108

@m3nu Merge this?

We can't add if-conditions like that. To pass arguments to the extract module, there should be a universal way that works the same for all modules.

But this feature is only for pdftotext module, how can we make it universal when no other module uses such extract layouts?

Other extract options may need special settings as well. If you want to pass those, there needs to be a universal way. Like ffmpeg has for example for filter options.
If you think it's not worth doing it properly we can't add this feature for now.


Checkpoint files possibly not created with correct name from fsi runs

Exploring the weirdness with checkpoint files on non-precursor checkpoint creation.
Problem:
During the precursor openfastcpp run, the code knows about all the turbines and so can properly write out turbine ids and openfast filenames, but this doesn't happen on subsequent restarts when each nalu instance only knows about a single turbine.

Opened against wrong remote

Opened against wrong remote

For future reference, it is possible to change the target branch on an open pull request.


Adding support for VPC MongoosT-50CM3 Throttle

Adding support for VPC MongoosT-50CM3 Throttle and added a workaround for split 32 button mode issue #75

This repo is closing in favour of a successor repo https://github.com/brammmers/edrefcard2
If this is still an issue, please re-open it there.
Thank you!


tests occasionally fail because no prometheus-tester units show up

From the logs:
INFO     juju.model:model.py:2679 Waiting for model:
  prometheus-tester (not enough units yet - 0/1)
# ...
INFO     juju.model:model.py:2679 Waiting for model:
  prometheus-tester (not enough units yet - 0/1)
FAILED

____________ test_prometheus_scrape_relation_with_prometheus_tester ____________
Traceback (most recent call last):
  File "/home/runner/work/prometheus-k8s-operator/prometheus-k8s-operator/tests/integration/test_charm.py", line 81, in test_prometheus_scrape_relation_with_prometheus_tester
    await ops_test.model.wait_for_idle(apps=[tester_app_name], status="active")
  File "/home/runner/work/prometheus-k8s-operator/prometheus-k8s-operator/.tox/integration/lib/python3.8/site-packages/juju/model.py", line 2677, in wait_for_idle
    raise jasyncio.TimeoutError("Timed out waiting for model:\n" + busy)
asyncio.exceptions.TimeoutError: Timed out waiting for model:
prometheus-tester (not enough units yet - 0/1)

For example a rerun of the following passed:
https://github.com/canonical/prometheus-k8s-operator/runs/4710553937?check_suite_focus=true#step:4:303

Possible solution:
    await ops_test.model.applications[app_name].remove()
+   await ops_test.model.block_until(lambda: app_name not in ops_test.model.applications)
    await ops_test.model.reset()


Clarify and prioritize overall project purpose

This PR proposes the following:

Make "collaboration and common purpose" our top priority.  If we are successful, it's because of our ability to work together.  This map is first and foremost a reflection of our community and we should prioritize our community perspective over any strict adherence to external principles.
Explicitly declare that we take our inspiration from paper maps from this region (however, this is secondary to our community opinion).

I'm overdue to make some trips to the library but perhaps some links to some paper map resources/indexes if anybody knows any would also be a good idea? Could even just be links to github comments where people have shared good pictures.

Directly including paper map samples in the repo itself would be problematic for a CC0 repository, but we could certainly describe the types of maps or link to my talk at State of the Map US: https://www.youtube.com/watch?v=4T8FxOPlqXA
It sounds like we need to create a section that specifically discusses our cartographic principles and I think that deserves its own discussion issue beyond this high-level purpose PR.


fix: Fail to deploy fate-operator successfully using make deploy

[bug] Fail to deploy fate-operator successfully using make deploy #19

/lgtm
/approve


Devices for Security Findings

OCSF security findings don't have a named relationship to devices, but security findings from EDR/EPP platforms often concern devices. This PR proposes we add this relationship. See enhancement-proposals/001-security-finding-to-device.md for details.

@srotsinha and @query-jeremy Maybe I missed what you meant: are you talking about the non-scalar observables, like securityFinding.observable.device as an alternative to securityFinding.device? I've kind of blocked those from my memory so I might have misinterpreted what you meant.

@srotsinha and @query-jeremy Were you talking about the non-scalar observables, like securityFinding.observable.device as an alternative to securityFinding.device? I've blocked those from my memory so I might have misinterpreted what you meant.

I think @query-jeremy was talking about scalar observables and why we shouldn't be using them in his proposal.
I, on the other hand, was proposing that make use of non-scalar observables. But we treat observables as nested data rather than a relationship for which query-engine would create plans fetch from different data sources and then join them.
I have mentioned my reasons above. But the TLDR is that customer data is very unpredictable and we may fail to map objects like user, device, file because ocsf does not describe a relationship. Extending schema is always possible but it is a very slow and lengthy process in which we require knowledge about multiple data sources and more rounds of discussion is required internally as well as within ocsf community.
We should talk about what the pros and cons are of using non-scalar observables. The query will become nested but our query expression can very well handle it.

To clarify: I think we should continue to use the scalar observables.
I'm now torn as to which approach would be best:

Add a device attribute.
Support non-scalar observables.

I think #2 is the right approach for most devices described in log data. Logs are point-in-time. Even CrowdStrike's agent ID isn't a permanent identifier, as it's lost when the agent is removed and reinstalled (something Matt S had real-world examples of this).

To clarify: I think we should continue to use the scalar observables.
I'm now torn as to which approach would be best:

Add a device attribute.
Support non-scalar observables.

I think #2 is the right approach for most devices described in log data. Logs are point-in-time. Even CrowdStrike's agent ID isn't a permanent identifier, as it's lost when the agent is removed and reinstalled (something Matt S had real-world examples of this).

I am not sure how common it is to reinstall agent. But I have faced issues with updating agent on my ubuntu machine in the past (haven't tried it on mac) and had to re-install

I forgot to mention that Eric confirmed CrowdStrike does indeed record point-in-time snapshots of the hostname and IP address, possibly other fields too.
On our platform today, observables are things that were observed when the event happened. If we want to maintain the semantics that

observables do not come from platforms other than the event source, and
observables contain only point-in-time snapshots of data, and
the rest of the related records represent the "current state"

then the right thing to do IMO is to put the device info in observables[n].device. It won't be very practical today, because we can't efficiently return the list of all observables when there is an any filter on observables. But that can be solved in time if we can stick to "observables do not come from platforms other than the event source".
We can abandon the semantics for observables, but I don't think it's going to be a simple matter.
My vote is:

add security_event.device which would contain the "current state" of the related device
follow-up queries on security_event.device for CrowdStrike will fetch the current device state
put CrowdStrike's snapshot of the device in observables[n].device
put the snapshot of IP address, hostname, etc in observables.value
the UI won't retrieve the snapshot data today, but eventually we will implement any filters in drivers, and then overfetching events won't be a problem.


Unhelpful error message when boxing blocks

The motivation is wrapping a block into a box in order to facilitate a nicer API. (using orElse { ... } { ... } instead of orElse(box { ... }, box { ... }).
I don't quite know whether this is just a bad error message or if it's an actual bug in Typer:
Here's the offending code:
interface OrElse {
  def orElse[T](m1: => T / {} at {}, m2: => T / {} at {}): T
}

def orElse[T] { m1: => T / {} }{ m2: => T / {} }: T / {OrElse} = {
  do orElse(box { m1 }, box { m2 })
}

which returns the following unhelpful message:

If I just ANF it, I get the same message. When I remove the braces, I get a different error:
interface OrElse {
  def orElse[T](m1: => T / {} at {}, m2: => T / {} at {}): T
}
def orElse[T] { m1: => T / {} }{ m2: => T / {} }: T / {OrElse} = {
  val f1 = box m1;
  val f2 = box m2;
  do orElse(f1, f2)
}

returns:

I think there are a few different issues here. The first error message is indeed confusing.
However, the second message is expected: you are passing f1 which has capture m1 to the operation which requires an empty capture set.
On a high level: I deeply understand the motivation of avoiding box in the API but I think it's not possible this way.
Maybe we can make it more lightweight by changing syntax somehow

On a high level: I deeply understand the motivation of avoiding box in the API but I think it's not possible this way.

I thought so, but I still wanted to report the confusing error message — dealing with boxing and captures is difficult already.

Update: the error is slightly different now (unfortunately still unhelpful though...)

Update: Current error on 0.3.0:

This issue popped up again, on 0.9.0, the following code:
record Container(eqz: Int => Bool at {})
def empty { eqz: Int => Bool }: Container =
  Container(box { eqz })

reports "Expected identifier" on the very last character (the closing paren) which is extremely unhelpful :(


manual cherry pick

Manual cherry pick of https://github.com/istio/istio/pull/23458 (Let --force really work in 'manifest apply') and https://github.com/istio/istio/pull/23613 (Guard against null bool in "manifest generate")

@esnible: 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.

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

Test name
Commit
Details
Rerun command

gencheck_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test gencheck_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.

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

Test name
Commit
Details
Rerun command

gencheck_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test gencheck_istio

integ-distroless-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-distroless-k8s-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.

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

Test name
Commit
Details
Rerun command

gencheck_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test gencheck_istio

integ-distroless-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-distroless-k8s-tests_istio

integ-mixer-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-mixer-k8s-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.

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

Test name
Commit
Details
Rerun command

gencheck_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test gencheck_istio

integ-distroless-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-distroless-k8s-tests_istio

integ-mixer-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-mixer-k8s-tests_istio

integ-telemetry-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-telemetry-k8s-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.

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

Test name
Commit
Details
Rerun command

gencheck_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test gencheck_istio

integ-distroless-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-distroless-k8s-tests_istio

integ-mixer-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-mixer-k8s-tests_istio

integ-telemetry-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-telemetry-k8s-tests_istio

integ-galley-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-galley-k8s-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.

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

Test name
Commit
Details
Rerun command

gencheck_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test gencheck_istio

integ-distroless-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-distroless-k8s-tests_istio

integ-mixer-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-mixer-k8s-tests_istio

integ-telemetry-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-telemetry-k8s-tests_istio

integ-galley-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-galley-k8s-tests_istio

integ-security-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-security-k8s-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.

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

Test name
Commit
Details
Rerun command

gencheck_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test gencheck_istio

integ-distroless-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-distroless-k8s-tests_istio

integ-mixer-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-mixer-k8s-tests_istio

integ-telemetry-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-telemetry-k8s-tests_istio

integ-galley-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-galley-k8s-tests_istio

integ-security-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-security-k8s-tests_istio

release-test_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test release-test_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.

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

Test name
Commit
Details
Rerun command

gencheck_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test gencheck_istio

integ-distroless-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-distroless-k8s-tests_istio

integ-mixer-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-mixer-k8s-tests_istio

integ-telemetry-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-telemetry-k8s-tests_istio

integ-galley-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-galley-k8s-tests_istio

integ-security-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-security-k8s-tests_istio

release-test_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test release-test_istio

integ-pilot-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-pilot-k8s-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.

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

Test name
Commit
Details
Rerun command

gencheck_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test gencheck_istio

integ-distroless-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-distroless-k8s-tests_istio

integ-mixer-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-mixer-k8s-tests_istio

integ-telemetry-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-telemetry-k8s-tests_istio

integ-galley-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-galley-k8s-tests_istio

integ-security-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-security-k8s-tests_istio

release-test_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test release-test_istio

integ-pilot-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-pilot-k8s-tests_istio

integ-operator-controller-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-operator-controller-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.

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

Test name
Commit
Details
Rerun command

gencheck_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test gencheck_istio

integ-distroless-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-distroless-k8s-tests_istio

integ-mixer-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-mixer-k8s-tests_istio

integ-telemetry-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-telemetry-k8s-tests_istio

integ-galley-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-galley-k8s-tests_istio

integ-security-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-security-k8s-tests_istio

release-test_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test release-test_istio

integ-pilot-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-pilot-k8s-tests_istio

integ-operator-controller-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-operator-controller-tests_istio

integ-local-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-local-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.

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

Test name
Commit
Details
Rerun command

gencheck_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test gencheck_istio

integ-distroless-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-distroless-k8s-tests_istio

integ-mixer-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-mixer-k8s-tests_istio

integ-telemetry-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-telemetry-k8s-tests_istio

integ-galley-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-galley-k8s-tests_istio

integ-security-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-security-k8s-tests_istio

release-test_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test release-test_istio

integ-pilot-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-pilot-k8s-tests_istio

integ-operator-controller-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-operator-controller-tests_istio

integ-local-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-local-tests_istio

lint_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test lint_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.

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

Test name
Commit
Details
Rerun command

gencheck_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test gencheck_istio

integ-distroless-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-distroless-k8s-tests_istio

integ-mixer-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-mixer-k8s-tests_istio

integ-telemetry-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-telemetry-k8s-tests_istio

integ-galley-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-galley-k8s-tests_istio

integ-security-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-security-k8s-tests_istio

release-test_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test release-test_istio

integ-pilot-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-pilot-k8s-tests_istio

integ-operator-controller-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-operator-controller-tests_istio

integ-local-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-local-tests_istio

lint_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test lint_istio

unit-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
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.

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

Test name
Commit
Details
Rerun command

gencheck_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test gencheck_istio

integ-distroless-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-distroless-k8s-tests_istio

integ-mixer-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-mixer-k8s-tests_istio

integ-telemetry-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-telemetry-k8s-tests_istio

integ-galley-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-galley-k8s-tests_istio

integ-security-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-security-k8s-tests_istio

release-test_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test release-test_istio

integ-pilot-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-pilot-k8s-tests_istio

integ-operator-controller-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-operator-controller-tests_istio

integ-local-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-local-tests_istio

lint_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test lint_istio

unit-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test unit-tests_istio

integ-multicluster-k8s-tests_istio
28ad7123bf6dafeebeac1f02edfc9d1f394b8695
link
/test integ-multicluster-k8s-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.

Closing because I based it wrong


⚠ TorrentLeech has degraded performance

In b8e303f, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 4021 ms

Resolved: TorrentLeech performance has improved in 16dc330 after 6 minutes.


quickstart project/tutorial not explained deeply enough...

I've used files from quickstart project, i've copied them to:
mypro\src\main\resources\public (as this is a place for public files in my Spring-Boot project)
also i've run npm install (there is node_modules folder within public folder now - not sure if it's a good idea to store 90MB of data in a public folder....)
Where should i place tsconfig.json ? in mypro\src\main\resources\public together with other files ?
However typescript plugin from Eclipse doesn't want to pickup app*.ts files.... (plugin works with simple typescript examples).
Quickstart tutorial lacks explanation of project structure and configuration and i guess that keeping node_modules (sources, configuration etc.) in a public folder is a bad habit.
In AngularJS1.4 it was enough to 'import' files with a < script src=... > tag in index.html
I'm confused now...

You can find the whole Quickstart app here:  Angular 2 Quickstart

yes, i know, i took code from it.
but how to use it/integrate with Java backend project ? i dont want to use nodejs

but how to use it/integrate with Java backend project ? i dont want to use nodejs

In fairness, that's a little beyond the scope of a quickstart, no?

And beyond the scope of this issue tracker I am afraid.


Adjacent links to the same CC URL

At the bottom of the page there are two links two CC links, "Creative Commons CC0 1.0 Universal Public Domain" and "Creative Commons CC0 1.0 Universal Public Domain Dedication" that both go to  https://creativecommons.org/publicdomain/zero/1.0/. This creates a lot of redundancy for screen readers and, apart from that, I can't really divine the difference between the designation and the dedication. Should be rewritten so that there is only one link.

Updated Footer language. We can now use the following footers:
The data produced through this project is covered by a Creative Commons CC0 1.0 Universal Public Domain dedication. By participating in this project, you agree that any copyrightable content you submit shall be subject this Creative Commons CC0 1.0 Universal Public Domain dedication.


Add Example of accessing Azure Settings from Blazor Server Side Service

Please add an example of accessing the Azure stored settings from a Blazor Server Side Service.  For example, accessing the database connection string stored in Azure App Configuration by the Server Side Blazor service.

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

ID: 2984100d-7d78-2115-40f4-4b9478a5a5bc
Version Independent ID: 24ac6adc-6779-b847-ce81-c9d36e4b067b
Content: Quickstart for Azure App Configuration with ASP.NET Core
Content Source: articles/azure-app-configuration/quickstart-aspnet-core-app.md
Service: azure-app-configuration
GitHub Login: @yegu-ms
Microsoft Alias: yegu

@leftmostlane Thank you for your feedback. I will assign this ticket to the doc author to work on.

@leftmostlane Thank you for your feedback! We're working on Blazer support. This is being tracked here.
#please-close


extract ErrorBoundary style

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Other... Please describe: Security improvement

What is the current behavior?
Inline styles are an attack vector 0, 1, 2. They may be prevented with the Content Security Policy style-src 'self'; as recommended by OWASP.
What is the new behavior?
This PR moves the inline style out to a stylesheet so users can use style-src 'self';.

If I understanding the issue: With this setting turned on inline styles are completely disabled. What we are doing here is not a security vulnerability because we aren't using CSS variables. But if one would want to protect against a potential security vulnerability they would want to enable this setting to disable inline styles.
Did you check that it works? I'm not seeing the CSS file being included anywhere.

Did you check that it works?

...nope 🤦♂
Upon checking, I realized that the dev-overlay, specifically shikiji, also uses inline styles, so this PR would not ultimately fix the problem.
A future possibility could be to put the dev-overlay in an iframe... I think that might isolate it from a Content-Security-Policy of style-src 'self';. However I'm not sure how to load Solid into something an iframe can host, so I'm closing.

we aren't using CSS variables

Ah, one of my OP's links was misleading; I removed it. CSS variables are not relevant to this issue.


Webdav access broken with 2.6

After upgrading when I try to mount a webdav space through Gnome's Nautilus my credentials get refused and the following gets logged:
2017-08-18 04:19:04:LOG_INFO:f75ab4abb33bcfd2a4853f45b6a5cf05:BasicAuth.php:114:validateUserPass: module not enabled for testchannel
Looking at the code it seems that the function check_module_access will only return true if the module is cdav and not dav. Adding dav made it work again...

Must be a regression from the cdav integration. I used DAV several days ago without issue. We may not require this function any more, but the correct fix if it is to remain is:
   protected function check_module_access($channel_id) {
        if($channel_id && in_array(\App::$module,[ 'dav', 'cdav', 'snap'] )) {
            return true;
        }
        $this->module_disabled = true;
        return false;
    }

Works for me now! Thanks, as always =]


Stop pulling in machine-os-content

As part of https://github.com/openshift/enhancements/blob/master/enhancements/ocp-coreos-layering/ocp-coreos-layering.md we have switched to a new rhel-coreos-8.
Really, nothing else besides the MCO should be pulling in or referencing the OS images.  I don't understand what cross-checking the image stream tag was ever doing at all.  The MCO owns rolling out the right OS version, and if that isn't correct something else went seriously wrong.
Closes: https://github.com/openshift/driver-toolkit/issues/101

/lgtm
/retest-required

/approve


Android: Bugfixes

Handfull of bugfixes for the current Beta including at least one critical fix.  Still holding back new features until this gets out to Prod.

Release as 8.0.0b4.

Beta is looking super clean.  I'm fine with promotion at any time.

Anything to add to this list?

The grid view comes to Android. Each feed/folder has its own List view setting.
New story pager has smooth transitions and fast updates.
Fix for Twitter videos.
Fix crash for viewing muted feeds.
Long press links and images without tags.
Fix issue where thumbnails and cached story imaged would be double downloaded.

Released to both the Play Store and updated the Amazon App Store.

New story pager has smooth transitions and fast updates

The new pager isn't until the next release, this one just contains a ton of cleanup in preparation for it.  User-facing changes should be minimal.

Fix issue where thumbnails and cached story imaged would be double downloaded

This bug never made it out of Beta, so shouldn't be relevant to Production users.
Other new stuff:

Can scroll all the way past the last story (for marking-read while scrolling through story list)

Other bugfixes:

Better login screen for very small devices
Thumbnail loading much more memory and network efficient
Story list position retained on rotate
Many Android 8 compatibility fixes
Colourblindness-friendly intel training

But mostly it is the fancy new story list with grid support and much, much smoother loading.

Blogged: http://blog.newsblur.com/post/176089041176/the-newsblur-android-app-hits-version-80-with-a

Looks great!
Blog post seems to have some issues with image aspect ratios - first
one is too tall, second one is too wide.
Nicholas

Yikes, thank you for letting me know. Looks like Tumblr rewrote my img tags when I hit preview one last time.


Docs/redux

Very detailed architecture description. LGTM 👍

TODO

[ ] Add directory structure overview
[ ] More introductory stuff in README.md
[ ] Describe missing parts (redux, services, utils)
[ ] Document testing practices


PropertyFieldColorPicker unable to set alpha value

Category
[ ] Bug
[ ] Question
Unable to set the alpha color property value in the PropertyFieldColorPicker control. Because of this when the focus is out of the property pane, the value goes back to RGB only instead of RGBA.
The color control should work as shown in the below example to get the RGBA values
https://casesandberg.github.io/react-color/
Note: Happens only when style: PropertyFieldColorPickerStyle.Full

Unfortunately, I can't seem to recreate this issue (or I'm not understanding). Here is a gif of what I am seeing:

Could you provide more details about the issue, specifically:

Does this only occur when dragging the alpha slider?
Does this only occur when editing the alpha value directly?
Are you using Reactive Property Changes (as shown in the gif)?

Thanks!

Hi,
The issue occurs when I close the property pane and come back to the web part. As you shown in this gif, I can not reproduce this issue when I change the value and keep the property pane open.
Could you please let me know the tool you are using to convert the screen recording to gif? I will record and share the behavior.

Thank you for the gif! This helped me recreate the issue and identify the problem. The issue occurs when the PropertyFieldColorPicker is the first (or only) property pane field and the style is set to Full.
The cause of the problem is how the ColorPicker control handles the blur event on the Hex textbox. When the Hex textbox loses focus, the value is evaluated and everything is updated. Since hex values don't represent alpha values, the alpha value is always set to 100%. This is always true and can be seen on the UI Fabric React Component's demo page as well (change the alpha value, click into the hex textbox, then click out. The alpha value will be reset to 100). This is expected behavior and is inline with the UI Fabric components (even if not great).
However, the confusion occurs because when the property pane is opened, focus is set to the first input in the property pane. When a PropertyFieldColorPicker is the first field, focus is set to the Hex textbox. This results in the confusing behavior of changing the color using the sliders (all the while the focus remains in the Hex textbox) only to have the alpha value reset to 100 when the Hex textbox loses focus.
I have solved this issue by including a hidden input above the ColorPicker when the style is set to Full. This has no effect when the PropertyFieldColorPicker is not the first field in the property pane, but intercepts the focus when it is. This eliminates the confusing behavior of letting users use the sliders to configure values only to have the alpha value reset on them. Here's what it looks like with the new behavior (green click indicator circle added):

Thank you very much for bringing this to our attention!

Hi Chris,
Thank you for your response.
I have verified the code change with version 1.5.1-beta.3404e60 and it looks good except the first time. When I change the Alpha value for the first time it goes back to hex value. Am I using the right latest version of your code?
Providing additional values such as R,G,B,Alpha and Hex on top of color outcome will help a lot.
Example: https://casesandberg.github.io/react-color/
background: rgba(${ this.properties.color.r }, ${ this.properties.color.g }, ${ this.properties.color.b }, ${ this.properties.color.a })

The fix has been merged and is now available in the v1.5.1 beta. If you want to test out the beta, here you can find more information how you can do this: https://sharepoint.github.io/sp-dev-fx-property-controls/beta/

Looks cool and issue resolved. Thank you Chris and Elio.
Can you do one more change? The icon we set using the iconName property is not aligned properly in the IE 11
Chrome

IE 11

Glad to hear it! Thanks again for taking the time to submit this issue!
As for the IE 11 alignment - Would you mind opening an additional issue for this one so that we can track it? Thanks!

Sure, Done #56


Feature request: Support attribute srcset

In my opinion attribute srcset would be great to be supported. I tried to provide a complete pull request introducing srcset-checks for elements img and source, however, I wasn't able to integrate this functionality to your tests. To be specific, tests checkLinksQueryHashes and localContentQueryHashesLinksToIgnore failed on files containing the new attribute tags.
Package srcset already parses the attributes content reliable, which should be a good choice to build upon.

This was on my TODO list, but I didn't want to have to deal with parsing all the different types of syntax. I love your idea of using a package for this and will look into it soon. Thank you!


[STORM-1051] Fix for flushMessagse NPE

STORM-763 introduced a situation in which flushMessages could receive batch = null, in which case, an NPE is thrown because we weren't validating != null

@schonfeld
Could you post a new PR which points to master, too?
Thanks in advance!

Closing in favor of PR #745


Fatal error when using Zend-service Twitter

Fatal error: Call to a member function connect() on a non-object in zendframework/zend-http/Zend/Http/Client.php on line 1358

This issue has been moved from the zendframework repository as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html
New issue can be found at: https://github.com/zendframework/zend-http/issues/80


Customize "error: ..." message (e.g. argument not recognized)

 Issue by Ghosty141
Sunday Mar 08, 2020 at 17:14 GMT
Originally opened as https://github.com/clap-rs/clap/issues/1734

Describe your use case
Since I'm using clap to create an interactive cl program, I'd like to skip whole "USAGE: " section since it shows very few useful information and takes up quite a lot of screen-space:
USAGE:
    myapp [SUBCOMMAND]

For more information try --help

Describe the solution you'd like
It would be great to customize the error message. For example, select which parts should be shown (error, usage, additional information).
Alternatives, if applicable
For now just cutting off everything after "USAGE:" kinda works but it's not really pretty.
With some mentoring I'd maybe try to implement it myself if this is considered a good addition.

 Comment by CreepySkeleton
Sunday Mar 08, 2020 at 18:26 GMT

It would be great to customize the error message.

I believe clap::App::template is exactly what you're looking for, isn't it?

 Comment by Ghosty141
Sunday Mar 08, 2020 at 19:49 GMT

Maybe it is and I don't exactly know who to use it but when I use .template("...") it does not change the error message that gets displayed when entering an invalid argument/subcommand.


Auto tagging : feature request

Feature request
If we give images and tags for training (without bounding boxes) to TuriCreate, it should be able to generate tags for unseen but similar images.

@johnyquest7 - I'm not sure I understand your feature request. Could you explain more or give an example? Also is this for the image classification toolkit?

@TobyRoseman
This is for image classification toolkit.
Lets say I have lots of satellite images and each image is tagged with land features and weather features like - farm land, forest, urban area, river, cloudy day, sunny day etc. Once we train a model using these images and tags, the model should be able to auto generate these tags for an unseen image.

@johnyquest7 - thanks for the description. This sounds like multilabel classification, which we are already tracking in #147.


The todo-databinding branch contains findViewById calls

While most of these calls are for layout files that don't actually do any data binding in itself (i.e., for layout files that do not have the <layout> tag), it might be a good idea to demonstrate best practices of the data binding library and eliminate all findViewById calls altogether.
If the maintainers agree with this, I could submit a PR.

Thanks for the issue. Are you suggesting creating layout tags in the activities' layouts? Like in https://github.com/googlesamples/android-architecture/blob/todo-databinding/todoapp/app/src/main/res/layout/taskdetail_act.xml
It seems overkill to do that to get a ref to the fab.

Yes that is what I was suggesting, at least in layouts where there is more than one view to find (maybe not for the fab case). I think eliminating findViewById calls is the "Level 0" of the data binding library.
Additionally there's looking at code like this where you already have an instance of ViewDataBinding. You don't need the findViewById calls there. mTitle and mDescription can be replaced with mViewDataBinding.addTaskTitle and mViewDataBinding.addTaskDescription respectively.

I don't see the data binding lib as a way to eliminate findViewById everywhere (there's butterknife for that), so I would only use it for actual data binding.
I agree on the second part. A PR would be very much appreciated!


Bluetooth: Controller: Properly terminate per adv HCI on failed rx

If one of AUX_CHAIN_IND is not received properly we need to send an
extra report over HCI to indicate that data are incomplete.
Signed-off-by: Andrzej Kaczmarek<EMAIL_ADDRESS>

extracted from https://github.com/zephyrproject-rtos/zephyr/pull/38146

@andrzej-kaczmarek please rebase and resolve conflict


HADOOP-17769. Upgrade JUnit to 4.13.2. fixes TestBlockRecovery

HADOOP-17769 Upgrade JUnit to 4.13.2
JUnit 4.13.1 has a bug that is reported in Junit issue-1652 Timeout ThreadGroups should not be destroyed.
The bug has been fixed in Junit-4.13.2

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 57s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
35m  9s

trunk passed

+1 :green_heart:
compile
0m 17s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
0m 17s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
mvnsite
0m 22s

trunk passed

+1 :green_heart:
javadoc
0m 19s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 18s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
shadedclient
52m 55s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 13s

the patch passed

+1 :green_heart:
compile
0m 12s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javac
0m 12s

the patch passed

+1 :green_heart:
compile
0m 11s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
0m 11s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 14s

the patch passed

+1 :green_heart:
xml
0m  2s

The patch has no ill-formed XML file.

+1 :green_heart:
javadoc
0m 14s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 13s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
shadedclient
17m 28s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
0m 14s

hadoop-project in the patch passed.

+1 :green_heart:
asflicense
0m 29s

The patch does not generate ASF License warnings.

74m 41s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3130/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3130

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell xml

uname
Linux 35beb5b44425 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / a03d4274161f2da3b810a978bf45c6df2c921e61

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3130/1/testReport/

Max. process+thread count
523 (vs. ulimit of 5500)

modules
C: hadoop-project U: hadoop-project

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3130/1/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

@amahussein can you update this file, as suggested on 2.10 PR
https://github.com/apache/hadoop/blob/10b79a26fe0677b266acf237e8458e93743424a6/LICENSE-binary#L505

@amahussein can you update this file, as suggested on 2.10 PR
https://github.com/apache/hadoop/blob/10b79a26fe0677b266acf237e8458e93743424a6/LICENSE-binary#L505

Thanks @ayushtkn and @jojochuang for the feedback.
I pushed another commit fixing the LICENSE-binary. Apparently, I missed it because I was searching for 4.13.1 while it was 4.12 in the LICENSE-binary .

Thanx for the update. Will push once the build completes.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 52s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
shelldocs
0m  1s

Shelldocs was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 14s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
22m 55s

trunk passed

+1 :green_heart:
compile
25m 15s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
22m 45s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
mvnsite
30m 48s

trunk passed

+1 :green_heart:
javadoc
10m 43s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
9m 46s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
shadedclient
38m 48s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 30s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
27m  5s

the patch passed

+1 :green_heart:
compile
29m  0s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javac
29m  0s

the patch passed

+1 :green_heart:
compile
25m 20s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
25m 20s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
27m 13s

the patch passed

+1 :green_heart:
shellcheck
0m  0s

No new issues.

+1 :green_heart:
xml
0m  2s

The patch has no ill-formed XML file.

+1 :green_heart:
javadoc
9m 26s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
7m 50s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
shadedclient
34m  1s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
984m 40s
/patch-unit-root.txt
root in the patch passed.

+1 :green_heart:
asflicense
1m 28s

The patch does not generate ASF License warnings.

1287m 57s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.namenode.TestDecommissioningStatus

hadoop.hdfs.server.balancer.TestBalancerWithHANameNodes

hadoop.hdfs.server.namenode.TestDecommissioningStatusWithBackoffMonitor

hadoop.hdfs.TestDFSShell

hadoop.hdfs.server.namenode.ha.TestEditLogTailer

hadoop.hdfs.server.namenode.ha.TestBootstrapStandby

hadoop.tools.dynamometer.TestDynamometerInfra

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3130/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3130

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell xml shellcheck shelldocs

uname
Linux 8f70c3d654bb 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 1f2ace03f64c00c64e2a306564f25e1406702898

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3130/2/testReport/

Max. process+thread count
2979 (vs. ulimit of 5500)

modules
C: hadoop-project . U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3130/2/console

versions
git=2.25.1 maven=3.6.3 shellcheck=0.7.0

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Merged to trunk. @amahussein the cherry-pick fails for 3.3 and 3.2, Would you be able to provide patch/PR for them?

Merged to trunk. @amahussein the cherry-pick fails for 3.3 and 3.2, Would you be able to provide patch/PR for them?

Thanks @ayushtkn for merging the PR!
Sure, I will backport into 3.3 and 3.2


Bugsnag: turn off trackInlineScripts

Description
See this document for rationale and more information.

To whoever might know: it looks like we're using @bugsnag/js here, and this option is only available in @bugsnag/browser. Are we adding plugins to Bugsnag manually?

I'm tempted to say this a breaking change that users should opt into rather than getting for free - we're changing default configuration.

I'm tempted to say this a breaking change that users should opt into rather than getting for free - we're changing default configuration.


Problem setting up the MacOS environment, the serialport gem is not maintained.

Problem: Unable to use bundle to install Ruby gem ruby-serialport
System info: Apple M1 with macOS Sonoma
ruby-serialport issue: [posix_serialport_impl.c: error: incompatible pointer to integer](posix_serialport_impl.c: error: incompatible pointer to integer)
The serialport gem is no longer maintained. I can pull the repo and apply the patch mentioned in the issue, but due to my lack of knowledge of Ruby/gem, it's not perfect, for some reason  I need to add RUBYLIB=ruby-serialport/lib to specify the serialport lib path. Maybe someone good at gem has a better solution.I wonder if this can be done with rust.

Any better solutions ? I also met this issue.

A better solution would be to use a maintained serialport gem to replace the current one. If you just want to run the tutorials, I think manual patch ruby-serialport and install will do, but if you are not familiar with ruby, you will need to spend some time figuring out how to install the gem locally.
links:

https://github.com/hparra/ruby-serialport/issues/74

yes, it's hard to know how to install gem locally.

bundle config --local path '.vendor/bundle'
run gem install --local pkg/serialport-1.3.2.gem --install-dir ../ruby-serialport-output -V intall gem to ruby-serialport-output
in Gemfile using gem 'serialport', path: "ruby-serialport-output" to specify path
bundle install
bundle list

❯ bundle list
Gems included by the bundle:
  * bindata (2.5.0)
  * bundler (1.17.2)
  * colorize (1.1.0)
  * elftools (1.2.0)
  * ruby-progressbar (1.13.0)
  * serialport (1.3.2)

Somebody give us a pull request for the book? This is something I know nothing about, but it sounds like @livexia has it figured out.

I don't think this solution is good or 100% correct, even though it works on my system. I came across this problem because I am helping someone run this tutorial. It fails on my own system, but had no problem on that person's system. Before this I know nothing about ruby, I think this is a good temporary solution, maybe someone with better knowledge should give a pr.

Thank you for reporting this issue. The author has been on vacation for over a year now.
This seems to be a Ruby issue, I will take some time to check, but I'm not sure if I can solve it, but it's worth fixing it.


WIP: add optional qlogging to quiche

This is still WIP but ready for review. I need to add tests for the new capability but will await design feedback.
The PR contains 2 commits, the older one uses a buffered serialization approach, the newer one replaces this with a streaming serialized approach. I left these so that we could compare the two, but we'd definitely squash this down before any merge.
To test this out, you can run quiche-client or quiche-server with the QLOGDIR environment pointing at a folder. The resulting trace will be written to a file such as client-{scid}.qlog which can then be imported to the visualization tool https://qvis.edm.uhasselt.be.

Okay... I've addressed a lot of the feedback comments, thanks!
This is now ready for another review. In the meantime I'll start on updating the qlog documentation ready to publish the new v0.2 of the crate before this PR lands.

Docs updated and comments addressed, PTAL

@LPardue looks like this needs to be rebased on master as there is a conflict. While you are at it, can you also submit the new qlog crate to crates.io and squash the commits in the PR? So we can finally merge this.


Compiling Problem (HELP ME)

Describe your problem here.
i tried to compile typing lime test windows after 2 seconds this appeared "source/psychlua/CallbackHandler.hx:55: characters 18-33 : Class<llua.Lua_helper> has no field sendErrorsToLua
export/release/windows/haxe/ApplicationMain.hx:153: characters 5-36 : ... Called from macro here
C:/HaxeToolkit/haxe/lib/flxanimate/3,0,4/flxanimate/frames/FlxAnimateFrames.hx:142: characters 21-27 : Field concat should be declared with 'override' since it is inherited from superclass flixel.graphics.frames.FlxAtlasFrames
C:/HaxeToolkit/haxe/lib/flxanimate/3,0,4/flxanimate/frames/FlxAnimateFrames.hx:142: characters 21-27 : Field concat overrides parent class with different or incomplete type
C:/HaxeToolkit/haxe/lib/flixel/5,6,2/flixel/graphics/frames/FlxAtlasFrames.hx:454: characters 18-24 : ... Base field is defined here
C:/HaxeToolkit/haxe/lib/flxanimate/3,0,4/flxanimate/frames/FlxAnimateFrames.hx:142: characters 21-27 : ... Different number of function arguments
source/backend/Paths.hx:605: characters 7-18 : flxanimate.FlxAnimate has no field loadAtlasEx (Suggestion: loadAtlas)
source/objects/Character.hx:304: characters 87-101 : flxanimate.animate.FlxAnim has no field lastPlayedAnim
source/objects/Character.hx:335: characters 20-26 : flxanimate.animate.FlxAnim has no field resume" Help. How do i Fix this?
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Yes I did all instances of TeaCall in the file source psychlua Hscript.hx  to Tea.

assuming you're using 0.7.2 or higher, you should look at this
https://github.com/ShadowMario/FNF-PsychEngine/wiki/Libraries-versions

Update your Libs and try using haxe 4.3.2 as recommended
Also you will want to set SScript to 7.7.0 since later versions break the Hscript and then TeaCall will work make sure to revert it back to TeaCall.
You can also use the BAT included.

Use the setup-windows.bat for the libraries

after I did as you said i compieled and showed me this:  ------------------------SScript 7.7.0 Macro------------------------
Special Thanks:

CrowPlexus

Checking version...
There was an error getting the latest version info, you may not have internet connection.
Continuing...
source/psychlua/CallbackHandler.hx:55: characters 18-33 : Class<llua.Lua_helper> has no field sendErrorsToLua
export/release/windows/haxe/ApplicationMain.hx:153: characters 5-36 : ... Called from macro here
C:/HaxeToolkit/haxe/lib/flixel-ui/2,5,0/flixel/addons/ui/FlxUICursor.hx:564: characters 9-11 : Warning : Potential typo detected (expected similar values are flixel.addons.ui.SortMethod.ID);   PLEASE HELP ME

you might want to completely reinstall haxe it fixed it for me

I also made a better setup-windows.bat
setup-windows.bat

you might want to completely reinstall haxe too. It fixed it for me
C:/HaxeToolkit/haxe/lib/flixel-ui/2,5,0/flixel/addons/ui/FlxUICursor.hx:564: characters 9-11 : Warning : Potential typo detected (expected similar values are flixel.addons.ui.SortMethod.ID); Ignore this error its nothing.
source/psychlua/CallbackHandler.hx:55: characters 18-33 : Class<llua.Lua_helper> has no field sendErrorsToLua
For this you can delete this part

export/release/windows/haxe/ApplicationMain.hx:153: characters 5-36 : ... Called from macro here
For this delete settings.cocoa, its the SScript settings. Its located in this folder

after users its your username for your computer.
When it asks you to enable macros for SScript press (n) and press and enter and the others is what you want.

Would recommend following this

or you can just
haxelib remove linc_luajit
haxelib git linc_luajit https://github.com/superpowers04/linc_luajit

THX SO MUCH GUYS I THINK ITS WORKING!!!

THX SO MUCH GUYS I THINK ITS WORKING!!!

no problem just close your issue


Simplify upgrading agent

Currently agent uses self signed key pair for ssl, this made upgrading rather difficult. Default installation should be as simple as it possible

disable ssl in agent at 6fe470751776520ed75b9d4146edc54a80323612


How to use a customized font in IOS?

I switched from SwiftUI's TextEditor to this CodeEditor. it really amazed me.
However, I found I cannot specify the font and font size as I did in the TextEditor. The code is as follow,
CodeEditor(source: $formulaInput, language: .python, theme: .default,
                       autoPairs: ["(" : ")", "{" : "}", "[" : "]"])
                .font(.custom("SF Mono Bold", size: fontSize))

Can you teach me how to use a customized font?
Thank you so much

The fontSize is just a regular binding of CodeEditor, e.g.
CodeEditor(
  source: $formulaInput, language: .python, theme: .default,
  fontSize: .constant(20),
  autoPairs: ["(" : ")", "{" : "}", "[" : "]"]
)

should do. Also check: https://github.com/ZeeZide/CodeEditor/blob/180bde07b44dea839b32873bd8586ba146fa9106/Sources/CodeEditor/CodeEditor.swift#L93
The font itself is part of the Highlightr theme: https://github.com/ZeeZide/CodeEditor/blob/180bde07b44dea839b32873bd8586ba146fa9106/Sources/CodeEditor/UXCodeTextView.swift#L57
I don't remember how you would change that as it seems to be accessed by name. Might require a custom Highlightr.


Sloppy handling of OS requirements for building test suite

The get_source.sh script assumes Ubuntu (or maybe Debian too), but this is not called out anywhere in the document.
README.md should plainly state which OS distro configurations have been tested and which packages (and known versions) need to be installed. Shell scripts should not run system commands prior to validating a supported distro.

will update shell and readme

fixed in d497d99

https://github.com/intel/rv-brs-test-suite/commit/d497d995f45dfac197ea6001c117b4f088503eab has other issues.
It's unnecessarily restrictive. For example, my WSL envo is Ubuntu 20.04.5 LTS and everything builds fine. So no point to restrict to 22.04. Worse, what happens when 24.04 or w/e comes out? If you want to enforce a version, it only makes sense to enforce the lowest tested version.
Here's what I would do:

Only run "apt" on Ubuntu and Debian
You can test for lowest supported version, BUT don't make failed checks an error. You could make this a warning and request user input, or just log a warning and continue... or add an option to make OS checks (including available space and w/e) ignorable.

have tested on 20.04 docker based image, will try on 24.04 and update script.

Will test other version and update script.


✨ Algorithm for finding valid physical parameters for a given charge distribution.

Description
This algorithm provides a highly efficient method for determining the physical parameters that make a given charge distribution physically valid. It also provides the excited state number for a given charge distribution. Thus, for a given physical parameter that makes the cds physically valid, it also provides information about whether the cds is the ground state or the n-th excited state.
This algorithm can be useful for extracting current physical parameters from AFM measurements.
Checklist:

[x] The pull request only contains commits that are related to it.
[x] I have added appropriate tests and documentation.
[ ] I have created/adjusted the Python bindings for any new or updated functionality.
[ ] I have made sure that all CI jobs on GitHub pass.
[ ] The pull request introduces no new warnings and follows the project's style guidelines.

@marcelwa I would love to get your feedback here after some architectural changes. After that I will add the Python bindings.


Seamless support for reading data with meta line of separator declaration used by MS Excel

This PR adds seamless support for reading data with meta line of separator declaration used by MS Excel.
The original discussion.
Sample data:
sep=,
id,first_name,last_name,username
11,"Rob","Pike",rob
2,Ken,Thompson,ken
4,"Robert","Griesemer","gri"
1,"Robert","Thompson","abc"
NA,"Robert","Abel","123

The code looks good, however my intent with this is to provide compatibility with encoding/CSV so I don't think I am going to pull this in.
Maybe you could make a package that wraps this library that would do what you want?  Would apply for reading and writing.

That's Ok, thank you anyway. i'll use my fork version.


[Snyk] Security upgrade bootstrap from 4.6.0 to 5.0.0

Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

dashboard/package.json
dashboard/package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Cross-site Scripting (XSS) SNYK-JS-BOOTSTRAP-7444580
  688

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Scripting (XSS)

superseded by #397


Can't share full screen with meeting at Fedora

In Fedora, user can't share entire screen at meeting, user can only share screen of special app.
This is bug from chromium https://bugs.chromium.org/p/chromium/issues/detail?id=682122

fixed with #2


Cleanup state of Main Activity

The second connection to Sonar Activity executed without reconnect.

done in #12


Rename ra_fmt -> fmt

This needs a better name than just fmt, we already have fmt. I'll think about a better name today!

Turned out that this crate could be removed entirely: https://github.com/rust-analyzer/rust-analyzer/pull/5743

Was just looking at that PR. 👍


Readium fails when using readium index page as source inside iframe on IE

I have an html page with iframe that has source to readium viewer link  as follows
 <iframe width="100%" height="500px"  id="ifjdfnkjvnjjbk"  src="http://example.com/cloud-reader-lite/index.html?epub=epub_content/test.epub"  runat="server"  ></iframe>
This does open on google chrome but it fails on IE. It shows the book title on top but fails to open the file the loading circle keeps on running and nothing is displayed

How could this be solved?

@mahag Are there any errors in the console?

Yes there is an access denied error even though I have added document.domain script as suggested when I looked about this issue on some forums.

I did try few things in order to skip this error. I added correct domain value in my html page and I was able to skip the error at that point of code but I received the same error on another location inside readium JavaScript file as shown in img

Any ideas?