Split a Dataset into n folds with the transform split function

Hello everyone,
I am looking for a way to split a data set into n folds, taking into account the class distribution. The documentation says that this is possible with the function transform split. Now I would like to know if this is also possible with a Python script?
I am asking this because I have an image dataset (the classes are unbalanced) that I want to perform k-fold cross validation on. I have tried to use the k-fold stratification method, but unfortunately failed. Now I am wondering if I can use the Datumaro method.

Hi @DP1701,
If your task is one of classification, detection, segmentation, or re-id, you can use task specific splitter for your purpose.
You can use Split class directly or call 'transform('split', ...).

Thanks for the answer but I get the following error message:
transform() takes 2 positional arguments but 5 were given
File "split_test.py", line 11, in <module>
    dataset = dataset.transform('split', dataset, 'detection', splits)
TypeError: transform() takes 2 positional arguments but 5 were given

I use this:
from datumaro.components.dataset import Dataset

path_to_dataset = '/Users/dennisp/Desktop/dataset'
dataset_format = 'coco_instances'

dataset = Dataset.import_from(path_to_dataset, dataset_format)

splits = [(f'fold{i}', 0.1) for i in range(10)]

# with transform
dataset = dataset.transform('split', dataset, 'detection', splits)
subsets = dataset.subsets()

Hi @DP1701
Let me correct the transform call like below.
dataset = dataset.transform('split', task='detection', splits=splits)

I close this ticket because it seems that your question is resolved.

Sorry I didn't get in touch sooner. Here is a short update. It works wonderfully when I use the following:
from datumaro.plugins.splitter import Split
...
dataset_format = 'coco_instances'
dataset = Dataset.import_from(path_to_dataset, dataset_format)

k_split_portion = 1.0/k_split
fold_portion = [(f'fold{i}', k_split_portion) for i in range(k_split)]

# Create folds
folds = Split(dataset, mode, fold_portion, seed=None)
folds_subsets = folds.subsets()
...

I have another question: How can I merge two subsets and export them as coco_instances so that I get one annotation file?
Currently I have used the following:
...
merge_dataset = Dataset(folds_subsets['fold0'])
subset_two = Dataset(folds_subsets['fold1'])
merge_dataset = merge_dataset.update(subset_two)
subset_one.export(export_path, dataset_format, save_media=True, reindex=True, image_ext='.png')

When I run this, I get two json files in the annotations folder and not one json file.
How could I implement it so that I get one annotation file?

Hi @DP1701,
I'm happy to hear that you successfully split your dataset.
For the second question, you've got two json files because the coco format generates separated annotation files for each subsets.
To get one annotation file, you need to aggregate subsets into one. Please refer the document.
Here is the code example:

from datumaro.components.hl_ops import HLOps
aggregated = HLOps.aggregate(merge_dataset, from_subsets=["fold0", "fold1"], to_subset="train")
aggregated.export(export_path, dataset_format, save_media=True, reindex=True, image_ext='.png')

I hope this will help.

Thank you for your interests @DP1701. From your report, we felt the need of data preparation for K-fold cross validation. We are going to get back with this feature in the near future. I will close this issue.


Make 'ResourceManagerStringLocalizer' overload internal

After the PR #91 and while we have InternalsVisibleTo in this line of https://github.com/aspnet/Localization/blob/dev/src/Microsoft.Framework.Localization/Properties/AssemblyInfo.cs#L8, it will be nice to modify the access modifier for ResourceManagerStringLocalizer in https://github.com/aspnet/Localization/blob/dev/src/Microsoft.Framework.Localization/ResourceManagerStringLocalizer.cs#L49 to internal rather than public because the intend for this overload to be used for testing purpose only
/cc @kirthik

It could be useful for other people testing outside of this assembly. This is probably fine as-is.

I'm still wondering why it should be public, while internal is a proper access specifier?!!

Because if it's internal then other assemblies that don't have an InternalsVisibleTo relationship with it won't be able to use it.

Of course @Eilon, but while the intend is for testing purpose, the InternalsVisibleTo is already there for the testing project that use it. Am I right?

@DamianEdwards thoughts?

IVT is only useful for the target assembly. If a random person with a random assembly wants to call the method, it must be public.

Yep, but while it located in Microsoft.Framework.Localization and almost the cases the dev deal with Microsoft.AspNet.Localization so internal still a valid option. I hope to hear opinions from other guys.
thanks @Eilon

This is the pattern we use almost everywhere in ASP.NET 5 and EF 7. If our own framework needs this, then we assume that others will need this.

Thanks for this clarification @Eilon


fix(dynamic_avoidance): not avoid objects on ego's path

Description
This PR fixes the bug of avoiding objects on the ego's path.

Tests performed

planning simulator
Effects on system behavior

Nothing
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@kosuke55 Could you approve the PR.


response_timeout leaks memory

Hey Folks,
Found an interesting case the other day where my server was running out of memory because handlers cancelled from a response_timeout would keep references to their local context. This may be an asyncio  misunderstanding on my part but I found this quite surprising, here's a test to illustrate (sorry if its a bit verbose):
import asyncio
import weakref
from sanic import Sanic, response

class LeakedObject:
    pass

leaked = weakref.WeakSet()
app = Sanic(__name__)

@app.route("/slow")
async def slow_handler(request):
    l = LeakedObject()
    leaked.add(l)
    await asyncio.sleep(2)
    return response.text('OK')

@app.route("/fast")
async def fast_handler(request):
    l = LeakedObject()
    leaked.add(l)
    return response.text('OK')

app.config.RESPONSE_TIMEOUT = 1

# this works
def test_completed_request_cleaned_up():
    results = [None, None]

    @app.listener('after_server_start')
    async def _collect_response(sanic, loop):
        results[0] = await app.test_client._local_request('get', '/fast')
        # at this point the request is done leaked should no longer have a
        # weakref to LeakedObject
        results[1] = list(leaked)
        app.stop()

    app.run(host='<IP_ADDRESS>', debug=True, port=app.test_client.port)
    resp, references_leaked = results
    assert resp.status == 200
    assert resp.body == b'OK'
    assert references_leaked == []
    app.listeners['after_server_start'].pop()

# this does not
def test_timed_out_request_cleaned_up():
    results = [None, None]

    @app.listener('after_server_start')
    async def _collect_response(sanic, loop):
        results[0] = await app.test_client._local_request('get', '/slow')
        # at this point the request is done leaked should no longer have a
        # weakref to LeakedObject
        results[1] = list(leaked)
        app.stop()

    app.run(host='<IP_ADDRESS>', debug=True, port=app.test_client.port)
    resp, references_leaked = results
    assert resp.status == 503
    assert resp.body == b'Error: Response Timeout'
    assert references_leaked == []  # <<< fails here
    app.listeners['after_server_start'].pop()

if __name__ == '__main__':
    test_completed_request_cleaned_up()
    test_timed_out_request_cleaned_up()

@ashleysommer I think you'd be the expert here, side note I can't quite figure out how to use  ReuseableSanicTestClient here but it could potentially reduce the testing boilerplate above

so I've found an issue with response_timeout_callback (just not my issue), namely when it tries to cancel the _request_handler_task the CancelledError gets caught in handle_request's try/catch which leads it to continue chugging along even though the transport is already closed by .write_error(). I think that gets solved  with this https://github.com/channelcat/sanic/compare/master...abuckenheimer:fix_response_timeout-1273 but that doesn't fix my first issue.

@abuckenheimer
This is a really interesting problem. Thank you for bringing it up.
The ReuseableSanicTestClient probably wouldn't help in this case, you're right it is very hard and counter-intuitive to use, it was made for one very specific test case. Your boilerplate code in your test is fine for testing this behavior.
I will look into fixing that _request_handler_task canceled issue you pointed out, and I'll dig a bit deeper into the memory leak you've described.

@abuckenheimer
I've gone down a very interesting rabbit hole with this one (specifically, the memory leak you were seeing)
Some findings:

I could only reproduce it with asyncio running in Debug mode. Eg.
app.run(host='<IP_ADDRESS>', debug=True, port=app.test_client.port)
If I turned off debugging with debug=False, the test was passing.
I removed uvloop to run sanic using the native CPython asyncio implementation. This allowed slightly easier debugging of the problem, but the results of the memory leak test were identical.
With debug mode on, even after modifying sanic heavily to ensure the response object and its context is destroyed, the context was still leaked.
I read some interesting articles and stackoverflow answers about using the sys module to test for reference count of an object (sys.getrefcount(x)). And also down the track, using the gc module to determine all referrers of a particular object (gc.get_referrers(x)).
After the modifications to sanic to ensure the response and context was destroyed, there was still 1 reference to the LeakedObject, and the reference was inside something called a frame object.
I believe this is caused by running asyncio (and uvloop) in debug mode, the library keeps a copy of the Task stack frame in memory, with a hard reference to all context variables, for debugging  purposes.
I found out that you should always call gc.cleanup() before calling gc.get_referrers() to force the garbage collector to run, to avoid giving false results.
After running gc.cleanup() the reported problem disappears. Doing that seems to force the final
referrer to let go of the object, and the object gets de-allocated.

