TestBuildIsShareableForCompatiblePlatform unclear what the intent is

Description
TestBuildIsShareableForCompatiblePlatform relies on AssertErrNotContains, which apparently does not test stderr, but stdout.
Since there is never anything on stdout in that case, none of these asserts are doing anything.
Furthermore, the test on a different architecture will only pass if emulation is enabled, but fail otherwise.
Put otherwise, the test right now is doing two things:

verify that we can build, which is already in another test
verify that we can build on a different platform, which assumes emulation is installed

It is unclear to me what the test intent is.
Can we clarify?
Thanks.
Tagging @Shubhranshu153
Steps to reproduce the issue
na
Describe the results you received and expected
na
What version of nerdctl are you using?
main
Are you using a variant of nerdctl? (e.g., Rancher Desktop)
None
Host information
No response

I will assume this was meant to test building on a different from the host but currently emulated platform, and rewrite the test accordingly.

i can fix it, AssertErrNotContains -> from the name i always assumed it checks stderr, probably a simple redirection of everything to stdout would fix it.
the test checks if a cross build is happening  then a tarball is created. so if the test are running on amd then that portion would run and test but if its running on arm it would get skipped.

i can fix it, AssertErrNotContains -> from the name i always assumed it checks stderr, probably a simple redirection of everything to stdout would fix it.

Probably this method should go. It does not really serve a purpose, since what we are checking here is that the build is successful (exitcode 0 should be sufficient).

the test checks if a cross build is happening then a tarball is created. so if the test are running on amd then that portion would run and test but if its running on arm it would get skipped.

This is not what is happening right now.
If you are on anything but armv7 (eg: arm64), the test will run, and will fail if emulation is not installed (which is the default).
The test should first verify that we support building on something else than the current platform, then build for that (and we should remove the hardcoded arm check).

Probably this method should go. It does not really serve a purpose, since what we are checking here is that the build is successful (exitcode 0 should be sufficient)

-thats not entirely correct, we are checking if the build was successful and export of the image is not a tarball for cases where we are not doing a cross platform build.
yeah agreed removing hardcoded stuff, not sure about a good way to test it out, let me check.

Probably this method should go. It does not really serve a purpose, since what we are checking here is that the build is successful (exitcode 0 should be sufficient)

-thats not entirely correct, we are checking if the build was successful and export of the image is not a tarball for cases where we are not doing a cross platform build.

Besides the fact that currently it is testing stdout instead of stderr - I don't think the above is accurate:
nerdctl build -t foo --platform linux/amd64 . 2>&1 | grep tarball
#5 sending tarball 0.1s done

nerdctl build -t foo --platform linux/arm64 . 2>&1 | grep tarball
#5 sending tarball 0.1s done

nerdctl build -t foo . 2>&1 | grep tarball
#5 sending tarball 0.1s done

Either way, no point in rehashing this - appreciate if you can fix it :-)
Thanks @Shubhranshu153

i see thanks, will update it to test it accurately.

Checked it again, i think we can remove the test creating a pr. Technically the unit test for this cover the different cases for the function itself.
But here is what my test outputs are, so if the platform matches or i dont give platform the tarball is not created, just curious whats your setup to be able to reproduce it, in the grand scheme of things i had thought this behaviour would be consistent in any system
[shubhum@lima-finch fix-shareability-test]$ sudo nerdctl build -t foo --platform linux/amd64 . 2>&1  | grep tarball
[shubhum@lima-finch fix-shareability-test]$ sudo nerdctl build -t foo --platform linux/arm64 . 2>&1  | grep tarball
#5 sending tarball
#5 sending tarball 0.3s done
[shubhum@lima-finch fix-shareability-test]$
[shubhum@lima-finch fix-shareability-test]$
[shubhum@lima-finch fix-shareability-test]$ nerdctl build -t foo . 2>&1 | grep tarball
[shubhum@lima-finch fix-shareability-test]$ nerdctl build -t foo . 2>&1 | grep tarball
[shubhum@lima-finch fix-shareability-test]$ sudo nerdctl build -t foo --platform linux/arm64/v7 . 2>&1  | grep tarball
#4 sending tarball
#4 sending tarball 0.2s done

But here is what my test outputs are, so if the platform matches or i dont give platform the tarball is not created, just curious whats your setup to be able to reproduce it, in the grand scheme of things i had thought this behaviour would be consistent in any system

I am on lima on an M2 - containerd v2.0.0-rc.3 - nerdctl main.
Not sure what the (other) differences could be here?

Checked it again, i think we can remove the test creating a pr.

That, or we clarify what the intention of the test was.
It is fine if we want to keep a test for cross-building, but we just have to make sure:

we skip the test if the host does not have support for cross building
we assert success somehow

checked with containerd v2.0.0-rc.4-56-ga44804738 -nerdctl main, my results were same,
The intention is simple:

Check that tarballs are not created if we dont do cross arch builds, this saves time.
i can a check if the host checks cross builds but not sure it is adding much value to the unit test as we already make sure the build is successful in other tests.

this is the pr: https://github.com/containerd/nerdctl/pull/3456

PR removed this test, so, closing.


Props are handled too simply

Currently this plugin takes the props and just makes them data of the root component, this is fine, but a bit simple for my needs. As an example, I use the props to deliver functional objects to the micro frontend that hide implementation detail of the shell e.g. a logger or an identity provider.
These function objects don't change and have no need to be reactive. Furthermore in Vue 3, due to how reactivity works, objects with methods on its prototype simply just break. It makes more sense for objects like these to be provided as props, or better yet as injectables (using provide).
Is it possible that we can introduce a new option for complex cases like these, e.g.
singleSpaVue({
    handleProps (props)
    {
        return {
            provide () {
                 return {
                     logger : props.logger
                 };
            },
            data () {
                return {
                    locale : props.locale
                };
            }
        };
    }
})

The plugin will take the result of this function and simply merge it into the application options that will be passed into Vue. If a handleProps function is not provided it will behave as normal.
The update lifecycle method would need to be a little smarter too by using vm.$data to derive the data keys that are actually reactive and only update those.
I'm happy to submit a PR, I just wanted my thought process to be validated before I dive into the code.

Hi @lsphillips, I'm open to this change. The handleProps name matches the pattern set by handleInstance, which is good. I am happy to review a PR implementing it. Please submit a documentation PR for it, as well. The file to modify for the documentation is https://github.com/single-spa/single-spa.js.org/blob/master/website/versioned_docs/version-5.x/ecosystem-vue.md


Update the Cloud Firestore Logo

[REQUIRED] Codelab info
https://codelabs.developers.google.com/codelabs/firebase-web/#2
[Optional] Suggested fix
Change the screenshot, since Firestore is out of Beta.

@JamesVanWaza fixed thank you!


TypeScript version?

Hi,
It is great but would you mind to create version with TypeScript?

It's a great idea. Tagging this as enhancement. PRs welcome.


single type does not merge - can't query all locales

I am using version 2.0.0-beta.0, so the code is in new syntax.
As you may know, the param locale=all does not work in Strapi, so the workaround to get all languages of a single type can be done like below:
singleTypes: [
          { singularName: 'home-page', queryParams: { locale: 'en' } },
          { singularName: 'home-page', queryParams: { locale: 'it' } },
          { singularName: 'home-page', queryParams: { locale: 'de' } }
        ],

The previous version works by merging all results into one, but now it only fetches the first entry (shown at build time).
Does anyone have an idea of how to fix it?

@remidej thanks for looking into this! I am having the exact same issue at the moment. Do you have any updates on this?

@jonathan-baernthaler I don't have any working solution at the moment. I have to either wait for the plugin to be fixed, or wait for the Strapi issue to be merged (https://github.com/strapi/strapi/issues/12638).

Hello, we have identified the issue and decided on the best way to fix it. You can expect a fix release next week

Hello, we have identified the issue and decided on the best way to fix it. You can expect a fix release next week

@remidej thanks that would be great. 👏

@jonathan-baernthaler I don't have any working solution at the moment. I have to either wait for the plugin to be fixed, or wait for the Strapi issue to be merged (strapi/strapi#12638).

I found a solution that is rather cumbersome but works for now. You are able to pull in the data via the gatsby-node.js (like the plugin does) and use custom graphql queries for your specific use case.  Maybe something you want to look into

@remidej another thing: I saw that you are working on the strapi-corporate-starter and its respective frontends with next and gatsby! first off great work! I really like the architecture, we are going to use its concepts for a project now.
but as you are aware v4 of strapi broke both frontends. Is it something that's going to be actively maintained in the future again?

Thanks @jonathan-baernthaler
We have ported the Next corporate starter on this repo. I don't know when we'll port the Gatsby version though.

This is implemented in the v2.0.0 stable release. See this section of the readme


Homebrew test block fails for 0.4.11

The test block probably just needs a minor update but I'm not sure what it should be.
Currently it runs
  test do
    require "socket"
    assert_match version.to_s, shell_output("#{bin}/fn --version")
    system "#{bin}/fn", "init", "--runtime", "go", "--name", "myfunc"
    assert_predicate testpath/"func.go", :exist?, "expected file func.go doesn't exist"
    assert_predicate testpath/"func.yaml", :exist?, "expected file func.yaml doesn't exist"
    server = TCPServer.new("localhost", 0)
    port = server.addr[1]
    pid = fork do
      loop do
        socket = server.accept
        response = '{"route": {"path": "/myfunc", "image": "fnproject/myfunc"} }'
        socket.print "HTTP/1.1 200 OK\r\n" \
                    "Content-Length: #{response.bytesize}\r\n" \
                    "Connection: close\r\n"
        socket.print "\r\n"
        socket.print response
        socket.close
      end
    end
    begin
      ENV["API_URL"] = "http://localhost:#{port}"
      ENV["FN_REGISTRY"] = "fnproject"
      expected = "/myfunc created with fnproject/myfunc"
      output = shell_output("#{bin}/fn routes create myapp myfunc")
      assert_match expected, output.chomp
    ensure
      Process.kill("TERM", pid)
      Process.wait(pid)
    end
  end

The error is
Testing fn
/usr/bin/sandbox-exec -f /tmp/homebrew20171013-65213-za5ga.sb /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/bin/ruby -W0 -I /usr/local/Homebrew/Library/Homebrew -- /usr/local/Homebrew/Library/Homebrew/test.rb /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/fn.rb --verbose
==> /usr/local/Cellar/fn/0.4.11/bin/fn --version
==> /usr/local/Cellar/fn/0.4.11/bin/fn init --runtime go --name myfunc

        ______
       / ____/___
      / /_  / __ \
     / __/ / / / /
    /_/   /_/ /_/

