Bad Sad on openapi gen-client command

Description:
I invoked ballerina openapi gen-client o2j:postmanclient collection.json on the json schema at:
https://schema.getpostman.com/collection/json/v2.1.0/draft-07/collection.json

it resulted in following error in the ballerina-internal.log file:
java.lang.NullPointerException
        at org.ballerinalang.openapi.CodeGenerator.generateBalSource(CodeGenerator.java:190)
        at org.ballerinalang.openapi.CodeGenerator.generate(CodeGenerator.java:121)
        at org.ballerinalang.openapi.CodeGenerator.generateClient(CodeGenerator.java:140)
        at org.ballerinalang.openapi.cmd.OpenApiGenClientCmd.execute(OpenApiGenClientCmd.java:94)
        at java.util.Optional.ifPresent(Optional.java:159)
        at org.ballerinalang.tool.Main.main(Main.java:57)

Steps to reproduce:

Create a ballerina project and add an module ( e.g. o2j)
Download the file https://schema.getpostman.com/collection/json/v2.1.0/draft-07/collection.json
run above command and observe the error in the log file.

Affected Versions:
1.2.8
OS, DB, other environment details and versions:
Related Issues (optional):
Suggested Labels (optional):
bug
Suggested Assignees (optional):

This issue closes since the above YAML file containing the parser issues.

In case if someone else too got the same error, I got this for openapi: 3.1.0. Changing to openapi: 3.0.1 resolved it.


Navigating to function contained in .fpkg results in error

Should probably be fixed in the LSP

I think this is OK for now.


Implement vim.ui.input over vim.fn.input

Ideally nvim-surround should allow the use of vim.ui.input to take in user inputs, to let users modify the look and feel of the inputs (e.g. using dressing.nvim). See #17 and the the latter half of #5

Closing this for now as I still have no clue how this would be done, and the plugin works just fine without vim.ui.input

I think I have a solution, if you want to reopen this.
Basic idea: the code here:
https://github.com/kylechui/nvim-surround/blob/04293f3ec9a635b53ff4f3b6fa8cc5789e17e51f/lua/nvim-surround/utils.lua#L49-L54
should be changed to use vim.ui.input. As I think you've noticed, it's not a trivial change - vim.ui.input is asynchornous and takes a callback.
If we don't want to change the API of get_input, probably adding plenary as a dependency and using its channels should be a sufficient, if imperfect, solution; we can have the main function block on recieving the value over the channel, and have the callback capture the sender and send the value to the outer scope. I can try to write a PR if you want.

@nihilistkitten If you would like to take a shot at it, be my guest! I will just note that depending on the complexity of the code, I reserve the right to reject such a PR as this is more of a "quality of life" enhancement as opposed to a "real" feature. With that aside, the main issue for me was that although I was able to get some working version done via coroutines, it required non-trivial refactors to all of the functions that called utils.get_input(), of which there are ~4 (not including the potential ones that users are using for custom surrounds). If you can somehow make this work without sacrificing on simplicity, I'll gladly merge this in :)

Totally makes sense. Should only be a few lines and not super complicated, if definitley moreso, and of course it does add a dependency (albeit a very common one). I'll take a shot at it tonight or tomorrow.

Bleh, ok. Looks like this is a lot more difficult than I thought. What we really want is the ability to block (in a sync context) on the return of an async function, but we don't seem to have that in the language ATM (at least as far as I can tell). Guess this should be close again, unless you want to leave it open for someone with more plenary experience than me :(.

Haha no worries; yeah there's no way to do it without treating "main" as another thread and having to do massive refactors to that instead (to my knowledge). Thanks for the attempt though :)


allocator instance support in sampling

Pull Request Test Coverage Report for Build 538699150

