New package: BellSoft.LibericaJRE.19.Full version 19.0.2+9

Pull request has been created with Komac v1.11.0 :rocket:

Resolve https://github.com/microsoft/winget-pkgs/issues/113719

Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-Domain


PinnedMessage error

"""module for processing pinned scraped messages"""
import constants.node_constants as nc
import constants.scraper_constants as sCons
from messages.fragments.content import Content
from messages.message import Message

class PinnedMessage(Message):
    """class for taking a pinned message and converting it to a
    message object for consumption"""

    def build_message(self):
        """builds a pinned message for output from a given livechat
        pinned message"""
        self.content_node = self.action[nc.ADD_BANNER_NODE]
        content = self.content_node[nc.BANNER_RENDERER_NODE][nc.LIVECHAT_BANNER_RENDERER_NODE]\
            [nc.CONTENT_NODE][nc.LIVECHAT_TEXT_MESSAGE_RENDERER_NODE]
        header = self.content_node[nc.BANNER_RENDERER_NODE][nc.LIVECHAT_BANNER_RENDERER_NODE]\
            [nc.HEADER_NODE]
        self.time_stamp = content[nc.TIMESTAMP_USEC_NODE]
        self.author = ''
        self.occurrence_timestamp = content[nc.TIMESTAMP_SIMPLE_TEXT_NODE][nc.SIMPLE_TEXT_NODE]
        content = header[nc.BANNER_HEADER_RENDERER_NODE][nc.TEXT_NODE]
        self.context_message = {
                "pinned_by" : self.runs_message_builder(content),
                "message" : content[nc.MESSAGE_NODE][nc.RUNS_NODE][0][nc.TEXT_NODE]
        }

    def generate_content(self):
        """generates pinned message output"""
        return Content(self.occurrence_timestamp, self.time_stamp, self.author, self.context_message, sCons.MESSAGE_TYPE_PINNED).object_output()

Needs to be modified like so,

"""module for processing pinned scraped messages"""
import constants.node_constants as nc
import constants.scraper_constants as sCons
from messages.fragments.content import Content
from messages.message import Message

class PinnedMessage(Message):
    """class for taking a pinned message and converting it to a
    message object for consumption"""

    def build_message(self):
        """builds a pinned message for output from a given livechat
        pinned message"""
        self.content_node = self.action[nc.ADD_BANNER_NODE]
        content = self.content_node[nc.BANNER_RENDERER_NODE][nc.LIVECHAT_BANNER_RENDERER_NODE]\
            [nc.CONTENT_NODE][nc.LIVECHAT_TEXT_MESSAGE_RENDERER_NODE]
        header = self.content_node[nc.BANNER_RENDERER_NODE][nc.LIVECHAT_BANNER_RENDERER_NODE]\
            [nc.HEADER_NODE]
        self.time_stamp = content[nc.TIMESTAMP_USEC_NODE]
        self.author = content[nc.AUTHOR_NODE][nc.SIMPLE_TEXT_NODE]
        self.occurrence_timestamp = content[nc.TIMESTAMP_SIMPLE_TEXT_NODE][nc.SIMPLE_TEXT_NODE]
        pinned_by = header[nc.BANNER_HEADER_RENDERER_NODE][nc.TEXT_NODE][nc.RUNS_NODE][-1][nc.TEXT_NODE]
        self.context_message = {
                "pinned_by" : pinned_by,
                "message" : content[nc.MESSAGE_NODE][nc.RUNS_NODE][0][nc.TEXT_NODE]
        }

    def generate_content(self):
        """generates pinned message output"""
        return Content(self.occurrence_timestamp, self.time_stamp, self.author, self.context_message, sCons.MESSAGE_TYPE_PINNED).object_output()

Please notice the pinned_by object addition, I understand there is a function to extract it automatically, but was to lazy to delve to deep in the code and figure out how it worked and decided to do it the easy way in turn. The error came from modifying the object content which at first had the necessary data to get the information for both pinned_by and message keys however since it got rewritten with different data in the original code it results in an error where it can't find the requested child data for message key.
Finally added self.author data to extract authorName

ahhhh good find,  i have no idea why content was pointed to the header, I think it should be fixed now.


DataGrid to Pandas DataFrame

Hello,
I have not found a way to transform the data from DataGrid to Python DataFrame.
Would appreciate if someone could share any solution for this.
Thanks

Isn't simply datagrid.data what you need?

Thank you for your quick reply.
I have a Pandas DataFrame and then use ipydatagrid to edit my data. When finished editing my data I would then like to Pandas DataFrametransform it back to a df.
Is there a solution for this?

Then what I just suggested works :)
Please look at this example: https://github.com/jupyter-widgets/ipydatagrid/blob/main/examples/CellEditing.ipynb
You'll see that datagrid.data provides the dataframe updated after edits

Thank you. I will have a look at it.

