Prepare for release v1.0.7

Signed-off by: Ian Vernon<EMAIL_ADDRESS>

This change is

test-me-please

test-missed-k8s

test-docs-please

Star Wars Demo tests failed for the K8s tests: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-K8s/1075/testReport/junit/k8s-1/8/K8sDemosTest_Tests_Star_Wars_Demo/

test-me-please

The Star Wars test failed because in the v1.0 branch, we never actually deploy the file https://github.com/cilium/star-wars-demo/blob/master/02-xwing.yaml , which contains the 'xwing' pods with label 'alliance', which the test expects to be deployed, but are not.
Applying the following patch makes the test pass locally for me. Some of the commented out / changed lines are to make the test run on my local setup, because go vet fails if I don't perform said modifications:
$ git diff
diff --git a/test/k8sT/Nightly.go b/test/k8sT/Nightly.go
index 7b2bf335c..1504a333a 100644
--- a/test/k8sT/Nightly.go
+++ b/test/k8sT/Nightly.go
@@ -49,7 +49,7 @@ var _ = Describe("NightlyEpsMeasurement", func() {
        endpointsTimeout := endpointTimeout * time.Duration(endpointCount)
        manifestPath := "tmp.yaml"
        vagrantManifestPath := path.Join(helpers.BasePath, manifestPath)
-       var lastServer int
+       //var lastServer int
        var err error

        initialize := func() {
@@ -99,7 +99,7 @@ var _ = Describe("NightlyEpsMeasurement", func() {
        })

        deployEndpoints := func() {
-               _, lastServer, err = helpers.GenerateManifestForEndpoints(endpointCount, manifestPath)
+               _, _, err = helpers.GenerateManifestForEndpoints(endpointCount, manifestPath)
                ExpectWithOffset(1, err).Should(BeNil(), "Manifest cannot be created correctly")
                res := kubectl.Apply(vagrantManifestPath)
                res.ExpectSuccess("cannot apply eps manifest :%s", res.GetDebugMessage())
diff --git a/test/k8sT/demos.go b/test/k8sT/demos.go
index cfa479049..9af501866 100644
--- a/test/k8sT/demos.go
+++ b/test/k8sT/demos.go
@@ -40,7 +40,7 @@ func getStarWarsResourceLink(file string) string {
 var _ = Describe(demoTestName, func() {

        var (
-               demoPath   string
+               //demoPath   string
                once       sync.Once
                kubectl    *helpers.Kubectl
                logger     *logrus.Entry
@@ -57,7 +57,7 @@ var _ = Describe(demoTestName, func() {
                kubectl = helpers.CreateKubectl(helpers.K8s1VMName(), logger)

                //Manifest paths
-               demoPath = kubectl.ManifestGet("demo.yaml")
+               //demoPath = kubectl.ManifestGet("demo.yaml")

                // TODO (ianvernon) - factor this code out into separate functions as it's
                // boilerplate for most K8s test setup.
@@ -125,6 +125,10 @@ var _ = Describe(demoTestName, func() {
                _, err = kubectl.WaitforPods(helpers.DefaultNamespace, fmt.Sprintf("-l %s", empireLabel), 300)
                Expect(err).Should(BeNil(), "Empire pods are not ready after timeout")

+               By("Applying xwing deployment")
+               res = kubectl.Apply(xwingYAMLLink)
+               res.ExpectSuccess("unable to apply %s: %s", xwingYAMLLink, res.CombineOutput())
+
                By("Getting xwing pod names")
                xwingPods, err := kubectl.GetPodNames(helpers.DefaultNamespace, allianceLabel)
                Expect(err).Should(BeNil())

I can post that patch as a PR into the v1.0 branch to stabilize it after this is merged. I'm going to merge this since the test failure does not affect the quality of the code that will actually be used in user environments.


GUACAMOLE-641: Add support for populating arbitrary parameter tokens from key vaults.

These changes add a new family of extensions, similar to "guacamole-auth-jdbc", which provide support for retrieval of secrets from key vaults: "guacamole-auth-vault". Initial support for Azure Key Vault is present through the "guacamole-auth-vault-azure" module, with the necessary structure in place to allow other implementations to be provided in the future.
The general support works as follows:

A JSON file is included within GUACAMOLE_HOME which defines a token/secret mapping. Besides defining tokens, the names of each secret may also contain tokens which allow the secret name to vary by connection ID, hostname, username, etc. There is a specific set of tokens available for use within secret names.
When a user attempts to connect to a connection or connection group, the key vault implementation is queried to retrieve each defined secret. Tokens are then defined for only those secrets whose names are fully defined within the current context and which have values stored within the vault.
The extension defining the connection is given these tokens. If the extension supports substitution of parameter tokens, as all current extensions do, those tokens will be substituted within the configuration of the connection. Guacamole extensions are not required to do this, however.
To avoid excessive retrieval requests, the result of retrieving a particular secret is cached (by default for 10 seconds).

This thus allows secret values like passwords and private keys to be stored off-site within the vault and retrieved dynamically based on context.

OK - I believe I've resolved all feedback thus far, as well as brought things up to date. I also:

Updated to the latest versions of the libraries.
Switched from JSON to YAML for the token/secret mapping file.
Renamed from guacamole-auth-vault-* to guacamole-vault-* (since it's not providing any kind of auth).
Renamed the only other non-auth extension (guacamole-auth-quickconnect) following the same logic.
Leveraged getPrivileged() for parameter retrieval, so that it's more likely that we'll be able to vary things based on connection hostname and connection username.

BUT, I've also converted this to draft because of difficulties in obtaining the license text from the "azure-annotations" dependency. To be clear, the pom.xml for that dependency clearly states that the library is MIT-licensed, as does the source for that library, however the full text of the license file can't currently be obtained because the corresponding git repository is private (presumably by mistake). See:
https://github.com/apache/guacamole-client/blob/54150829d3a60137582a912adb982799c39764b5/doc/licenses/azure-annotations-1.10.0/License.txt#L1-L28
As noted in the placeholder above, I've already emailed Microsoft about this, and they are tracking things down, but are unsure whether this will be resolved soon due to the holidays. I've asked whether they can send us the License.txt file manually for the time being so that we can move forward. We shall see...


macOS: switch to png for image

Used png for handling images on all platform. On macOS dropped the dependency on core-graphics since now an image is directly stored as png

Hey there, thanks for the PR. Similar to the first PR which tried this, can I ask that you do some basic compatibility testing to make sure this doesn't regress any major use cases? Some examples would be:

Copying image to and from Firefox, Chrome, and Safari
Copying image to and from Pages and/or Word
Copying image to and from TextEdit

Unfortunately at the moment I don't have access to an Apple device.


Issue with Scroll bar

Hi Isaac Mann,
How do i add scroll bar for context menu. i am  tried to use cdkScrollable ,but it is not working.
Could you please help me out this issue.
Attached the screen shot for your reference.

Try something like this:
.ngx-contextmenu > ul {
  overflow: auto;
  max-height: 200px;
}

Try something like this:
.ngx-contextmenu > ul {
  overflow: auto;
  max-height: 200px;
}

sub menu...
context-menu-content  ul {
    max-height: 500px;
    overflow: auto;
}


onReady doc duplicate of isReady

Subject of the issue
The function documentation for onReady is a duplicate of isReady and is not exactly appropriate.
Your environment

Not applicable

Steps to reproduce
https://github.com/angular-translate/angular-translate/blob/master/src/service/translate.js#L2255
Expected behaviour
Updated doc is provided.
Actual behaviour

Not applicable

Ehm.. $translate.isReady() returns a boolean, $translate.onReady(fn) consumes a callback and returns a promise.
Which one these behaviors make you feel these are duplicates of each other?

onReady does not "Returns whether the service is "ready" to translate (i.e. loading 1st language)."

Gotcha.
Thank you for clarification.


Demographics FHIR server Prio 2

After release we should resolve specific things into the demographics or foward it.
Like practioner etc. if we want to have them back convertable as resources, otherwise use identifiers ?!

see for example assigner in identifier.
We have one in openEHR but its sadly a string.
We could nevertthelss add a link via links or so.


Unable to renew cert after auto renew failed

Hello,
I had a renewal which failed. Tried a manual renewall using win-acme.v<IP_ADDRESS> which also failed.
Downloaded verson win-acme.v<IP_ADDRESS> and tried to create a new cert in test mode which resulted with some errors. Your help would be much appreciated.
[VERB] Checking [Manual] www.5220dessources.com
[DBUG] Loading signer from C:\ProgramData\win-acme\acme-staging-v02.api.letsencrypt.org\Signer_v2
[DBUG] Send GET request to https://acme-staging-v02.api.letsencrypt.org/directory
[DBUG] Send HEAD request to https://acme-staging-v02.api.letsencrypt.org/acme/new-nonce
[DBUG] Loading account information from C:\ProgramData\win-acme\acme-staging-v02.api.letsencrypt.org\Registration_v2
[DBUG] Send POST request to https://acme-staging-v02.api.letsencrypt.org/acme/new-order
[DBUG] Send GET request to https://acme-staging-v02.api.letsencrypt.org/acme/authz/tqwtgkK_iQUPW55yw3oahlh8hRHqLzwu18EgyjRSb8w
[INFO] Authorize identifier: www.5220dessources.com
[INFO] Authorizing www.5220dessources.com using http-01 validation (SelfHosting)
[DBUG] Submitting challenge answer
[DBUG] Send POST request to https://acme-staging-v02.api.letsencrypt.org/acme/challenge/tqwtgkK_iQUPW55yw3oahlh8hRHqLzwu18EgyjRSb8w/286722919
[DBUG] Refreshing authorization
[DBUG] Send GET request to https://acme-staging-v02.api.letsencrypt.org/acme/challenge/tqwtgkK_iQUPW55yw3oahlh8hRHqLzwu18EgyjRSb8w/286722919
[EROR] {
"type": "urn:ietf:params:acme:error:unauthorized",
"detail": "Invalid response from https://www.5220dessources.com/.well-known/acme-challenge/l9GCa2hpbPpRvqHffW4X9OQZuGKly1Pkm9L5Nt1uRsU [<IP_ADDRESS>]: "\r\n<html xmlns=\"http"",
"status": 403
[EROR] Authorization result: invalid
[EROR] Create certificate failed: Authorization failed

Hi,
Not an expert at all in certs, but I would check to see if authorization is on at your domain:80.  I think Let's Encrypt is sending an http get too port 80 as an anonymous user, when confirming that you control the domain.

I think your problem can be solved by going though https://github.com/PKISharp/win-acme/wiki/Validation-issues

Thanks for your reply
"Enable anonymous authentication to the /.well-known/acme-challenge/ path to allow access from the ACME server."
Where do i find this path?

@kiekar that path is in your IIS website folder.
If you have your site located at
c:\inetpub\wwwroot\5220dessources.com then that folder would typically reside at c:\inetpub\wwwroot\5220dessources.com\.well-known\acme-challenge\

make sure that path is accessible via a browser (anonymous access)
the folder may be hidden in Windows

Actually since you're using SelfHosting validation that path probably doesn't exist physically. But that 403 error page is not being returned by our program, which means that the requests to port 80 end up somewhere else. I'm suspecting some network-level shenanigans like a reverse proxy.

c:\inetpub\wwwroot\5220dessources.com.well-known\acme-challenge\

The website is located on a different partition. I added the path in the website root directory but still unable to create a cert

Have you managed to solve the problem? The domain seems to have a current certificate, so I assume yes?

I tried several times to solve the problems with no luck so I decided to switch from win-acme to certify the web app and all worked fine afterwards.
Thanks


{Upgrade} az upgrade: Fix grammar error: updates -> update(s)

Related command
az upgrade
Description
Fix grammar error found by https://github.com/Azure/azure-cli/issues/23470: updates -> update(s)

Upgrade


fix several Stitcher property set methods that called wrong NativeMethod...

It's terrible :weary:
Thanks!!


[Environment] Should add an option to remove 'under modification' Environments.

Description: If the during the process of environment creation user will face with some unfavourable situations, like an SS dropped down or the light went out, he can observe 'under modification' Environment. And the user can't remove it.
Improvement: should add in option to remove 'under modification' Environments in an hour for example, SS will check if the Environment under modification more then hour, the user get an option to remove it.

Destroy will cancel active workflow and destroy environment


weak

Describe the bug
Your project uses a large number of weak cryptographic algorithms. According to NIST recommendations, it is recommended to use cryptographic algorithms with a strength of 128 bits or more. Weak hash algorithms like MD5 are easy to find malicious collisions, while RC4 is easy to crack
Attachments
If applicable, please attach any files that caused problems or log files generated by the software.
Environment (please complete the following information):

OS: [e.g. macOS 10.14.2]
Java Version: [e.g. 11.0]
Ghidra Version: [e.g. 10.1.4]
Ghidra Origin: [e.g. official GitHub distro, third party distro, locally built]

Additional context
Add any other context about the problem here.
vul.json
The specific vulnerabilities and potential fixes are in the attached json file.
I strongly recommend that you conduct an in-depth analysis of these issues and take remedial measures as soon as possible after confirmation. If you require more detailed information or would like my assistance in reviewing a fix, please feel free to contact me.
We want to apply for CVE/CVND. Looking forward to hearing from you and can communicate further if you would like me to provide more details.

All of files you mentioned are Ghidra/Features/Base/data/typeinfo/golang/runtimesnapshot.go.
This project doesn't call them at all.
% curl -s -L https://github.com/NationalSecurityAgency/ghidra/files/14738483/vul.json | jq -r '.[] | .Slicing_Criteria.SourceFilename' | sort | uniq -c
     27 /data1/zyx/go/src/verify_dir/github.com/Nationalsecurityagency/ghidra/Ghidra/Features/Base/data/typeinfo/golang/runtimesnapshot.go

This is rather out-of-date relative to the 11.x release which now uses Java 17 and has made quite a few changes.  Any chance you could update this to reflect the latest Ghidra release.

Could you please indicate the procedures you used to generate the json file and how you consider the listed items as exploitable.  e.g., the presence of MD5 support itself does not consitute a security weakness

As @topia stated in their comment, the file that is listed in your report is not part of Ghidra, but is instead an 'example' binary used to extract type information from the golang runtime.
I'm marking this issue as closed.


README not up-to-date

The README still mentions M6 but it seems like the code has been updated to support RC3?

... Didn't read the part below:

If you are running the latest version of IDEA itself, or you only ever run tests from the command line, it's safe to upgrade this plugin to the actual most recent version, 1.0.0-RC3.


Update virtual-machines-windows-sql-server-iaas-faq.md

Clarify an important licensing question

@luisherring,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@luisherring : Thanks for your contribution to the Azure documentation! The author, @v-shysun, has been notified to review your proposed change.

@rothja - Please assist with the review of these proposed changes. Thanks

Hi @luisherring,
Once the changes suggested by @rothja are made, this PR can be approved. Also, please resolve the merge conflict.
Thanks,

@luisherring Hey. Because of the merge conflicts, I'd be willing to make this change for you. Or you could open a new PR with the change request. Sorry again for the delay. If you get access to the private repo, you can make small changes like this without approval.

@rothja Hi Jason, you should be able to pull this branch over to your local private repo to resolve the conflict and make the necessary edits. Try this:

Update your master branch, then create a new working branch.
In the new working branch, run:
git pull https://github.com/luisherring/azure-docs.git patch-2
This will pull Luis' patch-2 branch to your working branch and cause the conflict to occur locally for you.
Resolve the conflict, add, commit, and push to your fork. Then, create a pull request in the private repo. It should automerge. This allows you to merge the contribution and implement your feedback.
Come back to this PR and type the pound-sign-please-close comment to close it.

If this doesn't work, ping me. You might have to set the public repo as a remote for your private repo, but I don't think

@tysonn I'll try this. I'm making updating in azure-docs-pr right now to the same FAQ, so I'm going to let those changes publish first to have only one round of merge conflict resolutions.

I'm working on the FAQ in another PR (https://github.com/MicrosoftDocs/azure-docs-pr/pull/24681), and I made this change there. Luis is a Microsoft PM, so I think this should be OK with him. Please close this PR. Thanks.

@v-shils @GitHubber17 This PR can be closed, because the change has been made in a separate FAQ updated. Thanks!

#please-close


[FR]: easy way to choose the device from the list (device tracker)

Hi,
With all happines of using this integration, there is a small topic to consider.
Task: choose 3 devices you would like to track
Way to do it (today): open the list and ... find them.
Problem: if you have many devices, it take ages because:
1- list is not sorted - neither by MAC nor by hostname
2- with example of my network where I have 200+ devices it takes ages to find the one, especially if the hostnames sometimes are very similar and MAC are randomly displayed on the list - see below.
Question: is there any chance to have this taks improved a bit?  i.e.:

sort by MAC or
sort by hostname or
start finding what you are looking for by typing so that maching records would be left on the list

thank you

Interestingly, the HA selector to display these does not support searching. I wish it did as this is not the first use case where this would be VERY helpful. Maybe someday I'll try to submit a PR to HA to improve the selector.

I can sort the list. Do you think it would make the most sense to sort by IP address or Hostname?
I'd rearrange the display of the items as well maybe leaving off the mac altogether.

<IP_ADDRESS> - cam2front.local
cam2front.local - <IP_ADDRESS>

I would say: if sorted by hostname it would solve 90% the problem
And if searchable it would be 100% (together with sorted hostname)
So yes, this one:
cam2front.local - <IP_ADDRESS>
looks better for me
thank you

So thinking about this a little more, I'm going to update my thoughts. In looking at my device tracker list, there are some entries with hostnames and some without. Additionally, in those without a hostname, maybe seeing the MAC address might be helpful. I still 100% agree that the selector needs to be sorted.

If we sort by hostname, where do the items without a hostname go?
If we list hostname first, what should it look like if there isn't a hostname?
I now think we should continue to show the MAC address, just not first

Options:
(I vote for option 6. Second choice option 2)
1. Hostname first, sorted by hostname, then IP:

appletv.home - <IP_ADDRESS> [aa:bb:cc:dd:ee:ff]
my-device.home - <IP_ADDRESS> [11:00:11:00:11:00]
<IP_ADDRESS> [00:11:22:33:44:55]
<IP_ADDRESS> [11:22:33:44:55:66]

2. Hostname first, sorted by hostname, then IP:

appletv.home (<IP_ADDRESS>) [aa:bb:cc:dd:ee:ff]
my-device.home (<IP_ADDRESS>) [11:00:11:00:11:00]
<IP_ADDRESS> [00:11:22:33:44:55]
<IP_ADDRESS> [11:22:33:44:55:66]

3. Hostname first, sorted by IP:

my-device.home - <IP_ADDRESS> [11:00:11:00:11:00]
<IP_ADDRESS> [00:11:22:33:44:55]
appletv.home - <IP_ADDRESS> [aa:bb:cc:dd:ee:ff]
<IP_ADDRESS> [11:22:33:44:55:66]

4. Hostname first, sorted by IP:

my-device.home (<IP_ADDRESS>) [11:00:11:00:11:00]
<IP_ADDRESS> [00:11:22:33:44:55]
appletv.home (<IP_ADDRESS>) [aa:bb:cc:dd:ee:ff]
<IP_ADDRESS> [11:22:33:44:55:66]

5. IP address first, sorted by IP:

<IP_ADDRESS> - my-device.home [11:00:11:00:11:00]
<IP_ADDRESS> [00:11:22:33:44:55]
<IP_ADDRESS> - appletv.home [aa:bb:cc:dd:ee:ff]
<IP_ADDRESS> [11:22:33:44:55:66]

6. IP address first, sorted by IP:

<IP_ADDRESS> (my-device.home) [11:00:11:00:11:00]
<IP_ADDRESS> [00:11:22:33:44:55]
<IP_ADDRESS> (appletv.home) [aa:bb:cc:dd:ee:ff]
<IP_ADDRESS> [11:22:33:44:55:66]

7. IP address first, sorted by hostname, then IP:

<IP_ADDRESS> - appletv.home [aa:bb:cc:dd:ee:ff]
<IP_ADDRESS> - my-device.home [11:00:11:00:11:00]
<IP_ADDRESS> [00:11:22:33:44:55]
<IP_ADDRESS> [11:22:33:44:55:66]

8. IP address first, sorted by hostname, then IP:

<IP_ADDRESS> (appletv.home) [aa:bb:cc:dd:ee:ff]
<IP_ADDRESS> (my-device.home) [11:00:11:00:11:00]
<IP_ADDRESS> [00:11:22:33:44:55]
<IP_ADDRESS> [11:22:33:44:55:66]

I'd go with 6.

6 is ok


typo: Applicances -> Appliances

@nschonni : Thanks for your contribution! The author, @jimdial, has been notified to review your proposed change.

Editorial fix confirmed.


Responsive Image only 250px width?

Describe the bug
On large mobile Devices image in a two column layout is only 250px wide. i can see the 250px on the  in the generated html output. why is it there? why isn't it 100%? where can i change it?
To Reproduce
Steps to reproduce the behavior:

Create a file with this MJML code: <mj-section padding="0">      <mj-column>        <mj-text mj-class="headline2" css-class="columns-two-text-image_title"> Content Bock #2 </mj-text>        <mj-text css-class="columns-two-text-image_text"> Lorem ipsum dolor sit amet, consectetuer adipiscing elit, sed diam nonummy nibh euismod tincidunt ut laoreet dolore magna aliquam erat volutpat. Ut wisi enim ad minim veniam, quis nostrud exerci tation ullamcorper suscipit lobortis nisl ut aliquip ex ea commodo consequat. </mj-text>      </mj-column>      <mj-column>        <mj-image src="http://go.powerschool.com/rs/861-RMI-846/images/HE Newsletter Winter Webinar v3.jpeg" alt="Peopleadmin" align="center" css-class="columns-two-text-image_image" />      </mj-column>    </mj-section>
See error

Expected behavior
100% wide like on desktop view
MJML environment (please complete the following information):

OS: Big Sur
MJML Version 4.8.1
MJML App 3.0.3

Screenshots

mj-image has a fluid-on-mobile attribute for this behavior. cf https://documentation.mjml.io/#mj-image it relies on a media query for this so support may vary.

wow was searching for something like "250px" and found nothing.

The size will depend on the column size to avoid overflow issue in Desktop Outlook. It's a simple column width - (left+right padding) - (left+right border)


Invalid Http Method Recognize

Problem occur will having convert curl to python

curl
curl -G endpoint
--data key="key"
--data url="url"

python
import requests
data = { 'key': key,  'url': url}
response = requests.post(endpoint, data=data)

-- Valid Response should be:
import requests
data = { 'key': key,  'url': url}
response = requests.get(endpoint, params=data)

I looked into this and it is converting properly now as GET. Please test when convenient. Closing this for now, but feel free to reopen if you still have this problem.


Addsupport for wifi adapters for older NUCs

[page-] undefined

[page-] This issue has attached support thread https://jel.ly.fish/#/e97cc2cd-e64a-4daf-b8bf-641a84024f6a


Support for Laravel 5.8 but for yajra/laravel-datatables package

Hi,
I know the version 9.0 supports Laravel 5.8 but what yajra/laravel-datatables
I have this in composer file:
"yajra/laravel-datatables": "^1.0",
"yajra/laravel-datatables-oracle": "~9.0"

and the first one is not working.

Using "yajra/laravel-datatables": "^1.5" should be enough for 5.8.

Using "yajra/laravel-datatables": "^1.5" should be enough for 5.8.

Installation completed on mine. Thanks 👌


Fix the indentation of scaffold controller spec

The controller specs generated by scaffold command have inconsistent indentation. Like this:
  describe "GET #show" do
    it "assigns the requested user as @user" do
      user = User.create! valid_attributes
          get :show, params: {id: user.to_param}, session: valid_session
          expect(assigns(:user)).to eq(user)
    end
  end

Seems reasonable to me, thanks :)


[BUG] mpplus main menu - dialog not found

That'd be probably an easy one to fix. Seems the tool dialog is not found. Should it have been installed automatically on musicplayerplus package install or on mppinit. If yes, I can investigate further and provide more details. For now, just reportiong the issue:
1) Launch mpcplus in Current         6) ASCIImatics Animations          11) Blissify Audio Info Extraction  16) View Music Service Logs         21) Toggle Fullscreen (OFF)
2) Launch mpcplus with Kitty         7) Convert M4A files to MP3        12) Essentia Audio Info Extraction  17) List/Search/Select Media        22) Quit
3) Launch mpcplus with Gnome         8) Convert WAV files to MP3        13) AcousticBrainz Info Retrieval   18) Select Music Directory
4) Launch mpcplus with Retro         9) Import Music to Beets           14) Manage Downloads                19) Toggle Show Cover Art (ON)
5) Launch mpcplus with Tilix        10) Fetch Lyrics                    15) Manage Services                 20) Toggle Use Tmux (ON)
Please enter your command choice (numeric or text, 'h' for help): 18
/usr/bin/mpplus: line 2745: dialog: command not found

