cmd/go: go mod init writes info message to stderr

What version of Go are you using (go version)?
go version devel +81555cb4f3 Tue Aug 14 09:19:38 2018 +0000 darwin/amd64
Does this issue reproduce with the latest release?
Yes, latest master.
What did you do?
cd `mktemp -d`
go mod init example.com/foo > /dev/null

What did you expect to see?
I expect to see no output.
What did you see instead?
$ go mod init example.com/foo > /dev/null
go: creating new go.mod: module example.com/foo

go mod init writes this message to Stderr.
I would prefer if it wrote to stdout instead, so I could ignore the output in my script.
And I definitely don't want to ignore the stderr - in case of any error occurs.

Using standard error for non-error messages is fine. For example, the log package uses standard error by default.
If you want to ignore output as long as no error happened, simply keep the output somewhere, and only use it if the exit code is non-zero.

Thanks, that makes sense.
I just thought it's not an intended behaviour.
Closing.


doc: update README to align with the new project brand

Resolve #150

Codecov Report

Merging #158 (766c454) into main (0c1ec3b) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #158   +/-   ##
=======================================
  Coverage   89.13%   89.13%
=======================================
  Files          21       21
  Lines        1693     1693
=======================================
  Hits         1509     1509
  Misses        149      149
  Partials       35       35

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@priteshbandi @patrickzheng200 This is PR is ready to be merged. Would you mind doing it? Thanks


../dialogs/IntentRecognizerSet not found due to case issue?

I was trying to deploy a bot with botbuilder 3.6.0 to Heroku, however, build fails because IntentRecognizerSet is not found. I am assuming its because of the case sensitivity on Linux (which is what Heroku uses). Would someone happen to know a fix?

Hi @achadha0111, could you please send us the full error text you are getting? Thanks!

 Error: Cannot find module '../dialogs/IntentRecognizerSet' 2017-02-06T17:33:44.139141+00:00 app[web.1]:     at Function.Module._resolveFilename (module.js:469:15) 2017-02-06T17:33:44.139143+00:00 app[web.1]:     at Function.Module._load (module.js:417:25) 2017-02-06T17:33:44.139144+00:00 app[web.1]:     at Module.require (module.js:497:17) 2017-02-06T17:33:44.139144+00:00 app[web.1]:     at require (internal/module.js:20:19) 2017-02-06T17:33:44.139146+00:00 app[web.1]:     at Object.<anonymous> (/app/node_modules/botbuilder/lib/bots/Library.js:9:29) 2017-02-06T17:33:44.139146+00:00 app[web.1]:     at Module._compile (module.js:570:32) 2017-02-06T17:33:44.139147+00:00 app[web.1]:     at Object.Module._extensions..js (module.js:579:10) 2017-02-06T17:33:44.139148+00:00 app[web.1]:     at Module.load (module.js:487:32) 2017-02-06T17:33:44.139148+00:00 app[web.1]:     at tryModuleLoad (module.js:446:12) 2017-02-06T17:33:44.139149+00:00 app[web.1]:     at Function.Module._load (module.js:438:3)

@carlosscastro , any updates?

I agree that it sounds like a casing problem but I don't see a casing problem.  Are you able to figure out the exact place where we use the wrong case?

I am supposing, from the error message, it was the naming of the file IntentRecognizerSet.js which was preventing the file from being found when Heroku tried to build it. However, it seems to be working now, so I think we can close this issue. Thank you for looking into it though! Cheers.


[BUG] Can't delete macros because ui's overlap

Describe the bug
When in the GSE UI, I pick a macro, hit edit, hit delete
I can't confirm to delete because the editor window is on top of the confirm delete window
To Reproduce
Steps to reproduce the behavior:

The GSE UI
Click on a Macro
Go to Editor
Attempt to delete
See Error

Expected behavior
The confirm delete window being on top.

I can’t make this occur.  I need more information on How to replicate this.
Are you running any UI mods?  what version of GSE are you running?  What OS? What game version?  Does this occur when you disable all your other mods?  How big is the editor set to (you can see this in GSE’s options)?

Was finally able to replicate this in 3.0 -- As its blizzards code it makes it an interesting problem,


feat(cli): CLI containerization

This PR

adds this new feature

Related Issues

Fixes #7586

@MdSahil-oss if you say the pipeline work is too much for you, we can also create a follow-up ticket for that and I can take it over :)

@mowies I will finish the pipeline work very soon myself :)

@mowies can you have a look on the changes that I made related to codepipeline.

@MdSahil-oss thanks for the updates! I will do some final changes and debugging and push them directly to your PR when I'm done and then we can merge this.

@MdSahil-oss i found an issue that prevent us from building the CLI in docker. So this PR is blocked by https://github.com/keptn/keptn/issues/9462


Minor clean-up follow-ups for #86

Description of what I changed

This should make no functional changes (although regeneration of comboPooledDataSource in the old version of JdbcConnectionUtil.getConnectionObject in #86 might have had some performance implications and potentially some race condition issues which are resolved now).
E2E test

TESTED:
Didn't do any manual tests as I do not expect functional changes.
Checklist: I completed these to help reviewers :)

[x] My IDE is configured to follow the code style of this project.
No? Unsure? -> configure your IDE, format the code and add the changes with git add . && git commit --amend

[x] I am familiar with Google Style Guides for the language I have coded in.
No? Please take some time and review Java and Python style guides. Note, when in conflict, OpenMRS style guide overrules.

[ ] I have added tests to cover my changes. (If you refactored existing code that was well tested you do not have to add tests)
No? -> write tests and add them to this commit git add . && git commit --amend

[x] I ran mvn clean package right before creating this pull request and added all formatting changes to my commit.

[x] All new and existing tests passed.
No? -> figure out why and add the fix to your commit. It is your responsibility to make sure your code works.

[x] My pull request is based on the latest changes of the master branch.
No? Unsure? -> execute command git pull --rebase upstream master

hello @bashir2 this looks good to me ,,,
may be pending fixing the merge conflicts

Merging as the last Travis run was successful.


MultiPartFile in Model doesnot work

My application is simple spring-mvc (no spring boot)
I am using following version of swagger
        <dependency>
            <groupId>io.springfox</groupId>
            <artifactId>springfox-swagger2</artifactId>
            <version>2.5.0</version>
        </dependency>

        <dependency>
            <groupId>io.springfox</groupId>
            <artifactId>springfox-swagger-ui</artifactId>
            <version>2.5.0</version>
        </dependency>

I am having an REST endpoint that accepts Model(definition as below)
public @ResponseBody UploadResponse processSubmit(
       @ModelAttribute("uploadModel") UploadModel uploadModel )

However the UploadModel has one attribute as MultiPartFile type
@ApiModel
public class UploadModel {
     private MultipartFile inputFile = null;
      public MultipartFile getInputFile() {
        return inputFile;
    }

    public void setInputFile(MultipartFile inputFile) {
        this.inputFile = inputFile;
    }
}

I can see that this parameter is ignored and not listed in json as well as the Swagger UI.
Could you please tell me how can we fix this?

This may be a subtle usage of the multipart file, but if you change it to this signature it should work.
public @ResponseBody UploadResponse processSubmit(MultipartFile uploadModel )

Actually the model has multiple other attributes too.So I cannot change the definition.

Would you perhaps like to try and add support for this via a PR? I can help answer questions if any help is required. However this is not a priority feature to support.

I will go through the code and see if I can fix this.

Hey, what is stage of this feature request?

I'm also encountering something like this. I have a DTO/Model object that contains a Multipart file. The parameters show up in Swagger UI, including the MultipartFile, although when I select a file and submit the request my API just receives null for the file.

I also have the same issue. I can see all the fields, but the order and the required param is ignored. Version 3.0.0

I've encountered this problem, I thought it was caused by Spring Boot, but after trying to upload files using Postman they went through flawlessly. Also it's visible that swagger treats a RequestParam MultipartFile different than a MultipartFile inside a ModelAttribute complex object!


creating a middleware example for validating cookie

Just as there is a validate_jwt_token method which can be used in a middleware to protect routes unless there is a jwt token.
It would be useful if there is a validate_cookie method which could be also used in middleware to protect routes unless there is a valid session cookie.
not sure if this line is related https://github.com/authorizerdev/authorizer-go/blob/34c6220aeb6c36175b55645f744ce672080c4d85/get_session.go#L13
but I couldn't find anyway to do it currently.

I think the new API would make more sense because I am trying to validate and not issue a new one every time

Okay, I will try to add that by tomorrow

I have created github issue for same: https://github.com/authorizerdev/authorizer/issues/367

thanks

@c-nv-s added validate_session https://github.com/authorizerdev/authorizer-go/blob/main/validate_session.go

sorry to comment on a closed ticket but this feature could really do with this addition https://github.com/authorizerdev/authorizer/issues/379

also worth noting that the user must url.PathUnescape(theCookieValue) before passing it here https://github.com/authorizerdev/authorizer-go/blob/93f295f42bfbb0eaed8dbc12cdfdf7e4655021da/examples/validate_session.go#L23
otherwise it will return an error


Four Swords Net?

Would it be possible to program something like this to play four swords online?

The gba version? It's possible but would need a lot of code changes to the rom. Communication between the gamecube and gba works differently to communication between two gbas so that need updating. But also you to change what the game requires / what is being sent cope with the higher latency.
For example rather than receiving constant movement actions from a player you might need to just send several coordinates and have the game simulate movement between those points. (Not sure if this is how four swords works, just an example)
Unfortunately the decompilation project for four swords looks like it needs a lot of work before before those kind of changes can be easily done.


Milestones, epics

This repo needs some example epics and milestones, for illustration.

OK, made one epic (vandalism recovery) and a couple of milestones. Good enough for the CI tutorial in Oxford.


Add logger for the server APIs

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7909033948

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7909033948
Status: failure


Namespace tag should not be present in yaml if no namespace is selected while creating constraint.

What kind of request is this (question/bug/enhancement/feature request):
Bug
scenario 1
Steps to reproduce (least amount of steps as possible):

