text:
chore: bump nightwatch, add retries

#minor

Pull Request Test Coverage Report for Build 771444520

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.003%) to 84.465%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

Totals

Change from base Build 767758935:
0.003%

Covered Lines:
18929

Relevant Lines:
21331

💛  - Coveralls

authors: coveralls joshgummersall


text:
New array as model not working in the playground

Type relation PR seems to have broken arrays in the playground
Playground Link

nvm was an issue with web caching, works fine

source: gharchive/issue

repo: microsoft/cadl

license: MIT

license_type: permissive

license_source: github-api


text:
fixed titlebar

What?
When active, Calc title bar is impossible to read in Night Sky theme
Updated:

Cal title bar is visible in Night Sky theme
Remove the foreground and clean up the code

Any screenies?

source: gharchive/pull-request

authors: Poopooracoocoo huytheduong

repo: microsoft/calculator


text:
591/Set custom location file

Issue Number: (591)
Summary
Be able to set custom location data files.
Changes

Add support to set custom location files as part of the configuration of the application.
Kept one common directory structure for the location logic.

Checklist

[x] Local Tests Passing?
[x] CICD and Pipeline Tests Passing?
[x] Added any new Tests?
[ ] Documentation Updates Made?

Are there API Changes?
No
Is this a breaking change?
No
Anything else?
No

Codecov Report

Merging #149 (44da1e4) into dev (bb79dcb) will increase coverage by 0.32%.
The diff coverage is 84.07%.

@@            Coverage Diff             @@
##              dev     #149      +/-   ##
==========================================
+ Coverage   85.73%   86.06%   +0.32%
==========================================
  Files          52       54       +2
  Lines        1711     1758      +47
  Branches      194      199       +5
==========================================
+ Hits         1467     1513      +46
- Misses        191      192       +1
  Partials       53       53

Impacted Files
Coverage Δ

src/CarbonAware/src/Model/Location.cs
93.75% <ø> (-0.37%)
:arrow_down:

.../CarbonAware.LocationSources/src/LocationSource.cs
81.48% <81.48%> (ø)

...ionSources/src/Configuration/LocationDataSource.cs
88.46% <88.46%> (ø)

...e/src/Configuration/ServiceCollectionExtensions.cs
90.00% <100.00%> (+6.66%)
:arrow_up:

.../Configuration/LocationDataSourcesConfiguration.cs
100.00% <100.00%> (ø)

Wondering if we should add some documentation in Swagger about prefixing the location?

Wondering if we should add some documentation in Swagger about prefixing the location?

Good question. I was hesitant on adding another 'example' on the swagger doc where location is used, since it depends whether there is a configuration set or not. Document GettingStarted.md was updated with how to configure this functionality and how to use it.

authors: codecov-commenter juzuluag pritipath

repo: microsoft/carbon-aware-sdk


text:
Gem::ConflictError with new released cocoapods-downloader-2.0

Hi,
since last week everything was working fine, but unfortunately on 2023-10-26 cocoapods-downloader was released in version 2.0, see https://rubygems.org/gems/cocoapods-downloader/versions/2.0
Since then I get this error when running pod install:
Gem::ConflictError - Unable to activate cocoapods-azure-universal-packages-0.1.0, because cocoapods-downloader-2.0 conflicts with cocoapods-downloader (~> 1.0)
I guess in https://github.com/microsoft/cocoapods-azure-universal-packages/blob/main/cocoapods-azure-universal-packages.gemspec the runtime version dependency to cocoapods-downloader should be updated to 2.0 to stay compatible!?
Please can you try to have a look and fix it?
Thanks
Ben

Note: a workaround could be to run
gem install "cocoapods-downloader" -v 1.0
before building an app.

I too face this issue. Did gem install cocoapods-downloader -v 1.6.3 in appcenter-pre-build.sh but still facing this issue.

@zflorin Unfortunately I'm no longer at Microsoft so I don't have write permissions to this repo anymore. I'll reach out to some of my ex-colleagues to see if anyone can help!

@zflorin Unfortunately I'm no longer at Microsoft so I don't have write permissions to this repo anymore. I'll reach out to some of my ex-colleagues to see if anyone can help!

That would be very much appreciated so thanks a bunch @Frugghi 💪🏻

authors: BenjaminZaiser Frugghi Manojkumar-Subramaniam zflorin


text:
Add contents on BDD in automated-testing/unit-testing

Adding contents around BDD with "Given, When, Then"(GWT) format in addition to TDD with AAA format.
Ref:
https://github.com/microsoft/code-with-engineering-playbook/tree/master/automated-testing/unit-testing#test-driven-development

@akirakakar if you plan on working on the issue, please assign it to yourself, or if you don't have permissions, comment on the issue, so we know it is being worked on

@akirakakar if you plan on working on the issue, please assign it to yourself, or if you don't have permissions, comment on the issue, so we know it is being worked on

