Move all public methods to the api namespace

What do these changes do?
Move public methods to the api namespace.
Are there changes in behavior for the user?
The top level module no longer provides the cached_property and under_cached_property methods.  They have moved to propcache.api.cached_property and propcache.api.under_cached_property respectively.
Related issue number
#18

Is there prior art for this approach? Personally, my first thought seeing something like this, is that's it's not pythonic and just adds verbosity to imports (which then makes me start to think of Java imports shudders).

I had initially balked a bit to this approach but I’ve also been in a situation where you end up forced to have code loaded that isn’t needed anymore to maintain compatibility because it’s imported into init.py so I can see why it would be advantageous to avoid that problem in the future

I’ve also been in a situation where you end up forced to have code loaded that isn’t needed anymore to maintain compatibility because it’s imported into __init__.py

Arguably, you could use getattr for that (e.g. how gunicorn workers are imported in aiohttp.web). But, this looks like a single module library, so doesn't look like it'd benefit anyway. And for something like aiohttp, you'd decrease the amount of code loaded into memory, but sacrifice usability when every user now needs 20 lines of aiohttp imports in each of their modules.

I agree it’s very unlikely to be a problem in the future, we could revert the change since 1.0.0 failed to ship

I looked at what the change will be downstream to Home Assistant, and they’ll be 86 new line changes to add .api which I agree isn’t great

I don't mind too much, it doesn't make much difference for this library. Mainly just wanted to know @webknjaz's arguments before this becomes accepted as aio-libs standard practice, as I'm struggling to see any benefits. Whether you want to revert or not, |I'll leave with you.

Since 1.0.0 failed to upload to pypi anyways, I pulled it down from GitHub while we decide this so no downstream is affected

I don't mind too much, it doesn't make much difference for this library. Mainly just wanted to know @webknjaz's arguments before this becomes accepted as aio-libs standard practice, as I'm struggling to see any benefits. Whether you want to revert or not, |I'll leave with you.

I'll hold on doing anything until webknjaz has a chance to respond.

@Dreamsorcerer one place this is coming from WPS412 that I know you asked about in the past. The documented rationale there is incomplete, though.
It's more or less obvious that if you import a.b but a also imports c internally, you'll get a side effect of more things loaded than you expected. This is a simple case but as projects grow, I observed a lot of human error happening, leading to unobvious import loops (we have tests in aiohttp to fight this, of course, but does this mean that every project has to implement a test scanning the disk for Python modules and attempting to import them in isolation to see if something blows up? Probably not).
I'm seeing cases where projects cannot use regular module-level imports and stick them into functions that are called later because too many things end up being loaded and initialized on import depending on other imports to happen first to the point of being both ridiculous and really difficult to fix.
That's outside aio-libs, here we tend to have more robust structure, but that's at a cost of higher cognitive load. Implementing a good layout is something that may reduce said load and human factor-caused problems, virtually for free. I don't mind that the callers might need to type in 4 more bytes in their imports.
One case that came to my mind within aiohttp, for example, was emitting deprecations and using an in-project class for that, which effectively made it impossible to suppress because referencing the exception in the warning suppression settings meant that in needed to be imported and the import itself was causing the deprecation warning. It's a very real problem that we need to be conscious about.
The libs still need to expose some high-level API, it just doesn't have to be in a way that would cause such side effects. And I figured that api.py might be such a place.
As for keeping some imports in place, even temporarily, __getattr__() is a good solution, it could handle deprecations too, if needed.
I think this is a good layout to have but of course, we're not going to break existing APIs. Still, for new projects, I believe this is what they should be modeled after. You mentioned aiohttp probably thinking that I'd suggest changing it too (I'm guessing). But no, aiohttp already has some of such namespacing through such proxy entry points (like aiohttp.web). The only thing to mention is that maybe we need to pay more attention to such entry point modules regardless of what they are called (.api seems like a good name for smaller libs and could be universal, unlike bigger frameworks). And perhaps move more things under __getattr__() in some cases.
I think I saw a blog post or something on this years ago but can't find it. But the approach is good, and Anthony agrees that the linting rule is useful. I've been practicing this where I can for some time, and I'm pretty happy about what it brings to the table.

@Dreamsorcerer one place this is coming from WPS412 that I know you asked about in the past. The documented rationale there is incomplete, though.

But, the response there was to explicitly allow imports in libraries:

And with libraries it is totally fine to tweak imports. Libraries are going to use --i-dont-controll-code anyway.

The libs still need to expose some high-level API, it just doesn't have to be in a way that would cause such side effects. And I figured that api.py might be such a place.

I guess I'm not really seeing how it's any different if everything is imported into api.py instead of __init__.py...
The traceback for the mypy issue above doesn't even have __init__.py anywhere in it, the circular import doesn't seem like it'd be impacted by following this rule.

I guess I'm not really seeing how it's any different if everything is imported into api.py instead of __init__.py...

You're right. This aspect is not very different. But I think that it's still a good layout, since it enables us to expose things that are used more rarely in a more granular way.


T265: A safe way to provoke an exception while attempting to save a map

Required Info

Camera Model
T265

Firmware Version
<IP_ADDRESS>6

Operating System & Version
macOS Catalina

Kernel Version (Linux Only)

Platform

SDK Version
2.31.0, development, #5213 applied

Language
py

Segment
others

Issue Description
It is pretty simple:  Try to save a map early after brand new start. Happens regardless of pose tracking confidence. The reproducible exception is
null pointer passed for argument "buffer"
I think there could be a better way to express the inability to save a map, e.g. if there are not enough map points internally. A None for output_map would do it :)
The exception is the answer to this call:
           output_map = self.__pose_sensor.export_localization_map()

Problem seems to be fixed with fw <IP_ADDRESS>8, #5504

Nope, it is still there :) Most likely an easy to fix thingy...

Hi @neilyoung
Are you still experiencing this on the latest T265 f/w (v. <IP_ADDRESS>1)?
Thanks

Nvm...was able to reproduce by forcing an error on latest, so disregard last question.
That said,...Thanks for highlighting this. I will pass this enhancement request on to our dev team.
Thanks again.

@RealSenseSupport Hmm. Tried right now with 2.39.0. Wasn't able to reproduce so far. Firmware is <IP_ADDRESS>1

Hi @neilyoung
That's good to hear. The way I forced the error would never occur in practice. :)
And it has been a while so hopefully this has been improved since.
That said, at this time we have moved our focus to our next generation of products and consequentially will not be addressing new T265 related enhancements.
Thanks


:art: Replace tables and add styles (#705)

close #705

Summary by CodeRabbit

New Features

Introduced isEditable prop to various input and select components for editability control.
Added tooltip functionality in TabItemWrapper component.
Implemented new table structures using svelte-headless-table.
Added new button and tooltip components.

Bug Fixes

Fixed link title issue in the navigation bar.

Documentation

Updated CONTRIBUTING.md with clearer frontend technology details.

Style

Reorganized CSS variables and added theme color variables.

Chores

Updated dependencies and ESLint configurations.
Added new configuration files for Svelte project settings and UI components.

Svelte 5の新機能を利用すれば、実現できるかもしれません


Bugfix/tsv output

Coverage increased (+0.6%) to 99.415% when pulling e13841e5f376c5f07de3eecd24f0415b5709d653 on bugfix/tsv-output into cb727fa12ec3b2e20fef3ec8ee062c0d07604ac7 on develop.


running on windows  error

I am trying enduro on windows server 2012 but I got this error:
[09:54:40 |    +10.4] developer start
Juicebox is not set up
Unhandled rejection TypeError: Cannot read property '1' of null
    at Array.slice.call.map (C:\Users\adm-213100168\AppData\Roaming\npm\node_modules\enduro\libs\flat_utilities\enduro_h
elpers.js:52:110)
    at Array.map (native)
    at enduro_helpers.ensure_directory_existence (C:\Users\adm-213100168\AppData\Roaming\npm\node_modules\enduro\libs\fl
at_utilities\enduro_helpers.js:52:53)
    at babel.get_cultures.then (C:\Users\adm-213100168\AppData\Roaming\npm\node_modules\enduro\libs\pregenerator\pregene
rator.js:70:27)
    at tryCatcher (C:\Users\adm-213100168\AppData\Roaming\npm\node_modules\enduro\node_modules\bluebird\js\release\util.
js:16:23)
    at Promise._settlePromiseFromHandler (C:\Users\adm-213100168\AppData\Roaming\npm\node_modules\enduro\node_modules\bl
uebird\js\release\promise.js:510:31)
    at Promise._settlePromise (C:\Users\adm-213100168\AppData\Roaming\npm\node_modules\enduro\node_modules\bluebird\js\r
elease\promise.js:567:18)
    at Promise._settlePromiseCtx (C:\Users\adm-213100168\AppData\Roaming\npm\node_modules\enduro\node_modules\bluebird\j
s\release\promise.js:604:10)
    at Async._drainQueue (C:\Users\adm-213100168\AppData\Roaming\npm\node_modules\enduro\node_modules\bluebird\js\releas
e\async.js:143:12)
    at Async._drainQueues (C:\Users\adm-213100168\AppData\Roaming\npm\node_modules\enduro\node_modules\bluebird\js\relea
se\async.js:148:10)
    at Immediate.Async.drainQueues (C:\Users\adm-213100168\AppData\Roaming\npm\node_modules\enduro\node_modules\bluebird
\js\release\async.js:17:14)
    at runCallback (timers.js:637:20)
    at tryOnImmediate (timers.js:610:5)
    at processImmediate [as _immediateCallback] (timers.js:582:5)

C:\Users\adm-213100168\AppData\Roaming\npm\node_modules\enduro\libs\flat_utilities\enduro_helpers.js:52
        file_paths = Array.prototype.slice.call(arguments).map((file_path) => { return file_path.match(/^(.*)\/.*$/)[1]
})
                                                                                ^

TypeError: Cannot read property '1' of null
    at Array.slice.call.map (C:\Users\adm-213100168\AppData\Roaming\npm\node_modules\enduro\libs\flat_utilities\enduro_h
elpers.js:52:110)
    at Array.map (native)
    at enduro_helpers.ensure_directory_existence (C:\Users\adm-213100168\AppData\Roaming\npm\node_modules\enduro\libs\fl
at_utilities\enduro_helpers.js:52:53)
    at read (C:\Users\adm-213100168\AppData\Roaming\npm\node_modules\enduro\libs\pregenerator\pregenerator.js:51:19)
    at tryToString (fs.js:425:3)
    at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:412:12)