Create a cluster, navigate to dashboard, enable OPA gatekeeper.
Create a constraint in form view without namespace.

Result:
The constraint created has namespace:[]  because of which constraint behavior gets impacted.
One of the issue in upstream https://github.com/open-policy-agent/gatekeeper/issues/508
Expectation
In case of empty namespace, yaml should not have namespace tag.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-2463-head- e651c7a81
Installation option (single install/HA): Single

Verified in latest dashboard UI.
If no namespace is selected in constraint, no namespace tag is present in yaml.


Message chunk support

[x] chunk method (no syllables support)
[ ] feature flag
[ ] implement chunking to sendMessage method
[ ] tests

How can I handle reply_markup parameter when chunking?
Do I add it to the first message or the last message?

We need to find a way to not split words.

like this example  can we use wordwrap method for this issue

Ok, I can use the 4th parameter of wordwrap method.

But there's another problem... it doesn't support multibyte strings


package/esb: initial esbuild package rewrite

This PR adds the esb (short for esbuild) package to Bud. The purpose of this package is to provide common presets that viewers can use and extend in #383.
I'm also taking this opportunity to try and move esbuild fully to a virtual filesystem, so it's much easier to test different combos.

Note: Going to merge now and fix issues as they come up while working on the viewer.


[Bug]: Dice images not shown

Give a brief about the bug ✍
Description:
In the pig game the image of the dices not show when we role the dice ad also the thumbnail of the game is also not shown.

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @shaklan2001 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @shaklan2001 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


About the threshold of the anomaly

Thanks the great work!
Could you tell me the threshold of the recon_error, which used to distinguish between normal and abnormal.

Hi! I've been trying to replicate the MNIST experiment in the paper. I'm interest in this threshold too. In my experiments the reconstruction error doesn't see out of distribution for anomaly samples.


Fixed duplicate alt tags

Category

[x] Content fix
[ ] New article

Fixed duplicate alt tags

Docs Build status updates of commit b4d5a73:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Community/power-automate-vs-logic-apps.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add support for html in tooptip

This can be done by adding data-html="true" to the tooltip  element

This does not appear to work.  I tried adding this to the <a> in tmpls/month-day.html, but raw HTML was displayed.


Laravel 9 support

Description
Any plans to support Laravel 9?

As Laravel 8 introduces a way to enforce morph mapping, being explicit is favored over "magic code", and I don't have the time to maintain it, I'm probably going to archive this package. That means no more updates unfortunately. You're free to fork it though.


Failure to create DATA_asset in c2d-flow.md on Rinkeby

Describe the bug
I was running the c2d-flow.md on Rinkeby and ocean.assets.create() returned None
# Publish asset with compute service on-chain.
DATA_asset = ocean.assets.create(
    metadata=DATA_metadata,
    publisher_wallet=alice_wallet,
    encrypted_files=DATA_encrypted_files,
    services=[DATA_compute_service],
    erc721_address=erc721_nft.address,
    deployed_erc20_tokens=[DATA_datatoken],
)

To Reproduce
Steps to reproduce the behavior:

Setup environment to connect to Rinkeby

mkdir test
cd test/
python -m venv venv
source venv/bin/activate
pip3 install wheel
pip3 install --no-cache-dir ../
export OCEAN_NETWORK_URL=https://rinkeby.infura.io/v3/<infura_project_id>
export ADDRESS_FILE=/home/david/projects/ocean.py/test/address.json # copied from https://github.com/oceanprotocol/contracts/blob/v4main/addresses/address.json
export PROVIDER_URL=https://v4.provider.rinkeby.oceanprotocol.com
export METADATA_CACHE_URI=https://v4.aquarius.oceanprotocol.com
source secrets.sh # Set `TEST_PRIVATE_KEY1` and `TEST_PRIVATE_KEY2` using my own wallets containing Rinkeby ETH and Rinkeby OCEAN.

Execute the steps in c2d-flow.md

Expected behavior
ocean.assets.create() should return an Asset instance.

First wait for https://github.com/oceanprotocol/ocean.py/issues/793 to be merged

That issue already exists: #853. This one can be closed.


⚠ Quirk Club Argentina (Payment Gateway) has degraded performance

In 11cc7ec, Quirk Club Argentina (Payment Gateway) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/payment-gateway-ml) experienced degraded performance:

HTTP code: 200
Response time: 9438 ms

Resolved: Quirk Club Argentina (Payment Gateway) performance has improved in dce440f.


fix(ci): Set environment to prod

More: https://docs.github.com/en/actions/using-jobs/using-a-matrix-for-your-jobs

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


Send e-mail notification to user on email change