Syncing MusicPlayerPlus music directory, please wait ...

MusicPlayerPlus music directory synchronized.

Installing the dialog package manually fixes the issue.

I've added dialog to the dependency list for all platforms. It's installed by default in most distributions but to cover all distributions it needs to be explicitly listed as a dependency. Good catch. This issue should be resolved in the next release.


Add ConcurrentExecution step

ML-6015
Based on the preexisting internal _ConcurrentJobExecution class. With support for three concurrency mechanisms: asyncio, threading, and multiprocessing.

Do we want to address this issue within the current pull request?
To make mlrun context serializable isn't something we can do in storey anyway.

It seems that the 'pass_context' flag may not be the best solution for the problem
Why is it the flag's fault? :) Do you have some change in mind?

Do we want to address this issue within the current pull request?

To make mlrun context serializable isn't something we can do in storey anyway.

It seems that the 'pass_context' flag may not be the best solution for the problem

Why is it the flag's fault? :) Do you have some change in mind?

Can you check if indeed https://github.com/uqfoundation/dill can serialize lambdas?

@assaf758, I don't believe multiprocessing allows for replacing pickle with dill or anything else. Do you have a solution in mind?

@assaf758, it can, but it's not obvious how to use that to solve the problem. I'll try to find a way.

It turns out that mlrun GraphContext contains the entire graph, which even dill fails to serialize. Ironically, it's there for ParallelRun.