what is wrong?
Thanks

Thanks for the report. Let me investigate.

Hey @assiomatica , which version of node.js are you running? You can find out by running node -v

Hi,
thanks for answer
v7.1.0
first was older , and I have updated to the last but nothing change, I got the error the same.
BTW Nice work , congratulation, I really like it! I have tested on osx and works perfectly.
A.

Wow, thank you.
Just update enduro (npm i enduro -g) and it should work smoothly. Issue was with the path delimiter.

btw, @assiomatica could you confirm this was solved?

Thanks but seems the patch is not available on npm
I have removed and reinstalled enduro but the error is still there.
may be I have to install a branch?
Thanks
Antonio

Hi
I have installed it from git directly
npm install git+https://github.com/kiskadigitalmedia/Enduro.git -g
now is working well
thank you for support.
Antonio

You are the best, thank you. Feel free to post more stuff if you find something.


[No QA] Make BootSplash have better parity on web/desktop

Details
react-native-bootsplash returns a void promise for hide, show, and a promise that resolves to one of "visible", "hidden", "transitioning" for getVisibilityStatus. This PR just improves the parity in our web shim so that we don't have another problem like the one solved by this PR
Fixed Issues
n/a
Tests
Just do some basic regression testing to make sure the app still runs okay and doesn't have any console warnings.
Tested On

[x] Web
[ ] Mobile Web
[x] Desktop
[ ] iOS
[ ] Android

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging in version: 1.0.34-1🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Migrations unapply in the wrong order

Apply-Migration 0 unapplies in the wrong order. It starts with the oldest migration. The first should be last and the last should be first.

Looks like I missed a call to .OrderByDescending(m => m.Id) on line 131 of Migrator.cs.


[Feature]: add Support for Embed Messages for the Guard RateLimit

Feature?
please add the possibility to react with an Embed instead of an normal Message.
With this "Feature" we also have the possibility to send Ephemeral messages...
Package
@discordx/utilities

You can send ephemeral messages as simple message without embeds.
Embeds are tricky case, you can't guarantee that the bot has embed permission in that channel. Plain text messages are the best. Even I wanted the ability to send embeds for rate limits warnings but i can't trust the user server permissions.

But I agree about the possibility of having rate limit warning as ephemeral.

@drph4nt0m lol, forgot about that xD. For me it would be also enough to send limit warnings as ephemeral.
I will adjust the title and description accordingly.

@VictoriqueMoe

I'll do this when I can, currently I'm very unwell, but as soon as I'm better. I'll add it


ListSubscriptions{,Details}: include names prefixed docker-subscription-

The current subscriptions have a "Name" field set to "docker-subscription-"
Signed-off-by: David Scott<EMAIL_ADDRESS>

closing in favor of https://github.com/docker/licensing/pull/24


Develop detection tool

The primary job of this tool is to detect orca in passive acoustic datasets comprised of a pipeline consisting of stages to: 1) preprocess and standardize input data, 2) apply a model to identify the presence of orca within the dataset, 3) apply additional models to extract additional information about the orca call including signal type and pod, and 4) create a summary of the results of the pipeline indicating presence of orca along with time and any additional information.
The issue here is to apply the model(s) developed in  #2 to create stage 2 of the pipeline.

Developed with the PR #24


Server updates

Getting the server ready for the hackathon, etc.

I've refactored the server quite a bit to add some functionality. I'll add better deploy options (Docker, Heroku button, etc) in a follow-up.


Cant update conda

Problem Description
What steps will reproduce the problem?

I am trying to update conda using anaconda promt : conda update conda. It shows all the packages being downloaded, verified but when it starts to execute - this error occurs.
ERROR conda.core.link:_execute(502): An error occurred while uninstalling package 'defaults::anaconda-5.2.0-py36_3'.
PermissionError(13, 'Access is denied')
it happens on both Mac and Windows. I have full admin rights and I have installed the Anaconda for all users.
Any ideas what can I do here?
thank you
Stan

This is not a problem with Spyder but with Anaconda, so please report it here:
https://github.com/ContinuumIO/anaconda-issues/issues/
The only thing that occurs to me is that you should launch the Anaconda Prompt with admin privileges (on Windows) and run conda update spyder with sudo on macOS.
However, if you're the only user in your computer, you should remove your previous installation and reinstall Anaconda for your user only.

However, if you're the only user in your computer, you should remove your previous installation and reinstall Anaconda for your user only.

That's the recommended installation method because it's less prone to errors.

regarding Windows update - you are right. right click on Anaconda Prompt and run as admin - worked out.


Can you add support for Alienware X15 R1

Hello, Can you add support for the following device:
Device Name: Alienware X15 R1
Device Website: https://www.dell.com/en-us/shop/gaming-laptops/alienware-x15-gaming-laptop/spd/alienware-x15-r1-laptop
SDK AlienFX in GitHub : https://github.com/T-Troll/alienfx-tools
DxDiag.txt
USB Device Tree.txt

As far as i see into source code, Aurora should support Alienware trough Dell LightFX wrapper.
Try to locate lightfx.dll inside AWCC folder and copy it into Aurora one.