The latest version of devise (3.5.3) adds support for notifying users of a password change, which is a great addition, I've been doing this manually up until now.
Another thing that I am still doing manually, and is also considered best security practice, is notifying a users old email address when an accounts email has been changed. This has been proposed before (https://github.com/plataformatec/devise/issues/3691) and was marked as out of scope for Devise. Maybe with the latest change to (optionally) send a notification on password change, you might reconsider this feature request.
I'd be happy to send a PR if this is wanted.

+1 for email notification on the old email address regarding the email change. From a security standpoint this is really good to have and I'd advocate more sites to use this.
@josevalim Can you elaborate why think this is out of scope for Devise (#3691)? Or was that regarding the ability to cancel the change?

I think it's the same issue I've here.
My problem is about confirmation email when I change it. In devise higher than 3.5.3 confirmation email is not sent anymore.
It's a devise problem between version 3.5.3 and 3.5.4.
My test suit broke after this upgrade.
You can see this here:
passing: https://travis-ci.org/call4paperz/call4paperz/builds/109015545
broken: https://travis-ci.org/call4paperz/call4paperz/builds/109017932
the testing is not passing anymore is:
https://github.com/call4paperz/call4paperz/blob/feature/security-updates/spec/controllers/profiles_controller_spec.rb#L37

+1 for this too

@carlosantoniodasilva any update here?

can you guys confirm if this commit: https://github.com/plataformatec/devise/commit/1d57169c7bd12977a6697a5a06cda265442bb5c8
solve this problem?
tks

It looks like this sends an email when updating the email address from nil, but it wouldn't send an email when changing it from a previous valid email address?

Any update? How to send email to previous email on email change?

@marcosanfilippo please check #4455 for the new feature/option to enable that.


Python 3 (with tests)

@matclayton

Drops support for Django 1.7 (not supported, conflicts with jsonfield 2.0).
Add support Python 3.3, 3.4, 3.5 and 3.6.
Also adds config for for Travis CI.
Second class support for PyPy (tox tests pass) since Travis CI PyPy install is broken.
All tests pass. :slightly_smiling_face:

Test results from Travis CI at https://travis-ci.org/tysonclugg/django-experiments:

Test results from tox on my machine:

Thanks not looked at this in depth, more than happy to drop 1.7! We currently use codeship for CI, so I'd like to get it working on there as well. I'll need to take some time and figure it out, but once thats done, I'm happy to merge/ship this out.

I tried to look at your Codeship builds, but they're not public?  Travis is free for open source...

Great point, I have no idea how we didn't spot that! I'll try and open it up.

This PR resolves the following issues:

#143 Is Python3 supported?
#148 Hardcoded version of 'mock' and 'django-modeldict-yplan' breaks people's projects
#157 Add backwards compatible python3 support


Signature Validation Failed

I am trying to test the digital signatures in the sandbox and I am getting the following error :
{
  "errors": [
    {
      "errorId": 215120,
      "domain": "ACCESS",
      "category": "REQUEST",
      "message": "Signature validation failed",
      "longMessage": "Signature validation failed to fulfill the request."
    }
  ]
}

I am calling the Signature::getSignature and passing the 4 shown headers (the method documentation states to pass the map of headers).

Do I miss some headers to pass to the method ?
Also I cannot understand why Signature::getSignedRequest returns HttpServletResponse, I would expect the method to accept an HttpServletRequest and return an HttpServletResponse.
Thanks!

I've also tried to do it like this (like you do it in getSignedRequest by signing only the X_EBAY_SIGNATURE_HEADER and no other header that will be passed to the request) - but I am still getting the exact same error message :
      Map<String, String> headersMap = new HashMap<>();
      headersMap.put(Constants.X_EBAY_SIGNATURE_HEADER, jwe);
      String signatureHeader = signature.getSignature(headersMap, signature.getSignatureConfig());

Ok I figured it out - I had to configure the signatureComponents in order for it to work properly.


How to use depth GT for training?

Hi, thanks for your excellent job. I noticed that the depth GT provided in the dataset is not the actual depth of the scene but values between 0-254. Could you please share about how do you use the depth GT for training?

Sorry for the inconvience. We forget to include the information to docs.
Depth maps are stored as uint8 PNG images, where a value of 255 signifies an invalid pixel (i.e., out of range). For valid pixels, the depth in meters can be calculated by converting the uint8 value to a float and multiplying by $1000.0 / 255.0$:
$$
depth(u, v)=I(u, v) \times \frac{1000.0}{255.0}
$$
We have observed that the transformation process introduced some numerical errors. We plan to upload a version with improved accuracy shortly.

Hello,
I have downloaded the depths from huggingface and I found that the depth maps have many stripes. it seems like they are out of the range of int8. How can I convert them to true depths?

We have released a new version of depth image with more accurate precision. Please refer to Getting Started to download the data. The usage of the data can be found in toturial.ipynb.


Dynamic progress bar only prints when accessing progress bar

The current implementation only prints when indexing into the progress bar and doesn't expose the print_progress method.
This leads to problems when the last access to the progress bar sets e.g. an option to make the postfix text to "Finalize" it's not going to be printed for the last-to-finish bar.
I think the most formally correct way to implement this would be with either a proxy object (instead of returning a real reference from the accessor, or with a set_option / set_progress method that also takes an index.
What do you think?

There is a method in the progress bar class called .mark_as_completed() that you can call (by indexing) that will print progress one last time. This was written specifically for this purpose where properties like postfix_text need to be updated at the end of progress. Would this work for you?

Hmm, I see, yeah that might work. Anyways, I have some ideas on improvements for the dynamic progress bar. I'll try a PR when I have something.
My goal would be to have something similar to DNF:
https://user-images.githubusercontent.com/885054/78633663-71f2e400-78a2-11ea-9f51-c9de76c36d1e.png
so that one also sees some global info like curretn total download speed.

There is an additional issue tied to this where the initial start time is not set for the 1st printing, and then the elapsed time is some very long time :)

There is an additional issue tied to this where the initial start time is not set for the 1st printing, and then the elapsed time is some very long time :)

Yeah this should definitely be an issue of its own. I've noticed this on MultiProgress too.


Added DNR Changes

Added DNR Tests
Updated Test Recordings

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Communication.PhoneNumbers


New Event in codeunit 5942

Please add a new event in the OnRun trigger,
IF TemplateDisc.FIND('-') THEN
REPEAT
ContractDisc := TemplateDisc;
ContractDisc."Contract Type" := ServContract."Contract Type";
ContractDisc."Contract No." := ServContract."Contract No.";
ContractDisc.INSERT;
UNTIL TemplateDisc.NEXT = 0;
//New event
OnSetupServiceContract(ServContract,ContractTemplate);
Rec := ServContract;
where
[IntegrationEvent] OnSetupServiceContract(var ServContract: Record "Service Contract header";ContractTemplate: Record "Service Contract Template");
Thank you very much.

Please create request for latest code base, this code is not actual anymore for 15.X

No update, closing


Don't send response body on status 204 No Content

It's a protocol violation.

A 204 response is terminated by the end of the header section; it cannot contain content or trailers.

https://www.rfc-editor.org/rfc/rfc9110#section-15.3.5-5

Cherry-picked to v9.2.x


[qtbase]unable to locate ICU library icuind73.dll using windeployqt.debug.bat

Describe the bug
windeployqt.debug.bat failed to locate icuind73 but the dll already exists in the app dir.
Environment

OS: Windows
Compiler: MSVC 2022

To Reproduce
Steps to reproduce the behavior:

(path to vcpkg)/packages/qtbase_x64-windows/tools/Qt6/bin/windeployqt.debug.bat  app.exe

Expected behavior
windeployqt.exe should finish deploying .
Failure logs

[build] EXEC : warning : Translations will not be available due to the following error. [C:\program_on_git\APP\app.vcxproj]
[build]   Cannot open C:/program_on_git/SFR/sfr-vcpkg/packages/qtbase_x64-windows/translations/Qt6/catalogs.json
[build]   C:\program_on_git\APP\app\build\bin\Debug\app.exe 64 bit, debug executable
[build]   Unable to locate ICU library icuind73.dll

Additional context
But icuind73.dll has been copied to the app directory.

@QiuYilin Could you provide an app.exe to reproduce the issue?

I meet the same question.


[Microsoft.StorageSync] New API version 2022-06-01

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
We are looking to roll out the new API version ASAP, hopefully by the end of this month or early September.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
We expect to publish swagger ASAP, once the new API version is supported in our service and enabled in ARM (through the manifest).
If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following appy to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@ArcturusZhang before we can merge this PR, we need to fix the auto-generated readme file. Here's a PR into this branch to fix the readme file: https://github.com/Azure/azure-rest-api-specs/pull/20262
Please help merging that PR so that we can proceed with this one.

/azp run


support for Traditional Chinese - Handwriting keyboard

how does one enable Traditional Chinese - Handwriting keyboard for a given text field and support it?
looking for hints, actually want the user to type in Japanese-Kanji using Apple's iOS graphic kanji pad

is it possible to check if the iPhone has the  Traditional Chinese - Handwriting keyboard installed?
how to use it if available?

Hi! You can use any views with KeyboardKit so you can just create views or buttons that send a text action to the action handler.


Update reat-base-table to latest verison for React 17 support

Update react-base-table  to latest version
Fix small breaking changes

Benefits:

React 17 support
Smaller bundle size
Better table performance

There are some unrelated changes in the lock files due to known issues with lerna/npm and aliased packages (ex: "@fluid-tools/webpack-fluid-loader-previous"). I'll look into mitigating that.


Move reduction kernels to reduction module

I swear on the Zen of Python that I have only copy/pasted large blocks of code and not maliciously dissimulated opinionated and controversial changes in this PR.

Will merge when test can pass (must wait for https://github.com/soda-inria/sklearn-numba-dpex/actions/runs/4303504742/jobs/7503319872 to be completed first for new docker image to be available)


Export number of read_only_indices

I'm interested in getting a count of read_only indices in my ES as we get over the flood stage ( https://www.elastic.co/guide/en/elasticsearch/reference/master/disk-allocator.html ). This stat doesn't seem to be exported here, and I was just curious if there was a reason why or if you would welcome a PR for that?

PRs are always welcome 😃
Which API exposes the info about index being in read_only mode?

Looking at this a bit. Looks like GetSettingsReponse would provide it?
Would have to supply the request portion with the correct IndiciesOptions so that wildcards for _all work and such, and use the option that ignores unavailable indices.
https://github.com/elastic/elasticsearch/blob/v7.5.0/server/src/main/java/org/elasticsearch/action/admin/indices/settings/get/GetSettingsResponse.java


Integer validation problem

If id parameter except of integer value has a space symbol inside, this code will pass the validation without any problems:
        $v = Validator::make($request->all(), [
            'id'   => 'required|integer',
        ]);

        if ($v->fails()) {
            return $this->respondBadRequest($v->errors());
        }

while $request->input('id') still is a string with a space. I think, many developers use integer validation rule and are sure that the value is 100% integer after validation.

 123 will pass
123  will pass
123 will pass
1 23 will fail
PHP considers an integer with a leading or trailing whitespace as integer, that's why the validation passes.

Strings like 42 cats can be cast to 42 as an integer in PHP. No idea why they thought that would be a good idea.

Yeah that is strange, it makes you think twice before casting a string to integer.

PHP considers an integer with a leading or trailing space as integer

$a = '1 ';
var_dump($a); // string(2) "1 "

is_int($a) is false too.
May be you mean that GET parameters always are strings? That's true.
@GrahamCampbell so in this case validation doesn't really validate, but tries to convert a given string into integer, and if it's succeed, returns a positive result. But the string is not converted into integer after validation. And I'm sure many developers who use 'integer' validation rule are sure that after validation this string has an integer type, but it's still a string with any symbols including dangerous. They have to manually use intval() for all such parameters.

@Chimit The Validator doesn't modify the given attributes, it only validates. While many developers might expect [ ]1 to return false, some others would expect it to be true since filtering with FILTER_VALIDATE_INT will consider it true. So PHP considers it as a valid integer.

@DeeJaVu Can I ask why you are downvoting every single comment I make on any issue? I'm considering reporting you to GitHub.

Think of a form input where you type for age [26 ], the trailing space here will be ignored and the value will be validated as an integer successfully.

I'd be happy to see another stronger integer validator though. I'd like to see a PR to add a new validation rule that's more strict.

@themsaid sure, but it would be saved in the DB as integer 26 or string [26], not as string [26 ].
I'm not sure, but I think it's a very unclear place, and somebody could rely on validation and put such string into the DB without other sanitization.
At least the Laravel documentation should point on this moment. Or renaming "integer" to "integerable" is needed :)


The search-fn property "prev" of navigate does not contain the validated search

Describe the bug
The prev property of the navigate-search-fn does not contain the validated-search.
Your Example Website or App
https://stackblitz.com/edit/tanstack-router-thggwk?file=src%2Fmain.tsx
Steps to Reproduce the Bug or Issue

Go to https://stackblitz.com/edit/tanstack-router-thggwk?file=src%2Fmain.tsx
Open the console of the dev tools
Click on the navigate-button at the posts-route

See the console output

Expected behavior
The prev-prop should contain the search, which was validated.
Screenshots or Videos
No response
Platform

OS: Linux Ubuntu
Browser: Firefox
Version 120.0b7 (64-bit)

Additional context
No response

This issue still exists as of beta 246.


Setup Pipeline (Lint)

Fix the bash script linting process by disabling unnecessary SC1091 errors during the spellcheck execution.

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


Errors with a list of objects

I experimented with lists of object instances:
' Define an object class

class Object
  var myadjective = "UNINITIALIZED"

  def Describe(s)
    myadjective = s
  enddef

  def Write()
    print "I'm an ", myadjective, " object!";
  enddef
endclass

print "Object List Test";;

' Create a list of object instances

Obj1 = new(Object)
Obj1.Describe("awesome")

Obj2 = new(Object)
Obj2.Describe("ugly")

ObjectList1 = List(Obj1, Obj2)

Obj1 = Nil
Obj2 = Nil

for X in ObjectList1
  X.Write()
next

' Create another list of object instances

Obj3 = new(Object)
Obj3.Describe("unfathomable")

Obj4 = new(Object)
Obj4.Describe("ordinary")

ObjectList2 = List(Obj3, new(Object))

Obj3 = Nil
Obj4 = Nil

for X in ObjectList2
  X.Write()
next

' Try to create a third list of object instances

ObjectList3 = List(new(Object), new(Object))

Here is the output from running the program:
Object List Test

I'm an awesome object!

I'm an unfathomable object!
I'm an ordinary object!
Error:
    Ln 53, Col 35
    Code 28, Abort Code 3
    Message: Colon expected.

At line 42 it executes, but the second item in the list is Obj4, instead of an uninitialized fifth object instance.
At line 53 it fails.

@paladin-t curious about the status of this issue.


Please add license

Hey guys, excellent library!
Watching its development for some time already.
Could you pls specify the license for it?
For most of the OS community, that would be a crucial argument

Thank you for opening this issue. Just added Apache License 2.0 :)


Cull the unneeded dependencies!

Description
Enhancement
Reason to enhance/problem with existing solution

The less we support, the simpler this project will become! And the easier it is to optimize and improve on performance and code quality!
Improves delivery effectiveness! 🚢

Suggested enhancement

Remove unneeded babel plugins