The type of render function

As I see the prototype of render function is next:
function render(node:JSX.Element, parent:Element, mergeWith?:Element):Element;

In react I render the root component (i.e. starting from html node) using the document object as container (i.e. parent) node and it works fine for this case. I tried to do same using preact and it also works as expected.
Actually I think the prototype of render function must be like that:
function render(node:JSX.Element, parent:Element|Document, mergeWith?:Element):Element;

@katyo makes sense! Happy to accept a PR for this if you have the time.

Closing this, as this has been merged a while ago.


multi select for history (ctrl+r) puts timestamps in front of all but the first command

Before proceeding...

[ X] My CLIs meet the minimum version requirements
[ X] I have read through the troubleshooting guide
[ X] If I updated fzf.fish for the first time in over 6 months, I read the migration guides to make sure I'm caught up

Describe the bug
ctrl+r and then tab to select multiple entries puts the timestamp in front of the entries, like:

I imagine this is repeatable behavior for the package, but I'm happy to help debug of course if it's somehow only my setup. I'm also looking at the code now and trying to see if I could submit a pull request.
Environment
Versions installed:
bat 0.21.0 (405e5f74)
fd 8.4.0
fzf 0.31.0 (a0ef898)
fish, version 3.4.1

terminal: konsole
OS: Linux 5.18.14

Thanks much for putting together something so well thought out.

Ah yeah, I have a pull request, just needed to change a few lines. Here is the code snippet:
       set -U commands_selected # empty list to loop over selected commands, in case multiple                                                 for c in (string split \n $command_with_ts)                                                                                set --append commands_selected (string split --max 1 " │ " $c)[2]                                                                  end                                                                                commandline --replace -- $commands_selected
Incoming!

Hi @acegallagher, the search history feature does not multi-select by default, so you must've configured it to be multi-selectable. What's your use case for selecting multiple commands?


Added nullbyte tests for coreidle and fpgaconf

Broke up commits from PR #1000

Coverage increased (+0.002%) to 80.352% when pulling 748c3177206d529d0c11977da963d3a48510a6d6 on jelonanx/nullbyte-tests into 12f5a0911ff5645b3963791eaeee7f4a8f304e03 on master.


Invalid Date

I just downloaded vue-good-table this morning, basically I just copied everything just to see how it works. But on the date side (from the example, the "joined" field) I always get "Invalid Date".
And again for the record, basically I just install and run it. But the date always get "Invalid Date". And I tried changing the format, but still got the same thing.
Is there anything Im doing wrong?

hey @yudhiezk, thanks for trying out vue-good-table. It is difficult to say what is going on without seeing how you are using it. What does your code look like? What is the format you're using? What version of the plugin are you using?
I created the issue template so that I don't have to waste people's time asking the same questions... have you taken a look at one of the examples from the recipe page?
https://jsfiddle.net/aks9800/57a2a4ce/

Thanks, @yudhiezk


SP4: Support LineStyle in DataLogger and DataStreamer

Purpose:

Adds test for DataLogger LineStyle
Fixes bug 2972 - SP4: DataLogger should support LineStyle

Notes:

Used IsDifferentThan in the test assert to detect change in LineStyle, but it doesn't feel like the most precise way of testing this - open to suggestions here

Thanks @Gray-lab, this looks great! 🚀

Oh yeah, bonus points for the test that demonstrate this fix too! 🚀 ✨


Add experimental notice for migration

Purpose
$Subject
Fixes:
Examples
Checklist

[ ] Linked to an issue
[ ] Updated the specification
[ ] Updated the changelog
[ ] Added tests

@kalaiyarasiganeshalingam Are you change googlesheets in the separate PR?

@kalaiyarasiganeshalingam Are you change googlesheets in the separate PR?

Yes, https://github.com/ballerina-platform/persist-tools/pull/219


Update volumes section

Issue number: #15

I assigned #15 to myself because it needs to be modified a little differently.

@SamMacy This repository need a little different spin on how to do volumes.   I made the fix in https://github.com/Senzing/docker-db2-driver-installer/pull/17


Add nowait kvp removal form

Allows for one-shot removal calls that skip the unnecessary busy-wait.

verified to work
/lgtm

/cherry-pick v0.7


Custom Stream with async read

** Enhancement **
Right now, custom streams work as any streams can be converted to IRandomAceessStream. It can then be utilized via the library given that the underlying streams are seekable. But in the scenario where the streams readasync/read function has to wait for a few second to cache it up the UI seems to be freezed.
Sample
`
public override async Task ReadAsync(byte[] buffer, int offset, int count, CancellationToken cancellationToken)
{
var startOffset = _start + offset + Position;
int piece = (int)(startOffset / _piece);
if (_chunks.Contains(piece) == false)
{
    int pieceEnd = (int)((startOffset + offset + Position + 10*1024*1024)/_piece);
    pieceEnd = Math.Max(piece + 1, pieceEnd);
    _handler.FocusPieces(piece, pieceEnd, _lastPiece);
    while (_handler.HasPiece(piece) == false)
    {
        await Task.Delay(2000, cancellationToken).ConfigureAwait(false);
    }
    _chunks.Add(piece);
}
return await _fileStream.ReadAsync(buffer, offset, count, cancellationToken);

}
`
This is the actual code snippet from the implementation where the handler downloads and saves to file via chunks. While checking for piece, I have used a task.delay. The outsider functions HasPiece and FocusPieces are almost instant. This behaviour can also be observed removing the Read piece code and just introducing a delay before returning _fileStream.ReadAsync()
I have also tried with overriding stream.read() but that's also freezes the UI.
I am assuming this is the part which actually reads data from stream.
HRESULT hr = mss->fileStreamData->Read(buf, bufSize, &bytesRead);
Expected behaviour:
The main UI shouldn't freeze on streams which takes time to return. If mss is bypassed via the original new MediaPlaybackItem(MediaSource.CreateFromStream()), the app seems to work as it should be honouring the async delays.
Possible implementation:
https://ffmpeg.org/doxygen/trunk/async_8c_source.html#l00386
I would assume this might be complicated to implement. In that case, we can mark this as solved.

Hi,
Are you sure the problem isn't caused by the stream's custom implementation marshalling the read in the UI thread?
Our internal reads are done on background threads on which the MF callbacks into MSS are marshaled. It's very unlikely this will end up in the UI thread.
Also if the stream is written in c# you'll run into performance issues caused by GC freezes.
A full runnable sample might be easier to understand.
Thanks

You are absolutely right. The thread issue was mine. There was an info dispatcherque along with the reader implemented. Thanks a lot.
To be noted, not sure of the performance impact but the stream implemented in the C# works pretty well. No garbage freeze at all. The custom stream is nothing but a filestream with a lock to check if it has been successfully downloaded.
Detail (Not important)
I had a dispatcherque which shows progress and stat info whenever there was a buffer. As I found out, I couldn't simultaneously lock both ReaderAsync and Infos calling like mss.CurrentVideoStream.HardwareDecoderStatus both on the same thread. (Bad design)
Detail End
Again, thanks a lot for the fast response. As it was working on MediaSource, what I thought was wrong.

