Fix onboarding example code loading

In the new user, new sign-in case we were not setting the code in the KCL code editor, because we checked if a value was null before doing that, but just above had made the value coerce to "" if it was found to be null. Annoying little JS behavior. This fixes that issue and adds a Playwright test to verify the "fresh user" case always sees the onboarding code in their editor.

Sorry for the big diff in the Playwright tests; I also grouped our existing onboarding-related tests with this new one under an "Onboarding" test group.

This doesn't touch "sketch on face" so I find the one MacOS Playwright failure spurious; merging.

Resolves #2677


After release YM beta 5.6.0 taskbar buttons not working

Describe the bug
After updating YM to 5.6.0, the buttons on the YM preview are pressed, but nothing happens.
To Reproduce
Steps to reproduce the behavior:

Launch the application.
Find the application icon on the taskbar.
Hover the cursor, a preview with control buttons will appear.
Press the control button, for example, play \ stop.
Nothing will happen.

Expected behavior
I expected that when I pressed the play control button, music would start playing, but the state of the player did not change.
Screenshots/Video

Client info (please complete the following information):

OS: Windows 10 Pro 22H2
Modification version 1.6.0
Yandex Music version 5.6.0

Thanks for your report. I can reproduce this bug. Bug identified and will be fixed in next release.

Fix released 1.6.1


Failing when port is specified with labels

Resolves #522
Kompose will give FATAL error if labels are given but ports are not defined

cc @kadel @cdrage

@surajnarwade tests and docs update

@cdrage @surajssd needs review

LGTM :+1:

@surajnarwade tests and docs update

Would it be better if it return error here and leave the Fatalf() work to upper layer?
I'm investigating https://github.com/kubernetes-incubator/kompose/issues/464 and I think logrus.Fatalf(...) may be a broker because it finally invokes os.Exit()
Anyway, returning an error consists with other parts of the codes:
https://github.com/surajnarwade/kompose/blob/8198246bd8363642800941be75a2b9bf414ee9b5/pkg/loader/compose/compose.go#L352

@gitlawr , return error here, I didn't get you

@surajnarwade
I mean
return kobject.KomposeObject{}, errors.New("xxx defined in service xxx with no ports present. Issues may occur when bringing up artifacts.")

instead of
log.Fatalf(...)

@surajssd , needs review on this

@surajnarwade bear with me, also please add functional tests! so when I meant tests == (unit tests and functional tests)

@cdrage , @surajssd I have updated PR with changes and added tests as well

@kadel @surajssd @containscafeine can you please test now

@cdrage review please :)

All green now, @cdrage @surajssd


Python2 removal

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


ci: updated releaserc with a URL

PR summary

PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

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


Error: cannot find file

I'm using multithread_download() with url.
Downloading goes well in console with progress %. When it's done, it shows error 'cannot find file path './m3u8_To_Mp4.mp4'.
I tried create a file './m3u8_To_Mp4.mp4' advancedly but it still doesn't work.
Did I miss something?

直接说中文了
发现是_merge_to_mp4_by_ffmpeg()这个函数的问题，subprocess没有完成，然后在外面下载函数结尾打印下载信息的时候就找不到文件了。
我debug下，把cmd命令复制出来 在cmd里手动执行，是可以成功拼接mp4文件并置入指定目录的。
但是程序中的subprocess.Popen()并没有执行这个ffmepg指令。是否需要配置什么东西？

原来是在python环境中 没法使用ffmpeg（系统环境因为有系统变量所以可以吧）。这个我要研究一下了

破案了。我是在启动了pycharm之后添加的ffmpeg环境变量，运行时os不会去读新的环境变量而是启动时就加载好了。重启解决问题。。

Excuse-me, can you explain how did you fix it?

I've the same problem again and again.
segment set: |##################################################| 100.0% downloaded segments successfully! 2022-06-05 22:36:09,345 | INFO | decrypt and dump segments... 2022-06-05 22:36:09,389 | INFO | merging segments... Traceback (most recent call last):   File "get_tweet.py", line 54, in <module>     m3u8_To_MP4.multithread_download(video)   File "/home/pi/.local/lib/python3.7/site-packages/m3u8_To_MP4/__init__.py", line 95, in multithread_download     crawler.fetch_mp4_by_m3u8_uri(True)   File "/home/pi/.local/lib/python3.7/site-packages/m3u8_To_MP4/v2_abstract_crawler_processor.py", line 198, in fetch_mp4_by_m3u8_uri     printer_helper.display_speed(task_start_time, fetch_end_time, task_end_time, self.mp4_file_path)   File "/home/pi/.local/lib/python3.7/site-packages/m3u8_To_MP4/helpers/printer_helper.py", line 52, in display_speed     download_speed = os.path.getsize(target_mp4_file_path) / download_time / 1024   File "/usr/lib/python3.7/genericpath.py", line 50, in getsize     return os.stat(filename).st_size FileNotFoundError: [Errno 2] No such file or directory: './m3u8_To_Mp4.mp4'

I've the same problem again and again.
segment set: |##################################################| 100.0% downloaded segments successfully! 2022-06-05 22:36:09,345 | INFO | decrypt and dump segments... 2022-06-05 22:36:09,389 | INFO | merging segments... Traceback (most recent call last): File "get_tweet.py", line 54, in <module> m3u8_To_MP4.multithread_download(video) File "/home/pi/.local/lib/python3.7/site-packages/m3u8_To_MP4/__init__.py", line 95, in multithread_download crawler.fetch_mp4_by_m3u8_uri(True) File "/home/pi/.local/lib/python3.7/site-packages/m3u8_To_MP4/v2_abstract_crawler_processor.py", line 198, in fetch_mp4_by_m3u8_uri printer_helper.display_speed(task_start_time, fetch_end_time, task_end_time, self.mp4_file_path) File "/home/pi/.local/lib/python3.7/site-packages/m3u8_To_MP4/helpers/printer_helper.py", line 52, in display_speed download_speed = os.path.getsize(target_mp4_file_path) / download_time / 1024 File "/usr/lib/python3.7/genericpath.py", line 50, in getsize return os.stat(filename).st_size FileNotFoundError: [Errno 2] No such file or directory: './m3u8_To_Mp4.mp4'

@papillon-unix have you solved this problem? I'm running into the same issue