[ ] https://github.com/loganfsmyth/babel-plugin-transform-decorators-legacy
[ ] https://github.com/babel/babel/tree/master/packages/babel-plugin-transform-react-constant-elements
[ ] https://github.com/oliviertassinari/babel-plugin-transform-react-remove-prop-types

[x] isomorphic-fetch (we already use axios)

I'm referencing create react app's babel plugins as well, over here
Please suggest more if you find an unneeded dependency!

just leaving "babel-plugin-transform-react-inline-elements"?

You mean remove it?  🎉

no. I was just asking..

I might have to double check. It promises to make production code faster but I haven't seen many projects using it.

Some of the dependencies were already removed (linked PR's) so i suggest we close it. We can always try and clean up additional dependencies and open more issues.


Report unidentified on Microsoft Windows for foriegn items

Decided to smoke test what goes wrong when enumerating items in a directory, and it turns out that the function lookup_account_sid in meta\windows_utils.rs was the culprit! According to the Win32 API documentation on LookupAccountSidW:
If the function cannot find an account name for the SID, GetLastError returns ERROR_NONE_MAPPED. This can occur if a network time-out prevents the function from finding the name. It also occurs for SIDs that have no corresponding account name, such as a logon SID that identifies a logon session.

This can generally happen if you create a file or folder on a system using NTFS and copy it over to another system using NTFS as information about the name and domain are preserved and not pummeled. This may or may not happen and is probably dependent on the filesystem and or OS configurations. Instead of failing this function outright, and since we already know the item exists if we got up to this point, we should just report that we don't actually know what permissions we can use or, account name, and domain owns this item. There is probably some way to calculate this properly since Explorer does it, but for now this is good enough.
Fixes https://github.com/Peltoche/lsd/issues/356

Codecov Report

Merging #473 (64a834f) into master (1396c59) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #473   +/-   ##
=======================================
  Coverage   84.58%   84.58%
=======================================
  Files          35       35
  Lines        3353     3353
=======================================
  Hits         2836     2836
  Misses        517      517

Continue to review full report at Codecov.

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

Codecov Report

Merging #473 (64a834f) into master (1396c59) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #473   +/-   ##
=======================================
  Coverage   84.58%   84.58%
=======================================
  Files          35       35
  Lines        3353     3353
=======================================
  Hits         2836     2836
  Misses        517      517

Continue to review full report at Codecov.

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

Hey, thanks for looking into this. I think we could report something like this for now until we have a better way of figuring out the proper user.
That said, I am keeping the PR open for a few days just in case someone who had the original issue in https://github.com/Peltoche/lsd/issues/356 / or knows a bit more about how Windows permission system works has any suggestion on how to go about it.

Hey, thanks for looking into this. I think we could report something like this for now until we have a better way of figuring out the proper user.
That said, I am keeping the PR open for a few days just in case someone who had the original issue in https://github.com/Peltoche/lsd/issues/356 / or knows a bit more about how Windows permission system works has any suggestion on how to go about it.

I have had some time to think about a better way that we could do this. Just hardcoding "UNIDENTIFIED" might not be the best idea. We could make the Owner struct optional for windows and have it display - in case of error with the greyed out color. This will make sure we keep the output consistent. (Also just - instead of -\- which would be the case with this).

I have had some time to think about a better way that we could do this. Just hardcoding "UNIDENTIFIED" might not be the best idea. We could make the Owner struct optional for windows and have it display - in case of error with the greyed out color. This will make sure we keep the output consistent. (Also just - instead of -\- which would be the case with this).

I hope this satisfies the requirements, at least temporarily!

Actually re-read the requirements and saw that my previous commit did not actually satisfy those requirements. The error is now handled by the caller to ensure that the displayed metadata appears as - and not -\-.

Codecov Report

Merging #473 (bd458cc) into master (1396c59) will increase coverage by 3.86%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #473      +/-   ##
==========================================
+ Coverage   84.58%   88.44%   +3.86%
==========================================
  Files          35       36       +1
  Lines        3353     3540     +187
==========================================
+ Hits         2836     3131     +295
+ Misses        517      409     -108

Impacted Files
Coverage Δ

src/config_file.rs
69.23% <0.00%> (-11.04%)
:arrow_down:

src/flags/symlink_arrow.rs
96.00% <0.00%> (-4.00%)
:arrow_down:

src/meta/symlink.rs
63.82% <0.00%> (-3.52%)
:arrow_down:

src/main.rs
7.69% <0.00%> (-2.31%)
:arrow_down:

src/meta/filetype.rs
80.00% <0.00%> (-1.73%)
:arrow_down:

src/flags/recursion.rs
91.52% <0.00%> (-0.93%)
:arrow_down:

src/flags/sorting.rs
99.06% <0.00%> (-0.45%)
:arrow_down:

src/color.rs
78.64% <0.00%> (-0.36%)
:arrow_down:

src/app.rs
95.83% <0.00%> (ø)

src/meta/size.rs
94.15% <0.00%> (ø)

... and 17 more

Continue to review full report at Codecov.

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

Would be great to see this merged! I keep running into the 1332 error all over the place.

also, please check the edit locally, it seems there are syntax errors

Builds just fine on my end, looks like the issue is with the version of Rust that's used to build it.


Bot Authenticates but doesn't detect comments or DM's

Like the title says the bot will sign in and authenticate but will not detect DM's or comments of it being mentioned

Did you fill in the name of your bot in the config.yaml?

Yes I have filled the bots name in config

After restarting the bot for another time it seems to have noticed all of the DM's and mentions and is starting to send them out. Sorry for wasting your time.


Release 0.0.17

Codecov Report
Attention: Patch coverage is 93.54839% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 78.97%. Comparing base (2f657a0) to head (a5c7188).

Files
Patch %
Lines

src/sdr/_detection/_theory.py
92.50%
6 Missing :warning:

src/sdr/_detection/_non_coherent_integration.py
91.66%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #319      +/-   ##
==========================================
+ Coverage   78.62%   78.97%   +0.35%
==========================================
  Files          58       59       +1
  Lines        3508     3595      +87
==========================================
+ Hits         2758     2839      +81
- Misses        750      756       +6

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


refactor(ellipsis tooltip): create dialog content on hover

This allows the ellipsis tooltip to generate the dialog content on hover, previously it only worked when trigger is click. Also prevents an empty tooltip from being generated when there is no overflow.

Closing in favour of #1293


For 2013 'Sending 2000 bytes over commodity network' number is 0.0ns

Visit http://www.eecs.berkeley.edu/~rcs/research/interactive_latency.html and move slider to the year 2013. You will notice Send 2000 bytes over commodity network value to be 0.0ns which does not seem right. The micro-seconds value is OK.

I think this is fixed - at least I can't reproduce it any more. Could you please re-check and report if the issue is still happening?
BTW, the link now is http://people.eecs.berkeley.edu/~rcs/research/interactive_latency.html

@cdman Yes, it seems fixed now.

2020  Send 2000 bytes over commodity network value 44ns < memory 100ns which does not seem right.


[tensor] ColoTensor supports ZeRo

Usage:
chunk_size = 38 * 1024**2 if use_chunk else None
chunk_manager = ChunkManager(chunk_size, enable_distributed_storage=use_zero)
model = ColoDDPV2(model, chunk_manager)

chunk_size=None means chunk is not used.

The main concern is about suspending parameters.
For example
class Net(_)
  def __init__
       self.fc1 = Linear()
       self.suspend_param = torch.Paramer(..)

The self.suspend_param will occur in module.parameters(). So your DDPV2 will append it to the chunk manager.
Managing the state of the param will be a disaster for your design?

The main concern is about suspending parameters. For example
class Net(_)
  def __init__
       self.fc1 = Linear()
       self.suspend_param = torch.Paramer(..)

The self.suspend_param will occur in module.parameters(). So your DDPV2 will append it to the chunk manager. Managing the state of the param will be a disaster for your design?

Finally, we will use computation graph to derive the computation order of params and filter those unsed params. Chunk manager only manages used params


JPEG serializer

Tiled uses Pillow to serialize png files out of the box.  There is a custom exporter example for also handling jpeg.
Is there an strong reason not to put this in the core array serializer? We have  a lot of jpegs that we would like to serve via tiled.

Great minds think alike. @jwlodek took care of this: https://github.com/bluesky/tiled/pull/796
We should update the docs to use a more obscure format, so let's leave this issue open.

Awesome. Thanks!


Don't auto-replace PhasedXPowGate instances with XPowGate or YPowGate

This would make my life a lot easier :) so big +1 from me
I'm also mildly confused about the value equality special casing. Could you motivate this a bit more?

I special cased the equality because not doing so broke many, many tests. I believe all those tests could be rewritten, but it was tricky to do so.

Let's open an issue then for this then, since I think it seems strickly not necessary.

Automerge cancelled: A status check is failing.


SIL: Fix DynamicSelfType references from inside init accessor

We need the self metatype parameter to correctly lower DynamicSelfType in IRGen, so plumb this through to all calls of init accessors, and inside the prolog of an init accessor definition.
This does not break the public ABI, because init accessors are never public. Also for value types, the metatype is thin, so it should not change generated code.
For classes we need the metatype in the general case because of Self, but hopefully in most cases the init accessor can be inlined away and the value_metatype instruction subject to dead code elimination.
Fixes rdar://problem/119822466.

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test Linux

@swift-ci Please smoke test Linux

@swift-ci Please smoke test Windows

@swift-ci Please smoke test Linux

@swift-ci Please smoke test Linux


confusing term: GO:1990296 scaffoldin complex

Looks like it was created for bacterial structures (taxon restraint: prokaryota) but has no annotations. I stumbled over it looking for "scaffold complex" or "scaffolding complex" akin to MF GO:0032947 protein complex scaffold.

Can we fix the existing term so it's clear what it's for?

Would it be reasonable to create a CC term akin to GO:0032947 protein complex scaffold? Looks like I may be able to use the TG template for this.

PMID: 23219485 - Atg17-Atg31-Atg29 complex is a scaffold for the larger ATG1 complex (GO:1990316 ATG1/ULK1 kinase complex)
Thanks,
Birgit
Note to self: complex EBI-11686111