As long as you just pass the read operation to the base Stream implementation, you should be fine with the gc, the problem comes when you read actual bytes in c#, using byte arrays that you allocate with the new operator.
In the end you'll probably be able to intercept the read calls on the Stream that are coming from ffmpeginteropx, do not attempt to keep references to the byte arrays being passed there, as we recycle them and your data will be dirty.


How about use 'mvn versions' to update version ?

Why you need it?
https://github.com/alibaba/fescar/blob/40ea4eb9276eeeca17f9caecdea5ffe0b49ce292/changeVersion.sh#L4
I think replacement is not a very good method
How it could be?
What about use mvn versions:set -DnewVersion=NEW_VERSION

now it has been supported by another way.
I will close the issue, if you have other questions, please reopen it or create a new one


Defect: Lanes moves to top

When assigning or adding a label, the lanes moves to the top again.

Housekeeping

[ ] model updated
[ ] tests added

[ ] tests green

[ ] coding style checked
[ ] doku checked

Seems to be fixed now :)


checks.yml repo linter requires token

Issue Report
module-name: checks.yml repo linter requires token
Expected behavior
To be able to run the repo-scaffolder setup procedure for existing repos and have the checks work.
Actual behavior
The checks.yml workflow is set to run on pull requests. It then runs the newrelic/repolinter-action@v1 action with the output_type configuration set to issue instead of exit-code. The repolinter-action configuration requires a token value to be set with a valid Github Personal access token.
Steps to reproduce the behavior
Run the steps on an existing repo and and the checks won't pass. In my case the error message was:
Error: A fatal error was thrown.
Error: Octokit API call failed. This may be due to your token permissions or an issue with the GitHub API. If the error persists, feel free to open an issue.
Error: GET https://api.github.com/repos/XXXXX/XXXX/issues?creator=app%2Fgithub-actions&labels=cms-oss-tier4&state=all&sort=created&direction=desc returned status 403

I filed an issue instead of a PR as I was unsure of the preferred solution. Possible solutions include:

the output_type could be set to exit-code
add more steps to the setup procedure to create a token, add it to the secrets with the appropriate name

Closing, resolved in #130


Hide "publication overview" sidebar on the left when in split view, and when unused

Description of UX crash
The left sidebar now remains unused, but takes up a lot of space. Especially for librarians this is very annoying because they need to use their screens in split view in order to work efficiently.
Notes
"Deze view is nog steeds erg irritant. Daardoor moet ik heel vaak wisselen tussen groot scherm en half scherm"
"Ik kan mijn werk wel nog doen, maar het is heel irritant"
Screenshots

Desired result
Suggestion A [desired for now as the sidebar is unused]:

We hide it on the publications and datasets overview

Suggestion B:

We hide the sidebar when in split view in the publications and datasets overview

Suggestion C:

We make it collapsable

Fix with responsive design


Wait for interactive thread to finish clean up

I have not found a simple way to make the main thread wait for the timer to finish. As a result, the final cleanup step has been running after the REPL prints a new line, so the final frame isn't cleared.
Can anything like fetch or wait from Distributed.jl be made to work here? Otherwise, the signaling mechanism might have to be modified to allow reply signals from the interactive thread.

For now, I've just added sleep(0.5) in the macro after the close signal is sent.

The docs say wait or fetch should work. I'll experiment some more with the timer and spinner status.

The functiontimer_spin needs to wait until the timer is closed before returning, but I believe wait(Timer()) just wakes up the thread everytime the Timer elapses.

The next PR should fix this issue. Instead of waiting for the timer to be closed, timer_spin just waits until the spinner status is closed.
Additionally, I removed the const designation from the signaling function, since this caused issues with repeated calls.


Dr Elephand is not starting

Hi Guys, we have been trying to install Dr. Elephant and after a lot of troubleshooting we are stuck in this part:
[root@ip-172-31-37-252 bin]# ./start.sh
Using config dir: /usr/dr-elephant/app-conf
Using config file: /usr/dr-elephant/app-conf/elephant.conf
Reading from config file...
db_url:  localhost
db_name:  drelephant
db_user:  root
http port:  8081
This is hadoop2.x grid. Add Java library path: /lib/native
Starting Dr. Elephant ....
Dr. Elephant started.
but the process never start and looking in the logs we found this:
2016-04-22 20:34:50,374 - [ERROR] - from com.jolbox.bonecp.hooks.AbstractConnectionHook in main
Failed to obtain initial connection Sleeping for 0ms and trying again. Attempts left: 0. Exception: java.net.ConnectException: Conn
ection refused.Message:Communications link failure
The last packet sent successfully to the server was 0 milliseconds ago. The driver has not received any packets from the server.
/hadoop/dr-elephant-2.0.3-SNAPSHOT/logs
please let us know if somebody has been experience this issue or have any idea.

@alanheli92
there're something to be sure : mysql version 5.6.4+ which declared in Administrator Guide ;  env.  all set ;  elephant.conf is ready.

@canghaiyisu666, the mysql requirement has been changed to 5.5+. I had updated this in the developer docs but forgot to do so in the Administrator guide.
@alanheli92, there is a thread open in the mailing list on the same issue reported by Gabriel García. You might want to follow it up there.

The root cause of this problem is conflicting akka dependencies. If you have spark 1.5.0 or above, you need to exclude the akka dependencies.

Thanks, finally I could start the server but when I tried to connect with the web browser the dr.log displays this error message:
tail: dr.log: file truncated
Java HotSpot(TM) 64-Bit Server VM warning: ignoring option MaxPermSize=256m; support was removed in 8.0
Play server process ID is 9122
SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in [jar:file:/hadoop/dr-elephant-2.0.3-SNAPSHOT/lib/ch.qos.logback.logback-classic-1.0.13.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:/hadoop/dr-elephant-2.0.3-SNAPSHOT/lib/org.slf4j.slf4j-log4j12-1.7.10.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:/usr/hdp/<IP_ADDRESS>-3485/hadoop/lib/slf4j-log4j12-1.7.10.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
SLF4J: Actual binding is of type [ch.qos.logback.classic.util.ContextSelectorStaticBinder]
[info] play - database [default] connected at jdbc:mysql://localhost/drelephant?characterEncoding=UTF-8
[info] application - Starting Application...
[info] play - Application started (Prod)
[info] play - Listening for HTTP on /0:0:0:0:0:0:0:0:8082
[error] play - Cannot invoke the action, eventually got an error: javax.persistence.PersistenceException: Query threw SQLException:Table 'drelephant.yarn_app_result' doesn't exist
Bind values:[Tue Apr 26 15:35:05 UTC 2016]
Query was:
select count(*) from yarn_app_result t0 where t0.finish_time > ?
[error] application -
! @6ppoc7dnn - Internal server error, for (GET) [/] ->
play.api.Application$$anon$1: Execution exception[[PersistenceException: Query threw SQLException:Table 'drelephant.yarn_app_result' doesn't exist
Bind values:[Tue Apr 26 15:35:05 UTC 2016]
Query was:
select count(*) from yarn_app_result t0 where t0.finish_time > ?
]]
at play.api.Application$class.handleError(Application.scala:293) ~[com.typesafe.play.play_2.10-2.2.2.jar:2.2.2]
at play.api.DefaultApplication.handleError(Application.scala:399) [com.typesafe.play.play_2.10-2.2.2.jar:2.2.2]
at play.core.server.netty.PlayDefaultUpstreamHandler$$anonfun$2$$anonfun$applyOrElse$3.apply(PlayDefaultUpstreamHandler.scala:261) [com.typesafe.play.play_2.10-2.2.2.jar:2.2.2]
at play.core.server.netty.PlayDefaultUpstreamHandler$$anonfun$2$$anonfun$applyOrElse$3.apply(PlayDefaultUpstreamHandler.scala:261) [com.typesafe.play.play_2.10-2.2.2.jar:2.2.2]
at scala.Option.map(Option.scala:145) [org.scala-lang.scala-library-2.10.4.jar:na]
at play.core.server.netty.PlayDefaultUpstreamHandler$$anonfun$2.applyOrElse(PlayDefaultUpstreamHandler.scala:261) [com.typesafe.play.play_2.10-2.2.2.jar:2.2.2]
Caused by: javax.persistence.PersistenceException: Query threw SQLException:Table 'drelephant.yarn_app_result' doesn't exist
Bind values:[Tue Apr 26 15:35:05 UTC 2016]
Query was:
select count(*) from yarn_app_result t0 where t0.finish_time > ?
    at com.avaje.ebeaninternal.server.query.CQuery.createPersistenceException(CQuery.java:790) ~[org.avaje.ebeanorm.avaje-ebeanorm-3.2.2.jar:na]
    at com.avaje.ebeaninternal.server.query.CQueryEngine.findRowCount(CQueryEngine.java:110) ~[org.avaje.ebeanorm.avaje-ebeanorm-3.2.2.jar:na]
    at com.avaje.ebeaninternal.server.query.DefaultOrmQueryEngine.findRowCount(DefaultOrmQueryEngine.java:36) ~[org.avaje.ebeanorm.avaje-ebeanorm-3.2.2.jar:na]
    at com.avaje.ebeaninternal.server.core.OrmQueryRequest.findRowCount(OrmQueryRequest.java:235) ~[org.avaje.ebeanorm.avaje-ebeanorm-3.2.2.jar:na]
    at com.avaje.ebeaninternal.server.core.DefaultServer.findRowCountWithCopy(DefaultServer.java:1301) ~[org.avaje.ebeanorm.avaje-ebeanorm-3.2.2.jar:na]
    at com.avaje.ebeaninternal.server.core.DefaultServer.findRowCount(DefaultServer.java:1293) ~[org.avaje.ebeanorm.avaje-ebeanorm-3.2.2.jar:na]

Caused by: com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: Table 'drelephant.yarn_app_result' doesn't exist
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) ~[na:1.8.0_60]
at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62) ~[na:1.8.0_60]
at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45) ~[na:1.8.0_60]
at java.lang.reflect.Constructor.newInstance(Constructor.java:422) ~[na:1.8.0_60]
at com.mysql.jdbc.Util.handleNewInstance(Util.java:400) ~[mysql.mysql-connector-java-5.1.36.jar:5.1.36]
at com.mysql.jdbc.Util.getInstance(Util.java:383) ~[mysql.mysql-connector-java-5.1.36.jar:5.1.36]
Any help would be appreciated

And the web browser displays:
Oops, an error occured
This exception has been logged with id 6ppof9ljo.
The id error changes when I reload the page

you need to configure your mysql. Make the table drelephant as specified in the docs.


Add "--organization-id" argument for GCP assessments

New feature motivation
For GCP, the easiest way to run prowler is in the Cloud Shell where you are already authenticated to Google Cloud and prowler will use those creds. But if that account has access to multiple organizations then prowler, by default, will scan all projects from all those organizations given no --project-id filter. But what if those organizations contain hundreds/thousands of projects and you want  to run prowler just for one of them? Then you will end up with a super long command line containing hundreds/thousands of project ids.
Solution Proposed
It would be useful to have an argument like --organization-id that prowler can use to then list & scan all projects in the given organization id (and accessible by the credentials used).
Describe alternatives you've considered
Getting a list of project ids for an organization with the following command and pass that list to prowler.
gcloud asset search-all-resources --asset-types="cloudresourcemanager.googleapis.com/Project" --scope=organizations/123456789 --format=json | jq -r '.[].additionalAttributes.projectId'

