Wrong path response structure in docs when using OpenAPIv3

Hi,
First of all, thank you so much for this awesome extension! It really fullfills what I need.
I am just facing a little problem. At first i thought it was because i was using a application factory, but then i tried this simple code and I could not generate the openapi.json.
It throws the error TypeError: Object of type 'PetSchema' is not JSON serializable.
It sources back to
File "/flask_app/ext/flask_rest_api/spec/__init__.py", line 78, in _openapi_json
 def _openapi_json(self):
        """Serve JSON spec file"""
        # We don't use Flask.jsonify here as it would sort the keys
        # alphabetically while we want to preserve the order.
        return current_app.response_class(
            json.dumps(self.to_dict(), indent=2),
            mimetype='application/json')

Basically  json.dumps fails because it is passed the Marshmallow schema.
Could you please tell me what I do wrong?
Merci beaucoup !!!
app.py:
import marshmallow as ma
from flask import Flask
from flask.views import MethodView

from flask_rest_api import Api, Blueprint

from .config import Config

app = Flask('My API')
app.config.from_object(Config)
api = Api(app)

class Pet:
    pass

api.definition('Pet')

class PetSchema(ma.Schema):

    class Meta:
        strict = True
        ordered = True

    id = ma.fields.Int(dump_only=True)
    name = ma.fields.String()

class PetQueryArgsSchema(ma.Schema):

    class Meta:
        strict = True
        ordered = True

    name = ma.fields.String()

blp = Blueprint(
    'pets', 'pets', url_prefix='/pets',
    description='Operations on pets'
)

@blp.route('/')
class Pets(MethodView):

    @blp.arguments(PetQueryArgsSchema, location='query')
    @blp.response(PetSchema(many=True))
    def get(self, args):
        """List pets"""
        return Pet.get(filters=args)

api.register_blueprint(blp)

config.py
API_VERSION = '1.0.0'
OPENAPI_VERSION = '3.0.0'
OPENAPI_URL_PREFIX = "/apidoc/"
OPENAPI_JSON_PATH = 'openapi.json'
OPENAPI_SWAGGER_UI_PATH = "swagger-ui"
OPENAPI_SWAGGER_UI_VERSION = '3.18.2'

requirements.txt
Flask==1.0.2
flask-rest-api==0.7.0
apispec==0.39.0
itsdangerous==0.24
Jinja2==2.10
MarkupSafe==1.0
marshmallow==2.15.4
PyYAML==3.13
webargs==4.0.0
Werkzeug==0.14.1

Nevermind, I tried with changing the version to OPENAPI_VERSION to '2.0' and it now works!
:smile:

Hi. Thanks for the feedback and the bug report with the test case to reproduce the issue. This is really helpful.
I reworked it a bit.
import marshmallow as ma
from flask import Flask
from flask.views import MethodView

from flask_rest_api import Api, Blueprint

class Config:
    API_VERSION = '1.0.0'
    OPENAPI_VERSION = '3.0.0'
    OPENAPI_URL_PREFIX = "/apidoc/"
    OPENAPI_JSON_PATH = 'openapi.json'
    OPENAPI_SWAGGER_UI_PATH = "swagger-ui"
    OPENAPI_SWAGGER_UI_VERSION = '3.18.2'

app = Flask('My API')
app.config.from_object(Config)
api = Api(app)

class Pet:
    pass

@api.definition('Pet')
class PetSchema(ma.Schema):

    class Meta:
        strict = True
        ordered = True

    id = ma.fields.Int(dump_only=True)
    name = ma.fields.String()

class PetQueryArgsSchema(ma.Schema):

    class Meta:
        strict = True
        ordered = True

    name = ma.fields.String()

blp = Blueprint(
    'pets', 'pets', url_prefix='/pets',
    description='Operations on pets'
)

@blp.route('/')
class Pets(MethodView):

    @blp.arguments(PetQueryArgsSchema, location='query')
    @blp.response(PetSchema(many=True))
    def get(self, args):
        """List pets"""
        return Pet.get(filters=args)

api.register_blueprint(blp)

from pprint import pprint
pprint(api.spec.to_dict())

It is clear from the pprint output that the marshmallow Schema is not resolved.
This is indeed an issue in flask-rest-api.
In OpenAPI v2, the docs read
'responses': {200: {'description': 'Description', 'schema': ... }},