说中文我反而看不懂了 :(

Install ffmpeg to your system then try it again!
杨肥肥
@.***

------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2022年12月9日(星期五) 上午9:53
收件人: @.>;
抄送: @.>; "State @.>;
主题: Re: [songs18/m3u8_To_MP4] Error: cannot find file (Issue #8)
I've the same problem again and again.
segment set: |##################################################| 100.0% downloaded segments successfully! 2022-06-05 22:36:09,345 | INFO | decrypt and dump segments... 2022-06-05 22:36:09,389 | INFO | merging segments... Traceback (most recent call last): File "get_tweet.py", line 54, in <module> m3u8_To_MP4.multithread_download(video) File "/home/pi/.local/lib/python3.7/site-packages/m3u8_To_MP4/init.py", line 95, in multithread_download crawler.fetch_mp4_by_m3u8_uri(True) File "/home/pi/.local/lib/python3.7/site-packages/m3u8_To_MP4/v2_abstract_crawler_processor.py", line 198, in fetch_mp4_by_m3u8_uri printer_helper.display_speed(task_start_time, fetch_end_time, task_end_time, self.mp4_file_path) File "/home/pi/.local/lib/python3.7/site-packages/m3u8_To_MP4/helpers/printer_helper.py", line 52, in display_speed download_speed = os.path.getsize(target_mp4_file_path) / download_time / 1024 File "/usr/lib/python3.7/genericpath.py", line 50, in getsize return os.stat(filename).st_size FileNotFoundError: [Errno 2] No such file or directory: './m3u8_To_Mp4.mp4'
@papillon-unix have you solved this problem? I'm running into the same issue
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you modified the open/close state.Message ID: @.***>

I've installed ffmpeg on macOS

I've installed ffmpeg-python pacakge

Python version

PIP version

and still has issues with no such file or directory. Any ideas?


Socket mode - Using slash commands in DM/self message

I have set up a slack app and configured a slash command. A slack bot is created using Bolt SDK via socket mode.
The slash command is working from a channel, however it's not working within a direct message or message to self.
The bot token looks to be having scope to read/post direct messages
Here is the code
`const app = new Slack.App({
token: SLACK_SECRETS.SLACK_BOT_TOKEN,
signingSecret: SLACK_SECRETS.SLACK_SIGNING_SECRET,
socketMode: true,
appToken: SLACK_SECRETS.SLACK_APP_TOKEN,
extendedErrorHandler: true,
});
app.command('/restart', async ({ ack, body, client }) => {
//do something
await ack();
});
(async () => {
// Start your app
await app.start(3000);
console.log('⚡ Bolt app is running!');
})();
`

hi, @syedirfan33! Thanks for the question!
As mentioned here:

To post a message to a private channel, a bot user needs to be a member of the private channel.

As for DMs, it's similar. Your bot user needs to be a member of multiparty direct messages (mpim in APIs) or start a DM (im in APIs) with a user by im.open. Your bot user cannot jump into others' private conversations.

Have you tried testing the slash command in a DM with the bot?

Hi @hello-ashleyintech ,
I tried to use the slash command in a DM with the bot, it's working as expected. Could you elaborate how to get the slash command work within a direct message or self message? It looks like the bot could use user tokens and not sure how to use user tokens with bolt sdk.

If a Slack app has a user token to access a DM / private channel, the app is eligible to post a message to those conversations.

@syedirfan33 - glad to hear it worked!

Could you elaborate how to get the slash command work within a direct message or self message? It looks like the bot could use user tokens and not sure how to use user tokens with bolt sdk.

For sure! As cited from here:

If your app really needs the same access a user who installed the app has, or the app may work on behalf of users, your app may use user tokens over bot tokens. In this case, the app expects every single user in the workspace to install the app. Installation here means granting user-level permissions to the app.

Keeping the above in mind, you can think of a user token giving an app the ability to execute all actions that the user installing the app can (within the specified user scopes). To achieve this with Bolt, I recommend using the built-in OAuth functionality to achieve utilizing user tokens in your app! To learn more about user tokens, check out this guide. I hope this helps!

Thanks for answering the question @hello-ashleyintech


docs: Update template command

Update template command given the current one is not valid:
$ dotnet new -n Sandbox.Generated.Operator operator
Error: Invalid option(s):
  operator
For usage information, run:
   dotnet new -h

Thanks :-)

Happy to help!


"I'll add details soon" checkbox

Perhaps have a checkbox,
"I'll add more details soon"
that appends those words to the description when uploading.
As most users won't have time when uploading in the field to make a perfect submission yet.

Very interesting idea!
Ideally there would also be a way on desktop browser to see what images need to be perfected.
A possible implementation which does not make the Android UI even more crowded that it is already:

On mobile: When you have no time, just enter nothing in the "description" field.
On desktop: Go to a special webpage that lists all of your pictures that lack a description, newest first.

Or automatically add a Category:Bob_Nerblesome/Not_Finished_Editing_Yet_Don't_Get_Angry_at_me etc.
I don't think it is actually "legal" (as seen when attempting to do it from a browser) to get away without a Description.

@nicolas-raoul Please let me work on this issue. thx

@xiangl1111 Please read https://github.com/commons-app/commons-app-documentation/blob/master/android/Volunteers-welcome!.md and first choose issues that have the "bug" label, thanks!


Stuck while downloading large batch

Source: Gelbooru.com
What steps will reproduce the problem?

Download a tag with over 100k files
Cancel it after 10k images.
Resume the batch download.

What is the expected behavior? What do you get instead?
Download proceeds normally till the last file. Does not need user intervention to continue.
Download gets stuck after "reportedly" 19000 files are checked and needs the user to click "Skip" for it to continue. Reportedly because the counter always malfunctions.
How often does this problem occur?
Always.
What version of the program are you using? On what operating system?
v5.4.2
Windows 10 x64
Please provide any additional information below
Picture: http://i.imgur.com/gXW1zPv.jpg

New findings:

It gets stuck after completing a page.
It gets stuck at definite page numbers which are almost 20 pages apart. The file count being 18999, 37999 and so on.

New Findings:

It doesn't get get stuck at pages almost 20 pages apart.
It gets stuck at random pages but the processed file count always consistently has 999 at the end. The file count as I have discovered yet: 18999, 37999, 40999

New Findings:

Sometimes the client gets wild while processing files. Even though it gets the file from the first call, it still searches for an arbitrary number of possible extensions with "Image not found". When the image gets fully downloaded and all extensions had not been searched yet, it stops the search.

This also happens when a page which has at least one downloadable image, gets completed. It tries to find those images again but fails for some reason. It does this till I press "Skip". It still continues the search but stops when the next page gets loaded.

Continued from above. Videos for the findings.
Point 1: https://drive.google.com/open?id=0B-5SBhsDznYOMTBMNzk5V0ctcHM
Point 2: https://drive.google.com/open?id=0B-5SBhsDznYOc3oxb0ZHV3cyLUk

New Findings:

A crash happens randomly. Dumps: https://drive.google.com/open?id=0B-5SBhsDznYONjNocTdsQlpncHc

Now it consistently crashes, just like before... my dreams are tarnished yet again...

New Findings:

The app sometimes get stuck on a single file and infinitely loops over the extension list querying the api.
It can be stopped by clicking "Pause" but this introduces a new erroneous behavior instead. The downloads that are being downloaded do get paused but then it starts the waiting ones and continues as nothing happened.

It gets stuck at random pages but the processed file count always consistently has 99x (x being 5-9) at the end. In the log: It tries to find images from who knows where but fails for some reason. It does this till I press "Skip". Evidence: https://drive.google.com/open?id=0B-5SBhsDznYOMTBMNzk5V0ctcHM

Same thing here. Surprised the issue hasn't been labeled yet. This happens quite regularly with my large batch downloads.

Damn, I am getting a real urge to make my own app instead... even though I know, it would be an endeavor...


New Comments API

The refactoring of registering comment threas for Pull Request list view is done but review mode part is still in progress, following code needs to be improved

Register comment threads for pr, review and file scheme files
Update them properly when there are new comments update
Update commenting range when document is dirtly
Dispose them properly

@RMacfarlane thanks for your thorough review, I feel confident with our product quality as your review is always high quality ❤
I addressed your comments and fixed a few upstream issues (not in Insiders yet so please wait till Monday). In addition to issues you ran into, I also fixed a few bugs with Review and comments update. Since we didn't merge this pr when I was still in office, I need your help to merge this pr. You may still run into bugs so it would be great if you have a better understanding how comment threads cache is managed in this PR

PR List View

Raw comments are saved in fileChanges, which is the same as before
When an editor is being opened, we will check if we already create comment threads for the newly opened editor or not. If not, we create comment threads based on fileChange.comments.
When an editor is being closed, we will dispose all related comment threads.
When we refresh the PR, we will compare the cached comment threads for this PR, with the new comments, and update accordingly.
Note, we only update all comment threads for PR when users attempt to refresh the PR, which is the same behavior as before.
When we create/update/delete comments, we need to update fileChanges.comments and visible comment threads.

Review

Raw comments are saved in fileChanges, which is the same as before
We will create comment threads when we detect the workspace is in review mode.
When we open diff view from PR List View or Changes in PR, we will create comment threads when the editor is being opened, and dispose the comment threads when the editor is being closed. The logic is the same as what we have in PR List View
When we create/update/delete comments, we need to update all visible review/pr comment threads and all workspace file comment threads.

The challenge here is we now moved to push mode from previous pull mode. It means we need to manage the state/cache of comment threads. It can be simple if we only need to support file scheme but life is life, we need to support pr, review and file. So if you run into weird bug, look into how we cache comment threads, usually that's where the bug is from.
The code architecture can be improved by moving the visible editors change listener to PRDocumentCommentsProvider instead of registering in PRNode and remove any pr related code from ReviewDocumentCommentsProvider but we don't need to do that now.

Thanks for the clear explanation! I've done some more testing and found some more issues that need to be fixed, seems like a lot of them are upstream.

Without checking out a PR, open a file and leave a comment. Switch to another file and then back, comment is gone
Leave some text in a comment without submitting it, switch to another file and do the same. Switch back, the pending comment text is overwritten
If clicking the gutter icon quickly, mutiple empty comment threads can be created
No way to remove empty comment threads
Toggling reaction on something I haven't reacted to tries to remove the reaction and errors out
Checkout a PR, then return to master. No commenting ranges are shown on any file opened from the tree after that


Example with using statement fails to compile and run

Example with using statement fails to compile and run
In the Standard Numeric Format Strings article, the example in The General ("G") Format Specifier fails to compile. Error messages indicate that the culprit is the using System; statement.
If the using System; statement is removed, the code successfully compiles but throws an OutOfMemoryException.
This behavior has been reported in two issues in the dotnet/docs repo: dotnet/docs#11250 and dotnet/docs#11413.
Where did this error occur?

docs.microsoft.com

What kind of error was it?

using System statement can't be compiled.
Removing using System statement throws OutOfMemoryException.

Screenshots
See dotnet/docs#11413

@jonsequitur @LadyNaggaga can we please prioritize this? It's breaking some of our interactive tutorials experience.

@mairaw - @jonsequitur opened an issue here and is having convo with @BillWagner

The issue here is that this content was written to run against Try .NET's C# scripting backend, which allows both method declarations and top-level statements. It is now running under our Blazor runner and being inlined into a Main method for you, with normal C# language rules. This then leads content that already contains a Main to fail to compile.
You can fix this sample by removing the wrapping method, so that it looks like this:
double original = 0.84551240822557006;
var rSpecifier = original.ToString("R");
var g17Specifier = original.ToString("G17");

var rValue = Double.Parse(rSpecifier);
var g17Value = Double.Parse(g17Specifier);

Console.WriteLine($"{original:G17} = {rSpecifier} (R): {original.Equals(rValue)}");
Console.WriteLine($"{original:G17} = {g17Specifier} (G17): {original.Equals(g17Value)}");

// The example displays the following output:
//     0.84551240822557006 = 0.84551240822557: False
//     0.84551240822557006 = 0.84551240822557006: True


Prerelease versions should use "alpha.5" instead of "alpha005" format

Picking an arbitrary number of leading zeros as part of an alphanumeric identifier isn't leveraging the semver rules and limits readability and comparability. See below for some relevant sections and emphasized sentences relevant to this.
From semver.org

9.A pre-release version MAY be denoted by appending a hyphen and a series of dot separated identifiers immediately following the patch version. Identifiers MUST comprise only ASCII alphanumerics and hyphen [0-9A-Za-z-]. Identifiers MUST NOT be empty. Numeric identifiers MUST NOT include leading zeroes. Pre-release versions have a lower precedence than the associated normal version. A pre-release version indicates that the version is unstable and might not satisfy the intended compatibility requirements as denoted by its associated normal version. Examples: 1.0.0-alpha, 1.0.0-alpha.1, 1.0.0-0.3.7, 1.0.0-x.7.z.92

11.Precedence refers to how versions are compared to each other when ordered. Precedence MUST be calculated by separating the version into major, minor, patch and pre-release identifiers in that order (Build metadata does not figure into precedence). Precedence is determined by the first difference when comparing each of these identifiers from left to right as follows: Major, minor, and patch versions are always compared numerically. Example: 1.0.0 < 2.0.0 < 2.1.0 < 2.1.1. When major, minor, and patch are equal, a pre-release version has lower precedence than a normal version. Example: 1.0.0-alpha < 1.0.0. Precedence for two pre-release versions with the same major, minor, and patch version MUST be determined by comparing each dot separated identifier from left to right until a difference is found as follows: identifiers consisting of only digits are compared numerically and identifiers with letters or hyphens are compared lexically in ASCII sort order. Numeric identifiers always have lower precedence than non-numeric identifiers. A larger set of pre-release fields has a higher precedence than a smaller set, if all of the preceding identifiers are equal.

You are right as far as semantic versioning is concerned, but the leading zeros are a necessity because of limitations of NuGet (NuGet parses everything after the dash as alphanumerical, which is not in compliance with clause 11 of semver 2.0.0).
I suggest we don't change anything, as we don't want to get into the situation where -alpha.10 is considered lower in version than -alpha.2
(Also see Top10 NuGet patterns)

@AArnott You are quoting semver 2.0.0. The naming used to date is based on limitations in semver 1.0.0 (which is also the version @vweijsters is indirectly referencing). Previously NuGet did not support semver 2.0.0. That said:

Which version(s) of NuGet support semver 2.0.0?
While many users installing this package only work in Visual Studio 2015, that is not a requirement. It can be installed in projects that support Visual Studio 2010+, and will simply "light up" when the project is opened in 2015. As such, nuget restore for Visual Studio 2010+ cannot fail due to our naming convention.

nuget restore should always work. That doesn't do any version comparison -- it just gets exactly the version that was installed previously.
The good question though to discuss is whether NuGet's Install-Package and Update-Package commands will correctly interpret which package version is the 'latest'.
I'm very surprised to hear you say that you support VS2010. These are Roslyn analyzers we're talking about so how can they be expected to work on anything earlier than VS2015? I suppose if someone installs the Roslyn toolset and jimmy-rigs their older MSBuild projects to run the Roslyn compiler and wire up the custom analyzers than maybe it would work, but that's not a VS scenario--that's a command-line build scenario and as such seems less likely to be hurt by a possibly malfunctioning Update-Package command.
Now as to whether any version of the NuGet client properly supports this portion or version of semver, I haven't actually confirmed. I know it deviates in several ways from semver so this might be one of them. And I agree, if NuGet misbehaves we should stick to the alpha005 format you're already using. I'll ask the NuGet team and report back here.

@AArnott The analyzers only run if you edit with VS 2015 or build using those tools. Consider the OpenStack.NET project, where I added custom code analysis features to make daily work easier for users working with VS 2015. The analysis is also enabled in the automated build, but none of this impacts the ability for users to load this solution in Visual Studio 2010 through 2013 (2012+ recommended due to the test suite's use of async/await). However, since the analysis packages are listed in the packages.config files, the version numbers listed there have to work properly with versions of NuGet that are available/installed in these older environments.

The NuGet team reports that modern clients still only support semver 1.0.0. So I'll close the issue.


[Blog Comments] The Story of My Game - 12/29/2015

https://tadukooverse.github.io/blog/2015/12/29/the-story-of-my-game.html

page removed


[BUG] concurrent: LockShouldWithTryFinallyRule 规则 pmd

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

编写到一半不小心按到回车发出去了..重写.


Encryption for config files

Per customer commentary, certain industries prefer/require encryption of the configuration files.  Should we add a facility to do so?  Perhaps an override for 'Encryption/Decryption strategy' where the default is no encryption?

If you do that I'd suggest making it so that you can save binary files using this too. Right now if you notice there is a separate tools menu item for that. You could consider changing the format of the pcfg file to have more header info (besides just version) and then treat the rest of the file as encrypted, or binary, or pseudoxml.
Was this encryption on local drive, or only once deployed to the cRIO? If just on deploy, that might be an easier place to add it.


luaSpriteMakeGraphic error in 0.6.2

Describe your problem here.
Soo, i used lua script that says "luaSpriteMakeGraphic". I made it black, but why does this is white and transparent?

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
I edited only main menu state for remove some buttons and credit state to make it looks like a gallery

hxcpp issue, use git
haxelib git hxcpp https://github.com/HaxeFoundation/hxcpp/
Also, please, use makeGraphic
if you need to know more lua functions, you can look at this wiki: https://github.com/Meme1079/PsychWiki/wiki/Lua-Script-API

luaSpriteMakeGraphic is deprecated and you're using an unsupported version of Psych Engine.


Fix acknowledge abuse download of unflagged files

Currently, when the --acknowledge-abuse flag is given, plexdrive can only download abused files, files not flagged for abuse fail to download with the error "invalidAbuseAcknowledoment".
This PR tries to fix that by only adding the acknowledgeAbuse=true parameter to the download URL, if the --acknowledge-abuse was given and the download request failed with the "cannotDownloadAbusiveFile" error.
The drawback is that we do twice as many download requests for abused files, but a better solution would require to cache the abuse state of a file in the drive object metadata inside cache.bolt, which would require a lot more code changes.
Unrelated to the fix I've also moved the global acknowledgeAbuse option flag to be passed directly to the Downloader instance, instead of repeating it in each Request struct.

Do not merge yet, I'm waiting on feedback in the discussion of this issue in #402. I could only test the case for files which aren't flagged for abuse so far, since I don't have any flagged files.

@Moodkiller Not sure why the download of the flagged file doesn't work for you, even though --acknowledge-abuse is specified. Does it work with plexdrive 5.2.1 and --acknowledge-abuse?
I think you also need to be the owner of the file, to be able to download it.

Would it also be possible to add a logic that eventually skips / fails the download all together? Use case: The user hasn't specified --acknowledge-abuse but the file is flagged on drive.

If the download fails you get the "Could not read object ... / StatusCode: 403" error, but that is for each chunk. Plexdrive doesn't remember that something failed and that's not easy to change, since we would have to add some kind of blacklist of files that should instantly fail.


Add vscode via vendor package.

While #4010 resolves some incompatibilities with 1.60.0, it's still in review and needs additional testing. This PR updates lib/vscode to use a yarn package pointing to our VS Code fork. Once this is merged, any remaining fork clean up branches will be ported over to the vendored .
Fixes #3917
Notable changes from upstream

The log service now defers initialization.
Several terminal channels were added.
Several type definitions were adjusted.

Our changes

Remove asar linking.
Remove module lint check.
Use yarn for vscode vendoring.

How to review this PR
This is a somewhat unusual PR since we don’t yet have a diff to start with, however, I’ve made a branch on our fork with the previous changes form lib/vscode. This allows you compare all the changes from upstream, as well as the adjustments to our code to fix them in 1.60.0
https://github.com/cdr/vscode/compare/1.57.1-initial...cdr:code-server
Follow up
#4136 can be merged after this. They are only separated to make reviewing the changes easier to follow.

Remove asar linking.

Happy we're doing this. Those npm changes upstream were not fun to debug so glad we can avoid that.

Seems to be running just fine locally!
I do see this error but guessing extension related (not your changes):

Weird...a reload of code-server fixes it 🤷♂
Search for files and the terminal aren't working :(  I triedcd lib/vscode && yarn again but that didn't fix it.
https://user-images.githubusercontent.com/3806031/132900766-f0387543-838b-4bc6-ac59-941b59d0b422.mov

What am I missing? Do I need to manually delete node-modules?

The terminal process failed to launch: A native exception occurred during launch (The module '/Users/jp/Dev/code-server/vendor/modules/code-oss-dev/node_modules/node-pty/build/Release/pty.node' was compiled against a different Node.js version using NODE_MODULE_VERSION 89. This version of Node.js requires NODE_MODULE_VERSION 83. Please try re-compiling or re-installing the module (for instance, using npm rebuild or npm install).).

Yeah I tried deleting node_modules in lib/vscode and running yarn install but that didn't help.

Check the Node.js version that's used by Electron

I see the docs say this but I'm not sure how to check that

Oh no...I think I broke something. So I ran cd vendor && yarn install --modules-folder modules --ignore-scripts and now when I go back to the root and run yarn watch I get this error:
yarn run v1.22.10
$ VSCODE_IPC_HOOK_CLI= NODE_OPTIONS='--max_old_space_size=32384 --trace-warnings' ts-node ./ci/dev/watch.ts
[tsc] [11:59:26 AM] Starting compilation in watch mode...
[vscode] $ npm-run-all -lp watch-client watch-extensions
/bin/sh: npm-run-all: command not found
error Command failed with exit code 127.
[vscode] info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
vs code watcher terminated unexpectedly
killing vs code watcher
killing tsc
killing watch
error Command failed with exit code 127.

I'm wondering if I should not use --ignore-scripts when developing locally (i.e. running yarn watch).

@jsjoeio,
I recommend doing yarn clean to completely erase your modules, then yarn from the project directory. Vendor modules will automatically be installed and yarn watch can be run. However, if you encounter that terminal error, your node version differs from a built package and you’ll have to do the following:
 cd vendor/modules/code-oss-dev
 nom rebuild

From there, yarn watch should behave correctly.

Oh man... yarn clean didn't work.  Going to try the second approach

I'm using Node v14.17.0 if that helps. I wish I knew how to check which Node version corresponded to version 83 vs 89

I'm going to delete the code-server repo, reclone and try again

Still no luck. I'm going to blame my Node version. What version are you using @TeffenEllis ?

With regards to the Node module version errors, that might be because the code-oss-dev module we are pulling in has .yarnrc files? They tell Node to target Electron 13.1.8 instead of whatever version of Node you are running.

       v14.15.0   (LTS: Fermium)
       v14.15.1   (LTS: Fermium)
       v14.15.2   (LTS: Fermium) (83?)
       v14.15.3   (LTS: Fermium)
       v14.15.4   (LTS: Fermium)
       v14.15.5   (LTS: Fermium)
       v14.16.0   (LTS: Fermium)
       v14.16.1   (LTS: Fermium)
->     v14.17.0   (LTS: Fermium) (89)
       v14.17.1   (LTS: Fermium)
       v14.17.2   (LTS: Fermium)
       v14.17.3   (LTS: Fermium)
       v14.17.4   (LTS: Fermium)
       v14.17.5   (LTS: Fermium)
       v14.17.6   (Latest LTS: Fermium)

Video
✅ Besides Asher's requested changes, everything looks good on my side
https://user-images.githubusercontent.com/3806031/132926155-b4d1a513-fc6d-473c-9ad7-0bf7a8799bc0.mov

codecov/patch Failing after 1s — 33.33% of diff hit (target 64.12%)

I thought I disabled this in a previous PR but apparently I didn't 😅 Here is PR to fix that.
Feel free to rebase after that gets merged in.
https://github.com/cdr/code-server/pull/4159


New template

This branch is being used to keep track of the work in progress for the new template.
Did you update the documentation accordingly (yes/no)?
no
Did you read the CONTRIBUTING document (yes/no)?
yes
Summary
This is still a draft.
Details and comments

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: ismirlia:x: priti-ashvin-shah-ibmYou have signed the CLA already but the status is still pending? Let us recheck it.


Runtime Comments

Two things to consider
- generic node object type for runtime-created comments
- object may either be of the type UHeartComment_Text or UHeartComment_Image. Both would be stored in the same heart node, it's up to widgets to distinguish them.
- Copy editor authored comments into runtime as a UHeartComment_Text, and synchronize edits.

The first part of this was completed awhile ago, the second doesn't make sense really, so this is done now


encrypt and sign mail from keybase

When I receive a notification mail from keybase it is neither signed nor encrypted.
These mails may contain meta data that I don't think should get exposed.
Thanks
Jens

That would be a great addition.


Tools_v0.8.1 _08: Trails are not visible due to messed bubbles

Release: v0.8.1
Testing Link: http://<IP_ADDRESS>:8080/tools/
Module: Tools page (Bubble Chart)
Description:
Already reported on  /gapminder-tools-vizabi/issues  as :
https://github.com/Gapminder/gapminder-tools-vizabi/issues/56
But it needs to be logged here.

They are actually working but hidden below many other bubbles =) Roll the time slider a bit further and you can see it

Solution: the user created a mess by setting bubble sizes to a variable similar in many countries
let the user decrease the opacity of the non-selected:

or the max size of bubbles.

then the thing gets visible!
@k6Labz please not reopen :-)


