MLPP versus Pytorch C++

Great work! Very interesting!
In the README, you say that MLPP serves to revitalize C++ as a machine learning front-end. How does MLPP separate itself from the Pytorch C++ API? If you don't mind me asking, why not build wrappers around the already open-source and highly optimized Pytorch C++ code?
Thanks!

In the README, he also introduced

The intent with this library is for it to act as a crossroad between low-level developers and machine learning engineers.

Personally, I think that the author makes this repo to reconstruct ML algorithms, beyond NeuralNet stuffs. The key thing is that he explicitly reimplemented on pure C++, while LibTorch (PyTorch C++ API) is partially inherited Caffee framework and is extended with apex repo from NVIDIA (a bunch of dependencies).
Anyway, this is a great work and interesting for both education and research purposes towards for C++ programmers and enthusiasts.

Thank you for your thoughts @hungnphan, however my intent with this issue was not to poll people's opinion on MLPP but instead to understand the reason why MLPP was not built on top of Pytorch C++.
The relevant point you make is that Pytorch uses external libraries which is a purported drawback. However, the Pytorch C++ dependencies can all be installed directly through Conda without root privileges, so it's a non-issue.

Thank you for your thoughts @hungnphan, however my intent with this issue is not to poll people's opinion on MLPP but instead to understand why it was not built on top of Pytorch C++.
Your relevant point is that Pytorch C++ is built on top of external libraries which is a purported drawback. However, Pytorch C++ dependencies can be installed directly through conda without root privelages, so it's a non-issue.

Mon response
Hello @towermitten,
Thank you for the kind sentiment.
Regarding the distinction between PyTorch’s C++ API and ML++, the main differences lie in user support and syntax.
In terms of support for C++, PyTorch has glaring issues- various main functions existent in the Python API are absent, the documentation is lacking and does not contain as many examples, and not many people are willing to contribute. This of course is understandable as machine learning is primarily dominated by Python. ML++, meanwhile, is specifically designed to cater to C++ developers and low level engineers, and so emphasis will be given to C++ as opposed to other languages, such as Python.
Secondly, another significant drawback present in the C++ API in PyTorch is its complexity. The following is a standard end-to-end implementation of an artificial neural network in the PyTorch C++ API, via https://pytorch.org/cppdocs/frontend.html:
#include <torch/torch.h>

// Define a new Module.
struct Net : torch::nn::Module {
  Net() {
    // Construct and register two Linear submodules.
    fc1 = register_module("fc1", torch::nn::Linear(784, 64));
    fc2 = register_module("fc2", torch::nn::Linear(64, 32));
    fc3 = register_module("fc3", torch::nn::Linear(32, 10));
  }

  // Implement the Net's algorithm.
  torch::Tensor forward(torch::Tensor x) {
    // Use one of many tensor manipulation functions.
    x = torch::relu(fc1->forward(x.reshape({x.size(0), 784})));
    x = torch::dropout(x, /*p=*/0.5, /*train=*/is_training());
    x = torch::relu(fc2->forward(x));
    x = torch::log_softmax(fc3->forward(x), /*dim=*/1);
    return x;
  }

  // Use one of many "standard library" modules.
  torch::nn::Linear fc1{nullptr}, fc2{nullptr}, fc3{nullptr};
};

int main() {
  // Create a new Net.
  auto net = std::make_shared<Net>();

  // Create a multi-threaded data loader for the MNIST dataset.
  auto data_loader = torch::data::make_data_loader(
      torch::data::datasets::MNIST("./data").map(
          torch::data::transforms::Stack<>()),
      /*batch_size=*/64);

  // Instantiate an SGD optimization algorithm to update our Net's parameters.
  torch::optim::SGD optimizer(net->parameters(), /*lr=*/0.01);

  for (size_t epoch = 1; epoch <= 10; ++epoch) {
    size_t batch_index = 0;
    // Iterate the data loader to yield batches from the dataset.
    for (auto& batch : *data_loader) {
      // Reset gradients.
      optimizer.zero_grad();
      // Execute the model on the input data.
      torch::Tensor prediction = net->forward(batch.data);
      // Compute a loss value to judge the prediction of our model.
      torch::Tensor loss = torch::nll_loss(prediction, batch.target);
      // Compute gradients of the loss w.r.t. the parameters of our model.
      loss.backward();
      // Update the parameters based on the calculated gradients.
      optimizer.step();
      // Output the loss and checkpoint every 100 batches.
      if (++batch_index % 100 == 0) {
        std::cout << "Epoch: " << epoch << " | Batch: " << batch_index
                  << " | Loss: " << loss.item<float>() << std::endl;
        // Serialize your model periodically as a checkpoint.
        torch::save(net, "net.pt");
      }
    }
  }
}

The following is a very similar implementation, except in the ML++ framework:
int main() {
	MANN ann(inputSet, outputSet); // Input size implicitly 784- final output size implicitly 10
	ann.addLayer(64, “RELU”); // Hidden unit number, activation function
	ann.addLayer(32, “RELU”);

	ann.addOutputLayer(“Softmax , “CrossEntropy”);

	ann.MBGD(0.01, 10, 64, 1); // LR, epoch number, batch size, UI panel
}

As can be seen, implementing ML algorithms for C++ in already existing APIs is time consuming and verbose (given that the 2nd version was a lot shorter). ML++ tries to alleviate this by offering simpler implementations.
Regarding your second question on why I don’t simply utilize PyTorch implementations, as @hungnphan has pointed out, using tools such as PyTorch supported matrix operations, neural network packages etc. would abstract a lot of the key details of various machine learning algorithms, and I was interested in getting into the particulars of them.
Because of this, the speed of execution is an issue, and to try to ameliorate this, I now am looking into vectorization strategies.

Lack of support in Pytorch C++
Some of the issues you raise about Pytorch C++ are:

Documentation is lacking
Many missing implementations
Not many people are willing to contribute
The Pytorch development team is focused on Python, not C++

I agree with all of these specific points, but I feel that all of them could be just as easily addressed by building wrappers around Pytorch C++ all the while not sacrificing performance. This is exactly what Pytorch Lightning does (for Python).
Complicated Syntax of Pytorch C++
I don't believe your code-level comparison is very fair. The Pytorch C++ code does more than yours does:

Pytorch doesn't auto-generate a forward() method, and therefore doesn't force the model to use all layers sequentially (e.g. a ResNet: the most common deep architecture, or RNNs). If you did want this, you can use torch:nn:Sequential or just use a pre-trained AlexNet. It can also take arbitrary number of inputs.
The Pytorch code is specifying a dataset by name, and it is loaded using multithreading
The Pytorch code uses dropout

It is true that many of the other lines rarely change, even in (Python) Pytorch, which is why the Python-Pytorch wrapper library Pytorch Lightning was built.
Comments
Also regarding your last comment

the speed of execution is an issue, and to try to ameliorate this, I now am looking into vectorization strategies.

With all due respect, this is a massive understatement. Vectorizing your code, even if you use multithreading, will not begin to approach Pytorch C++'s performance on GPU. The only way to do this is to write CUDA kernels (which is doable, you could extend MLPP in this way). Also, Pytorch C++ can be distributed across multiple GPUs using a single line of code, or across multiple processes in a distributed setting. Deep learning beyond educational toy examples will require GPU utilization.
Also, you say

using tools such as PyTorch supported matrix operations, neural network packages etc. would abstract a lot of the key details of various machine learning algorithms, and I was interested in getting into the particulars of them.

That's reasonable, but MLPP purports to be focused specifically on revitalizing C++ as a machine learning front end. So the details/particulars of the machine learning algorithms shouldn't be relevant.

i think that the MLPP is not used for production level applications，it is for those c++ developers who are interested in Machine Learning.


Master -> Live

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

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/case-study-spatial-sound-design-for-holotour.md
:white_check_mark:Succeeded
View

mixed-reality-docs/directx-development-overview.md
:white_check_mark:Succeeded
View

mixed-reality-docs/eye-tracking.md
:white_check_mark:Succeeded
View

mixed-reality-docs/gaze-in-directx.md
:white_check_mark:Succeeded
View

mixed-reality-docs/hololens-known-issues.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound-design.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound-in-directx.md
:white_check_mark:Succeeded

mixed-reality-docs/spatial-sound-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound.md
:white_check_mark:Succeeded
View

mixed-reality-docs/voice-input-in-directx.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 703d5ba:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/case-study-spatial-sound-design-for-holotour.md
:white_check_mark:Succeeded
View

mixed-reality-docs/directx-development-overview.md
:white_check_mark:Succeeded
View

mixed-reality-docs/eye-tracking.md
:white_check_mark:Succeeded
View

mixed-reality-docs/gaze-in-directx.md
:white_check_mark:Succeeded
View

mixed-reality-docs/hololens-known-issues.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound-design.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound-in-directx.md
:white_check_mark:Succeeded

mixed-reality-docs/spatial-sound-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound.md
:white_check_mark:Succeeded
View

mixed-reality-docs/voice-input-in-directx.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit f540b1c:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/gaze-in-directx.md
:warning:Warning
View
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/case-study-spatial-sound-design-for-holotour.md
:white_check_mark:Succeeded
View

mixed-reality-docs/directx-development-overview.md
:white_check_mark:Succeeded
View

mixed-reality-docs/eye-tracking.md
:white_check_mark:Succeeded
View

mixed-reality-docs/hololens-known-issues.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound-design.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound-in-directx.md
:white_check_mark:Succeeded

mixed-reality-docs/spatial-sound-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound.md
:white_check_mark:Succeeded
View

mixed-reality-docs/voice-input-in-directx.md
:white_check_mark:Succeeded
View

mixed-reality-docs/gaze-in-directx.md

Line 171: [Warning] Cannot find bookmark '#fallback-solutions-when-eye-tracking-is-not-available' in 'eye-tracking.md'
Line 182: [Warning] Cannot find bookmark '#fallback-solutions-when-eye-tracking-is-not-available' in 'eye-tracking.md'

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 52f3622:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/gaze-in-directx.md
:warning:Warning
View
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/case-study-spatial-sound-design-for-holotour.md
:white_check_mark:Succeeded
View

mixed-reality-docs/directx-development-overview.md
:white_check_mark:Succeeded
View

mixed-reality-docs/eye-tracking.md
:white_check_mark:Succeeded
View

mixed-reality-docs/hololens-known-issues.md
:white_check_mark:Succeeded
View

mixed-reality-docs/openxr.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound-design.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound-in-directx.md
:white_check_mark:Succeeded