This passes:
import gc
def test_timed_out_request_cleaned_up():
    results = [None, None]

    @app.listener('after_server_start')
    async def _collect_response(sanic, loop):
        results[0] = await app.test_client._local_request('get', '/slow')
        # at this point the request is done. `leaked` should no longer have a
        # weakref to LeakedObject
        await asyncio.sleep(1)
        gc.collect()  # <<< only change
        results[1] = list(leaked)
        app.stop()

    app.run(host='<IP_ADDRESS>', debug=True, port=app.test_client.port)
    resp, references_leaked = results
    assert resp.status == 503
    assert resp.body == b'Error: Response Timeout'
    assert references_leaked == []  # <<< no longer fails here
    app.listeners['after_server_start'].pop()

good catch, my whole issue may come down to debug mode. What I find strange though is that I only found this bug because the memory leaks lasted for a long time which makes it seem like the garbage collector should have had plenty of time to do its thing. I did find this issue on windows though so there may be something else weird going on, will have to check in on it.

following up I didn't realize that gc only collects after a set number of object allocations so by having a handful of very large objects the threshold must not have been passed. Will consider this closed, thanks for your help @ashleysommer

This issue saved me big times.
Thank you so much @abuckenheimer


Quotes in --set parameter of redis update command given in the doc looks inaccurate

[Enter feedback here]
I followed az redis update doc to update shardCount for an existing redis instance.
If double quotes are included in --set parameter like --set "shardCount"="1", An usage error promoted to me
az redis update --subscription xxxxxx --name {my redis} --resource-group {my resourcegroup} --set "shardCount"="1" --output json
usage error: Empty key in --set. Correct syntax: --set KEY=VALUE [KEY=VALUE ...]

If try this, another error promoted to me
az redis update --subscription xxxxxx --name {my redis} --resource-group {my resourcegroup} --set \"shardCount\"=\"1\" --output json
Property '\shardCount\' not found on root. Send it as an additional property .
(InvalidRequestContent) The request content was invalid and could not be deserialized: 'Could not find member '\shardCount\' on object of type 'ResourceDefinition'. Path '['\\shardCount\\']', line 1, position 359.'.
Code: InvalidRequestContent
Message: The request content was invalid and could not be deserialized: 'Could not find member '\shardCount\' on object of type 'ResourceDefinition'. Path '['\\shardCount\\']', line 1, position 359.'.

If --set doesn't include double quotes like --set shardCount=1, command worked for me and updated the redis instance with a desired shard count in couple minutes later.
az redis update --subscription xxxxxx --name {my redis} --resource-group {my resourcegroup} --set shardCount=1 --output json
Thus, I am wondering the sample command with double quotes in --set parameter below in the doc is inaccurate.
az redis update --name MyRedisCache --resource-group MyResourceGroup --set "shardCount"="2"

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

ID: ecb66426-9b0f-be54-5530-0673e6821f39
Version Independent ID: 3d89a838-9a95-3cfe-ea58-7d7484e94956
Content: az redis
Content Source: latest/docs-ref-autogen/redis.yml
Service: cache
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

route to CXP team

@evanlwj Thanks for reaching out to us and reporting this issue. We are looking into this issue. We will provide an update once we get a chance.

@evanlwj Thanks for sharing the feedback. The above PR has been filed to address this issue. It is currently under review phase. In the meantime, if you have any questions, please let us know. We would be happy to help.

Thanks for an update. Just a reminder, I guessed --set "sku.capacity"="2" might have the similar issue (I didn't verify it myself).


Add Retry Configs to EgressConfig

This PR adds retry configs to the contract configuration in preparation
of supporting the delivery spec: #116
Signed-off-by: Pierangelo Di Pilato<EMAIL_ADDRESS>Part of #116
Proposed Changes

Add Retry Configs to EgressConfig

Release Note

- 🧽 Update or clean up current behavior
The data plane contract contains retry configurations.

The linter checks trailing whitespaces in the generated code.

/retest

/retest

Unit test failure is a general flakyness, not related to this one right?

Yes, micrometer metrics use atomic fields, and they never fail in Prow AFAIK.

/lgtm


Async implementation

Is there a plan to make the calls to models async?

+1, want to use ell but need async impl

+1, async would be great :)

I need it!!!

+1, I would also benefit from this enhancement to improve throughput.
Please let me know if there is anything I can do to assist!

@Colk-tech this is also a requirement for my usage. I've an have idea on how to implement this that I superficially sketched out
in the other issue https://github.com/MadcowD/ell/issues/193 @MadcowD opened. But didn't want to start without his approval.


Enabled additional rules for SwiftFormat

Added the rules mentioned in https://github.com/AlphaWallet/alpha-wallet-ios/issues/6583#issuecomment-1521104075

BTW, @eviltofu can you help add a TODO inside resources/SwiftFormatFiles.xcfilelist to ignore files such as modules/AlphaWalletFoundation/AlphaWalletFoundation/NFT/Enjin/API.swift that are autogenerated? Maybe R.generated too

BTW, @eviltofu can you help add a TODO inside resources/SwiftFormatFiles.xcfilelist to ignore files such as modules/AlphaWalletFoundation/AlphaWalletFoundation/NFT/Enjin/API.swift that are autogenerated? Maybe R.generated too

This should be in the config file.

I get
error: Unexpected end of file at 121:1 in /Users/jerome/Desktop/work/alpha-wallet-ios/AlphaWallet/Browser/ViewModel/Dapps.swift.

But this file is not in the AlphaWallet project. Can I remove this?

Exclude the file from SwiftFormat, but keep in repo. We are still maintaining it; just not including it

would you give this a quick pass to see if there's anything glaring? You can run it on a subset and look at diffs:

looks good, a lot of changes in Foundation and Web3 pods, but looks good


Bugs : dataset.map() is frozen on ELI5

Hi Huggingface Team!
Thank you guys once again for this amazing repo.
I have tried to prepare ELI5 to train with T5, based on this wonderful notebook of Suraj Patil
However, when I run dataset.map() on ELI5 to prepare input_text, target_text, dataset.map is frozen in the first hundreds examples. On the contrary, this works totally fine on SQUAD (80,000 examples). Both nlp version 0.3.0 and 0.4.0 cause frozen process . Also try various pyarrow versions from 0.16.0 / 0.17.0 / 1.0.0 also have the same frozen process.
Reproducible code can be found on this colab notebook , where I also show that the same mapping function works fine on SQUAD, so the problem is likely due to ELI5 somehow.

More Info : instead of map, if I run for loop and apply function by myself, there's no error and can finish within 10 seconds. However, nlp dataset is immutable (I couldn't manually assign a new key-value to dataset object)
I also notice that SQUAD texts are quite clean while ELI5 texts contain many special characters, not sure if this is the cause ?

This comes from an overflow in pyarrow's array.
It is stuck inside the loop that reduces the batch size to avoid the overflow.
I'll take a look

I created a PR to fix the issue.
It was due to an overflow check that handled badly an empty list.
You can try the changes by using
!pip install git+https://github.com/huggingface/nlp.git@fix-bad-type-in-overflow-check

Also I noticed that the first 1000 examples have an empty list in the title_urls field. The feature type inference in .map will consider it null because of that, and it will crash when it encounter the next example with a title_urls that is not empty.
Therefore to fix that, what you can do for now is increase the writer batch size so that the feature inference will take into account at least one example with a non-empty title_urls:
# default batch size is 1_000 and it's not enough for feature type inference because of empty lists
valid_dataset = valid_dataset.map(make_input_target, writer_batch_size=3_000)

I was able to run the frozen cell with these changes.

@lhoestq Perfect and thank you very much!!
Close the issue.

@lhoestq mapping the function make_input_target was passed by your fixing.
However, one I got this error in the final step of valid_dataset.map(convert_to_features, batched=True)
ArrowInvalid: Could not convert Thepiratebay.vg with type str: converting to null type
Do you have some ideas? (I am really sorry I could not debug it by myself since I never used pyarrow before)
Note that train_dataset.map(convert_to_features, batched=True) can be run successfully even though train_dataset is 27x bigger than valid_dataset so I believe the problem lies in some field of valid_dataset again .

I got this issue too and fixed it by specifying writer_batch_size=3_000 in .map.
This is because Arrow didn't expect Thepiratebay.vg in title_urls , as all previous examples have empty lists in title_urls

I am clear now . Thank so much again Quentin!

I'm getting a hanging dataset.map() when running a gradio app with gradio for auto-reloading instead of python


NLOHMANN_DEFINE_TYPE_INTRUSIVE_WITH_DEFAULT throw an exception with an empty json

Description
NLOHMANN_DEFINE_TYPE_INTRUSIVE_WITH_DEFAULT should not throw an exception when the json instance is empty and should use the default values of the member fields
Reproduction steps
run the Minimal code example and get the exception
Expected vs. actual results
don't expect to get exception with an empty JSON
expect that foo member fields are filled with they default values
Minimal code example
class Foo {
 public:
  NLOHMANN_DEFINE_TYPE_INTRUSIVE_WITH_DEFAULT(Foo, a, b)

  Foo() = default;

  int a{10};
  std::string b{"bar"};