In OpenAPI v3, they read
'responses': {200: {'description': 'Description', 'content': {'application/json': {'schema': ... }},

When using apispec, it is up the the user to use the correct structure in the YAML docstring. Then, apispec being aware of this, it looks in the right place when resolving schemas depending on the OpenAPI version.
When using flask-rest-api, this structure is written by the response decorator. Currently it does things the OpenAPI v2 way, so it is broken for OpenAPI v3. I didn't realized that until now.
This is not trivial to solve as Blueprint can't know the OpenAPI version when its decorators are executed. We'd need to defer the writing of the documentation structure to register_blueprint, when the app object is available.
This needs a bit of refactor. We'd need to store all information is a known internal structure until calling register_views_in_doc where the documentation structure would be created according to the OpenAPI version.

Also, there's a mistake in your code, the @ is missing before the definition decorator, so the call to the decorator is a no-op.
I fixed it in my version above.
I guess it happened when you minified the example. Anyway, this is not the cause of the problem.

I believe this is fixed in master (not released yet).
Thanks again for the comprehensive bug report.

Fix released in 0.8.0.

Same issue with parameters.
Working on it.

Documentation of requestBody for OpenAPIv3 fixed in master. Not released yet.


How to HTTP Post a File in WinRT?

I am having a hard time understanding how to make an HTTP post request in WinRT. I have an ASP.Net 6 Web Api, which I have already been able to access via a Python project and a C++ project (via Curl). The api can also be tested via Postman.
But now I have a WinRT project and have thrown together the following code. I want to be able to upload a file. In my case it is a point cloud in .ply format.
My Question:

Can someone tell me if this is the right way? Especially the lines
binaryContent.Headers().Append(L"Content-Type", L"image/jpeg"); and
HttpContentDispositionHeaderValue disposition{ L"form-data" };

    void HL2ResearchMode::SendPLY(std::wstring const& pointCloud)
    {
        OutputDebugString(L"--- SendPLY()\n");

        if (pointCloud.size() == 0)
            return;

        init_apartment();

        auto buffer{
            winrt::Windows::Security::Cryptography::CryptographicBuffer::ConvertStringToBinary(
                pointCloud,
                winrt::Windows::Security::Cryptography::BinaryStringEncoding::Utf8
            )
        };

        winrt::Windows::Web::Http::HttpBufferContent binaryContent{ buffer };
        // binaryContent.Headers().Append(L"Content-Type", L"text/plain;charset=utf8");
        binaryContent.Headers().Append(L"Content-Type", L"image/jpeg");

        winrt::Windows::Web::Http::Headers::HttpContentDispositionHeaderValue disposition{ L"form-data" };
        //winrt::Windows::Web::Http::Headers::HttpContentDispositionHeaderValue disposition{ L"multipart/form-data" };
        binaryContent.Headers().ContentDisposition(disposition);
        disposition.Name(L"fileForUpload");
        disposition.FileName(L"test.ply");

        winrt::Windows::Web::Http::HttpMultipartFormDataContent postContent;
        postContent.Add(binaryContent);

        winrt::Windows::Web::Http::HttpResponseMessage httpResponseMessage;
        std::wstring httpResponseBody;

        try
        {
            // Send the POST request.
            winrt::Windows::Foundation::Uri requestUri{ L"https://<IP_ADDRESS>:5001/api/meshes/uploadPointCloud" };
            winrt::Windows::Web::Http::Filters::HttpBaseProtocolFilter myFilter;

            auto fu = myFilter.IgnorableServerCertificateErrors();
            fu.Append(ChainValidationResult::Expired);
            fu.Append(ChainValidationResult::Untrusted);
            fu.Append(ChainValidationResult::InvalidName);
            fu.Append(ChainValidationResult::InvalidSignature);
            fu.Append(ChainValidationResult::InvalidCertificateAuthorityPolicy);

            winrt::Windows::Web::Http::HttpClient httpClient(myFilter);
            httpResponseMessage = httpClient.PostAsync(requestUri, postContent).get();
            httpResponseMessage.EnsureSuccessStatusCode();
            httpResponseBody = httpResponseMessage.Content().ReadAsStringAsync().get();
        }
        catch (winrt::hresult_error const& ex)
        {
            httpResponseBody = ex.message();
        }
        std::wcout << httpResponseBody;
    }

What are the follow lines for? Is this about assigning arbitrary names?

disposition.Name(L"fileForUpload");
disposition.FileName(L"test.ply");

Hi, @arsi0001, and thanks for filing this issue.
For feedback about missing, incorrect, or incomplete info in a specific topic, you can visit the topic and then Send feedback about > This page.
For questions/feedback/support about the platform (Windows and its APIs) or tools, you can click Send feedback about > This product, and you'll be taken to Microsoft Q&A (more info about that below).
You can use the info you've provided above with that product-feedback channel.
Here are all of the developer support resources, for completeness.

Microsoft Q&A is the right place to lobby for a change or a new feature, to make a suggestion, or to ask a product question.
For community assistance with development questions, there are our forums, and StackOverflow.
The Windows developer support site explains other support options.
The Visual Studio developer community is available, and responsive. The former User Voice feature is now there on the Suggest a feature tab, if you want to lobby for a change, a feature, or to make a suggestion.
For developer issues, or suspected product bugs, there's Microsoft Support; a support engineer will look at the repro and see whether the issue is a Microsoft product bug, or something else. If it's a bug with a Microsoft product then the ticket will be free of charge.

I'll go ahead and close this issue now, since it doesn't look like we're identifying any documentation that's wrong. You can always reply to a closed issue, or even re-open it if you feel it's been closed in error.
Thanks!
-Steve


Filter possible items for recommendation

Currently, the only possibility to filter items for a recommendation is the use of categories. Only one category can be used as a filter.
I.e. this scenario in an e-com shoe store is not possible: "Recommend shoes from category sneaker with colors black, white or gray."
I think it would be helpful to have the opportunity to filter by multiple categories, as well as by certain labels.

It could be implemented in 0.5.x


Update Discography “fill-murrays”

Automatically generated by Netlify CMS

:construction_worker: Deploy Preview for thar-ai processing.
:hammer: Explore the source changes: 9b964960a1a0b67c3dd40b292686aae3ce10d173
:mag: Inspect the deploy log: https://app.netlify.com/sites/thar-ai/deploys/60c61b00e530a70008a4199e


text tokenizer for beitv3?

Describe
Model I am using (UniLM, MiniLM, LayoutLM ...):
the tokenizer for visual image is using beitv2:
https://github.com/microsoft/unilm/blob/master/beit2/test_get_code.py
but the tokenizer for text is not mentioned?

Hi @PanXiebit Please refer to beit3.spm in the setup.

https://github.com/microsoft/unilm/blob/master/beit3/README.md#text-tokenizer

@donglixp @wenhui0924  Thanks, I'm now able to get tokens for text, but I'm having trouble with tokenizers for images.
I tried to deal with vision-language tasks, and then used the pre-trained model of "beit3_large, beit3_large_patch16_224.pth". I ran through test_get_code and got accurate results.
But three image tokenizer models are provided in beit2 TOKENIZER and I can't determine which image tokenizer model is used by beit3_large?

hi, I try to use beit-v3 to get the image cls embedding and text cls embedding, and then computer the spherical_dist_loss between them.
The prompt is "a fish on a bike", and the image is here

but the results is 1.2226.
And then I test two random vector?  the distance is also 1.2226. This is strange, could you give me some suggestions?
from torchscale.model.BEiT3 import BEiT3
from model_lib.beitv3.beit3.modeling_utils import _get_base_config, _get_large_config
import torch
import collections

import torch
import torch.nn.functional as F
from torchvision import transforms as pth_transforms
from timm.models import create_model
from model_lib.beitv3.beit2.modeling_vqkd import vqkd_encoder_base_decoder_1x768x12_clip
from PIL import Image

IMAGENET_INCEPTION_MEAN = (0.5, 0.5, 0.5)
IMAGENET_INCEPTION_STD = (0.5, 0.5, 0.5)

def init_beit(cktp_path="model_lib/beitv3/beit3_large_itc_patch16_224.pth"):
    args = _get_large_config()
    model = BEiT3(args)

    ckpt_state_dict = torch.load(cktp_path)["model"]
    new_ckpt_state_dict = collections.OrderedDict()
    for name, param in ckpt_state_dict.items():
        new_name = name.replace("beit3.", "")
        new_ckpt_state_dict[new_name] = param
    model.load_state_dict(new_ckpt_state_dict, strict=False)
    return model

def image_process(img_path, input_size=224):
    transform = pth_transforms.Compose([
            pth_transforms.Resize((input_size, input_size), interpolation=3),
            pth_transforms.ToTensor(),
            pth_transforms.Normalize(mean=IMAGENET_INCEPTION_MEAN, std=IMAGENET_INCEPTION_STD)
        ])
    print(f"Image transforms: {transform}")

    images = transform(Image.open(img_path).convert("RGB")).unsqueeze(0)
    return images

def spherical_dist_loss(x, y):
    x = F.normalize(x, dim=-1)
    y = F.normalize(y, dim=-1)
    return (x - y).norm(dim=-1).div(2).arcsin().pow(2).mul(2)

if __name__ == "__main__":
    from transformers import XLMRobertaTokenizer
    tokenizer = XLMRobertaTokenizer("model_lib/beitv3/beit3.spm")
    prompt = "a fish on a bike"
    text_tokens = tokenizer(prompt, return_tensors="pt")
    print("text_tokens: ", text_tokens.input_ids.shape)

    image_path = "model_lib/ldm/clip_guidance/a fish on a bike_sd.png"
    image_tokens = image_process(image_path)
    print("image_tokens: ", image_tokens.shape)

    beit3_model = init_beit()
    encoder_out = beit3_model(textual_tokens=text_tokens.input_ids,
        visual_tokens=image_tokens)

    multiway_split_position = encoder_out["multiway_split_position"]
    out = encoder_out["encoder_out"]
    vision_cls = out[:, 0, :]
    language_cls = out[:, multiway_split_position, :]
    print(vision_cls.shape, language_cls.shape)

    loss = spherical_dist_loss(vision_cls, language_cls)
    print("loss: ", loss) # sd: [1.1729], midjourney: [1.2337]

    # two random vectors
    test_a = torch.randn(1, 1024)
    test_b = torch.ones(1, 1024)
    loss = spherical_dist_loss(test_a, test_b)
    print("loss: ", loss)

The results are:
text_tokens:  torch.Size([1, 7])
Image transforms: Compose(
    Resize(size=(224, 224), interpolation=bicubic, max_size=None, antialias=warn)
    ToTensor()
    Normalize(mean=(0.5, 0.5, 0.5), std=(0.5, 0.5, 0.5))
)
image_tokens:  torch.Size([1, 3, 224, 224])
torch.Size([1, 1024]) torch.Size([1, 1024])
loss:  tensor([1.2226], grad_fn=<MulBackward0>)
loss:  tensor([1.2203])

You could follow https://github.com/microsoft/unilm/blob/master/beit3/get_started/get_started_for_retrieval.md for embedding usages.


Support Azure AD for China region

Issue:
https://github.com/rancher/rancher/issues/47251
Problem
The way we generate graph client uses an empty baseUrl for request adapter, which will be set as Global Azure Graph API as default. This will cause failure when getting resource information from Azure China.
Solution
The graph client uses the global graph API as the base url. To support custom graph API such as China region, we need to customize the base url when generating the client.
I checked issues from Microsoft Graph client SDK and found this issue. We need to set the correct base URL for different graph API.
We also need to improve the error message described here instead of just showing the error message like error status code received from the API which can’t give us more information such as response status code and the real error message.
Testing

Engineering Testing
Manual Testing

Register Rancher with Azure China and set the Redirect URI to <MY_RANCHER_URL>/verify-auth-azure.
Create a new client secret and save it.
Set required API permissions for Rancher.
Allow Public Client Flows.
Copy Azure China application data to the forms in Rancher AzureAD authentication provider. The endpoints should be selected for China.
Click Save and login to Azure account from the popup window.
The Azure AD auth provider becomes active.

Automated Testing

Test types added/modified:

Unit(Add TEST_AZURE_CHINA environment)

QA Testing Considerations

Regressions Considerations
We need regression testing for both Global and China Azure AD auth.

Validated on v2.9.5-alpha1 successfully. For detailed test content, please refer to the issue comment.


chore: standardize prettier configuration

use @ericcrosson/prettier-config
remove superfluous npm run-scripts
run prettier on all files

:tada: This PR is included in version 1.0.5-beta.1 :tada:
The release is available on npm package (@beta dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.2-beta.1 :tada:
The release is available on npm package (@beta dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.2.6-beta.1 :tada:
The release is available on npm package (@beta dist-tag)
Your semantic-release bot :package::rocket:


add reactjs resource

Added a ReactJS course

Please review

Hey, it seems to be conflicting with the main branch. would you look after this issue.
Thanks a lot for your contribution.
Looking forward for more such resources which helps others and make the community grow. ❤

i will surely do

please check @debrajhyper

Thanks a lot @swatiaggrawal for your contributions.
Looking forward for more contributions. ❤


[BUG/Help]  windows service 2022 环境下无法支行train.bat,报下面异常

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior

Expected Behavior
No response
Steps To Reproduce
执行 train.bat
Environment
- OS: Windows Service 2022
- Python:3.11
- Transformers:
- PyTorch:
- CUDA Support (`python -c "import torch; print(torch.cuda.is_available())"`) :

Anything else?
nothing

会一直创建临时目录，手动创建了提示的临时目录，再执行train.bat,又会报新临时目录 不存在，请部这些是不是死循环了？


About precise RoI pooling

Hello, it seems mmdetection does not support Precise RoI pooling yet. Do you have a plan for that?
If I have to implement the Precise RoI pooling myself, is there any suggestion or guidance?
Thank you.

You can migrate it from https://github.com/vacancy/PreciseRoIPooling.
Some researchers tried it before but the op itself does not boost the performance over RoI Align. Other components in IoU-Net are required to work with it.

You can migrate it from https://github.com/vacancy/PreciseRoIPooling.
Some researchers tried it before but the op itself does not boost the performance over RoI Align. Other components in IoU-Net are required to work with it.

The concept of "Precise ROI Pooling" is equivalent to ROIAlign with sampling_ratio=0, which is in fact already supported by the ROIAlign op.

The concept of "Precise ROI Pooling" is equivalent to ROIAlign with sampling_ratio=0, which is in fact already supported by the ROIAlign op.


Set year ranges or max/last year for viewMode: years

I cannot find anywhere in the documentation how to set the max year or a custom year range on the viewMode: 'years' option.
I am using a setup like this:
$('#date-input').datetimepicker({
    viewMode: 'years',
    maxDate: moment()
});

However, all the maxDate option does is prevent you from clicking a year in the future, it doesn't hide them.
This presents a dialogue like this:
http://i.stack.imgur.com/2hZbe.png
However, it should only show up to the current year (2016) so that more past years would fit on the page. In other words, the default range to go from 2005-2016 instead of from 2011-2022.

Sorry, but no. Any disabled date (including year selection) have a class of disabled. If you want to completely hide or remove them, it's up to you to implement that.


cannot move location counter backwards (from<PHONE_NUMBER>018958 to<PHONE_NUMBER>008000)

Hi, there,
I'm trying to use the provided toolchain to reproduce the experiments. Like the tutorail projects, I cloned the projects using command "./cloner.sh -s Template-USBCDCH743FUZZER -d ./FuzzTestH743", and then I opened this  projects in my own STM32CubeIDE 1.16.0. And I configured the toolchain  named "gcc-arm-none-eabi-10.3-2021.12-0x24000000". And finally, I configured the GCC compiler with optional options "-fno-common", ""-fsanitize=address", "-fsanitize-coverage=trace-pc".  And I canceled the "fcyclomatic-complexity" that provided by STM32's toolchain.
After the above operations, I tried to build the tutorial projects, and got following information. Could you please help me with it?
gcc-arm-none-eabi-10.3-2021.12-0x24000000/bin/../lib/gcc/arm-none-eabi/10.3.1/../../../../arm-none-eabi/bin/ld:../STM32H745ZITX_FLASH_MPU_ASAN.ld:95 cannot move location counter backwards (from<PHONE_NUMBER>018958 to<PHONE_NUMBER>008000)
collect2: error: ld returned 1 exit status

Hi, I will check what is going on. Meanwhile, I noticed you are cloning the H743 template but the error message mentions the H745, those are different MCUs, are you sure you are following the instructions as detailed in the video?

I reproduced the steps described in the readme for the latest CUBEIDE 1.17.0 and it works as expected.
It does require to disable the fcyclomatic-complexity option as you also mentioned but nothing else.
I noticed that you " configured the GCC compiler with optional options "-fno-common", ""-fsanitize=address", "-fsanitize-coverage=trace-pc"  However,  notice that that configuration is done only for the test.c file and not for the entire project.
Also, after cloning the template with the provided instructions you do not need to configure anything else in your IDE besides the toolchain. The template is already configured with the "-fno-common", ""-fsanitize=address", "-fsanitize-coverage=trace-pc" for the test.c file.

Thank you very much! After screen these steps again, I find I mistakely set "-fno-common", ""-fsanitize=address", "-fsanitize-coverage=trace-pc" for the whole projects, which cuased the error. And now, I can build the projects.


Fix flaky RuntimeWarning during array reductions

This is sort of a shot in the dark. We've got some (pretty rare) flaky RuntimeWarning: invalid value encountered in reduce warnings being emitted in test_reductions_2D (see full traceback below). The warning occurs when we're determining the output meta dtype using np.empty. This PR suggests we use np.ones instead of np.empty to make things more deterministic. Again, not totally sure if this actually solves the flaky warning, but it might, and the change here seems harmless anyways.

Okay, so I've run CI 6 times here and haven't seen the test_reductions_2D failure. It's a pretty rare failure, so maybe 6 times isn't enough, but the changes here seem harmless, regardless. Let's give it a try and I'll keep an eye out for test_reductions_2D failures


Core: Revise the async mechanisms

We have a number of parallel mechanisms in place for faster serialization and server communication.
We should make sure the appropriate async mechanisms is used in each case.
In particular:

calling .Result on a task will block the calling thread (used in a couple of places, make sure it's intended)
ensure (de)serialization context switching and waiting is done properly
never block the calling thread of operations.receive / etc  (might be UI thread) (issue described here: https://speckle.community/t/speckle-core-cant-receive-stream-data-on-specific-machine/2409/7 )

This ticket is a result of multiple related issues / questions.
We should also briefly document somewhere the parallelism involved and how it's done.

Modified a few .Result / .Wait with await, but there are still a few places, like the Transport sync method:
public string GetObject(string id)

Still has to block the calling thread and can deadlock UI calls
So for now, callers of the SDK, especially when calling methods from the UI thread, must call with Task.Run to switch to a background thread for the actual work


dialog_test.dart "Pressed button changes appearance and dividers disappear" broken on the Web

I enabled dialog_test.dart on the Web, but had to skip the test titled "Pressed button changes appearance and dividers disappear" as it fails a paint expectation. I couldn't figure out at a glance what's wrong. Leaving this issue to investigate in the future.
All other tests in dialog_test.dart are passing.

Still valid since the test is question is being skipped for web on latest master channel.
https://github.com/flutter/flutter/blob/a8281e31afa9dddfa0764f59128c3a2360c48f49/packages/flutter/test/cupertino/dialog_test.dart#L808-L919

This appears to have been fixed. I reenabled the test with #87538, and everything appears to be working fine.


[BUGFIX] Fix a bug in Auto Function.

Description
This PR is to fix this issue: https://github.com/apache/incubator-mxnet/issues/15183
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@mxnet-label-bot Add [Autograd. pr-awaiting-review]


Any plans to use URL rewrite middleware to redirect to https?

Sorry if this is the wrong place to post this question as this is not a real "issue" per se.
I'd like to know if you plan on adding the url rewriting middleware to redirect to https and canonical urls instead of the actual implementation, do you think it would be a useful thing to add?

There is one in ASP.NET Core 1.1. Redirect to Canonical URL middleware is planned. See the Projects tab, you can see it there.

Ahah yeah that's what I meant I was asking  if you planned on using it in your template.


Windows + GIT Bash: display issue

Steps to Reproduce:

Open VSCode
Run a cmd (with integrated GIT Bash, here yarn run xstart)
The result in the integrated terminal should be identical to external terminal.

Does this issue occur when all extensions are disabled?:
Yes
Please find attached two screenshots:
In the first one you can see a big white square of missing text:

In the second one you can see that the missing text is here (external terminal):

Question
Could you please advise on why some text is missing on integrated terminal ?

I'm seeing the same. Another effect of this that I've found is that running any command causes the terminal to scroll right to the bottom the first time it outputs a line (actually past the line it outputs). Might be a clue.
Other shells don't appear to be affected.

This is a bug in Windows 1809.
/duplicate https://github.com/Microsoft/vscode/issues/57803

#57803 was closed and "terminal.integrated.windowsEnableConpty": false did not change anything.

I've decided to update the #57803 comment as well as the saved reply I use, see below for more details on what I closed this.

This looks like an issue with our old terminal backend "winpty", going forward we are focusing our efforts on the new backend "conpty" which is build by the Console team at Microsoft and should fix many issues with the terminal under Windows. You can read more about getting ConPTY here
/duplicate https://github.com/Microsoft/vscode/issues/45693

FLY I am getting the same issue with either

"terminal.integrated.windowsEnableConpty": true,

or

"terminal.integrated.windowsEnableConpty": false,

@ArThoX what version of Windows are you on?

@Tyriar Windows 10 Pro 64-bits (build 17763), vscode 1.33.

conpty doesn't work on that build on Windows, see https://github.com/Microsoft/vscode/issues/57803#issuecomment-449495423


await missing from completions

PTVS.Dev.2015-05-28.VS.2015
async is there, but await is missing

It's there, but it doesn't show up as the first keyword on the line. Typing x = aw... will display it


On what Recipes should be included.

I have basically been accepting everything, including things I would not included under the umbrella of "based."
Now that we have "enough" recipes to make it a usable site, I want to be more restrictive in what is accepted.
Feel free to say your views on it.
On drinks and smoothies, there have been a couple smoothies or things like them and I frankly feel like they're a waste. That's not a real recipe. A "blueberry and banana smoothie" has its recipe in its name, more or less. Possibly some drinks might be allowed, but I am open to removing them altogether. I'm not sure if cocktails are really appropriate either, I mean I'm not necessarily against them, but I think a case can be made that it's not really relevant.
The only thing I have specifically not accepted is meat-substitute cringe which is an unambiguous case.
I will say that my own biases are to mostly not accept desserts.
Those of you who know me know that I am very anti-sugar and view it as the cause of all ills, but I suppose there could be based dishes with it.
New potential submission rules
These are to slim down submissions. Offer more suggestions.

Images must be included and must've been taken by the submitter. This encourages people to check their recipes one more time and not just submit a million things, but focus on what matters. I might say that images must be less than 150K.
Recipes should be Lindy (Lindy -> Based). That is, recipes should be longstanding folk recipes, not modern abominations. Most of what we have is fine, but some are border line.
People need to spend time to link redundancies. If a recipe is already present that shows you how to make boil pasta or some other ingredient, it should be linked in the recipe page.

On tagging
https://github.com/LukeSmithxyz/based.cooking/pull/240 Is a nice PR which I haven't merged that adds fasting information.
I will not be so cringe as to accept "diet" tags, but I think this is an acceptable and helpful thing.
The issue is whether new submitters can keep up with it.
I don't know if other people consider it a waste to have this, I might, just because you sort of know by looking at the ingredients what is acceptable, but I suppose it would be nice to have a tag or it.

I agree for most of this.
Most drinks as you point out probably don't need a recipe. I'd even go more general and say that if the name of the recipe fully explains it (like blueberry banana smoothie as an example), or something like a 'fruit salad' which is just combining whatever raw fruits you want, that we don't need a recipe for it.
I would personally like desserts, but you've stated from the beginning that you didn't want sugary things so I am okay with this.
I think requiring images would be good. I always find it useful to be able to see the end product and right now a lot of recipes are missing pictures.

I agree with most of what you have written up here except the part about images.
I know that having images makes the recipes a lot more clear to understand, and also makes them more presentable but most of my recipes are writen down in notebooks ive made through the years and when i travel.  It would be quite unfeasable for me to remake every recipe in my books just to share it.
I completely agree that overly sugary recipes, and things like the smoothie or alcoholic recipe you mention don't really have to be on the website since i don't think that that is the direction you'd like to take it.
On the topic of meat substitute recipes, i have collected a couple extremely delicious tofu recipes when i went to china, i'd consider these ok since they're traditional recipes and the tofu isn't used as a meat-substitute, its just another ingredient, but I'd like to hear what you guys think about these.

Good grammar and spelling should be a submission rule, if it isn't already imo. Also, how would you define a modern abomination?

Regarding drinks, I think they have a place on the site, but under the same requirements as the "normal" recipes, i.e. traditional and folksy is ok (like the Glühwein and maybe some teas, etc.).
Also I would personally like having basic/generic "recipes" for e.g. preservation methods and the like.
Other than that, I agree with quality over quantity.

Good grammar and spelling should be a submission rule, if it isn't already imo. Also, how would you define a modern abomination?

Sure it should be a rule, but I'm not going to be able to check it all. If I tell someone to fix an error, it'll will multiply the time I have to worry about the PR by more than 2.

On the topic of meat substitute recipes, i have collected a couple extremely delicious tofu recipes when i went to china, i'd consider these ok since they're traditional recipes and the tofu isn't used as a meat-substitute, its just another ingredient, but I'd like to hear what you guys think about these.

I am totally 100% fine with recipes with bean curd or anything else that might be used as a meat-substitute if it is indeed part of a traditional dish.

On drinks and smoothies, there have been a couple smoothies or things like them and I frankly feel like they're a waste.

Totally agree on smoothies. Maybe a single "this is the general method for making smoothies" page with a few different ingredients list suggestions could work, but yeah, ten different smoothies where ingredients are the title is spam.

Images must be included and must've been taken by the submitter.

I think a major problem with requiring images is that people are just not good at taking pictures (myself included), especially appealing pictures of food. The site would get filled with low contrast pictures of mush.

People need to spend time to link redundancies. If a recipe is already present that shows you how to make boil pasta or some other ingredient, it should be linked in the recipe page.

On the one hand, this sounds nice and DRY. On the other hand, even the most basic recipes often have variations and people have opinions on them. A recipe author might prefer a slightly different version of tomato sauce that his grandma always made rather than the one that's posted as a recipe here.
Requiring this sort of modularization also increases complexity - recipes become hyperlinked graphs instead of just a linear text.

Those of you who know me know that I am very anti-sugar and view it as the cause of all ills, but I suppose there could be based dishes with it.

I generally agree on anti-sugariness. I think the one case for desserts is similar to alcohol - in social settings.

It might be a good idea to request that ingredients aren’t measured exactly, so people can – to a certain extent – rely on their own judgement (wich is something you need if you want to cook without a recipe)
e.g.
a hand full of ...
five tablespoons ...
two teaspoons ...
This way imperial/metric confusion is not much of a concern anymore, and it’s easier to remember as well. (recipe independence lul)
I do realize this is not always ideal.

I'd argue that exact measurements can and are important in many cases (definitely not always). Cooking is literally chemistry for normal people.
Also a tbsp and tsp are imperial measurements. 1 tsp is roughly 5 mL.

It might be a good idea to request that ingredients aren’t measured exactly, so people can – to a certain extent – rely on their own judgement (wich is something you need if you want to cook without a recipe)

I like the idea, but i don't think it's enforceable.


Add logs to end-to-end tests

This PR pass around a Tracer in some of the HydraNode functions used for writing benchmark and end-to-end testing.
Only the waits are traced currently.

This is to help debugging the double-with-calls we just talked about in the daily ?

Yes, and possibly other troubles we might have later on running the tests. I debugged the issue by sprinkling putTextLn all over the place, so we might as well have proper logging in place.

Addressed comments


Change print to console.log in example code

When ran in the context of a browser, print() will call window.print() which opens a print dialog. It's very confusing pasting example code into the REPL only to have it open several print dialogs :)

The tour page is based on es6features, I don't really think it's a good idea to deviate from it in terms of changes. @thejameskyle thoughts?

In that case the example should also include a definition for the print function so people don't think there's a bug with the REPL where it tries to print the page.

Yeah I think this is a good change.


carla_nuscenes

What is the feature?
Hello, I am currently using the CARLA simulator to generate a dataset in the nuscenes dataset format. I would like to ask how to get the png map files in the maps folder in the nuscenes dataset. If I can get your answer, it will be very helpful to me, thank you.
Any other context?
I tried to use the top view in carla directly as a png map file, but there were some problems when generating a pkl file using the nuscenes devkit.

same question, have you solved this problem?


Adding @searchable directive to GraphQL models results in invalid instance type if instance type is set to t2.small.elasticsearch

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
19.6.0
Amplify CLI Version
12.1.1
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
Added AWS WAF and have added my own lambda functions that can be called through the graphql API, otherwise no.
Describe the bug
When I add @searchable with t2.small.elasticsearch to any @model it results in the SearchableStack failing to create. This applies to newly created models as well as already existing models.
Expected behavior
It should add the resource without error.
Reproduction steps

Add any model containing any data.  For example like this:

type Test @model {
id: ID!
name: String
}

Add @searchable directive to the model like this:

type Test @model @searchable {
id: ID!
name: String
}

run Amplify push and hit yes to every question

Will running the exact steps below in an empty directory create an Amplify project that demonstrates the bug behavior?

Not sure. Did not test in new project.
Project Identifier
52a5415a955eedae2497cf2f723fcd05
Log output

# Put your logs below this line

🛑 The following resources failed to deploy:
Resource Name: OpenSearchDomain (AWS::Elasticsearch::Domain)
Event Type: create
Reason: Invalid instance type: t2.small.elasticsearch (Service: AWSElasticsearch; Status Code: 409; Error Code: InvalidTypeException; Request ID: *some_id*; Proxy: null)

🛑 Resource is not in the state stackUpdateComplete
Name: OpenSearchDomain (AWS::Elasticsearch::Domain), Event Type: create, Reason: Invalid instance type: t2.small.elasticsearch (Service: AWSElasticsearch; Status Code: 409; Error Code: InvalidTypeException; Request ID: *some_id*; Proxy: null), IsCustomResource: false

Additional information
I am using region eu-north-1.
I solved my issue by using t3.small.elasticsearch instead of t2.small.elasticsearch like so:
"api": {
"name": {
"OpenSearchInstanceType": "t3.small.elasticsearch"
}
}
per the documentation for setting instance type for production environment.
It wouldn't surprise me if it weren't just t2 that doesn't work. Are there any others that may not work?
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @freesegways, Thank you for bringing this issue to our attention. We have tried to reproduce the problem by following the steps mentioned in the description, but we were unable to encounter the same issue. We were able to successfully modify the search instance type to t2.small.elasticsearch by updating the team-provider-info.json file as shown below:
categories": {
      "api": {
        "test": {
          "OpenSearchInstanceType": "t2.small.elasticsearch"
        }
      }

To further investigate the issue, could you please provide us with the failure logs? This will help us in understanding the problem better and assisting you with a solution.

Hi @freesegways  👋, We are closing this issue as we have not received any response from you. However, if you are still facing the same issue and require our assistance, please don't hesitate to leave a comment. This way, we can reopen the issue and provide you with better assistance.
Thank you!


chore(deps):  update geostyler-style to v9.2.0

:tada: This PR is included in version 6.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


errors: get rid of DvcLiveError calls.

DvcLiveError is an error that should be an abstract exception that lets end user know where it did originated.
We should replace current occurences of raise DvcLiveError with particular errors with naming related to the cause.

Hello, can I take this issue?

Hello, can I take this issue?

Sure!

Looks like this was fixed by https://github.com/iterative/dvclive/pull/135. Can we close this?


[BUG]: Repos[org][repo].Contents[path].GetAsync does not return file contents

What happened?
Calling the Repo[org][repo].Contents[path].GetAsync() function does not deserialize the response correctly.
In the HTTP response, there is a content key which should be deserialized, but the parser appears to look at content-file instead.
For now I am using a workaround by manually getting the request information and sending it to HttpClient, which provides the correct response.
Versions
GitHub.Octokit.SDK v0.0.23
.NET v8
Code of Conduct

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

Thank you for reporting.
I wonder if we have a broader pattern of schema errors with discriminator property names.

Hey, sure! Some rough code below

public class GitHubFileContentHttpClient(HttpClient httpClient)
{
    /// <summary>
    /// Sends a request for <see cref="WithPathItemRequestBuilder.WithPathItemRequestBuilderGetQueryParameters"/>
    /// </summary>
    /// <param name="requestMessage">Message after using <see cref="Microsoft.Kiota.Http.HttpClientLibrary.HttpClientRequestAdapter"/> ConvertToNativeRequestAsync</param>
    /// <param name="ct">Cancellation token</param>
    /// <returns>If the file was found, return <see cref="GetRepoContentResponse"/>. Else, return null</returns>
    /// <exception cref="Exception">Non success status code response, other than NotFound</exception>
    public async Task<(string, GetRepoContentResponse)?> GetFileContentAsync(
        HttpRequestMessage requestMessage,
        CancellationToken ct
    )
    {
        var rawRes = await httpClient.SendAsync(requestMessage, ct);
        if (rawRes.StatusCode == HttpStatusCode.NotFound)
        {
            return null;
        }

        if (rawRes.StatusCode != HttpStatusCode.OK)
        {
            throw new Exception(
            $"Expected success status code for {requestMessage.RequestUri} but received {rawRes.StatusCode}.");
        }

        var res = await rawRes.Content.ReadFromJsonAsync<GetRepoContentResponse>(cancellationToken: ct);
        if (res is null)
        {
            return null;
        }

        return (await httpClient.GetStringAsync(res.DownloadUrl, ct), res);
    }
}

To use

var kiotaFileRequestInformation =
    gitHubClient.Repos[orgName][repo.Name]
        .Contents[filePath]
        .ToGetRequestInformation(
            options =>
            {
                options.QueryParameters.Ref = branch.Name;
                options.Headers["accept"] = ["application/vnd.github+json"];
            });

var reqInfo = await requestAdapter.RequestAdapter
    .ConvertToNativeRequestAsync<HttpRequestMessage>(
        kiotaFileRequestInformation);

var fileContentResponse = await gitHubFileContentClient.GetFileContentAsync(reqInfo);
if (!fileContentResponse.HasValue)
{
    return;
}

I wonder if we have a broader pattern of schema errors with discriminator property names.

It could be the case, the OpenAPI spec file looks correct to me, so this could also be an upstream issue with Kiota

Hey @qin-guan we finally tracked this down - it has to do with the fact that our OpenAPI definitions for content does not have a discriminator in it to help it understand what, of the 4 types, should it be.
I'm working to get a PR in to change this in the GitHub OpenAPI definitions and will let you know when that has shipped.

@qin-guan Just a quick update.  I have created a PR for the GitHub OpenAPI descriptions that should fix this issue.  Once merged our auto generation should take over and update the the SDKs to fix the issue.  I'll let you know when that happens.

Hey @nickfloyd thanks for the updates!! So glad to see that a fix has been found, thank you! Happy to test it out once it's been released

Hey! Thanks for the updates yet again!! I look forward to seeing it out 🎉

This has been fixed and released: https://github.com/octokit/dotnet-sdk/releases/tag/v0.0.30


[Darwin] MTRCommandPath is missing (NSString *)description

Reproduction steps
While debuting an issue I noticed calling unlike MTRAttributePath, MTREventPath - MTRCommandPath doesnt overwrite the description method resulting in MTRClusterPath's description to be used.
Platform
darwin
Platform Version(s)
No response
Type
Unit tested
(Optional) If manually tested please explain why this is only manually tested
No response
Anything else?
No response

(lldb) po mtrPath.class
MTRCommandPath
(lldb) po mtrPath
<MTRClusterPath> endpoint 0 cluster 60


Filter and refresh does not update container height

Voila I filter my blocks, so some of them are removed, but the height of my container is not updated, even with event refreshWall().
Did I miss something ?

Sorry it must be my code because I made this codepen and the container height is dynamic when filtering...
http://codepen.io/migswd/pen/dPeedq


Document how to enable kroki.io as a diagramming service.

Will resubmit at a later stage. More work to be done.


Pool connection is never released (C++)

The following code hangs in OCISessionGet in the third iteration:
ocilib::Environment::Initialize(ocilib::Environment::Threaded);
ocilib::Pool connPool(db, user, password, ocilib::Pool::SessionPool, 1, 2);

for (int i = 0; i < 3; ++i) {
    ocilib::Connection connection = connPool.GetConnection();
}

The connections are never released to the pool. As far as I can tell, the Connection object is created as transient, and so the HandleDeleter is never called. But maybe I'm doing something wrong.

Brother, write your own thread pool. The performance of the built-in thread pool feels too low or always error-prone. I have given up using it. Write your own thread pool.

The code hangs in the 3rd iteration as by default, if the pool is exhausted, requesting a new connection blocks until one is available.

That's not the issue I was describing. The problem is that ~Connection does not release the connection back to the pool. Sample code using a connection also seems to rely on the destructor. Is it required to manually close the connection? If so, the documentation of Close should be clarified and state that a pooled connection is returned and not really closed.

Indeed, I missed that point.
I will investigate.
Regards,
Vincent.

Hi,
This is a bug introduced  by commit 8ad2fa43c8b9f5089fa3b26aa96f16b0f2eb20e5 in v4.7.5.
It is now fixed in v4.7.7 dev branch.
Regards,
Vincent


fix coredump when using "ovs-ofctl add-groups"

desc: When using ovs-ofctl add-groups with only "switch" argument, a
coredump is generated. The main reason is that the command "ovs-ofctl
add-groups" need two arguments but the limitation of min-args of this
command is set to 1.
Signed-off-by: Wang Yibo<EMAIL_ADDRESS>

Why my CI fails, I'm using a new fork repo to make this pull request.

Why my CI fails, I'm using a new fork repo to make this pull request.

At the time of a previous pull request there was no issues in FreeBSD CI.  But now there is one.  I sent a patch to fix it: https://mail.openvswitch.org/pipermail/ovs-dev/2021-May/382702.html .

Why my CI fails, I'm using a new fork repo to make this pull request.

At the time of a previous pull request there was no issues in FreeBSD CI. But now there is one. I sent a patch to fix it: https://mail.openvswitch.org/pipermail/ovs-dev/2021-May/382702.html .

Thanks for your reply. By the way, does my patch meet the merge requirements?

It's OK in general.  One thing is that commit Author field doesn't match with the Signed-off-by tag.  They should be the same.
$ git show 785f42ee8
commit 785f42ee8ae7e75d8f4b46eb61ec35d1005c7068
Author:     w00574186 <wangyibo***.com>
AuthorDate: Thu May 6 11:14:07 2021 +0800

    desc: When using ovs-ofctl add-groups with only "switch" argument, a
    coredump is generated. The main reason is that the command "ovs-ofctl
    add-groups" need two arguments but the limitation of min-args of this
    command is set to 1.

    Signed-off-by: Wang Yibo <bobxxwang***.com>

And a few nitpicks: Commit itself has no subject line, so it doesn't look pretty in the output of git log --oneline.  There should be a short subject line, empty line, and the rest of a description after that.  It's also a good idea to include a Fixes tag to the commit message to point to the commit that introduced the issue.  So, in the end, to be perfect, commit message should like something like this:
$ git show 785f42ee8
commit 785f42ee8ae7e75d8f4b46eb61ec35d1005c7068
Author: Wang Yibo <bobxxwang***.com>
Date:   Thu May 6 11:14:07 2021 +0800

    ovs-ofctl: Fix coredump when using "add-groups" command.

    When using ovs-ofctl add-groups with only "switch" argument, a coredump
    is generated.  The main reason is that the command "ovs-ofctl add-groups"
    need two arguments but the limitation of min-args of this command is
    set to 1.

    Fixes: 7395c05254df ("Implement OpenFlow 1.1+ "groups" protocol.")
    Signed-off-by: Wang Yibo <bobxxwang***.com>

OK, modified as required.

Applied as commit f173527ec8a42e79e0a322ebe4bed999ea9ba497 and backported down to 2.12.
Thanks!


Investigate thread contention on windows logfiles

Is there an existing feature request for this?

[X] I have searched the existing issues

Describe the Feature
in #4055 a single test failed because a process was already using the log file. I suspect this is because of how our tests are run and not because there is a glaring problem with dbt. It should be investigated though since windows tests might currently be non-deterministic with the new structured logging.
Describe alternatives you've considered
No response
Who will this benefit?
No response
Are you interested in contributing this feature?
No response
Anything else?


[Bug] Timeline step with at and duration not working

1. Describe the bug
Using the following code in which I am trying to make both timeline steps start at the same time but have the second one have twice the duration of the first one using at and duration is not working. I have tried using the at and duration parameters in different order but it hasn't helped.
`const item1= document.querySelector('.item-1')
const items2= document.querySelectorAll('.item-2')
const sequence = [
[item1, {transform: ['translate(0, 100px)', 'translate(0, 0)']}, {duration: 1}],
[items2, {opacity: [0, 1]}, {at: 0}, {duration: 2}]
// [items2, {opacity: [0, 1]}, {duration: 2}, {at: 0}]
]
timeline(sequence);`
2. IMPORTANT: Provide a CodeSandbox reproduction of the bug
https://codepen.io/offbeat-io/pen/VwxYjPj
3. Steps to reproduce
Steps to reproduce the behavior:

Go to 'https://codepen.io/offbeat-io/pen/VwxYjPj'
See error

4. Expected behavior
Both timeline steps should start at the same time with the second step taking more time to finish than the first one.

at is an animation option
{ at: 0, duration: 2 }
or
{ at: "<", duration: 2 }


feat: add string/base/last-code-point

Resolves #1702
Description

What is the purpose of this pull request?

This PR adds the package @stdlib/string/base/last-code-point
This pull request:

Adds the package @stdlib/string/base/last-code-point  which returns the last n Unicode code points of a string.

Related Issues

Does this pull request have any related issues?

#854
This pull request:

resolves #1702

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

changes have been made , this PR is ready for review


Fixed inconsistencies in HLSL ASM formatting

I noticed that there were inconsistencies and some errors with the HLSL ASM documentation and I have tried to fix them. More could be done, but I wanted to at least do these basic things first.
In the majority of instructions, a single destination register is called dest so I have made any dst or dst0 (where there is no dst1) match that.

@microsoft-github-policy-service agree

It seems the component selection annotations are a little inconsistent to:

✔ .swizzle any combination of xyzw (input)
✔ .mask any combination of xyzw (output)
❌ .write_mask any combination of xyzw (output) - duplicate of above?
❓ .select_component/.single_component_mask seem to be the same but could be replaced with .component
❓ .r Usually it's on a sampler, but sample_c and gather4 have it on srcResource (even though there's also srcSampler). I've only seen .x used which should be equivalent, but some things claim to require it and in other places it's optional.
❓ .xyzw store_uav_typed claims to require a fixed swizzle on dstUAV, but in all the sample/gather instructions srcResource doesn't have the same requirement (but I always see it in examples)

Some instructions add the negative and absolute modifiers to them, and I'm not sure if that means it's only supported on those instructions, or if all inputs variables support it and only those ones have been annotated.
Clarification on these points would be great, and I can fix those inconsistencies too.

Hi, @Nintynuts, and thanks for making all these changes and submitting this PR! :)
It's an unusually large PR, with 70 files changed and many changes per file. Every change needs to be read and reviewed (it's not enough just to scan or smoke-test), which in cases like this is expensive. I've looked through every change, and they all look safe. But I'll reach out to an SME to see whether they have the bandwidth to confirm the technical accuracy of the changes that need that level of review. Those changes are hidden within changes such as changing span elements (those elements shouldn't be in the content in the first place, so that's a problem on our side) and simple lexical transforms.
I have some ideas for what we might do differently in a case like this, either next time (or this time if I can't find an SME with the bandwidth to review this PR).
First, if there's a find-replace transformation, then it's much less expensive to let me know what the transformation is. For example, "Change all dst to dest in topics inside the direct3dhlsl folder" (you could communicate that in a PR where you make just the very first change). That way I can make and publish the transform on my end without needing to look at and review every one of potentially hundreds of individual changes.
Then, for changing technical details in descriptions (such as changing valid parameter ranges), it'd be great to describe those changes (and the reason for them) in a comment in the PR. Then the SME can read and confirm those descriptions and reasons, and I can just confirm the change has been made consistently. That'll be more economical, I think. I do see a list of items in the PR (for example, about component selection annotations), which is great. But I can't tell whether you actually made those changes in this PR, or if you're asking whether some info in the docs is correct or not. In any case I'll ask the SME to look at the comments, too, and see what they think.
Perhaps some of the questions or discussion could be done on Microsoft Q&A rather than in a PR, if that's a better workflow. But we'll see what we can do here first.
Thanks again! :)
-Steve

Thanks Steve, that's fair enough and I understand the need to verify it all.
My second comment here is a combination of observations and questions (particularly where there are question marks) which I wasn't confident to change before getting confirmation, so having a subject matter expert would be helpful.
Please let me know if anything specific of what I've said isn't clear.


Add Node Exporter Quickstart

Summary
A new quickstart for Node Exporter data ingested from a Prometheus Remote-Write configuration.  This is host/node data that details CPU/Memory/Storage/Network.

Pre merge checklist

[x] Did you check you NRQL syntax? - Does it work?
[x] Did you check your dashboard image quality? -  Do they look good?
[x] Did you check that your alerts actually work?
[x] Did you include an InstallPlan and Documentation reference?
[x] Did you check your descriptive content for voice, tone, spelling and grammar errors?
[x] Did you attach images of your dashboards to the PR so we can see them working?

Holding off on the merge while @shawnj's team gives this a review.

This seems like a lot visualizations in the first page. Could we have an overview page with Load 15min, CPU Seconds per mode, Memory Total Bytes, Storage Disk Bytes Read/Written , and Network Rx/Tx Bytes perhaps have the other viz in other tabs?

@lazyplatypus So put just the Network status on its own viz page?

@aswanson-nr Please feel free to merge this PR now.  Thanks!!


Not really

Azure offers IOS agents. No need to have a remote machine any longer

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

ID: 32f33eba-a747-c57b-02e3-853d2a8a5c3b
Version Independent ID: 1ddd9492-f5b1-d0ae-b813-c3838452a17a
Content: Install And Configure Tools to Build using iOS - Visual Studio
Content Source: docs/cross-platform/install-and-configure-tools-to-build-using-ios.md
Product: visual-studio-windows
Technology: vs-ide-mobile
GitHub Login: @corob-msft
Microsoft Alias: corob

@josersleal  Thank you for the feedback.
Is this a feature request or are you citing an issue with the article that refers to not needing a remote machine?
For feature requests, please consider starting a dialog with the product team using Suggest a Feature in the Visual Studio community.  For documentation issues, I can assign the author.

Closing the issue now, please comment below if there are further issues.


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative/serving-writers knative/eventing-writers
/assign knative/serving-writers knative/eventing-writers
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/hack 8d1e4cc...36b2b3c:
  > 36b2b3c add flag (# 224)
  > 547a2ca Start Signing our Releases (# 198)
  > 6c30196 :broom: Rewrite unit tests in Golang (# 215)
  > f9dc722 Format go code (# 218)
  > 664eac5 Fixing go.mod foreach (# 217)
  > 97fd663 :gift: Support multiple Golang modules in a single project (# 205)
  > f472a96 fix go run invocation of github hub cli (# 212)
  > 7d3bed1 go run... (# 210)
  > e81d63a Update community files (# 209)

/retest

/retest

/retest

/hold merging k8s deps bump first

/retest
/lgtm
/approve

/hold cancel


getProfile returns incorrect followingCount

In particular, it seems that the followingCount property of the Profile object is copying the likesCount property.
For example, this is (part of) one of the profiles I fetched:
{
  ...
  biography: '',
  followersCount: 3,
  followingCount: 82,
  friendsCount: 5,
  mediaCount: 0,
  isPrivate: false,
  isVerified: false,
  likesCount: 82,
  listedCount: 0,
  ....
}

I checked and the followersCount here is incorrect, but equal to the likesCount.
Tested with some other profiles and the patterns repeats itself
I'm on the most recent version

It seems that Twitter itself actually just uses the friends_count as the following count, and that is already being returned by the getProfile.
So as far as I know, you can just use the friendsCount entry as the nr of following.
Still, may as well remove the followingCount property, or give it the same value as friendsCount.

Fixed in #67


disable evasion

evasion is so awful

POV Captain evaded your Armblade 17 times in a row

might want to revert the armblade and esword buffs from armok too if you're gonna remove it, otherwise you'll have armblades beheading people in 2 hits lmao

might want to revert the armblade and esword changes from armok too if you're gonna remove it, otherwise you'll have armblades beheading people in 2 hits lmao

Isnt that the point of an overpriced 4 points weapon
wich makes you insta-valid to everyone, with sound AND visuals ,
requiring precision to do atacks  (ELSE you loose a lot of stanina on wideswings) in quick succession (they WILL run away else and then proceed to magdump every tider in maints JUST TO BE SURE)
This weapon REWARDS for using strategy and sting synergy

I fixed it though... Only head can evade now and other issues are fixed. I like current esword delimbing combat.

armblade users celebrating

I think we dont need this PR
Evading really DID get fixed and now applies only to Head
Since with "Fix Wheelchair Bound" people fall prone with 1 or more legs missing, Armblade will have better combat intereactions at LEAST against humanoids
(Ill break your knees - the Update)

yea no it is fine now my bad


Problem after reboot: policy restart=always with icc=false and linked containers

Steps to reproduce:
Run docker with --icc=false
Run docker-compose up  (with linked containers and restart=always policy)
Reboot OS couple times (min. 2)
Problem:
After first (most time second) reboot/shutdown and boot most of containers that use links are down.
Simply starting docker-compose up and every container is up again.
Other info:
When icc=true and with links between containers everything is fine.
When icc=false and without links between containers everything is fine too.
debug_log:
ESC[31mERROESC[0m[0002] leaving endpoint failed: a container has already joined the endpoint
ESC[31mERROESC[0m[0002] leaving endpoint failed: a container has already joined the endpoint
ESC[31mERROESC[0m[0002] leaving endpoint failed: a container has already joined the endpoint
ESC[31mERROESC[0m[0002] leaving endpoint failed: a container has already joined the endpoint
ESC[31mERROESC[0m[0002] leaving endpoint failed: a container has already joined the endpoint
ESC[31mERROESC[0m[0002] leaving endpoint failed: a container has already joined the endpoint
[...]
ESC[34mINFOESC[0m[0006] Loading containers: done.\
VirtualMachine Parallels
Tested on Docker version: 1.7.1, 1.8.1 and most recent 1.9-dev
Linux 3.19.0-25-generic #26~14.04.1-Ubuntu SMP Fri Jul 24 21:16:20 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
Containers: 39
Images: 410
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 488
Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.19.0-25-generic
Operating System: Ubuntu 14.04.3 LTS
CPUs: 2
Total Memory: 3.856 GiB
Name: XXX
ID: K4VD:3MVL:654M:XT2I:XSZU:M5PE:Y2NR:A5OO:RYGI:6SN7:PLCN:SDER
WARNING: No swap limit support

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Issue updated

ping @mavenugo

@mikechris this must be resolved now by #16910. can you please confirm ?

ping @mikechris

I'm going to close this issue as fixed. @mikechris, please feel free to open a new issue if the problem persists.


feat: add more source details

Description
Adding additional information to the dim_dbt__sources model including

The description of the source (the description is of the node and can also be used for models)
The database of the source
The load of the source

To Note
This change is a breaking change, because it changes the source table dbt_manifest_nodes which is used for the V2 set up (also breaks if V1 is being used).
Mitigation:

Drop table dot_manifest_nodes
Run dbt run-operation create_artifact_resources again
If V2 is used, do any migration steps for V2 again (possibly, we are still using V1 so I haven't looked into this)

Thanks for this contribution @LukasS29 - your work has been included as part of https://github.com/brooklyn-data/dbt_artifacts/pull/105 which is now merged.


Update bootstrap initializer banner

remove a misleading sentence

Please do not make direct changes to this file!

💭 prep & discussion:

https://github.com/heartcombo/simple_form-bootstrap/pull/359

fixes #1756
/cc @carlosantoniodasilva @collimarco

@m5o thank you


Dark theme issues

When using the relatively new "dark" theme that came out in a recent Chrome build, the templates panel is difficult to read.

+1 :)

Thanks @robdecker. We'll fix it.

Added the option to enable the Dark theme. Tagging a new release soon. See screenshot below.

Looks great!

@arshad - The last release I see in the Chrome webstore is from April 5. I am guessing this fix has not made it to the plugin there yet? Thanks.

I'll tag a new release tomorrow.

New version released with fix. Please update. Closing this issue. Thanks everyone.


asp0018 exception

Fixes #31964

Internal previews

📄 File
🔗 Preview link

aspnetcore/diagnostics/asp0018.md
ASP0018: Unused route parameter

@agc93 please review.

@martincostello  much appreciate!


Test and build plugin

Ensure the plugin is built similar to other existing plugins
/assign @psschwei

We're having some Slack discussions about this here and here.
Currently, the plan is to build the plugin manually for the time being (as we may want to release faster than the 6-week Knative cadence), then at some point move it into the release train.
The repo appears to be set up properly (save for one issue that #46 will fix); the next step is to see what happens when we cut a build. Ideally, it will automatically create a release with the appropriate binaries.

With release of v0.1.0 seems that we can close this one


Is it true that this library doesn't use the reflection package? You lied.

I thought there was some new hack to implement dependency injection without reflection and only through generics. A quick look at the code revealed that this was a lie.
// service.go:27
func generateServiceName[T any]() string {
	var t T

	// struct
	name := fmt.Sprintf("%T", t)   // <-- Here you use reflection !!!
	if name != "<nil>" {
		return name
	}

	// interface
	return fmt.Sprintf("%T", new(T))
}

It's this kind of PR (title, content) that drives people away from open source contributions.
Please consider the possibility that the author didn't deliberately lie. It would have been approximately 1000x friendlier and more useful to have said "hey, %T actually does reflection under the hood" and, ideally, offer a PR to change either the docs or the implementation.

Is there no way to use the full path to the package from go.mod in the service name instead of just the package in which it is located?
If you try to make Provide services with the same name and package in different subpackages it won't work
p1/p.Service
p2/p.Service

If someone else gets curious about the details of how it's using reflection, there's the piece of code (from fmt package)
https://cs.opensource.google/go/go/+/refs/tags/go1.20.1:src/fmt/print.go;drc=1a9893a969e0a73dc4f1e48ed40ccaf29ec238a6;l=698

@btvoidx: Please implement Injector with type key[T any] struct{}. I'm sure many people would be interested. Regarding changing the API: there is always the option to create github.com/samber/do@v2.

I think it is rather dishonest to still claim in the README to not use reflection when it has been pointed out that it does in fact uses it.


(BOLT-1100) Add install note about loading powershell module

On PS 2.0 and 3.0, the Bolt powershell module isn't automatically
loaded. This commit adds a note to the install docs, based on the
equivalent note from the pdk docs, explaining the steps to take to load
Bolt manually.

This is based very closely on the same note from the
PDK install docs.

CLA signed by all contributors.

Updated with @Iristyle's suggestions.

I tested these instructions out on a Windows 2008 host with PowerShell 2.0


[BUG] OpenIMSDKRN.sendMessage fail

OpenIM Server Version
3.8.0
Operating System and CPU Architecture
macOS (ARM)
Deployment Method
Docker Deployment
Bug Description and Steps to Reproduce
const options = {"groupID": "", "message": {"clientMsgID": "8711d448b1d7c519f0040ff78d7e7fb0", "contentType": 101, "createTime":<PHONE_NUMBER>536, "isRead": false, "msgFrom": 100, "sendID": "6111529997", "sendTime":<PHONE_NUMBER>536, "senderNickname": "curry", "senderPlatformID": 2, "seq": 0, "sessionType": 0, "status": 1, "textElem": {"content": "SD"}}, "recvID": "2004242507"}

try {
   const successMessage = await OpenIMSDKRN.sendMessage(options, uuidv4());
} catch(error) {
   console.log('发送消息失败:', error); // 发送消息失败:  Error: json: cannot unmarshal number 1.723458869536E12 into Go struct field MsgStruct.createTime of type int64
}

Screenshots Link
No response

Did you create the message via createTextMessage?
Note: Do not construct the message body yourself

use  const message = await OpenIMSDKRN.createTextMessage(text, uuidv4());
Based on the error message, it appears that the createTime field is being parsed as an int64 type in Go, while the number in the JSON is being interpreted in scientific notation (e.g., 1.723458869536E12), leading to a parsing failure.

I submit a PR

import com.alibaba.fastjson.serializer.SerializerFeature;

private String readableMap2string(ReadableMap map) {
    try {
      HashMap<String, Object> hashMap = map.toHashMap();
      return JSON.toJSONString(hashMap, SerializerFeature.WriteNonStringValueAsString);
    } catch (Exception e) {
        e.printStackTrace();
        return "{}";
    }
  }

use of SerializerFeature.WriteNonStringValueAsString will only convert Number types (such as int, long, float, double) to strings.

There are many other methods that also use map.toString() and you need to determine if you want to replace them all with readableMap2string

updated to the latest 3.8.1-rc.0

updated to the latest 3.8.1-rc.1

updated to the latest 3.8.1-rc.1

You are recommended to refer to the lower version https://github.com/openimsdk/open-im-sdk-reactnative/issues/32


Fix user commands in terminal with function calling

End-user friendly description of the problem this fixes or functionality that this introduces

[ ] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below
Fix agent crash when terminal commands are sent and function calling is enabled

Give a summary of what the PR does, explaining any non-trivial design decisions
When function calling is enabled, the agent expected all CmdRunAction to be "tools", but if they were sent by the user in the terminal, they were not, they were regular command actions.
This PR proposes to convert them into Messages as usual pre-function calling. Since they have source 'user', same as the next observation, and potentially more (a user message in chat probably follows too), they all get added to the same Message with role='user' when sent to the LLM.
On a side note, the concatenation... worked too well, so much so that Claude felt the need to help the human:
Prompt:
----------

... # other stuff done by the agent

----------

pwd
OBSERVATION:
/workspace[Python Interpreter: /openhands/poetry/openhands-ai-5O4_aCHf-py3.12/bin/python]
openhands@openhands-workspace:/workspace $
[Command finished with exit code 0]
print 5
print 55
hi there

LLM response:

23:56:25 - openhands:INFO: agent_controller.py:513
ACTION
[Agent Controller 6368bb2f-c5b4-4dc0-aea9-a973fe1afbc0] CmdRunAction (source=None)
THOUGHT: I notice you're trying to type commands but they seem to be jumbled together. Let me help you with the proper command. If you want to see the current working directory, you can use:
COMMAND:
pwd

😅
Maybe we should insert some mini-separators?

Link of any specific issues this addresses
Fix https://github.com/All-Hands-AI/OpenHands/issues/4783

Awesome that you're fixing this, I was just about to file a bug on this exact problem

It's a good question, it could go either way. It happens in the same time with the task, and we know we need to tell the agent when the user is making modifications, rather than just looking. 🤔
Please take a look also at the slack discussion.

Yes, it could go either way, made a UX suggestion to just have a checkbox on top of the terminal tab to let user decide whether his commands should be shown to the LLM

Currently:
----------

COMMAND:
pwd
OBSERVATION:
/workspace
[Python Interpreter: /openhands/poetry/openhands-ai-5O4_aCHf-py3.12/bin/python]
openhands@openhands-workspace:/workspace $
[Command finished with exit code 0]
print 99 ?

----------

I'll help you print the number 99 using Python in the IPython environment.


EOS 18914: The node for which conf_dir access is denied is unknown

Solution: Displaying the node-name of those nodes for which /var/lib/hare/node-name is not accessible
Signed-off-by: SUPRIT SHINDE<EMAIL_ADDRESS>

The patch seems to solve the said problem though by displaying the node name but in general the actual problem can be something else and this can one of the symptoms of it. But just in case the problem is seen only during Hare deployment then I think the error message looks good. @supritshinde , can you post some data from the actual crime scene?

I think the problem is more general: if anything goes wrong, hare-bootstrap script will not help to understand at which node the problem appeared.

@knekrasov I cannot confirm this. I think in some cases we can see where the problem is, but I'm not sure. It's better to revise the script(s) for this issue probably.
I'd suggest using this function for all ssh commands in the script:
myssh() {
    local hostname=$1
    shift
    ssh -n $hostname "$@" 2>&1 | sed "s/^/$hostname: /"
}

It will prefix each output line with the node name, for example:
$ set -o pipefail
$ myssh sage-tier1-1 ls bbb
sage-tier1-1: ls: cannot access bbb: No such file or directory
$ echo $?
2

Hi @mssawant While analysing the issue by @andriytk The permission denial error was given out for Access issue to the node name file inside confdir. This was seen after "Generating cluster Config.... line" And the code block after that wasn't completely executed so, before printing 'OK'. System throwed the error and exited. And, in that part of code. We echo name of node into the node name file. (just this single operation is happening nothing else). Here's the link to JIRA ticket https://jts.seagate.com/browse/EOS-18914

The issue is addressed in https://github.com/Seagate/cortx-hare/pull/1608
And that's better solution. Hence, Closing this PR.


[Flink-14942][state-processor-api] savepoint deep copy

What is the purpose of the change
Creating a new savepoint from an existing savepoint is broken when relocatable savepoint is introduced in Flink 1.11. The PR fix the issue by making a deep copy instead of shallow copy.
Brief change log
Copy the state files of untouched operators from the existing savepoint directory to the new savepoint directory.
Verifying this change
This change added tests and can be verified as follows by run SavepointDeeoCopyTest which:

bootstraps savepoint1 with Operator1 and make sure state is large enough to have state files in addition to _metadata
create savepoint2 based on savepoint1 by adding Operator2
verify all state files from savepoint1 are also presented in the savepoint2 directory
make sure reading the state of Operator1 from savepoint2 is succeeded, and the number of keys remains the same

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? Kind of... Shallow copy was used before, now it is deep copy
If yes, how is the feature documented? Removed the warning of shallow copy

You can apply this patch to SavepointWriterITCase to see the failure.
SavepointWriterITCase.patch.txt

Thanks for the hint. The test is passed now with my last commit.

Looks good, merging . . .

Looks good, merging . . .


deploy: Add prompt to beginning that ensures you've set up an AWS IAM role with the correct permissions

Confirm that all of the following permissions are actually needed w/ team

Prompt for:

[ ] IAMFullAccess
[ ] AmazonS3FullAccess
[ ] AmazonKinesisFirehoseFullAccess
[ ] AmazonTimestreamFullAccess
[ ] AWSLambda_FullAccess


Update and rename _articles/zh-hant/index.html to _文章/zh-hant/index.html

[ ] Have you followed the contributing guidelines?
[ ] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Sent from App for Gmail
Wednesday, November 17, 2021, 12:19 AM -0600 from x157660503  @.***>:

Have you followed the  contributing guidelines ?

Have you explained what your changes do, and why they add value to the Guides?
Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

You can view, comment on, or merge this pull request online at:
   https://github.com/github/opensource.guide/pull/2639
Commit Summary

d70f0c7 Update and rename _articles/zh-hant/index.html to _文章/zh-hant/index.html
File Changes
( 1 file )
R _文章/zh-hant/index.html (3)
Patch Links:
https://github.com/github/opensource.guide/pull/2639.patch
https://github.com/github/opensource.guide/pull/2639.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly,  view it on GitHub , or  unsubscribe .


Panic when trying git lfs clean

When executing git lfs clean on local repository that has downloaded the large files.
Windows 8.1
Git for Windows git version 2.6.2.windows.1
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x30 pc=0x4b9309]

goroutine 1 [running]:
github.com/github/git-lfs/commands.requireStdin(0x99e120, 0x34)
        C:/Users/techn/go/src/github.com/github/git-lfs/commands/commands.go:105 +0x49
github.com/github/git-lfs/commands.cleanCommand(0xb65d40, 0xb91620, 0x0, 0x0)
        C:/Users/techn/go/src/github.com/github/git-lfs/commands/command_clean.go:18 +0x71
github.com/github/git-lfs/vendor/_nuts/github.com/spf13/cobra.(*Command).execute(0xb65d40, 0xb91620, 0x0, 0x0, 0x0, 0x0)
        C:/Users/techn/go/src/github.com/github/git-lfs/vendor/_nuts/github.com/spf13/cobra/command.go:477 +0x40a
github.com/github/git-lfs/vendor/_nuts/github.com/spf13/cobra.(*Command).Execute(0xb68fa0, 0x0, 0x0)
        C:/Users/techn/go/src/github.com/github/git-lfs/vendor/_nuts/github.com/spf13/cobra/command.go:551 +0x59a
github.com/github/git-lfs/commands.Run()
        C:/Users/techn/go/src/github.com/github/git-lfs/commands/commands.go:88 +0x2a
main.main()
        C:/Users/techn/go/src/github.com/github/git-lfs/git-lfs.go:34 +0x135

goroutine 5 [syscall]:
os/signal.loop()
        c:/go/src/os/signal/signal_unix.go:22 +0x1f
created by os/signal.init.1
        c:/go/src/os/signal/signal_unix.go:28 +0x3e

goroutine 8 [chan receive]:
main.main.func1(0xc082106fc0, 0xc08200def0)
        C:/Users/techn/go/src/github.com/github/git-lfs/git-lfs.go:22 +0x74
created by main.main
        C:/Users/techn/go/src/github.com/github/git-lfs/git-lfs.go:32 +0x130

Thanks for the report. Is this happening during normal git lfs operation, or are you running git lfs clean manually? It is intended to be run by the internal git filters. While this panic is a legit bug that we should fix, it shouldn't be impacting normal usage of git lfs.
Also, what version of git lfs are you running?  Run git lfs version. This will let me match up line numbers from the trace.

I was running git lfs clean manually when trying to troubleshoot authentication issues and wanted to reset to ref-only state.
The git lfs version is git-lfs/1.1.0 (GitHub; windows amd64; go 1.5.1; git 258acf1)

Oh, interesting interpretation of the command :) The name clean actually refers to a filter triggered by gitattributes when a file is added. It has no relation to git clean. Thanks for the version details.

* git lfs clean:
    Git clean filter that converts large files to pointers.

Ah. It sounded like what I wanted to do, which was reset the large files back to pointers. I mentally skipped over the filter part of the help text as didn't understand it - however it now makes sense.
Is there another way to reset the local large files back to pointers?

I mentally skipped over the filter part of the help text as didn't understand it - however it now makes sense.

Thanks for explaining. Your interpretation makes a lot of sense to me now. I don't think it's worth renaming the command, but I should make the description more clear.

Is there another way to reset the local large files back to pointers?

Can you explain why you want this? Someone just asked about it recently in #951. I'm not opposed, just trying to understand why you want it.

Added this to my list of features to investigate for Git LFS v1.2: #844. It's something I want, I'm just not sure when I'll get around to it.

Hi there, this is a bit old but I ran into this bug as well when trying to debug this issue:

and version number
D:\dev\owlVR_2\Owls_v2>git lfs version
git-lfs/1.1.2 (GitHub; windows amd64; go 1.6; git bf6a6c9)
D:\dev\owlVR_2\Owls_v2>git lfs clean
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x30 pc=0x4b9f69]

goroutine 1 [running]:
panic(0x8f67c0, 0xc082008090)
        C:/Go/src/runtime/panic.go:464 +0x3f4
github.com/github/git-lfs/commands.requireStdin(0xa374c0, 0x34)
        C:/Users/techn/go/src/github.com/github/git-lfs/commands/commands.go:116
 +0x49
github.com/github/git-lfs/commands.cleanCommand(0xc2b2e0, 0xc4e708, 0x0, 0x0)
        C:/Users/techn/go/src/github.com/github/git-lfs/commands/command_clean.g
o:18 +0x75
github.com/github/git-lfs/vendor/_nuts/github.com/spf13/cobra.(*Command).execute
(0xc2b2e0, 0xc4e708, 0x0, 0x0, 0x0, 0x0)
        C:/Users/techn/go/src/github.com/github/git-lfs/vendor/_nuts/github.com/
spf13/cobra/command.go:477 +0x3fb
github.com/github/git-lfs/vendor/_nuts/github.com/spf13/cobra.(*Command).Execute
(0xc2e540, 0x0, 0x0)
        C:/Users/techn/go/src/github.com/github/git-lfs/vendor/_nuts/github.com/
spf13/cobra/command.go:551 +0x593
github.com/github/git-lfs/commands.Run()
        C:/Users/techn/go/src/github.com/github/git-lfs/commands/commands.go:99
+0x2a
main.main()
        C:/Users/techn/go/src/github.com/github/git-lfs/git-lfs.go:34 +0x135
:\dev\owlVR_2\Owls_v2>git lfs cleanpanic: runtime error: invalid memory address or nil pointer dereference[signal 0xc0000005 code=0x0 addr=0x30 pc=0x4b9f69]goroutine 1 [running]:panic(0x8f67c0, 0xc082008090)        C:/Go/src/runtime/panic.go:464 +0x3f4github.com/github/git-lfs/commands.requireStdin(0xa374c0, 0x34)        C:/Users/techn/go/src/github.com/github/git-lfs/commands/commands.go:116 +0x49github.com/github/git-lfs/commands.cleanCommand(0xc2b2e0, 0xc4e708, 0x0, 0x0)        C:/Users/techn/go/src/github.com/github/git-lfs/commands/command_clean.go:18 +0x75github.com/github/git-lfs/vendor/_nuts/github.com/spf13/cobra.(*Command).execute(0xc2b2e0, 0xc4e708, 0x0, 0x0, 0x0, 0x0)        C:/Users/techn/go/src/github.com/github/git-lfs/vendor/_nuts/github.com/spf13/cobra/command.go:477 +0x3fbgithub.com/github/git-lfs/vendor/_nuts/github.com/spf13/cobra.(*Command).Execute(0xc2e540, 0x0, 0x0)        C:/Users/techn/go/src/github.com/github/git-lfs/vendor/_nuts/github.com/spf13/cobra/command.go:551 +0x593github.com/github/git-lfs/commands.Run()        C:/Users/techn/go/src/github.com/github/git-lfs/commands/commands.go:99+0x2amain.main()        C:/Users/techn/go/src/github.com/github/git-lfs/git-lfs.go:34 +0x135

Any insight would be appreciated ! Thanks!
Dylan

The popup says that the "git-lfs" command does not exist. Check that it's installed properly, and that your native git tool has access to it. Specifically, it seems like your system path is missing something, or your git tool has a different environment. Please open a new issue with details, and we'll try to help the best we can.
The second error is totally unrelated. The git lfs clean command is not designed to be run by users, and must be called a very specific way by internal git filters. While it IS a bug we need to look at, I don't think it has anything to do with the first problem in the app popup window.
Can you run git lfs version so I can match the line numbers in the backtrace up with the correct code?

I copied the git-lfs.exe into the same folder as git.exe and this issue was resolved. It seems to be git not being able to find the lfs executable. Thanks for taking the time to respond !


undefined method when reinsertir battler

When removing an actor, and then re-inserting in the party a few turns later it throws a undefined method (move). I think this can be solved calling the inicialization of the script when adding the actor to the party, but i don't know how to achieve it

Hi. This is what we have placed in the Yanfly Blog :)
I also don't understand the report as it doesn't also state which script does it.
List of scripts used. This is important! If you use Victor’s scripts, then it’s a given Yanfly scripts won’t work due to low compatibility.
Description of the bug. Make sure to note HOW it happens and try to emulate it as much as possible.
Did you test it in a fresh new project? If no, then try at a new project first.
Did you load a save and encountered the bug there? If yes, then start a new game and see if the problem persists.
Did you check Yanfly’s Core first for bug fixes list? Is the bug mentioned there? If yes, then grab that script.
Did you read the features of the script first before assuming it’s not working like what you thought it would? If no, then read.
Did you try putting the script above/below other/certain scripts first? Give it a try, it might just fix it.
Give us a demo if possible. No need for the entire game, but just the area where it bugs out.


[DOCS ISSUE] Page: select

Hello, i'm starting a project with the new version and i was looking to add the showFields to the choices array but can't find how to do it, it is enabled on this version?

Yep! See the new "if" feature:
https://v3.docs.apostrophecms.org/guide/conditional-fields/
On Tue, Apr 12, 2022 at 3:32 PM Brian Mathias, Pesci Juliani <
@.***> wrote:

Hello, i'm starting a project with the new version and i was looking to
add the showFields to the choices array but can't find how to do it, it is
enabled on this version?
—
Reply to this email directly, view it on GitHub
https://github.com/apostrophecms/a3-docs/issues/196, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH27MT5DBBYJXDFZZKF7TVEXFTHANCNFSM5TIKD5YA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
THOMAS BOUTELL | CHIEF TECHNOLOGY OFFICER
APOSTROPHECMS | apostrophecms.com | he/him/his

It seems like that was the right answer!
This comes to another question, the conditions inside the if are done with mongo syntax?, i mean, i assume we do not ave the else as it is.
Many thanks!

It supports what you see in the documentation. We are following mongodb syntax as a model but all we support right now is implicit "and" (more than one property = they all must match), and explicit $or.


fq: 0.1.0 -> 0.2.0

Diff: https://github.com/wader/fq/compare/v0.1.0...v0.2.0
Description of changes
https://github.com/wader/fq/pull/537

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 207565 run on aarch64-darwin 1

1 package built:

fq


Object with index as keys should not equal array

expect({ 0: 'hello', 1: 'world' }).toEqual([ 'hello', 'world' ]);

I don't think this should pass, but it does. Is there something I am missing, or is this not intended?

JavaScript arrays are actually objects (with special semantics). I see no reason why that test shouldn't pass.

Node core no longer agrees: https://github.com/nodejs/LTS/wiki/Breaking-changes-between-v0.12-and-next-LTS-release#assert-check-object-prototype-in-assertdeepequal
https://github.com/nodejs/node/pull/621
Object.getPrototypeOf([0]) === Object.getPrototypeOf({'0': 0})
false


allow acceptssl to work with all commands

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Farmer's delight compat issues

Game crashes with farmer's delight
crash-2021-12-19_16.00.56-server.txt
crash-2021-12-19_16.00.59-client.txt

The client crash log doesn't mention More Bundles, it mentions Sodium. However, it is still possible that my mod may have caused that crash in a way. The server crash is definitely More Bundles fault and the root of the issue seems to be the same as this issue: https://github.com/BelgianSalamander/MoreBundles/issues/7. The error seems to be caused by farmer's delight adding a "Dispenser Behavior" before More Bundles has properly initialized and so the config. I should be able to fix this by loading the config as soon as it is needed even if it is before mod initialization.

I have been able to reproduce the crash and have now fixed it I will update the curseforge page soon.

This should be fixed in this version (https://www.curseforge.com/minecraft/mc-mods/more-bundles/files/3573607). Could you test it so I can close the issue? Thanks!

Yep, it works. Thank you for your hard work!


[WIFI-12539] Config S3 SDK tool to allow multiple providers

@Shodocan Good PR. I cannot test it but it all looks sound. Before I accept, I just need you to document these new parameters in the CONFIGURATION.md file.


[v2] README review

[x] 1st sentence: add "through events" at the end

[x] 2nd sentence: put "such as Clojure's ones" between parentheses

[x] Updates/cursor level: add to the example a simple syntax of nested data to better explicit what the first nested data example does

// Nested path
var newData = cursor.set({one: {two: 'world'}});

[ ] Give a way to easily find the list of possible events in the doc, their current formatting does not allow to spot them easily

[x] make explicit the difference between write and update events

[x] Advanced/polymorphisms 1st phrase: "THERE are many ways"

[x] tree/cursor.project: not very useful to users so put at the list bottom instead of the to

[x] tree/cursor.serialize: "avoiding computed data" is an obvious ref to facets, although for profane users it needs to be explicitely mentioned, AND moved down after facets are actually introduced

[x] tree/cursor.exists: in the first example (tree.exists()), maybe add something like "obviously always true, except when having specifically set the root of the tree to undefined using tree.set(undefined)" for better understanding

[x] Traversal: maybe put the root cursor part above the rest (before up)

[x] Computed data: in the second sentence, I would keep the old analogy with facets (as a "view" or "facet") that I personally find quite easier to apprehend

[x] Computed data example: typos before the last example command: "// Just note thaT computed data node IS read-only"

[x] History: "Baobab lets you record the SUCCESSIVE stateS"

[x] Philosophy: Maybe explicit UIs as User-Interfaces at least in the title

Danke schön @boogheta.

I would also prefer to have a formalized list of available methods for each datastructure with examples rather than just examples.
Something like:
<Object> Cursor
.merge(key:String, a:<Object>)
.merge(a:<Object>)
Merges objects shallowly.
Example
// cursor.get() = {x: "x"}
cursor.merge({y: "y"});
// cursor.get() = {x: "x", y: "y"}

This section of the documentation could indeed probably use more formal definitions.


devicelab: add more hot-reload metrics

Fixes #7238
/cc @johnmccutchan

lgtm but this does not fix #7238 because I also am requesting that these benchmarks be run against iOS and the simulators.

Gotcah, reworded.


Tally Eror!

Dear Dora Vota Team,
When I run tally (~20 DORA in Wallet).
The current MACI round is pending
startProcessing
Tally failed.
Query failed with (6): rpc error: code = Unknown desc = failed to execute message; message index: 0: Error calling the VM: Error executing Wasm: Wasmer runtime error: RuntimeError: Aborted: panicked at 'called Result::unwrap() on an Err value: NotFound { kind: "cosmwasm_std::math::uint256::Uint256" }', src/contract.rs:1019:16: execute wasm contract failed [!cosm!wasm/wasmd@v0.43.0/x/wasm/keeper/keeper.go:395] With gas wanted: '18446744073709551615' and gas used: '100303' : unknown request

Please help me!

@ltygmdev Please provide your contract address.

My contract address:
dora168qp56mkxzf2nv2yps8httwde8x3mmkj7jtw55scjupvljqztu8sk6wzpk

@ltygmdev  You may need to change the script version of the vota script, if you can I suggest you recreate a new round.
https://github.com/DoraFactory/vota-operator-scripts/releases


Breaks when other files than JS files are found

Hi Kyle,
thank you for this package, it would be really helpful for me right now! Unfortunately it seems to break when there are other files than JS files in from (in my case HTML files). When running mz via CLI I get: SyntaxError: Unexpected character '' (1:0). After removing the HTML files it works.
If you are open for it, I could also try to create a PR in the next days.
Thanks,
Michael

There's a skip config, and even a copyOnSkip config (to just copy them over). Does that work for you?

Oh yes, I had interpreted the skip option more like "skip these JS files", but obviously it could also be used for other files. Thanks a lot :)
One really minor thing I noticed: it would be very convenient if the to folder gets automatically created if it doesn't exist.

it would be very convenient if the to folder gets automatically created if it doesn't exist.

I considered that at one point, but it seemed a little more dangerous to have you specify a directory name (not a default one) that didn't exist, as it seemed like it might be a mistake (where you typo'd the target).
The output directory it does automatically create is the default ./.mz-build directory, as shown here: https://github.com/getify/moduloze/blob/2f3a1f99b800366d436d8c4efd191fd435afb4f8/src/cli.js#L239-L249
I could consider an option to override this and always create the target dir. But I think the way it currently is seems a safer default. Would a config option (like forceToPath or something) suffice?

In my opinion it's okay that a new directory gets created (in my use case a new directory is what I want) - if any the user could just rename it (correct me if I'm missing some possible negative side effects). But sure, an option like that would also be fine!