Error in test_roi_pooling_conv.py

When I tried to run the script it gave me the error
AttributeError: 'RoiPoolingConv' object has no attribute 'pool_list'
Also can you please create an example for using the scripts
thanks

Thanks for reporting this. I'm now aware of some issues with the tensorflow backend, it should be fixed over the next day or two.

Is RoiPooling essentially a "flattened" version of RoiPoolingConv, for each RoI ? In other words, it returns the various RoI feature vectors rather than the RoI feature maps?

@fferroni yes that's correct.
One advantage of RoiPooling of it is that you can pool at multiple sizes since the output is all flat. With RoiPooling, you can for example have a 1x1, a 2x2 and a 4x4, all flattened into a 1+4+16=21 length array.
But with RoiPoolingConv, if you have a 7x7 output, you can't have a different size one, since your array would be uneven.

Thanks. Yes, that sounds nice.
Given that you mention i.e. Fast RCNN in your readme, the output of this RoiPooling layer would then be passed through some FC layers and split into a softmax output and a bbox regressor output. But in Keras, how do you do this for multiple ROI? The output of this RoiPooling would be (None, nb_roi, nb_channels * num_outputs_per_channel) but the classification and regression needs to be done for each roi.

Hey,
The trick is to use TimeDistributed, which will apply it as you want. It's kind of awkward, but I see no other way to do it. I am working on a prototype faster-rcnn implementation right now, you can see the model definition here
Basically the layer look like this:
def classifier(base_layers,input_rois,num_rois,nb_classes = 21):

    pooling_regions = 7

    out_roi_pool = RoiPoolingConv(pooling_regions, num_rois)([base_layers,input_rois])
    out_class  = TimeDistributed(Convolution2D(...))(out_roi_pool)
    ...
    out_class  = TimeDistributed(Flatten())(out_class)
    out_class  = TimeDistributed(Dense(nb_classes,activation='softmax'),name='dense_{}'.format(nb_classes))(out_class)

    return (out_class)

For RoiPooling, use TimeDistributed(Dense(...)) instead of TimeDistributed(Convolution2D(...))

Ha! You beat me to it ;-)

Ha not really, mine is still very much a work in progress. It is quite a large task, I would certainly be interested if you want to collaborate or submit some pull requests.

I've finally fixed this. All test should now pass for both the theano and tensorflow backend (tested with keras 1.2.2, theano 0.8.2, tensorflow 1.0.0). Note that there are small differences in how rounding is handled by default, so results may not be exactly reproducible between pure numpy vs theano vs tensorflow is pooling sized that don't divide exactly (i.e. pooling size 16 into a size 3).
For a more complete example on using these layers, see the work on frcnn I've begun here:
https://github.com/yhenon/keras-frcnn
Please feel free to reopen this if you have issues.

@yhenon are u planning to do a pull request to keras?


feat(ci): add GH actions release pipeline

Adds GH actions release workflow:

builds docker image and published it to docker hub
copies k8s manifest to S3
supports building pre-release and changing from pre-release to release

Fixes #7

closing in favor of https://github.com/newrelic/nri-kube-events/pull/10