SAK-23873 - Problem with site manage quoting

Works for me locally.
What's really bad here is that there's nothing in the logs and there's no way to do a tool reset because there's no portal code displayed.

Nice find. LGTM

It looks to me like portal buffers code and prob scans for presence of HTML head tags.... so the HTML head tag in JS in site-manage-tool/tool/src/webapp/vm/sitesetup/chef_site-siteInfo-list.vm causes confusion?

Merged to get site-info back working again.

Thanks, rebuilding trunk now.


Fix mosdepth to 0.3.3 to resolve --d4 issue

mosdepth 0.3.4 was compiled without --d4. Likely the best solution is to fix 0.3.3 until the conda package is updated.

Thanks @aewebb80! I'll merge this once the tests run.

Happy to help @cademirch! The Kocher Lab at Princeton is planning on using  snpArcher as our GATK pipeline, I'll let you know if I catch anything else.

Happy to help @cademirch! The Kocher Lab at Princeton is planning on using  snpArcher as our GATK pipeline, I'll let you know if I catch anything else.

Great to hear! Let us know how it works for y'all.


Parameters array of objects BUG

How to use fully JSON multi depth object in request?
swagger: '2.0'
info:
  title: Bug array of objects
  description: 'Example: Parameters array of objects BUG'
  version: "1.0.0"