mixed-reality-docs/spatial-sound-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound.md
:white_check_mark:Succeeded
View

mixed-reality-docs/voice-input-in-directx.md
:white_check_mark:Succeeded
View

mixed-reality-docs/gaze-in-directx.md

Line 171: [Warning] Cannot find bookmark '#fallback-solutions-when-eye-tracking-is-not-available' in 'eye-tracking.md'
Line 182: [Warning] Cannot find bookmark '#fallback-solutions-when-eye-tracking-is-not-available' in 'eye-tracking.md'

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit b538c9e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/gaze-in-directx.md
:warning:Warning
View
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/case-study-spatial-sound-design-for-holotour.md
:white_check_mark:Succeeded
View

mixed-reality-docs/directx-development-overview.md
:white_check_mark:Succeeded
View

mixed-reality-docs/eye-tracking.md
:white_check_mark:Succeeded
View

mixed-reality-docs/hololens-known-issues.md
:white_check_mark:Succeeded
View

mixed-reality-docs/open-source-projects.md
:white_check_mark:Succeeded

mixed-reality-docs/openxr.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound-design.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound-in-directx.md
:white_check_mark:Succeeded

mixed-reality-docs/spatial-sound-in-unity.md
:white_check_mark:Succeeded
View

mixed-reality-docs/spatial-sound.md
:white_check_mark:Succeeded
View

mixed-reality-docs/voice-input-in-directx.md
:white_check_mark:Succeeded
View

mixed-reality-docs/gaze-in-directx.md

Line 171: [Warning] Cannot find bookmark '#fallback-solutions-when-eye-tracking-is-not-available' in 'eye-tracking.md'
Line 182: [Warning] Cannot find bookmark '#fallback-solutions-when-eye-tracking-is-not-available' in 'eye-tracking.md'

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Support Service Broker API 2.12

http://docs.cloudfoundry.org/services/api.html#changes-since-v211

Add context field to request body for provisioning and updating a service instance

Note that while the latest CF release (1.3.0) is sending the 2.12 context field it is still sending 2.11 in the X-Broker-Api-Version header. This was by design as we were waiting on consensus in the working group.
That has now happened but we are waiting for this PR to be accepted.

@Samze your PR is merged. I am happy to create a PR that implements 2.12 related changes.

Sounds good @mcelep. Yeah the OSBAPI 2.12 spec has been released.

@Samze @mcelep I have some WIP changes for the context field change. I'm hoping to get those changes done this week.

Fixed in https://github.com/spring-cloud/spring-cloud-cloudfoundry-service-broker/commit/8e5d87fad64e6c281afef0923d476a24fb629c65


StaleElementRecoveryException when sending a form by pressing ENTER in a text field

When sending a form (i.e. a search query in Jira) by pressing enter using our TextField's append method (basically WebElement#sendKeys(...)) a stale element recovery exception is thrown. This happens when the text of the field is read in order to generate the text changed event.
Issue came up during a workshop when trying to implement a search for our Cloud Jira instance:
https://github.com/testIT-WebTester/webtester-examples/blob/workshop-2015-12-04/workshop-2015-12-04/src/test/java/workshop/IssueCreationTest.java
Doesn't work:
public SearchResultPage searchForIssue(String query){
	quickSearch.setText(query).appendText(Keys.ENTER.toString());
	return create(SearchResultPage.class);
}

Works:
public SearchResultPage searchForIssue(String query){
	quickSearch.setText(query);
	quickSearch.getWebElement().sendKeys(Keys.ENTER);
	return create(SearchResultPage.class);
}

Simples solution would be to add an additional method to the TextField class:
public SearchResultPage searchForIssue(String query){
    quickSearch.setText(query).pressEnter();
    return create(SearchResultPage.class);
}

I also would add an appropriate warning to the JavaDoc of setText(...) and appendText(...) stating that these methods are not intended to send keys like WebElement#sendKeys(...).
@benhamidene @PMO-IT @andifalk Feedback?


Improvement: add Argument Completers to cmdlet parameters

To enhance the ease of use of the cmdlets, let's add argument completers to as many parameters as makes sense.  Like, auto-completer values for:

-Cluster
-VMHost
-VM
-Name for VMGroup, VMHostGroup, Rules, etc.
-AffineHostGroupName, -AntiAffineHostGroupName
..so on

Completed in 49fef76. Hurray!


Tutorial: Review and revise driving and acoustic pinger tutorials

Review the Driving and Acoustic pingers tutorials, linked here:
https://github.com/osrf/vrx/wiki/vrx_api_tutorials

Revised.


Removed condenscened language from settings

Fix #1394

CI checks are failing @kurianbenoy please fix formatting as explained in https://dvc.org/doc/user-guide/contributing/docs. Contact me with questions or if you need help.

Ping @kurianbenoy

@andronovhopf please just double check the few changes to blog articles. Thanks

@kurianbenoy before we do more changes - please, address @jorgeorpinel 's review and let's fix CI.

I have addressed comments of @jorgeorpinel reviews, I have only made changes to the first 10 files reviewed. I will make more changes soon.

Closing as stale, feel free to reopen, fix conflicts, address feedback.

Sorry, I was just busy with the DVC 1.0 release and couldn't get back to reviewing this. But it's still a valid PR.

It's going to require lots of conflict solving though. @kurianbenoy do you want to resolve those conflicts or rather branch off master again and re-apply all of these changes?

@jorgeorpinel creating a new branch off master might be a better idea. I agree it's good time to close this PR and reopen as a new PR again

Hi @kurianbenoy,
Are you having issues resetting your fork's master branch and re-applying your changes? It would probably be a better idea not to use master next time, BTW. For situations like this one.
Feel free to reach out to me if you need help with Git.


add documentation and save everything in function

hi @MimiYuchenGuo lots of work going on here, thank you! Could you please address the comments at some point? i will merge it any way so everyone can contribute and we have the package there


Add reduce sum to the GPU transformer (op::Sum)

What this PR does:

Enables sum reduce for tensors of arbitrary tensor size.

For tensors of <= 4-dimensions, the built in cudnn 4d tensor descriptor is used as the cudnn api suggests it is optimized for smaller tensors.
For tensors of greater than 4 dimensions the Nd tensor descriptor is used.

Added 5d tensor to scalar unit test.

Limitations:

Only supports float

TESTNOW


Use preserve-symlinks to solve React singleton issue

React singleton issue happen when multiple React versions detected on server runtime (this is requirement for new React context and hook), usually not happening on client-side since Webpack combine to single bundle, unlike server-side where don't bundle server modules.
This only happen on development since peerDependencies are not installed by Lerna and instead we specify them as devDependencies, that's where the issue happen. "react" should only installed on app-side, not only library-side.
This issue should not affect anything on production side since on user-side, they should only have one package.json and one "react" defined

Codecov Report

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

@@             Coverage Diff             @@
##           master     #179       +/-   ##
===========================================
- Coverage     100%   27.69%   -72.31%
===========================================
  Files           3       17       +14
  Lines          59      278      +219
  Branches       13       47       +34
===========================================
+ Hits           59       77       +18
- Misses          0      169      +169
- Partials        0       32       +32

Impacted Files
Coverage Δ

...ackages/cra-universal/src/config/webpack.config.js
0% <ø> (ø)

<EMAIL_ADDRESS>100% <0%> (ø)

packages/cra-universal/src/util/log.js
0% <0%> (ø)

packages/cra-universal/src/index.js
0% <0%> (ø)

packages/cra-universal/src/config/server/index.js
0% <0%> (ø)

packages/cra-universal/src/sample-module.js
100% <0%> (ø)

<EMAIL_ADDRESS>100% <0%> (ø)

packages/cra-universal/src/cli/start.js
0% <0%> (ø)

packages/cra-universal/src/util/path.js
0% <0%> (ø)

... and 5 more

Continue to review full report at Codecov.

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


[FLINK-15493][test] Add retry rule for all tests based on KafkaTestBase

Unchanged backport of #17973

@flinkbot run azure


Protocol inheritance and composition

Hello.
Is there a way to mocking protocols  that combine multiple protocols?
I try some like this, but got the error.
protocol ServiceA {
    func fooA()
}

protocol ServiceB {
    func fooB()
}

@Mockable
protocol SummaryService: ServiceA, ServiceB {}

Type 'MockSummaryService' does not conform to protocol 'ServiceA'
Type 'MockSummaryService' does not conform to protocol 'ServiceB'

Hey @VisualDeceit!
Unfortunately protocol inheritance is not supported. This limitation stems from the nature of peer macros: you simply have no access to any information about the protocols you inherit from so there is no way of conforming to them automatically.

I know its not ideal, but you can conform to the inherited protocols in an extension and still use the generated mock implementation for testing:
extension Mock SummaryService {
    func fooA() {}
    func fooB() {}
}


Added some icons. Added js to the hero block to group buttons for flexible styling

As a test for getting flexbox styling to work, I added custom js to hero block to move buttons into their own span, so flexbox can be applied. I don't know that this is the way we want to go for future styling but I wanted to make sure it was possible.
Test URLs:

Before: https://main--jmp-da--jmphlx.hlx.live/data-analysis-software
After: https://feature-hero-button-styling--jmp-da--jmphlx.hlx.page/data-analysis-software

URL for testing:

https://feature-hero-button-styling--jmp-da--jmphlx.hlx.page/data-analysis-software

Looks good!


Make quiet mode

Use quiet mode by default for service and quick look, but have standard script give more, better output

This ticket should not be addressed until after #66 is resolved

Make standard operation quiet mode, unless verbose option is given.  Also do log mode.


Implement background services using procfiles and honcho

Example usage can be found here: https://github.com/chvp/nixos-config/blob/main/shells/dodona.nix

Sure, I'll try to look into it tonight.

I took a stab at writing some documentation (sorry for the delay, life got in the way).

thanks!
bors merge


An error occurred (ConflictException) when calling the StartTranscriptionJob operation

We're getting this error in our cloudwatch logs when trying to transcribe voicemails through amazon connect.
[ERROR]	2022-01-28T19:17:02.683Z	82a9debb-b8e3-4b76-8672-d0cb9e3f755c	An error occurred (ConflictException) when calling the StartTranscriptionJob operation: The requested job name already exists. Use a different job name
I'm researching this now, but would like to know if anyone on this team has seen this before?

Job name is always the Contact ID. If you were testing and resubmitted the same s3 event, or you re-uploaded the recording to S3, or something like that with the same contact ID, this would happen.