I check.
Now Aurora uses old Gurjot95's low-level library code, so only v1 and v3 (ancient) devices supported. v2 not supported die to the bug in his code (they have 4-bit per color, not 8).
2devs: can you please uncomment this:
    /*
    LFXInit();
    */```

and this:
```/*
                for (uint devIndex = 0; devIndex < numDevices; devIndex++) {
                    uint descSize = 255;
                    char* description = new char[descSize];
                    byte devType = 0;
                    LightFXSDK.LFX_GetDeviceDescription(devIndex, description, descSize, &devType);
                } */```

into Devices/LightFX/LightFX.cs?

It will be enough to use both Dell's lib or my emulator.

Thank you @T-Troll  for helping me out, I found those files

zip file for AlienFX SDK folder
AlienFX SDK.zip


MM-48773: fixes position of ACK users popover

Summary
The fallback position for the post acknowledgements users popover should be bottom-start,
so if the popover doesn't fit at top-start, it would be placed at bottom-start,
and if it doesn't fit there, or in right, it will use the best-fit position.
Fixes both RHS, and Center Channel.
Ticket Link
https://mattermost.atlassian.net/browse/MM-48773
Screenshots

before
after

Release Note
NONE

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard

Cherry pick is scheduled.


loader: make one variable plural and clean up a comment’s wrapping

Commit message first line should be less than 50 characters.  I think the next lines can be 70.  I'll merge it once you clean up the commit, perhaps something like:
   loader: clean up comment and variable

   Make one variable's name plural to clarify that it contains more than one
   layer.  Also, clean up a comment that wrapped prematurely to the next line.

I don't know if I can change the merge request titles from inside GitHub which is why I'm requesting you do it.  If it's too much, I can change it next week.

Thanks for cleaning it up!


new gcc result

gcc gives more accurate warnings in one test

Hmm, it looks like the same warning is duplicated twice.  Can you check the GCC outputs before they are processed by csgrep?

The output from GCC looks correct.
\cc @davidmalcolm any idea why it is duplicated?

Sounds like a possible GCC bug; are you able to send me a reproducer as preprocessed source? (compile it with -E; see https://gcc.gnu.org/wiki/A_guide_to_testcase_reduction)

@davidmalcolm  I used check script [ $(gcc -fanalyzer testcase.i 2>&1 | grep -c "warning: allocated buffer size is not a multiple of the pointee's size" ) -ne 2 ] && exit 1 || exit 0
interesting that if you remove useless line int length = 9;, the warning is not duplicated anymore
--- /tmp/tmp.8JwjaC5XGI/testcase.i ---
main() {
  int length = 9;
  int *arr = __builtin_alloca(9);
}

[mockbuilder@e304d2b29b30 tmp.8JwjaC5XGI]$ gcc -fanalyzer testcase.i
testcase.i:1:1: warning: return type defaults to ‘int’ [-Wimplicit-int]
    1 | main() {
      | ^~~~
testcase.i: In function ‘main’:
testcase.i:3:14: warning: allocated buffer size is not a multiple of the pointee's size [CWE-131] [-Wanalyzer-allocation-size]
    3 |   int *arr = __builtin_alloca(9);
      |              ^~~~~~~~~~~~~~~~~~~
  ‘main’: event 1
    |
    |    3 |   int *arr = __builtin_alloca(9);
    |      |              ^~~~~~~~~~~~~~~~~~~
    |      |              |
    |      |              (1) allocated 9 bytes and assigned to ‘int *’ here; ‘sizeof (int)’ is ‘4’
    |
testcase.i:3:14: warning: allocated buffer size is not a multiple of the pointee's size [CWE-131] [-Wanalyzer-allocation-size]
  ‘main’: events 1-2
    |
    |    3 |   int *arr = __builtin_alloca(9);
    |      |              ^~~~~~~~~~~~~~~~~~~
    |      |              |
    |      |              (1) allocated 9 bytes here
    |      |              (2) assigned to ‘int *’ here; ‘sizeof (int)’ is ‘4’```


Defect: Browser refresh handling

Current functionality:
When a browser refresh occurs:
The browser always redirects to /intro.
Expected functionality:
When a browser refresh occurs:
If the last visited page was protected:
Check if user session is valid, if not redirect to login. If it is valid, navigate to last visited page
(Note: This looks like pseudocode, but it doesn't and likely won't be implemented this way)

I'm unable to replicate this behavior with /#/faq and /#/home. Also we want the default view to be home not intro (#62). Someone visiting https://loungeapp.co/ shouldn't be redirected to a login screen.

@tlr3552 Refreshing the page from /#/posting/view/-1 will result in navigation to the invalid route of /#/posting/view/.  As far as I know, we have not specified how this invalid route should be handled, but the automatic navigation would seem related to this bug.


Add upgrade suggestions

Detect when an older version of the extension is in use and suggest an upgrade, similar to gh.

The gh CLI team is working on bringing this to all extensions: https://github.com/cli/cli/issues/8183

Example of how this is implemented in gh-copilot: https://github.com/github/gh-copilot-internal/blob/188a3eada01526548fd530fffebfab9a3308ba55/cmd/root.go#L69-L99


Status

According to these two threads:
https://github.com/thomseddon/cakephp-oauth-server/pull/35
https://github.com/thomseddon/cakephp-oauth-server/issues/50
The plugin was being re-factored to handle the implicit grant type.. How has this progressed?
Thanks

Thom,
I maybe spending time with this plugin again soon and try to pick up where I left off.  Can we still use thomseddon/oauth2-php or we need a new package?

No that's fine, I never finished the refactored implementation of this module using the bshaffer library....if possible that would probably be the preferred way to go, oauth2-php is long dead

@rchavik This sounds good :)

Another alternative would be https://github.com/thephpleague/oauth2-server.
@iamdriz, but even that does not support Implicit Grant: http://oauth2.thephpleague.com/authorization-server/which-grant/#implicit-grant-section-42httptoolsietforghtmlrfc6749section-42

Yeah that may be a better option!

@rchavik Ah :-1: does it not? Why hasn't anyone implemented this grant type? As if that's the one wanted for most mobile applications to talk to an API with authentication I'd of thought it'd be the most used and implemented.

@rchavik @thomseddon What's happening with this plugin then? Thanks :)

Well, I was working on the client side of things.  Turns out that didn't require any changes on this package.
If you're after Implicit Grant, then I'm not sure. I'm not knowledgeable enough to implement it.

I haven't work on Cake for quite a long time and so haven't needed to implement this - I'm going to add an unmaintained flag to the readme


OS Independance for the Popen tar commands

Error:
[WinError 2] is reported from windows.
content:
Popen subprocess doesn't work properly using the tar arguments from the compiled arguments on Windows resulting in the above error.

Thanks much for the issue... I do not have ready accesss to a Windows dev environment but if you are willing to contribute a PR I'm delighted to include...

OK, I appreciate your taking the time to look at it. When I can, I will try to grab a Windows machine and see if I can help.


Inline tables with inf/nan as values fail to parse

In a regular table:
>>> toml.loads("[x.y]\nz = nan")
{'x': {'y': {'z': nan}}}