consumes:
  - application/json
produces:
  - application/json
paths:
  /someurl:
    post:
      parameters:
        - name: objects
          in: formData
          description: Array of questions @todo Array of objects
          required: true
          type: array
          items:
            type: object
            properties:
              title:
                type: string
      responses:
        200:
          description: An array of survey

Not a valid parameter definition

There's no support in the spec for complex objects in formData parameters.

Useless tool then...

Huh?  Ok, feel free to file a ticket against the specification.  Beats just complaining.


Automatic Field Resolver (High Order Field Resolvers)

I've been noticing that many field resolvers derivated from relations use to be simple. Simple enough to be able to resolve them using a simple query with TypeORM query builder. It automatically creates the query based on the relation decorators of fields of entity objects.
This is an example:
@Resolver(Post)
export class PostResolver {

    @FieldResolver()
    public author(@Root() post: Post): Promise<Person> {
        return getManager()
            .createQueryBuilder()
            .relation(Post, "author")
            .of(post)
            .loadOne();
    }

}

What I'm trying to say is: I think we should be able to resolve relation fields automatically using TypeORM query builder so that we don't have to code a lot of simple field resolvers. I think Vesper has this feature.
At least I think it would be amazing if we could somehow create high order field resolver dynamically, so we can create base resolvers that creates all field resolvers depending on the entity.
Maybe something like this:
export function createBaseFieldResolver<T extends ClassType, X extends ClassType>(
    field: string,
    arrayField: string,
    classType: T,
) {

    @Resolver({ isAbstract: true })
    abstract class BaseFieldResolver {

        @FieldResolver({ name: `${field}` })
        protected async singleFieldResolver(@Root() single: T): Promise<any> {
            return getManager()
                .createQueryBuilder()
                .relation(classType, `${field}`)
                .of(single)
                .loadOne();
        }

        @FieldResolver({ name: `${arrayField}` })
        protected async arrayFieldResolver(@Root() multiple: T): Promise<any[]> {
            return getManager()
                .createQueryBuilder()
                .relation(classType, `${arrayField}`)
                .of(multiple)
                .loadMany();
        }
    }

    return BaseFieldResolver;
}

The problem is that this only works for two fields, and I don't find any way to programmatically create a field resolver function for each relation field depending on the class type. Is it possible?
I'm new to TypeGraphQL, so please be patient. And sorry for my bad english.

I think that this will be covered by #44

I think that this will be covered by #44

I've been looking that issue. So the plan is resolving field automatically developing a typeorm helper plugin using a dataloader so queries are much less and optimized? Correct me if I'm wrong please.

It uses TypeORM entities metadata to create resolvers for joining relations.

Yes, it's very nice. I was using Vesper before, but I changed to use TypeORM + TypeGraphQL because of that double decorator entity definition and auto schema generation that I fell in love with. Great job by the way!
However, I miss automatic resolver functionality. I hope that comes soon!
What about being able to create high order dynamic field resolver? I suggest adding a @Field handler option so that handler can create dynamically it correspondent resolver and many others.
Let me explain the idea, entity will look like this:
@ObjectType()
@Entity()
export class Post extends BaseEntity {

    @Field(() => Int)
    @PrimaryGeneratedColumn()
    public postId: number;

    @Field(() => Int)
    @Column()
    public authorId: number;

    @Field()
    @Column()
    public name: string;

    @Field(() => Person, { handler: singleFieldResolver })
    @ManyToOne(() => Person, (person) => person.posts)
    @JoinColumn([
        {
            name: "authorId",
            referencedColumnName: "personId",
        },
    ])
    public author: Person;

    @Field(() => [Person], { handler: arrayFieldResolver })
    @ManyToMany(() => Person)
    public mentioned: Person[];
}

And this would be the high order resolvers that would create their resolvers:
        protected async singleFieldResolver(field: string, @Root() single: T): Promise<any> {
            return getManager()
                .createQueryBuilder()
                .relation(classType, field)
                .of(single)
                .loadOne();
        }

        protected async arrayFieldResolver(field: string, @Root() multiple: T): Promise<any[]> {
            return getManager()
                .createQueryBuilder()
                .relation(classType, field)
                .of(multiple)
                .loadMany();
        }

Again, I don't know even if this or similar is possible.

This should work:
@Resolver(of => Post)
export class PostResolver extends createBaseResolver(Post) {}

For multiple relations, you would need something like this:
@Resolver(of => Post)
export class PostResolver
  extends relationResolver(Post, "field1",
 	relationResolver(Post, "field2",
  		relationResolver(Post, "field3", class {})
  )
) {}

You would need to use the mixins pattern #359.

Closing for a housekeeping purposes 🔒


Fix various typos, repeated words, align some spelling to LDP.

partially revamped from #16712

fall thru -> fall through
time stamp -> timestamp
file name -> filename
host name -> hostname

Only fix a few low level =>low-level to reduce the PR extent.
Fix a few pointer compare test nits, while fixing some typos.
Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

I may submit a PR to backport documentation fixes only, if you want.

I may submit a PR to backport documentation fixes only, if you want.

Feel free to submit it.

Ping @openssl/commiters, needs a 2nd reviewer.

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

Merged, thanks for the contributions.


Remove duplicated mFrustumData clear in void DrawHelper::clear()

I've looked DrawHelper over, and it does not look like any of the member Vector instances was skipped over, so mFrustumData.clear() was just redundant.

Indeed, it was totally redundant. Merged.


RNG-179: Add fast loaded dice roller discrete sampler

Codecov Report

Merging #112 (926da96) into master (f97c0f8) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             master     #112    +/-   ##
==========================================
  Coverage     99.71%   99.72%
- Complexity     1294     1336    +42
==========================================
  Files           148      149     +1
  Lines          5231     5383   +152
  Branches        477      504    +27
==========================================
+ Hits           5216     5368   +152
  Misses           11       11
  Partials          4        4

Impacted Files
Coverage Δ

...ling/distribution/FastLoadedDiceRollerSampler.java
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Coverage increased (+0.006%) to 99.796% when pulling 926da9678a93dedb8a550ef96651797597d09d22 on aherbert:feature-RNG-179 into f97c0f8de15cbe5606c45c6ef5739d4ecd7cf821 on apache:master.


See reference, offset, (proportion?) for pegged and liquidity shapes

Story
As a When looking at a list of my orders (that include either pegged or liquidity shapes)
I want to see the Reference and Offset for each part of a pegged/liquidity shape order (as well as other information shown in order table)
So that I can confirm the status of my liquidity/pegged shape and decide if I wish to leave it, amend, or cancel
Language note: one summits a "pegged shape" that results in an order for each price+side combo in the shape. The submission of this whole shape is also often referred to as a pegged order (even when it results in multiple orders)

[ ] Decide what to do with proportion

Acceptance Criteria
See The Manage orders Acs Section on shapes. specifically 7003-MORD-017 & 7003-MORD-016. It contains some "shoulds" that could be considered along side this work and some context around the user task.

[ ] I can see the price reference used for that order (e.g. bid, mid, ask)
[ ] I can see the offset for that order (formatted with market.quote decimal places)
[ ] I can see the direction of that order (long/short)

Tasks

[x] Explore and sketch
[ ] Team and stakeholder review
[ ] Visual Design
[ ] Team review
[ ] Etc.

Sketch
https://www.figma.com/file/Vnf1XhB4p6Dlg9AnxbdE32/Untitled?node-id=0%3A1&t=LGWfisJuvce55GQi-1
Additional details / background info