  void log() {
    std::cout << "a: " << a << std::endl;
    std::cout << "b: " << b << std::endl;
  }
};

// with an empty json, an exception is thrown instead of getting default values
static void JsonTest_empty_json() {
  nlohmann::json json{};
  auto foo = Foo(json);
  foo.log();
}

int main(int argc, char **argv) {
  JsonTest_empty_json();
  return 0;
}

Error messages
No response
Compiler and operating system
clang++ / C++17 / Windows OS
Library version
ref: v3.11.2 - commit: 7d2f9ad500cf062394040f44d006791ff7cfc1c1
Validation

[X] The bug also occurs if the latest version from the develop branch is used.
[ ] I can successfully compile and run the unit tests.

I think there is already an issue for this.

It was not an issue, but a PR: #4507. @NissimBendanan can you check if that PR would be helping you?


Add hyperlinks to timestamps

Resolves https://github.com/Microsoft/vscode-pull-request-github/issues/110

I also made all hyperlinks underline on hover. Before, some of the hyperlinks were underlined without hovering and some were. To be consistent I made them all not underlined when not hovered.

The change looks good to me but IMHO the link for timestamp should not stand out, we may want to still use gray/white for it and show underline when hover.

Done!

Are there any other links that should should not be blue?

@JeffreyCA awesome, the code change is neat ;) Right now let's leave other links as they used to be.


Collection of Logs via Azure Monitor

I have added a section to collect the logs using Azure Monitor Service.
Problem: https://github.com/MicrosoftDocs/windows-itpro-docs/issues/2376

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

All of the suggestions applied.

@Justinha - Can you please assist to check, approve and merge the content of this PR based on issue #2376?
Thank you.


Requirements from Quasar team

Multi-chain setup
At this moment, the test framework setup takes a pair of chains (source and destination chains).
This is enough most of the times but becomes a blocker for more complex scenarios.
An example involving 3 chains (A, B, C): an IBC denom, originally from ChainA, needs to be transferred from ChainB to ChainC. The ChainA needs to be used as pass-through using the packet-forwarder feature so that the denon is valid in the destination ChainC.
Ideally, the ibc-test-framework should take an arbitrary number of blockchains as an input for the test setup.
All chains will be started, same as now, and it is for the developer to orchestrate the communication between these chains and test accordingly.
Note that we could also imagine future scenarios where multiple relayers are needed.
Changes
The main change is in the test setup.
ibctest/test_setup.go
func StartChainsAndRelayerFromFactory(
	t *testing.T,
	ctx context.Context,
	pool *dockertest.Pool,
	networkID string,
	home string,
	srcChain, dstChain ibc.Chain,
	f RelayerFactory,
	preRelayerStart func([]ibc.ChannelOutput, User, User) error,
) (ibc.Relayer, []ibc.ChannelOutput, *User, *User, error)

It currently takes two chains but should accept an arbitrary number, or a wrapping structure containing the various chains to be started.
Interchain Accounts support
The Interchain accounts feature is currently missing in the ibc-test-framework. Only the SendIBCTransfer is available on the type Chain interface.
The test framework should allow the developer to configure their ICA keepers accordingly and access the connections IDs necessary to start sending ICA packets.
Changes
A way to access the IBC connection IDs is needed to send ICA packets.
A new IBCConnection structure could be introduced containing more than what is currently in the ChannelOutput.
The appropriate genesis params will have to be set on the host chain (like the allow_list for the ICA messages).
Other

Ability to use locally built docker image

Implemented some of the pieces for testing interchain accounts in #103

We have a working Interchain Accounts test example here: https://github.com/strangelove-ventures/ibctest/pull/117
Support for interchain accounts in the main branch of the relayer will likely not land until we undergo the ChainProcessor refactor; progress can be tracked here https://github.com/cosmos/relayer/milestone/6
I also started to piece together a test utilizing the packet forward middleware. This one is not yet complete but it's in this branch if anyone is interested https://github.com/strangelove-ventures/ibctest/commit/eddb7e62bf7ea01af5042188c73558f184485691


Production queue: Problems with Corps/Army & Fleet/Armada

When ever i build Corps/Army & Fleet/Armada the production queue is "empty" but it is still building the unit.

With the current nightly 19.02.2017 7:50pm it's still buggy...

I have still the same problems like above, when I'm adding Corps to the queue.

When I have infantry corps in production, the right city view will not open:

When I open the city view in another city and change to a city with an infantry corp, the Queue won't show the correct Queue:

At the moment it looks like an Infantry Corp problem. Other units, like Artillery Corp works fine in the current game. But I had the problem at irregular intervals in the past too. So I have to pay attention for it now.

Seems weird to only affect one type of Corps but will look into it.


Can I define multiple methods in server/API/user. ts? For example, defining add and update methods?

Describe the feature
Can I define multiple methods in server/API/user. ts? For example, defining add and update methods?
Must there be a method to define a file?
Additional information

[ ] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide.
[X] Check existing discussions and issues.

See the docs file system based http method routing here: https://nuxt.com/docs/guide/directory-structure/server#matching-http-method


Perceptrons - the most basic form of a neural network

Written on 02/25/2017 14:00:43
URL: https://appliedgo.net/perceptron/

Migrated comment, written by Tom Kirby-Green on 04/11/2017 04:40:27
Thank you for the wonderful article. I'm currently exploring Deep Learning I my spare time and Go has been on my 'to-do' list to learn for a while now. Your article serves to push it up said list. Looking forward to reading your other posts.

Migrated comment, written by Christoph on 04/15/2017 16:41:24
Thanks! I think Go will find its place in the area of deep learning. It is one of the few languages that bring the speed of compilation and the simplicity of a scripting language together.

Migrated comment, written by slidenerd on 08/06/2017 12:43:47
nice tutorial, could you redo a version of this in python , not a go programmer

Migrated comment, written by cjoshmartin on 06/14/2018 06:24:34
How are the values of `a` and `b` set?

Migrated comment, written by Christoph on 06/14/2018 20:40:46
They are set in main(). I take a cheap and dirty shortcut here and declare a and b as global variables (see the code block after the Training headline). This saves a few function parameters, but frankly, I would not do it again this way. Global variables are only ok in very small entities of source code, and even there they can cause the reader to lose track of the data flow, as your question proves. Even I had to look twice until I found the place were a and be are set, and I am the one who wrote the code! (Even though it has been quite a while ago...)

Migrated comment, written by Romain Olympe on 12/27/2019 23:41:57
It's an amazing post thanks  a lot

Migrated comment, written by Fatih Erikli on 04/16/2021 23:43:07
So the story continues on https://nerver.net

Migrated comment, written by Fatih Erikli on 04/16/2021 23:43:24
#AppliedGo please approve my comment yoksa doverim


Two variables in native cube output are broken