There's nothing in the documentation about setting Contact ID, so we aren't actually setting the Contact ID anywhere; we're using default settings. Is this a testing limitation where two voicemails from the same number can't be processed at the same time?

Here's some more discussion around this:

leave VM, hang up. Repeat. I always let it get to VM and I always left one. I ruled out length as a factor in my tests.
With my streak of 10 in a row, only my first one fails. So maybe contactId overwrites and gets lost

As a developer, when using step functions, i've always appended a hash at the end of the id in question so that we don't have an issue with this.  Is it possible that the contact id could include something to make it unique?  The unfortunate thing is we don't control this on the SFDC side.

I see here where you're passing in the contact_id.

Contact ID is a unique identifier, and should never repeat on a given instance of Amazon Connect. If you run the update contact attributes API OR if kinesis emits the event twice, this could happen I suppose. Another option would be to delete the jobs after completion when we delete the transcript from S3. That could certainly reduce the instance of this.

If you want to change your implementation to remove the transcription job as well (as I will be doing in the next release), make the following changes:
In the AWSSCV_vmx_core_policy, add the action:
"transcribe:DeleteTranscriptionJob"
In the packager function, replace:
        # Delete the transcription
        try:
            transcript_delete = transcript_object.delete()

        except Exception as e:
            logger.error(e)
            logger.error('Record {0} Failed to delete transcript'.format(loop_counter))

with:
        # Delete the transcription job
        try:
            transcribe_client = boto3.client('transcribe')
            transcribe_client.delete_transcription_job(
                TranscriptionJobName=contact_id
            )

        except Exception as e:
            logger.error(e)
            logger.error('Record {0} Failed to delete transcription job'.format(loop_counter))

That will delete the job, inputs, and resulting transcription. Its a cleaner setup and should greatly reduce the liklihood of this happening.


Implement port allocation

What
Describe what the change is solving
Implement port allocation for EndpointBindings
How
Describe the solution
Use a port bitmap (from ngrok internal monorepo)
Use a static allocation strategy (noted in comments)
Use a fixed port range from 10000-65535 allowing for ~50k EndpointBindings :see_no_evil:
Breaking Changes
Are there any breaking changes in this PR?

#460  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @hjkatz and the rest of your teammates on  Graphite

Merge activity

Oct 23, 3:00 PM EDT: A user merged this pull request with Graphite.


Can it resolve those kind of images?

No. Need to adjust preprocessing and segmentation routines for that.


feat: added reading of conda-lock files

Allows reading of conda-lock files.

pretty cool! :)

I am wondering if this should go in conda-types or a seperate crate.

Added Digest parsing to rattler_digest crate, so that should work now!

Wow, that looks pretty slick! :)


Everything is NewClass?

On the master branch right now, practically all objects are referenced as NewClass:
// In Chrome dev tools console:
> var marker = L.marker();
> marker
NewClass {options: Object, _latlng: undefined, _initHooksCalled: true, constructor: function, initialize: function…}

Pretty much all objects are like this. It seems to be because they are all extending a distant object that is type NewClass. Is this a bug or is this just how JavaScript is? Would be nice if it said what class the object actually is, like Marker...
Any clues on this?

Pretty certain this is the same in 0.7
Class recognition in JS is hard!

Looks like back in 0.6 Class names didn't come up. It would just say something like e > {... } in the chrome console.

Unfortunately it's just how JS works. The only way to have pretty class names in the console combined with a sophisticated class system is transpiling code a-la Coffeescript or Traceur.


Compatibility with Preact

Hi!
Just wondering if anyone tried using the components with Preact. If not, I'd be happy to try it out and report back 🙂

You need to understand that this isn't a new component library - it's a pre-styled solution originating from @radix-ui and @tailwindlabs. Tailwind is framework-agnostic, but I'm not sure if radix-ui is compatible with preact.

Hey @jackHedaya, I'm curious to know if you've had a chance to try it out and could share your feedback?

Radix UI does not support preact, see this link
https://github.com/radix-ui/primitives/discussions/807#discussioncomment-3206140

Try Preact Aliasing.
Preact Aliasing is defined as:
Aliasing React to Preact At some point, you'll probably want to make use of the vast React ecosystem. Libraries and Components originally written for React work seamlessly with our compatibility layer. To make use of it, we need to point all react and react-dom imports to Preact. This step is called aliasing.
You can find more information about Preact Aliasing in the Preact documentation

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


[Derived Field] Integration tests for derived fields

Description

tests derived fields when defined in index mappings
tests derived fields when defined in query
tests mappings and settings updates for derived fields
tests index and search analyzer applicability on derived fields
tests query string
tests fetch fields and highlight on derived field of type text. Currently highlighter of type unified is supported.
tests ignore_malformed, format, source_indexed_field and properties for explicit type definitions.

The diff will reduce once preceding PRs are merged - https://github.com/opensearch-project/OpenSearch/pull/13592, https://github.com/opensearch-project/OpenSearch/pull/13717 and https://github.com/opensearch-project/OpenSearch/pull/13720
Related Issues
Resolves
https://github.com/opensearch-project/OpenSearch/issues/13715

https://github.com/opensearch-project/OpenSearch/issues/13143
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] API changes companion pull request created.
[x] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[x] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

flaky tests encountered in previous failing checks -
https://github.com/opensearch-project/OpenSearch/issues/10006
https://github.com/opensearch-project/OpenSearch/issues/13503
https://github.com/opensearch-project/OpenSearch/issues/13476
https://github.com/opensearch-project/OpenSearch/issues/13792
https://github.com/opensearch-project/OpenSearch/issues/13473


Rename "Compute@Edge" to "Compute" for Fastly

What is the feature you are proposing?
Fastly "Compute@Edge" was renamed to "Compute".

Fastly’s Compute platform, formerly known as Compute@Edge, helps you compile your custom code to WebAssembly and runs it at the Fastly edge using the WebAssembly System Interface for each compute request...
https://docs.fastly.com/products/compute

Fixed by #1664


Cinder Volumes Do Not Reattach On Node Shutdown

Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug
What happened:
Ran a deployment with a PV cinder volume.  Shutdown the node via Horizon console.  Pod was stuck in "Init" state waiting for the PV to attach.
What you expected to happen:
PV would detach from the shutdown node and reattached to the new node.
How to reproduce it (as minimally and precisely as possible):

Run a deployment with a PVC
Shutdown the node via Horizoon that has the deployment's pod running

Anything else we need to know?:
Example of hung pod:
test-686c498d6b-2d726                                 2/2     Terminating         0          179m    <IP_ADDRESS>    node2             <none>           <none>
test-686c498d6b-b7x2b                                 0/2     ContainerCreating   0          19m     <none>          node1             <none>           <none>

Environment:

openstack-cloud-controller-manager version:

docker.io/k8scloudprovider/cinder-csi-plugin:v1.15.0
quay.io/k8scsi/csi-attacher:v1.2.0
quay.io/k8scsi/csi-node-driver-registrar:v1.1.0
quay.io/k8scsi/csi-provisioner:v1.3.0
quay.io/k8scsi/csi-snapshotter:v1.2.0

OS (e.g. from /etc/os-release):
NAME="Ubuntu"
VERSION="18.04.2 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.2 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic

Kernel (e.g. uname -a):

Linux master1 5.0.0-23-generic #24~18.04.1-Ubuntu SMP Mon Jul 29 16:12:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Install tools:
kubeadm
Others:

Logic for selecting when and where to attach volumes isn't managed by CSI driver. It is done by attacher-detacher-controller in kubernetes manager. (code is located in kubernetes core repo)
Also this behavior is intended as control loop can not know if node is down or if there is just net-split and volume would still be in use.
If k8s logic would just forcefully detach volume and attach it to new node the data on volume might get corrupted.
In production environments you should never shutdown node before draining it. If node crashes it needs to be manually fixed or removed from cluster.

/remove-lifecycle stale