Closed this PR as i removed the repository by mistake :(


optional manual/automatic Geotagging

Similar to this video:
https://youtu.be/bEBWJnqzgU0
There are some captions for places like Spain

Any idea when this might be released?

@KyleKun any idea when this might be released?

Probably in January since Play Store review can take a while, but I'll try to get a new apk released here by next week so we can start the year with a brand new version 😁


add current_actor() SQLite function

I'm not sure how achievable this is. #49 explores creating something like updatable views for the turker use case. It'd be handy to have a trigger that can set last_edited_by = current_actor(), where current_actor() is evaluated in the context of the current request to get the user's ID.
Can ... can we do that?
Simon mentioned maybe you can abuse asgi_wrapper to get access to the request: https://discord.com/channels/823971286308356157/823971286941302908/1070385463640215702

Hmmm, this might work for read-only queries, but I think write queries are executed in another thread?
Maybe we could use datasette-rewrite-sql to rewrite the query to include a comment with the actor ID, and then, just before actually passing the SQL off to SQLite, we'd set a thread local.

https://blog.sqreen.com/asyncio/ seems relevant

Huh, would this also let you write views like:
SELECT row_level_security_view AS SELECT * FROM data WHERE approved_by = current_actor() OR current_actor() = 'superuser'

The view would only work in the context of Datasette, not via vanilla sqlite CLI.

The request that actor_from_request is passed is a datasette thing: https://github.com/simonw/datasette/blob/0b4a28691468b5c758df74fa1d72a823813c96bf/datasette/utils/asgi.py#L56
We could mint one in order to call actor_from_request...but I wonder if plugins depend on getting the same request (eg, to cache a lookup, to log things).
The actual auth happens here: https://github.com/simonw/datasette/blob/0b4a28691468b5c758df74fa1d72a823813c96bf/datasette/app.py#L1481-L1522
Maybe we can write a hookwrapper for actor_from_request? See https://pluggy.readthedocs.io/en/stable/#wrappers

We might also be able to have a hook that stashes the request in the current_task ? Then we could always inspect stashed_request.scope.actor.

Stashing the request seems a little less skeezy, then we don't have to interrogate the result object and risk Datasette's logic changing.

Even once we've stashed it, we have the problem that we're in some asyncio context, and the actual execution of the SQL may be in a different thread, e.g. for writes.
A not-very-good solution: in rewrite_sql replace all instances of current_user() with a literal value.
Ah, but this doesn't actually solve the case I care about, which is I want to be able to create views with RLS where clauses, and have triggers that do the right thing.

Another oddness:
I did create view xxx as select current_user(), then went to http://localhost:8001/cooking/xxx
actor_from_request hadn't/didn't run before rewrite_sql...maybe we need something earlier?

Alternate idea: datasette-rewrite-sql only patches the entry points like execute, execute_write, execute_write_many, execute_write_script.
This isn't actually what needs to be patched -- we need to patch execute_fn and execute_write_fn, to control setting the value right before we pass control to SQLite.

I have a buggy proof of concept, it's incorrect under concurrent load.
I added an actor_from_request hook like:
@hookimpl
def actor_from_request(datasette, request):
    _id = None
    if 'x-me' in request.headers:
        _id = request.headers['x-me']
    return {'id': _id}

And then tested by having multiple bash windows open like:
while :; do curl -H "x-me: $$" http://localhost:8001/cooking/xxx.json?_shape=array; echo; done

I expect the output from each window to always show the same value, but instead, I get interleaved results.
This means my understanding of something is flawed. I wonder if it works with num_sql_threads=1? I stash the request on the asyncio.current_task(), but when we transition into SQLite I stash the ID in a global...that might be flawed.

--config num_sql_threads=1 gives correct results, so yeah, a global is not an OK way to store this.

Using a threading.local() resolves this

This might be worth its own plugin, and we'll just depend on it.

Released as https://github.com/cldellow/datasette-current-actor


Service Injection in module blade template

So I am currently trying to do the following:
@inject('data', 'Modules\Atoz\Providers\AtozServiceProvider')

<div>
	Monthly Revenue: {{ $data->getData() }}.
</div>

But its throwing this error:
Unresolvable dependency resolving [Parameter #0 [ <required> $app ]] in class Illuminate\Support\ServiceProvider (View: /home/vagrant/Code/Mekon/Modules/Atoz/Resources/views/dash-widget.blade.php) (View: /home/vagrant/Code/Mekon/Modules/Atoz/Resources/views/dash-widget.blade.php)

Any ideas? I'm kid of at a loss... Everything else is functioning perfectly with the module.

I've never used that feature of laravel, and wont be any time soon so I can't really help you here.
I doubt you want to inject a service provider into a view, maybe a repository instead ? Make sure that repository is bound in the IoC container.


Highchart not displaying inside a directive

I've got a directive that contains highchart-ng and I need to update the series data inside the directive from my Service to fetch the data.
I got it working without the service - basically this works https://jsfiddle.net/w657bcdp/
However, if I update the series inside my service, it doesn't work. The debugger shows the chart series data being loaded but does not display.
...
var defer = $q.defer();
defer.resolve(DashbboardSrv.loadStats());

defer.promise.then(function(resp){

	...

	var data = [
                    [Date.UTC(1970,  9, 18), 0   ],
                    [Date.UTC(1970,  9, 26), 0.2 ],
                    [Date.UTC(1970, 11,  1), 0.47],
                    [Date.UTC(1970, 11, 11), 0.55],
                    [Date.UTC(1970, 11, 25), 1.38],
                    [Date.UTC(1971,  0,  8), 1.38],
                    [Date.UTC(1971,  0, 15), 1.38],
                    [Date.UTC(1971,  1,  1), 1.38],
                    [Date.UTC(1971,  1,  8), 1.48],
                    [Date.UTC(1971,  1, 21), 1.5 ],
                    [Date.UTC(1971,  2, 12), 1.89],
                    [Date.UTC(1971,  2, 25), 2.0 ],
                    [Date.UTC(1971,  3,  4), 1.94],
                    [Date.UTC(1971,  3,  9), 1.91],
                    [Date.UTC(1971,  3, 13), 1.75],
                    [Date.UTC(1971,  3, 19), 1.6 ],
                    [Date.UTC(1971,  4, 25), 0.6 ],
                    [Date.UTC(1971,  4, 31), 0.35],
                    [Date.UTC(1971,  5,  7), 0   ]
                 ];

    			scope.chartConfig.series = [{
					  	type: 'area',
					  	name : 'votes',
					    data: data,
					    tooltip: {
		                    valueDecimals: 2
		                }
					  }]
}
, function (err) {
    console.trace(err.message);
})

Any idea why this is happening?

Apparently I found the issue in my directive link function. I had wrapped the code above in a function within the link function itself and was causing issues in AngularJS probably.
The solution is to create the function with scope:
...
link: {
    	pre: function(scope, element, attrs) {

    			scope.chartConfig = {...}

                scope.loadStats();     //loadStats() wont' work.

               scope.loadStats = function(){...}
       ...
      }


[New game]: Word scribble game

🎮 Game Request
Game logic and basic description
Word Scribble is a word puzzle game where players rearrange a set of scrambled letters to form meaningful words. The game logic involves several key components to ensure an engaging and challenging experience for the player. Here's a detailed breakdown of the game logic for Word Scribble:

Game Initialization

Word List Selection: Choose a list of words appropriate for the game's difficulty level.
Random Word Selection: Randomly select a word from the list.
Scramble Letters: Shuffle the letters of the selected word to create the scrambled word presented to the player.

User Interface (UI)
Display Scrambled Word: Show the scrambled letters to the player.
Input Mechanism: Provide a way for the player to input their guesses (e.g., text input box, draggable letters).
Feedback System: Display feedback for correct or incorrect guesses.

Gameplay Loop
Start Game: Display the first scrambled word to the player.
Player Input: Allow the player to rearrange the letters or type their guess.

Check Guess:
Compare the player's guess to the original word.
If the guess is correct, move to the next word or end the game if it was the last word.
If the guess is incorrect, provide feedback and allow the player to try again.
Point down the features
Game points
Scoring System
Score Calculation: Assign points based on the accuracy and speed of the player's guesses.
Correct guess on the first try: Maximum points.
Correct guess after multiple attempts: Fewer points.
Bonus points for completing the game quickly.

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

Hey @codewithishu!
We are already having a similar game request in #3949 👀
Make sure you come up with a cool unique idea 😀
Waiting for your new game idea 💗.

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


Create related links macro and some other minor tweaks

Create related links macro. This makes related links more consistent across templates and addresses @duelj's styling issue.
Moves include files into _includes.
Fixes an issue with tag links not working in Blog and Newsroom.
Pulls tags into its own macro and adds comments to article.html.

Very nice :+1:

Gracias!


Address already in use after update

Hi all,
I am running this fantastic tool for a while. I want to update to the latest image, because of #356 (which got merged recently 🥳 ).
This is the error I get:

app_1  | [4/15/2020] [12:58:05 PM] [SSL      ] › ℹ  info      Let's Encrypt Renewal Timer initialized
app_1  | [4/15/2020] [12:58:05 PM] [SSL      ] › ℹ  info      Renewing SSL certs close to expiry...
app_1  | [4/15/2020] [12:58:05 PM] [IP Ranges] › ℹ  info      IP Ranges Renewal Timer initialized
app_1  | Uncaught Error: listen EADDRINUSE: address already in use :::3000

I used docker-compose pull and I was under the impression that it would be an easy upgrade. I tried  jc21/nginx-proxy-manager:2 and  jc21/nginx-proxy-manager:latest
Is there any way I can help debug this?

Nevermind, I found it. Apparently the newer version binds to port 3000, and I am also proxy'ing a port 3000. Solved!

In hindsight, it would be nice if https://github.com/jc21/nginx-proxy-manager/blob/bb0f4bfa626bfa30e3ad2fa31b5d759c9de98559/backend/index.js#L24 was an env-variable. Maybe I'll contribute that in the future!

Well no it doesn't matter what port the backend binds to inside of it's own network.

No, but if I already use port 3000 for nginx (streaming ports), it makes sense :) The message came from the container, not the host.

Ah yes of course

How do you feel about making it an ENV-var, @jc21 ?

I feel uneasy about it. The backend api can listen on any port easy enough with a env var deciding what that port is. But, the Nginx configuration forwards requests from the Admin UI to the API with <IP_ADDRESS>:3000 and that would need to be dynamic as well.

As a workaround for yourself, you can map ports in your docker-compose file 3000:3001 and then set up a stream for 3001 to forward wherever you like.

Hmm yes, did not think of that, good point @jc21. I changed my infrastructure to adapt with the change anyway.


O_VLIELAND: good records marked as outliers

The current outlier settings seem to mark good migration records for individual 5517506 as outliers (orange):

Resolved and data reuploaded.


New records from create are undefined (v1)

Sails version: v1.0.0-20
Node version: v6.3.0
NPM version: v3.10.8
Operating system: Win10

Minimal model:
module.exports = {
  attributes: {
    value: {
      type: 'string'
    }
  }
};

Creating new model within a controller (or wherever else, also tried in service):
Bar.create({
      value: 'foobar',
    }).exec(function(err, records) {
      if(err) {
        sails.log.error(err);
      }

      sails.log.debug(records);
    });

    return res.ok();
  }

Log:
debug: undefined

Just in case:

sails-hook-orm is at 2.0.0-7
Waterline is at 0.13.0-3

You should read the upgrading to v1 doc
Answer to your question link

Checked the docs, forgot the upgrade guide. Whoops.
Thanks.


Skip SSH key strength check for ECDSA keys

This appears only to be relevant for RSA and DSA keys and was already being ignored for newer Ed25519 keys, but ECDSA keys were still being checked.

@AgaDufrat I think you're right about updating that documentation. The second line in particular seems misleading: it won't necessarily have == in (that's just random chance that base16 worked out that way) and it doesn't have to end with an email address, although it does by default. I'll take a look at that at some point.


Fix ContinuousRectangleBorder bottom radius

Description
Currently bottom left radius is drawn on bottom right, and bottom right radius is drawn on bottom left. This commit fixes the behavior.
Related Issues
None.
Tests
I updated the existing test case, it used to set bottom right radius but check against an image with bottom left round corner, now the test set bottom left radius.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

@a9udn9u Do you have plans to come back to this PR?

Yes but unfortunately I don't have time right now, it would be great if someone can take over otherwise I might be able to work on this in mid-Dec.

@Piinks Are you interested in taking this on?

I can pick it up. 😉 I will close this to reopen as a new PR.


Update logic for determining list position

Fixes https://github.com/github/issues-graph/issues/825
This PR adjusts the logic used to calculate the positioning of lists within a comment to satisfy additional ul and ol usage.
See linked issue for additional context.

You may want to double-check these combos:

Nested tasklists
Nested mixes of checklists and tasklists
Tasklists/checklists within HTML tables (not MD tables)
Nested tasklists within HTML tables

You may want to double-check these combos:

Nested tasklists
Nested mixes of checklists and tasklists
Tasklists/checklists within HTML tables (not MD tables)
Nested tasklists within HTML tables

Tested the above to confirm that no functionality was lost. The current implementation of a tasklist remains encapsulated so shouldn't be adversely affected by the new positioning logic when support for nesting w/in tasklists is added. 👍


[typespec-python] A LRO not generate correctly as defined in tsp

The issue in found in azure-ai-documentintelligence package, we expect to have a LRO for analyzing stream doc: tsp link
But what we got is a LRO as an override of analyzing the non-stream operation:  sdk link
There should has some issues in dpg to generate analyzeDocumentFromStream so that we could have a seperate operation for stream doc.

@tadelesh Thanks for your reply. I'll check with service team if that's expected.

I'm closing this issue because it's a tsp issue, @YalinLi0312 please reopen if anything new happens. Thanks @tadelesh !

@tadelesh  We'll add analyzeDocumentFromStream in client.tsp(https://github.com/Azure/azure-rest-api-specs/pull/27240), but we still can't get it generated as multiple content types are not supported that reported in issue https://github.com/Azure/autorest.python/issues/1526. Do you have any plan for this feature?

@iscai-msft We want to generate an operation with multi content type defined in routes.tsp but the generator recommends to custom the operation in patch.py. We are not expecting such customizations in GA, can you give priority to this issue?

Before we can generated anything, we need a strong design proposition that works generically across all possible TSP we can think of.
It's a complex problem, we should rush it

closing because this was fixed when there are multiple streaming content types. For a mix of json and streaming content types, follow this issue https://github.com/Azure/autorest.python/issues/1526


fixed spelling mistake of function

Overview
This PR includes the following proposed change(s):

fix for activity page of isMobile function

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Screenshots
Please add any relevant UI screenshots if applicable.
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] New and existing unit tests pass locally with my changes

/rebase


bug(schematcis): Failed ng add. Main file not found

Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
16
Description
When I am using ng add @angular/material@17, I have the following error : Could not find the project main file inside of the workspace config (src).
Reproduction
Please do the following command :

ng new test-app
ng add @angular/material@17

Expected Behavior
Ng add works correctly
Actual Behavior
It is impossible to add material with schematics with application builder.
Environment

Angular: 17
CDK/Material: 17

Is this connected to https://github.com/angular/angular-cli/issues/26284?

Is this connected to https://github.com/angular/angular-cli/issues/26284?

I don't think so. The 26284 issue seems to be a typescript compilation problem. With this one the main.ts is not found with schematics in src folder. I am not sure.


Add BC Testing for prior client versions

Add a simple test harness linked to pulsar client 2.2.0 for backwards compatibility testing.

run java8

run java8 tests

run java8 tests


Fixed subreddit limited search

The option "restrict_sr=on" is need to make the result limited to the specific subreddit. Otherwise, we get results from unrelated subreddit.

We don't use any of these legacy searches here so I don't have an easy way to validate your changes.  However, since tests went ahead and passed, I'll accept the PR.  Thanks!


Transform the basetest binary into unit tests

The basetest binary is used to test the base library. At this time basetest is launched manually. We have to transform it into unit tests.

Do you need help for this one ?


Cross platform sockets

Cross platform sockets

Adding glz::socket and glz::server for TCP networking
Builds upon these low level tools to replace glz::asio_server with glz::repe_server and glz::asio_client with glz::repe_client

[!IMPORTANT]
This code is cleaner and more future proof using std::format. When this is merged we will drop support for GCC 12 to support std::format.

Removing asio
This change replaces the glz::asio_server and glz::asio_client with glz::repe_server and glz::asio_client.

Removes the asio third party dependency
Allows recursive request calls with clients
Allows us to develop networking for Glaze at a faster pace
Allows us to optimize networking
Allows tighter integration with RPC and network protocols (REPE, etc.)

@meftunca, I'm pinging you to let you know about this socket development for Glaze. After running into limitations with asio, wanting to push performance and ease of use, and realizing that asio does not accept much outside influence, we've decided to implement our own cross platform sockets in Glaze.
We're able to use a simpler approach by not needing to support older versions of C++ like asio. And, we can more cleanly handle various protocols and formats by using the same template options. The socket class is still very abstract, but it enables us to send either JSON or binary without the need for an additional protocol. But, we can also build a protocol like REPE on top of it (which I'm also refactoring) so that we can more efficiently handle RPC calls and more.