@TessFerrandez I haven't started to work on yet and don't want to block anybody to pick this up if they want.
I will ask you to assign me (as I don't seem to have permission to do so) if I start working on it before somebody picks up.

@TessFerrandez I haven't started to work on yet and don't want to block anybody to pick this up if they want.
I will ask you to assign me (as I don't seem to have permission to do so) if I start working on it before somebody picks up.

@akirakakar - this issue has been stale for some time, I will go ahead and close and you can reopen later if you find time to add the content

authors: TessFerrandez akirakakar


text:
Customize ADO - under code reviews - should be Azure DevOps

Replace all different abbreviations here to Azure DevOps for consistency
https://github.com/microsoft/code-with-engineering-playbook/blob/master/code-reviews/process-guidance/customize-ado.md

This is being fixed in #815

authors: TessFerrandez tompaana


text:
Docker compose lint errors for valid 'reset' tag

With this VS Code extension enabled, I get the following linting errors in a docker compose override file:

If I disable the extension, the errors go away:

The reset tag is fairly new to docker compose, in the last year or so. I believe it needs to be added to this extension.

Resolving as a duplicate of #145

repo: microsoft/compose-language-service


text:
NuGet package: CppWinRTPrecompiledHeader should be "." when not using PCH

In Microsoft.Windows.CppWinRT.targets, in this part of the code:
        <PropertyGroup Condition="('$(CppWinRTOverrideSDKReferences)' == 'true') or ('$(TargetPlatformVersion)' > '10.0.17708.0')">
            <CppWinRTUsePrefixes Condition="'$(CppWinRTUsePrefixes)' == ''">true</CppWinRTUsePrefixes>
            <CppWinRTPrecompiledHeader Condition="'$(CppWinRTPrecompiledHeader)' == ''">$(_PCH)</CppWinRTPrecompiledHeader>
        </PropertyGroup>

Should likely be something like this:
            <CppWinRTPrecompiledHeader Condition="'$(CppWinRTPrecompiledHeader)' == '' and '$(UsePCH)' == 'true'">$(_PCH)</CppWinRTPrecompiledHeader>
            <CppWinRTPrecompiledHeader Condition="'$(CppWinRTPrecompiledHeader)' == '' and '$(UsePCH)' == 'false'">.</CppWinRTPrecompiledHeader>

(Otherwise, when the project has PCH disabled, the generated code fails to compile.)

@jlaanstra

        <CppWinRTPrecompiledHeader Condition="'$(CppWinRTPrecompiledHeader)' == '' and '$(UsePCH)' == 'true'">$(_PCH)</CppWinRTPrecompiledHeader>
        <CppWinRTPrecompiledHeader Condition="'$(CppWinRTPrecompiledHeader)' == '' and '$(UsePCH)' == 'false'">.</CppWinRTPrecompiledHeader>

Why the second line? Not sure why we would set it to a single dot?

Actually I can't find anything docs about a UsePCH flag? Where is that coming from?

On the command line, the . means that no precompiled header should be used since the default is pch.h.
https://github.com/microsoft/cppwinrt/blob/0be01db74002bcbb6c9bcc3fd6f0ef88961bf71b/cppwinrt/main.cpp#L132
https://github.com/microsoft/cppwinrt/blob/0be01db74002bcbb6c9bcc3fd6f0ef88961bf71b/cppwinrt/main.cpp#L138-L141

But I have no idea what UsePCH is. 😜

I think that was just pseudo-syntax for "checking whether this project has precompiled headers enabled or not".
Here's the syntax added to the project file when enabling them in VS 2019. (They're off by default, at least for some templates.)
<PrecompiledHeader>Use</PrecompiledHeader>

I believe the valid values are Use, Create, or NotUsing. (I'm not sure what Create should do in this case; haven't checked exactly what it means.)

"Create" means create the PCH whereas "Use" means depend on the previously created PCH. Only the "pch.cpp" would have "Create" while everything else (that depend on the PCH) would have "Use".

authors: davidmatson jlaanstra kennykerr

repo: microsoft/cppwinrt


text:
CRUD API Plugin Schema shows warning in VSCode

Related to https://github.com/garrytrinder/dev-proxy-vsc-ext/issues/1#issuecomment-1894453537

We should set additionalProperties to false on all schemas and include $schema property in the schema.

repo: microsoft/dev-proxy


text:
500 Internal server error when trying to run portability analysis and generate DGML

Trying to run analysis against a 4.7.2 winforms project checking for .NET 5 and trying to generate DGML. Unchecking the DGML option allows it to work. Unfortunately the project is closed source, so will not be able to provide repro in this instance.

Duplicate of #896

Yes, it's known issue. Close this issue and use #896  to track it.

authors: Lxiamail lxman rsabirov

repo: microsoft/dotnet-apiport


text:
Remove unused 'request' dependency

Maybe I'm missing something but I don't see why it was added in
https://github.com/microsoft/dtslint/commit/3c57139d7e2afa4600846a00123843e85e42083e

@sanders Friendly ping 👐
Not blocker in any sense but I would love to remove a bunch of implicit devDependencies from my packages.

@sandersn If possible can you please release it sooner?
Previously it was just my paranoia about number packages in node_modules but now I get this message on every npm install/yarn install:

warning dtslint > request@2.88.2: request has been deprecated, see https://github.com/request/request/issues/3142

Done, as version 2.0.6.

@sandersn Thank a lot 👍
One less warning and 39 fewer potential security vulnerabilities in my node_modules folder.

authors: IvanGoncharov sandersn

repo: microsoft/dtslint


text:
Possible to call Azure FHIR API using MSAL authentication process

I am using MSAL daemon server authentication mechanism to call FHIR API using example specified in below link,
https://github.com/Azure-Samples/active-directory-dotnetcore-daemon-v2
I tried with sample application, able to retrieve access token using MSAL library but its giving forbidden error while calling FHIR API from daemon process, Is it possible to access FHIR API using daemon application?

Its mentioned in MSAL sample application that we need application permission in order to access API, let me know if it is possible to provide application permission for FHIR API from app registration, currently application permission for Azure Healthcare API is disabled and not able to add as shown in below image,

I am assuming that by "daemon server authentication" you mean service to service authorization and so you are using "client credentials flow" (in Azure AD parlance).
It is not exactly clear from this whether you are getting a 401 or 403 error code, but it sounds like 403. So if that is the case, the reason is probably that the service principal that you are getting a token for does not have access to the FHIR API. Did you give that service principal access? And is this specifically related to this fhir-server-samples repo or is this a general question related?

@hansenms
Yeah, you are right about below points,

Error Code : 403, Forbidden
Using Client Credential Flow, sample flow as,

But I am trying to access FHIR API from daemon process, and authentication mechanism is MSAL latest library, I will check details of service principals....
MSAL team is looking for someone from Azure Healthcare API's, please have a look into below git hub issue,
https://github.com/Azure-Samples/active-directory-dotnetcore-daemon-v2/issues/89

If you are getting a 403, it means that the service principal (the app registration) does not have access to the FHIR API. You need to grant it access. How you do that depends on whether you are using Azure RBAC or Local RBAC.
You need to take a look at your token (e.g. with https://jtw.ms) and figure out what the oid claim is. That is the object id of the identity that you need to grant access. It is slightly confusing because there is both and app id and an object id, but the object id is what matters. If you are using local RBAC, you simply add that oid to the list of allowed object ids per instructions above. If you are using Azure RBAC, follow the instructions there.
@ginalee-dotcom is on the Healthcare APIs team and can assist.

Issue resolved by following steps for Azure RBAC, I need to assign FHIR roles to desired app to access FHIR API from daemon or web api, I assigned role to my user name instead of App...
https://docs.microsoft.com/en-us/azure/healthcare-apis/configure-azure-rbac

authors: RahulKate hansenms

repo: microsoft/fhir-server-samples


text:
Cannot run built in scripts...

Type: Bug
Could not find a GenAiScript issue
Requested script: /_builtin/system.diagrams.genai.mjs
Known scripts:
genaisrc/release-notes.genai.mjs
Environment
vscode: 1.96.0
extension: 1.84.3
Extension version: 1.84.3
VS Code version: Code 1.96.0 (Universal) (138f619c86f1199955d53b4166bef66ef252935c, 2024-12-11T02:29:09.626Z)
OS version: Darwin arm64 24.1.0
Modes:

System Info

Item
Value

CPUs
Apple M3 Max (16 x 2400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
2, 2, 2

Memory (System)
64.00GB (3.13GB free)

Process Argv
--crash-reporter-id f55fe1a8-55ed-4e7c-b69f-475658ec7184

Screen Reader
no

VM
0%

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805cf:30301675
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
pythonnoceb:30805159
pythonmypyd1:30879173
2e7ec940:31000449
pythontbext0:30879054
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
dvdeprecation:31068756
dwnewjupyter:31046869
nativerepl2:31139839
pythonrstrctxt:31112756
nativeloc2:31192216
cf971741:31144450
iacca1:31171482
notype1:31157159
5fd0e150:31155592
dwcopilot:31170013
stablechunks:31184530

do you remember the steps to repro this?

#930

This issue might have been fixed with v1.83.4

authors: msanvido pelikhan

repo: microsoft/genaiscript


text:
Fix for port conflict with docker daemon.

The libnetwork logic uses a dummy proxy to bind the ports for communication between container and host. WS 2019 Aug 2020 patch onwards, Windows started doing an early port reservation which conflicts with the aforementioned logic for overlay and host publish mode for the ports.
Adding ExternalPortReserved flag to NatPolicy for HNS API V1. This is a flag exposed for docker to avoid port reservation conflict with external port
HNS API V2 will use NatFlags to check and see if ExternalPortReserved is set
(cherry picked from commit b85f3fdc17dad534a2cebbc67e0c18f77fb0fca8)
Signed-off-by: Ameya Gawde agawde@mirantis.com
Verified the fix with private HostNetSvc.dll from Windows networking team and corresponding change on libnetwork to send down the flag ExternalPortReserved

I assume we'll need to get out a tag of hcsshim containing this and get it into moby?

@dcantah - Yes, we'll need a tag. Also, if this looks good, will put up a PR against hcsshim's moby branch for backport.

authors: ameyag dcantah

repo: microsoft/hcsshim


text:
Array support for AnyOf

Adds support for AnyOf arrays as described in
fixes #3786

@microsoft-github-policy-service agree company="Twoday"

Also make sure you accept the CLA (see the bot reply)

Thanks for making the changes!

authors: baywet twodayfbh

repo: microsoft/kiota


text:
🐛 Signature for Trajectory.on_env_feedback() Interface should include the arguments used by the Actor class

Description

The Trajectory.on_env_feedback() interface defined here
only includes self as an argument.
However, the Actor() class defined here requires that the on_env_feedback() method include additional arguments:
self.trajectory.on_env_feedback(
                event, state_by_agent, action_by_agent, reward if reward is not None else self.env.metrics
            )

When we define a custom Trajectory, we can overwrite this method (and we do), but the difference in signature produces pylint errors:
W0221:Parameters differ from overridden 'on_env_feedback' method

As far as I can tell, every use of on_env_feedback requires that same signature, so it should be represented in the interface.
Screenshots

To Reproduce
Steps to reproduce the behavior:

Write a custom

Expected Behavior

Environment

MARO version (e.g., v0.1.1a1):
MARO scenario (CIM, Citi Bike):
MARO component (Simulation, RL, Distributed Training):
Orchestration platform (GraSS on Azure, AKS on Azure):
How you installed MARO (pip, source):
OS (Linux, Windows, macOS):
Python version (3.6, 3.7):
Docker image (e.g., maro2020/maro:latest):
CPU/GPU:
Any other relevant information:

Additional Context

That is correct @jreynolds01. Thanks for pointing it out. FYI, Trajectory has been revamped to AbsEnvWrapper in the upcoming version, so things will appear quite different. Stay tuned.

repo: microsoft/maro


text:
RichEditBox dont match Theme

Describe the bug
If you load a text to a RichEditBox when the Theme is in Dark mode the text will be black.
Steps to reproduce the bug

Just use the App XAML Controls Gallery version 1.2.16.0 and go to the RichEditBox control.
Go to the third example where you can upload a text file.

Make sure you are on dark mode and upload the text.

Expected behavior
The text should be white.

I am not sure if this is actually a bug. Per default text is black, so on black background you don't see anything. Similarly, if you load a file with white text into a RichEditBox in light theme, you can't read the text either.

Yeah, I think this is by design.  With RichEdit, the text color is associated with the text itself so when you copy black text into a text box its supposed to be black. The background is also black though so you get this behavior.  We could have xaml controls gallery be responsive to this scenario but I'm not sure that is worth it. @stmoy

The bug reported here doesn't repro on other RichEditBoxes - just this specific one that attempts to emulate a text editor (and specifically sets the text color to black). I think it'd be reasonable to move this over to the XCG repo. @chingucoding did you add this sample? Maybe instead of setting the color to black, it could be set to a ThemeBrush instead?

There are two things here:

When loading a RTF file, the RichEditBox not only copies the text, but also the font colors, resulting in a rtf file with black text not being readable on dark theme.
There were a few changes made to the samples to deal with the lighttheme on focus, which are not up to date anymore. I am working on reverting those changes and will create a PR for this soon.

Regarding 1, I don't think there is much to do beside overriding the font color delivered to us through the rtf file, @stmoy do you think we should do that kind of changes when loading an rtf file?

For #1, I don't think we should do any sort of shenanigans to the copied text in the sample. I think the absolute easiest thing would be to change the background color back to white just for this sample.

For #1, I don't think we should do any sort of shenanigans to the copied text in the sample. I think the absolute easiest thing would be to change the background color back to white just for this sample.

I think you would still hit this issue when loading an rtf file with white text. Maybe that is a little less common though?

I think you would still hit this issue when loading an rtf file with white text. Maybe that is a little less common though?

True, and probably. This isn't mean to be a proper editor, just a simulacrum of one. It's ok if the edge cases don't work.

If we change the background, we also need to adjust the foreground, foregroundhover, foregroundfocused ... values. I am not sure if we want to go down that road, especially as it suggest that you should use the RichEditBox with changed background and foreground values.

With this change, can we close this issue? https://github.com/microsoft/Xaml-Controls-Gallery/pull/524

Closing per the above PR

authors: CesGan StephenLPeters chingucoding stmoy

repo: microsoft/microsoft-ui-xaml


text:
Sometimes blank microsoft login page comes while trying to login using AzureAuth

Describe the bug

Sometimes while login with Microsoft login page blank screen appear with no textbox and label to enter password.
Steps to reproduce the bug
try to login with Azure Auth -> Microsoft Login screen should appear -> Sometimes that login screen is blank with no textbox to enter password.
Expected behavior
Microsoft Login screen should always appear properly with textbox and label, so user can enter password properly.
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.1.5
Windows version
Windows 11 (22H2): Build 22621, Windows 10 (21H2): Build 19044
Additional context
No response

@AnzarN, I'm closing this issue as it's not related to WinUI - maybe followup with other related reports


text:
Improve UI of MUXControlsTestApp

Description

This PR aims to improve the visuals and code of the MUXControlsTestApp
Motivation and Context

Even though designed for tests, having devs use the MUXControlsTestApp should not be impossible or unpleasant due to unreadable text and other UI (and code) that is not optimal to use.
How Has This Been Tested?

Started app. Tests should not be affected by this.
Screenshots (if appropriate):

Additional comments:

NavigationView: "BlankPage" is unused. Should we remove it?
ButtonInteractionPage and SliderInteractionPage are (at least for me) not visible in the MUXControlsTestApp. Are they expected to not be viewable?
Pivot: The CommandBar on the test page does nothing, can it be removed?
ScrollBar: The test page seems to have nothing rendered visible.
ProgressBar: Since we now have the NumberBox, should we switch away from the "Set ... Value" buttons and instead use bindings?
Happy holidays!

Increase and Improve Spacing between control property controls;

Increase Margins on pages and container regions;

Proper separation between example controls and the property forms;

Consistent layout for all test pages (perhaps include template XAML for developers and testers;

Increase and Improve Spacing between control property controls;
Increase Margins on pages and container regions;

This is something I already did on some pages,but not on all. If you find any pages that need improvement, feel free to tell me, and I will take a look 😊

Proper separation between example controls and the property forms;

I agree, however I don't know how you would do that without having pages contain weird layouts e.g. the NavigationView pages, where we use the NavigationView as root of the page. Do you have any ideas for this?

Consistent layout for all test pages (perhaps include template XAML for developers and testers;

Unfortunately, there are rarely any layouts/patterns that are used more than once or twice. This is due the fact that different controls have different ways to test them. So having a consistent layout among all pages might be quite difficult. But if you have an idea feel free to comment them :)

Increase and Improve Spacing between control property controls;
Increase Margins on pages and container regions;

This is something I already did on some pages,but not on all. If you find any pages that need improvement, feel free to tell me, and I will take a look 😊

Proper separation between example controls and the property forms;

I agree, however I don't know how you would do that without having pages contain weird layouts e.g. the NavigationView pages, where we use the NavigationView as root of the page. Do you have any ideas for this?

Consistent layout for all test pages (perhaps include template XAML for developers and testers;

Unfortunately, there are rarely any layouts/patterns that are used more than once or twice. This is due the fact that different controls have different ways to test them. So having a consistent layout among all pages might be quite difficult. But if you have an idea feel free to comment them :)

Its not about making each page the same layout, but the same separation between tests, properties, and the control example itself standing out from each other.

So it becomes obvious what the example is that you should be looking at.
What properties you can adjust.
What actions/tests you can perform

/azp run

/azp run

Its not about making each page the same layout, but the same separation between tests, properties, and the control example itself standing out from each other.

Thats a good point. I will see if what pages that need improvements in that area.
@kaiguo Thanks for starting azure!

/azp run

@chingucoding, there's one more AnimatedVisualPlayerTests.AccessibilityTest failing, could you take a look 😊?

@chingucoding, there's one more AnimatedVisualPlayerTests.AccessibilityTest failing, could you take a look 😊?

Sure :)

@kaiguo , I have reverted my changes for the AnimatedVIsualPlayer. Visual Studio did not want to run the test for the AnimatedVisualPlayer page, so I couldn't investigate the test failing.

@kaiguo Kai Guo FTE , I have reverted my changes for the AnimatedVIsualPlayer. Visual Studio did not want to run the test for the AnimatedVisualPlayer page, so I couldn't investigate the test failing.

I looked at the log, seems like some PropertyChangedEventWaiter timed out, I'm not sure why adding a column would make it time out. Reverting sounds fine to me, I'll re-run the tests.

/azp run

It seemed like that the test that was fixed with #1797 was failing, so I rebased this branch onto master. Hopefully that helps.

Increase and Improve Spacing between control property controls;
Increase Margins on pages and container regions;

This is something I already did on some pages,but not on all. If you find any pages that need improvement, feel free to tell me, and I will take a look 😊

Proper separation between example controls and the property forms;

I agree, however I don't know how you would do that without having pages contain weird layouts e.g. the NavigationView pages, where we use the NavigationView as root of the page. Do you have any ideas for this?

Consistent layout for all test pages (perhaps include template XAML for developers and testers;

Unfortunately, there are rarely any layouts/patterns that are used more than once or twice. This is due the fact that different controls have different ways to test them. So having a consistent layout among all pages might be quite difficult. But if you have an idea feel free to comment them :)

Its not about making each page the same layout, but the same separation between tests, properties, and the control example itself standing out from each other.

So it becomes obvious what the example is that you should be looking at.
What properties you can adjust.
What actions/tests you can perform

I've actually been wishing/hoping for this for a long time.  I think the best example of a good test page whose layout could be the base line is the Scroller's Exercise Scroller API page which @RBrid wrote.  The teaching tip, exercise swipe control api's, and RadioButtons test pages were built with this in mind. Personally I would love if all the pages got updated to look like this.

I've actually been wishing/hoping for this for a long time. I think the best example of a good test page whose layout could be the base line is the Scroller's Exercise Scroller API page which @RBrid wrote. The teaching tip, exercise swipe control api's, and RadioButtons test pages were built with this in mind. Personally I would love if all the pages got updated to look like this.

Agreed, those pages are better to use. I will try to get all pages into this pattern/layout.
However, on some pages I am not sure what to do since they have a lot of controls e.g. the ItemsRepeater overview page. Should I just propose changes there, and you just look if it makes sense? Or would it be better to update the questions at the top of this thread?

ItemsRepeater pages are purely for trying things out. They are not used by interaction tests so you can freely update them. Much of the other pages have interaction tests driving their pages, so you want to make sure atleast the UIA tree does not get changed much or be willing to update the tests as well.

ItemsRepeater pages are purely for trying things out. They are not used by interaction tests so you can freely update them. Much of the other pages have interaction tests driving their pages, so you want to make sure atleast the UIA tree does not get changed much or be willing to update the tests as well.

I see, thank you for the clarification!

Increase and Improve Spacing between control property controls;
Increase Margins on pages and container regions;

This is something I already did on some pages,but not on all. If you find any pages that need improvement, feel free to tell me, and I will take a look 😊

Proper separation between example controls and the property forms;

I agree, however I don't know how you would do that without having pages contain weird layouts e.g. the NavigationView pages, where we use the NavigationView as root of the page. Do you have any ideas for this?

Consistent layout for all test pages (perhaps include template XAML for developers and testers;

Unfortunately, there are rarely any layouts/patterns that are used more than once or twice. This is due the fact that different controls have different ways to test them. So having a consistent layout among all pages might be quite difficult. But if you have an idea feel free to comment them :)

Its not about making each page the same layout, but the same separation between tests, properties, and the control example itself standing out from each other.

So it becomes obvious what the example is that you should be looking at.
What properties you can adjust.
What actions/tests you can perform

Agreed, this is definitely a huge improvement. Thanks for doing this :)

/azp run

Looks like ProgressBar and TextCommandBarFlyoutTests are/were being reported as failing. Was this related to the AutoSuggestBoxTest?
What is the "Work Item" indicated in the log?
Link to Pipeline build:
https://dev.azure.com/ms/microsoft-ui-xaml/_build/results?buildId=56834&view=results

Looks like ProgressBar and TextCommandBarFlyoutTests are/were being reported as failing. Was this related to the AutoSuggestBoxTest?
What is the "Work Item" indicated in the log?
Link to Pipeline build:
https://dev.azure.com/ms/microsoft-ui-xaml/_build/results?buildId=56834&view=results

I think the work item here refers to the helix batch of work that the failing test was a part of. @kmahone is that right?

/azp run

@chingucoding These test failures look like real issue to me, will you be able to take a look?

@chingucoding These test failures look like real issue to me, will you be able to take a look?

Since the tests are not failing on my machine for some reason, I mostly reverted the pages in question as they are usable and having the tests be reliable is most important.

/azp run

/azp run

/azp run

Since the last CI run was successful, I think it is the best to merge this PR now, as a lot of pages have been improved. I have merged the latest commits from master in this branch to resolve potential merge problems.
If the CI build started by @ranjeshj finishes and is successful, I think it is best to merge this PR and make a separate PR for additional improvements.
Regarding latter, should I create an issue for this to collect items not covered by this PR?

Since the last CI run was successful, I think it is the best to merge this PR now, as a lot of pages have been improved. I have merged the latest commits from master in this branch to resolve potential merge problems.
If the CI build started by @ranjeshj finishes and is successful, I think it is best to merge this PR and make a separate PR for additional improvements.
Regarding latter, should I create an issue for this to collect items not covered by this PR?

Sure. We please do a follow up PR if it is getting too painful to merge.

Sure. Please do a follow up PR if it is getting too painful to merge.

If this is something, the WinUI team think is worth investing more time, sure.
Currently I do not have too much time to work further on this, so instead of having to continuously merge master into this PR, it is easier to merge this.

authors: StephenLPeters chingucoding kaiguo karenbtlai mdtauk ranjeshj


text:
Buffering IModelContentChange

I am looking for code to reduce several IModelContentChange to one if someone types several characters. I cannot find something like that in VS code repo. Does something like that exist?

I don't think we have code anywhere to "compact" changes like that.

authors: akosyakov alexdima

repo: microsoft/monaco-editor


text:
using auth = windows generates error: Unable to update the password...

Describe the issue
I'm logged into windows using my domain account.  Attempting to create a container with $auth = 'windows' and the the same windows domain account results in an error "Unable to update the password. The value provided for the new password does not meet the length, complexity, or history requirements of the domain."
I've read the other logged issues regarding the use of auth = windows and both of those were solved by going to auth = userpassword.  When I do that, the container is created successfully.  My issue is that I can't create a container using auth = windows and if my understanding is correct the password complexity validation comes from the domain.  Yet there must be a mismatch somewhere because I'm successfully logged into windows with the same account I'm trying to use for the container.
Scripts used to create container and cause the issue
$containerName = 'bcserver'
$credential = Get-Credential -Message 'Using Windows authentication. Please enter your Windows credentials for the host computer.'
$auth = 'Windows'
$artifactUrl = Get-BcArtifactUrl -type 'Sandbox' -country 'au' -select 'Latest'
New-BcContainer `
    -accept_eula `
    -containerName $containerName `
    -credential $credential `
    -auth $auth `
    -artifactUrl $artifactUrl `
    -imageName 'testimage' `
    -updateHosts

Full output of scripts
$containerName = 'bcserver'
$credential = Get-Credential -Message 'Using Windows authentication. Please enter your Windows credentials for the host computer.'
$auth = 'Windows'
$artifactUrl = Get-BcArtifactUrl -type 'Sandbox' -country 'au' -select 'Latest'
New-BcContainer `
    -accept_eula `
    -containerName $containerName `
    -credential $credential `
    -auth $auth `
    -artifactUrl $artifactUrl `
    -imageName 'testimage' `
    -updateHosts
BcContainerHelper is version 3.0.4-preview563
BcContainerHelper is running as administrator
Host is Microsoft Windows 10 Enterprise - 21H2
Docker Client Version is 20.10.14
Docker Server Version is 20.10.14
Removing container bcserver
Removing bcserver from container hosts file
Removing bcserver-* from container hosts file
Removing d:\containerhelper\Extensions\bcserver
Downloading application artifact /sandbox/20.0.37253.38528/au
Downloading C:\Users\<my domain user here>\AppData\Local\Temp\64d3ac7a-59db-41c0-95b9-a21b5e621342.zip
Unpacking application artifact to tmp folder using 7zip
Downloading platform artifact /sandbox/20.0.37253.38528/platform
Downloading C:\Users\<my domain user here>\AppData\Local\Temp\e8a70c6d-fddf-45a4-8d0e-86c844410b4f.zip
Unpacking platform artifact to tmp folder using 7zip
Downloading Prerequisite Components
Downloading d:\Artifacts.cache\sandbox\20.0.37253.38528\platform\Prerequisite Components\Open XML SDK 2.5 for Microsoft Office\OpenXMLSDKv25.msi
Downloading d:\Artifacts.cache\sandbox\20.0.37253.38528\platform\Prerequisite Components\IIS URL Rewrite Module\rewrite_2.0_rtw_x64.msi
Downloading d:\Artifacts.cache\sandbox\20.0.37253.38528\platform\Prerequisite Components\DotNetCore\DotNetCore.1.0.4_1.1.1-WindowsHosting.exe
Fetching all docker images
Fetching all docker volumes
ArtifactUrl and ImageName specified
INFO: Windows 10 21H1/21H2 images are not yet available, using 2004 as these are found to work better than 20H2 on 21H1/21H2
WARNING: Unable to find matching generic image for your host OS. Using mcr.microsoft.com/businesscentral:10.0.19041.1415
Image testimage:sandbox-20.0.37253.38528-au-mt already exists
Using image testimage:sandbox-20.0.37253.38528-au-mt
Creating Container bcserver
Style: sandbox
Multitenant: Yes
Version: 20.0.37253.38528
Platform: 20.0.37114.38311
Generic Tag: 1.0.2.1
Container OS Version: 10.0.19041.1415 (2004)
Host OS Version: 10.0.19044.1586 (21H2)
WARNING: Host OS is 21H1 and Container OS is 2004, defaulting to process isolation. If you experience problems, add -isolation hyperv.
Using process isolation
Using locale en-AU
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Files in d:\containerhelper\Extensions\bcserver\my:
- AdditionalOutput.ps1
- MainLoop.ps1
- SetupVariables.ps1
- updatehosts.ps1
Creating container bcserver from image testimage:sandbox-20.0.37253.38528-au-mt
a764ef89ce5302a2abcae6ed1eeddec9bab8e63e88f4a4f94d4170f3621f25fb
Waiting for container bcserver to be ready
Initializing...
Setting host.docker.internal to 10.1.10.65 in container hosts file (copy from host hosts file)
Setting gateway.docker.internal to 10.1.10.65 in container hosts file (copy from host hosts file)
Setting kubernetes.docker.internal to 127.0.0.1 in container hosts file (copy from host hosts file)
Setting host.containerhelper.internal to 172.26.240.1 in container hosts file
Starting Container
Hostname is bcserver
PublicDnsName is bcserver
Using Windows Authentication
Starting Local SQL Server
Starting Internet Information Server
Modifying Service Tier Config File with Instance Specific Settings
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Enabling Financials User Experience
Dismounting Tenant
Mounting Tenant
Mounting Database for default on server localhost\SQLEXPRESS with AllowAppDatabaseWrite = False
Sync'ing Tenant
Tenant is Operational
Creating http download site
Creating Windows user <my domain user here>
Unable to update the password. The value provided for the new password does not meet the length, complexity, or history requirements of the domain.

at <ScriptBlock>, C:\Run\SetupWindowsUsers.ps1: line 12
at <ScriptBlock>, C:\Run\navstart.ps1: line 199
at <ScriptBlock>, C:\Run\start.ps1: line 363
at <ScriptBlock>, <No file>: line 1Error
Initializing...
Setting host.docker.internal to 10.1.10.65 in container hosts file (copy from host hosts file)
Setting gateway.docker.internal to 10.1.10.65 in container hosts file (copy from host hosts file)
Setting kubernetes.docker.internal to 127.0.0.1 in container hosts file (copy from host hosts file)
Setting host.containerhelper.internal to 172.26.240.1 in container hosts file
Starting Container
Hostname is bcserver
PublicDnsName is bcserver
Using Windows Authentication
Starting Local SQL Server
Starting Internet Information Server
Modifying Service Tier Config File with Instance Specific Settings
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Enabling Financials User Experience
Dismounting Tenant
Mounting Tenant
Mounting Database for default on server localhost\SQLEXPRESS with AllowAppDatabaseWrite = False
Sync'ing Tenant
Tenant is Operational
Creating http download site
Creating Windows user <my domain user here>
Unable to update the password. The value provided for the new password does not meet the length, complexity, or history requirements of the domain.

at <ScriptBlock>, C:\Run\SetupWindowsUsers.ps1: line 12
at <ScriptBlock>, C:\Run\navstart.ps1: line 199
at <ScriptBlock>, C:\Run\start.ps1: line 363
at <ScriptBlock>, <No file>: line 1
New-BcContainer Telemetry Correlation Id: 58ecfeba-f493-4e9d-b61c-4c34b078586d
Initialization of container bcserver failed
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\3.0.4\ContainerHandling\Wait-NavContainerReady.ps1:42 char:17
+ ...             throw "Initialization of container $containerName failed" ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Initialization ...bcserver failed:String) [], RuntimeException
    + FullyQualifiedErrorId : Initialization of container bcserver failed
...

Additional context

does it happen all the time? Yes
did it use to work? I don't think it's ever worked for me

Not knowing what your Windows Password is, I expect it to not meet the default requirements of the Windows Server Core in the container.
You could try to change your Windows Password to "OuuCUN%$l52x1EFK" or another strong password and see if it works then.
It is likely not anything I can do anything about.

OK that explains the part I was missing.  The domain requirements in question are not the host domain, but the containers domain (i.e. in Windows Server Core).  I was incorrectly assuming that it would use the same requirements as used for the host.

Correct, containers cannot be domain joined.
For Windows Auth with containers you can use group managed service account.
The "native" Windows auth with containers lives off a functionality which dates back from Windows 3.11, where a user with the same username and password on two machines is assumed to be the same guy.

authors: BW-PA freddydk

repo: microsoft/navcontainerhelper


text:
Assign Premium Plan fails with version 6.2.92

A pipeline that last used to work >1week ago has stopped functioning today. The only visible difference is navcontainerhelper being updated to 0.6.2.92. The release notes indicate that there was some change regarding the Premium Plan for 15.x containers. Maybe something got botched for 14.x containers accidentally?
The error is:
Violation of PRIMARY KEY constraint 'User Plan$0'. Cannot insert duplicate key in object 'dbo.User Plan'. The duplicate key value is (8e9002c0-a1d8-4465-b952-817d2948e6e2, 27867dc1-70f8-4c53-92d5-d4c6746e0888).

Here is the complete output of that pipeline step:
2019-08-15T10:03:08.7725464Z ##[section]Starting: Prepare environment
2019-08-15T10:03:08.8042714Z ==============================================================================
2019-08-15T10:03:08.8043067Z Task         : PowerShell
2019-08-15T10:03:08.8043324Z Description  : Run a PowerShell script on Linux, macOS, or Windows
2019-08-15T10:03:08.8043744Z Version      : 2.151.2
2019-08-15T10:03:08.8043944Z Author       : Microsoft Corporation
2019-08-15T10:03:08.8044242Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/powershell
2019-08-15T10:03:08.8044540Z ==============================================================================
2019-08-15T10:03:11.0364769Z Generating script.
2019-08-15T10:03:11.0878355Z ========================== Starting Command Output ===========================
2019-08-15T10:03:11.1178881Z ##[command]"C:\windows\System32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command ". 'C:\Agent\_work\_temp\90f19a34-7cee-482c-bf81-c84a743086b9.ps1'"
2019-08-15T10:03:14.0327006Z TF_BUILD set, running under agent, enforcing Build flag
2019-08-15T10:03:14.0334369Z Build is true
2019-08-15T10:03:14.0341934Z Using passed password and NavUserPassword authentication
2019-08-15T10:03:14.5471515Z NavContainerHelper is version 0.6.2.92
2019-08-15T10:03:14.5526616Z Host is Microsoft Windows Server 2019 Datacenter - ltsc2019
2019-08-15T10:03:14.8641339Z Docker Client Version is 18.09.6
2019-08-15T10:03:15.0068662Z Docker Server Version is 18.09.6
2019-08-15T10:03:15.0254495Z Pulling image mcr.microsoft.com/businesscentral/sandbox:de-ltsc2019
2019-08-15T10:03:15.5735640Z de-ltsc2019: Pulling from businesscentral/sandbox
2019-08-15T10:03:15.5799714Z Using image mcr.microsoft.com/businesscentral/sandbox:de-ltsc2019
2019-08-15T10:03:15.9046896Z Removing C:\ProgramData\NavContainerHelper\Extensions\bleedingedge
2019-08-15T10:03:16.1425030Z Creating Container bleedingedge
2019-08-15T10:03:16.1431464Z Using license file ***
2019-08-15T10:03:16.2971140Z Version: 14.3.34444.35087-DE
2019-08-15T10:03:16.4583029Z Platform: 14.0.35036.0
2019-08-15T10:03:16.6175627Z Generic Tag: 0.0.9.91
2019-08-15T10:03:16.7721681Z Container OS Version: 10.0.17763.615 (ltsc2019)
2019-08-15T10:03:16.7728439Z Host OS Version: 10.0.17763.678 (ltsc2019)
2019-08-15T10:03:16.7822555Z WARNING: The container operating system matches the host operating system, but the revision is different.
2019-08-15T10:03:16.7828569Z If you encounter issues, you might want to specify -isolation hyperv
2019-08-15T10:03:16.7875331Z Using locale de-DE
2019-08-15T10:03:16.7896463Z Using process isolation
2019-08-15T10:03:16.7919299Z Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
2019-08-15T10:03:16.8433905Z Downloading C:\ProgramData\NavContainerHelper\Extensions\bleedingedge\my\SetupScriptsWildcard.zip
2019-08-15T10:03:16.9928857Z Extracting .zip file
2019-08-15T10:03:17.5553743Z Downloading C:\ProgramData\NavContainerHelper\Extensions\bleedingedge\my\license.flf
2019-08-15T10:03:17.8931156Z Files in C:\ProgramData\NavContainerHelper\Extensions\bleedingedge\my:
2019-08-15T10:03:17.9013867Z - AdditionalOutput.ps1
2019-08-15T10:03:17.9026580Z - AdditionalSetup.ps1
2019-08-15T10:03:17.9035032Z - license.flf
2019-08-15T10:03:17.9044234Z - MainLoop.ps1
2019-08-15T10:03:17.9057039Z - SetupCertificate.ps1
2019-08-15T10:03:17.9082265Z - SetupClickOnce.ps1
2019-08-15T10:03:17.9097745Z - SetupFileShare.ps1
2019-08-15T10:03:17.9117475Z - SetupNavUsers.ps1
2019-08-15T10:03:17.9446721Z - updatehosts.ps1
2019-08-15T10:03:17.9466573Z Creating container bleedingedge from image mcr.microsoft.com/businesscentral/sandbox:de-ltsc2019
2019-08-15T10:03:18.4673036Z 150be2a26104937387b781625fba91f70ec305e28c02afc954fb8cf947fe22cd
2019-08-15T10:03:21.9917528Z Waiting for container bleedingedge to be ready
2019-08-15T10:03:32.6455900Z Initializing...
2019-08-15T10:03:32.6466705Z Starting Container
2019-08-15T10:03:32.6472867Z Hostname is bleedingedge
2019-08-15T10:03:32.6479014Z PublicDnsName is bleedingedge.unitop.engineering
2019-08-15T10:03:33.8045172Z Using NavUserPassword Authentication
2019-08-15T10:03:42.6265747Z Starting Local SQL Server
2019-08-15T10:04:03.9501805Z Starting Internet Information Server
2019-08-15T10:04:03.9502324Z
2019-08-15T10:04:03.9502596Z Name                           Version          Source           Summary
2019-08-15T10:04:03.9502796Z ----                           -------          ------           -------
2019-08-15T10:04:28.3650427Z nuget                          2.8.5.208        https://onege... NuGet provi...
2019-08-15T10:04:28.3662195Z
2019-08-15T10:04:28.3671940Z ICloudBlob                         : Microsoft.Azure.Storage.Blob.CloudBlockBlo
2019-08-15T10:04:28.3679503Z                                      b
2019-08-15T10:04:28.3687548Z BlobType                           : BlockBlob
2019-08-15T10:04:28.3694652Z Length                             : 5645
2019-08-15T10:04:28.3702072Z IsDeleted                          : False
2019-08-15T10:04:28.3710392Z RemainingDaysBeforePermanentDelete :
2019-08-15T10:04:28.3718343Z ContentType                        : application/x-pkcs12
2019-08-15T10:04:28.3725913Z LastModified                       : 5/27/2019 12:11:45 PM +00:00
2019-08-15T10:04:28.3732893Z SnapshotTime                       :
2019-08-15T10:04:28.3740545Z ContinuationToken                  :
2019-08-15T10:04:28.3747731Z Context                            : Microsoft.WindowsAzure.Commands.Storage.Az
2019-08-15T10:04:28.3755892Z                                      ureStorageContext
2019-08-15T10:04:28.3762516Z Name                               : wildcard_unitop.engineering.pfx
2019-08-15T10:04:28.3766511Z
2019-08-15T10:04:28.3773234Z Importing Certificate to LocalMachine\my
2019-08-15T10:04:28.3779673Z DNS identity unitop.engineering
2019-08-15T10:04:28.3787167Z Modifying Service Tier Config File with Instance Specific Settings
2019-08-15T10:04:28.3793592Z Modifying Service Tier Config File with settings from environment variable
2019-08-15T10:04:31.0455758Z Setting ServicesUseNTLMAuthentication to true
2019-08-15T10:04:38.4296138Z Starting Service Tier
2019-08-15T10:04:38.4316242Z Registering event sources
2019-08-15T10:05:08.2727298Z Creating DotNetCore Web Server Instance
2019-08-15T10:05:08.6088758Z Enabling Financials User Experience
2019-08-15T10:05:08.6643677Z Using license file 'c:\run\my\license.flf'
2019-08-15T10:06:03.5066307Z Import License
2019-08-15T10:06:06.4106591Z Creating http download site
2019-08-15T10:06:06.4113709Z Setting SA Password and enabling SA
2019-08-15T10:06:10.6152769Z Creating GOBAdmin as SQL User and add to sysadmin
2019-08-15T10:06:11.9171316Z Creating SUPER user
2019-08-15T10:06:11.9177986Z Assign Premium plan for GOBADMIN
2019-08-15T10:06:11.9182102Z
2019-08-15T10:06:11.9188884Z AccountExpires         :
2019-08-15T10:06:11.9200157Z Description            :
2019-08-15T10:06:11.9205996Z Enabled                : True
2019-08-15T10:06:11.9213407Z FullName               :
2019-08-15T10:06:11.9220604Z PasswordChangeableDate : 8/15/2019 10:06:11 AM
2019-08-15T10:06:11.9227421Z PasswordExpires        : 9/26/2019 10:06:11 AM
2019-08-15T10:06:11.9234134Z UserMayChangePassword  : True
2019-08-15T10:06:11.9241356Z PasswordRequired       : False
2019-08-15T10:06:11.9247406Z PasswordLastSet        : 8/15/2019 10:06:11 AM
2019-08-15T10:06:11.9254155Z LastLogon              :
2019-08-15T10:06:11.9260658Z Name                   : PSAdmin
2019-08-15T10:06:11.9267666Z SID                    : S-1-5-21-2076345448-4069775801-2563238138-1001
2019-08-15T10:06:11.9273968Z PrincipalSource        : Local
2019-08-15T10:06:11.9280365Z ObjectClass            : User
2019-08-15T10:06:13.1949024Z
2019-08-15T10:06:13.1964754Z Assign Premium plan for GOBADMIN
2019-08-15T10:06:13.1992037Z Violation of PRIMARY KEY constraint 'User Plan$0'. Cannot insert duplicate key in object 'dbo.User Plan'. The duplicate key value is (8e9002c0-a1d8-4465-b952-817d2948e6e2, 27867dc1-70f8-4c53-92d5-d4c6746e0888).
2019-08-15T10:06:13.2017328Z The statement has been terminated.
2019-08-15T10:06:13.2040901Z at <ScriptBlock>, C:\Run\my\SetupNavUsers.ps1: line 40
2019-08-15T10:06:13.2053576Z at <ScriptBlock>, C:\Run\my\SetupNavUsers.ps1: line 32
2019-08-15T10:06:13.2058991Z at <ScriptBlock>, C:\Run\navstart.ps1: line 179
2019-08-15T10:06:13.2070211Z at <ScriptBlock>, C:\Run\start.ps1: line 121
2019-08-15T10:06:13.2075498Z at <ScriptBlock>, <No file>: line 1Error
2019-08-15T10:06:13.2085377Z Initializing...
2019-08-15T10:06:13.2090810Z Starting Container
2019-08-15T10:06:13.2101445Z Hostname is bleedingedge
2019-08-15T10:06:13.2106673Z PublicDnsName is *********************************
2019-08-15T10:06:13.2117868Z Using NavUserPassword Authentication
2019-08-15T10:06:13.2123156Z Starting Local SQL Server
2019-08-15T10:06:13.2136132Z Starting Internet Information Server
2019-08-15T10:06:13.2137316Z
2019-08-15T10:06:13.2148836Z Name                           Version          Source           Summary
2019-08-15T10:06:13.2157303Z ----                           -------          ------           -------
2019-08-15T10:06:13.2165720Z nuget                          2.8.5.208        https://onege... NuGet provi...
2019-08-15T10:06:13.2173381Z
2019-08-15T10:06:13.2179992Z ICloudBlob                         : Microsoft.Azure.Storage.Blob.CloudBlockBlo
2019-08-15T10:06:13.2188006Z                                      b
2019-08-15T10:06:13.2196886Z BlobType                           : BlockBlob
2019-08-15T10:06:13.2204815Z Length                             : 5645
2019-08-15T10:06:13.2212834Z IsDeleted                          : False
2019-08-15T10:06:13.2220062Z RemainingDaysBeforePermanentDelete :
2019-08-15T10:06:13.2228226Z ContentType                        : application/x-pkcs12
2019-08-15T10:06:13.2234946Z LastModified                       : 5/27/2019 12:11:45 PM +00:00
2019-08-15T10:06:13.2242486Z SnapshotTime                       :
2019-08-15T10:06:13.2249636Z ContinuationToken                  :
2019-08-15T10:06:13.2257456Z Context                            : Microsoft.WindowsAzure.Commands.Storage.Az
2019-08-15T10:06:13.2263967Z                                      ureStorageContext
2019-08-15T10:06:13.2271361Z Name                               : wildcard_unitop.engineering.pfx
2019-08-15T10:06:13.2275378Z
2019-08-15T10:06:13.2282812Z Importing Certificate to LocalMachine\my
2019-08-15T10:06:13.2290169Z DNS identity unitop.engineering
2019-08-15T10:06:13.2296417Z Modifying Service Tier Config File with Instance Specific Settings
2019-08-15T10:06:13.2303921Z Modifying Service Tier Config File with settings from environment variable
2019-08-15T10:06:13.2311333Z Setting ServicesUseNTLMAuthentication to true
2019-08-15T10:06:13.2319373Z Starting Service Tier
2019-08-15T10:06:13.2328302Z Registering event sources
2019-08-15T10:06:13.2399253Z Creating DotNetCore Web Server Instance
2019-08-15T10:06:13.2505519Z Enabling Financials User Experience
2019-08-15T10:06:13.2515253Z Using license file 'c:\run\my\license.flf'
2019-08-15T10:06:13.3118207Z Import License
2019-08-15T10:06:13.3244546Z Creating http download site
2019-08-15T10:06:13.3635888Z Setting SA Password and enabling SA
2019-08-15T10:06:13.3964766Z Creating GOBAdmin as SQL User and add to sysadmin
2019-08-15T10:06:13.4106382Z Creating SUPER user
2019-08-15T10:06:13.4156677Z Assign Premium plan for GOBADMIN
2019-08-15T10:06:13.4160292Z
2019-08-15T10:06:13.4177898Z AccountExpires         :
2019-08-15T10:06:13.4196330Z Description            :
2019-08-15T10:06:13.4222414Z Enabled                : True
2019-08-15T10:06:13.4806812Z FullName               :
2019-08-15T10:06:13.4833563Z PasswordChangeableDate : 8/15/2019 10:06:11 AM
2019-08-15T10:06:13.4880345Z PasswordExpires        : 9/26/2019 10:06:11 AM
2019-08-15T10:06:13.4902065Z UserMayChangePassword  : True
2019-08-15T10:06:13.4919020Z PasswordRequired       : False
2019-08-15T10:06:13.4937979Z PasswordLastSet        : 8/15/2019 10:06:11 AM
2019-08-15T10:06:13.4954986Z LastLogon              :
2019-08-15T10:06:13.4968883Z Name                   : PSAdmin
2019-08-15T10:06:13.5005091Z SID                    : S-1-5-21-2076345448-4069775801-2563238138-1001
2019-08-15T10:06:13.5034134Z PrincipalSource        : Local
2019-08-15T10:06:13.5051729Z ObjectClass            : User
2019-08-15T10:06:13.5060708Z
2019-08-15T10:06:13.5066199Z Assign Premium plan for GOBADMIN
2019-08-15T10:06:13.5085691Z Violation of PRIMARY KEY constraint 'User Plan$0'. Cannot insert duplicate key in object 'dbo.User Plan'. The duplicate key value is (8e9002c0-a1d8-4465-b952-817d2948e6e2, 27867dc1-70f8-4c53-92d5-d4c6746e0888).
2019-08-15T10:06:13.5298214Z The statement has been terminated.
2019-08-15T10:06:13.5308686Z at <ScriptBlock>, C:\Run\my\SetupNavUsers.ps1: line 40
2019-08-15T10:06:13.5311913Z at <ScriptBlock>, C:\Run\my\SetupNavUsers.ps1: line 32
2019-08-15T10:06:13.5315305Z at <ScriptBlock>, C:\Run\navstart.ps1: line 179
2019-08-15T10:06:13.5322665Z at <ScriptBlock>, C:\Run\start.ps1: line 121
2019-08-15T10:06:13.5409532Z at <ScriptBlock>, <No file>: line 1
2019-08-15T10:06:13.7183641Z ##[error]Initialization of container bleedingedge failed
At C:\Program

Files\WindowsPowerShell\Modules\navcontainerhelper\0.6.2.92\ContainerHandling\Wait-NavContainerReady.ps1:45 char:17

+ ...             throw "Initialization of container $containerName failed" ...

+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

    + CategoryInfo          : OperationStopped: (Initialization ...dingedge failed:String) [], RuntimeException

    + FullyQualifiedErrorId : Initialization of container bleedingedge failed
2019-08-15T10:06:13.7228850Z ##[error]PowerShell exited with code '1'.
2019-08-15T10:06:13.8019828Z ##[section]Finishing: Prepare environment

Also tested with 0.6.2.93 - same failure

Any chance that you could email me the SetupNavUsers.ps1 from that machine.
c:\programdata\navcontainerhelper\extensions\containername\my\setupnavusers.ps1
Just copy/paste the content here or in an email

$InstanceName = (Get-NAVServerInstance).ServerInstance.split("$")[-1]
if ($auth -eq "Windows") {
    if ($username -ne "") {
        if (!(Get-NAVServerUser -ServerInstance $InstanceName @tenantParam -ErrorAction Ignore | Where-Object { $_.UserName.EndsWith("\$username", [System.StringComparison]::InvariantCultureIgnoreCase) -or $_.UserName -eq $username })) {
            Write-Host "Creating SUPER user"
            New-NavServerUser -ServerInstance $InstanceName @tenantParam -WindowsAccount $username
            New-NavServerUserPermissionSet -ServerInstance $InstanceName @tenantParam -WindowsAccount $username -PermissionSetId SUPER
        }
    }
} else {
    if (!(Get-NAVServerUser -ServerInstance $InstanceName @tenantParam -ErrorAction Ignore | Where-Object { $_.UserName -eq $username })) {
        Write-Host "Creating SUPER user"
        New-NavServerUser -ServerInstance $InstanceName @tenantParam -Username $username -Password $securePassword -AuthenticationEMail $authenticationEMail
        New-NavServerUserPermissionSet -ServerInstance $InstanceName @tenantParam -username $username -PermissionSetId SUPER
    }
}
Get-NavServerUser -serverInstance $InstanceName -tenant default |? LicenseType -eq "FullUser" | ForEach-Object {
    $UserId = $_.UserSecurityId
    Write-Host "Assign Premium plan for $($_.Username)"
    $dbName = $DatabaseName
    if ($multitenant) {
        $dbName = $TenantId
    }
    sqlcmd -S 'localhost\SQLEXPRESS' -d $DbName -Q "INSERT INTO [dbo].[User Plan] ([Plan ID],[User Security ID]) VALUES ('{8e9002c0-a1d8-4465-b952-817d2948e6e2}','$userId')" | Out-Null
}
New-LocalUser -Name "PSAdmin" -Password (ConvertTo-SecureString -String "**************************" -AsPlainText -Force)
Add-LocalGroupMember -Group "Administrators" -Member "PSAdmin"
Add-LocalGroupMember -Group "Remote Desktop Users" -Member "PSAdmin"
``

It doesn't look like the new change from 0.6.2.92 is here at all.
New-NavContainer will add code to SetupNavUsers that I don't see here?
But it looks like you have overwritten the default behavior and have renamed the serverinstance???
Is there more in the file than this?

Can't say why... will have to delegate this to a DevOps Engineer to look into if and why that script was overridden. Anyway, that is a possible explanation why things stopped working,
AFAIK the guys did not rename the serverinstance. They did insert stuff to check if it is a 14.x or 15.x container because the service tier is called BC from 15 on. That way they could use the same scripts and would not need distinct ones per version. This is vital because at some point our automated systems will start to pull what is then bcsandbox and ... well. The road to hell is full of good intentions :-)

