Switch p7zip to actively maintained fork

Issue description
The original P7zip was last updated over 3 years ago and has accrued security vulns in the meantime. It has been (correctly) marked as insecure in the package definition, but it would be of more use if it were replaced by a fork that fixed these problems. I suggest this one, as it seems to be the most actively maintained.

Here's a diff between the latest release of p7zip and this fork, excluding the new bundled libraries (C/{fast-lzma2,lz4,zstd,zstdmt}), if anyone would like to take a look. I haven't pored over it in detail, but some notes:

This fork is linked from the 7-Zip site, so it's probably a reasonable canonical choice.

The bulk of the changes seem to be feature and documentation additions rather than refactors or security fixes.

It seems that the repository has commits identified as fixing CVE-2016-9296 and CVE-2018-5996, but nothing clearly marked for other vulnerabilities we have patches for. We should make sure it incorporates fixes for these before switching to it as a presumed-secure upstream; we can't stay on top of fixing and patching every single vulnerability that comes up ourselves, so having the known CVE patches incorporated seems like a reasonable baseline expectation.

The README doesn't mention the non-free licensing of the RAR code. Not really a big deal since this is a mess upstream too, just noting it.

I feel like the really big task in keeping p7zip updated will be keeping it up to date with 7-Zip upstream, which I assume it shares a lot of code with. It's not clear if there's been much thought put into that here; there have been 10 upstream 7-Zip releases since the last p7zip release so there's probably a significant amount of divergence already (3 years of work, probably including a lot of security fixes, compared to the ~month this fork has existed). Discussion in the thread about p7zip's fate seems to imply that the divergence is significant enough that it might be better to base a new p7zip on 7-Zip proper rather than the existing port.

Sorry for the negativity; these aren't intended to block this, just note down the concerns I'd have before committing to a fork. It would be great if we had a secure, updated p7zip package, since a lot of people are used to the interface, it has wide format support, and some software uses it internally. But from having skimmed over the code a bit and seen the existing vulnerabilities I know it'll be a big task to keep it updated and free of holes, and I think we should apply due diligence here given the liability it's been. I'd feel more comfortable re-enabling it once it's removed from builds (see e.g. #87837) and runtime dependencies where possible to reduce the general exposure.

Sorry for the negativity

No problem, this was going to be messy no matter what.
I'd agree that upstreaming the nixpkgs fixes should be first priority, and that a big 7zip --> p7zip refactor should be farther down the road. In the meantime, do you think we could simply switch to the new fork while keeping the patches in the build process? It look like that we'd have at least a few less vulns, which would be useful even if it's still marked as insecure since some people/processes are still using it anyway.
Do you think that can that be done now, or does it need to wait for #87837?
also: cc @7c6f434c (Michael Raskin), the maintainer.

We have 4 CVE patches. There is a changelog that claims to apply some patches for 3 CVEs, and the last patch is defacto applied. Are there any more CVE patches we know of?

Here are CVE lists for 7zip and pzip.
The nixfile's four patches are:

CVE-2016-9296
CVE-2017-17969
CVE-2018-5996
CVE-2018-10115

Szcnick's fork seems to have all 4, although 2018-10115 is only noted here (also CVE-2015-1038 is listed under 15.14 in the changelog).
Assuming Szcnick's fixes are good (and that 2015-1038 is a non-issue, which I think it is), switching now would provide equal security plus zstd support and the other new features.
Going off the 7zip list, I think this is what we got so far:

CVE
has patch
last vuln
note

CVE-2008-6536

4.5.7

CVE-2015-1038
y

CVE-2016-2334

16.00

CVE-2016-2335

15.05

CVE-2016-7804

windows-only

CVE-2016-9296
y

CVE-2017-17969
y

CVE-2018-5996
y

CVE-2018-10115
y

CVE-2018-10172

windows-only

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/p7zip-and-possible-rces/6951/16

Let's hope everything goes well…

What about the official release of 7zip itself? https://news.ycombinator.com/item?id=26438921


Add initial draft of Toolchain Hardening for C and C++ guide

As discussed during the C/C++ Compiler Options BP Guide call on 2023-02-15 I'm opening this draft PR in order to share the initial Markdown draft of the "Toolchain Hardening for C and C++" guide from Ericsson. There were already good suggestions for improvement during the call. We can use this PR to continue the discussion and start incorporating changes.

I have a number of comments, but I'm also really excited to see this. THANK YOU!
I'm very concerned about the term "Toolchain Hardening". We have a lot work going on to counter supply chain attacks, in part due to serious problems like the attack on SolarWinds' Orion. It's true that some people use that phrase, but also is used to describe hardening of build environments against external attacks, so it's a really ambiguous term.
I think the title should at least include the term "option" or "options", and maybe "option flags". These are option flags, if we put them in the title everyone will understand. Also, toolchains are lot more than compilers. Make and cmake are part of many toolchains, for example. So I'd use the term "compiler" - that's what everyone calls them.
So I'd title this something like this:

Compiler option hardening guide for C and C++

Feel free to move around the order of words, etc. But I think if "compiler", "option", "C" and "C++" are in the title, people are more likely to find it and will understand what they found :-).

FYI: We prefer markdown to meet certain lint requirements - you can see in "details" all the things it whines about :-).
We can forcibly merge it & then fix that stuff, if desired. I actually think that would be better - we do want to fix the problems, but the sooner we make this visible to others as a starting point, the better.


Create auto-comments.yml

I am solving THis  Isuue :https://github.com/Clueless-Community/Spectrum-UI/issues/230
I have created a Workflow that automatically commits when an issue is opened, a pull request is created, an issue is closed, or an issue is assigned or unassigned.
too run this workflow you need to enable the following permission

Feature Details:
When an issue is opened, the auto-comment should greet the author and provide a brief acknowledgement and request for additional context.
When a pull request is opened, the auto-comment should greet the author, express gratitude, and remind them to follow the project's contributing guidelines.
When an issue is closed, the auto-comment should thank the author for their contribution and encourage further engagement.
When an issue is assigned to someone, the auto-comment should notify the assignee and encourage them to start working on it.
When an issue is unassigned from someone, the auto-comment should notify the assignee about the change and suggest reassignment if they are offline.
Benefits:
Improved communication and engagement with contributors.
Provides clear instructions and acknowledgements for various events.
Enhances collaboration by setting expectations and providing reminders.
Reduces manual effort by automating comment generation.


[RFC] net: Split off net_app_settings lib to a separate directory under subsys/net/lib/

Over last time, there was a lot of talk that "net_app API does too much things at once", which leads to confusion. At the same time, net_app is (at least) 2 largely independent parts:

Client/server helper API.
Network setting module.

It should be noted that p.2 isn't even an "API", it's a module, or subsystem, which is driven by "declarative" Kconfig settings and just configures network environment on the system startup.
To make this distinction clear, I propose to move all file related to settings module from subsys/net/lib/app/ to subsys/net/lib/settings/. This would affect init.c, bt_settings.[ch], ieee802154_settings.[ch].
To make the move clean, we may also want to rename Kconfig options, e.g. NET_APP_AUTO_INIT => NET_SETTINGS_AUTO_INIT. (Or alternatively, we can use subsys/net/lib/config/ and NET_CONFIG_*).

@jukkar, @tbursztyka : Let me know if this makes sense, I can prepare a patch for 1.13.
@mike-scott, @mbolivar, @d3zd3z : FYI

+1 from me.
I just wonder if the settings could cause confusion with the settings API, so in this respect the config might be better choice.

I think this is a really good start. Regardless of how the networking code is used, it will have to be configured somehow.
My concern with this setup is that it seems overly tailored to testing, and isn't particularly useful for an application using Zephyr. It is reasonable that a testing environment uses a hard-coded network configuration. However, this doesn't really make sense for an actual application. In this instance, it is going to have to be one or more of:

Have a static configuration, but stored in some kind of configuration area. Maybe there is a UI or a web interface that allows the user to set this configuration.
DHCP. The system will need to try to get network configuration from dhcp (and any networking part of the app will need to wait for it). This is currently mixed into one of the many things that net_app is doing.

@jukkar:

I just wonder if the settings could cause confusion with the settings API, so in this respect the config might be better choice.

I assume you mean "settings in the non-volatile storage" API, CONFIG_SETTINGS. Using "config" sounds good then.

@d3zd3z:

My concern with this setup is that it seems overly tailored to testing, and isn't particularly useful for an application using Zephyr.

I can definitely ack that, but I don't see that as a concern. It all boils down to whether we work on a generic RTOS, or some shrink-wrapped functionality toolset. In my list, it's the first - at least, first of all. And then testing is very important part of it. Doing stuff for real-world production applications is important too, but there're supposedly people who concentrate on that, and such a support may be as well specific for each application/product (as an example, you talk about DHCPv4, but we were told that IPv6 will rule the world, and there's no need (or support) for DHCPv4 in it). I'd be happy to contribute to that part of development too, but hopefully, after the bugs in the IP stack are fixed, as that makes developing real-world apps hard at all.
All in all, it's interesting topic to discuss, but I guess it's outside the scope of this ticket.

+1 from here.  Especially with the move from net_app APIs to socket APIs scheduled for 1.13/1.14, I think the timing is right to move out the net config/settings.


AutoProxy Daemon is not working

×<EMAIL_ADDRESS>     Loaded: not-found (Reason: Unit<EMAIL_ADDRESS>not found.)
     Active: failed (Result: exit-code) since Sun 2024-08-11 13:15:24 IST; 2 days ago
   Main PID: 1953 (code=exited, status=1/FAILURE)
        CPU: 4ms