I wrote a simple socket service with io_uring and kqueue and I was able to handle more than 150k requests on average.  My personal interpretation is that libraries like asio will continue to be developed by people with different concerns to fit certain patterns. Doing an independent development on Glaze is really the right conclusion.
Is there any planning or roadmap for the Socket project?

https://github.com/meftunca/muxpp
To be honest, I'm not very knowledgeable about socket programming, but I wrote a learning project anyway. I designed an ExpressJS-like interface. I was hoping to complete phase 1 by finishing client-server communication by routing the final requests to the relevant EndPoints :D

It can also be more performant to route large requests to coroutines for workflow management, congestion reduction and performance.  I am sure it will be more performant if coroutine support is provided as an option.

After analyzing some socket code,  realized that you use threads very generously :) At first, I used threads generously in this way and I lost performance, let alone the performance I expected.

Thanks for the feedback. Yes, currently the threading is on a per client basis, which isn't the best approach for a lot use cases. I plan to add a thread-pool solution as well as coroutines in the future. This is just an initial implementation to figure out the API and architecture.

Just wondering: does this plan to support websockets? Or is there a way I can attach existing websocket to this?

@sam20908, yes within the next few months we plan to add Websocket support. The goal is to have the Websocket protocol separate from the socket code as well, so either the protocol or the socket code could be swapped out.
Right now we're working on cross platform coroutines with will be added to Glaze to support efficient server implementations.

Nice to hear! I realized that is there a way one can use their own connection? E.g. I use MBedTLS to create a TLS connection (adding TLS support for Glaze sounds like pain)

Yes, I'd like to use templates on the sockets and create a general wrapping interface that will allow any networking library to work with the rest of the protocols, coroutines, etc.

Hey,
Is there an ETA for when this PR will be merged into the main branch? I would love to use this for my current project, which would improve my current implementation of RPC.
Thanks

We're still figuring out the best approach to handling the underlying concurrency/scheduling. It's hard to say when this will get merged. I would say it will probably take until the end of September. We've decided to use https://github.com/NVIDIA/stdexec, which has been accepted into C++26. It could be sooner, but I don't want to disappoint.

btw im wondering if theres a reason why you're implementing ur own sockets instead of having the users getting their own sockets setup and just parse the json response using this library?

The socket implementation has to be somewhat tied to the scheduler (I think). I wish https://github.com/dietmarkuehl/stdnet were further along, but it isn't mature and is not cross platform. As much as possible we want to separate the sockets, scheduler, payload spec, and RPC spec, but we need cross platform versions of all these that we can iterate on to figure out a general solution.

I don't understand, there are cross platform libraries that handles socket connection. What does this do that other socket libraries don't

Currently Glaze uses asio, and I intend to still allow asio sockets. I've also experimented with libuv sockets. But, cross platform sockets is the easiest part of this implementation and it would be nice to not need third party dependencies.
There's also the fact that we'd like to eventually handle HTTP3 with QUIC, which asio doesn't have support for. One can argue that just simple UDP sockets can be used underneath a QUIC implementation, but then why bring in a big dependency like asio to use only a tiny piece of it. There are also some optimizations that get easier to manage when it's possible to manage the socket code oneself, particularly when it comes to header handling in various protocols. The project I'm working on has networking needs for various context, web UI, hardware interfaces with extremely low latency, and more, so there's a lot to consider.

ok that makes more sense, i look forward to use it in my project when websocket support comes lol


Fix  redundant type conversion

Changes

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[ ] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
Describe any user facing changes here, or delete this block.

Examples of user facing changes:
- API changes
- Bug fixes
- Any changes in behavior

@vdemeester needs /ok-to-test 🐱

/ok-to-test

/meow

/retest


ses_bounce_logging_blog.yml - Unsupported runtime specified for the Lambda Function

The Lambda Function Runtime withinn the 'ses_bounce_logging_blog.yml' template is specified as 3.6, this runtime is no longer supported causing the stack creation to fail.
Runtime needs to be increased to 3.9, but I don't know if any python code changes would be required to support this?

I just changed the runtime to 3.8. Not sure what the deprecation schedule is like but it seemed to deploy and work without problem

At the time of writing python 3.7 still has no deprecation date https://github.com/awsdocs/aws-lambda-developer-guide/blob/a83f69b/doc_source/lambda-python.md


Variable Cost Refactor Part 2: PowerSimulations

The second part of a significant refactor of Sienna cost data structures (part 1 is here). The main focus here is to bring PowerSimulations.jl up to date with the elimination of VariableCost in favor of FunctionData. That entails these changes in InfrastructureSystems:

Make PiecewiseLinearPointData use NamedTuples so it's even harder to mix up x and y
Add the notion of "raw data" to support storing data from FunctionData in other containers, like the ones in PowerSimulations

On the latter point — does this functionality belong in PowerSimulations?
Tests pass:

The group of four PRs this is a part of:

https://github.com/NREL-Sienna/InfrastructureSystems.jl/pull/332
https://github.com/NREL-Sienna/PowerSystems.jl/pull/1059
https://github.com/NREL-Sienna/PowerSystemCaseBuilder.jl/pull/73
https://github.com/NREL-Sienna/PowerSimulations.jl/pull/1061


Add new procedural article "Filtering the assignment overview" for GitHub Classroom docs

This issue is reserved for our MLH Fellowship program ✅
Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
This will be a new article in the GitHub Classroom section of the docs site called "Filtering the assignment overview."
This article could be placed in the "Teach with GitHub Classroom" topic, under "Edit an assignment."
What part(s) of the article would you like to see updated?
This issue is reserved for the MLH Fellowship program.
Why
Currently, the only information about the assignment overview page in the GitHub Classroom docs exists in the "Monitoring students' progress" sections of "Create an individual assignment" and "Create a group assignment - GitHub Docs." While these sections are useful for understanding the assignment overview page at a glance, they don't include any procedural information on filtering the assignment overview page for specific criteria. Creating a new procedural content on filtering the assignment overview page will provide teachers with more tools and showcase another useful feature in GitHub Classroom.
Use cases and audience
This article is needed to show teachers how they can filter the assignment overview for specific criteria so they can better track their students' progress. It also showcases a useful feature in GitHub Classroom, potentially generating interest for prospective users.
Getting started
You can get started by learning more about procedural content in our content model.
You can then view our Procedural content template and follow it to create the article.
The article should be created in the education/managing-your-coursework/teach-with-github-classroom directory.
The “Filtering the assignment overview” article should have the following sections, at least:

About filtering the assignment overview: this should be a brief conceptual intro to filters on the assignment overview page. We can introduce the assignment overview page and explain why a teacher might filter it.
Applying filters to the assignment overview: this should be a procedural section with instructions on filtering the assignment view. We should include instructions on using the dropdown menu filters available next to the search bar, and on filtering with the search bar itself. Feel free to reach out to me with any questions on available filters/testing 🙂

Additional information
This is for a special project and not currently available for contributors to work on.

👋🏼 @OlayinkaAtobiloye, here's the issue for the article I mentioned earlier! Let me know if you have any questions 🙂

Yea, no clue wtf to do with any of this

Still don’t understand
Get Outlook for iOShttps://aka.ms/o0ukef