I don't think we should do that. The MF term  "protein complex scaffold" (could probably be named better, is used to describe the activity of a protein  (often a WD repeat) which  provides a platform onto which c complex assembles. For example
For example, cdc11 for the septation iniitiation signalling complex
http://www.pombase.org/spombe/result/SPCC1739.11c
(note to self, we do not have this complex annotated)
or git5 in one of the fission yeast heterotrimeric G-protein complexes
its not the purpose of the complex per se.

Thanks, Val.
ATG17 (one of the members of my complex) in pombe and S. cerevesiae is annotated to GO:0032947 protein complex scaffold. The other 2 members have no MF annotation (so far) but ATG17 is the big member that dimerises and ATG31 binds to (ATF29 binds to ATG31).

Hi Birgit,
This appears to be a complex involved in membrane curvature, like the F-Bar proteins?
I'm not sure that its a scaffold in the sense of the MF GO term, but you are correct this  term is not  very clear.
Val

Any suggestions for clarifying the def of 'scaffoldin complex'?

Would it be reasonable to create a CC term akin to GO:0032947 protein complex scaffold?

Surely we wouldn't want both.  And you can use capable_of to link existing MF term to complexes.

Any suggestions for clarifying the def of 'scaffoldin complex'?

Can't help with that, not my term, just something I stumbled over. Sorry :(

Would it be reasonable to create a CC term akin to GO:0032947 protein complex scaffold?
Surely we wouldn't want both. And you can use capable_of to link existing MF term to complexes.

I think this complex initiates the assembly of the preautophagosomal structure (PAS), the first step of autophagosome formation. I haven't seen anything that it is actually involved in membrane curvature (but it might, given its characteristic crescent shape!).
So, I would still argue we might need a CC term that can group scaffolding complexes. Otherwise, they all remain annotated to 'protein complex' or I have to request specific, composition-based complexes for these terms.
In this case, the released complex is annotated to 'protein complex' for now but I could request
Name: Atg17-Atg31-Atg29 complex
Def: Scaffolding complex required for the assembly of the preautophagosomal structure (PAS), the first step of autophagosome formation. Constructively present under both growing and starvation conditions. Under conditions of nutrient starvation, it forms a complex with ATG1 and ATG13 to enhance the kinase activity of ATG1.

Hello @bmeldal
Should we obsolete 'GO:1990296 scaffoldin complex'? It was created in 2014 and never used yet.
Also the cited papers refer to another term, 'Cellulosome', which is a parent and has 3 annotations.
Thanks, Pascale

Happy with the obsoletion but better check with the DB that requested it. I've never used the term, it was a chance find that caused confusion.

The terms (the complex and the complex binding terms) were requested by MENGO for plant papers. @tberardini do you know anything about this ?
Thanks, Pascale

@pgaudet I don't know any more about this than what can be found with PubMed searches.
The term does have these relationships

is_a: GO:0043234 ! protein complex
relationship: part_of GO:0043263 ! cellulosome

The parent term 'cellulosome' is defined as:

An extracellular multi-enzyme complex containing up to 11 different enzymes aligned on a non-catalytic scaffolding glycoprotein. Functions to hydrolyze cellulose. Source: PMID:11601609, GOC:jl

I found this:
http://www.sciencedirect.com/topics/agricultural-and-biological-sciences/cellulosome

In general, the cellulosome can be divided into three modular protein components: (1) cell-surface proteins, (2) scaffold proteins, and (3) enzymes (Fig. 7A).

My guess is that the 'scaffoldin complex' = (2) scaffold proteins above.
I hope this information helps in deciding whether or not to keep the term.  Just because the term hasn't been used yet doesn't mean we should get rid of it.
One more point:

First discovered in the thermophilic anaerobe Clostridium thermocellum, cellulosomes have since been identified in a variety of other anaerobic bacteria, ruminal bacteria, and anaerobic fungi (Bayer et al., 2004).

Based on the identification in anaerobic fungi (above), the taxon restriction on 'cellulosome' needs to be broadened beyond Prokaryota.

Summary: I will merge the two terms:
the papers cited in the term (PMID:20373916, PMID:15197390) are actually annotated to the parent GO:0043263 cellulosome, indicating that the term is not really useful for annotation. Moreoverm the titles of these two papers mention “cellulosome”.
Thanks, Pascale

@bmeldal
Just checking one last time: the scaffoldin seems to be a subcomplex of the cellulosome. Is this a legit complex ? I can keep both terms and change the definition as suggested by Tanya, or merge.
Thanks, Pascale

If the subcomplex has a function on it's own then it would be legit, if it's only a pre-assembly construct I would say not.
From the thread above it looks like the latter and as it has no annotations I'd merge with the parent term and make it a NARROW synonym so at the the term is findable.


fix https://github.com/yegor256/takes/issues/945

move PqBaseMethod outside of parent interface
fix https://github.com/yegor256/takes/issues/945

@yegor256 check please

@Kartofanych what is the motivation for this? What problem are you trying to fix with these changes?

@Kartofanych what is the motivation for this? What problem are you trying to fix with these changes?
this is fix of issue https://github.com/yegor256/takes/issues/945
you asked to check issues that repository has - i found this one

@yegor256 Any updates on this? (course is already finished)

@Kartofanych the issue https://github.com/yegor256/takes/issues/945 was just a proposal, not an accepted bug (I closed it now). Please, look at the issues that are labeled as "bug" only


Why does the _uid is diff from 'for' attribute _uid value in label  in the same component when reload the window?

Version
v2.9.2
Reproduction link
https://codesandbox.io/s/youthful-hofstadter-5is4o?fontsize=14&hidenavigation=1&theme=dark
Steps to reproduce

reload window

What is expected ?
get same _uid in input ID and label 'for' attribute value
What is actually happening?
label 'for' attribute value is different with _uid in this component when  i reload window
Additional comments?
index.vue
<template>

    <Test></Test>

</template>

Test.vue
<template>

      label_uuid: {{_uid}}
      <input type="radio" :id="uuid" :value="_uid">

      <
      label for="{{label_uuid_for_value}}"
      />

      label_random: {{random}}
      <input type="radio" :id="random" :value="random">

      <
      label for="{{label_random_for_value}}"
      />

</template>

layout

    label_uuid: 9
    <input type="radio" id="9" value="9">

   < label for="59"/>

   label_random: 44530
    <input type="radio" id="44530" value="44530">

   <label for="64275"/>