The code that gets appended by new-navcontainer will use invoke-sqlcmd (which probably is what's failing) - it doesn't have -erroraction ignore - which I guess it should, but the reason for the error here is probably that you have taken over SetupNavUsers yourself (and then I have added something:-()
If you like, we could have a quick conf. call - where you go through the things you have done - and I can give you my recommendations as to what you should do differently (if anything).
It would also give me an understanding of the challenges you face.

Just one more info: This pipeline is the only one to use UserPWAuth. The other pipelines use Windows Auth and are running 0.6.2.93 flawlessly (for now).
We would gladly take You up on the kind offer for a conf call - but that will have to be next week when the guys who know what they are doing are available. Will send You an invite.

OK, see you then.
BTW - there is a variable in the container called $ServerInstance (defined in c:\run\ServiceSettings.ps1) which will hold the ServerInstance name in the container.

authors: RafaelKoch freddydk


text:
no gpu found on remote mode

Describe the issue:
Environment:

NNI version: 2.5
Training service (local|remote|pai|aml|etc): remote
Server OS (for remote mode only):ubuntu 18.04
Python version:3.6.9
PyTorch/TensorFlow version: pytorch 1.6.0+cu101
Is running in Docker?: YES

Configuration:

Experiment config (remember to remove secrets!): Mnist-pytorch
Search space:

Log message:

nnimanager.log:
[2022-01-12 14:41:40] INFO (NNIManager) Trial job v6Q3x status changed from WAITING to RUNNING
[2022-01-12 14:41:40] INFO (NNIManager) Trial job w1IS6 status changed from WAITING to RUNNING
[2022-01-12 14:41:40] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:41] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:42] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:43] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:44] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:45] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:45] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:46] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:47] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:48] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:49] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:50] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:51] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:51] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:52] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:53] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:55] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:55] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
[2022-01-12 14:41:56] INFO (TrialDispatcher) TrialDispatcher: 1 live env, and 1 reusable, but no GPU available so request a new one.
dispatcher.log:
[2022-01-12 14:41:30] INFO (nni.runtime.msg_dispatcher_base/MainThread) Dispatcher started
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) tpe_transform took 0.001629 seconds
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) TPE using 0 trials
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) tpe_transform took 0.001548 seconds
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) TPE using 0 trials
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) tpe_transform took 0.001542 seconds
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) TPE using 0 trials
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) tpe_transform took 0.001561 seconds
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) TPE using 0 trials
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) tpe_transform took 0.001513 seconds
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) TPE using 0 trials
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) tpe_transform took 0.001584 seconds
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) TPE using 0 trials
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) tpe_transform took 0.001514 seconds
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) TPE using 0 trials
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) tpe_transform took 0.001543 seconds
[2022-01-12 14:41:31] INFO (hyperopt.tpe/Thread-1) TPE using 0 trials