From: github-actions[bot] @.>
Sent: Saturday, May 27, 2023 9:27:19 AM
To: github/docs @.>
Cc: dy214 @.>; Comment @.>
Subject: Re: [github/docs] Add new procedural article "Filtering the assignment overview" for GitHub Classroom docs (Issue #24637)
A stale label has been added to this issue becuase it has been open for 60 days with no activity. To keep this issue open, add a comment within 3 days.
—
Reply to this email directly, view it on GitHubhttps://github.com/github/docs/issues/24637#issuecomment-1565580382, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A57AG7FIILKWBRM6L34R7LLXIITOPANCNFSM6AAAAAAWFYJWHE.
You are receiving this because you commented.Message ID: @.***>

We completed this internally, so I'll close it out now 🙂


Local development environment:

This aims at making it easy to set up a local environment for development:

a script to setup two kind clusters and to generate the manifests for their registration in kcp
a script for running kcp locally
an instruction page on how to set up a local development environment

Signed-off-by: Frederic Giloux<EMAIL_ADDRESS>

@bnallapeta @Roming22 could you please have a look? I would be glad to get that merged before it gets out of date and to keep it actualized as things progress

I'm OK with merging as is and starting iterating on smaller issues.


Update KubeFedConfig API fields to be optional

For fields we plan to provide a default such that we won't require the user to specify them, we need to make them optional in order to pass kubectl client side validation. CRD client side validation will be enabled by default in k8s v1.15. See https://kubernetes.io/docs/tasks/access-kubernetes-api/custom-resources/custom-resource-definitions/#publish-validation-schema-in-openapi-v2 for more details.
Additionally, optional fields are made pointers. This is to distinguish the user not specifying a value (since it's not required), from the user specifying an invalid value of 0 (for Duration and time) or "" for the empty string (should produce a validation error). It also makes serialized objects cleaner.

/lgtm

@font It looks like merging this broke master due to linting errors:
+echo Linting
Linting
+golangci-lint run
WARN [runner/megacheck] Can't run megacheck because of compilation errors in packages [sigs.k8s.io/kubefed/pkg/controller/webhook sigs.k8s.io/kubefed/pkg/controller/webhook/federatedtypeconfig sigs.k8s.io/kubefed/pkg/controller/webhook/kubefedconfig sigs.k8s.io/kubefed/pkg/webhook sigs.k8s.io/kubefed/pkg/apis/core/v1beta1/validation [sigs.k8s.io/kubefed/pkg/apis/core/v1beta1/validation.test]]: pkg/apis/core/v1beta1/validation/validation.go:174: cannot convert elect.ResourceLock (variable of type *v1beta1.ResourceLockType) to string and 32 more errors: run `golangci-lint run --no-config --disable-all -E typecheck` to see all errors
pkg/apis/core/v1beta1/validation/validation.go:174:88: cannot convert elect.ResourceLock (variable of type *v1beta1.ResourceLockType) to string (typecheck)
	allErrs = append(allErrs, validateEnumStrings(electPath.Child("resourceLock"), string(elect.ResourceLock),
	                                                                               ^
pkg/apis/core/v1beta1/validation/validation.go:198:84: cannot use health.PeriodSeconds (variable of type *int64) as int64 value in argument to validateGreaterThan0 (typecheck)
	allErrs = append(allErrs, validateGreaterThan0(healthPath.Child("periodSeconds"), health.PeriodSeconds)...)
	                                                                               ^
pkg/apis/core/v1beta1/validation/validation.go:199:87: cannot use health.FailureThreshold (variable of type *int64) as int64 value in argument to validateGreaterThan0 (typecheck)
	allErrs = append(allErrs, validateGreaterThan0(healthPath.Child("failureThreshold"), health.FailureThreshold)...)
	                                                                               ^
pkg/apis/core/v1beta1/validation/validation_test.go:341:59: cannot convert "NeitherConfigmapsOrEndpoints" (untyped string constant) to *v1beta1.ResourceLockType (typecheck)
	validElectorResourceLock.Spec.LeaderElect.ResourceLock = "NeitherConfigmapsOrEndpoints"
	                                                         ^
pkg/apis/core/v1beta1/validation/validation_test.go:361:73: cannot convert 0 (untyped int constant) to *int64 (typecheck)
	validPeriodSecondsGreaterThan0.Spec.ClusterHealthCheck.PeriodSeconds = 0
	                                                                       ^
pkg/apis/core/v1beta1/validation/validation_test.go:377:59: cannot convert "NeitherEnableOrDisable" (untyped string constant) to *v1beta1.ResourceAdoption (typecheck)
	validAdoptResources.Spec.SyncController.AdoptResources = "NeitherEnableOrDisable"

I'm very much confused as to how CI managed to pass for this PR.

Lint errors are addressed in #973


feat(DIA-5): cleanup of ArtistAbout tab

This PR resolves DIA-5
Description
This PR clears the AboutTab screen removed sections we don't want to use anymore and has minor adjustments to it.

Adds missing title to Genes section
Adds a "Read More" feature(agreed with Product) to the Bio and make the bio data equal to what we have in Force (will PR later on palette-mobile)
Removes unnecessary components: ArtistSeriesMoreSeriesFragmentContainer, ArtistNotableWorksRailFragmentContainer, and ArtistConsignButton

iPhone 14 Pro Max
https://github.com/artsy/eigen/assets/15792853/9d556a34-a466-4687-8714-fa3a7fdcb704
iPad  Pro (12.9-inches)
https://github.com/artsy/eigen/assets/15792853/3f653486-47e0-4a81-9277-282108574027
@artsy/diamond-devs
PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

removes old sections from artist about tab - araujobarret

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Cross-platform user-facing changes (removes old sections from artist about tab - araujobarret)

  Generated by :no_entry_sign: dangerJS against 1b36093ad1f10b8b28372f83fce1c96d90795940


Attempts to attack with Dox underwater

Dox will engage underwater targets, leading to them just standing there.

This is a hardcoded bug.


fix(#217) typo and grammar in interview content

Description
Closes #217
Fixes the typo and grammar problem described in #217
Type of change
Please select the option that is relevant

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist

[x] I have commented my code, particularly in hard-to-understand areas.
[x] All the required documentation is added.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.

Sorry voor deze PR, maar ik werd er gewoon geïriteerd door toen ik dit las...

@Elmar-Wiese Hartstikke bedankt! Hierdoor hebben we juist gekozen voor GitHub, zodat iedereen zelf wijzigingen kan doen. Top!


Does this plugin uses ~/.ssh/config and possible to use tunnel?

Is it possible to use this plugin with configuration from ~/.ssh/config ?
For me it looks that this file is not taken into account. because i configured a hostname in config file and it get "unresolved Address"
Is it possible to use this plugin with a tunneled connection?
Because i need a tunnel to configure to the host where the interesting data is available

Hallo @ehmkah
The SSH config file is only taken into account if the JMeter property jmeter.sshmon.sshConfig is set to the file location.
I'm assuming you have set this?
See https://github.com/tilln/jmeter-sshmon#jump-hosts-since-v13
However, your error suggests it fails to resolve the hostname as perhaps the there is no DNS entry on the JMeter side of the network, before the tunnel/jump host.
This may not work with the underlying library and hasn't been tested.

It looks like I missed this part in your documentation. Your description looks more or less like the configuration I am using. Si try it. Thanks for your help so far. 👍


got a fatal message when exec make

I got a following fatal message when exec make command, but build succeeded.
$ make install
fatal: No names found, cannot describe anything.

this occurs on following line at GNUmakefile.
VERSION := $(shell git describe --tags --abbrev=0)

fixed


How to configure the Customer Account API in hydrogen for login and other operations

What is the location of your example repository?
No response
Which package or tool is having this issue?
Hydrogen
What version of that package or tool are you using?
Hydrogen
What version of Remix are you using?
No response
Steps to Reproduce

Expected Behavior
Click login to stop showing site errors
Actual Behavior
Related to configuration items

In order to use the Customer Account API for logged in users, you need to follow the instructions you have in your generated README.md (same as in this link).

install ngrok
setup urls in shopify hydrogen store
add url to your code.

Here's a tutorial as well: https://shopify.dev/docs/custom-storefronts/building-with-the-customer-account-api/hydrogen


Add allow_stock_to_be_exceeded flag in OrderFulfill mutation

Added allow_stock_to_be_exceeded flag in OrderFulfill mutation. This flag is set to False by default. If set to True then we can fulfillment an order when stock is out of quantity.
In addition:

added migration for Stock model for quantity < 0 -> saleor/warehouse/migrations/0015_alter_stock_quantity.py
fixed typo in module name -> test_fulfullments_actions.py → test_fulfillments_actions.py

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-4421 Extend OrderFulfill mutation by new flag to allow fulfill lines without stocks

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


Child theme requires parent activation first to Work

Thanks for the good work on this, Ben!  We're going to use Roots extensively in our client projects.
It took me about an hour to realize an issue I was troubleshooting while child theming was due to not having activated Roots first. The child-parent theme stack was assuming the existence of some objects that hadn't been created because the activation functions hadn't run to build them yet.
I think the specific issue is beside the point, but I could be wrong. So: the issue was an "undefined object" error notice (with WP_DEBUG set to true) displaying in place of the primary_navigation menu. The notice referenced line 49 of roots/headers.php. This went away and menus behaved as expected after activating Roots and then activating the child.
To save other child theme-ing fools like me some time, I'd like to suggest adding a note about this pitfall in the child theme documentation. Better still, because it would be less brittle, would be to trick the Roots activation functions into running when a child theme is activated.
I'd like to help and can start work on a patch for this if desired, but it'll be awhile before I can write something I'm confident suggesting. I'm learning fast, but still a WP and PHP newbie.

Patches are always welcomed. This isn't a major issue so we'll leave this as is until you have a chance to work on it. I've updated the documentation here: https://github.com/retlehs/roots/wiki/Child-theme

i wonder if this is any different since we've implemented activation options (#6)


Implement a PDDL writer

Closes #13

@mikand You guys passing on tarski entirely?

@haz For PDDL printing yes: we will soon need Temporal Planning and tarski does not support it at all. We decided that relying on tarski was simply not worth it given how simple the printer is. Parsing is entirely another story, we are looking into that.

Bummer. Hope would be that efforts would combine (PR's for tarski rather than re-implementations). I guess the lack of temporal support is at least something that should be lofted as an issue to @miquelramirez & the tarski team.

Codecov Report

Merging #15 (05f4b34) into master (cb92b92) will decrease coverage by 2.54%.
The diff coverage is 73.14%.

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
- Coverage   85.30%   82.75%   -2.55%
==========================================
  Files          24       30       +6
  Lines        1660     2157     +497
==========================================
+ Hits         1416     1785     +369
- Misses        244      372     +128

Impacted Files
Coverage Δ

upf/problem_kind.py
100.00% <ø> (ø)

upf/simplifier.py
31.38% <31.38%> (ø)

upf/problem.py
85.71% <76.19%> (-1.69%)
:arrow_down:

upf/io/pddl_writer.py
83.52% <83.52%> (ø)

upf/pddl_solver.py
88.23% <88.23%> (ø)

upf/fnode.py
91.26% <100.00%> (+0.79%)
:arrow_up:

upf/operators_extractor.py
100.00% <100.00%> (ø)

upf/test/test_pddl_io.py
100.00% <100.00%> (ø)

upf/test/test_pddl_planner.py
100.00% <100.00%> (ø)

... and 8 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 cb92b92...05f4b34. Read the comment docs.


Security hardening guide doesn't discuss protecting access_key_id and secret_access_key

I think the lack of discussion on proper secrets handling for S3-compatible disks in Clickhouse configuration is a bit of a gap here:
https://github.com/Altinity/clickhouse-operator/blob/master/docs/security_hardening.md
Should probably explicitly cover from_env for that use-case.

yes, the best way is from_env + secret mounted in environment variable inside podTemplate.spec.containers
do you need a complete manifest?

I've already figured it out for myself, I was just thinking it was a missing scenario in the docs. I suspect a snippet w/ a small example would be sufficient to get others headed in the right direction.

@sporkmonger don't be shy to make pull request ;)

👋  I'm also looking into this and cannot find a first-class way to reference fromenv in ClickHouseInstallation
  configuration:
    clusters:
      - name: "simple"
    settings:
      storage_configuration/would_like_to_read_this_from_env: ???

Probably I've just missed docs somewhere, but I thought it was worth a mention!

@plantfansam you can use kubernetes secrets/configmap for it and customize podTemplate.spec to mount some files inside config.d
usually storage_confugration doesn't contains sensitive data
even when you use S3 disk
you can use IRSA metadata.annotations in spec.podTemplates[].spec.containers for it or provide environment variables with valueFrom + secretKeyRef inside spec.podTemplates[].spec.containers

Ah, that is super helpful @Slach, thank you!

@sporkmonger , agree we need docs on this. In fact, there are several ways to use S3 securely:

secret mounted via env variables, as discussed above.
service-account and IRSA
named collection defined in DDL -- this is more difficult to manage on infra side though

Secrets for configuration settings are supported in 0.23. See https://github.com/Altinity/clickhouse-operator/blob/0.23.0/docs/security_hardening.md#securing-clickhouse-server-settings


HPX 1.8.0 and Blaze issues

Expected Behavior
Blaze and HPX should work
Actual Behavior
We got many issues with deprecated namespaces for the executors and for_loop.
Steps to Reproduce the Problem
... Please be as specific as possible while describing how to reproduce your problem.

Build HPX 1.8.0
Build a application using blaze's hpx backend.

Specifications

HPX Version: 1.8.0 or master
Platform (compiler, OS): Not important

@diehlpk Could you confirm?

Yes, I made a pull request to Blaze a while ago.

I never tested it with 1.9.0, but the ticket can be closed.

@diehlpk 1.9.0 has not been published yet.


MODCLUSTER-754 Modify ResetRequestSourceImpl to send STOP request whe…

Issue: https://issues.redhat.com/browse/JBEAP-23679
Upstream issue: https://issues.redhat.com/browse/MODCLUSTER-754
Upstream PR: https://github.com/modcluster/mod_cluster/pull/602

@rhusar This is backport for https://github.com/modcluster/mod_cluster/pull/602.


glob("*") does not support matching non-utf8 filenames

 Issue by kballard
Wednesday Jan 29, 2014 at 23:12 GMT
For earlier discussion, see https://github.com/rust-lang/rust/issues/11916
This issue was labelled with: A-libs, A-unicode, E-easy, E-mentor in the Rust repository

glob::glob() does not have any support right now for matching non-utf8 filenames. Not only are its patterns restricted to strings, but it also explicitly skips any non-utf8 filenames it encounters (which should at least be able to match a * pattern).
Tasks that need to be done:

[ ] glob() needs to accept both strings and byte-vectors. It can do this using std::path::BytesContainer
[ ] glob() needs to process its pattern as a byte vector instead of a string, which will allow it to process filenames as byte vectors. This includes matching non-utf8 filenames against * and ? tokens (for the latter, matching a single byte is appropriate; ideally, it would match however many bytes are supposed to be consumed to create a U+FFFD REPLACEMENT CHARACTER as per the unicode standard)

This is a sub-task of #9639.

I've ran into this. I wanted to emulate globs on Windows, but it allows unpaired surrogates in filenames, and therefore filename patterns also need to support unpaired surrogates.
Looks like the original issue refers to an old Rust trait. I suppose post Rust-1.0 it's going to be glob<P: AsRef<OsStr>>(pattern: P).


Update maximum MTU for 10Gb NICs

10Gb switches use 9214 MTU on the interfaces vs 9000 for most 1Gb

cherry-picked in https://github.com/opnsense/core/commit/13f8ba23478e6b3f7cbfc17f594284f2bd3f1b81


Update netlink

update netlink version and remove out of tree modifications maintained in this repo.

@moshe010 PTAL

@moshe010 we could remove all things related to devlinkResource from mlxdevm-go now.
mlxdevm (gen)nl family does not support devlink resource only devlink family does. however that means we break API.

should we document on which kernel or OFED it should work? I remember it was required upstream fix...

should we document on which kernel or OFED it should work? I remember it was required upstream fix...

the upstream fix was eventually to the documentation :) saying that for some commands we may have empty msg.
see[1] so the fix was eventually in netlink lib[2] (the WA we are removing here essentially)
[1] https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next.git/commit/?id=8ad55b1e73c4e77656e2bea68fa2b484f33a6425
[2] https://github.com/vishvananda/netlink/pull/935


Failed to load config "airbnb-typescript" to extend from

Hey,
I tried to install and checkout, unfortunately it failed on command next lint
Error:
Failed to load config "airbnb-typescript" to extend from.
Referenced from<EMAIL_ADDRESS>error: script "lint" exited with code 1 (SIGHUP)

Would you be able to check if you've installed all required peer-dependencies? In this case, eslint-config-airbnb and eslint-config-airbnb-typescript?

I've updated the installation instructions to make the required peer-deps more clear.

sorry, I forgot to response! Thank you, now it's clear ❤


Suggestion: use metadata "kind" in query file instead of language_kind_map.lua

Thank you for creating such a great plugin!
I especially love the nvim-treesitter integration feature.
However, I have one issue. I use my own tree-sitter parsers and queries for some minor languages, and when I try to display its outline, I can't extend the outline just by editing the query file.
I read the source a little and found that it is because it uses language_kind_map.lua to determine the kind of items in the outline. Unless I edit the plugin itself, I cannot add outline kinds at will.
So I have a suggestion: why not specify the kind from the metadata of the query file with "#set!"?
Using (#set! "kind" "kind_name") syntax, the kind can be specified for each pattern, and in the future language_kind_map.lua will no longer be needed.
Currently I have edited the Lua and Rust query files to try it out. I'd appreciate your feedback.

Seems fine to me! I think that's a pretty reasonable refactor. I'm happy to merge this, or wait if you wanted to do a broader migration.

Thank you for the review!
I will do the same for the other languages. Please wait a moment.

I added "kind" metadata to the other query files, removed language_kind_map.lua, and confirmed that ./run_tests.sh prints "Success".
Please check it again!

Looks good to me! Thanks for the PR!


Make OWL model retrieved from Noctua more usable (handle the neo import problem)

This has come up in enough places it needs its own issue to refer to.  The problem is that go-cam OWL models include owl:imports http://purl.obolibrary.org/obo/go/extensions/go-lego.owl which includes https://github.com/geneontology/neo which is very large.  This makes it impossible to open the model in Protege and a hassle to deal with in code.  Since 99.9% of neo is irrelevant to any given go-cam model, lets find a way to reduce the import to something more manageable for downstream consumers.  (This may also impact developers working on the noctua/minerva stack but this issue is specifically in reference to consumers of the go-cams).
Ideas from @cmungall on https://github.com/geneontology/go-site/issues/558#issuecomment-377647594 :

We can provide a pre-generated combined OWL file with an import module merged in, that would allow for cross-model exploration.
For people who want to look at individual models in Protege, we can provide a docker container that makes a module and writes a catalog file. But really Protege needs better dynamic module creation abilities.

@goodb - is this issue now addressed with the more recent work on neo?
Thx.

@vanaukenk not quite.  Its related to what i was discussing with @balhoff at the end of the software call.  Hopefully close to closing.


Application lifecycle management: container start failed

Related Repo
https://github.com/open-cluster-management/multicloud-operators-subscription
What happened
After run make deploy-managed, the container start failed, got err msg as the following:
unknown flag: --cluster-namespace

After deleted - --cluster-namespace=managed_cluster_name # cluster1 in deploy/managed/operator.yaml,  the container start successfully, but then get error in log:
I0706 06:32:32.132637       1 manager.go:60] LeaderElection enabled as running in a cluster
I0706 06:32:32.222951       1 manager.go:118] Starting ... Registering Components for cluster: cluster1/cluster1
I0706 06:32:32.504304       1 namespace_subscriber.go:126] default namespace subscriber with id:cluster1/cluster1
I0706 06:32:32.505549       1 namespace_subscriber.go:129] Done setup namespace subscriber
I0706 06:32:32.509991       1 subscription.go:1013] No multiclusterHub resource found, err: the server could not find the requested resource
I0706 06:32:32.510045       1 controller.go:54] Add helmrelease controller when the remote subscription is NOT running on hub or standalone subscription
I0706 06:32:32.510384       1 helmrelease_controller.go:84] The MaxConcurrentReconciles is set to: 10
E0706 06:32:32.556900       1 placement.go:131] ACM Cluster API service NOT ready: no matches for kind "ManagedCluster" in version "cluster.open-cluster-management.io/v1"
I0706 06:32:32.586759       1 spoke_token_controller.go:78] Adding klusterlet token controller.
I0706 06:32:32.635489       1 lease_controller.go:72] trying to update lease "open-cluster-management-agent-addon"/"application-manager"
I0706 06:32:32.636434       1 manager.go:187] Starting the Cmd.
I0706 06:32:32.636588       1 leaderelection.go:243] attempting to acquire leader lease kube-system/multicloud-operators-remote-subscription-leader.open-cluster-management.io...
I0706 06:32:32.650651       1 lease_controller.go:113] addon lease "open-cluster-management-agent-addon"/"application-manager" updated
I0706 06:32:49.265782       1 leaderelection.go:253] successfully acquired lease kube-system/multicloud-operators-remote-subscription-leader.open-cluster-management.io
I0706 06:32:49.266868       1 sync_server.go:170] start synchronizer
I0706 06:32:49.309966       1 discovery.go:86] Synchronizer cache (re)started
I0706 06:32:49.310041       1 sync_server.go:190] stop synchronizer
E0706 06:32:49.310268       1 leaderelection.go:325] error retrieving resource lock kube-system/multicloud-operators-remote-subscription-leader.open-cluster-management.io: Get "https://<IP_ADDRESS>:443/apis/coordination.k8s.io/v1/namespaces/kube-system/leases/multicloud-operators-remote-subscription-leader.open-cluster-management.io": context canceled
I0706 06:32:49.310650       1 leaderelection.go:278] failed to renew lease kube-system/multicloud-operators-remote-subscription-leader.open-cluster-management.io: timed out waiting for the condition
I0706 06:32:49.310821       1 sync_server.go:231] stop synchronizer channel
E0706 06:32:49.310886       1 manager.go:191] controller was started more than once. This is likely to be caused by being added to a manager multiple timesManager exited non-zero