Runtime: go
Function boilerplate generated.
func.yaml created.
==> /usr/local/Cellar/fn/0.4.11/bin/fn routes create myapp myfunc
ERROR: no image specified
Error: fn: failed
<0> expected but was
<1>.

Thanks for your help.

We'll get this fixed shortly.

Thanks @treeder!

@ilovezfs fixed and new PR here: https://github.com/Homebrew/homebrew-core/pull/19675

Thanks for fixing this! I've shipped your PR :)


Replace set-env and add-path with secure ways on GHA

Replace set-env and add-path with the recommended ways since they're deprecated due to moderate security vulnerability (https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/).

Thanks :+1:


Drawing the outline of a rectangle

Hello,
this isn't as much of an issue, as it is a question.
Could you give a brief example of how to draw the outline of a rectangle inside the paint method of a widget?
I hope this is not the wrong place to ask this question and I don't bother the development of druid.
Thank you for reading trough this

If by outline you mean that you want to draw a rectangle with a border (or stroke the rectangle) you can use RenderContext::stroke or RenderContext::stroke_styled methods.
See example: https://github.com/xi-editor/druid/blob/master/src/widget/button.rs#L207

Thank you, this makes things much happier for me. Glad you showed this


Producers in MessageDeduplication may not expire deleted

Describe the bug
In the current MessageDeduplication implementation, when the producer is disconnected, the producer will be added to the inactiveProducers collection, waiting to be expired and deleted.
But before the expired deletion, if the broker restarts and these producers are no longer reconnected, then these producers will never be expired and deleted
To Reproduce

Change messageDeduplication config on standalone.conf:

brokerDeduplicationEnabled=true
brokerDeduplicationMaxNumberOfProducers=10000
brokerDeduplicationSnapshotFrequencyInSeconds=10
brokerDeduplicationSnapshotIntervalSeconds=60
brokerDeduplicationEntriesInterval=10
brokerDeduplicationProducerInactivityTimeoutMinutes=2

Start pulsar standalone.
Create some produces and send messages.
Close or stop these producers.
Restart broker.

Expected behavior
After waiting 2 minutes, all producers expired and delete.
Actual behavior
These producers will never be expired and deleted.
Desktop (please complete the following information):

OS: [e.g. iOS]

Additional context
Here is another question about the existence in MessageDeduplication, One of the issues references #15204.

Confirm that there is no such problem, it is my test scene is wrong


[Android] Fix wrong modal size in fullscreen

Display.getCurrentSizeRange() doesn't include the size of the status
bar, so Modal windows on Android have a gap in the bottom that is the
same size as the status bar. This checks the current theme and adds the
size of status bar to the modal window height if necessary.
Test plan (required)
Run a React Native app on Android with a theme that doesn't show status bar and launch a modal dialog. See issue #11872 for an example.

This doesn't look correct to me, I'm fairly certain this was fixed a few months back.
Have you tested this on several APIs (16 to 25), and different Themes?

Yes, I tested with android devices with different themes and different configuration(with hw buttons and on screen navigation bar). I can reproduce the issue in #11872 on all. The documentation for Display.getCurrentSizeRange() always return the size as if the status bar is there. See the relevant documentation: https://developer.android.com/reference/android/view/Display.html#getCurrentSizeRange(android.graphics.Point, android.graphics.Point)

@andreicoman11 I was wondering if there is actually a way to tell the first child inside ReactModalHostView to size itself as big as its parent. That way we wouldn't need to manually set the size here and the DialogRootViewGroup inside the Modal always sizes itself correctly.

@andreicoman11 any comments? I can provide a small test app to show that this is broken.

I need to have some free time to play with some of the Modals we already have. I'm fairly certain that they'll be working correctly.
I have a feeling that you are working with a theme that lets you draw the Modal underneath the status bar, which is not the general case and which we won't want to support by default. In that case, I recommend changing you modal to have top: status_bar_height in JS.

I tried setting top: status_bar_height but it doesn't work. My app is not drawing behind the status bar, it's a full screen app where the status bar is not there. I saw that there is at least one more issue that was opened last May, so it means there are people who might need this. And I don't think having a modal without a status bar is an odd corner case.
Here's a small test app: https://github.com/rgngl/gaptest
It uses the branch I created for the PR #11958 so you can try out both things at once.

I have noticed one thing: If the modal view gets an onSizeChanged after it shows up, it is correctly sized to fill the screen. But the initial size setting in the helper class is still wrong.
From what I can see, modal sizing was fixed in 4941cbcf1 where you removed the shadow node for modal host view but then it got broken again in 922cd6dd where you brought it back.

Checked this locally, it's working as expected.
You can take a look at the UIExample/Modal, and add the following to the container style:
    borderWidth: 5,
    borderColor: 'red',

This will show you what the actual bounds of the Modal are. And they are indeed correct:

Since this is working as expected, I will close this PR. Feel free to open a new one with a JS example where the modal size is incorrect.

Hi @andreicoman11 ,
I have already provided an example where it's not working. You can check the example I provided above. You can also see the screenshots I posted in #11872.  In the screenshot you shared the status bar is there, so yes, in that situation there is no problem. The gap under the model window appears when the status bar is not visible.
Display.getCurrentSizeRange used in ModalHostHelper always returns the maximum screen size without status and navigation bars. Therefore, if the status bar is not visible, the gap happens.
Please let me know if I am missing a point here.

@andreicoman11 this is definitely still an issue, please note as described by @rgngl the issue pertains to the status bar visibility.

This needs to get reopened and merged asap, the maintainer was arrogant to not fully read what is happening and test it for him self in the wrong way. This only happens in fullscreen without the status bar.

👍

Thanks @christopherdro !

Hi @christopherdro, is this going to merge after all?

Yes I just wanted to do one last test myself to make sure everything was working as expected.

@facebook-github-bot shipit


docs: Updated commands.md and created events.md

This is the Pull Request for Issue #12
I hope I did everything correct 🙃
If there are issues please let me know and I will try to fix them.
Greetings,
lokerhp

I changed the things, Not sure why I added the extra #, but they are gone now.


[SPARK-16425][R] describe() should not fail with non-numeric columns

What changes were proposed in this pull request?
This PR prevents ERRORs when summary(df) is called for SparkDataFrame with not-numeric or non-string columns. This failure happens only in SparkR.
Before
> df <- createDataFrame(faithful)
> df <- withColumn(df, "boolean", df$waiting==79)
> summary(df)
16/07/07 14:15:16 ERROR RBackendHandler: describe on 34 failed
Error in invokeJava(isStatic = FALSE, objId$id, methodName, ...) :
  org.apache.spark.sql.AnalysisException: cannot resolve 'avg(`boolean`)' due to data type mismatch: function average requires numeric types, not BooleanType;

After
> df <- createDataFrame(faithful)
> df <- withColumn(df, "boolean", df$waiting==79)
> summary(df)
SparkDataFrame[summary:string, eruptions:string, waiting:string]

How was this patch tested?
Pass the Jenkins with a updated testcase.

Hi, @shivaram .
Could you review this PR?

I'm not sure this is something we should be fixing just on R frontend.  What happens when we run the query from Scala / Python ? If we get the same error we should be fixing it in Scala ?

Test build #61933 has started for PR 14096 at commit f0bd1d6.

This failure happens only in SparkR because SparkR blindly try for every columns.

Currently, Scala/Python already do column-type checking for this.

At here.
https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala#L1922

Oh, I see your point.
The difference occurs at all column retrieval of SparkR.
We can make this consistently with Scala/Python by removing all column retrieval.
That would be more simpler!

I mean colList <- as.list(c(columns(x))). We should not do this.

FYI, here is the result of Scala example.
scala> val df = spark.read.json("examples/src/main/resources/people.json")
scala> df.withColumn("boolean", lit(true)).show()
+----+-------+-------+
| age|   name|boolean|
+----+-------+-------+
|null|Michael|   true|
|  30|   Andy|   true|
|  19| Justin|   true|
+----+-------+-------+
scala> df.withColumn("boolean", lit(true)).describe().show()
+-------+------------------+
|summary|               age|
+-------+------------------+
|  count|                 2|
|   mean|              24.5|
| stddev|7.7781745930520225|
|    min|                19|
|    max|                30|
+-------+------------------+

Yeah we should just call it with empty columns (instead of all the columns) and let the Scala side do the appropriate thing.

I'll update again. Thank you for fast review!

Test build #61933 has finished for PR 14096 at commit f0bd1d6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

Hi, @shivaram .
I updated the PR to call Scala directly.

Test build #61935 has started for PR 14096 at commit 97a158e.

Test build #61938 has started for PR 14096 at commit 65f2236.

Test build #61935 has finished for PR 14096 at commit 97a158e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

Test build #61938 has finished for PR 14096 at commit 65f2236.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test PASSed.

Test build #61941 has started for PR 14096 at commit 08672d9.

Test build #61941 has finished for PR 14096 at commit 08672d9.

This patch fails R style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #61942 has started for PR 14096 at commit c332c52.

Test build #61942 has finished for PR 14096 at commit c332c52.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

Hi, @shivaram .
Now, it's ready for review again.
Please let me know if there is something to do more.
Thank you!

LGTM. Thanks @dongjoon-hyun -- Merging this to master, branch-2.0

Thank you!


⚠ Fortnite has degraded performance

In 8bacec0, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1514 ms

Resolved: Fortnite performance has improved in a580137.


Add setting for path to jar file.

It would be useful to allow a setting to the tla2tools.jar file so one can maintain consistency with other users and a build server version of tla tools.

@scott-fleischman https://github.com/tlaplus/tlaplus/issues/393#issuecomment-561732695 is a related discussion.  https://github.com/tlaplus/tlaplus/issues/306 discusses TLA+ and continuous integration.

@lemmy, thanks for adding some context here.
There're several classpath-related features, like changing tla2tools.jar, adding community modules and other libraries. I think it could be implemented by some smart handling of the -cp option in the already existing Java Options setting.

Until this gets properly addressed, what about providing a nightly build of the extension that always includes the last version of https://nightly.tlapl.us

The feature is provided in the latest pre-release (how-to). Will be released soon.