I am just using the provided example trial---mnist-pytoch. The only change is config file. Here is my config.yml
nniManagerIp: 172.17.0.18
experimentName: MNIST
searchSpaceFile: search_space.json
trialCommand: python3 mnist.py
trialGpuNumber: 2
trialConcurrency: 8
experimentWorkingDirectory: '/app/nni/experiments'
maxTrialNumber: 20
useAnnotation: false
tuner:
  name: TPE
  classArgs:
    optimize_mode: maximize
trainingService:
  platform: remote
  machineList:
    - host: 10.10.30.25
      port: 8888
      user: root
      password: masike
      maxTrialNumberPerGpu: 4
      gpuIndices: '1,2'

The web UI keeps showing running and waiting without any progress.
The GPU NO.1 and 2 are not shown to be used, but I can see the python3 nni progress through htop
I am planning to test how to use nni to training on server GPUs, but this issue happened.  I also tried to reinstall NNI by the wheel.
Any suggestions would be appreciated!

Please check the output of nvidia-smi on the server.
If the GPU has other tasks running already, you can set useActiveGpu to true.

Hi, I switched it to false and true, and keep watching GPU to make sure there is no progress on the GPUs. The issue still happened.

I restarted the server and it works.

experimentName: MNIST
searchSpaceFile: search_space.json
trialCommand: python3 mnist.py
trialGpuNumber: 1
trialConcurrency: 8
maxTrialNumber: 20
tuner:
  name: TPE
  classArgs:
    optimize_mode: maximize