/assign @qiujian16

Should be fixed when https://github.com/open-cluster-management-io/multicloud-operators-channel/pull/9 and https://github.com/open-cluster-management-io/multicloud-operators-subscription/pull/15
are merged.

@xuezhaojun would  you  like to try again?

@xuezhaojun would you like to try again?

Sure， working on it now

@xuezhaojun would you like to try again?

It's OK with tutorial on the website now.


format wandb run name in pytorch checkpoint

Description & Motivation
It would be great to format the wandb run name:
wandb.run.name
into the checkpoint filename.
epoch: 10
loss: 0.5423
filename: "epoch{epoch}-loss{val_loss:.2f}" --> "epoch10-loss0.54"
filename: "{wandb.run.name}-{epoch}-{val_loss:.2f}" --> "swept_sky_54-epoch10-loss0.54"
I am not sure if this is already possible but I think the filename function in the ModelCallback can only access metrics so far.

@tobiascz What if you include the run name directly in the filename from the beginning?
logger = WandbLogger(...)
checkpoint = ModelCheckpoint(..., filename=f"{logger.experiment.name}-xyz.ckpt")

I am using Pytorch CLI:
  callbacks:     - class_path: lightning.pytorch.callbacks.ModelCheckpoint       init_args:         dirpath: "/home/Develop/project/checkpoints"         monitor: "val_loss"         save_top_k: 1         mode: "min"         filename: "{epoch}-{val_loss:.2f}-{wandb.run.name}"     #  wandb.run.name not accessible