Hello. I have the same problem.
After a node crash volumes stay in attached state (at least for Kubernetes, because Horizon says they are not attached to any node). How can I tell Kubernetes, to remove that node from cluster? It is removed automatically (kubectl get nodes doesn't show it). But volumes are not possible to reattach because of "already attached" state.
How to at least manually deattach volumes from dead/nonexistent node?

Seems like you have something broken in Openstack side. Horizon do not always show volume information correctly. So it is better to use cli as there you get actual data. I.e if server do not exist then horizon might not show any attachments on volume even volume might have attachments in it. So then the Cinder CSI will fail to attach as there is already attachments on the volume.
If you have OS cloud provider and you just delete the vm it should find it out and delete it from cluster and openstack should detach all volumes attached to that node. So everything gets just scheduled to other nodes.
What you described is that node do not exit but volumes can not be attached to new nodes. So then might be that something failed in communication between nova and cinder.
To fix this only way is to use cinder cli to reset the state of volume.
Following commands might help:
cinder reset-state --state available $id
cinder reset-state --attach-status detached $id
Kubernetes should never delete nodes. Only the openstack cloud provider might delete nodes if they do not exist any more.

@mape90 thanks for the informative answer


CI docs: Update publishing step for homebrew

Update step to use bump-formula-pr
brew bump-formula-pr --version=3.8.1 code-server

Related: https://github.com/Homebrew/homebrew-core/pull/70537#issuecomment-774351614

Closing via #2762


Docker packaging, adding CentOS 6 and CentOS 7 packaging

Note that this PR requires the following changes already committed :

https://github.com/servebox/omnibus/commit/02f567deadf78ec7049bab0b591af9651bfb5008
https://github.com/servebox/omnibus-software/commit/b95765ee08c8e4b1c97f90d7e32e1d138811fbb7
https://github.com/servebox/omnibus-software/commit/3ddc1fbaaec87486cf84336d6b5192193ab59483
https://github.com/servebox/omnibus-software/commit/f7cef09ea51e13492603ed44e6249265e0c7d1cf
https://github.com/servebox/omnibus-software/commit/40aff9997a64095e8de4bc5cea4ca5871023efcf
https://github.com/servebox/omnibus-software/commit/d0e01a7ef6fb732183b7e710306659d35277f390

No big changes, just update the way of detecting architecture 32bits vs 64bits.
debian packages have been built for Jessie, let me know if Wheezy should be included.

No more useful


Make Cmd.run typesafe

I notice there is a stale PR that tackles the same problem.
This PR goes a step further with argument inference while still allowing the special symbols to be used if the fn passed has arguments that would match the types of Dispatch or GetState (although I can't distinguish between them as the type symbol === symbol)

Is any of this using a new typescript feature that requires a version we don't currently require, or is it a breaking change at all? I think we currently require at least 2.8, but I'm not too familiar with typescript as I don't use it myself.

Is any of this using a new typescript feature that requires a version we don't currently require, or is it a breaking change at all? I think we currently require at least 2.8, but I'm not too familiar with typescript as I don't use it myself.

Yes it does require ^3.0.1 because of the generic rest parameter https://www.typescriptlang.org/docs/handbook/release-notes/typescript-3-0.html#generic-rest-parameters. I don't think its possible to infer the argument before that, but it is one of the more useful bits.

@laszlopandy can you review this? Since it requires typescript v3, it sounds like it'll be a breaking change to introduce. How widespread is typescript v3 usage? Is it a difficult upgrade from v2?

TS Upgrades are generally smooth, and v3 was released several months ago, so I don't anticipate updating to TS v3 to be a major issue for people. We have been updating to every minor version sequentially in <1 day over the last two years.
But if the API significantly changes to make it easier to typecheck that can be a larger breaking change and a hurdle towards upgrading.
I think it would be good if this change is merged and any API changes (if needed) be taken up in separate breaking changes.

@lorefnon awesome. Then I think we can just do a v6 to release this. There is another breaking change I want to release anyway

@domarmstrong

(although I can't distinguish between them as the type symbol === symbol)

Is this still true? Does this mean if you switch the arguments accidentally it doesn't give a type error?

@domarmstrong

(although I can't distinguish between them as the type symbol === symbol)

Is this still true? Does this mean if you switch the arguments accidentally it doesn't give a type error?

No, I've fixed this. It should correctly error now if you use the wrong symbol, which I've tested in the loopReducer.ts file.
-    [K in keyof T]: T[K] extends Dispatch | GetState
-      ? typeof dispatch | typeof getState | T[K]
-      : T[K];
+    [K in keyof T]: T[K] extends GetState
+      ? typeof getState
+      : T[K] extends Dispatch
+        ? typeof dispatch
+        : T[K];

Thanks @domarmstrong in that case I suggest we merge this now and deal with any new issues when we prepare for the next release. Thanks for your work on this.
(FYI: it's not necessary to force push to keep a single clean commit; we squash merge PRs in this repo).


Can't delete images

Hi @jessfraz
nice tool! Love it! Thanks for sharing it :)
I have pushed an image to my repo (https, but without auth, registry v2).
ls
REPO                TAGS
alpine              latest

manifest
{
   "schemaVersion": 2,
   "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
   "config": {
     "mediaType": "application/vnd.docker.container.image.v1+json",
     "size": 1512,
     "digest": "sha256:053cde6e8953ebd834df8f6382e68be83adb39bfc063e40b0fc61b4b333938f1"
   },
   "layers": [
     {
       "mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
       "size": 1991435,
       "digest": "sha256:b56ae66c29370df48e7377c8f9baa744a3958058a766793f821dadcb144a4647"
     }
   ]
 }

There is no fsLayers blobSum part used to delete / rm the image?
{
   "schemaVersion": 2,
   "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
   "config": {
     "mediaType": "application/vnd.docker.container.image.v1+json",
     "size": 1512,
     "digest": "sha256:053cde6e8953ebd834df8f6382e68be83adb39bfc063e40b0fc61b4b333938f1"
   },
   "layers": [
     {
       "mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
       "size": 1991435,
       "digest": "sha256:b56ae66c29370df48e7377c8f9baa744a3958058a766793f821dadcb144a4647"
     }
   ]
 }

Tried layers digest, but image is still there?
reg --registry https://... delete alpine:latest@sha256:b56ae66c29370df48e7377c8f9baa744a3958058a766793f821dadcb144a4647

Response:
Deleted alpine:latest@sha256:b56ae66c29370df48e7377c8f9baa744a3958058a766793f821dadcb144a4647

ls
REPO                TAGS
alpine              latest```

Any idea?

I think the registry I used for testing runs with delete disabled... I'll check this.

Enabled delete by ENV VAR REGISTRY_STORAGE_DELETE_ENABLED: true:
/ # echo $REGISTRY_STORAGE_DELETE_ENABLED
true

and inside the config /etc/docker/registry/config.yml:
/ # cat /etc/docker/registry/config.yml
version: 0.1
log:
  fields:
    service: registry
storage:
  cache:
    blobdescriptor: inmemory
  filesystem:
    rootdirectory: /var/lib/registry
  delete:
    enabled: true
http:
  addr: :5000
  headers:
    X-Content-Type-Options: [nosniff]
health:
  storagedriver:
    enabled: true
    interval: 10s
    threshold: 3

But deleted images still there...
reg --registry https://myregistry delete alpine@sha256:b56ae66c29370df48e7377c8f9baa744a3958058a766793f821dadcb144a4647
Deleted alpine@sha256:b56ae66c29370df48e7377c8f9baa744a3958058a766793f821dadcb144a4647

reg --registry https://myregistry delete alpine:latest@sha256:b56ae66c29370df48e7377c8f9baa744a3958058a766793f821dadcb144a4647
Deleted alpine:latest@sha256:b56ae66c29370df48e7377c8f9baa744a3958058a766793f821dadcb144a4647

Image is still listed:
reg --registry https://myregistry ls
Repositories for https://myregistry
REPO                TAGS
alpine              latest

It seems to work with debugging to get the correct ref:
2017/11/24 15:20:02 ref: sha256:4b8ffaaa896d40622ac10dc6662204f429f1c8c5714be62a6493a7895f664098

Delete works with that ref used. Usage is a little bit strange, but it works that way.

@pwFoo https://stackoverflow.com/questions/25436742/how-to-delete-images-from-a-private-docker-registry

Tested the latest release, but problem isn't fixed I think. Still need debug info to get the correct ref with reg from registry.
nwe-reg manifest openvpn:v0.0.1
{
   "schemaVersion": 2,
   "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
   "config": {
     "mediaType": "application/vnd.docker.container.image.v1+json",
     "size": 2059,
     "digest": "sha256:6a2e9a93fa6ffa47b83c32a7afb6fdff0fc1b765ef7c5a962e4e2e27d03a0c05"
   },
   "layers": [
     {
       "mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
       "size": 1991435,
       "digest": "sha256:b56ae66c29370df48e7377c8f9baa744a3958058a766793f821dadcb144a4647"
     },
     {
       "mediaType": "application/vnd.docker.image.rootfs.diff.tar.gzip",
       "size": 2487962,
       "digest": "sha256:500d7c908a09320a3ceb63b207aea199389dbecde18950363115287359ead8ee"
     }
   ]
 }

The workaround I use.
# nwe-reg -d manifest openvpn:v0.0.1 |grep "ref:"
[...]
2017/12/14 10:02:11 ref: sha256:fc785bb24e2b65fffff2ae72897d80c0c564100f167cfba08f114aaa7c4fd560


Fixed the build: use target java 1.7; removed unnecessary (default) m…

…aven repositories

I revoke this request, because, despite it compiles, I still cannot compile Paxtools project (sbgn-converter) using the refresh chilay-sbgn 2.0.1 snapshot.


[WIP] Allow Switch and Lan models to be taggable

Purpose or Intent
Allow Switch and Lan models to be tagged to support best-fit placement
Links

https://www.pivotaltracker.com/story/show/125152641

@gmcculloug the original description of this feature was "Allow for VDS switches to be tagged and support best fit placement.", does it make sense to tag the switch for this or just the lan since that is what is being picked in the end?  Or could it help to tag both?  (really I just don't know how tagging and provisioning works :smile:)

@agrare It could not hurt to support tagging on both models as users can define what the tagging means to them.  (See also http://talk.manageiq.org/t/should-all-models-be-taggable)
In provisioning, tags can be used in custom automate code to decide which resources to use.  For example, if the template you are provisioning is tagged as production a user could write a custom automate script that selections a lan with a matching production tag.

@agrare Tests are failing because of recent changes from PR #10230.  We will need to update the specs to validate the taggable? method against something other than the 'Lan' object since that now supports tags.  You could change it to use the Account model instead.

@gmcculloug thanks, changed over to use Account in that spec test

@miq-bot remove-label wip


Update the changelog file

Add v0.1.3 to the CHANGELOG.md

@alallema it could be good to add a new step in the CONTRIBUTING about updating the CHANGELOG :)

bors merge


Use a connectionpool with a limit of 100 connections to usdb.

The default http-pool is 10 connections. To increase this we must create an adapter and then mount that to the session.
Current value is 100 concurrent connections.
This improves the calls done with the session. So not video's downloading through yt-dlp.
Haven't found a way to parallize that more

In a quick and dirty test, this didn't seem to speed things up. I guess the bottleneck is the QThreadPool.maxThreadCount anyway. You could experiment with that and measure the difference it makes.

Further investigation required, closing for now.


How to insert (html)text in the editor?

Is there a way to insert text into the editor at the position of the cursor (without making a plugin)?

the control?

Sorry editor ;)

const editor = new Jodit()
editor.selection.insertHTMl('some html'); // will be inserted on cursor position

Thanks Valeriy for the answer.
But he problem is that I want to insert from 'outside' the editor, in this case by the ref.
So following your answer I should do this:
const editor = Jodit.make(this.editor.current)
editor.selection.insertHTML('some html')

But this will put the html in front of the existing text.

same issue

Yup, same issue. Anyone ever get it figured out?

Yup, same issue. Anyone ever get it figured out?

Actually I just figured it out. For anyone searching for this in the future, I was trying to open a custom dialog and insert some HTML at the cursor from there. On the click for opening the dialog I added pos = editor.selection.save(); (pos being a global variable). Then right before inserting the HTML do editor.selection.restore(pos) and it will put the cursor where it was when you executed the selection.save. That did it for me.

Actually I just figured it out. For anyone searching for this in the future, I was trying to open a custom dialog and insert some HTML at the cursor from there. On the click for opening the dialog I added pos = editor.selection.save(); (pos being a global variable). Then right before inserting the HTML do editor.selection.restore(pos) and it will put the cursor where it was when you executed the selection.save. That did it for me.

Hello Dev, can anyone tell me how can I solve the issue ? Facing the same issue

@DarinBeard can you please share your code?

I followed this thread and figured out the issue https://github.com/jodit/jodit-react/issues/51#issue-589056107

@DarinBeard i am trying the solution of save() and restore() before insertion but it is not working. Anyway you can share a snippet?

I fixed some of the known issues of Jodit in this repository, including this one.
See if it helps:
https://github.com/ShahrozeAli/react-jodit-fixes


Problem when importing models out of the terminal

🐛 Describe the bug
I have a conda environment and the latest versions of the libraries.
When I run my script in vscode or in the terminal it works perfectly.
But when I want to run my script, with the same conda environment in another execution environment like Python 4 Delphi, right on the line from super_gradients.training import models it gives me an error: AttributeError: 'NoneType' object has no attribute 'write' '
I have tried to follow the execution but I can't figure out why it gives me this error.
I normally use this Python 4 Delphi architecture with other environments and other libraries and it works without problems, because everything runs within the conda environment. but with this library it doesn't work for me.
Any idea why?
Versions
j

I think Python 4 Delphi does some magic with overriding stdin/stdouts which conflicts with some magic SG is also performing under the hood to log console outputs to log files. Unfortunately I don't think we are able to address this issue now. Probablly it relates to this issue but the fix is not on the roadmap yet.


Iati.cloud-2038

Added task  third party validation

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


Add etcd-operator to requiredPods when selfHostedEtcd is true

In some environments, pulling image from gcr.io is faster than quay.io. We might call etcdutil.Migrate() too early even before etcd-operator got started.
In this case, the user will have fail to wait etcd TPR to be ready: timed out waiting for the condition error and the bootstrap process got terminated.
This PR add etcd-operator to requiredPods when selfHostedEtcd is true. We'll make sure etcd-operator is running before calling etcdutil.Migrate().  Making the bootstrap process less likely to fail.

The code change makes sense. Please format the changed code.

OK to test

fixed the format. I was editing the file directly on an linux server.

@aaronlevy can you please take a look?


python version??

Which version of python is the author using : )  ?

Python 3

The master branch is for Python 2.7.
However, there is a python3 branch as well.

Excuse me for commenting to a closed issue, but could you possibly put in the readme that Python 3 is supported as well and there's a separate branch for that? I spent about an hour or two rewriting some lines in the code so that I can run it in Python 3...
Is there actually any reason why Python 3 branch is not master?

@kurapan Thanks -- good idea; updated now!

@ankush-me Thanks!


Take input of Sudo Password for CLI Commands

:nut_and_bolt: Description: What code changed, and why?
Added sudo_password support for HA cmd in ruby

:chains: Related Resources
https://chefio.atlassian.net/browse/CHEF-795
https://github.com/chef/automate/pull/7774
:+1: Definition of Done
:athletic_shoe: How to Build and Test the Change

Rebuild automate-cli (rebuild components/automate-cli/) or can use (akrishna/automate-cli/0.1.0/20230412082516), automate-ha-cluster-ctl (rebuild components/automate-cluster-ctl/) or can use (akrishna/automate-ha-cluster-ctl/0.1.0/20230411133555), and automate-ha-deployment(rebuild components/automate-backend-deployment/) or can use (akrishna/automate-ha-deployment/0.1.0/20230411141141).
Upload the above packages hab pkg upload <PATH_TO_PACKAGE_HART_FILE>.
Create an airgap bundle with new packages sudo ./chef-automate airgap bundle create -m latest_semver.json --channel dev. Update this latest_semver.json manifest file with your origins.
Note: Do not update the automate-cli origin in the manifest file,  just install the new automate-cli on Bastion.
Set the environment variable sudo_password. For eg. export sudo_password=1234.
Run deployment command with -E flag to preserve environment variable. For eg. sudo -E ./chef-automate deploy config.toml --airgap-bundle automate-4.5.104.aib. Refer here for passing environment variables to the sudo command.

:white_check_mark: Checklist
All PRs must tick these:

[ ] I have read the CONTRIBUTING document.
[ ] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[ ] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[ ] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[ ] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[ ] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[ ] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[ ] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable

Video: https://progresssoftware.sharepoint.com/:v:/s/ChefCoreC/EacHYiZu7aFBgNkiBpoRPwkB0f_Ngkk_CR2vOxTCdbtMfA?e=janCk5

Hi @AnkurMundhra could you please review this ticket https://chefio.atlassian.net/browse/CHEF-795 ?


Bug Fix and Enhancement

Bug Fix: The onScenePostEvents was not resetting the pressed state to false for unpressed buttons which made the C_button_released call return true always after releasing a button once.
Enhancement: Updated the onSceneFirstLoaded and onScenePostEvents to keep track of the last activated controller id. Also added a condition to get the last activated controller id.
This makes it possible to dynamically switch between gamepads and just use whichever controller the user picks up instead of having to designate a static controller/player id. Sorry for the whitespace differences.
I'm not a big github user.

@Bouh I wasn't sure this was the correct way to share changes with you, but I hope you will consider them.

This look good!
That mean the C_button_released condition return true only once when the button has been released. And it doesn't stay on false.
This is a breaking change in a good way because this mimic how it's done with the key released condition.
Thanks for this fix :+1:

Also if you can edit the version from 0.3.0 to 1.0.0.

I think there still might be a lingering bug where calling "Any Gamepad button was released" or "Any gamepad button was pressed" before called "Was button X released?" will prevent getting an accurate response.
There are also rare times where button presses are not detected.
I have not done problem isolation or debugging on either of these, so just letting you know for now.

Indeed I found an issue with released condition.
I fixed it by adding a case in C_any_button_released where if the button is released or not I save the appropriate value in the previousFrameStateButtons of the button.
You can try it with this example.
Let me know if you see something else.
gamepad 1.0.0 - example.zip

This seems like something that should get merged.
@krunkster were you able to test @Bouh latest update of the example game?

The merge will be made into a cleaner branch here:
https://github.com/GDevelopApp/GDevelop-extensions/pull/692


Covers node-hc failure test case using -f flag

Needed to cover the test case of starting the agent while running with node-hc which should fail with an error message of conflicting installations

LGTM


Circuits containing division operations not supported in QIR submissions

Under limited circumstances they are supported in QASM submissions, so this is a feature discrepancy.

Related: https://github.com/CQCL/pytket-qir/issues/104 and https://github.com/qir-alliance/pyqir/issues/165 .

Update: division operations with non-constant denominator shouldn't be supported at all, so it's actually a "bug" that they are in QASM.


Fatal error, can't open config file '/etc/redis/redis.conf'

As the title states, updated gitlab and also this redis image leads to
Fatal error, can't open config file '/etc/redis/redis.conf'

@cwildfoerster the same for me!
Using on Synology. Without user mappings it works.


Beyond Earth "space-breathing" datapack doesn't work

Title.
It seems like the .json for it is missing, and if it is in the datapack, it doesn't seem like the alien tf can "breathe" in space by default.
https://github.com/user-attachments/assets/7670edf7-290e-4314-b862-68482d533ec5
(tested in the 0.14.1 build assembled a couple of days ago)

Please reopen, and provide a log, if the bug is replicated.

strange, also v7.0 is a beta with some things possibly not working, does it work in the stable ver?

latest.log
the video is too big for github so here's the log


Added support for tcpdump command attachment

@matsvanmolle I believe tcpdump is not a command wrapper, as you cannot give it a specific command in its own command line. It is more of a CommandAttachment in benchkit methodology (something that you run in parallel with your actual benchmark command). Could you update your patch to comply to that?

Hi @matsvanmolle let me know once this is ready. You can also remove the "draft" status once you consider it is ready to be reviewed and integrated. Thanks again.

@matsvanmolle are you still considering this one?

@matsvanmolle are you still considering this one?

Yes, I will test this one in the coming week. Promised

@apaolillo I tested it and did some cleanup. However, I would like to point out that the password for sudo for tcpdump should be turned off.


c# example, doesn't seem to do anything ...

Doesn't seem to do anything, no errors no output, just nothing.
Am I missing something?

It depends on what you've done to run it.


Styling issues Tutorial

DESKTOP 1280x800

Tutorials main page: https://p5js-wip.online/tutorials/

[ ] Add padding above and below in the header section - applies for all resolutions
[ ] In the images preview for all sub sections: reduce the spacing between the image title and subtitle. For example: between Get Started and A tutorial that [..]
[ ] Remove grey line at the bottom of the page above the footer. Applies for all resolutions

Tutorials sub pages: https://p5js-wip.online/tutorials/get-started/

[ ] The language, accessibility and search buttons are pink and should be brownish/gray like the color in the header of the tutorial main page
[ ] At the bottom, related examples should have a line above and below, same for related references.
[ ] The image that is a preview on the tutorials main page for each tutorial should appear at the top of each sub page. Applies for all resolutions. IT doesn't happen for get-started for example.

TABLET 768x800

Tutorials main page: https://p5js-wip.online/tutorials/

[ ] In the header: Subtitle should be 24px and it is 30 px

Tutorials sub pages: https://p5js-wip.online/tutorials/get-started/

[ ] In the color-gradients tutorial the example code overflows the grey section to the right: https://p5js-wip.online/tutorials/color-gradients/ .
 . This happens also for mobile resolution.

MOBILE 428x800

Tutorials main page: https://p5js-wip.online/tutorials/

[ ] the description below each image should be 10px (but might be too small? ) It is currently 12px.

Thank you!

The image that is a preview on the tutorials main page for each tutorial should appear at the top of each sub page. Applies for all resolutions. IT doesn't happen for get-started for example.

Not sure if this is the case since those images are different. I think that it makes more sense to leave this decision to the authors of the blog content but pulled it out to highlight it.

Also, just noticed that the code examples no longer have JS syntax highlighting. @Qianqianye I see that in the Figma, tutorials don't have highlighting while examples do. Do you think it makes sense to just have highlighting everywhere? The code snippets in some tutorials are quite long, and I find them more difficult to read without it.

[ ] On Firefox mobile on Android, the code font doesn't seem to load, defaulting to a serif font. (This works in mobile Chrome and desktop Firefox though 🤔)

[ ] The breadcrumbs on a tutorial page show the name of the tutorial twice. Should the first one be "Tutorials"?

Also, just noticed that the code snippets in tutorials no longer have JS syntax highlighting. @Qianqianye I see that in the Figma, tutorials don't have highlighting while examples do. Do you think it makes sense to just have highlighting everywhere? The code snippets in some tutorials are quite long, and I find them more difficult to read without it.

Yes I actually worked pretty hard to get this styling in place. By default the Astro builder applies a nice dark theme to any bigger code blocks. I agree that it is easier to read and I would've preferred to not strip out the styles but it was done to match the Figma mockups

The image that is a preview on the tutorials main page for each tutorial should appear at the top of each sub page. Applies for all resolutions. IT doesn't happen for get-started for example.

I think this isn't necessarily implied from the designs though it is a little ambiguous. The images are similar but different in the Figma mockups and I think it better to leave flexibility to tutorial authors on whether they want a featured image at the top. Going to mark this one as done.

Closing as all items have been addressed, including elements flagged by Dave

Sorry for the delayed response. I agree with @davepagurek and @stalgiag that highlighted JS syntax would make the code snippet more readable. We probably miss catching it with the designers on this part. Would it make sense to have the JS syntax highlighted for the upcoming launch?

Sorry for the delayed response. I agree with @davepagurek and @stalgiag that highlighted JS syntax would make the code snippet more readable. We probably miss catching it with the designers on this part. Would it make sense to have the JS syntax highlighted for the upcoming launch?

Switching it back to the default Astro styling is simple. Custom styling won't make sense before launch. I can take a picture of the Astro default and you can decide if it works.


[Doc] add password descriptions to load docs

[Doc] add password descriptions to load docs
Signed-off-by: amber-create<EMAIL_ADDRESS>

@mergify backport branch-3.0


Make snippets context aware

Snippets now appear only where they make sense, e.g. resource snippets in a "resources" section, new-parameter snippet inside "parameters" (definitions), etc.
Fixes most of #654
Internal testcases link: https://microsoft.sharepoint.com/teams/azcode/_layouts/OneNote.aspx?id=%2Fteams%2Fazcode%2FSiteAssets%2Fazcode Notebook&wd=target(Testing.one|D12CBB51-9002-4371-B958-B88191DF5CF5%2FDRAFT NEW%3A Snippets Contextualization|61E92124-07C0-604D-9A8C-73C1E08B86AD%2F)

Could you please provide me access to internal testcases link? Thanks!


File Not Found error on running regular optimization py file mac

Getting below error if I run "python3 solve_regular.py"
Exported problem with name: mp_h5_regular_pS0ST
Traceback (most recent call last):
File "/Users/abchanda/Documents/fpl-optimization/run/solve_regular.py", line 19, in
result = solve_multi_period_fpl(data, options)
File "/Users/abchanda/Documents/fpl-optimization/run/../src/multi_period_dev.py", line 349, in solve_multi_period_fpl
process = Popen(command, shell=False)
File<EMAIL_ADDRESS>line 951, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File<EMAIL_ADDRESS>line 1821, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'cbc tmp/mp_h5_regular_pS0ST.mps cost column ratio 1 solve solu tmp/mp_h5_regular_pS0ST_sol_init.txt'

You should be seeing this error due to first solve step failing. Probably the instance if infeasible. Could you tell me your configuration (regular_settings.json content)?

Sure..
{
"horizon": 5,
"ft_value": 1.5,
"itb_value": 0.08,
"no_future_transfer": false,
"randomized": false,
"banned": [],
"locked": [],
"delete_tmp": true,
"secs": 300,
"use_cmd": false
}

Can you open a terminal and try running cbc to see if it is in your path variable?
If you don't get an error, navigate to "tmp" directory and try running the same line as above (if you have the .mps file still there):
cbc tmp/mp_h5_regular_pS0ST.mps cost column ratio 1 solve solu tmp/mp_h5_regular_pS0ST_sol_init.txt

If this works fine, then try using "use_cmd": true and let me know if the script works that way. Try setting horizon to 1 and make sure your credentials are correct.

If cbc fails, make sure you have the cbc binary downloaded and added to your path environment variable.
If the solve commands work and you see "infeasible" (this is less likely, but just in case), probably it is an input issue.

Thanks for the suggestions. Yes, cbc command works for me. I tried entering the line you pasted within the /tmp directory.
Below is what I got:
Welcome to the CBC MILP Solver
Version: 2.10.5
Build Date: Aug 12 2020
command line - cbc tmp/mp_h5_regular_pS0ST.mps cost column ratio 1 solve solu tmp/mp_h5_regular_pS0ST_sol_init.txt (default strategy 1)
Unable to open file ./tmp/mp_h5_regular_pS0ST.mps
Option for costStrategy changed from off to column!Order?
ratioGap was changed from 0 to 1
** Current model not valid
** Current model not valid
No match for tmp/mp_h5_regular_pS0ST_sol_init.txt - ? for list of commands
Total time (CPU seconds):       0.00   (Wallclock seconds):       0.00

try using "use_cmd": true and let me know if the script works that way.
Yes, I just tried it and it works now!!! :) thank you so much. Does it mean this is fixed now?

I see "current model not valid" error. Seems like there is still a problem. When you run the script with use_cmd, did it give you a solution back at the end?

Oh never mind, it was probably due to file not being there. Do you see the transfer suggestion at the end when you run the script now?

Yes, I see it now! The issue is fully resolved! Thanks a lot for your help and expertise!


Set default host and port in config file

It would be great if we can set default host and port for mopidy server somewhere in config file. For now I need to setup this for every new browser / device I want to connect.

@matmar-91 Can you please provide me with some examples of your configuration?
By default the web interface will use the same ip/domain and port 6680 for it's initial Mopidy connection. For example if your mopidy server was music.desktop then the web interface would connect to music.desktop:6680.
The only time I think this wouldn't work out of the box is if your webserver was hosted on a different machine to the mopidy server (ie spotmop.laptop as web interface and mopidy.desktop as mopidy server). In this instance the ~/.config/mopidy/mopidy.config file will not be relevant. The solution that comes to mind is to have an additional web interface config file, but this isn't a desirable solution.
I would be inclined to focus on #102 as this would provide synchronicity across your client devices.

Ok, you're right. For some reason i can't get it work first time but now i reset all settings and leave host and port blank and it worked. Probably before there was some other problem on my end. Thank you for your time and great work!

Sweet, glad you got it working :-)
The Reset all settings button has saved my bacon a few times too!


adding Serbian

adding Serbian

Hello, thanks a lot for your MR!
But it is missing the Serbian flag for the settings (here, settings will crash when you will try to open it).

I added it:
https://github.com/damienleroy/PowerAccent/commit/5c81757dfe905291af6968b3bd56d0752a81fa3e


Fix finding SFML 3 and add SFML 3 to CI pipeline

Currently in master it's not possible to find an installation of SFML 3. Using SFML 3 requires that someone already builds SFML 3 in their build tree and thus can't depend on a preexisting installation.
I then added SFML 3 to CI to ensure SFML 3 support is constantly maintained.

Some recent SFML 3 changes broke the SFML build again.


C#

Can this be done from the C# table storage library?

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

ID: 9693faf0-0f8b-b6ba-877d-183c6cabbc3f
Version Independent ID: dd8c8f1f-8c19-69f2-c025-513558380933
Content: Use a Windows VM system-assigned managed identity to access Azure Storage using a SAS credential
Content Source: articles/active-directory/managed-identities-azure-resources/tutorial-windows-vm-access-storage-sas.md
Service: active-directory
GitHub Login: @priyamohanram
Microsoft Alias: priyamo

@festivus Thanks for your feedback! We will investigate and update as appropriate.

@festivus As per my knowledge it is not possible using table storage library. However, I suggest you please open a new MSDN forum post with the details and community will help address your query. Please refer to below link to MSDN forum to ask new question -
https://social.msdn.microsoft.com/Forums/azure/en-US/newthread?category=windowsazureplatform&forum=WindowsAzureAD&prof=required.

@festivus We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Update Text and Screenshots

https://docs.microsoft.com/en-ca/power-bi/create-reports/power-bi-report-add-filter#add-a-filter-to-an-entire-page
https://docs.microsoft.com/en-ca/power-bi/create-reports/power-bi-report-add-filter#add-a-filter-to-a-visual
Both sections should be updated as Filters are now managed in Filters pane.
Regards,
Julie

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

ID: c9848d00-1a27-8d72-88cb-d74cc50c496e
Version Independent ID: 456b6ffd-5af0-a2d7-aa16-b0335ced93d0
Content: Add a filter to a report in Power BI - Power BI
Content Source: powerbi-docs/create-reports/power-bi-report-add-filter.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Hi, @DaxelCA -- good points, thanks! I'll add it to my list.

Hi @DaxelCA ,
Thank you for your suggestion.  We've made the update and set the article to republish.
All the best,
The DOCS team


Feature request

export request & response  to  gist
export curl command

Hi,
What's the curl command for ? To replay a request ?

@ottomao Yes, communicate with work mates, easy to replay the request with same environment.

And  I think No.2 is more valuable than No.1.

Some code might help: https://github.com/swagger-api/swagger-ui/blob/3ed8efe7360d62745c256d2f68fd30bbad7a58a2/src/core/curlify.js

@ottomao Here is my PR: https://github.com/alibaba/anyproxy/issues/155 Plz review, thx

Great feature ! Just get merged and released to 4.0@beta9.

I meant the curl feature has been merged. The export request & response to gist is not a high-priority feature.

@oboolean Totally agree.


How to add line breaks to StyledText Element?

I want to add line breaks to a StyledText Element but can't manage to do it.
If a styled text contains a line break "\n", I get the following error message:
"StyledText must not contain line breaks, use TextFragment.LINEBREAK for that"
But I can't find an example in the documentation that shows how use TextFragment.LINEBREAK.
Do you have an example or a alternative solution for this? I also tried to use .addMarkup() but that does not support PDType0Font only BaseFont.

Both StyledText and LineBreak are TextFragments and are used in a container, typically TextFlow. So just add your StyledText and LineBreaks to a TextFlow. Actually, this is what addMarkup() is doing under the hood.

Thanks for your fast response.
Can you give me an example how to create a LineBreak TextFragment?
A StyledText TextFragment is easy: StyledText styledText = new StyledText(text, fontsize, font, color);
But I can't find an example how to do the same with a LinkeBreak.

Just create a NewLine, e.g. new NewLine(fontSize)

Perfect thanks, it worked :) 👍


Drop the .cluster.local suffix

Description of the change
While .cluster.local is the standard suffix, it is configurable and not necessarily accurate.
Just using .svc and relying on ndots it used elsewhere through the helm chart, so drop the qualification here.
Existing or Associated Issue(s)
N/A
Additional Information
N/A
Checklist

[x] Chart version bumped in Chart.yaml according to semver. Where applicable, update and bump the versions in any associated umbrella chart
[x] Variables are documented in the values.yaml and added to the README.md. The helm-docs utility can be used to generate the necessary content. Use helm-docs --dry-run to preview the content.
[x] JSON Schema generated.
[ ] List tests pass for Chart using the Chart Testing tool and the ct lint command.

cc @priyawadhwa @haydentherapper @cpanato


[MachineLearning] Fix deserialize for ResourceType in MachineLearningEnvironmentContainerData & MachineLearningDataContainerData

This is a fix for issue: https://github.com/Azure/azure-sdk-for-net/issues/45884
We should remove this once we fixed: https://github.com/Azure/azure-sdk-for-net/issues/45900

API change check
API changes are not detected in this pull request.

Please note, we may still hit exceptions in debug mode, since the resource id is not in arm format,

Please note, we may still hit exceptions in debug mode, since the resource id is not in arm format,

Update: Bypass the validation for resource id for now.


Make catalogsource compatible with restricted SCC enforcement

Restricted SCC enforcement will be added with OCP 4.14
Updating the catalogsource to allow the operator to get deployed
Clusters that don't support the setting (<4.12) will ignore it

Jira: OSD-15620

/lgtm


[WIP] Transition matrices (general, life-cycle)

This PR contains preliminary work that seeks to construct state-transition matrices for any model in HARK, including life-cycle versions.
I am making the raw code public now because I expect to collaborate around these ideas with @wdu9, and in case other people are interested in contributing.
I think this is an important piece of functionality in our goal of communicating with the sequence-space-jacobian methods and tools.
Even though the two examples that I have included are infinite-horizon, the method works for LC parametrizations. It creates a list of transition matrices. One for every age
There are many things to be done:

[ ] Clean up the code and make the methods more readable and user-friendly (using labeled distributions and xarrays, for instance).
[x] Add an example showing that it works with LC models (we really need an init_portfolio_lifecyle param set).
[x] Incorporate death (the transition matrices currently are conditional on survival).
[x] Add tests that compare with the existing model-specific methods for correctness.
[ ] Add the required methods (state-to-state transition, distribution_engine, etc) to more of the existing models.

And here are the ususal to-dos.

[x] Tests for new functionality/models or Tests to reproduce the bug-fix in code.
[ ] Updated documentation of features that add new functionality.
[ ] Update CHANGELOG.md with major/minor changes.

great ideas! @AMonninger and I just came back from NBER ssj workshop and are blown away by their tools. Anything we can do to more tightly integrate with their framework will be very beneficial

The main idea that the PR works of off is that once you have solved a model, the only things that you need to produce a transition matrix that represents it are:

A transition equation: which takes the state today, policy functions, and shock realizations tomorrow; and transforms them into the state tomorrow.
The joint distribution of all the shocks in the model.

I have used the ConsPortfolioModel as a working example, and I added methods that construct both objects for it:
State transition equation:
https://github.com/Mv77/HARK/blob/c4c5064bea6665d096886fe049fc5ec3922c27fc/HARK/ConsumptionSaving/ConsPortfolioModel.py#L382-L425
Shock-distribution engine:
https://github.com/Mv77/HARK/blob/c4c5064bea6665d096886fe049fc5ec3922c27fc/HARK/ConsumptionSaving/ConsPortfolioModel.py#L324C14-L348

Once we have the transition equation, shock distribution, and the grids we want to use for the state-space, we can use the tools that

Find the distribution of a function of a matrix-valued random variable.
Clip points to a general grid using what the SSJ people call the "lottery method."

to produce the transition matrix. It's easy!:
https://github.com/Mv77/HARK/blob/c4c5064bea6665d096886fe049fc5ec3922c27fc/HARK/core.py#L941-L959

So, in principle, this means that if you write a state_to_state_transition method for your model and a method to produce the joint distribution of all shocks, I can get you the transition matrices.

You can tell the methods what "states" to keep track off. This is important because:

Sometimes we do not want to keep track of some states (e.g. we are using Harmenberg or we do not care about PLvl).
Some of the models have versions that augment the state-space. E.g., if in the portfolio model AdjPrb<1.0, the share and the Calvo shock become states.

Here are tests that show that the method can accommodate both versions of the portfolio model.

This will benefit from some ongoing work:

There is a lot of ugly code that finds the right parameters to use given the age of the agent and time_vary and time_inv. E.g., https://github.com/Mv77/HARK/blob/c4c5064bea6665d096886fe049fc5ec3922c27fc/HARK/core.py#L930-L937 . If @alanlujan91 gets his Parameters class working and merged in, this could be a lot cleaner, like Parameters[t].
This is a step towards thinking of models using stochastic transition equations---that is one of the things that needs to be defined for the method to work. I know @sbenthall has been giving some thought to model-representation.
There is also ugly code in packing and unpacking matrices that represent states. Labeled xarrays and functions that work with them would make the code more readable. For instance, having a dist_of_func method with labeled distributions as both inputs and outputs would avoid a lot of unpacking.

@alanlujan91 has pointed me to some work that might help with the cleaning up of stochastic transition equations using DiscreteDistributionLabeled and xarray. Linking for future self-reference:
https://github.com/econ-ark/HARK/pull/1283#issuecomment-1595065727
I think what I want might be possible using DiscreteDistributionLabeled.dist_of_func or lines like these:
https://github.com/econ-ark/HARK/blob/37134b9c9eea4591659b7cc398cf5894e44ba0e7/HARK/distribution.py#L1237-L1238

Just to make sure it's on your radar....
I see you're contributing this as additions to the ConsIndShock and ConsPortfolio models, and that this involves yet another representation of the model equations to be hard coded into the model.
Transition matrix methods are awesome, but it would be even more awesome if they were implemented generically, so that model-specific information could be passed to them.
What I've been working on in #1296 is a standalone engine for the Monte Carlo simulation that is currently bundled in AgentType. The idea is you pass it the equations and shock definitions you want, and it does the work.
Can something similar be done for transition equations?

Hey @sbenthall,
Yeah, one of the main points of this PR is to have the methods work for any model, receiving their transition equation as a parameter. The method find_transition_matrix belongs to AgentType and takes self.state_to_state_transition as given---whatever it is.
It moves us away from model-speficit methods like
https://github.com/econ-ark/HARK/blob/b123b0efbef934fe7a473e815c9333af3e59b964/HARK/utilities.py#L653-L655
There is not a current transition equation definition that is used across models in HARK, so I came up with my own placeholder and showed that it can be defined for multiple models easily. Once there is an agreed-upon representation, this code will be easy to modify to take that one and not the one I am using.
We agree with the goal of having a consistent representation of models that general methods can receive and work with. I am coding with that in mind.

To be clear, notice that the main work is being done in HARK.core.AgentType. That is where the "methods" are being added.
But there is no current representation of transitions that is common to ConsIndShock and ConsPortfolio. The additions to those files that you see are just a makeshift common representation for me to work with. Once there is a hark-wide standard, I'll work with it.
But yeah, if you check out the methods, they don't assume much about the model. They work in n-dimensions, you can tell them what subset of states to consider, they work in infinite horizon and life-cycle versions ... they just need a consistent representation for models. I know you are working towards that and I'll be happy to use it once it is agreed upon and merged in.

Super, @Mv77 .
@mnwhite I wonder if you ultimately want this sort of thing to be in AgentType (which I agree is the natural place to put it given the current architecture) or if it would make more sense to put it in something like HARK.simulation.transition_matrices. My preference is for the latter.... That could happen even if there isn't a HARK-wide standard for model equations. (It should be relatively easy to shift between ad hoc model equation formulations, since they do very similar things).
I don't mean for this to block the PR, which obviously does a lot of good as is.

I believe I have achieved what I wanted out of this PR in terms of scope. I now need to document things and clean up.
The PR uses representations of transition equations, which I know are in the works. I've introduced my own placeholder for two models.
I'd like to decide if I should wait for the canonical representation of transition equations that I know is in the works, or push towards merging this in with the placeholder representations.
@sbenthall @alanlujan91 @llorracc , could you offer any thoughts or updates?

My opinion is that you should go ahead with the clean up and we should merge this in as soon as it's ready.
This PR is good progress, and we can always do the work of aligning it with new transition equation representations later.
If you don't mind, I might go to you first to ask you to adapt this code once the new way of doing equations is settled. But in any case, this is important work to get into HARK!

I agree with Seb, that you should go ahead, and we should merge when it is ready.
My point is just to always have people thinking about the longer-term goal so that we do as much as we can to avoid building things in a way that, if we had only done it slightly differently, it would be much easier to upgrade to the new paradigm.


[Discussion] Can popover api be utilized in a11y-dialog?

Why a11y-dialog

Building interactions that rely on events. The native <dialog> element supports cancel and close events, but their behaviors aren’t reliable. The <dialog> does not support an open event at all.

https://developer.mozilla.org/en-US/docs/Web/API/Popover_API/Using#dismissing_popovers_automatically_via_a_timer
Maybe we can make a11y-dialog even more lightweight by adding opt-in to utilize popover api? A popover api plugin for a11y-dialog :thinking:

Probably worth getting @hidde’s opinion on whether it‘s worth it / a good idea as he is the person I can think of who knows most about the popover API.

It seems useful to me to include it optionally:

to allow folks to make a dialog that is top layer (by setting popover); this is especially helpful for folks that may have <dialog>s that are shown with showModal() in their applications. Such dialogs will be in the top layer, so any a11y-dialog-created dialog that wants to participate in the same stacking context would need to be in the top layer too  (and therefore need a popover attribute)
to allow for light dismiss (but I'm not sure if that's already built in to a11y-dialog?)

It would make sense (to me) though, to default to non-modal dialogs when popover is added.


Update lib.rs

fix

hvh is not a rust keyword? It doesn't make much sense to add it when the macro is primarily about replacing keywords.

hvh is not a rust keyword? It doesn't make much sense to add it when the macro is primarily about replacing keywords.


Backport PR bqplot#1644: fix: save_png doesn't work when character \xa0 is used (#1644)

The character \xa0 is converted to   when .outerHTML() is called, causing an error in the data-URL where the outer HTML is used.
(cherry picked from commit 4d5c2932d8d55e1af96e7129f0550c18c0c3b977)

References

Code changes

User-facing changes

Backwards-incompatible changes

Maybe we karma-webpack would need to be pinned, or we can try to update nodejs...


Email to signatory needs a very minor text change

Currently following email is sent to CLA Signatory.
Hello Vinod Kadam,
This is a notification email from EasyCLA regarding the project vinods15foundation_clagrp3. Vinod Kadam has designated you as being an authorized signatory for Proxima. In order for employees of your company to contribute to the open source project vinods15foundation_clagrp3, they must do so under a Contributor License Agreement signed by someone with authority to sign on behalf of your company.
After you sign, Vinod Kadam (as the initial CLA Manager for your company) will be able to maintain the list of specific employees authorized to contribute to the project under this signed CLA.
If you are authorized to sign on your company’s behalf, and if you approve Vinod Kadam as your initial CLA Manager for vinods15foundation_clagrp3, please review the document and sign the CLA.If you have questions, or if you are not an authorized signatory of this company, please contact the requester at<EMAIL_ADDRESS>If you need help or have questions about EasyCLA, you can read the documentation or reach out to us for support.
Thanks,
EasyCLA Support Team
Change "as being an" to "as an".

This is deployed to DEV, Staging & PROD.

Verified and found changes are reflected which is as follows:


AutoIndent not consistent when moving lines in JSON

Testing #29493

Steps to Reproduce:

open settings file (cmd+,).  mine has many settings, for example:

In the above example, put the cursor on "editor.renderWhitespace": "boundary", (line 13)
Move the line down to the "[markdown]": { section

Result: the "editor.renderWhitespace": "boundary", line is in properly indented.  This is goodness.

Now, move the line back up, outside of the "[markdown]": { section:

Result: the line is not outdented as expected.

This is because we don't have indentation rules for JSON. Now add a simple version which just checks brackets.


PC integration with Octane (#14)

Pc <-> Octane integration
JENKINS-45155

:white_check_mark: Build hpe-application-automation-tools-plugin latest.101-jnvexbgq completed (commit https://github.com/hpsa/hp-application-automation-tools-plugin/commit/79242a1513 by @radislavB)

Approved for test build

Please review sonar and codacy issues


[patch] Wrong range start value in debug message

After for(;;) soffset is beyond eoffset. )-;
rtcfx.diff.txt

Oops! Forgot about the sample of wrong text
RTCMemoryFixup:   RTCFX @ (DBG) boot-arg rtcfx_exclude specified, value = 80-FF
RTCMemoryFixup:   RTCFX @ (DBG) rtc offset token = 80-FF
RTCMemoryFixup:   RTCFX @ (DBG) rtc range from offset 100 to offset FF is marked as emulated

Thank you for the notice and fixe!


Mobile: Room members and chat scuffed

Describe the bug
The room members have a weird bug where they go up and down inconsistently
And chat spacing is scuffy sometimes
To Reproduce
Steps to reproduce the behavior:

Go to dogehouse.tv on mobile
Have your screen landscape
Scroll the member list up and down
See error

I don't have any steps for chat spacing, it just happens sometimes, probably because of some non-monospace font(?)
Expected behavior
The spacing should be consistent with the name and avatar
And the chat messages should have consistent spacing
Screenshots
Members scuffiness;
https://user-images.githubusercontent.com/81298611/115252018-611eb480-a12b-11eb-88e6-24e62b6145cf.mp4
Chat spacing scuffed;

What device are you on?
Xiaomi Pocophone F1 (Android 10)
Brave Mobile (1.22.71)
Additional context
n o

Here it's really visible
https://user-images.githubusercontent.com/81298611/115253556-c8893400-a12c-11eb-9d6c-a6e3b0c14d9e.mp4


TextArea with useId causes duplicate ID

Current Behavior
When using the id from React's useId on the Textarea component, it triggers a duplicate ID error.

It's worth noting this only seems to happen with the TextArea component.
i.e.
export function Screen() {
  const id = useId()

  return (
    <TextArea id={id} placeholder="Duplicate ID here" />
  )
}

Expected Behavior
To be able to use TextArea without a duplicate ID error
Tamagui Version
1.101.5

Platform (Web, iOS, Android)
only tested iOS

Reproduction
https://github.com/NerdCowboy/tamagui-textarea-bug

System Info
System:
    OS: macOS 14.5
    CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
    Memory: 5.36 GB / 64.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 15.0.0 - ~/.nvm/versions/node/v15.0.0/bin/node
    npm: 7.24.2 - ~/.nvm/versions/node/v15.0.0/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 126.0.6478.127
    Safari: 17.5
  npmPackages:
    @babel/runtime: ^7.24.6 => 7.24.6
    @manypkg/cli: ^0.21.4 => 0.21.4
    check-dependency-version-consistency: ^4.1.0 => 4.1.0
    eslint: ^9.3.0 => 9.4.0
    node-gyp: ^10.1.0 => 10.1.0
    prettier: ^3.2.5 => 3.3.0
    turbo: ^1.13.3 => 1.13.4
    typescript: ^5.4.5 => 5.4.5

Has anyone resolved this problem?
I'm experiencing the same error.

Same for me

I created a PR: https://github.com/tamagui/tamagui/pull/2917


iFrame not returning .text

I'm accessing a site where, for some reason, the iFrame won't fully load by itself in a browser tab...it needs the parent around it.
The .text method is returning an empty string on the iFrame:
ifrm = browser.iframe(:index => 0)
print "iframe html is " + ifrm.html      # works
print "iframe text is " + ifrm.text      # doesn't work, empty string

A 'hack' workaround is to save the .html to a local file, .goto that, then use .text.
Ugly, but it's working.

browser.iframe(:index => 0) is an iframe element in the top level browsing context. Within the scope of the top level browsing context it does not have any text inside it, because it is opaque. To display a value, you need to switch into the iframe's browsing context. In Selenium you must do that explicitly, in Watir it does it implicitly so long as you are working with elements: browser.iframe(index: 0).div(class: foo).text
In your case, this will switch you into the iframe context and provide what you are looking for:
iframe_html = browser.iframe(index: 0).element.html
iframe_text = browser.iframe(index: 0).element.text

@p0deje - what do you think about making defining the browser methods in the FramedDriver class such that it automatically switches into the browsing context for these methods directly?

Thank you both for your quick replies.  I'm not quite sure I completely follow, however, as I thought
ifrm = browser.iframe(:index => 0)
would give me the iframe's content.
I'm also unclear why
browser.iframe(:index => 0).html
gives me results, yet
browser.iframe(:index => 0).text
does not.

IFrames are DOMs within the DOM. The driver can not see inside a nested DOM without being specifically told to switch into it. Right now Watir is currently implemented to only do that if you specify an element to interact with as a child element of the IFrame
So:
browser.iframe(index: 0) gives you a Watir::IFrame object, and there is no reason at that point for Watir to instruct the driver to switch into its context.
Ah, I didn't realize #html is already being treated special.
I don't like the way it is being done, and we should have #text as an option as well.
If @p0deje likes #416 then we can give you that functionality in the next release.

fixed here - 6f55c8b2f463d5151b4cbacadd021685ed41ec70


汇总/更新 slack 信息

relates to https://github.com/wuhan2020/wuhan2020.github.io/issues/152

相关石墨文档，https://shimo.im/sheets/6YcV693Y3wqC6qJk/MODOC

Closed via https://github.com/wuhan2020/wuhan2020.github.io/pull/201


Poor handling of intermittent slow connections

The releng team is seeing that when pivnet is intermittently slow (requests taking >1 min sometimes) then we get consistent failures in the put step for this resource because the resource makes a lot of requests, and eventually one of them will fail before the whole operation succeeds.
It would be nice if we could specify a higher HTTP timeout for this resource so that put is more resilient to slowness on pivnet.

Hi, I'm just updating this issue with a code patch that Dennis & I paired on to temporarily work around the issues we were having in the releng delivery pipelines.
We didn't want to make a PR with these changes because they are a bit blunt, and we didn't test drive anything, so the code quality may not be fully up to snuff.  Also, for expediency, we just vendored in the dependent modules, and made changes to the dependent modules. You will probably want to unwind that change.
The gist of the changes was:

we changed the default HTTP connection timeout from 1 minute to 20 minutes
we made some changes to store the passed in lager logger so that we could use it to add additional logging
we added retry loops to most of the operations
we changed some internal code to fail instead of panicking, so that retry would be possible

We put these changes on a private fork in Dennis' org, so we figured the simplest way to deliver them would be in a patch file.  Dennis has created that file, and I have attached it here.
patch.txt
Let us know if having a pair would be helpful if/when you pick this up.
Thanks!

Thank you for the patch. We'd like the timeouts to be user-configurable, and might look at applying the retry logic as a wrapper around go-pivnet's http clients to reduce the duplication. You can follow our progress here https://www.pivotaltracker.com/story/show/173929827


Custom Mutation/Query

Let's say I want to create a signupWithFacebook mutation or loginWithFacebook query that receives a facebook token. These can't/shouldn't be implemented as db level procedures since they need to query the facebook api to validate the token. How would I go about implementing something like that?

So there are a few approaches to interacting with the world outside the database.

LISTEN / NOTIFY - you can have a script running in the background listening for when these mutations occur.

Write your mutation functions in something like Python or JavaScript and make whatever API calls you need.

Use a foreign data wrapper in combination with something like Multicorn. This would be an interesting approach.

I'm sure there's more ways, but these are what seem to come up most frequently when I've had similar questions.

👍
Personally, I bypass PostGraphQL for authentication and just use Passport.JS and direct database access.

@mgallagher
1: I can't use that since the API calls needs to be run before the mutation/SQL query is executed
2: I would have issues with such an approach for the following resons:

I doubt this scales well in terms of connection usage and resources usage on the db server.
It would be a security issue to start to open up a db server to outside world.
I don't think that writing bare javascript or pyton to interact with external services is a good solution.
3: Looks like it could work but it suffers from the same drawbacks of 2.

Have you guys thought of using hooks/events like before(mutationname, data) to be able to manipulate the incoming data before withPostGraphQLContext is run?

@benjie 👍  I see that would be the simplest approach I guess.

There's been a some discussion around various hooks but no definite plan that I'm aware of.

@benjie so do you still use database roles? Or do you implement authz another way?


Fix linux4_14 feature flags

Before this change, code intended for the linux4_14 feature would not get compiled in as expected. The reason is that the feature flags were not specified in the way Rust expects (this is a common footgun).
This change fixes the linux4_14 feature flag usage, and safely casts the ptid to the expected nix::unistd::Pid.

Nice catch and thanks for the PR.
This appears to close #31 too.

@rw it looks like there's another #[cfg(linux4_14)] in src/builder.rs. Do you mind correcting that one as well as part of this PR?

@rw it looks like there's another #[cfg(linux4_14)] in src/builder.rs. Do you mind correcting that one as well as part of this PR?

Could you link me to the line? I'm not seeing it :-]

@rw Sorry, my mistake. My repro's main branch was woefully out of date and it looks like src/builder.rs was corrected 17 months ago 🤦.
That should do it for the #[cfg(linux4_14)] references.

This looks good to me. Can you please push it and create a new release? I'm also running into this issue.
Thanks for fixing!

Thanks!