58 of 61 (95.08%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 94.41%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sampling/include/var_opt_sketch_impl.hpp
51
54
94.44%

Totals

Change from base Build 538593935:
0.004%

Covered Lines:
4526

Relevant Lines:
4794

💛  - Coveralls


[SPARK-16988] [Spark Shell] spark history server log needs to be fixed to show https url when ssl is enabled

spark history server log needs to be fixed to show https url when ssl is enabled

Can one of the admins verify this patch?

Thank you for your comments. I’ll fix my program

Hey @hayashidac , you did't need to close this PR. You should have just pushed your change...

I applied comments. Please check again.

Can one of the admins verify this patch?

sorry, I fixed it.

Jenkins, test this please.

Test build #67505 has started for PR 15611 at commit 6f26e30.

While I I noticed that all of spark.ui.ssl in SSLOptionsSuite.scala should be spark.ssl.ui. It's very minor so you can fix it too within this PR if you would like to.

Looking good, thank you. I also wouldn't mind fixing the "spark.ui.ssl" -> "spark.ssl.ui" issue here because it's a little bit related.

I fixed SSLOptionsSuite.scala too.

Test build #67505 has finished for PR 15611 at commit 6f26e30.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67505/
Test PASSed.

ok to test.

Test build #67513 has started for PR 15611 at commit af73518.

Test build #67513 has finished for PR 15611 at commit af73518.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67513/
Test PASSed.

LGTM. Merging into master and branch-2. Thanks @hayashidac .


Real World Maps using Infraworks

Hi,
I'm doing research on safety of AVs and I need real world 3d maps for the UE4 level. I've been web surfing for solutions for quite a while. Autodesk's Infraworks can easily connect to open databases such as OpenStreetMap and automatically build a 3d model that includes roads, buildings and ground imagery. Then you can export the model to an fbx format and import it in UE4. It would be great if the roadmap could be generated according to the roads in the fbx file.
Best,
Abol

Hi @abol-karimi
if you're looking for integrating real world road networks, there is already a plugin by one of the Epic Games developers that supports the generation of roads and buildings: https://github.com/ue4plugins/StreetMap
IMHO a deeper integration of realworld roads is difficult at the moment, but might be easier based on xml/osm data than fbx files, as they are text files for meshes. Still, creating real life like roads from open databases is a task that requires some work done by yourself. Curves or intersections are stored as multiple segments, I for myself use the road network as a base for dragging splines along the roads and attaching RoutePlanners.
Maybe you can somehow make use of the plugin. But I'd like to see open database integration too. :)

Hello @abol-karimi @p-schulz.
This is a feature that we absolutely want to incorporate in CARLA, but is not a easy task.
We are thinking on a procedural way to generate all these meshes, paths, buildings and so on, so you can use an easy tool to manually or automatically create all the roads you want. There is a lot of things to considere here, so as mentioned before, this is not trivial and we are slowly but carefully working on it.
Right now you can import models from CityEngine to Unreal Engine. It's a little bit tedious and we have no automated way to generate the paths for vehicles and pedestrians but... it's something.
Here you have a work in progress sample that @XGodina is working on, using CityEngine to generate the roads from OpenStreetMaps and our assets for buildings and so on.

This is a great idea, and it is a fantastic and futuristic feature for the Carla. I believe it is vital for the self-driving car research.

no automated way to generate the paths for vehicles and pedestrians but... it's something.

Hi! Thanks for contributing. This is exactly what I need. 👍
I wonder if there's any notable progress about this work or will you add such feature in future release?


Override JavaScript methods

Describe the bug
I have the following TypeScript class where one of the methods needs to be overridden in the implementation. I am ensuring this in the constructor.

Implementation of the above class in Python,

The foreign application implementing the above class is experiencing unexpected results even after the 'dummy_green_method' is overridden in the implementation. I expect the super constructor not to throw an error in the implementation because the method has been overridden. However, this is not the case. Please provide guidance on what I might be missing here.
Expected Behavior
The above implementation should throw an error while declaring an object.
Current Behavior
It's throwing an error because in the base constructor dummy_green_method is not overridden.
Reproduction Steps
Provided in the problem description.
Possible Solution
No response
Additional Information/Context
No response
SDK version used
1.69.0
Environment details (OS name and version, etc.)
macOS/13.6 (22G120)

I suspect this might not be possible as described here: https://aws.github.io/jsii/user-guides/language-support/callbacks/
However the case is definitely not clearly described in the docs. Needs checking.


[Improvement][Style] add autolink to issues in PR title and commit msg

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

What would you like to be improved?
It's common and considered best practice to be able to click links to corresponding issues/PRs/tickets in the commit msg. To achieve this features, Github's autolinked references and links^1 are used.
For example, Spark uses this feature to autolink Jiras.

Since Uniffle are using Github Issues to track issues/request/improvements, etc. The issue is effective the same as the Jira ticket.
I'd like to propose an suggestion to improve the code context experience: add autolink to issues in PR title and commit msg.
How should we improve?

Let's decide which form to include autolinks:

[#xxx], the simple form, https://github.com/apache/incubator-uniffle/commit/cf9afab471a9595d8298ebd65c4d25c080ad4bbf

[ISSUE #xxx] prefixed with the ISSUE

Enforce the pr title check.  Every PR should have an issue linked.
There's an exception rule for this: PR title prefixed with [Minor]/[MINOR] can be excused for this enforcement check.

Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Can we use the apache style like [UNIFFLE-120][Test] Fix msg ? And the uniffle-120 could be linked to github issue by https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/managing-repository-settings/configuring-autolinks-to-reference-external-resources

UNIFFLE-120 may be confusing, because it's usually a Jira id.
Currently I would not recommend Jira because the self registration is limited. We wish to make new contributors easier to join the project.
For how to configure the autolink, see https://cwiki.apache.org/confluence/display/INFRA/Git+-+.asf.yaml+features

The point to add issue id in the title is that we can links to issues and commits in Github's ecosystem. In most cases, only title is displayed for commits.

My point is, by just looking at the issue id, you don't get any information.
When looking at commit history, you got interested by the scope and type of the change first.
Then you start to looking for why it's needed, and find the related issue.
Just click the 3 dots, you could see it the description.

It's ok for me.

It's OK for me, thanks for your effort. @advancedxy

kaijchen zuston jerqi felixcheung WDYT?

also cc @xianjingfeng which was pointed to a wrong one.

It's OK for me.

Seems the following format is preferred now?
[#123] feat: add xxx feature
[#124] fix: exception in xxx

How about combine issue ID and semantic commit message ?
[<issue>]<type>(<scope>): <subject>

For example:
[#123]feat(operator): support feature xxx
[minor]refactor: fix typo in variable name

Hi everyone, I've edited existing open PR titles (to show examples to new PR). How do you think?


REST API is no longer an option in 6 countries

Due to a recent approval of a patent (in 6 countries) outlining rest API via a server, for same day settlement on payment between users, A rest API is no longer a valid option (not without challenged legal action).
see https://www.google.com/patents/WO2016099492A1
Connecting directly with the mobile methods is now the only option for direct connection then using the server as a validation point.
This probably needs to be removed from the wiki as a possible option or at least a note made about it.

IP concerns should be forwarded to<EMAIL_ADDRESS>


not able to find deviceupdate-agent in packages.microsoft.com

As mentioned on the page.
"Install the deviceupdate-agent on the IoT device. Download the latest Device Update Debian file from packages.microsoft.com:"
However I am unable to find the deviceupdate-agent.deb package.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9fba65fb-7e52-5f21-80ed-0744003598b7
Version Independent ID: 3cc6c5cd-1414-d9d9-5b9d-c2dc6170c225
Content: Complete a proxy update by using Device Update for Azure IoT Hub
Content Source: articles/iot-hub-device-update/device-update-howto-proxy-updates.md
Service: iot-hub-device-update
GitHub Login: @valls
Microsoft Alias: valls

@petoor Thanks for your comment. We will review the issue and get back to you shortly.

@petoor Are you saying Debian file is not available on https://packages.microsoft.com/debian/ ? or by running the command sudo apt-get install deviceupdate-agent is not downloading the package?

It might be avaiable but i for sure cant find it in https://packages.microsoft.com/debian/
Whats the path to the file?

@petoor Should be available here: deviceupdate-agent_0.8.2_public_preview_amd64.deb Even i haven't found it. If you are blocked and file is not downloaded when you run the command sudo apt-get install deviceupdate-agent, please download the .deb file from here: https://github.com/Azure/iot-hub-device-update/releases/tag/0.8.2
I will reach out internally and update you regarding the exact location of the file if available or is missing.

Thanks. Which versions of debian and ubuntu does it work on?

Every release has the details about it. Please see Notable Enhancements section for details.
https://github.com/Azure/iot-hub-device-update/releases

@AshokPeddakotla-MSFT  that doesnt answer the question.
Does this file work for old debian versions and if so, how old? The same goes for ubuntu

@petoor Apologies for not being so clear on this query. I'm able to find the package under this path : https://packages.microsoft.com/ubuntu/18.04/multiarch/prod/pool/main/d/deviceupdate-agent/

Does this file work for old debian versions and if so, how old? The same goes for ubuntu

I'm not seeing any restrictions on using the previous versions. Do you have any specific version which you have tried and not working?  It works with previous Debian versions as well and also works with the Ubuntu 18.04 as mentioned in the docs.
Do let us know if you have any further queries.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Support client params from a single positional arg

This is an alternative to those that don't prefer to use
**kwargs for client calls.
I'd like to get some feedback on this before merging.  We've had some concern about using **kwargs.   We'd also heard that using the CamelCasing in our **kwargs annoys tools such as pylint.  This adds an option to make calls via:
client = session.create_client('dynamodb')
client.describe_table({"TableName": "foo"})

thoughts?  cc @kyleknap @mtdowling @rayluo

Looks good. One thing that would be nice to check is that boto3, still works as expected. Maybe run the tests to make sure nothing is unexpectedly broken with the the resource interface. :ship:

Ran boto3 tests against this change, they passed.

Cool. Good with me to merge then.

Nice! Ship it! :shipit:

FWIW, the following should already work without this change:
client = session.create_client('dynamodb')
client.describe_table(**{"TableName": "foo"})

Why not just rely on that?

There seems to be an agreement with the original author of this PR (@jamesis) that it should not be merged (see #230). Perhaps this PR should be closed?

Yeah, I think fi we don't get feedback additional feedback by tomorrow, we can go ahead and pass on this pull request for the time being.

Closing due to lack of interest (https://github.com/boto/boto3/issues/230).  We can look at this again in the future if there's any future interest.


Cannot use login.live.com as an OIDC authority

Please follow the issue template below. Failure to do so will result in a delay in answering your question.

Library

[x]<EMAIL_ADDRESS>
Framework
Ad-hoc testing in Browser F12 console.
Description
Would like to use the login.live.com OAuth2 provider that is OIDC compliant by providing it as the authority.  However, a runtime error occurs when instantiating MSAL before any request is made.
Error Message
msal-browser.js:2173 Uncaught ClientConfigurationError: URL could not be parsed into appropriate segments. Given Error: Given url string: https://login.live.com/
at ClientConfigurationError.AuthError [as constructor] (https://alcdn.msauth.net/browser/2.0.0-beta.0/js/msal-browser.js:191:32)
at ClientConfigurationError.ClientAuthError [as constructor] (https://alcdn.msauth.net/browser/2.0.0-beta.0/js/msal-browser.js:416:32)
at new ClientConfigurationError (https://alcdn.msauth.net/browser/2.0.0-beta.0/js/msal-browser.js:1473:32)
at Function.ClientConfigurationError.createUrlParseError (https://alcdn.msauth.net/browser/2.0.0-beta.0/js/msal-browser.js:1508:20)
at UrlString.validateAsUri (https://alcdn.msauth.net/browser/2.0.0-beta.0/js/msal-browser.js:2173:48)
at AadAuthority.set [as canonicalAuthority] (https://alcdn.msauth.net/browser/2.0.0-beta.0/js/msal-browser.js:2570:42)
at AadAuthority.Authority (https://alcdn.msauth.net/browser/2.0.0-beta.0/js/msal-browser.js:2554:37)
at new AadAuthority (https://alcdn.msauth.net/browser/2.0.0-beta.0/js/msal-browser.js:2736:27)
at Function.AuthorityFactory.createInstance (https://alcdn.msauth.net/browser/2.0.0-beta.0/js/msal-browser.js:2817:28)
at new AuthorizationCodeModule (https://alcdn.msauth.net/browser/2.0.0-beta.0/js/msal-browser.js:2846:63)
Security

[ ] Is this issue security related?

Regression

[ ] Did this behavior work before?
Version:

MSAL Configuration
const msalConfig = {
  auth: {
    clientId: "292666", // valid LiveID app ID.
    authority: "https://login.live.com", //The MSA OIDC authority (e.g. https://login.live.com/.well-known/openid-configuration is off of this)
    redirectUri: "https://localhost",
  },
  cache: {
    cacheLocation: "sessionStorage", // This configures where your cache will be stored
    storeAuthStateInCookie: false, // Set this to "true" if you are having issues on IE11 or Edge
  }
};

Reproduction steps
const myMSALObj = new msal.PublicClientApplication(msalConfig);

Expected behavior
Able to continue on to requesting an auth code.
Browsers/Environment

[x] Chrome
[ ] Firefox
[ ] Edge
[ ] Safari
[ ] IE
[ ] Other (Please add browser name here)

@hpsin Thanks for raising this, @pkanher617 will be primarily driving these 2.0 bugfixes. Moving it over to the bug board to take a deeper look.

Additional details:

If you remove the segments check in line 2172 (  if (!components.HostNameAndPort || !components.PathSegments || components.PathSegments.length < 0) {) then the object can be created.  This line appears to assume that all authorities will have a /tenant/ section in their authority.
Once the object is created, authorization endpoint lookup fails because MSAL assumes that the authority is an AAD endpoint, and uses the internal lookup API to do discovery, which fails:
https://login.microsoftonline.com/common/discovery/instance?api-version=1.0&authorization_endpoint=https://login.live.com/oauth2/v2.0/authorize
There's a smart lookup that tries to do OIDC discovery, but only when the authority is a known OIDC endpoint:

if (this.isInTrustedHostList(this.canonicalAuthorityUrlComponents.HostNameAndPort)) {
                                return [2 /*return*/, this.defaultOpenIdConfigurationEndpoint];
...
    var AADTrustedHostList = [
        "login.windows.net",
        "login.chinacloudapi.cn",
        "login.cloudgovapi.us",
        "login.microsoftonline.com",
        "login.microsoftonline.de",
        "login.microsoftonline.us",
    ];

I think this is B2C logic  -if you can't find the AAD authority, fall back to checking B2C.  I'd have expected the reverse, where an authority is assumed to be 3rd party OIDC instead of a special Microsoft IDP that will be in a lookup on login.mso.com.

Once the object is created, authorization endpoint lookup fails because MSAL assumes that the authority is an AAD endpoint, and uses the internal lookup API to do discovery, which fails:
https://login.microsoftonline.com/common/discovery/instance?api-version=1.0&authorization_endpoint=https://login.live.com/oauth2/v2.0/authorize

@hpsin This part can be resolved by passing knownAuthorities: ["login.live.com"] to your msalConfig.auth config but it will then fail on the openid-configuration call because we currently only support the v2.0 endpoint. Supporting the v1 well-known endpoint is something we can look into for scenarios such as this.

I think this is B2C logic -if you can't find the AAD authority, fall back to checking B2C. I'd have expected the reverse, where an authority is assumed to be 3rd party OIDC instead of a special Microsoft IDP that will be in a lookup on login.mso.com.

The logic actually is as you expect, sort of. If the previously mentioned knownAuthorities parameter is provided, we assume B2C/ADFS. Then if not, we fallback to the AAD lookup. Additionally while in theory 3P OIDC IDPs could work, we don't officially support that scenario.

Thanks @tnorling ! That's helpful.  I'm a bit confused about what fails in the OIDC config part - .well-known/... is not a v2-specific item, so should work on all platforms. Note this isn't the v1 well-known endpoint, it's just "a well known endpoint off of an OIDC authority"
That said - ACK, thanks for confirming.  It would be great to see MSAL be used for other providers but I understand why it's not.

@hpsin To clarify, we currently build the well-known endpoint like so: authority + "/v2.0/.well-known/openid-configuration" unless the authority is an adfs authority in which case we leave off the "v2.0" portion of the url

Ahhh, right, That's a bummer :( I'd have assumed that people use the https://login.microsoftonline.com/common/v2.0/ as the authority.  Makes sense.

@hpsin  does @tnorling  suggestion using known authorities to skip openId config work for you

No, because it falls back to calling :
https://login.microsoftonline.com/common/discovery/instance?api-version=1.0&authorization_endpoint=https://login.live.com/oauth2/v2.0/authorize 400 (Bad Request)
That's with
var msalConfig = {
  auth: {
    clientId: "292666",
    validateAuthority:false,
    knownAuthorities: ["login.live.com"],
    authority: "https://login.live.com/",
    redirectUri: "https://localhost",
  },

Hmm I would not expect it to hit that endpoint when providing knownAuthorities. I just copied that config to try it myself and it's failing on the openId-config call as I would expect because it's trying to hit https://login.live.com/v2.0/.well-known/openid-configuration instead of https://login.live.com/.well-known/openid-configuration.
In any event, I don't expect this scenario to work without additional changes to the library so we'll need to discuss with the team if this should be a supported scenario and, if so, what those changes should look like.
To summarize there's 2 issues currently preventing the library from using login.live.com as an authority:

Library expects tenant to always be a part of the provided authority
Library always appends "v2.0" to the authority when making the .well-known/openid-configuration call

@jmckennon will address this alongside #2120

Are there any update or workarounds for this issue?

@tom171296 I'm currently working on this fix, I will link the PR when it's up.

🎉🎉🎉🎉🎊🎊🎊🎊🥳🥳🥳🥳
This is awesome. I can't wait to try it out! This is the first officially recommendable library for login.live.com in a while.


CronWorkflow name attribute question

The CronWorkflow object suspend and resume methods have a required name parameter.  Can this parameter be optional and default to the name attribute of the CronWorkflow?

Hey @bobh66! Thanks for submitting this question! 🙂
Yes, I think this would be a nice addition! I just looked through the CronWorkflow and I think I may have missed the fact that we merged this with a required parameter, rather than no name parameter, actually. So, I must first apologize for this... I will pay more attention to this type of detail going forward!
I think I completely agree with your implicit assessment that it should default to self.name upon submission. The service can still maintain the interface that requires a name because it can be used independently of the CronWorkflow object.
Would you like to submit a PR? 🙂

Thanks @flaviuvadan - I'll push a PR as soon as I'm done with the labels work.

Going to close this for now and continue the discussion on a PR 🙂 Thank you @bobh66!


Specifying comments for Intellisense in .idl files

I'm trying to export my .NET style comments to XML so they can be consumed by third party developers along with the resulting .winmd. Unfortunately it appears that .idl are either ignored entirely by Visual Studio or they're not properly supported.
If I place the documentation comments in the equivalent header they are indeed picked up and placed in the resulting .xml, but ofcourse the namespaces don't align with those used in the .winmd.
Am I doing something wrong or are they just not supported?

Hi @ChristopherLeaWacom -- this list is for feedback on our docs. Can you give me a URL for a doc relevant to your questions? This will help me route your question to the correct author. Thanks!

Apologies, I ended up here through lots of Googling and a similar issue posted here: https://github.com/MicrosoftDocs/winrt-related/issues/82.
Where else should I ask? I guess the fact there's no documentation about documentation is my issue, but it's hard to tell if it's even supported.

Hi, @ChristopherLeaWacom, your instinct that this isn't supported is correct, and we apologize for the gap in the tooling. XML documentation (as described in https://docs.microsoft.com/dotnet/csharp/programming-guide/xmldoc/xml-documentation-comments), and compiling with the /doc option is a solution for comments within .NET source code, but not for comments within an IDL file. The IDL comments are not processed even if you use /doc to compile C++/WinRT.
The only recourse currently (but it's manual, error-prone work), is to replicate the .idl content structure and copy-paste the comments. That would mimic the kind of .xml documentation file that's generated for C#. And then you'd see IntelliSense correctly light-up when ingesting, say, a NuGet package containing a C++/WinRT Component. Here's an example of how that'd look, IDL first and then .xml file.
namespace RuntimeComponent2
{
[default_interface]
/// <summary>
/// This class is a test class written in C++/WinRT
/// </summary>
runtimeclass Class
{
/// <summary>
/// This is the class constructor
/// </summary>
Class();
/// <summary>
/// This method does something new
/// </summary>
/// <returns>The result</returns>
Boolean DoSomething();
}
}

<?xml version="1.0"?>
<doc>
<assembly>
"RuntimeComponent2"
</assembly>
<members>
<member name="T:RuntimeComponent2.Class">
<summary>
This class is a test class written in C++/Winrt
</summary>
</member>
<member name="M:RuntimeComponent2.Class.#ctor">
<summary>
constructor of this class
</summary>
</member>
<member name="M:RuntimeComponent2.Class.DoSomething">
<summary>
This method does something new
</summary>
<returns>The result</returns>
</member>
</members>
</doc>

If you want to lobby for this feature, you could contact the
Visual Studio User Voice feedback website, or the Developer Community. Also, just FYI, for issues, or suspected product bugs, there's Microsoft Support; a support engineer will look at the repro and see whether the issue is a Microsoft product bug, or something else. If it's a bug with a Microsoft product then the ticket will be free of charge.
I'll leave this open for a while; please close if you feel like it's resolved.
~Steve

This issue feels as resolved as we can get it within this documentation-related feedback channel. So I'll go ahead and close it. Thanks! ~Steve


Change AdjustIndistinguishableColors to an enum setting instead of a boolean setting

Summary of the Pull Request
AdjustIndistinguishableColors can now be set to:

Never: Never adjust the colors
Indexed: Only adjust colors that are part of the color scheme
Always: Always adjust the colors

References
#13343
PR Checklist

[ ] Closes #xxx
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx
[ ] Schema updated.
[x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Validation Steps Performed

We probably want to map true to Indexed and false to Never for legacy purposes (otherwise users will get a settings error if they previously had this setting set)

And don't forget to update the schema/docs plz


add zabbix to "projects using duktape"

As per https://www.zabbix.com/documentation/4.2/manual/appendix/items/preprocessing_javascript (see the last paragraph), would be useful to add Zabbix in https://wiki.duktape.org/ProjectsUsingDuktape.html .

Thanks @richlv, I'll add it to the list.


closed model in vrp_initial_routes and minor comment fix

Hello,

When using an initial solution in routing, the model has to be closed, or else the solver will ignore the search parameters. This is a known issue (for lack of a better word) actually.
The model was not closed in the vrp_initial_routes file, so made the necessary changes.

Thanks Very Much.

Hello,
Any update on this?
Thanks! :-)

Let us release 9.5 today or tomorrow (hopefully). Then we will merge the PR.  Thanks


🐛Multiple wells are created when copying a log to another server

:bug: Describe the bug
When copying a log to a server where the parent well does not exist, a temporary well is created in addition to the actual well.

Avoid creating temporary wells as this behavior is not recommended.
To Reproduce
Steps to reproduce the behavior:

Go to the source server.
Find a log in a well that does not exist in the target server.
Right click the log and select "Copy 1 log to server" and select your target server.
When asked "The wellUid does not exist on the target server. Do you want to copy it and its parent well? (image below), select Copy
Then navigate to your target server fast, and you will see the temporary well.

Expected behavior
Not creating a duplicated well temporarily.

Cannot reproduce it, and I think it was solved with the use of tanstack query instead of navigationState.


Code to periodically clean up empty tserver nodes in ZK may need to be generlized

The Accumulo manager periodically calls LiveTserverSet.scanServers() which will delete any tservers nodes in ZK that have no locks among other things.  The functionality of periodically cleaning up empty ZK nodes could also be done for compactors and scan servers (not sure if code can be shared for this).  Is there similar code that cleans up scan server and compactor entries in ZK?  If not, then need to create it.  If there is then this issue can be closed.
This potential problem in exists elasticity, but its probably a need in earlier versions as well.  The following are a few links to code that do this clean up.
https://github.com/apache/accumulo/blob/39c92710683bf75c41fd70e1f21b39ad376dd84d/server/manager/src/main/java/org/apache/accumulo/manager/Manager.java#L1014
https://github.com/apache/accumulo/blob/39c92710683bf75c41fd70e1f21b39ad376dd84d/server/base/src/main/java/org/apache/accumulo/server/manager/LiveTServerSet.java#L223
https://github.com/apache/accumulo/blob/39c92710683bf75c41fd70e1f21b39ad376dd84d/server/base/src/main/java/org/apache/accumulo/server/manager/LiveTServerSet.java#L282

Found this code that cleans up compactors. Have not found anything for scan servers yet.
For scan servers, looked at what used the constant Constants.ZSSERVERS and did not see any cleanup code directly referenced from that.

dupe of #4559


[TF 2.0 API Docs] Added Returns and Raises sections to tf.image.convert_image_dtype

Added missing documentation sections. #29406

Closing this PR as this not against master, please open a new PR against master
CC @mihaimaruseac


Cast exception when calling increment

I am getting a cast exception when calling the increment method in some instances
java.lang.ClassCastException: com.google.appengine.api.datastore.Entity cannot be cast to com.theupswell.appengine.counter.data.CounterData
	at com.theupswell.appengine.counter.service.ShardedCounterServiceImpl$3.run(ShardedCounterServiceImpl.java:904)
	at com.theupswell.appengine.counter.service.ShardedCounterServiceImpl$3.run(ShardedCounterServiceImpl.java:900)
	at com.googlecode.objectify.impl.TransactorYes.transact(TransactorYes.java:102)
	at com.googlecode.objectify.impl.ObjectifyImpl.transact(ObjectifyImpl.java:178)
	at com.theupswell.appengine.counter.service.ShardedCounterServiceImpl.getOrCreateCounterData(ShardedCounterServiceImpl.java:899)
	at com.theupswell.appengine.counter.service.ShardedCounterServiceImpl$IncrementShardWork.run(ShardedCounterServiceImpl.java:447)
	at com.theupswell.appengine.counter.service.ShardedCounterServiceImpl$IncrementShardWork.run(ShardedCounterServiceImpl.java:412)
	at com.googlecode.objectify.impl.TransactorNo.transactOnce(TransactorNo.java:118)
	at com.googlecode.objectify.impl.TransactorNo.transactNew(TransactorNo.java:95)
	at com.googlecode.objectify.impl.TransactorNo.transact(TransactorNo.java:85)
	at com.googlecode.objectify.impl.ObjectifyImpl.transact(ObjectifyImpl.java:178)
	at com.theupswell.appengine.counter.service.ShardedCounterServiceImpl.increment(ShardedCounterServiceImpl.java:389)

Objectify best practice (https://github.com/objectify/objectify/wiki/BestPractices) recommends using your own objectify class to ensure entities are registered. This seems to be happening because entities are not registered because counter uses the objectify service provided and relies on the entities already having been registered else where.
It would be good if the ShardedCounterServiceImpl accepted an objectify service. For now I just create a factory that registers the entities before instantiating and returning ShardedCounterServiceImpl.

Hi, thanks for following up on this issue.  Yes, as you pointed out, currently appengine-counter doesn't automatically register its entities for you.  You'll need to manually register them along with your other entities -- a factory like you indicated is the ideal place.  Motomapia (created by Jeff, the creator of objectify) provides a good example:
https://github.com/stickfigure/motomapia/blob/master/src/main/java/com/motomapia/OfyFactory.java
We don't want ShardedCounterServiceImpl to accept an ObjectifyService in the constructor because this might cause problems.  See here for more details:
Can I use Dependency Injection with Objectify
However, thinking about this more, it seems like there's only ever a single ObjectifyFactory floating around, which is where these entities would be registered, so it might make sense to add another constructor to ShardedCounterServiceImpl that requires an ObjectifyFactory.  That way, when you construct the ShardedCounterService (or when you DI system does) appengine-counter could auto-register the entities.  I'll experiment with that to see if it can work.

Closing this as a non-issue, but capturing the enhancement request as part of #23.  Please re-open if this is still an issue for you.


Fatal error: Login: Checkpoint required. Point your browser to [link]

i did verify the login through that link in the browser [this was me]. But still it won’t let me login as it generate new "Fatal error: Login: Checkpoint required. Point your browser to [link]" with new link over and over even after verification.
is this just me?
i'm no programmer, correct me if i'm wrong. From what i understand, instagram detect my login on browser and my login on instaloader (terminal) is 2 different device/login. Is there any command like --cookies which you can f.e. grab cookies.txt from browser and use it on instaloader --cookies examples.txt
Thank you.

do you search for https://instaloader.github.io/troubleshooting.html#login-error ?

yeah i saw, then again i'm no progammer i don’t really know how it works lol


Update CHANGELOG.md for Grace Kelly

TODO:

[ ] Discourse post for the new release
[ ] A post about node ordering/running
[ ] Double check links
[ ] Update docs for new/changed functionality
[ ] Add contributors to the changelog

Any idea when Grace Kelly should land?

We're releasing Grace Kelly (pending any bugs found in the RC period) in early January

Feel free to install from the RC in the meantime if you have a pressing need for the features in this release, or if you're interested in helping us test :) https://pypi.org/project/dbt/0.12.2rc1/


Code Refactoring - Part 3 - Use the @agoracloud/common Package

This issue is part 3 of the multi part process of refactoring the code base.
In this issue, Waleed will work on removing all the existing copied over DTO classes from the server and use all the DTO available from the @agoracloud/common package.
To Do:

[ ] Remove all current DTO classes
[ ] Import all needed DTO classes from @agoracloud/common
[ ] Capitalize all form input field titles and custom validation messages

By integrating the mars-man/models package into the UI, @MarcDAFrame is using the DTO's from the @agoracloud/common package. We still need to clean up the old DTO classes but Marc was added as an assignee to track work.

Done 🎉

Done 🎉


Make initializer optional

Just a small fix (I think); an error was being thrown if no initializer directory existed. As the docs imply this is optional, I assume the code is supposed to enforce/support this.

Good catch, thanks! 🎅


Publish dialog revamp.

Progress bar
Icon preview
Public/private switch

Prototype:
https://app.atomic.io/d/nC4yDbM6PkgR
Note that the spinner on the "publishing" button should spin :)  – limitations of Atomic for animated elements
Comp file reference: Publish Dialogue 160322_v2

also #1079 #1178


React duplicate key warning with nl2br: true and two child strings

Hello! Great library. Please let me know if you have any questions on this bug!
Note: This also occurred in v3 (with a different key name in the error).
Repro
const opts = { nl2br: true };

const foo = `hello

`;
const bar = `hello

`;

const App = () => (
  <Linkify options={opts}>
    {foo} {bar}
  </Linkify>
);

Codesandbox: https://codesandbox.io/s/react-testing-library-demo-forked-tlok5c?file=/src/index.js
Expected result
No duplicate key warning. (Everything seems to be fine other than the warning in the console.)

Hi @aeharding, thanks for reporting, should be fixed in the latest v4.0.1 release


RuntimeError: HHblits failed

I can run alphafold, but everytime it  fails at   line 144 in  hhblits.py.
Does anyone face this situation?
Thank you!

I have the same error

I don't know why this error happens,  could anyone find the reason? thanks!

I don't know why this error happens, could anyone find the reason? thanks!

Alphafold worked for me when I used this this PR fix: https://github.com/deepmind/alphafold/pull/36
But when I use the last master commit https://github.com/deepmind/alphafold/tree/1e216f93f06aa04aa699562f504db1d02c3b704c
I have the error above

When I try and build the docker file (trying my best to use consistent versions of software), I always see warnings in the compilation of hhblits.  The warnings vary depending the base version of ubuntu used, because each version uses a different default compiler version.
FWIW, I see the fewest compilation warnings using ubuntu 16.04, which is the version currently in the repo's Dockerfile.  (Note that it is currently passed end-of-life and is no longer supported.)  My PR #36 used 20.04, which does display more warnings.
This is not to say the warnings are a problem.  Just an observation that you may get different behavior using different compilers, especially if there is some memory related problem.

When I try and build the docker file (trying my best to use consistent versions of software), I always see warnings in the compilation of hhblits. The warnings vary depending the base version of ubuntu used, because each version uses a different default compiler version.
FWIW, I see the fewest compilation warnings using ubuntu 16.04, which is the version currently in the repo's Dockerfile. (Note that it is currently passed end-of-life and is no longer supported.) My PR #36 used 20.04, which does display more warnings.
This is not to say the warnings are a problem. Just an observation that you may get different behavior using different compilers, especially if there is some memory related problem.

Hmmmm interesting.
In nvidia/cuda:11.0-cudnn8-runtime-ubuntu16.04 image after installing the build-essential i have
root@822f34c25aaf:/# gcc --version
gcc (Ubuntu 5.4.0-6ubuntu1~16.04.12) 5.4.0 20160609

In nvidia/cuda:11.0-cudnn8-runtime-ubuntu18.04 image after installing the build-essential I have
root@e7ecd162bf99:/# gcc --version
gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0

And the last image seems to work with the hhblits for me.
@LuGarfield can you change base image from nvidia/cuda:${CUDA}-cudnn8-runtime-ubuntu16.04 to nvidia/cuda:${CUDA}-cudnn8-runtime-ubuntu18.04 and check the pipeline again?

When I try and build the docker file (trying my best to use consistent versions of software), I always see warnings in the compilation of hhblits. The warnings vary depending the base version of ubuntu used, because each version uses a different default compiler version.
FWIW, I see the fewest compilation warnings using ubuntu 16.04, which is the version currently in the repo's Dockerfile. (Note that it is currently passed end-of-life and is no longer supported.) My PR #36 used 20.04, which does display more warnings.
This is not to say the warnings are a problem. Just an observation that you may get different behavior using different compilers, especially if there is some memory related problem.

i found these  warnings  when i compile docker env.

When I try and build the docker file (trying my best to use consistent versions of software), I always see warnings in the compilation of hhblits. The warnings vary depending the base version of ubuntu used, because each version uses a different default compiler version.
FWIW, I see the fewest compilation warnings using ubuntu 16.04, which is the version currently in the repo's Dockerfile. (Note that it is currently passed end-of-life and is no longer supported.) My PR #36 used 20.04, which does display more warnings.
This is not to say the warnings are a problem. Just an observation that you may get different behavior using different compilers, especially if there is some memory related problem.

Hmmmm interesting.
In nvidia/cuda:11.0-cudnn8-runtime-ubuntu16.04 image after installing the build-essential i have
root@822f34c25aaf:/# gcc --version
gcc (Ubuntu 5.4.0-6ubuntu1~16.04.12) 5.4.0 20160609

In nvidia/cuda:11.0-cudnn8-runtime-ubuntu18.04 image after installing the build-essential I have
root@e7ecd162bf99:/# gcc --version
gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0

And the last image seems to work with the hhblits for me.
@LuGarfield can you change base image from nvidia/cuda:${CUDA}-cudnn8-runtime-ubuntu16.04 to nvidia/cuda:${CUDA}-cudnn8-runtime-ubuntu18.04 and check the pipeline again?

@akhoroshev
my settings :
ARG CUDA_FULL=11.2.2
FROM nvidia/cuda:${CUDA_FULL}-cudnn8-runtime-ubuntu20.04

still have the problem

I met the same error. And notably, this happened when I submitted protein sequence longer than 1k AA. Protein smaller than 1k AA  could be calculated normally. So I suspected if the memory space for HHblits is matters.

Does anyone mange to solve this problem? I got the same red errors as posted by @LuGarfield and also the same initial error. So far I was not able to fix it. My protein is far below 1k AA, but I have only 16GB RAM. Could this be a problem?

Did you have the "segmentation fault" ?
I have met this error (and I have segmentation fault" when I only use ONE cpu for MSA. I solved this problem by give the it more CPUs and memory and it worked. After that, some friends occasionally asked me about this question and I usually suggesting them to add more CPUs. Because our HPC cluster only give maximum 6 CPUs for 1 GPU, so I let them use more GPU cards to have more CPUs, which means more memory.

I don't get a segmentation fault. So far I'm running it on my own computer and I used my NAS to store all the databases. I have a Intel® Core™ i9-9980HK CPU @ 2.40GHz × 16, a NVIDIA GeForce GTX 1650/PCIe/SSE2 and 16 GB of RAM. Could this be insufficient to run AlphaFold?

Hello, it seems same question~when i run multimer it show me this,but it works well when i run monomer.
Here are the errors:

If you have any ideas?Thanks!

This may be caused by insufficient RAM – if you are using less than 32 GB RAM then upgrading your RAM to 32 GB will probably solve the issue.
Another complete workaround for this is to run with --db_preset=reduced_dbs as specified in the README. This uses a different tool (jackhmmer) for the genetic search. Closing this issue for now, feel free to reopen if this is still a problem.

This may be caused by insufficient RAM – if you are using less than 32 GB RAM then upgrading your RAM to 32 GB will probably solve the issue.
Another complete workaround for this is to run with --db_preset=reduced_dbs as specified in the README. This uses a different tool (jackhmmer) for the genetic search. Closing this issue for now, feel free to reopen if this is still a problem.

Ok,thanks for your help


fix product search e2e test

this test is currently failing. Did the search result change again?
regression/mobile/product-search-store-flow-mobile
regression/product-search/product-search-store-flow

Added the fix in branch 1765.


Temporarily switches to Inquirer.js until enquirer fixes their output

This is temporary. Once upstream fixes their problem (including Gluegun updating to the new version) we'll want to revert this commit.
Fixes #634.

It may not be long. We can keep this PR in a holding pattern until close to launch and see if this issue is resolved first.
https://github.com/enquirer/prompt-list/issues/2

we can close this.  they released 0.1.2
https://github.com/infinitered/gluegun/pull/144

Yay @GantMan !


add deb security manifest

As specified in the OCI Factory's image maintainer agreement, ROCKs that include additional deb packages (in our case, ca-certificates) must render a security manifest; the purpose is to allow the security team to notify us in case of security issues with those packages.

Related: https://github.com/canonical/rockcraft/issues/438


Filters on include_context are ignored

Subject of the issue
Filters on include_context are ignored if passed in the wrong order
Your environment

Ruby version: 3.1.1
rspec-core version: 3.11.0

Steps to reproduce
# frozen_string_literal: true

begin
  require "bundler/inline"
rescue LoadError => e
  $stderr.puts "Bundler version 1.10 or later is required. Please update your Bundler"
  raise e
end

gemfile(true) do
  source "https://rubygems.org"

  gem "rspec", "3.11.0" # Activate the gem and version you are reporting the issue against.
end

puts "Ruby version is: #{RUBY_VERSION}"
require 'rspec/autorun'

RSpec.shared_context 'default context' do
  let(:value) { 1 }
end

RSpec.shared_context 'my context' do
  let(:value) { 2 }
end

RSpec.configure do |config|
  config.include_context 'default context', type: :bug_report
  config.include_context 'my context', :filter_me, type: :bug_report
end

RSpec.describe 'include filters', type: :bug_report do
  it 'should have value == 1' do
    expect(value).to eq 1
  end

  it 'should have value == 2', :filter_me do
    expect(value).to eq 2
  end
end

Expected behavior
The context should only be included on examples matching the filters
Actual behavior
The context is always included
Workaround
Inverting the filters order fixes the issue:
# frozen_string_literal: true

begin
  require "bundler/inline"
rescue LoadError => e
  $stderr.puts "Bundler version 1.10 or later is required. Please update your Bundler"
  raise e
end

gemfile(true) do
  source "https://rubygems.org"

  gem "rspec", "3.11.0" # Activate the gem and version you are reporting the issue against.
end

puts "Ruby version is: #{RUBY_VERSION}"
require 'rspec/autorun'

RSpec.shared_context 'default context' do
  let(:value) { 1 }
end

RSpec.shared_context 'my context' do
  let(:value) { 2 }
end

RSpec.configure do |config|
  config.include_context 'default context', type: :bug_report
  config.include_context 'my context', { type: :bug_report }, :filter_me
end

RSpec.describe 'include filters', type: :bug_report do
  it 'should have value == 1' do
    expect(value).to eq 1
  end

  it 'should have value == 2', :filter_me do
    expect(value).to eq 2
  end
end

I tested this on 4-0-dev and it was fixed (using the snippet above) closing for now but please re open if you find an issue against 4-0-dev


ci: Speed up MacOS CI

CI Vulkan-Tools build queued with queue ID 11701.

CI Vulkan-Tools build queued with queue ID 11701.

CI Vulkan-Tools build # 994 running.

CI Vulkan-Tools build # 994 passed.


Use all processors when installing zstd

Description
Speeds up my builds from 93s to 37s (four core) for the install-zstd.sh step
Related issue:
None
Check list

[n/a] Related issue / work item is attached
[n/a] Tests are written (if applicable)
[n/a] Documentation is updated (if applicable)
[x] Changes are tested and related VM images are successfully generated

One question i have yet to ask: Is this change stable in CI aswell just like using zstd in single-threaded mode?

@Kichura Can you approve it so the workflows can run and we can see if it's stable on CI?

@Kichura Can you approve it so the workflows can run and we can see if it's stable on CI?

I am not a member of this repo, therefore i can't greenlight this.


Initial work on UntypedObject coercions for binary operators

Enables coercions from UntypedObject to other types in binary and unary operators. Untyped/Blank combinations often coerce to Number. Includes tests for Blank() for many operators.

✅ No public API change.

What exactly are the coercion rules?
Are the any ambiguities, especially since + can be both number and date, how do we statically resolve?
ParseJson() + 5  // is this NumberAdd, or DateAdd?

Single type operator coerce support is limited to Number, Boolean, Text types. So I assume that this would be NumberAdd. DateAdd would require explicit cast.

In reply to:<PHONE_NUMBER>

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.

✅ No public API change.


How do you filter for message ID?

Hello,
we are building up a web project based on laravel-imap and tried to filter for the messageID which should be unique along a user's mailbox so that a user can select some messages and request to delete them.
I've tried several ways but can't figure out how to filter for the messageId in the Query::class.
My code so far:
[...]

        $messageID =<EMAIL_ADDRESS>
        $oClient = new Client([
            'username'      => $current_mailaddress->username<EMAIL_ADDRESS>            'password'      => $current_mailaddress->securetoken
        ]);
        $oClient->connect();

        $folders = $oClient->getFolders();

        $mails = [];

        foreach ($folders as $folder) {
                //$mails[] = $folder->query()->whereMessageId($messageID)->get(); --> this throws an InvalidWhereQueryCriteriaException
                $mails[] = $folder->query("Message-ID <". $messageID . ">")->get();
        }

The current version shows all available messages without filtering them for the messageId.
I've looked this up @ https://github.com/Webklex/laravel-imap/blob/master/src/IMAP/Query/WhereQuery.php
There's a public function whereMessageId($msg_id) which does not work.
But public function whereUnseen() works fine - why?
How can I filter all messages for a given message id without gathering them all and filter them with my own logic?
Thanks in advance & Kind Regards

Hi @buenyamin-olgun ,
the available search criteria depends on your mail provider. Owever if you take alook at your raw mail is there a line containing the string Message-ID <fooBar>? If it isn't present but lets say "MESSAGE-ID" you could use something like ->where("MESSAGE-ID <fooBar>") and use a custom search parameter.
I hope this helps a little bit :)

Hi @Webklex,
thank you for your answer.
There's the following line in every mail on my mail server:
Message-ID<EMAIL_ADDRESS>
But if I try to filter for this with $folder->query("Message-ID: <". $messageID . ">")->get();
there are 2 messages even though the message ID is completely different.
Could you help me with this?
Kind Regards

Oh, well let me do some quick tests :)

Alright, I'm out of luck. My mailserver doesn't understand the Message-ID command.

$oFolder->messages()->text('Message-ID: CA+=9s-NmW8JnZ7UWSY9y15EV_Chob6WLAJsbN8g-Pg3FFwQR=Q@mail.gmail.com')->get()
try this
for select mail when contain text [ Message-ID:  ]

Or the WhereQuery whereMessageId :
 $folder->query()->whereMessageId<EMAIL_ADDRESS>)->get();

Hi Webklex !
Ok ! Thx for  that clarification 😃


  (See pictures).Sometimes they even drive out of the road.
So I tried to use Vehicle Restriction provided by TM: PE, but it works on vacuum trucks.(See pictures)
The tunnel in picture doesn't look good because I made it just for test. It still happen in smooth tunnel.
Steps to reproduce

see above
Log files

https://ufile.io/7zg9812x
no error message occured
Savegame?

https://ufile.io/whkfs29c
My Mods collection：https://steamcommunity.com/sharedfiles/filedetails/?id=1697022785
Screenshots?

https://ufile.io/jjl7adio
https://ufile.io/whkfs29c
Notes or questions?

@Viklamvia You could very easily just drag and drop the pictures in the issue.
I can see you have prohibited city services. what if you also ban trucks as well?

I am not sure which color is which but you can tell from the colors what type of vehicle is a vacuum truck.

The first car is vacuum truck. It was assigned into service vehicle by color so I just ban city service. And this road is built in industry area which is designed for trucks so I didnt ban truck but I will try.

It's possibly a bug in TM:PE; vacuum trucks (WaterTruckAI) should certainly be treated as city services, we'll have to check.
Note also, however, that vehicle restrictions aren't outright bansl; they are pathfinding penalties. The pathfinder will try to find "cheapest route" (ie. route with least penalties). The "Vehicle Restriction Agression" setting determines how harsh the penalties are.

TM:PE doesn't interfere with vacuum truck process or selecting location where they should go. They are also correctly treated as city service. If issue persist, let us know.


Install fails, requirements could not be resolved

The installation as described in the Readme fails:
$ composer create-project pattern-lab/edition-development-twig edition-development-twig                                                                                Mo  2 Mär 2015 09:40:15 CET
Installing pattern-lab/edition-development-twig (v0.1.0)
  - Installing pattern-lab/edition-development-twig (v0.1.0)
    Loading from cache

Created project in edition-development-twig
Loading composer repositories with package information
Installing dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - pattern-lab/styleguidekit-assets-default dev-dev requires pattern-lab/core ~0.5 -> no matching package found.
    - pattern-lab/styleguidekit-twig-default dev-dev requires pattern-lab/styleguidekit-assets-default dev-dev -> satisfiable by pattern-lab/styleguidekit-assets-default[dev-dev].
    - Installation request for pattern-lab/styleguidekit-twig-default dev-dev -> satisfiable by pattern-lab/styleguidekit-twig-default[dev-dev].

Potential causes:
 - A typo in the package name
 - The package is not available in a stable-enough version according to your minimum-stability setting
   see <https://groups.google.com/d/topic/composer-dev/_g3ASeIFlrc/discussion> for more details.

Read <http://getcomposer.org/doc/articles/troubleshooting.md> for further common problems.

@vitaminless -
Sorry about that. I was trying to set-up a true beta version of the Twig work but messed up the dependencies. What's odd is that pattern-lab/core definitely has v0.5+ tags. I'm going to play with this a little bit this morning. Hopefully I can resolve it quickly.

@vitaminless -
Had this working again this morning but have run into another issue with the latest edition of Composer. You can try to install but let me know if it blows up for you too.

@vitaminless -
You should be able to install now. You should run the following before attempting to install the project though:
composer self-update

Please note that the Development Edition should be considered unstable. If you use it you will essentially always be running edge when you update your project components. The developer preview for the stable Standard Edition for Twig should be out today or tomorrow.

Sorry for late response, true it works now


{AKS} Fix #27461 az aks create: Update help information for option --node-osdisk-size

This checklist is used to make sure that common guidelines for a pull request are followed.
Update help information for option --node-osdisk-size. The disk size specified by this option is actually in GiB (instead of GB). Fix issue #27461
Related command

az aks create
az aks nodepool add

General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

AKS


Footer missing in the website.

There is footer missing in the website. The design I wish to add is as shown-

Your team can also share a better design for the footer if needed. @ishitaraina1807  Kindly assign me this issue, I will work closely with you and create a personalized footer for the website, using React.js and Tailwind CSS.

I have assigned. Thank you.

can you assign me plz?


Generate interpolation videos from trained models

I was wondering whether there's a possibility to share a script to generate videos from the trained model as shown on your project page. The .pt format is not easily usable for exporting videos from the original NVidia implementation. Any suggestion on how to implement it would be of much help.

Sure. We don't have it unified as a single script at this point, so I'll have to create one - but I don't think it will be an issue.
I'll add it to the to-do list and hopefully get around to it during the weekend.

Hey! Where will we be able to find the 'align_faces_parallel' script?
I am talking with respect to this line  from scripts.align_faces_parallel import align_face in the run_alignment function present in the google colab.

That script (and most of the inversion code) is part of ReStyle.

@samuelpietri I added a video generation script. It still requires you to supply your own initial and edited latent codes (or use the ones I uploaded as an example). Let me know if you need help with that.

It has been more than a month since the video generation script was uploaded so I'm going to go ahead and close this. Feel free to re-open if you still need something.


Refactor store_message function and add Simple Git Hooks

This pull request adds Simple Git Hooks for better code quality and consistency. It also updates the pre-commit-config.yaml and Makefile, and refactors the pre-commit installation in the Makefile. Additionally, the update_poetry.sh script is refactored to not install pipx anymore.

LGTM


BIG: Bad query url generated for equality compare

The following Metadata for 2 objects (Account having a User)

Simple, made for example.
The following query:
        var item = await c.For<OdataApi.ZZZAccount>()
            .Filter(x => x.User.Identity == identity)
            .FindEntryAsync();

should create a $filterr following the navigation property to User and comparing the IDentity there ;)
What happens is this:
http://localhost/ZZZAccount(12bbdd24-3833-4f93-9e7a-009382403841)
Seems it gets confused and tries direct access (.Key). Totally not what is wanted.
If I rename Identity on User into UserIdentity, it works and generates the Url in the form
http://localhost/ZZZAccount?$filter=User%2FUserIdentity eq 12bbdd24-3833-4f93-9e7a-009382403841
which is perfect. Seems to get confused - but we really standardize property names ;)

Is Identity a key for User? If so, AFAIK this is an expected behavior because $filter clause is supposed to be replaced with a key lookup (value in parentheses).

Ah, but you miss the point here. Compare the URL's - it takes the Identity of User and then chekcs Account. It literlaly checks the Identity ON THE WRONG SET.
http://localhost/ZZZAccount(12bbdd24-3833-4f93-9e7a-009382403841)
makes the key lookup on ZZZAccount.
But the filter is not on Account, it is on User/Identiy- With User being a navigation property.
So, the replacement mixes up object sets - which is why this is so bad.

Ah I see. Then it's really strange.

Can you send me a scheme for these?

Well, I don't ahve a separate test project and the metadata is long, but here are the parts i actually wanted to paste in anyway ;)

Thanks, I guess this should be sufficient.

I have a lot of them. We have standardized naming. The classes you get and see have no table attribute etc - they basically ONLY have a Key ever. They are DTO, the underlying is not exposed. One reason is to keep the assembly defining the api objects slim (and you pull in the whole aspnetcore webapi if you use any odata specific attributes.
We use Identity as property name on every object having an Identity (which is a "random" Guid assigned at creation).
THAT SAID: as this is a pure eror on the simple.odata.client side (i.e. based on the EdmModel) the backend implementation how they get the data should not matter. I t generates the wrong link based on the metadata. As long as the key in Account and User define Identity, it makes the wrong transform.  Simple odata client simply seems to et confused with identical names.
This test code:
    [TestMethod]
    public async Task Get2()
    {
        var c = new ODataClient(TestSetup.ClientSettings);
        var identity = await c
            .For<OdataApi.ZZZUser>()
            .Select(x => x.Identity)
            .Top(1)
            .FindScalarAsync<Guid>();

        var item = await c.For<OdataApi.ZZZAccount>()
            .Expand(x => x.User)
            .Filter(x => x.User.Identity == identity)
            .FindEntryAsync();
    }

with this metadata:

results in:
http://localhost/ZZZUser?$top=1&$select=Identity"
followed by
http://localhost/ZZZAccount(00fea0e2-9e93-4b56-881c-09f87522353a)?$expand=User
As you can see the identity is in the test code pulled from a user, so it should not be used in account.
Here are the model classes:
public class ZZZAccount
{

    [Key]
    public Guid Identity { get; set; }

    public int Id { get; set; }

    public string Name { get; set; }

    public ZZZUser User { get; set; }

}

public class ZZZUser
{

    [Key]
    public Guid Identity { get; set; }

    public int Id { get; set; }

}

As simple as it gets.

Thank you. This should be sufficient now.


Laravel 5.6

Laravel Version: 5.6.22
PHP Version: 7.2.4
Laravel-admin: ^1.5
Description:
Not fully functioning with laravel 5.6

I cant get the form fields to display, would this be related? It shows the List view but on edit or create there are no form fields

It should be ok now


[WIP] Refactor infrastructure layer

WHY
WHAT
REF

Coverage increased (+5.9%) to 68.663% when pulling a0adc4bf1d808f4cf5945bafb5d592546682b9a6 on izumin5210/refactor-fs into 54f2f5da43e67210bca4a010ca503d46c32874b7 on master.

Coverage increased (+4.8%) to 67.578% when pulling af73e784970e7f09c0cc5f3879750eacb23754a8 on izumin5210/refactor-fs into 54f2f5da43e67210bca4a010ca503d46c32874b7 on master.

Coverage increased (+9.0%) to 71.79% when pulling f5dbfa884cc7ef66be5b62814b6d658b09ee448a on izumin5210/refactor-fs into 54f2f5da43e67210bca4a010ca503d46c32874b7 on master.

Coverage increased (+8.04%) to 70.851% when pulling 0f3b19c92768805f697b625317af345f60b02479 on izumin5210/refactor-fs into 54f2f5da43e67210bca4a010ca503d46c32874b7 on master.

Coverage increased (+10.7%) to 73.474% when pulling 7fbb89e1db592033efbf1e1784b4cfb2288f68d2 on izumin5210/refactor-fs into 54f2f5da43e67210bca4a010ca503d46c32874b7 on master.


Add install and import guide for Python SDK

Added a short install and import tutorial for the
Python SDK. Each SDK should have instructions
on how to get the

Note to reviewers: this PR contains commits from a non-merged PR, (#403), since it was branched off that PR. Not sure if there is a way to avoid having this extra commits in here.

@aludvik you are right about the complication -it's branched off another branch I have ready to merge with another PR.

@aludvik By mistake, the most important file for this PR was not added last Friday: python_sdk_install.rst
I'll have to ask you to re-review.

This PR is being replaced by PR #431

Closing in favor of #431


Does it support transactions in asynchronous?

It was a mistake, I’m sorry, I close the issue

Hey @00Sergio00, don't worry.
Though maybe you could let the problem description here and give the solution to it yourself, in case someone else experience the same problem you did and find this link.


Error checking and return to caller

The translators generate a file with no indication of problems it has encountered. While error checking of the translators may be limited, there is still an opportunity to return ❌ errors and ⚠ warnings through a return value or callback. This would be helpful for a tool using this module to report errors to the user.

Refactor the .write() method to return an array of warnings and errors, and the buffer. A separate file will check for warnings and errors and handle them accordingly. The buffer will be written to a logical path. This will also help reduce the code repetition that Code Climate is reporting.

Get rid of the .write() method and make the translator simply a function that returns an object:
{
  warnings: [],
  errors: [],
  result: ...
}

This way the file system code is logically separated from the translation. Plus, translation may be desired and writing its result to a file may not. Translation and file system should really be decoupled.


Add support for Phi-3-vision series model

Add support for Phi-3-vision models (microsoft/Phi-3-vision-128k-instruct and microsoft/Phi-3.5-vision-instruct)

Hi @Isotr0py, thanks for creating this PR. Do you have an AWQ model of Phi-3 Vision that I can test?

I have uploaded the model to hf: Isotr0py/Phi-3.5-vision-instruct-AWQ.
Example code
from PIL import Image
import requests
from awq import AutoAWQForCausalLM
from transformers import AutoProcessor

model_id = "Isotr0py/Phi-3.5-vision-instruct-AWQ"

# Note: set _attn_implementation='eager' if you don't have flash_attn installed
model = AutoAWQForCausalLM.from_pretrained(
    model_id,
    device_map="cuda",
    trust_remote_code=True,
    torch_dtype="auto",
    _attn_implementation='eager'
)

# for best performance, use num_crops=4 for multi-frame, num_crops=16 for single-frame.
processor = AutoProcessor.from_pretrained(model_id,
  trust_remote_code=True,
  num_crops=4
)

images = []
placeholder = ""

# Note: if OOM, you might consider reduce number of frames in this example.
for i in range(1,2):
    url = f"https://image.slidesharecdn.com/azureintroduction-191206101932/75/Introduction-to-Microsoft-Azure-Cloud-{i}-2048.jpg"
    images.append(Image.open(requests.get(url, stream=True).raw))
    placeholder += f"<|image_{i}|>\n"

messages = [
    {"role": "user", "content": placeholder+"Summarize the deck of slides."},
]

prompt = processor.tokenizer.apply_chat_template(
  messages,
  tokenize=False,
  add_generation_prompt=True
)

inputs = processor(prompt, images, return_tensors="pt").to("cuda:0")

generation_args = {
    "max_new_tokens": 1000,
    "temperature": 0.0,
    "do_sample": False,
}

generate_ids = model.generate(**inputs,
  eos_token_id=processor.tokenizer.eos_token_id,
  **generation_args
)

# remove input tokens
generate_ids = generate_ids[:, inputs['input_ids'].shape[1]:]
response = processor.batch_decode(generate_ids,
  skip_special_tokens=True,
  clean_up_tokenization_spaces=False)[0]

print(response)

Outputs
To summarize, the presentation is composed of these sections:

- Presentation Overview:
The presentation, authored by Dinesh Kumar Wickramasinghe, a Senior Software Engineer from CMS Private Limited in Sri Lanka, focuses on Microsoft Azure. The content is divided into three sections: Hyper-scale, Enterprise, and Hybrid, each represented by a hexagon. The design is clean and modern, with a blue color scheme and a geometric pattern.

Steps I had to take to reproduce working generation:

Quantize the model. This can be done with quite a normal script.

from awq import AutoAWQForCausalLM
from transformers import AutoTokenizer

model_path = 'microsoft/Phi-3.5-vision-instruct'
quant_path = 'phi3.5-vision-instruct'
quant_config = { "zero_point": True, "q_group_size": 128, "w_bit": 4, "version": "GEMM" }

# Load model
model = AutoAWQForCausalLM.from_pretrained(
    model_path, **{"low_cpu_mem_usage": True, "use_cache": False}
)
tokenizer = AutoTokenizer.from_pretrained(model_path, trust_remote_code=True)

# Quantize
model.quantize(tokenizer, quant_config=quant_config)

# Save quantized model
model.save_quantized(quant_path)
tokenizer.save_pretrained(quant_path)

print(f'Model is quantized and saved at "{quant_path}"')

After quantization is done, I noticed that 3 files are mismsing. So you need to download modeling_phi3_v.py, preprocessor_config.json, processing_phi3_v.py

Inference works too, but I had to disable the fused layers which do not work with the custom modeling code (even with _attn_implementation='eager').

Error:
Traceback (most recent call last):
  File "/workspace/AutoAWQ/examples/generate2.py", line 34, in <module>
    generation_output = model.generate(
                        ^^^^^^^^^^^^^^^
  File "/workspace/AutoAWQ/awq/models/base.py", line 130, in generate
    return self.model.generate(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/transformers/generation/utils.py", line 2215, in generate
    result = self._sample(
             ^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/transformers/generation/utils.py", line 3206, in _sample
    outputs = self(**model_inputs, return_dict=True)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/torch/nn/modules/module.py", line 1562, in _call_impl
    return forward_call(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/root/.cache/huggingface/modules/transformers_modules/phi3.5-vision-instruct/modeling_phi3_v.py", line 1603, in forward
    outputs = self.model(
              ^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/torch/nn/modules/module.py", line 1562, in _call_impl
    return forward_call(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/workspace/AutoAWQ/awq/modules/fused/model.py", line 353, in forward
    mask = fused_utils.prepare_attention_mask(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/workspace/AutoAWQ/awq/utils/fused_utils.py", line 58, in prepare_attention_mask
    mask = torch.triu(mask, diagonal=start_pos + 1).type_as(type_as)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RuntimeError: CUDA error: device-side assert triggered
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1
Compile with `TORCH_USE_CUDA_DSA` to enable device-side assertions.

Inference code that works:
import torch
import requests
from PIL import Image
from awq import AutoAWQForCausalLM
from transformers import AutoProcessor, TextStreamer

# Load model
quant_path = "phi3.5-vision-instruct"
model = AutoAWQForCausalLM.from_quantized(quant_path, fuse_layers=False)
processor = AutoProcessor.from_pretrained(quant_path)
streamer = TextStreamer(processor, skip_prompt=True)

images = []
placeholder = ""

# Note: if OOM, you might consider reduce number of frames in this example.
for i in range(1,2):
    url = f"https://image.slidesharecdn.com/azureintroduction-191206101932/75/Introduction-to-Microsoft-Azure-Cloud-{i}-2048.jpg"
    images.append(Image.open(requests.get(url, stream=True).raw))
    placeholder += f"<|image_{i}|>\n"

messages = [
    {"role": "user", "content": placeholder+"Summarize the deck of slides."},
]

prompt = processor.tokenizer.apply_chat_template(
  messages,
  tokenize=False,
  add_generation_prompt=True
)

inputs = processor(prompt, images, return_tensors="pt").to("cuda")

generation_output = model.generate(
    **inputs,
    max_new_tokens=512,
    streamer=streamer
)

I removed the fuser for now so we can merge the PR. For optimized inference, it's probably best to use vLLM or other engines.


Machine class

What this PR does / why we need it: Adds a MachineClass type to the Machines API
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
xref #529
Special notes for your reviewer: I had to delete some generated code to make this compile. See https://github.com/kubernetes-incubator/apiserver-builder/issues/234
Release note:

NONE

@kubernetes/kube-deploy-reviewers

$ kubectl get machineclasses
NAME      AGE
medium    17s
small     12s

We may not want to merge this until the apiserver-builder issue is resolved because it will incur overhead every time someone re-runs the apiserver-boot with --generator apiregister set (at minimum, deleting 10 lines of code but much more likely confusion as to why the code stopped building).

FYI, this is on hold until Robby is back later this month.

/hold
We have migrated the generic parts of the Cluster API to to sigs.k8s.io/cluster-api. This PR modifies some of those files. It will have to be migrated to the new repo https://github.com/kubernetes-sigs/cluster-api.

I need to re-open this against the cluster-api repo. Closing this PR.