this is my main:
`def cli_main():
cli = LightningCLI(model_class=MyModel, datamodule_class=DataModuleHTC, run=False, save_config_kwargs={"overwrite": True})
cli.trainer.fit(cli.model, cli.datamodule)

if name == "main":
cli_lightning_logo()
cli_main()`
here wandb has not been initilaized yet. I think the wandb is known once the first logging has been done.

Oh ok, this is interesting. I don't actually know how to do this or whether it is possible to do elegantly. Right now I think you would need to create the callback programmatically and call into wandb to get the run name in the main script.


CRS graph attrubute wrong format

Hi there is an issue with the type of crs properties at the intermodal graph. You are using class 'pyproj.crs.crs.CRS' which is not compatible with .graphml format. I suggest using int value (e.g. 4326 or 32636). If you get the crs value at your graph from GeoDataFrame using .crs --> it would be better to convert it to int value using .to_epsg(). The reason I am writing this is that I can't cache my graph to file to skip its repeated calculations when I reload my notebook.
This small change won't affect any part of the code since while creating a gdf from the graph you can use both options of crs system setting (pyproj class and int in recent versions of geopandas)
I think I might create a PR with the fix

there is the same error with geometries in edge attributes. It is better to apply .wkt to each geometry to make it string type.

There is also format problem with osmid in edge attributes. Somewhere it has list type, which is incompatible with .graphml format

@GeorgeKontsevik hello, the graph made with GraphGenerator is OSMNX compatible, so the CRS is similar to GeoDataFrame CRS. You can load and save an obtained graph the following way:
g = GraphGenerator.from_graphml('input.graphml')
...
GraphGenerator.to_graphml(g, 'output.graphml')


Supporting 3rd party file system like Amazon S3

Deploying Vapid on services like now.sh or Heroku leaves us with the issue of non-persistent file storage.
A file storage adapter system would be great, with like S3 adapter at first and let do the community work on other adapters?

I followed up on the forums about this, but for future reference, here's the response:

Re: storage, unfortunately, there isn’t anything implemented or in the works. IMO, this is one of the biggest pains using Heroku and other ephemeral hosts. I was hoping that one of them would eventually offer something akin to Amazon’s EFS, where the mounted file system is a virtual connection to S3.

That said, I’m open to pull requests or discussions about finding a solution. Or promoting a similar hosting service, if we can find one that’s no ephemeral, which maybe is DigitalOcean?


YARN-11395. RM UI, RMAttemptBlock can not render FINAL_SAVING

In the YARN-1345 remove of FINAL_SAVING was missed from RMAttemptBlock
Same issue was present after YARN-1345 in YARN-4411
YARN-4411 logic was applied in this commit for FINAL_SAVING

Description of PR
How was this patch tested?
For code changes:

[ ] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m 13s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
17m 52s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
29m 22s

trunk passed

+1 :green_heart:
compile
10m 27s

trunk passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
compile
8m 56s

trunk passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
checkstyle
1m 46s

trunk passed

+1 :green_heart:
mvnsite
2m  2s

trunk passed

-1 :x:
javadoc
0m 53s
/branch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdkUbuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04.txt
hadoop-yarn-server-resourcemanager in trunk failed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04.

+1 :green_heart:
javadoc
1m 42s

trunk passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
spotbugs
4m  5s

trunk passed

+1 :green_heart:
shadedclient
24m 12s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 23s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 30s

the patch passed

+1 :green_heart:
compile
9m 49s

the patch passed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04

+1 :green_heart:
javac
9m 49s

the patch passed

+1 :green_heart:
compile
8m 44s

the patch passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
javac
8m 44s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
1m 40s
/results-checkstyle-hadoop-yarn-project_hadoop-yarn.txt
hadoop-yarn-project/hadoop-yarn: The patch generated 2 new + 144 unchanged - 1 fixed = 146 total (was 145)

+1 :green_heart:
mvnsite
1m 55s

the patch passed

-1 :x:
javadoc
0m 50s
/patch-javadoc-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-jdkUbuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04.txt
hadoop-yarn-server-resourcemanager in the patch failed with JDK Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04.

+1 :green_heart:
javadoc
1m 37s

the patch passed with JDK Private Build-1.8.0_352-8u352-ga-1~20.04-b08

+1 :green_heart:
spotbugs
4m  8s

the patch passed

+1 :green_heart:
shadedclient
24m 41s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
5m 26s

hadoop-yarn-common in the patch passed.

+1 :green_heart:
unit
102m 38s

hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 50s

The patch does not generate ASF License warnings.

268m 56s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5222/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5222

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 943383f50ecf 4.15.0-200-generic #211-Ubuntu SMP Thu Nov 24 18:16:04 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 46ff6f9b8a2d249c16e669f332503cdc994dce46

Default Java
Private Build-1.8.0_352-8u352-ga-1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.17+8-post-Ubuntu-1ubuntu220.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_352-8u352-ga-1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5222/1/testReport/

Max. process+thread count
901 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5222/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

Merged to trunk, branch-3.3 and branch-3.2.


Please add an option to export conversion error (and other errors)

Please add an option to export conversion error (and other errors). I suggest an option to export to .csv & .ods
If the log is already saved somewhere please add a button to "open the log file"

We're planning a revamp of our "report" feature and will make sure to include your suggestions.
Thanks! Cc @clairekuang

We have added a copy to clipboard option, closing for now!


Add calculator for decoding a message in hamming code form L ot R

Is your feature request related to a problem? Please describe.

A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Describe the solution you'd like
I would like to work on this issue

Go ahead
Deadline 20 May,2021


Multi character emoji's with fitzpatrick modifier is parsed as multiple emojis

Passing an emoji like "👩🏽⚖" to EmojiParser.extractEmojis() will parse it as two emoji's  👩🏾  ⚖
It looks like EmojiParser.getNextUnicodeCandidate does not take the fitzpatrick into account when trying to find the longest match.

Faced a similar issue: emoji 👨🏾🦳 (Man: White Hair) is parsed incorrectly. Hair tone is not handled, at the same time skin tone is parsed correctly.


PXT titled "Recommended browser for linux" on Google search

Looks like the search crawlers are getting confused with the browser recommendation popup.

Peli fixed this. Fix needs changes to be merged in master. I will close this once that happens.


Driver generator

Add the docs for new drivers. Close #557

Thank you for your contribution!
Before submitting this PR, please make sure:

[ ] Your code builds clean without any errors or warnings
[x] You have made the needed changes to the docs
[x] You have wrote a description of what is the purpose of this pull request above

IssueHunt Summary

Referenced issues
This pull request has been submitted to:

#557: Add-Driver fails to run

Codecov Report

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

@@             Coverage Diff              @@
##            master     #585       +/-   ##
============================================
- Coverage   100.00%   11.26%   -88.74%
============================================
  Files           12      146      +134
  Lines          444     5042     +4598
  Branches       101     1179     +1078
============================================
+ Hits           444      568      +124
- Misses           0     4474     +4474

Impacted Files
Coverage Δ

packages/base-driver/src/lib/notification.ts
0.00% <0.00%> (ø)

packages/plugins/formatter/extension.ts
0.00% <0.00%> (ø)

packages/extension/index.ts
0.00% <0.00%> (ø)

...plugins/connection-manager/explorer/SidebarItem.ts
0.00% <0.00%> (ø)

...ges/plugins/connection-manager/ui/components/A.tsx
0.00% <0.00%> (ø)

packages/base-driver/src/lib/require.ts
0.00% <0.00%> (ø)

...ager/ui/screens/Results/components/Table/index.tsx
0.00% <0.00%> (ø)

packages/vscode/utils/index.ts
0.00% <0.00%> (ø)

...onnection-manager/ui/screens/Settings/lib/steps.ts
0.00% <0.00%> (ø)

packages/extension/language-client/index.ts
0.00% <0.00%> (ø)

... and 124 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 94d3241...9f9ba83. Read the comment docs.


[CLI] Install Completed with Failures; Won't Update

New 16.04 install. Install finished with the following message
>> Completed with Failures -- Success: 283, Failure: 181
How do I figure out what failures were?
Now running sudo sift update fails
><EMAIL_ADDRESS>> sift-version: notinstalled