@scott-fleischman, the feature is released in v1.4. Here's the description of how it works: https://github.com/alygin/vscode-tlaplus/wiki/Java-Options#class-path


30 create footer component

hey Llamas I'm still working but I have some problem with css, and i was trying to fix but nop, tomorrow I'll push branch contact us for tonight I'll go to sleep.
by the way  have a look  file acces.log.js
I din't touche nothing from there

There are a number of problems still. We don't have a page to test the footer.

The reference from eg about-us page to the header and footer component is incorrect (wrong filename)
The reference to the style file is incorrect.

Is it possible to correct this ?

Also, please, exclude the access.log

@dirk-jacobs  i fixed and footer is finished so you can see it please

Thanks @RosaMoran I have few comments on the footer

the logo in the center of the footer styling needs modification
the Privacy Policy & Terms and conditions  should be clickable  elements

@Moamin-AbuEwaida  and @dirk-jacobs  i made that you suggested buddies, it's ready to be merged

Observations:

The 3 social media icons are linked to GitHub, but we can fix that later.
I think we haven't discussed what our terms and conditions are, so...
The phrase "code along with us" is linked to GitHub too, I don't know if that was the idea.
Some minor alignments touches are needed but, again, can be fixed later


model.clear should maintain the id.

Based on issue #565.

This seems to already be resolved in #565 and elsewhere.


Initial TinyGo, JS/TS, Grain API docs

Use the reactr api descriptors to expand the Runtime API docs with the recently added languages (Tiny Go, JavaScript and TypeScript, Grain).

@cohix @ospencer could you give me the up-to-date Status (stable/beta/preview/experimental etc.) and the short description of all the Runtime APIs? As seen in the docs right now. If the existing ones need updating, let me know, but I'm primarily interested in the new ones (Tiny Go, JS, Grain).

@flaki sure:
Rust: Stable
Go: Beta
AssemblyScript: Beta
Grain: Beta
JS/TS: Preview
Swift: Preview

Now this is another thing, it would be good to keep the ordering according to status consistent but that currently means rewriting all existing tabs manually, urgh. #27 can't come soon enough. Will try to look into that on Monday if i find the time.


Allow CTerm to consume mlTop/mlBottom

Currently, CTerm attempts to split configuration and constraints on the Kast passed to it. This allows it to handle mlTop() and mlBottom() as well.

closed in favor of https://github.com/runtimeverification/pyk/pull/621


Implement "Decide" step

Should have support an attribute containing an expression. Downstream flow direction is dictated by the outbound link whose name matches the expression result. Expression is evaluated against runtime values.

Duplicate of #125


[feat] Add opportunity to include .dll or .so libraries along with the binary

Describe the problem
Hi!
I'm using Vosk with Rust Tauri.
In order to work properly, Vosk libraries should be bundled with the Tauri binary or included in the PATH of a user.
With the Rust itself bundling Vosk libraries is easy and even can be automated with tools like cargo make.
But with Tauri, it does not work.
Because bundling is make with cargo tauri build command, and cargo make totally skipped in process.
Thanks!
Describe the solution you'd like
Allow bundling libraries like .dll or .so alongside with sidecars and resources.
This libraries should be placed with the app .exe file.
Alternatives considered
No response
Additional context
No response

Have you tried using https://tauri.app/v1/api/config#bundleconfig.resources ? also you can change the runner using https://tauri.app/v1/api/config/#buildconfig.runner which you maybe able to set to cargo-make binary if it exists in PATH.


[BUG]Send-PnPMail : Object reference not set to an instance of an object.

Notice
Many bugs reported are actually related to the PnP Framework which is used behind the scenes. Consider carefully where to report an issue:

Are you using Invoke-PnPSiteTemplate or Get-PnPSiteTemplate? The issue is most likely related to the Provisioning Engine. The Provisioning engine is not located in the PowerShell repo. Please report the issue here: https://github.com/pnp/pnpframework/issues.
Is the issue related to the cmdlet itself, its parameters, the syntax, or do you suspect it is the code of the cmdlet that is causing the issue? Then please continue reporting the issue in this repo.
If you think that the functionality might be related to the underlying libraries that the cmdlet is calling (We realize that might be difficult to determine), please first double check the code of the cmdlet, which can be found here: https://github.com/pnp/powershell/tree/master/src/Commands. If related to the cmdlet, continue reporting the issue here, otherwise report the issue at https://github.com/pnp/pnpframework/issues

Reporting an Issue or Missing Feature
Please confirm what it is that your reporting
Expected behavior
This should send email from configured mail -From
Actual behavior
Send-PnPMail : Object reference not set to an instance of an object
At line:1 char:1

Send-PnPMail -From "Email ID" -To  ...

  + CategoryInfo          : WriteError: (:) [Send-PnPMail], NullReferenceException
  + FullyQualifiedErrorId : EXCEPTION,PnP.PowerShell.Commands.Utilities.SendMail

Steps to reproduce behavior
Execute this line
Send-PnPMail -From "fromemailid" -To "toemailid" -Subject "Test message" -Body "This is a test message"
What is the version of the Cmdlet module you are running?
(you can retrieve this by executing Get-Module -Name "PnP.PowerShell" -ListAvailable)
Name                           Value

PSVersion                      5.1.22000.2538
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.22000.2538
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
Which operating system/environment are you running PnP PowerShell on?

[ ] Windows

Can you run it with -Verbose and share what it outputs?
Can you also share how you connect? You can leave out the environment specific values, just need to know which parameters you're using.

Can you run it with -Verbose and share what it outputs? Can you also share how you connect? You can leave out the environment specific values, just need to know which parameters you're using.

Yes, I ran with the -Verbose.

I connected the SharePoint sites with PnP Online by using below command.
Connect-PnPonline -Url $adminSiteURL -ClientId $spClientId -ClientSecret $spClientSecret -WarningAction Ignore
2)I got the Client ID and Client Secret from the [_layouts/15/appregnew.aspx] page.
3)I am sending email with below command.
Send-PnPMail -From "Email ID" -To "Email ID" -Subject "Welcome" -Body "Welcome to My World." -Verbose -Password password.
If i remove the From address, i can able to send the email, but i can't send email with -From paramter.
5)-From is mandatory to complete my work.

Error Details:
Send-PnPMail : Object reference not set to an instance of an object.
At line:1 char:1

Send-PnPMail -From  -To Si ...

  + CategoryInfo          : WriteError: (:) [Send-PnPMail], NullReferenceException
  + FullyQualifiedErrorId : EXCEPTION,PnP.PowerShell.Commands.Utilities.SendMail

Thanks for providing the additional details. Which PnP PowerShell version are you using?
The verbose output should also show if it sends out the mail using Graph or by another means. This cmdlet has been revamped not too long ago, so depending on the version you're using, I will know if it's going to work.
Having said that, the - by far - recommended approach for this scenario would be to create a proper Azure Active Directory / Entra ID Application Registration, assign it the Mail.Send permission on Microsoft Graph and connect using a client id and client certificate connected to that application registration. That's the most robust, future safe approach.

Friendly reminder. Can you share the above requested details?

Closing issue as no further response is being received


Not Starting on Quest

When I Launch BeepSaber on my Quest 2 it is just a black screen.

Are you using the .apk from the github releases or are you trying to launch from godot using the master branch? Could you try and see if adb shows some meaningfull error messages?: for example useing adb logcat -s godot:* GodotOVRMobile:*

I downloaded the .apk from the GitHub releases.

I'm getting the same thing (black screen).
I don't really know what I'm doing, so if I can do this differently let me know.

I have the same issue on the Quest 2. Is there something else I need to install?

All versions in GitHub Releases have this issue on Quest 2, presumably because of an error that occurs while loading the splash screen texture.

seems like something in recent Quest updates have broke it, you can use this fork that is ported to godot 4 and openxr:
https://github.com/leandrodreamer/BeepSaber/releases
(i keep it on my own fork because most of OQ_Toolkit is not ported)

Thank you, @leandrodreamer ! Your port works like a charm.
Before your post, I tried building 3.x version of Beep Saber myself using Godot 3.5.2. I followed these instructions on how to deploy the Oculus Mobile Godot plugin, but it didn't work. However I don't think it's worthwhile to debug the issue since Oculus Mobile plugin is deprecated and you've already ported the game to Godot 4 and OpenXR.
If anyone is curious about how Godot 3.x performs on Quest 2, this release of Godot Oculus Quest Toolkit contains a Beep Saber demo and it works on Quest 2.
@leandrodreamer if you have free time, I kindly encourage you to work on your Beep Saber fork. Or at least you can briefly describe what needs to be done about OQ_Toolkit so that others can continue. I think it might be better to continue the development in a separate repository since the original developer @NeoSpark314 seems to be inactive. Also I like how you renamed the game to "OpenSaber" in your fork, it fits better.

@leandrodreamer that works brilliantly on the Quest 2 - thanks :-)

Before your post, I tried building 3.x version of Beep Saber myself using Godot 3.5.2. I followed these instructions on how to deploy the Oculus Mobile Godot plugin, but it didn't work. However I don't think it's worthwhile to debug the issue since Oculus Mobile plugin is deprecated and you've already ported the game to Godot 4 and OpenXR.

i managed to port it myself to godot 3.5.2 by replacing oculus mobile with openxr, but again a lot of OQ_toolkit depends on it and also performance is not as good as it is in godot 4


feat(release): Use github actions to release binaries

HelloW,  You got a really nice project there. Congrats on that :clinking_glasses:
This pr will allow you to use github actions and release binaries of presenterm .
Few notes :

Don't forget to add CARGO_TOKEN as secret before merge for crates.io releases

Consider adding a CHANGELOG.md, You can use git-cliff for that. I'm not super familiar with this but still you can get some idea from my repos

Allow github actions to publish

Image ->

The process would go like this . Tested here

I added cargo profiles for release,bench,test,dev. It was helping me a lot to minimize the binary size and test with minimal efforts.

Feel free to make changes <3

I have added splashes. Previously, it would do nothing if no arguments were provided. You can see this in the image below.
POC:
Before :

Now :

If you have any comment on ascii or design let me know :100:

Awesome, thanks so much! I will add a changelog before next release :ok_hand:. Your readme looks nice but I think I'd prefer something a bit more minimalistic.


Feature/choose standard loadbalancer sku