or with python:
from google.cloud.asset_v1 import AssetServiceClient, SearchAllResourcesRequest

def get_projects_under_org(org_id):
    client = AssetServiceClient()
    request = SearchAllResourcesRequest(
        scope = org_id,
        asset_types = ["cloudresourcemanager.googleapis.com/Project"],
        read_mask = "additionalAttributes"
    )
    response = client.search_all_resources(request=request)
    projects = []
    for item in response:
        projects.append(item.additional_attributes.get("projectId"))
    return projects

Additional context
No response

Hi @ar3diu, this is a great idea, thanks! However, Prowler now is retrieving all the projects that are accesible through the given credentials here. Would not that be the same as the approach you are mentioning?

@sergargar - given my scenario, if you wanted to run prowler only on projects in an organization you have access to, today you'd have to manually paste the list of project IDs from that organization. Otherwise, prowler will also consider the projects you have access to from other organizations.

Okay @ar3diu, I see what you mean and it makes sense!
We will create this flag, meanwhile, you can use the flag --list-project-ids to copy the projects you are interested in.

Again, hard to know which projects belong to which organization when --list-project-ids returns hundreds/thousands of ids. I used the code from my initial message to get project ids for a specific org id and then feed that list to prowler.

Hi @ar3diu, I have included the new flag in https://github.com/prowler-cloud/prowler/pull/5524, let me know if it works for you 😄

@sergargar - It only gets projects that are located directly under an organization id. But projects can also be located in folders which then can have sub-folders and so on. So you need a way to recursively check in all folders and subfolders for projects. I tried this before but I couldn't find a solution and that's why in my python code above I chose to use the Cloud Inventory.

Okay, it makes sense now! Thanks for the info, I will change it including this.

@ar3diu, I have made the changes to recursively check in all folders and subfolders within the organization, let me know if it supports your use case!

So, for some reason I get an authentication error when using --organization-id:
2024-10-28 07:39:41,695 [File: gcp_provider.py:103]     [Module: gcp_provider]   CRITICAL: No Project IDs can be accessed via Google Credentials.

2024-10-28 07:39:41,695 [File: provider.py:231]         [Module: provider]       CRITICAL: GCPNoAccesibleProjectsError[204]: GCPNoAccesibleProjectsError[3002]: No Project IDs can be accessed via Google Credentials.

I run prowler in Cloudshell and everything works well when using the other arguments.
I also tried to re-authenticate but without success.

@ar3diu make sure that the used credentials have the role Cloud Asset Viewer (roles/cloudasset.viewer) or Cloud Asset Owner (roles/cloudasset.owner) on the organization level. Cloudshell may have these roles by default.
I will add that information in the docs. Let me know if it works!

Yes, that seems to have been the problem.
It seems to work ok now. Thanks for implementing this so quickly :raised_hands:


Update docs

Coverage increased (+0.3%) to 92.956% when pulling a782c804893deabe1582571c41bbf51a53efd5be on update-docs into 0fc71e736ee11bede2012266a62f0b32b07858b9 on main.

Coverage increased (+0.3%) to 92.956% when pulling a782c804893deabe1582571c41bbf51a53efd5be on update-docs into 0fc71e736ee11bede2012266a62f0b32b07858b9 on main.


Data Lake reference missing

Where does the "3" Map to for the data Lake comment?

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

ID: 1c9ef03f-cf87-a333-623b-afbd39fd75a2
Version Independent ID: 7ab8460b-b303-c829-4c45-462be21b38b2
Content: Scalability and performance targets for standard storage accounts - Azure Storage
Content Source: articles/storage/common/scalability-targets-standard-account.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

@mikeblakeuk Thanks for reporting this. I have assigned the issue to the content author to investigate further and update the document as appropriate.

@mikeblakeuk - Thanks, I've removed the footnote.
#please-close


error message at the end of activation logs is shown

Description
aio app run, aio app logs --tail and aio runtime activation logs --tail all three use lib runtime to implement activation log tailing, and in each command, after each log is printed the following error message is also shown:
The log message can't be retrieved, key not found: log_message
Expected behavior
no error message should be shown
Reproduce

init firefly app with UI and action generic
Terminal 1: aio app run
Terminal 2: aio app logs --tail
Terminal 3: aio runtime activation logs --tail
Invoke action generic from the UI
Check logs in all three terminals

Update: this also happens with regular aio log commands such as aio app logs aio rt activation logs <id> BUT also: wsk activation logs <id> meaning the problem doesn't come the aio tooling but from I/O runtime

Do you have a recent example activation id we can look at?

Are the log lines in question very long?

Do you have a recent example activation id we can look at?

here you go 387119b89e8d4817b119b89e8d08172f

Are the log lines in question very long?

Yes, what is the limit ? I think I know where the problem is, in example our template action we are logging the response body, which depending on the template might be very long (all the runtime swagger spec..):  https://github.com/adobe/generator-aio-app/blob/master/generators/add-action/generic/index.js#L30-L38

Yeah removing that line solved the issue, thanks
I will open a PR against our templates to get rid of that.
Would still be good to get the max size if you know @mcdan ?

@moritzraho - it's around 10k.
No we will not change the error message, we are just going to remove the limit when we implement the new logging infrastructure. So, this should all go away in the next few weeks.

ok perfect thanks for the details 🙂


Show executed commands when in debug mode (-X)