trainingService:
  platform: remote
  machineList:
    - host: 10.10.30.25
      port: 8888
      user: root
      password: masike
      maxTrialNumberPerGpu: 4
      gpuIndices: 1,3
      useActiveGpu: true

I am using this config to test mnist-PyTorch, but found gpuindices cannot work properly. I want to use GPU '1,3' to train 8 trials at the same time. However, The GPU '0,1,2,3' would all be used, 4 trials in waiting mode, 4 in running mode. Can anybody give me some suggestion how to fix it ?

Please check /tmp/nni-experiments/<experiment-id>/run.sh.
It should contain CUDA_VISIBLE_DEVICES=xxx.

Please check /tmp/nni-experiments/<experiment-id>/run.sh. It should contain CUDA_VISIBLE_DEVICES=xxx.

Hi, this is the command of run.sh: ' cd /tmp/nni-experiments/vyVEpsc1 &&             mkdir -p envs/EmvYz && cd envs/EmvYz && sh ../install_nni.sh && python3 -m nni.tools.trial_tool.trial_runner --job_pid_file /tmp/nni-experiments/vyVEpsc1/envs/EmvYz/pid             1>/tmp/nni-experiments/vyVEpsc1/envs/EmvYz/trialrunner_stdout 2>/tmp/nni-experiments/vyVEpsc1/envs/EmvYz/trialrunner_stderr             && echo $? date +%s%3N >/tmp/nni-experiments/vyVEpsc1/envs/EmvYz/code '
There is nothing about CUDA_VISIBLE_DEVICES.  Any approaches to fix it?