Thanks for submitting an pull request to pcf-pipelines.
To speed up the process of reviewing your pull request please provide us with:

A short explanation of the proposed change:

An explanation of the use cases your change solves:

Expected result after the change:

Current result before the change:

Links to any other associated PRs or issues:

[ ] I have viewed signed and have submitted the Contributor License Agreement

[ ] I have made this pull request to the master branch

[ ] I have run all the unit tests

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


NIFI-9253 Corrected SSLSocketChannel.available() for TLSv1.3

Description of PR
NIFI-9253 Corrects SSLSocketChannel to avoid intermittent unit test and runtime failures related to TLSv1.3.  The available() method implementation resulted in sporadic unit test failures for TLSv1.3 due to partial reads from the connected channel while attempting to determine available bytes.  Partial reads produced incomplete TLS messages, causing AES-GCM decryption failures reported as Tag mismatch messages with SSLException.
The previous unit test used a loop for checking the available() bytes, causing intermittent failures. The updated unit test class includes separate methods with and without calls to available() in order to reproduce the potential problem. The corrected available() method in SSLSocketChannel avoids direct reads from the socket channel and instead returns the size of the application data buffer.  This approach aligns with the documentation for InputStream.available(), which states that the method returns an estimate of the number of bytes that may be read, without blocking. Additional changes include refactoring and isolating the readChannel() and unwrap() methods in SSLSocketChannel for more streamlined handling of buffer underflow status.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically main)?

[X] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[X] Have you written or updated unit tests to verify your changes?
[X] Have you verified that the full build is successful on JDK 8?
[X] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

local build is happy and github checks are happy.  But i'm not sure I can give the code changes justice for a review.  Anyone else available to look?

local build is happy and github checks are happy. But i'm not sure I can give the code changes justice for a review. Anyone else available to look?

Thanks for the feedback @joewitt!
@gresockj can you take a look at the changes based on reviewing previous updates to this class?


Change the font of the application text

Suggestion / Feature Request
The default font is being used right now, would be nice to have good looking font, can check out google fonts for any good ones

assign this issue  to me @ken1000minus7
would like to work on this

Sure @Frey0-0 , assigning u 😄

hi, please assign me.

Sure @shrishti147 go ahead 😇

hey @ken1000minus7 can you assign this issue to me i can work on it

Hey @kashish0604 , thanks for your interest in contributing but this issue is already assigned 😅 . You can check out other issues or create new ones that may be relevant 😇

@shrishti147 any updates🤔 ??

i would like to work on this issue, assign pls

Lfg


Added Leap Year Calculator

Fixes Issue🛠

Closes #847
Description👨💻

Adding Leap Year calculator- Leap Year, people says that it comes once in 4 years but there are some exceptional cases, where leap year doesn't occur like in same pattern. For eg: 1896 is a leap year but after 4 years 1900 is not a leap year. To perform or check this I have added a new calculator called Leap Year Calculator. In this if we enter a year it will say whether it as Leapyear or not. It is applicable for all present, past and future.
Type of change📄

[x] New feature (non-breaking change which adds functionality)

How this has been tested✅
I have taken a list of years which may or may not be a leap year and checked . It is working correctly

Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Screenshots/GIF📷

https://github.com/Rakesh9100/CalcDiverse/assets/125359008/4dbd4a98-ce9a-4293-b0d7-96dc5fcc477d

Pls change the background as text is not visible clearly, enhance the UI and also add Readme file of your calculator in your folder @GnanariddhikaRavikumar

Pls change the background as text is not visible clearly, enhance the UI and also add Readme file of your calculator in your folder @GnanariddhikaRavikumar

@Rakesh9100  I will change it soon.

Enhance the UI slightly with some text and background colors @GnanariddhikaRavikumar

Sure @Rakesh9100

@Rakesh9100  I have altered the UI.

Congrats on getting your PR merged to the repo under GSSOC 2k24 🥳🎉 @GnanariddhikaRavikumar

Thank You @Rakesh9100


Pipeline Service troubleshooting Grafana Dashboard