It would be really handy for debugging purposes if the executed command lines (eg. dotnet build --flags.. etc) would be printed. To prevent spam during the build, perhaps only when run in debug mode (when Maven's -X flag is specified).

I agree, and it should be rather easy to implement.


Support typescript

None of the react-native hooks projects supports typescript. Please consider adding types.

Meanwhile would using this package work? https://github.com/react-native-community/hooks#usedeviceorientation
Adding types may take a while.

Hey @divyanshu013
Are you still looking to add type definition?
I can take it up and raise a PR.

Hey, sure, happy to accept PR. Alternatively, you can use the react-native-community package if that works.

Okay, react-native-community seems updated. Thanks for the heads up.


i met that when input some emoji  it will crash

i met that when input some emoji  it will crash and what can i do for it

Can you please post a new issue with more information? Please provide sample code, a stack trace, error output, etc.


do you have a plan to support es6?

do you have a plan to support es6?

I've found the es6-support branch. Do you plan to merge it to master?


html method adds automatically alt and title attributes if defined in…

… media file's meta.yaml file

Is this intended to add them if they are defined? or saying they are already added and you don't want them to be? Your title is not clear :)

This is a proposal for a new feature.
Currently alt and title tags have to be specified explicitly as parameter to the html call. But we also have the great meta.yaml feature were they can be specified in a companion file for each media file.
If you want to use the alt and title attributes specified in the meta.yaml file, then one has to assemble the img tag manually.
This PR would embed these attributes automatically should they be specified in the meta.yaml file and not specified as part of the html function call.

BTW for simplicity I propose:
        if (empty($attributes['title'])) {
            if (!empty($title)) {
                $attributes['title'] = $title;
            } elseif (!empty($this->items['title'])) {
                $attributes['title'] = $this->items['title'];
            }
        }

        if (empty($attributes['alt'])) {
            if (!empty($alt)) {
                $attributes['alt'] = $alt;
            } elseif (!empty($this->items['alt'])) {
                $attributes['alt'] = $this->items['alt'];
            }
        }

        if (empty($attributes['class'])) {
            if (!empty($class)) {
                $attributes['class'] = $class;
            } elseif (!empty($this->items['class'])) {
                $attributes['class'] = $this->items['class'];
            }
        }

Your re-factored version is much nicer and better to understand.

Fixed in develop with adjusted version. Thanks!

Would it make sense to even go a step further and add width and height as well?
This would be however much trickier as width/height is only known of the unmodified image. Once resize actions have been added, those attributes are invalid for the modified incarnation of the image.


Improve validation of field types in WAL

Since we moved the WAL out of the engine (https://github.com/influxdata/influxdb/pull/11364) the writing client is no longer protected by the error handling in the Cache:
https://github.com/influxdata/influxdb/blob/b9d5a7bec2529d7696afb016646f6eb0b4139c2c/tsdb/tsm1/cache.go#L74-L86
As such, the user will now see the following error if they have two series with different field types within a single batch:
edd@Alameda:~|⇒  influx write --bucket-id 0371a3adc1755001  @/tmp/writes.txt
Error: Failed to write data: http/handleWrite: incorrect value found in int64 slice: value.FloatValue.
Usage:
  influx write line protocol or @/path/to/points.txt [flags]

We should improve the error message by possibly using the same error message as the Cache. We can't remove the type checking from the cache because we need it for instances where a WAL is not in use.

@e-dard
Is it possible to add type checking to PointsToValues?
https://github.com/influxdata/influxdb/blob/b9d5a7bec2529d7696afb016646f6eb0b4139c2c/tsdb/tsm1/value.go#L53-L108

In the case of conflicting points in the same batch, I think it's going to be tricky to maintain the same semantics. It used to first validate all of the points against the index, then would write into the cache, and if any of those points were problematic, it would skip writing to the WAL (source: https://github.com/influxdata/influxdb/blob/1.8/tsdb/engine/tsm1/engine.go#L1348-L1356). That doesn't seem possible to maintain with the WAL first.
Given the above challenges, I propose that, as @zhulongcheng suggested, we report errors when converting into the values map, drop the conflicting points where first point sets the type for the rest of the points in the batch, and report a partial write error.
Since that's fairly small to implement, I'm going to start work on it and we can change directions if the proposal isn't sufficient.


Remove Jackson usage in more locations

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-ai-personalizer


format changes for flake8 linting

refactored code format to match flake8 specification

test suite will be fixed in following issue


Broken Orders (closed order)

NoMethodError in Orders#edit
Showing /app/app/views/orders/edit.html.erb where line #10 raised:
undefined method `category_id' for nil:NilClass
Extracted source (around line #10):
8
9
10
11
12
13
{
  id: id,
  category_id: item.category_id,
  item_id: item_id,
  quantity: quantity
}

Rails.root: /app
Application Trace | Framework Trace | Full Trace
app/models/order_detail.rb:10:in to_json' app/models/order.rb:47:in map'
app/models/order.rb:47:in to_json' app/views/orders/edit.html.erb:10:in block (2 levels) in _app_views_orders_edit_html_erb___2346109603271450612_69907163415280'
app/views/orders/edit.html.erb:6:in block in _app_views_orders_edit_html_erb___2346109603271450612_69907163415280' app/views/orders/edit.html.erb:3:in _app_views_orders_edit_html_erb___2346109603271450612_69907163415280'
lib/down_for_maintenance.rb:35:in `call'
Request
Parameters:
{"redirect_to"=>"orders",
"id"=>"5"}
Toggle session dump
Toggle env dump
Response
Headers:
None

We are scoping out 'deleted' items. Make sure we include them so lisa can see them

Resolved with #502


Added restaurant locations with dataset and Google Maps API

Got dataset about fast food restaurants from https://data.world/datafiniti/fast-food-restaurants-across-america, then used Google Maps API to display locations of these restaurants with markers.

Gotcha. Will definitely do next time!


How to compiler with the ARM cross-toolchain, such as x86_64_aarch64_arm_gnu compiler?

Dear devepolers:
I want to test the C++ code performance with gperftools on a ARMV8 platform such as ARM A53 and I have successfully tried a demo on a x64 linux OS. Thus the question is how can I compile the gperftools source code with x86_64_aarch64_arm_gnu?
Thanks very much and best regards.

Cross compiling works standard to any autoconf-based project. Just pass --host flag.

Dear alk,
I have tried the --host flag as following:
./configure --prefix=/home/projects/gperftools_setup --host=/home/projects/aarch_64/bin/aarch64-linux-gnu-g++
but it turned out to be invalid which throw an error: machine "aarch64-linux-gnu" not recognized.
Besides, is there any detailed document about the gperftools? Would you please share me a copy of that? Thanks.

Please consider reading standard instructions on cross compiling with autotools. There are plenty.
Standard way to use is something like ./configure --host=aarch64-linux-gnu
Put bin directory of your cross toolchain to PATH. And also make sure it works standalone. I.e. try to compile hello world or something like that to ensure it works.


Stop words

closes #111
I'm not sure I updated everything, can you tell me if I missed something?
Also we should not merge this until we release a new version of the tokenizer (look at the Cargo.toml I pushed)

Reopening the PR from another branch #128


{ border: 0; }

While doing some testing with this reset I ran into some issues where descendent elements were getting undesired borders. Adding * {   border: 0; }was useful for me.

@oste, great catch! Would you test this new version which includes several updates?

yep, looks good


Extended scenarios

Description
Created scenarios and appropriate behaviors with triggers based on most popular scenarios tested on a simple two lane road.
The behaviors are continuous, i.e. after termination the controls are not being reset to allow a seamless flow of controlled vehicle. These include:

DriveVehicleContinuous: Controlled vehicle will have the specified control values applied to itself with optional max_speed parameter to limit acceleration once desired speed is achieved.
DriveToLocationContinuous: Similar to DriveVehicleContinuous but with lateral control so that vehicle will point to target_location
FollowVehicleContinuous: Similar to DriveVehicleContinuous but with lateral control so that vehicle will steer towards target_vehicle

Triggers have been also updated to allow the scenarios:

TriggerOnLocation: check if a vehicle is within certain distance to a target location
TriggerOnStatusChange: check if a vehicle status value(s) have passed target value(s) (x, y, z, roll, pitch, yaw)

The scenarios included in this pull request consist of:

PassingFromOppositeDirections: Scenario used to calibrating and testing sensor models: ego vehicle passes other vehicle from opposite direction on a long straight.
OvertakingSlowTarget: Scenario used for testing a simple overtake maneuver. Ego car should overtake other vehicle with safe margin on a long straight.
FollowingAcceleratingTarget: Scenario used for testing whether ego car follows a slowly accelerating other vehicle correctly on a long straight.
FollowingDeceleratingTarget: Scenario used for testing whether ego car follows a slowly decelerating other vehicle correctly on a long straight.
FollowingChangingLanesTarget: Scenario used for testing a cut off situation where other vehicle is slowly changing lane to one occupied by ego.
DrivingOffDriveway: Scenario used for testing a situation where other vehicle is merging from covered driveway right in front of ego.
OncomingTargetDriftsOntoEgoLane: Scenario where oncoming other vehicle is slowly drifting from its lane onto one occupied by ego.

Almost all scenarios have different configuration based off TwoLaneRoad.xml, named with appropriate suffix: *Car, *Truck, *Motorcycle, *Bicycle. The updated list is available in Docs/list_of_scenarios.md
Where has this been tested?

Platform(s): Ubuntu 18.04.1 LTS
Python version(s): 2.7.15rc1, 3.6.7
Unreal Engine version(s): 4.21
CARLA version: 0.9.3

Possible Drawbacks
Hopefully none. "Extended scenarios" only extend the functionality.

This change is

I will create a new PR to development branch once everything is fixed


Changes to accessing Custom Data

I have recently updated ui-route from ver 0.2.15 to 0.3.1 and I noticed that accessing to custom data has been changed. With ver 0.2.15, I could use _.extend(newVar, $state.current.data) to initiate newVar using lodash. However, in ver 0.3.1, the properties of $state.current.data are in$state.current.data.__proto__. With this change, I have to do _.extend(newVar, $state.current.data.__proto__) to achieve the same result. Is this intended behavior?

While trying to create an example in plnkr, it turned out that the plnkr worked fine with just including the ui-router and angular. Then I debugged my code again, and noticed that in the debugger that $state.current.data was shown as 'extend', not object (see screen shot below).

Maybe something is wrong with my code, or some libraries are incompatible with ui-router. I will continue debugging my code. If you someone has seen similar problem and could provide some hints, it will be greatly appreciated.

Ok, I figured out the cause.
I have a state A.B.C which is abstract, with which I have the custom data associate. Also I define a controller for state A.B.C. Then I define non-abstract states such as A.B.C.1, A.B.C.2 etc. When A.B.C.1 is shown, it uses the controller associated with state A.B.C. The changes from ver 0.2.15 to 0.3.1 seems having  changed the way how the inherited custom data is packed. Instead of being an object, it is now an  'extend', which is created by the following code
	    // inherit 'data' from parent and override by own values (if any)
	    data: function(state) {
	      if (state.parent && state.parent.data) {
	        state.data = state.self.data = inherit(state.parent.data, state.data);
	      }
	      return state.data;
	    },

So to get around of this problem, I can do  _.extend(this, $state.$current.parent.data); and it works now.
However, I still question whether this is the intended behavior.

FYI, this is related to bug  #2631

Indeed. Thanks @paol

Closing as a duplicate since it seems duplicated, not just related

So to get around of this problem, I can do _.extend(this, $state.$current.parent.data); and in some places, I have to do _.extend(this, $state.$current.parent.parent.data); which is not pretty at all.

A prettier way to get the old behavior back is found at https://github.com/angular-ui/ui-router/issues/3045#issuecomment-249991317


Added 'countryCode' in response fields.

I just implemented it for Yahoo provider (this is the one I use for now).
I will add it for other providers during the week.
Tell me if you prefer to wait for other providers update before merging this pull request.

Nice! BTW it's better to update all providers :)

@jfsimon any news on this?

Sorry, I started... I need so many API keys! I'll try to finish this week.

I finally added countryCode field for all provider response.
Some services does not provide this information (Bing & Cloudmade), so it's set to NULL.

mmh unit tests?

Ping @jfsimon

Sorry for the delay :/

Mmmh thanks, but I have the following test suite output:

PHPUnit 3.6.10 by Sebastian Bergmann.

Configuration read from /Users/william/projects/Geocoder/phpunit.xml

...........................................EE.F...............S  63 / 108 ( 58%)
SSSSS.......................FF...............

Time: 29 seconds, Memory: 16.50Mb

There were 2 errors:

1) Geocoder\Tests\Provider\BingMapsProviderTest::testGetGeocodedDataWithLocalhost
Undefined index: countryCode

/Users/william/projects/Geocoder/tests/Geocoder/Tests/Provider/BingMapsProviderTest.php:91

2) Geocoder\Tests\Provider\BingMapsProviderTest::testGetGeocodedDataWithRealAddress
Undefined property: stdClass::$adminDistrict2

/Users/william/projects/Geocoder/src/Geocoder/Provider/BingMapsProvider.php:129
/Users/william/projects/Geocoder/src/Geocoder/Provider/BingMapsProvider.php:64
/Users/william/projects/Geocoder/tests/Geocoder/Tests/Provider/BingMapsProviderTest.php:101

--

There were 3 failures:

1) Geocoder\Tests\Provider\BingMapsProviderTest::testGetReversedDataWithRealCoordinates
Failed asserting that null matches expected 'FR'.

/Users/william/projects/Geocoder/tests/Geocoder/Tests/Provider/BingMapsProviderTest.php:167

2) Geocoder\Tests\Provider\OpenStreetMapsProviderTest::testGetGeocodedDataWithRealAddress
Failed asserting that null matches expected 60.453947176858.

/Users/william/projects/Geocoder/tests/Geocoder/Tests/Provider/OpenStreetMapsProviderTest.php:16

3) Geocoder\Tests\Provider\OpenStreetMapsProviderTest::testGetReversedDataWithRealCoordinates
Failed asserting that null matches expected 60.453947176858.

/Users/william/projects/Geocoder/tests/Geocoder/Tests/Provider/OpenStreetMapsProviderTest.php:62

FAILURES!
Tests: 108, Assertions: 698, Failures: 3, Errors: 2, Skipped: 6.

I launched tests on different servers, and I noticed that float assertions didnt work everywhere... so I fixed them as mentioned here: http://www.phpunit.de/manual/3.6/en/writing-tests-for-phpunit.html in example 4.24.
I fixed BingMapsProvider tests (I pushed wrong version oO).
No error for me with OpenStreetMapsProvider tests.
Results are null with GeoipProvider on geocoding an IP address, whatever it is... and the notice is: geoip_record_by_name(): Required database not available at /usr/share/GeoIP/GeoIPCity.dat. You can wget this file here: http://geolite.maxmind.com/download/geoip/database/GeoLiteCity.dat.gz

Thank you


Source depth

Reference issue
Implements #8512
What does this implement/fix?
Added function vertex_depth() to source_space.py to compute source depths for vertices.
Additional information
Used example from @wmvanvliet.

@wmvanvliet can you please have a loot at vertex_depths() and check that I've understood your example right. I also mention two problems in the documentation (importing SourceSpace and Forward properly, and an issue with the exception). Thanks!

Thanks @wmvanvliet. Now I can't figure out how to add vertex_depths to python_references.rst.

FYI @olafhauk my suggestions are often only approximate so they might or might not work... And also if you're on the "Files" view of the PR rather than the conversations one, you can click each suggested change in "add to batch" mode and make a single commit rather than several.

FYI @olafhauk my suggestions are often only approximate so they might or might not work... And also if you're on the "Files" view of the PR rather than the conversations one, you can click each suggested change in "add to batch" mode and make a single commit rather than several.

I guess I should have read this first... thx!

@wmvanvliet @larsoner I get ""TypeError: tuple indices must be integers or slices, not tuple" for _get_trans() (l. 3234) with testing dataset but not sample dataset.

@olafhauk see if my commits make sense to you

I'm ready for some criticisms.

ok so let's make trans optional. We agree here. I had not realized it was
not necessary if src from fwd or inv

agreed. Is it really a use case to pass info as None here?

agreed. Is it really a use case to pass info as None here?
…

I agree, also because distance to COG and distance to sensors are anticorrelated, which may be confusing.
What about
compute_distance_from_sensors()
compute_distance_from_centre()
?

what is the use for compute_distance_from_centre ?
keep the API with src not fwd. You can always pas fwd['src']

what is the use for compute_distance_from_centre ? keep the API with src not fwd. You can always pas fwd['src']
…

