Add testing for Xamarin

Based on recent reports things are generally working with improvements made to Mono and Xamarin in recent versions. We however don't have any automated testing and adding it would likely require different infrastructure from what we use for testing this for other .NET platforms.

Should we make this a more general cross-platform testing automation bug or file a new one for UWP so it can be prioritized separately?

@bricelam My intention is to track them as separate issues, precisely to prioritize them separately. I haven't created the UWP issue yet, mostly because UWP cannot be tested in 2.0 yet.

I've been mucking around with EF Core 2.0.1 and 2.0.2 with Xamarin.iOS lately, trying to get a meaningful cross-section of EF Core unit tests copied into and passing in a mobile app deployed to a real iOS device, i.e. AOT compiled. I'm doing this to get more confidence in EF Core for Xamarin mobile app development. (EF Core is working in samples I've tried, but they're relatively shallow.)
This remains a work in progress, but I thought I'd remark here about one specific class of issue I needed to work around, so far: there are handfuls of tests that use the Moq framework. Moq depends on generating code at runtime with Reflection.Emit, which isn't permitted for AOT compiled builds. I imagine this would also be an issue for UWP.
Fortunately, I noticed issue https://github.com/aspnet/EntityFrameworkCore/issues/7627 and it looks like 2.1.0's tests won't depend on Moq. 👍

@cwrea It's great to hear that you are doing this testing--really interested to see what you find. And, yes, Moq is gone in the current codebase.
@divega We should look into whether we can leverage the dotnet ci bot to run our tests on Xamarin. It's already running our PRs, so it seems not unreasonable to go the next step.

In late March, I did get mobile test-runner apps working with a meaningful subset of the EF Core unit tests. As you can see from the issue references just above, I'd already filed a couple of issues for Xamarin.Android and Xamarin.iOS — issues that prevented certain unit tests from running.
I put the rest on the back burner for a little while to wait for VS 15.7 to be released. I'd tried the previews and I knew 15.7 would include Xamarin updates that would address some of the test failures reported on Xamarin.iOS.
Since VS 15.7 was released earlier this week, I just published my results from using the official release. Please find them at Entity Framework Core 2.0.2 Unit Test Subset Results on Xamarin.Android and Xamarin.iOS.
The repository also contains the entire Visual Studio solution for the mobile test runner apps, including the subset of Entity Framework Core 2.0.2 unit test classes that I judged as meaningful to test on mobile.
The EF Core unit tests subset does not include tests that relate to benchmarks, design-time, and SQL Server. Also, tests that used Moq were commented out, and some tests that required SQL Server as accessory to test non-SQL Server functionality were adapted for Sqlite.
The results from both the Xamarin.Android and Xamarin.iOS test runner mobile apps (based on xUnit.net Runners for Devices) were my primary concern, but the solution also includes console-based projects for running the same tests on full .NET Framework and .NET Core. I added these console projects to sanity-check that the test class subset plus manual fixups did not break tests that already pass on those platforms.
There is also a UWP test runner app project. I did get the UWP runner working initially with default Debug project build settings — and all tests from the subset were passing for UWP x86 and x64 on those settings.
However, UWP builds with the all-important option "Compile with .NET Native tool chain" enabled would not compile, not on 15.6.x before, and not on 15.7 now:  error : ILT0005: '[...]\nutc_driver.exe [...]"' returned exit code 1.
This surprised me, as I expected if Xamarin.iOS could get an AOT-based build compiled, then so should the .NET Native tool chain for UWP. Plus, UWP is mentioned here as working with EF Core 2.0.1. I didn't venture any further down the UWP rabbit hole, but my guess is that if others have successfully been able to compile UWP native apps that use Entity Framework Core, then something else about the test runner app (size or complexity, perhaps) is causing the .NET Native tool chain to choke.

p.s. I'll add that I do intend to repeat this exercise for EF Core 2.1 when released, to check for regressions. At the same time, with the issues already identified on Android, iOS, and UWP being Xamarin/Mono framework or UWP tooling issues, I don't expect EF Core 2.1 would improve on current results on those platforms. Platform fixes are necessary, and I still have problems to isolate for filing a couple of new issues in the respective places.

Today I attempted to update my Xamarin-based test runners to use the EF Core 2.1 release.
I encountered a major issue. An iOS app that references EF Core 2.1 won't even build. I filed the following issue for Xamarin.iOS: #4168 - iOS app build fails when referencing package (e.g. Entity Framework Core 2.1) that depends on newer System.Memory.

@cwrea Thanks for trying this. We appreciate what you are doing a lot. I think you hit the same issue reported at https://github.com/aspnet/EntityFrameworkCore/issues/12081. It was suggested to copying System.Memory.dll as workaround, but we haven't had a chance to verify.

@divega You're welcome.
I actually just filed a similar issue for Xamarin.Android just a couple of moments ago: #1769 - Android app build fails when referencing package (e.g. Entity Framework Core 2.1) that depends on newer System.Memory. I only noticed your comment about #12081 after opening the Xamarin.Android issue — but I think my issue is in the right place. Includes a sample that reproduces the issue, and downgrading EF Core 2.1 to 2.0.3 gets the sample building and working.

My 2 cents: For the past couple of months I've been building an app which utilizes a common data services library (NetStandard v2.0) using EFCore.Sqlite. For me, cross platform mobile app development using c# with Xamarin.Android, Xamarin.iOS and NetStandard v2.0 libraries is a HUGE advantage. I feel that having an EFCore testing suite for Xamarin is a necessary step forward for EF Core ( and MS as the lead in cross platform development ).

More generally...
... I know that getting a fully automated EF Core test suite running on iOS and Android devices remains an open issue and presents a handful of challenges as originally noted above.
But maybe now is a good time for the Xamarin and Entity Framework teams at Microsoft to begin co-ordinating pre-release testing of EF Core on for iOS and Android? Start small, if necessary? Build a sufficiently non-trivial sample app on devices once in a while, to ensure no obvious regression before a release?
Consider:

Each of EF Core and Xamarin are high-profile strategic technology offerings from Microsoft.

Each is sufficiently mature and being used by Microsoft customers to solve real-world problems.

And yet, EF Core + Xamarin are not sufficiently mature __when used together__. (sorry!)

That remains a strategic gap. One of the early goals I remember Rowan Miller mentioning about EF7 way back in 2014 that got me interested, and paying attention ever since, was that not only would EF vNext work for cross-platform desktop and server OSs like macOS and Linux, but that EF vNext could/would, eventually, also work on mobile devices.
I assume Windows Phone was the likely intended benefactor of a mobile-capable version of EF as envisioned in 2014, but once Microsoft pivoted from competing as mobile OS vendor to providing awesome development tools for iOS and Android, I expected EF Core would eventually include Xamarin.iOS and Xamarin.Android as first class platforms.
That these two strategic Microsoft development technologies do now sorta, kinda, sometimes work together is nice, but it would be even better if they were intentionally made to work well together.
So now that desktop and server OSs have been given a highly-performant, mature, and feature-packed release of EF Core in v2.1, could mobile platforms please be shown love in a coming-soon release?
🙏 🙏

@cwrea, @ToddThomson thanks for your great feedback!

Build a sufficiently non-trivial sample app on devices once in a while, to ensure no obvious regression before a release?

We are discussing this with the Xamarin team. It seems they already regularly build certain samples and customer apps as a way to guard against regressions. It would be great if you can point us to good candidates. @cwrea I know from before that you have built some samples using EF Core with Xamarin together. I would like to know if you think they are sufficient or at least a good starting point. Also, if you can point us to the latest code, that would be great.

@smitpatel see https://github.com/aspnet/EntityFrameworkCore/issues/8792#issuecomment-393621259, for the iOS repro of the issue with System.Memory. We don't have a separate issue on the EF Core side tracking this.

@divega Worth investigating is xunit.runner.devices referenced on the xamarin app (iOS,Android). I have zero experience with this (xunit.runner.devices). However, I modified my EFCore Sqlite 2.1 test applet and was able to build and run the common netstandard 2 lib with EFCore Sqlite 2.1 and XUnit along with the mono android app with the xunit runner (2.4 beta nuget package).
I'll poke about to see what can be accomplished...

@divega I just ran your public class DbSetAsTableNameSqliteTest : DbSetAsTableNameTest test on my android phone. I just copied a couple of your EF Core test files into my net standard 2 lib with EFCore Sqllite.
I'd say that this approach may be a starting point to fit your needs (I will definitely be using it).

@divega Thanks! Here's a link to the Xamarin Todo sample that I ported from another ORM popular with Xamarin apps (SQLite.Net-PCL) to run on EF Core instead, some time ago: https://github.com/cwrea/XamarinTodo/tree/NETStd20_EFCore203. That's currently the latest, using EF Core 2.0.3. I haven't updated for EF Core 2.1 yet, mainly because the iOS build will fail due to open issue https://github.com/xamarin/xamarin-macios/issues/4168.
Also of interest— but not in the category of a simple sample—in my May 10 comment on this issue I mentioned and linked to another solution I hacked together more recently. The following repository contains both Xamarin.Android and Xamarin.iOS test runner apps. They exercise a meaningful subset of EF Core unit tests: https://github.com/cwrea/EFCoreCPTestSoln/tree/EFCore2.0.2_VS15.7. The test runner apps use the same xunit.runner.devices package that @ToddThomson just mentioned.
That test runners solution also isn't updated for EF Core 2.1 yet—or, rather, I haven't published my work-in-progress— also due to the same iOS build failure issue. But locally I do at least have a working version for Xamarin.Android and have been discovering a handful of new issues for Xamarin.Android that new tests in EF Core 2.1 have uncovered. Tomorrow, I could probably share a half-baked update to the solution that at least compiles and runs the EF Core 2.1 subset on Xamarin.Android. This solution I hacked together is far from perfect but might be a good starting point.

@divega Re:

There was another issue reported with System.Memory, but with different symptoms: https://github.com/aspnet/EntityFrameworkCore/issues/12081. It was suggested to copying System.Memory.dll as workaround, but we haven't had a chance to verify.