The following metrics are being queried:-
* pipelinerun_duration_scheduled_seconds
Using the expr - sum by (namespace) ((avg_over_time(pipelinerun_duration_scheduled_seconds_sum[1h]) / avg_over_time(pipelinerun_duration_scheduled_seconds_count[1h])))
We calculate average scheduling duration per PipelineRun in seconds for each namespace over the last 1 hour
* tekton_pipelines_controller_pipelinerun_duration_seconds
Using the epxr - sum by (namespace) ((avg_over_time(tekton_pipelines_controller_pipelinerun_duration_seconds_sum{status=\"success\"}[1h]) / avg_over_time(tekton_pipelines_controller_pipelinerun_duration_seconds_count{status=\"success\"}[1h])))
We calculate average duration of successful pipelineruns in seconds over one hour time window grouped by namespace.
Preview

As commented  Grafana Observational graphs for Pipelines Controller Monitoring #683 , I don't have the visibility if it's gonna look the same once we merge all of my PRs.
What we can do is go one by one over the PRs while merging. Once #683 is merged, I'll come over here again and rebase it.

Hello @gabemontero,  I want to clarify Pipeline Service troubleshooting dashboard is meant to get the stats from individual namespace (if you look at the queries, they're ultimately grouped by ns) while tekton_pipelines_controller_workqueue_depth gathers data from core tekton reconciler. And additionally, I've already added  the workqueue_depth metric here. Let me know your thoughts on this.

we'll want the throttled metrics on this panel too,

Sure

Hello @gabemontero, I want to clarify Pipeline Service troubleshooting dashboard is meant to get the stats from individual namespace (if you look at the queries, they're ultimately grouped by ns) while tekton_pipelines_controller_workqueue_depth gathers data from core tekton reconciler. And additionally, I've already added the workqueue_depth metric here. Let me know your thoughts on this.

Makes sense / works for me / thanks @ramessesii2

we'll want the throttled metrics on this panel too,

Sure

cool ... yeah we have the 1.11 bump passing in our test PRs.  @Roming22 will need to resolve with chains team how we deal with it long term in the continuation of https://github.com/openshift-pipelines/pipeline-service/pull/688 when he is back from PTO

In which  order do you want to merge this PR vs. https://github.com/openshift-pipelines/pipeline-service/pull/683 ?

I would like  683 to go first. Once that's done, I'll bump PipelineService SHA in infra for dev/stage. And will give this PR a rebase and will signal the merge.

I've rebased this PR after merge of #683 & have confirmed it both on Staging & using Inner Loop Observability local setup. And it's ready to be merged.

/test

/retest pipeline-service-test


Support for applications with dashes in its names

Support applications and profiles with dashes in its names in properties, json & yml endpoints.
See discussion on #384

How does this work for "/{label}/{name}-{profiles}.yml where there is a dash in request mapping url?

Merged (with polish - please use the coding conventions from README) in ba303fd.


Migrate to Storybook 8.2.6

Description
This upgrades the design-system/apps/docs example to latest stable Storybook version following the migration guide from Storybook.
Testing Instructions

cd examples/design-system
pnpm install
pnpm dev
view Storybook instance

Allow CI Workflow Run

[ ] approve CI run for commit: dc4aa9765b832d4ed16b35fc52b992cfc2a16a53

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer


shell: bluetooth shell asserted after 'bt init'

on bt_ready, ctx_shell would be NULL if not initialized before bt_enable.

Codecov Report

Merging #11924 into master will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11924      +/-   ##
==========================================
+ Coverage   48.11%    48.2%   +0.09%
==========================================
  Files         280      280
  Lines       43246    43259      +13
  Branches    10363    10364       +1
==========================================
+ Hits        20806    20854      +48
+ Misses      18285    18253      -32
+ Partials     4155     4152       -3

Impacted Files
Coverage Δ

include/logging/log_core.h
100% <0%> (ø)
:arrow_up:

include/logging/log_msg.h
90.9% <0%> (+1.81%)
:arrow_up:

misc/printk.c
84.93% <0%> (+3.12%)
:arrow_up:

subsys/logging/log_core.c
74.67% <0%> (+4.8%)
:arrow_up:

subsys/logging/log_output.c
69.4% <0%> (+9.58%)
:arrow_up:

Continue to review full report at Codecov.

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

Is this still relevant after #11604 ?
@nordic-krch FYI

Side comment: How we are going to support simultaneous access from several shell instances when ctx_shell is just a global variable?

@summer20100514 Your fix is fine but it is not enought to make bt commands work correctly.
We must implement: #11604 and #11897 which already implements your fix.
However I am not against merging your PR, it will make source code a bit more robust until we will have mentioned PRs in place.

@jarz-nordic any updates?

@summer20100514 : I am not BT expert but on my side example itself is working correctly:

From my perspective shell is now able to display text correctly from different threads, so any potential problems should not be related to it.

@jarz-nordic ok, one stupid question. How can I terminate the shell output after tapping bt scan? I use ctrl + C, but in vain.

It is up to command implementation not up to shell. I guess you can try bt scan off ?

@jarz-nordic it works, thank you!


Added the Document Chat plugin to the list

Submit your project for inclusion in XD-Awesome
Hey! We're happy you want to submit your project to XD-Awesome. Please create a pull request and ensure that you fill out the following template:
Document Chat
Description of your project: Chat plugin that uses the root node's pluginData to store messages, removing the need for an external server.
Link to your project: https://github.com/pklaschka/chat-xd-plugin
License: MIT
Category: Awesome Open Source Plugins
Why is it awesome?
Where do I begin 😜 ? First of all, it's a privacy-first chatting solution for the XD cloud documents with coediting. If that isn't awesome, I don't know what is. But more, there are some interesting technical implementations in this repository. Among other things, the plugin, behind the scenes, uses:

Typescript (with the official Typescript Definitions)
React for the panel
React Router
Cypress for component tests (including mocks for the relevant XD modules and a stylesheet to emulate XD's look and feel)
@fliegwerk/logsemts, which is a library that makes console logs much more enjoyable and is easy to integrate into XD
Fully automated CI builds and releases using GitHub actions

If you need more, please let me know, but at least to me, this is pretty awesome 😉
Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] I have read the CONTRIBUTING document.

Thanks Pablo!


find_program cache prevents finding previously unfound program

Describe the bug
eg, We start with gdb not installed then:
> xmake run -d
error: debugger not found!
> pacman -S gdb

- - - - - - -

> xmake run -d
error: debugger not found!

Expected behavior
For xmake to find gdb after installing it.
Error output
checking for Cuda SDK directory ... no
checking for Cuda SDK directory ... no
error: @programdir/core/main.lua:279: @programdir/modules/devel/debugger/run.lua:279: debugger not found!
stack traceback:
    [C]: in function 'error'
    [@programdir/core/base/os.lua:800]: in function 'raise'
    [@programdir/modules/devel/debugger/run.lua:279]: in function 'run'
    [@programdir/actions/run/main.lua:59]: in function '_do_run_target'
    [@programdir/actions/run/main.lua:83]: in function 'script'
    [@programdir/actions/run/main.lua:138]: in function '_run'
    [@programdir/actions/run/main.lua:201]:
    [C]: in function 'load'
    [@programdir/core/base/task.lua:519]: in function 'run'
    [@programdir/core/main.lua:277]: in function 'cotask'
    [@programdir/core/base/scheduler.lua:365]:

stack traceback:
	[C]: in function 'error'
	@programdir/core/base/os.lua:800: in function 'raise'
	@programdir/core/main.lua:279: in function 'cotask'
	@programdir/core/base/scheduler.lua:365: in function <@programdir/core/base/scheduler.lua:362>

Related Environment

xmake version:v2.5.4+202105180112
os: Manjaro
target platform: Linux

Additional context
The issue seems to be that the boolean false result from the cache is used here from the previous failed attempt at finding the tool:
https://github.com/xmake-io/xmake/blob/239c446b20ecceda838e0de77711be18805fab26/xmake/core/sandbox/modules/import/lib/detect/find_program.lua#L277-L281
... commenting it out fixes the issue. It has happened with doxygen and gdb so far.

Yes, this is not a bug. Usually xmake caches the detection results to avoid repeated detection. Of course, this will also make it impossible to detect changes in the user's environment in time, such as the installation of gdb later.
Therefore, you only need to execute xmake f -c after installing gdb / doxygen to clean up the configuration cache, and xmake will try to detect gdb again.

Ah, okay thanks. I thought that was for clearing a project config cache. I understand now, although my former self clearly needed a nicer error message. :)


Getting InvalidRequestError when running the backend with uvicorn

Bug Description
Hey everyone. I am trying to run the backend alone with uvicorn, and it seems like there is a duplicate table name or a duplicated module load or things of that nature.
Steps to Reproduce

Create a virtualenv: python -m venv .venv.
Activate the virtualenv: source .venv/bin/activate.
Install all required dependencies: pip install -r requirements.txt.
cd into the src folder: cd src
run the backend with uvicorn: uvicorn fides.api.main:start_webserver --reload

Expected behavior
The backend should run as expected.
Screenshots
I will provide a full stack trace instead of screenshots:

stack trace

INFO:     Will watch for changes in these directories: ['/home/harmouch/Desktop/open-source/fides/src']
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)
INFO:     Started reloader process [9053] using watchgod
Startup configuration: reloading = False, dev_mode = False
Startup configuration: pii logging = False
Process SpawnProcess-1:
Traceback (most recent call last):
  File "/home/harmouch/.pyenv/versions/3.9.10/lib/python3.9/multiprocessing/process.py", line 315, in _bootstrap
    self.run()
  File "/home/harmouch/.pyenv/versions/3.9.10/lib/python3.9/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/home/harmouch/Desktop/open-source/fides/.venv/lib/python3.9/site-packages/uvicorn/subprocess.py", line 76, in subprocess_started
    target(sockets=sockets)
  File "/home/harmouch/Desktop/open-source/fides/.venv/lib/python3.9/site-packages/uvicorn/server.py", line 60, in run
    return asyncio.run(self.serve(sockets=sockets))
  File "/home/harmouch/.pyenv/versions/3.9.10/lib/python3.9/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "uvloop/loop.pyx", line 1517, in uvloop.loop.Loop.run_until_complete
  File "/home/harmouch/Desktop/open-source/fides/.venv/lib/python3.9/site-packages/uvicorn/server.py", line 67, in serve
    config.load()
  File "/home/harmouch/Desktop/open-source/fides/.venv/lib/python3.9/site-packages/uvicorn/config.py", line 458, in load
    self.loaded_app = import_from_string(self.app)
  File "/home/harmouch/Desktop/open-source/fides/.venv/lib/python3.9/site-packages/uvicorn/importer.py", line 21, in import_from_string
    module = importlib.import_module(module_str)
  File "/home/harmouch/.pyenv/versions/3.9.10/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 850, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/home/harmouch/Desktop/open-source/fides/src/./fides/api/main.py", line 50, in <module>
    from fides.api.ops.api.v1.api import api_router
  File "/home/harmouch/Desktop/open-source/fides/src/./fides/api/ops/api/v1/api.py", line 1, in <module>
    from fides.api.ops.api.v1.endpoints import (
  File "/home/harmouch/Desktop/open-source/fides/src/./fides/api/ops/api/v1/endpoints/connection_endpoints.py", line 43, in <module>
    from fides.api.ops.schemas.connection_configuration import (
  File "/home/harmouch/Desktop/open-source/fides/src/./fides/api/ops/schemas/connection_configuration/__init__.py", line 3, in <module>
    from fidesops.ops.models.connectionconfig import ConnectionType
  File "/home/harmouch/Desktop/open-source/fides/.venv/lib/python3.9/site-packages/fidesops/ops/models/connectionconfig.py", line 96, in <module>
    class ConnectionConfig(Base):
  File "/home/harmouch/Desktop/open-source/fides/.venv/lib/python3.9/site-packages/sqlalchemy/orm/decl_api.py", line 75, in __init__
    _as_declarative(reg, cls, dict_)
  File "/home/harmouch/Desktop/open-source/fides/.venv/lib/python3.9/site-packages/sqlalchemy/orm/decl_base.py", line 126, in _as_declarative
    return _MapperConfig.setup_mapping(registry, cls, dict_, None, {})
  File "/home/harmouch/Desktop/open-source/fides/.venv/lib/python3.9/site-packages/sqlalchemy/orm/decl_base.py", line 177, in setup_mapping
    return cfg_cls(registry, cls_, dict_, table, mapper_kw)
  File "/home/harmouch/Desktop/open-source/fides/.venv/lib/python3.9/site-packages/sqlalchemy/orm/decl_base.py", line 310, in __init__
    self._setup_table(table)
  File "/home/harmouch/Desktop/open-source/fides/.venv/lib/python3.9/site-packages/sqlalchemy/orm/decl_base.py", line 799, in _setup_table
    table_cls(
  File "<string>", line 2, in __new__
  File "/home/harmouch/Desktop/open-source/fides/.venv/lib/python3.9/site-packages/sqlalchemy/util/deprecations.py", line 298, in warned
    return fn(*args, **kwargs)
  File "/home/harmouch/Desktop/open-source/fides/.venv/lib/python3.9/site-packages/sqlalchemy/sql/schema.py", line 577, in __new__
    raise exc.InvalidRequestError(
sqlalchemy.exc.InvalidRequestError: Table 'connectionconfig' is already defined for this MetaData instance.  Specify 'extend_existing=True' to redefine options and columns on an existing Table object.

Environment

Version: 1.8.5+632.gddc80928.dirty (unified-2 branch)
OS: ubuntu 20.04
Python Version: 3.9.10
Docker Version: Not required.

Additional context
That's pretty much it.
Question
Is there an easy way to run the backend only?

Thanks for the report. I haven't seen this error before, but I will look into it.
As for an easy way to run the backend only, not on its own. Things are fairly tightly tied to running in docker. nox -s dev -- shell will start the container with a terminal in the container to interact with.


Unable to run vite dev on v0.61.4

UnoCSS version
=0.61.4
Describe the bug
Error from running vite dev:
278 | function createSvelteScopedContext(configOrPath) {
279 |   const uno = createGenerator();
280 |   const ready = reloadConfig();
281 |   const loadConfig = createRecoveryConfigLoader();
282 |   async function reloadConfig() {
283 |     const { config, sources } = await loadConfig(process.cwd(), configOrPath);
                                            ^
ReferenceError: Cannot access 'loadConfig' before initialization.
      at /home/user/project_dir/node_modules/@unocss/svelte-scoped/dist/vite.mjs:283:39
      at reloadConfig (/home/user/project_dir/node_modules/@unocss/svelte-scoped/dist/vite.mjs:282:33)
      at createSvelteScopedContext (/home/user/project_dir/node_modules/@unocss/svelte-scoped/dist/vite.mjs:280:17)
      at UnocssSvelteScopedVite (/home/user/project_dir/node_modules/@unocss/svelte-scoped/dist/vite.mjs:258:19)
      at /home/user/project_dir/vite.config.js.timestamp-1721135614680-89ba977444b6d8.mjs:6:13

Doesn't happen on version =0.61.3
Reproduction
Run vite dev with this vite.config.js:
import { sveltekit } from "@sveltejs/kit/vite";
import { defineConfig } from "vite";
import UnoCSS from "@unocss/svelte-scoped/vite";

export default defineConfig({
  plugins: [UnoCSS(), sveltekit()],
});

System Info
No response
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Will be fixed by https://github.com/unocss/unocss/pull/3992 but for now you can also control click the (/home/user/project_dir/node_modules/@unocss/svelte-scoped/dist/vite.mjs:282:33) and just move the line const loadConfig = createRecoveryConfigLoader(); up so it is run before const ready = reloadConfig();.


Package ownership

I would like to use the package name mario. Attempting to upload gives a 403 error. I don't see any contact info, is it possible to identify the owning account and maybe email them? It's possible that it was me and I forgot the username.

The owner of that name seems to be @roll . Perhaps they can reply here and you can discuss a transfer.

@mar10py
Please try again. It's deleted now

@jamadden Thank you for figuring this out.
@roll Perfect, thank you for releasing!


fix jpeg quality encoding

jpgWriteParam was in fact not used, so compression quality was not functional.
added an explicit exception for users who are used to 0-100 integer for jpeg quality.

How about supporting LONG parameters on top of DOUBLE rather than issuing an error?

done.


Add more example scripts

From #89. While I think it is up to the user to read the reference documentation or view the Web API documentation in order to learn all the available endpoints, the examples should be expanded to contain at least an example from every API. They should be a bit more interesting than a single call and printing out the results. Let's make them personalised too.
Here are some:

Browse, Album: get album tracks of a new release
Artist: get albums of a user's top artist, get related artists of a user's top artists and highlight ones that were already followed
Browse, Follow: follow a spotify playlist in a category
Misc, Follow: follow an artist by searching their name
Misc, Browse, Playlist: create a playlist of recommendations based on a user's top tracks
Playlist, Track: analyse a track in a user's playlist
Library, Player: start playing a saved album

Who's to say there can't be a ton of more interesting examples, but these will do for now.


Mismatched instructions for SSH/HTTPs

Gitea version (or commit ref): 1.15.0+dev-494-g35f37a362 built with GNU Make 4.3, go1.16.5 : bindata, timetzdata, sqlite, sqlite_unlock_notify
Git version: 2.30.2
Operating system: Ubuntu-20.04 Docker install

Database (use [x]):

[X] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[X] Yes (provide example URL): https://try.gitea.io/adyanth/test
[ ] No

Log gist:

Description

See below. The instructions don't match the protocol selected after creating a repo, and on every reload.
Screenshots

After creating a repo

2nd repo

Switch to SSH

Switch back to HTTPS

Reload page

This is inconsistent behavior as one would expect the instructions to align with the selected scheme.

note: if you klick to ssh and back to https, it get changed to a valid https commands ...

Yes, also provided screenshots for the same.
Meaning it can show https commands, but won't do by default.


[Node.js] Fix issues found in UT

Update test cases and index.js for issues found during test

@halton PTAL, thanks.

@halton PTAL of the updated commit, thanks.


Silences failing test

Resolves #1138.
This pull request offers a temporary solution to skip the failing MQTT test as we continue to investigate an alternative to the q.m2m.io host.

Looks good, thanks @bswinnerton!

Nice work.

I intend on deploying this tomorrow, March 31st.


Quotation Marks are removed at the beginning and end of a dialog box

In D2E editor, events don't seem to like quotation marks at the beginning and the end of the dialog box. When I add it and save, it deletes them.

As a work around you can use 3 quotes at the start and end and 2 in the middle if required.  Or just use a space as the first character.  I will fix for 1.5

Looks like the same thing for periods too. I have used the workaround in my latest update to the quest.

I want to get Valkyrie 1.5 out to support RtL 1.4, this is going to require a bit more work, so changing target.

This is fixed for newly entered text, existing text should be unchanged.


Publish zipped docs in our releases

It would be useful to have a single .zip containing all our docs as part of our GitHub release. It could then be throw on an LLM by us or our users.
The zip should include:

All of https://rerun.io/docs/
Our API docs (Python, C++, Rust)

I think optimally we should have a button to download all docs on the docs page to make the flow of adding these to your AI coding assistant as easy as possible


aealite fixes: p2p acn support, tls support

Proposed changes
This trello ticket

added a test that ensures aealite type definition and .proto file in aealite is identical to packages/valory/protocols/acn

comes after #222  and #225
Fixes
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to agents-aea?
Put an x in the boxes that apply

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

Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the develop branch (left side). Also you should start your branch off our develop.
[ ] Lint and unit tests pass locally with my changes and CI passes too
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that code coverage does not decrease.
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Why not remove aealite entirely from this repo? It's a pain to develop here and atm we don't develop it anyway.

Why not remove aealite entirely from this repo? It's a pain to develop here and atm we don't develop it anyway.

That is fine with me. It was an ACN related ticket on the Trello board, which is why I decided to attempt to quickly cover it before heading to Montreux.

Codecov Report

Merging #226 (2a8bca5) into main (04db06d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #226   +/-   ##
=======================================
  Coverage   94.55%   94.55%
=======================================
  Files         312      312
  Lines       25157    25157
=======================================
  Hits        23786    23786
  Misses       1371     1371

Flag
Coverage Δ

unittests
94.55% <ø> (ø)

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

Continue to review full report at Codecov.

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


the headers are different per site

Describe the bug
On the main page the headings are positioned left off the text, on other pages above the text.
Expected behavior
The style of heading positions is the same across the site.

thank you for the feedback! fixed it!

please send a pull request then :)

fixed in #67


goog.isString broken when using advanced compilation

Since commit 9389e14bb122d8c97674fbdcac49045bce8dcf3b compiling using the advanced setting breaks the usage of the goog.isString method. It is returning undefined instead of a boolean.
If I remove the usage of goog.pubsub.PubSub (the module which was updated in that commit) then the compilation works fine again.
I had a quick look through the commit changes but I can't see anything obvious that would be causing this.

Since this isn't generally true, you will need to provide exact reproduction steps.

If I had to take a shot in the dark, my guess is that your webpage is stomping on variables in the global scope, and that goog.isString coincidentally got renamed to one of the variable names you're stomping on. If you remove Pubsub, it would change the renaming slightly, and a different variable would get stomped on.

I will put some time towards trying to reproduce in an isolated environment. This should confirm whether the problem lies in the application code or not.

Apologies this does appear to be our fault. We had added google analytics and forgotten to add an external.
Thanks for your help.


fixes: Menu_Toggle, New PR due to synchronization issues in last one

Closes #5645
Due to some errors in my approach My old PRs were not getting synchronized Consider this one.

Update your pull request from your fork

@stevepiercy Hey can you guide me why are these test failing ?
okiee i will colse it


Solutions in save-points 00 and 01 do not build/

I've cloned the repo and opened the BlazingPizza.sln solutions in the 00-starting-point and 01-Components-and-layouts folder, but when trying to build them I get the following build error:
C:\Users\johnsteed\.nuget\packages\microsoft.aspnetcore.blazor.build\3.0.0-preview5-19227-01\targets\Blazor.MonoRuntime.targets(633,5): error MSB3073: The command "dotnet "C:\Users\johnsteed\.nuget\packages\microsoft.aspnetcore.blazor.build\3.0.0-preview5-19227-01\targets\../tools/Microsoft.AspNetCore.Blazor.Build.dll" write-boot-json "obj\Debug\netstandard2.0\BlazingPizza.Client.dll" --references "C:\Users\johnsteed\Source\Repos\blazor-workshop\save-points\01-Components-and-layout\BlazingPizza.Client\obj\Debug\netstandard2.0\blazor\bootjson-references.txt" --embedded-resources "C:\Users\johnsteed\Source\Repos\blazor-workshop\save-points\01-Components-and-layout\BlazingPizza.Client\obj\Debug\netstandard2.0\blazor\embedded.resources.txt" --linker-enabled --output "C:\Users\johnsteed\Source\Repos\blazor-workshop\save-points\01-Components-and-layout\BlazingPizza.Client\obj\Debug\netstandard2.0\blazor\blazor.boot.json"" exited with code -2147450730.
I'm  running .NET Core 3 Preview 4 and have the latest Blazor tools for Visual Studio installed. I can't find any information with regard to exit codes of the dotnet CLI so I have no idea what the problem could be. Any ideas?

Using detailed build output, I've discovered the following:
It was not possible to find any compatible framework version
3>    The specified framework 'Microsoft.NETCore.App', version '3.0.0-preview5-27626-15' was not found.

Turns out Preview 5 was released... a few hours after I downloaded the latest preview. It builds fine on preview 5!


Sync eng/common directory with azure-sdk-tools for PR 2011

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/2011  See eng/common workflow

/check-enforcer override


Bump vpp-agent version

Coverage decreased (-0.02%) to 75.971% when pulling fd5326e4a3558c5bbed9d7eba741a368850a8428 on lukasmacko:master into 810c9c994fb68842e2cfe3c966a030a6e9810a8d on contiv:master.


Remove empty space in Preview

Removed extra spacing found in the "Preview Table of Contents"

I will accept this! Good luck on your course.


Workshop Module for Secrets Encryption

Add a new module that covers enabling Secrets Encryption with KMS as detailed here: https://aws.amazon.com/about-aws/whats-new/2020/03/amazon-eks-adds-envelope-encryption-for-secrets-with-aws-kms/

Submitted a pull request. (#699 ). It will be reviewed as soon as eksctl 0.15.0 is GA.


Add stop_evaluating attribute to Model class and add associated modification to evaluate method

Issue type
Feature Request
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.16.1
Custom code
No
OS platform and distribution
No response
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
The Model class implements the stop_training attribute, but no stop_evaluating one.
In fit, there is a correct action taken to stop training when stop_training is set to True, in a callback for instance:
for epoch, iterator in data_handler.enumerate_epochs():
        self.reset_metrics()
        callbacks.on_epoch_begin(epoch)
        with data_handler.catch_stop_iteration():
          for step in data_handler.steps():
            with trace.Trace(
                'train',
                epoch_num=epoch,
                step_num=step,
                batch_size=batch_size,
                _r=1):
              callbacks.on_train_batch_begin(step)
              tmp_logs = self.train_function(iterator)
              if data_handler.should_sync:
                context.async_wait()
              logs = tmp_logs  # No error, now safe to assign to logs.
              end_step = step + data_handler.step_increment
              callbacks.on_train_batch_end(end_step, logs)
              if self.stop_training:
                break

and validation:
# Run validation.
        if validation_data and self._should_eval(epoch, validation_freq):
          # Create data_handler for evaluation and cache it.
          if getattr(self, '_eval_data_handler', None) is None:
            self._eval_data_handler = data_adapter.get_data_handler(
                x=val_x,
                y=val_y,
                sample_weight=val_sample_weight,
                batch_size=validation_batch_size or batch_size,
                steps_per_epoch=validation_steps,
                initial_epoch=0,
                epochs=1,
                max_queue_size=max_queue_size,
                workers=workers,
                use_multiprocessing=use_multiprocessing,
                model=self,
                steps_per_execution=self._steps_per_execution)
          val_logs = self.evaluate(
              x=val_x,
              y=val_y,
              sample_weight=val_sample_weight,
              batch_size=validation_batch_size or batch_size,
              steps=validation_steps,
              callbacks=callbacks,
              max_queue_size=max_queue_size,
              workers=workers,
              use_multiprocessing=use_multiprocessing,
              return_dict=True,
              _use_cached_eval_dataset=True)
          val_logs = {'val_' + name: val for name, val in val_logs.items()}
          epoch_logs.update(val_logs)

        callbacks.on_epoch_end(epoch, epoch_logs)
        training_logs = epoch_logs
        if self.stop_training:
          break

The evaluate method doesn't include this behaviour:
for _, iterator in data_handler.enumerate_epochs():  # Single epoch.
        self.reset_metrics()
        with data_handler.catch_stop_iteration():
          for step in data_handler.steps():
            with trace.Trace('test', step_num=step, _r=1):
              callbacks.on_test_batch_begin(step)
              tmp_logs = self.test_function(iterator)
              if data_handler.should_sync:
                context.async_wait()
              logs = tmp_logs  # No error, now safe to assign to logs.
              end_step = step + data_handler.step_increment
              callbacks.on_test_batch_end(end_step, logs)
      logs = tf_utils.sync_to_numpy_or_python_type(logs)
      callbacks.on_test_end(logs=logs)

The current way to stop evaluation in a callback is to force raise a StopIteration() error as it will trigger the catch_stop_iteration() method. But you can only do it after step 1, otherwise the logs will be empty. This is clearly not clean.
Also, the catch_stop_iteration() except text is only related to the training. A more general text would help to debug:
@contextlib.contextmanager
  def catch_stop_iteration(self):
    """Catches errors when an iterator runs out of data."""
    try:
      yield
      self.sync()
    except (StopIteration, errors.OutOfRangeError):
      if self._inferred_steps is None:
        self._inferred_steps = self._current_step
      else:
        self._insufficient_data = True
        total_epochs = self._epochs - self._initial_epoch
        logging.warning(
            "Your input ran out of data; interrupting training. "
            "Make sure that your dataset or generator can generate at "
            "least `steps_per_epoch * epochs` batches (in this case, "
            "{} batches). You may need to use the repeat() function "
            "when building your dataset.".format(total_epochs *
                                                 self._inferred_steps))

Standalone code to reproduce the issue
import tensorflow as tf

model = tf.keras.Sequential(
    [
        tf.keras.layers.Dense(2, activation="relu", name="layer1"),
    ]
)

print(model.stop_training)
print(model.stop_evaluating)

Relevant log output
No response

Sorry, I was in a wrong Conda env and using TF 2.15.1
It seems to have been implemented with keras 3 and TF 2.16.1


Input size mismatch during evaluation

Hello,
Thanks for sharing your code !
I was able to use cityscapes dataset and train (just doing 2 classes for start).
However, during evaluation - on any jpg (either a jpg of training data, or new image)
python eval.py --data_path ./eval --load_from_checkpoint ./checkpoints/unet_example/model-146 --batch_size 1
I always get the exact same error:
ValueError: Input 0 of layer conv1_1 is incompatible with the layer: expected ndim=4, found ndim=3. Full shape received: [None, 256, 256]
(full trace here - https://pastebin.com/Rjzt5N7V )
it seems as if nothing I do can avoid that error.
I'll keep at it - but if anything comes to mind, appreciate any tips or ideas meanwhile.
Thanks again !

aha - as suspected, this is a rather cryptic 'image not found' error... i get it even when there are no images in the folder :)
We know that the train/test format is :
If you check loader.py. I have specify the format. If you have multiple labels.
You can organize the img and gt in this way. The following examples has 3 labels.
    Use the Keras data generators to load train and test
    Image and label are in structure:
        train/
            img/
                0/
                1/
                2/
            gt/
                0/
                1/
                2/
        test/
            img/
                0/
                1/
                2/
            gt/
                0/
                1/
                2/

yet - what of evaluate ?

i'm giving up, i can't get evaluation to work. moving to other implementation.

Sorry for the confusion. I have updated the code to solve the issue. Now it has unique dataLoader and better support multiclass segmentation.

that is an impressive update ! I quick re-trained and evaluated - fact is, I fear you've made it harder for single-class inference (if you care) - I had gone down to one class when I couldn't get N>1 to work, but I had to go back to N=2 for training to work after this update.  Furthermore, I got errors in utils.py - meanIU_per_class was not iterable, it's a (numpy float?) - np.ndenumerate(meanIU_per_class) worked for line 62 ,but then the for loop won't work...
This may all be a result of my own data setup at this point - I thought I was doing the right thing by putting per-class masks/image/gt (0 = car, 1=people) but I may have just not done so correctly. So I am not giving a DIFF as i don't know what my problems are, vs how your code is supposed to work.
but thanks again for sharing the code - very clean model.py , and at this point I have a nice intro to online image transformation w/ keras :)

@nanodust  Hi I realized that you made a mistake on separating the class of masks. That is not necessary. The only case you need to separate image or mask to different folder, 0/ , 1/, is if your image has an image label and you want that information. Otherwise, put all image/mask in 0/.

interesting ! ok I'll give it another shot....


Download mpd drm video

Hi,
i want to know how to download videos from WAVO
for exemple i want to download this video https://www.wavo.com/video/episode-1-91
MPD File of the episode : https://neunlds1610lon.akamaized.net/nlds_vod/osn/vod/drm/nlncp/2019/06/21/0135e6f4-54d4-3de4-9189-fceb0ac87980/v1/stream/0135e6f4-54d4-3de4-9189-fceb0ac87980_1_pc.mp4.mpd
and it has many segments m4s like: https://neunlds1610lon.akamaized.net/nlds_vod/osn/vod/drm/nlncp/2019/06/21/0135e6f4-54d4-3de4-9189-fceb0ac87980/v1/stream/0135e6f4-54d4-3de4-9189-fceb0ac87980_1_4500_video/Time=1574000.m4s
first of all i don't know how to download m4s video + audio and they are encrypted with DRM
if any one need to login to the website just send PM to my email that it mentioned on my profile
Could any one help me please ?

This is a Kodi add-on that helps the Kodi InputStream Adaptive add-on by downloading the required CDM libraries. It has nothing to do with downloading streams.

ok thanks


Update to the latest Taginfo version

Currently, we are using an outdated version of Taginfo, which needs to be updated due to recent changes in the repository https://github.com/taginfo/taginfo/commits/master. There are also some other related issues and improvements that need to be addressed.

https://github.com/OpenHistoricalMap/issues/issues/404
https://github.com/developmentseed/osm-seed/issues/318
https://github.com/OpenHistoricalMap/issues/issues/672
https://github.com/OpenHistoricalMap/issues/issues/548
https://github.com/OpenHistoricalMap/issues/issues/492

Updates here: https://github.com/developmentseed/osm-seed/pull/321

Taginfo has been successfully updated at https://taginfo.openhistoricalmap.org/. In addition to resolving the issue, I have optimized the containers and configured them to run in separate processes.

Web: It is going to use fewer resources and will pull the database from S3 to start running.
Data Processor: This job will run on a dedicated job node with an autoscaler, using sufficient resources. It will then upload the data to S3. This setup ensures that it only uses large resources for a few minutes, avoiding the prolonged use of extensive resources as was done in the past.

cc. @batpad @danrademacher


CLM-6003 Add documentation for AddResourceToQueue mutation

@rjschill87 Regarding the resourceId argument, is it okay to provide the example of a possible resourceId the way I did, suggesting a courseID?

@AlexONeillTI Yeah, that's fine. Only change I'd suggest is courseID -> Course ID since courseID is more of an arg name


Production performance impacted after v2.17.0-beta releases

[x] VM RAM 95%+

Reduce instance count or add more RAM
VM upgrading IN PROGRESS completed

[x] Azure outbound data over usage by ~4x (62.5 GB / 15 GB)

Needs more investigation
Free quota 15 GB, priced after
Put 2 app on the same VM instead of different so the data transfer is internal

This can't be done because the client will interact with the API directly. The website source code calls the API (https://dl-back.raenonx.cc).

[x] Bump next-auth version and related dependencies

User.created_at and User.updated_at are removed
Account.created_at and Account.updated_at are removed
Account.account_id, Account.provider_type, and Account.accessTokenExpires are removed
Session.access_token are removed

Currently monitoring performance.


Fix support for JWT.encode() with alg: 'none'

Currently, we have entries like supportedAlgorithms.define('none', 'sign', {}) in the algorithms registry, so when you try JWT.encode() where header: { alg: 'none' }, it throws a 'that's not a function' error.

see #33

This is handled in anvilresearch/jwa#4. Closing the issue.


"Push notifications for COVID Alert" not showing at all on Android

Environment :
Pixel 3a, Android 10, apk v15-a
Scenario :

Install the app
Navigate the onboarding screens until "Choose your province" screen
Click on "Turn on" for the "Turn on exposure notifications" prompt

Expected :
A second prompt to allow "Push notifications for COVID Alert" should have been displayed
Issue :
The prompt for "Push notifications for COVID Alert" never showed up on Android (phone notifications being ON or OFF)

@dsamojlenko
Possibly related?
https://github.com/CovidShield/mobile/issues/114#issuecomment-642954453

I think this is getting set in the background without the need for a popup. Correct me if I'm wrong @henrytao-me.

https://github.com/cds-snc/covid-shield-mobile/issues/271

Android doesn't require permission to show push notification. That's why it's not relevant / shows up in Android.
I think we can close this issue.

@srepussard I'm closing this based on @henrytao-me comment on android functionality, if you have follow up please reach out.

Thank you for the investigation. I'll edit the test cases to adapt to this behavior.


Added module for Ni-Black thresholding

This is in response to the feature request: 4024 and is same as PR.

@StevenPuttemans : Working on that.

:+1:

@StevenPuttemans : Added function prototype. Could you review this?
Thanks!

I do not have review powers, but I can take a look at it tomorrow!

Sure! Thanks a lot :+1:

@samyak-268, thanks a lot! Can you please add some doxygen comments with the function and parameter description?

@vpisarev : Sure, will be doing that very soon.

@vpisarev I added some doxygen comments along with the code.

@vpisarev @StevenPuttemans : I was thinking of adding some sample/tutorial code for the Niblack module. Should I send a fresh PR for that or updating this one would be fine?

O simply include the sample here!

Updated PR with the following changes:

Changed Niblack threshold from (mean - (k*std_dev)) to (mean + (k*std_dev)) which is the correct implementation as per the original text.
Added sample code that demonstrates the use of Niblack thresholding. The value of k in the sample can be modified using track-bar.

Adding the sample made me realize that I was missing a CV_EXPORTS_W in the header!

@samyak-268, thanks! May I ask you to do 2 more things:

as far as I can see, using double for stddev image is not mandatory in this case; can you change it do float? (which is ~5x faster than double on ARM)
can you add some simple regression test?

@vpisarev : Sure. I am a little busy for the next few days. Will look into this as soon as I find time. Hope that's all right.

ok, I have yet another request. This seems to be just one relatively small function. Can you please put it to ximgproc (extended/advanced image processing) instead of creating a whole new module?

Coming back to this PR after quite a long time!
@vpisarev : I have changed the type of Mat stddev from double to float. Regarding your other suggestion, what sort of checks is a regression test supposed to perform?  It would be helpful if you could guide me to some existing regression tests within the repo. Is the modules/ximgproc/test/ directory the right place to look for the same?

@StevenPuttemans : Included everything within ximgproc. The code no longer resides in it's own module. Is this what you were looking for?
Thanks!

Seems good to me, but you will need acceptance of @vpisarev

Hey! Just a gentle reminder. It's been quite a while, anything that needs to be done from my end in regards to this PR? Thanks!

@samyak-268 I am afraid not. Seeing that I know of PR's waiting since June ... I want to ask you to wait at least a little bit more :)

I get some strange results using your example:

It seems something goes wrong.

@mshabunin : A little busy for a few days. I'll look into your comments soon. Thanks!

I have written some statistical threshold functions some time ago from this paper http://www.math-info.univ-paris5.fr/~vincent/articles/DRR_nick_binarization_09.pdf .
I have never really used this, but the first impression looked good. Perhabs you can use this for your implementation.
void statisticalThreshold(InputArray inputImage, OutputArray outputImage, int maxValue, int blockSize, double k, int algorithmType) {
    CV_Assert(!inputImage.empty() && (inputImage.kind() == _InputArray::MAT || inputImage.kind() == _InputArray::UMAT));
    CV_Assert(outputImage.kind() == _OutputArray::MAT || outputImage.kind() == _OutputArray::UMAT);

    UMat _inputImage = inputImage.getUMat();
    outputImage.create(inputImage.size(), CV_8U);
    UMat _outputImage = outputImage.getUMat();
    int nCompareType = CMP_GT;
    //int nNumberOfPixels = _inputImage.total();

    UMat normImage, meanImage, meanImageSquare, tempImage1, tempImage2, varianceImage, stdDevImage;
    normalize(_inputImage, normImage, 0, 255, NORM_MINMAX, CV_32F);
    boxFilter(normImage, meanImage, CV_32F, Size(blockSize, blockSize), Point(-1, -1), true, BORDER_REFLECT101);
    sqrBoxFilter(normImage, tempImage1, CV_32F, Size(blockSize, blockSize), Point(-1, -1), true, BORDER_REFLECT101);
    multiply(meanImage, meanImage, meanImageSquare);
    subtract(tempImage1, meanImageSquare, varianceImage);

    // Mean and squared mean image with integral image
    //int windowSizeHalf = blockSize / 2;
    //copyMakeBorder(_inputImage, borderImage, windowSizeHalf, windowSizeHalf, windowSizeHalf, windowSizeHalf, BORDER_REFLECT101);
    //integral(borderImage, integralImage, integralSquareImage, CV_32F, CV_32F);
    //UMat addImage1, addImage2, stdDevImage;
    //add(integralImage(Rect(0, 0, _inputImage.cols, _inputImage.rows)), integralImage(Rect(blockSize-1, blockSize-1, _inputImage.cols, _inputImage.rows)), addImage1);
    //add(integralImage(Rect(blockSize-1, 0, _inputImage.cols, _inputImage.rows)), integralImage(Rect(0, blockSize-1, _inputImage.cols, _inputImage.rows)), addImage2);
    //addWeighted(addImage1, 1. / (blockSize*blockSize), addImage2, -1. / (blockSize*blockSize), 0, meanImage, CV_32F);
    //multiply(meanImage, meanImage, meanSquareImage, 1.0, CV_32F);
    //add(integralSquareImage(Rect(0, 0, _inputImage.cols, _inputImage.rows)), integralSquareImage(Rect(blockSize, blockSize, _inputImage.cols, _inputImage.rows)), addImage1);
    //add(integralSquareImage(Rect(blockSize, 0, _inputImage.cols, _inputImage.rows)), integralSquareImage(Rect(0, blockSize, _inputImage.cols, _inputImage.rows)), addImage2);
    //addWeighted(addImage1, 1. / (blockSize*blockSize), addImage2, -1. / (blockSize*blockSize), 0, squareMeanImage, CV_32F);
    //subtract(tempImage1, tempImage2, subImage, noArray(), CV_32F);
    //cv::sqrt(subImage, stdDevImage);

    switch (algorithmType) {
    case CV_THRESH_NIBLACK:
        nCompareType = CMP_LE;
    case CV_THRESH_NIBLACK_INV:
        sqrt(varianceImage, stdDevImage);
        scaleAdd(stdDevImage, k, meanImage, tempImage1);
        break;
    case CV_THRESH_SAUVOLA:
        nCompareType = CMP_LE;
    case CV_THRESH_SAUVOLA_INV:
        sqrt(varianceImage, stdDevImage);
        stdDevImage.convertTo(tempImage1, CV_32F, (1. / 128), -1.);
        tempImage1.convertTo(tempImage2, CV_32F, k, 1.);
        multiply(meanImage, tempImage2, tempImage1, 1.);
        break;
    case CV_THRESH_WOLF:
        nCompareType = CMP_LE;
    case CV_THRESH_WOLF_INV:
        sqrt(varianceImage, stdDevImage);
        multiply(stdDevImage, meanImage, tempImage2, (k / 128.));
        scaleAdd(meanImage, (1. - k), tempImage2, tempImage1);
        break;
    case CV_THRESH_NICK:
        nCompareType = CMP_LE;
    case CV_THRESH_NICK_INV:
        add(meanImageSquare, varianceImage, tempImage1);
        //scaleAdd(meanImageSquare, nNumberOfPixels / (nNumberOfPixels - 1.), varianceImage, tempImage2);
        sqrt(tempImage1, tempImage2);
        scaleAdd(tempImage2, k, meanImage, tempImage1);
        break;
    default:
        CV_Error(Error::StsBadArg, "Threshold type unknown.");
    }
    compare(normImage, tempImage1, meanImage, nCompareType);
    meanImage.convertTo(_outputImage, CV_8U, maxValue, 0);

    return;
}

thanks! let's put it in :+1:


feat: adds es language

Pages on which the translation should be:

[x] header
[x] footer
[x] SEO
[x] home
[x] what-is-ebpf
[x] get-started
[x] case-studies
[x] labs
[x] events
[x] applications
[x] infrastructure
[x] contribute

@xmulligan Hello! I made corrections, please check.


Error: Read timed out

The Pledges to my project stopped showing up. Did I do something wrong?
Thanx in advance guys :)
-ken

Which server are you using?

I think I used that secondary server with Joe, since you were gone that weekend I could not get the project file onto the vinumeris server. the domain started with the letter T i think, can't remember the exact name.

OK, then we need to ask @JoeSchmid about it.

Let me know if you need anything from me then, I'm here. My skype too is: kenCode_de

I'm kinda new to github so please ask him if ya could for me, thanx Mike :)

Steps to reproduce: Open Lighthouse on linux mint, sync with bitcoin network works fine, other projects I have imported load just fine and their pledges are viewable. click on my project though, and that error message above appears and the pledges are not viewable.

Hi Ken, it should be fixed now. I needed to restart the Lighthouse server. It's been quite stable, but in this case it seemed to stop responding to HTTPS requests. (It seemed to be responding to bitcoin network messages fine though.)

Yep, everything works perfectly again. Did I mention you guys are gods? Thanx again Joe and Mike :) I'll send it to you in a few days, but fyi: https://twitter.com/kenCode_de/status/565186437281099776


Consider caching AuditActor.GetLocalIpAddressAsync

Scenario: tested push locally with our shim auditing enabled (both ifX and fileStorage).
Profile shows 15.5% inclusive in SaveAuditRecordAsync (36.1% of ApiController::CreatePackageInternal stack). The cause is the AuditActor's call to NetworkInterface.GetIsNetworkAvailable which accounts for 14.6% of the 15.5%.

Caching AuditActor.GetLocalIpAddressAsync seems to reduce api push server latency by 18.5%.

Run
Baseline (avg)
Result (avg)
%Diff

1
336.61ms
274.04ms
-18.5%

2
309.71ms
252.58ms
-18.5%

Note that this optimization will apply to all auditing, not just api push.

Didn't get a baseline DEV0 run before optimization was deployed, but captured an INT0 baseline.

Scenario
Baseline Env
Baseline Result
Env
Result
% Diff

ApiPush
n/a
n/a
DEV0_1220
12669.2ms
n/a

ApiPush
INT0_1220
4249.3ms
INT0_X
n/a
n/a


[#28187][prism] worker shutdown, cleanup, log fail, port spec, grpc recv size

Some fixes while trying to use the experimental Swift SDK against prism.

Use an actual loopback address for endpoints instead of the net package listener default of unspecified.

IPv6 uses "[::]" to mean an unspecified address, which Go then treats as the loopback address. This isn't true across all SDKs, so we must explicitly indicate the loopback address.
Note: We create the server to listen to any traffic to that port however, so other connections, possibly off the machine, remain valid.

Increase default receive buffer to max.
Fail jobs if the SDK worker doesn't connect after 1 minute.
Error job failures on log on slog side.
Minor log clean up, to a debug log.

Related to #28187

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

Thanks for the review!


How to change user-agent?

Hi,
Sorry I am new to scraping and is trying to use your code.
In the use of this code you mentioned:
"Change the default python requests user-agent for a browser user-agent "
I don't quite know how to do that. Can anyone demonstrate/illustrate?
Thanks!

see https://github.com/ericfourrier/scrape-linkedin/issues/8


Resizing terminal will not resize the terminal chat widget

Repro:

Open terminal chat
Ask "list files in various ways"
Reduce height of terminal

@jrieken , we'd need a way to set max height for the chat widget, which there isn't currently AFAICT

@meganrogge You need to call layout with the sizes you want

@jrieken i tried that and didn't see the height taking effect, only the width.


Fix typos: connot->cannot concurrrent->concurrent

Signed-off-by: mooncake<EMAIL_ADDRESS>
What this PR does / why we need it:
Fix typos: connot->cannot concurrrent->concurrent
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

NONE

/assign @dims

/ok-to-test

/approve

/lgtm

@adisky  can you also add "/approve" label? thanks

/retest

/assign @dims

/override openlab/cloud-provider-openstack-acceptance-test-flexvolume-cinder

/lgtm
/approve