Distance from centre would provide a measure of depth purely based on anatomy, like distance from origin in a sphere. I for example would use it together with distance from sensors to see how they explain variation of resolution metrics. But I might be the only user.

@wmvanvliet @larsoner @agramfort I tried to add a test for the -trans option, but I must be misunderstanding something about transformations for source spaces and forward solutions, or about the testing dataset. I added some comments to test_compute_distance_to_sensors().
I also keep getting error messages from test_import_nesting, e.g. "Found un-nested import(s) for ["scipy submodules: ['special', 'constants', 'spatial']"]".

you should import these scipy modules within the functions to reduce import
time of mne

@olafhauk I pushed a fix
would it be useful to add this to an example?
I thinking of looking at this next to https://mne.tools/stable/generated/mne.sensitivity_map.html ?

@olafhauk I pushed a fix
would it be useful to add this to an example?
I thinking of looking at this next to https://mne.tools/stable/generated/mne.sensitivity_map.html ?

Correlating sensitivity with source depth? Good idea, I'll give it a try, but in a new PR?

I don't like much adding code in the public API without some example or narrative doc. These functions tend to not be discovered by users.

I don't like much adding code in the public API without some example or narrative doc. These functions tend to not be discovered by users.

Just when all my tests have passed I get more homework... ;-)
I can add a whole-brain distribution of "source depth" and then correlate the sensitivity values with it? I'm expecting the source depth distribution to look similar to the sensitivity map, and the correlation to be high.

:)
just display it so there is a usage example in the doc

Cool!


Prepare map visualization

Recovered cases from historic versions of Wikipedia pages
Scrape time series of recovered cases from historic versions of Wikipedia pages, such as
https://en.wikipedia.org/w/index.php?title=COVID-19_pandemic_in_Wisconsin
It turns out that we can get better (more complete) data from ’https://raw.githubusercontent.com/COVID19Tracking/covid-tracking-data/master/data/states_daily_4pm_et.csv .
The ultimate goal is to update https://www.endcoronavirus.org/map-visualization automatically, and updating the number of recovered cases is taking a lot of manual work currently.

Here is a heatmap display of recovered cases, as scraped from Wikipedia pages:

PDF version
Lots of missing values.
Compare this to the heatmap of recovered cases, taken from COVID19Tracking:


@TargetRequestFilter is not invoked with Junit5

I have the below code to add authentication to request header but it is not getting called.
    @TargetRequestFilter
    public void requestFilter(MockHttpServletRequestBuilder request) {
        request.header("username", "someuser");
        request.header("password", "somepassword");
    }

@TargetRequestFilter is invoked when using @RunWith(SpringRestPactRunner.class) but it is not called when using Junit5 + @ExtendWith(SpringExtension.class).

You don't need to use @TargetRequestFilter with JUnit 5, because you can just inject the request object into your methods. Just add it as a parameter to a @BeforeEach method or the @TestTemplate method.


fix: Add path and domain options to cookie deletion

What is it?

[x] Feature / enhancement
[ ] Bug
[ ] Docs / tests

Description
Allows users to set the path/domain when deleting a cookie. Cookies set with a path and/or domain need the fields set to be removed properly.
Fixes: #1990
Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

Amazing job Glenn!


Need a way to filter events

Not all exporters need all events, how should we filter them?
Proposal:

if there is no mention then log all events which are captured
add an "event_inclusion" line to the exporter which contains a list of patterns (regex?) which can be used on an event received to determine if it should be logged.

Should the logNotebookContentEvents flag (that decides whether to log the entire notebook content or not) also be a part of the filter?
If the events to log are decided based on individual exporters, the original activeEvents flag which controls the activated event for all the exporters would be redundant. Or the extension may filter the event library using activeEvents first and then event_inclusion?


Mac App Features

[x] Implement "Use Option as Meta Key".
[x] Implement Command-R as a toggle for reporting mouse events.
[ ] Print
[ ] Save selection
[ ] Save buffer
[ ] Preferences dialog
[ ] Clear scrollbar buffer
[ ] File/New Remote Connection dialog.
[ ] UI to switch themes (color scheme, font)
[ ] UI to select ansi color palette.

See also #21 for a list of features for the terminal view itself.
Should this surface the SSH capabilities from the SwiftTermApps, and have a terminal emulator that can do that?   And we could bring the UI from there which is mostly SwiftUI.

UI Switching of themes is on the SwiftTermApps repo, might be worth bringing - and also the spicy metal shaders.


Make firezone-id optional for linux

Users don't know how to generate this -- we should just generate it ourselves and save it to, e.g. /var/run/firezone/.firezone-id like the GUI clients do.

See https://github.com/firezone/firezone/blob/main/rust/gateway/src/main.rs#L71


Corrected version from v4->v5 in README_js.md

This was previously fixed in https://github.com/kelektiv/node-uuid/commit/880d24e35d2a393b115ce3687465866211ad0680 ,
but was regrtessed in README_js.md and hence in the newly generated README.md
Also reran npm run-script prepare to update README.md from the corrected version

Fixes #214


@ngtools/webpack eagerly reloading everything inside of src folder for ng serve

Bug Report or Feature Request (mark with an x)
- [ x ] bug report -> please search issues before submitting
- [ ] feature request

@ngtools/webpack is causing webpack-dev-server to eagerly rebuild for any change inside of the folder that contains the tsconfig file. This includes files that are not referenced by webpack at all. This can be a huge problem for development if your tsconfig.json file is in the root of your application folder as even git changes trigger a rebuild.
Versions.

@angular/cli: 1.0.2
node: 7.2.1
os: darwin x64
@angular/common: 4.1.1
@angular/compiler: 4.1.1
@angular/core: 4.1.1
@angular/forms: 4.1.1
@angular/http: 4.1.1
@angular/platform-browser: 4.1.1
@angular/platform-browser-dynamic: 4.1.1
@angular/router: 4.1.1
@angular/cli: 1.0.2
@angular/compiler-cli: 4.1.1
Repro steps.

Inside of Angular CLI
From a newly created project, start up the dev server and add an empty file in src without a file extension. The dev server will rebuild.
Inside of a custom project
If you are using @ngtools/webpack for your development build inside of a project with a custom webpack configuration (ejected projects also count), and your tsConfigPath references a tsconfig.json file in the root of the project, start up the dev server and stage a file in git. You will see the dev server rebuild.
The log given by the failure.

N/A
Desired functionality.

@ngtools/webpack should not be increasing the scope of watched files in the project beyond the boundaries of what has been configured inside of webpack itself. If the file is not in the bundle, it should not trigger a rebuild from webpack-dev-server.
Mention any other details that might be useful.

Sounds like a bug alright.
This is probably also related to some instances of https://github.com/angular/angular-cli/issues/1610 where IDE's 'safe-write' settings create temporary files. Knowing that rebuilds are triggered for all file changes really highlights @wulfsberg commment in https://github.com/angular/angular-cli/issues/1610#issuecomment-300298204.


Update README.md

This URL https://unicode-table.com/ has changed to https://symbl.cc/

Hey, thanks for the heads up, merged.


Support RESTful API documentation

The Framework should have an option to enable API documentation for RESTful Route handlers that exist in the application.
Some aspects need consideration:

Whether to enable API docs generation for any specific module or all modules.
Whether to generate API docs on the application booting-up phase or run a specific command line to trigger.
Whether to automatically generate API docs by source code analysis or using Gin Swagger and Swaggo with user manually inputted General API Info Comment

@vzzederzv  this ticket is related to #2 ?

Move to #2 as duplicated.

Move to https://github.com/trinitytechnology/ebrick/issues/2 as duplicated.

Move to https://github.com/trinitytechnology/ebrick/issues/2 as duplicated.


Suitability for very low-bandwidth links?