Some initial thoughts for discussion here:
https://www.figma.com/file/K49Nx7XfocShhHcwwQzQkm/Console-V2?node-id=3854%3A37007

Updated designs here: https://www.figma.com/file/K49Nx7XfocShhHcwwQzQkm/Console-V2?node-id=3862%3A37354

But lets discuss this properly

Closing this ticket. use this ticket instead https://github.com/vegaprotocol/frontend-monorepo/issues/3373


multiple routers/services entries no longer working for file provider yaml tcp routes

Do you want to request a feature or report a bug?

Bug

What did you do?
I upgraded a working traefik 2.2.11 deployment to 2.3.6.
Part of the configuration are templated tcp routers:
conf/dynamic/tcp-lb1-list.yaml
{{$ports := list "30022" "30122" "30143" "30222" "30322" "30422" "30432" "30522" "30622" "31432" "31532"}}
tcp:
   {{range $i, $p := $ports}}
   routers:
      tcp-lb1-{{$p}}-router:
         entryPoints:
            - "lb1v4t{{$p}}"
            - "lb1v6t{{$p}}"
         rule: "HostSNI(`*`)"
         service: "tcp-lb1-{{$p}}-service"
   services:
      tcp-lb1-{{$p}}-service:
         loadBalancer:
            servers:
               - address: "<IP_ADDRESS>:{{$p}}"
               - address: "<IP_ADDRESS>:{{$p}}"
               - address: "<IP_ADDRESS>:{{$p}}"
   {{end}}

What did you expect to see?
A set of working tcp routes from template expansion, like in 2.2:

What did you see instead?
In 2.3 only one route is present:

Output of traefik version: (What version of Traefik are you using?)

Version:      2.3.6
Codename:     picodon
Go version:   go1.15.6
Built:        2020-12-17T16:34:27Z
OS/Arch:      linux/amd64

What is your environment & configuration (arguments, toml, provider, platform, ...)?

linux amd64
file provider
docker

Doing some further investigation without templating, I think the template expansion works. v2.3 does no longer accept multiple routers and services -- so the template loop needs to be pushed one level down.
Not sure if this counts as a bug. It's a breaking change for such configs from 2.2 to 2.3, however.

Hello @schwebke,
Thanks for your interest in Traefik!
Since the templating is not the cause of your issue, can you rephrase the title?
Also, can you provide a short example of a configuration without templating that works with v2.2 and not with v2.3?

Hi @rtribotte,
here a minimal example resulting in two tcp routes with 2.2, and only one in 2.3:
tcp:
   routers:
      tcp-lb1-30022-router:
         entryPoints:
            - "lb1v4t30022"
         rule: "HostSNI(`*`)"
         service: "tcp-lb1-30022-service"
   services:
      tcp-lb1-30022-service:
         loadBalancer:
            servers:
               - address: "<IP_ADDRESS>:30022"
               - address: "<IP_ADDRESS>:30022"
               - address: "<IP_ADDRESS>:30022"
   routers:
      tcp-lb1-30122-router:
         entryPoints:
            - "lb1v4t30122"
         rule: "HostSNI(`*`)"
         service: "tcp-lb1-30122-service"
   services:
      tcp-lb1-30122-service:
         loadBalancer:
            servers:
               - address: "<IP_ADDRESS>:30122"
               - address: "<IP_ADDRESS>:30122"
               - address: "<IP_ADDRESS>:30122"

Written like this it does not look good with the duplicate routes/services keys -- using templates it was nice and concise...
I've also attached a minimal docker example to replicate this:
lb.tar.gz

Thank you for your quick feedback, it's clearer now.
Since the yaml format must be JSON "valid", duplicates key are illegal.
Yet, you are right that in v2.2 duplicate keys were allowed in the dynamic configuration, thus, this was not intended. It was too permissive.
In the v2.3 version of Traefik, the parsing of the dynamic configuration has evolved and relies now on the paerser library, which not allows duplicate keys. In fact, only the last key applies.
You need to remove the duplicate keys to make your configuration work again.
Unfortunately, I'll close this issue as a question since it's an intended change.


Getting Stalled Error during test run.

What App Center service does this affect?
App center xamarin uitest test runs
Describe the bug
Getting "Current test status: Stalled error (contact support for more info)" message when trying to run tests.
Expected behavior
The same tests were running without any error before and expecting the same.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS] Apple iPhone XS Max (12.3.1)
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6] Apple iPhone XS Max (12.3.1)

Additional context
Add any other context about the problem here.

@avanti-deodhar it's faster to contact our support through the blue chat icon in the App Center portal.


Railway yard and former train stations

Key points:

Add preset railway yard;
Add validation for train station with disused=yes or abandoned=yes and offer fix.

Railway yard preset
Railway yard is a station where freight trains stop. Some are former train stations. If passenger trains also stop, the mapper should use railway=station instead.
It should be mapped as a node at the center of the yard with the recommended tags:

name={string}
operator={string}
railway:ref={string}

Links:

https://wiki.openstreetmap.org/wiki/Tag:railway%3Dyard
https://wiki.openstreetmap.org/wiki/OpenRailwayMap: Freight Yards
Example of railway yard on OpenRailwayMap: "Coppermill Junction"

Currently iD doesn't recognize a node with railway=yard, so it displays a blank geopin and the default railway panel.
Validation and fix
I have found railway=station + disused=yes or abandoned=yes where there was a train station that was closed and repurposed as freight-only station. This is not only wrong but also might be a tagging for the renderer (example). In these cases the validation could help update the tags (example of yard after manual fix).
Please, validade railway=station + disused=yes as improper or outdated tagging and offer the possible fixes:

If freight trains still stop: replace it with disused:railway=station + railway=yard;
If freight trains no longer stop either: disused:railway=station.

The same validation and fix could be applied accordingly to abandoned=yes or demolished=yes. In both cases it's recommended to keep the other tags, such as name and ref.
Regards,
Igor

Thanks for implementing.
A small refinement: if disused=yes + railway=station + public_transport=station is found, then:
- disused=yes
+ disused:railway=yes
- railway=station
- public_transport=station <--- this one is missing.
Once iD replaces railway=station with disused:railway=yes, the public_transport=station is left behind. I believe some mappers added public_transport=station to former train stations because railway=station tag alone requires it.


Mobile prevent leaving input on clear

Closes
✅ Pull Request Checklist:

[ ] Included link to corresponding Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

Fix it 😈


Fix pbr shader compiliation error, #version has to be in the first line

I've had problems with compiling and running the pbr example:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Compilation("glslang_shader_preprocess:\nInfo log:\nERROR: 0:40: \'#version\' : must occur first in shader \nERROR: 0:40: \'#version\' : bad profile name; use es, core, or compatibility \nERROR: 0:40: \'#version\' : bad tokens following profile -- expected newline \nERROR: 3 compilation errors.  No code generated.\n\n\nDebug log:\n\n")', crates/bevy_render/src/pipeline/pipeline_compiler.rs:161:22

I've checked each shader, and only one shader hasn't had #version in the first line.
This change fixed my issue.

from https://www.khronos.org/opengl/wiki/Core_Language_(GLSL)#Version

The #version directive must appear before anything else in a shader, save for whitespace and comments. If a #version directive does not appear at the top, then it assumes 1.10, which is almost certainly not what you want.

so it should still work, strange that's an issue in your case?
anyway, doesn't hurt to have it first

bors r+


merge {tcp,udp}/message to message

Codecov Report

:exclamation: No coverage uploaded for pull request base (v3@cb5107a). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff          @@
##             v3     #357   +/-   ##
=====================================
  Coverage      ?   70.21%
=====================================
  Files         ?       66
  Lines         ?     5453
  Branches      ?        0
=====================================
  Hits          ?     3829
  Misses        ?     1228
  Partials      ?      396

Continue to review full report at Codecov.

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


Fix subLayers being affected by superLayer html

A sub-layer of a super-layer with Layer#html content set is not positioned correctly in the parent, as it is pushed down by the content. See this example.

Thanks man!

No prob! Thanks for merging :)


Doc Block Updates

I did something strange with the pull requests trying to merge them into one.. ended up in two parts..


Update torch pinned version due to CVE vulnerability warning

A small version update for the pinned version of torch/torchvision to address a critical severity CVE warning.

LGTM, thanks. Was this the CVE?

Yep!


Support for Materialize.css

https://github.com/Dogfalo/materialize seems to be a worthy competitor to bootstrap and support for materialize layouts would be great

Happy to add it if you can provide the implementation.


draft: add third_party/rebel

WiP

I don't know what this is about

Can you elaborate (sorry for clicking the wrong close button)

Can you elaborate (sorry for clicking the wrong close button)

Sorry, I wanted to upload it to my forked repo. It was by mistake.


Document what exceptions may be raised

I'd like to be able to write something which will catch any exceptions raised when performing a request, so I can handle them appropriately.  However, I'm not sure which exceptions might get raised; looking at the code, I think urllib{,2,3} exceptions can occur, as well as ones from requests.exceptions.
Ideally, I'd like to be able to do try:... except requests.exceptions.RequestException and have that catch any exceptions due to making the HTTP request; this could probably be accomplished by monkeypatching the urllib exceptions to add RequestException as a base class, though maybe that's too ugly. Some documentation of what I need to catch, though, would be a good start.

:+1:

@bitprophet has some further thoughts on connection issues that I may or may not agree with.

Yea, thoughts like "man, that @kennethreitz is such a jerk!". ;)
A weak Monday morning attempt to recapture my IRC arguments:
There are, IMO, two types of failure modes in this kind of library: network failures, such that you never connect to or hear back from the HTTP server; and HTTP failures, such that you successfully get back a non-200 response.
Kenneth's default is to suppress exceptions, return a valid Request object, and require the user to explicitly ask about exceptions via Request.raise_for_request. (IIRC he does this for both types of failures, but had a thought to create a 2nd Request.raise_* method so users could tell the failure types apart.)
My opinion is that this is un-Pythonic and surprising; somebody new to the library but not new to Python will be confused upon getting a successful-but-empty Request object, even if the server was down/DNS was down/bad URL/etc.
Furthermore, one will get a "farther down the road" traceback when trying to use the Request object as if it had succeeded, e.g. using its .content or .raw attributes, which is more confusing and harder to debug than an obvious, at-connection-time NetworkError or whatnot.
Such error suppression needs to have a serious, obvious benefit to offset the negative effects of forcing them to appear in strange ways farther down the stack. I don't see such a benefit here, other than Kenneth's noted use case of making N requests in a row/simultaneously, and not wanting connection errors to abort the entire run.
In that situation, my opinion is one should use an opt-in, "please suppress errors, I will check for them myself" setting, and that it should not be the default behavior.