Then it's confirmed a bug.
I have just taken over this part so it may take some time to fix it...
Sorry for inconvenience.

Then it's confirmed a bug. I have just taken over this part so it may take some time to fix it... Sorry for inconvenience.

Thanks for the folllow up

Sorry I have misunderstood the mechanism...
Please add following lines to your trial code and check the log:
import logging
import os
import nni
logging.getLogger('DebugCudaVisibleDevices').info(f'trial {nni.get_trial_id()}: "{os.getenv('CUDA_VISIBLE_DEVICES')}"')

Sorry I have misunderstood the mechanism...
Please add following lines to your trial code and check the log:
import logging
import os
import nni
logging.getLogger('DebugCudaVisibleDevices').info(f'trial {nni.get_trial_id()}: "{os.getenv('CUDA_VISIBLE_DEVICES')}"')

The code has a format problem and I fixed a bit, here is what I added.
import logging
import os
import nni

logger = logging.getLogger('DebugCudaVisibleDevices')
logger.info(f'''trial {nni.get_trial_id()}: "{os.getenv('CUDA_VISIBLE_DEVICES')}"''')

and run.sh shows
cd /tmp/nni-experiments/fqX4vUAg &&             mkdir -p envs/uDj0b && cd envs/uDj0b && sh ../install_nni.sh && python3 -m nni.tools.trial_tool.trial_runner --job_pid_file /tmp/nni-experiments/fqX4vUAg/envs/uDj0b/pid             1>/tmp/nni-experiments/fqX4vUAg/envs/uDj0b/trialrunner_stdout 2>/tmp/nni-experiments/fqX4vUAg/envs/uDj0b/trialrunner_stderr             && echo $? `date +%s%3N` >/tmp/nni-experiments/fqX4vUAg/envs/uDj0b/code

在上面的config配置下，我有个4卡服务器，如果四个gpu都是空闲的，trial会在每个gpu都占1000MB左右（每个gpu内存为11000MB）。我随便跑了个程序占满其中一个gpu，那么就会有几个trials fail，剩余的success，我跑的程序也不会被卡掉。现在无法占满gpu也无法控制制定gpu。我是docker run建立双端口，然后在docker里安的环境。

repo: microsoft/nni


text:
Add new neteventebpfext eBPF extension for processing network events.

Description
This PR:

Adds a new neteventebpfext eBPF extension for capturing network events.
Adds a new netevent_sim kernel driver, which simulates an event provider for testing purposes.
Adds an new netevent_monitor eBPF program that will be invoked by the neteventebpfext extension upon receiving "special" events (actual or simulator), which will store the events in a ring-buffer map.
Adds e2e tests in generating events from netevent_sim kernel driver, flowing from the neteventebpfext eBPF extension, into the  netevent_monitor eBPF program, into the ring-buffer map and ultimately into a UM app (i.e. the unit test) being called back upon every event insertion.
Removes duplicate store_helper library files, already implemented in EbpfApi.dll.

Testing
Local, CICD.
Documentation

README.md
docs\neteventebpfext.md

Installation
n.a.

As a general comment, this PR contains significant code re-org.  While this re-org is certainly worthwhile, it seems orthogonal to the actual extension code payload.  'As-is', reverting this PR (for any reason) will cause all that re-org to be reverted as well.
Is it possible to separate out the code re-org into a separate PR?

As a general comment, this PR contains significant code re-org. While this re-org is certainly worthwhile, it seems orthogonal to the actual extension code payload. 'As-is', reverting this PR (for any reason) will cause all that re-org to be reverted as well.
Is it possible to separate out the code re-org into a separate PR?

As a general comment, this PR contains significant code re-org. While this re-org is certainly worthwhile, it seems orthogonal to the actual extension code payload. 'As-is', reverting this PR (for any reason) will cause all that re-org to be reverted as well.
Is it possible to separate out the code re-org into a separate PR?

As a general comment, this PR contains significant code re-org. While this re-org is certainly worthwhile, it seems orthogonal to the actual extension code payload. 'As-is', reverting this PR (for any reason) will cause all that re-org to be reverted as well.
Is it possible to separate out the code re-org into a separate PR?

As a general comment, this PR contains significant code re-org. While this re-org is certainly worthwhile, it seems orthogonal to the actual extension code payload. 'As-is', reverting this PR (for any reason) will cause all that re-org to be reverted as well.
Is it possible to separate out the code re-org into a separate PR?