Aug 11 13:15:24 linux-ws1 systemd[1]<EMAIL_ADDRESS>Scheduled restart job, restart counter is at 5.
Aug 11 13:15:24 linux-ws1 systemd[1]: Stopped IITJ Autoproxy Daemon for user bcca.
Aug 11 13:15:24 linux-ws1 systemd[1]<EMAIL_ADDRESS>Start request repeated too quickly.
Aug 11 13:15:24 linux-ws1 systemd[1]<EMAIL_ADDRESS>Failed with result 'exit-code'.
Aug 11 13:15:24 linux-ws1 systemd[1]: Failed to start IITJ Autoproxy Daemon for user bcca.

Please check if a file named<EMAIL_ADDRESS>is present in /etc/systemd/system/ directory.

ok

yes

Please check if a file named<EMAIL_ADDRESS>is present in /etc/systemd/system/ directory.


Add support for new C# 6 keywords

C# 6 introduces two new keywords, nameof and when. (For reference, here's the Roslyn wiki on the subject which lists all the new features added to C# 6.)
This pull request adds support for the keywords to Razor/general .cs files so they'll be syntax highlighted in the editor.

@jamesqo, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

Just to clarify, the standalone tokenizer is not used for C# syntax highlighting of VS Code.
I you want to make changes there, make a pull request to https://github.com/atom/language-csharp

@aeschli I've already submitted pull requests there last week: atom/language-csharp#53 and atom/language-csharp#54. Would this still be useful to be merged?

Yes, I'll still merge your pull request, we will profit from the Razor fix. I just have to wait until we are done with the March endgame. The CSharp extension including the grammar is now managed at https://github.com/atom/language-csharp. I suggest we file a bug there to ask them to pick up the latest csharp grammar to include your fixes there.

@aeschli I've already submitted pull requests for C# 6 there which have been merged (see the links above). I guess I'll just wait until April for so this one can be too... :smile:


Client websocket disconnected

Describe the bug
The web socket is disconnected after few steps and with the error:
Error sending data to client
Client websocket disconnected (<CloseCode.ABNORMAL_CLOSURE: 1006>, None)
Error sending data to client
INFO:     connection closed
Steps to Reproduce

Start the backend and frontend
Ask Opendevin to create something
After few steps, some commands, the error appears

Expected behavior
Opendevin should continue working on the next step
Actual behavior
Opendevin stops and a new session should be started
Additional context

Several fixes went in this morning--should be fixed now. If you're still seeing issues, feel free to open another bug!


Parsing argument lists

The test does not fail anymore. Also changed a couple of tests to add some blank characters between the kernel name and the beginning of the arguments.

Found issue #57 after submitting this.


Update allow-or-prevent-custom-script.md

This document has been extensively rewritten based on the new "no script" feature that is rolling out. Main changes:

added a note on the "To allow custom script on OneDrive or user-created sites" section that states the UX is going away,
addede a whole new "Manage custom script from SharePoint tenant administration" section,
removed "HTML Field Security" row in the "Features affected when custom script is blocked" section,
added "Embed" web part row in the "Features affected when custom script is blocked" section.

@cmalonzo mentioning you here .. I will need other to review as well.

@PramodBalusu as well.

ready to

Hi @maggierui, this is now ready to be merged please.
Thanks a lot!


Added temp_ses_config.yml example file

This ses_config file structure should be used with 7_deploy_osh until the values provided in the example file taken from SES documentation can be tested.

The ses-ansible stuff needs to be updated to reflect this so our engcloud installation works and stage 2 can be used on a local SES install - have a look at ses-ansible/roles/setup_ses_configs/templates/ses-config.yml.j2


WIP : Enable custom domain with  DNS Provider

Description
Changes proposed in this pull request:

Enable custom domain with  DNS Provider

Related issue(s)

/test

/test all


Add airship and riak-haskell-client

raik-haskell-client? I don't see it on hackage.

Oops, that's the repo name not the hackage  package name.
It should be https://hackage.haskell.org/package/riak

Airship has some restrictive upper bounds that need to be relaxed before it can be incorporated in the nightly builds.
either-4.4.1 depended on by:

airship-<IP_ADDRESS> (==4.3.*). Tim McGilchrist<EMAIL_ADDRESS>@lambda_foo. @helium

tasty-<IP_ADDRESS> depended on by:

airship-<IP_ADDRESS> (==0.10.*). Tim McGilchrist<EMAIL_ADDRESS>@lambda_foo. @helium

tasty-quickcheck-0.8.4 depended on by:

airship-<IP_ADDRESS> (==0.8.3.*). Tim McGilchrist<EMAIL_ADDRESS>@lambda_foo. @helium

wai-<IP_ADDRESS> depended on by:

airship-<IP_ADDRESS> (==3.0.2.*). Tim McGilchrist<EMAIL_ADDRESS>@lambda_foo. @helium

warp-3.1.7 depended on by:

airship-<IP_ADDRESS> (==3.0.*). Tim McGilchrist<EMAIL_ADDRESS>@lambda_foo. @helium


fix crash when doing debugging output

Crashes hard when trying to do get() on str.
I tried to work out what your intention here was and fix it accordingly.
I may have understood it totally wrong. My Python-foo isn't terribly strong.
With this change it does output something, and it doesn't crash.

Thanks! Never noticed this in current testing environment since the error is handled. The debug log is there to help determine if get/post requests are actually supported for given vehicle. Nothing that really impacts the use of the library at the moment.


Request to add the propagated history to the root policy status

For the replicated policy, I find that we have added the event history to the policy status for checking the compliance changes on the replicated policy.
Do we also consider adding the event history to the rootPolicy for checking the propagated updates? as follows:
status:
...
       history:
          eventName: *,
          lastTimestamp: *,
          message: "Policy rootpolicy-1 was propagated to  cluster cluster1"

Hi @yanmxa! We don't add the history to the root policy for scalability. However, we could add a clusteradm get command that would fetch this information from all managed clusters. Would that suit your use case here?

Thanks @dhaiducek! It might indeed cause scalability issues. However, we can retrieve the information from the root policy event now!


Minor point - New and Make

Thanks for reworking this to be value oriented.  I revisited my project where I was using this and forgot I had reported the whole pointer thing and wondered "Why did I use pointers everywhere".  Then I came here and checked the issues to see that I had raised the issue with you and was pleased to see that v2 was now available.  That was perfect since I was rewriting my code to use values anyway.
However, one minor thing I noticed. You still have a function called New.  The Golang convention here is to use Make for functions that return values and New for ones that return pointers.  So I'd suggest you change New to Make.  You could include New as well...but it should return a pointer.  But I'm not how useful that would be for someone since the whole API is now value based.  But the value versions of the methods should work with the pointers (just not the other way around).  So it shouldn't hurt to have both New and Make.  I could submit a PR if you want.
Thanks again!

Thx for pointing that out, fixed in v3


feat: exclude external pages from Sitemap

Include pages with relative externalUrl with correct URL in Sitemap
Exclude pages with absolute externalUrl from Sitemap
Include pages with relative externalUrl with correct URL in Sitemap
Improve output XML (no more linebreaks and spaces)

The isset check for externalUrl did not work correctly before, pages with relativeUrl were still included in the Sitemap XML.
see sitemap.xml of blowfish.page and search for https://blowfish.page/users/

Thanks for this @fuse314


doc: more links

Description:
Cleanup README todos in #21
Related:
closes #21
TODO:

[x] Updated README documents
[x] Complete PR Body

!merge

!merge


Persist game data between sessions

Probably use localforage. Definitely save inventory; maybe save current site/overworld too?

See what it would take to recreate the current site if the app is reloaded - probably save the current SiteConfig, tilemap and maybe Area data? Should be simpler if we don't keep Stuff on the map indefinitely (see other issue)

We should be able to restructure a map to make things a little more straightforward for creating save data. See below. We also need to think through the logic for actually getting save data to and from localforage.
As for save data, we want to stick to the data bags needed to recreate everything for the map - we shouldn't have to serialize anything from Phaser I wouldn't think.

Start by selecting any instance-specific data for the SiteConfig that will be used to generate the map.
Not sure if this is really a separate step, but generate list of Area models, which includes "exits" to specific Sites in an Overworld Site.
First two pieces are used to generate the tilemap (should be persisted)
Use tilemap to generate list of Dust models

Also need to make sure we update the Dust model list every time more dust is cleared


No scrolling for .grid-content.shrink

When using the code below I often get a scroll bar shown for the element
<div class="grid-content shrink">
...
</div>

I'm wondering if scrolling can be disabled for the .grid-content.shrink selector.  I'll add it to my own SCSS but I'm wondering if it would be good to include by default as I'm not sure when you would want a scrollbar to appear in that case.

There shouldn't be a scroll bar if a grid element is shrunk, because "shrink" means "this element should only take up the space it needs". So using overflow: hidden (or overflow: visible to make using dropdown menus easier) as a default would make sense.


Refresh button doesn't close out network connections

Godot version
4.0 latest
System information
Windows 10
Issue description
Very Minor thing I saw and wasn't sure if it was a bug or not, when using the high-level multi-player and having a client connected to a server, if you use the 'restart' button connections to that server aren't terminated during the reload. Eventually these connections will timeout but it can cause unexpected behavior.
Steps to reproduce

Create a client > server application, have the server output when clients connect / disconnect via signals.
Hit the refresh / reload button in the godot editor on the server.

Minimal reproduction project
No response