I'm leaning more towards this myself.

Interesting.
FWIW, I certainly expected exceptions to be raised automatically; didn't even think to look for raise_for_* methods!  +1 to @bitprophet's suggestion of making this behaviour opt-in.
Making it easy to catch all possible exceptions raised by Requests (without doing an "except:" thing) would be helpful, though.  Either by some fiddling to rebase the exceptions (so I can do "except Requests.Error:") or by some nicer technique if you can come up with one...

I never did really finish my thoughts; I should point out that most of the above relates to the 1st category of errors, i.e. "did not receive a real HTTP response".
When actually receiving a non-200 response, or when one successfully connects but gets some sort of garbage back from the server (though that's kind of a gray area) I think the question of whether to raise or depend on the user to check, is not as clear cut.
I still think raising an exception would not be a bad thing (again with the "fail fast" philosophy) but I could see the argument for telling the user "you got some response; but you need to make sure it's the right kind."
~fin~

To be clear: this discussion is limited only to network and protocol exceptions.

I'm new to the Python scene, but I agree with @bitprophet. Requests that have no chance of bringing back any meaningful data should fail fast and fail loudly, especially when it comes to malformed URLs.


fix: Engine initialization did not respect OS env

Feature or Bugfix

Bugfix

Detail
Engine initialization did not respect OS env. When importing awswrangler, engine.register gets called. This function looks at only the value of the name parameter (None in this case) or the output of engine.get_installed().
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 456cc3fb5f4fbeb786d275e4967ac61d5d96e9b0
Result: SUCCEEDED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 456cc3fb5f4fbeb786d275e4967ac61d5d96e9b0
Result: SUCCEEDED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: fbb7947a47f4e7a0af42d9f5eb938e9505216e7e
Result: SUCCEEDED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 8aae9605f897dcac4154fc563dfa027ef3236e48
Result: SUCCEEDED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 8aae9605f897dcac4154fc563dfa027ef3236e48
Result: FAILED
Build Logs (available for 30 days)

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


[FEATURE REQUEST] Output doesn't contain enough info!

Is that possible to save output simply as on terminal results, I use |tee -a for more visual inspection. But the issue is that cat file doesn't contain the URL but just the paths this hard for later use (no ffuf banner like extensions) , -v wasn't convenient either.
Thank you for giving us FFUF.

Hi @alexerxx
Have you tried save it in a file?
https://github.com/ffuf/ffuf/issues/364#issuecomment-752736960

Hi @alexerxx, you can use -v to get the full URL.
Closing this by now, please re-open if you still have questions related to it.


Inconsistent builder method prefixes

We have a lot of builders in our API, and most of them define setters without any prefixes, so for example to set a message they would have MyBuilder message(Text message). This is currently inconsistent in our API, for example the CommandSpec.Builder defines them with a set prefix. We should decide for one style and then use it for all builders in the API consistently. We could basically choose from one of these options:

no prefix at all (as currently), e.g. builder.message(Text message).build()
set-prefix, e.g. builder.setMessage(Text message).build()
with-prefix, e.g. builder.withMessage(Text message).build()

I say remove set and with.

I prefer builders without a prefix for setters.

Same, I prefer without the set prefix.

I agree, no set prefix.

sure, not java-enterprise-verbose but works for me


Put information about stub files in the documentation

What is the request about
Describing in a documentation what to do, so your module has hints shown by IDE and types checking tools.
Reasoning
I have just started using pyo3 to create a Python package that I can use in my Python application. It took me some time googling on how to make the hints and typing which is a very common need in my opinion. The simple "How to" would be a nice improvement to the documentation and help newbies, especially that even although my code is working I am not sure if I have found the best solution.
Details of proposed solution
There are two issues I have found that are related to this topic:

https://github.com/PyO3/pyo3/issues/1112
https://github.com/PyO3/pyo3/issues/510

As far as I understood, the best way right now is to write the *.pyi files manually and just attach it to the package (with maturin it is easy).
I would be glad to help with this part of documentation, but I am not sure if above statement is completely true or we can somehow use comments/docstrings/signatures in rust code to make it happen.

Thanks, some additions to the guide would be very welcome. You're correct that the best solution at the moment is to maintain .pyi files manually. It would be awesome to have #510 implemented some day, however it's a future feature!


5.4.x

PHP 5.4, MySql 5.6

Hi thank you for pull request. I will take a look at this at the weekend.

Update and ship it

resolved merge conflict in php54 branch
I'm running a test build , when it works, I will do the php.ini fixes by my self, otherwise we wont go anywhere here.

test ok


Python input() function causes EOF error

The error occurs immediately when input() is called.

I want this fixed soon

I want getpass to work as well

I cannot reproduce it. Does it happen everytime?

Yes. It happens when input() is called from a script in the scripts directory, but not in the interactive console.

Actually, it does work if I enter the interactive console then exit before running the script.

But some other issues occur even if input() works.

Fixed by 152742beaf9bea1cc092d911e4831ddd87204cf9

yay


Removed references to Bitbucket

Replaced with Github. Updated links

Sorry about the confusion.  @john-shepherdson Are you handling these fixes?

Yes, see PR 121. Please wait for Jenkins job to run to ensure build has not
broken again.
John Shepherdson BSc (Hons) MSc CEng MIET
CESSDA Platform Delivery Director

T +44(0) 1206 872001
E @.*
*ORCiD *https://orcid.org/0000-0002-4402-9644

UK Data Archive
University of Essex

Legal Disclaimer: Any views expressed by the sender of this message
are not necessarily those of CESSDA ERIC or the UK Data Archive.
This email and any files with it are confidential and intended solely for
the use of the individual(s) or entity to whom they are addressed.
On Thu, 14 Sept 2023 at 10:16, Joshocan @.***> wrote:

Sorry about the confusion. @john-shepherdson
https://github.com/john-shepherdson Are you handling these fixes?
—
Reply to this email directly, view it on GitHub
https://github.com/cessda/cessda.guidelines.public/pull/117#issuecomment-1719071864,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZVEPS54QZG4QAOWM5BFTU3X2LDO3ANCNFSM6AAAAAA4WY342I
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add all versions of node/electron & use semver

Sorry for one big commit. Need major bump.
I changed getTarget to getTargets. Swap runtime and target/abi. Made arguments for functions mandatory (I think that clear is better than unclear).
node.json/electron.json can be updated with npm run update

any feedback?

not a fan of the breaking api changes. i already have a bunch of upstream deps.

personally i like the simplicity of the module as is. this seems to up the complexity a bit, but that's @lgeiger's call.

@fanatid Sorry for not getting back to you and thanks for taking the time to contribute. The updating of the json list is quite interesting!
Though I totally agree with @mafintosh in that I'm not a fan of the breaking changes and like the simplicity of of the module as it is now.
We have a few upstream packages depending on node-abi which would all need to be modified. And this change doesn't add benefit to any of them.
What is your use case where you need all the available targets (Why getTarget to getTargets)?

Thank you for feedback... I also don't like major changes. I'll try split this improvement for small parts. For beginning what you think about all versions node/electron in json files (and update script)?

Moved to #21


Suggestion for Assitant bot skills deployment steps

Project
https://github.com/Microsoft/AI/blob/4.4/docs/tutorials/assistantandskilldeploymentsteps.md
It wasn't clear after installing PowerShell core how you should run it. As doesn't show in the Start Menu after install. I had to launch via File Explorer where it was installed
C:\Program Files\PowerShell\6
Also there is a step to sign into az login that the script fails on if not done before. Worth adding as a step first.

Thanks - after install pwsh.exe is your path so Windows+R and pwsh should find it but I'll emphasise that in the docs.
az login is optional for most azure developers as they have already done this and we wanted to keep the instructions as simple as possible. Will review. Thanks!


Mismatch of roles after update

Description - What happened? *
We observed a inconsistency when the admin changes a users role. The role-definitions do not get updated in all relevant areas at the same time (see screenshot). I can understand, that it may be needed to log in again to finalize the change of a a users role. However on this I would prefer the process of the last version, where a user got automatically logged out when his role was changed as this did not lead to inconsistencies.
Expected Behavior *
/
Observed Behavior *
/
Steps to Reproduce
No response
Context Information
No response
Relevant log output
No response
Screenshots

Duplicate of #111


Update connector-azure-sql-database.md

Added a note on how ADF implicitly copies data into identity columns without the need of any one to execute and pre script

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

You should consider using the preCopyScript to set the identity insert option.
@linda33wj to look at this PR as well.

Hey @dcstwh ,
There is no need to explicitly have a pre script. ADF implicitly executes it and copies data into an identity column.

Right now, we do not have implicit logic to turn on/off identify.
Thank you for the feedback.  At this point, we do not plan to include this tip.
#please-close

@linda33wj - do comment as appropriate before we close this issue.

Hey @dcstwh ,
This is not a feedback. The tip which I have included is what is happening in ADF as of today.
ADF copies the data into an identity column in sink implicitly without we adding any pre script for Azure SQL database or synapse.

At this point, we do not plan to include the tip. Thank you.


google.api_core.exceptions.PermissionDenied: 403 IAM permission 'dialogflow.sessions.detectIntent' on 'projects/None/agent' denied.

I'm using flask as a dialogflow webhook !!

Duplicate of https://github.com/googleapis/dialogflow-python-client-v2/issues/180


New Folder "dialog" canceled

I accidentally clicked the checkbox to prevent new folder dialogs, and now I cannot create new folders. How can I restore this ability? I reset the settings to default, but that did not seem to work. Thank you.

After rebooting, I'm now able to create new folders.


Add unity scaffolds

Closes #???
Description

For contributor use:

[ ] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[ ] Code follows the standards mentioned here
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

Codecov Report

Merging #887 (1abeada) into master (39e521d) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #887   +/-   ##
======================================
  Coverage    2.61%   2.61%
======================================
  Files           5       5
  Lines         420     420
======================================
  Hits           11      11
  Misses        408     408
  Partials        1       1

Flag
Coverage Δ

unittests
2.61% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Remove payroll_run_id from Claim

I've also had to delete the specs for the data migration, because the column is no longer around. It's served its purpose though now.

 Remove payroll_run_id column on claims table


Documentation Inadequate

Hi,
Please make the documentation more partner-friendly. The current info is inadequate and creates a lot of confusion. You should update the PAL FAQ and documentation from the below document.
https://aka.ms/palfaq (found thru internet search)
There is no way for a customer or even Partner to verify where the PAL is actually applied in the Azure Portal (except the PowerShell cmdlet). There should be a simple report in the Portal itself showing Customer / Partner to which resources a partner MPN is applied. It is sad to see that you are releasing half-hearted features, thus creating more complications that consume much of our time in operational activities, instead of spending more face time with customers. The Partner Center Insights reporting has its own flaws, access issues, etc.
Look it from a Partner Perspective, where the partner business is spread across the Globe (Global SIs), where the project team has no motivation to use PAL. Why would you make such a complex system and push partners to do more?

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

ID: 5f901056-c225-b086-cdaa-c8bd978db64c
Version Independent ID: 9c067eb4-54cb-e141-d82f-e136ff3c671e
Content: Link an Azure account to a partner ID
Content Source: articles/billing/billing-partner-admin-link-started.md
Service: billing
GitHub Login: @dhirajgandhi
Microsoft Alias: banders

Assigning content owner @dhirajgandhi for visibility, review and document enhancement considerations.

Hello - The partner can see the list of customers and subscriptions in the partner center.  Due to security requirements, only partner admins can list of all the subscriptions. The partner admins typically go to partner center to manage their activities.  This is similar what DPOR has.

Thank you for your feedback, We will investigate the addition to current documentation and update.

Please update the last sentence: Note that you'll need to do this in your service provider tenant. For simplicity, we recommend creating a service principal account in your tenant that is associated your MPN ID and granting it Reader access to every customer you onboard. - In the example the RBAC Reader role is used and this is one of the roles that not eligible for PEC. Read Roles and permissions for the partner earned credit. https://query.prod.cms.rt.microsoft.com/cms/api/am/binary/RE3QuW2

How often does the 'My Insights Dashboard' get updated?  It seems stale.  Thanks!!

Updated article with https://github.com/MicrosoftDocs/azure-docs-pr/pull/113755
#please-close


feat: Add remark function and tests

Add remark functionality.

closing this as we're not implementing remark field anymore


Fix/332 WebUser using previous identity after disconnecting and connecting with a different one

The disconnect method was not cleaning the profileConnection instance property that is used to get the profile.

Can you also add a placeholder CHANGELOG entry with no date or release number

Done.
Not sure how the release works without the develop branch. Is it ok to merge on main?


Configure the room to calender.

Configure the room to calender.

this issue is replaced with a more specific one, closed.


Feat: Support Security Group custom description

Description
Just a small update to be able to customize the security groups description for both gitlab runner and docker machine instances.
Migrations required
NO
Verification
Verified on runner-docker example
Documentation
Documentation updated in runner-docker example
Please ensure you update the README in _docs/README.md. The README.md in the root can be updated by running the script ci/bin/autodocs.sh, requires terraform-docs version 0.8+

@pandarouxbsd thanks for your contribution 👍

@pandarouxbsd thanks for your contribution 👍


Ignore unique ID in bundles test

Request criteria

[X] I searched the zppy GitHub Discussions to find a similar question and didn't find it.
[X] I searched the zppy documentation.
[X] This issue does not match the other templates (i.e., it is not a bug report, documentation request, feature request, or a question.)

Issue description
#527 introduced a unique ID to file paths so that it would be easier to run jobs launched from different zppy branches concurrently. However, it appears that unique ID is being included in the bash content test. That needs to be ignored. I tried adding -I '{unique_id'} but that doesn't seem to be working. (In the example below, test_main_20231116 is the unique_id).
$  python -u -m unittest tests/integration/test_bundles.py
--- /lcrc/group/e3sm/ac.forsyth2/zppy_test_bundles_output/test_main_20231116/v2.LR.historical_0201/post/scripts/bundle1.bash       2023-11-16 19:16:20.766862000 -0600
+++ /lcrc/group/e3sm/public_html/zppy_test_resources/expected_bundles/bundle_files/bundle1.bash 2023-11-06 19:08:21.905791000 -0600
@@ -5,7 +5,7 @@
 #SBATCH  --job-name=bundle1
 #SBATCH  --account=e3sm
 #SBATCH  --nodes=4
-#SBATCH  --output=/lcrc/group/e3sm/ac.forsyth2/zppy_test_bundles_output/test_main_20231116/v2.LR.historical_0201/post/scripts/bundle1.o%j
+#SBATCH  --output=/lcrc/group/e3sm/ac.forsyth2/zppy_test_bundles_output/v2.LR.historical_0201/post/scripts/bundle1.o%j
 #SBATCH  --exclusive
 #SBATCH  --time=07:00:00

@@ -20,7 +20,7 @@
 fi

 # Script dir
-cd /lcrc/group/e3sm/ac.forsyth2/zppy_test_bundles_output/test_main_20231116/v2.LR.historical_0201/post/scripts
+cd /lcrc/group/e3sm/ac.forsyth2/zppy_test_bundles_output/v2.LR.historical_0201/post/scripts

 # Get jobid
 id=${SLURM_JOBID}

From the man pages for diff:
       -I, --ignore-matching-lines=RE
              ignore changes where all lines match RE

It's probably not catching it because the unique_id only appears in actual, not expected. The -I line worked for templateDir in other tests, because templateDir was indeed found in both (just set to different things). You need to run -I on the pattern that stays the same between actual and expected.


[interop][SwiftToCxx] Skip emitting an accessor method if its name collides with an accessor method of another property

Resolves rdar://119835520 and rdar://119835571.

@swift-ci please smoke test


Refactor products to use polymorphic relationships

Broke out manage_iq_products
Broke out team_member_products
Moved provisioning logic into product
Added test coverage

 Code Climate has analyzed this pull request.

I'll look at that failed Travis build.

https://github.com/projectjellyfish/ux/pull/250

Looks like you guys prefer a few different styles than we do. Fixing now.

There are is an example of a spec in this PR as well. This will also get us some test coverage around order items.

This LGTM. @stackus or anyone else?

:+1: for me

@jntullo ?

Yeah I looked it over this morning. :+1: There might be a gremlin in there but I don't see it.


Bluetooth extended advertising not work on STM32WB55

Describe the bug
I'm noticed that extended advertising not work on STM32WB55. It is tested on Nucleo-WB55 board with Zephyr 3.6.0 and stm32wb5x_BLE_HCILayer_extended_fw.bin V1.18.0 on radio CPU2.
If I using normal advertising it is work.
Warning is produced calling function "bt_hci_cmd_send_sync" at "bluetooth/host/adv.c" line 1214.
And exactly at "bluetooth/host/hci_core.c" line 333.
status = cmd(buf)->status;
if (status) {
LOG_WRN("opcode 0x%04x status 0x%02x", opcode, status);
net_buf_unref(buf);
Logs and console output
Here is part of log:
0:00:00.025,000]  blecpp: BLE device INIT
[00:00:00.025,000]  blecpp: Start BLE app
[00:00:00.051,000]  bt_hci_core: Identity: 02:80:E1:00:00:00 (public)
[00:00:00.051,000]  bt_hci_core: HCI: version 1.0b (0x00) revision 0x8077, manufacturer 0x0030
[00:00:00.051,000]  bt_hci_core: LMP: version 1.0b (0x00) subver 0x2177
[00:00:00.051,000]  bt_hci_core: opcode 0x2036 status 0x01
[00:00:00.051,000]  blecpp: Advertising ext. create failed(err -5)
Environment (please complete the following information):

OS: (MacOS aarch64)
Toolchain (Zephyr SDK)
Zephyr V3.6.0

[00:00:00.051,000] bt_hci_core: opcode 0x2036 status 0x01

This is the error "Unknown command", check that you have enabled support in the controller or contact your controller vendor.

This feature is not implemented. See https://github.com/zephyrproject-rtos/zephyr/issues/51430

@erwango, thank you for answer. Sorry for not search open issues.
Since there is no date when this features will be implemented I have a question.
Is it possible to implement under Zephyr RTOS STM32 Bluetooth SDK?
In short. Using "stm32wb5x_BLE_Stack_full_extended_fw.bin" on CPU2, use source code from STM32CUBE_WB SDK and integrate into Zephyr app. Is there any guides yet?

In short. Using "stm32wb5x_BLE_Stack_full_extended_fw.bin" on CPU2, use source code from STM32CUBE_WB SDK and integrate into Zephyr app. Is there any guides yet?

No, but here you'll find a discussion about preliminary work. You could ping guys there to know what is their status and if they're willing to collaborate and upstream a solution with your help.


I have updated my swift version from 3.2 to 4.2 and start getting error in api calling.

I am getting error after updating my swift versions from 3.2 to 4.2. Error is "RxAlamofireDomain error -1". Some time it getting successful and most of the time getting this error.

@Mobifyi Can you try and get request/response logs from both versions (3.2, 4.2)?

Hello @Mobifyi
I have the same issue so have you got any solution regarding this?
if you have solution then please help me.
Thank you


setup.py hardcoded to linux tensorflow

Fails to install when I try it on Mac
Complete output from command python setup.py egg_info:
tensorflow-1.6.0-cp35-cp35m-linux_x86_64.whl is not a supported wheel on this platform.
Traceback (most recent call last):
File "", line 1, in
File "coach/setup.py", line 63, in
shell=True)
File "/anaconda3/lib/python3.6/subprocess.py", line 291, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['pip install https://anaconda.org/intel/tensorflow/1.6.0/download/tensorflow-1.6.0-cp35-cp35m-linux_x86_64.whl']' returned non-zero exit status 1.