FWIW, for Xamarin.iOS a number of potential workarounds have been attempted e.g. https://github.com/xamarin/xamarin-macios/issues/4168#issuecomment-395767156 but none have succeeded so far.
I did find a related set of commits at https://github.com/xamarin/xamarin-macios/pull/4147 which pre-dated the issue by a couple of days, however it isn't clear to me when those changes will make it into a preview release and whether they would definitively address the System.Memory issue.
Even if the removal of the broken Xamarin.iOS facade assembly permits the netstandard2.0 version of System.Memory to be used by the build, I suspect that the AOT assertion failure seen as result of some of the promising workarounds may end up as the next hurdle.

Action item for @divega and @ajcvickers: review sample applications from the perspective of EF Core usage. Once we have something we will contact the Xamarin team for inclusion in regression protection tests.

@divega @ajcvickers You may want to look at NUnitLite in the xamarin.android repo.

Note from triage: focus has shifted to MAUI.


Sliding window implementation for training

I haven't found sliding window implementation for training as described in the Google paper (fig.3). Only first and last 180 frames were taken from each segment. Why don't you implement sliding window? Could you explain the reason?

A sliding window is not used during training for the paper I implemented.

I am sorry, if we speak about different papers. But according to the paper "Generalized end-to-end loss for  speaker verification" fig.3  batch construction for training is shown.

The images does not refer to the inference sliding window, but rather picking a random number of frames between 140 and 180 for each batch during training. And yes, I did not implement this, I may do it in the future.


Added Running section in README and RootChain migration file

Full list of changes:

Added 2_deploy_rootchain.js file, as the first migration file wasn't actually deploying the RootChain contract and it wasn't available in truffle console
Added a Running section which explains the rationale for the point above
Rewritten Migrations's contructor to get rid of the warning, as per Solidity ^0.4.22
Fixed some minor spelling errors in docs/rootchainFunctions.md

Pull Request Test Coverage Report for Build 339

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.701%

Totals

Change from base Build 325:
0.0%

Covered Lines:
166

Relevant Lines:
184

💛  - Coveralls


remove pCars2 'will start raining' prediction

Using the cloudBrightness to guess when it'll start raining was always a punt. It just doesn't work - remove it

made this default to disabled, and made the logic less likely to fire


How can i disable Analytics (just in local env) ? then enable it when I public my website on a real host

Simply, within the App\Http\Controllers\Admin\DashboardController.php, edit constructor.
For instance, change it to something like below:
public function __construct()
{
    if (App::isLocal()) {
        return "Disabled";
    } else {
        $this->period = 30;
        $this->limit = 16;
        $this->end = Carbon::today();
        $this->start = Carbon::today()->subDays($this->period);
        $this->country = env('ANALYTICS_COUNTRY');
        parent::__construct();
    }
}

It's not working !  This is my code
if (app('env') == "local") {             return "Disabled";         } else{             $this->period = 30;             $this->limit = 16;             $this->end = Carbon::today();             $this->start = Carbon::today()->subDays($this->period);             $this->country = env('ANALYTICS_COUNTRY');             parent::__construct();   }
Then i got this error
Google_Auth_Exception in OAuth2.php line 364: Error refreshing the OAuth2 token, message: '{ "error" : "invalid_client", "error_description" : "The OAuth client was not found." }'

Thank you !

I tried the solution it says
FatalErrorException in DashboardController.php line 49:
Class 'App\Http\Controllers\Admin\App' not found

@dezudas You need to import App class after declaring the namespace.
namespace App\Http\Controllers\Admin;

use App;
use App\Base\Controllers\AdminController;
use Carbon\Carbon;
use Illuminate\Support\Collection;
use LaravelAnalytics;


Pull to refresh #10 #63

Not the best implementation, but it works i guess?
Scroll up 3 times in a row to refresh the page (Edit: Forgot to mention, you have to be on top of the page and then scroll 3 times in a row)

Seems like a reasonable solution to me for a desktop app. It's not a feature you often see on desktop apps.

Nice! What about some visual feedback? If you check if the user is 0 from the top, and scroll x up then add class to animate hidden element, like https://github.com/apeatling/web-pull-to-refresh

http://i.imgur.com/Lr7OZMj.gif
When you scroll up 2 times it shows it and if you scroll one more time it reloads the page

Nice – I'll merge this for the next release

I'm closing this for now sorry. Too much in the codebase have changed since


error when using tkmatrix for star with radius > 1 solar radius

Hi!
Sorry to bother you again, but I am wondering if tkmatrix is compatible with stars greater than 1 solar radius. I was hoping to use this to do transit injection and recovery for planets around red clump stars, but I get the error: ValueError: radius_1 argument out of range . I looked into the lc.py file in ellc, and it looks like it raises an error any time the star has a radius greater than 1.
Have you tried using tkmatrix for stars with radii greater than 1? If not, I would suggest adding this to the readme!
Thanks so much.

Hi!
Thank you again for such a quick response. I think that the period does seem to be the issue, and it looks like I picked a weird system to start with. I am looking at TIC 15932677, a red clump star with a potential planet transiting every 1.4 days. So I let my period range go down to 1 day and masked the planet candidate (file attached). However, if this star actually has the radius predicted by lightkurve (10.8 Rsun) , this system does not even seem possible. I am attaching the .yaml file and traceback but I think you're right and I should just pick another star and pick a different minimum period. Not sure what is going on with this system.
Thanks again for the help
nohup.out.txt
TIC15932677.yaml.txt

Thank you so much, this is very helpful!

Thank you so much for all of your help! I was able to run it successfully on TIC 15932677 and a couple other red clump stars. If there is anything I can do to help going forward, please reach out!


Add readme for sites

Reference documentation for sites using wrangler:

[ ] double check the output of wrangler init and publish

i have a start of an edit of this that i'm doing for our QA sessions: https://gist.github.com/ashleygwilliams/e8ab664e4a2a84b84fab2edebea07426
i started using this but found some things confusing. let me know what ya'll think!

i have a start of an edit of this that i'm doing for our QA sessions: https://gist.github.com/ashleygwilliams/e8ab664e4a2a84b84fab2edebea07426
i started using this but found some things confusing. let me know what ya'll think!

I like yours a lot better as a tutorial (i.e. getting started with sites having never used wrangler). I think the readme that lives here (docs/content/sites.md) should be reference material more than a tutorial though. The tutorial version repeats a lot of the information (e.g. setting up personal domain v. workers_dev) in wrangler's root readme.
I suggest we:

move your tutorial to the docs as a boilerplate with this tutorial
clean up the sites.md I created to be pure reference (i.e. just defining the new commands and config) and link to that tutorial

Any thoughts on that as a solution? @ashleygwilliams

Going to reopen one that just points to https://sites.workers-tooling.cf/workers/sites


557 domain function mapping files

Summary
Functions have been added to derive the SIC - Domain mapping functions on sharepoint
Functions to produce the output for selective editing have been produced.
Minor changes to selective editing contributing method to join threshold values
Checklists

This pull request meets the following requirements:

[ ] installable with all dependencies recorded
[x] runs without error
[x] follows PEP8 and project specific conventions
[ ] appropriate use of comments, for example no descriptive comments
[ ] functions documented using Numpy style docstings
[ ] assumptions and decisions log considered and updated if appropriate
[ ] unit tests have been updated to cover essential functionality for a reasonable range of inputs and conditions
[ ] other forms of testing such as end-to-end and user-interface testing have been considered and updated as required
[ ] tests suite passes (locally as a minimum)
[ ] peer reviewed with review recorded

If you feel some of these conditions do not apply for this pull request, please
add a comment to explain why.

Made the changes you suggested @AntonZogk. I will move the script to create the outputs to the testing branch so they can be produced easily next week following 0.0.3 release. I'll create a branch for that now. Ready for this to be reviewed again :)

Forgot to push before re requesting my bad!


Command fails if no editors are open

Invoking generate application stack report when no editors are open fails with the following error.
I used the command palette (F1) to invoke the command.
Running the contributed command:'extension.fabric8AnalyticsWidget' failed.

This PR should fix it https://github.com/fabric8-analytics/fabric8-analytics-vscode-extension/pull/129/files


Add method to support dependent generators in forAll

This isn't meant to be committed as-is, it's a work in progress but wanted to get some feedback on it, especially if you think it's a good idea.
While writing https://github.com/ncredinburgh/QuickTheories/pull/33, I wondered why the ListDSLTest hard-coded the numbers 2 and 1 for the length of the lists rather than being generated too. I suspect the answer is at least partially that there isn't an elegant way to generate a value, generate a dependent value, and then use both in the test.
This code adds an equivalent to asWithPrecursor() with a Gen-producing function, as flatMap() is to map(). What do you think of the idea of this? Also, "withPrecursorGen" could do with a better name.
If this seems useful, I can add it to the other TheoryBuilders too. In the size-between tests generating (min, extra) and adding them in both the list generator and the check isn't as nice as it could be. I should be able to make it so you can chain withPrecursorGen, so it could be something like:
.forAll(integers().between(0, 10))
.withPrecursorGen(min -> integers().between(0, 10).map(n -> n + min))
.withPrecursorGen((min, max) -> lists().of(integers().all()).ofSizeBetween(min, max))
.check((min, extra, l) -> l.size() >= min && l.size() <= max);

Thanks for the PR, writing the response below was quite helpful in thinking about what direction to take things in.
The reason the list dsl doesn't accept a Gen of Integers for sizes is history and oversight - it should. Generators didn't use to compose together nicely. Now that they do the api ought to accept them whenever possible. The maps dsl was fixed to do this when it was merged in.
I agree that withPrecursorGen (with a name that was somehow better) would be useful in some circumstances, but I think I'd prefer not to add it as it adds complexity.
The first 0.1x cut of quicktheories had a lot of very complex code spread around the place - it was big relief to get rid of a lot of it in 0.2 and I want to avoid getting back to the situation again.
So, unless there's a compelling case I'd prefer not to expand the api in the theory builders (which requires changes in 4 or more places for consistency, and could require more if the airity is ever increased). In contrast adding flatMap to Gen required 1 line of code.
So I think where we are is we have an easily accessible api for basic operations available via the theorybuilders. This is going to stay fairly static unless there's a really strong use case.
If you want to do something more complex you need to work in terms of Gens. This makes sense as that complex work can then be easily re-used.