As expected.
But in an inline table:
>>> toml.loads("[x.y]\nz = 'nan'")
Traceback (most recent call last):
  File "toml/decoder.py", line 511, in loads
    ret = decoder.load_line(line, currentlevel, multikey,
  File "toml/decoder.py", line 778, in load_line
    value, vtype = self.load_value(pair[1], strictly_valid)
  File "toml/decoder.py", line 883, in load_value
    self.load_inline_object(v, inline_object)
  File "toml/decoder.py", line 677, in load_inline_object
    raise ValueError("Invalid inline table value encountered")
ValueError: Invalid inline table value encountered

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "toml/decoder.py", line 514, in loads
    raise TomlDecodeError(str(err), original, pos)
toml.decoder.TomlDecodeError: Invalid inline table value encountered (line 2 column 1 char 4)

Whereas I would have expected the same result as in the regular table case.

Note that inf, nan, +inf and +nan fail, but -inf and -nan succeed.


CI build of azps-7.1.0

Docs Build status updates of commit 1b4a9af:
:warning: Validation status: warnings

File
Status
Preview URL
Details

azps-7.1.0/Az.RedisCache/New-AzRedisCache.md
:warning:Warning
View (azps-7.1.0)
Details

azps-7.1.0/toc.yml
:warning:Warning
View (azps-7.1.0)
Details

azps-7.1.0/Az.ADDomainServices/Get-AzADDomainService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.ADDomainServices/Remove-AzADDomainService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.ADDomainServices/Update-AzADDomainService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Aks/Get-AzAksNodePoolUpgradeProfile.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Aks/Get-AzAksUpgradeProfile.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Aks/Start-AzAksCluster.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Aks/Stop-AzAksCluster.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.AppConfiguration/Get-AzAppConfigurationStore.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.AppConfiguration/New-AzAppConfigurationStoreKey.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.AppConfiguration/Remove-AzAppConfigurationStore.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.AppConfiguration/Update-AzAppConfigurationStore.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.ApplicationInsights/Get-AzApplicationInsightsWebTest.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.ApplicationInsights/Remove-AzApplicationInsightsWebTest.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.ApplicationInsights/Update-AzApplicationInsightsWebTestTag.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Automation/Start-AzAutomationDscCompilationJob.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.BotService/Get-AzBotService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.BotService/Remove-AzBotService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.BotService/Update-AzBotService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Get-AzCloudService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Get-AzCloudServiceNetworkInterfaces.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Get-AzCloudServiceOSFamily.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Get-AzCloudServiceOSVersion.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Get-AzCloudServicePublicIPAddress.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Get-AzCloudServiceRoleInstance.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Invoke-AzCloudServiceRebuild.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Invoke-AzCloudServiceReimage.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Invoke-AzCloudServiceRoleInstanceRebuild.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Invoke-AzCloudServiceRoleInstanceReimage.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/New-AzCloudService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Remove-AzCloudService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Remove-AzCloudServiceRoleInstance.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Restart-AzCloudService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Restart-AzCloudServiceRoleInstance.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Start-AzCloudService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Stop-AzCloudService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Switch-AzCloudService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.CloudService/Update-AzCloudService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Communication/Get-AzCommunicationService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Communication/New-AzCommunicationServiceKey.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Communication/Remove-AzCommunicationService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Communication/Set-AzCommunicationServiceNotificationHub.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Communication/Update-AzCommunicationService.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Compute/Get-AzGalleryApplication.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Compute/Get-AzGalleryApplicationVersion.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Compute/Get-AzVMRunCommand.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Compute/Get-AzVmssVMRunCommand.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Compute/New-AzVmss.md
:bulb:Suggestion
View (azps-7.1.0)
Details

azps-7.1.0/Az.Compute/Remove-AzGalleryApplication.md
:bulb:Suggestion
View (azps-7.1.0)
Details

This comment lists only the first 50 files in the pull request.

azps-7.1.0/Az.RedisCache/New-AzRedisCache.md

Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: './../../../Network/Network/help/New-AzVirtualNetwork.md'.

azps-7.1.0/toc.yml

Line 4080, Column 11: [Warning: bookmark-not-found] Cannot find bookmark '#import/export' in 'azps-7.1.0/Az.ImportExport/index.yml', did you mean '#import-export'?

azps-7.1.0/Az.ADDomainServices/Get-AzADDomainService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAdDomainServicesIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.ADDomainServices/Remove-AzADDomainService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAdDomainServicesIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.ADDomainServices/Update-AzADDomainService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAdDomainServicesIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Aks/Get-AzAksNodePoolUpgradeProfile.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAksIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Aks/Get-AzAksUpgradeProfile.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAksIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Aks/Start-AzAksCluster.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAksIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Aks/Stop-AzAksCluster.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAksIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.AppConfiguration/Get-AzAppConfigurationStore.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAppConfigurationIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.AppConfiguration/New-AzAppConfigurationStoreKey.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAppConfigurationIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.AppConfiguration/Remove-AzAppConfigurationStore.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAppConfigurationIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.AppConfiguration/Update-AzAppConfigurationStore.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAppConfigurationIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.ApplicationInsights/Get-AzApplicationInsightsWebTest.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IApplicationInsightsIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.ApplicationInsights/Remove-AzApplicationInsightsWebTest.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IApplicationInsightsIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.ApplicationInsights/Update-AzApplicationInsightsWebTestTag.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IApplicationInsightsIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Automation/Start-AzAutomationDscCompilationJob.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'NodeName' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.BotService/Get-AzBotService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IBotServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.BotService/Remove-AzBotService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IBotServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.BotService/Update-AzBotService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IBotServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Get-AzCloudService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Get-AzCloudServiceNetworkInterfaces.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'CloudService' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Get-AzCloudServiceOSFamily.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Get-AzCloudServiceOSVersion.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Get-AzCloudServicePublicIPAddress.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'CloudService' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Get-AzCloudServiceRoleInstance.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Invoke-AzCloudServiceRebuild.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Invoke-AzCloudServiceReimage.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Invoke-AzCloudServiceRoleInstanceRebuild.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Invoke-AzCloudServiceRoleInstanceReimage.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/New-AzCloudService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceExtensionProfile' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceNetworkProfile' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceOSProfile' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceRoleProfile' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Remove-AzCloudService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Remove-AzCloudServiceRoleInstance.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Restart-AzCloudService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Restart-AzCloudServiceRoleInstance.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Start-AzCloudService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Stop-AzCloudService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Switch-AzCloudService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'CloudService' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.CloudService/Update-AzCloudService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudService' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Communication/Get-AzCommunicationService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICommunicationIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Communication/New-AzCommunicationServiceKey.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICommunicationIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IRegenerateKeyParameters' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Communication/Remove-AzCommunicationService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICommunicationIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Communication/Set-AzCommunicationServiceNotificationHub.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ILinkNotificationHubParameters' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Communication/Update-AzCommunicationService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICommunicationIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Compute/Get-AzGalleryApplication.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IComputeIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MajorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MinorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'Patch' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MajorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MinorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'Patch' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Compute/Get-AzGalleryApplicationVersion.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IComputeIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MajorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MinorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'Patch' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MajorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MinorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'Patch' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Compute/Get-AzVMRunCommand.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IComputeIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MajorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MinorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'Patch' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MajorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MinorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'Patch' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Compute/Get-AzVmssVMRunCommand.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IComputeIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MajorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MinorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'Patch' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MajorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MinorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'Patch' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Compute/New-AzVmss.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'DomainNameLabel' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'Location' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ScaleSetName' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ResourceGroupName' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.1.0/Az.Compute/Remove-AzGalleryApplication.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IComputeIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MajorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MinorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'Patch' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MajorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'MinorVersion' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.
Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'Patch' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Cannot read data from specific XLS file

Dear authors,
i have to read data from a stream of an XLS file like this: example.XLS
I am using ExcelDataReader 3.6.0 to create a DataTable out of a stream:
private static DataTable GetData(Microsoft.AspNetCore.Http.IFormFile someFile)
{
   using Stream stream = someFile.OpenReadStream();
   IExcelDataReader? reader = ExcelReaderFactory.CreateBinaryReader(stream);
   DataSet dataSet = reader.AsDataSet();
   DataTable dataTable = dataSet.Tables.Cast<DataTable>().First();

   return dataTable;
}

In this way the resulting DataTable is empty. I have also tried to use ExcelReaderFactory.CreateReader(stream); but received the same result.
After opening the XLS file with Microsoft Office 365 (Version 2108 - Build 14326.21200), hitting the save button and closing it, I receive a DataTable with all the cells of the excel file. Here is the working file after saving (without changing anything): example_after_saving.XLS
Converting the file to XLSX (via Excel) without changing the content of the file works as well.
Unfortunately I am not able to change the incoming XLS file I have to process. I also thought of using e.g. Microsoft.Office.Interop.Excel to open the file and save it as XLSX. But this does not seem to work with streams and I would like to keep the current logic of my program and avoid saving files to disk.
Maybe you have got a suggestion for me on how to process these kind of XLS files.
Thank you!
Patrik

Thank you for the sample file.
Looks like the worksheet is missing the expected BOF record. Possibly we can ignore this.

Thank you very much for solving my problem!
Do you already know when version 3.7 is going to be published?

No, but I'll try to do a new prerelease soon.


新的docker镜像无法启动

报错信息如下：
novel    | [2018-12-14 13:31:49,682] 12-INFO log::info():l15: GET: http://r.qidian.com/?chn=1 novel    | [2018-12-14 13:31:49,974] 12-ERROR log::exception():l30: there are no users authenticated novel    | Traceback (most recent call last): novel    |   File "/opt/owllook/spiders/qidian_ranking.py", line 93, in save novel    |     upsert=True) novel    |   File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 56, in run novel    |     result = self.fn(*self.args, **self.kwargs) novel    |   File "/root/.local/share/virtualenvs/opt-zvmYt2-H/lib/python3.6/site-packages/pymongo/collection.py", line 995, in update_one novel    |     session=session), novel    |   File "/root/.local/share/virtualenvs/opt-zvmYt2-H/lib/python3.6/site-packages/pymongo/collection.py", line 851, in _update_retryable novel    |     _update, session) novel    |   File "/root/.local/share/virtualenvs/opt-zvmYt2-H/lib/python3.6/site-packages/pymongo/mongo_client.py", line 1243, in _retryable_write novel    |     return self._retry_with_session(retryable, func, s, None) novel    |   File "/root/.local/share/virtualenvs/opt-zvmYt2-H/lib/python3.6/site-packages/pymongo/mongo_client.py", line 1196, in _retry_with_session novel    |     return func(session, sock_info, retryable) novel    |   File "/root/.local/share/virtualenvs/opt-zvmYt2-H/lib/python3.6/site-packages/pymongo/collection.py", line 847, in _update novel    |     retryable_write=retryable_write) novel    |   File "/root/.local/share/virtualenvs/opt-zvmYt2-H/lib/python3.6/site-packages/pymongo/collection.py", line 817, in _update novel    |     retryable_write=retryable_write).copy() novel    |   File "/root/.local/share/virtualenvs/opt-zvmYt2-H/lib/python3.6/site-packages/pymongo/pool.py", line 579, in command novel    |     unacknowledged=unacknowledged) novel    |   File "/root/.local/share/virtualenvs/opt-zvmYt2-H/lib/python3.6/site-packages/pymongo/network.py", line 150, in command novel    |     parse_write_concern_error=parse_write_concern_error) novel    |   File "/root/.local/share/virtualenvs/opt-zvmYt2-H/lib/python3.6/site-packages/pymongo/helpers.py", line 155, in _check_command_response novel    |     raise OperationFailure(msg % errmsg, code, response) novel    | pymongo.errors.OperationFailure: there are no users authenticated
建议每次发docker包使用新的tag ，这样可以使用指定的tag运行 ，现在看tag全都是latest

日志：pymongo.errors.OperationFailure: there are no users authenticated


Introduce DropdownSearchField.Option

This allows a more comprehensible and compact usage of the
DropdownSearchField. In addition, due to replacing many HashMap
instances by the simpler DropdownSearchField.Option object, the memory
footprint will be reduced.
After applying #337, the class WebOpacAt needs to be adapted …

Seems legit to me ;)