SIFT not installed, unable to update
Error: SIFT not installed, unable to update
    at Object.<anonymous> (/snapshot/Users/ekristen/Development/sift/sift-cli/sift-cli.js:586:13)
    at Generator.next (<anonymous>)
    at Generator.tryCatcher (/snapshot/Users/ekristen/Development/sift/sift-cli/node_modules/bluebird/js/release/util.js:16:23)
    at PromiseSpawn.module.exports.PromiseSpawn._promiseFulfilled (/snapshot/Users/ekristen/Development/sift/sift-cli/node_modules/bluebird/js/release/generators.js:97:49)
    at Promise.module.exports.Promise._settlePromise (/snapshot/Users/ekristen/Development/sift/sift-cli/node_modules/bluebird/js/release/promise.js:574:26)
    at Promise.module.exports.Promise._settlePromise0 (/snapshot/Users/ekristen/Development/sift/sift-cli/node_modules/bluebird/js/release/promise.js:614:10)
    at Promise.module.exports.Promise._settlePromises (/snapshot/Users/ekristen/Development/sift/sift-cli/node_modules/bluebird/js/release/promise.js:693:18)
    at Async._drainQueue (/snapshot/Users/ekristen/Development/sift/sift-cli/node_modules/bluebird/js/release/async.js:133:16)
    at Async._drainQueues (/snapshot/Users/ekristen/Development/sift/sift-cli/node_modules/bluebird/js/release/async.js:143:10)
    at Immediate.__dirname.Async.drainQueues (/snapshot/Users/ekristen/Development/sift/sift-cli/node_modules/bluebird/js/release/async.js:17:14)
    at runCallback (timers.js:672:20)
    at tryOnImmediate (timers.js:645:5)
    at processImmediate [as _immediateCallback] (timers.js:617:5)

Try this for the time being.
apt-get remove python3-xlsxwriter
then
sudo sift install
Lets see where we get.
After that you can go into /var/cache/sift/<version> and grab all the log files and yaml files and send them to be for further troubleshooting.

Looking through the logs myself seems our web proxy is causing a lot of the failures. Thanks!


ADD community project pipeline

Description

Add community project about pipeline training and inference on website UI.

Fixes #721

Changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] Refactor (non-breaking change which refactors the code base)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Checklist

[x] My code follows the pre-commit style and check guidelines of this project.
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing tests pass locally with my changes

@vnk8071 Looks like the pre-commit checks have failed. Can you install pre-commit in your system as described here https://openvinotoolkit.github.io/anomalib/developer_guide/pre_commit_hooks.html. Then run pre-commit run --all and commit the modified files with the changes. Lemme know if something is not clear.

There seems to be some unintentional files committed to this PR. I can see some .py files in the notebooks folder.

@ashwinvaidya17 Thanks for your supported and I will close this PR.


TypeError: can only concatenate str (not "pandas._libs.tslibs.offsets.MonthEnd") to str

Hello,
I am trying to replicate the Automatic Hyperparameter Tuning using my dataset but encountered this error when using the predict method to forecast the next 12 months......

hey @beergil! The issue probably arises due to your ds column type. You could try Y_df['ds'] = pd.to_datetime(Y_df['ds']) to convert it to datetime type.

I have the same issue:

The solution from @FedericoGarza above worked.

Closing since casting to datetime seems to solve the issue. Feel free to reopen if you run into this issue again.


High-contrast mode: comments dropdown needs a border

This is a bug specific to Windows high-contrast mode. cc @kbayliss who did the testing for this.
Without the border, it’s hard to tell where the dropdown is:

The border could be added with a colour matching the background so it’s only visible in high-contrast mode.

@thibaudcolas  May I work on this issue?

Hello @thibaudcolas I'm new to open source. Can i work on this issue.

@Adyyousf thanks for offering. Please, do go ahead.
A general note for anyone who sees this - if you do not see a pull request referencing an open issue and the issue doesn't have a "Design decision needed", it means it is up for grabs and you can work on it. Ideally put a note on the open issue that you're working on it and will have a PR (shortly) in case others think about doing the same.

@Adyyousf that would be a pretty smashing first contribution! If you haven’t already, have a look at our Contributing to Wagtail guide first. For this specific issue, I expect the change is needed here:
https://github.com/wagtail/wagtail/blob/aff898ad8d12f9e9169e7364a776485c6ea56011/client/scss/components/_comments-notification-dropdown.scss#L31-L36
Fixing this might be as simple as adding a one-line:
border: 1px solid $color-text-base

This will need proper testing in High-contrast mode to confirm though.

I will start working on it and hopefully a PR will be up shortly

I have started working on this issue since no one has commented in a while. I will test and push a PR soon.


How to interact with checkbox in jest test?

I am testing with react-native-testing-library and would like to interact with a couple of checkboxes I have in one of my components.
Using fireEvent.press(getByA11yLabel('Drivers bag')); would throw error

No handler function found for event: "press"

I tried calling the onValueChange() like so getByA11yLabel('Drivers bag').props.onValueChange(); and that would throw error

TypeError: Cannot read property 'nativeEvent' of undefined

All I'm interested in testing is whether an effect would run when the user toggles the checkbox.

https://github.com/react-native-community/react-native-checkbox/blob/223e604f4fbe1abd57a476d743d294f6de82a38a/example/AppFunction.tsx#L25-L86
There are some examples to show how to change the value of checkbox outside. Please feel free to let us know if you have any trouble in the test of checkbox

do you still need a better implementation as a new Button just for testing? This is how I solved it:
it('performs onValueChange', () => {
  const func = jest.fn();
  const {getByTestId} = render(
    <CheckBox onValueChange={func} testID="myTest" />,
  );
  const checkBox = getByTestId('myTest');
  fireEvent(checkBox, 'onValueChange', {nativeEvent: {}});
  expect(func).toBeCalledTimes(1);
});

it('changes value', () => {
  let checked = false;
  const {getByTestId} = render(
    <CheckBox
      value={checked}
      onValueChange={() => (checked = !checked)}
      testID="myTest"
    />,
  );
  const checkBox = getByTestId('myTest');
  expect(checked).toBeFalsy();
  fireEvent(checkBox, 'onValueChange', {nativeEvent: {}});
  expect(checked).toBeTruthy();
});

The last one is not that much beauty, maybe someone can beautify it a little bit 👍

I went ahead with wrapping the label with a TouchableOpacity and testing the onPress callback on that.

Had a similar issue, ended up using this to toggle the checkbox:
fireEvent(getByRole('checkbox'), 'onValueChange', true)
Just wanted to add this in case it helps anybody else!

Had a similar issue, ended up using this to toggle the checkbox:
fireEvent(getByRole('checkbox'), 'onValueChange', true)
Just wanted to add this in case it helps anybody else!

Thanks!
I would like to add also that in my case, I needed to add an await
const checkbox = await findByRole('checkbox');
fireEvent(checkbox, 'onValueChange', true);

in this test how can i test the checked property on click: import {
render,
screen,
fireEvent,
cleanup,
waitFor,
} from "@testing-library/react";
import { axe } from "vitest-axe";
import { Checkbox } from "@components/Checkbox";
// clean up mocks after each test
afterEach(() => {
cleanup();
});
describe("Checkbox", () => {
it("should render the checkbox and label", () => {
const { getByRole, getByText } = render(
<Checkbox label="Test Checkbox" onChange={() => {}} />,
);
expect(getByRole("checkbox")).toBeInTheDocument();
expect(getByText("Test Checkbox")).toBeInTheDocument();
});
it("should call the onChange handler when clicked", async () => {
waitFor(() => {
const checkbox = screen.getByRole("checkbox");
fireEvent(checkbox, "onChange");
});
});
it("should show the check icon when checked", () => {
const { getByRole, container } = render(
,
);
fireEvent.click(getByRole("checkbox"));
expect(container.querySelector("svg")).toBeInTheDocument();
});
});


The predicted coordinates remain unchanged

Hi, thanks for your awesome work!
I have tried to reproduce your result and found the predicted tap coordinate of test dataset always been the same(as shown below).

The hyperparameter settings at runtime are as follows:
python main.py --data_root blip \
    --model FLAN-ALPACA-BASE \
    --epoch 10 --lr 1e-5 \
    --user_msg seq_all_hist1.0_hist4_future4_cuda1_evalstep2_imghist_db \
    --img_type blip --img_dim 1408 \
    --bs 8 --eval_bs 16 --input_len 512 --output_len 256 --eval_acc 40 \
    --transform_axis --warmup_ratio 0.05 \
    --all_data 1.0 \
    --use_history 4 \
    --use_future 4 \
    --eval_subset AITZ_processed/blip/all_blip \
    --output_dir experiments \
    --device cuda:0 \
    --seed 3407 \
    --use_img_history

I tried different hyperparameters, but the above problem still persists, just in different coordinates.
Do you know why this happens? Thanks!

Hi, I noticed that you passed the directory of ‘AITZ_processed’ dataset. Did you use the AITZ dataset instead of our processed AITW dataset?

Yes! I processed the AITZ dataset with reference to utis_data.py and fetch_features.py to ensure that the fields and meanings of the processed dataset are consistent with the fields and meanings of the dataset you provided. The data field of the processed data dictionary contains the following keys, which are finally stored according to the dataset structure mentioned in the readme:
keys = [
    "goal",
    "step_id",
    "image",
    "ui_positions",
    "ui_text",
    "ui_type",
    "result_touch_yx",
    "result_lift_yx",
    "result_action"
]

Could this problem be caused by the training data volume in the AITZ dataset being significantly smaller compared to the AITW training dataset?

Not sure if there is a mismatch between the coordinate style. I am not familiar with AITZ. Could you check the the specific transformed yx coordinates for AITZ? Maybe there is some problems during the transformation for the gold yx, which makes the model learn to predict everything to [0.2, 0.5].

 I followed your advice and checked all the target_touch and target_lift coordinates in the AITWDatasetImg instances, and I found that none of them are [0.2, 0.5], and the labels do not contain the tokenized [0.2, 0.5] (i.e., [18189, 6, 3, 12100]). BUT when I checked the labels passed to the model's forward function, I found that [18189, 6, 3, 12100] appears with a very high frequency, which I think is the source of the problem.
Thank you for your reminder.


More best practices?