This bug report is available on Nuxt community (#c10094)

Your value should be same on client and server e.g. the values that are generated in mounted. In your case they are different and that will lead to such behaviour. If u want to use random values change them in mounted, not created

Thanks, mounted is works! !! But only the tag "for" attribute has a problem, that's weird.
If this's done only in mounted, will it affect SEO?

if its done in mounted it will be changed only on client so it wont be in server generated html


Add endless mode

Run in endless mode, continuously logging anomalies to stdout.

 Builds on my mac, though.


How to make so that when you choosing the date the calendar is hiding

Hello. Thank you for the work done. I am using your framework in "single day" mode. All is fine, but there is one question: "How to make so that when you choosing the date the calendar is hiding?"

I had the same issue and created a pull request under #191 . You can either wait for the dev's to review it or use it from the pull request.

@FredUK's PR has been merged into the dev branch. I'll close this issue when I release 0.7


fix(L3): workaround status checks associated with protected branches

Pull Request
Prerequisites

[ ] My code follows the coding conventions
[ ] All tests pass

Description

Update lib update pipeline to work on main branch and push to same branch.
Workaround status checks in protected branches using a temporary branch that will be deleted by the pipeline at the end.

@yohanAnushka - can we check if this PR is still needed? was raised 5 months ago xD. if it's still needed, maybe we could rethink if we should adopt the same approach that @binaryunary used to bypass ADO pipeline from making direct commits

Lets go with Benny's workaround.


Terraform for ECS Deployments

There should be a terraform to create all the AWS infrastructure required to deploy applications on ECS.
Initial tasks:

Potentially use ECS blueprints as a starting point to create the cluster
Create RDS, RabbitMQ and other infrastructure required by the application components (re-use the existing EKS terraform library)
Create IAM Roles, Task Definitions and Services for each component
Choice between EC2/Fargate deployment models for services.

@niallthomson Please assign this to me.

Completed in #434


Add Result

Pull Request Description
Replace with a description of this pull request. Instructions for adding
projects are available in the README.
Acceptance Criteria
To be accepted into the Swift source compatibility test suite, a project must:

[X] be an Xcode or swift package manager project
[X] support building on either Linux or macOS
[X] target Linux, macOS, or iOS/tvOS/watchOS device
[X] be contained in a publicly accessible git repository
[X] maintain a project branch that builds against Swift 3.0 compatibility mode
and passes any unit tests
[X] have maintainers who will commit to resolve issues in a timely manner
[X] be compatible with the latest GM/Beta versions of Xcode and swiftpm
[X] add value not already included in the suite
[X] be licensed with one of the following permissive licenses:

BSD
MIT
Apache License, version 2.0
Eclipse Public License
Mozilla Public License (MPL) 1.1
MPL 2.0
CDDL

[X] pass ./check script run

Ensure project meets all listed requirements before submitting a pull request.

@swift-ci test

@swift-ci Please test

Thanks!


Reset button

For when you want to restart your combos.

@Louiso you can reset combo counter with atom dev tools


Update index.md

Fixed typo.

@r3nya thank you for your PR! Sorry that it took us so long to get to merging it. I've merged your change. You may also contribute translations for Cordova in the future using Crowdin. Thank you again for your contribution! :)

@dblotsky Hey! Thanks for your advice. I'm going to use crowdin. :+1:


Improve autofill subdomain/url handling

Task/Issue URL: https://app.asana.com/0/488551667048375/1203418026431060/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #2552   👈

This comment was auto-generated by Graphite.


Fix IS issues

TXLOCKREQUEST should be processed as normal tx plus some custom logic, should not "fake" inventory
should not create "fake" local lock, should instead keep track of orphan votes and reprocess them when corresponding TXLOCKREQUEST arrives
orphan vote time map should be indexed by full outpoint, not by txid of mn collateral
bump MIN_INSTANTSEND_PROTO_VERSION (i.e. requires proto bump to work on network scale but you can revert MIN_INSTANTSEND_PROTO_VERSION back to 70202 to test it locally).

utACK - I can see the changes you made priorly in your previous commit to txlocks and the txcandidates.


AppBar: pad toolbars not tab bars

The tab bar and its selection indicator should extend to the edges of the screen.
The horizontal padding that used to be applied to both the tab bar and the toolbar is now just applied to the latter.
Fixes #5122


ARK and NEO addresses are too similar

Hi,
I accidentally sent ARK coins to a NEO address, since the client considered it to be a valid address. I'm assuming my coins are lost for good. However, I would like to make sure others don't run into this issue in the future. Is there any way to prevent this (e.g. by giving a warning if the destination address doesn't have any transaction history?)

by giving a warning if the destination address doesn't have any transaction history?

I think this is quite a good idea. It also helps users of cold storage wallets, to encourage them they check twice their cold wallet address.

After filling the form to send ARK, there's a confirmation step with all the relevant information. Adding a new confirmation should be unnecessary.
Instead of your suggestion of triggering a warning if the destination address is empty, we could show it in that confirmation page. We could check that is an existent ARK address, or even a NEO one, and show different text depending on the case.
IMHO is a very uncommon problem, so this issue would be a not urgent enhancement. @boldninja what do you think?

Either one of these solutions sounds like it would prevent this issue in the future, thanks for the prompt replies!


duplicate entries with same primary index value

I've been noticing this bug happening and thought it was my code but after looking at the keys are defined in both local development and deno deploy, i think that's the issue
This is how the keys are defined locally(GUI using kv-insights)

And this is how it's seen on deno deploy

As you can see, KvId is an array of numbers locally while it's a string on deno deploy. I'm not using history because i don't know it's usage.
Here's the function that deals with creating new users if they weren't identified by their primary key
import { NextFunction } from "grammy";
import { GrammyContext } from "$grammy/context.ts";
import { logging } from "$utils/logging.ts";
import { getProfileLink } from "$utils/grammy.ts";
import { sendMessageToGroup } from "$grammy/helpers/sendMessage.ts";
import { db } from "$db/mod.ts";

export default async function attachUser(
  ctx: GrammyContext,
  next: NextFunction,
) {
  if (!ctx.chat?.id || !ctx.from?.id) {
    return;
  }

  const dbUser = await db.users.findByPrimaryIndex(
    "telegram_user_id",
    ctx.from.id,
  );

  if (!dbUser) {
    const cr = await db.users.add({
      telegram_user_id: ctx.from.id,
      metadata: {
        summary_language: ctx.from.language_code,
      },
    });

    if (!cr.ok) {
      throw new Error(`Failed to create user id ${ctx.from.id}`);
    }

    if (!IS_DEV) {
      await sendMessageToGroup(
        `👋 New user #joined: ${getProfileLink(ctx.from)} [id${ctx.from.id}]`,
      );
    }

    const user = await db.users.findByPrimaryIndex(
      "telegram_user_id",
      ctx.from.id,
    ); // using this because `add` doesn't return `value`

    logging.info("attachUser >> user", user?.id);

    if (!user) {
      throw new Error("User not found");
    }

    ctx.user = {
      id: user.id,
      ...user.value,
    };
    ctx.session = {
      ...ctx.session,
      user: {
        id: user.id,
        isPro: user.value.is_pro,
        isUser: true,
        accepted: true,
      },
    };
    logging.info(
      "attachUser >> user info attached to ctx after adding to db",
      user.id,
    );
  } else {
    ctx.user = {
      id: dbUser.id,
      ...dbUser.value,
    };
    ctx.session = {
      ...ctx.session,
      user: {
        id: dbUser.id,
        isPro: dbUser.value.is_pro,
        isUser: true,
        accepted: true,
      },
    };
    logging.info("attachUser >> existing user info attached to ctx", dbUser.id);
  }

  return next();
}

The actual document id seems to be a ulid string both on deploy and locally, based on your screenshots.
When it comes to the primary index (and all indices for that matter), this is a little more complicated:
Both locally and on deploy, the index value is serialized to a Uint8Array. I'm pretty sure your screenshots show that this is the case as well, as the Deploy KV view does not just display it as an array, but explicitly as a Uint8Array (check the HTML to see the entire key). So the last key entry should be a Uint8Array for both.
If you are having issues with the primary (or secondary) indices not matching between deploy and running locally, I may know why. Since the index values are serialized, it automatically chooses the best serialization strategy for the runtime that you are on. Since Deploy does not expose the internal Deno Core serialize methods, it defaults to the custom JSON serializer. Locally however, it uses Deno Core. This difference leads to different Uint8Arrays being created, which would not match. I agree that this is actually a problem. In the next patch, I will definitely make index-serialization default to the JSON serializer, and I'm considering removing the "auto" option for serialization altogether to make it very clear which serialization method is used.
thanks for bringing this issue to my attention.

Alright. Thanks for the clarification and yeah, it's really complicated from the looks of it. So i'm seriously looking forward to the next patch.
In the meantime, how do i solve this issue? I was about to delete the new _id__ by using db.users.delete when ctx.session.user.id and dbUser are available but don't have the same id but then doing it that way will retain the two entries of  __index_primary__ and i can't delete by primary index as it will delete both. Existing data in the old is what's important to me(No issue for me in local development as i'm the user but i'm afraid of its likelihood to occur in production on deploy for whatever reason)

Just be explicit and use serialize: "json" for all collections. This should work across from the current version to oncoming versions, and ensure that serialization is done the same locally as on deploy.

Thanks, that did the trick. I also had to wipe out the database both locally and on deploy because i was getting Cannot read properties of undefined (reading 'map') in places used by db.collection.deleteMany and other unknown locations(no biggie).


Cannot retrieve bootstrap password on first login to webgui

I'm very new to this.
I'm running:
ubuntu 22.04.2
docker 20.10.21
I installed rancher using the following command:
docker run -d --restart=unless-stopped
-p 80:80 -p 443:443
-v /opt/rancher:/var/lib/rancher
--privileged
rancher/rancher:latest
I can reach the prompt on the web gui asking me to run:
docker logs  container-id  2>&1 | grep "Bootstrap Password:"
to get my temporary password. When i run this command after getting the container ID using docker ps, the command returns nothing.
I have also tried setting the password by adding:
-e CATTLE_BOOTSTRAP_PASSWORD=password
to the docker run command, this did not work.
Thank you for the help.

@jakeconnolly you should also open an issue with https://github.com/rancher/rancher/issues. You're more likely to get help from the community there. We're focused on Rancher documentation and would need to reach out to an SME.


It works too well.
Code is ment to be buggy and trash.
Why does this work.
Illigal.

It works too well.
Code is ment to be buggy and trash.
Why does this work.
Illigal.

Thanks! I really believe that getting API data onto a spreadsheet should be an easy as building one.
Have a great day!
Bann


Amended Spelling Errors

Amended minor spelling errors in various files in the codebase.

okay will change that.

okay, will make the necessary changes.

Closing for now as I don't see any updates, let me know if I'm mistaken.


OCaml build of F* generates invalid constructors when extracting to F#

The OCaml build is adding bogus U__ prefixes to constructor identifiers when extracting F#.
The F# build seems to be ok here.
val maybe(#a #b:Type): b -> (a -> b) -> option a -> b
let maybe #_ #_ y f = function | None -> y
                               | Some x -> f x

Extracts to F# as
let maybe = (fun ( y  :  'Auu___5_19 ) ( f  :  'Auu___4_18  ->  'Auu___5_19 ) ( uu___6_36  :  'Auu___4_18 FStar.Pervasives.Native.option ) -> (match (uu___6_36) with
| FStar.Pervasives.Native.U__None -> begin
y
end
| FStar.Pervasives.Native.U__Some (x) -> begin
(f x)
end))

But extracts to OCaml as
let maybe:
  'Auu___4_18 'Auu___5_19 .
    'Auu___5_19 ->
      ('Auu___4_18 -> 'Auu___5_19) ->
        'Auu___4_18 FStar_Pervasives_Native.option -> 'Auu___5_19
  =
  fun y  ->
    fun f  ->
      fun uu___6_36  ->
        match uu___6_36 with
        | FStar_Pervasives_Native.None  -> y
        | FStar_Pervasives_Native.Some x -> f x

Note the FStar.Pervasives.Native.U__None vs. the correct FStar_Pervasives_Native.None

Just to clarify, do you mean “when extracting F*” at the start?

@iamrecursion I mean "when using the OCaml-generated F* binary".

Thanks for the clarification.

Fixed in #1192


NullPointerException when loading connector config properties in Console app

Describe the issue:
When loading the governance connector config properties in Console app, NullPointerException is thrown from [1] if the property list[2] contains null values.
Expected behavior:
List should be check for null values before looping through the properties.
Environment information:

Product Version: IS 5.11.0

[1] https://github.com/wso2/identity-api-server/blob/master/components/org.wso2.carbon.identity.api.server.identity.governance/org.wso2.carbon.identity.api.server.identity.governance.v1/src/main/java/org/wso2/carbon/identity/api/server/identity/governance/v1/core/ServerIdentityGovernanceService.java#:~:text=propertyRes.setName(property.getName())%3B
[2] https://github.com/wso2/identity-api-server/blob/master/components/org.wso2.carbon.identity.api.server.identity.governance/org.wso2.carbon.identity.api.server.identity.governance.v1/src/main/java/org/wso2/carbon/identity/api/server/identity/governance/v1/core/ServerIdentityGovernanceService.java#:~:text=Property property %3A connectorConfig.getProperties()

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Seek hangs on a truncated flac file

Hello!
I have a very large collection of flac files that I'm trying to process. A few of these seem to be truncated, so that the file metadata doesn't match the actual contents. When I try to seek past the end of the actually-available samples in the file, the seek operation hangs indefinitely.
Code looks like this:
with epath.Path(ap).open('rb') as f:
  sf = soundfile.SoundFile(f)
  print(len(sf) / sf.samplerate / 60)  # 119.2 minutes of audio
  for i in range(120):
    # File hangs on seek at i==12
    print('seek', i, flush=True)
    sf.seek(i * 60 * 32000)
    print('read', i, flush=True)
    audio = sf.read(10 * 32000)

It would be nice if an error were thrown instead of hanging indefinitely, as I could handle the error and move on.
Any recommendations for mitigations are also welcome. Thanks!

This is a duplicate of #379 I think. We're working on a solution in the mentioned issue. Feel free to help out!


Improve upgrade process

Create a new orphan branch releases containing release information. It will have following files:

info.json - information about newest release and certificates:

newest version number,
hash of the newest release,
file name of the script,
hash of the cacert.pem file,
extracted cacert needed only to get this information (github cacert),
signature of this file

wp_downlaoder.php
cacert.pem
README.md

If this will be implemented, following thinks can be removed from the source code:

downloading cacert.pem from curl.se and embedding its url/hash during the build.
prevention from downgrading, because update will always get latest version. It will be ensured by the following steps.

Reading release information consist of following steps:

download info.json
verify a signature
if HTTPS verification was not possible above:

apply extracted cacert from info.json
repeat from the beginning

Action running on schedule will (following step should also be in the release action):

download new cacert file only if .sha1 file contains different hash than in json, else use cacert from the last commit,
extract certificates needed by github from cacert,
create a new commit in releases branch if something was updated.

Extracting certificate only for github from cacert.pem:

function get_url($url, $cert) {
    $ch = curl_init($url);
    curl_setopt($ch, CURLOPT_USERAGENT, "WordPress Downloader PHP script");
	curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, true);
	curl_setopt($ch, CURLOPT_CAINFO, __DIR__ . "/" . $cert);
	curl_setopt($ch, CURLOPT_CAPATH, __DIR__ . "/empty");
    curl_setopt($ch, CURLOPT_FOLLOWLOCATION, true);
    curl_setopt($ch, CURLOPT_MAXREDIRS, 10);
    curl_setopt($ch, CURLOPT_HEADER, 0);
    curl_setopt($ch, CURLOPT_CERTINFO, 1);
    curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
    $res = curl_exec($ch);
    if (curl_error($ch)) {
        echo("\n" . curl_error($ch) . "\n");
        $res = false;
    } else {
        $status = curl_getinfo($ch, CURLINFO_HTTP_CODE);
        if ($status != 200) {
            echo("\nHTTP status code: $status\n");
            $res = false;
        }
    }
	if ($res !== false) {
		$res = curl_getinfo($ch, CURLINFO_CERTINFO);
	}
    curl_close($ch);
    return $res;
}

$list = get_url('https://raw.githubusercontent.com/kildom/wp_downloader/main/test.js', 'cacert.pem');
$certs = array();
foreach ($list as $c) {
	$x = openssl_x509_parse($c['Cert']);
	$x['-'] = $c['Cert'];
	$x['-in-cacert'] = false;
	array_push($certs, $x);
}

$list = file_get_contents('cacert.pem');
preg_match_all('/-----BEGIN CERTIFICATE-----.*?-----END CERTIFICATE-----/ms', $list, $m);
$cacerts = array();
foreach ($m[0] as $c) {
	$x = openssl_x509_parse($c);
	$x['-'] = $c;
	$x['-in-cacert'] = true;
	array_push($cacerts, $x);
}

$all = array_merge($certs, $cacerts);

function parseKeyIdentifier($key, $force) {
	$key = trim($key);
	$key = strtoupper(trim(preg_replace('/keyid\s*:?/i', '', $key)));
	if ($key == '' && $force) {
		echo('Invalid key!');
		exit(1);
	}
	return $key;
}

function find_root($c) {
	global $all;
	$subject = parseKeyIdentifier($c['extensions']['subjectKeyIdentifier'], true);
	$authority = parseKeyIdentifier($c['extensions']['authorityKeyIdentifier'], true);
	if ($c['-in-cacert']) {
		return $c;
	}
	foreach ($all as $cc) {
		$s = parseKeyIdentifier($cc['extensions']['subjectKeyIdentifier'], false);
		if ($s == $authority) {
			return find_root($cc);
		}
	}
	echo("Cannot find root for $subject!");
	exit(1);
}

$roots = array();

foreach ($certs as $c) {
	$r = find_root($c);
	$rs = parseKeyIdentifier($r['extensions']['subjectKeyIdentifier'], true);
	$roots[$rs] = $r;
}

$pem = '';
foreach ($roots as $root) {
	$pem .= trim($root['-']) . "\n\n";
}

file_put_contents('temp.pem', $pem);

$test = get_url('https://raw.githubusercontent.com/kildom/wp_downloader/main/test.js', 'temp.pem');
if ($test) {
	echo($pem);
} else {
	echo("Extracted certificate is invalid\n");
	exit(1);
}

Signing JSON:

$data['signature'] = '---SIGNATURE---';
$data = json_encode($data, JSON_PRETTY_PRINT);
$data = str_replace('---SIGNATURE---', sign($data), $data);

With this following levels of cacert workarounds can be used:

level 0 - use system cacerts
level 1 - securly download from github release branch
level 2 - use downloaded from github cacerts to download newest from curl.se

Done in v0.0.5


How do I setup another OPC DA Server?

Hey There,
The Websocket works with the Softing OPC Demo Server, I can read all of his Tag's.
But the Idea of this Webserver I guess is to read/write Data from multiple OPC DA Servers, right? so how can i change the origin OPC DA Server?
Thank you for help!

Hi there,
Great to hear it works well with your OPC Server!
It is a very rare situation that multiple OPC Servers co-exist in a production environment. It isn't by design to support multiple OPC Servers in the same box. But in Event Viewer you can find an entry like "Another OPC Server in the same box found and ignored: clsId progId" to indicate that you have multiple OPC Servers. I would suggest installing multiple servers one by one in each of your tests.
The whole ideas around it are,

Free everyone from DCOM and say goodbye to it in using legacy OPC Servers
Provide a cross-platform, cross-language access to legacy OPC Servers
Help legacy OPC Servers to enter mobile era and make native mobile development feasible / friendly
Revive legacy OPC Servers with modern technology to extend their lifetime since they are still widely adopted
Provide an Internet access port for Edge/Cloud use

Cheers!

Okay, my Issue ist more this :
I dont want to use the Softing OPC Demo Server, i'd like to replace him with another, local OPC DA Server in my Box.
Is there a possibility?

Uninstall Softing OPC Demo Server, and then see if you can use your local OPC DA Server.

This issue will be closed a week later if no further concerns.

Domenik001 - the feature you asked earlier is available now. Check the latest release and put your desired ProgID in config xml file and you are good to go. Let me know if you have questions. Cheers!


youtube-dl not working on Comedy Central

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

I'm trying to use youtube-dl to get some stuff off Comedy Central.  I keep getting the same error back, shown below.  I am using the most updated version (2000.05.03) and I've also updated Python.  I use a mac.  Command line (using verbose) along with output below:
$ youtube-dl -v http://www.cc.com/video-clips/kfpdwr/the-daily-show-with-jon-stewart-indecision-2000---2000-gop-debate
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'http://www.cc.com/video-clips/kfpdwr/the-daily-show-with-jon-stewart-indecision-2000---2000-gop-debate']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.05.03
[debug] Python version 2.7.2 (CPython) - Darwin-19.3.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg present, ffprobe present, rtmpdump Reason
[debug] Proxy map: {}
[ComedyCentral] the-daily-show-with-jon-stewart-indecision-2000---2000-gop-debate: Downloading webpage
[ComedyCentral] Downloading JSON metadata
[ComedyCentral] b502b41e-ed00-11e0-aca6-0026b9414f30: Downloading info
[ComedyCentral] b502b41e-ed00-11e0-aca6-0026b9414f30: Extracting information
[ComedyCentral] b502b41e-ed00-11e0-aca6-0026b9414f30: Downloading video urls
WARNING: Unable to download XML: <urlopen error [Errno 1] _ssl.c:503: error:140770FC:SSL routines:SSL23_GET_SERVER_HELLO:unknown protocol>
[download] Downloading playlist: Indecision 2000 - 2000 GOP Debate
[ComedyCentral] playlist Indecision 2000 - 2000 GOP Debate: Collected 0 video ids (downloading 0 of them)
[download] Finished downloading playlist: Indecision 2000 - 2000 GOP Debate

Outdated python.

Python is up to date.  Any other ideas?

No, it does not.

Python version 2.7.2


Feature request: allow filtering results with minBytes

Sometimes the linguist-js reports languages that have very few bytes. Could it be possible to add a new option to filter by minimum number of bytes for example:
linguist-js --minBytes 100

Could maybe do a bit more than that and allow something like linguist-js --min 5b / 7kb / 2%, say?

I've made a possible implementation of this feature - however I'm not 100% on the method I've chosen so feedback over at #35 would be welcome.


Add eccentricity parameter to fluid flow

The user will no longer provide the position of the rotor as
argument to instantiate fluid flow. Instead, they will provide
the eccentricity, which is the distance between the stator and
rotor centers. The rotor will always be placed at -45 degrees,
and theta will start counting from this point, as commonly seen
found in literature. Close #82

Codecov Report

Merging #83 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #83   +/-   ##
=======================================
  Coverage   57.17%   57.17%
=======================================
  Files           8        8
  Lines        1401     1401
=======================================
  Hits          801      801
  Misses        600      600

Continue to review full report at Codecov.

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


[Bug] Seems rectangly has it's width and length swapped for world edit commands?

Not sure how to go into a lot of detail here... When I use InfinityHammer with WorldEdit and utilize the hoe, it seems that rectangle measurements are ignored, instead treating it like a square using the width as an absolute.
A picture example:

Thanks, fix is coming today.

2991e84e4647ab8150c40b612ef956b04375c32d seems to have corrected the issue, closing.


Underline links in dropdowns

It can be hard to tell what link you're hovering on in the dropdown. Can we add the underline we use on tertiary links to add a bit of clarity?

cc: @deadlybutter

need me to work on this?

Yes please! :art:

hmm, shouldnt this be an issue at dosomething? All I need to do is add the -tertiary class to the list elements

Works for me.

apparently that doesn't work.... time to add it in neue!  ( @DFurnes can you reopen this? I haz no perms)

:dancer:


chore(coral): Update all unified related packages.

Linked issue
Resolves: #xxxxx
What kind of change does this PR introduce?

[ ] Bug fix
[ ] New feature
[ ] Refactor
[ ] Docs update
[ ] CI update
[x] deps update

Description
Updated unified and all related package we use for our markdown documentation.
I tested the functionality and it works fine 🎉

Hahaha got tired of the dependabot spam?

😬  No, I had a reminder for those, because they can't be updated separately. Got reminded of my reminder by dependabot this morning though, that's true :D


Cleanup extern SDL_Window* underlyingWindow;

There are a number of places with a comment about a hack, and then the following line of code:
extern SDL_Window* underlyingWindow;

We should clean these up, and find a standard way of accomplishing what these lines are used for.
For OPHD, these uses are tied to the popup error dialog boxes.
Related references:
SDL_GetWindowWMInfo
MessageBoxA

As part of this, we should clean up the include that occurs very late in Common.cpp (line 205):
#if defined(WINDOWS) || defined(WIN32)
#include <SDL2/SDL_syswm.h>

It's truly a hack. My biggest concern was exposing implementation details (which this does). We could hide it behind a void* and require a cast when it's used. I think it may also be used in the EventHandler for grabbing/warping the mouse? But I don't completely remember.
Anyway, it's bad and I know it (hence the comment).
As for the comment, I wanted to keep that close to where it's used though it's used for pop up message boxes which I ultimately want to make an in-game thing instead of using the system message boxes as the system boxes don't play nice with fullscreen modes (and sorta breaks the look/feel).

I would like to avoid casts where possible. Instead, we should perhaps look at adding functionality to NAS2D so we don't need access to this internal variable.
I checked the code more closely, and there are 3 instances of extern SDL_Window.

Used by HWND WIN32_getWindowHandle(), which is further used to generate modal popup boxes
Used by bool windowMaximized() to determine if the main window is maximized
Used by main to maximize the main window (if the maximized option is set)

There didn't seem to be a concept of a main window exported by NAS2D that was used anywhere. That might be one area to start with. I assume it would be represented by a Window object. Assuming yes, we can add methods to query and set the maximized state.
The modal popups might be a bit harder to deal with. I suspect the real solution would be to use in game dismissable (non-modal) messages. We would likely need to update GUI objects to support such messages. The messages would then be presented through normal UI objects, and so we shouldn't have need of the Windows HWND handle.
I think I might de-prioritize the dismissable messages until other UI work can be completed. We should probably focus on updates to the event handling, and layout manager first.

Adding reference to related upstream issue:
https://github.com/lairworks/nas2d-core/issues/701

I ultimately want to get rid of the modal dialog boxes which would make a Window object a concept that isn't really used in NAS2D. It's a case of 'using undocumented functions and/or features' using internal knowledge of how NAS2D handles things internally. It's prone to break should the underlying implementation change.
In OPHD's use of modal message boxes will be dropped in favor of non-native implementations (e.g., message boxes handled by OPHD's internal UI code). Those are there specifically to handle getting rid of the voice prompts that weren't useful in a quick way to get a release out the door. This is also the reason fullscreen mode was disabled in OPHD, something I want to reintroduce.
bool windowMaximized(); is only ever used in the WindowEventWrapper class, specifically in void onWindowResized(int w, int h). The check where it's used can probably be removed. If a window is maximized, we can store whatever values are in there, it doesn't actually matter because the window gets resized.