Hi,
Currently Mac is not officially supported by Coach. It was only tested on Ubuntu 16.04.
Nevertheless, I don't see a reason for it not to work. The tensorflow version being installed is dependent on your machine. If no GPU is available (which is probably the case with the Mac), the Intel Optimized Tensorflow version is used, otherwise, tensorflow-gpu is installed by default.
Could you please try changing 'pip install https://anaconda.org/intel/tensorflow/1.6.0/download/tensorflow-1.6.0-cp35-cp35m-linux_x86_64.whl' in setup.py to 'pip install tensorflow==1.6.0', and see if it helps resolving the issue?

Yes, this works. I reported the issue as I thought it was hard-coded to linux by mistake.

Sure. Thanks a lot for commenting that out.
Officially supporting Mac is planned to be added in the future.


How get Child control elements of "ToolBar Windows" use FromPoint function?

If you try to get "AutomationElement" when clicked on "Windows TaskBar" using "UIA3Automation.FromPoint()", the "FlaUi" returns different values, either "ToolBar" or "Pane" or "Button". How can you get all child "ToolBar" controls?

Can you show in FlaUInspect/Inspect what exactly you want to get and what you get?

I'll close this due to inactivity. Please re-open if you have more information.


fix(Sleek): update theme

fixes height of playbar when listening on a different device