Post-save error highlight doesn't work when project contains both main.rs and lib.rs

Looks like if a project contains both src/main.rs and src/lib.rs, post-save error highlight doesn't work.  If I run the cargo rustc command manually it just fails with the following error.
$ cargo rustc -- -Zno-trans -Zunstable-options --error-format=json
error: extra arguments to rustc can only be passed to one target, consider filtering
the package by passing e.g. --lib or --bin NAME to specify a single target
I have to add either --lib or --bin for the rustc command to work.
$ cargo rustc --lib -- -Zno-trans -Zunstable-options --error-format=json # works
$ cargo rustc --bin [BIN-NAME] -- -Zno-trans -Zunstable-options --error-format=json #works
Also setting RUSTFLAGS with "cargo build" seems to work -- should the plugin switch to it??
$ RUSTFLAGS="--error-format=json" cargo build # this also seems to work

good spot @omakoto
Not sure what the best fix for this would be, its a shame it can't just check both :/
if RUSTFLAGS works then maybe its easier to try that.

Hmm, except, cargo build fails in one of my projects unless I remove "no-trans":
$ RUSTFLAGS="-Zno-trans -Zunstable-options --error-format=json" cargo build --verbose
Build failed, waiting for other jobs to finish...
error: could not rename crate "/home/omakoto/cbin/src/rust/highlighter-rs/target/debug/build/bzip2-sys-b768200d1597b521/build_script_build"
Caused by:
No such file or directory (os error 2)
However "cargo rustc" works fine in the same project for both with --lib and --bin... :(
I guess it'd be too slow if you remove no-trans?

@nrc @dten what are your thoughts on this?

You should prefer RUSTFLAGS to cargo rustc, for sure.


passcode change - crashing after change of passcode

What happened:
Thank you. Here a few details. We could fix the problem with a workaround.
He was able to reproduce this error on the Lightwallet and repair it in the same way.
Fixed by following:
The user has now completely deleted all data in the app de-installed and re-installed and entered again new seed to recover the wallet and set a new password.
Everything works now.
"What still needs to be mentioned, the error probably only comes when you have changed the password and then enter the wrong password for the first time during the next action where the password is requested. Then comes the error message that you only have 2 attempts But if you then enter the correct password, then the app smears. The whole thing happens on a HUAWEI P2 Android 9.1.0 "
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:

@izzycsy @dwivedinalin can you try to reproduce this?

Acceptance Criteria: Able to change passcode on 2nd attempt
Android:

[ ] Settings > change passcode > enter wrong passcode (1st attempt) > enter correct passcode (2nd attempt) > change passcode

iOS:

[ ] Settings > change passcode > enter wrong passcode (1st attempt) > enter correct passcode (2nd attempt) > change passcode


variantdev/vals aws profile for SSM ignored with helmfile

Test 1: This works with vals-0.6.0
echo "foo: ref+awsssm://mypath/somekey?profile=myprofile&region=eu-west-1" | bin/vals eval -f -

Result:
SSM: successfully retrieved key=/mypath/somekey
foo: "banana"

Test 2: Using helmfile version v0.119.1
releases:
  - name: something
    namespace: kube-system
    chart: ./something
    values:
      - foo: ref+awsssm://mypath/somekey?profile=myprofile&region=eu-west-1

Result:
get parameter: NoCredentialProviders: no valid providers in chain. Deprecated.
	For verbose messaging see aws.Config.CredentialsChainVerboseErrors

Test 3: If I provide helmfile with AWS_PROFILE=profile-with-permissions as env with permission to that parameter it works.
(but I need to use ?profile for different parameters, also kubeconfig is using a different role with aws-iam-authenticator)
Test 4: If I provide helmfile with AWS_PROFILE=some-other-profile as env without permission to that parameter, says:
get parameter: ParameterNotFound:

It seems when using helmfile the ?profile part is being ignored.

My bad, I had two versions of helmfile on my system.
I will test further, thank you!


Look immediately when digging or activating a block

Somewhat related to #262
I was having some trouble digging blocks on a modded server and it turned out that the server actually checks if the player is looking directly at the block when digging or interacting with blocks and it fails otherwise.
Calling look() or lookAt() before digging or interacting with a block currently doesn't work in this case, even if we set force to true, because the position update is actually sent up to 50 ms after the call and the server receives the block_dig or block_place packet before it knows we moved.
This also affects functions like activateBlock() and it causes problems when opening chests or interacting with some other blocks, it does not happen on a Vanilla server though.
A simple workaround would be to always wait for the 'move' event before proceeding but I think it's better to send the position update immediately because otherwise the documentation is misleading, it says:

force - If present and true, skips the smooth server-side transition. Specify this to true if you need the server to know exactly where you are looking, such as for dropping items or shooting arrows.

but even when force is true you currently have to do something like this:
bot.look(yaw, pitch, true)
bot.once('move', function() {
  // drop item, shoot arrow...
})

That means we might send position updates faster than every 50ms. That's probably not a problem though.
Are you sure "force looking" is ok though ? look at that line https://github.com/andrewrk/mineflayer/blob/master/lib/plugins/physics.js#L238
Is the vanilla client properly displaying the bot look with this change ?

I did that in a7ba46e7473a73a7dc428476fde6554a8fd754c4 .
That's a better fix : no force move needed, no sending position faster than every 50ms needed.
That should fix your pb too on your modded server ;)

Nice, thanks!


Panic at startup

Hi, I'm getting a panic as soon as the node starts. My intention is to set up a validator but could not ever sync the testnet blockchain yet.
I installed the app as the setup instructions indicated, updated after the initial "panic" block and now running 0.13.6. Getting this:
2022-02-15T23:15:58Z INF Starting multiAppConn service impl=multiAppConn module=proxy
2022-02-15T23:15:58Z INF Starting localClient service connection=query impl=localClient module=abci-client
2022-02-15T23:15:58Z INF Starting localClient service connection=snapshot impl=localClient module=abci-client
2022-02-15T23:15:58Z INF Starting localClient service connection=mempool impl=localClient module=abci-client
2022-02-15T23:15:58Z INF Starting localClient service connection=consensus impl=localClient module=abci-client
2022-02-15T23:15:58Z INF Starting EventBus service impl=EventBus module=events
2022-02-15T23:15:58Z INF Starting PubSub service impl=PubSub module=pubsub
2022-02-15T23:15:58Z INF Starting IndexerService service impl=IndexerService module=txindex
2022-02-15T23:15:58Z INF ABCI Handshake App Info hash="���\x14��]�\"�b��~{�qk���z�\x022��\x1b��3�" height=337021 module=consensus protocol-version=0 software-version=0.13.6
2022-02-15T23:15:58Z INF ABCI Replay Blocks appHeight=337021 module=consensus stateHeight=337021 storeHeight=337022
2022-02-15T23:15:58Z INF Replay last block using real app module=consensus
2022-02-15T23:15:58Z INF multisig sign 5f3800e85d41cf4b7aabb4f7199c45d766b015034ed5d9a7b694c7c68bac1be6 completed
panic: error while routing signature to module evm: no command batch found to handle for signature 5f3800e85d41cf4b7aabb4f7199c45d766b015034ed5d9a7b694c7c68bac1be6

goroutine 1 [running]:
github.com/axelarnetwork/axelar-core/x/tss.handleMultisigSigns({{0x2383d48, 0xc000050048}, {0x23b4e40, 0xc006235180}, {{0xb, 0x0}, {0xc003f25440, 0x17}, 0x5247e, {0x2e3a2837, ...}, ...}, ...}, ...)
        /go/src/github.com/axelarnetwork/axelar-core/x/tss/abci.go:293 +0xb6e
github.com/axelarnetwork/axelar-core/x/tss.EndBlocker({{0x2383d48, 0xc000050048}, {0x23b4e40, 0xc006235180}, {{0xb, 0x0}, {0xc003f25440, 0x17}, 0x5247e, {0x2e3a2837, ...}, ...}, ...}, ...)
        /go/src/github.com/axelarnetwork/axelar-core/x/tss/abci.go:32 +0x745
github.com/axelarnetwork/axelar-core/x/tss.AppModule.EndBlock(...)
        /go/src/github.com/axelarnetwork/axelar-core/x/tss/module.go:150