This repo was designed to offer a library of boilerplate code to accelerate the development of eBPF extensions, and host multiple other extensions that build on top of that.
Upon its creation, it was self-checked in with the ntosebpfext placed in the root, therefore missing to offer a structured design to host other future extensions, in a well understandable, ordered way.
Happens that upon adding the second extension, the structure and other minor refactoring had to be adapted to have the repo comply to its design intent in the first place, which was missing.

authors: dv-msft gtrevi

repo: microsoft/ntosebpfext


text:
Use 64-bit worker for function

We want the .NET code running as 64-bit. This shouldn't affect the Python code.

Confirmed working.

repo: microsoft/onefuzz


text:
opgen: run lintrunner instead of black/isort

The SPACES linter has some terrible O(N^?) perf when explicitly passing paths that comprise tens of thousands of lines of code, so skip it here. The generator doesn't write lines with trailing spaces anyway.

Thanks for pointing out the issue with SPACES! I will take a closer look.
Regarding this change, I am curious of your thoughts on bringing in lintrunner as a build dependency, presuming we want to have this code gen process as a build step? I think it would be a more robust process relying on only isort and black, given that lintrunner is not working well on Windows yet and it also takes longer to run.
If we are going to not check in the generated code, lintrunner will ignore those files so nothing will be done.
If we are going to keep the generated code checked in, I think we can simply exclude them from lintrunner so that (1) the build is more robust, with less dependencies (2) the build is faster.
I do see the value in running mypy etc. on the generated code to make sure the types are right. I think this can be done by a manual run in lintrunner there.
What do you think?

authors: abock justinchuby

repo: microsoft/onnx-script


text:
Building from Source for Jetson

Hi,
I'm trying to build onnxruntime-genai with cuda support on Jetson device. I have onnxruntime-1.18.0 and onnxruntime-gpu 1.18.0.
I have placed the .so files in ort/lib/ and .h files in ort/include as instructed.
But when building, it fetches onnxruntime version 1.17.1 into the _deps. Does this version have to match the onnxruntime I have installed on the machine ? I'm not able to use CUDA to run genai.
Kindly help me on how to change the onnxruntime source version getting fetched by cmake to customizable option.
Thanks in advance.

Hi,
I'm trying to build onnxruntime-genai with cuda support on Jetson device. I have onnxruntime-1.18.0 and onnxruntime-gpu 1.18.0.
I have placed the .so files in ort/lib/ and .h files in ort/include as instructed.
But when building, it fetches onnxruntime version 1.17.1 into the _deps. Does this version have to match the onnxruntime I have installed on the machine ? I'm not able to use CUDA to run genai.
Kindly help me on how to change the onnxruntime source version getting fetched by cmake to customizable option.
Thanks in advance.

sample problem here

Can you share the logs of the build failure here please?

The build does not fail. It completes and generates a wheel. But when running, the library does not provide the GPU provider.

authors: PredyDaddy SRai22 natke

repo: microsoft/onnxruntime-genai


text:
OpenPai examples need to be updated

Organization Name:
CloudEdu

Short summary about the issue/question:
OpenPAI examples need to be updated. The examples use JSON but OpenPAI of v0.14.0 only accept YAML file when submit jobs.
Brief what process you are following:

How to reproduce it:

OpenPAI Environment: Ubuntu 16.04

OpenPAI version: v0.14.0
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Hardware (e.g. core number, memory size, storage size, GPU type etc.):
Others:

Anything else we need to know:

I agree with this. I've been confused when I come across JSON examples.

All outdated examples have been removed, pls refer to https://github.com/microsoft/pai/tree/v1.0.1/marketplace-v2 for examples.

authors: abuccts andrewjong victorming

repo: microsoft/pai


text:
[BUG] Page selectors are not being evaluated unless debug breakpoint is used

Context:

Playwright Version - 1.18.1
Operating System - Windows 10
.NET version - NET 6
Browser - Firefox
Extra - Visual Studio 2022, Fresh NET 6 Console Application

Code Snippet
using Microsoft.Playwright;

try
{
    var playwright = await Playwright.CreateAsync();
    var browser = await playwright.Firefox.LaunchAsync(new BrowserTypeLaunchOptions
                                                       {
                                                           Headless = false,
                                                           SlowMo = 50
                                                       });

    var context = await browser.NewContextAsync();
    var page = await context.NewPageAsync();

    await page.GotoAsync("https://login.aol.com/");
    await page.TypeAsync("id=login-username", "playwright_demo@aol.com");
    await page.ClickAsync("id=login-signin");

    var visible = await page.IsVisibleAsync("p[data-error]");

    if (visible)
    {
        Console.WriteLine("Element Found");
    }

    await Task.Delay(TimeSpan.FromMinutes(3));
}
catch (Exception e)
{
    Console.WriteLine(e);
}

Describe the bug
If you run the code above the element "p[data-error]" will not be found. However, if you put a breakpoint on the line
var visible = await page.IsVisibleAsync("p[data-error]");

and step through the code, it will find the element.

I guess the error element does not get immediately rendered but a little bit later. You can use
await page.WaitForSelectorAsync("p[data-error]");

instead which will wait a bit for the element being visible.
https://playwright.dev/dotnet/docs/api/class-page#page-wait-for-selector

Thanks, that worked great.

authors: FutureTD campersau

repo: microsoft/playwright-dotnet


text:
There is a problem with the gyroscope function in 【Controller】

code：https://makecode.com/_UaiihWYLaaEp
only "on tile left"work well

Duplicate of https://github.com/microsoft/pxt-arcade/issues/2349

repo: microsoft/pxt-arcade


text:
Set of SPY Bugs

I believe this happens for SPY in general, but it's definitely happening working with JS.