So the function bool windowMaximized(); can be safely removed. The WindowEventWrapper is there as a kludge -- the NAS2D Delegate class is no longer to bind to stand-alone functions. Don't know why -- this wasn't an issue before it was modified. I'm sure there's a way to do this without using a class as a wrapper but that's what it's there for.

I would be happy to see more integrated non-modal popup messages.
With that said, I think we could still make use of a Window class. At the very least, it would help improve the API. Currently the Renderer mixes quite a lot of different functionality together into one class. The Renderer has been pared down quite a bit, but it's still quite large. I think more of it should be separated out.
In particular, I think a Window class would make a better home for:

setting a window title
setting a window icon
setting/unsetting maximized
setting resizeability
setting minimum window size
setting window size
resize event notifications
input notifications (mouse, keyboard, touch, etc.)

The Renderer might represent a viewport within a window, which might very well take up the entire client area of the window. It would contain all the "draw" methods.
A few methods are a bit harder to class. Querying the desktop resolution seems more closely related to the Window than the Renderer, though it's not really a perfect fit since the desktop contains the Window, not the other way around. Perhaps it can be modeled as a desktop window, or perhaps it might evolve into yet another class.

None of this should be confused with the current "Window" class, which is perhaps more like a Container within a window. The actual OS window, along with a drawing viewport is currently combined into the Renderer object.