Thanks, that makes sense.


Resize issue

I got this error:
Traceback (most recent call last):
File "F:\EchoMimic\venv\lib\site-packages\gradio\queueing.py", line 624, in process_events
response = await route_utils.call_process_api(
File "F:\EchoMimic\venv\lib\site-packages\gradio\route_utils.py", line 323, in call_process_api
output = await app.get_blocks().process_api(
File "F:\EchoMimic\venv\lib\site-packages\gradio\blocks.py", line 2018, in process_api
result = await self.call_function(
File "F:\EchoMimic\venv\lib\site-packages\gradio\blocks.py", line 1567, in call_function
prediction = await anyio.to_thread.run_sync(  # type: ignore
File "F:\EchoMimic\venv\lib\site-packages\anyio\to_thread.py", line 56, in run_sync
return await get_async_backend().run_sync_in_worker_thread(
File "F:\EchoMimic\venv\lib\site-packages\anyio_backends_asyncio.py", line 2441, in run_sync_in_worker_thread
return await future
File "F:\EchoMimic\venv\lib\site-packages\anyio_backends_asyncio.py", line 943, in run
result = context.run(func, *args)
File "F:\EchoMimic\venv\lib\site-packages\gradio\utils.py", line 846, in wrapper
response = f(*args, **kwargs)
File "F:\EchoMimic\webgui.py", line 233, in generate_video
final_output_path = process_video(
File "F:\EchoMimic\webgui.py", line 169, in process_video
face_img = cv2.resize(face_img, (width, height))
cv2.error: OpenCV(4.10.0) :-1: error: (-5:Bad argument) in function 'resize'

Overload resolution failed:

src is not a numerical tuple
Expected Ptrcv::UMat for argument 'src'

Seems to be an issue with resizing. but it shouldnt be. any help suggestion?

Try with command line first.

Try with command line first.

i dont know how to do that and i dont want to run the interface like that either. the gradio is working fine. just this error relating to resizing

okay so it seems that echomimic only supports square aspect ratios, so images need to be cut at 512 x 512 or some other square resolution. which is strange because it seems to have a image size slider. i wonder what that is for.

Yes 512 x 512. just ignore the slider.

I get the same error, also with square images. no matter if I use my own images or the official test images.
In console it works great. Any idea how to solve the issue?

I found the solution.
Open webgui.py and change the part from line 162 to this:
# Crop and pad face_img
face_img = crop_and_pad(face_img, crop_rect)
if isinstance(face_img, tuple):
    face_img = face_img[0]  # Keep only the image part if crop_and_pad returns a tuple

# Crop and pad face_mask
face_mask = crop_and_pad(face_mask, crop_rect)
if isinstance(face_mask, tuple):
    face_mask = face_mask[0]  # Keep only the mask part if crop_and_pad returns a tuple

The reason it didn’t work was that crop_and_pad returned a tuple instead of a numpy.ndarray, causing an AttributeError when accessing .shape. Selecting only the first element resolved the issue.


[terra-list] defect fixes

Summary

Updated styles to remove markers  and fix dividerstyles for list items

Closes #
Deployment Link

https://terra-core-deployed-pr-#.herokuapp.com/
Testing

Additional Details

Thank you for contributing to Terra.
@cerner/terra

Can we WDIO test updated for list with dividers, section, section-header ??

Can we WDIO test updated for list with dividers, section, section-header ??

Added in https://github.com/cerner/terra-core/pull/3749/commits/17a1f59ee7172a9ed77445627718d8f6d9de7e4d

@sugan2416  @supreethmr
do you know when this will get released?


[Question] - Keep the connection stable and avoid client restart

Hi @antoniomika
I noticed, looking at the logs of the ssh client connected to the sish server, that there are frequent disconnections and restarts (the client is managed with autossh or systemd service).
The sish server is deployed on an AWS machine and my clients are also almost all on AWS so I don't think that the disconnections are caused by connection problems (in general the EC2 AWS have stable connections)
When the client logs out they show the message:
Connection to sish.mydomain.link closed by remote host.
Connection to sish.mydomain.link closed.

What can I do to avoid these frequent disconnections?
Should I use, client-side, ssh with any of the following parameters? (If so, what values do you recommend?)
-o ServerAliveInterval=??
-o ServerAliveCountMax=??
-o TCPKeepAlive=??

The parameters I use on the server side are these:
--idle-connection=true \
--idle-connection-timeout=5s \
--cleanup-unauthed=true \
--cleanup-unauthed-timeout=5s \
--cleanup-unbound=true \
--cleanup-unbound-timeout=5s \
--ping-client=true \
--ping-client-interval=5s \
--ping-client-timeout=5s \

Do you have any recommendations or tips for keeping the connection alive and avoiding frequent client restarts?
Are the values set on the sish server adequate to keep the connection stable and avoid restarts?
Thank You.
Fabio

Hello @manprint ,
The connection being closed by remote host makes me think that the connection is either unbound (no tunnel) or unauthed (just staying at password auth). The ServerAliveInterval should be at least the ping-client-interval (usually a bit more for buffer).
Could you share the full sish command you are using as well as the full autossh command? It could have something to do with the tunnels being used.
Also, any logs from sish (and exit codes, just in case sish is the thing that is crashing) would be helpful as well!

Hi, Thanks for your quick response.
This is my typical command when running sish with autossh:
autossh -M 0 -p 2222 -R subdomain:80:localhost:5000 sish.mydomain.it

I checked the sish logs and there are no crashes.

Hrm yeah, there's nothing complicated going on there then. I would try the ping-interval and timeout modifications.
The other thing it could be is something with the autossh monitoring forward acting weird? Though not sure why that would be.

Ok, let's try changing the timeout and ping-interval settings.
Do you have any values to suggest?

Hi @antoniomika
I'm testing with the following parameters:
--ping-client=true \
--ping-client-interval=30s \
--ping-client-timeout=30s \

The autossh command I'm using is the following:
autossh -M 0 -o "ServerAliveInterval 35" -o "ServerAliveCountMax 2" -p 2222 -R mysubdomain:80:localhost:5000 sish.mydomain.it

What do you think about it?
Suggestions?

Ok, thanks.
I'll leave it like this and check.
I'll let you know soon.
Fabio

Hi @antoniomika
I tested with the settings I wrote above.
The connection is much more stable.
In three days there was only one disconnection.
Thanks as always for the support.
Fabio

Awesome! I'm gonna close this for now! Again, if you're happy with sish and the work I'm doing, please consider sponsoring the project!
Best,
Antonio


Research: generating and storing keys

Research how to generate and store keys on embedded device with Rust.

Initial research finished. Additional research will go in parallel with making proof-of-concept.


Add query string support to List function

As a User
I need a way to query promotions by a name, value, type or active.
So that  I can retrieve a list of promotions as per the query
Acceptance Criteria
Given query string support to List functionality is setup
When a query string is provided
Then I should get the list of promotions as per the attributes specified in the query string (for example: name, type,...,value, active)

Update: Have implemented the query string capability for the List functionality.
For the remaining part:
Implement an Action endpoint route for your service -> Looking into an end point that can be added for our model.


Fixed some typos and cleaned up language in getting_started.ipynb.

Hi @davipatti, thanks for the contribution! :D
Just let me know if you're planning to add something else. Otherwise I'm going to merge.

This was it!
bambi is great - thanks for all the work!


wrong line and column numbers reported by longReporter

Expected behavior:
The reported line and column numbers should be the equal, regardless of whether longReporter, defaultReporter or tsc is used
Actual behavior:
with the longReporter, the numbers are smaller by one
gms@sirius:~/work/HOT/sandbox/build-gulp (master)$ tsc
src/app/index.ts(2,5): error TS2322: Type 'undefined' is not assignable to type 'number'.
gms@sirius:~/work/HOT/sandbox/build-gulp (master)$

gms@sirius:~/work/HOT/sandbox/build-gulp (master)$ gulp ts:tsc
[19:08:23] Using gulpfile ~/work/HOT/sandbox/build-gulp/gulpfile.js
[19:08:23] Starting 'ts:tsc'...
using defaultReporter:
src/app/index.ts(2,5): error TS2322: Type 'undefined' is not assignable to type 'number'.
[19:08:24] TypeScript: 1 semantic error
[19:08:24] TypeScript: emit succeeded (with errors)
[19:08:24] Finished 'ts:tsc' after 830 ms

gms@sirius:~/work/HOT/sandbox/build-gulp (master)$ gulp ts:tsc:long
[19:08:41] Using gulpfile ~/work/HOT/sandbox/build-gulp/gulpfile.js
[19:08:41] Starting 'ts:tsc:long'...
using longReporter:
[gulp-typescript] /home/gms/work/HOT/sandbox/build-gulp/src/app/index.ts(1,4): error TS2322 Type 'undefined' is not assignable to type 'number'.
[19:08:42] TypeScript: 1 semantic error
[19:08:42] TypeScript: emit succeeded (with errors)
[19:08:42] Finished 'ts:tsc:long' after 861 ms

+1

@gms1 can you please indicate why this issue was closed, would be nice to know if it's solved or not reproducible or whatever

I'm not sure ... because I have not used this project for a long time.
If not, after such a long time, it would be better to file a bug report regarding the current version, to gain more attention .. and a PR would not be a big deal either :-) and would probably be welcome


Add TNGF

Revise from previous PR comment
Add ID
Add UserLocationInfo
Change all indents to tab
Format file under ngapType & ngapConvert

Already fix errors caused by dependencies, @ianchen0119 please review it.
Tks !


SOLR-14680: Remove SimpleMap (affects NamedList)

and remove the rest of SolrJ org.apache.solr.cluster.api
NamedList:

new getOrDefault
remove forEachEntry, forEachKey, abortableForEachKey, abortableForEach
remove asMap, get(key, default)

https://issues.apache.org/jira/browse/SOLR-14680

This was a pain because SimpleMap was a little more entrenched.  It deserves a CHANGES.txt calling out briefly that NamedList is impacted, since this is so omnipresent in Solr.

I don't think I have the knowledge to weigh in here, but I did like that this removes 10x more lines than it adds!

If I should break this up, or file another JIRA, let me know.  The SimpleMap part feels kind of separate, especially since it impacts the ubiquitous NamedList.

I plan to merge this next week with the following CHANGES.txt under 9.8
CHANGES.txt, 9.8:
OTHER
* SOLR-14680: NamedList, deprecating methods: forEachEntry, forEachKey, abortableForEachKey, abortableForEach,
 asMap (no-arg only), get(key, default).  Added getOrDefault.  Deprecated the SimpleMap interface as well as the,
 entirety of the SolrJ package org.apache.solr.cluster.api, which wasn't used except for SimpleMap.  (David Smiley)

Don't plan to say anything in 10 since removing deprecated code doesn't deserve a mention, generally.
I'll do the 9x deprecations and addition of getOrDefault in another commit on branch_9x.


Library cannot be tested on standard npm runkit

@ljharb nice! Unfortunately I haven't been able to test https://npmjs.com/string.prototype.replaceall in runkit: https://npm.runkit.com/string.prototype.replaceall. It's getting Error: Cannot find module 'define-properties' error.

Turns out this is a missing top-level dep; it works locally because npm hoists it to the top.

I'll publish a patch shortly.

v1.0.2 is released; runkit works now.


Fix export restore PVC owner reference

Signed-off-by: Alex Kalenyuk<EMAIL_ADDRESS>
What this PR does / why we need it:
Currently this is
ownerReferences:
  - apiVersion: v1alpha1

But we want
ownerReferences:
  - apiVersion: export.kubevirt.io/v1alpha1

This gets validated with https://kubernetes.io/docs/reference/access-authn-authz/admission-controllers/#ownerreferencespermissionenforcement
which is enabled in OpenShift, but disabled in our kubevirtci clusters.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/cc @awels

Let wait on this until #8257 is merged.

/approve

/hold

/hold cancel

/retest


violation emails/GCS uploads now use the CSV format

Fixes issue #1048, the changes are fully tested; emails and GCS uploads work correctly.
Please ignore any commits prior to Mar 21, 2018 -- these have been merged to 2.0-dev already (with PR #1260). The first relevant commit is 7d8bcdf030430ee1eca5a369fe3925cea0f1123c.
In any case, please take a look and let me know what you think.

Henry, I had to rebase the branch in question and open a new PR (PR #1288) -- the latter addresses all your review comments and also fixes the email_violations notifier that was leaking temp files indeed.


keras Model.compile with loss/metrics dict and multiple outputs from same layer

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): ubuntu 16.04
TensorFlow installed from (source or binary): binary
Python version: ('v1.12.0-7354-gedbd8a15b9', '1.13.0-dev20190203') - also present in 2.0 preview
CUDA/cuDNN version: 10.0 / 7
GPU model and memory: quadro k620

Current Behaviour
Keras' model.compile with dict losses matches provided loss functions with outputs passed into the constructor via each output's layer name. This is not unique in the case where multiple model outputs come from the same layer.
Expected Behaviour
I would expect usages of lists/dicts to be consistent between losses, metrics, labels and outputs - i.e. if any are lists, they must all be lists of the same length, and if any are dicts they must all be dicts with the same set of keys. Requiring output layers to be named consistently with the key in the corresponding labels dictionary seems completely bamboozling to me, e.g. if building a hybrid model with a classification output and 'class_index' key in the labels one would have to name the final layer 'class_index' even if it outputs logits/softmax activations.
This is likely a breaking change. At the very least I would expect an error to be raised if output_names are not unique and users pass in dictionaries for either loss or metrics in Model.fit.
Code to Reproduce
import tensorflow as tf
# network that maps 1 input to 2 separate outputs
x = tf.keras.layers.Input(shape=(1,), dtype=tf.float32)
x2 = tf.keras.layers.Dense(2)(x)
y, z = tf.keras.layers.Lambda(
    lambda x2: tf.unstack(x2, axis=-1), name='x2_unstack')(x2)
# y = tf.keras.layers.Lambda(tf.identity, name='y')(y)
# z = tf.keras.layers.Lambda(tf.identity, name='z')(z)  # current work-around
model = tf.keras.models.Model(inputs=x, outputs=dict(y=y, z=z))

print(model.outputs)
# [
#  <tf.Tensor 'x2_unstack/unstack:0' shape=(None,) dtype=float32>,
# <tf.Tensor 'x2_unstack/unstack:1' shape=(None,) dtype=float32>
# ]
# somewhat unexpected as not the same as the value passed to constructor, but ok...
print(model.output_names)
# ['x2_unstack', 'x2_unstack']
# surely that's going to cause issues later...

model.compile('SGD', loss={'x': 'mse', 'y': 'logcosh'})
# ValueError: Unknown entry in loss dictionary: y.
# Only expected following keys: ['x2_unstack', 'x2_unstack']

model.fit(x=x_data, y=dict(y=y_data, z=z_data))  # expected way of using fit

Is this still an issue with latest version TF 2.4? Thanks!

Looks all good now.


#1284 Added SpEL support to @SchedulerLock name attribute

This PR adds SpEL support to @SchedulerLock "name" attribute (#1284)
I think it does not add much complexity to ShedLock.
What do you think @lukas-krecan?

By the way, the SpEL parser uses a TemplateParserContext with the default prefix #{ and suffix } in order to mix literal text with one or more SpEL evaluation blocks.
More info: https://docs.spring.io/spring-framework/docs/current/reference/html/core.html#expressions-templating

Looks good, thanks


quick search footer and tweaks

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] I have tested this PR in all supported browsers.
[ ] I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #2688

PR #2689   👈

This comment was auto-generated by Graphite.


Ensure Global Accelerator accelerator has flow logs enabled

Possible check to target the following resource

[ ] aws_globalaccelerator_accelerator

closing, tracking in spreadsheet

closing, tracking in spreadsheet


[Falcosidekick] Additional annotations + fix aad-pod-identity namespace

Signed-off-by: Annegies van 't Zand<EMAIL_ADDRESS>What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release
Any specific area of the project related to this PR?
/area falcosidekick-chart

What this PR does / why we need it:
Allow additional annotations on falcosidekick
Pretty sure there was a mix up with the eventhub.namespace value. This isn't a Kubernetes namespace, but an Event Hubs namespace (which contains the event hub instances).
Which issue(s) this PR fixes:
None
Fixes #
Special notes for your reviewer:
Checklist

[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido after the PR has been reviewed.
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @leodido

LGTM label has been added.  Git tree hash: a895bf37b2dc05f7a63433ffecde6c37e5b506f9

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Views and Twitter Blue verification appears on the right tweet sidebar when viewing media

Reproduced on version 2.22.1 on both Chrome Desktop and Firefox with "hide views / analytics under tweets". I assume this hasn't been fixed at HEAD.

"Twitter Blue verified checkmarks" also do not appear on the right tweet sidebar:

The view is an easy CSS fix, the other one not so much - the desktop media modals contain their own timeline which we're not handling, and when you switch to them the  doesn't change, so we need to:

Watch for a popup appearing and the URL changing to a photo URL
Start watching the media modal timeline (most of the current timeline watching stuff should work, probably need to be able to pass a different selector in), which will handle replacing the Twitter Blue checks
Bonus fun - if there's media in a reply the modal switches over to that, at which point the timeline gets replaced, so we need to watch the timeline container for the timeline being replaced
While the modal is open, hovering over stuff or anything which pops up a menu will continue to trigger popup detection, so we need to avoid having the media popup functionality above running in duplicate
Since the same media modal element stays the whole time, we should be able to use the existing functionality for doing something when a popup closes


(PUP-6739) Certain apps shouldn't fail if env is not local

Prior to this commit, we had a number of users running into an issue
where if they set the environment in either the toplevel or [main] of
puppet.conf, and that environment did not have a corresponding directory
in the local file system, commands like puppet config and puppet help would bail out. This commit adds in the ability to specify
environment requirements on the application level. Commands like puppet config and puppet help shouldn't require the environment to exist
locally.
This issue mainly surfaces because the concept of environment is used in
multiple contexts. The use case this change is meant to address is when
a user has set environment to refer to an environment that only exists
on the master. Under this scenario, puppet agent can check into the
master to request that specified environment. Although this change does
allow the user to continue to abuse how and where environment is set,
this is a common enough behavior we should have this buffer. This is
also useful if the user has accidentally mistyped their environment. If
this is the case, the user can still use puppet config set to update
that setting to the correct one.
This does not impact applications that should depend on the environment
existing locally, such as puppet apply and puppet module.

[root@b0q1ro8iuh21hk0 ~]# puppet config print environment
production
[root@b0q1ro8iuh21hk0 ~]# cat /etc/puppetlabs/puppet/puppet.conf
# This file can be used to override the default puppet settings.
# See the following links for more details on what settings are available:
# - https://docs.puppetlabs.com/puppet/latest/reference/config_important_settings.html
# - https://docs.puppetlabs.com/puppet/latest/reference/config_about_settings.html
# - https://docs.puppetlabs.com/puppet/latest/reference/config_file_main.html
# - https://docs.puppetlabs.com/puppet/latest/reference/configuration.html
[root@b0q1ro8iuh21hk0 ~]# puppet config set environment foo
[root@b0q1ro8iuh21hk0 ~]# cat /etc/puppetlabs/puppet/puppet.conf
environment = foo
# This file can be used to override the default puppet settings.
# See the following links for more details on what settings are available:
# - https://docs.puppetlabs.com/puppet/latest/reference/config_important_settings.html
# - https://docs.puppetlabs.com/puppet/latest/reference/config_about_settings.html
# - https://docs.puppetlabs.com/puppet/latest/reference/config_file_main.html
# - https://docs.puppetlabs.com/puppet/latest/reference/configuration.html
[root@b0q1ro8iuh21hk0 ~]# puppet config print environment
/opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/environments.rb:38:in `get!': Could not find a directory environment named 'foo' anywhere in the path: /etc/puppetlabs/code/environments. Does the directory
exist? (Puppet::Environments::EnvironmentNotFound)
        from /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/application_support.rb:29:in `push_application_context'
        from /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/application.rb:346:in `run'
        from /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/util/command_line.rb:132:in `run'
        from /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/util/command_line.rb:72:in `execute'
        from /opt/puppetlabs/puppet/bin/puppet:5:in `<main>'
[root@b0q1ro8iuh21hk0 ~]# vim /etc/puppetlabs/puppet/puppet.conf
[root@b0q1ro8iuh21hk0 ~]# cat /etc/puppetlabs/puppet/puppet.conf
[agent]
environment = foo
# This file can be used to override the default puppet settings.
# See the following links for more details on what settings are available:
# - https://docs.puppetlabs.com/puppet/latest/reference/config_important_settings.html
# - https://docs.puppetlabs.com/puppet/latest/reference/config_about_settings.html
# - https://docs.puppetlabs.com/puppet/latest/reference/config_file_main.html
# - https://docs.puppetlabs.com/puppet/latest/reference/configuration.html
[root@b0q1ro8iuh21hk0 ~]# puppet config print environment
production
[root@b0q1ro8iuh21hk0 ~]# vim /etc/puppetlabs/puppet/puppet.conf
[root@b0q1ro8iuh21hk0 ~]# cat /etc/puppetlabs/puppet/puppet.conf
environment = foo
# This file can be used to override the default puppet settings.
# See the following links for more details on what settings are available:
# - https://docs.puppetlabs.com/puppet/latest/reference/config_important_settings.html
# - https://docs.puppetlabs.com/puppet/latest/reference/config_about_settings.html
# - https://docs.puppetlabs.com/puppet/latest/reference/config_file_main.html
# - https://docs.puppetlabs.com/puppet/latest/reference/configuration.html
[root@b0q1ro8iuh21hk0 ~]# puppet config print environment
/opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/environments.rb:38:in `get!': Could not find a directory environment named 'foo' anywhere in the path: /etc/puppetlabs/code/environments. Does the directory exist? (Puppet::Environments::EnvironmentNotFound)
        from /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/application_support.rb:29:in `push_application_context'
        from /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/application.rb:346:in `run'
        from /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/util/command_line.rb:132:in `run'
        from /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/puppet/util/command_line.rb:72:in `execute'
        from /opt/puppetlabs/puppet/bin/puppet:5:in `<main>'
[root@b0q1ro8iuh21hk0 ~]# cd /opt/puppetlabs/puppet/lib/ruby/vendor_ruby/
[root@b0q1ro8iuh21hk0 vendor_ruby]# patch -p1 < ~/0001-PUP-6739-Fix-environment-mode.patch
patching file puppet.rb
patching file puppet/application.rb
patching file puppet/application/config.rb
patching file puppet/application/facts.rb
patching file puppet/application/help.rb
patching file puppet/application/resource.rb
patching file puppet/application_support.rb
patching file puppet/settings.rb
patching file puppet/util/environment_mode.rb
[root@b0q1ro8iuh21hk0 vendor_ruby]# cat /etc/puppetlabs/puppet/puppet.conf
environment = foo
# This file can be used to override the default puppet settings.
# See the following links for more details on what settings are available:
# - https://docs.puppetlabs.com/puppet/latest/reference/config_important_settings.html
# - https://docs.puppetlabs.com/puppet/latest/reference/config_about_settings.html
# - https://docs.puppetlabs.com/puppet/latest/reference/config_file_main.html
# - https://docs.puppetlabs.com/puppet/latest/reference/configuration.html
[root@b0q1ro8iuh21hk0 vendor_ruby]# puppet config print environment
foo

I'm not entirely convinced this is the best way to solve this problem. It seems quite heavy weight for what we're trying to do.
That, and there are a handful of comments pertaining to run_mode, which is what this work is based on, that aren't hugely inspiring.
115     # This sucks (the existence of this method); there are a lot of places in our code that branch based the value of
116     # "run mode", but there used to be some really confusing code paths that made it almost impossible to determine
117     # when during the lifecycle of a puppet application run the value would be set properly.  A lot of the lifecycle
118     # stuff has been cleaned up now, but it still seems frightening that we rely so heavily on this value.
119     #
120     # I'd like to see about getting rid of the concept of "run_mode" entirely, but there are just too many places in
121     # the code that call this method at the moment... so I've settled for isolating it inside of the Settings class
122     # (rather than using a global variable, as we did previously...).  Would be good to revisit this at some point.
123     #
124     # --cprice 2012-03-16

Why do you believe this is an equivalent pattern to run_mode? Other than that they both use an Application class method to set a property on the application.

CLA signed by all contributors.

Alright, next steps for this PR ideally would be to add in a warning if the user is trying to set the environment globally using puppet config set. I can either do that here, or in a seperate PR.

Maybe treat that as a separate PR.

If this is strictly a bug fix, I think it'd be good to target 4.10.x, but if it's too risky, then 5.3.x.

Also, I've updated this PR to target the 4.10.x branch

Could you add a test to spec/unit/application/config_spec.rb that the application's environment_mode is :not_required?
Also it would be good to have a unit test for the application_support change for the case where if Puppet[:environment] refers to a non-existent directory and the mode is :not_required then Puppet.lookup(:environments).get(Puppet[:environment]) should return a non-nil environment. But if mode is :local, then we would expect an exception to be raised, e.g. when running puppet apply

@joshcooper updated


wrapperscript cudaaligner problems

Hi,
I have problems with the racon wrapper script and using racon NOT on a gpu.
The error is: racon: unrecognized option '--cudaaligner-batches'
The racon version is v1.4.13
racon_wrapper --split 2500 -t 16 -u test.reads.fastq test.ovl.paf test.fasta > wrapper_test.fasta
[RaconWrapper::run] preparing data with rampler
[RaconWrapper::run] total number of splits: 3841
[RaconWrapper::run] processing data with racon
/path/racon: unrecognized option '--cudaaligner-batches'

Thanks,
Fabian

Hello Fabian,
this issue has been already fixed in commit https://github.com/lbcb-sci/racon/commit/31a3c92878d19a9ef31462193e6a7862e6fe1611 which is version 1.4.16. I did not publish a release yet.
Best regards,
Robert

Hi Robert,
Thanks a lot, then I will update it via git.
Cheers,
Fabian


Can LlamaGen predict a [EOS] token when inferencing?

No， we donot need to

If I simply add a [eos] token after tokenizing a image for training, and when inferencing, if [eos] is predicted, the model then stops generating. Will it work? What's your thought on this, thx!

emm.. It will work but it will be just useless since the model will do the exact step during inference since the image has a fix latent size.

But it will be usefull if you want to training on different spatial ratio image and add the information as the start token

Hi~
Adding a special token to enable different aspect ratio image generation is very promising. I will try this idea if possible in the future. Thx !

Yes, And my another suggest based on my training in many t2i models, add cross attention instead of add the token to the front will produce more promising result.


Netlify Functions build fix for #60

Did a bit more digging, looks like all I was missing was the addition of a included_files option for the functions to be able to "see" the full data/members folder since it's a dynamic input instead of a static one.
This time, I set up a Netlify account and tested it with a few options first before this one ended up working. So confidence is much higher now! 😅

Let's give it a go!
And thank you again for taking the time to do this, as well as work through the bug fixing iterations. I really appreciate it 🙂


name resolvers

[x] combine original icns/stars PRs into new PR/branch https://github.com/cosmology-tech/cosmos-kit/pull/126
[ ] move all into 2 packages name-services and name-services-hooks
[x] remove swr and just use a simple caching mechanism w/o new dependencies
[x] create a NameService class

let resolver = new NameService({
   defaultService: defaultSvc, // 'icns' | 'stargaze'
   services: [
      {
         name: 'icns',
         contract: 'osmo1xk0s8xgktn9x5vwcgtjdxqzadg88fgn33p8u9cnpdxwemvxscvast52cdd',
         chainName: 'osmosis',
         getQueryMsg: (address) => {
            return {
               icns_names: { address }
            }
         },
         slip173: 'osmo'
      },
      {
         name: 'stargaze',
         contract: 'stars1fx74nkqkw2748av8j7ew7r3xt9cgjqduwn8m0ur5lhe49uhlsasszc5fhr',
         chainName: 'stargaze',
         getQueryMsg: (address) => {
            return {
               name: { address },
            }
         },
         slip173: 'stars'
      }
   ]
})

Then imagining an instance could so something like:
resolver('dan.stars'); // => stars1lkjsadfljsdflkjsdflkjsdf

// this uses the stargaze resolver to get an osmosis address (but this requires changes to contracts)
resolver('dan.stars', 'osmosis');  // => osmo1alskjflkjsdflkjsdfkjsdf

No new packages, they are integrated in core and react


proof of humanity profile in the href on pages/Cases/CaseDetails/Overview.tsx

where do you  want to put the URL on the .env file or just use the "const"


Menu of upcoming & recent classes/events

Description
A menu of the upcoming &/or past classes/events. Where the user can click on the hyperlink and get a short blurb on the class instead of a link taking you to the meet up page.
Ask
I would like the user to be able to click on the hyperlink next to the upcoming &/or past classes to get a quick blurb of the description of the class. Not a link taking the user to Meet up.
To Do

[ ] Create a drop down box w/ hyperlinks
[ ] Create a menu on the side of the chapter pages

Additional Info
This would just be a quick place for a user to see what the chapter has been up to without leaving the GDI website.

@staceyturnage I am not 100% sure it's the best use of the page to add past classes. Meetup keeps an archive on hand and would like to not re-create too much content on the site.
Also, I am not sure that we need to add any extra verbiage, as we do post a rather large chunk of text on the page as is.
@kstack7 thoughts?

There is actually a great discussion on this over in #233 so these two can combine into one discussion as we move forward. The gist of it being that it'd be good to show "sold out" courses when possible but still three open courses without overwhelming the space.

@kstack7 cool! Did you want to close this issue and add to the discussion on the other issue or... ? I feel like this issue has a few more nuances, but will leave it up to you on you'd like to move forward.

@staceyturnage let's move this over to #233 since there is more traction over there. It'd be great to keep all our input on one issue so we can cover more in one merge :)


Firefox 73: WebVR requires a secure context

https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/73:

The deprecated WebVR API---which has been supplanted by WebXR, which supports both augmented and virtual reality applications---now requires a secure context using the HTTPS protocol to operate. This is due to the availability of sensitive input sources that may include private information (bug 1381645).

Would be probably more accurate to add an additional note to Firefox 73+ giving the added information, but since its deprecated anyway and support is dwindling…

Yeah, seems fine. Thanks!


Update minor version to 2 (0.2.0)

The version number 0.0.x has some issue with the SemVer.

Thanks for making this PR


virtualbox: add patch to enable building with linux kernel 5.11

Motivation for this change
Without this change, virtualbox's kernel module does not build on linux kernel 5.11.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[NA] macOS
[NA] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[NA] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[NA] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 118740 at abcb1938 run on x86_64-linux 1

4 packages marked as broken and skipped:

linuxPackages-libre.virtualbox
linuxPackages_hardkernel_4_14.virtualbox
linuxPackages_hardkernel_latest.virtualbox
linuxPackages_latest-libre.virtualbox

17 packages built successfully:

linuxPackages.virtualbox (linuxPackages_5_10.virtualbox)
linuxPackages_4_14.virtualbox
linuxPackages_4_19.virtualbox
linuxPackages_4_4.virtualbox
linuxPackages_4_9.virtualbox
linuxPackages_5_11.virtualbox (linuxPackages_latest.virtualbox)
linuxPackages_5_4.virtualbox
linuxPackages_hardened.virtualbox
linuxPackages_latest_hardened.virtualbox
linuxPackages_latest_xen_dom0.virtualbox
linuxPackages_latest_xen_dom0_hardened.virtualbox
linuxPackages_lqx.virtualbox
linuxPackages_testing_bcachefs.virtualbox
linuxPackages_xanmod.virtualbox
linuxPackages_xen_dom0.virtualbox
linuxPackages_xen_dom0_hardened.virtualbox
linuxPackages_zen.virtualbox

1 suggestion:

warning: unclear-gpl
gpl2 is a deprecated license, please check if project uses gpl2Plus or gpl2Only and change meta.license accordingly.
Near pkgs/applications/virtualization/virtualbox/default.nix:232:5:
    |
232 |     license = licenses.gpl2;
    |     ^

This is obsolete due to https://github.com/NixOS/nixpkgs/pull/123156


Update alter table table_constraint

Added ON DELETE CASCADE of edges. This is a new feature in CTP2.3 of SQL Server 2019 release.

@shkale-msft : Thanks for your contribution! The author, @VanMSFT, has been notified to review your proposed change.

Hi, I am Shreya Verma, PM with the SQL Server team and the graph feature. Made a quick edit in the documentation for a new feature that we released with CTP2.3

#sign-off

@shkale-msft: I'm sorry - only the author of this article, @VanMSFT, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

Hi @shkale-msft - We'll merge this PR. Please note that Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.
After you review staging and fix all validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.  Read about authors working in the public repo for more information.
If you need help getting set up to work in the private repo, see Set up Git repository locally for documentation.
Thank you


RxJava2 ?

Hi @yudhir would support in next release.

Can this be used alongside RxJava2 until then?

@yudhir Sure.

@jaychang0917 how to see logs from FbLoginHiddenActivity.java?  and test with rxjava?


Bulk download bug

I cannot bulk download tickers with . in the name. Steps to reproduce the problem are below:
import yfinance as yf
Running:
yf.Ticker('STX40.JO').history(period="1mo", interval='1d')
Works fine, whereas running:
yf.Tickers(['IEV', 'SPY', 'STX40.JO']).history(period="1mo", interval='1d')
Results in:
AttributeError: 'Tickers' object has no attribute 'STX40.JO'
I have stepped through the code with the debugger and the data is being successfully downloaded and parsed, but I suspect the issue is due to python attributes not being able to have . in their names. I did try using 'STX40-JO' for the ticker name, but this results in the error STX40-JO: No data found, symbol may be delisted
The same is true for exchange rates, for example the ticker 'USDEUR=X' has the same behaviour as above.

Cool


revert proxy statement cache & synchronized register method.

Pull Request Test Coverage Report for Build 7026

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.0008%) to 71.134%

Files with Coverage Reduction
New Missed Lines
%

sharding-proxy/sharding-proxy-frontend/sharding-proxy-frontend-mysql/src/main/java/org/apache/shardingsphere/shardingproxy/frontend/mysql/command/query/binary/close/MySQLComStmtCloseExecutor.java
1
0.0%

Totals

Change from base Build 7024:
0.0008%

Covered Lines:
11299

Relevant Lines:
15884

💛  - Coveralls


Tabs overflowing the segment they are attached to.

I'm adding dynamic created tabs to my porject, and when the number of tabs is big enough, the tabs headers overflow the segment they are attached to, and go offsrcreen.
By now, my current solution is that I limited the number of tabs I can create, depending on the screen resolution.
Is there any type of scrolling component that can be added to the tab header or a solution that doesn't need to limit the number of tabs?

Unfortunately, Semantic UI doesn't provide support for tab overflow out of the box, so this can't easily be done without me adding extra CSS to the library, which isn't a good idea due to themeing etc.
One option is to use a tabs menu:
<sui-tabset>

            Automatic
            Template Based
            Component Based

...

...

Lots of thanks!


Ideas

Hey! I recommend making it so the color you choose saves, and also a feature for splitting music (https://github.com/danielfrg/demucs-app)

also, maybe a discord for development updates, not really that big of a deal, but if you feel like it, you should

Thanks for the feedback! I implemented the saving colors feature, so your selected colors should persist between refreshes now. I don't want to get into paying a bunch of money for hosting stem splitting software or anything right now so unless there's some sort of free api for splitting stems, I don't think that that feature will be implemented. Even then, it may be out of scope.
Here's a discord I just made: https://discord.gg/jfnFhggJGZ. I'll probably set up an automated channel for site updates soon. How do you think I should show links to the discord and github on the site? Maybe text links in the top right or icons in the bottom right?

maybe like just a basic hyperlink in the top left would work


Landing Page - Hero Section - Illustration to Replace Stock Photo

Background
High level direction

Looking for illustrations that represent open source, working better together, community effort that helps our friends and love ones.
Imagery that shows a wide array, of multiple voices within our community being shared/listened to versus focusing in one individual

Tactical direction

Individuals are from across the communities of Louisiana
Diverse ethnicities and diverse genders
Individuals look up, down, and or to the right and left as if they’re looking at each other
Should have expressions of curiosity and interest

@wavemanstudio here you go.

@wavemanstudio
Like the matrix of these photos,
and I could see us representing a wide swath of the community,
specifically the state of Louisiana as part of hero image for this site.
Would be nice to reflect that similarly for a hero illustration on this landing page.
CC @iamjoshfrank @CheetoMao @gerardramos for visibility

@wavemanstudio - I moved this Issue into our Implement column with an indication you're currently working on this illustration. Our kanban for the Nightingale project is found here:
https://kanban.prod.revelry.net/app/revelry-foundation/nightingale
Feel free to hit "pause" if you're not actively working on this Issued.

@iamjoshfrank @jeremybrady - working on the grid of portraits, currently 8 columns by 4 rows of square portraits. Feeling like this is reasonable number I can finish EOD PST. But keeping it modular, so we can continue to expand on the dimensions if we find it necessary.

Here's the grid of portraits. We could sprinkle in the portraits from the other two illos if needed.


site url getting overridden

I have my site url as
site          = "https://<IP_ADDRESS>:9443/oauth2/authorize"
but once i call client.auth_code.authorize_url(:redirect_uri => redirect_uri) my url is changed to
https://<IP_ADDRESS>:9443/oauth/authorize?client_id=J7H_LoEIdaf9aVXF_opqtVMLgwoa&redirect_uri
So my oauth2/authorize is being replaced by oauth/authorize
Is there any way to fix this?

Figured out the answer
You need to pass the :authorize_url attribute.
require 'oauth2' client = OAuth2::Client.new('client_id', 'client_secret', :authorize_url => '/oauth2/authorize', :site => 'https://<IP_ADDRESS>:9443')

This needs to be in the documentation!

@KelseyDH Please submit a PR if you have time :)


Upstream HPPC-RT

Can some of the modifications be incorporated from https://github.com/vsonnier/hppcrt

HPPC-RT has a slightly different set of goals - that's why it's a parallel fork.


Confused in implementation

Hi,
I have read all documentation but I am so much confused what to use. Currently I have created templates in sendgrid I want to send emails using that templates. Please tell me which library I should install ? which code to use? Please guide me step by step
Thanks,

Hello @Ibad021,
If you use this library, here is the documentation for sending templates: https://github.com/sendgrid/sendgrid-php#templates
If you just want to use curl, here is an example: https://github.com/sendgrid/sendgrid-php/blob/master/example_v2_template.php
If you would like additional options or support, please reach out to<EMAIL_ADDRESS>Thanks!
With Best Regards,
Elmer

Thank you Elmer, You were so helpfull
On Thu, Mar 3, 2016 at 9:51 PM, Elmer Thomas<EMAIL_ADDRESS>wrote:

Closed #205 https://github.com/sendgrid/sendgrid-php/issues/205.
—
Reply to this email directly or view it on GitHub
https://github.com/sendgrid/sendgrid-php/issues/205#event-576521662.

--
Regards,

Ibad ur Rehman Khan*


ROX-22046: Paladin Cloud integration form

Description
Paladin Cloud integration form allows to configure integration with Paladin Cloud instances. This is new type of Cloud Source integrations, which will be extended with OCM integration type next release.
Three fields are required:

Name for the integration
Paladin Cloud endpoint
Token to access Paladin Cloud

For now we do not implement Test functionality either in the backend or frontend.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change

CI TBD
Manual test:

Connect local frontend to infra instance with cloud sources enabled
Check that create/update/delete/view functionality work as expected
Check that updateCredentials functionality works properly: credentials are modified when checkbox is checked, otherwise credentials are not sent to the backend

/test gke-ui-e2e-tests
/test ocp-4-11-ui-e2e-tests

/test ui-component-tests

/retest


add missing comma

@Mikhail-Zex There is a comma missing in install_requires since #858, which breaks the installation

Damn. I should have checked.  Thanks for the quick fix.


Fixed no intellisense on types

I removed the module declaration from the types file and updated a devDependency so ts would stop screaming about no http module dec.

Thanks :)


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v2.6.34 to before v4.14.241",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=dae52d009fc950b5c209260d50fcc000f5becd3c",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=e818f2ff648581a6c553ae2bebc5dcef9a8bb90c"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "dae52d009fc950b5c209260d50fcc000f5becd3c",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "e818f2ff648581a6c553ae2bebc5dcef9a8bb90c",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "media: ngene: Fix out-of-bounds bug in ngene_command_config_free_buf()\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.14.241 by commit e818f2ff648581a6c553ae2bebc5dcef9a8bb90c, it was introduced in version v2.6.34 by commit dae52d009fc950b5c209260d50fcc000f5becd3c. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001400


Add bytearray.count

@talhof8 thank you for contributing to this project!

https://github.com/RustPython/RustPython/pull/916 added bytearray.count(), so this PR is no longer relevant.

Codecov Report

Merging #928 into master will increase coverage by 0.49%.
The diff coverage is 78.26%.

@@            Coverage Diff             @@
##           master     #928      +/-   ##
==========================================
+ Coverage   63.97%   64.46%   +0.49%
==========================================
  Files          89       89
  Lines       15498    15791     +293
  Branches     3482     3566      +84
==========================================
+ Hits         9915    10180     +265
+ Misses       3278     3262      -16
- Partials     2305     2349      +44

Impacted Files
Coverage Δ

vm/src/obj/objstr.rs
74.87% <100%> (ø)
:arrow_up:

vm/src/obj/objbytearray.rs
70.35% <77.27%> (+1.46%)
:arrow_up:

vm/src/obj/objbytes.rs
73.94% <0%> (+0.48%)
:arrow_up:

vm/src/builtins.rs
68.39% <0%> (+0.71%)
:arrow_up:

vm/src/obj/objbyteinner.rs
74.66% <0%> (+5.75%)
:arrow_up:

vm/src/function.rs
58.29% <0%> (+7.1%)
:arrow_up:

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 5124563...b17cabe. Read the comment docs.


ZAP allocates memory for ZResources, but it never frees them.

ZAP may also need to free additional assets used by any given ZResource.

Anything in particular stand out to you? I know ZTexture and ZOverlay make allocations, but they also free them on destroy. I guess for Rooms/Scenes their commands need to be freed once all is done.

Anything in particular stand out to you? I know ZTexture and ZOverlay make allocations, but they also free them on destroy. I guess for Rooms/Scenes their commands need to be freed once all is done.

Anything in particular stand out to you? I know ZTexture and ZOverlay make allocations, but they also free them on destroy. I guess for Rooms/Scenes their commands need to be freed once all is done.

As far as what specifically comes to mind. This is not meant to be a 100% exhaustive list.

XML handle may not be closed.
File handles may not be closed.
Strings that are not stack variables need to be freed. Some come from XML, and it may handle freeing of those. We need to consult documentation.
ZFile.resources needs to free and destroy the ZResource on ZFile destructor.
ZFiles need to be destroyed after the XML file is done being processed. They are allocated memory in a few places, but never freed.
ZCollision allocates VertexEntry classes into the vertices vector and never frees them.
ZCollision allocates PolygonEntry classes into the polygons vector and never frees them.
ZCollision allocates a CameraDataList and never frees it.
ZCollision allocates WaterBoxHeader classes into the waterBoxes vector and never frees them.
ZCutscene allocates commands into the commands vector and never frees them.
ZDisplayList allocates ZDisplayLists into the otherDLists vector and never frees them.
File.ReadAllBytes allocates a char array that is never freed.
File.ReadAllText allocates a char array that is never freed.
SetListActor allocates ActorSpawnEntry to the actors vector and never frees them.
SetCutscenes allocates ZCutscene to the cutscenes vector and never frees them.
SetEntranceList allocates EntranceEntry to the entrances vector and never frees them.
SetLightSettings allocates LightingSettings to the settings vectors and never frees them.
SetMesh allocates MeshHeader and other objects, but never frees them.
SetPathways allocates PathwayEntry to the pathways vector and never frees them.
SetRoomList allocates RoomEntry to the rooms vector and never frees them.
SetStartPositionList allocates ActorSpawnEntry to the actors vector and never frees them.
SetTransitionActorList allocates TransitionActorEntry to the transitionActors vector and never frees them.
ZRoom.ExtractFromXML allocates multiple objects and never frees them.
ZRoom allocates room commands to the commandList vector and never frees them.
ZLimbStandard.FromRawData allocates a ZLimbStandard that is never freed or added to the ZResource list in a ZFile.
ZLimbLOD::FromRawData allocates a ZLimbLOD  that is never freed or added to the ZResource list in a ZFile.
There are ZDisplayList allocations throughout the code that are never freed or added to the ZResource list in a ZFile.

Anything in particular stand out to you? I know ZTexture and ZOverlay make allocations, but they also free them on destroy. I guess for Rooms/Scenes their commands need to be freed once all is done.

As far as what specifically comes to mind. This is not meant to be a 100% exhaustive list.

XML handle may not be closed.
File handles may not be closed.
Strings that are not stack variables need to be freed. Some come from XML, and it may handle freeing of those. We need to consult documentation.
ZFile.resources needs to free and destroy the ZResource on ZFile destructor.
ZFiles need to be destroyed after the XML file is done being processed. They are allocated memory in a few places, but never freed.
ZCollision allocates VertexEntry classes into the vertices vector and never frees them.
ZCollision allocates PolygonEntry classes into the polygons vector and never frees them.
ZCollision allocates a CameraDataList and never frees it.
ZCollision allocates WaterBoxHeader classes into the waterBoxes vector and never frees them.
ZCutscene allocates commands into the commands vector and never frees them.
ZDisplayList allocates ZDisplayLists into the otherDLists vector and never frees them.
File.ReadAllBytes allocates a char array that is never freed.
File.ReadAllText allocates a char array that is never freed.
SetListActor allocates ActorSpawnEntry to the actors vector and never frees them.
SetCutscenes allocates ZCutscene to the cutscenes vector and never frees them.
SetEntranceList allocates EntranceEntry to the entrances vector and never frees them.
SetLightSettings allocates LightingSettings to the settings vectors and never frees them.
SetMesh allocates MeshHeader and other objects, but never frees them.
SetPathways allocates PathwayEntry to the pathways vector and never frees them.
SetRoomList allocates RoomEntry to the rooms vector and never frees them.
SetStartPositionList allocates ActorSpawnEntry to the actors vector and never frees them.
SetTransitionActorList allocates TransitionActorEntry to the transitionActors vector and never frees them.
ZRoom.ExtractFromXML allocates multiple objects and never frees them.
ZRoom allocates room commands to the commandList vector and never frees them.
ZLimbStandard.FromRawData allocates a ZLimbStandard that is never freed or added to the ZResource list in a ZFile.
ZLimbLOD::FromRawData allocates a ZLimbLOD  that is never freed or added to the ZResource list in a ZFile.
There are ZDisplayList allocations throughout the code that are never freed or added to the ZResource list in a ZFile.


Error: EACCES at npm install with sudo

I run this command sudo npm install -g electron-prebuilt, then I get this error (not sure if it's electron fault):
$ npm install -g electron-prebuilt
/usr/bin/electron -> /usr/lib/node_modules/electron-prebuilt/cli.js

><EMAIL_ADDRESS>install /usr/lib/node_modules/electron-prebuilt
> node install.js

/usr/lib/node_modules/electron-prebuilt/install.js:15
  throw err
        ^
Error: EACCES, mkdir '/home/rc/.electron'

npm ERR<EMAIL_ADDRESS>install: `node install.js`
npm ERR! Exit status 8
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is most likely a problem with the electron-prebuilt package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node install.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls electron-prebuilt
npm ERR! There is likely additional logging output above.
npm ERR! System Linux 3.16.0-37-generic
npm ERR! command "/usr/bin/node" "/usr/bin/npm" "install" "-g" "electron-prebuilt"
npm ERR! cwd /home/rc
npm ERR! node -v v0.10.38
npm ERR! npm -v 1.4.28
npm ERR! code ELIFECYCLE
npm ERR! not ok code 0

To void this error, I just manually created /home/rc/.electron folder. Then it installs normally.

@rcotrina94 in the output you pasted it shows you didn't use sudo, it has $ npm install -g electron-prebuilt. Are you sure you typed sudo when you ran it? The EACCES error means it didn't have permission to create a new folder in the /home/rc/ directory

Getting the same problem. Creating the /root/.electron directory did not help.
> sudo npm install electron-prebuilt -g
/usr/bin/electron -> /usr/lib/node_modules/electron-prebuilt/cli.js

><EMAIL_ADDRESS>install /usr/lib/node_modules/electron-prebuilt
> node install.js

/usr/lib/node_modules/electron-prebuilt/install.js:15
  throw err
        ^
Error: EACCES, mkdir '/root/.electron'
    at Error (native)
npm ERR! Linux 4.0.1-1-ARCH
npm ERR! argv "node" "/usr/bin/npm" "install" "electron-prebuilt" "-g"
npm ERR! node v0.12.3
npm ERR! npm  v2.10.0
npm ERR! code ELIFECYCLE

Running Arch linux
> npm -v
2.10.0
> node -v
v0.12.3

This isn't an issue with electron, but with your installation of node/npm and the permissions on your machine. I'd probably purge node and npm and re-install them.

Uninstalling and re-installing nodejs and npm from my system fixed the issue for me. Thanks

In my case, npm install run successfully after creating .electron directory and granting write permission to it.
# at your home directory
> mkdir .electron
> chmod 777 .electron
> sudo npm install -g electron-prebuilt
> chmod 755 .electron

OSX 10.10.3
node v0.12.3
npm 2.9.1

This issue is now tracked at https://github.com/mafintosh/electron-prebuilt/issues/27.

Had the issue. Reinstall node/npm does not work. Only chmod 777 for ~/.electron solved it.
Not sure why .electron suddenly have no write perms

reinstalling node/npm did not work for me, so i tried the following and it with worked perfectly
sudo npm uninstall electron-prebuilt -g
rm -rf ~/.electron
sudo npm install electron-prebuilt -g

Can't install electron, tried everything. creating electron directory doesn't work as running the npm install electron -g again with sudo deletes the directory. this is annoying.


updating ReadMe for AKS Extension supporting CI/CD pipelines

💡 Linked with this work: https://github.com/Azure/vscode-aks-tools/pull/42

💡 Please note: you might have missed another place to add this commands which is line 6 or line 7 ( the bullet points which define what all commands fucntiaonlity does via this extension)


feat(bundler/NSIS): Add Korean to supported NSIS languages. Partially fixes #6906.

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[x] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[ ] I have added a convincing reason for adding this feature, if necessary

Other information
Relevant issue: #6906
Relevant PR: #6907 #6940

Thank you for your contribution :heart:.


pipx: fix build and refactor

Description of changes
fixes https://github.com/NixOS/nixpkgs/issues/298439

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 298528 at dddb894f run on x86_64-linux 1

2 packages built successfully:

pipx (python311Packages.pipx)
python312Packages.pipx


Deny assignments with Blueprint HOW TO EXAMPLE?

Hello All ,
I am re-opening this thread  https://github.com/MicrosoftDocs/azure-docs/issues/40720
I wanted to understand if there is an working Example on using the Deny Assignments with Blueprint, like how to use all those properties which are mentioned in this article:
https://learn.microsoft.com/en-us/azure/role-based-access-control/deny-assignments#deny-assignment-properties
Right now the only way I can see is thru ASSIGN the BLUEPRINT and LOCK/READONLY everything but dont know how this whole thing can be implemented with some of those Deny Assignment properties?
Any example is missing and that would really help on grasping the concept.

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

ID: 5bc146ac-d530-fccb-0157-43cfcaeab937
Version Independent ID: 84376ebf-b328-39b0-5435-569a31c879e1
Content: Understand Azure deny assignments - Azure RBAC
Content Source: articles/role-based-access-control/deny-assignments.md
Service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

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

I hope we get some answer as I totally understand that Deny Assignments can only be done currently using Powershell or CLI option as mentioned in this documentation: https://learn.microsoft.com/en-us/azure/governance/blueprints/create-blueprint-powershell#assign-a-blueprint
But issue here is where and how and what we put in that Json file to define all those Deny Assignment properties, I exported out a Json from existing deny assignment created but still it is not clear and hence an example/sample on how and what that Json would look like with the Deny Assignment properties.
Following is the extracted Json from portal from Activity log of Deny Assignment:

"{"properties":{"denyAssignmentName":"Deny assignment 'XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX' created by Blueprint Assignment '/subscriptions/YYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYY/providers/Microsoft.Blueprint/blueprintAssignments/Assignment-TestDenyAssignment22'.","description":"Created by Blueprint Assignment '/subscriptions/YYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYY/providers/Microsoft.Blueprint/blueprintAssignments/Assignment-TestDenyAssignment22'.","permissions":[{"actions":[""],"notActions":["Microsoft.Authorization/locks/delete","/read","Microsoft.Network/virtualNetworks/subnets/join/action"],"dataActions":[],"notDataActions":[]}],"scope":"/subscriptions/YYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYY/resourceGroups/RyanAnthonyrg/providers/Microsoft.Storage/storageAccounts/storexpkf4vrze5522","doNotApplyToChildScopes":false,"principals":[{"id":"00000000-0000-0000-0000-000000000000","type":"systemDefined"}],"excludePrincipals":[{"id":"898e31d8-0f81-4873-a35e-5660046257db","type":"servicePrincipal"}],"isSystemProtected":true},"id":"/subscriptions/YYYYYYYYYYYYYYYYYYYYYYYYYYYYYYYY/resourceGroups/RyanAnthonyrg/providers/Microsoft.Storage/storageAccounts/storexpkf4vrze5522/providers/Microsoft.Authorization/denyAssignments/XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX","type":"Microsoft.Authorization/denyAssignments","name":"XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX"}",

Hi @DeepMalh44
Thanks for your feedback. As indicated in the docs, you can't directly create your own deny assignments.
https://learn.microsoft.com/en-us/azure/role-based-access-control/deny-assignments#how-deny-assignments-are-created
Azure Blueprints and Azure managed apps are the only way that deny assignments are used within Azure. Resource locking in Azure Blueprints is an application of deny assignments. The following tutorial shows how to add a Blueprint resource lock.
https://learn.microsoft.com/en-us/azure/governance/blueprints/tutorials/protect-new-resources
Can you clarify the scenario you are trying to accomplish?
thanks

@rolyon We are trying the same thing and cannot figure out how to do the deny. We are trying to recreate the example in your own documentation here. We've gone through the documentation you have linked and it only mentions locks, not deny assignments. I can find no actual example of how to create a deny assignment via blueprints, only locks.

Hi @lotsahelp
It's my understanding that when you create a Blueprint lock, a deny assignment is applied.
https://learn.microsoft.com/en-us/azure/governance/blueprints/concepts/resource-locking#how-blueprint-locks-work

Hi @DeepMalh44 , @lotsahelp
Thanks for your feedback. Let us know if you have additional feedback. I'm going to close this feedback for now. If you have additional feedback, we can reopen.
thanks

#please-close


GBK编码的HTML字符串初始化Ji后里面的内容全乱码了

网页是GBK编码的，然后我这样初始化Ji
 let NSGBKStringEncoding = CFStringConvertEncodingToNSStringEncoding(CFStringEncoding(CFStringEncodings.GB_18030_2000.rawValue))
 var htmlString = String(data: data!, encoding: NSGBKStringEncoding)
 let ji = Ji(htmlString: htmlString!, encoding: NSGBKStringEncoding)

我这样转换后ji里的内容中文的全乱码了，不知道为什么
网站是梦幻西游论坛：http://my.netease.com/forum.php

@zixun Thanks for your issue, I'll take a look on that. Not sure whether it's related to this issue https://github.com/honghaoz/Ji/issues/5#issuecomment-135741737

@honghaoz  OK，thank you~~ waiting for you~

@honghaoz  I found the reason!! Its not the mistake of Ji,but me.
the HTML String from Web is based on GBK,but I change some HTML code by
 stringByReplacingOccurrencesOfString: withString:

and this api return a NSUTF8StringEncoding String.
if I tell the Ji the string encoding is UTF8,everything will work well.
if I not call the string api above,and tell Ji the encoding is GBK,it also work well~

@zixun Ahaha, cool, thanks for your responding! I believe this issue could be closed


Remove Ember.Assert for IntlGetResult

This will now display to others than they are missing a key in their
messages instead of a more cryptic message saying that a result is not
of type IntlGetResult.
Fixes issue #96.
Attempted to add a test to catch the case, but I test less than I should.  The assert.throws with a sub-expression seems not to pick up the error. Not sure if it's because the error happens in a different place than it's expecting due to it being a sub-expression.

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:

Not sure if the CLA check will re-run, but the CLA has been signed.

CLA is valid!

:+1:


Add support for an upload key.

Add option to require a key to upload packages.

Hello, @sverrirs,
Could you merge this PR from @manuelxmarquez i also need it ?
Thanks


Tooltip starts blinking with container "overflow: auto" css

I have a sidebar which I recently decided to make independently scrollable. When I set overflow: auto; to the sidebar-wrapper class or <aside> html element, v-tooltip with placement: right starts to blink and it's almost impossible to see what's in the popover.
When I switch placement of the popover to default blinking goes away. I've tried to disable all tooltip styles, but it didn't help.
What's going on? How can I set the overflow property without having this weird behavior?

I have the same problem, it was mentioned that you can fix it with this option, but it didn't work for me, maybe it works for you.
popperOptions: {
    modifiers: {
      preventOverflow: {
        escapeWithReference: true
      }
    }
  }

@pmochine that's incredible! It worked fine for me, thank you!

Better use:
popperOptions: {
  modifiers: {
    preventOverflow: {
      boundariesElement: 'window'
    }
  }
}

this is what worked for me:
import VTooltip from 'v-tooltip';
import Vue from 'vue';

Vue.use(VTooltip, {
  defaultBoundariesElement: 'window',
});

The fix worked for me. For reference, this is my usage:
<v-popover :popperOptions="{ modifiers: { preventOverflow: { boundariesElement: 'window' } } }">
    <slot name="button" />
    <template slot="popover">
        <slot name="content" @passEvent="passEvent($event)" />
    </template>
</v-popover>


Include sourcesContent in composed Hermes source map

React Native version: >= 0.60.2
Steps To Reproduce

Build a new RN app in release mode with Hermes.
The source map under generated/sourcemaps/react/ does not contain the sourcesContent field.

Describe what you expected to happen:
The source map should contain a fully-populated sourcesContent field.
More context: https://github.com/facebook/hermes/issues/85 opened by @bengourley

As far as I know nothing has changed, can anybody else shed any light?

Should this issue be on the Hermes repo?

No, the relevant build script is part of the RN+Hermes integration which lives in this repo.

This issue still exists.

https://github.com/facebook/metro/pull/574 will fix this. Apologies for the long delay.


lack of uuencode

Hi,
In file  MailMimeParser/src/Stream/PartStreamRegistry.php 'uudecode' is missing.
Daetails:
ZBateson\MailMimeParser\Stream::attachEncodingFilterToStream();
is
case 'x-uudecode':
         stream_filer_

should be
 case 'x-uuencode':
 case 'uuencode':
            stream_filter_

Could you correct this issue?

Hi @barszpaw,
Thanks for the suggestion.  Looking around at other implementations I see Python's mimetools library supports all of these: uuencode, x-uuencode, uue, x-uue
I'm assuming the creator's of the mimetools library saw those being used in enough places to warrant implementing them as acceptable... unless anyone else objects here, I'll go ahead with that for the next release (will target first of the month).


Update pekko-connectors-jms to 0.0.0+185-f7b9e3d4-SNAPSHOT

About this PR
📦 Updates org.apache.pekko:pekko-connectors-jms from 0.0.0+140-7d704044-SNAPSHOT to 0.0.0+185-f7b9e3d4-SNAPSHOT
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.apache.pekko", artifactId = "pekko-connectors-jms" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.apache.pekko", artifactId = "pekko-connectors-jms" }
}]

labels: library-update, early-semver-build-metadata, semver-spec-build-metadata, commit-count:1

Pekko connectors 1.0.0 has been merged