This was discovered by @aoloso when testing the compress-and-compare of @wmputman 's DYAMOND2 data. Namely, two variables in the CS native output are garbage. They are defined as variables but never set it looks like, so you get...uninitialized junk in them. Luckily they seem to be variables nothing cares about, but we should fix it.
The big broken variable is orientation and the other one I suspect is broken is cubed_sphere.  (I wonder if this is a bug in libz? Seems like zip should be able to compress anything even junk...)
Speaking with @tclune, he looked at this code:
https://github.com/GEOS-ESM/MAPL/blob/8e933d8adf111af0d4215ecf09c7e0dbe1388ff9/base/MAPL_CubedSphereGridFactory.F90#L908-L936
where it looks like cvar that was filled back in old Developer Git days (https://developer.nasa.gov/GMAO/GEOSagcm/blob/4efb0f3c6210c84fc2cf1380be34c37104ad591c/src/GMAO_Shared/MAPL_cfio/ESMF_CFIOSdfMod.F90#L2347-L2390) was commented out and then orientation was never filled. He suspects maybe this was a GNU compiler issue maybe? Unknown as this code has been commented forever it looks like here in GitHub.
I'm also concerned about the other char variable cubed_sphere where it is declared:
https://github.com/GEOS-ESM/MAPL/blob/8e933d8adf111af0d4215ecf09c7e0dbe1388ff9/base/MAPL_CubedSphereGridFactory.F90#L888-L893
but I can't see anything ever actually filling it.

Note: This might be an opportunity as well for @bena-nasa and myself. Perhaps we should come up with a special HISTORY.rc.tmpl we could have my nightly tests run. The reason this was never caught is that my nightly tests use the "standard" @sdrabenh / @lltakacs HISTORY.AGCM.rc.tmpl for most tests. They don't have cube-output so I don't have cube output.
But if @bena-nasa could dream up a crazy HISTORY with varying code paths it could be a good test.

@bena-nasa Did we ever fix this? I want to say you looked at it...

@bena-nasa Did your recent work on the cube sphere factories, etc. fix this?

Okay. This is a fun one. @bena-nasa and I looked at this and we figured out there might be NO way for either of these to be written out. Why? Well if you look at write_const_variables in pfio/NetCDF4_FileFormatter.F90:
https://github.com/GEOS-ESM/MAPL/blob/beba7fc571c9096b9b0897b4966d8ee0d10ba987/pfio/NetCDF4_FileFormatter.F90#L466-L484
A string would fall under class default and...fail!
So, as a stopgap, what I (and @bena-nasa) is proposing is multi-fold:

Remove the cubed_sphere variable. We do not see the need for it.
Comment out the orientation variable until pFIO supports string variables

You are right we cannot write string variable at this moment. Is that important ? can that be in attribute section?

@weiyuan-jiang At the moment it seems to be okay not to do. The issue would arise in the future if there was a "standard" cubed-sphere format where this would become important.

If there is a reasonably straightforward fix for this, my preference would be to enable this now.    Even if it has to be disabled for gfortran.


How I can use crates?

Hi, I use cargo for example test code from rmp crate:
extern crate rmp as msgpack;
extern crate rustc_serialize;

use rustc_serialize::Encodable;
use msgpack::Encoder;

fn main() {
    let val = (42u8, "the Answer");

    // The encoder borrows the bytearray buffer.
    let mut buf = [0u8; 13];

    val.encode(&mut Encoder::new(&mut &mut buf[..]));

    assert_eq!([0x92, 0x2a, 0xaa, 0x74, 0x68, 0x65, 0x20, 0x41, 0x6e, 0x73, 0x77, 0x65, 0x72], buf);
}

I add cargo.toml and this work fine from console, but how I can build project with visualrust? Maybe can add something like customising build process?

https://github.com/PistonDevelopers/VisualRust/issues/162

Duplicate od #162


line:506  position error

error
 506   *this.position += this.position.saturating_add(delta as u64);

fix
 506   *this.position = this.position.saturating_add(delta as u64);

Thank you for reporting this issue. I've released version 0.2.1 with the fix.


Possible infinite loop

That is a fun way to implement a sed bot.
I don't know if you still use this bot, but, just in case, I wanted to report that I think it is possible to get it stuck in a busy infinite loop with a message like
s/^/&/; :x; b x

because you are running sed without a timeout, and sed --sandbox does not prevent infinite loops.
o/
 emanuele6

Yup, that's true. I've just applied a band-aid that solves the "infinite-loop" part, but people spamming this command can still cause a DoS (Denial of Sed).
I'll probably make each request fork a process that outputs $((OFFSET+1)) and have a pipe on the other end that updates the cached offset if it increases.


refactor expression and logical plan

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

Project coverage is 71.24%.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #132      +/-   ##
==========================================
- Coverage   76.88%   71.24%   -5.64%
==========================================
  Files         124      124
  Lines       15902    15760     -142
==========================================
- Hits        12226    11228     -998
- Misses       3676     4532     +856

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


Add simple AWS client implementation

Resolves #169
Scope
Implemented:

AWS authentication client
AWS S3 client

*NOTE

Only environment credentials are supported in this Pull Request

Checklist

[x] GitHub issue exists for this change.
[ ] Unit tests added and they pass.
[x] Pylint 10.0/10.0 without bloating .pylintrc with exceptions.
[x] Review requested on latest commit.

create_package


Include public/private dependency status in cargo metadata

Problem
related to https://github.com/rust-lang/rust/issues/44663
Information about whether or not a dependency is public is not included in the output of cargo metadata.
This could be useful for custom subcommands. cargo override, would find this useful for https://github.com/eopb/cargo-override/issues/121
Proposed Solution
When -Zpublic-dependency is enabled, include a boolean flag in cargo metadatas output at packages[].dependencies[].public.
If  -Zpublic-dependency is not enabled,  this flag should not appear in cargo metadata output.
Notes
If this feature request is accepted, I'm happy to claim and take a crack at working on it

This seems like basic enough of metadata for us to include.

@rustbot claim


PipFunTest: Update expected test results

Signed-off-by: Sebastian Schuberth<EMAIL_ADDRESS>

Merging despite the unrelated PipenvFunTest failure on Windows.


Use keyof in Object.keys() when object is constant (as const)

Hi,
Initial reasons for not using keyof in Object.keys: https://github.com/microsoft/TypeScript/pull/12253#issuecomment-263132208
I know this subject is kind of a duplicate but I'd like to get it revisited as const assertions are a thing now.
If the object is constant, declared with a const assertion, wouldn't it make sense to return the Keys[] type instead of string[]?
Use Cases
Get better typing with Object.keys() when possible
Examples
const TestObject = {hey: "hoo",a: "bc"} as const;

const TestKeys = Object.keys(TestObject)

TestKeys should have type "hey" | "a"
Checklist
My suggestion meets these guidelines:

[X] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[X] This wouldn't change the runtime behavior of existing JavaScript code
[X] This could be implemented without emitting different JS based on the types of the expressions
[X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[X] This feature would agree with the rest of TypeScript's Design Goals.

A type originating in an as const expression doesn't really imply that an object isn't aliased.
There'd have to be an entirely new way of writing signatures that only applied to "closed" objects, then new type system information for tracking which objects have a provably-closed set of keys and which don't. This is a lot of hoops to jump through for the sake of one built-in API.
What you're basically asking for is exact types, #12936

Thanks, didn 't know about those exact types ;)

Maybe is there a way to define somehow properly Object.keys(...) to infer keys as Array<keyof T> for exact types and string[] otherwise? Because I didn't manage to find a solution how to combine these Exact<...> types with Object.keys ...


Roll Flutter from 2303f42250b1 to 1546fa08d324 (15 revisions)

https://github.com/flutter/flutter/compare/2303f42250b1...1546fa08d324
2023-02-09<EMAIL_ADDRESS>[flutter_tools] toolExit on sdkmanager exit during doctor --android-licenses (flutter/flutter#120330)
2023-02-09<EMAIL_ADDRESS>Added ListTile.titleAlignment, ListTileThemeData.titleAlignment (flutter/flutter#119872)
2023-02-09<EMAIL_ADDRESS>Support --local-engine=ios_debug_sim (flutter/flutter#119524)
2023-02-09<EMAIL_ADDRESS>Add missing parameters to SwitchListTile (flutter/flutter#120115)
2023-02-09<EMAIL_ADDRESS>Add proper disabled values for input chips (flutter/flutter#120192)
2023-02-09<EMAIL_ADDRESS>Update gallery.dart (flutter/flutter#120366)
2023-02-09<EMAIL_ADDRESS>Manual roll Flutter Engine from 2f2e2e27cb28 to eb346ba63f69 (7 revisions) (flutter/flutter#120364)
2023-02-09<EMAIL_ADDRESS>Roll Plugins from f59c08db3f27 to 73986f4cc857 (5 revisions) (flutter/flutter#120362)
2023-02-09<EMAIL_ADDRESS>Manual roll Flutter Engine from 89c8a1393d4b to 2f2e2e27cb28 (3 revisions) (flutter/flutter#120333)
2023-02-09<EMAIL_ADDRESS>Fix widget inspector null check (flutter/flutter#120143)
2023-02-09<EMAIL_ADDRESS>Revert "[web] Move JS content to its own .js files (#117691)" (flutter/flutter#120275)
2023-02-08<EMAIL_ADDRESS>[Tool] [Windows] Output build duration (flutter/flutter#120311)
2023-02-08<EMAIL_ADDRESS>Update .cirrus.yml (flutter/flutter#120315)
2023-02-08<EMAIL_ADDRESS>Manual roll Flutter Engine from cc4ca6a06ab3 to 89c8a1393d4b (6 revisions) (flutter/flutter#120319)
2023-02-08<EMAIL_ADDRESS>Reland "Stop recursively including assets from asset directories" (flutter/flutter#120312)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins
Please CC rmistry@google.com,stuartmorgan<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Plugins: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


[1.8.x] chore: update docs script

Release Note
NONE

Sparing some checks resources, this is going to run at night time only.


[BUG] Ansible commands failing

To Reproduce
follow the example below,
Commands you ran
```---
- hosts: localhost
  connection: local
  tasks:
    - name: Creating resource group - "{{ name }}"
      azure_rm_resourcegroup:
        name: "{{ name }}"
        location: "{{ location }}"
      register: rg
    - debug:
        var: rg

ansible-playbook create_rg.yml --extra-vars "name=rg-ans location=centralindia"

## Observed Behavior
PLAY RECAP ****************************************************************************************************************************************************************************************************
localhost                  : ok=1    changed=0    unreachable=0    failed=1    skipped=0    rescued=0    ignored=0

Include error output here
fatal: [localhost]: FAILED! => {"changed": false, "msg": "Failed to import the required Python library (ansible[azure] (azure >= 2.0.0)) on cc-29446bdf-669f964886-b942f's Python /usr/bin/python3. Please read the module documentation and install it in the appropriate location. If the required library is installed, but Ansible is using the wrong Python interpreter, please consult the documentation on ansible_python_interpreter"}

## Expected behavior
changed=1
A clear description of what you expected to happen instead.

## Is this specific to Cloud Shell?
Please verify if the same issue can be reproduced by running the same tool **outside Cloud Shell** - for example,
by installing it on your own computer. If so, it is likely to be a bug in that tool or in the Azure service it communicates with,
not in Cloud Shell. Please file the issue with the appropriate project.

## Interface information
How are you accessing Cloud Shell - https://shell.azure.com, https://portal.azure.com, via Windows Terminal, or some other method?
If a browser, which Operating System and browser are you using? (ex. Edge on Windows 10)

## Additional context
Add any other context about the problem here.

AB#14503580

@kunalworldwide : Can you reproduce this again today? The ansible team added some changes and we recently released an image.


[Windows] Accessibility seems broken when supplying actions or shortcuts to MaterialApp constructor

Hi,
If you create a project with the below code and run it, you should find that unless you comment out both the shortcuts and actions arguments, your tab key doesn't work in the resulting app.
If it does work, it could well be a screen reader thing, in which case try to run with Narrator running.
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';

/// An intent to print google.com.
class PrintGoogleIntent extends Intent {
  /// Create an instance.
  const PrintGoogleIntent();

  /// The hotkey to use.
  static const hotkey = SingleActivator(LogicalKeyboardKey.keyG, control: true);
}

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    var callbackAction = CallbackAction<PrintGoogleIntent>(
      onInvoke: (intent) => print('Opening google.com.'),
    );
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
      shortcuts: const {PrintGoogleIntent.hotkey: PrintGoogleIntent()}, // Comment me.
      actions: {PrintGoogleIntent: callbackAction}, // And me.
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      // This call to setState tells the Flutter framework that something has
      // changed in this State, which causes it to rerun the build method below
      // so that the display can reflect the updated values. If we changed
      // _counter without calling setState(), then the build method would not be
      // called again, and so nothing would appear to happen.
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        // Center is a layout widget. It takes a single child and positions it
        // in the middle of the parent.
        child: Column(
          // Column is also a layout widget. It takes a list of children and
          // arranges them vertically. By default, it sizes itself to fit its
          // children horizontally, and tries to be as tall as its parent.
          //
          // Invoke "debug painting" (press "p" in the console, choose the
          // "Toggle Debug Paint" action from the Flutter Inspector in Android
          // Studio, or the "Toggle Debug Paint" command in Visual Studio Code)
          // to see the wireframe for each widget.
          //
          // Column has various properties to control how it sizes itself and
          // how it positions its children. Here we use mainAxisAlignment to
          // center the children vertically; the main axis here is the vertical
          // axis because Columns are vertical (the cross axis would be
          // horizontal).
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            Focus(
              child: Text(
                '$_counter',
                style: Theme.of(context).textTheme.headline4,
              ),
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

[√] Flutter (Channel master, 2.11.0-0.0.pre.751, on Microsoft Windows [Version 10.0.22000.527], locale en-GB)
    • Flutter version 2.11.0-0.0.pre.751 at C:\Users\chris\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 21797cbb03 (10 hours ago), 2022-03-02 23:16:22 -0500
    • Engine revision 121376d39a
    • Dart version 2.17.0 (build 2.17.0-164.0.dev)
    • DevTools version 2.11.1

[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0-rc2)
    • Android SDK at C:\Users\chris\AppData\Local\Android\sdk
    • Platform android-31, build-tools 31.0.0-rc2
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Enterprise 2022 17.1.0)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Enterprise
    • Visual Studio Enterprise 2022 version 17.1.32210.238
    • Windows 10 SDK version 10.0.22000.0

[√] Android Studio (version 2020.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[√] VS Code, 64-bit edition (version 1.64.2)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.27.0

[√] Connected device (4 available)
    • Windows (desktop)       • windows • windows-x64     • Microsoft Windows [Version 10.0.22000.527]
    • Windows (UWP) (desktop) • winuwp  • windows-uwp-x64 •
    • Chrome (web)            • chrome  • web-javascript  • Google Chrome 98.0.4758.102
    • Edge (web)              • edge    • web-javascript  • Microsoft Edge 98.0.1108.56

[√] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Thanks for the report. I do see the same behavior upon running the code sample on stable and master.
With actions or shortcuts enabled, tab key doesn't focus on the elements on screen.
If we comment those out and run narrator + tab key, the focus properly switches between elements as below:

 stable, master flutter doctor
[√] Flutter (Channel stable, 2.10.1, on Microsoft Windows [Version 10.0.19042.1526], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.2)
[√] Android Studio (version 2021.1)
[√] VS Code (version 1.62.3)
[√] Connected device (3 available)
[√] HTTP Host Availability

• No issues found!

/cc @cbracken

Ah, OK, thank you.
Take care,
Chris Norman
On Wed, 4 May 2022 at 17:29, Greg Spencer @.***> wrote:

This is "as designed". The reason it stops working is that when you are
assigning the shortcuts and actions at the MaterialApp level, it is
replacing the default shorcuts and actions. If what you want to do is
add to them, then you need to add your shortcuts/actions to the defaults.
You can get the set of defaults with the WidgetsApp.defaultActions and
WidgetsApp.defaultShortcuts calls.
See the documentation for the actions attribute
https://master-api.flutter.dev/flutter/material/MaterialApp/actions.html
or the shortcuts attribute
https://master-api.flutter.dev/flutter/material/MaterialApp/shortcuts.html
on MaterialApp for more information.
—
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/99482#issuecomment-1117559647,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBUZ2PU3N65GB5JRADSN63VIKQXBANCNFSM5P2T5LZQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


IFX-Stepper: Show content between steps and short text information on the left hand side

Detailed description of what you want to happen

We would like to have content between the individual steps (see screenshot). Currently, content can only be on the left or right hand side of the stepper but not in between of the steps.

We would like to  have the option to place short text on the left hand side of stepper's status icons (see screenshot).

Upload a screenshot or reference an example if possible

Why do you need this feature?
It is predetermined in the Figma templates of Infineon.
Do you have a deadline?
As soon as possible, it is urgent.
Additional information
No response

Hi, here is the link https://www.figma.com/design/IbFyAFXTfnOdKrxBCNWQqm/IDC-Online---Main-Design?node-id=200-4000&node-type=frame&t=rEeVKP3Dt6yqssve-0

@UgurAyvaz Hello Ugur Ayvas, in order not to disappoint your users and still offer them something that works and is understandable, you can use our vertical stepper. However, this is not based on your stepper's interactive logic. But the idea is the same. I also created some quick mockups that maybe could help you and make understand how our stepper would look like in your context. We also recommend to user shorter labels for the steps (also shown in the mockup). I hope this helps you to implement a different looking and working stepper.
vertical-stepper.zip


Add privacy manifest file for iOS

You need to add PrivacyInfo.xcprivacy file following the new apple policy.
Refer link: https://developer.apple.com/documentation/bundleresources/privacy_manifest_files

@navaronbracke Because mobile_scanner uses MLKit from google, I guess they might need to update MLKit first, so you can pick up newer version. I believe they are using Disk Space API, which should be listed in their manifest. I found it using this tool: https://github.com/crasowas/app_store_required_privacy_manifest_analyser
Analyzing MLKitCommon ...
⚠  Missing privacy manifest file!
API usage analysis result(s): 3
[0] NSPrivacyAccessedAPICategoryFileTimestamp:stat,NSURLContentModificationDateKey:/Users/marcin.pawlowski/projects/flut/yosensi-mobile-flutter/ios/Pods/MLKitCommon/Frameworks/MLKitCommon.framework/MLKitCommon
[1] NSPrivacyAccessedAPICategoryDiskSpace:NSFileSystemFreeSize:/Users/marcin.pawlowski/projects/flut/yosensi-mobile-flutter/ios/Pods/MLKitCommon/Frameworks/MLKitCommon.framework/MLKitCommon
[2] NSPrivacyAccessedAPICategoryUserDefaults:NSUserDefaults:/Users/marcin.pawlowski/projects/flut/yosensi-mobile-flutter/ios/Pods/MLKitCommon/Frameworks/MLKitCommon.framework/MLKitCommon
🛠  Descriptions for the following required API reason(s) may be missing: 3
[0] NSPrivacyAccessedAPICategoryFileTimestamp
[1] NSPrivacyAccessedAPICategoryDiskSpace
[2] NSPrivacyAccessedAPICategoryUserDefaults

The February MLKit release https://developers.google.com/ml-kit/release-notes#february_20_2024 does have some updates, but no Privacy Manifest yet, I think.


When trying to run Process Hacker as admin, it won't launch.

Not able to launch Process Hacker 2 64bit as admin, 32bit works fine and I even reinstalled the program and restarted my PC. Also, when I try to inject a DLL with the 64bit version, it opens the file explorer to find the DLL and crashes Process Hacker.
Just tried to launch 64bit Process Hacker 2 as admin from the shortcut on desktop, which didn't work. Tried to inject a DLL as simple user, but as soon as I see the file explorer popping up, it crashes Process Hacker. Went into the folder where it is installed, running as admin didn't work too for the 64bit version, but the 32bit one worked for some reason and I was able to inject a DLL without it crashing Process Hacker.
I expected Process Hacker 2 64bit to run as admin and being able to inject a DLL into a program without crashing.
My OS is Windows 11 21H2.
As mentioned before, reinstalling didn't help and rebooting the PC didn't do it too.

A little update on the situation.
It fixed itself somehow, I can't explain how because I just wrote that bug report here and went watching YT videos and waiting for an answer. Then I decided to try it again and it worked.

Process Hacker 2 is old. Try the new nightly builds from here https://processhacker.sourceforge.io/nightly.php

Process Hacker 2 is old. Try the new nightly builds from here https://processhacker.sourceforge.io/nightly.php

Just installed, seems to work fine out of the box, I'm going to stick with it for now. Ty for your suggestion.


#202 - stats

This is still missing logic for the demand/compel statistics.
I'm not sure where they go.  Presumably they would end up in the nested execution logic?

Also left a few TODOs for the outstanding work.


Deduplicate project page entries

Adapting the forthcoming change from https://github.com/jenkinsci/jenkins/pull/6918 to deduplicate items from the project page and the sidepanel.
The change proposed removes the "Recent changes" object, because it's already covered by the sidepanel.

cc @jglick

All checks are passing, but auto merge is blocked by a missing review 👀

@jenkinsci/workflow-job-plugin-developers Do you mind reviewing this PR? The called forthcoming change is shipped in weeklys for a few weeks now.

 jglick merged commit 71bba12 into jenkinsci:master 11 hours ago

I was a little startled to see that until I realized

 jglick enabled auto-merge  2 months ago

and it just got approved, thus satisfying the merge condition. :grin:


Mark implicit parens generated

Hello all,
This  patch marks implicit parentheses generated and adds a helper function for marking tokens generated. It's a fall-out from the discussion in issue #1921.
Thanks a lot,
Matt

LGTM. Verified that it builds and tests fine. @clutchski: How can we be sure this is exhaustive?

@michaelficarra: It definitely covers all of the implicit tokens added in the Rewriter, which is easy to tell by looking at all splice operations that insert into tokens arrays.
I haven't touched the Lexer because there is no precedent for it and I'm not sure one makes sense. The work in the rewriter is essentially reducing multiple possible syntactic representations of an idea - e.g. explicit and implicit objects, one line & multiline conditionals - into a single form before passing them along for further processing. And when the rewriter marks tokens generated it is essentially saying "this comes from the implicit syntax". I'm not sure there is an analogous situation in the Lexer, whose tokens all come from explicit syntax (I think), even in the case where that syntax produces more than one token per 'idea', like string interpolations. I'm not 100% sure I'm correct here, but that's my current thinking. What say you?

ps - thanks for the review.

I did a more thorough search, and I'm satisfied.

Cool, thanks very much.

Quick note -- I screwed up the merge a minute ago, but it should be good now.


Does the rule: navigator.connection.rtt == 0 possibly cause false positives for some cases like Edge?

We have observed that 2%-3% Edge traffic that has navigator.connection.rtt set to 0. It's much higher than other headless signals. So I wonder if there can possibly be any false positives for the rule: navigator.connection.rtt == 0 on Edge browsers?
Actually I searched around the Internet, but didn't find a strong evidence to prove that navigator.connection.rtt == 0 indicates a headless bot, except here. Any answers would be appreciated. Thanks!

Under Android WebView navigator.connection.rtt == 0

Sir, is it possible to remove this navigator.connection.rtt == 0 flag from Android WebView

I'm not the developer of this project, but the rtt check was removed some time ago, if I remember correctly


Fix hatch handler

fix bulge property. Write bulge value for each point instead of only writing it in the loop level. @skymakerolof  let me know if it is more clear this way or I need to add more explanation.

Thank you!


getAppWidth/getAppHieght have margins in ui

When spawning objects in the either the right or bottom edges of the ui (meaning using either getAppWidth or getAppHieght)  the objects are placed either slightly away from the edge in case of the right or be slightly below the window in case of the bottom.
This was tested with the official pong tutorial app( see included images), as well as personal apps ( see included code).
To Reproduce you can do one of the following:

Run the pong example
Use the following code( if needed i have other examples with the same results):

`
@Spawns("border")
public Entity createBorder(SpawnData data){
    Path path = new Path();

    path.setStroke(Color.RED);
    path.setStrokeWidth(TILE_SIZE); // In my code 30 but could be any value

    ObservableList<PathElement> elements = path.getElements();

    // Sets Start point
    elements.add(new MoveTo(0,0));

    // Creates border lines
    HLineTo top = new HLineTo(getAppWidth()-(TILE_SIZE/2));
    VLineTo right = new VLineTo(getAppHeight()-(TILE_SIZE/2));

    HLineTo bottom = new HLineTo(0);
    VLineTo left = new VLineTo(0);

    // Actually adds the lines
    elements.add(top);
    elements.add(right);
    elements.add(bottom);
    elements.add(left);

    return entityBuilder(data)
            .type(EntityType.BRICK)
            .viewWithBBox(path)
            .collidable()
            .zIndex(-1)
            .build();

}

`
See the right paddle's margin compared to the left

Ubuntu 20.04.2 LTS 64-bit

Thanks for this. I suspect this is an issue related to window handling on Linux as I recall similar problems in the past. I will see if I can reproduce this in a pure JavaFX app, in which case that will suggest the issue comes from JavaFX, rather than FXGL.


Incompatible declaration in Fazland\ElasticaBundle\Elastica\Client:16

php7.0.18
symfony 2.8
fazland/elastica-bundle: dev-master
ruflin/elastica: dev-master
After commit:
https://github.com/ruflin/Elastica/commit/3e3a1affd1b8097ed942ae2364c454ae6872e201
getting warning:
ContextErrorException in Client.php line 16:
Warning: Declaration of Fazland\ElasticaBundle\Elastica\Client::request($path, $method = Elastica\Request::GET, $data = Array, array $query = Array) should be compatible with Elastica\Client::request($path, $method = Elastica\Request::GET, $data = Array, array $query = Array, $contentType = Elastica\Request::DEFAULT_CONTENT_TYPE)

Has been fixed in https://github.com/fazland/FazlandElasticaBundle/commit/795e188afb0cc1f006505bb5ebea246307166450 which added the $contentType optional argument to the method signature.
Are you sure you are using the dev-master version of elastica-bundle?

Yep. i'm using 'dev-master', and update has been called after this commit. But maybe smthng went wrong. Now all is ok after one more update. Thanks.


Cleanup network usage in NBitcoin #2

All instances where we now create Block, BlockHeader and Transaction has to be created by passing in the network's ConsensusFactory all other types does not require it.
This can be reviewed by commit.

All unit and integration tests pass.
Staking working as per normal.
Synced STRAT testnet
Synced BTC testnet up until block 100k

There are a few places where Network was used before and now it's not (nor consensusFactory). And vice-versa.
Can you please review to make sure there is no change in functionality?

@bokobza the only reason we actually have the Network class in bitcoin stream etc is to check the IsProofOfStake setting on Consensus (As well as to create the object). I feel that this needs to be looked at properly.

I'll give this the once-over.

Can you please merge from master and then wait for tests? appveyor is failing for some reason.
I'll be happy to merge afterwards.


Compilation fails on Linux due to inclusion of windows.h

What steps will reproduce the problem?
1. make lin
2.
3.

What is the expected output? What do you see instead?
g++ -c -Iinc -Iexpat  -o obj/a_env.o src/a_env.cpp
In file included from src/a_env.cpp:20:0:
inc/a_win.h:12:21: fatal error: windows.h: No such file or directory

g++ -c -Iinc -Iexpat  -o obj/a_db.o src/a_db.cpp
src/a_db.cpp:11:21: fatal error: windows.h: No such file or directory

What version of the product are you using? On what operating system?
ubuntu 12.04.5
g++ 4.6.3

Please provide any additional information below.

includes of windows.h should be wrapped within #ifdef _WIN32

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 5 Nov 2014 at 3:11

Hi, I have the same error when I try to upload a sketch in arduino. Where can I download the windows.h library? I am a newbie


Replace "Smart Clause" with "Clause Template"

The term "Smart Clause" is the IP of another company. We have to change all instances of "Smart Clause" to "Clause Template"

I want to work on this issue

@arteevraina thanks I've assigned it to you


Binding against SelectedText

    <abstractions:SegmentedControl SelectedText="{Binding SwitchValue}">
No property, bindable property, or event found for 'SelectedText'

What do you want to do with this?

I assumed we could use this instead of binding against the index to just get the text of whatever cell is selected.

Nope. Thats not the functionality of that property. I was planing to remove it.

Your call. I could see it being useful. If it shouldn't work though, you should remove it!

Hello sir. Can you show us about SegmentedControl binding items and selected text change? Because I try to use it with mvvm pattern. So please help me.

This property has been removed. Instead you will be able to bind to SelectedSegment from MVVM.


[ci] Add lint check for newlines

See #11560, CI should enforce that every file uses LF and has a trailing newline aside from exceptional cases
cc @Mousius @areusch

cc @hpanda-naut can you triage?


Playlist name

Hello,
Thank you, you did a great job;)
would it be possible to add a name to the playlist, example in the image
https://www.awesomescreenshot.com/image/3398659/3219a3900af44cc70ceb0e13679b84ea

Thx. Theoretically yes, but would require some work to make it intuitive, responsible and easy managable. I unfortunetely don't have time for this project, currently.


Troposphere library "AttributeError" fix and python3.9 runtime support

Description
zappa/core.py
method "add_description" in core.py line 2417 doesn't exists in the latest troposphere library, method "set_description" takes it's place. This change allows for "zappa deploy" and "zappa update" without errors.
GitHub Issues
https://github.com/zappa/Zappa/issues/1016
https://github.com/zappa/Zappa/issues/1002
https://github.com/zappa/Zappa/issues/1000

This would be huge!

This would be LIT 🔥

Yes, please! The old solution we had was to pin tropo to 2.7.0 but with the latest major version of setuptools (58.0+) conflicts with troposphere 2.7.0 because setuptools dropped support for use2to3 (issue). We can always pin setuptools to <58, but this would be a much better alternative. 🙌

Thanks for this fix @ghostmaster-ai, I just switched to your fork until a new official release is published

It isnt the same as this one? https://github.com/zappa/Zappa/pull/1026

Can some one look into this and close the PR? We are stuck with this!!! :(

Hey everyone, a team of contributors is back to support the amazing Zappa project.
The short term goal is to bring back CI releases and merge urgent bug fixes this week.
This would allow everyone to continue (or come back) to using this repo.
https://github.com/zappa/Zappa/pull/1026 is the active PR to watch for Python 3.9 support
I'm closing this PR as it's not relevant anymore


[tests-only][full-ci]Extend tests related to assign role

Description
This PR expand tests coverage for assign role
Related Issue

https://github.com/owncloud/ocis/issues/5564

Motivation and Context

How Has This Been Tested?

locally

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[x] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[x] Acceptance tests added
[ ] Documentation ticket raised:

LGTM information_source Also, remember that we need to get these php code changes to the stable as well. Probably after #5671

@saw-jan  Why backport unnecessary PHP code to a stable branch?


The units of the velocity passed into the GestureDetector's onDragEnd callbacks is not specified/documented.

Should be logical pixels per second, fwiw.

If we had a Velocity type that supported units, like Duration does, then reporting and comparing velocities might be a little easier.
See also: https://github.com/flutter/flutter/issues/785


adds possibility to issue custom HTTP requests

Implements feature request #180.

+1 would be helpful to do some tests with our http proxies.

We're currently debating whether or not we want to add this functionality to capybara-webkit. If enough people request the functionality we'll probably be for it.

If capybara-webkit is faster than rack-test then then I'd want to use it for the whole suite, including API tests.

@mike-burns: Which test suite are you talking about?

Ah sorry, I mean that this would be useful for my app's test suites.

As mentioned in #180, we haven't run into a desire for this, and we don't want to maintain code we don't use.
Closing, even though it seems like a cool feature.
Thanks,
-Mike


use SVG diagram instead of Cytoscape.js visualization

Currently we are using Cytoscape.js to visualize the graph like this:

However a much better visualization would work directly with the HITO diagram, which we soon release in version 24.03 in issue https://github.com/hitontology/hitontology.eu/issues/46.
This is quite tricky but should be doable.
Cytoscape.js would still be used in the background.

This is the new default approach now.


Add drag and drop functionality in ProgramSetting and FileSystemSettings

Add drag and drop functionality in ProgramSetting and FileSystemSettings
Signed-off-by:<PHONE_NUMBER>88@qq.com

Please try to send this pull request to V1.2.0 branch, not master. We removed Wox.Plugin.SystemPlugins in v1.2.0.
Also, please use english as commit messages. Thx.

Ok


Add NET_ADMIN capability to fix flannel on bare metal FCOS

Without the
capabilities:
  add: ["NET_ADMIN"]

services in the Kubernetes cluster are broken. For example, when accessing Kubernetes dashboard
Error trying to reach service: 'dial tcp <IP_ADDRESS>:8443: i/o timeout'

Running Typhoon with v1.18.5 Kubernetes on bare metal Fedora CoreOS cluster.
After adding NET_ADMIN capability and waiting for flannel pods restarts, services are working again.
Default template also contains this capability:
https://github.com/coreos/flannel/blob/master/Documentation/kube-flannel.yml

After reboot of the nodes it is broken again. Triggering the restart of the flannel pods fixes it. Need more investigation.


Update apacheds-all version 1.5.5 to 2.0.0-M24

Purpose
To update apacheds-all version to a later version which used in LDAP integration tests.
Goals
Make sure LDAP authstore work with latest apache DS.
Approach
Update apacheds dependancy used in integration test to a newer version.
Automation tests

Integration tests

yes

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Codecov Report

Merging #13842 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #13842      +/-   ##
============================================
- Coverage     64.49%   64.48%   -0.02%
  Complexity      426      426
============================================
  Files          2154     2154
  Lines         95396    95396
  Branches      12706    12706
============================================
- Hits          61530    61520      -10
- Misses        28854    28861       +7
- Partials       5012     5015       +3

Impacted Files
Coverage Δ
Complexity Δ

...allerinalang/net/jms/LoggingExceptionListener.java
50% <0%> (-50%)
0% <0%> (ø)

...rinalang/runtime/threadpool/ThreadPoolFactory.java
61.9% <0%> (-4.77%)
0% <0%> (ø)

...a/org/ballerinalang/bre/bvm/WorkerDataChannel.java
83.95% <0%> (-3.71%)
0% <0%> (ø)

...va/org/ballerinalang/channels/ChannelRegistry.java
94.59% <0%> (-2.71%)
0% <0%> (ø)

Continue to review full report at Codecov.

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


[3702328964] Feature/models url tests

3702328964
Description

removed the test for cloudfront URL avatars and replaced with avatar API url's and Model URLs.
lots of refactoring
fixed some broken tests
removed old legacy tests that seem to have no relevance now.

Changes
Added

Tests for avatar API and Model URL's

Updated

general refactoring of avatar loader tests

How to Test

Add steps to locally test these changes

Checklist

[ ] Tests written or updated for the changes.
[ ] Documentation is updated.
[ ] Changelog is updated.
[ ] QA Testing is updated.

QA Testing

Mention any useful test cases for this feature and add to QA Testing documentation.

Details

If more details are necessary to support the description with images and videos add them here.

🔴 I am getting errors

yep my bad there were a lot of errors actually as I forgot to push some changes :D
But it should all be fixed now


fix(ndt_scan_matcher): fix to pass clang-tidy's check

Description
(1)
To avoid warning: function 'callback_sensor_points_main' has cognitive complexity of 26 (threshold 25) [readability-function-cognitive-complexity], I changed from for statement to stl algorithm.
This change was made only to avoid warnings, and it is possible that readability has not improved but has actually decreased. I think it would be reasonable for this pull request to be judged as Reject on this reason. There is a possibility that it is better to properly separate the big function callback_sensor_points_main.
(2)
To solve warning: do not access members of unions; use (boost::)variant instead [cppcoreguidelines-pro-type-union-access], I added // NOLINT to those lines.
Tests performed
logging_simulator works well as before.
Effects on system behavior
There are no effects on system behavior.
Interface changes
There are no interface changes.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

This change was made only to avoid warnings, and it is possible that readability has not improved but has actually decreased. I think it would be reasonable for this pull request to be judged as Reject on this reason.

Honestly, I think the source code has become harder to understand :cry:

Thank you for your comment.
I will fix this pull request because I found it difficult to read when I read the changes again.


Unexpected Error Occurred while Connecting

Issue Type: Bug
[2022-11-04 10:13:09.082] [main] [info] update#setState idle
[2022-11-04 10:13:39.087] [main] [info] update#setState checking for updates
[2022-11-04 10:13:39.620] [main] [error] UpdateService error: Cannot update while running on a read-only volume. The application is on a read-only volume. Please move the application and try again. If you're on macOS Sierra or later, you'll need to move the application out of the Downloads directory. See https://github.com/Squirrel/Squirrel.Mac/issues/182 for more information.
[2022-11-04 10:13:39.620] [main] [info] update#setState idle
Azure Data Studio version: azuredatastudio 1.30.0 (59c4b8e90cf2d3a24bed72623197f10f8d090fdc, 2021-06-16T18:57:01.467Z)
OS version: Darwin x64 21.6.0

System Info

Item
Value

CPUs
Apple M1 (8 x 2400)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
1, 2, 2

Memory (System)
16.00GB (0.03GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

sql-migration
Mic
0.1.11

@emanuelaguna
It seems you may be running ADS from the Downloads directory.. Can you try moving the installation location to a different directory? Also, 1.30.1 seems to be a very old version of ADS, could you please try updating to a newer version as well?

Closing issue as inactive and possibly due to running ADS from Downloads directory.


module initialization error, lambda to sagemaker?

i am trying this tutorial
https://aws.amazon.com/blogs/machine-learning/call-an-amazon-sagemaker-model-endpoint-using-amazon-api-gateway-and-aws-lambda/
whenever i try the lambda function, i get this error
module initialization error: 'ENDPOINT_NAME'
this is the code i wrote
`import os
import io
import boto3
import json
import csv
grab environment variables
runtime= boto3.client('runtime.sagemaker')
ENDPOINT_NAME = os.environ['factorization-machines-2019-03-23-19-41-56-103']  //problem is right here.
def lambda_handler(event, context):
// rest of the handler
`

@ilyes-hamrouni
Thanks for using SageMaker!
The issue you describe seems to be a problem with the lambda code in the blog post, not the code in the example notebook here in this repo. The blog post has a comment feature, and I see the author has been responding to other users' questions there. I suggest you repost your question there.
I am closing this issue. Feel free to open a new one if you run into trouble with the notebook!

@ilyes-hamrouni
the line you are having trouble with should be:
ENDPOINT_NAME = os.environ['ENDPOINT_NAME']

if you have endpoint created then this line is supposed to be:
ENDPOINT_NAME = '<name_of_endpoint>' not os.environ['ENDPOINT_NAME']


Missing support for private certificate authorities

The docker command fails to get local issuer certificate, because...
Our GitLab server is signed by our company Root CA and Signing CA which the pr-agent doesn't know.
How can we make the pr-agent to trust our company Root CA and Signing CA ?
Thanks for your help

OK, I found a workaround, reusing the cacerts of my local ubuntu system, as pr-agent is using python3.
cacerts=/etc/ssl/certs/ca-certificates.crt
docker run -it --rm -v $cacerts:$cacerts:ro -e REQUESTS_CA_BUNDLE=$cacerts ...

It'd be a bit less cumbersome having a CLI param to skip TLS verify :smile:


Add a section for recent project activity

Using the pulls_since tool makes it so that we get news even when folks forget to mention it in the tracking issue, and it also gives credit to the PR author with @username.
See https://github.com/budziq/pulls_since for details, or run cargo install pulls_since to install it.

Thanks @fitzgen 👍


Wrong writing leads to loss of 'MayRef' responsiveness

unref should be placed in computed
like:

export function useMotionVariants<T extends MotionVariants>(variants: MaybeRef<T> = {} as MaybeRef<T>) {
  // Unref variants
   //↓↓↓↓↓↓↓
  // const _variants = unref(variants) as T

  // Current variant string
  const variant = ref() as Ref<keyof T>

  // Current variant state
  const state = computed<Variant | undefined>(() => {
    if (!variant.value) return
    //↓↓↓↓↓↓↓
    const _variants = unref(variants) as T
    return _variants[variant.value]
  })

  return {
    state,
    variant,
  }
}

Are you experiencing any issues related to the code you're talking about? Could you please provide a reproduction so I can look into it 🙏


Refactor preprocess_text and preprocess_html functions as class method;

It doesnt make sense to extend each class with preprocess methods when it doesnt work for native extensions;

Done!


get_pull_request_iteration_changes only return change_tracking_id

I try to use get_pull_request_iteration_changes  to get files that are change in this iteration as shown here.
I copy and paste the desired output from the documentation below.
{
"changeEntries": [
{
"changeTrackingId": 1,
"changeId": 1,
"item": {
"objectId": "e21e56d119ae81fb4ffebc4fefc6351f5b5ef888",
"path": "/new_feature.cpp"
},
"changeType": "add"
},
{
"changeTrackingId": 2,
"changeId": 2,
"item": {
"objectId": "5ec0f71ffb8b47bd4c0117f624647963e021f3d2",
"path": "/new_feature.h"
},
"changeType": "add"
}
]
}
However, when I use get_pull_request_iteration_changes, I can only see changeTrackingId. I wonder if what is the proper way to get above schema, particular the path of the item.
 client.get_pull_request_iteration_changes(repo_id, pull_request_id, 2, compare_to=1).as_dict()
{'change_entries': [{'change_tracking_id': 1}, {'change_tracking_id': 4}]}

Try this:
git_client = connection.clients_v5_0.get_git_client()
changes = git_client.get_pull_request_iteration_changes(repo_id, pr_id, 2, compare_to=1)
for change in changes.change_entries:
    pprint.pprint(change.additional_properties['item']['path'])

I am going to close this issue.  Please let me know if the above code sample was not helpful.


Allow for stdout to return the data in chunks

When I ran any ps-node command on windows it was failing with a undefined is not a function because output from the command list empty.
Tracked it down to the fact that the data piped in from wmic command is not guaranteed to come in line by line, but the exit function that finds the beginRow expects each line to be on its own. This fix splits the data as it comes in to fix this bug.

published in 0.0.5


orchestrator: Refactor into distinct packages

Before this change, all orchestration-related components were in the
same package, even though there were several independent pieces. They
shared code in sloppy ways, like the global orchestrator calling code in
replicated.go.
This refactors them into multiple pacakages:

manager/orchestrator/constraintenforcer
manager/orchestrator/global
manager/orchestrator/taskreaper
manager/orchestrator/replicated
manager/orchestrator/restart
manager/orchestrator/update

manager/orchestrator has a few shared helper functions, but very little
code stayed there. Also, there is a testutils package for shared testing
functions.
Note that a lot of the tests ended up under
manager/orchestrator/replicated since they depend on the replicated
orchestrator to exercise a different component (such as the task reaper
or restart manager).
cc @aluzzardi @dongluochen

Current coverage is 56.31% (diff: 78.43%)

No coverage report found for master at 5d0192b.

Powered by Codecov. Last update 5d0192b...1c14bc0

LGTM

LGTM


Jest snapshot and auto generated inputs(generative tests)

Do you want to request a feature or report a bug?
Feature
What is the current behavior?
I'm trying to use the built in generative testing capabilities of Jest( check.it ) together with snapshot testing, along the lines:
check.it('Should add numbers together', [gen.int, gen.int], (a,b) => {
    expect(sum(a,b)).toMatchSnapshot();
});

They work beautifully together and for each input I do get a correponding snapshot:
exports[`test Should add numbers together 1`] = `0`;

exports[`test Should add numbers together 2`] = `-1`;

exports[`test Should add numbers together 3`] = `-2`;
.........
.........

The only issue and the reason for opening up this discussion is that's impossible to see the input arguments in the snapshot file thus making it impossible to use them together in a reliable way.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal repository on GitHub that we can npm install and npm test.
What is the expected behavior?
I propose to allow the toMatchSnapshot matcher to receive a message as param and use them when generating the snapshot description. Eg:
    check.it('Should add numbers together', [gen.int, gen.int], (a,b) => {
    expect(sum(a,b)).toMatchSnapshot(`when arguments have values ${a} and {$b}`);
});

Should generate snapshots looking like:
  exports[`test Should add numbers together 1 - when arguments have values 1 and  -1`] = `0`;

Basically concatening the test description with the toMatchSnapshot message to form the full message.
Advantages:

More context when reviewing the snapshot files
Ability to use generative tests and data generators to also automate the input generation for React components, Redux reducers etc .  This is huge as would allow both the input and the output of a test to be generated and asserted automatically requiring only carefull review of the snapshot file.

Run Jest again with --debug and provide the full configuration it prints. Please mention your node and npm version and operating system.

Duplicate of #1392.


mmcsd: move block driver register to probe stage in order to read ext_csd reg

Summary
mmcsd: move block driver register to probe stage in order to read ext_csd reg
Impact
mmcsd
Testing
cortext M55

[Experimental Bot, please feedback here]
No, this PR does not meet the NuttX Requirements.
Missing Information:

Summary:

A clear explanation of why this change is necessary is missing. What problem does moving the block driver registration solve?
How does moving the registration to the probe stage enable reading the ext_csd register? Provide a technical explanation.

Impact:

The impact section is far too brief. Address all the bullet points (user impact, build impact, etc.) with "YES" or "NO" and provide details where applicable. Even if there's no impact, state "NO" explicitly for clarity.

Testing:

Insufficient Detail: "cortext M55" is not enough information.  Specify:

Host OS: (e.g., Linux Ubuntu 20.04)
Compiler Version: (e.g., GCC 10.3.0)
NuttX Configuration: (e.g., configs/cortex-m55/nsh)

Missing Logs: Provide actual testing logs from before and after the change to demonstrate the issue and the fix.

Recommendations:

Elaborate on the "Why": Clearly state the problem that this change addresses. Why is reading the ext_csd register important, and why was it not possible before?
Explain the Technical Details:  Describe the code changes and how they enable reading the ext_csd register.
Complete the Impact Section:  Carefully consider and address all impact points, even if the answer is "NO".
Provide Detailed Testing Information: Include specific host and target details, along with actual testing logs demonstrating the problem and the solution.

@gneworld please fix(https://github.com/apache/nuttx/actions/runs/11084304188/job/30799485530?pr=13705):
mmcsd/mmcsd_sdio.c: In function 'mmcsd_slotinitialize':
Error: mmcsd/mmcsd_sdio.c:4367:47: error: 'devname' undeclared (first use in this function); did you mean 'rename'?
 4367 |   finfo("MMC: %s %" PRIu64 "KB %s %s mode\n", devname,
      |                                               ^~~~~~~

@gneworld please fix(https://github.com/apache/nuttx/actions/runs/11084304188/job/30799485530?pr=13705):
mmcsd/mmcsd_sdio.c: In function 'mmcsd_slotinitialize':
Error: mmcsd/mmcsd_sdio.c:4367:47: error: 'devname' undeclared (first use in this function); did you mean 'rename'?
 4367 |   finfo("MMC: %s %" PRIu64 "KB %s %s mode\n", devname,
      |                                               ^~~~~~~

@xiaoxiang781216
done