This sample code could be even better, and enable more application insights features, if it used more examples of custom properties and custom metrics.  For example, the number of words found could be either another TrackMetric call itself, or could be sent in the customMetric parameter for the TrackEvent call.    I see that the TrackMetric call passes this.CommonProperties as the properties for the metric, but doesn't appear to pass those same properties for all the other TrackTrace/TrackEvent calls?
Likewise, If it isn't hooked up implicitly, using this.taskId as the telemetry client's operation.id field (and possibly the task's name as the operation.name field) would enable all of the telemetry in that one task to be correlated together in various places.

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

ID: a9e2f213-4ec9-2fd5-2847-c718e3190139
Version Independent ID: b46cb22d-c84d-606e-2b57-a80f417959d9
Content: Monitor Batch with Azure Application Insights
Content Source: articles/batch/monitor-application-insights.md
Service: batch
GitHub Login: @paselem
Microsoft Alias: danlep

@gardnerjr Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Closing doc issue and tracking as a code sample issue in Azure Batch Samples repo.
#please-close


Retrieve featuregate status in e2e tests

Read feature gate status instead of indirect checking to
determine whether a feature is enabled.
Signed-off-by: Weiqiang Tang<EMAIL_ADDRESS>

Codecov Report

Merging #1387 into master will decrease coverage by 19.45%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master    #1387       +/-   ##
===========================================
- Coverage   64.32%   44.87%   -19.46%
===========================================
  Files         159       74       -85
  Lines       12674     5585     -7089
===========================================
- Hits         8152     2506     -5646
+ Misses       3668     2801      -867
+ Partials      854      278      -576

Flag
Coverage Δ

#integration-tests
44.87% <100.00%> (-0.03%)
:arrow_down:

#kind-e2e-tests
?

#unit-tests
?

Flags with carried forward coverage won't be shown. Click here to find out more.


http.get throwing an exception on the iOS simulator

I made a simple demonstration app that does a http.get which throws an exceptions. The stocks example throws the same error.
Steps to Reproduce
Click button which invokes
var response = await http.get(_url)
Flutter Doctor
[✓] Flutter (on Mac OS, channel alpha)
• Flutter at /Users/toney/flutter
• Framework revision 5460e94488 (7 days ago), engine revision 76d5cf230a
[x] Android toolchain - develop for Android devices
x Android Studio / Android SDK not found. Download from https://developer.android.com/sdk/
(or visit https://flutter.io/setup/#android-setup for detailed instructions).
[✓] iOS toolchain - develop for iOS devices (Xcode 7.3.1)
• XCode at /Applications/Xcode.app/Contents/Developer
• Xcode 7.3.1, Build version 7D1012
[✓] Atom - a lightweight development environment for Flutter
• Atom installed; Flutter plugin version 0.2.2
Logs and Crash Reports
May  9 11:07:36 --- last message repeated 1 time ---
══╡ EXCEPTION CAUGHT BY NETWORKING HTTP LIBRARY ╞═══════════════════════════════════════════════════
The following NoSuchMethodError was thrown while sending bytes to the Mojo network library:
The null object does not have a getter 'iterator'.
NoSuchMethodError: method not found: 'iterator'
Receiver: null
Arguments: []
When the exception was thrown, this was the stack:
#0      Object._noSuchMethod (dart:core-patch/object_patch.dart:42)
#1      Object.noSuchMethod (dart:core-patch/object_patch.dart:45)
#2      MojoClient._send.<_send_async_body> (package:flutter/src/http/mojo_client.dart:183:39)
#3      _RootZone.runUnary (dart:async/zone.dart:1137)
#4      _Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:553)
#5      _Future._propagateToListeners (dart:async/future_impl.dart:639)
#6      _Future._completeWithValue (dart:async/future_impl.dart:426)
#7      _Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:481)
#8      _microtaskLoop (dart:async/schedule_microtask.dart:41)
#9      _startMicrotaskLoop (dart:async/schedule_microtask.dart:50)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

For more information about diagnosing and reporting Flutter bugs, please see https://flutter.io/bug-reports/.

Should also mention the same http.get works on the android simulator.

FYI @chinmaygarde @tvolkert

Note: this is on the alpha branch (7 days ago) in case that matters.  This file has been changed once since that time,here is the current head version of that line:
https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/http/mojo_client.dart#L183
I'm surprised this is a funny async stack.  Isn't there a stacktrace library to help make stacks across async boundaries like this readable?  Maybe we're not using it?  @devoncarew
Also a little surprised we don't have any flutter drive test which covers use of the http library in the simulator, definitely should fix that either with this change or shortly after.

I'm  guessing that the code that threw is running on the device, and that we're not using the stack_trace package there. I think it has performance implications? Or just if you have very very long chained async calls? @nex3 would know.
Here's where we use it in flutter_tools: https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/executable.dart#L80. If it does have perf implications, we could just enable it on the runtime when in --debug mode.

The thrown code is running in --debug mode on iOS simulator.

Filed https://github.com/flutter/flutter/issues/3810 about using package:stacktrace in flutter in --debug mode.

Fixed on master

Verifying ns_net in engine next.


Flickering during drag

I am trying to implement your great library in my project.
The problem is:
Sometimes if I drag an element, the preview free space is flickering between two states. It seems your directive is not sure, where to show the drop preview.
Unfortunately I can't reproduce this issue with your examples.
The examples always work fine.
Do you have a idea, what the problem might be? Could something in the CSS cause the problem?

@a5hik Great project, @JohannesAtGit is correct in a sense with the flickering. This seems to only occur when each sortables items are not the exact height. I encountered this issue when moving  a div with a height of 100pixels whilst the others are 500+ . Let me know your thoughts.


3 levels of nesting

Hi,
I have the following example:
Resources:
class Campaign extends RailsResource
    @configure({
     ...
      serializer: railsSerializer(() ->
        this.resource('emails')
        this.nestedAttribute('emails')
      )
    })

 class Email extends RailsResource
    @configure({
    ...
      serializer: railsSerializer(() ->
        this.resource('emailTemplate')
        this.nestedAttribute('emailTemplate')
      )
    ...
    })

class EmailTemplate extends RailsResource
    @configure({
     ...
    })

Controller:
$scope.campaign = new Campaign(
  emails: [
    new Email(
      emailTemplate: new EmailTemplate(
        html: 'some html...'
      )
    )
  ]
)
$scope.campaign.save().then((response) ->
  console.log response
)

When I get the request params on the backend they have the following structure:
{
  "campaign" => {
    "emails_attributes" => [
      "email_template" => {
        "html" => "some html .."
      }
    ]
  }
}

while I expect
{
  "campaign" => {
    "emails_attributes" => [
      "email_template_attributes" => {
        "html" => "some html .."
      }
    ]
  }
}

The email_template key should be called email_template_attributes otherwise the accepts_nested_attributes_for in the Rails models won't work.
Is there something I am missing or the 3 level resource nesting is not supported?

This seems like a bug. I don't see anything obviously wrong with your example.

On Feb 16, 2016, at 3:23 AM, Vladimir Aleksiev<EMAIL_ADDRESS>wrote:
Hi,
I have the following example:
Resources:
class Campaign extends RailsResource
@configure({
...
serializer: railsSerializer(() ->
this.resource('emails')
this.nestedAttribute('emails')
)
})
class Email extends RailsResource
@configure({
...
serializer: railsSerializer(() ->
this.resource('emailTemplate')
this.nestedAttribute('emailTemplate')
)
...
})
class EmailTemplate extends RailsResource
@configure({
...
})
Controller:
$scope.campaign = new Campaign(
emails: [
new Email(
emailTemplate: new EmailTemplate(
html: 'some html...'
)
)
]
)
$scope.campaign.save().then((response) ->
console.log response
)
When I get the request params on the backend they have the following structure:
{
"campaign" => {
"emails_attributes" => [
"email_template" => {
"html" => "some html .."
}
]
}
}
while I expect
{
"campaign" => {
"emails_attributes" => [
"email_template_attributes" => {
"html" => "some html .."
}
]
}
}
The email_template key should be called email_template_attributes otherwise the accepts_nested_attributes_for in the Rails models won't work.
Is there something I am missing or the 3 level resource nesting is not supported?
—
Reply to this email directly or view it on GitHub.

I figured out what was wrong with your example.  The call this.resource('emailTemplate') should be this.resource('emailTemplate', 'EmailTemplate').


Add Istio Ambient Mode Example and Update to 1.22

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

What is the outcome that you are trying to reach?
The outcome I'm trying to achieve is to enhance the repository by adding an example demonstrating how to provision an EKS cluster with Istio in Ambient mode. Additionally, updating the version to 1.22 is necessary.
Describe the solution you would like
I would like to propose adding documentation and Terraform scripts for deploying an EKS cluster with Istio in Ambient mode. This includes detailed instructions for installing Istio in Ambient mode.
Describe alternatives you have considered
An alternative approach could be to include comments within the main.tf file explaining the ambient settings and how users can customize them according to their requirements.
Additional context
The addition of this example will enhance the repository's documentation and provide users with a practical guide for deploying Istio in Ambient mode on Amazon EKS. Additionally, updating the version to 1.22 ensures that users have access to the latest features and improvements in Istio.

I would like to propose adding documentation and Terraform scripts for deploying an EKS cluster with Istio in Ambient mode. This includes detailed instructions for installing Istio in Ambient mode.

What is the motivation for this though, why is it useful - what scenarios should users consider this pattern, what problems does it solve, whats the overall value proposition?

@bryantbiggs Thank you for your comment and interest in the proposed addition.
Ambient mode in Istio offers a resource-optimized strategy tailored for efficient utilization within Amazon EKS environments. By documenting and providing Terraform scripts for this deployment mode, users can benefit from:

Resource Efficiency: Ambient mode optimizes resource utilization, making it suitable for scenarios where resource constraints are a concern.
Scalability and Flexibility: Users can easily scale their Istio deployments in Amazon EKS while maintaining efficient resource usage.
Enhanced Performance: Optimized resource utilization can lead to improved performance and cost savings in Amazon EKS environments.

Overall, Ambient mode introduces a highly efficient and optimized approach to deploying Istio on Amazon EKS. By providing documentation and Terraform scripts for Istio in Ambient mode, users gain access to a powerful deployment strategy that maximizes resource utilization, scalability, and performance. This addition empowers users to efficiently manage their Amazon EKS clusters with Istio, ensuring smooth and seamless deployments while minimizing resource overhead. We appreciate your feedback and welcome any further discussion or suggestions regarding this proposal.
We appreciate your feedback and welcome any further discussion or suggestions regarding this proposal.

resolved with https://github.com/aws-samples/istio-on-eks/pull/75