I honestly cant seem to understand why you're having the issue, but the existing code works for everyone else. Are you fully up to date with everything? Can you send a new screenshot of the issue where i can see all of the bottom.

@harbassan I believe I am fully up to date on everything, my Spicetify is v2.16.1 and my Spotify client is <IP_ADDRESS>6.gc5f8b819. I have downloaded the latest version of the theme repository, followed the setup instructions, and ran spicetify update. My OS is Zorin OS 16.2.
This is what my Spotify looks like with the latest version of the pushed theme changes:

Ah, I see the real issue now. For some reason your connect bar isn't visible, which is the thing thats supposed to push the whole bottom bar upwards. Do you have some sort of extension/snippet enabled that hides the connect bar?

I guess I must have been, none of my extensions seemed like they were having an effect, but when I did a hard reset and wiped all marketplace content it was back to working all of a sudden!
Thanks for your help, closing this out now.


'root_encrypted' -> bool

PR o'clock
Description
Follow-up PR after #761 failed as a result of encrypted key requiring a boolean
Checklist

[x] Change added to CHANGELOG.md. All changes must be added and breaking changes and highlighted
[x] CI tests are passing
[x] README.md has been updated after any changes to variables and outputs. See https://github.com/terraform-aws-modules/terraform-aws-eks/#doc-generation

@craig-rueda can you please update your branch.


IWW Issue #29, December 1st 🙀 2020

This pull request is for gathering links for the December 1st issue of the Immersive Web Weekly newsletter for the W3C Immersive Web Community Group.
If you have a link to a news article, event, artwork, or project that is directly related to the immersive web then we welcome your help!
To submit a link comment below using this template:
title: Example title of link
url: https://example.com/
author: Alice Smith
content: A short description of what is at the link and why it's related to the immersive web

Please keep in mind that comments here and all submitted links are public!
We keep each issue pretty short so we won't immediately use every submission but we keep track of them all and they may appear in a later issue.
Deadline
IWW goes out each Tuesday morning Pacific time.
Trevor will edit each issue on the Monday before (in this case that Monday is November 30th) so submissions after that will roll over for use in future issues.
If you need editorial support then email @TrevorFSmith<EMAIL_ADDRESS>

title: xrpet.me, WebXR hand inputs demo
url: https://www.xrpet.me/
tweet url: https://twitter.com/Tojiro/status/1333218545262444544?s=20
github: https://github.com/toji/xr-critter
author: Brandon Jones
content: "Pet me!" is a WebXR experience where you pet a cat-thing.

title: WebXR Scene Optimization
url: https://toji.github.io/webxr-scene-optimization/
author: Brandon Jones
content: A mini-tutorial covers some simple methods of optimizing 3D assets, especially for WebXR on mobile VR and AR devices, using free tools like Blender.

title: New game: Sol Attrition
url: https://twitter.com/wwwtyro/status/1332809571249893380
author: Rye Terrell
authorLink: https://twitter.com/wwwtyro/

title: Makepad
url: https://makepad.dev/
author: Rik Arends
authorLink: https://twitter.com/rikarends
content: A live-coding environment with VR support
https://github.com/makepad/makepad

title: Immersive AR Emulation
url: https://github.com/adarosecannon/immersive-ar-emulation
author: Ada Rose Cannon
authorLink: https://mobile.twitter.com/Lady_Ada_King/
content: >

  This is a project designed to allow you to test your immersive-ar session in a VR headset it acts kind of like a polyfill, if immersive-ar is already supported then it does nothing. But if immersive-vr is supported and immersive-ar it will let it work by:

In emulated immersive-ar draw a background scene behind your scene
Modifying navigator.xr.isSessionSupported to override requests for immersive-ar to let it be allowed.
Modifying navigator.xr.requestSession to return an immersive-vr session if an immersive-ar session is requested which has been modified such that:
It supports requestHitTestSource by testing against the rendered background scene
It modifies requestAnimationFrame so that each XRFrame returned has a replacement getHitTestResults


Add  targetUri in stackdriver_grpc_service

Add  targetUri in stackdriver_grpc_service
TODO: Make this an  option, so it can be configured
Signed-off-by: gargnupur<EMAIL_ADDRESS>[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ X] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Can we do bootstrap.validate() or test it with rc2?

@mandarjog : Yes, adding a test in bootstrap validation, in the meantime, can we get this pr going as the whole cherry-picks take a lot of time?

In response to a cherrypick label: new pull request created: #24953


Legacy Facebook adapter should be updated to use bot-specific config

One of the major limitations of the legacy Facebook adapters it is lack of support for multi-tenant bots.
In large part, this is because the spawned bot objects from this adapter use a adapter-wide global configuration option (configuration.access_token being the primary culprit) instead of a local config option (bot.config.access_token)
This would require a small change to the defineBot factory function to mix the global vars into the local config.

possible fix in https://github.com/howdyai/botkit/pull/1683


Feature: Voice Message Transcription

Automatically process voice message and provide transcriptions.

Added in https://github.com/NateShoffner/lanco-discord-bot/commit/75eadf76d681e1347b1f71bfc890ac22544e6fe8.
Currently limited to CPU-only (which is probably fine given most hosted instances won't have it anyway) because of Torch and OpenAI Whisper having conflicts.


Porting Mac native library to Maui

Will your tool help with that?

Hi, @420tech
Which Mac native library are you going to use? Creating a Mac Catalyst binding library may require a macbook, but I don't have one at the moment. You can tell me which Mac native library you need to use, I will create the Mac binding class library when I have a MacBook.

It is our own sound controller. We are migrating everything to Maui and we made this library our native Mac app.
Get Outlook for iOShttps://aka.ms/o0ukef

From: ZhangChi @.>
Sent: Thursday, January 5, 2023 5:20:02 PM
To: realZhangChi/MauiBinding @.>
Cc: Tim Belvin @.>; Mention @.>
Subject: Re: [realZhangChi/MauiBinding] Porting Mac native library to Maui (Issue #3)
Hi, @420techhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2F420tech&data=05|01||24788602a8584365fa3508daef84232b|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638085648051301052|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=eqbTq8Rn5aLWTZxZuK5rdpcw1u%2Fgg7FagkxmaThE084%3D&reserved=0
Which Mac native library are you going to use? Creating a Mac Catalyst binding library may require a macbook, but I don't have one at the moment. You can tell me which Mac native library you need to use, I will create the Mac binding class library when I have a MacBook.
—
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FrealZhangChi%2FMauiBinding%2Fissues%2F3%23issuecomment-1373010866&data=05|01||24788602a8584365fa3508daef84232b|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638085648051301052|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=G1wHtJFeCULQNFTYxLSAeiV3UJDr78Zk2zopV0rWR8E%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADLR3UK6LYPH6LYDHN6HZULWQ5XMFANCNFSM6AAAAAATSI6UPA&data=05|01||24788602a8584365fa3508daef84232b|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638085648051301052|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=7j6rLvd3n7gyXZNFhP2bJvDI2OzoJlx4FmkebIjhL1Q%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

I'm sorry that this project may not be of much help, you can use the dotnet new maccatalystbinding command to create a Mac OS binding class library for Maui, and refer to the xamarin documentation to bind your native library.

Thanks!
Get Outlook for iOShttps://aka.ms/o0ukef

From: ZhangChi @.>
Sent: Thursday, January 5, 2023 6:19:55 PM
To: realZhangChi/MauiBinding @.>
Cc: Tim Belvin @.>; Mention @.>
Subject: Re: [realZhangChi/MauiBinding] Porting Mac native library to Maui (Issue #3)
I'm sorry that this project may not be of much help, you can use the dotnet new maccatalystbinding command to create a Mac OS binding class library for Maui, and refer to the xamarin documentationhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flearn.microsoft.com%2Fen-us%2Fxamarin%2Fcross-platform%2Fmacios%2Fbinding%2F%3Fcontext%3Dxamarin%252Fmac&data=05|01||18a7a1a9151d4b15e1c908daef8c80ba|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638085683978785608|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=yQ2CDCnXQA9U1vy9%2FkhDJfD4dEvtHhrKmEAYRZkVM7E%3D&reserved=0 to bind your native library.
—
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FrealZhangChi%2FMauiBinding%2Fissues%2F3%23issuecomment-1373055247&data=05|01||18a7a1a9151d4b15e1c908daef8c80ba|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638085683978941805|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=xXb1zPNlnNFHxJUcjUBUpT3X252OzEyziVJFkK0bWnA%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADLR3UN7OEVCVJ6EHSAPUD3WQ56MXANCNFSM6AAAAAATSI6UPA&data=05|01||18a7a1a9151d4b15e1c908daef8c80ba|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638085683978941805|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=oO2JyopHy%2FZZZ6lBvwfPHF4i2eaIPSUPBZWZYs27mW0%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>