I believe this may not be a bug. I assume you refer to the button that restarts the running instance from the editor, and if so it just closes the process as if you were closing the window or press Alt+F4.
I'm not familiar with the new Multiplayer API but if you initiated the connections manually, you are probably expected to terminate them too - so you can do it e.g. in _exit_tree to make sure that connections are terminated properly when quitting.
On the other hand if the Multiplayer API initiates those connections automatically it could indeed be expected to terminate them automatically on close too.
Cc @Faless

I assume you refer to the button that restarts the running instance from the editor, and if so it just closes the process as if you were closing the window or press Alt+F4.

The debugger uses OS::kill() to stop the game process, so there is no way for the project to catch it.

I assume you refer to the button that restarts the running instance from the editor, and if so it just closes the process as if you were closing the window or press Alt+F4.

The editor debugger uses OS::kill() to stop the game process, so there is no way for the project to catch it.

Indeed. If that's a problem for more users we could maybe add an option so that the Stop / Restart buttons actually send a quit request to the running instance instead of kill. That would potentially be slower but might be useful in cases like this one.

I assume you refer to the button that restarts the running instance from the editor, and if so it just closes the process as if you were closing the window or press Alt+F4.

The editor debugger uses OS::kill() to stop the game process, so there is no way for the project to catch it.

Indeed. If that's a problem for more users we could maybe add an option so that the Stop / Restart buttons actually send a quit request to the running instance instead of kill. That would potentially be slower but might be useful in cases like this one.
If it's mostly useful for networked apps, it could be part of godotengine/godot-proposals#522 (comment) among advanced run options.

Yep this makes perfect sense. It is a bit of a 'stretch' to call this an issue, again it's only in editor but I think there would be a use for a graceful quit in-editor. I imagine this may be useful when building plugins around the editor as well. Appreciate the response!


⚠ Bofrost.be has degraded performance

In fba4286, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1988 ms

Resolved: Bofrost.be performance has improved in d3e9679.


Use nicer Debug implementations

Same as Display for now
Closes #503.

It appears as though nothing has been changed since my previous comment. Did you forget to push something?

My bad :) Should be good now.

Looks good now! I'll take a final look over once I'm on my laptop. I also want to look into the internal compiler error when generating coverage, as that should never happen (not your fault).

Unfortunately I am able to reproduce the internal compiler error on the currently nightly. This is unrelated to this PR, as I can reproduce it on the current main. Consequently I see no reason not to merge this.
Thank you!


Information on mmseqs cluster module

Information
Hi,
I am a bit confused with the MMseqs2 suite modules.
If I understood correctly:

when I do mmseqs search args, it uses the algorithm described in https://www.nature.com/nbt/journal/vaop/ncurrent/full/nbt.3988.html   (3 stages: prefilter, ungapped alignment, gapped SW alignment. The last one generates similarity file, giving the % of similarity for each pair of sequences)
when I run mmseqs linclust args, it executes the algorithm described in https://www.nature.com/articles/s41467-018-04964-5
For mmseqs cluster, I saw that it is slower than linclust, but also way more sensitive. So, I am using this module, to cluster sequences with min-seq-id = 0.8 (default of mmseqs). But I do not understand what is used. In the userguide, I saw :

mmseqs cluster: Clusters sequences by similarity. It compares all sequences in the sequence DB with each other using mmseqs search, filters alignments according to userspecified criteria (max. E-value, min. coverage,. . . ), and runs mmseqs clust to group similar sequences together into cluster' (p.17)

At the same time, in p.22, I saw:

The cascaded clustering workflow first runs linclust, our linear-time clustering module, that can produce clustering’s down to 50% sequence identity in very short time. To achieve lower sequence identities and/or to further improve the resulting clusters, we continue with three cascaded clustering steps

Cascaded clustering being the default for mmseqs cluster, this would mean that the search module is not used?
When using mmseqs search args I can get the similarity file in blast format (using convertalis). Therefore, is it also possible to get this file at the end of clustering step, if it uses mmseqs search algorithm as I saw p.14?
I don't know if my questions are really clear. Please do not hesitate to ask me if you need some more information.
Thanks a lot for your help!

Cascaded clustering being the default for mmseqs cluster, this would mean that the search module is not used?

In default, mmseqs cluster performs first linclust followed by cascaded clustering.

When using mmseqs search args I can get the similarity file in blast format (using convertalis). Therefore, is it also possible to get this file at the end of clustering step, if it uses mmseqs search algorithm as I saw p.14?

It is possible to get the similarity file by realigning the cluster. Following is an example:
mmseqs cluster seq clu tmp
mmseqs align seq seq clu aln
mmseqs convertalis seq seq aln aln.m8

In default, mmseqs cluster performs first linclust followed by cascaded clustering.

So, mmseqs search algorithm is never used for clustering?

Each iteration of the cascaded clustering performs a prefilter (prefilter), gapped SW alignment (align) and a clustering (clust).


Please make isEmulator and getDeviceName into constants

Feature Request
Please make isEmulator and getDeviceName into constants again.
Why it is needed
I'm using isEmulator and getDeviceName when bootstrapping reactotron, which is in turn tangled up with Redux bootstrapping. Having this async makes things veeery complicated.. :)

First let me say I feel the pain, and obviously am trying to find a balance
However - getDeviceName is not a constant in real life, so that one likely not.
isEmulator is potentially expensive even if static, which is why it is not static right now.
Is it possible to use those in your project via the xxSync APIs and if the chrome debugger is an issue, detect debugger and mock the results (https://github.com/react-native-community/react-native-device-info/issues/776#issuecomment-535627454)? Possibly best of both worlds and no need to deliberate here - esp re DeviceName which really is dynamic

That's what I'm doing now. Not really optimal though.
Could at least isEmulator be made constant? I see that at least for iOS it's basically using the same logic as getDeviceId which is already a constant - so shouldn't be that expensive, right? On Android there are a few string comparisons - is that what you mean by potentially expensive or am I missing something here?
One way to optimize - and I guess that goes for more calls than isEmulator - is to move some of the "constant logic" to JS so it's only executed when needed. Just keep the most basic stuff as native constants (on iOS getDeviceId and on Android the build constants needed) and move the string comparisons to JS so it's only executed when needed. I guess making the string comparisons into regexes could help a bit as well.
I think I understand what you're trying to accomplish with this (adding as little startup overhead as possible in this widely used library?). Just keep in mind that it's easy to overengineer this kind of stuff - which in the end is executed mostly native and will probably have very minor effects on performance compared to other factors. Ease-of-use and DX is obviously very important for these kind of generally used libraries as well - and in this case I'm not sure whether minor performance gains are more important.
Didn't really mean to write this long of a post - but I guess these are my 5 cents! :) Most of all - thanks for your work on this really useful library @mikehardy!

The thing that worries me about isEmulator is we're not getting a lot of android simulators out there - there is a PR which could add a lot more #713 - but I'm nothing if not pragmatic, and the current isEmulator check could be a constant to my way of thinking. You are absolutely correct it is not doing a whole lot currently.
Perhaps if I merge something like #713 (which is stale at the moment anyway) It becomes an optional async "deeper" isEmulatorComplete or something.
If you make a patch for isEmulator that follow the lines of the recent PRs re-constant-ifying (made up word) a couple things I'd merge it. It'll be another breaking change because of the promise semantics but I'm not getting much push back on these tiny changes for constants so I feel it can go out without a revolt


Provide clear and working instructions how to install platform.

Describe the bug
Hi. I would like to us this project to host a website for me and my friends. Unfortunately current installation instructions are not clear. I managed to follow them but when I enter my domain I am getting 500 error with information that tenancy not found for my domain. I managed to disable Tenancy Laravel plugin ( if it is not ready IT SHOULD not be enabled by default!) but that results in green page informing welcoming me to the project.
To Reproduce
Steps to reproduce the behavior:

Go to https://github.com/cgdprojects/genealogy
Click on try to follow installation procedure
Fail.

Expected behavior
Successful installation. Clear information what to do next.
Screenshots

Additional context
Version used: 10.0.5 and master.
I commented out  App\Providers\TenancyServiceProvider::class line in config/app.php but I have no idea if this is sufficient enough.
There is no list of required php extensions (if there any beyond Laravel required extensions?).
What versions of php are supported. What database engines are supported. Is Redis required?
What is "Broadcasting Setup" and why do I need it?
"Launch the site and log into the project with user<EMAIL_ADDRESS>password: password" ? where? how?
FYI: https://github.com/cgd-ui/genealogy/issues/218

Tenancy has issues but is required we are trying to fix the middleware.
PHP 8.1, optional pcntl for Horizon.
MariaDB 10, Mysql 8, postgresql should also work
broadcasting is for notifications
you launch the cgd-ui install for client side


Add NanoMsg package

GitHub repo:

https://github.com/nanomsg/nanomsg

Note: CMake based

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/33051794-add-nanomsg-package?utm_campaign=plugin&utm_content=tracker%2F1032790&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1032790&utm_medium=issues&utm_source=github).

I currently have a pending PR on nanomsg's repo (https://github.com/nanomsg/nanomsg/pull/934) that adds a nanomsgConfig.cmake file.
If it gets accepted, I will submit https://github.com/maddouri/hunter/tree/pr.nanomsg for a PR here.


Implementation of probe collection for every operation.

@jwustrack-amadeus and I opened a ticket  for requesting the feature of client side monitoring in the cpp sdk.
We propose to add a probe collection mechanism to AWSClient where all the operations are processed. The collection can be disabled with a compiler variable.
The user can then initialize the sdk with one or more probe handlers that will process a given probe.
We proposed a very simplistic implementation of the probe handler interface and we will propose later a gist of what we will do internally with the sdk.
For aggregation purposes, we prefered to add an abstract method to AmazonWebServiceRequest instead of using typeid.