Compiler won't convert the image code to text within instructions or hint
After dropping block into workspace, it does not generate another line at the end of the program, making it hard to add the next block
After dropping a block into workspace, cursor ends up at random point in line, so if user presses enter to make a new line, they will likely split the text they just added
//@hide doesn't seem to work in JS
blockconfig doesn't seem to work in JS
The cursor doesn't automatically snap to the first/indented space while you're dragging, so sometimes you have to drag the block to the left outside of the workspace to get the text to appear at the beginning of the line
Controller block says move [mySprite] with buttons but when you drop it, mySprite becomes null
The JS that comes from the overlap block:
sprites.onOverlap(SpriteKind.Player, SpriteKind.Player, function(sprite: Sprite, otherSprite: Sprite) {}
Is different than the JS we typically use:
sprites.onOverlap(SpriteKind.Player, SpriteKind.Player, function(sprite, otherSprite) {}


text:
Support for localized strings in simulator

This is a proposal to support localization in the static typescript code. The idea is to define a template literal (e.g. loc) for localized strings and have the compiler swap the localized string at compile time.

No plans

authors: abchatra pelikhan


text:
Round trip from blocks -> ts -> py -> ts -> blocks gives back identical blocks

After much ado, I settled on fixing this bug by centralizing and introducing a cache around the ts->py and py->ts transpile operations. This is done in the compiler code, so in the worker.
Probably a better, longer term fix would be to implement a "MultiViewFile" abstraction in the webapp that encapsulates all language conversions (including blocks, which was the difficult part that I punted from this PR), tracking equivalence and keeps the trio [blocks, ts, py] in sync. Discussed this in person with @riknoll . This would allow us to remove a lot of code in the webapp and elsewhere that does things like "convert from A to B and then back to A to see if things have changed" and it would likely eliminate several race conditions.
But this PR is not that, it just handles the ts <-> py case.
Fixes:
Bug repro:

open Space Destroyer in blocks
convert to python and back to blocks
notice blocks have changed

Can we revert the "generated" -> "outfiles"? it would remove a bunch of changes.

Can we revert the "generated" -> "outfiles"? it would remove a bunch of changes.

Yeah but I think it's worth it. The data structure we use almost everywhere is "CompileResult" which has an "outfiles" property. For whatever reason ts2py had it's own output data structure that had a "generated" property that contained exactly the same information. This was inconvenient in this PR because I was trying to standardize the interface. "TranspileResult" is a strict subset of "CompileResult".

Sounds good!

Can we get a build for testing?

Can we get a build for testing?

I forget the cmd for that.. what is it again? I can't find it in my notes

pxt uploadtrg

pxt uploadtrg

ty. my local build isn't working (I always use --rb and there's no --rb option for uploadtrg? Is that intentional), I'll try to get it sorted out..

What is -rb ?

rebundle

Any ticks to be added to measure where those happen?

Like ticks inside the transpile.py2ts and .ts2py functions? It'd would be very noisy. A single ts->py->ts round trip calls those functions ~6 times for unclear reasons.
Coincidentally, I think we'll get a perf win from this PR because of that inefficiency.
I believe the important tick is when the user switched languages which I believe we have already.

LGTM

Can we get a build for testing?

https://arcade.makecode.com/app/ae2dc3a32fd5dfc641beadfcad7f6ff70ffb8414-9ab6c5d9b6

project

Do you (also @pelikhan, @mmoskal ) know if anything with:
interface CompileOptions {
        fileSystem: pxt.Map<string>;
        target: CompileTarget;
        testMode?: boolean;
        sourceFiles?: string[];
        generatedFiles?: string[];
        jres?: pxt.Map<pxt.JRes>;
        hexinfo: HexInfo;
        extinfo?: ExtensionInfo;
        noEmit?: boolean;
        forceEmit?: boolean;
        ast?: boolean;
        breakpoints?: boolean;
        trace?: boolean;
        justMyCode?: boolean;
        computeUsedSymbols?: boolean;
        name?: string;
        warnDiv?: boolean; // warn when emitting division operator
        apisInfo?: ApisInfo;
        bannedCategories?: string[];
        skipPxtModulesTSC?: boolean; // skip re-checking of pxt_modules/*
        skipPxtModulesEmit?: boolean; // skip re-emit of pxt_modules/*

        syntaxInfo?: SyntaxInfo;

        alwaysDecompileOnStart?: boolean; // decompiler only
        allowedArgumentTypes?: string[]; // decompiler-only; the types allowed for user-defined function arguments in blocks (unlisted types will cause grey blocks)

        embedMeta?: string;
        embedBlob?: string; // base64

        /* @internal */
        ignoreFileResolutionErrors?: boolean; // ignores triple-slash directive errors; debug only
    }

changes when the dependencies change? Probably ApisInfo?
My goal is to create a hash representing the current dependencies.

We always reload the header when dependencies change.

We always reload the header when dependencies change. So resetting your cache is enough to handle this case.

Which header?

Using all the qNames as part of the cache key seems to be working well.
(btw I FAR prefer to use a stronger cache key than count on some event to clear the cache. Much less error prone, easier to refactor, no races, and it's truly functional. Invalidating a cache due to some event is a last resort and it's usually an anti-pattern. Caching should simply be memoization whenever possible. IMHO.)
However, now after I've merged master it appears things have regressed. I'm getting an error when going ts->py on Space Destroyer so I'm looking into that.

In CompileOptions.sourceFiles, you should be able to look at pxt.json. I don't know if that would mess up @mmoskal's two tab extension editing though.

header is the internal name for a script/program. We reload the entire user package (header) when pxt.json is modified because toolbox and other stuff need to be refreshed.  At that point, you should invalidate this cache.

See https://github.com/microsoft/pxt/blob/cf7e498116fef6b52cf22852454b4b0c713550a5/webapp/src/monaco.tsx#L1135

reloadHeaderAsync

I'd really think it's a bad idea to invalidate caches based on an event. This turned into a huge pain when I did the caching around the blockly flyout and it was brittle an hard to refactor around.
I think its better to just capture the dependencies in the cache key. Then they'll automatically be invalidated and there won't be any race between something trying to read the cache right when the dependencies are invalidated, etc.

Watching pxt.json is not enough. If you have a local dependency, the content of the dependency may have changed in another tab (this happens with double-tab testing). This is why we reload the program when you enter a tab.

Wait, I'm hitting an exception, need to investigate:
[ ] Fix null exception, probably https://github.com/microsoft/pxt-minecraft/issues/1472 will need to be fixed.

Cache needs to be cleared when project dependencies change

Fixed. See "resetCache"

authors: darzu jwunderl pelikhan riknoll

repo: microsoft/pxt


text:
vscode does not provide intellisense

The VSCode autocomplete option doesn't work for tensorflow and keras libraries; However i've installed python and pylance extension on it.
Here i'm trying to use layers or preprocessing from keras API but it doesn't show anything at all.

I used a virtual environment and made sure to have the tensorflow package installed in the environment.
I also set these for vscode, but it doesn't seem to work.
    "python.analysis.extraPaths": [
        "C:\\WorkSpace\\PyTest0802\\.venv\\Lib\\site-packages"
    ],
    "python.autoComplete.extraPaths": [
        "C:\\WorkSpace\\PyTest0802\\.venv\\Lib\\site-packages"
    ],

shouldn't have to set extra paths.
are you setting your environment? see https://code.visualstudio.com/docs/python/environments#_select-and-activate-an-environment

Sorry that my statement might not be clear enough.
Adding an extra path setting is just a workaround I've found, it just still doesn't solve the problem.
Without setting the extra path, intellisense just doesn't work like the picture I showed.
I'm not setting up the environment. I'm asking how to make intellisense work correctly. I have my environment set up. Create and activate a virtual environment, choose an interpreter for the virtual environment, and install these packages for the virtual environment. I have done all of these.

see this suggestion https://github.com/microsoft/pylance-release/issues/1066#issuecomment-1218230855

new stable release has the partial fix https://github.com/tensorflow/tensorflow/releases/tag/v2.10.0

authors: bschnurr xxc-zsz

repo: microsoft/pylance-release


text:
Incorrect typing error for Decimal operators

Environment data

Language Server version: 2020.9.6
OS and version: MacOs 10.15.6
Python version (& distribution if applicable, e.g. Anaconda): 3.8.1

Expected behaviour
from decimal import Decimal
x = Decimal("2.5")
y = Decimal("3.2")
z = x + y

Standard mathematical operators between two "Decimal" objects should not produce typing errors.
Actual behaviour
Using "basic" Type Checking Mode:
Operator "+" not supported for types "Decimal" and "Decimal"Pylance (reportGeneralTypeIssues)

I'm not able to repro the problem you're seeing. Perhaps you have some other "decimal" package installed that is overriding the stdlib version? If you click on Decimal and press F12 to "show definition", does it take you to the decimal.pyi stub file that ships with the Pylance extension?

Looks like the problem is with the Datastax Cassandra driver Decimal column type. While it's treated like a regular Decimal under the hood, it is indeed different. I'll manage around it for now... thank you.

authors: erictraut joshuamanns


text:
Fixed Issue with Range Slicing Bug

Fixes #1430

/azp run

repo: microsoft/qsharp-compiler


text:
Remove "Message:" prefix from the output from Message function (running in VS Code)

Describe the bug
Prefacing the output from the Message function with "Message:" (when running in VS Code) looks very jarring to me, and not what I expect. It makes sense to call out "Result" explicitly, since this happens once per program, and DumpMachine output is large enough that a "DumpMachine" prefix doesn't stand out, but the "Message" prefix looks noisy.
Screenshots
For example, in the following example I use newlines to add a break between different parts of the example, and having "Message:" strings makes it look odd:

System information

QDK 1.0.33

Hi, I'd like to clarify the requirements. Is it ideal to remove the message: and directly output the content in the message?

Yes

authors: colommar tcNickolas

repo: microsoft/qsharp


text:
[Fabric] Fix NativeComponentExample not rendering on Fabric

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[x] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[ ] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Summary
The NativeComponentExample is currently not rendering when building for fabric on iOS.
ref._internalInstanceHandle?.stateNode?.canonical is undefined.
https://github.com/microsoft/react-native-macos/blob/main/packages/rn-tester/NativeComponentExample/js/MyNativeView.js#L38
Upstreamed the latest MyNativeView.js from https://github.com/facebook/react-native/blob/main/packages/rn-tester/NativeComponentExample/js/MyNativeView.js
Changelog
[macOS][Fabric] - Fix NativeComponentExample to render correctly
Test Plan
Before
https://user-images.githubusercontent.com/96719/213311648-2b5641cd-7754-4048-8808-2ee6f5f061e9.mp4
After
https://user-images.githubusercontent.com/96719/213311320-097131ab-1978-4376-951f-76fc20c70460.mp4

Thanks for the fix! When I had the code looking like this before, it failed the newly added Fabric CI. I'm curious to see if it will pass now :)

Oops, this broke Paper on iOS 😓

I'll fix it so that we only render when fabric is available

My understanding is that Turbomodules is somewhat broken at this point in time and the actual paper implementation for RNTMyNativeView isn't getting linked :/.
I'm hoping this will be resolved after we sync to 0.71

Found a temporary fix that won't throw error when trying to render on Paper.
Paper

Fabric

authors: Saadnajmi shwanton

repo: microsoft/react-native-macos


text:
Create guide for how to publish to store

We have great docs covering how to create and app and get it running/editable/debuggable. But how do you ship it?
There are UWP docs on this of course, but they are surprisingly hard to find if you don't know what you're looking for (for example you're coming in as a RN/mobile/web developer, as many of our RNW customers are). And if you do find them... they focus a lot on exceptions to the process, not the standard flow. As someone who had published to the store before... I found it all very disorienting.
We should provide a guide that takes in our developer's perspective and helps them through this process.

Duplicate of #62

repo: microsoft/react-native-windows-samples


text:
Revert Conan changes.

Reverts #319 until the SPDX changes are removed from the Component Detection repo. As of now Conan changes are in v3.7.0 of Component Detection which contains some unwanted changes to how they detect SPDX files.

@sebasgomez238 Could you please share more information on SPDX changes on component detection library which is referred here in this PR?

repo: microsoft/sbom-tool


text:
.Net: generating parameter schema with enum types

when a function is using a parameter that is an Enum like:
    public enum MyEnum
    {
        ValueA,
        ValueB
    }

the schema produced by semantic kernel serialising KernelParameterMetadata will be :
{
 "enum":["ValueA","ValueB"]
}

according to the OpenAi documentation here enums look like this:

in  that case a parameter of type MyEnum should have been represented in the schema as:
{
 "type": "string"
 "enum":["ValueA","ValueB"]
}

in .NET Enums cannot extend string (like in typescript) in the dotnet iinteractive we provided a specific handling of Enum to align with OpenAI docs.
What are you thoughts?

@colombod I'd like to find out more about what you are doing in .Net interactive to see can we adopt the same approach

@colombod I'd like to find out more about what you are doing in .Net interactive to see can we adopt the same approach

To be honest I think your approach is correct from JSonSchema point of you. might be worth having some tests to cover enum in tool signature and also some integration test to verify that gpt calls will honour that. What do you think?

authors: colombod markwallace-microsoft

repo: microsoft/semantic-kernel


text:
[JS] Lightbot - Investigate and fix bug

Also fix #922
Initial message: 'What is the status of the lights'
[onTurnError] unhandled error: Error: The JSON returned had errors. Apply these fixes:
change the "instance.commands[1].type" property to be one of these values: DO,SAY
Error: The JSON returned had errors. Apply these fixes:

Tried to replicate bug, but bot is working fine. Looked to be most likely an AOAI problem.

repo: microsoft/teams-ai


text:
Strikethrough expands when zooming in or out

Windows Terminal version (or Windows build number)
1.4.3243.0
Other Software
should not matter
Steps to reproduce

Print strikethrough text (you can use python -c print('\x1B[9mstrikethrough\x1B[0m') or echo -e '\x1B[9mstrikethrough\x1B[0m' in bash)
Zoom in or out

Expected Behavior
The strikethrough shouldn't have expanded.
Actual Behavior

edit: this also seems to happen with underlines (\x1B[4m), double underlines (\x1B[21m), overlines (\x1B[53m), this that I don't know how to name (\x1B[7m) and hyperlinks (\x1B]8;;https://microsoft.com\x1B\\hyperlink\x1B]8;;\x1B\\)

Alas, this is one of the oldest issues in the console codebase!
/dup #32

authors: vulpinefoxxo zadjii-msft

repo: microsoft/terminal


text:
Using 'Toggle command palette' action from palette doesn't close palette

Windows Terminal version (or Windows build number)
1.9.1445.0
Other Software
No response
Steps to reproduce

Open the command palette
Search for: Toggle command palette
Hit enter to use that action

Expected Behavior
I'd expect the palette to close when using the Toggle command palette item in the palette. This is also what happens when you use the toggle shortcut while the palette is open.
Actual Behavior
The entered search text is cleared, and the palette remains open. The code responsible seems to be:
https://github.com/microsoft/terminal/blob/b2c2a4c159b7e918ea8aabc9e22f83d58a479002/src/cascadia/TerminalApp/AppActionHandlers.cpp#L369-L380
When the command palette is open, and you use the shortcut to toggle its visibility, CommandPalette().Visibility() correctly returns Visibility::Visible and the palette is closed. On the other hand, when using the entry in the palette, CommandPalette().Visibility() returns Visibility::Collapsed, which causes the palette to remain open.

Huh. Good catch!

I'll see if I can track down what's causing this.

authors: KnapSac zadjii-msft


text:
Secondary language always shown as GUI language even with language selection

Windows Terminal version (or Windows build number)
1.11.2421.0
Other Software
No response
Steps to reproduce
Install primary language Dutch - NLD
Install secondary language Russian - Rus
Start Terminal - Gui is in Russian, Select Dutch, no change.
Remove Russian, Gui in English again
Expected Behavior
Expected with primary language Dutch to have English UI
MAYBE: Because Dutch is not a standard language in Windows Terminal and Russian is, it skips Dutch and goes to Russian because that's the next language on the system that IS recognized.
Actual Behavior
Primary language Dutch gave Russian UI.

Sounds to me like this is just a variation on #10667. The Terminal will pick the first language that it's actually available in, and that set might need to be bigger.
You can manually set the language with the following setting

but that only lets you pick from the languages the Terminal is actually available in.
/dup #10667

authors: Truckland zadjii-msft


text:
If setting change requires restart, notify user

Summary
If a setting is changed that requires the Terminal to restart in order for the setting change to take effect, consider displaying a message to the user prompting them to restart their Terminal.
Also, consider making it easier to identify settings that require a Terminal restart if changed, perhaps using an emoji/visual glyph of some kind?
Related
This feature is related to #1252 suggesting that the Terminal should support restart/resume.
Proposed technical implementation details (optional)

This just needs some good good settings UI.

This needs to be parented to some good good settings UI deliverable.

authors: DHowett-MSFT bitcrazed