Thank you @martinRenou
Your line of code worked like a charm :)
Feel bad that I didn`t get it when I was looking at this example initially some weeks ago. Haha


Can't generate some charts in sketch 52.5 chart 4.6.0

I purchased chart plugin and I follow the instructions on the repo to for example to generate a pie chart:
Pie Chart, Donut Chart — categories: 1, items: 5 (categories count should be equal to canvases count);
I select one canvas(circle) and enter 1 and 5 and I only get 1 piece of the pie chart?
I can produce a bar chart but all the bars are the same color  and same for most of the others.
Now that you are charging for the plugin you may consider improving your instructions. Thank you.

@jz-nbme can you send me a screenshot of SetParams popup? It seems to me that your color palette has only one color.

@jz-nbme the problem here with a number of items. You have 4 colors in your palette but you want to generate a pie chart with 5 slices. You should add more colors to the palette.
Regarding the bar chart, all bars will have the same color. You can try to create a bar chart with categories: 2 and items: 10, for example.

Yes that seems to be working now, thanks.


generate BUILD dependency file

Hi, how you generate BUILD dependency file in the vendor directory?

I used gazel: https://github.com/mikedanese/gazel :D
Gazel seems to work well except for if a library is vendoring packages. PR #4 tracks most of the work I went through to get it working with this package

Thanks for the answer. I have found https://github.com/bazelbuild/rules_go/tree/master/go/tools/gazelle like too does the job.


student profile edit

Can you please squash these commits? This might be helpful: http://code.dblock.org/2015/08/31/getting-out-of-your-first-git-mess.html


Logo changes if size changes

I would like to embed the 16px upload icon, which works when I do
{% octicon "upload" height="16" %}

I would like to embed it at a larger scaling, e.g.
{% octicon "upload" height="40" %}

However, If I do this, I get the alternate logo with the bar on top rather than below.
Is this desired behaviour? Can I suppress it and get the smaller logo at a bigger scaling?

I would like to embed the 16px upload icon, which works when I do
{% octicon "upload" height="16" %}

I would like to embed it at a larger scaling, e.g.
{% octicon "upload" height="40" %}

However, If I do this, I get the alternate logo with the bar on top rather than below.
Is this desired behaviour? Can I suppress it and get the smaller logo at a bigger scaling?

Hmm, this is desired behaviour, as the icon with the bar on top is the 24px variant.
Whenever specifying a height, the module picks the best variant. What I mean by this is any icons with a height of more than 16px will default to the 24px variant.
I was not aware that there were icons with these types of differences between variants. I'm considering adding a new parameter variant or something similar that can be used to override this behaviour then.
Do you know if GitHub's helper libraries do anything like this?
Thanks for the heads up.

Hey,
thanks for your reply :)
As all of them are vector graphics, I'm a bit unclear on what constitutes a better version for one or the other size. I guess level of detail could be a thing, but in this case it seems kind of random to me. For me at least, I usually pick a specific icon I want to use, so I'd rather have more control than the convenience of the library automatically picking. That might also really give undesired behavior e.g. in the difference between mobile and desktop versions of the site.

Do you know if GitHub's helper libraries do anything like this?

I'm not sure what you are referring to :/

As all of them are vector graphics, I'm a bit unclear on what constitutes a better version for one or the other size. I guess level of detail could be a thing, but in this case it seems kind of random to me.

This is correct, but it's not random, however I see your point.

For me at least, I usually pick a specific icon I want to use, so I'd rather have more control than the convenience of the library automatically picking. That might also really give undesired behavior e.g. in the difference between mobile and desktop versions of the site.

Agreed, actually.
I was referring to the official Ruby and Javscript libraries as this library is meant to be similar.
Solution
I'll push out a new release soonish, allowing you to manually choose which variant you want to use, in your case:
{% octicon "upload-16" height=40 %}
Since you want the 16px variant.

@Aypac I've added this functionality, to use it, please upgrade to v1.1.0.
Overall I think it's a really positive change. Thanks for your feedback. :+1:
Please consider starring the project if you find it useful! It would also be nice to see what you make with it :smiley:

Thanks a bunch+ ⭐!
Have not had the time to test it yet, but will try it soon!
If I publish it, I will let you know :)


The birthday cake recipe doesnt require candles

Description

Sprite has candles. Recipe no has candles. What gives?
https://github.com/space-wizards/space-station-14/blob/976cae9c785fd8bb4ba1c7476f9a71694250288f/Resources/Prototypes/Recipes/Cooking/meal_recipes.yml#L1468
Reproduction

Screenshots

Additional context

Literally unplayable.
Someone fix this.

unrelated to this but a crafting recipe for candles would be cool too


WIP React refactor phase 2

Phase 2: move react -> shared

[x] Make symbolic link to shared under desktop
[x] move all paths to use shared
[x] Update resolve-assets -> resolve-root

some stuff got lost during the rebase, need to fix it up

[ ] TODO check a build

@keybase/react-hackers this can get looked at again, i need to test a build before we merge but take a peek. let's try and get this in early otherwise we'll have lots of conflicts

reopening against master

new PR: https://github.com/keybase/client/pull/1856


Submitting security notices

Greetings,
My company did a basic security analysis of OpenOnDemand. A minor issue was discovered, but I do not want to disclose it openly. I sent an email to the only team member that I have a contact for, but I've not heard back from them. The security tab for the Github project is also empty providing no assistance in reporting this issue.
What is the proper way to report security issues to the project? Can this be added to the Security tab for these projects please?
Thank you.

On our official project website, http://openondemand.org/, we indicate "If you have security concerns or think you have found a vulnerability in Open OnDemand, please contact us directly via email on the news list linked above".  The list can be accessed at: https://lists.osu.edu/mailman/listinfo/ood-users or directly via<EMAIL_ADDRESS>and emails sent to it are only distributed to the core project team members for moderation.  We'll look at providing this same info in the Security tab.

Ack! I looked right over that when I was on the page. Sorry. But thank you for pointing it out. I will send an email right now.
Thank you for also considering adding that to the Security tab. I will go ahead and close this out now.

Thanks for pointing it out.  I've put this info on the security tab now.

Sorry to reopen, but I got a bounce back from the email<EMAIL_ADDRESS>saying that I can't send emails. It redirected me to discourse.osc.edu which is just as public as this post. :-/

Sorry about that.  We had the list setup to reject messages, thinking we'd be copied on them as moderators.  I've toggled it to hold them instead.  Please try resending your message to the list.

Sent!

I just received an update that my email is being held for moderator approval. If you are comfortable with that, then feel free to close the ticket again. Thank you for your assistance!

Recieved via mailing list and responded.


新增隐私清单文件

https://developer.apple.com/documentation/bundleresources/privacy_manifest_files/describing_use_of_required_reason_api?language=objc

我这边调试无法运行，仅仅新增了一个隐私文件，没有改动任何逻辑代码

感谢PR，已经合并了。无法运行好像是因为IPHONEOS_DEPLOYMENT_TARGET太低导致的，我改为12了，我本地可以运行了

已发布3.8.5版本

你好,我现在已经更新为3.8.5但是通过脚本检查发现这边隐私清单问题还是没有得到彻底处理,下面是分析情况,目前使用的是3.8.5版本
Analyzing TZImagePickerController ...
⚠  Missing privacy manifest file!
API usage analysis result(s): 1
[0] NSPrivacyAccessedAPICategoryFileTimestamp:creationDate:/Users/xxxxxx/Pods/TZImagePickerController/TZImagePickerController/TZImagePickerController/TZImageManager.m
🛠  Descriptions for the following required API reason(s) may be missing: 1
[0] NSPrivacyAccessedAPICategoryFileTimestamp
底下是MJExtension已处理的隐私清单,是正常的.
Analyzing MJExtension ...
💡 Found privacy manifest file(s): 1
[0] /Users/js_zl/Desktop/xxxxxxx/Pods/MJExtension/MJExtension/PrivacyInfo.xcprivacy
API usage analysis result(s): 0
✅ All required API reasons have been described in the privacy manifest.

因为这个隐私清单文件存放在TZ.framework 的bundle目录下，脚本可能会检测不到
---- 回复的原邮件 ----
| 发件人 | @.> |
| 发送日期 | 2024年04月23日 11:04 |
| 收件人 | banchichen/TZImagePickerController @.> |
| 抄送人 | lxm1799 @.>,
Author @.> |
| 主题 | Re: [banchichen/TZImagePickerController] 新增隐私清单文件 (PR #1675) |
你好,我现在已经更新为3.8.5但是通过脚本检查发现这边隐私清单问题还是没有得到彻底处理,下面是分析情况,目前使用的是3.8.5版本
Analyzing TZImagePickerController ...
⚠ Missing privacy manifest file!
API usage analysis result(s): 1
[0] NSPrivacyAccessedAPICategoryFileTimestamp:creationDate:/Users/xxxxxx/Pods/TZImagePickerController/TZImagePickerController/TZImagePickerController/TZImageManager.m
🛠 Descriptions for the following required API reason(s) may be missing: 1
[0] NSPrivacyAccessedAPICategoryFileTimestamp
底下是MJExtension已处理的隐私清单,是正常的.
Analyzing MJExtension ...
💡 Found privacy manifest file(s): 1
[0] /Users/js_zl/Desktop/xxxxxxx/Pods/MJExtension/MJExtension/PrivacyInfo.xcprivacy
API usage analysis result(s): 0
✅ All required API reasons have been described in the privacy manifest.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


[-] Failed to register and start service for the vulnerable driver

I have this problem "Failed to register and start the service for a vulnerable driver". I tried to fix this by reinstalling vs2022, but nothing. My last option is to reinstall Windows, but it's so boring to do so.
Some solution?

Reinstalling windows was my solution


Add a way to update docs

Hey,
I have recently tried updating a doc, but ended up with creating a new one instead, which did have the same name, but the id after the name differed. As such, I had to manually remove the original and then embed the new one. There could be a way to simply overwrite (after confirming such action in a modal) a file that has the same name within a given workspace.
Thank you

i want this too, but keep in mind when you change the doc, you have to re-embed it anyway - so it's going to get a new guid/name.

i want this too, but keep in mind when you change the doc, you have to re-embed it anyway - so it's going to get a new guid/name.

@AntonioCiolino Yes, but the process could be more streamlined, not requiring manual removal of the old doc. It could be an optional switch.

This is something that would be a massive breakthrough in the space (and something we have a solution for) so that you can more easily manage and modify documents in place without a re-embed totally.

@timothycarambat
Is there a solution for this? Whenever I upload the same document per API, or a github repo, it always creates a new file.

I’m so confused. This seems like a feature request. The repo author agreed it is a good idea. They said there was a solution. Then, the issue was closed without outlining what the solution is, or where the feature went in terms of roadmap. A little more context would be appreciated. It would complement the product nicely.

Agreed


[BUG] Rapid json Accept() failure not caught

Describe the issue
When using RapidJson (TINYGLTF_USE_RAPIDJSON is defined), the JsonToString() function does not check the return code from the Accept() function
https://github.com/syoyo/tinygltf/blob/master/tiny_gltf.h#L3095
https://github.com/syoyo/tinygltf/blob/master/tiny_gltf.h#L3099
To Reproduce
This was discovered when we had some bad data (NaN's in some floats) and we were getting partial glTF files with no error codes.  When producing glb's, it would produce invalid glb's with bad json data which was unusable.
Expected behaviour
JsonToString() doesn't have a good way to report a problem.  We changed it to:
    if (!o.Accept(writer)) {
      return "tiny_gltf::JsonToString() failed rapidjson conversion";
    }

in both places.
Screenshots
N/A
Additional context
N/A

I've added a check of Accpet() return code. https://github.com/syoyo/tinygltf/commit/544969b7324cd6bba29f6203c7d78c7ea92dbab0
TODO: Better error reporting.


Feat: ETH plugin

Let me know if that makes sense, if this PR is going in, we can work on making a proper ETH SVG icon.
We could as well make this a generic crypto currency plugin.

Would be happy to include this - would it be possible to make this to accept configuration for multiple tickers, e.g. ["ETH","BTC","VET"] and render then side by side? I'm already working on a way to provide this type of configuration for each plugin

Sure, what about the icons?
On Sun 11 Mar, 2018, 1:33 AM Nick Tikhonov<EMAIL_ADDRESS>wrote:

Would be happy to include this - would it be possible to make this to
accept configuration for multiple tickers, e.g. ["ETH","BTC","VET"] and
render then side by side? I'm already working on a way to provide this type
of configuration for each plugin
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Hyperline/hyperline/pull/147#issuecomment-372061390,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABHi5_lXwCSkJGL2CaIfP1KvRun9f-lks5tdDF0gaJpZM4SlLn2
.


Buttongroup React V3

Description
Related commits:

https://github.com/adobe-private/react-spectrum-v3/commit/c9b37b8f9a5fceb2344303c19837042158f1a812#diff-9f636f1ba4c8f714b94783214246db78

To-do list

[ ] If my change impacts other components, I have tested to make sure they don't break.
[ ] If my change impacts documentation, I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

[x] This pull request is ready to merge.

Already covered in a pr that @lazd is working on


Add official Hatch badge

https://hatch.pypa.io/latest/next-steps/#community

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
40

Relevant Lines:
40

💛  - Coveralls


[BUG] Invalid resource type Microsoft.Web/sites/workflows requesting generic resource with an ResourceIdentifier

Library name and version
Azure.ResourceManager 1.10.1
Describe the bug
Request ArmClient.GetGenericResource(new ResourceIdentifier(stringVar)) where stringVar is the resource ID of a Logic Apps Standard workflow queried by REST API using $"/subscriptions/{subscription}/resourceGroups/{resourceGroup}/providers/Microsoft.Web/sites/{webApp}/workflows?api-version=2018-11-01";. I am certain the resource ID is correct as it is the exact value returned by the REST API, and have corroborated this by using az resource show --ids (the id) which works fine. The ID in question is (anonymised):
/subscriptions/{subscription id - verified correct}/resourceGroups/{rg name - verified correct}/providers/Microsoft.Web/sites/{logic app name - verified correct}/workflows/{workflow name - verified correct}
Expected behavior
The resource is returned
Actual behavior
Exception thrown:
InvalidOperationException: Invalid resource type Microsoft.Web/sites/workflows
Azure.ResourceManager.Resources.ResourceProviderCollection.GetApiVersion(ResourceType resourceType, CancellationToken cancellationToken)
Reproduction Steps
Use ArmClient.GetGenericResource with the ID of any Logic App Standard workflow
Environment
Windows 10
.NET SDK:
Version:           8.0.102
Commit:            64f1bc458e
Workload version:  8.0.100-manifests.3097af8b
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19045
OS Platform: Windows
RID:         win-x64
Base Path:   C:\Program Files\dotnet\sdk\8.0.102\
.NET workloads installed:
Workload version: 8.0.100-manifests.3097af8b
[aspire]
Installation Source: SDK 8.0.100, VS 17.9.34310.174
Manifest Version:    8.0.0-preview.1.23557.2/8.0.100
Manifest Path:       C:\Program Files\dotnet\sdk-manifests\8.0.100\microsoft.net.sdk.aspire\8.0.0-preview.1.23557.2\WorkloadManifest.json
Install Type:              Msi
Host:
Version:      8.0.2
Architecture: x64
Commit:       1381d5ebd2
.NET SDKs installed:
2.2.108 [C:\Program Files\dotnet\sdk]
3.1.426 [C:\Program Files\dotnet\sdk]
6.0.419 [C:\Program Files\dotnet\sdk]
7.0.116 [C:\Program Files\dotnet\sdk]
8.0.100 [C:\Program Files\dotnet\sdk]
8.0.102 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.All 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.25 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.27 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.14 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.16 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 8.0.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 8.0.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.0-rc.2.21480.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.25 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.27 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.14 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.16 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 8.0.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 8.0.2 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.25 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.27 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.14 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.16 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 8.0.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 8.0.2 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Other architectures found:
x86   [C:\Program Files (x86)\dotnet]
registered at [HKLM\SOFTWARE\dotnet\Setup\InstalledVersions\x86\InstallLocation]
Environment variables:
Not set
global.json file:
Not found
Learn more:
https://aka.ms/dotnet/info
Download .NET:
https://aka.ms/dotnet/download

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

HI @TWolversonReply , thank you for your feedback. Regarding your response, I have reconfirmed through Swagger that in the api-version 2018-11-01, there indeed isn't a resource called 'workflows'. However, we will be upgrading the api version of App Service in the near future, at which point we will upgrade to the 2023 api-version. In this version, we have added resources related to workflows. The latest version of AppService (api-version 2021-02-01) indeed does not include content related to workflows. This issue is likely caused by the REST API version and SDK version being out of sync. I believe this issue will be resolved after the new version is released.


(maint) mergeup 5.3.x -> master

Merging this PR is blocked on a green CI run of puppet-agent 5.3.x including puppet at https://github.com/puppetlabs/puppet/commit/d23ff65ebe872aa8df6de547dff3d6896d12feca

CLA signed by all contributors.

We have a green run on puppet-agent 5.3.x including puppet at https://github.com/puppetlabs/puppet/commit/d23ff65ebe872aa8df6de547dff3d6896d12feca
This PR should be ready to merge now.
cc: @geoffnichols , @branan , @McdonaldSeanp

Not seeing this, I opened https://github.com/puppetlabs/puppet/pull/6269, which has up to the latest passing 5.3.x SHA.

👍 @Magisus I am closing this PR.


CR line ending are not well treated

From VINCENT.guillaume.inp on September 20, 2013 05:26:58
What steps will reproduce the problem? 1. get or create a text file (text.txt) with two lines separated by 'CR' character as line ending (Mac OS X ending line).

Go to http://jquery-csv.googlecode.com/git/examples/file-handling.html 3. Open your file, jquery-csv parse file content but separate line on \n char. For exemple csv generated on mac with Excel doesn't work.

jquery-csv should handle csv file with \r or \n or \r\n line ending.
Attachment: text.txt
Original issue: http://code.google.com/p/jquery-csv/issues/detail?id=31

From<EMAIL_ADDRESS>on December 09, 2013 15:21:56
Status: Duplicate
Owner<EMAIL_ADDRESS>Mergedinto: 27


using sum() when there are commas in table cells?

This may be a hard limitation but, I was wondering, is it somehow possible to support math e.g. sum() functions when the numbers contain commas? E.g. see below:
| Item              | Grams |
| ----------------- | -----:|
| Whole Wheat Flour |   110 |
| Bread Flour       |   748 |
| Warm Water        | 1,691 |
| Salt              |    18 |
| Starter           |    40 |
| **Total Grams**   |   917 |
<!-- TBLFM: @>$2=sum(@I..@-1) -->

result:

(should be 2607)

and while doing this, maybe add a setting allowing the user to select what a decimal "point" is 🙈
(for me it's a comma 😆)


kctrl package output when wrong command/input should be consistent

What steps did you take:
kctrl
kctrl helps to manage packages and repositories on your Kubernetes cluster (package, version)

Usage:
  kctrl [flags]
  kctrl [command]

Package Commands:
  package     Package (available, install, installed, repository)

Available/Other Commands:
  help        Help about any command
  version     Print client version

Flags:
      --color                       Set color output (default true)
      --column strings              Filter to show only given columns
      --debug                       Include debug output
  -h, --help                        help for kctrl
      --json                        Output as JSON
      --kube-api-burst int          Set Kubernetes API client burst limit (default 1000)
      --kube-api-qps float32        Set Kubernetes API client QPS limit (default 1000)
      --kubeconfig string           Path to the kubeconfig file ($KCTRL_KUBECONFIG)
      --kubeconfig-context string   Kubeconfig context override ($KCTRL_KUBECONFIG_CONTEXT)
      --kubeconfig-yaml string      Kubeconfig contents as YAML ($KCTRL_KUBECONFIG_YAML)
      --tty                         Force TTY-like output (default true)
  -v, --version                     version for kctrl
  -y, --yes                         Assume yes for any prompt

Use "kctrl [command] --help" for more information about a command.

kctrl: Error: Invalid command - see available commands/subcommands above
❯ kctrl package
kctrl: Error: Use one of available subcommands: available, install, installed, repository
❯ kctrl package get
kctrl: Error: Use one of available subcommands: available, install, installed, repository

What happened:
kctrl without any command, prints help for command and output error, but kctrl package without any subcommand prints only error, and not help message that will help understand subcommand, flags, etc...
What did you expect:
I would expect the help to be provided so that I don't need to fail or have to do an additional explicit -h call to understand usage.
Environment:

kapp Controller version (execute kubectl get deployment -n kapp-controller kapp-controller -o yaml and the annotation is kbld.k14s.io/images): kctrl version 0.34.0
Kubernetes version (use kubectl version)

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

Out of curiosity, which one of these would you prefer? (the more verbose help section, or a more direct error message)

the more verbose output, which aligns with the behavior of most related CLIs like tanzu, pack, and many more in the k8s space. It seems current behavior is common to other Carvel CLIs (kapp at least) :-(
As mentioned, this output makes you issue an additional invocation:

kctrl does not provide any help on error, so requires always an additional invocation:

$ kctrl package available get
kctrl: Error: Expected package name to be non-empty

Docker provides an intermediate behavior where it doesn't print verbose help, but gives enough information to not require --help unless the format of params is unknown. For more complex commands, you still need to issue --help to understand how to use it.

$ docker port
"docker port" requires at least 1 and at most 2 arguments.
See 'docker port --help'.

Usage:  docker port CONTAINER [PRIVATE_PORT[/PROTO]]

List port mappings or a specific mapping for the container

$ docker run
"docker run" requires at least 1 argument.
See 'docker run --help'.

Usage:  docker run [OPTIONS] IMAGE [COMMAND] [ARG...]

Run a command in a new container

pack (and other tools) does provide enough information to not require additional usage.

$ pack builder create
Error: accepts 1 arg(s), received 0
Usage:
  pack builder create <image-name> --config <builder-config-path> [flags]

Examples:
pack builder create my-builder:bionic --config ./builder.toml

Flags:
  -c, --config string        Path to builder TOML file (required)
  -h, --help                 Help for 'create'
      --publish              Publish to registry
      --pull-policy string   Pull policy to use. Accepted values are always, never, and if-not-present. The default is always

Global Flags:
      --no-color     Disable color output
  -q, --quiet        Show less output
      --timestamps   Enable timestamps in output
  -v, --verbose      Show more output

It seems current behavior is common to other Carvel CLIs

we have received feedback before that dumping help text is confusing because it's just too much -- i agree with that assestment. the thinking is that in most cases user just forgot to add a flag/arg -- they know how to do it, but just forgot, or they want to see more targeted suggestion rather than a wall of help text. it would be useful to provide a bit more info in error messages of what is required -- specifically which flag should i provide.

Well, from a UX perspective, there's tradeoffs between usefulness and confusionness. Usefulness is when the tool helps you as it's difficult to use so it guide you to a correct use with the minimal amount of effort. Confusionness is when it gets always in the middle.
If you're an advanced user, you will most likely never miss a command/flag and if you do it will be very few times, so you will be confused very little. If you're a newbie user, you might miss a command/flag many times, so you will be helped a lot.
Sometimes our engineer eyes are not the best eyes to look at things, if we're targeting non-engineer users. It obviously depends on the tools, but from what I understand kctrl is a tool to help users manage (install, update, delete) packages on a cluster, so probably not same users as kapp/ytt/imgpkg users. Although I still find those, even if consistent with this tool, confusing to use.

i think we are saying the same thing but coming to different conclusions:
output below to me is not "helping the user", it just dumps a pile of text. in fact ive seen bunch of times where people when faced with below experience would be confused of what they have done wrong because it just looks like a wall of text.
$ pack builder create
Error: accepts 1 arg(s), received 0
Usage:
  pack builder create <image-name> --config <builder-config-path> [flags]

Examples:
pack builder create my-builder:bionic --config ./builder.toml

Flags:
  -c, --config string        Path to builder TOML file (required)
  -h, --help                 Help for 'create'
      --publish              Publish to registry
      --pull-policy string   Pull policy to use. Accepted values are always, never, and if-not-present. The default is always

Global Flags:
      --no-color     Disable color output
  -q, --quiet        Show less output
      --timestamps   Enable timestamps in output
  -v, --verbose      Show more output

docker cli approach seems like a reasonable design choice that still focuses on the error but also provides just enough context to know what's going on / what they can do to do more.

Still under discussion.


ref navbar links + pricing ver update

ticket
Made navbar content at least more in sync with lob.com's navbar (particularly under the Pricing tab), though the core of the openapi docs's navbar is from docs.lob.com. (Don't know where the js causing the hover-to-pull-down-menu is coming from though)
Put the variables (e.g. {{LOB_ASSETS_URL}} back into templates.hbs so it'll be easier to transition to having those variables based on dev mode (dev vs prod vs staging) later on. And since all these vars can be found on the public site, I didn't put them into Github secrets -- just slapped them into package.json

@jho44 All the images are broken when I run and test locally

I suspect this is because you haven't applied my redoc-cli patch to your installation of redoc-cli. If you run npx patch-package from the project root then run npm run redoc again, the proper docs should be generated.


Twipsy positioning on element inside of an element with a margin

I am instantiating a Twipsy object (with placement = right) on an element that is a descendent of an element that has a margin-top of 38px which isn't being accounted for when the tooltip is placed on the page (it's vertical offset is 38px too great). I assume this has something to do with the fact that the tooltip is being placed immediately after the opening body tag and the element I am instantiating it on is in a decedent of body.

could you provide a jsfiddle? thanks!

http://jsfiddle.net/SBjth/5/
Thanks

crazy things happen to the body when you give it a position of relative... so i would suggest not doing that. Also, for fun, try adding a border of 1px and watch it fix everything. weird.
Maybe @necolas has some clue into why the browser is doing weird stuff when you do that?

This is actually expected behaviour. It's because of "margin collapse". The descendant's margin is collapsing into the body's margin (although in this case there is none), therefore, it is as if the body has a top margin (despite a background color still being applied to the entire viewport).
So when the tooltip positioning includes that margin, the calculation is off. Adding a top border, or padding, or anything to trigger the creation of a new block formatting context (like overflow:hidden or display:inline-block, etc.) for the body will cause the descendant's margin to be contained within it - http://jsfiddle.net/SBjth/5/


Spectator Camera Breakes When Body Destroyed

The camera doesn't behave normally when killed by something that destroys the body. An example of this is the Locker.

I would recommenced creating a fallback to the normal spectator if the mod detects something went wrong

Should be fixed in latest commit


How to set Right side bar in  pushes the content to make space for the sidebar

hi ,
how to create right sidebar looks like left side bar,when i click right side bar the content cannot moved,i need same left side bar option.

Hello,
In your javascript file disable control-sidebar slide option. This can be done either by adding the following code before loading AdminLTE's app.js file:
AdminLTEOptions.controlSidebarOptions.slide = false;

Or by customizing the app.js file and modifying the following line:
  controlSidebarOptions: {
    //Which button should trigger the open/close event
    toggleBtnSelector: "[data-toggle='control-sidebar']",
    //The sidebar selector
    selector: ".control-sidebar",
    //Enable slide over content
    slide: false /* SET THIS TO FALSE */
  },


fix: padding-right for right-pane-tabs

Description
Fix #

Fixes padding on the right for the tab panel.
Additional context

What is the purpose of this pull request?

[ ] Bug fix
[ ] New feature
[x] Other

We could also do with margin-right. Then we don't need to wrap in another div

@ydcjeff could you please show how to do it? Maybe via suggestion


Format of log messages for unnamed clients

REPRO: Create a client directly using the factory, without providing a name.
var client = factory.CreateClient();
Ensure at least information logging is enabled, make a request and view the resulting log messages.
You will see that where no name exists the log message includes a double full stop (period) character before LogicalHandler.
System.Net.Http.HttpClient..LogicalHandler[100]
I believe the message should either remove the unnecessary period or perhaps use "Default" where the name is empty. I'm happy to put in a PR to address this, I would just need to know what the preferred solution should be in this case.
This is non-urgent so I would propose this for the 2.2 timeframe.

I think we should use default in this case.  - so the whole name would be System.Net.Http.HttpClient.default.LogicalHandler.
Feel free to send a PR 😆

@rynowak Happy to pop in a PR. Do you think this should be limited to the point of naming the loggers? Or would setting the name on clients to default rather than an empty string be preferred? I'm not sure if there's a reason it used the Options.DefaultName currently?

Yes, it uses Options.DefaultName on purpose. That way if you use the methods provided by options to interact with the client factory, it will affect the default client. Basically the design of named clients piggybacks on an existing feature.


+1 Error 503 when access from host/outside to container nginx-proxy

Dear @jwilder  and everyone
I am new in docker and I try to using docker with architecture let says :
my goal is microservice with everyone can access my hostname with port 80 and create multiple container which using port 80 to communicate from outside to container itself

and container 1 reproduce default.conf with error here
server_name _; # This is just an invalid value which will never trigger on a real hostname.
proxy_set_header Proxy "";
server {
        server_name _; # This is just an invalid value which will never trigger on a real hostname.
        listen 80;
        access_log /var/log/nginx/access.log vhost;
        return 503;
}
# api1.dev.local
upstream api1.dev.local {
                                ## Can be connected with "reverse-proxy" network
                        # api1-local
                        server <IP_ADDRESS>:80;
}
server {
        server_name api1.dev.local;
        listen 80 ;
        access_log /var/log/nginx/access.log vhost;
        location / {
                proxy_pass http://api1.dev.local;
        }
}
# api2.dev.local
upstream api2.dev.local {
                                ## Can be connected with "reverse-proxy" network
                        # api2-local
                        server <IP_ADDRESS>:80;
}
server {
        server_name api2.dev.local;
        listen 80 ;
        access_log /var/log/nginx/access.log vhost;
        location / {
                proxy_pass http://api2.dev.local
        }
}

i can access container 2 with curl localhost:3000 => OK
i can access container 2 with curl localhost:3001 => OK
i can access from host -> proxy -> container 2 with curl -H "Host:api1.dev.local" localhost => OK
i can access from host -> proxy -> container 3 with curl -H "Host:api2.dev.local" localhost => OK
but
i cannot access container 1 with curl localhost => 503
i cannot access from outside -> host => 503
may i missing something ?
here my reverse-proxy
#!/bin/bash
##########################################################################
# script to check if the jwilder proxy container is already running
# and if the ngnix-proxy network exists
# should be called before "docker-compose up -d"
##########################################################################

if [ ! "$(docker network ls | grep reverse-proxy)" ]; then
  echo "Creating reverse-proxy network ..."
  docker network create --driver bridge reverse-proxy
else
  echo "reverse-proxy network exists."
fi

if [ ! "$(docker ps | grep reverse-proxy)" ]; then
    if [ "$(docker ps -aq -f name=reverse-proxy)" ]; then
        # cleanup
        echo "Cleaning Reverse Proxy ..."
        docker rm reverse-proxy
    fi
    # run your container in our global network shared by different projects
    echo "Running Reverse Proxy in global reverse-proxy network ..."

fi

and here my docker-compose
version: "3.2"

services:
  api1-local:
    build: ./_atkrequest                 #path which Dockerfile is exists
    container_name: api1-local
    restart: always
    networks:
      - reverse-proxy
    expose:
      - 80
    ports:
      - "3000:80"                            #host:container
    environment:
      - VIRTUAL_HOST=api1.dev.local       #nginx-proxy host
      - VIRTUAL_PORT=3000                 #nginx-proxy port
      - HTTPS_METHOD=noredirect

  api2-local:
    build: ./_atkmanage                 #path which Dockerfile is exists
    container_name: api2-local
    restart: always
    networks:
      - reverse-proxy
    expose:
      - 80
    ports:
      - "3001:80"                            #host:container
    environment:
      - VIRTUAL_HOST=api2.dev.local       #nginx-proxy host
      - VIRTUAL_PORT=3001                 #nginx-proxy port
      - HTTPS_METHOD=noredirect

networks:
  reverse-proxy:
    external: true

thanks in advance
@ikwijaya

Mhh, what do you expect "curl localhost" to deliver? There is nothing to deliver, it is how it supposed to be.

solved issue man, i will close it.
my problem is the DNS not reverse, cause i have DNS server for manage it.

and another issue is
if i have at least 2 container which running with same port,
when 1 call 1 container from brower, it will be status 200 then 404 when i refresh
and same issue for another containeer too

i dont know what my problem is, but i suggest  maybe problem with load balancer.


Eslint upgrade

Bump prettier
Bump eslint
Upgrade eslint config to work with upgraded prettier (https://github.com/prettier/eslint-config-prettier/blob/main/CHANGELOG.md#version-800-2021-02-21)

TLDR from the README:
We use ESLint to find/automatically fix problems.
- react-app and react-hooks/recommended are important with react stuff.
- @typescript-eslint/recommended and @typescript-eslint/eslint-recommended as recommended defaults.
- unused-imports to automatically remove unused imports.
- simple-import-sort to automatically sort imports alphabetically. This is opinionated, but useful because it helps avoid merge conflicts with imports (and who doesn't like neat alphabetically sorted imports anyway).
- @typescript-eslint/explicit-function-return-type and @typescript-eslint/explicit-module-boundary-types are turned off to prioritise inferred return types over explicit return types. This is opinionated, but often times the inference Typescript makes is good enough, and sometimes help prevents type mismatches that are a pain to debug.
- @typescript-eslint/ban-ts-comment and @typescript-eslint/ban-ts-ignore are also turned off. This could possibly be temporary, but the ability to use @ts-ignore-like directives is certainly handy as an escape hatch as we encounter errors during the migration to TS.


Question

Not an issue but I was just confused as to what the "Display clear sequence" does, reading it I assumed it would show me the sequence on doors to take to make it to the end during the duty but I couldn't see any indication or guidance during any run, granted they weren't my own maps so I'm not sure if it would if it was my map.

"Display clear sequence" means it will show the order and run total of each time you've fully cleared that treasure map dungeon on the Duty Progress tab.
e.g.
1st clear: 12
2nd clear: 3
3rd clear: 7


Support last_value window function

What problem does this PR solve?
Issue Number: ref #7376
Problem Summary:
What is changed and how it works?
Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/cc @windtalker @SeaRise

/cc @windtalker @SeaRise

/cc @windtalker

/run-all-tests

/run-unit-test

/run-unit-test

/merge

/merge


LightGBM stuck on fitting with libomp 15.0.7 on new Apple M2 if n_jobs != 1

from lightgbm import LGBMRegressor
import numpy as np

x = np.random.random((100, 10))
y = x.dot(np.random.random((10,)))
l = LGBMRegressor(n_estimators=1) # doesnt work, with n_jobs=1 it works
l.fit(x, y)

Darwin PC0455 22.3.0 Darwin Kernel Version 22.3.0: Mon Jan 30 20:39:46 PST 2023; root:xnu-8792.81.3~2/RELEASE_ARM64_T6020 arm64
Name: lightgbm
Version: 3.3.5
==> libomp: stable 15.0.7 (bottled) [keg-only]
MacOs Ventura 13.2.1

Old somewhat related thread: https://github.com/microsoft/LightGBM/issues/4229
Downgrading libomp via homebrew is impossible, as older libomp versions are not compatible with M2.

Issue still persists with libomp 16.0.2

Facing same problem -- I have to set num_threads=1 otherwise kernel died shortly after starting the train job. Interestingly when I set a value >1 (for example 2 or 3), the kernel died after a few seconds while if I do not set any values at all (which default to 0), it almost died immediately.
I originally use homebrew to install lightgbm but switched to the build from github (here) due to the error, before I found this thread and setting num_threads=1.
I can only suspect the homebrew installation will also work with this workaround (haven't tested it)

@Zahlii we just released LightGBM v4.4.0, with some fixes to macOS support. Could you please check again and see if that resolves the issue?
I just ran the example you provided and it worked well for me.

macOS: 14.4.1 (23E224)
chip: M2
Python: 3.11.9
Python libraries: lightgbm==4.4.0, numpy==1.26.4, scikit-learn==1.15.0
OpenMP: libomp: stable 18.1.7 (bottled) [keg-only]

@jameslamb the problem is not fixed for me.
brew info libomp
==> libomp: stable 18.1.8 (bottled) [keg-only]
LLVM's OpenMP runtime library
https://openmp.llvm.org/
Installed
/usr/local/Cellar/libomp/18.1.8 (9 files, 1.7MB)
  Poured from bottle using the formulae.brew.sh API on 2024-06-27 at 01:50:02
From: https://github.com/Homebrew/homebrew-core/blob/HEAD/Formula/lib/libomp.rb
License: MIT
==> Dependencies
Build: cmake ✘, lit ✘
==> Caveats
libomp is keg-only, which means it was not symlinked into /usr/local,
because it can override GCC headers and result in broken builds.

For compilers to find libomp you may need to set:
  export LDFLAGS="-L/usr/local/opt/libomp/lib"
  export CPPFLAGS="-I/usr/local/opt/libomp/include"
==> Analytics
install: 60,255 (30 days), 181,591 (90 days), 515,596 (365 days)
install-on-request: 11,742 (30 days), 35,590 (90 days), 103,479 (365 days)
build-error: 1 (30 days)

My system is almost exactly as yours (M2 and everything)

sad 😭
ok thank you for letting us know, we'll try to investigate soon


Add post build action to Jenkinsfile to run e2e tests.

This change triggers the e2e tests to run on intg after an intg deployment has been started successfully. There is a potential issue in that the deployment may not have finished and the load balancer may not be sending traffic to the newly deployed version yet. As a small workaround to this, the e2e tests Jenkinsfile will be modified to include a 'quietPeriod' in which it waits a pre-configured amount of time to start building after a build is initially triggered.
A more sophisticated solution to this problem may exist, and I will look into it to see what can be done to fix it.

This looks good to me! Just one formatting comment.
We should have a think about what other jobs should trigger the end-to-end tests. Off the top of my head: the API deployment, Terraform environments, DB migrations.
This is another case where it would be handy to have this function in a shared Jenkins library. Are you planning to look into that as part of the GitHub branch work?

I was planning to start looking into that for the build status stuff but if it's useful for the e2e tests as well I can add a card to the e2e story too and look into both at the same time?


No way to edit header comment

Please provide a way to edit header comment.
Header comment is the very first comment in a PO file so it often contains useful information like copyright years and translator history.

The standard way of having this metadata is in the actual header. The de-facto convention of some FLOSS projects is to record history in the comment.
In my view, that is far less critical to have; that said, if this is important to you, I would definitely be happy to merge your patch implementing it.

Well some FLOSS projects just do not care about this comment. xgettext always generates POT files starting with header comment '# SOME DESCRIPTIVE TITLE  # Copyright (C) YEAR THE COPYRIGHT HOLDER' ....' which is intended to be edited by translators.

Again, PR welcome.

The standard way of having this metadata is in the actual header.

Is there an official standard about the PO files?

The de-facto convention of some FLOSS projects is to record history in the comment.

These "some" FLOSS projects include the Translation Project, formerly known as the GNU Translation Project. In order to submit a translation there, the header comment must contain the proper copyright holder and proper year, along with some other things; otherwise the submission is rejected. PO files created with Poedit need further manual editing with a text editor to make them valid for submission.
The Translation Project started in 1996, so it is nearly as old as GNU Gettext itself, and and one of the first places to collect translations in PO format. Therefore I'd say that their opinion of how the header comment should be used has slightly more noteworthiness than a random FLOSS project.
Just my 3 cents.


Document allow-http:false annotation is supported

It isn't listed anywhere that you can use allow-http:false with kube-lego.
I guess the annotation is in generic ingress TLS but there's no indication that it's supported on kube-lego.

Closing in favour of #173


keyword of the install method has a syntax error

@Swind This is a breaking change for those who are using this method.


feat(transformer/typescript): support transform exported TSModuleBlock

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #2074

PR #2075

PR #2076   👈

This stack of pull requests is managed by Graphite.

Merge activity

Jan 18, 10:16 AM: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.


Always be stuck when close lots of tab too quickly.

VSCode Version:1.43.0
OS Version:Linux x64 5.3.10-050310-generic
Electron:7.1.11
Chrome:78.0.3904.130
Node.js:12.8.1
V8:7.8.279.23-electron.0

Steps to Reproduce:
1.open lots of tab and fill some word into them.
2.close these tabs one by one quickly.

Does this issue occur when all extensions are disabled?: Yes

Is this still an issue with latest insiders https://code.visualstudio.com/insiders ?


'insert ... select ' task is not running on the worker-write node

(you don't have to strictly follow this form)
Bug Report
Briefly describe the bug
'insert ... select ' task is not running on the worker-write node
The result you expected
'insert ... select ' task is running on the worker-write node
How to Reproduce
Occurs when I perform the import below
INSERT OVERWRITE olap.olap_comm_deal_margin_agreement_indicator_di_cube_v37 SELECT shop_code,performance_city_code,city_code,agent_code,statistical_code,brand_code,is_core_30_city_yj_code,is_agent_city_code,battle_region_code,province_region_code,corp_code,marketing_code,area_code,shop_director_code,team_code,sub_brand_code,role_category_code,biz_type_code,dim_ref_employee_base_info_da_on_job_status_code,dim_ref_shop_brand_org_info_da_original_battle_region_code,dim_ref_shop_brand_org_info_da_original_province_region_code,dim_ref_shop_brand_org_info_da_original_performance_city_code,dim_ref_shop_brand_org_info_da_original_city_code,dim_ref_shop_brand_org_info_da_original_corp_code,dim_ref_shop_brand_org_info_da_original_region_code,dim_ref_shop_brand_org_info_da_original_marketing_code,dim_ref_shop_brand_org_info_da_original_area_code,dim_ref_shop_brand_org_info_da_original_shop_code,pt,stat_date,statistical_name,brand_name,is_core_30_city_yj_name,is_agent_city_name,is_has_lianjia,is_direct_sale,is_valid_agent,on_job_days_section,position_type,del_type_name,battle_region_abbr_name,province_region_abbr_name,performance_city_abbr_name,city_abbr_name,corp_name,marketing_name,area_name,shop_director_name,shop_name,team_name,agent_name,sub_brand_name,is_direct,role_category_name,is_effective,is_last_180d_linkshop,biz_type_name,corp_director_no,corp_director_name,marketing_director_no,marketing_director_name,area_director_no,area_director_name,dim_public_date_info_year,dim_public_date_info_month,dim_public_date_info_week,dim_public_date_info_sun_dt,dim_public_date_info_short_week_cn,dim_public_date_info_short_year_weeks,dim_ref_employee_base_info_da_is_link_agent,dim_ref_employee_base_info_da_job_level_seq_name,dim_ref_employee_base_info_da_agent_job_category_name,dim_ref_employee_base_info_da_on_job_status,dim_ref_shop_director_role_relation_da_is_link,dim_ref_shop_brand_org_info_da_original_battle_region_abbr_name,dim_ref_shop_brand_org_info_da_original_province_region_abbr_name,dim_ref_shop_brand_org_info_da_original_performance_city_abbr_name,dim_ref_shop_brand_org_info_da_original_city_abbr_name,dim_ref_shop_brand_org_info_da_original_corp_name,dim_ref_shop_brand_org_info_da_original_region_name,dim_ref_shop_brand_org_info_da_original_marketing_name,dim_ref_shop_brand_org_info_da_original_area_name,dim_ref_shop_brand_org_info_da_original_shop_name,dim_ref_city_base_info_da_city_level,dim_ref_city_base_info_da_is_bplus_city,original_team_code,original_team_name,original_agent_ucid,original_agent_name,dim_public_date_info_date,assess_income_amt_sum,same_shop_margin_assess_income_amt_sum,coop_agent_order_outer_rebate_amt_sum,receivable_assign_amt_sum,received_assign_amt_sum,sign_receivable_amt_sum,sign_receivable_pre_amt_sum,received_pre_amt_sum,valid_assess_income_amt_sum,person_create_amt_sum,s_person_create_amt_sum FROM load_temp.olap_olap_comm_deal_margin_agreement_indicator_di_cube_v37_850ad17938de4ae58906cdaca6cca05f SETTINGS max_execution_time = 3600;

Version
0.4.0

hi @bright-zy , sqls like 'insert ... select' currently can only be run in server node

worker-write
hi @smmsmm1988 ,
I hope to support execution on worker-write nodes in the future. Thanks!

worker-write
hi @smmsmm1988 ,
I hope to support execution on worker-write nodes in the future. Thanks!

hi @bright-zy, thanks for your suggestion! It may be a big feature for ByConity, and we will take it into consideration in future versions.


Behavior Test Framework for Bindings

OpenDAL's behavior test greatly enhances the stability and reliability of the openal rust core. I used to think bindings can use rust core directly and don't need to perform their own test frameworks. However, some users blocked by binding issues, for example: https://github.com/apache/incubator-opendal/issues/3037
So I think it's better to build a behavior test framework for all bindings instead. This test framework should reuse all our existing setup and workflows as much as possible. And our binding should only need to implement test case once and can run upon different services.
Design

Binding read .env like core does to init different services.
Binding implement behavior test like core does, for example, write a file and read it.

Tasks

[x] Move all fixtures from core/src/services/{service} to top-level fixtures/{service}
[x] Java

[x] Implement functions to read .env (and env) to init operator
[x] Implement test_write / test_read
[x] Add java test in service_test_s3.yml
[x] Migrate redis test to service_test_redis.yml

[x] Python

[x] Implement functions to read .env (and env) to init operator
[x] Implement test_write / test_read
[x] Add python test in service_test_s3.yml

[x] Node.js

[x] Implement functions to read .env (and env) to init operator
[x] Implement test_write / test_read
[x] Add node.js test in service_test_s3.yml

[x] Remove Cucumber based BDD Test

Other bindings can follow those work based on their own progress.
Notes
This issue is solely for bootstrapping the test framework. We will create separate tracking issues to address different bindings in the future.

cc @tisonkun @messense @suyanhanx for comments.

How do the variables in .env used? I don't find usage of those variables like OPENDAL_AZBLOB_CONTAINER.

How do the variables in .env used? I don't find usage of those variables like OPENDAL_AZBLOB_CONTAINER.

All variables in .env are treated as OPENDAL_{service}_{key}. Implementor should extract all OPENDAL_{service}_{key} = {value} to a HashMap and than passing to Operator::from_map() (based on binding's own API).
The rust implementation could be found here:
https://github.com/apache/incubator-opendal/blob/12bd7677462847f86714e311c098578912204859/core/tests/behavior/utils.rs#L48-L77

May I have a try of java? And should I complete the migration first?

Move all fixtures from core/src/services/{service} to top-level fixtures/{service}

And should I complete the migration first?

Cool, welcome to do this!

May I have a try of java?

Welcome! @tisonkun oversees the Java binding. Feel free to engage him in discussion or open a new issue if necessary.

FYI - The Java part is completed. Thanks to @G-XD's work.
TODO list is updated.

I'd like to try the Python part and I found that already has a behavior test based on memory, should I update it to the s3 service?

I'd like to try the Python part and I found that already has a behavior test based on memory, should I update it to the s3 service?

The behavioral tests mentioned in this issue are not quite the same as the parts we did before. The new tests will be a bit closer to real-world usage.
The test in Java may give you some help as an example.

I'd like to try the Python part and I found that already has a behavior test based on memory, should I update it to the s3 service?

The old behavior test will be replaced with the new test framework mentioned here. In the new test framework, we will:

reading .env files to bootstrap different test services.
support running tests based on service and test names.
implement tests like what we do in rust core and java binding.

I'd like to solve the Node.js part. Can I have a tried?

I'd like to solve the Node.js part. Can I have a try?

Welcome!

Most work has been done. Let's close.


what is the difference between promise and complete?

Velocity($('.coin'), {
  left: '+100'
}, {
  duration: 2000,
  complete: () => console.log('complete')
})
.then(() => console.log('resolve'))

Velocity($('.coin'), 'stop')

output:
resolve

complete not execute

@lianer The returned promise is resolved when the call's animation completes, regardless of whether it completed on its own or prematurely due to the user calling $element.velocity("stop"[, true]).
The promise is rejected when an invalid first argument is passed into a Velocity call (e.g. an empty properties object or a Velocity command that does not exist). The reject callback is passed the triggered error object.
Complete will only execute on completion of animation. Both of them are applied to different use cases.


[Broker] update SystemTopicClient.isSystemTopic  to BrokerService.isSystemTopic

Motivation
#14671  made changes on judging systemTopic, SystemTopicClient had been deleted.
All CI failed
/home/runner/work/pulsar/pulsar/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/BrokerService.java:[1528,36] cannot find symbol
  symbol:   variable SystemTopicClient
  location: class org.apache.pulsar.broker.service.BrokerService

Modifications
Change to isSystemTopic
Documentation
Check the box below or label this PR directly (if you have committer privilege).
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[x] no-need-doc
(Please explain why)

[ ] doc
(If this PR contains doc changes)

#14774  duplicate


[Task]: replace findDatabase in Action sql

What needs to happen?
replace findDatabase in Action sql
related to #2542
Issue Priority
Priority: 3
Issue Component
Component: Actions, Component: Database

Merge pull request https://github.com/apache/hop/pull/2864


[Request] - Toggle hotkey

Hey,
I've been using your extension for a while now and what I've noticed that I end up doing a lot is disabling the extension whenever the expanded type is too large.
Without the extension I can hold CMD and hover a variable and sometimes I'll get some type insight, so I was wondering if you could add a feature like that? maybe a way to manually show/hide expanded types based on hotkey

Same. It would be great to use the plugin optionally with a modifier key.

Hey @salmin89 and @ismailkarsli! Thanks for your feedback.
Handling how to display large types has been a continuous struggle which I’m trying to improve with each update. Currently I am working on improving nested types that have non-aliased sub properties, which currently are displayed even if your depth setting indicates they shouldn’t be. If possible, would you mind sharing the type that is displayed that’s too large, and what the resulting type preview from the extension is? I am hopeful my current changes for the next release address it address it already.
Also - Having you explored restricting the settings of the extension to display less properties, or truncate the max number of characters?
Regarding a hotkey to enable/disable, unfortunately VSCode does not expose an API to check for key presses the same way the default setting does to expand on types, but one option I could explore is creating a command for the extension to enable / disable, which then it might be possible to attach a key-bind to trigger that toggle command. Would that be acceptable in your case?
Thanks for using my extension and for your feedback! 😁

I just checked and there are many options in extension settings as you're saying. However, sometimes I need to see the class names and their wrappers rather than the object structure. For example;

Drizzle/TypeGoose models
External libraries
Node native functions

In other words, things that are not pure objects.
Toggling the feature without enabling/disabling the plugin would be enough for me.
I really appreciate this plugin by the way. I was thinking of making the same plugin and was glad to see that it was already done :)

I haven’t messed around with the max props and depth settings. I’ll definitely test changing those!
otherwise I’d be totally fine with a hot key as mentioned 🙂

Thanks for this extension, I was going to make a similar one a while back until I saw you'd make this already! It's been very nice to use :)

one option I could explore is creating a command for the extension to enable / disable, which then it might be possible to attach a key-bind to trigger that toggle command. Would that be acceptable in your case?

This would be exactly what I'd like myself—I appreciate being able to see the full contents by default, but would love to be able to hide/show them on command to reveal the TS type info that shows below, without needing to scroll down to it.
(I believe you actually already had this command implemented a couple months ago—toggleHover—and I regularly used it with a keybinding, along with VSCodeVim's gh keybinding for triggering a hover on the cursor)

If possible, would you mind sharing the type that is displayed that’s too large, and what the resulting type preview from the extension is? I am hopeful my current changes for the next release address it address it already.

Here's one example (fairly tame—some types can be much larger than this):

(move cursor into window, scroll down)

Hey @salmin89! Really sorry for the delay on this one. I just closed out PR #51 which will include this change in the next release, which I'm aiming to get released ASAP. 😄

@mylesmmurphy no worries! we all have jobs and family. This is just a bonus. I'm in no rush :)

I like the idea of toggling and maybe also some action functions to increase/decrease depth of an object on the fly. Also, some to disable unwrap for the hovered type.

Total typescript has some actions that can be taken from a hover.


Can I use third party lib in IncludeOS?

Hi:
I'm a beginner of Unikernel and I want to migrate my App to IncludeOS. But I have use some 3rd libs like folly, boost and zmq. Is IncludeOS support these 3rd libs?

You can use many 3rd party libs with IncludeOS, see the SQlite demo service for an example. If the libraries you want to use depend on pthreads you might run into some issues since this part of IncludeOS is still WIP.
Regarding boost, it depends on what parts of boost you want to use. For example, I use Boost CRC in a test service, and that works as expected, but there are probably some other libraries in Boost that will be slightly more challenging to get to work.
If you decide to try migrating your app to IncludeOS and run into any issues with any third-party libraries, feel free to file issues here or come into our gitter chat

Thanks, I'll try it.


This is from times long gone. Closing.


write list of use-cases for test cases

we need a list of use-cases that could be done in UI. Hierarchy (by user role) would be nice.
List has to be prioritised based on usage statistics - the more frequently use cases is run, the higher the priority should be

https://github.com/ydb-platform/ydb-embedded-ui/wiki/Use‐cases-grouped-by-role take a look please


Recipe Compile Error on "Installing all 3"

As of 'database >= v4.0.2', recipe 'database::mysql' has been removed from the cookbook.
Which causes errors when installing database according to instructions from readme

Installing all 3 - Database MUST come before Server
"recipe[database::mysql]",
"recipe[mysql::server]",
"recipe[zabbix]",
"recipe[zabbix::database]",
"recipe[zabbix::server]"

Result of installation
==> monitoring_server: ================================================================================
==> monitoring_server: Recipe Compile Error
==> monitoring_server: ================================================================================
==> monitoring_server:
==> monitoring_server: Chef::Exceptions::RecipeNotFound
==> monitoring_server:
==> monitoring_server: --------------------------------
==> monitoring_server: could not find recipe mysql for cookbook database

Thanks

Modify the metadata like this:
depends 'database', '= 2.3.1'
depends 'mysql', '= 5.6.3'
At chef client 12.4.3 (ubuntu 14.04) I have to add a package to build-essential::_debian.rb recipe:
package 'libmysqlclient-dev'


Reserved IPPools

PROPOSAL :
Hello,
It would be great if Calico will be able tag the IPPools as "Reserved"  to disable Pod to get scheduled from this IPPool without specify any Annotation that point this IPPool.
It would be great because when dealing with a lot of IPPools, pods get scheduled a lot on unwanted pools and that force me to specify the IPPool I want on EACH deployment.
Thanks !

@valentin2105 not sure I understand. How is this different from IPPool Disabled: true configuration to stop allocating new IPs from a particular IPPool?

@gunjan5 if I put Disabled an IPPool, can i still use it by choosing this pool with a Deployment annotation ?

No that won't work, if the pool is disabled, IPs won't be allocated from it until it is enabled again. But I think I understand what you're asking now, so you want to disabled the IPPool for pods to get IP from by default unless a pod/deployment specifically asks for it using an annotation? Is that correct?
Example:
You have:
pool-1, pool-2 - for all the other pods
pool-3 - not to be used by other pods, but only used by pods with an annotation asking for pool-3 specifically
Is that correct?

Yes its exactly that !
I looking to get "reserved" IPPool who is used only when I specify on the
Deployment.
You think is a useful feature ?
In my case, I use pool for give a dedicated IP to my DNS and Nginx-Ingress
to expose them on our DNS and reach them directly.
Le 2 févr. 2018 17:34, "Gunjan Patel"<EMAIL_ADDRESS>a écrit :

No that won't work, if the pool is disabled, IPs won't be allocated from
it until it is enabled again. But I think I understand what you're asking
now, so you want to disabled the IPPool for pods to get IP from by default
unless a pod/deployment specifically asks for it using an annotation? Is
that correct?
Example:
You have:
pool-1, pool-2 - for all the other pods
pool-3 - not to be used by other pods, but only used by pods with an
annotation asking for pool-3 specifically
Is that correct?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/projectcalico/calico/issues/1655#issuecomment-362499579,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL1ByWKBr1gYjP_hp_BZV8PjefJPofvcks5tQqyTgaJpZM4R2bGk
.

@gunjan5 Can you still force-allocate from a disabled pool by using the fixed IP annotation?

Hm, yeah. We have a concept of "Disabled" IP Pools, which won't be used for IPAM.
However, I think if you specify a disabled IP Pool in an annotation it will throw an error, rather than calling the specific IP Pool.
Would be nice to have a way of setting some pools aside as reserved.

@fasaxc you could still force-allocate, but then you need to pick an exact IP address, and it won't get tracked anywhere (so multiple pods could get the same IP because nothing is coordinating).
I think having a reserved class of IP pools probably seems sensible given the CNI plugin's selection features.

I have disabled one of my IPPool,
Then I tryed to deploy a deployment (1 replicas) with an IP of this Pool ,
It seem doesn't work :
failed: rpc error: code = Unknown desc = NetworkPlugin cni failed to set up pod \"nginx-ingress-01-6b844c5b9-vjkx7_nginx-ingress\" network: error getting IP from IPAM: The provided IP address is not in a configured pool\n"

@caseydavenport don't get me wrong, I think having a reserved class would be good too, was just trying to find a workaround.  I thought there was an option to "force allocate and claim" an IP

I don't think we have a "force allocate and claim" option - we have a "force allocate and don't claim" option. @gunjan5 is that right?
If you use cni.projectcalico.org/ipAddrsNoIpam but that only lets you specify an exact address, not an IP Pool.

I got it,
If you create a Disabled: true IPPool and then use cni.projectcalico.org/ipAddrsNoIpam, my deployment got the IP I want, and not other pod can take an IP from this Pool !!
So maybe, A reserved pod is useless if we can explain that in the documentation.

@caseydavenport yeah. We don't have a "force allocate and claim" option.
One solution is what @valentin2105 described, with Disabled pool and force allocate IP, but that will require specifying IP manually for each pod/deployment, not a pool

FWIW, with nodeSelector you can now create an IP pool which doesn't select any nodes, which effectively "reserves" that IP pool unless explicitly asked for.
It's a bit awkward, but works until there is something cleaner.

Couple of improvements here in v3.21:

IPReservations
allowedUses

I think we can probably close this, as it's been relatively inactive and we have several methods to achieve the original intent of the issue now


[FIX] divide all queries over threads

The last queries of the query file were not processed in case the queries could not be divided over the reads exactly ( nr_of_queries % nr_of_threads != 0).
Test parallel search (parallel IBF building requires larger dataset of >= 64 bins)

Codecov Report
Base: 90.90% // Head: 88.88% // Decreases project coverage by -2.02% :warning:

Coverage data is based on head (92e8ed9) compared to base (569ec1f).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #66      +/-   ##
==========================================
- Coverage   90.90%   88.88%   -2.03%
==========================================
  Files          24        1      -23
  Lines         638       27     -611
==========================================
- Hits          580       24     -556
+ Misses         58        3      -55

Impacted Files
Coverage Δ

include/valik/search/local_prefilter.hpp

...nclude/valik/search/write_output_file_parallel.hpp

src/valik_search.cpp

src/argument_parsing/top_level.cpp

src/argument_parsing/split.cpp

include/valik/build/call_parallel_on_bins.hpp

src/valik_build.cpp

include/valik/shared.hpp

src/argument_parsing/build.cpp

include/valik/split/reference_segments.hpp

... and 13 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fix solution application for keypath function subtype conversions

In #39612 we added subtyping for keypath-function conversions. However the implementation naively coerced the keypath expression itself to the inferred supertype, resulting in erroneous results. This patch updates the solution application logic to build the keypath-function conversion expression based entirely on the 'natural' keypath type, only converting to the inferred supertype at the end via the usual coerceToType machinery for function conversions.
Resolves #71423

@swift-ci please test

@swift-ci please test source compatibility

@xedin would love your eyes here when you have a chance!

@swift-ci please build toolchain macOS platform

@swift-ci please test macOS platform

@swift-ci please test Linux platform

All of the source compatibility failures are macro related, including penny-bot

@xedin as in, there's an unrelated macro issue here? Or you suspect this patch is somehow interfering with macro logic?

Macro issue is unrelated, I saw this happen in other PRs.

@swift-ci please smoke test macOS platform


Need a hint

Good time, please tell me how you can output the address, for example, only the city and the street with the house number?

you can take a look at the example app in the repo


updated readme showing a parallel_tests configuration

Solves bad xml due to misconfiguration #33

Thanks, I'll revise this in a bit.

link https://github.com/ArturT/knapsack/issues/10


[Bug] collision true. Can't move element to some coordinates

When collision is active, when trying to move an element it can only be moved to certain spaces not everywhere.
Reproduction: Can't drag on this repl element "0" down (y axis). Only works if collision is false.

It is intended. without the y-axis compression, it's hard to decide where collision target will be moved. plz refer to change log https://github.com/cuire/svelte-grid-extended/pull/27#issue-1785173048

Check #32


azure-mgmt-quota fails reference docs CI

Cannot produce SDK API Reference Documentation for azure-mgmt-quota because of errors in the docs CI process.
Instructions to fix:

Fix issue in package code to eliminate this error and check into main branch
Ensure output is correct in daily docs build (see @danieljurek or @sima-zhu for details)
Remove entry from $PackageExclusions in Language-Settings.ps1

Build: https://apidrop.visualstudio.com/Content CI/_build/results?buildId=260142&view=logs&j=fd490c07-0b22-5182-fac9-6d67fe1e939b&t=1e77462e-7711-551f-cf20-c31b880b493a&l=28529

<Matched Warning>: D:\a\1\s\dist_temp\83\azure-mgmt-quota-1.0.0a20211020001\doc\azure.mgmt.quota.aio.rst:4: WARNING: error while formatting arguments for azure.mgmt.quota.aio.AzureQuotaExtensionAPI: name 'AsyncTokenCredential' is not defined

##[error]<Matched Error>: C:\hostedtoolcache\windows\Python\3.6.8\x64\lib\site-packages\azure\mgmt\quota\aio\operations\__init__.py:docstring of azure.mgmt.quota.aio.operations.QuotaRequestStatusOperations.list:10: ERROR: Error parsing content block for the "list-table" directive: uniform two-level bullet list expected, but row 2 does not contain the same number of items as row 1 (1 vs 2).
.. list-table::
   :header-rows: 1
   * - Field

<Matched Warning>: C:\hostedtoolcache\windows\Python\3.6.8\x64\lib\site-packages\azure\mgmt\quota\aio\operations\__init__.py:docstring of azure.mgmt.quota.aio.operations.QuotaRequestStatusOperations.list:19: WARNING: Inline substitution_reference start-string without end-string.

##[error]     - Supported operators
   * -

Is there any possibility of getting these documents generated at this point in time?

@DuneganS
https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/612/Use-Docker-for-Package-Validation?anchor=reproduce-the-docker-errors
Here is the wiki for you to validate the errors in local. Ping me if anything unclear

@DuneganS If the docs passed the validation, then we can add the package back to docs.

Both of links as part of this inc show that I do not have access (and there is no way to request)

Also interested in getting this documentation

@DuneganS @ajhous44 Sorry for late reply. We grant the permission for all internal employees I think. If you are external customers, you can't run the internal docker, so the docs doesn't apply to you.


⚠ Bofrost.at has degraded performance

In e5a946e, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1199 ms

Resolved: Bofrost.at performance has improved in 751167b after 9 minutes.


Require user to accept EPSON's end user license before launch

Maybe zenity dialog will help?

Fixed in revision ae35a22.


add unzip step

One random thought for convenience—you could add hyperlinks to the next issue at the bottom of each one.

One random thought for convenience—you could add hyperlinks to the next issue at the bottom of each one.

I like this idea - we'll do that.


Suggest additional clarity as to which HW the document applies to.

Team,
I'm confused as to which hardware the document refers to in several respects.  The title of the document implies that you are imaging the console, although it's unclear as to whether you're actually imaging a console or a compute module.  It would also be beneficial to point out whether it's supported to image non-certified NUCs (and other certified compute modules).  In addition, it would be beneficial to point out whether NUCs need to be sourced as part of a package or whether it's ok to buy your own NUC (or other certified compute modules).  For example, other documentation refers to a TPM chip in the NUC.  Is the TPM chip present in all NUCs by default, or only those that are purchased as part of a certified Teams Room package?  Is this document applicable to only current Teams Rooms, also to previous dock-based setups, or both?  (There is a dock reference in this document that confused me in this regard).
There are various forum posts on the web where people are asking these questions and referring to this particular document, so this would be a good place to add this clarification.
Thank you!
David
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6f8b204f-9650-d268-8566-7a75ba0a1b6a
Version Independent ID: 9a963e63-6716-7522-8abe-c8a407d3a46d
Content: Configure a Microsoft Teams Rooms console - Microsoft Teams
Content Source: Teams/rooms/console.md
Service: msteams
GitHub Login: @dstrome
Microsoft Alias: dstrome

@davidjfriedman Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hi @davidjfriedman, thank you for engaging with the community here for the docs.
Non-certified NUCs/PCs are not supported. You can check the list of supported devices at the following link:
https://docs.microsoft.com/microsoftteams/rooms/requirements
This is an important document to check when you are deploying MTR devices. The console article is not enough.
The Teams Rooms app will not start until it verifies that you are using certified hardware. So no matter what, you must purchase an approved, certified hardware solution.
The ability to make your own image is only intended for large corporations to customize the image on certified hardware. It is definitely not a roll-your-own, ad-hoc solution.

@dariomws, thanks for your reply and clarification--much appreciated.  My concern is that the console article as it is written leaves this open to interpretation.  I understand that we may not be looking to turn the console document into another redundant requirements article, although there may be room to sprinkle several disclaimers and warnings to more clearly spell this out, possibly with additional references to the requirements document.
David

@davidjfriedman rechecking the article I found there is an important note about that:

[!IMPORTANT]
Microsoft Teams Rooms relies on the presence of certified console hardware. Even a correctly created image containing the Microsoft Teams Rooms console app will not boot past the initial setup procedure unless the console hardware is detected. For Surface Pro based solutions, the Surface Pro must be connected to its accompanying dock hardware to pass this check.

I'll add a link to the requirements article.


Popup on hover

Hi @Naimikan
Do you have a way in your directive to show up a popup when you hover over a marker ? (i use glLayers and glSources like so :
{
                id: 'medium-circle',
                source: 'medium-circle',
                type: 'circle',
                paint: {
                    'circle-radius': 7,
                    'circle-color': '#FFFFFF',
                    'circle-opacity': 1,
                    'circle-stroke-width': 3,
                    'circle-stroke-color': '#E2E70D'
                },
                popup: {
                    enabled: true,
                    onClick: {
                        message: '

HELLO

'
                    }
                }
            }

Works on click but i need to understand how to do it on hover with your directive.

HI @mm-ns!
I'm working on that feature. I will update a new version soon.
Thank you!

Cool :) Thanks! So far I have a workaround where I check on mousemove if im on one of my markers and then opening a popup.

Cool :) Thanks! So far I have a workaround where I check on mousemove if im on one of my markers and then opening a popup.

Hi @mm-ns, can you help me? How did you determine that element on 'mousemove' is marker? Did you compare coordinates?


v3 request - a config option to disable full Photo

analog to "Download Photo" option.

I just note here that this is not about disabling image access to users. I think it is a valid request for the owner to only allow js layered screens. (For the moment I just disable it by the contextMenu.photoMore splice thing, set to splice(0, 2))

This feature has been requested multiple times now. See https://github.com/electerious/Lychee/issues/243 and https://github.com/electerious/Lychee/issues/231. However, I'm still think that this is a user friendly feature and shouldn't be disabled. Photos may be displayed in a small size depending on the screen. Users should be able to see the full photo in this case.

I am with you, that from a user perspective the open form is best. My perspective was from the owners and design view, since as far as I remember, the full photo was just viewed by direct link using the browser. I fell over it, since I had images which were not as big as the actual browser screen. Then the full image was shown placed left-upper on a white background. And that looked bad. Therefore I needed a disable option. And it would be so easy to do, see my not very ambitious patch..
I am sorry I did not see these other request issues.


Time to publish

Hey @ShanewUU how about we turn on GitHub pages and publish the game?

@Docmilo maybe we should review the project files and ensure we are good to go. Can you create an issue and complete this task please?

GitHub pages turned on and the game is published at gameURL


Tests to Re-produce Inheritance Problem

In relation to #75, as per @nemesisdesign's suggestion.
I guess that the added test is pretty much self explanatory.

hey, could you rebase this? and do you have any solution in mind? if you are looking for takers of this, please tell. thanks for your efforts.

@auvipy Sorry, I have missed your comment.
It is a 2+ years old topic, and I am no longer using inheritance as I did before. For the sake of sanity, I am closing the PR.


Add support for Turbopack in Next.js 13

Describe the feature
Usually I use Turbopack instead of Webpack. I am experiencing an error rendering the root server when attempting to use Turbopack in conjunction with Million.js.
Additional information

[ ] Would you be willing to help implement this feature?

Hmm, what is this issue, please?

Having a similar issue with our NextJS + turbopack project here -- million gives errors relating to rules of hooks and making sure we installed the compiler correctly, but I triple-checked everything and it looks fine. On top of that, none of these problems are present when using webpack.

We'll look into that
Thank you for reporting the issue

Million still doesn't work with Turbopack enabled

Can confirm that this is still the case.
Next.js 14.2.3 + @million/lint 1.0.0-rc.13 does not work with Turbopack.
If you use next dev --turbo, remove --turbo.


Resetting database should take less time

SInce appears_in has an update trigger, it may be quicker to run a single update query in a BFS-style algorithm to update the values of all Ryu Numbers, rather than relying on reinsertion, which can get tedious.
PERSONAL NOTE: Use a new branch when trying this. I do NOT want to break anything already working lmao.

PERSONAL NOTE 2: If this gets working, and working well, seriously consider implementing a "recenter" function or something. Let's see how far away Mario really is from Goku >:)

I had created a whole separate branch for working on this issue in my local repository, and merges with a few other commits have brought this issue to a close!

1ba119c Removed file i/o entirely (optimized in 6cba3fd)
02de771 Optimized file insertion
c269aac Was the final nail in this coffin
Issue will only be completed once I also update the README, since it mentions the file i/o, but thats a job for in the morning...

README updated, so this issue is resolved as of commit 3c7014a!


Sometimes I can't continue typing text after I type in Chinese

Environment：Windows 10 1903
When this happens, I have to switch the English input method and then switch the Chinese input method to continue typing.
Previously this problem did not occur on Ubuntu 18.04.

In order to have any hope of reproducing and fixing this, we're going to need some detailed steps to reproduce the problem. This is one of the reasons we require the issue template to be filled out ...

Thank you for contributing!
We require the issue template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests. Please open a new issue and fill out the template for us.
Thanks for understanding and meeting us half way :grinning:


CMake AnyNewerVersion -> SameMajorVersion

We are not backwards compatible for version 2.0. Allowing users' 1.x code to build against 2.0 is almost 100% would lead to build failures. Would rather prevent that during configuration.
Switching to SameMajorVersion would likely be temporary, for a few 2.x releases, after which we would switch back.
An alternative to switching to SameMajorVersion would be to update CMake requirements to at least 3.19 (release 11/18/2020, so about a year older than CMake we are using right now) and do
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -136,7 +136,7 @@ configure_package_config_file(cmake/ArborXConfig.cmake.in
 )
 write_basic_package_version_file(${CMAKE_CURRENT_BINARY_DIR}/ArborXConfigVersion.cmake
   VERSION ${ARBORX_VERSION_STRING}
-  COMPATIBILITY AnyNewerVersion
+  COMPATIBILITY AnyNewerVersion <2.0
 )
 install(FILES
   ${CMAKE_CURRENT_BINARY_DIR}/ArborXConfig.cmake

This has a benefit of not having to revisit this again later.
Note: AnyNewerVersion was introduced in #984.

This means that users cannot support both 1.x and 2.x versions in their own code. That's going to be a pain.

How about we conditionally us a version range when cmake is recent enough and we defer the actual update of our cmake minimum requirement?

For the current CMake, we switch to SameMajorVersion?

This means that users cannot support both 1.x and 2.x versions in their own code. That's going to be a pain.

Fair point. Do you suggest sticking with AnyNewerVersion?
Do you envision users wanting to do that? In relation to that, we should probably also provide ARBORX_VERSION macro to ease code writing depending on ArborX version.

They could do find_package(ArborX REQUIRED) without specifying version

I want to be able to require a minimum version because older versions did not have ray tracing.

I want to be able to require a minimum version because older versions did not have ray tracing.

Why not require ArborX 2.0?

According to the standard ([cpp.cond].11)

After all replacements due to macro expansion and evaluations of defined-macro-expressions, has-include-expressions, and has-attribute-expressions have been performed, all remaining identifiers and keywords, except for true and false, are replaced with the pp-number 0, and then each preprocessing token is converted into a token.

This reads to me that for ARBORX_VERSION prior to #1190 would essentially be set to 0, meaning that the user code could correctly determine whether it is using version prior to 2.0 correctly even if ARBORX_VERSION is not defined.
I tested it and it indeed seems to work.

So, we have three ways to proceed:

Require CMake 3.19 and do
AnyNewerVersion <2.0
Switch to SameMajorVersion and require users to do find_package(ArborX 1.7...2.99) if they want to be able to handle both versions.
Leave things as they are with AnyNewerVersion.

In 2, if we start requiring CMake 3.19 in ArborX, this would essentially tell users what CMake to use implicitly.
I think option 3 is a mistake, as it break unsuspecting user's codes during compilation instead of configuration.
We need to move forward somehow.

The current status quo, that is option 3, is clearly not viable.
I think that bumping our minimum cmake version requirement is not an issue.  For reference, Trilinos is currently requiring 3.23.  On the Kokkos side, we will likely wait until next summer and the release of 5.0.

Let's go with option 2. It express better what is our current requirement, Option 1 and 2 are currently equivalent. They will only differ when we get to version 3 and at that time we can still decide to use option 1.

So, bump CMake (to 3.19?) and go with option 2?

So, bump CMake (to 3.19?) and go with option 2?

Yes, let's do that

For the record, CMake 3.16 was released November 2019, and CMake 3.19 November 2020.
Default versions:
Ubuntu 20.04: 3.16.3
Ubuntu 22.04: 3.22.1
Frontier: 3.20.4

Configuration errors in the Windows build:
CMake Error in CMakeLists.txt:
  IMPORTED_LOCATION not set for imported target "Kokkos::kokkoscore"
  configuration "RelWithDebInfo".

Weird.

Frontier has a 3.27.9 module.  I would require 3.22

I would require 3.22

Why 3.22?

I would require 3.22

Why 3.22? And what's Kokkos plan for CMake version update?

Arbitrary, based on the 22.04 LTS.
Unsure but likely >= 3.22

Windows build still fails.

Windows build still fails.

What about explicitly building for Debug via
cmake --build . --config Debug

?

I think we would rather need to specify the configuration when building ArborX since that's when it's complaining about all kind of other configurations for Kokkos missing.


Ale is too severe by default

I'm trying to learn python. I need the ale plugin to show the errors that I make.
This "ale" is so severe that I can't even past the code from the official python tutorials without an infinite number of warnings about style.
Can you please explain how to make a basic configuration that highlights just the errors whithout constantly shouting me that the code that I've pasted is wrong?
The default level is too severe. It's like starting a videogame from the last level instead of starting from simpler levels.
thanks :-)

You have to configure the linters to control which errors or warnings are shown. ALE just reports what they said.

LOL

ALE itself can't restrict how many warnings the linters themselves provide. This must be done through whichever linter you're using. You can find which linters are enabled by using the :ALEInfo command in Vim, then look for the "enabled linters" list. Then, go to Google, and look up "how to disable  warning for  linter"; for example, "how to disable missing docstring warning for pylint linter". This is out of scope of ALE.


WASM AdvanceComp

http://www.advancemame.it/download

AdvanceComp is still better than oxi in some cases. I'd like to keep this open. Or is the plan to close it but keep it in the "codecs support" project?

adding help wanted label incase anyone wants to tackle supporting this.

There is nothing speaking against providing multiple PNG encoders, so this one is definitely up for grabs.


Uploader - fix window auto update

Context
When update the window version, I see this error. I cannot use the app after update and I had to uninstall and install the app again to make it works.

Expected results

[ ] Can update to the latest version from the popup

Released in v1.2.2 🚀


Fix incorrect heap size value for boxed slice

Previously the GetSize impl for a boxed slice didn't have a bound on T, so get_size was being called on &T instead of T. This meant that the size of each element was always being calculated to the size of a reference. This change fixes that and adds a test.

@DKerp ping

@CeleritasCelery Ping

Hey @bircni Thanks for taking ownership of this code! I am going to switch to using get-size2 since it has this fix among other improvements.

No Problem @CeleritasCelery please feel free to raise other issues or pull requests !

the next version I publish will include your changes!

Also opened an issue here:
https://github.com/rustsec/advisory-db/issues/2078


revisit the ManagedClusterConditionAvailable condition

Describe the enhancement
Today the ManagedCluster condition ManagedClusterConditionAvailable status could be True, False and Unknown.
The status is set to false when kube API server is unavailable, and to unknown when lease stop update.
Some feedback shows that this condition will be flapping, for example, when API server is not stable. Log this issue to revisit the conditions to set ManagedClusterConditionAvailable status.

cc @qiujian16

cc @haowells


Make MultisigTransaction['value'] non-nullable

Summary
Prior to #1385, MultisigTransaction['value'] was not nullable. The Transaction Service returns a Uint256Field value, which is inherently a Field and returns a value by default.
Changes

Adjust the MuiltisigTransactionSchema to not have value as nullable.
Adjust associated tests.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.04%) to 91.845%

Files with Coverage Reduction
New Missed Lines
%

src/routes/transactions/entities/tests/human-description.builder.ts
4
60.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
6892

Relevant Lines:
7239

💛  - Coveralls