github.com/cosmos/cosmos-sdk/types/module.(*Manager).EndBlock(_, {{0x2383d48, 0xc000050048}, {0x23b4e40, 0xc006235180}, {{0xb, 0x0}, {0xc003f25440, 0x17}, 0x5247e, ...}, ...}, ...)
        /go/pkg/mod/github.com/cosmos/cosmos-sdk@v0.44.5/types/module/module.go:481 +0x33d
github.com/axelarnetwork/axelar-core/app.(*AxelarApp).EndBlocker(...)
        /go/src/github.com/axelarnetwork/axelar-core/app/app.go:667
github.com/cosmos/cosmos-sdk/baseapp.(*BaseApp).EndBlock(0xc001529380, {0x100000000000000})
        /go/pkg/mod/github.com/cosmos/cosmos-sdk@v0.44.5/baseapp/abci.go:210 +0x25d
github.com/tendermint/tendermint/abci/client.(*localClient).EndBlockSync(0xc000f5d260, {0xc000f5d260})
        /go/pkg/mod/github.com/tendermint/tendermint@v0.34.15/abci/client/local_client.go:288 +0xdf
github.com/tendermint/tendermint/proxy.(*appConnConsensus).EndBlockSync(0xc000eee5a0, {0xc000e01c20})
        /go/pkg/mod/github.com/tendermint/tendermint@v0.34.15/proxy/app_conn.go:89 +0x24

How should I continue?
Regards,
Mauro.

Hey, how're you running the node precisely?

Hi @milapsheth , started by following the instructions on the, now unavailable page https://docs.axelar.dev/#/validator-zone/setup/overview and it downloaded v0.13.5.
It said that, at some block, it will panic and I should run the script/node.sh. The app was upgraded to 0.13.6 and continued to run.
Then the testnet failure happened. After it was back online, I started the app again but here is when the crash appears.
Now I'm thinking if I must delete all the data and start to sync from scratch again.


Display and "save" the base image a notebook uses in a notebook cell

We have a little first draft system for pairing a notebook with a base image that looks like this ...

User says garden-ai notebook start foo.ipynb --base-image=3.10-torch
We save a mapping from current/directory/foo.ipynb to 3.10-torch in the user's local data
Then the user can say garden-ai notebook start foo.ipynb and we open up it up again in the right image.

This is nice as a quick first pass but ofc it's pretty fragile. If the user mvs the notebook to a different directory they'll be prompted again to specify the base image.
I suggest that when making a new notebook from a template we create a first cell that presents (in a nice user-readable format) what base image this notebook uses. If the user decides to change it later, that is well and good! They can just edit the cell.
This has a few benefits

We don't need a local DB mapping anymore
The user can move the notebook around or even send it to another machine and the notebook keeps its own metadata. (Especially useful if we're debugging someone else's notebook)

Assumptions:

Acceptance Criteria
Given..., when..., then...

Now that we let users specify a --requirements file, that's something we can also think about in this ticket. The base image we provide and the requirements a user has specified feel pretty linked in the user's mind. So we'll want to think about those holistically.


fix: add pirep field for distance and fixed distance calculation

This should fix the live map depiction of the distance.

@BossOfGames are you able to do a release for me?


PathFinder is easily confused

Fix it or remove in favour of: https://github.com/PurpleKingdomGames/indigo/pull/648

Despite fixing the issue, deprecating this in favour of something smarter may still be a good idea...


Added madgrad_wd optimizer from https://github.com/lessw2020/Best-Dee…

As per the request https://github.com/EleutherAI/gpt-neox/issues/210 madgrad_wd from https://github.com/lessw2020/Best-Deep-Learning-Optimizers is added to the optimizers list.

As per the request #210 madgrad_wd from https://github.com/lessw2020/Best-Deep-Learning-Optimizers is added to the optimizers list.

Oh wow, awesome! I'll take a look over this this evening.

The code all looks perfect to me. Have you tried it out yet @preethamgali ?
I'm not sure how madgrad handles fp16

Working great for me - merging it now. Thanks again @preethamgali


Logo size

Is there a constraint to the size of the logo (e.g. based on the "Community," "Planet," etc. buttons to the right of the logo) or could it be made larger? I'd suggest at least 100px height if possible--but I defer if there are web standards for sizing!

There is a constraint. I don't suggest changing its height. How about do I change its width?

I don't think changing the width would make sense if we can't change the height too. I don't mind closing this given the height constraint.


Stickers: Set Experiment to True

Feature Description
Set sticker experiment to default true.
Alternatives Considered

Additional Context

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Verified in QA


Bundle for client-side usage

Two things to do:

Expose as a global function in UMD build
Expose as an ESM

fixed in 3d2cd4f9dcd8f73d87e0666af9686367954ee22f


use const instead the magic number

I think const is better than the magic number

/lgtm
/approve


Snyk Scan parse: cvssScore null value causes exception for operator

Description
cvssScore values of null cause an exception in the Snyk Scan parser, which permits only integer or float.
Expected behavior
The vulnerability should be parsed without its cvssScore component, with field populated appropriately.
Environment information

Operating System: [e.g. Ubuntu 18.04
DefectDojo version 2.9.1

Logs
**An exception error occurred during the report import:'<=' not supported between instances of 'NoneType' and 'float' **
Sample scan files
https://pastebin.com/raw/KULwF8Hx

This isn't too tricky a fix and I can crack on with it along with testcase from my scan result sample - However, I'm unsure how the null value should be handled and could do with advice on that.

@P440Boyd thanks for your feedback. Create a PR and we will gladly help you.


Allow field init shorthand for component prop fields

Adds support for field init shorthand syntax, with tests.
let signal = create_signal(cx, 0);
view! { cx, Component { signal } }

This looks great! Thanks!


[BUG]: AA4PPA Can't share a deployment user profile to a team associated to an AAD security group

According to this documentation, https://learn.microsoft.com/en-us/power-platform/guidance/alm-accelerator/setup-deployment-user-settings it appears that it is possible to associate a deployment profile with a team that is associated with a security AAD group.
However, when I want to share this profile, I don't see the team
To Reproduce
Steps to reproduce the behavior:

Go to the environment that contains ALM
Click on teams
Create new teams
Select AAD security group
Select Members
Open AA4PPA
Select Deplyment User Profile
Select the deployment and click on share
in add user/team, type the name of the Team or advanced lookup and select teams

Ok, find the answer by mistake.
I have not seen the dropdown menu.

A step is missing in the documentation


Unnecessary space between badge and action

Testing #177709

This was preexisting, and I believe it's this way so that people don't mistaken the file decoration as a button (it's in its own grouping).


Update Get Started for 2.2

Fixes #9948
Internal review link

The tutorial already has "Click Accept to accept the privacy and cookie policy."  What is it missing?


Bug with side markers and brake lights

Hello! Been tweaking the code for own liking, (not a fan of the sequential flasher and the "fill and empty" of just brake lights) and I came across an issue.
I tested it with your original code on the sequential mode. I'll explain what happens and how.
1.) turn brakes on
2.)turn on hazards
3.) turn on left or right signal on stalk
4.) turn off hazards,
5.) turn hazards back on
The brake light keeps the side marker solid lit of whichever signal the stalk is set to. If you then switch the stalk to the other signal while hazards are on, and flip them off and on again, both side markers will be solid lit, no flashing.  If brakes come off, it will revert to sequential hazards again. This was all done on a breadboard.
It's extremely rare you'd encounter this when driving, but any chance of taking a look at it? Thanks!

To add, if I start the arduino with hazards on and brakes pressed, the lights all stay off. And if you power the arduino with the brakes on and one signal on, it will sequentially do the signal, but the brake side will be solid off.

Third issue.
1.) one tap left/right signal.
2.) flip stalk to other direction for any period of time longer than a tap
3.) turn off signal.
4.) one tap function finishes in initial direction
This one is a bit more dangerous. If you accidentally hit the one tap, then put the stalk in the other direction, it WILL properly flash while you keep the signal on. But as soon as you turn it off, it will flash 3 times in the tapped direction, no matter how many times it flashed at the initial tap. Also, if you tap one direction then immediately tap the other, only the initial tapped direction runs. I think ideally, it should overwrite to flashing the latest tapped direction.


Add missing upload stream function

Function uploadStream is required in the strapi v4 (ex. logo upload)

Sorry for writing here but the video previews are not working in admin panel because video URL does not have https:// in front of it.
I just tried with your plugin as well. below is the complete thread.
Can you please provide any help in this case?
https://forum.strapi.io/t/video-preview-not-working-when-using-strapi-with-digitalocean-spaces/21716

Is this needed to get this plugin to work with strapi 4.3.6 for media uploads?

Thanks for your response. I am using the latest version of strapi.
Yes, this is very crucial to view the video preview besides it also strips out https:// which does have issues on the API response on the client side, especially when doing static site generation.