Hi,
This isn't an issue, but I couldn't find contact information for you anywhere.
First, thanks for writing this.  It looks interesting!
I am interested in very low-bandwidth, long-distance radio links.  These may be via amateur radio protocols, via LoRA (my own lorapipe, https://github.com/jgoerzen/lorapipe works with that), or satellite.  Bandwidth of these links ranges, depending on the technology in play, from about 0.3Kbps to 100Kbps.  Many of them are actually packetized already, and may provide guarantees roughly similar to UDP (error-checked but not reliable, though most guarantee ordering if not delivery).  So you can perhaps see why I'm interested in this protocol.  Those that are packetized would tend to use packets ranging from about 32 bytes up to maybe several hundred bytes, with an ethernet-like MTU of 1500 being a rather high-end outlier.
My question is: what is the overhead of your reliable protocol when working with a pure binary stream?  Also, do you think it would be suitable for situations in which it may actually take a substantial fraction of a second for a packet to be transmitted (or maybe even several seconds for the very slowest)?  When dealing with a 300bps link, every added byte definitely counts!
Thanks!

I'm pretty sure this library is probably only appropriate as it is now for medium bandwidth games protocols over relatively high bandwidth links.  Nothing in this library currently even attempts to handle congestion control at all, so I definitely wouldn't try to use anything in this library as for example a general purpose replacement for TCP / SCTP / QUIC.  Those implementations of standard protocols are almost certainly going to be better engineered for general use than anything this library will probably ever have.
The reason for this library's existence is mostly because I found that on many platforms, multiplexing different streams using TCP or using SCTP / QUIC is a royal PITA.  You can't really rely on being able to use huge numbers of TCP ports, and the scope of existing SCTP implementations and integration with Rust is pretty grim, and QUIC is very early and heavyweight.  The realistic options that I saw open to me were to try to adapt a rust QUIC implementation OR make my own protocol, and I took the second option.  I don't really have to worry about the general use case, I'm trying to make a high player count networked game, so I really only personally care about that sort of use.  This library is designed for when the amount of bandwidth the game needs is much less than the bandwidth of the underlying link, so it sort of tries to optimize above all else for reduced latency.  It sounds like you need the opposite, a library for reliable connections that optimizes above all else for low bandwidth.
Now, if something like plain TCP is inappropriate for this case, or if in your environment real TCP is difficult for some reason and you need your own protocol, it might be useful to take a look at the way the reliable channel in turbulence is implemented, and maybe get some inspiration?  I think PROBABLY as it is the existing reliable channel is too optimized for latency and basically not at all optimized for bandwidth use (it always sends one ack packet per received packet, it constantly re-sends a small bit too much data in the expectation that the other side will process enough by the time it gets there, it could have a LOT better bandwidth efficiency!).  It might be sensible to use the multiplexing parts of this library with your own bandwidth sipping reliable protocol?  The existing reliable protocol is, imo, pretty self contained and hopefully easy to follow, but obviously I'm pretty biased here.  The only two files you need to understand the reliable protocol are the file that defines the send / receive buffers (windows) (https://github.com/kyren/turbulence/blob/master/src/windows.rs), and the channel implementation itself (https://github.com/kyren/turbulence/blob/master/src/reliable_channel.rs).  I'm not some genius internet protocol inventor, this is just kind of baby's first TCP, it was just remarkably hard to find pre-existing implementations of such things that don't come with a whole world of things I'll never need bolted on top and impossible to remove.
I imagine for low bandwidth though that you'd have a hard time beating TCP itself, so if you can figure out how to use a pre-existing TCP implementation that might be the first thing to try?

Thank you very much for this explanation!
So what drew me to your project is that it's not tied to UDP in the way so many others (the various QUIC implementations, for instance) are.  In my case, some of the devices I'm working with (eg, LoRA radios) have a serial interface (often packetized) with no IP stack whatsoever.  The overhead of running TCP/IP across a very slow link is significant due to the TCP and IP headers, and the performance is generally quite bad.  PPP with VJ header compression helps somewhat and it gets halfway decent, but it turns out that protocols from the modem era do pretty well here.    I'm probably going to go re-implement the UUCP protocol "i" in Rust and see how that goes.
Totally understood about QUIC.  Lots of good things about that protocol, but simplicity isn't one of them.

Oh okay that makes sense, I didn't think about at such low packet sizes and bandwidth limits that just the header overhead is significant!  I too wish there were more pluggable network things to do reliability and such which is kind of another reason this project exists (if there were some pluggable TCP like thing already I might have just used that).


[chore] update Go to 1.21, and update relevant pkgs

Desired Behaviour
Need to update Go to 1.21 in go.mod and in Dockerfile, and also Makefile

Minor chore on updating meshkit and meshery-adapter-library

@nebula-aac @theBeginner86 we might need to process this post-haste

Hey @yash37158, are you working on this issue?

you can go for it @AryanSharma9917

@yash37158 will i do it .

Yes @Ayush9026, you can start working on this

I forgot to mention that this is already done, however there seems to be an indirect package that creeped in that may affect how other adapters build using the latest from meshkit even without a release. I'm researching now.

If this issue is already resolved, we can close it. @nebula-aac


Add cloud storage for item resources/assets

Seems much more efficient to store it in code instead of an asynchronous call to download from cloud storage, issue not needed


Create common cmake modules for dependency resolution [ESD-1245]

This PR implements some common cmake functions used for the dependency resolution and provides standard Find* modules for common packages.
The two main new modules are:
GenericFindDependency
A function to search for a dependency. Searches in either bundled source code or system libraries. Configurable through either function parameters or global variables defined on the command line or elsewhere at configure time. Don't call directly, call from a Find* style cmake module.
SwiftCmakeOptions
Sets up and configures several generic options to enable or disable package features such as unit tests, documentation, or example code.
GenericFindDependency.cmake and SwiftCmakeOptions.cmake contain header with documentation of parameters and usage. The PR also includes several Find* modules for commonly used packages including all swift repositories. See other repos for examples of SwiftCmakeOptions.

Looks great so far @woodfell! Documentation is 💯

@martin-swift RE my response to your review comments above, I think there's some management of expectations to be done for this repo in terms of what uses common methods vs taking cmake modules provided for third party repos as they are. I don't know what the right mix is per se, but figured that I'd call it out as an open question as to how much modification is really necessary for such open-sourced reuse.

Placeholder request for ClangTools replacements scripts. Right now all the downstream PRs are defining dummy targets which shouldn't be merged into the trunk.

@benjaminaltieri Definitely. New clang tools support is in the works

@benjaminaltieri @jbangelo @martin-swift
I've finished off the new clang tools modules. I have split them up in to ClangTidy and ClangFormat since not every project uses both. Documentation at the header should be fairly self explanatory.
I have updated all the other PRs to use these modules and removed all the temporary hack targets. The only project which doesn't use the generic tidy/format targets is starling. It still uses this module but it has custom scripts in the repo. Starling is probably worth revisiting though, the script uses some hard coded flags to clang-tidy which passes, but as soon as it is run against the compile command database exported from cmake it throws loads of errors.

Should default SN .clang-tidy file be provided? See for example https://github.com/swift-nav/starling/blob/master/.clang-tidy

Should default SN .clang-tidy file be provided? See for example https://github.com/swift-nav/starling/blob/master/.clang-tidy

@jbangelo @martin-swift
I don't have any strong opinions on this, but I can look in to it. Does anyone else have any thoughts?

@jbangelo @martin-swift
I don't have any strong opinions on this, but I can look in to it. Does anyone else have any thoughts?

let's let individual repos decide what linting they use. it's important that the module supports pulling this in (possibly a custom script as well? i.e. not only .clang_tidy file)


AssertionError: MaskedFill can't differentiate the mask

hi, when i run the following command , i get a exception, can you help  me?
@yunjey
python3 main.py --mode='train' --dataset='CelebA' --c_dim=5 --image_size=128 --sample_path='stargan_celebA/samples' --log_path='stargan_celebA/logs' --model_save_path='stargan_celebA/models' --result_path='stargan_celebA/results'
The number of parameters: 44762048
Traceback (most recent call last):
File "main.py", line 106, in
main(config)
File "main.py", line 41, in main
solver.train()
File "/data1/software/deeplearning/StarGAN/solver.py", line 279, in train
accuracies = self.compute_accuracy(out_cls, real_label, self.dataset)
File "/data1/software/deeplearning/StarGAN/solver.py", line 145, in compute_accuracy
predicted = self.threshold(x)
File "/data1/software/deeplearning/StarGAN/solver.py", line 138, in threshold
x[x >= 0.5] = 1
File "/usr/local/lib/python3.5/dist-packages/torch/autograd/variable.py", line 85, in setitem
return MaskedFill.apply(self, key, value, True)
File "/usr/local/lib/python3.5/dist-packages/torch/autograd/_functions/tensor.py", line 440, in forward
assert not ctx.needs_input_grad[1], "MaskedFill can't differentiate the mask"
AssertionError: MaskedFill can't differentiate the mask

change lines of the method as follows:
x = (x >= 0.5).float()

@mrlzla thank you! it's a bug of pytorch in version 0.3. when i switch to pytorch 0.2,  everything runs well!!!!

@mrlzla Could you expand on your answer? What lines of what method do I have to change?

well,  you could just use torch.clamp(x , 0, 0.5) to limit the x in the range (0, 0.5)


ISSUE: After mod update, saves from previous version doesn't load and are deleted.

Hi,
I've looked at changelog, and I might find issue with your decision to temporarily remove Ancient Pact from Warlock list - at least I discovered this, since it was introduced at version <IP_ADDRESS> and I used that subclass, (good I'm doing backups, before updating, but many people could suffer) my saves were deleted and not loading any more. I believe it should be somehow still present and available if picked before, but not possible to pick for next characters till fixes comes, that way you won't break people saves :)
I hope it somehow helps your project, you're doing wonderful job!

Fixed on <IP_ADDRESS>


Add Fast-Kan (KAN-RBF)

Recent short paper reviews the relationship between RBF and KAN: https://arxiv.org/abs/2405.06721
The official paper's code is here: https://github.com/ZiyaoLi/fast-kan
A faster forked implementation is here: https://github.com/AthanasiosDelis/faster-kan
An alternative implementation of the original is here: https://github.com/Sid2690/RBF-KAN/blob/main/RBF_KAN.py
Any chance you are able to add these to the benchmark leaderboard?

I don't have access to my CUDA workstation now. When I can I will try to update. Or you can do a pull request if you want.

It is now added. It is the second best implementation in terms of efficiency. The ChebyKAN continues to be the first.


Crash when selecting multiple images from photos page

On  line 151 of FSGridViewController it also crashes on
    for (NSIndexPath *indexPath in self.selectedIndexPaths) {

I think there may be one or two other places that also iterates of selectedIndexPaths in this way. So, the fix should apply to all of them.

Oh boy I really did miss that one - version 0.1.5-beta should fix that. ;)


Support usage of absolute path for testRootPath

According description of testRootPath "Path to directory with tests (relative path from place, where save.properties is stored or absolute path)" support logic of usage of absolute path
Now supported only relative https://github.com/cqfn/save/pull/148

A sinppet on how we can get working directory in  PlatformUtils for linux:
import kotlinx.cinterop.ByteVar
import kotlinx.cinterop.allocArray
import kotlinx.cinterop.memScoped
import kotlinx.cinterop.toKString

val pwd: String = memScoped {
            platform.posix.getwd(allocArray<ByteVar>(platform.posix.PATH_MAX))!!.toKString()
        }

I think now it's redundant, after https://github.com/cqfn/save/issues/179, since now it is one entity (save.prop && testRootPath), which could be passed as relative either as absolute path, checked on perfomAnalysisTest, absolute paths worked fine


feat: Refactor asyncEvent middleware and add websocket support

SUMMARY
Migrate asyncEvent module from Redux middleware to a Promise-based event listener approach (with @suddjian) and add Websocket event transport support (extracted from https://github.com/apache/superset/pull/11498).
Resolves an incompatibility between async queries and dashboard native filters.
TEST PLAN
Verify async queries function properly with polling with and without dashboard native filters enabled.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Awesome @robdiciuccio! Testing/reviewing this weekend. Quick observation: you need to upgrade to npm@7, as the lock file will otherwise be downgraded from version 2 to 1.


Vision API call with wrong function name

In Step 9, the code includes a call to the Vision API, in the code it is vision.detectSafeSearch.
The correct function name is vision.safeSearchDetection.
This call appears in the code once in the cloud-functions-angular directory, but it also appears in the instructions.
A simple rename fixes the issue.

Fixed in https://github.com/firebase/friendlychat-web/commit/ec1c77aa59c6125ec0375137c5fc024c7c028059! thanks for the report @getzdan !


Adding NavBar on Profile page as well

On switching to profile page, the navbar disappears.
Though there's a home icon to return to HOME page, but i feel it would be better if NavBar is visible on profile page as well, to maintain uniformity throughout the site.
Also, the display mode switches to LIGHT mode automatically, on the profile page, and there's no option to switch to another mode on profile page.
I would like to work on this issue.

we do not need a navbar there , thanks


CONTENT RETRIEVED

CONTENT RETRIEVED
Acceptance Criteria
GIVEN a text editor web application
WHEN I reopen the text editor after closing it
THEN I find that the content in the text editor has been retrieved from our IndexedDB

Text editor content has been retrieved from IndexedDB


support verlator 5, fix #9

https://github.com/djg/verilated-rs/issues/9#issuecomment-1799352056

I've been using this branch for a while with Verilator 5.006 2023-01-22 and it's working really well. I'd love to see this branch merged in as it's the only dependency I have that needs a patch branch.


Update selector for External metrics type

The documentation for the selector field is incorrect. It says that selector should be a string but it actually expects a map. The above change worked for my use case.

Hi @mlagran1 .
Would you check that you have signed the CLA?

I signed it

/check-cla

Hi @mlagran1 .
Would you verify the email address you used to sign the CLA:
If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address.

:wave: @mlagran1 ,
Were you able to verify your GitHub and CLA information?

Hi @mlagran1 .
I plan on closing this pull request.
Would you verify the CLA or reach out if you need help?

Hi @mlagran1 .
I am closing this pull request.
At any time, you can reopen this pull request to continue to work on the changes.
/close


[Improvement][K8s Task] support the custom labels and selectors for creating the k8s job

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Currently, submitting jobs to k8s cluster will add default labels such as k8s.cn/layer and k8s.cn/name. However, in some practical scenarios, we usually manage resources on k8s by adding custom labels. In addition, we also use selectors to make some pods running on specific k8s nodes.
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I can help with that.


Feat/update monitor pages

@dalwar23 Could you please take at this PR please? 🙂

thanks for the MR.