I rebased my pr on your latest commits to use your new virtual method

GetServiceRequestName

However, I did change the prototype to make the method const and usable on a AmazonWebServiceRequest const reference.
Thank you in advance.

Hello,
I updated my PR since you made the method GetServiceRequestName const. There are much less changes in the pull-request. Can we have a first feedback ?
Regards,
Amine

Hi, thank you for submit this PR. We have a similar requirement internally and are still evaluating it.

Hello @singku,
Any update regarding this pull-request ?
Thanks.
Amine

We've implemented client-side monitoring in the SDK v1.6.0. You can learn about it here https://docs.aws.amazon.com/sdk-for-cpp/v1/developer-guide/sdk-metrics.html


[Event Request] Codeunit 10842 "G/L Entry Application"- OnBeforeNextLetter

Hi, would it be possible to get an event OnBeforeNextLetter codeunit 8614, procedure NextLetter, like this :

    procedure NextLetter(var Letter: Text[10])
    var
    IsHandled: boolean;
    begin
        IsHandled:= false;
        OnBeforeNextLetter(Letter, IsHandled);
        if IsHandled then
        exit;
        case Letter of
            '':
                Letter := 'AAA';
            'AAA' .. 'ZZY':
                IncrementAlphabeticString(Letter, 1, 3);
            'ZZZ':
                Letter := 'ZZZ.000000';
            'ZZZ.000000' .. 'ZZZ.999998':
                Letter := IncStr(Letter);
            'ZZZ.999999':
                Letter := 'ZZZ.AAAAAA';
            'ZZZ.AAAAAA' .. 'ZZZ.ZZZZZY':
                IncrementAlphabeticString(Letter, 5, 10);
        end;
    end;