Okay, I think I'm board here. Makes sense to me.


start of process of making lib usable elsewhere

I'd like to try and make the core functions usable as a client lib which means moving the accessing of the screen recording code outside of the WhipClient itself.
go.sum changes are just from my go mod tidy...
Made it so that it errored out if you don't get a 201 to the original POST.
Bit of a work in progress but didn't want to hold back from this first PR.
What do you think?

This is perfect, thank you very much @danjenkins !   I can merge it now.


Remove OLM and marketplace from standard installation

Change
Currently, monitoring is removed, we could do the same with OLM.
OLM is the greatest memory consumer after Kubernetes itself.
How
It is included in SriKrishna's PR https://github.com/code-ready/snc/pull/280.
Expected and mesured improvements
Pros
Cons

According to Praveen, first release of crc didn't include OLM but many users need it.

We should keep the OLM and marketplace on by default. One of the main use-cases for CRC is to work with operators. Either to use them as dependencies or to test ardevelop them, both requires OLM.
If we can have an easy way to disable it for users we should consider at that.

If I remove OLM and marketplace operator, I win 1G of RAM. (6395 vs 5289M)


Develop

Thanks for the plugin. I've updated some of the docs to remove code deprecated in cake4.

Great, can you target this at the cake4 branch please. The develop branch is for Cake 3

Thanks very much!


[Bug?] Meshes imported as 3D mesh (.obj) have visible UVs

Test blend file, exported .obj file and test Godot project are here:
https://www.dropbox.com/s/gki3a4l73ywhgsf/VisibleUVs.zip?dl=0
It's a simple unwrapped sphere exported to obj. UVs aren't visible only if option Include UVs isn't checked in Blender exporter.

Yes, this is unavoidable. Try exporting normals to your OBJ file and tell Godot to not generate them and it will work


Segmentation Fault when Compiling with LTO

The Bareflank build segfaults when using LTO to build the hypervisor. The error occurs when building libvmcs.so
Debugging with GDB reveals that in llvm/lib/Transforms/Utils/ValueMapper.cpp: line 140 a nullptr value is passed to in a cast operation, and a nullptr dereference eventually triggers the segmentation fault. This occurs due to a bug in MapValue() called in line 117 of ValueMapper.cpp, which has been addressed in the upstream releases for LLVM.
I'm not sure if we can pull changes from the upstream version, as there is a great deal of additional code between the 3.8 and 3.9 versions, including many new data structures and a refactor of the algorithm. I am checking to see if LTO will work using the 3.9 version of vanilla clang/llvm with Bareflank, and will update when testing is complete.
I have included a core dump for those interested. If you would like to reproduce the error, check out the fix_lto branch of the securesystemslab/hypervisor-super project and follow the directions on the branch's README.md. (Do note that the branch is using the Debug version of the multicompiler, which is quite large. I can provide a zipped version of the debug cross compiler built on Ubuntu, but even that is 5.5GB) You can change the build configuration from Debug in 'scripts/build_multicompiler.sh' and scirpts/build_gold.sh
To pass the correct LTO flags into the build, invoke make with:
$ CROSS_CXXFLAGS="-flto -v" make
To debug the core file use :
$ gdb /path/to/multicompiler/ld core
or more accurately for Bareflank:
$ gdb ~/compilers/clang_38/x86_64-elf-ld core
core.zip

Great. Could you share the link to the issue(or commit log) that has fixed this error?

This commit seems to be the critical fix for the error.
https://github.com/llvm-mirror/llvm/commit/d10baa15d2039f2663af4cb5baf0aabdae35348f
Its line 432 in this blame.
https://github.com/llvm-mirror/llvm/blame/release_39/lib/Transforms/Utils/ValueMapper.cpp

Thanks! I read the issue.
IMO, although we cannot automatically apply this patch to our 3.8 based multicompiler due to the different code bases, we can still fix this error by manually applying the same "solution" they used (i.e., handling the null when RF_NullMapMissingGlobalValues is set) with little effort.
We may need to version up multicompiler in the near future, but for now, fixing it seems to be good enough, I think.


Interactive Tutorial - Deploying an App

Hello;
the interactive terminal on the website https://kubernetes.io/docs/tutorials/kubernetes-basics/deploy-app/deploy-interactive/
is not working , its blank page , I am using google chrome

If you are using ad blocker, try disable it and reload the page.

@rayarazan It's working fine could you please recheck  at your end,

This might be similar to issue #32832
@rayarazan do you think this might have been caused by an ad blocker tool?

/close
Won't fix because we're removing Katacoda.


🐛 [BUG] LayerPopup控制台提示报错

🐛 Bug description [Please make everyone to understand it]

Please provide a link by forking these links LarkMap or GitHub repo, a minimal reproduction.

Required Link to minimal reproduction:

📷 Step to reproduce
🏞 Expected result
🚑 Any additional [like screenshots]

LarkMap Version:
Platform:

和 https://github.com/antvis/LarkMap/issues/192 问题一致，统一修复