Hi, Thanks for the contribution,
I just reviewed it and it is a great addition.
I'm sorry I couldn't review it sooner.


C3 instance's destroy method does not get called when directive's scope is destroyed

The situation where this is most apparent is this: if charts are created and destroyed many times, window resize event listeners get left hanging, which causes resizing browser window to get progressively slower.

I already wrote a quick and dirty fix for this, but did not have time to check if this has any side-effects: https://github.com/joukosaastamoinen/angular-chart/commit/d5a2a70c4a0cde72c4afd7b5b018de84206bbe1a

You are totally right, the c3.js event listeners have to be destroyed. I will add that in the next release.

I implemented your fix by listening to the $destroy event on the scope.
// Registers a $destroy listeners for cleanup purposes
//
scope.registerDestroyListener = function() {
  scope.$on('$destroy', function() {
      scope.chart.destroy();
      element.remove();
  });
};


Remove rspec version part1

Pull Request Test Coverage Report for Build 12

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+22.05%) to 94.521%

Totals

Change from base Build 10:
22.05%

Covered Lines:
69

Relevant Lines:
73

💛  - Coveralls

Pull Request Test Coverage Report for Build 12

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+22.05%) to 94.521%

Totals

Change from base Build 10:
22.05%

Covered Lines:
69

Relevant Lines:
73

💛  - Coveralls

Pull Request Test Coverage Report for Build 12

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+22.05%) to 94.521%

Totals

Change from base Build 10:
22.05%

Covered Lines:
69

Relevant Lines:
73

💛  - Coveralls

Pull Request Test Coverage Report for Build 12

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+22.05%) to 94.521%

Totals

Change from base Build 10:
22.05%

Covered Lines:
69

Relevant Lines:
73

💛  - Coveralls

Pull Request Test Coverage Report for Build 12

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+22.05%) to 94.521%

Totals

Change from base Build 10:
22.05%

Covered Lines:
69

Relevant Lines:
73

💛  - Coveralls


Script Runner and example job renaming

Adds slightly modified versions of RestApiScriptRunner and RestApiJobManagement from
https://github.com/sheehan/job-dsl-gradle-example
renames all example jobs for consistency and absence of redundancy

Details:

Script runners:
Irina had originally included these in the internal
jenkins-automation-private repo, for use with recovering the "Mother
Seed Job".
I think this is so useful for a local development workflow that I'm
bringing it into jenkins-automation
It will then be usable by any repo that uses jenkins-automation.
The idea is to use this as part of a local development workflow for
job-dsl jobs that mitigates the slow-and-annoying git workflow we have
ot use now

Example job renaming

These jobs are already in an example-jobs folder.
I see no need to have them have example or sample in the name.
In addition, they should be named consistently, not the island of misfit
names that they were prior to this commit.

What I'd like to see is some documentation (either in the PR, or preferably in readme markdown) on how to use the script runner.
If the solution is a link to a document elsewhere that's fine, but it also could be a code snippet that i can copy and paste into my dev environment.

@m3brown I'll definitely add that, but I am not certain of the particulars yet because I need to get this PR merged first so that I can test using it from within a JAC repo.
Ultimately, though, from any JAC repo you should be able to do this:
./gradlew rest -Dpattern=<pattern> -DbaseUrl=<baseUrl> [-Dusername=<username>] [-Dpassword=<password>]

pattern - ant-style path pattern of files to include
baseUrl - base URL of Jenkins server
username - Jenkins username, if secured
password - Jenkins password or token, if secured

👍 Should also be added to the starter repo

I think if you loaded those parameters from a config file, it might be better, and also serve as documentation.  That is, allow the settings to be defaulted from a file, but overridden from the command line.
I don't particularly like the pattern argument either.  Instead of loading files from a directory, we should be loading classes from a package using inspection.  So if I said something like '--package=jenkins.jobs', then any class under that package would go ahead and be run.

We can debate the particulars later. I need to merge this so that I can see if it even works from within a separate repo

Sweet. This works exactly as expected. I'll add docs to here and the starter repo soon (tracking this work in Jira).
@vickumar1981 I like the config file idea. However, I don't think the package thing applies, because jobs don't live in packages, just folders. The pattern arg lets you target specific job files:
./gradlew rest -Dpattern=jobs/environment_test_job.groovy, for example, which during development is probably what you're looking for

yeah, i understand what the pattern does.  I just think it's a limitation.  All your files have to be inside the same directory (what if I want different folders for my jobs?  does it handle recursion?  is there a switch for this?)
I could have multiple classpaths, for example, with jobs in different projects, but logically grouped under the same package.  What if I just send you a JAR file with all my jobs already compiled, for example? The latter method allows users of JAC greater flexibility.  Like in the environment_test_job.groovy example above, I just think it should be inside a package, and I don't care where that package is, so long as it's able to be located inside $CLASSPATH.

@vickumar1981 I think you are way overthinking this, tbh

@vickumar1981 Plus, ant pattern matcher is extremely flexible. I don't know if it'd support your JAR example, but that's not a thing I think would actually exist in real life anyway. And remember, this is a development-time tool, designed to help you iterate on jobs before pushing them

Sure, but I don't see why we're limiting it to run only files/scripts vs a more elegant and flexibile approach of loading the class dynamically.
task rest(dependsOn: 'classes', type: JavaExec) {
   main = 'jenkins.automation.rest.RestApiScriptRunner'
    ...
}

Why does gradle itself want you to specify a class instead of a file/directory for example, as to where the Main method is located?  I get that this is a single resource and not a pattern, but...
When you specify dependencies, when you declare a managed, or unmanaged resource, in basically every configuration option for gradle, sbt, maven, ant, you're specifying a package, b/c packages are how the JVM logically groups classes.  src/main/[groovy | scala | java] is automatically loaded into the CLASSPATH by convention, but we still allow a configurable option to modify/change that convention.
I'm just saying that if we follow what the JVM build tools do, then we won't be tying ourselves down as much.

@vickumar1981 I think this is beyond my JVM knowledge, and we should probably take it to pairing


Add eslint-loader to webpack's dev config

https://github.com/MoOx/eslint-loader

Won't do this, linting should stay out from the webpack build


lhash: Avoid 32 bit right shift of a 32 bit value

Fixes #17583

This pull request is ready to merge

Merged to all branches. Thank you for your reviews.


⚠ ClamAV has degraded performance

In 0b75580, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 66 ms

Resolved: ClamAV performance has improved in 28e9f7b.


Updating hub page with latest design and content

@wadepickett This is the hub page update. It successfully builds.


Color mode

Please please give us back the original color mode. The new colors look bright and neat, though. But after a few rounds it hurts my eyes. The original colors were not that beautiful, but they had sharp transitions to each other. At least let the player chose which color mode to pick. Would be a benefit for the game imho :D Great game!

+1 for that.
I am actually red/green blind and the new colors make the game even harder. The new color blind option isn't really a good alternative since the old colors had enough contrast to be fun to while being possible. The gameplay had a huge shift through that change.
Maybe implementing the new colors as difficulty level would be a better way.

Duplicate of #12.
I plan to add an option to revert back to the old color scheme in the next release as well as desaturate the current color scheme to make it easier on the eyes.
See my comments in the discussion in #12 for more info.

ed70b50b84a83d4fd4a863df7b53f46a773f2617 and c32013dfc331d4a05cff14b46aa672227e2db1d3

Nice game, for me the orginale colours worked beter.
But i can't choose those Nice colours any more.
Is it possible to make it an option like "classic" mode?
Yours kindly
Roel
The Netherlands

You should update to 1.2.2 and have a look into the settings -> "Old Color Scheme".
btw. Thanks for that :+1:


ci(kitchen+travis): replace EOL pre-salted images

Automated using https://github.com/myii/ssf-formula/pull/16

:tada: This PR is included in version 0.13.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


python310Packages.upcloud-api: 2.0.1 -> 2.5.0

Description of changes
Diff: https://github.com/UpCloudLtd/upcloud-python-api/compare/refs/tags/v2.0.1...v2.5.0
Changelog: https://github.com/UpCloudLtd/upcloud-python-api/blob/refs/tags/v2.5.0/CHANGELOG.md

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

5 packages built:

home-assistant-component-tests.upcloud
python310Packages.upcloud-api
python310Packages.upcloud-api.dist
python311Packages.upcloud-api
python311Packages.upcloud-api.dist


BrowseTheWeb to offer access to storages and cookies

We should add some abstraction layer inside BrowseTheWeb to offer access to

session storage
local storage
cookies
inside the current browser window.

added functions:

getBrowserCookies
addBrowserCookies
clearBrowserCookies
getBrowserStorageState <---- Returns storage state for the browser context, contains current cookies and local storage snapshot.

i didn't find anything for session storage

added actions Add, Fetch and Remove to access cookies, session storage and local storage.


Html commented frontmatter

Closes  #1428

Why does your commit contain Docker stuff?

@ariejan and woooosh, its gone :)
I do wonder if building with Docker would be welcome in a separate PR ...

@wenerme @bep @spf13

Merged in a1e3243
For future reference, please end the commit body with "Fixes #"; it saves me from doing it.


[AzureMonitorExporter] resolve AOT warnings

Related: #37734.
This PR mitigates AOT warnings in the Azure.Monitor.OpenTelemetry.Exporter library.
Azure.Monitor.OpenTelemetry.Exporter has 14 totals warnings

AzureMonitorExporterEventSource.cs

IL2026:RequiresUnreferencedCodeAttribute

Using member 'System.Diagnostics.Tracing.EventSource.WriteEvent(Int32,Object[])' which has 'RequiresUnreferencedCodeAttribute' can break functionality when trimming application code. EventSource will serialize the whole object graph. Trimmer will not safely handle this case because properties may be trimmed. This can be suppressed if the object is a primitive type.

The fix is to decorate these methods with [UnconditionalSuppressMessage("ReflectionAnalysis", "IL2026:RequiresUnreferencedCode", Justification = "Parameters to this method are primitive and are trimmer safe.")]

AzureMonitorStatsbeat.GetVmMetadataResponse & IngestionResponseHelper.GetErrorsFromResponse

IL2026:RequiresUnreferencedCodeAttribute

Using member 'System.Text.Json.JsonSerializer.Deserialize(String,JsonSerializerOptions)' which has 'RequiresUnreferencedCodeAttribute' can break functionality when trimming application code. JSON serialization and deserialization might require types that cannot be statically analyzed. Use the overload that takes a JsonTypeInfo or JsonSerializerContext, or make sure all of the required types are preserved.

IL3050:RequiresDynamicCodeAttribute

Using member 'System.Text.Json.JsonSerializer.Deserialize(String,JsonSerializerOptions)' which has 'RequiresDynamicCodeAttribute' can break functionality when AOT compiling. JSON serialization and deserialization might require types that cannot be statically analyzed and might need runtime code generation. Use System.Text.Json source generation for native AOT applications.

The fix is to use source generation.

Models\StackFrame

IL2026:RequiresUnreferencedCodeAttribute

Using member 'System.Diagnostics.StackFrame.GetMethod()' which has 'RequiresUnreferencedCodeAttribute' can break functionality when trimming application code. Metadata for the method might be incomplete or removed.

The fix is to decorate with UnconditionalSuppressMessage . GetMethod() may return null. In this case we will fall back to ToString().

LogsHelper.GetProblemId

IL2026:RequiresUnreferencedCodeAttribute

Using member 'System.Diagnostics.StackFrame.GetMethod()' which has 'RequiresUnreferencedCodeAttribute' can break functionality when trimming application code. Metadata for the method might be incomplete or removed.

The fix is to decorate with UnconditionalSuppressMessage . GetMethod() may return null. In this case we will fall back to ToString().

API change check
API changes are not detected in this pull request.

@vitek-karas, @Yun-Ting, @m-redding Please help with this review :)


Pkg.instantiate() after sysimage build

Yep, my change doesn't do a thing. Fixing..

Ok, I think this should work, since sysimage.jl is using (version 1 of) PackageCompiler with replace_default=true, so the new invocation of julia should be using the sysimage.


board specific code moved to boards directory

Board specific implementation are moved from arch folder to boards folder

Shoud we correct the commit message for this patch:
https://github.com/apache/incubator-nuttx/pull/1030/commits/7c504cc7174aaddcd9490095fccf969b35e63725
Remove this empty patch:
https://github.com/apache/incubator-nuttx/pull/1030/commits/4da5f19ed0f07bef1a75b656be4dd71141e34de9
Merge two nxstyle patch into one:
https://github.com/apache/incubator-nuttx/pull/1030/commits/0bf36ecd2c4fad39a71bcb16d538175b822e7fd8
https://github.com/apache/incubator-nuttx/pull/1030/commits/a56e9f01a491fd2438353fa2847b915dd62e6c1d
@RB-tel I also suggested before that it's better to enable renesas daily check if you take renesas support serously.

Shoud we correct the commit message for this patch:
7c504cc
Remove this empty patch:
4da5f19
Merge two nxstyle patch into one:
0bf36ec
a56e9f0
@RB-tel I also suggested before that it's better to enable renesas daily check if you take renesas support serously.

When I run nxstyle check on my repository, I do not get few violations. But when I do PR, the check fails with the violations. I will fix them. Please ignore the PR till I clear the checks.
I am new to github, so doing multiple commits. Will learn the process and try to avoid such things.

Maybe your branch contain the old nxstyle, the best workflow is:
git checkout master -- it's better to use other branch instead master
git fetch apache
git rebase apache/master
./tools/checkpatch.sh ....
git rebase --interactive -- squash or remove the patch as needed
git push -f origin master

Maybe your branch contain the old nxstyle, the best workflow is:
git checkout master -- it's better to use other branch instead master
git fetch apache
git rebase apache/master
./tools/checkpatch.sh ....
git rebase --interactive -- squash or remove the patch as needed
git push -f origin master

Thanks a lot. Will use the above suggested workflow.


Nightmare drops don't show up

I have had 3 drops from the nightmare. An orb, a hauberk and a pet.
Neither of the three showed up on Discord.

Should be fixed. Can re open this if needen.


Allow podman for normal build steps

What this PR does / why we need it:
If docker is present and working, continue choosing docker. If it is not
present try to fallback to podman. In case that KUBEVIRT_CRI
environment variable is set, try to use whatever is selected there.
Note that local test-cluster related operations like make cluster-up and make cluster-sync still require docker, but build steps like make, make generate or make functest work now with podman or docker.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:
NONE

/cc @EdDev
/cc @danielBelenky
/cc @dhiller

/cc @phoracek

This fails for me when trying to build tests with following:
$ hack/dockerized hack/build-func-tests.sh
hack/dockerized: line 13: docker: command not found
selecting podman as container runtime
Trying to pull docker.io/kubevirt/builder@sha256:e6a76ec71f2de87c3bc648bae87ab43a113540bf57da69f07f76bd0edfd500ab...ERRO[0002] Error pulling image ref //kubevirt/builder@sha256:e6a76ec71f2de87c3bc648bae87ab43a113540bf57da69f07f76bd0edfd500ab: Digest of source image's manifest would not match destination reference
Failed
Error: unable to pull kubevirt/builder@sha256:e6a76ec71f2de87c3bc648bae87ab43a113540bf57da69f07f76bd0edfd500ab: 1 error occurred:
        * Digest of source image's manifest would not match destination reference

/test pull-kubevirt-e2e-windows2016

@phoracek works for me:
$ podman pull docker.io/kubevirt/builder@sha256:e6a76ec71f2de87c3bc648bae87ab43a113540bf57da69f07f76bd0edfd500ab
Trying to pull docker.io/kubevirt/builder@sha256:e6a76ec71f2de87c3bc648bae87ab43a113540bf57da69f07f76bd0edfd500ab...
Getting image source signatures
Copying blob dfe6ecd6d77b skipped: already exists
Copying blob 5cf469971bf6 skipped: already exists
Copying blob 48699dca5da3 skipped: already exists
Copying blob 95060617e71b skipped: already exists
Copying blob b1833078a99d skipped: already exists
Copying blob d1de41b0fa4e skipped: already exists
Copying blob b0fe058ffda2 skipped: already exists
Copying blob aafcb618b9d8 skipped: already exists
Copying blob<PHONE_NUMBER>3d [--------------------------------------] 0.0b / 0.0b
Copying blob d0f8d8b071eb [--------------------------------------] 0.0b / 0.0b
Copying config c88a484b96 done
Writing manifest to image destination
Storing signatures
c88a484b960b86d5b88a8258df50914417b1dbcc4e67ae61a666216e6bc12a75

podman version:
$ podman version
Version:      2.0.4
API Version:  1
Go Version:   go1.14.6
Built:        Thu Jan  1 01:00:00 1970
OS/Arch:      linux/amd64

Must be my ancient Podman 1.4.2. I will upgrade my system and verify again.

/test pull-kubevirt-e2e-k8s-cnao-1.17
/test pull-kubevirt-generate

Commit message typo: “that that”

Fixed.

/test pull-kubevirt-e2e-k8s-cnao-1.17
/test pull-kubevirt-generate

After upgrading the system, it works fine with Podman 2.0.4.