[IntegrationEvent(false, false)]
local procedure OnBeforeNextLetter(var Letter: Text[10]; IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Food Delivery Landing Page(Responsive)

Excited to include a food delivery landing page built using only HTML and CSS, made responsive using media queries. A perfect starter website for any beginner.

Sure go ahead


Status wrong when turning on from config-ui-x

I currently am using this plugin with the Yeelight bulb and bedside lamp.
I try to turn them on from config-ui-x, the lights turn on, however, the status still shows off.
On Home.app however, it shows the correct status. I also can't turn it off too from the platform.
If I turn them on from Home.app, it's working as expected and the correct status is also shown on config-ui-x

Same problem for me! There is delay between change status.
If you wait about 20s, will status change?

Just tried it and looks like it changes to what it should be after 20 seconds.

+1

I tried to derive some state directly when it is set to emulate how the HomeKit UI does it.

Same here. Sometimes it takes 20 sec, sometimes 50.
Quite strange

Couldn't really reproduce this. I made quite some changes on the whole naming and characteristic setting recently. So it might be resolved with recent updates.


Presto CLI is firing 2 Queries when user submit  use information_schema;

During Initializing the Presto CLI  auto fires the 2 Queries
SELECT table_name FROM information_schema.tables WHERE table_schema = 'information_schema';
SHOW FUNCTIONS;
But after Presto CLI is user just give use information_schema; then also 2 Queries are get firs automatically.
So suppose user give use information_schema; 10 times then total 30 Queries  in Query Details.

The CLI has tab completion for table names and function names, which is why these queries are run. Are they causing a problem for you? They should be quick and cheap.

But I feel every time user give use schema; command these queries should not be fire. Yes I agree first time when launching CLI it is getting fire ok.

The table name completion is per schema, so it needs to be loaded for the current schema. I agree that functions could be loaded only once.

The table name completion is per schema -- But if u see suppose user give use 'default'; then it is loaded for current 'default' schema and user is inside the 'default' schema and again fire use 'default'(same schema ) again loaded.

We can handle this - functions could be loaded only once against this issue.

Yes, we could be smart and skip refresh if the current schema is the same. But I’m not sure it’s worth the trouble — how often do users run use for the schema they are already using?

We can handle this - functions could be loaded only once against this issue.


feat(android)!: specify export mode on BroadcastReceivers, requires compileSdk 33+

This PR introduces Android targetSdk 34 (Android 14) support.
Without this PR, devices running Android 14 will crash if the app using this dependency is using targetSdk=34.
An alternative approach would be to directly include androidx.core:core as a dependency of this project and use ContextCompat.registerReceiver instead.
More information can be found in this issue: https://github.com/react-native-netinfo/react-native-netinfo/issues/681
This is the smallest change that I can make and have this work, I think. Locally, in order to get this to build on my machine, I did also need to upgrade the gradle wrapper to work with newer JDKs. But I don't think those changes are necessary for this PR. We'll see what CI says :P

Looks like we just ran into the issue where CI doesn't actually even attempt to compile the java which is "awesome", so CI has nothing really to say about it. Will pull locally though and give it a run

let me know what your local results are! to be candid my react native experience is limited and I want a second opinion 🙇

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


AttributeError: module 'pyogg.opus' has no attribute 'opus_int32'

I'm unable to import OpusEncoder using the current master (3783343):
Traceback (most recent call last):
  File "/usr/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/kyle/projects/bitcast/mumbleice/mumbleice/__main__.py", line 44, in <module>
    icecast = IcecastConnector(cfg['server'], cfg.getint('port'),
  File "/home/kyle/projects/bitcast/mumbleice/mumbleice/bot.py", line 78, in __init__
    self.encoder = OpusEncoder()
  File "/home/kyle/projects/bitcast/mumbleice/pyenv/lib/python3.9/site-packages/pyogg/opus_encoder.py", line 20, in __init__
    self.set_max_bytes_per_frame(4000)
  File "/home/kyle/projects/bitcast/mumbleice/pyenv/lib/python3.9/site-packages/pyogg/opus_encoder.py", line 128, in set_max_bytes_per_frame
    self._max_bytes_per_frame = opus.opus_int32(max_bytes)
AttributeError: module 'pyogg.opus' has no attribute 'opus_int32'

Thanks for your feedback.
Could you please provide a bit more information about the code that generates the error? A minimal example would be ideal.
Also, have you looked at the example 03-write-ogg-opus.py? That shows usage of OpusBufferedEncoder which (internally) uses OpusEncoder.
Cheers,
Matthew

A basic proof of concept:
$ python3
Python 3.9.1+ (default, Jan 10 2021, 15:42:50)
[GCC 10.2.1 20201224] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from pyogg import OpusEncoder
>>> encoder = OpusEncoder()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/kyle/projects/bitcast/mumbleice/pyenv/lib/python3.9/site-packages/pyogg/opus_encoder.py", line 20, in __init__
    self.set_max_bytes_per_frame(4000)
  File "/home/kyle/projects/bitcast/mumbleice/pyenv/lib/python3.9/site-packages/pyogg/opus_encoder.py", line 128, in set_max_bytes_per_frame
    self._max_bytes_per_frame = opus.opus_int32(max_bytes)
AttributeError: module 'pyogg.opus' has no attribute 'opus_int32'
>>>

Thanks for that. I’ll have a look on Monday and get back to you.
Matthew

Found the error - I was missing libopusfile0. There should probably be better error handling here to report missing external libraries if the features that require them are used

Thanks for reporting back on the cause of the error.
I have committed changes to master that improve the exception messages when OpusFile is missing.  Would you like to confirm that they work as expected for you?
Also, your minimal example should now no longer error at all, given that it shouldn't have depended on OpusFile in the first place.  Again, would you be so kind as to double-check these changes for me?
Cheers,
Matthew

Yup, that works. Thanks!


gdbgui: init at <IP_ADDRESS>

Feature rich browser-based frontend with data structure visualizations
(like DDD), and gdb terminal access. Compatible with C, C++, golang,
Rust, fortran. Written in Python and JavaScript
gdbgui is at the top of the GDB Front Ends
list on https://sourceware.org/gdb/wiki/GDB Front Ends
Motivation for this change
gdbgui is not present in nixpkgs
Things done

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

[x] NixOS
[ ] macOS
[ ] 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 nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

This is my first "new package" PR so please let me know if I missed some requirement or convention. Thank you!

Cool!
It doesn't build for me (test phase fails):
$ nix-build -A gdbgui
...
wrapping `/nix/store/didsnhp5ixi1a9gqg6a588zv0gpqa0jr-gdbgui-<IP_ADDRESS>/bin/.gdbgui-wrapped'...
running install tests
running test
error: protocol not found
builder for ‘/nix/store/0rh11mh4lk772xs67k7x0y8f2xfwgqs2-gdbgui-<IP_ADDRESS>.drv’ failed with exit code 1
error: build of ‘/nix/store/0rh11mh4lk772xs67k7x0y8f2xfwgqs2-gdbgui-<IP_ADDRESS>.drv’ failed

Can you try with "nix.useSandbox = true" and see if you get the same issue? Looks like the tests does something that is unavailable in Nix sandboxes.

Looking at the error message I assume it tries to access /etc/protocols or /etc/services.
There is very ugly workaround for that: https://github.com/NixOS/nixpkgs/blob/master/pkgs/top-level/python-packages.nix#L2386
Maybe the test can be patched to disable that or if it is a single test it can be skipped.

@GrahamcOfBorg build gdbgui

Ok green light on darwin. Thanks


Update modulo for default avatars by id

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[x] Internal code
[ ] Library interface (affecting end-user code)
[ ] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
This was changed.

Don't users with discriminators still have % 5 ? Or is this waiting for the rollout to be completed

That discriminator check is still present in UserImpl and unchanged.


C++ temp converter

Pull Request Type
What kind of change does this PR introduce?
[x] Solved an issue/task
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:
What is the current behavior?

Issue Number: #1305
What is the new behavior?
Converts Celsius to Fahrenheit

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Feature: Add option to change settings via config file

in order to properly use the solution in a shared project folder one would want to be able to easily share the configuration in form of a (Json) file via git (examples from other tools: .eslintrc, .prettierrc, ...).
Your file should end on .json though to make editing them simple.

CLI might come in bit later.


Add new feature: Projects

Is your feature request related to a problem? Please describe.
Faults should be related to a Project. This would be a 1-M relationship
Describe the solution you'd like
Create a project model + controller, defining new relationships with faults and users.
A project should have a single user as an author and many faults. The schema would be something like:

id: primary_key
user_id: foreign_key
title: str
description: str

In another issue, faults should also have its schema updated to look like this:

id: primary_key
title: str
description: str
project_id: foreign_key
status: enum[OPEN, IN_PROGRESS, CLOSED, ABANDONED]

this can be thought more about later

Describe alternatives you've considered
N/A
Additional context
N/A

Part of this has been addressed in #29

Added and migrated projects
Established simple view and create form to make projects from UI

TODO:

Create 1-M relationship between faults and projects
Create additional columns for faults

Addressed in #33, need to update fault views and project views to be integrated


WIP: handle unrecognised APDUs / interaction with ledger-live while nanoapp is running

Current dependencies on/for this PR:

main

PR #21

PR #22   👈

This comment was auto-generated by Graphite.


Update config usage in working example

The config usage in the working example notebook is outdated as of this PR.  We should update it to reflect the current implementation in the deepdisc repo.  We should also identify parameters that the user is likely to want to change and place those in the dictionary that is input to the Informer/Estimator stages.  The other parameters can stay in the config file.

The idea here is to remove any parts of the code that change or set a config parameter, and put all of them in a "solo" config.  This makes it easy to use different models without changing the underlying code


Fix comparison of vector element

I am not sure if that is a problem but it seems to me.

It seems like a reasonable fix.
I don't have all the context though, this was introduced in d1571c4e80060ce5e753caf361889e285d1e9bc0, @vinx13?


Include pre-compiled libraries at release

Setup a CI to provide pre-compiled library packages for MBT-Go (and future FFI).
The libraries should be compiled for,

[ ] x86_64-unknown-linux-gnu
[ ] aarch64-unknown-linux-gnu
[ ] x86_64-apple-darwin
[ ] aarch64-apple-darwin
[ ] x86_64-pc-windows-gnu

The GH action matrix would look like,
- { os: ubuntu-latest, target: x86_64-unknown-linux-gnu }
- { os: ubuntu-latest, target: aarch64-unknown-linux-gnu }
- { os: macos-latest, target: x86_64-apple-darwin }
- { os: macos-latest, target: aarch64-apple-darwin }
- { os: windows-latest, target: x86_64-pc-windows-gnu }

Closes #155


03-configure-data-model-in-power-bi-desktop.md

English:
In the Options window, at the left, in the Current File group, select Data Load.
Current Translation:
在“选项”窗口左侧的“当前文件”组中，选择“数据负载”。
Suggested Translation:
在“选项”窗口左侧的“当前文件”组中，选择“数据加载”。

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

No locfile in iCMS, So I Re-index the file, waiting for result. ICMS QUERY

Run the bugfix tool and waiting for result.

fixed.


Remove unneccesary no-cache build arg

@ryanmattcollins Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@ryanmattcollins Thank you for signing the Contributor License Agreement!


How to log out user when hidden?

I cannot seem to find a good way to be able to log out while the sidebar is hidden.

Disable kiosk mode using the ?disable_km query string.


Contribution details UI bug

URL: https://opencollective.com/marshmallow/donate
Logged in as: sloria
Error message if any: In console: SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data
Expected result:
Browser: Firefox 60.0.1

This seems to have been fixed. The console error is still there, but it appears to be unrelated to this UI issue.


Enforce checks for unused imports and missed interpolation [databricks]

Signed-off-by: Gera Shegalov<EMAIL_ADDRESS>

build

build

build


Vapoursynth R49 and nasm

Vapoursynth in nixpkgs is still version R47 which is affected by serious (crashes on some CPUs, artifacts) bugs that were fixed in R49 (Audio support was also merged since then). Moreover R47 is the first vapoursynth version to not require nasm anymore (all asm was removed in favor of intrinsics). For details, the last few articles on the vapoursynth's blog are a good read. It'd be nice to have an upgraded package for it. It's my first issue so I'm not sure I did this correctly.

Tested and backported to 20.03 in b3fbd4f5b4d0ea1d0014ab9602b28bc8dce15331.


devops, automate tagged releases

Add functionality, so we can set a flag on a pr and then:

it should check which publishable are updated
update their patch version
publish them to NPM under the latest tag.

A follow up on the will be how to handle minor versions/

@SanderElias  So from what I can see, the workspace uses Nx, right? So GitHub Actions should be able to do the heavy lifting here I think. This is a link to an action that I use to run affected tests on libs/apps in a workspace, but you can do similar things to publish affected apps.
https://github.com/pjlamb12/ng-utah-nx-demo-app/blob/main/.github/workflows/nx-affected-tests.yaml
Essentially, you add a new target that can run a bash script if you need to, or a series of commands, and that would build and publish the library. The hard part would be determining the patch version manually if you can't use something like the standard-version package or something. I think there's a similar option for Nx workspaces.
I know at a high level how to do this, and could get most of it set up through GitHub Actions, but the specifics on updating the patch version and publishing with that new tag to NPM would be the trickiest part.
https://www.prestonlamb.com/blog/deploying-nx-affected-apps-from-github-actions

Most of the scripting needed for this is in place already.
The nx toolset only gives an offset of the latest nx build, not of a previous version released to NPM.

this is done


ImportError

Informations

Qiskit Ignis version: latest
Python version: latest
Operating system: windows 10

What is the current behavior?
Copying the code from the source for the quantum volume fitter but getting an import error:
----> 6 from ...utils import build_counts_dict_from_list
      7
      8 try:

ImportError: attempted relative import with no known parent package

Steps to reproduce the problem
Copy the code and try to run it
What is the expected behavior?
Should run without any errors
Suggested solutions

This code is not meant to be run independently, only when you call QVFitter as described in the tutorial:
https://qiskit.org/documentation/tutorials/noise/5_quantum_volume.html
Anyway, if you replace this line by:
from qiskit.ignis.utils import build_counts_dict_from_list
then it should work.
If you have any further questions - please ask them in the qiskit-ignis slack channel (see the link in the README file)


Migration , limitations and differences

Jose Alatorre from Main Sequence,
Hi, I tried to run FastTrackML against our current DB(running on Postgres) of MLFLOW but it was unsuccessful. A few initialization arguments seem incompatible, like backend-store-uri, default-artifact-root, and so on. I couldn't find in the documentation any indication regarding migrations or compatibility. Will be nice if the documentation is a little more extensive so more users can try this implementation with their current deployment. Bests!

Hi @JalatorrS1, thank you so much for your interest in FastTrackML, and sorry we didn't reply sooner to your issue!
We have been focusing mostly on the code so far, hence the lack of documentation — it is however a major goal of this project to have top notch documentation soon!
Until we have it, here is how you can run FastTrackML against an existing MLFlow database:

we support MLFlow databases created with MLFlow 1.21.0 to 2.2.2 — higher versions will require you to use FastTrackML 0.3.0 which we plan to release very soon (#303 adds support for these higher versions)
when using an existing MLFlow database, we will upgrade its schema to the one from MLFlow 1.29.0-2.2.2, so your database would then become unusable with lower versions of MLFlow — if you need to keep it compatible, please use a copy instead!
we will also alter the schema in other places that have remained compatible with MLFlow so far, but we can't guarantee it will always be — again, if in doubt, please use a copy instead!
in order to point FastTrackML to an existing database, it should be sufficient to pass it your existing Postgres database URI via the --database-uri parameter or via the FML_DATABASE_URI environment variable
the default artifact root can be set via the --artifact-root parameter or the FML_ARTIFACT_ROOT environment variable (only supported in FastTrackML 0.3.0 or in nightly releases, usable via the gresearch/fasttrackml:edge Docker image)
we currently only support artifacts stored on a filesystem or on S3 or S3-compatible storage
if you use an S3-compatible storage platform (like Minio), you need to set the FML_S3_ENDPOINT_URI environment variable to whatever your MLFLOW_S3_ENDPOINT_URL is pointing to

Here is a concrete example. If you used the following command to start an MLFlow tracking server, you can replace it with the equivalent FastTrackML command:
# MLFlow
mlflow server --backend-store-uri postgresql://postgres:postgres@localhost/mlflow --default-artifact-root s3://mlflow

# FastTrackML
fml server --database-uri postgresql://postgres:postgres@localhost/mlflow --artifact-root s3://mlflow

We also raised #304 to ensure we never break this compatibility again in the future.

Thank you Jonathan this is helpful I will give it another try when 0.3.0 is
released.
On Tue, Sep 12, 2023 at 7:13 PM Jonathan Giannuzzi @.***>
wrote:

We also raised #304 https://github.com/G-Research/fasttrackml/issues/304
to ensure we never break this compatibility again in the future.
—
Reply to this email directly, view it on GitHub
https://github.com/G-Research/fasttrackml/issues/288#issuecomment-1716124758,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH5TMAD32Z3E5CTALQ4ZNZDX2CJ2FANCNFSM6AAAAAA4HKYYZ4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @JalatorrS1, we have released FastTrackML 0.3.0. Please let us know how it goes!


Data Placement, Rack Awareness and Scheduling

Not sure if this really fits into the goals for this Incubator, if it does not, please feel free to close the issue.
At some level a BatchJob processing  a large corpus of data should have awareness of data (location, Skew etc)  for efficient scheduling.

If there are multiple copies of the data block, the schedule should be aware of their locations for enhance scheduling.
The scheduling algorithm should have ability to optimize the cost of data access: local disk --> Single Network Hop (within Rack) --> Multiple Network Hops.
Intelligence to address Data Skews: Performance of a BatchJob is directly related to the ability to address data skews efficiently. Schedulers should have intelligence to address the stragglers.

These requirements may be more appropriate for a layer above the scheduler, if so there should be a way to identify how these requirements would translate into changes at the scheduler layer.

This seems to me like it could be an application level concern. For example, in Spark/HDFS, the way we dealt with it was to teach Spark to use node-affinity (https://github.com/apache-spark-on-k8s/spark/issues/315). The scheduler respects affinity/anti-affinity fields which can be used for expressing different levels of locality.
cc @davidopp @liyinan926

I'm also thinking it should be handled by application, e.g. Spark; but for other batch job, e.g. MPI, it do not have such kind of framework to handle it. There's a proposal, named IndexJob, in k8s, does it make sense to support data aware scheduling for them? for example, after job_1 finished, its output is on local disk and shared disk, and job_2 prefer to use them, if not in local disk, job_2 will get from shared disk. I'd like to keep this open as a potential requirement.

Yeah, I agree this seems like an application-level concern.
That said, for the example @k82cn gave, my understanding of how local PVs will work is that if job_1 and job_2 both reference the same PersistentVolumeClaim, then job_2 will schedule onto the same node as job_1 (assuming other scheduling requirements are satisfied) even if job_2 is not created until after job_1 terminates.
There's some general discussion of data-locality scheduling in kubernetes/features#139

my understanding of how local PVs will work is that if job_1 and job_2 both reference the same PersistentVolumeClaim, then job_2 will schedule onto the same node as job_1 (assuming other scheduling requirements are satisfied) even if job_2 is not created until after job_1 terminates.

The output of job_1 is preferred, not required :). If job_2 can be scheduled to those host, it can improve the SLA; if not, it's also OK to run :).


[BCH] It is impossible to access to generateCashAddress() function

Its need to make it function public

Or better point it into the switch statement:


Scrolling up and down the scroll bar really quick freeze Visual Studio 2017

video repro -> https://youtu.be/GZuBC_5wr9w
Noticeable things:
-Scrolling up and down the scroll bar really quick freeze visual studio
-If scrolled slowly the scroll bar shrinks, as if it is re-calculating its size based on how far we went down the list.

This should be logged against the Color Theme Editor extension


Add a test to verify that child processes are isolated

This test verifies that the before  and  after events, which are executed for each child process, are isolated from each other in terms of memory and resource.

Thanks!


Create a filter for projects based on topics or languages

Search / filter has been implemented but it currently only works based on project description.

Topics included. Good enough.


Allow other PSR7 abstraction

Hello !
Thanks for this very usefull lib.
While I tried to integrate it in my projects, I was wondering why you choose ringcentral instead of guzzle psr7 implementation.
I move beyond that, and now my question is why don't let the developper choose its own implementation like in Buzz. Because it was the lib that inspired this one.
If you're agree, I can help you to make this move. I'm currently working on an adaption of this lib, to provide some sugar with Reactive Programming operators. The outcome of this discussion may lead to more or less work for the adaptation.
Thanks for your time

@Th3Mouk Happy to hear this project is useful to you!
This project is documented to use PSR-7 interfaces, the actual PSR-7 implementation is an implementation detail that really shouldn't matter for the most part. Accordingly, this implementation may also change over time and it is intentionally not covered in the documentation.
We're currently using ringcentral/psr7 because it's a relatively lightweight PSR-7 implementation that happens to to support all PHP versions this project currently supports. Other than that, I really don't care much about this implementation either.
If there are some good reasons to change this implementation, then I'm not opposed to this change. As far as I'm concerned, I'm not aware of anything that is missing from the current implementation. If you feel like filing a PR, go for it and I'm happy to take a look and review this. However, keep in mind that I very much care about maintainability and long term support, so I aim to avoid adding unneeded complexity and instead like to focus on adding new features for consumers of this package.
I believe this has been answered, so I'm closing this for now. Please come back with more details if this persists and we can reopen this :+1:

Thank you for responding so fully. I understand your commitment, I'm just taking care of my dependencies and I try to avoid as much as I can useless one. I'm agree ringcentral/psr7 is totally fine but my approach was to totally decouple the lib like httplug recommendation. No need to reopen the issue now I have the background of the project, and features take precedence over optimisations.


Add code and href in WiresharkSslMasterKeyHandler

Motivation:
WiresharkSslMasterKeyHandler JavaDoc can be improved by adding code and href into it.
Modification:
Added code and href.
Result:
Better JavaDoc

@normanmaurer Can you retrigger CI? Seems it failed due to flaky test.

@hyperxpro thanks a lot!

yaaaayyyy!


Create custom type for use with Ecto

Withdrawing this PR as it seems to fairly difficult to force ecto to compile before this in large projects.  Will consider making it a separate project.


Find an elegant way to determine the ARCH in the Makefile.

We'll need to add an issue to find an elegant way to determine the ARCH in the Makefile.
The current way doesn't break local e2e testing on arm architectures, but it will be slower because they will always build all the packages as part of the make test-e2e
Originally posted by @YrrepNoj in https://github.com/defenseunicorns/zarf/pull/392#pullrequestreview-910416749

An alternate to uname on mac is the arch command.  So after your initial ifeq ... Darwin ... you can use a single arch command to get the processor architecture arm, arm64 or amd64/i386...
The outputs here kinda illustrate what I'm trying to relay:
➜  zsh-z git:(develop) uname -s
Darwin
➜  zsh-z git:(develop) uname -p
arm
➜  zsh-z git:(develop) # arm may be too generic
➜  zsh-z git:(develop) arch
arm64

roger that. i'll take a look at the pipelines

I think this was reasonable handled in #428 by allowing default + overrides for the arch coming into the makefile.


Fix: add examples of multi-type failures

Clarify how multiple specified place types will fail.
Thank you for opening a Pull Request!

Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open a GitHub issue as a bug/feature request before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

@LesliSchauf checkout go/github to register your account.


Remove duplicate description tag

I have removed the duplicate meta description tag that had 2021 in the text.

@FroostyCodes thank you!


Playground

What does this PR do?
Add resource(s)
For resources
Description
Added an interactive course.
Why is this valuable (or not)?
Valuable
How do we know it's really free?
It is completely free as it doesn't charge anything on the platform
For book lists, is it a book? For course lists, is it a course? etc.
Interactive Playground
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

@eshellman There is some with issue with the alphabetical ordering of this file /home/runner/work/free-programming-books/free-programming-books/books/free-programming-books-bn.md. It makes it real hard to pass the checks kindly check that in.

@sohansai I can't because I don't know the language.


Formatting error on conflicting options

Hello,
The code displaying this issue is here: guiniol/dnsmole@cc868c1
If I do dnsmole --serve -k 34 toto, I get an error because --serve and -k are conflicting. The problem is the error message:
error: The argument '--serve' cannot be used with '--socket <socket>'

USAGE:
    dnsmole <target> --connect --socket <socket> <--serve|--connect>

For more information try --help

The usage line displays --serve twice when it can only be specified once. Also, --serve and --connect are part of an arggroup which is required.
EDIT: Ideally,  should be at the end since it is a positional argument

Thanks for filing this, I'll look into the root cause and post back with what I find.

Thanks a lot! ^^


feat: dashboard-products

Description
Feature to add endpoints /dashboard/product/{product_id}, /dashboard/products, and /dashboard/products/count

Related Issue (Link to issue ticket)
Related Issue will be up in a bit

Motivation and Context

How Has This Been Tested?

[ ] Unittest on local machine

Screenshots (if appropriate - Postman, etc):

Types of changes

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

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Thank you @joboy-dev for the correction and approval

@Takinnuoye5


AST: Remove some unnecessary getCanonicalType() calls

@swift-ci Please smoke test

It's quite a bit faster to do some of these manipulations on canonical types; getAs<> expands to a lot of code.  The queries on SILType, at least, seem like they ought to stay optimized.

Really? The set of operations that were "optimized" in this manner were completely arbitrary and the "optimized" getNominalParent() was even wrong.
I think we will get better performance wins by reworking the expression type checker and SIL optimizer, not hunting for small micro optimization like this.

Slava, micro performance isn't in opposition with macro performance.  And yes, we didn't have optimized access paths for literally everything; I don't see why that's an argument against having optimized access paths for anything.  You just seem to have launched onto a random crusade against getCanonicalType().
And how was getNominalParent() wrong?

@slavapestov and I talked about this, and I've submitted #8080 to revert most of these changes.


Briefly describe the dataset and the variables that you will visualize

Climate Action:

TBC
TBC

Gender equality:

Up to 1990, 18 countries in the world had been led by a female head of state or government. What was that number in 2021?
How many companies in the world have a woman as top manager or CEO?

I suspected that these surveys might have a different source of data too. I don't think it's a big problem as we can always come up with new questions.
For the data, any subset of gapminder is fine. We don't need to make it's complicated, but I do think we should make it interesting

As each of us will implement one plot in later milestone, maybe we can all draft a question based on the above 'summary gapminder dataset' and propose the interaction component as well?  Then, we can based on the finalized questions to draft the proposals.

Question 1 (Vanessa) :  The average number of children per woman in 1990 is 4.05, what is the average number in 2018? Ans: 2.74
Sample Plot:

Intereaction:
By year range:  1900-2018
By color: none, region, income group

I like this plan:
My question might be: In 1980, the difference in average life expectancy between European and Sub-saharan African countries was X years. What was it in 2018?
Plot:

Axes: Year range between 1980 and 2018 vs. life expectancy (num years)

Content: Lines for each country, coloured by region and average lines (maybe split into two charts)
Interactivity:

Tooltips for each country

filters for sub-regions

filters for time period

filters for within-region GDP percentiles?

Here is my take on this:

Our theme would be education and misconception, I would try to frame the questions around it.
I would try to avoid asking for specific numbers (e.g what will be in 2018), but rather than asking for what the audience believes.
For example, Vanessa's question can be: the average number of children per woman in Western countries was 2 times lower than Asian countries. What will be the ratio now? 1. Still double. 2. About 50% 3. The same. That way, the user can guess before they explore our graph
I think Julien's question is excellence, with minor improvement like instead of asking for X years, we replace with friendly stats like percentage or increase-decrease gap.

do you think rough drafts of the plot like Vanessa made are necessary for the mockup or is a written description enough for you to move forward?
I can use the description for the mockup, or even your hand drawing

About the dashboard layout, I'm not sure how to design it yet:

We can throw everything into 1 page and call it a day, most people would do this
Only show the graphs according to the type of questions (it helps if the questions are related)

About the dashboard layout, I'm not sure how to design it yet:

We can throw everything into 1 page and call it a day, most people would do this
Yes, I'd think so because this is what a dashboard should look (Data at-a-glance)

Only show the graphs according to the type of questions (it helps if the questions are related)
Say we have 4 questions, I'd imagine the page to be partitioned into 4 parts and with each showing

the corresponding questions/ans (static text)
plot
_ interactive user controls
Some would use a left/panel to show the interactive widgets, but I think it depends if all our 4 parts can share the same widgets.  For example, changing the a date filter would change all 4 plots at the same time

Do we want a map?
I don't think it is necessary

Do our dashboard just be data exploration? We just show the data from the table without any models or running predictions. I think we would do this but I want to double-check
Yes, I think dashboard is just an interface for displaying and analyzing data, not involving any prediction.

@SonQBChau regarding 2018 vs. now point you mentioned, the issue is that the Gapminder data we're looking at only goes until 2018 I believe, it's a bit of a stretch to say "now"

Yea I didn't realize that. If you want to be precise, we can use the exact number then

Question 3: Over the last hundred years, child mortality has dropped by an average of -400% for some high-income countries. For low-income countries, this number is: -400%, -200%, -100%, -50%.
Plot:

Bubble Charts
Axes: Year range between 1918 and 2018 vs. child mortality rate
Content: circle for each country, coloured by income and size in income

Interactivity:

Tooltips for each country
Slider years

Question 4:
In 1918, the ratio of income per capita at the 75% percentile to that at the 25% percentile was X. How has that ratio changed in 2018? (possible answers: higher, lower, stays the same)
Plot:
scatter plot with lines
Axes: year vs. income per capita
Content: each point represents the averaged income per capita for a year for a chosen region, the default is all the countries; size can be used to compare value amount
Interactivity:
Slider: year range between 1918 and 2018
dropdown: filter by country or region and by income group


Timeout during template clone

While trying to clone template to VM using vagrant-vsphere I am running into timeout error after 5 minutes:
==> default: Calling vSphere CloneVM with the following settings:
==> default:  -- Template VM: pcc-149-56-104-121_datacenter2487/vm/Templates/Linux/centos-7-server
==> default:  -- Target VM: pcc-149-56-104-121_datacenter2487/vm/TEST/kitchen-cookbooks-default-centos-7-server_default_1501830422774_3121
STDERR: Undefined namespace prefix: //soapenv:Body/*
---- End output of vagrant up --no-provision --provider vsphere ----
Ran vagrant up --no-provision --provider vsphere returned 1] on default-centos-7-server

The issue is also described here:
https://github.com/nsidc/vagrant-vsphere/issues/106
Seems like in my case, on OVH Private Cloude (vSphere 6), timeout is more or less after 5min:
I, [2017-08-04T09:21:09.865227 #30699]  INFO -- Kitchen: -----> Creating <default-centos-7-server>...
E, [2017-08-04T09:26:23.183730 #30699] ERROR -- Kitchen: ------Exception-------
E, [2017-08-04T09:26:23.183807 #30699] ERROR -- Kitchen: Class: Kitchen::ActionFailed

vagrant-vsphere uses .wait_for_completionwhich timeouts on the vSphere side.

What I can suggest is to, instead of .wait_for_completion, ask every a while if the VM is already cloned.

I'm not sure there is much I can do about this from the rbvmomi side. What you mentioned should work. You can increase the timeout for rbvmomi, but if the provider is where the timeout is occurring, I'm not sure what I could offer.


added automatic closing of accordion; closes #7

Now the accordion closes the previous question div automatically if the new one is clicked.

@ahmadsohail404 the PR is not marked as hacktoberfest-accepted please look into it. :)


Data structure

@olafveerman
This is the data we'll be needed for each page to build the charts. I'll also include an image of the charts we're planning to build. (it's a sketch on my notebook so may not be super clear)
Homepage
First part will be a summary of the year 2016 and we need an array with all districts with only the values for 2006 and 2016. When the value of 2006 is not available we should use the next available year.

licença geral
licença reduzida
max licença geral
max licença reduzida
tipos de estacionamento (only for 2016)
tipos de contingente (only for 2016)

Well also need to create a chart showing the change per year. For that we need the national aggregate of all years (2006 to 2016)

licença geral
licença reduzida
max licença geral
max licença reduzida

Last, we need some more generic information:

Total taxis licença geral (for 2006 and 2016) [This is computable from other values. Is it worth to include?]
Total taxis licença reduzida (for 2006 and 2016) [This is computable from other values. Is it worth to include?]
Total number of distritos
Total number of distritos - mobilidade reduzida (2006 and 2016)
Total number of municipios
Total number of municipios - mobilidade reduzida (2006 and 2016)

/national.json
{
  distritos: [
    {
      id
      name
      lic-geral: [
        {year: 2006, value: 1}
        {year: 2016, value: 1}
      ]
      lic-mob-reduzida: [
        {year: 2006, value: 1}
        {year: 2016, value: 1}
      ]
      max-lic-geral: [
        {year: 2006, value: 1}
        {year: 2016, value: 1}
      ]
      max-lic-mob-reduzida: [
        {year: 2006, value: 1}
        {year: 2016, value: 1}
      ]
      estacionamento: [livre, fixo]
      contingente: [a, b]
    }
    // ...
  ]
  national: {
    lic-geral: [
      {year: 2006, value: 1}
      // ...
    ]
    lic-mob-reduzida: [
      {year: 2006, value: 1}
      // ...
    ]
    max-lic-geral: [
      {year: 2006, value: 1}
      // ...
    ]
    max-lic-mob-reduzida: [
      {year: 2006, value: 1}
      // ...
    ]
    total-taxis: [
      {year: 2006, value: 1}
      {year: 2016, value: 1}
    ]
    total-lic-reduzida: [
      {year: 2006, value: 1}
      {year: 2016, value: 1}
    ]
    total-distritos: 20 // Total number of distritos
    total-distritos-reduzida: [ // Total number of distritos with taxis with licença mobilidade reduzida
      {year: 2006, value: 1}
      {year: 2016, value: 1}
    ]
    total-municipios: 309 // Total number of municípios
    total-municipios-reduzida: [ // Total number of municípios with taxis with licença mobilidade reduzida
      {year: 2006, value: 1}
      {year: 2016, value: 1}
    ]
  }
}

Each distrito
The charts on the distrito page will be very similar to the ones on the homepage, only the data will differ.
We won't have a dedicated concelho page. Instead we'll have a table listing the concelhos of the distrito and show some info on it, including:

change per year for both geral and reduzida
types of estacionamento available

We'll also have summary of the year 2016 and we need an array with all concelhos with values from 2006 to 2016.

licença geral
licença reduzida
max licença geral
max licença reduzida
tipos de estacionamento (only for 2016)

We'll also be showing the change in time for the distrito being displayed. For that we need the district aggregate of all years (2006 to 2016).

licença geral
licença reduzida
max licença geral
max licença reduzida

Last, we need some more generic information:

Total taxis licença geral (for 2006 and 2016) [This is computable from other values. Is it worth to include?]
Total taxis licença reduzida (for 2006 and 2016) [This is computable from other values. Is it worth to include?]
Total number of municipios
Total number of municipios - mobilidade reduzida (2006 and 2016)

/[district-name].json
{
  concelhos: [
    {
      id
      name
      lic-geral: [
        {year: 2006, value: 1}
        // ...
      ]
      lic-mob-reduzida: [
        {year: 2006, value: 1}
        // ...
      ]
      max-lic-geral: [
        {year: 2006, value: 1}
        // ...
      ]
      max-lic-mob-reduzida: [
        {year: 2006, value: 1}
        // ...
      ]
      estacionamento: [livre, fixo]
    }
    // ...
  ]
  district: {
    lic-geral: [
      {year: 2006, value: 1}
      // ...
    ]
    lic-mob-reduzida: [
      {year: 2006, value: 1}
      // ...
    ]
    max-lic-geral: [
      {year: 2006, value: 1}
      // ...
    ]
    max-lic-mob-reduzida: [
      {year: 2006, value: 1}
      // ...
    ]
    total-taxis: [
      {year: 2006, value: 1}
      {year: 2016, value: 1}
    ]
    total-lic-reduzida: [
      {year: 2006, value: 1}
      {year: 2016, value: 1}
    ]
    total-municipios: 309 // Total number of municípios
    total-municipios-reduzida: [ // Total number of municípios with taxis with licença mobilidade reduzida
      {year: 2006, value: 1}
      {year: 2016, value: 1}
    ]
  }
}

Does this look reasonable to you?
Given the charts we have to build I think this is the most straightforward structure. If you have other thoughts let me know.

Here are the chart sketches

Thanks for this @danielfdsilva Will work on it.
Totally agree on the totals, these were already removed in: https://github.com/olafveerman/taxis/pull/23

@olafveerman Stop the presses!!!
After the last meeting some things changed which are reflected in the new mocks here: https://projects.invisionapp.com/share/TEAPCWCBG#/screens/221974968_Desktop_HD_-_V1
Basically a lot of information is now shown on a "concelho" level even though we're on the homepage.
As of right now the data we'd need is:
each distrito:
  - lic geral (from 2006 to 2016)
  - lic reduzida (from 2006 to 2016)
  - max licenças (only the total is needed in this case) (from 2006 to 2016)
  - population (2016 only)

  - each concelho:
    - lic geral (2006 and 2016)
    - lic reduzida (2006 and 2016)
    - types estacionamento

I think this is the bare minimum data we need to build the homepage. There will still be quite some data that needs to be computed client side, but can be done. My worry here is the file size. You think we'd be better off pre-processing everything and serving data per section?
The same data will be needed for the "distrito" page level.

Right on time.
Not sure if file size is a big issue here. concelho.json is 750kb with data for all years for:
lic-geral
lic-mob-reduzida
max-lic-geral
max-lic-mob-reduzida
pop-residente


[WIP][SPARK-31197][CORE] Exit the executor once all tasks and migrations are finished built on top of on top of spark20629

What changes were proposed in this pull request?
Exit the executor when it has been asked to decommission and there is nothing left for it to do.
Why are the changes needed?
If we want to use decommissioning in Spark's own scale down we should terminate the executor once finished.
Does this PR introduce any user-facing change?
The decommissioned executors will exit and the end of decommissioning. This is sort of a user facing change, however decommissioning hasn't been in any releases yet.
How was this patch tested?
I changed the unit test to not send the executor exit message and still wait on the executor exited message.

Test build #123957 has started for PR 28817 at commit ea8efc7.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28579/
Test PASSed.

Test build #123957 has finished for PR 28817 at commit ea8efc7.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123957/
Test FAILed.

Test build #123983 has started for PR 28817 at commit a2c0557.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/28606/
Test PASSed.

Test build #123983 has finished for PR 28817 at commit a2c0557.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/123983/
Test FAILed.

cc @Ngone51 @jiangxb1987

cc @tgravescs too FYI

To be clear this a WIP PR and not yet ready for review. I created it to give additional context after talking with @agrawaldevesh about our respective goals. I'll try and get a "read for review" PR out next week.

any progress here?

It's been merged in the non-WIP version of this PR.


Removed an only that got left in the tests.

Description:
Removes a .only that got left in the tests.
Related issue(s):
Fixes #1161
Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report
Patch coverage has no change and project coverage change: +46.97 :tada:

Comparison is base (cb6e3e0) 27.57% compared to head (87e8382) 74.54%.

:exclamation: Current head 87e8382 differs from pull request most recent head a8c50bf. Consider uploading reports for the commit a8c50bf to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1162       +/-   ##
===========================================
+ Coverage   27.57%   74.54%   +46.97%
===========================================
  Files          31       31
  Lines        2165     2165
  Branches      427      427
===========================================
+ Hits          597     1614     +1017
+ Misses       1523      439     -1084
- Partials       45      112       +67

see 11 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[AutoPR compute/resource-manager] updated Reimage API documentation in compute json for ephemeral os disks

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4644

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/3459


bitwise operations in if structure

if(window.innerWidth < 1600 & window.innerWidth > 1200){
....
}
the "&" It allowed in javascript. It is a bitwise operation

http://diyo.netai.net/bitwise.png

This issue was moved to Microsoft/TypeScript#5775


Feedback Submission - docs - Off-Chain Features - /v3/concepts/off-chain-features

What browsers are you seeing the problem on?
Chrome
Feedback
Links of References at end of article are broken

Fixed! Thanks @kamikazebr for the flag on this. Much appreciated 🚀


I did a fresh install of build 3126 and installed this and it doesn't seem to be working.

Are you aware of this?

Same problem here.

Can confirm. Build 3126 and not working. Also did a fresh install via package control

'ST3 3126' on Windows, 'ST3 3125' on Mac.
'docblockr_python' can not work...

Sorry for the silence around this one. Rolling out an update right now that should resolve the problem.


SwiftDriver: honour SWIFT_DRIVER_SWIFTSCAN_LIB on Windows

We would previously not honour this flag on Windows as the layout of the toolchain is different on Windows from the other platforms. The computed path needing to be different bifurcated the implementation. Hoist a path for override handling into the shared path.

@swift-ci please test


Dropdown toggle attributes not added

I have the following menu structure:

And this is the output I got:

The required attributes to toggle the dropdown are missing, so the dropdown menu cannot be opened.

For anyone facing the same problem, add 'depth' => 2 to your wp_nav_menu call.
public function primarymenu() {
    $args = array(
        'theme_location'    => 'primary_navigation',
        'menu_class'        => 'navbar-nav',
        'depth'             => 2,
        'walker'            => new \App\wp_bootstrap4_navwalker(),
    );
    return $args;
}


Unable to locate package miopenkernels-gfx90a-110kdb

Cannot install miopenkernels Packages
I am working some tasks using ROCm55 and followed here to install the package, tried both name of miopenkernels-gfx90a-110 and miopenkernels-gfx90a-110kdb, but got the same error below,
#22 1421.7 ++ [[ rocm55 == \r\o\c\m\5\5 ]]
#22 1421.7 ++ apt install -y miopenkernels-gfx90a-110
#22 1421.7
#22 1421.7 WARNING: apt does not have a stable CLI interface. Use with caution in scripts.
#22 1421.7
#22 1422.7 E: Unable to locate package miopenkernels-gfx90a-110

Can anyone let me know if there is any changes of installation?
FYI, For ROCm542, ROCm54 or ROCm532, apt install -y miopenkernels-gfx90a-110kdb is working properly. Thanks.

I think the name of miopenkernels-xxx change to miopen-hip-xxx on ROCm5,5, If we want to install miopenkernels-gfx90a-110kdb, we can try miopen-hip-gfx90a-110kdb. Can anyone confirm this change? Thanks.

I think the name of miopenkernels-xxx change to miopen-hip-xxx on ROCm5,5, If we want to install miopenkernels-gfx90a-110kdb, we can try miopen-hip-gfx90a-110kdb. Can anyone confirm this change? Thanks.

That's correct: https://repo.radeon.com/rocm/apt/5.5/pool/main/m/

@junliume This documentation needs an update: https://docs.amd.com/bundle/MIOpen_gh-pages/page/install.html#installing-miopen-kernels-package

Hi, @jithunnair-amd  and @PeixuanZuo , thanks for the response. I can install miopen-hip-gfx90a-110kdb for ROCm55 now.
Do I need to keep miopenkernels-gfx90a-110kdb for ROCm542, or I can also use miopen-hip-gfx90a-110kdb ?
Thanks.

FYI for ROCm523, still need miopenkernels-gfx90a-110kdb  or miopenkernels-gfx908-120kdb, otherwise, can use miopen-hip-gfx90a-110kdb.


Fix bug with microphone prompt being shown despite plist setting 'Enable...

[AKOrchestra addInstrument]  tries to disable the audio input, but is does not effectively avoid the microphone security prompt because [AKInstrument init] calls [AKOrchestra start] - BEFORE any instruments are ever added. Disabling the audio input in [AKOrchestra addInstrument] is too late to avoid prompting for the microphone, since the AKOrchestra has already been started when any subclass of AKInstrument is created. Thus, moving the code to enable or disable the input in  [AKOrchestra start]'s !isRunning condition solves the issue.

Thanks!

Thank you for the great library!


Configure integration test runner

We need some sort of integration CI to exercise the service working against a PostgreSQL database. We should also think about how we consume results from Kubernetes clusters, and if we want to cache that information in a test suite or if we want that to be part of the integration test.
Let's us this issue to discuss possible solutions and approaches.

We might be able to re-use the OpenShift CI to test the compliance service.
https://docs.ci.openshift.org/docs/how-tos/onboarding-a-new-component/

Add the openshift-ci-robot as a collaborator and installed the OpenShift CI application.

I proposed a review to configure OpenShift CI for this repository
https://github.com/openshift/release/pull/29151

Proposed adding an OWNERS file since it's required for OpenShift CI.
https://github.com/rhmdnd/compserv/pull/42

Added 'vendor/' directory in a separate PR to get around issues in the openshift/releases patch https://github.com/openshift/release/pull/29151
https://github.com/rhmdnd/compserv/pull/45

Added the ability to spin up OpenShift clusters to the compserv CI here:
https://github.com/openshift/release/pull/29368

Closing this since we have an OpenShift CI cluster spinning up on each patch proposed to the project and running the deploy target. We will update this to include an e2e target when we have end-to-end testing to run.